A problem with switching from 0.3 to 0.4

I use mobc as my connection pool.
Here is the HelloWorld demo.
pub struct Global {
    redis: Pool<RedisConnectionManager<DefaultExecutor>>,
    postgres: Pool<PostgresConnectionManager<NoTls, DefaultExecutor>>,
}

async fn helloworld(cx: Request<Global>) -> String {
    match cx.state().redis.get().await {
        Ok(_) => "Rust rua!!!".to_string(),
        Err(_e) => "SERVER ERROR".to_string(),
    }
}

It's ok in tide 0.3, but it complied failed in tide 0.4
Here is the error message, my pool is sync + send + 'static, I don't know which part is unsync
error[E0277]: `(dyn futures_io::if_std::AsyncBufRead + std::marker::Send + 'static)` cannot be shared between threads safely
  --> api/src/main.rs:84:17
   |
84 |     app.at("/").get(helloworld);
   |                 ^^^ `(dyn futures_io::if_std::AsyncBufRead + std::marker::Send + 'static)` cannot be shared between threads safely
   |
   = help: the trait `std::marker::Sync` is not implemented for `(dyn futures_io::if_std::AsyncBufRead + std::marker::Send + 'static)`
   = note: required because of the requirements on the impl of `std::marker::Sync` for `std::ptr::Unique<(dyn futures_io::if_std::AsyncBufRead + std::marker::Send + 'static)>`
   = note: required because it appears within the type `std::boxed::Box<(dyn futures_io::if_std::AsyncBufRead + std::marker::Send + 'static)>`
   = note: required because it appears within the type `std::pin::Pin<std::boxed::Box<(dyn futures_io::if_std::AsyncBufRead + std::marker::Send + 'static)>>`
   = note: required because it appears within the type `http_service::Body`
   = note: required because it appears within the type `http::request::Request<http_service::Body>`
   = note: required because it appears within the type `tide::request::Request<Global>`
   = note: required because of the requirements on the impl of `std::marker::Send` for `&tide::request::Request<Global>`
   = note: required because it appears within the type `for<'r, 's, 't0, 't1> {tide::request::Request<Global>, &'r tide::request::Request<Global>, Global, &'s Global, &'t0 mobc::Pool<mobc_postgres::PostgresConnectionManager<tokio_postgres::tls::NoTls, mobc::runtime::DefaultExecutor>>, mobc::Pool<mobc_postgres::PostgresConnectionManager<tokio_postgres::tls::NoTls, mobc::runtime::DefaultExecutor>>, impl std::future::Future, ()}`
   = note: required because it appears within the type `[static generator@api/src/main.rs:34:52: 43:2 cx:tide::request::Request<Global> for<'r, 's, 't0, 't1> {tide::request::Request<Global>, &'r tide::request::Request<Global>, Global, &'s Global, &'t0 mobc::Pool<mobc_postgres::PostgresConnectionManager<tokio_postgres::tls::NoTls, mobc::runtime::DefaultExecutor>>, mobc::Pool<mobc_postgres::PostgresConnectionManager<tokio_postgres::tls::NoTls, mobc::runtime::DefaultExecutor>>, impl std::future::Future, ()}]`
   = note: required because it appears within the type `std::future::GenFuture<[static generator@api/src/main.rs:34:52: 43:2 cx:tide::request::Request<Global> for<'r, 's, 't0, 't1> {tide::request::Request<Global>, &'r tide::request::Request<Global>, Global, &'s Global, &'t0 mobc::Pool<mobc_postgres::PostgresConnectionManager<tokio_postgres::tls::NoTls, mobc::runtime::DefaultExecutor>>, mobc::Pool<mobc_postgres::PostgresConnectionManager<tokio_postgres::tls::NoTls, mobc::runtime::DefaultExecutor>>, impl std::future::Future, ()}]>`
   = note: required because it appears within the type `impl std::future::Future`
   = note: required because it appears within the type `impl std::future::Future`
   = note: required because of the requirements on the impl of `tide::endpoint::Endpoint<Global>` for `fn(tide::request::Request<Global>) -> impl std::future::Future {helloworld}`

@importcjj o/ thanks for reporting! At a glance I'm unsure what's causing this failure. Just to verify: which Rust version are you using? Did you only update the Tide version, or also Rust versions?
I just wrote a quick example to verify that we can indeed read from the global state:
use async_std::io;
use async_std::task;

/// Shared application state.
#[derive(Debug)]
struct State {
    name: String,
}

fn main() -> io::Result<()> {
    task::block_on(async {
        let mut app = tide::with_state(State { name: "nori".to_string() });
        app.at("/submit").post(|req: tide::Request<State>| {
            async move {
                let name = &req.state().name;
                tide::Response::new(200).body_string(name.to_string())
            }
        });
        app.listen("<IP_ADDRESS>:8080").await?;
        Ok(())
    })
}

I'd be happy to help you debug further if you could share a repro. I'm sorry this is happening, but thanks again for reporting!

@yoshuawuyts
My Rust version
stable-x86_64-unknown-linux-gnu (default)
rustc 1.39.0 (4560ea788 2019-11-04)

Here is a simple demo.
https://github.com/importcjj/tide-demo/blob/master/src/main.rs

@importcjj I updated to a recent nightly version, and the diagnostics were super helpful! Solved the problem, and included the code. The short version is: req.pool needed to be assigned to a temporary, which fixed the trait resolver problems.
I'm not exactly sure what caused this to trip up in a recent Tide version, but the problem is easily sidestepped. It seems somewhat similar to the lifetime inference changes that happened as part of 1.39-nightly, but you mentioned you'd been using 1.39-stable all along so I really don't know.
Happy this seems to have been resolved though. Let me know how this works out for you, and then I think we should be okay to close this. Thanks!
Screenshot

Code
use async_std::io;
use async_std::task;

use mobc::{ConnectionManager, runtime::DefaultExecutor, Pool, AnyFuture};

struct FooManager;

unsafe impl Send for FooManager {}
unsafe impl Sync for FooManager {}

struct FooConnection;

impl FooConnection {
    async fn query(&self) -> String {
        "nori".to_string()
    }
}

impl ConnectionManager for FooManager {
    type Connection = FooConnection;
    type Error = std::io::Error;
    type Executor = DefaultExecutor;

    fn get_executor(&self) -> Self::Executor {
        DefaultExecutor::current()
    }

    fn connect(&self) -> AnyFuture<Self::Connection, Self::Error> {
        Box::pin(futures::future::ok(FooConnection))
    }

    fn is_valid(&self, conn: Self::Connection) -> AnyFuture<Self::Connection, Self::Error> {
        Box::pin(futures::future::ok(conn))
    }

    fn has_broken(&self, conn: &mut Option<Self::Connection>) -> bool {
        false
    }
}

/// Shared application state.
struct State {
    pool: Pool<FooManager>
}

fn main() -> io::Result<()> {
    task::block_on(async {
        let pool = Pool::new(FooManager).await.unwrap();
        let mut app = tide::with_state(State { pool });
        app.at("/submit").post(|req: tide::Request<State>| {
            async move {
                let pool = &req.state().pool;
                let conn = pool.get().await.unwrap();
                let name = conn.query().await;
                tide::Response::new(200).body_string("hello".to_string())
            }
        });
        app.listen("<IP_ADDRESS>:8080").await?;
        Ok(())
    })
}

req.pool needed to be assigned to a temporary, which fixed the trait resolver problems.

@yoshuawuyts I know you said the ping wasn't a bug report, but I see that even though that change will be obvious to anyone reading the output with a moderate level of experience with Rust, that hint is not in the output and can be beyond the level of understanding of someone just starting out (or a decaffeinated experienced dev). That makes me think we might want to add a help: consider assigning this to a temporary variable message at the bottom in rustc for these cases. What do you think? If you agree, would you mind opening a ticket for it?

@estebank done!

Thank you!
But this error message really confused me.

@importcjj is the new error understandable to you? If not, please join the thread at the linked rust ticket.

@estebank  The new error message is clear enough, but I'm just confused about the tide state() API.
@yoshuawuyts Why not just use req.state.pool instead of req.state().pool? After all, they do the same thing, and the former doesn't cause errors.

@importcjj providing public field access, unless directly modifiable as part of a constructor, is somewhat of an anti-pattern. Not too thrilled about type inference having a bad time here, but my hopes is that as work on async/await continues, inference will eventually catch up.


Right shift and Usigned right shift operations give runtime errors for some valid cases

