Blazor WebAssembly with Hosted AAD B2C problem

Hi,
I followed the instructions. When I test, I was able to login. However, I got conflicting results on the redirect:

LoginDisplay's @context.User.Identity.Name! I was able to get the correct value
@body I got the "There was an error trying to log you in: 'AADB2C90205: This application does not have sufficient permissions against this web resource to perform the operation."

If I click on fetchData, blazor crashes.
Anyone followed the instructions and got it working? There are many placeholders that we need to populate. I wonder if I did something wrong. Any feedback would be helpful. Thanks.

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

ID: 7d682d4c-86b5-c47e-89a3-3684b2235320
Version Independent ID: a80e115c-466a-288d-a918-9ef7514f5add
Content: Secure an ASP.NET Core Blazor WebAssembly hosted app with Azure Active Directory B2C
Content Source: aspnetcore/security/blazor/webassembly/hosted-with-azure-active-directory-b2c.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @Alienroid ... The docs in this node are brand new ... we only put them up last week when 3.2-Preview 2 came out, so it's not surprising that a few rough spots are going to be hit with them.
Things might be a little rough in preview. We had some trouble with scopes in the AAD version of the topic that I'll be investigating further this week. Here with AAD B2C, I just discovered something flaky: This app worked just fine here last week. This morning, I fired it up, and it broke (even after clearing site data in developer tools). I noticed a console error about pop-ups being blocked. I disabled the pop-up blocker in Firefox ... cleared cookies, logged the user in again, and now the FetchData page works! It's difficult to document something with unexpected and strange behavior like that.
Try a couple of things ...

If you're using VS, make sure that you've selected the Hosted project (xxxxx.Server) when you select the VS Run button. Try it with each of the IIS Express profile and the app's profile (named after the app) in the Run button drop down. I usually use the app's profile and stick with the 5001 port for all of my testing instead of the randomly generated port for IIS Express (although that really should be ok if you want to test it that way).
Disable your browser's popup blocker like I just did. See if that helps.
Make sure that each time you fire the app back up (or make any changes on the Azure Portal side) that you log the user out and clear the cookies in the developer tools console. A lingering user cookie there can easily hose a changed config and give a false negative when you test a new setup.
Confirm a given behavior in two browsers when something breaks. Again, make sure cookies are cleared before you try and log the user in ... every single test.

Thanks Luke for getting back to me and with such detailed write up. I totally understand that this is only preview, I know this all to well. I'll try it out few more times and report findings here. Otherwise I'm just going to wait for future builds. Thanks again!
Get Outlook for iOShttps://aka.ms/o0ukef

