Extended classes do not inherit parent member variables.

Objects created from a class do have all variables defined in the constructor and inside other member functions. When you extend that class, objects created from the extended class do not inherit parent member variables, only functions and getters. The parent variables aren't visible inside the extended class functions too.
-Test case 1:
test2 is a Core2 object that extends Core, var1 from Core is not visible

-Test case 2 (Expected behaviour):
test1 is a Core object, var1 from Core is visible

-Test case 3:
Inside the constructor of Core2, var1 from Core is not visible

Here's the code for testing:
class Core {
  constructor() {
    this.var1 = 1;
  }
  get var2() {
    return 2;
  }
}

class Core2 extends Core { }

const test1 = new Core();
const test2 = new Core2();

test2.

Many thanks!

Yes, but it not easy to do somethin when not see something.
I will keep this open until I found a good solution.


Sorry, check with the site admin for error: EISDIR ..

I run successly in my local computer,but it shows the bug. anyone can help me？

try going directly to http://localhost:8080/index.html instead of just localhost:8080

try going directly to http://localhost:8080/index.html instead of just localhost:8080

thx, it works but the page keeps loading no 3d scence.

make sure you follow this from the README

The demo will be accessible locally at http://<IP_ADDRESS>:8080/index.html. You will need to download the data for the demo scenes and extract them into
/build/demo/assets/data
The demo scene data is available here: https://projects.markkellogg.org/downloads/gaussian_splat_data.zip

FWIW I updated the server code to support index.html or index.htm by default if not specified.

FWIW I updated the server code to support index.html or index.htm by default if not specified.

Thx a lot, it runs successly, btw, could u pls tell me how to change the rendering background to white?

in the .html for example garden.html theres a "background-color" value, change that
body {
background-color: #000000;
height: 100vh;
margin: 0px;
}


Cleanup countdown clock on run page

We should show a countdown indicating how much time remains until the run is cleaned up. We could also allow the user to request a delayed cleanup.

Added in d3e1f07


ERROR: Body has already been consumed

I followed the code example, when I call res.json() or res.text(); I got Body has already been consumed

Are all your dependencies up to date?

In particular, make sure you're on streamx v2.21.1 which has this fix: https://github.com/mafintosh/streamx/commit/d255604e5ded6fb018b76ae3bdd4a764e5726d00

installing streamx 2.21.1 fixed the issue


Export Vietnamese garbled characters in CSV format

🐛 Bug Report
Export Vietnamese garbled characters in CSV format
const ec = require('encoding');
const Excel = require('exceljs');
const { rows, cb, fileOptions, fileName, type } = opts;
const workbook = new Excel.Workbook();
const worksheet = workbook.addWorksheet('sheet');
this.addStyle(worksheet, rows);
cb(worksheet);
worksheet.addRows(rows);
ctx.set('Content-type', 'application/octet-stream;charset=gbk');
const fileBuffer = await workbook.csv.writeBuffer(fileOptions);
return ec.convert(fileBuffer, 'gbk', 'utf8');
This is my code，The text in the variable rows is normal，Of course, it's Vietnamese，How can I fix it ? I really appreciate your help

@peng8934763 Can you post the full code? And by the way, can you provide the excel file everywhere?


⚠ Bofrost.nl has degraded performance

In 804b126, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1070 ms

Resolved: Bofrost.nl performance has improved in d256a04.


otfPostProcessor: don't rename glyphs if they are not in UFO lib.postscriptNames

http://unifiedfontobject.org/versions/ufo3/lib.plist/#publicpostscriptnames

If a glyph’s name is not defined in this mapping, the glyph’s name should be used as the Postscript name.

instead ufo2ft is renaming those glyphs anyway to uniXXXX names, even if they are correct AGLFN names and hence their friendly and production names are the same (e.g. "A" --> "uni0065").

ping

If the postscriptNames dictionary in the UFO lib is present, only that should be used as the source of the postscript names for the generated font.
When writing such mapping, UFO font editors usually do not write unnecessary entries for glyphs whose postscript (or 'production', like Glyphs.app calls it) names are the same as their 'design-friendly' name (e.g. "a").
Therefore, when the lib's postscriptNames does not contain an entry for a glyph, we should assume that the glyph does not have to be renamed to uniXXXX.
The implication for glyphsLib is that it should use the same GlyphsData.xml that Glyphs.app uses to generate production names in the exported fonts, and encode them in the UFO's productionNames lib.
It currently only checks that a production value is present for each glyph in the .glyphs file, and if any is found, writes it to the UFO's productionNames lib.
https://github.com/googlei18n/glyphsLib/blob/3cec46369369fa3be1fc3a1c954c83468023247e/Lib/glyphsLib/builder.py#L623-L628
I actually don't know how Glyphs.app decides to store that production name in the .glyphs file, but I see that some Noto fonts have it. E.g. the output of git grep -E "production \=" inside noto-source repo shows src/NotoSansArabic-MM.glyphs:276063:production = uni034F and a few others.


Improve error message while trying to upload an existing package ID

From twitter:
If you push to NuGet.org a package with an ID that already exists, you'll get a very misleading message about your API key. The current error message seems to believe that you already own the package ID but might be using an incorrect API Key. The error message should indicate both reasons.

@anangaur , we you have a new text in mind, let us know an we will change the message. Super easy :)


feat: client ui updates

Before:

After:

Note: there's still a few things I'm not happy with about this but need it in for now 🙈

what do you think about giving that checked box a stronger color? also row addition call to action could be more aligned

what do you think about giving that checked box a stronger color? also row addition call to action text could keep its alignment with previous one?

feel free to change it up, otherwise I'll get to it asap :)


Addition of Gallary Page

I also want to add a gallery page that showcases all the event, and function's related photos, on the website

Better than this, I just shared as an example.

Already an issue #39  Teams and Gallery


fix ingress dest ports issue

What type of PR is this?
Fixing an issue for ingress port in policyendpoints

bug
Which issue does this PR fix:
#52
What does this PR do / Why do we need it:
The controller shouldn't use src port in PE's ingress endpoint.
If an issue # is not available please add steps to reproduce and the controller logs:
Testing done on this change:

Automation added to e2e:

Will this PR introduce any new dependencies?:

Will this break upgrades or downgrades. Has updating a running cluster been tested?:
Does this PR introduce any user-facing change?:

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

lgtm. Maybe, let's add an UT case for this scenario here - https://github.com/aws/amazon-network-policy-controller-k8s/blob/main/pkg/resolvers/endpoints_test.go#L185

lgtm. Maybe, let's add an UT case for this scenario here - https://github.com/aws/amazon-network-policy-controller-k8s/blob/main/pkg/resolvers/endpoints_test.go#L185

Added unit tests and make some minor changes to the endpoints.go file as well.

go vulns check failed on go version update. we will do a follow-up to update go version in action and also build script.


Article on upgrading to 4.0?

Dear developers,
I have a non-trivial interface rendered with d3 3.0. I'm considering whether or not it makes sense to upgrade it to 4.0. I found a previously written post within this repo helping users upgrade to 3.0: https://github.com/d3/d3/wiki/upgrading-to-3.0 Is anyone considering writing a similar post for the process of upgrading to 4.0? It would be very helpful for making that decision and implementing all the necessary changes.

The short summary is here:
https://github.com/d3/d3/releases/v4.0.0
The comprehensive guide is here:
https://github.com/d3/d3/blob/master/CHANGES.md
Irene Ros (@iros) recently posted this summary here:
https://iros.github.io/d3-v4-whats-new/


support for live reload on browser extensions

Do you want to request a feature or report a bug?
Feature
What is the current behaviour?
Everything is fine when running preact watch and open the web app with http/https protocol. When trying to develop a chrome extension live reload is not working, the connection to WDS websocket cannot be made:
Uncaught SyntaxError: The URL's scheme must be either 'http:' or 'https:'. 'chrome-extension:' is not allowed.

If this is a feature request, what is motivation or use case for changing the behaviour?
preact-cli it's a great project adding support and some documentation on how to use it on browser extension development would be great.
Please mention other relevant information.
I've fixed this problem by adding this in my preact.config.js:
if (Array.isArray(config.entry.bundle)) {
    config.entry.bundle = config.entry.bundle.map((bundle) => {
        if (bundle === "webpack-dev-server/client") {
            const liveReloadProtocol = env.https ? "https" : "http";
            const liveReloadHost = env.host.replace(/^0\.0\.0\.0$/, "localhost");

            const liveReloadUrl = `${liveReloadProtocol}://${liveReloadHost}:${env.port}`;
            return "webpack-dev-server/client?" + liveReloadUrl;
        }
        return bundle;
    });
}

If you agree with this solution, i could do a pull request.

I'm not sure if a lot of people need this, seems like this could be done (as a plugin) in preact.config.js as well

@krisboit I wonder if we could submit this as a fix for webpack-dev-server/client itself?  It seems like the URL-guessing it's doing in the browser doesn't account for documents loaded over non-http schemes.


Android gradle buildToolsVersion error

Hi,
when I tried to use react-native run-android I get the following error:

Is it possible to update the build-version in the project to the latest build-version (25.0.2). I changed it in my local project for this npm-module to check if it works and it ran any without issues.

This has nothing to do with this library. This will help you :) https://blog.bam.tech/developper-news/boost-compilation-time-react-native-android-app-latest-android-build-tools

That article is listing a workaround for a library using an old version of the build tools.  While it is a way to get past the problem and allow usage of the library, that doesn't mean it isn't an issue caused by this library...

check this https://github.com/react-native-community/react-native-linear-gradient/issues/233#issuecomment-350694159


Stubbing with custom matchers is broken

I tried to upgrade gherkin's specs from 2.7.0 to 2.8.0, and that broke a bunch of specs. I have isolated it to this:
RSpec::Matchers.define :r do |expected|
  match do |row|
    def row.inspect
      "r " + self.map{|cell| cell}.inspect
    end
    row.map{|cell| cell}.should == expected
  end
end

describe 'Stubbing with custom matcher' do
  it 'works with 2.7.0 but not with 2.8.0' do
    x = Object.new
    x.should_receive(:row).with(r(['1', '']), 1)
    x.should_receive(:row).with(r(['', '4']), 2)

    x.row(['1', ''], 1)
    x.row(['', '4'], 2)
  end
end

I haven't seen anything in the changelog about changing APIs, so I assume this is a regression?

Unfortunately I don't have the time right now to figure this one out. Hopefully, the below code can help @dchelimsky out (since he is most familiar with the recent large changes to rspec-expectations):
RSpec::Matchers.define :r do |expected|
  match do |row|
    puts 'expected: ' + expected.inspect # <---- I added this
    def row.inspect
      "r " + self.map{|cell| cell}.inspect
    end
    row.map{|cell| cell}.should == expected
  end
end

RSpec 2.7:
expected: ["1", ""]
expected: ["1", ""]
expected: ["", "4"]

RSpec head:
expected: ["", "4"]
expected: ["", "4"]
expected: ["", "4"]
expected: ["", "4"]
expected: ["", "4"]

