[Bug] The operator complains about not finding the admin credentials in the secret

Describe the bug
While deploying a Grafana instance, the operator complains in the logs:
2024-09-27T14:02:48Z	ERROR	GrafanaReconciler	failed to get version from instance	{"controller": "grafana", "controllerGroup": "grafana.integreatly.org", "controllerKind": "Grafana", "Grafana": {"name":"grafana","namespace":"grafana-main"}, "namespace": "grafana-main", "name": "grafana", "reconcileID": "5c7d64be-e954-4912-acb5-77ad160aafcf", "error": "fetching authentication information for version detection: fetching admin credentials: credentials not found in secret: grafana-main/grafana-admin-credentials"}

Version
v5.13.0
To Reproduce
Steps to reproduce the behavior:

Create a secret named grafana-admin-credentias
Our secret is managed by External-secrets Operator
apiVersion: v1
data:
  password: *****************
  user: *****
immutable: false
kind: Secret
metadata:
  annotations:
    reconcile.external-secrets.io/data-hash: ********
  labels:
    app.kubernetes.io/managed-by: grafana-operator
    reconcile.external-secrets.io/created-by: ******
  name: grafana-admin-credentials
  namespace: grafana-main
  ownerReferences:
  - apiVersion: external-secrets.io/v1beta1
    blockOwnerDeletion: true
    controller: true
    kind: ExternalSecret
    name: grafana-admin-credentials
type: Opaque

Create a Grafana object
apiVersion: grafana.integreatly.org/v1beta1
kind: Grafana
metadata:
  name: grafana

spec:
  deployment:
    spec:
      replicas: 1
      template:
        spec:
          containers:
            - name: grafana
              env:
                - name: GF_SECURITY_ADMIN_USER
                  valueFrom:
                    secretKeyRef:
                      name: grafana-admin-credentials
                      key: user
                - name: GF_SECURITY_ADMIN_PASSWORD
                  valueFrom:
                    secretKeyRef:
                      name: grafana-admin-credentials
                      key: password

Apply the manifests of the resources (ExternalSecret first so that the secret exists when the Grafana Operator tries to create the resources

Watch the logs

Expected behavior
According to the code I have read

https://github.com/grafana/grafana-operator/blob/2a634570b216163f7676487f606cd215f1573311/controllers/client/common.go#L12
https://github.com/grafana/grafana-operator/blob/master/controllers/client/grafana_client.go#L73-L95

The Operator should be able to get the Admin credentials without issues 🤷🏻
Runtime (please complete the following information):

OS: AWS managed nodes
Grafana Operator Version v5.13.0
Environment: AWS EKS, Kubernetes 1.30.4
Deployment type: Helm

Thanks for reporting this! This happens because the operator auto generates its own credentials in a secret with the name <instance>-admin-credentials.
This causes a conflict with the external secrets operator as they both try to take control over this secret.
The fix for this situation is to use another secret name for the synced secret. That way, the operator does not override the existing secret:
apiVersion: v1
stringData:
  password: fooBarBaz
  user: administrator
immutable: false
kind: Secret
metadata:
  name: grafana-custom-admin-credentials
type: Opaque

---
apiVersion: grafana.integreatly.org/v1beta1
kind: Grafana
metadata:
  name: grafana
spec:
  deployment:
    spec:
      replicas: 1
      template:
        spec:
          containers:
            - name: grafana
              env:
                - name: GF_SECURITY_ADMIN_USER
                  valueFrom:
                    secretKeyRef:
                      name: grafana-custom-admin-credentials
                      key: user
                - name: GF_SECURITY_ADMIN_PASSWORD
                  valueFrom:
                    secretKeyRef:
                      name: grafana-custom-admin-credentials
                      key: password


Honor ID on admonitions

Ids are not retained on admonitions.
[[note_xref]]
[NOTE]
.Title of the note
====
Block of text.
====

@mojavelinux Looking at the code, I guess most blocks don't comply with IDs. Maybe a generic solution would be great on this part. I don't know enough of ruby to start directly a fix. Do you have a path to a solution and do you think it can be handled by someone else?

We'll just need to take it one node at a time. When I created the EPUB3 converter, I must not have needed the IDs for the project I was using it for. But they should definitely be supported.
To learn what I'm changing, just study the PRs. I'll send one shortly for this issue.


The following assertion was thrown during paint(): RRect argument contained a NaN value. 'dart:ui/painting.dart': Failed assertion: line 46 pos 10: ''

Steps to Reproduce

Run flutter create bug.
Update the files as follows: ...
...

Expected results:
Actual results:

Logs

Hi @HiraGul ,
Thanks for filing the issue, Please provide the output of flutter doctor -v along with minimal reproducible code sample, With the exact steps to reproduce the issue, This helps to better investigate the issue.
Thank you.


New integration update feature

What changed?
I worked pretty tirelessly to make this work, and to create a positive user experience that I'm now very proud of. "Ease of use" was top of mind. Here are the things that changed.

.gitignore was update to include more things
path creations for the config file are now os independent
Added 4 new, styled display functions in jiu that leverage the emoji package. NOTE TO JOHN this requires the emoji package so that needs to be added to the main requirements.txt file
Added "updater_core" as a new Addon that inherits the "Addon" class. This leverages ipywidgets and uses the Output widget to communicate to the user.
A user now has the ability to re-install integrations from within the Updater addon with a click of a button.

After discussion with @JohnOmernik this morning, holding off on this PR for now until we work through the python setup.py install to pip install solution, since this PR won't work in an Windows env due to file locks of .egg files.


Can't restore stock firmware Acer CB311-10H

Hello,
I'm having issues restoring the stock firmware onto my Chromebook using the recovery media for the firmware.
The device is an Acer CB311-10H (board name KASUMI) running Fedora 40 and I'm using recovery media from the Acer C721.
The restore process seems to work up until it needs to grab the firmware from the USB drive, in which it errors out in the copying process.
I could be using the wrong recovery media but the C721 was the closest match I could find based off of the board name and specifications.
Any assistance/pointers would be greatly appreciated :)
Firmware script log

as the script log shows, it was unable to extract the firmware from the recovery USB for some reason:

cp: target 'Google_Grunt.11031.123.0.bin': No such file or directory
mv: cannot stat 'coreboot-Google_*': No such file or directory
ERROR: Unable to open /tmp/stock-firmware.rom for reading: No such file or directory

so I need to look at see 1) why that failed, and 2) why the script didn't catch the error and halt

Works perfectly now. Thank you very much for your help!

Hello
I have this error when I want to restore stock firmware using a ChromeOS Recovery USB in fedora 41:
cp: -r not specified; omitting directory '/tmp/tmp.QT9rIcvgVJ/'
Error: failed to extract firmware image for blorb using this recovery USB

@ynoalberto if not for the same device, then please open a new ticket

No. I open a section in issusse. Check it please

In that section I explains the device that I have


Export images in PDF or SVG format

Hi,
I am using nglviewer through https://github.com/nvelden/NGLVieweR/. In my Shiny-App I can make a snapshot to save the current view as an PNG image. I would like to export images in a scalable vector format such as SVG or PDF. I quickly went through the issues, but so far nobody asked for this feature?
I already asked in the NGLVieweR issues, but there seems to be no support for other image formats: https://github.com/nvelden/NGLVieweR/issues/3. Also in the manual NGLVieweR forwards me to you: 'For a full  list of options, see "makeImage" and "ImageParameters" in the official NGL.js manual'. I looked at the options but I do not find a option to choose the image format. Is this correct?
Best,
Christian

Hi @ChristianRohde !
Exporting a 3D scene as a 2D vector image is far from being trivial. One could write some routines to convert the 3D meshes in 2D triangles in a SVG document, but I doubt that the result will be any good. NGL leverages the Three.js library which has an SVG renderer, but the results are esthetically subpar with what can be seen in a 3D view (an example here that can give an idea). I might miss something though...
Unfortunately given the current resources for NGL development, I doubt that adding this feature would be feasible.
But if what you want is to get images with a high resolution, you can export as PNG and increase the resolution. There are options in the cartoon representations to augment the number of subdivisions in the spline to avoid artifacts at higher resolutions.


