MINIFICPP-732 - Add property to expose librdkafka "debug" values for …

…PublishKafka
Thank you for submitting a contribution to Apache NiFi - MiNiFi C++.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

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

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

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

[ ] Is your initial contribution a single, squashed commit?

For code changes:

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file?
[ ] If applicable, have you updated the NOTICE file?

For documentation related changes:

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

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

Merging this as it works and adds feature, property validation will be done in scope of the linked follow-up.


Advanced Syncing of Grouped Slicers documentation is incorrect

This page is not correct for the Advanced Sync slicers. You do not have to group the slicers by selecting 2+ slicers and then right-clicking and selecting Group.
You simply select each slicer you want to sync, one at a time, then type in the same Group name (whatever random text you want - it is unrelated to anything in the model) in the Advanced Slicer group name field. This is actually confusing terminology as I though the "group name" referred to in the Advanced Slicer field was the same Power BI object "group name" for visual grouping, but it isn't.
You can see in the image below the "Real Slicer" and "Pseudo Slicer" are not grouped by the Format/Group command, but simply have the same "group name" in the advanced slicer dialog box.

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

ID: 2f177d47-9600-eab6-a282-09d40d5c2dff
Version Independent ID: 792909b9-e2fa-8a31-1da9-2223307e799d
Content: Slicers in Power BI - Power BI
Content Source: powerbi-docs/visuals/power-bi-visualization-slicers.md
Service: powerbi
Sub-service: pbi-visuals
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Hi, @edhans -- good catch, you're right! Making the change today. It'll go live tomorrow Tues Oct 12 in the afternoon, Pacific time.


Adding a Maven Dependency for this Chuck Noris from Maven Repo Central

Adding a Maven Dependency for this Chuck Noris from Maven Repo Central

Completed


Default Theme: allow to link directly to overloaded signatures - Currently there is no way to link to the nth overloaded definition of a function

Search Terms
overloaded function method direct linking
Problem
Currently, it's not possible to link directly to a specific overloaded method definition.
Suggested Solution
Clicking on any method overload should change the current URL so that it can be linked directly:

Sorry if this has been asked before, could not find another issue asking for this

This is a good idea. I don't know if it is possible with the current themes + anchor generation, but would love to be proven wrong.


Check in Typescript 3.5.3

This is required for https://chromium-review.googlesource.com/c/chromium/src/+/1728584 to build successfully.

Is this still relevant?

I will close for now.


Revert "cleanup and implement toggle visibility on choose card"

This reverts commit 752cfd01e9281755cf85ed545f670d2a729dddcf.

Closed, git history fixed by alternative means


feat: Add target-environments-directory to the global config

Fixes: #997
Todo:

[x] Fix tests
[x] Add warning on already local to project installed environments
[x] Add documentation
[x] Add symlink from target directory folder to local .pixi folder.

We should probably also print a warning when we already see a populated .pixi directory to remove it because this setting is set.
Also, we might want to think about printing a warning when we see symlinks?
For example, your mounted devcontainer might have this setting enabled but locally you don't want this behavior. (although there are better options in general for pixi in devcontainers as elaborated in https://github.com/prefix-dev/pixi/issues/997#issuecomment-2018525722)

What do you mean with warning when symlinking?

When I have a filled .pixi directory instead of a symlink already present, we should warn the user instead of directly deleting it.

This PR move the complete .pixi dir to the specified target dir. But I see a situation where it makes sense to only move the "heavy" environments. So would it make sense to only move the .pixi/envs to the target? @pavelzw @baszalmstra?
Currently it is already weird as you can specify the configuration in the projects .pixi folder but then symlinking it is not possible.

@baszalmstra I looked at all your review points and updated the symlink function a little to also do something on windows. With adding a README.txt file to the .pixi folder to tell the user looking there that it was moved.

Fixed the unknown field issue:

Not sure yet why we run the config parsing twice.

A few thoughts that came up:
This configuration option is hard to "disable" locally since it mixes two things:

I want my .pixi folder external
Where to put the .pixi folders

My proposal would be to split the two options into something like the following:
external-pixi-folder: false / true (false is default)
external-pixi-folder-location: (path to pixi folder root thingy)

The external-pixi-folder-location would, by default, be inside the global pixi folder, e.g. under:
~/.pixi/pixi-envs-v1/<project-name>-<hash>/...

or we could also drop it next to the package cache (although I am unsure wether a "cache" location is the right place). On the other hand, it "is" a cache since the definition is in the lockfile).
The benefit of having the pixi envs next to the cache would be that hard linking should be possible under most configurations (while the home folder might be on a different partition, or NFS, etc.).

This configuration option is hard to "disable" locally since it mixes two things:

For documentation of thoughts, I believe these shouldn't be two things. You either want to move it or you don't. I can't see a situation where defining both has a beneficial experience as it is not explicit what it will be doing when you have those two defined but set to false.
What about this, we kinda keep the same for the global config, just a rename.
~/.config/pixi/config.toml:
pixi-folder-location = "/etc/pixi_environments"

Allow for relative paths in this configuration.
~/myproject/.pixi/config.toml:
# This just overrides the global configuration.
pixi-folder-location = "."

The only problem with that is that we move the complete .pixi folder to the new location. Thus a config.toml will also needs to move with the current design.
Sidenote: the name is not the best yet, as we pixi-folder might make users think its the ~/.pixi folder which listens to $PIXI_HOME.

I'm thinking of the same idea of:

My proposal would be to split the two options into something like the following:
external-pixi-folder = false  # or true (false is default)
external-pixi-folder-location = "/some/path/pixi-root"  # (path to pixi folder root thingy)

but instead using just one config
/// Option to toggle project environments to be detached from the project directory
enum DetachedEnvironments {
    Boolean(bool),
    Path(PathBuf),
}

detached-environments = Option<DetachedEnvironments>
//
// - None/Some(false): the default behavior, envs stored within `<project_root>/.pixi`
// - Some(true): toggle detached environments on, envs stored within `<pixi_cache_dir>/envs/<hash(project-x)>`
//    * `<pixi_cache_dir>` would be `XDG_CACHE_HOME/pixi` or its synonym
//    * By storing envs in `pixi_cache_dir` it is possible to leverage command like `pixi clean`/`pixi cache clean`
//      or whatever command we will introduce to cleanup caches with envs. It could be configurable, e.g.
//      `pixi clean --[no-]envs`
// - Some(my_path): toggle detached environments on, envs stored in `my_path/<hash(project-x)>`

This would be much more configurable yet concise and only one key is added.

@chawyehsu We all like your idea! I'm going to implement it like that including only moving the .pixi/envs folder and not the complete .pixi folder.


correct I2C_TIMEING_VAL_ERR_STR typo to I2C_TIMING_VAL_ERR_STR

 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.

Thanks for your contribution.


Errors list not Paging

I see in the screenshot from the Readme.md that it shows "Loaded 106 of 106 errors"
My Errors list initially loads 50 errors but no more than that.  It is not loading more errors when I scroll to the bottom of the list.
Is the Errors list supposed to support paging?
Using NuGet package version 2.0.7

Unfortunately, the changes brought by zolaikhahammad don't work in my case. There are 2 issues:

When my app customize the Elmah base path, the "Show Details" links are wrong.
Because I use a custom store (ravendb database), my error ids are of the form ElmahErrors/1234. This seems to render the link to a specific error page invalid because of the "/' and it redirects to the list of errors. Note that I have no problems with this customized id and the original ElmahCore package. Only with zolaikhahammad modifications.


Media Playback is stopped when application is closed

Steps to reproduce

Install application that can play media even when user kills the main process (for example VLC)
Play media (audio in VLC)
Close the app

✔ Expected Behavior
Media should continue playing as it is on Android device when user kills the main process
❌ Actual Behavior
Media playback (and probably whole application) stops
Other Software
VLC for Android 3.5.4
Please specify the version of Windows Subsystem for Android
2309.40000.10.0

This is by design in Windows Subsystem for Android. If there is no user visible UI including a Window media playback is stopped to match the behavior of other apps on the platform.

This sadly doesn't match the behavior of the Android itself:
https://github.com/microsoft/WSA/assets/40525630/a176085b-0178-4831-b4a4-ba2079e98aeb
There is still an UI (although just notification)


Added first & last property and also items_length attribute to the pagination

#567
first and last property added because of the mentioned issue.(you can find more info in the issue)
Added items_length attribute which shows the number of items for current page that is useful for showing the number of items in a page specially the last one if not total % per_page == 0.

Duplicate of #586


Can't use breeze is Scala 3.1.0

When I try to use this library in Scala 3 I get the following issue:
I am using cats-effect, spire already as is. Not sure if the cross-compilation is setup correctly.
error] Modules were resolved with conflicting cross-version suffixes in ProjectRef(uri("file:/Users/swoorup.joshi/personal/finflow-scala/"), "common"):
[error]    org.typelevel:spire-util _3, _2.13
[error]    org.scala-lang.modules:scala-collection-compat _3, _2.13
[error]    org.typelevel:algebra _3, _2.13
[error]    org.typelevel:spire-platform _3, _2.13
[error]    org.typelevel:spire _3, _2.13
[error]    org.typelevel:cats-kernel _3, _2.13
[error]    org.typelevel:spire-macros _3, _2.13
[error] stack trace is suppressed; run last common / update for the full output
[error] (common / update) Conflicting cross-version suffixes in: org.typelevel:spire-util, org.scala-lang.modules:scala-collection-compat, org.typelevel:algebra, org.typelevel:spire-platform, org.typelevel:spire, org.typelevel:cats-kernel, org.typelevel:spire-macros
[error] Total time: 3 s, completed 13 Jan. 2022, 6:04:14 pm
sbt:root>

Can be fixed by https://github.com/scalanlp/breeze/pull/833

Hi @dlwh Any chance for a new release? The last version in scaladex appears to be 10 months ago.
https://index.scala-lang.org/scalanlp/breeze/artifacts/breeze?binary-versions=_3#

doing it as we speak!
On Sun, Aug 21, 2022 at 8:57 PM Swoorup Joshi @.***>
wrote:

Hi @dlwh https://github.com/dlwh Any chance for a new release? The last
version in scaladex appears to be 10 months ago.
https://index.scala-lang.org/scalanlp/breeze/artifacts/breeze?binary-versions=_3#
—
Reply to this email directly, view it on GitHub
https://github.com/scalanlp/breeze/issues/835#issuecomment-1221772716,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAACLIJOI5DKMYPDQZVQWH3V2L3C5ANCNFSM5L3AELLQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Ok 2.1.0 should be hitting central as we speak.

Awesome

