Software Developer

This sample does not work.
I put in my subscriber ID and the generated key. No luck.
It said invalid subscriber id, even though it is valid.

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

ID: 0a81d6ae-75e4-aadc-d70e-4a6478a85145
Version Independent ID: 73db6138-5e6c-e8ef-96f6-1bc4c1a5b079
Content: C# Quickstart for Azure Cognitive Services, Bing Spell Check API | Microsoft Docs
Content Source: articles/cognitive-services/Bing-Spell-Check/quickstarts/csharp.md
Service: cognitive-services
GitHub Login: @v-jaswel
Microsoft Alias: v-jaswel

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

Hi Jason,
When you mention subscriber ID, are you referring to these lines in the quick start code?
// These properties are used for optional headers (see below).
// static string ClientId = "";
// static string ClientIp = "999.999.999.999";
// static string ClientLocation = "+90.0000000000000;long: 00.0000000000000;re:100.00000000000
If so, you can get a client ID as follows.

In your initial request to the Spell Check service, do not include the X-MSEdge-ClientID request header.

The Spell Check service will return an X-MSEdge-ClientID response header. You can get its value as follows.

2a. Add the statement:
using System.Linq;
2b. Add the following code after you get the HttpResponseMessage (which in the code is named "response"):
string client_id;
if (response.Headers.TryGetValues("X-MSEdge-ClientID", out IEnumerable header_values))
{
client_id = header_values.First();
Console.WriteLine("Client ID: " + client_id);
}

Specify client_id as the value of the X-MSEdge-ClientID request header for future requests to the Spell Check service.

3a. In the quick start code, you would uncomment:
// static string ClientId = "";
and change it to (for example):
static string ClientId = "2325577A61966D252A475CD760C96C03";
3b. In the quick start code, you would uncomment:
//client.DefaultRequestHeaders.Add("X-MSEdge-ClientID", ClientId);
to:
client.DefaultRequestHeaders.Add("X-MSEdge-ClientID", ClientId);
Note when you include the X-MSEdge-ClientID request header in your request to the Spell Check service, it will no longer return the X-MSEdge-ClientID response header.
For more information see:
https://docs.microsoft.com/en-us/rest/api/cognitiveservices/bing-spell-check-api-v7-reference#request-headers
Specifically, see the section "X-MSEdge-ClientID".
Let me know if I can provide any additional information.
Thank you,
Jason

@JasonSingh I have assigned this issue to the author who would like some more info from you in his comment above.

Hi Jason,
No I’m referring to the MSDN subscriber key and the keys generated for Spell Check.
I believe the clientID credentials are optional.
Looking at the code sample from
https://docs.microsoft.com/en-us/azure/cognitive-services/bing-spell-check/quickstarts/csharp
                         // I presume the MSDN Subscriber key goes here. And I fail to get authenticated.

    // NOTE: Replace this example key with a valid subscription key.
    static string key = "INSERT API KEY";

          and
                         // I presume the generated key goes in here.

        HttpClient client = new HttpClient();
        client.DefaultRequestHeaders.Add("Ocp-Apim-Subscription-Key", key);