feat(#710): Add org.eolang and org.eolang.jeo packages

In this PR I added support of FQN for all the generated objects. This involves adding of org.eolang and org.eolang.jeo packages to all the generated objects.
Closes: #710.
History:

feat(#710): add EoFqn and JeoFqn classes
feat(#710): try to apply EoFqn and JeoFqn to all base attributes. All tests fail!
feat(#710): fix most of the integration tests
feat(#710): fix all the integration tests
feat(#710): fix all the code offences
feat(#710): add one more puzzle

PR-Codex overview
The focus of this PR is to update XML element base attributes to use fully qualified names in the EO language representation classes.
Detailed summary

Updated XML element base attributes to use fully qualified names
Replaced hardcoded values with JeoFqn and EoFqn instances
Improved consistency and readability in the codebase

The following files were skipped due to too many changes: src/main/java/org/eolang/jeo/representation/xmir/XmlTryCatchEntry.java, src/main/java/org/eolang/jeo/representation/xmir/XmlAttribute.java, src/test/java/org/eolang/jeo/representation/bytecode/BytecodeMethodParameterTest.java, src/main/java/org/eolang/jeo/representation/xmir/XmlClass.java, src/main/java/org/eolang/jeo/representation/xmir/XmlAnnotation.java, src/test/java/org/eolang/jeo/representation/xmir/XmlAttributeTest.java, src/test/java/org/eolang/jeo/representation/xmir/XmlAttributesTest.java, src/main/java/org/eolang/jeo/representation/xmir/XmlNode.java, src/main/java/org/eolang/jeo/representation/xmir/XmlOperand.java, src/test/java/org/eolang/jeo/representation/xmir/XmlInstructionTest.java, src/main/java/org/eolang/jeo/representation/DataType.java, src/test/java/org/eolang/jeo/representation/directives/DirectivesFieldTest.java, src/main/java/org/eolang/jeo/representation/xmir/XmlMethod.java, src/test/java/org/eolang/jeo/representation/directives/DirectivesClassTest.java, src/test/java/org/eolang/jeo/representation/xmir/XmlTryCatchEntryTest.java, src/main/java/org/eolang/jeo/representation/directives/DirectivesSeq.java, src/test/java/org/eolang/jeo/representation/directives/DirectivesClassPropertiesTest.java, src/test/java/org/eolang/jeo/representation/xmir/XmlFrameTest.java, src/test/java/org/eolang/jeo/representation/directives/DirectivesAnnotationPropertyTest.java, src/test/java/org/eolang/jeo/representation/directives/DirectivesAnnotationsTest.java, src/main/java/org/eolang/jeo/representation/directives/EoFqn.java, src/test/java/org/eolang/jeo/representation/directives/JeoFqnTest.java, src/it/fails-if-bytecode-broken/verify.groovy, src/main/java/org/eolang/jeo/representation/directives/JeoFqn.java, src/test/java/org/eolang/jeo/representation/directives/HasMethod.java, src/it/eo-to-bytecode/target/generated-sources/opeo-xmir/org/eolang/spring/WithoutMetas.xmir, src/it/eo-to-bytecode/target/generated-sources/opeo-xmir/org/eolang/benchmark/BA.xmir, src/test/resources/org/eolang/jeo/Application.xmir

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Blocked by https://github.com/objectionary/eo/issues/3373#issuecomment-2358425399

@maxonfjvipon This PR blocks other issues in this repository. Are the packages  org.eolang and org.eolang.jeo suitable for your needs (at least for now), or maybe you need some significant changes?

@volodya-lombrozo at least for now it's ok

@rultor merge

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 35min)

@volodya-lombrozo Thanks for the contribution! You've earned +4 points for this: +4 as a basis; -8 for too many hits-of-code (1529 >= 200); -16 for way too many hits-of-code (1529 >= 800); -16 for the lack of code review; -4 for very few (5) comments; +44 to give you at least something. Please, keep them coming. Your running balance is +89.

@volodya-lombrozo Great work on your contribution! 🌟 You've earned +4 points, with adjustments for code volume and review. While there's room for improvement in code size and comments, we value your effort. Your running balance is now +89. Keep the contributions coming and remember, quality matters!

@volodya-lombrozo Great work on your contribution! 🌟 You've earned +4 points, with adjustments for code volume and review. While there's room for improvement in code size and comments, we appreciate your effort. Your balance is now +89. Keep the contributions coming and check out this article for tips on balancing speed and quality!

@volodya-lombrozo Thanks for your contribution! You've earned +4 points: +4 base, -8 for exceeding 200 hits-of-code (1529), -16 for surpassing 800 hits-of-code, -16 for no code review, -4 for only 5 comments, +44 adjustment. While the code volume is impressive, remember our policy values quality and review. Consider breaking larger contributions into smaller, reviewable chunks. Your balance is now +89. Keep improving and contributing!

@volodya-lombrozo Great job on your contribution! 🌟 You've earned +4 points, with adjustments for code size and review. While there's room for improvement in code conciseness and comments, we value your effort. Your balance is now +89. Keep up the good work and remember to focus on quality in your next submission!

@volodya-lombrozo Thank you for your contribution! We appreciate your effort, but there are some areas for improvement. Your submission of 1529 hits-of-code exceeds our guidelines, resulting in deductions. Additionally, the lack of code review and limited comments (5) have impacted your points. Despite these challenges, you've earned +4 points: +4 base, -24 for excess code, -16 for no review, -4 for few comments, +44 adjustment. Please focus on quality over quantity in future contributions. Your current balance is +89. Keep up the effort, and let's work on improving the review process!

@volodya-lombrozo Hey there! 👋 Thanks for your contribution! Your effort is appreciated, but it seems we've got some room for improvement. You've earned +4 points, but there were a few hiccups: -8 for exceeding 200 hits-of-code (1529!), -16 for going over 800, -16 for missing code review, and -4 for only 5 comments. We had to add +44 to keep you at the minimum 4 points. Remember, quality over quantity is key! Aim for fewer hits-of-code and make sure to get those code reviews. Keep the contributions coming, and let's work on boosting that +89 balance! 💪

@volodya-lombrozo Great work on your contribution! 🌟 You've earned +4 points, with adjustments for code volume and review. While there's room for improvement in code size and comments, we value your effort. Your balance is now +89. Keep the contributions coming and check out our quality guidelines for future submissions!

@volodya-lombrozo Great contribution! Here's your reward breakdown: +4 base points, -8 for 1529 hits-of-code (≥200), -16 for exceeding 800 hits-of-code, -16 for no code review, -4 for only 5 comments. We've added +44 to ensure a minimum reward. Your total: +4 points. Remember, smaller, quality contributions with thorough reviews are ideal. Keep them coming! Your balance is now +89.

@volodya-lombrozo Thanks for the contribution! You've earned +4 points for this: +4 as a basis; -8 for too many hits-of-code (1529 >= 200); -16 for way too many hits-of-code (1529 >= 800); -16 for the lack of code review; -4 for very few (5) comments; +44 to give you at least something. Please, keep them coming. Your running balance is +89.

@volodya-lombrozo Thanks for the contribution! You've earned +4 points for this: +4 as a basis; -8 for too many hits-of-code (1529 >= 200); -16 for way too many hits-of-code (1529 >= 800); -16 for the lack of code review; -4 for very few (5) comments; +44 to give you at least something. Please, keep them coming. Your running balance is +89.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 604 ms

Resolved: ClamAV performance has improved in 340974a after 17 minutes.


Start markdown

[ ] Turn on GitHub Pages
[ ] Outline my portfolio
[ ] Introduce myself to the world

M


You can't merge so why are you even reviewing code?


[🐞] Qwik - Build is Aggressively Treeshaking with Firebase

Which component is affected?
Qwik Rollup / Vite plugin
Describe the bug
I've been working with the Firebase team to get Firebase working on Edge functions.
https://github.com/firebase/firebase-js-sdk/issues/8355
Firebase now works on ever single Framework, even in Edge Environments (Bun, Deno, Cloudflare)... except on Qwik.
Reproduction
https://qwik-firebase-test.vercel.app/
Steps to reproduce
Here is issue on Firebase with deployment instructions:
https://github.com/firebase/firebase-js-sdk/issues/8506
Repo: GitHub
Demo with error: Vercel Edge
System Info
Vercel Edge (no way to currently test this on Vercel Functions as there is not a Plugin)

Additional Information
Here is what the Firebase team said after agressive testing:

I was able to reproduce the problem. It appears that something in the build pipeline is aggressively tree shaking portions of the Firebase JS SDK when it packages it for the edge runtime. The "service" mentioned in the log file is that fact that Auth cannot properly register itself with the instance of FirebaseApp (or FirebaseServerApp). This is due to the hooks it would use to register itself with the instance of App are inexplicitly missing from the SDK implementation.

Here is the link to the GitHub Issue:
https://github.com/firebase/firebase-js-sdk/issues/8506
Thanks,
J
Small Chance could be related to: https://github.com/QwikDev/qwik/issues/6322
Note: This could also be a Vercel Edge Plugin problem, but I'm not sure how the build process works.

So qwik has to assume that every import does not have side effects on the client.
The reason is that no code runs on the client except in response to events. All qrl segments are only importing code they are actually using.
Can it be that firebase relies on side effects on the client?
Do you know which code for missing?

Do you know which code for missing?

I don't know what this means.

Can it be that firebase relies on side effects on the client?

Can you be more specific about what you mean with side effects.
I can relay this to the Firebase issue.
Thanks,
J

I can confirm that the Firebase SDK does indeed use side effects on import.

@jdgamble555 As @DellaBitta confirms, importing the firebase SDK makes changes to the global object (side effects).
To solve the problem, the firebase import needs to happen in a different file. This file should export helpers that expose the desired firebase functionality.
Then when you use these helpers in your Qwik code, they won't be tree shaken and the firebase side effects can run on first import.

Hi @wmertens,
Could you clarify? The app should have a single point where it imports the Firebase SDK and then export helpers that should be imported throughout the rest of the application?
That is, this is an App level change and not something in the SDK source base? Or am I misreading your post.
Thanks!

@wmertens - Do you mean you can only import in one file here? Please clarify.
Also, is there any reason this would work in dev mode but not in production on Vercel?  Could this be a Qwik Vercel adapter problem?
J

@jdgamble555 that is odd. The Qwik transform is the same in dev and prod but Vite does tree shaking in prod.
It would be best to edit your firebase in node_modules to console.log() when it loads and see if that log is still present in the prod bundle.

So this is a Vite problem with Qwik? Why would this not effect other frameworks that use Vite?
If I edit something in node_modules it will just get overwritten in prod when the modules are installed.
J

Could you clarify? The app should have a single point where it imports the Firebase SDK and then export helpers that should be imported throughout the rest of the application?

@wmertens - Clarity for the Firebase Team would definitely help if you could. Thank you!

@jdgamble555 the node modules edit is just to figure out which code runs when.
And yes I think importing firebase in one file that exports helpers would help.

@wmertens - I'm using dynamic imports on everything Firebase, and it does work in npm run preview.

Would dynamic imports matter?
Does preview still use tree shaking? It seems to work fine in preview.

J

@wmertens - We have the Firebase team trying to help with this, but everyone seems to think this is a Qwik issue.

Is there anything that production does to the treeshaking differently than dev or prod?

This could be a simple fix if we can at least figure out the problem.
Thanks,
J

The best way to debug this is to work backwards. In the browser, find out
what isn't working, and then determine what code should have run to make it
work, and find where it should have been loaded, until you find a
difference.
The optimizer doesn't handle prod and dev differently tree-shaking wise,
but vite bundles things together in prod that might have side effects that
make it work.
Wout.
On Sat, 21 Dec 2024, 23:26 Jonathan Gamble, @.***>
wrote:

@wmertens https://github.com/wmertens - We have the Firebase team
trying to help with this, but everyone seems to think this is a Qwik issue.

Is there anything that production does to the treeshaking
differently than dev or prod?

This could be a simple fix if we can at least figure out the problem.
Thanks,
J
—
Reply to this email directly, view it on GitHub
https://github.com/QwikDev/qwik/issues/7052#issuecomment-2558231370, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAANNFXJUJT4MQPMQPRLJKL2GXFI3AVCNFSM6AAAAABRNZXC4WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJYGIZTCMZXGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

This is a server problem, not browser problem, so harder to debug.
What would vite do differently with Qwik that it wouldn't do in other frameworks?
This is only problematic with Qwik and on the server in production.
J


Allow administrators to create tokens for other users

It's more a feature request than a bug report

Gitea version (or commit ref): 1.15.0+dev-344-g5285a3e70 built with GNU Make 4.3, go1.16.4 : bindata, timetzdata, sqlite, sqlite_unlock_notify
Git version: 2.25.1
Operating system: Ubuntu 20.04

Gitea on docker

Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[X] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[X] No

Log gist: useless

Description
I'm using a SSO to handle connections on my apps. I don't have the hand on this SSO so I can't integrate my Gitea instance in this SSO. What I would like is to allow the users of my app to post issues on the gitea with their userID.
I started to develop this mechanism:

check if the username exists: GET /users/<username>
if it doesn't exist, create it: POST /admin/users
get the tokens of the user: GET /users/<username>/tokens
if in the list of tokens I don't have a "myapp" token, I create it: POST /users/<username>/tokens
with the token, I create the issue

The problem is that I can't create a token for another user, it has to be the user connected who creates his own token. I could create the same password for all my users, but it's not secure at all and I'd like my users to be able to connect to Gitea with their own password.
Would it be possible to allow administrators to create tokens for other users?
Screenshots
Useless

We have su feature, so this is unnecessary.

Closing, as you can use the sudo header per https://docs.gitea.io/en-us/api-usage/#sudo to impersonate other users.

OK but an explanation in the documention on how sudo works with Gitea API wouldn't be useless...


[do not merge] restores PR #381: Proposed additions to P4_16 spec: restrictions on use of stateful objects

This PR re-applies the changes that were submitted as part of #381 and which need to wait for the language working group discussion.

Merging per our discussion at the 10/23 LDWG meeting.


[Bug] Input Widget Regex validation not working as expected

Description
I wanted to create an input widget that allows numeric values greater that 0, and upto 2 places of decimal. I chose Data Type as Number, and set Regex to ^\s*(?=.*[1-9])\d*(?:\.\d{1,2})?\s*$ . On my local machine's terminal, I verified that the regex returns no match for input value 1.222 . But when I set the value of the input widget on Appsmith to 1.222, no error is shown.
Steps to reproduce the behaviour:
Add steps to reproduce this behaviour, include console / network logs & screenshots

Add a new Page
Drag drop input widget on it
Choose Data Type as Number
Set Regex to ^\s*(?=.*[1-9])\d*(?:\.\d{1,2})?\s*$
Deploy application
In the live app, set the value of input as 1.222
Regex validation doesn't fail, and input 1.222 is accepted

Expected behavior
Regex validation should fail
Important Details

Version: Cloud
OS: Windows 10
Browser: Chrome

The issue still persist.hence reopening the issue

@jsartisan this issue seems to have reopened.

@jsartisan this issue seems to have reopened.

@Nikhil-Nandagopal I just checked. I can see the error tooltip whenever the above regex is failing. Tried with the form widget also. - https://app.appsmith.com/applications/5ff584aa53b64c03e83001c4/pages/5ff584aa53b64c03e83001c6
1.22 is passing and 1.222 is failing.

@Nikhil-Nandagopal I just checked. I can see the error tooltip whenever the above regex is failing. Tried with the form widget also. - https://app.appsmith.com/applications/5ff584aa53b64c03e83001c4/pages/5ff584aa53b64c03e83001c6
1.22 is passing and 1.222 is failing.

This was a build problem when it got reopened. This is not an issue.
@prapullac Please verify this and close the issue if it looks good to you.

This was a build problem when it got reopened. This is not an issue.
@prapullac Please verify this and close the issue if it looks good to you.

The issue is fixed and working as expected

The issue is fixed and working as expected


Make timeout configurable

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io


chore: drop node 19, add node 20 and 21 to workflows

Using lts for browser tests. Removing node 19 and add node 20 and 21 to the workflows.

coverage: 94.331%. remained the same when pulling 0a5bc98a9cb9c8af4920daa978768f18c15057e2 on improve-workflows into 37deed262d4bc0788d32c66636495d10038ad398 on master.


File Manager: File compression to zip has no output

Hi,
While New->create new archive-> select .zip format ends up with message "compression has successfully finished." but it results in no output file.
Where as .tar.gz.is working fine.
Please check and confirm

Also delete file function is not working. After confirming to "Remove Selected" button, message "deleting selected, please wait..." disappears with no action.

Hi,
The most possible reason why this is happening is that you don't have zip command installed or it's not on the path. Try whereis zip on the CLI, what is the output? If non install zip.
It's working fine on my side.
The good point is to add a notification for the user to install zip.

Tried it?

Apologize for delayed response.
I checked, zip was not installed in my machine, after installing zip everything worked perfectly.
Now, the point is, instead of showing "compression has successfully finished." message to those users who don't have zip installed show them some error message like "zip is not installed, compression unsuccessful." or to prompt them to install zip before continuing.  It'll be clear to those users who don't have zip.
Thanks :-)

Yes, I think I'll add it to my extensions.

Alright,
17.84+ will show user-friendly message, in case zip command is not installed - Example

17.84 is out. Please upgrade.

That's perfect.... :-)

Thanks. :D


Fixes the typing issue in the palette

Before submitting the PR:

[ ] Does your PR reference an issue? If not, please chat to the team on Discord or GitHub before submission.

Does not reference an issue directly, but is related to #301's discussion.

[ ] Did you you update and run tests before submission using npm run test?

Didn't touch anything needing tests

[ ] Does your branch follow our naming convention? If not, please amend the branch name using branch -m new-branch-name

Yes

[ ] Did you update documentation related to your new feature or changes?

Yes
What does your PR address?
Fixes the typing issues introduced when we shifted to statically typing the Tailwind values.

@ryceg Thanks. I'll check this tomorrow.

Confirm fixed. Thanks for this!


Return error in locally

Is it possible to return a custom error in locally? Thanks :)

You can return whatever you want in locally.

But this would not trigger the failed callback. Basically my remotely returns a promise which is rejected with the actual error object but as my locally is called afterwards the error is forgotten. Any idea how to reflect this?

Your locally needs to not return a valid value for the failure handling to work. Alternatively, you can return a sentinel value in locally that your component can render appropriately into an error.

I don't want to handle the error in the component itself but instead it should be automatically handled in the container. This is nothing which one can do today if I understand the code correctly. Should this be a feature request?

Just throw in your locally.


[Bug] BracesFixer - Do add a line before close tag

Test for https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/1829

This will surely break most of our fixers?

I ran 1.12 with the default configuration of the this project on the sample code and only the BracesFixer touched. Maybe we should add this case to the SF2 integration test, not sure though..

When I said "this", I was more referring to that style of code, rather than that specific test case.

rebased, RTM if test are green

btw, 1.12 not 1.11? why?

my mistake, I tried and the commit can be picked as is on 1.11 without any problems.

OK, great !

Thank you @SpacePossum.


Add Permission Sensitive Cache Servlet

Currently, this code gets copied and pasted from one project to another and every developer seem to have their own version. Might be good to have one version in Commons that people can reuse.
Idea is for this servlet to have two configurable properties

The servlet path
The list of extensions to be removed from the URI so it resolves to a valid resource.

Thanks @justinedelson for pointing this out.

PR has been merged


Fullscreen as pip

Can you add a feature "like when we press fullscreen in any video(Ex youtube). It should become fullscreen in browser window not to whole pc screen.

Good point! I anyone knows a solution how to "fullscreen in a container", please help!

Removing fullscreen permission while allowing fullscreen in the events will fullscreen within the Flobro window, as @SilentBot1 implemented in his fork


Separate DDC control for audio / video

Before opening the issue, have you...?

[X] Searched for existing issues

Is your feature request related to a problem? Please describe
First, thank you very much. This app is exactly what I was looking for!
I am using USBC connector to my Dell monitor. If I turn on DDC I can control audio as expected but  every time I adjust brightness the monitor it flashes up a prompt on the monitor and I don't see a way to remove that.
Describe the solution you'd like
Allow the audio and brightness to have separate DDC settings so one can be on and not the other.
Describe alternatives you've considered
I am not aware of any except to have to fuss with the monitor switch every time I adjust brightness.
Anything else?
No response

You might want to state which device combination you're using.
I'm using MBA M2 + Dell U4320q and it works smoothly. No flashing.
Here's the monitor settings:

Thanks that seems to have helped.


Update CORS for Oxford Dictionary API

https://forum.oxforddictionaries.com/api/discussion/18/error-message-no-access-control-allow-origin-or-i-can-t-get-my-client-side-request-to-work

Unfortunately, we are unable to support client-side application requests (this includes JavaScript, JScript, VBScript, Java applets, ActionScript, etc.). This is because our API does not currently support CORS requests due to the potential implications for the security of our server. Instead, we suggest you to make the query reach your server side application, and then send the API request from the server rather than from the client.

Thank you for taking the time to work on a Pull Request for this project!
To ensure your PR is dealt with swiftly please check the following:

[x] Your submissions are formatted according to the guidelines in the contributing guide
[x] Your additions are ordered alphabetically
[x] Your submission has a useful description
[x] The description does not end with punctuation
[x] Each table column should be padded with one space on either side
[x] You have searched the repository for any relevant issues or pull requests
[x] Any category you are creating has the minimum requirement of 3 items
[x] All changes have been squashed into a single commit

Thanks!


Fix integration with Rack::File

Changed intoduced to rack in the commit below broke the way that prax
uses Rack::File. This change uses the new API.
https://github.com/rack/rack/commit/f244d4
https://github.com/rack/rack/commit/ed0d8c

Is it backward compatible?

With old versions of Rack? Probably not, but given that this change was made to Rack over a year ago I think it's fair to say that in versions of prax moving forward we're only going to support Rack >= 2.0.0

The Ruby version of Prax is maintenance only (use the Crystal version). I'd like to avoid introducing a change for Rack 2+ that breaks compatibility with older versions of Rack.
We can't use Rack::VERSION but maybe use Rack::RELEASE to support both versions?

@ysbaddaden how does the most recent version look?


fix(upgrade): add sleep to allow event to flush before panic

Which issue(s) this PR fixes:

Issue https://github.com/longhorn/longhorn/issues/9569
What this PR does / why we need it:
Add a sleep after Run() before panicking to let the queued event get flushed.  We thought that eventBroadcaster.Shutdown() would take care of that, and it did in some testing, but there is still a race and it does not work reliably.  QA found that while testing the feature and I did reproduce it - see the note in https://github.com/longhorn/longhorn/issues/9569#issuecomment-2425705470
Special notes for your reviewer:
This is the simplest fix.  We could do something more complicated, like calling Watch() on the events and waiting for it to appear, but there is no need to speed up the exit as soon as possible.  The upgrade will not go forward anyway.  The important thing is to guarantee the event can be found so the problem can be corrected.
Additional documentation or context

Ok, let's go with the simple sleep, then, and keep this in our back pocket in case problems still show up in testing.  Thanks!

@mergify backport v1.7.x v1.6.x


404 errors on portrejections detector and tcpconnectionsusage detector for linux sites

Maybe make these calls only when TCP Connection Analysis is opened or check for 404 and don't display a message box if site is linux etc. figure out the right thing to do.

This is deployed now in prod. Closing this bug


Fix documentation with remaining 'Item' instead of 'Collection'

Related Issue(s):

N/A

Proposed Changes:

Fixed small oversights in the documentation

PR Checklist:

[x ] This PR has no breaking changes.
[ x] I have added my changes to the CHANGELOG or a CHANGELOG entry is not required.

Thank you for catching this!


UI/UX for selecting a data format

Description:
Create a user interface that allows users to select the CSV format when downloading form responses.
User Story:
As a user, I want to choose the format (CSV) in which I can download bulk form responses for better customization.
Acceptance criteria

[ ] Screens and pages work on MS Edge and Google Chrome
[ ] Screens and pages work on Tablet and Desktop
[ ] Screens and pages meet WCAG 2.1 AA requirements
[ ] The number of times a format is used is tracked (Likely GA is the best place)

Next steps: updating wireframes based on scope and discussions, and booking some time for feedback/review


How to run wasm app built by trunk without recomiling?

I know perform trunk serve --open to compile app and run it. But I don't know to run built app directly.

Hello @nickhell. Are you asking how to get your app to run in the browser when people visit a specific site on the internet?

Yes, I wonder know how to deploy app as a server.
And could run app by local runtime just like  wasmer?

Generally speaking, WASM applications compiled using Trunk are intended to run in the browser as web applications. Fundamentally speaking, making this happen with Trunk is easy as the end result of a trunk build gives you a dist dir containing the index.html, the wasm code, and all other assets which are part of your app.
There are SO MANY ways to take that dist content and serve it on the internet. Here are the fundamentals:

you need a DNS domain (buy one from a registrar),
you need to associate that domain with a set of actual IPv4 or IPv6 addresses corresponding to your servers, or more often a load balancer.
head over to a cloud provider like DigitalOcean or the like. Getting some servers & and load balancers provisioned is DEAD SIMPLE and quite fast.
you can delegate the DNS authority of your domain to DigitalOcean (or whatever cloud provider you use), and have your new servers or LoadBalancer IPs associated with your DNS domain as A records. Or you can do this manually, but that has tradeoffs considering possible (inevitable) failure cases.
setup a CI pipeline (using Github Actions or the like) which ships your final dist dir over to your servers.

That is just one way, and is a bit old school. The modern way to do this would be to use a multi-stage docker build where you copy the contents of the dist dir into the container, and the final stage of your container would be based on nginx or whatever server you choose to go with. You could just write your own server in Rust too. Once you have your container, deploy it in a cloud provider's hosted container runtime, in a Kubernetes cluster, or whatever you choose to go with.
Your question is fundamentally an "operations" related question, and there is no one good answer, there are hundreds of good answers :).
I would recommend doing some research on this front, and determine what you think you can managed on your own given whatever time constraints you have.

How nice of you! Thanks a lot.


Available payment methods on Mollie not limited to assigned payment methods of the current Sales channel

If I change the payment method at Mollie or if the payment fails, the user sees all in Shopware enabled payment methods. The assignment to the corresponding sales channel is ignored.

Hi @AndreasA, this is correct. To avoid this, please choose the standard failed payment redirect (can be switched on/off in the plugin configuration).

@milan-kiener As far as I saw once I am on the mollie payment page, I can change the payment method there directly too which changing the payment fails handling will not resolve.


[TODO] Add Thai Whisper

Add Thai Whisper

https://huggingface.co/biodatlab/whisper-small-th-combined
https://huggingface.co/biodatlab/whisper-th-medium-combined
https://huggingface.co/biodatlab/whisper-th-large-combined

Add Thai Whisper

https://huggingface.co/biodatlab/whisper-small-th-combined
https://huggingface.co/biodatlab/whisper-th-medium-combined
https://huggingface.co/biodatlab/whisper-th-large-combined

Hello, I want to generate subtitles for Thai videos(mp4 format), which model should I choose to achieve the best results? I have tried whisper-large-v2 and v3 but both of them make a lot of mistakes. Do you have any recommends? Looking forward to your reply.


We get this lockup occasionally too leading to a crash. Here's the stack trace. The background threads are not related to AFNetworking -- just the main thread.

We get this lockup occasionally too leading to a crash. Here's the stack trace. The background threads are not related to AFNetworking -- just the main thread.
Thread 0:
0   libsystem_kernel.dylib               0x00000001ec1969d4 __ulock_wait + 8
1   libdispatch.dylib                    0x00000001ebfdab94 _dispatch_thread_event_wait_slow$VARIANT$mp + 44
2   libdispatch.dylib                    0x00000001ebfe6ec0 __DISPATCH_WAIT_FOR_QUEUE__ + 324
3   libdispatch.dylib                    0x00000001ebfe6ad8 _dispatch_sync_f_slow + 136
4   <>                           0x000000010266b8cc -[AFHTTPRequestSerializer HTTPRequestHeaders] (AFURLRequestSerialization.m:306)
5   <>                           0x000000010267075c -[AFJSONRequestSerializer requestBySerializingRequest:withParameters:error:] (AFURLRequestSerialization.m:1246)
6   <>                           0x000000010266c0dc -[AFHTTPRequestSerializer requestWithMethod:URLString:parameters:error:] (AFURLRequestSerialization.m:376)
7   <>                           0x0000000101e32a78 -[AFHTTPSessionManager dataTaskWithHTTPMethod:URLString:parameters:uploadProgress:downloadProgress:success:failure:] (AFHTTPSessionManager.m:276)
8   <>                           0x0000000101e32130 -[AFHTTPSessionManager POST:parameters:progress:success:failure:] (AFHTTPSessionManager.m:179)

Here's the code where it is locking up:
dispatch_sync(self.requestHeaderModificationQueue, ^{
        value = [NSDictionary dictionaryWithDictionary:self.mutableHTTPRequestHeaders];
});

I'm not quite sure why it fails to acquire the queue. I'm inclined to think that a dispatch_sync on the main queue to a lower priority queue is a possible cause.
Originally posted by @gestrich in https://github.com/AFNetworking/AFNetworking/issues/4425#issuecomment-726060079

version: 3.2.1


Instructions out of date

The Amazon tool suite referred to here is deprecated. These instructions should be rewritten to use awscli.
https://github.com/snowplow/snowplow/wiki/Enable-support-for-HTTPS#openssl-prep-keys

Thanks for raising! Assigning...

Wow, that's fast. Do you use some special tooling to respond that fast?

We get all notifications into our internal chat!

Very nice.

@alexanderdean - Deprecated the info and provided the link to the instructions from Amazon docs. It might not be sufficient but it's not something I could verify myself. Our own instructions could be provided when we, say, renew our certificate.

Thanks @ihortom - will leave this open till we re-write the section. We should probably also change these instructions to assume the user is working with an AWS-purchased certificate to simplify things.

moving to docs, closing


T9 #59

Hello @dexion
Thanks for the PR and sorry for the delay.
The AppCompatActivity has already been added to the project.
Unfortunately I won't add the T9 to the design sorry!
Cheers,
Olivier

You may leave strings <string name="button2_small_text">ABC</string> blank and get old visual style, only digits without chars. I think library should be more customizable. Someone want chars, someone none. Let them choose )


Extended LUT Support: Now accept external LUT netlists with embedded custom logic

Motivate of the pull request

[ ] To address an existing issue. If so, please provide a link to the issue.
[X] Breaking new feature. If so, please decribe details in the description part.

Describe the technical details

What is currently done? (Provide issue link if applicable)

Current OpenFPGA only support pure LUT logic in LUT circuit model.
It means that all the inputs and outputs are driven and drive the internal multiplexer of a LUT.
This imposes a limitation that modern LUT may contain embedded carry logic gates.

What does this pull request change?

This pull request introduces the extended support on LUT circuit modeling, where any customized LUT circuit model (with any embedded logic gates) is supported.
Here is a quick example:

The module of the super LUT can be described can be described using the following XML codes:
<circuit_model type="lut" name="frac_lut4_arith" prefix="frac_lut4_arith" dump_structural_verilog="true" verilog_netlist="${OPENFPGA_PATH}/openfpga_flow/openfpga_cell_library/verilog/frac_lut4_arith.v">
  <design_technology type="cmos" fracturable_lut="true"/>
  <input_buffer exist="false"/>
  <output_buffer exist="true" circuit_model_name="sky130_fd_sc_hd__buf_2"/>
  <lut_input_inverter exist="true" circuit_model_name="sky130_fd_sc_hd__inv_1"/>
  <lut_input_buffer exist="true" circuit_model_name="sky130_fd_sc_hd__buf_2"/>
  <lut_intermediate_buffer exist="true" circuit_model_name="sky130_fd_sc_hd__buf_2" location_map="-1-"/>
  <pass_gate_logic circuit_model_name="sky130_fd_sc_hd__mux2_1"/>
  <port type="input" prefix="in" size="4" tri_state_map="---1" circuit_model_name="sky130_fd_sc_hd__or2_1"/>
  <port type="input" prefix="cin" size="1" is_harden_lut_port="true"/>
  <port type="output" prefix="lut3_out" size="2" lut_frac_level="3" lut_output_mask="0,1"/>
  <port type="output" prefix="lut4_out" size="1" lut_output_mask="0"/>
  <port type="output" prefix="cout" size="1" is_harden_lut_port="true"/>
  <port type="sram" prefix="sram" size="16"/>
  <port type="sram" prefix="mode" size="2" mode_select="true" circuit_model_name="DFFRQ" default_val="1"/>
</circuit_model>

A new XML syntax is_harden_lut_port is introduced to specify those ports drive/are driven by internal hard logic of a LUT
Documentation has been updated with a detailed circuit model example and highlight in the technical highlights

Which part of the code base require a change
In general, modification on existing submodules are not acceptable. You should push changes to upstream.

[ ] VPR
[X] OpenFPGA libraries
[X] FPGA-Verilog
[X] FPGA-Bitstream
[X] FPGA-SDC
[ ] FPGA-SPICE
[ ] Flow scripts
[X] Architecture library
[X] Cell library

Checklist of the pull request

[X] Require code changes.
[X] Require new tests to be added
[X] Require an update on documentation

Impact of the pull request

[ ] Require a change on Quality of Results (QoR)
[ ] Break back-compatibility. If so, please list who may be influenced.

@tpagarani @lnsharma I am almost there in supporting the LUT with embedded carry logic.

I guess the previous doc changes are also added in the pull

I guess the previous doc changes are also added in the pull

Got it. Will merge master to dev so that the problem can be resolved.

@tpagarani @lnsharma I have finished my code changes here. Looking forward to the eblif to test adder functions.

@tpagarani @lnsharma I have finished my code changes here. Looking forward to the eblif to test adder functions.

I raised a PR to bump yosys submodule. Once this PR is merged, you can call custom yosys script (openfpga_flow/misc/quicklogic_yosys_flow_ap3_adder.ys) to synthesize in adder mode.


Change: change Vote<NID:NodeId> to Vote<C:RaftTypeConfig>

Changelog
Change: change Vote<NID:NodeId> to Vote<C:RaftTypeConfig>
This refactoring moves Vote from a per-NodeId type to a per-TypeConfig type,
to make it consistent with RaftTypeConfig usage across the codebase.

Part of: #1278

Upgrade tip:
Vote is now parameterized by RaftTypeConfig instead of NodeId

Change Vote<NodeId> to Vote<C> where C: RaftTypeConfig, for
example, change Vote<u64> to Vote<YourTypeConfig>.

This change is

To address #1278, types like Vote<NID>, LeaderId<NID>, and LogId<NID> need to be updated to use RaftTypeConfig. This configuration encapsulates all customizable associated types, rather than relying on a single NID: NodeID.

dose change Vote definition need to do some backward compatibility test?say: save some Vote data of old format, and read it using new format.

No. There is no data format change. The only thing changed is the way to declare the data format: Vote<NID> -> Vote<RaftTypeConfig>

dose change Vote definition need to do some backward compatibility test?say: save some Vote data of old format, and read it using new format.

No. There is no data format change. The only thing changed is the way to declare the data format: Vote<NID> -> Vote<RaftTypeConfig>

i see the new definition is
pub struct Vote<C: RaftTypeConfig> {
    /// The id of the node that tries to become the leader.
    pub leader_id: LeaderId<C::NodeId>,

    pub committed: bool,
}

so there is no format change. but my question is: since the new and old definition declare id as C::NodeId, what the point introduce RaftTypeConfig param instead of NID: NodeId?

so there is no format change. but my question is: since the new and old definition declare id as C::NodeId, what the point introduce RaftTypeConfig param instead of NID: NodeId?

In order to allow to define all of the data types in protobuf, such as Vote, LeaderId, LogId, these types can not use a Rust type field. But instead, they must reference the types with C::XXX, for example:
struct Vote<NID> {
 leader_id: LeaderId<_>,
 // ..
}

Should be updated to
struct Vote<C> {
 leader_id: C::LeaderId,
}


ImageMinify won't compress *.svg files after updating to v2.36<PHONE_NUMBER>39) via App Store

I am running:

OSX El Capitan v10.11.6
iMac Retina 5k, 27", Late 2015
Processor: 3.3 GHz Intel Core i5
Memory: 8GB 1867 MHz DDR3
Graphics: AMD Radeon R9 M395 2048 MB

ImageMinify was running fine (with a few fails here and there that i remedied by re-setting the "save" location), UNTIL the latest update to v2.36<PHONE_NUMBER>39) after which it'll compress *.jpg and *.png files, but NOT *.svg (the most valuable to me).

I've tried:

resetting the “save” location
shutting everything down and restarting my machine
uninstalling/re-installing

… all to no avail.
PLEASE HELP!
I need this to work for my job!
THANK YOU for this amazing software!
shigi
p.s.

is it possible to downgrade to the previous working version?
if so, can you tell me how?

@shigimcp
Thanks for your feedback.
We will check...
p.s.
old version,(no MAS) http://www.filefactory.com/file/gtihpvymxyh/ImageMinify-v2.19.0-darwin-x64.zip
Give me a email, I will send a Key to you.

HI!
you can email me here<EMAIL_ADDRESS>THANK YOU!

@shigimcp
We have already sent you an email.


TypeError: natten1dqkrpb() takes 4 positional arguments but 5 were given

Dear Taejun, dear Vitaly,
We really love your work and would like to point out that it's brilliant!
We'd love to try it out but are currently stuck with the latest error:
TypeError: natten1dqkrpb() takes 4 positional arguments but 5 were given
We're using Python 3.11.7 on a Apple M1 Silicon with manually installed Natten 0.14.4 - might there be a problem with Natten 0.14.4 beeing older than the latest version 0.17.1?
We'd love to hear from you!
Thank you so much and best regards!

TypeError                                 Traceback (most recent call last)
Cell In[6], line 1
----> 1 result = allin1.analyze('/Users/justus/Downloads/Happy.wav')
File /opt/anaconda3/lib/python3.11/site-packages/allin1/analyze.py:134, in analyze(paths, out_dir, visualize, sonify, model, device, include_activations, include_embeddings, demix_dir, spec_dir, keep_byproducts, overwrite, multiprocess)
131 for path, spec_path in pbar:
132   pbar.set_description(f'Analyzing {path.name}')
--> 134   result = run_inference(
135     path=path,
136     spec_path=spec_path,
137     model=model,
138     device=device,
139     include_activations=include_activations,
140     include_embeddings=include_embeddings,
141   )
143   # Save the result right after the inference.
144   # Checkpointing is always important for this kind of long-running tasks...
145   # for my mental health...
146   if out_dir is not None:
File /opt/anaconda3/lib/python3.11/site-packages/allin1/helpers.py:29, in run_inference(path, spec_path, model, device, include_activations, include_embeddings)
26 spec = np.load(spec_path)
27 spec = torch.from_numpy(spec).unsqueeze(0).to(device)
---> 29 logits = model(spec)
31 metrical_structure = postprocess_metrical_structure(logits, model.cfg)
32 functional_structure = postprocess_functional_structure(logits, model.cfg)
File /opt/anaconda3/lib/python3.11/site-packages/torch/nn/modules/module.py:1532, in Module._wrapped_call_impl(self, *args, **kwargs)
1530     return self._compiled_call_impl(*args, **kwargs)  # type: ignore[misc]
1531 else:
-> 1532     return self._call_impl(*args, **kwargs)
File /opt/anaconda3/lib/python3.11/site-packages/torch/nn/modules/module.py:1541, in Module._call_impl(self, *args, **kwargs)
1536 # If we don't have any hooks, we want to skip the rest of the logic in
1537 # this function, and just call forward.
1538 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
1539         or _global_backward_pre_hooks or _global_backward_hooks
1540         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1541     return forward_call(*args, **kwargs)
1543 try:
1544     result = None
File /opt/anaconda3/lib/python3.11/site-packages/allin1/models/ensemble.py:21, in Ensemble.forward(self, x)
20 def forward(self, x):
---> 21   outputs: List[AllInOneOutput] = [model(x) for model in self.models]
22   avg = AllInOneOutput(
23     logits_beat=torch.stack([output.logits_beat for output in outputs], dim=0).mean(dim=0),
24     logits_downbeat=torch.stack([output.logits_downbeat for output in outputs], dim=0).mean(dim=0),
(...)
27     embeddings=torch.stack([output.embeddings for output in outputs], dim=-1),
28   )
30   return avg
File /opt/anaconda3/lib/python3.11/site-packages/allin1/models/ensemble.py:21, in (.0)
20 def forward(self, x):
---> 21   outputs: List[AllInOneOutput] = [model(x) for model in self.models]
22   avg = AllInOneOutput(
23     logits_beat=torch.stack([output.logits_beat for output in outputs], dim=0).mean(dim=0),
24     logits_downbeat=torch.stack([output.logits_downbeat for output in outputs], dim=0).mean(dim=0),
(...)
27     embeddings=torch.stack([output.embeddings for output in outputs], dim=-1),
28   )
30   return avg
File /opt/anaconda3/lib/python3.11/site-packages/torch/nn/modules/module.py:1532, in Module._wrapped_call_impl(self, *args, **kwargs)
1530     return self._compiled_call_impl(*args, **kwargs)  # type: ignore[misc]
1531 else:
-> 1532     return self._call_impl(*args, **kwargs)
File /opt/anaconda3/lib/python3.11/site-packages/torch/nn/modules/module.py:1541, in Module._call_impl(self, *args, **kwargs)
1536 # If we don't have any hooks, we want to skip the rest of the logic in
1537 # this function, and just call forward.
1538 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
1539         or _global_backward_pre_hooks or _global_backward_hooks
1540         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1541     return forward_call(*args, **kwargs)
1543 try:
1544     result = None
File /opt/anaconda3/lib/python3.11/site-packages/allin1/models/allinone.py:51, in AllInOne.forward(self, inputs, output_attentions)
48 inputs = inputs.reshape(-1, 1, T, F)  # N x K, C=1, T, F=81
49 frame_embed = self.embeddings(inputs)  # NK, T, C=16
---> 51 encoder_outputs = self.encoder(
52   frame_embed,
53   output_attentions=output_attentions,
54 )
55 hidden_state_levels = encoder_outputs[0]
57 hidden_states = hidden_state_levels[-1].reshape(N, K, T, -1)  # N, K, T, C=16
File /opt/anaconda3/lib/python3.11/site-packages/torch/nn/modules/module.py:1532, in Module._wrapped_call_impl(self, *args, **kwargs)
1530     return self._compiled_call_impl(*args, **kwargs)  # type: ignore[misc]
1531 else:
-> 1532     return self._call_impl(*args, **kwargs)
File /opt/anaconda3/lib/python3.11/site-packages/torch/nn/modules/module.py:1541, in Module._call_impl(self, *args, **kwargs)
1536 # If we don't have any hooks, we want to skip the rest of the logic in
1537 # this function, and just call forward.
1538 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
1539         or _global_backward_pre_hooks or _global_backward_hooks
1540         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1541     return forward_call(*args, **kwargs)
1543 try:
1544     result = None
File /opt/anaconda3/lib/python3.11/site-packages/allin1/models/allinone.py:110, in AllInOneEncoder.forward(self, frame_embed, output_attentions)
108 hidden_states = frame_embed
109 for i, layer in enumerate(self.layers):
--> 110   layer_outputs = layer(hidden_states, output_attentions)
111   hidden_states = layer_outputs[0]
112   hidden_state_levels.append(hidden_states)
File /opt/anaconda3/lib/python3.11/site-packages/torch/nn/modules/module.py:1532, in Module._wrapped_call_impl(self, *args, **kwargs)
1530     return self._compiled_call_impl(*args, **kwargs)  # type: ignore[misc]
1531 else:
-> 1532     return self._call_impl(*args, **kwargs)
File /opt/anaconda3/lib/python3.11/site-packages/torch/nn/modules/module.py:1541, in Module._call_impl(self, *args, **kwargs)
1536 # If we don't have any hooks, we want to skip the rest of the logic in
1537 # this function, and just call forward.
1538 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
1539         or _global_backward_pre_hooks or _global_backward_hooks
1540         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1541     return forward_call(*args, **kwargs)
1543 try:
1544     result = None
File /opt/anaconda3/lib/python3.11/site-packages/allin1/models/allinone.py:170, in AllInOneBlock.forward(self, hidden_states, output_attentions)
167 NK, T, C = hidden_states.shape
168 N, K = NK // self.cfg.data.num_instruments, self.cfg.data.num_instruments
--> 170 timelayer_outputs = self.timelayer(hidden_states, output_attentions)
171 hidden_states = timelayer_outputs[0]
172 if self.cfg.instrument_attention:
File /opt/anaconda3/lib/python3.11/site-packages/torch/nn/modules/module.py:1532, in Module._wrapped_call_impl(self, *args, **kwargs)
1530     return self._compiled_call_impl(*args, **kwargs)  # type: ignore[misc]
1531 else:
-> 1532     return self._call_impl(*args, **kwargs)
File /opt/anaconda3/lib/python3.11/site-packages/torch/nn/modules/module.py:1541, in Module._call_impl(self, *args, **kwargs)
1536 # If we don't have any hooks, we want to skip the rest of the logic in
1537 # this function, and just call forward.
1538 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
1539         or _global_backward_pre_hooks or _global_backward_hooks
1540         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1541     return forward_call(*args, **kwargs)
1543 try:
1544     result = None
File /opt/anaconda3/lib/python3.11/site-packages/allin1/models/dinat.py:298, in _DinatLayerNd.forward(self, hidden_states, output_attentions)
295 if attention is None:
296   continue
--> 298 attention_output = attention(attention_inputs, output_attentions=output_attentions)
299 attention_output = attention_output[0]
301 if is_2d:
File /opt/anaconda3/lib/python3.11/site-packages/torch/nn/modules/module.py:1532, in Module._wrapped_call_impl(self, *args, **kwargs)
1530     return self._compiled_call_impl(*args, **kwargs)  # type: ignore[misc]
1531 else:
-> 1532     return self._call_impl(*args, **kwargs)
File /opt/anaconda3/lib/python3.11/site-packages/torch/nn/modules/module.py:1541, in Module._call_impl(self, *args, **kwargs)
1536 # If we don't have any hooks, we want to skip the rest of the logic in
1537 # this function, and just call forward.
1538 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
1539         or _global_backward_pre_hooks or _global_backward_hooks
1540         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1541     return forward_call(*args, **kwargs)
1543 try:
1544     result = None
File /opt/anaconda3/lib/python3.11/site-packages/allin1/models/dinat.py:193, in _NeighborhoodAttentionModuleNd.forward(self, hidden_states, output_attentions)
188 def forward(
189   self,
190   hidden_states: torch.Tensor,
191   output_attentions: Optional[bool] = False,
192 ) -> Tuple[torch.Tensor]:
--> 193   self_outputs = self.self(hidden_states, output_attentions)
194   attention_output = self.output(self_outputs[0])
195   outputs = (attention_output,) + self_outputs[1:]  # add attentions if we output them
File /opt/anaconda3/lib/python3.11/site-packages/torch/nn/modules/module.py:1532, in Module._wrapped_call_impl(self, *args, **kwargs)
1530     return self._compiled_call_impl(*args, **kwargs)  # type: ignore[misc]
1531 else:
-> 1532     return self._call_impl(*args, **kwargs)
File /opt/anaconda3/lib/python3.11/site-packages/torch/nn/modules/module.py:1541, in Module._call_impl(self, *args, **kwargs)
1536 # If we don't have any hooks, we want to skip the rest of the logic in
1537 # this function, and just call forward.
1538 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
1539         or _global_backward_pre_hooks or _global_backward_hooks
1540         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1541     return forward_call(*args, **kwargs)
1543 try:
1544     result = None
File /opt/anaconda3/lib/python3.11/site-packages/allin1/models/dinat.py:99, in _NeighborhoodAttentionNd.forward(self, hidden_states, output_attentions)
95 query_layer = query_layer / math.sqrt(self.attention_head_size)
97 # Compute NA between "query" and "key" to get the raw attention scores, and add relative positional biases.
98 # attention_scores = natten2dqkrpb(query_layer, key_layer, self.rpb, self.dilation)
---> 99 attention_scores = self.nattendqkrpb(query_layer, key_layer, self.rpb, self.kernel_size, self.dilation)
101 # Normalize the attention scores to probabilities.
102 attention_probs = nn.functional.softmax(attention_scores, dim=-1)
TypeError: natten1dqkrpb() takes 4 positional arguments but 5 were given

Hi, yes you should use the latest version of natten.
Their API changed at some point.
Please open the issue again if it doesn't get fixed even after the upgrade!
Thanks.

I was having the same issue.  Natten 0.17.1 will not work on MPS, so reverting to 0.14.4 works, however then we hit this error.   So, using natten 0.14.4...
Thankfully, the errors from there are simple enough to fix.
On dinat.py, just remove 'self.kernel_size' from the self.nattendqkrpb() and  self.nattendav() function calls on lines 99 and 111.
`    #attention_scores = self.nattendqkrpb(query_layer, key_layer, self.rpb, self.kernel_size, self.dilation)
attention_scores = self.nattendqkrpb(query_layer, key_layer, self.rpb, self.dilation)
# Normalize the attention scores to probabilities.
attention_probs = nn.functional.softmax(attention_scores, dim=-1)

# This is actually dropping out entire tokens to attend to, which might
# seem a bit unusual, but is taken from the original Transformer paper.
attention_probs = self.dropout(attention_probs)

# context_layer = natten2dav(attention_probs, value_layer, self.dilation)
context_layer = self.nattendav(attention_probs, value_layer, self.dilation)`


[feature] Convert the plugin to use plugin_platform_interface

Currently the plugin provides support for Android, iOS, MacOS and web.
However, the current implementation mixes Texture & HTMLElementView directly in the implementation
This can be improved by migrating the plugin to a Federated Plugin.
This refactor will include the following changes (on a high level)

a platform interface, i.e. MobileScannerPlatformInterface extends PlatformInterface which defines the current implementation instance and the interface for the public API.
a MethodChannel implementation for Android, iOS, MacOS, ...
a web implementation
the platform interface will have a method like Widget buildScannerView(); which is implemented using a Texture / HTMLElement view under the hood

A starting point would be to recreate the plugin using flutter create -t plugin mobile_scanner, which includes the platform interface boilerplate.
See also https://docs.flutter.dev/development/packages-and-plugins/developing-packages#federated-plugins
This would also make implementing https://github.com/juliansteenbakker/mobile_scanner/issues/14 a bit easier, since only the windows implementation needs to be set up using the PlatformInterface method

@teolemon I would like to start working on this in the coming days.
My plan is to set up the plugin in such a way that we abstract implementation details behind the platform interface.
This would hide implementation details (such as webId vs textureId) and make fixing things a bit easier (I think)

@juliansteenbakker Just an FYI: I'll be tackling this one next.
The plan is to indeed have a common platform interface, which hides implementation details (i.e. Texture vs HtmlElementView). I'll also convert the web implementation to a full blown static interop layer for ZXing (having a MethodChannel on the web is a little weird imo)
I'll make sure I don't forget the MacOS implementation along the way :)
This is going to be a big effort, so it might take a while to get it done.

