Implement typed errors for all codegen

This continues the work on typed errors started by #261, which implemented them for JSON protocol services.
This PR implements typed errors for all other services produced by our codegen, and does away with the catch-all AwsError type altogether.
S3 remains the odd man out, though I have put in a S3-specific catch-all S3Error there to replace AwsError.  Implementing typed errors for each S3 method like we've done in the codegen would require a ridiculous amount of typing, and is better handled later in a different, yet-to-be-opened S3 cleanup project.
This builds on #291, so that one will need to get merged first.

Looks good, thanks @adimarco !
Should we add a top-level description to the errors generated by this so they show up in the documentation?  Right now it's blank, as seen in this screenshot:

Yeah, I can add some top level description there, though it'll have to be something generic like "an enum type of all errors generated by create_cluster()" since the detailed error descriptions annotate the enum elements themselves.  I'll ponder wording.

It's a nice to have, don't think it should block this PR.

Opened an issue for the descriptions.  Go ahead and merge if you think it's ready.


Remove all EOL maintenance branches

For the sake of cleanliness, all of our old end-of-life maintenance version branches should be removed once confirmed that they are merged back into master (i.e. so that we aren't losing commits).
In addition:

Remove the "decoupler" branch
Remove all merged feature branches

Curently working on removing the decoupler branch

decoupler has been deleted

Release branches:
v1.13.x
v1.12.x
v1.11.x
v1.10.x
v1.9.x
v1.8.x
v1.7.x
v1.6.x
v1.5.x
v1.4.x
v1.3.x
v1.2.x
v1.1.x
v1.0.x
v0.9.x - PR https://github.com/SciTools/iris/pull/2853

Thanks @djkirkham.


HHH-18867 correctly implement the documented semantics of beginTransaction()

and clarify its semantics in the jdoc

[Please describe here what your change is about]

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.

@sebersole WDYT?

I dislike changing the semantics here yet again.  We've changed the semantics around Transaction a few times the last couple of releases.
But I agree this does match better with the JPA spec, so +1


open-api plugin does not obey "effective documentation" precedence for member structures

There appears to be a bug in the open-api plugin that leads to incorrectly applied documentation.  This is causing some friction for us, requiring refactoring of our model to display documentation correctly.  If the plugin behaved according to the documentation here: Effective documentation, we would not have any issue.
To repro:
model.smithy
$version: "2"

namespace example

use aws.api#service
use aws.protocols#restJson1

/// Cutting edge foo barring service, now with AI
@service(sdkId: "Foo")
@restJson1
service FooService {
    version: "2006-03-01"
    operations: [
        BarOperation
    ]
}

/// Performs a bar operation.
@http(uri: "/bar", method: "POST")
operation BarOperation {
    input: BarInput
    output: BarOutput
}

@documentation("It's a Quux")
structure Quux {
    quux: String
}

/// Input for the BarOperation.
structure BarInput {
    @required
    @documentation("It's a Bar")
    bar: Quux
}

/// Output for the BarOperation.
structure BarOutput {
    /// The result of the bar operation.
    result: String
}

smithy-build.json
{
  "version": "1.0",
  "maven": {
    "dependencies": [
      "software.amazon.smithy:smithy-aws-traits:1.51.0",
      "software.amazon.smithy:smithy-openapi:1.51.0"
    ]
  },
  "sources": ["model.smithy"],
  "plugins": {
    "openapi": {
      "service": "example#FooService",
      "protocol": "aws.protocols#restJson1"
    }
  }
}

This produces a model.json that correctly contains "It's a Bar" for the member's documentation:
 "example#BarInput": {
            "type": "structure",
            "members": {
                "bar": {
                    "target": "example#Quux",
                    "traits": {
                        "smithy.api#documentation": "It's a Bar", <-- correct
                        "smithy.api#required": {}
                    }
                }
            },
            "traits": {
                "smithy.api#documentation": "Input for the BarOperation."
            }
        },

however, the generated FooService.openapi.json contains a schema that looks like this:
{
        "schemas": {
            "BarOperationRequestContent": {
                "type": "object",
                "description": "Input for the BarOperation.",
                "properties": {
                    "bar": {
                        "$ref": "#/components/schemas/Quux" <-- missing "description"
                    }
                },
                "required": [
                    "bar"
                ]
            },
            "BarOperationResponseContent": {
                "type": "object",
                "description": "Output for the BarOperation.",
                "properties": {
                    "result": {
                        "type": "string",
                        "description": "The result of the bar operation."
                    }
                }
            },
            "Quux": {
                "type": "object",
                "description": "It's a Quux",
                "properties": {
                    "quux": {
                        "type": "string"
                    }
                }
            }
        }
    }

I expect to see:
                "properties": {
                    "bar": {
                        "$ref": "#/components/schemas/Quux",
                        "description": "It's a Bar" <--
                    }
                }

The example from https://smithy.io/2.0/spec/documentation-traits.html#effective-documentation works correctly.  This appears to specifically affect structure members.

With https://github.com/smithy-lang/smithy/pull/2402 you are able to add "addReferenceDescriptions": true to your openapi plugin config so you get member docs on $ref properties

Awesome, thanks!


[Brew] Cannot open details of installed package

Extension
https://www.raycast.com/nhojb/brew
Raycast Version
1.74.1
macOS Version
14.4.1
Description
Error:
TypeError: Cannot read properties of null (reading 'value')

NC:index.js:13:25465
index.js:13:25312
    at Array.map (<anonymous>)
Ji:index.js:13:25305

Steps To Reproduce

Open brew installed
Go open a package that is from a tap

Current Behaviour
No response
Expected Behaviour
No response

@vpukhanov I still have the issue. Copied error log:
TypeError: Cannot read properties of null (reading 'value')

OC:index.js:13:24813
index.js:13:24660
    at Array.map (<anonymous>)
Ji:index.js:13:24653

What's the package and tap that are causing the issue?

It seems to only happen for packages installed through taps:
For instance: tap sst/tap/sst and package sst
https://github.com/user-attachments/assets/8b0ab2a9-0efe-4032-9418-dc36dfc39ba5

#14596 Should fix this problem

Fixed
@raycastbot close this issue


ARO v4 On Boarding Script CLI syntax error (az group deployment)

Hi,
Regarding the documentations here:
https://docs.microsoft.com/en-us/azure/azure-monitor/insights/container-insights-azure-redhat4-setup
This bash script includes invalid CLI command:
curl -LO https://raw.githubusercontent.com/microsoft/OMS-docker/ci_feature/docs/aroV4/onboarding_azuremonitor_for_containers.sh
Error:
az deployment: 'group' is not in the 'az deployment' command group. See 'az deployment --help'. If the command is from an extension, please make sure the corresponding extension is installed. To learn more about extensions, please visit https://docs.microsoft.com/en-us/cli/azure/azure-cli-extensions-overview

Fix:
Update the command to the following:
az group deployment create ....

az -v
azure-cli                          2.1.0
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0a7aaa8a-a94b-f2ba-7365-b527519e9920
Version Independent ID: 86c97898-1d8a-69fe-35f4-613fc0e336fd
Content: Configure Azure Red Hat OpenShift v4.x with Azure Monitor for containers - Azure Monitor
Content Source: articles/azure-monitor/insights/container-insights-azure-redhat4-setup.md
Service: azure-monitor
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@mohamedsaif  Thank you for bringing this to our notice, we will look into it and get back to you at the earliest.

@mohamedsaif  Thanks again for bringing this to our notice. I see that you are also created a GitHub issue in respective OMS-docker repo. I have created a PR  to update the Az group deployment syntax in the script. Once its reviewed and approved the script will update accordingly.
Hope the above information helps. At this point it appears that there's no further edits or updates needed to the documentation. We will go ahead and close this issue, however if you have any further queries please tag us in your reply and will be happy to reopen the thread and continue the conversation. Thank you


Bump kube to 0.70.0. Release 0.15.0

Description
Bump kube to 0.70.0 for bugfix https://github.com/kube-rs/kube-rs/pull/852.
As this is a braking change I've updated to a new minor version.

Review Checklist

[x] Code contains useful comments
[x] (Integration-)Test cases added (or not applicable)
[x] Documentation added (or not applicable)
[x] Changelog updated (or not applicable)
[x] Cargo.toml only contains references to git tags (not specific commits or branches)

Once the review is done, comment bors r+ (or bors merge) to merge. Further information

bors r+


a bug report

Hello! Sorry, but I cannot reproduce this bug. How do you use this component?
The following code works fine:

Also try adding "noImplicitAny": false to your tsconfig.json

 <React.StrictMode>
            <HTMLFlipBook/>
 <React.StrictMode/>

worked well!
another bug. when flip from next to pre page, pre pages content appeared. its ok from pre to next ;


Suggestion

Great tool ❤❤
Can you introduce a silent flag that doesn't spit out output but just posts to slack.

Hi @nine9-nine9, thanks for the feedback & suggestion!
Looks like the workaround you're talking about could easily use &>/dev/null to suppress stdin messages.

How about adding which channel to send output to??


Support for type hints

The lack of type hinting is a pain for my development workflow, as tools like mypy outright refuse to analyze pyarr code, and even Pylance in VS Code doesn't seem to infer the types properly most of the time.
It should be fairly simple to add types, as most of the API returns dict[str, Any] or list[dict[str, Any]]. It's an implementation decision on whether or not it should be done using stubs or actually implementing it in the codebase.
I can begin working on this if I can find some time, but depending on how far we've gotten on switching to using asyncio, it might be worth waiting until after that's done just so merging won't be a pain.

asyncio support is still very rudimentary. I can handle merging if you want to push out a separate PR for type hints

Feel free to steal the type hints directly from #93

I can pick this up as ive got some time spare. I will use use union instead of pipe to keep it compatible with 3.9 unless you see any issues with doing that?

Yea that was my thinking, can always swap it at a later date when 3.9 is not as common

I didn't figure out a solution for this as well. I ended up just specifying any as one of the return types just to avoid the linting error.

Using TypedDict seems a little excessive. I see no downside to using assert isinstance(...) personally.

Should probably specify dict[str, Any] as opposed to a dict. Would prevent trying to get ret[0] for example (expecting a list to be returned instead of a dict).


Cached template strings causes issues with escaped strings

When using the cache on a particular component the cache causes the server to return some unexpected behaviour.
example: const MyComponent = (props) => (
{props.name}
)
MyComponent: {
  templateAttrs: ['name']
}

Now on the server side, I pass in something with a html equivalent value:
<MyComponent name={'My Name & name 2'} />
this gets put in the cache as something like:
${name}

When MyComponent is pulled from the cache and string replaced, & is placed instead of &.
The next time the component is rendered it returns &amp; If caching is not enabled the problem is not present.

@varsis are you able to reproduce this issue in a unit test?  We haven't encountered this with our use of this library...


[lib] Adds Missing Attributes to PerformanceResourceTiming

Summary:
Problem:
Flow's exported definition of PerformanceResourceTiming is missing some attributes. PerformanceResourceTiming's attributes are specified in the Resource Timing spec.
https://www.w3.org/TR/resource-timing-2/#sec-performanceresourcetiming
https://github.com/facebook/flow/issues/8590
Solution:
Add the missing attributes to the PerformanceResourceTiming class definition.
Test Plan:
make test

I signed the individual Facebook CLA, but this change is motivated by some work I'm doing for Twitter Inc.


Wrr traffic test changes

This PR modifies WRR weight assigment to queues to values that most vendor should be able to accept:

Highest weight value is 63 what prevent some vendor form scaling weight down and losing accuracy
SUM of WRR rates is <100 (27) what prevent some vendor form rounding weight down and losing accuracy

Consequently expected loss-rate per class (PHB) has been also modified.
It this tests suite AF4 and NC1 are served as sptrict priority, hence weights are irrelevant
WRR weights for for AF3, AF2, AF1, BE0 and BE1 with weight 12, 8, 4, 1 and 2 respectively.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 56.88%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1236

Relevant Lines:
2173

💛  - Coveralls

https://github.com/openconfig/featureprofiles/pull/1242  There is already PR with weights of 64, 16, 4, 1 and
1. Probably change the highest weight to 63 and it helps to cover 1:4 for few queues

FYI: Only issues representing the existence of a test should be part of the project lists.  I'm removing this closed item which is a change to a test from the two projects it's assigned to.


[Feature]: Enhance Accessibility Testing within PR flow

Provide a general summary of the suggestion
Accessibility is a core aim of the theme, making sure that content is accessible to all.
Context
Accessibility is something that all of us are responsible for, and have a duty to work on. To ensure the theme continues to aim for high standards, this should be a part of the PR check.
Possible Implementation

Microsoft Accessibility Insights as part of PR flow
Google LightHouse as part of PR Flow

Your Environment
No response

Blocked by https://github.com/microsoft/accessibility-insights-action/issues/629


Update mapping to OPC UA?

In V3.0 the mapping to OPC UA is described as in the following. In the meanwhile there are new activities, do we need an update?
Note1 : the mapping to the OPC UA are carried out in a joint working group[10] "I4AAS" between OPC Foundation, ZVEI and VDMA (https://opcfoundation.org/markets-collaboration/I4AAS/ [30].
Note 2: the different versions of the OPC UA Companion Specification for I4AAS can be found here: https://reference.opcfoundation.org//I4AAS//docs/, e.g. https://reference.opcfoundation.org/v104/I4AAS/v100/docs/ for release 1.00 [41].
draft: https://rwth-iat.github.io/aas-specs-antora/IDTA-01001/snapshot/IDTA-01001_Mappings.html#_format_normal_in_opc_ua

See https://github.com/admin-shell-io/aas-specs/issues/397

2024-11-27 TF Metamodel
see https://github.com/admin-shell-io/aas-specs/issues/397: chapters on OPC UA and AML were removed. So nothing to be changed.


fix(es/parser): Fix parsing of auto accessors

Description:
Related issue:

This is required for https://github.com/swc-project/swc/pull/6950.

Well this was not required


auth-mode server fails when Authorization header is passed.

Summary
We have argo server behind a reverse proxy that does github SSO.. Upgrading from argo 2.8 to 2.11, using --auth-mode server broke the UI. /api/v1/version was returned
{"code":16,"message":"no valid authentication methods found for mode client"}

Diagnostics
The error is caused because our reverse proxy adds an authorization header.
To recreate:

run argo server with auth-mode server
port-forward to argo-ui
This works curl http://localhost:2746/api/v1/version
This fails curl -H "Authorization: Bearer 1234"  http://localhost:2746/api/v1/version

This was tracked down to:
https://github.com/argoproj/argo/blob/f9b65c52321d6e49d7fbc78f69d18e7d1ee442ad/server/auth/gatekeeper.go#L117-L118
when only auth-mode server is active, authorization and cookie headers should be ignored.
What Kubernetes provider are you using?
GKE
What version of Argo Workflows are you running?
2.11.8

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

Why are you sending a header?

@tcolgate you are correct. This behavior changed in 2.9 version. Do you like to submit PR?

Sorry, I'm not going to have time, I'd need to understand things a bit better to be confident I'd not be breaking anything else.

Can you try this image argoproj/argocli:modes. This now only uses an auth mode if the correct header is supplied AND the mode is enabled.

Authorization: Bearer v2:.* + SSO = SSO
Authorization: Bearer .* + Client = Client
Authorization: Basic .* + Client = Client
Server = Server

I'm not totally sure what you're set-up is, so this may fix you bug, or may not.

This fix appears to have been recently undone. Was that intentional? https://github.com/argoproj/argo/pull/4496/files
Works in v2.12.0-rc1 but seeing this issue again in later rcs.

This fix appears to have been recently undone.

Do you mean you can point to a specific change that undoes it? Or do you actually mean the problem has started to appear again?

Both. It looks like the additional checks for the current auth-mode were removed in the linked PR: https://github.com/argoproj/argo/pull/4496/files

@simster7 -your PR?

@dylanhellems I'm having a hard time seeing what the issue is just looking at the changes from the PR you linked. Can you elaborate a bit more on what the issue is, as well as what lines in particular you think affected it? Some steps to reproduce would also help.

The original fix for this issue added checks to the GetMode conditionals to ensure that it didn't try to use an inactive auth mode (see: the && m[Client] in  https://github.com/argoproj/argo/pull/4412/files#diff-d26743300120981351842cdfdb25e21906ed62d3ca53c448ea6df863d7f1a90cR37).
The later PR removes those extra conditions (see: https://github.com/argoproj/argo/pull/4496/files#diff-d26743300120981351842cdfdb25e21906ed62d3ca53c448ea6df863d7f1a90cL37)
The problem is that if you send an Authorization header in a request to argo while using the "server" auth-mode, it will hit the client mode conditional and fail with a 401. This is a problem for our team since we are using istio as an authentication mechanism instead of argo's built-in auth, and istio is adding the header to requests.

I see what you mean, I hadn't considered the validity of using Authorization headers while using server mode. I'll get a PR in soon and this will be fixed in the next release

@dylanhellems The fix for this was just released in RC6. Could I ask you to test?

https://github.com/argoproj/argo/releases/tag/v2.12.0-rc6

@simster7 Working for me, thanks!


feat: add some prompts

Have you got any examples of what these look like? 😄

They will probably vary slightly based on rtv config, but here are the examples I got:
colorful skeuomorphic pattern, cgsociety

colorful glossy pebbles, highly detailed pebbles, cgsociety

colorful bubbles, 3d, cgsociety

colorful fractals, cgsociety

colorful mandelbrot fractals, cgsociety

3d pipes, shiny metal, cgsociety

First skeuomorphic is probably a bit boring, I personally find the pebbles most satisfying to look at 😄

😍 cool!


view is stuck in loading state after creating a generic imported cluster through cluster managementis

on master-head(9afe417

)
the following errors appear in the console when the user creates a generic imported cluster:
vue.runtime.esm.js:1888 TypeError: Cannot read property 'parentLocationOverride' of null
    at f.parent (Masthead.vue:180)
    at _n.get (vue.runtime.esm.js:4479)
    at _n.evaluate (vue.runtime.esm.js:4584)
    at f.parent (vue.runtime.esm.js:4836)
    at f.r (Masthead.vue?734d:1)
    at f.t._render (vue.runtime.esm.js:3548)
    at f.r (vue.runtime.esm.js:4066)
    at _n.get (vue.runtime.esm.js:4479)
    at _n.run (vue.runtime.esm.js:4554)
    at gn (vue.runtime.esm.js:4310)
vue.runtime.esm.js:1888 TypeError: Cannot read property 'isRke2' of null
    at f.r (provisioning.cattle.io.cluster.vue?08ea:1)
    at f.t._render (vue.runtime.esm.js:3548)
    at f.r (vue.runtime.esm.js:4066)
    at _n.get (vue.runtime.esm.js:4479)
    at _n.run (vue.runtime.esm.js:4554)
    at gn (vue.runtime.esm.js:4310)
    at Array.<anonymous> (vue.runtime.esm.js:1980)
    at ie (vue.runtime.esm.js:1906)

and the user's screen is in a loading state
reloading did not fix the issue, but going back to -> clusters -> new imported cluster allows the user to see the correct (registration command list) screen.

I've seen this before, but with an exception of something like Cannot read isRKE2 of null


/users/:user/completed_solutions returns 500 error "undefined method `code_challenge_review' for nil:NilClass"

Hello,
I can't see my submitted solutions, each time I enter page I keep getting:

@jhoffner  Will this going to be fix soon?

@kkorus he's on vacation right now.
By "submitted solutions", do you mean https://www.codewars.com/users/kkorus/completed_solutions?

Possibly related #944

@kazk
Yep, thats it, keep getting this error.

Ok, then this is the same issue as #944 which seems to have resolved itself :/

I think it can be closed.

@kkorus thanks!


Clarified Scheduled Task name and location.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Looks fine to me. Please merge.


FakeQuantWithMinMaxVarsPerChannel operation  - mismatch of returned values TF vs SameDiff

Failing test:
    public void testFakeQuantAgainstTF_1() {
        INDArray x = Nd4j.createFromArray(new float[]{ 0.7788f,0.8012f, 0.7244f, 0.2309f,0.7271f,
                0.1804f,    0.5056f,    0.8925f,    0.5461f,    0.9234f,
                0.0856f,    0.7938f,    0.6591f,    0.5555f,    0.1596f}).reshape(3,5);
        INDArray min = Nd4j.createFromArray(new float[]{-0.2283f,   -0.0719f,   -0.0154f,   -0.5162f,   -0.3567f}).reshape(1,5);
        INDArray max = Nd4j.createFromArray(new float[]{0.9441f,    0.5957f,    0.8669f,    0.3502f,    0.5100f}).reshape(1,5);

        INDArray out = Nd4j.createUninitialized(x.shape());
        val op = new FakeQuantWithMinMaxVarsPerChannel(x,min,max,out);

        INDArray expected = Nd4j.createFromArray(new float[]{0.7801f,    0.5966f,    0.7260f,   0.2320f,    0.5084f,
                0.1800f,    0.5046f,    0.8684f,    0.3513f,    0.5084f,
                0.0877f,    0.5966f,    0.6600f,    0.3513f,    0.1604f}).reshape(3,5);

        assertEquals(expected, out);
    }

cc @shugeo ^^^

There no mismatch with TF. min and max arguments should be 1D vectors with length as last dim of the input array.

TF returns 0.77700233 0.596913   0.72314    0.23104    0.50982356]
[0.17930824 0.50528157 0.86846    0.34995764 0.50982356]
[0.08735529 0.596913   0.6574     0.34995764 0.15974471]

Our out is
Quantized03:
[[0.777002, 0.596913, 0.723140, 0.231040, 0.509824]
[0.179308, 0.505282, 0.868460, 0.349958, 0.509824]
[0.087355, 0.596913, 0.657400, 0.349958, 0.159745]]


Ignore interpolated values for misspelled properties

When linting for misspellings, interpolated values within property names warn of a spelling mistake. It would be good to exclude these somehow.
For instance
$u-layout-divider-types: (
    bottom,
    left,
    right,
    top
);

@each $type in $u-layout-divider-types {
    .u-layout-divider-#{$type} {
        border-#{$type}: 1px solid #000 !important;
    }
}

results in "Property border- appears to be spelled incorrectly"
Using v1.7.0 of sass-lint through gulp-sass

This is 'fixed' in 1.8.0 but only because I've spotted a new bug! 😭


SwitchBot] Bot: Amplifier Device internal error due to device states not synchronized with server, or command format is invalid, statusCode: 190

