keep get  [WDS] disconnected!

OS?
Windows 10
Versions.
@angular/cli: 1.0.0-beta.32.3 [1.0.0-beta.24]
node: 6.9.1
os: win32 x64
@angular/common: 2.4.8
@angular/compiler: 2.4.8
@angular/core: 2.4.8
@angular/forms: 2.4.8
@angular/http: 2.4.8
@angular/platform-browser: 2.4.8
@angular/platform-browser-dynamic: 2.4.8
@angular/router: 3.4.8
@angular/cli: 1.0.0-beta.32.3
@angular/compiler-cli: 2.4.8
Repro steps.

use ng new PROJECT create a new project without any modification, keep get [WDS] disconnected! error after ng serve.

The error only appears under Chrome, both version 55 and 56, it works fine in Mircrosoft Edge and other browsers. I've searched the internet, most solution is about webpack, but I don't know how to config this with angular-cli. I wonder whether this is a bug related to Chrome or not, anyway, even a newly created project suffers.
Thank you for help!

@luzly have you tried to reset Chrome settings to defaults?

I have the same problem (in Chrome 56), but it only happens when I use ng serve --ssl

I had the exact same issue. when using SSL. What was happening is that the WebSocket would get closed/dropped with a handshake error. (I could see the error in Chromium, but not Chrome).  In my case the certificate that the "webpack-dev-server" is using is self-signed and so I had the option "chrome://flags/#allow-insecure-localhost" enabled to ignore the "page not secured" message. Installing the certificate in Chrome as Trusted Root fixed the "[WSD] Disconnected!" issue.
I got the certificate from "\node_modules\webpack-dev-server\ssl" and extracted it manually from the file then imported in Chrome and disabled the "allow-insecure-localhost". Importing the certificate via MMC didn't work for me either. This is on Windows 7.

Closing as answered, thanks @violinbg !

I still receive the [WDS] Disconnected! error message after recompiling.
I added the webpack-dev-server ssl certificate as well as my self-signed certificate to the Trusted Root Certificates in Google Chrome.
Any idea?

Same here

@veke With some of the recent updates of Chrome, the browser now rejects certificates that don't have SAN (Subject Alternative Name). If the cert doesn't have the "Subject Alternative Name" - you need to figure out how to create a new self-signed cert with SAN - I googled for a while and it wasn't as easy as I hoped.
So double check the certificate you are using has that, then install it to Chrome via Settings -> Show Advanced -> Manage Certs -> Then add it to "Trusted Root...";
You can also play with the "chrome://flags/#allow-insecure-localhost" option in Chrome or disable SSL in Angular.
I hope this helps somewhat.  As I pointed earlier - I had the issue because I was using HTTPS . If you are not or you still have the issue - I would suggest to download Chromium - you'll see more error messages in the Console Log - so it may help to figure out exactly what's happening.

@violinbg Thanks for the reply. Yes I'm using self signed certificate and need to run app in https, so I need to fix this somehow. I will investigate this more with your instructions. Thanks

Im still having this problem on win/chrome. Any of those tips mentioned earlier did not work for me.

Updating to chrome 59 seems to work on Mac.

I am still seeing this issue in Chrome 59. It seems to go away when disabling SSL on ng serve.

@hill0826 Hmm...verify you run latest angular?

Relates to https://github.com/webpack/webpack-dev-server/issues/941 and potentially https://github.com/webpack/webpack-dev-server/issues/851

@veke good question. yes, here's the output of ng version for me:
@angular/cli: 1.1.2
node: 8.1.2
os: win32 x64
@angular/common: 4.2.3
@angular/compiler: 4.2.3
@angular/core: 4.2.3
@angular/forms: 4.2.3
@angular/http: 4.2.3
@angular/platform-browser: 4.2.3
@angular/platform-browser-dynamic: 4.2.3
@angular/router: 4.2.3
@angular/cli: 1.1.2
@angular/compiler-cli: 4.2.3
@angular/language-service: 4.2.3

I'm on Windows 10 and Chrome 59. We don't have a strong requirement for SSL in development environments right now so we're going to disable it for the time-being, but it would be nice to have.
This issue seems to relate to https://github.com/webpack/webpack-dev-server/issues/941 and potentially https://github.com/webpack/webpack-dev-server/issues/851

