Cluster gets (re)created when running amp cluster rm

Description

When running amp -s localhost cluster rm after creating a cluster, the cluster does not get removed. The logs that are output are the same as when running amp -s localhost cluster create
Windows 10 Anniversary x64, docker 17.03.1, amp v0.12.0-dev
Steps to reproduce the issue:

create a cluster  amp -s localhost cluster create
remove the cluster  amp -s localhost cluster rm

Describe the results you received:
$ ./amp -s localhost cluster rm
[localhost:50101]
bootstrapping cluster
wait for cluster
ok
deploy amp monitoring stack to cluster - stage 1
deploy ampmon.1.stack.yml => amp
Updating service amp_elasticsearch (id: nvxesm4ys8fcs2c3qargenb4s)
Updating service amp_nats (id: d8e0do0gh68abao68xjd371ak)
Updating service amp_etcd (id: q1sl2vgxnqg50melmxt83il20)
ok
wait for all amp monitoring stage 1 stack service replicas to be running
ok
deploy amp monitoring stack to cluster - stage 2
deploy ampmon.2.stack.yml => amp
Updating service amp_ampbeat (id: w4c0mibn1qwnf22nack8lx7j5)
Updating service amp_kibana (id: 5as277v0swy49y3cjrdkilyjo)
ok
wait for all amp monitoring stage 2 service replicas to be running (540 sec)
ok
deploy amp stack to cluster
deploy amp.stack.yml => amp
Updating service amp_amplifier-gateway (id: qiibd3h9l0tlbapxfz4vhtm6y)
Updating service amp_agent (id: mwk1145u0vzravak6gidx1uwi)
Updating service amp_amplifier (id: z9upbi71y4myq1t2sl1i3i9jv)
ok
wait for all amp service replicas to be running (540 sec)
ok
Generating RSA private key, 2048 bit long modulus
............................+++
................................................+++
e is 65537 (0x10001)
Generating RSA private key, 2048 bit long modulus
............................+++
..+++
e is 65537 (0x10001)
Signature ok
subject=/C=US/ST=California/L=Santa Clara/O=Axway/OU=Atomiq/CN=dashboard.local.atomiq.io
Getting CA Private Key
588b1e61b81b8f4cc004a3ef212ae810b60303a5b2fa184e1cfd2730d29db673
checking if the certificate secret is already defined
deploy amp proxy stack to cluster
deploy ampproxy.stack.yml => amp
Updating service amp_proxy (id: yb9q9t0inga4rmfbvem1tv603)
ok
wait for all amp proxy service replicas to be running (540 sec)
ok

Cluster status: healthy

Describe the results you expected:
cluster was deleted
Additional information you deem important (e.g. issue happens only occasionally):
Output of docker version:
$ docker version
Client:
 Version:      17.03.1-ce
 API version:  1.27
 Go version:   go1.7.5
 Git commit:   c6d412e
 Built:        Tue Mar 28 00:40:02 2017
 OS/Arch:      windows/amd64

Server:
 Version:      17.03.1-ce
 API version:  1.27 (minimum version 1.12)
 Go version:   go1.7.5
 Git commit:   c6d412e
 Built:        Fri Mar 24 00:00:50 2017
 OS/Arch:      linux/amd64
 Experimental: true

Docker for Mac/Windows version, if applicable:
Version 17.03.1-ce-win12 (12058)

Output of amp version:
$ ./amp version
[cloud.appcelerator.io:50101]

Client:
 Version:       v0.12.0-dev
 Build:         056e5e9f
 Server:        cloud.appcelerator.io:50101
 Go version:    go1.8.1
 OS/Arch:       windows/amd64

Server:         not connected

Additional environment details (AWS, etc., if applicable):

It appears the proper way to delete (for now):
docker stack rm pinger
docker stack rm amp
cc @bladedancer

related to #1212
this will be fully fixed in next release, and right now if fixed in 0.12.0-dev when using the option -t latest (this uses the latest pushed version of the images, in particular the amp-bootstrap image related to this issue).

fixed with 0.12.0 release


demo site down

Will the demo site reopen?
For a quick demo of the plugin?

Hey, that was actually due to github maintenance. If you do a couple hard refreshes it should work fine.


Pin Hupper version to 1.7

Fixes #6578
Hupper introduced reloading pressing the Enter key on version 1.8 which does not work well with our Docker Compose dev environment.
This PR pins the version to 1.7 just before this change.

I released 1.9. Let me know if there’s still an issue after upgrading please!


add automated testing

Creates a bootstrap script to grab build & test dependencies, and a test
script to validate style and run tests. Finally, adds a minimal travis
config for running the above.

LGTM 👍 +1 . I'll merge this and #5 #6 #7 #8  together after testing it together @josephholsten

Thanks for the pull request! I've merged this.
golint is currently commented out in the tests, as I haven't yet merged the API incompatible changes (I don't really want to do this until we have the matching changes for terraform-provider-nsone ready to go).
I don't have any more time to look at this today, but I think between myself and @sarguru we should be able to get this done and the final set of changes merged some time in the next week.


Error on Startup after Migration to Nuxt Bridge on Fresh Project

Environment

Operating System: Darwin
Node Version:     v16.11.0
Nuxt Version:     2.16.0-27358576.777a4b7f
Package Manager<EMAIL_ADDRESS>Bundler:          Webpack
User Config:      target, head, css, plugins, components, buildModules, modules, vuetify, build
Runtime Modules:  -
Build Modules<EMAIL_ADDRESS>