From: Luke Latham<EMAIL_ADDRESS>Sent: Sunday, March 15, 2020 6:20:51 AM
To: dotnet/AspNetCore.Docs<EMAIL_ADDRESS>Cc: Alienroid<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [dotnet/AspNetCore.Docs] Anyone got this working? (#17319)
Hello @Alienroidhttps://github.com/Alienroid ... The docs in this node are brand new ... we only put them up last week when 3.2-Preview 2 came out, so it's not surprising that a few rough spots are going to be hit with them.
Things might be a little rough in preview. We had some trouble with scopes in the AAD version of the topic that I'll be investigating further this week. Here with AAD B2C, I just discovered something flaky: This app worked just fine here last week. This morning, I fired it up, and it broke (even after clearing site data in developer tools). I noticed a console error about pop-ups being blocked. I disabled the pop-up blocker in Firefox ... cleared cookies, logged the user in again, and now the FetchData page works! It's difficult to document something with unexpected and strange behavior like that.
Try a couple of things ...

If you're using VS, make sure that you've selected the Hosted project (xxxxx.Server) when you select the VS Run button. Try it with each of the IIS Express profile and the app's profile (named after the app) in the Run button drop down. I usually use the app's profile and stick with the 5001 port for all of my testing instead of the randomly generated port for IIS Express (although that really should be ok if you want to test it that way).
Disable your browser's popup blocker like I just did. See if that helps.
Make sure that each time you fire the app back up (or make any changes on the Azure Portal side) that you log the user out and clear the cookies in the developer tools console. A lingering user cookie there can easily hose a changed config and give a false negative when you test a new setup.
Confirm a given behavior in two browsers when something breaks. Again, make sure cookies are cleared before you try and log the user in ... every single test.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/AspNetCore.Docs/issues/17319#issuecomment-599209648, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH3FBMWK7GPIWHYEYXD3NTTRHTI3HANCNFSM4LKKHMMQ.

Update ... rather frustrating, but re-enabling the popup blocker in Firefox and testing again ... the app works as it did the other day. At this point, we can only draw one logical conclusion as to why ... Yes! ...
👽
... they are among us. :smile:
Seriously tho, I can't explain why it flaked out this morning.
I don't have anything actionable yet on your scenario/setup. I would like to investigate a related issue for the topic. You're making available the user data for the user, so the name populates, but the topic doesn't. I think I'd like to do that here.  Also, I want to take a look at the instruction for running the app and make sure that the topic guides devs to run the app with the Server project selected in Solution Explorer.

Hi Luke,
Should I follow up here or in the newer thread you’ve created?
I just did everything again. With firefox, I could log in and log out properly. Clicking on FetchData will crash due to … I haven’t looked into that yet, but it’s on the client side (server controller never got called).
However, if I try with chrome, right after login, I’ll get the error:
There was an error trying to log you in: 'AADB2C90205: This application does not have sufficient permissions against this web resource to perform the operation. Correlation ID: 2e123fba-35d4-4203-9449-6126d1c1ca86 Timestamp: 2020-03-17 00:10:41Z ' Login failed, and top right still shows ‘login’
If I refresh the page (f5), top right now shows my display name from @context.User.Identity.Name!
Alien!
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10
From: Luke<EMAIL_ADDRESS>Sent: Monday, March 16, 2020 10:13 AM
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [dotnet/AspNetCore.Docs] Blazor WebAssembly with Hosted AAD B2C problem (#17319)
Update ... rather frustrating, but re-enabling the popup blocker in Firefox and testing again ... the app works as it did the other day. At this point, we can only draw one logical conclusion as to why ... Yes! ...
👽
... they are among us. 😄
Seriously tho, I can't explain why it flaked out this morning.
I don't have anything actionable yet on your scenario/setup. I would like to investigate a related issue for the topic. You're making available the user data for the user, so the name populates, but the topic doesn't. I think I'd like to do that here.  Also, I want to take a look at the instruction for running the app and make sure that the topic guides devs to run the app with the Server project selected in Solution Explorer.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/AspNetCore.Docs/issues/17319#issuecomment-599658198, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH3FBMVXITAUO7CC5WG467TRHZM3NANCNFSM4LKKHMMQ.

Let's discuss Hosted with AAD B2C here. I'll use the other issue to work through the Azure Portal and scope address aspects of the Hosted with AAD. If you saw that issue, it looks like we'll be chatting with an Azure 🐈 for help with that scenario.

Clicking on FetchData will crash due to …

When you get to that, I had good luck 🍀 with Postman. I had the FetchData component write out the exact token it was going to use to the console ...
if (tokenResult.TryGetToken(out var token))
{

    Console.WriteLine($"Bearer {token.Value}");

    httpClient.DefaultRequestHeaders.Add("Authorization", $"Bearer {token.Value}");
    forecasts = await httpClient.GetJsonAsync<WeatherForecast[]>("WeatherForecast");
}
else
{
    Navigation.NavigateTo(tokenResult.RedirectUrl);
}

Over in Postman, I set up the request to the https://localhost:5001/WeatherForecast/ with the Authorization header.
The server sends back a reason for the API call failure. I think that would have been piped thru to logging if I had turned up the output, but finding a line in a sea of lines 😵  is a PITA. It was easier to just fire up Postman and make a direct request to the backend (Server app) and see what comes back.

if I try with chrome, right after login

... and you're completely zapping the cookies each test, right? ... and you're starting the app from the Server project? ... and your config matches the structure of what I posted :point_up:?
I've been working mostly in Firefox and Edge. My Chrome ate itself the other day, so I uninstalled but didn't get around to reinstalling yet. I'll check Edge in the morning to see how that's going. I'll get Chrome back on here, and see how that goes. I'll report back what I find tomorrow ........... IF I don't do a 1929 nose dive out the window after looking at the DOW futures in the morning. Fortunately, I live on the 1st floor, so I can't really hurt myself too badly!

I see discussions of that error, too, out there. For example wrt an Azure sample ...
https://github.com/Azure-Samples/active-directory-b2c-javascript-msal-singlepageapp/issues/4

I've gone through that thread before posted here. That thread seems old and not related. That said, I first gone through non-wasm case, blazor server side and seems to be working fine.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Luke Latham<EMAIL_ADDRESS>Sent: Monday, March 16, 2020 6:09:13 PM
To: dotnet/AspNetCore.Docs<EMAIL_ADDRESS>Cc: Alienroid<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [dotnet/AspNetCore.Docs] Blazor WebAssembly with Hosted AAD B2C problem (#17319)
I see discussions of that error, too, out there. For example wrt an Azure sample ...
Azure-Samples/active-directory-b2c-javascript-msal-singlepageapp#4https://github.com/Azure-Samples/active-directory-b2c-javascript-msal-singlepageapp/issues/4
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/AspNetCore.Docs/issues/17319#issuecomment-599826956, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH3FBMQES22HPZ4QLFASVYLRH3ETTANCNFSM4LKKHMMQ.

I wish there's private chat here. We can just chat about stock market or have some laughs. I got LinkedIn or fb if you want to connect jasonertw on Facebook. Futures is up tomorrow but I wouldn't buy anything now (except maybe uso)
I'll try postman later when I have time.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Jason Chen<EMAIL_ADDRESS>Sent: Monday, March 16, 2020 6:40:03 PM
To: dotnet/AspNetCore.Docs<EMAIL_ADDRESS>dotnet/AspNetCore.Docs<EMAIL_ADDRESS>Cc: Mention<EMAIL_ADDRESS>Subject: Re: [dotnet/AspNetCore.Docs] Blazor WebAssembly with Hosted AAD B2C problem (#17319)
I've gone through that thread before posted here. That thread seems old and not related. That said, I first gone through non-wasm case, blazor server side and seems to be working fine.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Luke Latham<EMAIL_ADDRESS>Sent: Monday, March 16, 2020 6:09:13 PM
To: dotnet/AspNetCore.Docs<EMAIL_ADDRESS>Cc: Alienroid<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [dotnet/AspNetCore.Docs] Blazor WebAssembly with Hosted AAD B2C problem (#17319)
I see discussions of that error, too, out there. For example wrt an Azure sample ...
Azure-Samples/active-directory-b2c-javascript-msal-singlepageapp#4https://github.com/Azure-Samples/active-directory-b2c-javascript-msal-singlepageapp/issues/4
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/AspNetCore.Docs/issues/17319#issuecomment-599826956, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH3FBMQES22HPZ4QLFASVYLRH3ETTANCNFSM4LKKHMMQ.

Just to update where we stand here ...
https://github.com/dotnet/AspNetCore.Docs/pull/17333 – Makes sure that the Display Name comes back for B2C scenarios. It also calls out that the project should be run from the Server app in all of the topics in this node.
@Alienroid ... Did you figure out your issue with your app? I don't have anything else actionable here at this time for the AAD B2C scenario. If you're still having trouble, you might need to either consult with the community and/or the product unit until it's worked out. If there's a gotcha that the topic needs to call out, I can re-open this issue and work it.

No update from my side. I’ll revisit in the future.
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10
From: Luke<EMAIL_ADDRESS>Sent: Sunday, March 22, 2020 8:53 AM
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [dotnet/AspNetCore.Docs] Blazor WebAssembly with Hosted AAD B2C problem (#17319)
Just to update where we stand here ...
#17333https://github.com/dotnet/AspNetCore.Docs/pull/17333 – Makes sure that the Display Name comes back for B2C scenarios. It also calls out that the project should be run from the Server app in all of the topics in this node.
@Alienroidhttps://github.com/Alienroid ... Did you figure out your issue with your app? I don't have anything else actionable here at this time for the AAD B2C scenario. If you're still having trouble, you might need to either consult with the community and/or the product unit until it's worked out. If there's a gotcha that the topic needs to call out, I can re-open this issue and work it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/AspNetCore.Docs/issues/17319#issuecomment-602230048, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH3FBMWTDAMAJYGOC6POGYLRIYYARANCNFSM4LKKHMMQ.

Ok ... let's close for now.

Hi Luke,
Just to update (since I got some feedback from other forums just now). Someone mentioned they have to:
“configure the RoleClaim at AddMsalAuthentication to match whatever B2C returns as a role claim”
I haven’t tried that.
However, I looked at the code more closely. It seems when I click on FetchData, after tokenResult.TryGetToken(out var token), the token is null, which caused my fetchData page to crash. Then I looked at LoginDisplay to see how it was able to get my name. LoginDisplay gets the name parsing from @context.User.Identity.Name! Where ‘context’ is AuthenticationState.
I got to figure out why token is null. I am also interested in how context is populated… it’s like magic there.
Regarding why token is null, I still think my config is wrong. But if that’s the case, why would context be correct?
One other thing I observed, from the person who suggested to config RoleClaim, his AddMsalAuthentication looks like:
options.ProviderOptions.DefaultAccessTokenScopes.Add("https://{domain}.onmicrosoft.com/API.Access");
but the template generated for me looks like:
options.ProviderOptions.DefaultAccessTokenScopes.Add("https://{domain}.onmicrosoft.com/https:// {domain}.onmicrosoft.com/111111-1111-1111-1111-111111111/API.Access");
Did you happen to jump out your first floor window? I think we still got 15-20% to go before we can have real recovery.
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10
From: Luke<EMAIL_ADDRESS>Sent: Sunday, March 22, 2020 9:21 AM
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [dotnet/AspNetCore.Docs] Blazor WebAssembly with Hosted AAD B2C problem (#17319)
Ok ... let's close for now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/AspNetCore.Docs/issues/17319#issuecomment-602234119, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH3FBMSFBAV6NYEVZ34QWZLRIY3G5ANCNFSM4LKKHMMQ.

The scope is important for the API access. The client can log the user in; but if the scope is wrong, the FetchPage component won't get an access token, as you're witnessing.

options.ProviderOptions.DefaultAccessTokenScopes.Add("https://{domain}.onmicrosoft.com/https:// {domain}.onmicrosoft.com/111111-1111-1111-1111-111111111/API.Access");

That looks like it may have happened from not having the right values passed into the dotnet new command when the app was created. It should look like this ...
options.ProviderOptions.DefaultAccessTokenScopes
    .Add("https://{TENANT DOMAIN}/{SERVER API CLIENT ID}/{SCOPE}");

{TENANT DOMAIN} is something like XXXXXXX.onmicrosoft.com ... same entry as the server app's Domain entry in the appsettings.json file.
{SERVER API CLIENT ID} is something like<PHONE_NUMBER>-1111-1111-111111111 ... make sure it's the server app client ID ... the same one in the server app's ClientId entry in the appsettings.json file.
{SCOPE} (going by the example in the topic) is API.Access.

15-20% to go before we can have real recovery.

... I hope it's only that much. 🙈 The DOW futures just tanked (again). :cry:

Might be to much to ask, but is there some way I can share my dotnet new command with you?
 I also sold my bonds
Get Outlook for iOShttps://aka.ms/o0ukef

From: Luke Latham<EMAIL_ADDRESS>Sent: Sunday, March 22, 2020 4:28:22 PM
To: dotnet/AspNetCore.Docs<EMAIL_ADDRESS>Cc: Alienroid<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [dotnet/AspNetCore.Docs] Blazor WebAssembly with Hosted AAD B2C problem (#17319)
The scope is important for the API access. The client can log the user in; but if the scope is wrong, the FetchPage component won't get an access token, as you're witnessing.
options.ProviderOptions.DefaultAccessTokenScopes.Add("https://{domain}.onmicrosoft.com/https:// {domain}.onmicrosoft.com/111111-1111-1111-1111-111111111/API.Access");
That looks like it may have happened from not having the right values passed into the dotnet new command when the app was created. It should look like this ...
options.ProviderOptions.DefaultAccessTokenScopes
.Add("https://{TENANT DOMAIN}/{SERVER API CLIENT ID}/{SCOPE}");

{TENANT DOMAIN} is something like XXXXXXX.onmicrosoft.com ... same entry as the server app's Domain entry in the appsettings.json file.
{SERVER API CLIENT ID} is something like<PHONE_NUMBER>-1111-1111-111111111 ... make sure it's the server app client ID ... the same one in the server app's ClientId entry in the appsettings.json file.
{SCOPE} (going by the example in the topic) is API.Access.

15-20% to go before we can have real recovery.
... I hope it's only that much. 🙈 The DOW futures just tanked (again). 😢
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/AspNetCore.Docs/issues/17319#issuecomment-602294127, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH3FBMSRSPIMHVJCMUEREJDRI2NJNANCNFSM4LKKHMMQ.

I don't have a secure way to receive it. I have to follow my company's (and MS's) security policies when it comes to something like that.
It's the one in the topic ...
https://docs.microsoft.com/en-us/aspnet/core/security/blazor/webassembly/hosted-with-azure-active-directory-b2c?view=aspnetcore-3.1#create-the-app
... which results in the config that I show at the bottom of my comment :point_right: https://github.com/dotnet/AspNetCore.Docs/issues/17319#issuecomment-599209648
It just sounds like it's only your scope that's off. I think if you work on the scope further, you'll probably be able to get it running. Just be sure to keep clearing those browser cookies between changes.

Hi Luke,
I got it to work. So here’s the weird part. I think I did the dotnet new properly. Appsettings.json is generated correctly. However, program.cs DefaultAccessTokenScopes is generated incorrectly.
Generated:
options.ProviderOptions.DefaultAccessTokenScopes.Add("https:// XXXXXX.onmicrosoft.com/https:// XXXXXX.onmicrosoft.com/11111111-1111-1111-1111-111111111111/API.Access");
Working solution (same as you mentioned):
options.ProviderOptions.DefaultAccessTokenScopes.Add("https://XXXXXX.onmicrosoft.com/11111111-1111-1111-1111-111111111111/API.Access");
It seems dotnet new used value from --app-id-uri instead of the value from --api-client-id
On a separate note, the doc had a minor error. It used "Instance": "https://login.microsoftonline.com/",
It should be “Instance”: “https://XXXXXX.b2clogin.com/”
Thanks again! Now I can think about using Azure AADB2C in the future. 😉
BTW, I think I found you on LinkedIn.
Jason
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10
From: Luke<EMAIL_ADDRESS>Sent: Sunday, March 22, 2020 5:11 PM
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [dotnet/AspNetCore.Docs] Blazor WebAssembly with Hosted AAD B2C problem (#17319)
I don't have a secure way to receive it. I have to follow my company's (and MS's) security policies when it comes to something like that.
It's the one in the topic ...
https://docs.microsoft.com/en-us/aspnet/core/security/blazor/webassembly/hosted-with-azure-active-directory-b2c?view=aspnetcore-3.1#create-the-app
... which results in the config that I show at the bottom of my comment 👉 #17319 (comment)https://github.com/dotnet/AspNetCore.Docs/issues/17319#issuecomment-599209648
It just sounds like it's only your scope that's off. I think if you work on the scope further, you'll probably be able to get it running. Just be sure to keep clearing those browser cookies between changes.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/AspNetCore.Docs/issues/17319#issuecomment-602303309, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH3FBMWKHDSP7YBI6AF5GXLRI2SLRANCNFSM4LKKHMMQ.

It seems dotnet new used value from --app-id-uri instead of the value from --api-client-id

That's right, but I think the Azure Portal defaults the value to the server app's client id. Since the same GUID is used in the default case, I used the same GUID (the one with 1's in it) in the topic's instructions and mention the it "might default the value" phrase ...

Server API app Application ID (Client ID) (for example, 11111111-1111-1111-1111-111111111111)
Server API app App ID URI (for example, https://contoso.onmicrosoft.com/11111111-1111-1111-1111-111111111111, the Azure portal might default the value to the Client ID)

On a separate note, the doc had a minor error. It used "Instance": "https://login.microsoftonline.com/",
It should be “Instance”: “https://XXXXXX.b2clogin.com/”

Yes! ... it's probably another one of my classic cut-'n-paste errors. 🙈 The file contents shown came from the Hosted with AAD topic. I'll fix that right now. Thanks for letting me know.

Hi Luke,
I am not sure where to ask this. So Blazor WebAssembly with Hosted AAD B2C authentication is working. However, there’s no documentation or sample code on how to add an userflow.
The examples from blazor server side only configures in startup.cs and it gets configuration from AppSettings.json. That’s the same as Blazor WebAssembly with Hosted (on the server side). However, the client side, we only have program.cs that configures ‘authentication’ so we can use authentication/login in LoginDisplay. However, no idea how we can do that for other userflow. I am not sure if AAD B2C for Blazor WebAssembly is completed or just lack of documentation? Is there some way we can subscribe to the status of the project? Thanks.
Regards,
Jason
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10
From: Luke<EMAIL_ADDRESS>Sent: Sunday, March 22, 2020 9:19 PM
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [dotnet/AspNetCore.Docs] Blazor WebAssembly with Hosted AAD B2C problem (#17319)
Closed #17319https://github.com/dotnet/AspNetCore.Docs/issues/17319 via #17408https://github.com/dotnet/AspNetCore.Docs/pull/17408.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/AspNetCore.Docs/issues/17319#event-3153562185, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH3FBMV3UBMLSG4DBQ4WR7DRI3PMNANCNFSM4LKKHMMQ.

So I have a similar issue that I can't seem to track down. In our B2C client's AD, we have two types of users. First there are those who are sourced via the Federated Azure Active Directory (which is our parent Azure AD that uses company credentials to validate any user). Then we have those who are sourced via the default Azure Active Active Directory on the B2C subscription.
For some reason, those who are users by virtue of the Federated Azure Active Directory cause a hard crash. It is happening at this line:
var tokenResult = await AuthenticationService.RequestAccessToken();
I am figuring that this has something to do with the scopes but I cannot seem to pinpoint what exactly and why that would only cause the issue for the non-federated AD users.

@adamasmar ... open that for engineering at https://github.com/dotnet/aspnetcore/issues.
Please add a cc: @guardrex to the bottom of your issue there so that I can track the discussion. I'm working on these new topics almost daily. If it turns out to be something good that we can surface in the doc, I'll open a new issue for it.

Hi Luke, I got it to work. So here’s the weird part. I think I did the dotnet new properly. Appsettings.json is generated correctly. However, program.cs DefaultAccessTokenScopes is generated incorrectly. Generated: options.ProviderOptions.DefaultAccessTokenScopes.Add("https:// XXXXXX.onmicrosoft.com/https:// XXXXXX.onmicrosoft.com/11111111-1111-1111-1111-111111111111/API.Access"); Working solution (same as you mentioned): options.ProviderOptions.DefaultAccessTokenScopes.Add("https://XXXXXX.onmicrosoft.com/11111111-1111-1111-1111-111111111111/API.Access"); It seems dotnet new used value from --app-id-uri instead of the value from --api-client-id On a separate note, the doc had a minor error. It used "Instance": "https://login.microsoftonline.com/", It should be “Instance”: “https://XXXXXX.b2clogin.com/” Thanks again! Now I can think about using Azure AADB2C in the future. 😉 BTW, I think I found you on LinkedIn. Jason Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10 From: Luke<EMAIL_ADDRESS>Sent: Sunday, March 22, 2020 5:11 PM To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [dotnet/AspNetCore.Docs] Blazor WebAssembly with Hosted AAD B2C problem (#17319) I don't have a secure way to receive it. I have to follow my company's (and MS's) security policies when it comes to something like that. It's the one in the topic ... https://docs.microsoft.com/en-us/aspnet/core/security/blazor/webassembly/hosted-with-azure-active-directory-b2c?view=aspnetcore-3.1#create-the-app ... which results in the config that I show at the bottom of my comment 👉 #17319 (comment)<#17319 (comment)> It just sounds like it's only your scope that's off. I think if you work on the scope further, you'll probably be able to get it running. Just be sure to keep clearing those browser cookies between changes. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#17319 (comment)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH3FBMWKHDSP7YBI6AF5GXLRI2SLRANCNFSM4LKKHMMQ.

I was getting the exact issue with error AADB2C90205 on 3.2.0-rc1.20223.4 and switched...
https://xxxxxx.onmicrosoft.com/https://xxxxxx.onmicrosoft.com/11111111-1111-1111-1111-111111111111/API.Access
TO
https://xxxxxx.onmicrosoft.com/11111111-1111-1111-1111-111111111111/API.Access
and it worked after the change.  Looks like it's still generating incorrectly in Program.cs?

This pattern ...

https://xxxxxx.onmicrosoft.com/https://xxxxxx.onmicrosoft.com/11111111-1111-1111-1111-111111111111/API.Access

... means that the options values weren't correct in the dotnet new command that generated the app.
I'm probably going to set up a table with example values in the topics soon that will make it easier to match what devs need to take from the Azure portal registration to the options for the command. That should happen fairly fast ... within a week or two.
The interesting thing about ...

https://xxxxxx.onmicrosoft.com/11111111-1111-1111-1111-111111111111/API.Access

is that it was failing earlier. They might have made some changes on the Azure side that make that scope work now. When I make my pass on these topics soon, I'll try that again and see if I get the same good result, then I'll update the topic for it.

I wrote a blog documenting this as a note: https://iheartms.azurewebsites.net/2020/03/23/troubleshoot-asp-net-core-blazor-webassembly-hosted-app-with-azure-active-directory-b2c/
But yes, I tried it on RC. If the command have --default-scope "demo.read", it generated
https://xxxxxx.onmicrosoft.com/https://xxxxxx.onmicrosoft.com/11111111-1111-1111-1111-111111111111/demo.readhttps://xxxxxx.onmicrosoft.com/https:/xxxxxx.onmicrosoft.com/11111111-1111-1111-1111-111111111111/demo.read
So, seems program.cs is still generating the wrong scope from:
dotnet new blazorwasm -au IndividualB2C --aad-b2c-instance "{AAD B2C INSTANCE}" --api-client-id "{SERVER API APP CLIENT ID}" --app-id-uri "{SERVER API APP ID URI}" --client-id "{CLIENT APP CLIENT ID}" --default-scope "{DEFAULT SCOPE}" --domain "{DOMAIN}" -ho -ssp "{SIGN UP OR SIGN IN POLICY}" --tenant-id "{TENANT ID}"
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10
From: Luke<EMAIL_ADDRESS>Sent: Sunday, May 3, 2020 6:04 PM
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [dotnet/AspNetCore.Docs] Blazor WebAssembly with Hosted AAD B2C problem (#17319)
This pattern ...
https://xxxxxx.onmicrosoft.com/https://xxxxxx.onmicrosoft.com/11111111-1111-1111-1111-111111111111/API.Accesshttps://xxxxxx.onmicrosoft.com/https:/xxxxxx.onmicrosoft.com/11111111-1111-1111-1111-111111111111/API.Access
... means that the options values weren't correct in the dotnet new command that generated the app.
I'm probably going to set up a table with example values in the topics soon that will make it easier to match what devs need to take from the Azure portal registration to the options for the command. That should happen fairly fast ... within a week or two.
The interesting thing about ...
https://xxxxxx.onmicrosoft.com/11111111-1111-1111-1111-111111111111/API.Access
is that it was failing earlier. They might have made some changes on the Azure side that make that scope work now. When I make my pass on these topics soon, I'll try that again and see if I get the same good result, then I'll update the topic for it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/AspNetCore.Docs/issues/17319#issuecomment-623216064, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH3FBMTSLLXVR6MRKVAMFNTRPYIBRANCNFSM4LKKHMMQ.

@guardrex there's a lot going on in the new command and I get that blazor has been a moving target and Azure B2C is new.  Here's my command if it helps...
dotnet new blazorwasm -au IndividualB2C --aad-b2c-instance "https://xxxxxx.b2clogin.com/" --api-client-id "{server guid}" --app-id-uri "https://xxxxxx.onmicrosoft.com/{server guid}" --client-id "{client guid}" --default-scope "API.Access" --domain "xxxxxx.onmicrosoft.com" -ho -ssp "B2C_1_signupsignin" --tenant-id "{tenant guid}"
And thanks @Alienroid for the blog and comments!

I think it might be nice to just have everyone go with ...
dotnet new blazorwasm -au IndividualB2C -ho

... and then change the default values that the template puts into the framework.
The command for this experience is 😵.

I totally agree with this. The cmd is way too long and confusing 😊
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: Luke Latham<EMAIL_ADDRESS>Sent: Sunday, May 3, 2020 6:53:03 PM
To: dotnet/AspNetCore.Docs<EMAIL_ADDRESS>Cc: Alienroid<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [dotnet/AspNetCore.Docs] Blazor WebAssembly with Hosted AAD B2C problem (#17319)
I think it might be nice to just have everyone go with ...
dotnet new blazorwasm -au IndividualB2C -ho
... and then change the default values that the template puts into the framework.
The command for this experience is 😵.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/AspNetCore.Docs/issues/17319#issuecomment-623225060, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH3FBMSVKQDWE2CVCEVZ7M3RPYNX7ANCNFSM4LKKHMMQ.

I have a similar problem: I can login to AAD B2C with the client and LoginDisplay is correct. But accessing fetchdata on the local server during development leads to a token invalid for this URI error.
During development my API is hosted on my local machine and not in Azure, how must this be configured to accept the token locally?
www-authenticate: Bearer error="invalid_token" or am I missing anything in the Azure config?

For anyone facing the above problem: solution is to add a call to
services.Configure<JwtBearerOptions>(AzureADB2CDefaults.JwtBearerAuthenticationScheme, options =>                     {                         options.Audience = "your Azure AP ID";                     });
to ConfigureServices

@ahlurien On the service side?

Like @ahlurien I had to confiure the audience in my server app to "api://{server app id}/API.Access" to make it work.
I am using AAD and I followed the doc https://docs.microsoft.com/fr-fr/aspnet/core/blazor/security/webassembly/hosted-with-azure-active-directory?view=aspnetcore-3.1

For me it generated the the DefaultAccessTokenScropeWrong. This was what I had there:
            `options.ProviderOptions.DefaultAccessTokenScopes.Add("https://applysolutionsb2c.onmicrosoft.com/https://applysolutionsb2c.onmicrosoft.com/[ID]/[API Name]");`

Correct would have been:
 options.ProviderOptions.DefaultAccessTokenScopes.Add("https://applysolutionsb2c.onmicrosoft.com/api://[ID]/[API Name]");
The documentation says you have to enter: api://41451fa7-82d9-4673-8fa5-69eff5a761fd
I think if you leave await the "api://" and add the Scope Name at the end it would work correctly from the get go.

I have more info on that ...
The doubling of the App ID URI is a known (but unresolved error). It's called out in the docs.
The App ID URI is in the api:// format IF your Azure tenant has a verified publisher domain.
The App ID URI is in the https:// format IF your Azure tenant has an unverified publisher domain.
If you have an https:// App ID URI, then under 3.x, you probably will need to remove the scheme. Under 5.x preview, you should be able to keep the scheme but the server API app will probably need an Authority entry in it's app settings file that matches the App ID URI.
These updates are in the 5.0 preview docs now; so if you flip the doc version selector over to 5.0 preview, you'll see the latest content. The latest guidance tho just went in over the last 72 hours or so and is subject to change without notice as we work the bugs out of the docs.


rs2::pipeline can't playback two IR streams from bag file

Required Info

Camera Model
D400

SDK Version
2.10.3

Issue Description
rs2::pipeline can't correctly playback two IR streams from bag file that recorded two IR streams.
Is this a specification of rs2::pipeline? Or is this a bug?
This code is record IR streams to bag file.
// Record IR Streams to File
rs2::config config;
config.enable_stream( rs2_stream::RS2_STREAM_INFRARED, 1, width, height, rs2_format::RS2_FORMAT_Y8, fps );
config.enable_stream( rs2_stream::RS2_STREAM_INFRARED, 2, width, height, rs2_format::RS2_FORMAT_Y8, fps );
config.enable_record_to_file( "two_ir_streams.bag" );

rs2::pipeline pipeline;
const rs2::pipeline_profile pipeline_profile = pipeline.start( config );

// Output Enable Streams Name
const std::vector<rs2::stream_profile> stream_profiles = pipeline_profile.get_streams();
for( const rs2::stream_profile& stream_profile : stream_profiles ){
    std::cout << stream_profile.stream_name() << std::endl;
}

// e.g. Get 100 Frames for Record
for( uint32_t i = 0; i < 100; i++ ){
    rs2::frameset frameset = pipeline.wait_for_frames();
}

Output is following. It is correctly output the two IR streams.
And, that recorded file can be playback two IR streams correctly using RealSense Viewer.
Infrared 1
Infrared 2

This code is playback IR streams from bag file that recorded two IR streams.
// Playback IR Streams from File
rs2::config config;
config.enable_device_from_file( "two_ir_streams.bag" );

rs2::pipeline pipeline;
const rs2::pipeline_profile pipeline_profile = pipeline.start( config );

// Output Enable Streams Name
const std::vector<rs2::stream_profile> stream_profiles = pipeline_profile.get_streams();
for( const rs2::stream_profile& stream_profile : stream_profiles ){
    std::cout << stream_profile.stream_name() << std::endl;
}

Output is following. Infrared 2 is missing. Why?
Infrared 1

I think this is a bug, try config.enable_all_streams and if the result is the same then this is a bug in config.h which the pipeline uses to resolve streams.
You can works around it by iterating all streams and enabling each one separately.

@zivsha I added config.enable_all_streams() and tried again.
But, the result didn't change. Infrared 2 is still missing.
  // Playback IR Streams from File
  rs2::config config;
  config.enable_device_from_file( "two_ir_streams.bag" );
+ config.enable_all_streams();

Infrared 1

I confirmed that  this problem can avoid in following method.
But, I think it is not smart method. I still think that this problem should be solved.
I think that rs2::pipeline should be play all streams that contained when playback the bag file by default.
@dorodnic What do you think?
// Playback IR Streams from File
rs2::config config;
config.enable_device_from_file( "two_ir_streams.bag" );

// Retrieve Each Streams that contain in File
rs2::context context;
const rs2::playback playback = context.load_device( "../two_ir_streams.bag" );
const std::vector<rs2::sensor> sensors = playback.query_sensors();
for( const rs2::sensor& sensor : sensors ){
    const std::vector<rs2::stream_profile> stream_profiles = sensor.get_stream_profiles();
    for( const rs2::stream_profile& stream_profile : stream_profiles ){
        config.enable_stream( stream_profile.stream_type(), stream_profile.stream_index() );
    }
}

rs2::pipeline pipeline;
const rs2::pipeline_profile pipeline_profile = pipeline.start( config );

// Output Enable Streams Name
const std::vector<rs2::stream_profile> stream_profiles = pipeline_profile.get_streams();
for( const rs2::stream_profile& stream_profile : stream_profiles ){
    std::cout << stream_profile.stream_name() << std::endl;
}

Infrared 1
Infrared 2

Hi @UnaNancyOwen
Yes, your assumptions about the API are reasonable, we should fix it.

@UnaNancyOwen, the problem has been addressed in 2.14.0, along with significant refactoring of pipeline request resolution. In particular, enable_all / default start for recordings will always select all recorded streams. We hope the new solution will offer more consistent and predictable behavior.
Thank you for reporting this issue.
If you have any problem with the new solution please open new issue (and reference this one).


enable single pass transform-evaluate for logistic regression

note this PR includes cumulative changes from
https://github.com/NVIDIA/spark-rapids-ml/pull/488
https://github.com/NVIDIA/spark-rapids-ml/pull/489

build

build

build

Overall looks good, except I am not very sure about the _ClassificationModelEvaluationMixIn class. Better to have Bobby or someone double-check and approve the PR.


Almond: '/usr/bin/unzip': No such file or directory

I just started playing around with this addon and when I requested a cat gif, got this error:

Looked into the problem a little bit more, the Dockerfile specifies the unzip to be installed, but also uninstalls it during the build cleanup.
My guess is that the package should be kept. Are you willing to accept a PR that keeps the unzip in the container?

Thanks!


Extend playlists functionality

[x] Allow users to enter a description when publish a playlist
[x] Allow users to reorder items in a playlist
[x] Add editing / delete options in the playlist fragment view
[x] Allow users to drag and drop items for reordering in a playlist
[x] Support other playlist fields: tags, language?
[x] Support the transient (Now playing) "Queue" playlist which should be reset when the user closes the app
[x] loop control / shuffle play
[x] playlist items while playing + allow to be minimized (similar to YT / odysee web)

All done. https://github.com/OdyseeTeam/odysee-android/pull/362 and https://github.com/OdyseeTeam/odysee-android/pull/366.


Move log location from temp folder to somewhere else

Or store them in a different folder inside the temp folder.

Either the temp folder should be relative to the project (.wurst folder could work, which is cleaned) to avoid problems when the project is on a different drive than the temp folder (https://github.com/peq/wurst4vscode/issues/13)
Or the relativize call must be replaced with something that handles multiple roots, examples here https://stackoverflow.com/questions/204784/how-to-construct-a-relative-path-in-java-from-two-absolute-paths-or-urls

moved to %t/wurst/wurst.log


Scraping other information besides profiles (company profiles, jobs)

Is there any interest in adding this sort of scraping? It would require changing the API as it's pretty much centered around profiles only. I have a potential use-case for it, though.

Yes, sure! let's do an experiment about that, just tell what are you thinking about :smile_cat:

Closing issue due to lack of feedback

fwiw would be useful to have company data too

Yea sorry I would find that helpful too - actually I was working towards that the last time I worked on this and then realized that there would be a number of changes needed...

if you guys create a "companyScraperTemplate.js" like profileScraperTemplate.js I can work on changing the rest of API

Hey,
As mentioned by @a10kiloham. I do want to work on Company data scrapper.
When I submit Company URL instead of the user Profile URL, the scrapper should go to the Company page then click on see all employee and then it will see all employee data.

It should click on each employee and fetch data from the profile as we already do (one by one operation).
Then, it should click on view more and continue its operation

Scrapper should repeat it's operation.
As you mentioned above, I tried Creating companyScrapperTemplate for Step 1 and created click functionality and see more functionality in those relative files.
But it does not seem to be working

if you guys create a "companyScraperTemplate.js" like profileScraperTemplate.js I can work on changing the rest of API


Fix the bug of not showing item's data

Description
Changes proposed in this pull request:

Fixed the bug that makes difficult to use the Panel in the Table's collapsible and children sections. The values from the item variable didn't load in some cases without that fix. Check the Lyczeq/VirtualServices in the cluster. The good example is the VirtualService in the ak namespace.
Fixed a KeyValuePair widget bug. To reproduce the bug, use the KeyValuePair widget and fill the fields. Then go to the YAML and remove the value and a :, 💥 view crashes.

Related issue(s)
Closes #1789

/retest

/retest

/retest

/retest

/retest

/retest


Service maintenance

[DL-5672]
Before embarking on more drastic changes to the application, we want to bring all services up to date. This PR aims to achieve that.
TODO: not sure if missing something...

I ran migrations from a fresh slate, and they worked as expected. ✅
I noticed the report-generation service was crashing; I changed an entry inside the service's volumes section as OP have done it here.

Can you also try it on your side and see if you get the crashing issue and if changing the volume mount fixes it?

I tried the producer/consumer flow between app-digitaal-loket and this app:

It correctly consumes the needed data, and all data is present in the database (no broken links or faulty data based on my local checks).
The submission shows up in the DatabankErediensten frontend table with all the data filled in.
However, clicking Bekijk on the DatabankErediensten frontend displays a blank page. After looking into it, it seems the cause is the enrich-submission service.

The snippet causing the issue is this:

v1.4.0: https://github.com/lblod/enrich-submission-service/blob/v1.4.0/lib/submission-document.js#L310-L324
v1.5.1 and above: https://github.com/lblod/enrich-submission-service/blob/v1.4.0/lib/submission-document.js#L310-L324

reqState.submissionGraph in v1.5.1 and above yields the wrong graph in DatabankErediensten, which leads to a blank page when trying to view a submission's details.

I ran migrations from a fresh slate, and they worked as expected. ✅

I noticed the report-generation service was crashing; I changed an entry inside the service's volumes section as the organisatie-portaal team has done it here.

Can you also try it on your side and see if you get the crashing issue and if changing the volume mount fixes it?

Good catch, as per usual! The service changed to a more modern JavaScript template, which uses this new mount point for external config.

I tried the producer/consumer flow between app-digitaal-loket and this app:
* It correctly consumes the needed data, and all data is present in the database (no broken links or faulty data based on my local checks).

* The submission shows up in the DatabankErediensten frontend table with all the data filled in.

* However, clicking `Bekijk` on the DatabankErediensten frontend displays a blank page. After looking into it, it seems the cause is the `enrich-submission` service.

The snippet causing the issue is this:
* `v1.4.0`: https://github.com/lblod/enrich-submission-service/blob/v1.4.0/lib/submission-document.js#L310-L324

* `v1.5.1` and above: https://github.com/lblod/enrich-submission-service/blob/v1.4.0/lib/submission-document.js#L310-L324

reqState.submissionGraph in v1.5.1 and above yields the wrong graph in DatabankErediensten, which leads to a blank page when trying to view a submission's details.

Because I also saw a blank screen before I made any changes to the stack, I assumed it was broken because I did not have the needed files on my local set-up. That was true, but not the only problem. After changing the enrich-submission-service and adding the files locally, it works now (apart for some errors on loading the remote-data URLs, but that is an issue in the data). See https://github.com/lblod/enrich-submission-service/pull/17


[Ruby] Changes to yardopts and gemspec for YARD migration

See https://github.com/GoogleCloudPlatform/google-cloud-ruby/pull/2232 for context.
We're migrating from jsondoc to YARD for ruby documentation. This requires a few changes to generated configuration files:

The .yardopts file needs a new line:

--markup-provider redcarpet

The gemspec file needs two new dependencies:

gem.add_development_dependency "redcarpet", "~> 3.0"
gem.add_development_dependency "yard", "~> 0.9"

See https://github.com/GoogleCloudPlatform/google-cloud-ruby/pull/2232/files for examples of where these lines should be added.

Oh never mind, this was already done: https://github.com/googleapis/gapic-generator/pull/2194/files


Doesn't get retweets

Hi, thank you for forking and fixing this!
I was scraping tweets based off their username and noticed everything under the retweet column is false, meaning no retweets have been scrapped

Hi
You mean the number of retweets?

Hi You mean the number of retweets?

Sorry no I mean there is a boolean retweet column that indicated whether or not something tweeted by a user is a retweet or not

Hi, any update on this issue?

I'm searching tweets by text and only getting non-retweets too. It is unable to pick up any retweets despite knowing for a fact that it should (using the Twitter direct API) and as you can see every tweet that is picked up has the retweet variable as FALSE.

@ClemLanoe in the wiki there's Retweets             (bool)   - Get retweets done by the user.
Haven't tried it yet but give it a shot if you haven't already.

@ClemLanoe in the wiki there's Retweets (bool) - Get retweets done by the user.
Haven't tried it yet but give it a shot if you haven't already.

Update: this doesn't work


Dates handled improperly on load

After speaking with @randyzwitch  we found that DATE types are handled in mapd 4.4 as epoch days rather than epoch seconds. After loading data via pymapd, dates can be found to be invalid:

Expected Behavior:

Dates show up when selected

Observed Behavior:

Dates do not show up -- invalid date shows up

Reproduction:
Create table in OmniSci:
CREATE TABLE sales (
saleDate DATE
)

load dates
import pandas as pd
import pymapd

mapdcon = pymapd.connect(user=mapduser, password=mapdpass, host=mapdhost, dbname=mapddbname, port=mapdport, protocol=mapdprotocol)

# Create some dates
dates = pd.date_range('20190101', periods=10)
df = pd.DataFrame(dates)

# load into mapd
mapdcon.load_table_columnar('sales', df, preserve_index=False)

Per internal discussion;

Yes, this will require an update in the connector. You will need to read the encoding on the SQLType of the column you are inserting data into. If the encoding is FIXED or NONE, you need to use seconds. If the encoding is DATE_IN_DAYS, you need to use days.


Build fails on Debian 5.10.0

Are you using the latest driver?
yes
Are you using the latest EVDI version?
1.9.0+dfsg
If you are using a DisplayLink device, have you checked 'troubleshooting'
on DisplayLink's website?
yes, HP Port replicator 3005
Is this issue related to evdi/kernel?
yes

Linux distribution and its version Debian Bullseye
Linux kernel version 5.10.0
Xorg version (if used)
Desktop environment in use

During "Building initial module for 5.10.0-21-amd64 following error occurs:
Bad return status for module build on kernel:5.10.0-21-amd64 (x86_64)
post installation script returns error code 10
I mention that on /var/lib/dkms/evdi/1.9.0+dfsg the tree "build" is not existing
(
initial problem I have, may contribute to the problem:
after installing displaylink driver, section Display Link info states (all other sections look like expected):
Driver version: 1.12.0
1.9.0+dfsg
DisplayLink service status:
EVDI service version /sys/devices/evdi/version not found
)
thanks for your help!

btw: v1.12.0 is showing the exact same symptom with empty service status and empty EVDI service version.

turned out that probably secure boot was not turned off :( sorry!


Improve presence detection using bluetooth object tracking

The shelly's installed across the house have bluetooth, which can be used to track other bluetooth objects in the house. Because there are many Shelly's triangulation can be used to determine which room the object is in.
There are a couple of options

https://espresense.com/
https://www.room-assistant.io/
ESPHome component https://community.home-assistant.io/t/esphome-component-for-espresense-like-room-tracking/386678/10

Ideally use something compatible with ESPHome, because this already works really good with Home Assistant.

Can we use https://www.home-assistant.io/integrations/mqtt_room/ together with ESPHome and the Bluetooth RSSI strength https://esphome.io/components/sensor/ble_rssi.html?

And/or use the bluetooth distance custom component mentioned here https://github.com/rpatel3001/BleDistance.

Setup iBeacon software on esp32?

In combination with https://owntracks.org/booklet/features/comparison/ seems you can do room occupancy tracking.

https://www.beaconzone.co.uk/blog/positioning-tracking-and-flow-using-bluetooth-signals/

https://github.com/dalehumby/ESPHome-Apple-Watch-detection

Tried to add
esp32_ble_beacon:
  type: iBeacon
  uuid: 12a13a53-905b-0654-3468-3956014a7888

to the supply-closet-shelly but this resulted in a crash loop. Got the logs from the device by connecting with the usb voltlog flasher and saw this message:
assert failed: s_prepare_reserved_regions memory_layout_utils.c:94 (reserved[i + 1].start > reserved[i].start)

Backtrace:0x40081aca:0x3ffe3390 0x400904f9:0x3ffe33b0 0x400962f1:0x3ffe33d0 0x4013db01:0x3ffe34f0 0x40127fee:0x3ffe3850 0x401264c8:0x3ffe3c00 0x40081c33:0x3ffe3c40 0x40079508:0x3ffe3c80 0x40079965:0x3ffe3cc0 0x400806cd:0x3ffe3df0 0x40007c15:0x3ffe3eb0 0x4000073d:0x3ffe3f20

ELF file SHA256: 966f2b982dd75de8

Rebooting...

Found this issue https://github.com/espressif/esp-idf/issues/2561 on the topic.

Fixed this particular issue  for shelly by changing the board type from esp32doit-devkit-v1 to esp32dev.

Some more software https://github.com/andrewjfreyer/monitor and https://github.com/andrewjfreyer/presence though both are no longer maintained it seems.

This would be perfect https://github.com/rpatel3001/BleDistance, but it seems it's not possible for the iPhone to advertise as an iBeacon while an app is in the background https://stackoverflow.com/questions/20121342/advertising-with-ibeacon-when-app-is-in-background.

What would be cool is to use the bluetooth_proxy feature of ESPHome https://esphome.io/components/bluetooth_proxy.html together with Home Assistant to let Home Assistant do triangulation of which ESPHome device is closer to the advertised bluetooth device. Reading this https://community.home-assistant.io/t/new-esp32-bluetooth-proxy-how-is-it-supposed-to-work/458377/40 this is not yet possible.

After discovery work in https://github.com/mvgijssel/setup/pull/122 figured out the following. For (unique) Apple watch tracking using bluetooth one of the following is necessary

Use the NimBLE stack in ESP-IDF which supports Resolvable Private Address (RPA). Open issue for bluedroid, the standard for ESPHome https://github.com/espressif/esp-idf/issues/2262 and https://github.com/espressif/esp-idf/issues/8613.

Use bluetooth classic instead of bluetooth low energy which is working for room assistant https://www.room-assistant.io/integrations/bluetooth-classic.html#requirements. Bluetooth classic example for esp-idf esp32 can be found here https://github.com/espressif/esp-idf/tree/1c84cfde14dcffdc77d086a5204ce8a548dce935/examples/bluetooth/bluedroid/classic_bt/bt_discovery.

Use the LocalName instead of the RPA as mentioned here https://stackoverflow.com/questions/66439992/home-automation-with-esp32-bluetooth-and-iphone-drives-me-crazy.

Because the Shelly’s are relatively close to each other in some places we get results that tend to jump around a lot. For example kitchen, supply closet and living room.
Therefore as an improvement try to implement https://github.com/ESPresense/ad-espresense-ips which should give x, y, z coordinates based on multiple nodes and use triliteration to figure out what room the object is in.
This requires the following steps:

[ ] Implement HACS in home assistant
[ ] Setup AppDaemon Apps in HACS
[ ] Setup the mentioned app inside AppDaemon


Sorting isn't disabled if ASC and DESC are applyed after each other

Description
Sorting in column header is not disabled correctly
DBeaver Version
23.0.0
Operating System
Any
Database and driver
Any
Steps to reproduce

Apply sort by DESC in column header
Apply sort by ASC in the same column header
Click disable order in column header

Expected: Sorting is disabled
Actual: Sorting returns to DESC
Additional context
Note: This is not reproducible the other way around (ASC -> DESC) only DESC -> ASC
https://user-images.githubusercontent.com/82543000/222984324-7ae0b1c2-8953-4718-b0aa-10742a449b22.mp4

I can confirm the bug provided by Matvey16, and add another detail.
It seems the 'Disable order by' option in column header always applies the 'next' option as shown in the dropdown list:

I.e.:

When selecting 'Order by ... ASC', the disable order makes a DESC ordering.
When selecting 'Order by ... DESC', the disable order makes a disabled ordering.

It has nothing to do with the ordering option previously selected.


Good article, but how do you implement an approval step if you are an enterprise that doesn't use Mailchimp, but hasn't yet migrated to o365?  I note that the Teams connectors are in Preview which seems to rule them out

[Enter feedback here]

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

ID: 616cbf28-e4c7-a539-911c-8c601031831a
Version Independent ID: cad3d887-b65e-b2e0-a6b6-49e6cdb9e108
Content: Build approval-based automated workflows - Azure Logic Apps
Content Source: articles/logic-apps/tutorial-process-mailing-list-subscriptions-workflow.md
Service: logic-apps
GitHub Login: @ecfan
Microsoft Alias: estfan

@MattNamaste Thanks for the feedback! We are currently investigating and will update you shortly.

Hello @MattNamaste, Thank you for for reaching out and apologies for the delay.
You can implement a customized approval process using Logic App as described by Derek Li.
If MailChimp is not a preferred option and depending on the services you want to use you can also create your own custom connector  instead.
As we primarily monitor this channel for doc improvements I will now proceed to close this issue out. Please tag me in your response if there are additional concerns and we will gladly continue with our discussion. Thank you!


remove redundant requirements

What does this PR do?

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

i removed redundant reqs from pyproject.toml since all these were already part of requirements.txt
any reason we need both ? cant we just keep requirements or toml file ?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Arf I read this PR too fast, I shouldn't have merged it. We need the pyproject.toml to specify dependencies for Poetry, then the requirements.txt is generated autoamtically.
Anyway, we are moving to a dedicated fork today or tomorrow so this won't have any impact.


Update java version property in pom.xml

This is implemented in rewrite-spring src/main/resources/META-INF/rewrite/spring-boot3-upgrade.yml

type: org.springframework.sbm.build.migration.actions.SetProperty
propertyName: "java.version"
propertyValue: "17"
description: Set Java version to 17


non-public apis used?iTuens Connect rejected

I also meet this problem,detail is suitename function is a non-public selector

why? I can found the function in NSUserDefaults.h of  system files.

@fandongtongxue    I also meet this problem, Did you found any sololution?

@AsHighAsHonor change to another thirdpart lib


added option to use Glyhpicons in bootstrap templates

When using the Bootstrap Theme Glyphicons can be enabled for the help icon rather than using Font Awesome.

@bvaughn I believe the CircleCI test failed because of a CircleCI DB outage. If there are any issues let me know!

Thanks for PR man! Added a couple of discussion points. Let me know what you think!

I am not able to determine what is causing the issue with CircleCI running the tests. Any suggestions?

Don't worry about the Circle CI tests. I never quote got those ironed out (as you may have noticed). As long as Travis passes, we're good.

Okay, so first off I'd like to apologize for being so slow with my response times. I've been super busy this week with my day job.
That being said- I like what you've done here. I'd like to propose one additional change though. Rather than just making the Bootsrap template customizable- let's do both it and the default formFor template. I think the way to do this is as follows:
• FormForConfiguration should default helpIcon to null. (This would also necessitate a small documentation update to setHelpIcon.)
• Both default and bootstrap templates should specify their own default icons but allow the helpIcon to override them if set like so:
bootstrap

Default

?

I think everything else you've done looks great. If you're willing to make this one additional small change- I'd greatly appreciate it and I'd be happy to merge the PR. Thanks so much for contrinbuting! :)

No worries! I am glad I was able to contribute and hopefully make this more friendly for future users as well as myself!
I definitely agree with your recommendations and I have went ahead and implemented those changes. If I missed something or you have any other suggestions please let me know! Thank you for the suggestions and welcoming PRs.

Thanks for the contribution, Spencer!


Update archives sha for standalone feature

Fixes #65

Hello, I am a code review bot on flows.network.
It could take a few minutes for me to analyze this PR. Relax, grab a cup of coffee and check back later. Thanks!

@jprendes Thanks for the fix!


docs: collapse codeblocks by default

Collapsed

Expanded

:tada: This PR is included in version 17.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Recurring problems with users being unable to change emails

Users are not able to change their emails when additional emails are added to COmanage:

MESH-Research/hc-admin-docs-support#144
MESH-Research/hc-admin-docs-support#137
MESH-Research/hc-admin-docs-support#112 (possibly)

User's email addresses are stored in shib_email usermeta, set in humanities-commons/hc-simplesaml.php::hcommons_set_shibboleth_based_user_meta(). They are set based on $_SERVER['HTTP_MAIL']  variable ('mail' header).

The header is set on login. Typo in logging code.

Log entries will be generated in hcommons_error.log recording the value of shib_email when a user logs in and when a user visits the settings general page. Users should try re-logging in and check the log entries.

Icebox pending user feedback

Two users are fixed, the other is non-responsive. I'm closing this for now. We can reopen if more users report issues.

Reopening this as a new issue and linking back to this one. Sorry!


CourseRosterExpansion

Code completed and tested which can expand or replace a course's roster.

Just tested this and it works!


HttpServletRequest m_QueryString isnull！

CppMicroServices/httpservice/src/HttpServletRequest.cpp getQuery_string fault

please add reproduction steps so someone can help diagnose the issue. thanks!


One link still broken

This is a follow-up to my previous issue #9293, which was closed following pull request #9341. There are still three links that are not correctly entered in the table:

PowerShell -> Management options: Set-MpPreference and Update-MpSignature.
Microsoft Azure -> Reporting options: after “…SIEM tool to report on “.

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

ID: af205543-b539-a7e8-a66c-f9365241fec9
Version Independent ID: b9d4b071-f501-51a8-8565-859df13b7fa7
Content: Deploy, manage, and report on Microsoft Defender Antivirus
Content Source: microsoft-365/security/defender-endpoint/deploy-manage-report-microsoft-defender-antivirus.md
Product: m365-security
Technology: mde
GitHub Login: @denisebmsft
Microsoft Alias: deniseb

@Hindy-UK thank you for your attention to detail and for catching that we had STILL missed those three links. They're fixed now, and you will see the update render live later today. My apologies for the inconvenience. I appreciate you helping us to improve the quality of our documentation. Thank you again, and I hope you have a great weekend!


Comments being removed

We use our styles at userstyles.org, so some css definitions have placeholder comments within the definition (see "placeholder" in the example below) which get replaced with user set choices:
.selector {
  /* comment 1 */
  color: /*[[placeholder]]*/ white;
  /* comment 2 */
}

With default settings, the result removes the essential placeholder comment:
.selector {/* comment 1 */
    color: white; /* comment 2 */
}

Looks like an issue with resetting decl.between. Thanks for reporting.

Released as 1.2.1.

Thanks for the fix!

Still one minor issue though. If I include a comment before the selectors, it adds an extra carriage return now:
Starting with:
/* comment 0 */
.selector {
  /* comment 1 */
  color: /*[[placeholder]]*/ white;
  /* comment 2 */
}

the result becomes:
/* comment 0 */

.selector {/* comment 1 */
    color: /*[[placeholder]]*/ white; /* comment 2 */
}

Any way to get the first comment directly above the selector, and the second comment back on it's own line? :grin:

This might need a bit more thought. For example if we have some CSS like this:
h1 {
  color: black; /* Don't change this line! */
}

The formatter would then break the semantics of the comment. I wonder if there is some way to tell if both the property value pair & the comment are on the same line.
https://github.com/ben-eb/perfectionist/issues/4

I got a little lost trying to read through the code, so I don't know how much help I could be with troubleshooting. I guess it's not a huge deal if the comment ends up on the next line, given you are trying to support different levels of formatting.
Adding an option for comment formatting would be awesome! :cake:


Allow for a url from a self-hosted GitHub Enterprise server instead of the public GitHub SaaS offering.

Certain users use self-hosted GitHub Enterprise server instead of the public GitHub SaaS offering. This would mean the GitHub repo URL is a custom one instead of the standard https://github.com/.
There is string validation that does not allow  code for this. Could we support these custom URLs?

There is no restriction on the repository_url that can be provided to the block. If a user has a different domain for their GitHub instance, they should be able to use that without issue. Do enterprise GitHub users see errors when using a non github.com URL?


Standardize use values

Standardize use values, they are not consistent in the rules

standardized on:
observation
reconnaissance
ground attack
fighter
bombing
seaplane fighter
added second flag for crew count, remove fighter flag


Swift 2.2 upgrade

Swift 2.2 changed @asmname to @_silgen_name

Thanks for the PR, I did a more complete 2.2 support fix here: https://github.com/neonichu/xctester/commit/07845f17a8881f0506cc1b7116a1694dda3d103f


initial commit of Set-DbaMaxDop

This PR includes the Test-DbaMaxDop as a changed file, can you remove the file and commit again please?

Done @dansqldba

Set of tests for this PR
SQL Server 2000

[ ] Don't run on version 2000 and throw an message

SQL Server 2005 - 2016

[ ] Run command on a single instance. Do not specify the -MaxDop parameter. Must change the MaxDop for the recommended one.
[ ] Run command on a single instance. Specify the -MaxDop parameter to other than the current value. Must change the MaxDop for the specified one.
[ ] Run command on a single instance. Specify the -Databases parameter and select one. Must give warning saying "Server '$servername' does not supports Max DOP configuration per database. Run the command again without -Databases parameter. Skipping."
[ ] Run command on mutiple instances. Do not specify the -MaxDop parameter. Must change the MaxDop for the recommended one.
[ ] Run command on mutiple instances. Specify the -MaxDop parameter to other than the current value. Must change the MaxDop for the specified one.
[ ] Run command on mutiple instance (one of them 2016) and specify -Database. Should see a warning for non-2016 version saying "Server '$servername' does not supports Max DOP configuration per database. Run the command again without -Databases parameter. Skipping.". On 2016 instance the database should be configured with recommended value.

SQL Server 2005 - 2014 ONLY

[ ] Run command on a single instance and specify -Database. Should see a warning "Server '$servername' does not supports Max DOP configuration per database. Run the command again without -Databases parameter. Skipping."

SQL Server 2016

[ ] Run command on a single instance and specify -Database. If using -Verbose should see a output "Server '$servername' supports Max DOP configuration per database."
[ ] Run command on a single instance and specify -Database. Do not specify the -MaxDop parameter. The configuration must be applied only on the Database(s) defined.
[ ] Preparation. Set the same MaxDop for every database (User databases). Set MaxDop on instance level to 0. Run the command on that instance. The result shoulb be all databases max dop set to 0 and instance set to the databases max dop. That way if we add new databases as the default is 0, all databases are inherith the instance level configuration.

Thorough AF 😍

Any takers?

[ ] Working/useful help content, including link to command on dbatools web site
[ ] All examples work as advertised
[ ] Does not contain template content
[ ] Does not contain excessive/unnecessary amounts of comments
[ ] Works remotely
[ ] Works locally
[ ] Works on lower versions or throws error specifying version not supported
[ ] Works with named instances
[ ] Works with clustered instances
[ ] Handles offline/read only databases
[ ] Supports multiple servers (at the command line or piped from Get-SqlRegisteredServerName)
[ ] No un-handled errors which stop the command working with multiple servers

[ ] Does not support "sql2005","sql2008" | Set-DbaMaxDop
[ ] Set-DbaMaxDop -SqlServer sql2005 -MaxDop 1 -Databases mydb should throw warning but it does not.
[ ] Set-DbaMaxDop -SqlServer sql2005,sql2008, sql2014 -MaxDop 10 -Data does not autopopulate - I'm assuming $sqlserver[0] was not used in the autopopulation thing. But should it? I kinda like that it doesn't.
[ ] I would expect for database DOP to change only if I specified -Databases, so Set-DbaMaxDop -SqlServer sql2016 -Verbose had two issues -

it changed all of my database dops (did it? Maybe it's just reporting for each one and it didn't change the database, just the instance over and over)
the output was wrong

If it is changing per db without dbs set, please only change instance level if databases is not set since it's inherited.

[ ] Set-DbaMaxDop -SqlServer sql2016 -Databases db1 -MaxDop 55
Instance                   : sql2016
Database                   : db1
OldDatabaseMaxDopValue     : 0
CurrentDatabaseMaxDopValue : 55
OldInstanceMaxDopValue     : 5
CurrentInstanceMaxDop      : 5

This made me worried that I had changed the server dop when I only intended to change the database dop. So I tested with -MaxDop 10 and saw it only changed the database.
Can  OldInstanceMaxDopValue be dropped and CurrentInstanceMaxDop  changed to InstanceMaxDop when -Databases is specified?

[ ] Set db3 to offline on sql2016 then ran Set-DbaMaxDop -SqlServer sql2016 -Databases db3  -- it just returned nothing

Changes made. Please re-test.
The only one will be for a future enhancement is the "Set db3 to offline on sql2016 then ran Set-DbaMaxDop -SqlServer sql2016 -Databases db3 -- it just returned nothing"


Provider complains about custom attributes

Issue
I use a plugin for Babel that annotates the React components ( https://github.com/fullstorydev/fullstory-babel-plugin-annotate-react ). When booting up, I get the following errors:
index.js:1 [react-use-intercom] some invalid props were passed to IntercomProvider. Please check following props: data-element, data-source-file.

Upon investigation, it looks like it complains about anything it doesn't know about in the ...rest spread:
  if (!isEmptyObject(rest) && __DEV__)
    logger.log(
      'error',
      [
        'some invalid props were passed to IntercomProvider. ',
        `Please check following props: ${Object.keys(rest).join(', ')}.`,
      ].join(''),
    );

I feel like this should be loosened up to at minimal ignore data- attributes.

@devrnt - Happy to make a PR but... moving these to warnings will still flag my system so I'd prefer for it to ignore any data- attributes. Its pretty typical for things to be tagged using that for testing/etc.

I see, ignoring the data-x attributes seems okay to me. Please add this extra check inside the __DEV__ guard (see above)  so it has only impact on the dev bundle. If you would be so kind to submit a PR, then I'm eager to approve
By the way: out of interest what does FullStory do with the data-x attributes?

@devrnt - Take a look at: https://github.com/devrnt/react-use-intercom/pull/480

By the way: out of interest what does FullStory do with the data-x attributes?

The custom attributes the plugin adds gives us the ability to define my explicit conditions for e2e and analytics. This plugin denotes all the components and their source for our team to identify root cause issues more easily.

v1.5.2


Modularize Home Screen

Dashboard elements in the home screen is now rendered as a list of components.  Also changed the rendering strategy to render each component as a separate <Accordion /> so that elements can be hidden (all expanded by default).
Coins is now first on the list so that stats can be viewed together, followed by GPU/CPU stats, overall stats, and  worker graphs.
Lastly updated the dashboard components to return a placeholder message if there is no data to render.
@evan-cohen What are your thoughts on using <p /> for placeholder text?  Should we be using <Typography /> instead?
NOTE: The list of components is currently hard-coded, but with the inclusion of CPU mining support the logic will be tweaked to selectively filter components dynamically.

@evan-cohen What are your thoughts on using <p /> for placeholder text? Should we be using <Typography /> instead?

IIRC, the advantages of using <Typography> are centered around the element being "registered" with MUI for consistent font family/size, theming, etc. I'm not opposed to using a paragraph tag provided there aren't any inconsistency issues with the placeholder text style (esp. while toggling between light and dark mode).

@evan-cohen What are your thoughts on using <p /> for placeholder text? Should we be using <Typography /> instead?

IIRC, the advantages of using <Typography> are centered around the element being "registered" with MUI for consistent font family/size, theming, etc. I'm not opposed to using a paragraph tag provided there aren't any inconsistency issues with the placeholder text style (esp. while toggling between light and dark mode).

I'm going to swap it over to use <Typography /> then just to be safe.


Update for 20241212

Hi! This is the friendly automated conda-forge-webservice.
I've started rerendering the recipe as instructed in #13.
If I find any needed changes to the recipe, I'll push them to this PR shortly. Thank you for waiting!
Here's a checklist to do before merging.

[ ] Bump the build number if needed

Fixes #13

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

ℹ The recipe is not parsable by parser conda-souschef (grayskull). Your recipe may not receive automatic updates and/or may not be compatible with conda-forge's infrastructure. Please check the logs for more information and ensure your recipe can be parsed.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12333895670. Examine the logs at this URL for more detail.

Hi! This is the friendly conda-forge automerge bot!
I considered the following status checks when analyzing this PR:

linter: passed
azure: failed

Thus the PR was not passing and not merged.

see https://github.com/conda-forge/torchvision-feedstock/issues/103

@conda-forge-admin please rerender

Hi! This is the friendly conda-forge automerge bot!
Commits were made to this PR after the automerge label was added. For security reasons, I have disabled automerge by removing the automerge label. Please add the automerge label again (or ask a maintainer to do so) if you'd like to enable automerge again!


Infer nullability from is not null clause

IE:
CREATE TABLE test(
  value TEXT
);

selectNulls:
SELECT value
FROM test;

selectNonNulls:
SELECT value
FROM test
WHERE value IS NOT NULL;

first one generates a boxed string, second one should just be the non null string.

#867


feat: wip on replacing artwork grids with more performant masonry #doNotMerge

This PR resolves []
Description

wip
PR Checklist

[ ] I tested my changes on iOS / Android.
[ ] I added screenshots or videos to illustrate my changes.
[ ] I added Tests and Stories for my changes.
[ ] I added an app state migration.
[ ] I hid my changes behind a feature flag.
[ ] I have prefixed changes that need to be tested during a release QA with [NEEDS EXTERNAL QA] on the changelog.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

cc @MrSltun

Closing for now, will revisit if they fix the issue from flashlist


Add zenhub release management

Fixes #102

Codecov Report

Merging #106 into master will increase coverage by 0.14%.
The diff coverage is 18.18%.

@@            Coverage Diff             @@
##           master     #106      +/-   ##
==========================================
+ Coverage   19.62%   19.77%   +0.14%
==========================================
  Files          10       11       +1
  Lines         540      617      +77
==========================================
+ Hits          106      122      +16
- Misses        434      495      +61

Impacted Files
Coverage Δ

loghub/cli/common.py
31.57% <0%> (-1.76%)
:arrow_down:

loghub/cli/main.py
12.98% <0%> (-1.1%)
:arrow_down:

loghub/core/formatter.py
7.23% <3.44%> (-0.43%)
:arrow_down:

loghub/core/repo.py
31.52% <33.33%> (+0.03%)
:arrow_up:

loghub/core/zenhub.py
34.14% <34.14%> (ø)

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 d52be39...7af89bc. Read the comment docs.


Can I do transfer learning training on my own data set?

Thank you so much for your contribution, it was an outstanding piece of work and very exciting!!! Can I train on my data set to give specific category labels?

Interested to know as well!

See #5

There is no helpful info in #5


What if a pre-build custom image have more than one prediction APIs?

I have a pre-build image which have more than one prediction APIs, /prediction1 and /prediction2 for example. I want to put this image into Kfserving which bind two prediction APIs with my custom APIs.
Can anyone share an example for my case? Thanks in advance.

/cc @Iamlovingit

@rtrobin Hi, we just added this new feature. You can look at this sample.

Hi @Iamlovingit . Perfect! It is just what I expected. I will try it soon and give you feedback.
Btw, I raised another question about custom url. I think the sample you mentioned in this post, should fix my problem too. Right?

@rtrobin yes, we relax the standard of url format, you can use your customized urls now.

@Iamlovingit I have tried your sample, but met some issues.
By running kubectl get inferenceservice customized-sample, it shows below. Which is not like the http://customized-urls-sample.default.<IP_ADDRESS>.xip.io in sample doc. Is the sample running on specific version?

NAME
URL
READY
DEFAULT
TRAFFIC
CANARY
TERAFFIC
AGE

customized-sample
http://customized-sample.test-serving.example.com/v1/models/customized-sample
True
100
.
.
.
10m

Also, running curl http://customized-sample.test-serving.example.com/v1/models/customized-sample:predict script, it shows curl: (6) Could not resolve host: customized-sample.test-serving.example.com.

It looks like you have not use the right version. You should update your code to latest, and build the KFServing, you can look at thsi section.

@Iamlovingit Hi, do you know which version could support this sample case? We are using the release version 0.3.0 right now. Do you know when to release next version? Or maybe which commit is about this feature? We could merge the commit in our build. Thanks.

The latest master branch is supoort this feature, you can pull the latest commit. Next week KFServing will cut the 0.4 release version. This feature will be included.

Thanks. I would wait to check it in next release.

/close
this feature is released in 0.4.


dd: Eliminate code duplication

dd contains functions dd_fileout and dd_stdout which repeat the same code. The same issue is present with duplicate write_blocks procedures. @miDeb suggested this patch, which solves this issue by creating a OutputTrait which can be implemented generically for all outputs. This seems to be a good solution, but others could be considered as well.

I've applied @miDeb's patch locally, and it's looking good. I will likely create a PR shortly.


fix(tests): improve sorting algos section

Pre-Submission Checklist

[x] Your pull request targets the staging branch of freeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[x] Closes currently open issue (replace XXXX with an issue no): Closes #14204

Description

@bonham000 as discussed, this PR makes improvements to the interview prep section's sorting algos challenges (Bubble Sort, Selection Sort, Quick Sort, Insertion Sort, Merge Sort):

prevents solving challenges with .sort() as initially raised in #14204
adds new test case that ensures the returned array has same members as original array (i.e. the only thing that has changed is the order)
removes un-needed "helper" function from boilerplate code - was not being utilized
add changes to instructions to point out arrays should be sorted from least to greatest
added note to explain that things are happening behind the scenes, user does not need to explicitly call the function, and to log as normal if they want to see what's going on
adds commented out test array to boilerplate so user can see array we are starting with
replaced <b> tags with <strong> for improved accessibility (how screen readers read tags is different)

Actually putting an explicit call to the sort functions in the editor ends up confusing things if user was to log array to see steps of sort. It would log the array that we are passing in from behind the scenes, as well as the array being passed in in the code editor. Adding a note to explain this seemed like the better option so if they do log, logs will not be confusing.

This looks good to me and after discussion with @no-stack-dub-sack earlier I'm going to merge this.

Satu


Validation failing for empty values

Unless I am missing something, most rules should not be validated (or validation should return true) if the input is null or an empty string (except for Required, Length and some others).
There are likely more rules failing for empty values, but the ones I found are:

Alpha
AlphaNumHyphen
AlphaNumeric

Can you please provide an example and describe the expected output contrasted with the actual output? Thanks!

apologies, issue is intended for siriusphp/validation


this Node Loop-generic_loop_Loop__42 has no value_infer on SSD-mobilenet-int8

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

Go to : https://github.com/onnx/models/blob/main/validated/vision/object_detection_segmentation/ssd-mobilenetv1/model/ssd_mobilenet_v1_12-int8.onnx

SSD-MobilenetV1-12-qdq
import onnx_tool
import numpy as np
modelpath = '/home/ssd_mobilenet_v1_13-qdq.onnx'
m = onnx_tool.Model(modelpath)
m.graph.shape_infer({'inputs': np.ones((1,383,640,3), dtype=np.uint8)})  # update tensor shapes with new input tensor
m.graph.profile()
m.graph.print_node_map()  # console print
m.save_model('/home/ssd_mobilenet_v1_13-qdq-mods.onnx', shape_only=False)  # only with weight tensor shapes and dynamic tensor shapes

See error
NotImplementedError(f'this Node {self.op_type}-{self.name} has no value_infer
this Node Loop-generic_loop_Loop__42 has no value_infer

Screenshots
If applicable, add screenshots to help explain your problem.

it's a known issue that Loop is not supported. But this model is not simplified.

The real model body does not have this Loop node. It should start with this Mul node.


Restart with failure for the last node

[Subject of the issue]
Restarted like the last  client sever for doc example after stop all nodes. The three other without problem restarted.
go run main.go server --data-dir ./test-chain-4 --chain genesis.json --grpc :40000 --libp2p :40001 --jsonrpc :40002 --seal
Description
2021-06-05T16:32:54.961+0200 [INFO]  polygon.jsonrpc: http server started: addr=<IP_ADDRESS>:40002
2021-06-05T16:32:54.961+0200 [INFO]  polygon.consensus.ibft: syncing past snapshots: from=0 to=4162
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0xc8ad97]
goroutine 1 [running]:
github.com/0xPolygon/minimal/consensus/ibft.(*Snapshot).Copy(0x0, 0x0)
/home/robert/Documents/polygonsdk/polygon-sdk/consensus/ibft/snapshot.go:367 +0x37
github.com/
Describe your issue in as much detail as possible here
Your environment

OS and version
version of the Polygon SDK
branch that causes this issue

Steps to reproduce

Tell us how to reproduce this issue
Where the issue is, if you know
Which commands triggered the issue, if any

Expected behaviour
Tell us what should happen
Actual behaviour
Tell us what happens instead
Logs
Please paste any logs here that demonstrate the issue, if they exist
Proposed solution
If you have an idea of how to fix this issue, please write it down here, so we can begin discussing it

@fastepic Hello, I'm sorry I couldn't answer this issue sooner. I had been trying to reproduce the issue in my local environment.
I think it's rare case. The issue seems to happen when node failed to load snapshot for IBFT and has empty snapshots on initializing. You can produce this error easily by deleting consensus/metadata and consensus/snapshots and starting node again. I'm not 100% sure the issue you faced are caused by same reason, but perhaps node couldn't store these data when you faced this issue.
I will fix corresponding code. Thank you for your report.

Thank you. It is possible that computer was in heavy work ( other processes ) when I test it so it can be correlate with your suggestion and was problem with storing data.

Hi, @fastepic. I've fixed the issue and the code has been merged into develop branch. I will close this issue for now. I'm sure this issue will no longer happen. But you can reopen this issue and continue discussion if you find related issue. Thanks.


cadquery-ocp 7.6.3a0 not working on Windows 7

The cadquery-ocp 7.6.3a0 version included a compiled vtk 9.1, which used the tbb12-XXX.dll. This dll had a call to Kernel32.dll for function "GetCurrentThreadStackLimits", and windows7 does not have it. It is only available on Windows 8+.
The offical vtk 9.1 package does not have this depenedency, thus, it can be executed on windows 7.
The old cadquery-ocp <IP_ADDRESS>'s tbb dll does not have this function call.

The win_amd64 wheel was built by running (link):
conda install -n cadquery-ocp -y ocp=7.6.* vtk=9.1.0

and then packaging the tbb12.dll that was installed.
I can see that https://conda.anaconda.org/conda-forge/win-64/tbb-2021.6.0-h91493d7_0.tar.bz2 was installed and I can see "GetCurrentThreadStackLimits" in tbb12.dll.  So I'm guessing that it's not a vtk problem, it's just that in Oct 2022, when the wheel was built, there was a newer version of tbb, and that was packaged into the wheel.
I don't have access to a Windows machine so it's hard for me to propose a fix, but @polycorn, if you are a conda user, perhaps you can find an install command that installs ocp 7.6, vtk 9.1, and doesn't install an incompatible version of tbb.  Wheels are built on Windows Server 2022 or 2019 so ideally you'd check on that, but I imagine checking on Windows 10 might be good enough.
If you do this, please paste the output of the install command and your Windows version.

Thanks for the reply ! I am not familiar with conda, but i will look into it later.
From what I have seen, https://anaconda.org/conda-forge/tbb/files has a version https://anaconda.org/conda-forge/tbb/2021.5.0/download/win-64/tbb-2021.5.0-h91493d7_5.tar.bz2, this tbb library is available to win7.

We have moved well past this version of OCP and VTK, so I'm going to close this. Please reopen if this issue persists with the latest wheels.


docker.io/rancher/k3os:latest: not found

Version (k3OS / kernel)
k3os version v0.10.0
5.0.0-43-generic #47~18.04.1 SMP Wed Apr 1 16:27:01 UTC 2020
Architecture
x86_64
Describe the bug
My k3os stopped functioning because k3os-system/apply-k3os-latest is stuck on this error:
Failed to pull image "rancher/k3os:latest": rpc error: code = NotFound desc = failed to pull and unpack image "docker.io/rancher/k3os:latest": failed to resolve reference "docker.io/rancher/k3os:latest": docker.io/rancher/k3os:latest: not found

To Reproduce
sudo crictl pull docker.io/rancher/k3os:latest
Expected behavior
without problem
Actual behavior
FATA[2020-04-25T11:49:45.325673572+10:00] pulling image failed: rpc error: code = NotFound desc = failed to pull and unpack image "docker.io/rancher/k3os:latest": failed to resolve reference "docker.io/rancher/k3os:latest": docker.io/rancher/k3os:latest: not found

Additional context
Looks like you have forgot to tag the latest image as latest

This is because you’ve specified a tag that doesn’t exist as the version for the k3os-latest plan? If you want the latest, remove the version on the plan and let the system upgrade controller figure out the latest from the channel (supplied by default).

@dweomer I don't think I have specified a tag anywhere. In k3os-system/system-upgrade-controller, there are errors:
system-upgrade-controller E0425 02:07:23.611340       1 controller.go:135] error syncing 'k3os-system/k3os-latest': handler sys │
│ tem-upgrade-controller: Get https://github.com/rancher/k3os/releases/latest: dial tcp: lookup github.com on <IP_ADDRESS>:53: read │
│  udp <IP_ADDRESS>:57031-><IP_ADDRESS>:53: i/o timeout, requeuing

https://github.com/rancher/k3os/issues/437

@dweomer I don't think I have specified a tag anywhere. In k3os-system/system-upgrade-controller, there are errors:
system-upgrade-controller E0425 02:07:23.611340       1 controller.go:135] error syncing 'k3os-system/k3os-latest': handler sys │
│ tem-upgrade-controller: Get https://github.com/rancher/k3os/releases/latest: dial tcp: lookup github.com on <IP_ADDRESS>:53: read │
│  udp <IP_ADDRESS>:57031-><IP_ADDRESS>:53: i/o timeout, requeuing

#437

@guiguan it looks like you are getting closer to the underlying issue. That's a DNS timeout.

@dweomer ohh, probably the coredns was not running because the k3os node (single node cluster) was cordoned off by a failure upgrade process. Now, I have removed the plan.upgrade.cattle.io/k3os-latest label on the node and did a reboot. The label is set automatically again and everything seems to be working fine. Although, still not sure what's the root cause of the upgrade failure, I know how to recover the error at least. Thank you for your timely replies and help. You can close this issue now.


Implement constant power mode

It would be so amazing if constant power and resistance modes were added I have a reload pro and most applications I require are constant power


Add items to antiquarian

Adds some items that are otherwise unavailable anywhere else to the antiquarian for purchase/trade
Adds exgrenade, minibomb, and grenadier webbing to purchase at inflated prices!

The pipebomb is plenty enough.


Fix kwargs usage for Ruby 2.7

Summary
Ruby 2.7 displays warning without separation of positional and keyword arguments. This PR intend to fix it when calling the method driven_by.
rspec-rails-4.0.0/lib/rspec/rails/example/system_example_group.rb:100: warning: Using the last argument as keyword parameters is deprecated; maybe ** should be added to the call
rails-8aa14248a74c/actionpack/lib/action_dispatch/system_test_case.rb:167: warning: The called method `driven_by' is defined here

Hurrah an easy Ruby 2.7 fix :joy:

Thanks  💚 💙 💜 💛 ❤


Imdb mobile

Codacy reports the the duplication of the main() function, which could be moved into the the common file, but I prefer to have the whole of the top-end of the code in a single file, especially since we're planning to support AoC at some point.