[jvm-packages]xgboost4j-spark has different result with xgboost

i train a model，respectively use xgboost4j-spark and xgboost。
same train_data, train_params。
but get different auc, the auc useing xgboost4j-spark model is lower than local xgboost.

It is as expected, distributed training will make candidate proposal less accurate

I read code: xgboost4j-spark calls xgboost4j , and xgboost4j calls c++ code with JNI.
i don't understand why distributed training will make candidate proposal less accurate.
@CodingCat

http://www.kdd.org/kdd2016/papers/files/rfp0697-chenAemb.pdf section 3.2
please add [jvm-packages] in your title when asking about jvm stuffs (which has been EXPLICITLY stated in the issue template)

for save/load model issues, it might be caused by 0/1-based issues (https://github.com/dmlc/xgboost/tree/master/jvm-packages)

Why let this issue to close? I have the same problem. I dump tree model and find it is caused by 0/1-based as its feature index is wrong. Then how to solve it when I train model use xgboost-spark, but use the trained model to predict in local?

in local , you also need to convert  feature index 1-based to 0-based.

thanks, @wangzhilong . Maybe convert feature index is the only method now.
I want to change MLUtils.loadLibSVMFile source code and recompile the loadLibSVMFile part to make it support 0-based. Do you have any better advise?

that's what I thought before,
you'd better don't do that,  because you will have to change many places.
and online use model, the index usually start form 0.


Add Popularity Endpoint

There appears to be a 'instruments/popularity' API endpoint which drives the "Top Movers" page. I'd love to be able to see this programmatically getting summarized market activity!

I'll inspect the endpoint tonight.

@Jamonek I'd love to help with this issue if you are looking for contributors?

Great work, @jasonleehodges !


Update Open chat filter with active open chat counts

The Operator panel has filters for the Chat List.
Update the count Open (X) where X = the number of chats that have "open": true. This happens when we first load all the chats, or when incoming ones (live) come in

I think this is also resolved right?

yes this is done


For Better

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Please delete this note before submitting the pull request.
For 1.12 Features: set Milestone to 1.12 and Base Branch to release-1.12
Help editing and submitting pull requests:  https://deploy-preview-9510--kubernetes-io-master-staging.netlify.com/docs/contribute/start/#submit-a-pull-request.
Help choosing which branch to use, see
https://kubernetes.io/docs/contribute/start#choose-which-git-branch-to-use.
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

/assign @mistyhacks

I disagree with both of these changes.
/reject

assign @mistyhacks

@mmsakura 👋 Thanks for opening this PR! The suggestions you've made seem well-intentioned but are either equivalent to the existing text (into vs. in) or incorrect (pluralizing singular nouns).
I'm closing this PR, but I encourage you to keep opening PRs whenever there are clear improvements to make.


Endpoint creation via API - cannot add port to URL

Bug description
When adding a port to URL in POST /api/endpoints, the FQDN is removed from the request and the request fails. Adding a scheme (tcp/http/https) in front of the URL prevents FQDN deletion, but results in a certificate invalid error, even with TLSVerify: false.
Expected behavior
URL should go through as is.
Steps to reproduce the issue:
POST with port number:
POST <portainer_url>/api/endpoints
Headers:
  Authorization: Bearer <jwt>
Form:
  Name: test-pi
  EndpointCreationType: 2
  URL: test-pi.iot.local:9001

Returns:
  details: Get "https:9001/ping": http: no Host in request URL

POST without port number:
POST <portainer_url>/api/endpoints
Headers:
  Authorization: Bearer <jwt>
Form:
  Name: test-pi
  EndpointCreationType: 2
  URL: test-pi.iot.local

Returns:
  details: Get "https://test-pi.iot.local/ping": context deadline exceeded

POST with port number and scheme (tcp/http/https results in the same error):
POST <portainer_url>/api/endpoints
Headers:
  Authorization: Bearer <jwt>
Form:
  Name: test-pi
  EndpointCreationType: 2
  URL: tcp://test-pi.iot.local:9001

Returns:
  details: Get "https://test-pi.iot.local:9001/ping": x509: certificate is valid for localhost, not test-pi.iot.local

The URL works when adding via the Environment UI:
# Works when added in UI
Name: test-pi
Environment URL: test-pi.iot.local:9001

Technical details:

Portainer version: 2.14.2
Docker version (managed by Portainer): 20.10.17
Kubernetes version (managed by Portainer): n/a
Platform (windows/linux): linux
Command used to start Portainer (docker run -p 9443:9443 portainer/portainer): docker-compose controlled
Browser: Chrome
Use Case (delete as appropriate): Using Portainer at Home.
Have you reviewed our technical documentation and knowledge base? Yes

@anthonyma94
I wanted to follow up on this request.  I wanted to echo the thread referenced below that the issue was resolved.
https://www.reddit.com/r/portainer/comments/wqx6w7/endpoint_creation_via_api_cannot_add_port_to_url/
You were also receiving the following error message:  certificate is valid for localhost, not test-pi.iot.local
The Portainer Server and Portainer Agent use, by default, self-signed certificates to communicate with each other. This can cause errors like you're seeing if the server isn't told to expect the self-signed certificates. You can define this by telling the API call to use TLS, but to skip verification on the certificates with:
TLS: true
TLSSkipVerify: true
TLSSkipClientVerify: true

The Portainer UI adds these options automatically when adding the environment. In an API call with httpie, this might look something like this:
http --verify=no POST https://portainer:9443/api/endpoints X-API-Key:myapikey Name==test-pi EndpointCreationType==2 URL==tcp://test-pi.iot.local:9001 TLS==true TLSSkipVerify==true TLSSkipClientVerify==true
I will open an internal request to have our API documentation updated.
Thanks!


Failure to parse nested default values

When attempting to parse the CRD referenced to nodejs in https://www.rabbitmq.com/kubernetes/operator/install-operator.html (i.e. https://github.com/rabbitmq/cluster-operator/releases/latest/download/cluster-operator.yml) crd2pulumi fails with an "error binding default value for property storage".
Expected behavior

Should compile CRD correctly
Current behavior

Fails with following error:
λ c:\dev\localApps\crd2pulumi\crd2pulumi.exe -n components\cluster-operator-trimmed.yml
panic: fatal: error An assertion has failed: could not parse Pulumi package. source error: could not import spec: binding types: failed to bind type kubernetes:rabbitmq.com/v1beta1:RabbitmqClusterSpecPersistence: error binding default value for property storage: default values may only be provided for boolean, integer, number, and string properties

goroutine 1 [running]:
github.com/pulumi/pulumi/sdk/v2/go/common/util/contract.failfast(...)
        /home/runner/go/pkg/mod/github.com/pulumi/pulumi/sdk/v2@v2.9.2/go/common/util/contract/failfast.go:23
github.com/pulumi/pulumi/sdk/v2/go/common/util/contract.AssertNoErrorf(0x12c1060, 0xc0012d60a0, 0x111c62a, 0x1e, 0x0, 0x0, 0x0)
        /home/runner/go/pkg/mod/github.com/pulumi/pulumi/sdk/v2@v2.9.2/go/common/util/contract/assert.go:47 +0x1ca
github.com/pulumi/crd2pulumi/gen.(*PackageGenerator).SchemaPackage(0xc000517ba8, 0x3)
        /home/runner/work/crd2pulumi/crd2pulumi/gen/generate.go:181 +0xbc
github.com/pulumi/crd2pulumi/gen.(*PackageGenerator).genNodeJSFiles(0xc000517ba8, 0x10fcb88, 0x4, 0xc00093d100, 0x1, 0x1)
        /home/runner/work/crd2pulumi/crd2pulumi/gen/nodejs.go:39 +0x4a
github.com/pulumi/crd2pulumi/gen.(*PackageGenerator).genNodeJS(0xc000517ba8, 0xc00008f600, 0xb, 0x10fcb88, 0x4, 0x1, 0xc00093d140)
        /home/runner/work/crd2pulumi/crd2pulumi/gen/nodejs.go:30 +0x4a
github.com/pulumi/crd2pulumi/gen.Generate(0xc00068b770, 0x0, 0x0, 0x0, 0x10fcb88, 0x4, 0x10fcb88, 0x4, 0x10fcb88, 0x4, ...)
        /home/runner/work/crd2pulumi/crd2pulumi/gen/generate.go:63 +0x287
github.com/pulumi/crd2pulumi/cmd.glob..func2(0x1ac8640, 0xc0002d88a0, 0x1, 0x2)
        /home/runner/work/crd2pulumi/crd2pulumi/cmd/root.go:153 +0x218
github.com/spf13/cobra.(*Command).execute(0x1ac8640, 0xc00003e090, 0x2, 0x3, 0x1ac8640, 0xc00003e090)
        /home/runner/go/pkg/mod/github.com/spf13/cobra@v1.0.0/command.go:846 +0x2a4
github.com/spf13/cobra.(*Command).ExecuteC(0x1ac8640, 0x4455a1, 0x1a51280, 0xc00005bf78)
        /home/runner/go/pkg/mod/github.com/spf13/cobra@v1.0.0/command.go:950 +0x350
github.com/spf13/cobra.(*Command).Execute(...)
        /home/runner/go/pkg/mod/github.com/spf13/cobra@v1.0.0/command.go:887
github.com/pulumi/crd2pulumi/cmd.Execute(...)
        /home/runner/work/crd2pulumi/crd2pulumi/cmd/root.go:165
main.main()
        /home/runner/work/crd2pulumi/crd2pulumi/main.go:25 +0x39

Steps to reproduce

Retrieve operator from https://github.com/rabbitmq/cluster-operator/releases/latest/download/cluster-operator.yml (tested with 1.5.0)
Trim away non-CRD elements from yaml
Run crd2pulumi -n cluster-operator-trimmed.yml
See error above.

Context (Environment)

It's likely the following section is responsible for the error:
              persistence:
                default:
                  storage: 10Gi
                description: The desired persistent storage configuration for each Pod in the cluster.
                properties:
                  storage:
                    anyOf:
                    - type: integer
                    - type: string
                    default: 10Gi
                    description: The requested size of the persistent volume attached to each Pod in the RabbitmqCluster. The format of this field matches that defined by kubernetes/apimachinery. See https://pkg.go.dev/k8s.io/apimachinery/pkg/api/resource#Quantity for more info on the format of this field.
                    pattern: ^(\+|-)?(([0-9]+(\.[0-9]*)?)|(\.[0-9]+))(([KMGTPE]i)|[numkMGTPE]|([eE](\+|-)?(([0-9]+(\.[0-9]*)?)|(\.[0-9]+))))?$
                    x-kubernetes-int-or-string: true

Affected feature

Looks similar to https://github.com/pulumi/crd2pulumi/issues/22

Fixed in https://github.com/pulumi/crd2pulumi/pull/136.


SSO with azure AD

Would love to have a guide in the docs for AAD SSO.

Hi @CasperNaudts
Thanks for using our Learn-LTI Tool. While we would work towards improving our documentation for AAD SSO, we could use this opportunity to understrand your use case in more detail and probably what specific aspect of AAD SSO you are looking an improvement for.

Hi @adbindal
Thanks for the quick response. At our school we almost exclusively use MS-teams for everything related to exchanging information between teacher and student. For our it courses we have been looking for a platform that allows the teacher to publish their own content related to hardware and so but for the software part we want an interactive environment. We already use MS learn but don't like the options for own content.
Our main goal is to let our students sign in with there office 365 login, and if posible get some MS-teams integration.
I am looking for a step by step guide to connect the SSO experience with AAD. I have a good knolage of azure services, but I am not a hero at applications in AAD.
thanks for the support

Hi @CasperNaudts
I checked with the team and please see if the details provided at https://docs.microsoft.com/en-us/azure/active-directory/manage-apps/plan-sso-deployment aligns to your requirement. We'd also request you to reach us out via our email<EMAIL_ADDRESS>to better understand your use case and provide relevant recommendations to you.
Thanks a lot for trying to use our product in your environment.

Closing the issue since it has not seen any activity for over a week now.


Change RHEL to use sudo instead of use root user

some strange requirement from ocp lol

technically it's still root, but using sudo. idk why this would be better on OCP tho.

Great start on this!


GitHub repo api calls should be authenticated

When going to Codebases screen, getting error: 'Failed to retrieve GitHub repo: https://github.com/pmuir-demo-3/fri0704171413.git'
Root cause seems to be unauthenticated calls to api.github.com
"https://developer.github.com/v3/#rate-limiting"
message
:
"API rate limit exceeded for 'ip-addr'. (But here's the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.)"
URL seems to be: https://api.github.com/repos/pmuir-demo-3/fri0704171413

@pmuir @joshuawilson the 60 requests an hour is fixed by adding Authorisation to Headers in this PR:
https://github.com/fabric8io/fabric8-ui/pull/616
according to the doc the limit is now 5000 requests an hour.
Let's close this issue.


Bump version to anonaddy's

The docker image is still at 1.1.0, while anonaddy is at 1.2.1. Is there any hint when it will be updated?

Sorry was trying to bump Alpine before but still failing. I just created the release.


feat: update start api to be async

Fixes #542
What Changed
https://github.com/getAlby/nostr-wallet-connect-next/assets/64399555/6c103fb4-6c7e-477a-be35-ca392e6d3a5b

Not sure if I should do this for wails, I think we should, but there isn't no option to check if the unlock password im wails, why is that? cc @rolznz

Not sure if I should do this for wails, I think we should, but there isn't no option to check if the unlock password im wails, why is that? cc @rolznz

@im-adithya we don't have an unlock screen on Wails because it can't be accessed from the outside. So I don't think it's necessary to make any changes there.

Makes sense, that's what i thought, leaving this for Wails 👍


Set starting Pre/CFR tab fix

Make sure only the starting tab gets .current style to avoid seeing more than one tab "active" at once.
Fixes eregs/notice-and-comment#244

This works, but isn't the problem in the underlying markup? If we fix it there, we fix it for non-JS users, too.

Tried to edit chrome.html to no avail but tracked it down finally to preamble-base.html. Works without JS now.


Dynamic countdown component injection from JSON

#744

Converted the countDown Cards to support JSON format
dynamically inserted the component generated from JSON

Mentors kindly review the same! Thank you!

we will replace these dynamic images with images related to the project. Something kind of given in pr #812
ask @smaranjitghose for it

we will replace these dynamic images with images related to the project. Something kind of given in pr #812
ask @smaranjitghose for it

Sir I only changed the count down components which doesn't have any images on them! Also the other parts of project in the page has be injected with the necessary images! Apart from new project to be added in page!


Multiple validators for single object

I am not sure if i am maybe going about this the wrong way, so thought i would ask here.
I have a single object that has 2 different methods, one for adding the object to the database and another method for building the object. When adding an object, it must have a Name and Reference, but when building the object it only requires a Name.
To do this i have created 2 separate validators, both implement IValidator, but one also implements an interface called IBuildValidator.  Is this the right way to go about this?  Should i have 1 validator and a property to determine if its the build rules or adding rules needed?
Also, how can i register all my validators?  I have tried using some code i found in the documentation (cant find it now), but it errored because i had multiple validators for one object.

Simplest thing to do would be to use RuleSets - this allows you to group rules together, which can be executed individually: https://github.com/JeremySkinner/FluentValidation/wiki/b.-Creating-a-Validator#rulesets
I'm not sure I understand what you mean about registering validators...do you mean with an ioc container?

Yeah, registering the validators with SimpleInjector throws an exception,
but maybe that is a question for them.  I will look at rulesets, thanks
On Thu, Nov 19, 2015 at 4:18 PM, Jeremy Skinner<EMAIL_ADDRESS>wrote:

Simplest thing to do would be to use RuleSets - this allows you to group
rules together, which can be executed individually:
https://github.com/JeremySkinner/FluentValidation/wiki/b.-Creating-a-Validator#rulesets
I'm not sure I understand what you mean about registering validators...do
you mean with an ioc container?
—
Reply to this email directly or view it on GitHub
https://github.com/JeremySkinner/FluentValidation/issues/164#issuecomment-158105655
.

Yes, that's what I'd expect to happen...unless the container has some way of conditionally resolving different concrete types per interface, you'll only be able to have one validator registered.

Ok, i will ask them about it.  I managed to get it working using autofaq,
just not SimpleInjector. Will ask them, and maybe post here encase it helps
someone else too.  Thanks again
On Thu, Nov 19, 2015 at 4:21 PM, Jeremy Skinner<EMAIL_ADDRESS>wrote:

Yes, that's what I'd expect to happen...unless the container has some way
of conditionally resolving different concrete types per interface, you'll
only be able to have one validator registered.
—
Reply to this email directly or view it on GitHub
https://github.com/JeremySkinner/FluentValidation/issues/164#issuecomment-158107047
.

On Thu, Nov 19, 2015 at 4:23 PM, Darren Gillard <
<EMAIL_ADDRESS>wrote:

Ok, i will ask them about it.  I managed to get it working using autofaq,
just not SimpleInjector. Will ask them, and maybe post here encase it helps
someone else too.  Thanks again
On Thu, Nov 19, 2015 at 4:21 PM, Jeremy Skinner<EMAIL_ADDRESS>
wrote:

Yes, that's what I'd expect to happen...unless the container has some
way of conditionally resolving different concrete types per interface,
you'll only be able to have one validator registered.
—
Reply to this email directly or view it on GitHub
https://github.com/JeremySkinner/FluentValidation/issues/164#issuecomment-158107047
.

Maintenance / cleanup: Closing this as the question appears to be answered and there hasn't been any activity for a few months.


[DNM] Test toolchain build

This PR exists to try toolchain build(s) before merging a revert of a revert

https://github.com/swiftlang/swift-docc/pull/1117
@swift-ci build toolchain


nexd: Allow devices to change organizations

Issue #1096
Signed-off-by: Russell Bryant<EMAIL_ADDRESS>

This isn't a full solution, yet. This properly updates the org on the device, but the tunnel IPs don't get reallocated. It's in the new org, but with an IP from the previous org's CIDR.

Seems to be working now with manual testing. I moved between 2 orgs several times.

I should update the e2e test that specifies an org to run a few times, switching back and forth between two orgs, to cover this case.

going to add this to #1095 as the test cases are going to overlap


Adding the possibility to use a vDataFrame in a SQL query

It would be nice to add the possibility of using SQL magic on a vDataFrame.

it is added in v0.6.3 which is soon available.


{Enhancement] : Removal of unnecessary picture

Get in touch section has a picture besides the form which is not matching the all over website and also if i open it in mobile there is no picture which will affect the User Experience. Also the alignment of form with the picture is not so good. So i will remove it or add any other substitute which can match properly.

In mobile view

Please assign me this under GSSOC'24.

Stylings are already fixed and hiding the image under medium screens is done purposely.


Feature/job specs pt2

targeting #172 til merged
tests and modifications to RapidproGroupJob
details in commits
there's a flaky admin_page_spec that i think we can ignore

Coverage increased (+2.0%) to 57.553% when pulling 8d6ea7950d4790b3c182bbef0adb12a233d5aef7 on feature/job-specs-pt2 into ee69aa39824b6fd8fff1d60f17de5630b6a3b300 on feature/job-specs.

TimeCop is weird on travis. I'm going to try to get it to work.


Refactor Filter as seperate component

Currently, the Filter is a child component of the graph. This entails a rerendering of the Filter when the graph is updated which should not be the case since the FilterStateStore gets updated when a graph is rerendered.

Solved by #227


Add dictionary input for setting aflr4AIM options.

Similar to recent updates in FUNtoFEM, this adds the ability to use dictionaries directly to set aflr4AIM options.

@bburke38 can you add this save_dict_options in for the TacsAim class too for consistency?

@bburke38 can we use this new verbosity flag to silence the caps print outs in the integration tests here and here?

@timryanb Yes, we should be able to use the verbosity flag to silence those print outs. Do you want me to make a new pull request just for those?

@bburke38 yes, that would be great


Add more OS icons.

It seems like right now, it's full of generic non-Marshmallow icons for NFC, Location, Time and Bluetooth. Since I'm on Samsung Galaxy S6 on Android Marshmallow, there are some Touchwiz-only Status icons, others are Goggle's version.

The app has the ability to add custom icons if you don't like the preset ones. The Marshmallow icons are for the most part the same as Google's icons apart from the battery icon. If you have any icon sources you would like to share that are under the creative commons license or similar, please make a pull request to add them to this repository as illustrated in the readme. Otherwise, I cannot add icons from specific operating systems as it would infringe their copyright.


App instance lives after app removed from recents when track player is intialized.

Describe the Bug
App instance lives even if we remove the app from the background when the Track player is initialized/running
Steps To Reproduce
We need to look at logs. the app should be killed when we remove the app from the recent no matter if the trackplayer is running. In this case, the Sounds stops playing but the instance lives and the app also don't get killed and the app resumes instead of restarting on opening the app again. When the trackplayer is not initialized or running the app gets killed properly.
Code To Reproduce
reproducible in the example app as well
Environment Info:
Paste the results of npx react-native info
info Fetching system and libraries information...
System:
    OS: macOS 11.6.1
    CPU: (8) x64 Intel(R) Core(TM) i5-8265U CPU @ 1.60GHz
    Memory: 2.64 GB / 20.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.14.0 - ~/n/bin/node
    Yarn: 1.22.17 - ~/n/bin/yarn
    npm: 8.3.1 - ~/n/bin/npm
    Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.10.2 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: DriverKit 21.2, iOS 15.2, macOS 12.1, tvOS 15.2, watchOS 8.3
    Android SDK:
      API Levels: 28, 29, 30, 31, 32
      Build Tools: 28.0.3, 29.0.2, 30.0.2, 31.0.0, 32.0.0, 32.1.0
      System Images: android-29 | Google APIs Intel x86 Atom
      Android NDK: Not Found
  IDEs:
    Android Studio: 2020.3 AI-203.7717.56.2031.7935034
    Xcode: 13.2.1/13C100 - /usr/bin/xcodebuild
  Languages:
    Java: 11.0.11 - /usr/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: 17.0.2 => 17.0.2
    react-native: 0.67.3 => 0.67.3
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Paste the exact react-native-track-player version you are using
2.2.0-rc2
Real device? Or simulator?
Real Device
What OS are you running?
Android 12
How I can Help
What can you do to help resolve this?
Currently have no idea of the new implementation
Have you investigated the underlying JS or Swift/Android code causing this bug?
I looked into the logs and confirmed
Can you create a Pull Request with a fix?
Sorry

Did you find a solution to this problem because i'm having it too :(

No i am still living with this issue and this happens in the android rewrite as well.

same :/ ... but thank you for the response

@mpivchev  it looks like this might be a duplicate (or at least addressed by) this issue?
https://github.com/doublesymmetry/react-native-track-player/issues/1377
@afkcodes can you confirm/deny?

@jspizziri yes. It's related.

Closed for duplication. Check #1377


Scanner does not scan/recognize crit substats

Current Behavior
Scanner normally scans everything, but when i upload it to genshin optimizer it appears that scanner skip over the crit substats, like it was purposefully avoiding them.

This is just a one example
Expected Behavior
The scanner to correctly exports the data
Reproducing a Behavior
Just straight up scanning normally reproduces this. Scanner delay does not change anything.
My scanning settings is as follows;

Device OS
Windows 11
Genshin Impact Version
4.5
Inventory Kamera Version
1.3.15
Screen resolution, screen mode, and UI scale
1920x1080 Fullscreen, 1920x1080 Borderless (both tested, both bugged)
Additional notes and remarks
No response
Evidence
genshinData_GOOD_2024_04_10_15_14.json
InventoryKamera.debug.log

artifact.json
This is what scanned artifacts look like (Its from "Log all screenshots" button) Seems like it does not recognize crit substats.
Preflight Checklist

[X] I have checked for similar issues in the issue page, and I have not found any similar to mine.
[X] I am using the latest version of the scanner.
[X] I tried setting the scanning delay to maximum, and it still doesn't work.
[ ] I have attached the video recording to the bug report.
[ ] I have attached the error log to the bug report.
[X] I am available to later explains the issue.
[X] I have disabled (or documented) all screen filtering software, plugins, and enhancements while scanning.
[X] I use default in-game keybindings or have configured keybindings for the scanner.

I have the same problem

I have the same problem to. Does anyone know the solution?

I had made an inquiry about this on Discord and provided the dev with logs, but still seems like it isn't fixed. Tried it out last hour, idk what is the cause.

I had the same problem it's caused by system language. If you're on a language with dotless i like Turkish scanner will read CRIT as crıt. Changing the system language to English fixed it

I had the same problem it's caused by system language. If you're on a language with dotless i like Turkish scanner will read CRIT as crıt. Changing the system language to English fixed it

THANK YOU SO MUCH for this. It indeed fixed it and its able to recognize crit now. But i want to use my pc with my native language so the problem still stands.


Use cardano-ledger for all minimum UTxO calculations

Issue Number
ADP-2144
Summary
This PR:

uses the ledger function evaluateMinLovelaceOutput to replace the use of Cardano API function calculateMinimumUTxO in all implementation code, for all eras.
repurposes the Cardano API function calculateMinimumUTxO for use as an oracle, to compare against when testing.
removes all special-casing for the Babbage era, and replaces it with a simpler implementation that works in all eras.
revises documentation for the {compute,isBelow}minimumCoinForUTxO functions to explain their intended purpose more clearly.

Context

The ledger function evaluateMinLovelaceOutput accepts an era-specific protocol parameters object and, unlike like the Cardano API, does not require that we convert to an era-agnostic protocol parameters object. Since values of the MinimumUTxO type already contain era-specific protocol parameters, this means we can avoid the extra complication of record conversion whenever we call:

isBelowMinimumCoinForUTxO
computeMinimumCoinForUTxO

Performance
Mainnet (Alonzo):
$ time curl -X POST http://localhost:8091/v2/wallets/042094088ed439a7b14e811e0781a00185b921c2/payment-fees -d '{"payments":[{"address":"addr1qylgm2dh3vpv07cjfcyuu6vhaqhf8998qcx6s8ucpkly6f8l0dw5r75vk42mv3ykq8vyjeaanvpytg79xqzymqy5acmqgyxuyr","amount":{"quantity":100000000,"unit":"lovelace"}}]}' -H "Content-Type: application/json"
{"deposit":{"quantity":0,"unit":"lovelace"},"estimated_max":{"quantity":187633,"unit":"lovelace"},"estimated_min":{"quantity":169021,"unit":"lovelace"},"minimum_coins":[{"quantity":999978,"unit":"lovelace"}]}
real	0m0,029s
user	0m0,006s
sys	0m0,000s

Preprod (Alonzo):
$ time curl -X POST http://localhost:8090/v2/wallets/1f82e83772b7579fc0854bd13db6a9cce21ccd95/payment-fees \
> -d '{"payments":[{"address":"addr_test1qrtez7vn0d8xp495ggypmu2kyt7tt6qyva2spm0f5a3ewn0v474mcs4q8e9g55yknx3729kyg5dl69x5596ee9tvnynq7ffety","amount":{"quantity":1000000,"unit":"lovelace"}}]}' \
> -H "Content-Type: application/json"
{"deposit":{"quantity":0,"unit":"lovelace"},"estimated_max":{"quantity":169021,"unit":"lovelace"},"estimated_min":{"quantity":169021,"unit":"lovelace"},"minimum_coins":[{"quantity":999978,"unit":"lovelace"}]}
real	0m0,029s
user	0m0,006s
sys	0m0,000s

Preview (Babbage):
$ time curl -X POST http://localhost:8090/v2/wallets/1f82e83772b7579fc0854bd13db6a9cce21ccd95/payment-fees -d '{"payments":[{"address":"addr_test1qrtez7vn0d8xp495ggypmu2kyt7tt6qyva2spm0f5a3ewn0v474mcs4q8e9g55yknx3729kyg5dl69x5596ee9tvnynq7ffety","amount":{"quantity":100000000,"unit":"lovelace"}}]}' -H "Content-Type: application/json"
{"deposit":{"quantity":0,"unit":"lovelace"},"estimated_max":{"quantity":187809,"unit":"lovelace"},"estimated_min":{"quantity":169197,"unit":"lovelace"},"minimum_coins":[{"quantity":995610,"unit":"lovelace"}]}
real	0m0,036s
user	0m0,003s
sys	0m0,003s

bors try

bors r+

bors r+

bors r+

I'm going to merge this manually, as:

We've now made 4 attempts to merge with bors
Each attempt has failed (on hydra) with test failures unrelated to this PR
We've categorised every test failure with a related issue number
Each hydra job failure, when re-run, has eventually succeeded:

https://bors-ng.aws.iohkdev.io/batches/22417
https://bors-ng.aws.iohkdev.io/batches/22418
https://bors-ng.aws.iohkdev.io/batches/22419
https://bors-ng.aws.iohkdev.io/batches/23238


The --read_archive cache doesn't delete once completely read

The cache doesn't delete once an achieve has been completely read. This can lead to buggy results.
To use the feature you must:

Delete the cache file any time you change what archive file you want to read from.
Delete the cache file manually after you're finished reading from a particular archive.

Fixed in v6 Beta 3.


avoid long tail tasks due to PrioritySemaphore, remaing part

This PR is to fix @jlowe 's comment (https://github.com/NVIDIA/spark-rapids/pull/11574/files#r1795652488) after https://github.com/NVIDIA/spark-rapids/pull/11574 is already merged.

build


pip install ERROR

In requirements.txt saying:
torch==1.3.0.post2
torchvision==0.4.1.post2
However, errors happend as follows:
ERROR: Could not find a version that satisfies the requirement torch==1.3.0.post2 (from versions: 0.1.2, 0.1.2.post1, 0.1.2.post2)
ERROR: No matching distribution found for torch==1.3.0.post2
ERROR: Could not find a version that satisfies the requirement torchvision==0.4.1.post2 (from versions: 0.1.6, 0.1.7, 0.1.8, 0.1.9, 0.2.0, 0.2.1, 0.2.2, 0.2.2.post2, 0.2.2.post3, 0.3.0, 0.4.1, 0.5.0)
ERROR: No matching distribution found for torchvision==0.4.1.post2
What is the correct version?

Sorry for the inconvenience.
I guess the reason for this error is this environment is for Mac OS. Since our previous Linux system is down, we are not able to export a Linux environment at this moment. However, we'll upload the environment for Linux as soon as it's fixed.

Thank you very much. It does help me a lot. Now I'd try other version to see if it is suitable for Linux.


[UI] Getting Started: Playing video leads to console error

Reported by @huangginny  on the CommonJS Trello board
Steps
Open console. Navigate to "getting started" and play a video.
Expected
Console should show nothing
Actual

@sloria any thoughts about this?

It looks like it's a google chrome issue, not an OSF issue.
From http://stackoverflow.com/questions/25814914/chrome-youtube-cast-sender-js-console-error:

... it appears that Chrome attempts to get cast_sender.js on pages that have YouTube content. I'm guessing when Chrome sees media that it can stream it attempts to access the Chromecast extension. When the extension isn't present, the error is thrown.

OK @samanehsan ... good find ... I'm going to Close. But we'll have for reference if needed. Thanks.

I should add that the 404 error when trying to get the youtube video is a separate issue, but it doesn't prevent the videos from playing. Not sure if this one is us or the embedded youtube player.


Should root update instructions include pausing automation?

Do we need to pause the Update Snapshot and Timestamp automation during the root signing week? If so, it should be added to documentation.

No - we don't. Root signing occurs on a branch and the automation runs on main.


Description for URLs

Hello,
I am planning to use YOURLS for my blog and while testing I came up with a thought about being able to add a description for an URL. That would be very useful when using multiple shorturls to one reference and given it away to twitter, facebook, friends etc. With an description you would be able to mark where which shorturl went to or just make some notes to it.
Of course you could use custom shorturls or title for that but that would not be useful if the description has to be a bit longer than just a few letters.
So my question is if it would be possible to implement.
Thanks!
Dragon013

Several issues about this. It'll be easily done with a plugin in the future.


p11-kit: 0.23.15 -> <IP_ADDRESS>

Motivation for this change
https://github.com/p11-glue/p11-kit/releases/tag/<IP_ADDRESS>
https://github.com/p11-glue/p11-kit/releases/tag/0.23.16
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Based on previous PR this would probably need to be targeted at staging branch
Otherwise :heavy_check_mark: on removing the two patches they were merged upstream.


Merge the Declarative DSL into the main repo trunk

Context

Contributor Checklist

[ ] Review Contribution Guidelines
[ ] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[ ] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[ ] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[ ] Ensure that tests pass sanity check: ./gradlew sanityCheck
[ ] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest

Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test RFN

I've triggered the following builds for you. Click here to see all build failures.

ReadyForNightly build

@bot-gradle test this

@bot-gradle test RFN

I've triggered the following builds for you. Click here to see all build failures.

ReadyForNightly build

@bot-gradle test RFR

I've triggered the following builds for you. Click here to see all build failures.

ReadyForRelease build

One thing must be done before merging: shading kotlinx-serialization into declarative-dsl-core.

It looks like don't need to do that:

the Gradle API jar generated by a distribution produced by this branch doesn't contain the kotlinx.serialization code
kotlinx.serialization is also filtered out from the plugin classloaders

Let's discuss and check this again later, but for now, it looks like this is a generically solved problem and we don't need to change anything.

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test RFM

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle merge

Can't merge a draft PR.

@bot-gradle merge

The merge queue build has started. Click here to see all failures if any.


[Acg Player][2.6.2]The app crashes if i click random play icon.

Actual behavior
App crashes if i click on random play icon.
Expected behavior
Expected that i can play song randomly by clicking random play icon.
How to reproduce

Install the apps and open the app
Click playlist
choose a playlist and click on that.
Click the random play icon.
See the error

Information

App version:v2.6.2
OS:Android 9.0
Device:Samsung A10

Recording Of The Bug
 youtube.com
Logcat

05-24 15:28:01.892 20703 20703 E AndroidRuntime: FATAL EXCEPTION: main
05-24 15:28:01.892 20703 20703 E AndroidRuntime: Process: top.geek_studio.chenlongcould.musicplayer.Common, PID: 20703
05-24 15:28:01.892 20703 20703 E AndroidRuntime: java.lang.IllegalArgumentException: bound must be positive
05-24 15:28:01.892 20703 20703 E AndroidRuntime: 	at java.util.Random.nextInt(Random.java:388)
05-24 15:28:01.892 20703 20703 E AndroidRuntime: 	at top.geek_studio.chenlongcould.musicplayer.activity.ListViewActivity.d(SourceFile:32)
05-24 15:28:01.892 20703 20703 E AndroidRuntime: 	at h.a.a.b.a.L.onMenuItemClick(Unknown Source:2)
05-24 15:28:01.892 20703 20703 E AndroidRuntime: 	at androidx.appcompat.widget.Toolbar$1.onMenuItemClick(SourceFile:2)
05-24 15:28:01.892 20703 20703 E AndroidRuntime: 	at androidx.appcompat.widget.ActionMenuView$MenuBuilderCallback.onMenuItemSelected(SourceFile:2)
05-24 15:28:01.892 20703 20703 E AndroidRuntime: 	at b.b.f.a.k.a(SourceFile:19)
05-24 15:28:01.892 20703 20703 E AndroidRuntime: 	at b.b.f.a.o.invoke(SourceFile:2)
05-24 15:28:01.892 20703 20703 E AndroidRuntime: 	at b.b.f.a.k.a(SourceFile:39)
05-24 15:28:01.892 20703 20703 E AndroidRuntime: 	at b.b.f.a.k.a(SourceFile:36)
05-24 15:28:01.892 20703 20703 E AndroidRuntime: 	at androidx.appcompat.widget.ActionMenuView.invokeItem(SourceFile:1)
05-24 15:28:01.892 20703 20703 E AndroidRuntime: 	at androidx.appcompat.view.menu.ActionMenuItemView.onClick(SourceFile:2)
05-24 15:28:01.892 20703 20703 E AndroidRuntime: 	at android.view.View.performClick(View.java:7339)
05-24 15:28:01.892 20703 20703 E AndroidRuntime: 	at android.widget.TextView.performClick(TextView.java:14222)
05-24 15:28:01.892 20703 20703 E AndroidRuntime: 	at android.view.View.performClickInternal(View.java:7305)
05-24 15:28:01.892 20703 20703 E AndroidRuntime: 	at android.view.View.access$3200(View.java:846)
05-24 15:28:01.892 20703 20703 E AndroidRuntime: 	at android.view.View$PerformClick.run(View.java:27788)
05-24 15:28:01.892 20703 20703 E AndroidRuntime: 	at android.os.Handler.handleCallback(Handler.java:873)
05-24 15:28:01.892 20703 20703 E AndroidRuntime: 	at android.os.Handler.dispatchMessage(Handler.java:99)
05-24 15:28:01.892 20703 20703 E AndroidRuntime: 	at android.os.Looper.loop(Looper.java:214)
05-24 15:28:01.892 20703 20703 E AndroidRuntime: 	at android.app.ActivityThread.main(ActivityThread.java:7073)
05-24 15:28:01.892 20703 20703 E AndroidRuntime: 	at java.lang.reflect.Method.invoke(Native Method)
05-24 15:28:01.892 20703 20703 E AndroidRuntime: 	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:493)
05-24 15:28:01.892 20703 20703 E AndroidRuntime: 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:964)