Describe Your Problem
Cannot use switch bot suddelny, any help? statusCode: 190
With the Switchbot app everything works. but getting an error with homekit.
Relevant log output
[26/10/2024, 12:05:33] [SwitchBot] Bot: Amplifier Device internal error due to device states not synchronized with server, or command format is invalid, statusCode: 190

Config for homebridge-switchbot
{
    "name": "SwitchBot",
    "credentials": {
        "token": "sensitive removed",
        "secret": "sensitive removed",
        "notice": "Keep your token a secret!"
    },
    "options": {
        "devices": [
            {
                "configDeviceName": "Amplifier",
                "deviceId": "sensitive removed",
                "configDeviceType": "Bot",
                "connectionType": "OpenAPI",
                "type": "switch",
                "mode": "switch"
            }
        ]
    },
    "_bridge": {
        "username": "sensitive removed",
        "port": 43432
    },
    "platform": "SwitchBot"
}

Screenshots
No response
Device & Model
HUB Plus F4 and SwitchBot V4.8
Node.js Version
v20.18.0
NPM Version
10.9.0
Homebridge Version
v1.8.4
Homebridge SwitchBot Plugin Version
4.62.0
Homebridge Config UI X Plugin Version
4.62.0
Operating System
Windows 10 64bit

Anyone? :/