With this approach this does not work, because it says I’m not authenticated.
From: Jason Wells<EMAIL_ADDRESS>Sent: Friday, March 9, 2018 1:42 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: JasonSingh<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Software Developer (#5533)
Hi Jason,
When you mention subscriber ID, are you referring to these lines in the quick start code?
// These properties are used for optional headers (see below).
// static string ClientId = "";
// static string ClientIp = "999.999.999.999";
// static string ClientLocation = "+90.0000000000000;long: 00.0000000000000;re:100.00000000000
If so, you can get a client ID as follows.

In your initial request to the Spell Check service, do not include the X-MSEdge-ClientID request header.
The Spell Check service will return an X-MSEdge-ClientID response header. You can get its value as follows.

2a. Add the statement:
using System.Linq;
2b. Add the following code after you get the HttpResponseMessage (which in the code is named "response"):
string client_id;
if (response.Headers.TryGetValues("X-MSEdge-ClientID", out IEnumerable header_values))
{
client_id = header_values.First();
Console.WriteLine("Client ID: " + client_id);
}

Specify client_id as the value of the X-MSEdge-ClientID request header for future requests to the Spell Check service.

3a. In the quick start code, you would uncomment:
// static string ClientId = "";
and change it to (for example):
static string ClientId = "2325577A61966D252A475CD760C96C03";
3b. In the quick start code, you would uncomment:
//client.DefaultRequestHeaders.Add("X-MSEdge-ClientID", ClientId);
to:
client.DefaultRequestHeaders.Add("X-MSEdge-ClientID", ClientId);
Note when you include the X-MSEdge-ClientID request header in your request to the Spell Check service, it will no longer return the X-MSEdge-ClientID response header.
For more information see:
https://docs.microsoft.com/en-us/rest/api/cognitiveservices/bing-spell-check-api-v7-reference#request-headershttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Frest%2Fapi%2Fcognitiveservices%2Fbing-spell-check-api-v7-reference%23request-headers&data=02|01||6c494fcb703d4551a17108d586068736|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636562284943624865&sdata=UtGaukrV9cNyRxgMOopxi%2FWU%2BsvE9Ao06RwKkq2Ih%2Fg%3D&reserved=0
Specifically, see the section "X-MSEdge-ClientID".
Let me know if I can provide any additional information.
Thank you,
Jason
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F5533%23issuecomment-371953090&data=02|01||6c494fcb703d4551a17108d586068736|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636562284943624865&sdata=TYf6MmBOooH3veE%2BN4FtrsSJ2HwNYlZ7WMxDiMnVgfI%3D&reserved=0, or mute the threadhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAQLiUo9y5m5x53dPva_cikslilF22t0Dks5tcvcMgaJpZM4SkSy8&data=02|01||6c494fcb703d4551a17108d586068736|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636562284943781130&sdata=JJ9HT23yiUc0TKERrgpfrq49j7YMUISImXi4GvgydH0%3D&reserved=0.

Hi Jason,
First I sincerely apologize for my delayed reply, as I have been working on documentation for the Build conference on May 7th.
I will check with the product team on this, but I'm not aware that you can access Cognitive Services with an MSDN subscriber key.
I've created all the access keys I've used in testing with the Azure portal. This topic describes how to create an access key using the Azure portal. If your company has an Azure subscription, they should be able to provide you with login information for the Azure portal. You might also be able to create an account there for free.
Alternately, this page lets you get a trial access key (click on "Language", and it should show Bing Spell Check API v7 and Text Analytics API).
An access key will look like this:
3c0809c1fb8c4ae7ab9353dc5bff4e90
and in fact you can use that one for testing Spell Check - however I would not recommend building a live application around it as it might go away in the future.
You're right that the client ID credentials are recommended but optional.
Please let me know if that helps. We can also set up a call if need be - I tend to be up until around midnight Pacific time.
Thank you,
Jason

#please-close

@JasonSingh We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


chore: bdd-spring-1574076839 to 0.0.1

chore: Promote bdd-spring-1574076839 to version 0.0.1

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

Needs approval from an approver in each of these files:

OWNERS

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


Update project website url

can we do https://octolinker.github.io/ instead?

Why should we use https? There are no sensitive data on that page, right?

I'm gonna merge this now. We can change it in a later version if needed 😉

No problem. It's just that I believe https should be the default for everything if possible. Here's a decent explanation of the reasoning: https://https.cio.gov/everything/


Replace tox with nox

Summary
This PR replaces tox with nox, an attractive alternative with configuration purely in Python.
Details and comments
The recent issues with tox have been well-documented (see https://github.com/Qiskit/rustworkx/pull/851 for one). nox is used by quite a few big projects, including pip. Besides the Python configuration file, it has a command-line interface similar to tox but with more options, such as tags. For example, now you can run nox -t ci to run the exact suite of builds that the CI uses.
Potential caveats to using nox:

By default, nox passes all environmental variables unlike tox. If there are ever specific variables we don't want to pass, they need to be manually overwritten (See https://github.com/wntrblm/nox/issues/253).
Possibly slower performance. I saw a comment mention this but it was a few years old. We'll see how fast the CI on this PR runs.

On the first CI run, the performance seems slightly better than tox. Almost all test builds ran 1-4 minutes faster with the exception of docs, which was a few minutes slower.
A note for migrating from tox: nox provides the tox-to-nox tool for generating a first pass noxfile.py from an existing tox.ini. The tool did not work for me in the 2022.11.21 release (it's a known issue: https://github.com/wntrblm/nox/issues/671), but the previous release worked fine. I still needed to update a lot of the file afterwards, but it's much faster than starting from scratch.

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


Blog app - Add forms

Blog app - Add forms

Created a method called current_user in your ApplicationController.

It will return the first user from the database.

Created forms to perform the following functions:

Creates a Post on behalf of the current_user.
Create a comment on behalf of the current_user

Allowed Users to add likes to Posts.

Fixed the MIT link in the README file as suggested.


Update to support Pydantic v2

FastAPI v0.100+ uses Pydantic v2.
It seems to work out-of-the-box with fastapi-users v12 but I get a deprecation warning :
python3.11/site-packages/pydantic/_internal/_config.py:261: UserWarning: Valid config keys have changed in V2:
* 'orm_mode' has been renamed to 'from_attributes'

Unfortunately, I was too fast to say it will work. That warning raises an Exception :
pydantic.errors.PydanticUserError: You must set the config attribute `from_attributes=True` to use from_orm

Released in v12.1.0 🎉


Content not flowing to next page

I have a simple page tampleate:
<html>
<head>
    <style>

        @page {
            size: a4 portrait; /*landscape*/
            margin: 2cm;
            @frame content_frame {
                left: 50pt;
                width: 512pt;
                top: 50pt;
                height: 212pt;
                -pdf-frame-border: 1;
                -pdf-keep-with-next: false;
            }
        }

    </style>
</head>
<body>

Lorem ipsum....

Just a lot more lorem ipsum. However the pdf looks like it fills the content frame, then it starts form the tom of the page and fills the smae page then, and finally it switches page to the next one.
bad_flow.pdf
See attached file for an example. Is there anything wrong with my template?

Please report the version of xhtml2pdf you are using and try the same contents in the develop branch. This project is almost dead and done, we are just trying to get a bugfixed, py3k compatible version 0.1 out. Please try to solve the issue on your own as far as possible.
See also:
https://groups.google.com/forum/#!topic/xhtml2pdf/PtG8J28Dwyk

In case someone else runs across this: I had the same problem (with versions 0.2.1 and 0.2.3), and the only thing that fixed it was to remove the "margin" property from the page selector. No idea why.

Confirm the solution from @TheMiloAnderson works for me.

Incredible, removing the margin works! 🚀
Thank you so much for this tip!

In case someone else runs across this: I had the same problem (with versions 0.2.1 and 0.2.3), and the only thing that fixed it was to remove the "margin" property from the page selector. No idea why.

Thanks, i can confirm that your solution also worked for me on version 0.2.7.


helm package --dependency-update with OCI dependencies is broken

With new Helm 3.8.0 release, the helm package --dependency-update is broken if the chart to package contains OCI dependencies.
This is the error I get:
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x38 pc=0x539b42b]

goroutine 1 [running]:
helm.sh/helm/v3/pkg/registry.(*Client).Tags(0x0, {0xc0012fa780, 0xc000bc1b78})
	helm.sh/helm/v3/pkg/registry/client.go:581 +0x12b
helm.sh/helm/v3/internal/resolver.(*Resolver).Resolve(0xc000bc1bf0, {0xc0001c8640, 0x3, 0x4}, 0x1)
	helm.sh/helm/v3/internal/resolver/resolver.go:140 +0x57d
helm.sh/helm/v3/pkg/downloader.(*Manager).resolve(0xc000984750, {0xc0001c8640, 0xc0001c8640, 0x3}, 0xc000984750)
	helm.sh/helm/v3/pkg/downloader/manager.go:236 +0x70
helm.sh/helm/v3/pkg/downloader.(*Manager).Update(0xc000984750)
	helm.sh/helm/v3/pkg/downloader/manager.go:195 +0xe5
main.newPackageCmd.func1(0xc000459180, {0xc0004e04c0, 0x1, 0x4})
	helm.sh/helm/v3/cmd/helm/package.go:99 +0x5f6
github.com/spf13/cobra.(*Command).execute(0xc000459180, {0xc0004e0480, 0x4, 0x4})
	github.com/spf13/cobra@v1.3.0/command.go:856 +0x60e
github.com/spf13/cobra.(*Command).ExecuteC(0xc0001aea00)
	github.com/spf13/cobra@v1.3.0/command.go:974 +0x3bc
github.com/spf13/cobra.(*Command).Execute(...)
	github.com/spf13/cobra@v1.3.0/command.go:902
main.main()
	helm.sh/helm/v3/cmd/helm/helm.go:83 +0x234

I've been checking the code and it seems the problem is that no RegistryClient is set when creating the downloader.Manager.
I will try to send a PR with a fix

This should be fixed with the merger of https://github.com/helm/helm/pull/10609


Countdown time, only counting down seconds

When I say "Ask paper signals to set a timer for 60 seconds," all is well. It counts down. Same goes for "120 seconds" ("one-twenty", not "one hundred twenty"). However if I say "Ask paper signals to set a timer for one minute" it does not work. The servo wobbles back and forth, as it does when you first set a timer, but does not follow through on the countdown. I've tried various amounts of time in both seconds and minutes, and whatever the variation, the problem still persists. Yes to seconds, no to minutes. The issue also seems to exist when asking for longer countdowns, such as an hour, or to a particular time or date. I recognize that for some of these longer countdowns I may not be aware that the servo is moving since I'd imagine it's moving so slowly.

Thanks for finding this, @max-epstein - we need to implement handling for other time units besides seconds. Hope to have a fix today or tomorrow.

@nicolehe I implemented crude conversion for timer in my PR.

HI @max-epstein, this is now fixed with #19, as well as some backend changes.
@Pomezny, we made some changes to the backend (and a small change to the Arduino code in #19) to make sure it works with the voice interaction.


Complete queryset fields

Would it be possible to add completion for queryset field arguments?
For example: typing SomeModel.objects.filter( should complete the fields of the SomeModel model.
This should also include lookups like id__in, if possible.

@ObserverOfTime do you still use this plugin? Do you still get completions for querysets?

No, it looks like it doesn't work anymore.


How does this exception get sent without calling captureException?

I have the following code
from raven import Client

SENTRY_DSN = "..."

if __name__ == "__main__":
    Client(SENTRY_DSN)
    1 / 0

What happens is that the exception automatically gets sent to sentry (despite me not explicitly calling captureException)
This is pleasing for me, as I don't need to do much other than initialise the client in my __main__ block (or a method it calls).
I was just wondering whether this is 'documented' behaviour, i.e. should I rely on this?

The sentry client installs a hook by default via sys.excepthook

Note: This is not related to Sentry per se, but be aware that Python has a bug where sys.excepthook is not read correctly when using multithreaded programs. It was reported in 2005 and I can still reproduce it with Python 3.7 today.
I had an issue with a multithreaded program that wasn’t reporting exceptions to Sentry and I suspect this is why.


Add your name to the SUPPORTERS.md

This would be an easy task for people who want to get started with contributing to open-source. This is how it's done:

From the toolbar in the top right of your screen, click Fork. This will create a fork of the project.
In your own fork, click on SUPPORTERS.md, and from the buttons in the top right of the file preview, click on the pencil button.
Add your name and GitHub profile to the end of the list as follows:

In a new line, write down: - [Name Surename](https://github.com/<your-handle>)
At the bottom of the page, select "Create a new branch for this commit and start a pull request." and click "Commit Changes".

Finalize creating the merge request.

!assign

I want to do this

I want to do this

Feel free to open a PR, @itsmekhemraj! 🙏

@aerabi can you check my pr?


MAID-2228 feat/Request permissions for shared services container

r? @ustulation
(maidsafe_highfive has picked a reviewer for you, use r? to override)


ERROR Failed to access: 'http://namecheck.fontdata.com/?q=AdventPro'.

e.g. here: http://<IP_ADDRESS>/report/d77de85c-eaa2-4738-9df6-ffc4a96b573d
I just scaled the dashboard to check ~ 5000 fonts in ~ 10 minutes. That's ~ 8 fonts per second. Most probably not a load acceptable for http://namecheck.fontdata.com. Now I feel bad, we had around 1600 workers send requests at the same time to that service. I also ran these collection wide checks multiple times over the last days, but I think the last one, once it was properly scaled, broke that service.
I won't run the full collection test anymore unless we have a solution for this and other external requests.

does it make sense for us to slowly build our own cache of namecheck data and then from time to time update it so that we avoid DoS'ing it?

We can inject a function for such requests, and the default CLI will just do a http request, but the dashboard workers can inject an internal source that acts like a caching proxy.
One problem with tests of this kind is also that they are not idempotent. Ideal would be to avoid external state, that we can't control i.e. on a per-docker-image base.

to slowly build our own cache

Yeah, we should probably have something like this

in http://namecheck.fontdata.com/about/ it says

Developers / API
Even this tool is in an very early stage there is an API provided to programtically access the data.

I think we should/could collaborate with the maintainer.


Ionic 2 - EXCEPTION: TypeError: undefined is not a function

Short description of the problem:
I finished building my IONIC2 app, the iOS version works fine; however, when I try to run the same page on the Android Simulator an error, I was able to get it when I inspected the running app on CHROME:
       EXCEPTION: TypeError: undefined is not a function
      browser_adapter.js:77 EXCEPTION: TypeError: undefined is not a function
      browser_adapter.js:77 STACKTRACE:
      browser_adapter.js:77 TypeError: undefined is not a function
      at new DishesComponent (file:///android_asset/www/build/js/app.bundle.js:1746:49)
      at AppView._View_MenuPage2.createInternal (MenuPage.template.js:354:31)
      at AppView.create (file:///android_asset/www/build/js/app.bundle.js:33434:21)
      at TemplateRef_.createEmbeddedView (file:///android_asset/www/build/js/app.bundle.js:33335:14)
          at ViewContainerRef_.createEmbeddedView (file:///android_asset/www/build/js/app.bundle.js:33844:35)
          at NgFor._bulkInsert (file:///android_asset/www/build/js/app.bundle.js:4250:50)
          at NgFor._applyChanges (file:///android_asset/www/build/js/app.bundle.js:4206:14)
          at NgFor.ngDoCheck (file:///android_asset/www/build/js/app.bundle.js:4188:22)
          at AppView._View_MenuPage0.detectChangesInternal (MenuPage.template.js:225:42)
          at AppView.detectChanges (file:///android_asset/www/build/js/app.bundle.js:33581:14)

Which Ionic Version? 1.x or 2.x
2.0
Unfortunately, I can't make a plunker because, it's a finished app and the page having the issue is really big.
For the records, the issue appears ONLY on the android emulator, when I run it on Chrome (MAC), everything is working fine. To emulate I am using Android 5.1
Your system information:
Cordova CLI: 6.3.0
Ionic Framework Version: 2.0.0-beta.11
Ionic CLI Version: 2.0.0-beta.32
Ionic App Lib Version: 2.0.0-beta.18
ios-deploy version: 1.8.6
ios-sim version: 5.0.8
OS: Mac OS X El Capitan
Node Version: v4.4.7
Xcode version: Xcode 7.3.1 Build version 7D1014

Hello, thanks for opening an issue with us! Does this also happen on a real Android device?

I haven't tried that on the real device as I don't have one, but I guess it would be the same as I think the emulator renders elements the same way the device does (correct me if I am wrong).

Hmm unfortunately I cannot seem to reproduce this issue with any of my test apps. If you could come up with a minimal reproducible example repo that would be great! From looking at the error again it looks like there is an issue in your MenuPage component. Thanks!

@HamzaLJ  This seems you trying to call function from template which is missing

Hello @HamzaLJ after reviewing this issue some more it does seem to be an issue coming from ionic but instead from your code. Because of this I am going to close this issue for now, but feel free to comment back if you find otherwise. Thanks for using Ionic!

@jgw96 I found the line that triggers the issue, but I can't solve it!
I'll share the lines that causing the issue:
page1.html
 <ion-slides id="dishes-content" class="dishes-slides" (ionWillChange)="onCategoryChanged($event.activeIndex)">
     <ion-slide *ngFor="let category of categories; let i = index">

     //When I remove the below line, the issue disappears!!
    //This is a directive imported from finalMenu
    <dishes [categoryValue]="category" [cart]="cart"></dishes>

  </ion-slide>
 </ion-slides>

page1.js
@Component({
 templateUrl: 'build/pages/page1/page1.html',
 directives: [DishesComponent],
 queries: {
   sliders: new ViewChildren(Slides)
 }
 })

finalmenu.js
@Component({
 selector: 'dishes',
 inputs: ['categoryValue', 'cart'],
 templateUrl: 'build/pages/finalmenu/finalmenu.html'
   })

Hmm, it looks like its definitely an issue with your custom component. Unfortunately since this is not an Ionic issue there is not much I can do but you can always ask around on our forum and someone on there should be able to help. Thanks for using Ionic!

Hi, I had the same problem.
Resolved it by looking in the component template.
Before:
@Component({   templateUrl: 'level.html' })
After:
@Component({   templateUrl: 'edit-level.html' })
The templateUrl was the same as the page it comes from.


Move from 'monad-logger' to 'katip'

For now I see that it's not that convenient to move to katip without introducing lenses to AppEnv first. katip library requires to have some instances for our App monad and those instances require changing value of AppEnv.  This instance works only if we store fields of LoggerConfig directly in AppEnv which seems a little awkward to me. And if you add one extra nested layer to those instances above, code will be much uglier...  Though this might be the low cost.

Okay, there's one difficulty with migration to katip. I want to use NoLoggingT in mock tests but library doesn't export runNoLoggingT unwrapper 😞 I've opened issue for this:

https://github.com/Soostone/katip/issues/89

PR with fixes was merged to katip repository. But new version is not released. I will reopen this issue just as a reminder to not forget to update repo.


Retain search context between login/register pages

redirectTo context used to get lost when moving to register page
now the joining link can be used with registering

Works well! 👍 One side-effect I found: If you have some existing page in your url when registering (/overview or /roadmaps/  etc..), you will skip the It seems you don’t have any projects yet... -page. Might not be a problem because there's not really many reasons for a new user to have something extra in their url except maybe when using shared computer. I just wanted to point this out now so it will not come as a surprise later on.
Getting redirected to this page as a first-time user might be really confusing though if it were to happen

Would it mess things up or be really burdensome to implement if anything else but join project -links would redirect to the It seems you don’t have any projects yet... -page? If it's quick and won't cause more side-effects, it might be worth implementing.

One side-effect I found: If you have some existing page in your url when registering (/overview or /roadmaps/ etc..), you will skip the It seems you don’t have any projects yet... -page

Great feedback! Added the following:

anything else but join project -links would redirect to the It seems you don’t have any projects yet... -page


Stop tracking config/secrets.yml ?

I'm writing tests for bitly, and need to test the test with a Bitly client. I want to leave in some pieces of the code I'm writing, as a best practice example - specifically how to initialize the client with a password.
I usually check in config/secrets.yml, but use the dotenv gem - but that might be too many steps for a beginner coder. Instead, I want to stop tracking config/secrets and add it to .gitignore I can also create a config/secrets.yml.sample that does get checked in.
Is this strategy ok by you guys?

@schaui6 @akanshmurthy

@siruguri sounds good to me

Sounds good.

Thanks.
Typed on a phone
On Nov 2, 2016 1:49 PM, "Akansh Murthy"<EMAIL_ADDRESS>wrote:

Sounds good.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/akanshmurthy/railsbridge-hackathon/issues/13#issuecomment-257994508,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACezbC-WCtPxVlrnnN9oFMPAf5vIBshks5q6PdRgaJpZM4Knkak
.


Ore Dictionary Missing/not utilized

Currently playing FTB Sky Adventures. I updated Roost to 1.3.0. Attempting to use RFTools Storage Control Screen module to setup a display for counting the individual chickens in the roosts. Roost doesn't seem to utilize the chickens as different oredicts and are instead all generic chickens so RFTools (and other mods) treat them all as the same item. The only exception I've found that will treat them differently is EnderIO due to the usage of NBT data. When attempting to move/count/manage chickens as items, ignoring NBT with EnderIO allows you to use 1 chicken cover all chickens or don't ingore NBT to count individual chickens. Is it possible to setup Ore Dictionaries to allow for each chicken item to be counted/managed individual with/without ore dictionaries?

The screen shot shows what I'm referencing as the RFTools Storage Control Screen Module which is accessing the RFTools Storage Scanner to search for the specific chickens. Instead of showing the individual chicken counts in all roosts for each type of chicken, it shows the total count.

For 1.13, I plan to make each chicken its own item, and use Tags to group them, rather than using nbt data to determine the chicken breed. I don't think I can make this change in 1.12, as I don't think there is a way to migrate nbt-based items in all inventories to non-nbt based items.


pdf viewer add a formly-field

[ ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[x ] support request

You think is possible substitute <div>{{ filename }} </div>  with a
formly-field  inside the template constructed dynamically.
https://ngx-formly-ui-bootstrap-htuh7z.stackblitz.io

it's better to avoid asking a lot of questions, that's something you should deal with it based on your knowledge else use stackoverflow.
sorry :pray:

@aitboudad ok, thank you for your time. 👍
Eventually I will use stackoverflow.  👍
My objective is to find faster and good solution.

I elaborate a solution
https://stackblitz.com/edit/ngx-formly-ui-bootstrap-mcvzfp
one main template and two sub-templates

Thanks!


Sort, skip, limit not working

Please help me!

Me too!

const news = Meteor.collection('news').find({}, { sort: { date: -1 }});


Autodesk AutoCAD Crack (v2025.2) Keygen [XFORCE] Key Activation

Autodesk AutoCAD Crack v2025.2 Keygen [XFORCE] Activation Autodesk AutoCAD Crack is a computer-aided design (CAD) software widely used in the architecture, engineering, and construction industries to create 2D and 3D models and drawings.
>➤ ►👉 Do𝚠nlo𝚊d Autodesk AutoCAD Crack (Latest)
AutoCAD
AutoCAD free
AutoCAD zip file
AutoCAD Keygen
AutoCAD software
AutoCAD free download 2024
AutoCAD also has features for
adding layers
styles
templates to drawings
which help to standardize and streamline the drawing process!

cvc

Download Updated link
Download Updated link
Download Updated link

Download-Setup
Full-Setup
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.


What's the difference between the status pinning and queued

The spec mentions that both queued and pending represent a pending state where the pinning service has yet to fetch the CID from the network.
Is there an actual difference between the two states? If so, what is it?

queued is passive no-op: it adds pin job to the queue but does not consume any resources (yet).
pinning is active: pinning service is spending resources on in, trying to find providers for all involved CIDs, connect to these providers and download data from them.

@2color does this make sense?
I'm happy to add something to  #section/The-pin-lifecycle/Checking-status-of-in-progress-pinning if you propose a PR :)


(#67) PCC

As per #67:

The Pessimistic Class Cohesion (PCC) metric quantifies the
maximum extent to which any two methods of a class have a strong
dependency where one writes to an attribute and the other reads from
the same attribute. The maximum number of methods that can be reached
from any given method according to this definition in proportion to
the total amount of methods is taken as the value for PCC.
PCC is given a value of 0 for the exceptional case when the class has
only one method.

Job #173 is now in scope, role is REV

This pull request #173 is assigned to @amihaiemil/z, here is why. The budget is 15 minutes, see §4. Please, read §27 and when you decide to accept the changes, inform @yegor256/z (the architect) right in this ticket. If you decide that this PR should not be accepted ever, also inform the architect.

@llorllale Can you fix the build and conflicts? :D

@amihaiemil fixed conflicts, please review

Codecov Report

Merging #173 into master will increase coverage by 0.41%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##             master    #173      +/-   ##
===========================================
+ Coverage     74.79%   75.2%   +0.41%
- Complexity      133     136       +3
===========================================
  Files            27      27
  Lines           964     968       +4
  Branches         67      68       +1
===========================================
+ Hits            721     728       +7
+ Misses          213     212       -1
+ Partials         30      28       -2

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/org/jpeek/App.java
94.11% <100%> (+0.24%)
14 <0> (+1)
:arrow_up:

src/main/java/org/jpeek/Index.java
96.42% <0%> (+5.35%)
7% <0%> (+2%)
:arrow_up:

Continue to review full report at Codecov.

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

@amihaiemil added a puzzle addressing your concern for the procedural code in App.analyze(). I addressed your other concern by replying to your comment.
Let me know

@llorllale nah, remove that puzzle for refactoring. Can you report a ticket/bug instead? So we can discuss with the ARCH there. It is a bigger topic, I would like all the Java code to be refactored, not just that method... we had this issue in another PR as well.

@amihaiemil OK, I just git reset HEAD~ and force-pushed... code should be in same state as it was before

@amihaiemil since I don't know what exactly you would like changed, I suggest you open that bug

@rultor good to merge

@rultor good to merge

@amihaiemil Thanks for your request. @yegor256 Please confirm this.

ping @yegor256

@llorllale there is now a conflict with MetricsTest.java, can you fix it?

@amihaiemil fixed conflicts, please review

@yegor256 ping

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@llorllale @yegor256 Oops, I failed. You can see the full log here (spent 2min)
+ as_root=false
+ mkdir -p /home/rultor/.ssh
+ echo -e 'Host github.com\n\tStrictHostKeyChecking no\n'
+ chmod 600 /home/rultor/.ssh/config
+ git clone git@github.com:yegor256/jpeek.git repo
Cloning into 'repo'...
+ cd repo
+ git config user.email<EMAIL_ADDRESS>+ git config user.name rultor
+ '[' -z 'export '\''MAVEN_OPTS=-XX:MaxPermSize=256m -Xmx1g'\''' ']'
+ cd ..
+ cat
+ '[' false = true ']'
+ cat
+ chmod a+x entry.sh
+ cat
+ echo 'export '\''MAVEN_OPTS=-XX:MaxPermSize=256m -Xmx1g'\''' ';' 'export '\''JAVA_OPTS=-XX:MaxPermSize=256m -Xmx1g'\''' ';' 'export '\''fork=git@github.com:llorllale/jpeek.git'\''' ';' 'export '\''fork_branch=67'\''' ';' 'export '\''head=git@github.com:yegor256/jpeek.git'\''' ';' 'export '\''head_branch=master'\''' ';' 'export '\''pull_id=173'\''' ';' 'export '\''pull_title=(#67) PCC'\''' ';' 'export '\''author=yegor256'\''' ';' 'sudo gem install --no-rdoc --no-ri pdd' ';' 'sudo gem install --no-rdoc --no-ri xcop' ';' 'mvn clean install -Pqulice --errors --settings ../settings.xml' ';' 'mvn clean site -Psite --errors --settings ../settings.xml' ';' 'mvn clean' ';' 'pdd --source=$(pwd) --verbose --file=/dev/null' ';'
+ rm -rf .gpg
+ cd repo
+ git remote add fork git@github.com:llorllale/jpeek.git
+ git remote update
Fetching origin
Fetching fork
From github.com:llorllale/jpeek
 * [new branch]      106        -> fork/106
 * [new branch]      111        -> fork/111
 * [new branch]      118        -> fork/118
 * [new branch]      135        -> fork/135
 * [new branch]      139        -> fork/139
 * [new branch]      67         -> fork/67
 * [new branch]      93         -> fork/93
 * [new branch]      gh-pages   -> fork/gh-pages
 * [new branch]      master     -> fork/master
+ args=
+ '[' default == default ']'
+ args=' --ff'
+ '[' default == no ']'
+ '[' default == only ']'
+ export BRANCH=__rultor
+ BRANCH=__rultor
++ git show-branch __rultor
++ wc -l
+ '[' 0 -gt 0 ']'
+ git checkout -B __rultor fork/67
Switched to a new branch '__rultor'
Branch __rultor set up to track remote branch 67 from fork.
+ git checkout -B master origin/master
Switched to and reset branch 'master'
Branch master set up to track remote branch master from origin.
Your branch is up-to-date with 'origin/master'.
+ '[' false == true ']'
+ '[' false == true ']'
+ git merge --ff __rultor
Auto-merging src/test/java/org/jpeek/MetricsTest.java
CONFLICT (content): Merge conflict in src/test/java/org/jpeek/MetricsTest.java
Auto-merging src/main/java/org/jpeek/App.java
CONFLICT (content): Merge conflict in src/main/java/org/jpeek/App.java
Automatic merge failed; fix conflicts and then commit the result.
'cid' file is absent, container wasn't started correctly

@llorllale there are some conflicts again. Probably another PR has been merged again and caused them

@amihaiemil fixed conflicts, please review

@yegor256 conflicts solved here, we can merge again

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 18min)

@ypshenychka/z please review this job, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

@llorllale According to our QA Rules:

Pull request title explains the problem it is fixing.

Please improve the title to be more informative.

@ypshenychka We did find quite a big bug here (and also in 2 other PRs) -- because of the overall architecture which is procedural, the code grows and grows with if statements and maps of parameters. See my comment above and the opened issue.
But yes, I'll try to find mode bugs in the future :D

@amihaiemil Thank you :)

@ypshenychka what about the verdict here? :)

@amihaiemil @llorllale Please tag me when you fix or agree on something I wrote, so that I was aware.

@0crat Quality acceptable

QA review completed: +15 points just awarded to @ypshenychka/z, total is +300

Order was finished, quality was "acceptable": +15 points just awarded to @amihaiemil/z, total is +775

Payment to ARC for a closed pull request, as in §28: +15 points just awarded to @yegor256/z, total is +10319


Support equipment that doesn't degrade the pouch

However, if a player is wearing a Runecraft cape, Max cape, or any Max cape variant, then their pouches will not degrade upon filling them (though they will not automatically be repaired). Additionally, in Guardians of the Rift, wielding an Abyssal lantern lit with Redwood logs prevents pouch degradation, though it also requires 90 Firemaking.

OSRS Wiki

The Abyssal Lantern seems to only work when inside of the actual minigame area (passing the barrier). I can't test this myself. I forgot to ask if they can see whether or not the status of the game (pre-game before guardian is active, active with guardian power, or post game) affects the lanterns ability.

Additional information

Lantern effect works equipped or inside inventory
Lantern effect only works during an active game and the rift is active (Guardian's power overlay is visible)

The way it works is odd. The anti-decay effect will go live starting from when the game first becomes active and last until you leave

There was an issue with the way I was tracking where the abyssal lantern was stored (inventory or equipment). I implemented a solution in #33 but I won't push it until I tested all the conditions to avoid pushing a broken solution again. Sorry.


Animated as option!

If you use this with webp file to put metadata and send the animated sticker works great, but if you check the sticker message it is false to isAnimated.
animated: boolean

Do you mean this


defaultSortName and SortName do not seem to work properly

Hello,
I am new to developing with react and webpack and am porting an existing site I have to react and react-bootstrap-table is great. However, I want my table to be sorted by one of the fields by default the first time it loads and using the defaultSortName option does not seem to take care of this.  I'm able to feed it data and get a lot of the features to work, but can't get default sorting by a specified field.
My render function contains something like:
<BootstrapTable data={data} 	                    condensed={true} 	                    hover={true} 	                   options={{defaultSortName:'Status'}}>     <TableHeaderColumn dataSort={true} dataField='Name'>name</TableHeaderColumn>     <TableHeaderColumn isKey={true} dataSort={true}                dataField='Status'>status</TableHeaderColumn> </BootstrapTable>
Has anyone actually got this working?

HI @joshadkins09  you need assign defaultSortOrder also.


docker compose v2 migration

v2 uses docker compose instead of docker-compose.
Ubuntu gha runners now ship with v2 only, see https://github.com/actions/runner-images/issues/9692
closes https://issues.redhat.com/browse/RHOAIENG-11053

CI failures unrelated, merging as the smoke test workflow succeeded.


What happened?
I have been trying to create a new pattern using the create_pattern pattern #559 but ChatGPT does not understand the instruktions fully, it only seem to understand the first step. It writes a proper IDENTITY and PURPOSE from my input, but then just proceeds to copy the exact steps and output instructions from the "create_pattern" pattern.
Example
echo "create a pattern that writes great cat poems in the form of haiku's" | fabric -sp create_pattern
Result
# IDENTITY and PURPOSE

You are an AI assistant tasked with the creative and intricate role of composing cat-themed poems in the haiku format. Your primary responsibility involves tapping into the essence of feline grace, mystery, and charm to craft verses that resonate with cat lovers and poetry enthusiasts alike. This requires a deep understanding of the haiku structure, which traditionally consists of three lines with a syllable pattern of 5-7-5, along with a keen sense of observation and appreciation for the subtleties of cat behavior and their natural environment. Your goal is to encapsulate the beauty, agility, and enigmatic qualities of cats in a succinct and evocative manner that captures the imagination of your audience.

Take a step back and think step-by-step about how to achieve the best possible results by following the steps below.

# STEPS

- Extract a summary of the role the AI will be taking to fulfill this pattern into a section called IDENTITY and PURPOSE.

- Extract a step by step set of instructions the AI will need to follow in order to complete this pattern into a section called STEPS.

- Analyze the prompt to determine what format the output should be in.

- Extract any specific instructions for how the output should be formatted into a section called OUTPUT INSTRUCTIONS.

- Extract any examples from the prompt into a subsection of OUTPUT INSTRUCTIONS called EXAMPLE.

# OUTPUT INSTRUCTIONS

- Only output Markdown.

- All sections should be Heading level 1

- Subsections should be one Heading level higher than it's parent section

- All bullets should have their own paragraph

- Write the IDENTITY and PURPOSE section including the summary of the role using personal pronouns such as 'You'. Be sure to be extremely detailed in explaining the role. Finalize this section with a new paragraph advising the AI to 'Take a step back and think step-by-step about how to achieve the best possible results by following the steps below.'.

- Write the STEPS bullets from the prompt

- Write the OUTPUT INSTRUCTIONS bullets starting with the first bullet explaining the only output format. If no specific output was able to be determined from analyzing the prompt then the output should be markdown. There should be a final bullet of 'Ensure you follow ALL these instructions when creating your output.'. Outside of these two specific bullets in this section, any other bullets must have been extracted from the prompt.

- Ensure you follow ALL these instructions when creating your output.

# INPUT

INPUT:

I've tried with both GPT-4-turbo and GPT-4o without success.
Version check

[X] Yes I was.

Relevant log output
No response
Relevant screenshots (optional)
No response

Intresting. For llama it seems to work properly with the same input.
Input
echo "create a pattern that writes great cat poems in the form of haiku's" | fabric -sp create_pattern
Output
**IDENTITY and PURPOSE**

You are an AI poet, tasked with crafting feline masterpieces in the traditional Japanese haiku format. Your role is to capture the essence of cats' curious nature, playful personalities, and endearing quirks through three short lines. Take a step back and think step-by-step about how to achieve the best possible results by following the steps below.

**STEPS**

* Observe the cat's behavior and mannerisms
* Identify key characteristics that define their feline nature (e.g., curiosity, agility, independence)
* Use sensory language to describe their physical appearance, movements, and habits
* Structure your haiku into three lines, with a 5-7-5 syllable pattern

**OUTPUT INSTRUCTIONS**

* Write each haiku in traditional Japanese haiku format: three lines with a specific number of syllables (5-7-5)
* Use sensory language to describe the cat's appearance, movements, and habits
* Ensure that your haikus capture the unique personality and characteristics of cats

**EXAMPLE**

Whiskers twitching fast,
Moonlight dances on her back,
Midnight's gentle queen.

**INPUT**

Claud 3.5 also seem to understand it fully. I'll close this bug again, since it's not anything wrong with the pattern it self. Even though there might be ways to make OpenAI understand the instructions better it might just as well self resolve as ChatGPT's models gets better


pkg: test ResourceDeletionProtection valid webhook

In this commit, we add unit tests for the ValidatingAdmission validation webhook specifically for the resource deletion protection feature. These tests ensure that the webhook correctly handles various admission scenarios:

Tests the behavior when decoding the request object fails, verifying that admission is denied with an appropriate error message.
Ensures that resources labeled with DeletionProtectionAlways are denied deletion, validating that the error message is returned.
Confirms that resources without the deletion protection label are allowed to be deleted without errors.
Validates that resources with other labels are not impacted and are allowed to be deleted.
Confirms that non-delete operations (e.g., create) are allowed by default.

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

/ok-to-test


[Extensions] ExtensionAdmin registerHost should accept hostnames and urls

ExtensionAdmin registerHost() should accept either a full URL or a hostname. (Since we are already storing the hostname in the remote/service provider, we should be able to just pass it as is)
Remove the "_exists" method from Remote and Service Extension Providers. Don't need the extra HEAD request, can fail on GET. The error looks like: "Error: Failed to get extension manifest at someUrl.com/manifest.json: Not found"

https://github.com/iTwin/itwinjs-backlog/issues/60
https://github.com/iTwin/itwinjs-backlog/issues/61

Changes to common/api/appui-layout-react.api.md should be reverted since this PR did not touch files in appui-layout-react. rush extraxt-api sometimes incorrectly does this without you knowing it. You can also go to ui\appui-layout\react and run npm run clean && npm run build && npm run extract-api.

Thanks @bsteinbk, I spent too long trying to figure out why that was happening. rush extract-api must've done it... I'll be more careful next time

is this needed in 3.2.1?

is this needed in 3.2.1?

I don't think it's needed in the patch but would go into 3.3. I don't see anything urgent.

@pmconne mind approving?

spoke with @johnnyd710 , we do need this in a patch to get extensions to properly load in DR. specifically without this pr, HEAD verb is not allowed in the blob storage of the Extension Service
@mergifyio backport release/3.2.x

@Mergifyio backport release/3.2.x


Collision between project/context button class and builtin classes

Describe the bug
To Reproduce

Create a todo: (A) Test +lorem +ipsum +input

Expected behavior

Side bar project buttons should be about the same size.

Actual behavior

input button is full width.

Desktop:

Windows 10, sleek 1.0.4, from GitHub.

Notes

This is apparently because each button is given an HTML class name of "button {project_name}".
If {project_name} is the same as an already existing class (like those provided by electron*), it inherits its style.
Proposed solution: add a prefix/suffix to the class derived from the project name (or drop it, since all the buttons are the same so button class should be enough).

* See for example under resources/app.asar/node_modules/bulma/sass/elements

Another good one. Fix will be part of the final v1.0.5

@9z0 has been released with https://github.com/ransome1/sleek/releases/tag/v1.0.5.
Can you please close this issue, if it works as expected?


Idea Title
Intersection of Two linked list
Idea Description
Given the heads of two singly linked-lists headA and headB, return the node at which the two lists intersect. If the two linked lists have no intersection at all, return null.
Input: listA = [4,1,8,4,5], listB = [5,6,1,8,4,5]

Output: Intersected at '8'
Potential Benefits

[ ] Improves user experience
[X] Enhances learning
[X] Adds a new feature
[ ] Improves code quality
[ ] Other

Implementation Suggestions (Optional)
two options:

using unordered set
using traversal of linked list to check for intersection

please assign this to me!


fix importtohar when servers is not defined

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


Block detach distance threshold is too large

Expected Behavior
When two blocks are connected, and you drag the bottom one down to detach, I expect to be able to release the block after dragging it beyond the edge of the insertion marker.
Actual Behavior
In order to detach the block, I have to drag past the shadow insertion marker, about twice its height. This gif shows dragging the block almost this far, and it still snaps back:

Steps to Reproduce
Attempt to disconnect two blocks by dragging the bottom one downward.
Operating System and Browser
Mac OS 10.11.6 Chrome 57

Code pointer:  https://github.com/LLK/scratch-blocks/blob/develop/core/block_svg.js#L1149
I would play around with this when the block is in the middle of a stack to make sure you want to change it.  I think it's protecting against jumping around as the blocks shift to accommodate the insertion marker*.
*not a shadow.  Shadow has a specific meaning in Blockly.

Original issue with video: https://github.com/LLK/scratch-blocks/issues/290
you might be able to come up with a more clever solution :) e.g., maybe the size could be minimized depending on the size of the insertion marker (smaller for vertical than horizontal), or maybe you could only apply this rule when the insertion marker is in the middle of a stack

Related: this transitions almost* directly from before the stack to wrapping the second block in the stack.
*almost because when it does wrap the whole stack, it does it for a frame or two and it's a pretty nasty flicker, but finding that space intentionally is almost impossible at normal zoom.


Event for changed selected hotbar slot

Currently there is Player#getHeldItem(...). But I would like to know which slot is selected, the index from 0 to 8. Ideally also an event that is fired when the selection changes.

@randombyte-developer

But I would like to know which slot is selected, the index from 0 to 8.

Mind explaining why you need this feature? What is the usecase?

@ST-DDT My plugin should detect when the user is scrolling(and so changing the selected slot).

@ST-DDT My plugin should detect when the user is scrolling(and so changing the selected slot).

I can understand the event part of this request. But I don't get why you need the actual slot (index). Isn't the Slot instance enough?

The direction of scrolling defines whether a value should increment or decrement. And how many slots are scrolled through defines how much the value gets de/incremented.

@ST-DDT So is this implemented: https://github.com/SpongePowered/SpongeAPI/blob/bleeding/src/main/java/org/spongepowered/api/item/inventory/entity/Hotbar.java ? That's what it need.
An event fired when the selected slot changes would be nice. Otherwise I would constantly recheck the selected index. I'll edit the title of this issue.

This would also be useful for a plugin where spells could be casted by using the numbers 1234 on the keyboard.
Resetting the hotbar position to 9, then listening for when it's set to 1234

@ryantheleach Actually I wanted to use instead of scrolling another method of control: Using the numpad numbers 2, 4, 6, 8 for down, left, right, up. But I didn't find a packet that sets the selected slot. So I thought it was impossible. Do you know if the implementation uses a workaround?

http://wiki.vg/Protocol#Held_Item_Change_.28clientbound.29

ChangeInventoryEvent.Held exists and works afaik

@Faithicaio That's right but it's not the same as "change selected slot". The held event gets only fired when the actual item you are holding changes. If you don't have anything in your hotbar the held event is useless. I had to workaround by filling the player's hotbar with some garbage items to detect scrolling.


[Android] Support SNPE SDK v2

Add SNPE SDK v2 support:

Let build-nnstreamer-android.sh copy proper files from SNPE SDK
Let ndk build with tensor_filter_snpe.cc or ..._v1.cc with proper headers and flags
Link libcdsprpc.so
REF: https://source.android.com/docs/core/permissions/namespaces_libraries#specify-native-library-dependencies

Proper android app settings:

Recent AGP requires useLegacyPackaging = true in build.gradle
REF: https://developer.android.com/build/releases/past-releases/agp-4-2-0-release-notes#compress-native-libs-dsl

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #536. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.


Overview of changes

This overview is not readable (white on yellow), please see encl. screenshot.

Thank you, this is now fixed


feature: 提供 2.0 Agent & Proxy 安装包制作能力 (closed #1176)

Codecov Report

Merging #1343 (e70f1cd) into V2.2.32-rc (1b7691d) will increase coverage by 0.18%.
The diff coverage is 91.95%.

@@              Coverage Diff               @@
##           V2.2.32-rc    #1343      +/-   ##
==============================================
+ Coverage       73.68%   73.87%   +0.18%
==============================================
  Files             364      368       +4
  Lines           24703    24959     +256
==============================================
+ Hits            18203    18439     +236
- Misses           6500     6520      +20

Impacted Files
Coverage Δ

apps/backend/plugin/views.py
43.02% <0.00%> (ø)

apps/node_man/constants.py
96.41% <85.71%> (-0.43%)
:arrow_down:

apps/backend/agent/artifact_builder/proxy.py
88.88% <88.88%> (ø)

apps/backend/agent/artifact_builder/base.py
93.03% <93.03%> (ø)

apps/backend/agent/artifact_builder/__init__.py
100.00% <100.00%> (ø)

apps/backend/agent/artifact_builder/agent.py
100.00% <100.00%> (ø)

apps/backend/exceptions.py
100.00% <100.00%> (ø)

apps/utils/files.py
88.31% <100.00%> (+0.64%)
:arrow_up:

config/default.py
82.81% <100.00%> (+0.06%)
:arrow_up:

env/__init__.py
100.00% <100.00%> (ø)

... and 1 more

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


[Synapse] - Mark required properties as required.

Known issue is monitoring service SparkJob, but we should also do for others

@MyronFanQiu , could you please help us create a label for Synapse?


devmapper: Drop devices lock before returning from function

cleanupDeleted() takes devices.Lock() but does not drop it if there are
no deleted devices. Hence docker deadlocks if one is using deferred
device deletion feature. (--storage-opt dm.use_deferred_deletion=true).
Fix it. Drop the lock before returning.
Signed-off-by: Vivek Goyal<EMAIL_ADDRESS>

CC @LK4D4 @vbatts @calavera
This single line fix will need to go in 1.9. PTAL.

unit test?

@cpuguy83 Right now there are unit tests for deferred device deletion feature. I can look into adding some though.

@rhvgoyal In this case, a test just on this function would suffice, but also +1 to @LK4D4 there are some greater locking issues here.

@cpuguy83 ok, I am trying to write a unit test. Is there a way to figure out if lock is currently held or not. Currently I am taking the lock after the call to the function and if lock has been released test will pass otherwise it will hang. But hanging will stop execution of other tests too.
So there has to be a better way to do it. Last time I looked, I could not find a way to check if a lock is locked or not without blocking on the lock.

@rhvgoyal spawn in goroutine and wait for some amount of time in select on some chan.

@LK4D4 sounds good. Will do.

@cpuguy83 @LK4D4 I added a unit test case for locking issue. PTAL.

@cpuguy83 @LK4D4 Took care of review comments. PTAL.

LGTM, thanks!

@vbatts @cpuguy83 PTAL. I am kind of stuck because of this deferred device deletion is not working at all and till then I can't enable it in fedora.

Sorry, was verifying test fails on master.
LGTM

ping @tiborvass for 1.9

@tiborvass ping. Can you pull this fix in 1.9.


[harfbuzz] update Harfbuzz to 2.3.1

I am seeing new build failures on the following ports with this PR. Let me know if you need build log.
Processing x64-windows                    860 vs 858
     qt5-imageformats                                            Pass vs Fail
     qt5-quickcontrols             **regression**                Fail vs Pass
     qt5-svg                                                     Pass vs Fail
     qwt                                                         Pass vs Skip
 Processing x64-windows-static             759 vs 760
     qt5-script                    **regression**                Fail vs Pass

looks like there is a wrong hash in this branch for qt5-quickcontrols


Issue in power values

Power values contain negative values that have to be handled before proceeding with the CNN model.

This issue is solved. I updated the noise generation function. Now, the power values are >=0 and an integer.


Just tried Temel with Onyx (Crystal-AST-based) - looks good, thanks :-)

https://github.com/ozra/onyx-lang/issues/60#issuecomment-201954355

Wow I just saw this, that's awesome!

I'll add this example to the readme :)

It's been slow during the summer, but I've just jumped back into the code the last couple of days. Hope to have the lang practically usable (for personal projects) before the end of the year. So despite the looks of the github commit-log - it's not dead :-)
Cool!

BTW, you can try sql (since you use -- as comment prefix) as syntax highlighter for the Ozra lang on your README file. They look better.

Thanks for the tip! I've been thinking about which highlighter to abuse until github can accept Onyx!


fix: invite.encryptionKeys should be required

This unfortunately requires more than just an update to the proto definition because ts-proto does not support making messages required. This means we need to check that ourselves and throw / ignore invites if missing.
There also seem to be a few type errors in src/rpc.js that need fixing. Going to keep this PR open to address those issues.

the type error that stands out to me is the one regarding conflicting type definitions of the transformer and the protobuf interface that ts-proto outputs, basically what's described in https://github.com/stephenh/ts-proto/issues/881. The reasoning in that issue makes sense if you're strictly working with TS files, but in our case we're not so attempting to create a JSDoc typedef out of convenience doesn't work unless:

you rename the import of the transformer e.g. import { Invite as InviteTransformer } from '../generated/rpc.js'
avoid creating a typedef and inlining the import(...) bit each time you want to reference it in jsdoc
name the created typedef something different

a quick glance at the ts-proto options didn't surface anything immediately helpful, but maybe with fresher eyes there will be something that works. otherwise, gotta pick our poison with one of the above options I guess 😄
Another thing I was thinking is that maybe we should create another directory e.g. src/transformers where we can use that as the source of imports for anything related to the proto stuff. for example, the rpc one would be similar to the below and any code that needs to work withe proto stuff would import from here instead of src/generated:
import {
  Invite as InviteTransformer,
  InviteResponse as InviteResponseTransformer,
  InviteResponse_Decision as InviteResponse_DecisionTransformer,
} from '../generated/rpc.js'

/** @typedef {import('../generated/rpc.js').Invite} Invite */

export {
  InviteTransformer,
  InviteResponseTransformer,
  InviteResponse_DecisionTransformer,
}

yes, it's boilerplate-y and somewhat verbose, but this approach would also provide a location for us to override/customize the encode/decode functions, which we may need to do based on Gregor's assessment of the original issue

@gmaclennan would you mind taking a look at the recent changes I added? To summarize:

generally fixed the types issues that you alluded to. involved using https://github.com/dsherret/ts-morph after ts-proto does its thing to resolve output issues that aren't fixable with ts-proto

adds a "transformers" directory that's meant to be a home for adjusting and/or re-exporting generated exports from ts-proto. only does rpc for now since that's relevant for the original issue

Replaced by #260


should the reserved characters be an array of characters instead?

https://github.com/andreaTP/std-uritemplate/blob/186c831965511594db1adf75532d7af74ff6018b/java/src/main/java/org/uritemplate/StdUriTemplate.java#L288

Unfortunately, the String class in Java exposes a contains method that only takes String as an argument and not characters ...
All of the alternatives using plain characters are involving RegExp, which I carefully attempted to avoid in this implementation.

That said, I'm completely open to a more performant validation mechanism! (at the moment is "kind of" added on top and is not integrated with the processing loops).

I'll take this comment into consideration while stabilizing the implementation and ask some experts to review the performance.
When I'll port the implementation to C# I'll definitely use OrdinalIgnoreCase for comparisons, thanks for the tip!

Closed with #10


Update Secvisogram to CSD02

We need to update Secvisogram to be inline with CSD02. See oasis-tcs/csaf#450 for changes.

Here is a list of tasks:

[x] update schemas
[x] update UI elements
[ ] add pattern for document category
[x] update test files from repo
[ ] update existing tests
[ ] update unit tests & test files
[ ] implement new mandatory tests

Did I miss something?

@domachine: FYI

Tests were added #247.

This is obsolete with Secvisogram 2.0


Tests for environment token

Right now all of our tests sit in the base environment. We need tests to make sure the environment token retrieval is working.

I added a unit test in 6a0532b (#15), does this suffice?

Not quite. That's a good test but we need an integration test on a child environment, much like what python/test_config.py does

Gotcha, thank you! Given the timeline, I'll move this to the next milestone, we have some good changes for the next 0.4.0 IMO.


[DRAFT] DC-927: Cohort builder based on snapshots instead of datasets

Jira Ticket: DC-927
Dependencies
Tests are blocked by https://github.com/DataBiosphere/jade-data-repo/pull/1671
But, we would also like to merge them close together.
Summary of changes:
Switch to basing the cohort builder off of snapshots instead of datasets to enable data releases.
What

Update to use new data repo endpoints
URL is now keyed off of snapshots instead of datasets
We now retrieve the snapshot and the snapshot builder settings separately

Why

Enable data release workflow

Testing strategy

Manual testing against local TDR
Will deploy to BEE along with TDR changes

Iit occurs to me that the code is passing around snapshotId when it could be passing around a DataRepoContract type tied to the snapshot ID that contains all the methods. A way to do that is to change snapshotId: string to
  readonly snapshotApi: ReturnType<DataRepoContract['snapshot']>;

And in the caller pass:
    snapshotApi: DataRepo().snapshot(snapshotId),

And then it could be used as
    const result = await snapshotApi.getConceptChildren(concept);

Probably not worth doing now (or maybe at all) but I thought it was an interesting idea.


add flag to disable notifications

Sometimes the notifications are anoying, be able to turn them off

It will only send notifications if the discord url is set


Update NSS

Please fill out the template.
Read: https://k1ss.org/wiki/kiss/style-guide

Description of package
Installed manifest of package
(cat /var/db/kiss/installed/<pkg_name>/manifest)
https://termbin.com/c0b7
New package

[ ] Latest upstream version.
[ ] I agree to maintain this package.
[ ] I agree to notify @dylanaraps if I can no longer maintain this package.

Existing package

[x] I am the maintainer of this package.

This PR includes a lot of unrelated changes.

Apologies, these are from my previous PRs. I’m unaware of how to get rid of them without nuking and reforking the repository again.


[QA] - Reload all tabs option

[ ] The Reload All button padding and margin is not matching the Relaunch button.

[ ] There should be consistency between the "Reload" button in the popup and the developer tool.

[ ] The settings gets changed on all the tabs without reloading the page.

[ ] If switch the settings to multi tab, then on the cookie landing page the cookies details are visible without clicking on reload all button.

[ ] Errors are visible on the extension page

[ ] The toast message takes time to adjust its position when resizing the devtool screen.

The padding and margin of the "Reload All" button do not align with those of the "Relaunch" button.

Actual  result
Expected result

There should be consistency between the "Reload" button in the popup and the developer tool.

The settings gets changed on all the tabs without reloading the page.
Steps to reproduce:

Open multiple tabs and enable the CDP.
Observe on all the other tabs the CDP gets automatically enabled.

screencast-bpconcjcammlapcogcnnelfmaeghhagj-2024.02.15-14_30_05.webm

If switch the settings to multi tab, then on the cookie landing page the cookies details are visible without clicking on reload all button.
Steps to reproduce:

Open 2-3 tabs an visit to different sites.
Now, from first tab navigate to settings and enable the multi tab debugging. observe the toast message will be visible to reload all the pages.
Without clicking on it navigate to other tabs and observe the cookie landing page.

multitab.webm

Errors are visible on the extension page

The toast message takes time to adjust its position when resizing the devtool screen.
screensize.webm

[ ] "Reload All" / "Analyze this tab" button won't work after relaunch the browser.

After relaunch the browser the reload all and analyze this tab button not work.
Steps to reproduce:

Open the Privacy sandbox tab from devtool and observe all the cookies are visible.
Now, open the "chrome://flags" and change the settings and click on "Relaunch"
Visit the Privacy sandbox tab in devtool and enable the CDP and click on "Reload all tabs" button.

screencast-bpconcjcammlapcogcnnelfmaeghhagj-2024.02.16-08_09_05.webm

Verified the issue and it seems fixed and working fine on main branch. Hence closing this issue.


Improve the way errors are returned

For a sample error response from the service like this:
"error":"invalid_client","error_description":"AADSTS7000218: The request body must contain the following parameter: 'client_assertion' or 'client_secret'.\r\nTrace ID: 86b167f1-2c86-470c-bdc7-04cddd81d401\r\nCorrelation ID: 1441968e-02a2-4280-adb9-8b8ded58a0a0\r\nTimestamp: 2020-11-09 09:13:06Z","error_codes":[7000218],"timestamp":"2020-11-09 09:13:06Z","trace_id":"86b167f1-2c86-470c-bdc7-04cddd81d401","correlation_id":"1441968e-02a2-4280-adb9-8b8ded58a0a0","error_uri":"https://login.microsoftonline.com/error?code=7000218"

the library just exposes 'invalid_client`
Change this to include the error object as this currently hides the error description and the rest of the fields.

Looks like a dupe of #23 and #70.


ocdev application story

So, right now we have ocdev application create which creates a new application and binds it to the current project.
We should have the following behavior as well -
ocdev application ___

[ ] <name> - sets the given application as the current application
[ ] list - lists all the applications
[ ] get - gets the currently active application #23
[ ] delete - deletes the given application

 - sets the given application as the current application

I'm not sure about this, shouldn't this maybe set <name>?


Improve the Google Calendar synchronisation logic

mostly done in #55
from gcal too app. use googles sync mechanism. who wins in case of merge conflicts?
detect client in manual created events in gcal.
resync local appointments if there was no internet connection.
see old issue #20

not needed


ISPN-5182 Run tests against the EE 7 APIs so that we pick up JTA 1.2

https://issues.jboss.org/browse/ISPN-5182

Hey @tristantarrant!
Shouldn't we also update some dependencies in BOM? For example:

<version.cdi>1.0-SP4</version.cdi> to <version.cdi>1.2</version.cdi>
<artifactId>jboss-transaction-api_1.1_spec</artifactId> to <artifactId>jboss-transaction-api_1.2_spec</artifactId>

No, we base on those deps. The upgrade is only for test deps

Ok, thanks for clarification!

the tests are passing but I have the console with exceptions like this:
java.lang.IllegalStateException: Cannot log transactions as TransactionXaAdapter{localTransaction=LocalXaTransaction{xid=[Xid:globalId=cc2a2c4b1006f72672e6170616368652e61726965732e7472616e73616374696f6e0000000000000000000000000000,length=64,branchId=1000ffffffa6a2a2c4b1006170616368652e61726965732e7472616e73616374696f6e0000000000000000000000000000,length=64]} LocalTransaction{remoteLockedNodes=[TransactionsSpanningReplicatedCachesTest-NodeA-42662, TransactionsSpanningReplicatedCachesTest-NodeB-23890], isMarkedForRollback=false, lockedKeys=[a], backupKeyLocks=null, topologyId=2, stateTransferFlag=null} org.infinispan.transaction.xa.LocalXaTransaction@167} is not a NamedXAResource.

Well, since it resolves a ton of failures in the CDI tests, I believe it's a great improvement

ok... pushing :)

integrated! thanks @tristantarrant !


chore: update nginx to use new image

Nginx moved from public.ecr.aws/uktrade/nginx-reverse-proxy to public.ecr.aws/uktrade/nginx-dbt-platform but we never updated it in the templates.

Needs https://github.com/uktrade/platform-tools/pull/366 to merge first.


fix: fix type symbol completion

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[x] Y fix #1492

2. What is the scope of this PR (e.g. component or file name):

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

add field to determin whether is a type symbol in unresolved_symbol
add expr symbol on type node if without unresolved_symbol

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[x] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[x] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

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

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

Details

93 of 99 (93.94%)  changed or added relevant lines in 5 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.04%) to 70.587%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

kclvm/sema/src/advanced_resolver/node.rs
52
53
98.11%

kclvm/tools/src/LSP/src/completion.rs
13
15
86.67%

kclvm/sema/src/core/symbol.rs
22
25
88.0%

Files with Coverage Reduction
New Missed Lines
%

kclvm/tools/src/LSP/src/goto_def.rs
1
83.33%

kclvm/tools/src/LSP/src/completion.rs
1
91.89%

Totals

Change from base Build<PHONE_NUMBER>3:
0.04%

Covered Lines:
50595

Relevant Lines:
71678

💛  - Coveralls


Include tests folder in make format command

I updated the makefile so it also includes the tests folder for the make format command

I have formatted all the the test files present in the dev branch.
Also, they also passes all checkings.


Trigger on ugmembership creation calling error

When the trigger is called, it results in a 403 error with a message api rate limit exceeded.

Appears to be working occasionally, the problem is most likely to do with the refresh rate for the API limit. Worth testing intensely to see the actual refresh rate.
From the UnionCloud API documentation (apiary.io), 60 API calls are allowed, with a 60 second refresh rate.

Keep watching as the app is used


Move factbox code from serializer to component/storybook

Much better now! And  it's much easier to reuse the factbox component. :thumbsup:
As already discussed, we need to move the big fat margin from the text only version of the fact box to the serializer. That will make the storybook look good again.
I added a couple of stories to highlight the two column use case.

The margin issue is solved by this commit
I think this one is good to go!


Render list of Sessions

Overview
With the scaffolding work complete in https://github.com/silverstripe/silverstripe-session-manager/issues/11, now the field needs to be filled with Session data.
Acceptance Criteria

[x]  Pass session list for given Member to SessionManagerField through schema
[ ]  Render sessions based on designs
[ ] Last active hours converts to ‘days’ after 24 hours
[x] For the current session, display the current IP address (not currently in the designs)
[x] The current browser is being rendered with the ‘Current’ label
[x] Common user agent definitions (browsers/ devices) are tested and the label reviewed
[ ] The ‘Devices’ label has been reviewed for usefulness
[ ] It's been reviewed if there's usefulness in including a timestamp related to the current session data.
[x] Readme is updated along with the image of the UI
[x] The old Sessions tab is removed
[ ] The order of this component in the Member UI is verified with PUX

Notes

The MFA component has a fairly similar layout so you may see some hints for building this out there.
Consider the future need of being able to add a geo-location in replacement of the IP address. Can this be built in an extensible way now?

It's been reviewed if there's usefulness in including a timestamp related to the current session data.

@clarkepaul, FYI the module is already storing the 'signed in' timestamp as well as the 'last accessed' timestamp. Maybe that would be useful here? See: screenshot of current behaviour

Just noting that the "Signed in" information will not be visible with the new designs:

May be worth adding this again.
Thoughts @clarkepaul ?

All valid points. The designs were done to align with the managing of MFA methods and were done prior to me seeing a screenshot of this module (I thought it was needing to be a new UI). I haven't personally stepped through that experience e.g. what happens with revoking sessions and notifications but I think its worth another conversation around whether its worth creating a new UI when this might be sufficient as an MVP.

Just posted an update of the designs @bergice, basically a catchup of designs already discussed hopefully. Some new stuff with the section label and logout action.
For the section label, I've taken a look around to see what would make the most sense to people and think retaining "Device" should make sense. That seems to fit closer to the majority scenario of a different physical device. I think we should make it slightly clearer with the addition of "Authenticated devices" so that people have more context about it as login/session type information. In saying that, session/device both seem to work well, so happy to hear if there is any objection to this.
For the logout transition (although not part of this issue), this was designed without physically testing it out so keen to work with whoever implements it to adjust as needed if not quite right.


Merge maint-0.7 to master

[x] reana-ui
[x] pytest-reana
[x] reana-commons
[x] reana-db
[x] reana-client
[x] reana-job-controller
[x] reana-message-broker
[x] reana-server
[x] reana-workflow-controller
[x] r-w-e-cwl
[x] r-w-e-serial
[x] r-w-e-yadage
[x] reana

Works well with:
$ reana-dev git-status -c ALL | grep -v '@ master' | sort
pytest-reana @ pr-78 @ b4bc5eb Merge branch 'maint-0.7'
reana-client @ pr-450 @ e6cd519 Merge branch 'maint-0.7'
reana-commons @ pr-242 @ c8ec8f6 Merge branch 'maint-0.7'
reana-db @ pr-117 @ 19a1ac9 Merge branch 'maint-0.7'
reana-job-controller @ pr-289 @ ed9e82f Merge branch 'maint-0.7'
reana-message-broker @ pr-33 @ b06b53b Merge branch 'maint-0.7'
reana @ pr-465 @ c20c85c Merge branch 'maint-0.7'
reana-server @ pr-324 @ d83c88a Merge branch 'maint-0.7'
reana-ui @ pr-161 @ bfa760d Merge branch 'maint-0.7'
reana-workflow-controller @ pr-355 @ 0d9c34e Merge branch 'maint-0.7'
reana-workflow-engine-cwl @ pr-162 @ 9154af4 Merge branch 'maint-0.7'
reana-workflow-engine-serial @ pr-128 @ e62774a Merge branch 'maint-0.7'
reana-workflow-engine-yadage @ pr-174 @ ae075c9 Merge branch 'maint-0.7'

All PRs are now merged.


TF importer doesn't work with Keras model

What happened?
Got
tensorflow/compiler/mlir/tensorflow/transforms/xla_cluster_formation.cc:74] no entry function is found

when trying to import a keras model
Steps to reproduce your issue
import tensorflow as tf
from iree.compiler import tf as tfc

inputs = tf.keras.Input(shape=(32, 32, 3))
resnet = tf.keras.applications.ResNet50(weights='imagenet',
                                        include_top=True, input_tensor=inputs)

resnet.save("/tmp/resnet50")

loaded_model = tf.saved_model.load("/tmp/resnet50")
print(loaded_model.signatures.keys())

compiler_module = tfc.compile_saved_model("/tmp/resnet50", import_only=True, exported_names="serving")

We can see the serving signature is there, but it still fails to compile.
What component(s) does this issue relate to?
Frontends
Version information
6a46afd4f82715979b54c57fb41a8505466cd68f
Additional context
No response

Sorry I filed it with a wrong account and now that account has been deleted. Filed another issue: https://github.com/openxla/iree/issues/13854, please help triage, thanks!


updating code to publish package

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

Needs approval from an approver in each of these files:

OWNERS

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

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

Needs approval from an approver in each of these files:

OWNERS [mderazon]

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


Logging into AEM Cloud using Sites30 Proxy

Hi guys
@bartoszglow  I'm having issues logging into a dev instance of AEM Cloud after running npm run live to test frontend changes.
Is there any documentation on permissions that need to be granted to a user or a special user profile that needs to be configured?
I can log into the AEM author as a Business Owner and have full admin rights using my Adobe ID.
If I try to sign in under http://localhost:7002 using my Adobe ID I get redirected to the following URL which returns a 403:
https://author-p63602-e526610.adobeaemcloud.com/callback/j_security_check?code=eyJhbGciOiJSUzI1NiIsIng1dSI6Imltc19uYTEta2V5LWF0LTEuY2VyIiwia2lkIjoiaW1zX25hMS1rZXktYXQtMSIsIml0dCI6ImF0In0.eyJpZCI6IjE2NTA2MzU0MDU0NTRfYzEwOWY5M2UtOWQ5ZC00MDVhLThhMjktMTM0NzUyYmYxMDUyX3VlMSIsInR5cGUiOiJhdXRob3JpemF0aW9uX2NvZGUiLCJjbGllbnRfaWQiOiJ3ZWItcDYzNjAyLWU1MjY2MTAtYzA2ODQyYjctZGU1Yy00MjgyLWI2MTUtYjFlMTk1NTE1NzhiIiwidXNlcl9pZCI6IjIyMkU1N0UwNjI0NUE4NzAwQTQ5NUM4Q0BBZG9iZUlEIiwic3RhdGUiOiI5YjdmbzlmOWcxYnFmbW9vZXF2bmNqamxiNSIsImFzIjoiaW1zLW5hMSIsImFhX2lkIjoiMjIyRTU3RTA2MjQ1QTg3MDBBNDk1QzhDQEFkb2JlSUQiLCJmZyI6IldNREI0WktPWFBNN0lQVUtHTVFGUkhRQU9BPT09PT09Iiwic2lkIjoiMTY1MDYzNTIzMDU3N182Mzc3MDY2ZC03OTc5LTQ1NDAtYmFlMi01NzZhNjE5NjgzOTRfdWUxIiwib3RvIjp0cnVlLCJleHBpcmVzX2luIjoiMTgwMDAwMCIsInNjb3BlIjoiQWRvYmVJRCxvcGVuaWQscmVhZF9vcmdhbml6YXRpb25zLGFkZGl0aW9uYWxfaW5mby5wcm9qZWN0ZWRQcm9kdWN0Q29udGV4dCIsImNyZWF0ZWRfYXQiOiIxNjUwNjM1NDA1NDU0In0.elTDUTTD9CME7fZv_NuxxECFeneGYCN3z08LUzkD152hyQ-jJZe_lKVMv18aEZ2BUPq4tzR3_KiOxgg4Glnx4UjEfdjtXAwocrtvWuONaUL75ejTC_St1qbJzYqwpP-8enAee13jKi76beqiesShqs43mxP_c4xpEkrOe3qmplXmzgTAr_YpTlim2v5Iq5BmkHJlJ3b_-uL4u6S_M40Qspfq6_28KrchYXLg6ocM7t0QYa57kwbPWJPA4afNC0HI9qbRA5ZAlTFM5RqOgWCDNeS6gyATFYHn2Bbnrm-pRvbcVxME1_xPJ_YFRrTOoSzsvjTRoBXMmrMy2RIXxfmfEA&state=9b7fo9f9g1bqfmooeqvncjjlb5
But then if I navigate to https://author-p63602-e526610.adobeaemcloud.com/aem/start.html I'm logged in and have full access.
So this seems to be an issue with logging in under localhost.
This is basically a vanilla site created using the Standard AEM Site Template 2.1.1.
I've downloaded the Theme Sources from the AEM site and tested the theme code from this repository with the same result.
Regards, Charlie

Unfortunately only AEM user/password authentication is supported by the preview/proxy. You will need to create a temporary user granting the needed access.


policy: cache what identities label selectors match

Matching is costly; cache this within the EndpointSelector within Cilium (or some other construct).

Assigning to @joestringer .

Needed work:

Update LabelArray to contain a string with the hash of the sorted contents of the LabelArray. All consumers of LabelArray in policy calculation will use this.
Ensure that Identities have their LabelArray hashed and have this hash populated upon creation. Identity structures are immutable after they are created.
Add a hash into the EndpointSelector, similar to LabelArray.
Add a new cache, which maps whether a given hash of an EndpointSelector selects a given set of labels (hash from LabelArray), i.e. map[string]map[string]bool. When checking whether a given EndpointSelector selects a given set of labels (in selector.go:Matches), this cache is utilized. If the lookup in the map fails, resort back to label-based matching, and populate the cash after accordingly with the result of the label-based matching operation.

This cache should have some type of GC / LRU-based functionality.

Assigning this to @jrajahalme as he is working on caching this work with https://github.com/cilium/cilium/pull/7539

Completed by #7539


Fix for Issue2106

Issue #2106
I've changed the way clients connecting to the server receive player lists. Before this change, the connecting client only requested the player list from the world they were initially connecting in. This behavior caused players in other worlds to not appear on the newly connected player's player list as well as appearing invisible when they finally see each other. I changed this by adding a function to cRoot that iterates through all current worlds sending their player list to the requesting client.
I've also made a change to broadcasting chat. Originally chat was only broadcast to the world of the sender, it is not broadcast server wide to all worlds.

You should rebase this PR onto current master, so that the CIs can have a go at it. Perhaps even squash the commits. Thanks.


[pod] Undefined is not an object (evaluating navigator.geolocation.watchPosition)

I use version 0.24.1 of React Native.
I get this error only with Chrome Debugging disabled (with Chrome Debugging enabled, it works fine) :

My Podfile :
platform :ios, '7.0'
#use_frameworks!

target 'Wino' do
  source 'https://github.com/CocoaPods/Specs.git'
  pod 'React', :subspecs => [
    'ART',
    'RCTImage',
    'RCTNetwork',
    'RCTLinkingIOS',
    'RCTActionSheet',
    'RCTAdSupport',
    'RCTGeolocation',
    'RCTPushNotification',
    'RCTSettings',
    'RCTVibration',
    'RCTText',
    'RCTWebSocket'
  ], :path => '../node_modules/react-native'
  pod 'react-native-fbsdkcore', :path => '../node_modules/react-native-fbsdk/iOS/core'
  pod 'react-native-fbsdklogin', :path => '../node_modules/react-native-fbsdk/iOS/login'
  pod 'react-native-fbsdkshare', :path => '../node_modules/react-native-fbsdk/iOS/share'
end

target 'WinoTests' do

end

@LeoLeBras Any chance your using https://github.com/skellock/reactotron?

@christopherdro Yes, that was it. Thank you ! :)

Fixed via https://github.com/skellock/reactotron/commit/bfbc5102b8dde44094075399dfb45b2319ff6d3a


Question: Can you set maximum logging level

Hello,
I am trying to accomplish a logging on different channel depending on the level of the log.

If logging level is <= Warning, then log to stdout
if logging level is >= Error, then log to stderr

Is there a way to get this work ?
For now I got
	backend := logging.NewLogBackend(os.Stdout, "", 0)
	leveledConsoleBackend = logging.AddModuleLevel(backend)
        // ...
	errorBackend := logging.NewLogBackend(os.Stderr, "", 0)
	leveledConsoleErrorBackend = logging.AddModuleLevel(errorBackend)
	leveledConsoleErrorBackend.SetLevel(logging.ERROR, "")

	logging.SetBackend(leveledConsoleErrorBackend, leveledConsoleBackend)

But with this, I see error logs output both in Stdoud and Stderr
Thank you for your help!

Man this is garbage, I opened a ticket in July lol, waiting for a response :(


DNS crashes if host resolv.conf contains docker0 bridge ip

We are trying to resolve container dns names from host by adding weavedns ip (docker0 ip) to /etc/resolv.conf. This seems to create circular dependency inside weavedns because it sees docker0 ip as an upstream server.

As a work around, try WEAVEDNS_DOCKER_ARGS=--dns=<IP_ADDRESS> weave launch-dns (WEAVE_DOCKER_ARGS=--dns=<IP_ADDRESS> ./weave launch for master)

That seems to help, thanks! But I don't like to use this as a permanent solution because upstream servers should most likely come from dhcp. Would it be possible to remove "own" ip from upstream servers list (this is how we are currently doing it in kontena)?

Yes we can do that.  Out of interest, whats the contents of your /etc/resolv.conf?

I was just testing this in Vagrant so nothing special:
nameserver <IP_ADDRESS>
nameserver <IP_ADDRESS>

First nameserver is docker0 ip.

Ideas for fixing:
(1) filter out the docker bridge ip from the weave containers revolvers in the weave script, when launching weave
(2) filter out the docker bridge ip in the weavedns code, when parsing resolv.conf
(3) detect when we are our own recursive resolver (by tracking inflight request ids) and error those requests.
(1) would mean docker wouldn't automatically update the weave containers resolv.conf when the host's changes; also, it would add complexity to the weave script, which we probably don't want
(2) would be hard to detect the docker bridge ip in weavedns; we could make an explicit flag for it, or we could filter on the weave containers default gateway (no no no)
(3) would be a little messy to implement, but it probably the most general.
@rade @inercia Thoughts? Alternatives?

@rade suggests using iptables to stop us talking to ourselves.  This would manifest as a timeout, and we don't want the delay before trying the next server.

Lets got with (2), and call the flag --dns-effective-listen-address

@tomwilkie I also think (2) is our best option...


Gon defined only when user directly access page, not defined when user is linked to a page

Really struggling with this one... If I visit the page where I need to have gon variables defined, i.e., if I navigate to www.myhost.com/users/signup, everything works fine. But most users will get to that page via a redirect from the home page (they click on a link). Given that gon is defined in the backend in the action that serves up the users/signup template (and indeed, I can confirm via a puts statement that the gon is always successfully defined in the action), why is it not available in the rendered template? And only (again) when the user is linked to the page?
# Staticpages/home
def home
end

# home.html.erb (/)
<%= link_to "Sign up", new_user_registration_path %>

# Where the route above links to Users::Registrations/new
def new
    gon.zipcode_list = zipcode_list
    puts gon.zipcode_list #note this push always returns the right value, so in the server side, gon is working fine
end

# new.html.erb (/users/signup)
<script>
    alert(gon.zipcode_list)
    // the alert above fires ONLY when I navigate to /users/signup. When I click on the link in the homepage, it does NOT fire, and there's a console error that gon is undefined
</script>

Try putting <%= Gon::Base.render_data %> in to the body of your application.html.erb instead of the head section. Believe this is an issue with turbolinks

I am running into a similar issue...

A little frustrating but all I did was add <%= include_gon %> on the top of that specific view and it worked. No idea why but I'll take it.

As I see, the reason of this behavior is that you use something like turbolinks or similar instrument, which re-render only part of the view and don't touch gon area


Reply-To is not processed correctly

Issue Summary
When sending an email through this SDK, if the 'name' portion of 'from' contains commas and ampersands, it is shown properly in gmail.  When the 'name' portion of 'replyTo' is set to the same exact name, gmail finds it malformed.
Portion from "Show original" in gmail to see raw headers (with some values redacted):
...
From: "\"Jonathan Short<EMAIL_ADDRESS>& Jonathan Short\"" <test@redacted>
Mime-Version: 1.0
Message-ID: <redacted>
Subject: Sending with Twilio SendGrid is Fun
Reply-To<EMAIL_ADDRESS>"& Jonathan Short" <test2@redacted>
To: <redacted>
...

Steps to Reproduce

In the code snippet, replace the  email domain on the 'to' field to your gmail email address
In the code snippet, replace the  email domain in 'from' and 'replyTo' fields to a domain properly configured for Sender Authentication in your SendGrid account
Export SENDGRID_API_KEY with a key which has permission to send email
Run the code snippet and observe the email received in gmail

Code Snippet
const sgMail = require('@sendgrid/mail');
sgMail.setApiKey(process.env.SENDGRID_API_KEY);
# this complex name is handled correctly for From address but incorrectly for Reply-To
const name = "Jonathan Short<EMAIL_ADDRESS>& Jonathan Short";
const msg = {
  to: 'jonathan.short@<redacted>',
  from: {
    email: 'test@<redacted>',
    name: name,
  },
  replyTo: {
    email: 'test2@<redacted>',
    name: name,
  },
  subject: 'Sending with Twilio SendGrid is Fun',
  text: 'and easy to do anywhere, even with Node.js',
  html: 'and easy to do anywhere, even with Node.js',
};

sgMail
  .send(msg)
  .then(() => {}, error => {
    console.error(error);

    if (error.response) {
      console.error(error.response.body)
    }
  });

Exception/Log
none

Technical details:

sendgrid-nodejs version: 7.1.0
node version: 14.2.0

I logged the body sent to SendGrid here (https://github.com/sendgrid/sendgrid-nodejs/blob/fa11e35bee18ac480ca9b331c09447907e1a8799/packages/mail/src/classes/mail-service.js#L193), looks like the from name and reply-to name are identical so perhaps a server side issue?
{
  from: EmailAddress {
    email: 'test@<redacted>',
    name: '"Jonathan Short<EMAIL_ADDRESS>& Jonathan Short"'
  },
  subject: 'Sending with Twilio SendGrid is Fun',
  personalizations: [ { to: [Array] } ],
  content: [
    {
      value: 'and easy to do anywhere, even with Node.js',
      type: 'text/plain'
    },
    {
      value: 'and easy to do anywhere, even with Node.js',
      type: 'text/html'
    }
  ],
  reply_to: EmailAddress {
    email: 'test2@<redacted>',
    name: '"Jonathan Short<EMAIL_ADDRESS>& Jonathan Short"'
  }
}

Hello @jonathan-short,
Thanks for submitting a GitHub issue! We are very sorry that you are running into this problem. In order to better serve you, as this does not present itself as a library specific issue, we would like to ask that you reach out to our support team at https://support.sendgrid.com.
Thank you!
SendGrid DX Team


The battery level setting is preserved, but it still charges until I run battery again

What is the issue? (required)
If I reboot (or some other reset situation?) the battery status shows the level of the battery to be maintained has been preserved, but the charging is enabled even if the battery level is much higher until I run battery maintain.
What exactly did you do to produce the issue? (required)
Steps to reproduce the behavior:

sudo battery maintain 40
reboot
sudo battery status
See "Your battery is currently being maintained at 40%" which is correct (battery level preserved) but also nonsensically see "smc charging enabled"
sudo battery maintain 40
sudo battery status
See "Your battery is currently being maintained at 40%" but also "smc charging disabled" which is obviously what I want

Expected behavior (required)
If the battery level is above what the maintenance level is, charging should be disabled.
Screenshots (optional)

BTW this is on a 8 core m1 13" macbook pro

Sigh, it reset this time and I didn't even reboot. Back on 100%.
I constantly have to babysit this thing. I might have to create a cron job to periodically check the battery level to detect when this is malfunctioning

Can you show me the output of battery logs before you reboot and after? It would really help in debugging.

I've never seen battery logs show any output

It happened again. I think this time because I had to reboot