Description
$title. Seems this only happens with nilable unsigned integer subtypes. According to the spec If the operator is >> or >>> and the left hand operand is a subtype of int:UnsignedK when K is 8, 16 or 32, then the static type of the result is int:UnsignedN where N is the smallest such K; otherwise, the static type of the result is int. In all cases the static type of the result is modified by the usual [singleton typing](https://ballerina.io/spec/lang/master/#singleton_typing) rules.
But currently these scenarios give runtime errors.
Steps to Reproduce
import ballerina/io;

function fn1() {
    byte x = 251;
    int:Signed8? b = -123;
    int:Signed8 b2 = -123;

    byte? x1 = x >>> b2;
    io:println(x1); // Output = 7
    byte? x2 = x >>> b; // Actual output = error: {ballerina}NumberConversionError {"message":"'int' value '-123' cannot be converted to 'byte'"} at test:init(test.bal:8)
    io:println(x2); // Expected output = 7
}

function fn2() {
    int:Unsigned8 x = 251;
    int:Signed8? b = -123;
    int:Signed8 b2 = -123;

    byte? x1 = x >>> b2;
    io:println(x1); // Output = 7
    int:Unsigned8?|float x2 = x >> b; // Actual output = error: {ballerina}TypeCastError {"message":"incompatible types: 'int' cannot be cast to 'lang.int:Unsigned8'"} at test:init(test.bal:32)
    io:println(x2); // Expected output = 7
}

public function main() {
    // fn1();
    fn2();
}

Affected Version(s)
Current master
OS, DB, other environment details and versions
No response
Labels
jBallerina
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

This happens because the BIR is incorrect. Consider the following Ballerina code.
function fn1() {
    byte x = 251;
    int? b = -123;
    int? x2 = x >>> b; //7
}

In its BIR we can see the following errors.
fn1 function() -> () {
    %0(RETURN) ();
    %1(LOCAL) byte;
    %3(LOCAL) int|();
    %5(TEMP) int;
    %6(LOCAL) int|();
    %8(SYNTHETIC) byte|();// can be int|(), but not necessary
    %9(SYNTHETIC) byte;
    %11(SYNTHETIC) int|();
    %13(SYNTHETIC) boolean;
    %14(TEMP) boolean;
    %15(TEMP) any;
    %22(TEMP) ();
    %25(TEMP) byte;// should be int
    %27(TEMP) byte;

    bb0 {
        %1 = ConstLoad 251;
        %5 = ConstLoad -123;
        %3 = <int|()> %5;//-123
        %9 = %1;//251
        %11 = %3;//-123
        %15 = <any> %9;//251
        %14 = %15 is ();
        %14? bb1 : bb2;
    }
    bb1 {
        %13 = ConstLoad true;
        GOTO bb3;
    }
    bb2 {
        %13 = %11 is ();
        GOTO bb3;
    }
    bb3 {
        %13? bb4 : bb5;
    }
    bb4 {
        %22 = ConstLoad 0;
        %8 = <byte|()> %22;
        GOTO bb6;
    }
    bb5 {
        %25 = <byte> %11;//-123 this should be cast to int // this throws the error 'int' value '-123' cannot be converted to 'byte'
        %27 = %9 >>> %25;
        %8 = <byte|()> %27;// can be <int|()>, but not necessary
        GOTO bb6;
    }
    bb6 {
        %6 = <int|()> %8;
        %0 = ConstLoad 0;
        GOTO bb7;
    }
    bb7 {
        return;
    }
}


WIP: Fix dependencies

swri_roscpp is not exporting its dependencies correctly, as reported in https://github.com/swri-robotics/marti_common/issues/703. This appears to be caused by ament() macros not understanding what an INTERFACE library is. This PR replaces a lot of the ament() calls with direct CMake calls so that dependencies are correctly exported and linked.

I can give it a look / test tomorrow

I can give it a look / test tomorrow

Thanks. I just tested it on my system by install swri_roscpp to my system-wide ROS install, and then building a fresh copy of swri_transform_util, which I think should pull in the header file that was previously causing you problems.

FWIW, it looks like the dependencies get caught now.
However, I don't see the benefit of the Interface Library.
I edited the CMakeLists.txt you had to use the correct ament calls, and I removed the whole library interface.
Instead of exporting the library target, I export the include directory. However, this seems to not work when trying to pickup the headers downstream. I looked in my install directory, and the files were actually in the correct spot.
Confused, I looked at the INCLUDE_DIRS set for swri_roscpp in my package that depends on it, and it looks like two swri_roscpp directories are set:

install/swri_roscpp/include/swri_roscpp
install/swri_roscpp/include

However, when I look at the "compile_commands.json" for the files, I see only -isystem <absolute_path>/install/swri_roscpp/include/swri_roscpp directory, so when trying to include anything like <swri_roscpp/swri_roscpp.hpp> it will not get picked up. I believe this is because of the rosidl_generate_interfaces used.
When I remove all of the code in the if (BUILD_TESTING ...) statement in swri_roscpp, this gets added to the compile command instead: -I<path>/install/swri_roscpp/include

I happened to look at the ROS2 sensor_msgs package, and I'm thinking this may be a cleaner approach to what I was trying: https://github.com/ros2/common_interfaces/blob/rolling/sensor_msgs/CMakeLists.txt. It definitely looks like there's some weirdness around interface only libraries and ament, but that approach looks a lot cleaner.

After doing some digging, I believe that creating the interface library is probably the way to go, mostly because a lot of things in the ament_cmake structure seem to be target based.


"access denied, got HTTP status code 403 (forbidden)" and in-stock GPUs show as not in stock

Hello,
I tried to set up this bot for some shops in my area (Germany). For some vendors, it says "access denied" (see log).
Also, i put in a link with a 3090 that I know is in stock just for testing, but the bot says it is not. When I run the container, it gives me warnings that it's only using generic scrapers.
Does this bot only work with US vendors? What do I need to do to make it work in my region? Or am I just missing a configuration step?
Thanks guys!
[LOG]
W2021-01-11 21:18:05,484 [root] warning: using generic scraper for url: https://www.computeruniverse.net/de/pny-geforce-rtx3080-xlr8-gaming-epic-x-rgb-m-10-gb-enthusiast-grafikkarte
I2021-01-11 21:18:05,485 [cmptrnvrs_nt_32] scraper initialized for https://www.computeruniverse.net/de/pny-geforce-rtx3080-xlr8-gaming-epic-x-rgb-m-10-gb-enthusiast-grafikkarte
...
I2021-01-11 21:24:10,540 [ltrnt_d_1] not in stock
I2021-01-11 21:24:12,588 [ltrnt_d_2] not in stock
I2021-01-11 21:24:14,555 [ltrnt_d_3] not in stock
I2021-01-11 21:24:17,413 [ltrnt_d_4] not in stock
I2021-01-11 21:24:19,400 [ltrnt_d_5] not in stock
I2021-01-11 21:24:21,141 [cskng_d_1] not in stock
I2021-01-11 21:24:23,381 [cskng_d_2] not in stock
I2021-01-11 21:24:25,687 [cskng_d_3] not in stock
I2021-01-11 21:24:27,841 [cskng_d_4] not in stock
W2021-01-11 21:20:29,411 [cmptrnvrs_nt_1] access denied, got HTTP status code 403 (forbidden)
W2021-01-11 21:20:31,593 [cmptrnvrs_nt_2] access denied, got HTTP status code 403 (forbidden)
W2021-01-11 21:20:33,859 [cmptrnvrs_nt_3] access denied, got HTTP status code 403 (forbidden)

Might want to try slowing down your refresh rate... some site admins set thresholds, and block when you refresh too much. Also, you could try scrambling/chunking your list of urls in your .yaml, if you're using a combined one (verses by domain), so that you aren't doing all the pulls at one site back-to-back. It might be valuable to have an argument to randomize the scrapes a bit, to make it look less structured/bot-like..

Might want to try slowing down your refresh rate... some site admins set thresholds, and block when you refresh too much. Also, you could try scrambling/chunking your list of urls in your .yaml, if you're using a combined one (verses by domain), so that you aren't doing all the pulls at one site back-to-back. It might be valuable to have an argument to randomize the scrapes a bit, to make it look less structured/bot-like..


Need Documentation on Markdown Custom Styling

It's not clear how to provide custom styling for the generated markdown.

My solution was to use global.

<SvelteMarkdown {source}></SvelteMarkdown>

I'll add this to the README. But the approach is either what @magnetenstad mentioned, or using your own custom renderer for the element you want. E.g. if you want to style paragraphs you'd create your renderer:

<slot />

And then pass it to SvelteMarkdown using the renderers prop:
<SvelteMarkdown source={source} renderers={{ paragraph: Paragraph }} />

@pablo-abc Perfect! Thank you for the guidance on that.

Adding here in-case someone stumbles upon this issue and wants a quick solution. It's exactly the same as solution by @magnetenstad but keeps the style local to current element.
// Required: id should be unique here

	<SvelteMarkdown source={tokens} />


Addition of Pairs problem under HackerRank

Problem Link - https://www.hackerrank.com/challenges/pairs/problem
I want to add the solution to this problem in Java under LSM SOC'21.
Please, assign me this.

Please, assign me this.

@jayant766 mention the language

Language:- C++, python.
Please, assign me this issue.


Add hololens2-deploying-remote-assist-using-shared-identity.md

Add article - Deploying HoloLens 2 with Remote Assist using a shared identity across multiple users.

Closed in favor of other PR.


Is it possible to export to an mtl file?

I know from #1 how to export as a ply file, however I have had no luck at exporting any textures. I saw that in shap_e/examples/example_data/cactus/ is an obj file and a mtl file, is this something that could be generated from shap-e?

Update, If anybody else is needs this, see PR #20

I have a basic alternative solution using the trimesh library.
Also, if you have this same concern as me: vertices generated by marching cubes are very dense, and vertex color is enough for the color resolution.
@torch.no_grad()
def decode_render(
    xm: Union[Transmitter, VectorDecoder],
    latent: torch.Tensor,
    cameras: DifferentiableCameraBatch,
    rendering_mode: str = "stf",
):
    decoded = xm.renderer.render_views(
        AttrDict(cameras=cameras),
        params=(xm.encoder if isinstance(xm, Transmitter) else xm).bottleneck_to_params(
            latent[None]
        ),
        options=AttrDict(rendering_mode=rendering_mode, render_with_direction=False),
    )
    arr = decoded.channels.clamp(0, 255).to(torch.uint8)[0].cpu().numpy()
    return decoded, [Image.fromarray(x) for x in arr]

render_mode = 'stf' # you can change this to 'stf' for mesh rendering
size = 2  # this is the size of the renders; higher values take longer to render.

cameras = create_pan_cameras(size, device)
for i, latent in enumerate(latents):
    decoded, images = decode_render(xm, latent, cameras, rendering_mode=render_mode)
    assert len(decoded['meshes']) == 1
    mesh = decoded['meshes'][0]
    trimesh.exchange.export.export_mesh(trimesh.Trimesh(
        mesh.vertices.cpu().numpy(),
        mesh.faces.cpu().numpy(),
        vertex_colors=mesh.vertex_colors.cpu().numpy()
    ), "barrel-gen.glb")

I have a basic alternative solution using the trimesh library. Also, if you have this same concern as me: vertices generated by marching cubes are very dense, and vertex color is enough for the color resolution.
@torch.no_grad()
def decode_render(
    xm: Union[Transmitter, VectorDecoder],
    latent: torch.Tensor,
    cameras: DifferentiableCameraBatch,
    rendering_mode: str = "stf",
):
    decoded = xm.renderer.render_views(
        AttrDict(cameras=cameras),
        params=(xm.encoder if isinstance(xm, Transmitter) else xm).bottleneck_to_params(
            latent[None]
        ),
        options=AttrDict(rendering_mode=rendering_mode, render_with_direction=False),
    )
    arr = decoded.channels.clamp(0, 255).to(torch.uint8)[0].cpu().numpy()
    return decoded, [Image.fromarray(x) for x in arr]

render_mode = 'stf' # you can change this to 'stf' for mesh rendering
size = 2  # this is the size of the renders; higher values take longer to render.

cameras = create_pan_cameras(size, device)
for i, latent in enumerate(latents):
    decoded, images = decode_render(xm, latent, cameras, rendering_mode=render_mode)
    assert len(decoded['meshes']) == 1
    mesh = decoded['meshes'][0]
    trimesh.exchange.export.export_mesh(trimesh.Trimesh(
        mesh.vertices.cpu().numpy(),
        mesh.faces.cpu().numpy(),
        vertex_colors=mesh.vertex_colors.cpu().numpy()
    ), "barrel-gen.glb")

what is 'trimesh.exchange'? it doesn't seem to exist in my library

couple follow up questions, how could this be implemented using other render modes (the others throw errors)?
how could this be implemented with pytorch3d (texture seems to be implemented differently vs a trimesh)?
can this be added as an example in the project?

I have a basic alternative solution using the trimesh library. Also, if you have this same concern as me: vertices generated by marching cubes are very dense, and vertex color is enough for the color resolution.
@torch.no_grad()
def decode_render(
    xm: Union[Transmitter, VectorDecoder],
    latent: torch.Tensor,
    cameras: DifferentiableCameraBatch,
    rendering_mode: str = "stf",
):
    decoded = xm.renderer.render_views(
        AttrDict(cameras=cameras),
        params=(xm.encoder if isinstance(xm, Transmitter) else xm).bottleneck_to_params(
            latent[None]
        ),
        options=AttrDict(rendering_mode=rendering_mode, render_with_direction=False),
    )
    arr = decoded.channels.clamp(0, 255).to(torch.uint8)[0].cpu().numpy()
    return decoded, [Image.fromarray(x) for x in arr]

render_mode = 'stf' # you can change this to 'stf' for mesh rendering
size = 2  # this is the size of the renders; higher values take longer to render.

cameras = create_pan_cameras(size, device)
for i, latent in enumerate(latents):
    decoded, images = decode_render(xm, latent, cameras, rendering_mode=render_mode)
    assert len(decoded['meshes']) == 1
    mesh = decoded['meshes'][0]
    trimesh.exchange.export.export_mesh(trimesh.Trimesh(
        mesh.vertices.cpu().numpy(),
        mesh.faces.cpu().numpy(),
        vertex_colors=mesh.vertex_colors.cpu().numpy()
    ), "barrel-gen.glb")

Where do I get Transmitter, VectorDecoder, and DifferentiableCameraBatch?

I have a basic alternative solution using the trimesh library. Also, if you have this same concern as me: vertices generated by marching cubes are very dense, and vertex color is enough for the color resolution.
@torch.no_grad()
def decode_render(
    xm: Union[Transmitter, VectorDecoder],
    latent: torch.Tensor,
    cameras: DifferentiableCameraBatch,
    rendering_mode: str = "stf",
):
    decoded = xm.renderer.render_views(
        AttrDict(cameras=cameras),
        params=(xm.encoder if isinstance(xm, Transmitter) else xm).bottleneck_to_params(
            latent[None]
        ),
        options=AttrDict(rendering_mode=rendering_mode, render_with_direction=False),
    )
    arr = decoded.channels.clamp(0, 255).to(torch.uint8)[0].cpu().numpy()
    return decoded, [Image.fromarray(x) for x in arr]

render_mode = 'stf' # you can change this to 'stf' for mesh rendering
size = 2  # this is the size of the renders; higher values take longer to render.

cameras = create_pan_cameras(size, device)
for i, latent in enumerate(latents):
    decoded, images = decode_render(xm, latent, cameras, rendering_mode=render_mode)
    assert len(decoded['meshes']) == 1
    mesh = decoded['meshes'][0]
    trimesh.exchange.export.export_mesh(trimesh.Trimesh(
        mesh.vertices.cpu().numpy(),
        mesh.faces.cpu().numpy(),
        vertex_colors=mesh.vertex_colors.cpu().numpy()
    ), "barrel-gen.glb")

Where do I get Transmitter, VectorDecoder, and DifferentiableCameraBatch?

I am thinking of the same question, which package do i need to import in my code to make them run?


Lens doesn't use MacOs DNS resolver

Describe the bug
Lens can't resolve DNS records when MacOs can.
We are connecting to development/production environments in AWS through VPN.
We are using private Route53 zones for internal DNS, so they are not visible worldwide.
Looks like Lens tries to resolve DNS using nameserver from /etc/resolv.conf which is usually router which not connected to VPN.
At the same time, MacOs DNS resolver can resolve records through VPN.
To Reproduce
Steps to reproduce the behavior:

Connect to VPN with private DNS on Route53
Add a cluster by private DNS name

Expected behavior
The cluster should be connected successfully
Screenshots

Environment (please complete the following information):

Lens Version: 3.1.0
OS: MacOs Catalina 10.15.3 (19D76)
Installation method: dmg

Logs:

Kubeconfig:
Quite often the problems are caused by malformed kubeconfig which the application tries to load. Please share your kubeconfig, remember to remove any secret and sensitive information.
apiVersion: v1
kind: Config
clusters:
- name: "dc1"
  cluster:
    server: "https://rancher.preprod.voicebase.com/k8s/clusters/c-56vd7"
    certificate-authority-data: ""
users:
- name: "dc1"
  user:
    token: ""

contexts:
- name: "dc1"
  context:
    user: "dc1"
    cluster: "dc1"
current-context: "dc1"

Additional context
Add any other context about the problem here.

Confirmed for a self hosted Kube cluster via VPN as well.

as a workaround until this gets fixed:
Add your resolved IP to your /etc/hosts file.

Is this getting picked up? Same error when using a private Azure DNS zone. kubectl and other tools pick it up, but Lens just throws out errors:
E1208 09:28:35.804660 80784 proxy_server.go:147] Error while proxying request: remote error: tls: internal error
remote error: tls: internal error


