Mask RCNN, HTC models, why only backbone has norm_cfg

I checked config files for mask RCNN, HTC,  I found only the backbone part has norm_cfg, could you explain why?
Why FCN mask head don't have norm_cfg ?
Thanks a lot!
model = dict(
type='HybridTaskCascade',
pretrained='torchvision://resnet50',
backbone=dict(
type='ResNet',
depth=50,
num_stages=4,
out_indices=(0, 1, 2, 3),
frozen_stages=1,
norm_cfg=dict(type='BN', requires_grad=True),
norm_eval=True,
style='pytorch'),

I also found config in "gn" and "gn+ws", In those config files, all parts have norm_cfg.
norm_cfg = dict(type='GN', num_groups=32, requires_grad=True)
model = dict(
pretrained='open-mmlab://detectron/resnet50_gn',
backbone=dict(norm_cfg=norm_cfg),
neck=dict(norm_cfg=norm_cfg),
roi_head=dict(
bbox_head=dict(
type='Shared4Conv1FCBBoxHead',
conv_out_channels=256,
norm_cfg=norm_cfg),
mask_head=dict(norm_cfg=norm_cfg)))

This is a common practice. For mask r-cnn, faster r-cnn, and retinanet, they only contain BN in backbone and do not use BN in the head. If you try to add BN in the head, the performance will drop since the batch size is small. However, you could also try to use GN in the head like FCOS, which could improve performance.

Thanks a lot for your comments

This is a common practice. For mask r-cnn, faster r-cnn, and retinanet, they only contain BN in backbone and do not use BN in the head. If you try to add BN in the head, the performance will drop since the batch size is small. However, you could also try to use GN in the head like FCOS, which could improve performance.

hi, i want to add shared_head in htc_roi_head, but assert not self.with_shared_head  # shared head is not supported in htc_roi_head.  So  if i add the shared_head what will happen?


Feature/security or

remove unnecessary comment,
add or-functionality to parser_test

@ubogdan I added or to the general check.
How can i use this functionality locally? I tried using the master branch and i saw my code in the vendor-folder, but it didn't work.

@ubogdan Hey ubogdan I've been looking atleast 5h for the reason my code doesn't work. Everything executes fine. Every tests is ok. But in the End the file does not contain the object parseSecurityComment returns. Could you give me a hint? I don't know where to search anymore.
The input comment seen in a Test:

The result of ParseSecurityComment:

Left -> comment in goFile right-> result in swagJson

Hey ubogdan sorry for the late response. The function works exactly as expected. It adds both values into the array which creates a security option with two options so to say. I do not understand where this gets written into a file because thats where the error must happen.

Hey ubogdan sorry for the late response. The function works exactly as expected. It adds both values into the array which creates a security option with two options so to say. I do not understand where this gets written into a file because thats where the error must happen.
This test works fine and it's exactly what i need in my swaggerfile

I made some more testing and found out that comments in Tests must be read correctly cuz following test ran succeeded. I really don't know it doesn't look like my code gets executed at all. Does the init command of the cli somehow use an older version of swag?


Added maxDepth config option

This PR introduces a new maxDepth() config option to limit the depth of the tree.

Thanks!


SignUp form is not validated

Description
I am able to signup with the invalid email and blank username.
This will only unreal users to access your website.
Steps to reproduce
This bug can be resolved by-
adding regex in all fields of signup page which will accept only valid entries.
Screenshots

Desktop (please complete the following information)
Windows, Crome
Smartphone (please complete the following information)
No response
Additional information
No response

I request you to assign me this issue in order to resolve it

this is duplicate issue, closing it


Route management

is there any option/api for managment routes?
How deal with it.
Routs basicually need baseURL, named route for generate url base on parameters and etc.
Thanks for your awesome project bwt

Thought I'd shime in here to ask my question: How do you integrate server side routing with client-side routing (as mentioned in the README)?

Answer to this came late but it's here: https://fastify-vite.dev/guide/router-integration


Using Context::Call at unary_expr & array extract value

And combine Array-extract with Unary-expr.
We don't have to manage so many things but the same concept.
Ref #79

array extract expression should be seen as binary expression.

This issue has to work with Template(Generic), so became not critical now.

To hard to implementing now, need a lot of refactoring, close


reference

https://docs.upbit.com/reference/전체-출금-조회
https://wikidocs.net/78954
https://packaging.python.org/en/latest/tutorials/installing-packages/

https://github.com/chaos314

https://github.com/uJhin/upbit-client
https://ujhin.github.io/upbit-client-docs/#8240d4d69a

https://github.com/Wollala/wollala-upbit

https://github.com/seunggabi/upbit-client
사용하면 될듯


JP translation 92% complete after stiching in prior work

The home stretch for JP!  ひじょにまんぞく！

Full attribution to original BSD copyright holders intact
Full commit history intact for translation strings (Chie Taguchi)
merged against current OPNsense .pot file using msgmerge

Holy mother of translations. Great work, thank you!! :)

@dotike small nitpicks:
https://github.com/opnsense/core/blob/master/lang/ja_JP.po#L1-L4
ja_JP.po:11428 (Send/Expect translation got lost... maybe intended?)

L1-4, totally a regression in all the merging of our old work, my bad.
--
This, I believe, is the intended translation, "Send/Expect",
https://github.com/opnsense/core/blob/master/lang/ja_JP.po#L11428
One of the lines in the source is:
$types = array("icmp" => gettext("ICMP"), "tcp" => gettext("TCP"), "http" => gettext("HTTP"), "https" => gettext("HTTPS"), "send" => gettext("Send/Expect"));
This is a case where a native Japanese speaker can disambiguate, the english term fits in clearly in the context of the rest of the list.

ack, thanks for double-checking


debug steamturbine and require input for Boiler

Require fuel_cost_per_mmbtu as an input for Boiler tech
Correct how steam turbine defaults are determined

I just realized that this PR was created back in March when we were last working on adding these techs into REopt_API. It should have some relevant changes.


[DOCFIX] Update List of Metrics table formatting

What changes are proposed in this pull request?
update plain text tables to table formatting
Why are the changes needed?
better user interface
Does this PR introduce any user facing changes?
webui

alluxio-bot, merge this please


Fix rest stop bug

What type of PR is this?
/kind bug
What this PR does / why we need it:

resolve scheduler not exit when call kubescheduler Stop
resolve restmapper getAPIGroupResourcesres error when restConfig is nil

Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
none
Does this PR introduce a user-facing change?

none

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

none

waiting for https://github.com/kubernetes/kubernetes/pull/116583 to merge

@fanhaouu can you rebase and resolve the conflict first?


Question about the Announcement

Just want to know if you could atleast tell us if we should have BTC prepared for the announcement. I would like to be prepared as much as I can and it would be better to have BTC on the platform.
Will the news in anyway be better if I had BTC on the platform?
Thanks for answering in advance

That's a really subjective sort of question...
The news will be what it is.. if you want to be in a position to buy because of the news.. well then best to have it on there I guess.

