Server inherited from ExperimentalCallbackService not responce

What version of gRPC and what language are you using?
1.23.0
What operating system (Linux, Windows,...) and version?
Windows 10 x64
What runtime / compiler are you using (e.g. python version or version of gcc)
MSBuilder (VS2017)
What did you do?

Get helloword.proto from example and replace service Greeter to this:

service Greeter {
  rpc TestUnary (HelloRequest) returns (HelloReply) {}
  rpc SayHelloEverySecond (HelloRequest) returns (stream HelloReply) {}
}

Generate source files
Client code:

#include <iostream>
#include "grpc++/grpc++.h"
#include "helloworld.grpc.pb.h"

using namespace helloworld;
using namespace grpc;

int main()
{
  auto channel = CreateChannel("localhost:50051", grpc::InsecureChannelCredentials());
  auto stub = helloworld::Greeter::NewStub(channel);
  std::unique_ptr<ClientContext> context = std::make_unique<ClientContext>();
  HelloRequest request;
  request.set_name("hello");
  HelloReply reply;
  auto status = stub->TestUnary(context.get(), request, &reply);
  context = std::make_unique<ClientContext>();
  auto reader = stub->SayHelloEverySecond(context.get(), request);
  while(reader->Read(&reply)){
    std::cout << reply.message() << "\n";
  }
  int val;
  std::cin >> val;
}

Server code:

#include <iostream>
#include "grpc++/grpc++.h"
#include "helloworld.grpc.pb.h"

using namespace helloworld;
using namespace grpc;

class GreeterServiceImpl2 final : public Greeter::Service {
  virtual ::grpc::Status TestUnary(::grpc::ServerContext*, const HelloRequest*, HelloReply* response) override {
    response->set_message("TestUnary from Server");
    return grpc::Status::OK;
  }

  virtual ::grpc::Status SayHelloEverySecond(::grpc::ServerContext*, const HelloRequest*,
                                             ::grpc::ServerWriter<HelloReply>* writer) override {
    HelloReply response;
    response.set_message("SayHelloEverySecond from Server");
    writer->Write(response);

    return grpc::Status::OK;
  }
};
class GreeterServiceImpl final : public Greeter::ExperimentalCallbackService {
  virtual ::grpc::Status SayHelloEverySecond(::grpc::ServerContext* context, const HelloRequest* request,
                                             ::grpc::ServerWriter<::helloworld::HelloReply>* writer) override {
    throw std::logic_error("Not implemented. And never be thrown");
  }

  virtual ::grpc::experimental::ServerWriteReactor<HelloRequest, HelloReply>* SayHelloEverySecond() override {
    throw std::logic_error("Not implemented.  And never be thrown");
  }

  virtual void TestUnary(::grpc::ServerContext* context, const HelloRequest* request, HelloReply* response,
                         ::grpc::experimental::ServerCallbackRpcController* controller) override {
    throw std::logic_error("Not implemented. And never be thrown");
  }
};

int main() {
  GreeterServiceImpl service;
  ServerBuilder builder;

  builder.AddListeningPort("<IP_ADDRESS>:50051", grpc::InsecureServerCredentials());
  builder.RegisterService(&service);
  std::unique_ptr<Server> server(builder.BuildAndStart());
  int val = 0;
  std::cin >> val;
}

GreeterServiceImpl2 works as expect. But when I try to use GreeterServiceImpl as service in main(), server successfully start with message I1028 12:26:24.451000000 11832 server_builder.cc:318] Callback server., but no one overloaded method in GreeterServiceImpl never calls and client hangs in line auto status = stub->TestUnary(context.get(), request, &reply);

What did you expect to see?
Methods in GreeterServiceImpl should be called after client invoked it.
What did you see instead?
Client hangs on trying to call server code and server service not work as expected

Hope, can you take a look please?

Reproduced with 1.25.0-pre1

Taking a brief look at this again, the experimental callback service would need the event manager poller to be working but we don't have that yet, so I guess this is expected.


Update instructions for adding folks to the team page in wordpress

Migrated from GitLab: https://gitlab.com/meltano/handbook/-/issues/51
Originally created by @afolson on 2022-01-25 19:44:21

Looks like there's a few things that have to happen: 1) add their profile to the "Team" section in the sidebar 2) Add their team profile to the actual team section on the about page 3) Sort the newly added name alphabetically by dragging it to the right place

View 3 previous comments from the original issue on GitLab


Upgrade nolus to v0.6.3

Please pull latest master branch into here, and then add the required properties you're missing (ibc, sdk, cosmwasm, etc.))


esbuild failed to process css background images

Command
build
Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
Error while processing images as css background images
Minimal Reproduction
place any image with name "timeline.png" to assets folder

.container {
  width: 300px;
  height: 300px;
  border: 1px solid red;
  background-image: url("~/src/assets/timeline.png")
}

Works perfectly with default builder, but browser-esbuild could not resolve it
Exception or Error
Could not resolve "~/src/assets/timeline.png" [plugin angular-css-resource]

    apps/esbuild-app/src/app/app.component.css:6:24:
      6 │   background-image: url("~/src/assets/timeline.png")
        ╵                         ~~~~~~~~~~~~~~~~~~~~~~~~~~~

  You can mark the path "~/src/assets/timeline.png" as external to exclude it from the bundle, which will remove this error.

I use fresh nx project
Looks like same problem as #24470

The linked issue (https://github.com/angular/angular-cli/issues/24470) is actually a bug regarding the Webpack-based builder.  The Webpack-based builder should work with the second variable based method example but does not whereas the esbuild-based builder will work with both examples from the issue.
As to this particular use case, the path (~/src/assets/timeline.png) specified in the url() uses the tilde prefix which is a Webpack specific feature.  This feature is also no longer recommended for use by Webpack as it will most likely stop working in a future Webpack release.
The path also appears to be referencing a resource that is also an application asset. If the file is intended to be bundled and processed within the application build, then it should be referenced via a relative path and not placed within a configured asset directory. Build processing will ensure the file is available in the output. If the file is intended to be copied directly to the output without any further processing then the file should be referenced by a root relative path based on the browser accessible application deployment location and placed within a configured asset directory.

@clydin thank you for detailed response. I remove ~ and ^ before file names, and looks like it works good! Would love if angular esbuild could provide details about it in error message. Maybe using that plugin angular-css-resource possible to do that?

problem with following new approach is that when starting project with command ng serve . upon reloading project on. code change the  background images often not getting found by server. but get found if restart server manually the images get loaded properly .


Can a function type alias have a type variable with the same name as itself?

Is this legal:
typedef F<F>(F f);

I can't even find a specification of what it means to have type parameters on a function type alias.
The specification says:

The effect of a type alias of the form  typedef T id (T1 p1, ..., Tn pn, [Tn+1 pn+1, ..., Tn+k pn+k]) declared in a library L is is to introduce the name id into the scope of L, bound to the function type (T1, ..., Tn, [Tn+1 pn+1, ..., Tn+k pn+k])  -> T.
The effect of a type alias of the form typedef T id (T1 p1, ..., Tn pn, {Tn+1 pn+1, ..., >Tn+k pn+k}) declared in a library L is is to introduce the name id into the scope of L, bound to the function type (T1, ..., Tn, {Tn+1 pn+1, ..., Tn+k pn+k})  -> T. . In either case, iff no return type is specified, it is taken to be dynamic.
Likewise, if a type annotation is omitted on a formal parameter, it is taken to be dynamic.

Notice the absence of type parameters from the above text.

Immediate thought: Sure, no problem.
The type parameter is a "local" variable scoped inside the typedef, the typedef's own name is declared in the scope outside it, so the local variable should shadow the outer one, and everything is honky dory.
As the opposing view, we don't allow type parameter of a class to have the same name as the class, or as any member. The members make sense, they can be seen to be at the same scope level as the type variables (whether they actually are or not, it's easier to not confuse anything), and having the same name as the class would probably preclude writing constructors, so it's better to disallow it.
We could say that a type parameter having the same name as the typedef itself is confusing, and should be disallowed, but we haven't done so yet, and there is no technical reason to do it.
All in all, I'd allow it, unless someone comes up with a good reason against.

It seems inconsistent to me to allow it for typedefs but not for classes. My belief is that inconsistencies like this make it harder for people to understand a language, so I would disallow it, if it were up to me.
@lukechurch What do you think the impact on usability would be?

The consistency I'm looking at is that nothing inside a class can have the same name, including constructors, which is the reason the class's own name is prohibited for members and type parameters.
It's not that there is a problem with the type parameter conflicting with the class, it's the conflict with the constructor that is the reason. That problem isn't there for the typedef (no internal declarations to conflict with).
We allow names to shadow, even closely declared ones, unless there is a reason not to.
If we had used new(x) or new.foo(x) for constructors instead of ClassName(x) and ClassName.foo(x), there wouldn't be a technical need to disallow the class name as a member name or type parameter name. It might be confusing, but lots of things can, and we have style guides to fix most of those, not language requirements.

My inclination would be to say that unlike classes where the class name is in scope in the body (and hence forbidding the shadowing is helpful), the typedef name is not in scope in the body and hence there is no reason to forbid it.

Technically, the name of the typedef is in scope inside itself, it's just not allowed to refer to itself (directly or indirectly, to avoid a structurally infinite type).

Any self reference in a typedef, either directly, or recursively via another typedef, is a compile time error.
You couldn't have a direct self reference if it wasn't in scope.

Still, that's not a reason to prohibit nested uses of the name. We do allow functions to have variables, or even parameters, that have the same name as the function, they're just shadowing the outer declaration and there is nothing inherently wrong with that, if used with care (well, readability, but nothing technically wrong).
typedef Foo Foo<Foo>(Foo x);
foo<foo>(foo){var foo;print(foo);}
class C<T> {  C(C, T) { print("$C $T"); } }
main() {
 new C("Hello", "world!");
 foo<int>("param");
 Foo<int> f = (int x) => x;
}

This runs today, in checked mode and analyzer strong mode, so we already allow the type parameter of a typedef to have the same name as the typedef.
I see no reason to change that.

No milestone now: The specification needs to be clarified as indicated, but this is independent of Dart 2.

So, in short, it's legal.
We may want to consider making it not legal, bu we haven't yet.


Edit user page to be possible to add 2FA

@FelipePSoares Forgive me, I'm trying to work out how you create a database migration in your project. Are you able to let me know?

@Dan-613 sorry I didn't plan this issue yet. I just create to don't forget. But answering your question.
To create a migration you need to build the project in release and then run the command appoint to persistence project then because exist 2 contexts you need to select the correct context.
Add-Migration {migration name} -Context EasyFinanceDatabaseContext

But I was thinking now, you don't need to change anything in the background to use 2FA. Because I'm using the C# default implementation and all the endpoints to use 2FA is ready created.

Using google authenticator compatible 2FA, I was planning on storing the 2fa secret key per-user so that it is defined on the fly and therefore doesn't need to be configured and stored within the code or during provisioning. So I was going to add the field string User.TwoFactorSecret

But this is exactly what is defined today, there is a field to save the code, isn't?

Ah. I didn't realise the field already existed in ef core.

But to be honest I don't know exactly how it works, because it's the new .net core 8 authentication. It's the first time that I'm using.
I just added this configuration and it created every register, login and logout endpoint.


[List] nestedItems support in @next

@oliviertassinari is the nestedItems prop removed from the new ListItem component or is it about to be ported sooner or later?
Thanks for your work!