Not sure why, but I'm getting what appears to be this error when I try to upgrade breeze version from "2.0.1-RC2" to "2.1.0".
On the off chance that someone can see a relevant clue in my stack dump:
My build compiles for "2.13.8" and "3.2.0", and there's no problem with the scala2 build.   Here's what happens during the scala3 compile:
[error] java.lang.RuntimeException: Conflicting cross-version suffixes in: org.typelevel:cats-kernel
[error]         at scala.sys.package$.error(package.scala:30)
[error]         at sbt.librarymanagement.ConflictWarning$.processCrossVersioned(ConflictWarning.scala:39)
[error]         at sbt.librarymanagement.ConflictWarning$.apply(ConflictWarning.scala:19)
[error]         at sbt.Classpaths$.$anonfun$ivyBaseSettings$71(Defaults.scala:3204)
[error]         at scala.Function1.$anonfun$compose$1(Function1.scala:49)
[error]         at sbt.internal.util.$tilde$greater.$anonfun$$u2219$1(TypeFunctions.scala:62)
[error]         at sbt.std.Transform$$anon$4.work(Transform.scala:68)
[error]         at sbt.Execute.$anonfun$submit$2(Execute.scala:282)
[error]         at sbt.internal.util.ErrorHandling$.wideConvert(ErrorHandling.scala:23)
[error]         at sbt.Execute.work(Execute.scala:291)
[error]         at sbt.Execute.$anonfun$submit$1(Execute.scala:282)
[error]         at sbt.ConcurrentRestrictions$$anon$4.$anonfun$submitValid$1(ConcurrentRestrictions.scala:265)
[error]         at sbt.CompletionService$$anon$2.call(CompletionService.scala:64)
[error]         at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
[error]         at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)
[error]         at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
[error]         at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
[error]         at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
[error]         at java.base/java.lang.Thread.run(Thread.java:834)
[error] (apps / update) Conflicting cross-version suffixes in: org.typelevel:cats-kernel
[error] Total time: 2 s, completed Sep 13, 2022, 1:42:13 PM

Can you use one of the SBT dependency tree plugins that are out there
different the dependency graph I suspect the conflict will be pretty clear
in there
On Tue, Sep 13, 2022, 12:49 PM philwalk @.***> wrote:

Not sure why, but I'm getting what appears to be this error when I try to
upgrade breeze version from "2.0.1-RC2" to "2.1.0".
On the off chance that someone can see a relevant clue in my stack dump:
My build compiles for "2.13.8" and "3.2.0", and there's no problem with
the scala2 build. Here's what happens during the scala3 compile:
[error] java.lang.RuntimeException: Conflicting cross-version suffixes in: org.typelevel:cats-kernel
[error]         at scala.sys.package$.error(package.scala:30)
[error]         at sbt.librarymanagement.ConflictWarning$.processCrossVersioned(ConflictWarning.scala:39)
[error]         at sbt.librarymanagement.ConflictWarning$.apply(ConflictWarning.scala:19)
[error]         at sbt.Classpaths$.$anonfun$ivyBaseSettings$71(Defaults.scala:3204)
[error]         at scala.Function1.$anonfun$compose$1(Function1.scala:49)
[error]         at sbt.internal.util.$tilde$greater.$anonfun$$u2219$1(TypeFunctions.scala:62)
[error]         at sbt.std.Transform$$anon$4.work(Transform.scala:68)
[error]         at sbt.Execute.$anonfun$submit$2(Execute.scala:282)
[error]         at sbt.internal.util.ErrorHandling$.wideConvert(ErrorHandling.scala:23)
[error]         at sbt.Execute.work(Execute.scala:291)
[error]         at sbt.Execute.$anonfun$submit$1(Execute.scala:282)
[error]         at sbt.ConcurrentRestrictions$$anon$4.$anonfun$submitValid$1(ConcurrentRestrictions.scala:265)
[error]         at sbt.CompletionService$$anon$2.call(CompletionService.scala:64)
[error]         at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
[error]         at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)
[error]         at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
[error]         at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
[error]         at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
[error]         at java.base/java.lang.Thread.run(Thread.java:834)
[error] (apps / update) Conflicting cross-version suffixes in: org.typelevel:cats-kernel
[error] Total time: 2 s, completed Sep 13, 2022, 1:42:13 PM
—
Reply to this email directly, view it on GitHub
https://github.com/scalanlp/breeze/issues/835#issuecomment-1245883605,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAACLILXXHHBMDVRMGH2PUDV6DLDTANCNFSM5L3AELLQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Can you use one of the SBT dependency tree plugins that are out there different the dependency graph I suspect the conflict will be pretty clear in there

Wow, thanks!  I did try that previously, but this time it showed that I have a conflict with circe, which requires an older version of cats-kernel.

Can you use one of the SBT dependency tree plugins that are out there different the dependency graph I suspect the conflict will be pretty clear in there
…
On Tue, Sep 13, 2022, 12:49 PM philwalk @.> wrote: Not sure why, but I'm getting what appears to be this error when I try to upgrade breeze version from "2.0.1-RC2" to "2.1.0". On the off chance that someone can see a relevant clue in my stack dump: My build compiles for "2.13.8" and "3.2.0", and there's no problem with the scala2 build. Here's what happens during the scala3 compile: [error] java.lang.RuntimeException: Conflicting cross-version suffixes in: org.typelevel:cats-kernel [error] at scala.sys.package$.error(package.scala:30) [error] at sbt.librarymanagement.ConflictWarning$.processCrossVersioned(ConflictWarning.scala:39) [error] at sbt.librarymanagement.ConflictWarning$.apply(ConflictWarning.scala:19) [error] at sbt.Classpaths$.$anonfun$ivyBaseSettings$71(Defaults.scala:3204) [error] at scala.Function1.$anonfun$compose$1(Function1.scala:49) [error] at sbt.internal.util.$tilde$greater.$anonfun$$u2219$1(TypeFunctions.scala:62) [error] at sbt.std.Transform$$anon$4.work(Transform.scala:68) [error] at sbt.Execute.$anonfun$submit$2(Execute.scala:282) [error] at sbt.internal.util.ErrorHandling$.wideConvert(ErrorHandling.scala:23) [error] at sbt.Execute.work(Execute.scala:291) [error] at sbt.Execute.$anonfun$submit$1(Execute.scala:282) [error] at sbt.ConcurrentRestrictions$$anon$4.$anonfun$submitValid$1(ConcurrentRestrictions.scala:265) [error] at sbt.CompletionService$$anon$2.call(CompletionService.scala:64) [error] at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264) [error] at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515) [error] at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264) [error] at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128) [error] at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628) [error] at java.base/java.lang.Thread.run(Thread.java:834) [error] (apps / update) Conflicting cross-version suffixes in: org.typelevel:cats-kernel [error] Total time: 2 s, completed Sep 13, 2022, 1:42:13 PM — Reply to this email directly, view it on GitHub <#835 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAACLILXXHHBMDVRMGH2PUDV6DLDTANCNFSM5L3AELLQ . You are receiving this because you were mentioned.Message ID: @.>

Thanks for the suggestion, an older version of circe was causing this problem, sorry for the noise!


Should considerations be normalized as a mean / percentage?

My understanding of utility AI is still nascent so this is part question part feature request — I'm in early stages of setting up and tuning my ai agent I find myself fiddling with consideration curve values a lot just to compensate for one behavior having a single consideration and another having many. This makes the system feel quite brittle, because when I add a new consideration to one behavior, I have to go and update consideration weights for other behaviors to account for the summed extra.
From my understanding, shouldn't the considerations be averaged to give the behavior a normalized score? So all behaviors are scored (for example) 0 - 1 based on the weighted curves of the considerations it has, no matter how many it has?

Ignore me I just discovered the mean option in consideration and sensor groups 🤦🏻♀
Maybe a documentation issue? Maybe a note in tutorial "next steps" would help, also noticed the tutorials don't mention behavior actions


bun install failing with "error: failed to enqueue lifecycle scripts for ... :ENOENT"

$ bun --version
1.0.35

bun install v1.0.35 (940448d6)
  📦 Installing [3/198]
error: failed to enqueue lifecycle scripts for @sveltejs/kit: ENOENT
  📦 Installing [11/198]
error: failed to enqueue lifecycle scripts for esbuild: ENOENT

 +<EMAIL_ADDRESS> +<EMAIL_ADDRESS> +<EMAIL_ADDRESS> +<EMAIL_ADDRESS> +<EMAIL_ADDRESS> +<EMAIL_ADDRESS> +<EMAIL_ADDRESS> +<EMAIL_ADDRESS> +<EMAIL_ADDRESS> +<EMAIL_ADDRESS> +<EMAIL_ADDRESS> +<EMAIL_ADDRESS>
 163 packages installed [52.00ms]
Failed to install 2 packages

Try using npm
npm i
npm run dev

That does work to get past that point, but the UI is then mostly unresponsive. I can get it to prompt for a project name, but it does nothing with whatever input I give it.

same issue in v1.1.29 (6d43b366)


No 509 problem again (AS Preview depends on PC language settings)

Describe the bug
As the title says.
To Reproduce
Steps to reproduce the behavior:
MultiLanguagePreviews Check for parts with annotations.
Expected behavior
Revert to the status resolved in #509
Screenshots
If applicable, add screenshots to help explain your problem.

By the way, fixing this part as shown in the attached image appears to solve the problem.
However, I think this response is probably not correct since the No. 627 response makes it look like this, is that correct?

I think this PR is affected by this issue.
We might have to divide the const for test and for preview.
https://github.com/DroidKaigi/conference-app-2023/pull/809

I'll divide these locale string :raised_back_of_hand:


advert data

Hi,
In the below image:

Any reason why NimBLE would not report a payload for the device starting E52E?
using the code:
  BLEAdvertisedDevice *advertisedDevice = ad->advertisedDevice;

  uint8_t* payload = advertisedDevice->getPayload();
  size_t payloadlen = advertisedDevice->getPayloadLength();
  if (payloadlen  && (maxlen > 30)){ // will truncate if not enough space
    strcpy(p, ",\"p\":\"");
    p += 6;
    maxlen -= 6;
    dump(p, maxlen-10, payload, payloadlen);
    int len = strlen(p);
    p += len;
    maxlen -= len;
    *(p++) = '\"'; maxlen--;
  }

  int svcdataCount = advertisedDevice->getServiceDataCount();
  if (svcdataCount){

both payloadlen and svcdataCount appear to be zero?
10:45:07.079 MQT: tele/tasmota_BLE3_98BFAC/BLE = {"details":{"mac":"E52E1C92779F","RSSI":-51}}
br,
Simon

I can't think of a reason for this. If you call the toString method on the advertisedDevice does it produce anything?

closing because the guy asking was not me.....


adds ANSI

Here is my contribution

 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: indra8981 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for taking the time to contribute this pull request. However, we are not able to accept contributions to this historical reference repository at this time.


Error while closing orders

Hi
I use Kotak Api to trade in bank nifty options.
i have encountered one error while closing the open order. The system i have coded is if an order is placed at a particular time (let's say 09:16) must be traded off at a particular time(let's say 09:45).
I had inserted one condition which checks at 09:45 if the order is traded or not and if not then it modifies the order to trade at 09:45.
In between, if i believe the market will not be in my favor, I modify the order myself manually before the coded time (09:45 in this example).
Doing this terminates my program at 09:45 giving the error message as "ks_api_client.exceptions.Api exception: (999113) Order modification not allowed as current order status is FIL, "){screenshot attached} and thereafter terminating the program for future trades.
please help on why this error occur and how to rectify it
Thanks