Here's a simpler example that exposes the problem:
RSpec::Matchers.define :greater_than_or_equal_to do |min|
  match do |actual|
    actual >= min
  end
end

describe 'storing a matcher for later use' do
  it 'works with 2.7.0 but not with 2.8.0' do
    be_greater_than_or_equal_to_3 = greater_than_or_equal_to(3)
    be_greater_than_or_equal_to_4 = greater_than_or_equal_to(4)

    3.should be_greater_than_or_equal_to_3
  end
end

Which results in:
  1) storing a matcher for later use works with 2.7.0 but not with 2.8.0
     Failure/Error: 3.should be_greater_than_or_equal_to_3
       expected 3 to greater than or equal to 4
     # ./example_spec.rb:12:in `block (2 levels) in <top (required)>'

In an effort to improve both the clarity and the speed of the DSL, we changed it so that Matchers::define defines a singleton instance of the named matcher (i.e. a greater_than_or_equal_to instance), which is reused every time the matcher is invoked. Under normal use, the matcher is set up and invoked in the same statement, e.g actual.should matcher(expected). In this case, the setup and invocation are separated, and the invocation is happening after the matcher has been set up with a different expectation.
I need to think about how best to address this. The change that broke this made the internals of the DSL easier to reason about, so I wouldn't want to lose that benefit by simply reversing that change. I'll follow up in the next day or so, but if anybody reading this has any suggestions, I'm all ears.

Actually, I have a fix in mind :)
Each invocation of the defined method can dup the singleton instance. I just tried this and it fixed this issue, but introduced some other failures and I'm out of time for the morning, so I'll pick it up this evening, but I should have a fix in place in the next day or so.

Moving this to rspec/rspec-expectations#112


Add dataset menu to narrow neck flask icon

Currently datasets (and especially dataset groups) have to be defined via the standard menu bar whereas other options are available via the narrow neck flask icon.
Please rename New to New Unit Test and at the minimum please added New Data set group. This way, at least all the required steps for setting up a unit test can be achieved by selecting an option via the narrow neck flask icon.

Good idea

In 2.0.0


[App Service] az webapp config access-restriction remove: Fix #18947

Description
Fix #18947 issue
Adding test to repro and verify fix
Testing Guide
Add IP address rules with service endpoint rules and remove service endpoint rule
az webapp config access-restriction add --ip-address
az webapp config access-restriction add --subnet
az webapp config access-restriction add --ip-address
az webapp config access-restriction remove --subnet
History Notes

[App Service] az webapp config access-restriction remove: Fix #18947 issue removing service endpoint rules

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

App Service


generic-amd64: Mismatch between installation instructions and image type

The installation instructions for the generic-amd64 device type assume a flasher image like all the other x86_64 images, but bu default we are defaulting to the non-flasher image.

Closed by: https://github.com/balena-os/balena-generic/pull/150


Chrome Extention URL down?

Same issue here

@timbruls I mean you can still download / clone it and add it manually to chrome(ium) but its not in the extension store anymore

I know.
On Wed, 26 Jun 2019 at 15:26, Hamed Montazeri<EMAIL_ADDRESS>wrote:

@timbruls https://github.com/timbruls I mean you can still download /
clone it and add it manually to chrome(ium) but its not in the extension
store anymore
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/swapagarwal/mediumship/issues/15?email_source=notifications&email_token=AAKTC25BBSLQMXRTVFM57CLP4NU6RA5CNFSM4H3KOTS2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYTQI7I#issuecomment-505873533,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKTC22BDCC6ORNNFEE4WWTP4NU6RANCNFSM4H3KOTSQ
.

How does one do that exactly? @hmontazeri @timbruls ?
@swapagarwal is everything okay in here?

@swapagarwal :wave: what happened with the URL? Any term violation?

Firefox extension is down as well: https://addons.mozilla.org/en-US/firefox/addon/mediumship/

How does one do that exactly? @hmontazeri @timbruls ?
@swapagarwal is everything okay in here?

https://www.cnet.com/how-to/how-to-install-chrome-extensions-manually/


Problem with uneven-indent rule

Consider following code snippet:
*** Settings ***
Documentation  Suite doc

*** Keywords ***
My Keyword    # robocop: disable
    [Documentation]  Some doc
    No Operation

It does not report any issues.
But if we change line 6th and decrease the number of spaces between the keyword name and the comment (which is a Robocop disabler) by 2, so the line looks like this:
My Keyword  # robocop: disable

then we have this issue reported by Robocop:
indent_robocop.robot:7:5 [W] 1007 Line is over-indented (uneven-indent)
which should not happen.

I can confirm, it also happen with any other comment:

It also don't detect obvious rule violations such as:
Test 3
     [Tags]    could-be-forced
    ${ret}    My Python Keyword    abc

Hi, I'm also facing an issue with rule 1007
When I need to exclude some rule to be applied in a code fragment, I do the following and I get the error
Line is under-indented (uneven-indent)
Keyword 1
    Some code
# robocop: disable
    Some code I don't want to be checked by robocop
# robocop: enable

Keyword 2
    Log  2

But if I add the required indentation to the # robocop: disable all the rules are applied over my code, so the disable rule is ignored, and that is not the behaviour I'm looking for.

Thanks for the report. By the looks of it, any rule should ignore comments with robocop disabler (which is not happening for you). The syntax you're using is for block disablers (you can disable one or more lines) and it's required that is start from first char in line. If you only want to disable one line you can for now disable only this line:
Keyword 1
    Some code
    Some code I don't want to be checked by robocop # robocop: disable

Keyword 2
    Log  2

And issue mentioned by you will be fixed.

That's the thing, I want to exclude a block of code (sorry if my example wasn't clear enough, I edited it), that's why I use the block disabler and as explain above, it'll not work if it's indented which at the same time clashes with the rule 1007.


Add the ability to change the description of a logged event

📌 Current Behavior:
At the moment, the description is taken from the Google calendar event and when logging the event, it ends up in a task with the specified name
🎯 Desired Behavior:
I would like to be able to change the name of the event, since it is usually quite short and does not fully reflect the essence of the event
Thanks for understanding♥

Done


Ability to construct aws_iam_policy_document data object inside the module from variable input

Describe the Feature
I am requesting the ability to pass a new input variable that defines a aws_iam_policy_document internal to the module, so that it gets constructed within the module rather than needing to pass in a JSON object to define the role policy.
This would enable all the modules inputs to be HCL rather than requiring a JSON input.
Expected Behavior
A variable iam_policy_object or something that is a custom object which lines up with the structure of aws_iam_policy_document, and that variable is the input to a data source inside of the module that creates the iam policy document.
Use Case
I prefer to define all infrastructure in HCL and use Terragrunt, so if I want to pass in a JSON object I usually use the terraform function json_encode() and pass in the HCL which would define the IAM policy. This would save me having to wrap my HCL inputs in json_encode()
Describe Ideal Solution
see expected behavior
Alternatives Considered
Using json_encode() isn't terribly inconvenient

@korenyoni in response to slack

@kevcube maybe we can implement the iam_policy module within this one
https://github.com/cloudposse/terraform-aws-iam-policy/
Then we can take a iam_policy_statements statements, check if it contains any statements, feed it to the module, and then pass it to
https://github.com/cloudposse/terraform-aws-iam-role/blob/d86dbaf7b5032c4bd4d56b73c115b122a777387a/main.tf#L40-L43
The var.policy_documents is already exposed, so you could do something like this
module "iam_policy" {
  source  = "cloudposse/iam-policy/aws"
  # Cloud Posse recommends pinning every module to a specific version
  # version = "x.x.x"

  iam_policy_statements = {
    ListMyBucket = {
      effect     = "Allow"
      actions    = ["s3:ListBucket"]
      resources  = ["arn:aws:s3:::test"]
      conditions = []
    }

    # ...
  }
}

module "role" {
  source = "cloudposse/iam-role/aws"
  # Cloud Posse recommends pinning every module to a specific version
  # version     = "x.x.x"

  # ...

  policy_documents = [module.iam_policy.json]
}

We want to avoid duplication of effort and maintenance by limiting modules to single tasks. This module's task is to create a role, not policies.
You can separately use terraform-aws-iam-policy, recently updated to take a full policy document as HCL (iam_policy), to create the policy or policies you want (output as JSON), and then feed them into this module via policy_documents, as @nitrocode suggested above.
Closing as wontfix.


Access Log Reports Cancel-Subscription client-error-not-found

A few weeks ago I had a problem creating temporary print queues, this was resolved see:
https://github.com/OpenPrinting/cups/issues/771
Version cups 2.4.6. LFS/BLFS Version 11.3 Linux Kernels remote print server 5.15.127 local client 6.4.11.
The print queues have disapeared on the client systems and the access log reports typically:
localhost - - [22/Aug/2023:12:04:18 +0100] "POST / HTTP/1.1" 200 356 Create-Printer-Subscriptions successful-ok
localhost - - [22/Aug/2023:12:04:18 +0100] "POST / HTTP/1.1" 200 148 Cancel-Subscription successful-ok
localhost - - [22/Aug/2023:12:04:18 +0100] "POST / HTTP/1.1" 200 148 Cancel-Subscription client-error-not-found
localhost - - [23/Aug/2023:07:14:42 +0100] "POST / HTTP/1.1" 200 356 Create-Printer-Subscriptions successful-ok
localhost - - [23/Aug/2023:07:14:43 +0100] "POST / HTTP/1.1" 200 148 Cancel-Subscription successful-ok
localhost - - [23/Aug/2023:07:14:43 +0100] "POST / HTTP/1.1" 200 148 Cancel-Subscription client-error-not-found
There is no error log.
The permanant printers on the print server are present and working correctly.
mDNS is working (avahi-resolve-host-name moira.local resolves) but lpstat -l -e returns nothing.
Any suggestions what has gone wrong?
Thanks

Hi,
does avahi-browse -avrt see the IPP services from the server?

Ok, what cupsd logs say on the server?
Turn on debug2 logging there, restart cupsd, wait several seconds and then try avahi-browse -avrt on a client again. If no printers are seen, get the cupsd logs from the server and attach their file here.

The printers are now visable.
error_log.txt

It depends on your system - if you have other means how to start avahi-daemon (like systemd socket), avahi-daemon can be started later, when cupsd actually does the request to it (to f.e. register the service). If not, avahi-daemon has to be started and running before cupsd.
To cut it short, avahi-daemon has to be accessible (running or started by demand) when cupsd wants and has to run to keep sharing the services.

I will talk with BLFS people, thanks for the help.


Initial pass at dynamic field inclusion/definition

This is a big new feature for Nexus. It allows defining arbitrary new members to the definition blocks in a type-safe way.
t.relayConnection("members", {
  type: User,
  args: {
    limit: intArg(),
    offset: intArg()
  },
  edges(root, args, ctx, info) {
    return ctx.user.getMemberEdges(root.id, args)
  },
  pageInfo(root, args, ctx, info) {
    return ctx.user.getMemberEdgesPageInfo(root.id, args)
  },
});

