Update README.md

Fixed broken link - removed extra parens from the url.

Thanks @Karla-Isabel-Sandoval !


Suggestion: Add more to the FAQ

Aside from dialyzer it would be useful to have comparisons to other Erlang type checking tools, particularly Gradualizer. I'm sure there are others that'd be good to provide, gradualizer is just the one I was personally curious in a comparison with :).

Thanks. Yes, it would be added as part of documentation soon.

Hey, any update on this? Or can you say here what the difference is with Gradualizer :)

After reading the docs and running some quick experiments, here are my preliminary conclusions about this.
I think the main difference is that eqWAlizer has two modes, strict and dynamic. Dynamic mode corresponds, in essence, to Gradualizer. Strict mode allows for stricter type-checking, by not only considering dynamic() to be equal to term(), but also by treating types differently internally, and by making sure that dynamic() can never appear during elaboration, type refinements, or type inference. While eqWAlizer is mostly developed with gradual mode in mind at the moment, we still try to keep strict mode in sync as it can sometimes provide better signal.
You can find more info about modes here.
EqWAlizer and Gradualizer also make different assumption about types. For example, eqWAlizer does not differentiate between numeric types (it consider them all equal to number()), while Gradualizer does. Gradualizer also makes different assumption about map types. For example, Gradualizer understands type #{integer() := integer()} as the type of maps that have at least one binding from an integer to an integer. In eqWAlizer, we considered such a type to be ambiguous and opted to simply interpret it as #{term() => term()} in strict mode, and #{dynamic() => dynamic()} in gradual mode.
On the topic of maps, eqWAlizer differentiates between dictionaries and shapes: shapes are special map types where the keys are all atoms that are known statically. This allows eqWAlizer to perform more precise type checking on maps in some instances.
More info about types and subtyping for maps.
Finally, eqWAlizer features stronger occurrence typing (type refinement) than Gradualizer. In particular, eqWAlizer is able to refine the type of expressions using information in arbitrary deep patterns and guards. See these snapshot tests as an example. Gradualizer only allows refinements based on failing clauses in very specific cases, as explained here.
Hope this clarifies things a bit, I'll add this info in the documentation after more experimenting.


Switch to sql for Wu realnames, fix #329.

realname is now fetched from the Wu sql database, which should preferred over LDAP since it is easier and more frequently edited, and the LDAP is not synced against it.

[x] Run the tests and see them pass
[x] Rebase your branch on top of develop
[x] Include tests for features you introduced / bugs you fixed

This fixes issue #329.

@lukasjuhrich: I hope I addressed the formatting issue, please have another look.

Awesome, thanks! I'm gonna merge&deploy tomorrow if I have time.


Adding "justDraggable" option to the "range" component

Short description of what this resolves:
This commit resolves #6956 issue.
Changes proposed in this pull request:
we added a new option named: "justDraggable" to "range" component that is false by default. while it is set to true, range is working only if user drags the knob. In other words, range value does not change by one click or dragging somewhere else in the range bar.

Ionic Version: 2.x
Fixes: #6956

What about knobOnly or thumbOnly?  We don't love justDraggable 😄 .
Give it some thought and see what you can come up with.  We'll do the same.
Thanks for the PR!
Thanks,
Dan

Hello Dan
knobOnly seems a better alternative.
Thanks
Mohi
On Mon, Aug 8, 2016 at 10:29 PM, Dan Bucholtz<EMAIL_ADDRESS>wrote:

What about knobOnly or thumbOnly? We don't love justDraggable 😄 .
Give it some thought and see what you can come up with. We'll do the same.
Thanks for the PR!
Thanks,
Dan
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/driftyco/ionic/pull/7496#issuecomment-238322602, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGWe0vIQeLBFVog97cbNks5XFD_U32bsks5qd252gaJpZM4JZU6Z
.

I'd like to propose an alternative solution: #9272.

Hello @mohi-ir, big thanks for submitting this PR. Would you be able to fix the merge conflicts please so we can continue to review?

@jgw96 Is there a preference for this solution over #9272? I can make my branch compatible again, but want to make sure you don't prefer to implement this instead.

@mohi-ir  @frederickjansen hi! it's time for us to start merging new features
I am evaluating both implementations, thanks!

Hello and thank you for contributing to Ionic! We have been working on porting all of the Ionic components to web components and have recently updated master to reflect this. This significant change has caused this pull request to break. While we really appreciate the time and effort you put into creating this, we are not able to merge it because of the newly introduced conflicts. We are extremely sorry about this. We will not be merging any more features in to v3. If this is a feature and you have the time, please resubmit this PR against the master branch. If this is a critical security issue in v3, we would greatly appreciate it if you would resubmit the PR against the new v3 branch. Thanks so much for your time!


Geniki Taxydromiki always returning no result

Describe the bug
All tracking numbers queried using Geniki Taxydromiki always return no result regardless of the actual status of the package.
Expected behavior
The module is expected to return the tracking history.

I'm gonna test this and update this ASAP :+1:

This should be fixed by commit 59206790cd35f3a54414cca73bd9eaf0ff8c581a - I will be testing it as soon as possible.


[hotfix] fix EXECUTION_FAILOVER_STRATEGY's comments error and its inf…

What is the purpose of the change
fix EXECUTION_FAILOVER_STRATEGY's comments error and its info message error
Brief change log
modified comments and info message
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)

can anybody take a look at it? it's pretty straightforward

Thanks @makeyang for your contribution.
Could you also regenerate the documentation to update the description for this option on the web page.
You can do it by:
mvn package -Dgenerate-config-docs -pl flink-docs -am -nsu

This is duplicate of #6303. Therefore I will close this PR. Feel free to reopen it if you disagree @makeyang


Section-ify mark and undo/redo

Codecov Report

Merging #277 (562c131) into master (3a86d4c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #277   +/-   ##
=========================================
  Coverage     72.54%   72.54%
  Complexity      907      907
=========================================
  Files           126      126
  Lines          2739     2739
  Branches        350      350
=========================================
  Hits           1987     1987
  Misses          581      581
  Partials        171      171

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[Bug]: Different reporter order leads to coverageMap missing in custom reporter starting in Jest 28

Version
29.5.0
Steps to reproduce

Clone repo: https://github.com/danieldiekmeier/jest-custom-reporter-reproduction
npm i
npm test

In the output and in the reporter.js file, you can see that I had to use a setInterval to keep my custom reporter running until after the CoverageReporter is finished collecting the coverage. In Jest 27, this wasn't necessary – by the time my custom reporter ran, the CoverageReporter was already finished.
Expected behavior
I expect my custom reporters to run last (like they did in Jest 27). Alternatively, it'd be fine if I could influence the order somehow.
Actual behavior
The coverage reporter runs after my custom reporter, making it impossible to use its results in custom reporters.
Additional context
This is the new code in Jest >=28.0.0, which registers the CoverageReporter after the custom reporter: https://github.com/facebook/jest/blob/v28.0.0/packages/jest-core/src/TestScheduler.ts#L329-L364
Up until Jest 27.5.1, the code looked like this, registering the CoverageReporter first: https://github.com/facebook/jest/blob/v27.5.1/packages/jest-core/src/TestScheduler.ts#L349-L380
This is probably a duplicate of the automatically closed #13479.
Environment
System:
  OS: macOS 13.3
  CPU: (10) arm64 Apple M1 Pro
Binaries:
  Node: 16.19.1 - ~/.asdf/installs/nodejs/16.19.1/bin/node
  npm: 9.6.0 - ~/.asdf/plugins/nodejs/shims/npm
npmPackages:
  jest: ^29.5.0 => 29.5.0

This can't be stale, it's still in triage!

Still needs triage!

@mrazauskas are you still up for sending a PR for this? 🙂

@mrazauskas ping 😀

Any news on this?

Uff.. I can’t find time unfortunately. Feel free to take it over.

@danieldiekmeier Hi! Have you got better solution than your current one? Are you going to open your PR?

@i-dont-know-javascript Hello! My solution was to stop using Jest. I never planned to open a PR about this, but even getting this bug triaged/acknowledged took literally MONTHS. I no longer care either way.

I, at @danieldiekmeiers previous company (Hi Daniel! 👋), still care though 😅


(maint) Update to latest ezbake

Ezbake 0.3.21 was just released, so this updates to use that version.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-enterprise.delivery.puppetlabs.net/job/enterprise_puppetdb_init-multijob_githubpr-stable/60/
Test PASSed.


Pixiv login error

Bug description
every time i try to log into pixiv i get hit with this error :
[12:55:29.461][Info] [www.pixiv.net] Logging in...
[12:55:31.115][Warning] [www.pixiv.net] No OAuth2 token type received: {"has_error":true,"errors":{"system":{"message":"Invalid grant_type parameter or parameter missing","code":1508}},"error":"invalid_grant"}
[12:55:31.116][Info] [www.pixiv.net] Login finished: failure.
it's been like this for a while now
Expected behavior
log into pixiv like it used to do normally .
System information
windows 10

Closing as duplicate of #2263
There doesn't seem to be a proper workaround as of now but I'm investigating


Data wipe and duplication while simultaneously indexing items in auto-batch

TLDR
I found the root cause of issue!
My guess is: Meilisearch fails to index items when individual items are added simultaneously in auto-batch mode on 1 core CPU.
I'm running 1 CPU AMD (can this be the cause?) and when I send a few documents at once to Meilisearch it creates items as duplicates.
Every time I do any indexing in my app I'm synchronizing indexing using wait_for_task from Ruby client, to ensure no new batch of documents are sent after previous completed processing. If I run a few indexing tasks in parallel I get data loss or a broken index (see below).
The following scenario describes how the data loss happens and the point where internal error  "message":"internal: Missing key in the documents database.. can be seen
Scenario
I launched Meilisearch in auto-batch mode:
/usr/bin/meilisearch --db-path /var/lib/meilisearch/data1.ms --env production --dumps-dir /var/lib/meilisearch/dumps --enable-auto-batching

Here is my index:
{
  "uid": "Test_staging",
  "name": "Test_staging",
  "createdAt": "2022-06-06T12:44:50.221884241Z",
  "updatedAt": "2022-06-06T17:41:11.056253545Z",
  "primaryKey": "id"
}

