data mode not working

Hello,
I am using vuetable-2 v1.6.3 via npm, and data mode seems not to be working in this version. Here is my Code:
<vuetable ref="vuetable"                               :api-mode="false"                               :data="myData"                               :fields="myFields"                               pagination-path=""                     ></vuetable>
Table headers (myFields) are appearing correctly but the table body says: "No Data Available" . At first I thought I had a configuration mistake but on my research I found this codepen you supplied some time ago. There you can see the same issue:
https://codepen.io/ratiw/pen/LyLpow
Please advise :) .

Wie oft muss ich mich noch abmelden?
Der Quark interessiert mich nicht!

@tbyte80 Use this updated codepen as a guide instead. Still finding time to finish tutorial on using Data mode.

@tbyte80 Also, it appears that Vuetable does not handle the data array correctly. I'll tag this as a bug and work on it later.

Fixed in v1.6.5


Fix issue #3: Change the name  in Install guide to use semantic version

Fix issue #3: I already changed the name to use semantic version style for later enhancement

Thanks


TSLint "interface-name" rule not correcly applied

Hello,
I do not want tslint to warn in case interface name does not start with 'I'. Also, I configured tslint.json with the rule:
"interface-name": false,
However, my typescript editor is still displaying a warning decorator saying "Interface name must be capitalized I" (whatever is the value of the interface-name rule, actually).
Note that if I run tslint using the command line, then rule is applied correctly.

this is the eclipse-tslint project (distinct from eclipse-typescript): https://github.com/palantir/eclipse-tslint

Closing since this project is not related to tslint.


Create Integration test api for reseting the state during tests

There is need to reset backend state in

ipaas-ui/e2e tests (@jludvice working on e2e)
ipaas-system-tests

The idea is to conditionally enable some MBean or jaxrs resource if we're in testing mode,
to allow set/reset test fixtures.
It was discussed on IRC with @chirino @rhuss and @iocanel
Consideration

JMX Mbean / jax-rs with @ConditionalOnProperty
how to do auth / disable auth?
in case of JMX MBean/jolokia, it would be required to log into the openshift console first before accessing this test resource from e2e tests.

QE could implement this test resource once we know which way to go.

@jludvice I think we can close this one as https://github.com/redhat-ipaas/ipaas-rest/pull/211 is merged already.
Thanks @chirino!

Agreed @dsimansk
Thank you @chirino
Closing issue


AutoIt <> AutoHotkey lexer

Please split this:
https://github.com/pygments/pygments/blob/master/pygments/lexers/automation.py
in two separate files:
https://github.com/pygments/pygments/blob/master/pygments/lexers/autoit.py
https://github.com/pygments/pygments/blob/master/pygments/lexers/autohotkey.py

Why? The location of a lexer is not very relevant, and we don't have a policy for one lexer per module.

It appears this ticket can be closed.

Why? The location of a lexer is not very relevant, and we don't have a policy for one lexer per module.

Because AutoIt and AutoHotkey are two different languages with completely different syntax.

I don't see the problem? We have plenty of languages with different syntax in the same file if they're somewhat related, and that applies here as well. See data.py or esoteric.py for instance.


Nice to have: an option to save filter setting in "Edit table"-->"Table data"

This would be nice to have. Our users want a way to set their filter, column widths, and hide/unhide. Then when they reopen DBeaver the view configuration is always re-opened. It doesn't seem to do this if you restart DBeaver.

already implemented


[BUG] I press play roblox and it takes me to this site (roblox failed to launch)

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[ ] I did not answer truthfully to all the above checkboxes.

What problem did you encounter?

Bloxstrap Log
N/A

i did i cannot find an answer.

I fixed the problem i opened roblox without bloxstrap (roblox player) i open it then close it and then open bloxstrap again how you do normally and it should work


chore(redux): add unit tests to cost model sources

add unit tests to cost model sources

Codecov Report

Merging #1575 into master will increase coverage by 0.91%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1575      +/-   ##
==========================================
+ Coverage   77.50%   78.41%   +0.91%
==========================================
  Files         233      233
  Lines        3832     3832
  Branches      704      704
==========================================
+ Hits         2970     3005      +35
+ Misses        772      739      -33
+ Partials       90       88       -2

Impacted Files
Coverage Δ

src/store/sourceSettings/reducer.ts
100.00% <0.00%> (+35.71%)
:arrow_up:

src/store/sourceSettings/actions.ts
100.00% <0.00%> (+60.00%)
:arrow_up:

src/store/sourceSettings/selectors.ts
91.66% <0.00%> (+66.66%)
:arrow_up:

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 a5242e5...dbfe7e4. Read the comment docs.

Thank you @ddonahue007 :bow:


px-control-locate's attribute "move-max-zoom" doesn't set the max zoom level.

In px-control-locate, if move-max-zoom is set to a different zoom value, it doesn't make any difference on the zoom level.  It seems locate always defaults to the highest zoom level.
Screenshot:

Code Snippet:
<px-map-control-locate
              position="bottomright"
              move-to-location
              last-found-location={{location}}
              move-max-zoom = "6"
              on-px-map-control-locate-error="handleerror"
            >
          </px-map-control-locate>

/cc @davidrleonard

Fixed in a147d666d26a6c64a6b818d53430e60125e8c3f8 (v1.2.1)


KYLIN-4576 add some hint of password length

Proposed changes
add some hint of password length in frontend password placeholder
Types of changes

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist

[x] I have create an issue on Kylin's jira, and have described the bug/feature there in detail
[ ] Commit messages in my PR start with the related jira ID, like "KYLIN-0000 Make Kylin project open-source"
[ ] Compiling and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If this change need a document change, I will prepare another pr against the document branch
[ ] Any dependent changes have been merged

Pull Request Test Coverage Report for Build 6033

0 of 0   changed or added relevant lines in 0 files are covered.
13 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.01%) to 27.955%

Files with Coverage Reduction
New Missed Lines
%

stream-core/src/main/java/org/apache/kylin/stream/core/storage/columnar/ColumnarStoreCache.java
1
56.82%

core-metrics/src/main/java/org/apache/kylin/metrics/lib/impl/MetricsSystem.java
5
60.0%

core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java
7
76.29%

Totals

Change from base Build 6029:
-0.01%

Covered Lines:
26087

Relevant Lines:
93319

💛  - Coveralls


Redundant css suggestion for color

VSCode Version: Any
OS Version: Any

Steps to Reproduce:

Use the below content and trigger suggestions at the end of the property value

.foo {
   color: #123
}

The suggestion list has 2 entries. 1 from emmet, and the other from the css extension repeating the value that is already typed.

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

Is this part of the css-language-service?

@praveenpuglia Yes, but it's ok to have the issue here.

@ramya-rao-a What are you asking for?
Suggesting #123 is working as expected from css. If you want exact match to come before others there are issue for it but I can't find.

@octref Why do we even suggest #123 in this case? It doesnt add any value.

@ramya-rao-a Well just found out I was the guy that opened the exact issue: https://github.com/Microsoft/vscode/issues/21362

IMO @ramya-rao-a is valid here.
If there's a known set of values for a css property, e.g. for display, we should show each value as a proposal.
In this case, we guessed possible values by looking at other locations. When doing that we should ignore the current value. If some other location has that value, then, ok.


Change the  color for different screens

