Wildfly 10 Issue:  WELD-001456: Argument resolvedBean must not be null

Hi.
Due to Glassfish being marked as not commercially supported anymore at some clients I cannot implement Glassfish as commercial support is an audit requirement. Thus I have been playing with this on WIldFly 10. I changed the persistence provider to Hibernate  in the persistence.xml as I expect at some clients they wont allow me to use EclipseLink either on their WildFly production servers.
I got the application to build but during deployment it stops at:
Deploying C:\wildfly-10.0.0.Final\standalone\deployments\JPAModellerDemo-1.0.war
{"WFLYCTL0080: Failed services" => {"jboss.persistenceunit."JPAModellerDemo-1.0.war#DEFAULT_PU"" => "org.jboss.msc.service.StartException in service jboss.persistenceunit."JPAModellerDemo-1.0.war#DEFAULT_PU": javax.persistence.PersistenceException: [PersistenceUnit: DEFAULT_PU] Unable to build Hibernate SessionFactory
Caused by: javax.persistence.PersistenceException: [PersistenceUnit: DEFAULT_PU] Unable to build Hibernate SessionFactory
Caused by: org.jboss.weld.exceptions.IllegalArgumentException: WELD-001456: Argument resolvedBean must not be null"}}
This seems like a known issue on WildFly see this link:
https://issues.jboss.org/browse/WFLY-2387
Also issue related logged on HIbernate JIRA
https://hibernate.atlassian.net/browse/HHH-8706
Also these StackOverflow items will be able to guide you.
http://stackoverflow.com/questions/35394384/org-jboss-weld-exceptions-illegalargumentexception-weld-001456-argument-resol
http://stackoverflow.com/questions/33891368/org-jboss-weld-exceptions-illegalargumentexception-weld-001456-argument-resolv
If there is a workaround in the current plugin? If not i am pretty sure I can figure it out but it is Fathers Day here today and I have to play dad to the kids for now.
I will try resolve and add notes when I get to it tomorrow or so.

Potential workaround found.
The problem loos like it related to the Entity Listener on the AbstractAuditingEntity  class.
I changed this
@MappedSuperclass @EntityListeners(AuditListner.class) public abstract class AbstractAuditingEntity implements Serializable {
To:
@MappedSuperclass //@EntityListeners(AuditListner.class) public abstract class AbstractAuditingEntity implements Serializable {
The error is resolved however running into SQL command statement errors on the deployment and run.
Caused by: javax.persistence.PersistenceException: Unable to execute JPA schema generation create command [INSERT INTO ACCOUNT(ID, LOGIN, EMAIL, FIRST_NAME, LAST_NAME, LANG_KEY, ACTIVATED, CREATED_DATE, PASSWORD_HASH) VALUES (1001,  'user'<EMAIL_ADDRESS>'User_F', 'User_L', 'en', 1, '2016-08-10 09:27:36.388', 'ee11cbb19052e40b07aac0ca060c23ee')]
Caused by: org.postgresql.util.PSQLException: ERROR: column "activated" is of type boolean but expression is of type integer.
Will resolve this and log an issue separately for the SQL statement. I do believe PostgerSQL is interpreting 1 as an int the statement should probably be like
 INSERT INTO ACCOUNT(ID, LOGIN, EMAIL, FIRST_NAME, LAST_NAME, LANG_KEY, ACTIVATED, CREATED_DATE, PASSWORD_HASH) VALUES (1001,  'user'<EMAIL_ADDRESS>'User_F', 'User_L', 'en', 1::bool, '2016-08-10 09:27:36.388', 'ee11cbb19052e40b07aac0ca060c23ee')

Hi.
I found several SQL related issues on PostgreSQL when trying to deploy on Wildfly. I will post them soon.I have fixed them as well. Got to the landing page but still cant log in. Will investigate and resolve later.

Thanks for reporting the issue. Currently we have tested it using Payara and MySql/Derby DB. Once you will done with all fix, Please update here.
Really thanks for your support :)

Hi jShiwaniGupta
I have found two or three PostgreSQL related SQL problems and also fixes. I will log these tonight on a new issue so we dont confuse the WildFly issue with the PostgreSQL issue.

i found the same issue in my project, and apply the steps above and it run, but when i try to login using user:admin pass:admin do nothing, any tips.
regards and great work :) 👍

@RandyHBH ,
Please share more details like App Server and DB name. And also attach the project.
If you are using Wildfly, please try the latest v2.6.4.

hi @jGauravGupta
i am using Wildfly 10.1.0, PostgreSql 9.4, i also tried with the latest version available and still i can't login with user:admin pass:admin. I attach the full project. regards
WildFlyJpaModeler.zip

i fix the isuue when try to login with JPA and Hibernate. I change in the auto-generated User class the way of fetch de authorities.
Before
@JsonIgnore
@ManyToMany
@JoinTable(name = "user_authority",
joinColumns = {
@JoinColumn(name = "user_id", referencedColumnName = "id")},
inverseJoinColumns = {
@JoinColumn(name = "authority_name", referencedColumnName = "name")})
private Set authorities = new HashSet<>();
Now:
@JsonIgnore
@ManyToMany(fetch = FetchType.EAGER)
@JoinTable(name = "user_authority",
joinColumns = {
@JoinColumn(name = "user_id", referencedColumnName = "id")},
inverseJoinColumns = {
@JoinColumn(name = "authority_name", referencedColumnName = "name")})
private Set authorities = new HashSet<>();
i found this isuue in stackoverflow and some info how to fix it using hibernate whitout change Lazy for Eager.
http://stackoverflow.com/questions/11746499/solve-failed-to-lazily-initialize-a-collection-of-role-exception#
http://docs.jboss.org/hibernate/orm/3.3/reference/en/html/performance.html#performance-fetching-initialization

"WELD-001456: Argument resolvedBean must not be null" issue tracking :
https://issues.jboss.org/browse/WFLY-2387


add download thumbnail

It may be necessary to review the thumbnail position/style in the frontend and the way to make the distinction between the thumbnail and the file when we upload them.

@arthurpar06 is this working as you expect?

@nabeelio for the upload and the backup everything works fine, after as I said it may be necessary to review the frontend part (image size and position) and add a label or something like that to differentiate the two <input type="file" /> in the admin

@arthurpar06 is this still active?

No, you can close it

Is it mostly working, though?

Yes I think we just had to revise the frontend a bit

@FatihKoz would you be able to pull this PR down and see what fix/changes it might need on the frontend?

Why not, but earliest possible time is Saturday or Sunday for a nice
check/update.
23 Ağu 2022 Sal 19:08 tarihinde Nabeel S. @.***> şunu
yazdı:

@FatihKoz https://github.com/FatihKoz would you be able to pull this PR
down and see what fix/changes it might need on the frontend?
—
Reply to this email directly, view it on GitHub
https://github.com/nabeelio/phpvms/pull/1395#issuecomment-1224286612,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARXKVMJ6UYP77PJ56OBQWDTV2TZQDANCNFSM5NVO3FBQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

@FatihKoz no rush at all

Just a question, do I need to pull down and create a new pr with my changes (which also will include fixes to current one, stlye fix and removal of down migration) ? Or should I only work on default theme frontend changes when able ?
I think I will not be able to commit on this PR 'cause I am not the owner and/or moderator etc.

I think we need more than just frontend/blade changes on this @nabeelio , there are some issues;

If there is no file uploaded along with thumbnail , it does not get uploaded (imagine a scenario like below)

path field: https://flightops.turksim.org/test.zip
thumbnail field: C:\Users\fatih\AppData\Local\Temp\phpD1E4.tmp

Model does not return full url for the thumbnail , it just returns what is recorded to database as expected
files/thumbnails/B8p07y7DdJP8Zbyl_startbootstrap-small-business-gh-pages_thumbnail.jpg

Second one is easy, it can be fixed with a model update to generate a new attribute like imageurl or it can be handled in blade with url helpers. First one needs some work.
Results from a quick look and test :(

@arthurpar06 do you want to fix this up or should I close it

I can try to take a look when I have some free time but I think we can close, it's not a revolutionary feature either


Add option to always reveal tests after state change

This PR fixes #151847
A new configuration parameter, named testing.alwaysRevealTestOnStateChange, is added. When enabled, after a test's state has changed, the tree view expands to reveal the corresponding item. The default value of the new parameter is set to false, which keeps the existing behavior in place.

@connor4312 Could you please review this?

Thanks for the PR! Since this option affects followRunningTest, it'd probably make sense to have the description be something like:

Always reveal the executed test when testing.followRunningTest is on. If this setting is turned off, only failed tests will be revealed.

@connor4312 Done.


pkg/ipcache: remove stale identity to IP mappings from XDS cache upon identity update for endpoint

If the identity is updated for an endpoint, the XDS cache not only needs to have the new
identity to list of IPs mapping updated, it needs to have the old identity to IP mapping
updated as well to account that the IP for which a key-value store Create or Modify event
no longer maps to its old identity. Delete the identity from the XDS cache if it no longer
maps to any IPs in the IPIdentityCache, or update it with the list of IPs that it still
maps to.
Signed-off by: Ian Vernon<EMAIL_ADDRESS>

test-me-please

test-me-please

Now the runtime tests fail to even start properly, seems completely independent of these changes!

    runtime: Documentation dependency 'snowballstemmer' not found.
    runtime: Run 'pip install -r Documentation/requirements.txt'
    runtime: Makefile:24: recipe for target 'check-requirements' failed
    runtime: make[1]: *** [check-requirements] Error 2

Did we upgrade the vagrant box?

test-me-please

@tgraf Vagrant box was very recently updated:
5c9efe0c00 (Eloy Coto       2018-03-29 16:00:03 +0200 108)         config.vm.box_version = "46"

test-me-please

test failed:
03:53:42   Updating Cilium stable to master [It]
03:53:42   /home/jenkins/workspace/Cilium-PR-Ginkgo-Tests-Validated/src/github.com/cilium/cilium/test/ginkgo-ext/scopes.go:270
03:53:42
03:53:42   Cannot curl app1-service
03:53:42   Expected
03:53:42       <bool>: false
03:53:42   to be true
03:53:42
03:53:42   /home/jenkins/workspace/Cilium-PR-Ginkgo-Tests-Validated/src/github.com/cilium/cilium/test/k8sT/Updates.go:123
03:53:42 ------------------------------
03:53:42 SSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSS
03:53:42
03:53:42 Summarizing 1 Failure:
03:53:42
03:53:42 [Fail] K8sValidatedUpdates [It] Updating Cilium stable to master
03:53:42 /home/jenkins/workspace/Cilium-PR-Ginkgo-Tests-Validated/src/github.com/cilium/cilium/test/k8sT/Updates.go:123
03:53:42
03:53:42 Ran 1 of 272 Specs in 626.282 seconds
03:53:42 FAIL! -- 0 Passed | 1 Failed | 0 Pending | 271 Skipped --- FAIL: TestTest (626.29s)

Build link: https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/1800/execution/node/59/log/
Will investigate tomorrow; triggering another build for now.

test-me-please


Remove old Symfony exception message expectation

Tests still pass so looks like a good clean up to me. Is this possible because the SF deps. version were bumped?

yep, dropping old symfony is the reason why we could clean this up

Thank you @julienfalque.


Upgrade spack release/v0.17 to VisIt 3.2.2

Is your feature request related to a problem?
Spack's develop branch supports VisIt 3.2.1. An LLNL user wants that on Spack's release/v0.17 branch...they need a python 3 enabled version of VisIt there.
I started a PR to Spack's release/v0.17 branch but then realized its @3.1.1 and quite a bit different from what is on develop for 3.2.1.
Not sure actually what the rules are in Spack for PRs to their release candidate branches. I put in an inquiry about it there.

@markcmiller86 I was looking at the visit package in the spack repo and I noticed that 24 days ago the release version of visit was updated to 3.2.2. The v0.18 release was done 16 days ago. Therefore, 3.2.2 is now in the public release and I believe that it's not possible to update a version in a patch release (v0.17.x) so this is probably resolved as good as it can be.

Ah, I see, you were wanting to confirm it works on our systems.

I believe this has been resolved as much as possible but more by @brugger1's efforts than mine.


fix: don't mutate the bounded text if not updated when submitted

Attempt fix the "undo twice for first time when bounded text properties updated by selecting text explicitly" mentioned in https://github.com/excalidraw/excalidraw/pull/4537
Previous

Now

I noticed a regression:

Create a sticky note and write line long enough to wrap one the next line. Confirm.

Edit again and delete the text, then write a single-line text. Confirm.

The text is misaligned.

I noticed a regression:

Create a sticky note and write line long enough to wrap one the next line. Confirm.
Edit again and delete the text, then write a single-line text. Confirm.
The text is misaligned.

Good catch!. Updated the implementation to cache originalText and comparing with editor value instead. Added spec for this case. Should be fine now.

🚀


Path to vault

path to the vault is currently the location where the executable is ran first. in data.rs, encryption and decryption functions only work when you're in the directory where the vault is.
  pub fn encrypt_directory() -> std::io::Result<()> {
      //! This function was specifically made to encrypt ./vault.tar.gz directory where encrypted data is stored
      //!
      //! ## Usage
      //! ```
      //!     let _ = encrypt_directory()
      //! ```

      println!("Locking data.....");
      let output = Command::new("tar")
          .arg("-czvf")
          .arg("./.vault.tar.gz")
          .arg(VAULT_PATH)
          .output()?;

      if output.status.success() {
          let s = String::from_utf8_lossy(&output.stdout);
          println!("{}", s);
          let output2 = Command::new("gpg")
              .args(["-c", "--no-use-agent", "./.vault.tar.gz"])
              .output()?;
          if !output.status.success() {
              let s = String::from_utf8_lossy(&output2.stderr);
              println!("Error: {}", s);
          } else {
              let _ = Command::new("rm")
                  .args(["-rf", "./.vault", "./.vault.tar.gz"])
                  .output()?;
          }
      } else {
          let s = String::from_utf8_lossy(&output.stderr);
          println!("Error: {}", s);
      }
      Ok(())
  }

  pub fn decrypt_directory() -> std::io::Result<()> {
    //! This function was specifically made to decrypt ./vault.tar.gz directory where encrypted data is stored
    //!
    //! ## Usage
    //! ```
    //!     let _ = decrypt_directory()
    //! ```

    let output = Command::new("gpg").arg("./.vault.tar.gz.gpg").output()?;
    if output.status.success() {
        let s = String::from_utf8_lossy(&output.stdout);
        println!("{}", s);

        let output2 = Command::new("tar")
            .args(["-xf", "./.vault.tar.gz"])
            .output()?;
        if output2.status.success() {
            let s = String::from_utf8_lossy(&output2.stdout);
            println!("{}", s);

            let _ = Command::new("rm")
                .args(["-rf", "./.vault.tar.gz"])
                .output()?;
        } else {
            let s = String::from_utf8_lossy(&output.stderr);
            println!("{}", s);
        }
    } else {
        let s = String::from_utf8_lossy(&output.stderr);
        println!("{}", s);
    }

    Ok(())
}

make path to the vault a constant accessible to all scripts in src. in data.rs functions for encryption and decryption  of the vault should work with path to vault being an absolute path.
in prompt.rs create a function vault_path() which gets the username from the terminal and  returns the path to the vault, path to vault should be something like  /home/$username/.vault.
vault_path() will be used in all other script to access the vault

Hello @db-keli
I just dropped a PR for this issue. https://github.com/db-keli/kelvin/pull/14
Can you check it out, please?

Fixed by @Nathy-bajo


added getAdversitingId

@aaustin  Tested

Whoa nice! Works in both instances?

Yes. I added Google Play services SDK (the package required for advertising id) to the Branchster project, ran it on your Samsung, got the advertising id.
When running the TestBed which doesn't have the google play services SDK, it just returns null.

:+1:

Just looked at the backend PR @qinweigong, which looks for key 'google_advertising_id'. This PR submits 'advertising_id'

seems like there's a disconnect here?

This PR wasn't coupled with the backend PR. Please see today's Android PR https://github.com/BranchMetrics/Branch-Android-SDK/pull/45, which is in line with the backend PR, both using "google_advertising_id".

I guess I don't understand why we needed anything else done in Android after this PR

The change in this PR was about retrieving the google advertising id (if Google Play service SDK is imported to the project) by ourselves.
Today's new PR is about accepting the google advertising id the partners retrieve by themselves and pass it to us in initSession call. This is per what you told me in Slack yesterday. Am I having misunderstanding here?

Maybe I should've just add today's change to this PR instead of creating another PR.

Ahhh - no - this was miscommunication. Sorry! Partners would never pass us Google Advertising ID via init since we can just read it directly from the Play Services framework. I think we've got it all covered in the last PR - can you just make sure the keys match server/SDK?

Oh, sure!


Report no valid files retrieved as info and continue.

    // TODO: Report no valid files retrieved as info and continue.

In file lib/skelo-utils.js
https://github.com/weblidity/skelo/blob/d5023fe7816567390d04bc9bcaa18abe0cf4227d/lib/skelo-utils.js#L75

No longer an issue


Allow test ratio to zero in WildFireSplitter

This PR aims to answer the issue : [DataSet] Allow test ratio to zero in WildFireSplitter #61
As explain in the issue :
At the moment the input ratios of WildFireSplitter cannot be equal to zero. I would like to add this option. Indeed we are going to do a lot of tests with different extraction methods (number of frames, spacing between frames ...). As the frame extraction is long I think we will save a lot of time if we don't have to do it every time for the test set.
PS: I have also commented the warinig
"Labeling. Maybe try to label the fire again
Distinct values of ids(340 ≠ 420)
f" ≠ {wildfire.metadata['fire_id'].max() + 1})", Warning)"

We haven't annotated the entire Dataset so we're going to get this warning every time.

Hi @MateoLostanlen,
Thanks for the PR! Is this ready for review?
If so, add some reviewers to it or we don't get notified :)

Yes it's ready :)


When order total is less than 0 no payment is made by mollie. So no need to…

When order total is less than 0 no payment is made by Mollie. So no need to check for active mandate.
We're using Cashier in a project where we currently support businesses to sell digital gift cards to support them during the Corona lock-down. These customers have a free account without a mandate (no need, since we'll pay them instead of they pay us).
The cashier package is still really useful l because of calculating transaction costs, invoicing etc. We just payout the "credit" amount that is calculated by cashier. It works really well for this.
However, this particular type of client does not have a mandate and we're not going to ask them to create one. This PR checks if the total order amount is 0 or less and if so, it won't throw an error when no mandate is set.
This will allow our particular client type and won't introduce breaking changes. I really hope you're accepting this PR!