here are settings for it:
{
  "displayedAttributes": [
    "*"
  ],
  "searchableAttributes": [
    "title_tags",
    "public_title",
    "file",
    "description"
  ],
  "filterableAttributes": [
    "assigned_to",
    "branch_id_number",
    "created_at",
    "priority",
    "project_id",
    "state",
    "sync",
    "updated_at"
  ],
  "sortableAttributes": [
    "branch_id_number"
  ],
  "rankingRules": [
    "branch_id_number:desc",
    "words",
    "typo",
    "attribute",
    "exactness"
  ],
  "stopWords": [],
  "synonyms": {},
  "distinctAttribute": "public_uid",
  "typoTolerance": {
    "enabled": true,
    "minWordSizeForTypos": {
      "oneTypo": 5,
      "twoTypos": 9
    },
    "disableOnWords": [],
    "disableOnAttributes": []
  }
}

Then I added 2 documents simultaneously but not in batch (I assume this causes an issue).
These documents:
[
{"id": 228142, "title": "asdsad", "state": "automated", "priority": "normal", "public_uid": "37ccf021", "project_id": 78207, "branch_id_number": 0},
{"id": 228143, "title": "something", "state": "automated", "priority": "normal", "public_uid": "39c6499b", "project_id": 78207, "branch_id_number": 0 }
]

One with the title "asdsad" other with title 'something' on different IDs
But Meilisearch contained the same document under both IDs. So I tried to re-add them via CURL:
I add 2 documents via CURL:
curl -X POST 'http://localhost:7700/indexes/Test_staging/documents?primaryKey=id' -H 'Content-Type: application/json' --data-binary '[{"id": 228142, "title": "asdsad", "state": "automated", "priority": "normal", "public_uid": "37ccf021", "project_id": 78207, "branch_id_number": 0}, {"id": 228143, "title": "something", "state": "automated", "priority": "normal", "public_uid": "39c6499b", "project_id": 78207, "branch_id_number": 0 }]' | jq
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   432  100   130  100   302  21666  50333 --:--:-- --:--:-- --:--:-- 72000
{
  "uid": 3894,
  "indexUid": "Test_staging",
  "status": "enqueued",
  "type": "documentAddition",
  "enqueuedAt": "2022-06-06T17:41:04.796354715Z"
}

They are added
curl -X GET 'http://localhost:7700/tasks/3894' | jq
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   321  100   321    0     0  53500      0 --:--:-- --:--:-- --:--:-- 64200
{
  "uid": 3894,
  "indexUid": "Test_staging",
  "status": "succeeded",
  "type": "documentAddition",
  "details": {
    "receivedDocuments": 2,
    "indexedDocuments": 4
  },
  "duration": "PT3.013141648S",
  "enqueuedAt": "2022-06-06T17:41:04.796354715Z",
  "startedAt": "2022-06-06T17:41:10.341110888Z",
  "finishedAt": "2022-06-06T17:41:13.354252536Z",
  "batchUid": 3893
}

But in fact the first document only is stored, and the second one is ignored.
There are documents named "something" and "adsad" I query both of them but in both cases, I receive the document #1
curl  -X GET 'http://localhost:7700/indexes/Test_staging/documents/228142' | jq
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   137  100   137    0     0  22833      0 --:--:-- --:--:-- --:--:-- 22833
{
  "id": 228143,
  "title": "something",
  "state": "automated",
  "priority": "normal",
  "public_uid": "39c6499b",
  "project_id": 78207,
  "branch_id_number": 0
}
curl -X GET 'http://localhost:7700/indexes/Test_staging/documents/228143' | jq
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   137  100   137    0     0  27400      0 --:--:-- --:--:-- --:--:-- 27400
{
  "id": 228143,
  "title": "something",
  "state": "automated",
  "priority": "normal",
  "public_uid": "39c6499b",
  "project_id": 78207,
  "branch_id_number": 0
}

I tried to write documents to new IDs, when I use a new ID which (in my case were 728142 and 728143) and in this case documents were store correctly. These IDs were not exist in database.
So my guess was that overwriting current data can be a problem.
Then I tried to delete items 228142 and 228143 from index and then re-add them:
{"uid":4011,"indexUid":"Test_staging","status":"succeeded","type":"documentDeletion","details":{"receivedDocumentIds":1,"deletedDocuments":1},"duration":"PT0.775224943S","enqueuedAt":"2022-06-06T20:16:51.462232736Z","startedAt":"2022-06-06T20:16:51.464172707Z","finishedAt":"2022-06-06T20:16:52.23939765Z","batchUid":4011}root@meilisearch0271ondebian10buster-s-1vcpu-2gb-amd-ams3-01:~#

But when I try to add new item I receive this internal error:
{"uid":4012,"indexUid":"Test_staging","status":"failed","type":"documentAddition","details":{"receivedDocuments":2,"indexedDocuments":0},"error":{"message":"internal: Missing key in the documents database..","code":"internal","type":"internal","link":"https://docs.meilisearch.com/errors#internal"},"duration":"PT0.003221089S","enqueuedAt":"2022-06-06T20:16:58.45716101Z","startedAt":"2022-06-06T20:16:58.

-X GET 'http://localhost:7700/indexes/Test_staging/documents/928142' | jq
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100    94  100    94    0     0  18800      0 --:--:-- --:--:-- --:--:-- 18800
{
  "id": 928142,
  "title": "asdsad",
  "state": "automated",
  "priority": "normal",
  "public_uid": "37ccf021"
}
 curl -X GET 'http://localhost:7700/indexes/Test_staging/documents/928143' | jq
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100    97  100    97    0     0  19400      0 --:--:-- --:--:-- --:--:-- 19400
{
  "id": 928143,
  "title": "something",
  "state": "automated",
  "priority": "normal",
  "public_uid": "39c6499b"
}

Meilisearch version: 0.27.1
Stats for this index:
{
  "numberOfDocuments": 99101,
  "isIndexing": false,
  "fieldDistribution": {
    "assigned_to": 99100,
    "branch_id": 99100,
    "branch_id_number": 99101,
    "changed_in_branch": 99100,
    "cid": 99100,
    "code": 99100,
    "created_at": 99100,
    "created_by": 99100,
    "created_in_branch": 99100,
    "deleted_in_branch": 99100,
    "description": 99100,
    "emoji": 99100,
    "environment": 99100,
    "file": 99100,
    "has_examples": 99100,
    "id": 99101,
    "import_id": 99100,
    "jira_link": 99100,
    "last_sync_id": 99100,
    "params": 99100,
    "position": 99100,
    "precondition": 99100,
    "priority": 99101,
    "project_id": 99101,
    "public_title": 99100,
    "public_uid": 99101,
    "recordings_count": 99100,
    "run_statuses": 99100,
    "state": 99101,
    "suite_id": 99100,
    "sync": 99100,
    "tags": 99100,
    "title": 99101,
    "title_tags": 99100,
    "updated_at": 99100
  }
}

To create index I used Ruby on Rails integration and Ruby client.
All interactions were re-played over CURL to demonstrate the issue.
I tried to change the primary key index from ID to a compound index "#{project_id}_#{public_uid}" and recreate index. I get absolutely the same behavior.
can this be that auto-batching breaks the atomic additions of documents, combining them in a batch even they were executed as separate requests?
Meilisearch is running on DigitalOcean droplet created via Marketplace

Hey @DavertMik,
Thank you very much for the detailed reproduction steps, I am happy to be able to reproduce it on my side and I am currently trying to find and fix the issue. The bug is also on the main branch not only v0.27.1.
Here is a simpler version of the steps to reproduce the bug, the first block of code is the list of documents you gave us and we store them in a documents.json file.
[
    {
        "id": 228142,
        "title": "asdsad",
        "state": "automated",
        "priority": "normal",
        "public_uid": "37ccf021",
        "project_id": 78207,
        "branch_id_number": 0
    },
    {
        "id": 228143,
        "title": "something",
        "state": "automated",
        "priority": "normal",
        "public_uid": "39c6499b",
        "project_id": 78207,
        "branch_id_number": 0
    }
]

We run the server with a debounce duration just to make sure that the auto-batching system creates a new batch from your two documents that were sent independently. If we don't add a debounce the auto-batching system just processes the two updates independently and everything works fine.
cargo run --release -- --no-analytics --enable-auto-batching --debounce-duration-sec 5 --log-level=debug

Now that the engine run just sends the two documents independently one just after the other, you have 5sec to send the second document.
cat documents.json | jq '.[0]' | xh 'http://localhost:7700/indexes/repro/documents?primaryKey=id' content-type:application/json
cat documents.json | jq '.[1]' | xh 'http://localhost:7700/indexes/repro/documents?primaryKey=id' content-type:application/json

Once the update has been processed, the first document that has been sent is displayed as the second. The bug is surely located in either the auto-batching system of Meiliseqrch or in the update transform system of milli.

@Kerollmops thank you very much for the quick look and for reproducing the issue. I spent 4 days catching it and that's so nice my efforts were meaningful

--debounce-duration-sec 5

So what is a quick fix for this error? Can I disable debounce like this so two updates were running independently?
--debounce-duration-sec 0

@irevoire I tried, 4 secs for indexing 2 documents is too slow for me.
Everything gets really really slow without auto-batching ... 🤢
so if no quick fix I can just update my application to combine all batched updates to be sent as 1 request

Hey @DavertMik,
Instead of releasing a quick fix, we released the real fix quickly :racing_car:
The v0.27.2 is on its way; it's already available in Brew. See the following gh-action to follow the release of the Debian package.
If the issue persists, please re-open the issue.
Thank you very much for the report, have a nice day.

Thank you so much for the FASTEST fix!

Is there an estimate for the Docker release?

Hello @DavertMik, sorry we are having unexpected issues with our docker CIs to build and push the docker images.
We are working on it, sorry for the inconveniance


Add IAM role for access to ES

what

Create IAM role for access to Elasticsearch
Update default Elasticsearch version from 6.2 to 6.5

why

Limit access among the cluster to the few who need it
Stay current

@osterman  @aknysh Comments addressed


Added profile

What is this PR about ?
This PR adds the code of my profile and project. I have followed all the instructions and executed them all

What is this PR about ?
This PR adds the code of my profile and project. I have followed all the instructions and executed them all

Where have you add the link url,
Read carefully and change accordingly and push the new changes

I have changed the Url


GEODE-4133: synchronize ClientHealthMonitor.getClientHeartbeats

Add generics and make some things private while I'm at it. eventually maybe we can fix GEODE-2678.

[x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[x] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[x] Is your initial contribution a single, squashed commit?

[x] Does gradlew build run cleanly?

[I don't think this is necessary, be] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>@WireBaron @PivotalSarge

@upthewaterspout It does look like copy-on-write. It was giving access to the actual map in getClientHeartbeats that really stuck out to me; I was just trying to do some cleanup with the rest of it. It may be fine to wrap the returned map in an UnmodifiableMap.
I'm somewhat reluctant to use an actual CopyOnWriteHashMap because that would change semantics, the code isn't all that well tested, and I wanted to do a quick improvement and make the heartbeats map not publically available.

@galen-pivotal . Ok, makes sense. Looks good then.


OPA gatekeeper 100.0.1+up3.6.0 stuck uninstalling CRDs in RKE1/RKE2 Windows clusters

Rancher Server Setup

Rancher version: v2.6.4-rc11
Installation option (Docker install/Helm Chart): n/a

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc):

Proxy/Cert Details:

Information about the Cluster

Kubernetes version: v1.22.7 (RKE1), v1.22.7 (RKE2)
Cluster Type (Local/Downstream): Downstream

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider): Windows

Describe the bug

To Reproduce

Install rancher-gatekeeper 100.0.1+up3.6.0 in a RKE1/RKE2 Windows cluster
Uninstall rancher-gatekeeper
Check gatekeeper pod associated with the CRD uninstall job

Result
The rancher-gatekeeper chart gets stuck uninstalling, seemingly due to a pod from the crd uninstall job getting scheduled in a windows node which may not be supported.

The changes that created this issue have been reverted but will be fixed in the next release.
Propose that this issue be assigned to Helm Yes. @SheilaghM

PR: https://github.com/rancher/charts/pull/1872

There is a PR for this, but we need to push to v2.6.6 because testing requires a setup that QA is unfamiliar with and they won't be able to test this in time.

@eliyamlevy The installation for gatekeeper 100.1.1+up3.7.1 which was added in https://github.com/rancher/charts/pull/1872 was erroring out because of the missing linux-node-tolerations & linux-node-selector templates. I have raised a PR https://github.com/rancher/charts/pull/1938 to fix this installation issue.
cc: @SheilaghM @MKlimuszka @samkulkarni20

Validated this issue on rancher v2.6-head and v2.7-head windows cluster - 1cp, 1etcd, 1wk (Linux), 1wk (Windows)
Validation steps:

Created a rancher v2.6-head and v2.7-head
Provisioned a k8s cluster v1.22.15-rancher1-1
Installed a OPA-gatekeeper v100.2.0+up3.8.1
Made sure the chart is getting installed and working correctly.
Uninstalled the OPA-gatekeeper then gatekeeper-CRD.

Result

Uninstall progress do not get stuck. Uninstallation was successful.
OPA gatekeeper chart was removed successfully.

Issue is not active anymore. Hence, closing the issue.

Also validated the same with OPA-gatekeeper 101.0.0+up3.9.0. Uninstallation was successful.


Runnable analysis

This moves this app to use the RunnableAnalysis class, to major benefit:

Less code.
No temp files necessary (because run handles dataframes)
We get to see the original values of the params

Overall, the interface was totally sweet and really nice. Loving it.

To test this, make some analyses, rerun them :)


