Playwright e2e tests have broken dependency due to dependabot updates

Bug description
Ref actions fail run: https://github.com/containers/podman-desktop-extension-bootc/actions/runs/10269395422/job/28414736482?pr=710
Operating system
Everywhere
Version
next (development version)
Steps to reproduce
No response
Relevant log output
Run yarn add -D @podman-desktop/tests-playwright@next
yarn add v1.22.22
[1/4] Resolving packages...
warning Lockfile has incorrect entry for<EMAIL_ADDRESS>Ignoring it.
warning Lockfile has incorrect entry for<EMAIL_ADDRESS>Ignoring it.
[2/4] Fetching packages...
[3/4] Linking dependencies...
warning "workspace-aggregator-87e5ce8b-f396-44f7-a576-3a905fab220e ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "workspace-aggregator-87e5ce8b-f396-44f7-a576-3a905fab220e ><EMAIL_ADDRESS>has incorrect peer dependency "<EMAIL_ADDRESS>warning "workspace-aggregator-87e5ce8b-f396-44f7-a576-3a905fab220e > frontend ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^5.0.0".
warning "workspace-aggregator-87e5ce8b-f396-44f7-a576-3a905fab220e > frontend ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning "workspace-aggregator-87e5ce8b-f396-44f7-a576-3a905fab220e > frontend ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning "workspace-aggregator-87e5ce8b-f396-44f7-a576-3a905fab220e > frontend > @sveltejs/vite-plugin-svelte ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^4.0.0".
warning " ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>error Invariant Violation: expected workspace package to exist for "@typescript-eslint/utils"
info Visit https://yarnpkg.com/en/docs/cli/add for documentation about this command.
    at invariant (/usr/local/lib/node_modules/yarn/lib/cli.js:2318:15)
    at _loop2 (/usr/local/lib/node_modules/yarn/lib/cli.js:91560:9)
    at PackageHoister.init (/usr/local/lib/node_modules/yarn/lib/cli.js:91619:19)
    at PackageLinker.getFlatHoistedTree (/usr/local/lib/node_modules/yarn/lib/cli.js:48551:20)
    at PackageLinker.<anonymous> (/usr/local/lib/node_modules/yarn/lib/cli.js:48562:27)
    at Generator.next (<anonymous>)
    at step (/usr/local/lib/node_modules/yarn/lib/cli.js:310:30)
    at /usr/local/lib/node_modules/yarn/lib/cli.js:328:14
    at new Promise (<anonymous>)
    at new F (/usr/local/lib/node_modules/yarn/lib/cli.js:5539:28)
Error: Process completed with exit code 1

Additional context
No response

I believe this was caused by #699, which bumped the eslint-plugin.

@deboer-tim that is correct. But I still do not understand why.

@benoitf suggested the problem is at yarn (1.2.2)[https://stackoverflow.com/questions/64060218/why-does-yarn-throw-invariant-violation-expected-workspace-package-to-exist-w]. We could probably add particular version (the latest/next) manually into package.json and then download deps. Simply do not use yarn add is a workaround.


Multi-line labels: express left, center, right justification (top/bottom too for vertical writing?)

major remaining issue: FlexSymbolizer also needs to read Justify and emulate flex-start, flex-end based on child bounding boxes


feat(prometheus): serialize resource as target_info gauge

Which problem is this PR solving?
This PR updates the PrometheusExporter to export Resource as a gauge-typed target_info metric according to the specification here.

When serializing resources, service.name and service.namespace are dropped from the resource attributes and used to construct the job label.

If job is already present in the resource, it is dropped instead.

When serializing resources service.instance.id is dropped from the resource attributes, and its contents are added to the instance label.

If job is already present in the resource, it is dropped instead.

Fixes #867
Short description of the changes
Export Resource as a single gauge-typed target_info metric.
Type of change

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

How Has This Been Tested?

[x] Added Unit tests
[x] Adapted existing tests.

Checklist:

[ ] Followed the style guidelines of this project
[ ] Unit tests have been added
[ ] Documentation has been updated

I think the lint failure was intermittent


rename/rebrand (opinionated)

In my humble opinion "image maniac" is not a very good name for an image viewing program. I'm not going to motivate that, and leave it for you to decide.
We can discuss possible alternative names here.

Agree, let think a better name here.

eye of bevy, aka. eob


enable custom "prop" to breakpoints

We are having trouble using styled-system v5 with material-ui because createMuiTheme will create the following breakpoint as an object

{
  "breakpoints": {
    "keys": [
      "xs",
      "sm",
      "md",
      "lg",
      "xl"
    ],
    "values": {
      "xs": 0,
      "sm": 600,
      "md": 960,
      "lg": 1280,
      "xl": 1920
    }
  },
}

can we have a prop on theme that says the key of breakpoints?
const theme = {
   customBreakpoints: 'ssBreakpoints',
   ssBreakpoints: [],
}

Material UI uses its own custom fork of Styled System, but you could probably add appropriate keys to the theme.breakpoints object that it returns that could work with Styled System.
E.g.
theme.breakpoints[0] = '40em'
theme.breakpoints[1] = '52em'
/// etc...

This library follows the Theme Specification, and it's not really in the interest of this effort to allow customization of the theme shape here

@oliviertassinari any way to use theme specification on material-ui?

@sibelius This is an interesting interoperability issue, I wasn't aware of it, thanks for raising. I have been keen on improving the breakpoint structure of the Material-UI themes since we first introduced it in v1 alpha. I'm not very happy with the way it can be used/customized right now. @jxnblk is right, Material-UI system started as a fork of smooth-ui's system, authored by an ex-coworker/friend to match our existing theme structure, also influenced by the great work done on styled-system.
While changing the theme structure can be a significative issue from a backward compatibility perspective for the developers using Material-UI, I still think that we have some room for changes and innovation in the version 5 (probably in May 2020).
Here is an idea to explore, we could be using the same structure as Bootstrap: https://getbootstrap.com/docs/4.3/layout/grid/#grid-tiers, (most of our users are coming from Bootstrap, we are doing our best to leverage common conventions developers have already learned).
const theme = {
  breakpoints: {
    values: {
      xs: 0,
      sm: 600,
      md: 960,
      lg: 1280,
      xl: 1920,
    },
    up,
    down,
    between,
    only,
  },
}

So dropping the keys and width property, it's redundant. It's also similar to Uber approach: https://baseweb.design/components/block/#responsive-layouts. We might even be able to introduce the change without a breaking change in v4 :).
I'm not sure how we could help in this case 🤔.

any way to use theme specification on material-ui?

I want each breakpoints value to be named. I don't see how an array structure is rich enough. Is there a way to do it?
I'm wondering, how are styled-components users writing media queries right now? We document the following in Material-UI, could we make it simpler?:
import styled from 'styled-components';

const StyledButton = styled.button`
  padding: 8px 12px;
  border: 1px solid;
  cursor: pointer;
  outline: none;
  font-size: 18px;
  ${props => props.theme.breakpoints.up('md')} {
    font-size: 16px;
  }
`;


[Snyk] Security upgrade io.streamnative:pulsar-broker from <IP_ADDRESS>-rc5 to <IP_ADDRESS>

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `maven` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

pom.xml

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

616/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 5.9
Denial of Service (DoS) SNYK-JAVA-COMFASTERXMLJACKSONCORE-3038426
io.streamnative:pulsar-broker:  <IP_ADDRESS>-rc5 -> <IP_ADDRESS>
No
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Denial of Service (DoS)

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


Add API to return licenses as a map in addition to list

I see that the license list is actually using a map internally, but the API returns a list Map::values(). Is it possible to add an API which also returns a Map with key String licenseId and value SpdxListedLicense? This would be more performant rather than having to recreate a map from the list.

Sounds like a useful API - if you'd like to create a PR, I can review merge.  Otherwise, I'll take a look at this after implementing the SPDX 3.0 support.

@dwalluck - Finished with the SPDX 3 upgrade - can you point me to the API that returns the list?


Client() stuck on listening for microphone.

I am using, Eleven labs, Text Gen UI and it's API, Whisper Live for transcription, but it seems to constantly listening to my microphone and  it never send the messages:
while True:
    user_message = client() # get the result of the transcription it's stays stuck here.

    history.append({"role": "user", "content": user_message})
    data = {
        "mode": "instruct",
        "stream": False,
        "messages": history
    }

    response = requests.post(url, headers=headers, json=data, verify=False)
    assistant_message = response.json()['choices'][0]['message']['content']
    history.append({"role": "assistant", "content": assistant_message})
    print(assistant_message)

Is there a way I can make it stop listening if there is silence after a certain amount of time?? Or something like that? Thanks

@JohnMama12 did you test the whisper-live client & server alone, without eleven labs and text gen ui? if not please do and if yes please let us know what issues were there?

Yes, I've tried it separately and it works fine, I just need to eventually stop listen after 2 secs or so.

@makaveli10  ?

@JohnMama12 we want to listen always if we want to transcribe everything, no?
Not sure how your chatbot setup looks like but we have used it in a chatbot assistant setup here => https://github.com/collabora/WhisperFusion and WhisperLive works fine for us.
If you can details the issue maybe we can understand better the issue with listening always.

@JohnMama12 we want to listen always if we want to transcribe everything, no? Not sure how your chatbot setup looks like but we have used it in a chatbot assistant setup here => https://github.com/collabora/WhisperFusion and WhisperLive works fine for us. If you can details the issue maybe we can understand the issue with listening always.

Thanks, but do you know if it would be possible not to use the default chatbot? would like to implement the chatbot externally, as I currently have using a server.

@makaveli10 WhisperFusion looks cool, but don't think it will work for me. I don't have a dedicated GPU for this and I'm running windows.  I don't need speech synthesizing. If you didn't understand my issue clearly, the issue is that yes it transcribes it nearly in real time, but I need it to eventually stop listening after a couple secs.  As I can't have have a conversation with the chatbot as it never stops listening. Do you understand my issue now?

@JohnMama12 you could just stop sending data if you get the eos notification:
https://github.com/collabora/WhisperLive/blob/8c36768f7f043e559c32b587ee35df2625c0fc7e/whisper_live/server.py#L184
I would also suggest to run the different services in threads. We can provide a more clear example based on the eos message that I mentioned, if you need it.
We use the end of sentence detection (eos) for WhisperFusion as well, to trigger the LLM part.

@JohnMama12 you could just stop sending data if you get the eos notification:
https://github.com/collabora/WhisperLive/blob/8c36768f7f043e559c32b587ee35df2625c0fc7e/whisper_live/server.py#L184
I would also suggest to run the different services in threads. We can provide a more clear example based on the eos message that I mentioned, if you need it.
We use the end of sentence detection (eos) for WhisperFusion as well, to trigger the LLM part.

Thanks, a step forward but it  still doesn't do what i want. I added a small delay after client() and then put self.clients[websocket].set_eos(True)
It seems to be listening once, but itstops listening right after the last thing I say but still  it doesn't go to the next piece of code and send the message...


 That's on me for not properly reviewing your last docs change.

@nataliescottdavidson it didn't give a very obvious error message, and it's pretty hard to preview the templates so no worries. But I do think the email will be better than the link so that's an improvement :)


Feature Request: eject load-cookie.html from the PUC repo, form new repo

eject the cookiesync and load-cookie logic from the puc; it is unrelated and we would like to be able to build and release one without the other.

kicked back to high, ready for dev to publish the docs


Release machinelearningservices microsoft.machine learning services 2024 04 01 preview

ARM (Control Plane) API Specification Update Pull Request
PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

[1] ARM review queue (for merge queues, see [4])
The PRs are processed by time opened, ascending. Your PR may show up on 2nd or later page.
If you addressed Step 1 from the diagram and your PR is not showing up in the queue, ensure the label ARMChangesRequested
is removed from your PR. This should cause the label WaitForARMFeedback to be added.
[2] https://aka.ms/azsdk/support/specreview-channel
[3] List of SDK breaking changes approvers in pinned Teams announcement
[4] public repo merge queue, private repo merge queue (for ARM review queue, [1])
If you need further help with anything, see Getting help section below.
Purpose of this PR
What's the purpose of this PR? Check all that apply. This is mandatory!

[x] New API version. (Such PR should have been generated with OpenAPI Hub).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix swagger quality issues in S360.
[ ] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to Step 2, "ARM Review", for this PR.

Breaking changes review (Step 1)

If the automation determines you have breaking changes, i.e. Step 1 from the diagram applies to you,
you must follow the breaking changes process.
IMPORTANT This applies even if:

The tool fails while it shouldn't, e.g. due to runtime exception, or incorrect detection of breaking changes.
You believe there is no need for you to request breaking change approval, for any reason.
Such claims must be reviewed, and the process is the same.

ARM API changes review (Step 2)

If this PR is in purview of ARM review then automation will add the ARMReview label.
If you want to force ARM review, add the label yourself.
Proceed according to the diagram at the top of this comment.

Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.
Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
Swagger-Suppression-Process
to get approval.
Getting help

For general PR approval workflow, see the diagram at the top of this comment.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

/azp run

Note to ARM reviewer: I have added a lintdiff approval to this PR because the check is timing out and not expected to complete. This is because of the number of files that need to be run thru this process. The reviewer needs to manually review all changes as they cannot rely on the automated checks.

FYI ARM Reviewer: This PR is a batching of API changes so we can introduce the version into main as a unit. All the changes in this PR are pre-approved.

@ms-henglu , The comments from yesterday have been addressed with this PR https://github.com/Azure/azure-rest-api-specs/pull/28460 . Could we now get the sign off on this PR please?

There's still a comment not responded: https://github.com/Azure/azure-rest-api-specs/pull/27840#discussion_r1539923808

@ms-henglu - This is the PR to address https://github.com/Azure/azure-rest-api-specs/pull/28495
the comment. Could you please review?

There's still a comment not responded: #27840 (comment)

/pr RequestMerge


Update index.md

I added a Trezor Wallet explanation and an Option C: Blue Wallet for the Emergency Recovery

There must be no newline inbetween a list, otherwise pandoc will not handle them correctly. Please remove all line breaks such that it looks like:
1. ...
1. ...
1. ...

There must be no newline inbetween a list, otherwise pandoc will not handle them correctly. Please remove all line breaks such that it looks like:
1. ...
1. ...
1. ...

Back to the drawing board :)
I noticed Blue Wallet only let's users import keys while connected to the Internet.
That's a bad idea.
It could work in an extreme emergency, but  it's not good enough for the guide.