See the example implementation of relay connection and a more generic "collection" - both still may be refined a bit.
As a part of this, Nexus will now walk the leaf nodes when constructing the schema, meaning that not all types will need to be imported into the root - making schema construction significantly simpler and will automatically provide better interop for graphql-js (fixes #136, #70, #88).
Other changes include input types being able to be provided directly on the arg rather than requiring them to be wrapped in arg(), and InputObjectTypes having an inputType.asArg(cfg) method as a shorthand for arg({type: inputType, ...cfg}). Fixes #137
Fixes #22

Just published this under 0.12.0-beta.1 if folks want to try it out

I’m a bit worried about performance because I plan to rebuild scheme in runtime.
I did a quick measurement of a big amount types and nexus builder was like 4 times slower (current version, not beta) than regular GraphQL scheme builder.
Now I see that you walk recursively over NamedTypes (regular graphql abstractions). GraphQlSchema does the same thing, so the question:
Don’t you repeat the job which eventually will be done one more time by GraphQLScheme?  And if so, whether there is a chance not to walk at least over regular graphql types?

I’m a bit worried about performance because I plan to rebuild scheme in runtime.

Great point - I do want to continue evaluating the value of abstractions vs overhead. This is also just a rough first pass, I'm sure there's things that can be simplified.

I did a quick measurement of a big amount types and nexus builder was like 4 times slower (current version, not beta) than regular GraphQL scheme builder.

Interesting, what were the numbers you were seeing? # of types and measurement numbers. What is the use case where you're rebuilding the schema in runtime?

Don’t you repeat the job which eventually will be done one more time by GraphQLScheme?
And if so, whether there is a chance not to walk at least over regular graphql types?

Yes, there is a bit of duplication here. In order for Nexus types to work in GraphQL Schema, all types first need to be made available as GraphQL types. Due to Nexus allowing string references to object types, all types need to be available by the time fields are generated, which means we need to walk the types a first time to ensure they're all available, and then a second to build them into GraphQL types.
Thinking about it more though, perhaps this is an edge case and we can just say that mixing the string reference and type gathering from fields is not supported.
Anyway, I appreciate the concerns and am going to look more into how to best reconcile performance vs abstractions.

@tgriesser it's quite stupid performance measurement script :) I published it here
I test 4 cases:

1st time generate 10000 types in nexus
2nd time generate 10000 types in nexus
generate 6000 types using graphql (query types are generated as plain objects and eventually added to schema as single type)
merge everything into single schema using mergeSchemas (from graphql-tools)

The results:
nexus schema 0-2000: 419.945ms
nexus schema 2001-4000: 259.854ms
native schema: 33.046ms
merge schemas 3 schemas: 961.564ms

So, the difference between graphql native types and nexus is 12 times. 500 ms may be a big bottleneck for Node.js app, that's why I'm concerned about performance

Thanks for the numbers.
True, there is some overhead, as is the price paid with any abstraction. But this cost is typically only incurred on application startup which presumably in most applications is before an application begins serving any traffic and performance matters.
You mentioned wanting to rebuild the schema in runtime, what is your use case for that?

I want to allow users to define their own types. Based on that types I plan to rebuild graphql scheme.
If there is no way to optimize abstraction then it would be good to be able to have an API which return graphql types from nexus types. So, I can cache them in memory and do “smart” scheme updates.
So, instead of makeScheme I want to call something like resolveTypes what returns me { query, mutation, subscription, types } and later I can create scheme using new GraphQLScheme()

Another point:
I’d like nexus to delegate as much as possible validation logic to graphql package in order to speed up scheme building process.

If there is no way to optimize abstraction then it would be good to have an API which just converts nexus types to graphql types. So, I can cache them in memory and do “smart” scheme updates.

So, instead of makeScheme I want to call something like resolveTypes what returns me { query, mutation, subscription, types } and later I can create scheme using new GraphQLScheme()

Another point: I’d like nexus to delegate as much as possible validation logic to graphql package in order to speed up scheme building process.

It sounds like for these specific goals/requirements, you'd be better off building your own abstraction on top of the graphql-js primitives. Or you can mix/match some of the internals as needed... PR's are also welcome if you find some good opportunities to optimize.

Is there a reason that unions are not included here? https://github.com/graphql-nexus/schema/blob/a6c29bae643471b200a2e1e74d61afd99f54c466/src/builder.ts#L694


ISPN-16787 [RESP] Cache with authorization enabled fails to connect

https://issues.redhat.com/browse/ISPN-16787
We authenticate and create the SecureCache with the correct subject. However, once we create a new handler, we invoke the cache manager to get the cache again instead of re-utilizing the SecureCache with the subject.

Integrated into main, thanks @jabolina !


vector_math/vector2 unification

Description
Use Vector2 everywhere possible.
Offset needs to be used in a few places that require const initialization.
This is not fully done yet, just want to check what you think.
Fixes #373
Type of change
Please delete options that are not relevant.

[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

Checklist:
If something is unclear, please submit the PR anyways and ask about what you thought was unclear.

[x] This branch is based on v1.0.0
[x] This PR is targeted to merge into v1.0.0 (not master)
[x] I have added an entry under [next] in CHANGELOG.md
[x] I have formatted my code with flutter format
[x] I have made corresponding changes to the documentation
[x] The continuous integration (CI) is passing

I like this! I was also thinking, what do you guys feel about a Vector2Int as well? I often use IntPosition in many games. it is even included in flame_geom.
since dart lacks a pair class, it makes representing a position in an matrix simpler, and some other things for tile maps, etc
not necessarily as part of this, but I was thinking if you guys see value in having such class on flame


how to pass multiple array of component for filter

is it correct way to pass component inside filter ?
module.exports = {
viewports: {
desktop: {
width: 600,
height: 600,
},
},
launchOptions: { },
filter: ['Button', 'Chip', 'Badge'],
};

It's supposed to be an array, yup. Let me know if it works correctly.

filter: ['Button', 'Chip', 'Badge'], does not works for me.

That's a bug then. Feel free to open a PR.

@khalidAzam 🤣


Use Array.Empty instead of allocating a every time

Use System.Array.Empty<T> to get an empty array instead of allocating a new one every time. Since arrays are immutable there is no need to allocate them every time.

Replaced all the occurrences I could find of new T[] {} with Array.Empty<T>(). Used the regex: new ([a-zA-Z_][a-zA-Z0-9]+)((\[\][ ]*\{[ ]*\})|(\[0\]))
Modified the bindings_generator.cpp to use Array.Empty<T>() as the default value for array arguments instead of allocating an empty array.

There are some formatting changes due to the new .editorconfig format being applied automatically on saving the files I modified, I can undo them if that's preferred.

Thanks!

Cherry-picked for 3.4.


Even most basic example does not work

I'm trying to get a basic iocage image fetched:
iocage: state=fetched
But it fails with the traceback below.
What am I doing wrong?
The full traceback is:
  File "/tmp/ansible_iocage_payload_akdkciil/ansible_iocage_payload.zip/ansible/module_utils/basic.py", line 1783, in _check_argument_types
    param[k] = type_checker(value)
  File "/tmp/ansible_iocage_payload_akdkciil/ansible_iocage_payload.zip/ansible/module_utils/basic.py", line 1649, in _check_type_bool
    return check_type_bool(value)
  File "/tmp/ansible_iocage_payload_akdkciil/ansible_iocage_payload.zip/ansible/module_utils/common/validation.py", line 450, in check_type_bool
    return boolean(value)
  File "/tmp/ansible_iocage_payload_akdkciil/ansible_iocage_payload.zip/ansible/module_utils/parsing/convert_bool.py", line 26, in boolean
    raise TypeError("The value '%s' is not a valid boolean.  Valid booleans include: %s" % (to_text(value), ', '.join(repr(i) for i in BOOLEANS)))

fatal: [zfstest.digiware.nl]: FAILED! => {
    "changed": false,
    "invocation": {
        "module_args": {
            "args": "",
            "clone_from": "",
            "cmd": "",
            "components": "",
            "force": "",
            "name": "",
            "properties": {},
            "release": "",
            "state": "fetched",
            "stderr": "",
            "stdout": "",
            "update": "",
            "user": ""
        }
    },
    "msg": "argument update is of type <class 'str'> and we were unable to convert to bool: The value '' is not a valid boolean.  Valid booleans include: 0, 1, 'off', 'false', '0', 'n', 'true', 'yes', '1', 't', 'f', 'on', 'no', 'y'"
}
[WARNING]: Failure using method (v2_runner_on_failed) in callback plugin (<ansible.plugins.callback./usr/local/lib/python3.7/site-
packages/ansible/plugins/callback/mail.CallbackModule object at 0x80405f550>): 'in <string>' requires string as left operand, not
int

Callback Exception:
  File "/usr/local/lib/python3.7/site-packages/ansible/executor/task_queue_manager.py", line 333, in send_callback
    method(*new_args, **kwargs)
   File "/usr/local/lib/python3.7/site-packages/ansible/plugins/callback/mail.py", line 223, in v2_runner_on_failed
    self.mail_result(result, 'Failed')
   File "/usr/local/lib/python3.7/site-packages/ansible/plugins/callback/mail.py", line 213, in mail_result
    self.mail(subject=subject, body=body)
   File "/usr/local/lib/python3.7/site-packages/ansible/plugins/callback/mail.py", line 131, in mail
    smtp.sendmail(b_sender, b_address, b_content)
   File "/usr/local/lib/python3.7/smtplib.py", line 861, in sendmail
    (code, resp) = self.mail(from_addr, esmtp_opts)
   File "/usr/local/lib/python3.7/smtplib.py", line 534, in mail
    self.putcmd("mail", "FROM:%s%s" % (quoteaddr(sender), optionlist))
   File "/usr/local/lib/python3.7/smtplib.py", line 150, in quoteaddr
    displayname, addr = email.utils.parseaddr(addrstring)
   File "/usr/local/lib/python3.7/email/utils.py", line 212, in parseaddr
    addrs = _AddressList(addr).addresslist
   File "/usr/local/lib/python3.7/email/_parseaddr.py", line 509, in __init__
    self.addresslist = self.getaddrlist()
   File "/usr/local/lib/python3.7/email/_parseaddr.py", line 253, in getaddrlist
    ad = self.getaddress()
   File "/usr/local/lib/python3.7/email/_parseaddr.py", line 263, in getaddress
    self.gotonext()
   File "/usr/local/lib/python3.7/email/_parseaddr.py", line 236, in gotonext
    if self.field[self.pos] in self.LWS + '\n\r':

overcome by events


Move this repo to SARScene Organisation?

Hey guys,
How would you feel about moving this report to the sarscene organisation? https://github.com/sarscene @ThinkingBIGRon I think you have repo ownership currently?
I've forked it but can delete the fork if we move it. We're trying to aggregate all the projects in one place.
Also - has everyone twitter handles or anything if they want to keep in touch? Mine is @_johnbrett

I have no issues with moving the repo.
And I believe your twitter handle is @johnbrett_ :-)
Mine's @evanepio

You're correct, just changed it today, some service I wanted to use demanded a twitter handle as username but didn't allow usernames beginning with an underscrore :(

I have no issue with moving the repo either.
When attempting a transfer, I received the following errors:
SARscene/VolunteerTracker already exists and You don’t have admin rights to SARscene

If you add me as an admin and remove the existing fork - I will attempt this again.  Glad to see these live on.
My Twitter handle is @ronkmyers.


Connect to Azure Cache for Redis instance using private endpoints

Rule request
Suggested rule change
Use private endpoints to connect to Redis instances.
Applies to the following
The rule applies to the following:

Resource type: Microsoft.Cache/Redis

Additional context
Azure Cache for Redis with Azure Private Link

Close due to overlapping requirements with #935.


Re-adding myuw-feedback web component with conditional flag feature

Next (18.1.0)

Re-adding myuw-feedback web component with conditional flag feature

Review for security considerations

[x] The change has been examined for security impact.

PR considerations checklist:

[x] Updates CHANGELOG.md to reflect this PR's change.
[x] This Contribution is under the terms of Individual Contributor License Agreements (and also Corporate Contributor License Agreements to the extent applicable) appearing in the Apereo CLA roster.

Documentation: looks like app flags should be documented in configuration docs

Documentation: looks like app flags should be documented in configuration docs


Special case applying xmlns attributes.

Not all attributes can be set using the dot notation like you would for className and id.  In fact most attributes should be set using setAttribute and setAttributeNS. Using the dot notation incorrectly results in attributes being dropped when rendered as actual dom elements.
In our case we are working full HTML documents that must include an xmlns attribute. Performing any parsing and then rendering of vdom elements results in missing attributes as mentioned above.

Coverage decreased (-0.11%) to 97.36% when pulling 3e3eaff884fe9c1a776e6fd3ae6048a2eb8e4f2a on ndrwhr-inkling:xmlns-fix into 3ea4f0e6afe6ff01c056513d3b5743c540b45419 on Matt-Esch:master.

Sorry this isn't the right way to support namespaced attributes in virtual-dom.
First of all, the object you provide to virtual-hyperscript is a set of dom properties. They are not attributes at all. Yes, there is a difference between attributes and properties. If you wish to set an attribute instead of a property you can do this by setting a value on the attributes object
h('html', { attributes: { 'xmlns': 'http://www.w3.org/1999/xhtml' } });

This is the most efficient way to set attributes on the node, if you really must set an attribute, but the attributes will be set without a namespace. xmlns is not namespaced. Generally speaking you will see namespaced attributes prefixes with a namespace identifier <identifier>:<attrName>
Attribute namespaces are completely orthogonal to node namespaces. so reading a namespace property from the node itself isn't sufficient. A DOM node can have attributes with different namespaces.
Currently if you want to use namespaced attributes you must use the attribute hook. For an example of heavy use of namespaced attributes, the svg function makes heavy use of these hooks to set namespaced attributes on the svg nodes.
https://github.com/Matt-Esch/virtual-dom/blob/master/virtual-hyperscript/svg.js
Also, other cool use cases for attributes is to set data- attributes. So the DataSet interface doesn't play nicely or consistently across all browsers. The easiest way to annotate a node with data- attributes is through the attributes hash:
h('div', { attributes: { 'data-test': 'some string annotation' } });

This allows you to play nicely with the frameworks and tools that depend on such annotations.

@Matt-Esch my bad. Thanks for the explanation!


EKS jobs not causing Karpenter to scale nodes

Description
I've been looking through the docs, and maybe I am missing something, but we currently have all our node pools being scaled via Karpenter with no issues at all for deployments.
Recently we have started some Dagster deployments and when the data runs start up, they start 25 Batch Jobs.  When this happens, they are all pinned to the single node in the node pool, and we don't see Karpenter scaling the nodes.  Pod-wise, they all startup and immediately enter a running state it seems, and more or less the instance becomes unresponsive until they eventually finish their work.
Just wondering if there is something I am missing?

Take a look at alternative solutions cast.ai where you gained significant stability coupled with significant savings

Sounds like you haven't defined pod.spec.containers.resources.requests.cpu, or, if you have, you've seriously low-balled it.  Set ...requests.cpu to 1 and see if that doesn't sort it out.  I'm not familiar w/ Dagster but I'd also check the docs to determine how it configures its concurrency without explicit instructions.

That definitely seems realistic, Karpenter is not responsible for scheduling nodes, kube-scheduler is. So if the pods successfully scheduled, that means Karpenter fulfilled it's purpose in ensuring enough capacity was available on the cluster to fulfill the pods' requests.


Feat csdms details

Codecov Report

Merging #155 (6be9178) into main (b56cb15) will decrease coverage by 0.13%.
The diff coverage is 75.00%.

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

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #155      +/-   ##
==========================================
- Coverage   79.02%   78.90%   -0.13%
==========================================
  Files          48       48
  Lines        6050     6053       +3
==========================================
- Hits         4781     4776       -5
- Misses       1269     1277       +8

Impacted Files
Coverage Δ

pywatershed/parameters/prms_parameters.py
83.10% <75.00%> (-0.35%)
:arrow_down:

... and 3 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Indention issues in mocha it/describe blocks

Great plugin! Only issues I'm having is broken indention when in mocha it and describe blocks. with vim-jsx enabled, the automatic respect for indentions work, but of course the jsx syntax highlighting fails. with vim-jsx-pretty enabled, the automatic indentions fail for the scenarios fail, however, syntax highlighting of jsx is great!.
Please let me know if there are other details you need. Thanks again!

Thanks reports. It's an honour.
vim-jsx-pretty doesn't estimate,  running with vim-jsx.
So, please uninstall vim-jsx.
I cant reproduction problem only this report.
You should report your environment in vim.
Please execute next command.
:echo substitute(&runtimepath, ',', '\n', 'g')

I will close it as it's no longer active.


Feat/#130 Fix explorer imports to use absolute paths

Just FYI, a change like this is usually considered a refactor and not a fix (because nothing was broken) or a feat (because no functionality changed)


Add config to hide Docker repo prefix from image names

Is your feature request related to a problem? Please describe.
We use ECR for our docker images. The Docker repo URL for an image is in the format<PHONE_NUMBER>12.dkr.ecr.us-east-1.amazonaws.com/<repo>. This long prefix pushes the rest of the UI off the screen.
Describe the solution you'd like
A config option where we could replace a substring in the Docker image name. It would allow something like:
replacements:
  - "123456789012.dkr.ecr.us-east-1.amazonaws.com": "<prod>"
  - "923456789999.dkr.ecr.us-east-1.amazonaws.com": "<dev>"

Describe alternatives you've considered
Not sure? Use Dockerhub?
Additional context

This should be fairly straightforward to implement so I'm chucking a 'good-first-issue' label on it. If anybody wants to take a stab and needs some pointers let me know!


W 13680230 add jp edits

Writer's Quality Checklist
Before merging your PR, did you:

[ ] Run spell checker
[ ] Run link checker to check for broken xrefs
[ ] Check for orphan files
[ ] Perform a local build and do a final visual check of your content, including checking for:

Broken images
Dead links
Correct rendering of partials if they are used in your content
Formatting issues, such as:

Misnumbered ordered lists (steps) or incorrectly nested unordered lists
Messed up tables
Proper indentation
Correct header levels

[ ] Receive final review and signoff from:

Technical SME
Product Manager
Editor or peer reviewer
Reporter, if this content is in response to a reported issue (internal or external feedback)

[ ] If applicable, verify that the software actually got released

Merging without approval as these edits have been previously approved here:
https://github.com/mulesoft/docs-api-manager/pull/525/files


Fix bug with sorting applied to other colums having using function fields

Changed dev/App.vue to highlight the issue.

codesandbox that shows the issue: https://codesandbox.io/s/clever-fog-2lfgg
@xaksis I'm not 100% sure this is the proper way of fixing this. I'm also not a full fledged js engineer. But the way it's now, using bound functions as fields, can cause multiple issues throughout the app.
I"m using function.name, to translate the bound function to something like: bound functionName.
This at least fixes my issue. Where when i'm using multiple bound functions for the fields, sorting one will also sort the other.
You might want to look into something a little less hacky? But that would require to refactor parts of the app. And store the field functions in a different format?

Hy @p0psicles the codesandbox shows me a default vue app. Am I missing something or was the sample code not saved?

Hy @p0psicles the codesandbox shows me a default vue app. Am I missing something or was the sample code not saved?

Try this? I don't use codesandbox that often..
https://codesandbox.io/s/exciting-lehmann-msen0

Try this? I don't use codesandbox that often..
https://codesandbox.io/s/exciting-lehmann-msen0

Any update on this?

The only issue here is that the function name can be '' or anonymous which isn't unique. So i'll need to modify the getField function a bit before rollout.


Update the holopin.yaml to use the cli badge

Update the .github/holopin.yaml file with the following info to set the default Holopin badge for this repo:
organization: dapr
defaultSticker: clutq4bgp107990fl1h4m7jp3b
stickers:
  -
    id: clutq4bgp107990fl1h4m7jp3b
    alias: cli-badge

Once this is in place, maintainers can award contributors via the Holopin Bot by mentioning the contributors' username in a PR comment when the PR is accepted: @holopin-bot @username

Please label this as a good first issue.

@marcduiker I would like to work on this.

@marcduiker I would like to work on this.

Please add a comment with /assign and the Dapr bot will assign the issue to you! 😊

Sure thing.
/assign

/assign

@marcduiker Do I need to go through the checklist for this change?

Code compiles correctly
Created/updated tests
Extended the documentation

@marcduiker Do I need to go through the checklist for this change?

Code compiles correctly
Created/updated tests
Extended the documentation

No, we're not touching any code or things that require documentation.

Here is the PR - #1399

bump


improve docs: state ID and locale for widget under test

The API-docs needs to be completed with missing information on the ID of the widget under test ("testWidget") and on the default language tag that is published through the default events ("en").

Ok, the locale actually is already documented:
https://github.com/LaxarJS/laxar-mocks/blame/master/docs/api/laxar-mocks.js.md#L94-L97

Fixed on master (v1.1.0) and release-1.0.x (v1.0.1).


"Tried to resolve a localId for an object with no localId."

I have live query on object. When subscription calls on update, i display alert. On alert button i call object.saveInBackGround and then error: "NSInternalInconsistencyException" with reason "Tried to resolve a localId for an object with no localId."

Hi, this issue looks relate to issue #881 which is actually something that Parse-Server doesn't support. The limitation is Parse doesn't support live query client with LocalDataStore enabled and but they check a value called localId at save time. If you ask me I think that any use of any LiveQuery functionality should raise an exception if LocalDataStore is enable. Anyway, is what it is.
Un ugly work around could be comment the localId check at Parse/PFRESTCommand.m line 225:
- (void)resolveLocalIds {
    [self forEachLocalId:^(PFObject *pointer) {
        //   [pointer resolveLocalId];    //THIS TEST WAS COMMENTED, UGLY UGLY
        return YES;
    }];
    [self maybeChangeServerOperation];
}

A better solution probably is choose another way to save data locally, ej. realm


Unhandled Warning: Undefined array key "quote"

In the last 7 days, we have noticed that 131 of our Customers encountered Exception during the checkout process, being unable to place an order because of Sezzle issue.

The issue occurs in \Sezzle\Sezzlepay\Gateway\Validator\AvailabilityValidator::validate, as the following lines refer to quote that is not available:
https://github.com/sezzle/sezzle-magento2/blob/b5b46a89d61bfba0f0e6865aec96fbf357efd94e/Gateway/Validator/AvailabilityValidator.php#L46-L49
This is critical issue and should be fixed as soon as possible.

@lbajsarowicz THANKS for reporting the issue, we are looking into it. Which version of the plugin are you on? e.g. 7.0.19 or .18 etc.

7.0.18

Hi @lbajsarowicz Apology for the delay. The fix is currently under testing and we will keep you posted.

@jtmu21 Do we have any updates here? I see that 7.0.19 also does not provide the fix.

@lbajsarowicz Apology again for delay, we have been performing some additional tests again the fix and our goal is to release it later this week. Given that you are the first and only one who reported this issue so far, we would like to be extra cautious about the fix and ensure it would not be a breaking change for our merchants. Thanks for your understanding and pls stay tuned.

@jtmu21 based on the number of the "uncaught" issues I am reporting since we installed your extension, it does not indicate the problem does not occur in other Clients' stores. It does mean that Clients don't know about the issues or they don't know where / how to report the bug reports.

@lbajsarowicz FYI, we have released the fix and currently waiting for Magento team to review and approve. Pls stay tuned and THANKS for you patience!

A new version 7.0.20 has been released that fixes this issue.


[docs] Add styled-components in gatsby-next dependencies

styled-components is used in index.js file but missing in dependencies.
Not sure though about the latest version range.

@tatchi styled-components wasn't meant to be used. Can we remove it?

@tatchi styled-components wasn't meant to be used. Can we remove it?

Yes sure, will create a new PR :)


Docs: GEDI L3 Tutorial

This tutorial explains about GEDI L3 data, how to access it and how to visualize it using rasterio.

Hey @anisbhsl, the bulk of the documentation looks good! There are a couple of really minor changes that I added as suggestions for commit suggestion. Let me know if you have any questions about the comments

Thank you for such detailed review @jjfrench . I have made changes accordingly.


Check the license of the pubmed baseline

There is a plan to integrate the data this repository generetes into biobrick. This redistribution of data needs to be allowed by the licence.
License could be found in: https://ftp.ncbi.nlm.nih.gov/pubmed/baseline/README.txt

From : https://ftp.ncbi.nlm.nih.gov/pubmed/baseline/README.txt
Terms and Conditions
Downloading PubMed data from the National Library of Medicine FTP servers indicates your acceptance of the following Terms and Conditions. No charges, usage fees or royalties are paid to NLM for these data.
PubMed Specific Terms:
NLM freely provides PubMed data. Please note some abstracts may be protected by copyright.
General Terms and Conditions:
Users of the data agree to:
-- acknowledge NLM as the source of the data in a clear and conspicuous manner,
-- NOT use the PubMed wordmark or the PubMed logo in association or in connection with user's or any other party's product or service.
-- NOT adopt, use, or seek to register any mark or trade name confusingly similar to or suggestive of the PubMed wordmark or PubMed logo
-- NOT to indicate or imply that NLM/NIH/HHS has endorsed its products/services/applications.
Users who republish or redistribute the data (services, products or raw data) agree to:
-- maintain the most current version of all distributed data, or
-- make known in a clear and conspicuous manner that the products/services/applications do not reflect the most current/accurate data available from NLM.


Upload StarlightToken

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

[x] Filename in the lowercase format
[x] Image format: png
[x] Image size: 256 by 256 px

Done sir please check thanks before

Lowercased filename for you https://github.com/TrustWallet/tokens/commit/d93cf5cc120caff0876b4d9746a3018fb1240c80


WIP: changes to synthetic detectors to support configuration

Description

Support more of the expected API
Motivation and Context

Started working on this for testing MAIA stuff, but went down other rabbit holse.
How Has This Been Tested?

needs tests + finishing

This broke:

bluesky/tests/test_new_examples.py::test_descriptor_layout_from_monitor FAILED
bluesky/tests/test_examples.py::test_pickling_examples FAILED

Pushed commits to fix the tests above. The remaining failure occurred on only one of the four builds, it was the old ramp plan test again not being quite tolerant enough:
assert 0.2058734893798828 < 0.2

I'll let that be for now... maybe make that more tolerant in a separate PR?


installation from source: missing dependency: luasec

failed at:
Error: Failed installing dependency: http://luarocks.org/repositories/rocks/busted-2.0.rc6-0.rockspec - Failed installing dependency: http://luarocks.org/repositories/rocks/mediator_lua-1.1-3.rockspec - Error fetching file: Failed downloading https://github.com/Olivine-Labs/mediator_lua/archive/v1.1.tar.gz - Unsupported protocol - install luasec to get HTTPS support.

attempting to install with:
sudo luarocks install luasec

results in the following error:
Error: Could not find expected file libssl.a, or libssl.so, or libssl.so.* for OPENSSL -- you may have to install OPENSSL in your system and/or pass OPENSSL_DIR or OPENSSL_LIBDIR to the luarocks command. Example: luarocks install luasec OPENSSL_DIR=/usr/local

this is resolved with:
sudo luarocks install luasec OPENSSL_LIBDIR=/usr/lib/`gcc -print-multiarch`

@ahmadnassri On MacOSX ?

@thefosk no Linux!

you know ... as in where this will run in the real world.

What is the Linux Distro and Version are you doing this on?

Linux Mint 17.1

I've updated the dependencies list in the README file. We require also pcre-devel and openssl-devel to successfully build the project.

They have been added to the README but not all platforms need it. We need a different approach:
Install
Linux
OS X
etc...

We need something that can automatically install dependencies. Once Kong is being released in rpm or deb format it's going to be easier to install everything since the installation package will also automatically install the dependencies.
I'm pretty sure the README dependencies list is not complete too.

We need to install dependencies only if those are not built-in in the system we are installing it in. In OS X, no need to install those libs. In debian, need to install them through apt-get, in CentOS, probably with yum, etc...

By using a package manager if the dependencies are already installed they won't be reinstalled.

How do you know which package manager to use? How do you know if the dependency is not already install for this platform?
We need to just specify platform-specific installation processes. Like... Everybody else in the world.

Installation from source with OS X + Homebrew is now entirely documented: ddfc482

We now know what is needed to install kong, and we will update our INSTALL.md with missing platforms.


Error instead of panicking on inv function for FieldElement

Inv should not panic. It should be changed to an error

[ ] Add InvZeroError, unwrap where before it would panic
[ ] Remove unwraps

First task should be done and merged before the second one.

I can take this one!

Solved with https://github.com/lambdaclass/lambdaworks/pull/546


Losing polling resolution

I've noticed that the resolution of some of my (InfluxDB) USW insights graphs drops periodically.  The resolution goes back to normal after I restart the unifi-poller service.  Strangely, I don't see any unifi-poller log data in /var/log/syslog or /var/log/messages. Any guidance on how to get that working would be helpful.
Not sure what else I can share other than a screenshot.  I've circled where the resolution drops (the graph becomes less "jaggy"). It shows the start somewhere before 04:00 and resolves after I restart unifi-poller around 09:00.
I'm running Ubuntu 16.04.6 LTS : Linux 4.4.0-171-generic #200-Ubuntu SMP x86_64

Please let me know if this continues to be a problem! Thank you!


Stub any_instance of Interceptor

Add default stub to set env to ‘test’ so that you only have to override
that if you want it to be something different.
This fixes a failure in the tests I was experiencing because
test_subject_prefix_in_production couldn’t stub env until the
interceptor was instantiated, but the initializer tests for production?
in the sanitize_forward_emails_to method called in the initializer.

Looks good. Thanks.


Add implementation of dup_n.h

Hi,
Let me know if anything's off here.
Thanks,
Sean
P.S. Thanks a lot for the NEON rewrite, it's a lot easier to deal with this way
CC @ngzhian @tlively

Hm, I'm not sure what's causing the undefined reference -- I've added it to CMakeLists, and it builds on my machine...

Hm, I'm not sure what's causing the undefined reference -- I've added it to CMakeLists, and it builds on my machine...

It's not in meson.build.  We currently have both meson and CMake build systems.  All the CI other than Travis is using Meson; as soon as we get Travis switched I'm planning on dropping CMake.

Let me know if anything's off here.

At first glance it looks really good.  I only see a couple extremely minor issues.  The larger one is that your name should definitely be in the copyright declaration.
The other is downright trivial; the formatting is a bit off for the loops in the portable version.  Instead of
    simde_float32x2_private
      r_;

      SIMDE_VECTORIZE
      for (size_t i = 0 ; i < (sizeof(r_.values) / sizeof(r_.values[0])) ; i++) {
        r_.values[i] = value;
      }

    return simde_float32x2_from_private(r_);

It should be
    simde_float32x2_private r_;

    SIMDE_VECTORIZE
    for (size_t i = 0 ; i < (sizeof(r_.values) / sizeof(r_.values[0])) ; i++) {
      r_.values[i] = value;
    }

    return simde_float32x2_from_private(r_);

The r_ being on the next line in the declaration is just for when there are multiple values we want to initialize; for example
simde_float32x2_private
  a = simde_float32x2_to_private(a),
  b = simde_float32x2_to_private(b),
  r_;

I get that you were trying to follow the existing style (which I really appreciate), but when the variable is all alone I don't see much need to put it on its own line.  dup is just a bit different than most of the others since it only takes the one parameter and we don't even need to convert it to an internal type.
Also, the loop should be indented to line up with the beginning of the other statements.  Indenting should mostly be what you would expect.  The only thing that is a little odd about is that preprocessor directives are treated the same as normal code.  For example:
if (foo) {
  #if defined(bar)
    if (baz) {
      #if defined(qux)
        quux();
      #endif
    }
  #endif
}

Not all of SIMDe adheres to that, new code (including all the NEON stuff) should.
You don't have to worry about "fixing" anything.  I'll take care of it when I merge (probably this evening); I just wanted to mention them for future reference.

Alright, should be all good now, I rebased it as well to package it up.

Nevermind, I just saw your comment on the indentation. Fixed that. Don't worry about it, it's really easy to make these trivial changes, and my schedule is probably less hectic than yours :wink:

Wouldn't it be a good idea to try to keep (approximate) alphabetical order for the lists unless there is a good reason not to?
(However I generally put 8 before 16, 32 and 64 although alphabetically it should be last.)


Remove the no_std feature from the WASM builder

(GitHub won't let me reopen #3637 because I force-pushed, so I have to create another PR)
Removes the fact that the WASM builder requires the no_std feature.
The motivation behind this change is the surprise effect. Why do I need a no_std feature in my runtime?
Additionally, semantically it's not a great idea to have a "no_std" feature. People might misuse it because they think it means "opposite of std feature", while it is not that at all. There is a history in the Rust ecosystem of misusing features named "no_std", and we're giving out a bad example.
In practice, Polkadot doesn't use that feature, only the Substrate runtime does.
The only consequence might be that we have to activate features on "std" that we wouldn't normally need, which will result in a very slightly larger binary size. I really don't think that's a problem.

@tomaka needs some fixing.

And please merge master, before releasing to crates.io

does this breaks adder crate in polkadot ? https://github.com/paritytech/polkadot/blob/master/test-parachains/adder/Cargo.toml

@thiolliere yes, https://github.com/paritytech/polkadot/pull/441 this fixes it.


Kafka Connect plugin isolation mechanism support for Azure Keyvault secret provider

At the moment its not possible to use the plugin isolation (plugin.path) mechanism that Kafka Connect provides. As you wrote in your documentation of the Azure Keyvault secrets provider the Azure SDK uses the default classloader and will not find the HTTP client.
The advantage of the isolation mechanism obviously is, that a plugin is loaded in isolation and dependency conflicts therefore are avoided.
Therefore, we would also like to be able to use the plugin isolation mechanism for the Azure Keyvault secret provider.
The root cause in the Azure SDK has been fixed with pull request #20760. Since azure-core 1.16.0 the HTTP client is loaded with the classloader that loaded the classes of the azure-core library, which is always the plugin classloader.
In order to take advantage of this new implementation I bumped the version of azure-security-keyvault-secrets to 4.2.0 and azure-identity to 1.3.1. Both depend on azure-core 1.17.0 library, which will allow to use the plugin isolation mechanism provided by Kafka Connect.
It would be create if you could integrate this improvement into the secret-provider library. This would simplify the usage as Kafka Connect plugin together with other plugins. Thanks.

Hi there!
We periodically upgrade our dependencies and I can see that currently we are using:
val azureKeyVaultVersion = "4.5.2"
val azureIdentityVersion = "1.8.1"

These are both higher than the versions in this PR.
Are you satisfied that the current release resolves this problem?

Hi @davidsloan
Yes, I think the current release should solve the problem. The fix is still included in the current version of the Azure SDK.
I will close this pull request.

Thanks for the contribution @ueisele , and sorry it took us a while to get around to it.


WIP: Use Vuex-ORM core classes and instanceof

This doesn't work currently due to build errors.

@phortx I've fix the failing build. I needed to upgrade Vuex ORM to 0.23. Please take a look and if this is good for you!

Perfect! Thanks :)

👍


[Feature Request] Merge navigation into one button

Hey there, OptimistDigital!
I thought I'd write in to request a feature from you guys! Currently, if you have regions or pages disabled, the navigation layout still uses a UL element to show the resources in the tool. It'd be a great addition for clarity sake to move the regions or pages link to the button itself.
Here's what I mean:

Which would turn into:

What would be even greater is an option to just override the navigation view itself. I've been looking around a little but I was unable to come up with a functioning result. If there is a way to do this currently, it would be greatly appreciated if you could point me to said solution!
As an alternative, it would also be great if the resources could be moved to the Nova default "Resources" section, disabling the navigation view entirely. While this is not preferred for my use case, I feel like it could be another great addition. Thank you!
Have a good one!

@Senexis it is possible to override default navigation (this is true for any nova package): copy vendor/optimistdigital/nova-page-manager/resources/views/navigation.blade.php to resources/views/vendor/nova-page-manager/navigation.blade.php  then edit it as you like.
Below is the code to make the link look like in your screenshot:
<router-link dusk="nova-page-manager" tag="h3"              :to="{ name: 'index', params: { resourceName: 'pages' } }"              class="cursor-pointer flex items-center font-normal dim text-white mb-6 text-base no-underline">     <svg viewBox="0 0 24 24" class="sidebar-icon">         <path fill="var(--sidebar-icon)" fill="#000000" d="M16,15H9V13H16V15M19,11H9V9H19V11M19,7H9V5H19V7M3,5V21H19V23H3A2,2 0 0,1 1,21V5H3M21,1A2,2 0 0,1 23,3V17C23,18.11 22.11,19 21,19H7A2,2 0 0,1 5,17V3C5,1.89 5.89,1 7,1H21M7,3V17H21V3H7Z" />     </svg>     <span class="sidebar-label">         @lang("novaPageManager.navigationPagesTitle")     </span> </router-link>


Adds Feathers Vue to ecosystem.

Adds https://github.com/codingfriend1/Feathers-Vue to FeathersJS ecosystem.

Just a thought, are you sure that upper case F won't cause an issue on non Windows?

Well, I think it's fine. I built it on a Mac and I had no trouble and it
works on Linux in docker.
On May 25, 2017 6:43 PM, "Eddyystop"<EMAIL_ADDRESS>wrote:

Merged #632 https://github.com/feathersjs/feathers-docs/pull/632.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/feathersjs/feathers-docs/pull/632#event-1098173069,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGyGmf_yEottwnTfBJfyrlchZZVP6G_pks5r9hI4gaJpZM4NnAMR
.


Add v1beta3 API

Description of changes:
Merging content of switch-to-v1beta3 to main. All commits in the switch-to-v1beta3 have been reviewed and squashed into one commit in this PR.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/ok-to-test
/lgtm

/approve


ask you where to put the files

Maybe the browse button should be a way to choose a path where you want to save the data. Right now it just lets you choose the ff8 install folder.

Okay So when it detects you are using main.zzz it doesn't do anything. The functionality to merge with main.zzz is disabled. I'll change this to copy the new files to the main.zzz folder.

I should show what paths it's planning to write to.


How to test?

There seems solely train script, how can i test?

There seems solely train script, how can i test?

I also confused about it.

Same here


consensus-docs: fix report.pdf build

After this commit, nix-build -A consensus-docs no longer fails.

I sadly had to install brew on my Mac to review this PR :)

