Error x509: certificate signed by unknown authority when trying to download vulnerability DB from private Registry

What steps did you take and what happened:
I installed the trivy-operator via kubectl in version 0.0.5 and made the necessary changes to pull all images from our internal registry (JFrog Artifactory). The operator is running fine and starts the scanning pods. Except for the changes to the image sources, the operator is running with the default config.
However, the logs show the following error: failed to download vulnerability DB: OCI artifact error: OCI artifact error: OCI repository error: Get \"https://packages.repo.internal.com/v2/\": x509: certificate signed by unknown authority\n"
This repository uses a self-signed certificate, so this error is expected.
In the documentation I found a parameter to set registries to unsecure: trivy.insecureRegistry.
So I added this to the trivy-operator-trivy-config configmap:
data:
  trivy.insecureRegistry.artifactory: "packages.repo.internal.com"

This seems to have no effect, the error is unchanged.
What did you expect to happen:
I would expect trivy to skip the certificate validation with this setting.
Anything else you would like to add:
Environment:

Trivy-Operator version (use trivy-operator version): 0.0.5
Kubernetes version (use kubectl version): v1.22.6
OS (macOS 10.15, Windows 10, Ubuntu 19.10 etc): Ubuntu 20.04

have you tried adding this param in configmap : trivy.serverInsecure: true ?

From my understanding this setting would not apply in standalone mode, or am I mistaken? I thought it would only apply to client server mode with an external trivy server.
I have now tried to add this setting, however the error is the same. I tried both settings at the same time and both settings separately, but I still get the same error in any case.

its look like a proxy or else is redirecting the download URL to https://packages.repo.internal.com/v2/ , have you tried to use trivy by it self without operator and see if you get the same issue?

I changed trivy.dbRepository in the deployment manifest from ghcr.io/aquasecurity/trivy-db to packages.repo.internal.com/github-proxy/aquasecurity/trivy-db, as I'm deploying trivy in an airgapped environment so I cannot access ghcr.io directly.
For the same reason trivy by itself only works if I download the vulnerability database manually and run trivy with --skip-update --offline-scan

Trivy has added support for it trivy#240, once it will be released we will update trivy version in trivy-operator

@mauricewittek FYI #164

I just tested with the new Trivy-Operator 0.0.8 which uses a trivy 0.29.1, but I still get the same error:
{"level":"error","ts":1655803858.6775062,"logger":"reconciler.vulnerabilityreport","msg":"Scan job container","job":"trivy-system/scan-vulnerabilityreport-6f9895d478","container":"39edf7f4-6bff-4611-9e40-fb0e586e23b9","status.reason":"Error","status.message":"2022-06-21T09:30:57.562Z\t\u001b[34mINFO\u001b[0m\tNeed to update DB\n2022-06-21T09:30:57.562Z\t\u001b[34mINFO\u001b[0m\tDB Repository: packages.repo.internal.com/docker-github-proxy/aquasecurity/trivy-db\n2022-06-21T09:30:57.562Z\t\u001b[34mINFO\u001b[0m\tDownloading DB...\n2022-06-21T09:30:57.611Z\t\u001b[31mFATAL\u001b[0m\tinit error: DB error: failed to download vulnerability DB: OCI artifact error: OCI artifact error: OCI repository error: Get \"https://packages.repo.internal.com/v2/\": x509: certificate signed by unknown authority\n","stacktrace":"github.com/aquasecurity/trivy-operator/pkg/vulnerabilityreport.(*WorkloadController).reconcileJobs.func1\n\t/home/runner/work/trivy-operator/trivy-operator/pkg/vulnerabilityreport/controller.go:323\nsigs.k8s.io/controller-runtime/pkg/reconcile.Func.Reconcile\n\t/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.12.1/pkg/reconcile/reconcile.go:102\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile\n\t/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.12.1/pkg/internal/controller/controller.go:121\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler\n\t/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.12.1/pkg/internal/controller/controller.go:320\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem\n\t/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.12.1/pkg/internal/controller/controller.go:273\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2\n\t/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.12.1/pkg/internal/controller/controller.go:234"}
Am I missing something in my configuration? I've set the following parameters in the deployment manifest:
trivy.insecureRegistry.artifactory: "packages.repo.internal.com"
trivy.serverInsecure: "true"

@mauricewittek you mention that you are air-gapped env. where do you store the vuln db you download  ?

@mauricewittek join our slack channel to discuss in details

We are using jFrog Artifactory as a Proxy for external Repositories. So if I want to use the image docker.io/aquasec/trivy-operator:0.0.8 I have to change the address to packages.repo.internal.com/docker-proxy/aquasec/trivy-operator:0.0.8. And this works, the image can be pulled and the operator pod is running in the cluster.
I used the same logic to change trivy.dbRepository from
  trivy.dbRepository: "ghcr.io/aquasecurity/trivy-db"

to
 trivy.dbRepository: "packages.repo.internal.com/docker-github-proxy/aquasecurity/trivy-db"

Assuming that the db could be pulled then.

when you resolve it with CLI it works I assume ,which repo type you define and what URL you put in ?

I'm afraid I don't understand what you mean with

when you resolve the vulndb with trivy CLI it works I assume

But I tried the documentation for using Trivy in Air-Gapped environments: https://aquasecurity.github.io/trivy/v0.29.1/docs/advanced/air-gap/#run-trivy-with-skip-update-and-offline-scan-option
And a $ oras pull packages.repo.internal.com/docker-github-proxy/aquasecurity/trivy-db:2 -a works fine and downloads the db.tar.gz.

@chen-keinan Hi, I am also having the same issue. In my case the SSL Inspection of Palo Alto messes up with the certificate which causes it to become "insecure". I am very much looking for the insecure flag so that I can set it in the config map and use the operator seamless. Thanks!

@chen-keinan  Can you please reopen the ticket. I am still facing the same issue.
I downloaded the latest operator, added the flag trivy.serverInsecure: "true" and set the trivy version to trivy.imageRef: "docker.io/aquasec/trivy:0.29.2" within the configmap trivy-operator-trivy-config and I am still getting the same message:
tinit error: DB error: failed to download vulnerability DB: OCI artifact error: OCI artifact error: OCI repository error: Get \"https://ghcr.io/v2/\": x509: certificate signed by unknown authority\n","stacktrace":"github.com/aquasecurity/trivy-operator/pkg/vulnerabilityreport.(*WorkloadController).reconcileJobs.func1\n\t/home/runner/work/trivy-operator/trivy-operator/pkg/vulnerabilityreport/controller.go:323\nsigs.k8s.io/controller-runtime/pkg/reconcile.Func.Reconcile\n\t/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.12.1/pkg/reconcile/reconcile.go:102\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile\n\t/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.12.1/pkg/internal/controller/controller.go:121\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler\n\t/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.12.1/pkg/internal/controller/controller.go:320\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem\n\t/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.12.1/pkg/internal/controller/controller.go:273\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2\n\t/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.12.1/pkg/internal/controller/controller.go:234"}

@chen-keinan Can you please reopen the ticket. I am still facing the same issue. I downloaded the latest operator, added the flag trivy.serverInsecure: "true" and set the trivy version to trivy.imageRef: "docker.io/aquasec/trivy:0.29.2" within the configmap trivy-operator-trivy-config and I am still getting the same message:
tinit error: DB error: failed to download vulnerability DB: OCI artifact error: OCI artifact error: OCI repository error: Get \"https://ghcr.io/v2/\": x509: certificate signed by unknown authority\n","stacktrace":"github.com/aquasecurity/trivy-operator/pkg/vulnerabilityreport.(*WorkloadController).reconcileJobs.func1\n\t/home/runner/work/trivy-operator/trivy-operator/pkg/vulnerabilityreport/controller.go:323\nsigs.k8s.io/controller-runtime/pkg/reconcile.Func.Reconcile\n\t/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.12.1/pkg/reconcile/reconcile.go:102\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile\n\t/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.12.1/pkg/internal/controller/controller.go:121\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler\n\t/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.12.1/pkg/internal/controller/controller.go:320\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem\n\t/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.12.1/pkg/internal/controller/controller.go:273\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2\n\t/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.12.1/pkg/internal/controller/controller.go:234"}

@nfsouzaj  this issue has not yet been released , it will be released with trivy-operator v0.0.9

@nfsouzaj trivy-operator v0.1.0 has been released checkout the fixes in notes


Break down ODistributedLifecycleListener

Also, a minor change to the Hazelcast config file to remove the Hazelcast warning at startup and closes #9244.

@tglman @luigidellaquila Somehow I cannot choose reviewers. Please have a look at this.

thanks @pxsalehi we are already on it ;-)
Thanks
Luigi


Add files via upload

Robson Ferreira
Título: Solicitação de Licenças de Estudante da JetBrains
Descrição:
Olá,
Sou um estudante do 3º período de Análise e Desenvolvimento de Sistemas na Faculdade UNINASSAU e estou atualmente aprendendo a programar em Java e Kotlin como parte do meu currículo acadêmico.
Gostaria de solicitar acesso às licenças de estudante da JetBrains para poder utilizar as ferramentas fornecidas por eles, como o IntelliJ IDEA e o Kotlin Plugin. Essas ferramentas são amplamente reconhecidas pela sua qualidade e eficiência no desenvolvimento de software, e acredito que ter acesso a elas seria extremamente benéfico para a minha formação acadêmica.
Agradeço antecipadamente pela consideração da minha solicitação.
Atenciosamente,
Robson Ferreira

Title: Request for JetBrains Student Licenses
Description:
Hello,
I am a student in the 3rd period of Analysis and Systems Development at UNINASSAU College, currently learning to program in Java and Kotlin as part of my academic curriculum.
I would like to request access to JetBrains student licenses to be able to use their provided tools, such as IntelliJ IDEA and the Kotlin Plugin. These tools are widely recognized for their quality and efficiency in software development, and I believe having access to them would be extremely beneficial for my academic development.
Thank you in advance for considering my request.
Sincerely,
Robson Ferreira

@RobsonFe Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

You can provide us with any of the following as the proof:

A copy of instruction (a hard-copy or electronic one) given to you by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that students use this email domain
A reference to a recognized journal where an article was published by someone from the school, with explicit affiliation to the school, and e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be verified independently.

Hello!
Mauricio de Nassau University or UNINASSAU is the biggest university in the Northeast of the country in Brazil.

School Official Web : https://www.uninassau.edu.br, https://vestibular.uninassau.edu.br and https://aluno.sereduc.com.

My College Enrollment: 01266993

Name: Robson Ferreira da Silva

College Addres: Av. Sen. Salgado Filho, s/n - Centro, Paulista - PE, 53401-440

Part of site about the course https://vestibular.uninassau.edu.br/curso/85/4700/3/cst-analise-e-desenvolvimento-de-sistemas/Paulista-PE in Portuguese.

Some images

Some proves about my college.

