Support comparing two linked properties

For example, to get all persons with more children than siblings: [Person objectsWhere:@"numberOfChildren > numberOfSiblings"]. See this Asana ticket in Core (private link).

I was mistaken. Realm supports comparing two properties, but not two linked properties. So this works:
[ModelA objectsWhere:@"int1 > int2"]

but this doesn't:
[ModelB<EMAIL_ADDRESS>> modelA.int2"]

I was mistaken. Realm supports comparing two properties, but not two linked properties. So this works:
[ModelA objectsWhere:@"int1 > int2"]

but this doesn't:
[ModelB<EMAIL_ADDRESS>> modelA.int2"]

Multilevel equality was added in #3419, but other comparisons still aren't supported.

This was mostly done in #2895. There are some complex predicates involving multi-level key paths that are not yet supported, but the link.property > link.otherProperty case definitely works.


feat: add option to filter targets

This change closes  #7191 by introducing a callback that can be used to decide if puppeteer should connect to a given target.

I have no idea how this PR works or what the changes mean, but in the release change log for version 9.1.0 this PR is the only one in that release.
Ever since upgrading to puppeteer 9.1.0 we started seeing strange errors from jest.
Our project has a bunch of *.test.js files and only a couple of them have anything to do with puppeteer. There are a couple that use jest-puppeteer.
Unscientifically, I found that our test suite stopped being so busted when we downgraded back to puppeteer 9.0.0.
Not sure who or how it helps.
Here's my issue about it
Here's my PR to hack around it

Hi @peterbe Thanks for bringing this to my attention!
I'll try to reproduce the problem and see what this might be about.

@peterbe Thanks for pinging - I think this is related to the fact that the filter can be async. I'll verify that now...

By default, puppeteer connects to all available targets (=pages and workers etc.). We wanted to give developers a way to control which ones to connect because puppeteer enables some CDP domains by default which might interfere with what that target is doing.
Unfortunately, I allowed the filter to be async and awaited its result. That led to a situation in which the targetInfoChanged event was processed before the target was created because it was still waiting for the return of the targetFilter.

Yeah I have just raised issue #7204 which turns out to be due to this.


Failed to download template icon when using http protocol and custom port

Environment

Walrus version:  main ed21adcc5770196dc4a5654502d4a98a0987a631
UI version: dev-2d5d418

Steps to reproduce

Add a template from self-hosted gitlab server as follows, using http protocol and the port is 8929.

Check the template information.

Result

Failed to download icon of template.

Check the request url, the protocol is incorrect and the custom port is missing.

Expected behavior

Display the template icon.
Additional context
Add any other context about the problem here.

Environment

Walrus version: main 829ac7a2a65c2ee5092f994e81f8b41214e376a4
UI version: dev-e2db642

Test Result:
walrus should use http as well. (--enable-tls=false)


Bounded cache

This is my proposal for #32. It enables users of Haxl to use caches of bounded size, but keeps "cache everything" as the default behaviour.
Please let me know if you have any remarks or want me to change anything.

Taking a glance over this, it is slightly intrusive and touches some performance-critical bits (e.g. DataCache.lookup).  We'll need to be careful it doesn't impact performance.

Yes, unfortunately it's rather intrusive. I'm also not sure whether the use case this opens up, transferring large amounts of data, is something that other Haxl users will want to do.

@simonmar, @kantp Hi, any update on this?

Hi @jkomyno,
no news from my side. I have moved away from the project that I had used this in (using a fork of the repository with this patch), so I didn't pursue this any further. If it's deemed useful, I'd be happy to help in getting it merged, though. But the performance concerns are a valid point of course, we'd need to make sure that performance is not impacted by this.

@jkomyno in principle I have no objection to including a bounded cache of some kind, but it needs to have zero (or close to zero) overhead.


Will this work with .net 6?

I noticed that this demo is using .Net Core 3.1. Can I apply this same logic to an api written in .Net Core 6?

@andywatermarkmedical you can, yes. I've added a separate branch to the repo containing a .NET 6 implementation. I'm just considering the simplest way to keep examples of both.


Channels where xp doesn't count # is missing before every channelname

Bugreport:
Bug reported by Monkeyyy11
What is the title of your bugreport?: Channels where xp doesn't count # is missing before every channelname
Is it a bugreport for the website or for the bot?: website
How can you reproduce the bug? (Explain in steps): Go on the website, log in, go a server dashboard, navigate to administration and see in generalsettings the "channels where xp doesn't count" setting https://i.imgur.com/52QdYcW.png
Which result would normally have to be?: Normally there should be a "#" before every channelname
What happens if you do the action now?: There isn't a "#". You can see it in the screenshot
ReportID: 34

➤ LenoxBot commented:
Approve:
reproducible
Wandi#9576

➤ LenoxBot commented:
Approve:
could be reproduced
Dadi#7808

➤ LenoxBot commented:
Approve:
reproducible
Fabiooo#3984


Future plans and continuation

I work for a software company and after a longer research we have found ZBoxFS which could solve couple of our challenges. What is annoying is that it seems the development of ZBoxFS is stopped at the end of 2021. Can someone tell me, what can be expected in the forthcoming years? Will the project be abandoned?

Any updates/insights on this?


Expose java_string_create(JNIEnv *, const char *, size_t) publicly.

I'm JNI-bridging a library (Abseil, used in WebRTC) that has a type (StringView) with non-NUL-terminated C strings, so this function is useful as part of the public API, and I don't see any reason it shouldn't be exposed.

Makes sense. There probably should just be an std::string_view taking function but this would be a breaking change. Maybe in next major release.


Unable to get multiple dataset from store procedure

I am trying to get multiple data sets from sp via FromSql in Dotnet core 2.2, but unfortunately, I am not getting success. does Dotnet core provide support to get multiple data set from one sp

Duplicate of #8127


Community cards implementation.

I'm interested in contributing to the community cards implementation. More of this can be found #903

I feel that in order to make the cards more accessible and easier for users to understand what each card means without looking at the names, we should incorporate an illustration or picture that describes the cards.
For the community cards, what does it entail?? How does it work?

@AceTheCreator @mcturco @derberg

I feel that in order to make the cards more accessible and easier for users to understand what each card means without looking at the names, we should incorporate an illustration or picture that describes the cards.

Great idea, @Mayaleeeee! Go go go 🚀

For the community cards, what does it entail?? How does it work?

@AceTheCreator You may want to give some direction on the intent of these cards from a UX standpoint. Why did we add them? What purpose are they serving? How will they help visitors get more information about the community?

Thanks @mcturco

@AceTheCreator You may want to give some direction on the intent of these cards from a UX standpoint. Why did we add them? What purpose are they serving? How will they help visitors get more information about the community?

Thanks, @mcturco
@Mayaleeeee  I'm really glad you're interested in implementing this.
Firstly, If I get you correctly @Mayaleeeee the first bullet point you indicated is about enhancing the existing cards we have on the community landing page that links to other pages.
Secondly, the goal of the community card is to make all contributors visible. A case study is this...
We are going to have a page /community/contributors where all the lists of contributors will be displayed, for each contributor card, it'll include the name of the contributor, projects contributed to, contributors' handles, and pictures and contribution metrics

That's nice.
Thanks @AceTheCreator

Hey Folks, here's the update of the community members card I'm working on, let me know now what you think about it.
[https://www.figma.com/file/NC2FUyFAccF08uP5RbsbAB/AsyncApi?node-id=908%3A169]

Hey guys, here's an update on the contributors' card I'm working on. Please do let me know what feel about it.

@Mayaleeeee This is looking great so far! I think the second option works better, but I am wondering if we really need the colored dots and if we can take down the intensity of the drop shadows. We should also add a light grey border and rounded corners to match other UI elements on the website.
I would also use the secondary/500 as an accent color, as this is what I have been doing on the rest of the website. The purple color is to be used sparingly so we do not overdo it.
Maybe you can also start exploring what this would look like on mobile devices?
Great work so far! 🎉

Alrighty @mcturco , thanks for the feedbacks.
I'll will definitely work on them and get back soon🥰🥰.

Okay @derberg
Are you suggesting that we do away with the categories and simply put all images of each contributor?
And regarding the social media, we just need to delete the linkedln icon right?

Hey Folks
I'll be updating this file very soon. @mcturco @derberg @AceTheCreator

@Mayaleeeee I noticed I missed this Issue...
I started working on the contributor's page with no idea you have an implementation for a community card in progress. What's the status of your progress?
You can also check out the PR for the contributor's page here #1122

Hey @AceTheCreator, I'm so sorry for the late response.
It's OK, I was supposed to update the file this week and make corrections based on what @derberg and @mcturco suggested, but I can't find the Figma file on my board.

So sorry to hear that Maya... I hope you resolve it

Yeah yeah @AceTheCreator
I'm working on a new one atm

Hey @derberg @AceTheCreator @mcturco
I've updated the community members' page, but I could not find an appropriate icon to replace LinkedIn @derberg. I'm open to suggestions, though.
Please check I out.

LGTM
I'm only missing:

under main header with page name, we should have a note that the list is sorted alphabetically, like in https://www.asyncapi.com/community/tsc
next to github and twitter icons we should have kinda globe (I think it is standard) that will represent link to user website

other than that, it is perfect.

Thanks @derberg
I'll work on it tonight and update it here.

Hey @derberg  , If @Mayaleeeee  is not working on this , can I take this up ?

Hi @reachaadrika nice to meet you.
I'm done with my internship, so I will resume my work in the community on Monday.
Thank you 🤗🤗

Hey @derberg @AceTheCreator @mcturco @All
I've updated the community members' page based on @derberg feedback; please check it out.
Thanks.

I think it looks good, nice job!
now we need to figure out how to power it with data 😄

Thank you

@allcontributors please add @Mayaleeeee for design

Hii. Absolutely love this Idea ❤
Is this implemented?
If it isn't is it ok If I pick this up. Cheers!

@akshatnema @Mayaleeeee @derberg Have any ideas in mind for this?

I think it looks good, nice job!
now we need to figure out how to power it with data 😄

@Mayaleeeee @AceTheCreator @derberg
Can I work on this issue?

hey folks. I'm not sure what should be the future of this issue. To implement the UI, we need data first - collecting them is super difficult and also brings costs that I'm not sure are worth it.
pinging @thulieblack that played with data some time ago, and @AceTheCreator that was involved in discussions

hey folks. I'm not sure what should be the future of this issue. To implement the UI, we need data first - collecting them is super difficult and also brings costs that I'm not sure are worth it.
pinging @thulieblack that played with data some time ago, and @AceTheCreator that was involved in discussions

Hello, @derberg
My name is Surur and I'm just starting a career in data analyst.
I have reached out to @thulieblack on this issue, we were able to discuss it on a call and I'm already working on it.
It's a WIP 😀
cc @thulieblack

oh, nice!

@Oloso-surur any updates?

@Oloso-surur any updates?

Hello everyone, here is an update on the contributors' data I'm working on.
You can view the data via this link.
cc @derberg @thulieblack  @Mayaleeeee @AceTheCreator @sambhavgupta0705

@Oloso-surur, great work, but I want to ask how you got the following data?

Hello everyone, here's how I got the data 👇🏽👇🏽👇🏽
To collect the data for each contributor, I accessed our Organization link on GitHub and manually retrieved the details for each profile.
After that, I carefully extracted the relevant information from each profile and manually entered the sorted data into a Google sheet by hand.
Here's the link to the google sheet with all the data.
cc @thulieblack @AceTheCreator @Mayaleeeee @derberg

MMH, I'm skeptical that you collected the data manually, and some fields are incorrect. We need data across all GitHub repos, and manually collecting them isn't the best solution at all.

we previously used GitHub archive to collect data, the downside to this was the amount of storage needed to store the data queries to the database.
Unless there is an alternative to this?

we can do the same using the GitHub API to collect all the data. we get the data in JSON format but if required we can convert to yaml or csv format as per the requirement.

Unless there is an alternative to this?

@thulieblack Analyzing the data via Github Archive will sure take time. We can't do it using github workflows. One of the possible solutions can be doing it manually over a Google collab where all the archive data will be given using a tar or zip file and all it's underlying files will be analysed. The Collab will give output in the form of json files, that can be committed to the github repo.
For more flexibility and long term solution, we should use a dedicated server/service to run the python scripts and analyse all the parameters using Github archive.

we can do the same using the GitHub API to collect all the data. we get the data in JSON format but if required we can convert to yaml or csv format as per the requirement.

I don't think Github API should be used because required information is given by github archive easily. So, we shouldn't waste our resources making various API calls to get the data.

@akshatnema are we gonna work on this one ??
If yes then we can submit it to bounty of the next quarter

@sambhavgupta0705 @akshatnema is this issue up for taking , then i would like to work on it .

Yes @GarvitSinghal47 , it is up
You can take it

just one more question which approach should i use to get the data as there are two approaches being discussed above one using api and other using archive.

Hi @derberg ,
Is this issue still open? I’d be happy to integrate the GitHub API for the Community cards. Let me know!


GitHub Action CI/CD - Fixed issue #595

🚀 🚀 Pull Request
All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Is your Pull Requests linked to an Issue

New Feature Submissions:

[x] Does your submission pass all tests?
[x] Does your submission not lower the coverage-rate?

Summary

The GitHub-Action CI/CD is now running full unit and coverage test for all (forked) branches.

LGTM


The creation process never completes within cloudformation

What steps did you take and what happened:
[A clear and concise description of what the bug is.]
from: https://aws.amazon.com/quickstart/architecture/heptio-kubernetes/
choose 👍
Deploy into a new VPC
follow the steps to fill in required information. then proceed with "Create".
What did you expect to happen:
I expected the VPC and Kubenetes cluster to be created.
The Nested stack completed as the status shows, but the master stack never completes, it shows "CREATE_IN_PROGRESS" forever.
The VPC is actually created, but if you check EKS service, no cluster is created, and no error messages in Cloudformation.
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Environment:

Kubernetes version: (use kubectl version):
OS (e.g. from /etc/os-release):

"but if you check EKS service"...?
This is not an EKS cluster.

Interesting.
What I observed:

The master stack completed after 6 hours, while I expect it to complete in 10 minutes.
If I run the same template in couldformation in a region that does not support EKS, it fails quickly. If I run it in a region that supports EKS, it behaved as what I described.

I will run it again and let you know.


Make the model traceable with Torchscript

Fixes #282
Goals :soccer:
This PR makes the model traceable with Torchscript (in non-strict mode strict=False).
Implementation Details :construction:
pred_metadata and model_outputs in the model output are commented out because it doesn't look like they are being used anywhere and it seems they were only left there as a reminder for a future task.
The checks on the shapes are skipped when we are tracing with Torchscript. In general, it is difficult to get around such checks on shapes when tracing because tracing can't deal with a control flow branch that depends on the input shape. This PR proposes that we simply skip the check and assume that the input will be valid when the model is traced.
Testing Details :mag:
python3 -m pytest tests/torch

Adding a note for future reference, enabling tracing for strict=True should also be fairly simple. Currently the model output is a dictionary, and with struct=True the output cannot be a mutable object. By implementing a converter that transforms a dictionary to an immutable object and back (e.g., named tuple), we should be able to also trace the model without specifying strict=False and without any breaking changes in the model output. For example,
converter = Converter(model, batch)
traced_model = torch.jit.trace(converter, batch)
intermediate_output: NamedTuple[str, torch.Tensor] = traced_model(batch)
final_output: Dict[str, torch.Tensor] = converter.to_original_format(intermediate_output)


Speakers echo microphone's sound

Hi,
when I open a page with the recorder, browser asks me to allow microphone. When I do it, speakers start echoing the sound from microphone, louder and louder.
How to fix this problem? I think the script should disable microphone after the recording is finished and enable it when I press 'start recording'.

The "kehers" fork of the project has implemented the fix + made other improvements. Read his blog post to see details.

i think there is an issue with kehers - the file time length is twice as recording

@intergleam i was about to find kehers' fork, but I wanted to know if you had issues with that fork?


Allow customizing the GCE metadata service address via an env var.

The goal here is to make it possible for a user of a binary that depends on
this library (eg the google cloud SDK) to be able to customize where it looks
for the GCE metadata service. (An adventurous user can already customize the
GCE metadata service location via the existing global vars in this library.)
The only bit of awkwardness here is really the test: since this is a top-level
statement, reloading is the only way to ensure it works.
PTAL @jonparrott
(and why yes I DID just completely copy-paste the commit description from the oauth2client PR.)

@craigcitro Did you want to merge in both libraries? (It's pretty unlikely oauth2client will ever get another release.)

@dhermes Yep, I purposely sent the PR for both.
@jonparrott mentioned the same about oauth2client, then remembered that he had to add something silly (update the classifiers?), so it was an opportune time for me to sneak it in. 😉

OK, updated to use the constants in environment_vars -- PTAL

@craigcitro LGTM, but Travis is mad about you not using 4 spaces for indentation. :)

As well he should be. Fixed.
(Do you guys autosquash or should I squash?)

(Do you guys autosquash or should I squash?)

We autosquash

Thanks, @craigcitro!

:car: squash


So, that memory leak...

Any ideas on how to mitigate it? I sampled at 10 per second (far too low) and ended up with gdb taking up almost 15GB, so I can't really use this seriously (and boy do I want to, we need to get to the bottom of some of our build times at Symmetry).

Fixing it will undoubtedly require delving into GDB internals.
As for pragmatic workarounds, my personal approach so far has been to have 64 GB of RAM on my main system, which has turned out quite useful in situations like these. You might have some luck with enabling memory compression (zswap) or a big pagefile, but that will likely skew the results of profiling as well.


cannot throw plugin.media.abstractions.mediaPermissionException on thread 1

I have this problem, I've been trying for a few days and I can not solve it
This is the code
using System;
using Xamarin.Forms;
using Xamarin.Forms.Xaml;
using Plugin.Media;
using Plugin.Media.Abstractions;
namespace AplicacionZeus
{
[XamlCompilation(XamlCompilationOptions.Compile)]
public partial class Agregar : ContentPage
{
private ImageSource imageSource;
private MediaFile file;
    public Agregar()
    {
        InitializeComponent();

    }
    private async void Añadir_Clicked(object sender, EventArgs e)
    {

        await CrossMedia.Current.Initialize();

        if (CrossMedia.Current.IsCameraAvailable && CrossMedia.Current.IsTakePhotoSupported)
        {
            var source = await DisplayActionSheet("¿De donde desea tomar la imagen?", "Cancelar", null, "Galeria", "Camara");

            switch (source)
            {
                case "Cancelar":
                    file = null;
                    break;
                case "Galeria":
                    file = await CrossMedia.Current.PickPhotoAsync(new PickMediaOptions
                    {
                        PhotoSize = PhotoSize.Small
                    });
                    break;
                case "Camara":
                    file = await CrossMedia.Current.TakePhotoAsync(new StoreCameraMediaOptions
                    {
                        PhotoSize = PhotoSize.Small,
                        SaveToAlbum = true
                    });
                    break;
                default:
                    break;
            }
        }
        else
        {
            file = await CrossMedia.Current.PickPhotoAsync(new PickMediaOptions
            {
                PhotoSize = PhotoSize.Small
            });
        }
        if (file != null)
            imageSource = ImageSource.FromStream(() =>
            {
                var stream = file.GetStream();
                file.Dispose();
                return stream;
            });

    }
}

}

I already put the permissions in the app and even then I still get that error

I would need to see a stack trace post it to http://gist.github.com

this is the code https://gist.github.com/Eliezer10/117f684fd123dc67f84a1d5211bd851e


Why is the license printed 3 times in the runtime ... ?

I can see that the license is printed three times in a row in the runtime version. And I've spent the whole day trying to get the right runtime version up and running, and still have no »luck«, as guessing is all I seem to be able to do just now.
And this just made me wonder (as I saw the license three times), is this project going down somehow? Can it still be trusted in a production pipeline?

The license being printed 3 times might be related to #1288. Apart from that, what are the problems you are facing?
There has just been a patch release (4.0.6) in November 2016. The project is active and can be used in production (in fact, it is used by many projects).


1.19 <IP_ADDRESS> MCA Starting Screen

The mca starting screen has a habit of making the origins selection not appear, but then again my origins rn has a habit of not loading right anyways, i made another problem for that tho.

1.19(.0) is out of support. Update to 1.19.2


Adding Flash Reduction Config Options

Implementing INCLUDE_BACKTRACE optimization. Now actually removes unwind
tables from PrintBacktrace.
Added and implemented INCLUDE_VSNPRINTF, which will drop code size by
13kB. (hello_world @ Os => 7.5kB from previously 20.7kB)
Resolves #704

Coverage remained the same at 76.791% when pulling 1b78eb73d378548318c6b738cbb654d29c4cdc45 on optional-unwind-tables into d48805d7fe20236b035df0be147fafa093066267 on master.


Replace BackpressureBufferLastOperator with onBackpressureLatest in RxJava 1.1

BackpressureBufferLastOperator class copied from SqlBrite can be replaced with the native onBackpressureLatest() operator, whic is available in RxJava 1.1.
See related issue in SqlBrite
https://github.com/square/sqlbrite/issues/73

Hi Tomáš, thanks for bringing this up! Switching from the custom BackpressureBufferLastOperator to the onBackpressureLatest() operator seems like the right approach.
Do you want to take a stab at making the change, or would you like me to go ahead and switch this over?

Thanks. I could try it, but I'll be able to do it end of this week


EditorFor templates referencing from different mvc areas

Feedback from UserVoice site https://aspnet.uservoice.com/forums/41201-asp-net-mvc/suggestions/2080561-editorfor-templates-referencing-from-different-mvc
Please triage as needed.
Allow support for when using EditorFor that you can reference a template that is in a different area

We have this support - just pass in an app-relative path to the custom CSHTML template:
@Html.EditorForModel("~/Views/CustomTemplate.cshtml")


Fresh issued refresh token expired

Have latest IdentityServer4 with configured refresh token flow for client:
...
AllowOfflineAccess = true
UpdateAccessTokenClaimsOnRefresh = true,
AccessTokenLifetime = 120,
RefreshTokenUsage = TokenUsage.OneTimeOnly,
RefreshTokenExpiration = TokenExpiration.Absolute,
AbsoluteRefreshTokenLifetime = 300
...

Wanted to test it in short period of time, to ensure that everything works. Refresh token requesting every 60 seconds (half of access token lifetime), but at 5-th refresh token request (300 seconds) getting error that refresh token expired.
I expected that every new refresh token will have fresh expire time (300 seconds), but seems like even when i get new refresh tokens every time, they have inherited expiration time from first-issued refresh token.
Am i doing something wrong?

Just encountered post https://karatejb.blogspot.com/2019/09/aspnet-core-identity-server-4-refresh.html that explained little bit detailed refresh token behavior.
So next configuration did the trick:
...
AllowOfflineAccess = true
UpdateAccessTokenClaimsOnRefresh = true,
AccessTokenLifetime = 120,
RefreshTokenUsage = TokenUsage.OneTimeOnly,
RefreshTokenExpiration = TokenExpiration.Sliding,
SlidingRefreshTokenLifetime = 300,
AbsoluteRefreshTokenLifetime = 0
...


feat(perception_online_evaluator): add predicted path variance

Description

add predicted path variance and update documents

Tests performed

psim
unit test
1: [==========] Running 27 tests from 1 test suite.
1: [----------] Global test environment set-up.
1: [----------] 27 tests from EvalTest
1: [ RUN      ] EvalTest.testLateralDeviation_deviation0
1: [       OK ] EvalTest.testLateralDeviation_deviation0 (1638 ms)
1: [ RUN      ] EvalTest.testLateralDeviation_deviation1
1: [       OK ] EvalTest.testLateralDeviation_deviation1 (1634 ms)
1: [ RUN      ] EvalTest.testLateralDeviation_oscillation
1: [       OK ] EvalTest.testLateralDeviation_oscillation (2657 ms)
1: [ RUN      ] EvalTest.testLateralDeviation_distortion
1: [       OK ] EvalTest.testLateralDeviation_distortion (2652 ms)
1: [ RUN      ] EvalTest.testLateralDeviation_deviation0_PEDESTRIAN
1: [       OK ] EvalTest.testLateralDeviation_deviation0_PEDESTRIAN (1635 ms)
1: [ RUN      ] EvalTest.testYawDeviation_deviation0
1: [       OK ] EvalTest.testYawDeviation_deviation0 (1646 ms)
1: [ RUN      ] EvalTest.testYawDeviation_deviation1
1: [       OK ] EvalTest.testYawDeviation_deviation1 (1644 ms)
1: [ RUN      ] EvalTest.testYawDeviation_oscillation
1: [       OK ] EvalTest.testYawDeviation_oscillation (2672 ms)
1: [ RUN      ] EvalTest.testYawDeviation_distortion
1: [       OK ] EvalTest.testYawDeviation_distortion (2649 ms)
1: [ RUN      ] EvalTest.testYawDeviation_oscillation_rotate
1: [       OK ] EvalTest.testYawDeviation_oscillation_rotate (2645 ms)
1: [ RUN      ] EvalTest.testYawDeviation_distortion_rotate
1: [       OK ] EvalTest.testYawDeviation_distortion_rotate (2661 ms)
1: [ RUN      ] EvalTest.testYawDeviation_deviation0_PEDESTRIAN
1: [       OK ] EvalTest.testYawDeviation_deviation0_PEDESTRIAN (1642 ms)
1: [ RUN      ] EvalTest.testPredictedPathDeviation_deviation0
1: [       OK ] EvalTest.testPredictedPathDeviation_deviation0 (1635 ms)
1: [ RUN      ] EvalTest.testPredictedPathDeviation_deviation1
1: [       OK ] EvalTest.testPredictedPathDeviation_deviation1 (1637 ms)
1: [ RUN      ] EvalTest.testPredictedPathDeviation_deviation2
1: [       OK ] EvalTest.testPredictedPathDeviation_deviation2 (1639 ms)
1: [ RUN      ] EvalTest.testPredictedPathDeviation_deviation0_PEDESTRIAN
1: [       OK ] EvalTest.testPredictedPathDeviation_deviation0_PEDESTRIAN (1633 ms)
1: [ RUN      ] EvalTest.testPredictedPathDeviationVariance_deviation0
1: [       OK ] EvalTest.testPredictedPathDeviationVariance_deviation0 (1640 ms)
1: [ RUN      ] EvalTest.testPredictedPathDeviationVariance_deviation1
1: [       OK ] EvalTest.testPredictedPathDeviationVariance_deviation1 (1638 ms)
1: [ RUN      ] EvalTest.testPredictedPathDeviationVariance_deviationIncreasing
1: [       OK ] EvalTest.testPredictedPathDeviationVariance_deviationIncreasing (1634 ms)
1: [ RUN      ] EvalTest.testPredictedPathDeviationVariance_deviationOscillating
1: [       OK ] EvalTest.testPredictedPathDeviationVariance_deviationOscillating (1643 ms)
1: [ RUN      ] EvalTest.testYawRate_0
1: [       OK ] EvalTest.testYawRate_0 (1731 ms)
1: [ RUN      ] EvalTest.testYawRate_01
1: [       OK ] EvalTest.testYawRate_01 (3345 ms)
1: [ RUN      ] EvalTest.testYawRate_minus_01
1: [       OK ] EvalTest.testYawRate_minus_01 (3336 ms)
1: [ RUN      ] EvalTest.testYawRate_1
1: [       OK ] EvalTest.testYawRate_1 (3345 ms)
1: [ RUN      ] EvalTest.testYawRate_minus_1
1: [       OK ] EvalTest.testYawRate_minus_1 (3341 ms)
1: [ RUN      ] EvalTest.testYawRate_5
1: [       OK ] EvalTest.testYawRate_5 (3355 ms)
1: [ RUN      ] EvalTest.testYawRate_minus_5
1: [       OK ] EvalTest.testYawRate_minus_5 (3394 ms)
1: [----------] 27 tests from EvalTest (60723 ms total)
1:
1: [----------] Global test environment tear-down
1: [==========] 27 tests from 1 test suite ran. (60723 ms total)
1: [  PASSED  ] 27 tests.

Effects on system behavior

Not applicable.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@ktro2828
could you please review this PR?


Huggingface Accelerate

Hi all,
Thank you for the great work.
I was wondering if this library was compatible with Huggingface accelerate.prepare() for training?
Thank you,
Enrico

Hi @conceptofmind,
No, the library is not compatible with accelerate.prepare() since it would be required to somehow integrate it on their side. Maybe it's possible, but I'd need to discuss with accelerate maintainers if that's a possibility.
For now I'm closing this issue but if anything comes out of it feel free to reopen it!


fix: ignore pod patching failures during startup

Summary
Patch failures in pod_webhook.Start() abort the controller process and may result in crash looping.
We've experienced this on a test cluster, where we have 8 pods out of ~1100 that fail return an error on the "no op" patch attempt in Start().
As I believe the code within Start() is an optimization to eagerly create CachedImages, but otherwise not required to function properly, this patch will instead log the err and continue on to the next pod.
Patch Failure Details
Here's a sample of the patch failure:
The Pod "xxx-yyy-zzz" is invalid: spec: Forbidden: pod updates may not change fields other than `spec.containers[*].image`,`spec.initContainers[*].image`,`spec.activeDeadlineSeconds`,`spec.tolerations` (only additions to existing tolerations),`spec.terminationGracePeriodSeconds` (allow it to be set to 1 if it was previously negative)
  core.PodSpec{
  	... // 15 identical fields
  	Subdomain:         "",
  	SetHostnameAsFQDN: nil,
  	Affinity: &core.Affinity{
  		NodeAffinity: &core.NodeAffinity{
  			RequiredDuringSchedulingIgnoredDuringExecution: &core.NodeSelector{
  				NodeSelectorTerms: []core.NodeSelectorTerm{
  					{
  						MatchExpressions: []core.NodeSelectorRequirement{
  							{Key: "xxx/availability", Operator: "In", Values: {"on-demand"}},
  							{Key: "xxx/arch", Operator: "In", Values: {"amd64"}},
+ 							{Key: "xxx/availability", Operator: "In", Values: []string{"on-demand"}},
  						},

I've no idea why the patch logic occasionally is converting from Values: {"on-demand"} to Values: []string{"on-demand"} only on a subset of pods.  A first thought was the match expressions objects being reordered & sorted based on the Key, however I was unable to replicate the error.  This style of Affinity config exists in many other pods in the cluster.

Hello and thanks for your submission.
As you understood, the goal of this piece of code is to eagerly create CachedImages on start, and failing to do so is delaying the usefulness of kuik, which we would like to avoid. However, crashing at start is even worse, thus I can agree to merge this PR until we find a more acceptable solution. I just would like to ask you to rename your commit in order to follow the conventional commit spec, the title of this PR would fit perfectly.
It is very weird because it looks like the empty patch sometimes just randomly mutate the pod. I would be interested in finding out if a retry mechanism could solve the issue and more importantly understand why this happens. Since it is related to #371, I suggest that we continue the discussion there if you're willing to help us understand why this happens and how to properly fix it.
Thanks again, I'm waiting for you to reword your commit before merging.


Chained Builder-Classes with one setter each for compile-time checking

Would it be feasible to generated a partial Builder class for each field, with each Builder having one setter that returns the next Builder, so I can ensure on compile-time that all fields have been set? Also they would be always set in the same order in the code.

Unfortunately the annotation processor can't reliably tell what fields have defaults set (the code in the constructor is opaque without an ugly javac-specific hack that may not work in future versions).
What I would like to do is use said javac-specific hack to check all fields have been set at compile-time; if it fails, or if your code is more complex than "set all fields in order", you only lose that check, rather than having invalid code generated.
Thanks for the suggestion!
Chris


Automatic Release generation and tagging

In gitlab by @sylvaingaudan on Apr 26, 2017, 12:16

In gitlab by @sylvaingaudan on Apr 26, 2017, 14:01
added ~1918909 label

In gitlab by @sylvaingaudan on Apr 28, 2017, 13:46
changed milestone to %4

In gitlab by @sfalquier on May 9, 2017, 09:13
added ~1918367 and removed ~1918909 labels

In gitlab by @sfalquier on May 12, 2017, 11:22
added ~1918368 and removed ~1918367 labels

In gitlab by @sfalquier on May 12, 2017, 11:22
assigned to @sfalquier

In gitlab by @sfalquier on May 15, 2017, 13:44
closed

In gitlab by @sfalquier on May 15, 2017, 13:44
removed ~1918368 label


instanceof check throws TS2454 variable is used before being assigned

TypeScript Version:  2.2.1, 2.3.2
--strictNullChecks
Code
playground
class Foo {};

let foo: Foo;

if (foo instanceof Foo) { // Throws TS2454 variable foo is used before being assigned
    // foo is Foo
}

class Bar {};
let bar: Bar | undefined;

if (bar instanceof Bar) {
    // bar is Bar
}

Expected behavior:
Unassigned left hand variables in a instanceof check should not throw TS2454.
instanceof can return false when left hand unassigned variables, or null, or undefined
Actual behavior:
Throws TS2454 variable foo is used before being assigned
Inconsistent behavior when variable type is a union with undefined

Why write a condition which is always false? That doesn't make any sense

You make a good point. However, the use-case I discovered it in was using a try-catch-finally.
The variable declared outside of the try-catch-finally may have been assigned anytime inside the try, catch,  or finally blocks thus instanceof has a possibility of returning true.
example:
class Foo {
    doSomethingThatMayThrow() {
        throw new Error('whoops');
    }
    cleanup() {
        // perform some kind of cleanup
    }
};

let foo: Foo;

try {
    foo = new Foo();
    foo.doSomethingThatMayThrow();
} finally {
    if (foo instanceof Foo) { // Throws TS2454 variable foo is used before being assigned
        // foo is still Foo
        foo.cleanup();
    }
}

I have this block of code that throws the same error. Is this related to this bug or am I doing something wrong?
async function foo() {
  let bar: IBar

  try {
    bar = await Bar
      .query()
      ....
  } catch (err) {
  ...
  }

  ctx.body = bar // <- Throws variable 'bar' is used before being assigned. (TS2454)

@demisx the error is correct; it's possible bar doesn't get assigned

@RyanCavanaugh Thanks. I see. I've updated the example above with a little more detail inside the catch {...}. Please note, that if I uncomment throw new Error() in the catch block, TS understands that this block will never exit and the error goes away.
What would be the right way to help TS compiler understand that ctx.throw() also never exits? This is the method's signature, but TS still doesn't see it as never exiting:
throw(...properties: Array<number | string | {}>): never;

return ctx.throw('some error')

@RyanCavanaugh That's what I did. 👍

if foo is not assigned at declaration site, consider declaring it using the ! modifier:
let foo!: Foo;

try {
    foo = new Foo();
    foo.doSomethingThatMayThrow();
} finally {
    if (foo instanceof Foo) {
        // foo is still Foo
        foo.cleanup();
    }
}

Is this actually best practice for try catch or is there some better pattern? I'm not a huge fan of forcefully removing any subsequent null checks.
In my code, I used if (foo! ... instead, which also seems to work. But as im a complete novice, better solutions are welcome.
let foo: Foo;

try {
    foo = new Foo();
    foo.doSomethingThatMayThrow();
} finally {
    if (foo! instanceof Foo) {
        // foo is still Foo
        foo.cleanup();
    }
}

This does seem odd; notice the error even if the variable's existence is checked before use:
let foo: string

try {
  foo = 'bar'
} finally {
  if (foo) console.log('done')
  //  ^^^ Variable 'foo' is used before being assigned. ts(2454)
}

Reproduced here: https://www.typescriptlang.org/play?#code/DYUwLgBAZg9jBcEDOYBOBLAdgcwLACgC0BPCAbwImjggF4IByAIwENUGCBfaLF4YUhXxV0UCAApYMAJQQAxjExIYoAHTAY2cQwAmikA2lcCQA

This does seem odd; notice the error even if the variable's existence is checked before use:
let foo: string

I see, the type declaration doesn't allow for the variable to be undefined. So the fix is to allow it:
let foo: string | undefined

If you have some code that can't possibly throw, it doesn't make sense to put it inside the try in the first place. If you do have code that can possibly throw, then it's definitely the case that foo might not be initialized yet.


Relax docutils version constraint

Refs: https://github.com/vscode-restructuredtext/snooty-parser/pull/15#issuecomment-813737514

I'm a little nervous about this since we interface with docutils internals, but tests do pass with 0.17, so it's probably okay.

Thank you for the PR!


Running Raw query with separate Where produces wrong sql (bracket)

Running raw query with separate condition adds closing bracket ')' after raw query. Closing bracket also seems to be missing for Where.
What version of Go are you using (go version)?
go version go1.10.3 linux/amd64
Which database and its version are you using?
sqlite3
Please provide a complete runnable program to reproduce your issue.
*edited
func TestSeparateConditionWithRaw(t *testing.T) {
	user1 := User{Name: "arandomuser"}
	DB.Save(&user1)

	DB.LogMode(true)
	defer DB.LogMode(false)
	// Outputs:
	// select id, name from users where 1=1) AND (name = 'arandomuser' ORDER BY "users"."id" ASC LIMIT 1

	var user2 User
	DB.Raw("select id, name from users where 1=1").
		Where("name = ?", "arandomuser").
		First(&user2)

	if user1.Id != user2.Id {
		t.Errorf("Wrong data returned")
	}
}

Similar/related

https://github.com/jinzhu/gorm/issues/1214
https://github.com/jinzhu/gorm/commit/f2fe351aa05655c528b139a3d28c519fcea9d429

I don't know if this was ever supposed to be used this way but if it can be done then I can't see any reason why not. Below fixes this:
func (scope *Scope) CombinedConditionSql() string {
	joinSQL := scope.joinsSQL()
	whereSQL := scope.whereSQL()

	if scope.Search.raw {
		// moved to scope.whereSQL()
		// whereSQL = strings.TrimSuffix(strings.TrimPrefix(whereSQL, "WHERE ("), ")")
	}

	return joinSQL + whereSQL + scope.groupSQL() +
		scope.havingSQL() + scope.orderSQL() + scope.limitAndOffsetSQL()
}

func (scope *Scope) whereSQL() (sql string) {
	...
	for index, clause := range scope.Search.whereConditions {
		if sql := scope.buildCondition(clause, true); sql != "" {
			if scope.Search.raw && index == 0 {
				andConditions = append(andConditions,
					strings.TrimSuffix(strings.TrimPrefix(sql, "("), ")"))
			} else {
				andConditions = append(andConditions, sql)
			}
		}
	}
	...
	} else if len(combinedSQL) > 0 {
		if scope.Search.raw {
			sql = combinedSQL
		} else {
			sql = "WHERE " + combinedSQL
		}
	}
	return

+1

I think using Raw and Where Order by etc. are necessary but I mainly use Raw for joining tables, and I would rather to use Where instead of unnecessary and complex function writing that will combine them also Where takes care of sql injection etc. also this function may be deprecated if we use scope etc. and we may have problem after updates so please consider offically supporting Raw, Where, Order By, Limit etc

Why is it closed? Support Ed in V2?

Raw is not expected to work with GORM API, check out http://v2.gorm.io/docs/sql_builder.html, I will update the document to make it clear
Thank you.


Create repository for core-provider-rke2

Our current builds package the provider binaries and rke2 binaries into a single image for all Kairos image flavors. Its beneficial to package just the provider binary aloing with any scripts/deps it needs in a separate 'core' image which can used by users wanting to use the provider with their own custom OS
Proposal is to create a docker repo - provider-rke2

See: https://github.com/kairos-io/kairos/issues/1010


Nexrender-Server: job is being taken many times by multiple nexrender-workers

Describe the bug
I have 8 workers that are connected to the same nexrender-server.
Once a job is created, 3 workers took it and render it at the same time instead of 1.
Information about environment

Which version of nexrender are you using? Latest
What kind of the setup do you have? Nexrender-server - ubuntu, 8 Nexrender-workers Windows 2018.
Are you using pre-built binaries or globally installed npm packages? NO

Expected behavior
Once a worker is getting the job, The next time jobs are being fetched by the worker, this particular job should not exist.
Screenshots

Apparently a kind of "data race" is happening. We need to rethink the way jobs are distributed. If I remember correctly currently jobs are selected on the worker side, which is the root of the issue, we would probably need to move selection process to the server.

I saw that at the nexrender-worker in the file index.js, you were trying to update the job state multiple times.
Maybe a good solution is to move
await client.updateJob(job.uid, job)
into the nextJob function right after picking up a random job, inside this if:
if (queued.length > 0) {            return queued[Math.floor(Math.random() * queued.length)]; }
Jonathan.

I believe if that even improves the situation a bit, it would still not fix the issue fully. Since there is some time while data is being sent, parsed and received. And there is a chance that all workers would ask for a job at exact same time, and they all would receive a list of jobs, and even at random, they would select the same job.

I'm using your project a lot lately since it will probably become some integral part of a startup project I'm trying to get going. I think I noticed the race condition, too, a few times. I support your idea to move the job distribution to the server. The server could ask which workers are free at the moment and then select one to assign the job to. I think that is what you had in mind, right?
Thank you for the work that you have already put into this neat tool. :-)
Frank


audit-avt1-guidelines-icons: form button needs label

Environment:

Operating system: Mac
Browser: Chrome
AVT :one:
Page: Guidelines - Icons: Library
This bug affects about 95 other issues

Detailed Description:
Form control element  has no associated label

What to do:
Add a for attribute to a visible label with the id of this control (e.g. );
Why is this important?
Associating a meaningful label with every UI control allows the browser and assistive technology to expose and announce the control to a user.

Fixed in monorepo: https://github.com/carbon-design-system/carbon/pull/7812

Fixed in PR: https://github.com/carbon-design-system/carbon/pull/7814


Atomic cross-chain swap between public and private chain

Hello there
Thank for the great documentation !
There is an error here : https://docs.symbolplatform.com/guides/secretlock/atomic-cross-chain-swap-between-NEM-public-and-private-chain.html
You said :
alice.token is defined in Symbol’s public chain
bob.token is defined in Symbol’s private chain
But Tx1 blocks 10 alice.token in the private chain, however alice.token isn't define in the private chain. So it's imposible.
(other transactions also don't work)
For the atomic swap will be succes, alice.token will be in the private chain and bob.token will be in the public chain.
Best !

Some of the pages have not been reviewed in a while and might contain errors. I've added this page to the scheduled global review.
Thanks for the report!


added animations to 'submit' and 'learn more' buttons

Pull Request Template

Please provide all of the details below. Pull requests missing any of the information below will be closed!
Providing the issue number/url helps us identify the problem being solved and keep the issues tab clean and tidy.

Summarize what this PR fixes:-
#fixes #561
This is not a fix. This is a request to add animations to the 'submit' and 'learn more' buttons.
Checklist
(Add an x between the brackets to check the items)

[x] I have summarize what this PR fixes above
[x] I have provided the issue ID/URL above
[x] I have not added Bootstrap or jQuery and used correct styling guide.
[x] This PR fixes only the issue mentioned above (create separate PRs for separate issues)
[x] I was the first one to claim the issue or I created the issue - I created the issue

I like these changes.


[Feature Request] Append/Prepend text to a specific note without opening Obsidian

Hello again @Vinzent03 !!!!!!
I'd like to make a request related to these amazing URI:
obsidian://advanced-uri?vault=<my_vault>&filepath=<my_file.md>&data=<my_data>&mode=prepend
obsidian://advanced-uri?vault=<my_vault>&filepath=<my_file.md>&data=<my_data>&mode=append
My suggestion is a new option to NOT open Obsidian while doing this URI.
In other words: append/prepend text to a specific note without opening Obsidian (background)
Why are you asking this?
Let me explain :)

This week I started using Obsidian Mobile on iOS;
I have a note in Obsidian called INP.md that I use to put everything that comes to my mind that I don't want to deal in the moment (TODO, reminders, links, short text ...)

My idea is to send (append/prepend) information to INP.md on Mobile using URI and the Shortcuts App. After searching in our forum, I wrote two shortcuts that uses your amazing plug-in to append/prepend information to this INP.md note.
—> This one uses the clipboard to prepend to the note

—> This one opens a modal that let me write what I want to remember and prepend to the note

This is amazing, but every new input that I send via URI, Obsidian Opens and I have to get back to the previous App that I was using.
Would be amazing to have a way to send information via URI on background (without opening Obsidian) :)

I hope my FR is feasible and I hope you liked :)
Thanks again for all your help. Is always a pleasure talking to you.
I wish you a fantastic day!!!!!!

Closing for now, because (as discussed in Discord) Obsidian doesn't provide an API for that. But they might not be able to implement that too. Some workarounds were discussed in Discord

100% agree @Vinzent03 !
Thank you very much for all your help. Lets hope that this could be a new feature in the future :)

⚠Update about this Issue:
I found a way to do what is described here using an iOS App called ToolboxPro.
This post in our Forum explain what you need to do.
Have a great day!


V0.1 release

Coverage decreased (-1.3%) to 89.745% when pulling 961487abf73a9a52dca3606164a6c9743e0cb26e on V0.1 into 1f7cbfc70da67aaebb4f663b0765c35c0a10f141 on master.


arch/arm: migrate to SPDX identifier

Summary
Most tools used for compliance and SBOM generation use SPDX identifiers This change brings us a step closer to an easy SBOM generation.
Impact
SBOM
Testing
CI

[Experimental Bot, please feedback here]
No, this PR description does not adequately meet the NuttX requirements. While it touches on some points, it lacks crucial details.
Here's why and what's missing:

Summary:  While it states the overall goal (SPDX identifiers for SBOM generation), it lacks specifics.

Missing: Which tools are being referred to?  What functional part of the code is changing (e.g., build system, kernel configuration, specific drivers)?  How are SPDX identifiers being added (e.g., new Kconfig options, source code annotations)?  A related NuttX issue number should be included if one exists.

Impact: Simply stating "SBOM" is insufficient.

Missing:  All the "Impact on..." sections need to be addressed explicitly with "YES" or "NO" and a description if "YES." For example, if no user changes are needed, explicitly state "Impact on user: NO". If the build system changes, state "Impact on build: YES (New build options are added to enable SPDX identifier generation)."

Testing: "CI" is not enough.

Missing:  Specify the host OS, CPU architecture, and compiler used for testing.  List the target architectures and boards tested.  Provide actual log output before and after the change, demonstrating the difference and verifying the intended functionality.  If the change only impacts the build system, show the relevant build output.  If it affects runtime behavior, show the relevant runtime logs.

Example of a better summary:

This change adds SPDX license identifiers to the Kconfig files for core NuttX components to facilitate SBOM generation.  Tools like [tool name] and [tool name] can now correctly identify the licenses of these components. This modifies the Kconfig files in the nuttx/ directory and adds a new configuration option CONFIG_SW_SPDX.

Example of a better impact description:

Impact on user: YES (Users will need to enable CONFIG_SW_SPDX to include SPDX identifiers in the build output.)
Impact on build: YES (The build system now generates SPDX tag output when CONFIG_SW_SPDX is enabled).
Impact on hardware: NO
Impact on documentation: YES (The Kconfig documentation and user guide will be updated to explain the new option and its usage.)
Impact on security: NO
Impact on compatibility: NO
Anything else to consider:  This change is a first step. Future PRs will add SPDX identifiers to other parts of the codebase.

Example of a better testing section:

I confirm that changes are verified on local setup and works as intended:

Build Host(s): Linux (Ubuntu 20.04), x86_64, GCC 9.4.0
Target(s): sim:qemu-x86_64, stm32f4discovery:nsh

Testing logs before change (build output with CONFIG_SW_SPDX=n):
... (relevant build output showing no SPDX info) ...

Testing logs after change (build output with CONFIG_SW_SPDX=y):
... (relevant build output showing SPDX tags) ...

@xiaoxiang781216 please ignore nxstyle errors


Check for any AOP-Proxy in ConfigurationPropertiesRebinder

Fixes #288 - PostProcessor getting applied more than once to the same bean.
#288 Might need more research, this is my naive fix of the problem. Please review carefully.

@Dav1dde Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@Dav1dde Thank you for signing the Contributor License Agreement!


Unsupported for vue3

When the component tried to start in vue3 it reports this error.

https://github.com/intlify/vue-i18n-next/issues/305
this commit gives hint for this problem

Is support for Vue 3 in the works? Awesome visualisation and would be great to see support for Vue 3 released

@brms2021 , a Vue 3 fork (partial functionalities) : https://github.com/la-fabrique/vue-d3-network-graph

When do you plan to add support for D3?

A lib é ótim, poderia ter par vue3


Fix: Unable to set unserialised objects using store value

Description
Fix: Unable to set unserialised objects using store value
PR fixes following issue(s)
Fixes #23201

if no issue exists, please create an issue and ask the maintainers about this first

Type of change

Bug fix (non-breaking change which fixes an issue)

Testing
How Has This Been Tested?

[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/build-deploy-preview skip-tests=true

/ok-to-test

/ok-to-test

/build-deploy-preview skip-tests=true

We need to show an error message when user tried to save an unserialized object using store value. Will pick this up in the next sprint.


Underlined Textbox with Icon renders Icon incorrectly, shows 3 dots

Hi. Thanks for this awesome framework!
I've recently upgraded my project's dependencies and I think InputGroup with Icon is rendering incorrectly. In this screenshot, you see two InputGroups and a Button with an Icon. The button with the icon renders correctly (I'm using FontAwesome as the theme icon family). However, the text inputs are not rendering their icons.
The Android project references the react-native-icons gradle subproject to copy the icons, and the VectorIconsPackage is referenced from MainApplication.
Am I missing something? These are my NPM dependencies. Thanks!
  "dependencies": {
    "color": "0.11.3",
    "native-base": 0.5.7",
    "react": "15.3.1",
    "react-native": "0.32.0",
    "react-native-vector-icons": "2.1.0"
  },

<InputGroup style={styles.login}>
  <Icon name="user" style={styles.whiteFont} />
  <Input placeholder="Username" style={styles.whiteFont}
    value={this.state.username}
    onChangeText={(txt) => { this.onUser(txt); }} />
</InputGroup>
...
<View style={styles.login}>
  <Button block onPress={() => { this.onLogin(); }}>
    <Icon name="lock" style={styles.whiteFont} />
  </Button>
</View>

I have this problem too, but on device (tested on Android) works well.

Did you try checking with react-native-vector-icons? It might be related to that.

@sankhadeeproy007 while it would be helpful to get feedback from the upstream project, the example code shows two different places where the Icon component is rendering. One works (the button), the other one doesn't (the input group), using the exact same icon definition. This tells me that there might be some unexpected behavior when the Icon component renders as a part of the input group.
Hope this helps. Thanks!

It is very hard to reproduce this without further details. Can you try it on a clean page with out any other dependencies?

Same problem, i actually filed this in react-native-vector-icons https://github.com/oblador/react-native-vector-icons/issues/294

Can you please try with different icons and check if the result is the same?

code works well in Android 6.0 device.
But in Android 7.1.1 device showing ... (dots) see attached screen

When i place same icon outside of input group it showing without any issue.
Using:
"react": "15.3.2"
"react-native": "0.32.0"
"native-base": "0.5.8"

I'm having this issue as well.

It renders correctly on my phone, just shows the three dots in the emulator. Strange.

Try with a lower fontSize for the icons please.

@sankhadeeproy007 I did try this. It just makes the three dots smaller.
<Icon name='ios-person-outline' style={{ color: '#fff', fontSize: 10 }} />
produces the following:

Try out this solution then?

No luck. The issue seems to be something within NativeBase's styling for InputGroup. My current code is:
<InputGroup>
  <Icon name='ios-person-outline' style={{ color: '#fff' }} />
  <Input
    value={this.state.username}
    style={styles.loginText}
    placeholder='Email'
    placeholderTextColor='#ffffff'
    onChangeText={(text) => this.setState({username: text})} />
</InputGroup>

Changing the InputGroup element to View renders the icon correctly. I tried setting a width and height to InputGroup, but results were the same.


jbig2dec cross fix

Description of changes
Finalize fix cross-compilation from #266254
In #266254 we set configureScript variable, but then we need remove manual autogen.sh invocation as well
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

CC @Mindavi @amjoseph-nixpkgs
Consider backport of #266254 and this to staging-23.11 as well. (I have ongoing branch for it)


sickchill supported?

I am having an issue with sickchill and deluged working together I never get pasted the conversion process where it says the dir cannot be deleted.
2021-06-05 09:10:12 - DelugePostProcess - INFO - Passing /var/lib/mythtv/videos/Flix/tvtime/xxxxxxx.mkv-convert directory to Sickbeard.
2021-06-05 09:10:12 - autoprocess.autoProcessTV - INFO - Opening URL: http://localhost:8081/home/postprocess/processEpisode?quiet=1&dir=%2Fvar%2Flib%2Fmythtv%2Fvideos%2FFlix%2Ftvtime%2FSxxxxxxxxxx.mkv-convert.
2021-06-05 09:10:12 - autoprocess.autoProcessTV - INFO - b''
2021-06-05 09:10:12 - DelugePostProcess - ERROR - Unable to delete temporary directory.
Traceback (most recent call last):
File "/home/owner/.sickchill/sickbeard_mp4_automator/delugePostProcess.py", line 172, in
os.rmdir(delete_dir)
OSError: [Errno 39] Directory not empty: '/var/lib/mythtv/videos/Flix/tvtime/xxxxxxx.mkv-convert'
currently tried sickrage and sickbeard settings same info in the logs!
xxxxxxxxxx presented to hide the name of conversion file

Sorry, running Debian with python3/pip3

Sickchill should be supported unless they changed something or deviated from Sickrage's API style. The directory error isn't a big deal and isn't breaking things. I'm guessing the API call isn't working and it looks like you're not getting any return from the API call
2021-06-05 09:10:12 - autoprocess.autoProcessTV - INFO - b''
I looks like you're routing this through Sickbeard which used a different API, try using this like its Sickrage instead

Trying it now,

Great it worked using sickrage, but I still get this......
2021-06-07 11:29:03 - SickbeardPostProcess - ERROR - Couldn't refresh Sickbeard, check your autoProcess.ini settings.
Traceback (most recent call last):
File "//home//owner//.sickchill//sickbeard_mp4_automator//postSickbeard.py", line 47, in
refresh = json.load(urlopen(sickbeard_url))
File "//usr/lib/python2.7/json/init.py", line 291, in load
**kw)
File "//usr/lib/python2.7/json/init.py", line 339, in loads
return _default_decoder.decode(s)
File "//usr/lib/python2.7/json/decoder.py", line 364, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "//usr/lib/python2.7/json/decoder.py", line 382, in raw_decode
raise ValueError("No JSON object could be decoded")
ValueError: No JSON object could be decoded

So it gets a little more tricky here since Sickchill and Sickrage sort of exist in this half compatible state with Sickbeard
autoprocessTV uses sickbeard settings, autoprocessTVSR uses sickrage settings
7d15b5aacdc21a1277e2e23f8d1f166bfc90691d
This patch I just uploaded should allow postSickbeard.py to fallback to sickrage settings if sickbeard fails allowing it to work with both programs
The old solution would have been to just include your Sickchill settings in both the Sickrage and Sickbeard section of autoProcess.ini (that would also probably still work but please test out this new update)

Nothing happened so far blank log!

Not sure what to make of that, does that mean its working or failing

Sorry it finally finished, with no error messages, don't know what the delay was about!

Good stuff. Going to close this then


Initial support for Azure LLM nodes.

There are still some rough edges due to underlying langchain and
openai API limitations, e.g. hwchase17/langchain#3769 and
openai/openai-python#411. Notably, you can't use the Azure and
non-Azure node types in the same server, since there's global openai
configuration needed to choose between the two. So it's probably best
to still leave the Azure node types commented out in the default
config. But with this PR, if you uncomment those nodes and start the
server with OPENAI_API_TYPE=azure, you will have working Azure nodes.
Fixes #332.

If that variable changes in between chain executions it still does not work?

hi, I install the latest version 0.0.87, but I can't the Azure LLM under the LLMs
how should I test the PR feature?


Import url cleanliness

Go supports Vanity urls for mod imports.  This would enable us to do something to the tune of:
import (
    github.com/Azure/GoSDK/core
    github.com/Azure/GoSDK/storage/blobs
    github.com/Azure/GoSDK/keyvault/keys
    ...
    )

Among other repo considerations, its worth considering a cleaner import story.

If we go this route I would also suggest that the URL be all lower-case characters as this has impact on systems with case-sensitive file systems (the capital 'A' in "Azure" is a real bummer).

What about something like go.azure.com/core ?


How to convert empty interface to other interface? the empty interface is just return value of functions.

What did you expect to see?
v = types.Types.CreateType(typename)
return Client(v)
which  types.Types.CreateType returns interface{} and Client is an interface
What did you see instead?
cannot convert v (type interface {}) to type Client:
interface {} does not implement Client (missing CreateClient method)

We don't the issue tracker to ask questions. Please see https://golang.org/wiki/Questions for good places to ask. Thanks.


Uncertainty calculation without Bayesian Approach

We should also implement an uncertainty calculation that does not use the Bayesian approach. To do this we can take the negative inverse hessian (which is given by the fit) as the covariance matrix. Then, to find the sigma errors, we just take the sqrt of the diagonals :)
http://bebi103.caltech.edu.s3-website-us-east-1.amazonaws.com/2018/tutorials/t6b_parameter_estimation_by_optimization.html

We also then need to output these

To do this, we need to calculate the Hessian of our minimization function (i.e. the likelihood function we have adopted). We can either do this analytically (gross) or use a numerical differentiation technique such as the python numdifftool grad/jac/hess functionality. Then, we take the negative inverse of the Hessian to calculate the covariance matrix. Once we have the covariance matrix, we can calculate the 1-sigma errors by taking the square root of the diagonals.


Requirements.txt is out of date.

the requirements.txt says that you require urllib3 1.x where in fact you are relying on urllib3 2.x. the json function only works urllib3 version 2.

Great catch, fixed in https://github.com/sethmlarson/pypi-data/commit/5df30e7c24732bf46de5cf134e69e226b8f64b00


Add build_overrides/pigweed_environment.gni

Problem
A fresh build complains missing build_overrides/pigweed_environment.gni file.
Change overview
Add an empty file to satisfy it.
Testing
Not needed.

Re-opening this. Looks like a good fix, why not take this?

Why was this generated file checked in?


Introduces a service registry for the incrementalSignal service.

This fixes an issue since, if using modern plugin syntax, plugins run in
their own class loader, meaning that the same class can't be used across
projects. This breaks the incremental signal up per project, using a
WorkerServiceRegistry so that it can be accessed by multiple tasks
within a project.
Rationale
I suspect that the issue revealed in #31 is related to Gradle's classloader isolation.  The fact that it works if you add the plugin as a classpath dependency in the buildscript, but doesn't if you add it using modern plugin syntax suggests that to be the case.
I believe that since the plugin is attempting to access the incremental signal from the KotlinCompile task, which uses an isolated classpath (my assumption), it receives an IncrementalSignal from a different classloader, which means it can't use the class.
This approach is modeled that used by aapt in the Android gradle plugin. Instead of using a shared BuildService that's accessed from multiple projects and tasks, this uses the build service to register a simple IncrementalSignalService with a WorkerActionServiceRegistry.  This allows the "service" to be shared among different tasks.
Testing
This is a bit of a pain to test, but I've set up a git repository for it.

Check out the test repository.
Run ./gradlew assemble
Notice the build fails with the following message:

Could not determine the dependencies of task ':module1:compileKotlin'.
> Could not create task ':module1:compileKotlinCheckIncrementalCompilationHephaestus'.
   > Cannot set the value of task ':module1:compileKotlinCheckIncrementalCompilationHephaestus' property 'incrementalSignal' of type com.squareup.hephaestus.plugin.IncrementalSignal using a provider of type com.squareup.hephaestus.plugin.IncrementalSignal.

Update the version of this branch in gradle.properties to 1.0.4-31-SNAPSHOT, build and deploy to maven local.
Switch the plugin imports in the test project's :app, :module1 and :module2 build files to comment the 1.0.3 version and uncomment the 1.0.4-31-SNAPSHOT variant. (IDEA "Replace in path" makes this much easier)
Run the build again with ./gradlew assemble.
The build now succeeds.

Fixes #31

 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.


UX Improvments

Also moved confetti on submit of "Complete Order" as sachin mentioned, they are expecting it to be "done" and may navigate away by that point just like you said, they want to see the tx on etherscan so we will show confetti right away as it is in the process of completing at that point already.

https://www.dropbox.com/s/mv0vatm59my3nb4/Screen Recording 2023-11-13 at 1.57.49 AM.mov?dl=0

Dooooooope:

Can you post screens, I want to see the block explorer link
Would suggest that instead of waiting to set the transaction on success, you set it on submit so they can see it while its mining
Put the confetti behind a feature flag

Screen recording:
https://www.dropbox.com/s/mv0vatm59my3nb4/Screen Recording 2023-11-13 at 1.57.49 AM.mov?dl=0

LGTM


Enforce PEP 570-style positional only syntax

flake8-pyi should forbid old-style positional only syntax. I.e., instead of def foo(__bar): ... the following should be used: def foo(bar, /): ....
It's too early to implement this at this moment, but I opened this issue so we remember it. See python/typeshed#4972 for current type checker support.

I really wish we could implement this feature now ☹ I find the __ syntax incredibly ugly, and it made the stubs look so weird to me when I first started contributing to typeshed (just because of how different it looks to anything you see in normal Python).


Generate a button to see changed documentation

I just found out about an awesome little feature that the sklearn devs built for their documentation.
They've configured their circle file to generate a folder with a list of files that (they think) have changed, and puts it in a folder.
https://github.com/scikit-learn/scikit-learn/blob/master/build_tools/circle/build_doc.sh#L134
and they have a tampermonkey/greasemonkey script that looks for that folder and generates a button that gives you a list of links to the changed HTML pages in the docs:
https://gist.github.com/lesteve/470170f288884ec052bcf4bc4ffe958a
It seems like this wouldn't be too tough to implement in MNE since we're already building with circle...could be a nifty feature to have for documentation improvements. WDYT @Eric89GXL ?

+1 would be cool to have
I'm not motivated to do it currently, but maybe someday :)

Closing as YAGNI but if someone is motivated feel free


Handle result in any OBX Set ID

Problem statement
Currently ReportStream requires that the COVID-19 result appears in the first OBX instance (typically Set ID = 1). Providence results may appear in any Set ID. While atypical for our senders, this is valid and allowed by the HL7 spec. So we need to be able to accommodate this.
This is a blocker for full onboarding of Providence.
What you need to know
Examples are in keybase under Providence.

sender_info/Providence/2022-08-26/ELR_Raw_message.txt
sender_info/Providence/2022-08-26/POC_Raw_message.txt

Note this samples will generate some other warning and filter issues. The related warning is "The code N for field OBX-5 is invalid."
You can also reorder the OBX segments in a working message to achieve the same outcome.
Acceptance criteria

[ ] Message with a result appearing in an OBX instance other than the first is successful.

Providence is currently on hold, could engrs shadow Joel/Jorge on this? cc: @BerniXiongA6

Met with Brick, James, and Ott today to go over findings so far and potential solution paths. The testing and research modifications are in the onboarding-ops/6733/results-any-obx branch.

have the coding completed - writing tests and commenting


activate event for vscode extension

VSCode Version: 1.37.0
OS Version: win10

Steps to Reproduce:
I am developing a lisp language extension based on Vscode. In my app I calls the "CreateProcess" windows API to launch the vscode, and then I want to pass some information to the extension, I see this page:
https://code.visualstudio.com/api/references/activation-events#onUri
I want to bring some information in the uri:
"activationEvents": [
"onUri"
]
It can provide call back for the "onUri" in extension side?
Or there is other way to pass information to extension from another process?

Does this issue occur when all extensions are disabled?: Yes/No

The slack is not good, no one answers the questions


A number of small additions and features to the drawing code

Includes:

progress on #773
including legends
wider wedges
better dashes
fixes to getAtomCoords()

The fixes look benign.  Would you like me to look at a ton of molecules to validate?  I'm partly assuming this could be considered a work in progress, and the results I've seen certainly look better so I'm happy merging.

It's definitely a work in progress. So I think it's fine to merge as is. I'll keep looking at molecules and tweaking things.

I'm happy merging, was there anything else we wanted to do first?

I think it's now ready to merge.

I'll do a sanity build on the Mac and merge.

@bp-kelley : did that sanity build go ok?


MRG: Allow reading MaxShield evoked files

Allows reading Evoked files that have MaxShield-type data in them, such as the ones that are produced during acquisition.
Ready for review/merge from my end.

Current coverage is 87.31% (diff: 95.74%)

Merging #3594 into master will increase coverage by 0.01%

@@             master      #3594   diff @@
==========================================
  Files           341        341
  Lines         60372      60397    +25
  Methods           0          0
  Messages          0          0
  Branches       9229       9234     +5
==========================================
+ Hits          52705      52733    +28
+ Misses         4915       4913     -2
+ Partials       2752       2751     -1

Powered by Codecov. Last update 97861e9...a6ee4d4

thx @Eric89GXL !


SonarCloud Quality Gate Code duplication rule too restrictive

SonarCloud Quality Gate failed.
  0 Bugs
  0 Vulnerabilities
  0 Security Hotspots
  14 Code Smells
 No Coverage information
 9.0% Duplication
Originally posted by @sonarcloud[bot] in https://github.com/odpi/egeria-connector-ibm-information-server/issues/743#issuecomment-1202728322

Checking for code duplication is most of the times very useful. However, also many times it is not easy to configure rule on how duplication is detected for given context and coding style.
Some of the Egeria frameworks and patterns in the code especially when writing connector are contradicting this to some extend and code gets duplicated. Example is where we are writing two connector provides/connectors to achieve same goal in different way. In cases like this we have to think about code duplication for limited scope (connector) and not across all connectors.
We are trying to keep this practice to minimum but there will be cases where duplicating code is just fine.
Default code duplication quality check we are using at the moment is <=3%. This is too restrictive and it is blocking us at the moment.
We need either to find smarter way to configure the rule (not sure if tools support this) or just simply increase the duplication threshold for certain repos.
any other thoughts?
cc: @planetf1

Agree with your summary
Some work is needed to add the GitHub action to run the scan on the repositories we are interested in. I've been doing this before vacation for the new HMS connector, though have a glitch to resolve. Once done we can adopt that pattern across our other repos that are not currently fully scanning (including this one)

I opened https://github.com/odpi/egeria/issues/6805 to track getting our sonar scan working across all repos


build: generated TAG from git-describe.

Automated tag-generation in Makefile. We can skip update TAG in Makefile in the release process.

git describe is possible to get old tag by mistake. Because git describe cannot get latest tag tagged on different branch. (e.g. master or develop)


how to fix this timed out issue?

➜  docker-magento2 git:(master) docker-compose up
Starting dockermagento2_dbdata_1
Starting dockermagento2_cache_1
Starting dockermagento2_clusterdata_1

ERROR: for db  UnixHTTPConnectionPool(host='localhost', port=None): Read timed out. (read timeout=60)
ERROR: An HTTP request took too long to complete. Retry with --verbose to obtain debug information.
If you encounter this issue regularly because of slow network conditions, consider setting COMPOSE_HTTP_TIMEOUT to a higher value (current value: 60).

Can you rerun it with --verbose?  It doesn't look like it's a problem with the images in this repository, rather a problem with your docker daemon as it's docker-compose that's reporting a problem.

 Retry with --verbose to obtain debug information.
If you encounter this issue regularly because of slow network conditions, consider setting COMPOSE_HTTP_TIMEOUT to a higher value (current value: 60).

This doesn't look like a problem with the images in this repository.  It looks like it's failing to boot your db container for some reason, but I'm not really sure.  I recommend you try updating docker, update docker-compose, or try using the mariadb image on its own or in another project to rule out a problem there.


Fix broken link

just fixes a broken link.
This attributes thing looks really cool 🎉

Thank you very much! 👍 😄


[1379] Fixed Pending Reservations link on Equipment Model page [v5.5]

Resolves #1379

merging based on review of #1387


urgent problems!-'user already taken' without sending account-verify-mails

I am using yii2-user to send account-verify-mails for my yii2-advanced project. However, it always seemed strange , as I config my 'mailer' part in common/config.php :

    'mailer' => [ // within my 'components' part

        'class' => 'yii\swiftmailer\Mailer',
        'useFileTransport' => false,
        'messageConfig' => [
            'from' =><EMAIL_ADDRESS>=> 'sender_name'],
            'charset' => 'UTF-8',
        ],
        'transport' => [
            'class' => 'Swift_SmtpTransport',
            'host' => 'smtp.163.com',
            'username' =><EMAIL_ADDRESS>            'password' => 'my_password_here',
            'port' => '25',
            'encryption' => 'tls',
        ]
    ],

and in dektrium codes,

    protected function sendMessage($to, $subject, $view, $params = [])

{
    $mailer = \Yii::$app->mailer;
    $mailer->viewPath = $this->viewPath;
    $mailer->getView()->theme = \Yii::$app->view->theme;
    if ($this->sender === null) {
        $this->sender = isset(\Yii::$app->params['adminEmail']) ? \Yii::$app->params['adminEmail']<EMAIL_ADDRESS>    }

    return $mailer->compose(['html' => $view, 'text' => 'text/' . $view], $params)
        ->setTo($to)
        ->setFrom($this->sender)
        ->setSubject($subject)
        ->send();  }

}
I also config 'adminEmail'  as<EMAIL_ADDRESS>within common/main.php . but the page just showed

 'user already taken'

and I receive NO mails from the project.
Is there anything to be really careful when using yii2-user with yii2-advanced template? thanks a lot!

Probably something is wrong with your smtp connection configuration. There should be no problems with using yii2-user with advanced template.


Need some music

We need to jam out to our game!

Added the music to our game, starts and stops accordingly!
9de48638caa1bdfdf60c5448e968d8f878a1ba27


Setting event hub policy

It is no longer possible to set an event hub policy other than "RootManageSharedAccessKey" which is set as a default. This is a security risk as it effectively provides admin rights on the namespace and event hub. Can you please fix?

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

ID: 0ca576e1-fafa-3a02-6d29-394f342bc737
Version Independent ID: a6a83de5-41fd-2dce-eb86-cee6c086c07d
Content: Create diagnostic settings to send platform logs and metrics to different destinations - Azure Monitor
Content Source: articles/azure-monitor/platform/diagnostic-settings.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @bwren
Microsoft Alias: bwren

@scunge6006 Thanks for your valuable feedback. When diagnostic settings are enabled to send logs to Event Hub, diagnostic settings creates event hubs in the event hub namespace if they do not exist. Therefore, it needs manage permissions to create them.
As this is more of feature request than documentation change, I would recommend you to navigate here and share your feedback or suggestions directly with the responsible Azure feature team and clicking the vote button of your suggestion to raise visibility and priority on it.
Hope this helps ! We will close the issue for now. If there are further questions , please revert and we will be glad to assist you.

Thanks for the update and suggestion. I will follow up with contacting the Azure team.
Miles
From: SwathiDhanwada-MSFT<EMAIL_ADDRESS>Sent: Wednesday, 1 July 2020 7:28 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Scott Hill, Miles<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Setting event hub policy (#57827)
[External Email] This email was sent from outside the organisation – be cautious, particularly with links and attachments.
@scunge6006https://github.com/scunge6006 Thanks for your valuable feedback. When diagnostic settings are enabled to send logs to Event Hub, diagnostic settings creates event hubs in the event hub namespace if they do not exist. Therefore, it needs manage permissions to create them.
As this is more of feature request than documentation change, I would recommend you to navigate herehttps://feedback.azure.com/forums/267889-azure-monitor-log-analytics and share your feedback or suggestions directly with the responsible Azure feature team and clicking the vote button of your suggestion to raise visibility and priority on it.
Hope this helps ! We will close the issue for now. If there are further questions , please revert and we will be glad to assist you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/57827#issuecomment-652243902, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AM6TUWKU22PKS5254EZVYA3RZLQQRANCNFSM4OHXSTBA.


Fixed most compiler warnings -Wall -Wextra

repro:
CFLAGS="-Wall -Wextra -Wunused-parameter -Wc++-compat" ./configure && make
which we use for perl, and libyaml is now included in cperl.
Tested with gcc-5 and clang-3.7
There are still a tons of format warnings (%d on 64bit) in
example-deconstructor.c which I skipped.

@rurban, thanks for the work. Most of this code is upstream, so it might take me a bit to get things sorted.

Should I file a PR to master also? Would be easier maybe

@rurban the canonical upstream is https://bitbucket.org/xi/libyaml
You can certainly try to get your changes there.
This repo is intended to be a GitHub fork that stays close to upstream. I haven't spent much time gardening it. I'll try to sync it with bitbucket now. Then I can merge your changes on a branch at least, and then try to pull into YAML::XS.
The YAML::XS repo needs a bit of gardening too.

@rurban can you redo this over the new master?
This repo is now under new management and your commits will be added promptly.

Sure, no problem


Added progress information

Update counts:
Total: 10, Success: 9, Invalid: 1, Failed: 0

or show progress bar:
##           10%

added


[WIP] Added charts to dashboard

Adds stats and charts to dashboard

Closed in favor of https://github.com/TryGhost/Admin/pull/1843


Show availability directly on map

This might be difficult to implement in a performant way as it would probably need many parallel requests to the data source. But it is regularly requested, so I should take another look at this at some point.

might be possible with the Chargecloud APIs, since these can give all details for all locations in a bounding box with one request


Add to Step bookmarklet page styling

@allanhaggett Most of the styling is done now, hoping you can take a look at some of the comments in the file, on the issue branch.

Branch: https://github.com/bcgov/learningcurator/tree/bcgov-noris/issue187


Team leaders form

@sjackman @cloucks @Brittdrog
This issue has a new home
https://github.com/ttimbers/October_2016_private/issues/4
Draft for the team leader application:
https://docs.google.com/forms/d/10n52V3flb7Z9z-ulBqAHS8tWVG2ov4b13BQLWYs4tRc/edit?usp=sharing
@DCGenomics please feel free to give us your feedback
cc: @ababaian @ttimbers @brunogrande @GalenWright @minisciencegirl @santina

Thanks @Brittdrog and @sjackman for your improvements!

Hey, I think the team leader application form looks great! Just a couple of suggestions, feel free to use them or not:

In one of the example forms, there was a field for approximate geographic location. I think this would be useful, as well as one for the applicant's related institute
Section 1, where it asks are you available for the three days - should we give a rough approximate hours expected?
Likewise with regards to times with the mentorship availability.

@GalenWright thanks for your great suggestions! @adrisua have the specific start and end times been decided for the three days?

We usually run from 9-6 with some networking time after.
This also seems to decrease burnout (instead of working 24 hours).  Many
folks will continue making commits after networking time.
On Mar 24, 2016 12:11 AM, "cloucks"<EMAIL_ADDRESS>wrote:

@GalenWright https://github.com/GalenWright thanks for your great
suggestions! @adrisua https://github.com/adrisua have the specific
start and end times been decided for the three days?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/hackseq/October_2016/issues/15#issuecomment-200654334

I'd like to find out what software they need for their analysis, make sure that there are Homebrew-science formula for that software, and possibly create a machine image with it preinstalled. That is, one machine image shared by all teams with all the software installed on it. I don't think this question needs to go on the original application. We can ask the team leaders later what software they need.

I wrote little blurbs for the team leader and mentor descriptions in the team leader application (partly drawn from the emails we will send out to potential team leaders). Please feel free to edit/improve them.

@ttimbers I also made a separate mentorship application form: https://docs.google.com/forms/d/1Mps9NRYI70d5b0kznMqbUINGSETwqNJz6ImyVPfy8xc/edit?usp=sharing

@cloucks thanks for making this. Can we add a couple more questions to assess mentors knowledge level and ask them to write a couple sentences on why they want to participate? Raising the bar a little will hopefully help ensure those who sign-up to become mentors, actually show up to the event.

Good idea @ttimbers

Great point!  Matt and I have a bunch of amis from previous teams that
might be helpful.  I believe we have a "base ami" too
On Mar 24, 2016 2:22 PM, "Shaun Jackman"<EMAIL_ADDRESS>wrote:

I'd like to find out what software they need for their analysis, make sure
that there are Homebrew-science formula for that software, and possibly
create a machine image with it preinstalled. That is, one machine image
shared by all teams with all the software installed on it. I don't think
this question needs to go on the original application. We can ask the team
leaders later what software they need.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/hackseq/October_2016/issues/15#issuecomment-200959015

@cloucks  Yes, dates are set.
The forms look great!!

@sjackman I have an AMI and a "manifest" of the software installed for the last hackathons. It's in a Google doc, I'll share it with you.
It's always hard to pin down just what is needed (and versions) so it's more of a best guess than perfect AMI. It's all manual ./configure && make installed into /opt, and then I added that to $PATH in /etc/profile.d/ just to make it easy for the users.

Actually, as an addendum, I had a lot of help/convenience for that AMI by using BioBuilds - https://biobuilds.org/downloads/ - a twice-yearly, curated, pre-compiled collection of common bioinformatics tools.
I'll also take a look at homebrew-science and a homebrew install for Linux next time.

When are the team leader project proposals due?

From discussions here @sjackman suggested we align team leader project proposals to be due on June 2nd to align with ASHG. I agree, what do others think?
@sjackman if we don't come to a consensus on the date before VanBUG today, let's just announce the April 30th date, and we can always "extend" it later if that is what we decide to do. Sound good?

If we can quickly decide on June 2nd, I'd like to announce that date.
@hackseq/steering_committee What do you all say?

Once we have decided, @ababaian can you update the web page with the project proposal due date?

If I remember, originally we wanted the participant applications to be due ~ June 2nd so they could book their plane tickets accordingly. That might not work if we added an additional team leader's choosing their teams to the process.
Unless we do something like

April 30th Team Leader Due.
May 15 Participant Due.
June 1st Decide on teams.

Not sure if that's feasible tbh. Using a June 2nd Team Leader date might be better.

I agree with @ababaian that we need to co-ordinate our application deadlines so that people can book their flights accordingly. I think we should probably keep April 30th for team leader applications and make participant deadlines June 2nd (coinciding with ASHG abstract deadlines). May 15th may be a bit of a tight deadline to advertise projects and get participant applications in.
We could always re-evaluate these deadlines if we do not have enough applicants?

April 30th is only two weeks away. Does that give enough time for people to hear about it by word of mouth and submit an application?

We could extend Britt's dates by two weeks, which would give the team leaders a month to put together applications and allow more time for word to spread about the event. It would also give us the opportunity to email the invited team leaders again, reminding them of hackseq in case they forgot about the previous email.

The motivation for the June 2nd decide-on-teams deadline was to get the info to the leaders and participants in time for them to schedule their travel. ASHG abstracts are due June 2nd, but won't hear back regarding acceptance until mid-August, and probably won't schedule their travel until they've heard back. How about this schedule?
Thu June 2: Team leader project proposal due
Thu Jun 30: Participant application due
Fri Jul 15: Decide on teams
https://www.ashg.org/2016meeting/pages/abstract_faq.shtml

Q. When will I be notified of acceptance?
A. The information will be available on the ASHG website in mid-August.

The hackseq account just received an email about the deadline for team leader applications. I looked on our website and could not find a date mentioned under http://www.hackseq.com/participate/.
@ababaian could you possibly include the team leader deadline, with the revised date, on this page? Thanks!

Done and added due date to the top of the application form.

@cloucks - do you think we should add the travel award application info to the team leader application forms? We could maybe send emails to the applicants who have already applied to inform them that they can apply for travel awards?

@Brittdrog I think it's a good idea to add the travel award info to the team leader application, and if only trainees are eligible we don't have to worry about the people who already applied.

@cloucks - Perfect!

@Brittdrog @cloucks Just wondering where the team leader applications live. How many do we have so far? @minisciencegirl are the twitter feeds being effective or should we look for alternative channels?

@adrisua @Brittdrog @minisciencegirl We're at 7 applications now (a new one came in today). You can access the team leader application form here (the hackseq email account has access): https://docs.google.com/forms/u/0/d/10n52V3flb7Z9z-ulBqAHS8tWVG2ov4b13BQLWYs4tRc/edit#response=ACYDBNgWYKcK99nIqN1pD6vcVrcVKaB6IGyMFn-kf83x4wlhRPv_0bxhRxoLzCw

@adrisua: I just looked up our Twitter Analytics and the data look better than I expected. We have 6.5K impressions for the last month, with about 238 impressions per day. This is not bad considering that we have only about 70 followers. A big thank you @adrisua for getting the ASHG account to retweet our tweets and follow us - that definitely drove traffic to our account.
For anyone interested in tracking our twitter impressions: https://analytics.twitter.com/user/hackseq/tweets
With that said, I don't think our Twitter feed should be the prime source of advertising.  In part because it does take time to build up followers. For team leader application, it was helpful to directly contact people (i.e. Galaxy, Deanna Church etc). Any other ideas for advertising the participant applications?

Let's extend the deadline to June 16th? @hackseq/steering_committee Early next week we could start sharing the info in our own departments, Faculties... @cloucks @Brittdrog Do you have a template we could use for departmental newsletters, mailing lists? I could reach out to the Faculty of Science and Forestry at UBC

A two-week extension seems reasonable.

I just checked the applications and we have 10 applications in. Maybe some of the hackseq members can put in an application too (I see @sjackman has already put in an application).
My worry is that people are already concerned about booking accommodation and we still need to get participants to apply. We are also getting contacted by team leaders to find out when they will find out whether their applications have been accepted.
Here is the previous template for sending out to departments if we still decide to extend the deadline (the deadline date will just need to be changed).

Shall we send out e-mails with abstracts to participants on Thursday June 9? We can in the next few days solicit more project proposals directly from people we know.

Sounds good to me.

I like that idea too.

New deadline for team leaders applications: June 8th. @ababaian @brunogrande @santina @ttimbers Would you mind sharing this in your network @Brittdrog thanks for the template: October_2016_private/TeamLeaders/Email_for_distribution_to_societies.pdf

@ababaian I may have misinterpreted it, but weren't we going to keep the team leader applications officially closed, but try and convince a couple of people to apply? Personally, I don't think we should announce an extended deadline again as it makes it look like we are struggling to get applications

Sorry, @ababaian didn't mean to tag you directly - it was meant for @adrisua

I agree with @GalenWright. I thought we were aiming for 10 applications to begin with?

10–20 as I recall.

I want to mention that we have 10 applications but only 9 submitted abstracts. There is one person who is unable to commit for the entire thing.

I think it is fair to give equal opportunity to everyone that would like to apply - since we are allowing late applications. Also, it will give our participants more options and diversity (if more people apply).

I shared the info with my departments (since I hadn't done it and people were asking me about it). Sorry for the confusion. If June 8th is still too late let me know.

No worries, I guess we can phrase it on twitter, "Due to popular demand, we've decided to extend our team leader applications..."

I'm not sure I agree with extending the deadline. 9 teams x 6 people per
team is 54 people. Plus all of us, and any mentors that wasn't too pop by
and help out. That's a big event!
On Jun 3, 2016 12:43 PM, "Galen Wright"<EMAIL_ADDRESS>wrote:

No worries, I guess we can phrase it on twitter, "Due to popular demand,
we've decided to extend our team leader applications..."
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hackseq/October_2016/issues/15#issuecomment-223675659,
or mute the thread
https://github.com/notifications/unsubscribe/AGjAeFxk9Ni3YI2MwZLhFKOjiQblj2Xrks5qIIP5gaJpZM4H2yiw
.

I'm not sure I agree with extending the deadline. 9 teams x 6 people per
team is 54 people. Plus all of us, and any mentors that wasn't too pop by
and help out. That's a big event!
On Jun 3, 2016 12:43 PM, "Galen Wright"<EMAIL_ADDRESS>wrote:
No worries, I guess we can phrase it on twitter, "Due to popular demand,
we've decided to extend our team leader applications..."
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hackseq/October_2016/issues/15#issuecomment-223675659,
or mute the thread
https://github.com/notifications/unsubscribe/AGjAeFxk9Ni3YI2MwZLhFKOjiQblj2Xrks5qIIP5gaJpZM4H2yiw
.