[WIP] Add S23K1XX and rddrone uavcan146

Untested
FYI @LorenzMeier, @igalloway
Do not MERGE!
This is built on current Nuttx master + han1raaijmakers-nxpDroneBoards pending PR to upstream.
han1raaijmakers-nxpDroneBoards PR will need to go upstream and be backported or PX4 nuttx will have to be updated.

@dinomani Could you please give this a go Monday?

I get an error building this PR:

I changed the Branch to Hans Nuttx branch and installed the Python extensions.

pip install jinja2

@LowOrbitIonCannon - there was a python update to master PX4. Most commands to update are emitted during the  the Cmake configure step on a clean build.  FYI @dagar but the jinga2 one is not,
pip install jinja2
will fix it.

Got it running. I had to setup Python3 from scratch

Cool! @LowOrbitIonCannon what does `i2cdetect -b n' for n = 0,1,2,4 show?

I connected a GPS device with Compass that is showing up at 0x1e.
Only 1 bus exists

GPS connected is a Drotek Sirius (M8N) with LIS3MDL magnetometer.

i tried building code with a simple adjustment of the  boards/nxp/rddrone-uavcan146/default.cmake
        SERIAL_PORTS
                GPS1:/dev/ttyS1

        DRIVERS
                gps

        MODULES

        SYSTEMCMDS
                i2cdetect
                top

But got an error during building
Memory region         Used Size  Region Size  %age Used
          vflash:         652 B         1 KB     63.67%
          pflash:         800 B         16 B    5000.00%
          dflash:      129452 B    1047536 B     12.36%
            sram:        3436 B       124 KB      2.71/home/dep00417/gcc-arm-none-eabi-7-2017-q4-major/bin/../lib/gcc/arm-none-eabi/7.2.1/../../../../arm-none-eabi/bin/ld: nxp_rddrone-uavcan146_default.elf section `__param' will not fit in region `pflash'
/home/dep00417/gcc-arm-none-eabi-7-2017-q4-major/bin/../lib/gcc/arm-none-eabi/7.2.1/../../../../arm-none-eabi/bin/ld: section __param LMA [0000000000000410,000000000000071f] overlaps section .text LMA [0000000000000410,000000000001fca8]
/home/dep00417/gcc-arm-none-eabi-7-2017-q4-major/bin/../lib/gcc/arm-none-eabi/7.2.1/../../../../arm-none-eabi/bin/ld: region `pflash' overflowed by 784 bytes
src/drivers/gps/libdrivers__gps.a(gps.cpp.obj): In function `GPS::setBaudrate(unsigned int)':
/home/dep00417/src/PX4-uavcan/build/nxp_rddrone-uavcan146_default/../../src/drivers/gps/gps.cpp:483: undefined reference to `tcgetattr'
/home/dep00417/src/PX4-uavcan/build/nxp_rddrone-uavcan146_default/../../src/drivers/gps/gps.cpp:521: undefined reference to `cfsetspeed'
/home/dep00417/src/PX4-uavcan/build/nxp_rddrone-uavcan146_default/../../src/drivers/gps/gps.cpp:526: undefined reference to `cfsetspeed'
/home/dep00417/src/PX4-uavcan/build/nxp_rddrone-uavcan146_default/../../src/drivers/gps/gps.cpp:531: undefined reference to `tcsetattr'
src/systemcmds/top/libsystemcmds__top.a(top.c.obj): In function `top_main':
/home/dep00417/src/PX4-uavcan/build/nxp_rddrone-uavcan146_default/../../src/systemcmds/top/top.c:75: undefined reference to `init_print_load_s'
/home/dep00417/src/PX4-uavcan/build/nxp_rddrone-uavcan146_default/../../src/systemcmds/top/top.c:94: undefined reference to `print_load'
/home/dep00417/src/PX4-uavcan/build/nxp_rddrone-uavcan146_default/../../src/systemcmds/top/top.c:82: undefined reference to `print_load'
/home/dep00417/src/PX4-uavcan/build/nxp_rddrone-uavcan146_default/../../src/systemcmds/top/top.c:84: undefined reference to `print_load'
/home/dep00417/gcc-arm-none-eabi-7-2017-q4-major/bin/../lib/gcc/arm-none-eabi/7.2.1/../../../../arm-none-eabi/lib/thumb/v7e-m/fpv4-sp/hard/libm.a(lib_a-w_sqrt.o): In function `sqrt':
w_sqrt.c:(.text.sqrt+0x84): undefined reference to `__errno'
w_sqrt.c:(.text.sqrt+0xa6): undefined reference to `__errno'
collect2: error: ld returned 1 exit status
%
ninja: build stopped: subcommand failed.
Makefile:200: recipe for target 'nxp_rddrone-uavcan146' failed
make: *** [nxp_rddrone-uavcan146] Error 1

I guess the available memory configuration settings need to be adjusted according S32K146. @davids5 ?

@dk7xe -
I have created branches
https://github.com/PX4/NuttX/tree/master_nxp_drone_boards
https://github.com/PX4/NuttX-apps/tree/master_nxp_drone_boards
These are equal to upstream NuttX what had the S32K PR (et al) merged today.
These branches will be rebased on upstream NuttX and can have the NXP commits on top. That we can submit upstream and then rebase.
This branch is pointing to master_nxp_drone_boards in the PX4 Nuttx and Apps
I will also backport the imx ,Kinetis and S32K PR to px4_firmware_nuttx-8.2
When that is done the team can work on the stable px4_firmware_nuttx-8.2 or the master_nxp_drone_boards version of nuttx.

Closing this PR as the board and architecture support have been merged to PX4 master. We can keep the branch around for a while if anyone would like to continue working with newer NuttX.


Issues with Dual Boot

Hello,
Thank you for your contributions and commitment to this project.
I observed that with recent EFI's (including the current one), for dual boot mode, the windows drive does not show up. only the Mac OS drive.
Could you please advice on how to enable this feature again as it was present in older EFI versions.
I have tried to run a comparison but no luck.
Any input on this will be very helpful.
Thank you for your kind consideration.

Hello,
Thank you for your contributions and commitment to this project.
I observed that with recent EFI's (including the current one), for dual boot mode, the windows drive does not show up. only the Mac OS drive.
Could you please advice on how to enable this feature again as it was present in older EFI versions.
I have tried to run a comparison but no luck.
Any input on this will be very helpful.
Thank you for your kind consideration.

Try latest files from repo


description = "@Size: Min - Max (until #1244 gets fixed)" when param annotated with @Min and/or @Max but #1244 is already fixed

2.9.0 still contains the above message but #1244 was fixed in 2.5. Will submit a fix shortly.

Fixed via #2429


Windows binaries doesn't simply work

... somehow, this issue never got reported.
Binary is still dynamically linked:
PS > dumpbin.exe /dependents .\fc2-live-dl-go-windows-amd64.exe
Microsoft (R) COFF/PE Dumper Version 14.40.33812.0
Copyright (C) Microsoft Corporation.  All rights reserved.

Dump of file .\fc2-live-dl-go-windows-amd64.exe

File Type: EXECUTABLE IMAGE

  Image has the following dependencies:

    avcodec-60.dll
    avformat-60.dll
    avutil-58.dll
    KERNEL32.dll
    msvcrt.dll

  Summary

        1000 .CRT
       68000 .bss
       53000 .data
        1000 .edata
        2000 .idata
       1C000 .pdata
      490000 .rdata
       17000 .reloc
      46E000 .text
        1000 .tls
        1000 .xdata

Issue appeared after migrating from MXE to Portage. Diff

Rollback to MXE for version v1.7.1 and v1.8.0, and maybe for all future version.

Closed with release v1.7.1 and commit<PHONE_NUMBER>8097f8cd1bb6207ec7d2206000fb68


What is the development status?

As title..

I looks quite abandoned, that's a shame because I desperately need this lib now and I also don't have to maintain...
Anyone up for a fork with PydanticV2 support and other goodies?