I'm ok with it, just add some decent tests to OrderTest.
You probably will have problems running the rest of the OrderTest methods without the required secrets for Mollie. Just focus on your specific methods, I expect yours will not be interacting with Mollie.
PS. Sounds like a great initiative. 👍

Thanks @sandervanhooft
And yes, the reason I didn't create any tests is that I couldn't figure out why I can't get them to pass (without my changes). I tried to paste in my mollie test key but even then I couldn't get all of them to pass.
I'll give it another shot to test my changes. If you can give me any pointers on how I can make all tests pass I'd appreciate it. Also for maybe some future PR's. I love to see all tests green before I commit :-)

There is a test instruction at the end of the readme, but it seems two secrets are missing:
MANDATE_PAYMENT_PAID_ID
SUBSCRIPTION_MANDATE_PAYMENT_PAID_ID
I'll look into drafting a test manual with some more details.

Thanks @sandervanhooft!
I think I get it now, the key names are very descriptive. I'll create all the mandates, payments etc and will then enter the values in phpunit.xml.
I'll try to create the tests this evening or tomorrow morning. Thanks again for your time!

LGTM!

@stefro I'll try to release this tonight so you don't have to wait until next week.
Will be released as a patch (1.12.x).

Thanks for merging!

No problem. And it's also shaving off a few calls to the Mollie API. Everybody wins 😄 .