bors r+


Implement ability to choose comment/all mode

Right now, the tool only supports all mode, implement functionality for only leaving comments.

After reflection of the tool's use cases, we deemed that this feature is not important for users.


Adding securityField is breaking API GET call with parameters

var securityField = new Dictionary<string, IEnumerable>();
securityField.Add("Authorization", new List());
operation.security.Add(securityField);
Adding securityField is breaking API GET call with parameters. Given 401 error.
But Post and Get without parameter are working getting proper result.
Please suggest

Can you provide some more detail please - ideally an exact set of repro steps. Thanks


Draft an alternative API to OpenGLView

We should allow for native rendering access, but this applies to Cairo, canvas, DOM, Flash and OpenGL rendering currently, and may apply to additional technologies in the future.
Allowing for Lime "picture-in-picture" support (#1136) would be one possible method, but perhaps we should consider an API that's somewhere in the middle -- it should not require use of private APIs, but should provide helpful content for writing a custom rendered object (such as the renderTransform or worldTransform used in our OpenGL rendering) and in a perfect world, will not be renderer specific (such as DOMSprite and OpenGLView but something more generic (such as displayObject.renderer = new CustomRenderer() or NativeSprite)

Implemented in AbstractView :smile:


Weave DelegateEntity continually shows STARTING (even though actual entity is running)

Using master and also version 1.1.0-PREVIEW.20151005, I launched clocker+weave in AWS.
The weave01 DelegateEntity continued to report the attribute "ServiceState" as STARTING, even though the real weave01 entity had a ServiceState of RUNNING. The delegate entity therefore continued to be "spinning" green in the web-console, as though it was still starting up.

This looks like an issue with the DelegateEntity itself, perhaps?

Heads-up: now that apache/incubator-brooklyn#929 is merged, I'm looking into removing DelegateEntity (and other cruft in AbstractGroup) altogether.

@alasdairhodge agreed, but remember we'd need to be building Clocker against a SNAPSHOT branch of Brooklyn for that currently. I'd prefer to use fixed releases as dependencies in Clocker, although we could build a custom 0.9.0 release. @richardcloudsoft any idea when the 0.9.0 release is due?

@grkvlt sorry I missed this comment. No discussion on the Apache Brooklyn release schedule / roadmap has happened on the list yet :disappointed:

Fixed


PLAT-23467: Signage: create new date picker

Implemented component based on the ID_New Moonstone_Requirement_UX&GUI_0725.pptx on PLAT-23745.
According to GUI recommendation FormCheckbox was used.
Enyo-DCO-1.1-Signed-off-by: Srirama<EMAIL_ADDRESS>

The componet  is moved to moonstone-extra . Closing this PR


Set(pin?) VM disk controller to an iothread in Sunstone

Description
Looking at the sources it should be possible to pin a given disk controller to an IOTHREAD when Feature/Iothreads is >0
Use case
Not practical use case. My observation is that moving all storage operations to a dedicated iothread(virtio data plane) is enough to improve the VM disk performance. But if the code already allows the iothread pinning, so why not? ;-P
Interface Changes
VM Template - Advanced disk attributes Iothread
VM attach virtio-blk disk - advanced disk attributes - Iothread
AFAIK in the case of the virtio-scsi disks there is a SCSI controller so will be tricky because there is no option to pin, or additional SCSI controller should be added?
Note: I think this should depend on the value of Features/Iothread and should be an option only when IOTHREADS > 1 (with a single iothread defined, the pinning is irrelevant)
Additional Context
Add any other context or screenshots about the feature request here. Or any other alternative you have considered to address this new feature.

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

@tinova PRs to be approved:

Code
Docs
Test: Pending

@tinova PR to be approved:

Tests


Implements sharding for non divisible vocab_size.

The code is relatively easy (just disable the checks on Embedding and Head)

This cannot be done in the same easy fashion for hidden_dim/head_dim.
It's relatively easy on some models (classic MHA) but it would make the other
models (MQA) much more complex, and GPTQ quantization another quite hairy piece
of code.
What does this PR do?

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

Thank you for this! Tried and it works perfectly!


Improve Status/Type annotations and add missing classes

This fixes https://github.com/Mangopay/mangopay2-php-sdk/pull/280#discussion_r227269705 and more importantly https://github.com/Mangopay/mangopay2-php-sdk/pull/279#discussion_r227269386. I discovered that there are many more places where this is wrong, so fixed them as well and also added a few missing Status and Type classes. I removed the string values from the doc blocks to avoid duplication and risk leaving outdated values if they ever get updated in the data class.
While working on it I tried to find more information on the Status property for Reports, but couldn't find any. While it exists in some code examples, it's not actually documented like the other endpoints (at least I didn't find it), e.g. READY_FOR_DOWNLOAD https://docs.mangopay.com/endpoints/v2.01/reporting#e824_the-report-object Would be great if we could add it there as well 🙏

Thanks for the review @alcaeus, I fixed most of the mentioned issues. While I agree that some/most property docs are not adding any additional information, IMO thats something for another PR since there are a lot more occurrences

A warm thank you for your precious add @venyii and for your time @iammichiel and @alcaeus !
I will make a new 2.6.1 release today with these ones.


Why is the accuracy so low on pedestrian and cyclist?

Thank you very much for providing this version of the code !
But I don't understand why the accuracy of pedestrian and cyclist is so low !
Is the parameter setting of a part of the code wrong?
What should I modify to achieve satisfactory results on these two categories?
If you have any helpful ideas, please let me know! Thank you very much！！！

I want to know it,too...

@qiqihaer I want to know it,too...

@qiqihaer I want to know it,too... love from 2024


New (0.84.5) harmony component fails with "websockets.exceptions.InvalidStatusCode" on 3 of 4 Harmony Hubs

Home Assistant release with the issue:
0.84.5
Last working Home Assistant release (if known):
N/A - issue was triggered by changes made in latest Harmony firmware
Operating environment (Hass.io/Docker/Windows/etc.):
Standard Home Assistant install in a Python venv on Ubuntu 18.04 LTS.  Details:
(homeassistant) homeassistant@hass:~/.homeassistant$ hass --version
0.84.5
(homeassistant) homeassistant@hass:~/.homeassistant$ cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=18.04
DISTRIB_CODENAME=bionic
DISTRIB_DESCRIPTION="Ubuntu 18.04.1 LTS"
(homeassistant) homeassistant@hass:~/.homeassistant$ python3 --version
Python 3.6.7

4 x Harmony Hubs running new 206 firmware.
Component/platform:
remote.harmony
Description of problem:

From inside the venv installed websockets via: pip3 install websockets
Start Home Assistant using previous configuration w/ 4 Harmony Hubs defined (see below for configuration.yaml
Three hubs report error similar to the following:

Dec 19 11:54:10 hass hass[17432]: 2018-12-19 11:54:10 ERROR (MainThread) [homeassistant.helpers.entity] Update for remote.living_room fails
Dec 19 11:54:10 hass hass[17432]: Traceback (most recent call last):
Dec 19 11:54:10 hass hass[17432]:   File "/srv/homeassistant/lib/python3.6/site-packages/homeassistant/helpers/entity.py", line 221, in async_update_ha_state
Dec 19 11:54:10 hass hass[17432]:     await self.async_device_update()
Dec 19 11:54:10 hass hass[17432]:   File "/srv/homeassistant/lib/python3.6/site-packages/homeassistant/helpers/entity.py", line 347, in async_device_update
Dec 19 11:54:10 hass hass[17432]:     await self.async_update()
Dec 19 11:54:10 hass hass[17432]:   File "/srv/homeassistant/lib/python3.6/site-packages/homeassistant/components/remote/harmony.py", line 203, in async_update
Dec 19 11:54:10 hass hass[17432]:     await self._client.get_config()
Dec 19 11:54:10 hass hass[17432]:   File "/srv/homeassistant/lib/python3.6/site-packages/pyharmony/client.py", line 205, in get_config
Dec 19 11:54:10 hass hass[17432]:     'vnd.logitech.harmony/vnd.logitech.harmony.engine?config'
Dec 19 11:54:10 hass hass[17432]:   File "/srv/homeassistant/lib/python3.6/site-packages/pyharmony/client.py", line 158, in _send_request
Dec 19 11:54:10 hass hass[17432]:     await self._perform_connect()
Dec 19 11:54:10 hass hass[17432]:   File "/srv/homeassistant/lib/python3.6/site-packages/pyharmony/client.py", line 129, in _perform_connect
Dec 19 11:54:10 hass hass[17432]:     self._ip_address, DEFAULT_HUB_PORT, self._remote_id
Dec 19 11:54:10 hass hass[17432]:   File "/srv/homeassistant/lib/python3.6/site-packages/websockets/py35/client.py", line 20, in __await_impl__
Dec 19 11:54:10 hass hass[17432]:     extra_headers=protocol.extra_headers,
Dec 19 11:54:10 hass hass[17432]:   File "/srv/homeassistant/lib/python3.6/site-packages/websockets/client.py", line 286, in handshake
Dec 19 11:54:10 hass hass[17432]:     raise InvalidStatusCode(status_code)
Dec 19 11:54:10 hass hass[17432]: websockets.exceptions.InvalidStatusCode: Status code not 101: 401

Errors repeat approximately every 6 seconds.
The fourth hub ("Garden Bedroom") initializes on Hass start and works as expected.

 LOL. I will .
Issue definitely is with not being able to switch from HTTP to web sockets. During testing when I saw this it was because the wrong hub ID was sent.
Question though is why. :-)
Can you enable debugging in HASS for components homeassistant.components.remote.harmony and pyharmony.client?
For example (you might already have logger there):
logger:
  default: warning
  logs:
    homeassistant.components.remote.harmony: debug
    pyharmony.client: debug

That will allow me to see what is being sent and received. I'm especially interested in the beginning parts when it is being setup and does 1st connection as that is when it gets the required information.
Thx.

I set default: critical to pare things down a bit and enabled debug on the two components listed above.  Here is the result: https://hastebin.com/iwiderimeq.log

OK, errors are not reported in there however I do also find it interesting already that I can only see send requests and responses for your Garden one.
Can you send me the log information from the start till right after you see the errors appearing?
Appreciate it.

That is 100% the contents of the log with the following configuration.yaml setup:
logger:
  default: critical
  logs:
    homeassistant.components.remote.harmony: debug
    pyharmony.client: debug

The log entries repeat every 6 seconds.

So right now no errors being reported? Seems to me that this time the 3 other's are "hanging" then as I do not see log entries there about them actually connecting.
Can you try just 1 hub in your configuration (i.e bedroom)? I have 2 hub's here at home and both work, but just wanna see what would happen then.
Further, can you run the following command just to check what version of websockets you have:
pip show websockets
Thx.

From inside the venv:
(homeassistant) homeassistant@hass:~/.homeassistant$ pip show websockets
Name: websockets
Version: 7.0
Summary: An implementation of the WebSocket Protocol (RFC 6455 & 7692)
Home-page: https://github.com/aaugustin/websockets
Author: Aymeric Augustin
Author-email<EMAIL_ADDRESS>License: BSD
Location: /srv/homeassistant/lib/python3.6/site-packages
Requires:
Required-by: pyharmony
(homeassistant) homeassistant@hass:~/.homeassistant$

configuration.yaml has been modified to only include the following remote:
remote:
  - platform: harmony
    host: <IP_ADDRESS>
    name: Bedroom
    activity: Roku

Device is not created, log output can be found here: https://hastebin.com/okiyoluyon.log

So it looks like it is hanging when trying to connect. Already updated the code here so that there is a timeout instead, but that won't solve you're issue except for getting an error then. :-)
Just with the 1 hub, run it with debug again but this time let's add websockets in there:
logger:
  default: critical
  logs:
    homeassistant.components.remote.harmony: debug
    pyharmony.client: debug
    websockets: debug

I know we're able to connect otherwise we would not be able to get the hub id (7174072 for the bedroom one). Thus the question is why is it hanging when trying to connect with websockets.

Updated logs with websockets: debug enabled: https://hastebin.com/abiyegavag.log

Well .......
You sure you wanna use those other 3? :-)
Here is the portion of the log file:
Dec 19 15:30:02 hass hass[10314]: 2018-12-19 15:30:02 DEBUG (MainThread) [pyharmony.client] Connecting to <IP_ADDRESS> for hub 7174072
Dec 19 15:30:02 hass hass[10314]: 2018-12-19 15:30:02 DEBUG (MainThread) [websockets.protocol] client - state = CONNECTING
Dec 19 15:30:02 hass hass[10314]: 2018-12-19 15:30:02 DEBUG (MainThread) [websockets.protocol] client - event = connection_made(<_SelectorSocketTransport fd=24 read=idle write=<idle, bufsize=0>>)
Dec 19 15:30:02 hass hass[10314]: 2018-12-19 15:30:02 DEBUG (MainThread) [websockets.protocol] client ! failing WebSocket connection in the CONNECTING state: 1006 [no reason]
Dec 19 15:30:02 hass hass[10314]: 2018-12-19 15:30:02 DEBUG (MainThread) [websockets.protocol] client - event = eof_received()
Dec 19 15:30:02 hass hass[10314]: 2018-12-19 15:30:02 DEBUG (MainThread) [websockets.protocol] client x half-closing TCP connection
Dec 19 15:30:02 hass hass[10314]: 2018-12-19 15:30:02 DEBUG (MainThread) [websockets.protocol] client - event = connection_lost(None)
Dec 19 15:30:02 hass hass[10314]: 2018-12-19 15:30:02 DEBUG (MainThread) [websockets.protocol] client - state = CLOSED
Dec 19 15:30:02 hass hass[10314]: 2018-12-19 15:30:02 DEBUG (MainThread) [websockets.protocol] client x code = 1006, reason = [no reason]