Still looking for a solution :/

I’m seeing this too, I tried to clean up some extra accessories from my cache and now nothing works. I’ve also tried rolling back a few versions and the issue persists.

adding additional information: upgraded to version 4.1.0 after restoring my setup and it working properly on 3.8.2 and the bot became unresponsive again. No error messages in the logs, but that may be because it’s grouped with a bunch of other accessories. It seems like something breaks between 3.8.2 and later versions but it’s hard to tell what without a lot of testing.

Thanks! I've rolled back to v3.5.0 and its working now!

@doogfa can you change the label on this to bug? i’m going to try to get some extra logs to file a new issue but if it’s happening to both of us there is probably a bigger problem

I am also getting the issue with my curtains.

Same issue with some mini switch plugs.
11/29/2024, 11:00:47 PM] [SwitchBot] Plug Mini (US): Printer Device internal error due to device states not synchronized with server, or command format is invalid, statusCode: 190
[11/29/2024, 8:59:20 PM] [SwitchBot] Plug Mini (US): Office 2 Device internal error due to device states not synchronized with server, or command format is invalid, statusCode: 190
[11/29/2024, 8:59:20 PM] [SwitchBot] Plug Mini (US): Office 1 Device internal error due to device states not synchronized with server, or command format is invalid, statusCode: 190
Then I also see.
[12/1/2024, 6:48:33 AM] [SwitchBot] Failed to get device status: Connect Timeout Error (attempted addresses: <IP_ADDRESS>:443, <IP_ADDRESS>:443, <IP_ADDRESS>:443, timeout: 10000ms)
[12/1/2024, 6:48:33 AM] [SwitchBot] Error making request: Failed to get device status: Connect Timeout Error (attempted addresses: <IP_ADDRESS>:443, <IP_ADDRESS>:443, <IP_ADDRESS>:443, timeout: 10000ms)
[12/1/2024, 6:48:38 AM] [SwitchBot] Failed to get device status: Connect Timeout Error (attempted addresses: <IP_ADDRESS>:443, <IP_ADDRESS>:443, <IP_ADDRESS>:443, timeout: 10000ms)
[12/1/2024, 6:48:38 AM] [SwitchBot] Error making request: Failed to get device status: Connect Timeout Error (attempted addresses: <IP_ADDRESS>:443, <IP_ADDRESS>:443, <IP_ADDRESS>:443, timeout: 10000ms)

went to version 3.8.1 and got [12/2/2024, 8:58:16 AM] [SwitchBot] Requests reached the daily limit, statusCode: 190


Add support for onPermissionRequestResult callback

On lollipop, while requesting permission, we need to listen to onPermissionRequestResult - http://developer.android.com/training/permissions/requesting.html
I've yet to test this, but just wanted feedback on whether this approach is good or we want to use a different approach.
cc @mkonicek @kmagiera @foghina

cc @andreicoman11

This doesn't need to go through ReactContext or ReactInstanceManager.
We have a working version internally, namely a PermissionsModule that accesses an activity interface PermissionAwareActivity and passes a PermissionsListener (much like the one here) which gets the callback. We can probably open source that module at this point, we've been using it for a while now. What do you guys think? cc @foghina @mkonicek

@andreicoman11 That would be awesome. Right now we've no option, but to crash when permission is denied.

Yes let's open source the internal implementation. Thanks for bringing this up @satya164! Should we close this PR?

@mkonicek Yeah, sure :)

Hi folks,
I've made some experimental module to deal with this kind of permissions request of Android M:
https://github.com/lucasferreira/react-native-android-permissions
But you need to setup your targetSdkVersion to 23 to use them.
Best Regards ;)

@andreicoman11 Any updates on this? I'll love to use it in my modules, as well as update modules such as Geolocation to use this.

Finally got around to open sourceing this: https://github.com/facebook/react-native/commit/b7352b46671d09471d6aa9355ea41368ea05df96

That's awesome. Thanks @andreicoman11


Removed twice declaration AttrTypeByName

The interface AttrTypeByName was declared twice in attr_builder.h.

Nagging Reviewer @mhong: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 14 days with no activity and the awaiting review label has been applied.


Some scikit-learn estimators no longer work with array_function enabled

There's nothing dask-ml can do I think. Just filing this here till I can figure out the right thing to do.
In [1]: import dask.array as da

In [2]: import dask_ml.datasets

In [3]: X, y = dask_ml.datasets.make_classification(chunks=50)

In [4]: import sklearn.linear_model

In [5]: clf = sklearn.linear_model.LogisticRegression()

In [6]: clf.fit(X, y)
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-6-b0953fbb1d6e> in <module>
----> 1 clf.fit(X, y)

~/Envs/dask-dev/lib/python3.7/site-packages/sklearn/linear_model/logistic.py in fit(self, X, y, sample_weight)
   1536
   1537         multi_class = _check_multi_class(self.multi_class, solver,
-> 1538                                          len(self.classes_))
   1539
   1540         if solver == 'liblinear':

TypeError: 'float' object cannot be interpreted as an integer

Previously, that would have been converted to an ndarray by scikit-learn and fitted.

For the record, the error is from the fact that np.unique(da.Array) no longer converts to an ndarray
With array function disabled
In [1]: import numpy as np; import dask.array as da

In [2]: np.unique(da.arange(12))
Out[2]: array([ 0,  1,  2,  3,  4,  5,  6,  7,  8,  9, 10, 11])

With array function enabled
In [1]: import numpy as np; import dask.array as da

In [2]: np.unique(da.arange(12))
Out[2]: dask.array<getitem, shape=(nan,), dtype=int64, chunksize=(nan,), chunktype=numpy.ndarray>

The length of that array is unknown, leading to the nan shape, which breaks _check_multi_class.

The simplest fix, from dask-ml's  point of view, is for scikit-learn to convert that array-like to a concrete ndarray. But is that appropriate for all arrays? Is it appropriate for all estimators? Some estimators, like RandomForestClassfiers accept 2d output, which may be sparse. But the unique set should presumably be small.

The code snippet in original issue now works on sklearn 0.22 because of https://github.com/scikit-learn/scikit-learn/pull/14702


Nothing happens when sending over 10kb

Found there is a limit on max data size to emit and it is about 10kb. Any hints on how to fix it? Can contribute if it is easy enough issue for a WebRTC newbie.

could you give me/us some code?

Is this when you are sending a blob of data nested in a packet e.g. {stuff: {things: <blob of data>}}?

No, I mean just string. Sending HTML inside of blob just crashes a client.
2015-09-14 12:19 GMT+02:00 tomcartwrightuk<EMAIL_ADDRESS>
Is this when you are sending a blob of data nested in a packet e.g. {stuff:
{things: }}?
—
Reply to this email directly or view it on GitHub
https://github.com/socketio/socket.io-p2p/issues/12#issuecomment-140029125
.

Maximum string length workaround
function sendData(data, name) {
var chunkLength = 16000;
var chunksNumber = Math.ceil(data.length / chunkLength);
for (var i = 0; i < chunksNumber; i++) {
var chunkStart = i * chunkLength;
var chunkEnd = Math.min((i + 1) * chunkLength, data.length);
var chunk = data.substring(chunkStart, chunkEnd);
p2p.emit(name, chunk);
}
p2p.emit(name, 'end');
}

Looks like the sort of thing we should do - I will have a go at pushing it down into the parsing lib and do a release of that.

This is a major problem.
Hit it when trying to send an large JSON encoded string.  The decodeString method is trying to parse chunks instead of the entire message.  Ends up killing chrome in my case. (sending an object with a base64 dataUrl for an image i want to serialize).

BTW, sending the exact same message through a socket.io server doesn't have this problem.  Seems there's no cap on the size of the ws frame.

This is a WebRTC data channel limitation, not WS

Wouldn't socket.io-stream be a grate solution to this?
You can stream, pipe, pause, resume and use a high watermark

When sending files over 10kb, it gives the following error on console:

Uncaught TypeError: this._appendBuffer is not a function

Code is like below from https://github.com/tomcartwrightuk/socket.io-p2p-parser/blob/master/index.js#L374
BinaryReconstructor.prototype.takeBinaryData = function(binData) {
  this.buffers.push(binData);
  if (this.buffers.length == this.reconPack.attachments) { // done with buffer list
    this.reconPack.data['data'] = this.buffers.reduce(function(prev, curr, idx, arr) {
      return this._appendBuffer(prev, curr);
    })
    binary.reconstructPacket(this.reconPack, [this.reconPack.data['data']]);
    var packet = this.reconPack
    this.finishedReconstruction();
    return packet;
  }
  return null;

};

