Add php-cs-fixer support

This PR Will:

Add PHP constraints: "^5.6 || >=7.0 <7.3"
Add php-cs-fixer to require-dev;
Add composer scripts: check-style & fix-style
Add CONTRIBUTING.md file
Update README.md file

We can check the code standard using git hook pre-commit like this:
composer.json
"scripts": {
    "...": "...",
    "post-install-cmd": "php cghooks add --ignore-lock"
},
"extra": {
    "hooks": {
         "pre-commit": "composer fix-style",
         "post-commit": "echo committed!"
    }
}

We can also provide more details this process in the Contributing.md file ...
What do you think?
P.S: (Sorry, English is not my native language :sweat_smile:. Learning every day... :muscle:!)

This is amazing! 🎉 💯 I really like the Contributing.md. Good job 👍
Could you add the check-style check to .travis.yml? So that it fails if the style is incorrect.

Done!

Check code standards with php-cs-fixer
Add composer cache directory ( accelerate the build process... )
Add 'travis_retry' ( prevents Travis stay stuck...)

Ok one last thing. There's this message in travis

You are running PHP CS Fixer with xdebug enabled. This has a major impact on runtime performance.
If you need help while solving warnings, ask at https://gitter.im/PHP-CS-Fixer, we will help you!

Can you check it out? It's nothing major, and I'll merge if you're not able to.

Humm... Of course! I'll check :smiley:

See: https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/1bf490754d9f1168e501c4e4eb8491a73c231f08/.travis.yml#L20
I'll add this ...


[Bug] Cursor and Character Shift

Reproducible in vscode.dev or in VS Code Desktop?

[X] Not reproducible in vscode.dev or VS Code Desktop

Reproducible in the monaco editor playground?

[x] Not reproducible in the monaco editor playground

Monaco Editor Playground Link
No response
Monaco Editor Playground Code
No response
Reproduction Steps
No response
Actual (Problematic) Behavior
No response
Expected Behavior
No response
Additional Context
Hi,
I don't know how to show the problem, it's not a problem that happens all the time. Sometimes I have it in Codepsaces and vscode.dev and also in monaco editor which I started to create today.
A video of the problem:
Screen recording 2023-11-14 11.16.48.webm
I have to refresh the page several times each time to solve the problem.

I found the solution: #392


NSI to MBIR parameter conversion

This PR contains the updates regarding the parameter calculation from NSI parameters to MBIR parameters.
With this update, we are able to consistently produce high-resolution MBIR reconstructions for NSI datasets.
Interface change
Function "NSI_load_scans_and_params" accepts a new variable "geom_report_path", which specifies the path to "Geometry Report.rtf" file.

Implementation
This PR contains a lot of changes, but the core changes are the following two lines in "preprocess.py":

These two lines correspond to the following two functions that we discussed and agreed on:

Dependency change:
A new dependency "striprtf" is added to "requirements.txt". This package is used to read rtf file.
Image quality testing:
As we saw in my weekly slides, the image resolution is tested with five datasets. In all cases the nominal MBIR parameters produce the best reconstruction quality and resolution.
Demo scripts:
The NSI demo scripts are updated accordingly.
you may run "demo_nsi_preprocess.py" to ensure that there are no bugs in the code. I sent out the updated datasets in a separate email.

Close this PR for further modifications.

Modification finished. PR is ready for testing. You may test the code by running demo_nsi_preprocess.py. This demo takes ~20 minutes on a Mac studio.

I modified the comments and the demo scripts based on the feedback this morning.
Could you test the code by running "demo_nsi_proprocess.py", and accept the PR?

I ran the demo, and it worked fine.


Error in web chat "Authorization for Microsoft App ID failed with status code Forbidden"

I have integrated my bot with Facebook, skype & web chat. Fb & Skype are working fine. But I am getting forbidden exception for web chat. It is not sending the replies. I have updated the framework to 3.1.0 as well.
Thanks in advance.

Are you using the Direct Line secret in your web chat url ?

Yes...I'm using key mentioned in the embedded code.

can you tell me your bot id?

msboombot

Hi,
I'm still getting the same error. Any update on this issue.

Hello, thanks for your patience. I can see the issue occurring. Are you using C# or Node.js ?

I am facing this same issue while trying to update my bot, it fails on skype and telegram, haven't tested other channels. My bot is in C#

Hi @manojkota - can you share a code sample of your dialog structure and how you are sending messages?

I'm generating random messages based on internal process. But my basic implementation is this.
var reply = await connector.Conversations.SendToConversationAsync(item);

I'm having the same issues :( BotID  "eldur3"

Thanks @manojkota and @liiver - at this point it would be very helpful for us to diagnose your issue if you can provide a more complete code sample which reproduces the issue. We cannot reproduce the scenario so if you can isolate a code block that will be valuable for us.

For me it seems similas as with @manojkota
So debugging it also locally, the exeption is either thrown in
await connector.Conversations.ReplyToActivityAsync(reply);
or
await Conversation.SendAsync(activity, () => new MainDialog());
i.e. it It does not seem to go onwards from there to my MainDialog() which is a LuisDialog
But in the first example, there is no LuisDIalog involved, but the same error. Seem bug-ish to me.
Using BotBuilder 3.2.0
Code example:
            if (activity.Type == ActivityTypes.Message)             {                 await Conversation.SendAsync(activity, () => new MainDialog());             }             else if (activity.Type == ActivityTypes.ConversationUpdate)             {                 ConnectorClient connector = new ConnectorClient(new Uri(activity.ServiceUrl));                 Activity reply = activity.CreateReply(activity.From.Name);                 await connector.Conversations.ReplyToActivityAsync(reply);             }

any updates on this one? Seems to be some bug then or?

@manojkota and @liiver it would be good if you can share the code that repros this. Creating a repo with the code that have a repro for this issue will help us diagnose this issue.

Mine is a private repository. I can't provide access to it.

@manojkota would be good if you can create a project that has the minimal code to repro this issue.

@msft-shahins, somehow the issue is resolved on its own. Thank you for your help.

I have the same issue. I have a well working bot that replies to messages in Slack and Skype. for my Slack users the bot also sends scheduled message every morning. I have a very simple code that once worked and than it started giving the "Authorization for Microsoft App ID ... failed with status code Forbidden" error .
Using latest bot framework NuGet  Microsoft.Bot.Builder 3.5.
I triple checked I have my MicrosoftAppId and MicrosoftAppPassword in the app config file, also tried to specify explicitly in the code below. Same error.
Here is the test code that I thought should work:

private void button1_Click(object sender, EventArgs e)
{
var connector = new ConnectorClient(new Uri("https://slack.botframework.com"), ".. my app id removed here...", "...removed password...");
var conversation = new ConversationAccount(true, "B0GRPT96X:T04EVT5HU:C0L57SCKY");
var botAccount = new ChannelAccount("B0GRPT96X:T04EVT5HU", "grasp");
IMessageActivity message = Activity.CreateMessageActivity();
message.From = botAccount;
message.Conversation = conversation;
message.ChannelId = "slack";
message.Text = "some text";
message.Locale = "en-En";
connector.Conversations.SendToConversation((Activity)message);
}

If I call

bool t = MicrosoftAppCredentials.IsTrustedServiceUrl("https://slack.botframework.com");

it returns false.
Thanks for your help.

@danmarshall , @msft-shahins  - I just noticed this issue was closed. Can you provide any guidance on the error I still get? Or should I open a new issue. thanks


Convenience methods for range-check operations

So, I find myself often doing something like val.lanes_ge(start) & val.lanes_le(end) and was wondering how well an alternative like val.lanes_range(start..=end) would fit on the provided SIMD types. I don't know of any platform which would provide this as a built-in operation, and it would be equivalent to just running the operations in sequence.
I would offer a PR for this, but I mostly ask since I'm not sure if it would be reasonable to offer "convenience" operations like this instead of purely ones based upon actual SIMD instructions. To fully clarify, the implementation would be effectively:
use core::ops::{RangeBounds, Bound};

#[inline]
fn simd_range(self, other: impl RangeBounds<Self>) -> Self::Mask
{
    let start = match other.start_bound() {
        Bound::Included(&start) => Some(self.simd_ge(start)),
        Bound::Excluded(&start) => Some(self.simd_gt(start)),
        Bound::Unbounded => None,
    };
    let end = match other.end_bound() {
        Bound::Included(&end) => Some(self.simd_le(end)),
        Bound::Excluded(&end) => Some(self.simd_lt(end)),
        Bound::Unbounded => None,
    };
    match (start, end) {
        (Some(start), Some(end)) => start & end,
        (None, Some(one)) | (Some(one), None) => one,
        (None, None) => Self::Mask::splat(true),
    }
}

Which would (presumably) optimise down to just self.simd_ge(start) & self.simd_le(end) for self.simd_range(start..=end), but I haven't tested to verify this claim. The self.simd_range(..) seems strange to support but RangeBounds doesn't explicitly deny it.

While it's not a priority by any means, I don't think a convenience like this is out of the question. There are some operations which we offer that are almost purely programmatic convention and have no hardware equivalent, and others which we offer not because the hardware can do better, but because the compiler can.

What about providing a SimdRangeBound trait, implemented over all of the ranges in std::ops?

I'm not sure why you'd need to create a separate trait for this.

The intention there is to keep it similar to the scalar API.  It's not clear to me that this function should be implemented on vectors themselves.

I see, you're saying that they should be inherent methods on the ranges, but since this crate can't do that, until it's merged into libstd, we should provide an extension trait instead.

Yeah, more or less.  Though RangeBounds is already a trait, I was suggesting implementing a trait over that, rather than on the ranges themselves.


(fosshub.extension) Download URL within "xdts"-attribute

The package does no longer work for me because the download url of the binaries is stored within the xdts-attribute of the link now instead of the data-attribute.
https://github.com/chocolatey/chocolatey-coreteampackages/blob/e17820954cdce01bf1979a4ecaa7b7baa0612706/extensions/chocolatey-fosshub.extension/extensions/Get-UrlFromFosshub.psm1#L33

This is what happens when I try to use it in the Irfanpackage:
C:\ChocolateyPackages_PUBLISHED_\irfanview>cinst irfanview.<IP_ADDRESS>61128.nupkg -force
Chocolatey v0.10.3 Professional
Installing the following packages:
irfanview.<IP_ADDRESS>61128.nupkg
By installing you accept licenses for the packages.
irfanview v<IP_ADDRESS>61101 (forced)
irfanview package files install completed. Performing other installation steps.
'kzvfk3wd.pwm' is of content type 'text/html; charset=utf-8'
This is try 1/3. Retrying after 300 milliseconds.
Error converted to warning:
Could not find a part of the path 'C:\Users\bcurran\AppData\Local\Temp\chocolatey\irfanview\<IP_ADDRESS>61101\kzvfk3wd.pwm.istext'.
This is try 2/3. Retrying after 400 milliseconds.
Error converted to warning:
Could not find a part of the path 'C:\Users\bcurran\AppData\Local\Temp\chocolatey\irfanview\<IP_ADDRESS>61101\kzvfk3wd.pwm.istext'.
Maximum tries of 3 reached. Throwing error.
Download of 'kzvfk3wd.pwm' (-1 B) completed.
'bvvv2fks.ico' is of content type 'text/html; charset=utf-8'
Download of 'bvvv2fks.ico' (-1 B) completed.
Attempt to use original download file name failed for 'IrfanView.html'.
Copying irfanview
from 'IrfanView.html'
Cannot find path 'C:\ChocolateyPackages_PUBLISHED_\irfanview\IrfanView.html' because it does not exist.
ERROR: Chocolatey expected a file to be downloaded to 'C:\Users\bcurran\AppData\Local\Temp\chocolatey\irfanview\<IP_ADDRESS>61101\irfanviewInstall.exe' but nothing exists at that location.
The install of irfanview was NOT successful.
Error while running 'C:\ProgramData\chocolatey\lib\irfanview\tools\chocolateyInstall.ps1'.
See log for details.
Chocolatey installed 0/1 packages. 1 packages failed.
See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).
Failures