@stefo Released in 1.12.2 . (More info)

@sandervanhooft @stefro I believe there's still an issue with this solution. Sure it does solve that a mandate guard is not needed when the total order amount is negative. But how about orders that are being paid by credit, they also don't need to guard for a mandate, right?
Currently the mandate check happens before the processing of the billable's credit:
        $minimumPaymentAmount = $this->ensureValidMandateAndMinimumPaymentAmountWhenTotalPositive();
        $this->update(['mollie_payment_id' => 'temp_'.Str::uuid()]);

        DB::transaction(function () use ($minimumPaymentAmount) {
            $owner = $this->owner;

            // Process user balance, if any
            if($this->getTotal()->getAmount() > 0 && $owner->hasCredit($this->currency)) {
                $total = $this->getTotal();
                $this->balance_before = $owner->credit($this->currency)->value;

                $creditUsed = $owner->maxOutCredit($total);
                $this->credit_used = (int) $creditUsed->getAmount();
                $this->total_due = $total->subtract($creditUsed)->getAmount();
            }

            $totalDue = money($this->total_due, $this->currency);

        ......

Shouldn't we move the $minimumPaymentAmount = ... inside of the transaction, after $this->total_due has been calculated and base it upon this total_due?

@lexdewilligen Yes, that makes sense to me!


can not hijack a job

when i run
fly -t test  hijack  -j api-dev/build-api-image it returns a list of containers i can hijack but when i choose one it just sits there. eventually dying with the error.
websocket: unexpected reserved bits 0x40
everything else works, i just can't hijack a job.

I need help replicating this as fly intercept is working as expected on our Concourse.
Are you still experiencing this issue?
What version of concourse-up are you running?
Does it fail connecting to all containers?
What image is running on the containers that are producing this error?
Do you have any firewalls/proxies between you and the Concourse?

I'm closing this due to inactivity. If you're still having an issue on the latest version of concourse-up please feel free to open another issue.


Save map position in redux to restore when needed

Closes #113

I think that selectedFeature is like the focus of an input, when you navigate it is lost and is ok. So it should be forgotten.


what's the performance comparison between apollo 8.0 with apollo 9.0 or apollo 10.0

We appreciate you go through Apollo documentations and search previous issues before creating an new one. If neither of the sources helped you with your issues, please report the issue using the following form. Please note missing info can delay the response time.
System information

OS Platform and Distribution (e.g., Linux Ubuntu 18.04):
Apollo installed from (source or binary):
Apollo version (3.5, 5.0, 5.5, 6.0):
Output of apollo.sh config if on master branch:
I understand that in Apollo 8.0 and earlier versions, the perception module operates through a pipeline, with functions communicating via parameter passing .
In Apollo 9.0 and 10.0, the perception module's components communicate using cyber::Writer.
I want to know whether communicating using cyber::Writer is slower than passing parameters through functions.
Are there any performance comparison indicators？

Steps to reproduce the issue:

Please use bullet points and include as much details as possible:

Supporting materials (screenshots, command lines, code/script snippets):

Just for CyberRT perfomance between Apollo10.0 and Apollo 9.0
CyberRT Performance Report

I want to know whether communicating using cyber::Writer is slower than passing parameters through functions

these two are not in the same dimension.
cyber document:  https://github.com/ApolloAuto/apollo/blob/master/cyber/README.md


Use with cert manager

I am looking to get this working with cert manager so that the certificate keys dont need to be generated manually before hand. What is the best way to do this?

That's the way we do it as well, for example, this is how we have CertManager create certs for one of our own instances:
apiVersion: cert-manager.io/v1alpha3
kind: Certificate
metadata:
  annotations:
    meta.helm.sh/release-name: RELEASE
    meta.helm.sh/release-namespace: example
  creationTimestamp: "2020-05-18T17:45:50Z"
  generation: 1
  labels:
    app.kubernetes.io/managed-by: Helm
    cluster-name: RELEASE
  name: RELEASE-cert-postgresql
  namespace: example
  resourceVersion: "669303993"
  selfLink: /apis/cert-manager.io/v1alpha3/namespaces/example/certificates/RELEASE-cert-postgresql
  uid: d73244fb-b380-42ff-abc8-33e7609fc9fd
spec:
  dnsNames:
  - RELEASE
  - RELEASE.example
  - RELEASE.example.svc.cluster.local
  duration: 8760h0m0s
  ipAddresses:
  - <IP_ADDRESS>
  issuerRef:
    group: cert-manager.io
    kind: Issuer
    name: example-self-signer
  keyAlgorithm: rsa
  keyEncoding: pkcs1
  keySize: 4096
  renewBefore: 720h0m0s
  secretName: RELEASE-cert-postgresql
  subject:
    organizations:
    - Timescale, Inc.

You can then point to the resulting secret by setting .Values.secretNames.certificate in values.yaml.


jquery version download

as you linked jquery demo , cant find out how to download files!

@0wczar do you plan to publish the jquery demo? we are making some changes and would like to push that into the jquery version


change: y-final from U+1540 to U+1429

ᑖᓂᓯ!
This is just to change the Y-final. I had also though it was U+1540, but after I did an informal survey (asking Cree writers), they almost unanimously prefered the way U+1429 CANADIAN SYLLABICS FINAL PLUS renders; in most fonts, it renders like the other finals, whereas U+1540 renders in the center of the text and is the only final to do so.
Just FYI, I've come up with a "definitive" list of syllabics for writing in ᓀᐦᐃᔭᐍᐏᐣ! https://github.com/UAlbertaALTLab/nehiyawewin-syllabics

What's the NARROW NO-BREAK SPACE used for?

It's used as a morpheme separator by a few very particular writers; not everybody uses it, but those that do use it REALLY want it! It's a space that's half as wide as a normal space, but importantly, it does NOT break words! This is used in, e.g., «ᑭᑲ ᐚᐸᒥᑎᐣ» ("I will see you") to separate the gramatical morphemes «ᑭᑲ» from the rest of the word.

I'm looking forward to working with you more!

Likewise 😄

Interesting! Do other writers use hyphens (like this SRO passage I got from this Wikipedia page), or just write them as a single word, or both?

tl;dr: both!
Some people use hyphens in SRO, as dictated in How to Spell it in Cree. It helps for writing long words with multiple preverbs like ninôhtênêhiyawi-kiskinwahamâkosin.
However, some writers skip the hyphens, especially in shorter words like ê-wâpamât/ewâpamât. Additionally, the diacritic on the ê is omitted by some writers, since there is no such *e grapheme in "strict" SRO!


Remove Analog Scale widget?

The analog scale widget was developed for a particular clinical trial use case (for OpenClinica). It has to rigidly look a particular way in a particular form to be certified for certain trials. I think my rationale for putting it in Enketo Core (long ago) was that others could perhaps benefit from it.
However, likely nobody outside OpenClinica knows about it. It was never added to the default widgets configuration in Enketo Express either (and not to ODK Collect).
It might be more efficient for OpenClinica to add/move this widget to their fork of Enketo Express and continue working on it there.
Would anybody have any objections to removing this widget from Enketo Core (to avoid confusion)? Or is anybody excited about this potential code cleanup?

I think it makes sense to maintain separately!


【Feature】Pods created by cloudtty should follow the namespace owned by the user.

Overview
The current cloudtty pods are by default created in the default namespace (default ns), which can lead to the following issues:

Permission leakage: If a user does not have permission to access the default namespace, but attempts to run cloudtty pods in the default ns, it may result in permission leakage and pose security risks.
Insufficient resources in the default namespace: Since all cloudtty pods are running in the default ns, it may cause resource scarcity issues, especially in clusters with limited resources. This can potentially disrupt the normal operation of user applications.

Recommendations

Create a dedicated cloudtty namespace (cloudtty ns) specifically for running cloudtty pods. However, there may still be permission leakage concerns even with this approach.
Ensure cloudtty runs in namespaces with appropriate permissions:

For users with full namespace permissions across the cluster, it is recommended to place the cloudtty pods in the default namespace.
For users with partial namespace permissions or permissions limited to a single namespace, it is suggested to run the cloudtty pods in the namespace where the current operation is being executed.

您好，您发的邮件我已经收到了，我会尽快查看的，么么哒。


MF-1603 - Env file contains an invalid value

What does this do?
This pull request fixes the .env file.
Which issue(s) does this PR fix/relate to?
Resolve #1603
List any changes that modify/break current functionality
N/A
Have you included tests for your changes?
N/A
Did you document any new/modified functionality?
N/A
Notes

Please sign off your commits.

@zhang-chuanfeng Please update your branch and we can merge

Codecov Report

Merging #1604 (e75c2bb) into master (d30e6ba) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1604      +/-   ##
==========================================
+ Coverage   68.93%   68.95%   +0.01%
==========================================
  Files         136      136
  Lines       11058    11058
==========================================
+ Hits         7623     7625       +2
+ Misses       2788     2786       -2
  Partials      647      647

Impacted Files
Coverage Δ

things/service.go
66.07% <0.00%> (+0.71%)
:arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

Thank you @zhang-chuanfeng !


question

is there a way how i can set it up with my account name? such as it being https://daveied.github.io/webretro/ just incase my school blocks your website i can just fork it and make my own website from there

Yes: I'm in a hurry to catch a bus right now, but if you'll give me 20 minutes I can write the instructions down.

^


I can't get it to read the file continuously, putting each new line in the Topic

I have FilePulse correctly configured, so that when I create a file inside the reading folder, it reads it and ingests it in the topic.
Now I need to do continuous reading of each of the files in that folder, since they are continually being updated.
I have to change any property of properties file?
My filePulseTxtFile.properties:
name=connect-file-pulse-txt
connector.class=io.streamthoughts.kafka.connect.filepulse.source.FilePulseSourceConnector
topic=lineas-fichero
tasks.max=1
#File types
fs.scan.filters=io.streamthoughts.kafka.connect.filepulse.scanner.local.filter.RegexFileListFilter
file.filter.regex.pattern=.*\.log$
task.reader.class=io.streamthoughts.kafka.connect.filepulse.reader.RowFileInputReader
#File scanning
fs.cleanup.policy.class=io.streamthoughts.kafka.connect.filepulse.clean.LogCleanupPolicy
fs.scanner.class=io.streamthoughts.kafka.connect.filepulse.scanner.local.LocalFSDirectoryWalker
fs.scan.directory.path=/home/ec2-user/parser/scanDirKafka
fs.scan.interval.ms=10000
#Internal Reporting
internal.kafka.reporter.bootstrap.servers=localhost:9092
internal.kafka.reporter.id=connect-file-pulse-txt
internal.kafka.reporter.topic=connect-file-pulse-status
#Track file by name
offset.strategy=name
Thanks a lot!

Hi @guilleGH
For doing continious reading you need to add this property read.max.wait.ms which is the maximum time (in milliseconds ) the RowFileInputReader will wait for more bytes after hitting end of file.
In addition, if new files are added to the input directory they will not be processed because tasks are still running for previous files. So, you can configure the allow.tasks.reconfiguration.after.timeout.ms to force all tasks to be restarted after a given period.

Hi @guilleGH
For doing continious reading you need to add this property read.max.wait.ms which is the maximum time (in milliseconds ) the RowFileInputReader will wait for more bytes after hitting end of file.
In addition, if new files are added to the input directory they will not be processed because tasks are still running for previous files. So, you can configure the allow.tasks.reconfiguration.after.timeout.ms to force all tasks to be restarted after a given period.

thank you very much, it works

thank you very much, it works


Card component

Description
Adding in the three new card variants that are used on nhs.uk and have tested well, these are the card with an arrow, transparent card, and top task.

https://github.com/nhsuk/nhsuk-service-manual-community-backlog/issues/254
https://github.com/nhsuk/nhsuk-service-manual-community-backlog/issues/472

Checklist

[x] Tested against our testing policy (Resolution, Browser & Accessibility)
[x] Follows our coding standards and style guide
[x] CHANGELOG entry

All looks good to me 👍

Changelog looks good to me 👍


Video browser stuck in loading v3

Describe the bug
Locally the video browser works but when deployed to a Sanity studio the loading spinner is stuck and videos are not displayed, this is for both choosing a video from a field and from the tabs at the top of the studio with the path /mux.
Videos will upload and display on the front end and can query data, but they dont show in the studio in the browser/select window.
Ive tried different API keys, signed URLs but nothing works.
To Reproduce
Install and deploy a clean sanity project e.g. https://github.com/sanity-io/sanity-template-nextjs-clean with mux plugin and deploy to a Sanity studio.
Expected behavior
Videos load
Screenshots

Which versions of Sanity are you using?
@sanity/cli (global)          3.60.0 (latest: 3.68.3)
@sanity/assist                 3.0.8 (latest: 3.0.9)
@sanity/eslint-config-studio   4.0.0 (latest: 5.0.1)
@sanity/icons                  3.5.5 (up to date)
@sanity/vision                3.68.3 (up to date)
sanity                        3.68.3 (up to date)
What operating system are you using?
Which versions of Node.js / npm are you running?
10.1.0
v20.8.0

@jackgregory exactly the same issue here!

I have a same problem. But when I open console i saw this problem cors.


Observable.fromInputStream emits bytes whose length is less than the chunkSize

Not sure if this is a bug or is intended, but using Observable.fromInputStream (and its variants) emits bytes whose length is less than the chunkSize in a situation like below:

InputStream actually has more bytes to stream than the chunkSize
InputStream.read results in bytes streamed less than the chunkSize in monix.reactive.internal.builders.InputStreamObservable (an internal Monix class)

Coming from the standpoint of the official API of InputStream.read, nothing is violated:

public int read(byte[] b)
throws IOException

Reads some number of bytes from the input stream and stores them into the buffer array b. The number of bytes actually read is returned as an integer. This method blocks until input data is available, end of file is detected, or an exception is thrown.

If the length of b is zero, then no bytes are read and 0 is returned; otherwise, there is an attempt to read at least one byte. If no byte is available because the stream is at the end of the file, the value -1 is returned; otherwise, at least one byte is read and stored into b.

This contract has existed for a long time and is an accepted fact.
However, there are situations when one wants to fill the given buffer even though InputStream.read cannot - by continuously calling it until the buffer is fiiled - or - the stream is exhausted.
Now, if the original intention of Observable.fromInputStream is to fill the chunkSize, there needs to be made some fixes to monix.reactive.internal.builders.InputStreamObservable to continuously read from the InputStream until the buffer is filled or the stream ends - otherwise, I am suggesting a variant of this method that does so.
As a motivating example, I was trying to download a 10 Gb file from a Hadoop HDFS file system and split it into size 1 GB each, so I used Observable.fromInputStream(Task(<hdfsInputStream>), chunkSize = 1e9), but each emission from Observable.fromInputStream is actually only of size 65KB instead of chunkSize (1 GB). I had to roll out my own InputStreamObservable implementation to continuously read from the InputStream until it hits the chunkSize given.
Thoughts?

I am now pretty convinced this is a bug. Reading from an input stream must be done in a loop, otherwise it won't fill the given buffer, rendering it pointless.

I agree it is surprising behavior and could be improved but I need to brush up on InputStream to be sure
Could you show your implementation?

Sure. In InputStreamObservable.fastLoop, the part where it reads the input stream:
val length = blocking(fillBuffer(in, buffer)) // instead of the old in.read...
In fillBuffer:
@tailrec
  private def fillBuffer(in: InputStream, buffer: Array[Byte], bytesRead: Int = 0): Int = {
    if (bytesRead >= buffer.length) bytesRead                 // buffer is filled
    else {
      val batchRead = in.read(buffer, bytesRead, buffer.length - bytesRead)
      if (batchRead > 0)                 // stream is not exhausted
                                         // and we read something, recurse to fill the buffer
        fillBuffer(in, buffer, bytesRead + batchRead)
      else {                            // stream has ended
        if (bytesRead <= 0)             // we read nothing, just return an EOF
          -1                            // stops the observable
        else                           // we read something
          bytesRead                    // return the bytes we read first
      }
    }
  }

I did recursion because I think it was cool, but a normal while loop would also suffice. I am also not sure if it is best to use nested if-else for readability or just use pattern matching on the ints.
I hope it illustrates the point, though.

@Avasil I have time to do this, submitting a PR.
I used a property test to validate - please review when you have time.


Update /350cup and /tiershift

I'm not entirely sure why redirect to /data, but only sometimes? What was wrong with old code, this just seems more complicated for no real reason.

/tiershift does need to be updated, but I don't think delegating to /data is the right choice.

@Zarel @TheImmortal

Bumping again @Zarel @TheImmortal


head: canonical link only if required

Currently the canonical link is set on EVERY page, which is not required. It makes sense to add canonical only to pages that have multiple URLs (including forwards) that they can be reached at.

non-issue.


Fixed #35689 -- Added init method for LabelCommand.

Trac ticket number

ticket_35689
Branch description
The missing_args_message attribute of the django.core.management.base.LabelCommand is hard-coded using the default value of label = "label".
Checklist

[x] This PR targets the main branch.
[x] The commit message is written in past tense, mentions the ticket number, and ends with a period.
[x] I have checked the "Has patch" ticket flag in the Trac system.
[x] I have added or updated relevant tests.
[x] I have added or updated relevant docs, including release notes if applicable.
[x] I have attached screenshots in both light and dark modes for any UI changes.

@gfabbretti8 Thanks for implementing my ticket!
@sarahboyce Should we take care of the case where label = None? I like defining None as the way to disable things. In this case, this would disable the custom message and use the one for the parser... In fact, BaseCommand takes missing_args_message = None as the default.

@sarahboyce Should we take care of the case where label = None? I like defining None as the way to disable things. In this case, this would disable the custom message and use the one for the parser... In fact, BaseCommand takes missing_args_message = None as the default.

Yes you're right, we should handle these (with tests) and missing_args_message having a custom message (as someone might have hard coded this given it's current issues)
@gfabbretti8 do you mind making some adjustments?

Yes @sarahboyce, I can make the changes, but I need a couple of clarifications.
If I got things right, as per @paduszyk suggestion's, we would like for the 'missing_args_message' to be 'None' when 'label=None'.
class LabelCommand(BaseCommand):
    ...
    label = "label"
    missing_args_message = "Enter at least one %s."

    def __init__(self, *args, **kwargs):
        super().__init__(*args, **kwargs)

        if self.label == None:
            self.missing_args_message = None
        else:
            self.missing_args_message = self.missing_args_message % self.label

Something like should be ok then, right?
The second question I have is about testing. It seems to me that there is no class in django extending class LabelCommand with label=None. Then, what is the correct way to test this? Should I create a dummy class inside a test to verify that when label=None then also missing_args_message is None or there is a best way to proceed?

Here is an example test, we will also want a test for when missing_args_message = None.
--- a/tests/admin_scripts/tests.py
+++ b/tests/admin_scripts/tests.py
@@ -25,6 +25,7 @@ from django.core.management import (
     color,
     execute_from_command_line,
 )
+from django.core.management.base import LabelCommand
 from django.core.management.commands.loaddata import Command as LoaddataCommand
 from django.core.management.commands.runserver import Command as RunserverCommand
 from django.core.management.commands.testserver import Command as TestserverCommand
@@ -2262,6 +2263,13 @@ class CommandTypes(AdminScriptTestCase):
         out, err = self.run_manage(args)
         self.assertOutput(err, "Enter at least one label")

+    def test_custom_label_command_custom_missing_args_message(self):
+        class Command(LabelCommand):
+            missing_args_message = "Missing argument."
+
+        with self.assertRaisesMessage(CommandError, "Error: Missing argument."):
+            call_command(Command())
+
     def test_label_command_multiple_label(self):

This currently fails with a TypeError. We can fix it with something like this:
if self.missing_args_message == LabelCommand.missing_args_message:
    self.missing_args_message = self.missing_args_message % self.label

As self.label is used in more than just this error, I don't think we need to worry about if the label is set to None :thinking:


Parsing Error in "ATTENDEE" line

Hi, the following link fails to parse:
vobject.base.ParseError: At line 12: Failed to parse line:
ATTENDEE;RSVP=TRUE;CN=John Doe;PARTSTAT=NEEDS-ACTION;ROLE=REQ-PARTICIPANT;SCHEDULE-STATUS="3.7;Invalid Calendar User";INVALID CALENDAR<EMAIL_ADDRESS>

It looks like you're missing a double-quote somewhere.

Maybe. But this is a read-life example.

What do you mean it's a "read-life example"? The property is incorrectly formatted, it correctly throws a ParseError.

Well, IMHO the parser needs to be resilient against this kind of errors. The event was generated by some software – wich might be buggy, but insisting on this does not help the user having this event.


[Suggestion] Time Line

Hello. Suggestion to add a Timeline page like Nostalrius had and all the other vanilla servers :)