Problem is that "this" in the line "return this._appendBuffer(prev, curr);" corresponds to Window object. Since  there's no Window._appendBuffer, it prints the given error. Why is "this" defined incorrectly?
"this" is defined correctly in the first line of if statement and first line of BinaryReconstructor class.
How can we fix this?


"network_state" value in telemetry is string, should be json

network_state value in telemetry is string, it should be json, in "msg":"system.interval"
Need to check whether this applies to any other values in other messages.
eg:
"network_state":"{\"peerId\":\"QmaGMX8QCeeiq79YiHc7ndR2npwbk2HdcmFFkYWLBHmznu\",\"listenedAddresses\":[\"/ip4/<IP_ADDRESS>/tcp/30333\"],\"externalAddresses\":[\
...

Yeah, we just need to drop the serde_json::to_string.


Dreame A1

Good morning,
I tested with the A1 robot lawn mower and I managed to retrieve the status and battery level.
How could I retrieve the correct commands (Start, Stop, ...)?
thank you

need the account with the device in
<EMAIL_ADDRESS>

Hello, I can't find how to launch a specific zone, any ideas?
THANKS
Nicolas
Le jeu. 18 juil. 2024 à 12:40, TA2k @.***> a écrit :

need the account with the device in
@.***
—
Reply to this email directly, view it on GitHub
https://github.com/TA2k/ioBroker.dreame/issues/1#issuecomment-2236186517,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL4FNXIH6ACOZKFHAHBP4ALZM6LSPAVCNFSM6AAAAABLCFSFFGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZWGE4DMNJRG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

GitHub version has customCommand
the default value is also the command you need to send for zone 1 you can adjust the zone


implement item remove mechanism

reasons for removal
-> after usage or after time or after x usages

Already implemented and merged in #130


Add script to disable all mentioned telemetry

something like the below script:
https://gist.github.com/sneak/c6845e95a9308a65ea3d29422f5a92ac
just more complete, which include all programs described in this repo.

I actually plan to release a cli tool that would do exactly that: run on your system and configure all known telemetry channels. Meanwhile, it's pretty easy to generate shell scripts using existing data so I might as well do this sometime soon.


Implements #537 Add support for Monaco diff editor

Closes #537

Two new components: pe:monacoDiffEditor/ and <pe:monacoDiffEditorFramed />
The component API (XHTML properties) and client-side API (widget methods) are mostly identical to the code (non-diff) editor. There are a few new properties to control the behavior of the original editor (the one to the left of the diff editor), such as "originalDisabled" or "PrimeFaces.widget.monacoDiffEditor.getOriginalValue()"
The "value" property must be an instance of "MonacoDiffEditorModel", which is essentially an ordered pair of two strings.
For each showcase example, there's now a new example with the diff editor.
Fixed a bug with the generation of locales, some i18n keys were missing.
Some other minor bugs and inconsistencies were fixed, corrected some spelling errors and dead links.

I tested it against the showcase, but would be great if somebody could run the showcase too and see if you spot any errors.

I will test tomorrow when I am in the office

I did a fresh pull of your branch and get this error on mvn clean install
 --- frontend-maven-plugin:1.12.1:npm (generate-locales) @ resources-monacoeditor ---
