NIFI-8116 The old peers do not be deleted when the URIs of the remote…

… process group are changed
Description of PR
https://issues.apache.org/jira/browse/NIFI-8116
When we start transmitting for RPG, Peer Persistence will restore the peers retrieved from disk. It only checks whether the protocol has changed but it doesn't check the URIs.
If the user changes the URIs to another instance, RPG not only keeps the old instance but also add the new instance to the state. the RPG will have two different instances at the same time and show the errors when transmission.
For this PR, I stored additional information into persistence files, this is used to check if the current target instance is matches with state persistence file in the disk.
For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[X] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[X] Has your PR been rebased against the latest commit within the target branch (typically main)?

[X] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[X] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[X] Have you written or updated unit tests to verify your changes?
[X] Have you verified that the full build is successful on JDK 8?
[X] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

Thanks for the fix @s9514171! I had a few concerns that i commented inline but i think they will be very simple to address. I was able to replicate the issue, though, so it means it'll be an easy thing to verify. Thanks!

I have added the prefix when writing to the state files and checked the pattern when restoring the peers
Thanks for your view and suggestion 👍

Thanks for the fix @s9514171! I had a few concerns that i commented inline but i think they will be very simple to address. I was able to replicate the issue, though, so it means it'll be an easy thing to verify. Thanks!

I have added the prefix when writing to the state files and checked the pattern when restoring the peers
Thanks for your view and suggestion 👍

Thanks @s9514171. This looks good to me now. I was able to test and everything looks good. +1 merged to main.

Thanks @s9514171. This looks good to me now. I was able to test and everything looks good. +1 merged to main.


Add "Local Storage Operator" subcomponent to hardcoded list of "extras"

Codecov Report

Merging #300 (9fb68b1) into master (7c70d79) will decrease coverage by 0.12%.
The diff coverage is 25.00%.

:exclamation: Current head 9fb68b1 differs from pull request most recent head 1a7983a. Consider uploading reports for the commit 1a7983a to get more accurate results

@@            Coverage Diff             @@
##           master     #300      +/-   ##
==========================================
- Coverage   30.65%   30.53%   -0.13%
==========================================
  Files          55       57       +2
  Lines        5349     5470     +121
  Branches     1179     1187       +8
==========================================
+ Hits         1640     1670      +30
- Misses       3608     3699      +91
  Partials      101      101

Impacted Files
Coverage Δ

elliottlib/cli/__main__.py
0.00% <0.00%> (ø)

elliottlib/cli/attach_bugs_cli.py
0.00% <0.00%> (ø)

elliottlib/cli/create_placeholder_cli.py
0.00% <0.00%> (ø)

elliottlib/cli/create_textonly_cli.py
0.00% <0.00%> (ø)

elliottlib/cli/find_bugs_qe_cli.py
0.00% <0.00%> (ø)

elliottlib/cli/verify_attached_bugs_cli.py
0.00% <0.00%> (ø)

elliottlib/openshiftclient.py
0.00% <ø> (-14.64%)
:arrow_down:

elliottlib/runtime.py
15.47% <ø> (ø)

elliottlib/tarball_sources.py
32.22% <ø> (ø)

elliottlib/util.py
38.46% <ø> (-2.37%)
:arrow_down:

... and 7 more

Continue to review full report at Codecov.

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


Fix small problems in use_case test

Presently,

Fix the problems that appeared in the tests. Now all tests should pass.
Add one more test in LoginInteractorTest for coverage.
Add one comment to SignupInteractorTest for how to use this test.

Update for test coverage. Still waiting for the connection to check some tests.


unnecesary ? statement

New Pull Request Checklist

[X] I have read the Documentation
[X] I have searched for a similar pull request in the project and found none
[X] I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)
[X] I have added the required tests to prove the fix/feature I am adding
[X] I have updated the documentation (if necessary)
[X] I have run the tests and they pass

This merge request fixes / refers to the following issues: ...
Pull Request Description
I just remove some unnecessary or implicit ? statements as well a unnecessary wrapper

Is the package alive too?

The original dio package seems to be mantained again: https://github.com/flutterchina/dio/issues/1234. Maybe you could send your PR to them too?


Missing JSON file info in case of errors

In case of errors while loading the configuration, the path and name of the affected JSON file is not printed to debug log file any more:
ERROR - Slicing is not supported! Please update the expression '${params.global.param1}[-1]'.
UNKNOWN - Loading of JSON configuration file failed!