This isn't a bug. But I want to if there is way to customize  element's color based on which screen it is on. I used customisation to change it to black for home screen. I want blue for login screen, yellow for settings screen and so on.
 <StyleProvider style={getTheme(myStyle)>
can we something like
set toolBarDefaultBg(color)
  {
    toolBarDefaultBg: color
  },

Or is there any built-in way to accomplish this.

either you can create new variables for each page and use that in header by importing it and passing the it to below props
iosStatusbar="light-content"
androidStatusBarColor='blue' /instead of blue you can use commonColor.settingPageColor
where CommonColor is imported from ejected NB theme and settingPageColor is defined by you .
in header will work for you .

@AshwinTayson Solution by @GeekRishabh should work in this case
Closing this issue


[Rendering] Action with Style destructive renders with poor constrast in "Standard" Design and not visible in High Contrast mode

Target Platforms
WPF
SDK Version
Bot Framework v4
Application Name
Microsoft Teams
Problem Description
Hi,
I am not sure if this is the right place, if not, could you please point me to the correct place to report this?
We have developed a bot application that uses adaptive cards to interact with its users. We recently noticed, that some of our action buttons get very poor contrast in the Windows Teams client (not Preview Version)
An adaptive card generated by our bot application renders very bad in Windows Teams Client: Version <IP_ADDRESS>78 (64-Bit).
Standard design has bad contrast and high contrast design has the text rendered white on a white background.
Also High Contrast Design does not render Placeholder Text very well...
See screenshots below.
Kind Regards,
Tobias Stähli
Screenshots

Card JSON
{
  "type": "AdaptiveCard",
  "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
  "version": "1.4",
  "body": [
    {
      "type": "ColumnSet",
      "columns": [
        {
          "type": "Column",
          "width": "stretch",
          "items": [
            {
              "type": "TextBlock",
              "size": "large",
              "text": "Microsoft Teams Testalarm (TS)",
              "wrap": true
            },
            {
              "type": "TextBlock",
              "spacing": "small",
              "isSubtle": true,
              "text": "Sie sind dabei, diesen Alarm auszulösen"
            }
          ]
        },
        {
          "type": "Column",
          "width": "auto",
          "items": [
            {
              "type": "Image",
              "url": "https://na-teams-bot-develop.novalink.ch/static/images/alert_icon_46_neg.png",
              "width": "50px",
              "height": "50px",
              "backgroundColor": "#FF00FF"
            }
          ]
        }
      ]
    },
    {
      "type": "Container",
      "items": [
        {
          "type": "TextBlock",
          "text": "Individueller Alarmtext:"
        },
        {
          "type": "Input.Text",
          "placeholder": "Individuellen Alarmtext eingeben...",
          "id": "CustomAlertText",
          "value": ""
        },
        {
          "type": "TextBlock",
          "text": "PIN"
        },
        {
          "type": "Input.Text",
          "placeholder": "PIN-Code zum Auslösen",
          "id": "pinCode",
          "isMasked": true,
          "style": "password"
        }
      ],
      "separator": true
    }
  ],
  "actions": [
    {
      "type": "Action.Execute",
      "title": "Alarm auslösen",
      "data": {
        "value": "triggerable_alert_triggered",
        "NAAlertCookie": "some value"
      },
      "style": "positive"
    },
    {
      "type": "Action.Execute",
      "title": "Abbrechen",
      "data": {
        "value": "abort_triggerable_alert",
        "NAAlertCookie": "some value"
      },
      "style": "destructive"
    }
  ]
}

Sample Code Language
No response
Sample Code
No response

@tstaehli
If you examine your card, you will notice that there are no places to configure the parameters you desire. It is intentional such parameters are supposed to be configured by host apps, in your case, it is Microsoft Teams. You should relay this concern to the Microsoft Teams team.

@jwoo-msft Thank you for your quick reply.
I'm not sure I fully understand your answer. Is it, that I have reported the issue to the wrong project (If so, which one is the correct one) or that I have used properties in the adaptive card, that I should not have used? So basically is it a bug in microsoft or did I use stuff I should not have used?
Thank you

@tstaehli, you've reported to the wrong project.
You can test your card using our designer, which offers a close approximation of how cards would render in the respective hosting app.

You can choose the host app and select between light/dark mode.
Your card renders correctly. Attributes such as background and button colors are configured by the hosting app. You should reach out to the team that owns the app rendering the AC and report this issue. Teams also have their own designer.


Apotheosis bookshelves dont work

Vanilla bookshelves use a property called "enchanting power" that is part of the block and set to 1. This mod reads the enchanting power of all surrounding blocks and adds them up to calculate the amount for the bookshelves requirement.
Apotheosis does not use this mechanic and did not set this property for their bookshelves.
I need to find a fix for this. Maybe a tag for blocks where if it does not have this property but is tagged it will default to enchanting power of 1 (= vanilla bookshelf).

Mojang has introduced a new tag for this in a recent snapshot for 1.20


define CSRs reserved for trapping to HS-mode

This patch defines a range of CSRs that always trap to hypervisors, namely 0x400-0x5FF. Future releases of the standard should not define vs* CSRs that map to this range. These could be used to enable paravirtualization capabilities for the HEE, including:

reporting of time when no host CPU was available to run the guest

reporting host wallclock time at VM boot

requesting the host to flush the TLB on the next guest entry (which avoids IPIs on overcommitted host)

enabling asynchronous page faults (which could be delivered through scause>=16)

Standard CSRs in these ranges can be used to define paravirtualized interfaces that are accessible from multiple hypervisors.
Because U-mode CSRs in those range are all defined as standard CSRs rather than custom, the patch also redefines the range 0x4C0 to 0x4FF to custom read/write.

I'm having trouble understanding the rationale here.  SBI calls seem more appropriate for a paravirtualization interface.  This is an enormous fraction of the CSR space to allocate for this purpose, and by contrast, the SBI-call space is effectively unlimited.
Separately, I do not see justification in allocating more custom CSR space at this time.

This is an enormous fraction of the CSR space to allocate for this purpose

Note that this patch does not allocate CSR space for paravirtualization. It just says that there are no vs* registers allocated in this range, which actually matches how the current specification allocates hypervisor-level CSR addresses (compare 0x200-0x2FF to 0x600-0x6FF for h* in Table 2.4). The idea is to guide the choice of CSR addresses in future extensions to the privileged specification.
Paravirtualization (whether standard or hypervisor-specific) is one possible use, but the important part is that CSRs trap to HS-mode.

It is an allocation, as this part of the address space can no longer be used for hardware CSRs in those modes.
Why not employ a level of indirection (e.g. ECALL with an x-register calling convention)? In addition to the larger address space, this approach will perform better, since software won’t have to decode instructions to determine the trap cause.

Fair enough...


What are the defaults for these switches?

It would be handy to know what the defaults for these switches are; particularly the -consoleloggerparameters and -verbosity switches.

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

ID: e6254da8-fbe7-cc2a-2c10-1ba918a61039
Version Independent ID: 7d3100b1-8ab0-f17f-6494-d183bdeb9081
Content: MSBuild Command-Line Reference - Visual Studio
Content Source: docs/msbuild/msbuild-command-line-reference.md
Product: visual-studio-dev15
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

Verbosity has a default of "Normal".

And consoleloggerparameters follows the overall verbosity (so defaults to normal) and includes a Summary.

Updated the command line topic to add the default for verbosity and /clp.


add typescript .d file

@amibenson @krystianity What is the reason only a few methods made in into the ts definition? I'm now missing a lot of functionality that is implemented in the lib/Bitstamp.js file, but not described in index.d.ts

@amibenson @krystianity What is the reason only a few methods made in into the ts definition? I'm now missing a lot of functionality that is implemented in the lib/Bitstamp.js file, but not described in index.d.ts


nkn version 1.2.1 ERR_PACKAGE_PATH_NOT_EXPORTED when running example

After installing the version 1.2.1 and trying to run an example I get
      throw e;
      ^

Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: No "exports" main defined in<EMAIL_ADDRESS>    at new NodeError (node:internal/errors:371:5)
    at throwExportsNotFound (node:internal/modules/esm/resolve:429:9)
    at packageExportsResolve (node:internal/modules/esm/resolve:683:3)
    at resolveExports (node:internal/modules/cjs/loader:482:36)
    at Function.Module._findPath (node:internal/modules/cjs/loader:522:31)
    at Function.Module._resolveFilename (node:internal/modules/cjs/loader:919:27)
    at Function.Module._load (node:internal/modules/cjs/loader:778:27)
    at Module.require (node:internal/modules/cjs/loader:999:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object.<anonymous> (/Users/sce9sc/Documents/Work/testNknLatestNode/node_modules/nkn-sdk/lib/multiclient/multiclient.js:8:35) {
  code: 'ERR_PACKAGE_PATH_NOT_EXPORTED'
}

I've got this issue before in another project. It seems to be a npm issue. The solution that worked for me was to remove node_moodules and package.lock/yarn.lock (depending on which on you use) and do npm install/yarn again.

I tried it in a new and clean project with no module and no package.json and still got the same message.

I tried a few nodejs version and confirmed this issue with some new nodejs versions. This should have been fixed in v1.2.2 now.


Use kubeadm/v1beta3 API for Kubernetes 1.22+ clusters

What this PR does / why we need it:
kubeadm has added a new v1beta3 API in v1.22. They recommend using it for 1.22+ clusters. The v1beta2 API will be deprecated in a future release.
A list of changes since v1beta2 can be found here: https://kubernetes.io/docs/reference/config-api/kubeadm-config.v1beta3/
Besides the new bootstraptoken API, there are no other changes that affect us.
Since kubeadm APIs are still part of kubernetes/kubernetes, this PR ports the new APIs to our codebase to avoid vendoring k/k.
Does this PR introduce a user-facing change?:
Use kubeadm/v1beta3 API for Kubernetes 1.22+ clusters

/retest


TryToResolve should accept a Sequence and not a List

As suggested by @alessiostalla

We used Iterable in the end


Save matplotlib plots as pdf

Currently it is only possible to save matplotlib plots as png in the script 07_meld_pipeline.Rmd .
Using dev = c("png","pdf"), in the setup chunk results in the following error:
Error: RuntimeError: invalid 'path' argument

It looks like a pathname issue, but considering everything works fine with the png files the real issue must be elsewhere.

It is now integrated to the main reticulate package, closing this.


com.github.dockerjava.api.exception.DockerClientException: Failed to parse dockerCfgFile: /Users/sscarduzio/.docker/config.json

Now that we managed to sidestep the JVM crash  (thanks!!), a new issue bubbled up.
org.testcontainers.containers.ContainerFetchException: Can't get Docker image name from org.testcontainers.images.builder.ImageFromDockerfile@1c6e0a08

	at org.testcontainers.containers.GenericContainer.getDockerImageName(GenericContainer.java:739)
	at org.testcontainers.containers.GenericContainer.logger(GenericContainer.java:278)
	at org.testcontainers.containers.GenericContainer.start(GenericContainer.java:162)
	at org.testcontainers.containers.GenericContainer.starting(GenericContainer.java:531)
	at org.testcontainers.containers.FailureDetectingExternalResource$1.evaluate(FailureDetectingExternalResource.java:29)
	at org.junit.rules.RunRules.evaluate(RunRules.java:20)
	at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
	at org.junit.runner.JUnitCore.run(JUnitCore.java:137)
	at com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:68)
	at com.intellij.rt.execution.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:51)
	at com.intellij.rt.execution.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:242)
	at com.intellij.rt.execution.junit.JUnitStarter.main(JUnitStarter.java:70)
Caused by: com.github.dockerjava.api.exception.DockerClientException: Failed to parse dockerCfgFile: /Users/sscarduzio/.docker/config.json
	at com.github.dockerjava.core.DefaultDockerClientConfig.getAuthConfigurations(DefaultDockerClientConfig.java:295)
	at com.github.dockerjava.netty.exec.AbstrDockerCmdExec.getBuildAuthConfigs(AbstrDockerCmdExec.java:37)
	at com.github.dockerjava.netty.exec.BuildImageCmdExec.resourceWithOptionalAuthConfig(BuildImageCmdExec.java:27)
	at com.github.dockerjava.netty.exec.BuildImageCmdExec.execute0(BuildImageCmdExec.java:107)
	at com.github.dockerjava.netty.exec.BuildImageCmdExec.execute0(BuildImageCmdExec.java:18)
	at com.github.dockerjava.netty.exec.AbstrAsyncDockerCmdExec.execute(AbstrAsyncDockerCmdExec.java:56)
	at com.github.dockerjava.netty.exec.AbstrAsyncDockerCmdExec.exec(AbstrAsyncDockerCmdExec.java:21)
	at com.github.dockerjava.netty.exec.AbstrAsyncDockerCmdExec.exec(AbstrAsyncDockerCmdExec.java:12)
	at com.github.dockerjava.core.command.AbstrAsyncDockerCmd.exec(AbstrAsyncDockerCmd.java:21)
	at org.testcontainers.images.builder.ImageFromDockerfile.resolve(ImageFromDockerfile.java:124)
	at org.testcontainers.images.builder.ImageFromDockerfile.resolve(ImageFromDockerfile.java:30)
	at org.testcontainers.utility.LazyFuture.getResolvedValue(LazyFuture.java:20)
	at org.testcontainers.utility.LazyFuture.get(LazyFuture.java:27)
	at org.testcontainers.containers.GenericContainer.getDockerImageName(GenericContainer.java:737)
	... 11 more