[INFO] Running 'npm run generate-locales' in C:\Temp\monaco-diff\monacoeditor\src\main\js
[INFO]
[INFO] ><EMAIL_ADDRESS>generate-locales
[INFO] > node src/scripts/generateLocales.js
[INFO]
[INFO] C:\Temp\monaco-diff\monacoeditor\src\main\js\node_modules\mkdirp\lib\path-arg.js:20
[INFO]       throw Object.assign(new Error('Illegal characters in path.'), {
[INFO]                           ^
[INFO]
[INFO] Error: Illegal characters in path.
[INFO]     at pathArg (C:\Temp\monaco-diff\monacoeditor\src\main\js\node_modules\mkdirp\lib\path-arg.js:20:27)
[INFO]     at Function.mkdirp.native (C:\Temp\monaco-diff\monacoeditor\src\main\js\node_modules\mkdirp\index.js:26:46)
[INFO]     at main (file:///C:/Temp/monaco-diff/monacoeditor/src/main/js/src/scripts/generateLocales.js:266:18)
[INFO]     at file:///C:/Temp/monaco-diff/monacoeditor/src/main/js/src/scripts/generateLocales.js:296:1
[INFO]     at ModuleJob.run (node:internal/modules/esm/module_job:185:25)
[INFO]     at async Promise.all (index 0)
[INFO]     at async ESMLoader.import (node:internal/modules/esm/loader:281:24)
[INFO]     at async loadESM (node:internal/process/esm_loader:88:5)
[INFO]     at async handleMainPromise (node:internal/modules/run_main:65:12) {
[INFO]   path: 'C:\\C:\\Temp\\monaco-diff\\monacoeditor\\target\\git',
[INFO]   code: 'EINVAL'
[INFO] }

[INFO]   path: 'C:\C:\Temp\monaco-diff\monacoeditor\target\git',

The C:\\C:\\ is probably the issue. Not sure where that comes from, may have to try it on a windows system

Alright, hope it's fixed, if you could try again
ps, I wonder if the gitlab workflow could do without running the PrimeFaces tests (-DskipTests / -DskipITs)? Since I suppose we only want to check if it builds against the current snapshot? https://github.com/primefaces-extensions/primefaces-extensions/blob/master/.github/workflows/build.yml#L32

works!

Is it safe for me to merge this?

Yeah if it compiles for you too and you can run the showcase, it's fine now.

I do have one question though from looking at your code.  So if you pass the Monaco Editor 2 strings it handles doing all the differencing and display in the JS code ?  If so that is AWESOME!  Let me know.

Exactly, you pass <p:monacoDiffEditor value="bean.value" /> a MonacoDiffModel, which is just a POJO with two strings. Rest is done by Monaco editor.

amazing!


Enormous memory consumption with large BAM file

Hi,
I'm trying to launch a stringTie assembly on a rather large BAM file (81G), and the call systematically ends with an 'error allocating memory'. I monitored it and saw that it used up to 128 GB of ram (it keeps increasing, and decreases only by tiny amounts from time to time). I tried reducing the number of threads, and using the default -s 1000000, but I get into the same problem. This is puzzling since loci are supposed to be processed one by one and I cannot understand how loading a maximum of 1000000 read pairs, even multithreading, could ever use so much memory. Is this somehow expected?
The reads have been aligned to a spiked-in human genome using HISAT, and I'm using stringTie -G with the refseq annotation. (The fact that it's spiked-in means that technically there are a bit more than 100 chromosomes, if that's of any relevance.)
(I've tried compiling and running the v1.0.4 software on two different linux x86 clusters.)
Thanks in advance,
Pierre-Luc

Hi Pierre-Luc,
We are working on a version of StringTie to reduce the large memory footprint when spliced reads artificially join distinct bundles together. Unfortunately HISAT has a tendency to predict many such splices reads, which results in a high memory usage. Due to this issue, at this point we don't recommend using StringTie with HISAT. Please use TopHat until we finish fixing this problem.

Thanks a lot for the hint!

Hi,
do you think that STAR could be used for this purpose ?

STAR poses the same problems as HISAT unfortunately. We will have a fix for this soon.

I recently released HISAT2, which is a successor to both HISAT and TopHat2.  Please use HISAT2 with an option, --dta, which should work well with StringTie.  HISAT2 is faster and much more memory efficient than STAR and provides better alignment quality.  The HISAT2 website is at https://ccb.jhu.edu/software/hisat2

We just released version 1.1.0 of StringTie which now runs using less than 1Gb memory for almost all samples.


containers: add note for lmp-device-register

Signed-off-by: Ricardo Salveti<EMAIL_ADDRESS>

Docs for f8111ddcbbfcafd62049e73b4adbf64a1df0eaa8 are browesable at: https://ci.foundries.io/projects/fio-docs/builds/324/docs/artifacts/html/index.html


@Sendable with function type defined as a typealias doesn't compile

Describe the bug
Adding the @Sendable attribute to a function type defined as a typealias doesn't compile.
Steps To Reproduce
typealias Callback = () -> Void

func foo(callback: @Sendable Callback) {} // error: @Sendable attribute only applies to function types
func bar(callback: @Sendable () -> Void) {} // no error

Expected behavior
foo(callback:) should compile just as bar(callback:) does.
Screenshots

Environment (please fill out the following information)

OS: 13.0 Beta (22A5311f)
Xcode: Version 14.0 beta 5 (14A5294e)

Also tested with docker swift:5.6.2-focal

Re-opening because it was accidentally closed by Doug's PR and it's not fixed on main (but my PR should do it).

Just checked with 5.8 and latest nightly main and the issue remains.

@hborla Sorry for direct pinging, but on Swift forums you asked for feedback on issues with adopting Swift concurrency adoption, and I just hit this. We have completion callbacks defined as typealiases and this bug gets in the way when I try to add an @Sendable.
(I hope this comment was appropriate. It felt weirder to open a forum thread about this, than @ here).


The error message and the if condition do not agree.

The if condition requires both elements to be non null, while the error message requires one of them.
Also the error message does not help much when debugging IMHO.
https://github.com/jetstack/cert-manager/blob/e51edb398eeba82b83b5e64301342fecd27b5d51/pkg/controller/certificates/sync.go#L80

If you check the implementation of CommonNameForCertificate and DNSNamesForCertificate, you should notice that if either a CN or at least one dnsName is set, both fields will be populated.
An example:
if you set the common name to example.com, but no dns names (on your Certificate resource):

expectedCN = example.com
expectedDNSNames = []string{example.com}

if you do not set the common name, but set the dnsNames field to "example.com, foo.com":

expectedCN = example.com
expectedDNSNames = []string{example.com, foo.com}

certificate must specify at least one of dnsNames or commonName is meant to indicate that a user has not specified any dns names or a common name on their certificate. If you've got an improvement for the message, I'd love to hear your thoughts and would be happy to create (or even better, accept!) a PR 😄


Convert JSON parsing to json4s

Addresses #130.

I would recommend to use jsoniter-scala instead.
It is handy, safe and the most efficient JSON parser in the JVM world.

I would recommend to use jsoniter-scala instead.
It is handy, safe and the most efficient JSON parser in the JVM world.

I'm pretty far into this already, but is your tool able to manually parse JSON?  I need to be able to do that for debugging and for tracking progress in some cases.  If so, how does it compare in terms of code readability?  json4s's (json \ "key").extract[Type] is pretty convenient.  Performance in reading JSON isn't as important to me here, as it only has to happen at human speeds and the various places where the UI could stall aren't where it's parsing JSON.


Equip Fairdrive Desktop App with MyData operator Award Logo

Is your feature request related to a problem? Please describe.
No response
Describe the solution you'd like
FDA was awarded the MyData operator 2023 award and can as such sport a MyData operator Award logo.
The logos can be found in various formats in: https://drive.google.com/drive/folders/1ztFXVWhmjXJ01VcTqi1SMn_8gje4k6LO?usp=share_link
Please equip the App itself with logo you find appropriate in appropriate size and location.
Please equip also the page with instructions with appropriate variant of the logo.
The logo if clicked, should lead to: https://www.mydata.org/participate/awards/
The page explains about the awards.
And congratulations!
Describe alternatives you've considered
No response
Additional context
No response

Added the white logo in about modal. @crtahlin @tfius


SoundSpace

sound-spaces
Project: RLR-Audio-Propagation
Audio Sensor

CVPR 2022
Audio-Adaptive Activity Recognition Across Video Domains
Wnet: Audio-Guided Video Semantic Segmentation via Wavelet-Based Cross-Modal Denoising Networks
Finding Fallen Objects Via Asynchronous Audio-Visual Integration
Audio-Visual Speech Codecs: Rethinking Audio-Visual Speech Enhancement by Re-Synthesis
Egocentric Deep Multi-Channel Audio-Visual Active Speaker Localization
Learning to Answer Questions in Dynamic Audio-Visual Scenarios
Audio-visual Generalised Zero-shot Learning with Cross-modal Attention and Language
Self-supervised object detection from audio-visual correspondence


Fix typo: "use" -> "used"

Fix a typo in the doc comment of the Metadata type.

Thanks!
@bors r+

:pushpin: Commit 2aeb5b5da31fc549f7f43b9e3dcaeb85110e4dc7 has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit 2aeb5b5da31fc549f7f43b9e3dcaeb85110e4dc7 with merge b28be289136aa346572514aeab07df74b9508d2b...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing b28be289136aa346572514aeab07df74b9508d2b to master...


set color with Row#color=

Hi
I think it's convenient if we can set color of row at once.
With this method, we can emphasize special row without knowing a bit complex style option.
Something like ....
  p.workbook.add_worksheet(:name => "Table") do |sheet|
    people.each do |person|
      row = sheet.add_row([person.name, person.age])
      row.color="00FFFF00" if person.salary > 100_000_000
    end
  end

Would you be kind enough to please rebase and push?


Engdocs 633

New user docs for Extensions.

@aevesdocker Thanks for the PR. Added some minor suggestions. Looks good overall.


External login, language param

Hello,
It is possible to pass language to external login form? As I see in examples issues 154 "ExternalLogin" form accept only email and password. It would be a nice feature if form has language param.

It would be a nice feature if form has language param.

It has. (not documented)
$ssoHash = \RainLoop\Api::GetUserSsoHash(
<EMAIL_ADDRESS>    'super-puper-password',
    array('Language' => 'ru_RU')
);

Thanks, but it looks like it works only for local SSO. External SSO pass only two params:
public function ServiceExternalSso()
....
$sResult = \RainLoop\Api::GetUserSsoHash($sEmail, $sPassword);
index.php?ExternalLogin form seems also do not accept lang param.(Or maybe i missed something)
So is there any way to pass lang via external POST request?

Thanks, but it looks like it works only for local SSO.

Yes you are right. So far, this is not possible right now.


FIX: Changed the text 'quot;' to an actual Quotation.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: oathgg sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Docs Build status updates of commit 2eabe00:
:clock10: Full build: the average full build time is 6 min(s) 34 sec(s), this is based on the last 7 build(s) for this repository.
:warning: This is a FULL build due to a recent Docs Build service upgrade, normally no action is required on your part. In the result you find subsequent builds still performing full builds, please contact your repository admin to manually perform a full build on the 'live' branch. For more detailed info, please click here.

Docs Build status updates of commit 2eabe00:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamicsax2012-msdn/how-to-modify-parameter-values-in-a-pre-method-event-handler.md
:bulb:Suggestion
View
Details

dynamicsax2012-msdn/how-to-modify-parameter-values-in-a-pre-method-event-handler.md

[Suggestion] Missing attribute: author. Add the current author's GitHub ID. NOTE: This Suggestion will be elevated to a Warning on 9/1/2019.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@Khairunj Do I still have to do something?


Exponential Moving Average of Model Parameters

Ref: https://www.tensorflow.org/api_docs/python/tf/train/ExponentialMovingAverage
Having a class that expresses the concept of moving average is likely useful for many models. We can think about how to express the weighing for more diverse use cases.

https://github.com/dmlc/gluon-nlp/blob/qanet/scripts/question_answering/util.py#L74


Plugin creates a dart file in every opened Flutter project

The plugin also creates a "widget.preview.dart" file in Flutter projects that doesn't use this package

Fixed in v2022.1.5. Update has been released, will be available in a couple days probably.


Delete Unused import

Delete Unused import
Simplify assertion
Fix javadoc
Remove Unused assignment

Coverage decreased (-0.03%) to 84.106% when pulling f80d26db7dec86d899a94c80cc97ad24e6f87940 on arturobernalg:feature/improvements into ffbcdc03dea137eb09a647cc1e22a66039c0d967 on apache:master.

@arturobernalg
May you please rebase on master?

HI @garydgregory
Done
TY


add/update README file in input data folder - version explanation

Explain the versioning of input data, maybe add metadata description.
https://ns2806k.webs.sigma2.no/EMERALD/EMERALD_platform/inputdata_fates_platform/

New location for input data: https://ns2806k.webs.sigma2.no/EMERALD/EMERALD_platform/inputdata_noresm_landsites/v1.0.0/default/
@lasseke could you add a readme file in there with creation date and ref to the repository/scripts or something?


[boot/sign]zip: Fix some details

@AdrianDC These changes don't follow your code style and not need to be merged, so if you want cherry-pick then or just pick-up the code :)


bidirectional_rnn Error

Hi,  the version of my tensorlfow is 1.2.0 . when I run deepsleepnet, there is a error:
File "/home/zjm/deepsleepnet/deepsleep/model.py", line 340, in build_model
outputs, fw_state, bw_state = tf.nn.bidirectional_rnn(
AttributeError: 'module' object has no attribute 'bidirectional_rnn'
when I change  tf.nn.bidirectional_rnn  into tf.nn.bidirectional_dynamic_rnn, the error is:
File "/home/zjm/deepsleepnet/deepsleep/model.py", line 395, in init_ops
network = self.build_model(input_var=self.input_var)
File "/home/zjm/deepsleepnet/deepsleep/model.py", line 347, in build_model
initial_state_bw=self.bw_initial_state
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/rnn.py", line 375, in bidirectional_dynamic_rnn
time_major=time_major, scope=fw_scope)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/rnn.py", line 574, in dynamic_rnn
dtype=dtype)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/rnn.py", line 637, in dynamic_rnn_loop
for input in flat_input)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/rnn.py", line 637, in
for input_ in flat_input)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/tensor_shape.py", line 649, in with_rank_at_least
raise ValueError("Shape %s must have rank at least %d" % (self, rank))
ValueError: Shape (3072, 10) must have rank at least 3.
How can I fix bidirectional_rnn Error ?
thanks

I think Tensorflow 1.2.0 change the name of the function to something ealse. If you wanna use 1.2.0, you need to change several function names to make it run.
Our code is implemented based on Tensorflow 0.12.1


[docs beta] "Scrolling to an element" example does not work on Safari

Checking the beta docs, I noticed that one of the examples was not working. At first it was not working at all, but after I added a style of overflow-x: auto to the <ul> element, then I could see the desired effect after clicking on the buttons. I could later check that the example works as expected (without any changes in Chrome). My first guess is that Safari doesn't scroll because the element has an implicit overflow as visible.
Please note that this issue is also present on the challenge 3, as it basically uses that same principle to scroll the images.
Steps

Open https://beta.reactjs.org/learn/manipulating-the-dom-with-refs#example-scrolling-to-an-element on Safari
Click on the buttons

The culprit is the scrollIntoView(). The method is not fully supported by Safari. You can make it work partially by changing behavior: 'smooth' to behavior: 'auto' as smooth is not currently supported by Safari. [0]
[0] https://caniuse.com/?search=scrollIntoView


About Camera Pose Control

Hi, Thanks for open-sourcing this great work! I wonder if we can apply camera control onto this model to enable arbitrary camera trajectory control to make videos.

That sounds like a great idea, but we haven’t tried it with this model yet. Currently, it remains a more general video generation model.


[Macros] Ability to opt-out from auto-propagation of attrs/modifiers

DeclarationMacro now have two optional requirements that is defaulted to 'true':
static var propagateFreestandingMacroAttributes: Bool { get }
static var propagateFreestandingMacroModifiers: Bool { get }
My implementing these and returning 'false', macro expansion systam won't copy attributes and/or modifiers to the expanded declarations.
rdar://110364154
I will add a test case with actual plugins in swift repo.

@swift-ci Please test


Test data for csv aggregation

Accounts for samples with or without sampleid column, if sampleid column not present the script will take the sampleid value from the stem of the filename (1.tif -> 1)
Created a new folder because the script expects a folders with all present CSVs to have a similar format, all other test dataset folders had markers.csv

Suggestion - exclude 2 of the input files. Subset data to be smaller.
Question, what is the Phenotyped and how is it different to Miguel's and Chiara's recently merged Phenotyped.csv?


[1082] remove the equipment item index page

Resolves #1082

link in navbar?

Remove equipment tab from the user navbar
Remove all other links to equipment_items_path, scoping to the model or request_back when appropriate

Note that in removing the equipment items links I changed the 'equipment' link for checkout persons to link to the equipment models index and also adjusted the view permissions such that it gives them a little bit more useful information

Looks good, the only change I'd make is replacing the "View all" link on the bottom of the resource views with a link to the nested equipment item index for the item's equipment model if we have it. Otherwise feel free to squash / merge.

yes

SQUASH IT.

:boot:


Help

How Can I access to JavaScript Variable in Symfony 3 Eric GELOEN
setJavascriptVariable() method does not exit help please

@zabaworld1994 You need to rely on setVariable instead.

thank you sir for you fast reply please can you help me  more i  did setVariable("mymap") in then controller
now please how can i get the google map objet in the twig view to use javascript code

Just pass it to the view as you would pass any template variables and then use the helpers according to you templating framework (Twig or PHP) (as documented here).

thank you Issue resolved problem was using document.ready fires too early
using window.load everything worked great


WIP: Dagster EPA CEMS Prototype

This PR contains a Dagster EPA CEMS prototype.
#1406 contains our reasoning for exploring Dagster. #1432 contain our goals for the prototype.
README
Setup
Recreate your pudl-dev environment to install dagster and dagit.
To start the dagster UI (dagit), run this command from the pudl root directory:
dagit -m pudl.dagster_etl

Then start the dagster-daemon in a separate terminal window:
dagster-daemon run

Run the ETL
Dagit should be running at http://<IP_ADDRESS>:3000/. You will see a window with three tabs, Overview, Launchpad, and Runs. The Overview tab contains a DAG of our CEMS pipeline. You can click on the different nodes to explore information about the op.
To process all of Idaho's data, go to the Launchpad tab. The Launchpad tab contains a little text editor to specify a run config. It's kind of like our settings yaml files. Copy and paste this config into the window:
ops:
  gather_partitions:
    config:
      states:
        - ID

Click, Launch Run in the very bottom right corner of the screen. That should kick off a run and bring you to a window with a fun gantt chart and logging information. A green circle should say "Success" at the top of the screen when the run finishes. The run will take a couple of minutes (I will write up my findings on performance).
Once the run finishes, you can go to the Asset window in the upper right-hand corner of the screen. Click on the hourly_emissions_epacems asset. There should be a list of processed partitions. You can click on each partition to view its metadata, like the number of rows and location.
Findings
If you review the code or run the ETL, feel free to add your findings to #1432.

I was able to run the prototype ETL as you described. I think something's wrong with how the dependencies are specified though -- it did all of the extraction before starting any of the transformation, and then it did all of the transformation before starting any of the loading.  So it's not surprising that it would be unable to do the full dataset. Rather than having all load depend on all transform, and all transform depend on all extract, we want to define longitudinal dependencies, in which there are dependencies between the different steps within each partition, but there are no dependencies between partitions.
It seems like the right way to do that might be to define a reusable @graph that connects the E-T-L steps for a given partition, and then hand that graph the set of partitions to work on?

Ah good point, I think it makes more sense to define a graph and feed it partitions.

It took about 6 minutes for my run of all years of the ID data to complete. In the gantt chart it looks like there's a pretty substantial fixed overhead associated with each partition (the light-blue section before the green computation?). It would be good to understand what's happening inside there, and if they can be shortened. We've also talked about potentially just chunking up the CEMS processing by year (since every year is about the same amount of data, and tractable) and then using record groups inside the Parquet files to allow individual selection of state/year combinations without requiring the whole file to be scanned.

We have decided to move forward with Dagster. See epic #1487 for the full reasoning.
We will not merge in this PR but wil keep the branch around for reference.


Fix message if no files found

Description of the Change
Previously would print a warning for each missing file/dir, then an error if no valid files were found.
Now it just prints an error for each missing file/dir (so we dont have to print a final error). Will still exit if no valid files were found.
UPDATE:
Im not sure if errors are right for most of these situations, if we only want errors to show if the test is not going to run then this is the wrong approach.
Will look at using warning and changing to errors only if nothing will run.
Scenarios:
Nothing given
Command: mocha
New output:
✖ Cannot find any files matching pattern "test/"

Previous output:
Warning: cannot find any files matching pattern "test/": 'test/'
✖ No test files found

Difference = No warning and slimmer message
Nothing found
Command: mocha -R json-stream --no-config "test/integration/fixtures/glob/**/*-none.js"
New output:
✖ Cannot find any files matching pattern "test/integration/fixtures/glob/**/*-none.js"

Previous output:
Warning: cannot find any files matching pattern "test/integration/fixtures/glob/**/*-none.js": 'test/integration/fixtures/glob/**/*-none.js'
✖ No test files found

Difference = No warning and slimmer message
1 found but 1 not found:
Command: mocha -R json-stream --no-config "test/integration/fixtures/glob/**/*.js" "test/integration/fixtures/glob/**/*-none.js"
New output:
✖ Cannot find any files matching pattern "test/integration/fixtures/glob/**/*-none.js"
["start",{"total":2}]
["pass",{"title":"should find this test","fullTitle":"globbing test should find this test","duration":0,"currentRetry":0}]
["pass",{"title":"should find this test","fullTitle":"globbing test should find this test","duration":0,"currentRetry":0}]
["end",{"suites":2,"tests":2,"passes":2,"pending":0,"failures":0,"start":"2019-01-03T16:57:02.725Z","end":"2019-01-03T16:57:02.727Z","duration":2}]

Previous output:
Warning: cannot find any files matching pattern "test/integration/fixtures/glob/**/*-none.js": 'test/integration/fixtures/glob/**/*-none.js'
["start",{"total":2}]
["pass",{"title":"should find this test","fullTitle":"globbing test should find this test","duration":0,"currentRetry":0}]
["pass",{"title":"should find this test","fullTitle":"globbing test should find this test","duration":0,"currentRetry":0}]
["end",{"suites":2,"tests":2,"passes":2,"pending":0,"failures":0,"start":"2019-01-03T17:53:56.843Z","end":"2019-01-03T17:53:56.845Z","duration":2}]

Difference = warning replaced with slimmer error
1 found but 2 not found:
Command: mocha -R json-stream --no-config "test/integration/fixtures/glob/**/*.js" "test/integration/fixtures/glob/**/*-none.js" "test/integration/fixtures/glob/**/*-none-again.js"
New output:
✖ Cannot find any files matching pattern "test/integration/fixtures/glob/**/*-none.js"
✖ Cannot find any files matching pattern "test/integration/fixtures/glob/**/*-none-again.js"
["start",{"total":2}]
["pass",{"title":"should find this test","fullTitle":"globbing test should find this test","duration":0,"currentRetry":0}]
["pass",{"title":"should find this test","fullTitle":"globbing test should find this test","duration":0,"currentRetry":0}]
["end",{"suites":2,"tests":2,"passes":2,"pending":0,"failures":0,"start":"2019-01-03T16:57:02.725Z","end":"2019-01-03T16:57:02.727Z","duration":2}]

Previous output:
Warning: cannot find any files matching pattern "test/integration/fixtures/glob/**/*-none.js": 'test/integration/fixtures/glob/**/*-none.js'
Warning: cannot find any files matching pattern "test/integration/fixtures/glob/**/*-none-blah.js": 'test/integration/fixtures/glob/**/*-none-blah.js'
["start",{"total":2}]
["pass",{"title":"should find this test","fullTitle":"globbing test should find this test","duration":0,"currentRetry":0}]
["pass",{"title":"should find this test","fullTitle":"globbing test should find this test","duration":0,"currentRetry":0}]
["end",{"suites":2,"tests":2,"passes":2,"pending":0,"failures":0,"start":"2019-01-03T17:54:34.393Z","end":"2019-01-03T17:54:34.395Z","duration":2}]

Difference = warnings replaced with slimmer errors
1 found:
Command: mocha -R json-stream --no-config "test/integration/fixtures/glob/**/*.js"
New output:
["start",{"total":2}]
["pass",{"title":"should find this test","fullTitle":"globbing test should find this test","duration":0,"currentRetry":0}]
["pass",{"title":"should find this test","fullTitle":"globbing test should find this test","duration":0,"currentRetry":0}]
["end",{"suites":2,"tests":2,"passes":2,"pending":0,"failures":0,"start":"2019-01-03T17:01:17.934Z","end":"2019-01-03T17:01:17.938Z","duration":4}]

Previous output:
["start",{"total":2}]
["pass",{"title":"should find this test","fullTitle":"globbing test should find this test","duration":0,"currentRetry":0}]
["pass",{"title":"should find this test","fullTitle":"globbing test should find this test","duration":0,"currentRetry":0}]
["end",{"suites":2,"tests":2,"passes":2,"pending":0,"failures":0,"start":"2019-01-03T17:55:06.996Z","end":"2019-01-03T17:55:06.999Z","duration":3}]

Difference = nothing
Applicable issues
Fixes #3646
(semver-minor)

Coverage decreased (-0.003%) to 90.726% when pulling 964f3638e0e8df6cb4e07c21bfb607f61941ceeb on craigtaub/fixNoisyMessage into 1f325e746033f698c6a040329a2fd7670daaf274 on master.


feat: sign goreleaser release using keyless cosign

Summary

Tested via test repo: https://github.com/floreks/goreleaser-cosign-test
It uses keyless signing with GitHub oidc issuer.
I have used a makefile target that can verify the file signature.
$ make verify
Enter version to verify: 1.0.1
Downloading checksums.txt for tag v1.0.1...
Verifying signature...
tlog entry verified with uuid: 0c626127c3f863670ff3f6cb4faa73358c68fd1724ad5ae87ee6b5018bb262a0 index: 132147841
Verified OK
Verifying archives...
goreleaser-cosign-test_1.0.1_Darwin_amd64.tar.gz: OK
goreleaser-cosign-test_1.0.1_Darwin_arm64.tar.gz: OK
goreleaser-cosign-test_1.0.1_Linux_amd64.tar.gz: OK
goreleaser-cosign-test_1.0.1_Linux_arm64.tar.gz: OK
goreleaser-cosign-test_1.0.1_Windows_amd64.tar.gz: OK

Labels

Test Plan

Checklist

[ ] If required, I have updated the Plural documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] I have added a meaningful title and summary to convey the impact of this PR to a user.
[ ] I have added relevant labels to this PR to help with categorization for release notes.