ERROR - Verification against JSON schema failed: 'Additional properties are not allowed ('testdict4' was unexpected)'. Please put the additional params into 'params': { 'global': {...}
UNKNOWN - Verification against JSON schema failed: 'Additional properties are not allowed ('testdict4' was unexpected)'. Please put the additional params into 'params': { 'global': {...}

ERROR - A substitution in key names is not allowed! Please update the key name "par${...}am2"
UNKNOWN - Loading of JSON configuration file failed!

This should be restored. Expected is something like:
ERROR - Expecting property name enclosed in double quotes: line 18 column 46 (char 598)
ERROR - Nearby: '... "]},\n                           "019" : {"020" : {: : "022"}}\n         ...'
ERROR - In file: 'D:/tmp_files/config/testconfig.json'
UNKNOWN - Loading of JSON configuration file failed!

Hello Holger,
An error message like:
ERROR - Expecting property name enclosed in double quotes: line 18 column 46 (char 598)
ERROR - Nearby: '... "]},\n                           "019" : {"020" : {: : "022"}}\n         ...'

is just printed if there is an syntax error in a JSON configuration file while loading a configuration file.
In the case you mentioned in this ticket, the error is raised by our JsonPreprocessor package that Slicing is not supported! Please update the expression '${params.global.param1}[-1]'.
So, in this situation we just need to log out the path of the configuration file if needed.
Thank you,
Son

Hi Son,
I wrote "... something like...". This is an example only. Simply print out the information that is available. And available should be at least the affected JSON file (like in the past).

Hello Holger,
I pushed the change to stabi branch to update error log message when loaded configuration failed.
I also update the reference logs of selftest.
Thank you,
Son

Hi Son,
in log files I still get errors like
ERROR - Slicing is not supported! Please update the expression '${stringval}[1:]'.
UNKNOWN - Unable to load the test configuration. The test execution will be aborted!

With path and name of affected JSON file are still missing.
Some TestsuitesManagement/JsonPreprocessor versions ago every error in a configuration file caused to print path and name of the affected configuration file to log file. In between this disappeared.
You should check what went wrong in the code within the last weeks and you should restore the missing information.

Hello Holger,
Thank you for your finding! I pushed commit 59946bae1b9 to solve your review comment.
Thank you,
Son

Hooray - the missing file is back!

Released with RobotFramework AIO 0.12.0


Wrap long array and slice patterns.

Closes #4530.

Thank you! The changes themselves are fine, but this would constitute a "breaking" formatting change (since basically everything does), so we'll need to version gate the new formatting and leave the old path as-is


North carolina

Load process for NC, closes #55

Going to close this since it is superseded by the PR on the datamodel1.1 branch


More Postgres fixes to support AR 7.1

Hi @enebo
Here you have more fixes for postgres, the failing tests are under 100
failing tests are in attached file
postgres_rails_71.txt

@JesseChavez thanks for the updates.  It is getting closer!


Does'nt work in material-ui Drawer

I'm trying to use it in material-ui drawer but it is not working
https://codesandbox.io/s/lpp9xn4j0l

Also noticed this, anyone know how to solve this?


Webhook error: "Webhook handler failed. View logs."

Stirpe webhook gives error 400 with message: Webhook error: "Webhook handler failed. View logs."  Stripe's logs doesn't show that anything would be wrong. Api-keys and secrets etc are double-checked. Stripe and supabase setup done according to the README.md of this repository. My other production app using this template with Stripe webhooks also broke in prod.
Anyone have idea why?

Can you check your function log on Vercel?

My function logs on Vercel just say awaiting new requests, so probably no functions has even been ran there.

My function logs on Vercel just say awaiting new requests, so probably no functions has even been ran there.
@rannikkopyry
no. Vercel function logs only work in live mode, they are not installed. You should go back to the functions tab, and once you're there go to your website on a different tab and try to log in, check-out any product with price, then head back to your vercel functions tab and then you'll see new messages. Those you can paste here to analyze.

hope this helps. btw did you solve the issue? or u still having trouble?

Still having trouble... here are the logs
Could not find a relationship between 'products' and 'prices' in the schema cache
{   code: '42P01',   details: null,   hint: null,   message: 'relation "public.prices" does not exist' }
{   code: '42P01',   details: null,   hint: null,   message: 'relation "public.products" does not exist' }
Don't know if you can say anything based off those, I'm kind of lost :D

I have exactly the same error. There is a duplicate thread here with some more info, but still it seems like something is wrong with the starter (webhooks?)

For me the solutions was that "supabase integration" was pointing to some other supabase project (wrong
NEXT_PUBLIC_SUPABASE_ANON_KEY, SUPABASE_SERVICE_ROLE_KEY, NEXT_PUBLIC_SUPABASE_URL)
Check those as they may not be updated properly if you have multiple supabase projects going on.

Thanks I checked the other thread and got a working solution from there. Needed to run the schema.sql in supabase sql editor. Link to the other thread for anyone with the same problem. https://github.com/vercel/nextjs-subscription-payments/issues/168


Module not found error in 0.2.0

It looks like I can't import vue2-timepicker in my Webpack project anymore:
ERROR in ./node_modules/vue2-timepicker/src/index.js
Module not found: Error: Can't resolve './vue-timepicker' in '[..]/node_modules/vue2-timepicker/src'
 @ ./node_modules/vue2-timepicker/src/index.js 1:0-44 2:15-28

However, changing the first line of /src/index.js to
import VueTimepicker from './vue-timepicker.vue'

Fixes the issue.
Thank you for this very useful package.

Thanks for the bug report. Fix released in v0.2.1


Provide a way to get configuration "into" MediaSupport

The MediaSupport class will need the ability to receive some kind of configuration at construction time.  The forcing function is #878 which requires configuration of some kind to "get to" a MessageBodyWriter, namely a ThrowableBodyWriter.

@romain-grecourt Feel free to chime in.


Change docker image

changed image to use ghcr.io one
add a simple PR test

Fixes #86

Why is this needed @gruebel ?


selectYear functionality not working

When I changed the year SelectYear method is not firing. Please help me with this issue.

same here
        $('#calendar').on('selectYear', function(event, activeYear) {              console.log(activeYear)         });
I have this and nothing happen :(


Merge conflict viewer unable to open file

Type: Bug
I initiated a git rebase -i master on my feature branch, open source control tab and click the first file under Merge Changes. It shows null and unable to open file. But clicking the open file icon correctly opens it.
VS Code version: Code 1.70.2 (e4503b30fc78200f846c62cf8091b76ff5547662, 2022-08-16T05:38:59.695Z)
OS version: Darwin arm64 21.4.0
Modes:

System Info

Item
Value

CPUs
Apple M1 Pro (10 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
5, 5, 5

Memory (System)
16.00GB (0.07GB free)

Process Argv
--crash-reporter-id de0a3a9e-7715-4efc-a88a-81303a63d000

Screen Reader
no

VM
0%

Extensions (11)

Extension
Author (truncated)
Version

emojisense
bie
0.9.1

solargraph
cas
0.23.0

vscode-eslint
dba
2.2.6

gitlens
eam
12.2.0

godot-tools
gee
1.3.1

vscode-docker
ms-
1.22.1

remote-containers
ms-
0.245.2

color-highlight
nau
2.5.0

ruby
reb
0.28.1

vscode-icons
vsc
11.16.0

vscode-ruby
win
0.28.0

(1 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vsdfh931cf:30280410
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411cf:30557515
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
pylanb8912:30545647
vsclangdc:30486549
c4g48928:30535728
hb751961:30553087
dsvsc012cf:30540253
azure-dev_surveyonecf:30548226
2144e591:30553903

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.71.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Upgraded to 1.71.0, bug is still there. Seems to be a file that was renamed. So opening the diff of the "removed" file should presumably result in This file has been deleted message while the "created" file should show the added diff.

Because I have a similiar issue, just 1.69.2 + merge instead of rebase. Working from windows on remote linux server.
All files are renamed either on source / target / both branches.
If you hit F1 and open "Developer: Show Window Log" whats the trace?
[2022-09-08 11:41:32.301] [renderer1] [error] Error: Die Datei wurde nicht gefunden.
    at g.handleSetInputError (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2716:96747)
    at async g.setInput (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2716:95987)
    at async l.doSetInput (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2714:6429)
    at async l.doOpenEditor (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2714:4791)
    at async l.openEditor (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2714:3783)
    at async vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2916:16970
    at async vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2716:26090
    at async Pt.open (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2724:49808)

main log shows a lot of:

[2022-09-08 11:22:04.320] [main] [info] Extension host with pid 4072 exited with code: 0, signal: null.

Is it the same for you?

/duplicate https://github.com/microsoft/vscode/issues/161321

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Move away from notebook tests in pyvespa

Implement integration tests in python files instead. We would like to move away from the nbdev library.

Fixed by #55


Regression: Unable to build when using env vars in app config

Expected Behavior
yarn --cwd packages/backend build succeeds
Actual Behavior
yarn run v1.22.18
$ backstage-cli package build
Building app separately because it is a bundled package
$ /Users/mkantzer/Drizly/backstage/node_modules/.bin/lerna --concurrency 4 run --scope app build
lerna notice cli v4.0.0
lerna notice filter including "app"
lerna info filter [ 'app' ]
lerna info Executing command in 1 package: "yarn run build"
lerna ERR! yarn run build exited 1 in 'app'
lerna ERR! yarn run build stdout:
$ backstage-cli package build
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
lerna ERR! yarn run build stderr:
(node:17192) [DEP0148] DeprecationWarning: Use of deprecated folder mapping "./" in the "exports" field module resolution of the package at /Users/mkantzer/Drizly/backstage/node_modules/yaml/package.json.
Update this package.json to use a subpath pattern like "./*".
(Use `node --trace-deprecation ...` to show where the warning was created)
Loaded config from app-config.yaml

Error: Configuration does not match schema

  Config must have required property 'baseUrl' { missingProperty=baseUrl } at /app
  Config must have required property 'baseUrl' { missingProperty=baseUrl } at /backend

error Command failed with exit code 1.
lerna ERR! yarn run build exited 1 in 'app'
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Command 'yarn' exited with code 1

error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Steps to Reproduce
Set app-config to something like:
app:
  baseUrl: ${BASE_URL}

backend:
  baseUrl: ${BASE_URL}

And then run yarn build.
Context
We're trying to supply values to app-config.yaml at runtime instead of build time, using Environment Variable Substitution. However, it looks like recent changes to the backstage-cli have introduced problems with this.
This used to be supported, with specific steps taken to allow for lax evals:

https://github.com/backstage/backstage/issues/5751
https://github.com/backstage/backstage/issues/5520

Possibly Related issues:

https://github.com/backstage/backstage/issues/11718

Additionally, the --lax flag is still supported by config:
yarn backstage-cli config:print --lax works, and prints a config YAML with x in place of the substituted env vars.

Sorry to hear that you're having issues, it does not look like the configuration file is available at build from the error.
You are changing directory which might be the cause of this. Try building with --config passing the full path app-config.yaml

And be sure to have BASE_URL set during the build!

So my intention is to not specify baseUrl until runtime, at which point I will specify it via the BASE_URL env var. I do not plan on having a value for it at build-time, as we'd like to re-use the container between environments.
Is this not supported? Am I misunderstanding the purpose of baseURL?

@mikekantzer-drizly The build needs a base URL set as it's part of the Webpack configuration. Only the path actually matters right now in practice though. You could set it to something like http://example.com if you really don't want any real data as part of the build

Sounds good, thanks!


RenderObject.debugCheckingIntrinsics mode should catch exceptions and explain what's going on

It's easy to get into situations where you throw during the RenderObject.debugCheckingIntrinsics mode.
It's hard to understand why your code is getting called in that mode.
We should catch all exceptions and include detailed information about what's going on and how to handle it.

cc @vlidholt

Does this still a valid issue?


unable to set background-color as none

Here is my code
new_style = {"background": "none", "background-color": "none"}

app = rx.App(style=new_style, title="OneCloud")

Still, it's giving background-color as background-color: var(--color-page-background)

I am directly rendering html page with rx.html() so need chakraui for specific page , is it is possible to disable ?

if you don't need chakra at all, try to set rx.App(overlay_component=None) and ensure you do not use any other chakra components in the app.
Currently the default overlay component shows a connection error banner when the websocket goes down and this relies on chakra.

@masenf How to disable for a particular page ?


[Play Store] Unable to publish apps on Play Store using Horologist

Play Console requires compileSdk to be less or equal than 33 for Wear apps but horologist libs are using compileSdk = 34, so every apk using horologist can't be published

Do you mean Play requires target sdk to be between 30 and 33?
If so, then it shouldn't be a problem that Horologist is using compileSdk = 34.
What's the error message you are seeing when trying to publish?


REQUEST: New membership for

GitHub Username
claude-zq
Requirements

[x] I have reviewed the community membership guidelines (https://github.com/cloudwego/community/blob/main/COMMUNITY_MEMBERSHIP.md)
[x] I have joined to the Slack Channel and Feishu Group Chat (for Chinese)
[x] I am contributing (any of the following apply: issues, discussions, PRs, reviews) to 1 or more CloudWeGo repositories

List of contributions to CloudWeGo
Hertz

Issue

[Proposal: add paseto middleware · Issue #618 · cloudwego/hertz (github.com)](https://github.com/cloudwego/hertz/issues/618)

Feature

[Time-sharing car rental system kit in the cloud-native era use hertz and kitex](https://github.com/cloudwego/biz-demo/blob/main/README.md)
[hertz-contrib/paseto: Paseto middleware for Hertz. (github.com)](https://github.com/hertz-contrib/paseto)
[feat: add consul example and fix some errors by Claude-Zq · Pull Request #33 · hertz-contrib/registry (github.com)](https://github.com/hertz-contrib/registry/pull/33)
[feat: add csrf example by Claude-Zq · Pull Request #60 · cloudwego/hertz-examples (github.com)](https://github.com/cloudwego/hertz-examples/pull/60)

Doc

[docs: add csrf docs by Claude-Zq · Pull Request #458 · cloudwego/cloudwego.github.io](https://github.com/cloudwego/cloudwego.github.io/pull/458)
[doc: update csrf docs by Claude-Zq · Pull Request #464 · cloudwego/cloudwego.github.io](https://github.com/cloudwego/cloudwego.github.io/pull/464)
[doc: add http2 docs by Claude-Zq · Pull Request #466 · cloudwego/cloudwego.github.io](https://github.com/cloudwego/cloudwego.github.io/pull/466)
[docs: add Etag middleware by Claude-Zq · Pull Request #529 · cloudwego/cloudwego.github.io](https://github.com/cloudwego/cloudwego.github.io/pull/529)
[docs: fix README example by Claude-Zq · Pull Request #2 · hertz-contrib/csrf (github.com)](https://github.com/hertz-contrib/csrf/pull/2)

Cr

https://github.com/cloudwego/community/pull/62/files
https://github.com/hertz-contrib/paseto/pull/2

Others

https://bytedance.feishu.cn/minutes/obcnoen1msed482zpf8f24bp

Thank you for your contribution.

@Claude-Zq  add freecar to the contributions, although the project is not in the official repositories, it is a good example of using hertz and kitex.


$$_gendir issue

Here's the repo https://github.com/scttcper/ngx-toastr
Angular cli is used to manage the tests/github pages/demo
Command used is ng github-pages:deploy --aot
package.json is in repo
I'm on osx with node v7.5.0 and have tried with both yarn and npm.
ERROR in ./src/$$_gendir/app/app.module.ngfactory.ts
Module not found: Error: Can't resolve '../../../lib/toast-component.ngfactory' in '/Users/scoope7/toastr-ng2/src/$$_gendir/app'
 @ ./src/$$_gendir/app/app.module.ngfactory.ts 37:0-67
 @ ./src/main.ts
 @ multi ./src/main.ts

Could be the same as or related to #4551

You do not state which version of Angular CLI you are using.
Note: as of version 1.0.0.beta.28.3 github-pages:deploy has been removed.
You have to use a different npm package in future.

@catull sorry. Its @angular/cli beta.30
I was unaware of the github-pages being removed. Either way, ng build --prod --aot throws the same error.

Could you please elaborate? Are you talking about node_modules?
What did you do to fix it?

Comigo o problema era o<EMAIL_ADDRESS>Atualizei para @1.2.7 e resolveu...

check your package.json has the same value like that
"@angular/animations": "4.0.2",
"@angular/common": "^4.0.2",
"@angular/compiler": "4.0.2",
"@angular/core": "4.0.2",
"@angular/forms": "4.0.2",
"@angular/http": "4.0.2",
"@angular/platform-browser": "4.0.2",
"@angular/platform-browser-dynamic": "4.0.2",
"@angular/router": "4.0.2",


Using the automatic carousel of Swiper Vue.js, even after the mouse is moved in, it will still carousel on its own

Check that this is really a bug

[X] I confirm

Reproduction link

Bug description
Using the automatic carousel of Swiper Vue.js, even after the mouse is moved in, it will still carousel on its own
Expected Behavior
Use the automatic carousel of Swiper Vue.js. After moving the mouse in, you want to stop the automatic carousel and continue with it when the mouse leaves
Actual Behavior
After the mouse is moved in, it still automatically rotates on its own
The code is as follows：
<template>

        {{ item.name }}:
        {{ item.value }}

      <swiper
        direction="vertical"
        :loop="true"
        :loopAdditionalSlides="1"
        :autoplay-delay="1000"
        :autoplay="true"
        :centeredSlides="true"
        :modules="[Autoplay, EffectFade]"
        @slideChange="onSlideChange"
        ref="swiperRef"
      >
        <swiper-slide v-for="(slide, index) in slides" :key="index">

{{ slide.title }}

        </swiper-slide>
      </swiper>

</template>

Mouse in error:swiperRef._value.stopAutoplay is not a function
Swiper version
11.0.5
Platform/Target and Browser Versions
window,Chrome is already the latest version Version 120.0.6099.130 (official version) (64 bit)
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[X] I'm willing to open a PR

Do you want to ask a question? Are you looking for support? Stack Overflow or Swiper Discussions is the best places for getting support

Please, don't use GitHub issues for questions


Randomize the RNG

Terrain generated from a build version is not randomly generated, while the terrain generated from the editor is.
Not sure as to the cause, yet, but this is to track the issue.

Terrain generated from the editor was apparently not randomly generated.  Issue is FastNoiseLite.seed is not randomized like RandomNumberGenerator.seed or Global.seed.


added RD niall-merrigan

Docs Build status updates of commit 8f00984:
:warning: Validation status: warnings

File
Status
Preview URL
Details

rd/index.yml
:warning:Warning

Details

rd/map.yml
:warning:Warning

Details

rd/media/profiles/niall-merrigan.jpg
:white_check_mark:Succeeded

rd/niall-merrigan.yml
:white_check_mark:Succeeded

rd/toc.yml
:white_check_mark:Succeeded

rd/index.yml

Line 0: [Warning] Invalid file link:(~/about).
Line 0: [Warning] Invalid file link:(~/CodeOfConduct).
Line 0: [Warning] Invalid file link:(~/map).

rd/map.yml

Line 0: [Warning] Invalid file link:(~/about).
Line 0: [Warning] Invalid file link:(~/CodeOfConduct).
Line 0: [Warning] Invalid file link:(~/map).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Daily command from another Discord bot

I'm trying to create a bot to send daily messages like $daily fractals for example but the GW2 Bot doesen't read the commands from my bot.
Do you know why it can be?

The bot ignores messages from other bots, fully intended. This is to prevent loops and abuse.
There's $daily notifier if you want the bot to post dailies every day at reset time.


Inconsistent types for route argument setting vs getting

According to it's docblock, RouteInterface::setArgument()'s $value argument needs to a string, whereas RouteInterface::getArgument() returns mixed, and accepts mixed type for its $default parameter.
Will be happy to provide a PR to resolve the inconsistency, but need to know which solution you prefer: should route arguments always be strings, or should we accept mixed for the value? The latter makes more sense to me - in my own case, I was trying to set a boolean flag on the route as argument (worked perfectly, except that static analysis complains about violating the parameter type).

It should be string... https://github.com/slimphp/Slim/blob/3.x/Slim/Interfaces/RouteInterface.php#L28
string | null, I suspect.


Make creates lots of prcessors and overloads server, which kills compilation

I am trying to compile racon.
make modules finishes fine.
I then tried "make tools" and got lots of "out of memory errors" and:
g++: internal compiler error: Killed (program cc1plus)
So then I tried:
make -j 1 tools
This should limit the jobs to 1; however, look at the attached screenshot -

What the....?
Compilation is being killed by the OS
How do I fix this?
Cheers
Mick

FYI I substituted all instances of "-j" with "-j 1" in the Makefile and this solved the issue

Whoops :-)
Thanks for reporting this.
I pushed a fix removing all -j entries from the Makefile.
Best regards,
Ivan.


Improve interface between Trainer and argparse

🚀 Feature
Current Trainer.add_argparse_args(parser_param) creates a new ArgumentParser object, passing the parser_param object as a parent to the newly created parser. Trainer.add_argparse_args() returns the new ArgumentParser object to its caller.
An improved interface would simply add all arguments to parser_param without creating a new object. To maintain backwards compatibility with almost all existing code, Trainer.add_argparse_args() may simply return the parser_param object.
Motivation
The current interface makes impossible using advanced argparse features like grouping and subparsing. Due to the huge number of parameters in Trainer, isolating them in a group (instead of leaving them intermixed with everything else) is a common use case.
Pitch
A small change in pytorch_lightning.utilities.argparse_utils.add_argparse_args, essentially removing the one line that creates a new ArgparseParser object:
parser = ArgumentParser(parents=[parent_parser], add_help=False,)

Alternatives
The only alternatives I could think of are horrible:

Clients subclassing Trainer and overriding add_argparse_args (breaking abstraction);
Clients calling Trainer.add_argparse_args with a clean parser and using protected members of ArgparseParser to copy the arguments onto a group or subparser of the actual parser (breaking abstraction in an even worse way).

Duplicate of https://github.com/PyTorchLightning/pytorch-lightning/issues/3076 (requires the same fix)
Note that we plan to migrate our argparse functionalities to use https://github.com/PyTorchLightning/pytorch-lightning/pull/4492. Please have a look to see if it fits your use case!

Sorry about the duplication — somehow I missed that issue in my searches.
Yep, the #4492 PR would make those questions moot.  It looks amazing! :-)


built: update lint version to 1.6.0

@rultor merge it.

@rultor merge it.

@rafamizes OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge it.

@rafamizes Oops, I failed. You can see the full log here (spent 17s)
+ '[' true = true ']'
+ cat
+ chmod a+x entry.sh
+ cat
+ echo 'export '\''head=git@github.com:dartoos-dev/json_cache.git'\''' ';' 'export '\''pull_id=24'\''' ';' 'export '\''fork=git@github.com:rafamizes/json_cache.git'\''' ';' 'export '\''fork_branch=17'\''' ';' 'export '\''head_branch=master'\''' ';' 'export '\''pull_title=built: update lint version to 1.6.0'\''' ';' 'export '\''author=rafamizes'\''' ';' 'flutter clean' ';' 'flutter pub get' ';' 'flutter test --no-pub' ';' 'flutter pub publish --dry-run' ';'
+ sensitive=()
+ rm -rf .gpg
+ cd repo
+ git remote add fork git@github.com:rafamizes/json_cache.git
+ git remote update
Fetching origin
Fetching fork
From github.com:rafamizes/json_cache
 * [new branch]      17         -> fork/17
 * [new branch]      master     -> fork/master
+ args=
+ '[' default == default ']'
+ args=' --ff'
+ '[' default == no ']'
+ '[' default == only ']'
+ export BRANCH=__rultor
+ BRANCH=__rultor
++ git show-branch __rultor
++ wc -l
+ '[' 0 -gt 0 ']'
+ git checkout -B __rultor fork/17
Switched to a new branch '__rultor'
Branch __rultor set up to track remote branch 17 from fork.
+ git checkout -B master origin/master
Switched to and reset branch 'master'
Branch master set up to track remote branch master from origin.
Your branch is up-to-date with 'origin/master'.
+ '[' true == true ']'
+ git checkout __rultor
Switched to branch '__rultor'
Your branch is up-to-date with 'fork/17'.
+ git rebase master
First, rewinding head to replay your work on top of it...
Applying: test: JsonCacheWrap, JsonCacheMem and JsonCacheFake.
Using index info to reconstruct a base tree...
M	CHANGELOG.md
M	README.md
M	analysis_options.yaml
M	lib/src/json_cache.dart
M	lib/src/json_cache_fake.dart
M	lib/src/json_cache_mem.dart
M	test/json_cache_mem_test.dart
M	test/json_cache_wrap_test.dart
Falling back to patching base and 3-way merge...
Auto-merging README.md
CONFLICT (content): Merge conflict in README.md
error: Failed to merge in the changes.
Patch failed at 0001 test: JsonCacheWrap, JsonCacheMem and JsonCacheFake.
The copy of the patch that failed is found in: .git/rebase-apply/patch

When you have resolved this problem, run "git rebase --continue".
If you prefer to skip this patch, run "git rebase --skip" instead.
To check out the original branch and stop rebasing, run "git rebase --abort".

'cid' file is absent, container wasn't started correctly


Add a TextDocumentContentProvider for source-generated files

Resolves https://github.com/dotnet/vscode-csharp/issues/6426
This allows the Roslyn LSP to return document references with a roslyn-source-generated URI, and we'll turn around and ask the LSP for those files when they're opened.
For now we don't worry about refreshing files if they've changed, just to keep this simple.
Requires https://github.com/dotnet/roslyn/pull/75180

We should definitely have a test for this. I think it is fine if it comes in a follow up.


Create addimage.py

feature to add images to gallery directly via command. (only admins can do it :P)

Glad you like it 🥰😎


New AC code. Manufacturer: Apton, Model: AFC-100T

ir-code.json.txt

Is there any problem with my code?

You need to repeat all fan modes and temperatures for dry mode

my dry mode only has auto, doesn't have level1,level2, level3

I know, but you have to repeat them. Otherwise the smartir will throw key was not found error

I repeat them, and re-upload the file, please take a see.

Thanks!


Deprecated config values aren't properly setting their equivalent replacements with a default

Pelican Version:
7.6/7.7
Pelican Service:

[ ] Client
[ ] Plugin
[ ] Registry
[ ] Director
[ ] Origin
[ ] Cache
[ ] Other (please give the detail)

Describe the bug
Various config values that were deprecated in the config overhaul between 7.5 and 7.6 aren't being properly mapped to their new equivalents.
For example, setting Origin.Mode: s3 doesn't correctly update Origin.StorageType: s3.
I suspect this is because we load in defaults from config/resources/defaults.yaml, which are interpreted as user inputs and override the deprecated default mapping that happens here
To Reproduce
In an affected version, set any deprecated config value who's new equivalent is included in config/resources/defaults.yaml. The value associated with the old config will not be mapped to its new config key.

The default being overridden in the issue should now inherit the correct values from default.


KeyError: 'measure_tools'

I have just started using ocp viewer. I managed to install it and execute the viewer.
However starting a minimal example results in KeyError: 'measure_tools'
Minimal example:
import cadquery as cq
from build123d import *
from ocp_vscode import *
b = Box(1, 2, 3)
b = fillet(b.edges(), 0.2)
c = Pos(2, 2, 0) * Cylinder(0.5, 2)
show(b, c)
Error:
Traceback (most recent call last):
File "cadquery_test.py", line 14, in
show(b, c)
File "C:\Users\stefa\AppData\Local\Programs\Python\Python310\lib\site-packages\ocp_vscode\show.py", line 441, in show
measure_tools = conf["measure_tools"]
KeyError: 'measure_tools'

Looks like you are using the extension from the Marketplace (1.x) and the latest version of ocp-vscode (2.x9
We are currently in a beta test phase that requires ocp-vscode 2 to be available.
Either install ocp-vscode from within the extension in vs code (Library manager) that installs ocp-vscode 1.x or load and install the vsix from github.
I will try to finally launch 2.x in the VS Code marketplace this weekend

These are my versions. As you said: Its a little bit strange as ocp_vscode 2.0.6 is installed but ocp_vscode 1.2.2 is running... I am not aware that I installed something by hand. Just clicked on the "install ... buttons"

downgraded ocp_vscode to 1.2.2 and it works. Thanks

I am not aware that I installed something by hand. Just clicked on the "install ... buttons"
hmm, interesting, I then need to check the commands what went wrong.
But glad it is working again

Indeed, for quick starts I did not check the ocp_vscode version.
Fixed it in the release, coming soon

2.0.13 is on VS Code Marketplace. So this shouldn't be a problem any more


Make the "basics" w/o a proxy easier

I feel like I should be able to do something like:
SOMETHING = require("can-ssr/something");

app.use("/", SOMETHING({
  config: 'public/package.json!npm'
});

as a replacement for:
app.use("/", function (req, res) {
		var pathname = url.parse(req.url).pathname;

		render(pathname).then(function(html) {
			var dt = config.doctype || doctype;
			res.send(dt + '\n' + html);
		});
	});

We have express middleware now.


OpenSslX509TrustManagerWrapper in netty-handler is not Java 6 compatible

Netty 4.y should work on Java 6, it's not the case of the OpenSslX509TrustManagerWrapper which fails to load the class:
java.lang.NoClassDefFoundError: Could not initialize class io.netty.handler.ssl.OpenSslX509TrustManagerWrapper
	at io.netty.handler.ssl.ReferenceCountedOpenSslContext.chooseTrustManager(ReferenceCountedOpenSslContext.java:541)
	at io.netty.handler.ssl.ReferenceCountedOpenSslServerContext.newSessionContext(ReferenceCountedOpenSslServerContext.java:142)
	at io.netty.handler.ssl.OpenSslServerContext.<init>(OpenSslServerContext.java:350)
	at io.netty.handler.ssl.OpenSslServerContext.<init>(OpenSslServerContext.java:333)
	at io.netty.handler.ssl.SslContext.newServerContextInternal(SslContext.java:422)
	at io.netty.handler.ssl.SslContextBuilder.build(SslContextBuilder.java:447)
	at ...

The problem is rooted in using javax.net.ssl.X509ExtendedTrustManager which was introduced in Java 7.
The netty-handler versions with the issue: 4.1.31 - 4.1.34 (current)

@kwart hmmm will need to check... you are interested in providing a PR ?

@kwart fixed in https://github.com/netty/netty/pull/9065

You're too fast :+1: :) I'll check the PR in our tests. Thanks.

@kwart let me know once you know the outcome as I plan to release a new version tomorrow ;)

All our TLS tests passed on Java 6 with the fix. Well done!


Add shannonbradshaw to members list.

Hello,
Adding myself, shannonbradshaw, to the kubeflow members list. I've mainly been working on the docs.
My contributions are:
https://github.com/kubeflow/website/pull/2645
https://github.com/kubeflow/website/pull/2655
https://github.com/kubeflow/website/pull/2650
https://github.com/kubeflow/website/pull/2641
https://github.com/kubeflow/website/pull/2638
https://github.com/kubeflow/website/pull/2630
@kimwnasptd and @RFMVasconcelos I believe I need two sponsors for this. Would you be willing to sponsor me?

/lgtm
Good to see you @shannonbradshaw!

/ok-to-test

/lgtm
/approve
Thank you for your contributions!
Welcome to Kubeflow!


[FLINK-23102][runtime] Accessing FlameGraphs while not being enabled …

…returns an exception
What is the purpose of the change
Fixed 'Unable to load requestd file ...' error returned when accessing flame graph while it is not enabled.
Brief change log

flink-runtime/src/main/java/org/apache/flink/runtime/webmonitor/WebMonitorEndpoint.java
flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/job/JobVertexFlameGraphHandler.java

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no

Looks like also need to fix this from the tests:

RestAPIStabilityTest.testDispatcherRestAPIStability:96->assertCompatible:183 The API was modified in a compatible way, but the snapshot was not updated. To update the snapshot, re-run this test with -Dgenerate-rest-snapshot being set.

The API was modified because the flamegraph handler is now always being registered. Please also regenerate the documentation.
So, essentially you need to run locally:
mvn package -Dgenerate-rest-docs -pl flink-docs -am -nsu -DskipTests
mvn verify -Dtest=RestApiStabilityTest -Dgenerate-rest-snapshot -pl flink-runtime

Do we consider FLINK-22527 to provide more friendly hint to tell users how to enable flame graph?

Interesting...that may indeed improve usability a lot. Do you feel it should be included in this PR or rather a separate one? I would be fine with either since it is quite similar to this ticket anyway

@NicoK , I think we could include that hint in this PR. From my point of view, throwing rest exception with messages instead of return null might be enough to tell users.

@zentol I executed the command mvn verify -Dtest=RestApiStabilityTest -Dgenerate-rest-snapshot -pl flink-runtime but got a no tests to run error. Is there anything wrong with this command? The test RestApiStabilityTest actually locates in the module flink-runtime-web.

In the commit fd2bdde, if user accessed the flame graph tab with flame graph setting disabled, it would result in an exception which tells the user that flame graph is disabled and how to enable it. It will not send back an empty flame graph in this case.

@paul8263 Could you share what the UI would be if user clicked but flamegraph not  enabled?

Hi @zentol @NicoK ，I separated the erroneous state(disabled, waiting for the first samples, etc.) of flame graph by assigning different negative value for flameGraph.endTimestamp. The frontend will display a human readable explanation based on this. I also resolved FLINK-23101 with the same commit as I changed both the request handler and the frontend template. It might not be a good practice to split this commit. If there are anything else for me to do, please comment below. Many thanks.

Sorry @paul8263 for letting you wait. I think, the code looks good in general also with respect to improving how the different states are shown, solving FLINK-23101 along the way (and thus making #16419 obsolete).
Before we can merge, can you two things?

Rebase onto latest master - that should help solving the other test failures in the CI build (I'm expecting any but RestAPIStabilityTest to be unrelated, but want to be sure)
RestAPIStabilityTest.testDispatcherRestAPIStability is still failing, please run the following two commands and commit the diff (I'm afraid, there were a few typos in @zentol's commands above)mvn package -Dgenerate-rest-docs -pl flink-docs -am -nsu -DskipTests
mvn verify -Dtest=RestAPIStabilityTest -Dgenerate-rest-snapshot -pl flink-runtime-web

It fails for me but generates the needed changes (and if you run again without -Dgenerate-rest-snapshot it succeeds.

Hi @NicoK , I successfully regenerated rest_v1_dispatcher.html and rest_api_v1.snapshot by running those commands. But I am still confused about the first point "Rebase onto latest master". Could you please tell me what I should do? Thanks.

Hi @zentol , I did the rebase and now it should be OK. Thank you for your advice.


README - Note on Arduino Serial Monitor NL/CR

In Arduino Serial Monitor, both Newline and Carriage return appear to work. In the README, mention that either will work.

Yup... I coded it to accept either :) Will make a note.
On Sat, Jul 23, 2016 at 11:48 AM, Dan<EMAIL_ADDRESS>wrote:

In Arduino Serial Monitor, both Newline and Carriage return appear to
work. In the README, mention that either will work.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/ESPSerialWiFiManager/issues/8, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA6a6oesXYM1ndGSx3UhH-yuZFh2Zmdsks5qYjfGgaJpZM4JTYhI
.

Added note. Still have to do more README details though


[PeptideTermMod:Lysine not cleaved at N-term]

This seems like a trypsin-specific modification. Should we uncheck it when searching other enzymes? It's popping up in nonspecific searches...

Maybe we should have protease-specific mods?

Why is this even a thing? Does anybody know?

GG is  a protease specific mod that occurs upon cleaveage of ubiquitin side chains. There are others, and these tags change with other proteases.

So lysine not being cleaved is indicative of ubiquitin? I am confused. Isn't this just a missed cleavage that happens to be on the N-terminus?

No lysine still gets cleaved but there is an addition two amino acid tag hanging off the side

I'm getting it for nonspecific searches where
"[PeptideTermMod:Lysine not cleaved at N-term]LWGKQKNK"
has the same mass as
"KLWGKQKNK"
so there doesn't appear to be a GG?

I don't know what that is. I was talking about protease specific mods. I don't think there should be any for non-specific.
The notation you write looks like a uniprot mod. check uniprot ptmlist

I removed this mod in #1512 along with a few other ones.
These seem weird to treat as mods for the above reasons (ambiguity, handling it in 2 places [mods and missed cleavages]) but also because we didn't even check to see if the previous amino acid is a K, to my knowledge...
We could write code for interpreting mass-differences as truncations, unsearched missed cleavages, etc. in an open-mass search. not sure how we should approach it in a notch search, if at all

we have protease-specific mods now ("AspN Digested", "Trypsin Digested"). "Trypsin Digested" includes GG on K. I think this issue has already been resolved but maybe I'm misinterpreting...


"I ran into an error while running this command.

Command that caused the error
gif image: [attachment] togif: true"
Image that caused the error
https://cdn.discordapp.com/attachments/1115886477696569434/1125650118318759977/image_20_-removebg-preview.png
Steps to reproduce
Message: DiscordRESTError: Interaction has already been acknowledged.
at RequestHandler.request (/home/esmBot/esmBot/node_modules/.pnpm/oceanic.js@1.7.0/node_modules/oceanic.js/dist/lib/rest/RequestHandler.js:96:15)
at RequestHandler.authRequest (/home/esmBot/esmBot/node_modules/.pnpm/oceanic.js@1.7.0/node_modules/oceanic.js/dist/lib/rest/RequestHandler.js:81:21)
at RESTManager.authRequest (/home/esmBot/esmBot/node_modules/.pnpm/oceanic.js@1.7.0/node_modules/oceanic.js/dist/lib/rest/RESTManager.js:47:29)
at Interactions.createInteractionResponse (/home/esmBot/esmBot/node_modules/.pnpm/oceanic.js@1.7.0/node_modules/oceanic.js/dist/lib/routes/Interactions.js:68:29)
at CommandInteraction.defer (/home/esmBot/esmBot/node_modules/.pnpm/oceanic.js@1.7.0/node_modules/oceanic.js/dist/lib/structures/CommandInteraction.js:176:46)
at GIFCommand.acknowledge (file:///home/esmBot/esmBot/classes/command.js:59:30)
at GIFCommand.run (file:///home/esmBot/esmBot/classes/imageCommand.js:34:49)
at default (file:///home/esmBot/esmBot/events/interactionCreate.js:37:39)
at runMicrotasks ()
at processTicksAndRejections (node:internal/process/task_queues:96:5) {
code: 40060
}
Stack Trace: DiscordRESTError: Interaction has already been acknowledged.
at RequestHandler.request (/home/esmBot/esmBot/node_modules/.pnpm/oceanic.js@1.7.0/node_modules/oceanic.js/dist/lib/rest/RequestHandler.js:96:15)
at RequestHandler.authRequest (/home/esmBot/esmBot/node_modules/.pnpm/oceanic.js@1.7.0/node_modules/oceanic.js/dist/lib/rest/RequestHandler.js:81:21)
at RESTManager.authRequest (/home/esmBot/esmBot/node_modules/.pnpm/oceanic.js@1.7.0/node_modules/oceanic.js/dist/lib/rest/RESTManager.js:47:29)
at Interactions.createInteractionResponse (/home/esmBot/esmBot/node_modules/.pnpm/oceanic.js@1.7.0/node_modules/oceanic.js/dist/lib/routes/Interactions.js:68:29)
at CommandInteraction.defer (/home/esmBot/esmBot/node_modules/.pnpm/oceanic.js@1.7.0/node_modules/oceanic.js/dist/lib/structures/CommandInteraction.js:176:46)
at GIFCommand.acknowledge (file:///home/esmBot/esmBot/classes/command.js:59:30)
at GIFCommand.run (file:///home/esmBot/esmBot/classes/imageCommand.js:34:49)
at default (file:///home/esmBot/esmBot/events/interactionCreate.js:37:39)
at runMicrotasks ()
at processTicksAndRejections (node:internal/process/task_queues:96:5)
Self-hosted instance?
Yes
Error file
https://github.com/esmBot/esmBot/files/11944552/error.txt

Additional context
No response

Duplicate of #336


Add Eth1DepositsCache cache for eth1 data for block production

Adds two new repository to the DB and their wrapper Eth1DepositsCache. Part of #1518. See #1518 for an overview of the target architecture that this component will be part of.
The current repositories eth1Data and depositData will be deprecated. Eth1 data is best fetched and stored separately: (1) blockHash fetched from block headers directly, and (2) depositCount and depositRoot fetched from the deposit logs directly. To do so it's necessary to store each deposit log blockNumber and index, so just keeping depositData is not sufficient.
Most of the logic added in this PR is to convert non-consecutive deposit logs into consecutive consumable eth1Data.

Superseded by https://github.com/ChainSafe/lodestar/pull/1518


React.createClass deprecated

In your code you use React.createClass which is now deprecated. Please rewrite - replace.
When used in React Native on a page, Radio Form onPress={(value) => {this.setState({value:value})}} ...  'this' is undefined so doesn't work/errors out.
Show full example how to implement in a page. It also errors that the var RadioButtonProject is never called.

In case if you are looking for an alternative react-native-radio-buttons-group


Code coloring error

Atom version: 1.27.2 x64

You didn't close the bracket, what happens if you do close it?

This happens:

Thanks for reaching out!
We require the template to be filled out on all new issues and pull requests. We do this so that we can be certain we have all the information we need to address your submission efficiently. This allows the maintainers to spend more time fixing bugs, implementing enhancements, and reviewing and merging pull requests.
Thanks for understanding and meeting us half way :grinning:


Can't able to Play the Video from Server

Streaming Media Cordova Plugin Version 0.1.4
Cordova version 5.3.3
Tested in Samsung Tab 4.4.2 KitKat, Moto E 4.4.4 KitKat, Sony M2 5.5.1 Lolipop
Can't able to play youtube video, and can't able to play video file from server. But it is working fine in local.
The code is below..
function playVideo() {
         var videoUrl="https://www.youtube.com/watch?v=gKBiIWY7FYw";
          console.log('local url      :'+videoUrl);
          var options = {
            successCallback: function() {
              console.log("Video was closed without error.");
            },
            errorCallback: function(errMsg) {
              console.log("Error! " + errMsg);
            }
          };
          try
          {
           window.plugins.streamingMedia.playVideo(videoUrl,options);
         }
         catch(Error)
         {
             console.log("error from server or local"+Error);
         }

      }

What is the error?


Multilevel submenu support

My goal is to have 3 level architecture for my sidebar. Depending on the user role I'm trying to render different sidebar components on the runtime. Everything works perfectly fine except I am not able to add 3rd level menu item.
What I am intending to achieve is having a menu item for each project which contains workbooks as a child menu item. Similarly, each workbook should contain child menu items for its views. In this layered architecture, only views (3rd level items) should contain links to redirect users. The other two, namely projects and workbooks are just expandable menu items.
            foreach ($projs as $proj){
                $event->menu->add([
                    'key' => $proj->name,
                    'text' => strtoupper($proj->name)
                ]);
                foreach ($proj->workbooks as $workbook){
                    $event->menu->addIn($proj->name, [
                        'key' => $workbook->name,
                        'text' => $workbook->name
                    ]);
                    foreach ($workbook->views as $view){
                        $event->menu->addIn($workbook->name, [
                            'text' => $view->name,
                        ]);
                    }
                }
            }

With the provided code I am able to build a menu that has 2 levels => projects and workbooks(child of projects). However, for some reason, it does not display 3rd level. I have tried to look for the official Wiki pages and look through already closed Github issues, but I could not find any relevant case.
I have also tried to use the "submenu" property in order to add items dynamically within the loop, but no success.
I would highly appreciate any input on the issue. Sorry in advance for being a complete newbie

Update!
I managed to solve the problem by the following code.
        foreach ($projs as $proj){
            $event->menu->add([
                'header' => strtoupper($proj->name),
            ]);
            foreach ($proj->workbooks as $workbook){
                $arr = [];
                foreach ($workbook->views as $view){
                    array_push($arr, [
                        'text' => $view->name,
                        'url' => url('/') .
                            '/dashboard/'
                            . $proj->id
                            . '/'. $workbook->id
                            . '/'. $view->id,
                        'shift' => 'ml-3'
                    ]);
                }
                $event->menu->add([
                    'key' => $workbook->name,
                    'text' => $workbook->name,
                    'submenu' => $arr
                ]);
            }
        }

Basically, I first generate the array which contains all 3rd level menu items (views in my case) which I later attach as a submenu property to the 2nd level menu item (workbooks in my case)
Although it did the trick, I would still be glad to hear if you have better solutions to offer.
Thanks!

Hi @nihadatakishiyev , I believe your first problem can be solved if you define an url or route attribute for the most inner menu items (the views). Give a read to the first section of Menu Configuration, there says:

You can specify the set of menu items to display in the left sidebar and/or the top navbar. A menu item representing a link should have a text attribute and an url (or route) attribute.

Even more, I have created the next test code for you particular case in the AppServiceProvider.php file, and all worked fine:
<?php

namespace App\Providers;

use Illuminate\Support\ServiceProvider;
use Illuminate\Contracts\Events\Dispatcher;
use JeroenNoten\LaravelAdminLte\Events\BuildingMenu;
use \stdClass;

class AppServiceProvider extends ServiceProvider
{
    /**
     * Register any application services.
     *
     * @return void
     */
    public function register()
    {
        //
    }

    /**
     * Bootstrap any application services.
     *
     * @return void
     */
    public function boot(Dispatcher $events)
    {
        // Create views.

        $view1 = new \stdClass();
        $view1->name = 'View1';
        $view1->url = 'view/1';

        $view2 = new \stdClass();
        $view2->name = 'View2';
        $view2->url = 'view/2';

        $view3 = new \stdClass();
        $view3->name = 'View3';
        $view3->url = 'view/3';

        $view4 = new \stdClass();
        $view4->name = 'View4';
        $view4->url = 'view/4';

        // Create workbooks.

        $wb1 = new \stdClass();
        $wb1->name = 'Workbook1';
        $wb1->views = [$view1, $view2];

        $wb2 = new \stdClass();
        $wb2->name = 'Workbook2';
        $wb2->views = [$view3, $view4];

        // Create projects.

        $proj1 = new \stdClass();
        $proj1->name = 'Project1';
        $proj1->workbooks = [$wb1];

        $proj2 = new \stdClass();
        $proj2->name = 'Project2';
        $proj2->workbooks = [$wb2];

        $projs = [$proj1, $proj2];

        // TEST

        $events->listen(BuildingMenu::class, function (BuildingMenu $event) use ($projs) {

            foreach ($projs as $proj) {

                $event->menu->add([
                    'key' => $proj->name,
                    'text' => strtoupper($proj->name)
                ]);

                foreach ($proj->workbooks as $workbook) {

                    $event->menu->addIn($proj->name, [
                        'key' => $workbook->name,
                        'text' => $workbook->name
                    ]);

                    foreach ($workbook->views as $view) {
                        $event->menu->addIn($workbook->name, [
                            'text' => $view->name,
                            'url' => $view->url,
                        ]);
                    }
                }
            }
        });
    }
}

Thanks a lot, @Shidersz. It worked like a charm.


Order Sections within a Monograph

STORY
As a press user, I would like to be able to order sections within a monograph, so that I can reflect the ordering and structure of the primary narrative in the UI for accessing associated assets.
NOTES
For the first pass, do the simplest possible thing.  This will facilitate having a more detailed discussion about refinements like drag-and-drop vs. assigning order numbers, etc.
ACCEPTANCE

[x] I can re-arrange the order of sections within a Web UI

If you click the "File Manager" button, you can reorder all the sections and assets that are attached to the monograph by using the drag and drop functionality.

Now that the user roles are in, you can't just create monographs anymore.  Here's what you'll need to do to test this story:
Run this rake task to create the super admin user:
rake admin
And make sure redis is running.
Then you can log into the UI with that user and create a monograph with sections and files attached.  At the bottom of the monograph show page, click "File Manager" (we should change the name of that button), and you should be able to reorder the files and sections.
The re-ordering won't take effect unless you click the 'Save' button.
Then when you go back to the monograph show page, the files and sections are each listed separately, but each list should be in the correct order.
We'll probably change how these lists are displayed at some point (like maybe they won't be in separate lists), but this was just a first pass.


Usage of this plugin for Private Azure Cloud

Hi can this plugin be used for private azure cloud ? What Authentication is required and where can we provide in in app-config file?
An example would be good to have here.
Thanks

Hi Mehul.
What do you mean by a private Azure Cloud, Azure stack HCI?
For authentication, we use managed identity in our setup. However, the backend plugin uses the azure identity plugin, which works with all the methods for authenticating against Azure.
I did not think of having a specific config block for this in app-config, but thats probably a good idea. For now, you'll need to use environment variables if authenticating with an app registration.


Sigil not working with assert macro.

The sigil seems to not be working with ExUnit's assert macro.
To reproduce:
test "sigil test" do
  assert {:a, ~M{a?: 1, b}} = {:a, %{a?: 1, b: 2}}
  assert b
end

It gives the following error:
** (CompileError) test/lib/testapp/sigil_test.exs:10: undefined function b/0
    (stdlib) lists.erl:1338: :lists.foreach/2
    (stdlib) erl_eval.erl:670: :erl_eval.do_apply/6
    (elixir) lib/code.ex:376: Code.require_file/2
    (elixir) lib/kernel/parallel_require.ex:59: anonymous fn/2 in Kernel.ParallelRequire.spawn_requires/5

Since assert is a macro as well, maybe there's some weird behavior in the macro's compile order.

I've just filed elixir-lang/elixir#6925, as my testing seems to show the issue is actually assert not preserving the new variables in the corner case you found.  I'll stay on top of that, and update this issue accordingly.

Awesome, thanks for the quick response and analysis.

With the release of elixir 1.6, this is now fixed. Let me know if you run into any other problems.  Thanks for the bug report!


accel & gyro calibration helpers

This is a small (hopefully more digestible) piece of incremental work broken out of https://github.com/PX4/Firmware/pull/15235 that takes the existing calibration helper class used in sensors moved to lib/ and expands them to handle accel/gyro calibration parameter interactions.

Tested

gyro calibration
accel & gyro enable parameter

CAL_ACCx_EN/CAL_GYROx_EN

I've included a simple quick accel calibration helper for bench testing, the test rack, etc. It quickly sets the accel calibration offsets using vehicle_attitude (if available) or simply normalizing the vector to 1G. I definitely wouldn't encourage it for normal usage, but it's useful for testing to quickly populate the calibration parameters.
nsh> commander calibrate accel quick
nsh> INFO  [commander] Accelerometer quick calibration
INFO  [sensor_calibration] ACC 3670026 EN: 1, offset: [-0.0044 0.0009 -0.0910] scale: [1.0000 1.0000 1.0000]
INFO  [sensor_calibration] ACC 3932170 EN: 1, offset: [0.1982 0.0412 -0.2945] scale: [1.0000 1.0000 1.0000]
INFO  [sensor_calibration] ACC 4259850 EN: 1, offset: [0.5936 0.1140 0.0580] scale: [1.0000 1.0000 1.0000]

nsh> param show CAL_ACC*
Symbols: x = used, + = saved, * = unsaved
x + CAL_ACC0_EN [36,62] : 1
x + CAL_ACC0_ID [37,63] : 3670026
x + CAL_ACC0_XOFF [38,64] : -0.0044
x + CAL_ACC0_XSCALE [39,65] : 1.0000
x + CAL_ACC0_YOFF [40,66] : 0.0009
x + CAL_ACC0_YSCALE [41,67] : 1.0000
x + CAL_ACC0_ZOFF [42,68] : -0.0910
x + CAL_ACC0_ZSCALE [43,69] : 1.0000
x + CAL_ACC1_EN [44,70] : 1
x + CAL_ACC1_ID [45,71] : 3932170
x + CAL_ACC1_XOFF [46,72] : 0.1982
x + CAL_ACC1_XSCALE [47,73] : 1.0000
x + CAL_ACC1_YOFF [48,74] : 0.0412
x + CAL_ACC1_YSCALE [49,75] : 1.0000
x + CAL_ACC1_ZOFF [50,76] : -0.2945
x + CAL_ACC1_ZSCALE [51,77] : 1.0000
x + CAL_ACC2_EN [52,78] : 1
x + CAL_ACC2_ID [53,79] : 4259850
x + CAL_ACC2_XOFF [54,80] : 0.5936
x + CAL_ACC2_XSCALE [55,81] : 1.0000
x + CAL_ACC2_YOFF [56,82] : 0.1140
x + CAL_ACC2_YSCALE [57,83] : 1.0000
x + CAL_ACC2_ZOFF [58,84] : 0.0580
x + CAL_ACC2_ZSCALE [59,85] : 1.0000
x + CAL_ACC_PRIME [60,86] : 3670026

 573/1324 parameters used.

Broken out of https://github.com/PX4/Firmware/pull/15235 for incremental review, but might as well bring it in with https://github.com/PX4/Firmware/pull/15235 where's it's all been tested together.


Sample improvements

The following improvements in separate commits:

Use @AutoConfigureHttpGraphQlTester in tests to have the tester initialized by Boot.
Remove attributes from @SchemaMapping in ReviewsController since default conventions match in this case.
Use ClassNameTypeResolver in the config for reviews. It handles straight forward type mappings and is customizable.

Awesome! Thanks for the fixes!


geerlingguy.drush : Do not run Composer as root/super user!

Issue Type

Bug Report / Support Request

Your Environment
Vagrant 1.8.7
VirtualBox 5.1.4r110228
ansible <IP_ADDRESS>

Your OS

macOS (Sierra)

Full console output
https://gist.github.com/vyasamit2007/4c8e497a61c73aecb878b60fb24b161e
Summary
geerlingguy.drush role is not downloading correctly. I've tried it with become: yes & become: no with no luck. If I remove the line, it works correctly but my drush doesn't work. This is the exact error:
TASK [geerlingguy.drush : Install Drush dependencies with Composer.] ***********
fatal: [npro]: FAILED! => {"changed": true, "cmd": "/usr/bin/composer install --prefer-dist --no-interaction", "delta": "0:00:00.116993", "end": "2017-01-05 15:05:26.455792", "failed": true, "rc": 2, "start": "2017-01-05 15:05:26.338799", "stderr": "Do not run Composer as root/super user! See https://getcomposer.org/root for details\nLoading composer repositories with package information\nInstalling dependencies (including require-dev) from lock file\nYour requirements could not be resolved to an installable set of packages.\n\n  Problem 1\n    - Installation request for phpunit/phpunit 4.8.31 -> satisfiable by phpunit/phpunit[4.8.31].\n    - phpunit/phpunit 4.8.31 requires ext-dom * -> the requested PHP extension dom is missing from your system.\n\n  To enable extensions, verify that they are enabled in your .ini files:\n    - /etc/php/7.1/cli/php.ini\n    - /etc/php/7.1/cli/conf.d/10-opcache.ini\n    - /etc/php/7.1/cli/conf.d/10-pdo.ini\n    - /etc/php/7.1/cli/conf.d/20-apcu.ini\n    - /etc/php/7.1/cli/conf.d/20-apcu_bc.ini\n    - /etc/php/7.1/cli/conf.d/20-calendar.ini\n    - /etc/php/7.1/cli/conf.d/20-ctype.ini\n    - /etc/php/7.1/cli/conf.d/20-exif.ini\n    - /etc/php/7.1/cli/conf.d/20-fileinfo.ini\n    - /etc/php/7.1/cli/conf.d/20-ftp.ini\n    - /etc/php/7.1/cli/conf.d/20-gettext.ini\n    - /etc/php/7.1/cli/conf.d/20-iconv.ini\n    - /etc/php/7.1/cli/conf.d/20-json.ini\n    - /etc/php/7.1/cli/conf.d/20-phar.ini\n    - /etc/php/7.1/cli/conf.d/20-posix.ini\n    - /etc/php/7.1/cli/conf.d/20-readline.ini\n    - /etc/php/7.1/cli/conf.d/20-shmop.ini\n    - /etc/php/7.1/cli/conf.d/20-sockets.ini\n    - /etc/php/7.1/cli/conf.d/20-sysvmsg.ini\n    - /etc/php/7.1/cli/conf.d/20-sysvsem.ini\n    - /etc/php/7.1/cli/conf.d/20-sysvshm.ini\n    - /etc/php/7.1/cli/conf.d/20-tokenizer.ini\n    - /etc/php/7.1/cli/conf.d/20-yaml.ini\n  You can also run `php --ini` inside terminal to see which files are used by PHP in CLI mode.", "stdout": "", "stdout_lines": [], "warnings": []}

Also, I've selected php version as 5.6, though its going in 7.1 configs, not sure why.
Thanks!

This should work out of the box. What changes have you made? If you've added become that means you're editing the playbook? This is not supported.

The project was created long before and have loads of project specific changes, now php repos have been changed and it got changed how it works so we've updated just the sections which needs to be updated.

Fair enough. In this case the error is:

phpunit/phpunit 4.8.31 requires ext-dom * -> the requested PHP extension dom is missing from your system.
To enable extensions, verify that they are enabled in your .ini files:
/etc/php/7.1/cli/php.ini
...

ext-dom should be included with php-xml. Note that composer believes the config files should reside in /etc/php/7.1 while it seems you have configured the VM to install php 5.6.
You probably missed something essential when you cherry picked updates.

Also, Drupal VM should nowadays be configurable enough that you dont need to "hack" internal tasks. If you contain all your custom changes to config.yml and post-provision-tasks, upgrades will be a lot easier.
If there's anything you find that cant be overridden, feel free to open up an issue and we can see if there's something Drupal VM can do to make it easier.

Could you post your config.yml and any other relevant changes you've made to Drupal VM. Maybe we can spot something.

@vyasamit2007 - Yeah, is it also possible for you to identify what version of Drupal VM you're running? Is there a CHANGELOG included?

No CHANGELOG file and also I don't think there is any file mentioning the Drupal VM version.
Here is my config.yml file : https://gist.github.com/vyasamit2007/1e8b4fb6d6a245a207e2309d3e3a7f3b

Any chance you could upload a zip of the drupal vm files you have?
A couple of other things:

Are you attempting to re-provision an existing box? Does this happen if you build a new one with vagrant destroy; vagrant up?
Does the installed PHP version report 5.6 or 7.1 (vagrant ssh -c 'php -v')

Also, here's a config.yml with only your custom overrides https://gist.github.com/oxyc/ced11adb7b853336fdfd49f105965cdb that should work for the latest version of Drupal VM.
From the looks of it you're using a local codebase and not a composer project which is the default? If you use a local codebase you should also set build_composer_project: false

Yes, I guess. I've destroyed multiple times after changing something but that happened, each time.
It's installing PHP 7.1.
I'll try to create a box using build_composer_project: false option and with the config file you've provided. BUT I'm not sure if it's related to this particular issue.

I'll try to create a box using build_composer_project: false option and with the config file you've provided. BUT I'm not sure if it's related to this particular issue.

Could you send a zip of all the drupal-vm files? The issue is most likely that you havent made all the necessary updates to the drupal vm files. Maybe we can figure out what's missing. Eg. if you hack drupal core files your chance for support on DO drops drastically :)

@vyasamit2007 any updates on this? Did you manage to get it working?

I'll close this as there's been no updates for some time. I hope you got it working and if not, please re-open and send the drupal-vm files and I'll have a look.
I'd still suggest upgrading to a later version of Drupal VM though, that's the most stable solution.


Complex Queries

Add a way to get views like:

[ ] Members of a given subteam who do/don't have a given badge
[ ] People with a given role who have more/less than N hours in a category
[x] Drop all expired active entries
[x] Drop all active entries

These exist on the search branch but need some prettification


No audio when screen sharing

using the latest plugin and flutter/dart version.
testing on android sdk 30
i have implemented foreground service for the screen share, but there is still no audio. do i need to implement audio in the foreground service? if so, how?

You can create another MediaStream by call getUserMedia({'audio': true, 'video': false}) and get AudioTrack from it. After that, add the track into main stream before.
But, on iPad you will facing an issue, some time you will receive logs about can not start record screen from RelayKit after at least one time successfully. That so weird!.


Bind to localhost only in order to avoid firewall prompts

This little change makes the annoying firewall prompts on Mac OS (probably Windows as well) disappear. It's most likely not upstreamable in this from as this behavior would probably have to be made configurable, so this PR can serve as a place to discuss whether and how to include it.

hmmm, I'm not sure that I like this change for the reasons that you detailed already, and I agree that it would be difficult to upstream in its current state. I have a simple suggestion though.
In WireServer.hpp add an override to TCPSocketServer::listen which takes a tcp::endpoint, and then refactor WireServer.cpp like so:
void TCPSocketServer::listen(const port_type port) {
    listen(tcp::endpoint(tcp::v4(), port));
}

void TCPSocketServer::listen(const tcp::endpoint endpoint) {
    doListen(acceptor, endpoint);
    acceptor.set_option(tcp::no_delay(true));
}

That way we can just restrict the bind to localhost in our call to TCPSocketServer::listen (which fwiw is in AAcceptanceTestRunner::SThreadFunc). What do you think about this?

That sounds like a great idea! How should we go about moving this forward? (i.e. who should do the work?)

@mak-ableton I just pushed a fixup with the aforementioned suggested changes. If you are also ok with this, then let's squash the branch and fix the commit message. I'll take care of upstreaming the resulting patch to cucumber-cpp.

@nre-ableton: Perfect, thanks!

@mak-ableton Sorry for the delay on this PR, I've been busy with some other stuff recently. Anyways, I've rebased/squashed and think this one is ready to merge. After we merge it to master (which at some point we will move our project references to that branch instead of the old octo one), we'll upstream it. @ala-ableton, are you also ok merging this patch?

@nre-ableton: shouldn't we merge the commits from ableton-initial-migration first?

@ala-ableton I don't think it's necessary to wait for that, since we will want this patch in our master at any rate, and it doesn't depend on any of the patches there.

Then LGTM :+1:

Sorry for the delay on this PR, I've been busy with some other stuff recently.

@nre-ableton no worries! On the contrary, I find it awesome that I can just draft a patch and you take care of getting it in shape and upstreaming it. Thank you! 😄

@mak-ableton BTW I've been doing a lot of debugging on macOS recently, and I completely appreciate how necessary this patch is. That dialog is super annoying!

That dialog is super annoying!

Especially since the VPN client requires that the firewall is enabled. Disabling the firewall was my old workaround for this dialog :)

@mak-ableton after talking it over with @ala-ableton, we've decided to try to upstream the patch first rather than merging it to our fork's master right away. Upstream PR is here in case you want to follow it: https://github.com/cucumber/cucumber-cpp/pull/158

This commit (or rather, one that looks a bit like it) has been merged to the upstream master.


Added destroy method

This just destroys the archive's metadata and content feed. Will look at adding a flag for destroying mounts in the near future.

@mafintosh @andrewosh is there anything else to change here?

Anything else I should do to get this in?

Hey @RangerMauve I'm really sorry this has taken an eternity, but let's get this in today. I'm gonna fix the Travis issue to turn this green, then rebase + publish.
Think it's totally fine to have this be a shallow deletion for now. There are a few other methods (like createDiffStream) that are shallow too, so it's not too confusing.

Published 10.11.0

Awesome! Thank you. 💜


[URGENT] Possible website hijack?

I'm getting abusive, sexual pop-up ads from the site accessing from a mobile device.

note Id?

URGENT!!!! PLEASE
Porn in notes!!!
I use notehub at school!!

@jesuslalvaro so please share the note id you visit at school? :)


in persian formatter digits do not convert to persian digits

when build formatter by language-strings/fa the text convert to persian language but the digits don't convert to persian digits.

I rely on pull request for translations. The language strings can take function that can convert digits to the right format. You're welcome to make a pull request. I will be happy to help. We basically just need a function to convert to Persian digits.

While this was not possible in the past, it might be possible to use the Intl module to convert the numbers to various locales now.


Confusing blade/message after selecting 'Swap'

The blade / message seems to serve no purpose.

This is tracked by
RDBug 9224675:Update the text on the Ibiza Slots blade to be more descriptive to click on slots command above - this will go live with our next Ibiza deployment

Closing issue as it's a duplicate


When attempting to connect, if the user is coming through mobile web, deeplink to Phantom app's browser with the webpage URL

Is your feature request related to a problem? Please describe.
Since mobile web users will not be able to sign (no way for mobile Chrome or mobile Safari to have wallets), if the wallet they are trying to connect to has a mobile app, they should be deeplinked to that mobile wallet app's browser for the webpage to be properly interacted with / wallet connected.
This is the same as how coinbase-wallet-sdk does.
Describe the solution you'd like
^
Describe alternatives you've considered
Copy & paste URL into mobile wallet app's browser.

Duplicate of #312, #311, #309. This is up to mobile wallets to implement first and cannot be added by wallet-adapter until the wallets support it.

Phantom Wallet added deep links. Do you plan to integrate it? I tried it - almost got through, but have issues with base58 encryption. I can share my code if you want.

Can you please share your code?

Can you please share your code?

Sure - hope you can help... The native app opens correctly. When returning to the website it shows an decryption error.
https://codesandbox.io/s/phantom-wallet-provider-rd8l0j?file=/src/PhantomProvider.tsx


Fix displayName broken by nested assignment

Hi. This PR resolves an issue I ran into while using this plugin. Namely, when the plugin attempts to pull the identifier out of an AssignmentExpression which is nested (eg A = B = 'foobar') it will choose the innermost identifier (B) while the others are ignored. This (as I explain below) may not be desirable.
This PR adds a preferOuterAssignment option for your babel config which enables this alternate behavior.
Quick example:
For the following snippet:
export const DoubleAssignment = Discarded = styled.div`
  width: 50%;
`;

if your babel config has:
  "plugins": [
    ["babel-plugin-styled-components", {
		"ssr": false,
		"fileName": false,
		"displayName": true,
		"transpileTemplateLiterals": false
	}]
  ]

your emit will look like:
export const DoubleAssignment = Discarded = styled.div.withConfig({
  displayName: "Discarded"
})`width:50%;`;

by enabling the preferOuterAssignment you'll get an emit which looks like:
export const DoubleAssignment = Discarded = styled.div.withConfig({
  displayName: "DoubleAssignment"
})`width:50%;`;

Real-world example where this be necessary
I first noticed this when some of my components were not getting their displayName set correctly while others where.

relevant snippet:
import React from 'react';
import styled from 'styled-components';
import ContentContainer from './ContentContainer';

export const ScaffoldingContainer = styled.div`
	background: red;
`;

export const ScaffoldingWrapper = styled.div`
    margin: 4px;
`;

export const Scaffolding = ({ children }) => (
    <ScaffoldingContainer>
        <ScaffoldingWrapper>
            <ContentContainer>
                { children }
            </ContentContainer>
        </ScaffoldingWrapper>
    </ScaffoldingContainer>
);

export default Scaffolding;

After some experimentation, I found that by moving ScaffoldingContainer and ScaffoldingWrapper into a seperate file, things got a bit better:

fileName is false. Therefor where the components are located should not determine their displayName
I dug deeper and discovered the issue when looking at the emitted code.
var ScaffoldingContainer = _c = styled_components__WEBPACK_IMPORTED_MODULE_2__["default"].div.withConfig({
  displayName: "_c",
  componentId: "sc-19b0fwj-0"
})(_templateObject());
var ScaffoldingWrapper = _c2 = styled_components__WEBPACK_IMPORTED_MODULE_2__["default"].div.withConfig({
  displayName: "_c2",
  componentId: "sc-19b0fwj-1"
})(_templateObject2());
var Scaffolding = function Scaffolding(_ref) {
  var children = _ref.children;
  return /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_1___default.a.createElement(ScaffoldingContainer, null, /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_1___default.a.createElement(ScaffoldingWrapper, null, /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_1___default.a.createElement(_ContentContainer__WEBPACK_IMPORTED_MODULE_3__["default"], null, children)));
};
_c3 = Scaffolding;
/* harmony default export */ __webpack_exports__["default"] = (Scaffolding);

var _c, _c2, _c3;

$RefreshReg$(_c, "ScaffoldingContainer");
$RefreshReg$(_c2, "ScaffoldingWrapper");
$RefreshReg$(_c3, "Scaffolding");

It's actually react-refresh/babel (the successor to react-hot-loader) that is causing the issue. It looks like when components in the same file reference each other it injects some helper code and creates this issue with nested assignment (ie: var ScaffoldingContainer = _c = ...). When ScaffoldingContainer and ScaffoldingWrapper were moved to a seperate file and they did reference other components (unlike Scaffolding) that code never got injected.
So why am I PR'ing this repo and not facebook/react?
It's quite simple. The likelihood that if I reported this in facebook/react (and possibly pmmmwh/react-refresh-webpack-plugin) have the react team decide it's a large enough issue that they rework their experimental mechanism and so anytime in the near future is a lot less likely than a simple PR here which is opt-in (via preferOuterAssignment) despite being non-breaking (at least in this repo's test suite). This has the added benefit of (possibly) resolving other related issues.

This needs a rebase

@probablyup I'll rebase off master tonight

Done.

@rozzzly what if we enabled this by default for NODE_ENV='development' since the refresh plugin isn't used in production or testing?

@probablyup I added this as an opt-in flag because while it didn't break any of the tests and I can't think of any situations where this would break the displayName for anyone, didn't want to make that assumption.
If you're comfortable with that, you'll not hear any objections from me. It would probably simplify things as far as documentation goes: A few moments ago I realized, my previous commits never updated the README.md with the name of the option/explanation. When I checked the README.md, I found that the options are not documented there but on the styled-components website. The site has multiple sections on tooling all concatenated into one page, and I feel adding this long technical explanation multi-paragraph explanation about what preferOuterAssigment is and when/why it should be enabled would just be more clutter. I think it would be preferable to just have it be the default and briefer explanation without comparing emit code, etc.

preferOuterAssigment
Enabled by default since v1.11.2, this option ensures that when there is nested assignment such as:
export const DoubleAssignment = Discarded = styled.div`
  width: 50%;
`;

that the component shows up as DoubleAssignment instead of Discarded. Even when your code does not contain nested assignment, this issue can be caused by other babel plugins changing the emitted code before it reaches babel-plugin-styled-components. For more details, see PR#279
You can disable this feature by setting the preferOuterAssignment option to false:
{
  "plugins": [
    [
      "babel-plugin-styled-components",
      {
        "preferOuterAssigment": false
      }
    ]
  ]
}

Ha I ran into this myself and opened #355. Is there anyway I can help push this along to get merged and released?

Also in terms of option or not. It seems that react-refresh always prefixes their names with an underscore so we could also only skip the inner assignment if it's an underscore name.


OTEL Collector Upgrade to 0.33.1 on all patterns

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
OTEL Collector Upgrade to 0.33.1 on all patterns
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Completed. Closing this issue.


feat: Update OKD/OCP Readme for 4.15/4.16

Update Tested Versions for OKD/OCP Installation

Here is more info about the issue:
I deployed latest OKD version with longhorn 1.6.2 : 4.15.0-0.okd-2024-03-10-010116
Tried to access the UI using route "Log in with OpenShift" shows the following error:

Checking on the openshift-authentication pod logs

Eventually found out service account exists but no secret token attached to it.

As in Kubernetes v1.24
The LegacyServiceAccountTokenNoAutoGeneration feature gate is beta, and enabled by default. When enabled, Secret API objects containing service account tokens are no longer auto-generated for every ServiceAccount. Use the [TokenRequest](https://kubernetes.io/docs/reference/kubernetes-api/authentication-resources/token-request-v1/) API to acquire service account tokens, or if a non-expiring token is required, create a Secret API object for the token controller to populate with a service account token by following this [guide](https://kubernetes.io/docs/concepts/configuration/secret/#service-account-token-secrets). (https://github.com/kubernetes/kubernetes/pull/108309, [@zshihang](https://github.com/zshihang))

In order to fix this either create a tokenRequest or a long-lived secret as mentioned in the PR
Refer:
https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG/CHANGELOG-1.24.md#urgent-upgrade-notes
https://kubernetes.io/docs/reference/access-authn-authz/service-accounts-admin/#auto-generated-legacy-serviceaccount-token-clean-up

@mantissahz please help review this.

Updated to include the OKD 4.16 Builds, while not full stable channel, they are releasing "next" builds.
While not recommended it is possible to upgrade from 4.15 -> 4.16.
New Builds on 4.16 seem to be fine.

Hi @ArthurVardevanyan,
Could you help test the Longhorn 1.7.0 as well?

Hi @ArthurVardevanyan,
Could you help test the Longhorn 1.7.0 as well?

Part 1
Started Clean on
OKD 4.15.0-0.okd-2024-03-10-010116
Longhorn 1.6.2

Part 2
Upgraded to 1.7.0, deleted the two below service objects manually, since they don't appear to be in 1.7.0
longhorn-engine-manager and  longhorn-replica-manager
Git Delta on my Repo: https://github.com/ArthurVardevanyan/HomeLab/commit/e3efac38c3bf0854e955f5d3737cfe33d764db5f

Part 3
Then I did the manual steps to force a 4.15->4.16 Upgrade, (again, not generally recommended yet)
Final
OKD 4.16.0-0.okd-scos-2024-08-21-155613
Longhorn 1.7.0


Is there any benchmark result for NGT QG/QBG?

Hi Team,
Thanks for the great work on quantinized graph.
I wonder is there benchmark result for quantized graph  build? Since build speed is also very important on production use cases

For QB build times, ANN Benchmarks is a useful reference. Each dataset page contains various evaluation results like this.

Thanks for the reply.
NGT seems not listed in GIST and some other dataset. Is there any advice on improving NGT index build speed? This is also very important for real world production

In general, there is a trade-off between build time and performance. For NGT, its build time can be reduced by changing parameters, but performance is also reduced. On the other hand, since NGT supports multi-threading for building the index, multi-core can significantly reduce the build time.

Thanks for the clarifiction. Great work, will do more benchmark on it


avoid DeprecationWarning from SciPy 1.8 (cupyx.scipy.sparse)

This small change is to avoid a new DeprecationWarning I noticed when using SciPy 1.8.0rc2
DeprecationWarning: Please use `spmatrix` from the `scipy.sparse` namespace, the `scipy.sparse.base` namespace is deprecated.
  from scipy.sparse.base import spmatrix

/test mini,head

/test mini,head

Would you merge this PR as the CI failure is not related to this change, @kmaehashi? (I'll look into TestLOBPCG's failure aside)

Thanks! @grlee77 @kmaehashi


flutter run failed with macOS Ventura, iOS 16.

Steps to Reproduce

flutter run # on macOS Ventura, iOS 16.

Expected results:
run flutter app with iOS 16.
Actual results:
it failed. but iOS 15.5 simulator runs well

Logs

[+3151 ms] 2022-06-08 23:42:37.046 ios-deploy[37293:911262] [ !! ] Unable to locate DeviceSupport directory with suffix
'DeveloperDiskImage.dmg'. This probably means you don't have Xcode installed, you will need to launch the app manually and
logging output will not be shown!
[   +1 ms] Script started, output file is /dev/null
[        ] Script done, output file is /dev/null
[        ] ios-deploy exited with code 0
[   +1 ms] Could not run build/ios/iphoneos/Runner.app on 00008101-00095C883801401E.
[        ] Try launching Xcode and selecting "Product > Run" to fix the problem:
[        ]   open ios/Runner.xcworkspace
[        ] Installing and launching... (completed in 6.0s)
[        ] Error launching application on iPhone 12.
[   +1 ms] "flutter run" took 20,005ms.
[   +1 ms]
           #0      throwToolExit (package:flutter_tools/src/base/common.dart:10:3)
           #1      RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:712:9)
           <asynchronous suspension>
           #2      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1195:27)
           <asynchronous suspension>
           #3      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #4      CommandRunner.runCommand (package:args/command_runner.dart:209:13)
           <asynchronous suspension>
           #5      FlutterCommandRunner.runCommand.<anonymous closure>
(package:flutter_tools/src/runner/flutter_command_runner.dart:281:9)
           <asynchronous suspension>
           #6      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #7      FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:229:5)
           <asynchronous suspension>
           #8      run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:9)
           <asynchronous suspension>
           #9      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #10     main (package:flutter_tools/executable.dart:94:3)
           <asynchronous suspension>

➜ flutter doctor -v
[✓] Flutter (Channel master, 3.1.0-0.0.pre.1166, on macOS 13.0 22A5266r darwin-arm, locale ko-KR)
    • Flutter version 3.1.0-0.0.pre.1166 at /Users/changjoopark/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4ec296513c (2 hours ago), 2022-06-08 08:48:07 -0400
    • Engine revision faa2ee8055
    • Dart version 2.18.0 (build 2.18.0-170.0.dev)
    • DevTools version 2.14.0

[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at /Users/changjoopark/Library/Android/sdk
    • Platform android-32, build-tools 32.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] VS Code (version 1.67.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.42.0

[✓] Connected device (3 available)
    • iPhone 12 (mobile) • 00008101-00095C883801401E • ios            • iOS 16.0 20A5283p
    • macOS (desktop)    • macos                     • darwin-arm64   • macOS 13.0 22A5266r darwin-arm
    • Chrome (web)       • chrome                    • web-javascript • Google Chrome 102.0.5005.61

[✓] HTTP Host Availability
    • All required HTTP hosts are available

Hello @ChangJoo-Park. Looks like you need to install Xcode 14 beta to add support for iOS 16 beta. Since this is not an issue with Flutter, I am closing this issue. Feel free to comment if you disagree

Hello @ChangJoo-Park. Looks like you need to install Xcode 14 beta to add support for iOS 16 beta. Since this is not an issue with Flutter, I am closing this issue. Feel free to comment if you disagree

Hello @exaby73, flutter builds well. but launch with debug mode not working. When I launch my app with xcode, it working well. How do I run flutter with flutter_tools

Unable to locate DeviceSupport directory

From the logs, Flutter is trying to look for devices support files which are included with Xcode. I'm not sure how Xcode works but Flutter needs these files in order to run your app. These would be installed for iOS 16 with Xcode 14. Please upgrade Xcode to v14 beta to most probably fix this issue.
You can alternatively try the steps in this StackOverflow thread. This is for iOS 14 but it's exactly what you are experiencing.


Fields for chef servers and orgs should disallow whitespace-only values

Describe the bug
A clear description of what the bug is.
To Reproduce
Create a server:
$ curl -sSkH "api-token: $TOK" -X POST \
> -d '{"name": "SERVER_NAME2", "description": "SERVER_DESCRIPTION2", "fqdn": "example.com", "ip_address": "<IP_ADDRESS>"}' \
> https://a2-dev.test/api/v0/infra/servers?pretty
{
  "server": {
    "id": "b900bfa6-c699-4bb3-bb4e-146901de1162",
    "name": "SERVER_NAME2",
    "description": "SERVER_DESCRIPTION2",
    "fqdn": "example.com",
    "ip_address": "<IP_ADDRESS>",
    "orgs_count": 0
  }
}

Grab the id value from above and plug in here to add an organization (notice the name field is all blanks):
$ curl -sSkH "api-token: $TOK" -X POST \
-d '{"name": "      ", "admin_user": "ADMIN_USER", "admin_key": "KEY"}' \
https://a2-dev.test/api/v0/infra/servers/b900bfa6-c699-4bb3-bb4e-146901de1162/orgs?pretty
{
  "org": {
    "id": "fdb262cc-a60a-44f6-bc8f-bef4a9b040b8",
    "name": "      ",
    "admin_user": "ADMIN_USER",
    "admin_key": "8a367010-1f5b-4cf1-8d8d-24a51ce39565",
    "server_id": "b900bfa6-c699-4bb3-bb4e-146901de1162",
    "projects": [
    ]
  }
}

Note that an empty field correctly triggers an error; it is only a problem for a non-empty value consisting of just whitespace.
Expected behavior
The operation should fail, and report preferably this text
(it is being introduced in PR #3197):
"an org name is required and must contain at least one non-whitespace character"
Note that you can see how such checks are being introduced across many resources in PR #3197.
Note the same fix is needed for every field the user may provide.

Yes, As of now this is pending to add the proto3 buff side validation. I have the plan to tackle in https://github.com/chef/automate/issues/3240

@vsingh-msys Please note that I am actually not suggesting protobuf validation. We had been using that for all auth services but have started to move away from it with the PR I mentioned above, #3197.
The reason is that the error messages from protobuf validation were NOT user friendly. With protobuf validation we had either:
(A) The rather esoteric "runes" term used here (when using min_len):
value length must be at least 1 runes
(B) The strange-looking regex for "not whitespace" (when using regex \\S+):
value does not match regex pattern \"\\\\S\""
So we instead let the service handle it with the message I indicated: an org name is required and must contain at least one non-whitespace character

Ohh, I was misunderstood it, but adding validation at service level looks interesting will look and revert you accordingly thanks


add new command and pictures from bot upon receipt

.shun - useful for theme days : http://cf067b.medialib.glogster.com/media/17/17f24350f6b62a842781e184ada36673ea04677cbcf0ae6a0ab1eacdd2fb3d86/shun-the-non-believer-jpg.jpg

Added!

Awesome!  <3


DNM: empty commit to test libdispatch change

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

Please test with following PR:
apple/swift-corelibs-libdispatch#519
@swift-ci please build toolchain

Linux Toolchain (Ubuntu 16.04)
Download Toolchain
Git Sha - a3c0e27c09d6f5e341398de9291ffd05d53d86ba
Install command
tar zxf  swift-PR-27372-280-ubuntu16.04.tar.gz
More info

macOS Toolchain
Download Toolchain
Git Sha - a3c0e27c09d6f5e341398de9291ffd05d53d86ba
Install command
tar -zxf  swift-PR-27372-382-osx.tar.gz --directory ~/

@swift-ci please smoke test