Not a problem . I'm sure the admin can answer me or not since he is the
one who knows the news. I think any investor would want to be prepared
atleast. Thanks for your comment though.
------ Original Message ------
From: "Blitzkitten"<EMAIL_ADDRESS>To: "monyxcoin/monyx-coin"<EMAIL_ADDRESS>Cc: "2themoonandback"<EMAIL_ADDRESS>"Author"
<EMAIL_ADDRESS>Sent: 2018-02-19 12:26:08 AM
Subject: Re: [monyxcoin/monyx-coin] Question about the Announcement
(#242)

That's a really subjective sort of question...
The news will be what it is.. if you want to be in a position to buy
because of the news.. well then best to have it on there I guess.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/monyxcoin/monyx-coin/issues/242#issuecomment-366592021,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AieP-U0wmXITbRHMnTb3UKZWibjum1yOks5tWQXwgaJpZM4SJ9Ns.

I doubt you'll get an answer to be honest...
what you're asking for is a pretty big 'spoiler alert'.. also making devs responsible for your level of reaction to news is a risk they shouldn't take.
If they say 'yes, get your money ready' and then you feel let down by the news, who is responsible?
As an investor you are prepared, you know there is going to be a news update

How about you just let the Admin answer so we don't even have to
speculate what he is going to say or better yet lets wait for his
response or non response than have a talk sorry it's just more efficient
for my time.
Thanks.
------ Original Message ------
From: "Blitzkitten"<EMAIL_ADDRESS>To: "monyxcoin/monyx-coin"<EMAIL_ADDRESS>Cc: "2themoonandback"<EMAIL_ADDRESS>"Author"
<EMAIL_ADDRESS>Sent: 2018-02-19 1:10:36 AM
Subject: Re: [monyxcoin/monyx-coin] Question about the Announcement
(#242)

I doubt you'll get an answer to be honest...
what you're asking for is a pretty big 'spoiler alert'.. also making
devs responsible for your level of reaction to news is a risk they
shouldn't take.
If they say 'yes, get your money ready' and then you feel let down by
the news, who is responsible?
As an investor you are prepared, you know there is going to be a news
update
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/monyxcoin/monyx-coin/issues/242#issuecomment-366597138,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AieP-Z1FsbKN0dysBlDgOVJzgrXkYk6zks5tWRBcgaJpZM4SJ9Ns.

No problem..
Have a good one

you too :)
------ Original Message ------
From: "Blitzkitten"<EMAIL_ADDRESS>To: "monyxcoin/monyx-coin"<EMAIL_ADDRESS>Cc: "2themoonandback"<EMAIL_ADDRESS>"Author"
<EMAIL_ADDRESS>Sent: 2018-02-19 1:13:59 AM
Subject: Re: [monyxcoin/monyx-coin] Question about the Announcement
(#242)

No problem..
Have a good one
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/monyxcoin/monyx-coin/issues/242#issuecomment-366597565,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AieP-ZJbVvFsmHNFwuFcrWuvjquJPkaSks5tWREngaJpZM4SJ9Ns.

 You would be better off playing with hot wheels cars than investing in ICOS!

Oh man shutup...  Are you the admin
? Are some psychic who can see the future? Is Monyx paying you hourly
for support?
Shutup no one asked you for help.

You would be better off playing with hot wheels cars than investing in
ICOS!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/monyxcoin/monyx-coin/issues/242#issuecomment-366714636,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AieP-Wlckni9kmxa33dCqcyxUQKBKutJks5tWYnIgaJpZM4SJ9Ns.

Chass3768 you’re right on the money buddy


Update CONTRIBUTING about staging

Because #37 and #52,
need to explain the 3 principles of issue #52.

wait and see #37

As we discussed @guillaumearm we should merge both redux-polyglot and react-redux-polyglot NPM modules in the same repository to ensure full synchronicity and simpler tests.
I close this PR.

OK to delete branch doc/contributing @guillaumearm ?


Update CONTRIBUTING.markdown

Remove reddit link as it's not working anymore

@ahmadalfy remove the reddit link since this board is "banned"


Update offline_server.py

disabling gradient calculations when running the rnnt the model

Thanks! Could you please also do this for the streaming server?


Relaton XML in DB have funny names

Sometimes relaton makes files with name like:
iso_iec_guide_99_2007,_1.1,_modified_\342\200\224_the_notes_have_been_changed..xml

The names should only include the document identifier, not the modification or clause ID. riboseinc/tc211-termbase/issues/7

it's a problem of riboseinc/tc211-termbase/issues/7


Added wildcard css tag support

Requesting review.
Attention please on CssTokensBuilder, I've made it in reverse.

Review please.


[question] Where is the output of the hooks being logged to?

I tried adding a done hook to list the files in my CI, but I can't see any output of it even when the logLevel is set to "info". Maybe I'm using it wrong, any help would be appreciated: Here's my config:
export default defineConfig(async () => {
  return {
    root: '.',
    input: {
      path: './openapi.yml',
    },
    output: {
      path: './src/gen',
    },
    logLevel: 'info',
    hooks: {
      done: ['pwd', 'ls -lah', 'prettier --write .'],
    },
    plugins: [
      createSwagger({ output: false }),
      createSwaggerTS({ output: 'models.ts' }),
      createSwaggerClient({ client: './src/client.ts', output: 'requests.ts' }),
      createSwaggerTanstackQuery({ client: './src/client.ts', output: 'hooks.ts' }),
    ],
  };
});

And the output:
✔ 💾 Config loaded(kubb.config.ts)
✔ 🚀 Build completed(./openapi.yml)
✔ 🪂 Executed hooks(done) pwd
✔ 🪂 Executed hooks(done) ls -lah
✔ 🪂 Executed hooks(done) prettier --write .

  Plugins:      4 successful, 4 total
Generated:      4 files
     Time:      19.085s

Thank you for this wonderful lib!

5min work so here is the PR already: https://github.com/kubb-project/kubb/pull/243


Since Angular 17, interface ServeBuilderOptions should have property buildTarget but not browserTarget

Command
other
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
I'm writing a custom installation schematics for my lib, and I get a TS error, when trying to set inside the angular.json (TS typeWorkspaceSchema) the property in "buildTarget" in the "serve" architect section (TS type ServeBuilderOptions)
AFAIK, the name of the property got changed only in Angular 17.
Minimal Reproduction
/**
 * In angular.json: use "noSsr" configuration in "serve" architect section
 */
function useNoSsrConfigurationInNgServe(
  spartacusOptions: SpartacusOptions
): Rule {
  return (tree: Tree, context: SchematicContext): Tree => {

    const { path, workspace: angularJson } = getWorkspace(tree);
    const projectName = getDefaultProjectNameFromWorkspace(tree);

    const project = angularJson.projects[projectName];
    const architect = project.architect;
    const serve = architect?.serve;
    const configurations = serve?.configurations;
    const production = configurations?.production;
    const development = configurations?.development;

    const updatedAngularJson = {
      ...angularJson,
      projects: {
        ...angularJson.projects,
        [projectName]: {
          ...project,
          architect: {
            ...architect,
            serve: {
              ...serve,
              configurations: {
                ...configurations,
                production: {
                  ...production,
                  buildTarget: `${production?.buildTarget},noSsr`,  // <-- TS error, `buildTarget` property doesn't exist in `ServeBuilderOptions`
                },
                development: {
                  ...development,
                  buildTarget: `${development?.buildTarget},noSsr`, // <-- TS error, `buildTarget` property doesn't exist in `ServeBuilderOptions`
                },
              },
            },
          },
        },
      },
    };

    tree.overwrite(path, JSON.stringify(updatedAngularJson, null, 2));
    return tree;
  };
}

(the full context and license of the snippet above can be found in the original repo and PR here)
Exception or Error
No response
Your Environment
Angular CLI: 17.0.5
Node: 18.15.0
Package Manager: npm 9.5.0
OS: darwin arm64

Angular: 17.0.5
... animations, cli, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... platform-server, pwa, router, service-worker, ssr

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1700.5
@angular-devkit/build-angular   17.0.5
@angular-devkit/core            17.0.5
@angular-devkit/schematics      17.0.5
@schematics/angular             17.0.5
ng-packagr                      17.0.2
rxjs                            7.8.1
typescript                      5.2.2
webpack                         5.76.3
zone.js                         0.14.2

Anything else relevant?
I've applied locally a workaround:
// Fix typing bug in @schematics/angular/utility/workspace-models since v17:
declare module '@schematics/angular/utility/workspace-models' {
  interface ServeBuilderOptions {
    /**
     * Since ng17 it's no more "browserTarget" but "buildTarget" property
     */
    buildTarget: string;
  }
}

Hi @Platonn, anything which is not exported directly from @schematics/angular/utility is considered as private API.


igc pipeline --tekton doesn't create git or image resource

To reproduce
igc pipeline --tekton
Getting git parameters
  Project git repo: https://github.com/csantanapr-cn-toolkit-tests/cn-node-typescript.git
? Provide the username: csantanapr
? Provide your password/personal access token: [hidden]
? Provide the branch the pipeline should use: master
Setting up dev namespace
Checking for existing namespace: dev
Setting up pull secrets
Setting up tls secrets
Adding pull secrets to serviceAccount
Copying ConfigMaps
Copying Secrets
Copying Tekton tasks
Copying Tekton pipelines
Copying Jenkins credentials
Creating service account: pipeline
Creating Git PipelineResource
Creating Image PipelineResource
? Select the Pipeline to use in the PipelineRun: igc-nodejs
Creating PipelineRun for pipeline: igc-nodejs

Then check the resources
oc get pipelineresource
No resources found in dev-cs1 namespace.

tkn resources ls
No pipelineresources found.

By default, the igc pipeline command will put the resources in the dev namespace if you don't specify one using the -n arg. At the moment, the CLI is not aware of the current project you are on when using OpenShift

Using old version of cli


Facebook stops responding after a while

We are also using the Facebook client, and we seem to be experiencing the same issue as reported by Ran (reported on your weblog). We request the facebook api every 30 seconds for an update, and do that for multiple users (separately in their own respective loops). After a while, facebook stops responding, and all requests freeze and return nothing. Only after a restart of node everything works fine again, until facebook stops responding again. Do you have any idea? Could it be caused by the fact that every request is done with the same agent?
I don't believe we are hitting some Facebook limit, otherwise a restart wouldn't help. Any help here is greatly appreciated.
The freezing issue could be related to an https issue in node, see https://github.com/joyent/node/issues/728 which is fixed in node after v0.4.12, but this is just a wild guess.

This is the code, which does the request:
https://github.com/DracoBlue/node-facebook-client/blob/master/lib/facebook-client/FacebookClient.js#L18
It actually does not handle the case of an timeout on it's own, but waits for an error event. Can you confirm that those requests are made, but not closed?
What version of nodejs are you using?

We wrapped it in a timeout, and at some point we only get timeouts:
facebookClient.getSessionByAccessToken(self.auth.token)(function(fbSession) {
  var timedOut = false;
  var callTimeout = setTimeout(function() {
    timedOut = true;
    return callback(new Error("We had some problems contacting the Facebook API"));
  }, 18000);
  fbSession.graphCall('/me/home', params)(function(response) {
    if (timedOut) return null;
    clearTimeout(callTimeout);
    // do stuff with response

So far it has been hard to reproduce. It seemed we could reproduce it once or twice by switching from one wireless network to another (or cable). And only after a restart of node facebook would stop producing timeouts.
We use node v0.4.12

I guess this would leave the "dead" request still in the FacebookClient.
Can you put your setTimeout-handling into the FacebookClient's doRequest method and request.abort() (remember to try+catch) the request in case of timedOut is true?

Makes sense to have the timeout handling in the actual facebookClient's doRequest. I'll see what I can do, and try to roll it out afterwards to see if it still generates the issue.

We found out that by pointing api.facebook.com and graph.facebook.com to nowhere in our hosts file, we would get the timeout events just like you would expect, only when we stopped pointing the urls to nowhere, the timeouts would continue (and would need a restart).
When we moved the timeout logic into the client, it seemed to have fixed this issue. doRequest() now looks like this:
function doRequest(host, port, path, secure, method, data, parser) {
return function(cb) {
    var protocol = http;
    if(secure) protocol = https;
    var options = {
        host: host,
        port: port,
        path: path,
        method: method || 'GET'
    };
    var timedOut = false;
    var callTimeout = null;

    var request = protocol.request(options, function(response){
        response.setEncoding("utf8");

        var body = [];

        response.on("data", function (chunk) {
            body.push(chunk);
        });

        response.on("end", function () {
            if (timedOut) return null;
            clearTimeout(callTimeout);
            cb(null, parser(body.join("")));
        });
    });

    callTimeout = setTimeout(function() {
          console.debug("[FacebookClient] TIMEOUT");
          timedOut = true;
          request.abort();
          return cb(new Error("Facebook API timeout"));
        }, 10000);

    if (data != null) {
        request.write(querystring.stringify(data))
    }
    request.end();
};
};

The callback now contains err and response. I set the timeout on 10 sec, but it would perhaps be nice to have that configurable. If you can confirm it, and patch the code, then we will role it out in production to see if it fixes our problem.


Incorrect links on Contribute page

Affected page

https://contribute.freecodecamp.org/#/index?id=translations
To reproduce

The Italian and Portuguese curriculum links in the "Translations" section are both pointing to the Spanish page.
Recommended fix, suggestions

Change the Italian link to https://www.freecodecamp.org/italian/learn
Change the Portuguese link to https://www.freecodecamp.org/portuguese/learn/
The file that needs to be updated: https://github.com/freeCodeCamp/freeCodeCamp/blob/main/docs/index.md#translations

Screenshots

This issue is open for first-time contributors only.
Please make sure you read our contribution guidelines. Join us in our chat room or the forum if you need help contributing, our moderators will guide you through this.
Sometimes we may get more than one pull requests. We typically accept the most quality contribution followed by the one that is made first.
Happy contributing!

Hi fixed this issue #44809


Modify spread typescript definition

Seems like the accurate typescript definition for spread?  Maybe I'm missing something I'm not too active with open source projects so I apologize if this is incorrect.

Coverage remained the same at 92.876% when pulling 45d64cb40dc6498c75f6da4256d1320958221625 on jonathanfishbein1:master into 8332392bfdf55602d4e0cd120f00d58b381063a4 on mzabriskie:master.

Closing in favor of #419.


New version: ProximalOperators v0.11.0

Registering package: ProximalOperators
Repository: https://github.com/kul-forbes/ProximalOperators.jl
Created by: @lostella
Version: v0.11.0
Commit: 3951d475a21c1690566bb61ff86e67a1bef10bfc
Reviewed by: @lostella
Reference: https://github.com/kul-forbes/ProximalOperators.jl/commit/3951d475a21c1690566bb61ff86e67a1bef10bfc#commitcomment-39554140

Is the automatic BREAKING label due to the more restrictive dependencies versions?


Add GraphQL Support

Add GraphQL support to the API Gateway, with ability to consolidate responses from multiple services, bases on the GraphQL schema definition.

Is this what you are looking for? https://github.com/MerlinLabs/moleculer-graphql#readme

My new implementation: https://github.com/moleculerjs/moleculer-apollo-server


[iOS] Replace iOS underlying implementation

https://github.com/jathu/UIImageColors is archived and quite outdated.
Consider a popular fork: https://github.com/FelixHerrmann/UIImageColors

Thanks! New library looks cool, but I'll wait for it to have more traction (it's currently at 37 stars) especially since the entire library was re-written. There are no issues in consuming the unmaintained version of the library at the moment but I'll keep an eye out 👍


Selenoid did not delete running containers when exits with panic

I use GGR 1.1.3 and Selenoid 1.1.1
And periodically Selenoid stops with this error:
panic: send on closed channel

goroutine 426219 [running]:
github.com/aerokube/selenoid/service.wait.func1(0xc4201df940, 0x16, 0xc42041b080)
	/home/travis/gopath/src/github.com/aerokube/selenoid/service/service.go:62 +0x115
created by github.com/aerokube/selenoid/service.wait
	/home/travis/gopath/src/github.com/aerokube/selenoid/service/service.go:68 +0x8c

After this Selenoid stops and left all containers with browsers as is.
As i understand, this could happen because of network issues.
Expected behavior:

Selenoid delete all browser containers after it exits with error.

If you need some more information, please ask.
I will try to provide as much information as needed.

@mdolinin could you please provide your config.json? Or at least what is your browser name and version?

Here is browsers.json that i use.
{
    "chrome": {
        "default": "latest",
        "versions": {
            "latest": {
                "image": "selenoid/chrome:53.0",
                "port": "4444"
            }
        }
    }
}

@mdolinin thanks. Will investigate soon.

@mdolinin please try aerokube/selenoid:latest to see if the issue is fixed.

@mdolinin any updates?

@vania-pooh I started to use latest version yesterday. Looks like working great.
But i couldn't reproduce those network issues.
Thinks that our IT's fixed them. Now trying come up with how to reproduce this locally.
May be you have some ideas?

It is hard to reproduce such issues locally, that is why we missed that
bug... Thank you for pointed to it.
2017-05-05 9:44 GMT+03:00 Mikhail Dolinin<EMAIL_ADDRESS>
@vania-pooh https://github.com/vania-pooh I started to use latest
version yesterday. Looks like working great.
But i couldn't reproduce those network issues.
Thinks that our IT's fixed them. Now trying come up with how to reproduce
this locally.
May be you have some ideas?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/aerokube/selenoid/issues/66#issuecomment-299394294,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABWNbSJCaHDuccJdHWMAfnUwhczgRg9lks5r2sVZgaJpZM4NPXkz
.

Thanks guys for this great tool. I think we can close this bug. I will reopen it, if it reproduced again.


Using Error Redirection in place of LastExitCode check

Based on better way of handling internal errors.

Looks fine.


DNM: try out v0.14.0 preview build

Awesome, looks good, nothing to do here!
Just wanted to check that https://github.com/dagger/dagger/pull/8669 hadn't done something super weird.


feat: remove wagmi/provider

Description

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #4304   👈

This comment was auto-generated by Graphite.


Playlist question

Hi there,
You already helped me a lot with other questions, thank your for that, but I have a question again. I think it's just a tiny one.
Within the VideoProvider.js I got the full movie source list:
for (var i=0, l=response.data.trailers.youtube.length; i<l; i++) {
   videos.push({src: "https://www.youtube.com/watch?v=" + response.data.trailers.youtube[i].source});
}

In the movie-detail.html page I can change the source from the first one to the second and third as well:
source:

Second trailer

Third trailer

using this function:
controller.setVideo = function setVideo(index) {
   controller.API.stop();
   controller.video.trailers[0] = controller.video.trailers[index];
   $timeout(controller.API.play.bind(controller.API), 100);
};

I see the movie source is changing, but loaded video always the same.
http://plnkr.co/edit/KdzxVscatU8lpKXrxmji
The third movie has more sources for testing (The Hunger Games)
Do you know where is the problem?
Thank you

Here's my plunk: http://plnkr.co/edit/SmcX1Itj0iQMaAbh3egW?p=preview
The problem was on several places, so here's a resume:

moviedetails.js: You were setting video.trailers[0] = video.trailers[index], so, you were overriding one values with new ones, which was bad for several reasons. Also, that result in a broken binding or something like that, because it was not triggering the $watch on the youtube plugin. I've changed to have the trailers in a variable and the current video in a new one called source.
movie-detail.html: I've changed here to set the new binding with source.
youtube.js: This one was tricky, since it is different from the latest release. On your youtube.js file there was an extra this. at this line: https://github.com/NamPNQ/bower-videogular-youtube/blob/master/youtube.js#L50

Oh,...I couldn't have figured it out by myself. It was a tricky one for me. Really thank you, I hope you didn't spend too much time on it. And sorry about the youtube.js file.
Thousand thanks, again!

Hi,
About this playlist. As I mentioned, working like a charm.
But it seems, the vg-complete doesn't activate.
If you place your latest plunk the vg-complete="controller.onCompleteVideo()" within movie-detail.html and within the moviedetail.js:
controller.onCompleteVideo = function onCompleteVideo() {
   console.log("complete");
};

...but the function doesn't run.
Do you think because of the source is different?

No, I think that you need to update the Youtube plugin and add that feature, since it seems that complete event is not supported yet.

Ok, thanks guys! I'm going to try.


Update Code of Conduct

What is changed and how it works?

Issue Number: ref #15273
What's Changed:
Update Code of Conduct

Check List
Tests

No code

Release note
None

/merge


Integration is not showing up

Hi, I've installed this integration via HACS but it's not showing up in Configuration -> Integrations. Thoughts?
I looked at the logs and there were no errors for it.
It's installed and booted:

shows up in HACS:

nothing in Configuration -> Integration

Press F5/Refresh while on the integrations page.

I swear I was doing it and nothing, it's working now after clearing cookies and refreshing. Thx!


"Round Robin" load balancing

Hi,
I see there's another question on queue broadcast, but what if I just want to "randomly" distribute requests to multiple, equal, micro services?  It seems that this is supported in the queue-world, but from what I understand in your documentation, every seneca listener will create its own queue.
I would still want the result of the operation back in my callback, so it's not "fire and forget".
Does this make sense with seneca, and/or is it already possible?
Multiple processors would make sense from a scaling and performance perspective.

I fired up the examples, using the following syntax to run a local docker Rabbit MQ:
docker run -d --hostname my-rabbit --name some-rabbit -p 8080:15672 -p 5672:5672 rabbitmq:3-management

... and found that "round robin" distribution is indeed what is used.  Pardon my ignorance.

Nothing to forgive here.
But that's correct. If you spawn multiple microservices, what you'll get is multiple consumers on the same queue. The broker (RabbitMQ, zmq, Kafka, etc.) will handle the balancing between them.


Add ability to pass extra properties to renderIndicator

It would be great to make it possible to pass extra properties like navigation details or image details to renderIndicator, so one can build more complex UI in there. For gallery apps, you can sometimes find the date of when the photo was taken, back navigation button etc. Right now there is no way to provide that.

Ah, this can be actually easily done with closure.


[wasm] Wasm code related to transaction building is deprecated and requires refactoring

ViewProtocolService on the extension side must support such requests related to transaction building as follows

rpc
Witness
rpc
WitnessAndBuild
rpc
AuthorizeAndBuild
rpc
Authorize

At the same time as the current wasm code allows you to do Authorize, Witness, Build in one function
It is necessary to make separate wasm functions for Authorize, Witness, Build that can be called from TS for proper implementation of the rpc requests given above

can we close this issue?

Yeah, I'm closing


Show a failed activity log when attaching one storage account via name & key

Storage Explorer Version: 1.14.0-dev
Build: 20200518.1
Branch: master
Platform/OS: Windows 10/ Linux Ubuntu 16.04/ macOS Mojave
Architecture: ia32/x64
Regression From: Previous release (1.13.0)
Steps to reproduce:

Select one storage account -> Right click it.
Select 'Copy Primary Key' -> Open Connect Dialog -> Select 'Attach a storage account name and key'.
Paste the Account key -> Input the Account name and Display name.
Click 'Next' -> Click 'Connect'.
Check the Activity log.

Expect Experience:
Show a successful activity log.
Actual Experience:

The attached storage account displays well.
Show a failed activity log.
"message": "Unable to determine custom connection type from connection string."

Verified this issue on build master/20200520.4. Fixed.


fs.ls return object could implement the iterator protocol

Would love it if the object returned by fs.ls() implemented the Iteration protocol:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Iteration_protocols
This would allow things like:

for (let item of items) {
  console.log('item', item)
}

let arr = Array.from(items)

A newer branch of the sdk is changing the reeturn type to an array, so closing.


groveelectromagnet: Initial implementation

This module implements support for the Grove Electromagnet.
Signed-off-by: Zion Orent<EMAIL_ADDRESS>Signed-off-by: Jon Trulson<EMAIL_ADDRESS>

Thanks again!  Merged and closing.


Events.getActor may return empty actor

Current functionality assumes that ActorContext is filled in the nearest to the event CommandContext and for all events emitted as a result of the command processing it is true. But, if an event was emitted as a result of the @Reaction to another event, the nearest CommandContext of the current EventContext will be empty, because the EventContext OriginCase is equal to EVENT_CONTEXT is such case.
Maybe it worth changing implementation in a way, that will allow obtaining actor from the nested command contexts if needed, e.g. smth like that may work:
public static UserId getActor(EventContext context) {
        EventContext nestedContext = getNestedContext(context);
        CommandContext commandContext = nestedContext.getCommandContext();
        return commandContext.getActorContext()
                             .getActor();
    }

private static EventContext getNestedContext(EventContext context) {
        EventContext currentContext = context;
        while (currentContext.getOriginCase() == EventContext.OriginCase.EVENT_CONTEXT) {
            currentContext = currentContext.getEventContext();
        }
        return currentContext;
    }

We have to use Events.findCommandContext() in Events.getActor() implementation. And tweak getActor documentation accordingly — as currently it tells nothing about the @React use-case.

Also, we should check if there is a case where an empty actor is returned. I would return Optional.empty() if such a scenario is possible.

Resolved in #922.


update to v0.330

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Fix the issue of getting event type

Description
Fix the issue of getting event type in Publish and Subscribe methods to get the event type from the passed object itself instead of using the passed generic type.
Motivation
Build new abstract layers on EiffelEvents SDK and enable extensibility.
Exemplification
Use:
eiffelEvent.GetType().Name
Instead of:
typeof(T).Name
Benefits
This will help the SDK to get the real event type instead of the generic type passed to the Publish method that can be abstract also.
Possible Drawbacks
NA

Closed here #88


Feature suggestion: strict mode on Translations

Hi,
I have an simple and useful feature suggestion: a strict mode on translations.
When developing a large multi-language system, sometimes we forget to translate some strings, and the software may go to production with something like client.name_label on the UI.
My suggestion is to add a option on app.php configuration file to enable a strict mode on translations, where when some translation key is missing, it throw an exception.
Maybe this can go to 5.2 or either to 5.1 since it's not a breaking change, can be disabled by default.
I appreciate if we can think about it! :+1:

Will go to either 5.2 or 5.3, but not to 5.1.

Please review our contribution docs. We don't process proposals here.


Cannot refresh while debugging

Issue
I cannot refresh the browser while debugging my razor page.
Steps to Reproduce

Start debugging with the browser.
Modify the cshtml for the current page.
Click the BrowserLink refresh button.

I am shown a dialog stating:

Do you want to stop debugging?

If I click OK, then the following happens:

Debug session terminates.
VS builds my project.
The browser refreshes.

Workaround
Use the refresh button on the browser.
Environment Information
Microsoft Visual Studio Professional 2019
Version 16.6.5
I was not having this problem with version 16.6.4.

I have this same issue in Microsoft Visual Studio Professional 2022 (64-bit)
Version 17.2.5


[env] Don't run next assess when failNow is called

What type of PR is this?
/kind bug
What this PR does / why we need it:
When writing tests, this allows the writer to control whether the test should continue running the next assess when a failure occurs. Basically, they can chose between using t.FailNow() or t.Fail() depending on if the failure should stop the test or not. It works independently from the -fail-fast flag specific to the framework and allows a more granular control on test failures.
Which issue(s) this PR fixes:
Fixes #386
Special notes for your reviewer:
I tried to add a new test for this case but couldn't find a way to make it work. Since failNow is called, the test fails and I couldn't find a way to expect a test to fail in go :sweat_smile:. Happy to add one if someone has an idea on how to do it though.
Code to reproduce
You can use the following code to illustrate what I'm trying to solve
package envfuncs_test

import (
	"context"
	"os"
	"testing"

	"sigs.k8s.io/e2e-framework/pkg/env"
	"sigs.k8s.io/e2e-framework/pkg/envconf"
	"sigs.k8s.io/e2e-framework/pkg/features"
)

var testenv env.Environment

func TestMain(m *testing.M) {
	testenv = env.New()
	os.Exit(testenv.Run(m))
}

func TestFailNow(t *testing.T) {
	feat1 := features.New("fail now").
		Assess("Assess 1", func(ctx context.Context, t *testing.T, cfg *envconf.Config) context.Context {
			t.Log("Assess 1 (should be printed)")
			t.FailNow()
			return ctx
		}).
		Assess("Assess 2", func(ctx context.Context, t *testing.T, cfg *envconf.Config) context.Context {
			t.Log("Assess 2 (should NOT be printed)")
			return ctx
		}).
		Feature()

	feat2 := features.New("succeed").
		Assess("Assess 1", func(ctx context.Context, t *testing.T, cfg *envconf.Config) context.Context {
			t.Log("Assess 1 (should be printed)")
			return ctx
		}).
		Assess("Assess 2", func(ctx context.Context, t *testing.T, cfg *envconf.Config) context.Context {
			t.Log("Assess 2 (should be printed)")
			return ctx
		}).
		Feature()

	testenv.Test(t, feat1, feat2)
}

func TestFail(t *testing.T) {
	feat := features.New("fail (not now)").
		Assess("Assess 1", func(ctx context.Context, t *testing.T, cfg *envconf.Config) context.Context {
			t.Log("Assess 1 (should be printed)")
			t.Fail()
			return ctx
		}).
		Assess("Assess 2", func(ctx context.Context, t *testing.T, cfg *envconf.Config) context.Context {
			t.Log("Assess 2 (should be printed)")
			return ctx
		}).

Stopped running the following assess if FailNow() was called during a test

Additional documentation e.g., Usage docs, etc.:

cc @harshanarayana

/retest


Add my files for odin sign up form

Odin sign up form

Odin sign up form

Odin sign up form

yo bruh,how bout updating this to be descriptibe enough

linters are failing


scoop install qtemu error, url is not valid

Bug Report
Package Name: [qtemu]
Current Behaviour
scoop install extras/qtemu
Installing 'qtemu' (2.1) [64bit] from extras bucket
The remote server returned an error: (400) Bad Request.
URL https://www.carlavilla.es/qtemu/qtemu_portable_x86_64.zip is not valid

same problem


Allow Package Scanner to scan larger files using the VirusTotal v3 API

Checklist

[X] I have verified this is the correct repository for opening this issue.
[X] I have verified no other issues exist related to my problem.
[X] I have verified this is not an issue for a specific package.
[X] I have verified this issue is not security related.

What You Are Seeing?
Packages with files larger than 200MB are not uploaded to VirusTotal and get the message "File is too big to scan - the limit is 200MB and this file is 255.74MB"
What is Expected?
Virus scanner should allow up to 650MB as per https://github.com/VirusTotal/vt-cli/issues/33
How Did You Get This To Happen?
Submitted package that referenced files that are larger than 200MB
┆Issue is synchronized with this Gitlab issue by Unito

I ran across this issue as well with Laravel Herd, as it is 229MB, just 29MB over the 200MB limit.
This limitation, however, only applies to the API. If you guys would take advantage of the official VirusTotal CLI, you can still upload files beyond 200 MB.
Command example:
vt scan file .\Herd-1.8.0-setup.exe
``

It spits out the checksum:
![image](https://github.com/chocolatey/home/assets/49938263/523de4f7-dd91-416b-b229-4cf62fb184ab)

And then, of course you can find the file using the checksum through the API or web:
https://www.virustotal.com/gui/file/b3fde098ff5e69685950495cd1c19a9b7507dea182ae2f87e1a3d7740f3cba5e


Operational-Best-Practices-for-NZISM.yaml is too long to be used

AWS Config seems to have a limit of 51000 chars for Compliance Pack files.
The NZISM file is ~51461 char, so can't be used directly.
If these files are driving the "Sample template" options in the management console, its also preventing NZISM being chosen from there as well, it will fail with the same error about the file being too long.
My workaround thus far has been to remove some rules, for services that we know we don't use.

We've trimmed some comment characters from the yaml to bring it under the 51200 limit. Console deployments will complete now. Please try again.


Updates to Virtual environments and split out "Other"

Delete mamba and merge to conda.
Add poetry as amber.

Move out other to a TL page and think of a cooler title than "other".

Move psutil to "Benchmarking"

Closed v


added sitemap.xml

Pull Request
Title
added sitemap.xml
Description
added sitemap.xml for crawlers
Checklist

[x] I have tested the changes locally
[x] Documentation has been updated (if necessary)
[x] Changes are backward-compatible

Screenshots (if applicable)

@mastansayyad please review this PR

@RadhikaMalpani1702 please let me know the reason...


build: use prettier instead of tslint for linting (#552)

close #552
This PR replaces duplicated tslint with prettier. The settings is same as other Vuex ORM family libs (such as Axios and Soft Delete).
Type of PR:

[ ] Bugfix
[ ] Feature
[x] Code style update
[ ] Refactor
[x] Build-related changes
[ ] Documentation
[ ] Other, please describe:

Breaking changes:

[x] No
[ ] Yes

Details
@cuebit Could you take a look at the settings and such? I don't think there should be problem while test is passing, though lot of file changed so just want double check this one 🙇

Yea I’ll cross reference with a branch I created but never pushed to upstream haha.


How to store the result in boomerang?

I am using boomerang.js to get the page loading time. The result displays on same page only. I want store that data in my salesforce. How to do this? How can i store the data using beacon url? How can i retrieve the stored data?
Here is my code
<apex:page controller="jstrackerexample">

Boomerang Exp

I do not see any code attached.  I'm also not sure about how salesforce works, but in general, to store the beacon data you will need to write a piece of server side code using any server side language like php, java, node.js, ruby, etc.  This code would listen for HTTP requests and when a beacon comes in, it will parse it and write the results to a database.  You can then use SQL to query it.
Other people have used Google Analytics or Piwik to do the data storage.  There are also people who use a couchdb url as the beacon, so the data goes directly into couchdb.  Or you can use some of the freely available servers like https://github.com/nature/boomcatch or a commercial solution like the one from SOASTA's mPulse.

Other people have used Google Analytics or Piwik to do the data storage.

Just lurking here...do you know of any documentation for using GA or Piwik for such storage? Thanks!

I'd posted this to quora a while ago: http://www.quora.com/How-good-is-Yahoos-Boomerang-code-for-measuring-page-performance-Is-it-worth-the-integration-effort/answer/Philip-Tellis
If you don't have a quora account, these links will be useful:
https://gist.github.com/tmahesh/833066
http://benjamin-balet.info/developpement/performances/performance-ressentie-avec-piwik-et-boomerang/
http://www.slideshare.net/justindorfman/a-web-perf-dashboard-up-running-in-90-minutes-presentation


Add MapLibre goodies

From ATIP: more basemaps, the geocoder, the street view widget, etc

Problems with switching basemaps:

[ ] Extra images we load get clobbered
[ ] The z-ordering needs to vary with the basemap. Just restoring the old layer config doesn't work


Feature Request - add way to ship the notify to elasticsearch

Already WIP: https://github.com/capitalone/cloud-custodian/pull/1612
The purpose of this feature request is to collect the data from the notify action into desired endpoints whether it be elasticsearch or logstash or any future endpoint. This will enable users that want to create a dashboard of resources surveyed and actions taken.
@kapilt, right now it is an amalgamation of the indexer and mailer. The pattern for use would be to have notify send to SNS for fanout to the mailer SQS and this dispatcher SQS.
I didn't think it would be a good idea to burden either existing tool with a refactor that generalizes the purpose of either one. My thoughts are that eventually people may want to migrate their mailer implementations to use this tool rather than add elasticsearch indexing into the mailer tool.

why not just add an sqs subscriber to the extant indexer instead of duplicating it?

the mailer already has an sqs subscriber, dispatching in this context is notifying to an sns topic and subscribing an sqs queue consumer for the functionality of interest.

Whatever we end up naming it, I think it would be better to create a new tool for this purpose and eventually migrate the mailer functionality into it. I think that putting this feature request into the mailer would be a schema breaking change for existing users and would be confusing under the "mailer" namespace

this doesn't have anything to do with the mailer or a new tool though, it has to do with not trying to multiplex on sqs, and instead setup sns as a pub/sub for dispatching. then individual tools that  want to consume just hook to their own sqs queue. thats alot better than trying to reimplement broadcast logic and all of the tools in yet another tool. ie. think microservices on the downstream not monoliths and use the platform features instead of reimplementing.


Issues in linking virtual machine to git hub

Hi
Could you please help with the steps to push the data from VM to Github.I get permission denied error and could not commit .Please help

No. Check Stackoverflow as I am busy in other things


Add realtime.kernel option to enable PREEMPT_RT patches

Description of changes
NVIDIA seems to have some level of support for running a kernel with the PREEMPT_RT patchset. They include the patches under rt-patches/ and have instructions to enable the realtime kernel here:
https://docs.nvidia.com/jetson/archives/r35.1/DeveloperGuide/text/SD/Kernel/KernelCustomization.html#to-build-the-real-time-kernel
This adds a hardware.nvidia-jetpack.kernel.realtime option which, if set to true, enables the PREEMPT_RT patches and kernel options.
Testing
Only tested this builds. Will test on device soon.

Tested working on a Xavier AGX. Some basic tests using cycletest show decreases in maximum latency from ~10ms down to ~50μs.

Going to merge this. Anyone who wants to use this should just be aware that (for the forseeable future) it's going to be a configuration that is less well tested than the standard configuration.


PCB export issue

Hello!
I have created this PCB in KiCad 7.0:
problematic.zip

When exported as gerber, things looks as they are intended:

However, when exported using the Aisler plugin, or if the kicad_pcb fileis imported directly on Aisler.net, I get following result.

Hi! I guess this issue is more KiCad related. If I open the generated gerber files using the Ucamco reference viewer I get a list of issues. These issues also cause our CAM to misbehave.

But as stated in the issue description, this does not occur if I generate the gerber with KiCad without the plugin.

These gerbers are generated directly from KiCad
grb.zip

You are right on this! However I have no clue what causes KiCad to plot these corrupted Gerbers as the plugin actually uses the same interface as the UI. I'll check with the KiCad developers!

Seems there is a problem using push to aisler using kicad...it connecting to microsoft edge without loading the files...


Possible Typo - "Migration policy customizations"

The feature named "Migration policy customizations" might not make sense. The word "migration" should likely be "mitigation".
Please consider one of the following:

"Mitigation policy customization."
or
"Customization of Mitigation Policies"

Please review when you get a chance.
Thank you,

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

ID: 6f38b1bd-2ab0-71f1-a6e9-925b7af3389d
Version Independent ID: 9528f5cf-16e1-0550-dc3e-edc135c82167
Content: Azure DDoS Protection Standard Overview
Content Source: articles/virtual-network/ddos-protection-overview.md
Service: virtual-network
GitHub Login: @KumudD
Microsoft Alias: kumud

@ALLWORLD7 , Thank you for your feedback. We will investigate and get back to you shortly.


generate trapeze

Hello,
With imgaug, I would like to transform the image to a trapeze. How I can do this ?
+--+      ++
|  | to  / \
+--+    +---+

Thanks

Hm, I don't think this is currently possible. You could use two shearing commands via Affine, i.e. shearing once to the right until only a triangle is remaining, then once back to the center. That should create a triangular output, but the content would be the bottom left triangle of the image, not the whole image's content squeezed to a triangle.
You could probably hack PerspectiveTransform to alter its sampling process. Currently it places randomly four points on the image and performs a four-point perspective transform towards these points. The sampling could be changed to form a triangle, but the perspective transformation would still output a rectangular image.

Thanks.
May be, I will add a new perspectivetransform ;-)


Invalid pallet session/pallet staking flow: purge_keys can decrement consumer ref count on wrong controller.

if I have 3 account: stash, controller_1 and controller_2.
I bound stash to controller_1,
then call set_keys from this controller_1, controller_1 has one consumer refcount increase.
then call unbond, and later call withdraw_unbonded.
then bond stash to controller_2,
than call purge_keys from this controller_2, the consumer refcount decrement silently fails and logs an error.
While it is not possible to do an attack with this. I think it is still an error to have a dec_consumer which fails silently. And an account with a consumer refcount which is not removable.
Fixing this issue can also fix ppl having issue with unbonded stash and controller still being used by pallet_session https://github.com/paritytech/polkadot/issues/2654, but I think it is more a UI issue.
Maybe a solution could be to have a consumer refcount increase on the stash instead of the controller in pallet_session.

the invalid refcount is still sensible to me, it is not harmful, but it shouldn't be possible to trigger this


make perl6- prefix optional

Hi!
When create directory to hold project data, can we make "perl6-" prefix optional? Or maybe even create directory without it. What do you think?

I spend quite some time thinking about that matter. My observation is that quite a few ppl use some sort of prefix for projects perl6- and p6- are common judgeing by the ecosystem. It allows for having Perl5 and Perl 6 projects in the same directory without causing confusion. Github got their topic system now but it doesn't translate to URLs. So from the standpoint of a casual user it's easiert to spot what this is about. And it's not just the directory name because the github repo name is derived from it too. There may be good reason to implement the same module for Ruby and Perl 6 (I know, wishful thinking :).
I settled for the longer form mostly for URL/github repo name clarity. And if you autocomplete in a shell it doesn't make any difference.
So no I don't like provide even the option to remove the prefix. If you really don't linke it you can always rename the directory with a shellscript wrapper or just use a local fork so tweak it (maybe not the best option right now but it wont change all that much in the future).
If you have good arguments for why the prefix is a bad idea please share.

Hi, more or less I just wanted to hear your arguments for naming conventions. It is hard to anticipate if it will be good or bad choice . But I see your points. And changing directory, alter .git/config afterwords manually is not a big deal. Probably we can close this now.

And yeah, as maijor part of my Perl6 modules are Sparrowdo ones, my naming convention is "Sparrowdo-*" )))

To be honest I only use some kind of prefix if there is going to be a clash with some existing repository I might have, but hey.

The prefix can now be set in the meta6.cfg with create.prefix=your-prefix-here-.

Thanks!


[iOS Google Maps] Fix update camera always animated

Does any other open PR do the same thing?
No
What issue is this PR fixing?
iOS GoogleMap update camera always animated.
fitToElements, fitToSuppliedMarkers and fitToCoordinates has animated parameter but real implantation ignore this parameter, always call animateWithCameraUpdate.
How did you test this PR?
run on iOS simulator (iOS 11.4)

pull request merged in master
thanks @a821246710


Opening a dialog hangs when GTK window is open

I'm trying to use this crate with tao; GTK fork of winit. Since GTK is already in dependencies, using gtk3 feature of this crate can avoid many dependencies (e.g. async runtime). So I added it in my Cargo.toml:
rfd = { version = "0.14.1", default-features = false, features = ["gtk3"] }
tao = "0.28.0"

However, when calling rfd::FileDialog::pick_files after opening a GTK window, the method call hung forever and eventually OS suggested to kill the application process.
Here is a reproduction. This code opens one window. Once the window is initialized, it tries to open a file dialog with FileDialog::piick_files. Expected behavior is that a file dialog opens successfully and the window can continue to work after closing the dialog. The actual behavior was that it hung forever.
use tao::{
    event::{Event, WindowEvent, StartCause},
    event_loop::{ControlFlow, EventLoop},
    window::WindowBuilder,
};

fn open_dialog() {
    let res = rfd::FileDialog::new()
        .add_filter("text", &["txt", "rs"])
        .add_filter("rust", &["rs", "toml"])
        .set_directory(".")
        .pick_files();
    println!("The user choose: {:#?}", res);
}

fn main() {
    let event_loop = EventLoop::new();

    let mut window = Some(
        WindowBuilder::new()
            .with_title("A fantastic window!")
            .with_inner_size(tao::dpi::LogicalSize::new(300.0, 300.0))
            .with_min_inner_size(tao::dpi::LogicalSize::new(200.0, 200.0))
            .build(&event_loop)
            .unwrap(),
    );

    event_loop.run(move |event, _, control_flow| {
        *control_flow = ControlFlow::Wait;

        match event {
        Event::WindowEvent {
            event: WindowEvent::CloseRequested,
            window_id: _,
            ..
        } => {
            // drop the window to fire the `Destroyed` event
            window = None;
        }
        Event::WindowEvent {
            event: WindowEvent::Destroyed,
            window_id: _,
            ..
        } => {
            *control_flow = ControlFlow::Exit;
        }
        Event::MainEventsCleared => {
            if let Some(w) = &window {
                w.request_redraw();
            }
        }
        Event::NewEvents(StartCause::Init) => {
            println!("opening dialog");
            open_dialog();
        }
        _ => (),
        }
    });
}

I'm not familiar with GTK's architecture, but it seems that gtk3 feature of this crate does not work when GTK main thread (or main loop) is already initialized and used by someone else. When using xdg-portal feature, this issue doesn't occur.
Environment:

Rust: 1.75
OS: Ubuntu 22.04
rfd: 0.14.1

This is my fault and also GTK's fault--see https://github.com/PolyMeilex/rfd/pull/152.
GTK can only be used from the thread that it was initialized on. AFAIK, there's no way to get this thread and tell it to do things if we didn't initialize it ourselves. There's a quote from an old version of the GTK documentation that doesn't seem to be online anymore, but you can still find references to this snippet floating around:

GTK+, however, is not thread safe. You should only use GTK+ and GDK from the thread gtk_init() and gtk_main() were called on. This is usually referred to as the "main thread".

I'm not sure how this can be fixed without some sort of cross-crate effort to share the GTK+ main thread.


extension frictionless[wkt] not found

The processing of the wkt format moved to an extension. Somehow finding this extension does not work. Although frictionless[wkt] is installed I get this error message:
[error] Please install "frictionless[wkt]"

Here is a minimal example to reproduce the problem:
docker run -it --rm python:3-alpine /bin/sh
pip install frictionless
pip install frictionless[wkt]
touch test.csv
echo '{"name":"test","path":"test.csv","schema":{"fields":[{"name":"geometry","type":"string","format":"wkt"}]}}' > test.json
frictionless extract test.json

Thanks @jze, unfortunately grako (that gives us WKT support) doesn't work in Python3.10+. I've added better error message


Allow dart to set an environment variable in the parent process

I'm creating a package 'dshell' that is designed to make it easy to write cli scripts. Essentially the aim is that dshell should allow you to do everything that bash does, but using dart.
https://pub.dev/packages/dshell
As an example:
I'm currently trying to write the install script for dshell and need to reconfigure the current shell's 'PATH' so that after the install completes, the user can immediately run dshell commands without requiring a reboot or restart the terminal.
To do this, dshell needs to update the parent process's PATH environment variable.
Additionally, user of dshell are also likely to want to modify the parent process's environment variables for all sorts of reasons as bash users can currently do using 'export XXX=yyyy'.
Looking through the dart:io api I can't find any method that meets the above requirements.
I'm guessing the obvious path is to make Platform.environment writable?

as bash users can currently do using 'export XXX=yyyy'.

I'm pretty sure no shell (not even bash) allows you to modify the parent process's environment (nor should they!)
Plus, that's not even a shell limitation-- it's intentional on the part of the operating system. A program with this ability would, for instance, be able to overwrite the parent's SHELL variable and make it trivial to obtain root access

It's been a long time, but I'm pretty sure you can do it from C.
On Sun, 12 Jan 2020, 12:11 pm Alex Brooke<EMAIL_ADDRESS>wrote:

as bash users can currently do using 'export XXX=yyyy'.
I'm pretty sure no shell (not even bash) allows you to modify the parent
process's environment (nor should they!)
Plus, that's not even a shell limitation-- it's intentional on the part of
the operating system. A program with this ability would, for instance, be
able to overwrite the parent's SHELL variable and make it trivial to
obtain root access
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/sdk/issues/40069?email_source=notifications&email_token=AAG32OBRPKQIIR4NK2XUV6LQ5JU4ZA5CNFSM4KFUHIXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIWO5SQ#issuecomment-573370058,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAG32OE22WGROG27PFC4HS3Q5JU4ZANCNFSM4KFUHIXA
.

So looks like I'm wrong.
There is a way to do it on windows but lots like it's not completely
reliable.
https://www.perlmonks.org/?node_id=658253
As such I will close this issue.
On Sun, 12 Jan 2020, 12:11 pm Alex Brooke<EMAIL_ADDRESS>wrote:

as bash users can currently do using 'export XXX=yyyy'.
I'm pretty sure no shell (not even bash) allows you to modify the parent
process's environment (nor should they!)
Plus, that's not even a shell limitation-- it's intentional on the part of
the operating system. A program with this ability would, for instance, be
able to overwrite the parent's SHELL variable and make it trivial to
obtain root access
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/sdk/issues/40069?email_source=notifications&email_token=AAG32OBRPKQIIR4NK2XUV6LQ5JU4ZA5CNFSM4KFUHIXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIWO5SQ#issuecomment-573370058,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAG32OE22WGROG27PFC4HS3Q5JU4ZANCNFSM4KFUHIXA
.


Work around potential race in PipeWriter

Make sure we always await the last flush task before calling FlushAsync
again instead of preemptively calling FlushAsync and checking to see
if the ValueTask is incomplete before bothering to acquire the _flushLock
This now acquires the _flushLock fore every call to Response.Body.WriteAsync
whereas this only happened for truly async writes before.
I don't think this is a big concern since this should normally be uncontested,
and DefaultPipeWriter.FlushAsync/GetResult already acquire a lock.

Addresses #8843
@benaadams
This really just #10126, but GitHub wouldn't let me reopen the PR.

Nothing changed yet. I'm still going to benchmark/profile this.

Our TE benchmark servers were unavailable this afternoon, but I did get some microbenchmark results from Http1WritingBenchmark. I made a new Http1LargeWritingBenchmark for larger unawaited writes as well. I fixed the old benchmark (which was hanging previously) and pushed the new new benchmark as part of this PR.
I'm not sure it's worth keeping the new benchmark. I can remove it before merging the PR.
I think the big takeaway is the perf impact is negligible and the allocations are unchanged.
master
Http1WritingBenchmark
 Method | WithHeaders | Chunked | OnStarting |     Mean |    Error |   StdDev |        Op/s | Allocated |

----------- |------------ |-------- |----------- |---------:|---------:|---------:|------------:|----------:|
WriteAsync |       False |   False |      Async | 548.2 ns | 10.96 ns | 21.64 ns | 1,824,087.4 |       0 B |
WriteAsync |       False |   False |       None | 554.4 ns | 10.49 ns | 18.65 ns | 1,803,866.9 |       0 B |
WriteAsync |       False |   False |       Sync | 555.6 ns | 11.16 ns | 20.40 ns | 1,799,721.0 |       0 B |
WriteAsync |       False |    True |      Async | 561.4 ns | 11.16 ns | 23.55 ns | 1,781,149.0 |       0 B |
WriteAsync |       False |    True |       None | 554.6 ns | 11.07 ns | 22.11 ns | 1,803,227.6 |       0 B |
WriteAsync |       False |    True |       Sync | 564.4 ns | 11.30 ns | 21.50 ns | 1,771,860.8 |       0 B |
WriteAsync |        True |   False |      Async | 792.8 ns | 15.70 ns | 27.91 ns | 1,261,393.3 |       0 B |
WriteAsync |        True |   False |       None | 683.5 ns | 13.27 ns | 19.45 ns | 1,463,063.0 |       0 B |
WriteAsync |        True |   False |       Sync | 706.8 ns | 13.98 ns | 24.84 ns | 1,414,790.9 |       0 B |
WriteAsync |        True |    True |      Async | 811.5 ns | 14.10 ns | 18.82 ns | 1,232,335.7 |       0 B |
WriteAsync |        True |    True |       None | 684.4 ns | 13.65 ns | 20.84 ns | 1,461,183.1 |       0 B |
WriteAsync |        True |    True |       Sync | 729.7 ns | 14.40 ns | 21.55 ns | 1,370,412.3 |       0 B |
Http1LargeWritingBenchmark
                  Method |     Mean |     Error |    StdDev |  Op/s |  Gen 0 | Allocated |

---------------------------- |---------:|----------:|----------:|------:|-------:|----------:|
WriteAsync | 2.360 ms | 0.0471 ms | 0.0812 ms | 423.8 | 3.9063 |  198.3 KB |
WriteSegmentsUnawaitedAsync | 2.109 ms | 0.0420 ms | 0.1005 ms | 474.2 |      - |   5.48 KB |
This PR
Http1WritingBenchmark
 Method | WithHeaders | Chunked | OnStarting |     Mean |    Error |   StdDev |        Op/s | Allocated |

----------- |------------ |-------- |----------- |---------:|---------:|---------:|------------:|----------:|
WriteAsync |       False |   False |      Async | 584.6 ns | 11.63 ns | 29.38 ns | 1,710,563.9 |       0 B |
WriteAsync |       False |   False |       None | 573.2 ns | 11.47 ns | 20.08 ns | 1,744,692.8 |       0 B |
WriteAsync |       False |   False |       Sync | 562.2 ns | 11.29 ns | 22.01 ns | 1,778,846.3 |       0 B |
WriteAsync |       False |    True |      Async | 569.3 ns | 10.78 ns | 12.41 ns | 1,756,619.1 |       0 B |
WriteAsync |       False |    True |       None | 561.2 ns | 11.16 ns | 22.80 ns | 1,782,038.0 |       0 B |
WriteAsync |       False |    True |       Sync | 565.1 ns | 11.52 ns | 23.28 ns | 1,769,587.7 |       0 B |
WriteAsync |        True |   False |      Async | 809.5 ns | 15.01 ns | 14.74 ns | 1,235,369.9 |       0 B |
WriteAsync |        True |   False |       None | 701.1 ns | 13.79 ns | 23.41 ns | 1,426,337.7 |       0 B |
WriteAsync |        True |   False |       Sync | 735.4 ns | 14.55 ns | 20.87 ns | 1,359,716.2 |       0 B |
WriteAsync |        True |    True |      Async | 809.5 ns | 17.90 ns | 15.86 ns | 1,235,302.2 |       0 B |
WriteAsync |        True |    True |       None | 695.8 ns | 13.73 ns | 22.56 ns | 1,437,196.3 |       0 B |
WriteAsync |        True |    True |       Sync | 719.4 ns | 14.38 ns | 24.02 ns | 1,390,017.6 |       0 B |
Http1LargeWritingBenchmark
                  Method |     Mean |     Error |    StdDev |  Op/s | Allocated |

---------------------------- |---------:|----------:|----------:|------:|----------:|
WriteAsync | 2.340 ms | 0.0463 ms | 0.0823 ms | 427.3 | 198.24 KB |
WriteSegmentsUnawaitedAsync | 2.190 ms | 0.0436 ms | 0.0691 ms | 456.6 |      5 KB |
Summary for all runs
BenchmarkDotNet=v0.10.13, OS=Windows 10.0.17763
Intel Xeon W-2133 CPU 3.60GHz, 1 CPU, 12 logical cores and 6 physical cores
.NET Core SDK=3.0.100-preview5-011568
  [Host]     : .NET Core 3.0.0-preview6-27708-04 (CoreCLR <IP_ADDRESS>672, CoreFX 4.700.19.25613), 64bit RyuJIT
  Job-NXGMUT : .NET Core 3.0.0-preview6-27708-04 (CoreCLR <IP_ADDRESS>672, CoreFX 4.700.19.25613), 64bit RyuJIT

@aspnet-hello triage https://dev.azure.com/dnceng/public/_build/results?buildId=186220

This comment was made automatically. If there is a problem contact<EMAIL_ADDRESS>I've triaged the above build. I've created/commented on the following issue(s)
https://github.com/aspnet/AspNetCore-Internal/issues/2407
https://github.com/aspnet/AspNetCore-Internal/issues/2381

So it seems like this might cause ~1% RPS regression in the TechEmpower plaintext benchmark. Should I merge and see what our benchmark infrastructure has to say? We can always revert. @anurse @davidfowl
master
RequestsPerSecond:           2,119,125
Max CPU (%):                 92
WorkingSet (MB):             61
Avg. Latency (ms):           2.03
Startup (ms):                283
First Request (ms):          152.37
Latency (ms):                0.6
Total Requests:              31,963,034
Duration: (ms)               15,080
Socket Errors:               0
Bad Responses:               0
SDK:                         3.0.100-preview5-011568
Runtime:                     3.0.0-preview6-27711-04
ASP.NET Core:                3.0.0-preview6-19253-01

RequestsPerSecond:           2,115,538
Max CPU (%):                 97
WorkingSet (MB):             61
Avg. Latency (ms):           1.74
Startup (ms):                285
First Request (ms):          154.07
Latency (ms):                0.59
Total Requests:              31,918,276
Duration: (ms)               15,090
Socket Errors:               0
Bad Responses:               0
SDK:                         3.0.100-preview5-011568
Runtime:                     3.0.0-preview6-27711-04
ASP.NET Core:                3.0.0-preview6-19253-01

RequestsPerSecond:           2,099,329
Max CPU (%):                 98
WorkingSet (MB):             61
Avg. Latency (ms):           1.73
Startup (ms):                284
First Request (ms):          155.22
Latency (ms):                0.56
Total Requests:              31,683,698
Duration: (ms)               15,093
Socket Errors:               0
Bad Responses:               0
SDK:                         3.0.100-preview5-011568
Runtime:                     3.0.0-preview6-27711-04
ASP.NET Core:                3.0.0-preview6-19253-01

This PR (halter73/8843)
RequestsPerSecond:           2,094,538
Max CPU (%):                 97
WorkingSet (MB):             62
Avg. Latency (ms):           1.65
Startup (ms):                287
First Request (ms):          153.8
Latency (ms):                0.56
Total Requests:              31,603,413
Duration: (ms)               15,090
Socket Errors:               0
Bad Responses:               0
SDK:                         3.0.100-preview5-011568
Runtime:                     3.0.0-preview6-27711-04
ASP.NET Core:                3.0.0-preview6-19253-01

RequestsPerSecond:           2,044,282
Max CPU (%):                 94
WorkingSet (MB):             62
Avg. Latency (ms):           1.94
Startup (ms):                284
First Request (ms):          154.75
Latency (ms):                0.54
Total Requests:              30,849,914
Duration: (ms)               15,093
Socket Errors:               0
Bad Responses:               0
SDK:                         3.0.100-preview5-011568
Runtime:                     3.0.0-preview6-27711-04
ASP.NET Core:                3.0.0-preview6-19253-01

RequestsPerSecond:           2,094,747
Max CPU (%):                 95
WorkingSet (MB):             62
Avg. Latency (ms):           1.83
Startup (ms):                276
First Request (ms):          155.8
Latency (ms):                0.57
Total Requests:              31,574,261
Duration: (ms)               15,073
Socket Errors:               0
Bad Responses:               0
SDK:                         3.0.100-preview5-011568
Runtime:                     3.0.0-preview6-27711-04
ASP.NET Core:                3.0.0-preview6-19253-01

> dotnet run --server http://<redacted>:5001 --client http://<redacted>:5002 -n Plaintext -j https://raw.githubusercontent.com/aspnet/Benchmarks/master/src/Benchmarks/benchmarks.plaintext.json --warmup 0 --duration 15 -x 2 -i 7 --outputFile "..\..\AspNetCore\src\Servers\Kestrel\Core\src\bin\Release\netcoreapp3.0\Microsoft.AspNetCore.Server.Kestrel.Core.dll" --aspNetCoreVersion "Latest" --runtimeVersion "Latest" --self-contained

This is on our physical Linux hardware. I was running the benchmark on Windows this afternoon and I was getting ~2.1 million RPS but when I ran more benchmarks tonight I was only getting ~630K RPS with the exact same configuration, so I switched to Linux. @sebastienros

1% seems within a margin of error to me, but I don't have enough experience in this area to know for sure. Personally I'm OK with a merge and watch approach, as long as someone is on point to actually verify that the regression is just noise (or acceptable).

This is a correctness fix, right? The alternative here is a race that causes incorrect behavior?

The alternative is a less locky solution (something that tries to avoid the lock until we need to queue tasks).
cc @DamianEdwards @sebastienros

The alternative is a less locky solution (something that tries to avoid the lock until we need to queue tasks).

Exactly. This is what we had before. We thought is was correct, but #8843 suggested otherwise. Hopefully the difference I saw was just statistical noise, but I don't think so.

@sebastienros Do you know what was wrong with the Windows server last night? I haven't tried it since?

This is on our physical Linux hardware. I was running the benchmark on Windows this afternoon and I was getting ~2.1 million RPS but when I ran more benchmarks tonight I was only getting ~630K RPS with the exact same configuration, so I switched to Linux. @sebastienros


Fixed collection sorting on document and image chooser

Fixes #5887

Now handled by the nested collections feature coming in 2.11 (#6141).


Added the comments and added a few lines of code to help with debug. …

…Also added a few lines of code to crank the indicator when user dismissed or reentered the app.

Be sure to add a project item for this pull request since it is done to complete one of the to-dos in the v0.2 agenda project


_ Patch Compile Tests _

+1 :green_heart:
mvninstall
5m  8s
the patch passed

+1 :green_heart:
compile
0m 31s
the patch passed

+1 :green_heart:
javac
0m 31s
the patch passed

+1 :green_heart:
shadedjars
7m  1s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 28s
hbase-client in the patch failed.

_ Other Tests _

-1 :x:
unit
0m 59s
hbase-client in the patch failed.

29m 29s

Subsystem
Report/Notes

Docker
Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1822

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux d5b68c644245 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 716702a349

Default Java
2020-01-14

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-client.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-client.txt

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/1/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-client.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/1/testReport/

Max. process+thread count
200 (vs. ulimit of 12500)

modules
C: hbase-client U: hbase-client

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 35s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+1 :green_heart:
mvninstall
4m 54s
master passed

+1 :green_heart:
compile
0m 30s
master passed

+1 :green_heart:
shadedjars
8m 10s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 33s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 31s
the patch passed

+1 :green_heart:
compile
0m 30s
the patch passed

+1 :green_heart:
javac
0m 30s
the patch passed

+1 :green_heart:
shadedjars
7m  2s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 27s
the patch passed

_ Other Tests _

-1 :x:
unit
0m 54s
hbase-client in the patch failed.

29m 18s

Subsystem
Report/Notes

Docker
Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1822

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 387d68260fa4 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 716702a349

Default Java
1.8.0_232

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/1/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-client.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/1/testReport/

Max. process+thread count
337 (vs. ulimit of 12500)

modules
C: hbase-client U: hbase-client

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
2m 27s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
5m  0s
master passed

+1 :green_heart:
checkstyle
0m 35s
master passed

+1 :green_heart:
spotbugs
1m 19s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 38s
the patch passed

-0 :warning:
checkstyle
0m 32s
hbase-client: The patch generated 1 new + 14 unchanged - 0 fixed = 15 total (was 14)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
14m 19s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1.

+1 :green_heart:
spotbugs
1m 48s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 15s
The patch does not generate ASF License warnings.

39m  6s

Subsystem
Report/Notes

Docker
Client=19.03.9 Server=19.03.9 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1822

Optional Tests
dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle

uname
Linux 134a3f9e412a 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 716702a349

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-client.txt

Max. process+thread count
85 (vs. ulimit of 12500)

modules
C: hbase-client U: hbase-client

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) spotbugs=3.1.12

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 31s
Docker mode activated.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m  1s
the patch passed

+1 :green_heart:
compile
0m 29s
the patch passed

+1 :green_heart:
javac
0m 29s
the patch passed

+1 :green_heart:
shadedjars
5m 42s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 26s
hbase-client in the patch failed.

_ Other Tests _

+1 :green_heart:
unit
1m  4s
hbase-client in the patch passed.

24m  4s

Subsystem
Report/Notes

Docker
Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/3/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1822

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 65002013bd79 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 716702a349

Default Java
2020-01-14

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/3/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-client.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/3/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-client.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/3/testReport/

Max. process+thread count
292 (vs. ulimit of 12500)

modules
C: hbase-client U: hbase-client

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/3/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 46s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+1 :green_heart:
mvninstall
4m 19s
master passed

+1 :green_heart:
compile
0m 27s
master passed

+1 :green_heart:
shadedjars
6m 35s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 25s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 50s
the patch passed

+1 :green_heart:
compile
0m 29s
the patch passed

+1 :green_heart:
javac
0m 29s
the patch passed

+1 :green_heart:
shadedjars
6m 21s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 22s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m  6s
hbase-client in the patch passed.

25m 43s

Subsystem
Report/Notes

Docker
Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/3/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1822

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux c13b5f1181d6 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 716702a349

Default Java
1.8.0_232

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/3/testReport/

Max. process+thread count
339 (vs. ulimit of 12500)

modules
C: hbase-client U: hbase-client

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/3/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

LGTM. Shouldn't cause log spam.

So, can you approve it? Thanks.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 50s
the patch passed

+1 :green_heart:
compile
0m 30s
the patch passed

+1 :green_heart:
javac
0m 30s
the patch passed

+1 :green_heart:
shadedjars
6m 40s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 25s
hbase-client in the patch failed.

_ Other Tests _

+1 :green_heart:
unit
1m 23s
hbase-client in the patch passed.

29m 28s

Subsystem
Report/Notes

Docker
Client=19.03.11 Server=19.03.11 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/4/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1822

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 542e5551bf55 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / f2fde77fc3

Default Java
2020-01-14

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/4/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-client.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/4/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-client.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/4/testReport/

Max. process+thread count
196 (vs. ulimit of 12500)

modules
C: hbase-client U: hbase-client

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/4/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 22s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
3m 43s
master passed

+1 :green_heart:
checkstyle
0m 29s
master passed

+1 :green_heart:
spotbugs
1m  1s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 20s
the patch passed

+1 :green_heart:
checkstyle
0m 27s
hbase-client: The patch generated 0 new + 12 unchanged - 2 fixed = 12 total (was 14)

+1 :green_heart:
whitespace
0m  1s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
11m 16s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1.

+1 :green_heart:
spotbugs
1m  6s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 15s
The patch does not generate ASF License warnings.

30m  7s

Subsystem
Report/Notes

Docker
Client=19.03.11 Server=19.03.11 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/4/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1822

Optional Tests
dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle

uname
Linux 9950d60a08fa 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / f2fde77fc3

Max. process+thread count
94 (vs. ulimit of 12500)

modules
C: hbase-client U: hbase-client

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1822/4/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) spotbugs=3.1.12

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.


Snapped_Waypoints - LineString or MultiPoint

As discussed here https://github.com/graphhopper/graphhopper/commit/8890b52c83d09c179577a072d296206e6e408a5f#commitcomment-17876754
I think we should set the snapped_waypoints as MultiPoint instead of LineString. The actual content stays the same, so this is basically only semantics. Examples can be found here: http://geojson.org/geojson-spec.html#id3
I just copied both examples for easier discussion:
{
   "type": "MultiPoint",
   "coordinates": [ [100.0, 0.0], [101.0, 1.0] ]
}
{
   "type": "LineString",
   "coordinates": [ [100.0, 0.0], [101.0, 1.0] ]
}

Thanks&Done!

Thanks for the fast fix. So I'll close this issue.

@karussell I just saw that the tests are failing. But it seems to be unrelated to this issue.

Yeah, strange. Happens rarely ... opened an issue


Fixed broken accordion for Job list view.

Closes: #2555
What's Changed

This reverts extras/inc/job_table.html to the previous version from commit d80a3cf94928f988a16a5d99d7fdc4ade025782f.

TODO

[ ] Explanation of Change(s)
[ ] Added change log fragment(s) (for more information see the documentation)
[ ] Attached Screenshots, Payload Example
[ ] Unit, Integration Tests
[ ] Documentation Updates (when adding/changing features)
[ ] Example Plugin Updates (when adding/changing features)
[ ] Outline Remaining Work, Constraints from Design

This does reintroduce the hypothetical issue that #2317 was attempting to fix, i.e. that this may not work correctly if we have multiple different groupings that slugify down to the same string, but IMHO that's an unlikely corner case that I'm OK with not mitigating at this time.

Right yeah, that's the part I wasn't clear on, so thanks for clarifying. I think the only thing would be to combine the slug with the index. But I didn't really spend time experimenting.


1.12 branch will not build

Description
running gradlew build after downloading the 1.12 fork results in an error
Base information
i am using java SE dev kit 8 as the gradle wrapper version is old
Crash report
FAILURE: Build failed with an exception.

* What went wrong:
Could not resolve all dependencies for configuration ':apiCompileClasspath'.
> Could not resolve mcp.mobius.waila:Hwyla:1.8.23-B38_1.12.
  Required by:
      project :
   > Could not resolve mcp.mobius.waila:Hwyla:1.8.23-B38_1.12.
      > Could not get resource 'http://tehnut.info/maven/mcp/mobius/waila/Hwyla/1.8.23-B38_1.12/Hwyla-1.8.23-B38_1.12.pom'.
         > Could not GET 'http://tehnut.info/maven/mcp/mobius/waila/Hwyla/1.8.23-B38_1.12/Hwyla-1.8.23-B38_1.12.pom'.
            > tehnut.info

According to the HWYLA readme, we need to update how we download it in the gradle buildscript.

by the way if/when you fix this you have to update to hwyla Hwyla:1.8.26-B41_1.12.2 as the older versions (or at least B38) is not on maven anymore


Support for .list and .help

As oodle gets bigger and more commands are being added it would be useful to have a command like ".list" which lists all commands then for seeing how to use a command: ".help " to tell you how to use a certain command.

Will implement by this week.


Create a python  script to determine the performance and accuracy of the datasets

What exactly are the requirements for this. What input should the script accept and what output should it provide?

Currently these are the following parameters we are considering,
1.Data Outliers ( can use Z-Score)
2.Data Bias and Skew Level in dataset
3.Data Timeliness (like check the data's update level eg:- recent data)
4.Data Linearity ( *keep this at last )
Note:
For Initial Phase these parameters should analysed the JSON and CSV file formats later we can expand.
Your Contributions and Suggestions are most welcomed

Ok. So given a file, said parameters should be evaluated for each numerical column, that's it? I have some questions about how to find a solution to implementing this:

For requirement 1, at what z-score should a value be considered a data outlier? 1, 2, 3?
How should I evaluate the data bias and skew levels based on the data from a file?
As for the timeliness, you want to know when a file was last modified, that's right? How can that be checked?
Could you please explain what you mean by data linearity? It's not totally clear how to evaluate it.

I will try to implement this, but it's a complex problem and I'm afraid I might not have the skills, so please excuse me if I can't find a solution and I end up throwing the towel.

Appreciation for your interest, Yes for a given file if the column is find as Numeric column then these parameter should be evaluated.

For Outlier in Initial you can try with Threshold Value of 3 for Z-Score.
I too worked for the Logic for it , Since at each project the bias and its need constantly change it's hard to derive a common solution, sure will work and find a logic for it and let you know bro.
Timeliness here refer to Recentness of a data for example if a dataset contain data up to 2020 when we try to validate dataset if we can find the data are relatively Old or New it's easy for user to choose which will best suit for his/her Project.
And your idea of Last modified is also nice.User can know the constant update of dataset through it.
I'm not very much confident in its usage. But in some project of ML, Linearity of data is checked to find right model to fit.

You can try as much as you can bro, there is no problem , we can collaborate and work on it. I hope you get some clarity.

Thank you for your help @Gladwin001 . I've completed a script that finds the outliers in the data. It reads a file, finds its numerical columns, calculate mean and standard deviation of all columns to calculate Z-scores and then finds the outliers. The outliers are then put into a list that is then nicely printed on the terminal.
Right now, the code lives on the fork I created this morning in a separate branch as demanded in the README for this project. It only implements the first requirement? Should I request a pull now or should I delay it and  try to implement the other features before creating the pull request?
To be honest, I think the first requirement was probably the easiest and the other ones will require much more cleverness and resourcefulness. I'm now going to try to find a way to solve another one, but I don’t guarantee I'll find anything. Have you made any progress?

You can implement one feature and then make a pull request.

@rtiop are u working on this issue?

How should it be implemented? I haven't made any progress on this since I implemented find_outliers.py. The issue is that I cannot think of a way to evaluate the four parameters @Gladwin001 mentioned when I first started working on this. Here are a few problems we face:

Data outliers: this was the only one I was able to implement.
Data Bias and Skew Level: To find out about this would require comparing a file against a truly representative dataset. A script that only has access to one file cannot know if it is biased or skewed.
Data Timeliness: Since each file, be it CSV or JSON, is structured differently, there is no way to create a script that consistently and accurately returns the timeliness of any file.
Data Linearity: I still do not know what it is, how to measure it and what are its uses.

It's a complicated issue. But could I know how all of this will be used later? Knowing what exactly is needed would help me know what it is that we are trying to achieve.

Once again thanks for your contribution, Appreciate your contribution via find_outliers.py. Here are some answers for your doubts,

Data Outlier: Implemented through Z-Score and Threshold Value
Data Bias and Skew Level: I have working on it. I think we can find it with Quartile Range Concept and I will try to Implement it in short days.
Data Timeliness:  There are multiple ways to find timeliness and as you mentioned each are structured differently so try to find an alternative ways.
Data Linearity: Currently I have not focus on this parameter, In future if it didn't have proper use case we can discard it.

What is our exact need is,
If an developer searching for an dataset, We like to give as much as useful parameter to developer which helps to choose a right dataset for his project.You can also suggest any other useful parameters to us.

The issue with data timeliness
@Gladwin001  I've taken a look at most of the data sets, and most of them do not have any timestamp or any information that could tell us more about its timeliness. A way to address this problem would be to manually check all data sets and match all of them to a date, but that's unpractical and not a good idea. Therefore I cannot write a Python script that consistently finds the data timeliness. The underlying issue is that we do not have the metadata to solve this.
Metadata
A way to address this issue would be for all data sets to have an accompanying CSV/JSON or .txt file that includes useful metadata about the file (such as its origin, a timestamp, a description, etc). To implement this would require a lot of work, but it would empower us to give all this information to the developers. Nevertheless, I think it should be done. It would be a worthwhile investment, albeit a costly one. I know it's not a simple issue but it would be good for DataStorehouse. It would let us give  more information to the users.
At a lower level, it would require us to find this information for already existing files and make contributors find it for new files, so will be hard to implement and a lot of hours put into it.
What I've done
Given the aim to give as much as useful parameter to developers, I've refactored my code to make it easier to add new functions to find data. I'm going to create a pull request so you can all see the details, but broadly it makes it easier to add data-finding functions to a unified script (analysis.py).
Note: I won't be able to work on this repository for some weeks starting Thursday since I'll be on vacations. Thank you for your understanding.
Summary

We should rethink the way we try to solve this issue.
Having a standardised  way to store metadata would help
I'm going to create a pull request to refactor some code and make it easier to implement new functions

Thanks for your contribution @rtiop