This feature has been added, please refer to: https://github.com/yesilmen-vm/YesilCMS/commit/717f2ced29d51e7697577555058817437255d8d8
Demo: https://yesilcms.page/en/timeline


Update Song “2023-04-17/index”

Automatically generated by Netlify CMS

👷 Deploy Preview for gleeful-crepe-2139ca processing.

Name
Link

🔨 Latest commit
84863caf2305fa5fd9c88774e8d836ce4dd5b6b0

🔍 Latest deploy log
https://app.netlify.com/sites/gleeful-crepe-2139ca/deploys/6543e3a3bc73960008e8b4e0


Increase padding between top of onboarding screen content to top app bar

Build 227:

In design:

Looks good


[stable/oauth2-proxy] fix PodDisruptionBudget.

Also change homepage.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

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

/assign @Miouge1

For reference, see https://github.com/helm/charts/pull/18073

Looks good, but it would have been nice to do /ok-to-test to see the CI values file in action (I didn't know that that was possible!)

Bot can't merge until tests pass
And since we are collaborator, we do not need ok-to-test when we create a PR! Small privileges. ;)

OK got it, so it's just that the circleci output doesn't show up in the "checks" tab, but I tracked down the CI output to here: https://prow.k8s.io/view/gcs/kubernetes-jenkins/pr-logs/pull/helm_charts/19394/ :)
Cool stuff!


