Overloading arrow function return type should allow union type

TypeScript Version:  v3.5.1 (On TypeScript Playground)
Search Terms:
Arrow function overload
Code
const foo1 : {
    (a: number): number;
    (a: string): string;
} = (a: number | string): number | string => a;

// Type '(a: string | number) => string | number' is not assignable to type '{ (a: number): number; (a: string): string; }'.
//   Type 'string | number' is not assignable to type 'number'.
//     Type 'string' is not assignable to type 'number'.

const foo2 : {
    (a: number): number;
    (a: string): string;
} = (a: number | string): number & string => a as any;

function foo3(a: number): number;
function foo3(a: string): string;
function foo3(a: number | string): number | string {
    return a;
}

Expected behavior:
The return value of implementation of overloading arrow function should be number | string which is allowed with function.
Actual behavior:
TypeScript expect the return value of implementation of overloading arrow function should be number & string.
Playground Link:
Link
Related Issues:

This is correct error.
If you check how you should use result of this function (result of the implementation, skip overloads for now because implementation has no knowledge of input->output types mapping) you would expect that implementation will support both of this cases without type errors.
const a: number = foo1(someNum);
const a: string = foo1(someStr);

this can be achieved only by a type that is assignable to number and string at the same time and this is number & string.
If you'll have function that has signature (a: number | string) => number | string e.g.
(a: number | string) => Math.random() > 0.5 ? Number(a) : String(a);

then it won't work in case described above.
In case of foo1 you can use
const foo1 : {
    (a: number): number;
    (a: string): string;
} = (a: number | string): any => a;
                          ^^^

But users will see only 2 correct signatures.

@IllusionMH My problem is the result is not consistent with function.
For example,
function foo3(a: number): number;
function foo3(a: string): string;
function foo3(a: number | string): number[] | string {
    return a;
}

It shows

This overload signature is not compatible with its implementation signature.

The error is on the overload signature.
However, the error in arrow function overload is on the implementation signature. As a result, function can return string | number or string & number  without error.
If I use any, I will lose the type check on return type.

It can't be consistent with functions with current syntax. Function overloads has special syntax and therefore can provide special handling of return types.
Your examples are regular function assignment and without special syntax are indistinguishable from incorrect implementations.
type Overloaded = {
    (a: number): number;
    (a: string): string;
};

declare function correct(a: number | string): number & string;
declare function incorrect(a: number | string): number | string;

const f1: Overloaded = correct;
const f2: Overloaded = incorrect; // correctly errors

You don't want to allow second case.
To request similarity you should create proper feature request/proposal with convincing use cases, instead of using Bug report template.

I can see no reason not to use the same algorithm which is used for a function overload declaration.
It's true that the function declaration algorithm would be better if the return values in the implementation were type checked against the signatures in context.  We can even simulate it in user code:
function foo3(a: number): "num";
function foo3(a: string): "str";
function foo3(a: bigint): never;
function foo3(a: number | string | bigint): "num"|"str";
function foo3(a: number | string | bigint): "num"|"str" {
    if (typeof a === "string"){
        const ret = "str";
        // Conceptual simulation of type checking
        // Unfortunately we cannot write typeof foo3(a) in user code,
        // but that lookup can be done in typescript internals.
        // Obviously would not deliberately enter infinite loop.
        const testret = foo3(a);
        ret satisfies typeof testret;  // OK
        return ret;
    }
    else if (typeof a === "number"){
        const ret = "num";
        const testret = foo3(a);
        ret satisfies typeof testret; // OK
        return ret;
    }
    else if (typeof a === "bigint"){
        const ret = "num";
        const testret = foo3(a);
        ret satisfies typeof testret; // error
//          ~~~~~~~~~
//Type 'string' does not satisfy the expected type 'never'.(1360)
        return ret;
    }
    throw undefined;  // a==="bigint" should take this branch.
}

The obvious solution then is to unify the treatment of function declarations and assignment of arrow functions to overloads types,
and ALSO upgrade BOTH with the above return value checking.  This same reasoning applies to generic functions as well.
Setting the return type of the implementation with the intersection of return types just breaks the existing type checking (which although not perfect is still better than nothing).

How is it possible that the "correct" function returns string & integer at the same time ? The type C = A & B in TS is used to tell the compiler that the variable of type C will be both A and B at the same time, correct ?


Ragnarok Online

Shows error dialogue d3denum init and cannot init d3d then exit after launching the game.
cnc-ddraw.log

The game uses Direct3D which isn't supported by cnc-ddraw


Update NavigationView pane scrolling behavior

Description

Previous behavior was not ideal, this PR updates the behavior.
New behavior:

If both collections fit, both are shown
If one of them needs scrolling, and the others height is below half of the available pane height, the exceeding item scrolls
If both need more than half of the available height, both get half of the available space

In addition to that, the scrollviewer for the panefooter will try to keep the settings item in view when changing size to ensure it is visible when it was before resizing.
Motivation and Context

Closes #3447
How Has This Been Tested?

Add new interaction tests.
Screenshots (if appropriate):

/azp run

Here a gif showing the new behavior of sizing for menu items and panel items (@YuliKl @anawishnoff @StephenLPeters FYI):

Shouldn't there be a Separator line above the Footer items, it would make the overflow regions more obvious

Shouldn't there be a Separator line above the Footer items, it would make the overflow regions more obvious

Very good idea, added that now:

/azp run

@chingucoding Looks like there are a few test failures.

Taking a look now.

Looks like there were calculation issues (off by 1 px), tests should reflect that now. Also VisualTreeVerification tests failed, files should be updated now.

/azp run


Add support for Ultra

We should add docs for Ultra usage as well (--application.profile true --application.profileType ultra)

Will add the docs. Also the tests are failing which is not right, they are quite stable. The tool also just got support for output filenames, so this can be used instead of taking whatever 'json.gz' is in the folder.


Add GetResource gRPC service and endpoint.

/tests

The format of the resource description will require some thinking, but this PR doesn't determine that.

Hi @fitzthum ,
We have created a new issue to discuss the definition of resource description. Please help to have a review also.
Issue: https://github.com/confidential-containers/attestation-agent/issues/44

Ok, I guess we are waiting for image-rs support to get merged?

\tests

I think this PR can get merged first and don't need to wait for image-rs, so that we can start adding KBC support for get resource :)


Getting unicode with string when doing an inspect on a container that uses --net host.

Description of problem:
Getting unicode with string when doing an inspect on a container that uses --net host.
Client:
 Version:      1.8.3
 API version:  1.20
 Go version:   go1.4.2
 Git commit:   f4bf5c7
 Built:        Mon Oct 12 05:39:44 UTC 2015
 OS/Arch:      linux/amd64

Server:
 Version:      1.8.3
 API version:  1.20
 Go version:   go1.4.2
 Git commit:   f4bf5c7
 Built:        Mon Oct 12 05:39:44 UTC 2015
 OS/Arch:      linux/amd64

Containers: 8
Images: 1182
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 1198
 Dirperm1 Supported: true
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.19.0-28-generic
Operating System: Ubuntu 15.04
CPUs: 4
Total Memory: 11.43 GiB

Linux 3.19.0-28-generic #30-Ubuntu SMP Mon Aug 31 15:52:51 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux

Environment details (AWS, VirtualBox, physical, etc.):
Physical.
How reproducible:
Always.
Steps to Reproduce:
docker inspect $(docker run -d --net host alpine sh) | grep \"Gateway

or
docker inspect --format={{.NetworkSettings.Gateway}} $(docker run -d --net host alpine sh)

Actual Results:
See above steps to reproduce commands.
Expected Results:
Would get the docker host's IP address.
Additional info:

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

@GordonTheTurtle not sure how your parser works, but I thought I did what you wanted based off of https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues. Someone let me know if @GordonTheTurtle actually needs more info or is just naive.
Thanks!

I'm not sure why docker would provide this info with --net=host since docker did not create or manage it.
Also the gateway would be the host's gateway, not the host's IP address.


Show JSX if PropVal is a React element

Issue:
What I did
JSX elements within prop values are now rendered completely (in Story Source and Prop Table). The behaviour can be reverted by configuring showSourceOfProps: false.
This yields some errors in the source code generation (with indentation).
I think it would be easier to do the JSX source generation outside of React components since there are a lot of "inverted" dependencies. Rendering the parent depends often on whether the child has multiple lines of code or just a single one.
There is some paddingLeft used for indentation  currently what breaks copy & paste behaviour. I think some of the issues outlined could be fixed by using paddingLeft in some cases but I'd rather not do that (because copy & paste).
Would you be open to a diffrent approach on rendering the source code?

How to test
Create a story with a property value of type JSX element.
  .addWithInfo('Test', () => {
    const Test = ({ foo, bar, children, arr }) => (

{foo}

{bar}

      {arr}

{children}

);

    Test.propTypes = {
      foo: PropTypes.node.isRequired,
      bar: PropTypes.node,
      arr: PropTypes.arrayOf(PropTypes.node),
      obj: PropTypes.objectOf(PropTypes.node),
      children: PropTypes.node.isRequired,
    };

    Test.defaultProps = {
      bar:
default works in PropTable!
,
      arr: null,
      obj: null,
    };

    return (
      <Test foo={
} arr={[<Test foo={
} />, <Test foo={
} />, <Test foo={
} />, <Test foo={
} />, <Test foo={
} />]}>
        <Test
          foo={
foo
bar

}
          bar={
test
}
          arr={[<Test foo={
} />, <Test foo={
} />, <Test foo={
} />, <Test foo={
} />, <Test foo={
} />]}
          obj={{ foo: <Test foo={
} />, bar: <Test foo={
} />, }}
        >

children

        </Test>
      </Test>
    );
  })