@navaronbracke @juliansteenbakker I assigned it to you

@juliansteenbakker As I finally have some time to address this, I have started the work on this issue.
I hope that with this refactor I can also address several reported issues.

Thank you very much @navaronbracke ! If you're interested, i can add you as a maintainer of this package. I unfortunately don't have enough time to keep it properly maintained right now, so some help is always welcome 😄

I would be interested in that. I think with multiple maintainers working on it, we can divide the workload a bit better.


[ci] Add CI check to ensure consistent version of OTEL semconv

Problem
As seen in this PR #4645, we used a random collection of semconv versions.
Proposal

Add a script to search the code for all usages of "go.opentelemetry.io/otel/semconv/v1.xx.x" and fail if they are not identical, with an appropriate error message and the exact files that are behind.
Run that script as part of make lint

Stretch goal

A script that automatically fixes the versions (suggest running the script in the output of the lint script)
Another check that determines the latest version of semconv available (1.20.0 is the latest right now), and ensure that all files reference that version.

Hello @yurishkuro, I'm interested in working on this issue. Could you please assign it to me? Thanks!

Hey @yurishkuro as I was trying to setup the project I encountered this issue

I have already tried removing the go.sum and then regenerated it using go mod tidy but still if gave me the same error. I am new to contributing and stuff, can you suggest me what should I do next ?