Add encrypted data to vanchor event

Summary of changes
Changes introduced in this pull request:
Reference issue to close (if applicable)

Closes #171

LGTM if you think this is the way we can merge.

yes, it does save a step so let's split it.


Add View tree traversal JQuery inspired assertions

Was just writing a test and really wanted to write something like:
assertThat(viewGroup).has(textView).withText("fest!");

or perhaps
assertThat(viewGroup).has(TextView.class).withText("fest!");

is better design.

We have no plans for this. It's a fundamental deviation from the core principles of this library (which is itself merely a platform adapter for another library not offering anything along these lines).


Cannot run the code because the feature size is incorrect when training or testing

Traceback (most recent call last):
File "/home/NerVE-main/network/grid.py", line 73, in forward
feature_grid = self.grid_feat(temp_grid).permute((1,2,3,0))
RuntimeError: Expected 5-dimensional input for 5-dimensional weight [128, 128, 3, 3, 3], but got 4-dimensional input of size [128, 64, 64, 64] instead

I encountered the same problem when running demos. Hope for a solution.


PayPal button won't start incontext checkout with Compatibility view mode and X-UA-Compatible

Description
PayPal Checkout button is shown but won’t when clicked work with Compatibility view mode and X-UA-Compatible, showing this message on console. I waited for a minute, but I still get the same.