"WebRTC server not available"

Hi
I am getting the above error
The RTSP streams work fine in VLC on multiple computers. IT was also working fine on my raspberry pi, I have now swapped to an ubuntu based VM
They are hikvision cameras through an NVR

Also working through camera & stream component within HA

Try to remove go2rtc binary from your Hass config folder and restart Hass

Try to remove go2rtc binary from your Hass config folder and restart Hass

Thanks tried and stream is semi working . However I have to keep restarting gortc add on as the stream just stops loading on dashboards, any tips?

Had the same problem after i migrated from a rpi 4 to a intel nuc (w/proxmox). The go2rtc binary was for aarch64 (rpi) and thus did no longer work, causing the "webrtc server not available" message. I tried removing and reinstalling webrtc but it didnt help - i manually had to remove the go2rtc binary, restart HA and it apparently downloaded the correct binary.
@AlexxIT Maybe add some check at startup to see if the arch has changed to prevent this?

Perhaps there was an error when downloading go2rtc binary. And you need to delete it manually, so that it can be downloaded again.
I'll think about some validation after download.

Had the same problem after i migrated from a rpi 4 to a intel nuc (w/proxmox). The go2rtc binary was for aarch64 (rpi) and thus did no longer work, causing the "webrtc server not available" message. I tried removing and reinstalling webrtc but it didnt help - i manually had to remove the go2rtc binary, restart HA and it apparently downloaded the correct binary.
@AlexxIT Maybe add some check at startup to see if the arch has changed to prevent this?

I have exactly this problem => how can I remove the go2rtc binary? => for beginners please

Its in /config. Open HA in your browser and choose terminal.
cd /config
rm go2rtc
Thats it.

Its in /config. Open HA in your browser and choose terminal. cd /config rm go2rtc Thats it.

thanks a lot, it works. Thanks so much!!!

https://github.com/AlexxIT/WebRTC/releases/tag/v3.6.0


TPM using Windows PowerShell. For details, see

[Enter feedback here]

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

ID: f424ce3a-a0fc-9c92-928b-8fdb40d6ac43
Version Independent ID: f424ce3a-a0fc-9c92-928b-8fdb40d6ac43
Content: Manage TPM commands - Windows Security
Content Source: windows/security/hardware-security/tpm/manage-tpm-commands.md
Product: windows-client
Technology: itpro-security
GitHub Login: @vinaypamnani-msft
Microsoft Alias: vinpa

I'm assuming this is an accidental feedback, so closing. If I'm incorrect, please reopen and provide more detail.


Siemens | Teamcenter Suite

Please provide the following information.

Vendor Name : Siemens
Product Name : Teamcenter Suite
Version(s) affected : All versions
Status:  Affected
Update Available: Yes
Notes : None
References : https://cert-portal.siemens.com/productcert/pdf/ssa-661247.pdf
Last Updated:  12/15/2021

Thank you for submitting the issue.  Updated.


Only local scope first class

Hi, i want to take the local scoped class and use it in my components, but i don't want to have to import every name from the file

and the way it works now i would have to do

which makes it more difficult to read, is it possible to only format the wrapper class?
Thanks :)