@floreks can we just create our own keypair and sign w/ those?  I feel like that's the only way to be safe that it can be used by enterprises that need cosign validation given limitations in their ecosystem

Sure, you'd need to add a key secret to the repo secrets for GH action to use it since I don't have permissions to add secrets. You should be able to generate it with cosign.

@michaeljguarino I can access the settings view now but it's almost empty. Only Pages and Moderation Options are available.

Ok I don't see any way for us to give you those perms w/ less than admin, so just generated the keypair for you, they're in COSIGN_PRIVATE_KEY and COSIGN_PASSWORD secrets.  Also the cosign.pub file at the root of the repo will be the public key.


[Feature request] View white space

Add an option to show white space.

You do not care and so do i. /Closed


Can't revisit tabs that were visited before initializing the Scrolling Tabs

Any tabs that were visited before initializing the Scrolling Tabs can't be opened after calling the plugin. Here is a Plunk that is showing the issue. Just click on a different tab and try to go back to "Tab Number 2".
As you can see in the JS file, I called click() before calling the plugin, but the same issue is happening if you call Bootstrap's tab('show') method. We actually run into the issue by calling the tab('show') method in order to show the correct tab depending on the URL. The reason why we cannot move the initialization of the plugin before our tab('show') call is because all the HTML elements are hidden and we only make them visible after the correct tab is selected. On the other side, this plugin expects the Tabs to be visible in order to calculate the needed widths.
The reasons why this is happening is combination of these two:

On the first activation of a tab, Bootstrap stores the pointer to the Anchor element in a data property. It then reuses the same element on the next click.