FIL implies that the order was traded. thats why your modification request wasnt going through

How should I insert the condition (via if loop) for capturing this order
status in my algo as the same terminates when this modification is not
possible.
On Tue, 26 Apr 2022 at 13:56, dhwaj singhal @.***>
wrote:

FIL implies that the order was traded. thats why your modification request
wasnt going through
—
Reply to this email directly, view it on GitHub
https://github.com/paramatrixtech/ksapi/issues/23#issuecomment-1109503313,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AY4CRABYMUOD6L2U2K5KTHDVG6SDVANCNFSM5UHQWNFQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Split replicated policy reconciles

The first 2 commits are refactors that are not all necessary for the other part of the PR. I can slim down those commits if it helps this PR review. I would recommend looking through the commits individually.
The new replicated policy reconciler helps with two things:

When a replicated policy is updated, other related replicated policies are not recalculated.
The root policy reconciler no longer creates/updates/deletes replicated policies itself, via custom concurrency logic. Instead, it simply sends events to the new reconciler for those policies to recalculate their correct states.

It does not "intelligently" handle updates to replication decisions, eg to not recalculate the replicated policy on managed clusters that should not have changed.
This also adds configurable concurrency to the root policy reconciliation, and the new reconciler has configurable concurrency. The new default root policy concurrency is 2, and the replicated policy concurrency is 10 (up from the default of 5 that we had in our bespoke concurrency logic).

For reference, the resource version caching reduced the number of reconciles done in the e2e tests by about 35%, from 577 to 364

I am worried that the test failure is because the replicated policy reconcile is now happening too fast? From the failing test:
Expected
  ... "label-vendor-test":"auto-detect\n", "label-vendor-test-two":"Fake\n" ...
to equal
  ... "label-vendor-test":"Fake\n", "label-vendor-test-two":"Fake\n" ...
...

both of these are from the managed cluster label - the first uses `.ManagedClusterLabels`, the second uses a lookup. So I wonder if the update to the ManagedCluster triggered the reconcile, but when the ManagedCluster was grabbed from the cache wasn't updated yet... because the ManagedCluster that the template-utils grab for the lookup *is* correct.

So I'm not really sure what to do about that. It seems to be happening inconsistently. Previously, I think if this happened, it would get handled by the duplicate reconcile.

Added some changes after more discussion with @mprahl. In particular, a race condition where the predicates might not correctly filter out events caused by the reconciler itself (causing an unnecessary duplicate reconcile) has been addressed by adding a set of locks.
AFAIK, there are not any more currently requested changes to this PR; it should be ready for another look.


Wrong resolution upon 2nd pipenv install

Describe your environment

macOS 10.12.6
Python version: any python version, bug seen with 2.7 and 3.6
Pipenv version: 9.0.0

Expected result
Expecting pipenv install to yield same result consistently, in this particular case setuptools 33.1.1 should be resolved (due to pex constraint)
Actual result
pipenv resolves setuptools 33.1.1 on 1st installation, but not on 2nd
Steps to replicate
You can reproduce like so (in any folder, using a scratch one here).
First time around, everything looks good (pex wants setuptools<34), note the minor annoying question "Did you mean pexpect?":
pipenv --rm ; rm -f Pipfile* ; pipenv install pex ; pipenv install pytest --dev ; pipenv graph
...
Did you mean pexpect? [Y/n]: n
Installing pex…
...
pex==1.2.15
  - setuptools [required: >=20.3,<34.0, installed: 33.1.1]
...
pytest==3.3.1
  - setuptools [required: Any, installed: 33.1.1]
...

Now delete venv and install again -> wrong setuptools is resolved:
pipenv --rm ; pipenv install ; pipenv graph
...
pex==1.2.15
  - setuptools [required: >=20.3,<34.0, installed: 38.2.4]
...
pytest==3.3.1
  - setuptools [required: Any, installed: 38.2.4]
...

Also note that pipenv install acted here as-if I state --dev... wasn't expecting pytest to be installed here (was expecting it to be installed if I stated --dev).

Found a workaround for this, by running:
pipenv --rm ; pipenv install
pipenv run pip install -e .

pip install notices the disparity and fixes it.
Can't do pipenv install '-e .': that doesn't help (setuptools version is still broken).

@zsimic thanks for the report. Just want you to know you are not crazy, this is a real issue introduced in 9.0 which always installs dev packages by default. It is fixed in #1173 and will be released shortly. I’m going to close this for now, but sorry for the inconvenience!


Leaving unnecessary dust behind

Hi there! Thanks for the great bot! FYI I noticed it leaves unnecessary dust behind. Please see the three buy orders and then the sell order. As you can see it buys 3x 0.09, but only sells 0,26 (and not 3x 0.09 = 0.27). I am running the latest version of the bot.

Should you need anything else, please let me know!
By the way, I read the documentation in the issues list and I checked https://codepen.io/chrisleekr/full/mdMEEKj, but I disagree that it must leave the tiniest amount behind. The fees get deducted automatically from either QUOTE or BASE currency (depending on SELL or BUY), so you do not have to take this into account.
Maybe somebody can recommend how to adjust place-sell-order.js so it will sell everything?

Going to change this place-sell-order.js so it will sell everything. I think it needs sufficient BNB in the account to prevent it from running into fee issues. This is a human action. The BNB is then used to pay the fees.
const orderQuantityWithPercentage = parseFloat(
_.floor(
// Changed line below so it will sell everything, will keep sufficient BNB in account to use as fee
// freeBalance * sellPercentage - freeBalance * sellPercentage * (0.1 / 100),
freeBalance * sellPercentage,
lotPrecision
)
);

I have been trying out the bot now for a few weeks and I am really enthusiastic about it. It does indeed leave some dust behind, but that only happens once. WIth once, I mean the first time a buy and sell occurs, after that the dust remains the same, it does not leave more dust. I still think leaving this dust behind is not necessary when you use BNB for fees, but since it does it only once for me the issue is not big enough to file it as a bug. Maybe another suggestion is to give explanation why it leaves this dust once behind. I did read the articles, but I could not understand.


WIP: [CI:DOCS] Cirrus: Implement new retrospective task

NOTE: Sadly there's no way to actually test this before merging it.  However, there is extensive testing enabled at the source, including self-testing changes to the cirrus-ci_retrospective container image itself.
For reasons such as releasing software or notifying people
about various conditions, a method is required to detect
and execute commands/scripts when all Cirrus-CI operations
have definitively concluded.  However, due to the nature of
automation, there are occasional and unpredictable needs
to manually perform these activities.
Given existing familiarity and dependence on Cirrus-CI, add a
'retrospective' task with a 'manual' trigger and no dependencies.
Remove other notification/monitoring operations in anticipation
this task will take them over in a future commit.  Add a
github action that will atomicly and trigger the new task
on branch and tag push, under one of two conditions:

The new github Action detects the Cirrus-CI 'success' task
finished running and was not manually canceled.
A person pressed the manual 'trigger' button on the new
'retrospective' task, from the status page, on any build.

Also: E-mail me immediatly if there is ever a problem with the
magic post-cirrus triggering automation.
Signed-off-by: Chris Evich<EMAIL_ADDRESS>

@edsantiago @rhatdan @mheon PTAL.  Once working, my intention is to use the new retrospective Cirrus-CI task for our branch failure notifications and for building/uploading releases.
Unfortunately, due to security limitations (that we actually want) the github action-magic cannot be tested in a PR, it only runs directly from the master branch.  If it helps, I can clone this PR over into automation_sandbox and merge it there first.  Let me know if that would be helpful.

(added comment reference to JSON format)

@edsantiago @rhatdan @mheon PTAL at a clone of this repo. where both Cirrus-CI and the new github action are operating as intended on the master branch (after push):

Cirrus-CI run: https://github.com/containers/automation_sandbox/runs/577159168
Github Action: https://github.com/containers/automation_sandbox/actions/runs/75323865

The action detected conclusion of the Cirrus-CI success task, and subsequently triggered the manual retrospective task (where releases, notifications, or other follow-up activities will happen)

Here's an example pull-request https://github.com/containers/automation_sandbox/pull/81/checks and the corresponding github action: https://github.com/containers/automation_sandbox/runs/577261181?check_suite_focus=true

Finally, an example from pushing a new tag: https://github.com/containers/automation_sandbox/runs/577296274 and corresponding github action: https://github.com/containers/automation_sandbox/runs/577297684?check_suite_focus=true
Oh! Well now, will you look at that, there's a non-conclusive task status from Cirrus-CI.  That should never happen, as the Github Action is set to trigger (literally) upon completion.
@edsantiago @rhatdan @mheon un-PTAL. This is a bug in Cirrus-CI or in Github that I need to follow up on.  We can't have our "all done" thing happening before "all done" is for real.

/approve

Update: This PR is blocked on debugging/investigating a status race or sync. problem regarding tag-push.  Since this code can only run from the master branch (by design) I need to do that work in a sandbox repository.

@cevich Anything goin on with this PR?

@cevich Anything goin on with this PR?