@AugustToko can i know why you added  duplicate lebel? Or can you add yhe reference?

@AugustToko can i know why you added duplicate lebel? Or can you add the reference?

The logic of fast play and random play is the same.
This is a legacy issue and I will correct the name.
In addition, this logic has been fixed.

Oh but fast play working  fine in my device💪

So dear can you remove  the duplicate  lebel  now?

So dear can you remove the duplicate lebel now?

ok and you can try the new version to solve your problem
On the release page

Still fetching the same problem

Still fetching the same problem
I downloaded the latest version.

Ok, I got it wrong. I thought it was a random play of the main page. 😰It has been fixed.

Haha ,not need to be sad its ok

When ypu will release  the fixed  version

When you will release the fixed version
Compiling, going to upload after a while

Ok waiting  for that to test it again😜

Ok waiting for that to test it again😜

It is done
I am very grateful to you for taking the time to participate in this project.🙃

Omg! Its working  fine now.Great♥♥

Omg! Its working fine now.Great♥♥

That's great, you can check it out by the way https://github.com/AugustToko/ACG-Player/issues/8 ，It should also be fixed


DBZ-5854 Bump Infinispan dependencies' versions.

Running this locally against Oracle 19C with Infinispan buffer enabled; waiting for test status.

@Naros Do we have test results? Thanks