Ser Educacional Web Site

My Login on web site

My conversation about JetBrains tools and my e-mail from college

About my period at school, 1 year to complete my course.

My College Address

Link to Google Maps
Thank you in advance for considering my request.
Sincerely,
Robson Ferreira


Append git version to cifmw galaxy collection

As setup_molecule is usually main installation method for installing cifmw.general galaxy collection,
append short git hash to its version for easier identification what code version is (or was) used,
otherwise it always states only:
$ ansible-galaxy collection list |grep cifmw
cifmw.general                 1.0.0
This change makes it slightly more useful
(as long as there is git command and git repo available, ignored otherwise):
cifmw.general                 1.0.0-a44ca164
As a pull request owner and reviewers, I checked that:

[X] Appropriate testing is done and actually running

recheck

/approve
/lgtm


Error：Exception: MoE JIT is designed to work on sample size = 800, while receiving sample size = 1600 (> 800)

It is expected to set an upper bound of sample size when creating tutel.moe_layer(..).
If not set by default, the upper bound will be automatically set to be the batch size from the first time of data forwarding.
To set the upper bound manual in your case, please add this line after the creation of tutel.moe_layer(..):
class MyModel(Module):

  def __init__(self):
    ...
    self._moe_layer = tutel_moe.moe_layer(..)
    self._moe_layer.expected_sample_size = 1600  # Add this line to set the upper bound manually
    ...


feat: add environment file for configuring docker compose

Description of the PR
Adds an environment file that can be used configure docker compose. At the moment, just image names and GraphQL API ports are configurable through the file.
@lumjjb I tried using ghcr.io/guacsec/guac:v0.1.0 but it doesn't seem to work. As far as I can see from Helm charts /opt/guac/ is moved to /cnb/process. And also there's no healthcheck image. Would it be OK to unify these two approaches and let folks use tagged images in compose as well? Please let me know if I can help with anything.
PR Checklist

[x] All commits have a Developer Certificate of Origin (DCO) -- they are generated using -s flag to git commit.
[ ] All new changes are covered by tests
[ ] If GraphQL schema is changed, make generate has been run
[ ] If collectsub protobuf has been changed, make proto has been run
[ ] All CI checks are passing (tests and formatting)
[ ] All dependent PRs have already been merged

@sunnyyip that's great, thanks ... I'll give it a test once it's done then ... and let me know if I can do anything else to help

@sunnyyip that's great, thanks ... I'll give it a test once it's done then ... and let me know if I can do anything else to help

The new release workflow is merged. You can test after the next release when we build a new image with the updates.

@dejanb if you rebase on main, it will fix the CI issues.

Pushed a fix that should hopefully fix test issues
https://github.com/guacsec/guac/pull/901/commits/e31a57c25406c066ca2cf1c822cd03954786ec49

Yup! That test is being flaky. That will fix it. Looks like you need to sign the commit and should be good to merge.

Sorry about that. Pushed it now


904 error message for statutory declaration field

ISSUE 904

Nice fix for #904
As FYI, I found 2 bugs associated with the Create Operations flow:
https://github.com/bcgov/cas-registration/issues/985
https://github.com/bcgov/cas-registration/issues/986


Respect //@ts-nocheck in TS files

Fixes parts of #19573 kinda.
Seeing as we already support //@ts-ignore, //@ts-nocheck is really just a file-scoped ignore, so there's nothing new here, really. It's also not really a big change.

@typescript-bot pack this

Approved in today's design meeting ❤


Misuses of cryptographic APIs

Hi
The following lines have cryptographic API misuses.  File name => server/src/main/java/com/adobe/testing/s3mock/domain/FileStore.java: Line number => 764: API name => MessageDigest:First parameter (with value "MD5") should be any of {SHA-256, SHA-384, SHA-512} File name => server/src/main/java/com/adobe/testing/s3mock/util/HashUtil.java: Line number => 70: API name => MessageDigest:Unexpected call to method reset on object of type java.security.MessageDigest. Expect a call to one of the following methods digest,update File name => server/src/main/java/com/adobe/testing/s3mock/domain/FileStore.java: Line number => 830: API name => MessageDigest:Operation on object of type java.security.MessageDigest object not completed. Expected call to digest, update

As intended.


[2183] Check what data you need content change

Context
We are updating our guidance page.
Changes proposed in this pull request

Remove allocations content and replace with trainee start date and id
Remove placement details content and replace with schools

Guidance to review
visit /guidance and review the content changes in line with the Trello ticket

 [S] Content change: Check what data you need

looks good!


Changing project name

Done:

Have changed the directory names to be UniExplore, and certain files where this file name needs to be referenced.
Made it so UniExplore shows at the top of all the pages.

To Test:

Make sure the app runs as normal.
View the app name at the top of all the pages.

 Create a name for the app


Add more documentation

This PR intends to document the whole project more thoroughly.

I'm not finished with this yet. The PR is for anyone to submit constructive criticism and/or request documentation in lacking areas.

I'm ready to merge this if there are no further comments.

Sure.

bors r+


Fix draggable handling in palette

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes
There's a draggable handling problem in the 1.0-beta version of Node-RED. When users drag a node to the outside central workspace, the node will return to the incorrect location (left edge of the palette) because $('#' + e.target.id).offset().left; always has 0 as the value currently. Therefore, I replaced the code with getPaletteNode(nt) to get the appropriate location.

Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the mailing list/slack team.
[x] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage remained the same at 77.534% when pulling dddfb1ec08fdfb0eaeb6b24b9309bbfd490b28d9 on kazuhitoyokoi:dev-fixnodedraggablehandling into 6fc9c03d707710446752763bd1e9d45b1c0f56e9 on node-red:dev.

Thank you!


fix property for column definitions in documentation overview

Hi,
only a small adjustment because I was wondering why the columns didn't render. In the documentation overview colDef is used, which does not work for me in version 3.2.2. Using columnDefs however, does.

thanks. i've made this change locally as i'm in the middle of refactoring and don't want to risk having to do a merge!!


List sizes should not be members

List sizes should be determined by the list object not a member variable.

This is done.


Expected network bandwidth for D2ds_v4 is confusing.

Size
Expected network bandwidth  (Mbps)

Standard_D2ds_v42
1000

Standard_F2s_v24
5000

i benchmarked with speedtest within two VMs, d2ds-v4 and f2s-v2, and found the inbound/outbound network performance are same.
so what does Expected network bandwidth really mean? should i perform the test by accessing the VM from outside?

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

ID: 6a43e5b4-7893-dcd7-839f-e394912d1bd9
Version Independent ID: 560d2e4a-6a12-48ac-dff7-44f036450250
Content: Ddv4 and Ddsv4-series - Azure Virtual Machines
Content Source: articles/virtual-machines/ddv4-ddsv4-series.md
Service: virtual-machines
Sub-service: vm-sizes-general
GitHub Login: @brbell
Microsoft Alias: brbell

@enihcam Thank you for your feedback . We will investigate and update the thread.

Hi @enihcam , thanks for reaching out. Expected network bandwidth means the maximum network throughput limit. If you used some benchmarking tool to test the network bandwidth/throughput between the two VMs, that is, D2ds_v4 and F2s_v2, then you will get the lower throughput for the VM that has the lower expected network bandwidth.
To test the throughput, you must use both VMs of same size.
Please refer to the below docs for more clarity and understanding on the same.
https://docs.microsoft.com/en-us/azure/virtual-network/virtual-machine-network-throughput
https://docs.microsoft.com/en-us/azure/virtual-network/virtual-network-bandwidth-testing
Hope this helps. Thank you.

Also, the network bandwidth allocated to each virtual machine is metered on egress (outbound) traffic from the virtual machine. All network traffic leaving the virtual machine is counted toward the allocated limit, regardless of destination. For example, if a virtual machine has a 1,000 Mbps limit, that limit applies whether the outbound traffic is destined for another virtual machine in the same virtual network, or outside of Azure.


Add support for LOCAL_PEER_CRED

On FreeBSD and its derivatives, this socket option gets the credentials
of the connected peer.

bors r+


Angular Nativescript "EmojiPicker" and "EmojiLabel" makes error on page

Hi.
When I add
	<EmojiPicker row="0" #myEmojiPicker></EmojiPicker>
			<Button row="1" text="Toggle Popup" (tap)="onButtonTap()" class="btn btn-primary btn-active"></Button>
			<EmojiLabel #myEmojiLabel row="2" text="Have fun with the nativescript-emoji-picker😃"></EmojiLabel>
			<Button row="3" text="Copy text to label" (tap)="onCopyBtnTap()" class="btn btn-primary btn-active"></Button>

to view, I get issue:
CONSOLE ERROR file:///node_modules/@angular/core/fesm5/core.js:5847:0 ERROR Error: Uncaught (in promise): TypeError: undefined is not a constructor (evaluating 'new viewClass()')
createView@file:///node_modules/nativescript-angular/view-util.js:215:0
createElement@file:///node_modules/nativescript-angular/renderer.js:138:0
createElement@file:///node_modules/nativescript-angular/renderer.js:415:0
createElement@file:///node_modules/@angular/core/fesm5/core.js:28475:0
createElement@file:///node_modules/@angular/core/fesm5/core.js:26511:0
createViewNodes@file:///node_modules/@angular/core/fesm5/core.js:27450:0
callViewAction@file:///node_modules/@angular/core/fesm5/core.js:27805:0
execComponentViewsAction@file:///node_modules/@angular/core/fesm5/core.js:27724:0
createViewNodes@file:///node_modules/@angular/core/fesm5/core.js:27517:0
createRootView@file:///node_modules/@angular/core/fesm5/core.js:27403:0
callWithDebugContext@file:///node_modules/@angular/core/fesm5/core.js:28411:0
create@file:///node_modules/@angular/core/fesm5/core.js:18750:0
createComponent@file:///node_modules/@angular/core/fesm5/core.js:18861:0
loadWithFactory@file:///node_modules/nativescript-angular/common/detached-loader.js:38:0
activateOnGoForward@file:///node_modules/nativescript-angular/router/page-router-outlet.js:283:0
activateWith@file:///node_modules/nativescript-angular/router/page-router-outlet.js:261:0
activateRoutes@file:///node_modules/@angular/router/fesm5/router.js:2315:0
file:///node_modules/@angular/router/fesm5/router.js:2266:0
forEach@[native code]
activateChil<…>
(ts part also added exactly same as in demo, and module is imported)
The same issue/error in demo-angular application
node v10
nativescript v6
angular v8

Hi,
are you using IOS or Android?
At the moment, the plugin works only in Android as I couldn't find a native library which provides an emoji picker. I forgot to add this to the readme.

Yes, we are trying to use this picker. I have built it on android today and cant see smile picker. Could you send some screen shot how does it look like? And are you going to get ios version to work in near feature ?

for me it looks like this and toggle popup button does not work