To answer your question on the rendering part, I don't think any of us are married to a specific method. A lot of the decisions were made way back when it was hard to predict the complexity of info.
If you think there's a more logical way to render it, we'd love a PR.

I'll try to elaborate on a more robust way to render the source. I think the copy & paste use case is big enough issue for me to stop using padding-left as indentation.
I'll add an example to the kitchen sink. Thanks for the heads up.

@sventschui could you have a look at the merge conflicts? 🙇

@ndelangen I think you can close this PR. This will yield some errors. The source code generation thingy needs to be refactored


Prevent possible nil pointer in go client round tripper.

Happened to me while having debug enabled and the client received an error.
I find the round tripper a bit suspicious anyway... log.Fatal is very harsh?

Happened to me while having debug enabled and the client received an error.
I find the round tripper a bit suspicious anyway... log.Fatal is very harsh?

Yes, can't remember why i added this log.Fatal, could you please add simply
return resp, err
instead ?

In the first case it's not possible because there is no response yet and it would interrupt the request. :/
Just tried to resolve it with fmt.Printf() instead.


Inconsistent behaviour when switching between two local networks (subnet)

Context
Working on an example app to interact with a subnet contract.
Users will be able to transfer STX and NFTs between a L1 (in my case, the local devnet) and a L2 (the local subnet).

Issues
Extension Network selection
In the wallet main page, when switching between the L1 and L2 networks, the wallet inconsistently displays my L1 or L2 balance, regardless of the selected network.
Flow

from L1 (local devnet) I can see my L1  balance ✅
switch to L2 (local subnet)
still see L1 balance ✅
refresh, refresh, refresh, sill seeing L1 balance ❌, until eventually...
see L2 balance ✅
switch back to L1
refresh, refresh, refresh, sill seeing L2 balance ❌, until eventually...
see L1 balance ✅

Note: no issue to report when switching between mainnet/testnet/devnet 👍
Note2: reproduced in Firefox and Chrome

In transaction pop up
When performing a transaction, and passing the following network to openContractCall const l2Network = new StacksMocknet({ url: L2_URL });
The popup page will inconsistently show L1, L2 (and sometime I believe testnet) balances.
And show contradictory data like showing the L1 balance) with 140STX bet saying insufficient founds to transfer 1 STX. Or the transaction preview looks good but it fails to transfer (while it should, because I already transferred 10STX to the L2 and the API does return 10STX as a balance.

Here it shows the tow issues:
- displaying L1 balance in the top
- not performing the transaction because of "insufficient balance" (while L2 has 10STX)

I know it may be hard to debug or reproduce because it's about seeing inconsistent data (at least it's consistently inconsistent). I'm available for a call if needed

https://user-images.githubusercontent.com/911307/225629216-6087e17d-f2ee-46ef-a88b-dcc4c7d515cf.mov
How to reproduce
Here are the step to boot a local mocknet and subnet APIs with clarinet (version 1.5 or higher - clarinet --version).
Create an empty project with clarinet new proj
In settings/Devnet.toml, un-comment or add:
enable_subnet_node = true
disable_subnet_api = false

Start project with clarinet integrate
In the top right "service" section, should appear:
stacks-api           http://localhost:3999/doc
subnet-api           http://localhost:13999/doc

Then the subnet can be added to the wallet

From there, the behavior described above should be reproducible (even though you can't really deposit found on the subnet)

Hey @alter-eggo
Things seem way more stable in the wallet view. The balance are accurate!
In the transaction popup on the other hand, I still see calls to the testnet api
Here, I call callContract with const l1Network = new StacksMocknet({ url: "http://localhost:3999" }); ✅

Here, I call  callContract with const 21Network = new StacksMocknet({ url: "http://localhost:13999" }); ❌

Thanks @alter-eggo for your help.

first problem regarding network selection is fully resolved on the dev branch
the second part is my bad, it was fixed by adding, I thought the key was optional and how to do with network protection or smth like that, but it seems like it's actually mandatory, everything works well we a key.

👉 Maybe the key field could be mandatory, and we shouldn't be able to add a network without it?


Add theme to Force 2017 website

as per
https://github.com/force11/force2017/issues/9#issuecomment-299326160 .

Might want to include a logo if we are going to create one.

Just checked - couldn't find it on
http://www.force2017.org/
or https://www.force11.org/meetings/force2017 , so
pinging @sthagstrom again.

Noting that this still hasn't been added to the site. @sthagstrom

It is located as part of the logo on https://www.force2017.org/ but is still not on https://www.force11.org/meetings/force2017 that I can see at least. Also, do we need it more prominently expressed on the first site than simply in the logo?

I forwarded the above comment on to Stephanie via email. :)

I heard back from Stephanie and she said that the second of the links is unpublished and traffic is being directed to the site at Copernicus instead. So, if we would like the theme more prominently stated someway on the first link, we need to forward that information to Stephen Diggs who will be acting as the connection point to Copernicus about the website.


Markdown syntax highlighting in admonition code blocks is semi-broken in Obsidian v0.13.30 and v0.13.31

Behavior in v0.13.30 and v0.13.31:

In v0.13.23:

So the syntax highlighting is still there, but more similar to the old editing mode. Also math formulas are no longer rendered inside the admonition block.

Is this in live preview?

@EHBaozi
Are you still experiencing this issue?


API endpoints for software on agents

We need API endpoints for viewing and manipulating what software is available on what agents.

These were added in #495


Update some of the models and converters to better match spec

The DID Document appears to support more formatting of data now than at the last update of this repo.
This PR adds in:

the option of having a context object. E.g:
"@context": [          "https://www.w3.org/ns/did/v1",          {              "@base": "did:example:1234"          }      ],
the option of having a service endpoint object. E.g:
"serviceEndpoint": {                  "origins": [                      "https://example.com/"                  ]              }

There is still more to be done to this repo but this helps with some of the current state of DID documents

Hey @juanfranblanco, I am working on the DID team in Microsoft, we are currently looking to make use of the dotnet models. This is an at least initial PR to get use to a state of us being able to use this repo.
The other thing would be working towards having this repo published as a nuget package. Even as something that is labeled as an alpha or preview since it isn't at a production state, possibly this is something someone with write access would be better at doing if that's possible? I'll also keep trying to get everything updated to spec when I get the time.
Many thanks, Adam.


horizontally align footer links to seperate them further

add final update before submitting: footer spacing and checkboxes in subscription form


refactor: Remove usages of serde_v8::Value

Removes usages of serde_v8::Value in favor of manually handling v8::Value in:

op_leak_tracing_get_all
op_get_promise_details
op_get_proxy_details

Codecov Report
Attention: Patch coverage is 34.61538% with 17 lines in your changes are missing coverage. Please review.

Project coverage is 80.90%. Comparing base (a736d86) to head (2e8a73d).

Files
Patch %
Lines

core/ops_builtin_v8.rs
34.61%
17 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #677      +/-   ##
==========================================
- Coverage   80.94%   80.90%   -0.04%
==========================================
  Files          97       97
  Lines       24425    24433       +8
==========================================
- Hits        19770    19767       -3
- Misses       4655     4666      +11

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


expose unplugin options as a type that can be imported

e.g.
import type { UnpluginVueRouterOptions } from 'unplugin-vue-router'

Current workaround:
import VueRouterVitePlugin from 'unplugin-vue-router/vite'
type UnpluginVueRouterOptions = Partial<Parameters<typeof VueRouterVitePlugin>[0]>

Thanks! The option type should be exposed directly in src/index.ts as well

I think it's imported but not exported from src/index.ts.


Introduce Project Editing APIs

Purpose

Introduce Project Editing APIs
Resolves #25455

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

@azinneera This PR topic isn't adherign to our best practices. Refer Contributing Guide for more details.
A properly formed Git commit/PR subject line should always be able to complete the following sentence:
If applied, this commit/PR will your subject line here

Merged to a different branch. Hence closing.


Rubicon-adapter: Integration questions

Hi,
Can you point me to the right documentation or the person to answer a question regarding rubicon adapter integration?
We got the following config for s2s adapter at client side
pbjsConfig.s2sConfig = {
    accountId: '1',
    bidders: ["rubicon"],
    enabled: true,
    timeout: Settings.prebidBidderTimeoutMS,
    adapter: 'prebidServer',
    endpoint: 'https://pbs.xxxxxxxx.com:8080/openrtb2/auction',
    cookieSet: false
};

The server forwards it correctly, but receives a http 400 from rubicon. I guess we need syncEndpoint or cookieSetUrl specified as well. What is the correct value for that?
Secondly, we get some errors back from Rubicon. Is this the right place to discuss that or should we go through the account manager route?

If anyone is blocked on similar issues, here are the relevant docs

https://github.com/rubicon-project/prebid-server-java/blob/master/docs/endpoints/cookieSync.md
https://github.com/rubicon-project/prebid-server-java/blob/master/docs/developers/cookie-syncs.md
https://github.com/rubicon-project/prebid-server-java/blob/master/docs/developers/gdpr.md

Closing the issue


Implement option to download VCard

Styling is left up to the maintainer :)

Thanks @tmueller for the extremely useful feature 🎉


Definition of Sector should be updated

Definition of Sector should be updated
A sector is a subdivision of something

Related to #57

I'll close this as its the same issue as #30


fix: typo in README.md

represention -> representation

Hi! Thank you very much for pointing this out!


kinetic-devel: select CMake build type if not configured.

As per subject.
See the commit comment for rationale.
Summary: CPU usage for an unoptimised build is quite a bit higher than for an optimised one.
If user has not explicitly set a build type, select RelWithDebInfo, which is equivalent to Release with debug symbols enabled.

This is inspired by how MoveIt detects and configures this.