@usr-ein thank you for answering. You can fork the repo and implement the things you need. It has a very simple structure and pattern. I was thinking of something like this but a framework (FastAPI, Starlette, Flask, etc.) agnostic version.

@usr-ein take a look at fastcrud, check what else you need and maybe open an issue! I'm trying to keep a project like this going


Feature/datainit

The purpose of this change is to get our data in line with what already exists in the node api project and ensure that the return values match as well.

Looks like we were testing at about the same time!  I missed an endpoint when I was fixing the responses; it should be fixed with my last commit.

Nice job.


fix typos

Thanks, @dylanratcliffe, merge this if these are the corrections you referred to.

@binford2k That's them


crane create transformer to remove namespaces

Create crane transformer to automatically remove namespaces

Open an issue to the https://github.com/migtools/crane repository to track it
Probably a separate repo is needed like https://github.com/migtools/crane-plugin-openshift for the OpenShift plugin
Link to developing custom plugins

Moved to #50


Support of select query from subcolumns with WHERE condition

Please add support of select query from subcolumns with WHERE condition for MongoDB:
Select column1, column2.. (or/and subcolumn) from collection_name where column_name.subcolumn_name [condition] [value]
e.g.
select * FROM customers4
WHERE Rentals.staffId=1

It is actual:

when the sub-column is not on the second, but on the third level. E.g.

SELECT * FROM test. "123", where rent. "Movie title". Amount = 5.99;

when there are multiple columns with sub-columns, and the selection comes from the second one

SELECT * from restaurants where grades.grade = ‘A’

closed


Inconsistency in the description/more info sections

Description
The deployment description is inconsistent, for some deployments it only shows a brief message with no link to the docs like full,micro vms.

also for full vm it still has description below the title which was replaced a while ago with the one above it in all other solutions.

and for other solutions it's a full description with details and links for the documentation.

and for most of the solutions the more info icon displays the same text that's already displayed in the description section.

Suggestions
The section above each deployment should be consistent for all solutions.
or
For a cleaner look i would suggest removing the description from each solution and having full description + link to the documentation in the more details section instead of displaying the same data in multiple places. Suggested look:
Remove all descriptions from the UI and have the i icon contain all of the needed description and manual links

Removed description & included docs to each solution info except for Freeflow as it's not yet included in the manual.

verified on devnet playground descriptions are now removed except for freeflow


Add method to retrieve the list of available repositories

Retrieves a list of repositories from the /v2/_catalog endpoint.

Looks good, should merge it?

LGTM. Thank you.


Add more clear information on how to get involved

Recently @yuvipanda has been doing some really nice work to improve the developer / contributor documentation for JupyterHub. I worry that the documentation for a package may actually not be a great entry point for certain kinds of potential Jupyter contributors.
How do people feel about the following:
Turn the "community" page into a drop-down list in the nav-bar. The current page will direct to a new page called "email lists" (since it is just a list of email lists). A new page will be added something along the lines of "get_involved.md" that will give information about