paypal.version: 4.0.235

IE11

The site that shows PayPal button is added in Compatibility View in IE11.

The site has <meta http-equiv=“X-UA-Compatible” content=“IE=edge” />

paypal.isEligible() = true

console logs
ppxo_button_pre_template_click {“timestamp”:1542788763503,“windowID”:“a8f1ad8947”,“pageID”:“47cf83f4ac”,“prev_corr_ids”:“”,“referer”:“developer.paypal.com”,“host”:“developer.paypal.com”,“path”:“/demo/checkout/“,”env”:“production”,“country”:“US”,“lang”:“en”,“uid”:“0060ed9143_ga4dumrshi2dk”,“ver”:“4.0.235”}

The issue also occurs at https://developer.paypal.com/demo/checkout/#/pattern/client.

The issue occurs consistently.

Steps to reproduce

Add the site that shows PayPal button in Compatibility View in IE11.
Add <meta http-equiv=“X-UA-Compatible” content=“IE=edge” />
Load the site, you will see the PayPal button, but when the button is clicked, nothing happens.

Affected browsers

[ ] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge
[x ] IE
[ ] Chrome Mobile/Tablet
[ ] Safari Mobile/Tablet
[ ] Web View / Safari ViewController
[ ] Other

Hey @sshimizupp, the fact that the pop-up never happens makes me suspect you might be trying to render the button in Intranet Mode. Could you please check this page out and see if it helps you?
https://github.com/paypal/paypal-checkout/blob/master/docs/debugging/ie-intranet.md

Hi @mcntsh, I'm aware of this intranet mode issue and this time is a different issue. The site is not in Intranet mode but added to Compatibility view mode.  I understand we don't support Compatibility view mode as IE works in IE7 mode, but when there is meta tag X-UA-Compatible IE=edge, it should disable Compatibility view mode, but when I click the button I see ppxo_button_pre_template_click. Seems like some necessary codes would not be loaded.
Below is for Classic NVP integration and it is old, but we recommended adding this meta tag then. Could you please check if this is still true?
https://developer.paypal.com/docs/classic/express-checkout/in-context/javascript_advanced_settings/#browser-support
Thanks,

If the site is in compatibility mode, there is no way for the PayPal button to work. Which mode do you see being enabled in the IE11 developer console?

Even when in Edge mode, I see the issue.

OK I was able to repro just now. Even though IE dev-tools shows "Edge", the button is still being loaded with an IE7 user-agent. So this is unlikely to be something we're going to fix. We would just recommend avoiding setting compatibility mode for any site where you're rending the paypal button.


Can skywalking ClassMatch support regex classname match?

Is there a way to match classes using regex expression like spring aop does?
what I wanted
  protected ClassMatch enhanceClass() {
        return new RegClassMatch("com.xxx.*Dao");
  }

by the way, all the dao classes didn't implement a common parent class, so I cannot use HierarchyMatch to match all the dao classes.

You could, just implement a new ClassMatch, which is based on byte-buddy API. I think we are also welcome you contribute this.

I implement a classmatch , but no class was instrumented.
public class RegClassNameMatch implements IndirectMatch {

    private List<String> matchClassNames;
    private Pattern pattern;

    public RegClassNameMatch(String regex) {
        if (regex == null) {
            throw new IllegalArgumentException("reg class names is null");
        }
        pattern = Pattern.compile(regex);
        matchClassNames = new ArrayList<String>();
    }

    @Override
    public ElementMatcher.Junction buildJunction() {
        ElementMatcher.Junction junction = null;
        for (String name : matchClassNames) {
            if (junction == null) {
                junction = named(name);
            } else {
                junction = junction.or(named(name));
            }
        }
        return junction;
    }

    @Override
    public boolean isMatch(TypeDescription typeDescription) {
        Matcher matcher = pattern.matcher(typeDescription.getTypeName());
        if (matcher.matches()) {
            matchClassNames.add(typeDescription.getTypeName());
            return true;
        }
        return false;
    }
}

Skywalking execute buildJunction first and then isMatch , therefore matchClassNames is empty. In my guess, method buildJunction() is responsible to match the class(cause I didnot find doc on the method).
Am I doing this right?
Any suggestion on how to implement buildJunction()?

buildJunction take charge for this. You need to look byte-buddy document to understand how to build it.


Allow a dash(-) at the first character of a plain scalar

According to the YAML spec, a dash(-) at the first character of a scalar is allowed. link
However, fkYAML didn't support such cases and threw an exception, e.g., with the following input:
foo: -bar

So, this PR has fixed the issue and the fix has also been tested by the added test cases.

Pull Request Checklist
Read the CONTRIBUTING.md file for detailed information.

[x] Changes are described in the pull request or in a referenced issue.
[x] The test suite compiles and runs without any error.
[x] The code coverage on your branch is 100%.
[x] The documentation is updated if you added/changed a feature.

Please don't

The C++11 support varies between different compilers and versions. Please note the list of supported compilers. Some compilers like GCC 4.7 (and earlier), Clang 3.3 (and earlier), or Microsoft Visual Studio 13.0 and earlier are known not to work due to missing or incomplete C++11 support. Please refrain from proposing changes that work around these compiler's limitations with #ifdefs or other means.
Please refrain from proposing changes that would break YAML specifications. If you propose a conformant extension of YAML to be supported by the library, please motivate this extension.
Please do not open pull requests that address multiple issues.

coverage: 100.0%. remained the same
when pulling 1be03fdf19bc73f7b4e1f229be0395445d56a8c5 on bugfix/allow_dash_as_1st_plain_scalar_char
into dc4e22d61dc2433e31be837476523b5ce413be28 on develop.


[docs] Added missing queries parameter to batch endpoint

Related to spotify/heroic#150

Great stuff, thanks!


Update chart victoria-metrics-k8s-stack to 0.21.0

This PR contains the following updates:

Package
Update
Change

victoria-metrics-k8s-stack
minor
0.20.1 -> 0.21.0

[!WARNING]
Some dependencies could not be looked up. Check the warning logs for more information.

Release Notes

VictoriaMetrics/helm-charts (victoria-metrics-k8s-stack)
v0.21.0
Compare Source
Release notes for version 0.21.0
Release date: 2024-04-18

TODO

bump version of VM operator to 0.43.0

updates CRDs definitions.

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

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

This PR has been generated by Renovate Bot.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.


Bigtable write samples

Looks like there are linting errors (bundle exec rubocop) causing the presubmit failure.

Looks like there are linting errors (bundle exec rubocop) causing the presubmit failure.

Yup, fixed it :)

@jsimonweb Hey could you take a look at this if you have time or assign it to someone? thanks

FYI, google-cloud-bigtable/v0.5.0 has been released which allows omitted or nil GcRule arguments.


Look like bootstrap dropdown option for single selection select2

Hi,
Can you guys add support to look the select2 box similar to bootstrap dropdown for single selection select2 box?
I just removed the border for select2 and added border and border radius for dropdown box class in select2 css. Its nearly similar to bootstrap dropdown. But only issue is the down and up arrow next to the selected item moved to the right. Pl have a look at the attached image.

Is there any changes that I can apply to move the arrow next to the selected Item?

There is a Bootstrap theme available for Select2 that makes it look like a standard <select>: https://github.com/select2/select2-bootstrap-theme


[New game]: Sci-Fi Alchemy

🎮 Game Request
Sci-Fi Alchemy is all about science and fiction game.Can you find all the elements?
Point down the features
Game points will be awarded on the basis of elements found
Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @S-ishita !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹


Failures with imported modifiers and helpers repro

I recently updated to V4 of ember-template-imports, but have found widespread breakages across our app as a result.
It seems like the template compilation process isn't dealing properly with imported functions and other components, but plain classes seem to be ok.
This PR just adds failing tests to demonstrate the problem, I'm unsure how to fix, but happy to give it some time.
Screenshot from our CI

Something weird is going on because it's failing when running locally, but not in CI 🤔 (for the repro)

Sorry, super weird, definitely failing locally

I'm seeing the same thing with "ember-cli-babel": "^8.2.0" and "ember-template-imports": "^4.0.0".

I'm using<EMAIL_ADDRESS>as well

@HeroicEric are you also using embroider?

@NullVoxPopuli no.On Oct 20, 2023, at 6:48 AM, NullVoxPopuli @.***> wrote:
@HeroicEric are you also using embroider?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

@NullVoxPopuli no

more minimal repro here: https://github.com/ember-template-imports/ember-template-imports/pull/211/files#diff-99353e7756ea20bb4fdcc1b1067c0f004b188d13df85162e6ef34b074dd97a2cR26
gonna close this.
issue: https://github.com/ember-template-imports/ember-template-imports/issues/207
thanks for the report!