Back to the drawing board :)
I noticed Blue Wallet only let's users import keys while connected to the Internet.
That's a bad idea.
It could work in an extreme emergency, but  it's not good enough for the guide.

Oh bummer, unfortunate they don't support an airgap :(
Closing for now, please reopen/edit if you disagree.

Oh bummer, unfortunate they don't support an airgap :(
Closing for now, please reopen/edit if you disagree.


Add capabilties to use crtmqm flags such as (-fs, -lp, -ls , -lc , -lf,  -h,  -u, -p)

Is your feature request related to a problem? Please describe.
As part of the create queue manager task, there must be a way to configure the queue manager at creation time with the file system size, the number of primary secondary logss, logs file sizes, and so on.
Describe the solution you'd like
I'd like to be able to configure the queue manager at creation time with all the parameters I require for my environment.
Describe alternatives you've considered
Not to use the create queue manager role and resort to using shell commands to create the queue managers that way I need them.
Additional context
This also falls inline with adding RDQM capabilities. As of today, my team and I have developed several roles to build an RDQM environment. I'd have loved to see this as part of the mq-ansible repo.

I agree that there should be the capability to create queue managers with these flags - in addition to those needed to create RDQM type queue managers.  I am just getting into learning Ansible, so I am far from being qualified to provide any kind of meaningful input.  But, could variables be set for each of these flags that have null values if they are not needed and if needed, set with both the flag and the value, e.g. "lfsize: -lf 65535", if needed or "lfsize: ", if not needed?  And then the crtmqm command would look something like "crtmqm {{ lfsize }} QM1.  That way the user has full control in determining when a non-default flag is used.


Dashboard UI Fixes - SEAB 4712/SEAB 5198/SEAB 4713

Description
Mix of Dashboard related UI fixes so let's dive in:
My-Sidebar:
Dividing line was missing the purple highlighting when visiting the dashboard page.  The routerLink needed to be updated, and cleaning up I removed the [routerLinkActiveOptions]="{ exact: true }" since that was needed originally when the dashboard button would link to the homepage (ie. routerLink="" ).

Before
After

Added the "User Guide" link to the sidebar:

Before
After

Navbar:
"My Dockstore" button was using the wrong icon, should be the smaller light purple one:

Before
After
Mocks

Small updates to the new dropdown, bolded text, new "Log out" icon, colour changes and some additional padding.

Before
After
Mocks

Looking at it now, there is no line in the mocks, so this mat-divider should be removed:
https://github.com/dockstore/dockstore-ui2/blob/385cce574745a7d079138a907900750be6f06650/src/app/navbar/navbar.component.html#L105
I'll wait for any additional feedback before pushing anything.
Organization Box:
Pending request/invitation icons should be greyed out if equal to zero.

Before
After

Noticed pending requests/invitations weren't updating so I changed them to observables, updating as expected now.
https://github.com/dockstore/dockstore-ui2/blob/385cce574745a7d079138a907900750be6f06650/src/app/home-page/widget/organization-box/organization-box.component.ts#L23-L24
https://github.com/dockstore/dockstore-ui2/blob/385cce574745a7d079138a907900750be6f06650/src/app/home-page/widget/organization-box/organization-box.component.ts#L42-L43

Empty card was updated with the circle org icon:

Before
After

Added the search bar (similar implementation to entry-box.component), changed "Recent Activity" to "My Organizations", and switched from using the organization Name to the DisplayName.

Before
After

Changes to the Org box still aren't complete, we have a ticket to add a new API that returns a user's organization events (SEAB-5266) and a follow-up ticket to use that API and display a user's organization events following the mocks (SEAB-5267).
Last change, the pending requests/invitations links were updated to go to the "Requests" tab of the account page:
routerLink="/accounts" [queryParams]="{ tab: 'requests' }"
Same for the link on this organization pop-up which previously linked to the default tab on the accounts page:

Finally, just an overview of the current state of the Dashboard:

Review Instructions
Issue
SEAB-4712
SEAB-5198
SEAB-4713
Security
If there are any concerns that require extra attention from the security team, highlight them here.
Please make sure that you've checked the following before submitting your pull request. Thanks!

[x] Check that your code compiles by running npm run build
[x] Ensure that the PR targets the correct branch. Check the milestone or fix version of the ticket.
[x] If this is the first time you're submitting a PR or even if you just need a refresher, consider reviewing our style guide
[x] Do not bypass Angular sanitization (bypassSecurityTrustHtml, etc.), or justify why you need to do so
[x] If displaying markdown, use the markdown-wrapper component, which does extra sanitization
[x] Do not use cookies, although this may change in the future
[x] Run npm audit and ensure you are not introducing new vulnerabilities
[x] Do due diligence on new 3rd party libraries, checking for CVEs
[x] Don't allow user-uploaded images to be served from the Dockstore domain
[x] If this PR is for a user-facing feature, create and link a documentation ticket for this feature (usually in the same milestone as the linked issue). Style points if you create a documentation PR directly and link that instead.
[x] Check whether this PR disables tests. If it legitimately needs to disable a test, create a new ticket to re-enable it in a specific milestone.

