[Question] Why am I getting an invalid type exception when using PromptDialog.Choice forwarding to a custom SuggestedActionDialog??

App ID: 5f35fe55-4e26-49d1-b3a0-46a6192f6dd1
SDK Platform: .NET
Active Channels: Skype, direct line, webchat
Deployment Environment: Azure app service

Issue Description
I'm getting an invalid type exception when using PromptDialog.Choice.
I am forwarding to a custom SuggestedActionDialog from the QnaMakerDialog. The prompt is supposed to show two or more buttons, which the user must select. The first time the dialog is started, the exception mentioned above is thrown at this line: var selectedButton = await result;
invalid type: expected TestBot.Model.Button, have Boolean inside the ResumeAfterConfirmation method.
Code Example
`[Serializable]
public class SuggestedActionDialog : IDialog
{
private readonly DynafleetSuggestedAction _dynafleetAction;
    public SuggestedActionDialog(DynafleetSuggestedAction dynafleetAction)
    {
        _dynafleetAction = dynafleetAction;
    }

    public Task StartAsync(IDialogContext context)
    {
        context.Wait(MessageReceivedAsync);
        return Task.CompletedTask;
    }

    private async Task MessageReceivedAsync(IDialogContext context, IAwaitable<IMessageActivity> result)
    {
        var activity = await result;
        var nextAction = _dynafleetAction;
        var options = nextAction.Buttons;
        var descriptions = nextAction.Buttons.Select(s => s.Caption);

        PromptDialog.Choice<Button>(context, ResumeAfterConfirmation, options, nextAction.Title, descriptions: descriptions);
    }

    private async Task ResumeAfterConfirmation(IDialogContext context, IAwaitable<Button> result)
    {
        try
        {
            var selectedButton = await result;
            if (selectedButton.SuggestedAction != null)
            {
                var nextAction = selectedButton.SuggestedAction;
                var options = nextAction.Buttons.ToList();
                var descriptions = nextAction.Buttons.Select(s => s.Caption).ToList();

                PromptDialog.Choice<Button>(context, ResumeAfterConfirmation, options, nextAction.Title, descriptions: descriptions);
            }
            else
            {
                await context.PostAsync(selectedButton.Value);
                context.Done(string.Empty);
            }
        }
        catch (Exception e)
        {
            Trace.TraceError(e.ToString());
        }
    }
}`

Funny thing is: I catch the error in the catch block, the user sees the PromptDialog a second time but can select a button and the dialog continues as expected.

I've tried now with a simpler example:
`private async Task MessageReceivedAsync(IDialogContext context, IAwaitable result)
{
var activity = await result;
PromptDialog.Confirm(context, Confirmed, "Have question");
}
    public async Task Confirmed(IDialogContext context, IAwaitable<bool> argument)
    {
        bool isCorrect = await argument;
        if (isCorrect)
        { }
        else
        { }
    }`

The bot shows the prompt with 2 buttons, but without selecting an option, the if(isCorrect) statement is handled. When I select the yes button, I get the same prompt once again.

I just tested your code example in a dialog (below) in the emulator and got the following results, did I miss something?  It seems I am unable to reproduce.
        public Task StartAsync(IDialogContext context)
        {
            context.Wait(MessageReceivedAsync);

            return Task.CompletedTask;
        }
        private async Task MessageReceivedAsync(IDialogContext context, IAwaitable<object> result)
        {
            //var activity = await result as Activity;
            var activity = await result;
            PromptDialog.Confirm(context, Confirmed, "Have question");
        }
        public async Task Confirmed(IDialogContext context, IAwaitable<bool> argument)
        {
            bool isCorrect = await argument;
            if (isCorrect)
            { }
            else
            { }
        }

I think the issue starts somewhere in the beginning of my bot code. Take a look at my root dialog:
`public Task StartAsync(IDialogContext context)
{
context.Wait(MessageReceivedAsync);
return Task.CompletedTask;
}
    private async Task MessageReceivedAsync(IDialogContext context, IAwaitable<object> result)
    {
        if (!await IsRegistered(context))
        {
            Trace.TraceInformation("Calling login dialog first.");
            var loginDialog = new FormDialog<LoginForm>(new LoginForm(), LoginForm.BuildForm, FormOptions.PromptFieldsWithValues);
            await context.Forward(loginDialog, ResumeAfterRegistration, context.Activity, CancellationToken.None);
        }
        else
        {
            Trace.TraceInformation("Calling QnA dialog.");
            await context.Forward(new QnaDialog(), ResumeAfterQna, context.Activity, CancellationToken.None);
        }
    }

    private async Task ResumeAfterRegistration(IDialogContext context, IAwaitable<LoginForm> result)
    {
        var loginData = await result;
        Trace.TraceInformation("Resuming after registration: " + loginData);
        context.Wait(this.MessageReceivedAsync);
    }

    private Task ResumeAfterQna(IDialogContext context, IAwaitable<IMessageActivity> result)
    {
        //var temp = await result;
        Trace.TraceInformation("Resuming after QnA");
        context.Wait(this.MessageReceivedAsync);

        return Task.CompletedTask;
    }`

Notice the var temp = await result in comment in the ResumeAfterQna method. If I uncomment this, I get the same Invalid Type exception, expecting an IMessageActivity, but receiving a bool.
Since I don't need the result of the QnA dialog here, I just commented it out. But then I got more or less the same issue in the SuggestedAction Dialog, mentioned in my first post. Expecting a button, instead receiving a bool.
This is the flow:
RootDialog -> Forward to LoginDialog or QnaDialog -> In QnaDialog, either post message or forward to SuggestedAction dialog.
This is the content from the RespondFromQnAMakerResultAsync inside the QnaDialog:
`
var answer = result.Answers.First().Answer;
var qnaAnswerData = answer.Split(new[] { ";" }, StringSplitOptions.RemoveEmptyEntries);
var reply = ((Activity)context.Activity).CreateReply();
            foreach (var data in qnaAnswerData)
            {
                if (data.ContainsVideoLink())
                {
                    reply.Attachments.Add(new VideoCard()
                    {
                        Media = new List<MediaUrl>() { new MediaUrl(data.Replace(ApplicationConstants.AttachmentPrefix.VIDEO, string.Empty)) },
                        Autoloop = false,
                        Autostart = true,
                        Aspect = "16:9",
                        Image = new ThumbnailUrl("http://www.dynafleethelp.volvotrucks.com/content/dam/volvo/volvo-trucks/masters/dynafleet-help/tutorials/1860x1050-tutorials-basic-administration.jpg/_jcr_content/renditions/1860x1050-tutorials-basic-administration-teaser2.jpg")
                    }
                    .ToAttachment());
                    await context.PostAsync(reply);
                }
                else if (data.ContainsImage())
                {
                    reply.Attachments.Add(new HeroCard()
                    {
                        Images = new List<CardImage>() { new CardImage(data.Replace(ApplicationConstants.AttachmentPrefix.IMAGE, string.Empty)) },
                        Text = "SCREENSHOT"
                    }
                    .ToAttachment());
                    await context.PostAsync(reply);
                }
                else if (data.ContainsSuggestedActions())
                {
                    Trace.TraceInformation("Calling Suggested Action dialog.");
                    var dynafleetAction = JsonConvert.DeserializeObject<DynafleetSuggestedAction>(data.Replace(ApplicationConstants.AttachmentPrefix.SUGGESTEDACTIONS, string.Empty));
                    await context.Forward(new SuggestedActionDialog(dynafleetAction), ResumeAfterSuggestedAction, message, CancellationToken.None);
                }
                else
                {
                    reply.Text = data;
                    await context.PostAsync(reply);
                }
            }

`

Situation update.
When I switch from the Microsoft.Bot.Builder.CognitiveServices.QnAMaker.QnAMakerDialog to the QnAMakerDialog from Gary Pretty ( https://www.nuget.org/packages/QnAMakerDialog/ ), the issue is resolved.
Implementation is the same.
So I guess the Microsoft QnAMakerDialog is messing things up?

It's possible, but we are not seeing any other reports of this.  In this case you may way to report the issue in the Conative Services repo

@FranciscoPonceGomez Do you have any insight on this? I know you have been looking into QnAMaker.

@JasonSowers i think you are right: https://github.com/Microsoft/BotBuilder-CognitiveServices/issues/39


CMakeLists.txt

Can't run cmake because there's no txt file. Please add it to the package ASAP please.

Solved my issue by copying nessary CMakeLists.txt from developer's another project.
To solve this:

Copy CMakeLists.txt in the project's root folder.
Copy CMakeListsSRC.txt in /src folder (where the Main.cpp, yml file is). Rename the txt file to CMakeLists.txt
and you're good to go.
CMakeLists.txt
CMakeListsSRC.txt


[Mod Compat] EMI is not grabbing unique JEI recipes for current Create Mod version 0.5.1i (Forge 1.20.1)

Currently the newest version of Create, 0.5.1i will have it's recipes working fine in JEI but once EMI is added will no longer show unique create recipes (No pressing recipes, mixing recipes, sanding recipes ect.). This is specific to Create, tested with Botania and Mekanism both showing unique recipe types such as Mana Conversion.
Downgrading to Create 0.5.1h or lower or removing EMI and just using JEI fixes the issue.
It seems like it will not cache the recipes on a fresh install but upon downgrading and then reupgrading Create versions causes the recipes to suddenly appear as normal. (Tested in 2 instances, one where EMI was installed fresh, one where I had changed Create versions back and forth, recipes only appear in the ladder)

Forgot to mention the way I tested if it was working was looking at the uses for Rose Quartz (right click) and recipe for Sheets of any material
Fresh install:

Recipe Loaded in previous create version and then updated to current:

All recipes will be loaded correctly in the second instance, even ones that were not originally looked at such as pressing and mixing recipes

This is me testing on a fresh version of 1.20.1 with the latest Create, EMI, and JEI versions


Make FileUtils#calculateRoots more efficient

Instead of a quadratic algorithm (comparing every
file against every other file), it now uses an
O(nlogn) algorithm, sorting the files first and
then removing all those that are covered by their
predecessors.

Considering this test case:
FileUtils.calculateRoots(files("a/a", "a/a-1", "a/a/a"))

I would expect the output to be ["a/a", "a/a-1"] but this algorithm generates ["a/a", "a/a-1", "a/a/a"] because character "-" is smaller than "/".

Good point, I'll add some more coverage.

@lixiangconan can you have another look? I've changed to a segment-based comparison, which fixes the issue you pointed out while keeping the main loop very simple.

Can you try this out on your project in its current state? I'd like to avoid further micro-optimizations if it already does the trick :)

I profiled runPlayBinary on our project using an implementation based on String#compareTo, and I'm seeing 2.80% of CPU samples in StringUtils#split disappear. This alternative implementation does need to read characters of the string using String#charAt, which could be a few instructions more expensive per character than the individual String#compareTo calls on each path segment because it does bounds checking, but I wasn't able to see this show up in the CPU profile.
I do see a reduction in memory allocation in my build during hot reload by 1 GB of short-lived objects related to splitting by segments:

538 MB of char[] arrays
303 MB of String
120 MB of ArrayList resizing when running StringUtils#split
58 MB of String[] arrays

The hacked together and untested implementation I used, without comments, was follows:
String leftPath = left.getPath();
String rightPath = right.getPath();

int len1 = leftPath.length();
int len2 = rightPath.length();
int lim = Math.min(len1, len2);

int k = 0;
while (k < lim) {
    char c1 = leftPath.charAt(k);
    char c2 = rightPath.charAt(k);
    if (c1 != c2) {
        if (c1 == File.separatorChar) {
            return -1;
        }
        if (c2 == File.separatorChar) {
            return 1;
        }
        return c1 - c2;
    }
    k++;
}

return len1 - len2;

LGTM!


Whats the process of creating a module with cocoapods dependency

native module i'm building uses firebase(installed through cocoa pods). I added "$(SRCROOT)/../../../ios/Pods/**" to header search path in the library native module and now in the Project that uses the native module I keep getting an error at  import firebase that  No such module 'Firebase'  .   I have ran react-native link  <module_name>
I also added the question on stack overflow https://stackoverflow.com/questions/53911013/

+1
I'm interested in this as well.
this might help: https://io.bilby91.com/posts/create-react-native-ios-native-module


[EDMTP- 248] Add accordion, add select options to branch

Hi, @ldzmitrenka, a couple of concerns:
1) (critical) Functionality.