@miguelprada wrote:

Some quick tests from my laptop show a reduction of around 20% in CPU load with this.

great. That was the point of the PR.
You should still be able to override the build type by configuring it in your catkin_tools profile or by adding -DCMAKE_BUILD_TYPE=Debug (fi) on the catkin_make command line.

Could you merge @miguelprada?

Thanks :+1:


New arg split-by-locale-to added to translate.js

This PR adds the functionality to translate.js to split the translations into separate files based on local. The arg to do so is called 'split-by-locale-to', and I'm open for suggestions on the naming of the arg. This functionality will reduce the manual work of code splitting / lazy load translations based on locale.
The PR was based on the discussions in this issue. However, the implementation differs a bit from the solution presented here. The reason for the difference is that translate.js outputs default to stdout, which would be messy to deal with when we want to output to several files. The solution here could have been used if a new script was created, however in my opinion it seems like a better option to add an arg instead to the existing script.
NOTE: I'm probably overseeing something, but I was not able to run any linter or any tests for the project before the commit. Seems like there are no eslint config? And there are only test scripts and a jest config added for the demo app?

I like where this PR is going!
Could you add a test script in this package.json file? Something named like test-translate-fbts-into-output-dir...
It's similar to this:
https://github.com/facebookincubator/fbt/blob/fd07a6b20d440d49b013472d50ed6f4ac9cf2594/demo-app/package.json#L15
Finally, please add a new entry in the CHANGELOG file for posterity! :-)

Fixed all the issues and added the stuff requested in commit 27cc3d412d8bb15b04d84110b6d26555708511a8.

Finally, please add a new entry in the CHANGELOG file for posterity! :-)

Not sure if I'm suppose to do this since I don't know the commit hash (you might squash and merge). I also don't have access to publish to npm and I think we should add a minior to babel-plugin-fbt (0.9.14 -> 0.10.0), which I think should be reflected in the changelog. This work should probably be made of the one in charge of doing the releases for fbt right?
Let me know if there is anything else that needs to be changed / added! :)

Got another question. Should I add the test-translate-fbts-into-output-dirscript to the live-demo-app? Not really sure what the purpose of that is compared to the regular demo-app 🤔

Fixed all the issues and added the stuff requested in commit 27cc3d4.

Finally, please add a new entry in the CHANGELOG file for posterity! :-)

Not sure if I'm suppose to do this since I don't know the commit hash (you might squash and merge). I also don't have access to publish to npm and I think we should add a minior to babel-plugin-fbt (0.9.14 -> 0.10.0), which I think should be reflected in the changelog. This work should probably be made of the one in charge of doing the releases for fbt right?
Let me know if there is anything else that needs to be changed / added! :)

Feel free to add the CHANGELOG entry without the proper git commit hash. You could simply use the github PR number link instead.
I can do that too if you prefer, no worries.

Got another question. Should I add the test-translate-fbts-into-output-dirscript to the live-demo-app? Not really sure what the purpose of that is compared to the regular demo-app 🤔

There's no need to add it to the "live demo app" because I don't expect it to remain in the repo for much longer. I'll probably archive it in a git tag and remove it from the repo since it's kind of a duplicate of the normal demo app.

I think we should add a minior to babel-plugin-fbt (0.9.14 -> 0.10.0), which I think should be reflected in the changelog.

@AdamRamberg Sorry but I'm not really sure what you meant here? 🤔

I think we should add a minior to babel-plugin-fbt (0.9.14 -> 0.10.0), which I think should be reflected in the changelog.
@AdamRamberg Sorry but I'm not really sure what you meant here? 🤔

Since we are adding functionality to babel-plugin-fbt(output-dir arg to the translate.js script) my thought was that we should probably increase the minior version of babel-plugin-fbtbefore it gets published to npm next time. Currently it's at 0.9.14 and wlll be at 0.10.0 after. This should also be reflected in the changelog, eg:
babel-plugin-fbt versions
0.9.14:
c11e9fd Enable custom Babel plugins for FBT collection

0.10.0:
#73 New arg `output-dir` added to the translate script. The output translation files will be split by locale.

So should I increase the minior version of babel-plugin-fbt and add a comment to the changelog? :)

Thanks for the clarification! I understand what you meant now.
For now, let's edit the CHANGELOG like follow:
### babel-plugin-fbt versions

- Next MINOR version
  - #73 New arg `output-dir` added to the translate script. The output translation files will be split by locale.
- 0.9.14:
  - [c11e9fd](https://github.com/facebookincubator/fbt/commit/c11e9fd) Enable custom Babel plugins for FBT collection

I would prefer using this format to avoid setting hard expectations on the future npm version numbers.
I'll take care of updating the CHANGELOG and the version numbers before publishing to npm. :-)

I just added an entry to the CHANGELOG.md file. Sorry for the world's slowest copy and paste ;)
Let me know if there are any other changes required before merging.

👍
Updated code looks good and my translation tests were successful too.

@jrwats, I think we'll also need your approval before merging this PR.

LGTM, but you've already approved it.  We'll need to bump some internal tools checksums for the changes is all.


Update axios to version 1.7.9 and add @types/axios as a dev dependency

Description
Upgrade the axios version to latest and added the type definition to resolve the issue reported.
Type of Change

[X] 🐛 Bug fix (non-breaking change which fixes an issue)
[ ] ✨ New feature (non-breaking change which adds functionality)
[ ] 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] 📚 Documentation update

Pre-flight Checklist

[X] Changes are limited to a single feature, bugfix or chore (split larger changes into separate PRs)
[X] Tests are passing (npm test) and code is formatted and linted (npm run format && npm run lint)
[X] I have reviewed contributor guidelines

Additional Notes

Hey thank you for the PR 🙏 I'm not accepting PRs that modify the package-lock for security reasons but I will update axios myself when I get the chance (or feel free to re-open with just the package change).

No issues 👍


Typo in the "Examples" section

This example:  "deviceTrustType	AzureAD, ServerAD, Workplace    device.deviceOwnership -eq "AzureAD"" has a typo.  "device.deviceOwnership" should be "device.deviceTrustType"

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

ID: 5f7a4070-1be8-5f1f-0092-68fb6b9e2305
Version Independent ID: cd34312f-b507-affe-555e-1e0c63e7d494
Content: Rules for dynamically populated groups membership - Azure AD - Microsoft Entra
Content Source: articles/active-directory/enterprise-users/groups-dynamic-membership.md
Service: active-directory
Sub-service: enterprise-users
GitHub Login: @barclayn
Microsoft Alias: barclayn

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

@PJMarcum
Thanks for contributing towards enhancement of the document. Change has been performed, once approved by author. It will reflect on the document itself. We are going to close this thread. Please feel free to reach out for further assistance.


Mocking out response when running in browser testing

Hi
I am writing some karma tests which need to mock out the response of the xhr request.
We are utilising this library to do a request and is wrapped around a promise. However when running in browser testing it seems it is instantiating a new window.XMLHTTPRequest as when i inspect it in the karma debugger for the XMLHttpRequest is not there for the URL i am trying to hit.
I have tried to add the following option as per docs
xhr: window.XMLHttpRequest
But then this breaks it and it hangs and does not get a response from the endpoint.
Any help or resolution on this would be great

I can't get much information out of the writeup to be able to help.
There are multiple options, but before I start guessing what happened, just try putting this somewhere right after you plug in the thing you're using for requesting stuff:
xhr.XMLHttpRequest = MockXMLHttpRequest

I assume you fixed it, since you didn't engage back. Feel free to reopen if needed.


[mllib] [python] Add LassoModel to all in regression.py

Add LassoModel to all in regression.py
LassoModel does not show up in Python docs
This should be merged into branch-1.3 and master.

Test build #28446 has started for   PR 4970 at commit c2cb533.

This patch merges cleanly.

Test build #28446 has finished for   PR 4970 at commit c2cb533.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28446/
Test PASSed.

LGTM. Merged into both master and branch-1.3.


Ipam error from docker containerizer.

Running mesos scale test. This run was with 20 nodes, creating 100 containers on each.
2 failed on different nodes. Diags are from one of these. The container that failed is towards the end of list (see mesos agent logs). I can't see any restarts in the logs.
docker.log:May 04 14:50:44 dlagent-011 docker[1828]: time="2016-05-04T14:50:44.376509850Z" level=warning msg="Unable to connect to plugin: /run/docker/plugins/calico.sock:/IpamDriver.RequestAddress, retrying in 1s"
docker.log:May 04 14:50:45 dlagent-011 docker[1828]: time="2016-05-04T14:50:45.376803106Z" level=warning msg="Unable to connect to plugin: /run/docker/plugins/calico.sock:/IpamDriver.RequestAddress, retrying in 2s"
docker.log:May 04 14:50:47 dlagent-011 docker[1828]: time="2016-05-04T14:50:47.378181479Z" level=warning msg="Unable to connect to plugin: /run/docker/plugins/calico.sock:/IpamDriver.RequestAddress, retrying in 4s"
docker.log:May 04 14:50:51 dlagent-011 docker[1828]: time="2016-05-04T14:50:51.378846368Z" level=warning msg="Unable to connect to plugin: /run/docker/plugins/calico.sock:/IpamDriver.RequestAddress, retrying in 8s"
docker.log:May 04 14:50:59 dlagent-011 docker[1828]: time="2016-05-04T14:50:59.440497346Z" level=error msg="Handler for POST /v1.23/containers/de73e797cc698a2dfd545e65a4124f4df848503a242285087d1ab2c58627624f/start returned error: Post http://%2Frun%2Fdocker%2Fplugins%2Fcalico.sock/IpamDriver.RequestAddress: http: ContentLength=56 with Body length 0"

