1.30.0 error

The error is as follows
使用pnpm，报错如下
><EMAIL_ADDRESS>docs:dev
> vitepress dev docs

failed to load config from /Users/dsh/files.localized/code/book-doc/src/apps/vitepress/docs/.vitepress/config.mts
failed to start server. error:
Cannot find module<EMAIL_ADDRESS>imported from<EMAIL_ADDRESS>Error [ERR_MODULE_NOT_FOUND]: Cannot find module<EMAIL_ADDRESS>imported from<EMAIL_ADDRESS>    at finalizeResolution (node:internal/modules/esm/resolve:257:11)
    at moduleResolve (node:internal/modules/esm/resolve:913:10)
    at defaultResolve (node:internal/modules/esm/resolve:1037:11)
    at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:650:12)
    at #cachedDefaultResolve (node:internal/modules/esm/loader:599:25)
    at ModuleLoader.resolve (node:internal/modules/esm/loader:582:38)
    at ModuleLoader.getModuleJobForImport (node:internal/modules/esm/loader:241:38)
    at ModuleJob._link (node:internal/modules/esm/module_job:132:49)

😂 测试出来了，跟pnpm没关系。
作者你给改坏了啊，上一个版本 1.29.0 还是好的，你对他做了什么啊。

你是不是提取类型声明文件的时候，向外暴漏的错了

Hello, I apologize for the inconvenience caused by the recent changes. It was my mistake for not checking it in depth after deployment.
This issue was fixed in version 1.30.2, and upgrading the package to that version or higher should fix the issue.
Thank you for reporting the issue.

您好，对于最近的更改给您带来的不便，我深表歉意。这是我的错误，在部署后没有深入检查。
该问题已在版本中修复1.30.2，将软件包升级到该版本或更高版本应该可以解决该问题。
感谢您报告此问题。

Haha, I actually wrote a sidebar generation plugin for vuepress a long time ago. But it is no longer maintained. Anyway, applaud you for making vitepress more useful.

Hello, I apologize for the inconvenience caused by the recent changes. It was my mistake for not checking it in depth after deployment.
This issue was fixed in version 1.30.2, and upgrading the package to that version or higher should fix the issue.
Thank you for reporting the issue.

Haha, I actually wrote a sidebar generation plugin for vuepress a long time ago. But it is no longer maintained. Anyway, applaud you for making vitepress more useful.


Refactor and removed deprecated methods

Replace this paragraph with a description of what this PR is changing or adding, and why. Consider including before/after screenshots.
Every PR must update the corresponding documentation in the code, and also the readme in english with the following changes.
Pre-launch Checklist