irfanview (exited -1) - Error while running 'C:\ProgramData\chocolatey\lib\irfanview\tools\chocolateyInstall.ps1'.
See log for details.

Wtf.. Now its fdlw...

There is no such thing.
Script simply needs to be more intelligent. Take all the attributes of the link and find the one that contains full URL. You are being too precise and thats the problem.

I guess this is why a few of my installed packages aren't updating right now. :)
If I may make @majkinetor's suggestion more concrete:
- $regexPattern = "<a href=`"$hrefText`".*(?:data|xdts)=`"(.*)`""
+ $regexPattern = "<a href=`"$hrefText`".*=`"(https://download.fosshub.com/[^`"]+)`""

The existing regex looks like it would be too greedy if fosshub changed the order of attributes so the url wasn't last or if more quotes appeared later on the same line, hence changing the captured .* to explicitly exclude quote characters. Things will get more fun if fosshub decides to obfuscate the download url and use javascript to deobfuscate it.

I'm reopening this, as @majkinetor said the script needs to be more intelligent.
@welwood08 That's definitely an option to parse the download url, mind opening a PR for it?

Unfortunately 0.6.0 isn't working for me either. Am I doing anything wrong?
C:\WINDOWS\system32>choco install irfanview -version <IP_ADDRESS>61201
Parsing -version resulted in error (converted to warning):
Cannot bundle unregistered option '-e'.
Chocolatey v0.10.3 Professional
Installing the following packages:
irfanview
By installing you accept licenses for the packages.
IrfanView v<IP_ADDRESS>61201 - Possibly broken
irfanview package files install completed. Performing other installation steps.
'dhoiniqh.1s0' is of content type 'text/html; charset=utf-8'
This is try 1/3. Retrying after 300 milliseconds.
Error converted to warning:
Could not find a part of the path 'C:\Users\bcurran\AppData\Local\Temp\chocolatey\IrfanView\<IP_ADDRESS>61201\dhoiniqh.1s0.istext'.
This is try 2/3. Retrying after 400 milliseconds.
Error converted to warning:
Could not find a part of the path 'C:\Users\bcurran\AppData\Local\Temp\chocolatey\IrfanView\<IP_ADDRESS>61201\dhoiniqh.1s0.istext'.
Maximum tries of 3 reached. Throwing error.
Download of 'dhoiniqh.1s0' (-1 B) completed.
'ne5znfyd.2vd' is of content type 'text/html; charset=utf-8'
Download of 'ne5znfyd.2vd' (-1 B) completed.
Attempt to get headers for https://download.fosshub.com/Protected/expiretime=1480626267;badurl=aHR0cDovL3d3dy5mb3NzaHViLmNvbS9JcmZhblZpZXcuaHRtbA==/98c3cd7cd42077cdf61aded602f315d84cffa2bd5b14485b0672ebd736edcee0/IrfanView/iview442_x64_setup.exe failed.
The remote file either doesn't exist, is unauthorized, or is forbidden for url 'https://download.fosshub.com/Protected/expiretime=1480626267;badurl=aHR0cDovL3d3dy5mb3NzaHViLmNvbS9JcmZhblZpZXcuaHRtbA==/98c3cd7cd42077cdf61aded602f315d84cffa2bd5b14485b0672ebd736edcee0/IrfanView/iview442_x64_setup.exe'.
The remote server returned an error: (403) Forbidden.
Downloading irfanview 64 bit
from 'https://download.fosshub.com/Protected/expiretime=1480626267;badurl=aHR0cDovL3d3dy5mb3NzaHViLmNvbS9JcmZhblZpZXcuaHRtbA==/98c3cd7cd42077cdf61aded602f315d84cffa2bd5b14485b0672ebd736edcee0/IrfanView/iview442_x64_setup.exe'
ERROR: The remote file either doesn't exist, is unauthorized, or is forbidden for url 'https://download.fosshub.com/Protected/expiretime=1480626267;badurl=aHR0cDovL3d3dy5mb3NzaHViLmNvbS9JcmZhblZpZXcuaHRtbA==/98c3cd7cd42077cdf61aded602f315d84cffa2bd5b14485b0672ebd736edcee0/IrfanView/iview442_x64_setup.exe'.
The remote server returned an error: (403) Forbidden.
The install of irfanview was NOT successful.
Error while running 'C:\ProgramData\chocolatey\lib\IrfanView\tools\chocolateyInstall.ps1'.
See log for details.
Chocolatey installed 0/1 packages. 1 packages failed.
See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).
Failures

irfanview (exited 404) - Error while running 'C:\ProgramData\chocolatey\lib\IrfanView\tools\chocolateyInstall.ps1'.
See log for details.

I noticed this earlier and assumed it was a fosshub problem since my feeble attempt to download via browser also failed. My currently proposed solution won't automatically fix the irfanview package or any other consumer outside this repository since it seems fosshub has started requiring referrer on the actual download urls.

There must be a way to put a referrer into the helper. I know I've seen it somewhere in one before.

@bcurran3 there is, I've already tried that with the qbittorrent package, doesn't work.
Fosshub just keep redirecting, which clears the referrer header

also noticed that the expiretime they set in the url, expires approximally 4-5 seconds after html page have been requested.

Those bastards! :)
Any chance you might want to put your efforts towards a helper for FileHippo, MajorGeeks, and/or others?

it certainly seems like that would be a better use of our time, instead of trying to fix fosshub.

I've fixed the IrfanView package by just including the binaries now. No more problems for awhile. :)

@bcurran3 unfortunately that will be in violation with their license:
3) You may not distribute, rent, sub-license or otherwise make available to others the Software
or documentation or copies thereof, except as expressly permitted in this License without prior
written consent from IrfanView (Irfan Skiljan). In the case of an authorized transfer, the
transferee must agree to be bound by the terms and conditions of this License Agreement.

He actually has conflicting license statements. I do plan to e-mail him and clear it up.
From included LICENSE.txt:
IrfanView's webpage at http://www.irfanview.com/ defines the license as "This work is licensed under a Creative Commons Attribution-ShareAlike 4.0 International License."
From: http://creativecommons.org/licenses/by-sa/4.0/
LICENSE
You are free to:
Share — copy and redistribute the material in any medium or format
Adapt — remix, transform, and build upon the material
for any purpose, even commercially.
The licensor cannot revoke these freedoms as long as you follow the license terms.
That's for IrfanView.
Unfortunately there is no express statement that plug-ins are under the CC license.

                                    ^^^^^^ errr, explicit.

FYI: I could no longer find the reference to the CC license, only the EULA


Added indexOf function

Function that takes in an array and an element and returns the index of that element.
If the element doesn't exist returns -1.

[x] Running yarn lint does not trigger any linter warnings
[x] The test of the method you have fixed is passing
[x] You have written a new skeleton method for someone else to work on!

Sorry dude, this function has already been added in a PR that was submitted before yours. Have to close this now but thanks for contributing anyways!


ETFE-2530 Pre validate trader EIS integration

Is there going to be a separate bearer token for this API?


feat: select language screen

Also closes https://github.com/equinor/mad/issues/259

Could you change the "Submit" button to say "Save" instead?

@EmilieOlsen I can! But I don't know what "Save" is in Portuguese. Do you have any Portuguese friends who could help with that translation?

Google translate says "Salvar". But its Google translate, so i dont know how trustworthy this is 🙈

We trust Salvar is the correct translation!

In Salvar we trust


Getting react hooks error in the host application - Cannot read properties of null (reading useEffect)

Describe the bug
Getting the hooks error when tries to load a remote component running on the latest version of React, Next and nextjs-mf.
Scenario
Host running on older version:
module federation : 6.2.3,
next:12.2.0,
react: 17.0.2
Remote running on the latest version of all the above packages.
My remote application works in standalone mode, as well as some of the components are exposed via mf, and consumed by host.
The problem is, I am either able to run the host application or the remote application.
If I make the shared config of remote to {}, I am able to run the remote application but get the error in the host application.
If I make the shared config as  this
const shared = {
  react: {
    singleton: true,
    import: false,
    requiredVersion: false
  },
  'react-dom': {
    singleton: true,
    import: false,
    requiredVersion: false
  }
}

I get the react not found error in the remote application but my host works in this case.
P.S: I can't update the host packages to latest version as it loads many other modules, which starts breaking.
Reproduction
https://github.com/a-utkarsh/next-mf-poc.git
Used Package Manager
npm
System Info
System:
    OS: Linux 5.15 Ubuntu 22.04.4 LTS 22.04.4 LTS (Jammy Jellyfish)
    CPU: (16) x64 13th Gen Intel(R) Core(TM) i7-1360P
    Memory: 1.27 GB / 7.54 GB
    Container: Yes
    Shell: 5.1.16 - /bin/bash
  Binaries:
    Node: 18.17.0 - ~/.nvm/versions/node/v18.17.0/bin/node
    npm: 9.6.7 - ~/.nvm/versions/node/v18.17.0/bin/npm

Validations

[X] Read the docs.
[X] Read the common issues list.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Module federation issue and not a framework-specific issue.
[X] The provided reproduction is a minimal reproducible example of the bug.

Upgrade<EMAIL_ADDRESS>and set shareStrategy as loaded-first , the issue should be fixed
detail commit: https://github.com/2heal1/next-mf-poc/commit/5b36d480750f82a5fcc6a1d7eff7db13c3008132

Hi @2heal1 I tried the solution you mentioned in the commit, but whenever I am updating nextjs-mf to the latest version I get this error in the host application.
Error: Cannot find module '/home/autkarsh/projects/next-mf-poc/host/node_modules/next/dist/compiled/lib/util/makeSerializable'
I also tried to run your branch fix/2898, but I am getting this error when I am trying to do npm ci in the host.
npm ERR! code ENOTFOUND npm ERR! syscall getaddrinfo npm ERR! errno ENOTFOUND npm ERR! network request to https://bnpm.byted.org/styled-jsx/-/styled-jsx-5.0.7.tgz failed, reason: getaddrinfo ENOTFOUND bnpm.byted.org npm ERR! network This is a problem related to network connectivity. npm ERR! network In most cases you are behind a proxy or have bad network settings.

I fix lockfile and push again , @a-utkarsh  you can try again

Hi @2heal1 thanks, it worked.  But can you please tell me, is there anyway I can do the same with the older version of module federation in the host. I don't want to modify anything the host application. In my use case, both host and  remote application independently gets deployed. That's why I don't want any dependency on host.

Hmm i'm afraid it can not , because host use<EMAIL_ADDRESS>which is not using mf 2.0 , so many features can not work

@ScriptedAlchemy , do you have any ideas to solve this issue ? They just use mf 1.0 + next.js

You are missing getInitialProps or get server props. https://module-federation.io/practice/frameworks/next/index.html
You must have a data method on document and app at least, otherwise it will SSG and there will be no react to begin with.

Id also suggest upgrading from v6 to v8
Whatever the problem is, will not be retroactively fixed and alot has changed since v6 - but at a glance the problem is you have no data methods in the pages so theres no react on the server


Module path emitted twice for the compiler if module-path is present in compiler arguments

This happens if gradle's handling of modularity is turned off (modularity.inferModulePath.set(false)).
If lazy compiler argument providers are used with JavaCompile and these pass --module-path then the argument is doubled in the actual compiler argument execution. This is caused by DefaultJavaCompileSpec's getModulePath() trying to infer the value of module-path:
   @Override
    public List<File> getModulePath() {
        if (modulePath == null || modulePath.isEmpty()) {
            int i = compileOptions.getCompilerArgs().indexOf("--module-path");
            if (i >= 0) {
                // This is kept for backward compatibility - may be removed in the future
                String[] modules = compileOptions.getCompilerArgs().get(i + 1).split(File.pathSeparator);
                modulePath = Lists.newArrayListWithCapacity(modules.length);
                for (String module : modules) {
                    modulePath.add(new File(module));
                }
            } else if (modulePath == null) {
                modulePath = ImmutableList.of();
            }
        }
        return modulePath;
    }

I don't know what the reason for trying to infer module path from lazy arguments in compile spec is so I don't know the proper fix - it seems that either compilerArgs should be filtered from the parsed module-path arguments or the entire block should be removed.

At the time this is done, lazy and eager compiler arguments are already unified and joined into one list.
So it happens the same, whether the module path is configured eagerly or lazily, it doesn't matter.
And it also happens the same whether inferModulePath is enabled or disabled, as long as the inferred module path is empty, for example because you have no modules on the compile classpath, or because you set classpath = files().
If you have inferModulePath enabled and the inferred module path is non-empty, it is even more weird, because then you get two different --module-path arguments, the first from the inferred module path and the second from the custom module path and the custom should actually win as it comes later. So that the custom compiler args are investigated for the module path only if the inferred module path is empty is maybe also not correct.
Maybe @jjohannes or @asodja remember about something here.

This sounds like a regression. Or a bug that was overlooked all the time.
IIRC, the code cited in the description was meant to allow users (or plugins) to explicitly set the --module-path and opting out of the "inferModulePath" behavior.  However, even if it would work like that, it is not really complete, as there is no similar code to also control the -classpath manually.
However you put it, this seems to be incomplete and inconsistent as it is now.
As long as there are situations Gradle does not support, but which are needed to support all kinds of classpath/module-path combinations that Java itself allows (#25954, https://github.com/gradle/gradle/issues/12630#issuecomment-1125105629), I think this should be improved.
If we ignore the description (and discussion) and only consider the title, this is probably a duplicate of #12630


Ps dm branch 1

Initial pull request for CDA RDBMS capabilities.

There are some comments added to the requested changes. A few of the suggestions had issues and I was unable to get them to work so I reverted to original code. Comments were added for those in all cases.
The remaining items we have not commented on we do not have time to take on. Can the PR be accepted without those changes?
Also, we corrected some configuration validation issues around the Merged and Raw settings.

There are some comments added to the requested changes. A few of the suggestions had issues and I was unable to get them to work so I reverted to original code. Comments were added for those in all cases.
The remaining items we have not commented on we do not have time to take on. Can the PR be accepted without those changes?
Also, we corrected some configuration validation issues around the Merged and Raw settings.

I will defer to @gw-vsasidharan here

There are some comments added to the requested changes. A few of the suggestions had issues and I was unable to get them to work so I reverted to original code. Comments were added for those in all cases.
The remaining items we have not commented on we do not have time to take on. Can the PR be accepted without those changes?
Also, we corrected some configuration validation issues around the Merged and Raw settings.

I was hoping that the code in OutputWriter could be moved to a separate DBOutputWriter class, without significant effort. Because, if we do that, it's still ok for the code have some warts in it, since it will be all localized to just that class and not the trait itself. Are you of the opinion that it's actually more involved than it seems? I haven't done a through review of the extent of work involved, so happy to be corrected here.

@gw-vsasidharan  We're working through some issues with the schema evolution, so we'll be getting that wrapped up prior to anything else. We'll review the level of effort to move the OutputWriter code after completing the smaller changes and correcting the schema evolution issue.

@gw-vsasidharan We're working through some issues with the schema evolution, so we'll be getting that wrapped up prior to anything else. We'll review the level of effort to move the OutputWriter code after completing the smaller changes and correcting the schema evolution issue.
Thanks Scott. And just wanted to reiterate - if it wasn't for the desire to avoid undue changes to the generic trait, I wouldn't have imposed on you.


Sorry, that page does not exist, 404 on user_timeline

Summary
Can't get news feed from tomorrow
Reproduction Steps
When i'm trying call self.twitter_api.user_timeline i got this error. It's starting tomorrow after new twitter limit announced
Traceback (most recent call last):   File "twitter_news.py", line 121, in get_tweets     tweets = self.twitter_api.user_timeline(screen_name=source,   File "parser/venv/lib/python3.10/site-packages/tweepy/api.py", line 33, in wrapper     return method(*args, **kwargs)   File "parser/venv/lib/python3.10/site-packages/tweepy/api.py", line 46, in wrapper     return method(*args, **kwargs)   File "parser/venv/lib/python3.10/site-packages/tweepy/api.py", line 415, in user_timeline     return self.request(   File "parser/venv/lib/python3.10/site-packages/tweepy/api.py", line 274, in request     raise NotFound(resp) tweepy.errors.NotFound: 404 Not Found 34 - Sorry, that page does not exist
Minimal Reproducible Example
No response
Expected Results
Get tweets from my feed
Actual Results
404 on this url https://api.twitter.com/1.1/statuses/user_timeline.json
Twitter API Access Plan
free
Tweepy Version
4.14
Checklist

[X] I have searched for duplicate issues.
[X] If applicable, I have shown the entire traceback.
[X] If applicable, I have removed any visible credentials from my code and/or screenshots.

Additional Context
No response

Twitter API v1.1 isn't available with Free access.
https://twittercommunity.com/t/announcing-new-access-tiers-for-the-twitter-api/188728
https://developer.twitter.com/en/docs/twitter-api/getting-started/about-twitter-api
https://developer.twitter.com/en
For code block usage, see https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/creating-and-highlighting-code-blocks.


Lip sync support

Add lip sync support. This will likely need to be done as a GDNative library for performance reasons. Rust will probably be the implementation language.
uLipSync (MIT) can be used as a reference even though it's a Unity library.
godot-audio-processing Can be used as a reference for feeding data from Godot and maybe interpreting the lip synced data?
NOTE: There's no license for godot-audio-processing, but I did ask on Discord if I could use the repo as a reference

personal fork of uLipSync
personal fork of godot-audio-processing

@you-win
Would you be interested Text to Speech LipSync?

@GeorgeS2019 looks like #90 will be handling your request, as your request is more about TTS than lip syncing. This issue is tracking only lip sync support.
On that note, #88 implements basic lip sync support. I will need to add phoneme (viseme?) detection in a later PR.


Fixes #216.

Thanks @roll
Think this is what you've asked for? Moved property up to main api.

Thanks. I'll check what wrong with tests tomorrow


Docker image for 3DCityDB v5

This PR adds an edge image for the 3DCityDB v5. The image is built from the latest commit to v5-devel branch and published to Dockerhub and Github packages:

3dcitydb/3dcitydb-pg-v5:edge
ghcr.io/3dcitydb/3dcitydb-pg-v5:edge

docker run --name 3dciytdb -p 5432:5432 -d \
    -e POSTGRES_PASSWORD=<theSecretPassword> \
    -e SRID=<EPSG code> \
    [-e HEIGHT_EPSG=<EPSG code>] \
    [-e GMLSRSNAME=<mySrsName>] \
    [-e POSTGRES_DB=<database name>] \
    [-e POSTGRES_USER=<username>] \
    [-e POSTGIS_SFCGAL=<true|false|yes|no>] \
  3dcitydb/3dcitydb-pg-v5:edge

Please rename the ENV variable GMLSRSNAME to SRS_NAME. We try to be independent from GML as much as possible.

Will we have a separate release track for v5?

Well, no, this is not planned currently.

I renamed the variable, but only in the Docker init script 3dcitydb-initdb.sh. I left CREATE_DB.sql, CREATE_DB.shand CREATE_DB.bat untouched, where GMLSRSNAME still exists.


Add missing arrow in documentation

If I'm not mistaken arrows is missing in those functions.

Whoops! Thanks.

You don't need an arrow here

@stevenmiller888 Only if I didn't also have a : there.
render ({ children }) {
  return <button class="my-button">{children}</button>
}

vs
render: ({ children }) => {
  return <button class="my-button">{children}</button>
}

Whoops you're right

Pretty much why I don't use those fancy functions in object literals. It's too confusing with the 10 different ways of declaring functions.


TAJO-1875: Resource leak after a query failure

The main reason is that wrong resource release routine.
When TaskManager receives a task start event e, it checks that e is the first task start event of the current eb. If so, it create a context for the current eb. Here, if an error occurs while creating a eb context, the resource assigned for e must be released.
However, in the current implementation, the resource release routine requires a task attempt id which will not exist with an eb start failure. As a result, resource manager loses the resource assigned for e.

+1 Great!!
I verified 'mvn clean install’ and the test failure is not related to this patch.


fix: GitLab labels

ensure the branch name is populated even when we are not running a merge request job (for branch job for example)
see https://docs.gitlab.com/ee/ci/variables/predefined_variables.html

CI_COMMIT_BRANCH is "Available in branch pipelines, including pipelines for the default branch. Not available in merge request pipelines or tag pipelines."
and CI_MERGE_REQUEST_SOURCE_BRANCH_NAME is "The source branch name of the merge request."

Thank you, @vbehar!


Update CSV recipe to use pyarrow.csv instead of pandas

Addresses https://github.com/apache/arrow-cookbook/pull/49

@westonpace  @jorisvandenbossche I merged your suggestions, I think this should now be ready to go


Fixed *.t file to test passes.

Now,PSGI is not enabled installed via panda
https://gist.github.com/hiroraba/ff863ee1f2da79ba9c59
so fixed it test passes.
https://travis-ci.org/hiroraba/perl6-psgi/builds/94352322

sorry! same pull request is opened.


Multisocket architechure not recognized by boot2Docker for windows workstation

Hi,
Docker Version
Client:
Version:      1.12.0
API version:  1.24
Go version:   go1.6.3
Git commit:   8eab29e
Built:        Thu Jul 28 23:54:00 2016
OS/Arch:      windows/amd64
Server:
Version:      1.12.0
API version:  1.24
Go version:   go1.6.3
Git commit:   8eab29e
Built:        Thu Jul 28 23:54:00 2016
OS/Arch:      linux/amd64
I am running docker on windows workstation and I have found that boot2Docker fails to recognize the number of sockets on the Machine, cores per socket and threads per core. Here is the output of the lscpu and cpuinfo.
log_lscpu.txt
log_cpuinfo.txt
and here is the image of the actual hardware configuration

That looks correct on the cores seen by Linux (it looks like windows calls them "Logical processors").  So Windows sees 2 threads for each of 12 cores, while Linux sees 1 thread for each of 24 cores.
I would assume that it does not see the 2 sockets because that is not exposed through the VM interface, but I don't have any machine I could even test that theory.  Are the two sockets seen by a different Linux VM, maybe Debian, Ubuntu, or CentOS?  Perhaps we are missing a kernel config for it to recognize multiple CPU sockets.

Are the two sockets seen by a different Linux VM, maybe Debian, Ubuntu, or CentOS?
I can test this theory and see if a linux VM can recognize the 2 sockets. Though, It would be great to know if there is kernel_config entry for this.

I've looked through the kernel config flags that mention "CPU" and can't find anything dealing with multiple physical cpus.  I also looked at my VirtualBox installation on Windows and couldn't find a setting for giving the guest different physical cpus, but I only have one physical cpu.

Hi,
I checked it on a Ubuntu16.04 VM, and it did not recognize the multiple sockets. Here is the output,

All I am concerned is, if it cannot recognize multiple sockets, will it affect the performance, if I extend the use of processors for my simulation from more than 6? As there will be 6 cores per socket, with 2 threads per core.


Modularise discovery

LGTM, other than the redundant dependencies.

Which redundant dependencies?

None apparently :) Merging.

Is there a reason this PR was done against a separate branch and not master?
Give the high risk of conflicts (because of the file rename) this should go into master ASAP.

Originally I forked the master branch t owork on modularising things, and by coincidence Anton and I started working at modularising librespot at the same time. I was going to modularise the audio_backend and spirc as well before merging, but given the potential changes in https://github.com/librespot-org/librespot/pull/125, it's probably sensible to merge the progress thus far sooner rather than later. Will let the CI run, then merge this to master.


Add Ting Kai information to AboutUs

Add About Us page information for @Ferusel.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@9a3fba4). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #98   +/-   ##
=========================================
  Coverage          ?   72.15%
  Complexity        ?      399
=========================================
  Files             ?       70
  Lines             ?     1232
  Branches          ?      125
=========================================
  Hits              ?      889
  Misses            ?      311
  Partials          ?       32

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Update developer guide: Target user profile, value proposition and user stories

Update developer guide: Target user profile, value proposition and user stories according to #18

LGTM


BUG: footer alignment needs to be done

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Hello @Its-Aman-Yadav please assign this issue to me under gssoc 24


Mrtk version next internal defintions

Overview
Initial check-in of new definitions for MRTK-vNext

Target Sprint 1: #1903

See https://github.com/SimonDarksideJ/MixedRealityToolkit-Unity/pull/1 with completed changes.

So these data structures will be serialized then?


Is uint8 supported?

Describe the bug
I expect the dataset to store the data in the uint8 data type, but it's returning int64 instead.
While I've found that datasets doesn't yet support float16 (https://github.com/huggingface/datasets/issues/4981), I'm wondering if this is the case for other data types as well.
Is there a way to store vector data as uint8 and then upload it to the hub?
Steps to reproduce the bug
from datasets import Features, Dataset, Sequence, Value
import numpy as np

dataset = Dataset.from_dict(
    {"vector": [np.array([0, 1, 2], dtype=np.uint8)]}, features=Features({"vector": Sequence(Value("uint8"))})
).with_format("numpy")

print(dataset[0]["vector"].dtype)

Expected behavior
Expected: uint8
Actual: int64
Environment info

datasets version: 2.12.0
Platform: macOS-12.1-x86_64-i386-64bit
Python version: 3.8.12
Huggingface_hub version: 0.12.1
PyArrow version: 11.0.0
Pandas version: 1.5.3

Hi ! The numpy formatting detaults to int64 and float32 - but you can use uint8 using
ds = ds.with_format("numpy", dtype=np.uint8)

If the feature type is Value("uint8") then it's written an uint8 on disk using the uint8 Arrow dtype.
e.g.
ds = Dataset.from_dict({"a": range(10)}, features=Features({"a": Value("uint8")}))
ds.data.nbytes
# 10

Oh, I understand now.
The data was stored in uint8 from the beginning (when the dataset returns int64).
Thank you for your time!
My question is fully resolved.


Deliberately cause a test to fail.

CI Experiment. Please do not merge!
An experimental PR to check that CI is working correctly.
I changed one function in order to trigger the following test failure:
Failures:

  test/Codec/Binary/Bech32Spec.hs:387:9:
  1) Codec.Binary.Bech32, Roundtrip (dataPartFromText . dataPartToText), Can perform roundtrip conversion
       Falsifiable (after 1 test and 2 shrinks):
         DataPart "4"
         Nothing /= Just (DataPart "4")

  To rerun use: --match "/Codec.Binary.Bech32/Roundtrip (dataPartFromText . dataPartToText)/Can perform roundtrip conversion/"

  test/Codec/Binary/Bech32Spec.hs:429:9:
  2) Codec.Binary.Bech32, Constructors produce valid values, dataPartFromText
       Falsifiable (after 3 tests and 2 shrinks):
         [DataChar {getDataChar = 'q'}]
         input:  [DataChar {getDataChar = 'q'}]
         output: Nothing

         Nothing /= Just True

  To rerun use: --match "/Codec.Binary.Bech32/Constructors produce valid values/dataPartFromText/"

The Travis CI integration should also report a failure.

Closing. Build failed, as expected: https://travis-ci.org/input-output-hk/bech32/builds/650333453?utm_source=github_status&utm_medium=notification

Pretty sure I tested something like this when I first put this package in its own repo :).


Improve implicit properties collection speed, Date mutability

When using date transforms, mutation made by methods (setTime, etc.) are reflected in the backed property (string / timestamp), so it is no longer required to treat dates as immutable objects.
Performance improvements for implicit property transform collections.

Coverage decreased (-0.8%) to 88.806% when pulling 0593e9633ea381cd29d1f9544c8cc5fcde77e22d on improve-collection-speed into f3badb28524a4a2128c8164543b9f0a191d60c9f on master.


feat(nightwatch): Add support for Nightwatch v1.x

Add support for Nightwatch v1.x

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

Codecov Report

Merging #137 into master will decrease coverage by 0.81%.
The diff coverage is n/a.

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

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: ruben0626:x: m0t01b6m0t01b6 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Terminology files not matching specification

The specification states that event instead of Event as listed in EHRbase.
See: https://github.com/openEHR/terminology/blob/master/openEHR_RM/en/openehr_terminology.xml#L38
and https://github.com/ehrbase/ehrbase/blob/develop/base/src/main/resources/terminology.xml#L1220
This leads to invalid outputs from ehrBase (AQL and return=representation). However, I changed all occurrences in 19b3796099f9e80ea5a6560bfa61f3157d354b78 but however even with a fresh database it will be always created with an uppercase E. (Update: I had to run mvn compile to avoid using an already built V3__terminology.jar file.)
Also: This is not the only inconsistency I saw there. In EHRbase 435 is laboratory while the specification says episodic. Those things might need to be changed as well.
In combination with #113 this caused my problems described in #109.

@nr23730 can you clarify what "ehrBase 435 is"?

Sorry, if it was not clear enough. I meant the terminology IDs in specification vs those in EHRbase..
Compare https://github.com/openEHR/terminology/blob/master/openEHR_RM/en/openehr_terminology.xml#L37
with https://github.com/ehrbase/ehrbase/blob/develop/base/src/main/resources/terminology.xml#L1222

@nr23730 the issue is the entry for "episodic" for the composition.category is currently under discussion, I'm not sure why it appeared in the openEHR terminology repository, already asked the question there.

Basically 435:: episodic is there because I jumped the gun on publishing it in the terminology spec.  Can reallocate episodic to another number


Not getting redirected to Solid Community Login as expected in Chrome

I've been building a demo app using @solid/react in preparation for a talk I'm giving on Solid to NationJS, a big JavaScript conference in Washington, DC. The app is functionally similar to some of the profile viewers out there, but takes close design inspiration from a certain well known social network.
My MVP is functionally complete and up at https://5bfb1fc2b13fb10671d78e6e--competent-babbage-885c43.netlify.com/. Unfortunately, I can't seem to login using my Community pod using Chrome. Clicking on "Solid Community" in popup.html triggers a request that Chrome seems to block. All other OIDC providers work in Chrome as expected, and Solid Community works in Firefox. It appears that Chrome is blocking the request for safety reasons.

Any ideas?
-Sean

Could not reproduce on Chrome 70.0.3538.102 on macOS.
What version are you using?

Just tried to disable privacy badger - and now it works even in normal browser mode. Fun thing is that it continued to work after I enabled Privacy Badger again. Duh?

This might be related to https://github.com/solid/node-solid-server/issues/921

"It was not possible to login - the request was blocked by your browser.

Probably an't do that, because net::ERR_BLOCKED_BY_CLIENT does not bubble up to JavaScript. The request fails silently from our perspective; same with cross-origin issues.
We can set a timeout, but we do not know the reason.

Note to self: ask people (in issue template?) to list their browser extensions

Yeah, interesting it is not just me... I suppose we should reach out to the Privacy Badger team.

Ah and that is fixed on latest version.

If popup.html embed images and javascript etc. from external services then I guess Privacy Badger does it job as it is supposed to and how I want it to. In that case popup.html is nothing better than embedded Facebook trackers - you allow a central actor to be notified of each and every Solid login attempt from all over the web by all Solid users - even if they choose to login with their own private Solid/webId provider.
Glad you fixed it - it really is a relevant privacy issue.

Yeah, had to trade UX for privacy.

And now we are bashing privacy issues, then I believe your recommended usage is just as bad. The code example starts with this line:
<script src="https://solid.github.io/solid-auth-client/dist/solid-auth-client.bundle.js"></script>

And that is needed on ALL pages that uses Solid since it contains the session tracking code.
This means github.io now tracks all Solid websites that uses your recommended usage.
Please promote a downloadable local version to be installed on the individual websites AND favor that over github.io.

Yeah, there we have to trade DevX for privacy too 👍

Unfortunately, the screenshot above is already the latest version that does not include any external resources, so Privacy Badger must be triggered by something else.

I was looking into this, since I still have this issue on NSS 5.0.0 beta.2. In the current version, there are no external dependencies, so we have concluded that the cause for this is unknown, right?
I was thinking that we might file an issue with Privacy Badger, but simply saying "Privacy Badger doesn't work with Solid" isn't a terribly good report. Are we able narrow down what the issue might be?

I would suggest to have a public demo somewhere, and already get the discussion going over at Privacy Badger. They might be able to say what we should look for (for instance, switch on certain debug flags).

Right!
I'll create a test account on dev.inrupt.net and include a password in the report.

Arrggh! So, it works for me on https://dev.inrupt.net/ ... @JornWildt , @bushidocodes , can I ask you to create an account there and see if you are able to reproduce?

I could(!) reproduce it somewhat consistently on for instance https://solidrc.azurewebsites.net/home/login# which is running an older version of "login.html". But it is not easy - you have to use the old login.html from at least four different websites before it triggers Privacy Badger.
So I have https://elfisk.solid.community/, https://solidrc.azurewebsites.net/home/login#, https://localhost:5001/home/login, https://solidrc.jorn:5001/home/login, https://solidrc.other.dk:5001/home/login#  where the last three all refers to my test project on localhost (using the "hosts" file on Windows to direct trafic).
Then, at some point, I see solid.community blocked by PB and unblock it, whereafter stuff works again. Unfortuantely, when I block solid.community again, things still work. Then I add a new name for locahost - and then that one is blocked and unable to login. So PB is a bit tricky to figure out.
Anyway - the blocked request is always https://solid.community/.well-known/openid-configuration which makes sense, because why should solidrc.azurewebsites.net reach with "fetch" to solid.community? Except that it happens when I click on a button (the "Login with Solid Community" button), so I initiate the request myself - but apparently PB is unable to detect that.

If it "works for you" then try running the new server version on localhost and add a couple of entries for <IP_ADDRESS> in the "hosts" file, such that you can use the same local server on different hostnames. Then, after logging in via solid.community from a couple of hostnames (4 times works for me), PB should start blocking it - it last it does for me on the old server.

I just tried a few logins on dev.inrupt.net (for instance https://aviana.dev.inrupt.net/common/popup.html) and PB blocks https://solid.community/.well-known/openid-configuration when I click "Login" and "Solid community". So the new server has the same issue with PB.

OK, I managed to reproduce it now, I think. At least I saw the problem I saw in https://github.com/solid/node-solid-server/issues/883 . I saw it when I logged into my own site https://solid.kjernsmo.net/ with the test identity https://testpb.dev.inrupt.net/profile/card#me identity. Not a particularly spectacular fail, but a fail nonetheless ;-)

OK; great! I think I have enough to submit to PB. :-)

Oh, they have some stuff in https://github.com/EFForg/privacybadger/wiki/Find-out-why-Privacy-Badger-is-blocking-a-domain if you want to help out submitting it, @JornWildt . If you do, I can follow up with posting an account and password to help them look into it.

Tried it and got:
**** ACTION_MAP for solid.community
elfisk.solid.community {
"dnt": false,
"heuristicAction": "block",
"nextUpdateTime":<PHONE_NUMBER>370,
"userAction": "user_allow"
}
solid.community {
"dnt": false,
"heuristicAction": "block",
"nextUpdateTime":<PHONE_NUMBER>811,
"userAction": ""
}
**** SNITCH_MAP for solid.community
solid.community [
"localhost",
"elfisk.dk",
"solidrc.azurewebsites.net"
]
Don't know how much it helps ...

OK, yeah, don't know how much it helps, but I'll submit to see if they can figure out something.


Add commands package.

Things this will depend on:

command-specific help
command-specific permissions (maybe?)
command-specific parsers
command-specific schemas
good up-front design


TE-9781: Fixed false-positive reports in ClientInterfaceRule and ApiI…

Developer(s): @asaulenko

Ticket: https://spryker.atlassian.net/browse/TE-9781

Release Group: https://release.spryker.com/release-groups/view/3795

merge: merge

Release Table

Module
Release Type
Constraint Updates

ArchitectureSniffer
patch (currently 0.x)

Module ArchitectureSniffer
Change log
Fixes

Adjusted ClientInterfaceRule so it is not applied to dependency interfaces.
Adjusted ApiInterfaceRule so it correctly inspects method docblocks with example code snippets.
Fixed ModuleConstantsFormingConstantValuesRule

Improvements

PHP 8 compatibility
Added BridgeClassNamingRule and BridgeInterfaceNamingConventionRule

You had the wrong header for Fixes
Also, I added the things on master, so we can now make one clean release together with the other PHP8 releases.


GA handling is different

This article fails to mention that GA accounts do not honor these settings

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

ID: 18640c6c-b52c-84e3-59d8-90f2a663cc6b
Version Independent ID: 77b249dc-4440-0334-31d7-b05d30f52657
Content: Azure Active Directory self-service password deep dive
Content Source: articles/active-directory/authentication/concept-sspr-howitworks.md
Service: active-directory
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

I cover it in the policies article in depth. https://docs.microsoft.com/en-us/azure/active-directory/authentication/concept-sspr-policy#administrator-reset-policy-differences
Do you think it would be helpful to cover it in this article as well maybe in a shortened note format? If so where do you think it would make most sense?

How does that look?


Theme aware plugin

Summary of the Pull Request
This PR adds feature to make plugin responsive to theme change.

PR Checklist

[x] Applies to #4122
[x] CLA signed. If not, go over here and sign the CLA
[x] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments

Updated theme manager to add a ThemeChanged event which fires when the system theme is changed.
Updated PublicAPIInstance to allow plugins to access the current system theme and subscribe to theme change events.
Update UWP applications to change icons on theme change.

Validation Steps Performed
Manually validated that icons change on changing themes.

Note: Please look at file src/modules/launcher/Plugins/Microsoft.Plugin.Calculator/Main.cs to get an idea of handling theme change in plugins.

@niels9001, this would be needed then for #4149

@somil55 app.xaml has a conflict

@crutkas @somil55 - Have merged the code from this PR into https://github.com/microsoft/PowerToys/pull/4149 so that should one now be good to go (and I guess we can close this one?).

Great :). If this gets merged in I'll update PR #4149 to update Calculator, Shell and WindowWalker.
@somil55 The default 'app_error' and 'app' icons in ...\launcher\modules\Wox.Infrastructure\Wox.cs (line 41 and 42) need to be theme aware as well - as they show up whenever an icon cannot be loaded. I don't dare to touch those since it's core functionality. Would you mind having a look at that as well for this PR?

Great :). If this gets merged in I'll update PR #4149 to update Calculator, Shell and WindowWalker.
@somil55 The default 'app_error' and 'app' icons in ...\launcher\modules\Wox.Infrastructure\Wox.cs (line 41 and 42) need to be theme aware as well - as they show up whenever an icon cannot be loaded. I don't dare to touch those since it's core functionality. Would you mind having a look at that as well for this PR?

@niels9001 I have updated the code to update error/default icon on theme changes. All error/default icon references across the project are also using ImageLoader now.

@alekhyareddy28 @somil55 That's interesting. With the move to the new wave of Fluent icons (with colors) we actually don't need to make them themeing depended, right? They will look fine either way.
With some apps still using the monochrome MDL2 style that might be an issue?

@alekhyareddy28 @somil55 That's interesting. With the move to the new wave of Fluent icons (with colors) we actually don't need to make them themeing depended, right? They will look fine either way.
With some apps still using the monochrome MDL2 style that might be an issue?

Yes @niels9001, you're right. There are a few packaged applications which don't show up with fluent icons; calculator, windows security and snip & sketch are a few examples for which we would need this.

Calculator seems to be changing but not this app. Can you pls look into it?

This seems to be happening because the logic for fetching UWP icons returns the first valid path. I will update it to first look for theme-based icons.


Fix user plugins case suite

Description
When submitting a case-suite that uses userPlugins, registration of the userPlugin with the plugin manager is attempted multiple times. The plugin manager does not allow this, and the case-suite fails.
To fix this, the registration of userPlugins now checks if the user plugin name has already been registered with the plugin manager.

Checklist

[x] This PR has only one purpose or idea.
[ ] Tests have been added/updated to verify that the new/changed code works.

[x] The code style follows good practices.
[x] The commit message(s) follow good practices.

[ ] The release notes (location doc/release/0.X.rst) are up-to-date with any bug fixes or new features.
[ ] The documentation is still up-to-date in the doc folder.
[ ] The dependencies are still up-to-date in setup.py.

Somehow I have never run into this issue... I wonder what is different between your case suite and mine.


Add linalg.eigh, linalg.eigvalsh to ChainerX

This is work-in-progress pull request!
This PR wraps cuSOLVER's Eigen decomposition.
With reference to #6423 and #6764

No hurry if you want to focus on the other PRs first but I left some initial comments.

Jenkins, test this please.

Jenkins CI test (for commit ddf042bde18b48338f6284d40c7f1f1596fe61bf, target branch master) failed with status FAILURE.

Jenkins, test this please.

Jenkins CI test (for commit 5dd57863084973f22dfe2c5d0a10e2e28d28b00f, target branch master) failed with status FAILURE.

Jenkins, test this please.

Jenkins CI test (for commit 5dd57863084973f22dfe2c5d0a10e2e28d28b00f, target branch master) succeeded!

Sorry for another round of review comments but could you fix the above?

These two routines suffer from the same issue as the one reported here https://github.com/chainer/chainer/pull/7329#issuecomment-522829589, but fails not only for cuda but both native and cuda. NumPy returns empty arrays so we should fix these ones too.

Inputs with shape (0, 0) are now working. The problem was that the leading dimension argument can't be less than one when calling LAPACK/cuSOLVER.

Jenkins, test this please.

Jenkins CI test (for commit 0b9e32dfc72d83362eafb7b9eb350cc2d6fc1d7c, target branch master) succeeded!

Jenkins, test this please.

Jenkins, test this please.

Jenkins CI test (for commit 0b9e32dfc72d83362eafb7b9eb350cc2d6fc1d7c, target branch master) failed with status FAILURE.

Jenkins, test this please.

Jenkins CI test (for commit 0b9e32dfc72d83362eafb7b9eb350cc2d6fc1d7c, target branch master) failed with status FAILURE.

Jenkins, test this please.

Jenkins CI test (for commit 1d349891e9645a70bf9f175ad2fe6d1789dd9524, target branch master) failed with status FAILURE.

Jenkins, test this please.

Jenkins CI test (for commit 1d349891e9645a70bf9f175ad2fe6d1789dd9524, target branch master) failed with status FAILURE.

We're currently trying to address flaky tests in Chainer (tests that fail quite often, but randomly). I tried to run this tests around 1000 times locally and noticed it is too. Could you possibly take a look at it?
> python -m pytest -rfx -x tests/chainerx_tests/unit_tests/routines_tests/test_linalg.py -k 'Eig or eig'
=================================================== test session starts ====================================================
platform linux -- Python 3.6.1, pytest-4.1.1, py-1.7.0, pluggy-0.7.1
rootdir: /home/vincent/repos/chainer, inifile: setup.cfg
plugins: xdist-1.26.1, repeat-0.7.0, metadata-1.8.0, html-1.20.0, forked-1.0.2
collected 5452 items / 5312 deselected

tests/chainerx_tests/unit_tests/routines_tests/test_linalg.py ..............................F

========================================================= FAILURES =========================================================
________________________ test_Eigh_param_15_{UPLO='L', in_dtypes='float64', shape=(6, 6)}[native:0] ________________________

...

>           raise AssertionError(f.getvalue())
E           chainer.testing.function_link.FunctionTestError: backward is not implemented correctly
E
E           (caused by)
E           AssertionError: check_backward failed (eps=0.001 atol=1e-05 rtol=0.005)
E           inputs[0]:
E           array([[0.63003572, 0.73949666, 0.30431390, 0.78106292, 0.84906733, 0.13343415],
E                  [0.77523393, 0.72043896, 0.35694555, 0.20228585, 0.1443275 , 0.46263248],
E                  [0.44895464, 0.98560085, 0.45669856, 0.50982009, 0.67137867, 0.71678652],
E                  [0.80079176, 0.49129112, 0.44117445, 0.37909192, 0.24211678, 0.05098325],
E                  [0.85323461, 0.4698262 , 0.24974733, 0.30615825, 0.44239657, 0.95101930],
E                  [0.67536627, 0.92138063, 0.80352809, 0.06498898, 0.07995298, 0.44205577]], shape=(6, 6), dtype=float64, device='native:0')
E           grad_outputs[0]:
E           array([0.69785838, -0.50916007, -0.38296303, -0.21680939, -0.50352705, -0.52525478], shape=(6,), dtype=float64, device='native:0')
E           grad_outputs[1]:
E           array([[-0.42395427, 0.94644767, -0.10463601, 0.45856564, -0.93298551, -0.77470524],
E                  [-0.73019518, 0.37148137, 0.95580271, -0.8257174 , 0.20094679, 0.71898474],
E                  [0.18891407, -0.9121845 , -0.01234777, -0.46408905, 0.96048164, 0.88721807],
E                  [-0.05649843, 0.45397163, 0.71113162, 0.54623588, -0.63286487, -0.15408585],
E                  [-0.95843457, -0.95512276, 0.57650616, 0.54538773, -0.16373386, 0.37241348],
E                  [-0.34651667, 0.99726244, -0.75112784, -0.75493156, -0.3493308 , 0.02286855]], shape=(6, 6), dtype=float64, device='native:0')
E           directions[0]:
E           array([[-0.00618424, 0.08321978, -0.36595368, 0.33599586, -0.10806886, 0.24544865],
E                  [0.11989653, -0.03817258, -0.18779766, 0.13898499, -0.10236622, -0.23798269],
E                  [0.12837853, 0.04892894, -0.27415405, -0.02997159, 0.05766378, -0.05279465],
E                  [0.24968716, -0.07334048, 0.04902351, -0.04082709, 0.25993498, 0.21250433],
E                  [0.10434886, 0.18213748, -0.04959657, 0.08947205, 0.22029317, 0.18337490],
E                  [0.11972817, 0.18409781, -0.20667259, 0.03948545, -0.05358864, 0.13837495]], shape=(6, 6), dtype=float64, device='native:0')
E           gradients (numeric):  array(0.05349447, shape=(), dtype=float64, device='native:0')
E           gradients (backward): array(0.04892034, shape=(), dtype=float64, device='native:0')
E
E           x: numeric gradient, y: backward gradient
E           Not equal to tolerance rtol=0.005, atol=1e-05
E
E           Mismatch: 100%
E           Max absolute difference: 0.00457413
E           Max relative difference: 0.09350161
E            x: array(0.053494)
E            y: array(0.04892)
E
E           assert_allclose failed:
E             shape: () ()
E             dtype: float64 float64
E             i: (0,)
E             x[i]: 0.05349447366484204
E             y[i]: 0.048920342974239195
E             relative error[i]: 0.09350160715372591
E             absolute error[i]: 0.004574130690602848
E             relative tolerance * |y[i]|: 0.000244601714871196
E             absolute tolerance: 1e-05
E             total tolerance: 0.000254601714871196
E           x: 0.05349447
E           y: 0.04892034

chainer/gradient_check.py:537: FunctionTestError

Thanks for the fix but it's still a bit flaky in my environment (6 failures in 5000 runs). How about relaxing the tolerances?

Sorry but the way in which kernels are registered has been changed in https://github.com/chainer/chainer/pull/7970. Could you make similar changes to this PR?

Modified the input construction using https://github.com/chainer/chainer/pull/8077 and it seems to address the flakiness. Could you adopt it?

This should be stable for the backward tests now. LGTM!

Jenkins, test this please.

Jenkins CI test (for commit 421cce554655ea01e089240667fad27a3bf1390f, target branch master) failed with status FAILURE.

Jenkins, test this please.

Jenkins CI test (for commit 421cce554655ea01e089240667fad27a3bf1390f, target branch master) failed with status FAILURE.

Not related
=========================== short test summary info ============================
FAIL ../../../repo/tests/chainerx_tests/unit_tests/routines_tests/test_creation.py::test_Meshgrid_param_5_{input_arrs=6}_param_1_{indexing='ij'}[float16-cuda:0]
===== 1 failed, 126209 passed, 5672 skipped, 41 xfailed in 2009.64 seconds =====

Jenkins, test this please.

Jenkins CI test (for commit 421cce554655ea01e089240667fad27a3bf1390f, target branch master) succeeded!


Please help to get git diff into prompt :)

Hey thanks for this great plugin, I would try to have something like this as a prompt for my commit messages, but have no idea how to get my local git diff into this prompt. Any suggestions? :)

Hi @dvogt23 ,
Thanks! You can also modify the prompt templates programmatically and I would do it like that:

Create a lua function to get the output of git diff
Put that into an appropriate prompt
Put that prompt into a temporary require('gen').prompts['someTmp']
Invoke it with :Gen someTmp

Would that work for you?

Seems strange, did try it like this:
function get_git_diff()
  -- Define the command to get the git diff
  local command = "git diff"

  -- Use io.popen to run the command and capture its output
  local handle = io.popen(command)
  local result = handle:read("*a") -- Read all output
  handle:close()

  return result
end

require("gen").prompts["Commit summary"] = {
  prompt = "You are an expert developer specialist in creating commits. Provide a super concise one sentence overall changes summary of the user "
    .. get_git_diff()
    .. " output following strictly the next rules: - Do not use any code snippets, imports, file routes or bullets points. - Do not mention the route of file that has been change. - Simply describe the MAIN GOAL of the changes. - Output directly the summary in plain text.",
  replace = true,
}

but getting always like this as output:
The user output shows the main goal of the changes as "Added new features to improve the performance and user experience of the application."
I think I have to chain the output of the first prompt, to this:
You are an expert developer specialist in creating commits messages.
	Your only goal is to retrieve a single commit message.
	Based on the provided user changes, combine them in ONE SINGLE commit message retrieving the global idea, following strictly the next rules:
	- Always use the next format: \`{type}: {commit_message}\` where \`{type}\` is one of \`feat\`, \`fix\`, \`docs\`, \`style\`, \`refactor\`, \`test\`, \`chore\`, \`revert\`.
	- Output directly only one commit message in plain text.
	- Be as concise as possible. 50 characters max.
	- Do not add any issues numeration nor explain your output.

Is this possible somehow? 😆

Just wanted to share my solution to this problem if you are still struggling with this. It's missing error handling but should work most of the time.
{
  'David-Kunz/gen.nvim/issues/101',
  opts = { display_mode = 'split' },
  config = function(_, opts)
    local gen = require('gen')

    gen.setup(opts)

    gen.prompts['Commit'] = {
      prompt = function()
        local diff = vim.system({ 'git', 'diff', '--staged' }, { text = true }):wait()

        return [[
          ...prompt...

          Git diff: ]] .. diff.stdout
      end,
      replace = true,
      extract = '```$filetype\n(.-)```',
    }
  end
}

Great, thanks @Rasmus-Bertell now Im able to get the summary. I have to chain the output of summary promt to another one for getting a commit message. Do you have an idea?

No, sorry. This covers my needs so I'm not motivated to explore it further.

Hi @dvogt23 ,
I'm afraid follow-up prompts are not yet possible. Only manually with :Gen Chat.
I'll chink about supporting prompt chains.

For now, would it be possible to put everything in one prompt?

The problem is that many models have a hard time generating proper commit messages from just the diff, or at least that's what I have noticed, especially if the diff is long. So I think the solution is to first generate a short summary of changes with one prompt and pipe it to another one that makes the actual commit message.


web3torrent: fix PEER_TRUST

At the moment, the leecher always pays for wire.requests.length blocks, instead of PEER_TRUST blocks. See proof below.
There is an assumption that the last piece is the first to be downloaded with the torrent.downloaded === 0  condition.

This PR fixes both of those.
Proof of point 1.
// Current logic
let amountToPay = BUFFER_REFILL_RATE.sub(
      WEI_PER_BYTE.mul(BLOCK_LENGTH).mul(PEER_TRUST - wire.requests.length)
    );

// Substitute BUFFER_REFILL_RATE definition
const amountToPay2 = utils
      .bigNumberify(WEI_PER_BYTE.mul(BLOCK_LENGTH * PEER_TRUST))
      .sub(WEI_PER_BYTE.mul(BLOCK_LENGTH).mul(PEER_TRUST - wire.requests.length));

// What is the above in bytes (instead of wei)
const bytesToPayFor = utils
      .bigNumberify(BLOCK_LENGTH * PEER_TRUST)
      .sub(BLOCK_LENGTH * (PEER_TRUST - wire.requests.length));

// What is the above in blocks, instead of bytes
const blocksToPayFor = utils.bigNumberify(PEER_TRUST).sub(PEER_TRUST - wire.requests.length);

// Big numbers are confusing! And not needed here :)
const blocksToPayFor2 = PEER_TRUST - (PEER_TRUST - wire.requests.length);

// Algebra is fun without a bunch of wrappers!
const blocksToPayFor3 = wire.requests.length;

I would add in the DONE listener ( before closing the channels).
Something that logs the final channels status, and the amount paid to each wire, before closing it: I did this in local, for example:
log.info({torrent, channels: this.paymentChannelClient.channelCache}, '<< Torrent DONE!');
      torrent.wires.map(wire => {
        const {peerAccount, peerChannelId} = wire.paidStreamingExtension;
        const channelOfWire = this.paymentChannelClient.channelCache[peerChannelId];
        if (channelOfWire) {
          const balance = bigNumberify(channelOfWire.beneficiaryBalance).toString();
          log.info(`<< ${peerAccount} Final Balance: ${balance} Downloaded ${wire.downloaded}`);
        }
      });

This would give us a final status check:

But I tested it locally, and it worked okay everytime!. Merge away!

I would add in the DONE listener

@spersico good idea. Let's do that in a separate PR.


Faile when I try to download any container

Expected behavior
I expect to install the contanier without problem
Actual behavior
This error is showed in the last part:
Failed to register layer: re-exec error: exit status 1: output: ProcessBaseLayer C:\ProgramData\Docker\windowsfilter\681be7b8764fb1152e444a01c7b20c896fc3dffd4b6e2aea6fc3008a8953f921: The parameter is incorrect.
Information about the Issue
Steps to reproduce the behavior

Downloading a container doesn't have any parameter - Sound more like running a container issue.
Can you provide the steps to reproduce this as, previously requested in the issue template?


[Xcode 11] swift package generate-xcodeproj with RealmSwift dependency does not build.

Bug: An Xcode 11 project which is generated from an executable-type Swift Package which has RealmSwift as a dependency does not build.
Goals
The goal is to have an Xcode 11 project which is generated from an executable-type Swift Package which has RealmSwift as a dependency that can successfully build and run.
Expected Results
What Swift Package that builds from the command line is also expected to build in an Xcode 11 project generated from the same package.
Actual Results
Build fails as show below in the steps to reproduced.
Steps to Reproduce

Setup an executable type Swift Package with RealmSwift as a dependency.

Example Package.swift
GitHub repository marc-medley/RealmSPMXcode is an example Swift Package with RealmSwift as a dependency.

Verify build from the command line on has only a few warnings.

swift build

# RealmSPMXcode/.build/checkouts/realm-cocoa/Realm/RLMRealm.mm:135:17:
# warning: method definition for 'privilegesForRealm' not found [-Wincomplete-implementation]
# @implementation RLMRealm {

# RealmSPMXcode/.build/checkouts/realm-cocoa/include/Realm/RLMRealm.h:658:1:
# note: method 'privilegesForRealm' declared here
# - (struct RLMRealmPrivileges)privilegesForRealm;

## ... snip ...

# warnings generated.
# [139/139] Linking RealmSPMXcode

Generate an Xcode 11 project from the Swift Package.

swift package generate-xcodeproj
# generated: ./RealmSPMXcode.xcodeproj
open RealmSPMXcode.xcodeproj/

Build in Xcode with "My Mac" as the target.

UNITO-UNDERSCORE!Errors!UNITO-UNDERSCORE!01!
UNITO-UNDERSCORE!Errors!UNITO-UNDERSCORE!02!
UNITO-UNDERSCORE!Errors!UNITO-UNDERSCORE!03!
UNITO-UNDERSCORE!Errors!UNITO-UNDERSCORE!04!
Workaround: Edit RealmCore 5.23.5/realm/util/terminate.cpp to remove the REALM_VER_CHUNK expansion related error:
REALM_NORETURN void terminate(const char* message, const char* file, long line) noexcept
{
    std::stringstream ss;
    // ss << file << ":" << line << ": " REALM_VER_CHUNK " " << message << '\n'; // :BEFORE:
    ss << file << ":" << line << ": [realm-core-5.23.5] " << message << '\n';    // :AFTER:
    terminate_internal(ss);
}

After the about workaround more errors occur:
UNITO-UNDERSCORE!Errors!UNITO-UNDERSCORE!05!
Notice that .build/checkouts/realm-cocoa/Realm/ObjectStore/src/impl/results_notifier.hpp
and .build/checkouts/realm-cocoa/Realm/ObjectStore/src/impl/results_notifier.cpp are in the same directly.
Next attempted workaround ...
// #include "impl/results_notifier.hpp"
#include "results_notifier.hpp"

... leads to yet more errors:
UNITO-UNDERSCORE!Errors!UNITO-UNDERSCORE!06!
Code Sample

Example Package.swift
GitHub repository marc-medley/RealmSPMXcode is an example Swift Package with RealmSwift as a dependency.

Version of Realm and Tooling
Realm framework version: REALM_COCOA_VERSION 3.21.0
Realm Object Server version: REALM_VERSION 5.23.5
Xcode version: 11.2
iOS/OSX version: macOS Mojave 10.14.6 (18G1012)
Dependency manager + version: swift-tools-version 5.1

swift build and swift generate-xcodeproj do significantly different things and AFAICT it isn't possible to support both of them at the same time.

I have several projects which build and run with both in the Swift Package Manager CLI swift build environment and as an Xcode project created by swift generate-xcodeproj. Both can work.
It's acutally a more rigorous process to test with both because sometimes one environment identifies weaknesses which the other environment does not. I've been using this multi-environment approach for almost 2 years on multi-environment Linux/iOS/macOS C/Swift headless libraries.
The version auto-generated for the Xcode environment by SPM has certain GUI debugging conveniences. However, a package library which builds and runs warning-free across all three environment (swift build macOS, swift generate-xcodeproj and swift build Linux) is pretty solid.
So, at least in my experience, this multi-enviroment approach can work; and, it can also help provide higher quality code.

Well, if you can figure out a way to deal with the problems I'd certainly be interested in them, but I couldn't. The REALM_VER_CHUNK error is because .define("REALM_VERSION_STRING", to: "\"\(coreVersionStr)\"") results in -DREALM_VERSION_STRING="5.23.6" being passed to the compiler with swift build, but -DREALM_VERSION_STRING=5.23.6 with the generated xcode project (note how it has stripped the quotes). The header inclusion failures are because the generated xcode project ignores .headerSearchPath("Realm/ObjectStore/src").

@tgoyne Thank you for the clarifying information. It's apparent that there are some upstream Package Manager issues to be addressed for a swift package generate-xcodeproj with RealmSwift dependency to build successfully.
The following issues have now been reported as SwiftPM bugs:

SR-11789: generate-xcodeproj does not preserve quotes in .define(...)
SR-11790: generate-xcodeproj fails to provide .headerSearchPath(…) includes

Hi, any update on this please.

Closing this as we don't need to generate an xcodeproj to be able to open a Package project as the warning mentions when running swift package generate-xcodeproj
Xcode can open and build Swift Packages directly. 'generate-xcodeproj' is no longer needed and will be deprecated soon.
If this is needed for other uses case, @AdieOlami @marc-medley please reopen the issue.


Added android:appCategory.

This pr is basically making an app able to be detected as a game on android.

Is there anything wrong with the pr?

The implementation looks fine, but it seems isGame is deprecated.

Is ok?

Will this be pushed?

What do you think of #1735 as a solution?

What do you think of #1735 as a solution?

It's better then this implementation

You can close this if you want

Might leave this open until we get more feedback on my version.


meta update

Fixes #3738

@rachelappel,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot


Slope Painting

Description
The ability to paint materials based on slope normals, this is a feature in a lot of other terrain tools and would be similar to the auto-shader implementation. This would allow much easier organic looking hand painted terrain instead of needing to rely on an external program.

Duplicate #282


Added vectors of expressions

Probably conflicts with #38 heavily, i.e. needs manual merge

Probably conflicts with #38 heavily, i.e. needs manual merge


Event Request - OnAfterPostWhseActivityLine

We are backporting some code for our app. But we're missing an event in codeunit 7324 "Whse.-Activity-Post" for BC15 and BC14
From BC16 onwards we have the following, can we get the same event (same location) in older versions of BC

If you need events for for BC15 and BC14 you should open Support Request to CSS through PartnerSource portal or contact your Service Account Manager (SAM) in the local subsidiary.


Fix/jwt vc to credential

Fix date format timezone
Fix json value for jwt vc (required for json schema verification)

@waltkb please review this one. thx


Failed to start app

Sorry if this is a noobish question. When I try to start app I get the following error:
C:\Users\William\Desktop\newstand\node_modules\express-jwt\lib\index.js:20
  if (!options || !options.secret) throw new Error('secret should be set');
                                   ^
Error: secret should be set
    at module.exports (C:\Users\William\Desktop\newstand\node_modules\express-jwt\lib\index.js:20:42)
    at Object.<anonymous> (C:\Users\William\Desktop\newstand\dist\server\auth\auth.service.js:9:19)
    at Module._compile (module.js:624:30)
    at Object.Module._extensions..js (module.js:635:10)
    at Module.load (module.js:545:32)
    at tryModuleLoad (module.js:508:12)
    at Function.Module._load (module.js:500:3)
    at Module.require (module.js:568:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (C:\Users\William\Desktop\newstand\dist\server\api\user\user.router.js:14:12)
    at Module._compile (module.js:624:30)
    at Object.Module._extensions..js (module.js:635:10)
    at Module.load (module.js:545:32)
    at tryModuleLoad (module.js:508:12)
    at Function.Module._load (module.js:500:3)
    at Module.require (module.js:568:17)
[nodemon] app crashed - waiting for file changes before starting...

Where do I need to set the secret to make it work? Thanks for sharing your code with the community.

Hey @billbarni - certainly understandable why you came across this error. I just committed some changes that include a '.env-sample' file, which contains environment sensitive variables you will need (including the secret).. To ensure a working app on your end - pull the latest code, rename '.env-sample' to '.env' and enter the necessary API variables in the .env file.
Note: While Google and FB oAuth credentials aren't required (you can use local authentication), you will need API credentials from getstream.io.
thanks for checking this project out! I will close this issue, but feel free reach out with anything else.

Now Im getting this error when trying to create a new local account:
[0] 2017-10-26T16:30:25.364-0200 I COMMAND  [ftdc] serverStatus was very slow: { after basic: 0, after asserts: 433, aft
er backgroundFlushing: 433, after connections: 433, after dur: 433, after extra_info: 433, after globalLock: 433, after
locks: 433, after network: 433, after opLatencies: 433, after opcounters: 433, after opcountersRepl: 433, after repl: 55
3, after security: 553, after storageEngine: 553, after tcmalloc: 553, after wiredTiger: 1073, at end: 1683 }
[1] [HPM] Error occurred while trying to proxy request /api/users/create from localhost:4200 to http://localhost:3000 (E
CONNREFUSED) (https://nodejs.org/api/errors.html#errors_common_system_errors)
[1] [HPM] Error occurred while trying to proxy request /api/users/create from localhost:4200 to http://localhost:3000 (E
CONNREFUSED) (https://nodejs.org/api/errors.html#errors_common_system_errors)

Ok great. I was just about to respond to say that I'm unable to reproduce the error on a fresh install. Also, this project is obviously in its infancy with plenty to do so feel free to create a PR if you want to contribute.


feat: track hashing

This PR extends the tree insertion metrics added in #3855 with stats about the hashing operations.

Benchmark results
No metrics with a significant change found.

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit e28a6bf1 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
128 txs

l1_rollup_calldata_size_in_bytes
45,444
179,588
716,132

l1_rollup_calldata_gas
221,004
866,300
3,447,500

l1_rollup_execution_gas
934,148
3,968,774
24,791,036

l2_block_processing_time_in_ms
1,499 (-1%)
5,135 (+1%)
21,161 (-1%)

note_successful_decrypting_time_in_ms
305 (-2%)
931 (+1%)
3,372 (-1%)

note_trial_decrypting_time_in_ms
22.8 (-1%)
57.7 (+17%)
133 (-1%)

l2_block_building_time_in_ms
14,922
59,494
237,818

l2_block_rollup_simulation_time_in_ms
10,534
42,118
168,680

l2_block_public_tx_process_time_in_ms
4,355
17,296 (+1%)
68,805

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
15,149 (+1%)
29,130 (+2%)

note_history_successful_decrypting_time_in_ms
2,425 (+11%)
4,361 (+1%)

note_history_trial_decrypting_time_in_ms
145 (+25%)
184 (+8%)

node_database_size_in_bytes
3,620,753
3,729,775

pxe_database_size_in_bytes
29,923
59,478

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
213
44,681
25,561

private-kernel-ordering
165
43,753
14,809

base-rollup
1,067
128,338
881

root-rollup
79.1 (-2%)
4,088
677

private-kernel-inner
292 (+1%)
71,208
25,561

public-kernel-private-input
206
31,863
25,561

public-kernel-non-first-iteration
203
31,905
25,561

merge-rollup
7.66 (-8%)
2,608
881

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
2 leaves
8 leaves
16 leaves
32 leaves
128 leaves
64 leaves
512 leaves
1024 leaves
2048 leaves
8192 leaves

batch_insert_into_append_only_tree_16_depth_ms
11.3
19.6
13.8
18.1
23.8
65.4
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
N/A
N/A
N/A
84.1
50.5
242
473
915
3,579

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
N/A
N/A
N/A
118
60.0
395
738
1,414
5,591

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
64.9
N/A
N/A
N/A
N/A
N/A
N/A
N/A

hash_append_only_tree_16_depth_count
16.9
17.5
23.0
31.6
47.0
143
N/A
N/A
N/A
N/A
N/A

hash_append_only_tree_32_depth_count
N/A
N/A
N/A
N/A
N/A
159
96.0
543
1,055
2,079
8,223

hash_duration_append_only_tree_16_depth_ms
0.00
0.00
0.00
0.00
0.00
0.00
N/A
N/A
N/A
N/A
N/A

hash_duration_append_only_tree_32_depth_ms
N/A
N/A
N/A
N/A
N/A
0.00
0.00
0.00
0.00
0.00
0.00

hash_indexed_tree_20_depth_count
N/A
N/A
N/A
N/A
N/A
202
106
707
1,395
2,771
11,027

hash_indexed_tree_40_depth_count
N/A
N/A
N/A
95.1
N/A
N/A
N/A
N/A
N/A
N/A
N/A

hash_duration_indexed_tree_20_depth_ms
N/A
N/A
N/A
N/A
N/A
0.00
0.00
0.00
0.00
0.00
0.00

hash_duration_indexed_tree_40_depth_ms
N/A
N/A
N/A
0.00
N/A
N/A
N/A
N/A
N/A
N/A
N/A

Miscellaneous
Transaction sizes based on how many contracts are deployed in the tx.

Metric
0 deployed contracts
1 deployed contracts

tx_size_in_bytes
15,455
32,224

Transaction processing duration by data writes.

Metric
0 new commitments
1 new commitments

tx_pxe_processing_time_ms
374 (-1%)
1,085

Metric
1 public data writes

tx_sequencer_processing_time_ms
535


temporary fix for calico-cloud search page issue

The issue stems from  docusaurus.config.js line   399  onlyIncludeVersions: [/'current',/'3.16'/,'3.15'/],
Only 3.16 is included for calico cloud, changing this to current does not add any of the revelant results to search so minor adjustments were made in the search behavior as to acomodate and address this issue while the proper way of fixing this is assessed.

Product Version(s):

Issue:

Link to docs preview:

SME review:

[ ] An SME has approved this change.

DOCS review:

[ ] A member of the docs team has approved this change.

Additional information:

Merge checklist:

[ ] Deploy preview inspected wherever changes were made
[ ] Build completed successfully
[ ] Test have passed

@HugoJLFerreira This appears to be hard coded to the 3.16 version. Am I right in thinking we'll need to update this manually when we promote a new version?


merge from template-node-app

package.json

chore(deps): add ^8.3.20 remove ^8.3.18 (devDependencies.npm-pkgbuild)

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

npm package (@latest dist-tag)
v5.0.105
GitHub release

Your semantic-release bot :package::rocket:


Please add workaround for forced YouTube translated video titles

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2016.09.24. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2016.09.24

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[x] Feature request (request for a new functionality)
[ ] Question
[ ] Other

YouTube has added a feature where it allows users or channel owners to contribute translations of video titles. This feature has been around for some time but rolled out in increments hitting users in different regions.
On YouTube's web interface whether the original title (desired) or one of the translated titles are shown depends on the choice of interface language. If the language interface is selected to be other than English then YouTube will show the non-English titles in their original form.
When using youtube-dl the desired outcome is for %(title)s to be the original title since this allows for better accounting of the saved material. There is no guarantee user-contributed or machine translations are accurate. These titles also make searching among files saved by youtube-dl harder than it should be.
The problem has now hit youtube-dl because it requests en-US (or en-GB for embedded SWF) content from YouTube via hardcoded 'hl=en' parameter in multiple places in extractor/youtube.py. This results in YouTube presenting youtube-dl with (often inaccurate or nonsensical) translated titles.
Please make the location, 'gl=??', and language 'hl=??', configurable via commandline switches. The parameter hl=?? is in particular effective for working around this issue. Currently, changing the occurrences of hl=en to, for example, hl=fr causes YouTuble, and consequently youtube-dl, to present the correct original title of videos.
Example
Video code: 2sKgVz79NZE
Original correct title (in Japanese, accessible via YouTube API or a browser where YouTube language interface has been set to something other than English): "人を怒らせる方法 「放棄」"
Translated title (acquired with --get-filename -f22 -o '%(title)s.%(ext)s' to youtube-dl, or a browser with YouTube interface language set to English): "How to make peopke angry 'renounse'.mp4"

For interested, there are some more details on IRC.

Seems like a doable issue. I'm  not advanced enough to solve it yet, but wish I could.

This seems to be the default behaviour with video titles for me at the moment. Descriptions are still localised, though, and I imagine implementing the specific feature rcajal described would solve this.

I think this is the relevant issue - is there anything we can do to force a title language? I'm trying to download some videos from nicovideo.jp (niconico) and the titles are downloaded in English when available. I'd prefer to have the titles downloaded in Japanese. On the website, the way to force Japanese is to swap the website language to it (from English).

@yan12125 Hi,
I propose you use this translate-shell tool :
To translate to English (default) :
$ youtube-dl --ignore-config --get-filename XMaxBthVLx4 | trans -b
273 The Word of Grace is full of power - XMaxBthVLx4.mp4

To translate to French (for example) :
$ youtube-dl --ignore-config --get-filename XMaxBthVLx4 | trans -b :fr
273 La Parole de Grâce est pleine de puissance - XMaxBthVLx4.mp4

You could call it by a sub-process through a new --translate or --trans option.

@sebma; The important part isn't being able to read the title or not, it's being able to avoid the translations Youtube provides.

its not duplicate. its problem with wrong interpretation description by youtubue-dl

it's the same, youtube-dl is using the description from the English interface, while what you're getting in the browser is the description from the Hindi interface.

FYR: --add-header 'Accept-Language: zh-CN' works fine for me when downloading a playlist from YouTube.

Thanks, that's very useful information.

--add-header 'Accept-Language: zh-CN'  does not work

--add-header 'Accept-Language: zh-CN' does not work

Are you downloading a playlist?

--add-header 'Accept-Language: zh-CN' does not work

Are you downloading a playlist?
Yes. It still returns english title.

"D:\GreenSoft\youtube\youtube-dl.exe" --playlist-items 81 "https://www.youtube.com/channel/UCBJmYv3Vf_tKcQr5_qmayXg/videos" --get-filename  --add-header "Accept-Language: zh-CN"
State Banquet Master Chef - Stir-fry kale borecole with pork and peanuts.-Tt959-
hPKh0.mp4


http://nixos.org/channels -> https://nixos.org/channels.

http://nixos.org/channels/ first issues a redirect to https:// before forwarding
to actual channel URL on Cloudfront. As we don't follow the redirect chain this
broke extraction of the channel's commit. Also see #26.
Fixes #25.

Thanks!


Panic on minimize event with error STATUS_ACCESS_VIOLATION or STACK_BUFFER_OVERRUN depending on example

Hello, I'm getting weird panics when window is getting minimized on Windows.
hello_world.rs panics with: STATUS_ACCESS_VIOLATIONand every other example except raytracing (not tested) panics with: STACK_BUFFER_OVERRUN.
I found it really hard to debug, second error code is definitely weird because it doesn't leak memory or anything, general usage even becomes 2x smaller for few seconds before panic 😕.

I don't have an explanation or answer to that problem, but I too experience this, and I've seen this same error occur on every ash-based render app in Rust I've tried.  kajiya is one of them.  Perhaps there's some edge case that needs handled with winit and the Windows API side of things that nobody has took the time and effort to do?

I spammed the swapchain.rs with a bunch of trace!() logs and narrowed this down to incorrect usage of the destroy function.
Thanks for the bug @HeavyRain266 and the helpful analysis @seabassjh! I don't use Windows often and missed this.
There may be more clean-up needed; and thought put into #36 before it merges but I may not have the time to do that over the next week or so. I will attempt to merge this as soon as I can.

Thanks for investigating this, I'm also not using windows often too, Wayland extension used by winit  to share window surface with the Compositor doesn't  specify such event a minimize so I couldn't encounter that crash anytime soon but potential users of my game or engine could on either windows or macOS through moltenvk.
Here is said extension spec generated from XML: https://wayland.app/protocols/xdg-shell


Uninstall: remove CephFilesystemSubVolumeGroup on external mode uninstall

Fix offboarding in the consumer cluster by removing the relevant created external resource
Signed-off-by: kesavan<EMAIL_ADDRESS>

/approve
On Ohad's behalf, there is a comment pending to be taken care but that can be taken in a follow-up PR

/cherry-pick release-4.10

/retest


Supported languages are undiscoverable

What version of OpenRewrite are you using?
I'm not, I just learned about it at a conference presentation and wanted to learn more about capabilities
How are you running OpenRewrite?
I'm not, I just learned about it at a conference presentation and wanted to learn more about capabilities
What is the smallest, simplest way to reproduce the problem?
Find someone who has never heard of this software. Point them at the documentation (optionally on a mobile device). Ask them to answer the question "what languages are supported by OpenRewrite?" in 30 seconds.
What did you expect to see?
Some sort of easily discoverable enumeration of supported languages.
What did you see instead?
Nothing obvious (to me). Not even an FAQ.
What is the full stack trace of any errors you
N/A
Are you interested in contributing a fix?
Happy to review a PR

Hi @andrewpollock ! Thanks for your suggestion to document the supported languages in the OpenRewrite docs as well. It's true that these aren't explicitly listed for now, although there's hints as to what's supported in the recipe catalog. Support is evolving fast, but perhaps not so fast the docs can't keep up. :)
We did document our current support and future plans in terms of language support over on https://docs.moderne.io/references/supported-languages
& https://docs.moderne.io/references/language-roadmap
Perhaps we can feature some of the same information in these docs as well.
Hope the above links already help; I'll ask @mike-solomon if he can work in a similar section in the docs here.
Out of curiosity: what conference presentation did you watch that brought you here? :)

Out of curiosity: what conference presentation did you watch that brought you here? :)

I'm live filing from "Scaling the Security Researcher to Eliminate OSS Security Vulnerabilities Once and For All - Jonathan Leitschuh, Open Source Security Foundation" at OSS EU

Thanks for the suggestion! I've gone ahead and added this. You should see it available in a few minutes once the deploy finished.

👋 This talk still has legs it seems! 😂🙂