Caused by: java.io.IOException: Invalid Auth config file
	at com.github.dockerjava.core.AuthConfigFile.loadConfig(AuthConfigFile.java:146)
	at com.github.dockerjava.core.DefaultDockerClientConfig.getAuthConfigurations(DefaultDockerClientConfig.java:292)
	... 24 more

I recently logged in into my docker Hub, maybe that's the cause?
$ docker -v
Docker version 17.06.0-ce, build 02c1d87
I'm running OSX El Capitan.

This is my config.json
{
        "auths": {
                "https://index.docker.io/v1/": {}
        },
        "credsStore": "osxkeychain"
}

I think it's known issue, in docker-java ( https://github.com/docker-java/docker-java/issues/806 ). We upgraded docker-java dependency to 3.0.12 in 1.4.0 and going to release it really soon :D
Jitpack is also an option to try, see https://www.testcontainers.org/usage.html#maven-dependencies

Oh nice, I didn't know jitpack.. BTW I tested removing that config.json file for the moment. Now it works again. Woohoo! 🚀

@sscarduzio nice! AFAIK "credsStore": "osxkeychain" was the cause, docker-java wasn't expecting fields other than they know :D

I'm closing as a duplicate of https://github.com/testcontainers/testcontainers-java/issues/293, to be released as 1.4.0, but feel free to reopen if it doesn't help

I have the same problem with oracle-xe, but there the latest version of that is 1.2.1 Will there be a 1.4.x for oracle-xe as well, containing this fix?

There seems to be a 1.3.1, but it's not yet in MavenCentral.
As a fallback, you can always resort on the Jitpack repository.
I'll look into upgrading to 1.4.x.


Release new version of raiutils with OD support

Description

Checklist

[ ] I have added screenshots above for all UI changes.
[ ] I have added e2e tests for all UI changes.
[ ] Documentation was updated if it was needed.

Codecov Report

Merging #2244 (e51d13c) into main (8541be6) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #2244   +/-   ##
=======================================
  Coverage   91.88%   91.88%
=======================================
  Files          35       35
  Lines         702      702
=======================================
  Hits          645      645
  Misses         57       57

Flag
Coverage Δ

unittests
91.88% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

raiutils/raiutils/version.py
100.00% <100.00%> (ø)


NodePort Local in Antrea

Describe what you are trying to solve
Current NodePort implementation using kube-proxy provides an easy way to expose a service’s port on a kubernetes’ node’s port. Irrespective of where the actual pod runs, nodeport exposes a port on every node. While, this is a quick and easy solution to create a service of type loadbalancer (and any application pod) for external connectivity, it has the following drawbacks:

Exposes the port on each node, even if the pods are not on it. This means that a secondary hop of load balancing is carried by kube-proxy to re-distribute the traffic amongst the pods.

Non deterministic if ports at a given node are not available. Since it exposes the same port on each node, there might be possibilities where a given port is not free on a node. There’s no way to control this behavior other than running health checks on each node:port to figure out if it’s working.

Session persistence does not work. For load balancers wanting to do consistent hash or session persistence directly to the application pod from outside the kubernetes clusters, nodeport isn’t a feasible solution.

It requires a range of ports to be exposed in all kubernetes nodes. This ends up hitting the port range limitations as the number of services (of type nodePort) keeps on increasing.

Describe the solution you have in mind
Proposed here is a solution that solves the above mentioned problems using a new concept called “NodePortLocal” (NPL) implemented in the Antrea Node Agent.
NodePortLocal provides an easy way to expose Kubernetes PODs on a Node's IP:Port.
In order to accomplish this task, NPL programs iptable DNAT rules on each node.
Test plan
NPL can be tested without the need of an ingress controller. Based on the user scenario, a service can be annotated with  kubernetes.io/nplep: true and the traffic workflow can be tested. I will add more details on this as we iterate the PRs.
Additional context
The design document can be accessed here: https://docs.google.com/document/d/1SoAKBvbbt2QMnGmt53E8ja0ZxCv_T4r1zLVnMzpykTg/edit

It's probably worth calling out that the behavior you mentioned about kube-proxy varies depending on what is set on the externalTrafficPolicy field. Using externalTrafficPolicy=Local with kube-proxy does address many (if not all?) of the issues you listed here.

+1 on what @andrewsykim said.

When externalTrafficpolicy=Local is set, a healthcheckNodePort will be allocated, to which LB can configure healthchecks to decide if backend pod is running on that node. healthCheckNodePort can be used to customize that port: https://kubernetes.io/docs/tasks/access-application-cluster/create-external-load-balancer/#preserving-the-client-source-ip

it's not completely non-deterministic as users can provide the nodePort field, but you're right that it's possible one port is not available on one of the nodes;

at least according to the doc: https://kubernetes.io/docs/concepts/services-networking/service/#proxy-mode-ipvs. SessionAffinity(and a corresponding timeout) can be used to control the persistence behavior on kube-proxy. If it works as documented, then as long as AVI has session affinity configured, the expected behavior could be achieved;

Added in #1459


Make fix for quotemark rule more readable

PR checklist

[ ] Addresses an existing issue: #0000
[ ] New feature, bugfix, or enhancement

[ ] Includes tests

[ ] Documentation update

What changes did you make?
In vscode, the string for implementing the fix looks like
Fix "' should be ""

and this isn't very readable. So I added spaces, punctuation to make the fix-text more readable.
Is there anything you'd like reviewers to focus on?
Do you have some other ideas for how to make the fix-text more readable?

Thanks for your interest in palantir/tslint, @aozgaa! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.

You'd need to fix the tests as well.

I don't think this rule's text should be changed because of the way another app displays it.


BAH-1577 Create a doctor and add their privileges/role using CSV

Please add in the README about all the inserts in initilizer liquibase being temporary (only for testing)

Sure Nouman,We will update.


griddle-react 0.2.15 is a patch release with incompatible changes, should be 0.3.0?

Glad to see the react 0.14 updates, but releasing it as a patch release broke lots of existing builds -- npm uses a caret in the saved version string by default. Because 0.2.14 -> 0.2.15 is just a "patch" release, npm tries to update.
Unfortunately, existing code and other dependencies that aren't ready for React 0.14 prevent me from updating. This broke the build until I traced it to griddle-react's new dependency.
Normally, I'd just pin the version to 0.2.14 in my own package.json and call it good, but I'm sure I'm not the only one this affected.  This is my public service announcement and a request for a better fix.
See this for an example of another project and how they resolved it by making a new patch release reverting to React 0.13 and then releasing the React 0.14 update as a minor version bump.
https://github.com/callemall/material-ui/blob/master/CHANGELOG.md#0125

You're completely right. I'll get moving on pushing what's currently 0.2.15 into 0.3.0 and moving over a downgraded version into 0.2.16.
Thanks for putting together the issue!

Alright -- an older version has been pushed to 0.2.16 to support React 0.13, while the current version is now 0.3.0.
<EMAIL_ADDRESS><EMAIL_ADDRESS>Thanks again for creating the issue!


Remove useless PhpDoc comments and minor improvements

Remove useless PhpDocs comments.
Improving Twig locales() function.

@20uf I think I understand your downvote ... but don't be afraid because this won't break PHPStorm autocompletion. Most of these annotations are indeed useless because PHPStorm already can infer their types (through the constructor or other assignments). There's at least one string annotation that we'll loose, but string typehint is not very useful anyway.
Moreover, we now use PHP 7.1, so we can typehint everything and remove all PHPdoc.

Thank you @yceruto.

The typehint does not completely replace the annotations and the strong typing is not fully respected, for example the type of return is not specified.


npm module doesn't require folder_name

Description
I install and require elasticsearch in my code, but the compiled js errored with
Uncaught Error: Cannot find module 'elasticsearch/src/lib/loggers' from 'elasticsearch/src/lib/log.js'

Expected behavior
It should require node_modules/elasticsearch/src/lib/loggers/index.js
Actual behavior
As error message.
demo project
run npm start and open development tool console for the error message.
If I modify the module, fix the require syntax to require("./loggers/index.js"),
it success and error at next folder requirement.
I've tried to alter the brunch config's like
javascripts: {
      joinTo: {
        'app.js': /^(app|node_modules)/
      }
}

but still no luck.
Environment

Brunch: 2.9.1
Node: v7.0.0
NPM: 4.0.1
Operating system: macOS 10.12.1

package.json contents
{
  "name": "brunch-app",
  "version": "0.1.0",
  "scripts": {
    "start": "brunch watch --server",
  },
  "dependencies": {
    "elasticsearch": "^12.0.1"
  },
  "devDependencies": {
    "auto-reload-brunch": "^2.0.0",
    "brunch": "^2.0.0",
    "clean-css-brunch": "^2.0.0",
    "css-brunch": "^2.0.0",
    "javascript-brunch": "^2.0.0",
    "uglify-js-brunch": "^2.0.0"
  }
}

brunch config contents
exports.files = {
  javascripts: {joinTo: 'app.js'},
  stylesheets: {joinTo: 'app.css'},
  templates: {joinTo: 'app.js'},
  plugins: {
    babel: {}
  },
  npm: {
    enabled: true
  }
};

Hi, @taiansu! Thanks for reaching out!
Your issue is related to problems with deppack module.
Here are other issues: deppack related issues.
We're working on it. I believe we will fix that as soon as it possible.

@Anaphase currently we are aimed to replace deppack with another bundler, possibly rollup or browserify. That will fix a lot of issues, including this one. However, it requires a rewrite of Brunch core that is happening in #1664.

+1 for using rollup as a bundler. (tree-shaking for the win!)
Actually, it would be great if bundlers were pluggable!

@taiansu did you manage to get this error solved?

@rafbgarcia Ah no, I just switch to other packaging tools after that.

I just lost many hours I guess related to this. Using Brunch because it's by default with Phoenix but I just can't get a simple npm module that require some lodash function to work properly (unless I import them myself in my app.js ...)
And this issue is open for a year and a half ? I was excited to use Brunch because it seems simple and I hate Webpack but I guess I've no choice.


[submission] Vanilla JavaScript

Did some light Q&A. Would appreciate some feedback!

@staltz I think this is a complete, working submission.  Any chance I could get this reviewed or merged? Thanks!


(core) - Skip registration of non-function nodes

fixes: #295

Thank you! I sincerely appreciate the prompt reply and then fix


Optimize Updater Implementations

Some early ND4J profiler results on the updater implementations (running on a simple MLP):

Mixed order results in Adam (x2), Nesterov (x1), RMSProp (x1)
Overall: seems like updaters are indeed a bottleneck