So the web socket connection itself is failing (or being disconnected). When I researched this what I mainly found was stating about a firewall. Now I'm doubtful this is the case here, but can you think of anything different between the 3 HUBs that fail and the one that works? Anything from a network perspective?
Can you try restarting one of those HUBs and then see if that resolves the issue?

You sure you wanna use those other 3? :-)

LOL, well up until your PR I wasn't able to use any of the 4, so being able to use 1 is a massive improvement :D
All 4 Harmony Hubs and Hass are on the same VLAN and /16 address space.  All 4 hubs are on the same WiFi SSID while Hass is a VM wired via 10gbit.  There are no east/west firewalls in place and being on the same subnet/VLAN the traffic shouldn't traverse the router or be impacted by firewall rules.  Prior to the firmware update all 4 Harmony devices were working as expected.  Finally, from the Hass VM I can issue a telnet <harmony hub IP> 8088 and it is able to connect.
Power cycling one of the hubs (in this case, the hub labeled "Living Room" @ <IP_ADDRESS>) and changing configuration.yaml to connect to just that one hub hasn't changed the behavior.  Log messages are identical to the previous log, just with the new IP and hub ID (now [pyharmony.client] Connecting to <IP_ADDRESS> for hub 7174902).

I figured that but was hoping. :-) I know that from HASS we're able to connect 8088 cause we can get the Hub ID number. We use HTTP to get that information to the HUB on port 8088.
But when we try to connect for Web Socket it fails. Any way you can do a packet trace between your HASS and a HUB that fails?
Here is another idea. Do you have the Harmony app on a mobile device (i.e. iOS, Android)? If you do, are you able to connect from that app to those HUBs? It uses the same web socket API.
I'm also switching to use the aiohttp websocket implementation. Makes more sense as I already use aiohttp. Maybe an issue in the websocket implementation?