how to run decoding on chain model built by myself

Guenter,
Thanks for your work, I successfully compiled and tested your code. And also I can do decoding using your pre-trained model (kaldi-generic-en-tdnn_f-r20190609) now. Great work!
Meanwhile, I am learning Kaldi, and almost finished my first model building using one of Mandarin recipes (aishell). And I just want to know whether it is possible to run decoding task using your code with my own model?
My model is a chain model  which was built following the standard recipe (kaldi/egs/aishell/s5/local/chain/run_tdnn.sh)
One possible difficulty I realized is how to collect all the needed data files into model directory, please advice if there is a instruction document on which files are needed and the corresponding source directory of kaldi. By reading your code, seems just collect all the files mentioned in the following code:
    ****cdef unicode mfcc_config           = u'%s/conf/mfcc_hires.conf'                  % self.modeldir
    cdef unicode word_symbol_table     = u'%s/%s/graph/words.txt'                    % (self.modeldir, self.model)
    cdef unicode model_in_filename     = u'%s/%s/final.mdl'                          % (self.modeldir, self.model)
    cdef unicode splice_conf_filename  = u'%s/ivectors_test_hires/conf/splice.conf'  % self.modeldir
    cdef unicode fst_in_str            = u'%s/%s/graph/HCLG.fst'                     % (self.modeldir, self.model)
    cdef unicode align_lex_filename    = u'%s/%s/graph/phones/align_lexicon.int'     % (self.modeldir, self.model)**

    **self.ie_conf_f.write((u"--cmvn-config=%s/conf/online_cmvn.conf\n" % self.modeldir).encode('utf8'))**
    self.ie_conf_f.write((u"--ivector-period=%d\n" % online_ivector_period).encode('utf8'))
    **self.ie_conf_f.write((u"--splice-config=%s\n" % splice_conf_filename).encode('utf8'))**
    **self.ie_conf_f.write((u"--lda-matrix=%s/extractor/final.mat\n" % self.modeldir).encode('utf8'))
    self.ie_conf_f.write((u"--global-cmvn-stats=%s/extractor/global_cmvn.stats\n" % self.modeldir).encode('utf8'))
    self.ie_conf_f.write((u"--diag-ubm=%s/extractor/final.dubm\n" % self.modeldir).encode('utf8'))**
    **self.ie_conf_f.write((u"--ivector-extractor=%s/extractor/final.ie\n" % self.modeldir).encode('utf8'))**
    self.ie_conf_f.write((u"--num-gselect=%d\n" % num_gselect).encode('utf8'))
    self.ie_conf_f.write((u"--min-post=%f\n" % min_post).encode('utf8'))
    self.ie_conf_f.write((u"--posterior-scale=%f\n" % posterior_scale).encode('utf8'))
    self.ie_conf_f.write((u"--max-remembered-frames=1000\n").encode('utf8'))
    self.ie_conf_f.write((u"--max-count=%d\n" % max_count).encode('utf8'))
    self.ie_conf_f.flush()**