The kinds of repositories users may want to be involved with.
The kinds of contributions Jupyter can use (here it's basically to make it clear that we welcome all kinds of contributions, not just tech contribs)
The kinds of tools that Jupyter stuff uses
Next steps to repo-specific stuff to get involved.

What do folks think? cc also @jzf2101 since I know she's done a lot of work trying to welcome in new people

I've been thinking about this as has @tgeorgeux and @willingc ?

https://whatcanidoformozilla.org/ is one of my personal favorites.

I made this https://github.com/jupyter/help/blob/master/CONTRIBUTING.md

I just discovered that there is already some sort of "contributor guide" documentation here:
https://jupyter.readthedocs.io/en/latest/contributor/content-contributor.html
but the only way to get to it (AFAICT) is from the Jupyter "community" page here:
http://jupyter.org/community

Since this ticket is more than three years old, I'm going to close it.


Remove Old Migration Code

Runtimes from our various networks have been successfully upgraded to the latest in Master.
Thus this PR removes any existing on_runtime upgrade code so it is not run a second time.
The following runtime upgrade migrations have been removed:

Democracy
Indices
Offences
Staking
Transaction Payment

Additionally, this PR removes old staking payout code that is no longer relevant after 84 eras since the migration from Lazy Payouts, and removes the MigrateEra storage item. It is important that any network that did migrate from lazy payouts to simple payouts waits until 84 eras have passed else some stakers may not get their rewards.
NOTE: This may effect the Call enum for staking. Is this an issue?

A secondary warning. I already wrote this in the main description, but this PR should only be merged if we are 84 eras after our last migration from Lazy Payouts. Otherwise, we should leave the staking code in.


provide slack group invite link

Do you want to request a feature or report a bug?
Feature
What is the current behavior?
No slack invite!
If this is a feature request, what is motivation or use case for changing the behavior?
Provide a link / badge for inviting to the slack group.

This is no feature 😅 and let's continue the discussion in #117. No need of opening multiple issues for the same thing 😄

Errr... ok! 😆


FileActivity: Make font size bigger

You will not maximize the screen before viewing the screen if you merge this

I suggest you to enable the ability to download APK file from the Continuous Integration because I cannot afford to set up an Android Development Environment, which means I am not able to see what the commits really do

发件人: Commit451Bot<EMAIL_ADDRESS>发送时间: 2018年7月16日 0:30
收件人: Commit451/LabCoat
抄送: Travor Liu; Author
主题: Re: [Commit451/LabCoat] FileActivity: Make font size bigger (#20)
Merged #20https://github.com/Commit451/LabCoat/pull/20.
―
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Commit451/LabCoat/pull/20#event-1734266495, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AXt_8YV0J8krNJrDrK8Wkq4cOmqE4mp4ks5uG25CgaJpZM4VQEZK.

Yep, this is set up on GitLab.com


VPN Configuration for Clippy

Motivation
We learned in https://github.com/giantswarm/roadmap/issues/2011 that we have to add Glippy to our GS VPN Configuration in order to secure and access it over our GS VPN.
Todo

[ ] Add Glippy to our GS VPN Config
[ ] Document the process on how to add a new Management Cluster to our VPN
[ ] Store the document of the process in our intranet as subsite under https://intranet.giantswarm.io/docs/product/providers/capz/

Outcome

We have Glippy secured via GS VPN access.
We have a document with a recipe to add new installations to our GS VPN configuration.

From @nprokopic in https://github.com/giantswarm/roadmap/issues/2011
========
Here we configure shared network range to include glippy MC network range as well as a larger network range that will accommodate all WCs created in glippy.
The suggestion is to use 10.x.y.0/20 range for glippy shared IP network range via the VPN connection (MC and all WCs).
This will enable us to deploy, for example, 16 clusters in total (counting all MCs and WCs) with 10.x.y.0/24 network range (256 addresses in total), where cluster network is split in the following way:

10.x.y.0/25 for workers subnet. This is 128 addresses in total, where 5 are reserved by Azure, which gives us 123 addresses for worker nodes.
10.x.y.128/25 divided into:

10.x.y.128/26 for control plane subnet
10.x.y.192/26 for bastion and other subnets that we will possibly need.

The smallest possible WC that we can create with the current WC setup would be 10.x.y.z/27, where:

10.x.y.z/28 is for workers subnet, which gives us 16 addresses (11 for nodes, 5 reserved by Azure)
10.x.y.z/29 is for control plane, which gives us 8 addresses, so 3 usable, where 1 is required for the internal load balancer, so we get 2 for control plane nodes, meaning we can have only 1 control plane node.

To expand a bit there, by using 10.x.y.0/20 network range for an MC (and its WCs), we get the following possibilities and constraints (max addresses, workers and CP nodes):

Total WCs
WC mask
Addresses
Workers
CP nodes

8
/23
512
251
123

16
/24
256
123
59

32
/25
128
59
27

64
/26
64
27
11

128
/27
32
11
1

The above examples show that this should be more than enough for testing purposes.
=====

We should also check if we can use the private DNS zones for WCs/MCs via the VPN or if we have to use a public DNS zone for WCs/MCs.


fix(ardour-git): move symlinks to postinst & fix various install issues

Add python as a build dependency

Still fails.

I’m gonna test manually when I get home.

Ok turns out the CI was installing the repo one... New issue now.

Try adding libasound2 to the build deps list.

Why can’t debian detect the alsa package?


quiz

Score: 3/5
Great job. The only errors you made were in the first question and with leaving out the end statements in your if...else statements. The syntax goes as follows:
if foo
   bar
else
   baz
end


Crashes on ios6 BAD_EXEC_

Hi mhergon,
i was so happy to implement your subtitle class into my app. but. it turns out it crashes on iOS 6.1 i get an bad exec when i start a movie, stop it, then play it again. its the same in your example. any idea?

I have same problem.
I only add a line #import <> ...


dotenv-hs failing to build in Nix

This happens because in the Build process the tests are run, but the file .dotenv.safe is not included in the cabal. Therefore, two test fail.

Solved with #109.


Adds a new peering ring mode to next hop selection strategies.

Adds a new peering consistent hash ring mode to next hop strategies.  This lets you build an edge group
of caches that share their cache.  When a request comes to a cache, that caches peers are checked using
consistent hash for the requested object.  If the peer does not have the object in it's cache, it will request the
object from an upstream parent or origin.  In the case where a request caches to the host itself, the request is
forwarded to the upstream parent or origin.  A new configuration parameter to the next hop strategies.yaml.
allows you to disable caching when a requested object is fetched from a peer in the cache group or pod.  This
allows for cache sharing among hosts at the edge.

[approve ci autest]

I'm going to move this to 9.2.x. Please make an argument for why we need this into 9.1.x, this late in the game :-).

@zwoop It's not that big of a change but, it does add the peering capability we talked about.  I think it would be a big plus to get it into 9.1.   It is a feature that we wish to use at Comcast and Verizon wishes to use it as well.  Walt has written an autest specifically to test this feature.  However, it cleanly cherry picks to 9.1 so, we can always just cherry pick it to our internal build.


Ways for a beginner developer to help

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

I've been donating for yt-dl for years, now there is not a monetary donation option anymore... But I'm studying to become a software developer, is there anyway a beginner dev could help? I've searched for easy issues to solve but I think I can't help with those... Probably no experience with python doesn't help much, in my case I know C and I'm starting with Java.
What would be the main concepts in python or other technologies what would be useful to contribute to the project? Thx in advance.

Have a look at issue #29310: the same advice applies.
Compared with C you have the advantage with Python that you can try out expressions at the Python prompt, in case the documentation is unclear (and bearing in mind that some features are known to differ according to Python version).
To do any coding you'll need a fork of the project from which you can submit Pull Requests. In case you don't feel confident to submit code as PRs, or your offerings aren't accepted, use your own fork (dogfood): easier in a Unix-like OS than Windows. Make a branch mymaster as a copy of master; make your change your change in a branch based on master and then submit a PR to your mymaster. Serve mymaster to yourself.

And you should use a powerful IDE which can tell you (most of the time) what's wrong, like PyCharm since this is Python after all. Will take a good deal of time to conf, but it's infinitely worth it.


Migrate to Dart 3

This is an attempt to migrate the plugin to Dart 3 and improve the performance. Dart 3 introduces sound null safety, and hence have updated the analysis_options to use the language features and improve the performance. That being said, Dart 3 also introduces other good features like records, pattern matching, and also native interop. Would be happy to contribute, and add use those features.

Thanks a lot for your contribution, awesome thinking about updating to Dart 3!

Merged to develop from here.
Thx a lot @AyushBherwani1998

Happy to contribute @Plumillon. I'll send few more PRs to cleanup and use Dart 3 features.


Improve IntelliJ setup instructions

Closes https://github.com/SAP-samples/cloud-cap-samples-java/issues/354

Doesn't IntelliJ have a feature similar to `Maven -> Update project ...' as I know it from Eclipse.

It has something like that, but it can't detect the additional source and resource locations that we contribute with our CDS Maven Plugin. That only works in VS Code and Eclipse..


feat(app): Added i18n in LoginPage

[x] Added i18n in Login page

[x] wrap login client component with a server component
[x] added metadata title


[FIX] useDimensions initial value

If we just use useDimensions at a component that is not rendered as a first component, like a modal, that can be instantiated after some orientation change, so the dims (initialValue) will be wrong.
Correct me if I wrong.
Feel free to close this if doesn't make any sense to v3.
Thanks in advance @wcandillon.

@crutchcorn opened a new pull request at #89.


Code only fetching open interest until October, instead of December

Lake API version: 0.8.0
Python version: 3.11.6
Operating System: MacOS Ventura 13.2

Description
I'm trying to fetch open Interest for Futures coins from 01-06-2022 until now, however for some coins it only fetches until 23-10-2023. Some of these include BANDUSDT, AVAXUSDT, ATAUSDT and so on.
What I Did
Heres the code i ran:
import lakeapi
import datetime
import matplotlib.pyplot as plt

def fetch_oi(symbol):

    books = lakeapi.load_data(
        table="open_interest",
        start=datetime.datetime.strptime('2022-06-1', '%Y-%m-%d'),
        end=datetime.datetime.strptime('2023-12-02','%Y-%m-%d'),
        symbols=[symbol],
        exchanges=["BINANCE_FUTURES"],
    )
    # print(books.columns)
    # books.set_index('origin_time', inplace = True)
    books = books[['origin_time','open_interest']].reset_index(drop=True)

    books.to_csv(f'/ultimate_oi/{symbol}.csv', index=False)
    return True
    # books[['open_interest','origin_time']][:2000].plot()

Solution: upgraded to the latest version laekapi==0.12.0


sampler.posterior(return_as_dict=True, equal_weight=True) gives TypeError: unhashable type: 'numpy.ndarray'

When sampling posterior into equal weights I get this error.
  File "/home/albert/git/jaxns-cosmology/jaxns_cosmology/nautilus_sampler.py", line 113, in run_sampler
    samples, _, _ = sampler.posterior(return_as_dict=True, equal_weight=True)
  File "/home/albert/miniconda3/envs/jaxns_cosmology_py/lib/python3.10/site-packages/nautilus/sampler.py", line 573, in posterior
    points = points[select]
TypeError: unhashable type: 'numpy.ndarray'

Hi @Joshuaalbert, thanks for raising this issue! Can you tell me which nautilus version you used? It looks like you used the latest release, version 0.7.4. It just so happens, I believe, that I fixed the exact issue you're describing two days ago. Does the issue still occur with the latest version in main?

Hi @Joshuaalbert, I'm assuming the issue is resolved in 1.0. Please re-open this issue if you keep encountering problems. Thanks for reporting the bug!


(docs) Remove note about ssh_authorized_key resources not being purgable

Since you can actually purge those now.
When this is merged, please resolve DOC-1186.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1734/


Fix IndexError when message key is empty

If the message variable is empty, it raises an IndexError.
I had the issue with a 'card.failed-3ds' transaction.

Thanks for the report & fix!
Because our SDK libraries are automatically generated, I couldn't merge your PR directly in the repository, but appropriate changes were added to our generation tools. The version 6.11.1 contains that fix.
Thanks again!


[e2e]: nodejs push main custom_publish slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/7650530831
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.nodejs.push.main.custom_publish.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.nodejs.push.main.custom_publish.slsa3.yml
Trigger: push
Branch: main
Date: Thu Jan 25 06:08:19 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/7664573311
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.nodejs.push.main.custom_publish.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.nodejs.push.main.custom_publish.slsa3.yml
Trigger: push
Branch: main
Date: Fri Jan 26 06:09:11 UTC 2024
Tests are passing now. Closing this issue.


Signmessage command does not work with Segwit address

Signmessage in XSN wallet, should sign the address and verify address owner is true.
This works for legacy addresses however; when using SegWit addresses, it returns the following:
Address does not refer to key (code -3)

Address was unable to be signed as expected with that command.

Yes, tried on two different wallets - same issue.

Wallet version 1.0.17

Windows

The address is a p2sh address, it doesn't have an associated public or private key to sign and verify messages with. The underlying script has a private key associated with it, but not the address itself.
https://github.com/bitcoin/bitcoin/issues/10542


Can you add data statistics based on url to the open API

Can you add data statistics based on url to the open API , or is there any other development method

you can pull URL counts per website using GET /api/websites/{websiteUuid}/metrics, unless you are looking for other type of metrics for URL.


Rich text formatting is not rendered when using the Streaming I/O

Hello there,
In my application I'm building an Excel document row by row and stream each row when it's done. One cell has rich formatting - some words have specific colours. I think I'm doing the right thing but in the Excel document the cell has the value [Object object].
I guess this is the toString() of the object instead of the correct rendering.
Here is a little example that is failing for me:
var fs = require('fs');
var exceljs = require('exceljs');

var stream = fs.createWriteStream('richtTextStream.xlsx');
var wb = new exceljs.stream.xlsx.WorkbookWriter({'stream': stream, 'useStyles': true});
var ws = wb.addWorksheet('rich');
var row = ws.getRow(2);
var cell = row.getCell(2);

cell.value = {
  richText: [
    {
      font: {color: {argb: '99E391'}},
      text: 'This part '
    },
    {
      font: {color: {argb: 'FF6565'}},
      text: ' and that one.'
    }
  ]
};

row.commit();
ws.commit();
wb.commit();

If I use a non streaming workbook and write the file using wb.xlsx.writeFile() then I can see the correct formatting.
Any idea what is going on? Is there some missing rendering logic in the Streaming I/O?
Thanks!

Seems is need to set up "useShareStrings": true in order to make it work
var fs = require('fs');
var exceljs = require('exceljs');

var stream = fs.createWriteStream('richtTextStream.xlsx');
var wb = new exceljs.stream.xlsx.WorkbookWriter({'stream': stream, 'useStyles': true, 'useSharedStrings': true});
var ws = wb.addWorksheet('rich');
var row = ws.getRow(2);
var cell = row.getCell(2);

cell.value = {
  richText: [
    {
      font: {color: {argb: '99E391'}},
      text: 'This part '
    },
    {
      font: {color: {argb: 'FF6565'}},
      text: ' and that one.'
    }
  ]
};

row.commit();
ws.commit();
wb.commit();

make sense as the only way that i can find in the openxml format in order to have different rich text formats in the same cell is by using the sharedString.xml file.
Should useSharedStrings be by default true ?

Hi,
Same, duplicate text with

worksheet.getCell('F1').value = {

  richText: [
    {font: {bold: true}, text: 'N° DOSSIER :'},
  ]
};

worksheet.getCell('H1').value = {
  richText: [
    {font: {bold: true}, text: 'NOM Projet'},
  ]
};

This issue is still not fixed in version 1.0.0 . richText repeated across all worksheets.

I ran into the same problem with all the richText cells have the same value as the first richText cell, any workaround or any news of a possible fix? Version: 4.2.0


Joeychrys/issue11

Hey I created a rating component. Cheers!

Hey @joeychrys @bacali95,
Just wanted to say you're doing an awesome job!
The core team (including myself) from the main library is now working on a new release and after that we'll hop in to help with the docs of the React library.

Awesome! Let me know if you need any help on the docs.

Hey @joeychrys @bacali95,
Just wanted to say you're doing an awesome job!
The core team (including myself) from the main library is now working on a new release and after that we'll hop in to help with the docs of the React library.

Awesome! Let me know if you need any help with the docs.

I just added support to React 18 in the main branch which requires some changes to make it work, can you please rebase and apply what I did to these components?

I sure can!


Update tweepy and chardet Python modules

New Tweepy has Python 3 support. I remember I've updated chardet becase some script required newer version of it, but sadly I can't remember details (it was quite some time ago).

@abbradar fails to build https://travis-ci.org/NixOS/nixpkgs/builds/101463988#L8728

Hm, turses requires turses == 3.3.0 (and as seen on their GitHub1 it really wants 3.1.0). What do we do in those cases?

If different versions are required then we include different versions. You can update tweepy to the latest version and have tweepy_0_3 = tweepy.override for turses.
cc @garbas for turses.


reversed js-file order because setting inheritanceConfig to false cre…

At the moment there seems to be a problem with template-inheritance and the console-command sw:theme:dump:configuration. When i have a theme and i dont want to have inheritanceConfig enabled the order of js-files in the dumped config is reversed.
This means, that plugin js-files are concatted first, which leads to a failing dependency with jquery. Probably im doing it wrong, but this looks like a bug to me.

@benboit, I closed this PR because we are tracking this issue as ticket http://jira.shopware.de/?ticket=SW-12782 in our bug-tracker.
A bugfix is already in QA and should be merged to the 5.1 branch soon.
Thanks for your help.


make cards' layout w/o js usable

Set cards' "position:absolute" with javascript and default to float:left so design will be workable w/o js.

closing b/c scope of CMS has changed to backend-only


Docs consolidation #trivial

This moves the emission/docs/ documentation to docs/, as well as cleanup the existing docs a bit. I moved Emission's old changelog into the docs/ folder too. Many docs still point to the Emission repo, but they all use commit hashes in the URL, so they'll continue to work. It would be tedious to update them to point to Eigen, so I'm skipping that for now.
I also updated the thanks.md file, thanking significant contributors to the app.

awesome work, thanks 🙏


Data loss if same file is saved in multiple fields

If a file is saved in multiple instances, then removing a single instance removes the file and cause 'file not found' error for every other instance.
We need to check if the file is referenced in any other object before deleting.

If I understand what you suggesting, this app would have to load all data in the database and search all of it before deleting any file. Please correct me if I am misunderstanding. But if what I understood is the case then I think this would be firmly a "wontfix".
Multiple instances referencing the same file is not a case that is supported and I would recommend you consider using a file manager app that centralizes references to a file such as django-filer or django-filebrowser or roll your own that supports your use case.

Okay. I understand.
Can you suggest, how can i delete an instance without deleting its associated file?

Will this issue be solved when the “don’t delete default file” feature is merged to master? Or is this a different issue from that?

I dont think so.
Besides, the issue would actually be a 'wontfix' if we have to search every FileField in database before deleting a file.
This case is, i think, not common, that same image would be referenced in multiple models. For this, i have added a setting field which specifies a list of model names for which file should not be deleted. This was the patch for my situation.

resolved with #47


[bazel] Splitting components in sw/device/lib/base

This is a draft as it depends on: https://github.com/lowRISC/opentitan/pull/8762 and tries to address an issue in the comment of that PR that I think should be beyond its scope.


Done, but let me know if you still see anything.


⚠ Secret Website 3 has degraded performance

In c5cba28, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 11936 ms

Resolved: Secret Website 3 performance has improved in e1660fb.


Can't seem to receive /notices

I don't see it anywhere, not in chat nor in "console." :c

Will be fixed when I release the next version of WinIRC


Force single app instance on windows

[X] Complete Mattermost Contributor Agreement
[X] Execute npm run prettify to format codes
[X] Write about environment which you tested

Operating system: Windows 10 64Bit

fixes issue #236
This fix may be problematic, if running multiple instances is required.

Thank you.
This should affect all operating systems, from reading the code. (which is good I guess)

@jgis can you please add your change to the changelog.md?
I just tested windows 10 and it seems to work flawless, good work.
For anyone willing to test, just grab a build from https://circleci.com/gh/mattermost/desktop/575#artifacts

This will address this uservoice also https://mattermost.uservoice.com/forums/306457-general/suggestions/15149430-keep-only-one-running-instance-of-the-desktop

This doesn't affect OS X because originally OS X app works as a single app instance.

This affects Ubuntu Linux 14.04 (64Bit). Same change in behavior as in Windows.

I added a test for this in make-single-instance branch (3b1381b). The test succeeded in CircleCI, which is Linux. So it's ready to merge.
@jgis Please include the commit in this PR.

Fantastic!! Big thank you @jgis!
Would love to see you join our Desktop community channel if you'd like (and already haven't). We have lots of discussion there around the desktop app across the community and Mattermost team.

Thanks.


Add tests for SE2BoxConstraint

We didn't add tests for SE2BoxConstraint when it was introduced in #135.

Resolved by #135.


Nb mb 6298 move detail shipment type

Description
As a Office user I should see a readable Shipment type displayed in the Requested Shipment Cards
Reviewer Notes

Is there anything you would like reviewers to give additional scrutiny?
Setup
Add any steps or code to run in this section to help others prepare to run your code:
make db_dev_e2e_populate
make server_run
make_client_run

Code Review Verification Steps

[ ] If the change is risky, it has been tested in experimental before merging.
[ ] Code follows the guidelines for Logging
[ ] The requirements listed in
Querying the Database Safely
have been satisfied.
Any new migrations/schema changes:

[ ] Follow our guidelines for zero-downtime deploys (see Zero-Downtime Deploys)
[ ] Have been communicated to #g-database
[ ] Secure migrations have been tested following the instructions in our docs

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] User facing changes have been reviewed by design.
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change
this article explains more about the approach used.