Tests passed, merged.


Update to reflect changes to nightly compiler

In addition to the changes in this commit, the shell line
export RUST_TARGET_PATH=`pwd`

must be run in order for the OS to work. I am unsure of where (in the makefile) to put this.

My commit should set the variable, but it still doesn't work for me. I'm getting the following error:
error: failed to run `rustc` to learn about target-specific information

Caused by:
  process didn't exit successfully: `rustc - --crate-name ___ --print=file-names --sysroot /home/pounce/.xargo -Z force-unstable-if-unmarked --target x86_64-ESALP --crate-type bin --crate-type rlib` (exit code: 101)
--- stderr
error: Error loading target specification: Could not find specification for target "x86_64-ESALP"
  |
  = help: Use `--print target-list` for a list of built-in targets

error: `"cargo" "build" "--release" "--manifest-path" "/tmp/xargo.joWJvGe1eSZN/Cargo.toml" "--target" "x86_64-ESALP" "-p" "alloc"` failed with exit code: Some(101)
note: run with `RUST_BACKTRACE=1` for a backtrace
make: *** [Makefile:80: xargo] Error 1

The syntax might vary across different shells. I think RUST_TARGET_PATH=$(pwd) is the most common syntax, maybe that works.

@phil-opp
Yes, I did this but deleted it when I copied it to a different place xD I'll fix it

