Async method not executed!

I'm building an application, where my repository have all my database access logic, all of me repository methods are async
some simple queries such as
public async Task<Product> GetByIdAsync(string id)
{
      return await  CurrentContext.Products.FindAsync(id);
}

it does not work some time, for example when i try to send an id for my product, the CPU goes up to 70% using, and no result is returned,  and some time the same code when i search for another product it gets it without and problem
in the case that it didn't work, i tried everything but there is nothing worked, and after converting my async query to sync it worked beautifully
public Product GetByIdAsync(string id)
{
      return CurrentContext.Products.Find(id);
}

I'm using EF Core 2.0 and SQL Server 2017

@A7medRamadan Unfortunately there is not enough information here for us to be able to effectively investigate the issue. Can you post a project or complete code listing that will allow us to reproduce what you are seeing?

@ajcvickers This is not an option for me to post all of my project, but as i defined earlier.
in my computer everything works great but when i moved my project to another computer with less resources the problem starts to shows up

@A7medRamadan I understand that it may not be possible for you to send us your produce code, but if you are able to create a small project that reproduces what you are seeing, then we can investigate. As it is now, there is no way for us to reproduce what you are seeing which means we are not able to investigate in any reasonable way.

EF Team Triage: Closing this issue as the requested additional details have not been provided and we have been unable to reproduce it.
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.


MSRPC over HTTPS for Exchange/Owa Reverse Proxy

Do you want to request a feature or report a bug?

Feature
What did you expect to see?
Microsoft RPC over HTTPS traeffic passed through "traefik" to the Exchange 2016/2019, so that I can use Outlook Anywhere.
What did you see instead?
MSRPC over HTTPS traeffic not passed through. Error message instead: "Failed to load resource: the server responded with a status of 500 (Internal Server Error)".
Output of traefik version: (What version of Traefik are you using?)
traefik version
Version:      2.3.2
Codename:     picodon
Go version:   go1.15.3
Built:        2020-10-19T18:36:22Z
OS/Arch:      linux/amd64

Additional Information
Currently the setup is working with Microsoft IIS ARR as the reverse proxy. Changing that by passing all 80/443 traffic from IIS to traefik, resulting in the error code 500 above.
If I want to use Apache as a reverse proxy instead, I have to install an additional module for MSRPC (https://directory.fsf.org/wiki/Mod_proxy_msrpc). Maybe "traefik" needs something similar?

Hey @dataCore,
thanks for your interest in the project.
To me, it looks like MSRPC seems to be a different protocol, no? Or is it some sort of HTTP in between? I tried to give a quick research and it looked like, that its a protocol which negotiates some further ports (TCP and UDP) and stuff. Am I wrong?
Can you explain the background a bit more, please? :) Like, what do you expect, how should it be working, what is a desired configuration? So we can potentially follow a bit better :)
Thanks!

Hi @SantoDE, thanks for your reply!
To me, this is also some weird microsoft stuff/protocol and I'm not a Microsoft Exchange expert either. My goal: I just want to replace our working, but old IIS ReverseProxy with traefik:2.3 as a better ReverseProxy...
In my understanding, Microsoft Outlook (E-Mail Client from Microsoft), running on client laptops outside our network, uses this protocol over port:443 to communicate with the Backend Microsoft Exchange 2016 server. The server itself has multiple pages/protocol to serve different things (owa, api, ecp, ews, mapi, oab and RPC) over his port:443.
3 Urls are used (over 443) and affected of the problem:

autodiscover.example.ch (returning some xml to configure (I think) the clients)
mail.example.ch (same as owa.example.ch)
owa.example.ch (same as mail.example.ch)

Our working setup:
[External Outlook Client: mail.example.ch:443] --> Portforward:443 --> [IIS Reverse Proxy] --> [Route to: internal Exchange2016:443]
Our not working setup:
[External Outlook Client: mail.example.ch:443] --> Portforward:443 --> [Traefik Reverse Proxy] --> [Route to: internal Exchange2016:443]
"not working": the webpage is working (owa) but the client (outlook) can't connect and under https://autodiscover.example.ch/Autodiscover/Autodiscover.xml apear a "ErrorCode:500 Internal Server Error" instead of an XML.
Anyway - next I will try to use tcp with HostSNI instead of http/https for the affected urls and will see, if it will work that way.
Stay tuned :) and thanks for your great work!!

update on this:
I had to do the ssl/cert things directly on the exchange server. So I had to use tcp passthrough
config example:
tcp:
  routers:
    https.mail.yourdomain.com:
      rule: HostSNI(`mail.yourdomain.com`)
      entryPoints:
        - https
      service: mail.yourdomain.com
      tls:
        passthrough: true

  services:
    mail.yourdomain.com:
      loadBalancer:
        servers:
          - address: internalExchangeHostName01:443

can be closed


deviations from tf/keras conventions

this looks like a fantastic resource, but I am struggling to get it to work. My application uses complex custom pre-processing functions, implemented as .map() on tf.data.Dataset objects. There are not yet many examples using this library, and the ones I found use relatively simplistic datasets. I've struggled to adapt the examples to my dataset, and have hit many errors trying to format my dataset appropriately. I think I need to complete pre-processing as a tf.data.Dataset, then convert to a dict for input to the model.
From a user's point of view, I think it would be ideal if adversarial training could be activated by just a boolean option to the compile() function (with some customisable hyperparameters). At the moment, it seems that there are a few places where the nsl functions are not compatible with the tf/keras functions. Could you please explain the reason for these design choices, and whether you intend to change them in the future?