Screenshots
If this PR makes visible UI changes, an image of the finished UI can help reviewers and casual
observers understand the context of the changes. A before image is optional and
can be included at the submitter's discretion.
Consider using an animated image to show an entire workflow instead of using multiple images. You may want to use GIPHY CAPTURE for this! 📸
Please frame screenshots to show enough useful context but also highlight the affected regions.

Warnings

:warning:
This PR does not include changes to unit tests, even though it affects app code.

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against 8bf3efa86bd1197529ca1d2582bb8970472f31f4


Not possible to create Machine Learning Registry with existing Storage Account / Container Registry

Bicep version
0.20.4
Describe the bug
According to the documentation (https://learn.microsoft.com/en-us/azure/templates/microsoft.machinelearningservices/registries?pivots=deployment-language-bicep), it should be possible to create a Machine Learning Registry using an already existing storage account and container registry. However, when I try to deploy the ML Registry, I get the following error (and the same for the c ontainer registry, when I use a system-created storage account):
{
    "code": "UserError",
    "message": "Invalid registry request: Creating registry using an existing storage account is not permitted",
    "details": []
}

Is the documentation inaccurate or am I understanding the error message wrong? Should it be possible to create a ML Registry using an existing storage account and container registry?
To Reproduce
A simple bicep template for reproducing the error:
param location string = resourceGroup().location

resource storageAccount 'Microsoft.Storage/storageAccounts@2023-01-01' = {
  name: 'teststorage${uniqueString(resourceGroup().id)}'
  location: location
  sku: {
    name: 'Standard_LRS'
  }
  kind: 'StorageV2'
  properties: {
    publicNetworkAccess: 'Enabled'
  }
}

resource containerRegistry 'Microsoft.ContainerRegistry/registries@2022-12-01' = {
  name: 'testacr${uniqueString(resourceGroup().id)}'
  location: location
  sku: {
    name: 'Standard'
  }
  properties: {
    publicNetworkAccess: 'Enabled'
  }
}

resource mlRegistry 'Microsoft.MachineLearningServices/registries@2023-06-01-preview' = {
  name: 'test-registry'
  location: location
  identity: {
    type: 'SystemAssigned'
  }
  properties: {
    publicNetworkAccess: 'Enabled'
    regionDetails: [{
      location: location
      storageAccountDetails: [
        {
          userCreatedStorageAccount: {
            armResourceId: {
              resourceId: storageAccount.id
            }
          }
        }
      ]
      acrDetails: [
        {
          userCreatedAcrAccount: {
            armResourceId: {
              resourceId: containerRegistry.id
            }
          }
        }
      ]
    }]
  }
}

The docs and the wizard in the Azure portal both indicate that an existing storage account and container registry should be ok.
Could it be that the compiler gets thrown off by the Bicep code indicating you want to create a new storage account instead of importing an existing one?
resource storageAccount 'Microsoft.Storage/storageAccounts@2023-01-01' existing = {
  name: 'teststorage${uniqueString(resourceGroup().id)}'
}

Unfortunately changing the template to reference an existing storage account and container registry didn't help. Even when I hardcoded the resource IDs I got the same error.

Did a quick repro here, and I'm getting similar errors so it's not just your environment.
Running through the wizard in the Portal I noticed a few things though:

You cannot select existing storage or registries. It will be auto-created for you.
The storage and registry created will be created in a new separate resource group. (The ML Registry will be placed in the RG you specify.)

So, if it's a bug in the docs or the resource provider I can't tell, but it's not straight-forward to deploy what looks like valid Bicep.


Purpose of finally

What is the purpose of the finally operator function? These two code blocks seem identical to me:
const someFunction = () => console.log('foo');
someObservable.pipe(
  tap({
    error: someFunction,
    complete: someFunction
  })
)

someObservable.pipe(
  finally(() => console.log('foo'))
)

What purpose is finally serving other than being a more concise version of tap, if any? If that is all it is doing, then maybe we can add something to the documentation to reflect that? Or consider removing finally to make the API more concise? Or perhaps my personal understanding is lacking 😆

Moving to discussion, just now seeing it should be there https://github.com/ReactiveX/rxjs/discussions/6191


Add pylib benchling doc

this looks pretty rough; I think we would need to have some code that parses this and outputs it in a more reasonable format.  this actually exists to some extent in "generateOperatorDocs.py" - I think something similar would need to be run on pylib to generate this page

this looks pretty rough; I think we would need to have some code that parses this and outputs it in a more reasonable format. this actually exists to some extent in "generateOperatorDocs.py" - I think something similar would need to be run on pylib to generate this page

Yeah I agree, it looks cluttered. I'll see what I can do.

@vagie-ganymede - can you push the updated version of the Benchling docs to this site


GRPC transport

The javascript version only provides thrift as a transport option. Is there anything that would prevent using the GRPC version available in other languages?

Thanks for opening this issue! We recently added support for proto over http transport; Is this sufficient?

Hi @ruiaraujo! I'm going to be investigating adding gRPC support.
Two questions:

Are you using the tracer for node, the browser, or both? I anticipate that node may be ready first.
Have you had a chance to try proto over http?

Node.
Not yet because somehow our collector has issues with this and I would like to either see a benchmark or benchmark it myself before changing anything.
The issues we are seeing is some 415 on our collector.

@iredelmeier any chance to work on this? Our node apps are out in the rain compared to the Golang based friends. 😄

Resurrecting this issue! I think with  https://github.com/lightstep/lightstep-benchmarks available now we may be able to run a benchmark between proto and thrift transports. The other thing to consider is the library size, with proto transport included it definitely significantly increases the size. Since you're using the tracer on Node & not browser that may not be relevant.

@sbaum1994 it would be very helpful to have support for the JS version in the benchmark, it would encourage me to showcase some patches we can do the reduce the overhead in the tracer implementation.


Modify Thread Content

I want to run my project on CircleCI and get the reports on Slack. I want to get the failures in a thread since they take too much space. I can edit fails[] in the layout/block generator under src, but it doesn't affect thread messages. I have to edit node modules, but that doesn't work with CircleCI.

Altering the layout of the threads is currently not supported.  I will look into supporting this.


Review-Fuction: Regression dialogue, Precision disapears after last reviewed segment of species

Describe the bug
A clear and concise description of what the bug is.
When reviewing segments with the review function of the Birdnet-Analyzer, after being done with the last segment of one species/class, the small box in the regression-window showing the precision values for different thresholds disappears.
That happens, once you are done with all the segments available for a species/class. Then the little box on the right, down at the "Regression" part, disappears.
I am unsure if that behaviour is on purpose.
Additionally, it would be very convenient, to save these results in some form of table. As I understand it now, one must at least screenshot the result, to save it for later.
To Reproduce
Steps to reproduce the behaviour:
For the GUI:

Go to Review
Click on select input directory
choose a directory with segments extracted in advance
review segments for a given species
see the "Regression" pictured below, with the precision values for different thresholds
review until you reach the last segment of the given species
In the "Regression" figure, see that the precision values for different thresholds are gone now.
For the CLI provide us with the arguments used.

Expected behavior
In "Regression" figure, t the precision-values for different thresholds should remain, after review of last segment.
Desktop:

OS:  Win10
Version 1.3.1

This might happen because we hide the threshold values if the are > 1.
So if the last review-segment pushes all the thresholds above 1 the box will disappear.
Otherwise i could not reproduce this issue.
Saving the result as a table is a good idea.
(Other then screenshotting the result you can also reopen the folder with the BirdNET GUI to access the results, but i see that this is not really convenient)


displayName not working with styled-components/macro

Hi there!
styled-components/macro allows the developer to enjoy the benefits of the displayName property without needing to tinker with a babel config / eject a create-react-app application.
Here's a CodeSandbox showing the issue:
https://codesandbox.io/s/cold-flower-5orvj
Expected behavior: The <Thing> component should render as a div with class="App__Thing-sc-123abc"
Actual behavior: The <Thing> component is rendering as a div with class="sc-abc123".
I checked, and this appears to have broken with the 2.0.0 release; it worked immediately previous, in 1.13.3.
Let me know if I can do anything else to help!

This seems to be fixed in the latest release

@joshwcomeau Fixed in v2.0.6?

I tried v2.0.7,  and I got the same error

it's working now in v2.0.7!


A better hook for checking new windows

Article:
https://www.codeproject.com/Tips/1057230/Windows-Resize-and-Move
Sources:
WindowsMover-src.zip
private readonly int windowMessage_ShellHook;
public WindowsHookForm()
{
    windowMessage_ShellHook = WindowsApi.RegisterWindowMessage("SHELLHOOK");
    WindowsApi.RegisterShellHookWindow(this.Handle);
}

public event Action<IntPtr> WindowCreatedEvent;
public event Action<IntPtr> WindowActivatedEvent;
public event Action<IntPtr> WindowDestroyedEvent;

protected override void WndProc(ref Message message)
{
    if(message.Msg == windowMessage_ShellHook)
    {
        WindowsApi.ShellEvents shellEvent = (WindowsApi.ShellEvents)message.WParam.ToInt32();
        IntPtr windowHandle = message.LParam;

        switch(shellEvent)
        {
            case WindowsApi.ShellEvents.HSHELL_WINDOWCREATED:
                if(WindowCreatedEvent != null)
                    WindowCreatedEvent(windowHandle);
                break;

            case WindowsApi.ShellEvents.HSHELL_WINDOWACTIVATED:
                if(WindowActivatedEvent != null)
                    WindowActivatedEvent(windowHandle);
                break;

            case WindowsApi.ShellEvents.HSHELL_WINDOWDESTROYED:
                if(WindowDestroyedEvent != null)
                    WindowDestroyedEvent(windowHandle);
                break;
        }
    }
    base.WndProc(ref message);
}

Additional source:
https://docs.microsoft.com/en-us/windows/desktop/api/winuser/nf-winuser-registershellhookwindow

The event messages received are only those sent to the Shell window associated with the specified window's desktop. Many of the messages	are the same as those that can be received after calling the SetWindowsHookEx function and specifying WH_SHELL for the hook type. The difference with RegisterShellHookWindow is that the messages are received through the specified window's WindowProc and not through a call back procedure.

https://docs.microsoft.com/en-us/windows/desktop/api/winuser/nf-winuser-setwindowshookexa

https://www.meziantou.net/detect-the-opening-of-a-new-window-in-csharp.htm


Support C++ exceptions

Using our RTTI information we can identify and symbolize both the throw metadata and the attached catch routine. This should be done in two passes. First pass should symbolize the throw metadata and the second pass should rewrite callsite to illustrate the branching, possibly with an xref to the catch routine and for the non-catch routine we should just be able to make the throw function returnable.
The end result should look something like this, the throw is highlighted orange to illustrate it could jump to the catch routine instead of continuing. Not sure how "correct" this is.

We could also inline the non-exceptional path like so:


(SP: 2) Change working hours ProfileItem

Change Set up your working hours to Education add description on your own

Design needed @Nastia197

This issue is not relevant


Need to Bundle All IconPacks?

Is your feature request related to a problem? Please describe.
I'm working on building a final version of the product I'm working on which utilizes MahApps Metro IconPacks. My application only uses the FontAwesome and Material icon packs but I noticed that I need every IconPack DLL for the application to run. This is very annoying since I am required to include unnecessary DLLs in my application's install for icon packs that the application doesn't use.
Describe the solution you'd like
Only need to include DLLs for IconPacks being used.
Describe alternatives you've considered
At the moment I've downgraded to version 2.3.0 since that includes both FontAwesome and Material but it would be nice if this was a feature so I can start using the newer versions again.
Additional context
Not really, love this library and it's been great to use during development of my application!

@ekarkosApollo You can also use only the NuGet packages for this 2 Icon Packs.
So de-install the all-in IconPacks NuGet package and use only these 2 ones

https://www.nuget.org/packages/MahApps.Metro.IconPacks.FontAwesome/
https://www.nuget.org/packages/MahApps.Metro.IconPacks.Material/

Ah perfect I hadn't seen that before... Thanks!

@ekarkosApollo np. The wiki should help here (I hope so) https://github.com/MahApps/MahApps.Metro.IconPacks/wiki/Install

Single .dll for all icons may look good!

Hi @Azhe403 ,
as @punker76 already mentioned you can just install the ones you need. I really like the flexibility here and I don't see a need to change this. If you don't want to ship too much files to the customer just select the ones you need or ship a single exe-File.
Happy coding,
Tim


[Angular 15] ng test: Add CLI flags to generate unit test report(xml) and code coverage report(Cobertura/JaCoCo) for CI

Command
test
Description
As per Angular CLI v15 release notes, karma.conf.js file will no longer be generated by default for newly generated applications/libraries.
What is the expected workflow for a developer to generate unit test report(xml) and code coverage report(Cobertura/JaCoCo) for Azure DevOps CI?
Below are the tasks available to collect unit test and code coverage reports in Azure DevOps:

Publish Test Results task

Publish Code Coverage Results task

Previous request: #21219
Describe the solution you'd like
Provide CLI flags to generate unit test and code coverage reports
ng test --project command-center --no-watch --code-coverage --code-coverage-report-format cobertura --tests-report-format junit
Or, provide a command to generate a default karma.conf.js file
ng generate tests-config --project command-center
Describe alternatives you've considered
No response

In this case since you require customization it’s recommended that you create your own Karma configuration file.

How should I generate my own Karma configuration file (Assuming I have no knowledge of previous angular releases where a default karma.conf.js file was present)?
This is how a tsconfig.json file can be generated:

tsc --init

We are exploring various possibles to generate such config. Having it in a hidden directory, using a schematic, or coping it from a documentation portal.
Whichever approach we go for will be documented before the final v15 release.


include math.h

Otherwise, compilation fails with error: call of overloaded 'abs(double)' is ambiguous

This has already been fixed on main (just a few minutes ago). I'll release 10.6.1 in a day or two.


fixed spelling in readme

Thank you very much. I actually disagreed with the first two until I looked them up and double checked.


Some applications of Stacks and Linked List

Hi guys as you know linked list and Stack  are abstract data types with its own applications and types , I have uploaded some applications of stack and linked list
Hope this will help you guys , Thank you...!

Bro plz merge it quickly bro as i am late for the fest , 🙏Plz add the hacktoberfest-accepted lable so as to not get excluded

files coded in c are not accepted

can you reopen it bro i will upload something in c++


Shopware 5 Rest API Filter Orders

Description
How to filter orders based on complex conditions
I need to filter orders from Shopware 5 by applying complex conditions. For example: I want to filter orders whose create time is greater than x date and have status either 1 or 250. I tried using the filter based on query params (?filter[0][property]=xx&....) but seems do not work. Is there any way to get this working?

Hi @tekbird,
this is the shopware 6 repository. You can find the shopware 5 sources here:
https://github.com/shopware/shopware

HI @OliverSkroblin
I could not find a place to create issue in Shopware5 repository.

development is also for shopware 6.
Please use the issue tracker for shopware 5 issues: https://issues.shopware.com/


Fix Typo

Fixes #40041

Internal previews

📄 File
🔗 Preview link

docs/csharp/language-reference/compiler-messages/static-abstract-interfaces.md
Static abstract and virtual interface member errors and warnings

Closed in favor of #40042


Fix NPE in notification controller

Fixes npe with notification controller when no notifications are returned from the service

Coverage remained the same at 35.857% when pulling 7b1986ca3c6d1f089bae2e4f3c7e7c3b05940f87 on npe-notification-controller into f1ed5bd053811bc1e482a472f1bfcb7e9fd8b695 on master.


Implements BIG sync commands in the ULL

Implements handling of the BIG sync commands in the ULL.
LLL support and handling of ACAD (biginfo) remaining.

Buildkite fails on a non-related regression as far as I can tell. To be examined.


Problem about tissue segmentation

Tried to perform tissue segment as listed here
https://stereopy.readthedocs.io/en/latest/Tutorials/Tissue_Segmentation.html
But when I run from 'stereo.image.tissue_cut import SingleStrandDNATissueCut' , an erro occurs.
The necessary packages mentioned in the tutorial have been installed.
Traceback as follow:
ModuleNotFoundError                       Traceback (most recent call last)
File ~/miniconda3/envs/st/lib/python3.8/site-packages/stereo/image/init.py:6
5 from .pyramid import merge_pyramid, create_pyramid
----> 6 from .segmentation.segment import cell_seg
7 from .segmentation.seg_utils.v3 import CellSegPipeV3 as cell_seg_v3
File ~/miniconda3/envs/st/lib/python3.8/site-packages/stereo/image/segmentation/segment.py:7
6 from stereo.constant import VersionType
----> 7 from stereo.image.segmentation.seg_utils.v1 import CellSegPipeV1
8 from stereo.image.segmentation.seg_utils.v1_pro import CellSegPipeV1Pro
ModuleNotFoundError: No module named 'stereo.image.segmentation.seg_utils'
During handling of the above exception, another exception occurred:
ImportError                               Traceback (most recent call last)
Cell In[4], line 1
----> 1 from stereo.image.tissue_cut import SingleStrandDNATissueCut
File ~/miniconda3/envs/st/lib/python3.8/site-packages/stereo/image/init.py:24
11 except ImportError as e:
12 #     errmsg = """function merge_pyramid, create_pyramid, cell_seg, cell_seg_v3, tissue_cut,
13 # cell_seg_deepcell is not import at stereo.image module.
(...)
21 # ************************************************
22 #     """
23     errmsg = "Some necessary modules may not be installed. More information please refer to the documentation."
---> 24     raise ImportError(errmsg)
ImportError: Some necessary modules may not be installed. More information please refer to the documentation.

It seems that stereopy was not installed fully, you can try to reinstall stereopy:

pip install stereopy==1.5.0 --force-reinstall --no-cache-dir

Then, install the necessary dependencies.


Stop / finalize server instance on python delete and process end

📝 Description of the feature
Hi,
it would be great, if you could exit the server instance (check in license) once one uses the python del operator and once the python process finishes.
💡 Steps for implementing the feature
No response
🔗 Useful links and references
No response

Hi @dkowollik do the following two options not deliver what you need in this instance? Thanks

Client.exit
Launching using context

Hi @waltersma,
You are right, this is doing the job, if everything works fine. Client.exit() does not work, if an error occurs during the script execution at an earlier stage. The server process stays alive and keeps the license. If that happens, one has to search for the process and kill it manually. As a workaround for this, I implemented the following for now:
if __name__ == '__main__':

    prime_client = prime.launch_prime(prime_root=r'C:\Program Files\ANSYS Inc\v242\meshing\Prime')
    try:
        main(prime_client)
        prime_client.exit()
    except:
        t, v, tb = sys.exc_info()
        prime_client.exit()
        six.reraise(t, v, tb)

It seems like each pyansys package uses a different interface for launch and exit (for example: launch, start, stop, restart, exit, with). It would be great to have identical interfaces to launch and close the server. Other pyansys packages also kill the server process once the script ends, an error occurs or the client instance has been deleted. To be more precise the server process is stopped, if it was launched during the session by the client.
There exists a generic pyansys interface that uses the python builtin weakref finalize implementation, see:
https://github.com/ansys/ansys-tools-local-product-launcher/blob/38840c7f8bd01ccc529d6b76d151c1f503d0cb4a/src/ansys/tools/local_product_launcher/product_instance.py#L52
I talked to @greschd, who gave me the above link and implemented the plugin for pyACP. He can help you and explain this much better than I can.
Thanks.

Hi @waltersma,
There is another advantage to use this finalize approach.


Visibility into the concurrent edge sessions and bandwidth utilization through the customer edge routers

Need visibility into concurrent edge sessions, new edge sessions initiated per second, bandwidth utilization  ( Tx & Rx ), etc through a specific customer edge router as a way to measure traffic on a customer edge router for capacity planning.

Bandwidth utilization metrics are all there. We have ingress/egress/fabric/rx/tx for each router.
For new edge sessions, I'm guessing you mean circuits, not sessions (please correct me if I'm wrong). We have circuit events which list the path, including the initiating router, so you should be able to derive a new circuits per second for a given router.
Concurrent circuits we don't currently have a metric for. Which of the following are you interested in?