variant, margins and multiple don't seem to work at all. I don't see them anywhere in XML.
properties mentioned above need to be specified for the whole accordion, not each panel.
there is no ability to specify active or disabled properties which are mentioned in the spec

2) Naming. I suggest renaming accordionTabs to accordions or accordionPanels and @PlaceOnAccordion to @PlaceInAccordion
3) Validation. Сurrently, the user can specify both tabs and accordionTabs, and use both @PlaceOnAccordion and @PlaceOnTab annotations. However, only accordions are rendered in this case. Maybe it makes sense to add some kind of validation to warn the user?
4) DependsOn. there is no ability to conditionally hide accordion panels with dependsOn. Seems like we need something like DependsOnPanel. Probably this can be done in a separate ticket. @smiakchilo, @lexastef, what do you think?


Discretization and Categorical features

Hi,
I have some questions that I will be glad to have some precise answers to them:
1- What are the benefits of applying discretization in the case of LimeTabularExplainer, and what are the disadvantages? In other words, what I will gain if I applied the discretization and what I will lose if I have not applied it?
2- I tried to implement all of the discretizers in LIME, each one result in different explanations. Accordingly, I need to know what is the best discretizer out of them, and why the quartile discretizer has been chosen to be the default one?
3- Why the categorical features have to be LabelEncoded?

Finally, we use a One-hot encoder, so that the classifier does not take our categorical features as continuous features. We will use this encoder only for the classifier, not for the explainer - and the reason is that the explainer must make sure that a categorical feature only has one value.

What does it mean that the explainer must make sure that a categorical feature only has one value? Does LIME sum back the different categories from the same categorical feature in case of OneHotEncoding, or it treats them as if each one is a  new separate feature?
Thanks.

1 and 2: See #196, #113, #265. Different discretizations give you different 'interpretable features', from which you get (different) explanations. You need a granularity that makes sense to the user, but also is able to capture model behavior. The straw man is the discretization that says 'every value falls into the same bin'. Clearly, we won't get useful explanations out of that. The other extreme is saying 'either the value is =5.3323 or it's not', which is also not very useful.

What does it mean that the explainer must make sure that a categorical feature only has one value? Does LIME sum back the different categories from the same categorical feature in case of OneHotEncoding, or it treats them as if each one is a new separate feature?

If you have a variable that encodes Sex (Male or Female) as a categorical feature, you want to generate perturbed samples of people who are either male or female. If you use OneHotEncoding, you get a feature for male and one for female, and LIME has no way of knowing that they are tied such that only one of them can be on.

@marcotcr I got what you say. However, I can not really interpret the results out of discretized LIME, In other words, what exactly each feature mean, can you describe the results in this image https://user-images.githubusercontent.com/45386621/78400902-d76f7800-75f7-11ea-9286-108e886df292.PNG

I don't know if I can explain it better than I did here. Did you have any specific questions that are not addressed there?

We pick one of the bins according to its frequency in the training set. Then, we generate a value in that bin by a truncated norm within that bin. Thus, no, categorical features will not be normally distributed. You can think of the simple case where the training set is distributed in the following way, assuming quartile discretization: [0.1, 0.1, 0.1, 0.7]. When sampling from this feature, 70% of values will fall into the last quartile, 10% in each of the other quartiles. I guess it will be a mixture of truncated normals.

3 - I don't think I understand what you mean by ' in a lower feature space than the original space as well.' Sorry!

@marcotcr  For point three, what I mean by lower feature space, for example, if the original space work on 345 features, the interpretable space may work on 100 features. For example, for Images, we might have the original dimension of 100100100, and the lower interpretable space works only on 505050 is the same holds for TabularData?

No, for tabular data we use the same features. Discretization changes the values that the features can take, but it's still the same features.