Preliminary results on a simple 3 layer MLP (layer size: 512) + Adam updater; Cuda 8 (Windows, Titan X)
org.deeplearning4j.nn.multilayer.MultiLayerNetwork.fit:1475  101006 us
org.deeplearning4j.nn.updater.MultiLayerUpdater.update:141  72000 us  (71%)

Interestingly, those results don't seem to depend much on layer size... perhaps this is due to the kernel launch time on windows? (Linux Titan X results from @crockpotveggies are exactly the same)
https://gist.github.com/AlexDBlack/849fe516e7ae6ec0924d3dbabb26a66b
Update: using new "--profile" option benchmark repo: Windows, Cuda 8, Titan X, no cuDNN
AlexNet, momentum:
org.deeplearning4j.nn.multilayer.MultiLayerNetwork.fit:1051  160016 us
org.deeplearning4j.nn.updater.MultiLayerUpdater.update:141  87500 us
AlexNet, Adam:
org.deeplearning4j.nn.multilayer.MultiLayerNetwork.fit:1051  208016 us
org.deeplearning4j.nn.updater.MultiLayerUpdater.update:141  135500 us
AlexNet, SGD:
org.deeplearning4j.nn.multilayer.MultiLayerNetwork.fit:1051  112000 us
org.deeplearning4j.nn.updater.MultiLayerUpdater.update:141  40000 us

How to actually fix this:

Fix the mixed orders/bad strides in the updaters, like Adam and Nesterovs
In DL4J, combine updaters where possible (i.e., for params and gradients with contiguous updater configuration)

https://github.com/deeplearning4j/nd4j/pull/1767
Remaining improvements will likely have to come on the DL4J side - ND4J side looks good currently in profiler: https://github.com/deeplearning4j/nd4j/issues/1766


Why doesn't work emmet media query? Specifically @m and @media?

Why doesn't work emmet media query? Specifically @m and @media?

VSCode version: 1.32.3
Windows 8.1x64:

Steps to Reproduce:

I want to using emmet
When i press tab @media or @m i was expecting this.

Look here please:

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

Hi @baxebu, I realize you opened this issue over a year ago, but I noticed on 1.49.3 that there's a @media screen abbreviation that is now suggested if you type @m inside of a style tag within an HTML file. Does this work for your purposes?

Nvm, labelling this as a bug because @m and @media both expand properly on other editors.

Closing in favour of the more general issue #74505


Undefined variable & index

Hi,
how can I fix this logfile entrys?
[Thu Dec 01 10:24:07 2011] [error] [client <IP_ADDRESS>] PHP Notice: Undefined variable: args in /var/www/vhosts/quqii.com/httpdocs/api/twitter/EpiTwitter.php on line 19, referer: https://www. ...
[Thu Dec 01 10:24:07 2011] [error] [client <IP_ADDRESS>] PHP Notice: Undefined index: port in /var/www/vhosts/quqii.com/httpdocs/api/twitter/EpiOAuth.php on line 146, referer: https://www. ...
[Thu Dec 01 10:28:21 2011] [error] [client <IP_ADDRESS>] PHP Warning: Invalid argument supplied for foreach() in /var/www/vhosts/quqii.com/httpdocs/api/twitter/EpiOAuth.php on line 116, referer: https://twitter.com/oauth/authorize
I tried it with my poor knowledge, but its not working after my correction :-)
Kind regards,
Carlo

Unsure what the problem is. There's nothing referencing args on line 19 of EpiTwitter.
https://github.com/jmathai/twitter-async/blob/master/EpiTwitter.php#L19
The other lines don't match up either. Are you using the latest version of the code from Github?


Motion & Fitness remains unavailable after being switched on in Privacy & Security

Bug summary
When Motion & Fitness is switched off in Privacy & Security settings, requesting the permission returns 'unavailable'. If I send the user to Privacy & Security, and they turn on Motion & Fitness, then the user taps the back to app button, then the app is not reset and the permission continues to return unavailable. Restarting the app is required to resolve the issue.
This could be a bug with this package or possibly an iOS bug?
Note this issue is on a physical device running iOS 16.7.1. Testing on a simulator is not possible with Motion & Fitness permissions.
Library version
3.8.0
Environment info
System:
    OS: macOS 13.6
    CPU: (10) arm64 Apple M1 Pro
    Memory: 543.77 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.12.1 - ~/.nvm/versions/node/v18.12.1/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v18.12.1/bin/yarn
    npm: 8.19.2 - ~/.nvm/versions/node/v18.12.1/bin/npm
    Watchman: Not Found
  Managers:
    CocoaPods: 1.12.1 - /Users/xxxxx/.rbenv/shims/pod
  SDKs:
    iOS SDK:
      Platforms: DriverKit 23.0, iOS 17.0, macOS 14.0, tvOS 17.0, watchOS 10.0
    Android SDK:
      API Levels: 26, 27, 28, 29, 30, 31, 32, 33, 34
      Build Tools: 29.0.2, 30.0.2, 30.0.3, 31.0.0, 32.0.0, 33.0.0, 33.0.1
      System Images: android-26 | Google APIs ARM 64 v8a, android-26 | Google APIs Intel x86 Atom_64, android-28 | Google APIs ARM 64 v8a, android-29 | Google APIs ARM 64 v8a, android-30 | Google APIs ARM 64 v8a, android-31 | Google APIs ARM 64 v8a, android-31 | Google APIs Intel x86_64 Atom, android-31 | Google Play ARM 64 v8a, android-32 | Google APIs ARM 64 v8a, android-33 | Google APIs ARM 64 v8a
      Android NDK: Not Found
  IDEs:
    Android Studio: 2021.2 AI-212.5712.43.2112.8609683
    Xcode: 15.0/15A240d - /usr/bin/xcodebuild
  Languages:
    Java: 11.0.18 - /opt/homebrew/opt/openjdk@11/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: 18.2.0 => 18.2.0
    react-native: 0.71.12 => 0.71.12
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Steps to reproduce

Install app on iOS physical device.
Ensure Motion & Fitness is disabled in Privacy & Security settings
Call requestPermission (in code below) - user to sent to Privacy & Security -> Motion & Fitness
Switch on Motion & Fitness
Click the back button to return to the app
Call requestPermission - observe that user is sent to Privacy & Security -> Motion & Fitness again - check permission returned 'unavailable'
Kill and restart the app
Call request permission - observe user is not sent to the app settings where they can enable Motion & Fitness for the app

Reproducible sample code
import {
  check,
  PERMISSIONS,
  PermissionStatus,
  request,
} from 'react-native-permissions';

const requestPermission = useCallback(async () => {
    const status = await checkPermission();
    const motionAndFitnessBlocked = status === 'blocked';
    const motionAndFitnessUnavailable = status === 'unavailable';

    if (motionAndFitnessUnavailable) {
      await Linking.openURL('App-prefs:Privacy&path=MOTION');
      return;
    }

    if (motionAndFitnessBlocked) {
       await Linking.openSettings();
      return;
    }

    try {
      return await request(PERMISSIONS.IOS.MOTION);
    } catch (error) {
      return 'unavailable';
    }
  }, []);

const checkPermission = async () => {
    try {
      const result = await check(PERMISSIONS.IOS.MOTION);
      return result;
    } catch (error) {
      return 'unavailable' as PermissionStatus;
    }
  };

I tried it with a simple experiment and this appear to be an iOS issue. I removed all code in motion permission check except this:
switch ([CMMotionActivityManager authorizationStatus]) {
  case CMAuthorizationStatusNotDetermined:
    return resolve(RNPermissionStatusNotDetermined);
  case CMAuthorizationStatusRestricted:
    return resolve(RNPermissionStatusRestricted);
  case CMAuthorizationStatusDenied:
    return resolve(RNPermissionStatusDenied);
  case CMAuthorizationStatusAuthorized:
    return resolve(RNPermissionStatusAuthorized);
}

Then:

I perform a check
It replies with CMAuthorizationStatusNotDetermined (which is later mapped to denied in this library - it's the android naming)
I open Privacy & Security settings to set Motion & Fitness to off
I go back to the app, perform a check again, it still replies with to CMAuthorizationStatusNotDetermined (denied)
I try to perform a request, it does not prompt the user, reply with blocked
After an app restart, everything is OK, the iOS API return the correct value / perform correctly

The issue is similar the other way around:

I open Privacy & Security settings to set Motion & Fitness to off
I perform a check
It replies with CMAuthorizationStatusDenied (which is later mapped to unavailable in this library)
When I try to request the permission, it does not prompt the user, reply with blocked
I open Privacy & Security settings to set Motion & Fitness to on
I go back to the app, perform a check again, it still replies with to CMAuthorizationStatusDenied (unavailable)
When I try to request the permission, it does not prompt the user, reply with blocked
After an app restart, everything is OK, the iOS API return the correct value / perform correctly

iOS restarts the app when you change the Motion & Fitness setting for the app itself - perhaps they forgot to do this with the Privacy & Security equivalent.
Thanks for looking into the issue.

I'm closing this as unfortunately we can't do anything 😕


Change default network provider

I opened an issue for weave problem https://github.com/weaveworks/weave/issues/3239, we can use flannel until it will be fixed.
Signed-off-by: Lukianov Artyom<EMAIL_ADDRESS>

ok to test

ci test please

retest this please

retest this please

retest this please

Confirmed locally that everything works. Merging to unblock people on reprovisioning.


About the number of slots

Hi, I have a question about the parameter nslots in the code.
Usually, nslots = dim * dim, then we rotate the ciphertext with nslots.
But if we have the ciphertext with more slots,  we cannot do the following operations like rotate.
For example, we have ctxt with 300 slots, but for our 16*16 matrix, we only need 256 slots. You know, in the BGV ciphertext, we cannot choose the exact slots in the ciphertext.

Hi, @K-miran . it seems that the number of slots in the ciphertext equals to HEmatpar.nslots. So for the 4*4, 16*16, 64*64 matrix, nslots can only be 16, 256, 4096? If we have the ciphertext with 300 slots for the 16*16 matrix, what should we do? I mean, in the following rotations on ciphertexts, we have to change all of the indexes and mappings? In this case, we have extra 44 slots.
Thanks in advance!


Fix incorrect ElementalArea.OwnerClassName on save

OwnerClassName in the ElementalArea table can become incorrect in certain scenarios*, and there is currently no way to fix this short of manually editing the database.  An incorrect OwnerClassName breaks the behaviour of BaseElement->getPage(), which breaks all kinds of other things in BaseElement that depend on it.
For example BaseElement->canView() will always return false.  I suspect a lot of people are working around this issue by putting canView{ return true } in all their block types.  That's what we were doing until I decided to really dig into the root cause of why BaseElement->canView() was failing.
This change modifies ElementalAreasExtension->ensureElementalAreasExist(), to verify that the OwnerClassName is correct, and re-saves it with the correct value if it is wrong.
*For example, apply ElementalPageExtension to Page, create a Page, then switch ElementalPageExtension to OtherPage, and switch your created page type to OtherPage.  The OwnerClassName in the DB will still be "Page" and there is no way to fix this currently.

Thanks for the patch @bcairns! There'll be a bunch of linting issues - you can probably run phpcbf across your code to clear that up.

@bcairns would you have time to finalise this? It's nearly there!

Thanks for the reminder @michalkleiner . We spoke on Slack and I offered to finalise this PR. I haven't had a chance to get around to it yet though.

I've created a new PR from the latest 4 branch
I've done the code in outside of ::ensureElementalAreasExist() as suggested

#816 has been merged, so closing this one out.


Line endings

in https://github.com/Zac-HD/flake8-trio/commit/490a77585f40c19b0fb06d494d4317a6e505f444 you converted all files to CRLF line endings. Was this intentional?
It's not a huge problem for me, but it makes diffs on my local machine have ugly ^M's and apparently the remotes are stored with LF endings so whenever I try to diff or rebase against a remote branch it's a mess. So unless you don't mind I'd like to convert all files back to LF and add a .gitattributes file to hopefully stop it from happening in the future. But I'll wait with it until there's no open PR's to avoid merging nightmares.
If it was intentional I'll add a .gitattributes file indicating CRLF is preferred, and see if I can get git to play nicely with that on my local machine.

Definitely an accident 😅, fix would be appreciated!


Web browser in VNM fails to render some webpages

Tom Johnstone from Swinburne will look into this. He says it is a known problem with web browsers accessed via VNC.

Solved :) We found out that it's possible to add more shm space in docker. VS code had the same problem. Solution is to start with more shm:
sudo docker run --shm-size=1gb -it --privileged --name neuromachine -v ~/vnm:/vnm -e USER=user -p 8080:8080 vnmd/neuromachine:20210826