Updates to the organization-box.component following Charles' feedback: totalOrgs remains constant,  removed forEach(), added this.listOfOrganizations = []; in the error block, and added a "No matching search results." text when returned result is empty.
Quick demo of the new behaviour:
https://user-images.githubusercontent.com/97123241/220383586-f7864e86-949e-4bd8-a348-7d10616a465a.mp4
Also made a followup ticket SEAB-5268 to update how search is handled in the entry-box.component to match.

Last change: "User Guide" updated to "Quickstart":


Fix: Hot Reload Blocking

Hot reload loop wasn't correctly handling the socket stream when closed.
Closes #2123

Thank you!!!


Adds ability for explorer to sort file numbers in natural order.

Adds ability for explorer to sort file numbers in natural order, example 10 comes after 2.

@id-aditya please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree [company="id-aditya"]

@microsoft-github-policy-service agree company="id-aditya"

@id-aditya the command you issued was incorrect. Please try again.
Examples are:
@microsoft-github-policy-service agree

and
@microsoft-github-policy-service agree company="your company"

@microsoft-github-policy-service agree

@microsoft-github-policy-service agree

agree

@lioman @lramos15 Took hints from the previous PR, what's the chance you can help me publish this?
This PR sorts file in explorer from this:

base16.tsx
base32.tsx
base64.tsx
base8.tsx

To this:

base8.tsx
base16.tsx
base32.tsx
base64.tsx

Thanks

@roblourens

@bpasero

@lramos15
Explanation of the "explorer.sortOrderNumeric" Feature
In the current version of VS Code, the "explorer.sortOrder": "filesFirst" setting allows for sorting files and folders in a specific order but does not naturally sort files that contain numbers. For example, files like file1.txt, file2.txt, and file10.txt are sorted as file1.txt, file10.txt, file2.txt, which is not intuitive for many users who expect numerical order.
To address this issue, I have introduced a new setting called "explorer.sortOrderNumeric". When this option is set to true, files that contain numbers will be sorted in a natural numeric order. With this setting enabled, files will be sorted as file1.txt, file2.txt, file10.txt, providing a more intuitive and user-friendly experience.
Here’s how the settings would work together:

"explorer.sortOrder": "filesFirst": This setting still controls the general sorting behavior of files and folders.
"explorer.sortOrderNumeric": true: This setting applies a natural numeric sort order to the files.

This addition aims to enhance the usability of file sorting in the Explorer, particularly for users who work with files named with numbers.

Update on the Sorting Issue
Initially, I was using "explorer.sortOrderLexicographicOptions": "unicode", which caused issues with the sorting order of files. After switching to "explorer.sortOrderLexicographicOptions": "default", the sorting problem was resolved.
Given that the default lexicographic option addresses the issue, the need for this PR might be reconsidered. However, I’m leaving the final decision to the maintainers on whether to close the PR or keep it open for potential future scenarios where "unicode" sorting might still be preferred.
Thank you for reviewing this contribution.

Closing for now as we hope to make explorer sorting extensible in the future rather than continuing to expand the core sort options.


Spawnmenu

[Wiremod] lua/wire/client/customspawnmenu.lua:676: Tried to use a NULL Panel!
1. GetParent - [C]:-1
2. unknown - lua/wire/client/customspawnmenu.lua:676

Could you provide steps onto how to replicate that?


Move CatalogKey, ColumnMetadata, ColumnMap, CatalogTable to dbt-common

resolves #

Description

Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[x] I have run changie new to create a changelog entry

Codecov Report
Attention: Patch coverage is 0% with 18 lines in your changes are missing coverage. Please review.

Project coverage is 64.04%. Comparing base (ddb28aa) to head (18fb414).

Files
Patch %
Lines

dbt_common/contracts/metadata.py
0.00%
18 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #147      +/-   ##
==========================================
- Coverage   64.39%   64.04%   -0.35%
==========================================
  Files          51       51
  Lines        3134     3151      +17
==========================================
  Hits         2018     2018
- Misses       1116     1133      +17

Flag
Coverage Δ

unit
64.04% <0.00%> (-0.35%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add RHEL nodes

CentOS was doing the jobs but it was missing to link for Red Hat
distribution and the choise to don't use epel repository.
Tested and works on RHEL 7.3
Signed-off-by: Cyril Lopez<EMAIL_ADDRESS>

I'm confused about how centos_repository would be set to an empty string on a RedHat node.

@smuth4, so the name should not so clear...  It's to choose to witch repo will be use for redis and rabbimq. It continue to be epel by default but you can define centos_repository as an empty string to don't enable other repo.
`-      enablerepo: epel

 enablerepo: "{{ centos_repository }}"`

Ah, I thought you were saying disabling epel was necessary for RedHat to work, but now I see you're actually adding two features. Thanks for the PR!


Minor edits of comments

Nothing bad about fill, we just use the other way everywhere else in the code.


Transformers should be annotated with codemeta.json files

Please describe your request. Is your feature request related to a problem?
Problem is that we don't have a consistent way of annotating our algorithms with metadata, and therefore, they aren't machine searchable, e.g. by schema.org. Further, this can provide more complete information to Zenodo (e.g. authors and intent etc) when we archive released versions of a transformer.
Also, there is a lot of hard-coding of information that could be put into a json file and used from there
Describe the solution you'd like
Have users put algorithm information into a codemeta.json file, and perhas have the template populate some fields automatically when a new transformer is created.
See https://codemeta.github.io/
Additional Info
codemeta doesn't support the workflow information (inputs, outputs, etc) (e.g. in the NIST plugin.json) but maybe a single file could contain both the codemeta w/ jsonld and the workflow info

Related to https://github.com/zenodo/zenodo/issues/1504 ;-)


chore(npm): sync package.json from npm-package-template

engines, version, release scripts for semantic-release, mocha, jsdoc and istanbul

Current coverage is 90.90%

Merging #6 into master will not affect coverage as of 4c4d7e3

@@            master     #6   diff @@
=====================================
  Files            1      1
  Stmts           11     11
  Branches         1      1
  Methods          0      0
=====================================
  Hit             10     10
  Partial          0      0
  Missed           1      1

Review entire Coverage Diff as of 4c4d7e3

Powered by Codecov. Updated on successful CI builds.


Dark mode & Light mode

Hi, first of all awesome library you have made here :) just having an issue when dark mode or light mode changes while app is running or in background.
I have colours set to change either white or black, colours will change but the selected radio button is always the wrong colour.
For instance radio buttons are all set white (dark mode enabled). User now changes to light mode, radio buttons change to black, however when you select a radio button which ever one is selected will be the old colour in this case white making it appear invisible. Example pictures below of how this looks vs how it should look without the bug.
The issue only happens when switching between modes while app is running otherwise programmatically setting the colour does not reproduce the bug.

Solved. Turns out I had not been setting the colour on each property - Doh!
Fixed it by setting the colours as needed on below properties.
.setTitleColor(.white, for: .normal)
.indicatorColor = .white
.tintColor = .white
.iconColor = .white


[FLINK-19832][coordination] Do not schedule shared slot bulk if some slots have failed immediately

If a physical slot future immediately fails for a new SharedSlot in SlotSharingExecutionSlotAllocator#getOrAllocateSharedSlot but we continue to add logical slots to this SharedSlot, eventually, the logical slot also fails and gets removed from the SharedSlot which gets released (state RELEASED). The subsequent attempts to add logical slots in the loop of allocateLogicalSlotsFromSharedSlots will fail the scheduling
with the ALLOCATED state check because it will be RELEASED.
The subsequent bulk timeout check will also not find the SharedSlot and fail with NPE.
Hence, such SharedSlot with the immediately failed physical slot future should not be kept in the SlotSharingExecutionSlotAllocator and the logical slot requests depending on it can be immediately returned failed. The bulk timeout check does not need to be started because if some physical (and its logical) slot requests failed then the whole bulk will be canceled by scheduler.
If the last assumption is not true for the future scheduling, this bulk failure might need additional explicit pending requests cancelation. We expect to refactor it for the declarative scheduling anyways.

Maybe the state check in SharedSlot.allocateLogicalSlot is wrong? Maybe we simply need to return a failed logical slot future if the internal state of the shared slot is not Allocated?

Thanks for the review @tillrohrmann.
I agree we do not need this Allocated state check in SharedSlot.allocateLogicalSlot.
I decided to try removing it and splitting the shared and logical slot allocation as we discussed offline.
This looks to me simplifying everything because we use all shared slots (already failed or not) to start the bulk check.
The bulk will be just stopped immediately if any physical slot has already failed.
If this is still too complicated or no time, I will change it to the precondition that the physical slots are not expected to fail immediately.

I've added a test case for the problem and rebased on the latest master. Once AZP gives green light, I will merge this PR.


Version solving failed due to dart SDK

I'm trying to install the stripe_payment package when I use the most recent version (stripe_payment: 0.1.0) I get the following error

When trying to get stripe_payments(0.0.8) I get an unresolved reference error

Flutter Doctor

The version of the Dart SDK in the stable release of flutter is 2.1.0-dev.9.4 which is technically not >= 2.1.0. I do not know if it was the intention of the stripe_payment package to restrict usage of the package from the stable release, but it should be raised on https://github.com/jonasbark/flutter_stripe_payment.


Exclude examples from distribution

This PR suggests exclusion of examples directory from git distribution package (so, no example dir in vendors, except if installed with prefer source)

This change is

Thanks!


Add cx-policy to the default context

We'll have a new namespace for legal definitions of parts of the policy.
Also the ODRL profile well be coming from this namespace.
"cx-policy": "https://w3id.org/catenax/policy/"

Also it is easier for the consumers, when using the given context, they terms are easier to read instead of expanded IRIs.
Can this please be added to the default context to also appear in the catalog?
Thanks,
Matthias

To look at adding this, we would need a valid Json-Ld context resolvable from that URL.