Total concurrent circuits on this router
Concurrent circuits initiated on this router
Concurrent circuits initiated or terminated on this router

Let me know, and we can update this issue accordingly.

@plorenz the following may be needed.
Total concurrent circuits on this router ( initiated + terminated) at any given second
Count of new circuits initiated on this router per second
Count of new circuits terminated on this router per second
Total concurrent circuits terminated on this router at any given second
Total concurrent circuits initiated on this router at any given second

Ok, I think that's doable. Moving this to the fabric project, as that's where the work need to be done


Get all tests to pass locally with no special configuration

Description
FIXES: #1107
Tests
python -m pytest and pytest should both now work when run from repository root.
Checklist
Before submitting this PR, please make sure (put X in square brackets):

[x] I have performed a self-review of my code.
[x] I have necessary comments in my code, particularly in hard-to-understand areas.
[x] I have run end-to-end tests tests and provided workload links above if applicable.
[x] I have made or will make corresponding changes to the doc if needed.

Important: do not merge yet! - I want to solve your grain issue [ https://github.com/AI-Hypercomputer/maxtext/actions/runs/12421511705/job/34681866302 ] first; which I think this commit fixed https://github.com/google/grain/compare/ce66a97aa88cc4618001ca9ba80c119178f8af4a...08f7cb175da8ae3c150027500a47797a8f9ddd6c
EDIT: Can you ping the guys? - Looks like they just need to release anew google/grain/issues/662

The grain issue might be solved by https://github.com/AI-Hypercomputer/maxtext/pull/1109

Good to see. Also related is your recent PR #1113


Docker Daemon process Consuming complete host memory 15gb RAM

What kind of request is this (question/bug/enhancement/feature request):
Issue with Memory consumption on rancher hosts
Steps to reproduce (least amount of steps as possible):
We are using Rancher v1.6.18. We have around 220 stacks running in rancher farm.
Infra Details:
3 Rancher Masters
1 Rancher Db Host
70 Rancher Hosts
It is seen that the Docker process consumes complete memory of the Rancher host. This make the Rancher hosts unresponsive.
In order to resolve this we need to clean up the whole Rancher Hosts which include:
Stopping all the stacks
Removing Docker images, containers, Volumes
Removing /var/lib/docker, /var/lib/rancher
Reboot the Rancher host
Adding back to Rancher farm
Current Resource Limits:
Memory: 13000
mCPU: 4
Host Details:
16GB RAM
Rhel 7
Can you let us know how can we restrict the memory for docker process on Rancher Host?
Is there a way we can Manage the CPU and Memory usage?
Also should we upgrade to 1.6.26 version? Will this help resolve the Memory issue on the rancher hosts?
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
Rancher 1.16.8
Installation option (single install/HA):
HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):
Custom
Machine type (cloud/VM/metal) and specifications (CPU/memory):
VM
Kubernetes version (use kubectl version):
We are using Cattle