@marcotcr  So to sum up, x' only represents a binary interpretable version but with the same number of features. What I see also that the similarity is done between x' and z' both as a binary interpretable version, not in between x and z in the original space, is that correct?
I will be glad if you can have a look at the illustration of the algorithm I provided I think the only difference that x' should be considered only as a binary interpretable feature (not in lower feature space).
Moreover, can you have a look at point 5 regarding the ridge regression and inform me if my understanding is correct or not?
If we applied the discretization, does it will be applied to all of the features (numerical and categorical) or to the categorical only?
For your comment here, I think you by ([0.1, 0.1, 0.1, 0.7] that this is how only one feature is discredited. Accordingly, we gonna pick the last bin because it has the highest frequency and we will generate value from it by using a truncated norm, is that correct?

We pick one of the bins according to its frequency in the training set. Then, we generate a value in that bin by a truncated norm within that bin. Thus, no, categorical features will not be normally distributed. You can think of the simple case where the training set is distributed in the following way, assuming quartile discretization: [0.1, 0.1, 0.1, 0.7]. When sampling from this feature, 70% of values will fall into the last quartile, 10% in each of the other quartiles. I guess it will be a mixture of truncated normals.

In practice, what we do is first select K features (to make sure the constraint is met) with whatever feature selection method is picked (lasso, highest weights, or whatever), and then learn a ridge regressor with small regularization (for numerical stability) to get the actual weights for explanations.

If we applied the discretization, does it will be applied to all of the features (numerical and categorical) or to the categorical only?

Only to continuous features

For your comment here, I think you by ([0.1, 0.1, 0.1, 0.7] that this is how only one categorical feature is discredited. Accordingly, we gonna pick the last bin because it has the highest frequency and we will generate value from it by using a truncated norm, is that correct?

Yes, this is for one feature (notice I'm not telling you what the bins are, just that they are quartiles). We are not going to pick the last bin, we will sample the bin according to these probabilities (i.e. the last bin will be picked ~70% of the time)

In practice, what we do is first select K features (to make sure the constraint is met) with whatever feature selection method is picked (lasso, highest weights, or whatever), and then learn a ridge regressor with small regularization (for numerical stability) to get the actual weights for explanations.
Yup, but even after picking our K features, we might have several ridge regressors that fit and we will pick the one that contributes the least to the loss, so G here can be a set of several ridge aggressors, is that correct? Or is it only one ridge regressor that fits perfectly to our situation and what we mean by g belong to G, that is g can be considered for example as a linear model or decision tree, but it is only one model not several from the same type?

Yes, this is for one feature (notice I'm not telling you what the bins are, just that they are quartiles). We are not going to pick the last bin, we will sample the bin according to these probabilities (i.e. the last bin will be picked ~70% of the time)

Ahh, so for this specific feature, what is going to be done, is that we gonna truncate from all of the bins and the final result for these features will be the result of multiplication of all of these truncated norms from each of the bins.
Something more, can you provide me how can we describe the interpretable version x' for the case of tabular data? In the case of Images, it might be superpixels, in case of text it will be words, how should it be for tabular data?

On Ridge: yes, we only have a single model that optimizes our locally weighted loss, out of the class of all ridge regressors. g belonging to G means that, yes.
For tabular data, you can think of the discretized version (if you use it) as the interpretable representation. If you don't use it, the original representation is already interpretable, and thus x = x'.

For tabular data, you can think of the discretized version (if you use it) as the interpretable representation. If you don't use it, the original representation is already interpretable, and thus x = x'.

So, if we applied the discretization, we can say that the similarity is done between x' and z' which are discretized versions. If the discretization is false then the similarity is done between x and z which are the same as the original version, correct?

Please note that LabelEncoder is not meant to be used with categorical features.
https://scikit-learn.org/stable/modules/generated/sklearn.preprocessing.LabelEncoder.html
I'm afraid this might not work well with Sklearn's Pipeline.


conf_cookbook not used in all resources

the conf_cookbook property for using an erb template from another cookbook is not used or set in all resources / generally not consistent

not being asked for


User.ReadBasic.All Application Permission for App Registration?

Hi Yvan,
It recently came to my attention that Microsoft added User.ReadBasic.All Graph Permission to Application Permissions.
Will the Permission User.ReadBasic.All be supported by EntraCP or is there a specific reason why User.Read.All has to be used?
Kind regards,
Claudio

Hi @Kinumikao, thank you very much for raising this, I was not aware about this permission.
I will be more than happy to use User.ReadBasic.All instead of User.Read.All if it does not cause side effects, I will test it asap and let you know here.

Hi @Kinumikao, finally I tested the permission User.ReadBasic.All. Unfortunately, it cannot replace User.Read.All because it denies critical requests such as the one below:
/users?$count=true&$filter=( (startswith(UserPrincipalName, 'testEntraCPUser_001') and UserType eq 'Member') or (startswith(Mail, 'testEntraCPUser_001') and UserType eq 'Guest') ) or startswith(DisplayName, 'testEntraCPUser_001') or startswith(GivenName, 'testEntraCPUser_001') or startswith(Surname, 'testEntraCPUser_001') or startswith(Mail, 'testEntraCPUser_001')&$select=Id,UserType,Mail,UserPrincipalName,DisplayName,GivenName,Surname,Mail,DisplayName,Mail,MobilePhone,JobTitle,Department,OfficeLocation&$top=30
I did not test further, but my guess is that getting the property UserType is rejected with permission User.ReadBasic.All


WIP: Virtual sensors / remap sensors

Closes https://github.com/mne-tools/mne-python/issues/1080. Here's what you get when you remap mag to grad for sample.

Still needs to be made nicer with tests and better options in API.
cc @Anevar @agramfort

We should think of a good way to convert the info structure -- otherwise you'll get plots with units all wrong. If we convert 204 grads to 204 mags, how do we specify the channel names of the new virtual channels?

@agramfort @mainakjas take a look here:
https://github.com/fieldtrip/fieldtrip/blob/master/template/layout/neuromag306cmb.lay
combined grads + mags layout

how is that related? the lay does not tell you the 3d locations of sensors

their orientation

the lay does not tell you the 3d locations of sensors

that's indeed true.

These virtual channels create a discrepancy between the channel names and channel type which causes a lot of code to break. How do we tell the evoked methods that these are virtual channels and should be treated differently?

I can probably make it work if we make new channel types vmag and vgrad -- then we treat these differently in all the pick methods. Thoughts?

-1 for new types. The units match the meg ch type

I agree on the new types, the idea is that we are interpolating/synthesizing/simulating data that could/would have been recorded on real, standard sensors. We could give the new channels distinctive names like MEGInterp001 by default so they know the data are interpolated/synthesized, and strongly emphasize on the docstring that data are synthesized/interpolated.

I think a simple naming scheme by default '%s_interp_%s' % (ch_type.upper(), idx) would be a good scheme if users pass new_ch_names=None. If they want something more compact, they should override the default verbose name.

@mainakjas this seems to be close to working, I added the 0.9 tag in hopes that you can get this done in the coming week. Sound feasible?

@Eric89GXL I think we can have it ready in the coming week definitely

You need to change the coil type

I made it work for grads too. Don't ask my how or why. This is for somato data. Have a look:

I made it work for grads too. Don't ask my how or why. This is for somato data. Have a look:

Wow, those are actually pretty close. I see your hack in the code, I too am confused by it :(

ok hack doesn't work any more @agramfort It seems I had misunderstood the eventual goal. When the 'to_type' was 'grad', the interpolation was being done on the original 'mag' channel locations using the 'grad' data. However, what we want is to interpolate at the 'grad' channel locations using the mag data. This turns out to be more challenging somehow :) It still works for grad to mag but not the other way round.

The interpolated grad maps basically look like the mags. Changing the coil type is not going to do anything -- I've tried all possible combinations. I think now I need to understand what's going on :) @Eric89GXL any hints?

I'd take a look at mne_map_data to figure out what it does. It's possible that the field interpolation code probably only into magnitude sensors, not gradient sensors. mne_map_data should have to do both, though.

so mne_map_data is not part of mne-python code? It's in the C code, right? That's why I was confused when you first mentioned it. I'll dig into it tomorrow. Thanks!

Yep, you'll need to dig through the C code.
In some (hopefully many) of the functions I translated, I tried to note function names or filenames that they were drawn from. My convention has been to drop the mne from in front of things, turning e.g. mne_compute_forward into _compute_forward. That might help you find correspondences between code you look at and code in mne-python. Feel free to beef up the mne-python comments as you go if it helps you navigate the C code -- I tried to put enough in there to be able to follow, so you looking through the two is a nice test case.

Nice additional comments @mainakjas. Let me know if you need help, I might have time next week to dig into this, too. I have a feeling this PR might end up holding up the release, the others seem more straightforward. But it's probably worth it to get this in. WDYT @agramfort @dengemann?

I think I'm on to something (not sure). I'll try for one or two more days. If it does not work, @agramfort suggested that we could simply do it for mags for now and extend it to grads later.

Yes but that would be so unsatisfying :)

Holy cow, I just translated the C code and it works like magic. Pushing the changes now.

the C code is readable after all :)

tell us when to review ie you have tests + travis happy

yeah, I gave up on installing all those libraries. I just read it and translated it :)

I think of Matti's C code as a treasure trove embedded in a sea of C boilerplate. Once you find what you're looking for, the code is indeed quite clear.
I can't decide if I'm more excited about the fact that it now seems to be working, or that there is now someone else who we can bother to help translate Matti's code when necessary :)

now that the code works we need to agree on API for the different use cases. @mainakjas can you make a proposition with snippets? thx

For virtual evoked, we can do:
evoked = evoked.compute_virtual_evoked(to_type='mag')
which will map all the existing MEG channels to mag locations in evoked.info and return an evoked with 102 channels. 'to_type' can be either 'mag' or 'grad'.
For bad channel interpolation, we can simply do:
evoked = evoked.interpolate_bads_meg()
For head position, I am not sure. But maybe the user can provide a trans file/matrix between the two head positions?
evoked = evoked.change_head_position(trans)
Wdyt?

For virtual evoked, we can do:
evoked = evoked.compute_virtual_evoked(to_type='mag')

I would say.
evoked = evoked.as_type('mag')
a method of evoked that contains evoked is unnecessary.

For bad channel interpolation, we can simply do:
evoked = evoked.interpolate_bads_meg()

+1
and we could have an
evoked.interpolate_bads()
to repair all channels.

For head position, I am not sure. But maybe the user can provide a trans file/matrix between the two head positions?
evoked = evoked.change_head_position(trans)

I would use a function here that takes a list of evoked and puts them
all in the same head position.
my 2c

On Sat, Apr 18, 2015 at 1:09 PM, Alexandre Gramfort <
<EMAIL_ADDRESS>wrote:

For virtual evoked, we can do:
evoked = evoked.compute_virtual_evoked(to_type='mag')

I would say.
evoked = evoked.as_type('mag')
a method of evoked that contains evoked is unnecessary.

+1 I like this.

For bad channel interpolation, we can simply do:
evoked = evoked.interpolate_bads_meg()

+1
and we could have an
evoked.interpolate_bads()
to repair all channels.

I knew that was coming :)

For head position, I am not sure. But maybe the user can provide a trans
file/matrix between the two head positions?
evoked = evoked.change_head_position(trans)

I would use a function here that takes a list of evoked and puts them
all in the same head position.

makes sense. So, it would put them all in the head position of the first
evoked of the list?

my 2c
—
Reply to this email directly or view it on GitHub
https://github.com/mne-tools/mne-python/pull/1943#issuecomment-94156012.

makes sense. So, it would put them all in the head position of the first
evoked of the list?

+1

are we happy with the example now? I'll add tests if the example looks good. I'll remove the add_title function in the example once @cmoutard's colorbar fix is merged.

Eventually we probably do want to be able to specify the head position instead of just taking the first one, since for example we translate all head positions to the median (in quaternion space) head position when doing translations. If trans=None then use first head position. Sound reasonable?

can you see what happens when you apply this to CTF data?

please add tests now. thanks @mainakjas !

@agramfort topomap doesn't seem to be working on spm face CTF data. It isn't able to detect grad channels (even though picks finds grad channels). I think this should be dealt in a separate PR

On Sat, Apr 18, 2015 at 3:46 PM, Eric Larson<EMAIL_ADDRESS>wrote:

Eventually we probably do want to be able to specify the head position
instead of just taking the first one, since for example we translate all
head positions to the median (in quaternion space) head position when doing
translations. If trans=None then use first head position. Sound
reasonable?

yes, that sounds good.

—
Reply to this email directly or view it on GitHub
https://github.com/mne-tools/mne-python/pull/1943#issuecomment-94165838.

When building the docs, I get this error:
Exception occurred:
  File "/home/mainak/anaconda/lib/python2.7/site-packages/pyface/ui/qt4/code_editor/pygments_highlighter.py", line 39, in get_tokens_unprocessed
    for item in action(self, m):
TypeError: 'NoneType' object is not callable

anyone else had this issue?

 Only thing is that I am unable to build the docs -- travis seemed to have failed due to some doc issue. Otherwise, I am good to go

i tried
evoked.copy().pick_types(meg='grad').as_type('mag').plot_topomap(ch_type='mag')

and I get a cryptic error ""ValueError: No channels match the selection.".
can you make it more explicit?

LGTM
works great !

My comments are mostly nitpicks on what seems like great code. Otherwise LGTM!

@mainakjas address @Eric89GXL's comments than +1 for merge !
next PR will be interpolation of bad meg channels !
great job @mainakjas !

all comments addressed @agramfort . I removed n_jobs for now because it doesn't help much with few channels + it's not implemented for _do_cross_dots -- so, it's only half the job (literally :)

LGTM but I'd actually prefer something like ch_type as parameter name. reading as_type and then to_type somehow distracts me. Also this would be a name that we already have in the context of selecting channels for plotting. What do others think?

I agree. I can change it to ch_type

I'm fine with ch_type

cool +3 on that already, @agramfort +0 or +1 and good?

@dengemann any other comments? I'll address them if not

@mainakjas other wise it looks good to me!

Really nice work!

Coverage increased (+0.0%) to 90.22% when pulling ce8549a9b50ece03af3b757d2c87cb409a01e661 on mainakjas:virtual_sensors into f18f2921a3233a524c562cf23b0502dbf2881fcf on mne-tools:master.

Coverage increased (+0.0%) to 90.22% when pulling ce8549a9b50ece03af3b757d2c87cb409a01e661 on mainakjas:virtual_sensors into f18f2921a3233a524c562cf23b0502dbf2881fcf on mne-tools:master.

Merged, thanks @mainakjas!

great job @mainakjas !


Add Microsoft Azure logo

The icon I want is:

Name: Microsoft Azure
URL: https://azure.microsoft.com/en-in/
Link to icon: https://upload.wikimedia.org/wikipedia/commons/thumb/f/fa/Microsoft_Azure.svg/150px-Microsoft_Azure.svg.png

Vector version of the logo: https://upload.wikimedia.org/wikipedia/commons/f/fa/Microsoft_Azure.svg
Official style guide or brand guide:

I've tried to make a start here: https://github.com/jmb/SuperTinyIcons/blob/ms-azure/images/svg/azure.svg but have only managed 1591 bytes so far. The gradients are most likely the issue, even though I've rounded a lot of the values.

If you can bake-in the gradientTransform and gradientUnits attributes, that should save a couple hundred bytes, but as far as I know you have to do that manually and it's a bit of a pain to do accurately. I might work on it if I have the time. Even then you'll be short, but it might be possible to make up the rest with path optimization and maybe even reducing the number of stops in the gradients.

I've given that a go and now have 1333 bytes. There must be some more optimisations to find! :)

Mind sharing that version?

Down to 1243 bytes - in the ms-azure branch on my fork: https://github.com/jmb/SuperTinyIcons/tree/ms-azure


Quite a few new tests added for Faster Log and Faster KV to get functional coverage

Added tests to make sure every feature for Faster Log and Faster KV have at least one corresponding test.

I see there are other changes to Master that I didn't have. Will do due diligence and merge that before doing a PR