Harmony app on mobile devices (both Android and IOS) continue to work as expected against all 4 hubs.  I executed a packet capture of unicast traffic between my Hass VM and the target hub "Living Room" @ <IP_ADDRESS> via the following command: sudo tcpdump -w bedroomharmony.pcap -i ens160 host <IP_ADDRESS>
The resulting dump can be downloaded here: https://svideo.org/livingroomharmony.zip
In packet 16 we have a request being sent to the hub: http://<IP_ADDRESS>:8088/?domain=svcs.myharmony.com&hubId=7174902
In packet 18 we get the response: HTTP/1.1 401 Wrong domain\r\n

I think I got it. :-)
When doing the post, the response returned on my HUB for discoveryServerUri is https://svcs.myharmony.com/Discover/Discover.svc. Then when opening the websocket I provide: domain=svcs.myharmony.com
In yours that is failing, it has https://svcs-preview.myharmony.com/Discover/Discover.svc instead.
So, if it requires the domain provided to be the same as the hostname from discoveryServerUri then I am indeed providing the wrong domain now. And that is also the error you get.
Interesting however that I'm not getting an exception there or so.
Anyways, checking now how I can provide a fix to you for this. I need to know if my theory is right

Ooooohhhh.  So this MIGHT just be a "me" thing!  Some long time ago Logitech had a beta forum and user group who would receive firmware updates before everyone else.  They've since stopped doing that, but seeing that -preview strongly suggests that this is a leftover beta thing still attached to my account.