I don't really see why. As I understand, the proposal is to add a prefix declaration, not the inclusion of a remote context - more like "edc": "https://w3id.org/edc/v0.0.1" instead of "http://www.w3.org/ns/odrl.jsonld"

What terms are defined in the context? We need at least a valid Json-Ld context somewhere.

@matgnt - I understand this feature request as merely aimed at contraction of IRIs "https://w3id.org/catenax/policy/Membership" to cx-policy:Membership. There's no remote vocabulary to be pulled in, right?

Yes, correct @arnoweiss : NO remote context is required for this.

OK, if there are no terms to add, then it is fine to add the prefix definition as is.

Closed by https://github.com/eclipse-tractusx/tractusx-edc/pull/1201


[prim_flop_en] Dependency fix

Signed-off-by: Michael Schaffner<EMAIL_ADDRESS>

This is related to #10805.

LGTM!


[plot] Another behavior change with silx 0.10.0

This one has been discovered by BCU colleagues using PyMca on the RNICE cluster or on windows.,

When clicking on the toggle points/lines/pointandlines icon any curve having points and lines is not shown.

This is again affecting PyMca https://github.com/vasole/pymca/issues/430 because the default visualization mode for SCAN data is lines with points and nothing is shown on the plot.

PyMca is almost useless now on RNICE.

It seems to be related to https://github.com/silx-kit/silx/commit/9c65671c13998efe320a85550be8a9e7fed1f155#diff-1c496ea291692f8f461862a24f8d5181
By one hand there is a break of compatibility by setting the DEFAULT_PLOT_CURVE_SYMBOL to nothing (an empty string or a space) instead of a circle.
That does not explain why requesting line and symbol one gets nothing.

The getting nothing is because there is another change making of just maker to take precedence over line+maker and that also affects PyMca.
Setting back DEFAULT_PLOT_CURVE_SYMBOL = 'o' solves most issues (except the fact that one does not get the line plus point as first choice in PyMca).

Solves most issues but then the tests do not pass.


iOS TinyWebDB needs to warn of invalid values

Describe the bug
From the forum: The iOS version of TinyWebDB does not appear to do anything when using a custom TinyWebDB server. However, on further investigation it appears that the values were not JSON-encoded, which the iOS version expects.
At the moment, we log the error using NSLog, but this isn't particularly valuable to an end-user of App Inventor. Instead, we should either call WebServiceError or dispatch an error via the form, so that the information can be properly communicated to the user via the blocks.
Affects

[ ] Designer
[ ] Blocks editor
[x] Companion
[ ] Compiled apps
[ ] Buildserver
[ ] Debugging
[ ] Other... (please describe)

Expected behavior
When the contents of TinyWebDB are not parseable as JSON, a user-visible error should be provided.
Steps to reproduce

Set up a TinyWebDB instance
Populate key-value pairs where the value is not valid JSON
Try to read the values on iOS and observe that nothing happens in the app

Hey @ewpatton, I had a look into the issue and would like to work upon solving it. Would you please assign this issue to me


⚠ ClamAV has degraded performance

In e82a5d4, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 329 ms

Resolved: ClamAV performance has improved in 034a394.


Add GDT_Int8, test GDAL 3.7 on CI and refactor workflow

[x] I agree to follow the project's code of conduct.
[x] I added an entry to CHANGES.md if knowledge of this change could be valuable to users.

Closes #403

bors try

r? @jdroenner

bors r+

@lnicola 🙇 🙇 🙇


gulp build not working

gulp build results in [08:59:38] No gulpfile found

It was working. gulp outdated.


Initial UBI9 images for EAP 8.1

JDK17 and JDK21 images on UBI9 for EAP 8.1
Behave tests of this ubi9 branch are currently running there, both are green:

http://brontes05.lab.eng.brq2.redhat.com/job/imgtest/job/eap8.1JDK17UBI9images/3/
http://brontes05.lab.eng.brq2.redhat.com/job/imgtest/job/eap8.1JDK21UBI9images/5/

Still some references to osbs, we will replace that with konflux integration

@jmesnil , this should be good to merge.

@jfdenise thanks!


Range partitioning for native parallel batch indexing