Yes.  I might close it :cry: or re-implement it to deal with the new reality: We cannot rely on tag-push triggering of Cirrus-CI workflow that is accountable on the Github side (like for PR's and Branches).
We've got an internal e-mail thread going with @mheon and @baude (and you) about needs relating to release automation.  Choices made there will impact the re-implementation or closure of this PR.
This topic is a top-priority of mine at the moment.

@cevich Still need this?

I'm going to close this for now in favor of a new PR (coming soon) that will do something less fragile for tags.


Ubuntu 22.04 with self-hosted

Hello! Please indicate why when using ubuntu 22.04 with self-hosted it is necessary to run it with root privileges? we have the case that we use it with ECS on AWS and with version 20.04 you don't need to run it with root privileges.

Hi @lesmorlopez2412 - Thank you for your question.  This board is for documentation issues. This looks like a product question. Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow


WIP: Replace easymde with textarea

WIP on the easymde replacement. Basically it rips out EasyMDE without much second thought. Issue commenting works but other things may be broken.
TODOs:

[ ] Fix up loose ends, quote reply, image paste, etc
[ ] Replace Tribute with something better, remove RequireTribute
[ ] Finish style removal, remove _editor.less
[ ] Add toolbar when editing markdown files in the repo
[ ] Fix review textarea
[ ] Re-Implement fullscreen side-by-side (maybe for another PR)

Fixes https://github.com/go-gitea/gitea/issues/10729 and a probably 30 other issues.

Our users really really like the WYSIWYG editor.
I think this has to be an option to switch between editors. In fact we need to have the option to use easymde as the main editor too.

WYSIWYG

You can't really call it that. It's more like a strange mixup between HTML and Markdown inside a buggy editor that is not accessible. Markdown was not designed for this and neiter GitHub or GitLab offer such a mode. If anything, it should be opt-in.

Rebased and kept the CodeMirror files in the repo for now make the diff more manageable.
Any more feedback on whether we can go ahead with this?
To me it's still a clear case. I curse at EasyMDE almost daily when it decides to delete entered text because of its atrocious handling of backtick characters.

I still think a majority of users would prefer a plaintext editor, it's only natural for developers.

While I understand what is going on here, as I implemented the original File editing feature for Gogs back in 2016 with CodeMirror and have made a few fixes once we all switched over to Gitea, just want to say we have thousands of users and not a single one is a developer. We use this for content, in various forms (Markdown, TSV, USFM, RST). We would have to figure out how we can properly have some sort of view and WYSIWYG.

how many of those thousands of users have to, occasionally, use gitea from a mobile device?

@igalic at least a hundred use our site directly, https://git.door43.org, but good point, as mobile apps have nothing to do with this feature. :)

unless the mobile app is Firefox, in which case, good luck, https://github.com/mozilla-mobile/fenix/issues/4208

We would have to figure out how we can properly have some sort of preview and WYSIWYG.

There's a "Preview" Tab right above the editor, is that really so hard to discover? I think I would implement some form of side-by-side editor, but the rendering has to be done by the server to match the actual output so there will be a slight delay. Ccurrently what EasyMDE shows may totally not match what the final result will be.

Just adding my thoughts:

I only write markdown and don't want to rely on any special markup in the editor
preview: works fine!
side-by-side should be optional IMHO and sounds more like a plan for the future, unless you're really fond of working on it

Oh and thank you very much for working on it! :100:
I'm really looking forward no longer to see weird editor bugs or hear users complaining about Gitea being inaccessible to mobile devices

I plan to pick this up again given the positive feedback. I think it outweights the negative ones :)

@silverwind I vote for this, I think we can move on, the usage of SimpleMDE in our JS is less messy now after some refactorings, we can use EasyMDE to replace it (and continue to refactor)

Oh sorry, maybe I misunderstand some comments. Maybe we are discussing about two options: EasyMDE vs textarea?
Well, I like the WYSIWYG-like editor (although it is not really WYSIWYG). But I will vote for a quick and easier switch.

If we can switch to EasyMDE as soon as possible, I prefer this way. And some features of EasyMDE are really good (eg: draft)

If we can switch to textarea more easily, that's also fine. Maybe we should also implement a draft mechanism by ourselves.

Which way seems easier?

This is now too far out of date to be any use, but we should still pursue it.


Allow for customization in Helm based operator

Feature Request
Is your feature request related to a problem? Please describe.
I have a set of charts, for which some resources need to be customized prior to installation (e.g. specifying a URL value in a ConfigMap that points to a Route installed ).
I have a set of charts that need to be applied in a certain order (e.g. as above, I need the chart providing the Route to be installed before its dependent).
I need to modify resources as part of the installation (e.g. applying labels to namespaces).
Describe the solution you'd like
It would be nice if the charts were rendered in a more controlled fashion, in a system where listeners could be registered to alter or enhance the behavior (as in the examples above).
We've started doing something similar to this in Maistra and I've recently refactored the generic rendering bits into a simple framework for the istio/operator.  Would something like this framework be beneficial for the operator-sdk?  If so, I'm happy to extract the generic portions I contributed to Istio for the SDK.

You can see how the framework is used here.  The basic framework utilizes json merge patch and k8s strategic merge patch to process updates to resources and uses labels to prune created resources.  The extension simply tells the framework:

what set of charts to use
what order to apply the charts
what labels to apply to the created resources
what resource types are being managed (so the labelled resources can be pruned, as necessary)

@rcernich This sounds interesting. A few questions:

I have a set of charts, for which some resources need to be customized prior to installation (e.g. specifying a URL value in a ConfigMap that points to a Route installed ).

Do you envision the collection of charts having their lifecycle managed under one CR or multiple? Is there a parent chart that has one or more dependent subcharts?

I have a set of charts that need to be applied in a certain order (e.g. as above, I need the chart providing the Route to be installed before its dependent).

If I understand this correctly, this part would be a big change from the existing design, which is that a single CR reconciliation results in a single Helm lifecycle action (e.g. install, update, uninstall), and a single Helm release object. Does this imply multiple helm release objects per CR?

I need to modify resources as part of the installation (e.g. applying labels to namespaces).

Basically a way to patch or inject things into the release manifest that's not in the chart template and controlled by values? This is definitely something we've thought about. The helm operator itself currently does this to inject owner references into the release manifest.

It would be nice if the charts were rendered in a more controlled fashion, in a system where listeners could be registered to alter or enhance the behavior (as in the examples above).

Our idea for this is to refactor the helm-operator into a library that has extension points for adding custom logic. For example, interfaces that can be wrapped or overridden for choosing a release name, rendering the manifest, adding custom finalizers, executing the actual release action (maybe similar to HTTP middlewares where you get a next handler and have the chance to execute code before and/or after calling next.ExecuteRelease or something).
In your scenario, are you envisioning that operator developers would write Go code to augment and change how the Helm release is managed, or are you thinking that this functionality could be added to the native helm-operator and configured entirely via watches.yaml and maybe other templates, patch files, etc.?

Answers:

The reconcile would be driven off a single CR.  Currently, the framework would only support a single chart, which could have multiple subcharts.  That said, invocations could be chained together to support multiple charts.
No helm release objects are used.
Correct.  One thing we do on Maistra is to modify a ConfigMap, which contains fields for various URLs, by looking them up from Route objects that were created by other subcharts.  This allows things to be wired correctly across the install.
Users would write custom go code to handle callbacks.

Here's basically how the operator works:

Render charts (effectively helm template)
Group manifests by chart name (map chart-name => manifest-list)
Create/update objects in manifestList for each chart (order of chart processing can be specified)
For each object, allow user to modify the object before it is applied.
If the object already exists, create a patch and apply it (this protects against things like overwriting immutable fields that have been set by k8s, e.g. Service.Spec.ClusterIP)
After manifest processing is complete, objects not aligned with the current CR are pruned.

To facilitate patching, objects are annotated in the same way they would if a user created them using kubectl apply.  This annotation is used as the source for the patch in a three way merge strategy (along with the installed structure and the new structure).
To facilitate pruning, all objects are labelled with an owner and a generation annotation is added.  For update processing, any labelled resources whose generation doesn't match are deleted.  For deletion all owned resources with a generation label are deleted.  (Deleting derived resources is avoided by asserting the existences of the generation annotation on the object.)
Theoretically, you could use this as a generic helm rendering engine, as all it does by default is add the labels and annotations to the objects (as opposed to using the helm release mechanism).
Currently, this mechanism has not been used in an upgrade scenario, so I don't know if there are appropriate hooks that would support migration.  I can say that it does correctly overlay rendered objects, so the resources created reflect the latest state of the CR.
HTH

I like the idea of removing the requirement for storing release state in a separate object. It sounds like a combination of labels and annotations on the release resources are sufficient for recreating the install, upgrade, uninstall lifecycle. And I really like the idea of the apply-like annotation for the 3-way merge. This is something I'd be interested in investigating more as a standalone design change.

Create/update objects in manifestList for each chart (order of chart processing can be specified)

Helm (and tiller) currently render the manifest for the entire release and then sort it based on kind before installing (and the order is reversed for uninstall). It intermingles objects from various charts. Would it be sufficient to use Helm's sorting algorithm by default, but allow it to be overridden? Or is it important to handle each chart individually? More generally, could the helm operator allow the entire rendering process to be overridden with a custom Render function provided by the operator developer?

Theoretically, you could use this as a generic helm rendering engine, as all it does by default is add the labels and annotations to the objects (as opposed to using the helm release mechanism).

Would there be a mechanism to rollback a failed install or upgrade?  An example of this is when a release attempts to change a service type from NodePort to ClusterIP. Helm fails the upgrade, leaves the previous release state in place (unless the user passes --force, in which case the service is deleted and recreated), and creates a new failed release object. The helm operator automatically rolls back in this scenario.
Also, one of the features of the helm release mechanism is chart hooks, which are currently supported in the helm operator. Would this alternate mechanism be able to handle chart hooks?
Along those same lines, I think it would be nice to allow arbitrary go code execution before and after the release. This would give operator developers the chance to change the manifest, run custom migration code, handle errors, etc. to handle complex logic that is too complex or not possible to handle with native Helm and chart hooks.
Lastly, have you seen the set of changes coming in Helm 3? Is there anything there that would need consideration?

Hey, sorry I've been on PTO...

Helm (and tiller) currently render the manifest for the entire release and then sort it based on kind before installing (and the order is reversed for uninstall). It intermingles objects from various charts. Would it be sufficient to use Helm's sorting algorithm by default, but allow it to be overridden? Or is it important to handle each chart individually? More generally, could the helm operator allow the entire rendering process to be overridden with a custom Render function provided by the operator developer?