Diags here https://transfer.sh/1Boiv/diags.tar.xz

I just hit this same error during a libnetwork ST. My hunch is its because the STs and Dave's scale test use dind. Some more verbose errors around when it happens:
level=error msg="Handler for DELETE /v1.23/containers/calico-etcd returned error: No such container: calico-etcd"
level=error msg="Handler for DELETE /v1.23/containers/calico-etcd-ssl returned error: No such container: calico-etcd-ssl"
level=warning msg="Security options with `:` as a separator are deprecated and will be completely unsupported in 1.13, use `=` instead."
level=error msg="Handler for DELETE /v1.22/containers/host1 returned error: No such container: host1"
level=warning msg="Unable to connect to plugin: /run/docker/plugins/calico.sock:/Plugin.Activate, retrying in 1s"
level=warning msg="Unable to connect to plugin: /run/docker/plugins/calico.sock:/Plugin.Activate, retrying in 2s"
level=error msg="Handler for DELETE /v1.23/containers/0e4372bff052 returned error: Driver aufs failed to remove root filesystem 0e4372bff0526271b0436cc10b71b12aef0846038908fa5e062128e85e5e8c5a: mount still active"
level=error msg="restartmanager canceled"
level=warning msg="Unable to connect to plugin: /run/docker/plugins/calico.sock:/Plugin.Activate, retrying in 4s"
level=error msg="Handler for DELETE /v1.23/containers/calico-etcd returned error: No such container: calico-etcd"
level=error msg="Handler for DELETE /v1.23/containers/calico-etcd-ssl returned error: No such container: calico-etcd-ssl"
level=warning msg="Security options with `:` as a separator are deprecated and will be completely unsupported in 1.13, use `=` instead."
level=error msg="Handler for DELETE /v1.22/containers/host1 returned error: No such container: host1"
level=warning msg="Unable to connect to plugin: /run/docker/plugins/calico.sock:/Plugin.Activate, retrying in 8s"
level=error msg="Post http://%2Frun%2Fdocker%2Fplugins%2Fcalico.sock/Plugin.Activate: dial unix /run/docker/plugins/calico.sock: connect: connection refused"
level=error msg="Error saving new container to disk: open /var/lib/docker/containers/0e4372bff0526271b0436cc10b71b12aef0846038908fa5e062128e85e5e8c5a/config.v2.json: no such file or directory"
level=error msg="Clean up Error! Cannot destroy container 0e4372bff0526271b0436cc10b71b12aef0846038908fa5e062128e85e5e8c5a: No such container: 0e4372bff0526271b0436cc10b71b12aef0846038908fa5e062128e85e5e8c5a"
level=error msg="Handler for POST /v1.22/containers/create returned error: open /var/lib/docker/containers/0e4372bff0526271b0436cc10b71b12aef0846038908fa5e062128e85e5e8c5a/config.v2.json: no such file or directory"
level=error msg="Handler for DELETE /v1.22/containers/host2 returned error: No such container: host2"
level=warning msg="Unable to connect to plugin: /run/docker/plugins/calico.sock:/Plugin.Activate, retrying in 1s"
level=warning msg="Unable to connect to plugin: /run/docker/plugins/calico.sock:/Plugin.Activate, retrying in 2s"
level=warning msg="Unable to connect to plugin: /run/docker/plugins/calico.sock:/Plugin.Activate, retrying in 4s"
level=warning msg="Unable to connect to plugin: /run/docker/plugins/calico.sock:/Plugin.Activate, retrying in 8s"
level=error msg="Post http://%2Frun%2Fdocker%2Fplugins%2Fcalico.sock/Plugin.Activate: dial unix /run/docker/plugins/calico.sock: connect: connection refused"
level=error msg="restartmanager canceled"
level=error msg="restartmanager canceled"
level=error msg="Handler for DELETE /v1.22/containers/host1 returned error: No such container: host1"
level=warning msg="Unable to connect to plugin: /run/docker/plugins/calico.sock:/Plugin.Activate, retrying in 1s"
level=warning msg="Unable to connect to plugin: /run/docker/plugins/calico.sock:/Plugin.Activate, retrying in 2s"
level=warning msg="Unable to connect to plugin: /run/docker/plugins/calico.sock:/Plugin.Activate, retrying in 4s"
level=warning msg="Unable to connect to plugin: /run/docker/plugins/calico.sock:/Plugin.Activate, retrying in 8s"
level=error msg="Post http://%2Frun%2Fdocker%2Fplugins%2Fcalico.sock/Plugin.Activate: dial unix /run/docker/plugins/calico.sock: connect: connection refused"
level=error msg="Handler for DELETE /v1.22/containers/host2 returned error: No such container: host2"
level=warning msg="Unable to connect to plugin: /run/docker/plugins/calico.sock:/Plugin.Activate, retrying in 1s"
level=warning msg="Unable to connect to plugin: /run/docker/plugins/calico.sock:/Plugin.Activate, retrying in 2s"
level=warning msg="Unable to connect to plugin: /run/docker/plugins/calico.sock:/Plugin.Activate, retrying in 4s"
level=warning msg="Unable to connect to plugin: /run/docker/plugins/calico.sock:/Plugin.Activate, retrying in 8s"
level=error msg="Post http://%2Frun%2Fdocker%2Fplugins%2Fcalico.sock/Plugin.Activate: dial unix /run/docker/plugins/calico.sock: connect: connection refused"

Closing for the reason in my last comment, and since we haven't repro'd this in quite some time.


Using on a  instead of  doesn't show calendar always

I want to use the datetimepicker on div so that it will be shown always. However, when I use the following code, I have to click on "XXX" to see the calendar. Is this an issue?

XXX

Did you try using the inline option?
inline:true,
https://xdsoft.net/jqplugins/datetimepicker/


[Icon request] Logseq

Icon source: https://github.com/logseq/logseq/blob/master/resources/icon.png
App name: Logseq
Alternatively, how to change this to SVG, so I can follow the guideline and send a PR?

The process is literally in the README :)

Hi! I cloned the repo, prepared the icon, sent it to /svg/ with proper filename and created corresponding /mappings/ file.
However, the final step is not clear to me.
`source ./path/to/icon_map_fn.sh
icon_map "${app_name}"
symbol_ligature="${icon_result}"`
There is no icon_map_fn.sh file! Any help on that? Thanks in advance :)

Hey! This package uses an npm package to compile the fonts and generate generate_icon_map_fn.sh.
pnpm i
pnpm run build

I personally use pnpm but npm will work as well if you don't have pnpm installed!

Please also submit a PR after testing your changes :)


Is there a way to hook into an "afterMigrate" event?

I'm trying to figure out a way to always run code after a migration happens, in my case I'm using mysql and I'd like to ensure a set of views is up to date so that we don't have to manually re-create our views after the underlying schema of the table changes.
I noticed there is an endMigration event in the driver, maybe there is a way to hook into this?
Or perhaps if I put an index.js file in the migrations directory I can monkey patch the up function to call my recreate views methods after migrations are done?
Thanks for your help.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/35929151-is-there-a-way-to-hook-into-an-aftermigrate-event?utm_campaign=plugin&utm_content=tracker%2F73887&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F73887&utm_medium=issues&utm_source=github).

So you want to execute something after every migration? Or after all mgrations have been done?
endMigration is actually intended for transactions, currently there is no such hook though. Could you please specify what your use case is, so that I can better understand what exactly you need?
Thank you

No problem, here is what I'm trying to do:
I have some views in MySql, one unfortunate thing about MySql views is if you create them as SELECT * FROM tbl_name; it will expand the select to give you all of the columns at create time.  The problem is, that if we then want to add or remove a column from the underlying table through a migration, the view needs to be re-created so that it sees the new or removed column.
We could manually do this in all migrations that change the underlying table, however then we have to remember to do this every time we change a table referenced by a view.  I was hoping we could simple re-create all of our views anytime a migration is run.  We could do this ourselves anytime we run db-migrate, but I was hoping there might be a built-in way to hook into the library.

hum, okay. While I think that putting this after every migration would be really heavy and would really slow down your migrations. To say I'm not a big fan of: Doing something always to not be able to forget to do it, b/c this most often also means: I don't heave an idea what actually is happening. And as soon as you add more stuff to those hooks, everything gets worse. I would prefer control and clean migrations over your approach, but finally this is your decision ;)
So generally I don't have a problem with adding such kind of hooks to db-migrate, currently db-migrate does not have any hooks to intercept such steps like you just explained, so this would be a new feature. If you really need this now, you can go ahead and create a PR for this, that feature wouldn't be that big though.

Ok, thanks.  I agree this approach wouldn't be super clean and the efficiency of the migrations would suffer, but I wanted to explore my options.  I'll let you know if I end up wanting to go this route and I'll submit a PR if I do.

To also mention, db-migrate from 0.10.x on does have a programable api, so you would be at least able to  do stuff like the following:
https://github.com/db-migrate/api-examples/blob/master/promise.js
Maybe this already helps you, as if you do everytime the same thing after a single migration, it shouldn't make a difference for you if you do this same thing just once after all migrations have been executed.

Thanks for that, I did see that and thought about using it, although we are currently calling the db-migrate command instead of using the APIs on our webservers, we could switch to this method and leverage that feature like you mentioned.
Thanks for the help!


[EDIT] to resolve pronoun ambiguity in "Genesis warns..."

Entry
November 21, 2022
"Genesis warns of bankruptcy if they can't raise $1 billion"
Changes

Genesis Global Trading has reportedly been telling investors that they may need to file for bankruptcy if their attempts to raise at least $1 billion in new capital don't succeed.

ought to read

Genesis Global Trading has reportedly been telling investors that Genesis may need to file for bankruptcy if its attempts to raise at least $1 billion in new capital don't succeed.

