use this plugin in combination with serverless lambda docker functionality

Hello,
In my  project I'm using functions that are built using a docker container, using the build in serverless functionality, but I also want to create non docker functions with the serverless-python-requirements plugin. The serverless-python-requirements tries to create a zip for each function, which is something I want to prevent for the docker container ones. Is there a way to exclude a function from the functionality from this plugin or turn the plugin off for a function?
My docker function definition looks like this

I have tried this too and couldn't find any way to ignore python-requirements module or specify requirements file when deploying a lambda with ecr image.  Please provide a fix, it would be greatly appreciated.

I've just run into this issue myself, I would also like this plugin to ignore certain function definitions in the serverless file.

Hey @araker - could you please post the output/error that you're getting when running deploy/package?

If it's helpful, when I include a lambda container function in the serverless.yml file that also has the plugin declared I get the following error:
TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string or an instance of Buffer or URL. Received undefined
      at readFile (fs.js:343:10)
      at go$readFile (/home/ubuntu/.nvm/versions/node/v14.17.0/lib/node_modules/serverless/node_modules/graceful-fs/graceful-fs.js:118:14)
      at readFile (/home/ubuntu/.nvm/versions/node/v14.17.0/lib/node_modules/serverless/node_modules/graceful-fs/graceful-fs.js:115:12)
      at go$readFile (/home/ubuntu/tmp/jw-met-proj-sls/node_modules/graceful-fs/graceful-fs.js:118:14)
      at Object.readFile (/home/ubuntu/tmp/jw-met-proj-sls/node_modules/graceful-fs/graceful-fs.js:115:12)
      at Object.readFile (/home/ubuntu/tmp/jw-met-proj-sls/node_modules/universalify/index.js:5:57)
      at Object.tryCatcher (/home/ubuntu/.nvm/versions/node/v14.17.0/lib/node_modules/serverless/node_modules/bluebird/js/release/util.js:16:23)
      at Object.ret [as readFileAsync] (eval at makeNodePromisifiedEval (/home/ubuntu/.nvm/versions/node/v14.17.0/lib/node_modules/serverless/node_modules/bluebird/js/release/promisify.js:184:12), <anonymous>:14:23)
      at injectRequirements (/home/ubuntu/tmp/jw-met-proj-sls/node_modules/serverless-python-requirements/lib/inject.js:23:6)
      at /home/ubuntu/tmp/jw-met-proj-sls/node_modules/serverless-python-requirements/lib/inject.js:128:13
      at tryCatcher (/home/ubuntu/tmp/jw-met-proj-sls/node_modules/bluebird/js/release/util.js:16:23)
      at MappingPromiseArray._promiseFulfilled (/home/ubuntu/tmp/jw-met-proj-sls/node_modules/bluebird/js/release/map.js:68:38)
      at MappingPromiseArray.PromiseArray._iterate (/home/ubuntu/tmp/jw-met-proj-sls/node_modules/bluebird/js/release/promise_array.js:115:31)
      at MappingPromiseArray.init (/home/ubuntu/tmp/jw-met-proj-sls/node_modules/bluebird/js/release/promise_array.js:79:10)
      at Promise._settlePromise (/home/ubuntu/tmp/jw-met-proj-sls/node_modules/bluebird/js/release/promise.js:601:21)
      at Promise._settlePromise0 (/home/ubuntu/tmp/jw-met-proj-sls/node_modules/bluebird/js/release/promise.js:649:10)
      at Promise._settlePromises (/home/ubuntu/tmp/jw-met-proj-sls/node_modules/bluebird/js/release/promise.js:729:18)
      at Promise._fulfill (/home/ubuntu/tmp/jw-met-proj-sls/node_modules/bluebird/js/release/promise.js:673:18)
      at MappingPromiseArray.PromiseArray._resolve (/home/ubuntu/tmp/jw-met-proj-sls/node_modules/bluebird/js/release/promise_array.js:127:19)
      at MappingPromiseArray._promiseFulfilled (/home/ubuntu/tmp/jw-met-proj-sls/node_modules/bluebird/js/release/map.js:108:18)
      at Promise._settlePromise (/home/ubuntu/tmp/jw-met-proj-sls/node_modules/bluebird/js/release/promise.js:609:26)
      at Promise._settlePromise0 (/home/ubuntu/tmp/jw-met-proj-sls/node_modules/bluebird/js/release/promise.js:649:10)
      at Promise._settlePromises (/home/ubuntu/tmp/jw-met-proj-sls/node_modules/bluebird/js/release/promise.js:729:18)
      at _drainQueueStep (/home/ubuntu/tmp/jw-met-proj-sls/node_modules/bluebird/js/release/async.js:93:12)
      at _drainQueue (/home/ubuntu/tmp/jw-met-proj-sls/node_modules/bluebird/js/release/async.js:86:9)
      at Async._drainQueues (/home/ubuntu/tmp/jw-met-proj-sls/node_modules/bluebird/js/release/async.js:102:5)
      at Immediate.Async.drainQueues [as _onImmediate] (/home/ubuntu/tmp/jw-met-proj-sls/node_modules/bluebird/js/release/async.js:15:14)
      at processImmediate (internal/timers.js:462:21)

Thank you @davehemming :bow: For now the workaround is to dpeloy such functions as a separate stack, I'm also in the process of getting more familiar with the plugin and I'm hoping to find some time in the upcoming weeks to investigate bugs like this one.

Thank you @pgrzesik, much appreciated.  This is the workaround that I'm using currently and it's working fine.

The latest master code seems fixed this issue but the published version is still old. Could we have a new version released?


Extend cf push logs to encompass the whole error

Please provide details on the following items. Failure to do so may result in deletion of your feature request.
What's the user value of this feature request?

For example here I push a go app with an old buildpack
±  |main {1} ?:1 ✗| → cf push test_app -b go_buildpack
Pushing app test_app to org org / space space as admin...
Packaging files to upload...
Uploading files...
 150.13 KiB / 150.13 KiB [======================================================================================================================================================================================================================] 100.00% 1s

Waiting for API to complete processing files...