Thanks!


PodUnschedulable condition is added over and over instead of being replaced

Is this a BUG REPORT or FEATURE REQUEST?:
We add pod conditions to the VMI, to make it more obvious if things go wrong. This logic does not seem to work. Instead of replacing an existing condition, it just appends it over and over.

Uncomment only one, leave it on its own line:

/kind bug

/kind enhancement

What happened:
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
just create a VMI without a PVC: kubectl.sh create -f  examples/vmi-windows.yaml.
status:
  conditions:
  - lastProbeTime: null
    lastTransitionTime: null
    message: 'failed to render launch manifest: didn''t find PVC disk-windows'
    reason: Unschedulable
    status: "False"
    type: PodScheduled
  - lastProbeTime: null
    lastTransitionTime: 2019-08-08T10:48:25Z
    message: 'failed to render launch manifest: didn''t find PVC disk-windows'
    reason: FailedPvcNotFound
    status: "False"
    type: Synchronized
  - lastProbeTime: null
    lastTransitionTime: null
    message: 'failed to render launch manifest: didn''t find PVC disk-windows'
    reason: Unschedulable
    status: "False"
    type: PodScheduled
  - lastProbeTime: null
    lastTransitionTime: null
    message: 'failed to render launch manifest: didn''t find PVC disk-windows'
    reason: Unschedulable
    status: "False"
    type: PodScheduled

Anything else we need to know?:
Environment:

KubeVirt version (use virtctl version):
Kubernetes version (use kubectl version):
VM or VMI specifications:
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

@davidvossel @cynepco3hahue @slintes
The yaml will grow over time until it has the max size of what is allowed in etcd.

ouch, good catch.
@rmohr Are you already looking into it? Else I can do.

Would be great if you can.

Instead of replacing an existing condition, it just appends it over and over.

whoa, good catch. That's not an obvious unit test case :)

/reopen
https://github.com/kubevirt/kubevirt/pull/2583#issuecomment-520122520


Translation selected but is not showing on Kata

I created a Typescript translation tonight to my Kata.
https://www.codewars.com/kata/help-suzuki-complete-his-chores
I published the translation then approved it but it has not appeared on the Kata and no honor was given. I've seen this behavior before on other Kata and had the issue one time before for a translation someone else did on my Kata and Jake had to fix it.

As you can see Translation shows as "closed" but it does not appear on Kata.

The translation is now appearing on the Kata so the issue has now been resolved.

Thanks. The background processing queue had an outage so thats why this happened.

Awesome thanks Jake.


Consider adding a lower-bound on time between manual- & auto-searches

We'd like to avoid spamming remote sources with requests. Currently we do this for bulk-search where we introduce a delay between requests (and bulk-search is not meant to be fast/interactive anyway). However there is still the potential for a user to start a playlist and then spam "next" to blast through many tracks and trigger searches for all of them.
How common is this? I don't know. I would expect it to be quite rare, so maybe its not worth the implementation/maintenance complexity.
We could let it allow some burst (so the user can make, say, 3 requests in quick succession before getting throttled) and then have it add a delay before each search that...maybe ramps up with additional requests until it reaches the delay we use in bulk-search? Seems like it'd feel bad if we snapped right up to the bulk-search delay but maybe if the burst allowance is large enough its fine? This is the sort of thing that metrics are meant to help with.

While I understand your concerns, being honest I don't see this as a problem. Why would you manually download a hundred of lyrics when you can use the bulk search? Makes zero sense to do that.
Regular users don't hack features to bypass artificial limitations. Developers do. And developers (or like-minded users) would simply use other tools for this (much more suitable), not foobar with manual playback.
Just my POV so you don't speed energy with things which are not relevant at all compared to other things.


feat: add ShowsRailHomeViewSection tracking

This PR resolves ONYX-1262
Description
This PR adds legacy tracking to shows rails sections.
https://github.com/user-attachments/assets/15cc66cb-3bfe-40c2-b400-91832e74d2fe

PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

add ShowsRailHomeViewSection tracking - mounir

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Dev changes (add ShowsRailHomeViewSection tracking - mounir)

  Generated by :no_entry_sign: dangerJS against 0f255e4e8f760277a13dded191f72bd0ce99583a


Better OSGi support for saaj-api

JAX-WS relies heavily on java.util.ServiceLoader for pluggable implementations of the various JAX-WS APIs. The ServiceLoader does not work properly in an OSGi environment, and hence getting a SOAP client up-and-running using Jakarta saaj-api requires some hackery with the thread context classloader.
In order for soap-api to work properly in finding saaj implementations in an OSGi environment without TCCL hacks, it needs to be properly configured. There are a couple of possibilities:
Use org.glassfish.hk2.osgiresourcelocator
This is a bundle that examines all bundles as they are loaded, looking to see if they have any files in their META-INF/services directory, and registering those that do. Clients can then call the utility methods in org.glassfish.hk2.osgiresourcelocator to find and instantiate them.
All of the other JAX-WS API components in Jakarta support OSGi in their respective FactoryFinder implementations using the org.glassfish.hk2.osgiresourcelocator bundle:
ws-api:
https://github.com/eclipse-ee4j/jax-ws-api/blob/9e05dfabc3a244a68604b143302d7eff5e7b8a07/api/src/main/java/javax/xml/ws/spi/FactoryFinder.java#L72-L75
bind-api:
https://github.com/eclipse-ee4j/jaxb-api/blob/55180e9082f0172019341dd5f8c44eabe42555cc/jaxb-api/src/main/java/javax/xml/bind/ContextFinder.java#L309-L310
However, it seems that the equivalent support is missing from the soap-api implementation.
Use the OSGi ServiceLoader mediator standard
More information about this here: https://osgi.org/specification/osgi.cmpn/7.0.0/service.loader.html
Apache Aries SPI-Fly is the reference implementation: https://aries.apache.org/modules/spi-fly.html
Pro/Con comparison of the two approaches
Arguments in favour of GlassFish's osgiresourcelocator implementation:

Using the GlassFish osgiresourcelocator method is consistent with other Jakarta EE implementations like bind-api and jax-ws-api.
As a result of the first point, this is easier to implement in the short-term.

