chore(deps): update renovate/renovate docker tag to v36.107.1

This PR contains the following updates:

Package
Update
Change

renovate/renovate
minor
36.100.0 -> 36.107.1

Release Notes

renovatebot/renovate (renovate/renovate)
v36.107.1
Compare Source
Bug Fixes

data: automatic update of static data (#24627) (0d38ef1)

Documentation

typo: update upgrade-best-practices.md (#24630) (c767e74)

Miscellaneous Chores

deps: update pnpm to v8.7.6 (#24633) (2c733ea)

v36.107.0
Compare Source
Features

migrate: regexManagers -> customManagers (#24451) (6f9d37a)

Miscellaneous Chores

deps: update dependency @types/diff to v5.0.4 (#24617) (4761306)

v36.106.2
Compare Source
Bug Fixes

regEx definitions that used regex literal and separate flags (#24626) (6759c6f)

Miscellaneous Chores

deps: update dependency graphql to v16.8.1 (#24624) (4a7ece1)
deps: update dependency markdownlint-cli2 to v0.10.0 (#24628) (71a86f2)
rename fixture package file and lock files so they do not trigger osv scanner (#24625) (e4e3b9b)

v36.106.1
Compare Source
Build System

deps: update dependency @renovatebot/ruby-semver to v3.0.15 (#24620) (d484df8)

v36.105.0
Compare Source
Features

override docker and helm package metadata  (#19901) (3092e42)

v36.103.0
Compare Source
Features

bun: support multiple lock files (#24594) (bb6bc35)

Documentation

add security preset page and opentelemetry to sidebar (#24592) (cdb2636)

v36.102.0
Compare Source
Features

datasource/docker: Add support for Google Application Default Credentials (#23903) (37a0bcf)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

:tada: This PR is included in version 5.2.6 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Docker images tagged "nightly" are not actually built nightly

📑 I have found these related issues/pull requests
n/a
🛡 Security Policy

[X] I agree to have read this project Security Policy

Description
From https://hub.docker.com/r/louislam/uptime-kuma/tags:

Is this expected?
👟 Reproduction steps
See https://hub.docker.com/r/louislam/uptime-kuma/tags
👀 Expected behavior
Docker images tagged nightly are built and published nightly.
😓 Actual Behavior
Docker images tagged nightly haven't been published in a couple months.
🐻 Uptime-Kuma Version
n/a
💻 Operating System and Arch
n/a
🌐 Browser
n/a
🖥 Deployment Environment

Runtime: n/a
Database: n/a
Filesystem used to store the database on: n/a
number of monitors: n/a

📝 Relevant log output
No response

nigtly is not a supported docker tag
See https://github.com/louislam/uptime-kuma/security/policy


Delay of execution [cover]

last working Home Assistant release (if known):
0.94.3
Operating environment (Hass.io/Docker/Windows/etc.):
raspberry pi 3 ->Hass.io
Component/platform:
Configuration
Description of problem:
I have made  covers like this
 platform: template
    covers:
      volet_iban:
        friendly_name: "Volet Iban"
        open_cover:
          - service: switch.turn_off
            entity_id: switch.fermer_volet_iban
          - service: switch.turn_on
            entity_id: switch.ouvrir_volet_iban
          - delay: '30'
          - service: switch.turn_off
            entity_id: switch.ouvrir_volet_iban
        close_cover:
          - service: switch.turn_off
            entity_id: switch.ouvrir_volet_iban
          - service: switch.turn_on
            entity_id: switch.fermer_volet_iban
          - delay: '30'
          - service: switch.turn_off
            entity_id: switch.fermer_volet_iban
        stop_cover:
          service: switch.turn_off
          entity_id: switch.ouvrir_volet_iban, switch.fermer_volet_iban

If i use the switch i've got the device responding directly, if i use the cover i've got 4-5 seconds delay.

Is this still an issue?


[fix] sys trusted_url_validator_spec

Coverage remained the same at 81.404% when pulling 36127d4ca49bc0b4809c50a9bfd09ca9aeff21a7 on fix-trusted-url-validator-spec into f2aefc4adba7b561301ced3c6eba846b025bab60 on master.


Misc. examples updates

README.md add blurb about building examples
fmt/clippy/small change to examples/simple.rs

Looks like there are some compilation failures. I think those types are platform dependent.

Looks like there are some compilation failures. I think those types are platform dependent.

I'll take a look at it next week, thanks!

Looks like there are some compilation failures. I think those types are platform dependent.

I'll take a look at it next week, thanks!

Sorry I didn't get back to this, I'm dealing with an illness.

No worries. I think I made most of the same changes in my PR, but if I missed anything feel free to send a new PR. I hope you feel better soon!


nsp fails to report advisory 31 for semver 2.3.2

I created a small test project with the following package.json
{
"name": "test_nsp",
"version": "0.0.0",
"description": "",
"dependencies": {},
"devDependencies": {
"grunt": "^0.4.5",
"grunt-cli": "^0.1.13",
"grunt-nsp": "~2.1.2",
"semver": "^2.3"
}
}
I am using node version v0.10.40 and npm version 3.3.9.  npm install puts version 2.3.2 of semver in node_modules/semver, which should be covered by advisory #31.  I ran npm shrinkwrap -dev, and confirmed that 2.3.2 of semver appears in npm-shrinkwrap.json.  However, grunt nsp reports "No known vulnerabilities found" as does "node_modules/grunt-nsp/node_modules/nsp/bin/nsp check ./npm-shrinkwrap.json".

@plynchnlm can you email<EMAIL_ADDRESS>both your package and shrinkwrap and I can create test cases and figure out what is going on?

@plynchnlm no need I think I can replicate

@plynchnlm this is a valid bug. Not sure quite what is going on but were looking into it.

Thanks!  I'm glad to here it is not just me.

Ok, I'm pretty sure this has to do with dev dependencies not being checked. when i move it from dev to regular dep it finds it jsut fine. Now I'm checking with shrinkwrap

I'm not sure how to interpret your statement, but IMO dev dependencies should be checked, because they are often involved in building files/packages used in production, even if they are not used directly in production, so a flaw in a dev dependency could affect production.

yeah, we talked about it today and i think you're right. there's been enough confusion around the behavior regarding devDependencies that we'll be making changes to check them by default. give us a little time, but we'll get it going in the next couple of weeks.

Just adding a :+1: for checking dev deps. This was just confusing me ... I was getting a warning for a dep on one project, but not another, only diff being it was a dev dep in one, and not the other.
In my case it was missing vulnerabilities in a shrinkwrap file created with npm shrinkwrap --dev.

closing in favor of #87


configparser.NoSectionError: No section: 'BloodHound'

Describe the bug
CME not working with some commands
To Reproduce
Steps to reproduce the behavior:
run the command  cme smb <IP> -u <user> -p <pass> -M handlekatz
or cme smb <IP> -u <user> -p <pass> -M nanodump
Crackmapexec info

OS: Debian
Version of CME 5.2.2
Installed from pipx

Additional context
Note - Also tested on a different machine with python 3.8 installed
Traceback (most recent call last):
  File "/root/.local/bin/cme", line 8, in <module>
    sys.exit(main())
  File "/root/.local/pipx/venvs/crackmapexec/lib/python3.9/site-packages/cme/crackmapexec.py", line 254, in main
    asyncio.run(
  File "/usr/lib/python3.9/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/usr/lib/python3.9/asyncio/base_events.py", line 642, in run_until_complete
    return future.result()
  File "/root/.local/pipx/venvs/crackmapexec/lib/python3.9/site-packages/cme/crackmapexec.py", line 102, in start_threadpool
    await asyncio.gather(*jobs)
  File "/root/.local/pipx/venvs/crackmapexec/lib/python3.9/site-packages/cme/crackmapexec.py", line 68, in run_protocol
    await asyncio.wait_for(
  File "/usr/lib/python3.9/asyncio/tasks.py", line 442, in wait_for
    return await fut
  File "/usr/lib/python3.9/concurrent/futures/thread.py", line 52, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/root/.local/pipx/venvs/crackmapexec/lib/python3.9/site-packages/cme/protocols/smb.py", line 125, in __init__
    connection.__init__(self, args, db, host)
  File "/root/.local/pipx/venvs/crackmapexec/lib/python3.9/site-packages/cme/connection.py", line 62, in __init__
    self.proto_flow()
  File "/root/.local/pipx/venvs/crackmapexec/lib/python3.9/site-packages/cme/connection.py", line 98, in proto_flow
    if self.login() or (self.username == '' and self.password == ''):
  File "/root/.local/pipx/venvs/crackmapexec/lib/python3.9/site-packages/cme/connection.py", line 275, in login
    if self.plaintext_login(self.domain, user, password): return True
  File "/root/.local/pipx/venvs/crackmapexec/lib/python3.9/site-packages/cme/protocols/smb.py", line 366, in plaintext_login
    add_user_bh(self.username, self.domain, self.logger, self.config)
  File "/root/.local/pipx/venvs/crackmapexec/lib/python3.9/site-packages/cme/helpers/bloodhound.py", line 7, in add_user_bh
    if config.get('BloodHound', 'bh_enabled') != "False":
  File "/usr/lib/python3.9/configparser.py", line 781, in get
    d = self._unify_values(section, vars)
  File "/usr/lib/python3.9/configparser.py", line 1149, in _unify_values
    raise NoSectionError(section) from None
configparser.NoSectionError: No section: 'BloodHound'

A workaround is to disable Bloodhound by adding the following to the ~/.cme/cme.conf configuration file:
[BloodHound]
bh_enabled = False

A workaround is to disable Bloodhound by adding the following to the ~/.cme/cme.conf configuration file:
[BloodHound]
bh_enabled = False

Edit: Apparently by upgrading crackmapexec using pip, the configuration file cme.conf remains the same and needs to modified manually

I've tried to do that and now it says:
    sys.exit(main())
  File "/root/.local/pipx/venvs/crackmapexec/lib/python3.9/site-packages/cme/crackmapexec.py", line 254, in main
    asyncio.run(
  File "/usr/lib/python3.9/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/usr/lib/python3.9/asyncio/base_events.py", line 642, in run_until_complete
    return future.result()
  File "/root/.local/pipx/venvs/crackmapexec/lib/python3.9/site-packages/cme/crackmapexec.py", line 102, in start_threadpool
    await asyncio.gather(*jobs)
  File "/root/.local/pipx/venvs/crackmapexec/lib/python3.9/site-packages/cme/crackmapexec.py", line 68, in run_protocol
    await asyncio.wait_for(
  File "/usr/lib/python3.9/asyncio/tasks.py", line 442, in wait_for
    return await fut
  File "/usr/lib/python3.9/concurrent/futures/thread.py", line 52, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/root/.local/pipx/venvs/crackmapexec/lib/python3.9/site-packages/cme/protocols/smb.py", line 125, in __init__
    connection.__init__(self, args, db, host)
  File "/root/.local/pipx/venvs/crackmapexec/lib/python3.9/site-packages/cme/connection.py", line 62, in __init__
    self.proto_flow()
  File "/root/.local/pipx/venvs/crackmapexec/lib/python3.9/site-packages/cme/connection.py", line 100, in proto_flow
    self.call_modules()
  File "/root/.local/pipx/venvs/crackmapexec/lib/python3.9/site-packages/cme/connection.py", line 132, in call_modules
    self.module.on_admin_login(context, self)
  File "/root/.local/pipx/venvs/crackmapexec/lib/python3.9/site-packages/cme/modules/nanodump.py", line 53, in on_admin_login
    with open(self.nano_path + self.nano, 'wb') as nano:
FileNotFoundError: [Errno 2] No such file or directory: '/tmp/shared/nano.exe'

It seems from the module's code that it tries to decode the base64 encoded binary and write it to disk but fails for some reason.

A workaround is to disable Bloodhound by adding the following to the ~/.cme/cme.conf configuration file:

[BloodHound]

bh_enabled = False

Edit: Apparently by upgrading crackmapexec using pip, the configuration file cme.conf remains the same and needs to modified manually

Thanks this workaround is good enough to solve :)

Remove the config file ~/.cme/cme.conf and then you are good to go

Just stopped by to thank you all for this post.  REALLY needed the new modules on a pentest and was running into this error.  Now I'm GTG.  Have a great weekend.

i disable bh_enabled = False
and same problem even remove cme.conf
same i cant solve this 10 days same in everything

Remove the config file ~/.cme/cme.conf and then you are good to go

Not working

Remove the config file ~/.cme/cme.conf and then you are good to go

Not working

This worked for me - THANKS!


Fix support for page listener

Facebook is now killing Messenger Platform for Pages!
Now apps want to be reviewed must be associated to a business. Sadly, they will ask you to provide documents to prove that your business is real and is making money.
With this change we can use this api instead of the dead one provided by Facebook.
Please merge this change, so the poor like me can have some hope.. and not something like Chatfuel which has a very limited range of features.

"Please merge" you don't have to ask us to merge... just make sure that everything works, and after some people verify it, we will merge
anyway... screw you Zuckerberg :/

@ravkr hey hey let’s not be aggressive here.
Seems good to me, I’m trusting you that this works.


sphinxsearch: Add support for MySQL & xmlpipe2

Motivation for this change
This PR obsoletes #79032.
I added support for MySQL sources, and optionally xmlpipe2. I don't use xmlpipe2 myself, so I didn't test that. I'll test MySQL source in the next few days in a small production environment. :)
The version and main source were pulled in as arguments? Not sure what that was about, overriding those should be done with overrideAttrs. I also enabled parallel builds (works fine for me), and added myself as maintainer. The 2.X branch of sphinxsearch isn't actively supported anymore though, so I don't expect any updates. The 3.X line isn't open source (yet), and has enough incompatibilities to go into its own sphinxsearch3 package anyways.
Things done

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

[x] NixOS

[ ] 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 nixpkgs-review --run "nixpkgs-review wip" no packages depend on this
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Closure size change (second is new):

/nix/store/id1lrdwxwwlndsb5w3hr4v69zlhg5i4l-sphinxsearch-2.2.11	  57.2M
/nix/store/6kdis2k84mfzdabmx5cfs6djy95zrk8g-sphinxsearch-2.2.11	  58.3M

Whoops. I built this against release-19.09 and it seems I didn't correctly rebase to master. will fix.

Ok, should work now. Closure sizes:

sphinxsearch on master
/nix/store/id1lrdwxwwlndsb5w3hr4v69zlhg5i4l-sphinxsearch-2.2.11	  57.2M
sphinxsearch on pr, enableMysql = false
/nix/store/8asbjxjxzbilhc25r1mj01zs2slhg71b-sphinxsearch-2.2.11	  57.2M
sphinxsearch on pr, enableMysql = true (default)
/nix/store/qjhabpy7dhvs8s3cmhzvzz2kkasvk6p2-sphinxsearch-2.2.11	  73.5M
sphinxsearch on pr, enableMysql = true, enableXmlpipe2 = true
/nix/store/z39nqslg1z21x3kc5smxcyqfqpp06ixk-sphinxsearch-2.2.11	  73.8M

Running in production, works just fine with a mysql source :+1:

Can I get a review on this one? :) @jtojnar @vvs-

Thanks for the review @jtojnar! Fixed and tested compilation in all build configurations.

@GrahamcOfBorg build sphinxsearch

Build failure on darwin :( seems like it's due to this upstream bug: http://sphinxsearch.com/bugs/view.php?id=2578
I imported a workaround for this particular issue from Homebrew:
https://github.com/Homebrew/homebrew-core/pull/8619/files#diff-df176d75fcc42a0ade34983e9ffc3193R47
I don't have a macOS to test on, unfortunately.

@GrahamcOfBorg build sphinxsearch

:tada:

Thanks.


Watchdog Rewrite to Support Python Interactive Shell

Start with the why:
Using the Python interactive shell is quite useful for discovering code-base functionality.  As it stands now, DepthAI cannot support the interactive shell because the watchdog detects the delay from the shell, assumes something has gone wrong, and reboots the DepthAI device.
Supporting the Python Interactive Shell will allow easier discoverability of DepthAI functionality.
Move to the how:
Figure out a way to configure and handle the watchdog so that python interactive shell can be used
https://luxonis.slack.com/archives/CG1PHMY6M/p1602495326208900, see idea https://luxonis.slack.com/archives/CG1PHMY6M/p1602508153217000
Move to the what:
Support the Python Interactive Shell in Gen2 Pipeline Builder (#136).

With Gen2 approach, this issue is now resolved. I also did quite a bit of tests with the interactive shell and it worked as intended.
I'll close this for now


Move system monitoring into background loop

At the moment the pi user is gathering alle the info for the display in a regular loop. This puts quite some load on the sytsem. The idea is to push that system monitoring loop into the background process that updates in different time intervals system values into the redis db. And the pi user running the infoscreen simply calls that values from the redis cache to display.
Having the pi user gathering the system infos with a lot of sudos creates lots of pam sessions & inflates the auth.log. This would be reduced because the background process is run by root and would not need sudo.

Thats what being prepared in the following branch:
https://github.com/rootzoll/raspiblitz/tree/2388redis

OK this tested goff now over RCs .. closing issue for release.


ingressgateway-certs related volume shall be removed from istio-ingressgateway

Bug description
from istio 1.6, the SDS is enabled by default for ingressgateway, but after installation, I still can find following mount point in istio-ingressgateway pod/deployment, but they are useless, user just need to add credentialName in Gateway to enable secret related info.
    - mountPath: /etc/istio/ingressgateway-ca-certs
      name: ingressgateway-ca-certs
      readOnly: true
    - mountPath: /var/run/secrets/kubernetes.io/serviceaccount
      name: istio-ingressgateway-service-account-token-jkwds
      readOnly: true

From output of "istioctl profile dump demo", I found the following secretVolumes section under values.istio-ingressgateway.
        secretVolumes:
        - mountPath: /etc/istio/ingressgateway-certs
          name: ingressgateway-certs
          secretName: istio-ingressgateway-certs
        - mountPath: /etc/istio/ingressgateway-ca-certs
          name: ingressgateway-ca-certs
          secretName: istio-ingressgateway-ca-certs

[ ] Docs
[x] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[x] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Expected behavior
secretVolume is not in ingress gateway deployment and pod
Steps to reproduce the bug
istioctl install --set profile=demo
check istio-ingressgateway deployment or pod yaml
Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
istio 1.6.8
How was Istio installed?
Environment where bug was observed (cloud vendor, OS, etc)

Although Gateway SDS was enabled by default from 1.6.0, File mounted gateway should continue to be available to help users to transition to secure gateway SDS.

There is a plan to use gateway yaml instead of ingress gateway deployment yaml and egress gateway depolyment yaml. And there is also a new file-based SDS feature. We could improve the file-mount approach on gateway for both ingress and egress gateways.

I think we should keep this. It doesn't hurt to have it, right? And allows people to continue to use mounted certs - which is supported

Closing per ^


Clean up and removal of double slashes

PR checklist

[ ] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs- [ ] If necessary, also make a PR on the nf-core/chipseq branch on the nf-core/test-datasets repository.
[x] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

Double slashes in directory urls were reported as bug in nf-core/atacseq https://github.com/nf-core/atacseq/issues/332


[AIRFLOW-620] Feature to tail custom number of logs instead of rendering whole log

Dear Airflow Maintainers,
This is a change in the way we render logs. This PR will give a dropdown of number of lines to be tailed with a refresh button. On refresh webserver fetch selected number of lines from local worker logs or remote worker logs. On can toggle this feature using config variable tail_logs in webserver, default number of lines to tail via config variable num_lines in webserver and items in dropdown can be controlled via config variable tail_lines_list in webserver in csv format
Please accept this PR that addresses the following issues:
https://issues.apache.org/jira/browse/AIRFLOW-620
Testing Done:
Manually tested on all major browsers.
This will add a dropdown of number of lines to be tailed and refresh button to get latest logs instead of refreshing the whole page.

Codecov Report

Merging #3992 into master will decrease coverage by 0.16%.
The diff coverage is 41.11%.

@@            Coverage Diff             @@
##           master    #3992      +/-   ##
==========================================
- Coverage   75.49%   75.32%   -0.17%
==========================================
  Files         199      199
  Lines       15947    16023      +76
==========================================
+ Hits        12039    12070      +31
- Misses       3908     3953      +45

Impacted Files
Coverage Δ

airflow/bin/cli.py
62.93% <0%> (-1.51%)
:arrow_down:

airflow/utils/log/file_task_handler.py
77.14% <33.33%> (-12.27%)
:arrow_down:

airflow/www/views.py
68.76% <70%> (-0.09%)
:arrow_down:

airflow/www_rbac/views.py
71.89% <70%> (-0.15%)
:arrow_down:

airflow/jobs.py
82.48% <0%> (+0.35%)
:arrow_up:

Continue to review full report at Codecov.

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

@phani8996 can you post some gifs on this feature in work?

@msumit I am not able to upload gif. I am attaching images for reference.

@msumit This isn't the equivalent of tail -f -- it doesn't stream the logs in as they happen, it's just tail -n 300 etc.

That's true sumit. This feature doesn't tail logs in fly mode. I am planning to add that in a separate PR as this brings a lot of changes to current log viewing mechanism and this maybe unstable and experimental.

Tests seems to be failing at places unrelated to my code changes. Do anyone has any idea what is causing and how to fix it?

@phani8996 Can you rebase onto master? To make sure that you're up to date.

@phani8996 Can you rebase onto master? To make sure that you're up to date.

It is upto date with master.


:sparkles: middleware support

#15

[ ] README.md update
[ ] Test with some external project
[ ] Publish

I delete codeclimate that is not binded to our linter

It will be published with #26


CEL expressions approver design

This PR contains an initial design proposal allowing policy authors to use CEL expressions to express policy rules. If this is ever implemented, it should address https://github.com/cert-manager/approver-policy/issues/62.
Also includes an initial design template copied from cert-manager.
/cc @inteon @SgtCoDFish

I have now updated the proposal and moved the previous proposal into the alternatives. The new proposal is now significantly closer to the upstream CEL validations rules API. Please take another look. I would love to see some progress on this design proposal so we eventually can move over to implementing it. 😸
/cc @jsoref @inteon @SgtCoDFish @wallrj

I have initiated the work on implementing this new feature: https://github.com/cert-manager/approver-policy/pull/277
Please take a look if you have time.

After working on improving the API docs and understanding more of the current API behavior, I have updated the proposed design (and examples with the following):

Note: To ensure we don't break behaviors in the existing API version, any validations can never allow a request otherwise denied by the values/value and required fields.

I think that is our safest option. The next API version could make this feature even more user-friendly. But I think it will be acceptable as proposed now.
I am almost ready to move over to the implementation of this, so I would prefer a more "formal" approval of this design proposal by merging this PR. 😉 PTAL!

Just wanted to add that the design I read just over 2 weeks ago looked great. It covered a few things I had not thought of and there was nothing I disagreed with. Now work has started, I will attempt to look over for understanding.
The only minor thing is that there are a few empty sections in the design doc, might it be worth either a quick comment of N/A if the section isn't applicable.

/retest

/unhold
/ok-to-test

/lgtm


Improve JSON response structure

Issue
Improve JSON response structure
Update the livenessImages field to use the proper JSON array format instead of a string representation.
Found here
https://github.com/smileidentity/flutter/blob/ea1bc1d44043d9f66df0c03d8a42aac9de97ce6b/ios/Classes/SmileIDSmartSelfieEnrollment.swift#L63
More Details:
The issue seems to be caused by the livenessImages field being a string representation of an array instead of a proper JSON array. Updating the server/API to return the livenessImages field as a JSON array should resolve the issue.
Expected Behavior:
The application should be able to parse the JSON response correctly and handle the livenessImages field without crashing.
Environment:

Device: iPhone 11 (iOS 17.X.X)
Library Version: v10.0.10

Attachments:
response.json

We don't get the images from the api/server actually
The array is the file path for the images stored in the device itself, after capture
We can however clean that to represent an actual json array

The array is the file path for the images stored in the device itself, after capture
We can however clean that to represent an actual json array

This would be very helpful, it's currently inconsistent with the Android library as that library returns an array of liveness image path


Secured HTTPS requests

Add Gin framework https://github.com/hackpgh/DINGUS/commit/250c156693cf8a4393724b405903a5a984fede3c


[annoy] new port

[x] Changes comply with the maintainer guide
[x] The name of the port matches an existing name for this component on https://repology.org/ if possible, and/or is strongly associated with that component on search engines.
[x] Optional dependencies are resolved in exactly one way. For example, if the component is built with CMake, all find_package calls are REQUIRED, are satisfied by vcpkg.json's declared dependencies, or disabled with CMAKE_DISABLE_FIND_PACKAGE_Xxx
[ ] The versioning scheme in vcpkg.json matches what upstream says.

no, using current master with version-date for lastest upstream CMake patches

[x] The license declaration in vcpkg.json matches what upstream says.
[x] The installed as the "copyright" file matches what upstream says.
[x] The source code of the component installed comes from an authoritative source.
[x] The generated "usage text" is accurate. See adding-usage for context.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is in the new port's versions file.
[x] Only one version is added to each modified port's versions file.

Tested usage successfully by the annoy:x64-windows triplet:
    # this is heuristically generated, and may not be correct
    find_package(Annoy CONFIG REQUIRED)
    target_link_libraries(main PRIVATE Annoy::Annoy)


npm init rust-webpack my-app will install to the current directory

The expected behavior would be to install it to create the my-app directory and install it inside there.

@sendilkumarn I suggest to then in the README change it to:
mkdir my-app
cd my-app
npm init rust-webpack

What do you think?

well npm init rust-webpack my-app does that right?

That's exactly what I'm trying to say: this doesn't work. Please try the command in your terminal and you will see it will just install the template in the current directory.

Nope, it installs in the directory specified. Can you please check?
What is the OS, you are using?

OS: OSX 10.13.6
node v10.9.0 (npm v6.2.0)

From the log it seems that you are using the old version of the library.
Can you please try updating the library?

@sendilkumarn how can I upgrade running  npm init or what do you mean?

@fitzgen Can we release a new version? Please remove cargo.lock in the new version. It causes issue.
@nikgraf I am re-opening this, please try to use the new version and let us know if you have the same error.
Can you also share us more info? like the create-rust-webpack.js from your installation ? It will be inside the global node_modules folder.

Can you please try updating the library?

npm  i -g create-rust-webpack

Published.

Thanks @fitzgen
@nikgraf it should be working now without any problem. Please try it inside a new directory.


[bug] Improve multi-Trailblazer support from reliquary-archiver

Issue Template
Description
reliquary-archiver generates character json for each of the trailblazer's paths with mixed results. When importing both characters and relics this can work fine while others it is a grab-bag of issues including incorrect LC assignments, missing builds from other paths or even relics not assigning to the correct build.
Steps to Reproduce

Generate reliquary archive json
Import relics and characters from generated archive

Expected Behavior

Each trailblazer path available to the player has a build present in the characters tab
Each build is correctly named for the selected gender and path
Each build as the correct light cone equipped
Each build has the correct relics equipped

Actual Behavior

In the current state (at the time of this post) Harmony trailblazer is treated as Stelle (Destruction)

this is due to IceDynamix/reliquary-archiver#16 and will be fixed in an upcoming release

Due to the above Stelle (Destruction)'s relics will be replaced by Stelle (Harmony)'s
Relics do not neccesarily match the build used in-game

this is due to IceDynamix/reliquary-archiver#12. A fix for this is in progress

The equipped LC is the same across builds regardless of path\

same issue as above

Screenshots
Current State

In upcoming update

Hey! Thanks for the heads up - just to clarify, do these issues get fixed if the new reliquary patch goes out with the correct trailblazer path / name?

That is correct. The issue with HMC getting imported as Stelle (Destruction) is already in dev and will get fixed as soon as a release is made. Everything else will be fixed with IceDynamix/reliquary-archiver#28 is merged, and I can't imagine a new release being cut without it.
Also, I'll have a PR up for this issue soon.


Restructure and clean up README

Quick summary:

Lots of the front-page links are either dead (Gmail-hacks) or obsolete (old-style Firefox extension). Removed.
Examples for creating extension promoted one using less reliable loading techniques and an obsolete version of Gmail.js. Restructured and reordered. (This is not ego-stroking by pushing by boilerplate repo first. This has been the source of numerous issues recently.)
Be more up front about what the library does and doesn't.
Try to make summary sound a little more "sexy".

@KartikTalwar : I think this is much better... But it's your project, and definitely your front-page. I'm not pushing this if you don't like it.
Let me know if there is some changes you disagree about.

Sorry for the delayed response - I was traveling. LGTM!


Overhaul AuthorizationService

(If this PR fixes a github issue, please add Fixes #<xyz>.)
Fixes #
(or if this PR is one task of a github issue, please add Master Issue: #<xyz> to link to the master issue.)
Master Issue: #
Motivation
With the addition of the more granular allowTopicOperation, the old canProduce/canConsume/canLookup usages of AuthorizationService were redundant. This mr switches all the code to use the allowTopicOperation instead of the latter three.
Also in PulsarAuthProvider, we enhance the default to make use of the existing interfaces.
Modifications
Describe the modifications you've done.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes / no)
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

Is the PR just focused on the topic operation? I think the namespace ops and tenant ops might need a look too.

This approach is requiring too many changes in the code base. I will re-work this one.


Feature request - Upload the plugin to Spigot

EN (I speak english, so I will use google translate to translate to japanese)
I am hoping you can upload this to Spigot, I currently run a MC server and I cannot directly download from sources other than Bukkit and Spigot. Also, there is a lot of plugin limiations. I am requesting if you can upload it to Spigot. If you can, thank you. If you can't, oh well.
JP
これをSpigotにアップロードできることを願っています。現在、MCサーバーを実行しており、BukkitとSpigot以外のソースから直接ダウンロードすることはできません。また、プラグインの制限がたくさんあります。 Spigotにアップロードしていただけませんか。できればありがとうございます。できない場合は、まあ。

First, thank you for contributing to this project.
We will let you know when the upload to Spigot is complete.

Currently, project leader @peyang-Celeron doesn't seem to be very active about uploading to Spigot.
She says:

"Is it really good to upload a plugin that throws an exception and erases machine learning data from the moment it starts?"

I want to upload to Spigot like you too, but it seems that I need to convince @peyang-Celeron first.

Currently, project leader @peyang-Celeron doesn't seem to be very active about uploading to Spigot.
She said:

"Is it really good to upload a plugin that throws an exception and erases machine learning data from the moment it starts?"

I want to upload to Spigot like you too, but it seems that I need to convince @peyang-Celeron first.

Isn't this what hypixel uses for watchdog? Can you send me videos on how the plugin works? And where the issue is?

Isn't this what hypixel uses for watchdog?

Sorry, hypixel doesn't use this plugin.
This plugin is inspired by hypixel, so this plugin is not a leak or reverse engineering.

Can you send me videos on how the plugin works?

Yes. here is a working video.

And where the issue is?

It is that the machine learning data implemented in the plugin is deleted when the server is stopped (#42), and the plugin throws an exception when the server is started.
The former is a fatal bug that needs to be fixed.
In the latter case, the plugin itself will start normally, but the cause is unknown.

Yes. here is a working video.

What server is  that, I also have the same client, may I try testing it out on that server?
Also, I know little about coding, but I have been learning Java, I could help with English translations if you like!

What server is that, I also have the same client, may I try testing it out on that server?

Start the server locally and deploy the built source code directly.
Therefore, the server running in the video is a local server, not a server published on the Internet.
It may be a bit inconvenient, but for normal testing you should deploy it to your server using an IDE such as IntelliJ IDEA.

I could help with English translations if you like!

Thank you very much!
We decided on the multilingual file path, save a translated text as en.yml in the following path:
src/main/resources/message/en.yml

Check here for contributing guides.

I don't know how to setup a local server, can you show me how?

Also, I will be happy to help, I have dug around for a while in Hypixel's backend and I do know what most staff plugins / messages do.

Updated #65

I know it has been a while but I am deciding to continue working on it a bit more.

I will close it, PSAC has moved to PSAC-improved


Add docs for caveats, chores

Docs: Add caveats and alternatives
Chore: Remove extraneous foo.js
Upgrade: npm audit fix, npm update

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


systemctl failed to connect to bus: no such file or directory

Windows 10. Debian Stretch on WSL, After today's upgrade to Build 18351.18h1 release<PHONE_NUMBER>.
So, I can't run MySQL :(

use system instead on systemctl

Thanks bro, it helps me a lot

Thanks bro, it helps me a lot

How so ,any idea?it didn't come out of any reply and service still cannot be access


⚠ Samsung has degraded performance

In 1009d9a, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2173 ms

Resolved: Samsung performance has improved in 27ec7f2.


Fixed processing of job submitter

Why are these changes needed?
Fixes errors introduced by PR that added job submitter to API server
Related issue number

Checks

[x ] I've made sure the tests are passing.
Testing Strategy

[x ] Unit tests
[ x] Manual tests
[ ] This PR is not tested :(

@z103cb, @astefanutti please, take a look
cc @kevin85421


Compile time error: expected ‘)’ before ‘PRIx64’

Error when running dbuild/make
Building CXX object layers/CMakeFiles/VkLayer_utils.dir/vk_layer_utils.cpp.o
In file included from /home/pbrubeck/Vulkan-LoaderAndValidationLayers/layers/vk_layer_utils.h:31:0,
                 from /home/pbrubeck/Vulkan-LoaderAndValidationLayers/layers/vk_layer_utils.cpp:33:
/home/pbrubeck/Vulkan-LoaderAndValidationLayers/layers/vk_layer_logging.h: In function ‘VkBool32 log_callback(VkFlags, VkDebugReportObjectTypeEXT, uint64_t, size_t, int32_t, const char*, const char*, void*)’:
/home/pbrubeck/Vulkan-LoaderAndValidationLayers/layers/vk_layer_logging.h:265:53: error: expected ‘)’ before ‘PRIx64’
     fprintf((FILE *)pUserData, "%s(%s): object: %#" PRIx64 " type: %d location: %lu msgCode: %d: %s\n", pLayerPrefix, msg_flags,
                                                     ^
make[2]: *** [layers/CMakeFiles/VkLayer_utils.dir/vk_layer_utils.cpp.o] Error 1
make[1]: *** [layers/CMakeFiles/VkLayer_utils.dir/all] Error 2
make: *** [all] Error 2

My specs
Centos 7.2
Kernel 3.10.0-327.13.1.el7.x86_64
gcc version 4.8.5 20150623 (Red Hat 4.8.5-4) (GCC)
Cmake version 2.8.12

This code builds fine with all the tool chains we are using.  If you remove the "#", does the problem go away for you?

This is a C99 feature and it seems possible that -std=c99 needs to be specified as a compiler option for the version of gcc used here.
Closing based on age, lack of follow-up, and the likelihood that the above explanation will help.

I get this as well under the current master.
gcc 4.9.2
CentOS 6.8
cmake version <IP_ADDRESS>
kernel 2.6.32-573.26.1.el6.x86_64
The problem is that the header vk_layer_logging.h is being included from c++ files.   vk_layer_logging.h uses the c99 header include inttypes.h rather than the C++ version cinttypes.  Given that vk_layer_logging.h is in C++ not C I believe that the proper fix would be to switch to C++ headers.

Thanks, I saw that cinttypes might be a possible solution, but couldn't reconcile it working with other compilers.  I'll reopen and look at this again.

Fixes pushed to master in https://github.com/KhronosGroup/Vulkan-LoaderAndValidationLayers/commit/37fb5d229d5163e1c1f817810dafd5e3e4c33a1d.
This repo builds now on CentOS 7 with gcc 4.8.5 and cmake 2.8.12.

https://github.com/KhronosGroup/Vulkan-LoaderAndValidationLayers/commit/414e9a4117b500eac650d8b8f01a6e1b22d05aae#diff-81e715a0f07036945919bca76416f7df
This commit broke this again by removing the # sign at the end of the first string before the PRIx64.


Archieved flag for DDO database in case of disaster recovery?

If block couldn't be updated (due to db connection problems or other factors) and DDOs are deleted => should set in DDO mapping a field for archieved DDOs?
see https://github.com/oceanprotocol/ocean-node/pull/332/files/28745f334c4a8cf4b6a5ef45998706cd7ee4d823#diff-f3e4b58444c79318209094dc30b3b6317640cfa8e2838d8ec84121a201d09b18R171

Discussed in meeting => execute update last indexed block then delete ddo, if deletion was not successful => revert block

will tackle in #332


Allow Microsoft Teams to share a fancy zone in a meeting

Description of the new feature / enhancement
When sharing screen in a Microsoft Teams meeting, it'd be great to have the capability to share a fancy zone only.
Scenario when this would be used?
Here's my use case, I have an 34-inch ultra wide screen of 21:9 ratio. Sharing my entire monitor to a meeting participant with FHD monitor (say 23 inch 16:9 monitor) makes the other party hard to see my content. Sure I can share a Window but it's a workaround, and does not have the same flexibility of sharing a fancy zone.
Supporting information
No response

Great idea, Tracked in #279

This is our how team is viewing this grouping, a “virtual monitor”.

Coming back to this topic: what's the original ticket number (if this is a duplicate)? I've set up my zones but I am not able to share a specific one in teams, just the usual things: screen or program. Was the original idea implemented or still waiting?

I'm sorry, but I read through #279, and don't see that the request is the same thing at all. Its title is "Maximize/Full Screen window within a zone (virtual monitor)" The OP here has asked to be able to share a zone, instead of a window, which means an application, or a screen, which is one monitor. FancyZones lets me create a 1920x1080 zone that I'd like to share in a meeting so that I can drag in different apps into that zone. IOW, I need to be able to share different apps on-the-fly. For example, if I have Azure DevOps open in a browser, and Visual Studio in another, I can look at code with a client but drag the browser into that zone so we can look at work items.

@NancyFolsom  how we are viewing #279 is it would hit that requirement.  Teams and other apps do not have the concept of FancyZones, it does however have it with applications and monitors. We have ideas on how to maybe do treat zones in FZ as an 'app' but then the questions arise of how does the content flow through and is that universal across ALL the apps that do this style screen sharing.
Versus if we trick the OS to think it is a monitor ... stuff just works and you can full screen applications / videos :)
This is why we group these asks against that ask.

@NancyFolsom how we are viewing #279 is it would hit that requirement. Teams and other apps do not have the concept of FancyZones, it does however have it with applications and monitors. We have ideas on how to maybe do treat zones in FZ as an 'app' but then the questions arise of how does the content flow through and is that universal across ALL the apps that do this style screen sharing.
Versus if we trick the OS to think it is a monitor ... stuff just works and you can full screen applications / videos :)
This is why we group these asks against that ask.

Fair enough. It was a big confusing to see the issue stated as being maximize w/i a zone. Maybe a tweak to the title would help? Anyway, thanks, again, for explaining.

I also wait that teams application sharing would work with the zones of FancyZones. But until this is not implemented i use RegionToShare


Java project loading infinitely

When I open a java project, the extension starts loading the project but never finishes, which prevents me from accessing automatic imports, or simply creating a new file with the base of a class.
Clean java language server workspace don't work
I'm using fedora 38, Project Manager for Java v0.23.6 and vscode v1.90.0

What is the project you are working on? Is that sharable?

This is just the basic project to create a minecraft mod with forge


IndependentMOGP with inputs grouped by feature

Currently the IndependentMOGP only has support for input vectors whose type is MOInputIsotopicByOutputs. This PR adds support for MOInputIsotopicByFeatures.
Something annoying happened with the things in the test file, which meant that the diff looks unpleasant. Not sure how to fix that. Apologies! Nothing in the @testset "MOInputIsotopicByOutputs" begin block has changed, just the @testset "MOInputIsotopicByFeatures" begin block has been added.
Hopefully the tests are sufficient to convince you of correctness -- I've constructed a GP equivalent to an IndependentMOGP using the LinearMixingModelKernel from KernelFunctions with identity mixing matrix, and checked that they agree on the various quantities of interest.

@thomasgudjonwright I've made another pass over the naming and comment.
I've removed the method which applied toMOInputIsotopicByFeatures, because the thing I need is the same in both cases.
I've also renamed the function to make it clearer what it's doing.
Also, why on earth has our CI stopped running...

@thomasgudjonwright I've made another pass over the naming and comment.
I've removed the method which applied toMOInputIsotopicByFeatures, because the thing I need is the same in both cases.
I've also renamed the function to make it clearer what it's doing.
Also, why on earth has our CI stopped running... 😬

@thomasgudjonwright I've made another pass over the naming and comment.
I've removed the method which applied toMOInputIsotopicByFeatures, because the thing I need is the same in both cases.
I've also renamed the function to make it clearer what it's doing.
Also, why on earth has our CI stopped running... 😬

Seems like the CI is running again 🥳 ! LGTM once the github-actions comments are resolved!

Seems like the CI is running again 🥳 ! LGTM once the github-actions comments are resolved!

On my screen it appears that only the format suggestions are running :( None of the tests appear to be run. Definitely not keen to merge without them running.

Ah. The repository had apparently disabled CI due to inactivity. Seems to be running fine now. Will merge when CI passes.

Ah. The repository had apparently disabled CI due to inactivity. Seems to be running fine now. Will merge when CI passes.


Completed the tasks

all tasks need to be checked to see they works fine

all files looks fine could be added to the graham cluster


Apple-Silion Macs don't support SVE instructions for Highway.

I'm using 2021 M1 Max MacBook Pro to compile vectorlite.
One of its dependenies, highway, is using SVE which Apple-Silicon Macs don't support.
**Version of C++:**
/Library/Developer/CommandLineTools/usr/bin/c++ -v
Apple clang version 16.0.0 (clang-16<IP_ADDRESS>)
Target: arm64-apple-darwin24.0.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin

**Error detail:**
[19/41] /Library/Developer/CommandLineTools/usr/bin/c++ -DHWY_STATIC_DEFINE -DTOOLCHAIN_MISS_ASM_HWCAP_H -DTOOLCHAIN_MISS_SYS_AUXV_H -I/Users/xiaobao/project/vectorlite/vcpkg/buildtrees/highway/src/1.2.0-9e5d3e7297.clean -fPIC -g -std=c++17 -arch arm64 -isysroot /Library/Developer/CommandLineTools/SDKs/MacOSX15.0.sdk -fPIC -fvisibility=hidden -fvisibility-inlines-hidden -Wno-builtin-macro-redefined -D__DATE__=\"redacted\" -D__TIMESTAMP__=\"redacted\" -D__TIME__=\"redacted\" -fmerge-all-constants -Wall -Wextra -Wconversion -Wsign-conversion -Wvla -Wnon-virtual-dtor -Wcast-align -Wfloat-overflow-conversion -Wfloat-zero-conversion -Wfor-loop-analysis -Wgnu-redeclared-enum -Winfinite-recursion -Wself-assign -Wstring-conversion -Wtautological-overlap-compare -Wthread-safety-analysis -Wundefined-func-template -fno-cxx-exceptions -fno-slp-vectorize -fno-vectorize -fdiagnostics-show-option -fcolor-diagnostics -Wc++2a-extensions -fmath-errno -fno-exceptions -Wno-psabi -MD -MT CMakeFiles/hwy_contrib.dir/hwy/contrib/sort/vqsort_i32a.cc.o -MF CMakeFiles/hwy_contrib.dir/hwy/contrib/sort/vqsort_i32a.cc.o.d -o CMakeFiles/hwy_contrib.dir/hwy/contrib/sort/vqsort_i32a.cc.o -c /Users/xiaobao/project/vectorlite/vcpkg/buildtrees/highway/src/1.2.0-9e5d3e7297.clean/hwy/contrib/sort/vqsort_i32a.cc
FAILED: CMakeFiles/hwy_contrib.dir/hwy/contrib/sort/vqsort_i32a.cc.o
/Library/Developer/CommandLineTools/usr/bin/c++ -DHWY_STATIC_DEFINE -DTOOLCHAIN_MISS_ASM_HWCAP_H -DTOOLCHAIN_MISS_SYS_AUXV_H -I/Users/xiaobao/project/vectorlite/vcpkg/buildtrees/highway/src/1.2.0-9e5d3e7297.clean -fPIC -g -std=c++17 -arch arm64 -isysroot /Library/Developer/CommandLineTools/SDKs/MacOSX15.0.sdk -fPIC -fvisibility=hidden -fvisibility-inlines-hidden -Wno-builtin-macro-redefined -D__DATE__=\"redacted\" -D__TIMESTAMP__=\"redacted\" -D__TIME__=\"redacted\" -fmerge-all-constants -Wall -Wextra -Wconversion -Wsign-conversion -Wvla -Wnon-virtual-dtor -Wcast-align -Wfloat-overflow-conversion -Wfloat-zero-conversion -Wfor-loop-analysis -Wgnu-redeclared-enum -Winfinite-recursion -Wself-assign -Wstring-conversion -Wtautological-overlap-compare -Wthread-safety-analysis -Wundefined-func-template -fno-cxx-exceptions -fno-slp-vectorize -fno-vectorize -fdiagnostics-show-option -fcolor-diagnostics -Wc++2a-extensions -fmath-errno -fno-exceptions -Wno-psabi -MD -MT CMakeFiles/hwy_contrib.dir/hwy/contrib/sort/vqsort_i32a.cc.o -MF CMakeFiles/hwy_contrib.dir/hwy/contrib/sort/vqsort_i32a.cc.o.d -o CMakeFiles/hwy_contrib.dir/hwy/contrib/sort/vqsort_i32a.cc.o -c /Users/xiaobao/project/vectorlite/vcpkg/buildtrees/highway/src/1.2.0-9e5d3e7297.clean/hwy/contrib/sort/vqsort_i32a.cc
**fatal error: error in backend: Calling convention SVE_VectorCall is unsupported on Darwin.**
c++: error: clang frontend command failed with exit code 70 (use -v to see invocation)
Apple clang version 16.0.0 (clang-16<IP_ADDRESS>)
Target: arm64-apple-darwin24.0.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin
c++: note: diagnostic msg:

I've run into the same problem. Following https://github.com/google/highway/issues/2356, I tried setting -DHWY_DISABLED_TARGETS=HWY_ALL_SVE in vectorlite/vcpkg/ports/highway/portfile.cmake, but that didn't work (seems that that flag would be passed to CMake, and does not affect preprocessor definitions)
Did you manage to get this to work?

I've run into the same problem. Following google/highway#2356, I tried setting -DHWY_DISABLED_TARGETS=HWY_ALL_SVE in vectorlite/vcpkg/ports/highway/portfile.cmake, but that didn't work (seems that that flag would be passed to CMake, and does not affect preprocessor definitions)
Did you manage to get this to work?

yeah, I've contacted the author and he also suggested adding the flag to highway. But CMake would never pass this flag down into the library. So I installed LLVM 15 using Homebrew and switch the toolchain from Xcode to LLVM.
And it works in my case to compile vectorlite on my M1 Max machine.

Hi, Highway TL here. The recommended -DHWY_DISABLED_TARGETS=HWY_ALL_SVE is a compiler flag and can be passed there via CMake by export CXXFLAGS=-DHWY_DISABLED_TARGETS=HWY_ALL_SVE before running CMake.
FYI we are also in touch with Arm on fixing this compiler issue.


Node.js & React Native

Title / Título
Node.js & React Native
Description / Descripción
If you're a web developer who has been itching to get into native development, now is your chance. In this series we'll be talking all about React Native and Node.js. React Native allows you to build native iOS and Android applications in JavaScript, and React.js. Node.js allows you to create servers to communicate with your app to provide a better experience inside the app.
Speaker Bio / Bio del Ponente
Italo Brito Brasil - Fullstack Developer with more then 10 years working with web technologies and many systems and apps in Brazil. Design Thinker and Leader of IxDA Teresina, BeerJS & @21unicorns.
Evangelist JS & React Native where i teach about new approaches to create nice apps with great UIs and with a good expe
iesede-flyer.pdf
rience too.
Level / Nivel
Intermediate
Duration / Duración
one hour
Constraints / Condicionantes
October, November

Closing after talking to @jordiperezdelolmo, please reopen when necessary.


feat: add bitrate to ios and audio channels to android

Description

add bitRate to iOS (default: 128000, same as Android)
add audioChannels to Android (default: 2, same as iOS)

Thank you 😆


feat: introduce FirebaseException StackTrace support

Description
Currently whenever a FirebaseException is thrown, any stack traces provided with a genuine error are lost. This PR adds support for an optional stack trace, along with updating the current reworked plugins to forward the trace on.
Example:

Related Issues
N/A
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] If the pull request affects only one plugin, the PR title starts with the name of the plugin in brackets (e.g. [cloud_firestore])
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See [Contributor Guide]).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the [Flutter Style Guide].
[x] I updated pubspec.yaml with an appropriate new version according to the [pub versioning philosophy].
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the [CLA].
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

Note this moves changes to an "unpublished" state; no version bumps or change log version changes.

Would be nice if this could be reappears alongside Crashlytics, @Ehesp what do you think?

Waiting on https://github.com/FirebaseExtended/flutterfire/pull/3700 to land for this one

Closing for now given how far behind it is.


ref(tests): move to using the TestCase classes DRF provides

Simplifies many calls
By setting a default DRF testing format (was already done in the settings file anyway), moving to using APITestCase and friends, plus credentials / login then the tests overall look cleaner and in many cases easier to grok
More information is at http://www.django-rest-framework.org/api-guide/testing/
Note that there is no need to use json.dumps when sending data as DRF takes care of that

Current coverage is 82.45%

Merging #562 into master will not affect coverage as of 2a75f6d

@@            master    #562   diff @@
======================================
  Files           36      36
  Stmts         2303    2303
  Branches       331     331
  Methods          0       0
======================================
  Hit           1899    1899
  Partial        115     115
  Missed         289     289

Review entire Coverage Diff as of 2a75f6d

Powered by Codecov. Updated on successful CI builds.


some optimizations to inline map

Commit Message: optimization to inline map
Additional Description:
This PR contains some useful improvements to inline map lib from recent practices. This patch contains following changes:

Update instantiation of inline map. Now the inline map could be created at stack.
Lazy memory allocation. Memory will be only allocated when first inline entry is added.
Continuous memory for inline entry values and flags.
More benchmark tests.

Risk Level: low.
Testing: unit.
Docs Changes: n/a.
Release Notes: n/a.
Platform Specific Features: n/a.

hmmm, I didn't update the description.
/wait

-----------------------------------------------------------------------------
Benchmark                                   Time             CPU   Iterations
-----------------------------------------------------------------------------
BM_InlineMapConstructAndDestruct/0        154 ns          154 ns      5343948
BM_InlineMapConstructAndDestruct/1        191 ns          191 ns      3377202
BM_InlineMapConstructAndDestruct/2        372 ns          372 ns      1870727
BM_InlineMapSet/0/0                     12357 ns        12357 ns        56254
BM_InlineMapSet/1/0                     15058 ns        15058 ns        46970
BM_InlineMapSet/2/0                      7120 ns         7120 ns        97133
BM_InlineMapSet/2/1                      1817 ns         1817 ns       386748
BM_InlineMapGet/0/0                      4585 ns         4585 ns       153700
BM_InlineMapGet/1/0                      4950 ns         4950 ns       141624
BM_InlineMapGet/2/0                      6530 ns         6530 ns       108060
BM_InlineMapGet/2/1                     0.000 ns        0.000 ns<PHONE_NUMBER>

From latest benchmark, the inline map pretty good performance for get/set operation (if the inline handle is used). But the destruction of inline map take much more time than normal flat_hash_map.
This means the inline map is not appropriate for scenarios where users will create the map instances frequently.

can you do a before/after on this benchmark with/without this change?
From what I understand, you will need to do a heap allocation either way (within inline-array and unique_ptr for the object, or with inline object and indirection for the array).

/wait
clarifying my request for benchmarks: you have new benchmarks in this PR. Can you patch them into a client without the impl changes so you ca do a proper a/b on all the benchmarks?
Thank you!

From what I understand, you will need to do a heap allocation either way (within inline-array and unique_ptr for the object, or with inline object and indirection for the array).

Haha, 😄 we still reduce once time heap allocation: the validity flags. In previous impl, we always need do a heap allocation for the validity flags (because we store the validity flags in the std::vector), now, we needn't do that.

will try to generate a bechmark for previous implementation.

Yeah I guess you are making 2 changes here, which are in some sense orthogonal

you are putting the value-array and the bools into one allocation
you are changing changing the heap allocation from the whole object to just the array

You could have done 1 without 2, or 2 without 1 :)
Just be aware that when you start allocating blocks of memory and offsetting into them with a lot of casting, this is hairier stuff with lots of opportunity for alignment issues on various platforms., and will require more scrutiny.

/wait

Benchmark result of previous implementation:
--------------------------------------------------------------------------------------------
Benchmark                                                  Time             CPU   Iterations
--------------------------------------------------------------------------------------------
BM_InlineMapConstructAndDestruct/0                     0.342 ns        0.342 ns<PHONE_NUMBER>
BM_InlineMapConstructAndDestruct/1                      55.9 ns         55.9 ns     13666244
BM_InlineMapConstructAndDestruct/2                       483 ns          483 ns      1454890
BM_InlineMapConstructAndDestructWithSingleEntry/0        130 ns          130 ns      5352604
BM_InlineMapConstructAndDestructWithSingleEntry/1        242 ns          242 ns      2883499
BM_InlineMapConstructAndDestructWithSingleEntry/2        563 ns          563 ns      1300264
BM_InlineMapSet/0/0                                    12569 ns        12569 ns        56541
BM_InlineMapSet/1/0                                    14862 ns        14862 ns        46831
BM_InlineMapSet/2/0                                     7612 ns         7612 ns        94023
BM_InlineMapSet/2/1                                     2219 ns         2219 ns       305484
BM_InlineMapGet/0/0                                     4655 ns         4655 ns       149487
BM_InlineMapGet/1/0                                     5301 ns         5301 ns       135805
BM_InlineMapGet/2/0                                     6523 ns         6523 ns       106430
BM_InlineMapGet/2/1                                    0.000 ns        0.000 ns<PHONE_NUMBER>

Benchmark result of latest implementation:
--------------------------------------------------------------------------------------------
Benchmark                                                  Time             CPU   Iterations
--------------------------------------------------------------------------------------------
BM_InlineMapConstructAndDestruct/0                     0.341 ns        0.341 ns<PHONE_NUMBER>
BM_InlineMapConstructAndDestruct/1                      8.54 ns         8.54 ns     82885993
BM_InlineMapConstructAndDestruct/2                      8.06 ns         8.06 ns     87560516
BM_InlineMapConstructAndDestructWithSingleEntry/0        133 ns          133 ns      5093443
BM_InlineMapConstructAndDestructWithSingleEntry/1        194 ns          194 ns      3605611
BM_InlineMapConstructAndDestructWithSingleEntry/2        363 ns          363 ns      1958655
BM_InlineMapSet/0/0                                    12555 ns        12554 ns        53261
BM_InlineMapSet/1/0                                    15175 ns        15175 ns        45922
BM_InlineMapSet/2/0                                     7524 ns         7524 ns        94286
BM_InlineMapSet/2/1                                     1926 ns         1926 ns       355645
BM_InlineMapGet/0/0                                     4718 ns         4718 ns       150265
BM_InlineMapGet/1/0                                     5140 ns         5140 ns       131661
BM_InlineMapGet/2/0                                     6583 ns         6583 ns       106319
BM_InlineMapGet/2/1                                    0.000 ns        0.000 ns<PHONE_NUMBER>

From latest benchmark, the inline map pretty good performance for get/set operation (if the inline handle is used). But the destruction of inline map take much more time than normal flat_hash_map.
Compare to previous implementation, the new implementation has similar (or even minor better?) get/set performance. And has better construct/destruct performance obviously.
This means the inline map is not appropriate for scenarios where users will create the map instances frequently.

Another interesting thing is I has compared the source code of flat_hash_map and our InlineMap. They has similar memory allocation/deallocation policy (lazy memory allocation, iterate the backing array to destruct the valid elements one by one).
But the flat_hash_map has obvious better construct/destruct performance than our InlineMap, even both they are empty.
I didn't find a possible reason from the source code.
And I did a tricky test. I removed all logic in the ~InlineMap and still get a poor result. But if I remove the whole ~InlineMap, I get a comparable result with flat_hash_map. I guess the explicit ~InlineMap prevent the compiler from doing some special optimizations. I will try to find why, when I have some free time.

I thinik the reason for the destruct slowness is that if there are a lot of inline keys then you must test each bool on destruction.
You could optimize this a little by exiting out of the loop in the dtor when you find size() elements. But maybe it's worth condidering, for this reason, storing the has-value bit as one bit per bit. You could then stride through the array of words and only look at the individual bits of a non-zero word.
I'm not sure if it's better to store 8 bits in uint8_t or 32 bits in a uint32_t; it depends on typical usage.
In contast, the Envoy header-maps don't need to do this because they also have a linked-list of valid entries.

I thinik the reason for the destruct slowness is that if there are a lot of inline keys then you must test each bool on destruction.

No, I don't think this is the cause. As I said and the benchmark results, even the descriptor contains zero inline key, the dctor is still much slower.

will try the bit flags tomorrow and output the benchmark results.

Here is another benchmark result of version that applied following modifications:

bit validity flags are used. I stored 8 bits in uint8_t.
calculate offset at runtime and eliminate the inline_entries_valid_.
evaluate 8 bits at same time when do the destruction.

Here is the commit (store it in another branch): https://github.com/wbpcode/envoy/commit/ad887f6d884c2c931cb4c8d09c99759aecc70a6a
--------------------------------------------------------------------------------------------
Benchmark                                                  Time             CPU   Iterations
--------------------------------------------------------------------------------------------
BM_InlineMapConstructAndDestruct/0                     0.348 ns        0.348 ns<PHONE_NUMBER>
BM_InlineMapConstructAndDestruct/1                      7.92 ns         7.92 ns     87707109
BM_InlineMapConstructAndDestruct/2                      7.91 ns         7.91 ns     88441953
BM_InlineMapConstructAndDestructWithSingleEntry/0        124 ns          124 ns      5573799
BM_InlineMapConstructAndDestructWithSingleEntry/1        190 ns          190 ns      3715836
BM_InlineMapConstructAndDestructWithSingleEntry/2        262 ns          262 ns      2893055
BM_InlineMapSet/0/0                                    12329 ns        12329 ns        54324
BM_InlineMapSet/1/0                                    14860 ns        14859 ns        35298
BM_InlineMapSet/2/0                                     8652 ns         8652 ns        80875 # Set inline entries by normal string key.
BM_InlineMapSet/2/1                                     3614 ns         3614 ns       194483 # Set inline entries by inline handle.
BM_InlineMapGet/0/0                                     4732 ns         4732 ns       146205
BM_InlineMapGet/1/0                                     5092 ns         5092 ns       136984
BM_InlineMapGet/2/0                                     6657 ns         6657 ns       104969
BM_InlineMapGet/2/1                                    0.000 ns        0.000 ns<PHONE_NUMBER>

From the benchmark result, the bit validity flags reduce the overhead of destruction when the map instance only contains little inline entries.
But it obviously degrade the performance of set (by inline handle) operations. And the get operations' performance is also penalized slightly.
Considering the gain, penalty, and the additional complixity (bit operations), I think it would be better to keep current implementation. 🤔

/wait

friendly ping @jmarantz. Is there any new concerns?

Do we want to continue iterating on this in github if there's no compelling application yet?
I don't think we normally create utility classes in the Envoy codebase with no pending uses. Maybe we could remove this from github for now and you could leave it in a branch until we find a place that can benefit from it?

My initial target is refactor the filter state. But from the benchmark, the gain is not obvious. So, yeah, I think we can split it out. I can add this in my folk (I still want to use it for per_filter_config, but I have no plan to contribute this to upstream in short term)

What alignment problem? I don't think you have one if you use malloc. The only exception might be if you have a data structure requiring SSE operations.

Hmm, seems there are some info gaps.
I initially thought the different type may have different alignment requirement. And the new uint8_t[] or new char[] cannot meet the requirement.
And then you tell me if malloc is used, then there are no alignment problem. But from stackoverflow's discussions, it's possible the default alignment of system may cannot match the requirement of special types'.
Maybe I have no enough related knowledge, so I mis-understood something. I think we can discuss this problem offline, if you are willing to and when we have free time. (I still want get a clear answer to this)


Cloud Translation GoogleApiException "The service is unavailable at this time. [503]"

OS: Windows 10
.NET version: 4.8
Package name and version: Google.Cloud.Translation.V2 (1.2.0-beta02)

I'm trying to translate the attached japanese text to english using this code:
_translationClient.TranslateHtmlAsync(text, "EN", "ZH");
This results in following exception:
The service translate has thrown an exception: Google.GoogleApiException: Google.Apis.Requests.RequestError The service is unavailable at this time. [503] Errors [ 	Message[The service is unavailable at this time.] Location[ - ] Reason[backendError] Domain[global] ]
The strange thing is that there are some texts, which work and some texts, which don't work. I can provide some other failing examples as well if it is needed.
TranslationText.txt

I strongly suspect this isn't a client library issue, but a Translation service issue, in which case it'll be best to follow one of the support routes listed in the Translation support page. But I'll try to a) reproduce it with client libraries; b) reproduce it in the API explorer, just to check.

Hmm... I reproduced it once, but then while fiddling with the text to try to cut down the repro a bit, the error stopped. Could you retry and see if it's been fixed for you now? It could be that it was a temporary backend issue.

Okay for this example it seems to work now. But I have another example, which still fails.
But you are right, this seems to be an error with the service and not with the library.
TranslationText2.txt

Right - I'm afraid there's not a lot I (as a client library maintainer) can do at this point, in that case, so I'm going to close this issue. I suspect the best route is via the google-translate-api Google group, which is monitored by the Translation team themselves. If you link to the same problematic file there, hopefully they'll be able to work out what's going on.


Allow MQTT client name configuration

Accept MQTT client name configuration to allow for multiple instances.
Fixes #93

Thanks for your effort, but I prefer a solution without adding another setting key.


Revert "reconcile crd to desired state"

Reverts kubevirt/kubevirt#5310
This change causes CRDs to be updated on every pass since the kubernetes API creates them as "v1" but they are returned as v1beta1 and compared against a v1beta1. v1 has new defaulted fields which makes the EnsureCustomResourceDefinitionV1Beta1 think that the existing object and the desired object are different. We should wait until https://github.com/kubevirt/kubevirt/pull/5017 is merged to fix this.

/release-note-none
/lgtm
/approve

Can we protect against such an issue with a small e2e test/check?

@EdDev we could add an e2e test. Kedar and I have been talking about which resources to have these tests for. The problem I see is that we are going to be reconciling 17 different resource types and adding a tests for each one could increase testing time by at least a minute per test since operator functional tests are ran serially

@EdDev we could add an e2e test. Kedar and I have been talking about which resources to have these tests for. The problem I see is that we are going to be reconciling 17 different resource types and adding a tests for each one could increase testing time by at least a minute per test since operator functional tests are ran serially

If the logic is generic and not per type, then you can just pick a single resource type as a ref check.


Add-WindowsFeature FS-NFS-Services not found

There is a typo here.  It should be:
Add-WindowsFeature FS-NFS-Service

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

ID: 06290341-e4fb-48cf-7e26-113810965321
Version Independent ID: cede637e-427b-b684-786c-d3222811352c
Content: Deploy Network File System
Content Source: WindowsServerDocs/storage/nfs/deploy-nfs.md
Product: windows-server-threshold
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

Looks like this was fixed - ready to close issue?


[HUDI-3689] Disable flaky tests in TestHoodieDeltaStreamer

What is the purpose of the pull request
Two flaky tests in TestHoodieDeltaStreamer are disabled in this PR to unblock CI:
testHoodieAsyncClusteringJob (follow-up: HUDI-3710)
testParquetDFSSourceWithoutSchemaProviderAndTransformer (follow-up: HUDI-3707)

Verify this pull request
This pull request is a trivial rework / code cleanup without any test coverage.
Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build passes.  Landing this now to unblock CI.  Also canceling the CI run for this PR to save resources.


vault: fix base64decode option handling

The value does not need to be decoded from the kubernetes secret itself, but rather not to be encoded when written to the vault secret.
This is achieved by passing string instead of []byte values to vault.

thank you for the contribution!


nit: Is ruff in Alpha or Beta?

https://github.com/charliermarsh/ruff/blob/main/pyproject.toml#L26 says Alpha
but the URL https://beta.ruff.rs would imply Beta.
People are often reluctant to use pre-release (especially Alpha) software.

Beta, will fix!


Condition 'turn.recognized.score' for intent trigger not working

Describe the bug
Condition for intent trigger doesn't work.
I created a brand new LUIS resource in Azure.
In Composer I specified intent "Cancel" with the following trigger phrases:

cancel
quit
abort
exit
never mind
forget about it
back

and added condition:
=turn.recognized.score>=0.93
Yet, during the dialog, when I input for example number '1' (answering question "How many users would you like to add"), the bot not only treats the number as an intent to cancel but is 0.9 sure it's the correct intent... To make matters worse it doesn't follow the condition specified by me above and triggers the Cancel intent...

Version
2.1.2

Browser

[ ] Electron distribution
[ ] Chrome
[ ] Safari
[ ] Firefox
[x] Edge

OS

[ ] macOS
[x] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

Create a brand new LUIS in Azure.
Create a new intent with trigger words as above.
Add a turn.recognized.score condition with a large value.
Build the bot.
During conversation insert a number as an answer.
Bot redirects to the intent despite being less confident than specified in condition.

Expected behavior
Bot disregards the number input either due to it being irrelevent to trigger phrases or condition turn.recognized.score

Screenshots

Additional context
none

Hi @krassussofficial, I'm looking into this.

@krassussofficial, I'm still looking into this, will report back once I find more info.

Hey @krassussofficial,
Initially, I was able to reproduce this issue on an empty bot. I found a workaround that might help.
The workaround is to Emit a custom event. Here are the steps:

Create an Empty Bot in Composer and provide LUIS authoring key

Create a new intent recognized trigger named "Cancel" with the trigger phrases provided in the above issue

Set the condition to =turn.recognized.score>=0.93
In this new Cancel trigger, click the + icon and select Send a response
Write: This is cancel intent

Add a new Dialog with the name of "NumberDialog" and click OK

In this new NumberDialog, click the + icon and select Number from Ask a question
In Bot Response, write: Enter a number.
Select the User Input tap, in Property tab, write: user.myNumber
click the + icon and select Send a response with the value of "you entered ${user.myNumber}"

In the greeting trigger, select the + sign and then select Emit a custom event from the Access external resources drop-down list.

In the Properties pane on the right side of Composer, select the abc button under the Event name box and then select abc custom event. Enter a name(ex: myEvent) and set Bubble event to be true.

Select + Add from the dialog toolbar then select Add new trigger.

In the trigger creation window, select Custom events from the drop-down list.

Enter the event name of the Emit a custom event(myEvent) you created in the previous section in the What is the name of the custom event? field. Select Submit.

Now you can begin a new dialog in this custom event trigger

Output:

For further information on Custom events, have a look at this documentation.
Try the above steps, let me know if this works.

Closing due to inactivity, feel free to re-open if you are still having issues.


Remove rebase info in Contributing.rst

In middle of our CONTRIBUTING.rst file, we have a section titled "BEFORE MERGING": https://github.com/PyAbel/PyAbel/blob/master/CONTRIBUTING.rst#before-merging
In this section, we provide information about how to rebase a PR. I've always struggled to rebase my PRs, and I guess the advantage is that the PR ends up getting merged with only a single commit. For the past year or so, GitHub has changed the "Merge" button to provide options to either (1) squash and merge, (2) create a merge commit, or (3) rebase and merge. I think that the squash and merge option makes it pretty easy to reduce any PR to a single commit while pressing the merge button.

So, two questions:

Should "squash and merge" be our default merging behavior? It seems that others have concluded that this is the best practice: https://rietta.com/blog/github-merge-types/

If we are squashing and merging, then we don't need to ask contributors to endure the pain of rebasing their PRs and we can also remove that section from CONTRIBUTING.rst.

I'm almost sure that we don't want automatic squashing, as this combines all changes into a single commit, thus losing all history of incremental changes, which might be sometimes important in retrospective analysis. I mean, for repositories with thousands of pull requests, squashing might be desirable to keep the history clean (without intermediate edits), but this is not our case, and if we ever get a PR with every single letter as separate commits, :–) we can ask the author to combine them in a more appropriate way manually...
Regarding rebasing, the reason for doing it is explained in the first paragraph of the mentioned “See this wiki for more information.” Basically, it ensures that there will be no merge conflicts and that the PR author can actually see and test the final results locally (which is better that only relying on our CI unit tests). In principle, it is not really required if the changes are completely unrelated (thus rebasing is just desirable, not strictly required), but recommending it doesn't harm in any case.

Okay, what you are saying makes sense. I think that big projects like suashing so that they can have one commit per PR and the summary message for that commit will relate to the mission of the whole PR, and not just be whatever the last commit happens to be, like "fixed those typos".
But yes, I think that you are right that we can keep everything the way it is for now.


Add unit test for React views

Article

Drop react-addons-test-utils and replace with Airbnb's enzyme


Install ClassifiedTracing filter after TraceInitializerFilter

Problem
The ClassifiedTracing filter was being applied before the TraceInitializerFilter in Router's client stack, such that response classification annotations were being set on the parent server request span, rather than on the client span that was being classified.
Solution
Move the ClassifiedTracing filter to immediately after the TraceInitializerFilter.  This creates a 1-to-1 mapping of client request to classification annotation.

⭐  lgtm. thanks for adding tests!

⭐ 🎯


requests module not installed on pip install

Not exactly solved yet, will close on next release 0.0.19


ES6 issue

I tried to update the settings to ES6 by using:
{"esversion":6}
but it didn't work, the warnings are still there, HELP!

Hi there! Your message is missing some important information; the project's contribution guidelines has advice for how to file a great support request.

Closing this to keep things organized. We can re-open once we have more information.


Change "MyStudentChart" to "MyUCSDChart"

Summary

Name of MyStudentChart card did not reflect current capabilities of card.
Changelog

[General][Fix] - Rename MyStudentChart to MyUCSDChart
[General][Fix] - Internal card_id is also renamed
Test Plan

Tested with dummy Default Cards List API with MyStudentChart changed to MyUCSDChart, all functions of the card worked (hide, show, move, etc.). Tested on iPhone 11 Pro simulator.

Closing in favor of #943


Updated countries.geo.json to include Singapore

Also edited polygon for Malaysia. Previously polygon for Malaysia incorrectly included Singapore

just curious why this PR is not merged yet

(FYI) This is how the polygon looks like for Singapore

Location seems right.

FWIW: this repository holds (low-resolution) country coastline coordinates for all countries in the edited countries.geo.json, and each such country as a separate file named after the id of each said country polygon, so drawing either on a map will be the visual shape of that country, not some kind of territorial claim to waters. This PR wrecks both of those constraints, and edits far more than it claims:

Does anyone know how I can change the "Multipolygon" to "polygon"? I do not think is a simple CTRL+F situation.

Hi, can I check where the Singapore.geo.json file is located? I cant seem to find it within the files. Thanks!

Is there any reason this hasn't been merged yet?


xsd 1.1 assert not validated by xerces 2.12.1-xml-schema-1.1

You will not be able to reproduce this because I commended out the offending line in tokenscript.xsd

Download xerces 2.12.1-xml-schema-1.1

$ wget -O - https://archive.apache.org/dist/xerces/j/binaries/Xerces-J-bin.2.12.1-xml-schema-1.1.tar.gz|tar -zxvf -

Run the validator

$ java -classpath xerces-2_12_1-xml-schema-1.1/xercesImpl.jar:xerces-2_12_1-xml-schema-1.1/xercesSamples.jar:xerces-2_12_1-xml-schema-1.1/xml-apis.jar sax.Counter -s COFI.xml
[Error] tokenscript.xsd:55:78: s4s-elt-invalid-content.1: The content of '#AnonType_token' is invalid. Element 'assert' is invalid, misplaced, or occurs too often.
COFI.xml: 1616 ms (52 elems, 53 attrs, 0 spaces, 22526 chars)

To reproduce this problem, uncomment the two lines mentioned in #388 and edit the test xml file (in this case COFI but any tokenscript file will do) to use the edited tokenscript.xsd then you can see this problem.
Note that I am already using the version of xerces that supports xml-schema 1.1

Hi Weiwu,
I have created xerces based utility to validate the XML using the XSD file and below are the details.
Command:
$ java -classpath "xercesImpl.jar;xercesSamples.jar;xml-apis.jar;xpath2-1.2.0.jar;XMLValidator.jar" XMLValidator H:/alphawallet/TokenScript/schema/tokenscript.xsd H:/alphawallet/tokenscripts/COFI.xml
Note: You need to replace ; with : (for unix) while adding JAR files in classpath.

All the required JAR files are attached here.
xerces-2_12_1-xml-schema-1.1.zip
**Arguments:
xerces-2_12_1-xml-schema-1.1.zip
**
1. First argument is the absolute path to xsd file.
i.e.	H:/alphawallet/TokenScript/schema/tokenscript.xsd
2. Second argument is the absolute path to XML file that needs to be validated against the XSD mentioned in argument 1
i.e. H:/alphawallet/tokenscripts/COFI.xml
Now you can validate XML against the XSD 1.1 using this package.

Tracking of requirement details from mail conversation:
It seems that XML Schema 1.1 is only supported by either Xerces 2.12
(the version with XML schema 1.1 support) or with Saxon. Saxon's
opensource version, at a glance, only support XSLT and XQuery, since
there is no mentioning of validation in the manual.
--
Looking at the usage side, we need a tokenscript commandline tool
ourselves that can do the following sub commands:
$ ts validate tokenscript.xml
$ ts sign tokenscript.xml -o tokenscript.signed.xml
$ ts c14n tokenscript.xml -o tokenscript.canonicalised.xml
$ ts verify tokenscript.xml
(ts is the name of this imaginary commandline tool)
Since these are just standard xml stuff the tool actually has nothing
to do with tokenscript - it involves tokenscript if it checks stuff
lively, like verify if the smart contract addresses in it are valid.
I'd like to commission such a tool to be built based on xerces, since
it seems to be the only tool currently that I tested is able to
validate an XML document without canonicalizing it first (since we
uses entity references in XML). xmllint, for example, fails with
entity references and I expect it to have poor support of XInclude.
If you are interested let me share documents on what I think should be done.

Once you have the validator, we will need a Pull-Request that not only
returns the xmlschema 1.1 rules that I commented out (2 lines), but
also change the schema's root element according to this article:
https://www.oxygenxml.com/doc/versions/22.1/ug-editor/topics/set-xml-schema-version.html
Otherwise, some tools will still process it with schema 1.0.
Let me know how you progressed on this! Thanks.

https://github.com/AlphaWallet/TokenScript/issues/395#issuecomment-716395559

Whatever I have here is initial version, we can eventually convert this to your suggested approach in the requirement document.
I can improve the Java Code to take default XSD path from the github and then we just need to pass the xml name to the command. We can also give the option to refer the local XSD schema for the validation.
My idea is to write a shell script where we can pass the action(validate, sign, c14n, verify) as command line argument and then based on action appropriate Java class will be invoked.

My idea is to write a shell script where we can pass the action(validate, sign, c14n, verify)
as command line argument and then based on action appropriate Java class will be invoked.

If you do so you will have to produce 2 versions (.sh and .bat) and they may behave a bit different depending on MacOS/Ubuntu. It's no harm if the content is extremely simple, you just need to keep it minimal and test it on all OSes, but in this case it's expected to be complicated - i.e. the fact that you can concatenate means it's not going to be simple at all, or you will have to manage a lot of intermediary files. See the example of "Multi-command processing" below:

Let's say xmlsec.jar for now, has 4 sub commands.
$ java -jar xmlsec.jar val tokenscript.xml
$ java -jar xmlsec.jar  sign [-o tokenscript-signed.xml | -d output.dir/] tokenscript.xml
$ java -jar xmlsec.jar c14n [-o tokenscript-signed.xml | -d output.dir/] tokenscript.xml
$ java -jar xmlsec.jar verify tokenscript.xml

The first and last commands also have a long form (validate and canonic, respectively). The second and the third command has an output. If unspecified, it will simply be tokenscript-signed.xml (that is, take the input file name, remove the extension and add -signed.xml, following the convention set by Android apk files).
Each sub-commands has their own parameters
For example, sign has --key
Multi file processing
It should be possible to process multiple files in all of the commands. For example:
$ java -jar xmlsec.jar val */*.xml

Which validates every XML files under every directory.
For the commands that has an output, either -o or -d should be used. But if there are multiple input file, then only -d is allowed. -d causes the output of the same filename under the directory specified.
Multi-command processing
It should be possible to concatenate commands. The most typical use-cases are:
$ java -jar xmlsec.jar val c14n sign verify tokenscript1.xml tokenscript.2xml

This causes the tokenscript files to be validated, canonicalized, signed and verified, and outputs tokenscript1-signed.xml and tokenscript2-signed.xml. (the verify subcommand is smart enough to know that the output file should be used to verify not the original input file). If one of the sub-command fails, the next sub-command is not executed; but if an input file caused one of the sub-command to fail, the next file in queue is processed.

This errror seem to be in the schema. Can you make a PR and link back to this issue?
$ LANG=en_US java -classpath XMLValidator.jar:xpath2-1.2.0.jar:xercesImpl.jar:xercesSamples.jar:xml-apis.jar XMLValidator schema/tokenscript.xsd ../token-api-poc/tokenscripts/COFI.xml
COFI.xml is not valid because
cvc-identity-constraint.4.3: Key 'typeRef' with value 'Transfer' not found for identity constraint of element 'token'.

This was my next finding and actually I am  not getting the ERROR that you
reported either with XERCES or with oxygen editor but getting the error
that you just reported.
[image: image.png]
Schema is expecting below XML block in the XML file. Do you mean that I
should fix the schema and make type attribute optional?
<asnx:module name="ENS-Events">

More details:
I will have to make the type attribute optional from the below XML snippet,
Right now it is compulsory, But the given XML does not have any
referenced XML block for  *type="Transfer" *
ts:origins
<ethereum:event contract="CoFiToken"
filter="from=${ownerAddress}" type="Transfer"/>
</ts:origins>
....
On Mon, Oct 26, 2020 at 4:54 PM Weiwu Zhang<EMAIL_ADDRESS>wrote:

This errror seem to be in the schema. Can you make a PR and link back to
this issue?
$ LANG=en_US java -classpath XMLValidator.jar:xpath2-1.2.0.jar:xercesImpl.jar:xercesSamples.jar:xml-apis.jar XMLValidator schema/tokenscript.xsd ../token-api-poc/tokenscripts/COFI.xml
COFI.xml is not valid because
cvc-identity-constraint.4.3: Key 'typeRef' with value 'Transfer' not found for identity constraint of element 'token'.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/AlphaWallet/TokenScript/issues/395#issuecomment-716485859,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE47O3MSJV6SUOMOX4J52SDSMVL77ANCNFSM4S23YHYQ
.

Schema is expecting below XML block in the XML file. Do you mean that I should fix the schema and make type attribute optional?

Then the validator is working correctly except the error reported isn't human readable!

Keeping it open when there is a tool so the xsd 1.1 stuff can be uncommented as the documents on how to validate it gets updated.

Further communication Updates from Telegram:
Weiwu: Stay connected you need to prioritise making the commandline tool that supports only validate (using the schema location in the xml header only - i have a reason for that) and canonicalisation, and support multi file processing and multi command processing. You should not proritise xml signing and verification as I can get by with sectool for the next a few weeks.

Why we need cannibalization? just want to know little bit more details about cannibalization in our existing stuff.
we actually don't need that, just entity dereference. So anything that can correctly read a XML file with entity reference in it and is able to serialise it into a single XML file will do the job for now.

xmlsectool vs Core Java xerces based validator:
Looks like the main focus of the xmlsectool is signing of the XML document. I also do not find the xmlsectool documentation clear. There is very little information available. If you have found different detailed official documentation than mentioned below Please direct me there.
https://wiki.shibboleth.net/confluence/display/XSTJ2/xmlsectool+V2+Home

https://wiki.shibboleth.net/confluence/display/CONCEPT/MetadataCorrectness#MetadataCorrectness-SchemaValidation.5

I do not see any special advantage of using xmlsectool for schema validation and entity de-referencing, So I am in favour of writing our own simple tool using xerces JAR.

Hi Weiwu,
I have completed the multi-file validation and attached is the Java Code.
Can you create separate repository where I can commit the code. If  I have created my private repository but can not add collaborator as I do not have enterprise git subscription.
XMLValidator.zip
I will start on entity de-referencing. Where we will store the de-referenced file? OR do we need to override the same XML? For now I can create the new XML file to save the result of de-referenced action.

Hi Weiwu,
I am committing my changes in forker repository - https://github.com/darakhbharat/TokenScript.git.
Created new directory named xml-validation-against-xsd-1.1 to commit the changes.
Overview:

Completed the basic working features for xml validation against xsd-1.1
Completed basic working Entity de-referencing feature.
Current implementation Supports multi command and multi-file validation.

Here is the command:
$ java -classpath "xercesImpl.jar;xercesSamples.jar;xml-apis.jar;xpath2-1.2.0.jar;XMLValidator.jar" XMLValidator -val -deref H:/alphawallet/TokenScript/schema/tokenscript.xsd H:/alphawallet/tokenscripts/COFI.xml

The command requires minimum three arguments,

the action: -val, -deref
XSD file location
the XML file location or the directory. If dir is provided then all the XML file present there will be validated.

Things needed to be improved:

Later I will also remove the dependency of XSD file location and it will pick the default if not provided.
Well formatted output
remove xml:base attribute from the de-referenced result.
Where to store the de-referenced result? Right now I am creating separate file by appending -deref to the original xml file name.


Solve Daily Problem: 719-kth-smallest Distance

719. Find K-th Smallest Pair Distance
Hard
The distance of a pair of integers a and b is defined as the absolute difference between a and b.
Given an integer array nums and an integer k, return the kth smallest distance among all the pairs nums[i] and nums[j] where 0 <= i < j < nums.length.
Example 1:
Input: nums = [1,3,1], k = 1
Output: 0
Explanation: Here are all the pairs:
(1,3) -> 2
(1,1) -> 0
(3,1) -> 2
Then the 1st smallest distance pair is (1,1), and its distance is 0.
Example 2:

Input: nums = [1,1,1], k = 2
Output: 0
Example 3:

Input: nums = [1,6,1], k = 3
Output: 5

Constraints:

n == nums.length
2 <= n <= 104
0 <= nums[i] <= 106
1 <= k <= n * (n - 1) / 2

Solved:
class Solution:
  def smallestDistancePair(self, nums: list[int], k: int) -> int:
    nums.sort()

    def numPairDistancesNoGreaterThan(m: int) -> int:
      count = 0
      j = 1
      # For each index i, find the first index j s.t. nums[j] > nums[i] + m,
      # so numPairDistancesNoGreaterThan for the index i will be j - i - 1.
      for i, num in enumerate(nums):
        while j < len(nums) and nums[j] <= num + m:
          j += 1
        count += j - i - 1
      return count

    return bisect.bisect_left(
        range(0, nums[-1] - nums[0]), k,
        key=lambda m: numPairDistancesNoGreaterThan(m))


YHSI: Export grid to csv

On export include fields: YHSI ID, Site name, NTS Map, Community, Site Category, Lat, Long, Category of Property.

Old system had ability to export records to excel- Export FN CSV, Export Community CSV, Export XML, Print List. Add this functionality to YHIS. Do not limit export number to 1000 records!


Index

Here are the issues:
X51006913024_Dropout2d_Grid.pdf : list index out of range
Is there a limit to the size of each PDF？？？

Related to #15