Phonegap Build no longer supports this plugin. Please, can you update the plugin?

When I try to build my App, the PhoneGap build shows the error message:
"plugin unsupported: de.appplant.cordova.plugin.background-mode"
This plugin worked very well 'till last month. I guess the Phonegap Build don't support the older versions of cordova device (0.2.12) anymore and now I have to use the npm repo version (cordova-plugin-device).
Please, can you update this plugin and put it at NPM repo?
Thanks,
Hugo.

I got the same error.
ionic plugin add https://github.com/katzer/cordova-plugin-background-mode.git
Fetching plugin "https://github.com/katzer/cordova-plugin-background-mode.git" via git clone
Repository "https://github.com/katzer/cordova-plugin-background-mode.git" checked out to git ref "master".
Installing "de.appplant.cordova.plugin.background-mode" for android
Notice: org.apache.cordova.device has been automatically converted to cordova-plugin-device and fetched from npm. This is due to our old plugins registry shutting down.
Dependent plugin "cordova-plugin-device" already installed on android.
Installing "de.appplant.cordova.plugin.background-mode" for ios
Notice: org.apache.cordova.device has been automatically converted to cordova-plugin-device and fetched from npm. This is due to our old plugins registry shutting down.
Dependent plugin "cordova-plugin-device" already installed on ios.
Saving plugin to package.json file

same problem here

Hi guys, Mr Katzer have updated this plugin and now it's on NPM repo. I just used the <plugin name="cordova-plugin-background-mode" source="npm" /> and PhonegapBuild builded my App again! I have tested and yes, my App came to live again, hehehee... Thank you very much, Katzer.

Yes!! Working for me now too! And was thinking the whole weekend I would need to update this plugin myself today, what a nice surprise.

:+1:

Thank you so much for the information. I was able to build my app after using the plugin. However the app is showing unable to connect to server. How can i correct this?


feat: use minimal box

What I did
Use minimal box type
So it'll look like this now:
                              SYMBOL Gas
                        ╷              ╷      ╷      ╷      ╷
  Method                │ Times called │ Min. │ Max. │ Mean │ Median
╶───────────────────────┼──────────────┼──────┼──────┼──────┼────────╴
  callContractCMethodC3 │ 1            │ 7977 │ 7977 │ 7977 │ 7977
                        ╵              ╵      ╵      ╵      ╵

How I did it

How to verify it

Checklist

[ ] All changes are completed
[ ] New test cases have been added
[ ] Documentation has been updated

For comparison:
Simple:
                              SYMBOL Gas

  Method                  Times called   Min.   Max.   Mean   Median
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
  callContractCMethodC3   1              7977   7977   7977   7977

Minimal:
                              SYMBOL Gas
                        ╷              ╷      ╷      ╷      ╷
  Method                │ Times called │ Min. │ Max. │ Mean │ Median
╶───────────────────────┼──────────────┼──────┼──────┼──────┼────────╴
  callContractCMethodC3 │ 1            │ 7977 │ 7977 │ 7977 │ 7977
                        ╵              ╵      ╵      ╵      ╵

Heavy head (previous default):
                              SYMBOL Gas
┏━━━━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━┳━━━━━━┳━━━━━━┳━━━━━━┳━━━━━━━━┓
┃ Method                ┃ Times called ┃ Min. ┃ Max. ┃ Mean ┃ Median ┃
┡━━━━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━━╇━━━━━━╇━━━━━━╇━━━━━━╇━━━━━━━━┩
│ callContractCMethodC3 │ 1            │ 7977 │ 7977 │ 7977 │ 7977   │
└───────────────────────┴──────────────┴──────┴──────┴──────┴────────┘

@banteg

still thinking the "simple" style would be superior, if you look at professionally typeset tables, none of them have vertical bars, it's a poor style popularized by ms excel. any article about how to make tables better would argue the same. for numeric columns it's better to align right so you can visually compare the difference magnitude. for decimal numbers it's best to align on the dot, but not sure rich supports it out of the box.


New Beginning

Basic toolkit:

linear and slope regression by Mueller
Numeric PD calculation, Gaussian/maximal error propagation estimation for vectorized data
Weighted average of a set of data values

Basic documentation:

Introductory pages
Manual on basic tools
a short intro to Octave and GPR-related statistics

Remaining todo:

Numeric PD calculation, Gaussian/maximal error propagation estimation for vectorized data
Weighted average of a set of data values
docs:

Introductory pages
Manual on basic tools
a short intro to Octave and GPR-related statistics

Reconsidered having the initial documentation in English. Since the first target group would be GPR, it is best to start the docs in German.


Enhancement on Substrate Node Template

Updated Substrate Node Template version to 3.0.0
spec_version in runtime is set to 100
Added a documentation on how to release Substrate Node Template
Misc minor enhancement
Copying documentations that are currently in Substrate Node Template back to here (upstream)
The node-template-release.sh script is directly using the path passed in instead of forcing to use a relative path.

thanks @kianenigma for the review. Addressed all of your concerns, and about docker script I will deal with it in a separate issue.

@apopiak thanks for the review 🙏


[9.x] Adds Str::space() helper

While this is a pretty nominal feature, I have reached for the functionality in many projects, so I thought I would see about getting it merged into the framework. This pull request adds a Str::space() helper that will join the passed words with a space and ignore nullable values. In particular, this is useful when joining model attributes that could be null. For example:
// Could be: "John", "John Doe", or "John James Doe"

Str::space($contact->first_name, $contact->middle_name, $contact->last_name);

The filter callback itself specifically checks for a nullable value or an empty string so as to allow for things like the digit 0 in the scenario that someone might desire to pass that.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


Do not fail for source artifact as its added as tasks

Temp fix to unblock source code artifacts execution on xplat agent.

Hi @pagarwal-msft, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Pradeep Agarwal). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

We should do the change specific to an artifact type, Please do not go ahead with this change


Added animatable maxRadius on PieElement

I wanted to animate the max radius of a PieElement, without moving the center.
So I added the property "maxRadius" to PieElement.
Check-out the example 2 to see the effect.
Cheers !

Thank you for contributing. I added maxRadius and minRadius in version 1.1.0, checkout.


⚠ Bofrost.at has degraded performance

In 5c319be, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2388 ms

Resolved: Bofrost.at performance has improved in 0b27bde after 7 minutes.


Check dependency compatibility with Granta MI STK 2.3.1

It looks like there is a dependency conflict between the STK as shipped, and the dependencies of openapi-common, as we expect users to want to use both, we should verify that they can be installed side-by-side

ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
ansys-openapi-common 1.0.0b1 requires importlib_metadata<2.0,>=1.0; python_version < "3.8", but you have importlib-metadata 3.4.0 which is incompatible.
ansys-grantami-bomanalytics 0.1.0.dev0 requires importlib_metadata<2.0,>=1.0; python_version < "3.8", but you have importlib-metadata 3.4.0 which is incompatible.


screenshot support

It would be wonderful if there was a reliable method to capture a screenshot or screen grab of the stream, perhaps on a timer.
I'm currently using this bit of ffmpeg to make an attempt at capturing a screenshot every 15 seconds, it really isn't reliable and seems to spawn a lot of ffmpeg processes (I guess they're orphaned).
exec_push /usr/local/bin/ffmpeg -i rtmp://live.watchme.io/$app/$name -probesize 20M -analyzeduration 400000000 -f image2 -vf fps=fps=1/15 -y -updatefirst 1 /home/ubuntu/public_html/thumb/$name.png;

Having support for something like this would be amazing.

@arut
my setting
recorder preview {
record keyframes;
record_path /tmp/hls;
record_max_frames 10;
record_interval 5s;
exec_record_done ffmpeg -i rtmp://<IP_ADDRESS>:1935/hls/${name} -vcodec png -y /tmp/hls/$name.png;
}
i got message: nginx: [emerg] "recorder" directive is not allowed here in /usr/local/nginx/conf/nginx.conf:200
the line is :  recorder preview {
can you help me?

@SeoulTsai Can I see the whole rtmp server setup in your nginx.conf?

FYI for anyone who gets this, it's because record is under the Application context. If it's outside, you will get that error. It should be.
application live {
    ...
        # make previews
        recorder preview {
                record keyframes;
                record_max_frames 4;
                record_path /data/preview;
                record_interval 30s;
                exec_record_done ffmpeg -i $path -vcodec png -vframes 1 -an -f rawvideo -s 320x240 -ss 00:00:01 -y /data/broadcast/$name.png;
         }
     ...
}#end application


Logging PEM Formatted Private Key

Hi Team,
I noticed recently that there is a DEBUG log that logs PEM formatted private key.
https://github.com/SAP/cloud-security-xsuaa-integration/blob/main/token-client/src/main/java/com/sap/cloud/security/mtls/SSLContextFactory.java#L145
Should we remove it? As far as I know any secrets should not be logged.

I've changed it in that way, that it prints only the first and the last 7 characters.


[lit-next] Remove unneeded scripts

Now they are in the root package.json.
Probably from a wrong merge.

Thanks!


Cannot get repman to realize stable release after history rewritten (self hosted gitlab)

Hello, I have - what I believe is a bug - to report.
I had commit in repository tagged as 1.0.0 and repman was registering it as stable release.
I rewritten the history and marked different commit with same tag.
Now repman won't recognize the stable release.
I tripple checked the tag is indeed there.
Whats even more interesting: Now the package cannot be deleted and repman removes the webhook and then crashes into 500.

We will check this use case and let you know if it is actually a bug. Thanks for reporting 👍

Okay, so we decided to recreate the whole organization. So I tried to delete the original one and got.... Error 500.
Can you help us debug this? Our sys-admin was not able to find anything in repman logs.

Hi @josefsabl, can you double check the logs? They are located in repman's var/logs directory

I am sorry, I am polluting this thread with possibly unrelated stuff. But we have serious problems running repman and we constantly get various 500 responses. We did reset repman completely only to get more sorts of errors:

After clean install and addition of user we were getting 500 on the homepage. Few docker restarts sorted this out.
Now, I want to reset my password / and or create new user. Application says it sent an e-mail but it is never delivered (possibly problem with our infrastructure, but I don't know).
Now, I want to add an existing user to the newly created organization and I, again, get 500. What is interesting, that symfony's exception page is displayed. I assume admin runs it in debug mode (which sort of helps us atm).

I am not familiar with contents of symfony's error page so I rather not post it here, but the error message is:
The stream or file "/app/var/log/prod.log" could not be opened in append mode: failed to open stream: Permission denied
in /app/vendor/monolog/monolog/src/Monolog/Handler/StreamHandler.php (line 111)
We are running it in docker so I assume this may be problem with docker configuration. Are we doing anything wrong?
Thank you!

This problem seems to be on our side.
a) Turned out there were some incorrect permissons set for logs so parts of application could not write to them.
b) Now we see that the real problem standing behind few of 500 was invalid certificate for the https communication.
Still researching it, will post more, once we find out.

Okay, so we solved the 500. The problem was with sending e-mail as we are using our own mailserver which uses self-signed certificated. We had to enable debug mode for Symfony to see the error and by digging in the stack trace we managed to guess the problem was with sending e-mail.
We solved it like this.
MAILER_DSN=smtp://mx1.ourcompany.cz:25?verify_peer=0
This however is probably not related to the potential bug I was reporting in the first place. We just solved the "bug" by reinitializing everything.
So it is up to you if you close this :-)
Thank you very much!