Motivation
Currently, native parallel batch indexing (#5543) only supports hashed partitions. Add support for range partitioning so that native parallel batch indexing has feature parity with hadoop batch indexing.
Proposed changes
For native parallel batch indexing hash partitioning, ParallelIndexSupervisorTask.runMultiPhaseParallel() runs two phases:

PartialSegmentGenerateTask: Each worker takes a firehose split and creates partial segments by hashing rows from the split
PartialSegmentMergeTask: Each worker merges the set of partial segments for each partition

For range partitions, the PartialSegmentMergeTask can be reused, but partial segment generation needs a different algorithm:

Each worker takes a firehose split and generates an approximate histogram in memory that records the frequency of each key (timestamp adjusted by query granularity and dimension value tuple) in the split. In particular, quantiles.ItemsSketch from DataSketches can be used as the implementation of the approximate histogram.

The most accurate ItemsSketch of k=32768 (which also uses more space) has a normalized rank error of 0.009% and for extremely large streams (>1B items), retains no more than 600k items. Assuming each item is about 1KB (~500 characters in the dimension value), the histogram will use a max of about 600MB memory, so it would not need to be periodically flushed to disk. I’m not sure what serialized size this translates to, but it’s possible that it’s too large to add as part of the task report payload, in which case it’d need to be written to disk and then copied to the supervisor.
- k can be exposed as a configurable value in the ingestion spec, in case users wish to reduce the memory consumption
NOTE: Using ItemsSketch will require moving DataSketches from extensions to core, which may block https://github.com/apache/incubator-druid/pull/8647

The supervisor gets all the partial ItemsSketches generated earlier (either via the TaskReport payload or by copying the serialized ItemSketches from disk) and then merges them by using quantiles.ItemsUnion. From the merged ItemsSketch, it uses ItemsSketch.getQuantiles() to creates partition ranges that satisfy the criteria specified in SingleDimensionPartitionsSpec.

Because ItemsSketch yields estimated quantile values, enforcing maxRowsPerSegment in the partitionSpec can only be done during PartialSegmentMergeTask when the final merged segments are created.

Each worker receives the list of partitions ranges and for each firehose split, creates partial segments by using the partition ranges.

NOTE: In this initial version of implementing native parallel batch range partitioning, specifying the partition dimension is required (unlike hadoop batch indexing).
Rationale
Some other alternatives considered were:
Exact Histogram
Because an exact histogram would use much more memory than an approximate histogram (e.g., one from DataSketches), it would need to be periodically flushed to disk. To facilitate subsequent merging of histograms without consuming too much memory, the histogram would need to be serialized in sorted order so that a heap-merge can later be used. The main disadvantage of the exact histogram approach is reduced performance due to the amount of disk I/Os.
Operational impact
None expected.
Test plan
Unit tests and integrations tests, similar to the ones for native parallel batch indexing hashed partitioning.
Future work
Automatic Partition Dimension Selection
This is a niche feature of hadoop batch indexing, and can be added later. Determining the best partition dimension can be complicated as many factors need to be considered (e.g., impact on query speed, impact on ingestion speed, etc.).

@ccaominh +1 for moving of DataSketches to core.
I don't think PR #8647 would block the move to core (or vice versa), as it just renames imported packages. But it probably requires some minimal coordination.
I guess if we create a separate PR for move-to-core, it would even makes such coordination easier.

I think I'm fine with separating moving data sketches to core and this issue as long as we call out that range partitioning is available only when the data sketches extension is loaded until it gets moved into core.


Question of dynamic node classification & edge classification?

Hii Yu, thanks a lot for your nice work and codes sharing!! I have a question about the dynamic node classification, after reading the codes, I found in the DG_data/DATASETS_README.md that it says the title of label column is the edge_label , but indeed it's the dynamic node classification task of src_node (shown in train_node_classification.py file). So there's a minor misleading issue here. Moreover, do you have plan to support edge classification in DyGLib? Thanks in advance! ^0^

Hi,
Thanks for your appreciation of our work! ^_^
Name of edge_label
Your understanding of edge_label is correct! In fact, we use the same column names as previous works do. We name node labels as edge_label because, in each interaction (i.e., edge), the involved source node is associated with a label, which means the number of node labels is equal to the edge number. In scenarios where the node tends to be assigned with a label after a number of interactions, I think using node_label is more appropriate. But it will be also fine to replace edge_label with node_label if you can understand better from the latter!
Edge classification task
Actually, the current link prediction task is a binary classification task for edges (with label 0 or 1) since most existing datasets support such a setting. If your datasets have more than two edge label classes, you can simply do the following things to achieve multi-class edge classification.

adjust the loss function from BCELoss to CrossEntropyLoss here;
set the output_dim of MergeLayer to the number of edge label classes here.

Please feel free to contact me if there are any further questions.

Thanks a lot for quick reply! The first question of edge_label has been well clarified. There's a concern in second Q of edge classification. Since the future link prediction usually uses concat(dynamic src_node embedding, dynamic dst_node embedding) to predict whether an interaction occurs at target timestamp, the edge classification uses concat(dynamic src_node embedding, temporal interaction feature, dynamic dst_node embedding) to classify the label of target edge. So I wanna know there's a small gap of these two tasks right? (If I want to modify into an edge classification task, I need do some small modifications basic on node_classification better. Correct me if I'm wrong.) Thanks for clarification!

Based on the current implementation, it is easier to realize edge classification using the link prediction code. It is a convenient extension to combine temporal interaction features with dynamic src_node embedding and dynamic dst_node embedding. And the above two modifications I mentioned are also straightforward to do.
Currently, the node classification code is unsuitable since its pipeline is first using the pre-trained temporal GNN models (derived from the link prediction task, and the parameters are freezed) to obtain node embeddings and then only finetuning the classifier. Such a design is used to tackle the imbalanced node labels problem following previous works, but this issue is unlikely to happen on the edge classification task.

Thanks for the clarification and it's all clear now.

Based on the current implementation, it is easier to realize edge classification using the link prediction code. It is a convenient extension to combine temporal interaction features with dynamic src_node embedding and dynamic dst_node embedding. And the above two modifications I mentioned are also straightforward to do.
Currently, the node classification code is unsuitable since its pipeline is first using the pre-trained temporal GNN models (derived from the link prediction task, and the parameters are freezed) to obtain node embeddings and then only finetuning the classifier. Such a design is used to tackle the imbalanced node labels problem following previous works, but this issue is unlikely to happen on the edge classification task.

Hi,
I'm interested in "Such a design is used to tackle the imbalanced node labels problem following previous works",
Could you please tell me what the "previous works" are? I want to take a look at them! Thanks!


Incorrect statement when importing APIs using CI/CD pipeline

Description:
Need to remove the following line in document [1] as it is incorrect.
When you import an API, regardless the status of the imported API it will be added with the created state and you need to sign in to the Publisher and publish the API.
The status of the API is preserved when importing an API as the --preserveStatus flag is set to true by default.
[1] - https://docs.wso2.com/display/AM260/Migrating+the+APIs+to+a+Different+Environment

Updated in [1] based on the offline discussion that I had with Kavishka.
@kavishkafernando , please let us know if any further changes are required in the documentation in this regard.
[1] https://docs.wso2.com/display/AM260/Migrating+the+APIs+to+a+Different+Environment


Question 19: Remove Nth Node from end of Linked List

Hey @Vanshika2063, I would like to provide an implementation for this problem using fast and slow pointers in C++ would you please assign me this issue?!

sure

Hey @Vanshika2063 I have addressed this issue in #138

hey @Vanshika2063 I would like to provide a solution for this problem. Would you please assign this issue to me?


Remove HF update, doesn't work

Topic: remove_hf_update

Reviews in this chain:
└https://github.com/riffusion/riffusion/pull/67 Remove HF update, doesn't work

#
head
base
diff
date
summary

0
5d618dd1
2695af97
diff
Jan 6 17:01 PM
1 file changed, 30 deletions(-)


Does it support Chinese charaters in the pdf documents?

I tried to modify the index in li = pdf.table_to_list(cells, pages)[5], it shows that this tool supports Chinese characters(it may has nothing to do with characters?).


I need help

I can't find the widget window in the game after opening the state component, I've opened the display widget in Hunterpie and the initialization widget is also open.

 Make sure you're selecting the right game using the selection box on the top left of your settings window.

Also, you must restart HunterPie after creating and configuring your abnormalities tray.

Thanks, here's my problem, I didn't find this option, sorry


feat(boringssl) use local toolchain

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


Add terraform settings block and data block, fix depends_on attributes

Add the class Terraform to represent terraform settings
Add the class Data to represent terraform data
https://www.terraform.io/docs/configuration/syntax-json.html#terraform-blocks
Add depends_on string conversion for the Block base class so depends_on will dump properly

That's a lot :-) I need to find time to have a closer look at all this. Sorry for the delay.

Have you tested this against Terraform 0.12 ? The current automated tests don't cover Terraform 0.12.
I started the feature/terraform_0.12 branch specifically for Terraform 0.12 with the intention to rewrite a lot of code. There is probably a lot of really good stuff in your pull request but I may have to work out how everything will fit together.

Sorry for the big PR, we are in the middle of moving to terrascript so I took my liberties and implemented features as I went along.
We just rebuilt our test environment with terraform 0.12.6 and terrascript today , and it worked fine

No problem at all. It's actually really nice (and a bit scary) to realise that other people find my little project useful. I am going to merge your pull request this evening (Melbourne time).
I might also temporarily create a separate branch just for your pull requests as I wanted to continue making (potentially major) changes to a (renamed) terraform_0.12 branch. I'll keep you updated.

Do you have any specific plans for Terraform 0.12 (although that is working now) and a 0.7 release? We would love to send more PRs to help out.

Let me get back to you later today, I am currently at work...

Hi, there are now separate Git develop-0.x branches for Terrascript 0.6.x (Terraform 0.11.x) and Terrascript 0.7.x. (Terraform 0.12). Please submit any future Pull Request to one of those branches. There is some more information about this in the README.md file of the develop branch.


Controlling Speaker volume

Hello,
i need to control volume of two buzzers, but i havent found the "volume" function or anything like it.
Can i control volume or not?
Thanks,
Jonathan.

(for documentation purposes)
There is no volume control for the square wave output.  You could "mix" the outputs by connecting the outputs (through resistors) and then use a variable resistor to control the volume.


Can't drag cells over pandas report output

Open notebook with pandas profile report output (@claudiaregio's Titanic.ipynb)
Drag markdown cell to scroll notebook
When I am dragging over the pandas output, the notebook bounces back up so I can't scroll down
Maybe the output is intercepting the drag event

After reproing this:

https://github.com/microsoft/vscode/issues/131937
Can't paste copied cells

@mjbvz this involves dragging a markdown cell over an output that contains an iframe. We get drag events where the clientY is relative to the position on the output's iframe, not the main webview. We can check event.view to see that it comes from another Window but there are no numbers on the event that give what we need. Best idea I have is to find the matching iframe element and check its offset to calculate the real position we need. Do you have any better ideas?

Trying to fix this issue and ended up with calculating the real clientY by e.clientY + e.view.frameElement.getBoundingClientRect().y. It works with the pandas output in the test notebook but I don't have confidence that it will always work, especially when the iframe's origin is unique (then e.view.frameElement would be undefined) or the output is a nested iframe (it makes the offset check diffcult).
I wonder if we should do something similar as how we handle scrollbar dragging for notebook. When we are dragging the vertical scrollbar, we will add a transparent div on top of the webview, to ensure that the drag event is not stolen by the webview (which will break the scrollbar smooth scrolling). Maybe we can do the same for markdown dragging.

I think that's probably what we have to do

I think that adding a transparent div that intercepts drag events makes sense. Will take a look in October

@roblourens or @claudiaregio  Can you please share the ipynb file I can use to test this? I'm not familiar with using pandas


APPT-258 added tests for E2E single/weekly create availability wizard

@sambiramairelogic Review comments update please take a look.

Its updated now take a look


IPASIP-1960: Update base image to latest node 14

LGTM, let's rebuild and push to Docker Hub once this is merged

Merging this from @Tim-W-James approval


Investigate Play Integrity API

https://developer.android.com/google/play/integrity/setup
implementation 'com.google.android.play:integrity:1.4.0'

Questions:

What are typical use cases for Play Integrity API for games?
How can game developers detect when a user is playing the game from a hacked version, and prevent access?
Is it only possible to detect when the apk is tampered with or can it also detect situations when downloaded content has been tampered with?

Answers from Google:
Play Integrity API has essentially 3 areas of checks:
[a] It can detect if the APK has been modified from what Google Play recognizes. This is primarily focused on the APK and not additional content that might be downloaded. There are some soft checks on additional assets downloaded using Play Asset Delivery, but there are no checks for assets downloaded from a CDN.
[b] It can detect if the user has a license for the game from Google Play (i.e. they installed or paid for the game on Google Play)
[c] Finally, it can get a verdict on the integrity of the underlying Android device to ensure that it has not been modified / tampered with.
You can think of this as several levels of checks / security. I think point [c] is a critical element of Play Integrity and forms a foundation for any additional anti-cheat / anti-hack solutions. Essentially, it's important to secure the integrity of the underlying device to build trust from any additional signals coming from the device.


In Bicep [providers] function return type is Array, but in ARM it is Object

Bicep version
Bicep CLI version 0.2.328 (a13b032755)
Describe the bug
The [providers] function return type is Array, but according to ARM it is Object, here
To Reproduce
output tes1 array = providers('Microsoft.Insights', 'components')
Additional context
does not run after compilation

Until this has been fixed, is there a way to achieve the equivalent of ARM template
providers('Microsoft.Cache', 'redis').apiVersions[0] ?
If bicep providers() outputs an array, is it possible to get the apiVersions from a specific index on that array?

Until this has been fixed, is there a way to achieve the equivalent of ARM template
providers('Microsoft.Cache', 'redis').apiVersions[0] ?
If bicep providers() outputs an array, is it possible to get the apiVersions from a specific index on that array?

You can workaround this for now with any(providers(Microsoft.Cache', 'redis')).apiVersions[0]


Move repostitories(Openjscad, CSG.js , openscad translator) to a Github Organization

Problem
Right now we are facing a few headaches because of the dependencies between packages, forks of forks , outdated or not maintained repos that we depend upon , lack of node.js packaging for some of them etc
Proposal

Move / ask the authors to move these repostories to a common Organization : https://github.com/jscad
The list of packages should be :

OpenJSCAD.org (this package, which itself uses a number of other packages in the list)
CSG.js (currently up to date version maintained by @joostn resides at https://github.com/joostn/OpenJsCad/blob/gh-pages/src/csg.js
openscad-openjscad-translator by @garyhodgson

Make sure that all of these modules are useable in all these environments:

browser
node.js (+ Browserify for the above)

All of this would reduce headaches, make common work easier , PR's easier, avoid 'dead forks', and allow all these projects to move forward faster & better .
Would love to hear what everybody (especially @spiritdude, @z3dev, @dav-m85, @mtippett)  thinks, and if I have missed something, please post here !

I think this would be great. We also have the OpenJsCad organization (https://github.com/OpenJsCad) set up by Stefan (@bebbi) a while ago. @z3dev you asked a while ago to be transfered ownership, apologies for not replying - you can use this organization name if you want. (I think @bebbi you would agree too?)
Whatever the name, I'd be happy to transfer joostn-csg.js to the organization, there are no active maintainers currently and I don't have the time.

An update.  I moved my personal code to https://github.com/jscad/csg.js, and it worked fine.  I'll try to see what to do about the format output.

Also pinging you @garyhodgson for the translator, this is the 'move I mentioned' : would you be ok with transfering it ? :)

Hi.  Sure no problem. I thought it was no longer necessary as you have a fork in the jscad org already.
I just tried to transfer the project and got the following message: "jscad/openscad-openjscad-translator already exists and You don’t have the permission to create repositories on jscad"
Open to suggestions 😺

@garyhodgson you can now transfer it, I removed the fork :)
thanks for merging the PR too !

thanks @garyhodgson  :)

I'm the current maintainer of the atom-scad-preview package which is really handy to edit and preview models side by side. I would like to see somewhere repository with already created basic models such as nuts, bearings and so on. Seems like jscad github org is a perfect place for that repo.

@apla that would be a good idea, for the most 'standard' of parts , I think it would be a very good idea we could reuse / collaborate with http://www.bolts-library.org/en/index.html since there are a TON of standardized measurements (I used to use those regularly with CoffeeScad)

I see the that https://www.npmjs.com/package/@jscad/csg was moved out. However no PR to make OpenJSCAD use this yet? Will this work happen in this repo?

Another piece I one could consider splitting out (probably after the initial pieces here are done) is the 3d-view widget. I think that would be useful in other places that use CSG.js also

Hi @jonnor, you may want to check the dev branch where the csg move out is actually happening. I don't know when it is planned to be merged into master, it is however under active development.
Totally agree on the 3d-view widget.

@jonnor & @dav-m85 as @dav-m85  said, most of the work is done in the dev branch, the main thing remaning before master is

dealing with the messy drag & drop code : I am currently working on it although a bit slow, because day job:)
spliting out all the formats handling (https://github.com/Spiritdude/OpenJSCAD.org/tree/dev/src/io) , although that might have to wait until after the next release of Openjscad / merge into master

As far as the 3D view widget goes : I will be honest, while it makes sense to have it 'swappable' /split out,  the quality of the code there is really not up to the level of the rest (Ie, no seperation of concerns between view/ camera controls/event handling),  messy structure etc :)

This is all done, closing !


fix: update metric processor for the readRecords and remove readSampledRecords from metrics selector

Fixes metric processor for the readRecords.
Video
https://user-images.githubusercontent.com/20871783/203730031-382d18f2-f42d-4e00-afed-8a1269bd39c7.mov
Signed-off-by: Qiuxia Fan<EMAIL_ADDRESS>

Let's adopt the latest readRecords and remove the deprecated readSampledRecords
Ref https://github.com/apache/skywalking-query-protocol/blob/90724a9c4982ea806d6ad58304ed70d3c3758248/metrics-v2.graphqls#L156-L159

Removed readSampledRecords from metrics selector


PusherChannelsFlutterPlugin.authorize Firebase Crashlytics

Hey,
I'm getting this error in Firebase Crashlytics dashboard, but it doesn't come when I debug. Does anyone know why this error is coming?
Fatal Exception: java.lang.NullPointerException:
at com.pusher.channels_flutter.PusherChannelsFlutterPlugin.authorize(PusherChannelsFlutterPlugin.java:13)
at com.pusher.client.channel.impl.PrivateChannelImpl.getAuthorizationResponse(PrivateChannelImpl.java:12)
at com.pusher.client.channel.impl.PrivateChannelImpl.authorize(PrivateChannelImpl.java:2)
at com.pusher.client.channel.impl.PrivateChannelImpl.toSubscribeMessage(PrivateChannelImpl.java:6)
at com.pusher.client.channel.impl.PresenceChannelImpl.toSubscribeMessage(PresenceChannelImpl.java)
at com.pusher.client.channel.impl.ChannelManager.lambda$sendOrQueueSubscribeMessage$0(ChannelManager.java:10)
at com.pusher.client.util.Factory.lambda$queueOnEventThread$0(Factory.java:3)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1137)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:637)
at java.lang.Thread.run(Thread.java:1012)

Are you able to share the code in use?
Is the auth endpoint accessible from the device?

This is onAuthorizer function. I test but it's works very well. But some users get crush.

@benw-pusher  if you want I can share other code in use about pusher. But I think everything is same like example project.

Can you clarify: does this only occur in production, when testing you don't see this?

It was occur in the production. No, when I test it in the debug mode I never see it.

Thanks for confirming. This suggests there is a difference in the response received between your test and your prod setups - which is also suggested by the stack trace showing the error at 'getAuthorizationResponse. Are you able to share the responses from each environment?
Or there could be an inability to access the auth endpoint in prod, or a specifiec variable/parameter is not being passed/set .

We haven't had a response here so will close. If you need further support pease get in touch at https://messagebird.com/support-center/Pusher-request-new


Multiple requests suggested way

Hi,
I am trying to do multiple requests without having to wait for each one to finish. I was using grequests but i am wondering if there is a better way to handle it.

grequests is the best way to handle it at the moment if you like using gevent. Your other options are trequests, treq, async-requests, and there are likely some others floating around that I don't know about.


Localise Horizontal Scaling page in Bengali

The Bengali version of Horizontal Scaling is not available.
Link: https://glossary.cncf.io/horizontal-scaling/
@waleed318  Can you assign it to me? I want to work on this issue.

Hello sajdakabir, I am Urdu glossary approver please contact your bengali team. 🙂

@jihoon-seo Can I work on this issue?

@sajdakabir  already Localized by  someone
https://glossary.cncf.io/bn/horizontal-scaling/

@mitul3737 This issue is already fixed.

The issue is already fixed in this PR. I am closing this issue.


Failed LN transaction shown in Histoey

I have tried to send quite big LN transaction from my bode over Sphinx.
The transaction never happend (checked via RTL) but it is visible in history (screenshot).

The 10 sats transaction worked and was routed. The 900k transaction was not.
It is not clear from UI that it haven't worked.

We'll make that clear if it failed or not in an upcoming release. Is the balance on the dashboard correct?

The balance is AFAIK correct


4.4 dev 6 crashes on game export headless

Tested versions
Reproducible on 4.4 dev6
System information
Ubuntu 24, Godot 4.4 dev
Issue description
When I try to export a game in godot 4.4 dev6 using the headless option, the editor crashes with the following error.
The command to export is
godot -v --headless --path game/ --export-release macOS export_game.zip
================================================================
handle_crash: Program crashed with signal 11
Engine version: Godot Engine v4.4.dev6.official (1f47e4c4e3a09a422e96880a7918d986dd575a63)
Dumping the backtrace. Please include this when reporting the bug to the project developer.
[1] /lib/x86_64-linux-gnu/libc.so.6(+0x45250) [0x7f457f377250] (??:0)
[2] godot() [0x475b918] (??:0)
[3] godot() [0x622ea0] (??:0)
[4] godot() [0x6376d3] (??:0)
[5] godot() [0x639056] (??:0)
[6] godot() [0x64c3c3] (??:0)
[7] godot() [0x48ca7dd] (??:0)
[8] godot() [0x43e1714] (??:0)
[9] godot() [0x2681085] (??:0)
[10] godot() [0x532442] (??:0)
[11] godot() [0x41c57a] (??:0)
[12] /lib/x86_64-linux-gnu/libc.so.6(+0x2a3b8) [0x7f457f35c3b8] (??:0)
[13] /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0x8b) [0x7f457f35c47b] (??:0)
[14] godot() [0x43c7ca] (??:0)
-- END OF BACKTRACE --
================================================================
Aborted