We rely on the helm ordering of resources, but also insert OpenShift specific resources in the appropriate locations (e.g. routes after services).
The ordering I was speaking of was specific to the chart ordering.  We organize the manifest list by charts, as opposed to having a single list of all resources from all charts, then apply resources for each chart, based on some custom ordering (i.e. apply chart a, chart b, chart d, chart c, etc.).  The use case for this is for when there is a dependency from one chart to another chart, or when a resources needs to be updated dynamically using information from a resource in another chart (e.g. the host name from a route that's referenced in a config map).

Would there be a mechanism to rollback a failed install or upgrade? An example of this is when a release attempts to change a service type from NodePort to ClusterIP. Helm fails the upgrade, leaves the previous release state in place (unless the user passes --force, in which case the service is deleted and recreated), and creates a new failed release object. The helm operator automatically rolls back in this scenario.

I haven't looked into anything like this.  Theoretically, this would involve a rollback of the custom resource and/or adding something like --force.  To be honest, there's work to do with respect to updates.  Using the patching mechanism was a simple way of avoiding having to write custom merge logic for every resource.

Also, one of the features of the helm release mechanism is chart hooks, which are currently supported in the helm operator. Would this alternate mechanism be able to handle chart hooks?

I think there's some overlap with this and hooks.  Part of the reason for developing this was to avoid using jobs to patch the install (i.e. install it correctly from the start).  That said, this mechanism doesn't currently look for or run hooks (it basically just calls helm template and manually creates/updates the rendered resources).

Lastly, have you seen the set of changes coming in Helm 3? Is there anything there that would need consideration?

I have not and don't know.


chore(POINT-941): Add bancontact to JS

Description
We want to add bancontact to our JS so it's shown in the front-end. The issue we’re trying to solve is that currently, the merchant is unaware a card is Bancontact - therefore if a customer wrongly selects this, it could lead to failures.
Solution

Add these values for bancontact:
6703
487104
670305
670330
670319
670300
670342
487109
670397
670375
670310

Jira Issue
https://checkout.atlassian.net/browse/POINT-941

@processout/frontend-developers Can I have a review here, please?

I believe you need to run make build as well

Done: 2b05132a642b17ca05718c710be673c1dd864e6a, Thanks!

LGTM!
The attached ticket talked about updating this switch statement for card number lengths based on scheme. There is no carte bancaire case, nor any new one for the bancontact case.
Do we know if that's what we want? Are the defaults for min/max length of the card number good enough here (min 12, max 19)?

We will raise a separate PR for that.

I guess the pin is same as normal card, 3 digits, right?

I have attached a test


Update README page with link to viewer and update validator links

Add link to viewer on README page. Update validator links to use released version.

https://github.com/will-moore/ome2024-ngff-challenge/tree/readme_udpate looks good - merging so the page is up to date for tomorrow...


Error while retreiving restaurant infrormation

There is an error while getting restaurant information
Here is the request information:

The error in the API-gateway server says that kotlinx.serialization.MissingFieldException: Field 'ownerId' is required, but there is no request parameter required for that:

@mosayed01 @NadaFeteiha @HaidyAbuGom3a @mustafa-sab3alarab

I updated the branch, but then I encountered another error:

After investigating the issue, I discovered that the problem was caused by a hardcoded local URL in the ktor client.


SwiftVirtualNetwork missing Location property

Ported from https://github.com/Azure/azure-sdk-for-go/issues/3874
The SwiftVirtualNetwork struct is missing a Location property which is expected by the API
https://github.com/Azure/azure-rest-api-specs/blob/master/specification/web/resource-manager/Microsoft.Web/stable/2018-02-01/WebApps.json#L21274
PUT https://management.azure.com/subscriptions/xxx/resourceGroups/hbtest_webapp_vnet/providers/Microsoft.Web/sites/hbtestwebappvnet/config/virtualNetwork?api-version=2018-02-01 HTTP/1.1
Host: management.azure.com
User-Agent: Go/go1.11.4 (amd64-windows) go-autorest/v11.2.8 Azure-SDK-For-Go/v24.0.0 web/2018-02-01 Terraform/0.11.9 terraform-provider-azurerm/dev
Content-Length: 201
Authorization: Bearer xxx
Content-Type: application/json; charset=utf-8
Accept-Encoding: gzip

{"properties":{"subnetResourceId":"/subscriptions/xxx/resourceGroups/hbtest_webapp_vnet/providers/Microsoft.Network/virtualNetworks/hbtest_webapp_vnet/subnets/subnet"}}

HTTP/1.1 400 Bad Request
Cache-Control: no-cache
Pragma: no-cache
Content-Length: 407
Content-Type: application/json; charset=utf-8
Date: Tue, 15 Jan 2019 12:09:43 GMT

{"Code":"BadRequest","Message":"The parameter location has an invalid value.","Target":null,"Details":[{"Message":"The parameter location has an invalid value."},{"Code":"BadRequest"},{"ErrorEntity":{"ExtendedCode":"51008","MessageTemplate":"The parameter {0} has an invalid value.","Parameters":["location"],"Code":"BadRequest","Message":"The parameter location has an invalid value."}}],"Innererror":null}

After replacing Kind (which doesn't appear to be needed) with Location the request succeeds
PUT https://management.azure.com/subscriptions/xxx/resourceGroups/hbtest_webapp_vnet/providers/Microsoft.Web/sites/hbtestwebappvnet/config/virtualNetwork?api-version=2018-02-01 HTTP/1.1
Host: management.azure.com
User-Agent: Go/go1.11.4 (amd64-windows) go-autorest/v11.2.8 Azure-SDK-For-Go/v24.0.0 web/2018-02-01 Terraform/0.11.9 terraform-provider-azurerm/dev
Content-Length: 222
Authorization: Bearer xxx
Content-Type: application/json; charset=utf-8
Accept-Encoding: gzip

{"location":"uksouth","properties":{"subnetResourceId":"/subscriptions/xxx/resourceGroups/hbtest_webapp_vnet/providers/Microsoft.Network/virtualNetworks/hbtest_webapp_vnet/subnets/subnet"}}

HTTP/1.1 200 OK
Cache-Control: no-cache
Pragma: no-cache
Content-Type: application/json
Date: Tue, 15 Jan 2019 12:42:27 GMT
Content-Length: 467

{"id":"/subscriptions/020ef497-e33d-4705-859b-08663256a4bc/resourceGroups/hbtest_webapp_vnet/providers/Microsoft.Web/sites/hbtestwebappvnet/config/virtualNetwork","name":"virtualNetwork","type":"Microsoft.Web/sites/config","location":"UK South","properties":{"subnetResourceId":"/subscriptions/020ef497-e33d-4705-859b-08663256a4bc/resourceGroups/hbtest_webapp_vnet/providers/Microsoft.Network/virtualNetworks/hbtest_webapp_vnet/subnets/subnet","swiftSupported":true}}

@lkems - any update on when location could make it into the specs? I'd like to get support for this into Terraform

@jhendrixMSFT - do you know anyone else that could pick this up please?

@petrpod are you the correct person to route this?

Per an internal discussion the change has been rolled out to the service, working on getting the swagger updated.

Is this still an issue? I cant find any documentation regarding "SwiftVirtualNetworkProperties" I got this while configuring the "Microsoft.Web/sites/networkConfig@2020-06-01" which is also undocumented.
"The property "vnetRouteAllEnabled" is not allowed on objects of type "SwiftVirtualNetworkProperties". No other properties are allowed. If this is an inaccuracy in the documentation, please report it to the Bicep Team.bicephttps://aka.ms/bicep-type-issues"


Characters assigned to wrong RIL_WORD block, 0 % confidence.

Current Behavior
Upon recognition with language model tessdata 4.1.0 eng.traineddata, characters that are a bit too wide apart sometimes get sorted into two words (false whitespace), depending on the ROI that was set with TessBaseAPI::SetRechtangle.
This is an expected edge case due to different segmentation, depending on the placement of the ROI. Thresholding method is Tiled Sauvola.
The recognized characters, however, seem not to be assigned to the correct word block (RIL_WORD):

Note how the recognized words are "20M" and "110210A" instead of the clearly boxed "29M1" and "10210A" substrings in the image (neglect the "9" as "0" misrecognition).
Note that the confidences are 0.0% on each of the words.

Below you can see the total image and the ROI (orange rectangle). The black area in the center is a customer logo that had to be removed for uploading. During OCR, this area was not black, but printed like the other text.

Changing the ROI slightly (moving the bottom right corner a bit more outward) removes the split of the "29M110210A" line, though the confidence is low (22.3 %).

Side note:
And suddenly an additional "0" has sneaked in, making it "290M110210A"!
Layout analysis (TessBaseAPI::AnalyseLayout) shows a tiny fragment inside the "M". Is that the "0"?

Changing the ROI yet a bit more (moving the bottom right corner yet further outward) then gives a good confidence (80.4 %).

I thought that maybe I am observing an edge case where the segmentation goes from OK to really bad, but this seems not to be the case. TassBaseAPI::GetThresholdedImage gives these nearly identical images for the bad and the good case:

Expected Behavior
This should either read "20M1" and "10210A" or "20M110210A" (neglect the "9" as "0" misrecognition) and not have zero confidence.

Suggested Fix
No response
tesseract -v
tesseract 5.3.2
leptonica-1.83.1 (Oct 27 2023, 14:15:52) [MSC v.1929 LIB Release x64]
(null)
Found AVX2
Found AVX
Found FMA
Found SSE4.1
Operating System
Windows 10
Other Operating System
No response
uname -a
No response
Compiler
MSVC 16.11.26
CPU
No response
Virtualization / Containers
No response
Other Information
C++ Application, TessBaseAPI. Custom GDI visualization, hence only cropped screenshots with slightly differing alignment.

Please provide input images and example C++ code that demonstrate your problem.

Input image

Code
#include "leptonica/allheaders.h"
#include "leptonica/pix_internal.h"
#include "tesseract/baseapi.h"
#include "opencv2/imgcodecs.hpp"
#include "opencv2/imgproc.hpp"
#include <memory>

int main() {
    cv::Mat in_img = cv::imread("./input.png", cv::ImreadModes::IMREAD_GRAYSCALE);
    tesseract::TessBaseAPI tess;

    // Set tesseract parameters.
    tess.Init(".", "eng");
    tess.SetVariable("thresholding_method", "2"); // Tiled Sauvola
    tess.SetPageSegMode(tesseract::PageSegMode::PSM_SINGLE_BLOCK);
    tess.SetImage(in_img.data, in_img.cols, in_img.rows, in_img.channels(), static_cast<int>(in_img.step1()));

    // Output thresholded image.
    std::unique_ptr<Pix, void(*)(Pix*)> thrs_pix(tess.GetThresholdedImage(), [](Pix* val) { pixDestroy(&val); });
    cv::Mat out_img(cv::Size(thrs_pix->w, thrs_pix->h), CV_8UC1);
    for (uint32_t y = 0; y < thrs_pix->h; ++y) {
        for (uint32_t x = 0; x < thrs_pix->w; ++x) {
            l_uint32 val;
            if (0 == pixGetPixel(thrs_pix.get(), x, y, &val)) {
                out_img.at<unsigned char>(y, x) = val ? 255 : 0;
            }
        }
    }
    cv::cvtColor(out_img, out_img, cv::COLOR_GRAY2BGR); // prepare colored output image

    // Perform recognition and extract image information.
    if (0 == tess.Recognize(nullptr)) {
        std::unique_ptr<tesseract::ResultIterator> res_iter(tess.GetIterator());

        if (res_iter != nullptr) {
            cv::Mat curr_img;

            // Generate output image for symbols and words.
            for (auto block_level : { tesseract::PageIteratorLevel::RIL_SYMBOL , tesseract::PageIteratorLevel::RIL_WORD }) {
                cv::cvtColor(in_img, curr_img, cv::COLOR_GRAY2BGR); // prepare colored current image
                res_iter->Begin();

                do {
                    // Only text blocks.
                    if (PTIsTextType(res_iter->BlockType())) {
                        cv::Point2i p1, p2;

                        if (res_iter->BoundingBox(block_level, &p1.x, &p1.y, &p2.x, &p2.y)) {
                            // Draw bounding box.
                            cv::rectangle(curr_img, cv::Rect(p1, p2), cv::Scalar(0, 255, 0));

                            // Prapare text output.
                            const int font = cv::HersheyFonts::FONT_HERSHEY_PLAIN;
                            cv::Size text_size;

                            // Write Confidence.
                            std::stringstream conf;
                            conf.precision(0);
                            conf << std::fixed << res_iter->Confidence(block_level) << '%';
                            text_size = cv::getTextSize(conf.str(), font, 1.0, 1, nullptr);
                            cv::putText(curr_img, conf.str(), cv::Point2i(p2.x - text_size.width - 2, p2.y - 2), font, 1.0, cv::Scalar(255, 100, 0));

                            // Write detected text (OpenCV does only have ASCII, but close enough).
                            std::unique_ptr<const char[]> raw_text(res_iter->GetUTF8Text(block_level));
                            if (raw_text != nullptr) {
                                text_size = cv::getTextSize(raw_text.get(), font, 1.0, 1, nullptr);
                                cv::putText(curr_img, raw_text.get(), cv::Point2i(p1.x + 2, p1.y + text_size.height + 2), font, 1, cv::Scalar(0, 0, 255));
                            }
                        }
                    }
                } while (res_iter->Next(block_level));

                // Stack current image on top of output image.
                cv::vconcat(curr_img, out_img, out_img);
            }
        }
    }

    cv::imwrite("./output.png", out_img);
    return 0;
}

Output

Remarks
The program above reproduces the error shown in the original issue post, but in a self-contained program. Hence coloring, fonts etc are deviating.
The output consists of three stacked augmented verisons of the input image:

Recognized words
Recognized symbols
Threshold image (for visual proof of Tesseract's working space)
Each word or symbol comes with it's bounding box (green), the recognized text (red) and the confidence (blue).

Dependencies

Tesseract
Leptonica
OpenCV

Setup
To execute the program, you need to put the input image into the executable's current directory as "input.png".
Also, you need the english language model from here in the same folder.
The output will be saved as "output.png" in the same folder.
Discussion
As you can see in the output image provided, the word "29M1" is recognized as "29M" with 0% confidence, albeit consisting of three characters '2', '9' and 'M' with above 90% confidence each. The 'M' is a misdetection of the actual printed "M1".
Noticeably, the next character might screw things up: the first '1' of "10210A" gets detected as 3 different Symbols, '1', '1' and 'T', where the glitched '1' and 'T' seem to share the exact same location. They got a higher bounding box than the neighboring characters but are only 1 px wide. It seems, those glitched symbols screw up the word "29M110210A", divide it in two parts and subsequently set their confidences to zero.
Detail shot from our customer application (I can zoom in there, but the boxes are drawn 0.5 pixels off - it is just a quick debug view):

And just for funsies, on the left side the word "paper" is recognized from random cracks. With 16% confidence, which is infinitely more than the 0% for second line of the actual printed text.

I just manually preprocess image based on documentation:

and the result is:
tesseract input4175p.png -
9200795018 -
20M110210A

=>

tesseract is not suitable for text detection (usually)
tesseract is OCR engine for good output there is a need to give a good input image.

Thank you. As you guessed, text detection is what we aimed for.
Just to reemphasize, I was neither being thrown off by the random junk being detected outside the obvious text label or by the inserted blank between '1' and '1'.
What caught my attention was that

"M1" became "M"
"1" became "11" (and this was not a '1' being carried over the blank, it was a coincidentally occuring actual '1' that was detected with a very deformed bounding box)
The confidence dropped to 0 %
and the broken overlapping bounding boxes left of the second '1' glyph in the second line.
And all that while the same image rotated 1 ° or 2 ° to the left or right yielded OK results.

I think this is dangerous: there is a continuous sweep of angles the image can be rotated for good results. and then, amodst those, there is a discontinuity in the results where obvious recognition artifacts screw up the result.
Even for non-optimal inputs the reults should not glitch out like that.
But I understand, there is machine learning behind the scenes and those models tend to have that kind of discontinuity issues.


Shortcuts and room turn up down

added: keywords “party”, “lounge”, “chill”
added: turn (room name) up|down

Oops. Wasn't really meaning this to merge to yours…but there are definitely elements you may want.

Would love to merge Next/Prev & State into master, as they were on my list of future additions.  If you're interested in contributing, please drop the custom intents and the app id, and we should be mostly good to go.
Volume also looks good, and a future enhancement could be to change relative volume by a value (eg "increase volume by 10" or "add 10 to volume") rather than setting to an absolute.

I was also going to do the current state enhancement this weekends, thanks for saving me some time and agreed it would be great to see in the main branch,

@kevnull, LMK if you plan to do this, or if one of us should add it to our todo list.  Thanks!

@rgraciano I somehow never saw this and then was traveling for a few months. Did you end up getting these features in?

A bunch of similar changes have been added since then. Take a look at the
newest version and we can circle back if there are gaps
On Mon, Feb 29, 2016 at 4:45 PM Kevin Cheng<EMAIL_ADDRESS>wrote:

@rgraciano https://github.com/rgraciano I somehow never saw this and
then was traveling for a few months. Did you end up getting these features
in?
—
Reply to this email directly or view it on GitHub
https://github.com/rgraciano/echo-sonos/pull/5#issuecomment-190472015.


[Dockerfile] Update Rust, Zig version & add Windows ARM64 target

@messense

Please limit the change to Dockerfile, FYI we are using zig 0.10.1 because the compatibility better than newer versions.


libuvc: 0.0.6-0 in 'lunar/distribution.yaml' [bloom]

Increasing version of package(s) in repository libuvc to 0.0.6-0:

upstream repository: https://github.com/ktossell/libuvc.git
release repository: https://github.com/ktossell/libuvc-release.git
distro file: lunar/distribution.yaml
bloom version: 0.5.26
previous version for package: 0.0.5-3

@7675t This fails to build on all platforms.
Please comment here if you are planning on releasing a fixed version soon, otherwise we will roll back the previous version for the sync tomorrow. Thanks!

@mikaelarguedas Sorry, it lacks a dependency, which I cannot know with local environment. I fixed it now. If it is still failing, please roll back to previous version. Thank you so much.

Thanks @7675t for the quick fix!
If you want to be able to validate your fixes locally you can run the same jobs as the buildfarm using https://github.com/ros-infrastructure/ros_buildfarm/blob/master/doc/jobs/release_jobs.rst#run-the-release-job-locally

Thanks, it is valuable tips for me. I will try it.


Webpack could not deduce aliases from TS / JS config file.

Describe the bug
After setting up, and using path mappings in tsconfig.json file, webpack gives module resolution error.
Module not found: Error: Can't resolve '@module' in '/path/frontend/src/file'
 @ ./src/App.tsx 7:0-46 22:30-42
 @ ./src/index.tsx 7:0-24 11:33-36

Did you try recovering your dependencies?
Yes
Which terms did you search for in User Guide?
Webpack resolve, alias
Typescript path mappings
Module not found: Error: Can't resolve '@module' in 'path/to/file'
Environment
Using yarn v1.22.17
Node: v14.17.6
OS: MacOS 12.2.1
CPU: Apple M1
Steps to reproduce

yarn create react-app frontend --template typescript
Add baseUrl and paths to compilerOptions in tsconfig.json file
yarn start

Expected behavior
Webpack should get module aliases from tsconfig.json file.
Actual behavior
Webpack gives Module not found error.

+1
Same here

+1
the same

...and my axe!


OSError: /home/work/Downloads/BEVFormer_tensorrt/TensorRT/lib/libtensorrt_ops.so: cannot open shared object file: No such file or directory

I have the following problems when I run"sh samples/test_trt_ops.sh", how can I solve them?
OSError: /home/work/Downloads/BEVFormer_tensorrt/TensorRT/lib/libtensorrt_ops.so: cannot open shared object file: No such file or directory

I encountered the same problem
How did you solve it?

I encountered the same problem How did you solve it?

Make sure your GPU's SM version is greater than or equal to 7.5, indicating that you need to use an NVIDIA GPU of the 20 series or higher. Also, ensure that the path in DCMAKE_TENSORRT_PATH=/path/to/TensorRT corresponds to your installed TensorRT path, not the TensorRT path within the project.

I solved it after building and installing TensorRT correctly


HTTPS support

Likely using cloudflare, there's a good little tutorial by @winston here: https://github.com/jollygoodcode/jollygoodcode.github.io/issues/9

Could you use Let's Encrypt?
References

https://letsencrypt.org/
https://twitter.com/letsencrypt
https://fnord.no/posts/2015-11/letsencrypt/
https://en.wikipedia.org/wiki/Let's_Encrypt

@jasnow I suspect that would mean we'd have to pay $20 for the heroku ssl addon every month

Nevermind (smile)

Yup. It's free if you use Cloudflare (Universal SSL) + Heroku. Don't even have to turn on the Heroku SSL addon. :)
One more thing that I would suggest adding too - https://www.deppbot.com. It's a little service I am running that issues bundler updates as PRs. Free for open source! Cheers!

@winston we're using https://libraries.io to open issues about upgrades and then letting people open pull requests themselves but thanks for the offer.

:+1:


Added/dictionary filter using multiple keys

Checklist

[x] I checked the Contributing Guidelines before creating this request.
[x] New extensions are written in Swift 5.0.
[x] New extensions support iOS 10.0+ / tvOS 9.0+ / macOS 10.10+ / watchOS 2.0+, or use @available if not.
[x] I have added tests for new extensions, and they passed.
[x] All extensions have a clear comments explaining their functionality, all parameters and return type in English.
[x] All extensions are declared as public.
[ ] I have added a changelog entry describing my changes.

          1 Warning

:warning:
The source files have been modified. Please consider adding a CHANGELOG entry if necessary.

          1 Message

:book:
Thank you for submitting a pull request to SwifterSwift. The team will review your submission as soon as possible.

SwiftLint found issues
Warnings

File
Line
Reason

DictionaryExtensions.swift
206
Return arrow and return type should be separated by a single space or on a separate line. (return_arrow_whitespace)

  Generated by :no_entry_sign: Danger

Don't forget the CHANGELOG

@guykogus  can you please review it, I pushed the latest changes with swiftformatter


Update kube-router.rst

typo fix

This change is

test-me-please

Thanks for the fix @carsonoid ! Can you please sign-off your commit per the Developer's Certificate of Origin ?

Sign off added

test-docs-please

@cilium/janitors this a PR being merged to v1.2. Should we merge it to v1.2 or merge it to master and backport it to v1.3 and v1.2

this a PR being merged to v1.2. Should we merge it to v1.2 or merge it to master and backport it to v1.3 and v1.2?

It's fine to merge it into 1.2 and cherry-pick it into master and 1.3


Activity bar gear badge is not a perfect circle

👀

I believe this got broken very recently, maybe with trusted work:

Actually @sbatten there's a much bigger issue here: the 1 on that last screenshot is about having an update available. It should not show with a shield.

This was broken via https://github.com/microsoft/vscode/commit/7c3f60f86e6776c4a339c23f85fd9a11c3424bbb due to a conflict with padding in the activity bar badge and should be resolved in today's insiders, has a separate issue for tracking.
The (1) issue is also filed separately


feat: encrypt send data and save it locally to (1) show a history of sends, and (2) help debugging

Once a user sends funds to someone, there is currently no way for the sender to see who the funds were sent to. The only available information is the stealth address it what sent to. There are two main benefits to letting users see a history of where they sent funds:

It's very helpful for users to have a history of their sends so they can remember who they've sent funds to
When users are unable to receive funds, this helps debug as we can verify what the sender did

To do this, we'd need to encrypt the following data and save it in locally in the browser:

The recipient ID entered in the form
Whether advanced mode was used (i.e. recipient's pubkey or stealth key registry)
The random number?

Options to do the encryption:

Require users to sign the message before sending, so we can use their normal Umbra-specific private key for the encryption
Only support this feature for wallets that support eth_getEncryptionPublicKey and eth_decrypt RPC methods. Then we'd get their encryption pub key, generate a random private key, and save the corresponding random pubkey along with the encrypted data

For completion, I'll mention the option of storing the data unencrypted in local storage but this seems like a very bad idea. A browser extension could probably just read from local storage and de-anonymize all your transactions

I can pick this up. Would we potentially want to store these messages outside of the browser in order to provide a consistent experience across browsers?

@garyghayrat looks like you assigned yourself to this, do you still want to work on it or should we reassign to @alexkeating?
Re implementation, some thoughts:

Having a database store the encrypted sends would definitely be better UX across browsers, and more robust since it persists when browser storage is cleared. If we go this route, let's discuss with @apbendi  to see if there's any concerns around us having a DB with (encrypted) user info + any tooling preferences
Until then, we can save everything locally to start. Local storage might fill up quick (I forget the size limit offhand) so we might want to use IndexedDB instead, but not all browsers support that and some use WebSQL. localForage is a wrapper around all browser storage APIs to make that easy, but it also hasn't been updated in a year
We should probably go with option (1) above, i.e. have them sign and use their viewing key to encrypt. But some users might not care about storing their history, and maybe don't like the UX of now having to sign before sending? So two options are (a) make signing the message behave like "logging in" on any page, or (b) give users the option to turn it off and keep the current behavior. I think I lean towards (a) right now to keep it simpler, curious to get thoughts from @apbendi too

@mds1 Let's assign it to @alexkeating . Thanks for picking it up!

That makes sense, go with option 1 for now. I am a little unclear by what you mean by "make signing the message behave like "logging in" on any page" in terms of UX.

Ah sorry, to clarify I mean:

Right now, signing a message only happens when you view a page that requires it, and I believe it happens lazily (i.e. we don't prompt for a signature until it's needed, on the Receive and Setup pages)
But perhaps we change this to just requesting a signature at the same time a wallet is connected. But on second thought this might be bad UX because if your wallet it autoconnects its not great to ask for your signature right away.

So maybe we stick with requesting signature lazily, and just request when they click the send button. This might be confusing for existing users though since it wasn't required previously, so we should also add a corresponding FAQ question and maybe som info/help text?


Cannot connect to any saved connections since 1.35.0

Azure Data Studio Version: 1.35.0
OS Version: Arch Linux (5.16.10-arch1-1)

When trying to connect to any of the saved conections after upgrading to 1.35.0, the server icon briefly shows a spinner then reverts back to the "not connected" state (icon with red dot). Downgrading to 1.34.0 solves the issue. The issue is also present in the insiders build.
When trying to connect from within the query editor (Connect button in the toolbar) it shows the following message:

Error: The name org.freedesktop.secrets was not provided by any .service files

After enabling this service the connection dialog hangs for a few seconds, then responds with the following

Error: No such interface 'org.freedesktop.Secret.Collection' at object path '/org/freedesktop/secrets/aliases/default'

FTR I get the following output from keepassxc which provides the org.freedesktop.secrets service while trying to connect in azuredatastudio:

QDBusConnection: error: could not send reply message to service "": Marshalling failed: Invalid object path passed in arguments
QDBusConnection: Could not emit signal org.freedesktop.Secret.Service.CollectionChanged: Marshalling failed: Invalid object path passed in arguments

I'm not too familiar with the secrets service, is this now a requirement to use azuredatastudio? I don't see this mentioned in the changelog at all.
Steps to Reproduce:

Expand any saved connection in the connections tree.
Icon briefly shows a spinner, then reverts.

Does this issue occur when all extensions are disabled?: Yes

@abist This looks like it might be caused by the secret storage changes you made.
@sn4kebite The issue seems similar to https://github.com/microsoft/vscode-docker/issues/1515 - in there they mention that installing gnome-keyring fixes it. Could you see if that works for you?

Thanks for checking. @kburtram FYI for triaging

Hi @sn4kebite
Curious to know if the latest release v1.39 is able to unblock you, could you please try the same and confirm?
Thanks for taking time!

Without gnome-keyring installed the server icon just shows a spinner. However with gnome-keyring installed I have had no issues for the last few releases that I can recall, so it seems to be working fine.

Thanks! I will close this then, please feel free to open a new issue if you need additional support :)


Improve TS support for enums

If you export an enum from Rust it's possible to provide values for multiple variants on the frontend due to the fact that | is typescript is not exclusive or.
I don't think this is a problem with adjacently tagged enums.
This article is a good rundown: https://effectivetypescript.com/2021/11/11/optional-never/
Eg. this demo needs to not compile:
const a: /* Specta output */ { a: string } | { b: string } = {
  a: "abc",
  b: "cde"
};

I think we would have Specta generate { a: string; b: never; } | { a: never; b: string; } instead which would cause the appropriate errors.
For personal reference this came up with search.paths in SD.

Just stumbled into this. Would be very nice to have this feature!


Global event request

Greetings - For an add-on we're porting, we have some logic we'd like to run right before any transaction (anywhere in NAV) finishes (i.e. the point of a commit occurring, whether explicitly coded or just triggered by the end of a transaction).  Would it be possible to create a global OnBeforeCommit event we could subscribe to in order to implement our logic?

Interesting idea! However, this request goes beyond the scope of this repository. Please submit your idea to Business Central - Ideas, where it will end up as a feature request. Thanks!


Improved Documentation

Fixed Grammatical Issues and Typos.

Please have a look at the commits.
Your PR consists of already merged commits too.

Done! @the-dagger

Thank you!


UHF-8441: Custom source plugin for Hearing migrate

See #554 for instructions

Codecov Report

:exclamation: No coverage uploaded for pull request base (UHF-8441_hearing_api@c046f22). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head 3c7ddd9 differs from pull request most recent head 7c5b45f. Consider uploading reports for the commit 7c5b45f to get more accurate results

@@                   Coverage Diff                   @@
##             UHF-8441_hearing_api     #561   +/-   ##
=======================================================
  Coverage                        ?   12.74%
  Complexity                      ?      236
=======================================================
  Files                           ?       30
  Lines                           ?      902
  Branches                        ?        0
=======================================================
  Hits                            ?      115
  Misses                          ?      787
  Partials                        ?        0

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

replaced with external entities


feat(caas): backport from caas

Description of your changes

backport from caas
update

XAKS

add region as input
add providerConfig as input

XCluster

add region as input
add providerConfig as input
using patchSets

XNetwork
XService

add flux
use prometheus as default

rename folder package -> apis
enable usages for XAKS, XServices and XApp
switch crossplane.yaml to root folder
update Makefile

Fixes #
I have:

[x] Read and followed Upbound's contribution process.
[x] Run make reviewable to ensure this PR is ready for review.
[x] Added backport release-x.y labels to auto-backport this PR, as appropriate.

How has this code been tested
uptest

/test-examples

/test-examples

/test-examples

/test-examples

followup via: https://github.com/upbound/platform-ref-azure/pull/44


[Feature]: Overwrite images uploaded to Roboflow

 new_project.upload(
        image_path=image_path,
        annotation_path=annotation_path,
        split="train",
        is_prediction=True,
        overwrite=True)

I could not find overwrite parameter in (https://github.com/roboflow/roboflow-python/blob/main/roboflow/core/project.py). Could you please add owerwirting existing images

I am encountering the same problem. I would love to update/overwrite my object detection annotations via API as described in the comment by xmsi.
However, I get the following error message:

'hint': 'This image was already annotated; to overwrite the annotation, pass overwrite=true as a query parameter.'

Is there anything planned in this regard? Or might there be a workaround for overwriting existing annotations?

I am also encountering the same problem.

I am also encountering the same problem.

There's a pull request pending. Might work in the future.
#216


[Bug]: Error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports. Check the render method of Edit.

Contact Details
No response
What happened?
List what you are trying to do?Error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports. Check the render method of Edit.

Bug prevalence
.
AdminJS dependencies version
.
What browsers do you see the problem on?
No response
Relevant log output
.

Relevant code that's giving you issues
.

Please fill in the form, this issue is missing relevant information


Nnayer/hyperband

Added init_model parameter to train() to enable continued training

 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: nandini269 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Unable to delete (deregister) card

I am not able to deregister card, because of the following exception:
The cli_auth parameter is required
The code used to deregister card is as follow:
$deleteCardRequest = $gateway->deleteCard($_POST);
$tpayResponse = $deleteCardRequest->getData();

It looks like the cli_auth param is not read correctly from tpay notification payload, neither directly nor from params given to deleteCard() method.
The POST data:
Array (
[type] => deregister
[test_mode] => 1
[cli_auth] => t5a9d52f5d82da88a36f4984c59816ef86827dcd
[date] => 2018-03-05 15:25:41
[sign] => d1a966903cd2c1b3bcd2677bdb71666352ea75b9c1492ac90f6b898b4c5c543bf83371b0b36e9e4ec0aca1ba33fd041b1a66e977f77bd0d076e1a0324e7f9c7f
)

You should use setToken method before sending deregister request.
Please see the following examples:
Setting a token from purchase response data (non-3DS credit card):
$purchaseResponse = $gateway
->purchase($params)
->setDescription("payment for order xyz")
->setAmount(10.00)
->setCurrency('PLN')
->setOrderId(32145)
->setCardSave()
->send();
if ($purchaseResponse->isPaid()) {
$token = $purchaseResponse->getToken();
$deregisterResponse = $gateway
->deleteCard($params)
->setToken($token)
->send();
var_dump($deregisterResponse->isSuccessful());
}
Settign a token from notification (3DS protected credit card):
$deregisterResponse = $gateway
->deleteCard($params)
->setToken($_POST['cli_auth'])
->send();
var_dump($deregisterResponse->isSuccessful());
or
$completePurchaseResponse = $gateway->completePurchase(array())
->setAmount(10.00)
->setCurrency('PLN')
->setOrderId(32145)
->getData();
$deregisterResponse = $gateway
->deleteCard($params)
->setToken($completePurchaseResponse['cli_auth'])
->send();
var_dump($deregisterResponse->isSuccessful());
Where $params is an array containing CreditCard object initialised like:
$params = array('card' => new CreditCard($formInputData));

Thanks @piotrjozwiak, your comment clears a lot.
However, I think it's not clear from documentation nor from examples how to deregister card with tpay library. I would suggest to write it in documentation for further users.

@pszalko sure, I will make some updates :)

Hello, is there any complete example for just deregister card?


Crashing on incoming call

I successefully installed the Twilio client phonegap.
I can do call but It crash on incoming call in Android Studio and in the app on Android.
I checked the log, and when it crash i have no message.
Also, status message update on call and on end call but not on ready.

I'm using the latest Twilio Client Android library
https://www.twilio.com/docs/client/android
The status message are ok now but the app still crash on incoming call.

I had the same issue the other day. The solution for was to put ALL of the files from the lib folder of the the Twilio SDK in the lib folder of your android project.

Thanks @ari3l - I was unable to replicate this myself, and if you don't use all of the libraries, you'll definitely see a crash. I'll close this issue.


multiple domains and offline password change

Hello guys,
Everything works fine but recently a client asked me if it was possible to change password outside of their site and with another domain.
For example:

I changed the domain for the correct domain --- works fine
change password with another domain (since this client has multiple domains *.pt and *.local)
offline password change, is it possible?

I just added
"DefaultDomain": "domain1.pt", "domain2.pt", "domain.local",// Set your default AD
cheers

I don't understand the question.
You are trying to have a common PassCore to serve internal and different domains but you want to use the DefaultDomain setting?

I don't understand the question.
You are trying to have a common PassCore to serve internal and different domains but you want to use the DefaultDomain setting?

that is correct, adn how about offsite? need to publicate at IIS ?

Keeping status at #219


Normalizing Kalman Filter (local/univariate version)

Issue #, if available:
Description of changes:
Adding Normalizing Kalman Filter (univariate version). This is done by including additional output_transform argument in DeepStateEstimator.
Currently updated only the smoke test to verify these changes. Will add more tests after getting feedback from Emmanuel.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@rshyamsundar would you have the CRPS-SUM test set metrics for this method on the datasets you considered somewhere for me? Thanks in advance.

@rshyamsundar would you have the CRPS-SUM test set metrics for this method on the datasets you considered somewhere for me? Thanks in advance.


Letsencrypt for Docker

Issue #471 (Also includes #575).
I pulled this from a project I am running in production. First time working with the cookiecutter template, so I may have missed something, but the three generation cases (w/ certbot and docker, w/o certbot, w/ certbot and w/o docker) worked by a manual test on my laptop (OS X).

I am using docker-compose v1 syntax for volumes, does this need modification @jayfk ?
There is extraneous space added in docker-compose.yml on generation through the cookiecutter template.
Letsencrypt recently changes there name to certbot, not sure about naming.
There may be a better approach to dhparams.pem generation, currently specifying in documentation and providing a warning.

Hope this is helpful to what you were working on @chrisdev. Caddyserver may be a better way to go.

@mjsisley i'm no docker expert but followed your docs and i generated the CSR request and certbot responded!
However, I had to change the last line of the compose/nginx/Dockerfile
from:
ADD /start.sh
to:
ADD start.sh /start.sh

@chrisdev Missed that on the transfer from COPY to ADD and only tested the cookiecutter generation. as this arrangement has been working for me. Thanks for testing it and catching that!
Not sure where to cite sources... but this was mostly adopted from here with a few adjustments.
There may be a more concise way to accomplish the same thing. Caddyserver may be a better option as well. But maybe this is a starting point.

@mjsisley great job!

Thanks @chrisdev
The bullet list above (extra whitespace, name certbot or letsencrypt or both, and docker-compose v1 volume syntax) could be addressed, but these aren't critical for it to work.
I think this is ready to merge if those can be adjusted later.

@mjsisley I need to fix the author on the commits for this, since they ended up in my name. To do that, I need the email you used to register your SSH key on GitHub (probably your github personal mail).

@pydanny<EMAIL_ADDRESS>


CircleCI Preview for all PRs including forks

It seems a PR from a fork doesn't launch the CircleCI build (see #19). We want to enable this so we can do an easy preview.

I think I've enabled this, but we'll need a new PR from a fork to know for sure.

This did work. See PR #25.


Fix for Metabase v0.34.0

Fix for #43
I have just built the driver successfully, but haven't tested it! Can somebody do it?

@pyrooka thanks for your help! However, Metabase 0.34 uses the java.time types now, and the ClickHouse JDBC driver does not support JDBC 4.2.

Sorry, I don't really know what should I do. To be honest I've never used ClickHouse just saw your issue and dropped a quick fix. So what do you suggest?


upstream merge

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

291 of 336 (86.61%)  changed or added relevant lines in 16 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+1.2%) to 89.742%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

csvkit/utilities/csvjoin.py
3
4
75.0%

csvkit/utilities/in2csv.py
14
17
82.35%

csvkit/cleanup.py
53
57
92.98%

csvkit/utilities/csvpy.py
0
18
0.0%

csvkit/cli.py
65
84
77.38%

Files with Coverage Reduction
New Missed Lines
%

csvkit/cli.py
2
83.74%

Totals

Change from base Build<PHONE_NUMBER>:
1.2%

Covered Lines:
1566

Relevant Lines:
1745

💛  - Coveralls


verbose flag doesn't serve the purpose

- Do you want to request a feature or report a bug?
bug report
- What is the current behavior?
We've the --verbose flag as part of status command which shows up local environment information useful while submitting bug reports. But it doesn't display the information as expected.
https://github.com/netlify/cli/blob/d54b500c548c0e45d9a496eec988726e23cb67da/src/commands/status/index.js#L78
Moving the above block just below the respective line would serve the purpose.
https://github.com/netlify/cli/blob/d54b500c548c0e45d9a496eec988726e23cb67da/src/commands/status/index.js#L12
- If the current behavior is a bug, please provide the steps to reproduce.
Type in netlify status --verbose
If you're logged in, the respective information is being displayed or else an appropriate warning shows up.
- What is the expected behavior?
netlify status --verbose displays local environment information.

I can shoot a PR 👍

A PR would be awesome @jamesgeorge007


fix warning message

/usr/lib/syncplay/syncplay/ui/GuiConfiguration.py:323: SyntaxWarning: "is not" with a literal. Did you mean "!="?

Cheers


New permissions in v2.0.0

I'm curious about why the extension needs the new permissions for all tab data access in FF and read browser history in chrome.  Previously it only need it for youtube.com and studio.youtube.com, which makes sense.

It seems to have been an accident, if you look at the changelog file (https://github.com/RomainLK/ytc-filter/blob/master/changelog.md) it seems like it'll be rolled back in v2.0.2

@matthew0x40 Ah, didn't go through the changelog looking for a mention of it.  Thanks for the heads up, I see that 2.0.2 no longer is prompting me for the additional permission in Chrome.


[FLINK-2248][client]Add flag to disable sysout logging from cli

Enables disabling of sysout messages on cli via a flag q.

Thanks for the pull request. Could you add a test case to check whether the output is suppressed when -q is provided? Also, it would be great if you could update the documentation with a hint to the new flag.

@mxm , I've added unit tests to verify disabled logging.

Thanks for adding the tests! This looks good to merge. Could you update https://ci.apache.org/projects/flink/flink-docs-master/apis/cli.html ?

@mxm , updated.

Thanks. One more thing. Could you also update https://ci.apache.org/projects/flink/flink-docs-release-0.9/setup/config.html ? You introduced a new config entry.

Done.

Thank you! Merging.

Actually, I'm not sure about the config entry. We already have a lot of config entries and I don't really see the purpose of making print output of the client a system wide setting. What was your rational behind introducing the config parameter?

There was no specific need for it. However, since the CliFrontend only passes the Client the configuration, I decided to include it with that.
Further, I think while writing programs using flink [flink-example-esqe] and running them in IDE, it would  be nice to have a config entry to turn off the log. It gets distracting sometimes.
[Although this PR doesn't quite achieve that. That would require some changes in the Local Executor].

yarnfifo case failure.
nothing to do with the changes made in this PR.

There was no specific need for it. However, since the CliFrontend only passes the Client the configuration, I decided to include it with that.

There is a method on the Client to set the printing.

Further, I think while writing programs using flink [flink-example-esqe] and running them in IDE, it would be nice to have a config entry to turn off the log. It gets distracting sometimes.

If you want to achieve that, you can adjust the log4j.properties file and set logging to OFF.
I would be for excluding the config entry because I don't think people will use it.

Yeah. I think we should remove it then. Too many entries tend to confuse people.

I've updated the code to remove any changes in Configuration.

Thank you for your patience :) I will merge this once the tests have passed.

Sure. No problem. :)

Travis build passes. You can merge it @mxm


Publish aggregator image in CI

Done as part of https://github.com/wellcomecollection/concepts-pipeline/issues/5