[TYPO] Says Pix instead of pick

Description
In the About Project section. The current pull says, "..you can pix whatever color you prefer..." in the 4th sentence. When I believe it means to say pick.
Expected Behavior
In the About Project section the 4th sentence should read, "...you can pick whatever color you prefer.."
Actual Behavior
4th sentence of the About Project section reads, "...you can pix whatever color you prefer..."
Possible Fix
Replace "x" with "ck"
Steps to Reproduce

Fork the project from  github
Pull up the project from a local server host.
Note the 4th sentence of the About Project section.

Context

It might just be flavor text. In which case this is a great choice! Some times these things can be dry. Maybe add a smiley face ":)" to it .

@ParticleMonk Can you assign it to me, I'll get it done ASAP!

I have fixed this issue. Refer to pull request #4908.
@ParticleMonk


Always force overwrite links

Verification

[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

Provide a detailed description of the proposed feature
I'm sorry for posting this as an issue here, I would have rather opened a discussion about it, if that where available here. Sorry again, you might not like that I'm simply asking a question and am not sure if I even have a feature request.
homebrew on my machine takes quite some time for any package to install. And after "minutes" or even hours it stops continuing because it asks me questions like this:
Error: Could not symlink include/textstyle/stdbool.h
Target /usr/local/include/textstyle/stdbool.h
is a symlink belonging to gettext. You can unlink it:
  brew unlink gettext

To force the link and overwrite all conflicting files:
  brew link --overwrite gettext

To list all files that would be deleted:
  brew link --overwrite gettext --dry-run

Which I always answer by tediously copying and pasting. Yes please overwrite, I don't care!
So what I'm asking myself is, is there any options that allow to simply tell brew to just always overwrite links? I tried this but it doesn't seem to work. I still get stops during longish runs:
brew install --quiet --overwrite --force somepackage

If such an "always overwrite links" option currently is not available then this would actually be the feature I would like to request. Thanks for you consideration! Please get back to me with any thoughts, improvement suggestions of my workflow or criticism on my report! All welcome! Thanks a lot for your time and efforts!

I'm sorry for posting this as an issue here, I would have rather opened a discussion about it, if that where available here.

We do have a discussions page for these sorts of questions.
Does brew install --overwrite and/or brew upgrade --overwrite not do what you need it to?

Probably need a specific example in order to provide context. Like what exact command you were running when it errored and at least some command output before the error.
Without any extra info, I am guessing this is related to linking dependencies since I think --overwrite should work but only for input formula. If so, as far as I can tell, we don't support this. A hack would be passing in everything in dependency tree $(brew deps ...), but that will result in a marking all formulae as installed_on_request and making it difficult to clean up no longer used formulae.

I'm sorry for posting this as an issue here, I would have rather opened a discussion about it, if that where available here.

We do have a discussions page for these sorts of questions.
Does brew install --overwrite and/or brew upgrade --overwrite not do what you need it to?

I'm very sorry I missed that discussions is open in this report.
As said: install --overwrite still asks when dependency links are happening.

As said: install --overwrite still asks when dependency link conflicts are happening.That usually happens with long running compiles. In the end it often takes me literally days to install one piece of software because every other dependency throws such a link overwrite conflict...

Can you be more specific here?

I'll post a report when not AFK

Does that clarify what I'm asking?

Yes, thanks. It is indeed unexpected/a bug that brew upgrade --overwrite will not overwrite in this case. Reopening.

==> Installing curl dependency: brotli
...
Possible conflicting files are:
/usr/local/bin/brotli -> /usr/local/Cellar/brotli/1.0.9/bin/brotli

This is confusing. Very strange that brotli can't handle brotli's own files.
May need to understand your setup better with brew config/brew doctor output.


match_dir fix

Fixes #108 by properly modifying dirs in place.

@glennmatthews thank you so much for your time. Merged!

@Nurdok can we get a new release with this fix? With 0.4.1, pep257 keeps reporting errors about files in my .tox directory no matter how I specify match-dir.

Released in 0.5.0
http://pep257.readthedocs.org/en/latest/release_notes.html#march-14th-2015

Awesome, thanks!


Fix Code sample is missing issue

According to this CATS report:
https://contentqacats.azurewebsites.net/CheckResults/2019080900284/Report/Failed/Missing code samples

I found one issue. Hang on!

The builds look good, @mairaw.

Now the regular expression from the code matches the description 🎉


fix: Add docs for passThroughArgs

Related Issues/PRs

#xxx
What changes are proposed in this pull request?
Add some docs to explain how LightGBM passThroughArgs works
How is this patch tested?

[ ] I have written tests (not required for typo or doc fix) and confirmed the proposed feature/bug-fix/change works.

Does this PR change any dependencies?

[ ] No. You can skip this section.
[ ] Yes. Make sure the dependencies are resolved correctly, and list changes here.

Does this PR add a new feature? If so, have you added samples on website?

[ ] No. You can skip this section.
[ ] Yes. Make sure you have added samples following below steps.

Find the corresponding markdown file for your new feature in website/docs/documentation folder.
Make sure you choose the correct class estimators/transformers and namespace.
Follow the pattern in markdown file and add another section for your new API, including pyspark, scala (and .NET potentially) samples.
Make sure the DocTable points to correct API link.
Navigate to website folder, and run yarn run start to make sure the website renders correctly.
Don't forget to add <!--pytest-codeblocks:cont--> before each python code blocks to enable auto-tests for python samples.
Make sure the WebsiteSamplesTests job pass in the pipeline.

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Word andPhrases Score
CorrectnessScore
Scorecard
Processed

website/docs/features/lightgbm/about.md

link
:warning:

website/versioned_docs/version-0.10.0/features/lightgbm/about.md

link
:warning:

website/versioned_docs/version-0.10.1/features/lightgbm/about.md

link
:warning:

website/versioned_docs/version-0.10.2/features/lightgbm/about.md

link
:warning:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Word andPhrases Score
CorrectnessScore
Scorecard
Processed

website/docs/features/lightgbm/about.md

link
:warning:

website/versioned_docs/version-0.10.0/features/lightgbm/about.md

link
:warning:

website/versioned_docs/version-0.10.1/features/lightgbm/about.md

link
:warning:

website/versioned_docs/version-0.10.2/features/lightgbm/about.md

link
:warning:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

/azp run

Codecov Report

Merging #1749 (8f3749f) into master (2a89e13) will decrease coverage by 4.38%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1749      +/-   ##
==========================================
- Coverage   85.99%   81.61%   -4.39%
==========================================
  Files         271      271
  Lines       14126    14126
  Branches      737      737
==========================================
- Hits        12148    11529     -619
- Misses       1978     2597     +619

Impacted Files
Coverage Δ

...re/synapse/ml/lightgbm/params/LightGBMParams.scala
100.00% <100.00%> (ø)

.../execution/streaming/continuous/HTTPSourceV2.scala
0.00% <0.00%> (-92.15%)
:arrow_down:

...che/spark/sql/execution/streaming/HTTPSource.scala
0.00% <0.00%> (-90.00%)
:arrow_down:

...ql/execution/streaming/continuous/HTTPSinkV2.scala
0.00% <0.00%> (-89.75%)
:arrow_down:

...ql/execution/streaming/DistributedHTTPSource.scala
0.00% <0.00%> (-78.46%)
:arrow_down:

...om/microsoft/azure/synapse/ml/io/IOImplicits.scala
10.16% <0.00%> (-61.02%)
:arrow_down:

...he/spark/sql/execution/streaming/ServingUDFs.scala
0.00% <0.00%> (-52.95%)
:arrow_down:

...t/azure/synapse/ml/core/schema/SparkBindings.scala
60.00% <0.00%> (-40.00%)
:arrow_down:

...icrosoft/azure/synapse/ml/io/http/HTTPSchema.scala
51.77% <0.00%> (-36.88%)
:arrow_down:

...n/scala/org/apache/spark/injections/UDFUtils.scala
75.00% <0.00%> (-12.50%)
:arrow_down:

... and 5 more

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

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Word andPhrases Score
CorrectnessScore
Scorecard
Processed

website/docs/features/lightgbm/about.md

link
:warning:

website/versioned_docs/version-0.10.0/features/lightgbm/about.md

link
:warning:

website/versioned_docs/version-0.10.1/features/lightgbm/about.md

link
:warning:

website/versioned_docs/version-0.10.2/features/lightgbm/about.md

link
:warning:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

/azp run


If an error occurs, the table still gets created

It was my mistake for faulty syntax, but still got an unexpected result, that artisan reports an error, the table gets created, but the migration is not reported as having occurred in the migrations table, and so doesn't get rolled back.
Migration code:
public function up()
{
    //

    Schema::create('test_table', function (Blueprint $table) {
        $table->bigIncrements('id');
        $table->primary('id');
    });
}

/**
 * Reverse the migrations.
 *
 * @return void
 */
public function down()
{
    //
    Schema::dropIfExists('test_table');

}
}

Run the migration. Get error. Rollback. Table still there.
I understand, I did it wrong, but still suggest changing the process. Understanding that different DB drivers have different limitations as far as transactions go. But if a Migration causes an error and doesn't get inserted into the migrations table, perhaps that table should be dropped outside of the down() method.

Laravel Version: 5.6.18
PHP Version: 7.1.14
Database Driver & Version: MariaDB 5.7 (wow! I need to upgrade....)

This sounds more like a feature suggestion. Could you perhaps expand further on the logic that can be used to do this properly and file an issue at https://github.com/laravel/ideas/issues?