Do we need it? The documentation is nesting the ListItem without much issue. I guess extra flexibility is allowing us to remove that feature.

Where is the doc using nested ListItem? I tried to browse the /docs path of the alpha but I didn't find nested ListItem 🤔
BTW GitHub search feature sucks, you cannot search in specific branch :\

There is that AppDrawerNavItem.

@damianobarbati ListItem can accept other ListItems as children.  There is no need for a dedicated nestedItems property since the children property can serve this purpose.

Sounds fair, I'm closing it.

@oliviertassinari @kgregory I'm adopting the docs solution with further css tweaking which is necessary to proper align text and secondary action icon: the previous nestedItems taking care of everything was handy!

@damianobarbati Do you see room for a good abstraction here? That's something we could think of, just want to keep things as simple and flexible as possible.

@oliviertassinari honestly the only scenario I can see for nesting/using children here is to render nested list items just like the old nestedItems used to do: every other case can be handled with the left/middle/right content/icon.
That's why I don't think we should abstract here but rather provide the specific and most used case without having the devs taking care of this everytime.
This is my personal opinion though: I agree with keeping everything as clean as possible and let devs extends functionalities when needed! :)

Long story made short: wrap ListItem in a div and an adjacent Collapse container with nested ListItems.
Credits to @oliviertassinari (I think Fade component is not working flawlessly as Collapse!).

My solution for reference:
import React from 'react';
import PropTypes from 'prop-types';
import autobind from 'autobind-decorator';
import { connect } from 'react-redux';
import { startCase, map } from 'lodash-es';

import classNames from 'classnames';

import { withStyles, createStyleSheet } from 'material-ui/styles';

import Paper from 'material-ui/Paper';
import List, { ListItem, ListItemSubheader, ListItemIcon, ListItemText, ListItemSecondaryAction } from 'material-ui/List';
import Badge from 'material-ui/Badge';
import IconButton from 'material-ui/IconButton';
import Divider from 'material-ui/Divider';
import Switch from 'material-ui/Switch';
import Collapse from 'material-ui/transitions/Collapse';

import {
    NavigationExpandMore as ExpandMoreIcon,
    NavigationExpandLess as ExpandLessIcon,
} from '@damianobarbati/mdi';

const styleSheet = createStyleSheet('Nav', theme => ({
    root: {
    },
    subList: {
        paddingLeft: '18px',
    },
}));

@connect((state, props) => ({
}))
@withStyles(styleSheet)
export default class Nav extends React.Component {
    static propTypes = {
        dispatch: PropTypes.func.isRequired,
        classes: PropTypes.object,
        entries: PropTypes.array.isRequired,
        secondaryEntries: PropTypes.array,
    }

    componentWillMount () {
        this.setState({});
    }

    render () {
        const { classes, entries, secondaryEntries = [] } = this.props;

        const renderListItem = ({ name, caption, icon, secondaryIcon, notificationsCount, action, secondaryAction, entries = [] }, nested = false) => {
            const expanded = !!this.state[name];
            const expand = () => this.setState({ [name]: !expanded });

            return (
                <div key={name} className={classNames({ [classes.subList]: nested })} >
                    <ListItem dense={false} disableGutters={false} button={true}>

                        { icon && notificationsCount &&
                        <ListItemIcon onClick={action}>
                            <Badge badgeContent={notificationsCount || ''}>{icon}</Badge>
                        </ListItemIcon>
                        }
                        { icon && !notificationsCount &&
                        <ListItemIcon className={entries.length ? classes.listItemIconWithNested : ''} onClick={action}>
                            {icon}
                        </ListItemIcon>
                        }
                        <ListItemText primary={name} secondary={caption} onClick={action} className={entries.length ? classes.listItemTextWithNested : ''} />
                        { (secondaryIcon || !!entries.length) &&
                        <ListItemSecondaryAction onClick={entries.length ? expand : secondaryAction} className={entries.length ? classes.listItemSecondaryActionWithNested : ''}>
                            <IconButton onClick={entries.length ? expand : secondaryAction}>{entries.length ? (!expanded ? <ExpandMoreIcon /> : <ExpandLessIcon />) : secondaryIcon}</IconButton>
                        </ListItemSecondaryAction>
                        }
                    </ListItem>
                    { !!entries.length &&
                    <Collapse  in={expanded} transitionDuration={'auto'} unmountOnExit>
                        { entries.map(entry => renderListItem(entry, true)) }
                    </Collapse>
                    }
                </div>
            );
        };

        return (
            <Paper className={classes.root}>
                <List>
                    { entries.map(entry => renderListItem(entry)) }
                </List>

                <Divider />

                <List>
                    { secondaryEntries.map(entry => renderListItem(entry)) }
                </List>
            </Paper>
        );
    }
}

@damianobarbati Thanks for sharing the solution 😄 !

Also, why not using material-ui-icons?

Because I'm heavily using material-design-icons as well 😢
I had to create a repo merging both the worlds https://www.npmjs.com/package/@damianobarbati/mdi

You mean https://materialdesignicons.com/?

Yep!


Unable to build with Carthage

Hi,
With the latest version (1.1) I am running in to issues building the project with Carthage.
"error: Build input file cannot be found: '/path/to/xcodeprojectdir/Carthage/Checkouts/BetterSegmentedControl/Example/Pods/Target Support Files/BetterSegmentedControl/BetterSegmentedControl-Info.plist'"
The only .plist file in that directory is "Info.plist" so I guess that needs to be updated somewhere. (Perhaps this is the Xcode migration tool breaking something?)
Thanks,
Geoff

I edited the references to Target Support Files/BetterSegmentedControl/BetterSegmentedControl-Info.plist in /Example/Pods/Pods.xcodeproj/project.pbxproj to be Info.plist and that seems to fix it