This plugin clones the tab elements while initializing

So when you try to navigate back to the "Tab Number 2", Bootstrap tries to activate the element it stored in the data property, which doesn't actually exist on the HTML page anymore.
I really like your library, so we kept using it and fixed this issue by clearing out the specific data property that Bootstrap stores (it's the same for Bootstrap 3 and 4) after the tab elements are cloned. It still works because next time the tab is clicked, Bootstrap will store the correct element, which is exactly what happens for the other tabs that weren't visited before this plugin was initialized.
@mikejacobson let me know what you think and I can send a PR with our fix.

Wow, your debugging and fix are very impressive. It's somewhat of an edge case but I would love to see your PR with the fix. Please go ahead and submit it when you get a chance. Thanks for using my plugin!


Mvp

Fixes https://github.com/vaadin/components-team-tasks/issues/228

This change is

Review status: 15 of 16 files reviewed at latest revision, 11 unresolved discussions, some commit checks failed.

Comments from Reviewable

Review status: 15 of 16 files reviewed at latest revision, 11 unresolved discussions, some commit checks failed.

vaadin-input.html, line 47 at r1 (raw file):
Previously, platosha (Anton Platonov) wrote…
Should we turn all the native attributes to use attributes? paper-input does that even for name, list, pattern and placeholder and title.

I'd rather not having those values in the dom if not absolutly necessary.

vaadin-input.html, line 233 at r2 (raw file):
Previously, manolo (Manuel Carrasco Moñino) wrote…
As discussed in chat, apparently is not needed right now although iron-input does it. I will remove it until we observe a related issue.

Done.

wct.conf.js, line 10 at r3 (raw file):
Previously, platosha (Anton Platonov) wrote…
Should we test on iPad as well?

Done.

test/validate.html, line 16 at r3 (raw file):
Previously, tomivirkki (Tomi Virkki) wrote…
Might make sense

Done.

Comments from Reviewable

Review status: 11 of 16 files reviewed at latest revision, 5 unresolved discussions.

vaadin-input.html, line 173 at r5 (raw file):

             * https://developer.mozilla.org/en/docs/Web/HTML/Element/input#attr-type
             */
            type: {

We shouldn't have type at this point at least.

Comments from Reviewable

Review status: 11 of 16 files reviewed at latest revision, 5 unresolved discussions.

vaadin-input.html, line 173 at r5 (raw file):
Previously, Saulis (Sauli Tähkäpää) wrote…
We shouldn't have type at this point at least.

How do you define a password input then ?

Comments from Reviewable

Reviewed 1 of 10 files at r2, 1 of 1 files at r4, 2 of 3 files at r5, 4 of 5 files at r6.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

Comments from Reviewable

Review status: all files reviewed at latest revision, 2 unresolved discussions.

vaadin-input.html, line 173 at r5 (raw file):
Previously, manolo (Manuel Carrasco Moñino) wrote…
How do you define a password input then ?

That question will be answered when we will start working on the "custom validators" and "prevent invalid input" tickets. What we do know for sure is that we don't want people to expect that something like type="checkbox" or type="button" would work, hence it's better to remove the property for now.

Comments from Reviewable

Review status: 13 of 16 files reviewed at latest revision, 2 unresolved discussions.

vaadin-input.html, line 173 at r5 (raw file):
Previously, Saulis (Sauli Tähkäpää) wrote…
That question will be answered when we will start working on the "custom validators" and "prevent invalid input" tickets. What we do know for sure is that we don't want people to expect that something like type="checkbox" or type="button" would work, hence it's better to remove the property for now.

Thinking more about this, I feel like we could have  a common vaadin-native-input or whatever name,  with the type and all common stuff, then  all other vaadin form controls could wrap or extend it exposing only the native input api that we want for it.  For instance for vaadin-input we support text/password/url/email ... via attributes or something.
Anyway removing the type for now.
Done.

Comments from Reviewable

Reviewed 7 of 18 files at r1, 4 of 10 files at r2, 1 of 1 files at r4, 1 of 3 files at r5, 3 of 5 files at r6, 3 of 3 files at r7.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks broke.

Comments from Reviewable


should we support Base.ReinterpretArray when casting Julia arrays to Python?

https://github.com/Suzhou-Tongyuan/jnumpy/blob/65cea6cc6a750911ab2c1545d94af26008e7b240/TyPython/src/CPython.Julia.jl#L80

I think it's fine when x is a Base.ReinterpretArray and StrideArray. Also need to check the type of parent(x), make sure it's a Array would be good.

For now, we support no-copy casting of ReinterpretArray when the condition below is true:
https://github.com/Suzhou-Tongyuan/jnumpy/blob/1d7e2007abb10fcc29cb4efb0e190c4829bd40f6/TyPython/src/CPython.Julia.jl#L80


Grid areas don't appear to always update during hot reload

Not entirely sure, but without a deep investigation it seems that way

Turns out the GridParentData's row/col information wasn't being updated every time it needed to be.
Now it always is before laying out. Tiny bit slower, way safer.


Facing issue reconnecting to video call with new token but same identity

I'm facing a similar issue to this. But no concrete solution was found. Looking forward to getting a reply if this is an issue or am I
doing something wrong here?
https://github.com/twilio/twilio-video.js/issues/1048
Thanks

Hi @him2016 . Thank you for reaching out regarding this issue. Can you please fill out the issue template? This will help us resolve your issue faster by providing specific reproduction steps. Thanks!

Steps for reproduction-

Connect a client to a room using auth token with identity XYZ (Client is connected and no issue)
Client disconnects the call
Client again tries to connect using a new token created for the same identity XYZ (Client gets connected and immediately disconnected throwing this exception TwilioError: Participant disconnected because of duplicate identity)

Hi @him2016 . Thank you for the reproduction steps. I have verified that this issue does not occur in this app. Are you facing this issue in this app or in your own?

I found this issue when changed the provided implementation from passcode to token generated on the server. I guess it is due to the singleton object of RoomManager which is created using Dagger. Any thoughts will be helpful.

Hi @him2016 . Do you mind sharing your code that swaps out the passcode logic and a stack trace if it is available? Otherwise It will be hard for us to troubleshoot your exact problem. Also, we swap out the passcode solution with another token generated solution in the internal flavor of the application. You can see how it's done by taking a look at this class. Just note that the internal flavor only works for internal Twilio employees, but it can help show you how the passcode solution can be properly swapped out.

Ok i'll go through the internal flavor application code. If still any issue is found, will share the code snippet.. Thanks.

Hi,
I just went through the code in the shared file. The authToken token implementation is similar. The issue I am getting is inside the RoomManager class. As soon as I try to connect with a new token created using the same identity, the 'onDisconnected' is called with this exception:
com.twilio.video.TwilioException: Participant disconnected because of duplicate identity

Hi @him2016 . Thanks for taking a look at that code. This looks like expected behavior from the Video SDK. You will see this error if you attempt to connect to a room that already contains a participant with the same identity. You would need to change the identity if you would like to connect to a room with the another participant. Do you mind sharing the Room SID after you experience this error? This can be found in the application logs by filtering on sid. It will help us troubleshoot further if needed. Thanks!

I tried to connect with same identity once it was disconnected for the room and I received the disconnect callback. This is the sid I found in the logs: RM68741678564ff1c674b99a38b9ee39fa.

@Alton09 Any progress as to what is causing this and how to handle it?

Hi @him2016 . From looking at the room SID you provided, I do see the Participant disconnected because of duplicate identity but this is due to the fact that the participant is never disconnected before the new one with the same identity joins the room, thus resulting in the error as expected.
I recommend taking a look at the community flavor of the video app as is without any modifications to see how it handles this scenario as I've verified that it handles it properly. Here's the doc on how to quickly get it up and running. Please feel free to reach out for help if needed. Hope this helps!

Hi @him2016 . From looking at the room SID you provided, I do see the Participant disconnected because of duplicate identity but this is due to the fact that the participant is never disconnected before the new one with the same identity joins the room, thus resulting in the error as expected.
I recommend taking a look at the community flavor of the video app as is without any modifications to see how it handles this scenario as I've verified that it handles it properly. Here's the doc on how to quickly get it up and running. Please feel free to reach out for help if needed. Hope this helps!

@Alton09 ok thanks for checking.. will try the same scenario again with the code provided..

@Alton09 ok thanks for checking.. will try the same scenario again with the code provided..

@him2016 Closing this one for now. Feel free to reopen if you are still experiencing this issue.


can grpc-python provide an api for loop socket message in client-side?

As i know,  callback witch add by future's add_done_callback actually run in another thread，this does not work for me. I want to get the socket looper through an api so I can execute the callback in the appropriate thread, does this api already exist?

This issue is being closed because it has been marked with the label kind/question.
Please post questions on https://groups.google.com/g/grpc-io so that the wider community can help answer.

@mnff gRPC Python has a background thread which handles the communication between Python and C extension. Callbacks will be scheduled to that thread. And currently, we don't have an API for your usage.
However, in the callback, you can use Python threading module to delegate your work to another thread.

your suggestion is useful to me, thank you very much!

@mnff gRPC Python has a background thread which handles the communication between Python and C extension. Callbacks will be scheduled to that thread. And currently, we don't have an API for your usage.
However, in the callback, you can use Python threading module to delegate your work to another thread.

@lidizheng your suggestion is useful to me, thank you very much!

This issue is being closed because it has been marked with the label kind/question.
Please post questions on https://groups.google.com/g/grpc-io so that the wider community can help answer.

@yashykt https://groups.google.com/g/grpc-io is 404


No logs in Azure

When running a python function app locally with "func host start" you can see the logs in the console, you can see how the host is started up and you get the log messages from your function app written in the terminal. You can also see any errors there might happen.
When I publish the same function to let it run in Azure these logs are no where to be found. I have enabled Application Insights, but there aren't any information about server response time, server requests or availability, no logs. Same if I go to the monitor tab under the function app, there are no entries even when the app is running and working.
An example of an app behaving like this is the following example: https://github.com/yokawasa/azure-functions-python-samples/tree/master/v2functions/blob-trigger-watermark-blob-out-binding
The app is working when I publish it to my Azure account, images are watermarked, but I can't see anything about the actual execution.
Can someone please share some information on how to enable the logging for python functions, or verify this is a bug in the python worker.

To anyone else who should experience a similar problem.  The logs were there all the time, the problem was I didn't put in a working search query.
I found the search query below useful to start with to get the logs. You can then dive into the query language to get a more specific output.
Go to Application Insights - Logs (Analytics) and put in the follow query and click run:
union traces| union exceptions| where timestamp > ago(7d)| order by timestamp asc


Too many things appear as buttons

I'm using material-ui and their menu-item component makes use of a <span [...] type=button>. As a result of the changes in 9a3b1b3, my menu-items now look like big buttons. Is there a reason the selectors were widened or can we revert these changes?

@OscarMarshall, hi! Thanks for reporting this. AFAIK type="button" is not a valid attribute for that element, it's only valid for <input> and  <button> elements. The proper way to indicate that a <span> it's a button or will behave like one should be role="button".
Regarding the reason, was to lower the specificity, see #438.
Maybe you could open an issue in their repo about this.

Sorry for my delayed response, but @battaglr addressed this concern. We use type attribute values only valid on <button> and <input>.
FWIW, the type attribute may be used on <menu> with a value of context or list, and it may be a valid mime-type when used on <embed>, <object>, <script>, <source>, or <style>. The usage of <span type="button"> is not valid HTML.


Karpenter on EKS fails to remove 'uninitialized' taint from nodes

Description
Observed Behavior:
Karpenter provisions a new node, and it successfully joins the EKS cluster. However, the taint "node.cloudprovider.kubernetes.io/uninitialized=true:NoSchedule" is not removed, preventing the node from being scheduled with pods from deployments. As a result, the node only runs the default add-on pods (e.g., kube-proxy, VPC CNI) but does not receive workload pods.
If the taint is manually removed from the node, everything works as expected, and the pods are scheduled without issues.
This behavior was observed in version 0.37 and persists after updating to version 1.0. It leads to availability issues, as pods remain in a Pending state despite the node being available.
Karpenter does not generate any logs about this issue, providing no insight into why the taint is not being removed.
K8s Events:
not all pods would schedule, xxxx/xxxx-64f49f59f4-jgdmj => would
  schedule against uninitialized nodeclaim/live-mnxjv,
  node/ip-172-16-76-204.sa-east-1.compute.internal
  xxxx/xxxx-7864b5b8-qlsqx => would schedule against
  uninitialized nodeclaim/live-mnxjv,
  node/ip-172-16-76-204.sa-east-1.compute.internal
  xxxx/xxxx-85f77756f4-wtc2s => would schedule against
  uninitialized nodeclaim/live-mnxjv,
  node/ip-172-16-76-204.sa-east-1.compute.internal

Node Clain Condition Message:
KnownEphemeralTaint "[node.cloudprovider.kubernetes.io/uninitialized=true:NoSchedule](http://node.cloudprovider.kubernetes.io/uninitialized=true:NoSchedule)" still exists

Node Taint:
  taints:
    - effect: NoSchedule
      key: live
    - effect: NoSchedule
      key: node.cloudprovider.kubernetes.io/uninitialized
      value: 'true'

Expected Behavior:
When Karpenter provisions a new node and it successfully joins the EKS cluster, the taint "node.cloudprovider.kubernetes.io/uninitialized=true:NoSchedule" should be automatically removed once the node is fully initialized. This would allow pods from deployments to be scheduled on the node alongside the default add-on pods (e.g., kube-proxy, VPC CNI).
Nodes should not require manual intervention to remove the taint, and workload pods should be scheduled as soon as the node becomes available, ensuring no pods remain in a Pending state due to this issue.
Reproduction Steps (Please include YAML):
This issue is intermittent and does not occur with every node provisioned by Karpenter, making it difficult to consistently reproduce. However, the following steps outline a typical setup where the issue has been observed:
Deploy Karpenter on an EKS cluster.
Create a provisioner with support for Spot and On-Demand Instances.
Trigger workloads that scale the cluster and provision new nodes.
Occasionally, newly provisioned nodes will join the cluster but retain the taint "node.cloudprovider.kubernetes.io/uninitialized=true:NoSchedule", preventing workload pods from being scheduled.
Versions:

Chart Version:  v1.0.8
Kubernetes Version (EKS): 1.30

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

:+1:

Hello,
I just created an issue that I think is related to this problem this morning: https://github.com/aws/karpenter-provider-aws/issues/7424.
We for example have at the moment a Nodeclaim in non-ready state (Ready False) for 40 minutes. As you described the node is correctly started (it has the daemonset running etc).
Conditions:
  - lastTransitionTime: "2024-11-22T11:37:51Z"
    message: KnownEphemeralTaint "node.cloudprovider.kubernetes.io/uninitialized=true:NoSchedule"
      still exists
    reason: KnownEphemeralTaintsExist
    status: "False"
    type: Initialized
  - lastTransitionTime: "2024-11-22T11:37:00Z"
    message: ""
    reason: Launched
    status: "True"
    type: Launched
  - lastTransitionTime: "2024-11-22T11:37:24Z"
    message: Initialized=False
    reason: UnhealthyDependents
    status: "False"
    type: Ready
  - lastTransitionTime: "2024-11-22T11:37:24Z"
    message: ""
    reason: Registered
    status: "True"
    type: Registered

As said in my issue we also don't understand why it happens because it seems random, for some nodes it works as expected.

I am facing the same problem.
This issue is greatly complicating my situation.
Version
Kubernetes Version (EKS): 1.30

can you share your config nodepool & ec2nodeclass ?

NodePool
apiVersion: karpenter.sh/v1
kind: NodePool
metadata:
  name: "live"
spec:
  template:
    spec:
      expireAfter: Never # 30 * 24h = 720h
      requirements:
        - key: kubernetes.io/arch
          operator: In
          values: ["amd64"]
        - key: kubernetes.io/os
          operator: In
          values: ["linux"]
        - key: karpenter.sh/capacity-type
          operator: In
          values: ["spot", "on-demand"]
        - key: "karpenter.k8s.aws/instance-hypervisor"
          operator: In
          values: ["nitro"]
        - key: karpenter.k8s.aws/instance-size
          operator: In
          values: [large, xlarge, 2xlarge]
      nodeClassRef:
        group: karpenter.k8s.aws
        kind: EC2NodeClass
        name: live
      taints:
        - key: live
          effect: NoSchedule
  disruption:
    consolidationPolicy: WhenEmptyOrUnderutilized
    consolidateAfter: 3m
  limits:
    cpu: "300"
    memory: 600Gi

EC2NodeClass
apiVersion: karpenter.k8s.aws/v1
kind: EC2NodeClass
metadata:
  name: live
spec:
  amiSelectorTerms:
    - alias: al2023@latest
  role: "EKS_Karpenter_Node_Role"
  blockDeviceMappings:
    - deviceName: /dev/xvda
      ebs:
        volumeSize: 60Gi
        volumeType: gp3
        #iops: 10000
        encrypted: true
        deleteOnTermination: true
        #throughput: 125
  subnetSelectorTerms:
    - tags:
        karpenter.sh/discovery: "k8s-live"
  securityGroupSelectorTerms:
    - tags:
        karpenter.sh/discovery: "k8s-live"
  tags:
    Name: karpenter.sh/live

hello @mcorbin,
you can try test with this above config.
nodepool-ec2nodeclass.yaml
apiVersion: karpenter.sh/v1beta1
# apiVersion: karpenter.sh/v1
kind: NodePool
metadata:
  name: nodepool-large-spot
spec:
  template:
    metadata:
      labels:
        ProjectCost: SRE
        environments: SRE-Dev
        billing-team: SRE
      annotations:
        teams/owner: "SRE"
    spec:
      requirements:
        - key: kubernetes.io/os
          operator: In
          values:
          - linux
        - key: kubernetes.io/arch
          operator: In
          values:
          - amd64
        - key: karpenter.sh/capacity-type
          operator: In
          values:
          - spot
        - key: karpenter.k8s.aws/instance-family
          operator: In
          values:
          - t3a
          - t3
        - key: karpenter.k8s.aws/instance-size
          operator: In
          values:
          - large
      nodeClassRef:
        name: nodeclass-xlarge-spot
  limits:
    cpu: 24
  disruption:
    budgets:
    - nodes: 10%
    consolidationPolicy: WhenEmpty
    consolidateAfter: 24h
    expireAfter: 120h
---
apiVersion: karpenter.k8s.aws/v1beta1
# apiVersion: karpenter.k8s.aws/v1
kind: EC2NodeClass
metadata:
  name: nodeclass-large-spot
spec:
  amiFamily: AL2 # Amazon Linux 2
  role: eksctl-KarpenterNodeRole-eks-sre-op
  blockDeviceMappings:
  - deviceName: /dev/xvda
    ebs:
      deleteOnTermination: true
      iops: 3000
      throughput: 125
      volumeSize: 30Gi
      volumeType: gp3
  subnetSelectorTerms:
    - tags:
        karpenter.sh/discovery: eks-sre-op
    - id: subnet-1234 #     zone: ap-southeast-1c
    - id: subnet-2456 #     zone: ap-southeast-1a
    - id: subnet-789 #     zone: ap-southeast-1b
  securityGroupSelectorTerms:
    - tags:
        karpenter.sh/discovery: eks-sre-op
    - id: sg-234
    - id: sg-45645
    - id: sg-7567
  amiSelectorTerms:
    - id: "ami-123" # EKS v1.31 AMD_AMI_ID
    - id: "ami-234" # EKS v1.31 ARM_AMI_ID
    - id: "ami-456" # EKS v1.31 GPU_AMI_ID
  tags:
    ProjectCost: SRE
	environments: SRE-Dev

hello @silvadavitor
This my config above, i dont use taint.
You can try remove it and test on the new dev environment eks cluster.
In my case I apply this config with apiVersion: karpenter.k8s.aws/v1 then not work but when I was apply this config with apiVersion: karpenter.k8s.aws/v1beta1 then working oke.

Hello @mcorbin,
Have you restart deployment karpenter service ?

We'll test this, at the moment I don't see any pending nodeclaim since this morning but our clusters just started to autoscale with the week starting.

@mcorbin We have also not experienced this issue anymore. To ensure reliability, we conducted several tests by forcing 10 to 100 NodeClaims, and everything worked without errors.
That said, we are also unsure about the root cause of the problem or why it stopped occurring.

To ensure reliability, we conducted several tests by forcing 10 to 100 NodeClaims, and everything worked without errors.

Same, we a did rollout of probably hundreds of nodes and we can't reproduce :/


cgroups: use Set instead of Apply in Freeze

So Apply function of freezer can be as sample as other subsystems.
Signed-off-by: Qiang Huang<EMAIL_ADDRESS>

LGTM

LGTM


Adds allergies exercise

Hi, this PR is to add a new exercise for allergies, here is a link to the problem-specifications
I'm still learning Crystal, so feel free to point out anything that needs to be change or improved.
Thanks!

Updates:

Rebased master
fixed conflict with config.json
regenerated readme
Added generator to match the new canonical-data.json
Updated example.cr to match new canonical-data.json


Adding basic vim move keys (j, k, l, h and V) for selection mode

Adding basic vim moves on selection mode: (h, l; j, k, and V)
Example:
elia-vim-movements_ 2024-06-13 09-10.webm

I've added vim key support in 1.9.0.