why the inputs have to be dicts, rather than directly using tf.data.Dataset objects? (I have to call a
why you require the labels to be specified, instead of inferring them from the structure of the model? Is it intended to support a use case where only some of the outputs are used for adversarial training?

I have also hit other errors, which imply that adversarial objects do not yet support all of the functionality of the keras library. I've been able to work around these ones, but they might be useful to help you identify functionality that still needs to be implemented.

Providing an argument to the Adam optimizer gave an error, but simply using the string 'adam' works fine.

adv_model.compile(optimizer='adam',
            #optimizer=keras.optimizers.Adam(0.0001),
            loss=keras.losses.sparse_categorical_crossentropy,
            metrics=['accuracy'])

trying to use class_weight when training the model gave an error. Commenting it out works fine.

adv_model.fit(adv_train_dataset,
          epochs=n_epochs,
        #   class_weight=class_weights,
          validation_data=adv_valid_dataset,
          validation_freq=1)

Hi @chrisrapson, thanks for your feedback.

why the inputs have to be dicts, rather than directly using tf.data.Dataset objects?

Regarding the input format, nsl.keras.AdversarialRegularization can handle tf.data.Dataset input, as long as each batch in the dataset is a dictionary of tensors. For example, in this tutorial the train_set_for_adv_model and test_set_for_adv_model are tf.data.Dataset objects.
The main reason of using the dictionary format is to enable accessing input labels in the Model.call() method, in order to compute supervised losses for adversarial attacks. (Input labels are generally not available to the call() method during model.fit(x=images, y=labels), so we need both images and labels in the argument x.) The dictionary format also supports use cases where the input contains multiple features (e.g. image plus metadata, multiple text fields, and structured data).

whether you intend to change them in the future?

TensorFlow 2.2 introduced Model.train_step(), which might be a better place for adding the regularization logic because it provides access to input labels naturally. If going this route, we can lift the constraint of each batch being a dictionary of tensors. However, the NSL library currently supports back to TF 1.15, so we cannot make the switch now. If you are interested, PRs are always welcome!

why you require the labels to be specified, instead of inferring them from the structure of the model?

Since the input batch is expected to be a dictionary of tensors mixing features and labels, users have to specify which keys in the dictionary are labels. Note that a Keras model may have multiple outputs (example), in such case a mapping between outputs and loss functions has to be provided by the user.

Providing an argument to the Adam optimizer gave an error,
trying to use class_weight when training the model gave an error.

I will look into those errors.

Providing an argument to the Adam optimizer gave an error,

I cannot reproduce this using TensorFlow 2.3 or 2.5. As in this colab the adversarially regularized model can be trained with an Adam(2e-3) optimizer.

trying to use class_weight when training the model gave an error.

Right, AdversarialRegularization doesn't support class_weight for now. The class_weight arg is handled in Keras internals before calling into the AdversarialRegularization subclass. But at that point Keras doesn't know what the input labels look like, due to the label_key setting in AdversarialRegularization. We might be able to add class_weight support after migrating to the Model.train_step() approach.

Closing this issue for now. Feel free to reopen if you have any follow-up questions.


Find common errors in code

Hyperapp debug should be able to find common errors, like not wrapping text in text('...'), or using the wrong casing for event names.
After finding the error, it should be able to make a recommendation on how to fix the error, and the exact location of the error.

My initial thought is something like:
import withDebug, { devMode } from 'https://unpkg.com/hyperapp-debug?module=1';
import * as hyperapp from 'https://unpkg.com/hyperapp?module=1';

const { app, h } = devMode(hyperapp);
// Now app and h report warnings/common mistakes
// This is separate from `withDebug` which communicates with the devtool

Some suggestions on things to warn about (unsure how common they are):

Forgetting to wrap strings in text, e g h('h1', {}, 'hello')
Forgetting the property object in h e g h('h1', text('hello'))
Passing mulitple children as separate args, rather than an array:

h('div', {},
  h('h1', {}, text('hello'),
  h('p', {}, text('foo bar baz')
)

Nested arrays in children:

h('div', {}, [
  h('h1', {}, text('hello')),
  [
    h('p', {}, text('some paragraph')),
    h('p', {}, text('another paragraph')),
  ]
 ])

Using plain string for style attribute: h('button', {style: 'color: #f00; border-radius: 5px'}, text('click me'))

I think it's worth noting that once hyperapp has type definitions available, VSCode or other type-aware editors will warn you about these sorts of mistakes (and many others) anyway. What this sort of library could add on top of that is more helpful messages on how to fix the errors. (Because the type errors VSCode will report are probably quite inscrutable)


Fix Optimizer Conversion Issue (Integer Keys)

Summary
There are optimizers (notably QNSPSA) that use integer keys in their json. This creates serialization issues because Python's built-in json module forces conversion to strings.
Fixes #42

Adds QNSPSA to optimizer encoding/decoding tests
Adds  __use_int_keys__ key into serialized JSON accommodate integer keys.

Details and comments
Adding integer key checks for serializing custom objects that are not Optimizers (which have the settings property) involves adding the __uses_int_keys__ property to default() in the RuntimeEncoder class. It should really only be added to classes that need it.

Replaced by https://github.com/Qiskit-Partners/qiskit-ibm/pull/50


Stop find operation at last match of the file

VS Code version
OS

1.14.0
Win x64 pro (french)

It may be just me who didn't find the function or setting to enable. Please enlighten me.
Steps to reproduce

Open any file
Run the find command, CTRL+F
Query a string you know it exists in multiple place in file
Use F3 or next button to move to next match

Results
When you reach the last occurrence, the find command keep searching by jumping to the first occurrence at the top of the file. You're in a search loop.
Expectations
When last search occurrence is reached, most of text/code editor will prompt the user if he wants to keep searching the file from the first line. This is a feature I need and should be a toggle one.

I aggree this is a missing feature.
Currently, you must always know on which line the search began to avoid a loop.
It would also be useful when navigating to the next / previous difference.

Positively agree. I've just been searching a term in 5,000 lines of a file, until I noticed I've been searching the same occurrences over and over and over and over again.

This issue should be resolved when #69741 was fixed. The feedback provided when doing searches is much much better now, I think you'll need an insiders build to try this now since its probably not yet in the monthly release.

This issue should be resolved when #69741 was fixed. The feedback provided when doing searches is much much better now, I think you'll need an insiders build to try this now since its probably not yet in the monthly release.

The only hint is the number of found occurrences is displayed as well as the one currently selected. Still no alert when it detects you have cycled through all occurrences.

We added editor.find.loop to control if search restarts from the beginning of the document when you are at the end of the document.


Initial code commit

Basic functionality stub including one documentation piece. Used to test the contribution process.

This commit is part of #7

Hi Mario,
Can you add a simple copyright/license template to the files?  Something along (ofcourse your version)
// SPDX-FileCopyrightText: 2023 Contributors to the Eclipse Foundation
//
// See the NOTICE file(s) distributed with this work for additional
// information regarding copyright ownership.
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
//     http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
//
// SPDX-License-Identifier: Apache-2.0

[like]  Mario Ortegon Cabrera reacted to your message:

From: Naci Dai @.>
Sent: Tuesday, January 30, 2024 1:05:48 PM
To: eclipse-sdv-blueprints/insurance @.>
Cc: Mario Ortegon Cabrera @.>; Author @.>
Subject: Re: [eclipse-sdv-blueprints/insurance] Initial code commit (PR #12)
Hi Mario,
Can you add a simple copyright/license template to the files? Something along (ofcourse your version)
// SPDX-FileCopyrightText: 2023 Contributors to the Eclipse Foundation
//
// See the NOTICE file(s) distributed with this work for additional
// information regarding copyright ownership.
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
//
// SPDX-License-Identifier: Apache-2.0
—
Reply to this email directly, view it on GitHubhttps://github.com/eclipse-sdv-blueprints/insurance/pull/12#issuecomment-1916804661, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARO5DJ3BU66PJCI6TMG3INDYRDV2ZAVCNFSM6AAAAABCQ2TIAGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJWHAYDINRWGE.
You are receiving this because you authored the thread.Message ID: @.***>

Hi Naci,,
Sure! Thanks for the feedback, that is why I wanted to try with a fairly simple commit.
Do we also need this for the documentation? Or only for the source files?
Thanks,
From: Naci Dai @.>
Sent: Tuesday, January 30, 2024 2:06 PM
To: eclipse-sdv-blueprints/insurance @.>
Cc: Mario Ortegon Cabrera @.>; Author @.>
Subject: Re: [eclipse-sdv-blueprints/insurance] Initial code commit (PR #12)
Hi Mario,
Can you add a simple copyright/license template to the files? Something along (ofcourse your version)
// SPDX-FileCopyrightText: 2023 Contributors to the Eclipse Foundation
//
// See the NOTICE file(s) distributed with this work for additional
// information regarding copyright ownership.
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
//
// SPDX-License-Identifier: Apache-2.0

Reply to this email directly, view it on GitHubhttps://github.com/eclipse-sdv-blueprints/insurance/pull/12#issuecomment-1916804661, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARO5DJ3BU66PJCI6TMG3INDYRDV2ZAVCNFSM6AAAAABCQ2TIAGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJWHAYDINRWGE.
You are receiving this because you authored the thread.Message ID: @.***>

Yes it is best to have it in your source code, and all initial commits.

@nacidai Hi Naci, I have added a new commit to the PR with the licenses and additional documentation.

This PR also closses #2. The code includes a .devcontainer that installs the required python components.


Autorest Regen Preview to 3.0.0-alpha.20230627.9 by Mike Harder from refs/pull/3524/merge

Triggered from https://github.com/Azure/autorest.csharp/pull/3524

API change check
API changes are not detected in this pull request.


NPM Audit - dependency update

To be merged only after PR #49

Would it be possible to rebase and have #50 and #49 as the same PR?
Sorry for the abandonment of the repo, I'm aiming to transfer control soon.


Not scrollable filechooser

Hi,
Is it possible to have a filechoosericonview not scrollable?
I use it on a raspberry pi with a pitft resistive touchscreen, and when I want to select a file the resistive screen scroll the view instead of selecting the file...
Would it be possible to have instead of a scrollable view a "multipages" view?
Thanks in advance,

github issues are for bug reports, not support requests. Please ask on the user mailing list or on the #kivy IRC channel for support.
That said, filechooser uses a scollview, and you probably can manually disable scrolling on it. Also, we're unlikely to introduce new features into filechooser without a significant re-write, which is not likely to happen anytime soon.


Could you tag it with a version number?

Hi, I would like to use your library, but it does not have a version number - I can only install it via composer if I use * as the version. My employer does not like that (if a backward incompatible update is made to the library, it will automatically be pulled in on the next composer update, even if it has a new major version number). It may well be that you don't intend to make any updates, but as a matter of good practice, a tag with a version number would be appreciated. Thanks!

Sorry, my bad. It does have a version number - perhaps you could just update your install instructions for composer to use 1.* or 1.1.*?


fix flake8 config param

configFilePath is already a absolute path, so it should not be joined with projectPath.
Currently it find's tox.ini in /Users/user/Development/project/ and runs flake8 with
'--config /Users/user/Development/project/Users/user/Development/project/tox.ini'

Makes sense, thanks!

Published in v1.13.4. 🎉


Vue 3.0.11 error with SSR

Version
3.0.11
Reproduction link
https://github.com/theoribeiro/vue-next-ssr
Steps to reproduce

Run buildnpm install
npm run build

Try to rendernode render.js

What is expected?

App

Comp

What is actually happening?
    [Vue warn]: resolveComponent can only be used in render() or setup().
    (node:22835) UnhandledPromiseRejectionWarning: TypeError: Cannot create property '__props' on string 'comp'
        at normalizePropsOptions (webpack://app/./node_modules/@vue/runtime-core/dist/runtime-core.cjs.js?:1737:30)
        at createComponentInstance (webpack://app/./node_modules/@vue/runtime-core/dist/runtime-core.cjs.js?:6379:23)
        at renderComponentVNode (webpack://app/./node_modules/@vue/server-renderer/dist/server-renderer.cjs.js?:159:22)
        at ssrRenderComponent (webpack://app/./node_modules/@vue/server-renderer/dist/server-renderer.cjs.js?:484:12)
        at ssrRender (webpack://app/./src/app.vue?./node_modules/vue-loader/dist/templateLoader.js??ruleSet%5B1%5D.rules%5B1%5D!./node_modules/vue-loader/dist/index.js??ruleSet%5B1%5D.rules%5B2%5D.use%5B0%5D:15:81)
        at renderComponentSubTree (/Users/theoribeiro/Dev/playground/vue-next-ssr/node_modules/@vue/server-renderer/dist/server-renderer.cjs.js:222:13)
        at renderComponentVNode (/Users/theoribeiro/Dev/playground/vue-next-ssr/node_modules/@vue/server-renderer/dist/server-renderer.cjs.js:173:16)
        at renderToString (/Users/theoribeiro/Dev/playground/vue-next-ssr/node_modules/@vue/server-renderer/dist/server-renderer.cjs.js:412:26)
        at Object.<anonymous> (/Users/theoribeiro/Dev/playground/vue-next-ssr/render.js:4:1)
        at Module._compile (internal/modules/cjs/loader.js:1075:30)
    (Use `node --trace-warnings ...` to show where the warning was created)
    (node:22835) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 2)
    (node:22835) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

This is the same repro with updated dependencies as the one in https://github.com/vuejs/vue-next/issues/2918 that was marked as a duplicate of https://github.com/vuejs/vue-next/issues/2863.
However, https://github.com/vuejs/vue-next/issues/2863, is marked as closed and solved but it does not solve either this issue nor https://github.com/vuejs/vue-next/issues/2918.

Duplicate of https://github.com/vuejs/vue-next/issues/3393
const nodeExternals = require('webpack-node-externals')
// and
externals: [nodeExternals()],


[SPARK-7026] [SQL] fix left semi join with equi key and non-equi condition

When the condition extracted by ExtractEquiJoinKeys contain join Predicate for left semi join, we can not plan it as semiJoin. Such as
SELECT * FROM testData2 x
LEFT SEMI JOIN testData2 y
ON x.b = y.b
AND x.a >= y.a + 2

Condition x.a >= y.a + 2 can not evaluate on table x, so it throw errors

Test build #30798 has started for   PR 5643 at commit 6eb62d2.

Test build #30798 has finished for   PR 5643 at commit 6eb62d2.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30798/
Test FAILed.

Test build #30800 has started for   PR 5643 at commit a99f492.

I am not sure it is suitable to broadcast a hashmap contains key and related rows, this maybe much bigger than the old hashset, may cause OOM issue.

@scwf Of course we can go the old way when there's no additional conditions.

Test build #30805 has started for   PR 5643 at commit cf435db.

Test build #30800 has finished for   PR 5643 at commit a99f492.

This patch passes all tests.

This patch merges cleanly.

This patch adds the following public classes (experimental):

    class FreqItemset(namedtuple("FreqItemset", ["items", "freq"])):

This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30800/
Test PASSed.

Test build #30808 has started for   PR 5643 at commit 19201e0.

Test build #30812 has started for   PR 5643 at commit 41c20d5.

Yes, i understand that when no additional condition it go the old way. i mean when there are additional conditions, your broadcasting hashmap may be much bigger since you also kept the related rows, which may leads to OOM.
I fixed my PR, please have a look.

Test build #30805 has finished for   PR 5643 at commit cf435db.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30805/
Test PASSed.

Test build #30808 has finished for   PR 5643 at commit 19201e0.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30808/
Test PASSed.

Test build #30812 has finished for   PR 5643 at commit 41c20d5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30812/
Test PASSed.

This LGTM

Test build #30818 has started for   PR 5643 at commit 75b8a64.

Test build #30818 has finished for   PR 5643 at commit 75b8a64.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30818/
Test FAILed.

retest this please.

Test build #30822 has started for   PR 5643 at commit 75b8a64.

Test build #30822 has finished for   PR 5643 at commit 75b8a64.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30822/
Test PASSed.

Test build #30834 has started for   PR 5643 at commit a7c6cc4.

Test build #30834 has finished for   PR 5643 at commit a7c6cc4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30834/
Test PASSed.

Test build #30905 has started for   PR 5643 at commit d29f9a6.

Test build #30905 has finished for   PR 5643 at commit d29f9a6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30905/
Test PASSed.

Test build #30924 has started for   PR 5643 at commit 90a69ec.

Test build #30924 has finished for   PR 5643 at commit 90a69ec.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30924/
Test FAILed.

retest this please.

Test build #30988 has started for   PR 5643 at commit ddadf9f.

retest this please.

Test build #31091 has started for   PR 5643 at commit ddadf9f.

Test build #31091 has finished for   PR 5643 at commit ddadf9f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

ping can you update this?

Merged build triggered.

Merged build started.

Test build #33063 has started for   PR 5643 at commit 8ef50d4.

Test build #33063 has finished for   PR 5643 at commit 8ef50d4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33063/
Test PASSed.

Mind bringing this up to date?

Merged build triggered.

Merged build started.

Test build #35076 has started for   PR 5643 at commit 8b8a992.

Test build #35076 has finished for   PR 5643 at commit 8b8a992.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class IsNull(child: Expression) extends UnaryExpression with Predicate
case class IsNotNull(child: Expression) extends UnaryExpression with Predicate

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #35084 has started for   PR 5643 at commit 592794d.

Test build #35084 has finished for   PR 5643 at commit 592794d.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #35086 has started for   PR 5643 at commit 455b890.

Test build #35086 has finished for   PR 5643 at commit 455b890.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #35090 has started for   PR 5643 at commit 15f9707.

Test build #35090 has finished for   PR 5643 at commit 15f9707.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #35241 has started for   PR 5643 at commit ad0ad59.

Test build #35241 has finished for   PR 5643 at commit ad0ad59.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Thanks for working on this and sorry for the delay reviewing.  Would be great if this could be updated in time for Spark 1.5!

Merged build triggered.

Merged build started.

Test build #37346 has started for   PR 5643 at commit cc09809.

Test build #37346 has finished for   PR 5643 at commit cc09809.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
trait HashSemiJoin

Merged build finished. Test PASSed.

@marmbrus

Thanks!  Merging to master.


Fix for Tutorial: Azure SignalR Service authentication with Azure Fun…

…ctions
This fixes a small typo to make sure the user opens the correct function.json file for SignalR Service negotiating when deploying to Azure

@flts : Thanks for your contribution! The author, @sffamily, has been notified to review your proposed change.

confirmed, thanks!
#sign-off


fix: JB-509 - Pay modal: Make pay button bigger to match cancel button size

What does this PR do and why?
Provide a description of what this PR does. Link to any relevant GitHub issues, Notion tasks or Discord discussions.
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] (if relevant) I have tested this PR in all supported browsers.
[ ] (if relevant) I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #3762

PR #3764

PR #3765   👈

This comment was auto-generated by Graphite.

@wraeth-eth started a stack merge that includes this pull request via Graphite.

Graphite rebased this pull request as part of a merge.

@wraeth-eth merged this pull request with Graphite.


No remote monit

Remove the (not used) ability to call monit remotely from MonitInterface.

OCD run https://ocd.appscale.com:8080/job/Daily Build/1204/

:+1:


Different sized backups when using "etcdctl backup" command

Question
When I take backup using etcdctl, I get different sized backups when running command repeatedly.
Is this expected behavior? What's the reason behind such huge difference?
/ # etcdctl backup --data-dir /var/etcd/ --backup-dir /tmp/bckup
/ # du -h /tmp/
59.3M   /tmp/bckup/member/wal
13.5M   /tmp/bckup/member/snap
72.7M   /tmp/bckup/member
72.7M   /tmp/bckup
/ # rm -rf /tmp/bckup/
/ # etcdctl backup --data-dir /var/etcd/ --backup-dir /tmp/bckup
/ # du -h /tmp/bckup/
5.6M    /tmp/bckup/member/wal
16.7M   /tmp/bckup/member/snap
22.3M   /tmp/bckup/member
22.3M   /tmp/bckup/

This is the actual data size:
/ # du -h /var/etcd/
305.2M  /var/etcd/member/wal
74.2M   /var/etcd/member/snap
379.4M  /var/etcd/member
379.4M  /var/etcd/

etcdserver keeps N historical WAL + Snapshot files. See configuration. So if one of them is broken, your server might still be able to come up. Backup only saves the latest one. So there is a difference.


fix for handling braces in path

should fix #287 and #614

Looks like it's causing a few tests to consistently fail.
Did you try a split-string based approach as suggested in #614?

I took braces because of discussion in #287, I honestly did not test it on Linux, since my main windows, will try to fix tests asap.

Coverage decreased (-7.6%) to 90.873% when pulling 91e275204bbddbacfa20b2b5793229fc68198dc2 on MhMadHamster:master into 3d917818c957088b62c160e426c1a99787ae6319 on paulmillr:master.

Coverage decreased (-7.6%) to 90.873% when pulling b149ec01880c9c82f61569a2266e47ced1bddaa8 on MhMadHamster:master into 3d917818c957088b62c160e426c1a99787ae6319 on paulmillr:master.

Coverage decreased (-7.6%) to 90.873% when pulling b149ec01880c9c82f61569a2266e47ced1bddaa8 on MhMadHamster:master into 3d917818c957088b62c160e426c1a99787ae6319 on paulmillr:master.

closing since i dont have macos so finding the problem causing the test fall would take some time for sure

Coverage increased (+0.02%) to 98.471% when pulling dd7192369b1ac8d5312f66cc73cf6c7958e67fb3 on MhMadHamster:master into 3d917818c957088b62c160e426c1a99787ae6319 on paulmillr:master.

https://travis-ci.org/paulmillr/chokidar/jobs/260414604 this one looks really weird because in last build it was failing on braces test and i changed only braces test, @es128 can you maybe restard this particular job please, if you have access, will much appretiate it.

@MhMadHamster thanks this is really good work. I got the one job in Travis passing after a few retries - this isn't unusual, the CI envs are fickle, especially for OS X.
There is one last problem to solve - the "check ignore after stating" test is failing on Windows with this patch.
https://ci.appveyor.com/project/es128/chokidar/build/387

@es128 tbh this test is failing for me even on the current release of chokidar, using windows 10 node 6.9.1 npm 3.10.8 😕 will look into that issue.

@MhMadHamster oh yeah? It wasn't failing on appveyor before this patch. I'm not sure what the issue is. Sporadic failures are just noise, but when the same test fails on every version in both modes it's indicative of something needing to be adjusted.

@es128 yeah, I remember that it wasn't failing before, at least on in my environment.

Thanks for checking into it! ❤

Well, I tracked down problem down to anymatch tho I'm not sure what's the better way to resolve it, the problem with windows is: in the failing test we have variable testDir which is getting compared in ignore function to the path of watched dirs/files, the testDir itself is created using default nodejs path module, so the path inside using backslashes, tho when the function in ignored, getting called by anymatch, it's been called 2 times once with path "as is" and once with normalized path and I'm not sure what was idea behind that, therefore when we're calling the first time with value "as is", we're correctly getting false, but the second time we're calling with normalized path i.e. with forward slashes and getting true when we're expecting false.
The easy way to fix this is to use normalized path for test itself, tho as i said i'm not sure what was the idea behind calling once with path as is and once with normalized path, ideally, I think, path should be normalized in every case where it's being used, before doing anything with it.

So if you use upath.normalizeSafe(path) inside the options.ignored function in that test it'll fix it? I'd be fine with that change.
The path doesn't exactly change deliberately between ignore checks, it's just that the first time it's based on user input and the second time it's based on readdirp output and gets normalized. Separate work is going to have to be done to update glob-parent to latest and deal with the inconsistencies that result out of the windows paths. It doesn't need to be a barrier to landing this PR.

Although we do have to question why it worked before and why this patch had an impact.

So if you use upath.normalizeSafe(path) inside the options.ignored function in that test it'll fix it? I'd be fine with that change.

yes both for path passed in function and testDir

Although we do have to question why it worked before and why this patch had an impact.

In fact this patch do not have impact on this test, i believe breaking changes can be found here, since then check for normalized path was added.
So if you'll run chokidar tests with anymatch 1.3.0 it'll run just fine but if you update anymatch to 1.3.2 it'll fail (on windows only)

ooook I see, thanks for clarifying. Sorry for putting you through all this, and thank you for your diligence.
Let's just update the test to use the normalized path and land this PR.

Coverage increased (+0.02%) to 98.471% when pulling 752dc81cf2abb0f366eab69b2d1c8eb19dc69d0b on MhMadHamster:master into 3d917818c957088b62c160e426c1a99787ae6319 on paulmillr:master.

@es128 can you restart failing jobs please, thanks.


Question regarding merging 3 VLP-16

Hello, I am working with a setup which merges 3 VLP-16 together and publishes the combined point cloud w.r.t baselink. Among the 3 llidars. one is placed horizontally and other 2 are tilted at a certain angle. So I was wondering what things I have to change inside config/vlp16_example.xml file.
N.B: When I imported my combined point cloud data with vlp16_example.xml file, it was looking good. I can see the point clouds and the rings generated by 3 different pointclouds well. Though it looks satisfactory visually, I wanted to reassure that I am not loosing any pointcloud data after importing inside SuMa.

Thanks for the info. Actually I am just using it to generate the 'poses.txt' file so that I can import the pointclouds inside 'point labeler' in kitti format. I tried top single point cloud, a combined 3 pointcloud and a combined 3 point cloud with filters (downsampled). However, the combined 3 pointcloud performs the best while others perform good (filtered one) and average (single pointcloud).


[ci] fix artifact name conflict

issue: https://github.com/openthread/openthread/actions/runs/7752469334/job/21142026450?pr=9825


Add quick start instructions in the quick start section of the website

Addition of some handy quick start instructions are needed the the quick start guide of the documentation.

Working on it !


OS 4.7.1

Pull Request Test Coverage Report for Build 1013

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 58.535%

Totals

Change from base Build 1011:
-0.1%

Covered Lines:
991

Relevant Lines:
1693

💛  - Coveralls


[core-http] Should we extract TokenCredential to a smaller library?

While @ShivangiReja has been updating core-amqp and event-hubs to support @azure/identity, we've noticed that it seems pretty wasteful to depend on the entirety of @azure/core-http just to get TokenCredential and its related types.
Would it make sense to create a @azure/core-auth library that just contains these interfaces and small helpers?
/cc @ramya-rao-a @bterlson @schaabs @AlexGhiondea

I think that's a great idea. Do you have an idea about which other types and helpers you'd want to put here?

Yep, here's a list:

TokenCredential
GetTokenOptions
AccessToken
isTokenCredential

My plan would be to create another core library containing these interfaces and function and then use it in core-http, re-exporting those types so that a consumer of core-http doesn't have to directly depend on core-auth also.
I'll kick off a draft PR so that we can see how it looks and whether it makes sense in the short term.

Created PR #3964 with a full implementation of this idea, let me know what you think!

Users of identity with event-hubs, etc will pull in core-http as a transitive dependency, enabling the interaction with AAD to get tokens without core-amqp having to be involved (or depend directly on core-http).

According to the new plan, core-amqp would take dependency on core-auth instead of core-http. So, why would user of identity with event-hubs pull core-http? Or were you talking about what's happening currently?

identity needs core-http to make the request to AAD for the token that will be given to core-amqp.  core-amqp or event-hubs would never be involved in that process though, everything will be handled in the TokenCredential implementation from identity, invoked by the user of event-hubs.


docs: fix typo README.md

"develoeprs" → "developers"
Fixed a typo to make the document clearer. Hope this fix helps.

Thanks for making this change @osrm!


Remove unused vitalfrog github reference

Description
Remove unused "github.com/vitalfrog/jsonl" from tests files to avoid go mod issues
https://github.com/simonfrey/jsonl/blob/main/reader_test.go#L7C2-L7C30
https://github.com/simonfrey/jsonl/blob/main/writer_test.go#L5
go mod tidy issue
$ go mod tidy
go: finding module for package github.com/vitalfrog/jsonl
go: found github.com/vitalfrog/jsonl in github.com/vitalfrog/jsonl v0.0.0-20230718192858-195f01ae6957
go: github.com/firebolt-analytics/auth0-parser imports
        github.com/simonfrey/jsonl tested by
        github.com/simonfrey/jsonl.test imports
        github.com/vitalfrog/jsonl<EMAIL_ADDRESS>parsing go.mod:
        module declares its path as: github.com/simonfrey/jsonl
                but was required as: github.com/vitalfrog/jsonl

Hi, it seems to be blocking any go.mod updates after adding this lib.
Can we please fix this?
Thanks


SqlAG: The cluster does not have permissions to manage the Availability Group

Details of the scenario you tried and the problem that is occurring
I'm receiving the following error message when using the SqlAg resource to create an availability group: -
VERBOSE: [TESTNCLSQL1]: LCM:  [ Start  Resource ]  [[SqlAG]ConfigureAG]
VERBOSE: [TESTNCLSQL1]: LCM:  [ Start  Test     ]  [[SqlAG]ConfigureAG]
VERBOSE: [TESTNCLSQL1]:                            [[SqlAG]ConfigureAG] Information: PowerShell module SqlServer not found, trying to use older SQLPS module.
VERBOSE: [TESTNCLSQL1]:                            [[SqlAG]ConfigureAG] Importing PowerShell module SQLPS.
VERBOSE: [TESTNCLSQL1]:                            [[SqlAG]ConfigureAG] Connected to SQL instance 'testnclsql1'.
VERBOSE: [TESTNCLSQL1]: LCM:  [ End    Test     ]  [[SqlAG]ConfigureAG]  in 1.1110 seconds.
VERBOSE: [TESTNCLSQL1]: LCM:  [ Start  Set      ]  [[SqlAG]ConfigureAG]
VERBOSE: [TESTNCLSQL1]:                            [[SqlAG]ConfigureAG] Information: PowerShell module SqlServer not found, trying to use older SQLPS module.
VERBOSE: [TESTNCLSQL1]:                            [[SqlAG]ConfigureAG] Importing PowerShell module SQLPS.
VERBOSE: [TESTNCLSQL1]:                            [[SqlAG]ConfigureAG] Information: PowerShell module SqlServer not found, trying to use older SQLPS module.
VERBOSE: [TESTNCLSQL1]:                            [[SqlAG]ConfigureAG] Importing PowerShell module SQLPS.
VERBOSE: [TESTNCLSQL1]:                            [[SqlAG]ConfigureAG] Connected to SQL instance 'testnclsql1'.
VERBOSE: [TESTNCLSQL1]:                            [[SqlAG]ConfigureAG] The recommended account 'NT SERVICE\ClusSvc' is missing one or more of the following permissions: Trying with 'NT AUTHO
RITY\SYSTEM'.
VERBOSE: [TESTNCLSQL1]:                            [[SqlAG]ConfigureAG] Getting the effective permissions for the login 'NT AUTHORITY\SYSTEM' on 'MSSQLSERVER'.
VERBOSE: [TESTNCLSQL1]:                            [[SqlAG]ConfigureAG] Information: PowerShell module SqlServer not found, trying to use older SQLPS module.
VERBOSE: [TESTNCLSQL1]:                            [[SqlAG]ConfigureAG] Importing PowerShell module SQLPS.
VERBOSE: [TESTNCLSQL1]:                            [[SqlAG]ConfigureAG] Connected to SQL instance 'TESTNCLSQL1'.
VERBOSE: [TESTNCLSQL1]:                            [[SqlAG]ConfigureAG] The account 'NT AUTHORITY\SYSTEM' is missing one or more of the following permissions: Connect SQL, Alter Any Availabil
ity Group, View Server State
VERBOSE: [TESTNCLSQL1]: LCM:  [ End    Set      ]  [[SqlAG]ConfigureAG]  in 0.7650 seconds.
PowerShell DSC resource MSFT_SqlAG  failed to execute Set-TargetResource functionality with error message: The cluster does not have permissions to manage the Availability Group on
'TESTNCLSQL1\MSSQLSERVER'. Grant 'Connect SQL', 'Alter Any Availability Group', and 'View Server State' to either 'NT SERVICE\ClusSvc' or 'NT AUTHORITY\SYSTEM'.

I've specified PSDscRunAsCredentials as the SQL service account so expected this creation to be carried out using that account, but according to the verbose output it first tries to use 'NT AUTHORITY\ClusSvc' (which I don't think exists in SQL 2016?) then 'NT AUTHORITY\SYSTEM'.  Note that the SQL service account (domain\testsqlagent) has the sysadmin role in SQL and has been used successfully by other resources without issue.
The DSC configuration that is using the resource (as detailed as possible)
AvailabilityGroup = 'testnclsql';
InstanceName = "MSSQLSERVER";
NodeName = "testnclsql1";
SQLAccountCreds = New-Object System.Management.Automation.PSCredential ('testncl\testsqlAgent', (ConvertTo-SecureString $SQLcreds -AsPlainText -Force));

SqlAG ConfigureAG
{
    DependsOn = ("[xCluster]ensureCreated"),("[SqlSetup]" + $Node.NodeName),("[SqlServerEndpointState]EnableEndpoint")
    Ensure = 'Present'
    Name = $Node.AvailabilityGroup
    InstanceName = $Node.InstanceName
    ServerName = $Node.NodeName
    AvailabilityMode = 'SynchronousCommit'
    AutomatedBackupPreference = 'None'
    FailoverMode = 'automatic'
    PsDscRunAsCredential = $Node.SQLAccountCreds
}

I think that's all the relevant code, please let me know if I've missed something.
Version of the operating system and PowerShell the target node is running
Caption                                OSArchitecture Version    MUILanguages $PSVersionTable.PSVersion
-------                                -------------- -------    ------------ -------------------------
Microsoft Windows Server 2016 Standard 64-bit         10.0.14393 {en-US}      5.1.14393.1770

SQL Server edition and version the target node is running
Edition  : Standard Edition
Version  : 13.0.1601.5
Language : 1033

What SQL Server PowerShell modules, and which version, are present on the target node.
Name  Version Path
----  ------- ----
SQLPS 1.0     C:\Program Files (x86)\Microsoft SQL Server\130\Tools\PowerShell\Modules\SQLPS\SQLPS.psd1

Version of the DSC module you're using, or write 'dev' if you're using current dev branch
Name         Version  Path
----         -------  ----
SqlServerDsc <IP_ADDRESS> C:\Program Files\WindowsPowerShell\Modules\SqlServerDsc\<IP_ADDRESS>\SqlServerDsc.psd1

Hi @iain83, The message indicates the fact that the Windows Server Failover Cluster (WSFC) does not have the credentials necessary to log into SQL Server and manage the Availability Group. The WSFC is required for Availability Groups to work.
To resolve the issue, it is recommended to create a login for NT SERVICE\ClusSvc as this follows the model of least privilege. Once the login is created, grant it the Connect SQL, Alter Any Availability Group, and View Server State permissions.

Thanks for the response @randomnote1 .
This is my second time using this code to deploy SQL, the first time round I managed to get this working without adding ClusSvc, unfortunately it was a manual undocumented change (should have used DSC!) so I'm now stuck.  I'm sure it was a computer/person account in AD that needed permissions over another account, but I've since removed and recreated those accounts also.
Do you have any idea what it may be?

Chances are you added the required permissions to NT AUTHORITY\SYSTEM. I think the AG wizard does that on its own if you use it to create the AG.
No worries, I did a ton of that kind of stuff when I was learning DSC! It's part of the learning process! 😄

Thanks again @randomnote1 .  I don't remember doing that, but you never know.  I've been battling with the SqlServerPermissions resource to create 'NT Service\ClusSvc' with various errors.  Eventually I've resolved everything but am now getting a very ambiguous error.  Here's the code: -
ClusSvcCreds              = New-Object System.Management.Automation.PSCredential ('nousername', (ConvertTo-SecureString $ClusSvcPass -AsPlainText -Force));

Importing the parameter as a PSCredential object has never worked for me, so I'm converting it with the above.
SqlServerLogin CreateCluSsvc
{
   DependsOn = ("[SqlSetup]" + $Node.NodeName)
   Ensure = 'Present'
    ServerName = $Node.NodeName
    InstanceName = $Node.InstanceName
    Name = 'NT SERVICE\ClusSvc'
    LoginCredential = $Node.ClusSvcCreds
    LoginType = 'SqlLogin'
    LoginMustChangePassword = $false
    LoginPasswordExpirationEnabled = $false
    LoginPasswordPolicyEnforced = $false
    PsDscRunAsCredential = $Node.SQLAccountCreds
}

The error message I now get is: -
VERBOSE: [TESTNCLSQL1]: LCM:  [ Start  Resource ]  [[SqlServerLogin]CreateCluSsvc]
VERBOSE: [TESTNCLSQL1]: LCM:  [ Start  Test     ]  [[SqlServerLogin]CreateCluSsvc]
VERBOSE: [TESTNCLSQL1]:                            [[SqlServerLogin]CreateCluSsvc] Information: PowerShell module SqlServer not found, trying to use older SQLPS module.
VERBOSE: [TESTNCLSQL1]:                            [[SqlServerLogin]CreateCluSsvc] Importing PowerShell module SQLPS.
VERBOSE: [TESTNCLSQL1]:                            [[SqlServerLogin]CreateCluSsvc] Connected to SQL instance 'testnclsql1'.
VERBOSE: [TESTNCLSQL1]:                            [[SqlServerLogin]CreateCluSsvc] Getting SQL logins
VERBOSE: [TESTNCLSQL1]:                            [[SqlServerLogin]CreateCluSsvc] 2018-03-26_12-16-58: Getting the login 'NT SERVICE\ClusSvc' from 'testnclsql1\MSSQLSERVER'
VERBOSE: [TESTNCLSQL1]:                            [[SqlServerLogin]CreateCluSsvc] 2018-03-26_12-16-58: The login 'NT SERVICE\ClusSvc' is Absent from the 'testnclsql1\MSSQLSERVER' instance.
VERBOSE: [TESTNCLSQL1]:                            [[SqlServerLogin]CreateCluSsvc] 2018-03-26_12-16-58: The login 'NT SERVICE\ClusSvc' on the instance 'testnclsql1\MSSQLSERVER' is Absent rath
er than Present
VERBOSE: [TESTNCLSQL1]:                            [[SqlServerLogin]CreateCluSsvc] 2018-03-26_12-16-58: The login 'NT SERVICE\ClusSvc' on the instance 'testnclsql1\MSSQLSERVER' is a  rather t
han SqlLogin
VERBOSE: [TESTNCLSQL1]:                            [[SqlServerLogin]CreateCluSsvc] 2018-03-26_12-16-58: The login 'NT SERVICE\ClusSvc' on the instance 'testnclsql1\MSSQLSERVER' has PasswordEx
pirationEnabled set to  rather than False
VERBOSE: [TESTNCLSQL1]:                            [[SqlServerLogin]CreateCluSsvc] 2018-03-26_12-16-58: The login 'NT SERVICE\ClusSvc' on the instance 'testnclsql1\MSSQLSERVER' has PasswordPo
licyEnforced set to  rather than False
VERBOSE: [TESTNCLSQL1]: LCM:  [ End    Test     ]  [[SqlServerLogin]CreateCluSsvc]  in 1.1720 seconds.
VERBOSE: [TESTNCLSQL1]: LCM:  [ Start  Set      ]  [[SqlServerLogin]CreateCluSsvc]
VERBOSE: [TESTNCLSQL1]:                            [[SqlServerLogin]CreateCluSsvc] Information: PowerShell module SqlServer not found, trying to use older SQLPS module.
VERBOSE: [TESTNCLSQL1]:                            [[SqlServerLogin]CreateCluSsvc] Importing PowerShell module SQLPS.
VERBOSE: [TESTNCLSQL1]:                            [[SqlServerLogin]CreateCluSsvc] Connected to SQL instance 'testnclsql1'.
VERBOSE: [TESTNCLSQL1]:                            [[SqlServerLogin]CreateCluSsvc] 2018-03-26_12-16-58: Adding the login 'NT SERVICE\ClusSvc' to the 'testnclsql1\MSSQLSERVER' instance.
VERBOSE: [TESTNCLSQL1]: LCM:  [ End    Set      ]  [[SqlServerLogin]CreateCluSsvc]  in 0.5160 seconds.
PowerShell DSC resource MSFT_SqlServerLogin  failed to execute Set-TargetResource functionality with error message: Creation of the login 'NT SERVICE\ClusSvc' failed due to a failed
operation.
    + CategoryInfo          : InvalidOperation: (:) [], CimException
    + FullyQualifiedErrorId : ProviderOperationExecutionFailure
    + PSComputerName        : localhost

If I manually try to add 'NT SERVICE\ClusSvc' I receive an "not a valud name because it contains invalid characters" error, which is presumably what is causing the DSC error?  I've been unable to find anything about manually creating the ClusSvc account, do you know of any guides or do you know how I can correct this problem?

Well, that error isn't very helpful!
Take a look at the example for creating an Availability Group. It has an example of how to create NT Service\ClusSvc in it here.

@iain83 You should add the account as a WindowsUser instead of SqlLogin.
Saw some (minor) erroneous output in the verbose output above, not related to this issue, so I will add a new issue to track that.

Thanks @johlju , I spotted the differences between the example and my code earlier.  Rookie mistakes on my part!  The AG is created and I just got the AG listener up and running too.
Thanks for your work on this module, it's been really helpful.

@iain83 Thanks for using it, and most credit should go to the community for contributing to the resources, like @randomnote1's work on the AG resources . :smile:
Closing this issue as this one was solved by changing to 'WindowsUser'.


[ru] Localize api-gateway.md

Description
This PR adds RU localization of API Gateway
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

@tym83 please approve 😃


Automatically escape JSON attributes starting with @

From<EMAIL_ADDRESS>on December 13, 2011 12:44:05
E.g. this should work:
expect("body.@id").
instead of
expect("body.'@id'").
Original issue: http://code.google.com/p/rest-assured/issues/detail?id=138

From<EMAIL_ADDRESS>on December 13, 2011 05:05:10
Status: Fixed


Don't use sudo to build docker image

The docker cli should not need sudo to talk to the docker daemon. On a linux system you can do sudo usermod -aG docker $USER and you'll get access to the docker.sock (check the ownership of the sock to make sure you're doing the right usermod). On mac os x the docker for mac will set up docker properly, no sudo needed.
Open to a PR to fix this?

Sure. We touched upon this in the thread on HN, but due to popular demand I don't see why not drop it. Please don't forget the -s flag when committing, to make the DCO bot happy 😄


trying to connect to a kubernetes cluster created with tools/operator using java SDK

I am super new to fabric. Can someone please point me to how i would go about connecting a java app using the fabric java SDK ?  I can contribute a sample app as a PR if that would be something people can use.
I am trying to make sense of this fabric-samples java project. I noticed the operator generates connection profiles and ca certs for all orgs. However its not clicking how my client can talk to the cluster stood up by the operator using those generated files.
Can anyone help ?

`//        public static boolean USE_PROFILE = true;
public static boolean USE_PROFILE = false;
public static String BUILD_DIR = "build/fabric-000";
public static String CRYPTO_DIR = BUILD_DIR + "/crypto-config";

public static String ORG1_PEER = CRYPTO_DIR + "/peerOrganizations/org1";
public static String ORG1_ORDERER = CRYPTO_DIR + "/ordererOrganizations/ordererorg1";

public static String ORG1_CA_CERT = ORG1_PEER + "/ca/ca.org1-cert.pem";

//    public static String ORG1_CA_CERT =ORG1_ORDERER+"/ca/ca.ordererorg1-cert.pem";
public static String ORG1_CONNECTION_PROFILE = "connection-profile/connection_profile_org1.yaml";

//    public static String ORG1_SERVER_CERT = "orderers/orderer0-ordererorg1.ordererorg1/tls/server.crt";
private static NetworkConfig networkConfig;

public static void main(String[] args) throws Exception {

    File yamlFile = Paths.get(BUILD_DIR, ORG1_CONNECTION_PROFILE).toFile();
    networkConfig = NetworkConfig.fromYamlFile(yamlFile);

    networkConfig.getOrdererNames().forEach(ordererName -> {
        try {
            System.out.println("orderer: " + ordererName);

            Properties ordererProperties = networkConfig.getOrdererProperties(ordererName);
            Properties testProp = getEndPointProperties("orderer", ordererName);
            ordererProperties.setProperty("clientCertFile", testProp.getProperty("clientCertFile"));
            ordererProperties.setProperty("clientKeyFile", testProp.getProperty("clientKeyFile"));
            networkConfig.setOrdererProperties(ordererName, ordererProperties);
        } catch (InvalidArgumentException e) {
            throw new RuntimeException(e);
        }
    });

    networkConfig.getPeerNames().forEach(peerName -> {
        try {
            System.out.println("peer: " + peerName);

            Properties peerProperties = networkConfig.getPeerProperties(peerName);
            Properties testProp = getEndPointProperties("peer", peerName);
            peerProperties.setProperty("clientCertFile", testProp.getProperty("clientCertFile"));
            peerProperties.setProperty("clientKeyFile", testProp.getProperty("clientKeyFile"));
            networkConfig.setPeerProperties(peerName, peerProperties);

        } catch (InvalidArgumentException e) {
            throw new RuntimeException(e);
        }
    });

    HFCAClient hfcaClient;

    if (USE_PROFILE) {
        //Check if we get access to defined CAs!
        NetworkConfig.OrgInfo org = networkConfig.getOrganizationInfo("org1");
        NetworkConfig.CAInfo caInfo = org.getCertificateAuthorities().get(0);
        Properties caProps = caInfo.getProperties();

        hfcaClient = HFCAClient.createNewInstance(caInfo);
        CryptoSuite cryptoSuite = CryptoSuiteFactory.getDefault().getCryptoSuite();
        hfcaClient.setCryptoSuite(cryptoSuite);
        caProps.put("allowAllHostNames", "true");
        System.out.println("hfcaClient CAName:" + hfcaClient.getCAName());

//            props.put("pemBytes", (byte[]) pemBytes);
//            Object pemBytes = caProps.get("pemBytes");
//            if (!(pemBytes instanceof byte[])) {
//                throw new RuntimeException("Expecting bytes in pemBytes");
//            }
} else {
File pemFile = Paths.get(ORG1_CA_CERT).toFile();
if (!pemFile.exists()) {
throw new RuntimeException(String.format("Missing pem file Could not find at location: %s", pemFile.getAbsolutePath()));
}
        Properties props = new Properties();
        props.put("allowAllHostNames", "true");
        props.put("pemFile", pemFile.getAbsolutePath());

        hfcaClient = HFCAClient.createNewInstance("https://localhost:7054", props);
        CryptoSuite cryptoSuite = CryptoSuiteFactory.getDefault().getCryptoSuite();
        hfcaClient.setCryptoSuite(cryptoSuite);
    }

    HFCAInfo info = hfcaClient.info(); //makes actual REST call.
    System.out.println("info CAName:" + info.getCAName());

//        Collection<NetworkConfig.UserInfo> registrars = caInfo.getRegistrars();
//        NetworkConfig.UserInfo registrar = registrars.iterator().next();
//        registrar.setEnrollment(hfcaClient.enroll(registrar.getName(), registrar.getEnrollSecret()));
}
`
in both cases (when i enable using the profile or when i use the pem file directly i get this :
orderer: orderer0-ordererorg1 peer: peer0-org1 PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target javax.net.ssl.SSLHandshakeException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target 	at java.base/sun.security.ssl.Alert.createSSLException(Alert.java:131) 	at java.base/sun.security.ssl.TransportContext.fatal(TransportContext.java:325) 	at java.base/sun.security.ssl.TransportContext.fatal(TransportContext.java:268) 	at java.base/sun.security.ssl.TransportContext.fatal(TransportContext.java:263) 	at java.base/sun.security.ssl.CertificateMessage$T13CertificateConsumer.checkServerCerts(CertificateMessage.java:1340) 	at java.base/sun.security.ssl.CertificateMessage$T13CertificateConsumer.onConsumeCertificate(CertificateMessage.java:1215) 	at java.base/sun.security.ssl.CertificateMessage$T13CertificateConsumer.consume(CertificateMessage.java:1158) 	at java.base/sun.security.ssl.SSLHandshake.consume(SSLHandshake.java:396) 	at java.base/sun.security.ssl.HandshakeContext.dispatch(HandshakeContext.java:445) 	at java.base/sun.security.ssl.HandshakeContext.dispatch(HandshakeContext.java:423) 	at java.base/sun.security.ssl.TransportContext.dispatch(TransportContext.java:182) 	at java.base/sun.security.ssl.SSLTransport.decode(SSLTransport.java:171) 	at java.base/sun.security.ssl.SSLSocketImpl.decode(SSLSocketImpl.java:1475) 	at java.base/sun.security.ssl.SSLSocketImpl.readHandshakeRecord(SSLSocketImpl.java:1381) 	at java.base/sun.security.ssl.SSLSocketImpl.startHandshake(SSLSocketImpl.java:441) 	at java.base/sun.security.ssl.SSLSocketImpl.startHandshake(SSLSocketImpl.java:412) 	at org.apache.http.conn.ssl.SSLSocketFactory.createLayeredSocket(SSLSocketFactory.java:570) 	at org.apache.http.conn.ssl.SSLSocketFactory.connectSocket(SSLSocketFactory.java:554) 	at org.apache.http.impl.conn.DefaultHttpClientConnectionOperator.connect(DefaultHttpClientConnectionOperator.java:142) 	at org.apache.http.impl.conn.PoolingHttpClientConnectionManager.connect(PoolingHttpClientConnectionManager.java:376) 	at org.apache.http.impl.execchain.MainClientExec.establishRoute(MainClientExec.java:393) 	at org.apache.http.impl.execchain.MainClientExec.execute(MainClientExec.java:236) 	at org.apache.http.impl.execchain.ProtocolExec.execute(ProtocolExec.java:186) 	at org.apache.http.impl.execchain.RetryExec.execute(RetryExec.java:89) 	at org.apache.http.impl.execchain.RedirectExec.execute(RedirectExec.java:110) 	at org.apache.http.impl.client.InternalHttpClient.doExecute(InternalHttpClient.java:185) 	at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:83) 	at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:56) 	at org.hyperledger.fabric_ca.sdk.HFCAClient.httpPost(HFCAClient.java:1326) 	at org.hyperledger.fabric_ca.sdk.HFCAClient.info(HFCAClient.java:559) 	at application.java.Network.main(Network.java:104)

@mbwhite Im not sure what unable to find valid certification path to requested target means in the error stack, is it looking for a truststore rather than a single pem?
Any chance you can provide some insight here, as I don't think its an actual issue with the network we generate from the operator tool.

@bestbeforetoday might be the best placed if I'm honest from a client SDK perspective.
@ymolists to clarify were you deploying Fabric into K8S using your own deployments?

Thank Matthew. He is using our Fabric-Test tool to deploy the network. I know it works with the Node SDK, but I've never used the Java SDK for anything.

@mbwhite as @Iindluni  said am using fabric-test/tools/operator to deploy the cluster on k8s.
i observed the following on the ca-org1 pod
ca
2021/03/11 17:31:12 [INFO] The issuer key was successfully stored. The public key is at: /etc/hyperledger/fabric-ca-server/IssuerPublicKey, secret key is at: /etc/hyperledger/fabric-ca-server/msp/keystore/IssuerSecretKey
ca
2021/03/11 17:31:12 [DEBUG] Intializing revocation authority for issuer 'ca0-org1'
ca
2021/03/11 17:31:12 [DEBUG] Initialize Idemix issuer revocation key material
ca
2021/03/11 17:31:12 [INFO] Idemix issuer revocation public and secret keys were generated for CA 'ca0-org1'
ca
2021/03/11 17:31:12 [INFO] The revocation key was successfully stored. The public key is at: /etc/hyperledger/fabric-ca-server/IssuerRevocationPublicKey, private key is at: /etc/hyperledger/fabric-ca-server/msp/keystore/IssuerRevocationPrivateKey
ca
2021/03/11 17:31:12 [DEBUG] Intializing nonce manager for issuer 'ca0-org1'
ca
2021/03/11 17:31:12 [INFO] Home directory for default CA: /etc/hyperledger/fabric-ca-server
ca
2021/03/11 17:31:12 [DEBUG] 1 CA instance(s) running on server
ca
2021/03/11 17:31:12 [INFO] Operation Server Listening on <IP_ADDRESS>:9443
ca
2021/03/11 17:31:12 [DEBUG] TLS is enabled
ca
2021/03/11 17:31:12 [DEBUG] TLS Certificate: /etc/hyperledger/fabric/artifacts/tlsca.org1-cert.pem, TLS Key: /etc/hyperledger/fabric/artifacts/tlsca-priv_sk
ca
2021/03/11 17:31:12 [DEBUG] Could not load TLS certificate with BCCSP: Could not find matching private key for SKI: Failed getting key for SKI [[243 56 28 210 58 129 20 52 139 112 209 223 11 153 56 252 91 213 147 18 125 184 245 193 9 67 8 183 128 57 154 255]]: Key with SKI f3381cd23a8114348b70d1df0b9938fc5bd593127db8f5c1094308b780399aff not found in /etc/hyperledger/fabric-ca-server/msp/keystore
ca
2021/03/11 17:31:12 [DEBUG] Attempting fallback with certfile /etc/hyperledger/fabric/artifacts/tlsca.org1-cert.pem and keyfile /etc/hyperledger/fabric/artifacts/tlsca-priv_sk
ca
2021/03/11 17:31:12 [DEBUG] Client authentication type requested: noclientcert
ca
2021/03/11 17:31:12 [INFO] Listening on https://<IP_ADDRESS>:7054

it seems like the container is not able to load the ca cert file and defaulting to use the tls cert instead. ? After i noticed that i changed my above java code to use the tls cert instead of the ca cert i was using and voila !
here is where i used the the tls certs in the new code
I checked the cert saved in the connection profile  by the fabric-test tool is using the ca cert not the tls cert ? that section of the code is failing for now.


V2 - Issues with the email sent to contributor (who is assigned with company-owner role) to complete Oraganization profile

Prerequisite

Create a CLA group with CCLA enabled
Add github repository for the CLA group
Generate template

Steps to Follow

Login to Github as corporate contributor (with private email in github) who has LFID
Navigate to the repository used in prerequisite
Create PR
CLA check fails
Click on CLA NOT SIGNED
Contributor is redirected to Contributor Console
Click on Proceed as a Corporate Contributor
Click on + Click Here button in drop down in Select Company screen
Enter below details
companyName Comp1Jul102020
companyWebsite https://Comp2Jul102020.com
Click Next
Observed the contributor is assigned with company-owner role and and email is sent to the user to complete organization profile
However observed below issues with the email sent
Issue1
Email subject is different and needs to be changed

Issue2
When clicked on the link in the email and logged in, company-owner is redirected to Dashboard instead of Organization Profile page in LFX

Tested and observed that Issue2 is still reproducible. When clicked on the link in the email and logged in, company-owner is redirected to Dashboard instead of Organization Profile page in LFX

Verified that Issue1 is fixed

@nirupamav could you test the redirect again as on my end it goes to the organization profile page

Testing blocked due to #1683

Verified that the issues are fixed


How to maximize the windows ?

This will probably sounds too dumb but how can I maximize the oni? I tried setting the background image size to either "cover" or "contain" but doesn't seem like its working.

Hi @HiCodin ,
Which platform are you using? Are you looking to have ONI default to maximize on startup? I don't think there is actually a setting for that, but it would make sense to have one for sure

I am on El Capitan and yes, i'd like to have oni default to maximize on startup or at least be able to specify the size. I thought it's already in the config and i just can't find it. :D

Seems like the work here would be:

Add a new config setting, like oni.editor.maximizeOnStart to Config.ts
In our index file, add a call to the browser window API that maximizes our current window: https://github.com/electron/electron/blob/master/docs/api/browser-window.md

@extr0py ,  I think we need to add an "OnStart" field in config.json instead of having editor.action and editor.actionOnStart for everything. Just a thought

@extr0py not to be dense, but would this be the UI or regular index

@extr0py I don't mean to be destroying your inbox, just looking for input on the last comment. I know I'm asking enough that you might as well do it yourself, but this is a learning process for me too :)

Hey @bert88sta , oh no worries :) It's great to see your initiative! Sorry if my responses are a bit delayed... I promise I'm not ignoring you, work has just been taking up a lot of my time this past couple of weeks.
You've done an excellent job diving in - yes, this method remote.BrowserWindow.getFocusedWindow().maximize() is exactly what we need.
In terms of implementation, it does seem like we have two potential ways we can implement it:

main.js - on the upside, this would let us maximize immediately, but on the downside, there isn't any config plumbing there (at the moment).
'browser\src\index.tsx` - on the upside, this has config stuff already there - like setting the font value. The downside though is as you mentioned it starts out not fullscreen and then maximizes.

I think, for now, implementing in browser\src\index.tsx would be the most straightforward, since the config info is already there. Ideally, we can improve startup time so that delta is less visible. I like to defer to the simplest solution barring any other constraints. Somewhere in that start method - probably towards the top. Plus, if we can fix the startup time, it begins a non-issue
For the project structure, you have a great point, it might be worth adding some notes in the CONTRIBUTING.md about the structure. I'm thinking of some starting notes like:

\

main.js - entry point for the app that creates the app window
browser\src - ONI application code (UI / neovim integration)

browser\src\index.tsx - initialization logic that starts up Neovim, hooks up to the interop layer
browser\src\UI - folder that contains all the rich UI components (React + Redux)
browser\src\Services - contains editor services like live evaluation, quick open, error management

vim - packaged vim plugins, default plugins
plugin_api - API surface that is exposed to javascript plugins

But it'd be helpful to get your ideas since you have a fresh perspective-
Hope that helps! Thanks for your contributions

@HiCodin:
Add to your Config.json
editor.fullScreenOnStart : true
as an option. See PR #203

@bert88sta @extr0py Thanks for the hard work. the fullscreen option is working perfectly. 👍

Awesome, thanks for confirming @HiCodin - and thanks for the fix @bert88sta ! Will close this out. We might want to add this to the README.md as well as one of the configuration options-


Issue in getting timezone info when hosted in Linux

Below method is giving TimeZoneInfo when we hosted in windows but it is failing to give timezone infor  when we hosted in Linux
TimeZoneInfo.FindSystemTimeZoneById("Pacific Standard Time")
Error:  The time zone ID 'Pacific Standard Time' was not found on the local computer

Does https://devblogs.microsoft.com/dotnet/cross-platform-time-zones-with-net-core/ address some of your concerns?


This browser or app may not be secure.

Any ideas on how to fix this? I've tried copying my firefox profile folder from my local comp into this scripts profile folder but no success

Are you using the correct firefox profile? It works for me when I specify a profile path.
# export PROFILE_PATH="C:\Users\chena\AppData\Roaming\Mozilla\Firefox\Profiles\1cyrfe0a.default-release-1676168063686"
uploader = YouTubeUploader(video_path, metadata_path, profile_path=os.getenv('PROFILE_PATH'))
was_video_uploaded, video_id = uploader.upload()

@chenaaron3 Had a try with your method but still error:
TypeError: Firefox.__init__() got an unexpected keyword argument 'profile_path'

@chenaaron3 Had a try with your method but still error:
TypeError: Firefox.__init__() got an unexpected keyword argument 'profile_path'

@chenaaron3 Had a try with your method but still error:
TypeError: Firefox.__init__() got an unexpected keyword argument 'profile_path'

Same error

Are you using the correct firefox profile? It works for me when I specify a profile path.
# export PROFILE_PATH="C:\Users\chena\AppData\Roaming\Mozilla\Firefox\Profiles\1cyrfe0a.default-release-1676168063686"
uploader = YouTubeUploader(video_path, metadata_path, profile_path=os.getenv('PROFILE_PATH'))
was_video_uploaded, video_id = uploader.upload()

This method solved my problem, and I can now log into YouTube.

Are you using the correct firefox profile? It works for me when I specify a profile path.
# export PROFILE_PATH="C:\Users\chena\AppData\Roaming\Mozilla\Firefox\Profiles\1cyrfe0a.default-release-1676168063686"
uploader = YouTubeUploader(video_path, metadata_path, profile_path=os.getenv('PROFILE_PATH'))
was_video_uploaded, video_id = uploader.upload()

this doesnt work for me and i constantly get An error occurred during upload: YouTubeUploader.init() got an unexpected keyword argument 'profile_path' did you modify your version?
edit: i was using the pypi latest version which frankly isnt updated to the most recent version of this github repo, i just manually replaced the code in my venv and it may work now

My previous question was that I had never used Firefox's profile feature before, so I couldn't understand the instructions in the installation guide. Later, I searched online about profiles and learned that it's a Firefox feature for customizing profiles. You first create a Firefox profile, then fill in the profile path in the program. I've been using this program for several months now, and the main part of the program works fine.

profile_path

Can you tell me the string of the profile_path?

no, its different for each person but you would find it in about:profiles 90% of the time

I just want to confirm that you have added the correct Firefox profile path. I think you should be able to use it normally.


Problems when canceling a gate repair

When the gate is destroyed and you cancel the repair, this happens

@TracyTheBoy The bug here is that the gate looks like it's in-tact despite not being repaired, right?

Yes that is the bug. You can still path through it so it is just a visual issue


chore: split Algebra.Submonoid.Membership

This file had two unrelated parts, one about big operators and one about other facts about membership.

Will need a further shake.

Thanks!
bors merge


Report if VeraPDF crashes

In rare conditions verapdf can run into an exception, that currently returns an empty xml as result but it should be a proper error

to fix your trouble check this solution click
maybe this will solve your problem.

+1


What about emojis?

If the goal is to fit as much text as possible in a tweet, why not using emojis as well?

Good thought. You are describing another encoding which already exists: Ecoji.

Ecoji encodes 10 bits per emoji * 140 emoji per Tweet / 8 bits per byte = 175 bytes per Tweet.
Base2048 encodes 11 bits per character * 280 characters per Tweet / 8 bits per byte = 385 bytes per Tweet, which is more than twice as efficient.

I see. My suggestions was using both Base2048 characters and emojis, rather then only emojis.

The efficiency gains from that would be negligible because emoji are so large (effectively twice the size of a Base2048 character) and there are so few of them.
Base2048 encodes 22 bits per 2 characters. Let's say we wanted to increase that to 23 bits per 2 characters, by adding emoji. We would need 2^22 ~= 4,000,000 emoji to do this. However, there are only approximately 1000-2000 of them.

Oh, so an emoji counts as two characters on Twitter, I was missing that piece of info.
Got it, thanks! 🙂


Add end-to-end behavioural tests for the python CLI

Fixes #970.
This PR includes end-to-end behavioural test for the spark_rapids CLI. A high level directory structure and more details on usage and debugging are provided at docs.
Setup
From the <repo_root>/user_tools directory, run the following command to install the required dependencies:
pip install behave
# or
pip install .[tests]

Running Tests
Tests can be run using 'behave' cmd or using 'tox' cmd.
Basic Usage:
behave <options>
# or
tox -e behave -- <options>

Run All Tests:
behave
# or
tox -e behave

Skip building the Tools jar during setup.
behave -D build_jar=false   # Skip building the Tools jar during setup (default: true)
# or
tox -e behave -- -D build_jar=false

More details are provided in the docs.
Notes

Most of the steps are straight forward. Below are the implementation details for some key steps:

"<cli>" is not installed

Create a mock CLI cmd and prepend to PATH

Qualification tool JAR crashes

Mimic crashing by starting a thread that watches for 'QualificationMain' and kills it

HDFS is "running"

Download hadoop binaries and setup a single node HDFS

Impact on Github Actions runtime:

Previously runtime: ~10min.
After including these tests: ~12min

Behave Official Docs: https://behave.readthedocs.io/en/stable/tutorial.html

Also all should be zstd compressed unless we are explicilty testing without it

Thanks @tgravescs. Compressed all event logs to zstd format.

The test_event_log_1 and test_event_log_2 should be more descriptive

Renamed to valid_eventlog_1.zstd

so valid_event_log_1 and 2 aren't exactly what I meant.  what is in that event log? I saw:
| platform | eventlog              |
| onprem   | valid_eventlog_1.zstd |
| dataproc | valid_eventlog_1.zstd |

So why not put onprem, or yarn eventlog, dataproc event log... what code was run in that to generate it?

@tgravescs

So why not put onprem, or yarn eventlog, dataproc event log... what code was run in that to generate it?

Renamed the event log to join_agg_on_yarn_eventlog.zstd
It is a simple query that reads a parquet of 100k rows and does a join -> aggregation -> join.


Storage queue from Alan's PR

[x] Code, Mono -> Flux

[x] Whether merge in DequedMessage and PeekedMessage? No merge. Confirmed from Alan.

[x] Code, getURL() or others? File an issue

[x] Why 'Azure' in AzureQueueStorageBuilder? Already renamed in code.

[x] connectionString() or credential() in builder patter

[ ] Code Is there ever a time a user would want to mutate or instantiate this class? To me it would be reasonable to make this an immutable class? It is from autogeneration. Will sync with Jianghao after blob change.

[ ] Code POJO or Fluent. Will do with POJO

[x] Code This class API and the name of the class are not obviously related. Is UpdatedMessage really the right name here? This data model is used in QueueAsyncClient for updateMessage API, so the name is by design.

Two undone in checklist.
One will follow up with Jianghao.
One will address in this PR.
Other checklist please refer the comments in the list.

Need to have code snippet in next PR.


Resolvers is not a fonction

After updating the package to the latest version and updating all the spread operators to fonctions the project couldnt work. Im having an error in the chrome console telling me that "resolvers" is not à fonction.
Is there anything else to do to upgrade from the previous version of the package ?

Can you share the lines of code where the error is thrown?
Have updated both, the npm package and meteor package?

Both npm package (1.3.0) and meteor package (2.1.0) is updated. I just uninstall them and install again but still have the issue.
Lines of code concerned :
resolvers.js
import { Meteor } from 'meteor/meteor';
import { Resolvers } from 'meteor/nicolaslopezj:apollo-accounts'

const usersResolvers = {
	Query: {
		me (roott, args, context) {
			const { userId } = context ? context : { userId: null };
			if (userId)
			return Meteor.users.findOne({ _id: userId });
		},
		users (root, args) {
			return Meteor.users.find().fetch();
		}
	},
	Mutation: {
		...Resolvers()
	}
};

export default usersResolvers;

Error message in Chrome console

Finally got it working just by deplacing my schema, server side instead of lib.


Uncaught Exception: Attempting to call a function in a renderer window that has been closed or released. Function provid

An uncaught exception was reported.
Info:
OS: win32
Arch: ia32
GPMDP Version: 3.0.1
Time: Sun Mar 20 2016 19:17:31 GMT-0500 (Central Daylight Time)
Error:
Attempting to call a function in a renderer window that has been closed or released. Function provid
Error: Attempting to call a function in a renderer window that has been closed or released. Function provided here: Object.<anonymous> (C:\Users\JonBons\AppData\Local\GPMDP_3\app-3.0.1\resources\app\build\inject\GPMInject\interface\mouseButtonNavigation.js:8:8.
    at BrowserWindow.callIntoRenderer (C:\Users\JonBons\AppData\Local\GPMDP_3\app-3.0.1\resources\atom.asar\browser\lib\rpc-server.js:201:19)
    at emitTwo (events.js:92:20)
    at BrowserWindow.emit (events.js:172:7)

This issue was created automatically inside the uncaughtException handler

Closing this automatically reported issue as a duplicate of #470


VcPkg: use version-semver

@antkmsft can you clarify why version-semver is the preferred name to use? Is there a doc somewhere that recommends using it? The command that converted the CONTROL file to vcpkg.json didn't' use it. I am wondering why.


chore: add dependency scan on PR

Description of changes
This PR adds dependency scan on each PR in order to prevent introduction of vulnerable dependencies. This shortens existing feedback loop, i.e. we'll get failed check before we even merge code.
References:
https://stackoverflow.com/questions/71068152/how-to-trigger-dependabot-scan-on-developer-pull-requests
https://github.blog/changelog/2022-04-06-github-action-for-dependency-review-enforcement/

Issue #, if available

Description of how you validated changes
Happy case

Failure injection.

Checklist

[x] PR description included
[x] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
[ ] Pull request labels are added

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #11676 (6b9bf0f) into dev (77e8657) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev   #11676   +/-   ##
=======================================
  Coverage   50.24%   50.24%
=======================================
  Files         705      705
  Lines       34080    34080
  Branches     6997     6997
=======================================
+ Hits        17123    17124    +1
+ Misses      15477    15476    -1
  Partials     1480     1480

Impacted Files
Coverage Δ

packages/amplify-cli/src/index.ts
66.66% <0.00%> (+0.85%)
:arrow_up:

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


Development active?

Hi, I just wanted to ask if development is still active or not since I'm making a very similar project.

Hi, yes. I created the package to add material design in scope of my project and adding the features required for it.


TfLite stablehlo add sort_op

Adds stablehlo_sort implementation
Adds stablehlo sort unit tests
Adds Comparator Subgraph

We just noticed that sort doesn't support bf16 and f16, as in the subgraph we are adding nodes for comparison which calls ops from comparisons.cc which currently doesen't have support for these data types.
Can we add support for bf16 and f16 in comparisons.cc?


Create an unified coverage report for local unit tests and instumented tests

Changes Unknown when pulling e95b51ed1c376101449259be7db166489ba1df94 on feature/coverage into ** on master**.


Updating the ROS 2 roadmap

This PR updates the ROS 2 roadmap to be a list of quarter goals. The list of feature ideas has been move to Feature-Ideas.
This is my first time contributing here, and using RST. Please point out all of my mistakes.
Signed-off-by: Nate Koenig<EMAIL_ADDRESS>

It would be good to copy the roadmap file to the new name without making any changes in a first commit (which allows being able to traverse the history of that file). All changes should than happen in a separate second commit. The PR should be merged without squashing at the end of the day.
Please check the style guide for something like: each sentence should start on a new line. Also please see the CI output for other minor things to address. Once CI passes the generated pages will be test-deployed and can viewed through the link mentioned.
It would be good to also comment with the items OR commits to for Galactic in the corresponding meta ticket: ros2/ros2#954

Just a friendly bump. Let's try to not let things languish in documentation.

copy the roadmap file to the new name without making any changes in a first commit (which allows being able to traverse the history of that file). All changes shou

It would be good to copy the roadmap file to the new name without making any changes in a first commit (which allows being able to traverse the history of that file). All changes should than happen in a separate second commit. The PR should be merged without squashing at the end of the day.
Please check the style guide for something like: each sentence should start on a new line. Also please see the CI output for other minor things to address. Once CI passes the generated pages will be test-deployed and can viewed through the link mentioned.
It would be good to also comment with the items OR commits to for Galactic in the corresponding meta ticket: ros2/ros2#954

Thanks for the pointers. I'll make a new PR with your suggestion about commits.
For the Galatic ticket, are you thinking of including a link from this document to that meta ticket or something else?

I'll make a new PR with your suggestion about commits.

No need to create a new PR - you can update this one and rewrite the history as necessary before merging.

For the Galatic ticket, are you thinking of including a link from this document to that meta ticket or something else?

It would be good to comment on the meta ticket with the items OR commits to for Galactic since that is the place where all the TSC members (and others) post their commitment. So users have a single point to read what is expected to be done for Galactic.

I'll make a new PR with your suggestion about commits.

No need to create a new PR - you can update this one and rewrite the history as necessary before merging.

For the Galatic ticket, are you thinking of including a link from this document to that meta ticket or something else?

It would be good to comment on the meta ticket with the items OR commits to for Galactic since that is the place where all the TSC members (and others) post their commitment. So users have a single point to read what is expected to be done for Galactic.

Apologies. I did the usual and messed up my git commits. I'll fix it all shortly.

I'm tapping out after a few struggles with git. #829 replaces this PR.


What is the point? Why not just "Add to Desktop" ?

I tried it out and.. well.. why not just do a borderless window by adding it to the desktop via chrome? Seems like a lot of extra work for the same end result. Or am I missing something :o

Well, for one, I prefer not to rely on Chrome for my apps, when possible. Also, many people use Firefox and AFAIK it's not possible there.
Another reason is that, eventually, this application can be improved beyond the web apps' features, if the author so desires, just like the Unofficial Facebook Messenger desktop client, which started as a wrapper too.
What I would like to see from this project is full integration with my DE: notifications, tray icon, maybe a chat menu integration for Ubuntu Unity users... things like that, which you can't do by using the Chrome app.

Thanks for the answer :)
I didn't think of things like tray icons when I posted it.
Cheers!


Cordova google maps tracker

Hello,
Can we use this plugin to track position  and drow route from a point A to point B?
Thanks

This plugin doesn't provide direction feature at this time.

And real live tracker ?

No plan to implement the feature at this time.

thanks


specify minimum dependency version

Also h5py is not required for install but for tests

This is mostly complete, waiting for pyuvdata version to increment to 1.3.8. This version will include an i/o bug fix for reading PAPER data.


why the answer is so short ??  is there any problem??

Describe the bug
why the answer is so short ??  is there any problem??
To Reproduce
$open_ai = new OpenAi($open_ai_key);
    $complete = $open_ai->completion([
        'model' => 'text-davinci-003',
        'prompt' => $prompt,
        'temperature' => 0.9,
        //'max_tokens' => 4096,
        'frequency_penalty' => 0,
        'presence_penalty' => 0.6,
    ]);

$prompt = "我家有只狗叫三万，你猜猜它为什么叫这个名字";
I believe the answer is long, but the answer is :
"{"id":"cmpl-.......","object":"text_completion","created":1676702955,"model":"text-davinci-003","choices":[{"text":"？\n\n这可能是你家","index":0,"logprobs":null,"finish_reason":"length"}],"usage":{"prompt_tokens":44,"completion_tokens":16,"total_tokens":60}}
"
why the answer is so short ??  is there any problem??
Code snippets
No response
OS
centos7.2
PHP version
PHP 7.4
Library version
openai3

No, not an issue please increase your max_tokens variable

No, not an issue please increase your max_tokens variable

thx
I need set max_tokens variable every time ？？ that's not humable.

@wuqiu2 what does it mean "that's not humable."


[Vssue]leetcode discussion

https://ox4f5da2.github.io/leetcode/leetcode.html

Welcome to this website! If there are problems with JS code or you have better suggestions, you can put forward your suggestions in the comments area. 🎉🎉🎉


Update cluster-autoscaler version

I added the usage of a debug logger to the cluster-autoscaler that shows all requests/responses made toward the Hetzner API when specifying a high log level (--v=5 or higher). I needed this because I saw "unknown" errors from the Hetzner api when trying to scale nodes.
I saw that a fork of the autoscaler repository is used here.
How often is the fork updated to the latest version of the base repo?
Sorry if this is the wrong issue format but it didn't really fit the other templates.

@karsten42 We last updated it about a month ago or so, pending merges and release of some changes on the mother repo. @Silvest89 do you know if the needed changes have been released already or not on the mother repo?

@Silvest89 FYI, please your input is needed bro 🙏

@mysticaltech Yes we can revert back to the cluster-autoscaler. They have released them.

@Silvest89 Thanks, appreciate it! Will do the change 🙏


Removing leftoiver merge information in the README.md file.

@jmbmxer

cleaning up my crappy readme! thanks :  )


Saaly/typescript rich binding

This PR introduces a sample for rich binding in TypeScript requested in #42

tagging @anirudhgarg  and @amamounelsayed

LGTM. this will work if the body of the message is a JSON string that you can then deserialize with GetValue. if the body of the message isn't JSON this is going to fail.
Are we worried about that, for this sample?

@ryancrawcour I updated the ReadMe about the json value type. For non json payloads, the parsing will need to be manual anyway.

For non json payloads, the parsing will need to be manual anyway.

Makes sense. Thanks @Sarah-Aly


Strange scoping behaviour in functions

Hello,
There is a set of functions in the custom.ts in this snippet illustrating what I think might be broken variable scoping. Is this a bug or can someone explain what's going on here? Variable scoping behaviour seems to have changed in the last day or so.
https://makecode.microbit.org/_YdiVFx1vu8TV
@whaleygeek
Rgds,
Rae

@mmoskal, can you look at this. Related to https://github.com/Microsoft/pxt/issues/2251 ?

/**
 * Use this file to define custom functions and blocks.
 * Read more at https://makecode.microbit.org/blocks/custom
 */

const val = 0x40;

/**
 * Custom blocks
 */
//% weight=100 color=#0fbc11 icon=""
namespace custom {
    const val2 = 0x40;

    //% block
    export function foo(): number {
        return 0x40;
    }

    //% block
    export function bar(): number {
        return val;
    }

    //% block
    export function bar2(): number {
        return val2;
    }

    //% block
    export function bar3(): number {
        const val3 = 0x40;
        return val3;
    }

}

Inlining the custom code into main makes the thing work:
https://makecode.microbit.org/_LW4PWRCYPXCu

Something odd is going on with respect to code gen in custom.cs
/**
 * Custom blocks
 */
//% weight=100 color=#0fbc11 icon=""
namespace custom {
    let life42 = 42

    //% block
    export function life(): number {
        return life42;
    }

yields
var __main__1 = entryPoint = function (s) {
var r0 = s.r0, step = s.pc;
s.pc = -1;
while (true) {
if (yieldSteps-- < 0 && maybeYield(s, step, r0)) return null;
switch (step) {
  case 0:

    s.lastBrkId = 1;
    r0 = pxsim.Array_.mk(0);
    s.tmp_0 = r0;
    r0 = globals.freqTable___110;
    pxtrt.decr(r0);
    r0 = s.tmp_0;
    globals.freqTable___110 = (r0);
    s.lastBrkId = 2;
    s.tmp_0 = { fn: _life_146, parent: s };
    s.pc = 2;
    return actionCall(s.tmp_0)
  case 2:
    r0 = s.retval;
    s.tmp_1 = r0;
    setupResume(s, 3);
    pxsim.basic.showNumber(s.tmp_1, 150);
    checkResumeConsumed();
    return;
  case 3:
    r0 = s.retval;
    s.lastBrkId = 3;
    r0 = 42;
    globals.life42___147 = (r0);
  case 1:
    return leave(s, r0)
  default: oops()
} } }
__main__1.info = {"start":0,"length":0,"line":0,"column":0,"endLine":0,"endColumn":0,"fileName":"pxt_modules/core/dal.d.ts","functionName":"<main>"}
__main__1.continuations = [ 3 ]

Note above that the global initialization is dead code, coming to late. When the TS above is inlined in main.ts, on the other hand, we get proper code:

var __main__1 = entryPoint = function (s) {
var r0 = s.r0, step = s.pc;
s.pc = -1;
while (true) {
if (yieldSteps-- < 0 && maybeYield(s, step, r0)) return null;
switch (step) {
  case 0:

    s.lastBrkId = 1;
    r0 = pxsim.Array_.mk(0);
    s.tmp_0 = r0;
    r0 = globals.freqTable___110;
    pxtrt.decr(r0);
    r0 = s.tmp_0;
    globals.freqTable___110 = (r0);
    s.lastBrkId = 2;
    r0 = 42;
    globals.life42___145 = (r0);
    s.lastBrkId = 4;
    s.tmp_0 = { fn: _life_146, parent: s };
    s.pc = 2;
    return actionCall(s.tmp_0)
  case 2:
    r0 = s.retval;
    s.tmp_1 = r0;
    setupResume(s, 3);
    pxsim.basic.showNumber(s.tmp_1, 150);
    checkResumeConsumed();
    return;
  case 3:
    r0 = s.retval;
  case 1:
    return leave(s, r0)
  default: oops()
} } }
__main__1.info = {"start":0,"length":0,"line":0,"column":0,"endLine":0,"endColumn":0,"fileName":"pxt_modules/core/dal.d.ts","functionName":"<main>"}
__main__1.continuations = [ 3 ]

Repro - ordering of files seems to be the issue. This doesn't work
basic.showNumber(cc.life())

namespace cc {
    let life42 = 42

    //% block
    export function life(): number {
        return life42;
    }
}

This does work:
namespace cc {
    let life42 = 42

    //% block
    export function life(): number {
        return life42;
    }
}

basic.showNumber(cc.life())

This error is consistent with TS/JS semantics.

Fixed

Thank you very much team.
Rae