Full log
Preloading project configuration(1).log
Steps to reproduce

Fork the following project

https://github.com/svelezw4games/godot-4-3d-third-person-controller

Checkout this branch

migrate_to_4_4

run the following command

godot -v --headless --path game/ --export-release macOS export_game.zip
Minimal reproduction project (MRP)
Reproducible project can be found on the following link
https://github.com/svelezw4games/godot-4-3d-third-person-controller
On this branch
https://github.com/svelezw4games/godot-4-3d-third-person-controller/tree/migrate_to_4_4

Can you try https://nightly.link/godotengine/godot/workflows/runner/master?preview and see if it still fails?
I want to narrow down some crash bugs.


Dependency clap has MSRV 1.7

[Manually transferred to this new account]
This is too high for qiskit. Can we:

replace clap
Use an earlier version of clap
Find a different solution.

@mtreinish wrote
In general I'm not sure we need clap for anything in a parser library. Clap is a command line argument parser, if the crates we export as a library shouldn't need an argparser as we won't have a command line interface. If you want to build a standalone parser utility on top of the parser lib then clap would make sense, but as a binary crate it could have an independent msrv.

Turns out that clap already is a dev-dependency. We can skip building the example code, which uses clap with cargo test --lib ...
I guess that clap really does need 1.7.0. Some of the crates that say they need 1.7.0 apparently don't. I can edit the version requirement and they build and tests succeed.
In any case clap is no longer an immediate problem.


Update log4j to 2.17

Update log4j to 2.17 to manage the new CVE-2021-45105

Dup, already done. We’ll have a 1.2.3 release this week. https://github.com/opensearch-project/OpenSearch/issues/1772


Problem with ros2_ouster package using colcon build

Have a problem building the ros2_ouster package with colcon build.
using ros2 foxy on ubuntu 20.04
cloned using: git clone -b foxy-devel https://github.com/ros-drivers/ros2_ouster_drivers
dependencies: rosdep install --from-paths src --ignore-src --rosdistro=foxy -y
then build: colcon build
Get sthis output:
Starting >>> ouster_msgs
Finished <<< ouster_msgs [0.49s]
Starting >>> ros2_ouster
--- stderr: ros2_ouster
CMake Error at /usr/lib/x86_64-linux-gnu/cmake/pcl/PCLConfig.cmake:62 (message):
common is required but boost was not found
Call Stack (most recent call first):
/usr/lib/x86_64-linux-gnu/cmake/pcl/PCLConfig.cmake:367 (pcl_report_not_found)
/usr/lib/x86_64-linux-gnu/cmake/pcl/PCLConfig.cmake:551 (find_external_library)
CMakeLists.txt:17 (find_package)