Arguments in favour of the ServiceLoader mediator implementation:

It is an OSGi standard rather than a proprietary one.
Consumer bundles' Java code doesn't need to be explicitly written to support it - SPI-Fly can re-write them automatically. (Compare this with the GlassFish implementation, where a client must explicitly call the GlassFish module to find implementations in an OSGi context.)
The Require-Capability/Provide-Capability syntax of the ServiceLoader mediator makes the dependencies explicit and understandable by the OSGi resolver module. This means that missing implementation modules will be discovered at bundle resolve time rather than at runtime. For example, if soap-api/saaj-api declared the following header"

Require-Capability: osgi.serviceloader; filter:="(osgi.serviceloader=javax.xml.soap.SAAJMetaFactory)";cardinality:=multiple
...then the OSGi resolver won't resolve the bundle (and hence won't start it) unless there is another bundle installed (eg, saaj-impl) with the corresponding Provide-Capability header declaring that it contains an implementation of javax.xml.soap.SAAJMetaFactory.

Integrates with OSGi's in-built services infrastructure - ServiceLoader implementations are automatically registered as OSGi services in the framework, and benefit from all of the service management functionality that an OSGi framework provides.

Conclusion
For what little my recommendation may be worth, I recommend the following:

Make it a medium-to-long-term goal to migrate all of the Jakarta API bundles and their reference implementations to use the ServiceLoader mediator specification, for all of the long-term benefits that provides.
Possibly in the short-term, add the few lines needed to javax.xml.soap.FactoryFinder to support loading SAAJMetaFactory implementations in an OSGi context without TCCL hacks.

As a bit of background, I feel strongly about using the ServiceLoader mediator approach in the long term, based on my own recent (and painful) experience in trying to get a SOAP client up-and-running in an OSGi container. I started with a simple "Hello World"-type implementation that I had working in plain Java, and tried to migrate it to OSGi. I had to keep running it only to get ClassNotFoundException errors at run time. I'd have to find the bundles containing the missing classes, add them, and try again. This took several iterations (plus a patch to soap-api in my working copy to use osgiresourcelocator) before I finally got a working client. Declaring the dependencies explicitly would make this process much easier and less error-prone.

added org.glassfish.hk2.osgiresourcelocator so it is consistent with related APIs.
Keeping this open as a reminder about other (probably better) option.

Thanks @lukasj! What's the time frame for the next release?

@kriegfrj I see I forgot to reply here... API 1.4.2 is out for some time already and new release of implementation (1.5.2) went out about a week ago

@kriegfrj I see I forgot to reply here... API 1.4.2 is out for some time already and new release of implementation (1.5.2) went out about a week ago

Thanks @lukasj, i did notice that 1.4.2 had gone out thanks! Does 1.5.2 have the Service Loader Mediator support? :)

@kriegfrj no, I'm quite busy with moving everything to jakarta namespace

@kriegfrj no, I'm quite busy with moving everything to jakarta namespace

:+1: It's ok, as a short term workaround @rotty3000 has made a patch to SPI-Fly so that you can manually override the serviceloader mediator's metadata for bundles that are missing it. So we can get around it now. But it will still be good if the bundle includes it because then it can help with the OSGi resolver ensuring that the right bundles are present.

I'm a bit confused by the abundance of packages in mvncentral - which one is the correct one representing this saaj-api implementation in version 1.4.2?

see https://eclipse-ee4j.github.io/metro-saaj/

Thanks for the link, but I simply can't get this to work in the OSGI Equinox environment. @kriegfrj did you manage to use the osgiresourcelocator approach? Do I understand you right, that there is a spi-fly implementation that allows me to combine all of this?

@col-panic , I did manage to get it to work using Equinox and osgiresourcelocator with the fix in this issue. Make sure you have the hk2 bundle installed and configured to start before the jax-ws bundles.

@kriegfrj thx for the help. Unfortunately all of this does not play well. Application is Eclipse Equinox based with JAXB parts, serving and clienting in JAX-WS and serving and clienting in JAX-RS - transition to Java 11 with all of this seems not feasible at the moment.

Hi @col-panic , with Java 11 i remember i needed to put an --add-module on the command line for some of the xml support... though I've never done server-side, my code was only for a client.

What about following http://docs.osgi.org/reference/portable-java-contracts.html#references and use JavaSOAP for Contract ID? Would that be good enough? Note that that reference is missing the javax.xml.soap package/JSR 67. I'm also not sure if (and if so, then when) this list will adopt jakarta names.

I am also facing issues while using saaj-api in OSGi and getting errors like javax.xml.soap.SOAPException: Unable to create message factory for SOAP: Unable to create SAAJ meta-factoryProvider com.sun.xml.messaging.saaj.soap.SAAJMetaFactoryImpl not found. On debugging the MessageFactory code, it seems the context classloader does not have any class and hence the above error. Could someone please guide me to the steps to use saaj-api with Java 11?
Note: If I use the plain jars and add them to classpath, it works but does not work with plugin based dependency.


add login route and login form

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 12 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-2.2%) to 68.229%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/pages/Login/Login.tsx
0
12
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-2.2%

Covered Lines:
232

Relevant Lines:
333

💛  - Coveralls


Partially regenerate codepoint widths from Emoji 13.0

This removes all glyphs from the emoji list that do not default to
"emoji presentation" (EPres). It removes all local overrides, but retains
the comments about the emoji we left out that are Microsoft-specific.
This brings us fully in line with the most popular Terminals on OS X,
except that we squash our emoji down to fit in one cell and they let
them hang over the edges and damage other characters. Oh well.
Refs #900, #5914.
Detailed Description of the Pull Request / Additional comments
Late Friday evening, I tested my emoji test file on iTerm2. In so doing, I realized
that @j4james and @leonMSFT were right the entire time in #5914: Emoji
that require U+FE0F must not be double-width by default.
I finally banged up a powershell script that parses the UCD and emits a codepoint
width table. Once checked in, this will be definitive.
Validation Steps Performed

Nah it's fine go

Thanks for doing this! Making only EPres Emojis wide makes total sense.


[Issue] Surface docker build output as part of azd deploy output

Right now, during azd deploy no information is shown around building a docker image. In the template I'm working on, the pip install  takes a good while (not an azd problem :D) so this feels like a really long time without any progress (like ~5 mins):

I think it'd be really helpful to be able to see some of the steps happening under the hood. That way, the user is able to see exactly what step it's working on. We could consider surfacing the information in a "window" (similar to what pip install looks like below) to make it clear that this is a subprocess as part of azd deploy.

Adding @Austinauth.
currently, we don't have multi-line temporal log support for outputs.  We only have single-line temporal log support.

@vhvb1989 Can you explain a little further? I'm not sure what that means.

@Austinauth Right now, we support single line status/progress (e.g. azd provision; each operation gets it's own line).
As part of azd deploy, we run docker build to build the container image before sending it off to ACR and then using it in the deployment. However, we don't actually show the Docker output as part of the azd deploy command. Instead, the user just sees image #1.
Image 2 is what the user sees if they run docker build outside of azd. This ask is to show users that progress and surface it in azd deploy .

@savannahostrowski is the image in your post above about the average length of the typical Docker output? Could it be substantially longer?

It's a fixed-height, dynamic window.  The output scrolls up as the build progresses.

After speaking with @wbreza here is the initial proposal for surfacing Docker output (or any output) in a sub-step. Consider these mock-ups as mid-fidelity and open to feedback. Wallace will be researching the feasibility of the output viewport illustrated in the screens below.
1) Output displayed during a sub-step as it completes

Docker output is displayed in a fixed-height, dynamic-width viewport beneath sub-steps where surfacing Docker output is critical. Viewport should be capped at roughly 12-14 lines (we can tweak as needed).
Content within the output viewport should maintain the style of the service it originates from (unless its easier for us to apply our own styles).
This pattern should scale to work for other types of output.
Nice to have:

Ability to scroll up and down to view lines of text out of the view port.
Ability to scroll up and down using keyboard inputs (↑ and ↓)

2a) Completion of a sub-step that surfaced Docker output

Upon successful completion of the sub-step Docker output should disappear, only persisting if the user ran the command with the -verbose flag. If this ends up feeling awkward or confusing in realtime we can adjust.
2b) Failure of a sub-step that surfaced Docker output

On error/failure the sub-step follows the traditional sub-step failure pattern UX, however the Docker output persists so that the user can better troubleshoot the cause of the problem.
Feedback needed:

What is the appropriate Error hint text when a sub-step fails as part of an azd deploy? Try again?
What is the appropriate Error hint text when a sub-step fails as part of an azd up? The same thing we have for when the provisioning step fails?

@rajeshkamal5050 @Austinauth Let's chat about this in the biweekly demo on 2/21.

@rajeshkamal5050 Does engineering have feedback on this?

@vhvb1989 please take a look at the above proposed mockups. Remember you said it will be complex to get this done. Keeping it for the April fit-n-finish iteration and can be re-accessed based on the scope of work.

We can remove the ability to scroll up and down through the output if needed. From what I understand this wold complicate things quite a bit.


clickhouse: toYYYYMM, add / subtract dates_and_times

@twheys hi. just added a few functions for working with dates.
Could you add me to contributors? I just don’t want to sync fork.
JFYI: I use pypika in production. I can add some functionality from time to time + fix problems.
Thank you.

Coverage increased (+0.02%) to 97.491% when pulling 91a19b2e5889050de1966ac9abf2dcdc74475970 on d-ganchar:feature/CH_support into 216895329f845b7a65f5fc4326d861582f14b85c on kayak:master.

@d-ganchar appreciate all of your contributions so far, but I'm not sure if I am ready to add external collaborators to this project. Perhaps in the future. Would like to write a proper contribution guide before then and configure some permissions here on github before.

@d-ganchar appreciate all of your contributions so far, but I'm not sure if I am ready to add external collaborators to this project. Perhaps in the future. Would like to write a proper contribution guide before then and configure some permissions here on github before.

@twheys everything is fine? :)

Sorry, busy month, a bit slow to respond lately. Will check.

Looks good! Thanks, merging and publishing


Doesn't seems to work with non https url