Could you kindly elaborate a little about the source directory of those needed files?
Thanks!
Kelvin

Guenter,
Have figure it out, :-) thanks!
Kelvin

@kelvinqin As aishell is a popular recipe, would you mind sharing your solution? :-)


need help

cloud 9 is asking for credit card for signup please tell me alternative of this because i don't have credit card...and i want to make a full stack project in one month becoz i have to submit it as a major project in college plz help me..and my project topic is placement portal based on html css js as afront end and node.js as a back end..i need help plz fix this issue

@atulsingh378, sorry, we can't help with your issue.  your issue is nothing with fcc repo.   FCC site and cloud 9 is separate. if you have an issue with cloud 9. you need to talk to cloud 9 support.
have a nice day.


Add esatic to JetBrains

https://www.esatic.ci/

@mcoul8 I corrected the filepath for you and merged the request. Everything is OK. Thank you.


Settings editor - shift+enter should search backwards

Open settings editor
press cmd+f to do a search
enter searches forward, shift+enter should search backwards

Also consider hooking up F3 and cmd+g

It is supported now.


Go build rules: only stamp actual go_binary outputs [phab:D865]

Created by schroederc at 2016-04-29 16:27:13:
The build rules currently stamp all linked Go binaries with the data
from stable-status.txt and volatile-status.txt, but this isn't really
needed for test binaries.  Also, since all actions depending on
volatile-status.txt will have to rerun for each build since it embeds a
timestamp, this change will improve the runtime for a null Bazel build.