I did as you suggested (https://github.com/laravel/ideas/issues/1159). Will close here. Thanks!


MAINT: Enable "no response" bot

Overview
Closes #3422
Add the "no response" bot, to close issues with the "awaiting feedback" label where the original author has not replied. For more info about the bot see: https://github.com/lee-dohm/no-response
This is probably less impactful that the "stale bot" over on #3424 , so we should probably focus on that first.
Checklist

[X] All pre-commit checks pass.
[X] Unit tests added (if fixing a bug or adding a new feature)

Abandoning for now as we have the stale bot working, and I think we should keep things simple and stick with just one bot.


Pentest preparation tasks

Description
This is a tracking issue to capture some of the technical work needed to get to pentest readiness.
Acceptance Criteria

[x] mock out email sending in local environments;
[x] remove SendGrid from dependencies;
[x] remove autoindex and Access-Control-Allow-Origin header from /tiles and /glyphs;
[x] create DAG to generate /glyphs in new environments;
[x] review MOVE dependencies to see which ones are essential;
[x] coordinate around process changes:

[x] CD from deploy-qa branch in QA, rather than master
[x] check sandbox stack, ensure user stories work

[x] configure Content-Security-Policy header in nginx;
[x] deploy new version to QA and:

[x] ensure above fixes / changes;
[x] ensure environment crosstalk has been solved.

Marking as closed after recent deployment 🎉


Router.route('/foo').put() doesn't exist.

react version: n/a
node version: 10.7.0
npm (or yarn) version: 6.2.0

Relevant code or config

In lib/router.js, only route.get is defined.

What you did:
const express  = require('jest-express');
const router = express.Router();
const dummy = (req, res) => res.send('blurg');

// these 'verb' functions must be chainable to work properly
router.route('/:id')
  .get(dummy)
  .put(dummy)
  .delete(dummy);

module.exports = router;

What happened:

Reproduction repository:

Problem description:
Suggested solution:

@jasonseminara, thank you for submitting an issue. I am at Node Summit 2018 this week. If you would like to submit a PR adding this functionality, it would be greatly appreciated. If not I will try and get a fix for this resolve in between sessions.

Awesome! I just updated my code locally to finish my project. I'll convert to ts and create a PR

:tada: This issue has been resolved in version 1.6.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


remove default CSS rule in boilerplate?

hi there,
Would you accept a PR that eliminates the body rule in styles.css? I wasn't aware it was in there, and I hosed a lot of people's themes when they installed my plugin :-)
Let me know and I'd be happy to PR it.

Hasn't been a problem for about a year and a half, but I guess I can change it.

Fixed in 64e86498772c97c6f49fd28698405547fd44dc60


Notebook preload scripts load after the renderer scripts

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

VS Code Version:  Insider
OS Version:  N/A

Steps to Reproduce:

Create a renderer with preload scripts using the new contribution notebookPreload

Add a console.log into this script

Create an output with HTML in a notebook

Add a console.log into the output HTML

Keep the notebook open and reload VS Code
You'll notice that the output HTML script is executed first

I.e. the preloads are loaded after the renderer scripts.

Closing this as I can no longer repro this issue, possible i had an older version of insiders or the like.

I can still repro this issue, I think you should reload vscode with all editors closed, then open the notebook.
Seems to be a timing issue with activation of the extensions or the like.
@mjbvz this blocks https://github.com/microsoft/vscode-jupyter/issues/6404 which was on the iteration plan, it kinda works. Kind of based on the timing of the loading of the pre-load scripts.

This bug has been fixed in the latest release of VS Code Insiders!
@DonJayamanne, you can help us out by commenting /verified if things are now working as expected.
If things still don't seem right, please ensure you're on version a803dc359aa1ed48c7acf074dd81d6af1525c8c8 of Insiders (today's or later - you can use Help: About in the command palette to check), and leave a comment letting us know what isn't working as expected.
Happy Coding!

/verified


Update countBadge.css

reduce the CSS required to align the problems count-badge
N.B the commit message should just be "reduce the CSS required" the previous fix does align vertically.
RE  #132456

@Un-index, thank you very much for the contribution. I have checked out the branch but as you can see from the screenshot below, adding vertical-align: text-top; does not address the vertical alignment issue.

Weird. It seems to be working for me:

I just pulled your branch, compiled and run it. As you can see from the screenshot below, the monaco-count-badge does have vertical-align set but the alignment does not work for me as expected. The badge continues to be aligned to the bottom of its container.

Could you tell me how to compile from my branch so I could further see what's happening?

You can find the list of prerequisites here and the list of steps to build and run here.

I built it from my repo and it seems to be aligned:

I have tested your pull request on macOS (using scripts/code.sh) though I would not have expected to have different behaviour on different operating systems. I wonder if it might be related to zoom levels. Are you using any custom zoom level on your machine? I am using "window.zoomLevel": 1 on the machine on which I am testing. Also are you saying that you have confirmed that your approach does not work in yarn web? If so, we need to opt for a solution that works on all runtime platforms (desktop, and web).

I meant it could be possible that it doesn't work there (was assuming that's where you tested it) although I don't see any reason why that could happen so it probably works there too (problems are not showing up for me there so I can't test that).
I've tested the build on another Windows device and it seems to working there as well. I've tested with multiple zoom levels  ("window.zoomLevel":  -1, 0, 1, 2) and that seems fine too.

I did revert my change as while the fix was working it was too generic. I will take another look at this later this week.

Maybe just one more person could test it to see if this fix is specific to my device?

@lszomoru  hi, it's been a while
do you think we should close this issue since there might be things that warrant more directed activity

@Un-index, I still want to get to the bottom of this, but will push this out to October.

Closing this as beyond stale.


Add WebAssembly

I wondered whether watching a Content-Type or a extension, but I choose the former. Because after I saw codes roughly, I noticed implement of "scripts" make browser evaluate a script. I don't know can browsers evaluate wasm byte-codes directly, but I tried it and it worked. Though it seemed strange action, so I decide to watch headers.

I have remembered I found a problem, though it doesn't relate directly. But its scope is broad.

Thanks!


Audit all watchers to look for missing boto pagination tokens.

Audit Summary
The security_monkey package is missing pagination on the following calls:

IAM User get_all_user_policies, get_all_access_keys, get_all_mfa_devices, get_all_signing_certs
IAM Role list_role_policies
Redshift describe_clusters

Impact
Typical AWS limits on user policies, access keys, mfa devices, and signing certs attached to a single user would likely prevent this from being an issue.
Typical AWS limits on role policies attached to a single role would likely prevent this from being an issue.
An environment which has a large number (>100) of redshift clusters may be affected by this issue.

http://docs.aws.amazon.com/redshift/latest/APIReference/API_DescribeClusters.html

Needs Work
IAM Users

Using a marker
https://github.com/Netflix/security_monkey/blob/master/security_monkey/watchers/iam_user.py#L63
Need to add it to get_all_user_policies
http://boto.readthedocs.org/en/latest/ref/iam.html#boto.iam.connection.IAMConnection.get_all_user_policies
https://github.com/Netflix/security_monkey/blob/master/security_monkey/watchers/iam_user.py#L96
Need to add it to get_all_access_keys
http://boto.readthedocs.org/en/latest/ref/iam.html#boto.iam.connection.IAMConnection.get_all_access_keys
Need to add it to get_all_mfa_devices
http://boto.readthedocs.org/en/latest/ref/iam.html#boto.iam.connection.IAMConnection.get_all_mfa_devices
Need to add it to get_all_signing_certs
http://boto.readthedocs.org/en/latest/ref/iam.html#boto.iam.connection.IAMConnection.get_all_signing_certs

IAM Roles

Using a marker in one place
https://github.com/Netflix/security_monkey/blob/master/security_monkey/watchers/iam_role.py#L61
Need to add it to list_role_policies
http://boto.readthedocs.org/en/latest/ref/iam.html#boto.iam.connection.IAMConnection.list_role_policies
https://github.com/Netflix/security_monkey/blob/master/security_monkey/watchers/iam_role.py#L92

Redshift

Needs a marker
https://github.com/Netflix/security_monkey/blob/master/security_monkey/watchers/redshift.py#L56
http://boto.readthedocs.org/en/latest/ref/redshift.html#boto.redshift.layer1.RedshiftConnection.describe_clusters

Complete The following have all their markers / pagination.
-IAM Group-

Using a marker
https://github.com/Netflix/security_monkey/blob/master/security_monkey/watchers/iam_group.py#L48
https://github.com/Netflix/security_monkey/blob/master/security_monkey/watchers/iam_group.py#L67
https://github.com/Netflix/security_monkey/blob/master/security_monkey/watchers/iam_group.py#L86

-IAM SSL-

Using a marker
https://github.com/Netflix/security_monkey/blob/master/security_monkey/watchers/iam_ssl.py#L141

-Security Groups-

get_all_security_groups does not take a marker
http://boto.readthedocs.org/en/latest/ref/ec2.html#boto.ec2.connection.EC2Connection.get_all_security_groups

-S3-

get_all_buckets does not take a marker
http://boto.readthedocs.org/en/latest/ref/s3.html#boto.s3.connection.S3Connection.get_all_buckets
get_policy does not take a marker
http://boto.readthedocs.org/en/latest/ref/s3.html#boto.s3.bucket.Bucket.get_policy
get_versioning_status does not take a marker
http://boto.readthedocs.org/en/latest/ref/s3.html#boto.s3.bucket.Bucket.get_versioning_status

-SSH Keys (Keypairs)-

get_all_key_pairs does not take a market

-RDS-

Using a marker
https://github.com/Netflix/security_monkey/blob/master/security_monkey/watchers/rds_security_group.py#L63

-SES-

list_identities does not take a marker
http://boto.readthedocs.org/en/latest/ref/ses.html#boto.ses.connection.SESConnection.list_identities
list_verified_email_addresses does not take a marker
http://boto.readthedocs.org/en/latest/ref/ses.html#boto.ses.connection.SESConnection.list_verified_email_addresses

-SQS-

get_all_queues does not require a marker
http://boto.readthedocs.org/en/latest/ref/sqs.html#boto.sqs.connection.SQSConnection.get_all_queues

-SNS-

Using marker on get_all_topics
https://github.com/Netflix/security_monkey/blob/master/security_monkey/watchers/sns.py#L90
Using marker on get_all_subscriptions_by_topic
https://github.com/Netflix/security_monkey/blob/master/security_monkey/watchers/sns.py#L108
get_topic_attributes does not require a marker
http://boto.readthedocs.org/en/latest/ref/sns.html#boto.sns.SNSConnection.get_topic_attributes

-ELB-

Using a marker
https://github.com/Netflix/security_monkey/blob/master/security_monkey/watchers/elb.py#L133

-EIP-

http://boto.readthedocs.org/en/latest/ref/ec2.html#boto.ec2.connection.EC2Connection.get_all_addresses
no marker required

IAM Role list_role_policies was corrected in commit e29e56bae3fdcf611d74df86322e7a1aa3cf71d2

Redshift describe_clusters was corrected in commit caec7ad88f68e5b63d22fbe909d735ec7ab66bdd

IAM User get_all_user_policies, get_all_access_keys, get_all_mfa_devices, get_all_signing_certs was corrected in commit c034b41ce7572922d39f530bcc53a65f9216ee73


Keybind for toggling live sharing

Maybe with a little status window you can use to see if it's enabled?

Fixed in 5da02b3357fbecd3e2bcbcd5fb8b73fe360c4266