I'll try to make this plugin work on IOS as soon as I can.
I also added screenshots to the readme as you requested.
Are you sure that the onButtonTap method here
<Button row="1" text="Toggle Popup" (tap)="onButtonTap()" class="btn btn-primary btn-active"></Button>

is actually called?
You can check this for example by adding a console.log("Method called!"); line to the top of this method.
If this works, can you console.log(this.myEmojiPicker) to the console and see if it is undefined?

Hi, sorry for long time answering, I was switched to another urgent tasks. I got to work plugin on android, I can see picker, toggle button works. It was my mistake, I commented out some line when played with that on IOS. But I still need ios target. Hope you will be able to fix iso soon!Thank you!

Hi,
I'm glad that you resolved your issue.
I'm very sorry that I cannot provide an IOS version soon. I simply don't know much about IOS developement and it is not easy to do without a Mac.
But feel free to fork this repo and implement this by yourself. It shouldn't be too complicated with https://github.com/isaced/ISEmojiView
I would be very happy about a pull request!


TN-2417 provide ugly workaround to failed query

Hopefully a temporary workaround, run a second query for the entire json document if the query for the json path field fails.

Hello @pbugni! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file portal/models/questionnaire_response.py:

Line 375:40: E251 unexpected spaces around keyword / parameter equals

@ivan-c should we include a rollback of yesterdays SQLAlchemy downgrade attempt, as that didn't prove to be the problem?

@ivan-c should we include a rollback of yesterdays SQLAlchemy downgrade attempt, as that didn't prove to be the problem?

I'd prefer to minimize change to master until we can figure this out. I'll merge what we have in master into develop and restore Wayne's changes there; they'll become part of the next release.

@ivan-c should we include a rollback of yesterdays SQLAlchemy downgrade attempt, as that didn't prove to be the problem?

I'd prefer to minimize change to master until we can figure this out. I'll merge what we have in master into develop and restore Wayne's changes there; they'll become part of the next release.

Sure, sounds wise.

Looks good, thanks for branching from master!
Is there anything else (metadata, system state) we would want to log? Nothing comes to mind, but I'm not sure we'll be able to reproduce with QNR.id only

I really don't know what else to log.  We can't reproduce at all, but at least we'll have a pointer to the db row that behaved poorly.


PDI-15196 - Fixed skipping rows if json field is null, empty or if th…

Fixed skipping rows if json field is null, empty or if there are no matches with json fields and at the same time there are no additional not null row fields

Corrected filtering out rows and avoiding excess processRow calls.

Reworked process of result rows' storage and converting.

Changed 'Ignore Missing Path' behavior. Now it is unchecked it will throw an exception if there are at least one null json value in output row after parsing. Previously it failed only if  all the rows misses the same path.

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f plugins/kettle-json-plugin/build.xml clean-all resolve jacoco && ant -Dtestreports.dir=bin/reports/integration-test -f plugins/kettle-json-plugin/build.xml jacoco-integration checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  plugins/kettle-json-plugin/src/org/pentaho/di/trans/steps/jsoninput/JsonInput.java
  plugins/kettle-json-plugin/src/org/pentaho/di/trans/steps/jsoninput/exception/JsonInputException.java
  plugins/kettle-json-plugin/src/org/pentaho/di/trans/steps/jsoninput/reader/FastJsonReader.java
  plugins/kettle-json-plugin/test-src/org/pentaho/di/trans/steps/jsoninput/JsonInputTest.java

Still Broken Tests:
org.pentaho.di.trans.steps.jsoninput.JsonInputIntTest.testNoInput:
:red_circle: java.lang.NullPointerException
java.lang.NullPointerException
	at java.util.concurrent.ConcurrentHashMap.putVal(ConcurrentHashMap.java:1011)
	at java.util.concurrent.ConcurrentHashMap.put(ConcurrentHashMap.java:1006)
	at org.pentaho.di.core.variables.Variables.copyVariablesFrom(Variables.java:83)
	at org.pentaho.di.core.variables.Variables.initializeVariablesFrom(Variables.java:135)
	at org.pentaho.di.trans.Trans.initializeVariablesFrom(Trans.java:4465)
	at org.pentaho.di.trans.Trans.<init>(Trans.java:481)
	at org.pentaho.di.

org.pentaho.di.trans.steps.jsonoutput.JsonOutputTest.testCompatibilityMode:
:red_circle: java.lang.NullPointerException
java.lang.NullPointerException
	at java.util.concurrent.ConcurrentHashMap.putVal(ConcurrentHashMap.java:1011)
	at java.util.concurrent.ConcurrentHashMap.put(ConcurrentHashMap.java:1006)
	at org.pentaho.di.core.variables.Variables.copyVariablesFrom(Variables.java:83)
	at org.pentaho.di.core.variables.Variables.initializeVariablesFrom(Variables.java:135)
	at org.pentaho.di.trans.Trans.initializeVariablesFrom(Trans.java:4465)
	at org.pentaho.di.trans.Trans.<init>(Trans.java:481)
	at org.pentaho.di.

org.pentaho.di.trans.steps.jsonoutput.JsonOutputTest.testNonCompatibilityMode:
:red_circle: java.lang.NullPointerException
java.lang.NullPointerException
	at java.util.concurrent.ConcurrentHashMap.putVal(ConcurrentHashMap.java:1011)
	at java.util.concurrent.ConcurrentHashMap.put(ConcurrentHashMap.java:1006)
	at org.pentaho.di.core.variables.Variables.copyVariablesFrom(Variables.java:83)
	at org.pentaho.di.core.variables.Variables.initializeVariablesFrom(Variables.java:135)
	at org.pentaho.di.trans.Trans.initializeVariablesFrom(Trans.java:4465)
	at org.pentaho.di.trans.Trans.<init>(Trans.java:481)
	at org.pentaho.di.

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.trans.steps.jsoninput.exception.JsonInputException

Complexity New Coverage: 50.00%
Instruction New Coverage: 50.00%
Line New Coverage: 50.00%
Method New Coverage: 50.00%

org.pentaho.di.trans.steps.jsoninput.JsonInput

Branch Change: -1.08%:small_red_triangle_down:
Complexity Change: -1.39%:small_red_triangle_down:
Instruction Change:  + .99%
Line Change:  + .21%
Method Change:  + .35%

org.pentaho.di.trans.steps.jsoninput.reader.FastJsonReader

Branch Change:  + 3.04%
Complexity Change:  + 1.50%
Instruction Change: -.99%:small_red_triangle_down:
Line Change: -.40%:small_red_triangle_down:
Method Change:  + 2.69%

org.pentaho.di.trans.steps.jsoninput.reader.FastJsonReader.TransposedRowSet

Branch Change:  + 12.50%
Complexity Change:  + 5.56%
Instruction Change: -7.23%:small_red_triangle_down:
Line Change: -9.47%:small_red_triangle_down:

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f plugins/kettle-json-plugin/build.xml clean-all resolve jacoco && ant -Dtestreports.dir=bin/reports/integration-test -f plugins/kettle-json-plugin/build.xml jacoco-integration checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  plugins/kettle-json-plugin/src/org/pentaho/di/trans/steps/jsoninput/JsonInput.java
  plugins/kettle-json-plugin/src/org/pentaho/di/trans/steps/jsoninput/exception/JsonInputException.java
  plugins/kettle-json-plugin/src/org/pentaho/di/trans/steps/jsoninput/reader/FastJsonReader.java
  plugins/kettle-json-plugin/test-src/org/pentaho/di/trans/steps/jsoninput/JsonInputTest.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.trans.steps.jsoninput.exception.JsonInputException

Complexity New Coverage: 50.00%
Instruction New Coverage: 50.00%
Line New Coverage: 50.00%
Method New Coverage: 50.00%

org.pentaho.di.trans.steps.jsoninput.JsonInput

Branch Change: -1.03%:small_red_triangle_down:
Complexity Change: -1.37%:small_red_triangle_down:
Instruction Change:  + .99%
Line Change:  + .23%
Method Change:  + .35%

org.pentaho.di.trans.steps.jsoninput.reader.FastJsonReader

Branch Change:  + 3.04%
Complexity Change:  + 1.50%
Instruction Change: -.99%:small_red_triangle_down:
Line Change: -.40%:small_red_triangle_down:
Method Change:  + 2.69%

org.pentaho.di.trans.steps.jsoninput.reader.FastJsonReader.TransposedRowSet

Branch Change:  + 12.50%
Complexity Change:  + 5.56%
Instruction Change: -7.23%:small_red_triangle_down:
Line Change: -9.47%:small_red_triangle_down:

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f plugins/kettle-json-plugin/build.xml clean-all resolve jacoco && ant -Dtestreports.dir=bin/reports/integration-test -f plugins/kettle-json-plugin/build.xml jacoco-integration checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  plugins/kettle-json-plugin/src/org/pentaho/di/trans/steps/jsoninput/JsonInput.java
  plugins/kettle-json-plugin/src/org/pentaho/di/trans/steps/jsoninput/exception/JsonInputException.java
  plugins/kettle-json-plugin/src/org/pentaho/di/trans/steps/jsoninput/reader/FastJsonReader.java
  plugins/kettle-json-plugin/test-src/org/pentaho/di/trans/steps/jsoninput/JsonInputTest.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.trans.steps.jsoninput.exception.JsonInputException

Complexity New Coverage: 50.00%
Instruction New Coverage: 50.00%
Line New Coverage: 50.00%
Method New Coverage: 50.00%

org.pentaho.di.trans.steps.jsoninput.JsonInput

Branch Change: -2.22%:small_red_triangle_down:
Complexity Change: -2.73%:small_red_triangle_down:
Instruction Change:  + .61%
Line Change: -.37%:small_red_triangle_down:
Method Change: -3.99%:small_red_triangle_down:

org.pentaho.di.trans.steps.jsoninput.reader.FastJsonReader

Branch Change:  + 6.32%
Complexity Change:  + 3.30%
Instruction Change:  + .80%
Line Change:  + .60%
Method Change:  + 2.69%

org.pentaho.di.trans.steps.jsoninput.reader.FastJsonReader.TransposedRowSet

Branch Change:  + 12.50%
Complexity Change: -11.11%:small_red_triangle_down:
Instruction Change: -21.51%:small_red_triangle_down:
Line Change: -19.47%:small_red_triangle_down:
Method Change: -20.00%:small_red_triangle_down:

Please, do not merge it!


Add BuildKonfig to support production api url

Issue

close #232

Overview (Required)

Kotlin Multiplatform Project does not support product flavor. So we need to provide additional property in order to determine flavors.

Specify default flavor in root gradle.properties
buildkonfig.flavor=dev
buildkonfig.flavor=prod

and also we can pass value via CLI $ ./gradlew build -Pbuildkonfig.flavor=prod
Links

https://github.com/yshrsmz/BuildKonfig#product-flavor

👀