DeviceParamFixture.DeviceInitializeAndTeardown ND hang on N300

This is ND and can be reproduced by repeating some tests:
./build/test/tt_metal/unit_tests_fast_dispatch --gtest_filter=*DeviceParamFixture* --gtest_repeat=-1 --gtest_break_on_failure
Repeating all tests (iteration 34) . . .

Note: Google Test filter = *DeviceParamFixture*
[==========] Running 4 tests from 1 test suite.
[----------] Global test environment set-up.
[----------] 4 tests from DeviceInit/DeviceParamFixture
[ RUN      ] DeviceInit/DeviceParamFixture.DeviceInitializeAndTeardown/0
                  Metal | INFO     | Initializing device 0. Program cache is NOT enabled
                  Metal | INFO     | AI CLK for device 0 is:   1000 MHz
                  Metal | INFO     | MMIO Device 0 : Tunnel 0 : Device 0
                  Metal | INFO     | MMIO Device 0 : Tunnel 0 : Device 1
                  Metal | INFO     | Closing device 0
                  Metal | INFO     | Disabling and clearing program cache on device 0
[       OK ] DeviceInit/DeviceParamFixture.DeviceInitializeAndTeardown/0 (25 ms)
[ RUN      ] DeviceInit/DeviceParamFixture.DeviceInitializeAndTeardown/1
                  Metal | INFO     | Initializing device 0. Program cache is NOT enabled
                  Metal | INFO     | AI CLK for device 0 is:   1000 MHz
                  Metal | INFO     | Initializing device 1. Program cache is NOT enabled
                  Metal | INFO     | AI CLK for device 1 is:   1000 MHz
                  Metal | INFO     | MMIO Device 0 : Tunnel 0 : Device 0
                  Metal | INFO     | MMIO Device 0 : Tunnel 0 : Device 1
                  Metal | INFO     | Closing device 0
                  Metal | INFO     | Disabling and clearing program cache on device 0
                  Metal | INFO     | Closing device 1

Repeating all tests (iteration 23) . . .

Note: Google Test filter = *DeviceParamFixture*
[==========] Running 4 tests from 1 test suite.
[----------] Global test environment set-up.
[----------] 4 tests from DeviceInit/DeviceParamFixture
[ RUN      ] DeviceInit/DeviceParamFixture.DeviceInitializeAndTeardown/0
                  Metal | INFO     | Initializing device 0. Program cache is NOT enabled
                  Metal | INFO     | AI CLK for device 0 is:   1000 MHz
                  Metal | INFO     | MMIO Device 0 : Tunnel 0 : Device 0
                  Metal | INFO     | MMIO Device 0 : Tunnel 0 : Device 1
                  Metal | INFO     | Closing device 0
                  Metal | INFO     | Disabling and clearing program cache on device 0
[       OK ] DeviceInit/DeviceParamFixture.DeviceInitializeAndTeardown/0 (26 ms)
[ RUN      ] DeviceInit/DeviceParamFixture.DeviceInitializeAndTeardown/1
                  Metal | INFO     | Initializing device 0. Program cache is NOT enabled
                  Metal | INFO     | AI CLK for device 0 is:   1000 MHz
                  Metal | INFO     | Initializing device 1. Program cache is NOT enabled
                  Metal | INFO     | AI CLK for device 1 is:   1000 MHz
                  Metal | INFO     | MMIO Device 0 : Tunnel 0 : Device 0
                  Metal | INFO     | MMIO Device 0 : Tunnel 0 : Device 1
                  Metal | INFO     | Closing device 0
                  Metal | INFO     | Disabling and clearing program cache on device 0
                  Metal | INFO     | Closing device 1

Repeating all tests (iteration 323) . . .

Note: Google Test filter = *DeviceParamFixture*
[==========] Running 4 tests from 1 test suite.
[----------] Global test environment set-up.
[----------] 4 tests from DeviceInit/DeviceParamFixture
[ RUN      ] DeviceInit/DeviceParamFixture.DeviceInitializeAndTeardown/0
                  Metal | INFO     | Initializing device 0. Program cache is NOT enabled
                  Metal | INFO     | AI CLK for device 0 is:   1000 MHz
                  Metal | INFO     | MMIO Device 0 : Tunnel 0 : Device 0
                  Metal | INFO     | MMIO Device 0 : Tunnel 0 : Device 1
                  Metal | INFO     | Closing device 0
                  Metal | INFO     | Disabling and clearing program cache on device 0
[       OK ] DeviceInit/DeviceParamFixture.DeviceInitializeAndTeardown/0 (25 ms)
[ RUN      ] DeviceInit/DeviceParamFixture.DeviceInitializeAndTeardown/1
                  Metal | INFO     | Initializing device 0. Program cache is NOT enabled
                  Metal | INFO     | AI CLK for device 0 is:   1000 MHz
                  Metal | INFO     | Initializing device 1. Program cache is NOT enabled
                  Metal | INFO     | AI CLK for device 1 is:   1000 MHz
                  Metal | INFO     | MMIO Device 0 : Tunnel 0 : Device 0
                  Metal | INFO     | MMIO Device 0 : Tunnel 0 : Device 1
                  Metal | INFO     | Closing device 0
                  Metal | INFO     | Disabling and clearing program cache on device 0
                  Metal | INFO     | Closing device 1

https://github.com/tenstorrent/tt-metal/actions/runs/9604648260/job/26490801278
https://github.com/tenstorrent/tt-metal/actions/runs/9570597099/job/26385904897

@aliuTT - can you help triage this please?

I wasn't able to repro locally last week. Still trying to find a setup where this fails. I may have to grab a cloud VM for this debug.

@aliuTT is this stale?

It's old, but I see DeviceInit/DeviceParamFixture.DeviceInitializeAndTeardown/0 is still disabled on main for N300. It may still be reproducible. I suspect maybe the CloseDevice call in that test is not correct for N300 somehow, I can take a look tomorrow or next week.

I tested locally. Issue appears resolved. Test re-enabled on main


Add support for tagging templates

Great project to simplify and track CloudFormation deployments! I needed the ability to tag my CF templates resources at the template level so I added it. Thanks for considering merging it back in!

@andyday I've been using this successfully since I posted this PR. Let me know if you'd like to see any changes to get this addition merged. Thanks!

@andyday Would love to see this one merged!
@kmcgill88 Thx for the contribution!

Sorry for the delay.   Please resolve conflicts.

Awesome! I'll get it cleaned up soon!

@andyday something is up with travis. error Couldn't find the binary (standard | snazzy) && nyc mocha. Tests pass locally.

@andyday let me know if I can do anything to get this in.

remove the node 4.3 config in the travis config file.  replace it with node 8

Goode to go!

Published to npm as 1.5.0


Use patch-operator for ad-hoc resource patches

The component now requires component patch-operator to be present in the cluster instead of component resource-locker.
There's limited support for transforming existing legacy adhoc patches into valid patch-operator Patch resources to make migration to the new major version simpler.
See also:

https://github.com/projectsyn/component-resource-locker/pull/73
https://github.com/projectsyn/component-patch-operator/pull/9

Checklist

[x] The PR has a meaningful title. It will be used to auto generate the
changelog.
The PR has a meaningful description that sums up the change. It will be
linked in the changelog.
[x] PR contains a single logical change (to build a better changelog).
[x] Update the documentation.
[x] Categorize the PR by adding one of the labels:
bug, enhancement, documentation, change, breaking, dependency
as they show up in the changelog.

I've successfully tested migration to patch-operator on a cluster using legacy ad-hoc patches using this feature branch for component adhoc-configurations.


[Feature Request]: Allow for chart aggregation at calendar month intervals

Provide a clear and concise description of what you want to happen.
Charts in PRISM are currently show rolling 10-day products for all data layers regardless of their length. For example a 1-month chart using the date 11-June-2024 is an aggregate of data from 21-May-2024 to 20-June 2024 and a new one-month aggregate is available every 10-days.
Users would also like to be able to access monthly intervals which are more clearly understood as fixed per calendar month rather than rolling every 10-days. For example, they would want to see monthly aggregates for all of Jan, Feb, Mar, etc.
This is required for monthly rainfall aggregate and monthly rainfall anomaly. I believe we can simply use r1h for aggregate and r1q for anomaly of the 3rd dekad (i.e. the 21st day of each month) for each month and use that as the calendar month value.
This data is already available through the HDC stats API but we do need a modification to the frontend. My suggestion is to add a toggle below the chart which appears if either 1-month rainfall aggregate or anomaly is chosen which allows a user to optionally display data by calendar month.
In the example below, the chart on the left is aggregated by calendar month.