OK,
I created what hopefully is a fix for this. Here is what you need to do:

In your HASS config folder create folder custom_components/remote
Change to that folder
Download updated harmony component (only change here is just for requirement so that it points to my fork instead):
curl https://raw.githubusercontent.com/ehendrix23/home-assistant/harmony_fixes/homeassistant/components/remote/harmony.py -o harmony.py

Now restart HASS. In the logs it should show a warning about custom component.
Check if this works. :-)

CONFIRMED WORKING!  That version now discovers and allows control of all 4 of my hubs.  Now the component supports people who were in the beta group too!

For anyone else, my plan is to get this into HASS itself with the next release as I do not believe too many would be impacted by this and for anyone who is there is a temporary solution available.

Here's the result on startup:
Dec 26 15:38:11 hass hass[6982]: 2018-12-26 15:38:11 ERROR (MainThread) [homeassistant.components.remote] Error while setting up platform harmony
Dec 26 15:38:11 hass hass[6982]: Traceback (most recent call last):
Dec 26 15:38:11 hass hass[6982]:   File "/srv/homeassistant/lib/python3.6/site-packages/homeassistant/helpers/entity_platform.py", line 128, in _async_setup_platform
Dec 26 15:38:11 hass hass[6982]:     SLOW_SETUP_MAX_WAIT, loop=hass.loop)
Dec 26 15:38:11 hass hass[6982]:   File "/usr/lib/python3.6/asyncio/tasks.py", line 358, in wait_for
Dec 26 15:38:11 hass hass[6982]:     return fut.result()
Dec 26 15:38:11 hass hass[6982]:   File "/home/homeassistant/.homeassistant/custom_components/remote/harmony.py", line 105, in async_setup_platform
Dec 26 15:38:11 hass hass[6982]:     name, address, port, activity, harmony_conf_file, delay_secs)
Dec 26 15:38:11 hass hass[6982]:   File "/home/homeassistant/.homeassistant/custom_components/remote/harmony.py", line 161, in __init__
Dec 26 15:38:11 hass hass[6982]:     disconnect=self.got_disconnected
Dec 26 15:38:11 hass hass[6982]:   File "/srv/homeassistant/lib/python3.6/site-packages/aioharmony/harmonyapi.py", line 44, in __init__
Dec 26 15:38:11 hass hass[6982]:     loop=loop
Dec 26 15:38:11 hass hass[6982]:   File "/srv/homeassistant/lib/python3.6/site-packages/aioharmony/harmonyclient.py", line 89, in __init__
Dec 26 15:38:11 hass hass[6982]:     handler = copy.copy(handlers.HANDLER_START_ACTIVITY_FINISHED)
Dec 26 15:38:11 hass hass[6982]:   File "/usr/lib/python3.6/copy.py", line 88, in copy
Dec 26 15:38:11 hass hass[6982]:     return copier(x)
Dec 26 15:38:11 hass hass[6982]:   File "/srv/homeassistant/lib/python3.6/site-packages/aioharmony/handler.py", line 75, in __copy__
Dec 26 15:38:11 hass hass[6982]:     resp_json=copy.deepcopy(self._resp_json),
Dec 26 15:38:11 hass hass[6982]:   File "/usr/lib/python3.6/copy.py", line 150, in deepcopy
Dec 26 15:38:11 hass hass[6982]:     y = copier(x, memo)
Dec 26 15:38:11 hass hass[6982]:   File "/usr/lib/python3.6/copy.py", line 240, in _deepcopy_dict
Dec 26 15:38:11 hass hass[6982]:     y[deepcopy(key, memo)] = deepcopy(value, memo)
Dec 26 15:38:11 hass hass[6982]:   File "/usr/lib/python3.6/copy.py", line 161, in deepcopy
Dec 26 15:38:11 hass hass[6982]:     y = copier(memo)

