Vulnerability Report's report.artifact.digest field is the artifact's image_id not digest

What steps did you take and what happened:
We've been trying to understand why the report.artifact.digest value doesn't match the actual image digest.
After digging a bit through the code, we realized that the value of report.artifact.digest is actually the image_id.
here's an example:
report:
  artifact:
    digest: sha256:dde063663e676fb551255dcc41f7b67fbb913becaccadde12da2c3a0abc236eb
    repository: <image repo>
    tag: v0.47.3
  os:
    family: alpine
    name: 3.18.4
  registry:
    server: <registry>

per above the digest is sha256:dde063663e676fb551255dcc41f7b67fbb913becaccadde12da2c3a0abc236eb
however, when we scan the same image with trivy cli, we get the following:
trivy i --format json <registry>/<image repo>:v0.47.3

{
  "SchemaVersion": 2,
  "CreatedAt": "2024-09-09T21:20:23.838423-04:00",
  "ArtifactName": "<registry>/<image repo>:v0.47.3",
  "ArtifactType": "container_image",
  "Metadata": {
    "OS": {
      "Family": "alpine",
      "Name": "3.18.4"
    },
    "ImageID": "sha256:dde063663e676fb551255dcc41f7b67fbb913becaccadde12da2c3a0abc236eb",
    "DiffIDs": [
      "sha256:cc2447e1835a40530975ab80bb1f872fbab0f2a0faecf2ab16fbbb89b3589438",
      "sha256:8d7be0d3cbdd44bc6fa563f60e8bc1a373d5e60f035d8c089918a2bcc92fee25",
      "sha256:94e5b30b4e4869f384d25800811d245e2a09d04eeb6295a0eadd34ef3bf62677",
      "sha256:f9733b6837c92fb7242b3513ccc51ed17051488e31115c5dd57ac1ee229f64e9",
      "sha256:1b1eb4aa471edd83d2c07371ae5c7a43435e8f445d749f055f48cb915ed52c21",
      "sha256:1624be5bf90422992672ea3698f7f48b1835627808d52126c46d4539e14eeb7e",
      "sha256:654cfc253da32ff012d018bb56208d1c5de1657e1f39e8f3a8c78bd016c14834",
      "sha256:bd978dcc58af2e02683ddcadad5142fea925718a2154791a45f41e5944fc8c91"
    ],
    "RepoTags": [
      "<registry>/<image repo>:v0.47.3"
    ],
    "RepoDigests": [
      "<registry>/<image repo>@sha256:cf3e3b3a98edde46eac58d4745a2467b9da2bc49a11191565a893ffac38034a3"
    ],

as you can see, the report.artifact.digest value is Metadata.ImageID in the Trivy report:
"ImageID": "sha256:dde063663e676fb551255dcc41f7b67fbb913becaccadde12da2c3a0abc236eb",

The actual Digest is in the Metadata.RepoDigests field:
    "RepoDigests": [
      "<registry>/<image repo>@sha256:cf3e3b3a98edde46eac58d4745a2467b9da2bc49a11191565a893ffac38034a3"
    ],

What did you expect to happen:
We expect that report.artifact.digest would show the actual image digest value, instead of image_id.
perhaps it's worth to include both?
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Environment:

Trivy-Operator version (use trivy-operator version): v0.22.0
Kubernetes version (use kubectl version): 1.28.13
OS (macOS 10.15, Windows 10, Ubuntu 19.10 etc): N/A

Encountered the same issue :/ I guess the source of confusion is the fact, that what Trivy and Docker call image digest, Kubernetes calls imageID (which is a totally different entity) - e.g. for image quay.io/jetstack/cert-manager-controller:v1.15.1 for linux/amd64, Kubernetes reports:
❯ k get pod -o json cert-manager-6947bb5c7f-k7vwj | jq -r '.status.containerStatuses.[].imageID'                                                                (syn-s001-eu-west-aks-loki1-001-admin/cert-manager)
quay.io/jetstack/cert-manager-controller@sha256:057ace5734b53ebdec20b3a0d182c37c9ef0014e9f6364f9b99018e76936e984

while Docker shows:
❯ docker image ls --digests --no-trunc quay.io/jetstack/cert-manager-controller
REPOSITORY                                 TAG       DIGEST                                                                    IMAGE ID                                                                  CREATED        SIZE
quay.io/jetstack/cert-manager-controller   v1.15.1   sha256:057ace5734b53ebdec20b3a0d182c37c9ef0014e9f6364f9b99018e76936e984   sha256:1d33ee05f4c869bf0591c4b176f943833fcdb9e7e48bbd219347eba87f4a7801   4 months ago   66.4MB

By looking at the logic here, if imageRef is of form xxxx@sha256:yyy, then the image digest reported by Trivy Operator is correct. In other (most) cases, incorrect imageID is returned, instead of actual digest. Will submit PR to fix this.

Hey @d-mankowski-synerise! thanks for raising the PR, I think that makes sense.
@chen-keinan sorry for tagging, but was hoping you'd have a chance to take a look and consider the proposed change? I think this makes sense overall and would be better aligned.

Thanks for responding @itaysk! looking forward for the PR to be reviewed, in the meantime if there's anything we can help with - let us know.


Creating a cluster from a Cluster Template Revision  that has K8s version set to latest (with/without override), shows incorrect k8s version in drop down.

Rancher server version - v2.3.1-rc4
Steps to reproduce the problem:
Scenario 1:
Create a CTR (Cluster template revision) that has K8s version set to latest and override .
Create a cluster using this CTR.
Notice that there is incorrect version presented in the K8s dropdown menu

Scenario 2:
Create a CTR that has K8s version set to latest .
Create a cluster using this CTR.
Notice that there is incorrect version presented in the K8s dropdown menu

Available in UI tag latest-2.3

Tested with K8s v2.3-head with UI tag latest-2.3
Scenario 1:
Create a CTR (Cluster template revision) that has K8s version set to latest and override .
Create a cluster using this CTR.
Correct version is presented in K8s dropdown menu as below:

Cluster gets created successfully with the correct K8s version
Scenario 2:
Create a CTR that has K8s version set to latest without an override
Create a cluster using this CTR.
The correct revision is presented in K8s dropdown as below:

Cluster gets created successfully with the correct K8s version


Add CSS 'color' options aside from colours

Add a block which is the same as the CSS 'width: auto' block, but on the left-hand-side has a dummy input saying color:.

Complete in abc4d66


Feed Screen Post Card re-design

OD Hack
This issue is part of ODHack 4.0:
description - https://x.com/OnlyDust_com/status/1791035111992934563
guidelines - https://onlydust.notion.site/ODHack-Common-Guidelines-Project-leads-5e21836f85ad493f9ca429770af0ad40
Please read the guidelines and make sure you follow them.
Please write your thoughts and how long you think it will take to complete the issue in the comments.
Feed Screen Post Card re-design
Post card should be re-designed to match the new design. The new design is available at Figma: https://www.figma.com/design/NIJ68MV1AeI8y283woXiM0/JoyBoy?node-id=101-13187&t=SqYlHcodPjhrLdTM-0
Current Post card implementation can be found at: https://github.com/keep-starknet-strange/joyboy/blob/main/JoyboyCommunity/src/shared/components/Post.tsx
You should implement the design as close as possible and should match 1:1 with or without an image. You should only implement the post card design, not the whole feed screen.
Please battle test it before doing a PR.
Communication
https://t.me/JoyboyStarknet/949
May the Joy be with you!

Hey

i would like to work on this

Hi @bhavyagosai What's the status of this issue? If you can't continue we have to assign this to someone else before the OD Hack ends

Hey, sorry for the delay...I was a bit under the weather yesterday. Let me finish this up by tonight and raise a PR!


Is a reference to EpisodeOfCare needed in the nl-core-SOAPReport profile?

In the STU3 profile that relates to SOAPReport a reference to an EpisodeOfCare is made using an extension with the notion this will need to be included in the R4 profile.
This extension is only in the profile on Observation not in the Composition.
Do we need to include an extension in the nl-core- profile for SOAPReport. If so, to Observation, Composition or both?

Gerelateerd: https://bits.nictiz.nl/browse/MM-1521

This comes down to the fact that within the context of general practitioners, en event is coupled both to an encounter (implemented using the Encounter resource) and to an episode (implemented using the EpisodeOfCare resource). In STU3, both these resources could be referenced using .context, but this was a limited solution as the cardinality of this element was 0..1 For this reason, the .context element was (in words) constrained to Encounter only, and an extension was added to refer to the EpisodeOfCare. In R4 .context is already limited to Encounter only.
The extension could be added if there's a functional backing for it. However, at the moment this is lacking.


Failed Spring app initialization on Heroku

Though all the tests have passed and the WAR app has been deployed to Heroku, the application failed to start because of a failure to find and read from pacdots.json.

While I diagnose and repair the problem, the server has been redeployed temporarily without any pacdots.

Fixed in above-referenced commit. The file-reading configuration has been tweaked to make it work properly.


Change package structure

The majority of this package is in TwoLayerDNS which is getting quite cluttered. I think that this repo and the package should be renamed to TwoLayerDirectNumericalShenanigans.jl (or something similar) with a focus on DNS in two layer systems. Then the code can be re-organised into scripts but in a single module. This in conjunction with #54 will make everything much cleaner and easier to develop and manage.

It also avoids calling using DNCS and using DNCS.TwoLayerDNS when really only need TwoLayerDNS.

With these changes a rename to something like TwoLayerDirectNunmericalShenanigans.jl seems appropriate but I will merge at this point and try this in another PR.


Loose comments from editor (emacs) perspective

Loose comments from editor (emacs) perspective:

Buffers (open files) can be not yet saved and therefore differ from what is on disk currently. Generally we want to operate with the visible contents.
Files-while-editing have the tendency to by not-valid Haskell. Or at least no type correct. There were multiple issues raised so that haskell-mode works with somewhat valid Haskell instead of totally not working.
Editors have cursor position that can serve as indicator what to act on.
Editors have marked region that can serve as indicator what to act on.
Smooth interaction requires latency hiding because 200ms is when people start to notice non-smoothness. Editor needs to be ready to handle longer-running queries, actually any query can run 'too long'. This means communication needs to be asynchronous.

Great to get this input. Some comments

ghc-mod has a map-file mechanism whereby the unsaved buffer is transferred to the backend for error checking or other processing.

invalid files are a problem, but I think being able to bring the full power of the GHC parser and/or the approach taken in Yi (originally) or http://blog.haskell-exists.com/yuras/posts/incremental-lexer.html will help.

The editor context should definitely be available to plugins. How precisely to communicate available commands/features from a plugin to an editor is something we still need to get to grips with.

Same as 3

The session needs to run locally, and be fast. Hopefully this can be achieved by splitting out the comms into multiple logical channels (possibly all multiplexed onto a single one for emacs) and ensuring that the backend is always "live" in the sense of responding on channels not explicitly marked as possibly blocking.

I think I got my point across so we can close this issue.


本地跑demo   报js 错误

本地跑demo   报js 错误
npm install
npm start
http://localhost:5566/
页面空白  报js错误
index.js:8 Uncaught TypeError: Cannot read properties of undefined (reading 'common')
at<EMAIL_ADDRESS>(index.js:8:1)
at webpack_require (bootstrap:19:1)
at ./src/index.ts (global.scss?7fa5:1:1)
at webpack_require (bootstrap:19:1)
at 1 (mockService.ts:118:1)
at webpack_require (bootstrap:19:1)
at bootstrap:8

duplicate with #2184


openssh-askpass keeps poping up

VSCode Version: 1.2.1
OS Version: Ubuntu 16.04

I am not sure what is causing this, but whenever I one of my projects I keep getting this popup. It locks up everything so I couldn't do a print screen. Anyways No matter what I do, it keeps opening. If I Press Cancel or Ok it goes away then pop back up a few minutes later. If I try entering a password, or Yes/No into the input box it still comes back a few minutes later.

Very likely caused by git.

You can disable auto fetching with "git.autofetch": false in your settings.

I had this too, it kept trying to add a new host to .ssh/known_hosts but my permissions were set to 500 on that file. Change the permissions to to 600 with chmod 600 ~/.ssh/known_hosts and it should be able to add the new host.


Fcm GAPIC type conversion

Add private method that converts from public messaging.Message type to the similar but generated IMessage type.

Also sync up/rebase with the base branch so the gulpfile changes are included in the revision history. Without it we cannot run integration tests on this branch directly.
I was able to run integration tests by manually merging the gulpfile changes to this branch on my local machine:
% npm run test:integration -- --grep messaging

><EMAIL_ADDRESS>test:integration /Users/hkj/Projects/firebase-admin-node/public
> mocha test/integration/*.ts --slow 5000 --timeout 20000 --require ts-node/register "--grep" "messaging"

  admin.messaging
    ✓ send(message, dryRun) returns a message ID

Nice work 👍


Update talkback

Description

This PR updates the talkback package to the latest version. Talkback is used to save the API request responses for visual regression and e2e tests in the frontend.
Testing Instructions

The CI should pass
Checklist

[ ] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[ ] My pull request targets the default branch of the repository (main) or a parent feature branch.
[ ] My commit messages follow best practices.
[ ] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[ ] I tried running the project locally and verified that there are no visible errors.
[ ] I ran the DAG documentation generator (if applicable).

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Doesn't it affect the saved API responses? What is the usual process to generate and test them?

CI is passing so it seems like Talkback v4 is compatible with v3 response format.

Doesn't it affect the saved API responses? What is the usual process to generate and test them?

The new version did not change the way responses are written: https://github.com/ijpiantanida/talkback/blob/v4.0.0/CHANGELOG.md#v400
The usual process to generate the tapes would be to run tests with UPDATE_TAPES set to true: UPDATE_TAPES=true just frontend/test:playwright. This would only add new tapes, not replace the old ones.
You could also remove all tapes and run the tests the same way (UPDATE_TAPES=true just frontend/test:playwright). This would save all new tapes, but the some tests would fail due to different responses: for instance, the number of sources is different.


Sheet

Describe the bug
Sheet with several columns with sort and filter options. When you add movable option for rows and columns. On one column you do not have sort and filter. So when switch the place on this one with other column with filter and sort the header is moved, but the sort and filter options stayed on the same place as before. I guess this behaviour is bug.
Reproducer
No response
Expected behavior
When you move a column i expect to move with sort and filter optons as well.
PrimeFaces Extensions version
7.0.3
JSF implementation
No response
JSF version
2.0
Browser(s)
Chrome

Please add a reproducer

Updated with Stack overflow info.  My guess is these features were just never tested together as movable columns was adding way after sorting/filtering.

In a cellSelect listener sheet.getSelectedColumn() also reports wrong column index.
pe:sheet widget should do hot.toPhysicalColumn(col), because moving columns changes their visual indexes (which is what most of HOT's API uses).

PR?


Writing Unit Tests for packaged desktop apps

I have a packaged a desktop app to access some new features.
In order to test this app, this tests need to run in an app container (I think it's calld this). As far as I know there is currently only one project type that support this. Uwp test projects. But A UWP test project can't reference a .Netcore 3.1 App.
The documentation says to test your project (and I agree) but we can't only relay on manual tests for this. So how do I write unit test for this kind of projects?

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

ID: 12379f43-c827-669f-6aeb-4b4d8650cfdf
Version Independent ID: fadedb99-8dfe-7371-105a-2944bc8d9766
Content: Prepare to package a desktop application (MSIX) - MSIX
Content Source: msix-src/desktop/desktop-to-uwp-prepare.md
Product: msix
GitHub Login: @Huios
Microsoft Alias: tajimha

Hi @LokiMidgard, this issue is more relevant to the Microsoft Project Reunion, creating an issue/feature proposal here (https://github.com/microsoft/ProjectReunion) would be the right next step.
Thanks!


[enhancement] update readme

hi i use fake-kubelet before, and would like to have a try for kwok.
is there any guide for how to use kwok?
look forward for readme enhancement. :)

This work will be done after the first release


zoom-us: <IP_ADDRESS>8 -> <IP_ADDRESS>8

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for zoom-us is: zoom.us video conferencing application
meta.homepage for zoom-us is: https://zoom.us/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found <IP_ADDRESS>8 with grep in /nix/store/9b2rvn8bh9l6iyxx9lbw74j0l33m3h4s-zoom-<IP_ADDRESS>8
found <IP_ADDRESS>8 in filename of file in /nix/store/9b2rvn8bh9l6iyxx9lbw74j0l33m3h4s-zoom-<IP_ADDRESS>8

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
zoom-us

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/9b2rvn8bh9l6iyxx9lbw74j0l33m3h4s-zoom-<IP_ADDRESS>8 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A zoom-us https://github.com/r-ryantm/nixpkgs/archive/9e6a70d060a2bd7ea328cb7aa29bad10456a3d2d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/9b2rvn8bh9l6iyxx9lbw74j0l33m3h4s-zoom-<IP_ADDRESS>8
ls -la /nix/store/9b2rvn8bh9l6iyxx9lbw74j0l33m3h4s-zoom-<IP_ADDRESS>8/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

zoom-us

Maintainer pings
cc @danbst @tadfisher @doronbehar for testing.

Result of nixpkgs-review pr 220257 run on x86_64-linux 1

1 package built:

zoom-us


Removes the implementation from the test

The test was overwriting the students' solution, forcing them to update the test to make it work.

My mistake


Re-Enable num_centroids to  approx_percentile_cont

Which issue does this PR close?
Closes #777.
Supersedes #794 #785 #739 #732.
Rationale for this change
When approx_percentile_cont was migrated to a UDAf upstream, the num_centroids argument was removed from its fluent API.
This works around that by invoking the AggregateFunction::call directly.
What changes are included in this PR?
Though approx_percentile_cont is re-enabled, this actually matches the behavior of the last release.
Are there any user-facing changes?
No.

@andygrove this is ready to go, and then the datafusion-python release is also ready


Likely to get merged into A-Frame proper?

Are there any plans to submit/get this included in A-Frame natively?

Great question. I suspect the Aframe team might be working on something like that. I personally haven't heard any plans for it though.


Move button to components group

:tada: This issue has been resolved in version 0.8.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Solved issue #408

Related Issuse
Html tag is no more visible in Rules for Roots of Quadratic Equation
Closes: #408
Describe the changes you've made
A clear and concise description of what you have done to successfully close your assigned issue. Any new files? or anything you feel to let us know!
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.

Screenshots

Original
Updated

original screenshot

Well Done :)

Thank you :)


.gitlab-ci.yml running successfully but issues.md file is not created in gitlab repository

Hi, with the release of v2, the default output mode is no longer via a file (since most CI environments by convention prefer to expose outputs from the command line to users)
Possible solutions include:

using the command-line output from v2 as the primary method of feedback
pinning your configuration to use LAMS v1
passing the output parameter to v2 to get the markdown output

https://github.com/looker-open-source/look-at-me-sideways#command-line-arguments
However, if you are passing the markdown output parameter and it is not working as expected, please let me know!


Add a high-quality set of maintained examples

Each benchmark needs an example of using the benchmark with a high-quality Python script. This issue just serves to get feedback and enumerate benchmarks that need to be written

This is a WIP -- closing as issue is generic and we won't be tracking here :)


Documentation references obsolete information

The Connection quality hyperlink doesn't point to .NET (UWP) documentation but to some obsolete JavaScript documentation.
Same for Near field communications.

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

ID: cc6fe7fc-701a-54ef-dbaf-eadc7c7d3164
Version Independent ID: 173b69f0-1cfa-a5bf-6861-11a615247bb9
Content: Which networking technology?' - UWP applications
Content Source: windows-apps-src/networking/which-networking-technology.md
Product: uwp
Technology: communication
GitHub Login: @stevewhims
Microsoft Alias: stwhi

Hi, @SetTrend, and thanks for filing this issue.
I've changed both urls so that they link to the XAML variant of the topic (rather than to the JavaScript variant). Note, though, that both those variants are UWP topics (they're not .NET topics). But we  link to the XAML one by default, and we weren't doing that here.
Thanks!
Steve

Thanks, @stevewhims!
Yet, it's not only the other programming language, it also seems an issue to me that the corresponding documentation is outdated. It bears the following warning message:

We're no longer updating this content regularly. Check the Microsoft Product Lifecycle for information about how this product, service, technology, or API is supported.

Shouldn't the links rather point to current versions of the product?

Hi, @SetTrend, yes you're right that those topics are in the archive. But if we don't have anything that was written more recently, that we're updating regularly, then I think it's ok to link to the archive; those topics will still contain valuable info for Windows 10 devs.
Thanks!
Steve


Issue with g_time and gTime usage in the code

Hi @HeryMwenegoha,
Thank you for this great library! I’ve encountered an issue related to the time classes. The code predominantly uses g_time, but there are instances where gTime is also used. When I attempt to standardize everything to g_time, I encounter formatting errors, particularly with time calculations (e.g., subtraction and comparison).
Could you clarify how I should handle this ambiguity? Should I be using g_time exclusively, or is gTime meant to be used in specific scenarios? Any guidance on how to resolve the formatting errors when switching would be appreciated.
Thanks for your help!
Iulian Diaconescu.

hi, IulianDiaconescu01, have you addressed the 'gTime' problem?  I met the same problem when I tried to run  GNSS-RX on matlab2023b. The software breaks at line 68 of the "sat.m"  and reminded me of the "Unrecognized function or variable 'gTime'".


C++ coredump in LightGBM::Predictor::Predictor by GOMP_parallel

Description

This is the brief coredump stack caused by parallelly invoking LGBM_BoosterPredictForMat, which seems to be realted with openmp, how to troubleshoot this problem?

Reproducible example

Environment info
LightGBM version or commit hash:
v3.3.3
Command(s) you used to install LightGBM

Additional Comments

Thanks for using LightGBM!
A screenshot and a version number is not enough information for maintainers to help you. Please provide at least the following additional information that was requested in the issue template:

exact commands you used to install LightGBM
a minimal, reproducible example which reproduces the issue


Added topic Await and Async in Javascript

Description 📜
I have added and discussed topic JavaScript Await/Async.
Fixes Issue #721

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Doc#umentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[x] Java#Script
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@paramsiddharth pls review my PR

@Creatoon @dakshsethi @s4-2 Please review.

@daminimehra Please follow the PR format mentioned in the contribution guide: https://github.com/paramsiddharth/gwoc-js-roadmap/releases/latest/download/JavaScript.Roadmap.for.GWoC.pdf

@daminimehra Please follow the PR format mentioned in the contribution guide: https://github.com/paramsiddharth/gwoc-js-roadmap/releases/latest/download/JavaScript.Roadmap.for.GWoC.pdf

Okay Sir!

Thank you so much for the amazing PR, Damini! Sadly another PR for the same issue under documentation has been merged. If you wish to contribute further to the same issue, you can create a PR improving the merged document or add your contribution in another format. Additionally, you can create multiple PRs for different issues, or create your own based on the roadmap here: https://github.com/paramsiddharth/gwoc-js-roadmap/releases/latest/download/JavaScript.Roadmap.for.GWoC.pdf
Good luck! :) Do join our acknowledgement session on Saturdays, 7 PM! We'd love to see you there.


Replaces single quotes in vbscript with double.

String literals in VBScript require double-quotes. There may be other places in the codebase where this is an issue.

Will submit to dev branch, sorry.


Write upgrade guide from v2 to v3

We have released v3.0.0, but we don't have the upgrade guide yet.
It's better to have one even if compatibility is mostly kept.

The release notes of alpha/beta/rc/major versions might be helpful to make a new one.


do not add files in parallel

adding files in parallel might produce inconsistent hashes for the same folder. we'll also run out of file descriptors pretty fast.
off topic, similar to a tarball, you can actually add non-files to a hyperdrive archive. this is needed for things like a container but we can add that later here.

+1


Restarting Session on Every Message

When the processor receives custom metadata it
was always restarting the session.
Bug introduced on:
https://github.com/RasaHQ/rasa/pull/5611
Status (please check what you already did):

[N/A] added some tests for the functionality
[N/A] updated the documentation
[N/A] updated the changelog (please check changelog for instructions)
[X] reformat files using black (please check Readme for instructions)

Thanks for submitting a pull request 🚀 @Ghostvv will take a look at it as soon as possible ✨

Lint code is giving a false positive (again LOL), as you can see below, the pointed error doesn't exist.
import numpy as np
import pytest

from rasa.nlu.training_data import TrainingData
from rasa.nlu.featurizers.dense_featurizer.lm_featurizer import LanguageModelFeaturizer
from rasa.nlu.utils.hugging_face.hf_transformers import HFTransformersNLP
from rasa.nlu.constants import TEXT, DENSE_FEATURE_NAMES, INTENT
from rasa.nlu.training_data import Message

@pytest.mark.parametrize(
    "model_name, texts, expected_shape, expected_sequence_vec, expected_cls_vec",
    [
        (
            "bert",
            ["Good evening.", "here is the sentence I want embeddings for."],

Hey @lluchini
Thanks a lot, that fixes the problem! 👍
I think it also makes a lot of sense to add a test that would check this behavior. As we remember, in the previous PR we didn't include a test that would catch this bug.
wdyt?

Hi @degiz,

As we remember, in the previous PR we didn't include a test that would catch this bug.

We indeed included some tests, but we didn't saw this one coming.

wdyt?

I think you're completely right, but to add a test to this bug/behavior I'll need to dig more into the currents tests a see how to test effectively.
Because of the COVID-19 I'm working more from home than I was at the office. Can you wait a few more days so I can manage some time to do it?
Thanks.

Hey @lluchini

Can you wait a few more days so I can manage some time to do it?

Sure, take your time. Thanks for doing this!

Hey @lluchini
I think I'll try to wrap up this PR later today.

Hey @lluchini
I think I'll try to wrap up this PR later today.

I was planning to finish it this weekend...

Please let me know what you decided...

Hey @lluchini
Sorry for the rush, I want to make sure this fix gets in the next patch release.
I've prepared the commit here #5965, if you want you can chery-pick it here, and we will merge it as part of this PR.

Hey @lluchini
Sorry for the rush, I want to make sure this fix gets in the next patch release.
I've prepared the commit here #5965, if you want you can cherry-pick it here, and we will merge it as part of this PR.

@degiz
No problem at all!
It's me who should apologize for the delay, and I'm really very sorry!
Cherry-pick was done.

@lluchini could you please reformat the code using "black" formatter?

@lluchini could you please reformat the code using "black" formatter?

@degiz
Done!

D:\rasa>black  rasa                                                                                reformatted
D:\rasa\rasa\core\actions\action.py                                                                         All done! ✨ � ✨
1 file reformatted, 204 files left unchanged.


yarn run fails in WSL2 with "Error: spawn /mnt/c/Windows/System32/WindowsPowerShell/v1.0/powershell.exe ENOENT"

Does anyone know why and what package tries to point to powershell.exe? I'm not able to get it running in WSL 2
/usr/bin/node /usr/lib/node_modules/yarn/bin/yarn.js run dev
yarn run v1.22.17
$ vite --port 3333 --open

  vite v2.9.5 dev server running at:

  > Local: http://localhost:3333/
  > Network: use `--host` to expose

  ready in 847ms.

  > Inspect: http://localhost:3333/__inspect/

node:events:504
      throw er; // Unhandled 'error' event
      ^

Error: spawn /mnt/c/Windows/System32/WindowsPowerShell/v1.0/powershell.exe ENOENT
    at Process.ChildProcess._handle.onexit (node:internal/child_process:283:19)
    at onErrorNT (node:internal/child_process:478:16)
    at processTicksAndRejections (node:internal/process/task_queues:83:21)
Emitted 'error' event on ChildProcess instance at:
    at Process.ChildProcess._handle.onexit (node:internal/child_process:289:12)
    at onErrorNT (node:internal/child_process:478:16)
    at processTicksAndRejections (node:internal/process/task_queues:83:21) {
  errno: -2,
  code: 'ENOENT',
  syscall: 'spawn /mnt/c/Windows/System32/WindowsPowerShell/v1.0/powershell.exe',
  path: '/mnt/c/Windows/System32/WindowsPowerShell/v1.0/powershell.exe',
  spawnargs: [
    '-NoProfile',
    '-NonInteractive',
    '–ExecutionPolicy',
    'Bypass',
    '-EncodedCommand',
    'UwB0AGEAcgB0ACAAIgBoAHQAdABwADoALwAvAGwAbwBjAGEAbABoAG8AcwB0ADoAMwAzADMAMwAvACIA'
  ]
}
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Process finished with exit code 1

Found out it has to do something with yarn. Running npm run dev works

@mklueh It's because you have to tell (in settings) that IntelliJ to use Node & yarn from WSL. Otherwise, It may use the windows one.
You can configure it by searching Node in Settings, it will autodetect node if it's installed on WSL, you just have to pick it.

@mklueh It's because you have to tell (in settings) that IntelliJ to use Node & yarn from WSL. Otherwise, It may use the windows one.
You can configure it by searching Node in Settings, it will autodetect node if it's installed on WSL, you just have to pick it.

No, this is a node error, which means it is already the right path and the application is starting. Not related to the IDE. The problem is probably with the --open flag that tries to open a Browser, and assumes it's running on Windows instead of detecting WSL correctly. I had this issue now numerous times with different CLI tools such as Netlify as well.

I'm using react instead of vite for a different project but I'm facing the exact same issue with the exact same error message, may I ask how did you manage to fix it?


HDDS-7155. [DiskBalancer] Create interface between SCM and DN

What changes were proposed in this pull request?
This ticket is for creating the interface between SCM and DN. The new interfaces are as follows:

SCM -> DN

SCM send DiskBalancerCommand to datanodes to control/update diskBalancer

DN -> SCM

DN report DiskBalancerReport to SCM via heartbeat

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-7155
How was this patch tested?
unit test.

@ChenSammi @ferhui @lokeshj1703 @siddhantsangwan @sodonnel @neils-dev @JacksonYao287 Could you help to review this PR?

Hi @symious , Sammi @ChenSammi gave a comment 'Please use uint64 instead. diskBandwidth is a number without fraction. We can change it in next PR.' in the last PR. Could you change it in this PR?

Hey @symious , I finished one round of review and left some comments.

@ChenSammi Thank you for the detailed review. I will update the PR later.
For the "Double" type issue, I will create another PR in master, hope you can have a look and we can rebase and use it in the current branch. Thank you again.

@ferhui Thanks for updating the patch.
@ChenSammi Conflicts after rebased has been solved, please have a look.

@lokeshj1703 Updated the patch according to the review.
Except for the loop part, I'll come back to that part later.

@lokeshj1703 Updated the loop part, please have a check.

@ChenSammi @myskov @lokeshj1703 Thanks for your careful reviews! Do you have any other comments?

Seems no further comments. Will merge it tmr so that @symious can continue.

The patch LGTM, +1.

@symious Thanks for your contribution. @myskov @ChenSammi @lokeshj1703 Thanks for your careful reviews! Merged.

@ChenSammi @ferhui @lokeshj1703 @myskov Thank you for the review.


Capture frames from video

Is it possible to capture frames from the video ? I tried
const frame = captureVideoFramethis.player.getInternalPlayer())
but it results in captureVideoFrame is not defined no-undef.
any suggestions on how to handle the problem?

but it results in captureVideoFrame is not defined no-undef.

Make sure you include the capture-video-frame.js code somewhere in your app. Note that the jsFiddle solution in https://github.com/CookPete/react-player/issues/341#issuecomment-368820250 includes it in the Resources list.

The const frame = captureVideoFramethis.player.getInternalPlayer()) doesnt update, even after seeking to a new time in the video using this.player.seekTo()

Once I use this, I could resolve this issue.
Actual code is below:
function captureVideoFrame(video, format, quality) {
    if (typeof video === 'string') {
        video = document.getElementById(video);
    }

    format = format || 'jpeg';
    quality = quality || 0.92;

    if (!video || (format !== 'png' && format !== 'jpeg')) {
        return false;
    }

    var canvas = document.createElement("CANVAS");

    canvas.width = video.videoWidth;
    canvas.height = video.videoHeight;

    canvas.getContext('2d').drawImage(video, 0, 0);

    var dataUri = canvas.toDataURL('image/' + format, quality);
    var data = dataUri.split(',')[1];
    var mimeType = dataUri.split(';')[0].slice(5)

    var bytes = window.atob(data);
    var buf = new ArrayBuffer(bytes.length);
    var arr = new Uint8Array(buf);

    for (var i = 0; i < bytes.length; i++) {
        arr[i] = bytes.charCodeAt(i);
    }

    var blob = new Blob([ arr ], { type: mimeType });
    return { blob: blob, dataUri: dataUri, format: format };
};

Once I use this, I could resolve this issue. Actual code is below:
function captureVideoFrame(video, format, quality) {
    if (typeof video === 'string') {
        video = document.getElementById(video);
    }

    format = format || 'jpeg';
    quality = quality || 0.92;

    if (!video || (format !== 'png' && format !== 'jpeg')) {
        return false;
    }

    var canvas = document.createElement("CANVAS");

    canvas.width = video.videoWidth;
    canvas.height = video.videoHeight;

    canvas.getContext('2d').drawImage(video, 0, 0);

    var dataUri = canvas.toDataURL('image/' + format, quality);
    var data = dataUri.split(',')[1];
    var mimeType = dataUri.split(';')[0].slice(5)

    var bytes = window.atob(data);
    var buf = new ArrayBuffer(bytes.length);
    var arr = new Uint8Array(buf);

    for (var i = 0; i < bytes.length; i++) {
        arr[i] = bytes.charCodeAt(i);
    }

    var blob = new Blob([ arr ], { type: mimeType });
    return { blob: blob, dataUri: dataUri, format: format };
};

Awesome It was very helpful.
Thank you for you article.
I expect you do best continue.


the order of the log

When I trigger an action that will update a Component,
The log from the component's life-cycle methods will be print before the action log.
I thought it would be in the opposite sequence and in the time-line order.

It does. Can you show example?

Thanks for your quick response.
So I have a MessageList component. It will print componentDidUpdate on every update.
When I add a new message, and the component did updated, and print like this.

As you can  see,  the componentDidUpdate comes before  ADD_MESSAGE.
If you like some additional information. just let me know.

Same issue here, I am using the store to decide whether to show a ActivityIndicator or not.
As you can see in the screenshots below, the actual value on the store of "isLoading" is already turned to false before the action the logger prints it on the screen.
Before the action was logged (isLoading already changed).

After the action was logged

I'm seeing the same thing. Anyone found a workaround? I've tried without thunks, and only one middleware: redux-logger. Still no go.

This works:
<Button onPress={() => this.props.dispatch({type: 'APP_NAME',name: 'test-' + new Date().getTime()})} title="This works" />
This doesn't:
<Button onPress={() => setTimeout(() => this.props.dispatch({type: 'APP_NAME',name: 'test' + new Date().getTime()}), 2000)} title="This doesn't work" />

@pkl This is because redux-logger is capturing the prev state, action, and next state as the action is dispatched, then logging them all together after it is completed. Since, store subscriptions are synchronous, components will receive their new props before redux-logger does the actual logging, hence why you see you components logging before redux-logger.

You're probably right. But in my example button #2 it's no async going on, I'm basically saying wait 2 seconds before dispatching this action.
Now it seems like the component receives it's props before the action is dispatched. It makes no sense (to me :))

Get confused for the same issue now...

same here. It's so obvious to me to first log the action, then render the component that are consequence of the action, that I lost a whole day debugging to understand what I was seing in my logs.

Have this problem too

this logger works as expected
import { combineReducers } from 'redux'
import currentUser from './currentUser'

const logger = combinator => (state, action) => {
  const before = state
  const after = combinator(state, action)

  console.group(action.type)
  console.log('Before state:', before)
  console.log('Action:', action)
  console.log('After state:', after)
  console.groupEnd()

  return after
}

const mainReducer = combineReducers({
  currentUser,
})

export default logger(mainReducer)


Increase maximum suffixes for DSL methods.

Fixes #606

Coverage remained the same at 100.0% when pulling 1a6422e95d8bc31b1ae219a1416c99e71c47fd73 on indices into ea85669d3733d657b2b2ba2cc4c38d2d94fb1af2 on master.


Standardize formatting of manifests/s/Synology/NoteStationClient/2.2.3-653

Microsoft Reviewers: Open in CodeFlow

@wingetbot run


⚠ Processors has degraded performance

In fd32e87, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 59 ms

Resolved: Processors performance has improved in cd1fbaa.


<EMAIL_ADDRESS>error

Hi guys!
I am trying to use this plugin in Angular 12 but I have this error:
external "https://cdn.jsdelivr.net/npm/<EMAIL_ADDRESS>- Error: The target environment doesn't support dynamic import() syntax so it's not possible to use external type 'module' within a script
I am trying to solve this and read the 8 closed issues but I am not getting the correct answer.
Any help please?

In order to be able to handle issues more systematically, I have added a form for bug reporting. I kindly ask you to provide details about this issue as detailed below:
To Reproduce
A minimal working example that reproduces the behavior would be much appreciated.
Or else the environment in which the behavior occurs needs to be specified:

Which JavaScript environment (Node/browser/other)?
Which framework(s)?
Which devtools, including their configuration files?
package.json content
Any other context about the problem

Hi! I am just puting the import

And then, this part in the code:

I am trying to use it in browser app but it brokes when I compile the project. The red rectangle that I show you, if I don't put it or just comment that, the project compiles perfect.
What I use is Ionic 5.34.
Package.json content:

That's it. Just that. I hope I can solve this because is very urgent.
Thanks!!

And... to compile, I am using: ionic cap copy android

And more info, when I try to compile with ionic cap copy android:
external "https://cdn.jsdelivr.net/npm/<EMAIL_ADDRESS>- Error: The target environment doesn't support dynamic import() syntax so it's not possible to use external type 'module' within a script
[ERROR] An error occurred while running subprocess ng.
    ng.cmd run app:build exited with exit code 1.

    Re-running this command with the --verbose flag may provide more information.

(Image) ->

As I am not familiar with Iconic and the Angular build process, I can only give general advice here.
I suggest to bundle zbar-wasm with your app (instead of trying to load it at runtime) as explained here.
The important points with bundling are:

Have your bundler replace 'https://cdn.jsdelivr.net/npm/<EMAIL_ADDRESS>with '@undecaf/zbar-wasm' in the bundles. This config file demonstrates this for Rollup and this one for esbuild. Other bundlers will require equivalent configurations.

Let your bundler copy the file zbar.wasm to the same path as the bundle that contains the rest of @undecaf/zbar-wasm. This is also shown in the config files mentioned above.

Okay thanks, I will try this. How do I get the zbar.wasm file?

It is part of the package @undecaf/zbar-wasm in node_modules.

Closing this issue as it has been inactive for more than 4 weeks.


feat: Do not delay response handling the first time

Previously, you'd have to wait for aWriteInterval ms before the first
response handling happens.
This would also delay the creation of the spec's file.
Now, it's not delayed for the first time, just like it should.
I tested this & it works perfectly.
This is compatible with #31 & #32!
Signed-off-by: Kipras Melnikovas<EMAIL_ADDRESS>

Thanks, though it is not about response handling but about dumping specification json into a file. Response handling happens before in index.js:155

Oh yeah that makes sense; this is what I wanted - a faster dump of the json file:D


Prepare Materials for 1st SC Meeting

Title of ticket: Prepare Materials for 1st SC Meeting
Description
Meeting July 13 (TIME TBD - follow up w/ KG)
Requested - Draft Agenda, powerpoint, and potential speaker (Agile Coach?)
KP liked idea of having an Agile Coach (Matt) present to the committee giving an overview of what they can expect to see and need to focus on for typical Agile projects.
DOD

[ ] Confirm Availability of Agile Coach
[ ] Draft Agenda
[ ] Draft Slides
[ ] Sent to KG by Friday for KP Review

Thought I would include both @mpilchar and @arielleandrews on this ticket.
I was given this task this morning - still waiting to confirm actual time for the meeting on July 13th - see description above

Kris has confirmed that the SC meeting will be on July 13th from 11AM - 1145
Assuming our agile coach Matt is off this week, hopefully we can find someone from the lab that could speak to that audience for a portion of the meeting?
I'll look at one of Loren's powerpoints from the IMML committee to see if we can repurpose it for this audience, and we'll try and come up with an Agenda.
What kind of timing do we think we want from the person from the lab... 10 minutes sufficient for this audience? That can help guide any draft agenda. Maybe we put them towards the end of the Agenda so they will be available for Q and A's as well?

@arielleandrews @mpilchar - just sent a mtg invite for tomorrow to talk steering committee. I'll try and tweak some slides for the group ahead of our call, don't know if we'll need the full hour but we can fill it with other items if need be.

@m-prodan I have reached out to the lab and Jesus - re: the agile coach. Will update on this when we meet tomorrow

Materials sent to KP today to be distributed to members - all assets in place, closing ticket


the result of poolCluster.of()

the result of poolCluster.of() is different from the result of mysql.createPool().
this always course confusion.
Why  not make them the same?
thanks

Hi @bjshijh, the entire pool cluster system was a community contribution, which certainly has some oddities (for example, it came with almos no documentation).
From what I know of the pool cluster is does not return the same object because it's return object actually can consist of multiple pool objects. For example, if you have a cluster of 5 pools and call .of('*'), we cannot return the same object as mysql.createPool() because we have to return an object that can actually aggregate all those pools together, including with the necessary logic for fail over and balancing.
Do you have any suggestions on how to solve this? If the result of .of() made sense to provide that pool object, it would do that, but it currently is returning an object that is much more complex than that. If you think it will help, always feel free to even provide suggestions via. a pull request.

Hi @bjshijh, I hope the information has helped you! I didn't hear back with any suggestions in over a month, so closing the issue for now.


Gradle Install is not working

Hi All
compile 'com.github.stfalcon:chatkit:0.2.2' is added
gradle return Error:(47, 13) Failed to resolve: com.github.stfalcon:chatkit:0.2.2
Please hrlp
Regards
Taka Wong

Great.
Thanks for your effort

Hi guys
did the issue get fixed?
I still not able to install thru gradle :(
Best Regards

Hi, @takawww ! I can't reproduce this issue.


split location name and address into two columns

/ be consistent about it

Just wanted to follow this issue so I can correct the zap when it's ready

Is a change to the events schema or processing of scraped items? Right now, the events schema says the location field should have a "name" field which is the street address... I suppose that's confusing.

@r-wei yes I think it counts as a change to the events schema. Right now, the "location name" has a mix of building names and street addresses. I'd like to separate these so the address is formatted as its own column and the building name falls under "location name."
Let me know if this needs further clarification, I'm happy to refine more details.

The original spec is confusing on this point and I'm +1 for splitting this out in a more reasonable way.  We should not forget to ping the opencivicdata people about this; I don't think their event spec is in wide use and isn't as robust as some of the other data they model. It would be nice to see an update to event spec.

resolved using mapzen api


Detected Material Entity Equiv Class Restriction

It seems to me that the equiv class restriction on DetectedMaterialEntity is too narrow.
obo:BFO_0000040 (material entity) and (ro:participates_in some cco:DetectionEventSensorProcess)
For example, the material object that does the detecting also participates in this process, so would be classified as a Detected Material Entity as well. Likewise for an agentive operators of the detection artifact, etc.
Can this be relaxed to a subclass restriction?

@aellenhicks-jhuapl Thanks for the catch. Will update asap.


Test with slow network

Looks alright on slow networks. Might want to consider preloading JSON data on hover.


build: bump Rust and cargo-stylus versions

Bump:

[x] Rust to nightly-2024-07-25
[x] cargo-stylus to v0.5.3

Resolves #294
PR Checklist

[ ] Tests
[ ] Documentation

Is it ready for the review @bidzyyys ?

Is it ready for the review @bidzyyys ? I think we can merge it

Not yet, I want to find a smarter way for handling Rust version management, but tight now we have higher priorities than this.

Superseded by #375


Do not track password fields in change/submit_form events, fixes #521

This removes password fields from being tracked in form events that have been triggered as part of the enableFormTracking feature. Fixes issue #521

@miike has signed the CLA.

Awesome - thanks @miike !

Closing since merged :+1:


Initial setup of repository

Initial setup of repository
:recycle: Current situation & Problem
Currently, the repo is mostly empty except for a template package.
:gear: Release Notes

Initial setup of the SpeziChat repo which previously was a target in the SpeziML repo
Improved README and documentation via DocC

:books: Documentation
--
:white_check_mark: Testing
UI Tests are included
:pencil: Code of Conduct & Contributing Guidelines
By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

[X] I agree to follow the Code of Conduct and Contributing Guidelines.

@PSchmiedmayer Same as in https://github.com/StanfordSpezi/SpeziSpeech/pull/2, I think what's left here is a proper DOI as well as the push to the Swift Package Index (the latter one can probably only be done after merging to main, the links to the index are already set correctly, that's thy the Markdown Link Check fails).

@PSchmiedmayer PR is ready for another round of review. The only left TODO will be incorporated once SpeziSpeech is merged and tagged.

@PSchmiedmayer Similar to https://github.com/StanfordSpezi/SpeziSpeech, please merge the PR and tag a release :)

Thank you for all the setup here @philippzagar! I have merged the PR, tagged a release, and it should also start the process of creating a DOI for the repo.
Feel free to create a similar PR to Spezi Speech once all the processes are finished. Thank you for all the work in setting these things up!

Thanks @PSchmiedmayer, will add the DOI as well as SPI things soon!


fix windows double character

fix for https://github.com/rhysd/tui-textarea/issues/14
a change in crossterm 0.26 (https://github.com/crossterm-rs/crossterm/pull/745) causes key release event to be generated on windows. The input handler of textarea was treating those as additional keystrokes. The same bug was apparent in all the samples
Note I also added some logging. I did not remove it as the plumbing is useful to have (I think)

This crate must support both crossterm v0.25 and v0.26
Logging code with simplelog is not related to issue which this PR want to fix

(I hope i have updated PR correctly - still git noob)
Logging in samples removed
I tested all relevant samples with tui (cross 25) and ratatui (cross 26)

@pm100 There are still some unnecessary changes. But it's OK. I'll cherry-pick necessary part of your changes.


Fix for the thread contention issue

* Changed retry during reading from result table to use exponential backoff
* Used ForkJoinPool instead of FixedThreadPool
* Added more logging

what's the motivation for usgin ForkJoinPool instead of FixedThreadPool?

ForkJoinPool is designed to be used for CPU-intensive workloads.
A customer suggested using it to improve situations when we run too many reading threads inside of one task.


Issue/5125 add compile time metrics collector (#5277)

Pull request opened by the merge tool on behalf of #5277

Merged in 85b39f6257fbb9c41c10f39e98c56bf6f074dbc1


Add PropertyNameGenerator as an optional Parameter

Is your feature request related to a problem? Please describe.
Currently, CodeGeneratorSettings does not provide an option to define a custom PropertyNameGenerator. In my projects this does not match the PascalNaming convention, as the properties are named like this:
[System.Text.Json.Serialization.JsonPropertyName("business_partner")]
public int Business_partner { get; set; }

[System.Text.Json.Serialization.JsonPropertyName("main_consultant_number")]
public int Main_consultant_number { get; set; }

Describe the solution you'd like
I would like to see an option in CodeGeneratorSettings that allows for the integration of a custom PropertyNameGenerator like the given sample below:
//Refitter.Core/CSharpClientGeneratorFactory.cs
internal class CSharpClientGeneratorFactory(RefitGeneratorSettings settings, OpenApiDocument document)
{
	public CustomCSharpClientGenerator Create()
    {
		var csharpClientGeneratorSettings = new CSharpClientGeneratorSettings
		{
			CodeGeneratorSettings = { PropertyNameGenerator = settings.CodeGeneratorSettings.PropertyNameGenerator ?? new CustomCSharpPropertyNameGenerator(), },
		};
	}
}

//Refitter.Core/Settings/RefitGeneratorSettings.cs

/// <summary>
/// Provide settings for Refit generator.
/// </summary>
[ExcludeFromCodeCoverage]
public class RefitGeneratorSettings
{
	/// <summary>Gets or sets the property name generator.</summary>
	[JsonIgnore]
	public IPropertyNameGenerator PropertyNameGenerator { get; set; }
}

Describe alternatives you've considered
NSwag unfortunately does not meet our logging requirements, as we need the method names from the RestMethodInfo.
Additional context
If there is any other way to customize the property names please let me know.
/cc @christianerbsmehl

@fsamiec Thanks for taking the time to report this. Unfortunately, I'm too busy with other things to look into this. If you find a solution yourself and create a pull request, then I will find the time to review and make sure your fix gets merged in and released. Otherwise, this will have to wait a few weeks
/cc @christianerbsmehl

thank you for merging. 👍
i will add another pull request, based on your feedback.


Readme updates

Changes proposed by this PR

Fixes broken GIF link and updates Office Hours agenda link

Release Note

PR Checklist
Note: Please do not remove items. Mark items as done [x] or use ~strikethrough~ if you believe they are not relevant

[ ] Linked to a relevant issue. Eg: Fixes #123 or Updates #123
[ ] Removed non-atomic or wip commits
[ ] Filled in the Release Note section above
[ ] Modified the docs to match changes

Still runnning validation tests


 There are multiple use-cases for this, the most 'pressing' being killing ephemeral or disposable peers. This would entail adding the REST API endpoint and command-line interfaces. The question is, is this a good idea? Is it 'safe'?

Closing this as it is no longer needed.


#3823: Demo file for mistral TT model

This Branch contains Demo file for Mistral TT model.

Here is the passing CI Links
[post-commit] Run all post-commit workflows Link
Scheduled slow dispatch frequent build and run Link


http3: sniff HTTP Content Type from first Write, analogous to http.ResponseWriter

This should be pretty straightforward change which can mostly be copied from the standard library HTTP implementation. See https://github.com/caddyserver/caddy/issues/5096#issuecomment-1268784887 for more discussion and links to code.

Hello @marten-seemann, does this issue still need help? If so, would you please take a look at my PR?


Add SageMaker RL Tic-Tac-Toe Example

Issue #, if available:
None
Description of changes:
The example notebooks already available for SageMaker RL are great, but I was hoping to create one that used a custom Gym environment for a very simple task that required almost no background knowledge (playing a game of tic-tac-toe).  In addition, I wanted to focus the evaluation of the output on the tangible interaction with the endpoint in a game-like setting (where RL is common).
If the cost of maintaining this example would be greater than the value of having it, feel free to leave unmerged and close the pull request.
Thanks!
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I'm satisfied with the code quality and I've verified it works properly in a SageMaker notebook instance. Local mode is not included in the code by default, but it is easy to switch by changing the training instance type. The game play with a hosted endpoint is compelling.
Approving the PR


Manually emit onElementsRemove

Currently whenever i hit on delete btn onElementsRemove is being emitted is there a way i cound click some button (html button on element/node) on a element/node and make it emit onElementsRemove.
Thanks in advance.

onElementsRemove is just the event handler when you delete an element by pressing the key that is configured by deleteKey. You could also use a button or any other user action to alter your elements array. To get the selected elements you can listen to the onSelectionChange event.

@moklick hello, do you have an example of delete by button?


single child codepath errors, so use multi-child codepath instead

Thanks for this great tool! I just needed to comment out a few lines to get it to work for 1 child.

Thanks for this. I only ever had two children in the system so no opportunity to test the single-child codepath.


Add support for Carthage

Carthage - A simple, decentralized dependency manager for Cocoa by the Github team

Ignore Carthage/Build as a file in .gitignore

Carthage will create that folder as a symlink, so it has to be ignored as a file

Make project build-able for Carthage

Add project file
Share scheme

I am using Carthage in my own project and using it to manage AMScrollingNavbar.
PS.
Great work. Thank you for all the time this project saved me.

Thank you @NicholasTD07
I've been meaning to add Carthage support in the next few days, this PR is more than welcome :)
:+1:

@andreamazz My pleasure. :D


Sprint 1

Here's what I think would be good for you to get done in the first sprint, which will last a little over 1 week. Let's say this sprint will end June 12th. Wednesdays are the best days for me to do a mentorship session with you. Essentially for each Sprint, we'll just take some of the items from here: https://github.com/lastmjs/software-issues/issues/119 And we'll try to get through the ones that make sense by the end of the Summer.

[x] HTML and CSS

[x] HTML & CSS is Hard: No 1
[x] HTML & CSS is Hard: No 2

[ ] JavaScript

[x] Reading

[x] Eloquent JavaScript: Introduction
[x] Eloquent JavaScript: Chapter 1

[x] Videos

[x] Crockford on JavaScript - Volume 1: The Early Years

[x] Exercises (talk to me before you install Node.js)

[x] Nodeschool: JavaScripting

[x] Personal project

[x] Decide on what project you want to pursue

Completed sprint 1 - ready for sprint 2!

Nice, how did the Nodeschool exercises go?

Really well! I only had to get help with one of them. It was a really good overview. I'd love more practice in some exercises for loops and functions to really fully grasp those.

Is it possible to get a list of sprint 2? I'd love to keep going! I'm off today and tomorrow from work but would still love to continue on learning!

Sorry, I posted it earlier today. Let me know if you have any questions


fix docker 128 error

Fixes #19814
Internal review URL
Per @travisgosselin

In the ASP.NET Core world driven inside a container, there is next to zero reasons I can think of in which I would want to enforce reloading of that configuration file. Once my container is built and pushed to its container repository with an immutable version it does not change (only environment variables on deploy modify its behavior). In this straightforward scenario, there is very little reason to turn on “reloadOnChange”. Turning it off, lead to slightly less number of occurrences of this problem.

@travisgosselin writes

The documentation is not great in telling you the default.

Just  above Combining service collection
Using the default configuration, the appsettings.json and appsettings.Environment.json files are enabled with reloadOnChange: true. Changes made to the appsettings.json and appsettings.Environment.json file after the app starts are read by the JSON configuration provider.
See #22392

@BrennanConroy  can you recommend a reviewer?

While this addresses the first and simplest change to help reduce the 128 user limit, I think its additionally necessary to address the potential root user usage in the Dockerfile and encourage the usage of other users on their runtime image given scaling implications when deployed to a single host with multiple container instances running (random user option described in https://github.com/dotnet/AspNetCore.Docs/issues/19814). Additionally using a non-root user in your Dockerfile also has security implications on what ports are available by default to expose (i.e. port 80 not accessible by default..).

@danmoseley  @danmoseley @ericstj  @pranavkm @glennc @spboyer  this is a serious problem Docker folks are frequently hitting that  needs to be documented how to avoid. See #19814

@danmoseley @danmoseley @ericstj @pranavkm @glennc @spboyer @BrennanConroy  3 minute review  this is a serious problem Docker folks are frequently hitting that needs to be documented how to avoid. See #19814


Another font for icons

Hi,
i am tring change font of icon by class, but not working:
<script type="text/javascript" src="../system/vendor/nette/forms/src/assets/live-form-validation.js">
<script type="text/javascript">

    LiveForm.setOptions({
        messageErrorPrefix: ' <i class="fa fa-exclamation-triangle" aria-hidden="true"></i> '
    });

</script>

Is right?
Thanks

Hi,
I think that should work. What's the real result? Some error or it uses original prefix? Or...?

Yea, it show original Options, without change.

I tried it in my test project and it works correctly. Are you sure you have most recent version? Download e.g. from https://github.com/Robyer/nette-live-form-validation/archive/v1.6.1.zip

Thanks, i was have 1.5.x version. Now it works.


Does not work (out of the box) in a minified app

Exception in thread "main" kotlinx.serialization.SerializationException: Serializer for class 'SarifSchema210' is not found.
Mark the class as @Serializable or provide the serializer explicitly.
        at kotlinx.serialization.internal.Platform_commonKt.serializerNotRegistered(Platform.common.kt:91)
        at kotlinx.serialization.internal.PlatformKt.platformSpecificSerializerNotRegistered(Platform.kt:29)
        at kotlinx.serialization.SerializersKt__SerializersKt.serializer(Serializers.kt:60)
        at kotlinx.serialization.SerializersKt.serializer(Unknown Source)
        at io.github.detekt.sarif4k.SarifSerializer.toJson(SarifSerializer.kt:23)

The proguard/r8 rules actually exist in https://github.com/Kotlin/kotlinx.serialization/blob/master/rules/common.pro, but only since 1.5.
#43 would solve this immediately, workaround:
implementation("io.github.detekt.sarif4k:sarif4k:0.5.0")
implementation("org.jetbrains.kotlinx:kotlinx-serialization-json:1.6.3")

We could update it to the latest version, or 1.5.0 at least. We are at 0. version so this shouldn't be a huge problem.


Server-side rendering guide needs more clarification

Hi, I have checked docs several times, but still can't figure out next questions:

Do I need to include fonts and katex.js files if I am doing server side rendering (i.e. renderToString)? I am including katex.min.css file and it looks fine for simple examples, however not sure if it's enough for complex math stuff.
Is it possible to completely inline styles while rendering server side (like shiki does)?

Thanks in advance for the answer.

Fonts should be included on the client side. Therefore, complete inlining is not possible at this time. (HTML style attribute doesn't support @font-face)
katex.css should also be included on the client side.
katex.js is not needed on the client side if the rendering is done on the server side.


Have info print conmon/oci runtime information

We need into to identify the OCI runtime and conmon used by podman.
Signed-off-by: Daniel J Walsh<EMAIL_ADDRESS>

@baude PTAL this is a followon to your earlier patch to add path and package version.
@vrothberg Could you tell me how to implement the equivalent of rpm -qf /usr/bin/runc with apt?

@lsm5 if you know how to do this with apt, I would appreciate your input also.

@rhatdan For Debian-based distributions you can use dpkg -S $PATH. However, I am a bit concerned about this PR as distributions may choose different paths (e.g., /usr/sbin/runc on openSUSE). I would personally prefer to execute something like runc --version and parse that.

@rhatdan scratch my previous comments regarding different paths. I should have had a closer look at the code, and as we're getting the paths from the runtime, everything looks good to me.

@vrothberg Added a patch for dpkg support

LGTM (checked on openSUSE)

I'm a little hesitant to start going through package versions manually - where do we stop? Arch is a popular distro, should we be including pacman in there?
LGTM, but I remain hesitant.

Please wait before merge, want to test on Debian/Ubuntu before.

@rhatdan I tested it via make shell and found that we need to check for != "Uknown" to make it work. Once that's changed, feel free to merge.

take it or leave it comment, else LGTM

I will leave it up to a someone from Arch to ask for this.  I think this information is critical to make debugging issues easier.  And when this gets into RHEL for support people to know what people are using.

@rh-atomic-bot r=baude


Manual distribution of FMUs over the network

It should be possibe to pass an URL to a running proxy-server, which will subsequently download and make the FMU available over the network.

Feature has been added


Added support for linked readme files.

Hello there!
I needed all other readme pages linked to the main one compiled with typedoc, so I added support for this. This code is not perfect and unfortunately I don't have time to work on this further so someone will have to take over. PM me.
But this works nice till now. Note that I have set the default themes to be installed from the forked repository atm. Also note that media is not handled by this implementation yet, only the readme files.

I am not sure why this is breaking, it passes all tests locally.

Thanks for sharing your changes. Unfortunately this PR needs some updating to be mergeable including some changes to the approach. For now I am going to close this but if you would like to update it, please leave a comment and we can discuss what changes need to be made. Thanks!


Formfields in a Form lose values on scroll/kb popup

You can see this with the 'Text fields' sub-app in the 'Gallery' app.  Pop up a keyboard, enter values, scroll around.  Happens on iOS and Android.  Hidden/scrolled out fields get set back to their initial value(s).
This seems to be a bug or incomplete implementation.  I would think a normal expectation would be to create the form and whatever values are entered in the fields while the form is active should remain until the form is either submitted or dismissed (or initial values explicitly restored.)  You shouldn't have to do anything out of the ordinary to preserve values through scrolling/keyboard popups or whatever else might inadvertently destroy the fields while using it (the form.)
If otherwise, the demo needs to be beefed up to show how to preserve values through scrolling/hiding/other operations while the form is active.

Yup. Dupe of #11500.


Add support for CFN APIGW changes from Sept. 2018

Description:
CFN recently announced additional support for some APIGW resources. We need to add support in SAM for these new changes. Also see the forums for information about the recent update.
In particular, we would like to add the (new) fields AccessLogSetting and CanarySetting to AWS::ApiGateway::Stage (docs) inside of SAM as well as decide if any of the other new features should be supported by SAM.

+1

Released!


package the repository so that it can be installed

I'd like to use this project.
Rather than copying the .py files to my own project though, I prefer to install it.
I've setup the repository so that it can be installed with pip.
I hope this is helpful to you too.

hey! thank you so much! I will try to look at this during the weekend


How to catch dev dependencies that actually need to be included in the distributed presentations of Meltano?

Just got bit by this with packaging in https://github.com/meltano/meltano/pull/6276:
It's a transitive dependency required by tox, but tox is only a dev dependency. So, in CI tests, things look good because dev dependencies are installed by poetry install. But, when meltano is installed from an upstream source or by another tool like pipx, which is the case in end-to-end tests here (pipx install ..), then the transitive dev dependency packaging is not installed and errors are encountered: https://gitlab.com/meltano/legacy-ci/meltano/-/jobs/2632994539#L1237.
$ meltano add orchestrator airflow
2022-06-23T19:10:30.061252Z [info     ] Environment 'dev' is active
No module named 'packaging'

I guess one easy way to ensure we don't fall in that scenario is to have tests that don't rely on Poetry to install Meltano.

Export the lockfile from from poetry to pin deps in a requirements.txt as part of the release process? 🤔

@pandemicsyn and @edgarrmondragon - What do you think of this being a part of

#6218?

At least when running in CI, we could use a pipx-installed version of Meltano instead of the Poetry-installed one.

At least when running in CI, we could use a pipx-installed version of Meltano instead of the Poetry-installed one.

Probably a better discussion for #6218, but sounds fine  - at some point we'd probably want a test mirroring the contributor guide which might end up using poetry.

The SDK had this exact issue in the v0.6.0 release so I went ahead and started a PR with nox-poetry, which runs tasks against the installed form of the package using the wheel.
https://github.com/meltano/sdk/pull/780


New version: Zimwiki.Zim version 0.75.0

- [X] Have you signed the Contributor License Agreement?

[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.5 schema?

Microsoft Reviewers: Open in CodeFlow

/AzurePipelines run


How protractor can detect javascript errors ?

Hello,
I start doing my first tests and I was wondering how protractor can make the test failed if an error is raised in the javascript console.
I start my tests by just loading the page, and tests results where successfull.

But when I'm loading the page manually and check the console, I got those errors :

Can anyone help me to understand how to make the test fail in this case ?
Thanks
Bobby

@hanssgo & @juliemr
I have tried your solutions to get the JS error from browser console. The issue is that it shows all the errors from the console and I want to check the JS error only. The "type" argument from the browser log is showing empty. Is there anyway I get filter the JS Errors only?


Input type file not working

None of the examples on file input in the documentation works. Neither does it work when downloaded.
However, the the examples on file input in the documentation for the PRO version works fine.

i am having the same issue

i am having the same issue

having the same issue

i added this to my base js file that runs across the website
$(".inputFileVisible").click(function(){
    $(this).siblings("input[type='file']").click();
});
$(".inputFileVisible").focus(function(){
    $(this).siblings("input[type='file']").focus();
});
$(".inputFileHidden").change(function(){
    if(this.files.length > 0){
        $(this).siblings(".inputFileVisible").val(this.files[0].name);
    }else{
        $(this).siblings(".inputFileVisible").val("");
    }
});

+1

Hello guys,
Thank you for your interest in working with our products, this issue is solved in the latest version (V2.0.4).
All the best,
Stefan

I'm using version 2.0.4 and I'm still having this issue.

Hello again,
it was a misunderstanding, this feature is available only for the PRO version, it was mistakenly left in the documentation, also it had not been added in the product. Sorry about this!
*we've updated the documentation for the Free version
All the best,
Stefan

Hey Stefan
Thanks for replying. So is there any file input styling for the free version?
Thanks
Tony


Programmatically sending input from the client

Hi
How can I programmatically send input from the client? Doing something like:
term.writeln('ls -al');
will write the text to the terminal, but it doesn't send it to the server (even when hitting return afterwards).
Thanks!

You can access the javascript Terminal object via window.term.

Thanks for the quick reply. My issue is not how to access the Terminal object; that seems to work fine - the problem is that using term.write() displays the text on the terminal UI, but never sends it to the server.
For example, in the screenshot below, I first sent the "select version()" command using the Javascript console. That never gets sent to the server. If I then type the same command into the terminal window, it works fine and I get the version info I expect.

example code to send ls to the terminal:
var script = document.createElement('script');
script.src =<EMAIL_ADDRESS>script.onload = function() {
	var input = document.querySelector('textarea');
	var keyboard = Keysim.Keyboard.US_ENGLISH;
	keyboard.dispatchEventsForInput("ls", input);
	keyboard.dispatchEventsForAction("enter", input);
};
document.body.appendChild(script);

Yes, that works. Thanks!


Will the code be open sourced soon?

Hello, I am very interested in your research content. I am very much looking forward to the code being open sourced soon, it will be a great help in my work and I will be very grateful！

Recent FSOD methods have significantly superior performance to this work, and they have released codes, why not choose other methods?


fix: format code snippets properly

Code snippets that are separated by new blocks were not treated with the same tab_space, resulting in inconsistent formatting between the blocks. To make this consistent, the parser will now collect all of the code snippet first, then process all of them at once with a uniform tab_space value that was calculated when the code block is encountered at first.
Modified the unit test to grab the tab_space like how the parser will compute it. Also added an additional unit test for testing code snippet with blocks and refactored the existing unit test to use parameterized style instead.
Fixes b/219067629.

[x] Tests pass

Thank you! Filed #194 to keep track of converting rest of comments to the docstring format.


Yarn action for i18n string replacement

https://github.com/Kuadrant/kuadrant-console-plugin/issues/116


Continuous integration

So if I got this right, currently the continuous integration publishes a new nuget version for every pushed commit?
If so, I suggest creating another branch called "development" and merging when we are ready to ship a new version.
We should also use something like SemVer for version numbers and maybe a changelog.
What do you think?

So if I got this right, currently the continuous integration publishes a new nuget version for every pushed commit?

You're right. The reason I added continuous integration was to quickly respond to users wishes.

If so, I suggest creating another branch called "development" and merging when we are ready to ship a new version.

Yes, this is no longer avoidable.

We should also use something like SemVer for version numbers and maybe a changelog.

If there are no new protocols, you're right. We can use.


gh-2378 updated references

Related Issue

Resolve #2378

Codecov Report

Merging #2379 (25109b6) into develop (3fe0943) will increase coverage by 9.72%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #2379      +/-   ##
=============================================
+ Coverage      66.13%   75.85%   +9.72%
+ Complexity      2464      185    -2279
=============================================
  Files            958       11     -947
  Lines          31404      584   -30820
  Branches        3820       50    -3770
=============================================
- Hits           20769      443   -20326
+ Misses          9006      100    -8906
+ Partials        1629       41    -1588

Impacted Files
Coverage Δ
Complexity Δ

.../uk/gov/gchq/gaffer/commonutil/ByteBufferUtil.java

...gaffer/federatedstore/FederatedStoreConstants.java

...andler/impl/FederatedAddGraphWithHooksHandler.java

...chq/gaffer/store/operation/handler/MapHandler.java

...ffer/operation/impl/compare/ElementComparison.java

...ffer/serialisation/util/JsonSerialisationUtil.java

...er/named/view/serialisation/TypeReferenceImpl.java

...gaffer/store/operation/resolver/ScoreResolver.java

...accumulostore/retriever/AccumuloItemRetriever.java

...andler/javardd/ImportJavaRDDOfElementsHandler.java

... and 920 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #2379 (25109b6) into develop (3fe0943) will increase coverage by 9.72%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #2379      +/-   ##
=============================================
+ Coverage      66.13%   75.85%   +9.72%
+ Complexity      2464      185    -2279
=============================================
  Files            958       11     -947
  Lines          31404      584   -30820
  Branches        3820       50    -3770
=============================================
- Hits           20769      443   -20326
+ Misses          9006      100    -8906
+ Partials        1629       41    -1588

Impacted Files
Coverage Δ
Complexity Δ

.../uk/gov/gchq/gaffer/commonutil/ByteBufferUtil.java

...gaffer/federatedstore/FederatedStoreConstants.java

...andler/impl/FederatedAddGraphWithHooksHandler.java

...chq/gaffer/store/operation/handler/MapHandler.java

...ffer/operation/impl/compare/ElementComparison.java

...ffer/serialisation/util/JsonSerialisationUtil.java

...er/named/view/serialisation/TypeReferenceImpl.java

...gaffer/store/operation/resolver/ScoreResolver.java

...accumulostore/retriever/AccumuloItemRetriever.java

...andler/javardd/ImportJavaRDDOfElementsHandler.java

... and 920 more

Continue to review full report at Codecov.

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


Fix macos test

What does this PR do?
Tested on 100 runs on my machine, stopped failing. Before it was at 50%
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


VS Code hangs when setting bind property in nshtml files

Visual Studio Code: 1.28.2 (1.28.2)
dotnet --info
.NET Core SDK (reflecting any global.json):
Version:   2.1.403
Commit:    04e15494b6
Runtime Environment:
OS Name:     Mac OS X
OS Version:  10.14
OS Platform: Darwin
RID:         osx-x64
Base Path:   /usr/local/share/dotnet/sdk/2.1.403/
Host (useful for support):
Version: 2.1.5
Commit:  290303f510
.NET Core SDKs installed:
2.1.403 [/usr/local/share/dotnet/sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.1.5 [/usr/local/share/dotnet/shared/Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.5 [/usr/local/share/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.5 [/usr/local/share/dotnet/shared/Microsoft.NETCore.App]
Steps to reproduce

Create a public class in a .cs file
add a public property in the class
create a page component in a .cshtml file
add @inherits and name the class
add a <input type="text" bind="@prop"/> element

Expected behaviour
VS Code should keep running.
Actual behaviour
After typing @ the entire VS code environment freezes.  After a period of the time, the OS gives a dialog box saying the window has stopped responding, and offers to close, continue waiting, or reopen.
Workaround:
type the @prop string somewhere else, and then copy/paste it into place.

Yes I've experienced this issue as well, although it keeps happening in regular asp.net core project as well so it has nothing to do with Blazor. I suggest opening an issue in the code repository for VS Code!
Best regards

Also I have experienced the behavior with both 2.1.403 and 2.2.100-preview3-009430 so it's not directly related to your runtime either.

Thanks for the post. Since we don't currently have Blazor support in VS Code, it's not specifically expected to work. This should all start to work properly once VS Code gains Blazor support, which will hopefully be in the next month or two.

This issue was moved to aspnet/Razor.VSCode#229


Feature Request: Java module compatibility

for details: https://github.com/conveyal/kryo-tools/issues/3

Thanks for the feature request, but I have no idea what you're asking for.
I'm not planning to spend time to figure out what it is that you would like to have. Please take some effort to explain this or otherwise I'm just going to close this feature request.

I sincerely apologize for the oversight. It escaped my attention that there is an existing project with an identical name. https://github.com/vertical-blank/sql-formatter

That's pretty interesting info. Apparently that project is a Java port of this JavaScript library, which in turn is a port of a PHP library.

That's pretty interesting info. Apparently that project is a Java port of this JavaScript library, which in turn is a port of a PHP library.

都是非常不错的 SQL 格式化项目，收藏了。


Contribute skyAppLinkExternal

We need to navigate to an other SPA but maintain service and environment context. So, we need to sidestep the Angular router but still get the same behavior as skyAppLink to maintain the runtime context for the SPA.

First commit for review on overall approach before writing tests

Codecov Report

Merging #366 into master will decrease coverage by 0.05%.
The diff coverage is 93.33%.

@@            Coverage Diff             @@
##           master     #366      +/-   ##
==========================================
- Coverage   99.22%   99.17%   -0.06%
==========================================
  Files          71       72       +1
  Lines        1685     1700      +15
  Branches      264      267       +3
==========================================
+ Hits         1672     1686      +14
- Misses         13       14       +1

Flag
Coverage Δ

#builder
100% <ø> (ø)
:arrow_up:

#runtime
94.06% <93.33%> (-0.05%)
:arrow_down:

#srcapp
100% <ø> (ø)
:arrow_up:

Impacted Files
Coverage Δ

lib/sky-pages-module-generator.js
100% <ø> (ø)
:arrow_up:

...time/directives/sky-app-link-external.directive.ts
93.33% <93.33%> (ø)

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 06bc08e...9651764. Read the comment docs.

@Blackbaud-PaulCrowder, I addressed your feedback and have written tests against the new approach, which leverages most of the url logic behind RouterLinkWithHref

Hey @Blackbaud-MitchellThomas this looks good to me.  Looks like there's a conflict with master now.  Can you get that addressed?  We can then get it approved and merged in.

@Blackbaud-PaulCrowder Can you confirm the current state meets your "changes requested?"


Alvastia Chronicles [PCSG01243]

App summary

App name: Alvastia Chronicles
App serial: PCSG01243
App version: 01.00

Vita3K summary

Version: v0.1
Build number: 1395
Commit hash: https://github.com/vita3k/vita3k/commit/76189aa

Test environment summary

Tested by: IllusionMan1212
OS: Windows 10 Pro
CPU: Intel Core i5 7200-U
GPU: AMD Radeon 520
RAM: 8 GB

Issues
Unicorn crash because it's a unity game
Screenshots
None at the moment
Log
vita3k.log

Vita3K v0.1.5 2609-653b0fa1
stuck on the first developers screen
vita3k.log

as of https://github.com/Vita3K/Vita3K/commit/c453f24cd9b995d76f8d5f81f4631ab31d6f3de9, the game can go ingame (although i cannot past thru the character screen because i can't understand japanese.)
Screenshots

Logs
vita3k.log
Recommended Labels

Ingame +
unity


Add approle authentication

Hi,
I added approle authentication method for vault : https://www.vaultproject.io/docs/auth/approle
I modified the fastify version supported by your plugin in order to be able to use it with fastify 4.x.x

Hey :wave: , thanks a lot for the pull request. Will take a look, but haven't worked with vault since writing this plugin. So i'll take your word that you have tested it :sweat_smile:


⚠ GitLab Server has degraded performance

In b84c382, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 7107 ms

Resolved: GitLab Server performance has improved in 2eadd19.


Engine won't start unless previous version is still installed

This is on macOS. The current version is 1.0.0-RC2 and the previous version is 1.0.0-RC.
If I delete, rename or move the previous version in the trash the engine won't start (there is no Java process). If the previous version is kept were it is, the current version works. I can also delete the previous version and download and install it at the exact same place and it will work too.

Oh damn, yes I can confirm. I just deleted the previous version and the engine won't start…

If I remove the "pipeline-ui" directory in "~/Library/Application Support" (after I put the 1.0.0-RC app in the trash) , the engine starts correctly.

@NPavie can you have a look at this one?

Yes

@marisademeglio i found the problem.
I was storing and reusing the pipeline and jre pathes from previous install, it works on windows as the package name had not changed so the application install folder remained the same (but its not the case on macos as app are bundled with their displayed name).
I fixed that by removing the use of the settings' path for this (this was a relicate of when i was testing the use of a system-wide pipeline 2 install with the app).

Also, this should fix a side effect i had not come to think of :
I found out you can launch the app from the DMG, and if i change a setting in this app running from the DMG, it would break the app the same way.

Fixed


Add Vue.js questions / answers

Add questions and answers that might test core Vue concepts and mimic questions that might be asked in phone screens. Should be concise enough to fit into 1-2 sentences. Fill in here under 'Vue' deck  and submit pull request.
Example of question types / length

o/ I'll take care of this.

Great additions by @omegavesko . Still need many more - thanks all


idris2: prefer chez 10 over chez-racket

Description of changes
Prior to version 10.0.0, Chez Scheme did not support Apple Silicon. This lead to Idris 2 bootstrapping with the Racket fork of Chez Scheme on Darwin platforms. As of v10.0.0, Chez Scheme has upstreamed enough of the Racket fork that the Racket team has stated it will not need to maintain the fork going forward.
This PR switches to using Chez Scheme for all platforms as long as v10.0.0+ is available.

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 297348 run on aarch64-darwin 1

2 packages built:

idris2
idris2Packages.idris2Lsp

Result of nixpkgs-review pr 297348 run on x86_64-darwin 1

2 packages built:

idris2
idris2Packages.idris2Lsp

Result of nixpkgs-review pr 297348 run on aarch64-linux 1

1 package built:

idris2 (idris2Packages.idris2)

I'm slightly confused as to why on aarch64-linux nixpkgs-review didn't try to rebuild the LSP package, but I'm not concerned that the LSP package wouldn't build given it doesn't directly need Scheme to build.

@fabianhjr may as well get this in before the next nixpkgs release.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1624


Rescan control

I needed to rescan today, but I accidentally selected the default start date (Oct 2018) instead of the correct much more recent birthday. “Cancel Rescan” only lets me pause the rescan as far as I can tell, but I would like to restart the rescan from the proper date and skip a few extra years of scanning.
Also, I think having date selection is a good UI, but in this case I had the birthday block handy from another source, so I actually had to convert that back to a date rather that just provide the exact block.

You can reset the app in this case. Canceling the rescan would leave the app in an undesirable state.


Update documentation about caching of unit tests

Fixes outdated documentation on caching of unit tests.
Context
Previous documentation referenced https://issuetracker.google.com/issues/79122161?pli=1 which is duplicate of https://issuetracker.google.com/issues/115873047 which is completed.
Contributor Checklist

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

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

@bot-gradle test ReadyForNightly

OK, I've already triggered the following builds for you:

ReadyForNightly build

@bean5 Could you please ensure that your commits are signed-off?

Hello @bean5,
Unfortunately, your commit lacks a sign-off, while Gradle only accepts signed-off commits.
If you accept the developer certificate, please update the commit by adding a sign-off and submit the pull request again. Due to the lack of activity for some time, I'm closing this PR. Feel free to comment if you think we should reopen it.

@h0tk3y I will submit the pull request again.

Re-submitted at #28737


Need 'brush_transfer' func for 3.0/3.1

Godot version:
3.1  alpha
Issue description:
Miss function in Image Class compared with Godot 2.1.

Why was it removed in Godot 3?


How to set proxy for chatgpt-reborn?

Describe the feature
How to set http proxy for chatgpt-reborn?

You could use Cloudflare to proxy OpenAI's API, and change the baseurl in the extension's settings.
const TELEGRAPH_URL = 'https://api.openai.com';

addEventListener('fetch', event => {
  event.respondWith(handleRequest(event.request))
})

async function handleRequest(request) {
  const url = new URL(request.url);
  url.host = TELEGRAPH_URL.replace(/^https?:\/\//, '');
  const modifiedRequest = new Request(url.toString(), {
    headers: request.headers,
    method: request.method,
    body: request.body,
    redirect: 'follow'
  });
  const response = await fetch(modifiedRequest);
  const modifiedResponse = new Response(response.body, response);
  modifiedResponse.headers.set('Access-Control-Allow-Origin', '*');
  return modifiedResponse;
}

Thank you for your reply. I configure http.proxy in vscode. And it worked


Is Llama 3.3 model actually Llama 3.1?

What is the issue?
Hi! When I was reading the logs while using llama3.3 (pulled with ollama pull llama3.3) I saw the model name was Llama3.1 (see screenshot).

Is this expected?
Thanks!
OS
Linux
GPU
Nvidia
CPU
Intel
Ollama version
0.5.2-rc2

#8010

Thanks @rick-github . Closed as duplicated of https://github.com/ollama/ollama/issues/8010


Minor typo the title for a link

The first paragraph of the article instructs the following to search for the package: Enter one of the package names below to search for it online. But one of the package name given is Microsoft.UI.Xaml.Controls. And when you search for such a name in VS2017 package Manager, a package with this name does not exist. I think, the title should read as Microsoft.UI.Xaml. However, the link for the title is correct.

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

ID: 716795b6-2bcc-8ff5-f999-5e1de88a04e0
Version Independent ID: 590dd55a-a259-e6dc-972e-531dda080cd9
Content: NuGet package list for the Windows UI Library - Windows UWP applications
Content Source: winrt-related-src/toolkits/winui/nuget-packages.md
Product: uwp
Technology: winrt-reference
GitHub Login: @mijacobs
Microsoft Alias: mijacobs

Thank you, @saf-itpro! I checked in a fix, it should be live later today.