Failed   <<< ros2_ouster [1.32s, exited with code 1]
Summary: 1 package finished [2.00s]
1 package failed: ros2_ouster
1 package had stderr output: ros2_ouster

You’re missing and install of PCL.

I have the same issue, however I do install PCL by below commands:
$ sudo apt install libpcl-dev
$ sudo apt isntall libpcl-conversions-dev

So which PCL do I miss? So that I can install it back. Thx


Adjust Statistics::total_byte_size in Filter with a projection

Is your feature request related to a problem or challenge?
@Dandandan  pointed out in https://github.com/apache/datafusion/pull/13187/files#r1824330274 that when applying a projection to a filter the resulting StatisticsCalculation is not properly updated

I think the global stats (total_byte_size) are not correct either, doesn't take into account the reduced number of columns. It should do something similar as stats_projection for ProjectionExec

However, I did not want to try and add that in the bugfix PR https://github.com/apache/datafusion/pull/13187 because:

The total_byte_size calculation in  filter also needs to take estimated selectivity into account
The calculation of total_byte_size in stats_projection is also somewhat suspect as it only accounts for "fixed sized" rows but still claims the size is known precisely

Describe the solution you'd like
Account for the projection in the filter statistics calculation somehow
Describe alternatives you've considered
https://github.com/apache/datafusion/blob/ac79ef3442e65f6197c7234da9fad964895b9101/datafusion/physical-plan/src/projection.rs#L261-L260
Additional context
No response

Take


Speedup encoding process

I don't think that it is right place to ask questions and for assistance, but as far as there is no official forum (and many others are doing the same) - I'll do it here.
I've written openh264-based plugin for VLC and using it in one of my projects. (soon will make it public on github, hello there: #2293, it is here)
I have a very specific problem that requires 24/7 full HD video recording. Also it must not consume a lot of CPU and be high-quality (in my case bitrate is not important at all, let's consider to have approximately infinite disk space).
I am using this settings right now:
    p_sys->sSvcParam->iUsageType = SCREEN_CONTENT_REAL_TIME;
    p_sys->sSvcParam->fMaxFrameRate  = 25;
    p_sys->sSvcParam->iPicWidth      = p_enc->fmt_in.video.i_width;
    p_sys->sSvcParam->iPicHeight     = p_enc->fmt_in.video.i_height;
    p_sys->sSvcParam->iTargetBitrate = 2500000;
    p_sys->sSvcParam->iMaxBitrate    = UNSPECIFIED_BIT_RATE;
    p_sys->sSvcParam->iRCMode        = RC_QUALITY_MODE;
    p_sys->sSvcParam->iTemporalLayerNum = 1;
    p_sys->sSvcParam->iSpatialLayerNum  = 1;
    p_sys->sSvcParam->bEnableDenoise    = 0;
    p_sys->sSvcParam->bEnableBackgroundDetection = 0;
    p_sys->sSvcParam->bEnableAdaptiveQuant       = 0;
    p_sys->sSvcParam->bEnableFrameSkip           = 1;
    p_sys->sSvcParam->bEnableLongTermReference   = 0;
    p_sys->sSvcParam->uiIntraPeriod  = 25;
    p_sys->sSvcParam->eSpsPpsIdStrategy = CONSTANT_ID;
    p_sys->sSvcParam->bPrefixNalAddingCtrl = 0;
    p_sys->sSvcParam->iComplexityMode = LOW_COMPLEXITY;
    p_sys->sSvcParam->iEntropyCodingModeFlag = 0;

    p_sys->sSvcParam->sSpatialLayers[0].uiProfileIdc       = PRO_BASELINE;
    p_sys->sSvcParam->sSpatialLayers[0].iVideoWidth        = p_enc->fmt_in.video.i_width;
    p_sys->sSvcParam->sSpatialLayers[0].iVideoHeight       = p_enc->fmt_in.video.i_height;
    p_sys->sSvcParam->sSpatialLayers[0].fFrameRate         = 25;
    p_sys->sSvcParam->sSpatialLayers[0].iSpatialBitrate    = 2500000;
    p_sys->sSvcParam->sSpatialLayers[0].iMaxSpatialBitrate    = UNSPECIFIED_BIT_RATE;
    p_sys->sSvcParam->sSpatialLayers[0].sSliceCfg.uiSliceMode = SM_SINGLE_SLICE;

What can be changed in settings to make encoder be more CPU-friendly? Right now it requires apox 30% of 4-core i5 4460 3.2 GHz to make screen recording. I.e. one core is practically 100% busy with encoding and other 3 at about 10% load. Tested on Windows 7 embedded.
Also are there any plans to make hardware acceleration inside openH264?

I think this issue can be closed as I figured out that this is VLC who consumes all this CPU time. In fact it I figured out with profiling the VLC that it spends approx. 80% of time just waiting though occupying CPU time.
Now I am going to write my own wrapper around OpenH264 to make 24/7 screen recording and I hope I wont spent CPU time such a useless way.


Allow global functions with no arguments to be called without parenthesis

Currently, a global function call must either have an argument, or use parenthesis. e.g. rand(), floor 5, floor(5).
This is because having the name by itself, e.g. rand could also refer to a local variable. The variables being passed to the view template are not currently known at compile time, and this creates an ambiguity.
One possibility would be to change the Variable AST node type to VariableOrGlobalFunc, and make the decision at runtime based on if the current scope contains such a variable or not (local variable takes precedence over global function or constant).

Was fixed in c9d5c132762f9c2120bb8caa98a9a47fc89fb3e1 as part of #78


Add prop for selected tab style

I understand tab implementation may be changing soon but I've added a prop so the selected tab can have differing style.

I would let @joenoon review this since he's working on tab.

@andoru I haven't tested it, so please correct me if I'm wrong this is a breaking change?

@andoru I dont think this still would apply to master, so closing for now.  If you want to reopen a new one against master please do.


Error when parsing CSV file 0.9.11

This lib is awesome. It works perfectly for xls and xlsx. I upgraded to 0.9.11 and I thought I would be able to read CSV files. However, I am receiving the following error in the console:
The same file works in the In-Browser Demo but not in my code but I don't see a difference
Uncaught Error: Corrupted zip : can't find end of central directory
at ZipEntries.readEndOfCentral (jzip.js:2264)
at ZipEntries.load (jzip.js:2326)
at ZipEntries (jzip.js:2150)
at JSZip.module.exports [as load] (jzip.js:567)
at JSZip (jzip.js:514)
at read_zip (xlsx.js:14064)
at Object.readSync [as read] (xlsx.js:14097)

If the file works in the demo but not in your code, there's most likely a problem in your integration code.

confirm you are using the right version: console.log(XLSX.version) just before the part that reads in the file.

just before reading the file, try to console.log the data just before calling read, and confirm that it looks like the original file

if the content is the same and you are using the right version, share the code snippet that actually reads the file (the part including the XLSX.read call).  But a glance at the line numbers suggests that you may be using an older version -- readSync is defined in line 16114 in version 0.9.11

Thanks for the quick reply!
So I confirmed with XLSX.version that the version is still 0.8.7. However, I also confirmed that there are 2 xlsx.js files in separate libs in bower_components. One is located in js-xlsx (0.9.11) and the other is in xlsx-js (0.8.7). I tried removing them both and reinstalling with bower, but they both came back. Is this right?

Our bower.json is configured to publish as js-xlsx.  The manifest page appears to be correct.
xlsx-js points to a separate fork.
It's likely that you either included both names in your bower.json, or a dependency pulls in the other version.  I would check your local bower.json first.
The correct name is noted in the README: https://github.com/sheetjs/js-xlsx#installation

The xlsx-js lib (0.8.7) gets installed due to another lib "tableexport.js" that has a dependency on xlsx-js (^0.8.7) and my bower.json installs js-xlsx 0.9.11, thus making two separate instead of updating the older version.


[Feature] Add PDF theme

It would be awesome if the theme included a PDF viewer theme in the future. Logseq PDF CSS is available at https://github.com/logseq/logseq/tree/master/src/main/frontend/extensions/pdf.

Thanks @Bad3r , I personally do not read PDF in Logseq muchand I will do it later if I have time next month. Meanwhile I have marked the issue as help wanted to see if some other can take the task.

@Bad3r High key interested in your notes about awk. 😊
Finished an espanso shell cmd using it yesterday.

Please try out my custome.css, see #101 .


add important eslint rules from leaflet repository and fix resulting warnings

My intention for this pr was to make this repo even better, but feel free to just ignore it 😄.
To make eslint rules more consistent with leaflet, this pr adds the rules from the leaflet repository that are handled as errors and fixes the resulting warnings,
Besides this it:

updates eslint to the current version
adds 'npx' to the package.json script for linting, to make it runnable on *nix and windows alike
adds the package-lock.json to git, to make all users of this repo have the same dependencies installed
adds a .gitignore file, to prevent the node_modules directory to find its way to git

I suggested a more extensive approach adding prettier and stylelint with #115. What do you think of it?
It might also be a good option to merge your PR and then I rebase my PR to discuss the introduction of prettier and stylelint later?

This pr is no longer necessary - I support pr #115 instead


exclude less-useful, harder-to-create artifacts

Editing in Makefile! Would be better to make this a configuration.

#PROJECT_FOLDERS = sqlschema shex shacl protobuf prefixmap owl jsonschema jsonld graphql excel
PROJECT_FOLDERS = sqlschema owl jsonschema jsonld  # MAM 2023-10-05

see also config.yaml and config.yaml.bok


fix image-classification import error with python3

fix import error with python3

alexnet and googlenet are wrongly switched #4826


dimensionality of sensitivities

@feigelman The documentation says that the dimension of the sensitivity output dMRE is N × T × d, with "d is the number of model parameters + the number of species". For dSigma it is N × N × T × d.
Is it correct that the first length(theta) sensitivities are the sensitivities with respect to the parameters? Then I would expected that length(theta)+1:d are the sensitivities with respect to the initial mean, correct?
What happens of the mean determined by the steady state?
Is the sensitivity with respect to the initial covariances calculated?

Hi Jan
yes, I believe that is how I had implemented it.  The sensitivities w.r.t.
model parameters, the w.r.t. to initial means.
I'm not sure what happens if the mean is determined by the steady state. In
that case the sensitivity w.r.t initial mean should be defined and computed
in the same way, right? It's just that if one is computing the mean using
the steady state, the sensitivity w.r.t. to the initial mean is not of
interest.
I never implemented the sensitivities for initial covariance, and see it on
my 'todo' list from 2016. :/
Thanks,
Jsutin
On Sun, Apr 22, 2018 at 4:52 AM, Jan Hasenauer<EMAIL_ADDRESS>wrote:

@feigelman https://github.com/feigelman The documentation says that the
dimension of the sensitivity output dMRE is N × T × d, with "d is the
number of model parameters + the number of species". For dSigma it is N × N
× T × d.
Is it correct that the first length(theta) sensitivities are the
sensitivities with respect to the parameters? Then I would expected that
length(theta)+1:d are the sensitivities with respect to the initial mean,
correct?
What happens of the mean determined by the steady state?
Is the sensitivity with respect to the initial covariances calculated?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ICB-DCM/LNAplusplus/issues/22, or mute the thread
https://github.com/notifications/unsubscribe-auth/ADnEQXgKAwpJrqvBrf8eOtWn5JeazEXwks5trG8bgaJpZM4Te1cj
.

--
Justin Feigelman

Hey Justin,
thanks for the confirmation. I will check the case with analytical steady state. The sensitivities with respect to the initial variances will be posted as a separate issue. I would be nice to have this at some point. For this paper, it is however not so important.
Greetings,
Jan

Okay, has been clarified in documentation.


[BUG]: Cannot drop Unique Constraint MySQL

What version of drizzle-orm are you using?
0.28.1
What version of drizzle-kit are you using?
0.19.12
Describe the Bug

Create a mysql table with a unique constraint
Drizzle-kit push
Delete the unique constraint
Drizzle-kit push

The ORM attempts to do the following:
ALTER TABLE MyTableDROP CONSTRAINTMyTable_MyKey_key``
whereas it should be executing this:
ALTER TABLE MyTableDROP INDEXMyTable_MyKey_key``
This is preventing pushes to the DB and errors in the console.
Expected behavior
Key should be dropped and drizzle-kit should push without issues.
Environment & setup
Planetscale DB connection in node.js 18 environment

I think we need to use something like this instead for MySQL:
ALTER TABLE MyTable DROP FOREIGN KEY MyTable_MyKey_key

Source: https://stackoverflow.com/a/14122155

I'm also experiencing the same issue now. Foreign Keys are being supported by PlanetScale, the feature is currently in beta.

Was fixed in<EMAIL_ADDRESS>


unmet peer dependency

Running
npm install grunt-jsdoc --save-dev

causes:

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/31168881-unmet-peer-dependency?utm_campaign=plugin&utm_content=tracker%2F294645&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F294645&utm_medium=issues&utm_source=github).