(paste the output here)

Docker version (use docker version):
1.13.1

(paste the output here)

Don't see why an issue is needed for this, let's continue in https://forums.rancher.com/t/rancher-v1-6-18-facing-memory-issues-on-rancher-host/14030


⚠ Bofrost.at has degraded performance

In d46dac5, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2199 ms

Resolved: Bofrost.at performance has improved in afdf888.


Can't install

Does not install. If does with --legacy-peer-deps flag > breaks code

This is not a bug. This is due to the peer deps requiring React 16. It looks like you are attempting to install with React 17+. I will test and release a patch to include react 17 and 18 as peer dependencies.
See package lock: https://github.com/smhaley/react-text-annotate-blend/blob/main/package.json#L43

Following up. I've released a patch on npm: 1.0.5.
This patch increases the peerDeps to cover React 16, 17, and 18. This should fix your install issues.
I've also includes a directory with minimal examples to help get you started: https://github.com/smhaley/react-text-annotate-blend/tree/main/examples

Great. Will update!


No support for Ubuntu 19.04 yet - does not install because of broken dependencies

Problem encountered on https://dotnet.microsoft.com/download/linux-package-manager/ubuntu18-04/sdk-current
Operating System: Linux Ubuntu 19.04 - x64
After performing installation commands, I get the following output:
The following packages have unmet dependencies:
dotnet-sdk-2.2 : Depends: aspnetcore-runtime-2.2 (>= 2.2.4) but it is not going to be installed
Depends: dotnet-runtime-2.2 (>= 2.2.4) but it is not going to be installed
E: Unable to correct problems, you have held broken packages.

Duplicate of #2526


Add namespace to kustomization.yaml to reduce manual steps to recreate px-finagle demo

The instructions in this repo's README required some hidden steps in order to recreate the px-finagle demo manifest. When kompose is run it erases the namespace: px-finagle defined in each deployment and service config in the kubernetes/ directory.
This change instructs kustomize to do the namespace annotation, so the files don't need to be manually changed after kustomization.
Testing done

Ran kompose and kustomize and diffed it against the pixie cli manifests (P391)
Validated that without this change the resulting manifests are missing the px-finagle namespace (P392)

@vihangm can you review this since it seems you reviewed (or merged) the change to add the kustomization.yaml?


storing block by hight instead of hash

Description

Storing block by hight instead of hash
This PR save the blockchain size more than 5%

Checklist

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] My code follows the code style of this project.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.
[ ] CHANGELOG is updated.

Don't we need to update wallet history based on height instead of hash?

@nagarajmanjunath Thanks


[Request for Comment] Swallow up bad queries

In #230, an operator was added that allowed a "partial" transformation of components, with the caveat that a user sending a bad query to a child component would be interpreted into HaltHF (a runtime error); in the PR thread, we discussed the possibility of changing this so that it would instead result in Nothing getting returned from query. This is an implementation of that idea (thanks to @garyb for your help!).
Now, I am not certain this is actually a good idea, which is why the PR is entitled "Request for Comment". My concern is that this may obscure the correctness conditions of a component and hide actual bugs in code (if you agree with me that a bug in a program should be fatal, rather than being "handled" in-program). However, one might argue that we have already made this design decision by having query return Maybe rather than have a fatal error in case the child component is not present, and so we should perhaps be consistent. What do people think?

Would be good to get the thoughts of those of you out there who have used this a bit @natefaubion @parsonsmatt @themoritz @cdepillabout
And @jdegoes of course, if you're not in full holiday mode yet :smile:

I think the distinction between "this query will can't ever possibly work" and "this query doesn't work right now" is important. I think an exception is fine. Handling this case is not something that should be happening in a shipping app.

Thanks for your input, @natefaubion; I think I agree.

@garyb and I have also discussed this a bit, and I think we should not proceed. The difference between an incorrect component and an expected failure case is important, and I don't want to obscure it.

Agreed -- Nothing carries much less information than an exception and is harder to debug in the case where I did something wrong.

If this is about broken-by-definition queries, then throwing an exception is correct (it can always be caught and dealt with). OTOH, query returning Maybe makes perfect sense since components may or may not exist at the time they are queried and client-code needs to be able to decide what that means.

@jdegoes Thank you for clarifying the status of not-present components!


exclude precision and scale from @Column for Double fields

According this question on stackoverflow please correct DefaultCustomizations.xml to exclude precision and scale property from @Column for Double fields.

Release 2.1.1 includes this commit 9ee6d18 to remove the scale="10" precision="20" attributes from the xsd:float and xsd:double customizations.

thank you!


C++ client-server hello world

Hi,
A have a newbie request:  I'm unable to find documentation describing what I need to do to compile and link a service that I write.
I see the code in test/cpp/*, and interop/{client,server} are particularly helpful as an example.  But there are no Makefiles  there, and the top-level file is 10K lines :(
-Mark

By the way, we've made efforts to improve that documentation, and created example projects here: https://github.com/grpc/grpc-common/tree/master/cpp - could you see if this seems better for you or if you still have comments about it ?

done