Reproduction
https://github.com/timgoeller/nuxt-bug
Describe the bug
When generating a new empty Nuxt 2 project and applying the steps provided in the Nuxt Bridge getting started guide (https://v3.nuxtjs.org/getting-started/bridge/), calling yarn dev throws an error when using nuxi dev (using nuxt still works).
The error message is as follows:
 ERROR  Cannot read properties of undefined (reading 'callHook')                                                                           16:23:36

  at Object.invoke (node_modules/nuxi/dist/chunks/dev.mjs:6773:31)
  at runMicrotasks (<anonymous>)
  at processTicksAndRejections (node:internal/process/task_queues:96:5)
  at async _main (node_modules/nuxi/dist/chunks/index.mjs:384:7)

I think it might have to do with this recent change, even though it seems like no one else encountered this issue yet: https://github.com/nuxt/framework/commit/4bd7adae4a7d43f7b906abec1149688d307954a3#diff-86f78f1673e500e055e8a1f266252abe62af18f0b52424f09275d7f2b1e96d3f
Additional context
No response
Logs
yarn run v1.22.17
$ nuxi dev
Nuxt CLI v3.0.0-27386759.b449d0b                                                                                16:23:25
                                                                                16:23:25
  > Local:    http://localhost:3000/
  > Network:  http://<IP_ADDRESS>:3000/

ℹ Preparing project for development                                                                                16:23:28
ℹ Initial build may take a while                                                                                16:23:28
ℹ Discovered Components: .nuxt/components/readme.md                                                                                16:23:28
✔ Builder initialized                                                                                16:23:28
✔ Nuxt files generated                                                                                16:23:28

✔ Client
  Compiled successfully in 7.54s

✔ Server
  Compiled successfully in 6.49s

ℹ Waiting for file changes                                                                                16:23:36

 ERROR  Cannot read properties of undefined (reading 'callHook')                                                                           16:23:36

  at Object.invoke (node_modules/nuxi/dist/chunks/dev.mjs:6773:31)
  at runMicrotasks (<anonymous>)
  at processTicksAndRejections (node:internal/process/task_queues:96:5)
  at async _main (node_modules/nuxi/dist/chunks/index.mjs:384:7)

Anyone fix this issue? Followed the documentation on the nuxt bridge page and came across this error. The core team seriously roll out buggy features.

@Cally99 If you are encountering an issue, please create a new issue with a reproduction and we'll look into it. 😉

@danielroe there is another issue with the same problem and a repo where you can reproduce the problem: https://github.com/nuxt/bridge/issues/314

Any movement on this?

If you are still getting this error while debugging your migration process, I encountered severe issues in development and according to the migration guide that says that you can disable Bridge if you set bridge: false in your nuxt.config.js file.
If you do this, you have to start nuxt with the legacy script, because nuxi dev doesn't work.
Use nuxt dev

Any updates on this?
` ERROR  Cannot start nuxt:  Cannot read properties of undefined (reading 'callHook')                                                                                05:45:23
at load (node_modules/nuxi/dist/chunks/dev.mjs:6780:33)
at async Object.invoke (node_modules/nuxi/dist/chunks/dev.mjs:6828:5)
at async _main (node_modules/nuxi/dist/cli.mjs:50:20)`
I really have no idea of even where to start looking.

Facing similar issue when starting dev server. nuxi build is working though.
Terminal Output:

Browser:

Facing similar issue when starting dev server. nuxi build is working though.
Terminal Output:
Browser:

I have the exact same issue:


Templateutils: Separate template utility code into a separate package

This PR moves the template code in ciao-cli into a separate package called templateutils.  The old template code has been enhanced by:

Some documentation
More unit tests.
Some new functions, e.g., table, sort, cols, head, rows, tail.
The ability to select which functions to make available to script writers.
The ability to add your own functions.

Here's an example of some of the new functions in action
ciao-cli workload list -f '{{table (cols . "ID" "Name" "RAM" "Vcpus")}}'
ID                                   Name                         RAM     Vcpus
b1650b0a-2813-4382-b08f-f9b45713ae13 Fedora test VM               128     2
528c7f8d-c87e-4b55-8aae-b292c0213cfa Clear Linux test VM          128     2
e56801a4-a6e5-4d9b-bdf7-4d0ebb389067 Ubuntu latest test container 128     2
28cc2f7d-7371-4c81-912b-245877b186f2 Debian latest test container 128     2

Coverage increased (+0.2%) to 65.943% when pulling 82af2b3056c59ed6b7f16dfcaf4bb7d86d952984 on markdryan:templateutils into 15cef664197b1e82c2dcf6bc22f0ab9bc174edb7 on 01org:master.

Coverage increased (+0.2%) to 65.943% when pulling 82af2b3056c59ed6b7f16dfcaf4bb7d86d952984 on markdryan:templateutils into 15cef664197b1e82c2dcf6bc22f0ab9bc174edb7 on 01org:master.

Coverage increased (+0.2%) to 65.959% when pulling 89074a60a4b9dc2c1d40f9b865a599c6ea594c74 on markdryan:templateutils into 15cef664197b1e82c2dcf6bc22f0ab9bc174edb7 on 01org:master.

Coverage increased (+0.2%) to 65.955% when pulling a1e525668fcc3d8a43e97e3539417a02babdc23f on markdryan:templateutils into 15cef664197b1e82c2dcf6bc22f0ab9bc174edb7 on 01org:master.

lgt

lgtm. only nit is that OptAllFNs -> OptAllFns naming.

Coverage increased (+0.2%) to 65.946% when pulling a1e525668fcc3d8a43e97e3539417a02babdc23f on markdryan:templateutils into 15cef664197b1e82c2dcf6bc22f0ab9bc174edb7 on 01org:master.

Coverage increased (+0.2%) to 65.951% when pulling 6782174fcf5663dc075ef7eb952d81207111381a on markdryan:templateutils into 15cef664197b1e82c2dcf6bc22f0ab9bc174edb7 on 01org:master.

Coverage increased (+0.3%) to 66.021% when pulling 6782174fcf5663dc075ef7eb952d81207111381a on markdryan:templateutils into 15cef664197b1e82c2dcf6bc22f0ab9bc174edb7 on 01org:master.

Coverage increased (+0.3%) to 66.021% when pulling 6782174fcf5663dc075ef7eb952d81207111381a on markdryan:templateutils into 15cef664197b1e82c2dcf6bc22f0ab9bc174edb7 on 01org:master.

My only real problem with this PR is the risk Google's golang team sees the useful types of things Mark is doing and tries to hire him away from us ;)

Coverage increased (+0.2%) to 65.953% when pulling b80a98b332f825052835f04a8bb5d5432cd0367a on markdryan:templateutils into 15cef664197b1e82c2dcf6bc22f0ab9bc174edb7 on 01org:master.


Update cuongval.toml

Resubmit without editing signed toml file as instruct by Bengt

Heya! Unfortunately you will have to resubmit the entire toml and resign it. The signature is made over your net address.
So go through the entire process again and sign it again (just don't change the metadata fields this time! and also use the IP:PORT form instead of a dns like you did in this pr). Thank you!

@trandinncuong please confirm receipt

Hello Bengt Lofgren,
Thank you for you informations, actually, I did follow your instruction in my last PR. However, I closed that PR and created new toml file then resubmit PR again with newly signed toml file and did not edit metadata field, please check Github
Regards
Tran Dinh Cuong
On Friday, December 15, 2023 at 04:20:38 PM GMT+7, Bengt Lofgren @.***> wrote:
Heya! Unfortunately you will have to resubmit the entire toml and resign it. The signature is made over your net address.
So go through the entire process again and sign it again (just don't change the metadata fields this time! and also use the IP:PORT form instead of a dns like you did in this pr). Thank you!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Yes, have I did it, please check github PR
On Friday, December 15, 2023 at 04:20:38 PM GMT+7, Bengt Lofgren @.***> wrote:
Heya! Unfortunately you will have to resubmit the entire toml and resign it. The signature is made over your net address.
So go through the entire process again and sign it again (just don't change the metadata fields this time! and also use the IP:PORT form instead of a dns like you did in this pr). Thank you!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Set the appropriate parameters when calling the Logger method

Not enough arguments for Logger's method.
Set the appropriate parameters when calling the Logger method.

Let me verify it a little bit more;
Before I do that, let me close this pull request.


Document redirects.txt

Follow up to https://github.com/cilium/cilium/pull/34233
Add some documentation describing redirects.txt and also update the check target to check redirects. Actually building the docs will also check this, but make check will do it without generating them.

/test


consider supporting conditional/lazy caching of some assets

Sometimes images are conditionally downloaded, particularly responsive images that use <img srcset> to make the browser determine which one of a set to load, as described in Responsive Images: If you’re just changing resolutions, use srcset..
I'm not sure how often they're used in apps generally, much less offline-able apps hosted on GitHub Pages, but they're mentioned in Introduction to Service Worker at least, so at least some folks have spent some time thinking about how to handle them when using Service Worker to cache app assets.
Since the promise of Oghliner is to simply cache your whole app offline, I'm loathe to complicate the interface with conditional/lazy caching of assets. But perhaps it would make sense to include some support for caching one of several variants of an image.
For example, we might let users specify assets that shouldn't be cached immediately but should be cached once requested, something like:
{
    "filesToCacheLazily": [
        "images/logo-lo-res.png",
        "images/logo-hi-res.png",
    ],
}

Then the worker can cache whichever variant the browser ends up requesting.
Before we do something like this, however, we should have better use cases for it, ideally in the field.

IIRC sw-precache allows you to define a set of dynamic URLs (that I guess are cached on request), or maybe it was broccoli-serviceworker.

https://github.com/jkleinsc/broccoli-serviceworker/blob/345faa10a9a1b148171a5095eb725fe8ea10c4ff/lib/service-worker.js#L170

This could be useful for localization files as well.


Update to Xcode 14.2

Update to cipd package https://chrome-infra-packages.appspot.com/p/flutter_internal/ios/xcode/ios/+/build_version:14C18 to support using iOS 16.2 in devicelab.
Fixes https://github.com/flutter/flutter/issues/117505
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

iOS 16.2 not downloading:
== Runtimes ==
iOS 16.0 (16.0 - 20A5339d) - com.apple.CoreSimulator.SimRuntime.iOS-16-0

https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8794118111164297857/+/u/list_simulators/stdout
It is installing the new 14c18 version of Xcode though
https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8794118111164297857/+/u/install_xcode/execution_details
But not the runtime
https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8794118111164297857/+/u/install_xcode/stdout
Should be installing
https://chrome-infra-packages.appspot.com/p/flutter_internal/ios/xcode/ios_runtime/+/hBrR2grfHQr_HT7umFHekdhf6cn9WBrwIutdgJBxMtcC


PSD-1145: Product search

Re-adding product search
Description

Re-adding product search and removing case count on business and product lists.
Checklist:

[x] Automated checks are passing locally.
[x] CHANGELOG updated if change is worth telling users about.

General testing

[x] Test without javascript
[x] Test on small screen

Accessibility testing

[x] Works keyboard only
[x] Tested with one screen reader
[x] Zoom page to 400% - content still visible
[x] Disable css - does content make sense and appear in a logical order?

Thanks!


Utility classes should not have public constructors

This pull request is focused on resolving occurrences of Sonar rule squid:S1118 - Utility classes should not have public constructors
You can find more information about the issue here:
https://dev.eclipse.org/sonar/coding_rules#q=squid:S1118
Please let me know if you have any questions.
Kirill Vlasov

See my comment here: https://github.com/graphhopper/graphhopper/pull/629#issuecomment-204449727


corrected "less" to "fewer"

Coverage remained the same at 100.0% when pulling f578e1a55e7ba811a8060caa54e4ed0dbe37578e on Thebigbignooby:patch-2 into 700d1fb2d3543bbf5cccb21489ae882a33cb213e on request:master.

Awesome, thank you!


Bump to latest kairos-agent version for the collector

This may fix the issue with building the collector for go 1.19 which is needed for fips enabled builds

nah this doesnt fix it :/


Getting error while adding a card

Describe the bug
When trying to add a card using eventBus getting following error
LaneHelper.js?bc88:57 Uncaught TypeError: Cannot read property 'find' of undefined
at eval (LaneHelper.js?bc88:57)
at Array.filter ()
at Object.appendCardsToLane (LaneHelper.js?bc88:57)
at Object.appendCardToLane (LaneHelper.js?bc88:84)
at boardReducer (BoardReducer.js?f5a0:23)
at dispatch (redux.js?00d8:205)
at Object.eval [as addCard] (redux.js?00d8:462)
at Object.publish (BoardContainer.js?3573:93)
at Tasks.eval [as addCard] (index.js?3067:21)
at onClick (index.js?3067:54)
To Reproduce
Steps to reproduce the behaviour:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Additional context
Using mongo DB to store data.

Could you show the data you publish to eventBus?

@dapi, I have tried using the data which was available on the storybook which is following
this.state.eventBus.publish({type: 'ADD_CARD', laneId: `${this.props.boards.lanes[0].id}`, card: {id: "M1", title: "Buy Milk", label: "15 mins", description: "Also set reminder"}})

You don’t need to use ``` symbols in such cases.
You component props, which you access with this does not containt boards

Sorry. This is not react-trello issue.

@akkias Sorry. This is not react-trello issue. Close it


Is there any way that we can set automatic refresh on access_token exp time instead of id_token exp time

Describe the bug
I have implemented oAuth with automatic refresh in my angular 10 project as described in Readme file and it is working fine but the  problem here is, automatic refresh is working on id_token exp time and the exp time of id_token is very short (5 min) so after every 0.75 % time of exp it is getting refreshed.
I want to setup automatic refresh on access_token exp time is there any way to do that.
Your any help will be apricated.
Stackblitz example
NA
To Reproduce
Expected behavior
Is there any way that we can set automatic refresh on access_token exp time instead of id_token exp time

I think it should happen based on the access token expiry time?
It would help if you could add a Stackblitz example or another way to reproduce it? Or perhaps you could check if my sample repo when reconfigured against your IDS displays the same behavior, and provide us with a redacted set of responses?
Alternatively you could dig a bit into the source code of the lib and point us to the lines that are causing this issue? If it's immediately clear from the code that this is a bug a repro would be less needed.

Then again I also see #633 which seems to be a super similar report? 🤔

@jeroenheijmans  no it is not same my question is can we have something to check expiry time of access_token instead of id_token

Ah yes I see how it would be different!
Digging a little in the code I think this is the cause:
https://github.com/manfredsteyer/angular-oauth2-oidc/blob/d95d7da788e2c1390346c66de62dc31f10d2b852/projects/lib/src/oauth-service.ts#L420-L428
combined with the listener to that event:
https://github.com/manfredsteyer/angular-oauth2-oidc/blob/d95d7da788e2c1390346c66de62dc31f10d2b852/projects/lib/src/oauth-service.ts#L256-L270
which does not filter on the type of event.
Basically, whenever either of the tokens you've asked for expires, both are requested anew.
What strikes me as odd though is that when you ask for the id_token, then why would you want to ignore the expiration of said token? If it expired you need a renewed one, non?
Still, I can imagine it being a feature to filter the expirations that should trigger asking for a new token. 👍

In my condition id_token time out is very less 5 mins and we can not change it and because of which when I use setupAutomaticSilentRefresh it is reauthenticating in every 5 min that overhead I dont want for every 5 min of interval where as in access_token we can able to set time out about 2-3 hrs so that refresh token request will execute once in 2 hrs

But if the token expires, you need to get a fresh one, right? Or you'd be sitting on an expired token?
Put differently, if you had a feature to 'ignore' the id_token expiration time, the library would be holding on to an expired token. That seems to open to weird bugs at best, a security issue at worst.
I've tagged it "feature-request" either way. I'm just not sure if it's a good feature to have for all users of the library. Instead, a workaround or useful interception point for special cases like yours might be the better way forward? Speaking of which, even though I personally dislike inheritance for customization, that is the choice made for this library, and it gives you a workaround already: create a sublcass of the OAuthService and override the setupExpirationTimers() method to ignore id_token timers. That should fix your immediate issue with relative ease?

@jeroenheijmans Your argument seems reasonable:

But if the token expires, you need to get a fresh one, right? Or you'd be sitting on an expired token?

However, the possibility to refresh the id_token is not always present in each identity provider implementation since the RFC does not require it. (See #1157)
So applications have to deal with an expired id_token which is imho not a real problem since the application may extract the user info from the id_token when it is received and remember it as long as it is able to refresh the access_token.


FEATURE: Allow Categories to follow remote Actors

Allows a Discourse category to follow any ActivityPub actor, including another Discourse category.

is it possible to configure the category to only follow remote actors, i.e. without enabling the publishing features? This probably has some repercussions for the "Federation" tab and the settings in the category.

Yes, this would essentially be a category-specific version of the site-wide publishing restriction added in https://github.com/discourse/discourse-activity-pub/pull/45. I think the way to implement this is to have a multi-select category setting with a list of published Activities (remove all of them to disable publishing in that category). This would be only slightly more work than a straight boolean setting and more powerful.

I wonder as well a bit if enabling both outbound publishing and inbound receiving of content (follows) can quickly become confusing. I don't know if this makes a lot of sense, but maybe for a v1 we can separate the two functionalities. I.e. have a way to allow category A to either publish content on Mastodon or receive content from follows.

I think separating them would be more confusing as a v1 tbh as the norm in federation is bi-directional. Accepting incoming and disabling outgoing (or selectively disabling outgoing as suggested above) would be a good feature, but I think it's a more advanced configuration. As you mentioned in our PM, disabling publishing raises some UX questions as you'll have topics with both published and un-published posts. Not an insurmountable UX challenge, but still slightly more advanced than just a fully federated category.


Documentation - No Mention Of Latest Firmware Or Where To Get It

I can see on the web configuration page that the unit provides means to flash a firmware update, but as far as I know none were provided.  I'm not sure if one is available either because there is no documentation sharing this information.
This is what the units we purchased shipped with:
Name: DMC60C
FW Version: 0x117
Please add somewhere in the repo the latest available firmware for the unit and where it can be acquired.  Thanks!

Whoops!  Meant to post this at https://github.com/Digilent/dmc60c-frc-api


Add e2e support for specifying XNNPACK thread count

This PR adds the iree-compile options xnnpack-threads that specifies the number of threads in the XNNPACK threadpool. The option is currently plumbed through to each xnnpack dialect op after legalization. In order to achieve the use of flags, this PR also makes some minor changes to the pass autogeneration of legalize-xnnpack.
Current limitation:

The threadpool is created every time an op is called. This can be fixed by

storing the threadpool after it gets created for the first time in the void* context variable that gets passed to every executable plugin function
creating xnnpack dialect ops for creating the threadpool, which can then be inserted as part of the MLIR

Addressing changes in follow up PR.


AWS region ap-south-1(Mumbai) not available

While creating the cluster using AWS option the region ap-south-1(Mumbai) is not listed in available options

The region needs to be added to this list.

Added via this pull request.


fix uehaj2022 / latteart#21

respected sir,
This PR contains commits for issue 21 alone. kindly review it for any changes or mistakes. Otherwise this is ready to merge.
regards


Update scalafmt-core to 3.0.4

Updates org.scalameta:scalafmt-core from 3.0.0 to 3.0.4.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (3.0.0).
You might want to review and update them manually.
project/plugins.sbt

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

labels: library-update, semver-patch, old-version-remains

Superseded by #113.


AssertionError: Explainer lime -- training_data should be an instance of Tabular.

I'm trying to run OmniXAI in ML Workflow tutorial, in the cell:
explainers = TabularExplainer(     explainers=["lime", "shap", "mace", "pdp", "ale"],     mode="classification",     data=train_data,     model=gbtree,     preprocess=lambda z: transformer.transform(z),     params={         "lime": {"kernel_width": 3},         "shap": {"nsamples": 100},         "mace": {"ignored_features": ["Marital Status"]}     } )
I got this error:

AssertionError: Explainer lime -- training_data should be an instance of Tabular.

How can I solve it?

Please make sure that train data is an instance of the Tabular class.

I solved it. Thank you.


livenessProbe and readinessProbe for Console

Hi, I see this block code where I can input some probe for liveness and readiness, but I can't find any default value that I can use to determine if the console is healty or ready.
Is possible to know if the console implements any of those? I tried the knowledge base but couldn't find anything useful.
https://github.com/aquasecurity/aqua-helm/blob/07321c6ef6934a9a4a8dc97c096f10e837269037/server/templates/web-deployment.yaml#L175-L182

Yea, I would think that all liveness and readiness probes should be configured out-of-the-box, as they are responsible for determining application health/readiness. That's not quite something an end user should have to know about as the app needs to expose those itself.
The block of code referenced is actually visible in all deployments, not just Console.

@amalucelli thanks for reporting. Fixed see above PR.

The block of code referenced is actually visible in all deployments, not just Console.

So the above PR only added probes to one deployment, the Console. Other charts here, such as the Enforcers, still don't seem to have any probes... Should a separate issue be opened to address those...?
The PR also implemented identical liveness and readiness probes against the / endpoint -- do the Console and other Aqua tools not have their own dedicated healthcheck endpoints that perform appropriate status checks? (e.g. /healthz that checks if DB is responsive etc)


Update Composer dependencies (2023-02-06-00-09)

Visual regression test passed!
View the visual regression test report


How to document spread operator

How do we document object spread operators in functions
/**
 * @param {object} cat - cute
 * @param {function} blender - pokey
 * @param {array} cubes - things to pass to blender
 * @return {string}
 *   result of blender
 */
const doThing = ( {cat, blender}, ...cubes ) => blender(cat, ...cubes)

+1

+1

+1

+1

Also please see #1163 , seems like code with spread operator can't be even parsed by jsdoc currently.

2

How is this solved nowadays?

@param {...object} myParam
works for me

See #987 and #1234 for examples of how to document this. I filed jsdoc3/jsdoc3.github.com#159 to add something about this to the docs.

There is still no mention of the spread operator though, so I filed jsdoc3/jsdoc3.github.com#183 to add to the docs.

I had a different need. I was doing my React component and I needed to document spread operator inside object destructuring. Seems that VSCode or JSDocs can not handle this properly.
I tried this way: (I've simplified)
`
/**

@typedef {{ value: number }} MyObj
@param {{ label?: string, rest: ...MyObj }} props
*/
const MyTextInput = ({ label, ...rest }) => {
But "rest" gets the weird type{ rest: any, MyObj: any }`. Yes, rest is considered as an object that has a property "rest" and another property "MyObj".

I also tried this way:
`
/**

@typedef {{ value: number }} MyObj
@param {{ label?: string, ...rest: MyObj }} props
*/
const MyTextInput = ({ label, ...rest }) => {
But "rest" gets the weird type{ rest: MyObj }`

So I found a solution that works for now:
`
/**

@typedef {{ value: number }} MyObj
@param {{ label?: string } & MyObj} props
*/
const MyTextInput = ({ label, ...rest }) => {
`

This time, "rest" type is correctly assigned to an object with the properties of MyObj.

if i have something like this:
/**
 * @param {function} successCallback
 * @param {function} errorCallback
 * @param {SessionRequest=} sessionRequest
 */
function requestSession (successCallback, errorCallback, sessionRequest) {
  api.requestSession(successCallback, errorCallback, sessionRequest)
}

and i would want to just write
function request (...args) {
  api.requestSession(...args)
}

What would be the best way to write my jsdoc?
I guess i could just write const request = api.requestSession.bind(api) but rather not

if i have something like this:
/**
 * @param {function} successCallback
 * @param {function} errorCallback
 * @param {SessionRequest=} sessionRequest
 */
function requestSession (successCallback, errorCallback, sessionRequest) {
  api.requestSession(successCallback, errorCallback, sessionRequest)
}

and i would want to just write
function request (...args) {
  api.requestSession(...args)
}

What would be the best way to write my jsdoc?
I guess i could just write const request = api.requestSession.bind(api) but rather not

if i have something like this:
/**
 * @param {function} successCallback
 * @param {function} errorCallback
 * @param {SessionRequest=} sessionRequest
 */
function requestSession (successCallback, errorCallback, sessionRequest) {
  api.requestSession(successCallback, errorCallback, sessionRequest)
}

and i would want to just write
function request (...args) {
  api.requestSession(...args)
}

What would be the best way to write my jsdoc?

Do correct me if I'm wrong but I think you could do:
/**
 * @param {function} successCallback
 * @param {function} errorCallback
 * @param {SessionRequest} sessionRequest
 */
function requestSession (...args) {
  api.requestSession(...args)
}

/**
 * @param {...number} args - array of number arguments
 */
export const a = (...args: number[]) => {
	return args
}

this will works, but the parameter name is fixed to args
indirect source: https://jsdoc.app/tags-type.html


Changed type of last4 to string

Description
Changed type of last4 in the users table from integer to string. This fixes the issue of the last4 digits of a credit card not showing all 4 digits of the credit card if the credit card's last 4 digits are something like 0005. Before this fix those 4 digits were stored as 5 in our database instead of 0005.
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[x] This change requires a documentation update

Change status

[x] Complete, tested, ready to review and merge

How Has This Been Tested?
Tested through front end app by using a stripe test card to confirm that the last 4 digits of the credit card show up as a string ('0005') instead of an integer (5).
Checklist:

[x] My code follows the style guidelines of this project

[x] I have performed a self-review of my own code

[ ] My code has been reviewed by at least one peer

[ ] I have made corresponding changes to the documentation

[x] My changes generate no new warnings

[x] There are no merge conflicts

 Fix following bug: When credit card updates and last four digits are something like 0005, the last 4 digits posted to the user object shows up as •••• •••• •••• 5 instead of •••• •••• •••• 0005.


Remove ChainExpression and ChainElement?

foo?.() is represented in our AST as (slightly abbreviated):
Expression::ChainExpression(
    ChainExpression {
        expression: ChainElement::CallExpression(
            CallExpression {
                callee: IdentifierReference { name: "foo" },
                arguments: [],
                optional: true,
            }
        )
    }
)

What do ChainExpression and ChainElement add here? They appear to just duplicate the same information that optional field holds.
Couldn't it just be:
Expression::CallExpression(
    CallExpression {
        callee: IdentifierReference { name: "foo" },
        arguments: [],
        optional: true,
    }
)

Or, if we want to follow Babel and shave 8 bytes off CallExpression, remove the optional field and use a separate enum variant:
Expression::OptionalCallExpression(
    CallExpression {
        callee: IdentifierReference { name: "foo" },
        arguments: [],
        // No `optional` field - `Expression::OptionalCallExpression` performs same function
    }
)

Same goes for the MemberExpression variants of ChainElement - they all also have an optional field.
Note: ESTree has ChainExpression and ChainElement. But I don't see the advantage of that design.

No, AST nodes are defined by the spec. In this case https://tc39.es/ecma262/#sec-optional-chains

ChainExpression is also really useful when we need to transform the syntax.

It's in the spec, but what does it actually do? What syntax cannot be expressed in AST without it?

Yes you can bend the AST to make it terser or more verbose ... but we don't design the AST, the spec does.
For runtimes, it needs to strictly follow the runtime semantics defined in https://tc39.es/ecma262/#sec-optional-chaining-chain-evaluation


[minor] spelling mistake in Links section

Stanford Question Asnwering Dataset and Lederboard should be Stanford Question Answering Dataset and Leaderboard

Haha thanks, fixed!


YAML File Weights Documentation

Hi,
This isn't really a bug for you but the guide you link to for random weights in YAML files is missing some key syntax and probably causing a lot of headaches, the listed syntax for random weights is
color:
- {2::red
  | 3::blue
  | 1::green
  }

Which does not work and actually breaks the wildcard key. To properly use weights it needs to be wrapped in quotes like
color:
    - "{8::red
        | 3:blue
        | 1::green
       }"

For anyone trying to do this note you don't use the normal list format because then you'd have to add a list entry for everytime you want an item weighted.

Thanks for that. We have just added explicit support for weights in yaml and json. The official documentation will be updated soon, but you can see examples here: https://github.com/adieyal/dynamicprompts/pull/97

This is awesome, thanks so much for adding that! Feel free to close this out I will once the docs are updated.


[BUG] SAS token SAS_EXPIRY_TIME encoding results in an invalid signature

Describe the bug
After introducing https://github.com/Azure/azure-sdk-for-java/pull/20520 SAS query params are sanitized but it seems like the SAS_EXPIRY_TIME gets modified and it invalidates the provided signature.
Exception or Stack Trace
<?xml version="1.0" encoding="utf-8"?><Error>AuthenticationFailed<Message>Server failed to authenticate the request. Make sure the value of Authorization header is formed correctly including the signature.
RequestId:76972c6c-201e-0090-1453-582f4d000000
Time:2021-06-03T08:38:25.4063780Z</Message><AuthenticationErrorDetail>Signature did not match. String to sign used was rwdl

2021-07-20T13:21:00Z
/blob/<account>/<container>

2018-11-09
c

To Reproduce
Provide a SAS token with SAS_EXPIRY_TIME without seconds, i.e. se=2021-07-20T13%3A21Z&sp=rwdl&sv=2018-11-09&sr=c&sig=<redacted> that converts the http request query params into se=2021-07-20T13%3A21%3A00Z&sp=rwdl&sv=2018-11-09&sr=c&sig=<redacted> making the signature invalid. This can be reproduced with curl easily.
Expected behavior
Generate a valid http request
Setup (please complete the following information):

Version of the Library used: 12.11.1

Additional context
Add any other context about the problem here.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[X] Bug Description Added
[X] Repro Steps Added
[X] Setup information Added

@gapra-msft could you take a look into this?

Hi @fcofdez,
Thank  you for reporting this issue. We will take a look at what could be causing the problem and get back to you if we have any questions.

@fcofdez, can I get some better information about how exactly you're encountering this? You're specifically pointing to a PR that just aligned connection string SAS ingestion with what every other SAS ingestion mechanism already did.

Are you encountering this new issue when providing a SAS through connection strings?
If you try to ingest a SAS via a different mechanism in the library, do you hit this same error?

Before the mentioned PR?
After the mentioned PR?

Where are you getting this SAS?

@jaschrep-msft we provide the SAS token using a connection string, but as far as I can see in the code that shouldn't make any difference.
new BlobServiceClientBuilder()
            .connectionString(connectionString)

Where are you getting this SAS?
It comes from our CI infrastructure as we caught this running integration tests.

I think the problem comes when the SAS query parameters are manipulated, as it seems like the signature is computed using the SAS_EXPIRY_TIME literal value. To test this theory I run a HTTP request using curl with the SAS query parameters as they are provided in our tests and the request worked correctly. When I modified the SAS_EXPIRY_TIME query parameter and included seconds to it (as https://github.com/Azure/azure-sdk-for-java/pull/20520/files#diff-119366ace1e5390d2aff952c5a6b8a0292d3a98638687277f8c5f41b373c4090R90 does), the request failed.

I've been checking the docs https://docs.microsoft.com/en-gb/rest/api/storageservices/formatting-datetime-values and it seems like YYYY-MM-DDThh:mm<TZDSuffix> is a valid date format. Just to add more context to the issue.

As a final note, the token was generated with the CLI
az storage container generate-sas

Any news on this?

I'm wondering if there's been some progress on this?

Hey, @fcofdez. Apologies for the delay. It looks like we slipped on this quite a bit, so thank you for following up.
I think you are correct that the PR in question will break a sas that doesn't have seconds. The code will eventually hit a line that tries to parse and then encode the sas query parameters and in so doing formats the date values it finds as "yyyy-MM-dd'T'HH:mm:ss'Z'".
We will work on getting a fix out for this


JSONPath based access to DataNode fields

will it include ad-hoc filters and projections? or will it be narrow down dot separated with optional index access only?

My initial feeling was for the purposes of the get on DataNode, we could support full JSONPath syntax.  For the purposes of using this for https://github.com/twosigma/webtau/issues/221 we only allow dot and index access but I haven't fully thought this through yet.  Wanted to get the issue raised so we don't forget.

Sorry, I meant for #221 we allow dot, index and filtration but no projection.

Discussed this with @MykolaGolubyev, we propose to allow dot and index based access only.  For example:
http.get('/foo/123') {
    body.get("a.b.c").should == "blah"
    body.get("a.b.d").should == ["first", "second", "last"]
    body.get("a.b.d[0]").should == "first"
    body.get("a.b.d[-1]").should == "last"
}


Youtube live event not announced

Hey!
I added couchbot to my discord server. Works nice except one thing.
I always making live event not "stream now". Somehow it's not announced on the discord feed however when the stream ends the vod is posted as uploaded video so it's detecting my channel.
Any suggestion? Or it's not working with events just stream now?
edit: this is the current config:

Owner Go Live Channel: general
Owner Published Channel: general
Owner Twitch Channel Feed Channel: Not Set
Go Live Channel: general
Published Channel: general
Greetings Channel: Not Set
Allow @ Role: True
Allow Thumbnails: True
Allow Greetings: True
Allow Goodbyes: False
Allow Live Content: True
Allow Published Content: True
Allow Owner Twitch Channel Feed: False
Use Text Announcements: False
Use YTG URLS For VOD Content: False
Live Message: Stream start!
Published Message: New vid!
Greeting Message: HF!
Goodbye Message: Default
Stream Offline Message: Default
Mention Role: Everyone
Time Zone Offset: 0

Regards,
Matthew

I'd be interested to know your server name so I can look up the config on my end. Events and Stream Now are essentially the same thing on the back end - no real distinction - and I've used both in testing/recently and had no issue.

Do you need a simple invite? Also probably the bot was offline when I finished the stream because now it didn't posted the finished vod either.

Yeah, I'm completely rewriting him. His data structure wasn't scalable to the numbers we've grown to. He goes offline every so often for an issue I could bandaid, but I'd rather just rewrite.
Just the name of the server/your discord name should suffice.

kEn's homeland is the name
Yeah I noticed multiple disconnects but it's okay and really appreciate your work :)

My Couchbot is also refusing to announce my YouTube streamers and it seems like it won't announce Picarto either.
Also this might be unrelated but I also can't check the live message for Picarto users.
My server is called Dauntless Unofficial Streaming Team

@lieblingkira Your config is wrong :) Hope in http://discord.couchbot.io - mention this post, and I can get you setup and fixed!!

@k3nnny I see nothing wrong :( Join the discord listed above - and let's work through this :)

I had 3 streams at the weekend and none of them announced in the feed. The vods are posted as video thats working fine. I'll join your server soon :)

Have same problem. When i removed all files from "live/youtube" folder it worked fine one more time. Cleaning folder before stream is a questionable solution, but it works :D
p.s. sorry for crappy english.

@RazCallahan I assume this is in the Self Host Bot?
@k3nnny Drop by the Discord, I have some further questions for you.


Feature request: Changes API

First, I should say that you have done a fantastic job with json-server!  Thank you for all of your hard work.
This evening, I ran into an issue where I would like to fake push notifications when data changes.  Would it be possible to implement a feature in json-server where, if data changes (POST, PUT, PATCH, DELETE), all ports on which json-server is listening will receive notification?  The payload would need to contain sufficient information to identify what changed and how.

This could be implemented with a variety of technologies:

Webhooks
SSE (Server-sent Events)
Websockets (this is already an issue in #870)

This would be an awesome feature!

Any updates on any of these issues?
They are all important and thus it seems weird if there is no interest to support them.
I tried to integrate ws along with json-server, but the server didn't seem to handle the upgrade requests in any way.
Anyone knows of a workaround to integrate websockets with json-server?

Hey @Gr3at can you share an example of this?

Hi @xabitrigo
I've added a base ws -- json-server integrated mock server that fits my needs in a GitHub repo.
You can check it here.

Thanks! It seems easier than I thought. It will be useful!!


File status in gutter

[ ] icons for inserted, modified lines
[ ] icon between lines for removed lines
[ ] gray out icons when changes are staged

So basically GitGutter?

Ha - pretty much, although I don't think GitGutter disambiguates between stated and unstaged changes in a file.  But perhaps that'd work better as a PR for GitGutter.  I'll close and revisit if necessary.


修复当设置条件为 Long转String时，但 java.util.List依然转换为array的bug

修复当设置条件为 Long转String时，但 java.util.List依然转换为array而不是array的bug，该bug导致前端用string在array 中无法查找定位。

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

https://github.com/alibaba/fastjson2
欢迎参与 fastjson2的开发


Add a link to the Q# language design repo

The list of repos in https://docs.microsoft.com/en-us/quantum/resources/contributing/ should include the recently created Q# language and core libraries design repo: https://github.com/microsoft/qsharp-language.

I opened a PR #1081 to solve this


Base implementation of inspect request impl.

There is a problem here with the inspect_request_impl function not being invoked which prevents handling input like "? std::vector" that is supposed to display documentation to the user.

@SylvainCorlay do you have a clue?

Hey there, sorry for not getting back to you earlier.
The way to trigger these inspect requests from a JupyterLab UI is to open the "Contextual Help" from the "Help" topbar menu.
Then any keystroke will trigger an inspect request when the contextual help is open.

Hi!
I have this working locally and will add some changes here (as discussed with @vgvassilev via Skype) as soon as I finish cleaning my tests up. (later today)

Hourray, this ia amazing!
Ping @loicgouarin for awareness.

Do we have a clang-format configuration for this project? If not, shall we use the standard one from llvm?

@SylvainCorlay what is the way to move forward here?

@mvassilev and @ioanaif thank you for your PR. It's a nice feature and it's great that it's working again! I'm really sorry for the delay and I will try to make a review soon. I also need to look at what @SylvainCorlay did in this project. Since I wrote xeus-cling, I haven't spent much time contributing to it. I will try to change that ;).
@vgvassilev we have no clang-format for this project for now but I can add one.

@SylvainCorlay, @gouarin, this looks mostly okay to me.
@SylvainCorlay, can you take a look and answer to the remaining questions?
I have a question on my own: Is there a way to avoid putting share/xeus-cpp/tagfiles/cppreference-doxygen-web.tag in the repository and download it from somewhere instead?

I have a question on my own: Is there a way to avoid putting share/xeus-cpp/tagfiles/cppreference-doxygen-web.tag in the repository and download it from somewhere instead?

I don't think this file exist otherwise.


Add vertical and horizontal orientation at the same time

Is it possible to use both orientations in a single page at the same time?

You can have a scrollview/listview As a child to achieve the behavior , I’ll still look into this 🙂

@triniwiz okay thanks. I have a dynamic pager and another with only 2 items. In 1 I have an exoplayer and in another I only show info. I can't access the video by id so I can pause or start it. How could I solve that?


tSQLt Test Adapter failing with an exception error

tSQLt Test Adapter for VisualStudio 2015 Update 3 stopped working with the error message which is below. I wondering is there a way to solve this, maybe some of the configurations are not correctly set. The most intriguing thing, that another project which has same setup of tSQLt tests works perfectly with the same VS and adapter.
-- tSQLt Test Adapter
v0.70
-- RUNSETTINGS FILE:

-- ERROR MESSAGE:
------ Discover test started ------
tSQLt Test Adapter, searching for tests...
tSQLt-Test-Adapter: Adding filter: Tests
An exception occurred while test discoverer 'tSQLtTestDiscoverer' was loading tests. Exception: Object reference not set to an instance of an object.
========== Discover test finished: 0 found (0:00:00.253) ==========
-- Folder structure

Tests

up_Test
-- up_Test.sql ( test schema)
-- test 01 test name.sql (test sp)
[....]

Can you try with this build or send me some sql files the reproduce it?
https://github.com/GoEddie/tSQLt-TestAdapter/releases/tag/vs2017


Added "FAQ" to the headline for SEO

The FAQ page was not found by the search engine when looking for "FAQ", because it did not contain this abbreviation.

Thank you @tjochmann !!

Thx @tjochmann !!


Feature request: Set document language with parameter for changeLanguageMode command

It would be useful if you could optionally pass a parameter to the workbench.action.editor.changeLanguageMode command to directly set the desired language instead of opening the language list.
See also the following questions regarding this issue:

http://stackoverflow.com/questions/40200788/how-to-change-language-mode-of-vscode-programmatically
http://stackoverflow.com/questions/41639887/how-can-i-create-a-custom-keybinding-to-change-syntax-highlighting-to-a-specific
https://social.msdn.microsoft.com/Forums/vstudio/en-US/25fd8d95-9de3-4465-8c64-82d68dcd2e47/shortcut-how-to-switch-between-typescript-and-html-with-one-or-two-shortcuts

And this comment: https://github.com/Microsoft/vscode/issues/8729#issuecomment-277838057
Ideally you would also be able to set a specific language with a keyboard shortcut, though I'm not sure if passing command parameters is possible in keybindings.json. If not that would be another issue I guess.

closing this as dupe of #1800. Tho it talks about a command this will be an API call like document.languageId = 'foolang'


Agenda: 14 Aug 2018

Please note we've moved back to the main community channel (#rust-community) to increase our reach. Once we've reached sufficient momentum we'll move back to the #rust-content.
5:30-6pm UTC on #rust-community on irc.mozilla.org. EIAL (Easy IRC Access Link): https://kiwiirc.com/client/irc.mozilla.org:+6667/#rust-community
Meeting logs (IRC): https://mozilla.logbot.info/rust-community
Agenda (shortened URL to this issue: bit.ly/2GSLLYA

Announcements - @booyaa 1 mins
Discord - @booyaa 1 mins
New Rust Site + Read Rust - @booyaa 5 mins

23 mins remaining
Please comment below if you wish to add anything to the agenda!

@booyaa

If anyoe wants to see the original email, I'll need to ask Wesley if he's okay for me to forward it on. I reckon he'll be okay

Yep that's fine.

Folks here's Wesley's email: https://gist.github.com/booyaa/60f537633f9a5b45c294ba84fefef297

Closed by #39


Can't build ld.lld: error: undefined symbol: cv::VideoCapture::VideoCapture(std::__cxx11::basic_s tring<char, std::char_traits, std::allocator> const&, int)

Gentoo Linux
installed from portage (so build from source)
OpenCV version: 4.8.1
rustc version: 1.73.0 (cc66ad468 2023-10-03) (gentoo)
build.txt

That's a very weird error to be honest. Judging by the 3 linking errors you're getting it looks like OpenCV functions are not accepting std::string typed arguments. The only situation that I can think about when it might be happening is that some of the so files are from OpenCV 3.x branch. In that branch the cv::String type was a separate definition unlike in 4.x where it's an alias to std::string. Can you double check that your OpenCV installation is clean?


Update for OpenBSD 7.6 release

OpenBSD 7.6 release is now available => https://www.openbsd.org/76.html
Please, could you update your vmactions/openbsd-vm action for this release? => use it as default release

fixed, use the latest v1


Update to work

This updates the url lists, the style targeting on Linkedin (and minorly on twitter)
Most importantly with this update is the tweak to the title update logic.
Also updates the prettier script to target only the src code

Wow, thanks for doing this! It’s been a while!!

😂 @1hauck and I have been talking intermittently over the last monthish and are looking to fix it up on Lever again; we're just waiting on the sandbox environment

ahh I see lol


all the dags are reporting errors airflow.exceptions.SerializedDagNotFound:

there are lot of dags which are reporting this error .. I am not sure what's the reason
it started to appear suddenly.
I am running airflow v2.1.1

@KulykDmytro @gowdra01 So it looks like it's an upstream airflow issue given https://github.com/apache/airflow/issues/18843?


Can you add unit tests?

It's not really clear at all how to set up the tests so they work properly... could you guys consider adding unit tests to this repo?

Tests have been added (a583221b2840cb88f57c6e40590aa2a325a530d8).


bug: Doesn't compile on bash

Did you check existing issues?

[X] I have read all the tree-sitter docs if it relates to using the parser
[X] I have searched the existing issues of tree-sitter-rust

Tree-Sitter CLI Version, if relevant (output of tree-sitter --version)
No response
Describe the bug
Doesn't compile on bash. Please, escape the # char in line 41 of the Makefile:
change
41: SONAME_MINOR := $(shell sed -n 's/#define LANGUAGE_VERSION //p' $(PARSER))

to
"41: SONAME_MINOR := $(shell sed -n 's/\#define LANGUAGE_VERSION //p' $(PARSER))"

Steps To Reproduce/Bad Parse Tree

run make on bash

Expected Behavior/Parse Tree
success
Repro
make

Can't reproduce.

Neither can I.

after make is run i have got the message:
Makefile:41: *** unterminated call to function 'shell': missing ')'.  Stop.
my shell is
$ sh --version
GNU bash, version 4.4.20(1)-release (x86_64-redhat-linux-gnu)
The problem is unescaped # char in line 41. Today i met with the same problem in the tree-sitter-c and tree-sitter-cpp parsers too.


Detect FreeBSD, use FreeBSD date(1)

Apologies for missing this in my initial pass -- FreeBSD's date(1) is not compatible, argument-wise. We can test uname as a fairly primitive check to vaguely see if we're on a FreeBSD system and use the appropriate date(1) arguments, falling back to the otherwise expected GNU/Linux assumption otherwise.
This is the last nit I have with build system foo, I'll move on to actually launching aprx and figuring out if there are any kinks remaining. A cursory glance at the pretty awesome documentation and serial-handling bits leads me to believe there's probably not much further work needed for FreeBSD bring-up. The only /dev assumption seems to be the reasonable /dev/null exists assumption, and everything else looks pretty configurable.

Whoops, omitted:
Tested with: FreeBSD-current
Tested with: Debian

Looking broader, is there other platforms where we'd see BSD date instead of GNU date? Would it make sense to try and match any "BSD" in the UNAME for this substitution, or should we leave that for the chump who wants to build Aprx on the third platform?

That's fair -- I hadn't looked into other platforms' date(1), but I've
arrived at the following list of confirmed BSD date(1) users:

FreeBSD (Given)
TrueOS (technically given -- closely FreeBSD-based)
NetBSD
OpenBSD
DragonflyBSD
Mac OS Sierra (Safe to assume earlier versions, as well -- OS X is
loosely BSD derived)

Given this list, I think it would be safe to amend my PR to look for "BSD"
or "Darwin" in uname and assume BSD date(1). There are other edge cases
(like TrueOS, I believe -- I don't have access to a TrueOS system at the
moment to test) that don't present themselves specifically as *BSD or
Darwin, but I'm not sure if it's important to address all of those with
uname tests.
It would be nice if there were an easy way to test whether date(1) has a
--date option or not, but I can't think of a way off the top of my head to
do this other than to just try a date --date=0 | grep -i illegal, and
that just feels dirty.

date might come back with an error code which we could capture somehow, but if there isn't an obvious and clean solution, I'm ok with starting with just FreeBSD/Linux. I've been perfectly happy only supporting Linux for the last two years, and I'd gladly take only those two platforms if it means more attention to more important parts of the codebase than the build system.

Oy, it's clearly time to go to sleep. =) Re-wrote this to fallback to BSD style on error and redirect stderr to /dev/null here as well. This works well enough that I can at least turn it into a port and any real errors cropping up won't be masked by false-positive errors.

I liked the first version more, even though it only supported FreeBSD. Much more clear what was going on.

Excellent, thanks! I'll be able to dork around with aprx here in the coming
weeks, so we'll see how the rest of this goes. =)
On Mar 21, 2017 11:11 PM, "Kenneth Finnegan"<EMAIL_ADDRESS>wrote:
Closed #46 https://github.com/PhirePhly/aprx/pull/46.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PhirePhly/aprx/pull/46#event-1009794782, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAncrv35EWicBTtTUXH07dWSmUF_TH7lks5roJ9YgaJpZM4MWYj8
.


How to use this with go 1.21?

gomod2nix seems to work with go_1_21 if you rebuild the binary manually, either by

installing it using go install:$ go install https://github.com/nix-community/gomod2nix@latest

building it using buildGo121Module:{
  fetchFromGitHub,
  buildGo121Module,
}: let
  pname = "gomod2nix";
  version = "1.5.0";
  src = fetchFromGitHub {
    owner = "nix-community";
    repo = "${pname}";
    rev = "v${version}";
    hash = "sha256-v2WMKnkpkz5YLYTBwh0NVgi3Xl4En249LCdrKTA4Nek=";
  };
  vendorHash = "sha256-4XoSRJKxLBYOZi/cEruFISn1KsxQZTDvnMfdS+oNK98=";
  subPackages = ["."];
in
  buildGo121Module {
    inherit pname version src vendorHash subPackages;
  }

You can then use buildGoApplication by setting go = pkgs.go_1_21:
buildGoApplication = {
  # (...)
  go = pkgs.go_1_21;
  # (...)
};

Here's a full flake.nix example:
flake.nix
{
  inputs = {
    nixpkgs = {
      url = "github:NixOS/nixpkgs/23.11";
    };
    gomod2nix = {
      url = "github:nix-community/gomod2nix";
      inputs.nixpkgs.follows = "nixpkgs";
    };
  };

  outputs = { self, nixpkgs, gomod2nix }:
    let
      allSystems = [
        "x86_64-linux" # 64-bit Intel/AMD Linux
        "aarch64-linux" # 64-bit ARM Linux
        "x86_64-darwin" # 64-bit Intel macOS
        "aarch64-darwin" # 64-bit ARM macOS
      ];
      forAllSystems = f: nixpkgs.lib.genAttrs allSystems (system: f {
        inherit system;
        pkgs = import nixpkgs {
          inherit system;
        };
      });
    in
    {
      packages = forAllSystems ({ system, pkgs, ... }:
        let
          buildGoApplication = gomod2nix.legacyPackages.${system}.buildGoApplication;
        in
        {
          default = buildGoApplication {
            # Required args.
            name = "nix-gomod2nix-example";
            src = ./.;

            #
            # In the latest versions of Go, the go.mod can contain 1.21.5
            # In that case, if the toolchain doesn't match, the go build operation will
            # try and download the correct toolchain.
            #
            # To prevent this, update the go.mod file to contain `go 1.21` instead of `go 1.21.5`.
            go = pkgs.go_1_21;

            # Must be added due to bug https://github.com/nix-community/gomod2nix/issues/120
            pwd = ./.;

            # Optional flags.
            CGO_ENABLED = 0;
            flags = [ "-trimpath" ];
            ldflags = [ "-s" "-w" "-extldflags -static" ];
          };
        });

      devShells = forAllSystems ({ system, pkgs }: {
        default = pkgs.mkShell {
          packages = with pkgs; [
            go_1_21
            gotools
            gomod2nix.packages.${system}.default # gomod2nix CLI
          ];
        };
      });
    };
}

If you drop this into a Go project, change the name argument value, and run nix develop, you'll get a shell containing gomod2nix.
Once you've ran gomod2nix, you'll have a gomod2nix.toml file in the directory.
You can then run nix build and you'll get a Go binary out.
Full example repo at https://github.com/a-h/nix-gomod2nix-example


Idea

If you have a great idea but it involves big changes, please file a ticket before making a pull request!

My idea would be to have a Powermode for Maaspower that send MQTT Messages and Topics to a Broker.
The idea behind is that you then can use Mosquitto  to control devices over Microcontrollers, or also node-red on a raspberry pi. Or any other MQTT Client, like a Powershellscript on a Windows Server that then start and stopp Hyper-V VM´s.
Example
MAAS Config
Broker url :  < your Broker >
Topic : < MQTT Topic like "location/devicetype/mydevice1">
Message: < ON ,OFF >
Would this

fit the scope of the project
?

Kind Regards
Ben3d

This sounds pretty good to me.
I don't know much about MQTT. But happy to look at PRs on this idea.


Can't find Wiki/ Documentation.

I was using 'com.theartofdev.edmodo:android-image-cropper:2.8.0' previously, Now I would like to build with this library.
But unfortunately the samples are with Kotlin and I would like some Java examples/ documentation. I can't find the wiki page of this repo. Kindly help.

Migration is here https://github.com/CanHub/Android-Image-Cropper#migrating-from-android-image-cropper
We've ditched Java samples but you can call the code from Java.

They are also in the README now :)
We'll focus on Kotlin now. It's state of the art.

Kindly share the old Java sample. I badly need to use this in one of my school project.

https://github.com/CanHub/Android-Image-Cropper/commit/ce3bf1b476e2ce0404abc1ba9c245c68dec42f23


如何去掉定位定位的权限

感谢作者解决iOS 14相册的权限问题，太给力了，我想再问一下代码里设置了cameraCanLocation=NO，info.plist也没有配置Location Always Usage Description，但是安装后，系统设置里还是有位置的权限。

cameraCanLocation 这是NO了并不会去请求定位权限

对，不会去请求，但是iOS14上，神奇的是系统设置里有为位置权限的入口，而且提交版本AppStore那边发邮件提示这个Your app's code references one or more APIs that access sensitive user data. The app's Info.plist file should contain a NSLocationWhenInUseUsageDescription key with a user-facing purpose string explaining clearly and completely why your app needs the data. Starting Spring 2019, all apps submitted to the App Store that access user data are required to include a purpose string. If you're using external libraries or SDKs, they may reference APIs that require a purpose string. While your app might not use these APIs, a purpose string is still required. You can contact the developer of the library or SDK and request they release a version of their code that doesn't contain the APIs. Learn more (https://developer.apple.com/documentation/uikit/core_app/protecting_the_user_s_privacy).

这个问题，只有把定位功能单独抽出来了来解决了。。

麻烦了麻烦了，看你有没有时间处理

定位问题已经修复了，可以查看最新demo。我这边用真机没有再出现这样的情况了

666，太厉害了，感谢作者！

你那边是正常的吗?

刚试了，正常啦


:bug: FIxed URI not updating on Clear content, minor formData improve…

…ment

Hey @liyasthomas,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: dbd3b050-590a-11ea-822e-0bf67ba4204b


IntrospectionError on Class/KClass recursive comparison

Hi
I encountered an issue with recursive comparison of Java/Kotlin class types. Basically those two tests are failing with the following error:
    @Test
    fun test() {
        val a: Any = Integer::class
        val b: Any = Long::class

        assertThat(a).usingRecursiveComparison().isEqualTo(b)
    }

    @Test
    fun test2() {
        val a: Any = Integer::class.java
        val b: Any = Long::class.java

        assertThat(a).usingRecursiveComparison().isEqualTo(b)
    }

org.assertj.core.util.introspection.IntrospectionError:
Can't find any field or property with name 'timestamp'.
Error when introspecting properties was :
- No getter for property 'timestamp' in java.lang.ref.SoftReference
Error when introspecting fields was :
- Unable to obtain the value of the field <'timestamp'> from <java.lang.ref.SoftReference@60b33bc>
	at app//org.assertj.core.util.introspection.PropertyOrFieldSupport.getSimpleValue(PropertyOrFieldSupport.java:88)
	at app//org.assertj.core.api.recursive.comparison.RecursiveComparisonConfiguration.dualValueForField(RecursiveComparisonConfiguration.java:631)
	at app//org.assertj.core.api.recursive.comparison.RecursiveComparisonConfiguration.lambda$getNonIgnoredActualFieldNames$3(RecursiveComparisonConfiguration.java:609)
	at java.base@<IP_ADDRESS>/java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:197)
	at java.base@<IP_ADDRESS>/java.util.stream.ReferencePipeline$2$1.accept(ReferencePipeline.java:179)
	at java.base@<IP_ADDRESS>/java.util.HashMap$KeySpliterator.forEachRemaining(HashMap.java:1707)
	at java.base@<IP_ADDRESS>/java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:509)
	at java.base@<IP_ADDRESS>/java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:499)
	at java.base@<IP_ADDRESS>/java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:921)
	at java.base@<IP_ADDRESS>/java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
	at java.base@<IP_ADDRESS>/java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:682)
	at app//org.assertj.core.api.recursive.comparison.RecursiveComparisonConfiguration.getNonIgnoredActualFieldNames(RecursiveComparisonConfiguration.java:615)
	at app//org.assertj.core.api.recursive.comparison.RecursiveComparisonDifferenceCalculator.determineDifferences(RecursiveComparisonDifferenceCalculator.java:276)
	at app//org.assertj.core.api.recursive.comparison.RecursiveComparisonDifferenceCalculator.determineDifferences(RecursiveComparisonDifferenceCalculator.java:189)
	at app//org.assertj.core.api.RecursiveComparisonAssert.determineDifferencesWith(RecursiveComparisonAssert.java:1258)
	at app//org.assertj.core.api.RecursiveComparisonAssert.isEqualTo(RecursiveComparisonAssert.java:160)

Those are simple examples for reproduction of the issue I encountered it in my code while comparing more complex objects of a class with KClass property.
assertj version: 3.20.2

https://github.com/assertj/assertj/blob/5faf53f07ce3fe685e9d3da271f19eb52e829b7a/assertj-core/src/main/java/org/assertj/core/api/AbstractAssert.java#L1090
AbstractAssert::usingRecursiveComparison is not public API.

At first glance, I don't see a correlation with the error above.
Could you please elaborate, @quaff?

At first glance, I don't see a correlation with the error above.
Could you please elaborate, @quaff?

It's not exposed API, assertThat(a).usingRecursiveComparison().isEqualTo(b) can not be compiled by javac, I think it's not expected behavior if it could be compiled by kotlin.

It is public in AbstractObjectAssert
https://github.com/assertj/assertj/blob/5faf53f07ce3fe685e9d3da271f19eb52e829b7a/assertj-core/src/main/java/org/assertj/core/api/AbstractObjectAssert.java#L1219

Your Java example is unrelated to the Kotlin one from @mkapiczy since you're passing a Class instance to assertThat(), which implicitly exposes Class assertions where the recursive comparison is not offered by design.

It seems that kotlin KClass is using AbstractObjectAssert and java class is using ClassAssert where it's protected.

Indeed: any type that doesn't have AssertJ specific assertions implicitly falls back to object assertions, where the recursive comparison is available.

However, my original issue was not with a direct comparison of KClass types but more complex domain types with a nested KClass property field which is too complex to share in a simple reproducible example.

Correct, that's the use case we should analyze to see how we can make it better.
I'll mark this and the previous comment as off-topics to avoid stealing the focus from the original problem.

I executed the second example on the latest code:
@Test
fun test2() {
    val a: Any = Integer::class.java
    val b: Any = Long::class.java

    assertThat(a).usingRecursiveComparison().isEqualTo(b)
}

which fails with:
java.lang.AssertionError:
Expecting actual:
  java.lang.Integer
to be equal to:
  long
when recursively comparing field by field, but found the following difference:

Top level actual and expected objects differ:
- actual value  : java.lang.Integer
- expected value: long

almost the same way as what happens in Java:
@Test
void test2() {
  Object a = Integer.class;
  Object b = Long.class;

  assertThat(a).usingRecursiveComparison().isEqualTo(b);
}

java.lang.AssertionError:
Expecting actual:
  java.lang.Integer
to be equal to:
  java.lang.Long
when recursively comparing field by field, but found the following difference:

Top level actual and expected objects differ:
- actual value  : java.lang.Integer
- expected value: java.lang.Long

It seems that the Kotlin b gets the Class instance of the long primitive type instead of the corresponding wrapper type. I'm not familiar enough with Kotlin to explain why the wrapper type instance is assigned to a while the primitive type instance is assigned to b, but I consider it unrelated to AssertJ and so what I experience looks good to me.

Running:
@Test
fun test() {
    val a: Any = Integer::class
    val b: Any = Long::class

    assertThat(a).usingRecursiveComparison().isEqualTo(b)
}

or:
@Test
fun test() {
    val a = Integer::class // KClass<Integer>
    val b = Long::class    // KClass<Long>

    assertThat(a).usingRecursiveComparison().isEqualTo(b)
}

yields:
java.lang.AssertionError:
Expecting actual:
  class java.lang.Integer (Kotlin reflection is not available)
to be equal to:
  long (Kotlin reflection is not available)
when recursively comparing field by field, but found the following difference:

field/property 'jClass' differ:
- actual value  : java.lang.Integer
- expected value: long

Aside from the long vs. java.lang.Long type also mentioned in the previous comment, the behavior looks good to me.

In 7406133f21ed66e3908a50c581ad0f0aaf059aa6, we decided to avoid the recursive comparison for JDK types and just rely on what equals does, even if some JDK classes don't override it. See also #2466 and #2450 for more background.
This change was released in 3.23.0.
@mkapiczy please upgrade to 3.23.0 or 3.24.2 (latest) to get the behavior I mentioned above.


How to authenticate users only once ?

The web application that we are building would need the user to authenticate himself over Google Ads, so that only the accounts accessible for him will be fetched and processed. Hence when the user hits the application url, the oauth page comes, where he logs in and allows access for our application. This will be a one time activity and then every time the user accesses the application url, since the auth token is already available in the request params, the user will not be asked to authenticate again, instead he will be automatically redirected to the main page of our application. We were able to enable this feature in our previous project, where we had used Adwords API, but using Google Ads API, we are unable to fix this issue.
This is the code that we are having :
UserAuthorizer userAuthorizer = UserAuthorizer.newBuilder()
.setClientId(
ClientId.of(adwordsProperties.getClientId(), adwordsProperties.getClientSecret()))
.setScopes(Arrays.asList(adwordsProperties.getScope()))
.setTokenStore(new MemoryTokensStorage())
.setCallbackUri(URI.create(callbackUrl)).build();
We have chosen the web app flow and here is the request URL been created :
<EMAIL_ADDRESS>In our previous application using Adwords API, we had implemented the same flow and there was no oAuth request created for every hit.
https://accounts.google.com/o/oauth2/auth?access_type=offline&client_id=xxx&redirect_uri=http://localhost:8080/oauth2callback&response_type=code&scope=https://www.googleapis.com/auth/adwords
We are trying to find out what it is that we are missing in our authentication process, due to which the user is prompted to go through oauth every time the application url is accessed.

Closed per internal discussion - this is an OAuth issue, not related to Java client library.


if there is no error ,please do not open the syntastic_auto_loc_list!

sometimes there is onle warning for java,the plugin will open the syntastic_auto_loc_list,i want it do not open the list windows when there are only some warning and without errors!
how can i config it,please?

Currently syntastic can't do that.  It is however possible to disable warnings, by setting g:syntastic_<filetype>_<checker>_quiet_messages to an apropriate value.  See :help 'syntastic_quiet_messages' for details.


cli: Make usage dynamically generate the list of subcommands

Previously, quilt usage used a hardcoded list of subcommands to
display to the user. This makes usage generate the list of subcommands
at runtime from those that are defined in cli.go.
I was considering writing a general purpose method to word wrap a
string to specified number of characters but decided that such a method
would require too much code to make it general (especially considering
it would only be used for printing the usage string).

Did you consider using this? https://github.com/mitchellh/go-wordwrap
The hashicorp stuff is typically very good

@ejj : yep, I saw that. I just didn't think it was worth it to bring in a whole other dependency for a small part of a relatively minor feature.

Luise proposed wrapping here: https://github.com/quilt/quilt/pull/941. Not sure if it's relevant.

@luise Updated with your feedback (and to match the new project name)


initial review version

Overall, there are potentially fairly large changes that would be expected for this to reach the BDE test driver standard (e.g., what is tested and in which order in the first 10 cases). However, I will inquire with Mike as to what is expected from this SOW.

Had a short discussion with Mike. The TLDR is that having the first test cases match what we do for other value-semantic types would make sense.  Please take a look at baltzo_localtimedescriptor for an example showing what is expected of the first cases. I suggest as a first iteration that we just reorder the test cases to vaguely match the order that we need (also, consulting nullablevalue TD for reference would probably help):

breathing test
default ctor, primary manipulators, and dtor
value ctor
basic accessors
print and output operators (see localtimedescriptor case 12 - just print "not implemented" in verbose mode)
equality-comparison operators
copy constructor
move constructor
swap
copy assignment
move assignment
bslx streaming (same deal as 5)

Again, just reorder existing cases for now (unless you see that matching this order is straightforward).

Overall, there are potentially fairly large changes that would be expected for this to reach the BDE test driver standard (e.g., what is tested and in which order in the first 10 cases). However, I will inquire with Mike as to what is expected from this SOW.

Had a short discussion with Mike. The TLDR is that having the first test cases match what we do for other value-semantic types would make sense. Please take a look at baltzo_localtimedescriptor for an example showing what is expected of the first cases. I suggest as a first iteration that we just reorder the test cases to vaguely match the order that we need (also, consulting nullablevalue TD for reference would probably help):

breathing test
default ctor, primary manipulators, and dtor
value ctor
basic accessors
print and output operators (see localtimedescriptor case 12 - just print "not implemented" in verbose mode)
equality-comparison operators
copy constructor
move constructor
swap
copy assignment
move assignment
bslx streaming (same deal as 5)

Again, just reorder existing cases for now (unless you see that matching this order is straightforward).

Re-arranged the tests as suggested. As well as any further suggestions, please advise on

current tests 13 and higher - I couldn't find where to fit them
negative only test - see testCase10e and testCase10f
not splitting copy and move testing
where to put test print statements and descriptions

There is also a test macro comment which is unresolved. Where inside the function would you like them ?

Overall, there are potentially fairly large changes that would be expected for this to reach the BDE test driver standard (e.g., what is tested and in which order in the first 10 cases). However, I will inquire with Mike as to what is expected from this SOW.

Had a short discussion with Mike. The TLDR is that having the first test cases match what we do for other value-semantic types would make sense. Please take a look at baltzo_localtimedescriptor for an example showing what is expected of the first cases. I suggest as a first iteration that we just reorder the test cases to vaguely match the order that we need (also, consulting nullablevalue TD for reference would probably help):

breathing test
default ctor, primary manipulators, and dtor
value ctor
basic accessors
print and output operators (see localtimedescriptor case 12 - just print "not implemented" in verbose mode)
equality-comparison operators
copy constructor
move constructor
swap
copy assignment
move assignment
bslx streaming (same deal as 5)

Again, just reorder existing cases for now (unless you see that matching this order is straightforward).

Re-arranged the tests as suggested. As well as any further suggestions, please advise on

current tests 13 and higher - I couldn't find where to fit them
negative only test - see testCase10e and testCase10f
not splitting copy and move testing
where to put test print statements and descriptions

There is also a test macro comment which is unresolved. Where inside the function would you like them ?


feat: Add support for datasource which can't be self-hosted for air-gapped version

Description
In earlier PR we opted out for plugins that can't be self-hosted for airgap thinking that as public internet is not available there might not be any usecase for these plugins. But we are seeing multiple levels of air gapping needed where customers still want the support for a few plugins like S3, Redshift, Firestore etc. which are offered by hosting providers.
https://theappsmith.slack.com/archives/C02K2MZERSL/p1681285395230519?thread_ts=1680632400.272319&cid=C02K2MZERSL
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Manual

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test

/ok-to-test


improve performance of encode()

Context
I was running the Chrome profiler for Node.js and found a nice heatmap on BIPF's index.js. I am trying to optimize seekKeyCached (i'll get to that soon), but I also stumbled upon a couple other easy wins.
Problem
The typeof encodingLengthers[type] !== 'function' isn't that cheap to run, see the before and after heatmaps. (Take a close look at the left-side measurements on the if condition) There's 74.8ms versus 6.5ms.
Solution
getType always returns the numbers we want, it doesn't return unexpected numbers, it will just return undefined when the type is invalid, so we can just directly check for type === undefined. And since it's possible to use undefined as a var name, it's safer to compare with void 0 (a synonym for undefined).
Before

operation, ops/ms
BIPF.encode 1.9179133103183736
JSON.stringify 4.557885141294439
BIPF.decode 7.1022727272727275
JSON.parse 5.408328826392645
JSON.parse(buffer) 5.537098560354374
JSON.stringify(JSON.parse()) 3.114294612270321
BIPF.seek(string)<PHONE_NUMBER>142857
BIPF.seek2(encoded)<PHONE_NUMBER>555555
BIPF.seek(buffer)<PHONE_NUMBER>909091
BIPF.seekCached(buffer) 1428.5714285714287
BIPF.seekPath(encoded) 500
BIPF.seekPath(compiled)<PHONE_NUMBER>666666
BIPF.compare()<PHONE_NUMBER>0689657

After

operation, ops/ms
BIPF.encode 1.9105846388995031
JSON.stringify 4.739336492890995
BIPF.decode 7.710100231303007
JSON.parse 6.079027355623101
JSON.parse(buffer) 5
JSON.stringify(JSON.parse()) 2.6874496103198067
BIPF.seek(string)<PHONE_NUMBER>30303
BIPF.seek2(encoded) 500
BIPF.seek(buffer)<PHONE_NUMBER>666666
BIPF.seekCached(buffer) 1250
BIPF.seekPath(encoded)<PHONE_NUMBER>956522
BIPF.seekPath(compiled)<PHONE_NUMBER>555555
BIPF.compare()<PHONE_NUMBER>142857
BIPF.seek(uniqueMsg)<PHONE_NUMBER>307693
BIPF.seekCached(uniqueMsg) 1000

Good point! I missed that one.
Just added a commit to change the if inside encodingLength.

Good point! I missed that one.
Just added a commit to change the if inside encodingLength.


Add update-site for scalariform

This change works in the same way as the change for the
scala-refactoring update-site in commit b6ed45040c1.

wrong merge branch


Golang plugin system

I'm not sure if this is the appropriate place to ask, but I'm wondering why the Golang runtime spawns a subprocess instead of using the plugin system to dynamically load the function.
Did you decide one is better than the other for Golang?

in some cases the other process is not a go function but an arbitrary binary (the go proxy is used for running functions from multiple languages now including swift, ruby, php, ...).
this may be the primary reason - @sciabarra can you comment?

The initial design was actually to build an executable including the proxy code to get a single executable. Community feedback was to build something that can work with other languages, most notably swift. Currently, the runtime spawning an executable is actually used for Go, Swift, PHP, Python, Rust, Java and Ruby. The communication layer is the old Unix pipe that is very fast and efficient. It was carefully measured and using it improved performances of other runtimes from a factor of 10 to 100. I  also did an experiment using a plugin and I did not notice any improvement in speed than using a pipe. Data are passed in-memory anyway using a pipe.


”Steierdorf” is listed in the drop-down menu

Description:
"Steierdorf" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Steierdorf" in the search bar.

Expected results
The location is listed in the drop-down menu as "Steierdorf, Caraș-Severin(Anina)".


Empty metatags on Tags page

Tags are categories which should have their unique description and metatags.
Is there a way to add unique meta data for every tag page? For example, for articles I can use Frontmatter to overwrite default meta tags.
So how can I write custom meta tags for every tag page? I mean /tags/*
Thanks!

In the new v2 version based on the app directory structure, you would be able to customize the metadata export file for that purpose.

great news! when do you plan to release the new version?

It's going to be a new version of the template. Currently working on it, releasing it maybe in a week's time?


DSD-189 initial setup for text input

Fixes #DSD-189
https://jira.nypl.org/browse/DSD-189
This PR does the following:
-Adds TextInput component
Front End Review:

[ ] View the example in Storybook

The placeholder text's style and color don't match the Figma designs.

@kaplanmichael
I've addressed all the issue that you noted in your review.  Where do we go from here?


Show a successful activity log when pasting one existing empty emulator blob container

Storage Explorer Version: 1.16.0-dev
Build: 20201010.6
Branch: main
Platform/OS: Windows 10
Emulator: <IP_ADDRESS>
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Install and run Emulator -> Launch Storage Explorer.
Expand Local & Attached -> Storage Accounts -> (Emulator - Default Ports) (Key) -> Blob Containers.
Create a new blob container and copy it -> Right click the Blob Containers node -> Paste the blob container.
Check the activity log.

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

More Info:

Show a failed activity log when pasting one existing table.

This issue doesn't reproduce for nomarl accounts.

edge case won't fix


260 user messages

@snasif Kevin needed this merged so you'll need to do a new PR if you're still on this branch


⚠ Fetchers has degraded performance

In b16d29f, Fetchers ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 371 ms

Resolved: Fetchers performance has improved in b484c11.


Add sap fieldglass approval connector

Description
Generate the SAP Fieldglass Approval connector using the official OpenAPI specification.
https://api.sap.com/api/approvals/resource
Fixes https://github.com/ballerina-platform/ballerina-extended-library/issues/275
One line release note:

Generate the SAP Fieldglass Approval connector using the official OpenAPI specification.

Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Not tested
Test Configuration:

Ballerina Version: SL Beta 6
Operating System: Ubuntu 20.04
Java SDK: 11

Checklist:
Changes to OpenAPI definition

[x] Modified info section of the OpenAPI definition file - Example

description - what the connector is about
x-ballerina-init-description OpenAPI extension - connector initialization details
x-ballerina-display OpenAPI extension - connector name and path to the connector icon

[x] Added proper description each API Key, if API Key authentication is defined in OpenAPI definition file

Changes to connector module

[x] Added license header at the top of each .bal file.
[x] Added Package.md as per the guide here
[x] Added Module.md for the module of the  connector as per the guide here
[] Added connector icon to the root of the connector. Icon needs to be a png of 200x200 px size with name icon.png
[x] Added Ballerina.toml file with following information. For keywords refer to guide

Verifying connector module

[x] Compiled the connector successfully with the targeted ballerina version.
[ ] Conducted smoke tests on the connector (Recommended when OpenAPI definition is obtain from an unofficial source)

Security checks

[x] Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines?
[x] Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets?

Need to add icon

Need to add icon

Added


Use DbProviderFactory to merge db health checks

https://github.com/Xabaril/AspNetCore.Diagnostics.HealthChecks/issues/1

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

I'm closing because this PR is stale, and we have moved the source code for this project. If this is something you would still like to contribute, open a new PR on this repo: https://github.com/aspnet/AspNetCore.
Thanks,
Nate


[Feature] Support storeValue within IIFE bindings

Summary
Support storeValue within IIFE bindings
For instance:
{{(() => {
    let a=["None","Alpha","Beta","Gamma","Delta"];
    storeValue("nameTypes", a, false);
    return "Page Title";
})()}}

Motivation

Use it as a workaround for the on page load event

@rishabhsaxena we don't plan to support this because these types of functions should not be triggerable in display properties. This will be supported in the JS Editor


Background color is applied in the wrong place when rendered segment type is not a TextExt

In the RichText demo applying styles works correctly, with the exception of the background color. When the editor just has text, there is no issue. When the text has an image insert before the selection, the application of the background color is negatively offset by the number of images before the selection.

Meh... I guess my title doesn't quite state the issue correctly either...
Still, the problem occurs in ParagraphText. I think there's a line somewhere in there that only considers TextExt objects when constructing the Path background shape.

Aye. The issue arises here. We'd need a way to determine whether a given rendered node shares the same background color as another consecutive one while accounting for different subclasses of Node that are used to render the segment on the screen. While this issue only mentions the background color, it could equally apply to the rtfx-specific border and underline styles, too.
Due to accounting for non-text objects, we'd also need to calculate the length of a segment to determine how long the range of that segment's background color or border is. Calling SegmentOps#length on the SEG object would easily calculate its length, but it would mean adding the ops object as one of the constructor parameters of ParagraphBox and ParagraphText.
One way would be to add a lambda (Function) that can be passed an argument and return an Optional<Paint> to determine whether two nodes share the same background color. If the the passed-in object is be a StyledSegment<SEG, S>, this would help us determine the background color for some use cases (e.g. when inline css is used) but not for others (when a style class is used). However, passing in the rendered node with type Node will require instanceOf checks and casting before the background color could be determined. (To get around this, we'd need to use the Visitor pattern and make all rendered nodes implement it. Definitely not something I like).


Add tests for IW debugger using the Jupyter Protocol on the web

[x] Test debugging local connections using the Jupyter protocol (done here #10263)
[x] Test debugging remote connections using the Jupyter protocol (done here #10263)
[ ] Test debugging remote connections using the Jupyter protocol on the web

Blocked on Add tests for IW on web

I assume this isn't actually done?

@DonJayamanne this seems like a debt item--if not, are there verification steps for this?


Add subtract function for dynamic template

Signed-off-by: Sai Sindhur Malleni<EMAIL_ADDRESS>Description
Fixes

Closing this out as we now have the sprig library


Add Emerging Tech to league-content

Adding more information to Emerging technologies section of League Content

@jlooper Check out this PR


noUncheckedIndexedAccess does not narrow properly with "prop" in obj

Bug Report
Apologies if this is a duplicate, but I wasn't able to find anything.
🔎 Search Terms
noUncheckedIndexedAccess property in object
⏯ Playground Link
Playground link
💻 Code
declare function invariant(condition: boolean): asserts condition;

function foo(obj: Readonly<Record<string, string>>) {
  invariant(obj && "test" in obj);
  // this one below does narrow to `string`
  // invariant(!!obj["test"]);

  const x = obj["test"] // string | undefined but should be string
}

🙁 Actual behavior
x is typed as string | undefined
🙂 Expected behavior
x should be typed as string

Seems related to https://github.com/microsoft/TypeScript/issues/41848

This isn’t really related to noUncheckedIndexedAccess—in narrowing doesn’t ever take undefined out of the domain of a property: https://www.typescriptlang.org/play?noUncheckedIndexedAccess=true&ts=4.3.0-beta#code/CYUwxgNghgTiAEA3W8D2AjAVgLngb3gBcQBnQgflzJgEsA7Ac3gF8BuAKHZoDN4AKAETEyA+PTRYAlPnbx4YVHTLwAHvAC8EzAG0hpQgIC6HOQqWF4ATw1aAdMMIdm7IA
You should check for equality / non-equality with undefined if you want that behavior.

Not sure I understand. The type of obj cannot have undefined values in it as per its definition.
How does undefined fit into Readonly<Record<string, string>>?

Ah, I see. When you set --noUncheckedIndexedAccess, the read type of property access on a Readonly<Record<string, string>> goes from string to string | undefined. But we don’t currently have the ability to model the difference between properties that are undefined and properties that are missing. If we were to introduce a missing type in the future, we would be able to implement the behavior you expect here—the read type from that index signature would be string | missing, and you could know that it’s not missing via an in test. But right now, because we lack the concept of missing, an in test on an object with an index signature does nothing.

With exactOptionalPropertyTypes, unchecked index access could map to
missing and not undefined.
Then key in guarded access should work equally on optionally declared properties and index signatures.

This seems like a bug rather than a design limitation then.

I agree that this is neither a design limitation nor working as intended. This is simply a bug and @derolf's example proves it.


CSS Not Working Well

Browser: Chrome
OS: macOS v.13.2
Processor: M1 Chip
CRA: v.6.2.0
I doubt this is an issue with your code and more of an issue with my code. So here's the thing...
I have a react app (CRA, craco v.6) that I've been working on and I was looking for a theme to make it look better. So I started with the home page. I downloaded your repo and slowly moved necessary files to my app.
Now, I believe I have everything set but not all the styling is being used.

I'm using tailwind npm<EMAIL_ADDRESS>and tailgrids v.2.0.1. Do I need to use the latest version of tailwind?

This template is independent from TailGrids, did you run build command before trying to use it?