shahms commented inline (context lost during migration) at 2016-04-29 16:32:42:
wrap at 80

shahms accepted this revision at 2016-04-29 16:32:42

schroederc wrote at 2016-04-29 16:37:13:
Wrap

schroederc closed via commit 780aace2a4737c3b90b5fc9eaa3566efc2c42860 at 2016-04-29 16:37:27


bug: pg_sequence not found when write data with polars

Describe the bug
RisingWave does not have the pg_sequence view. When polars using method Dataframe.write_database to writes data and specifies if_table_exists as replace, the generated sql will use this view, which leads to the error.
Error message/log
2024-03-27T03:33:06.617702+08:00 ERROR        rw-main handle_query{mode="simple query" session_id=1 sql=SELECT pg_catalog.pg_attribute.attname AS name, pg_catalog.format_type(pg_catalog.pg_attribute.atttypid, pg_catalog.pg_attribute.atttypmod) AS format_type, (SELECT pg_catalog.pg_get_expr(pg_catalog.pg_attrdef.adbin, pg_catalog.pg_attrdef.adrelid) AS pg_get_expr_1
FROM pg_catalog.pg_attrdef
WHERE pg_catalog.pg_attrdef.adrelid = pg_catalog.pg_attribute.attrelid AND pg_catalog.pg_attrdef.adnum = pg_catalog.pg_attribute.attnum AND pg_catalog.pg_attribute.atthasdef) AS "default", pg_catalog.pg_attribute.attnotnull AS not_null, pg_catalog.pg_class.relname AS table_name, pg_catalog.pg_description.description AS comment, pg_catalog.pg_attribute.attgenerated AS generated, (SELECT json_build_object('always', pg_catalog.pg_attribute.attidentity = 'a', 'start', pg_catalog.pg_sequence.seqstart, 'increment', pg_catalog.pg_sequence.seqincrement, 'minvalue', pg_catalog.pg_sequence.seqmin, 'maxvalue', pg_catalog.pg_sequence.seqmax, 'cache', pg_catalog.pg_sequence.seqcache, 'cycle', pg_catalog.pg_sequence.seqcycle) AS json_build_object_1
FROM pg_catalog.pg_sequence
WHERE pg_catalog.pg_attribute.attidentity != '' AND pg_catalog.pg_sequence.seqrelid = CAST(CAST(pg_catalog.pg_get_serial_sequence(CAST(CAST(pg_catalog.pg_attribute.attrelid AS REGCLASS) AS TEXT), pg_catalog.pg_attribute.attname) AS REGCLASS) AS OID)) AS identity_options
FROM pg_catalog.pg_class LEFT OUTER JOIN pg_catalog.pg_attribute ON pg_catalog.pg_class.oid = pg_catalog.pg_attribute.attrelid AND pg_catalog.pg_attribute.attnum > 0 AND NOT pg_catalog.pg_attribute.attisdropped LEFT OUTER JOIN pg_catalog.pg_description ON pg_catalog.pg_description.objoid = pg_catalog.pg_attribute.attrelid AND pg_catalog.pg_description.objsubid = pg_catalog.pg_attribute.attnum JOIN pg_catalog.pg_namespace ON pg_catalog.pg_namespace.oid = pg_catalog.pg_class.relnamespace
WHERE pg_catalog.pg_class.relkind = ANY (ARRAY['r', 'p', 'f', 'v', 'm']) AND pg_catalog.pg_table_is_visible(pg_catalog.pg_class.oid) AND pg_catalog.pg_namespace.nspname != 'pg_catalog' AND pg_catalog.pg_class.relname IN ('taxi_zones') ORDER BY pg_catalog.pg_class.relname, pg_catalog.pg_attribute.attnum}: pgwire::pg_protocol: error when process message error=Failed to run the query: Failed to bind expression: (SELECT json_build_object('always', pg_catalog.pg_attribute.attidentity = 'a', 'start', pg_catalog.pg_sequence.seqstart, 'increment', pg_catalog.pg_sequence.seqincrement, 'minvalue', pg_catalog.pg_sequence.seqmin, 'maxvalue', pg_catalog.pg_sequence.seqmax, 'cache', pg_catalog.pg_sequence.seqcache, 'cycle', pg_catalog.pg_sequence.seqcycle) AS json_build_object_1 FROM pg_catalog.pg_sequence WHERE pg_catalog.pg_attribute.attidentity <> '' AND pg_catalog.pg_sequence.seqrelid = CAST(CAST(pg_catalog.pg_get_serial_sequence(CAST(CAST(pg_catalog.pg_attribute.attrelid AS REGCLASS) AS TEXT), pg_catalog.pg_attribute.attname) AS REGCLASS) AS OID)): Feature is not yet implemented: pg_catalog.pg_sequence is not supported, please use `SHOW` commands for now.
`SHOW TABLES`,
`SHOW MATERIALIZED VIEWS`,
`DESCRIBE <table>`,
`SHOW COLUMNS FROM [table]`