BRP System Ordering

Objective

Attempts to fix #16042

Solution

Added a new RemoteSystem SystemSet for the BRP systems.
Changed the schedule on which these systems run from Update to Last.

Testing

I did not test these changes and would appreciate a hand in doing so. I assume it would be good to test that you can order against these systems easily now.

Migration Guide

process_remote_requests, process_ongoing_watching_requests and remove_closed_watching_requests now run in the Last schedule. Make sure you use RemoteSystem SystemSet in case you need to order your systems against them.

I mentioned it in the issue, but I'd consider adding a custom schedule for this, to ensure BRP-unaware users don't mix their systems with BRP systems.
In that case a system set seems pretty unnecessary, at least for the current system complexity

@MiniaczQ should this custom schedule run before Last? Wondering this since if it runs after, Last won't technically be the schedule that runs last anymore.

I've rebased the branch and pushed changes to use a custom schedule with some assumptions. Please correct me if I did something wrong.

lgtm

@mockersf I've updated the migration guide (although there's really nothing to migrate per se, just a note) post merge since it had outdated information, is that enough or is this required to be changed somewhere else? I'm thinking about the website generated pages and such.

migration guide isn't really required for this as it's new, there's a release note https://github.com/bevyengine/bevy-website/pull/1773 but it doesn't mention anything about ordering


How can I do to connect c# mvc with jasper server for to get report

Hi, I´m new with jasper report, I have my server with Jasper server and I ready can to deploy my report, but I need to show my report in my app with view mvc and c#, I saw tutorial about of theme and  comment on the jasper client for .net, but and I don´t know about how can I do to show my report in page MVC I hav this code
var rs = new ReportingService("http://localhost:8080/", "jasperadmin", "jasperadmin");
var report = await rs.RenderByNameAsync("xxx", new { xxx = "***" });
anyone have some code for to show report in c# mvc, for to show report in app
I see this code but I don´t understand how to setup my server http://localhost:8080/", "jasperadmin", "jasperadmin") for to get my report that I design in my jaspersoft studio and deploy in my jaspersoft server, this is the code I saw:
public ActionResult Index()
{
return View();
}
    [EnableJsReport()]
    public ActionResult Invoice()
    {
        HttpContext.JsReportFeature().Recipe(Recipe.ChromePdf);

        return View(InvoiceModel.Example());
    }

    [EnableJsReport()]
    public ActionResult InvoiceDownload()
    {
        HttpContext.JsReportFeature().Recipe(Recipe.ChromePdf)
            .OnAfterRender((r) => HttpContext.Response.Headers["Content-Disposition"] = "attachment; filename=\"myReport.pdf\"");

        return View("Invoice", InvoiceModel.Example());
    }

    [EnableJsReport()]
    public ActionResult InvoiceWithHeader()
    {
        HttpContext.JsReportFeature()
            .Recipe(Recipe.ChromePdf)
            .Configure((r) => r.Template.Chrome = new Chrome {
                HeaderTemplate = this.RenderViewToString("Header", new { }),
                DisplayHeaderFooter = true,
                MarginTop = "2cm",
                MarginLeft = "1cm",
                MarginBottom = "2cm",
                FooterTemplate = " "
            });

        return View("Invoice", InvoiceModel.Example());
    }

    [EnableJsReport()]
    public ActionResult Items()
    {
        HttpContext.JsReportFeature()
            .Recipe(Recipe.HtmlToXlsx);

        return View(InvoiceModel.Example());
    }

    [EnableJsReport()]
    public ActionResult ItemsExcelOnline()
    {
        HttpContext.JsReportFeature()
            .Configure(req => req.Options.Preview = true)
            .Recipe(Recipe.HtmlToXlsx);

        return View("Items", InvoiceModel.Example());
    }

    [EnableJsReport()]
    public ActionResult InvoiceDebugLogs()
    {
        HttpContext.JsReportFeature()
            .DebugLogsToResponse()
            .Recipe(Recipe.ChromePdf);

        return View("Invoice", InvoiceModel.Example());
    }

I saw this in example, in this foroum the workflow is design Jaspersoft studio-> deploy Jaspersoft server with parameter-> and I need to see on my app mvc c# in my view
best regards

JasperReports is a completely different technology.
This is the repo for jsreport .net client.... you are probably at the wrong place.


Include please build tool

The please build tool would be nice to have represented.

Hello! I've just seen this. I can probably take a crack at this :)


Feat- Added method to check if payment is enabled, and fixed tests

Tests to be fixed

@MSghais You can merge this, the tests are fixed, and added the changes you wanted in the nameservice.cairo

Tests are not fixed, you broke 84 test and more. Please check @PoulavBhowmick03


Provide a way to enforce a 0 return code from 'tsc'

TypeScript Version:  2.3.4
Code
tsc -p client 2>&1 && compile another project
tsc -p client >/dev/null 2>&1 && compile another project

Expected behavior:
In a real project. Sometime I cannot fix all ts error. But it's ok to emit js files.
The tsc command seem always return error code.
Actual behavior:
tsc -p client 2>&1 || compile another project

Is the problem that you want to be able to get a 0 return code so that you can continue build scripts? I'm assuming so and changing this issue's title to reflect that.

CC @bowdenk7

Yes @DanielRosenwasser. That's what I'm saying. Sorry for my bad English.

No worries! Thanks for filing this issue, this is something that we've experienced ourselves as well.

Is adding || true on the command line good enough?
e.g.
tsc -p client 2>&1 || true && compile another project

@gcnew This worked. But after you resolve all ts problem. tsc will return 0.

@gcnew It doesn't work in Windows which is the real pain-point in our experience.

Declining due to shortage of demand


BVAL-535 Reduce the number of compilation warnings in the API

https://hibernate.atlassian.net/browse/BVAL-535

Build triggered. sha1 is merged.

Build started sha1 is merged.

Build finished. 8 tests run, 0 skipped, 0 failed.

Merged; thanks, @gsmet.


(git.install) GH-1867: powershell 2 compatibility

Description
fixes GH-1867

:white_check_mark: Package verification completed without issues. PR is now pending human review

:white_check_mark: Package verification completed without issues. PR is now pending human review


An error occurred during the fetch of repository 'local_config_cuda'

Click to expand!
Issue Type
Build/Install
Source
source
Tensorflow Version
tf 2.8
Custom Code
Yes
OS Platform and Distribution
Linux Ubuntu 20.04
Mobile device
No response
Python version
3.9
Bazel version
4.2.1
GCC/Compiler version
9.4.0
CUDA/cuDNN version
CUDA 11.7 /cuDNN 8
GPU model and memory
A100 80G
Current Behaviour?
A bug happened!

Standalone code to reproduce the issue
git clone https://github.com/tensorflow/tensorflow tensorflow -b v2.8.0 --depth=1

Relevant log output
ERROR: An error occurred during the fetch of repository 'local_config_cuda':
   Traceback (most recent call last):
        File "/home/xizhang/software/dpmd/tensorflow/third_party/gpus/cuda_configure.bzl", line 1401, column 38
, in _cuda_autoconf_impl
                _create_local_cuda_repository(repository_ctx)
        File "/home/xizhang/software/dpmd/tensorflow/third_party/gpus/cuda_configure.bzl", line 1076, column 27
, in _create_local_cuda_repository
                cuda_libs = _find_libs(repository_ctx, check_cuda_libs_script, cuda_config)
        File "/home/xizhang/software/dpmd/tensorflow/third_party/gpus/cuda_configure.bzl", line 606, column 21,
 in _find_libs
                _check_cuda_libs(repository_ctx, check_cuda_libs_script, check_cuda_libs_params.values())
        File "/home/xizhang/software/dpmd/tensorflow/third_party/gpus/cuda_configure.bzl", line 501, column 28,
 in _check_cuda_libs
                checked_paths = execute(repository_ctx, [python_bin, "-c", cmd]).stdout.splitlines()
        File "/home/xizhang/software/dpmd/tensorflow/third_party/remote_config/common.bzl", line 230, column 13
, in execute
                fail(
Error in fail: Repository command failed
Expected even number of arguments
ERROR: Error fetching repository: Traceback (most recent call last):
        File "/home/xizhang/software/dpmd/tensorflow/third_party/gpus/cuda_configure.bzl", line 1401, column 38, in _cuda_autoconf_impl
                _create_local_cuda_repository(repository_ctx)
        File "/home/xizhang/software/dpmd/tensorflow/third_party/gpus/cuda_configure.bzl", line 1076, column 27, in _create_local_cuda_repository
                cuda_libs = _find_libs(repository_ctx, check_cuda_libs_script, cuda_config)        File "/home/xizhang/software/dpmd/tensorflow/third_party/gpus/cuda_configure.bzl", line 606, column 21, in _find_libs
                _check_cuda_libs(repository_ctx, check_cuda_libs_script, check_cuda_libs_params.values())
        File "/home/xizhang/software/dpmd/tensorflow/third_party/gpus/cuda_configure.bzl", line 501, column 28, in _check_cuda_libs
                checked_paths = execute(repository_ctx, [python_bin, "-c", cmd]).stdout.splitlines()
        File "/home/xizhang/software/dpmd/tensorflow/third_party/remote_config/common.bzl", line 230, column 13, in execute
                fail(
Error in fail: Repository command failed
Expected even number of arguments
INFO: Found applicable config definition build:cuda in file /home/xizhang/software/dpmd/tensorflow/.bazelrc: --repo_env TF_NEED_CUDA=1 --crosstool_top=@local_config_cuda//crosstool:toolchain --@local_config_cuda//:enable_cuda
INFO: Found applicable config definition build:cuda in file /home/xizhang/software/dpmd/tensorflow/.bazelrc: --repo_env TF_NEED_CUDA=1 --crosstool_top=@local_config_cuda//crosstool:toolchain --@local_config_cuda//:enable_cuda
WARNING: The following configs were expanded more than once: [cuda]. For repeatable flags, repeats are counted twice and may lead to unexpected behavior.
ERROR: @local_config_cuda//:enable_cuda :: Error loading option @local_config_cuda//:enable_cuda: Repository command failed
Expected even number of arguments

Hi @Johnsyisme
Sorry for the late reply. I'm able to clone the tensorflow repository without any problem on Ubuntu. I observed that you are using GCC 9.4.0 and CUDA 11.7 which is incompatible with TF v2.8.0. Could you please create a virtual environment and try to install the tensorflow as mentioned in this official document link and have a look at the compatible tested build configurations as well.  Please find the attached screenshot for reference.
Thank you!

Hi, is this solved?


crop_resize() error

Traceback (most recent call last):
  File "problem7.py", line 565, in <module>
    affine_ten_img, random_ten_param = ten_crop(pil_img, ori_gt)
  File "problem7.py", line 499, in ten_crop
    ori_img, affine_img, cropped_img, new_gt = RAO.all_activities()
  File "problem7.py", line 448, in all_activities
    new_gt)
  File "problem7.py", line 400, in random_occlusion
    new_gt)
  File "problem7.py", line 330, in crop_resize
    crop_img = TF.crop(img,
  File "/home/sravani/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/torchvision/transforms/functional.py", line 453, in crop
    raise TypeError('img should be PIL Image. Got {}'.format(type(img)))
TypeError: img should be PIL Image. Got <class 'torch.Tensor'>

Hi, just transform the image to pil image here,  using output=
Image.fromarray(input).
I also updated codes in GitHub by using csv as the input. You can also
check it out.
Wenjing
On Mon, Mar 1, 2021, 7:45 AM SravaniTeeparthi29<EMAIL_ADDRESS>wrote:

Traceback (most recent call last):
File "problem7.py", line 565, in
affine_ten_img, random_ten_param = ten_crop(pil_img, ori_gt)
File "problem7.py", line 499, in ten_crop
ori_img, affine_img, cropped_img, new_gt = RAO.all_activities()
File "problem7.py", line 448, in all_activities
new_gt)
File "problem7.py", line 400, in random_occlusion
new_gt)
File "problem7.py", line 330, in crop_resize
crop_img = TF.crop(img,
File "/home/sravani/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/torchvision/transforms/functional.py", line 453, in crop
raise TypeError('img should be PIL Image. Got {}'.format(type(img)))
TypeError: img should be PIL Image. Got <class 'torch.Tensor'>
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ntswj1/DataAugmentation/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH6CU2TX3IYTOBPQLC7PFGTTBOSBJANCNFSM4YMVNADQ
.

Traceback (most recent call last):
  File "DataTransformations.py", line 611, in <module>
    main()
  File "DataTransformations.py", line 607, in main
    affine_ten_img, random_ten_param = ten_crop(pil_img, ori_gt)
  File "DataTransformations.py", line 559, in ten_crop
    ori_img, affine_img, cropped_img, new_gt = RAO.all_activities()
  File "DataTransformations.py", line 513, in all_activities
    new_gt)
  File "DataTransformations.py", line 465, in random_occlusion
    new_gt)
  File "DataTransformations.py", line 400, in crop_resize
    MaxCol-occ_j)
  File "/home/sravani/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/torchvision/transforms/functional.py", line 453, in crop
    raise TypeError('img should be PIL Image. Got {}'.format(type(img)))