My server doesn't have https and when I put my link (http://my.server.com) to my server I have this error on ios
When I change my URL to https://my.server.com the message doesn't appear but I have a timeout error (expected because I don't have https)
It was working last time I entered the server URL. (I had a cache issue so I disconnected and now I'm stuck out of my server)

Maybe it's linked to this : https://stackoverflow.com/questions/32631184/the-resource-could-not-be-loaded-because-the-app-transport-security-policy-requi

This just started happening to me as well. It used to work in previous versions, I think something broke with the last update. For reference I've always used Tailscale as a VPN and it worked fine, but now I'm getting the same issue as OP.

Hey, this appears to be a regression after updating some dependencies. I'm incredibly sorry that happened. I'm releasing a hot fix for this right now, that should hit both stores asap. Thanks for reporting this!

I am still getting this error in IOS. As per appstore the version is 2.3.1 and this should be fixed. Not sure how to check the version inside the app itself. Works with https, with http and port number in URL I get the same error.

This is still an issue for me even after the hot fix.

Same here still having the same issue on the newest app version.

Hey all, I did a little digging. As it turns out the App Transport Security settings are not additive but rather exclusionary. I left a setting in for local connections, and it ended up overriding the default one that allows HTTP for all origins. I'm releasing v2.3.2 now, which I've tested to be a definitive fix for these issues. They're in the queue for both TestFlight and the App Store and I expect at least one of those to be released in the next 24 hours. Incredibly sorry again for the regression! Thanks for all your patience and your continued usage of Fintunes!


[Event Request] codeunit 90 "Purch.-Post" - OnBeforeArchiveSalesOrders

Please add new event in function ArchiveSalesOrders
local procedure ArchiveSalesOrders(var TempDropShptPostBuffer: Record "Drop Shpt. Post. Buffer" temporary)
var
    SalesOrderHeader: Record "Sales Header";
    SalesOrderLine: Record "Sales Line";
    IsHandled: Boolean;
begin
    IsHandled := false;
    OnBeforeArchiveSalesOrders(TempDropShptPostBuffer, IsHandled);
    if IsHandled then
        exit;

[IntegrationEvent(false, false)]
local procedure OnBeforeArchiveSalesOrders(var TempDropShptPostBuffer: Record "Drop Shpt. Post. Buffer" temporary; var IsHandled: Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 41235.


Temporary pin Elastic/Kibana images to release-calient-v3.9

Description

For PR author

[ ] Tests for change.
[ ] If changing pkg/apis/, run make gen-files
[ ] If changing versions, run make gen-versions

For PR reviewers
A note for code reviewers - all pull requests must have the following:

[ ] Milestone set according to targeted release.
[ ] Appropriate labels:

kind/bug if this is a bugfix.
kind/enhancement if this is a a new feature.
enterprise if this PR applies to Calico Enterprise only.

Tested on a GCP k8s cluster and the provision will success.


bring back old offers method, to allow getting more than 32 offers

@alex-pcln needs functionality to scrape more than 32 offers, so we decided to bring back the old method where we scrape offers by scrolling thorugh the page.
see issue #2

fixed in pr #28


A user should be able to ask directions

"How to go from here to the city center"
Return a Google Maps link


ci: version check fails if version was decreased

Task linked: CU-2fkkub0 Version check fails if version was decreased


ref: Update applySettingDefaults typings to use accurate return type

Internal, non-breaking change. applySettingDefaults enforces existence of all options namespaces, so they should not be optional anymore.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.9%) to 64.948%

Totals

Change from base Build<PHONE_NUMBER>:
-0.9%

Covered Lines:
90

Relevant Lines:
117

💛  - Coveralls


Fix falsy values such as '0' in style keys.

I discovered this trying to use flexShrink: 0.

NOTE: I tested this in my project, but did not run jsxstyle's tests because I couldn't get contextify to compile and didn't want to muck with it. I'm reasonably confident that this fixes the problem.

@brandonbloom good catch, thanks! I’ll run tests in a bit to see if everything is still green.

I’ve refactored the way we generate style keys and this bug is fixed as a byproduct of those changes. We now use a version of dangerousStyleValue (https://git.io/v7zKG) to stringify the style prop value and that function takes valid falsy values into account.
I’ve also refactored our tests so they’ll actually run reliably 😅
Thanks for the PR!


Testnet hotfix/v1.0.2 fast syncing

Purpose

Merging https://github.com/Taraxa-project/taraxa-node/pull/1726 based on master to testnet through new branch(due to conflicts)
How does the solution address the problem

Changes made

prnet cleaned-up successfully


Choose between names, scientific notation, and long form

Put in an option to show numbers using shortened names, scientific notation, or simply the entire number. Note that showing the entire number will require inserting commas in appropriate places.

Don't forget the adjustable significant figure count for the scientific notation.


Fix isolate:false for nested rules with media queries

Not sure how can I test that.

Is there a proper way to create conditional rule without jss-nested?

merged


Availability of other fonts

Hi,
I would like to use e.g the "Menlo Regular for Powerline" in combination with the awesome fonts.
As far as I understand I would need to patch these myself?
Is there a a simply way to do this?

This project uses FontForge, which has some pretty good documentation here. I haven't used it myself.

@flrgsr You can look in the patching-strategy branch for some script


#586 use label and width for non dropdown selector

bump version too

No
On Wed, Feb 1, 2023, 12:17 Fabien Lelaquais @.***>
wrote:

@.**** approved this pull request.
👌
Would that impact 2.1?
—
Reply to this email directly, view it on GitHub
https://github.com/Avaiga/taipy-gui/pull/597#pullrequestreview-1278914651,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVQBA5COHWNU3HSBGNR2WHLWVJA3RANCNFSM6AAAAAAUNSQ4DE
.
You are receiving this because you were assigned.Message ID:
@.***>


Pool Performance

Hello,
i am using mysql-simple in a batch job, i've to run such a query around 50.000.000 times:
i use the default pool:
let pool = mysql::Pool::new(connection_string).unwrap();

and this code to run the query:
let mut result = match pool.prep_exec(
    &format!("SELECT x.x_id from {}.page where x.title = :title and x.foo = 0 LIMIT 1", database),
    params!{"title" => (Value::from(title))}
) {
    Ok(r) => r,
    Err(e) => { panic!("");}
};

i tryed to run this query in multiple threads (binary and the mysql server most time dont hit 30% cpu usage).
On my local pc the throughput don't increase (or change) if i run the same query using multiple threads. i also tryed to obfuscate  the query to prevent caching (using a SELECT ... and [random] = [random]).
the only real difference i can see by running multiple threads is that on osx kernel_thread starts to consume up to 100% cpu.
can you give me a hint how i can increase the load to the mysql server?

Thanks for report!
I am experiencing similar problem but i can't test possible fixes until wednesday.
Could you please test changes provided in #54 in case of single and multiple threads.
For this you need to clone blackbeam-pool-performance branch and point it's directory in your Cargo.toml like this:
[dependencies.mysql]
path = "/path/to/cloned/blackbeam-pool-performance/branch"

Well. #54 is definitely a bug, but it affects performance only for cases where number of threads is greater then pool.min.
My issue with batch job (around 30.000.000 small updates) was caused by unnecessary file IO on each iteration and was not related to a pool performance. Before i fixed this unnecessary file IO issue top shows me that mysql consumption is around 15% CPU and binary consumption is around 5% CPU. After fix - mysql around 250% CPU and binary around 150% CPU (on a server with 8 cores). So it looks like pool was not involved.
But pool do have overhead, so you will perform better if you take one connection per thread from a pool before entering into query loop.
I also have a few changes which reduces pool overhead for multi-threaded scenarios. I will publish them soon.

Please update to v6.0.0


Notify not working after writing

Describe the bug
Notify not working after writing to BLE device on Android.
Plugin version:

@capacitor-community/bluetooth-le: 0.4.0

Smartphone (please complete the following information):

Device: OnePlus 8
OS: Android 11

Logs from Android Studio
D/BluetoothGatt: setCharacteristicNotification() - uuid: 6e400003-b5a3-f393-e0a9-e50e24dcca9e enable: true
D/Device: resolve: setNotifications|6e400001-b5a3-f393-e0a9-e50e24dcca9e|6e400003-b5a3-f393-e0a9-e50e24dcca9e Notification enabled.
V/Capacitor/Plugin: To native (Capacitor plugin): callbackId: 23593037, pluginId: BluetoothLe, methodName: write
V/Capacitor: callback: 23593037, pluginId: BluetoothLe, methodName: write, methodData: {"deviceId":"E9:4C:F8:D1:E2:6C","service":"6e400001-b5a3-f393-e0A9-e50e24dcca9e","characteristic":"6e400002-b5a3-f393-e0A9-e50e24dcca9e","value":"fd 7b 22 43 61 65 73 63 22 3a 7b 22 69 76 22 3a 22 44 44 30 43 43 42 39 46 33 37 38 30 39 42 34 44 38 30 30 41 37 34 41 38 38 35 38 37 34 41 31 42 22 2c 22 6b 65 79 49 6e 64 65 78 22 3a 31 7d 7d fe"}
D/Device: resolve: write|6e400001-b5a3-f393-e0a9-e50e24dcca9e|6e400002-b5a3-f393-e0a9-e50e24dcca9e Characteristic successfully written.
V/Capacitor/BluetoothLe: Notifying listeners for event notification|E9:4C:F8:D1:E2:6C|6e400001-b5a3-f393-e0a9-e50e24dcca9e|6e400003-b5a3-f393-e0a9-e50e24dcca9e
D/Capacitor/BluetoothLe: No listeners found for event notification|E9:4C:F8:D1:E2:6C|6e400001-b5a3-f393-e0a9-e50e24dcca9e|6e400003-b5a3-f393-e0a9-e50e24dcca9e

Thanks for using this plugin. Unfortunately, I cannot reproduce this issue. Could you show the code where you are doing the startNotifications and write? Are you using BleClient as mentioned in the README? Does it work on iOS or web?

Yes sure. i'll share it with you.
I imported the plugin as mentioned:
import {BleClient, numbersToDataView, numberToUUID, textToDataView} from '@capacitor-community/bluetooth-le';
starting notifications:
`
async main(): Promise {
try {
await BleClient.initialize();
      this.BleTestDevice = await BleClient.requestDevice(
          {
            name: 'AXASWODEMO'
          });

        await BleClient.connect(this.BleTestDevice.deviceId);

        console.log('connected to device', this.BleTestDevice);

        await BleClient.startEnabledNotifications(
            value => {
                console.log(value)
            },
        );

        await BleClient.startNotifications(
            this.BleTestDevice.deviceId,
            TEST_SERVICE,
            TX_CHARACTERISTIC,
            value => {
                this.parseHeartRate(value)
            }
        ).then(value => {
            console.log('Listening to notfications')
        });

    } catch (error) {
        console.error(error);
    }
}`

For writing i use a button with the following function:
`    async write(): Promise {
try {
await BleClient.initialize();
        const caesc = '{"Caesc":{"iv":"DD0CCB9F37809B4D800A74A885874A1B","keyIndex":1}}';
        await BleClient.write(
            this.BleTestDevice.deviceId,
            TEST_SERVICE,
            RX_CHARACTERISTIC,
            textToDataView(this.hex_to_ascii(START_BYTE) + caesc + this.hex_to_ascii(END_BYTE))
        );
        console.log('Caesc is geschreven naar device');
    } catch (e) {
        console.log(e)
    }
}`

I havent tried it on web or ios.

Thanks for the details. There have been a few fixes n v0.5.0 (for Capacitor v2) and v1.0.0-2 (for Capacitor v3).
Could you try with the latest version?

Hi, I updated the version to v.0.5.1 but still experienced the same problem

Unfortunately I still cannot reproduce this. What's your @capacitor/core and @capacitor/android version?
Could you add the logs from Android Studio when running this with v0.5.1?

@pwespi Im actually having an issue with this too - Im either not understanding the docs correctly or Im missing something.
Im subscribing to the same Nordic NUS characteristic as @swendleysprott trying to listen for the data like this:
  async connectToDevice(deviceId) {
    console.log("Stop scanning first...")
    await this.BluetoothStopScanning();

    // this will check whether we are already connected or not and is we are, disconnect first
    await this.BluetoothDisconnect();

    console.log("Connecting to: " + deviceId);
    await BleClient.connect(deviceId);
    console.log("connected to device");
    this.setState({connectedToDeviceId: deviceId});

    await BleClient.startNotifications(deviceId, NUS_SERVICE_UUID, NUS_TX_CHARACTERISTIC, value => {
      console.log('received data from NUS: ' + value);
    });
    console.log("subscribed to characteristic");
  }

What Im seeing in logcat is this:
2021-07-30 00:15:26.566 20712-25608/com.example.app V/Capacitor/BluetoothLe: Notifying listeners for event notification|F6:5F:B4:EE:1A:B3|6e400001-b5a3-f393-e0a9-e50e24dcca9e|6e400003-b5a3-f393-e0a9-e50e24dcca9e
2021-07-30 00:15:26.566 20712-25608/com.example.app D/Capacitor/BluetoothLe: No listeners found for event notification|F6:5F:B4:EE:1A:B3|6e400001-b5a3-f393-e0a9-e50e24dcca9e|6e400003-b5a3-f393-e0a9-e50e24dcca9e

Which seems like im correctly subscribed to the charactersitic, but Im not seeing the data in console.log (via Chrome DevTools connected directly to Android 11)
So my question would be - is there a reason the subscribed data are not passed to the startNotifications() value handler? Am I understanding the docs correctly here?

@pwespi @hecko
I am too having the same issue.
2021-08-03 22:59:16.775 12580-12719/io.ionic.starter V/Capacitor: callback: 14554192, pluginId: BluetoothLe, methodName: startEnabledNotifications, methodData: {}
2021-08-03 22:59:16.779 12580-12580/io.ionic.starter I/Capacitor/Console: File: http://localhost/ - Line 213 - Msg: undefined
2021-08-03 22:59:16.779 12580-12580/io.ionic.starter I/Capacitor/Console: File: http://localhost/js/chunk-69312c9a.3b039f35.js - Line 3 - Msg: enable notification on:E8:EB:1B:53:D0:99
2021-08-03 22:59:16.780 12580-12719/io.ionic.starter V/Capacitor/Plugin: To native (Capacitor plugin): callbackId: 14554193, pluginId: BluetoothLe, methodName: addListener
2021-08-03 22:59:16.780 12580-12719/io.ionic.starter V/Capacitor: callback: 14554193, pluginId: BluetoothLe, methodName: addListener, methodData: {"eventName":"notification|E8:EB:1B:53:D0:99|49535343-FE7D-4AE5-8FA9-9FAFD205E455|49535343-1E4D-4BD9-BA61-23C647249616"}
2021-08-03 22:59:16.782 12580-12719/io.ionic.starter V/Capacitor/Plugin: To native (Capacitor plugin): callbackId: 14554194, pluginId: BluetoothLe, methodName: startNotifications
2021-08-03 22:59:16.782 12580-12719/io.ionic.starter V/Capacitor: callback: 14554194, pluginId: BluetoothLe, methodName: startNotifications, methodData: {"deviceId":"E8:EB:1B:53:D0:99","service":"49535343-FE7D-4AE5-8FA9-9FAFD205E455","characteristic":"49535343-1E4D-4BD9-BA61-23C647249616"}
2021-08-03 22:59:16.783 12580-12662/io.ionic.starter D/BluetoothGatt: setCharacteristicNotification() - uuid: 49535343-1e4d-4bd9-ba61-23c647249616 enable: true
2021-08-03 22:59:16.802 12580-12593/io.ionic.starter D/Device: resolve: setNotifications|49535343-fe7d-4ae5-8fa9-9fafd205e455|49535343-1e4d-4bd9-ba61-23c647249616 Setting notification succeeded.
2021-08-03 22:59:16.804 12580-12580/io.ionic.starter I/Capacitor/Console: File: http://localhost/ - Line 213 - Msg: undefined
2021-08-03 22:59:16.822 12580-12593/io.ionic.starter V/Capacitor/BluetoothLe: Notifying listeners for event notification|E8:EB:1B:53:D0:99|49535343-fe7d-4ae5-8fa9-9fafd205e455|49535343-1e4d-4bd9-ba61-23c647249616
2021-08-03 22:59:16.822 12580-12593/io.ionic.starter D/Capacitor/BluetoothLe: No listeners found for event notification|E8:EB:1B:53:D0:99|49535343-fe7d-4ae5-8fa9-9fafd205e455|49535343-1e4d-4bd9-ba61-23c647249616
2021-08-03 22:59:16.865 12580-12593/io.ionic.starter D/BluetoothGatt: onConnectionUpdated() - Device=E8:EB:1B:53:D0:99 interval=36 latency=0 timeout=500 status=0
2021-08-03 22:59:17.330 12580-12593/io.ionic.starter D/BluetoothGatt: onConnectionUpdated() - Device=E8:EB:1B:53:D0:99 interval=24 latency=0 timeout=512 status=0
2021-08-03 22:59:17.846 12580-12593/io.ionic.starter V/Capacitor/BluetoothLe: Notifying listeners for event notification|E8:EB:1B:53:D0:99|49535343-fe7d-4ae5-8fa9-9fafd205e455|49535343-1e4d-4bd9-ba61-23c647249616
2021-08-03 22:59:17.846 12580-12593/io.ionic.starter D/Capacitor/BluetoothLe: No listeners found for event notification|E8:EB:1B:53:D0:99|49535343-fe7d-4ae5-8fa9-9fafd205e455|49535343-1e4d-4bd9-ba61-23c647249616
2021-08-03 22:59:18.881 12580-12593/io.ionic.starter V/Capacitor/BluetoothLe: Notifying listeners for event notification|E8:EB:1B:53:D0:99|49535343-fe7d-4ae5-8fa9-9fafd205e455|49535343-1e4d-4bd9-ba61-23c647249616
2021-08-03 22:59:18.882 12580-12593/io.ionic.starter D/Capacitor/BluetoothLe: No listeners found for event notification|E8:EB:1B:53:D0:99|49535343-fe7d-4ae5-8fa9-9fafd205e455|49535343-1e4d-4bd9-ba61-23c647249616
2021-08-03 22:59:19.870 12580-12593/io.ionic.starter V/Capacitor/BluetoothLe: Notifying listeners for event notification|E8:EB:1B:53:D0:99|49535343-fe7d-4ae5-8fa9-9fafd205e455|49535343-1e4d-4bd9-ba61-23c647249616
2021-08-03 22:59:19.871 12580-12593/io.ionic.starter D/Capacitor/BluetoothLe: No listeners found for event notification|E8:EB:1B:53:D0:99|49535343-fe7d-4ae5-8fa9-9fafd205e455|49535343-1e4d-4bd9-ba61-23c647249616

@pwespi
I think I found the issue.
It is because I use the UPPER CASE for UUID. But the listener list key for the callback is lower case.
@Fllynn @hecko
The current workaround is to use lower case only in UUID.

Well it has replaced the undefined with

2021-08-05 15:23:41.306 22984-27937/com.example.app V/Capacitor/Plugin: To native (Capacitor plugin): callbackId: 120255447, pluginId: BluetoothLe, methodName: addListener
2021-08-05 15:23:41.306 22984-27937/com.example.app V/Capacitor: callback: 120255447, pluginId: BluetoothLe, methodName: addListener, methodData: {"eventName":"notification|F4:4B:64:42:E7:6A|6E400001-B5A3-F393-E0A9-E50E24DCCA9E|6e400001-b5a3-f393-e0a9-e50e24dcca9e"}
2021-08-05 15:23:41.315 22984-27937/com.example.app V/Capacitor/Plugin: To native (Capacitor plugin): callbackId: 120255448, pluginId: BluetoothLe, methodName: startNotifications
2021-08-05 15:23:41.316 22984-27937/com.example.app V/Capacitor: callback: 120255448, pluginId: BluetoothLe, methodName: startNotifications, methodData: {"deviceId":"F4:4B:64:42:E7:6A","service":"6E400001-B5A3-F393-E0A9-E50E24DCCA9E","characteristic":"6e400001-b5a3-f393-e0a9-e50e24dcca9e"}
2021-08-05 15:23:41.320 22984-28093/com.example.app D/BluetoothGatt: onConnectionUpdated() - Device=F4:4B:64:42:E7:6A interval=36 latency=0 timeout=500 status=0
2021-08-05 15:23:41.322 22984-27858/com.example.app D/Device: reject: setNotifications|6e400001-b5a3-f393-e0a9-e50e24dcca9e|6e400001-b5a3-f393-e0a9-e50e24dcca9e Characteristic not found.
2021-08-05 15:23:41.323 22984-27858/com.example.app D/Capacitor: Sending plugin error: {"save":false,"callbackId":"120255448","pluginId":"BluetoothLe","methodName":"startNotifications","success":false,"error":{"message":"Characteristic not found."}}
2021-08-05 15:23:41.337 22984-22984/com.example.app E/Capacitor/Console: File: http://localhost/ - Line 426 - Msg: Uncaught (in promise) Error: Characteristic not found.
2021-08-05 15:23:42.705 22984-27937/com.example.app V/Capacitor/Plugin: To native (Capacitor plugin): callbackId: 120255449, pluginId: BluetoothLe, methodName: removeListener
2021-08-05 15:23:42.705 22984-27937/com.example.app V/Capacitor: callback: 120255449, pluginId: BluetoothLe, methodName: removeListener, methodData: {"eventName":"onScanResult","callbackId":"120255444"}
2021-08-05 15:23:42.713 22984-27937/com.example.app V/Capacitor/Plugin: To native (Capacitor plugin): callbackId: 120255450, pluginId: BluetoothLe, methodName: stopLEScan
2021-08-05 15:23:42.714 22984-27937/com.example.app V/Capacitor: callback: 120255450, pluginId: BluetoothLe, methodName: stopLEScan, methodData: {}

@Fllynn The log shows your notification is rejected, I think you need to check whether your characteristic UUID is correct. I believe at least android support both upper and lower cases UUID.
They are different from your previous log.
New: 6e400001-b5a3-f393-e0a9-e50e24dcca9e|6e400001-b5a3-f393-e0a9-e50e24dcca9e
Old : 6e400001-b5a3-f393-e0a9-e50e24dcca9e|6e400003-b5a3-f393-e0a9-e50e24dcca9e
BTW, your service UUID is still in Capital.

Even with the right UUID and lower cases same error
2021-08-05 16:14:18.378 6099-9286/com.example.app V/Capacitor/Plugin: To native (Capacitor plugin): callbackId: 93262798, pluginId: BluetoothLe, methodName: addListener
2021-08-05 16:14:18.378 6099-9286/com.example.app V/Capacitor: callback: 93262798, pluginId: BluetoothLe, methodName: addListener, methodData: {"eventName":"notification|F4:4B:64:42:E7:6A|6e400001-b5a3-f393-e0a9-e50e24dcca9e|6e400001-b5a3-f393-e0a9-e50e24dcca9e"}
2021-08-05 16:14:18.385 6099-9286/com.example.app V/Capacitor/Plugin: To native (Capacitor plugin): callbackId: 93262799, pluginId: BluetoothLe, methodName: startNotifications
2021-08-05 16:14:18.386 6099-9286/com.example.app V/Capacitor: callback: 93262799, pluginId: BluetoothLe, methodName: startNotifications, methodData: {"deviceId":"F4:4B:64:42:E7:6A","service":"6e400001-b5a3-f393-e0a9-e50e24dcca9e","characteristic":"6e400001-b5a3-f393-e0a9-e50e24dcca9e"}
2021-08-05 16:14:18.388 6099-9253/com.example.app D/Device: reject: setNotifications|6e400001-b5a3-f393-e0a9-e50e24dcca9e|6e400001-b5a3-f393-e0a9-e50e24dcca9e Characteristic not found.
2021-08-05 16:14:18.389 6099-9926/com.example.app D/BluetoothGatt: onConnectionUpdated() - Device=F4:4B:64:42:E7:6A interval=60 latency=0 timeout=400 status=0
2021-08-05 16:14:18.389 6099-9253/com.example.app D/Capacitor: Sending plugin error: {"save":false,"callbackId":"93262799","pluginId":"BluetoothLe","methodName":"startNotifications","success":false,"error":{"message":"Characteristic not found."}}
2021-08-05 16:14:18.399 6099-6099/com.example.app E/Capacitor/Console: File: http://localhost/ - Line 426 - Msg: Uncaught (in promise) Error: Characteristic not found.

@Fllynn  no I meant you used the service UUID as the characteristic. which results in "characteristic not found"
6e400001-b5a3-f393-e0a9-e50e24dcca9e -> service
6e400002-b5a3-f393-e0a9-e50e24dcca9e -> TX characteristic
6e400003-b5a3-f393-e0a9-e50e24dcca9e -> RX characteristic

Okay, so with the fixed UUIDs still the callback is undefined
2021-08-05 22:06:40.376 5268-9692/com.example.app V/Capacitor: callback: 114486288, pluginId: BluetoothLe, methodName: startNotifications, methodData: {"deviceId":"F4:4B:64:42:E7:6A","service":"6e400001-b5a3-f393-e0a9-e50e24dcca9e","characteristic":"6e400003-b5a3-f393-e0a9-e50e24dcca9e"}
2021-08-05 22:06:40.379 5268-9607/com.example.app D/BluetoothGatt: setCharacteristicNotification() - uuid: 6e400003-b5a3-f393-e0a9-e50e24dcca9e enable: true
2021-08-05 22:06:40.387 5268-9943/com.example.app D/BluetoothGatt: onConnectionUpdated() - Device=F4:4B:64:42:E7:6A interval=36 latency=0 timeout=500 status=0
2021-08-05 22:06:40.433 5268-9943/com.example.app D/Device: resolve: setNotifications|6e400001-b5a3-f393-e0a9-e50e24dcca9e|6e400003-b5a3-f393-e0a9-e50e24dcca9e Setting notification succeeded.
2021-08-05 22:06:40.438 5268-5268/com.example.app I/Capacitor/Console: File: http://localhost/ - Line 213 - Msg: undefined

@majianjia you suggestion to change UUIDs to lowercase seems to work around the issue in my case. Ill investigate a bit more and get back to this thread in a few hours. Good catch!

@majianjia I can actually confirm this being the case. the UUIDs need to be lowercase for notifications to work properly.
In my other Java code I actually have this:
import java.util.UUID;

...

private static final UUID UART_TX_CHARACTERISTIC_UUID =
      UUID.fromString("6E400003-B5A3-F393-E0A9-E50E24DCCA9E");

...

mTXCharacteristic = service.getCharacteristic(UART_TX_CHARACTERISTIC_UUID);

...

          setNotificationCallback(mTXCharacteristic)
              .with(
                  (device, data) -> {
                    final String text = data.getStringValue(0);
                    log(LogContract.Log.Level.APPLICATION, "rx: \"" + text + "\"\n");
                    mCallbacks.onDataReceived(device, text);
                  });
          requestMtu(260).enqueue();
          enableNotifications(mTXCharacteristic).enqueue();

which works and might help improving this codes Kotlin implementation

Hi all
Thank you all for reporting and  debugging this issue. I can confirm that using uppercase UUIDs caused this behavior. I just released v1.1.3 which allows uppercase as well as lowercase UUIDs and should solve this.
@Fllynn In case v1.1.3 does not solve your problem, please open a new issue.
Thanks again.
PS: it was actually documented in the README that the plugin expects lowercase UUIDS, but it was easy to miss and the plugin should have handled this better.

@pwespi Yes indeed, it is on the front page of the document. Just saw that this afternoon.
Great job on making this plugin, appreciated it.


Adjust find_packages to make work in a wheel.

Prior to this change, setup.py bdist_wheel does not include
mkcodes.py, and this mechanism is used by pip install .
Also add gitignore to make development a little cleaner.

I think it's because this is a "module" and not a "package" which would be a directory with __init__.py in it.  I didn't look too much into terminology, but this was enough to get it working.  Ref https://www.python.org/dev/peps/pep-0561/#packaging-type-information

This PEP does not support distributing typing information as part of module-only distributions. The code should be refactored into a package-based distribution and indicate that the package supports typing as described above.

Makes sense to me. I found this documentation more clear:

If your project contains any single-file Python modules that aren’t part of a package, set py_modules to a list of the names of the modules (minus the .py extension) in order to make setuptools aware of them.


bump to latest sync plugin version

waiting on new version of plugin to show up in https://updates.jenkins-ci.org/download/plugins/openshift-sync/

[test]

@bparees This is not reflected in the latest image! (Just in dev image; I though that one was deprecated?).
$ docker pull openshift/jenkins-1-centos7:dev && docker run -it --rm -u 0 openshift/jenkins-1-centos7:dev /usr/bin/grep 'openshift-sync' /opt/openshift/base-plugins.txt
openshift-sync:0.0.12

$ docker pull openshift/jenkins-1-centos7:latest && docker run -it --rm -u 0 openshift/jenkins-1-centos7:latest /usr/bin/grep 'openshift-sync' /opt/openshift/base-plugins.txt
openshift-sync:0.0.11

@tnozicka our image build infrastructure is having some issues, the new image hasn't been built/published yet.  hopefully soon.

@bparees  ok, I will wait :)
(I just wanted to make sure you know about it.)

@tnozicka try now (centos image)

@bparees thanks, it works now


color disappears when piping

Searching for word 'title' in my source code.
pt --color title | less
or
pt --color --file-search-regexp='\.go$'  | less
The color does not appear.  Color appears when not piped. It isn't less, it's the pipe.  Color is also gone in the case of:
pt --color title | cat
--
Description:    Ubuntu 14.04 LTS
Codename:       trusty
Linux sda8 3.13.0-29-generic #53-Ubuntu SMP Wed Jun 4 21:00:20 UTC 2014 x86_64 GNU/Linux
jessie/sid
$ pt --version
1.6.2

Afaict this its broken in 2.1.0, making it impossible to get colored, grouped, paged output. It'd be nice for either explicit --color to force color on even when piped, or add a force color option.

ag has a --pager option, so we can use ag --pager=less KEYWORD.


feat: add option for keda addon container-service

Description
This is adding in the option to enable Keda on an AKS cluster.
I would probably do this one after https://github.com/Azure/bicep-registry-modules/pull/1682 as i used my main branch that has the cost anlayasis changes in it.

Pipeline Reference

Pipeline

Type of Change

[ ] Update to CI Environment or utlities (Non-module effecting changes)
[X] Azure Verified Module updates:

[ ] Bugfix containing backwards compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json:

[ ] Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
[ ] The bug was found by the module author, and no one has opened an issue to report it yet.

[ ] Feature update backwards compatible feature updates, and I have bumped the MINOR version in version.json.
[ ] Breaking changes and I have bumped the MAJOR version in version.json.
[ ] Update to documentation

Checklist

[X] I'm sure there are no other open Pull Requests for the same update/change
[X] I have run Set-AVMModule locally to generate the supporting module files.
[X] My corresponding pipelines / checks run clean and green without any errors or warnings

@PixelRobots The PR diff is showing changes from your cost-analysis PR as well. Can you please clean this up?

That should be resolved now.

@PixelRobots code looks good now. Just need to get a green pipeline and we can merge this.

It's a strange one. One of the jobs keeps failing. All others are ok. I think it might be something I am doing but not an issue with the bicep.
Any ideas?

And we have green! Seems it was an Azure region and it was being slow or something.


Not working on sketch 52

@lightsbee this is a known issue, thank you for reporting - https://github.com/sonburn/symbol-organizer/issues/91


[RFC] Optimize configurations in Learner.

Currently the configuration steps in Learner is bloated in both size and complexity.  I think we have all come to agreement that it needs better organization.  Other than simplifying future maintenance, there are other benefits on this:

Parameter validation, the configuration happens in both initialization and during each update so copies of parameters are stored in various places, including both Learner and GradientBooster.  So we don't know exactly when a parameter should be used nor which copy is actually used.

Smooth out the path for our plan of transitioning to JSON format since one big issue is keeping compatibility, and I want the JSON format to be a fresh start.

To do so, we need to draft a testing suit for current binary model IO so that later refactoring won't break the existing model.  Then we need to decide what style of parameters management we want:

A global one like most of other gbm libraries, all parameters stored in one config class, then we pass it through the whole library.
Many local parameter structs just like current XGBoost.

Personally I don't have a preference.
I am struggling whether should JSON happen first or optimization for Learner.  But parameter validation is definitely coming after we reorganize those configurations.
@RAMitchell @CodingCat @hcho3

One funny thing, I tried to move tree updater configuration into GBM, the Learner.SLOW_CheckMultiBatch test gave me a dead lock in col_maker.

During many previous fixes and refactoring PRs including #4553,  I came to realize that how should we proceed on parameter validations.  There are two fundamental problems in current XGBoost internal parameters handling:

The semantic of Configure and Init is not clear, and they depend on each others.  In many places one has to allocate memory for components (Init) to do further configurations, example are gbm asking updaters to configure themselves.  But it's also not possible to do Init without first configuring stuffs out, example like creating updaters after selecting tree methods.
Some configurations has to be done after seeing the DMatrix, like num_feature, n_gpus(to be deprecated).  While our abstract Configure is not prepared for this, it has no parameter for DMatrix.
Most of the configurations can be changed during each boost iteration.  Our configuration is also not prepared for this.  It doesn't know whether those string parameters are changed or not, so we just plug some partial configuration into Update or DoBoost.

My proposal is,

Firstly we clear up the semantic of Init and Configure.  There should be no public Init method.  All initialization should be a part of Configure.  And the configuration should be capable of detecting whether parameters has changed, like comparing new and old dmlc::Parameter.
List all parameters that are dependent on DMatrix, then move them into DMatrix or just deprecate them.  Algorithms should not be dependent on those data distributions parameters.  Since n_gpus is about to be deprecated by @RAMitchell .  I think this is doable.  One particular exception is Predictor, we can just allocate both cpu_predictor and gpu_predictor, and choose to use one of them during iteration without deleting the other.
Configure for every iteration.

Clearing up config/init methods is a solid plan. These methods are often considered a 'code smell' and mean that we can't be sure if an object is safe to use or what state it might be in.

Another thing that may help improve clarity is parameter structs being immutable. It's very hard to tell what's going on if they read state in from external parameter dicts then change the value. Some examples: num_class, updater.

@RAMitchell

parameter structs being immutable.

I totally agree.  But let me get configuration right first.  Fixing its bugs is exhausting and wasting time.

Will leave further development to parameters validation.


How coercion works behind the scenes

Coercion and Abstract operations in Javascript.
Proposed title of the article
How coercion works behind the scenes
Proposed article introduction
You must have experienced a case where people are trying to bash Javascript by showing how bizarre it is while surfing the internet. This is one of the cases that you must have found :
console.log(""==[])
// this returns true

And you see, "Weird" is a short-lived term unless you comprehend how it's actually working. This is one of the cases where if you don't understand coercion, this will seem a like a bug in JS. Therefore, understanding coercion in JavaScript properly is important because not only it will help new developers to write better and understandable code, but it will also help other developers to avoid bugs that can occur due to not understanding coercion.
In this article, I will explain how coercion works in JavaScript and how it works behind the scenes. We will also have a look at abstract operations, which is the core concept behind coercion.
Key takeaways
At the end of this article, readers would be able to:

Readers will know what coercion in Javascript is.
Reader will know what abstract operations are.
Reader will know how coercion works under the hood.

Conclusion
In this article, we leaned what coercion is and how it works under the hood using abstract operations. We also saw implicit versus explicit typecasting and learned about the concept of boxing in Javascript.
Javascript being a beginner-friendly language comes with its caveat around typecasting, therefore it is important to know all these concepts to avoid these caveats as this will help you avoid bugs in your code and your coercion to your advantage instead of avoid it.

@ravi-ranjan-singh still working on this?

@WanjaMIKE I have the draft nearly ready but I saw my topic wasn't approved.