Tracking issue: https://github.com/risingwavelabs/risingwave/issues/1695

To Reproduce
Run the following code twice.
import polars as pl

def send_csv_records(
    endpoint: str,
    tbl_name: str,
    conn_string: str,
    write_disposition: str = "replace",
    engine: str = "sqlalchemy",
):
    df = pl.read_csv(endpoint).rename(
        mapping={
            "LocationID": "location_id",
            "Borough": "borough",
            "Zone": "zone",
            "service_zone": "service_zone",
        }
    )

    return df.write_database(
        table_name=tbl_name,
        connection=conn_string,
        if_table_exists=write_disposition,  # type: ignore
        engine=engine,  # type: ignore
    )

if __name__ == "__main__":
    rw_host = "localhost"
    rw_port = 4566
    rw_db = "dev"
    rw_user = "root"

    num_records = send_csv_records(
        # endpoint="https://s3.amazonaws.com/nyc-tlc/misc/taxi+_zone_lookup.csv",
        endpoint="./taxi+_zone_lookup.csv",
        tbl_name="taxi_zones",
        conn_string=f"postgresql://{rw_user}@{rw_host}:{rw_port}/{rw_db}",
    )

    print(num_records)

The original code comes from: https://github.com/iobruno/data-engineering-zoomcamp/blob/80f2e1a2ff881cb5dfa174144c5ef9ae7c9e8f09/module6-stream-processing/risingwave/seed.py

Expected behavior
No response
How did you deploy RisingWave?
Build from source.
The version of RisingWave
e3f3fb8f89bbb4879855297f65965fdcc4f8eb9d
Additional context
No response

Closed because https://github.com/risingwavelabs/sqlalchemy-risingwave/pull/29 was merged and the problem was solved.
To get polars working, pip install sqlalchemy-risingwave and then connect using conn_string=f"risingwave://{rw_user}@{rw_host}:{rw_port}/{rw_db}".


Support for Private Registries

I don't see a way of performing calls such as Image.search and Image.push on privately hosted Docker Registries. Is there something I've overlooked or can this be a Feature Request?

Seems like this would fit a separate gem that paired well with ours.  If you release it, we'll be happy to link to it!


How to get an instance later in another component?

How to get an instance later in another component?

I created a first time Select (like in the documentation).
Now I would use an instance to change the values of this select in another component where I don't have a config from point one.

It's possible to do this on this plugin?
Thanks

Closing as @hope55 gave a good answer

Closing as @hope55 gave a good answer


Release Google.Cloud.BigQuery.DataTransfer.V1 version 3.3.0

Changes in this release:
New features

Add owner email to TransferConfig message (commit 5939502)
Allow customer to enroll a datasource programmatically (commit 5939502)

Documentation improvements

Improvements to various message and field descriptions (commit 5939502)

Created release for Google.Cloud.BigQuery.DataTransfer.V1-3.3.0


It should be possible to change the trigger for UncontrolledTooltip

Issue description

components: UncontrolledTooltip
reactstrap version 6.0.1
react version 16.3.2
bootstrap version ^4.1.1

What is happening?
It is not possible to change the trigger of the UncontrolledTooltip. It is always hover
What should be happening?
It should be possible to change the trigger for UncontrolledTooltip like this
<UncontrolledTooltip trigger="focus" placement="right" target="id">
           Some Text
</UncontrolledTooltip>

Popper.js Supports this see here: https://github.com/FezVrasta/popper.js/blob/master/docs/_includes/tooltip-documentation.md

I have been wanting to better align tooltip and popover (as well as their uncontrolled counterparts). It seems like they do similar things in similar ways except the trigger is different... but the trigger, as this issue mentions, should be able to be changed. Technically, you could have a popover work on mouse events and a tooltip work on click events. The only real different between the two would be the default events it uses and the classNames which align with the bootstrap styles for each.
If anyone wants to take a stab at resolving this, please keep the above in mind and also address this similar concern with the popover component (maybe by creating a base component with all of the logic and having other components which implements the defaults for tooltip and popover.)

@TheSharpieOne  react-popper has an events prop. Wouldn't passing a prop to react-popper solve this?

react-popper has eventsEnabled which defaults to true. It controls eventsEnabled from Popper.js which just determines if it should listen for resize and scroll events to update.

I'll give this a shot.

Will do for popover too...

@TheSharpieOne I want to build this like you said (by creating a base component with all of the logic and having other components which implements the defaults for tooltip and popover.), but I have never written tests, so when I write tests, should I write them for the base component or for the Tooltip component?


Changing nickname of Pokemon is not working