So I think what has happened is that the Info.plist has changed, but because Pods/* is gitignored the new one BetterSegmentedControl-Info.plist isn't being added (whereas the Info.plist perhaps outlives the gitignore rule and so is still included?) so when Carthage tries to build it isn't there

Is this still an issue?

Yes, it's still an issue.

v1.2.1 still an issue.

This is still an issue even now :(

Support for Carthage has been removed for now due to issues with Xcode 12.

@gmarm FWIW our fork from Sep 2019 builds just fine with Carthage on Xcode 12 targeting iOS 14

Could you please open a PR for this? I was struggling yesterday and although the scheme was shared and commited to git, carthage build --no-skip-current was failing with Dependency has no shared framework schemes.


"encx264: unparsed options:"

Hi
First of all thanks for these great scripts!
I'm not sure if this is an actual issue, but whenever i use transcode-video.sh i'm getting the following line:
"encx264: unparsed options: vbv-maxrate=3000:vbv-bufsize=1500:crf-max=25"
If i just read this line, it seems like those parameters are not used for converting?
CLI program versions:
HandbrakeCLI: "HandBrake 0.10.1<PHONE_NUMBER>) - Darwin x86_64"
mkvpropedit: "mkvpropedit v7.7.0 ('Six Voices') 64bit built on Mar  1 2015 18:00:50"
MPlayer: "MPlayer 1.1-4.2.1 (C) 2000-2012 MPlayer Team"
mkvmerge: "mkvmerge v7.7.0 ('Six Voices') 64bit built on Mar  1 2015 18:00:50"
ffmpeg: "ffmpeg version 2.6.1 Copyright (c) 2000-2015 the FFmpeg developers
built with Apple LLVM version 6.1.0 (clang-602.0.49) (based on LLVM 3.6.0svn)
configuration: --prefix=/usr/local/Cellar/ffmpeg/2.6.1 --enable-shared --enable-pthreads --enable-gpl --enable-version3 --enable-hardcoded-tables --enable-avresample --cc=clang --host-cflags= --host-ldflags= --enable-libx264 --enable-libmp3lame --enable-libvo-aacenc --enable-libxvid --enable-libfaac --enable-libfdk-aac --enable-libx265 --enable-nonfree --enable-vda"
mp4track: "mp4track - MP4v2 2.0.0"

Conrad,
You are very welcome! Glad you like them. And it seems you're reading your console output and log files. Excellent! Everyone should do that.
But, not to worry. That misleading "encx264: unparsed options" message is simply telling you that HandBrake itself didn't parse those encoder options and instead passed them straight through to x264, the transcoding library within HandBrake. So, they're definitely being used.
You can verify this yourself with the MediaInfo tool available here or via brew install media-info from Homebrew.
When you inspect a transcoded file like this:
mediainfo "/path/to/Movie.mkv"

... MediaInfo will show you all the options used by x264.
This isn't the first time someone has asked me about the "encx264: unparsed options" line. I should really add that to the README. :)
Thanks for your question!


Pressing keyboard up-arrow maximizes app to full screen (sometimes)

Not sure where the focus is but it must be out of any tiled SSH session. I usually work with anywhere from 2 to 6 tiles, each with multiple tabs, all SSH. When focus is inside any of the tiled sessions, up arrow is interpreted by my shell and responds as expected. From time to time, focus must be somewhere on the app but not within a session. In that case, the up arrow maximizes the app window to full screen. This was not happening in <IP_ADDRESS>.
I've seen this multiple times but cannot reproduce at will. Don't know what window state for the app triggers the behavior.

OS: Windows 10 enterprise
Browser Brave
Version: <IP_ADDRESS>

Please let me know if you can reliably reproduce this issue so I can look into it further.

Please confirm if these steps can also reliably reproduce the issue for you:

Have focus on SuperPuTTY
WindowsKey + L to lock Windows
Ctrl+Alt+Del to login
refocus on SuperPuTTY (if lost)
press up-arrow

To reproduce this issue with Win+L, you have to press arrow key other than ↑ when SuperPutty is already maximized.
Pressing arrow key is triggering same behavior with Win+→, Win+←, Win+↑, Win+↓.

Oh, another prerequisite is that you'll need at least a session open for this bug to kick-in. It doesn't have to be connected though, but an empty SuperPuTTY (fresh launched) just won't cut it.

I used git-bisect to find the commit that introduced this bug. And it is 9cf0816.


Missing source maps

Sourcemap for<EMAIL_ADDRESS>points to missing source files
Sourcemap for<EMAIL_ADDRESS>points to missing source files
Sourcemap for<EMAIL_ADDRESS>points to missing source files
Sourcemap for<EMAIL_ADDRESS>points to missing source files

Getting a lot more...

Closed with https://github.com/lightning-js/renderer/pull/33


Blank document when I add a page

When I use AddPage I just get a white/blank file, and can't seem to figure it why, I even tried with the basic examples. Also auto page break point doesn't work. If I use AddPage after I have written to the document I get some infinite loop as it crashes.
Thanks

Can you include the smallest possible program that causes these problems?
Does the following program work for you?
package main

import (
  "fmt"
  "os"

  "github.com/jung-kurt/gofpdf/v2"
)

func main() {
  pdf := gofpdf.New("P", "mm", "A4", "")
  pdf.AddPage()
  pdf.SetFont("Arial", "B", 16)
  pdf.Cell(40, 10, "Hello, world")
  err := pdf.OutputFileAndClose("hello.pdf")
  if err != nil {
    fmt.Fprintln(os.Stderr, err)
  }
}

Thank you for the reply, turns out it was my issue, I was doing all the drawing in the header function and I guess when you call AddPage from the header function some kind of recursion forms.

Thanks for the followup!


cursor really hard to see light mode windows

idk how to fix this but it is near impossible to see the cursor on windows light mode when over the editor because the cursor text thing is also white, maybe just my computers or something
https://github.com/KittyCAD/modeling-app/assets/1445228/d460f1ac-0d02-41f8-8ac7-10ba39b1c987

OH MY GOD IN THE VIDEO ITS NOT WHITE

ill keep this open in case others hit this but very weird

Jess, it's Lena and Michael with the Loza team.  We've been trying to reach you.  Please check your KittyCAD email!  Urgent!

OMG that's crazy lol

URGENT!  Hi Jess.  We need instructions TODAY.  Please check email and respond ASAP.  Thank you.

i closed since this only happens on one windows machine i have and is super weird


Add source location tracking to RemoteWidget

Use case: Visual debugger.
See https://github.com/flutter/flutter/issues/90218#issuecomment-1879579898 for more context.
In a Visual Debugger I would like to be able to highlight the RFW text sections that map to a particular widget being displayed. This is similar to Chrome DevTool's 'Elements' feature.
Ideally we can map from Widget to text section and from text section to Widget.

Would we want this to work for the binary format also?
What kind of API would you want for this?

Re the binary format: If we went from rfwtext to binary, yes, that would be helpful. I think we would also need to embed the rfwtext in the binary as well.
I'm interesting in building a visual debugger tool for rfw, that is, I can type rfwtext into a box and see live updates and also map between the rfwtext and the widget tree:

I'd also like to map between DynamicContent keys and the widget tree.
My concern is that complicated UIs will be hard to debug when using rfw as an intermediate representation and having a simple visual debugger would help.

I'm having trouble understanding what the API should look like to be useful. Would you be able to provide some sample code that shows how you would write the visual debugger against a fictional ideal package:rfw API that does what you need?

Let me sketch in more detail here:
Let's assume I have something like the following (see my screenshot as well for more context)
The following code would run every time I _rfwText is changed (i.e. each keystroke). Essentially giving a live preview of whatever I've typed or pasted into my text box.
RemoteWidgetLibrary remoteLib = parseLibraryFile(_rfwText);
_remoteWidgetLibrary = remoteLib;
_runtime.update(mainName, _remoteWidgetLibrary!);

My tool is also displaying the root widget as follows:
RemoteWidget(
                  runtime: _runtime,
                  data: _data,
                  widget: const FullyQualifiedWidgetName(mainName, 'root'),
                  onEvent: (String name, DynamicMap arguments)

I would like an API that does something like the following:
// Encodes the location of some statement/section inside RFW text.
class SourceLocation {
  // Offset into RFW text.
  int offset;
  // Length from RFW text
  int length;
}

SourceLocation findSourceLocationForWidget(Widget w);

Where w is required to be a child of RemoteWidget.
My debugger would determine the Widget (w) under my mouse cursor and then call findSourceLocationForWidget so that I can change my text display to, for example, underline/highlight the section of RFWText that created w.
I could also build a reverse map, where I look at where in the text box my cursor is and determine the Widget it is creating and then add a box around the widget.

I actually have a basic demo here: https://github.com/johnmccutchan/rfwpg

Interesting. Let me think about how that might work.

(note to self: we'd need to use a BuildContext/Element as the key, not a Widget.)

Similarly, I'd like to be able to identify sections of rfw text that are keys to something in DynamicContent and do a mapping between a display of the entire DynamicContent tree and the rfw text.

Please take a look at https://github.com/flutter/packages/pull/5876 and let me know if it is sufficient for your needs. Thanks!


TokenMutator returns Mutants and more precise mutation location

This PR changes the return type of token mutators from String to Mutant. By doing so, a token mutator now has more control over the information (e.g. location of the mutation, description, etc.) of each Mutant mutated by it.
This change means:

From the outside, Weapon-regeX should have NO behavioral difference.
Mutate/Apply a RegexTree to a TokenMutator now returns a sequence of Mutant. This is also the case, for TokenMutatorJS and MutantJS.
All existing token mutators that are descendants of TokenMutator have been adapted to this change.
Any user-define token mutator that is a descendant of TokenMutator needs to adapt to this change. (👇 Guide)
TreeMutator now only traverses the tree, repacks Mutants with (more) complete patterns, and not create entirely new Mutants. New Mutants are only created by the token mutators.

This PR also updates the existing token mutators to produce Mutants with more precise mutation locations, enabled by the above change. (👇 Mutation location changes)
Mutation location changes
BOLRemoval
// Before
"^HelloWorld" -> "HelloWorld"
 └──────────┘
// After
"^HelloWorld" -> "HelloWorld"
 └┘

EOLRemoval
// Before
"HelloWorld$" -> "HelloWorld"
 └──────────┘
// After
"HelloWorld$" -> "HelloWorld"
           └┘

GroupToNCGroup
// Before
"(HelloWorld)" -> "(?:HelloWorld)"
 └───────────┘
// After
"(HelloWorld)" -> "(?:HelloWorld)"
 └┘

LookaroundNegation
// Before
"(?=HelloWorld)"  <-> "(?!HelloWorld)"
 └─────────────┘       └─────────────┘
"(?<=HelloWorld)" <-> "(?<!HelloWorld)"
 └──────────────┘      └──────────────┘
// After
"(?=HelloWorld)"  <-> "(?!HelloWorld)"
 └──┘                  └──┘
"(?<=HelloWorld)" <-> "(?<!HelloWorld)"
 └───┘                 └───┘

CharClassNegation
// Before
"[HelloWorld]" <-> "[^HelloWorld]"
 └───────────┘      └────────────┘
// After
"[HelloWorld]" <-> "[^HelloWorld]"
 └┘                 └─┘

CharClassChildRemoval
// Before
"[HelloWorld]" -> "[HellWorld]"
 └───────────┘     └──────────┘
// After
"[HelloWorld]" -> "[HellWorld]"
      └┘

POSIXCharClassNegation
// Before
"""\p{Alpha}""" <-> """\P{Alpha}"""
   └────────┘          └────────┘
// After
"""\p{Alpha}""" <-> """\P{Alpha}"""
   └──┘                └──┘

QuantifierRemoval
// Before
"(HelloWorld)*" -> "(HelloWorld)"
 └────────────┘
// After
"(HelloWorld)*" -> "(HelloWorld)"
             └┘

QuantifierNChange
// Before
"(HelloWorld){10}" -> "(HelloWorld){10,}", "(HelloWorld){0,10}"
 └───────────────┘
// After
"(HelloWorld){10}" -> "(HelloWorld){10,}", "(HelloWorld){0,10}"
             └───┘

QuantifierNOrMoreModification
// Before
"(HelloWorld){10,}" -> "(HelloWorld){9,}", "(HelloWorld){11,}"
 └────────────────┘
// After
"(HelloWorld){10,}" -> "(HelloWorld){9,}", "(HelloWorld){11,}"
             └────┘

QuantifierNOrMoreChange
// Before
"(HelloWorld){10,}" -> "(HelloWorld){10}"
 └────────────────┘
// After
"(HelloWorld){10,}" -> "(HelloWorld){10}"
             └────┘

QuantifierNMModification
// Before
"(HelloWorld){10,20}" -> "(HelloWorld){9,20}", "(HelloWorld){11,20}",
 └──────────────────┘    "(HelloWorld){10,19}", "(HelloWorld){10,21}"
// After
"(HelloWorld){10,20}" -> "(HelloWorld){9,20}", "(HelloWorld){11,20}",
             └──────┘    "(HelloWorld){10,19}", "(HelloWorld){10,21}"

QuantifierShortModification
// Before
"(HelloWorld)*" -> "(HelloWorld){0,0}", "(HelloWorld){0,2}",
 └────────────┘    "(HelloWorld){1,1}"
// After
"(HelloWorld)*" -> "(HelloWorld){9,20}", "(HelloWorld){11,20}",
             └┘    "(HelloWorld){10,19}", "(HelloWorld){10,21}"

QuantifierShortChange
// Before
"(HelloWorld)*" -> "(HelloWorld){0}"
 └────────────┘
// After
"(HelloWorld)*" -> "(HelloWorld){0}"
             └┘

QuantifierReluctantAddition
// Before
"(HelloWorld)+" -> "(HelloWorld)+?"
 └────────────┘
// After
"(HelloWorld)+" -> "(HelloWorld)+?"
             └┘

Adapt token mutator to return Mutant
Due to the return type of token mutators change from String to Mutant, any token mutator that is a descendant of TokenMutator needs to adapt to this.
This PR also provides some helper functions to make this adaptation easier. These are extension methods that convert a String into a Mutant within the scope of each TokenMutator by filling in the Mutant data with the information from the containing TokenMutator.

Convert a mutated pattern string into a Mutant at the provided location.

"Pattern".toMutantAt(loc: Location)

Convert a mutated pattern string into a Mutant with the location taken from the provided token.

"Pattern".toMutantOf(token: RegexTree)

Convert a mutated pattern string into a Mutant with the location starts from the start of the provided token and ends at the start of the token's first child. (*)

Sample results: GroupToNCGroup, LookaroundNegation, CharClassNegation, POSIXCharClassNegation

"Pattern".toMutantBeforeChildrenOf(token: RegexTree)

Convert a mutated pattern string into a Mutant with the location starts from the end of the provided token's last child and ends at the end of the token. (*)

Sample results: QuantifierNChange, QuantifierNOrMoreModification, QuantifierNOrMoreChange, QuantifierNMModification, QuantifierShortModification, QuantifierShortChange, QuantifierReluctantAddition

"Pattern".toMutantAfterChildrenOf(token: RegexTree)

(*) If the given token has no child, the location of the given token is considered to be the location of the mutant
Example
Here is an example of a simple case of adaptation. This will produce the same Mutants as before this PR.
// Before
object PredefCharClassNegation extends TokenMutator {
  // ...
  override def mutate(token: RegexTree): Seq[String] = (token match {
    case pdcc: PredefinedCharClass =>
      Seq(pdcc.copy(charClass = pdcc.charClass.toggleCase))
    case _ => Nil
  }) map (_.build)
}

// After
object PredefCharClassNegation extends TokenMutator {
  // ...
  override def mutate(token: RegexTree): Seq[Mutant] = (token match {
    case pdcc: PredefinedCharClass =>
      Seq(pdcc.copy(charClass = pdcc.charClass.toggleCase))
    case _ => Nil
  }) map (_.build.toMutantOf(token)) // Convert into `Mutant`
}

See the file changes of this PR for more examples.

Wow, what a wonderfully documented PR! Code looks good to me. Clean and descriptive!

Thanks!

If I understand correctly, the locations of mutants should now be much more accurate (smaller)? We'll have to start using this in Stryker4s soon 😃.

The mutation location should be more accurate now and better tell the actual character range that was mutated in the input regex string. This is especially useful in regex with long containers (group, character class, concat, or, …) and regex syntax on long containers.

If the positions of mutants have changed, shouldn't some tests also have had to change? Or maybe there's a couple missing for mutant positions

Indeed, we yet to have tests that check for mutation locations 😅, but we definitely will when we rework and reorganize the tests.


how i can make it the default file explorer?

hello how i can make it the default file explorer? also can you remove that emoji from icon?

It's NOT recommanded to set ExplorerEx as default because it's in develop and not stable yet.
If you really want to, this can be done by modifing the registry. you can EDIT ExplorerEx/ExplorerEx/Reg/SetEx.reg and REPLACE G:\\Source\\C#\\ExplorerEx\\ExplorerEx\\build\\ExplorerEx.exe to the actual ExplorerEx excutable path in your computer. REMIND that use \\ as the path separator.
You can restore it by ExplorerEx/ExplorerEx/Reg/Restore.reg. However, I only tested it on Windows 11. It's recommanded to backup the RegKeys in SetEx.reg and make your own Restore.reg.
After changing the registry, pressing win + e or opening explorer from start menu should launch ExplorerEx. However, when some application launch explorer directly (for example, the folder will open after unzipping an achieve), ExplorerEx will not launch. I may fix this by developing a Shell Extension later.
As for the Icon, I'm not a professional icon designer so I'm not going to change it by myself. Any idea of a new icon is appreciated!
Best wishes : )

Now you can set it to default by switching on in the settings!

Now you can set it to default by switching on in the settings!

yes i can see  (BHO plugin ExplorerEx new version 0.4) but it's not working, delete explorer proxy.reg file is created. but still not replaced by default file explorer.
can you tell me how i can do manually in regedit thanks

After switch on set default, a .reg file should be created in the ExplorerEx executable folder, you can use this to manually set default.

thanks it's working now, but there's a little bug when default file explorer appear in 1 sec and disappear.

see this gif screen recording
https://files.catbox.moe/l3y4vj.gif


[NO_TICKET] Experiment with the Home Page and Top Navigation Menu

Changelog
Added
As an experiment, I decided to play with the home page of the portal.
I'm submitting this PR to avoid losing my local changes. Last time I tried it locally and haven't saved the code I used.
For some reason, many Font Awesome icons didn't work for me, including fa-solid fa-up-right-from-square that Docsy is using.
https://deploy-preview-242--cobalt-docs.netlify.app/
In the first column of cards, I added links. The second column contains buttons.

Preview This Change
To see how this change looks in production, scroll down to Deploy Preview. Select the link that looks like https://deploy-preview-<num>--cobalt-docs.netlify.app/
Variables
Help us support a “Write once, publish everywhere” single source of truth. If you see a line that looks like:
{{% asset-categories %}}
You’ve found a shortcode that we include in multiple documents.
You’ll find the content of the shortcode in the following directory:
https://github.com/cobalthq/cobalt-product-public-docs/tree/main/layouts/shortcodes
That shortcode has the same base name as what you see in the PR, such as asset-categories.html.
Checklist for PR Author
[ ] Did you check for broken links and alt text?
Be sure to check for broken links and Alt Text issues. We have a partially automated process,
as described in this section of our repository README:
Test Links and Alt Attributes.

Suggestion from Courtney: change the footer color.


Battle replay Feature Suggestion?

Hello, I'd like to know if it would somehow be possible for a showdown replay saved as a file which can be then imported into the 3ds games as a battle replay?
Battle replays on the 3ds are only 11kb in size so I don't think it'd be too difficult to record and format all of the actions that happen in a showdown battle into the appropriate hexadecimal values so the 3ds games can read this?

This would be really nice, but unfortunately two things prevent this:

As far as I know, no one has managed to successfully reverse-engineer exactly how a 3DS replay is stored.

In-game replays store starting teams, starting RNG seed, and a list of decisions. We don't yet have full RNG accuracy – see #1157 – so we wouldn't be able to reproduce the same games as in-game.

Normally I'd leave open feature requests that are unlikely to implement, on the off chance someone else wants to, but for this one, I think I'll close it. If Project Pokémon ever gets to the point where this is possible, someone will probably do it then.

I'd like to do some digging into replay files, do you know of any documentation that has been compiled of such research?
I know of this tool: https://github.com/Lyviaff/7G-Replay-Editor
It doesn't edit anything aside from graphical aspects of a replay however. Could be useful to know what areas NOT to edit though.
I'm currently editing hex values of a replay file I have to get a feel for what variables do what, I'm sure this would help this project out.  I'm fairly new to this however and would appreciate any advice on what other steps I can take to see if we can make this happen!

Oh, I think that means they've at least managed to decrypt replays. This could be really useful, actually.

I'm also doing some testing to figure out what offsets are responsible for what , from what I can tell , the beginning bits are all the graphics that show up in the VS Recorder, things like Outcome of the battle , player passport pictures, sprites of team, maybe even type of battle, although my research so far is strictly for singles no restrictions since I feel this would be the most used format.
I'm going to be testing out Pokemon with very little variations to see if I can pinpoint the areas responsible for things like PID,Species,Gender,Nature,Ability,Item, IV's, EV's and such.
Should I post my findings on here or is another method preferred? First time I've done this sort of thing, haha!

This is the sort of research Project Pokémon is very interested in, so you might want to post it in their forums. Do link your thread from here, though, because I'm also interested in this.

Sorry for not replying here for a while, I've made considerable progress with this. I'm currently in the process of "making" my own replay based on a showdown replay. I'm merely putting everything I've found in my research to recreate a battle's actions into a replay for the 3ds games. So far it is going well,
I think I have figured out how the battle actions and Pokemon data works. However I have come across the problem of the RNG. Things like Min/Max Damage, Critical Hits, and Additional Effects of moves. I believe I found the location of the initial seed though. After examining multiple replays I found that 4 bytes were always different unless I was looking at two replays of the same battle. I changed these 4 bytes and then an additional effect that triggered on turn one of my battle (Ancient Power's +1 in all stats) did not trigger.
Given the information of where the seed is and many replays to study the results of a seed, is there anything you think you can do to generate a seed that would give the exact same results as a showdown replay ?
I.e.
Turn 1
1st player: Miss Attack ( of X accuracy)
2nd player: No effect (Minimum Damage)
Turn 2
1st player: Critical Hit (Minimum Damage)
2nd Player: Scald inflict burn on 1st Player .
I'm not sure how Battle RNG works as most of the RNG research I've seen is related to encounters and hatching eggs. However, shouldn't the replay store the frames or other information of every turn? Looking into the replays hasn't yielded anything indicating storage of other information except a starting seed at the beginning of the battle, I'm wondering if this is the starting seed at the beginning of the battle in real time or if the game just generates a seed that replicates the battle as it happened?
By the way would running PokeCalc during a battle yield any useful information? I tried doing so on my two consoles while watching a replay thinking that maybe some seed would show up on both simultaneously, but the tool is made for shiny encounters so I don't know if battle RNG information would even be displayed here.

@Marty-D How much work would it be to update the sim to be RNG-compatible with US/UM?
@ReZeroEmiria This is really interesting how far you've gotten. The main problem is that unless we know exactly how the RNG works at every step, we can't match it.
I'm not saying it's impossible, but it would require detailed knowledge of how the in-game RNG works.

I couldn't even give an educated guess since I don't know anything about the in-game RNG.

An in-game replay should store:

starting teams
starting seed
everyone's choices every turn

All other information is re-simulated live when the games play back a replay.
The starting seed is the same starting seed as the battle itself.
I don't know anything about PokéCalc so I don't know how it would be used.

@Zarel I went on the Project Pokemon Discord and one of the users said the following:
"new battle -> get a random seed
any RNG call -> advance RNG, get next random value
it'd be a lot of trial and error
pokecheck still has the battle video src code available
http://vsplayer.pokecheck.org/?src
might be beneficial to try to get bvid battle exectution working first, as it's the 'known correct' result
gen6/7 use the same RNG, it's their new standard
https://github.com/wwwwwwzx/3DSRNGTool/tree/master/3DSRNGTool/RNG"
They're far more knowledgeable on this matter than I, I just figured this may be of help, if you'd like , join the discord and check the dev channel: https://discord.gg/66PzPgD

@kwsch What would really help me is a list of example replays, with a list of exactly where the RNG was called, and how the value was used each time.

afaik nobody has logged bvid->rand call results; I don't own a 3ds and am not particularly interested in this area...
I haven't bothered checking lately, but based on what I've seen from browsing the disassembly, it should be roughly the same rand() call ordering as prior games but with a different rand() method generating the random values.
https://www.smogon.com/forums/threads/past-gen-rng-research.61090/page-31#post-3693066
The game codebase doesn't change drastically between generations, just updates to add in new mechanics / structures. Shouldn't be too hard to plot rand results if a similar tool to RNG Reporter's Researcher window is made. I'm pretty sure the battle RNG uses its own dedicated SFMT.

@kwsch PS has never been accurate, but there's not an obvious reason why, to me. It would be very useful to compare the RNG calls PS makes with the RNG calls the games make.
I don't even need you to log RNG calls directly from the games; it would be enough to log RNG calls from the Pokécheck Gen 5 VS player.
I guess I could do it myself, but I don't particularly understand the code and am in the middle of some other work.


When creating a new sqlalchemy session, get default values from the data source configuration

Currently when we create a new sqlalchemy session from a datasource we are ignore the values set  from the configuration.
Let's get the default values set in the current data source.

Let's also set auto_commit to False and expire_on_commit to True by default.

The parameters and autobegin and  autoflush  where set to True by default.
Those are the default values for a session in the sqlalchemy.


Switch to gmavenplus and update groovy

This means we can now run the groovy tests with JDK 11.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/168/


[i2c] sda interference during host enable

the current i2c assumes that when the host function is enabled, if sda is already low, then it is registered as bus interference.
@a-will pointed out that is not true in a multi-host situation.
Since the current i2c explicitly points out it does not support multi-host, this is not something that needs to be urgently fixed.
Filing this away as something to be improved in the future when multi-host is added.

since this is actually only needed for multi-host, which we do not support, i'm kicking this to backlog.
Let me know if anyone objects.

Out of scope for D2 signoff, part of D3.


[GLUTEN-1983][VL] Fix serialization errors by removing unused VeloxColumnarToRowExec.LOG

What changes were proposed in this pull request?
Remove the unused VeloxColumnarToRowExec that's causing Task not serializable errors.
Caused by: org.apache.spark.SparkException: Task not serializable
	at org.apache.spark.util.ClosureCleaner$.ensureSerializable(ClosureCleaner.scala:444)
	at org.apache.spark.util.ClosureCleaner$.clean(ClosureCleaner.scala:416)
	at org.apache.spark.util.ClosureCleaner$.clean(ClosureCleaner.scala:163)
	at org.apache.spark.SparkContext.clean(SparkContext.scala:2578)
	at org.apache.spark.rdd.RDD.$anonfun$mapPartitions$1(RDD.scala:860)
	...
Caused by: java.io.NotSerializableException: com.palantir.sls.logging.log4j.slf4j.SlsLog4jSlf4jLogger
Serialization stack:
	- object not serializable (class: com.palantir.sls.logging.log4j.slf4j.SlsLog4jSlf4jLogger, value: com.palantir.sls.logging.log4j.slf4j.SlsLog4jSlf4jLogger@7d832514)
	- field (class: org.apache.spark.sql.execution.GlutenColumnarToRowExec, name: LOG, type: interface org.slf4j.Logger)
	- object (class org.apache.spark.sql.execution.GlutenColumnarToRowExec, GlutenColumnarToRowExec
+- ^(2) ProjectExecTransformer [i_item_sk#63]
   +- ^(2) GlutenFilterExecTransformer ((isnotnull(i_manufact_id#76) AND (i_manufact_id#76 = 350)) AND isnotnull(i_item_sk#63))
      +- GlutenRowToArrowColumnar
         +- BatchScan ...

The field wasn't used. VeloxColumnarToRowExec calls logInfo on the org.apache.spark.internal.Logging trait it inherits.  That one correctly handles serialization (Github).
How was this patch tested?
Existing tests.

Thank you, @PHILO-HE.
It seems that CI flaked. I can't find the test that failed with Spark 3.2. The Spark 3.3 failure is below but that failure doesn't seem related to the change here.
- should be able to resolve a persistent view *** FAILED ***
  Exception thrown while executing query:

  ...

  == Physical Plan ==
  AdaptiveSparkPlan isFinalPlan=true
  +- == Final Plan ==
     *(1) ColumnarToRow
     +- LoadArrowData
        +- ColumnarAQEShuffleRead local
           +- ShuffleQueryStage 0
              +- ColumnarExchange rangepartitioning(f#738792 ASC NULLS FIRST, 1), ENSURE_REQUIREMENTS, [plan_id=1179598], [id=#1179598], [OUTPUT] List(d:StringType, e:IntegerType, f:DecimalType(2,1)), [OUTPUT] List(d:StringType, e:IntegerType, f:DecimalType(2,1))
                 +- ^(16811) NativeFileScan parquet default.t2[d#738790,e#738791,f#738792] Batched: true, DataFilters: [], Format: Parquet, Location: InMemoryFileIndex(1 paths)[file:/opt/gluten/gluten-ut/spark33/target/scala-2.12/test-classes/unit..., PartitionFilters: [], PushedFilters: [], ReadSchema: struct<d:string,e:int,f:decimal(2,1)>
  ^(16811) Substrait plan:
  {
    "relations": [{
      "root": {
        "input": {
          "read": {
            "common": {
              "direct": {
              }
            },
            "baseSchema": {
              "names": ["d", "e", "f"],
              "struct": {
                "types": [{
                  "string": {
                    "nullability": "NULLABILITY_NULLABLE"
                  }
                }, {
                  "i32": {
                    "nullability": "NULLABILITY_NULLABLE"
                  }
                }, {
                  "decimal": {
                    "scale": 1,
                    "precision": 2,
                    "nullability": "NULLABILITY_NULLABLE"
                  }
                }]
              },
              "partitionColumns": {
                "columnType": ["NORMAL_COL", "NORMAL_COL", "NORMAL_COL"]
              }
            }
          }
        },
        "names": ["d#738790", "e#738791", "f#738792"]
      }
    }]
  }
  +- == Initial Plan ==
     Sort [f#738792 ASC NULLS FIRST], true, 0
     +- Exchange rangepartitioning(f#738792 ASC NULLS FIRST, 1), ENSURE_REQUIREMENTS, [plan_id=1179547]
        +- FileScan parquet default.t2[d#738790,e#738791,f#738792] Batched: false, DataFilters: [], Format: Parquet, Location: InMemoryFileIndex(1 paths)[file:/opt/gluten/gluten-ut/spark33/target/scala-2.12/test-classes/unit..., PartitionFilters: [], PushedFilters: [], ReadSchema: struct<d:string,e:int,f:decimal(2,1)>

  == Exception ==
  org.apache.spark.SparkException: Job aborted due to stage failure: Task 0 in stage 1489.0 failed 1 times, most recent failure: Lost task 0.0 in stage 1489.0 (TID 1742) (e5425e0a1fcf executor driver): java.lang.ArithmeticException: Decimal precision 20 exceeds max precision 2
  	at org.apache.spark.sql.errors.QueryExecutionErrors$.decimalPrecisionExceedsMaxPrecisionError(QueryExecutionErrors.scala:1013)
  	at org.apache.spark.sql.types.Decimal.set(Decimal.scala:123)
  	at org.apache.spark.sql.types.Decimal$.apply(Decimal.scala:578)
  	at org.apache.spark.sql.types.Decimal.apply(Decimal.scala)
  	at io.glutenproject.vectorized.ArrowWritableColumnVector$DecimalAccessor.getDecimal(ArrowWritableColumnVector.java:1010)
  	at io.glutenproject.vectorized.ArrowWritableColumnVector.getDecimal(ArrowWritableColumnVector.java:728)
  	at org.apache.spark.sql.catalyst.expressions.GeneratedClass$GeneratedIteratorForCodegenStage1.processNext(Unknown Source)
  	at org.apache.spark.sql.execution.BufferedRowIterator.hasNext(BufferedRowIterator.java:43)
  	at org.apache.spark.sql.execution.WholeStageCodegenExec$$anon$1.hasNext(WholeStageCodegenExec.scala:760)
  	at org.apache.spark.sql.execution.SparkPlan.$anonfun$getByteArrayRdd$1(SparkPlan.scala:364)
  	at org.apache.spark.rdd.RDD.$anonfun$mapPartitionsInternal$2(RDD.scala:890)
  	at org.apache.spark.rdd.RDD.$anonfun$mapPartitionsInternal$2$adapted(RDD.scala:890)
  	at org.apache.spark.rdd.MapPartitionsRDD.compute(MapPartitionsRDD.scala:52)
  	at org.apache.spark.rdd.RDD.computeOrReadCheckpoint(RDD.scala:365)
  	at org.apache.spark.rdd.RDD.iterator(RDD.scala:329)
  	at org.apache.spark.scheduler.ResultTask.runTask(ResultTask.scala:90)
        ...

Ah, same failure here: https://github.com/oap-project/gluten/pull/1968#issuecomment-1592688883

Oops, didn't know the empty commit would dismiss the review.

Oops, didn't know the empty commit would dismiss the review (and not re-trigger CI 😅).

Don't worry. Sometimes, CI is not stable. Just manually re-triggered the CI.

@rshkv, please rebase the code. The reported failure is occasionally produced and temporarily disabled by us in https://github.com/oap-project/gluten/pull/1996/files. Thanks!

Thank you, @PHILO-HE. Rebased.

@rshkv Thanks for improving this piece of code! Could you please kindly share your use case? Based on the failure log, looks like you are trying to serialize the plans, which is not covered by any existing Gluten unit tests.
thanks, -yuan


"zwave.scene_activated" automation trigger not working in 0.47.0

Home Assistant release (hass --version):
0.47.0
Python release (python3 --version):
docker
Component/platform:
automation
Description of problem:
Automations based on "zwave.scene_activated" triggers are not working after the update to 0.47.0.  The events are originating from a aeotec minimote.  I can see log messages when I press buttons but no automations are triggered.
Example of log line on button press:
2017-06-18 22:28:20 INFO (MainThread) [homeassistant.core] Bus:Handling <Event zwave.scene_activated[L]: entity_id=zwave.living_room_remote_13, node_id=13, scene_id=1>
Expected:
Automations to trigger on minimote button presses.
Problem-relevant configuration.yaml entries and steps to reproduce:
- alias: Downstairs Minimote Button 1
  trigger:
    platform: event
    event_type: zwave.scene_activated
    event_data:
      entity_id: living_room_remote_13
      scene_id: 1
  action:
      service: switch.toggle
      entity_id: switch.living_room_floor_lamp_switch_7_0

Your entity_id on the automation is wrong. It should be zwave.living_room_remote_13

A lot of the examples I saw for the minimote didn't include that and it had been working for quite some time.
Was there something in 0.47 that made this stricter?

So this was a side-effect of #7786. Previously the entity_id attribute was being populated with the object ID, which wasn't actually an entity ID. It's more correct now, but it is a breaking change that sneaked through without a warning. I'll try to get a notice on the release notes.
https://github.com/home-assistant/home-assistant/pull/7786/files#diff-34736f9c4da86b6334cf5d201370707fL326 , it now is https://github.com/home-assistant/home-assistant/pull/7786/files#diff-1ce5c5e5b016f2be573987a5af3ca18fR175


[Bug] Unexpected error: multiple updates to a row by the same query is not allowed

Cloudberry Database version
Cloudberry Database 1.0.0+9b9dd58 build commit:9b9dd5856d002c43efc16e861d05514d08951a00
What happened
/code/cbdb_testrepo_src /code
/code/cbdb_testrepo_src/test_framework/tinc.py discover mpp/gpdb/tests/storage/access_methods
....
DeltaCompressionTestCase.test_delta_out_of_order_update_delete ... 369.79 ms ... FAIL
The subsequent data miss match. I believe it is due to the update statement failure.
122c120
< UPDATE 15
---
> ERROR:  multiple updates to a row by the same query is not allowed
127,130c125,128
< 1|2147483651|2014-07-29|14:22:23.77689|2014-07-30 14:22:58.356229|2014-07-31 05:22:23.776892+08
< 1|2147483651|2014-07-29|14:22:23.77689|2014-07-30 14:22:58.356229|2014-07-31 05:22:23.776892+08
< 1|2147483651|2014-07-29|14:22:23.77689|2014-07-30 14:22:58.356229|2014-07-31 05:22:23.776892+08
< 2|2147483650|2014-07-29|14:22:23.776899|2014-07-30 14:22:58.35623|2014-07-31 05:22:23.776888+08
---
> 1|2147483648|2014-07-29|14:22:23.77689|2014-07-30 14:22:58.356229|2014-07-31 05:22:23.776892+08
> 1|2147483648|2014-07-29|14:22:23.77689|2014-07-30 14:22:58.356229|2014-07-31 05:22:23.776892+08
> 1|2147483648|2014-07-29|14:22:23.77689|2014-07-30 14:22:58.356229|2014-07-31 05:22:23.776892+08
> 2|2147483630|2014-07-29|14:22:23.776899|2014-07-30 14:22:58.35623|2014-07-31 05:22:23.776888+08
131a130
> 10|2147479999|2014-07-31|14:22:23.778899|2014-07-30 14:22:58.357229|2014-07-31 05:22:23.778899+08
135,136c134
< 10|2147483660|2014-07-31|14:22:23.778899|2014-07-30 14:22:58.357229|2014-07-31 05:22:23.778899+08
< 10|2147483660|2024-07-30|14:22:24.778899|2014-07-30 10:22:31|2014-07-31 05:22:24.776892+08
---
> 10|2147499999|2024-07-30|14:22:24.778899|2014-07-30 10:22:31|2014-07-31 05:22:24.776892+08
139,141c137,139
< 12|2147483661|2014-07-30|14:22:23.776892|2014-07-30 14:22:58.366249|2014-07-31 05:22:23.777899+08
< 13|2147483661|2014-07-30|14:22:23.776892|2014-07-30 14:22:58.366249|2014-07-31 05:22:23.777899+08
< 13|2147483661|2014-07-30|14:22:23.776892|2014-07-30 14:22:58.366249|2014-07-31 05:22:23.777899+08
---
> 12|2147483660|2014-07-30|14:22:23.776892|2014-07-30 14:22:58.366249|2014-07-31 05:22:23.777899+08
> 13|2147483660|2014-07-30|14:22:23.776892|2014-07-30 14:22:58.366249|2014-07-31 05:22:23.777899+08
> 13|2147483660|2014-07-30|14:22:23.776892|2014-07-30 14:22:58.366249|2014-07-31 05:22:23.777899+08

What you think should happen instead
No response
How to reproduce
/code/cbdb_testrepo_src /code
/code/cbdb_testrepo_src/test_framework/tinc.py discover mpp/gpdb/tests/storage/access_methods
Operating System
centos7
Anything else
No response
Are you willing to submit PR?

[ ] Yes, I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct.

Test outputs and expected results are in the following attachment.DeltaCompressionTestCase.test_delta_out_of_order_update_delete.zip
It is related to PR https://github.com/cloudberrydb/cloudberrydb/pull/603

As confirmed with Zijie, these are as expected, thus I updated test cases accordingly.


Missing file in repo

There seems to be a missing file in the GitHub repo. I had to manually install bootstrap.min.css.map to C:\Users\me\miniconda3\envs\openai\Lib\site-packages\streamlit_local_storage\frontend\build.

This has been rectified in the latest update - the .map file is now added to the build folder (manually - like you, had to find it online).


[MACOS] EXC_BAD_ACCESS - Stack overflow in (null)

I have an issue on MacOS where if a flutter_inappwebview is launched when the app is not in the foreground, when the view that holds the webview instance is dismissed, the app will hard crash with the attached stack trace. It appears that there is a self reference in listeners for _finishClosingWindow causing a stack overflow.
Is it also expected that a WebView will not appear when the WebView is instantiated while the app is not in the foreground?
flutter_inappwebview: 6.0.0-beta.31
Flutter Doctor:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.7.12, on macOS 13.4 22F66 darwin-arm64, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.1)
[✓] VS Code (version 1.85.0)
[✓] Connected device (2 available)
[✓] HTTP Host Availability

• No issues found!

Can you post an example code and steps to reproduce the issue?

Maybe it could be fixed on the current latest release 6.0.0-rc.1.
Let me know if it works, so I can close this issue.

@pichillilorenzo I actually upgraded to -rc.2 on Sunday and have been testing it ever since. The crashes are fixed. However, I still have the question of the (kind of understandable, but nonetheless strange) behavior of the InAppBrowser not popping up when the App is not in the foreground. It still seems to be able to interact with the underlying InAppWebView, it's just that there is no window opened in that case.
To be clear, it was doing this before with the 6.0 beta releases, it's not new.

Sorry, but without an example, I don't understand how this could be possible.
Are you able to screen-record to show me what is the problem?


Quests Enhancements

In-Game Enhancement Ideas: What would be great is if we can enter additional commands for quests with pagination if needed
/jobs quests start <JobName> - start the quest for JobName
Permission: jobs.command.quests.admin.start
/jobs quests stop  - stop the quest for JobName
Permission: jobs.command.quests.admin.stop
/jobs quests start all - start all the quests for all jobs
Permission: jobs.command.quests.admin.start
/jobs quests stop all - stop all the quests for all jobs
Permission: jobs.command.quests.admin.stop
/jobs quests list  - lists all quests for player which would show the JobName, Quest Completed or Incomplete
Permission: jobs.command.quests.list
/jobs quests list  - lists all quests for JobName which would show all the quests by number, name, action, target, etc
Permission: jobs.command.quests.list
/jobs quests list all - list all enabled quests for all JobNames which would show similar to above
Permission: jobs.command.quests.list
/jobs quests edit  - this would open an edit UI to allow the admin to change the quest details including enabling/disabling
Permission: jobs.command.quests.edit
Console Enhancement Ideas:
Same as above, but no pagination

Additional quest suggestion here. Would it be possible to have it so a quest has to be completed within a specified region?
  ``1:
    Name: "Kill Mobs"
    Objectives:
    - Kill;Zombie;64
    - Kill;Skeleton;32
    RewardCommands:
    - "money give [playerName] 250"
    - "msg [playerName] Challenge completed"
    - "playsound minecraft:ui.toast.challenge_complete master [playerName] ~ ~ ~ 100 1.75 1.0"
    RewardDesc:
    - "Rarity: Common"
    - "Task: Kill 64 zombies and 32 skeletons in the arena!"
    - "Reward: $250"
    Worlds:
    - "World_Nether"
    Region:
    - "Arena"
    Chance: 30``

Excellent idea @Drewflav - being able to select a region (WorldGuard), Residence and/or Subzone (Residence Plugin) would be a great option to have.

@Drewflav This can be a huge large memory leak to the server, so the server can crash, whatever the server capacity. The server may collapse because PlayerMoveEvent and chunk remain on the server, so it takes up a lot of space and the Bukkit events are only deleted when plugin is disabled.
@smmmadden This is not a good idea because of server ram usage. The same chunk ram digest is in the Explorer job.

makes sense. Sounds like a bug for Mojang to work out so that isolation can be trapped to specific chunks or coordinates and not cost in performance or memory consumption.

Yes, this known bug is still for Mojang and does not fix it for some reason, so we need to manually fix it in the plugin code to avoid consuming the server.

Could it be tied into the restricted areas?

@Drewflav Hmm.. May be possible

It's time to announce, added editquest sub-command, which can list the quests and edit them. https://github.com/Zrips/Jobs/commit/0f9ed9e2c6df868e5069038fc91550bd4625495f
For start and stop commands, not sure this needed, but also is not easy to code, because we have a lot of datas, which needs to save/load on plugin stop to/from database and this causing the server slower loading at startup.


Node.Spec.ProviderID not updated after Move

After a clusterctl move from say a bootstrap cluster to a target cluster, the K.Node.Spec.ProviderID still contains the UID of the old BMH object.
Simply updating the Node.Spec.ProviderID is not currently allowed as validation checks prevent Node.Spec.ProviderID to be changed after it has been set to a non-nil value (see https://github.com/kubernetes/kubernetes/pull/51761)
Forbidden: node updates may not change providerID except from "" to valid, []:
Possible Solutions:
1) Provide an option to have ProviderID be something other than UID of BMH -- for example: BMH 'Namespace/Name' may be unique for most environments.
2) Don’t use BMH.UID which is generated automatically during object creation but instead create a new field (say BMH.ProviderID) that we can populate with a random id. This field would be copied over as part of the move.
3) Open an issue with K/K to allow updates to Node.Spec.ProviderID -- ie. Relax the validation checks.
/kind bug
xref: https://github.com/metal3-io/baremetal-operator/issues/431
@kashifest @maelk

Thanks, we'll take care of this in v1alpha4. I'd rather go with not using the UID, but another unique identifier such as namespace/name combination. We could also not tie it to the bmh anymore if it makes sense, especially considering the incoming metadata change that allow us a lot of flexibility

We could also keep it as is, considering that the provider id in our case is just used to match the machine and the node, and that it still appears in the machine correctly.

I'm fine with either Name+Namespace or a static ProviderID value that's not tied to UID.
The bug will manifest anytime someone tries to map K.Node <--> BMH.

Isn't it right now already a static ProviderID value ? It is created as the BMH UUID, but afterwards, for example if you do a move, then the only place you can find it is in the Metal3Machine Spec. So after a move you can still do the mapping with an intermediary step : K.Node <--> M3Machine <--> BMH if needed. If you take this approach from the start, whether we are using the original node UUID or a random UUID does not matter, does it ?

Yes, with the intermediary step things work out fine. However, for future reference, we should probably document that direct K.Node <---> BMH is not advisable b/c of the move operation.

Agreed with that. So we could solve this issue with a documentation update, right ?

/remove-lifecycle stale

/assign


errors: Temporary and Timeout error factories and checkers

User description

IsTimeout() and CheckIsTimeout()
IsTemporary() and CheckIsTemporary()
NewTemporaryError() and NewTimeoutError()

PR Type
enhancement, documentation

Description

Introduced TemporaryError struct to represent temporary and timeout errors.
Added methods IsTemporary, IsTimeout, Temporary, and Timeout to TemporaryError.
Implemented NewTimeoutError and NewTemporaryError functions to create specific error instances.
Added CheckIsTemporary and IsTemporary functions to check for temporary errors without and with unwrapping, respectively.
Added CheckIsTimeout and IsTimeout functions to check for timeout errors without and with unwrapping, respectively.
Updated README to document the new error handling functions and types.

Changes walkthrough 📝
Relevant filesEnhancement

errors.goIntroduce new error types and checkers for temporary and timeout errors.

errors.go
Introduced TemporaryError struct with methods IsTemporary, IsTimeout, Temporary, and Timeout.  Added functions NewTimeoutError and NewTemporaryError to create specific error types.  Implemented CheckIsTemporary and IsTemporary functions to check for temporary errors.  Implemented CheckIsTimeout and IsTimeout functions to check for timeout errors.

+113/-0

Documentation

README.mdUpdate README to document new error handling functions and types.

README.md
Updated documentation to include new error handling functions and types.  Added entries for IsTemporary, CheckIsTemporary, IsTimeout, CheckIsTimeout, TemporaryError, NewTemporaryError, and NewTimeoutError.

+3/-0

💡 PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

PR-Agent was enabled for this repository. To continue using it, please link your git user with your CodiumAI identity here.
PR Code Suggestions ✨
Category	Suggestion                                                                                Score
Maintainability
Simplify the Error method of TemporaryError by replacing the second switch statement with if statements
In the Error method of TemporaryError, the second switch statement can be simplified by combining the conditions.
errors.go [95-101]
-switch {
-    case !w.timeout:
-        return cause
-    case cause == "":
-        return "time-out"
-    default:
-        return fmt.Sprintf("%s: %s", "time-out", cause)
+if !w.timeout {
+    return cause
 }
+if cause == "" {
+    return "time-out"
+}
+return fmt.Sprintf("time-out: %s", cause)

Suggestion importance[1-10]: 6
6

@karasz do you agree with this suggestion? ---^

@karasz do you agree with this suggestion? ---^
imho the switch variant is more elegant and provides the same functionality. It might happen that somewhere the compiler gets smarter with the if construction (although I domn't see why would that be), but I do not think that the gain (if any) would be relevant. Obviously without a code tracing I cannot say what speed advantages one has over the other. Let's be more lelegant in this case I would say and stick with the switch.

v0.13.2 tagged


fix and test for SyncReturn of mirrored type

Changes
Fixes #1313
Checklist

[x] An issue to be fixed by this PR is listed above.
[x] New tests are added to ensure new features are working. End-to-end tests are usually in the ./frb_example/pure_dart example, more specifically, rust/src/api.rs and dart/lib/main.dart.
[x] The code generator is run and the code is formatted (via just precommit).
[x] If this PR adds/changes features, documentations (in the ./book folder) are updated.
[x] CI is passing.

/cc @Desdaemon for reviewing :)


cannot build the project...

Hi, I'm using vs 2015 and can't build the project. the error is unable to locate nuget.exe....
btw, is this api compatible with xbee xtend?

Sorry about that, I have been futzing with it lately in an effort to port it to .net core and I may have inadvertently checked something in. I'll look at it as soon as I can.
I've never tested it against xtend so my guess is that it doesn't work but I'm also not really sure what the xtend api looks like.
On Mon, Mar 28, 2016 at 4:11 AM -0700, "fvarelie"<EMAIL_ADDRESS>wrote:
Hi, I'm using vs 2015 and can't build the project. the error is unable to locate nuget.exe....
btw, is this api compatible with xbee xtend?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub


fix(lint): no-unused-vars + inline import/no-unresolved

:tada: This PR is included in version 1.0.0-alpha.13 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


🐛 Download and store manifests from provider URL

What this PR does / why we need it:
Currently Cluster API Operator keeps refetching data from the specified URL after successful provider installation. It means that in case of network error Operator can't download manifests again and marks providers as unhealthy.
To prevent this, we fetch manifests only once in the beginning and store them in a configmap. For all the following reconciliations we take data from the configmap instead of downloading it from github/gitlab.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #119

/hold cancel


[FEATURE] Commas in Economy

I want money to show with commas. $10,000.00 instead of $ 10000.00
I like vault placeholder,
%vault_eco_balance%
%vault_eco_balance_commas%
It gives user option to use both with and without commas.

This feature has been implemented and will be included in the next update.


videojs-dash.js:146 Uncaught TypeError: Cannot read property 'className' of null

videojs version 5.8.8,
If there are two types, 'rtmp/mp4' and 'application/dash+xml'. When I change 'rtmp/mp4' source to 'application/dash+xml' , video tag can't be created and error is:
   videojs-dash.js:146 Uncaught TypeError: Cannot read property 'className' of null

change to videojs version 5.0.0, still error

Could you provide a reduced test case or jsbin link for this issue?

Closing because of lack of activity, if this is still an issue please reopen. Thank you.

Yes still an issue on firefox. You can check it out here: fartingpuppy. com


Manually update v3 canary package

Summary of the changes
Manually update canary packages to be v3.0.0-canary.0
Manually update regular packages to v3.0.0-alpha.0
Related issue
N/A
Checklist
General

[x] Changes to docs package checked and committed.
[x] All acceptance criteria reviewed and met.

Testing

[x] Relevant unit tests and visual regression tests added.
[x] Visual testing against Figma component specification completed.
[x] Playground stories in React Storybook up to date, with any prop changes and additions addressed.
[x] Compare performance of modified components against develop using Performance addon in React Storybook.

Accessibility

[x] Accessibility Insights FastPass performed.
[x] A11y unit test added and yields no issues.
[x] A11y plug-in on Storybook yields no issues.
[x] Manual screen reader testing performed using NVDA and VoiceOver.
[x] Manual keyboard testing for keyboard controls and logical focus order.
[x] Correct roles used and ARIA attributes used correctly where required.
[x] Logical heading structure is maintained, and the HTML elements used for headings can be changed to fit within the wider page structure.

Resize/zoom behaviour

[x] Page can be zoomed to 400% with no loss of content.
[x] Screen magnifier used with no issues.
[x] Text resized to 200% with no loss of content.
[x] Text spacing increased as per the WCAG 1.4.12 success criterion with no loss of content.

System modes

[x] Browser setting 'prefers reduced motion' tested. No animations or motion visible whilst this setting is on.
[x] Windows High Contrast mode tested with no loss of content.
[x] System light and dark mode tested with no loss of content.
[x] Browser support tested (Chrome, Safari, Firefox and Edge).

Testing content extremes

[x] Min/max content examples tested with no loss of content or overflow.
[x] All prop combinations work without issue.
[x] Tested for FOUC (Flash of Unstyled Content) in both SSR (Server-Side Rendering) and SSG (Static Site Generation) settings.
[x] Controlled and uncontrolled input components tested.
[x] Props/slots can be updated after initial render.

seems to be a lot of relative path entries for other packages - looking into this

seems to be a lot of relative path entries for other packages. i pulled this down & did local install\bootstrap and a lot of relative content was removed:

is it worth regenerating the canary-react package-lock.json and pushing up?

Just tried this and it's not changed anything within the file for me, could it be a difference in dev environment thing?


No Auth Provider found for name "gcp"

For the following code I get an error when trying to connect to a GKE API server with v3.0.0.
FWIW this worked in v3.0.0.beta.0
I have a version of apimachinery that's about 28 days old. Could it have to do with some library incompatibility?
F0712 12:20:52.391235   71066 main.go:73] Could not create Kubernetes ThirdPartyResource API client: No Auth Provider found for name "gcp"

package main

import (
	"flag"

	"github.com/golang/glog"

	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
	"k8s.io/apimachinery/pkg/runtime"
	"k8s.io/apimachinery/pkg/runtime/schema"
	"k8s.io/apimachinery/pkg/runtime/serializer"
	"k8s.io/client-go/kubernetes/scheme"
	"k8s.io/client-go/rest"
	"k8s.io/client-go/tools/clientcmd"
)

type Thing struct {
	metav1.TypeMeta `json:",inline"`
	ObjectMeta      metav1.ObjectMeta `json:"metadata"`
}

// Required to satisfy ObjectMetaAccessor interface
func (m *Thing) GetObjectMeta() metav1.Object {
	return &m.ObjectMeta
}

type ThingList struct {
	metav1.TypeMeta `json:",inline"`
	ListMeta        metav1.ListMeta `json:"metadata"`
	Items           []Thing         `json:"items"`
}

// Required to satisfy ListMetaAccessor interface
func (m *ThingList) GetListMeta() metav1.List {
	return &m.ListMeta
}

func main() {
	kubeconfig := flag.String("kubeconfig", "", "The path to a kubeconfig. Default is in-cluster config.")

	flag.Parse()

	config, err := clientcmd.BuildConfigFromFlags("", *kubeconfig)
	if err != nil {
		glog.Exitf("Could not read kubeconfig %s: %v", *kubeconfig, err)
	}

	groupVersion := schema.GroupVersion{
		Group:   "ianlewis.org",
		Version: "v1alpha1",
	}

	config.GroupVersion = &groupVersion
	config.APIPath = "/apis"
	config.ContentType = runtime.ContentTypeJSON
	config.NegotiatedSerializer = serializer.DirectCodecFactory{CodecFactory: scheme.Codecs}

	schemeBuilder := runtime.NewSchemeBuilder(
		func(scheme *runtime.Scheme) error {
			scheme.AddKnownTypes(
				groupVersion,
				&Thing{},
				&ThingList{},
			)
			return nil
		},
	)
	schemeBuilder.AddToScheme(scheme.Scheme)

	_, err = rest.RESTClientFor(config)

	if err != nil {
		glog.Exitf("Could not create Kubernetes ThirdPartyResource API client: %v", err)
	}

	glog.Infof("OK")
}

$ go version
go version go1.8 darwin/amd64

It seems you need import the auth plugin package
_ "k8s.io/client-go/plugin/pkg/client/auth/gcp"
// or
_ "k8s.io/client-go/plugin/pkg/client/auth"

Is this working as intended? Is there any benefit to not loading auth plugins by default?

Is this working as intended? Is there any benefit to not loading auth plugins by default?

Yes and this is documented in a couple places. The benefit is not overloading regular clients with unnecessary imports. E.g you don't have to import OpenID Connect and Azure packages.
Going to close since it looks like this is working as intended.

@ericchiang Can you point me to where it's documented?
It wasn't here:
https://github.com/kubernetes/client-go/tree/master/examples/in-cluster-client-configuration
https://github.com/kubernetes/client-go/tree/master/examples/out-of-cluster-client-configuration
I just guessed based on reading the source code. It feels like that could be improved. Not that there are not really any "docs" to speak of yet.
While I understand the need to avoid importing packages that aren't needed I question the wisdom of this a bit. I can see folks creating apps and integrations that don't work w/ GKE/Azure/Openid because they don't import the plugins. It feels weird to drop this all of a sudden since it worked in v2.0.0 and v3.0.0.beta.0

@IanLewis a few of the examples have lines like this https://github.com/kubernetes/client-go/blob/53c7adfd0294caa142d961e1f780f74081d5b15f/examples/out-of-cluster-client-configuration/main.go#L31
The actual change was made here https://github.com/kubernetes/kubernetes/pull/41532 during the 1.6 release. Are you sure this changed between v3.0.0.beta.0 and v3.0.0? I'm not seeing them imported by default in v3.0.0.beta.0
https://github.com/kubernetes/client-go/blob/v3.0.0-beta.0/kubernetes/clientset.go
https://github.com/kubernetes/client-go/blob/v2.0.0/kubernetes/clientset.go#L35-Lundefined
I can add a changelog item for the v3.0.0.beta.0

@ericchiang I suppose you're right that it was removed between v2.0.0 and v3.0.0 but still the issue stands.

More folks noticing this on twitter: https://twitter.com/davecheney/status/915770572017037312

FYI, I hit this error message, and the 2nd Google search result led me here. Glad this has the solution!
As for where to document, perhaps somewhere near the docs for "gcloud container clusters get-credentials"?

I think the error error message could tell people what to do without them googling to firgure it out. e.g. append this:
suggestion: add this import: _ "k8s.io/client-go/plugin/pkg/client/auth/gcp"

This issue is happening in my kubectl client after upgrading gcloud to v299.0.0. Is it relathere a bug somewhere?


Hyper on run command opens the cmd console as well

Hi,
I installed Hyper for Windows 10 and selected the "Install Hyper CLI command in PATH". Now when I type hyper at the Run command, it opens the Hyper terminal as well as the cmd console as well.

I can confirm this. I have similar issues with other node apps and determined that in those apps it is caused by how node child_process works. A similar type of issue would occur if you use PM2 to run a node app that forks a process. To get around the issue, I had to do this in some of my apps:
spawn( 'node' [ path.join( __dirname, 'something.js' ) ], { windowsHide: true } )

This may not be the most reliable way to achieve this. Problem is likely coming from the last line in this block
https://github.com/zeit/hyper/blob/34fb41c23b26695b12ec4c8cef28bc4539e781b3/cli/index.js#L185-L197


Make witdh of orphaned serial dialog wider

fixes #1094

[x] EN and DE work
[x] I adjusted some layout options and removed the individualized dimensions from the DE version

Ooooo.


TI flow incorrect

Describe the bug
As a Trusted Intermediary, the current flow is:

Login in as a TI
Land on the Program Cards page
Click on "Add a Client"
Land on Client Roster page

Landing directly on the Program Cards page is confusing and has led to TIs applying for programs for themselves (not on behalf of clients). To avoid confusion, TIs should land directly on the Client Roster page.
Screenshots
Current State

Desired Flow (TIs should land on the Client Roster directly after logging in)

Fixed in #4821


Migration to Storybook 4 issue with webpack config

Hi Team,
During migration the storybook to new version 4.0.7, there is an issue in starting the storybook. Webpack is freaking out. The application is based on Create-React-App 1 and storybook has an extended webpack.config.js.  There are two issues:

Failed to load presets
Cannot read property 'rules' of undefined (During the loading custom webpack config task)

The webpack config looks like:
const path = require("path")
const ForkTsCheckerWebpackPlugin = require('fork-ts-checker-webpack-plugin');

module.exports = (baseConfig, env, config) => {
  baseConfig.module.rules.push({
    test: /\.css$/,
    loaders: [
      'style-loader',
      'css-loader'
    ],
    include: path.resolve(__dirname, "../")
  });
  config.module.rules.push({
    test: /\.tsx?$/,
    loader: [
      'thread-loader',
      'ts-loader?happyPackMode=true',
      'react-docgen-typescript-loader'
    ]
  }, {
    test: /\.md$/,
    loader: [
      'thread-loader',
      'markdown-loader'
    ]
  })
  config.plugins.push(new ForkTsCheckerWebpackPlugin({
    checkSyntacticErrors: true,
    watch: path.resolve(__dirname, "../stories"),
    workers: ForkTsCheckerWebpackPlugin.TWO_CPUS_FREE,
  }))
  config.resolve.extensions.push(".ts", ".tsx")
  return config;
}

Waiting for some pointers from your side in order to rectify the problem.
Thanks in advance
Amit

So, I understand that before migration to v4 everything worked ok with this custom webpack.config.js? Can you also share some stack trace / error log?

I tried by removing the rule as suggested above, but still getting the same error.

On a new project (CRA 2.1.1 + Storybook 4.0.1) https://storybook.js.org/configurations/typescript-config/ didn't work for me today.

v4.1.0-alpha already supports TS out of the box for the CRA2 users, without creating a custom webpack.config. Can you check this ?

@amittkSharma , do you have a reproduction repo?

@igor-dv: Will create one tomorrow and send you the repo name.
Thanks

Amit

@igor-dv: Here is the basic repo structure, please take a look at it. In my system, I am not able to reproduce the above error as there were some updates in the past but feel free to check it.
Thanks
Amit

This is issue is closed as reproduction repo is provided to the concerned team.

Amit

Same problem here. @amittkSharma have you found a solution?


Great work implementing local storage!

Form Validation:
The current form validation only checks if the email is in lowercase. It would be beneficial to also check if the email is in a valid format and if the other fields in the form are filled in. This could be done using a regular expression for the email validation and simple checks for the other fields.

Thanks for finding the bug, It will be fixed for better user experience.


charts/timescaledb-single: Allow to set priority class

What this PR does / why we need it
Adds functionality to set priority class
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #607

Special notes for your reviewer
Checklist

[x] Chart Version bumped
[x] CLA signed

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Mayer Maximilian seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi everyone. Any chance that this will be merged? Anything I can do to help? @mathisve


Parse error

http://bsc1.cn/download/dicoms/1
http://bsc1.cn/download/dicoms/2
http://bsc1.cn/download/dicoms/3
http://bsc1.cn/download/dicoms/4
http://bsc1.cn/download/dicoms/5
http://bsc1.cn/download/dicoms/6

Please provide more information.

DICOM Dump with Data Dictionary
I use this page to parse those files get this error.
Status:Ready - no pixel data found; file size 6.802 MB; parse time 3ms; SHA1 = 3f4d602033b499b1099ecd72ae3727848aa99147 Digital X-Ray Image Storage - For Presentation; Implicit VR Little Endian: Default Transfer Syntax for DICOM

I don't konw, Is the file standard or my setting's problem.

It seems like #89 , But his file can be parsed successed, mine can not.

I try the dcm4che(A Java dicom util) to paser these files, parse successed.
This is my code.
`DicomInputStream dicomInputStream = new DicomInputStream(stream2);
Attributes attributes = dicomInputStream.readDataset(-1, -1);
        attributes.setString(Tag.SpecificCharacterSet, VR.CS, "GB18030");//  <======

        String studyId = attributes.getString(Tag.StudyID); // study id
        String studyuid = attributes.getString(Tag.StudyInstanceUID); // study uid
        String seriesId = attributes.getString(Tag.SeriesInstanceUID); //series id
        String instanceId = attributes.getString(Tag.SOPInstanceUID); // instance id
        String seriesNumber = attributes.getString(Tag.SeriesNumber);
        String instanceNumber = attributes.getString(Tag.InstanceNumber);
        String patientID = attributes.getString(Tag.PatientID);
        String patientName = attributes.getString(Tag.PatientName);
        String bodyPart = attributes.getString(Tag.BodyPartExamined);
        String modality = attributes.getString(Tag.Modality);
        Date studyTime = attributes.getDate(Tag.StudyDate);
        String accessionNumber = attributes.getString(Tag.AccessionNumber);
        String transferSyntax = dicomInputStream.getTransferSyntax();
        String patientAge = attributes.getString(Tag.PatientAge);`

I watch the source code and try to fix this problem, But failt, I have no idea yet

I took a look at this. It looks like the problem is that the file has multiple Transfer Syntax UIDs: at offset 260 there's one for JPEG Lossless, Non-Hierarchical (Process 14) and at offset 356 there's one for Implicit VR Little Endian. Given that the file is actually explicit VR little endian and that the implicit VR transfer syntax is located after the file meta information, I'm assuming the second transfer syntax is a mistake. Indeed, if you delete the 26 bytes starting at offset 356, you'll find the file can be parsed properly.

So these files are not standard DICOM file, If the DCM4CHE can parse successed, we can use it's logic to deal with this problem.

@suoc I'm going to go ahead and close this issue since we know what the problem is and that it's not related to the parser itself.


[Bug][jira plugin] devlake collection jira server information

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
lake-v0110-devlake-1    | time="2022-07-20 01:19:53" level=error msg=" [task #24] [jira] retry #2 for http code error[404]:[{"errorMessages":["用户“aa@bbb.com”不存在"],"errors":{}}]"
lake-v0110-devlake-1    | time="2022-07-20 01:19:54" level=error msg=" [task #24] [jira] retry #2 for http code error[404]:[{"errorMessages":["用户“cc@bbb.com”不存在"],"errors":{}}]"
lake-v0110-devlake-1    | time="2022-07-20 01:19:56" level=error msg=" [task #24] [jira] retry #2 for http code error[404]:[{"errorMessages":["用户“ggg@bbb.com”不存在"],"errors":{}}]"
execute task fail
What you expected to happen
collection jira server information success
How to reproduce
curl http://<IP_ADDRESS>:8080/rest/agile/1.0/board/85/issue?expand=changelog&jql=ORDER+BY+created+ASC&maxResults=100&startAt=0
"changelog":{"startAt":0,"maxResults":13,"total":13,"histories":[{"id":"148308","author":{"self":"http://<IP_ADDRESS>:8080/rest/api/2/user?username=aaa","name":"aaa","key":"aaa","emailAddress":"aa@bbb.com","avatarUrls":{"48x48":"http://<IP_ADDRESS>:8080/secure/useravatar?ownerId=aaa&avatarId=11401","24x24":"http://<IP_ADDRESS>:8080/secure/useravatar?size=small&ownerId=aaa&avatarId=11401","16x16":"http://<IP_ADDRESS>::8080/secure/useravatar?size=xsmall&ownerId=aaa&avatarId=11401","32x32":"http://<IP_ADDRESS>:8080/secure/useravatar?size=medium&ownerId=aaa&avatarId=11401"},"displayName":"sss
no accountID tag
modify apiv2models/user.go
func (u *User) getAccountId() string {
if u == nil {
return ""
}
if u.AccountId != "" {
return u.AccountId
}
//add this code,run success,no test for jira cloud
if (u.AccountId == "") && (u.Name != "") {
return u.Name
}
return u.EmailAddress
}
Anything else
No response
Version
v0.11.0
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@jinzhu2002 Interesting, so right now you can make sure that you can get jira user info by the token, but devlake still return 404?

yes ，jira collection tasks run success，no any Errors。but no test for jira cloud

Right now, jira collection can work successfully.

@warren830 I encounter the same problem, how to solve it?

Hi @kyleliu1008
From @jinzhu2002 's suggestion, you can modify this file plugins/jira/apiv2models/user.go to the below
func (u *User) getAccountId() string {
	if u == nil {
		return ""
	}
	if u.AccountId != "" {
		return u.AccountId
	}
	//add this code,run success,no test for jira cloud
	if (u.AccountId == "") && (u.Name != "") {
		return u.Name
	}
	return u.EmailAddress
}

we will fix this bug in the next version

This should not be closed until we solve the problem in our main branch.


Images Not Aligned In Two Column Layout

When I try to have a slide with two columns and an image on the right side, I have to align that image when I generate my slides. Here's an example slide:
# Foo

- Foo bullet
- Bar bullet
- Baz bullet

{.column}

![](https://upload.wikimedia.org/wikipedia/commons/thumb/2/2f/Google_2015_logo.svg/1200px-Google_2015_logo.svg.png)

Thanks for the tool! Really enjoy being able to create my decks and have them be available for PRs!

I have the same problem. It does not seem possible to have an image on the right side :/

Yeah, I'll have to rework how images are handle. They're not currently associated with the text containers/shapes, which is what the columns.
Will think about how to handle this better.

Fixed in master, will be in next release (0.5). Should get around to pushing that shortly.

Nice! Thank you @sqrrrl !!


enh: chat auto tag

Just a quick note. The side bar may need a way to collapse/resize and/or have a scrollbar for tags so that the chat history list isn't shrunk down so much. At this point, I can only see it.

@TFWol already addressed.

Oh, is that from Dev branch?

@TFWol Yes, I'd appreciate if you could hop on our dev branch so we can iron out all the bugs!

Added to dev!


[KYUUBI #1067]catch exception if logRoot is not exist

BUGFIX:
#1067
This PR is just for version_1.3.0
Why are the changes needed?

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Codecov Report

Merging #1154 (2d9aaa5) into branch-1.3 (a03014d) will decrease coverage by 0.05%.
The diff coverage is 44.44%.

@@               Coverage Diff                @@
##             branch-1.3    #1154      +/-   ##
================================================
- Coverage         80.58%   80.52%   -0.06%
  Complexity           11       11
================================================
  Files               146      146
  Lines              5382     5387       +5
  Branches            638      640       +2
================================================
+ Hits               4337     4338       +1
- Misses              668      670       +2
- Partials            377      379       +2

Impacted Files
Coverage Δ

...rg/apache/kyuubi/engine/spark/SparkSQLEngine.scala
65.27% <40.00%> (-1.87%)
:arrow_down:

...pache/kyuubi/engine/spark/events/EngineEvent.scala
85.48% <50.00%> (-2.66%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a03014d...2d9aaa5. Read the comment docs.


updated rdp shortpath public

Hi,
I've added a connection diagram to understand more the connection flow for RDP Shortpath for public networks.

I'm unable to add the svg file to the media folder.

Learn Build status updates of commit 05318d4:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/virtual-desktop/rdp-shortpath.md
:warning:Warning

Details

articles/virtual-desktop/rdp-shortpath.md

Line 124, Column 1: [Warning: file-not-found - See documentation] Invalid file link: 'media/azure-virtual-desktop-rdpshortpath-public-connection'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Discussing internally.
#hold-off

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


why

Can't find a usable init.tcl in the following directories
This probably means that Tcl wasn't installed properly.

Hi @yiyinying, there isn't enough information here to provide a helpful response. The error you've provided is related to the TCL library not being installed or discoverable on your machine. That's a system level dependency that wouldn't usually be handled by pipenv.
I'm going to close this since the requested information wasn't provided and we don't have a path forward. Feel free to reopen when more you have more details. Thanks!


Covid-19 Detection using Chest X-Rays

Hey @smaranjitghose  Can I work on this issue?

Can i add my project to this ?

Can i add my project to this ?

As in? Your trained model?

My Repo : https://github.com/Siddhant-K-code/COVID-19-RAPID-TESTER

My Repo : https://github.com/Siddhant-K-code/COVID-19-RAPID-TESTER

Sure, you can give a pr with this but I would like to see the sensitivity, specificity, F-1 Score, and Recall as well


Issue with k8s.io/docs/concepts/configuration/assign-pod-node/

**This is a...bug report or feature request. i am not sure **

[x] Feature Request
[x] Bug Report

Problem:
While using replication factor >1; i intend to run same container on different nodes in cluster, and i also want them to be deployed on certain nodes,so i provided replication count and node selector in yaml like below..
apiVersion: extensions/v1beta1
spec:
replicas: 3
.........
........
........
nodeSelector:
node: node04
node: node05
node: node16
3 containers got deployed but all landed on node 16!!! I want one container per node and on node 04,05, and 16
Proposed Solution:
Page to Update:
https://kubernetes.io/docs/concepts/configuration/assign-pod-node/

kubectl version
Client Version: version.Info{Major:"1", Minor:"5", GitVersion:"v1.5.2", GitCommit:"a55267932d501b9fbd6d73e5ded47d79b5763ce5", GitTreeState:"clean", BuildDate:"2017-04-14T13:36:25Z", GoVersion:"go1.7.4", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"5", GitVersion:"v1.5.2", GitCommit:"a55267932d501b9fbd6d73e5ded47d79b5763ce5", GitTreeState:"clean", BuildDate:"2017-04-14T13:36:25Z", GoVersion:"go1.7.4", Compiler:"gc", Platform:"linux/amd64"}

@ankitpateledu This looks less like an issue with docs and more like a request for support. I encourage you to bring your question to the #kubernetes-users channel in Kubernetes slack. Please feel free to re-open this issue if there's a specific docs change you'd like to address here.