[CommentFix] [CodeLittleFix] Fix indentation of function comment and little code change

I fixed comment indentation to clarify the comment paragraph structures,
and changed ForEach-Object to foreach expression to fasten the result.

I don't agree with these changes.

The manifest was autogenerated by New-ModuleManifest. I'd rather keep with the output as it is generated.
I prefer the comment-based help as it is which is also consistent with the samples in the docs for comment-based help.
$Command (singular) as the parameter name is consistent with how PowerShell cmdlets treat parameters that take lists.
I prefer ForEach-Object over foreach.
You've removed some optional tokens but I prefer keeping them.
Format-WslArgument doesn't need to be an advanced function.

Thank you very much for your clear explanation of rejection.
I have been convinced.


[BUG]: Inserts do not work, using the example with postgres.

orange Version
4.3.0
Database
Postgres
Database Version
16.2
Operating System
Windows 11
Description
Postgres tables and columns are automatically stored in lowercase if they are not surrounded by quotes.
ERROR  [nuxt] [request error] [unhandled] [500] column "isActive" of relation "customer" does not exist

Expected Behavior
I am not sure, because if you made it case insensitive it would break CREATE STATEMENTs where quotes are applied.

Another error I got using the example:
ERROR  [nuxt] [request error] [unhandled] [500] invalid input syntax for type integer: "true"

Are you using the sql  example  in the docs ?
That is sqlite - not Postgres.
You need to quote all columns and tables in the create schema.

I guess I could add sql for Postgres in the docs as well

Try this , you can find more examples in the tests https://github.com/alfateam/orange-orm/edit/master/tests/initPg.js

CREATE TABLE customer (
    id SERIAL	 PRIMARY KEY,
    name TEXT,
    balance NUMERIC,
    "isActive" BOOLEAN,
);

CREATE TABLE "_order" (
    id SERIAL PRIMARY KEY,
    "orderDate" TIMESTAMP,
    "customerId" INTEGER REFERENCES customer
);

CREATE TABLE "orderLine" (
    id SERIAL PRIMARY KEY,
    "orderId" INTEGER REFERENCES "_order",
    product TEXT,
    amount NUMERIC(10,2) NULL
);

CREATE TABLE package (
    "packageId" SERIAL PRIMARY KEY,
    "lineId" INTEGER REFERENCES "orderLine",
    sscc TEXT
);

CREATE TABLE "deliveryAddress" (
    id SERIAL PRIMARY KEY,
    "orderId" INTEGER REFERENCES "_order",
    name TEXT,
    street TEXT,
    "postalCode" TEXT,
    "postalPlace" TEXT,
    "countryCode" TEXT
)

Try this , you can find more examples in the tests https://github.com/alfateam/orange-orm/edit/master/tests/initPg.js
CREATE TABLE customer (
    id SERIAL	 PRIMARY KEY,
    name TEXT,
    balance NUMERIC,
    "isActive" BOOLEAN,
);

CREATE TABLE "_order" (
    id SERIAL PRIMARY KEY,
    "orderDate" TIMESTAMP,
    "customerId" INTEGER REFERENCES customer
);

CREATE TABLE "orderLine" (
    id SERIAL PRIMARY KEY,
    "orderId" INTEGER REFERENCES "_order",
    product TEXT,
    amount NUMERIC(10,2) NULL
);

CREATE TABLE package (
    "packageId" SERIAL PRIMARY KEY,
    "lineId" INTEGER REFERENCES "orderLine",
    sscc TEXT
);

CREATE TABLE "deliveryAddress" (
    id SERIAL PRIMARY KEY,
    "orderId" INTEGER REFERENCES "_order",
    name TEXT,
    street TEXT,
    "postalCode" TEXT,
    "postalPlace" TEXT,
    "countryCode" TEXT
)

Thank you for your quick responsese! I know that the SQL was for sqlite. I updated it to fit the postgres dialect and ended up with the same statements (almost). Just forgot to also add quotes to the table names, because I normally do not name tables in camelCase (I normally use SCREAMING_SNAKE_CASE for everything in databases, so that there are no case issues). Massive skill issue on my side. Sorry for stealing your time like that. I wish orange-orm would let me define the actual tables created in the database and not just a type map. That way, I wouldn't have had that issue.

No problem. I am glad to help.
Yes, I will create cli command for generating sql from the map. Also, the other way around.


Coordinate axis convetions

What are the coordinate conventions used for camera pose matrices in cameras.npz?  Are you using the NeRF/OpenCV standard of "+Z down the optical axis, +X to the right and +Y is down"?

Some resources I found useful:
https://github.com/autonomousvision/differentiable_volumetric_rendering/blob/master/FAQ.md
https://github.com/autonomousvision/differentiable_volumetric_rendering/issues/3
https://github.com/autonomousvision/differentiable_volumetric_rendering/issues/64


Komga tracking not enabled by default when adding an entry to the library

Steps to reproduce

Install de Komga extension (1.4.50) and setup
Explore the newly configured Komga server
Long tap on an entry to add it to the Mihon library
Open the newly added entry, tracking is not enabled by default