The original form leads the reader to the idea that the investors may need to file for bankruptcy (which may be true, but that doesn't seem to be the idea that the sentence is intended to convey).
I'd also change the headline from

Genesis warns of bankruptcy if they can't raise $1 billion

to

Genesis warns of bankruptcy if it can't raise $1 billion

In general, it's okay to use the plural pronoun for the singular noun antecedent, but in this context it is confusing.
Agreement
By submitting this issue, I irrevocably agree to release my contribution under the CC-BY 3.0 License. I agree that a hyperlink or URL is sufficient attribution under the Creative Commons license.
Please attribute this contribution to:
Jesse Berry
You can leave this blank if this is a change so small as to not be copyrightable.

Ah, good spot! Thanks, fixed.


Focus on Editor when clicking on Timer Bar

📒 Description
This PR introduce the fix that we can click on Timer Bar to open Editor and focus on specific fields.
🕶 Types of changes
Bug fix (non-breaking change which fixes an issue)
🤯 List of changes

[x] Introduce ClickableImageView
[x] Logic for detect mouse down (same old approach) and open Editor with specific focus fields.

👫 Relationships
Closes #3245
🔎 Review hints

Start any TE
Click on Description, Project label and Tag icon on Timer bar -> Editor opens and focus on Description, Project Textfield or Tag view -> 💯

Everything works as expected. 👍


feat(tokenUnpauseTransaction): Implement TokenUnpauseTransaction E2E tests: TCK

Description:
This PR implements the TokenUnpauseTransaction tests documented in test-specifications/token-service/tokenUnpauseTransaction.md.
Related issue(s):
#283
Checklist

[x] Documented (Code comments, README, etc.)
[x] Tested (unit, integration, etc.)

Tests are passing in JS! 🎉


base-line shift super and sub not working?

According to the resvg-test-suite, both, Chrome and resvg render the following test case correctly:
<svg id="svg1" viewBox="0 0 200 200" xmlns="http://www.w3.org/2000/svg"
     font-family="Noto Sans" font-size="26">
  <title>Deeply nested `super`</title>

  <path id="crosshair" d="M 20 100 L 180 100 M 100 20 L 100 180"
        stroke="gray" stroke-width="0.5"/>

  <text id="text1" x="30" y="100">
    A
    <tspan id="tspan1" fill="green" baseline-shift="super">
      B
      <tspan id="tspan2" fill="green" baseline-shift="super">
        C
        <tspan id="tspan3" fill="green" baseline-shift="super">
          D
          <tspan id="tspan4" fill="green" baseline-shift="super">
            E
            <tspan id="tspan5" fill="green" baseline-shift="super">
              F
            </tspan>
          </tspan>
        </tspan>
      </tspan>
    </tspan>
  </text>

  <!-- image frame -->
  <rect id="frame" x="1" y="1" width="198" height="198" fill="none" stroke="black"/>
</svg>

However, in Chrome I get this result:

While in resvg, the result looks like this (ignore the dark background color):

I believe the one from Chrome is correct? In addition to that, I also get a warning Warning (in usvg_parser::svgtree:282): Failed to parse baseline-shift value: 'super'.

I believe the one from Chrome is correct?

No. Both are correct. The vertical offset used by sub/super is undefined and a renderer can use whatever we want.
resvg uses font metrics, while some apps simply hardcode those values.
Remember that SVG text is a joke and it's not possible to get reproducible output by design. You have to use PDF for this. Or simply convert text to paths.

Warning (in usvg_parser::svgtree:282): Failed to parse baseline-shift value: 'super'.

Meh... this is just a spurious warning. We're trying to parse baseline-shift as a number first and then as a string. Therefore any non-number value produces a pointless warning... Created #676

Okay, good to know. Thanks!


[cem-plugin-expanded-types]: Array not expanded

Hi there. I just noticed that specifying an array of a given type does not expanded →
type DayName = 'Monday' | 'Tuesday' | 'Wednesday' | 'Thursday' | 'Friday' | 'Saturday' | 'Sunday';

// … later specifying a property as

@property({type: Array})
weekDays: Array<DayName> = []; // → no expandedType generated

What would you expect the expanded type to be?


Optimize OptionTypeConverter

The original version used slow reflection with no caching.
This version uses optimized reflection (both caching and pre-computed readers/constructors).
Briefly verified to work manually. Assuming there are unit tests already in place for this code.

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


Move errors to FAQ

Move ARC and finalConfigure errors to FAQ
Standardize style guideline wording on ‘CORRECT’ and ‘WRONG’

i like it.
LGTM with comment

@brunobowden Good idea with this commit.


Change UI element reference formatting to bold

Our current style guidance for UI element references in the docs is contrary to both Google's and Microsoft's style guidance on how to represent UI elements in docs (i.e. bold and verbatim).
Therefore, the purpose of this PR is to update these UI element references in the docs to match standard documentation conventions used elsewhere.
Doing this has the added benefit of disambiguating key term formatting (i.e. italics) from UI element references.
Using the following regex to locate existing text between underscores:
(?:^| )[^@#\s_]*(_([^_\n]+)_)

Here's an example of what these updates will look like in the docs:

Note: This PR also took the opportunity to:

Standardise and improve other aspects of the docs to bring them more in line with our style guides (e.g. removes fancy formatting from headings and replaces unnecessary HTML markup in callouts with their Markdown equivalents.
Implement updates/fixes to anything I came across (as part of these doc updates) that were incorrect/out of date.
Also implement as many updates/fixes as possible to the old Pipeline Settings UI references, which were overloaded to refer to either the general Pipeline Settings page (of all pipelines within an org), or the Settings page for a specific pipeline.

Thanks very much @meiqimichelle , I think you captured pretty much all the main wording updates I implemented. The rest were formatting changes or minor wording clarifications (in reference to ambiguous UI elements).

Thanks very much @meiqimichelle , I might wait until tomorrow morning to see if either @mbelton-buildkite (if he's available then and I'm sure he'd be happy for this to be merged in without additional review) or @DanielOaks would like to review this first. Otherwise, I'll merge it in by midday tomorrow as it's going to start holding up other work.


Darwin(mac/OSX) has different head name of COMMAND -> COMM

in darwin, the ps output header is COMM, instead of COMMAND
to keep the consistency, should rename COMM to COMMAND in darwin platform.
$ ps -A -o comm,ppid,pid,stat
COMM              PPID   PID STAT
/sbin/launchd        0     1 Ss
/usr/libexec/Use     1    43 Ss

this bug broken my code under darwin. hope this fix will publish to npm soon.

could we fix this issue?

@zixia you could make a PR

@whitecolor thanks.
It's already in my pull request #17
please have a look over there. :)

I myself waiting for @indexzero

ping @indexzero

https://github.com/zixia/ps-tree/pull/1 couldn't one allow more maintainers on this repository?

Lets close also this @zixia
We have https://github.com/indexzero/ps-tree/pull/17


Generate documentation with Dokka

https://github.com/Kotlin/dokka

Plan:

split deployment of mkdocs to two steps: building and deploying to gh-pages branch
before deploying but after building mkdocs, build the Dokka docs and move them to the mkdocs dir
push everything to gh-pages branch, using https://github.com/JamesIves/github-pages-deploy-action

TODO: split to lib API docs and action wrappers docs, as suggested in https://kotlinlang.slack.com/archives/C02UUATR7RC/p1675413475512599?thread_ts=1675412896.761829&cid=C02UUATR7RC. To see how to do it, see https://kotlinlang.slack.com/archives/C0F4UNJET/p1675513702502919?thread_ts=1675415950.687779&cid=C0F4UNJET.


Starting with 0.22.0 package for macOS is more than double the size

It seems that in 0.22 NW.js changed what is included in the download for macOS, switching to symbolic links for some of the nested folders.
Before 0.22 this is what you normally would have in the nwjs package (actual folders with content).

In 0.22 instead of some of the folders that it used to have now it has symbolic links.

In the package for the app we end up having triple of original content.

One copy ends up in "nwjs Framework.framework", and two more copies in "nwjs Framework.framework/Versions/Current" and "nwjs Framework.framework/Versions/A" (the original content for this distro).

nwjs-builder-phoenix started to dereference symlinks when I started using lerna, which automatically creates symlinks between packages, and the test for building wouldn't work without doing it.
I don't like symlinks at all and avoid using it especially in distribution, so I post an issue on nwjs/nw.js#5873 and wait for the answer.
If it's expected for later versions of NW.js, I am going copy NW.js while keeping the symlinks unchanged, and then copy project files with dereferenced symlinks. But these symlinks should fail someone who want to build macOS apps from Windows.

If you are interested, commenting out this line should work (try finding the corresponding line in ./node_modules/nwjs-builder-phoenix/dist/lib/Builder.js).

When I discovered the issue last week I added extra step in our build process to remove content of "${name}.app/Contents/Versions/${version}/nwjs Framework.framework/Versions" after nwjs-builder-phoenix is done and before we run appdmg to create images, that way we don't have any symlinks or duplicates in our distro.
I think I'll wait for response to the ticket about intent of that change in NW.js distro and if that's here to stay then I'll test how it works without dereferencing. Or maybe some logic can be added to skip "**/nwjs Framework.framework/Versions" as part of the mentioned copy operation.

According to @rogerwang, it's an intended change. I am preparing for an update for this.

Hopefully this will be fixed in nwjs-builder-phoenix 1.10.2, please report back when you have time. I am having a slow Internet connection at the moment and can't test for it completely.

Thank you, seems to be all fine now. I was trying to test it yesterday, but was hitting issues with signature verification after install of the package downloaded from internet, wanted to make sure that signing issues were not related to use of symlinks.


[Release] Automatize source verification task submission

The workflow would look like this:

git push -u apache release-<version>
git push -u apache release-<version>-rc<rc-number>
git push -u apache apache-arrow-<version>

dev/release/02-source.sh <version> <rc-number>
dev/release/03-source-verify.sh <version> <rc-number>

Where 03-source-verify.sh would create a pull request and submit crossbow source verification tasks by either:
a. placing a github comment triggering the comment bot
b. calling crossbow locally then placing a comment to the PR using the same archery.crossbow.CommentReport class
I'd suggest to use gh cli tool for implementation.
The resulting PR should look like this https://github.com/apache/arrow/pull/12262
Opinions @kou?
Reporter: Krisztian Szucs / @kszucs
Note: This issue was originally created as ARROW-15456. Please see the migration documentation for further details.

Duplicate of #32012


Missed step when setting up an A record for SWA

In the section "Set up with an A record" and subsection "Validate ownership" after step 4 "Copy the value of the stableInboundIP property and set it aside in a text editor for future use. This is the IP address of your regional Static Web Apps host." there should be step to go to the custom domain section, that I think got missed, before moving to step 5 "Select + Add."

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

ID: ff29b66e-ca9b-11f6-e79f-a3b4fc722e65
Version Independent ID: 37abf881-e871-e577-ab1d-1d7b404ca943
Content: Set up an apex domain in Azure Static Web Apps
Content Source: articles/static-web-apps/apex-domain-external.md
Service: static-web-apps
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

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

rishabkumar7, We have raised a PR, and the published changes would reflect soon in the doc. Thanks for taking time to share your feedback.

Just a question @AjayKumar-MSFT, documentation is open-source, right?
I can contribute in future feedback and bug reports?

Rishabkumar7, Yes, You may contribute to Azure doc. This guide covers some general topics related to contribution and refers to the contributor guide for more detailed explanations when required.
https://github.com/MicrosoftDocs/azure-docs/blob/main/CONTRIBUTING.md
Thanks for your interest and taking the time to contribute to the Microsoft Azure docs.


connection_made method didn't invoked

the connection_made method of class ProtocolConsumer didn't invoked when the connection is made. such as - telnet localhost 8060

Where is the code and the test case to reproduce your issue!?

the code from the echo server examples:
https://github.com/quantmind/pulsar/blob/master/examples/echo/manage.py
and i add the connection_made method for class EchoProtocol as follows:

class EchoProtocol(pulsar.ProtocolConsumer):
'''An echo :class:~.ProtocolConsumer for client and servers.
The only difference between client and server is the implementation
of the :meth:`response` method.
'''
separator = b'\r\n\r\n'
'''A separator for messages.'''
buffer = b''
'''The buffer for long messages'''

def connection_made(self, connection):
    self.logger.info('connected')

when i run the server and telnet from another terminal, when connected, the connection_made method do noting!

The connection_made in the ProtocolConsumer is fired when the Connection (the protocol) creates a new consumer, i.e. when it needs to build the consumer to consume data received from the connection transport.
Therefore, simply connecting won't fire and connection_made event on the ProtocolConsumer but it will fire a connection_made event on the Connection.
Try sending a message and you should see the event fired.
To have connection_made event on the Connection, you need to add handler to it rather than the ProtocolConsumer.

you mean class EchoProtocol(pulsar.async.events.EventHandler): ?

when i use class EchoProtocol(asyncio.Protocol): , it still does not work:(, can you give some examples for this ?

There was a small bug which I've fixed.
I've updated the example, it should display the message on connection_made event

Yes, the updated example works well, but how can i do connection_made and connection_lost in ProtocolConsumer not outside?


Annotation processor fails to generate custom many-to-many join table entity

My data model has an entity A related many-to-many* to an entity B.
@Entity
public class AbstractA {

    @Key
    protected long id;

    @ManyToMany
    @JunctionTable
    protected List<AbstractB> bList;

}

@Entity
public class AbstractB {

    @Key
    protected long id;

}

I would like to specify a custom type in the @JunctionTable, becase I would like to customize the join table.
So I started by copying the default join table generated by requery in my sources (changing just its name), but the annotation processor fails to generate it.
@Entity(
        model = "default",
        stateless = false
)
@Table(
        name = "A_B"
)
public abstract class AbstractAB implements Serializable {
    @ForeignKey(
            delete = ReferentialAction.CASCADE,
            update = ReferentialAction.CASCADE,
            references = A.class
    )
    @Key
    protected Long AId;

    @ForeignKey(
            delete = ReferentialAction.CASCADE,
            update = ReferentialAction.CASCADE,
            references = B.class
    )
    @Key
    protected Long BId;
}

How can I solve this?

Have you tried setting @JunctionTable(type= AbstractAB.class) ? that should allow you to use your custom definition for the table.

Yes, but the code generation fails.
I'll prepare a sample project rosso reproduce the issue and let YouTube
know.
Il 28 nov 2017 08:49, "Nikhil Purushe"<EMAIL_ADDRESS>ha
scritto:

Have you tried setting @JunctionTable(type= AbstractAB.class) ? that
should allow you to use your custom definition for the table.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/requery/requery/issues/707#issuecomment-347440879,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABFlcxQyHHwET8vAzIl1MAyicC9hhnO_ks5s67sJgaJpZM4P3B1G
.


Include $LORDS to the website

Describe the feature you'd like to request
There is currently no trace of $LORDS mentioned on our website. I believe we should add it as well as few links on where to trade it
Describe the solution you'd like to see
Perhaps the price can be added just below the Realms.World logo on the left menu
Additional information
No response

A few useful links, the main ones-
AVNU TRADING (L2): https://app.avnu.fi/en?tokenFrom=0x124aeb495b947201f5fac96fd1138e326ad86195b98df6dec9009158a533b49&tokenTo=0x49d36570d4e46f48e99674bd3fcc84644ddd6b96f7c741b1562b82f9e004dc7&amount=10000
EKUBO POOL: https://app.ekubo.org/charts/LORDS/ETH
L1 TRADING: https://dexscreener.com/ethereum/0x5104846dc1c3c7697b856dc69e350ae4dcc47e24


[release-4.13] OCPBUGS-27534,OCPBUGS-27554,OCPBUGS-27559,OCPBUGS-27619,OCPBUGS-27639,OCPBUGS-27644: bump github.com/go-git/go-git/v5 to v5.11.0

Description of the change:
This is a cherry-pick of https://github.com/openshift/ocp-release-operator-sdk/pull/364
Motivation for the change:
Addresses the following CVEs:
https://github.com/advisories/GHSA-mw99-9chc-xw7r
https://github.com/advisories/GHSA-449p-3h89-pw88
Checklist
If the pull request includes user-facing changes, extra documentation is required:

[ ] Add a new changelog fragment in changelog/fragments (see changelog/fragments/00-template.yaml)
[ ] Add or update relevant sections of the docs website in website/content/en/docs

/label backport-risk-assessed

https://mastern-jenkins-csb-openshift-qe.apps.ocp-c1.prod.psi.redhat.com/job/ocp-common/job/ginkgo-test-vm/52846/testReport/
pre merge testing

/label cherry-pick-approved


Updating references to repo: s/toolkit/gapic-generator/

Codecov Report

Merging #1930 into master will increase coverage by 0.16%.
The diff coverage is 93.5%.

@@             Coverage Diff              @@
##             master    #1930      +/-   ##
============================================
+ Coverage     85.73%   85.89%   +0.16%
- Complexity     4792     4890      +98
============================================
  Files           441      451      +10
  Lines         19199    19533     +334
  Branches       2009     2043      +34
============================================
+ Hits          16460    16778     +318
- Misses         2052     2062      +10
- Partials        687      693       +6

Impacted Files
Coverage Δ
Complexity Δ

...oogle/api/codegen/viewmodel/StaticLangApiView.java
100% <ø> (ø)
2 <0> (ø)
:arrow_down:

...gle/api/codegen/viewmodel/DynamicLangXApiView.java
81.81% <ø> (ø)
13 <0> (ø)
:arrow_down:

...va/com/google/api/codegen/config/MethodConfig.java
73.58% <ø> (ø)
27 <0> (ø)
:arrow_down:

...ain/java/com/google/api/codegen/util/TypeName.java
88.63% <0%> (-11.37%)
15 <0> (ø)

...com/google/api/codegen/configgen/ConfigHelper.java
36.84% <0%> (ø)
4 <0> (ø)
:arrow_down:

...e/api/codegen/gapic/PackageNameCodePathMapper.java
66.66% <0%> (-33.34%)
2 <0> (ø)

...google/api/codegen/viewmodel/MethodSampleView.java
100% <100%> (ø)
2 <2> (?)

...google/api/codegen/viewmodel/ClientMethodType.java
100% <100%> (ø)
1 <0> (ø)
:arrow_down:

...m/google/api/codegen/config/GapicMethodConfig.java
66% <100%> (+0.34%)
17 <0> (ø)
:arrow_down:

...egen/transformer/py/PythonMethodViewGenerator.java
100% <100%> (ø)
3 <3> (?)

... and 56 more

Continue to review full report at Codecov.

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


Extract fails when a filesize of 0 bytes is encountered.

Same for Parse. Tested by writing white space to empty files and Parse and Extract worked as expected. Function stops at first encounter.

The problem seems to be in the Pullstream lib: https://github.com/EvanOxfeld/node-pullstream/blob/master/pullstream.js#L29
changing that to if (typeof len === 'undefined') { fixes it on our end

:+1:  Thanks @austinhallock . Solved my issue :)

I've already created pull request for pullstream ... hope they will merge it soon.

+1 thanks @austinhallock . You solved my issue.


⚠ Agora Aveiro has degraded performance

In c14227e, Agora Aveiro (https://agoraaveiro.org) experienced degraded performance:

HTTP code: 200
Response time: 6419 ms

Resolved: Agora Aveiro performance has improved in 28671be after 10 minutes.


remove unused GHCJS Git submodule

i.i.r.c. I created this Git submodule to use with Travis. It's unused by Travis and I don't think it's used anywhere else.

Thanks!  Merged


Clarify how to properly test compliance?

I am new to Fantasy-Land and algebraic types in general, so please correct me if any of the following is incorrect or off-base.
One of the nicest things a specification can do is provide a compliance suite for implementers to confirm their work, if at all possible. As far as I can see, FL doesn't quite include such a suite.
This repo includes a set of namespaced laws in curried function form (not all of the same type signature), which return booleans. That seems somewhat on the way towards a compliance suite, and indeed FL's own test.js appears to check that the laws hold for its internal/example Id and Sum types.
It would seem to be possible for any implementer to import those laws in their own project and directly invoke the functions with correct arguments (as deduced from reading the source code) — which is perhaps the intent of the authors of this project. However, I wonder if it is not possible to provide something more cohesive, documented, straightforward, and foolproof?
For example, the Promises/A+ organization provides a compliance suite which can be run programmatically using a thin adapter module. The implementer does not need to know how to properly test their Promise library, just how to bridge their API to the expected API.
Is there any reason why FL cannot or would not follow suit (no wordplay intended)? I am just curious.

Will fantasyland/fantasy-laws#1 get us some of the way there, @glebec?

@davidchambers — ah, I didn't notice that PR.
It's definitely a step in that direction especially assuming that the README outlines how developers can use the law functions to confirm their libraries work correctly.
One thing to note is that the laws are not the totality of the spec, considering the spec requires additional constraints (e.g. "A value that implements the Ord specification must also implement the Setoid specification").
If a more integrated and thorough spec built on top of the laws sounds like something we'd like to add to Fantasy-Land, I'd be interested in taking a crack at it. I don't know if that would live in this repo, the Fantasy-Laws repo, or a new repo; I also don't know if we'd have preferences in terms of testing framework etc. I'm happy to hear your opinions.

One thing to note is that the laws are not the totality of the spec, considering the spec requires additional constraints (e.g. "A value that implements the Ord specification must also implement the Setoid specification").

Good point. This is actually encoded in sanctuary-type-classes. For example, this is the definition of Z.Monoid:
//# Monoid :: TypeClass
//.
//. `TypeClass` value for [Monoid][].
//.
//. ```javascript
//. > Monoid.test('')
//. true
//.
//. > Monoid.test(0)
//. false
//. ```
var Monoid = $('Monoid', [Semigroup], {empty: Constructor});

I've added an item to the to-do list in fantasyland/fantasy-laws#1:

Verify dependencies as discussed in fantasyland/fantasy-land#243

@glebec, fantasy-laws is now available via npm. Can this issue be closed, do you think?

Given that I'm not actively looking at this project in any case, from an issue author perspective I am certainly okay with closing the issue. As to whether fantasy-laws satisfies the goals stated above, at first glance it looks sufficient, though I haven't examined it exhaustively.


feature_request.yaml for vueuse?

Describe the bug
The feature request.yaml template says Propose a new feature to be added to VueUse lol
Reproduction
f
System Info
f

Used Package Manager
pnpm
Validations

[X] Have tested with the latest version. This is still alpha version and sometime things change rapidly.
[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

Still a missed reference. I believe on line 13


FAQ and Quickstart Guide.

I've added a FAQ (Only four questions for now, but as more come along they can be added) along with a reasonably detailed Quickstart Guide. I also reformatted the pages to fit with markdown best practice.
I'm pretty sure I covered everything in that guide, but if I missed anything let me know.
I haven't done any changes to make the new page appear because I'm not sure if it's automatic or not. If it's not, you'll probably have to do it on your end.

Looks pretty good, minor typos notwithstanding. I'll merge the PR and make some fixes on my end.
The wiki has its own workflow to update pages automatically when the wiki folder changes.


Vars overridable playbook level

I'd like to override the variables used in this particular role when invoking it from a playbook.

Aren't currently overridable 🤔 ? We use this role in our playbooks and we override the vars:
        - name: install agent via newrelic-cli
          include_role:
            name: newrelic_cli
          vars:
            repo_endpoint: "https://download.newrelic.com/"
            recipe: "newrelic-infra"
            region: "INVALID_REGION_WILL_MAKE_IT_FAIL"
            nr_api_key: "user-api-key"
            nr_account_id: "nr-account-id"

Aren't currently overridable 🤔 ? We use this role in our playbooks and we override the vars:
        - name: install agent via newrelic-cli
          include_role:
            name: newrelic_cli
          vars:
            repo_endpoint: "https://download.newrelic.com/"
            recipe: "newrelic-infra"
            region: "INVALID_REGION_WILL_MAKE_IT_FAIL"
            nr_api_key: "user-api-key"
            nr_account_id: "nr-account-id"

When using it as role in roles vars won't get overridden;
- name: Install NRIA
  hosts: linux
  become: true
  gather_facts: yes
  vars:
    repo_endpoint: "https://nr-downloads-main.s3.amazonaws.com/"
    recipe: "newrelic-infra"
    region: "{{ lookup('ansible.builtin.env', 'NEW_RELIC_REGION') }}" # US | EU | STAGING
    nr_api_key: "{{ lookup('ansible.builtin.env', 'NEW_RELIC_API_KEY') }}"
    nr_account_id: "{{ lookup('ansible.builtin.env', 'NEW_RELIC_ACCOUNT_ID') }}"
  roles:
    - caos.ansible_roles.newrelic_cli

It seems like using include_role is able to override a role var. Playbook variables can't ovveride var roles, that is why I'd need them in defaults/
https://github.com/newrelic-experimental/caos-ansible-roles/pull/70#issuecomment-1968709820

Ansible does apply variable precedence, and you might have a use for it. Here is the order of precedence from least to greatest (the last listed variables override all other variables):

It seems like using include_role is able to override a role var. Playbook variables can't ovveride var roles, that is why I'd need them in defaults/ #70 (comment)

Ansible does apply variable precedence, and you might have a use for it. Here is the order of precedence from least to greatest (the last listed variables override all other variables):

Didn't know this, thanks for the explanation :) ✅


Force single data dir for scrub tests

This is for CASSANDRA-11693.
Since range-aware JBOD is introduced, there is a chance that zero byte SSTable is created (not actually on the disk, just internally) and generation number goes out of sync with what tests expect.
I propose to simply disable JBOD configuration for scrub tests, since it is not the target of the tests anyway.

The change looks fine, +1. If you're happy with the behavior of C* here, which doesn't sound too unreasonable.


Generate human-readable page with packages installed in user image

You can inspect images/user to get a sense of what packages are installed
in your user environment. This is clunky, and not as user friendly as I would
like.
Instead, I want us to auto-publish a page with the following information each
time we push an image:

List of python packages (conda list & pip list)
List of apt packages (apt list --installed)
List of R packages (installed.packages())

This lets users see what's already installed at a glance, and not ask for
libraries we might already have installed

could a first step here to be parse that repository's config files to auto-generate a page like this? That's something I could try coding up quickly rather than relying on querying a deployed hub (also this will change if/when a hub customizes its own environment so having a centralized docs source won't be as useful then)

@choldgraf sounds like a great first step! But probably not high priority...

Also we should totally write a similar plugin that uses jupyter book PDF conversion under the hood as well. That way people could write notebooks with jupyter book syntax and render them as single-page PDFs on the fly


Rename and refactor ScrollView hierarchy

This prepares us for a CustomScrollView that takes slivers.

... but you should add control tests for the new "custom" named constructors.


[HUDI-5913] Table can not read correctly when computed column is in the midst

Change Logs
The computed column(eg : process_time as proctime() ) will cause a select exception because it is not included in the data file schema, so we need to ignore the computed column
Problem
create table if not exists ods_test_hudi (
    `id` int,
    `name` string,
    `create_date` timestamp(3),
    `process_time` AS PROCTIME(),
    PRIMARY KEY(`id`) NOT ENFORCED
)
WITH (
    'connector'='hudi',
    'table.type'='MERGE_ON_READ'
)
env.sqlQuery("select * from ods_test_hudi /*+ OPTIONS('read.start-commit'='earliest')*/").execute().print();

exception:

 Caused by: java.lang.ArrayIndexOutOfBoundsException: -1
	at org.apache.hudi.table.format.cow.ParquetSplitReaderUtil.lambda$genPartColumnarRowReader$0(ParquetSplitReaderUtil.java:118)
	at java.util.stream.IntPipeline$4$1.accept(IntPipeline.java:250)
	at java.util.Spliterators$IntArraySpliterator.forEachRemaining(Spliterators.java:1032)
	at java.util.Spliterator$OfInt.forEachRemaining(Spliterator.java:693)
	at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:481)
	at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:471)
	at java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:708)
	at java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
	at java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:499)
	at org.apache.hudi.table.format.cow.ParquetSplitReaderUtil.genPartColumnarRowReader(ParquetSplitReaderUtil.java:120)

Risk level (write none, low medium or high below)
none
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

The failed test case should not be caused by this patch: https://dev.azure.com/apache-hudi-ci-org/apache-hudi-ci/_build/results?buildId=15661&view=logs&j=3b6e910d-b98f-5de6-b9cb-1e5ff571f5de&t=30b5aae4-0ea0-5566-42d0-febf71a7061a&l=321013


Suggestions for the .NET Core 2.2 Preview 1

Suggestion :

The page1 an page2 navbar links should link to real pages and they should be about and contact, That will be helpful to demonstrate the routing, and how you link to other pages.
For better UI consistency the page1 and page2 navbar links should be aligned to the right (in mvc project)
Change the url from https://example.com/?action=Index&controller=Home to htps://example.com/Index or Home to make it SEO friendly
Add button bootstrap class to the counter button

This issue was moved to aspnet/Templating#712


Support groovy compilation

Annotation processing works with Groovy compilation now.
It's be great if this plugin supported this.

Do you think the plugin should automatically set compileGroovy.groovyOptions.javaAnnotationProcessing to true? Or only configure the compileGroovy and compileTestGroovy tasks?

I must admit that I am new to Gradle.
Gut feeling from looking at your source is that the compile*Groovy tasks need configuring.
Just to give you some context I noticed this issue when trying to use the plugin on a mixed mode groovy/java project. Without the plugin things actually compiled OK using gradle, but Idea is broken. With the plugin, Idea is OK, but won't compile with gradle.

Taking a look at the groovy compiler I am not even sure this is possible.
Groovyc doesn't seem to have a -s option.
http://groovy-lang.org/groovyc.html

Annotation processing for Groovy involves generating Java "stubs" that are then processed by JavaC.

So possibly the -Fflag option could be used?

Actually, we don't even need to think about it, Gradle (apparently) already handles everything correctly; the only question really is: should compileGroovy.groovyOptions.javaAnnotationProcessing be forcefully set to true or left to its default value? (I lean towards the latter)

For me at least this didn't work.
compileGroovy {
        groovyOptions.javaAnnotationProcessing = true
    }

produced no generated source code and the compilation failed.
Using:
compileGroovy {
        groovyOptions.javaAnnotationProcessing = true
        options.compilerArgs += [
            "-s", "${project.buildDir}/generated/source/apt",
        ]
    }

did work, but obviously the source is not actually generated in the correct location (needs to be split in to main and test). Also the target directory needs to be created by hand.

Should be fixed in e930273ec6bd3fbdca253319ad1289653e89973c but I haven't actually tested IDE integration. Will cut a release shortly.

v0.6 released: https://plugins.gradle.org/plugin/net.ltgt.apt/0.6


Download failure

Hi Ego4d team,
While I am downloading the dataset and have almost finished all of them, I suddenly encounter error botocore.exceptions.EndpointConnectionError: Could not connect to the endpoint URL: "https://ego4d-consortium-sharing.s3.amazonaws.com/public/v1/ego4d.json".
Is it because of the aws s3 issue? Could you please have a check?

Seems to be resolved after while


[v2.8] Bump k8s.io/kubernetes to v1.28.6

What this PR does / why we need it:

Which issue(s) this PR fixes
Issue #
Special notes for your reviewer:
Checklist:

[ ] squashed commits into logical changes
[ ] includes documentation
[ ] adds unit tests
[ ] adds or updates e2e tests
[ ] backport needed

Looks like this PR needs a rebase.


git 1.2.0-release branch compile is error  !!!!!!!

1.2.0-release
I want to add my sms to Aert-moudle   add is ok ,i  want to compile all project in idea  for:
mvn -U clean package -Prelease -Dmaven.test.skip=true
I add my maven settings mirror  for:

alimaven
central
aliyun maven
https://maven.aliyun.com/nexus/content/repositories/central/

    <!-- 中央仓库1 -->
    <mirror>
        <id>repo1</id>
         <mirrorOf>central</mirrorOf>
        <name>Human Readable Name for this Mirror.</name>
        <url>http://repo1.maven.org/maven2/</url>
     </mirror>

     <!-- 中央仓库2 -->
     <mirror>
         <id>repo2</id>
         <mirrorOf>central</mirrorOf>
         <name>Human Readable Name for this Mirror.</name>
         <url>http://repo2.maven.org/maven2/</url>
     </mirror>

but  make it  is error
error message is :
[INFO] Scanning for projects...
Downloading from nexus: http://repo.maven.apache.org/maven2/org/apache/apache/21/apache-21.pom
[ERROR] [ERROR] Some problems were encountered while processing the POMs:
[FATAL] Non-resolvable parent POM for org.apache.dolphinscheduler:dolphinscheduler:1.2.1-SNAPSHOT: Could not transfer artifact org.apache:apache:pom:21 from/to nexus (http://repo.maven.apache.org/maven2): Transfer failed for http://repo.maven.apache.org/maven2/org/
apache/apache/21/apache-21.pom 501 HTTPS Required and 'parent.relativePath' points at wrong local POM @ line 49, column 10
my dependencies  is all ok ,why it not compile ???????

my add is it ,  but it not compile

    <mirror>
        <id>alimaven</id>
       <mirrorOf>central</mirrorOf>
        <name>aliyun maven</name>
        <url>https://maven.aliyun.com/nexus/content/repositories/central/</url>
     </mirror>

    <!-- 中央仓库1 -->
    <mirror>
        <id>repo1</id>
         <mirrorOf>central</mirrorOf>
        <name>Human Readable Name for this Mirror.</name>
        <url>http://repo1.maven.org/maven2/</url>
     </mirror>

     <!-- 中央仓库2 -->
     <mirror>
         <id>repo2</id>
         <mirrorOf>central</mirrorOf>
         <name>Human Readable Name for this Mirror.</name>
         <url>http://repo2.maven.org/maven2/</url>
     </mirror>

Effective January 15, 2020, The Central Repository no longer supports insecure communication over plain HTTP and requires that all requests to the repository are encrypted over HTTPS.
If you're receiving this error, then you need to replace all URL references to Maven Central with their canonical HTTPS counterparts:
Replace http://repo1.maven.org/maven2/ with https://repo1.maven.org/maven2/
Replace http://repo.maven.apache.org/maven2/ with https://repo.maven.apache.org/maven2/
If for any reason your environment cannot support HTTPS, you have the option of using our dedicated insecure endpoint at http://insecure.repo1.maven.org/maven2/
referer: https://support.sonatype.com/hc/en-us/articles/360041287334

whether your problem has been solved?


Added manpage for git-gud

Fixed #144
Wrote and added a manpage for git-gud. Will possibly submit another PR soon to add this to the setup.py script, so it is installed in the correct man directory.

Can you push your updates?

I made the requested changes and just pushed them.


SSL Certificate problem

Guzzle version(s) affected: x.y.z
Description

How to reproduce

Possible Solution

Additional context

Looks like nothing is here


adding the rect3d primitive

Objective
Fixes #12388
Solution

Removing the plane3d and adding rect3d primitive mesh

Nice work! Need to rename the existing Rectangle primitive to Rect2d I think. Thoughts on adding dir2 to Rect3d? With transforms it's in for a penny in for a pound.

So in the end the decision was to use Plane3d for a finite and InfinitePlane3d for an infinite plane, so all the confusing naming convention with the rectagnle is gone now :)

This should be ready for final review

I suspect this is the base of the upcoming work to  have infinite plane meshes?

Yes, I'll try to review this soon. I've had less time for reviews lately.

Thanks for reviewing! I have resolved the merge conflict as well, just awaiting your thoughts on my replies! :)

This should be ready for merge @alice-i-cecile

I think we need another approval first. Maybe @Jondolf can give it a look?

I'm going to go ahead and mark this as ready for final review, given Alice's conditional approval has been addressed.


Read parameter values from parent classes in configuration files

Description
Fix #2864
Currently, during instantiation of a task, values of parameters are only checked in the following order:

Positional arguments
Keywords arguments
defaults of task

How I changed this:

Positional arguments
Keywords arguments
defaults of task
defaults of parents task

Have you tested this? If so, how?
"I ran my jobs with this code and it works for me.

As I commented in the issue, I'm not sure what you're doing there is something we see as a valid use case. At least not the way I understood it.
But I can see why one would want this too. Regardless this change would eventually require tests and docs. I'm also thinking if one should have this behavior configurable or not.
By the way by defaults of task you meant defaults of task (based on what's read in config) right?

As I commented in the issue, I'm not sure what you're doing there is something we see as a valid use case. At least not the way I understood it.

In what is this not a "valid" use case?

But I can see why one would want this too. Regardless this change would eventually require tests and docs. I'm also thinking if one should have this behavior configurable or not.

Configurable? What do you mean?

By the way by defaults of task you meant defaults of task (based on what's read in config) right?

Yes, exactly!

Unfortunately, I am unable to give this PR a go. It still doesn't solve the issue I reported in this comment.


One tail vs two tailed GO term

Added an option in the cli health to pick if you want to include negative Z-scores or not when looking at  go-term enrichment

I'll merge this once I have tests passing again. We fell behind a bit on adding tests :frowning_face:

You want this closed?

I just submitted a new one with the fixes..my bad
On Tue, May 16, 2017 at 9:00 PM, Rob Schaefer<EMAIL_ADDRESS>wrote:

You want this closed?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/schae234/Camoco/pull/41#issuecomment-301964108, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ANr0AA4zaFH4NMkDvcO7xAbm7_qB1smyks5r6lTSgaJpZM4KHwGf
.

--
Jean-Michel Michno (Meesh)
Ph.D. Candidate
Bioinformatics and Computational Biology
University of Minnesota
<EMAIL_ADDRESS>952-212-6188


Pass data as such for post request when uploading files

For multipart form data Requests handles the encoding of data.

@bulkan do you have any review comments?


first pass at builder-base install of EKS-D built go 1.15.15

Issue #, if available:
Description of changes:
A conversation starter about how we want to perform the install of the patched 1.15 golang version from the RPMs we've generated.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/retest

/test builder-base-tooling-presubmit-2022

hold until following prod 1.24 release

superseded by https://github.com/aws/eks-distro-build-tooling/pull/494