Staging app and tracing logs...
   Downloading go_buildpack...
   Downloaded go_buildpack
   Cell f294008c-3e15-4287-aa1c-5333b2c36d94 creating container for instance de81ee86-6314-4b3d-aa4a-fb7500abcb35
   Cell f294008c-3e15-4287-aa1c-5333b2c36d94 successfully created container for instance de81ee86-6314-4b3d-aa4a-fb7500abcb35
   Downloading app package...
   Downloading build artifacts cache...
   Downloaded app package (1004.5K)
   Downloaded build artifacts cache (156.9M)
   -----> Go Buildpack version 1.8.42
          **WARNING** buildpack version changed from 1.9.35 to 1.8.42
          **WARNING** [DEPRECATION WARNING]:
          **WARNING** Please use AppDynamics extension buildpack for Golang Application instrumentation
          **WARNING** for more details: https://docs.pivotal.io/partners/appdynamics/multibuildpack.html
   -----> Installing godep 80
          Copy [/tmp/cache/final/dependencies/ea2dee71d88cce172a78209425e7d9fa69203481c57c75b313e2408c83b759a7/godep-v80-linux-x64-cflinuxfs3-b60ac947.tgz]
   -----> Installing glide 0.13.3
          Copy [/tmp/cache/final/dependencies/e0175241cc9dda53f333835198167170cda881c2154d211230494738e0e678ae/glide-v0.13.3-linux-x64-cflinuxfs3-ef07acb5.tgz]
   -----> Installing dep 0.5.4
          Copy [/tmp/cache/final/dependencies/67686fc44a190142e7c4d969fd78f5aadbc37080e7de0595c60adbc4d3ad410e/dep-v0.5.4-linux-x64-cflinuxfs3-79b3ab9e.tgz]
   -----> Installing go 1.12.7
          Download [https://buildpacks.cloudfoundry.org/dependencies/go/go1.12.7.linux-amd64-cflinuxfs3-040d6b6f.tar.gz]
          **WARNING** go 1.12.x will no longer be available in new buildpacks released after 2020-02-01.
          See: https://golang.org/doc/devel/release.html
          **WARNING** Installing package '.' (default)
   -----> Running: go install -tags cloudfoundry -buildmode pie .
BuildpackCompileFailed - App staging failed in the buildpack compile phase
FAILED

If I access to the app's logs I get a more actionable error
   2021-09-24T22:42:28.67+0000 [STG/0] OUT -----> Running: go install -tags cloudfoundry -buildmode pie .
   2021-09-24T22:42:40.38+0000 [STG/0] ERR # golang.org/x/net/http2
   2021-09-24T22:42:40.38+0000 [STG/0] ERR vendor/golang.org/x/net/http2/client_conn_pool.go:305:6: undefined: errors.Is
   2021-09-24T22:42:40.40+0000 [STG/0] OUT        **ERROR** Unable to compile application: exit status 2
   2021-09-24T22:42:40.96+0000 [STG/0] ERR Failed to compile droplet: Failed to run finalize script: exit status 12
   2021-09-24T22:42:40.97+0000 [STG/0] OUT Exit status 223
   2021-09-24T22:42:41.12+0000 [STG/0] OUT Cell f294008c-3e15-4287-aa1c-5333b2c36d94 stopping instance de81ee86-6314-4b3d-aa4a-fb7500abcb35
   2021-09-24T22:42:41.12+0000 [STG/0] OUT Cell f294008c-3e15-4287-aa1c-5333b2c36d94 destroying container for instance de81ee86-6314-4b3d-aa4a-fb7500abcb35
   2021-09-24T22:42:41.17+0000 [API/0] ERR Failed to stage build: staging failed

Describe the solution you'd like
I would like to see more of the app's log before the cli stop reporting the logs

Thank you for your submission. If you still run into this issue with the latest CLI v8,  please open a new ticket with the necessary information mentioned in our issue template


List index out of range when running pappy --lite without SSL certificates

$ pappy --lite
could not load certificate pair: open /home/jan/.pappy/certs/server.pem: no such file or directory
Would you like to generate the certificates now?
(Y/n) y
Certificates generated to /home/jan/.pappy/certs
Be sure to add /home/jan/.pappy/certs/server.pem to your trusted CAs in your browser!
Traceback (most recent call last):
  File "/usr/bin/pappy", line 11, in <module>
    load_entry_point('pappyproxy==0.3.1', 'console_scripts', 'pappy')()
  File "/usr/lib/python3.7/site-packages/pappyproxy/pap.py", line 142, in start
    main()
  File "/usr/lib/python3.7/site-packages/pappyproxy/pap.py", line 134, in main
    interface_loop(client)
  File "/usr/lib/python3.7/site-packages/pappyproxy/console.py", line 32, in interface_loop
    cons.cmdloop()
  File "/usr/lib/python3.7/site-packages/cmd2/cmd2.py", line 3598, in cmdloop
    parser = argparse.ArgumentParser()
  File "/usr/lib/python3.7/argparse.py", line 1641, in __init__
    prog = _os.path.basename(_sys.argv[0])
IndexError: list index out of range

look like it's a duplicate of #21


fix: reenable table chart page length

SUMMARY
Bump @superset-ui/legacy-preset-chart-table to fix missing
page length support.
See https://github.com/apache-superset/superset-ui/pull/522
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
N/A
TEST PLAN
Added unit-test in superset-ui.
Manual testing:

Go to a table chart
Change page length in Customize
Make sure pagination still works

ADDITIONAL INFORMATION

[ ] Has associated issue:
[x] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #9884 into master will decrease coverage by 11.80%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #9884       +/-   ##
===========================================
- Coverage   71.24%   59.44%   -11.81%
===========================================
  Files         584      400      -184
  Lines       30789    12717    -18072
  Branches     3234     3234
===========================================
- Hits        21936     7559    -14377
+ Misses       8744     4973     -3771
- Partials      109      185       +76

Flag
Coverage Δ

#cypress
?

#javascript
59.44% <ø> (ø)

#python
?

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


Allow overriding HTTP client of a Spine Web JavaScript client.

The current version of a Spine Web client for JavaScript provides a high-level API for interacting with a Spine-based backend and carries all HTTP-related functionality. It makes tricky to:

add custom HTTP headers, e.g. "Authorization";
use non-CORS request mode;
expand client to interact with custom backend endpoints (non "/command" or "/query" ) that use similar protocol, e.i. Protobuf request/response body and the same set of auth headers;

To solve above-mentioned issues in Pure Photos we use our custom implementation of HttpClient and inject it after Spine Web client instantiation.

@YegorUdovchenko I am not sure whether this is an old issue, or a new one. But just in case: #194.

It's been released with 1.9.0.

@YegorUdovchenko I am not sure whether this is an old issue, or a new one. But just in case: #194.

Sorry, missed this update.


Issue upgrading from v0.4.0 to v0.5.0

Hi,
We're running a kops cluster with a manually installed version of v0.4.0 which has been working with no issues.
We're attempting an upgrade on to v0.5.0 (specifically v0.5.0-alpine-3.7) with the following server arguments:
      - name: aws-iam-authenticator
        image: [...]:v0.5.0-alpine-3.7
        args:
        - server
        - --config=/etc/aws-iam-authenticator/config.yaml
        - --state-dir=/var/aws-iam-authenticator
        - --generate-kubeconfig=/etc/kubernetes/aws-iam-authenticator/kubeconfig.yaml
        # only for kops (https://github.com/goodfoo/aws-iam-authenticator#kops-usage)
        - --kubeconfig-pregenerated=true
        - --backend-mode=ConfigMap,CRD,File

I added backend-mode flag so that it uses our existing configmap, prior to adding this the daemonset did not log anything past waiting for informer caches to sync (logs below).
Post-upgrade, all kubectl commands fail with: error: You must be logged in to the server (Unauthorized).
Token verification appears to work, i.e. running:

aws-iam-authenticator token -i CLUSTER
aws-iam-authenticator verifiy -i CLUSTER -t TOKEN

The daemonset's logs have the below, with the final log line repeating:
time="2020-02-04T12:24:06Z" level=info msg="creating event broadcaster"
time="2020-02-04T12:24:06Z" level=info msg="setting up event handlers"
time="2020-02-04T12:24:06Z" level=info msg="starting mapper \"ConfigMap\""
time="2020-02-04T12:24:06Z" level=info msg="starting mapper \"CRD\""
time="2020-02-04T12:24:06Z" level=info msg="starting aws iam authenticator controller"
time="2020-02-04T12:24:06Z" level=info msg="waiting for informer caches to sync"
time="2020-02-04T12:24:06Z" level=warning msg="Unable to re-establish watch.  Sleeping for 5 seconds"
time="2020-02-04T12:24:11Z" level=warning msg="Unable to re-establish watch.  Sleeping for 5 seconds"
time="2020-02-04T12:24:16Z" level=warning msg="Unable to re-establish watch.  Sleeping for 5 seconds"
time="2020-02-04T12:24:21Z" level=warning msg="Unable to re-establish watch.  Sleeping for 5 seconds"
time="2020-02-04T12:24:26Z" level=warning msg="Unable to re-establish watch.  Sleeping for 5 seconds"
time="2020-02-04T12:24:31Z" level=warning msg="Unable to re-establish watch.  Sleeping for 5 seconds"
time="2020-02-04T12:24:36Z" level=warning msg="Unable to re-establish watch.  Sleeping for 5 seconds"
time="2020-02-04T12:24:41Z" level=warning msg="Unable to re-establish watch.  Sleeping for 5 seconds"

Are we missing an upgrade step? I've tried regenerating the certs that kops uses and rolling the masters but nothing seems to help.

/assign @wongma7

@chinthakagodawita I think what you want is File,CRD. When you say "existing configmap", do you mean the configmap that's mounted into the pod? Because mode ConfigMap refers to the EKS method of watching a ConfigMap which has only ever existed in EKS whereas File refers to the statically mounted file (which is also a configmap...not the best naming.) The documentation isn't very explicit about what the values for this flag mean exactly, let me fix it https://github.com/kubernetes-sigs/aws-iam-authenticator#configure-iamidentitymapping-custom-resource-definitions
However, either way you should not be getting Unauthorized errors because it's supposed to fall back in the order you specified. Do you have more logs from the authenticator showing 'access denied' errors

@wongma7 That might be it. I should be able to give it another test in the next week or two, will update you once I know more.

I'm working on upgrading Kops' integration with aws-iam-authenticator and am running into a similar issue. the new --backend-mode flag description states:

The first one that returns a matching mapping wins

My cluster does not have an EKS ConfigMap or any IAMIdentityMapping resources, only the ConfigMap volume mount like what was required before 0.5.0. Without specifying --backend-mode, aws-iam-authenticator chooses to use the CRD mapping and subsequently doesn't detect any mappings, so all authn fails. I have to explicitly set --backend-mode=File in order to preserve the pre-0.5.0 behavior. This makes me think the flag's description is not accurate, or we could improve the logic of how each backend mode is initialized and used.
Logs without --backend-mode=File:
time="2020-02-28T13:22:29Z" level=info msg="creating event broadcaster"
time="2020-02-28T13:22:29Z" level=info msg="setting up event handlers"
time="2020-02-28T13:22:29Z" level=info msg="starting mapper \"CRD\""
time="2020-02-28T13:22:29Z" level=info msg="starting aws iam authenticator controller"
time="2020-02-28T13:22:29Z" level=info msg="waiting for informer caches to sync"

and then no further logs from any of the failed authn requests.
Logs with --backend-mode=File:
time="2020-02-28T13:24:29Z" level=info msg="mapping IAM role" groups="[foo]" role="arn:aws:iam::000000000000:role/foo" username="foo:{{SessionName}}"
time="2020-02-28T13:24:29Z" level=info msg="loaded existing keypair" certPath=/var/aws-iam-authenticator/cert.pem keyPath=/var/aws-iam-authenticator/key.pem
time="2020-02-28T13:24:29Z" level=info msg="listening on https://<IP_ADDRESS>:21362/authenticate"
time="2020-02-28T13:24:29Z" level=info msg="reconfigure your apiserver with `--authentication-token-webhook-config-file=/etc/kubernetes/aws-iam-authenticator/kubeconfig.yaml` to enable (assuming default hostPath mounts)"
time="2020-02-28T13:25:04Z" level=info msg="STS response" accesskeyid=xxx accountid=000000000000 arn="arn:aws:sts::000000000000:assumed-role/foo" client="<IP_ADDRESS>:60230" method=POST path=/authenticate session=foo userid=xxx
time="2020-02-28T13:25:04Z" level=info msg="access granted" arn="arn:aws:iam::000000000000:role/foo" client="<IP_ADDRESS>:60230" groups="[foo]" method=POST path=/authenticate uid="aws-iam-authenticator:000000000000:xxx" username="foo"

Is this expected? I know this project is pre 1.0 release so breaking changes can be expected, but

the flag's description seems to be inaccurate
it'd be great if a required change like this was prominently mentioned in the release notes

I'm happy to open a PR if we can agree on the expected behavior and flag description.

I can confirm @rifelpet's findings. Looks like that backend-mode flag was it.
When I run v0.5.0 with --backend-mode=ConfigMap,CRD,File, I get the same logs as I posted above with the same authentication errors. I still get authentication errors if I run it with --backend-mode=File,CRD,ConfigMap.
If however, I run with just --backend-mode=File, I get the below logs and kubectl is able to authenticate successfully:
time="2020-03-04T10:21:15Z" level=info msg="starting mapper \"File\""
time="2020-03-04T10:21:15Z" level=info msg="mapping IAM role" groups="[system:masters]" role="arn:aws:iam::XXX:role/ROLE_NAME" username=kubernetes-admin
time="2020-03-04T10:21:15Z" level=info msg="mapping IAM role" groups="[system:bootstrappers aws:instances]" role="arn:aws:iam::XXX:role/KubernetesNode" username="aws:{{AccountID}}:instance:{{SessionName}}"
time="2020-03-04T10:21:15Z" level=info msg="mapping IAM role" groups="[system:masters]" role="arn:aws:iam::XXX:role/ROLE_NAME" username="admin:{{SessionName}}"
time="2020-03-04T10:21:15Z" level=info msg="loaded existing keypair" certPath=/var/aws-iam-authenticator/cert.pem keyPath=/var/aws-iam-authenticator/key.pem
time="2020-03-04T10:21:15Z" level=info msg="listening on https://<IP_ADDRESS>:21362/authenticate"
time="2020-03-04T10:21:15Z" level=info msg="reconfigure your apiserver with `--authentication-token-webhook-config-file=/etc/kubernetes/aws-iam-authenticator/kubeconfig.yaml` to enable (assuming default hostPath mounts)"
time="2020-03-04T10:22:11Z" level=info msg="STS response" accesskeyid=ACCESS_KEY_ID accountid=XXX arn="arn:aws:sts::XXX:assumed-role/ROLE_NAME/TS" client="<IP_ADDRESS>:60564" method=POST path=/authenticate session=TS userid=USER_ID
time="2020-03-04T10:22:11Z" level=info msg="access granted" arn="arn:aws:iam::XXX:role/ROLE_NAME" client="<IP_ADDRESS>:60564" groups="[system:masters]" method=POST path=/authenticate uid="aws-iam-authenticator:XXX:USER_ID" username="admin:TS"

I've opened a PR to fix this, please take a look at https://github.com/kubernetes-sigs/aws-iam-authenticator/pull/303/files#diff-04c6e90faac2675aa89e2176d2eec7d8R85

Looks like #303 is merged and is part of v0.5.1-alpha.1, thanks @wongma7!


swiss: optimize bucket init by setting control bytes simultaneously

This is a small optimization to set all the controls bytes in a group to empty in a single assignment when initializing a bucket or when using Clear, rather than doing so in a loop.
This saves around 2% when growing an initially empty map, at least at these sizes:
goos: linux
goarch: amd64
pkg: github.com/cockroachdb/swiss
cpu: Intel(R) Xeon(R) CPU @ 2.30GHz

                                   │   old       │              new                   │
                                   │   sec/op    │   sec/op     vs base               │
MapPutGrow/swissMap/Int32/6-16       205.8n ± 0%   200.4n ± 1%  -2.65% (p=0.000 n=10)
MapPutGrow/swissMap/Int32/12-16      495.4n ± 0%   481.7n ± 0%  -2.77% (p=0.000 n=10)
MapPutGrow/swissMap/Int32/18-16      948.5n ± 1%   910.0n ± 1%  -4.06% (p=0.000 n=10)
MapPutGrow/swissMap/Int32/24-16      1.067µ ± 1%   1.036µ ± 0%  -2.91% (p=0.000 n=10)
MapPutGrow/swissMap/Int32/30-16      1.847µ ± 1%   1.764µ ± 1%  -4.49% (p=0.000 n=10)
MapPutGrow/swissMap/Int32/64-16      3.780µ ± 1%   3.591µ ± 1%  -4.99% (p=0.000 n=10)
MapPutGrow/swissMap/Int32/128-16     7.618µ ± 3%   7.294µ ± 1%  -4.25% (p=0.000 n=10)
MapPutGrow/swissMap/Int32/256-16     14.81µ ± 0%   14.59µ ± 1%  -1.44% (p=0.000 n=10)
MapPutGrow/swissMap/Int32/512-16     28.90µ ± 2%   28.17µ ± 1%  -2.55% (p=0.000 n=10)
MapPutGrow/swissMap/Int32/1024-16    56.89µ ± 0%   55.31µ ± 1%  -2.79% (p=0.000 n=10)
MapPutGrow/swissMap/Int32/2048-16    111.8µ ± 1%   110.4µ ± 1%  -1.28% (p=0.000 n=10)
MapPutGrow/swissMap/Int32/4096-16    273.2µ ± 1%   275.5µ ± 1%  +0.82% (p=0.029 n=10)
MapPutGrow/swissMap/Int32/8192-16    613.0µ ± 1%   618.8µ ± 1%       ~ (p=0.052 n=10)
MapPutGrow/swissMap/Int32/32768-16   2.683m ± 0%   2.692m ± 0%  +0.34% (p=0.023 n=10)
MapPutGrow/swissMap/Int32/65536-16   5.552m ± 1%   5.613m ± 1%  +1.10% (p=0.002 n=10)
geomean                              20.17µ        19.75µ       -2.09%

Tested on a n1-standard-16 GCE VM. (Intel Haswell, and seemingly slower than the n1-standard-16 used in the README benchmarks).

This change is

Thanks for the contribution.


APIs to define formatting rules

I am trying to add a formatter to a custom language in Monaco Editor. I know we can use
monaco.languages.registerDocumentFormattingEditProvider(...)
monaco.languages.registerDocumentRangeFormattingEditProvider(...)
monaco.languages.registerOnTypeFormattingEditProvider(...)

In each of the above APIs, we need to define a provider, e.g., provideDocumentFormattingEdits that returns a TextEdit. In the TextEdit, we need to return a text, which will replace the initial text.
One way to construct the text of TextEdit, is to manually transfer the initial text by e.g., replace or inserting \n. But, I am wondering whether there is any existing APIs to simplify this process. For example, is there any way to set indentation/newline rules based on tokens?
Does anyone know?

@chengtie I'm trying to define a formatter for a custom language and I coudn't find a proper guide/documentation. When you say

In the TextEdit, we need to return a text, which will replace the initial text.

What do you mean? Could you please explain with an example?
And also, suppose I want to insert a tab space \t after every word, what would the TextEdit look like?

@chengtie I'm trying to define a formatter for a custom language and I coudn't find a proper guide/documentation. When you say

In the TextEdit, we need to return a text, which will replace the initial text.

What do you mean? Could you please explain with an example?
And also, suppose I want to insert a tab space \t after every word, what would the TextEdit look like?

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


Feature Request: environment variables (e.g. for proxy support)

As an enterprise user that is working behind an internet proxy, I would like to be able to pass environment variables that are used during the initialization of the runner to the constructs so that I can correctly configure the proxy URL and no-proxy list.

Hey @Obirah ,
actually you can pass env vars to the runner:
https://github.com/pepperize/cdk-autoscaling-gitlab-runner/blob/main/API.md#environmentoptional-
Which configuration properties are missing exactly to you https://docs.gitlab.com/runner/configuration/advanced-configuration.html ?
You may open a PR adding them.
Regards,
Patrick

Hey Patrick,
thank you for the quick response! I'm using the environment property to pass my proxy environment to runner, however I don't think that this is sufficient for my setup:
I need the proxy to be in effect here so that this call to packages.gitlab.com goes through the proxy.
Optionally, it would also be alright to be able to replace https://packages.gitlab.com with our internal GitLab RPM mirror, however I don't think this is really feasible as https://packages.gitlab.com/install/repositories/runner/gitlab-runner/script.rpm.sh will also point to the original URL and not the mirror.

Hey @Obirah ,
you may refactor the init config step repositories to be passed by the construct's props. Then you pass your own InitConfig to setup your private mirror.
The step packages may also bei refactored to pass your own docker+machine gitlab fork binary.
I guess, you also want to provide a hardened or customized AMI for the manager instance.
That combination could give you the desired flexibility.
I considered CDK escape hatches with InitConfig won't work very well.


Improvement Suggestion

Having UI status as boolean is prone to bugs. In this case, it's correct, but in other cases, the user can end up with error and loading turned on due to bad code, causing a bug. By having status as a string and not boolean, we can be safe against this type of bug.

Thanks for the PR. I'm not sure what you mean by "the user can end up with error and loading turned on due to bad code" though. Do you have an example to share with us?

Sure, let's say that I can't use finally for a reason (i.e. can't transpile it), so I just remove it when following the example:
...
  mounted () {
    axios
      .get('https://api.coindesk.com/v1/bpi/currentprice.json')
      .then(response => {
        this.info = response.data.bpi
      })
      .catch(error => {
        console.log(error)
        this.errored = true
      })
  }
...

If I do that, I'll end up with errored = true and loading = true, and UI might break.
If instead of boolean statuses, we make it strings, this will never happen:
...
  mounted () {
    axios
      .get('https://api.coindesk.com/v1/bpi/currentprice.json')
      .then(response => {
        this.info = response.data.bpi
      })
      .catch(error => {
        console.log(error)
        this.status = 'errored'
      })
  }
...

Is that clear?

Sure, let's say that I can't use finally for a reason (i.e. can't transpile it), so I just remove it

I'd say this "problem" applies for any kind of code modification/removal logically. When one alters some code, they should be aware of the effect(s). I don't see this as the code's problem, but the developer's.
Also, I'd personally frown upon using strings as logical flags, and when it must be the case, I'd always turn the strings into constants.
As such, I'm closing this PR. Thanks a lot for your contribution though, and looking forward to the next ones!


i18n(es): Translated i18n.md

What kind of changes does this PR include?

Changes or translations of Starlight docs site content

Description

What does this PR change? Give us a brief description.
Translated i18n.md to spanish

Gracias por la revisión @Waxer59 😄


[accessibility] Convey context menu items as buttons

Assistive technology such as screenreaders convey the context menu items (<a>) as a "link". I believe in more cases than not the context menu items behave more like buttons than links. Therefore I suggest using role=button for the context menu items.
Notes:
The W3C Maps for HTML Community Group implemented this for a custom element here.
This would also align with Leaflet controls:
https://github.com/Leaflet/Leaflet/blob/bdd7ebf836201550f6d173943b728098b83f40d5/src/control/Control.Zoom.js#L89-L91
(alternatively, use <button> instead.)

Also see the WAI-ARIA Authoring Practices recommended keyboard interactions for buttons.

Also see the WAI-ARIA Authoring Practices recommended keyboard interactions for buttons.


onOp capturing each cell changes so that causing overhead to capture the required items

<Workbook  ref={workbookRef} data={settings} onOp={onOp} onChange={onChange} />
const onOp = useCallback((op: Op[]) => {
//const socket = workbookRef.current;
console.log(JSON.stringify({ req: "op", data: op }));
}, []);
Need a option to capture the onOp on specific columns or ignore for specific columns so that overhead for frequently change data will be quick and simple for advance logics.

Ok, we will look into it.


Add performance logging documentation

The build is failing but link this page into the website, update index.rst to add it to the table of contents at the bottom of the page.

Fixed! It's ready for review now.


Fix learning rate problems in run_classifier.py

Don't do warmup twice (in BertAdam and manually)
Compute num_train_steps correctly for the case where gradient_accumulation_steps > 1. The current version might lead the the LR never leaving the warmup phase, depending on the value of gradient_accumulation_steps.

With these changes I get > 84% accuracy on MRPC, without them its around 77%.

Thanks @matej-svejda, I was a bit late on this PR. I've made a small commit to make the notation more explicit (removed t_total which was mainly a duplicate of num_train_steps and renamed num_train_steps in a more explicit num_train_optimization_steps).
Merging this now


[Program]: Write a R program to print ascii value of a string

Description
Write a R program to print ascii value of a string
ASCII value is the numeric representation of a character. For example, the ASCII value of the character 'A' is 65. The ASCII value of the character 'a' is 97. The ASCII value of the character '0' is 48. The ASCII value of the character ' ' is 32.
Input  : ABC
Output : 65 66 67

How to contribute

Save the solution in program/program/print-ascii-value-of-a-string/print_ascii_value_of_a_string.r file

!assign


please post a video for installation of openpose

First of all to be frank i became fan of your  work on tello openpose .Sir i had been working for past four weeks to install tello openpose and make it work on tello  ubuntu 18.04 but getting errors and i referred issues  posted in the repository still i was not able to solve it.
please provide video for installation of your openpose layer and installation of all dependancies  and eveything required to implement on tello drone please sir
            thank you sir

Thansk Geaxgx, Hi Vishnuproject Below link also helped me install openpose and create and openpose portable folder
https://blog.techbridge.cc/2019/01/18/openpose-installation/
https://yuanze.wang/posts/build_openpose_python_api/


Fix bug in validation code

There is a but in the validation code in ingest-api main which is caused by a the header of a call to entity-api not being a dict.
There is one other problem in ingest-api which is caused by the fact that ingest_validation_tools hard codes the entity-api url to that in PROD in the file upload.py _cedar_url_checks().

PR: https://github.com/hubmapconsortium/ingest-api/pull/488

The issue of ingest_validation_tools only working in PROD because it hardwires a call to entity-prod in the code, is not fixed here. The ingest_validation_tools should pull the first part of the path from a config file. I believe that the authors are working on this.

@ChuckKollar thanks for tracking the issue with this new card. Yes we'll have to wait until the PSC team addresses this issue in the Ingest Validation Tools.


[iOS FCM] Library -lGoogleToolboxForMac not found on Xcode build

Expected Behaviour
Actual Behaviour
Xcode failed to build due to error
ld: warning: directory not found for option '-L/Users/yingweiloke/Library/Developer/Xcode/DerivedData/TeamRN-greqqswbqeruppgwymcpsslcmaxe/Build/Products/Debug-iphoneos/GoogleToolboxForMac'
ld: warning: directory not found for option '-L/Users/yingweiloke/Library/Developer/Xcode/DerivedData/TeamRN-greqqswbqeruppgwymcpsslcmaxe/Build/Products/Debug-iphoneos/Protobuf'
ld: library not found for -lGoogleToolboxForMac
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Reproduce Scenario (including but not limited to)
Install the plugin v2.0.0 and build and run the project with cordova cli. App built and run.
Open the .xcodeproj file on Xcode and build -> failed.
Platform and Version (eg. Android 5.0 or iOS 9.2.1)
iOS 10.3.1
Cordova CLI version and cordova platform version
cordova --version                          6.5.0
cordova platform version ios        4.4.0
Plugin version
2.0.0-rc3

Fixed in https://github.com/phonegap/phonegap-plugin-push/pull/1715

I installed the plugin with commit but I'm still seeing the same issue

It definitely works -- did you grab the plugin from the 2.0.x branch?
cordova plugin add https://github.com/phonegap/phonegap-plugin-push.git#v2.0.x

I did cordova plugin add  https://github.com/phonegap/phonegap-plugin-push#e5739b8721eb7210a333725089fec3b36db5b348 and I confirmed that the line <framework src="GoogleToolboxForMac" type="podspec" spec="~> 2.1.1"/> is in the plugin.xml

Let me try it out using your method. Will let you know.

Using your plugin sha hash, I was able to build successfully. Repro steps:
$ cordova create gtm-test
Creating a new cordova project.
$ cd gtm-test
$ cordova platform add ios
Using cordova-fetch for<EMAIL_ADDRESS>Adding ios project...
Creating Cordova project for the iOS platform:
	Path: platforms/ios
	Package: io.cordova.hellocordova
	Name: HelloCordova
iOS project created with<EMAIL_ADDRESS>Discovered plugin "cordova-plugin-whitelist" in config.xml. Adding it to the project
Installing "cordova-plugin-whitelist" for ios
Adding cordova-plugin-whitelist to package.json
Saved plugin info for "cordova-plugin-whitelist" to config.xml
--save flag or autosave detected
Saving<EMAIL_ADDRESS>into config.xml file ...
$ cordova plugin add https://github.com/phonegap/phonegap-plugin-push\#e5739b8721eb7210a333725089fec3b36db5b348
Installing "phonegap-plugin-push" for ios
Adding phonegap-plugin-push to package.json
Saved plugin info for "phonegap-plugin-push" to config.xml
$ open platforms/ios/Podfile
$ open -a Xcode platforms/ios
$ cordova --version
7.0.1
$ cordova platform ls
Installed platforms:
  ios 4.4.0
Available platforms:
  android ~6.2.2
  blackberry10 ~3.8.0 (deprecated)
  browser ~4.1.0
  osx ~4.0.1
  webos ~3.7.0
$ xcodebuild -version
Xcode 8.3.2
Build version 8E2002

From above, I opened up the Podfile to verify GTM was there, and I built the project in Xcode 8.3.2 (after setting the correct provisioning profile, cert, etc). It built with no errors.

Ok, I found out the problem. I had been opening the project on Xcode by platforms/ios/my-app.xcodeproj instead of opening the platforms/ios directory. It was working until the pods thing created as a Xcode project in v2.x.x.
Thanks for your help and the steps to repro!

Great! you're welcome :)

The issue persists on a fresh clone of a project that uses this plug-in. (v2.0.0-rc5)
When using ionic cordova build ios --confirm on a fresh clone of the project (it runs npm install then cordova platform add ios then cordova build ios), I get the same errors. (plus GoogleService-Info.plist missing)
It's possible to work around the build issue by running:
ionic cordova plugin remove phonegap-plugin-push
ionic cordova plugin add phonegap-plugin-push@latest

seems it is not related to the plugin version, but need the action of plugin remove and add
as it ok for
ionic cordova plugin remove phonegap-plugin-push
ionic cordova plugin add<EMAIL_ADDRESS>but will fail if just edit config.xml for the plugin

Thanks for all your help. This issue should be documented in Installation guide as well.

This issue is still reproducible on 2.2.2

This is a Cordova plugin, pod install seems to fail or some files are missing.
Manually going to platforms/ios and running pod install there fixes the issue

Because I can't run it manually (must compile with fastlane on our CD server) i worked around adding a custom hook:
Add a file scripts/pod_install.js:
const { exec } = require('child_process');

module.exports = function(context) {
    console.log("Running pod install");
    exec('cd ' + context.opts.projectRoot + '/platforms/ios && pod install');
    console.log("Done");
}

Then add this line inside config.xml (under the iOS platform):
<hook src="scripts/pod_install.js" type="after_plugin_install" />


Add How to Debug dynamic query/layout rendered

With so much interaction to OOB APIs , How to debug our inputs/code on schedule board would be a best to have documented.
Thanks !

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

ID: 853da741-6ae3-8964-b6ff-7f58d70be56f
Version Independent ID: 2a8010cd-785a-aa26-6ea0-f1084edda417
Content: Extending URS - Find resources by language - a step by step guide
Content Source: ce/common-scheduler/developer/extending-urs-step-by-step.md
Service: crm-online
Technology: field-service
GitHub Login: @FieldServiceDave
Microsoft Alias: daclar

Thanks for your feedback and request! We have an update for this URS doc in our backlog, and I've added your feedback to that task :)


Data folders lack current data used

Should we add the new data files?
We can override .gitignore rule set using:
git add -f

Or we can just let the user make data all over again.

By removing the data folder?


[Jenkins-plugin] Add an optional secondary branch to EntityLatestJenkinsRunCard

Feature Suggestion
Add the option to have a secondary branch to be displayed in the EntityLatestJenkinsRunCard in case the first one doesn't exist. The idea here it's to solve the problem between the "main" and "master" branches.

Possible Implementation
Two possible implementations:

First one:
Add a second optional branch to the EntityLatestJenkinsRunCard: the plugin should be able to use the second branch when the first does not exist in the project.
It could be something like this:

<EntityLatestJenkinsRunCard  branch="master" secondarybranch="main" />

Second one:
Another solution could be the possibility of adding the default branch in the annotation itself:

annotations:
    jenkins.io/github-folder: 'folder-name/project-name/default-branch'

Context
Due to the renaming of the default GitHub branch there are some new repositories with the main branch by default and others with the master branch, which causes that in some cases the card cannot show the latest build.

Could we make it so that if if no branch is given at all to EntityLatestJenkinsRunCard, it is smart enough to try both master and main? I feel that this would give the best end user experience with little implementation overhead

Sure! that's probably the best approach to solve the problem


Update ExDoc

It would be nice if ExDoc could be updated to v0.30.3.  This fixes an issue with the light theme on mobile (the black bar at the top should be white):

@weiss, this is done in main already, but simply not released yet.

Yup, was pushed into main after my PR, I'll close this, thanks.
(I wonder why CI doesn't seem to run into https://github.com/elixir-lang/ex_doc/issues/1745 BTW.)

v0.2.19 released


Removing Boto3 as dependency

Earth2Studio Pull Request
Description

Swapping out boto3 for s3fs. Boto3 increased install time of earth2studio substantially, should avoid in future.
Closes: https://github.com/NVIDIA/earth2studio/issues/52
Checklist

[x] I am familiar with the Contributing Guidelines.
[x] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.
[x] The CHANGELOG.md is up to date with these changes.
[x] An issue is linked to this pull request.

Dependencies

/bossom-ci

/bossom-ci

/blossom-ci


Jump to definition requires a reload after build

If I load a project with dependencies into VS Code which has not been built when I right click on a symbol to go to definition (run a textDocument/definition request) it returns an empty result. This is a known issue.
If I then build the project and repeat I still get an empty result. I would assume once the project is built SourceKit-LSP should be able to return a position for the symbol.
I have to restart VS Code (ie restart the SourceKit-LSP server) for SourceKit-LSP to return a valid position for the symbol.

rdar://99329579

This look like it is resolved, will re-open if I see it again

Re-opening saw it again

Actually it is just slow to respond. Is there a way we can indicate the LSP server is loading the index after a build, or whatever it is doing post build that means it doesn't have the built symbols available.

I’m adding a progress indicator for package reloading in https://github.com/apple/sourcekit-lsp/pull/767. That’s all I can think of that might take a non-trivial amount of time and happens on the SourceKit-LSP side after the project has been modified.

Oh, and now I’ve been able to reproduce the issue as well, independent of the work done progress for package loading.
The issue appears to be that we don’t get compiler arguments if the dependencies have not been checked out and we don’t reload the package after the dependencies are checked out.
Steps to reproduce:

Set SOURCEKIT_LOGGING=99 in the VSCode Swift plugin settings
Clone the soucekit-lsp repository
swift package reset
Open the project in VSCode
Open DocumentTokens.swift
Notice that DocumentTokens.swift gets opened with compiler arguments
Run a build
Close and re-open DocumentTokens.swift
Notice that it still gets opened without compiler arguments

What’s happening is that after building, you need to make an edit to the file to get jump-to-definition working. Could that match what you’ve been seeing @adam-fowler?
On a technical level

When we open the file, we build an AST for it. That AST has errors because the dependent modules haven’t been built yet.
Now we build the dependent modules and the dependent module files exist
If you now do a jump to definition request again, we perform a cursor info request to get the USR of the symbol we do jump-to-definition for but since the file hasn’t changed, cursor info re-uses the previously built AST and is thus unable to resolve the symbol (because at the point that the AST was created the module that contains the symbol didn’t exist yet)
If you make an edit to the file (that’s not an edit inside a comment), we stop re-using the previously built AST. At that point, you see semantic highlighting appearing for the symbol in the other module and jump-to-definition starts working.

The way I would like to fix this, is to migrate cursor info to solver-based, which removes the need for this AST reuse altogether. This is a long-term effort though. I don’t have a good idea for a short-term workaround and since things recover after the first edit, I believe there’s nothing immediate we need to or should do here.

What’s happening is that after building, you need to make an edit to the file to get jump-to-definition working. Could that match what you’ve been seeing @adam-fowler?

I'll have to check that. It could be.
Given you are saying the correct solution is a long-term effort is there anything temporary I could do to resolve this issue. I know when a build finishes could I get VSCode to send some request, notification that would trigger an AST rebuild

Other than making an edit to the file, unfortunately I don’t think there’s anything you could do here to work around the issue. I’m sorry.


fix: transitionDuration should be number

hi, i think  transitionDuration should be number.

transitionDuration is a number or a string.
https://masonry.desandro.com/options.html#transitionduration
I can't merge this in as is sorry.

This will be fixed once https://github.com/eiriklv/react-masonry-component/pull/122 is merged.


Issue a certificate move workload to top

This adds a first step where we issue a demo cert and look into how cert-manager handles it first, then we do the workload then the ingress.

/assign @munnerz


Make each project a unique page

Projects pages need to be unique pages, should fix problem where doesn’t load first time, also will help with accessibility

This can be done later, post launch.


[feature request] allow set prefetch on dispatcher

right now, the prefetch on dispatcher is set to 1. This seems to be very restrictive, especially for long-running consumers.
I'm wondering why 1 is chosen?
event with keda to scale it based on the number of messages, it still seems inefficient.
Is there a plan to allow set the prefetch count or increase the default?
thx!

You mean here:
https://github.com/knative-sandbox/eventing-rabbitmq/blob/main/cmd/dispatcher/main.go#L51
Is this something that would be enough to specify at the Broker level?  Since dispatcher is for each Trigger, being able to configure it would mean changing the API and that won't be doable in the near future due to v1 work. However, adding that configuration option and plumbing that into a dispatcher seems like it would be fairly straight forward.
As far as changing the default, I'm not sure if there are any downsides to just changing the default from 1, I'm not sure why the value was chosen honestly. So changing that to something else seems reasonable too.

@vaikas thx for the quick response! Right, I'm referring to the code you linked.
I think in the long run, it probably makes the most sense to allow flexibility on the trigger level.
But if we can allow the config on the broker level in the short run, I think it would offer a lot of flexibilities, which is good enough

/assign
/remove-lifecycle stale


Not so satisfied with the result of my own dataset. Display code request for debugging.

Hello Yiming,
Thanks again for sharing this wonderful work! While I was testing on my own data. The result came out to be not so satisfying. Is it possible that you can share the display code for incremental reconstruction? So I can try the locate my problem. If not, any help for me to start with will be much appreciatted.

I didn't write the code for online visualization. I refer you to https://github.com/zju3dv/NeuralRecon/blob/master/utils.py#L246
If you'd like to save incremental reconstruction mesh, you need to call this function
https://github.com/neu-vi/PlanarRecon/blob/main/utils.py#L434 in each iteration.

Reference in new

I didn't write the code for online visualization. I refer you to https://github.com/zju3dv/NeuralRecon/blob/master/utils.py#L246
If you'd like to save incremental reconstruction mesh, you need to call this function https://github.com/neu-vi/PlanarRecon/blob/main/utils.py#L434 in each iteration.

Thanks a lot!

@ChiyuanFeng Hi, I am a student from China. I am also using planarrecon to test my data recently, can I add your WeChat to have a discuss? Best of luck with your research！


Enable Github Two-Factor Requirement

As of this moment this is not a requirement to be a member of the org.
It is put forward for consideration that this should be enabled to help guarantee that Typelevel organization members are secure in their interaction with code.
A note that if enabled will silent kick anyone who does not have it enabled.

Members, billing managers, and outside collaborators who do not have two-factor authentication enabled for their personal account will be removed from the organization and will receive an email notifying them about the change.

Yes, but it would be good to have a grace period.  This currently applies to nine humans, at least a few of which are highly active.
It also applies to @typelevel-bot.  We can work around that, but it will tie into #32.

If this is a hardship for anyone, please speak up here or reach out privately, and we can seek a solution that's both inclusive and secure.
/cc @typelevel/maintainers

Howto: https://docs.github.com/en/authentication/securing-your-account-with-two-factor-authentication-2fa
I just sent an e-mail or tweet to everyone outstanding.
I also added 2FA to @typelevel-bot.  Contact me for the TOTP secret.

Everyone has been contacted.  Most responded, one opted out, and everyone has been told they're welcome back.  I say it's safe to flip the switch.

Done.  If anyone wants back in, please contact an admin.


Check that particle names on STAT table match the fallback names...

...in each axis registry at the Google Fonts Axis Registry
(issue #3022)
This pull request is marked as a DRAFT because it still lacks code-tests. I want to get feedback from @m4rc1e regarding whether this implementation correctly encodes the checking procedures he had in mind for the STAT table when he requested it at #3022. If Marc agrees that this is indeed what we should be checking for, then I'll put the effort to write a few test-case examples in a code-test for this check.

thanks for all the feedback!

@m4rc1e, I'm merging this now because it is already useful. If possible, please open a new issue attaching a format-4 table so that we can expand the implementation to support that as well.

This looks pretty good! I agree it's now worth merging. Thank you so much for adding this.


Provide a way of knowing which emails failed/succeeded in sending

Is your feature request related to a problem? Please describe.
When sending multiple emails, an error is returned when any one of them fails. How do I know which one failed? I have, let's say, 100 emails to send. I get an error back ... now what? Which one(s) do I try resending? Which ones do I mark internally as having been sent / failed to send?
Describe the solution you'd like
Perhaps it makes sense to return an array of errors, 1:1 with the messages, guaranteed to be the same length as the messages? That makes it slightly annoying for the checkConn-type global failure. OTOH, this not exactly the common case, so OK to just make N copies of that error in an array? Not exactly idiomatic Go, but I don't know what is in this case.
Describe alternatives you've considered
Perhaps something can be added to the Msg itself indicating send status? Seems dodgy to mutate inputs though, and what happens if the object is reused somehow (e.g. resent), could make it unclear whether the status is "correct" or not.
Additional context
Note that I'm not using this library yet. Just shopping around for a lib that can send multiple emails in a single connection.

Hm, answering myself, perhaps the solution is to just Send() with one Msg per invocation, reusing the same client. But then why does Send() take multiple args?

Thanks for the issue @imirkin. That's actually an interesting problem. I assume most sending mail servers would actually provide an error message that holds i. e. the recipients mail address, but that's probably not the default for everything. I'll think about it and come up with a possible solution and open it for discussion.

I don't think that mail servers return errors in any standardized format. A human reading the error could maybe work it out, but even that's not guaranteed. Note that the emails may be going to the same recipients, etc.

I am proposing this change: https://github.com/wneessen/go-mail/compare/main...feature/90_provide-a-way-of-knowing-which-emails-failedsucceeded-in-sending
It introduces the SendError type which implements the error interface. A new senderror field has been added to the Msg as well, so introduce this type to it.
I've also added different error variables that indicate the different things that can go wrong during mail delivery. These variables can be checked for, for each Msg using the errors.As method
The Error() method of SendError will return a detailed error string on why the Msg could not be delivered.
Additionally, HasSendError() and SendError() methods have been added to Msg. While HasSendError() simply returns a bool in case a Msg failed during delivery, the SendError() will return the full SendError error interface.
I think it's fair that we reset the Msg.senderror to nil before we Client.Send it. Since the send error indicate an error during the mail delivery, in my opinion it should be reset when a re-send is initiated, so that the senderror field always represents the latest delivery error. The send error should be checked after mail delivery and before a retry is started.

While working on a test for the different test types, I noticed and implementation issue, so the commits made, are not final yet.

I think there's probably enough info, but it's also important to be able to distinguish between a "retriable" error, e.g. server just plain disconnects (let's say it got restarted), and a permanent error, e.g. "cannot deliver to recipient", etc. I believe the error codes from SMTP commands indicate the kind of error that it is.

True. That would involve some SMTP response message parsing. Let me check what can be done there

I'ver overhauled the commit and fixed the error handling. We've also now a isTemp field in the error, that indicates if it's a retryable error or not.

Took a look at the code because I'm interested in the isTemp flag.
I think it would make sense to add this method to the error type as well because error values would be checked first.
I'd want to use it like this:
if err := client.DialAndSendWithContext(ctx, msg); err != nil {
	var sendErr mail.SendError
	if errors.As(err, &sendErr) && sendErr.IsTemp() {
		// retryable
		return nil, fmt.Errorf("%w: %s", ErrTemporary, sendErr.Error())
	}
	// probably non-retryable
	return nil, fmt.Errorf("%w: %s", ErrSending, err.Error())
}

Also consider removing the client_send string prefix from the error messages, because this can look strange or redundant when library users wrap errors.

Good point indeed @iwittkau. I've adapted the proposal and changed the Send method to return the SendError instead of a fmt.Error, so we can use the code you proposed for checking for temporary errors after delivery.

I'm pretty happy with the proposal and created a corresponding PR. If no further comments or suggestions are added to this discussion, I'll merge it tomorrow and release v0.3.7.


[sql lab] WIP. Autosync queries periodically

@mistercrunch - I've implemented merging logic.
The fact that I change the query id in the object broke the results display,
Do you know how could I fix that?

@bkyryliuk let's sit down together and push this through today


eartag: 0.4.0 -> 0.4.1

Description of changes
https://github.com/knuxify/eartag/releases/tag/0.4.1

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 238359 run on aarch64-darwin 1

1 package built:

eartag


[Bug] Export>.pdf>about:blank issue

Is there an existing issue for this?

[ ] Yes, I have searched the existing issues and none of them match my problem.

Product Variant
Cloud (https://rxresu.me)
Current Behavior
Hello, first of all, I congratulate you for this project you developed. I have benefited from it many times, but today I have a problem. When I want to export it as .pdf, a blank tab opens. I do not have this problem when I download a json file. I would be very happy if you could help me.
Expected Behavior
I just want to download
Steps To Reproduce
Export>.pdf>about:blank
What browsers are you seeing the problem on?
Firefox, Chrome, Microsoft Edge
What template are you using?
None
Anything else?
No response

SOLVED:
When we download and import it as a .json file, we can download it as a .pdf file.


Publish package to pip

We would like to publish the package to pip to make it easier to install. Unfortunately, the name tools and py-tools is already used on pip, so we need to find a better alternative and rename this package.
Options so far:

py-science-tools
science-tools
zwicker-tools
py-zwicker

Any suggestions?

You might find the setup of our utility library helpful: https://github.com/tum-esm/utils
I use PDM to build and publish releases on PyPI: pdm build && pdm publish.
Looking forward to more tools from your group we can reuse :D

Yeah, I think it is a good idea to have such an utils package to get people easily started. We don't have any technical issues publishing on pip, it's rather that we currently picked a bad package name – you guys apparently did a better job :)

I think zwicker-tools is nice :) I think you can leave the "py" in the repo name but don't need it in the PyPI package.


enhance(build): finish to log compilied pkg.

This pull request includes a little change to the DrainState implementation in the src/cargo/core/compiler/job_queue/mod.rs file. The change enhances the logging and tracking of the job queue's progress by adding detailed logging for different types of artifacts.
Enhancements to logging and tracking:

src/cargo/core/compiler/job_queue/mod.rs: Added detailed logging for the completion of a unit of work, including specific statuses for documentation, scraping, and compilation. This helps in tracking the progress of the job queue more effectively.

Saw you just closed it, though thanks for the contribution anyway!
The major issue of console output overhaul is https://github.com/rust-lang/cargo/issues/8889, which you might be interested in. One of complaint is the current output too noisy, which is on the opposite side of this PR.


Adjusts expectations of data reporting of loopback spans

The latest version of Brave corrects a data merge problem where shared
spans (created via Tracer.joinSpan) would lose some data. Along the way,
this fixes some misuse of Tracer.joinSpan which should only be used
for RPC server spans.
See https://github.com/openzipkin/brave/pull/734 applied in Brave 5.1.3

cc @zeagord @shakuzen

Codecov Report

Merging #1034 into master will not change coverage.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #1034   +/-   ##
=========================================
  Coverage     70.74%   70.74%
  Complexity      628      628
=========================================
  Files           119      119
  Lines          2806     2806
  Branches        298      298
=========================================
  Hits           1985     1985
  Misses          626      626
  Partials        195      195

Impacted Files
Coverage Δ
Complexity Δ

.../instrument/rxjava/SleuthRxJavaSchedulersHook.java
77.19% <100%> (ø)
5 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


Trouble to play tone if is short.

Hello even in slow tempo, it is not playing 1/16th tones and shorter. Even in tempo 60 it is not generating E5, F6 below.
Am I using it wrong?
My code:
CALLING
static void Main(string[] args)
        {
            double tempo = 60;
            List<ISampleProvider> melody = new();

            for (int i = 0; i < 4; i++)
            {
                var binaryNumber = 1 << i;
                var tones = new List<double> { Tones.A4, Tones.B4, Tones.C5, Tones.D5, Tones.E5, Tones.F6 };

                var tone = Generator.GetTone(tones.ElementAt(i), tempo, binaryNumber);

                melody.Add(tone);
            }

            foreach (var tone in melody)
            {
                using (var wo = new WaveOutEvent())
                {
                    wo.Init(tone, true);
                    wo.Play();
                    while (wo.PlaybackState == PlaybackState.Playing)
                    {
                        Thread.Sleep(1000);
                    }
                }
            }
        }

Generator
    internal class Generator
    {
        public static ISampleProvider GetTone (double tone, double tempo, int lengthOfNote)
        {
            var result = new SignalGenerator()
            {
                Gain = 0.2,
                Frequency = tone,
                Type = SignalGeneratorType.Sin
            }.Take(TimeSpan.FromSeconds((60.00 / tempo) / lengthOfNote));

            return result;
        }
    }

Tones (only few of them)
    public class Tones
    {
        ...
        public const double A4 = 440.00;
        public const double Ais4 = 466.16;
        public const double B4 = 493.88;
        public const double C5 = 523.25;
        public const double Cis5 = 554.37;
        public const double D5 = 587.33;
        public const double Dis5 = 622.25;
        public const double E5 = 659.25;
        public const double F5 = 698.46;
        ...
     }

I am sorry. Big hole in musical theory. (Wrong formula in .Take)

I am sorry. Huge hole in musical theory. Works good.


change help text position

Changes in this PR:
Change help text position to adhere to GDS
Trello card / Rollbar error (etc)
https://trello.com/c/NU1ZhWGa/830-help-text-change-position-inline-with-gds-styling-in-all-instances-on-pui-awaiting-768
Screenshots of UI changes:
Before

After

[ ] Requires env variable(s) to be updated

Codecov Report

Merging #676 (2eddcb4) into main (817b775) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #676   +/-   ##
=======================================
  Coverage   88.79%   88.79%
=======================================
  Files          21       21
  Lines        1017     1017
  Branches       56       56
=======================================
  Hits          903      903
  Misses         97       97
  Partials       17       17

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Post processing effect is applied to all cameras

Bevy version
18d001d27ce475388743d9825f2833ce51f5b20d
AdapterInfo { name: "NVIDIA GeForce GTX 1650 with Max-Q Design", vendor: 4318, device: 8081, device_type: DiscreteGpu, driver: "NVIDIA", driver_info: "545.29.02", backend: Vulkan }

What you did
I tried to add a post-processing effect to my game, based on the post_processing example.
What went wrong
This worked fine, except that it was applied to all cameras. This was confusing, as the example specifically states:
        // Add the setting to the camera.
        // This component is also used to determine on which camera to run the post processing effect.

To reproduce:
Modify the post_processing example by replacing the camera spawning code with this
    // camera
    commands.spawn(Camera3dBundle {
        transform: Transform::from_translation(Vec3::new(0.0, 0.0, 5.0))
            .looking_at(Vec3::new(-1.0, 0.0, 0.0), Vec3::Y),
        camera_3d: Camera3d {
            clear_color: ClearColorConfig::None,
            ..default()
        },
        camera: Camera {
            order: 1,
            ..default()
        },
        ..default()
    });
    commands.spawn((
        Camera3dBundle {
            transform: Transform::from_translation(Vec3::new(0.0, 0.0, 5.0))
                .looking_at(Vec3::new(1.0, 0.0, 0.0), Vec3::Y),
            camera_3d: Camera3d {
                clear_color: ClearColorConfig::Default,
                ..default()
            },
            camera: Camera {
                order: 0,
                ..default()
            },
            ..default()
        },
        // Add the setting to the camera.
        // This component is also used to determine on which camera to run the post processing effect.
        PostProcessSettings {
            intensity: 0.02,
            ..default()
        },
    ));

This effectively creates two cameras, one rendering on top of the other without a clear color.

This looks fine at first glance, I fully expect the chromatic aberration to be on both cubes (at least I think that should be expected). However, the chromatic aberration on the left cube is twice as strong as on the right, leading me to believe that it is applied twice, even though only one camera has the component.
In my own experiments, I also noticed that all cameras are affected, even those that render to textures.
There is another comment in the example that I don't know how to interpret:
https://github.com/bevyengine/bevy/blob/18d001d27ce475388743d9825f2833ce51f5b20d/examples/shader/post_processing.rs#L131-L134
This sounds like what I want, but I don't know how to actually do this.

You are absolutely right. I forgot to make sure it only runs on a specific camera when writing the example.
You can add the PostProcessSettings component to the ViewQuery and it will correctly only run for that camera.
    type ViewQuery = (
        &'static ViewTarget,
        // This makes sure the node only runs on cameras with the PostProcessSettings component
        &'static PostProcessSettings,
    );

// ...
// make sure you also modify the run() function
(view_target, _post_process_settings): QueryItem<Self::ViewQuery>,


Error using ng config cli.packageManager yarn

Bug Report or Feature Request (mark with an x)

X Feature request #10558
Versions

NPM:  6.1.0
Angular CLI: 6.0.8
Node: 10.4.0
OS: darwin x64 High Sierra
Yarn: 1.7.0
Repro steps

Use Yarn instead of NPM
Run:
ng set --global packageManager=yarn
Log message returned:
get/set have been deprecated in favor of the config command.
Run:
ng config cli.packageManager yarn
Returns the following error:
Cannot read property 'value' of null TypeError: Cannot read property 'value' of null     at ConfigCommand.set (/usr/local/lib/node_modules/@angular/cli/commands/config.js:207:36)     at ConfigCommand.run (/usr/local/lib/node_modules/@angular/cli/commands/config.js:181:18)     at /usr/local/lib/node_modules/@angular/cli/models/command-runner.js:278:30     at Generator.next (<anonymous>)     at fulfilled (/usr/local/lib/node_modules/@angular/cli/models/command-runner.js:4:58)
The log given by the failure

Cannot read property 'value' of null TypeError: Cannot read property 'value' of null     at ConfigCommand.set (/usr/local/lib/node_modules/@angular/cli/commands/config.js:207:36)     at ConfigCommand.run (/usr/local/lib/node_modules/@angular/cli/commands/config.js:181:18)     at /usr/local/lib/node_modules/@angular/cli/models/command-runner.js:278:30     at Generator.next (<anonymous>)     at fulfilled (/usr/local/lib/node_modules/@angular/cli/models/command-runner.js:4:58)
Desired functionality

I'm new to using Yarn.  I usually use npm but my team uses yarn instead of NPM, so I need to configure this change. I don't know if this is a bug or something else is wrong based on the error message.
Mention any other details that might be useful

I found this feature update, but I don't know if anyone else has experienced the issue I am having.
Issue #10558

Just to add to the issue, I am experience the same identical error when attempting to change the packageManager to yarn.
Versions:
Angular CLI: 6.0.8
Node: 10.4.1
npm: 6.0.8
yarn: 1.7.0
PS C:\Windows\system32> ng config cli.packageManager yarn
Cannot read property 'value' of null TypeError: Cannot read property 'value' of null     at ConfigCommand.set (C:\Users\Redirection\goldip2\AppData\npm\node_modules\@angular\cli\commands\config.js:207:36)     at ConfigCommand.run (C:\Users\Redirection\goldip2\AppData\npm\node_modules\@angular\cli\commands\config.js:181:18)     at C:\Users\Redirection\goldip2\AppData\npm\node_modules\@angular\cli\models\command-runner.js:278:30     at Generator.next (<anonymous>)     at fulfilled (C:\Users\Redirection\goldip2\AppData\npm\node_modules\@angular\cli\models\command-runner.js:4:58)

Try ng config --global cli.packageManager yarn.
Unfortunately, the error message is currently not very useful and error messages in general are in the process of being improved.

This does not work with the latest cli.

This didn’t work with Angular CLI 6.2.1
~ ng config -g cli.packageManager yarn
This command can only be run inside of a CLI project.

Confirm that, have to create a junk project (ng new junk), let it fails because no NPM, go into "junk" folder newly created, type "ng config -g cli.packageManager yarn", go to parent folder, delete "junk" folder, and now I can use "ng new myProjet" with yarn. Bad.


terminal doesn't have scrollbar

VSCode Version:1.2
OS Version:win10 10240

Steps to Reproduce:

ctrl+`
terminal doesn't have scrollbar (not working on my laptop except  mouse scroll)

Duplicate of https://github.com/Microsoft/vscode/issues/6602


WhatsApp won't remember credentials

Description
Everytime I close the app, it will ask me to log in again when the app is launched again.
What am I missing?
Details

Are you nativefying a public website?

Feature request? Have you looked at nativefier --help to see if an existing option could fit your needs?
Yes

Full nativefier command used to build your app: nativefier --name "WhatsApp Desktop" "https://web.whatsapp.com"  --platform "linux" --icon "/home/caligaris/whatsapp.png" --show-menu-bar --disable-dev-tools --single-instance

Version of Nativefier: v7.6.7

Version of node.js: v8.10.0

OS: Ubuntu 18.04

Same here with waveapps

I believe that this issue is solved. I just tried to reproduce the issue and it actually works fine.

"I believe that this issue is solved. I just tried to reproduce the issue and it actually works fine."

@JMarteF thanks for the feedback! Can you share your nativefier options (full line used to nativefy Whatsapp), in case you're doing something the others aren't?
@bakeiro @TheGoddessInari @giovannicaligaris do you confirm the problem is gone?

It looks like WhatsApp changed the default storage away from session.

@TheGoddessInari can you precise what you mean by "session"? SessionStorage?

@ronjouch Yes, sessionStorage. Electron doesn't offer an easy way to persist sessionStorage as Chrome does by default, which some badly-made web apps, like formerly WhatsApp, T-Mobile DIGITS, and others, depend on being just another localStorage.
Convincing the apps themselves to change is generally difficult with webdevs thinking that Chrome = "the web".


Windows 10 -> link Sabaki with SGF Files

Dear yishn,
your program is really awesome.
I love it!
Thanks for that!
My problem is I wanna link all my sgf or other GO Files with Sabaki. So when I click them it automatically opens up. But I always get a "wrong parameter" error.
Should this work? Or is it not supposed to work?
Thanks in advance!

It is supposed to work, of course. Can you send a screenshot of what error you get?

Of course :)
Here you go!
http://imgur.com/a/ouR8A

I just uninstalled it and then reinstalled it.. i connect it again to the SGF Files.. now it seems working :) This can be closed! 👍 Thanks anyways!

I'm glad this is resolved!


Fix WASM-client disconnection error

In this PR, we add a fix that solves the error that the wasm-client freezes when attempting to close the sockets when disconnecting.
I've also added a commit to add the gbn subsystem to the wasm-client's logger. This simplifies debugging errors quite a lot. Let me know if you don't think that commit is worth including though :)

Updated the PR with adding a callback that's executed once the disconnection has completed, so that the frontend only updates once we're fully disconnected.
I noticed that for checking wether we are connected or not, we instead use a ticker that checks the IsConnected func every 200ms to determine wether the connection setup has completed or not. If you'd like me to do the same for the disconnected version, I could of course do so, so let me know if I should change this :)!

Thanks a lot for the reviews @guggero & @jamaljsr!
@jamaljsr, just wanted to check, is setting the debuglevel to info when you start the wasm-client an ok way for you app-devs to get rid of the extra gbn subsystem logs? I.e. with the example-server the log level is set to trace which is why the extra log lines are shown. https://github.com/lightninglabs/lightning-node-connect/blob/d8c9f92056e2b5a0c908514b2159d01738f7afb7/example/index.html#L54C14-L54C32
But in a real setting when using the wasm-client, I'd imagine that the debuglevel would be set to info, which removes the log extra lines you're referring to. Or does setting the debuglevel to info remove too much useful information for you?

But in a real setting when using the wasm-client, I'd imagine that the debuglevel would be set to info, which removes the log extra lines you're referring to. Or does setting the debuglevel to info remove too much useful information for you?

Ah yes, great suggestion about tweaking the debuglevel param. While setting it to info wouldn't be ideal because it produces no output at all, I was able to fiddle with it and find the optimal level of logging with debuglevel=debug,GOBN=info,GRPC=info. This is even better than what we had previously as it was a bit too verbose before. This is perfect, thanks! 🙌

Updated the PR with adding a callback that's executed once the disconnection has completed, so that the frontend only updates once we're fully disconnected.

I prefer the callback approach over polling, so it looks good to me. My only nitpick would be to rename disconnected to onDisconnect

Ah yes, great suggestion about tweaking the debuglevel param. While setting it to info wouldn't be ideal because it produces no output at all, I was able to fiddle with it and find the optimal level of logging with debuglevel=debug,GOBN=info,GRPC=info. This is even better than what we had previously as it was a bit too verbose before. This is perfect, thanks! 🙌

Awesome :rocket!

I prefer the callback approach over polling, so it looks good to me. My only nitpick would be to rename disconnected to onDisconnect

Great ok! I've renamed the callback to onDisconnect with the latest push. I saw that we also define some callbacks as config options instead: https://github.com/lightninglabs/lightning-node-connect/blob/d8c9f92056e2b5a0c908514b2159d01738f7afb7/example/index.html#L56-L58
If you'd like me to do the same for the onDisconnect callback let me know, but I found this approach I've implemented in the PR a bit cleaner.

Just added some docs to the Disconnect to include info that the function expects that the onDisconnect is passed as an argument. I also reformatted the line breaks in the function.


Tap Bing Ads - Target Redshift

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2659
Originally created by @mohsin9 on 2021-03-18 22:11:09

meltano               | INFO Incremental state has been updated at 2021-03-18 20:23:45.698878.
meltano               | DEBUG Incremental state: {'bookmarks': {'accounts': {'last_record': '2021-02-23T21:29:00.960000+00:00'}, '141559394_ad_performance_report': {'request_id': '1238132885391'}, '141559394_keyword_performance_report': {'request_id': '1238132885442'}}}
meltano               | DEBUG Deleted configuration at /home/ubuntu/dev-meltano/bing-to-postgres/meltano-bing-topostgres/.meltano/run/elt/2021-03-18T202303--tap-bing-ads--target-redshift/bff5d897-cf63-4ce1-ac38-68d9005a9e8b/target.config.json
meltano               | DEBUG Deleted configuration at /home/ubuntu/dev-meltano/bing-to-postgres/meltano-bing-topostgres/.meltano/run/elt/2021-03-18T202303--tap-bing-ads--target-redshift/bff5d897-cf63-4ce1-ac38-68d9005a9e8b/tap.config.json
meltano               | ERROR Extraction failed (1): }
meltano               | DEBUG ELT could not be completed: Extractor failed
Traceback (most recent call last):
  File "/home/ubuntu/.local/lib/python3.8/site-packages/meltano/cli/elt.py", line 236, in _run_elt
    await _run_extract_load(elt_context, output_logger)
  File "/home/ubuntu/.local/lib/python3.8/site-packages/meltano/cli/elt.py", line 274, in _run_extract_load
    await singer_runner.run(
  File "/home/ubuntu/.local/lib/python3.8/site-packages/meltano/core/runner/singer.py", line 253, in run
    await self.invoke(
  File "/home/ubuntu/.local/lib/python3.8/site-packages/meltano/core/runner/singer.py", line 227, in invoke
    raise RunnerError("Extractor failed", {PluginType.EXTRACTORS: tap_code})
meltano.core.runner.RunnerError: Extractor failed
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
  File "/home/ubuntu/.local/lib/python3.8/site-packages/meltano/cli/elt.py", line 224, in _redirect_output
    yield
  File "/home/ubuntu/.local/lib/python3.8/site-packages/meltano/cli/elt.py", line 245, in _run_elt
    raise CliError(f"ELT could not be completed: {err}") from err
meltano.cli.utils.CliError: ELT could not be completed: Extractor failed
meltano               | ELT could not be completed: Extractor failed
[2021-03-18 20:23:45,815] [31668|MainThread|meltano.cli.utils] [DEBUG] ELT could not be completed: Extractor failed
Traceback (most recent call last):
  File "/home/ubuntu/.local/lib/python3.8/site-packages/meltano/cli/elt.py", line 236, in _run_elt
    await _run_extract_load(elt_context, output_logger)
  File "/home/ubuntu/.local/lib/python3.8/site-packages/meltano/cli/elt.py", line 274, in _run_extract_load
    await singer_runner.run(
  File "/home/ubuntu/.local/lib/python3.8/site-packages/meltano/core/runner/singer.py", line 253, in run
    await self.invoke(
  File "/home/ubuntu/.local/lib/python3.8/site-packages/meltano/core/runner/singer.py", line 227, in invoke
    raise RunnerError("Extractor failed", {PluginType.EXTRACTORS: tap_code})
meltano.core.runner.RunnerError: Extractor failed
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
  File "/home/ubuntu/.local/lib/python3.8/site-packages/meltano/cli/__init__.py", line 43, in main
    cli(obj={"project": None})
  File "/usr/lib/python3/dist-packages/click/core.py", line 764, in __call__
    return self.main(*args, **kwargs)
  File "/usr/lib/python3/dist-packages/click/core.py", line 717, in main
    rv = self.invoke(ctx)
  File "/usr/lib/python3/dist-packages/click/core.py", line 1137, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/lib/python3/dist-packages/click/core.py", line 956, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/lib/python3/dist-packages/click/core.py", line 555, in invoke
    return callback(*args, **kwargs)
  File "/home/ubuntu/.local/lib/python3.8/site-packages/meltano/cli/params.py", line 23, in decorate
    return func(*args, **kwargs)
  File "/home/ubuntu/.local/lib/python3.8/site-packages/meltano/cli/params.py", line 57, in decorate
    func(project, *args, **kwargs)
  File "/home/ubuntu/.local/lib/python3.8/site-packages/meltano/cli/elt.py", line 136, in elt
    run_async(_run_job(project, job, session, context_builder, force=force))
  File "/home/ubuntu/.local/lib/python3.8/site-packages/meltano/cli/utils.py", line 49, in run_async
    loop.run_until_complete(coro)
  File "/usr/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
    return future.result()
  File "/home/ubuntu/.local/lib/python3.8/site-packages/meltano/cli/elt.py", line 213, in _run_job
    await _run_elt(project, context_builder, output_logger)
  File "/home/ubuntu/.local/lib/python3.8/site-packages/meltano/cli/elt.py", line 245, in _run_elt
    raise CliError(f"ELT could not be completed: {err}") from err
meltano.cli.utils.CliError: ELT could not be completed: Extractor failed
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
  File "/home/ubuntu/.local/lib/python3.8/site-packages/meltano/cli/__init__.py", line 51, in main
    raise CliError(str(err)) from err
meltano.cli.utils.CliError: ELT could not be completed: Extractor failed
ELT could not be completed: Extractor failed

Extractor failed
Addl info in slack: https://meltano.slack.com/archives/C013EKWA2Q1/p1616105486054000?thread_ts=1616087161.050400&cid=C013EKWA2Q1

View 2 previous comments from the original issue on GitLab


Ignored accounts

This PR does a small refactoring to remove some duplication (09f582e) and then adds an IGNORED_ACCOUNTS environment variable that is a comma separated list of GitHub usernames to ignore.
This is necessary because there are currently a few very abusive accounts (spammy, DMCA takedowns) that have installed several probot apps on hundreds or thousands of repositories, which is causing some pain.

I'll be reporting this issue to teams at GitHub, but here's what I think would make this change unnecessary:

The ability for an app to uninstall itself from an account (in the case of abuse, lack of payment, etc)
Some indication via the API that the account is spammy


Tailored error reporting when dynamic casts have nonsensical type args

This inserts a suitably named function into the stack trace whenever a dynamic cast failure involves a NULL source or target type. Very often, crash logs include backtraces with function names but no log output; with this change, such a backtrace might look like the following -- note TARGET_TYPE_NULL in the function name here to mark the missing type information:
 frame #0: __pthread_kill + 8
 frame #1: pthread_kill + 288
 frame #2: abort + 128
 frame #3: swift::fatalErrorv()
 frame #4: swift::fatalError()
 frame #5: swift_dynamicCastFailure_TARGET_TYPE_NULL()
 frame #6: swift::swift_dynamicCastFailure()
 frame #7: ::swift_dynamicCast()

Resolves rdar://130630157

@swift-ci Please test

Cool!!


Add wall state

Closes #17
Tho, the character doesn't perform a nice and rewarding arc when wall jumping just yet, this is something that will really improve the feeling of the wall jump. To achieve that I think we must somehow accelerate the Air state towards the Wall if it transitioned from the Wall state.
I couldn't think of a way to achieve that without messing things around, so I rather get your feedback on that and see what we can do.

Ah, I finally see what you meant with treating the wall as a ground: adding gravity towards the wall. It makes sense if you want the player to be able to climb on a single wall. Let's see, after testing, for the full game at least I'd do the opposite: push the player away from the wall, so you use wall jump more like a double jump.
The idea, design-wise, is to always favor the hook and limit the other abilities.

push the player away from the wall

This is quite what is happening currently if you try to climb the wall, wall jumping, you will have a bad time, you basically move small bits vertically, so, efficiency wise, is always better to favor the hook.
For vertical corridors tho, e.g. 2 parallel walls, it works nicely

About the wall state's feel:

If you press the direction away from the wall, the player falls too fast. You want to have the player still stick for a few frames to give them the time to jump in the opposite direction. You can use a timer for that. It just needs to be 0.1 seconds or less. Right now if you press left and jump, or right and jump, you'll often miss the jump as you start to fall.
As I wrote in the comments above, we can't control the slide, and the character is sliding too slowly to me. Also, even if you hit a wall during a fall, there's no inertia: the character will stick to the wall instantly and slow down a lot.

About wall jumps:

We could have an optional gravity modifier in the air state or a wall jump sub-state. But as it makes it easier to climb walls, I'd actually go in the opposite direction game design-wise. I'd rather make the wall jump go less up and have more horizontal thrust. That's how the jump works in the game: you don't jump high, it mostly helps you hop over gaps and go farther forward horizontally. Because the hook gives you the verticality.

Long review, sorry for that! But talking about game feel makes for long comments 😉

Alright, PR updated, lemme know if it seems nice now so I can squash everything :grin:

So I checked your work too, sorry I couldn't get into the code, it's getting late here and I still want to finish some things so I just checked feeling of the gameplay.

it seems there's a bug, if you jump on a wall, say on a wall on the right side for this example, and you release holding move_right (D) after a few milliseconds, you can see the Player actually jumps up the wall, might have to do with move_and_slide
I think the character falling too slowly still applies
more on the feeling side, I'd say that when we jump the wall we should have a horizontal motion on the opposite direction for a few hundreds of a second in which the Player isn't able to move back towards the wall. At the moment the Player probably goes into the Air state as soon as it leaves the wall and you can move the Player back towards the wall ever so slightly, but I think we should prohibit this for a short amount of time, the idea to wall jump I think is to propel the Player in the opposite direction (it's easier to jump horizontally than vertically in reality), but we'll have to see what Nathan says here as well

The wall jump feels a lot better already!

it seems there's a bug, if you jump on a wall, say on a wall on the right side for this example, and you release move_right (D) after a few milliseconds, you can see the Player actually jumps up the wall

Confirmed

I think the character falling too slowly still applies [...] the player should be able to fall with friction for a short amount of time and then transition to regular fall

As you slide down and you won't jump up that much from a wall, you already have a short timeframe to perform a useful wall jump. Keeping friction is completely fine to me.
My main issue with it is the timing is not reliable. That's the problem when you rely on arbitrary values like "friction" to achieve the effect. It can feel okay in some cases, but you can't control precisely for how long the player is going to stick to the wall. I'll have to test it more to pinpoint ways I think we can improve it. But for now it's time to merge.


Tiktok, Qtweb for web extractor

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm requesting a feature unrelated to a specific site
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2023.02.17 (update instructions) or later (specify commit)
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
Hi there, I have a suggestion. I'm wondering if it's possible to use the QTweb engine to download TikTok videos. It appears that the extractor you're using may not extract TikTok profiles because it doesn't load all the necessary requirements to scrape the website. I've heard that 4ktokkit uses the QTengine to download TikTok videos, so maybe you could try incorporating it into yt-dlp.
Also, it would be great to have an option to download videos without the watermark and in an uncompressed format, similar to how it's done on this website: https://musicaldown.com/en.
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

#4996


Fix typo in RTTI declaration

Just fixing a small typo in the RTTI declaration of VisualInteractiveObject I discovered when toying around with the RTTI macros. Funny enough, it worked before.

Can we make this being detected by the compiler somehow?

It was in fact discovered by the compiler as I changed the REGOTH_DECLARE_RTTI macro. The only possible way that comes into my mind (and which triggered it here for me) is accessing a member/method of the class passed to the macro. The compiler then bails out when you use the wrong class (whos header is hopefully not included). Something simple like
using _TypeCheck = decltype(&classname::getRTTI);

at the end of the macro works, however it is equally ugly.

Can you add something like this to the PR?

I added something like this with the latest commit and it's not even that ugly. It basically makes sure, that getRTTI has the same return type as getRTTIStatic while checking if the passed class name is known in the current file.

Good job! That's indeed a very nice solution.


all: add support for FreeBSD 12

This is a meta bug for tracking the status of the issues for supporting FreeBSD 12. FreeBSD 12, which will be released in CY18, will contain a few breaking changes on ABI and a few packages in standard library (and probably x/sys/unix repository) need to find out some good way to fill the ABI gap between FreeBSD 11 or below and FreeBSD 12 or above.

#22448 os, syscall, x/sys/unix: add support for FreeBSD 12

I compiled successfully Go 1.10 on FreeBSD 12 / arm. Basic programs seem to work for me: is there anything notably broken?

@paulzhol,
Can you please file a separate issue for your concern about kevent?

@mikioh I don't think there's a need for another issue, core doesn't use EVFILT_TIMER so the new absolute timer API won't be used anyhow. We can continue using to the current (COMPAT_FREEBSD11) sysnum on all versions.

@mikioh I've updated the wiki. I see there's a syscall section in doc/go1.12.html (autogenerated?) from Brad's 'RELNOTE=FreeBSD breakages in syscall, use build tags' comment in the CL. I'll update it as well.


getText from pinView

hi,
how can I get text or char from pinView to compare written text and my default pin.Thanks in advance

@ozgekaplan try this, mDefaultPin.equals(pinView.getText().toString())


Adding fields to Role and Task

add Role.gather_facts
add Task.vars

Looks good to me. I think we should merge it. Do we want one more pair of eyes or should I merge?

+1 to merge


the results of AMASS are very different trained on torch1.6 and torch1.7

I am very confused about the AMASS results on subject 50002 trained on different versions of pytorch.
According to your environments, it suggests to install pytorch==1.7.0. However the results trained on pytorch1.7 is  less than the results I train on pytorch==1.6.0 significantly. I am very confused about it.
the training loss, and evaluation IoU is shown as(blue is trained on torch==1.6.0, and red is trained on 1.7.0):

Looking forward to your reply.

I find the main issue is the broyden method can not converge to cvg_thresh(1e-5). Many (xd_opt -xd_tgt) coverage to 1e-4 trained on RTX3090, pytorch1.7.0. I do not know how to solve it, could you give me some suggestion?

Hi @lingtengqiu, I've actually encountered the same problem on RTX3080. Later I switched to RTX2080 and haven't gotten the problem anymore. It seems to be a low-level problem and even hardware dependent, I am sorry that I don't really know how to solve this for RTX30xx.
The training curve for PyTorch1.6 seems good - is it also trained with RTX3090? and how does the result look qualitatively? if the qualitative result looks good then I would suggest just using PyTorch1.6. If there are incompatibilities between the code and PyTorch1.6 I am happy to resolve them.

Thanks for your detailed answer.
The training curve for pytorch1.6 is the result of the model trained on RTX2080 and A-100.  I find the main issue is the broyden method on RTX-30 series could not converge to 1e-5 (which leads to the number of valid verts is small, many pts are unable to train). If I modify the cvg_thresh to 1e-3, the training curve is good on RTX-30.  I guess it is the implementation of graphic computing is different on 3090 ,compared with other devices.

Thank you for the information. Then it's probably easiest to change GPU ( Given that A100 also works fine for you, I suspect that any model earlier than 8.6 should be okay https://en.wikipedia.org/wiki/CUDA#:~:text=A100 40GB%2C A30-,8.6,-GA102%2C GA104%2C GA106)
Setting a larger cvg_thresh (e.g. 1e-3) on RTX30xx might cause problems based on my experience - you might get artifacts (noisy spikes) on the output mesh because the loose cvg_thresh could introduce false correspondences.


[Logging] Cloudwatch output UI does not capture all required arguments

When creating a [Cluster]Output for Cloudwatch, Rancher does not currently expose all the form options to minimally make the output functional.
Missing are two string inputs, log_group_name and log_stream_name. Without these (minimum log_group_name, but while we're at it, let's add log_stream_name), the output does not function.

Output form as it exists today. Would need to add two string fields, "Log Group Name" and "Log Stream Name"

We should add a Configuration section below Access with two text fields - Log Group Name and Log Stream Name.
These should be wired to the cloudwatch config log_group_name and log_stream_name.
You can use edit as yaml to see where in the object structure these need to go.
We should verify that when entered, these values get wired in correctly (check with edit as yaml).

@nwmac Would a ticket need to be opened on the backend to make log_group_name a required field?

Rancher version v2.6-head (09/22/2021) 223ec88
On this build I see the new fields added to the UI. If I edit them I ended up with the following fields filled in the YAML:
  cloudwatch:
    aws_key_id:
      valueFrom:
        secretKeyRef:
          key: key
          name: izaackey
    aws_sec_key:
      valueFrom:
        secretKeyRef:
          key: secret
          name: izaacsecret
    log_group_name: rancher-izaac
    log_stream_name: rancher-izaac-stream
    region: us-east-2

Generated fluentbit.conf
<label @2da173fa9306e44bbc7e1f0f6f1b6d45>
  <match **>
    @type cloudwatch_logs
    @id clusterflow:cattle-logging-system:test1:clusteroutput:cattle-logging-system:test1
    aws_key_id <REDACTED>
    aws_sec_key <REDACTED>
    log_group_name rancher-izaac
    log_stream_name rancher-izaac-stream
    region us-east-2
    <buffer tag,time>
      @type file
      chunk_limit_size 8MB
      path /buffers/clusterflow:cattle-logging-system:test1:clusteroutput:cattle-logging-system:test1.*.buffer
      retry_forever true
      timekey 10m
      timekey_wait 10m
    </buffer>
  </match>
</label>

v2.6.0 UI

v2.6-head UI

Checked both ClusterOutput and Output.


 With git 2.1.4 I also see this problem:
% git clone git@github.com:thoughtbot/clearance.git
Cloning into 'clearance'...
remote: Counting objects: 13889, done.
remote: Compressing objects: 100% (6/6), done.
error: object 5e82b08e97aa30596775a703591ced10bb718961:contains zero-padded file modes
fatal: Error in object
fatal: index-pack failed

StackOverflow recommends a git-fast-export pipe git-fast-import, with the warning that it will change SHAs.

cf. https://github.com/jasmine/jasmine/issues/1052

I don't know what "contains zero-padded file modes" means and why its a problem? Or why a dangling commit would be a problem? WIthout knowing this it's hard to judge if rewriting history to solve this is appropriate.

Forgot to add, which I discovered later, that this is what causes it to fail when I clone a fresh repo:
[transfer]
  fsckObjects = true
[fetch]
  fsckObjects = true
[receive]
  fsckObjects = true

As far as I can tell, the only problem with zero-padded files is that if you have two identical files one with zero-padded file modes and one without, they will have different SHAs.
I don't think this is worth rewriting the whole repo for.

I don't think this is worth re-writing history for. If someone else has another fix or feels strongly that this should be fixed, please let me know.