Is there anything else you can add about the proposal? You might want to link to related issues here, if you haven't already.
No response

Not captured here as it's not the current priority, but we would also eventually want to apply this to NDVI, which does not have monthly aggregation already provided by the HDC stats API


M3-1510 Add pagination to UserLanding.

Purpose

Add pagination to users table.
Update how gravatar URLs are acquired.
Added memoizee to the product for easy memoization of the gravatar request.

User large prod account to test.

Coverage increased (+0.001%) to 27.064% when pulling 6f55f39627a1866ddc75725049a8c57fed63a620 on rjstires:M3-1510 into e8f4174eb9d2bf7e928c60ece9a8fc9398c48353 on linode:develop.


[TEMP] Implement temporaly  repr for multiindex

When I checked repr for MultiIndex,
It was confusing because it was printed as 'Index' instead of 'MultiIndex'.
>>> import databricks.koalas as ks
>>> import pandas as pd
>>> import numpy as np
>>>
>>> arrays = [[1, 1, 2, 2], ['red', 'blue', 'red', 'blue']]
>>> idx = pd.MultiIndex.from_arrays(arrays, names=('Index', 'color'))
>>> pdf = pd.DataFrame(np.random.randn(4, 5), idx)
>>> kdf = ks.from_pandas(pdf)
>>>
>>> kidx = kdf.index
>>> kidx
Index([(1, 'red'), (1, 'blue'), (2, 'red'), (2, 'blue')], dtype='object', name='number')

Of course, maybe it's not really necessary,
but I suggest changing the MuntiIndex repr temporarily to avoid confusion.

then we can get repr result like this
>>> kidx = kdf.index
>>> kidx
MultiIndex([(1, 'red'), (1, 'blue'), (2, 'red'), (2, 'blue')], dtype='object', name='number')

Please check this suggestion when someone available.
Thanks :)

Codecov Report

Merging #693 into master will decrease coverage by 2.36%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master     #693      +/-   ##
==========================================
- Coverage   93.54%   91.18%   -2.37%
==========================================
  Files          32       32
  Lines        5455     5457       +2
==========================================
- Hits         5103     4976     -127
- Misses        352      481     +129

Impacted Files
Coverage Δ

databricks/koalas/indexes.py
96.33% <50%> (+4.74%)
:arrow_up:

databricks/koalas/usage_logging/__init__.py
23.14% <0%> (-74.08%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50% <0%> (-50%)
:arrow_down:

databricks/koalas/__init__.py
76.92% <0%> (-7.7%)
:arrow_down:

databricks/conftest.py
90.69% <0%> (-6.98%)
:arrow_down:

databricks/koalas/frame.py
92.95% <0%> (-1.79%)
:arrow_down:

databricks/koalas/namespace.py
87.75% <0%> (-1.64%)
:arrow_down:

databricks/koalas/plot.py
93.26% <0%> (-1.35%)
:arrow_down:

databricks/koalas/utils.py
97.31% <0%> (-0.68%)
:arrow_down:

databricks/koalas/testing/utils.py
77.35% <0%> (-0.63%)
:arrow_down:

... and 3 more

Continue to review full report at Codecov.

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


fix: 5681 - startActivityAndCollapse + android 14

What

Looks like we use a kotlin method deprecated in android 14.
That said, I don't even know what a tile is.
cf. https://stackoverflow.com/questions/77547985/launch-an-activity-from-tileservice-for-android-14-is-not-allowed

Fixes bug(s)

Fixes: #5681

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 6.80%. Comparing base (4d9c7fc) to head (784826c).
Report is 355 commits behind head on develop.

Additional details and impacted files
@@            Coverage Diff             @@
##           develop   #5684      +/-   ##
==========================================
- Coverage     9.54%   6.80%   -2.75%
==========================================
  Files          325     404      +79
  Lines        16411   21550    +5139
==========================================
- Hits          1567    1466     -101
- Misses       14844   20084    +5240

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

I believe it's the tiles you see in the notification drawer: https://developer.android.com/about/versions/14/behavior-changes-14#tiles-launch-api, and there's a new recommanded method

Now flutter run works, but I still cannot see a related tile.
Last change was in January 2023. Not sure how relevant it still is.
cf. #2333.


When editors sorted alphabetically next editor hotkey ignores it

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: Version: 1.68.1 (Universal)
OS Version: os x 12.3.1

Steps to Reproduce:

explorer.openEditors.sortOrder set to alphabetical
open multiple editors
try to navigate between them with View: Open Next Editor hotkey

You'll see that editors switched as if explorer.openEditors.sortOrder is set to editorOrder (default)

That setting has no impact on editors, only "Open Editors" view.

The described behavior is how it is expected to work. If you disagree, please explain what is expected and what is not in more detail. See also our issue reporting guidelines.
Happy Coding!


Add column origin attribute

Add feature origin attribute to column schema
Closes #927

@tamargrey agreed, that's a good parameter for read_file


fix missing strikethrough styling for scm resource states

This PR fixes fixes #87668 - in which strikethrough is missing from scm provider resources
Note that there are 7 more references in various css files to .monaco-icon-label-description-container - which no longer appears to exist in the code. I have not yet analyzed whether they should also be updated to the new classes.

I note that the tests failed due to hygiene checks, but this seems to have been caused by the update to TS 3.8 about an hour ago and is not related to the change...

Having looked through the other instances a bit more I believe they should all be updated as well - haven't managed to actually produce the right conditions yet though.
For example suggest.css implies deprecated items should have strike through, but functions don't appear to have the deprecated tag applied to them when I make them deprecated via the comments, and as such the deprecated class isn't applied. (Probably something I'm doing wrong).
A couple of them refer to previous minor styling issues in the tab header, I wasn't able to reproduce the actual original issues with the incorrect css - nevertheless by inspecting the elements, the current css is clearly not applied as intended

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

I've updated the PR to update all the styles that had the incorrect class name.
I was able to prove that deprecated strikethrough was broken by modifying the sample completion-sample extension.
I did clearly see that the correct overflow / text-overflow styles were applied to tabs following my change - though I couldn't see any visible difference from when they were not working.
Same for the quick open dialog and the custom view tree (e.g. the npm scripts view) - flex: 1 was not applied, but I could not see any visible difference when it was applied.
I haven't worked out how to produce an element with .remote-help-tree-node-item-icon


cue/fmt: various formatting issues

There is an extra tab indentation in the following after cue fmt
machine_type: [
		if arch == "amd" {"n2-standard-2"},
		if arch == "arm" {"t2a-standard-2"},
		"unknown arch",
][0]

cue fmt actually turns the expected output into the unexpected (with an extra tab)
exec cue fmt file.cue
cmp file.cue golden.cue

-- file.cue --
arch: string
machine_type: [
	if arch == "amd" {"n2-standard-2"},
	if arch == "arm" {"t2a-standard-2"},
	"unknown arch",
][0]
-- golden.cue --
arch: string
machine_type: [
	if arch == "amd" {"n2-standard-2"},
	if arch == "arm" {"t2a-standard-2"},
	"unknown arch",
][0]

If you remove the [0] from the end, it no longer has the extra tab indentation

Here's another one
exec cue fmt file.cue
cmp file.cue golden.cue

-- file.cue --
_foo: {
	#tool: string
	#arch: string

	// skip_create_image: true
}
-- golden.cue --
_foo: {
	#tool: string
	#arch: string

	// skip_create_image: true
}

What this results in currently is an injected newline
_foo: {
	#tool: string

	#arch: string

	// skip_create_image: true
}

If I put a field at the end, then the newline is not added
_foo: {
	#tool: string
	#arch: string

	// skip_create_image: true
	foo: _
}

cue 0.7 improve some formatting situations, but in some cases it is worse:
nicer:
-	templates: [ for name, tmpl in _template {tmpl}]
+	templates: [for name, tmpl in _template {tmpl}]

not great:
 #XX: {
-	env: [string]:           string | bool | int
+	env: [string]:                                    string | bool | int
 	database: [Name=string]: #DatabaseConfig & {name: Name}

@eonpatapon I believe that bug was resolved in v0.9.0 already, but please say if not - ideally with a new issue, as it's hard to track and resolve issues which concern multiple bugs at once.
@verdverm the first of your bugs should be resolved by the stack up for review at https://review.gerrithub.io/c/cue-lang/cue/+/1196135. The second of your bugs, where definitions get separated by empty lines, appears to be this bit of code in cue/format:
			if hasDocComments(x) {
				switch x := list[i-1].(type) {
				case *ast.Field:
					if internal.IsDefinition(x.Label) {
						f.print(newsection)
					}

				default:
					f.print(newsection)
				}
			}

That is, it seems like the formatter aims to separate definitions if they contain any doc comments. In your particular example the comment should not be considered a doc comment, so I think it's an edge case bug.