npm warns you because you need grunt in your project. See https://docs.npmjs.com/files/package.json#peerdependencies


Deprecate hasPermission method for 2.0.0 release

Remove the hasPermission method for the 2.0.0 release.
Add deprecation notice for 1.7.0 release.

What will be the recommended way to prevent an immediate iOS permission dialog on first app start? I delay the first init call until I gracefully on-board the user to enable push notifications. I use the hasPermission check then to fire the init on deviceready
The API calls felt a bit awkward, but it was the only option.

I'm also interested in the details reasoning around this deprecation/issue. hasPersmission is the start of something cool that would enable devs using our plugin to increase UX and conversion, and I don't quite understand what's wrong with it.

Well, it doesn't really work. When you call hasPermission and it returns false what does that mean? Does it mean the user has selected no to the allow push dialog or does it mean the allow push dialog hasn't been shown yet? Because it can mean either of those things.
On iOS it's probably better for conversion if you provide your own UX before you may a call to PushNotification.init() so you can ready the users that the dialog is coming and they should say "Okay".
On Android the PermissionUtils class uses reflection which is slow on Android and there are problems with older versions of Android (4.1.2) causing crashes. As well it uses a method to figure out if the permission is allowed should only be used by System apps so I'm a bit worried that Google may complain someday about this code.
Happy to have a discussion on why it should be kept around though or fixes to make it more reliable.

We need to call PushNotification.init onload to account for a change in the registration ID. But only selectively. We do not want to intrude new users with an instant plea for permissions. If we can still maintain this via another API change, then hasPermssions serves to purpose for the majority of use cases.
Check out the autoRegister option in the OneSignal SDK

Well, we can partially solve that by asking: do I already have a registrationId provided by the plugin previously? if (yes) then {I need to call .init() ondeviceready to check for changes on that id} else {I can delay the call to .init() to the point where the user is ready to be optimally converted}..
Good points, thank you for explaining @macdonst.

@macdonst
"When you call hasPermission() and it returns false what does that mean? Does it mean the user has selected no to the allow push dialog or does it mean the allow push dialog hasn't been shown yet?"
If you're calling hasPermission() after init() it can only mean one thing (no ?) ... The user has selected 'no' on the dialog.
hasPermission() is very convenient if you want to coach the user into activating the messages (after they actively refused them). "Hey user x , this feature is awesome, please enable it" ...

Exactly what @dbpieter said. This is how it works on React Native and is very useful for informing the user about the missing permission. Init tricks can only go so far because this is also needed when the app returns from the background because the use might have disabled the permission from the settings.

Maybe more correct name for this would be canReceivePushMessages()?

@macdonst
Use case for using the HasPermission:
I do not let the user use the app until the registration is done
--> I call has Permission
if NO Then I display a popup than cannot be removed by the user explaining that we need "push"
--> if yes, I hide the popup and go to next page...
Because we cannot know wether the user allowed or not the "push" on the "native dialog"
I loop on this function...
Without this function "HAS PERMISSION", how can I "disable" the user to go to next page?

@macdonst
maybe my explanation was not very precise.
One easier way to say it:
I need a popup to be displayed until the user has autorized the push.
Today it's possible only because of the "hasPermission" function.
If tomorrow you remove it, how can we do it?

Okay, let it never be said I don't listen to people. I've removed the deprecation notice but we need to make this API better.

I agree with @macdonst and I have a specific example where the current implementation of hasPermissions breaks down:
Call init on app load to check if we need to update the push id (in this case lets assume they already gave us permission).

If the user disabled push since the last time they opened the app then we would get false from hasPermissions

We would also get false if the user deleted and redownload the app or if the user got a new phone and logged in with their old account.

In the first case we'd want to record that we don't want to send pushes anymore while in the second we'd want to prompt them again and get their new id. Has anyone encountered this issue and come up with a solution?

@jworm7 mentioned scenarios where hasPermissions() fails, but not by it's own fault. It can be helped by other practices to make it work reliably.
You can and should identify, by many different ways, an installation of your application so you would know when [2] is the case. That would leave [1] an easy-to-discern case by exclusion.

[2] using device.uuid from cordova you can close-to-safely identify the device itself, so you'd know when it's a new installation from an old user on a new device:

{
  "registrationId": "new",
  "device.uuid": "new",
  "user": "old",
  "UUID.v4()": "new",
  "hasPermission()": false
}

[2] using UUID.v4()(node-uuid) you can generate a key to identify any and every installation of your application, so you would know when it's simply a new installation from an old user on an old device:

{
  "registrationId": "new",
  "device.uuid": "old",
  "user": "old",
  "UUID.v4()": "new",
  "hasPermission()": false
}

[1] if the app doesn't have permission to use push notifications but is a known installation, you know it's <an installation that has never had push notifications enabled>localStorage.getItem('registrationId') === null or <an installation that has had push notifications enabled once but had it disabled recently somehow>localStorage.getItem('registrationId') !== null:

{
  "registrationId": "old|new",
  "device.uuid": "old",
  "user": "old",
  "UUID.v4()": "old",
  "hasPermission()": false
}

I don't think there's anything inherently wrong with the idea, but hasPermission being set to false before the user has accepted or denied the permission is wrong.

Closing, we decided not to deprecate it.

@macdonst - I realise this is no longer being deprecated, but just wanted to give a suggestion on how to potentially solve this issue (assuming this method has the available data).
Why can't we have values, undefined, false and true.

Undefined means the permission has not yet been set
False means the permission was expressly denied
True means the permission was expressly granted

Happy to create a new issue to get this done if you think it could work?

@jacquesdev agree, but do you encounter an issue when user select allow permission but has permission still on false status?


Appstudio purge rhtap-demo-component-idhn

Pipelines as Code configuration removal

Pipelines as Code CI has failed.There was an error creating the PipelineRun: rhtap-demo-component-idhn-on-pull-request-
creating pipelinerun rhtap-demo-component-idhn-on-pull-request- in namespace rhtap-demo-dev-rhxz-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: rhtap-demo-component-idhn-on-pull-request-
creating pipelinerun rhtap-demo-component-idhn-on-pull-request- in namespace rhtap-demo-dev-vkyt-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: rhtap-demo-component-idhn-on-pull-request-
creating pipelinerun rhtap-demo-component-idhn-on-pull-request- in namespace rhtap-demo-dev-lvfr-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request