Link to German 3.0 BY port is broken on German deed

Describe the bug
The link to the German 3.0 port is broken in the German deed.
To Reproduce
Steps to reproduce the behavior:

Go to https://creativecommons.org/licenses/by/3.0/de/
Click on Lizenz
The corresponding URL (https://creativecommons.org/licenses/by/3.0/de/legalcode.de) leads nowhere.

Expected behavior
The correct URL for the legal code is https://creativecommons.org/licenses/by/3.0/de/legalcode
Additional context
I would guess this is a Transifex problem, but I am not sure. This is a lower priority item, given that it relates to pre-4.0 versions but I wanted to track it so it is not lost. We got an email about in Freshdesk.

This also affects BY-ND, we got a report of it in Freshdesk. https://creativecommons.org/licenses/by-nd/3.0/de/


Could not fix range for AssignOp at line X

It's running into this error, here X > total line count. Any ideas or thoughts would be greatly appreciated.

Most likely it's rewriting it internally and causing the line numbers not
to match. Can you share the code that breaks it?
On Fri, Jan 22, 2016 at 10:57 AM patelsan<EMAIL_ADDRESS>wrote:

It's running into this error, here X > total line count. Any ideas or
thoughts would be greatly appreciated.
—
Reply to this email directly or view it on GitHub
https://github.com/eventualbuddha/decaffeinate/issues/115.

Ping @patelsan.

@eventualbuddha Sorry for the late reply, I was able to zero down on the line causing it - seems to be related to RegEx, here is a snippet to reporduce this
class ControllerBase
  @register:(name = null)->
    name ?= @name || @toString().match(/function\s*(.*?)\(/)?[1]

@patelsan Odd, that example [seems not to crash](http://eventualbuddha.github.io/decaffeinate/repl/#?evaluate=false&code=class ControllerBase
  %40register%3A(name %3D null)->
    name %3F%3D %40name || %40toString().match(%2Ffunction\s*(.*%3F)\(%2F)%3F[1]
) with the latest version, although it leaves name ?= … which is clearly wrong (#38). Perhaps that's not the part that causes it?

@patelsan #38 is now fixed, but your code also showed causes another bug. I opened #118 to track it.

And now #118 is fixed. Added #119 to track the remaining issue from this one.


Support Swift 4.2

When can we expect an update for swift 4.2? Do you need any help with this?

I've released v0.6.0.
Please check it out.


Tests: drop testAutolinkOutputPath

This somehow uses the wrong Toolchain.  Using GenericUnixToolchain on
Windows or Darwin is not guaranteed to work.  This is required to enable
the test suite on Windows.

Actually, @artemcm was able to help isolate where the problem is getting introduced and we can keep this enabled!


[DOC] Publishing docs on cuda_memory

Feature request

It looks like __cuda_memory__ is used for some things internally and is checked in a few places. There even appear to be docstrings about it within the code. However it doesn't seem to show up in the docs online. Would be good to publish these to the webpage.

@jakirkham Do you still see any need for this? I'm not sure I'd want to publish anything about it without good reason, as I don't think there's been much thinking through it as an external API.

I don't think so. At the time it wasn't clear if we should be using this in other libraries like RMM for better interoperability with Numba. Though I think __cuda_array_interface__ and other Numba & RMM improvements have obviated the need here


LNURLP as an optional field

Return success for lnurlp queries when username is undefined or null instead of returning 400 error message in accountDefaultWallet query
This should resolve "Account creation temporarily unavailable" error in UI

Closing as the issue is not the lnurlp but querying be username


dumper-fuzzy barfs a hairball now and then

25-dumper-fuzzy.js sporadically barfs a hairball like the one shown below.
Can anyone give a few tips/hints how to track this down without perhaps ripping fast-check open and inspecting the entrails there?
     Property failed after 69 tests
{ seed:<PHONE_NUMBER>, path: "68:2:2:2:1:3:2:2:4:3:3:3:3:3:3:4:4:3:3:3:3:4:5:3:10:10:9:8:8:8:8:8:8:8:9:9:9:9:9:9:9:9", endOnFailure: true }
Counterexample: [{"":{"$,":{}}},{"flowLevel":0}]
Shrunk 41 time(s)
Got error: AssertionError [ERR_ASSERTION]: { '': { '$': null, null: {} } } deepStrictEqual { '': { '$,': {} } }

Stack trace: AssertionError [ERR_ASSERTION]: { '': { '$': null, null: {} } } deepStrictEqual { '': { '$,': {} } }
    at W:\Users\Ger\Projects\sites\library.visyond.gov\80\lib\js\js-yaml\test\25-dumper-fuzzy.js:44:16
    at Property.predicate (W:\Users\Ger\Projects\sites\library.visyond.gov\80\lib\js\js-yaml\node_modules\fast-check\lib\check\property\Property.generated.js:34:105)
    at Property.run (W:\Users\Ger\Projects\sites\library.visyond.gov\80\lib\js\js-yaml\node_modules\fast-check\lib\check\property\Property.generic.js:19:31)
    at runIt (W:\Users\Ger\Projects\sites\library.visyond.gov\80\lib\js\js-yaml\node_modules\fast-check\lib\check\runner\Runner.js:84:32)
    at check (W:\Users\Ger\Projects\sites\library.visyond.gov\80\lib\js\js-yaml\node_modules\fast-check\lib\check\runner\Runner.js:175:11)
    at Object.assert (W:\Users\Ger\Projects\sites\library.visyond.gov\80\lib\js\js-yaml\node_modules\fast-check\lib\check\runner\Runner.js:179:15)
    at Context.<anonymous> (W:\Users\Ger\Projects\sites\library.visyond.gov\80\lib\js\js-yaml\test\25-dumper-fuzzy.js:38:8)
    at callFn (W:\Users\Ger\Projects\sites\library.visyond.gov\80\lib\js\js-yaml\node_modules\mocha\lib\runnable.js:387:21)
    at Test.Runnable.run (W:\Users\Ger\Projects\sites\library.visyond.gov\80\lib\js\js-yaml\node_modules\mocha\lib\runnable.js:379:7)
    at Runner.runTest (W:\Users\Ger\Projects\sites\library.visyond.gov\80\lib\js\js-yaml\node_modules\mocha\lib\runner.js:535:10)
    at W:\Users\Ger\Projects\sites\library.visyond.gov\80\lib\js\js-yaml\node_modules\mocha\lib\runner.js:653:12
    at next (W:\Users\Ger\Projects\sites\library.visyond.gov\80\lib\js\js-yaml\node_modules\mocha\lib\runner.js:447:14)
    at W:\Users\Ger\Projects\sites\library.visyond.gov\80\lib\js\js-yaml\node_modules\mocha\lib\runner.js:457:7
    at next (W:\Users\Ger\Projects\sites\library.visyond.gov\80\lib\js\js-yaml\node_modules\mocha\lib\runner.js:362:14)
    at Immediate.<anonymous> (W:\Users\Ger\Projects\sites\library.visyond.gov\80\lib\js\js-yaml\node_modules\mocha\lib\runner.js:425:5)
    at runCallback (timers.js:794:20)
    at tryOnImmediate (timers.js:752:5)
    at processImmediate [as _immediateCallback] (timers.js:729:5)

Hint: Enable verbose mode in order to have the list of all failing values encountered during the run
  Error: Property failed after 69 tests
  { seed:<PHONE_NUMBER>, path: "68:2:2:2:1:3:2:2:4:3:3:3:3:3:3:4:4:3:3:3:3:4:5:3:10:10:9:8:8:8:8:8:8:8:9:9:9:9:9:9:9:9", endOnFailure: true }
  Counterexample: [{"":{"$,":{}}},{"flowLevel":0}]
  Shrunk 41 time(s)
  Got error: AssertionError [ERR_ASSERTION]: { '': { '$': null, null: {} } } deepStrictEqual { '': { '$,': {} } }

  Stack trace: AssertionError [ERR_ASSERTION]: { '': { '$': null, null: {} } } deepStrictEqual { '': { '$,': {} } }
      at W:\Users\Ger\Projects\sites\library.visyond.gov\80\lib\js\js-yaml\test\25-dumper-fuzzy.js:44:16
      at Property.predicate (node_modules\fast-check\lib\check\property\Property.generated.js:34:105)
      at Property.run (node_modules\fast-check\lib\check\property\Property.generic.js:19:31)
      at runIt (node_modules\fast-check\lib\check\runner\Runner.js:84:32)
      at check (node_modules\fast-check\lib\check\runner\Runner.js:175:11)
      at Object.assert (node_modules\fast-check\lib\check\runner\Runner.js:179:15)
      at Context.<anonymous> (test\25-dumper-fuzzy.js:38:8)

  Hint: Enable verbose mode in order to have the list of all failing values encountered during the run
      at Object.throwIfFailed (node_modules\fast-check\lib\check\runner\utils\RunDetailsFormatter.js:108:11)
      at Object.assert (node_modules\fast-check\lib\check\runner\Runner.js:183:31)
      at Context.<anonymous> (test\25-dumper-fuzzy.js:38:8)

https://github.com/dubzzz/fast-check/blob/master/documentation/1-Guides/Tips.md#replay-after-failure is this what you are asking for?

👍
That should do it for now; I do wonder why the fuzzy test generator only triggers this once in a while, but I guess it's tickling an edge case there, maybe in test generation, maybe in the library itself.
Time to find out...

I think fuzzy data is pseudo-random, and some rare edge case may not happen every time.

Indeed, it's a game of change every time, but when I filed this issue, the reported test failure (bug) was only happening very rarely, about once every 20 test runs (make test).
I increased the number of fuzzer runs (https://github.com/GerHobbelt/js-yaml/commit/587b02c0d91901e00571bbe086e06d403084bb57) not to have a guarantee but merely to increase the likelyhood that any bug triggered by the fuzzer would get reported on a make test run.
When a weird bug is reported more often it's more likely to be investigated than a fluke that happens rarely -- I know I shouldn't treat error reports like that, but sometimes I was working on something else and then the bug would turn up, but it would quickly scroll off screen and I would have a WTF moment. :wink:
Now, at 3000 rounds, I feel more certain that the many edges of js-yaml get tested more thoroughly and will be reported often enough that I will notice them, even when I'm really looking for other things.
Let's say I'm trying to mix the human condition with the technical reality of (pseudo)random-driven test suites like 25-dumper-fuzzy. :smile:


refactor(torii-server): combine tonic and warp

Refactor of the warp server to allow for better request forwarding of gRPC requests. Instead of using warp to combine these two services, this PR uses hyper.
torii-grpc's server route has been removed.
NOTE
There's a bug in the tonic client where the URI path is not respected in Endpoint, but this issue doesn't exist if torii-client is compiled to wasm32-unknown-unknown.
See: https://github.com/hyperium/tonic/issues/1314

pretty janky solution tbh

I'm not sure how to check if the /graphql endpoint is working correctly or not. I've only tested it by opening the playground.
@broody can you help confirming if it works as expected?


PowerToys.Run does not launch apps through the user account that has started PowerToys

Microsoft PowerToys version
0.56.2
Running as admin

[X] Yes

Area(s) with issue?
PowerToys Run
Steps to reproduce
Ensure PowerToys is not running. Find PowerToys.exe, shift+rightclick, run as different user. After PowerToys started, Alt+Space to launch PowerToys.Run, start cmd. In cmd, enter whoami. The user shown is not the "different user" that was used to start PowerToys.
✔ Expected Behavior
PowerToys.Run should use the same user that was used to start PowerToys. To show expected behaviour, run cmd (without using PowerToys.Run) as a different user, and in cmd run whoami, the name of the different user should then be shown.
❌ Actual Behavior
PowerToys.Run ignores the user that was used to start PowerToys.
Other Software
cmd (from Microsoft Windows [Version 10.0.19044.1586])

It does. I think this is the case of using the shell as a way to start PowerToys Run as not administrator. We use the shell for it and this looks like a side effect from it. Many users scenarios are not well supported by PowerToys.

To add some details the "different user" that I'm running as has admin rights. Hopefully this can be a feature that can be added. Ideally I would like to run as both a different user and as an admin (in this scenario, I would hope to be prompted for a username and password at startup, but not be prompted afterwards).

@jaimecbernardo
Should we mark this as duplicate of #xx. This allows to run apps and commands as different user.

As I imagine in the past we keept the permission/context that runner has. But because then everything is open with admin permission if runner has it, we changed behavior to start pt run always in non elevated context.

As I imagine in the past we keept the permission/context that runner has. But because then everything is open with admin permission if runner has it, we changed behavior to start pt run always in non elevated context.

What I'm asking for is the option to override this default behaviour. I'd like to be able to start PowerToys, enter my alternate user credentials once, using a user that has admin rights, and not be prompted for UAC again any time I use PowerToys.Run until I close and reopen PowerToys.
I realise that what I've described will never be the default behaviour, as it'll not be seen as following "best practices" for managing user rights, but I would like for PowerToys to give me the option to do this. Believe it or not, the request is being made as it would on-balance permit greater user restrictions elsewhere in IT systems I work with, thereby improving security overall without negatively impacting on user workflows (I wouldn't want to go into the exact specifics for security reasons, I just wanted to highlight that this request will enable better workflows in some cases).


Fix OP image

Change OP image broken

I changed the file name and put the old name back in the README

Thank you!


NOISSUE - Remove twins-service mqtt dependency and publish notifs to nats

What does this do?
Currently, twins service depends on mqtt to publish notifications. This PR removes twins service mqtt dependency. Twins service now publishes notifications related to twins CRUD and state saving directly to NATS if user provides a valid Mainflux channel ID. If no channel id is provided, the service works but does not publish notifications.

Codecov Report

Merging #1042 into master will increase coverage by 0.02%.
The diff coverage is 90.9%.

@@            Coverage Diff            @@
##           master   #1042      +/-   ##
=========================================
+ Coverage   78.98%     79%   +0.02%
=========================================
  Files          95      95
  Lines        6623    6622       -1
=========================================
+ Hits         5231    5232       +1
+ Misses       1080    1078       -2
  Partials      312     312

Impacted Files
Coverage Δ

twins/service.go
41.42% <90.9%> (-0.35%)
:arrow_down:

things/service.go
90.44% <0%> (+1.47%)
:arrow_up:

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 a30a3b9...fcc9886. Read the comment docs.


Doxygen comment review

updated the header files by adding missing doxygen comments

That should complete the docs in a slightly verbose manner and I can review (simplifying and reducing) as needed.


Add SSTO Project from SpaceDock

This pull request was automatically generated by SpaceDock on behalf of Monniasza, to add SSTO Project to CKAN.
Please direct questions about this pull request to Monniasza.
Mod details:
name = /mod/2417/SSTO%20Project
author = Monniasza
abstract = SSTO project add wide range of high-performance spacecraft parts
license = MIT
Homepage =
description =
Supported mods:
SpaceY (both mods) - for large nuclear engines
KSP Interstellar Extended - for NSABRE and SuperJet oxygenless operation
Making History - Size 1.5 nuclear engine
HGR - high performance chemical engines
Planned supported mods:
HGR - Different size 1.5 nuclear engine
Sea Dragon - ultra-large nuclear engine
Behemoth Aerospace Engineering - high power large nuclear engines
Mk2 expansion - Mk2 nuclear engines
Mk3 Expansion = Mk3 nuclear engines
Bluedog Design Bureau - High performance chemical and nuclear engines
Recommended mods:
Kerbal Engineer Redux - to correctly display performance of nuclear jets
Parts:
Engines:

@Monniasza , the description consists of lists of other mods. What does this mod do?

@Monniasza , the description consists of lists of other mods. What does this mod do?

See here

@Monniasza , the description consists of lists of other mods. What does this mod do?

This mod adds engines, aerodynamics, electric parts, air liquefiers and utilities.
This mod changes tech requirements od other parts too.


Model generation

It looks like you moved TestFileGenerationFactory.java into a directory named "org.finos.legend.sdlc.generation.file" instead of "org/finos/legend/sdlc/generation/file".

Fixed. thanks


[BACKLOG-40063] - Address review comments

Addresses review comments of https://github.com/pentaho/pentaho-kettle/pull/9283

 Note:

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.

🐸 JFrog Frogbot

:white_check_mark: Build finished in 29m 33s
Build command:
mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl core

:ok_hand: All tests passed!
Tests run: 1270, Failures: 0, Skipped: 1    Test Results

:information_source:  This is an automatic message


Missing base hooks: leadership, update-status, storage

New hooks exist since 1.23 of Juju which are not modeled in the base layer:

leader-elected
leader-settings-changed
{storage}-storage-attached
{storage}-storage-detaching
update-status

where {storage} is defined under the top level storage key in metadata.yaml
It'd be great to have them there by default

+1 to this
Missing update-status, and upgrade-charm hooks leave me with reactive charms that don't respond properly when decorating with @hook() decorators, as the hook exits 0 - never reaching the proper hook context in the reactive methods.

I'm a little bit hesitant about including update-status by default, because would trigger any matching handler every time the hook was invoked, every 5 minutes.  While this should be fine if the charms are coded well and are properly idempotent and properly gate behavior behind "if changed" clauses, it could still lead to more actions being taken on update-status than might be expected.
In some cases, this could actually be useful behavior, though, since sometimes it can be difficult to get a service to retry a hook that resulted in "blocked" status, but I wonder if it might not be worth gating it behind some sort of layer option?

Would it be possible for charm build to sniff all @hook decorators and add
hooks that are in a whitelist that wouldn't be added by default?
On Mon, Nov 16, 2015 at 3:41 PM Cory Johns<EMAIL_ADDRESS>wrote:

I'm a little bit hesitant about including update-status by default,
because would trigger any matching handler every time the hook was invoked,
every 5 minutes. While this should be fine if the charms are coded well and
are properly idempotent and properly gate behavior behind "if changed"
clauses, it could still lead to more actions being taken on update-status
than might be expected.
In some cases, this could actually be useful behavior, though, since
sometimes it can be difficult to get a service to retry a hook that
resulted in "blocked" status, but I wonder if it might not be worth gating
it behind some sort of layer option?
—
Reply to this email directly or view it on GitHub
https://github.com/juju-solutions/reactive-base-layer/issues/4#issuecomment-157163036
.

Not really, because of the external / non-python handlers; it's not really possible to introspect them.
I should perhaps clarify that I'm not entirely adverse to including update-status  by default, but we would just need to make sure we call it out as a consideration in the docs.

juju-solutions/reactive-base-layer#12
juju/charm-tools#67
juju-solutions/reactive-base-layer#13

These are all available now.


Cache middleware

I think we should have a decent HTTP cache middleware (like faraday-http-cache) with pluggable backend (a simple interface of get/put) while handling all of the HTTP cache stuff

@teamon you have an idea of implementation details?

There are many options, from super simple like "cache all GET for X seconds in ets" to full like faraday-http-cache
Here is very old take on it (with old tesla) - https://gist.github.com/teamon/696232ca6e5a2955052914605c4e05c0#file-more_middlewares-ex-L1 baced by file (don't do it ;))

I like having "official" middleware for general use case, but I'm not sure it's a good idea to add all of them into the main package.

Let's continue in #264


The cloud data sources Q&A should be clarified.

The comment about not needing a gateway for cloud data sources such as Azure SQLDB is correct--but incomplete thus misleading. If a cloud service like Azure SQLDB resides within a corporate virtual network, behind a firewall, or requires a VPN, then it would require a gateway. More and more companies are implementing virtual networks to enhance security so it would be helpful to update the Q&A with a reference to that to make people aware that it's not always so black and white. Thanks!

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

ID: d7103a5b-7dd8-c9df-0302-f374292a0934
Version Independent ID: 2d08eb1c-6e9e-fb1f-c305-b75ae4bf70fc
Content: On-premises data gateway FAQ - Power BI
Content Source: powerbi-docs/service-gateway-onprem-faq.md
Service: powerbi
Sub-service: powerbi-gateways
GitHub Login: @mgblythe
Microsoft Alias: mblythe

Hi, @sqlchick -- thanks for your suggestion! I've assigned it to the author.

Thanks @sqlchick - we're redoing the gateway docs and I'll make sure we clarify this Q&A.
@v-doklo - the new version of this article is better, but please take a look at this specific Q&A and make sure it doesn't say "No" for cloud sources.

We moved the FAQ as part of a doc rewrite: https://docs.microsoft.com/en-us/data-integration/gateway/service-gateway-onprem-faq. We clarified the point that you brought up:
Question: Do I need a gateway for cloud data sources like Azure SQL Database?
Answer: No. In general, the service will be able to connect to that data source without a gateway. However, for on-premises data sources, or if the data sources reside behind a firewall, or requires a VPN, or are on virtual networks, a data gateway may be needed.
Thanks!


Update gsm_sys.h

This has already been fixed, but on development branch. It will be merged to master on next release.


Develop Resource Page: Event Diversity - Kid Friendliness

This issue is for coordinating around developing the resource page for Event Diversity - Kid Friendliness.
The resource page lives in the following google document for easy editing, commenting, and revision:
https://docs.google.com/document/d/1jiylWjkLm04paBcflkNn0-yahvy958ShiUbrKrG20Zs/edit?usp=sharing
Once the page is in a decent shape, we will put a copy of it in the repository as a markdown file.
To work on this issue, feel free to leave a comment here and start editing the document. For questions, feedback requests, and other coordination tasks, please leave a comment here on the issue.
Please edit the google document directly and add your ideas to it. We can help with cleaning it up later.

This one was off to a good start, so I did a bit of minor cleanup and now it's ready for review.


Move some functions out of utilities

Some functions are too specific to be in utilities

run_smart (never used)
entropy
unbias probabilities

Utilities is too generic and contains too much stuff: it should be split into more specific files. This may be annoying for backward compatibility though...
Maybe leave utilities.py with from <file> import *


Run application with Docker

I am hoping someone will find this useful. Has added instruction in README too. Simple as it gets.
Docker build & run time on my machine was (Stat) :
real    2m4.055s
user    0m0.606s
sys     0m0.088s

Wow - this is great! Thanks for contributing!


check_service should be able to check multiple windows services at once

Is your feature request related to a problem? Please describe.
I'm checking multiple windows-services on a host with almost the same name - they just differ by a serial number. Currently, each windows-service needs its own check.
Describe the solution you'd like
--service accepts also regular expressions
two new arguments:
--warning, --critical,  accepting ranges: e.g. at least 10 but not more than 20 Windows-Services must meet --status
Alternatives
No response
Additional context
No response

Something like that?
service.exe --service="^win.*" --status=running --warning=5:
4 services named "^win.*" with status "running" found, but expected "5:" [WARNING].

Service Name        ! Status  !  :-)


Text is cut

Merci ! thanks for your awesome library, when I put a long message  it's cut, can you fix it ?

Thanks for your feedback @Dave181295, the message text will be update to 3 lines in the next release.

Resolved #21. New release


StartView from year not working with specific option

Test Steps:

Look the Demo page: http://eternicode.github.io/bootstrap-datepicker/
Choose the following options: - startView: 2, minViewMode: 1, autoclose: true
Open datepicker (it is opening on year selection which is good)
Select a date
Click away and open again datepicker

Actual Result:
Datepicker opening on month selection
Expected Result:
Datepicker should open on year selection

Fixed in #1304


Custom Extension - Error in https.request - The "listener" argument must be of type Function. Received type object

Version: 1.43.1 (user setup)
Commit: fe22a9645b44368865c0ba92e2fb881ff1afce94
Date: 2020-03-18T07:01:20.184Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.17134
Background:
I'm developing a VSCode extension which makes web requests to a server. The extension is installed on my instance of VSCode when the error is produced. I have not see the error while debugging the extension.
Steps to Reproduce:
This problem is hard to reproduce. Using the code below, the error is randomly reproduced. Once it occurs, it will continue to occur until I restart VSCode.
I believe the error is occurring due to this line:
https.request(url, options, (res) => {...})
I figured maybe the VSCode API is taking a wrong path every once in a while and does not properly handle the signature overloads in all cases. It seems that sometimes the API expects the second parameter to always be a function.
Based on this hypothesis, I changed the code to nest the URL content into the options parameter and only pass https.request(options, function). I have not reproduced the error in a week since I modified the code in this way. I was reproducing the error daily, with the same rate of usage.
Here are the signatures for Node https.request:
https.request(options[, callback])
https.request(url[, options][, callback])
Please let me know if I can provide more information.
const https = require('https')

/**
 * @param {string} url https.request url
 * @param {RequestOptions} options https.request options
 * @returns {Promise} Resolves with the request and response object.
 */
function makeRequest(url, options) {
	return new Promise((resolve, reject) => {
		const req = https.request(url, options, (res) => {
			const chunks = []
			res.on('data', (chunk) => {
				chunks.push(chunk)
			})
			res.on('end', () => {
				if (res.statusCode !== constants.statusCodes.ok) {
					reject(new Error(`Error accessing URL ${url}. StatusCode: ${res.statusCode}`))
					return
				}
				res.body = Buffer.concat(chunks).toString()
				resolve({
					req,
					res
				})
			})
		})

		if (options.body) {
			req.write(options.body)
		}
		req.on('error', (e) => {
			reject(e)
		})
		req.end()
	})
}

Stack trace:
onDidChangeNotification (notificationsAlerts.ts:40)
(anonymous) (notificationsAlerts.ts:26)
fire (event.ts:587)
addNotification (notifications.ts:183)
notify (notificationService.ts:101)
(anonymous) (mainThreadMessageService.ts:83)
_showMessage (mainThreadMessageService.ts:44)
$showMessage (mainThreadMessageService.ts:38)
_doInvokeHandler (rpcProtocol.ts:398)
_invokeHandler (rpcProtocol.ts:383)
_receiveRequest (rpcProtocol.ts:299)
_receiveOneMessage (rpcProtocol.ts:226)
(anonymous) (rpcProtocol.ts:101)
fire (event.ts:587)
fire (ipc.net.ts:453)
_receiveMessage (ipc.net.ts:733)
(anonymous) (ipc.net.ts:592)
fire (event.ts:587)
acceptChunk (ipc.net.ts:239)
(anonymous) (ipc.net.ts:200)
t (ipc.net.ts:28)
emit (events.js:203)
addChunk (_stream_readable.js:295)
readableAddChunk (_stream_readable.js:276)
Readable.push (_stream_readable.js:210)
onStreamRead (internal/stream_base_commons.js:166)

Does this issue occur when all extensions are disabled?: n/a I could not do this since my extension is the one with the error.

@chrmarti could this be related to the magic that allows extensions to use a proxy if a site has one?

@samusstrike What are url and options when this fails?

Here's an example of the URL and options provided to https.request. The "RequestOptions" type is that of the Node JS http RequestOptions: https://nodejs.org/docs/latest-v12.x/api/http.html#http_http_request_url_options_callback
Note the URL and options are the same in the case of success or failure.
const url = "https://hostName/app/request?login=true"
/**
 * @type {RequestOptions}
 */
const options = {
	method: "GET",
	headers: {
		username: "asdf",
		password: "12345"
	}
}
const req = https.request(url, options, (res) => {
	//...
})

Could you wrap your code in a try-catch and log the error with its stacktrace? (The stacktrace you posted shows the notification's code path.)

I will work on getting the results. I am reverting to using my old code on my custom extension. It may take me a day or two to get the repro results due to the apparent random behavior. Thanks!

I created an independent extension with this repro: https://github.com/samusstrike/vscodeExtensionRequestIssue.git
After a lot of debugging, I was able to identify an exact repro case. You can choose to use the repository above and use the extension debugger. Or you can use the compiled vsix file on the repo.
Instructions to reproduce using vsix file:

Disable all extensions

To rule out other extensions causing the issue.

Install the extension https://github.com/samusstrike/vscodeExtensionRequestIssue/raw/master/extrequesttest-0.0.1.vsix

This contains the command to trigger the https.request

Open VSCode (only the one extension above enabled)

Execute the command "vscodeExtTest.testRequestWithIssue"
Result: No error should occur

Install extension "SQL Server (mssql)" "ms-mssql.mssql"

Open a new editor tab and set the language to SQL

This is the key to the randomness. If a file is open with the SQL language set, it will trigger the mssql extension to do black magic.

Execute the command "vscodeExtTest.testRequestWithIssue"
Result: The error will occur: "The "listener" argument must be of type Function. Received type object"

Execute the command "vscodeExtTest.testRequestWithoutIssue"

This is to show that not passing the URL as the first argument to https.request() will allow the code to work.
Result: No error should occur.

The following is copied from the console.
TMScopeRegistry.ts:45 Overwriting grammar scope name to file mapping for scope source.sql.
Old grammar file: file:///c%3A/Users/user/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/extensions/sql/syntaxes/sql.tmLanguage.json.
New grammar file: file:///c%3A/Users/user/.vscode/extensions/ms-mssql.mssql-1.9.0/syntaxes/SQL.plist
register @ TMScopeRegistry.ts:45
console.ts:137 [Extension Host] logFile for SqlToolsResourceProviderService.exe is c:\Users\user\AppData\Roaming\Code\logs\20200325T031000\exthost1\ms-mssql.mssql\resourceprovider.log
console.ts:137 [Extension Host] This process (ui Extenstion Host) is pid: 45776
console.ts:137 [Extension Host] Old log files deletion report: {}
console.ts:137 [Extension Host] The "listener" argument must be of type Function. Received type object
t.log @ console.ts:137
console.ts:137 [Extension Host] Trace: Repro test	at Object.testRequest (c:\Users\user\.vscode\extensions\test.extrequesttest-0.0.1\app.js:119:11)
t.log @ console.ts:137
notificationsAlerts.ts:40 Error occurred: The "listener" argument must be of type Function. Received type object
onDidChangeNotification @ notificationsAlerts.ts:40

This is an issue with agent-base 4.3.0 not supporting the 3-argument request() added in Node.js v10.9.0 (https://github.com/TooTallNate/node-agent-base/issues/29):
https://github.com/TooTallNate/node-agent-base/blob/560f111674af84dec46a4c3070ddf3b22edd3e76/patch-core.js#L13
agent-base 4.3.0 patches the http/s modules, so any extension loading that module will side-effect all other extensions.
I see agent-base 4.3.0 in use also in other extensions. As a workaround we can hand out copies of http/s to extensions, so they end up modifying their own copy and won't affect others.

@samusstrike Thanks for the sample to reproduce the issue!

@chrmarti I'm glad it helped. Thank you for your time looking into this. I'm glad there's a workaround for now.


[Bug] DeprecationWarning: concurrency_count has been deprecated. Set the concurrency_limit directly on event listeners e.g. btn.click(fn, ..., concurrency_limit=10) or gr.Interface(concurrency_limit=10). If necessary, the total number of workers can be configured via max_threads in launch().

Checklist

[X] 1. I have searched related issues but cannot get the expected help.
[X] 2. The bug has not been fixed in the latest version.

Describe the bug

Reproduction
lmdeploy serve gradio http://localhost:23333
--server-name <IP_ADDRESS>
--server-port 6006
Environment
gradio                    4.27.0
gradio-client             0.15.1
lmdeploy                  0.4.0

Error traceback
No response

Please try gradio<4.0.0
https://github.com/InternLM/lmdeploy/blob/main/requirements/serve.txt#L1

Cool. Thank you.


In Technologies used the attached links were mismatched so fix that

Thank you for raising an Issue Ji @Priyal-tech
I would really appreciate if you can make a PR and contribute to the repository.

i would like to contribute to this issue


Fail to parse exponential number

As exponential number end by E[0-9] the plugins fail

Hi, could you please be more precise?
thanks


[Enhancement] (nereids)implement CreateCatalogCommand in nereids

What problem does this PR solve?
Issue Number: close #42594
Related PR: #xxx
Problem Summary:
[Enhancement] (nereids)implement CreateCatalogCommand in nereids
Release note
None
Check List (For Author)

Test

[x] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[ ] No.
[ ] Yes.

Does this need documentation?

[ ] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall

run buildall

run buildall

run P0

run buildall

run buildall


Add inline mapping to SCVS BOM Maturity Model

A proposal has been suggested that the CycloneDX specification add native support for the SCVS BOM Maturity Model to the schema itself. This may likely be a JSON-only enhancement, but XML support may be considered.
For example, the definition of a /components/[]/supplier looks like:
"supplier": {
  "title": "Component Supplier",
  "description": " The organization that supplied the component. The supplier may often be the manufacturer, but may also be a distributor or repackager.",
  "$ref": "#/definitions/organizationalEntity"
}

could be enhanced to:
"supplier": {
  "title": "Component Supplier",
  "description": " The organization that supplied the component. The supplier may often be the manufacturer, but may also be a distributor or repackager.",
  "$ref": "#/definitions/organizationalEntity"
  "meta:scvs:version": "1.0",
  "meta:scvs:structure": "urn:owasp:scvs:bom:structure:inventory",
  "meta:scvs:taxonomy": [
    "urn:owasp:scvs:bom:provenance:role:supplier"
    "urn:owasp:scvs:bom:provenance:role:supplier:org",
    "urn:owasp:scvs:bom:provenance:role:supplier:non-org"
  ]
}

By supporting SCVS BOM Maturity Model in the schema itself, this would allow tool makers to leverage the model and have a build-in mapping. This will aid in:

SBOM quality tools and score calculations
Make policy and governance easier

SCVS profiles could be used together with CycloneDX natively

Aid in conversions between BOM formats

@Mariuxdeangelo I hope to use/add a "meta:scvs" styled key to each data type in the schema (much like added for "meta:enum".

I guess e could even grate an XSD extension that would allow adding these flags to out XSD files ....
Not so sure about the protobuff schema files ...

Yeah, not too worried about protobuf. But if we find a way to do this in the XSD, that would be great. If not, the JSON Schema is our reference implementation for the specification and the Ecma standard, so its obviously a priority.

By supporting SCVS BOM Maturity Model in the schema itself, this would allow tool makers to leverage the model and have a build-in mapping.

@stevespringett if we added those markers to a schema, how would this help anybody?
Is there any existing/discrete tool that would read those fields? If there is no such tool yet, where is the spec of such a tool - I mean i could probably come up with some alternatives to this very proposal, so that such a tool would still be functioning ...
I find the proposal waaaay to vague, so I can not see any value in it yet. I would even say it is bloat I would avoid having in the schema files.

The CycloneDX SBOM Utility that Matt has created, currently uses Go annotations to do somthing similar. Its very manual however. The idea is to remove the annotations that are specific to the tool, and use the mapping provided by the schema itself.  In doing so, other tools would be able to adopt the same approach.

In doing so, other tools would be able to adopt the same approach.

This would require other tools to follow the same annotation style we would come up now.
I don't see that as a desirable way to work.
here is another approaches, which is more versatile in many means:
Somebody provides mapping of each SCVS "flag" to (multiple) actual BOM-document targets in form of JSONPath(JSON and YAML documents), XPATH(XML documents), FieldMask(Protobuf documents).
Such mapping is document centric (not schema- nor specification-centric).
Such mapping would not be part of any schema file, but independent.
Such mapping is portable.
Such mapping is machine readable.
example for CycloneDX JSON:

urn:owasp:scvs:bom:core:timestamp -> $.metadata.timestamp
urn:owasp:scvs:bom:resource:software:sbom -> $..externalReferences[?($.type=='sbom')].url

This solution is actually one that the SCVS BOM Maturity Model WorkGroup should have come up with in the first place, since this is the most interoperable way (I can think of) of mapping their "Identifier" to an actual data structure.
This is the relevant part: mapping to data structures, since they should not care for capabilities(schema or specification) but actual SBOM data.
These mappings can be provided by the SCVS community without any need for interaction with SBOM standards working groups, so this makes them even independent.


rclip: 1.10.0 -> 1.10.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for rclip is: AI-Powered Command-Line Photo Search Tool
meta.homepage for rclip is: https://github.com/yurijmikhalevich/rclip
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.10.1 with grep in /nix/store/15b0daa0wajpq06jb7xj520kacrxzxbh-rclip-1.10.1
found 1.10.1 in filename of file in /nix/store/15b0daa0wajpq06jb7xj520kacrxzxbh-rclip-1.10.1

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

rclip

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/15b0daa0wajpq06jb7xj520kacrxzxbh-rclip-1.10.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A rclip https://github.com/r-ryantm/nixpkgs/archive/22b3487860c61629ce06e1e9e869037e4544a79b.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/22b3487860c61629ce06e1e9e869037e4544a79b#rclip

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/15b0daa0wajpq06jb7xj520kacrxzxbh-rclip-1.10.1
ls -la /nix/store/15b0daa0wajpq06jb7xj520kacrxzxbh-rclip-1.10.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages built:

rclip
rclip.dist

Maintainer pings
cc @iynaix for testing.

@NixOS/nixpkgs-merge-bot merge


fix: allow sar_ent_slot_serial ingame

This PR patches sar_ent_slot_serial to change the entity's serial number on the next load if it can't do it immediately.

all my homies hate windows


Merge in Herbie's rival-apply code, add a REPL

This code comes directly from the separate-sampling branch on Herbie, just split into multiple files (for maintainability) and with a brand-new, minimal-effort REPL to demonstrate the evaluator.

An example of the REPL:
~/herbie/rival $ racket main.rkt
> (eval (- (exp (PI)) (PI)))
19.999099979189477
> exit


Prevent upgrade cta appearing on accounts that have no upgrade option.

When first configuring Portal, it is reasonable and straight-forward to connect a Stripe account as instructed, but then disable the paid tiers to delay presenting paid memberships until the site is mature enough to warrant paid memberships.
This results in a very functional Portal, which prompts for, processes and accepts free membership. After logging into a free account however, the Portal displays a call-to-action paragraph encouraging the user to upgrade to a paid account. Obviously this is confusing for the user, given that there are no paid options.
The code already performs a series of checks to ensure it is worth making a call to action. Given that a function to check for the existence of paid accounts is also provided, it seems a simple oversight that it is not included in the checks. This PR adds that check.
The result is no change for sites that have paid memberships, or do not have Stripe configured. But if the only membership plan is free, then the cta text is not added to the page, just as it would not be if Stripe was not configured or the subscription was complimentary or cancelled.

Hey @hraftery , thanks for raising this PR and sorry about the delay in getting this merged. The change completely makes sense and was definitely an oversight for upgrade CTA to be visible when there were no paid plans available. This is now merged and will be released this week 😄


Change text of buttons

Hey how I can change the text from buttons "done" and "skip"
I not found method for do this, please help me

That is currently done through styles:
<style name="CustomWelcomeScreenTheme" parent="WelcomeScreenTheme">
    <item name="welcomeButtonSkipText">your string</item>
    <item name="welcomeButtonDoneText">your string</item>
</style>

Then, you would apply it by calling .theme(R.style.CustomWelcomeScreenTheme) on your builder


fix: checksum calculation for fr/nif

Fixes #55 - this adds checksum verification for Frace / NIF formats.

You are amazing. How do you handle<PHONE_NUMBER>000 scenarios?


Score board needs to show total score.

It does show total score in text form (current, total). Closing this will work on it in the Game Over summary.