[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making or feature I am adding, or @jonataslaw said the PR is test-exempt.
[ ] All existing and new tests are passing.

Thanks, kindly release this so we can benefit, because currently cant use this latest code.

Thank you for your contribution!

Could you please publish these changes? Developers face numerous challenges, and many deprecated methods were removed in Flutter 3.22.0. This candidate version is working well, so it is ready for publication.


Vorkath Poison Timer

Like the chinchompa plugin , just on Vorkath or around it just when the poison splat special starts , so you have an idea how long is left.

https://github.com/runelite/runelite/pull/3903
Just found , apologies


Allowing granular access for data-stream related transport actions

Signed-off-by: Sandesh Kumar<EMAIL_ADDRESS>opendistro-for-elasticsearch/security pull request intake form

Category: New feature

Github Issue #1168

Description of changes: Datastream support for OS is in progress: https://github.com/opensearch-project/OpenSearch/issues/675
With this PR, we want to ensure that granular access for data stream related transport actions can be provided from security plugin.
This PR has a dependency on the above OS issue to be resolved and the linked PR to be merged to OS.

The 4 transport actions that we use to access for data stream related operations in OpenSearch:
(1).  indices:admin/data_stream/create
(2). indices:admin/data_stream/get
(3). indices:admin/data_stream/delete
(4). indices:monitor/data_stream/stats

Fixed the index resolution for (1)
Added unit tests for all 4 transport actions.

Why these changes are required?
To enable granular access for above mentioned transport actions by security plugin.

What is the old behavior before changes and new behavior after changes?
Transport action (1) does not resolves correctly. (Details in github issue)

Testing done:

Unit tests pass now.
Tested the transport action behavior with users by giving them multiple index-patterns in permission: 'pattern*', 'pattern', '*'. Under all these permissions, the data stream creation happens as per the index pattern specified.

TO-DOs, if any: No

Is it backport from main branch? No

By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or
(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or
(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.
(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Please fix the failing CI

The core artifacts are ready in -rc1 and were required for my changes to build and CI to pass. Therefore, changing adding the version_qualifier as discussed in https://github.com/opensearch-project/security/pull/1174
Also, removed the -beta1 from OpenSearch version in pom.xml

Codecov Report

Merging #1170 (2db7bed) into main (5a81a42) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main    #1170      +/-   ##
============================================
+ Coverage     64.57%   64.59%   +0.01%
- Complexity     3172     3175       +3
============================================
  Files           244      244
  Lines         17123    17125       +2
  Branches       3035     3036       +1
============================================
+ Hits          11058    11062       +4
+ Misses         4520     4519       -1
+ Partials       1545     1544       -1

Impacted Files
Coverage Δ
Complexity Δ

...earch/security/resolver/IndexResolverReplacer.java
63.63% <100.00%> (+0.21%)
59.00 <0.00> (+1.00)

.../dlic/auth/ldap2/LDAPConnectionFactoryFactory.java
58.01% <0.00%> (+1.52%)
24.00% <0.00%> (+1.00%)

Continue to review full report at Codecov.

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

The core artifacts are ready in -rc1 and were required for my changes to build and CI to pass. Therefore, adding the version_qualifier as discussed in #1174
Also, removed the -beta1 from OpenSearch version in pom.xml

We’ll have the CI changes in the other PR: https://github.com/opensearch-project/security/pull/1179. Please wait on #1179 to be merged.

The core artifacts are ready in -rc1 and were required for my changes to build and CI to pass. Therefore, adding the version_qualifier as discussed in #1174
Also, removed the -beta1 from OpenSearch version in pom.xml

We’ll have the CI changes in the other PR: #1179. Please wait on #1179 to be merged.

Thanks Chang for the response!
I will re-base from main branch once #1179 is merged. That will remove any CI related changes in this PR.

#1179 is merged.
Merged from the main branch to get the required CI related changes.
No CI related changes are part of this PR now.


Tunnel doesn't start, no feedback

Hi Phil! Thanks for making this extension available -- I'm a fan of ngrok and VSC, so putting'em together seems like a great idea.
I just installed the extension and I'm afraid when I run ngrok: start I get absolutely no indication of anything happening. The command palette just goes away and that's it.
Versions:

VSC: 1.44.1
ngrok: 2.3.35
ngrok for VSCode: 1.1.0

I do have a tunnel defined in my ngrok config file and have given the extension the path to the config file. I've confirmed that I can run the tunnel from the command line. I do get error messages if I put a malformed path to the config file. Running ngrok: dashboard from the command palette gives me an error that ngrok isn't running.
Any ideas?

Hi @chuck-wood, I'm guessing you're on Windows or Linux? I think that I've messed up and bundled the Mac binary for ngrok with the extension so it only works on Macs right now. I've raised issue #7 to cover this.
Hopefully we can git this sorted shortly, thanks for trying out the extension!

Windows indeed, sorry for leaving that out. Thanks for the quick reply!

Could you try installing and using version 1.2.0 please? It should be working now.

Okay, I updated by reloading VSCode. I ran ngrok: start and it popped up a toast saying that it was downloading the ngrok binary (on a side note, I already have it installed so it would be cool if there was an option to point at an existing binary). After a bit I ran ngrok: start again, it prompted me to select a profile, and it ran successfully. Thanks for the quick turnaround!

Thanks for testing @chuck-wood, glad it works!
The binary download is just how the ngrok npm module works right now. I agree, it's worth seeing if you can point it at an existing installation, but this was definitely the most straightforward way to get it to work to start with.
Let me know if you have any other feature requests!

Ah I see. No worries, it's great to make use of tools to build things quickly. Cheers!


Pass base_dir to model files can be loaded for auto-tp/meta-tensor.

This PR will enable auto-tp and meta-tensor feature to work with all models including Llama2.
Usage example with https://github.com/microsoft/DeepSpeedExamples/tree/master/inference/huggingface/text-generation
Run Command:
deepspeed --num_gpus=2 inference-test.py --model meta-llama/Llama-2-7b-hf --use_meta_tensor --checkpoint_path=/home/user/.cache/huggingface/hub/models--meta-llama--Llama-2-7b-hf/snapshots/6fdf2e60f86ff2481f2241aaee459f85b5b0bbb9/

FYI. @lekurile @molly-smith @mrwyattii
This is connected to the recently merged #4147 that fixed AutoTP with Meta Tensor support.


Fast repeated mouse clicks not registered in Chrome

When clicking repeatedly and fast enough on a link the app is not responding (i.e. no trigger of OnLocationChange). Using keyboard to "click" the links works as expected.
The Example.elm app has this problem (tested in Chrome on Mac OS).

Investigated this and it seems this is a problem in plain javascript implementation (see gist) using window.onhashchange also so maybe close this issue. Would be nice if someone had any workaround/info on it. Is it a "security" feature in chrome to prevent links changing while user is about to click them maybe?

Based on what you are saying, I am going to assume that the behavior is the same in JS, without Elm involved at all. In that case, this issues should be closed.
I don't have any particular insight on the browser behavior unfortunately.
If you find out what the root cause is, and find that other projects protect against it in some way, please open a new issue explaining the full situation!


Lwrp

changed definition for cpan_module to a lwrp.

We now have a fully idempotent LWRP included in the cookbook


Query editor should not send whitespace characters to engine for execution

Azure Data Studio Version: master

Steps to Reproduce:

Type a query such as SELECT * FROM sys.tables;
Run the query multiple times

Expected: The results are returned every time
Actual: Sometime the query results aren't returned an the the messages say "0 results returned"
Note: This doesn't repro consistently but I've seen it occur a few times in ~10 minutes of testing.  This appears to be a regression from the query streaming changes since it only recently started happening.

This may have been related to running a query with blank lines selected, which now has the behavior outlined above.  I think this is still a regression, but less serious than losing resultsets.

I can't seem to repro this.

Actually this doesn't look like a regression from Nov, though I thought we did have this behavior at some point.
The repro is something like the following.

Type a query with some SQL and some whitespace, such as below (note the space characaters.

SELECT * FROM sys.tables;

Select the SQL content and F5

Select the space characters and F5

Expected: In step 3 there is no action since only whitespace is highlighted
Actual: Results from step 2 go away and the space characters are passed to engine for execution.
Note: This is same behavior as SSMS, so I'll update the title and move to backlog as suggestion for consideration.

Closing older issues.


feat: get files for a session, skip files with wrong extension in uploads

Makes understanding why uploads failed simplre
Proposed Changes?

How did you test it?

Notes for the reviewer

Screenshots (optional)

Checklist

[ ] I have updated the referenced issue with new insights and changes
[ ] If this is a code change, I have added unit tests
[ ] I've used the conventional commit specification for my PR title
[ ] I updated the docstrings
[ ] If this is a code change, I added meaningful logs and prepared Datadog visualizations and alerts

Stale: closing


chore(upgrade): v1.6.0 to v1.7.0

Upgrade Polkadot-sdk to v.1.7.0.
Update weights to reflect the new version.

Notable Changes:

Allow custom error types in Jsonrpsee

For more details, please refer to:
Release Notes
#1870

Looks great!
I think there might be some questions about how this back-pressure implementation which supports 65 messages per connection by default might have some effects on our apps subscribing to the chain.

Thats a good point. Ill create an issue so that we can properly configure backpressure.


Remove tidyverse loading message

Possibly best in the _quarto.yml add message: true as well as echo and eval
https://github.com/nhs-r-community/health-inequalities-notebook/blob/e1f5ca7e39b911d107129926e781ef206b5e0a68/content/imd.qmd#L91

Can't put the execution code in the _quarto.yml and the IMD particularly needs a lot of the echo and eval removed as not necessary. The default is eval: true and echo: true.
If #| warning: false is put in the first chunk with library(tidyverse) this applies to the rest of the .qmd and doesn't need repeating in the later R chunk.

Good point! I used {tidyverse} because it is recognisable to very new users. If this were a package I've recently found https://ropensci.org/blog/2019/12/08/precompute-vignettes/ which could help with showing familiar code but without having to depend upon a series of other packages that it's only the vignette that needs.


Precompute S2K message schedules for short password-lengths

From<EMAIL_ADDRESS>on July 14, 2014 00:28:23
e2e.openpgp.IteratedS2K.getKey is slow for large c. Really really slow. 25 seconds for SHA2-512 at c=255.
Something easy to do for, e.g., 64-byte blocksize hashes:
For passwords < 56 bytes, create an array of all the blocksize-filling salted_passphrase rotations; then absorb these blockwise until the bytecount condition is met. goog.Crypt.Sha1 has a fast-path for blocksize calls: 25% speedup in this case. Don't think that goog.Crypt.Sha2 or Sha2_64bit do; but probably better, in any event, to import the code and use computeChunk_ directly for the repeated steps.
Even better:
Precompute message schedules for all rotations; then do as above, but only applying the MD update steps. Relative workfactor is from .57 to .66 for SHA instances at c=255. Cost breakdown attached.
(The bitops numbers are irrelevant here; the "p(rocessor)ops" numbers are relevant, but the operation costing may not be right for V8 -- the model I drew these numbers from was intended for (and verified on) a specific processor.)
Attachment: s2k.markdown
Original issue: http://code.google.com/p/end-to-end/issues/detail?id=113

Any updates on this?

Status of Closure?
On Wed, Jun 10, 2015 at 10:21 AM Krzysztof Kotowicz <
<EMAIL_ADDRESS>wrote:

Any updates on this?
—
Reply to this email directly or view it on GitHub
https://github.com/google/end-to-end/issues/150#issuecomment-110793413.

coruus, can you send this as a pull request?
https://github.com/coruus/e2e/commit/909622dcfef2fbf0bf4aa047d813f7dfd1527e44
not sure if it's ready or you had something pending


ci: fix failing tests

Fixed examples:

[x] examples/rust-wasm-cross
[x] examples/phoenix
[x] examples/python-django
[ ] examples/opentelemety-collector
~won't launch on macos-13. firewall issue? Tried disabling firewall, still nada.~
the readiness_probe on clickhouse is broken on macos13. curl-ing against the http port work. Only happens in GHA 🤷 clickhouse-client works as expected.
[x] examples/supported-languages
odin needs nixpkgs bump for darwin fixes.
~unison-ucm doesn't have aarch64-linux builds.~
[ ] examples/ruby-on-rails: flaky on macos. timeouts too short?

Fixed languages:

[x] languages/odin made gdb and ols optional depending on system.

Broken tests:

[x] devenv-test-cli
[x] devenv test
[ ] tests/mysql fails on macos

Other changes:

Changed .setup.sh to run inside the devenv shell. This helps test manual user setup, like initializing a project and installing dependencies.
Implement .patch.sh script support in devenv-run-tests. This runs outside of the shell, before .setup.sh. This should be used to patch files, like devenv.nix.
Replaced uses of example.com with localhost.

thinking about it some more, the .patch.sh is more of a workaround for whatever the underlying issue is. Why does running the setup stuff break in enterShell, but not when run before the processes are launched? Race condition?

Alright, I'm out of steam. The last two failures are on macOS.

examples/opentelemetry-collector — the clickhouse healthcheck fails to go green on the hosted x86 runner. I used tmate to run it manually, and as a separate process, and it works correctly in those situations. The http health check works as expected as well. Super weird!
tests/mysql — fails on our self-hosted macos runner. There's a warning about not being able to resolve the hostname. Haven't looked into it too much.

Needs #1263.


Show the roster before asking user to login.

ALMOST READY FOR REVIEW (keeping notes)
Note: these changes are only visible on dev
New:

clicking "Next" from the splash page goes directly to the roster
the invite FAB will always be visible
clicking on the FAB opens a page that looks like the existing login page
clicking on a network from the new "invite-user" page will prompt the user to login with that social network
depending on the network, after login, the user will get further instructions for inviting a friend
"Log out" from the settings panel will log you out of networks, but does not move you back to the splash screen
if you choose to start copypaste, you will be logged out of all other social networks first
if the user "hasSeenWelcome" (they've clicked through the welcome messages), they will not see the splash screen again; opening uProxy will always open the roster
the option to enter an invite has been moved from the network specific invite page (e.g. the red page to invite a Gmail contact) to the generic invite-user page

TODOs:

Update website screenshots, instructions and map from social network name to social network display name (will do when design for this PR is confirmed!)

Mocks:

Screenshots:

Sign in dialog for public (social1) networks:

Sign in dialog for invite (social2) networks:

What settings looks like if you're not logged in:

Proposal:
I think one-time connection can be combined with social networks:

Thoughts?

A few more things left to internationalize, but this is ready for review!

There's a lot of variable initialization taking place within the constructor functions here, any reason for that?

Reviewed 4 of 11 files at r1, 9 of 10 files at r2, 1 of 1 files at r3.
Review status: all files reviewed at latest revision, 31 unresolved discussions, some commit checks failed.

src/generic_ui/polymer/contact.html, line 132 [r2] (raw file):
Was this fixed? (also, it seems to me that it would make sense for pending requests to appear as offline)

src/generic_ui/polymer/invite-user.html, line 20 [r3] (raw file):
What are you overriding with the !important?

src/generic_ui/polymer/invite-user.html, line 46 [r3] (raw file):
This (and kind of the size of the file) makes my brain scream "separate element"

src/generic_ui/polymer/invite-user.html, line 153 [r3] (raw file):
ping on this (and the following similar notes)

src/generic_ui/polymer/invite-user.html, line 192 [r3] (raw file):
?

src/generic_ui/polymer/invite-user.html, line 272 [r3] (raw file):
It seems like it would be easier to just have a single template that uses selectedNetworkName instead of n, would their be a problem with that?

src/generic_ui/polymer/invite-user.ts, line 10 [r3] (raw file):
I think I saw this function already...

src/generic_ui/polymer/invite-user.ts, line 16 [r3] (raw file):
 :string

src/generic_ui/polymer/invite-user.ts, line 22 [r3] (raw file):
Can we do a better type than any here?

src/generic_ui/polymer/invite-user.ts, line 29 [r3] (raw file):
This looks incredibly strange with the indentation, can you either move core.sendEmail to the previous line or indent it?

src/generic_ui/polymer/invite-user.ts, line 56 [r3] (raw file):
Can you fill in the todo?

src/generic_ui/polymer/invite-user.ts, line 123 [r3] (raw file):
Why the empty line?

src/generic_ui/polymer/invite-user.ts, line 127 [r3] (raw file):
I think the norm for javascript is to use for (var i in model.onlineNetworks) {, any reason you did it this way? (edit: Ben pointed out it may be possible to use for of loops even with our build target now, maybe try that?)

src/generic_ui/polymer/invite-user.ts, line 143 [r3] (raw file):
This seems like it would be better to have initInviteForNetwork handle logging in if you are not logged in (or just doing the invite if you are).

src/generic_ui/polymer/invite-user.ts, line 180 [r3] (raw file):
I think this is duplicated in settings.ts, can you check that?

src/generic_ui/polymer/invite-user.ts, line 211 [r3] (raw file):
This feels like the kind of hack we should be solving by using build flags.  I realize that we do not have a good way to do build flags right now, and am fine with this because of that, but wanted to note it.

src/generic_ui/polymer/invite-user.ts, line 213 [r3] (raw file):
For this and the following functions, can you change the argument name to networkName?  That would make it a bit more clear as to what is happening.

src/generic_ui/polymer/invite-user.ts, line 223 [r3] (raw file):
I'd consider leaning towards networkNamesChanged for these function names, just to make things more obvious of why the function is being called (consistent with Polymer's attributeChanged convention)

src/generic_ui/polymer/invite-user.ts, line 230 [r3] (raw file):
If this is specifically for Github, can we name it as such?

src/generic_ui/polymer/network-invite-user.html, line 41 [r3] (raw file):
Let's not use deep for our own elements, just edit uproxy-app-bar so you can supply this.

src/generic_ui/polymer/network-invite-user.html, line 48 [r3] (raw file):
What are you overriding here?

src/generic_ui/polymer/network-invite-user.html, line 61 [r3] (raw file):
What's our convention for spaces in the {{ tags?

src/generic_ui/polymer/network-invite-user.html, line 63 [r3] (raw file):
More specific ID?

src/generic_ui/polymer/network-invite-user.html, line 64 [r3] (raw file):
Spaces around comparison operator.

src/generic_ui/polymer/network-invite-user.html, line 80 [r3] (raw file):
Spaces around comparison operator.

src/generic_ui/polymer/network-invite-user.html, line 96 [r3] (raw file):
General comment on the file: It seems like thes two components in here are pretty much ind

src/generic_ui/polymer/network-invite-user.ts, line 21 [r3] (raw file):
This looks like the same code as in invite-user.ts, is there a way to remove the duplication?

src/generic_ui/polymer/network-invite-user.ts, line 46 [r3] (raw file):
Let's move this to the previous line?

src/generic_ui/scripts/ui.ts, line 210 [r3] (raw file):
Any reason for the extra line here?

src/generic_ui/scripts/ui.ts, line 1153 [r3] (raw file):
We have the same logic in another place, can you move this check somewhere else perchance?

src/generic_ui/scripts/ui.ts, line 1233 [r3] (raw file):
Same comment on loop iteration as above.
Also, it seems like this this will cause us to unnecessarily loop if we do not need to.  How about replacing this function with this.showInviteControls = this.showRosterBeforeLogin || _.some(this.model.onlineNetworks, (network) => { return this.supportsInvites(network.name); }); (with, you know, some spacing thrown in and my typos corrected)?

Comments from the review on Reviewable.io

Review status: 2 of 15 files reviewed at latest revision, 29 unresolved discussions.

src/generic_ui/polymer/contact.html, line 132 [r2] (raw file):
This fixes it.
I think we should just show what the social provider gives us: if a user is online but not a friend, the social provider should not leak the status to uproxy

src/generic_ui/polymer/invite-user.html, line 20 [r3] (raw file):
Oops! Don't think we need that

src/generic_ui/polymer/invite-user.html, line 46 [r3] (raw file):
We're going to end up deleting everything that isn't "roster-before-login", so it will only be like this in the transitional phase

src/generic_ui/polymer/invite-user.html, line 153 [r3] (raw file):
Thanks -- I've translated the ones that I think won't change in the near future. Some of the quiver stuff is still untranslated because that will probably change soon!

src/generic_ui/polymer/invite-user.html, line 192 [r3] (raw file):
Removed

src/generic_ui/polymer/invite-user.html, line 272 [r3] (raw file):
Done.

src/generic_ui/polymer/invite-user.ts, line 10 [r3] (raw file):
This is also transitional, before the roster-before-login flow is pushed to prod, we will have the old code calling this

src/generic_ui/polymer/invite-user.ts, line 16 [r3] (raw file):
Done.

src/generic_ui/polymer/invite-user.ts, line 22 [r3] (raw file):
Done.

src/generic_ui/polymer/invite-user.ts, line 29 [r3] (raw file):
Done.

src/generic_ui/polymer/invite-user.ts, line 56 [r3] (raw file):
Done.

src/generic_ui/polymer/invite-user.ts, line 123 [r3] (raw file):
Just to split the functions for the new flow (again because this is transitional). I took it out in case that's what you would prefer.

src/generic_ui/polymer/invite-user.ts, line 127 [r3] (raw file):
Done.

src/generic_ui/polymer/invite-user.ts, line 143 [r3] (raw file):
Do you mean change initInviteForNetwork to:
    if (!model.getNetwork(networkName)) {
      this.$.loginToInviteFriendDialog.open();
      return;
    }
    this.selectedNetworkName = networkName;
    if (networkName == "GMail" || networkName == "GitHub") {
      // After login for these networks, open another view which allows users
      // to invite their friends.
      this.fire('core-signal', { name: 'open-network-invite-dialog' });
    } else if (networkName == "Facebook-Firebase-V2") {
      this.sendToFacebookFriend();
    }

I don't feel that that's too different.

src/generic_ui/polymer/invite-user.ts, line 180 [r3] (raw file):
Just checked, doesn't look like it

src/generic_ui/polymer/invite-user.ts, line 213 [r3] (raw file):
Done.

src/generic_ui/polymer/invite-user.ts, line 223 [r3] (raw file):
Done.

src/generic_ui/polymer/invite-user.ts, line 230 [r3] (raw file):
Done.

src/generic_ui/polymer/network-invite-user.html, line 41 [r3] (raw file):
Done.

src/generic_ui/polymer/network-invite-user.html, line 48 [r3] (raw file):
global.css styles "body /deep/ a"

src/generic_ui/polymer/network-invite-user.html, line 61 [r3] (raw file):
When we move to Polymer 1.0 it has to be no spaces

src/generic_ui/polymer/network-invite-user.html, line 63 [r3] (raw file):
Done.

src/generic_ui/polymer/network-invite-user.html, line 64 [r3] (raw file):
Done.

src/generic_ui/polymer/network-invite-user.html, line 80 [r3] (raw file):
Done.

src/generic_ui/polymer/network-invite-user.ts, line 21 [r3] (raw file):
It will be removed from invite-user.ts once the new flow is launched (because sendToGMailFriend will only be called here)

src/generic_ui/polymer/network-invite-user.ts, line 46 [r3] (raw file):
Done.

src/generic_ui/scripts/ui.ts, line 210 [r3] (raw file):
Removed

src/generic_ui/scripts/ui.ts, line 1153 [r3] (raw file):
Since this will be removed soon (TODO above) I'd prefer to keep it simple.  I.e. in the near future this logic will only be needed in networkNamesChanged in invite-user (and not here) so there wont be a duplication problem.
(I say "simple" because I imagine the alternative is to move code from polymer (networkNamesChanged) into ui.ts, where it wont be required at all, soon)

src/generic_ui/scripts/ui.ts, line 1233 [r3] (raw file):
Done.

Comments from the review on Reviewable.io

Reviewed 13 of 13 files at r4.
Review status: all files reviewed at latest revision, 6 unresolved discussions.

src/generic_ui/polymer/invite-user.html, line 46 [r3] (raw file):
Okay, I guess that's fine for now.

src/generic_ui/polymer/invite-user.html, line 153 [r3] (raw file):
Okay, that's fine.  I think it would be good in general if we got into the habit of translating everything immediately, but that's your area.

src/generic_ui/polymer/invite-user.ts, line 10 [r3] (raw file):
Okay :(

src/generic_ui/polymer/invite-user.ts, line 123 [r3] (raw file):
Ehh, would have been fine either way.

src/generic_ui/polymer/invite-user.ts, line 143 [r3] (raw file):
Definitely not very different, however, I cannot think of a situation where you would want to call initInviteForNetwork and not have it handle the case where the user is not logged in (right now, that does not get checked at all, so I have no idea what would happen if at some later time in the future someone made the mistake of calling it for a network not logged in)

src/generic_ui/polymer/invite-user.ts, line 180 [r3] (raw file):
My mistake, the code in settings.ts (logOut), while different, attempts to do the exact same thing though.  Can we refactor them to only be in one place (FWIW, this version is better overall)

Comments from the review on Reviewable.io

One more small request - when you click "Log-out of uProxy" in the settings menu, can we have the settings menu close?  It used to go back to the splash page, which I don't think we want anymore, but now it's not so obvious what happens because the settings drawer is blocking the roster.  Thanks!

Comments from the review on Reviewable.io

Adding confirmation for logout before copypaste:

Review status: 9 of 17 files reviewed at latest revision, 2 unresolved discussions.

src/generic_ui/polymer/invite-user.ts, line 143 [r3] (raw file):
Ah I tried to change it and I realized it requires fixing a greater race condition in ui.ts. I added a TODO and github issue.

src/generic_ui/polymer/invite-user.ts, line 180 [r3] (raw file):
Done, moved to ui.ts

Comments from the review on Reviewable.io

Done, PTAL!

:thumbsup:

Reviewed 8 of 8 files at r5.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks broke.

Comments from the review on Reviewable.io


DOC: interpolate.{RegularGridInterpolator, interpn} add note explaining added dim when xi.ndim == 1

Reference issue

Follow up to #16508
What does this implement/fix?

Adds note explaining corner case where extra dimension is added to the returned array when xi.ndims ==1.
Additional information

LGTM, merged. Thanks Jake!


Loosens dependency version requirements

The strict dependency versions is causing conflicts with projects as these dependencies release new versions.
This sets the minimum versions required.

Thank you very much for contributing! Unfortunately I didn't have much time lately to maintain the project, but seeing people interested in it sparks enthusiasm a lot!


add note about wildcard support on conditions

See https://github.com/kyverno/kyverno/issues/2165 for reference.

Thanks @MarcusNoble!

Thank you for adding this, @MarcusNoble !


Add -SelectColumns parameter to Get-AzTableRow command

Adding the -SelectColumns parameter to specify which columns the query should return. This can result in significant performance benefits for tables with many columns when only a subset is needed.

@paulomarquesc - Thanks for the review, I made all the suggested changes. Let me know if there is anything further.

Just executed the unit tests and it is passing ok.
https://dev.azure.com/paulomarquesc/AzureRmStorage/_build/results?buildId=1283&view=results
I'll merge shortly.

Published on PSGallery as version 2.1.0.


Tweedie Regression Post-Rebase

re-submitting after rebase... originally in reference to issues #1392 and #681

please fix code style error in https://travis-ci.org/dmlc/xgboost/jobs/173553871

@tqchen sorry, thought I took care of that.  fixed now.

thanks, this is merged


[RFR]: Default redirect disables passing redirect to SimpleForm

The redirect is defaulted in EditController and should be passed down to SaveButton. The defaultProp in SaveButton overrides this.
<Edit>
<SimpleForm redirect={false}>
</SimpleForm>
</Edit>

This currently does not work, the redirect still is list. (I know I can pass custom toolbar).

The redirect prop of the form dictates what happens to the form when submitted with Enter. The redirect prop of the Save button dictates what happens to the form when submitted with the button.
If you override the redirect prop of the SaveButton, it's no longer possible to have two save buttons with different redirects (save=>list and save=>add).
This is properly documented, I don't see the bug.

The redirect prop of the form dictates what happens to the form when submitted with Enter.

Yeah, so this does not work, this PR fixes this behaviour.

If you override the redirect prop of the SaveButton, it's no longer possible to have two save buttons with different redirects (save=>list and save=>add).

This PR only removes the default value of redirect on the SaveButton this should only be set explicit.
SaveButton passes redirect to handleSubmitWithRedirect, but because it's defaulted to list in SaveButton this will always be list unless specified explicitly.
handleSubmitWithRedirect in SimpleForm uses this.props.redirect as default value (which is passed from EditController). As SaveButton always passes a value (because of the default redirect prop) this value will never be used.

I'm also being hit by the issue fixed by this PR (on 2.0.0-beta4). The instructions given in the docs for custom redirects do not change where the "Save" button redirects to (on edit or update views).

Thanks!


Implement aclosing without a generator

Not sure exactly why, but using a generator with @asynccontextmanager to implement it leads to a potential issues with generator didn't stop after athrow() which I can't reproduce in isolation, but appear when running many tests.
This seems to fix it  🤷
I'm thinking the issue is probably something either exiting the context aclosing context manager early, cancellation or gc of the underlying generator.

Nice, this fixes something I was just about to report :D


Use shards version in crystal init tool

Use the shards version command introduced by https://github.com/crystal-lang/shards/pull/148 to populate the VERSION constant inside generated projects with crystal init.
I moved the VERSION constant into the main "example.cr" file, instead of a version.cr file too. I guess that will be a divisive change, but I don't think a seperate version.cr gains us anything.

I'm against merging version.cr into the main module. Keeping it a separate file is IMO cleaner and would be compatible with Ruby convention which does the same (see rails' version.rb). I see no need to clutter main module with declarations having great time in their own company.

Hello @RX14, thank you for adding this to crystal init. I completely forgot about that feature and sending the PR to use it :man_facepalming:
I would personally disagree on moving the VERSION constant within the main module code. I believe it adds value beyond the file location itself, but also some orientation in relation to code structure (ie. hey, you can place the files inside src/name/... and will be loaded by require)
Cheers.

@RX14 could you reduce this PR to using shards version? version.cr has it's pros and cons and should probably be discussed separately.

One nice thing about the lib/project/version.rb in Ruby is that the version file can be loaded independently of the full library if all you want is the version.  This is especially used in many .gemspec files.  Not sure that applies to Crystal projects or not.

Hello @RX14, thank you for your response.
I think you're basing your opinion in comments like this and this?
(Refs from #2788 and #2690).
If that is the logic that is driving your change, will be great if you can add those details in the commit message, so future developers will be able to find the logic of that decision.
Thank you.

Moving the version out, is a welcome improvement. Every file that is no essential is clutter. Especially when Shard.yml already holds a version number.
And if i am honest, the amount of "useless" files, is too much.
.editorconfig
.gitignore
.travis.yml
.LICENSE
.README.md
shard.yml  << The only important file beyond Source

What if i do not use GIT. What about the whole .editorconfig. Travis.yml. I do not need a licence file as its proprietary. This buildup is good if people are building shard to host on GIT but not everybody uses git in there tasks or needs all that clutter.
Unfortunately i am forced to build shards to use Sentry ( directory watch and recompile on change ) or else i will not have stumbled on this clutter.
Its only a quick delete to get ride of the files but at first it was overwhelming seeing so many files being created. Other languages do not go that far.
Maybe have crystal init or crystal init_git as a alternative between a clean shard and a git shard?

@Wulfklaue please keep this discussion focused on the PR. You could open a new issue for your comment.
But keep in mind: nobody needs to use crystal init, you can bootstrap a new project easily by hand. It's just a tool to get you started with a set of opinionated defaults.

Updated with a new longer commit message.

@RX14 The sentence We can use it here to automatically generate the version constant for libraries directory from shard.yml, meaning it will never be out of sync. looks a bit off. It seems like something is missing - or libraries directory is just not a great term. The linebreak after We should be removed.

It's a typo: directly instead of directory

Fixed typo.

Rebased to fix travis, hope this can get reviewed.

@RX14 With example/version.cr removed it is questionable if the require "./example/*" should stay in example.cr, considering it advocates against a flat file structure.

Can't we just use a makeshift "YAML parser" like this?
File.read_lines("#{__DIR__}/../shard.yml").find(&.starts_with?("version:")).try &.byte_slice(8).strip

Maybe put in a method Crystal.shards_version with some additional logic to correctly resolve shard.yml path relative to calling file.

I'll vote to close: as said in my comment, moving crystal init to the existing shards init will prevent us from a mess resulting from a mutual dependence of crystal <=> shards

@j8r that's a different path and probably a good idea, but both changes are not mutually exclusive. It doesn't make a difference if the file is created by crystal init or shards init, extracting the version from shard.yml (on every compilation) shouldn't depend on the shards command anyway.

It seems this got stalled on a dispute on whether we should all new projects generated with crystal init should depend  on the shards  command being available.
While one could argue that's already the case as soon as you add a single dependency, I  can see the point. Given this doesn't seem to really fix a pain point as evidenced by the lack of discussion and voting on this and no issues referencing this, I'll close this for now.


Update index.html

Fixed Issue #16817

Hi @shrey-sachdeva2000, thanks for the PR! Please follow instructions at https://git.k8s.io/community/CLA.md#the-contributor-license-agreement to sign the CLA so that we can review this PR, thanks!

Hey @shrey-sachdeva2000 👋- this has been hanging out here awhile so I'm going to close for now. Please feel free to reopen this when you have time to revisit this and sign the CLA. Thanks!


bug(parser): evaluating Terrform functions with null values as params

when running a scan of a terraform file and a variables file such as:
main.tf
resource "aws_vpc" "my_vpc" {
  cidr_block = "<IP_ADDRESS>/16"

  tags = {
  Name = format("my-vpc-%s", var.name)
  Name2 = format("my-vpc-%s", var.name2)
  }
}

variables.tf
variable "name" {
  type        = string
  default     = null
  description = "just a name"
}

variable "name2" {
  type        = string
  default     = "name2"
  description = "just another name"
}

kics scan will fail with the following error:
                   .0MO.
                   OMMMx
                   ;NMX;
                    ...           ...              ....
WMMMd     cWMMM0.  KMMMO      ;xKWMMMMNOc.     ,xXMMMMMWXkc.
WMMMd   .0MMMN:    KMMMO    :XMMMMMMMMMMMWl   xMMMMMWMMMMMMl
WMMMd  lWMMMO.     KMMMO   xMMMMKc...'lXMk   ,MMMMx   .;dXx
WMMMd.0MMMX;       KMMMO  cMMMMd        '    'MMMMNl'
WMMMNWMMMMl        KMMMO  0MMMN               oMMMMMMMXkl.
WMMMMMMMMMMo       KMMMO  0MMMX                .ckKWMMMMMM0.
WMMMMWokMMMMk      KMMMO  oMMMMc              .     .:OMMMM0
WMMMK.  dMMMM0.    KMMMO   KMMMMx'    ,kNc   :WOc.    .NMMMX
WMMMd    cWMMMX.   KMMMO    kMMMMMWXNMMMMMd .WMMMMWKO0NMMMMl
WMMMd     ,NMMMN,  KMMMO     'xNMMMMMMMNx,   .l0WMMMMMMMWk,
xkkk:      ,kkkkx  okkkl        ;xKXKx;          ;dOKKkc

Scanning with Keeping Infrastructure as Code Secure v1.7.11

Preparing Scan Assets: Done
Error: failed to read sources: failed to save file content: json: error calling MarshalJSON for type model.Document: value is not known
Usage:
  kics scan [flags]

Flags:
  -m, --bom                           include bill of materials (BoM) in results output
......

Closed by #6801


Commit vizualisations to repo so they can be shown in docs.

The visualisations in the docs need the HTML files generated in the tutorials to be committed to the repo.
Ideally we would deploy off another docs branch but that can be left for now.

Fixed in #1032


Use Recording memory APIs in the benchmark.

This allows us to see how the memory usage is affected with different kernel implementations.
Debugging http://b/183232978

looks like we need to include the recording allocator in the BUILD file for the benchmarks.

fixed.


Why is there no spi::blocking::Read trait

There is a blocking write trait, but it looks to me that the read is missing

The upcoming embedded-hal 1.0 does have SPI read.
In 0.2 you can use transfer with a zerofilled buffer.

Thank you
as for transfer, it seems to only work if there are as many bytes sent as received (or it is not well documented enough)

That's right (at least in the current 0.2 version of embedded-hal). It's because SPI must always send as many bytes as it receives. You can just put dummy bytes to make sure you read as much as you want (and remember the first bytes "received" will be during the initial transmission, so you probably want to discard them).

i'm not sure the spi peripheral i'm using respects this kind of rule
but as long as what i need is in the next version, all is good for me


fix: s3 bucket ownership issues

Couple of changes made to how S3 bucket handles ACL and ownership.
Breaking change was made to create logging bucket. I've added a default so that it does not break for users from the older version that do not want to have bucket access logging enabled.

Have rebased to latest master and fixed conflicts.

Pushed a fix on the resources being moved for clarity.


"Extensions" output channel?

Just wondering whether the "Extensions" output channel is actually used for anything? I have checked it when troubleshooting extensions many times and don't think I've ever actually seen any output in there

This is being used by debug adapter
https://github.com/microsoft/vscode/blob/ef802ebe7af412860387ac984682f6c86880af5c/src/vs/workbench/contrib/debug/node/debugAdapter.ts#L260-L263
Feel free to remove it if you are not using.

Oh, wow 🪃
No clue about that

I guess it was introduced in https://github.com/microsoft/vscode/commit/45de4af158df229ae9991cb21c252296aa1b62a8, previously showed errors from all extensions, now I think those errors go to other general logs, and probably just this one usage got left behind. @isidorn any idea if there was some discussion about this in the past? Or I will remove it

@roblourens not aware of anyone using it. I think we can go ahead and remove it.
fyi @weinand

bit off-topic here but I was looking around for an issue around a "combined" log (showing the channel inline) - is there such a thing and I'm just missing it? that way I wouldn't have to click around into various logs when I'm trying to figure out what's going on with vscode - I realize I could tail the various files and write a script to make that more convenient
I was also thinking maybe an extension could fill that need but then I saw https://github.com/microsoft/vscode/issues/106993#issuecomment-694902345

@sandy081 there are only 10 open issues tagged output - altho this issue is about output and got labeled debug instead - I looked at the debug tagged issues too and didn't see it - should I create one?
https://github.com/microsoft/vscode/issues?q=is%3Aissue+is%3Aopen+sort%3Areactions-%2B1-desc+label%3Aoutput+


fix(ui): valueId inputs not showing correctly

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 34 (0.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.04%) to 22.753%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/Gateway.ts
0
3
0.0%

lib/ZwaveClient.ts
0
9
0.0%

server_config.js
0
22
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
3819

Relevant Lines:
17742

💛  - Coveralls


Watch List in debug

Godot version:
3.06
My current game has over 20,000 lines of code with 30 .gd scripts.  I would absolutely love a "watch" list for user inputted variables while debugging.  A place you can type the variables you need to see and monitor them throughout the debug process.  This way a user doesn't have to click through a bunch of nodes to get view the needed variable.  I have used this in other environments with great success.  Can this feature be added?

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


invaliod block type error

**Etcher version:**etcher 1.1.2
**Operating system and architecture:**Windows 10 64 Bit
**Image flashed:**rasbian stretch
**Do you see any meaningful error information in the DevTools?**No  on first tempt  i could open dev tool with Ctr +Alt+ I, later DEv Tools did not open again

Hi, I used several micro SD cards (16GB, 32 GB: always the same error: invalid block type at Zlib._handle.onerror (zlib.js:355:17) , the same error at the different micros SD cards. Cleaned the cards with diskpart, the same error occured: how solve this problem? Other Programm? (DiskImager?)
Thanks for answering
my EMAIL<EMAIL_ADDRESS>

Can you please verify the exact version of Raspbian Stretch that you're using (a download link is ideal) as that'll make it easier for us to investigate this problem.

Thanks  for your answer.
I used the newest version from www..raspberrypi.org/downloads/Raspbian: Version September 2017,release 2017-09-07,Kernel Version 4.9
Sincerely yours
H.Maurer
Von: Andrew Scheller<EMAIL_ADDRESS>Gesendet: Donnerstag, 19. Oktober 2017 01:28
An: resin-io/etcher<EMAIL_ADDRESS>Cc: hbmau<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [resin-io/etcher] invaliod block type error (#1792)
Can you please verify the exact version of Raspbian Stretch that you're using (a download link is ideal) as that'll make it easier for us to investigate this problem.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/resin-io/etcher/issues/1792#issuecomment-337756559 , or mute the thread https://github.com/notifications/unsubscribe-auth/AfYwfMXouYoL0YSnNme3Osx9FOAFAbrYks5stol6gaJpZM4P-Uzh .  https://github.com/notifications/beacon/AfYwfC1dQeEUEDdFSYxDcccP39jonOF1ks5stol6gaJpZM4P-Uzh.gif

@hbmau thanks, we'll have a look!

I’d like to inform you about the following:
I just tried etcher with an external USB Card writer, the same problem, with the same error announcement. So it is not the older build in card reader of my PC.
Mabee some (Micro) SD Cards don’t function orderly?
I’l be really excited about a possible solution of this problem!
Sincerely yours
H.Maurer
Von: Jonas Hermsmeier<EMAIL_ADDRESS>Gesendet: Donnerstag, 19. Oktober 2017 09:03
An: resin-io/etcher<EMAIL_ADDRESS>Cc: hbmau<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Betreff: Re: [resin-io/etcher] invalid block type error (#1792)
@hbmau https://github.com/hbmau  thanks, we'll have a look!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/resin-io/etcher/issues/1792#issuecomment-337817970 , or mute the thread https://github.com/notifications/unsubscribe-auth/AfYwfKN9chmR3kBnXiB5xGTEhW7cpa54ks5stvQ_gaJpZM4P-Uzh .  https://github.com/notifications/beacon/AfYwfC6bdIJWN4mJGnqvErumhKRuQrjlks5stvQ_gaJpZM4P-Uzh.gif

This has nothing to do with the SD-cards or their adapters – the error is about the compressed blocks in the .zip file; I've now tested both, Raspbian Stretch w/ Desktop and Stretch Lite straight from the .zip, and both worked just fine.
My guess here is that your downloads have corrupted due to network issues or so – you should be able to check your downloaded images against the checksums provided on https://www.raspberrypi.org/downloads/raspbian/.

And if you're unsure how to varify checksums, I just did a quick search and found https://bhoover.com/how-to-verify-checksum-windows/


ran into an issue!

Error while creating build definition: The classic pipelines are disabled for this project / organization. Error while creating release definition: The classic pipelines are disabled for this project / organization.

Error while creating build definition: The classic pipelines are disabled for this project / organization. Error while creating release definition: The classic pipelines are disabled for this project / organization.

any solution for this??


updated images gii

Screen of Gii Doc is out of date.

Template paths are partially missing which isn't good.
In start-gii-crud.png there's a tooltip leftover.
start-gii-model-preview.png was displaying "diff" thing which was good for showing off features. Also there's something in the model class field (right side of the field).

@githubjeka would you like to apply changes suggested by @samdark, or you have objections?

It's hard work to do screenshots. Tommorow to get involved in my work I will create new screenshots before.

for better review: it would be great to only replace the english images and we copy them ourselfs because github does not display all images in the diff.


enabling ssl/tls in grpc request throwing error while running testplan

I have given cacert in system.properties javax.net.ssl.trustStore=cacert.p12
I see below error.
Response code:500
Response message:Exception: io.grpc.StatusRuntimeException: UNKNOWN: Channel Pipeline: [WriteBufferingAndExceptionHandler#0, DefaultChannelPipeline$TailContext#0].

Hello. Have this issue been resolved? If so, what was the solution?

Please check PR https://github.com/zalopay-oss/jmeter-grpc-request/pull/47, hope it can be solved!

Hello @huynhminhtan , unfortunately it doesn't. Turns out it was the lack of support for TLS NPN as the current implementation forces ALPN.
I've been able to fix it in my own fork since this one didn't accept my own branch push so that I could create a PR.
There's the fix: https://github.com/yanpaulo/jmeter-grpc-request/commit/2e681dff47dd3e51fbe25f8a3856757304c34e87
Also, how can I create a PR in this repo?

Hi @yanpaulo,
It's great that you solved your problem.
Of course, you can make a merge request, why did you close it #49?

I've created a net PR as the previous one weren't soo good. Prease review #50 .
It will solve the issue and still adds a neat feature on folder selections.

I have being working with this plugin since last week, and I am getting this error.  Could there be another reason?
Response code:500
Response message:Exception: io.grpc.StatusRuntimeException: UNAVAILABLE: io exception
Channel Pipeline: [SslHandler#0, ProtocolNegotiators$ClientTlsHandler#0, WriteBufferingAndExceptionHandler#0, DefaultChannelPipeline$TailContext#0]

Hello! I've been getting the same error as @mmahmutyilmaz. I have a pks12 file but am not sure how to pass it in metadata?

I have being working with this plugin since last week, and I am getting this error. Could there be another reason?
Response code:500 Response message:Exception: io.grpc.StatusRuntimeException: UNAVAILABLE: io exception Channel Pipeline: [SslHandler#0, ProtocolNegotiators$ClientTlsHandler#0, WriteBufferingAndExceptionHandler#0, DefaultChannelPipeline$TailContext#0]

also facing the same issue do u have any update regarding this?


util: testing: Move core docstrings test logic into main package

We should refactor tests/test_docstrings.py into dffml/util/testing/docstrings.py and make it so that the plugins all have a file under their tests which imports and uses the docstring test code so that the plugins docstrings are always tested.
The goal here is to take out as much code as possible from tests/test_docstrings.py and make it into something which the plugins could import from plugins. For example, say we ran dffml service dev create model dffml-model-myslr, and wrote some python examples for the model within the docstring using >>>  syntax. We also wrote a util.py with a function and that function has >>>  style examples in it (see dffml/util/data.py for examples). The goal of this issue is that to test both these examples. We only need to add the following to a tests/test_docstring.py file:
from dffml.util.testing.docstrings import make_docstring_tests

make_docstring_tests()

Our new function, make_docstring_tests(), should be essentially the same as the last two for loops in the the old version or this, tests/test_docstrings.py with the main dffml package, not the plugin you just created. Wherever you see sys.modules[__name__], you'll want to use the tricks that we used in docs/_ext/literalinclude_relative.py to grab the stack frame of the caller. The stack frame contains the information and variables associated with the calling function, see https://docs.python.org/3/reference/datamodel.html#frame-objects for more details on the frames properties.
We might be able to setattr directly on the frames f_globals, to add the tests to the global variables of the caller, which is in this case the new tests/test_docsting.py file we've created in our new plugin. If this doesn't work for some reason try looking in frame's f_globals for the __name__ variable. We should be able to use the value of that in place of __name__ from the old code.

Is it okay if I take this up?
Some pointers would do me good, I looked around tests/test_docstrings.py, the flow is somewhat clear, but the proposed util: "so that plugins have a file under their tests" isn't very clear to me.
Any guide would help :)

@aitikgupta I've updated the issue body, sorry for the slow response! Let me know if you have further questions.
You may have moved on to other things, so for anyone else reading this, the issue is up for grabs, subject to the contributing guidelines: https://intel.github.io/dffml/master/contributing/dev_community.html#communication-style

I'm still working on this.


Invalid Binding on NumberBox with unitialized int DP

Describe the bug

The following code does not create a valid binding:
IDL
    //[MUX_DEFAULT_VALUE("1")] <-- Adding this fixes the issue
    Int32 MyInt;

XAML
<!-- Maximum does not update correctly. -->
<muxc:NumberBox Maximum="{Binding RelativeSource={RelativeSource TemplatedParent}, Path=MyInt, Mode=OneWay}"/>
<!-- Updates correctly! -->
<TextBlock Text="{Binding RelativeSource={RelativeSource TemplatedParent}, Path=MyInt, Mode=OneWay}"/>

This is probably related to the fact that NumberBox.Maximum is a float, while MyInt is an int. When MyInt is not default valued, binding might not know the type and thus discard the binding. This might be related to the WinUI code gen for DPs.
Expected behavior

Binding should update.
Screenshots

Version Info

https://github.com/microsoft/microsoft-ui-xaml/commit/44767d110716d560286072b66ea5bf1f1ccf7b61
NuGet package version:

Windows app type:

UWP
Win32

Windows 10 version
Saw the problem?

Insider Build (xxxxx)
Yes

May 2020 Update (19041)

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Xbox

Surface Hub

IoT

Additional context

Thanks for filing this issue @chingucoding. I agree this is clunky and should be fixed.

Thanks for filing this issue @chingucoding. I agree this is clunky and should be fixed.


[2] Use webpack for dev server and build (new)

Reopening https://github.com/dcos/dcos-ui/pull/52 against feature branch.

Reviewer: @mlunoe

@orlandohohmeier @mlunoe, had to close https://github.com/dcos/dcos-ui/issues/52 and reopen here against feature branch. I have made amendments based on your comments.


wavesurfer not rendering wav file from blobstorage in chrome

I am trying to visualize a waveform from a .wav file stored in a blob container.I generate the SAS string and I get a uri that I pass to the wavesurfer.load( uri ) method. In firefox it works fine but in Chrome i get the exception unable to decode audio data.
uri=https://azureaccount.blob.core.windows.net/container/fileName.wav?sv=2016-05-31&sr............
wavesurfer.load(uri);
It happens the same if I store the file locally and I try to play it from my project folder.
Any ideas?
Thanks

Can you load the blob directly using loadBlob: https://github.com/katspaugh/wavesurfer.js/blob/master/src/wavesurfer.js#L383

I found out that the problem is that I have a 2KHz(sample rate) file and that Chrome is not able to decode it.It happens the same if I just try to play it using a html  tag. In firefox it works fine. Any ideas on how to play a 2khz file in chrome,or how to resample it to make it work?

you can use ffmpeg to convert and resample it, see stackoverflow/google etc.

I leave this in case it is useful for someone in the future, if the file is wav, when loading a blob with the latest version you must define the mime type, but I have found that if the mime type is audio/wav it does not play, but if we "force" it to audio/mpeg it does open:
const mimeType = 'audio/wav'; // Your function to get the mime type

const blob = new window.Blob([new Uint8Array(buffer)], {
  // Try to force loading as mp3 if not supported
  type: MediaSource.isTypeSupported(mimeType)
    ? mimeType
    : 'audio/mpeg',
});

wavesurfer.loadBlob(blob).catch((error) => {
    console.error('Error loading audio', error);
});


Fix mvc bootrapping

Q
A

Documentation
yes

Bugfix
yes

BC Break
no

New Feature
yes

RFC
yes/no

QA
yes/no

Description
Provide option to enable MVC Application bootstrapping.
False by default, to not break any existing app.
See Issue #106 .

I like it.
Regarding the

When enabling it, make sure not to do any HTTP-Only related stuff on Module's onBootstrap method.

Is there currently a way to tell if we are in a laminas-cli run context?
The old laminas-mvc-console package had a delegator factory which was used  to create a ConsoleRequest instead of the HttpRequest.
Would it maybe make sense to define a namespaced const in bin/laminas?
Something like
define('Laminas\Cli\RUN_CONTEXT', true);

So that a hypothetical onBootstrap method could bail out?
public function onBootstrap(EventInterface $e)
{
    // do general purpose bootstrapping

    if (defined('Laminas\Cli\RUN_CONTEXT')) {
        return;
    }

    // do http-only bootstrapping
}

I'm just spit-balling here. :sweat_smile:

@steffendietz

Would it maybe make sense to define a namespaced const in bin/laminas?

Pollution of the global namespace is not an option and must be avoided.

Pollution of the global namespace is not an option and must be avoided.

That's why I was specifically talking about a namespaced constant.

Is there currently a way to tell if we are in a laminas-cli run context?

Not AFAIK.
But: This should not be necessary. As told in the documentation you should not do any heavy stuff in onBoostrap(). Only prepare Services, Listeners, ...   things that ~aren't~ shouldn't be expensive at all. i.E.: setting up a Listener on MvcEvent::EVENT_DISPATCH ist totally fine. It's not executed, because lamas-cli doesn't fire MvcEvent::EVENT_DISPATCH.
If you've got something that really should not be executed in cli context but is currently located in onBoostrap(), it's very likely not in the right place. Think about putting the code behind an Event that's only fired in http-context  (that's what i do most of the time).  Use either EVENT_DISPATCH / EVENT_DISPATCH_ERROR, EVENT_RENDER / EVENT_RENDER_ERROR, ...
... but that's OT and should be diskussed in the Forum.

Regardless of the patch direction, I expect tests to verify what's being done here.

May i 'expect' your help on this, @Ocramius?
I don't know how to set up an environment for the test. And I've no time to figure it out, atm. I tried some bits, without success. I'm not familiar with the used vfs and don't want to mess the tests up by introducing something totally different.

I would rather prefer Mvc application to provide config/container.php which would init and potentially bootstrap the Mvc before it would return container for laminas-cli to consume.
Bootstrapping Mvc unconditionally is not something I am comfortable with. That is a decision that needs to be made consciously for every specific application.

@Xerkus
Any discussion on different solutions / ideas should go into #106.

Indeed. My bad.


Add support for creating a VPN Gateway subnet

An Azure VPN Gateway requires a subnet named GatewaySubnet to work properly:
https://docs.microsoft.com/en-us/azure/vpn-gateway/vpn-gateway-about-vpn-gateway-settings#gwsub
Pulumi transforms all subnet names to lowercase and appends a random eight character string.  This should not happen for subnets named GatewaySubnet.

You should be able to set a name property on the Subnet to force that specific name to be used. When the name is left off, Pulumi provides a default with some randomness appended to ensure uniqueness by default. But in scenarios where a fixed name is required, that name property ought to enable full control.

I am setting the name property on the subnet.  Pulumi sets the name to lowercase and appends eight random characters.
Tested with both python and javascript and get the same result.  I believe it is this line that is the culprit:
https://github.com/pulumi/pulumi-azure/blob/28b906b7dfc3000a3eca2885613529cc9fa2203b/resources.go#L436-L441

I tried the following:
const azure = require("@pulumi/azure");

let resourceGroup = new azure.core.ResourceGroup("server", {
    location: "West US",
});

let network = new azure.network.VirtualNetwork("server-network", {
    resourceGroupName: resourceGroup.name,
    location: resourceGroup.location,
    addressSpaces: ["<IP_ADDRESS>/16"],
    subnets: [{
        name: "default",
        addressPrefix: "<IP_ADDRESS>/24",
    }],
});

let subnet = new azure.network.Subnet("server-subnet", {
    name: "thenameipicked",
    resourceGroupName: resourceGroup.name,
    virtualNetworkName: network.name,
    addressPrefix: "<IP_ADDRESS>/24",
});

exports.subnetName = subnet.name;

The result was subnets named default and thenameipicked:

Do you have a code snippet you could share to highlight the issue you are seeing?

I am seeing the same results as you.  I was confusing the resource name ("server-subnet" in your example) with the name property.  I hadn't seen any examples using that property and IntelliSense told me the first parameter was called name.
Sorry for wasting your time.


Save the current state of the banned list

Purpose
It would be helpful if the banned list would persist between vscode sessions. So, this will save the state of the banned users into this extension's global state.
Changed Files

ChatClient.ts Included a parameter to pass the global state object to the Themer class.
Themer.ts Added the logic needed to add/remove the banned users to/from the global state of the extension.
extension.ts Refactored the ChatClient to be instantiated from within the activate function in order to pass parameters only available when vscode activates the extension (the globalState object).

@all-contributors please add @parithon for ideas, code, documentation

:shipit:


Download Excel action is not displayed [BUG]

Prerequisites

[x] Checked if your Laravel Excel version is still supported: https://docs.laravel-excel.com/3.1/getting-started/support.html#supported-versions

[x] Able to reproduce the behaviour outside of your code, the problem is isolated to Laravel Excel.

[x] Checked that your issue isn't already filed.

[x] Checked if no PR was submitted that fixes this problem.

[x] Filled in the entire issue template

Versions

PHP version: 7.3.12
Laravel version: 6.18.26
Package version: maatwebsite/excel 3.1.21 | maatwesite/laravel-nova-excel 1.2.1

Description
After installing the package following the instructions in https://docs.laravel-excel.com/nova/1.1/getting-started/installation.html and adding Maatwebsite\LaravelNovaExcel\Actions\DownloadExcel action to nova user resoucre actions() list, the action is not displayed.

Package auto discovery is enable so I did not add both excel and laravel-nova-excel packages to config/app.php file

Both verify and reset password actions are display on the details page.
Steps to Reproduce

Upgarde laravel 5.6 with nova package application to laravel 6.x
Run composer require maatwebsite/laravel-nova-excel
Add use Maatwebsite\LaravelNovaExcel\Actions\DownloadExcel; to Nova\User resource
Add new DownloadExcel to Nova\User resource actions list

Expected behavior:
DownloadExcel should be displayed on the resource index page.

Actual behavior:
Actions is not displayed on the index page

Additional Information
Any additional information, configuration or data that might be necessary to reproduce the issue.

Now this is embarrassing but there was actually no bug. I did not select what I wanted to download so the action was never presented. So sorry guys for the false alarm.


GCP Workflows without YAML

Im excited about GCP Workflows but not excited about using YAML as a programming language.  Any existing plans to support Workflows in Pulumi?

Hi @tomdavidson
So Workflows are now supported in pulumi-gcp - https://www.pulumi.com/docs/reference/pkg/gcp/workflows/workflow/
Please let us know if there are any further issues
Paul


Remove fix for "double namespace" imports

In 54dde2e59955318937a3d21f88b34deb9bc74a4f a fix was implemented for "double namespace" imports, where a module is imported like
import urllib.parse

and then referenced like
urllib.parse.urlencode(some_string)

The issue causing this behavior was actually the handling of "nested" attributes originally added in 77f9c3a2712592fd12fe10e6bfa00c8a4b89b9a6, and since removed e4362ecc0c66ed16ca541f386d4b651800240959. By performing the removal in e4362ecc0c66ed16ca541f386d4b651800240959, the test suite passes without the "double namespace" fix.
Removing this code also removes two lines that would never actually be called when running the code, and therefore could not have any test coverage.

Fantastic :clap:


Transparent buttons on hover with colored navbar and config mode: 'ios'

Ionic version:
[ ] 1.x
[x] 2.x
I'm submitting a ...
[x] bug report
[ ] feature request
[ ] support request
Current behavior:
Forcing the iOS style in IonicModule.forRoot results in a strange hover effect on buttons (normal buttons and the back button) in the navigation bar when the navigation is colored at the same time. The button text disappears because it gets the same color as the navigation bar.
This happens only in the browser with ionic serve (http://localhost:8100). It doesn't happen with http://localhost:8100/?ionicplatform=android or http://localhost:8100/?ionicplatform=ios.
The issue is not present with mode: 'android' (the hover color is dark).
Expected behavior:
The button should not get an hover effect or the effect should be a different color as the navigation bar.
Steps to reproduce:
Plunker: http://plnkr.co/edit/97f1WQpYHRk7wL1aXry0?p=preview

Start new project
Add mode: 'ios' to src/app/app.modules.ts:

IonicModule.forRoot(AppComponent, {
  mode: 'ios'
})

Add the following header to a page:

<ion-header>
  <ion-navbar color="primary">
    <ion-title>Page</ion-title>

    <ion-buttons start>
      <button ion-button>Button Text</button>
    </ion-buttons>

  </ion-navbar>
</ion-header>

Related code:
See above.
Other information:
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Cordova CLI: 6.4.0
Ionic Framework Version: 2.0.0-rc.6
Ionic CLI Version: 2.1.18
Ionic App Lib Version: 2.1.9
Ionic App Scripts Version: 1.0.0
ios-deploy version: 1.9.0
ios-sim version: 5.0.13
OS: macOS Sierra
Node Version: v6.9.1
Xcode version: Xcode 8.2.1 Build version 8C1002

Hello, thanks for using Ionic! We will take a look into this.


Budget section missing from Collective page

There was a support request related to https://opencollective.com/DragonetMC so I went to look at their expenses and budget, but the whole section seems to be missing from their page. In the host dashboard I can see they have multiple past expenses.

@Betree do you have an idea?

Will be resolved in https://github.com/opencollective/opencollective-frontend/pull/3095
In the mean time you can check the transactions on https://opencollective.com/DragonetMC/transactions


Fix mssql driver crashing

Since mssql driver has not been maintained, locked driver version and monkey patched it to fix couple of problems of leaking connections / hanging and crashing when connections are closed unexpectedly.
We should eventually change mssql dialect to use directly tedious driver.
I did run this stress test script overnight and it didn't crash and memory consumption didn't grow either (it was stable between 150MB - 400MB)

Is this problem now fixed with the release of 4.2.1? (we got a PR merged that fixed the depletion of connections error - https://github.com/tediousjs/node-mssql/pull/683)

@elhigu We can probably try updating driver, removing monkey-patching and running existing stress tests?..

It would be very helpful if this limitation was removed because we can't upgrade our knex version as we need the mssql fix in production ;)

@dhensby Doesn't monkey-patching introduced by @elhigu solve the same issue as well in newer knex version?

@kibertoad TBH, I'm not sure and it's not something we'd be prepared to put time into investigating. We had issues on azure in our production application which we spent a long time debugging and resolving, culminating in getting https://github.com/tediousjs/node-mssql/pull/683 merged.
It's not going to make business sense for us to downgrade from a known working patch "just" so we can upgrade knex.
The reason to upgrade knex would be so that knex.schema.unique and knex.schema.dropUnique do as expected. The work around to that is far more simple and less risky than rolling back a working patch and risking depleting our connection pool in prod again.
For now we'll use knex.schema.unique([cols], 'our_own_index_name'); and knex.schema.dropIndex([cols], 'our_own_index_name'); which works and I'm sure you can appreciate is insignificant in terms of risk to prod vs downgrading the mssql lib.

@dhensby That makes sense. I'll try to get stress tests working locally on my machine to get the answer.

@dhensby Wait, was that version ever released? https://www.npmjs.com/package/mssql still shows 4.1.0 as the latest published one.

https://www.npmjs.com/package/mssql/v/4.2.1 - it seems to be @next for some reason, but we're definitely using it...
It's not the most maintained package out there ;)

@dhensby Does stability-wise (other than this specific issue) 4.2.1 perform same or better than the current "stable" 4.1.0 for you? In general it is preferable to avoid "next" dependencies in libraries, I wonder if we should make an exclusion for this particular case.

Yeah, that's a good point, I hadn't realised it was a "next" release, though, anyone wanting to run mssql needs the fix that is in 4.2.1.
In production we haven't had any issues with MSSQL, the library or other symptoms since upgrading to 4.2.1, so we are happy with its stability.
Another option would be to allow users to set some kind of config to opt-out of the 4.1.0 restriction so those that want to run the higher versions can?
This is the diff between 4.1.0 and 4.2.1 (https://github.com/tediousjs/node-mssql/compare/85c14ff5b01b1ae0575591c7e4f1c7327cf82e51...master) and it doesn't look too risky to me.

I've asked that they release 4.2.1 as stable (https://github.com/tediousjs/node-mssql/issues/703) though not sure how likely that is to be actioned :(

I'd still rather get rid of mssql driver and use tedious directly than update knex to support newer mssql. It will be easier to make to work in a robust manner than trying to fix mssql to work well.
I'm open to help / review if anyone is going to change mssql -> tedious, but I really don't want to work with node-mssql's code anymore.

@elhigu I can absorb the effort of updating the supported mssql version as that doesn't sound like lots of work, but mssql -> tedious is going to require much more elaborate knowledge of SSQL and Tedious, and I don't even use MSSQL myself :D

Not much more plain sql knowledge is needed... node-mssql already uses tedious pretty straightforward manner, but ot just forces to use one additional (and buggy) generic-pool for getting connections which causes most of the problems.

There is already a PR to replace generic-pool with Tarn.js for node-mssql. Would working on it to make it merged help alleviate the pain?

Whilst I totally understand that and using the mssql package has caused us a lot of pain too, I don't think that larger objective should block derestricting the package currently supported.
For lots of developers (especially ones in our position) throwing out node-mssql is non-starter and simply means we won't be able to upgrade knex, which would be a shame

There is already a PR to replace generic-pool with Tarn.js for node-mssql. Would working on it to make it merged help alleviate the pain?

I would rather see PR which allows to use node-mssql without any additional pool on top of tedious. Tarn is developed in terms of knex, so it might not be the best choice for mssql driver, but yeah with that it would be definitely easier to fix at least.

Whilst I totally understand that and using the mssql package has caused us a lot of pain too, I don't think that larger objective should block derestricting the package currently supported.

For lots of developers (especially ones in our position) throwing out node-mssql is non-starter and simply means we won't be able to upgrade knex, which would be a shame

How are you actually using mssql driver directly? What difference it makes for you if you have mssql wrapping tedious when you are actually using knex to interface it?

We only use knex for our DB migrations / schema building. All our DB interfacing happens through stored procedures and an internal DB SDK that provides an API to those stored procedures.
As knex doesn't support stored procedures we don't use knex for our actual DB SDK and so use the mssql package to bind the params and execute the SPs.

In that case you can still use knex the same way that before and node-mssql for the same stuff you are using it currently.
Knex just internally wont use node-mssql anymore.

That's a good point ;)


Flashing CC2531 stops after an arbitrary number of pages

I am trying to flash my CC2531. I have followed the instructions in the book "Control Your Home with Raspberry Pi" pp 189-193. I managed to connect everything as prescibed, download the zip file CC2531_DEFAULT_20211115.zip, unzip it on my Pi. As prescribed, I do ./cc_erase (system response ID = b524. erase result = 00a2). Then I do ./cc_write CC2531ZNP-Prod.hex and the writing starts. It should write all 128 pages. However, the writing stops after an arbitrary number of pages (17, 39, 2, 53, ...) and then stops, and gives the message "flash error ... have you erased before write?". I tried several times. What could go wrong? Thanks for any help.

Not received any answer. My initial flash configuration used a GPIO extension board, a breadboard and a link from the indicated IO ports to the connection board. This gave aforementioned problems. Therefore I disconnected the Pi from the flat cable and directly coupled the Pi to the connection board. Then the flashing was completed without problems. Thus the problem was in the intermediate connection.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_ff8c1ef7-b2f5-47fa-a78b-377a4c87febd/pull/1 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target_ff8c1ef7-b2f5-47fa-a78b-377a4c87febd.git

Docs Build status updates of commit 1346cc0:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-UserAccountBuildPermissionInsufficientError] Owner v-alji no longer has Read permission of the repository https://github.com/opstest2/docs-archive-test-source. It might be due to recent GitHub changes on SSO. Please ask the owner v-alji to logon to Docs Portal https://ops.microsoft.com once again. If it is not the case, please ask anyone of Admin permission of the repo to assign a new owner via Docs Portal https://ops.microsoft.com

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Fix for TimeSlots

This small pull request fixes Issue #1442
This issue occurs due to Pull Request #1204
In the case where the range of an event is less than the step, this unmodified line will cause the position of the event to be positioned too early.
The bug, as demonstrated here
When the step attribute is set to the minimum time, this fixes the issue, as seen here, but this forces a change in UI, which is not desirable.
While this current fix resolves the issue to correctly push any events past the calendar height back within, it has the unfortunate side effect of utilizing the step attribute to determine the height. For an arbitrarily large step, this will push it to the top of the large step block.
I welcome any feedback prior to modifying the tests.

I can confirm that this PR solves events being offset, as reported in #1396

@MincedCutlet Can you please update the failing tests. I think the assertions should be .toBeGreaterThanOrEqual to pass but I am not sure if that's the desired assertion.

Is this getting anywhere? I'm having multiple problems displaying events...

Any chance of merging this?
It fixes this offset issue:

Any chance of this being merged in the near feature?

need to see if tests pass, i can't push to this branch and i can't restart manually.


Clean up the Makefile

Using Godep requires that we build the app from within a correctly structured workspace.  This is primarily managed by the CI Jenkins job, and this change gives the option for placing the built binary somewhere specific.

:+1:


missing license

Hello,
I have found these java api useful but don't know what license is applicable for it.
Could you please add one ? If is fully available - could you please add Apache standard license?
thank you in advance,
Sergey

Hello,
It is cool you think my code is useful. I added license.


Refactor vector_argminmax into reduce_to_scalar

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

13 of 13 (100.0%)  changed or added relevant lines in 3 files are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.1%) to 83.767%

Files with Coverage Reduction
New Missed Lines
%

mlir_graphblas/engine.py
1
75.57%

mlir_graphblas/tests/test_profile.py
4
51.95%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
2864

Relevant Lines:
3419

💛  - Coveralls

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

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

Details

13 of 13 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 83.913%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
2869

Relevant Lines:
3419

💛  - Coveralls


Best way to totally remove an app deployed via mup?

What is the cleanest way to remove a Meteor app that was deployed via mup? (but not the rest of the dependencies such as Node, Phantom, Mongo, etc)
Thanks for all your help in answering my questions, @arunoda!

It hard, try to reset the VM.
On Sat, Jun 13, 2015 at 7:17 AM Sam<EMAIL_ADDRESS>wrote:

What is the cleanest way to remove a Meteor app that was deployed via mup?
(but not the rest of the dependencies such as Node, Phantom, Mongo, etc)
Thanks for all your help in answering my questions, @arunoda
https://github.com/arunoda!
—
Reply to this email directly or view it on GitHub
https://github.com/arunoda/meteor-up/issues/478.

It's not a VM, this is a production server with more than one Meteor app on it.

Other then first stop the app.
Then remove /opt/
Then remove /etc/init/.conf
You need to remove mongo via apt-get
On Sat, Jun 13, 2015 at 7:19 AM Sam<EMAIL_ADDRESS>wrote:

It's not a VM, this is a production server with more than one Meteor app
on it.
—
Reply to this email directly or view it on GitHub
https://github.com/arunoda/meteor-up/issues/478#issuecomment-111652475.

I don't want to remove Mongo (or Node, Phantom, etc) since other apps depend on those.
So to remove a single Meteor app, just use mup to stop it, then remove /opt/ and /etc/init/.conf. Thank you!

yes. it is.
On Sat, Jun 13, 2015 at 7:22 AM Sam<EMAIL_ADDRESS>wrote:

I don't want to remove Mongo (or Node, Phantom, etc) since other apps
depend on those.
So to remove a single Meteor app, just use mup to stop it, then remove
/opt/ and /etc/init/.conf. Thank you!
—
Reply to this email directly or view it on GitHub
https://github.com/arunoda/meteor-up/issues/478#issuecomment-111653786.

+1 thx.

I'm noticing something related to removing an app.  I enabled SSL with Stud.  Got it working.  But I want to stop using it because I'm going to do my SSL through nginx (try at least).  It looks like Stud automatically keeps working on any app I deploy - even if I comment out the ssl mup.json entries in my new apps.    Is there a way to temporarily disable it?
I'm just noticing it still automatically working on all my newly deployed apps that use a different name. And I double checked it wasn't the old app still running.  I followed the above steps and confirm its not an old app running.

+1 thank you!

I could not find /etc/init/<appName>.conf. Is mupx not using Upstart?

mupx is docker centric and nginx too so your assumption is correct.

And where lives the upstart conf file. Which path?


Why do we use --pathpath for "ProjectPath"?

from Root.go:
RootCmd.Flags().StringVarP(&utils.Flags.ProjectPath, "pathpath", "p", ".", "path to serverless project")

Never understood why the flag is --pathpath and cannot explain this to users.

I guess it is a typo of projectpath?


Introduce window-function

I have a dataset with column Class which takes one of categorical values like Dog, Cat etc...

[ ] I need to select a subset of the records with 5 records in each class.
[ ] I need the same but the subset has to be randomized.

This is how it looks like in IBIS (with randomization):
# Create a random column for ordering
random_col = table.mutate(random_val=ibis.random())

# Define a window function partitioned by 'Class' and ordered by the random column
window = win.Window(partition_by='Class', order_by=random_col.random_val)

# Add a row number column based on the window function
ranked_table = random_col.mutate(row_number=ibis.row_number().over(window))

Merged. Will continue in the follow-up issues https://github.com/iterative/datachain/issues/522 and https://github.com/iterative/datachain/issues/523


[GHSA-329j-jfvr-rhr6] Apache Spark vulnerable to Improper Privilege Management

Updates

Affected products
Description
References

Comments
https://issues.apache.org/jira/browse/SPARK-41958 indicates it was fixed in 3.3.3/3.4.0, not only 3.4.0

cc @wangyum, the release manager of Apache Spark v3.3.3


Formatter adds space in js variable name with "-"

Input
The code looked like this before beautification:
console.log('input-fullname: ' + request.body.input-fullname);
console.log('input-email: ' + request.body.input-email);

Expected Output
There should be no change to this code. The code should have looked like this after beautification:
console.log('input-fullname: ' + request.body.input-fullname);
console.log('input-email: ' + request.body.input-email);

Actual Output
The  code actually looked like this after beautification:
console.log('input-fullname: ' + request.body.input - fullname);
console.log('input-email: ' + request.body.input - email);

Steps to Reproduce
I am using js-css-html formatter in vscode. The formatting happens on save and breaks js code.
Environment
OS: mac osx
Settings
Default settings in vscode.

Hi,
This is the expected output. Properties accessed using the dot notation cannot have "-" in them, otherwise it would be indistinguishable from the minus operator (remember that spaces are optional).
Thus,
request.body.input-fullname
will be interpreted as
request.body.input - fullname
As you can see, the formatter actually helps finding such bugs.
If you need to do this, however, there is a way:
request.body["input-fullname"]
(There is no difference between the dot notation and the bracket notation in JavaScript.)
Reference:
https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Operators/Property_accessors


The '' is ignored.

It seems that the tag <View> is ignored even it contains a id

Can you please insert screenshot or explain the specific situation that you have?

I am sorry but this is how it behaves right now. I don't plan to change the implementation because It may confuse other users.
Maybe try to use the Cover in this way.


How set updated_at as null in default?

Laravel Version: 5.2
PHP Version: 5.6
Database Driver & Version: 5.2

Description:
Hello. When i install laravel database and getting started with eloquent, i'm use created_at and updated_at field. In $user->save() action created_at field set current timestamp and updated_at set as null. If i'm updated $user, updated_at set current timestamp.
But today i'm use eloquent and create $user. In this action field created_at and updated_at set same value. This is bug or i'm early doing something wrong?..
And how i can set updated_at null value when create (not update) model object?..
Thank you.

Please ask questions on the forums.


#chat/registry

Add registry manager
update client api with specs
add Concurrency

This PR is pointing to main after the develop merge, need to fix it


Add test coverage report in CICD

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
You can refer to https://golangci-lint.run/usage/linters/
Describe alternatives you've considered
No response
Additional context
No response

Merged into main


Correct node visitor to correctly resolve hints to class parameters

This should close #281 .

@fabpot Please, review this one. It should fix error introduced in 4.0.1 of not properly parsing class parameters.

Thank you @TiS.


Can't open a file from a diff

Issue Type: Bug
Click on a file in the tree to open a diff, then there is no way to quickly jump to the editable version of the file (assuming you have checked it out locally). I would expect an Open File button in the editor toolbar like the standard diff (and GitLens), as well as have a command right from the tree to open the file directly.
Extension version: 0.1.4
VS Code version: Code - Insiders 1.28.0-insider (8330a695712c43f861ce5c47533d82637d3cb374, 2018-09-13T11:25:00.283Z)
OS version: Windows_NT x64 10.0.17758

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700K CPU @ 4.00GHz (8 x 4008)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
31.93GB (18.75GB free)

Process Argv
C:\Users\Eric\AppData\Local\Programs\Microsoft VS Code Insiders\Code - Insiders.exe

Screen Reader
no

VM
0%

I noticed that this works if I open the diff from the Changes in Pull Request tree, but not the other tree -- imo it should work the same for both (again assuming I have it checked out)

Duplicate of https://github.com/Microsoft/vscode-pull-request-github/issues/268
Yeah, we should have this for both - the reason that the behavior is different between trees is that in the GitHub Pull Requests tree, the extension registers a document content provider and calculates and returns the content based on patches fetched from GitHub, since the commits in question might not exist locally. When checking out we do a fetch so we can just diff local commits in the Changes in Pull Request tree


Adding Tutorial for Optimized LEMP with Wordpress

Thank you very much for your submission, it has just been finally reviewed and was accepted into our repository. However, I must inform you that a check of the mail address you have attached to your Github profile has found that said address is not currently connected to any existing netcup customer account. Could you please add the mailing address you have used to register at netcup to your Github account and add a comment on here, so I may forward that to the department responsible for handling vouchers? Alternatively, you may also send a mail to me at<EMAIL_ADDRESS>if that is preferred.


Japanese Support

Not completed as of yet but I'm working on it slowly.

Wow, thank you!
Two changes are required though:

rename poduim folder to podium
Yii 2 core uses ISO code ja without locale part JP so I guess it's not necessary - please rename ja-JP to just ja (unless it's needed like in case of pt-BR)

Hopefully that fixed it, I'll be working offline for the rest of the translations then will update this afterward! :+1:

Cheers!

Actually, it supposed to be ja/podium instead of podium/ja 😄 I will modify it myself.

Ah sorry was in a hurry. xD


TCPStatsClient: handle broken pipe error, reconnect if it happens

In TCP mode, I was getting pipe errors, and the statsd client doesn't ever recover from these errors on its own.  This patch makes it reconnect in case it gets a pipe error.

This is intentional. In TCP mode, it's up to the consuming application to decide how to handle disconnections.
The reconnect method is provided so that applications can do this, but this library isn't going to decide to go into an infinite loop on its own.

Hm, OK, fair enough.


[api-docs] Update the embed description character limit.

A new commit was made into the api-docs repo. This issue was created automatically. Need to review if we made any change to the library for this commit.
https://github.com/discord/discord-api-docs/commit/aa5022364275072f775d162777cd5d3c348957ff

ncn


Add possibility to specify timeout of fetching URL

Fixes #3
Also fixed one failing test and skipped other failing test, which can not be easily fixed

Looks great. Please bump the version in pubspec and add a changelog entry and I'll merge / publish it. Thanks for your contribution!


Update GIF on README.md

The gif shows an enumerated table of contents, but v0.2 does not enumerate the toc anymore.

v0.3 adds a new option to optionally enable enumeration of the table of contents as well. Default is still no enumeration. So the GIF does not have to be replaced.


Fix #13195 - Update way of parsing date

Brief Information
This pull request is in the type of:

[X] bug fixing
[ ] new feature
[ ] others

What does this PR do?
The PR updated way of parsing 7th part of date string from simply treat it as milliseconds to pick first 3 digits as milliseconds to avoid unexpected result when backend API return date string with precision of microseconds.
Fixed issues
#13195
Details
Before: What was the problem?
Though 3 digits milliseconds is widely used in most situations, some date formatter will return date string 6 digit microseconds (yyyy-MM-ddTHH:mm:ss.micros) . ECharts will treat micro seconds as milliseconds when parsing DateTime string and cause unexpected result.
After: How is it fixed in this PR?
Keep conistent with default constructor of JavaScript Date - only pick first 3 digits when the length of input part between dot and zulu time marker is longer than 3 digits.
Usage
Are there any API changes?

[ ] The API has been changed.

Related test cases or examples to use the new APIs
NA.
Others
Merging options

[ ] Please squash the commits into a single one when merge.

Other information

We really need this fix in our app please can you fix the conflict / merge it ?

We can merge it after the code style change and conflicts resolved.

I close this because due to some reason I can not find my original branch. And considering the timeline last too long, I have to create a new local branch based on latest master and applied my change on that. Please refer to the new PR #15410 for reference.


resizing the canvas element whenever the update function gets executed

Previously when updating the canvas (as result of the resize event) the following condition was preventing the canvas to be actually updated with the new size:
if (devicePixelRatio !== window.devicePixelRatio) {
This resulted in the canvas cutting off part of the chart on the far right side:

I am now moving this condition to control the canvas.scale method only.

Just noticed, I think the call on line 740
https://github.com/davidmarkclements/d3-fg/blob/903b393ef38f009d0167a7caf71b11f841262962/index.js#L740
is always immediately followed by a call to update() which calls adjustForHighDpiScreens again on line 324
https://github.com/davidmarkclements/d3-fg/blob/903b393ef38f009d0167a7caf71b11f841262962/index.js#L324
so the one on line 740 is probably not necessary.


Target classes instead of elements in stylesheets from a specific PR

Bug Report
What is the current behavior?
Many global styles are being affected by element-applied stylesheets from https://github.com/OperationCode/operationcode_frontend/pull/451 - I dropped the ball in not catching those styles, but we're very aware now. Going forward, all styles must be applied to classes or elements as children of classes.
What is the expected behavior?
All styles must be applied to classes or children of classes.
Resolving this issue will also close #511
How do I resolve this issue?
This ticket has been marked beginner-friendly, so I'd like to apply some hints. Looking at the PR above (#451) you can quickly find the stylesheets in question. Anywhere where you see styles applied to a specific element, you should instead put them as children of a class or directly apply them to a class. This may require you create classes and target them in the relevant .js file with className="newClassName"

In-progress at #552

PR #557 should fix some of the issues.

Resolved in #557