TypeError: img should be PIL Image. Got <class 'torch.Tensor'>

I am still getting the same error with updated code too

It is a pytorch version problem. For me it worked with version 1.7.1.


Issue with Delta Callback function Registered to Update Topic

Describe the bug
In v1 of the Shadow library, the changes that were made caused the Delta Callback to be registered to the Update Topic. This was due to the variable callbackInfo being passed into the AwsIotShadow_SetUpdatedCallback function instead of the pointer, pCallbackInfo.
System information

Version of Amazon FreeRTOS: master (c334212b824)

Expected behavior
Delta callback should only be called with Delta topic data, not with Update topic data
Additional context
See PR #825 for the change that needs to be made.

Tom, Thanks very much for this.  We are working on merging the PR.

Closing the issue as #825 has been merged.   Thanks for pointing this out.


Theory guide cfd dem

Description of the problem

Lethe did not have an Unresolved CFD-DEM section in the theory guide

Description of the solution

Started it.

Comments

Further updates on it could include weak forms and an algorithm chart.

@voferreira can you upload the images in .png instead of .jpg? Also, if possible, can you put the original .svg (if this is what you used) in the repository?

@blaisb Professor, I added a warning on the void fraction section, talking about the cell to particle size proportion. I forgot to put it there in the previous version. I'm not sure about the placement. What do you think?


Prepare for new release of Jira

Workflow links: https://github.com/atlassian-labs/atlassian-slack-integration-server/actions/runs/2924158293, https://github.com/atlassian-labs/atlassian-slack-integration-server/actions/runs/2924159355.

This issue will be addressed in #219


synapse server can not restart after homeserver.yaml updated

install & updated yaml according to instruction,then restart synapse
then always got message like
--Job for matrix-synapse.service failed because the control process exited with error code.
--See "systemctl status matrix-synapse.service" and "journalctl -xe" for details.
just wonder why?

You can use journalctl -u matrix-synapse for the complete log, press G to jump to the end.
Also please use codeblocks for commands/code/logfiles.

To be able to help you, we need

meaningful logs, covering the the time period between starting Synapse and it producing the error message
to know which Synapse version you are using and how you installed it
to know which synapse-auto-accept-invite version you are using and how you installed it.

Note: journalctl -eu matrix-synapse will take you straight to the bottom of the logs, which is useful when trying to see why it's not starting up.
It would be useful to know what your changes were.

I'm going to close this, since we can't help you without logs. @NewOcean11: please re-open if you manage to get logs.


Environment variables used to set auth doesn't work

I am trying to use the tool with a target repository that requires authentication. Using the yaml everything works as expected:
target:
   auth:
     # username is the username used to authenticate against the target chart repo
     # `TARGET_AUTH_USERNAME` env var can be used instead of this entry
     username: "robot$chart-syncer"
     # password is the password used to authenticate against the target chart repo
     # `TARGET_AUTH_PASSWORD` env var can be used instead of this entry
     password: "somePasswrod"

but when using the env. variables (both TARGET_AUTH_* and TARGET_REPO_*) with the same values, the authentication doesn't work:
POST chart status Code: 401, Message: {"code":401,"message":"UnAuthorized"}

Thanks for the issue! I created this PR with a fix #30


Update README.

Includes changes from the README doc by Ava, Emily, and I.

Could we remove the gifs with real receipts in them?

Could we remove the gifs with real receipts in them?

Done.


Add jars to maven central repository

Please add these to the maven central repository

I'll work on it, but why not use it via jitpack.io?

@mhewedy everything that's not Maven Central is a bit tougher to work with. If you're behind a repository manager like Nexus you'll need to add the repository there and probably add some access rules. If you're not behind a repository manager you'll need to add it to settings.xml and hope it behaves well. Adding it directly to pom.xml is not recommended. Adding it to Maven Central adds a little bit of security and convenience.

There would be very serious security concerns with adding some random 3rd party remote repository to our nexus installation. For the most part, if your stuff isn't in the maven central repository with PGP signatures, most people can't/won't use it.

@marschall @joineral32 thanks for clarifications, I am working on it.

I've created this ticket to deploy the project to the maven repository https://issues.sonatype.org/browse/OSSRH-28783

Issue fixed, and you can include it in mvn (gradle and other build tools) without adding jitpack, however it still not searchable in mvn central repo.

The sync with Maven Central may take a few hours, it's available now:
http://search.maven.org/#search|ga|1|a%3A"spwrap"


Todo list crud

In this Todo list crud apply the following features 💯

Created Array of objects
Add Todo dynamically 👍
Update Todo 👍
Delete Todo 👍
setup the project according to webpack and ES linters 👍

Thanks for the review


Update README URLs based on HTTP redirects

Created with https://github.com/dkhamsing/frankenstein
GitHub Corrected URLs

Was
Now

https://github.com/openaustralia/planningalerts-app
https://github.com/openaustralia/planningalerts

Other Corrected URLs

Was
Now

https://netguru.co/opensource
https://www.netguru.co/opensource

@ReadmeCritic thanks for your contribution!

:tada:


RFC assignee and sub-team

These are referenced in the README without an explanation of how they come to be. We'll need to address that.

Is the RFC assignee always going to be a maintainer?
I take it that the sub-team is not necessarily a team consisting of just maintainers. Would that be accurate?

Let's discuss how this work and I can put in the necessary changes via a new PR.

sub-team is a Sawtooth reference, I meant to change all those to 'maintainers'. As for assignee, I think yes.

Alright, will do.


Do not throw in normalize_type_impl for AbstractT.

Autocompletion for react components doesn't work with current master. It happens because state field has abstract type.
More concise example where autocompletion doesn't work:
class A {
  state: $Abstract<{}>;
}

class B extends A {
  render() {
    this.<cursor here>
  }
}

@facebook-github-bot shipit

thanks :)


Fix for Welsh upload documents

https://dluhcdigital.atlassian.net/browse/FS-4032
Change description
Adding fix for welsh documents not appearing in the areas that they should be

[ ] Unit tests and other appropriate tests added or updated
[ ] README and other documentation has been updated / added (if needed)
[ ] Commit messages are meaningful and follow good commit message guidelines (e.g. "FS-XXXX: Add margin to nav items preventing overlapping of logo")

How to test
Create a welsh application a check the areas where document uploads should be. All document upload section is a good example
Screenshots of UI changes (if applicable)

Great work 🔥. One question, as welsh language is handled in assessment backend no need to change anything in assessment frontend right?

Yes, I had no need the change the frontend. I wanted to avoid it so the back end handles it all


Add support for .napari-hub folder

Summary
To improve clarity around the scope and separation of concerns between the napari hub configuration folder (currently .napari/) and the npe2 manifest file (canonically src/napari.yml), we will add support for .napari-hub/ with the goal of deprecating our support for .napari/.
Outcome
For any metadata that we currently direct plugin developers to set via files in .napari/, they will be able to set through the same files in .napari-hub/.
How it will work
We will add .napari-hub/ as a source for the relevant metadata, taking precedence over .napari/ if both are found.

implementation: https://github.com/chanzuckerberg/napari-hub/blob/9137a96a8d07904b60edccc604ea4ff951bbdb56/backend/utils/github.py#L107


Move "About" and "Change Log" from Accounts to Settings screen

I'm starting to move functionality out of the Accounts Activity so we can get rid of it. The idea is to switch to a navigation drawer that contains an account switcher.
This PR moves the About dialog. It is now a separate Activity. I'm not sure where the "Change Log" option should end up. For now it's a separate entry in the main settings list.

How about moving the changelog into About? It's not like we need to optimize for clicks here, and it seems like the right place to me

Works for me. How would you do it? Option in the overflow menu? Button at the bottom of the screen?

Overflow should be fine

Changed.


SAK-49917 dashboard. Improve responsiveness on small displays

https://sakaiproject.atlassian.net/browse/SAK-49917

Found an issue with the image updating. I'll fix that before merging. Thanks for reviewing @ottenhoff


Is there any way to display text with Rich Text Format?

Is there any way to display text with Rich Text Format with proper stylings?
eg.
"Hello World"
Thanks!

If you're asking if you can set the color, weight, and style of text, then yes, certainly that is possible.  The README covers both creating your own style classes for styles you use repeatedly and applying one-off styles to paragraphs and text blocks.  There are also several examples of this kind of thing in the Caracal example repo.
If you're asking if you can paste HTML snippets directly into the document, then the answer is no, you cannot do that.