You can obviously assign only the first wrapper class and using a css pre-processor (like SASS) nest inner and logo like so:
.wrapper {     color: blue;     .inner {         color: red;         .logo {             color: yellow;         }     }

You can do that with :local(...), but I would not recommend it.


Merge updated main to develop

Can one of the admins verify this patch?

Jenkins please retry a build


unable to get examples to work

Followed the instructions for the babylon demo folder.  it says go to root folder to run npm install and then npm run start-babylon.  Two browser tabs open up, but when clicking on any of the demo links get lots of compile error like this:
ERROR in ./packages/amazon-sumerian-hosts-babylon/src/Babylon.js/HostObject.js 8:0-64
Module not found: Error: Can't resolve '@babylonjs/core/Loading/sceneLoader' in '/Users/username/pp/amazon-sumerian-hosts/packages/amazon-sumerian-hosts-babylon/src/Babylon.js'
Are there some missing instructions?

@hchou-godaddy I just tried this myself and didn't experience any issues. Let's see if we can figure out what's different on your system.
What operating system are you using? (I'm on MacOS.)
Give this a try...
From the root folder of the repository, run git clean -xfd to remove all old build files.
From the root folder of the repository, run npm install. Confirm that you don't see any errors or warnings in the output.
From the root folder of the repository, run npm run start-babylon. Confirm that you don't see any errors in the output. Leave this terminal window open and running.
In the browser tab for the demos, see if you can successfully run the "Hello World" demo. If the demo gets stuck on the "loading" screen, open the browser console and look for errors. (If this happens it's usually because you've failed to add your Cognito Identity Pool ID to the application's code.)


mgmt, update test on webpubsub

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

/azp run java - webpubsub - mgmt - tests

/azp run java - webpubsub - mgmt - tests

/azp run java - webpubsub - mgmt - tests


[BUG] Address CVE-2022-42889 by updating commons-text

What is the bug?
The CVE-2022-42889 was published affecting commons-text < 1.10.0
What is the expected behavior?
Update commons-text < 1.10.0
Do you have any additional context?
Add any other context about the problem.

Will there be new docker images published with the updated version?

@reta can you provide additional information on the impact of the vulnerability on OpenSearch is the actual vulnerable code being used?

@reta can you provide additional information on the impact of the vulnerability on OpenSearch is the actual vulnerable code being used?

@semmlfra I have not looked into the ways to reproduce the issue in OpenSearch (focusing on update first), but the commons-text is coming as transitive depedencies in a few plugins (security and ml-commons notably) which might be explotable through 3rd party libraries.

@reta can you provide additional information on the impact of the vulnerability on OpenSearch is the actual vulnerable code being used?

@semmlfra I have not looked into the ways to reproduce the issue in OpenSearch (focusing on update first), but the commons-text is coming as transitive depedencies in a few plugins (security and ml-commons notably) which might be explotable through 3rd party libraries.

Thanks @reta for providing this additional information


implement grand sub entry status summary bar

would like to have in shot/asset group sub entries
[ | inprogress | retake | done | delivered ]

now invalid


TensorFlow/examples The gesture_classification Install Dependencies FAIL

As the picture shows

Please provide details about what platform you are using (operating system, architecture). Also include your TensorFlow version. Also, did you compile from source or install a binary?
Make sure you also include the exact command if possible to produce the output included in your test case. If you are unclear what to include see the issue template displayed in the Github new issue template.
We ask for this in the issue submission template, because it is really difficult to help without that information. Thanks!

I run the TensorFlow Lite example from HERE.
I choice the gesture_classification.
When I finish to train my model by web  and download it.
Then I use Colabs to open the tensorflowjs_to_tflite_colab_notebook.ipynb.
In the Install Dependencies part, I saw Error msg like the image.
Is that correct?

Looks like this is  TensorflowJS issue. You can post this issue on TensorFlowJS repository for better and faster help. Let us know if you have any clarification. Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Add a link to openFPGALoader?

OpenFPGALoader is supporting DirtyJTAG, it would be good to add at least a link to the project, if not a nice picture and a screenshot:
https://github.com/trabucayre/openFPGALoader

Will do. A cheap ECP5 board + DirtyJTAG tutorial would be ideal.

There is a good blogpost here:
https://blog.pcbxprt.com/index.php/2020/07/19/running-risc-v-core-on-small-fpga-board/

What do you think of #74?


Add support for end-breaths and similar to JPCVVC Phonemizer.

Adding notes to the ends of phrases with "R", "-", "息" or "吸" will now add the ending transition or end breath to the end of the syllable.

Additionally fixed an issue with conflicting duplicate "R" in dictionary.


Next.js

@vlasenko Have you considered next.js possibilities? They have their own webpack / command line tools and whatnot. Let me know if anything comes to mind immediately, but I'm considering it would be nice to be able to have that as another option.

@justinr1234 They don't support building React Native project with Webpack


Day 209

Good morning! Here's your coding interview problem for today.
This problem was asked by YouTube.
Write a program that computes the length of the longest common subsequence of three given strings. For example, given "epidemiologist", "refrigeration", and "supercalifragilisticexpialodocious", it should return 5, since the longest common subsequence is "eieio".

https://www.researchgate.net/publication/221313835_Two_Algorithms_for_the_Longest_Common_Subsequence_of_Three_or_More_Strings


Stage 3 shows no such file or directory

I followed the instruct line by line, and when bash run.sh --stage 3 --stop-stage 3, it showed no such file or directory, what can I do? Thanks.
Describe the bug
tools/format_data.sh --nj 32 --feat-type wav --feat raw_wav/dev/wav.scp raw_wav/dev data/dict/lang_char.txt
split: illegal option -- -
usage: split [-a sufflen] [-b byte_count] [-l line_count] [-p pattern]
[file [prefix]]
ls: raw_wav/dev/log/wav_.slice: No such file or directory
cat: raw_wav/dev/log/wav_.shape: No such file or directory
tools/format_data.sh --nj 32 --feat-type wav --feat raw_wav/test/wav.scp raw_wav/test data/dict/lang_char.txt
split: illegal option -- -
usage: split [-a sufflen] [-b byte_count] [-l line_count] [-p pattern]
[file [prefix]]
ls: raw_wav/test/log/wav_.slice: No such file or directory
cat: raw_wav/test/log/wav_.shape: No such file or directory
tools/format_data.sh --nj 32 --feat-type wav --feat raw_wav/train/wav.scp raw_wav/train data/dict/lang_char.txt
split: illegal option -- -
usage: split [-a sufflen] [-b byte_count] [-l line_count] [-p pattern]
[file [prefix]]
ls: raw_wav/train/log/wav_.slice: No such file or directory
cat: raw_wav/train/log/wav_.shape: No such file or directory
Desktop (please complete the following information):

OS: Mac os

got it


[APIView] APIView Reviews / Revisions / Comments Structure

Problem Summary

There are multiple independent reviews with the same package name that don't share history (Comments and Revisions) and cannot be diffed across revisions resulting in the following difficulties.
- Difficulty identifying which review/revision to work with
- Difficulty finding old review/ revisions that have been previously interacted with (approved, reviewed, commented on)
- Difficulty viewing comment from related review / revisions
- No ability to diff across related review/ revisions

Proposed Solution

Restructure APIView to group all revisions regardless of the type under one Review for one package / language.
e.g. Azure.Template for C# should have one Review with all C# template revisions grouped under it.
and @azure/template for JavaScript should have one Review with all JS template revisions under it

Proposed Plan

[x] Create a clone of the Database and restructure 'schema' to support proposed plan.
[x] Update App logic to run off clone of database (Estimated Completion: Friday 3rd November 2023)
[x] Update Pipeline Automation to call updated API (Estimated Completion: Friday 10th November 2023)
[x] Estimated deployment to prod is Nov 27th

Deployed on 12/8


Nice but

I suggest creating an installer (with framework 462 for an exe) instead of a nuget package.
Two reasons:

Install once, and allow all applications to use it is easier
if the application is aspnet core with framework, it is not compatible

Thank you for your feedback, @olivier-voutat; that's an interesting suggestion.
It looks like it will soon be possible to install tools like dotnet-transform-xdt globally, and call them as if they were executables (details here and here). I think this would cover your first use case without giving up on cross-platform advantages. As the global tool support is still being actively developed, I plan to evaluate it in more depth once it stabilizes (likely after .NET Core 2.1 is released).
The current dotnet-transform-xdt tool works fine in an ASP.NET Core application targeting the full .NET framework. You can test this by cloning the samples repository and changing e.g. the WebServerSettingsSample.csproj file from:
<TargetFramework>netcoreapp2.0</TargetFramework>

to
<TargetFramework>net462</TargetFramework>

Run dotnet restore, then dotnet publish -c Release, and the tool will be invoked to transform Web.config. I hope this helps.

It is what I did. And I put the exe in my C:\Program Files\dotnet folder (in the TFS agents servers too). So no need to have it in all the projects.

Glad to hear that you found a workaround.
I will close this issue and open one to track making dotnet-transform-xdt available as a global tool.

@olivier-voutat a preview version of the .NET CLI global tool is now available; details here.
A standalone executable for .NET 4.6.1 or later is also produced on every AppVeyor build: https://ci.appveyor.com/project/nil4/dotnet-transform-xdt/build/artifacts


Rename modules in datacreators

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (263bc5e) 100.00% compared to head (9eacc7f) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #270   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          319       319
  Lines         9870      9873    +3
  Branches       952       952
=========================================
+ Hits          9870      9873    +3

Impacted Files
Coverage Δ

src/gravitorch/data/datacreators/__init__.py
100.00% <100.00%> (ø)

src/gravitorch/data/datacreators/base.py
100.00% <100.00%> (ø)

src/gravitorch/data/datacreators/caching.py
100.00% <100.00%> (ø)

src/gravitorch/data/datacreators/hypercube.py
100.00% <100.00%> (ø)

src/gravitorch/data/dataloaders/collators/base.py
100.00% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


neovimUtils: put customRC before plugins

This matches the behaviour of home-manager better
CC @teto
See also https://github.com/NixOS/nixpkgs/pull/120445

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Added a release notes entry if the change is major or breaking
[x] Fits CONTRIBUTING.md.

customRc should go after plugins (as it's usually done in a vim config). You can use beforePlugin if you want to pass a config before the plugins..

Ah I see, thanks for the feedback on this and the other issue, @teto. Sound's like they can both be closed :)


[DNM⚠] Change how we configure Redis

Context

For faster test runs, we want to be able to run our test suite in parallel using multiple cores. This is some of the configuration change required to support that.
Changes proposed in this pull request
The commits are a bit wordy but to summarize:

We plan to use parallel_tests to split up our test suite and run each set of tests in a different process.
Each process needs a unique Redis database to ensure test runs remain free of side effects.
To support this, update Redis config so that it follows these conventions:

Use REDIS_URL wherever it's already set, meaning these changes should have no effect on any existing environment that sets this variable.
Use Redis database 0 in development (was previously 1, as defined by .env)
Use Redis database 9 when running tests normally (no change here)
Use Redis databases 1-n when running tests in multiple processes, where n is determined by how many processes are created.

To ensure these conventions hold, access Redis via the ApplyRedisConnection class.

Guidance to review

Per commit recommended, commit messages give a bit more detail explaining the changes.
Do the new conventions make sense?

Link to Trello card
https://trello.com/c/ySrLogAF

Things to check

[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] API release notes have been updated if necessary
[ ] New environment variables have been added to the Azure config

This looks good to me and I like the Redis.current approach. Does it work with parallel_tests?

This looks good to me and I like the Redis.current approach. Does it work with parallel_tests?

Does it work with parallel_tests?

Yes, just about to raise the PR adding that in.

Does it work with parallel_tests?

Yes, just about to raise the PR adding that in.


Create Request Mappings for the Students list page

Create a controller, necessary services and repositories to handle Requests for the Students list page.

Include student's name and surname in JSON output

Show whether a user offered help to a particular student's card.


make fsop timeout injectable

We have a weak build machine for React Native app: a poor Mac Book Air with HDD, which is very slow.
So, fs operations sometimes fail because of timeout.
We likely don't care about build time, but it's pretty annoying to restart builds sometimes.
In this PR I make timeout injectable via environment variable.
USAGE:
export this variable into environment. It's measured in miliseconds.
Example, for 1 minute timeout:
export REACT_NATIVE_FSOP_TIMEOUT=60000
If you don't specify it, it will remain default value: 15000
This should not break anything, but repair slow builds.
Related to : #9373 , #8794 (I think so)
This PR should handle case of issue and close #9373

By analyzing the blame information on this pull request, we identified @bestander and @davidaurelio to be potential reviewers.

@facebook-github-bot shipit

Thanks for importing.If you are an FB employee go to Phabricator to review internal test results.


Enhance README: Shields to capture the attention of potential contributors

I noticed you have opened issues labeled help wanted. I recommend enhancing the README to make the call for new contributors more prominent.
Changes Made:

Shields: Added Shields.io badges at the top of the README

[![GitHub repo Good Issues for newbies](https://img.shields.io/github/issues/ashutosh-rath02/git-re/good%20first%20issue?style=flat&logo=github&logoColor=green&label=Good%20First%20issues)](https://github.com/ashutosh-rath02/git-re/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22) [![GitHub Help Wanted issues](https://img.shields.io/github/issues/ashutosh-rath02/git-re/help%20wanted?style=flat&logo=github&logoColor=b545d1&label=%22Help%20Wanted%22%20issues)](https://github.com/ashutosh-rath02/git-re/issues?q=is%3Aopen+is%3Aissue+label%3A%22help+wanted%22) [![GitHub Help Wanted PRs](https://img.shields.io/github/issues-pr/ashutosh-rath02/git-re/help%20wanted?style=flat&logo=github&logoColor=b545d1&label=%22Help%20Wanted%22%20PRs)](https://github.com/ashutosh-rath02/git-re/pulls?q=is%3Aopen+is%3Aissue+label%3A%22help+wanted%22) [![GitHub repo Issues](https://img.shields.io/github/issues/ashutosh-rath02/git-re?style=flat&logo=github&logoColor=red&label=Issues)](https://github.com/ashutosh-rath02/git-re/issues?q=is%3Aopen)

Welcoming Message: Enhanced the README with a more prominent and inviting message to welcome new contributors.

Why This Matters:

Attract New Contributors: Considering there is a number of opened issues labeled 'Help Wanted' - a welcoming and appealing README is one of the most efficient ways to attract new contributors

Additional Resources:
For more insights into building a strong open-source community, check out the Building Community guide.
🚀 Thank you for considering this pull request! 🚀

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

This issue has been automatically closed because it has been inactive for more than 30 days. If you believe this is still relevant, feel free to reopen it or create a new one. Thank you!


http_parser should support more time zones than GMT

Per http://freesoft.org/CIE/RFC/1123/99.htm
"There is a strong trend towards the use of numeric timezone indicators, and implementations SHOULD use numeric timezones instead of timezone names. However, all implementations MUST accept either notation. If timezone names are used, they MUST be exactly as defined in RFC-822."

This issue has been moved to dart-lang/http_parser#1.


Remove rtw89 firmware

Description of changes
The rtw89-firmware package have been removed and replaced with a throw, however I didn't just want to remove it in case people have older versions of nixpkgs so I made it conditional if the package evals or not.
With this in place I updated my flake input to use that branch and I can build the system without getting the throw error from this package.
Things done

[X] Tested the changes in your own NixOS Configuration
[X] Tested the changes end-to-end by using your fork of nixos-hardware and
importing it via <nixos-hardware> or Flake input

Isn't this PR related to #563?
(also not to be bikeshedding, but i feel like switching on the version is better than trying to evaluate it).

Seems to be a duplicate yes


[WIP] Upgrade controllers, k8s version, and node images in one shot.

This has been move to metal3-dev-env
All except k8s version are move. We have decided to add 'k8s version' and should be included there.


Fix key generation in local-up-cluster.sh

Was writing key to a location that non-sudo cannot typically write to. Changed to /tmp to match other file outputs in local-up-cluster.sh, and ensured parent dir exists if needed

@deads2k fyi

@nikhiljindal PTAL

Thanks! Verified that this works.

Gonna test this locally and merge.

WFM/LGTM, merging.


Name inconsistencies

Card name tends to be used as an id and is not always consistent between the source for card text and the source for card metrics. This needs to be harmonized. Encoding problems may be present.

All cards reconciled.


Split all git logic in editor to different authoring api service

Api will be roughly based on editor server api:
https://github.com/Soluto/tweek/blob/master/services/editor/server/serverRoutes.js
Exposes:
key editing (/keys/)
schema editing (/schema)
search (/search)
search-index (/search-index)
manifest listing (/manifests/, /manifests/*)

#491


Fix assert crash when stopping log

When we exit mavlink-router we call logger->stop() to stop
and properly finalize the flight stack log. However this
call may want to send something to the flight stack to stop
sending the log.
We don't care about that call as if we are really stopping the flight
stack should detect the other side died/disconnected after a timeout.
Instead of marking the mainloop as not initiliazed upon signal,
serialize it with the other shutdown task in the main loop.
This fixes the crash:
Logging target system_id=1 on 00001-2021-12-29_23-57-32.bin
mavlink-routerd: ../src/mainloop.h:77: static Mainloop&Mainloop::get_instance(): Assertion `_initialized' failed.
Aborted

Doesn't work for me :
mavlink-router version v2-146-gebf5b4a
Opened TCP Client [4]to_SITL: <IP_ADDRESS>:5762
File 00001-2021-12-30_21-51-14.bin not read-only yet, marking as RO
Logging target system_id=1 on 00002-2021-12-31_02-12-56.bin
^Cmavlink-routerd: ../src/mainloop.h:78: static Mainloop& Mainloop::get_instance(): Assertion `_initialized' failed.
[1]    182433 IOT instruction (core dumped)  mavlink-routerd -c ../mavlink-router.conf -t 0

I used CTRL-C to quit.
Mavlink-router was connected to ArduPilot SITL with this config file :
[General]
MavlinkDialect=ardupilotmega
Log=/tmp/flightstack
ReportStats=false
FcuId=1

[TcpEndpoint to_SITL]
Mode = Normal
Address = <IP_ADDRESS>
Port = 5762
RetryTimeout=2

and launched with mavlink-routerd -c ../mavlink-router.conf -t 0

@khancyr are you sure you executed the mavlink-router binary after applying this PR?

@lucasdemarchi yep.
I just did another test by putting printf where you did the changes
mavlink-router version v2-146-gebf5b4a+
Opened TCP Client [4]to_SITL: <IP_ADDRESS>:5762
Logging target system_id=1 on 00005-2021-12-31_13-40-58.bin
^Cno _initialized = false;_initialized = false;
mavlink-routerd: ../src/mainloop.h:78: static Mainloop& Mainloop::get_instance(): Assertion `_initialized' failed.
[1]    184379 IOT instruction (core dumped)  mavlink-routerd -c ../mavlink-router.conf -t 0

so it looks good but we still have a get_instance() triggered before stop

The remaining error comes from TcpEndpoint::close() that call Mainloop::get_instance().remove_fd(fd); Then it crash on the get_instance() there !

Oh. Ok, thanks for testing. So we should move this further down, or even just remove the setting back to -1


Connections have unique names instead of using host/IP

The Incorrect connection settings from the "code-for-ibmi.connectionSettings" array are used when multiple connections to the same Host System are defined in the “code-for-ibmi.connections" array.
The problem does not occur if the “CASE” of the host names is different.
Example
Problem occurs when connections are defined as 1) user1@pub400 and 2) user2@pub400.
But does not occur when connections are defined as 1) user1@pub400 and 2) user2@PUB400.
I’m Using version 0.4.1.
Might be nice if I could give the connection and connsectionSettings names.  This would solve the above issue and allow multiple connections to share the same settings.   See below.
"code-for-ibmi.connectionSettings": [
{
"connectionSettingsName": "config1",
"sourceFileList": [
"TESTLIB/QRPGLESC"
]
},
{
"connectionSettingsName": "config2",
"sourceFileList": [
"USERLIB/QRPGLESC"
]
}
],
"code-for-ibmi.connections": [
{
"name": "connection1",
"host": "pub400",
"port": 22,
"username": "userx",
"privateKey": null,
"useConnectionSettingsName": "config1"
},
{
"name": "connection2",
"host": "pub400",
"port": 22,
"username": "usery",
"privateKey": null
"useConnectionSettingsName": "config2"
},
{
"name": "connection3",
"host": "pub400",
"port": 22,
"username": "userz",
"privateKey": null,
"useConnectionSettingsName": "config1"
}
],
`
This might also address solve issue “Multiple setting per environment #104”.
Maybe a future enhancement could allow the connection settings from a different name to be dynamically selected without disconnecting.

Yes, it might be a good idea. It's one of those things which I will need to sit down and actually do - since another change of the configuration could be breaking.
I will look into it, thanks!

FYI I am working on a connections pane and have had to implement this in order to make it sensible, the commit is sat on top of the initial connection pane work, so is unfortunately not self contained, and can't be cherry picked

PR #128 Implemented this as a feature because I require it for another feature I am working on, so split out as per the 1 feature perr PR rule


Shuttle Arrival needs to check for occupied space before arriving

specifically centcom
evac shuttle and cargo shuttle often end up at centcom warped inside of each other
maybe have only evac shuttle warp to dock and make all other shuttles warp nearby area (like they used to)
hurg

happens for ftling to planets too, you can spawn inside a rock

The planet issue is something else entirely.


Implement DOI support and minting

Initially review how DOIs are handled today. Is this in a workflow step, in H2 somewhere, etc?

All we need to do is call Sdr::Repository.register with assign_doi: true. The rest will just work.
https://github.com/sul-dlss/hungry-hungry-hippo/blob/main/app/services/sdr/repository.rb#L31

I recall that we thought about this when it was built and intentionally did it in a way that was (hopefully) easily reusable. Our future selves are now very happy with this decision!

Mockup for the collection form where these settings are chosen: https://www.figma.com/design/g46Nfl5EHG7DfwqupK3G4X/H3-2024?node-id=789-4224&t=9soX0awq2YXWVQRo-4
Mockup with all the use cases for the deposit form. Just click on the "A DOI will/not be assigned text" to see the other use cases. https://www.figma.com/proto/g46Nfl5EHG7DfwqupK3G4X/H3-2024?page-id=1092%3A2227&node[…]width&content-scaling=fixed&starting-point-node-id=1114%3A2853

Note the mockup where the user gets to choose is awaiting one more piece of text to be added.


Add "connection to shape" anchor

We can currently anchor to the various X points around a shape but there is no anchor to shape as internally used in the "connect and clone" feature.  I'd like the ability to anchor to the shape so if I move the shape, the best anchoring point is automatically selected.

Added in <IP_ADDRESS>. Use snapToPoint=1 in the cell or connecting edge style.


are my targets correct?

https://github.com/department-of-reuse/DoR/blob/main/workflow/targetted_venues.md
please check

I think so - we could make a tiered / prioritized approach, ICSE/ASE/FSE, followed by ICSME etc. I think we talked about this before but some out of band samples would be interesting to pull in weird outliers (e.g., CAISE, ICSOB, MISQ, ICML)

complete 2020
move on to 2021


Error structure format

As a developer, I'd like a better Error structure from the GnoNative library.
For example, when I have to deal with an Error (eg: ErrUnknownAddress) thrown by GnoNative and I have to convert it to JSON, I get:
{"name": "ConnectError", "rawMessage": "ErrUnknownAddress(#206)", "code":2, "metadata":{"map":{}}, "details":[]}

It would be great if we had a better structure, eg:
{
  "type": "ErrUnknownAddress",
  "code": "206",
  "message": "The address provided is unknown on the blockchain.",
  "documentation_url" : "[https://gno.land/api/docs/errors/ERR-206](https://gno.land/api/docs/errors/ERR-206)"
 }

There is already a JavaScript error data structure. @D4ryl00 and @iuricmp to coordinate to try it.

Wait for the Gno devs to update GRPC to return JSON results and see if the panic response also has helpful JSON.


Ring Api seems down

<
-->
Home Assistant release with the issue:0.68.1

Last working Home Assistant release (if known):
**Operating environment (Hass.io/Docker/Windows/etc.):Raspian **

Component/platform:Ring

Description of problem:
I am trying to add Ring, when i am restart the service i got:
https://api.ring.com/clients_api/session
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

Traceback (if applicable):

Additional information:

Can confirm a similar issue. None of my configuration has changed. It would appear they have done something on their end.

This is apparently an issue due to Ring updating their authentication mechanism for oAuth. @tchellomello authors the library that we use, and has already acknowledged the issue. (Thanks!)
https://github.com/tchellomello/python-ring-doorbell/issues/94

same here :cry:

Yes, it's a known issue as Ring changed their authentication method we will need to update our library.

Guys, please forgive me, but I'm migrating over to the Ring ecosystem from Nest and this api wrangling is new to me. Could someone please help point me in the right direction to setting this up?
I'm hoping my end goal will be iSpy integration and Samsung Smart things since that's what everything else is in our house.
Truly appreciate any help you can offer!

@jo3see currently the Ring is broken within Home Assistant but it should be fixed as we are finishing the PR to address this problem and some users already reported the PR fixed the issue. See more at https://github.com/tchellomello/python-ring-doorbell/issues/94
I will finish the unittests for the new PR and then we can merge and update it within HA too.


fix: Update firestore class to accept the databaseId from the configuration file

Update the firestore class to accept the databaseId.

As the default the @google-cloud/firestore is using the "(default)" named firestore database if we don't set databaseId field.
Set the default database Id as "(default)" explicitly when there are no databaseId field originated from the cluster.stateDatabase configuration.

@henrybell Would you check this PR?

Hi @changsoo621 -- thanks for the PR! There are a couple of points to mention:

Before any commits can be merged to a Google repo, you'll need to (digitally) sign the CLA, which is checked as part of our CI.

I have some upcoming and related changes to the Terraform configuration that will also be needed to put this change live, as well as to the documentation. I should be able to get this in this week. I think it may make sense to combine these two changes to make sure they play nicely together.

So that I can make sure this fits with your requirements, could I please ask you to raise an issue on this repo that describes the aim? Once we are aligned and you have signed the CLA I will add you as a co-author and get the changes submitted.
Thanks again!

Thank you @henrybell, however I cannot sign in to the CLA. Would you help me?

Thanks @changsoo621, I will look into this and follow up here. Thanks for your patience.

Hi @changsoo621, I created this draft PR that incorporates:remove App Engine and includes the functionality

Hi @changsoo621, I created this draft PR that incorporates:

Removing App Engine dependency for Firestore and Cloud Scheduler
Adding configurable Firestore database ID (i.e. the functionality of this PR)
Updating tests, documentation, and other config

You are added as co-author on the PR -- many thanks for raising this.
I still need to discuss this with my colleagues, but it would be great if you could please check that it meets your requirements. Thanks!

Thank you @henrybell I will check the PR and review it!

Closing as addressed in #51.


Register Package

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/113254
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v1.0.0 -m "<description of version>" 5e1fc778d7d095ed62bb8757b9ee7fbf6e6a66ac
git push origin v1.0.0


Where's your copyright? :(

Hello there,
This template looks neat!
The MIT license requires users (us) to give you credit.

Conditions: License and copyright notice

I haven't seen your name anywhere in your demo's source-code.
How would you like to be thanked? :-) imo it's possible to have minified files with a copyright header, would this be enough?
Cheers!

Thanks! Sorry about the confusion regarding the license and copyright. I've pushed a change that preserves the top comment in the JS/CSS and I added a bit of info in there. That's more than enough for me as far as credit goes. Hope that clears it up!


fix: plonk scs serialization issues

panic: error reading scs : cbor: exceeded max number of elements 134217728 for CBOR array

--> seems now we can do<PHONE_NUMBER> as max size, easy fix, lucky for us :)

panic: cbor: cannot unmarshal map into Go struct field cs.system.Blueprints of type constraint.Blueprint

--> suspect this comes from BlueprintLookupHint
We need to figure out a better way to serialize blueprints, this fix should do for now, but we need to re-serialize the circuits (cc @AlexandreBelling )

To confirm, looks good to me. I guess it is not so bad that lookup blueprint is in constraint package. Maybe we want to put the blueprints into separate package in the future so that constraint package wouldn't be too big.


Why is font selection {left, center, right}?

usage: python-escpos set [-h] [--align {left,center,right}]
                         [--font {left,center,right}]
                         [--text_type {B,U,U2,BU,BU2,NORMAL}] [--width WIDTH]
                         [--height HEIGHT] [--density DENSITY]
                         [--invert INVERT] [--smooth SMOOTH] [--flip FLIP]
python-escpos set: error: argument --font: invalid choice: '1' (choose from 'left', 'center', 'right')

Why is font selection "left/center/right" and not "0/1"?

python-escpos version: 3.2.dev16+g99501cc
python version: CPython v3.11.2
platform: Linux-6.6.31+rpt-rpi-v8-aarch64-with-glibc2.36
printer driver USB is usable: True
printer driver File is usable: True
printer driver Network is usable: True
printer driver Serial is usable: True
printer driver LP is usable: True
printer driver Dummy is usable: True
printer driver CupsPrinter is usable: True
printer driver Win32Raw is usable: False

Hi @gloriouslyawkwardlife and thanks for the report!!.
You are right, this is certainly a typo in the list of valid arguments in the set sub-command of the CLI.
IMHO although the set() method in the API admits both the index based and the literal based arguments for the font parameter, the arguments of the --font parameter of the set sub-command in the CLI should be the literal based 'A' | 'B' instead of 0 | 1 to keep consistency with the same parameter in other sub-commands. See barcode as an example where the valid arguments for the font parameter are in fact the strings 'A' or 'B'.
Are you OK with this @patkan @gloriouslyawkwardlife ?

I should have said 'A'/'B' after reading the printer docs; too used to booleans. LOL. But "left/center/right" just doesn't make sense, especially when in the library itself the text_block function calls for using A or B.


Workday to AD provisioning custom attribute issue

The Azure AD provisioning service supports the ability to customize your list or Workday attribute to include any attributes exposed in the Get_Workers operation of the Human Resources API.

In one case we need workday attribute which is not available in Get_Workers menthod of Human Resource API from workday. How can we manage this case ? How to handle such technical constraints.
We need to sync that attribute from Workday to on Prem AD. @cmmdesai

Reference link:
https://docs.microsoft.com/en-us/azure/active-directory/saas-apps/workday-inbound-tutorial#customizing-the-list-of-workday-user-attributes

@AmitavaHazra
Thanks for your feedback! We will investigate and update as appropriate.

@AmitavaHazra Please check with your Workday subject matter expert if the attribute you want in Get_Workers response can be exposed using a Workday concept called integration system attribute.

There's a related feedback item as well that you can follow. https://feedback.azure.com/forums/169401-azure-active-directory/suggestions/36448306-support-for-workday-integration-system-custom-at

@cmmdesai,
Workday SME customized request call to Get_workers process to get custom attribute.
This is something you had feedback for MS. Can you please tell me whether Azure user provisioning service can make this customized request to API to retrieve custom attribute ? Is this something already supported in this solution.??
https://feedback.azure.com/forums/169401-azure-active-directory/suggestions/36448306-support-for-workday-integration-system-custom-at
wd:RequestCriteria
wd:FieldAndParameterCriteriaData
wd:ProviderReference
<wd:ID wd:type="IntegrationSystemID">YOURINTEGRATIONID</wd:ID>
</wd:ProviderReference>
</wd:FieldAndParameterCriteriaData>


[bundle-size] Convert to TypeScript

Actually caught a few bugs this way!
Semantic changes:

Convert to esm code
Upgraded probot to v13
Database retrieval method now does the type coarsening from string to the relevant types, instead of having the callers do it
Correctly return from the /report endpoint if it sends a 202, without accidentally sending a 500 after the response has closed

Testing changes:

Changed the signature of installApiRouter to make it easier to test without having to raise an entire Probot app (only Express)
Avoid using jest.mock, which has issues with esm modules
Import JSON fixtures directly instead of doing fs.readFile

GitHub's diff algo struggles with connecting the .js and .ts files, I ran difftastic with -M10% to ensure most files get connected correctly and put the result in this zipped HTML file (GitHub won't let me upload an .html file): difft.zip

Tested the happy paths by deploying to gcloud (not promoted to prod)

This version fails on a database connection issue. I reverted to the previous version and am working on a fix


Fix failed java build

I think this was a bug from d4503a1f0e62d38f7a12ba9748744c5490c915c4.

[ ] Did you create an Apache Jira ticket?  (Request account here, not required for trivial changes)
[ ] If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
[ ] Did you squash your changes to a single commit?  (not required, but preferred)
[ ] Did you do your best to avoid breaking changes?  If one was needed, did you label the Jira ticket with "Breaking-Change"?
[ ] If your change does not involve any code, include [skip ci] anywhere in the commit message to free up build resources.

It looks like that commit also broke something else which is fixed in https://github.com/apache/thrift/pull/3017/files#diff-dac985b5f3c62d122459132ffb79349f124eaf7f54eea6eba1d3b9d19b4d6a33R640-R644

It looks like that commit also broke something else which is fixed in https://github.com/apache/thrift/pull/3017/files#diff-dac985b5f3c62d122459132ffb79349f124eaf7f54eea6eba1d3b9d19b4d6a33R640-R644

Thanks. Since that PR is not merged yet, I'm going to incorporate that part of the fix into this PR instead.

cc @emmenlau I listed you as co-author as you did the part of fix in 1a4bf78ad09841c3c84bfed7826b871f4dfc9876


Allow hiding available tags in NavigatorCard.vue

Bug/issue #, if applicable: 100278813
Summary
Allows hiding any available tags in the NavigatorCard search
Dependencies
NA
Testing
To test this you would need to extend the NavigatorCard and apply the hideAvailableTags: true prop.
Assert no tags are shown, no matter what navigator items you have.
Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran npm test, and it succeeded
[x] Updated documentation if necessary

@swift-ci test

@swift-ci test

@swift-ci test


Luck mod not affecting monster crits

Edit: Seem to work fine/ as intended. Sorry. Turns out that if after a monster rolls a crit vs a PC, if I have the PC token selected when click the crit button on the chat card, it rolls the crit as the selected token, no the monster that originally made that attack. So the luck mod is not applied in case that because the selected token's target didn't have a luck mod! I'll just have to be careful to make sure I keep the monster's token selected before I make its crit roll.

Please install manually with the following URL  and test it. Let me know the result.

Fixed in 0.13


[C++] Use native PEXT on AMD Zen 3

According to various sources, BMI2 instructions PDEP and PEXT are not microcoded anymore on AMD Zen 3 CPUs, meaning we don't need the software emulation there.
Reporter: Antoine Pitrou / @pitrou
Note: This issue was originally created as ARROW-10548. Please see the migration documentation for further details.

@felipecrv

The main challenge is to find a Zen 3 CPU to validate the change, otherwise this should be easy enough.

@jgehrcke Do we have a machine with a recent AMD CPU in our benchmarking fleet?


Free text filter should be made less prominent in annotation search

E.g., at least make it only one row. Perhaps make it something you explicitly need to open
There are arguments for removing altogether. Some arguments for keeping it:
1 Users who are not steeped in the data models of the GOC may not know exactly how to find what they are looking for. They may just have a string that matches terms, families, genes. They may even be hazy about the distinctions here. But this may be better served in what amigo calls 'medial search'. Note that medial search for http://amigo.geneontology.org/amigo/medial_search?q=foo now provides links for users to pages like http://amigo.geneontology.org/amigo/search/annotation?q=foo which use the free text filter. I think this needs supported, but this can be done without the box.
2 We used to be able to type solr queries in this box. I loved this, and maybe there's a handful of others who would love this. But better served by an optionally opened 'advanced' search
3 Sometimes it's just faster to get at what you want the unstructured route. Facet drilling can be tedious.
Note that this all applies to annotation search. It's obviously more useful when you are searching through a list of things (terms, genes) than associations

See also: http://jira.geneontology.org/browse/GO-688

As a simple initial compromise, the textbox could be hidden at first, opened after the user clicks "filter by free text"

On 07/01/2015 19:33, Chris Mungall wrote:

See also: http://jira.geneontology.org/browse/GO-688
—
Reply to this email directly or view it on GitHub
https://github.com/kltm/amigo/issues/158#issuecomment-69077667.

ah was I using a free text search?...I wassure that I selected a GO ID
from the options which popped up....will try that again

On 07/01/2015 19:35, Chris Mungall wrote:

As a simple initial compromise, the textbox could be hidden at first,
opened after the user clicks "filter by free text"
—
Reply to this email directly or view it on GitHub
https://github.com/kltm/amigo/issues/158#issuecomment-69078083.

Hi Chris,
I definitely wasn't doing a free text search
I started from here
go.geneontology.org/amigo
typed cellular_component and selected
and selected the option(cellular_component GO:0005575)
Amigo then says "forwarding to GO:0005575"
Then I selected "link to all direct and indirect annotations to
"cellular component"
so I'm sure that I  shouldn't have any MF or BP annotation in my results?
VAl

Val, you "typed cellular_component " ; that's the key: typed; it treats it as free text, with and without the "_".

I'm still catching up, but http://jira.geneontology.org/browse/GO-688 seems to be a separate issue from this one. My druthers would be to delete this last set of comments.

Based on observations from @ValWood and comments from @dosumis I have categorized this as a bug that "affects usability"


bump and log have no subcommands

Description
Currently, cider bump <part> and cider log <type> are not subcommands. Thus, something like cider bump --help doesn't print the correct information.
You can take a look at how the change looks like here: https://github.com/marvin-kolja/cider/commit/473acd1786e0ed614294c793a43a11d9bf782b6f
Current Behaviour
$ dart bin/cider.dart bump --help
Bump project version

Usage: cider bump [arguments]
-h, --help               Print this usage information.
    --[no-]keep-build    Keep the existing build
    --[no-]bump-build    Also bump the build
    --build              Sets the build to the given value
                         (defaults to "")
    --pre                Sets the pre-release to the given value
                         (defaults to "")

Run "cider help" to see global options.

Wanted behaviour
$ dart bin/cider.dart bump --help
Bump project version

Usage: cider bump <subcommand> [arguments]
-h, --help    Print this usage information.

Available subcommands:
  breaking   Bump the breaking version
  build      Bump the build version
  major      Bump the major version
  minor      Bump the minor version
  patch      Bump the patch version
  pre        Bump the pre-release version
  release    Bump the release version

Run "cider help" to see global options.

$ dart bin/cider.dart bump breaking --help
Bump the breaking version

Usage: cider bump breaking [arguments]
-h, --help               Print this usage information.
    --[no-]keep-build    Keep the existing build
    --[no-]bump-build    Also bump the build
    --build              Sets the build to the given value
                         (defaults to "")
    --pre                Sets the pre-release to the given value
                         (defaults to "")

Run "cider help" to see global options.

Thank you. The change you're suggesting seems to be a nice UX improvement, wanna open a PR?

Fantastic, I'll do that.


Pretty print HTML

Our storybook Render component outputs ugly HTML.

Can I work on this?

Yeah @tanvinerkar

Looking here might help: https://github.com/vuejs-norte/puto-framework/blob/master/stories/Render.js


[DOC]: Wiki & README

Describe the solution you'd like
A simple to follow and clear readme to guide the user how to install and launch the application.
Additional context
Wiki should be more detailed written with GitHub Wiki page for the current repository.
Tasks

[ ] README.md
[ ] Wiki Structure

 Wiki & ReadMe

 README.md


Clamp BackgroundHiveSplitLoader concurrency to usable parallelism

Avoids allocation a greater level of concurrency to split loading than can actually be used based on the number of partititions being loaded. The effective usable concurrency in split loading can be significantly lower than the configured split loader concurrency value when the number of partitions scanned is small, since split loading is single threaded within each partition anyway. This makes it feasible to configure a much higher concurrency value than before without overcommitting threads to loading tasks unnecessarily.

merged, thanks!


Ejecting create-react-app causes babel-loader to be missing

Describe the bug
When ejecting a create-react-app project, running the dev server produces this error:

To Reproduce

Create a new "Vanilla React" project
Eject
Start development server

Expected behavior

It should run the server without falling over!

Environment (please complete the following information):

OS: MacOS
Version: 0.2.0
Node version: 8.11.1 (NPM v. 6.1.0), as well as Node 10.8.0 / NPM 6.4.0

Additional context
I originally thought this was an NPM bug, but we switched to Yarn, so I'm guessing it's that eject doesn't automatically reinstall dependencies... and it should, so I'm not sure what's up with that.

Whoops, this is a duplicate of https://github.com/joshwcomeau/guppy/issues/68


After update of an AKS cluster, Keda is unable to connect to the Kubernetes API

Report
After updating the k8s cluster to a new version (from 1.22.5 to 1.22.6), the keda-metrics pod is unable to connect to the kubernets api. The debug log shows an
I0328 14:44:00.734194       1 main.go:156] keda_metrics_adapter "msg"="KEDA Version: 2.6.1"
I0328 14:44:00.734467       1 main.go:157] keda_metrics_adapter "msg"="KEDA Commit: efca71d6bc770408468a9e1a4b3984f7136c0967"
I0328 14:44:00.734553       1 main.go:158] keda_metrics_adapter "msg"="Go Version: go1.17.3"
I0328 14:44:00.734626       1 main.go:159] keda_metrics_adapter "msg"="Go OS/Arch: linux/amd64"
I0328 14:44:00.742713       1 with_retry.go:171] Got a Retry-After 1s response for attempt 1 to https://<IP_ADDRESS>:443/api?timeout=32s
I0328 14:44:01.745991       1 with_retry.go:171] Got a Retry-After 1s response for attempt 2 to https://<IP_ADDRESS>:443/api?timeout=32s
I0328 14:44:02.749603       1 with_retry.go:171] Got a Retry-After 1s response for attempt 3 to https://<IP_ADDRESS>:443/api?timeout=32s
I0328 14:44:03.754102       1 with_retry.go:171] Got a Retry-After 1s response for attempt 4 to https://<IP_ADDRESS>:443/api?timeout=32s
I0328 14:44:04.757609       1 with_retry.go:171] Got a Retry-After 1s response for attempt 5 to https://<IP_ADDRESS>:443/api?timeout=32s
I0328 14:44:05.761270       1 with_retry.go:171] Got a Retry-After 1s response for attempt 6 to https://<IP_ADDRESS>:443/api?timeout=32s
I0328 14:44:06.764257       1 with_retry.go:171] Got a Retry-After 1s response for attempt 7 to https://<IP_ADDRESS>:443/api?timeout=32s
I0328 14:44:07.767366       1 with_retry.go:171] Got a Retry-After 1s response for attempt 8 to https://<IP_ADDRESS>:443/api?timeout=32s
I0328 14:44:08.770036       1 with_retry.go:171] Got a Retry-After 1s response for attempt 9 to https://<IP_ADDRESS>:443/api?timeout=32s
I0328 14:44:09.773508       1 with_retry.go:171] Got a Retry-After 1s response for attempt 10 to https://<IP_ADDRESS>:443/api?timeout=32s
E0328 14:44:10.776583       1 main.go:100] keda_metrics_adapter "msg"="unable to construct new client" "error"="Get \"https://<IP_ADDRESS>:443/api?timeout=32s\": EOF"
E0328 14:44:10.777143       1 main.go:218] keda_metrics_adapter "msg"="making provider" "error"="unable to construct new client (Get \"https://<IP_ADDRESS>:443/api?timeout=32s\": EOF)"
E0328 14:44:10.777175       1 main.go:177] keda_metrics_adapter "msg"="unable to run external metrics adapter" "error"="unable to construct new client (Get \"https://<IP_ADDRESS>:443/api?timeout=32s\": EOF)"
I0328 14:44:56.589822       1 main.go:156] keda_metrics_adapter "msg"="KEDA Version: 2.6.1"
I0328 14:44:56.589880       1 main.go:157] keda_metrics_adapter "msg"="KEDA Commit: efca71d6bc770408468a9e1a4b3984f7136c0967"
I0328 14:44:56.589887       1 main.go:158] keda_metrics_adapter "msg"="Go Version: go1.17.3"
I0328 14:44:56.589892       1 main.go:159] keda_metrics_adapter "msg"="Go OS/Arch: linux/amd64"
I0328 14:44:56.593960       1 with_retry.go:171] Got a Retry-After 1s response for attempt 1 to https://<IP_ADDRESS>:443/api?timeout=32s
I0328 14:44:57.596947       1 with_retry.go:171] Got a Retry-After 1s response for attempt 2 to https://<IP_ADDRESS>:443/api?timeout=32s
I0328 14:44:58.600431       1 with_retry.go:171] Got a Retry-After 1s response for attempt 3 to https://<IP_ADDRESS>:443/api?timeout=32s
I0328 14:44:59.604105       1 with_retry.go:171] Got a Retry-After 1s response for attempt 4 to https://<IP_ADDRESS>:443/api?timeout=32s

We also completely removed keda, including crd's and reinstalled keda, but this error remains. It seems that the Keda operator is depended on the metrics pod.
Expected Behavior
Able to connect and get Keda up and running
Actual Behavior
Error in the log, restarting pod(s)
Steps to Reproduce the Problem

Update an AKS cluster
Check the pods
Check the logs

Logs from KEDA operator (Debug)
1.64847757568208e+09	ERROR	Failed to get API Group-Resources	{"error": "Get \"https://<IP_ADDRESS>:443/api?timeout=32s\": EOF"}
sigs.k8s.io/controller-runtime/pkg/cluster.New
	/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.0/pkg/cluster/cluster.go:160
sigs.k8s.io/controller-runtime/pkg/manager.New
	/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.0/pkg/manager/manager.go:313
main.main
	/workspace/main.go:86
runtime.main
	/usr/local/go/src/runtime/proc.go:255
1.6484775756822596e+09	ERROR	setup	unable to start manager	{"error": "Get \"https://<IP_ADDRESS>:443/api?timeout=32s\": EOF"}
main.main
	/workspace/main.go:96
runtime.main
	/usr/local/go/src/runtime/proc.go:255

KEDA Version
2.6.1
Kubernetes Version
1.22
Platform
Microsoft Azure
Scaler Details
azure-pipelines
Anything else?
No response

Can you check and see if another external metrics adapter (or if any external metrics adapter) is installed? kubectl get apiservice | grep external.metrics - may be related to the issue that only one can be configured per cluster #470

Hi @jeffhollan,
Thanks, this is the output from the kubectl command, so just one is running:
v1beta1.external.metrics.k8s.io        keda/keda-metrics-apiserver   False (MissingEndpoints)   15h

That missing endpoint concerns me.

Looking in to this.

@Evantage-WS
I tried replicating this issue but failed. Metrics and operator both seem to be healthy. Below is my precise versions info
KEDA Version: 2.7.1
Go Version: go1.17.9
Kubernetes 1.22  {"version": "v1.22.6"}
I understand you've mentioned you've updated aks cluster's version from 1.22.5 to 1.22.6. Does the same issue occur if your cluster is created directly with 1.22.6 version?

I have raised this issue over a month ago, so we removed this cluster and reïnstalled it. From that moment on it is working again. Because of the lack of support on Keda we are considering another solution.

For what it's worth, it's an open source product and this felt through the cracks as we have a lot of request. Feel free to nudge us again in the future.

I really understand this, but it has been 6 weeks between the answers................................

Well it was opened on March 28 and last reply was on 29th, feel free to bump next time because we get a lot of requests and sometimes things fall through the cracks :)
Sorry.

If you are on Azure and want support, you can use the AKS add-on for KEDA: https://docs.microsoft.com/en-us/azure/aks/keda-about

Hello,
I recently had the same symptom when upgrading to AKS 1.22.6 version.
With that I'm having problems with keda, I'm not able to remove it also due to v1beta1.external.metrics.k8s.io
Erro.......
ramps@NT0876526MT1580:~$ helm uninstall keda -n keda
Error: uninstallation completed with 1 error(s): unable to build kubernetes objects for delete: resource mapping not found for name: "v1beta1.external.metrics.k8s.io" namespace: "" from "": no matches for kind "APIService" in version "apiregistration.k8s.io/v1beta1"
ensure CRDs are installed first
ramps@NT0876526MT1580:~$ kubectl get apiservice | grep external.metrics
v1beta1.external.metrics.k8s.io        keda/keda-operator-metrics-apiserver   True        690d
How could I get around this situation without failing to break anything else?
How can I remove keda using helm and try to do a new clean and fresh install?

After upgrading from K8s 1.21 to 1.22.11 I observed the following error when trying to run a third-party tool locally (popeye):

unable to retrieve the complete list of server APIs: external.metrics.k8s.io/v1beta1: the server is currently unable to handle the request

After digging around I came up with the following NetworkPolicy which resolved my problem.
apiVersion: networking.k8s.io/v1
kind: NetworkPolicy
metadata:
  name: keda-metrics-adapter-ingress
  namespace: my-ns
spec:
  ingress:
  - from:
    - namespaceSelector:
        matchLabels:
          control-plane: "true"
          kubernetes.io/metadata.name: kube-system
      podSelector:
        matchLabels:
          component: tunnel
  podSelector:
    matchLabels:
      app.kubernetes.io/component: metrics-adapter
      app.kubernetes.io/name: keda
  policyTypes:
  - Ingress

I'm not sure if this helps the issue reported, but I thought I'd share so it might help someone.


Bug in loading-unloading dependent fixtures with foreign keys

From docs
As we described earlier, a fixture may depend on other fixtures. For example, a UserProfileFixture may need to depends on UserFixture because the user profile table contains a foreign key pointing to the user table. The dependency is specified via the yii\test\Fixture::$depends property, like the following,
It was so when $this->resetTable(); was in unload method
We loaded fixtures in load method, and reversly unloaded it in unload method.
So there were no problems with foreign keys
Then $this->resetTable(); was moved in  load method. We in default order we try to resetTable.
So dependent don't work properly

related to #5442

Duplicates #5442


fix: upgrade go-releaser config

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

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

Details

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 90.909%

Totals

Change from base Build<PHONE_NUMBER>4:
0.0%

Covered Lines:
450

Relevant Lines:
495

💛  - Coveralls


Add a post-build step that project-specfic code can use

This is first to support building the cosmosis standard library inline in the image.

Let's try this!


Add pull request options and change view-profile, list-repos

Add a pull request option with checkout locally, merge and comment option #19
handle null cases in view profile where bio etc is not added
pagination has been removed from list-repos as per #20

Great work.


ERROR: We detected an unhandled promise rejection. The server will be terminated. The error message was:  Cannot read property 'replace' of undefined

OpenTest version number: 1.1.4

Server process throws:
"ERROR: We detected an unhandled promise rejection. The server will be terminated. The error message was:  Cannot read property 'replace' of undefined"
As soon as I try to create a test session from an existing template saved in '...\opentest\test-repo'
Steps to reproduce:

Create subfolder 'templates' under '...\opentest\test-repo'
Create new template saved as 'smoke_tests.yaml' with content:
description: Smoke tests
sessionLabel: Smoke tests
tests:
- name: Todoist_login
- path: .
In dedicated CMD, launch server process with: "opentest server server"
In dedicated CMD, launch actor process with: "opentest actor"
Refresh localhost:3000 tab browser,
Click Session > Create Session From Template...
Select visible "smoke_tests" option then click "CREATE SESSION"

Expected Results:
a) A new testing session is launched and successfully executed
b) No error is thrown in the server's dedicated CMD
c) No error is thrown in the actor's dedicated CMD
Actual Results:
a) No new testing session is created; upon opening again the Session > Create Session From Template... option, the popup window shows a continuous loading icon
b) server CMD throws:
please see 'Relevant log information:' section
c) actor's CMD throws:
"Warning: failed to communicate with sync server. Original error was: Connect to localhost:3000 [localhost/<IP_ADDRESS>, localhost/0:0:0:0:0:0:0:1] failed: Connection refused: connect"
Observations:

Repeated restarts of the server and actor processes does not stop the issue from reoccurring.
Existing number of test sessions displayed in localhost:3000 is currently 40, some of them are failed, some passed.

Relevant log information:
C:\Users\jxkhr\Documents\opentest>opentest server server
OpenTest server 1.1.4, commit b66f5666d4ff411b239b3d82efe727fdbf241cf0
Running in working directory "C:\Users\jxkhr\Documents\opentest\server"
Using test repo directory "C:\Users\jxkhr\Documents\opentest\test-repo"
The OpenTest server is listening on port 3000
Open a browser and navigate to http://localhost:3000
Established WebSocket connection for session Y2n6H94qiig2iYm7DV7wiGvdtavfscBU
ERROR: We detected an unhandled promise rejection. The server will be terminated. The error message was:  Cannot read property 'replace' of undefined
TypeError: Cannot read property 'replace' of undefined
at C:\Users\jxkhr\AppData\Roaming\npm\node_modules\opentest\node_modules\opentest-server\dist\lib\test-repo.js:177:45
at Array.forEach ()
at Object. (C:\Users\jxkhr\AppData\Roaming\npm\node_modules\opentest\node_modules\opentest-server\dist\lib\test-repo.js:176:17)
at Generator.next ()
at C:\Users\jxkhr\AppData\Roaming\npm\node_modules\opentest\node_modules\opentest-server\dist\lib\test-repo.js:7:71
at new Promise ()
at __awaiter (C:\Users\jxkhr\AppData\Roaming\npm\node_modules\opentest\node_modules\opentest-server\dist\lib\test-repo.js:3:12)
at Object.getTestsForTemplate (C:\Users\jxkhr\AppData\Roaming\npm\node_modules\opentest\node_modules\opentest-server\dist\lib\test-repo.js:161:12)
at C:\Users\jxkhr\AppData\Roaming\npm\node_modules\opentest\node_modules\opentest-server\dist\controllers\api.js:249:48
at Generator.next ()

Thank you for the very detailed issue report. I confirmed this issue is caused by missing error handling logic. It's not a critical issue, since the error is triggered by using an incorrect test session template file. The issue is happening when a test session template fails to specify the name for a test. The test session template you used is
description: Smoke tests
sessionLabel: Smoke tests
tests:
  - name: Todoist_login
  - path: .

This template actually defines 2 tests: one with name Todoist_login and no path property, and one with just the path property and no name. It's basically equivalent with this:
description: Smoke tests
sessionLabel: Smoke tests
tests:
  - name: Todoist_login
    path: undefined

  - name: undefined
    path: .

When the OpenTest server code tries to parse the second test, it fails because of the missing name property. I'm adding error handling code, but please note that this error can only happen while using an incorrect test session template file.
Thanks again for the bug report - it's one of the better ones I've seen, for sure.

Thank you as well Adrian!
Could you please give me a hint on how should I properly define the template file to describe only one test for instance? :-)

A template file with only one test looks like this:
description: Smoke tests
sessionLabel: Smoke tests
tests:
  - name: Todoist_login
    path: smoke-tests

When adding multiple tests, you need one - for each test, like this:
description: Smoke tests
sessionLabel: Smoke tests
tests:
  - name: Test 1
    path: smoke-tests

  - name: Test 2
    path: smoke-tests

Or, you can specify the tests by their tags, like this:
description: Smoke tests
sessionLabel: Smoke tests
includeTestsWithTags: smoke

And, of course, for this to work, you must add the smoke tag to your smoke tests, like this:
description: First smoke test
tags: [smoke]
...

You can find more information here. Is this what you were looking for?

You can find more information here. Is this what you were looking for?

Yes, thank you Adrian, I've realized my "-" typo next to "path" was causing the error.
All the best!