(This is my first issue report on github, so let me know if I'm doing this wrong)
nicknamePokemon() is only able to update Pokemon that were caught before the new update.  Oddly enough, it looks like it's able to rename Pokemon that hatched from eggs yesterday, but the eggs were from before the update.

Maybe this is related to the latest encryption changes (which we don't support just yet)? Seems a little odd that it would depend on the time the Pokémon have been caught but it's not completely impossible.

It's working now, I think it was my bad code.  Sorry about that!


feat(modal): Add size option to modal component

Add a size option to modal component, with three possibilities: small, medium(default value) and large.
Doc:

Current coverage is 100% (diff: 100%)

Merging #185 into master will not change coverage

@@           master   #185   diff @@
====================================
  Files          19     19
  Lines         340    343     +3
  Methods         0      0
  Messages        0      0
  Branches        0      0
====================================
+ Hits          340    343     +3
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last update 1789993...5fd09dc


Rework and migrate HA configuration system

[BUG]/[FEATURE] title
What changes were proposed in this pull request?
Restore master bind host/port configurations and clarify
it's part of the configuration migration plan, I left comments on the code blocks to explain what I changed.
Why are the changes needed?
I misunderstand the previous design, hope this change makes it clear enough.
What are the items that need reviewer attention?
Related issues.
Related pull requests.
How was this patch tested?
/cc @related-reviewer
/assign @main-reviewer

cc @AngersZhuuuu @zwangsheng

cc @AngersZhuuuu @waitinfuture @FMX thanks


Selenium got stuck in spamcallsStart

Traceback (most recent call last):
File "Moriarty-Project/DeepInfo/risks.py", line 32, in spamcallsStart
spam_risk=WebDriverWait(self.driver, 10).until(EC.element_to_be_clickable((By.XPATH, "/html/body/div[2]/div[1]/div/div[1]/div[1]/div/h3/strong"))).text
File "/usr/local/lib/python3.7/dist-packages/selenium/webdriver/support/wait.py", line 89, in until
raise TimeoutException(message, screen, stacktrace)
selenium.common.exceptions.TimeoutException: Message:
Stacktrace:
#0 0x556d5f1fbe89
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "Moriarty-Project/DeepInfo/risks.py", line 45, in spamcallsStart
spam_risk=WebDriverWait(self.driver, 10).until(EC.element_to_be_clickable((By.XPATH, "/html/body/div[2]/div[1]/div/div[2]/div[1]/div/div[1]/span[2]"))).text
File "/usr/local/lib/python3.7/dist-packages/selenium/webdriver/support/wait.py", line 89, in until
raise TimeoutException(message, screen, stacktrace)
selenium.common.exceptions.TimeoutException: Message:
Stacktrace:
#0 0x556d5f1fbe89
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "Moriarty.py", line 175, in
MP.run()
File "Moriarty.py", line 145, in run
self.spamcalls_()
File "Moriarty.py", line 122, in spamcalls_
spamcalls().spamcallsStart(self.phoneNumber)
File "Moriarty-Project/DeepInfo/risks.py", line 57, in spamcallsStart
spam_risk=WebDriverWait(self.driver, 10).until(EC.element_to_be_clickable((By.XPATH, "/html/body/div[2]/div[1]/div/div[2]/div[1]/div/div[1]/h4"))).text
File "/usr/local/lib/python3.7/dist-packages/selenium/webdriver/support/wait.py", line 89, in until
raise TimeoutException(message, screen, stacktrace)
selenium.common.exceptions.TimeoutException: Message:
Stacktrace:
#0 0x556d5f1fbe89
Hope you will find time to adjust your awesome product.

Hello, this project needs an update. Im thinking of moving this project from selenium to playwright but currently I don't have time for an update. But I will update this tool when Im available. Thanks for your interest in this project.

Good luck, mate!


getting cannot set blockOwnerDeletion, ownerReference resource you can't set finalizers

Hi,
I'm trying to install clickhouse-operator in Openshift 4.2 cluster, following the doc provided in this github here
however, after apply a simple ClickHouseInstallation I got some error event about unable to set finalizers on certain resources and stuck there, no pod spawned.
Can you help on this one?

it seems I have to manually update operator cluster role with the following finalizers in order to make this works.
  - verbs:
      - update
    apiGroups:
      - clickhouse.altinity.com
    resources:
      - clickhouseinstallations/finalizers
      - clickhouseinstallationtemplates/finalizers
      - clickhouseoperatorconfigurations/finalizers

closed via https://github.com/Altinity/clickhouse-operator/pull/783


[Feature request] Change project closing behavior

Hello,
Currently :

When a project contains unpushed commits, a warning is shown ;
When "prevent closing project" is checked, closing the warning will always keep the project open ;
When "prevent closing project" is unchecked, closing the warning will always close the project ;

What I recommend (option A) :

Remove the "prevent closing project" setting ;
Replace the "Please push before closing!" injunction with a "Would you like to close the project anyway?" question ;
Close the project when the primary OK button is clicked or when the Enter key is pressed ;
Keep the project open when a new secondary Cancel button is clicked, or when the Esc key is pressed, or when the top-right close button is clicked ;

Or do the opposite (option B) :

Remove the "prevent closing project" setting ;
Keep the injunction ;
Keep the project open when the primary OK button is clicked, or when the Enter key is pressed ;
Close the project when a new secondary No button is clicked, or when the Esc key is pressed, or when the top-right close button is clicked ;

Or allow both options, and eventually migrate users who checked the "prevent closing project" setting to option B, and the users who didn't to option A.
Thanks

Hi @KaKi87 - this is a great suggestion, thank you for submitting it.
Option A seems like a better UX in my opinion, so let's proceed with that one.
How does something like this look?

As you suggested, hitting Enter will proceed to allow the project to close, and hitting Esc will prevent the project from closing (ie, keep it open). Of course, clicking on the respective buttons do as you would expect. I opted to not have just "OK" and "Cancel" buttons for explicitness of behavior.
I will have a PR up shortly, if you would like to review it, that would be great! If I don't hear back from you in a few days, I will assume you have no interest in reviewing the PR and proceed to merge & release a version with this change. Let me know either way!

Hey,
This is perfect !
I'm not qualified for code review, if that's what you meant.
Thanks

I'm not qualified for code review, if that's what you meant.

You're more qualified than you may think ;)
I'll go ahead and merge it in then, but you can totally take a look at the PR; a quick breakdown:

GitPushReminder.java is simply

changing the dialog type (from messageDialog to OkCancelDialog),
the text of buttons and icon
checking if the user clicked 'ok' (ie, close anyway) or not

SettingsConfigurable.java and SettingsManager.java is simply removing the "prevent closing project" option
*.properties are just user-facing string properties; the two files are identical

Got it, thanks !
PS: I just noticed that you're an engineer at LinkedIn, so firstly I'm flattered by your consideration of me, secondly, may I ask how does Microsoft take that you prefer JetBrains IDEs over Visual Studio ?

No problem.

...how does Microsoft take that you prefer JetBrains IDEs over Visual Studio ?

Not sure, I have not directly asked them this question ;) That being said, most tech companies would prefer their employees are as productive as possible; sometimes that means using different tools (ie, IDEs) for various things. I likely can't comment on any specifics, of course, but I know that I am not the only engineer using a JetBrains IDE.

@all-contributors please add @KaKi87 for ideas


On language switch always error RangeError: Maximum call stack size exceeded no idea why ?

Error message: client.js?06a0:96 RangeError: Maximum call stack size exceeded     at Observer.toString (<anonymous>)     at klona (index.js?3aed:11)     at set (index.js?3aed:2)     at klona (index.js?3aed:48)     at set (index.js?3aed:2)     at klona (index.js?3aed:48)     at set (index.js?3aed:2)     at klona (index.js?3aed:48)     at set (index.js?3aed:2)     at klona (index.js?3aed:48)
src code:
onChange(locale) {
       var language = locale.code.toLocaleLowerCase();
       this.$i18n.setLocale(language);
       this.$router.replace(this.switchLocalePath(language));
     },

Why are you trying to setLocale from locale change event?

Why are you trying to setLocale from locale change event?

Hi rchl because i have a vue select element in the navbar which listens onChange events montha this was working absolutely ok, but since se update in my techbstack it breaks with the error callstack size, though the cookie is set ok so the language switch works on the cookie but some how not routing correct, i have no idea ...

removed onChange for nuxt-link now it works

Also a note that this.$router.replace(this.switchLocalePath(language)); call seems redundant because setLocale should change the route itself, if it's necessary.

hi !
I'm just using this.$i18n.setLocale(language);
and it works 100% fine only IF "nuxt-i18n": "^6.17.0",
any other version after that drops the error. Maybe not the first time but randomly after some lang switching

any help ?

Can't help without access to broken repo.


Postal changes DKIM prefix dynamically.

Hello!
I got an issue. The postal changes the DKIM prefix key dynamically like once in 24h. So the DKIM records are become not valid.
One day it is was like:
postal-dv5esd._domainkey
And now it becomes:
postal-ek3edu._domainkey
Why may this happen? How can I get a static DKIM key prefix?

I found the keys in the database. However, that is a better approach to add both keys or delete and leave one?

Postal doesn't change the prefix for a domain but if you added a new domain, or deleted and re added one, it would probably change. You also only have the one installation of Postal and one database server right?
You should use whatever is currently displayed on the DNS page.

@willpower232
In my DB have two dkim identifier: the first organization, the second server(it was by default). In my DNS setting panel, these keys alternate. And I get an error when this happens. Currently, I added both keys to my DNS records, and it seems that it works. However, why it may happen? And what is a better approach to fix it?

I found more detailed behavior. We have one user in  Postal(admin).  And two-person login to there. The first user in the admin panel sees one DKIM, the second another.

If the server you added the domain to is in the organization that already had it, you don't need to add the domain again.
If you added a user in the UI, they aren't an admin by default so can't see as much of the system. We've just been using the original make user command from the CLI to create our users as we don't need to differentiate between them.


It seems "Only SQL Server authentication is supported" is not a limit…

…ation any longer
We even have an article explaining how to enable AAD integrated authentication at https://docs.microsoft.com/en-us/azure/azure-arc/data/connect-active-directory-sql-managed-instance.

@nachoalonsoportillo : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"


Hello, here is my map case, but it is arranged to why it cannot run.

Uploading 新建文件夹.rar…

I'm trying to download your map, but nothing happens. Seems like it's just the link to the issue.

Excuse me, here is the map file I added.  The points of some agents at run time are shown to be invalid, and can run when I only keep the first agent, but cannot find the path.
all_in_one_example.zip

Excuse me, here is the map file I added.  The points of some agents at run time are shown to be invalid, and can run when I only keep the first agent, but cannot find the path.
------------------ 原始邮件 ------------------
发件人:                                                                                "PathPlanning/AA-SIPP-m"                                                                                @.>;
发送时间: 2021年6月3日(星期四) 晚上11:17
@.>;
<EMAIL_ADDRESS>主题: Re: [PathPlanning/AA-SIPP-m] Hello, here is my map case, but it is arranged to why it cannot run. (#11)
I'm trying to download your map, but nothing happens. Seems like it's just the link to the issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
从QQ邮箱发来的超大附件
all_in_one_example.zip (1.1K, 无限期)进入下载页面：http://mail.qq.com/cgi-bin/ftnExs_download?k=7d343366af9ec6c98819914240650b190f0d5251525209001b0307505e480d0301501e0400525d1b53050b0300525b5303020150667339575a586c0f083a5658536b561e0708495a531a490f166504&t=exs_ftn_download&code=643ffe96

Your input is not correct. Size=1 is the radius of the agent. With such a big size the agent occupies 9 cells and they all should be traversable, but your start of agent 1 is placed near the obstacle.
Set the size to 0.5.


Compatibility with safeVisionary2

Can the presented solution also be used with safeVisionary2 device, which also has CoLa-2 protocol, for extracting data from it without using a UDP communication?

The CoLa-2 protocol is used only for parameterization. The data streaming differs between the safe and non-safe devices. Thus, while it should be possible to reuse the CoLa-2 protocol layer to interact with the safeVisionary2 device you will find no support here for the different streaming protocol.