Expected behavior
Step 3 should add the Komga's entry to Mihon library and enable tracking.
Actual behavior
Step 3 adds the Komga's entry to Mihon library but does not enable tracking.
EDIT: I had the same issue with Tachiyomi.
Crash logs
No response
Mihon version
0.16.3
Android version
Android 13
Device
Pixel 4a
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version 0.16.3.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Is your extension name matching eu.kanade.tachiyomi.extension.all.komga.Komga ? (https://github.com/mihonapp/mihon/issues/304)
From what I remember, it was working as long as you have the right name and had enabled it in the settings.

I can reproduce this from original tachiyomi and is probably related to #88 and can confirm tracking works just the tracker doesn't get added automatically on long press.

Is your extension name matching eu.kanade.tachiyomi.extension.all.komga.Komga ? (#304) From what I remember, it was working as long as you have the right name and had enabled it in the settings.

I'm gonna have to check that. But yes as @THElegend5 pointed out, the tracking works but you have to manually enable it for every entries you add to the library, instead of it being enabled automatically when long pressing an entry.

Is your extension name matching eu.kanade.tachiyomi.extension.all.komga.Komga ? (#304) From what I remember, it was working as long as you have the right name and had enabled it in the settings.

I checked, it is matching the name you mentioned : eu.kanade.tachiyomi.extension.all.komga.

Issue still persists on Mihon 0.16.5
Komga Version is 1.4. 57 and the name is 'all.komga'
Also happens on latest preview build of TachiyomiSY

Closed via #1298


Keep track of activity's channels

Changes proposed ✍
This PR contains code for the following.

Creating a migration for adding a column called activityChannels in Settings table.
This Column is kept up to date using the add activity API, when new channels are encountered, they are then updated in the settings table in the respective platform.

Checklist ✅

[ ] Label appropriately with Feature, Enhancement, or Bug.
[x] Tests are passing.
[x] New backend functionality has been unit-tested.
[ ] Environment variables have been updated:

[ ] Local frontend configuration: frontend/.env.dist.local, frontend/.env.dist.composed.
[ ] Local backend: backend/.env.dist.local, backend/.env.dist.composed.
[ ] Configuration docs have been updated.
[ ] Team members only: update environment variables in override, staging and production env. files and trigger update config script.

[x] API documentation has been updated (if necessary) (see docs on API documentation).
[x] Quality standards are met.
[x] All changes have been tested in a staging site.
[x] All changes are working locally running crowd.dev's Docker local environment.

 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.


does this support more than 1 acc or just one

it doesnt

just one account

just one account

ye i saw i kinda need unlimited but okay


chore: update repository template to c0ade567

Updated repository templates to https://github.com/ory/meta/commit/c0ade56771b1a8bad929f03807de5c01b71c1e57.

Coverage remained the same at 43.429% when pulling f33eb4067ecdce710970fd0af88e1d373ebd4add on meta-12-15-20-07-46-58 into d0c037489810d5f03877c070fbe31fa1d8572992 on master.


changing background color / link colors (this could be user error!!)

I am trying to change the background of the title side and the link color. I have added
$link-color: #70001A;

and
#title-slide {
  background-size: cover;
  background-color: #fff;
  color: #404040;
}

to my custom CSS but the former seems to be defaulting to the theme regardless of what I put in and the latter does this weird thing:

I'm not sure what's up with the first one, but the second one is because quarto does title styling weirdly: https://github.com/malcolmbarrett/kakashi-quarto-theme/blob/c147c0f2b2fe3e2d7efeca21de0843f693fec421/_extensions/kakashi/_extension.yml#L23-L25

How are you including your css file?

Ohhh so do I need to update the yaml for the title? This is how I include
my css:
format:
kakashi-revealjs:
theme: [custom.scss]
slide-number: true
chalkboard: true
Lucy D'Agostino McGowan, PhD
Assistant Professor
Department of Statistical Sciences
Wake Forest University
✉ @.***
🌐 lucymcgowan.com https://www.lucymcgowan.com
On Fri, Jan 20, 2023 at 1:41 PM Malcolm Barrett @.***>
wrote:

How are you including your css file?
—
Reply to this email directly, view it on GitHub
https://github.com/malcolmbarrett/kakashi-quarto-theme/issues/3#issuecomment-1398789793,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACAKSGNF2I6HVRRCIJODZITWTLL7FANCNFSM6AAAAAAUB2IQ4Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

Ohhh so do I need to update the yaml for the title?

I think so
I wonder if you need something like theme: [kakashi.scss, custom.scss]?

Do you mean I would have the first link to the path within the extension
(like [_extensions/malcombarret/kakashi/kakshi.scss, custom.scss]? as
written it says the first doesn't exist.
On Fri, Jan 20, 2023 at 2:13 PM Malcolm Barrett @.***>
wrote:

Ohhh so do I need to update the yaml for the title?
I think so
I wonder if you need something like theme: [kakashi.scss, custom.scss]?
—
Reply to this email directly, view it on GitHub
https://github.com/malcolmbarrett/kakashi-quarto-theme/issues/3#issuecomment-1398823135,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACAKSGL6OLNAUZ7KFKVWC5TWTLPU7ANCNFSM6AAAAAAUB2IQ4Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hm I don't know. I thought it might resolve it itself

Does this work? I don't know why it would, just thinking through what's different about the link color
$pink: #70001A;

:( did not work. I think the key is anything that is input as a quarto variable in the .scss doc seems to not be able to be "overwritten" by my custom .scss file? (so less a problem with $pink and more with $link-color)

I wonder if it's because I'm not using !default in the sass file https://sass-lang.com/documentation/variables#default-values
Will you try adding that in your local copy of the kakashi.scss file, like $link-color: $pink !default;?

That's it!

Fixed on main. Also fixed a few other things so you might want to copy it over


To add a prompt showing the user limit per day

The custom code editor currently uses the free tier version of judge0 , so the requests is limited to 50 requests per day.
Need to add this prompt to users who are currently using the system

where we can show the number of requests on the code editor ?

@nikkivirtuoso just adding a prompt in the home page would be enough, just as note or instructions for someone using the app
No need to do operations based on error or so

if tihs issue is still open, i can take it

Can you review it now?


Docs: Explain scope Field within configuration Contribution Point...

the docs for the configuration contribution point make no mention of the scope field - yet we are warned about setting it / not setting it under certain circumstances. what does scope do?
docs:
https://code.visualstudio.com/docs/extensionAPI/extension-points#_contributesconfiguration
example:
https://github.com/Microsoft/vscode-extension-samples/blob/master/configuration-sample/package.json#L42
warning:
https://github.com/Microsoft/vscode/issues/36623
https://github.com/Microsoft/vscode/wiki/Extension-Authoring:-Adopting-Multi-Root-Workspace-APIs#run-time-diagnostics
thanks 🙏

This issue was supposed to be created in Microsoft/vscode-docs.

ah, whoops! thanks @KamasamaK
closing


ImageCache storeImage not using data from download

Is there a reason why storeImage computes its data from the UIImage instead of using the nsdata retrieved from the download? for jpg images although quality is at 100% it is compressed again.
(which also results in way bigger images than the ones that could be received from the download)
I'm wondering, are there any performance reasons?
Or could this be a usefull PR?

It is more likely a historical reason. There is no original data input for this API. Currently, it could ensure the data is valid and could represent an image. However, this is not a must in my opinion since we should  be able to trust the data from sever here. Another reason is that the ImageCache could be used separately.  This could make sure even if the user did not pass in an originalData, we could also store the image correctly.
A PR on it is warmly welcomed. We should respect the originalData and store it directly. If it is lack of the data, we should convert the input image to data and store the result data instead.


antigen theme issue?

MacOS 10.12.5
iTerm 3.0.15
zsh 5.2
antigen 79782288ef286fda8b48f1b3a82aa6f9bdf16864
using git method that cloned the repo to my local.
setup .zshrc with example
`source $HOME/dotfiles/antigen/antigen.zsh
Load the oh-my-zsh's library.
antigen use oh-my-zsh
Bundles from the default repo (robbyrussell's oh-my-zsh).
antigen bundle git
antigen bundle heroku
antigen bundle pip
antigen bundle lein
antigen bundle command-not-found
Syntax highlighting bundle.
antigen bundle zsh-users/zsh-syntax-highlighting
Load the theme.
antigen theme robbyrussell
Tell Antigen that you're done.
antigen apply
`
restart iTerm, and it's stopping in Installing
Installing robbyrussell/oh-my-zsh!... Installing zsh-users/zsh-syntax-highlighting!...
removed antigen theme robbyrussell and rm -rf ~/.antigen, terminal is working but without theme but don't think any bundles are working.

@sthtodo It's an issue with latest release, specifically this issue.
Locally I could interactively use it without issue, but I'm gonna take a deeper look.
Thanks for reporting.

 Let us know of any issue if you have!


pdfx: Migrate to package:web to support WASM

The short version:

Migrate usage of dart:html to package:web
Migrate usage of dart:js, dart:js_util, and package:js to dart:js_interop

This will allow compiling to WASM engine instead of just JavaScript on web.
Full migration guide: https://dart.dev/interop/js-interop/package-web

@SergeShkurko is there a plan for a new version with this change? ☺

@SergeShkurko do you plan to release a new version? ☺


test: use call GlobalFilter.set()

Description
Update call from Settings to GlobalFilter dedicated API.
Motivation and Context
Use existing API which is used everywhere in the code.
How has this been tested?
Unit test.
Types of changes
Internal changes:

[x] Tests (prefix: test - additions and improvements to unit tests and the smoke tests)

Checklist

[x] My code follows the code style of this project and passes yarn run lint.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] My change has adequate Unit Test coverage.

Terms

[x] My contribution follow this project's contributing guide
[x] I agree to follow this project's Code of Conduct

Thanks!


Wrong version in plugin.xml

package.json correctly contains 1.2.0
plugin.xml still says 1.1.0
Just lost a bit of time trying to work out why even after pulling from Github cordova plugin list was still reporting version 1.1.0!

Fixed in 1.2.1


Application fails after injecting the dependency

Is there are pre-requisites for the tool? I have a java 11 springboot project and it fails after injecting the dependency
`***************************
APPLICATION FAILED TO START

Description:
An attempt was made to call a method that does not exist. The attempt was made from the following location:
com.maciejwalkowiak.spring.boot.startup.StartupEventsInitializer.initialize(StartupEventsInitializer.java:17)

The following method did not exist:
'org.springframework.core.metrics.ApplicationStartup org.springframework.context.ConfigurableApplicationContext.getApplicationStartup()'

The method's class, org.springframework.context.ConfigurableApplicationContext, is available from the following locations:
jar:file:/home/deniz/.m2/repository/org/springframework/spring-context/5.2.9.RELEASE/spring-context-5.2.9.RELEASE.jar!/org/springframework/context/ConfigurableApplicationContext.class`

It must be Spring Framework 5.3+. I'll update the readme

Looks like it also requires BufferingApplicationStartup class that was added in Spring Boot 2.4. I think it would be nice to add to the documentation (readme) as well.


Notebook cell list focus/selection model

The notebook editor is a list widget/view under the hood, thus users can use their familiar list view keybindings to navigate and select cells in the notebook editor. It works well with single selection mode but fell apart after we turned on the multiple selection mode.
Selection/Focus model difference
The multi-selection/focus mode in the builtin list view seems inherits the concept of selection/focus of the File Explorer (VS Code one, not the OS one), and the major difference we are running into is

List View

Single click on an entry selects and focuses on the entry
Arrow up/down will move the focus to previous/next entry and it does not move or expand the selections. Side effect of this is focused items don't necessarily need to be selected items.

Notebook

Single click on an entry selects and focuses on the entry
Arrow up/down will move the focus and selection to previous/next entry
Shift+Arrow up/down will expand the selection, and also move the focus to previous/next entry
Click+Arrow up/down behaves the same

The list view behavior makes sense for the File Explorer as it's a master-details view, the selected entry might be opened in the editor pane so arrow up/down moves the focus, but not the "selection". However in Notebook, it's simply a list, pretty close to Finder, arrow keys just moves the focus/selection up and down. The only difference between Notebook list and Finder is probably that in Finder, there is no "focus" but in notebook, the lastly activated item is the focused element.
Proposal 1
When @roblourens and me discussed about this, we talked about if we should align completely with the default list view. The catch is it feels weird in notebook list, even for the simplest scenario

Click the first cell, it's selected and focused
Arrow down, the second cell is focused, the first cell is selected. The first cell has a selection background while the second cell has a focus border

It's beyond my expectation and IMHO users will get confused too. The advantage of this solution is though all list commands and features just work seamlessly, and we don't need to tweak how selections/focus are handled, which will definitely lead to interesting bugs.
Proposal 2
The other way around is keeping our current UX in notebook, but I found that I'm almost fighting against the list view logic (how it setFocus and setSelection):

Arrow down, calls setFocus only
Shift + Arrow down, calls setFocus and then setSelection

Whether the selection should be updated depends on if a setSelection call is invoked and we can't tell that when setFocus is invoked. Almost every list view navigation/selection related commands leverage setFocus and setSelection to archive the file_explorer_style experience. To completely workaround this in notebook, it seems we have to

Implement our own list down/up, expandDown/up, selectDown/up commands, etc
For focusFirst, focusLast, focusPageUp, focusPageDown commands, then setFocus is called, make sure the focused element is also selected

The drawback is list commands are no longer useable in Notebook and we may also want to disable list commands completely otherwise users will run into weird state as list commands can call setFocus and setSelection in a wrong order (wrong = not what we expect).

@joaomoreno I'd like to hear your thoughts on this, especially how to approach this if we want a different focus/selection model than the default one. Currently I prefer the second approach but found it challenging to implement.

cc @misolori and @roblourens for inputs.

That is: what do you expect to happen when pressing all combinations of Ctrl/Shift with arrow keys

sorry that I didn't make it clear:

arrow down, moves the focus/select to next
shift arrow down, expands the selection to next, move focus to next

now we don't use ctrl/alt modifier yet so above two are the only major difference and what we expect in a notebook list.

After talking to @rebornix I created the following items:

https://github.com/microsoft/vscode/issues/118044
https://github.com/microsoft/vscode/issues/118045
https://github.com/microsoft/vscode/issues/118046

Already supported via List View's new navigation model.


main.py very informal message

Hey @shajin-sha I think that this message is not very informal message. I think it will be better to write a nice message, like Hi There 👋, thanks for trying out bionic. We the team of Bionic wish you a very happy development with Bionic.

Closed #16


Support conditional requires with pip as well

Use python version specification in the list of required packages
instead of runtime condition so that it works with pip as well.

@sbertramrh, this should fix your issue on el8.

Thanks for the patch, this is way better than what I introduced into setup.py. No idea why I didn't do this in the first place.


store/postgres: make sure only one graph-node runs database migrations

We had the problem that if multiple graph-nodes started up simultaneously,
all of them would try to run database migrations. Only one of them would
succeed, and the others would have their migrations rolled back. That
causes issues with slow or expensive migrations because it forces the
database to do a lot of unnecessary work.
With this change, only one graph-node will migrate the database, all the
others will wait for it to finish by blocking on a new
__graph_node_global_lock table.

I tested this by running cargo test while creating/locking __graph_node_global_lock in a psql session and making sure the tests succeeded once I committed the txn in the psql session.

Rebased to latest master and made the requested changes


Recent changes break highlights from other sources

Hi,
I recently tried coc.nvim and saw highlights disappear after updating gruvbox. The issue is that
https://github.com/ellisonleao/gruvbox.nvim/blob/08a551cc8140e6ca41b5de6cabf75d02a246c330/lua/gruvbox/init.lua#L83
was previously conditional on vim.g.colors_name. It can be fixed by executing colorscheme gruvbox within init=function() ... end when using lazy.nvim.

require('lazy').setup({
...
  {
    'neoclide/coc.nvim',
    branch='release',
    build=':CocInstall coc-pyright',
  },
...
  {
    'ellisonleao/gruvbox.nvim',
    priority=1000,
    init=function() vim.cmd([[colorscheme gruvbox]]) end,
  },
...
})

...

-- vim.cmd([[colorscheme gruvbox]])

and a few hl links in a FileType autocmd (disabled while testing)

@notEvil thanks. Can you explain a little bit more what are the highlights with issues? Are those from Coc itself or Gruvbox? Can you try to inspect some of them using the :Inspect command and see what are the outputs please?

top before
bottom after moving colorscheme gruvbox to init
Don't think you need to investigate any further. The change in https://github.com/ellisonleao/gruvbox.nvim/commit/a749759e66341696a08f5e1bc572b4ccb121281b obviously clears the highlights on colorscheme gruvbox that coc sets up earlier.

I see, will rollback that change until i figure out another way

done in https://github.com/ellisonleao/gruvbox.nvim/commit/65f2b9e3b554c4579129e393d76044068ab34eed


How to use PEFT to develop new parameter efficient fine tuning methods

Can I use PEFT to develop new parameter efficient fine tuning methods? Are there any interface to allow me to do this?

Hello @GeyuanZhang, Thank you for your interest in adding new methods to the repo.  Please find below rough guidelines on how the library is structured, the interfaces to implement and suggestions going forward:

Each parameter-efficient fine-tuning method is in a separate file in the tuners folder : peft/src/peft/tuners at main · huggingface/peft · GitHub
Main class if PeftModel in peft/peft_model.py at main · huggingface/peft · GitHub with downstream nlp task specific class implementations, e.g., PeftModelForSeq2SeqLM
Each tuner has following interfaces, taking peft/lora.py at main · huggingface/peft · GitHub as example:
A config extending PeftConfig
A tuner-specific class implementing the PEFT logic for the corresponding method.
Currently, only LoRA method modifies the base model in place by injecting additional layers. Prompt learning based methods (Prompt Tuning, P-Tuning and Prefix-Tuning) are not modifying the base models and their logic is coupled to the nlp task-specific classes such as PeftModelForSeq2SeqLM
Going forward, all the methods should implement the logic in the way LoRA integration is implemented by injecting/modifying the base model so that it is widely applicable to most of the models and without having to depend on the nlp task-specific classes such as PeftModelForSeq2SeqLM. This has made it quite popular among practitioners.
Give proper citations to method, code and implementation utilities
Write a few tests and example notebooks to test out the peft method and have it as ready to use artefact for end users. This has particularly helped with the adoption of PEFT.

Follow the LoRA implementation as a reference and develop the new method following it

Hello @GeyuanZhang, Thank you for your interest in adding new methods to the repo. Please find below rough guidelines on how the library is structured, the interfaces to implement and suggestions going forward:

Each parameter-efficient fine-tuning method is in a separate file in the tuners folder : peft/src/peft/tuners at main · huggingface/peft · GitHub
Main class if PeftModel in peft/peft_model.py at main · huggingface/peft · GitHub with downstream nlp task specific class implementations, e.g., PeftModelForSeq2SeqLM
Each tuner has following interfaces, taking peft/lora.py at main · huggingface/peft · GitHub as example:
A config extending PeftConfig
A tuner-specific class implementing the PEFT logic for the corresponding method.
Currently, only LoRA method modifies the base model in place by injecting additional layers. Prompt learning based methods (Prompt Tuning, P-Tuning and Prefix-Tuning) are not modifying the base models and their logic is coupled to the nlp task-specific classes such as PeftModelForSeq2SeqLM
Going forward, all the methods should implement the logic in the way LoRA integration is implemented by injecting/modifying the base model so that it is widely applicable to most of the models and without having to depend on the nlp task-specific classes such as PeftModelForSeq2SeqLM. This has made it quite popular among practitioners.
Give proper citations to method, code and implementation utilities
Write a few tests and example notebooks to test out the peft method and have it as ready to use artefact for end users. This has particularly helped with the adoption of PEFT.

Follow the LoRA implementation as a reference and develop the new method following it

Thanks a lot!

Hello @GeyuanZhang, Thank you for your interest in adding new methods to the repo. Please find below rough guidelines on how the library is structured, the interfaces to implement and suggestions going forward:

Each parameter-efficient fine-tuning method is in a separate file in the tuners folder : peft/src/peft/tuners at main · huggingface/peft · GitHub
Main class if PeftModel in peft/peft_model.py at main · huggingface/peft · GitHub with downstream nlp task specific class implementations, e.g., PeftModelForSeq2SeqLM
Each tuner has following interfaces, taking peft/lora.py at main · huggingface/peft · GitHub as example:
A config extending PeftConfig
A tuner-specific class implementing the PEFT logic for the corresponding method.
Currently, only LoRA method modifies the base model in place by injecting additional layers. Prompt learning based methods (Prompt Tuning, P-Tuning and Prefix-Tuning) are not modifying the base models and their logic is coupled to the nlp task-specific classes such as PeftModelForSeq2SeqLM
Going forward, all the methods should implement the logic in the way LoRA integration is implemented by injecting/modifying the base model so that it is widely applicable to most of the models and without having to depend on the nlp task-specific classes such as PeftModelForSeq2SeqLM. This has made it quite popular among practitioners.
Give proper citations to method, code and implementation utilities
Write a few tests and example notebooks to test out the peft method and have it as ready to use artefact for end users. This has particularly helped with the adoption of PEFT.

Follow the LoRA implementation as a reference and develop the new method following it

@pacman100 Thanks a lot!


Issue when pasting a copied URL from the address bar of a browser

When pasting a copied URL from the address bar of a browser into a new post, instead of the URL being inserted (like in Twitter), the title of the page and domain of the URL appear.

If this is a bug, please mark it as such.

Can you show a screenshot or video or give more details please?

https://github.com/bluesky-social/social-app/assets/6194332/db9e87e8-b894-4c9c-9132-2ebb29f3cab5

Okay thanks for the video. I decided to delete it so we don't share your desktop any longer than necessary.
I'm not sure what's causing that, but that's absolutely not bluesky.  As far as I know, our app wouldn't be able to produce that information even if we wanted to.
Sorry I can't help, good luck

It doesn't happen in Twitter though.

This should fix it:
https://stackoverflow.com/questions/67560944/how-to-force-browser-to-paste-url-and-not-page-title-in-edge

What do you think?

I'll be darned, you might be right. I apologize for dismissing this prematurely. I'll look into it.

Any news?

@estrattonbailey

Just cosmetics, but maybe it coudl also be fixed:
Chrome on Mac, first time I see this today: in the composer, copying an URL shows the last letter of the URL black, as if it was unliked.

After posting, the links look normal https://bsky.app/profile/manuschwendener.bsky.social/post/3k6bgsmtfzo2v

Found https://github.com/bluesky-social/social-app/issues/1293#issuecomment-1694682160, which seems related, but is closed
"whether or not the underlining happens doesn't actually reflect what will be sent on the post"

To reproduce (Chrome on Mac):
add a carriage return before you paste the URL: both URLs start on a new line. The display bug does not happen if I just paste the URL after the text.
Also happens if I first just paste the URL after the text and then insert the CR before the link; also, if the CR is not right in front of the link.

Same on Firefox on Mac.
And: the number of Carriage returns determines how many characters do not look linked before submitting the post.

I thought this would be an easy fix.

I hope it gets fixed by the time a new release gets used.

 It's up to the browser and OS to use the correct clipboard buffer, and the solution linked above doesn't work out of the box. We'll keep this one in the backlog for now 👍
For the time being, users who wish to use Edge can either elect to disable this pasting behavior as noted in the link above, or use the built-in Right Click > Paste as > Plain text to work around it.

I wonder how Twitter does it.

I switched to using Thorium (which is based off of Chromium) and no longer have this issue. It would still be nice to get it fixed for those that still use Edge, etc.

From #4301 (cc: @mary-ext):

could you try seeing if Ctrl+Shift+V would give you the URL? that should get it to do a plain-text paste I believe.

It works just fine (on Windows, Edge)! Will do it for me, but I imagine other users will remain low-key frustrated.
Also noticing that this link: https://reactrouter.com/en/main/guides/api-development-strategy#current-future-flags looks like this: API Development Strategy v6.23.1 | React Router
Whereas this link: https://bsky.app/ looks like this: Following — Bluesky (bsky.app)
In the post composer, bsky.app looks like a clickable link, so obviously there is something going on that is making that one works better.


img data-src attribute

If you use data-src attribute angular loads the image depending the position of the attribute.
http://jsfiddle.net/ryo2vg88/3/

I'm not sure what you are trying to do, but if you want to provide a dynamic URL for an image you should use ngSrc directive: https://docs.angularjs.org/api/ng/directive/ngSrc
Having both src and data-src on one element doesn't make much sense.

Just try to use data-src attribute to lazy load images.

http://jsfiddle.net/ryo2vg88/5/

+1
Hey @pkozlowski-opensource.
Can you please elaborate:

 adds a src on the image tag (expected)
 doesn't do anything (expected).
 replaces src with data-src attribute in chrome, safari (webkit), not in firefox (not expected).
 replaces src with data-src attribute in firefox, not in webkit browsers (not expected).

So when you both have a data-src and src attribute on an image angular somehow tries to resolve it but it doesn't work consistently across browsers!


[$500] Test

Test
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01b3c10fe3e8118504
Upwork Job ID:<PHONE_NUMBER>200601088

Ignore, testing.


Add storybook

refs: #19

npm audit reveals that there is a high security vulnerability with markdown-to-jsx. However, this code should not be executed in a production environment. refs: #12


chore: move examples into dedicate crates

Closing as the eventually-app-example has been moved out of this repository.


Opening any project creates empty folder in userdata

Issue description:
Opening any project in Godot automatically creates an empty directory for user data. My app_userdata is completely littered with random projects I opened over the months. When I eventually need to use the directory, seeing all these random folders just puts me off.
Isn't it possible to create the directory only if it's actually used?
Steps to reproduce:

Create a project
Open it
Yet another empty user directory in your collection

I don't think empty folders in app_userdata are that big an issue, but we can think of possibilities to avoid it. The advantage of the current system is that we know that user:// always exists (the game creates it when starting if missing).
Without this, it means that any time you want to write something to user:// (be it logs, config or custom data), we need to first ensure that the directory exists and create it if not. So that's one added check on each write operation instead of a single check when starting the application.

Maybe have it as per-project setting whether to create the userdata directory? There's just lots of projects, like demos or tutorials or simple games, that don't use it.

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


add dep for logexpfunctions

Codecov Report

Merging #6 (3e97859) into main (f0546af) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #6   +/-   ##
=======================================
  Coverage   97.60%   97.60%
=======================================
  Files           2        2
  Lines         167      167
=======================================
  Hits          163      163
  Misses          4        4

Continue to review full report at Codecov.

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


Disabling EEPROM?

Not really an issue, but hope you can help.
I'm trying to get this working on an RFDuino, but this does not have any EEPROM. I looked at the code and it looks like the Arduino Due has an exception to use flash. Any ideas how to get this to work with an RFDuino or is there a way the EEPROM calibration data can be hard coded?
Thanks!

Simplest way would be to hack this file:
https://github.com/richards-tech/RTIMULib-Arduino/blob/master/libraries/CalLib/CalLib.cpp
The calLibRead() function could be modified to populate the calData structure with hard coded values.
As for using flash as EEPROM, that is not really useful on the Due anyway as it gets erased on every upload. The RFDuino does has the ability to use flash:
https://github.com/RFduino/RFduino/blob/master/libraries/RFduinoNonBLE/examples/Flash/FlashStructure/FlashStructure.ino
So another way would be to modify CalLib.cpp to use these functions instead of the EEPROM calls. Don't know if the data will survive an upload though.

Got it working by changing CalLib.cpp:
#ifdef RFDUINO
// RFduino version

void calLibErase(byte device)
{

}

void calLibWrite(byte device, CALLIB_DATA *calData)
{

}

boolean calLibRead(byte device, CALLIB_DATA *calData)
{
    calData->validL = CALLIB_DATA_VALID_LOW;
    calData->validH = CALLIB_DATA_VALID_HIGH;
    calData->magValid = true;
//    calData.pad = calData.pad;
    calData->magMin[0] = -35.05; //x
    calData->magMin[1] = -52.18; //y
    calData->magMin[2] = -28.62; //z
    calData->magMax[0] = 64.28; //x
    calData->magMax[1] = 60.31; //y
    calData->magMax[2] = 66.83; //z
    return true;
}

#elif __SAM3X8E__
// Due version

For anyone looking for a same solution, simply replace
#if __SAM3X8E__
// Due version

in the original file for the above code.


More descriptive error for INVALID_NUMBER_MESSAGE

A more descriptive error for INVALID_NUMBER_MESSAGE so that people who are new to manim aren't confused.

Thats a shame, It was funny :(
Just kidding, thank you !

@huguesdevimeux thanks for your code review! Are you on the manim-community channel on Discord? If you are, but you weren't able to merge this PR (it appears you don't have write access to the repo, please let @eulertour know).


NODE_GYP_FORCE_PYTHON doesn't affect gyp-mac-tool

Node 12.14.1
macOS 10.15.5
node-gyp 5.1.1
Setting the environment variable NODE_GYP_FORCE_PYTHON seems to work for most things, but it doesn't change the Python version used by "gyp-mac-tool".  I think the problem is that even though "gyp-mac-tool" is a Python program, it is just run as a generic executable with a shebang line:
#!/usr/bin/env python

So it ends up using whatever "python" is in the PATH, instead of what PythonFinder returns.
I'm not sure how to fix this...

Maybe run "mac-gyp-tool" as "$PYTHON mac-gyp-tool" instead of as a generic executable?  Does this require changes to gyp?
Maybe node-gyp can do some temporary symlink/PATH stuff to ensure the right Python gets run?

Yes, it requires changes to gyp (well, gyp-next).
Another option: start mac-gyp-tool as a shell script, then re-execute it as a python script with an appropriate interpreter; see nodejs/node@c6da5c8cdfae9e80d63f090c4289fd146c3175df for an example. That's different from inheriting NODE_GYP_FORCE_PYTHON at configure time, however.
I'm curious though: what issues does using a different python binary cause? Doesn't the script work with both python 2 or 3?

I'm curious though: what issues does using a different python binary cause? Doesn't the script work with both python 2 and 3?

Yes, the latest script works with both Python 2 and 3.  But I recently had to deal with a situation where a 3rd party module used node-gyp but didn't list it as a dependency, so the "install" step used up whatever is globally installed.  When I install the latest Node 12.x with nodenv, I get an older version of node-gyp that doesn't work on Python 3.
This is easy to fix on my machine, but I needed things to work on all ~50 of my coworkers machines as well.  If NODE_GYP_FORCE_PYTHON worked, that would have been an easy temporary workaround.
Hopefully the Python 2 vs 3 thing won't be an issue anymore, but if NODE_GYP_FORCE_PYTHON affects most of gyp's Python code, it's confusing when it doesn't affect "gyp-mac-tool" (or any other scripts), especially when you're trying to debug what's going on.


[ZEPPELIN-3007] display a note name without any path in the tab title

What is this PR for?
This PR modifies the display of the name of the note.
Now, instead of the full path, only the name of the note is displayed.
What type of PR is it?
Improvement
What is the Jira issue?
ZEPPELIN-3007
Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

Tested this branch.

Things little bit inconsistent to me is when renaming note name although i don't have any good idea on it.
@tinkoff-dwh @felixcheung what do you guys think?

I think it's better to show a pop-up when clicking on the name of the note. As on the main page.

What do you think about it?

@Leemoonsoo
What about this option?

Now note name clipped on the left.

@tinkoff-dwh Looks good to me!
Merge to master if no further comment.


Clean Up CLI Tests

Remove unnecessary baseCommand override in WskAction and always ensure that the CLI exists instead of just doing so in a few test suites.

PG3 928 🔵

@csantanapr, I believe this can be merged.

PG2 1892 in progress.


Replace kfdefs with DataScienceCluster resource and changer default ns to opendatahub for make deploy

With ODH v2 now on the fast channel, it's time we remove mentions of kfdefs, and replace them with DataScienceCluster resource. The resources should be included in their own sub folder like we had/have with kfdef here.
Provide typical DSC example with just dashboard + datasciencepipelines, a README on how to use it, and also a DEV example utilizing the manifests field, here. The example should include one pointing to the main branch with inline comments on suggestions for tags/etc. Now that kfdef parameters are gone, how can we deploy a custom DSPO image with v2 operator, like when we are testing a PR, is there a way? if so provide suggestions within the readme.
Also it seems odh v2 operator now deploys apps to opendatahub namespace, confirm this your self first. Once confirmed, update the make deploy command to deploy to this namespace by default instead of odh-applications.

here's an example DSC I used:
kind: DataScienceCluster
apiVersion: datasciencecluster.opendatahub.io/v1
metadata:
  name: default
spec:
  components:
    dashboard:
      managementState: Managed
    datasciencepipelines:
      devFlags:
        manifests:
          - uri: https://github.com/opendatahub-io/data-science-pipelines-operator/tarball/main
            contextDir: config
            sourcePath: base
      managementState: Managed

@HumairAK it looks like this issue depends on https://github.com/opendatahub-io/opendatahub-operator/issues/673.

@hbelmiro in this case we  can leave the dev example out but do the rest, let's do it this way:

Provide the DSC for end user
Provide a work around for dev to deploy a DSC deploying from a fork/branch, with list of commands to scale down odh operator and patch in the deployment image with a dev image
Create a follow up issue on updating this workaround with proper image fields once implemented in odh operator (mark this blocked, link to the blocking issue, and also mark it good-first-issue)


Various warnings due to Pydantic protected namespaces, such as UserWarning: Field "model_name" in JinaEmbeddings has conflict with protected namespace "model_".

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
from langchain_community.embeddings import JinaEmbeddings, LlamaCppEmbeddings, TensorflowHubEmbeddings, GooglePalmEmbeddings

Error Message and Stack Trace (if applicable)
/app_venv/lib/python3.10/site-packages/pydantic/_internal/_fields.py:132: UserWarning: Field "model_name" in JinaEmbeddings has conflict with protected namespace "model_".

You may be able to resolve this warning by setting `model_config['protected_namespaces'] = ()`.
  warnings.warn(
/app_venv/lib/python3.10/site-packages/pydantic/_internal/_fields.py:132: UserWarning: Field "model_path" in LlamaCppEmbeddings has conflict with protected namespace "model_".

You may be able to resolve this warning by setting `model_config['protected_namespaces'] = ()`.
  warnings.warn(
/app_venv/lib/python3.10/site-packages/pydantic/_internal/_fields.py:132: UserWarning: Field "model_url" in TensorflowHubEmbeddings has conflict with protected namespace "model_".

You may be able to resolve this warning by setting `model_config['protected_namespaces'] = ()`.
  warnings.warn(
/app_venv/lib/python3.10/site-packages/pydantic/_internal/_fields.py:132: UserWarning: Field "model_name" in GooglePalmEmbeddings has conflict with protected namespace "model_".

You may be able to resolve this warning by setting `model_config['protected_namespaces'] = ()`.
  warnings.warn(

Description
I am trying to use the langchain library to develop an AI application, however, when importing certain libraries I am getting warnings regarding protected namespaces.
System Info
System Information

OS:  Darwin
OS Version:  Darwin Kernel Version 23.6.0: Wed Jul 31 20:48:52 PDT 2024; root:xnu-10<IP_ADDRESS>0.5~1/RELEASE_ARM64_T6020
Python Version:  3.10.14 (main, May  3 2024, 16:41:18) [Clang 15.0.0 (clang-15<IP_ADDRESS>)]

Package Information

langchain_core: 0.3.12
langchain: 0.3.4
langchain_community: 0.3.3
langsmith: 0.1.137
langchain_google_vertexai: 2.0.5
langchain_text_splitters: 0.3.0

Optional packages not installed

langgraph
langserve

Other Dependencies

aiohttp: 3.10.10
anthropic[vertexai]: Installed. No version info available.
async-timeout: 4.0.3
dataclasses-json: 0.6.7
google-cloud-aiplatform: 1.70.0
google-cloud-storage: 2.18.2
httpx: 0.27.2
httpx-sse: 0.4.0
jsonpatch: 1.33
langchain-mistralai: Installed. No version info available.
numpy: 1.26.4
orjson: 3.10.10
packaging: 24.1
pydantic: 2.9.2
pydantic-settings: 2.6.0
PyYAML: 6.0.1
requests: 2.32.3
requests-toolbelt: 1.0.0
SQLAlchemy: 2.0.21
tenacity: 8.5.0
typing-extensions: 4.12.2

I'm working on a PR for this now 👍

The PR has been merged. Just waiting for a new release version.


[incubator/jaeger] added credentials for jaeger cassandra schema job

What this PR does / why we need it:
The job that installs schema in Jaeger Cassandra did not have credential support earlier, this adds it
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @naseemkullah

/lgtm


feat: enable no_std support for pbjson

Closes: #156
Background
This PR addresses the no_std compatibility issue with serde feature in ibc-rs. This is caused by the recent implementation of ProtoJSON serialization and deserialization for the ics23 Protobuf definitions using pbjson, and then re-exporting the ics23 type in ibc-proto-rs. Some of our users by then (starting from IBC-rs v0.41.0) are experiencing compilation errors.
To meet this immediate need and the lack of activity in the pbjson crate for months, we have taken the initiative to feature no_std support in the informalsystems-pbjson crate and have it published.
Before finalizing this change, I’d like to cross-check with @hdevalence to ensure this addition is in line with their systems too and maintain the expected behavior.
Subsequent PR in ibc-proto-rs: https://github.com/cosmos/ibc-proto-rs/pull/118
Subsequent PR in ibc-rs: https://github.com/cosmos/ibc-rs/pull/790

If nobody objects, let's merge this on Tuesday next week (August 15th).


Billing Email Template Logo not Formatted for Small Screens

Describe the bug
On the weekly billing report email the exchange lab logo formatting breaks down when the screen shrinks to a certain size. We want to maintain the formatting in the smaller pane as well.
Additional context
Add steps to reproduce the behaviour or any other context
Definition of done

[ ] Exchange Lab logo looks the same for smaller screen sizes

No longer relevant for Khurrum. @MaxWardle is this still required?


Propose a shorter  stack quick start guide

I though the Guide was a bit overwhelming for a beginner. I propose to link to a quick start document pointing to the full stack guide.
nb. the link point to my forked repository if this is accepted, one must not forget to point to the commercialhaskell/stack repository.

Let's wait for your guide to get merged into Stack's master branch, and then include this.


chore(package): ensure node_modules satisfies package.json

Codecov Report

Merging #1540 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1540   +/-   ##
=======================================
  Coverage   99.74%   99.74%
=======================================
  Files         141      141
  Lines        2384     2384
=======================================
  Hits         2378     2378
  Misses          6        6

Continue to review full report at Codecov.

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


Update rancher image version for e2e tests

Summary
Fixes #12082

Technical notes summary

E2E test should point to the latest backend, this PR takes care of it.
Areas or cases that should be tested

n/a
Areas which could experience regressions

n/a
Screenshot/Video

n/a
Checklist

[x] The PR is linked to an issue and the linked issue has a Milestone, or no issue is needed
[x] The PR has a Milestone
[x] The PR template has been filled out
[x] The PR has been self reviewed
[x] The PR has a reviewer assigned
[x] The PR has automated tests or clear instructions for manual tests and the linked issue has appropriate QA labels, or tests are not needed
[x] The PR has reviewed with UX and tested in light and dark mode, or there are no UX changes

It would be good to update the local command too https://github.com/rancher/dashboard/blob/master/package.json#L29 otherwise the PR looks good

good catch, thank you

the namespace picker flackyness could be down to https://github.com/rancher/dashboard/blob/43a3bfdab928c0d8b3d2ae9b2c3ad8fba3ac09f1/cypress/e2e/tests/pages/explorer2/namespace-picker.spec.ts#L203 being inline rather than gating the rest on then. just a guess though

the namespace picker flackyness could be down to
https://github.com/rancher/dashboard/blob/43a3bfdab928c0d8b3d2ae9b2c3ad8fba3ac09f1/cypress/e2e/tests/pages/explorer2/namespace-picker.spec.ts#L203
being inline rather than gating the rest on then. just a guess though

Tried it before locally but that didn't fix it, I'll re-try.


Problem P3 ( std::runtime_error)

P3-1] Table 2 took 35.7457 sec, wrote<PHONE_NUMBER> right entries
[P3-2] Table 2 took 18.212 sec, wrote<PHONE_NUMBER> left entries,<PHONE_NUMBER> final
[P3-1] Table 3 took 25.6894 sec, wrote<PHONE_NUMBER> right entries
[P3-2] Table 3 took 17.4704 sec, wrote<PHONE_NUMBER> left entries,<PHONE_NUMBER> final
[P3-1] Table 4 took 35.5207 sec, wrote<PHONE_NUMBER> right entries
[P3-2] Table 4 took 18.3492 sec, wrote<PHONE_NUMBER> left entries,<PHONE_NUMBER> final
[P3-1] Table 5 took 35.7135 sec, wrote<PHONE_NUMBER> right entries
[P3-2] Table 5 took 17.7532 sec, wrote<PHONE_NUMBER> left entries,<PHONE_NUMBER> final
[P3-1] Table 6 took 37.7248 sec, wrote<PHONE_NUMBER> right entries
[P3-2] Table 6 took 19.5287 sec, wrote<PHONE_NUMBER> left entries,<PHONE_NUMBER> final
terminate called after throwing an instance of 'std::runtime_error'
what():  thread failed with: thread failed with: input not sorted
Abortado (`core' generado)

I have the same machine, with 128gb ram, 5950x, both with 110g - 2 and 2tb SSD (the second 0 problems)

+1, I've got the same issue. 128gb ram, with dual xeon E5-2643,
might there be a broken ram bar?

+1 Same issue.  144GB RAM/120 set for Temp, Virtual Machine, Xeon Gold 6130.  Errors out during [P1] T3

Same here, it happens randomly on a box running only one chia_plot -r6 -u 256, 16GB ram, one older Intel Core i7-2600K, Ubuntu 20.04 server freshly installed and doing nothing else.
Here's the backtrace from the core dump:
#0  __GI_raise (sig=sig@entry=6) at ../sysdeps/unix/sysv/linux/raise.c:50 #1  0x00007fc6a3f59859 in __GI_abort () at abort.c:79 #2  0x00007fc6a432e951 in ?? () from /lib/x86_64-linux-gnu/libstdc++.so.6 #3  0x00007fc6a433a47c in ?? () from /lib/x86_64-linux-gnu/libstdc++.so.6 #4  0x00007fc6a433a4e7 in std::terminate() () from /lib/x86_64-linux-gnu/libstdc++.so.6 #5  0x00007fc6a433a799 in __cxa_throw () from /lib/x86_64-linux-gnu/libstdc++.so.6 #6  0x00005555827c9600 in Thread<std::pair<std::vector<phase1::entry_xm<4>, std::allocator<phase1::entry_xm<4> > >, unsigned long> >::wait (this=0x7ffd5ae56eb0) at /usr/include/c++/9/ext/new_allocator.h:89 #7  0x00005555827c9689 in Thread<std::pair<std::vector<phase1::entry_xm<4>, std::allocator<phase1::entry_xm<4> > >, unsigned long> >::close (this=0x7ffd5ae56eb0)     at /home/vince/chia/chia-plotter/include/chia/Thread.h:86 #8  0x0000555582820ffd in phase1::compute_matches<phase1::entry_xm<4>, phase1::entry_xm<4>, phase1::tmp_entry_x, DiskSort<phase1::entry_xm<4>, phase1::get_y<phase1::entry_xm<4> > >, DiskSort<phase1::entry_xm<4>, phase1::get_y<phase1::entry_xm<4> > > > (R_index=<optimized out>, num_threads=6, L_sort=0x7ffd5ae57130, R_sort=0x7ffd5ae57180, L_tmp_out=0x7ffd5ae57590, R_tmp_out=<optimized out>)     at /usr/include/c++/9/bits/stl_algobase.h:222 #9  0x0000555582824b17 in phase1::compute_table<phase1::entry_xm<4>, phase1::entry_xm<4>, phase1::tmp_entry_x, DiskSort<phase1::entry_xm<4>, phase1::get_y<phase1::entry_xm<4> > >, DiskSort<phase1::entry_xm<4>, phase1::get_y<phase1::entry_xm<4> > > > (R_tmp=0x0, L_tmp=0x7ffd5ae574a0, R_sort=0x7ffd5ae57180, L_sort=0x7ffd5ae57130, num_threads=6, R_index=4) at /usr/include/c++/9/chrono:331 #10 phase1::compute (input=..., out=..., num_threads=6, log_num_buckets=8, plot_name=..., tmp_dir=..., tmp_dir_2="/mnt/chian/chia/") at /home/vince/chia/chia-plotter/include/chia/phase1.hpp:480 #11 0x000055558282b389 in create_plot (num_threads=6, log_num_buckets=8, pool_key_bytes=..., farmer_key_bytes=std::vector of length 48, capacity 64 = {...}, tmp_dir="/mnt/chian/chia/", tmp_dir_2="/mnt/chian/chia/")     at /usr/include/c++/9/bits/basic_string.h:936 #12 0x000055558276b0a2 in main (argc=<optimized out>, argv=<optimized out>) at /home/vince/chia/chia-plotter/src/chia_plot.cpp:350

It must be something else, At first i tryed it with a ramdisk 110g, then this error occured. Then I tryed to Plot only with nvme, and the error Occured again. Then I Tryed with another server (other mainboard, same specs). Then I changed the os several Times (I tryed Cent, Ubuntu Server 21, Debian, Win Server 2008. I think its some failed garbage Collection Problem, but I don't really know...

I didn't look into the code yet and can't be sure the error is caused by just one issue, but I have a few interesting notes about the way it reproduces on my machines:

Seems to be much more frequent on SSD than on HDD.
Didn't try with RAM drive
Seems to be harder to reproduce with more cpu load(from other processes)
Rebooting machine changes things, after some reboots it can do hundreds of plots without an issue(had it few times, rebooted for other reasons), after some reboots, stuff keeps crashing in early stages, usually P1 tables 2-3.
Doesn't ever crash in P1 table 1.

Env: Arch, builds of 3 different commits from master. 2 different machines, both i7-7700K, 6 different nvmes, 1 ssd, ~6 different HDDs, 3 different sets of RAM(2 of them worked flawlessly for years including much more intensive tasks than this).
All in all, I believe this has nothing to do with RAM stability or any kind of system stability. Seems to be an issue with thread synchronization(though I'm really curious what sort of system state created at boot influences this, perhaps something to do with interrupts handling by one or the other core).

+1
terminate called after throwing an instance of 'std::runtime_error'   what():  thread failed with: thread failed with: thread failed with: input not sorted
5950x (used 24 of 32 cores), with 110G RAMDisk of 128 GB RAM, running on WSL Ubuntu 20.04.
Never had problems at hundreds of other plots...
Reducing to 16 threads made me 16 more plots without issues since, without a reboot...


[improvement] [maven] skip license-maven-plugin task

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
When I execute mvn package or mvn install, it will take a lot of times(cost 45 min in my computer).
And I find it might be caused by license-maven-plugin task, the <skip>true</skip> is not work.
And after I remove the license-maven-plugin it will just cost 8 min to execute mvn install.
I find there are may need to add other parameters to skip license-maven-plugin task.
https://www.mojohaus.org/license-maven-plugin/add-third-party-mojo.html
SeaTunnel Version
dev
SeaTunnel Config
no

Running Command
no

Error Exception
no

Flink or Spark Version
No response
Java or Scala Version
No response
Screenshots
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

After #1376 merged, now execute mvn clean install only cost 9 min, great.


EDL Adapter 'append' file

Hello !
Maybe I miss something, but it will be very useful to append edl file has new track. Ex:
eld-01 : [---clip01---][---------clip02---------][-clip03-]
eld-02 : ...................[-clip02.1-][-clip02.2-].............
This is a common use case to manage splitscreen effect.
pseudo code:
timeline = otio.adapters.read_from_file(edl-1)
otio.adapters.append_from_file(edl-2, timeline=timeline)

Thanks a lot ! .

You can read multiple EDLs into a multi-track timeline like this:
# make an empty timeline
timeline = otio.schema.Timeline()
for path in paths:
  # read an edl
  edl = otio.adapters.read_from_file(path)
  # append all of the edl tracks onto our timeline
  timeline.tracks.extend(edl.tracks)
# write the result to disk
otio.adapters.write_to_file(timeline, "multitrack.otio")

The EDL adapter only supports a single video track, but may have multiple audio tracks. If you want just the video track, then timeline.tracks.extend(edl.video_tracks())

It's perfect ! Thank you so much.