@hill0826 Strange, I have angular 4.2.2 and cli 1.1.2 on mac and works on me. And on windows 7 chrome 59 works for me too. (don't have access to see exact module versions, but are quite similar)

I have the latest CLI version and Chrome 59 installed and I still have the issue but not as often as with previous version of Chrome and CLI.
I added both my self-signed certificate for the website and the certificate that comes with webpack-dev-server in the Thrusted Root Certificates.

Do you guys have a green bar in Chrome when you have the redirect issue? If you don't it's most certainly a SSL issue. You can see the actual error in the console log if you try with Chromium. It would probably be something like "SSL handshake failed".
Last time I had the issue was with the latest Chrome version refusing to accept self-signed certificates with no SAN. The certificate that comes with the "webpack-dev-server" doesn't have SAN. So I created a new cert with SAN and it's working fine at this time.

@violinbg Do you have some easy guide to create self signed cert with SAN?

@veke:
I don't know about easy, but here are some very strong hints:
https://bugs.chromium.org/p/chromium/issues/detail?id=704199#c6
I also collected some links here:
https://github.com/webpack/webpack-dev-server/issues/941#issuecomment-310639952

@veke
you can also use the same package that WDS uses to do it manually:
$ node
> const selfsigned = require("selfsigned")
undefined
> const attrs = [{ name: "commonName", value: "localhost" }, { name: "subjectAltName", value: "localhost" }]
undefined
> const pems = selfsigned.generate(attrs, {
...     algorithm: "sha256",
...     days: 30,
...     keySize: 2048
... });
undefined
> const fs = require("fs");
undefined
> fs.writeFileSync("/tmp/server.pem", pems.private + pems.cert, { encoding: "utf-8" });

This seems to have fixed it for me; included a PR

@cwalv Thanks. I will test this.

@cwalv
That attributes containing subjectAltName is not the Chrome is complaining. It is SAN extension field and should be included in the second options object. You can see the options from selfsigned manual.
extensions: [{ name: 'basicConstraints', cA: true }], // certificate extensions array

@veke,
So did you manage to create one with node.js? Just wonder if it's easier because all the instructions for OpenSSL look a bit complicated.
I made my a cert with the PowerShell command New-SelfSignedCertificate, but it's only available in Windows 10 (and maybe 8).

@violinbg @cwalv
const selfsigned = require("selfsigned");
const attrs = [{ name: "commonName", value: "localhost" }];
const pems = selfsigned.generate(attrs, {
    algorithm: "sha256",
    extensions: [{
        name: 'subjectAltName',
        altNames: [
            {
                type: 2, // DNS,
                value: "localhost"
            },
            {
                type: 6, // URI
                value: "localhost"
            }, {
                type: 7, // IP
                ip: "<IP_ADDRESS>"
        }]
    }],
    keySize: 2048
});

const fs = require("fs");

fs.writeFileSync("./ssl/server.pem", pems.private + pems.cert, { encoding: "utf-8" });

@veke Thanks! I'll keep that script handy :)

Edited extension parameters. Removed URI and IP. Still got green certificate.

i am using nginx but the cert generated by selfsigned(node) not working? some idea?

Did you added in your os too?

@veke yes :(

if you have latest browsers and cli installed, I have no idea. This is strange that some work for other and other not. Try to add those IP and URI fields back to script.
{
   type: 6, // URI
   value: 'http://example.org/webid#me'
},
{
    type: 7, // IP
    ip: '<IP_ADDRESS>'
  }

What is the error in your certificate?

@veke this is the error:
"There are issues with the site's certificate chain (net::ERR_CERT_AUTHORITY_INVALID)."

@veke the code for generate cert:
const fs = require("fs");
const selfsigned = require("selfsigned");
const attrs = [{ name: "commonName", value: "mydomain.m" }];
const pems = selfsigned.generate(attrs, {
    algorithm: "sha256",
    keySize: 2048,
    extensions: [{
        name: "subjectAltName",
        altNames: [{
            type: 2, // DNS
            value: "localhost"
        },
	{
   	    type: 6, // URI
            value: 'https://my.mydomain'
        },
        {
            type: 7, // IP
            ip: '<IP_ADDRESS>'
        }
       	]
    }]
});

fs.writeFileSync("./server.crt", pems.cert, { encoding: "utf-8" });
fs.writeFileSync("./server.key", pems.private, { encoding: "utf-8" });

I deal this stuff only with local development. Im not even sure you can use these in real domains :D
I've seen commonName values *domain.com sometimes, maybe they need to match to those subjectAltname fields somehow... I have no idea, sorry.

@fernandops26  Certificates are read by the browser. The browser goes to a certification authority to validate the certificate. The certification authority says if the certificate is correct. With a self signed certificate there is no certification authority - so the browser can't trust it.
For a local dev environment, you can see its your certificate and so can choose to trust it - but its still not a correct certificate as it's self signed and doesn't have a certification authority associated with it. So the browser raises this ERR_CERT_AUTHORITY_INVALID.
The trick (which I still don't know how) is to get the browser to trust the certificate for your dev environment. You shouldn't ever trust a self signed certificate from  real server and system admins shouldn't have a self signed certificate on a server to make it work.
This is my "basic" understanding of certificates at the moment

I used @veke code and it worked:

generate your own cert using this code.
install that cert in User Store > Trusted Root Certification (windows 10)
Tell webpack to use that cert by "--https --cert ./ssl/server.crt --key ./ssl/server.key
Profit
Take printscreens and write a readme.txt in ./ssl for next year.

With this reference you can also solve the problem:
https://alexanderzeitler.com/articles/Fixing-Chrome-missing_subjectAltName-selfsigned-cert-openssl/

[SOLVED] run the angular app using npm start instead of  ng serve

thanks @abergs for your solution. That worked for me.

I created file vue.config.js in root project with content below:
module.exports = {
  devServer: {
    disableHostCheck: true
  }
}

To me solved!

@farnetani Are you using Vue? I think you're in a wrong repo. If you could explain and demonstrate that your solution will work in Angular CLI then it would be very much helpful.

I finally got the issue resolved by simply checking my package.json to see if there's a webpack dependency, in my case, there was none so I ran the following"
npm install webpack-dev-server --save-dev
I hope this helps someone.

@william-creative-cx
Didn't really fix the WDS issue but reminded me to fix my dev dependencies so thanks!

Hi,
As suggested earlier in my case running angular 8 (Ubuntu OS) with "sudo npm start" seems to be working correctly now.
Seems that this is not general for everyone.
BR,
Zhi


Git: ignore vscode settings

Signed-off-by: arthur yang<EMAIL_ADDRESS>Ignore vscode settings for better IDE exprience.

Consider just setting up global ignore file on your computer https://help.github.com/en/github/using-git/ignoring-files#configuring-ignored-files-for-all-repositories-on-your-computer instead of doing this PR

Consider just setting up global ignore file on your computer https://help.github.com/en/github/using-git/ignoring-files#configuring-ignored-files-for-all-repositories-on-your-computer instead of doing this PR

Thanks. Will close this PR.


Add bio to Free Code Camp portfolios

GitHub recently introduced a bio field. We should also pull this from GitHub and store it on each user's object the same way we pull their name and location from GitHub.

[ ] Update user schema to store bio
[ ] Update GitHub auth logic to request and store bio
[ ] Update code portfolio view to display bio if present

For reference, here's my public info: (https://api.github.com/users/quincylarson). You can see the new attribute is simply "bio":
{
  "login": "QuincyLarson",
  "id": 985197,
  "avatar_url": "https://avatars.githubusercontent.com/u/985197?v=3",
  "gravatar_id": "",
  "url": "https://api.github.com/users/QuincyLarson",
  "html_url": "https://github.com/QuincyLarson",
  "followers_url": "https://api.github.com/users/QuincyLarson/followers",
  "following_url": "https://api.github.com/users/QuincyLarson/following{/other_user}",
  "gists_url": "https://api.github.com/users/QuincyLarson/gists{/gist_id}",
  "starred_url": "https://api.github.com/users/QuincyLarson/starred{/owner}{/repo}",
  "subscriptions_url": "https://api.github.com/users/QuincyLarson/subscriptions",
  "organizations_url": "https://api.github.com/users/QuincyLarson/orgs",
  "repos_url": "https://api.github.com/users/QuincyLarson/repos",
  "events_url": "https://api.github.com/users/QuincyLarson/events{/privacy}",
  "received_events_url": "https://api.github.com/users/QuincyLarson/received_events",
  "type": "User",
  "site_admin": false,
  "name": "Quincy Larson",
  "company": "Free Code Camp",
  "blog": "http://www.FreeCodeCamp.com",
  "location": "San Francisco, California, US",
  "email": null,
  "hireable": null,
  "bio": "I'm a teacher at Free Code Camp. I write articles about technology at medium.com/@quincylarson and quora.com/profile/Quincy-Larson",
  "public_repos": 22,
  "public_gists": 38,
  "followers": 932,
  "following": 13,
  "created_at": "2011-08-17T04:08:08Z",
  "updated_at": "2016-06-03T21:48:07Z"
}

Mind if I take care of this one?

@jacobbearden, definitely!
Please, check our Guidelines for Contributing and if you need any assistance reach out to the Contributors Chat room.

Can I work on this one?

@VishalRam, still looking to help? If you are, go right ahead and let us know :smile:

@atjonathan - Yes. I will work on this and send you a Pull request by weekend. Is that good?

@VishalRam, please hold back on this one. It might be better to choose a different issue to work on. I'm currently working on a revamp of the profile page, and I have already started integrating this and other new features. Happy Coding and Sorry about that :smiley:


How to authenticate SPFx app on SharePoint Page Load

Category

[ ] Question

Description
I want to create SPFx webpart which will get all users from Azure Active Directory using Microsoft Graph API.
I have used SPFx, Adal.js and react to do it.
I added my SPFx webpart in ShrePoint Site. SPFx webpart should able to get context as logged in user and load data.
I have attached part of code. Please find in attachment.
Expected or Desired Behavior
It should get the context of signed in user using cached and loads data.
Observed Behavior
I am not getting context once app part loads. I had put sing in buttons which prompts for microsoft login but at that time it gets user and logged in users so i am not coming back to sharepoint page from login prompts.
Code.txt
If you have any idea please let me know.

Currently this is in dev preview and you should be using the graph http client.  Please see the documentation at https://dev.office.com/sharepoint/reference/spfx/sp-http/class/graphhttpclient.


Allow to use AddGearbestApi using MS DependencyInjection extension

Allow to use AddGearbestApi using MS DependencyInjection extension
Remove static constructor
Fix Start and End date deserialization

Double check!


Fix bug that caused repair run to fail if nodes are going up or down

This PR fixes a tiny bug that caused repair runs to fail while collecting information about whether a repair could be run.  It seemed to happen when it contacted a node that was coming up or going down.  I described the problem more fully in issue #89

+1

:+1:


Cake 0.26.0 compatibility

Changed unit test to run under netcoreapp2.0
That enabled me to throw away the net461 target of the unit tests
Changed net461 to net46 instead which the Cake.Core is aiming for.
Changed netstandard1.6 to netstandard2.0
Version locked the cake version used to build Cake.IIS to 0.22.0 - this is just a best practise thing to version lock cake.
Updated .gitignore to add tools/packages.config which cake uses for version locking.

Fixes #47

Try v0.4.0


Changing mail report to use unique temp folder for saving temp files …

…needed for the mail.

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 2

Complexity increasing per file
==============================
- Ginger/GingerCoreNET/WorkSpaceLib/ReportsInfo.cs  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 2

Complexity increasing per file
==============================
- Ginger/GingerCoreNET/WorkSpaceLib/ReportsInfo.cs  1

See the complete overview on Codacy


Extend functionality to include all running applications

I don't have any applescript experience, but I was wondering if it was possible to extend the functionality to include resizing all running applications. I was thinking something along the lines of ps -ax | grep /Application/ integrated with what you have written. But I don't know applescript, so I don't actually know how you've implemented the window resizing.

I was thinking about extending the script to different applications, and I'll probably do this in the future. However, it would require adding more arguments, so it wouldn't be as convenient. I don't really get what you meant by saying:

I was thinking something along the lines of ps -ax | grep /Application/ integrated with what you have written

Could you please explain?

An app worth looking at that is doing this http://spectacleapp.com

If you are willing to do a bit more setup, [Hammerspoon] (http://www.hammerspoon.org/go/#winresize) lets you to write Lua code to control the OS and allows hotkey bindings. You can replicate all the functionality that Spectacle provides and much more.
You also might consider using Javascript Automation for OS X. This walkthrough is a nice place to start and here are the docs from Apple.

I'm not that eager to use it for different applications, because it isn't it's initial purpose. I created this script just to be able to resize the terminal windows with simple commands, and not add any more global keyboard shortcuts. If you want to have a universal support it's probably best to stick to something else. I'm wondering if it makes sense to create just another one window manager. Are there any features that you're missing in these that already exist?


Release latest version ?

I see the gem is a bit outdated on rubygems: https://rubygems.org/gems/twitter-typeahead-rails
Would you mind releasing the latest version ? :smiley:

+1

1

The examples given on http://twitter.github.io/typeahead.js/examples/ do not work with 0.10.5 (version packaged in this gem). It took me too long to figure out that it was not me doing something wrong...

Dittoing bosskovic, I just wasted a half hour on version mismatch issues.

Beware, the latest version (0.11) is broken with remote sources, I would wait before embedding it in that gem. The issue is here https://github.com/twitter/typeahead.js/issues/1185. The previous documentation can be accessed from here: https://github.com/twitter/typeahead.js/blob/v0.10.5/doc/jquery_typeahead.md

Done


[Bug] "Synchronize" after starting a feature branch while inside another feature branch

When you are in a feature branch A and decide to start another feature branch B (or hotfix, release), this new branch is properly created from "develop", but Idea keeps the files cached. Therefore, in your new branch B you will see all the changes you did in branch A (which are not in develop). If you try to commit, you will also see that these changes from A will be committed as well.
Solution is to click on "Synchronize" button in Idea (Ctrl + Alt + Y) after starting a new branch, which forces Idea to reload all files from the hard drive.
Could you add a similar "Synchronize" call to the plugin?

Do you have this option in Intellij disabled?

Nope, I have it enabled (checked). Frame activation is when you switch from one tab (file) to another, and it seems that when I do that, the contents are actually synchronized.
However, when I'm working on one file in a feature branch A, commit, then create a new feature branch B (through giflow plugin), the contents are not synchronized. So if I want to work on the same file, I have to remember to manually synchronize, or at least switch to a different file and back. Otherwise, I contaminate my new feature branch with everything that I did in the old branch.

Alright - thanks for the clarification


Supporting for docker tmpfs volume mounts in steps

fix #862

Could you provide some details what you are trying to achieve by using a tmpfs, so we can maybe help you implementing it?

Using docker's tmpfs as the volume mount type for the working dir (/woodpecker) instead of a local volume. See also linked issue.
This allows for all actions within /woodpecker to happen in the host's memory instead of on disk. This can result in a step up of up to 20x for certain r/w intensive tasks and also save a lot of r/w cycles on the linked disks.
Downside: no state persistence between containers in a pipelines. I.e. cloning must be disabled and everything must happen in one step.
I want to add a boolean flag (currently named use_tmpfs) to allow for this and users which know what to do can then set it on a step level.
While there is already an undocumented tmpfs step option, this one is different as it would add an additional tmpfs mount somewhere else. This would require moving all files there first and then executing any task.

Wouldn't it be possible to pass a tmpfs volume across all steps of a workflow? We could maybe add it as option to https://woodpecker-ci.org/docs/next/usage/workflow-syntax#workspace.

Wouldn't it be possible to pass a tmpfs volume across all steps of a workflow?

Of course but this wouldn't help/solve the case I am describing. These mounts would live outside of /woodpecker and always require manual action to read/write files to/from this directory.
The additional issue is that you don't always want this. I.e. it only really helps for instances with a lot of RAM and you can use that RAM instead of a physical disk. On small VMs/homelabs with like 4 GB of RAM, you don't wanna use this as otherwise it will stall the host completely. And setting a sensible default there is not so easy (by default all available memory is provided for the mount).
So to me it makes most sense to have both options as "enabled on request" including a size limitation of how much RAM should be used for the mount. And maybe even a top-level env var which allows to prevent these kind of mounts on the admin side. And "proper" documentation on how and when to use this option.
Example: https://vladmihalcea.com/how-to-run-integration-tests-at-warp-speed-with-docker-and-tmpfs/

Of course but this wouldn't help/solve the case I am describing. These mounts would live outside of /woodpecker and always require manual action to read/write files to/from this directory.

I was talking about the workspace. My idea was basically to allow a user to use a tmpfs for the /woodpecker workspace instead of the normal volume.

Regarding the permission part, we could automatically add a label tmpfs=true to pipelines in case a tmpfs workspace (or later on maybe volume) was requested, so only agents with tmpfs=true or tmpfs=* would pick such pipelines.

I basically wanted to suggest to allow a user to use a tmpfs for the /woodpecker workspace instead of the normal volume

That is the exact purpose of this PR 😄
How the step setting should be named then and structured is TBD. I already tested it in a previous version (not in this PR) by hardcoding the TMPFS creation. Worked fine.
Now I wanted to get it working using a new step variable. Which is where I am stuck now. Do you see what is missing so that the variable is actually honored?

Regarding the permission part, we could automatically add a label tmpfs=true to pipelines in case a tmpfs workspace (or later on maybe volume) was requested, so only agents with tmpfs=true or tmpfs=* would pick such pipelines.

That might be one option but I guess you need a config setting on the admin side to allow/disallow this for pipelines in general. And/or to limit the size of the tmpfs mount. A label can surely also be added but I think there's no way around global options which apply to all agents, so that this feature doesn't require a dedicated agent necessarily.

Stale.


Add vscode.workspace.fs.chmod().

(see #84175)
For the Python extension, we need the ability to change file permissions.  For example: https://github.com/microsoft/vscode-python/blob/master/src/client/activation/languageServer/downloader.ts#L165.  Currently we are using node's fs.chmod().
@jrieken

According to https://github.com/microsoft/vscode/issues/84175#issuecomment-551445383, this is a dupe of #73122.  However, that issue is about seeing if a file is readonly or not.  This one is about changing the permissions on the filesystem (including the "executable" bit).  A more closely related issue is #48659.

this is a dupe of #73122. However, that issue is about seeing if a file is readonly or not. This one is about changing the permissions on the filesystem (including the "executable" bit). A more closely related issue is #48659.

Once we start on this, we won't do something half-basked


Localize

Hi, thanks for a nice work. I am using the EmailComposer in my SwiftUI app. It works great! I am going to localize the app for other languages. My question is if you plan to add support for this? I attach how I use EmailData.
var body: some View {
    let emailData = EmailData(subject: "Invitation shared game", recipients: [email], body:
"""
Hi,
Click on the attached link to connect.

\(url!)

PS.
Shared play takes place via the internet and Apple iCloud.

In case of problems check the following:
- Is there a good connection to the internet?
- If your team does not appear in the app, click on the attached link again.
DS.
""")

Hi again,
I'm a beginner at translation / localization, sorry.. But, I have now found an easy solution to my earlier question. I used a variable instead of static text. For example for the subject: var emailSubject = String(localized: "Invitation shared game").

Hi @MartinAtElitappar,
Thank you for using EmailComposer, I'm really glad it's useful for your project! Right now localization related additions are not in my plans, however that's not a problem for you anymore as I understand. What you did to localize your texts is the best and most proper approach, I'd do the same to be honest, so keep going with it.
Thanks again and stay safe!


Version 8.0.8 documentation site broken navigation

What RMWC Version are you using [major.minor.patch]: 14.0.5

Describe the bug with as much detail as possible:

It is not possible to navigate the version 8.0.8 documentation after the release of version 14

Team question: can you build the base url with an env variable? Jens wonders.


Let GrpcService specify a maximum bound for grpc-timeout

Currently, armeria has a useClientTimeoutHeader option to decide whether to respect the grpc-timeout request header or not.
Since useClientTimeoutHeader = true delegates the timeout value to the client-side, it is potentially dangerous.
On the other hand, setting useClientTimeoutHeader = false completely ignores the client header and Armeria's requestTimeout is used.
We may want to offer users a middle ground: Armeria server tries to respect the grpc-timeout header, but clamps the timeout according to a bound.
In order to support this, we may deprecate the previous useClientTimeoutHeader and accept a function which determines how to decide the request timeout. This might also be useful if users want to set timeouts depending on the remote.
@Deprecated
GrpcServiceBuilder useClientTimeoutHeader(boolean useClientTimeoutHeader) {}

interface GrpcRequestTimeoutMapper {

  long map(RequestHeaders headers, ServiceRequestContext ctx);

  static GrpcRequestTimeoutMapper USE_CLIENT_TIMEOUT_HEADER = (headers, ctx) -> {
    // https://github.com/line/armeria/blob/e263d76deff8f936a58db69de6ad63d0c0458407/grpc/src/main/java/com/linecorp/armeria/server/grpc/FramedGrpcService.java#L226-L254
  }

  static GrpcRequestTimeoutMapper USE_SERVER_REQUEST_TIMEOUT = (ignored, ctx) -> TimeUnit.MILLISECONDS.toNanos(ctx.requestTimeoutMillis());
}

GrpcServiceBuilder requestTimeout(GrpcRequestTimeoutMapper mapper) {}

...
GrpcRequestTimeoutMapper mapper;
...
HttpResponse doPost {
  // returns nanos
  long timeout = mapper.map(headers, ctx);
  if (timeout <= 0) {
    ctx.clearRequestTimeout();
  } else {
    ctx.setRequestTimeout(TimeoutMode.SET_FROM_NOW, Duration.ofNanos(timeout));
  }
}
...

ref: https://discord.com/channels/1087271586832318494/1245876539682324642/1245948161835667528

grpc-timeout is not just a number. A value representing a time unit is appended to the value.
It is a hassle to parse the value.
How about passing the parsed grpc-timeout value to the interface?
interface GrpcClientTimeoutHandler {
    static GrpcClientTimeoutHandler enabled() {
       return Function.identity();
    }

    static GrpcClientTimeoutHandler disabled() {
       return (ctx, timeout) -> -1;
    }

    static GrpcClientTimeoutHandler withBuffer(long bufferMillis) {
       return (ctx, timeout) -> timeout + buffer;
    }

    long apply(ServiceRequestContext ctx, long timeoutMillis);
}

Also possibly related https://github.com/line/armeria/issues/3155

er. (The only concern I have is nanos vs. millis

If it is considerable, Duration may be an alternative.

I wonder if any of the server side flags affect the timeout behavior?

defaultRequestTimeoutMillis
defaultResponseTimeoutMillis

etc. IIRC some of those caused connection to be terminated pre-maturely before client initiates a timeout.

Timeout does not cause connections to be terminated. In HTTP/2, the timeout sends an error response and the connection will be reused.

I wonder if any of the server side flags affect the timeout behavior?

Possible. We are going to provide an option to allow grpc-timeout but the maximum value is limited by defaultRequestTimeoutMillis.


Datafusion crossjoin

Codecov Report

:exclamation: No coverage uploaded for pull request base (datafusion-sql-planner@75116de). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head c3d9b0d differs from pull request most recent head a514782. Consider uploading reports for the commit a514782 to get more accurate results

@@                    Coverage Diff                    @@
##             datafusion-sql-planner     #521   +/-   ##
=========================================================
  Coverage                          ?   52.74%
=========================================================
  Files                             ?       71
  Lines                             ?     3473
  Branches                          ?      699
=========================================================
  Hits                              ?     1832
  Misses                            ?     1518
  Partials                          ?      123

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Support for Many-to-Many relationship

I am developing a tree structure with "detachable" sub-trees, and for what I have seen the current TreeNode and TreeNodeForeignKey are not suited for this.

To accomplish this kind of structure I need to use a Many-to-Many  parent (and also I need to define some extra fields on the M2M relation-table).
I do not need to query the whole graph, just individual trees (which can intersect with each other as shown in figure).
In MPTT I would have used the TreeManyToManyField, but since that project is not maintained I would like to accomplish something similar with this library.
I undestrand that it is not an easy step but I would like to see if it is feasable.

Hi,
django-mptt's TreeManyToManyField is just a m2m field with some additional goodies and doesn't influence the core behavior of MPTT at all. It couldn't; django-mptt assumes that each node has at most one parent and django-tree-queries does the same.
The tree_depth and tree_path attributes of the nodes in query tree C are ambiguous depending on whether you come from A or B and are undefined when coming from A and B at the same time. So, as I see it, django-tree-queries can't help much without first thinking about the value these tree attributes should have.
--
But maybe I'm misunderstanding and you "just" need a many to many field where the label shows the tree structure somehow? It should be easy to extend django-tree-queries with a TreeNodeManyToManyField (see https://github.com/matthiask/django-tree-queries/blob/main/tree_queries/fields.py , there's not much there -- deconstruct avoids creating migrations when replacing a ForeignKey with a TreeNodeForeignKey, formfield only ensures that the model field uses the TreeNodeChoiceField form field). The TreeNodeMultipleChoiceField already exists https://github.com/matthiask/django-tree-queries/blob/main/tree_queries/forms.py
You could even do this in your own project. Or submit a pull request with the new model field.

Hi @matthiask, thanks for your response.
To better explain my use case the trees I am willing to use are "Procedures", where each node is a precise step and which could be a sub-procedure. My aim is to have sub-procedures reusable, without duplicating their sub-tree.
This is why current TreeNodeForeignKey, that is Many-to-One (leaves to root), poses a limit.
I understand that both depth and path are not absolute in this structure, since they depends on the entry-point (and now I realize that my drawing shows an "happy scenario", where depth of C is the same in both A and B).
Also I understand that having the same sub-tree used twice in a single tree could get the query a bit complicated and result not so easy to understand.
A -- X -- C
  \- Y -- Z -- C

I will look deeper to see how this kind of recursive query could be built, because to have a "deterministic" path and depth will require a join to handle the M2M.

I have found that for my use-case the django-postgresql-dag could be a better suite. Since with an M2M the tree becomes a graph (I have re-read my graph-theory books and now I am starting hating to have accepted a feature which involves this elements).

Great! Yes, a DAG library is the way to go, and this is quite definitely out of scope for django-tree-queries.


Fulfill and hash

Make necessary modifications to work with upcoming change brought by https://github.com/bigchaindb/bigchaindb/pull/1899

[ ] update docs (basic & advanced usage, hand-crafting tx)

Codecov Report

Merging #343 into master will not change coverage.
The diff coverage is n/a.


Question about integration of Plan2Explore

Hi, I'd like to integrate the Plan2Explore algorithm into the latest dreamerv3 repo. The original code of Plan2Explore in the dreamerv2 repo seems to differ quite a lot from the current layout of the dreamerv3. And the current usage of some advanced Jax features in JAXAgent seems quite complicated and I'm not sure which part of the code should I begin with first.
May I ask, do you have any advice on how to integrate the Plan2Explore?
Thank you very much.

Hi, we removed that from the code during cleanup but it should be quite easy to add back. You don't need to make any changes to jaxagent.py. Just add the ensemble network with a stop gradient to the model graph and include it in the optimizer, and then add the disagreement to the rewards?


Adding my name to contributors for my approved question

🛠 Fixes Issue

➕ Changes Introduced
I submitted 2 interview questions and they got approved
📄 Note To Reviewers

📷 Screenshots

Can you link the issue where we had this discussion, and fix the PR title? Please read the contribution guideline first.

https://github.com/ykdojo/defaang/issues/232
we had this discussion on issue number 232
yes fixing the title

we had this discussion on issue number 232
https://github.com/ykdojo/defaang/issues/232
title fixed


Add yarn.lock file

#102

Coverage remained the same at 82.026% when pulling 47f4884966b3545b1a52471021ebacf73b878819 on add-yarn-lock-file into 818b2d7dfaf6dfe8c3b02435636c4320c1d867fd on develop.


Allows support for '...' in bindings with select_merge macro

This PR fixes a bug that causes an exception when '...' is used in the bindings for select_merge. This is done by unquoting the query prior to the Builder.Select.merge/2 call. Currently, '...' only appears to work when a select_merge is already used previously in the query.
This bug is also in Ecto version 2.2 and maybe earlier.
Resolves #2605

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


utter actions are not recognized during domain sanity check for intents with trigger actions

Rasa version: 1.10.9
Python version: 3.7
Issue:
If a response is listed as the triggers action for an intent, the domain sanity check gives an error because the action is not found in the actions list.
e.g.
- handoff:
    triggers: utter_greet

see https://github.com/RasaHQ/financial-demo/issues/59.
This issue is new in 1.10.9.
Error (including full traceback):
  File "/home/henry/.venv/bin/rasa", line 10, in <module>
    sys.exit(main())
  File "/home/henry/.venv/lib/python3.7/site-packages/rasa/__main__.py", line 92, in main
    cmdline_arguments.func(cmdline_arguments)
  File "/home/henry/.venv/lib/python3.7/site-packages/rasa/cli/train.py", line 76, in train
    additional_arguments=extract_additional_arguments(args),
  File "/home/henry/.venv/lib/python3.7/site-packages/rasa/train.py", line 50, in train
    additional_arguments=additional_arguments,
  File "uvloop/loop.pyx", line 1456, in uvloop.loop.Loop.run_until_complete
  File "/home/henry/.venv/lib/python3.7/site-packages/rasa/train.py", line 101, in train_async
    additional_arguments,
  File "/home/henry/.venv/lib/python3.7/site-packages/rasa/train.py", line 172, in _train_async_internal
    new_fingerprint = await model.model_fingerprint(file_importer)
  File "/home/henry/.venv/lib/python3.7/site-packages/rasa/model.py", line 290, in model_fingerprint
    domain_without_nlg = Domain.from_dict(domain_dict)
  File "/home/henry/.venv/lib/python3.7/site-packages/rasa/core/domain.py", line 162, in from_dict
    **additional_arguments,
  File "/home/henry/.venv/lib/python3.7/site-packages/rasa/core/domain.py", line 431, in __init__
    self._check_domain_sanity()
  File "/home/henry/.venv/lib/python3.7/site-packages/rasa/core/domain.py", line 1049, in _check_domain_sanity
    incorrect_mappings,
rasa.core.domain.InvalidDomain: Intent 'handoff' is set to trigger action 'utter_greet', which is not defined in the domain.

Command or request that led to error:
rasa train

Content of domain file (domain.yml) (if relevant):
session_config:
  session_expiration_time: 0
  carry_over_slots_to_new_session: true

intents:
- transfer_money:
    use_entities: []
...
- handoff:
    triggers: utter_greet
- help

responses:
  utter_greet:
  - text: Hi! I'm your Financial Assistant!
...

actions:
- action_account_balance
- action_session_start
- action_restart
- action_credit_card_balance
- action_recipients
- action_handoff
- action_handoff_options

May be have some bug.
rasa 1.10.3 work fine.

@Dustyposa yes, this was introduced in 1.10.9

closing this as MappingPolicy is removed from Rasa Open Source


Multiplication following a int8 transposed convolutions isn't constant folded

Description of the bug:
Original Issue: https://github.com/tensorflow/tensorflow/issues/57680
Opening on behalf of @lgeiger
When converting a model that used int8 quantization aware training, conversion of transposed convolutions followed by a scalar multiplication fails.
The converter isn't able to correctly constant fold the per-tensor fake quantized weights and the scalar multiplication which is a common patter when using transposed convolutions followed by batch normalisation layers. This is a follow-up issue to #53766
1. System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): macOS / Ubuntu
TensorFlow installation (pip package or built from source): pip
TensorFlow library (version, if pip package or github SHA, if built from source): 2.10.0 / 2.11.0-dev20220913

2. Code
A minimal reproduction of the issue is available in this notebook. Re-run the notebook to show netron visualisations showing the conversion problem.
Actual vs expected behavior:
No response
Any other information you'd like to share?
No response

This issue originally reported by @lgeiger has been moved to this dedicated repository for ai-edge-torch to enhance issue tracking and prioritization. To ensure continuity, we have created this new issue on your behalf.
We appreciate your understanding and look forward to your continued involvement.


Version Information

Run go version to get version of Go from the VS Code integrated terminal.

go version go1.22.2 linux/amd64

Run gopls -v version to get version of Gopls from the VS Code integrated terminal.

golang.org/x/tools/gopls v0.15.3
<EMAIL_ADDRESS>h1:zbdOidFrPTc8Bx0YrN5QKgJ0zCjyGi0L27sKQ/bDG5o=
<EMAIL_ADDRESS>h1:9F2/+DoOYIOksmaJFPw1tGFy1eDnIJXg+UHjuD8lTak=
<EMAIL_ADDRESS>h1:ofyhxvXcZhMsU5ulbFiLKl/XBFqE1GSq7atu8tAmTRI=
<EMAIL_ADDRESS>h1:2O2DON6y3XMJiQRAS1UWU+54aec2uopH3x7MAiqGW6Y=
<EMAIL_ADDRESS>h1:SernR4v+D55NyBH2QiEQrlBAnj1ECL6AGrA5+dPaMY8=
<EMAIL_ADDRESS>h1:5BMeUDZ7vkXGfEr1x9B4bRcTH4lpkTkpdh0T/J+qjbQ=
<EMAIL_ADDRESS>h1:vcVnuftN4J4UKLRcgetjzfU9FjjgXUUYUc3JhFplgV4=
<EMAIL_ADDRESS>h1:ScX5w1eTa3QqT8oi6+ziP7dTV1S2+ALU0bI+0zXKWiQ=
<EMAIL_ADDRESS>h1:uH9jJYgeLCvblH0S+03kFO0qUDxRkbLRLFiKVVDl7ak=
<EMAIL_ADDRESS>h1:KUas02EjQK5LTuIx1OylBQdKKZ9jeugs+HiqO5HormU=
<EMAIL_ADDRESS>h1:oFEHCKeID7to/3autwsWfnuv69j3NsfcXbvJKuIcep8=
<EMAIL_ADDRESS>h1:G3QvahNDmpD+Aek/bNOLrFR2XC6ZAdo62dZu65gmwGo=
<EMAIL_ADDRESS>h1:lyBNOm8Wo71UknhUs4QTFUNNMyxy2JEIaKKo0RWOh+8=
go: go1.22.2

Run code -v or code-insiders -v to get version of VS Code or VS Code Insiders.

1.88.1

Check your installed extensions to get the version of the VS Code Go extension

v0.41.2

Run Ctrl+Shift+P (Cmd+Shift+P on Mac OS) > Go: Locate Configured Go Tools command.

Environment
GOBIN: undefined
toolsGopath:
gopath: /root/go
GOROOT: /usr/local/go
PATH: /root/.vscode-server/bin/e170252f762678dec6ca2cc69aba1570769a5d39/bin/remote-cli:/usr/local/go/bin:/root/go/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/root/.local/bin:/root/.local/bin
Tools
go:	/usr/local/go/bin/go: go version go1.22.2 linux/amd64

gopls:	/root/go/bin/gopls	(version: v0.15.3 built with go: go1.22.2)
gotests:	/root/go/bin/gotests	(version: v1.6.0 built with go: go1.22.2)
gomodifytags:	/root/go/bin/gomodifytags	(version: v1.16.0 built with go: go1.22.2)
impl:	/root/go/bin/impl	(version: v1.1.0 built with go: go1.22.2)
goplay:	/root/go/bin/goplay	(version: v1.0.0 built with go: go1.22.2)
dlv:	/root/go/bin/dlv	(version: v1.22.1 built with go: go1.22.2)
staticcheck:	/root/go/bin/staticcheck	(version: v0.4.7 built with go: go1.22.2)

Go env
Workspace Folder (server):
GO111MODULE=''
GOARCH='amd64'
GOBIN=''
GOCACHE='/root/.cache/go-build'
GOENV='/root/.config/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='amd64'
GOHOSTOS='linux'
GOINSECURE=''
GOMODCACHE='/root/go/pkg/mod'
GONOPROXY=''
GONOSUMDB=''
GOOS='linux'
GOPATH='/root/go'
GOPRIVATE=''
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/usr/local/go'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/usr/local/go/pkg/tool/linux_amd64'
GOVCS=''
GOVERSION='go1.22.2'
GCCGO='gccgo'
GOAMD64='v1'
AR='ar'
CC='gcc'
CXX='g++'
CGO_ENABLED='1'
GOMOD='/xxx/go.mod'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -m64 -pthread -Wl,--no-gc-sections -fmessage-length=0 -ffile-prefix-map=/tmp/go-build445704314=/tmp/go-build -gno-record-gcc-switches'

Workspace Folder (web):
GO111MODULE=''
GOARCH='amd64'
GOBIN=''
GOCACHE='/root/.cache/go-build'
GOENV='/root/.config/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='amd64'
GOHOSTOS='linux'
GOINSECURE=''
GOMODCACHE='/root/go/pkg/mod'
GONOPROXY=''
GONOSUMDB=''
GOOS='linux'
GOPATH='/root/go'
GOPRIVATE=''
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/usr/local/go'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/usr/local/go/pkg/tool/linux_amd64'
GOVCS=''
GOVERSION='go1.22.2'
GCCGO='gccgo'
GOAMD64='v1'
AR='ar'
CC='gcc'
CXX='g++'
CGO_ENABLED='1'
GOMOD='/xxx/go.mod'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -m64 -pthread -Wl,--no-gc-sections -fmessage-length=0 -ffile-prefix-map=/tmp/go-build300344583=/tmp/go-build -gno-record-gcc-switches'

Describe the bug
go 1.22 alredy fix the loop vairable issue, why waring "loop variable v captured by func literal loopclosure" (go.mod go version 1.22.2)
waring link https://pkg.go.dev/golang.org/x/tools/go/analysis/passes/loopclosure
Steps to reproduce the behavior:
package main

import "fmt"

func main() {
	elem := make([]string, 10)
	for _, v := range elem {
		go func() {
			fmt.Println(v)
		}()
	}
}

Screenshots or recordings

I tried on go1.22.2 darwin/amd64 and the issue was not reproducible.

Looks like the GoVersion is < 1.22.0 for the file. This value is derived from the go.mod file for the module, the build tags for the file, and the compiler version used to build gopls (and the -lang flag in some rare cases). The compiler version used to build gopls is >= 1.22 so that is not the issue.
What does the "go 1.XY" line say in the go.mod file? (Or is there no go.mod file?)

Looks like the GoVersion is < 1.22.0 for the file. This value is derived from the go.mod file for the module, the build tags for the file, and the compiler version used to build gopls (and the -lang flag in some rare cases). The compiler version used to build gopls is >= 1.22 so that is not the issue.
What does the "go 1.XY" line say in the go.mod file? (Or is there no go.mod file?)

go.mod: go 1.22.2

Run Ctrl+Shift+P (Cmd+Shift+P on Mac OS) > Go: Locate Configured Go Tools command.

try this command
go clean -cache -modcache

then Run Ctrl+Shift+P (Cmd+Shift+P on Mac OS) > Go: Install/Update Tools command.

I tried on go1.22.2 darwin/amd64 and the issue was not reproducible.

try this command
go clean -cache -modcache

then Run Ctrl+Shift+P (Cmd+Shift+P on Mac OS) > Go: Install/Update Tools command and select all options

I'm seeing the same issue. Looks like a possible regression -> https://github.com/golang/go/issues/65910. gopls 0.15.2 works without issue.
Following the same steps from that issue:
gopls version
Build info
----------
golang.org/x/tools/gopls v0.15.3
<EMAIL_ADDRESS>h1:zbdOidFrPTc8Bx0YrN5QKgJ0zCjyGi0L27sKQ/bDG5o=
<EMAIL_ADDRESS>h1:9F2/+DoOYIOksmaJFPw1tGFy1eDnIJXg+UHjuD8lTak=
<EMAIL_ADDRESS>h1:ofyhxvXcZhMsU5ulbFiLKl/XBFqE1GSq7atu8tAmTRI=
<EMAIL_ADDRESS>h1:2O2DON6y3XMJiQRAS1UWU+54aec2uopH3x7MAiqGW6Y=
<EMAIL_ADDRESS>h1:SernR4v+D55NyBH2QiEQrlBAnj1ECL6AGrA5+dPaMY8=
<EMAIL_ADDRESS>h1:5BMeUDZ7vkXGfEr1x9B4bRcTH4lpkTkpdh0T/J+qjbQ=
<EMAIL_ADDRESS>h1:vcVnuftN4J4UKLRcgetjzfU9FjjgXUUYUc3JhFplgV4=
<EMAIL_ADDRESS>h1:ScX5w1eTa3QqT8oi6+ziP7dTV1S2+ALU0bI+0zXKWiQ=
<EMAIL_ADDRESS>h1:uH9jJYgeLCvblH0S+03kFO0qUDxRkbLRLFiKVVDl7ak=
<EMAIL_ADDRESS>h1:KUas02EjQK5LTuIx1OylBQdKKZ9jeugs+HiqO5HormU=
<EMAIL_ADDRESS>h1:oFEHCKeID7to/3autwsWfnuv69j3NsfcXbvJKuIcep8=
<EMAIL_ADDRESS>h1:G3QvahNDmpD+Aek/bNOLrFR2XC6ZAdo62dZu65gmwGo=
<EMAIL_ADDRESS>h1:lyBNOm8Wo71UknhUs4QTFUNNMyxy2JEIaKKo0RWOh+8=
go: go1.22.2

go env
GO111MODULE=''
GOARCH='arm64'
GOBIN=''
GOCACHE='/Users/dominic/Library/Caches/go-build'
GOENV='/Users/dominic/Library/Application Support/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='arm64'
GOHOSTOS='darwin'
GOINSECURE=''
GOMODCACHE='/Users/dominic/go/pkg/mod'
GONOPROXY='gitlab.morningconsult.com'
GONOSUMDB='gitlab.morningconsult.com'
GOOS='darwin'
GOPATH='/Users/dominic/go'
GOPRIVATE='gitlab.morningconsult.com'
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/usr/local/go'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/usr/local/go/pkg/tool/darwin_arm64'
GOVCS=''
GOVERSION='go1.22.2'
GCCGO='gccgo'
AR='ar'
CC='clang'
CXX='clang++'
CGO_ENABLED='1'
GOMOD='/Users/dominic/Projects/loopclosure/go.mod''
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -ffile-prefix-map=/var/folders/r3/5ch28yq55hv_8fk9bd_7jdnm0000gn/T/go-build1471649584=/tmp/go-build -gno-record-gcc-switches -fno-common'

What did you do?
With the following main.go:
package main

import (
	"fmt"
)

func main() {
	for i := range 10 {
		go func() { fmt.Println(i) }()
	}
}

and go.mod
module example.com/loopclosure

go 1.22.2

I ran
gopls check main.go

What did you see happen?
/Users/dominic/Projects/sandbox/main.go:9:27-28: loop variable i captured by func literal

What did you expect to see?
No error message

Thanks, yes this is a regression in<EMAIL_ADDRESS>specifically this CL: https://go.dev/cl/576678
That code should be comparing language versions, not toolchain versions :(
Fix incoming. To disable this analysis in the meantime: "analyses": { "loopclosure": false }

Aha, we didn't catch this because it isn't broken at master.
The problem is that gopls-release-branch.0.15 lacks https://go.dev/cl/567635.

Closing as this has been cherry-picked to the release branch, and we are preparing the v0.15.4 release.
The release branch was fixed in https://go.dev/cl/579758 (the issue number was wrong in that CL, which is why it wasn't associated here).

@synhi suggestion worked for me.
I am using nvim:

Updated gopls with :Mason + U
Ran go clean -cache -modcache
Than, go mod tidy
Finally, restarted nvim my lsp :LspRestart

I did everything apart from 2 and it failed. It was only after that command that it solved for me
Hope it helps

Thanks @0xBradock. For what it's worth, steps (2) and (3) should not be necessary. Simply updating gopls to the latest version and restarting should fix the problem.


[gz-sensors7] New port

[x] Changes comply with the maintainer guide
[x] The name of the port matches an existing name for this component on https://repology.org/ if possible, and/or is strongly associated with that component on search engines.
[x] Optional dependencies are resolved in exactly one way. For example, if the component is built with CMake, all find_package calls are REQUIRED, are satisfied by vcpkg.json's declared dependencies, or disabled with CMAKE_DISABLE_FIND_PACKAGE_Xxx
[x] The versioning scheme in vcpkg.json matches what upstream says.
[x] The license declaration in vcpkg.json matches what upstream says.
[x] The installed as the "copyright" file matches what upstream says.
[x] The source code of the component installed comes from an authoritative source.
[x] The generated "usage text" is accurate. See adding-usage for context.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is in the new port's versions file.
[x] Only one version is added to each modified port's versions file.

This port is depends on gz-rendering7 that depends on ogre-next. That way it skip in the CI.
I tested it on windows and linux.

Tested usage successfully by gz-sensors7:x64-windows:
Total install time: 2.9 h
gz-sensors7 provides CMake targets:

  # this is heuristically generated, and may not be correct
  find_package(gz-sensors7 CONFIG REQUIRED)
  target_link_libraries(main PRIVATE gz-sensors7::core gz-sensors7::requested gz-sensors7::gz-sensors7)

wow, why is it 2.9h?
On my pc on windows it takes several minutes.
I will post time soon.

On windows 11:
Elapsed time to handle gz-sensors7:x64-windows: 1.9 min
Total install time: 1.9 min
gz-sensors7 provides CMake targets:
this is heuristically generated, and may not be correct


API Documentation

As an API user
I want clear and comprehensive API documentation
So that I know how to extract useful data from the API

see
https://github.com/impactoss/impactoss-server/blob/master/doc/swagger-api.yaml


Remember last "page" (including the current song) when you exit Google Music

If you exit Google Music while a song was playing and then you open it again, Google Music starts at the "main" Google Music page, and it forgets the song that you were playing, the playlist (or album, artist, etc) that you had selected.
It would be really nice if when you reopen Google Music is went back to the previous song (either by default or as an option). It would be even better if it remembered your "browse history" and when you reopened the window you could even go back using the navigation buttons (or the 3 finger swipe).
I think this is the only "big" issue I have left with Google Music which is really awesome already :-D

@jacobwgillespie @chrismou What is the viability of a feature like this?

It could probably be implemented inside the JS library (local cache of current location)...

@jacobwgillespie Do you think the google-music.js addition will help us with this at all? Or will we just have to implement our own caching of the current location?

Possibly - it's technically not part of the Google Music functionality though, so I'd guess we'd want our own JS library.  It'd be pretty simple to implement, basically just subscribe to the page location, store its value in localStorage on change, then when the page loads initially check to see if there's a stored location and redirect there.  We could add an option in the Preferences to opt-in to that behavior (which would just control whether the script was injected or not).
If gmusic.js decides to interface with Google Music's routing though we might want to tie into that.  Which could be cool if it did - we could add convenience methods like go-to-artist, go-to-album, etc.


DTU_traning.rar 404

Hi, great work,
But the traing dataset link of MVSformer is 404, could you please release a new link,

I find the training dataset is correct， maybe you need a VPN

Thanks for your answering, I did switch on the vpn before, maybe something wrong with my network, it can be downloaded now, best wishes

Thanks for your answering, I did switch on the vpn before, maybe something wrong with my network, it can be downloaded now, best wishes


Send prompt to only users with shorts

Able to send a prompt to only users with shorts enabled from feature flag

#747


HDFS-16795. Use secure XML parsers (#4979)

Move construction of XML parsers in HDFS
modules to using the locked-down parser factory
of HADOOP-18469.
Contributed by P J Fanning
Change-Id: I9e21228eeebff699ebd22f46a99722cb9efb0cf4

Description of PR
How was this patch tested?
For code changes:

[ ] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 48s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 3 new or modified test files.

_ branch-3.3 Compile Tests _

+0 :ok:
mvndep
15m 22s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
27m  1s

branch-3.3 passed

+1 :green_heart:
compile
4m  3s

branch-3.3 passed

+1 :green_heart:
checkstyle
1m 14s

branch-3.3 passed

+1 :green_heart:
mvnsite
2m 34s

branch-3.3 passed

+1 :green_heart:
javadoc
2m 23s

branch-3.3 passed

+1 :green_heart:
spotbugs
6m 17s

branch-3.3 passed

+1 :green_heart:
shadedclient
28m 37s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 27s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 14s

the patch passed

+1 :green_heart:
compile
3m 55s

the patch passed

+1 :green_heart:
javac
3m 55s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 58s

the patch passed

+1 :green_heart:
mvnsite
2m 11s

the patch passed

+1 :green_heart:
javadoc
2m  0s

the patch passed

+1 :green_heart:
spotbugs
6m  1s

the patch passed

+1 :green_heart:
shadedclient
28m  9s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
2m 18s

hadoop-hdfs-client in the patch passed.

-1 :x:
unit
213m 46s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 59s

The patch does not generate ASF License warnings.

349m 15s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.datanode.TestDataNodeRollingUpgrade

hadoop.hdfs.server.blockmanagement.TestBlockTokenWithDFSStriped

hadoop.hdfs.server.datanode.TestBPOfferService

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5016/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5016

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux d469669f2241 4.15.0-191-generic #202-Ubuntu SMP Thu Aug 4 01:49:29 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.3 / 47ce9fafb31f804c3d6342ffbd2d24bd33340353

Default Java
Private Build-1.8.0_342-8u342-b07-0ubuntu1~18.04-b07

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5016/2/testReport/

Max. process+thread count
2167 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs-client hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5016/2/console

versions
git=2.17.1 maven=3.6.0 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.


Re-Implement Pod Status Indicators

Based on changes in explorer.ts the pod status indicators need to be re-implemented to take into account getPods and centralize logic based on the items noted in #285

Closed via. #308


Armor 2

Armor 2 dat is displaying as an image file with the February 2016 update. Armor 1 is unaffected.

Fixed.


How to edit the slash commands

I want to edit the slash commands, to add my own  , how can i do that. Could  you guys also please develop a nice documentation

You can join the fork project with the aim of customization and documentation: https://github.com/sergeysova/craft

I have the same question, do you have a good idea ？

Any update on this?

https://novel.sh/docs/guides/tailwind/slash-command you can now customize each extension, especially the slash command feature


SSH tunnel cannot be used after VPN connection

System information:

Windows 10、Mac OS
7.1.3

Describe the problem you're observing:
Some of my databases need to be accessed through the office host. When I connect to the office network through VPN, I can't use SSH tunnel to connect to the office host, but I can connect directly through the SSH client
ERROR:
Cannot establish tunnel
timeout: socket is not established
Steps to reproduce, if exist:
Include any warning/errors/backtraces from the logs

Please write SSH host in Main tab of connection settings and do not fill SSH tab. Does it help?

Hello, I had this problem and found a solution, use variables instead in SSH tab, ${host}, ${user}, ${password}, not sure if this is a bug but it works for me.


SetCumulVarSoftUpperBound on End node does no effect

Hi! I try to use SetCumulVarSoftUpperBound to stimulate each vehicle to have average orders count.
Short example:
avg_done_count = 10
routing.AddConstantDimension(1, 999999999, True, 'total_done')
total_done_dimension = routing.GetDimensionOrDie('total_done')
for route in routes:
    _i = routing.End(route)
    total_done_dimension.SetCumulVarSoftUpperBound(_i, avg_done_count, 1000)

And this does not add any value to total cost.
But when i make it for all orders instead of Ends it works:
avg_done_count = 10
routing.AddConstantDimension(1, 999999999, True, 'total_done')
total_done_dimension = routing.GetDimensionOrDie('total_done')
for order in orders:
    _i = routing.NodeToIndex(order)
    total_done_dimension.SetCumulVarSoftUpperBound(_i, avg_done_count, 1000)

Why it does not work with Ends?
or-tools version: 6.10.6025

Ran into the same issue. @smagafurov did you find an answer to your question?
In my case I am trying to a cost to a vehicle "working" during overtime, and the way I thought to do that is to set a soft upper bound on the time dimension of the end node

@smagafurov did you find an answer to your question?

No.
My workaround to apply SetCumulVarSoftUpperBound on orders instead of Ends solves my task.
You can try this workaround too.

Thanks @smagafurov , I'm afraid that doesn't work for my use case.
However, I noticed I am using release 6.10 of ortools which has a method called SetEndCumulVarSoftUpperBound which does the trick. This method was removed in the later versions and judging by that commit SetCumulVarSoftUpperBound should now also work for Ends.

Closing now. Reopen if it does not work with 7.3


Add workspace service as an optional authorization source

Requires an offline DB update.

@briehl some small spec updates from the previous version I showed you.

Looks okay to me.  I'll check out the spec change later tonight. But it should be okay.


Introduce Domain-scoped event-triggers

It should be possible to define event triggers which can only affect entities within the same domain.
This will have important performance implications, improves correctness and security.

This issue is blocked by #2062


Best way to handle large sets of tiles?

I'm writing a top-down shooter using a tile-based map system. Each tile is its own body, which makes it incredibly slow when I've got 64 tiles by 64 tiles on a single map. I've been looking at Matter-JS docs for awhile, and I haven't found many in-house solutions to this. My best guess so far is that I should convert all the tiles to a single body using fromVertices(...).
Is this a good idea? Or is there something more efficient or easier I can explore, like a mergeBodies(...) function?
Thanks

My best guess so far is that I should convert all the tiles to a single body using fromVertices(...).

Probably not a good idea, in this case it's better to use multiple bodies so that the broad-phase can be more efficient.
I'm interested to know what exactly is slow, since I assume the tile bodies are static? How many dynamic bodies do you have?


tags are not been generated

Hi,
I'm trying to use latest version of SEO, but I don't get the tags in my html
I do see the tab title changing but view source does not show the title tag + when I share the link on whatsapp I do not see the title
here is my code:
<SEO
title={addr}
description={addr}
siteUrl={window.location.href}
image={{
src: 'https://myurl/images/sold-logo.png'
}}
/>

Can you share the markup that is generated?

its a single page application so the markup is the react standard markup - html that links to main.js

The tab title do change

My ContactUs component looks like that:
return (
    <Panel title="צור קשר">
      <SEO
        title="סולד טכנולוגיות - צור קשר"
        description="צור קשר"
        siteUrl={window.location.href}
      />

When I use google dev tools to inspect, I do see the generated meta tags
<title>צור קשר</title>
<meta name="description" content="צור קשר" data-react-helmet="true">

but when I share the link via whatsapp/facebook I get some old default title.
Is there a cache on whatsapp/facebook? how can I refresh it?
Thanks
Dekel

You might try triggering a new crawl with Facebook debugger https://developers.facebook.com/tools/debug/

Cool!
According to the debugger, og:description and og:title are missing
BUT I do see it when I inspect the page via google dev tool bar.
[image: image.png]
On Sun, Aug 9, 2020 at 2:13 AM Jonathan Golden<EMAIL_ADDRESS>wrote:

You might try triggering a new crawl with Facebook debugger
https://developers.facebook.com/tools/debug/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/americanexpress/react-seo/issues/22#issuecomment-670983686,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZ3NNCHQD37ZHBCWCX7OPDR7XLZDANCNFSM4OWPLCMQ
.

Anyone?

Hey @dyaacov if you can see the tags when inspecting the element but when you go to view source you can't see them it might be to do with your app not doing server side rendering. I think when React runs in the client it renders the additional tags as a SPA would do, but if you are not doing server side rendering, the tags will not be visible in the original html before the javascript runs, hope that helps? 👍

Hi,
I knew that... I thought react-seo brings some new solution to the SPA app
thanks anyway
Dekel
On Thu, Aug 20, 2020 at 5:12 PM Ruben Casas<EMAIL_ADDRESS>wrote:

Hey @dyaacov https://github.com/dyaacov if you can see the tags when
inspecting the element but when you go to view source you can't see them it
might be to do with your app not doing server side rendering. I think when
React runs in the client it renders the additional tags as a SPA would do,
but if you are not doing server side rendering, the tags will not be
visible in the original html before the javascript runs, hope that helps?
👍
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/americanexpress/react-seo/issues/22#issuecomment-677690759,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZ3NNFJ6CYQ2WMQRNXQ7C3SBUVLLANCNFSM4OWPLCMQ
.


Only require gil for pyproj_log_function when needed

[x] Fully documented, including history.rst for all changes and api/*.rst for new API

Minor improvement:

Hmmm, nevermind. Does not seem to be consistent. This change may or may not be beneficial.

As long as the function is not called from a function releasing the GIL, this change can in principle have no effect I think. Declaring a function with 'nogil' is the prerequisite to use that function in a with nogil block, but you still need to release the GIL somewhere with such a block


Controlling PHP-ext log verbosity

Following the install instructions for the GRPC PHP-ext via Pecl, on Linux environments with a glibc < 2.17 I get the following info log:
env_linux.c:77] Warning: insecure environment read function 'getenv' used

Default log verbosity should have been changed to ERROR in > 1.0.0 according to https://github.com/grpc/grpc/pull/7438. Setting the GRPC_VERBOSITY environment variable doesn't seem to have an impact with the compiled Pecl extension.
Related to https://github.com/grpc/grpc/issues/8104 and https://github.com/grpc/grpc/issues/7620.

I'm seeing this same issue with the Node library.

Are there any news regarding this issue?

any chance that 79764a3bfbdec89b93b2f4a52b73e4b2ec275226 is supposed to fix this just @ncteisen forgot to link this issue?
based on that commit, this should be fixed in grpc 1.6.0-pre1 unfortunalley the 1.6.0 version doesn't have a stable release on https://pecl.php.net/package/grpc

Yep, that should fix it! (It's from pull req #11770
https://github.com/grpc/grpc/pull/11770).
Could you verify that by running at head? Or if that is not easy to do,
will you verify the fix once we release a version with the fix included?
On Mon, Sep 18, 2017 at 10:45 AM, Ferenc Kovacs<EMAIL_ADDRESS>wrote:

any chance that 79764a3
https://github.com/grpc/grpc/commit/79764a3bfbdec89b93b2f4a52b73e4b2ec275226
is supposed to fix this just @ncteisen https://github.com/ncteisen
forgot to link this issue?
based on that commit, this should be fixed in grpc 1.6.0-pre1
unfortunalley the 1.6.0 version doesn't have a stable release on
https://pecl.php.net/package/grpc
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/issues/9509#issuecomment-330246057, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AF3QLFDbWYk4py1OWWCEt8m0J-CdIlbYks5sjoHugaJpZM4Lx0d_
.

I would prefer to have an 1.6.0(or 1.6.1) stable on pecl, but I can compile from source (but a bit messy because you package different config.m4 for pecl what you have under src/php/ext/grpc/config.m4)

Let me know if you verify the fix from source. Thanks!
On Mon, Sep 18, 2017 at 11:09 AM, Ferenc Kovacs<EMAIL_ADDRESS>wrote:

I would prefer to have an 1.6.0(or 1.6.1) stable on pecl, but I can
compile from source (but a bit messy because you package different
config.m4 for pecl what you have under src/php/ext/grpc/config.m4)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/issues/9509#issuecomment-330254142, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AF3QLFQs48yT2_IXQN_2qmRiE3ylqxZnks5sjoe_gaJpZM4Lx0d_
.

with 1.6.0 stable I don't get the "Warning: insecure environment read function 'getenv' used" message

Great! Let's close this once a stable version with my fix is in

@stanley-cheung made a stable pecl release for 1.6.0 six days ago, I was testing with that, so you should be fine to close this right now.


Add modern js cheatsheet

What does this PR do?
Add resource(s)
For resources
Description
modern js cheatsheet
Why is this valuable (or not)?
because it is modern .
How do we know it's really free?
that repo
For book lists, is it a book? For course lists, is it a course? etc.
its cheatsheet
Checklist:

[ ] Read our contributing guidelines
[ ] Search for duplicates.
[ ] Include author(s) and platform where appropriate.
[ ] Put lists in alphabetical order, correct spacing.
[ ] Add needed indications (PDF, access notes, under construction)

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Duplicate of #6078


5.6.0-beta.3 - no download packages even though it was tagged 5 days ago

As the title says, its been 5 days since the tag. The previous beta release was back in June. It would be good to have more active releases.

@vraravam Thank you for letting us know, I have synced with @vantezzen on the issue and deleted the pre-release for now. We are taking care of a couple of issues and will come back with a new release soon enough 💪


Update gamelift.cpp

 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.

What's the purpose of this PR?


[BUG] : 2,3,4,5K , Multiple Kills not working

https://github.com/5stackgg/api/commit/db9e57d9eadcde748bf4b2eecc5f25c685492da2


Tutorial broken due to HELM version / NGINX Version

This tutorial (like all the others for NGINX ingress with AKS) is broken.
There is a mismatch in the digests and the NGINX Version is not compatible with the chart.

Digest
it´s necessary to add:
--set controller.admissionWebhooks.patch.image.digest=""
--set defaultBackend.image.digest=""

Controller image
to fix the nginx either helm version 3.36.0 must be specified
or
the nginx image v1.0.0 must be used.

Please fix

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

ID: 1bf9c2f3-caeb-0fb1-f089-3f7ca4fad729
Version Independent ID: 929253d7-c763-6429-561f-529d39b5284d
Content: Ingress controller on internal network - Azure Kubernetes Service
Content Source: articles/aks/ingress-internal-ip.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

Closing as dulpicate #81350


Enable/Disable return of background to Streamlit

Right now backgroundColor is returned to Streamlit but not backgroundImage
Add a prop to choose to return or not backgroundColor/backgroundImage

...for now I suppose one can merge the background image with the JSON data on the Python part so closing for now


Fixes #58

Changed line 12 of Dockerfile from:
RUN npm install --quiet
to
RUN yarn install --no-progress
Tested Docker compose command from README.md with no issues.

This is ready to go :)
Dockerfile composes faster and some prompts have been removed to avoid any confusion.


[Bug]: Most windows apps

What happened?
Where has the Settings app gone? Where is the Microsoft Store? Where is the File Explorer??????
When clicking those apps it just sends you to the rickroll page.
PLEASE BLUEEDGE FIX THIS!!!!!!!!!!!!!!!!!!!
To reproduce the behavior?
First step: Remove the Rickrolls.
2nd step: ReAdd the apps that are missing.
3rd step: Remove the BSOD.
What browser are you seeing the problem on?
Microsoft Edge

It's april fools day

Fixed


Add info on the Windows Azure CRP Certificate Generator

Product Backlog Item 5516322: [axelg] Document multiple "Windows Azure CRP Certificate Generator" certificates in the CSS Wiki + in a public doc -<PHONE_NUMBER>00283

@axelgMS : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hello Akshay,
I submitted a doc change in the https://docs.microsoft.com/en-us/azure/virtual-machines/extensions/features-windows - to describe the “Windows Azure CRP Certificate Generator” certificates.
Goal is to deflect cases and have a public note describing these.
--> can you please review & approve https://github.com/MicrosoftDocs/azure-docs/pull/41136?
-Axel
From: PRMerger14<EMAIL_ADDRESS>Sent: Monday, October 21, 2019 4:33 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Axel Guerrier<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Add info on the Windows Azure CRP Certificate Generator (#41136)
@axelgMShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FaxelgMS&data=02|01|Axel.Guerrier%40microsoft.com|fe743bb8578d43ca1d1c08d75633890b|72f988bf86f141af91ab2d7cd011db47|1|0|637072651667027990&sdata=6F%2Ba9X5ZkVl3k4QeW62sGTh09ogt6rPNHr%2B8ljCAtNA%3D&reserved=0 : Thanks for your contribution! The author(s) have been notified to review your proposed change.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fpull%2F41136%3Femail_source%3Dnotifications%26email_token%3DACPXLYC4GHKA7Z27MFIQA2DQPW4QXA5CNFSM4JC6FQT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEB2Q4WQ%23issuecomment-544542298&data=02|01|Axel.Guerrier%40microsoft.com|fe743bb8578d43ca1d1c08d75633890b|72f988bf86f141af91ab2d7cd011db47|1|0|637072651667037987&sdata=q6v17HhiNttzX9VOPXsSu4uSuqXqT3w4Wd89Z1KORmU%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACPXLYD46KQ4WUHKESR42L3QPW4QXANCNFSM4JC6FQTQ&data=02|01|Axel.Guerrier%40microsoft.com|fe743bb8578d43ca1d1c08d75633890b|72f988bf86f141af91ab2d7cd011db47|1|0|637072651667037987&sdata=s29tw2z2rrShYsUbcwwodR03c5cAtqYuZvwXqGQl3X8%3D&reserved=0.

@axayjo - Could you review the proposed changes? Thanks.

@Akshay<EMAIL_ADDRESS>gentle ping - can you please review and approve that PR ?
From: Axel Guerrier
Sent: Wednesday, October 30, 2019 2:59 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Akshay Joshi<EMAIL_ADDRESS>Cc: Mention<EMAIL_ADDRESS>Axel Guerrier<EMAIL_ADDRESS>Subject: RE: [MicrosoftDocs/azure-docs] Add info on the Windows Azure CRP Certificate Generator (#41136)
Hello Akshay,
I submitted a doc change in the https://docs.microsoft.com/en-us/azure/virtual-machines/extensions/features-windows - to describe the “Windows Azure CRP Certificate Generator” certificates.
Goal is to deflect cases and have a public note describing these.
--> can you please review & approve https://github.com/MicrosoftDocs/azure-docs/pull/41136?
-Axel
From: PRMerger14 <notifications<EMAIL_ADDRESS>Sent: Monday, October 21, 2019 4:33 PM
To: MicrosoftDocs/azure-docs <azure-docs<EMAIL_ADDRESS>Cc: Axel Guerrier <Axel.Guerrier<EMAIL_ADDRESS>Mention <mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Add info on the Windows Azure CRP Certificate Generator (#41136)
@axelgMShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FaxelgMS&data=02|01|Axel.Guerrier%40microsoft.com|fe743bb8578d43ca1d1c08d75633890b|72f988bf86f141af91ab2d7cd011db47|1|0|637072651667027990&sdata=6F%2Ba9X5ZkVl3k4QeW62sGTh09ogt6rPNHr%2B8ljCAtNA%3D&reserved=0 : Thanks for your contribution! The author(s) have been notified to review your proposed change.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fpull%2F41136%3Femail_source%3Dnotifications%26email_token%3DACPXLYC4GHKA7Z27MFIQA2DQPW4QXA5CNFSM4JC6FQT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEB2Q4WQ%23issuecomment-544542298&data=02|01|Axel.Guerrier%40microsoft.com|fe743bb8578d43ca1d1c08d75633890b|72f988bf86f141af91ab2d7cd011db47|1|0|637072651667037987&sdata=q6v17HhiNttzX9VOPXsSu4uSuqXqT3w4Wd89Z1KORmU%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACPXLYD46KQ4WUHKESR42L3QPW4QXANCNFSM4JC6FQTQ&data=02|01|Axel.Guerrier%40microsoft.com|fe743bb8578d43ca1d1c08d75633890b|72f988bf86f141af91ab2d7cd011db47|1|0|637072651667037987&sdata=s29tw2z2rrShYsUbcwwodR03c5cAtqYuZvwXqGQl3X8%3D&reserved=0.


Update modules.py

Seems like just a small copy/paste typo where the pitch controller was being used instead of the energy controller

PR https://github.com/ming024/FastSpeech2/pull/102 actually finds another place where this error occurs so I will close this one.


Add support for inserts via GCP cloud function and pub/sub

Added support for handle_tests_results and insert_rows to use a new insert_rows_method method called gcp-cloud-function which calls a UDF to push results into BigQuery via Pub/Sub rather than direct insert queries. This significantly increases Elementary's capacity to insert records/test results when using BigQuery.
Here is an example configuration in dbt_project.yml. You identify a UDF that publishes to a topic (first argument of SELECT), and then define 3 pubsub topics to send data to. Those topics are defined to pass records straight to BigQuery, which is a simple option. I did also define schemas for those tables (all of this is created in Terraform) which is not ideal, it means this is coupled to schema/code changes for those tables.
  # This is to prevent BQ from exceeding the query size limit, specifically used to prevent errors writing Elementary metadata.
  insert_rows_method: gcp-cloud-function
  insert_rows_udf: "_project_._dataset_.publish_to_pubsub_function"
  insert_rows_topics: {
    "data_monitoring_metrics": "projects/_project_/topics/elementary-monitoring-metrics-topic",
    "test_result_rows": "projects/_project_/topics/elementary-test-result-rows-topic",
    "elementary_test_results": "projects/_project_/topics/elementary-elementary-test-results-topic"
  }
  query_max_size: 100000

This is an example for publish_to_pubsub_function:
CREATE OR REPLACE FUNCTION project.dataset.publish_to_pubsub_function(pubsub_topic STRING, json_data STRING, attributes STRING) RETURNS STRING REMOTE WITH CONNECTION .... OPTIONS (endpoint = ......, max_batching_rows = 500);

HI @jordanwillifordcruise !
Thanks a lot for this contribution and sorry for the delay in our response.
This is pretty cool! I am a bit worried that this setup would be a bit too complicated for most users, though maybe it's fine as an advanced flag.
It would help though if you can provide instructions on how to set this up, ideally without terraform (I'm fine with manual commands in the GCP UI) - so we can reproduce + add to our docs.
Also - we recently merged another contribution that was related to BQ insert_rows issues. Would that help your use case?
Thanks,
Itamar

HI @jordanwillifordcruise ! Thanks a lot for this contribution and sorry for the delay in our response.
This is pretty cool! I am a bit worried that this setup would be a bit too complicated for most users, though maybe it's fine as an advanced flag. It would help though if you can provide instructions on how to set this up, ideally without terraform (I'm fine with manual commands in the GCP UI) - so we can reproduce + add to our docs.
Also - we recently merged another contribution that was related to BQ insert_rows issues. Would that help your use case?
Thanks, Itamar

I totally agree that the setup is pretty complicated.  I will try to add some more content around infra setup without Terraform.
I am happy to see that chunk PR, and we have seen that issue too, but the one that this PR fixes is related to the number of small writes to a BQ table in a day.


empty blueprint must create empty filter iterator

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.
@geirst please review and merge
@havardpe FYI

Build failure is unrelated: [INFO] node-admin ......................................... FAILURE [ 38.055 s]