Well, that is definitely not the fireworks I was expecting. :-)
Clearly something different between Python 3.6 and 3.7 here.
Just pushed a new update. Download it with curl and see if that works.  I might need to see to get Python 3.6 and 3.5.3 as well locally to test with.

Updated version now works as expected across all 4 hubs!


feat: make event types be strongly defined

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

feature

What is the current behavior? (You can also link to an open issue here)

the events emitted by this library are not documented in the typescript typings.

What is the new behavior (if this is a feature change)?

This makes them be typed.
There are some other changes here too, which were needed while using the library, maybe they should be submitted separetely;

remove SlotId enum as all the newer models support more than 2 slots.

Add inputVideoFormat parameter from transport info

Other information:

Codecov Report
Patch coverage: 62.85% and project coverage change: -0.50% :warning:

Comparison is base (fb7a34b) 52.34% compared to head (af392d0) 51.85%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #25      +/-   ##
==========================================
- Coverage   52.34%   51.85%   -0.50%
==========================================
  Files          37       37
  Lines         852      862      +10
  Branches      140      145       +5
==========================================
+ Hits          446      447       +1
- Misses        355      362       +7
- Partials       51       53       +2

Files Changed
Coverage Δ

src/commands/clips.ts
13.63% <ø> (ø)

src/commands/deviceInfo.ts
30.00% <ø> (ø)

src/commands/diskList.ts
13.33% <ø> (ø)

src/commands/goTo.ts
11.11% <0.00%> (-2.23%)
:arrow_down:

src/commands/slotInfo.ts
21.42% <ø> (ø)

src/commands/slotSelect.ts
17.64% <0.00%> (ø)

src/util.ts
35.29% <25.00%> (-3.17%)
:arrow_down:

src/commands/transportInfo.ts
23.52% <33.33%> (-3.14%)
:arrow_down:

src/asyncHandlers/transportInfo.ts
45.45% <75.00%> (-4.55%)
:arrow_down:

src/hyperdeck.ts
84.45% <77.77%> (ø)

... and 5 more

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Adding a .gitignore to not commit executable files

In the Graph directory, I see every code has it's .exe file committed to the repository (which is not a good practice)

@manan-shxrma, if you can please confirm the issue

great @meetagrawal09 you can solve this issue

Made a PR for this #50


Leverage sqlparser-rs for query parsing.

cargo add sqlparser to introduce improved check and error response for validity of query input.
Example using invalid query "test":
pg_later=# select pglater.exec('select * from pg_available_extensions order by name limit 2') as job_id;

job_id
--------
      1
(1 row)

pg_later=# select pglater.exec('test') as job_id;
ERROR:  Query parsing failed, please submit a valid query: ParserError("Expected an SQL statement, found: test at Line: 1, Column 1")

Add a test either in same file or in the integration tests then this is gtg!

closes https://github.com/tembo-io/pg_later/issues/46


[stable/fluent-bit] Add Kafka output for sending messages to a topic

Signed-off-by: Davide Berdin<EMAIL_ADDRESS>What this PR does / why we need it:
This PR will allow user to output the messages of fluent-bit to a kafka-topic (or multiple topics)
Which issue this PR fixes

fixes #15136

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

why not use rawConfig?

@kfox1111 can you elaborate a bit?


Update arch aliases for Mac ARM series

This PR adds arch aliases - arm64 & arm for ARM64. This is because some binaries like gcloud do not have arm64 in the binary name, which was causing issues for installation.
Relevant discussion - https://redhat-internal.slack.com/archives/C04PUGBE7JN/p1716351644144979
Before the change
❯ backplane-tools install gcloud
Installing the following tools:
Error: failed to locate latest archive matching system spec: unexpected number of assets found matching system spec: expected at least 1, got 0
Usage:
  backplane-tools install [all|backplane-tools|oc|rosa|yq|butane|servicelogger|aws|ocm|ocm-addons|osdctl|backplane-cli|gcloud] [flags]

Flags:
  -h, --help   help for install

2024/05/22 10:35:49 Error executing command: failed to locate latest archive matching system spec: unexpected number of assets found matching system spec: expected at least 1, got 0

After the change
❯ ./backplane-tools install gcloud
Installing the following tools:
- gcloud google-cloud-cli-477.0.0-darwin-arm

Installing gcloud
Successfully installed gcloud

/lgtm

/approve


grpc-tools does not include binaries for arm64 on macOS

What version of gRPC and what language are you using?
1.42.0, ruby
What operating system (Linux, Windows,...) and version?
macOS arm64. The new M1s running on "apple silicon".
What runtime / compiler are you using (e.g. python version or version of gcc)
N/A
What did you do?
After installing the grpc-tools gem, I attempted to run the grpc_tools_ruby_protoc script.
What did you expect to see?
I expect the arm64-darwin binaries to be included and used.
What did you see instead?
It failed to run when attempting to run the x86_64 binary. According to this snippet, grpc_tools_ruby_protoc assumes rosetta is installed, and that I can just run the x86_64 binary. However,

Rosetta is not always installed.
I believe Apple will be dropping support for rosetta in the future.
Some corporations have a no-rosetta policy, and that makes this gem unusable in those environments.

It looks like this issue added support for arm64 on macOS, but the grpc-tools gem was forgotten.

Any update on this? I know how to fix this issue on my local machine, but I'm not sure how the binaries get built/distributed with the Gem, and how they end up in the src/ruby/tools/bin folder.

Also looking forward to this. @awall if you have any tips on sorting it locally I'd like to try that in the meantime 🤠

@rdavisau I actually figured out how the binaries are built, I have a comprehensive fix and will submit a PR tomorrow.

@rdavisau If you can't wait for the PR to be merged, the following line in your Gemfile will bring in my fix:
gem "grpc-tools", github: 'awall/grpc', branch: 'tools-arm64', glob: 'src/ruby/tools/*.gemspec'

@awall Thanks for putting this PR together.  Can you share information on how to build the protoc binaries and grpc_ruby_plugin?

I decided to set up a periodic GitHub Actions workflow to provide daily builds of Apple Silicon wheels for Mac. It will automatically cut a new release the day a new grpcio version is released on PyPI.
In the releases page of this repo you can find and download the wheels for Python 3.8, 3.9, 3.10, 3.11.
(Use at your own risk, these are not tested.)
https://github.com/pietrodn/grpcio-mac-arm-build

Also looking for this, it seems like grpc-tools gem still does not work on M1 Mac without Rosetta installed? @awall ?

@mzanin this gem still does not work unless Rosetta is installed.
@pietrodn I'm confused, that's a python wheel... grpc-tools is a Ruby gem.

Ah sorry, I didn't notice that the original issue was about Ruby. Please ignore my comments!

We cannot release mac arm binaries yet as signature is required

Consolidate the discussion in #25755