No support for billing_portal webhook events

Describe the bug
I get the following error
Dec 06 03:52:39 ubuntu-4gb-hel1-1 llm-templates[68831]:     Error("unknown variant `billing_portal.session`, expected one of `account`, `capability`, `application`, `application_fee`, `fee_refund`, `balance`, `bank_account`, `billing_portal.configuration`, `card`, `charge`, `checkout.session`, `coupon`, `customer`, `discount`, `dispute`, `file`, `invoice`, `invoiceitem`, `issuing.authorization`, `issuing.card`, `issuing.cardholder`, `issuing.dispute`, `issuing.transaction`, `mandate`, `payment_intent`, `payment_link`, `payment_method`, `payout`, `person`, `plan`, `price`, `product`, `promotion_code`, `quote`, `refund`, `review`, `setup_intent`, `subscription`, `subscription_schedule`, `tax_id`, `tax_rate`, `test_helpers.test_clock`, `topup`, `transfer`", line: 9, column: 40),

in the following code
    match Webhook::construct_event(&payload.contents, stripe_signature.signature, &endpoint_secret) {
        Ok(event) => {
            if let Err(e) = process_stripe_event(&event, &mut connection) {
                eprintln!("Failed to process stripe event: {:?}", e);
                return Status::InternalServerError; // Log and handle processing error
            }
            Status::Ok
        }
        Err(e) => {
            dbg!(e);
            Status::BadRequest
        }
    }

To Reproduce
Have stripe send a billing portal webhook
Expected behavior
no Error
Code snippets
No response
OS
MacOS
Rust version
cargo 1.74.0-nightly (59596f0f3 2023-09-29)
Library version
0.26.0
API version
0.26.0
Additional context
No response

I think this issue is still relevant. I just got the same error with version 0.37.0.
It looks like the default / #[serde(other)] is not working.
According to the documentation of #[serde(other)] it is only for "[...] a unit variant inside of an internally tagged or adjacently tagged enum".


RubyInstallerTeam.RubyWithDevKit.3.2 version 3.2.2-1

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.
Resolves #110553

@stephengillie why the pull request keep failing.

@stephengillie, This pull request keep gave me Internal-Error-Dynamic-Scan

@stephengillie, This pull request keep gave me Internal-Error-Dynamic-Scan

Hi @BrandonWanHuanSheng,
I've been working to manually validate. This package installs numerous applications, and running them simultaneously for the Defender scan is causing the manual pipeline VMs to run out of memory. Sorry for the delay - I'll get them validated before my end of day today.


Kylin-3409 NPE on Metrics creation.

Error occurs when queryType != OLAPQuery.EnumeratorTypeEnum.OLAP (i.e. LOOKUP_TABLE...).
Handle this case.
KYLIN-4161 - duplicate i think.

Can one of the admins verify this patch?

Codecov Report

Merging #942 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master     #942      +/-   ##
============================================
- Coverage     25.57%   25.56%   -0.02%
+ Complexity     6139     6138       -1
============================================
  Files          1412     1412
  Lines         85072    85075       +3
  Branches      11928    11930       +2
============================================
- Hits          21760    21752       -8
- Misses        61206    61214       +8
- Partials       2106     2109       +3

Impacted Files
Coverage Δ
Complexity Δ

.../apache/kylin/rest/metrics/QueryMetricsFacade.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...he/kylin/dict/lookup/cache/RocksDBLookupTable.java
72.97% <0%> (-5.41%)
6% <0%> (-1%)

.../apache/kylin/cube/cuboid/TreeCuboidScheduler.java
63.84% <0%> (-2.31%)
0% <0%> (ø)

...rg/apache/kylin/cube/inmemcubing/MemDiskStore.java
69.3% <0%> (-0.92%)
7% <0%> (ø)

Continue to review full report at Codecov.

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

Pull Request Test Coverage Report for Build 5223

0 of 21 (0.0%)  changed or added relevant lines in 1 file are covered.
6 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.005%) to 28.051%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

server-base/src/main/java/org/apache/kylin/rest/metrics/QueryMetricsFacade.java
0
21
0.0%

Files with Coverage Reduction
New Missed Lines
%

core-dictionary/src/main/java/org/apache/kylin/dict/lookup/cache/RocksDBLookupTable.java
1
81.08%

core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java
5
77.81%

Totals

Change from base Build 5214:
-0.005%

Covered Lines:
23864

Relevant Lines:
85075

💛  - Coveralls

Check list

[ ] Step 1. [OPTIONAL]Understand background/root cause/design basically(one hour EST) . Its issue type is

[ ] bug fix
[ ] new feature
[ ] enhancement

[ ] Step 2. Test cases are designed and documented (30 minutes EST).
[ ] Step 3. Prepare specific env, for example:

mock data (maybe some specific data type)
test env (maybe install a RDBMS instance).

[ ] Step 4. Verify and make sure test cases passed.
[ ] Step 5. Paste manual important steps and screenshots here (20 minutes EST).

If you find difficulty in pick most important evidence, please attach diagnosis package.

[ ] Step 6. Do more check in user perspective (20 minutes EST)

[ ] Doc need be updated? And if it is updated? Ask help for release manager if so.
[ ] If it is a breaking change so we should notify Kylin community? Ask help for release manager if so.

[ ] Step 7.  Summarize this test (20 minutes EST).

Total estimate

4 hours for small issue from optimistic estimate.

Note
If you find some unexpected and unrelated error/mistake, please DO report it if it is truly a mistake, I think we may research and fix it in the future.
If you find background information/root cause analysis is not complete or ambiguous, please try to contact to author or do a quick research and record what you find. It is a good chance to learn something interesting.
If you find it is not easy to design testcase, please notify release manager.

Design test cases
Prepare test env

create cube and build segment (2012-01-01 ~ 2013-01-01)

dimension : price, part_dt
measure : sum(price)

Case-1
Make sure the sql result is null and then make sure there's no exception about "Write metric error.java.lang.NullPointerException" in kylin.log
select sum(price) from kylin_sales where part_dt > '2014-01-01'

Prepare data and env

[x] Built on commit_SHA : d14eef5fee3ef5a305b35b9121bde28fc2400d9d

[x] Hadoop env : CDH 5.7

[x] Just sample cube is OK.

Verify the result
Check the kylin.log and then make sure there's no exception about "Write metric error.java.lang.NullPointerException" in kylin.log

Sumary
From the query log, we can see that the metrics are updated correctively.


[Bug]: Chromium can not be found

Bug expectation
I wanted to deploy my node app on a server (render.com). So I installed chromium via npm i chromium.
Locally works everything great but after deployment via render.com I get this error message:
{"message":"Could not find Chromium (rev. 1083080). This can occur if either\n 1. you did not perform an installation before running the script (e.g. npm install) or\n 2. your cache path is incorrectly configured (which is: /opt/render/.cache/puppeteer).\nFor (2), check out our guide on configuring puppeteer at https://pptr.dev/guides/configuration."}
Bug behavior

[ ] Flaky
[ ] PDF

Minimal, reproducible example
const browser = await puppeteer.launch();

Error string
{"message":"Could not find Chromium (rev. 1083080). This can occur if either\n 1. you did not perform an installation before running the script (e.g. npm install) or\n 2. your cache path is incorrectly configured (which is: /opt/render/.cache/puppeteer).\nFor (2), check out our guide on configuring puppeteer at https://pptr.dev/guides/configuration."}
Puppeteer configuration
No response
Puppeteer version
19.6.1
Node version
16
Package manager
npm
Package manager version

Operating system
Linux

Looks like a duplicate of https://github.com/puppeteer/puppeteer/issues/9581


Etch-a-sketch using js

I would like to work on this issue as a part of gssoc'21.


Unclear how client is connecting?

Hi there.. I've been trying to use your example code with the latest version of GRPC and GRPC-web. I've got everything set up the same as you and am stuck trying to understand how this works:
https://github.com/johanbrandhorst/grpcweb-wasm-example/blob/master/frontend/frontend.go#L46
You provide no address in your example. When I do this I run into the following error:
initializing client failed: could not connect to gRPC server: failed to build resolver: passthrough: received empty target in Build()

Makes sense. How is it meant to connect without knowing the address? Bit confused about how this could work in your example at all.
So I try to address the issue by supplying the actual server address (ie. localhost:1000). The obvious problem here is that the client is using standard grpc.. which uses sockets. And sure enough; it gives me an error indicating as much:
rpc error: code = Unavailable desc = connection error: desc = "transport: Error while dialing dial tcp: lookup localhost: Protocol not available"

I'm at a bit of a loss trying to understand how your example is meant to work unfortunately.
I realize this is a 4-year-old repo, but sadly I can't find any other resources that are covering what I am trying to do (ie. connect from go-wasm via grpc). In fact, the grpc-web repo seems to be entirely focused on facilitating grpc connections via the web from the server. I see no mention of doing it from the client side in Go (only typescript).
Hopefully, you can provide some insight. Thank you!

 I apologize for the confusion. My presentation at dotGo 2019 talks a bit more about how this works but the basic thing you're missing is that you need to use a specific branch of grpc-go that I created in 2019. You can see it defined here:
https://github.com/johanbrandhorst/grpcweb-wasm-example/blob/7ffce7e6aa30b47cf59bd277f2d849093e330a63/Gopkg.toml#L18-L21
The branch is still available here: https://github.com/johanbrandhorst/grpc-go/tree/add-grpc-web-client. You should be able to get this all building by ensuring that your go.mod points to this branch. The reason Dial("") works in this case is because it will use the address of the website that it is hosted on. This works in JS, so it works in Go wasm. It will also avoid any CORS or HTTPS issues, which is nice :).
With all that said, this demo never made its way into the mainstream grpc-go distribution, mainly because there (AFAIK) still is no "transport" interface in which you could switch in a grpc-web transport, and as such, shouldn't be used for anything serious. But it's a cool tech demo!

I just pushed up a commit that migrates to modules, that should allow you to run the make serve and have it all work :). I should migrate the protobuf generation and stuff too but it'll take a bit longer. Feel free to reach out on Gophers Slack if you want to chat more about this.

Thanks for the swift response @johanbrandhorst! I indeed missed that detail. I'll have a play with your fork to see if I can get it working with the latest grpc version. Much appreciated! 🙏

I went and updated the protobuf generation and asset embedding too, feel free to have a look it should be significantly easier to use. Good luck adapting the code to the modern codebase, it was a hack back then and it'll be the same today I expect 😁.

Above and beyond, much appreciated @johanbrandhorst, thank you!


Adding --cluster flag causes traefik to crash

Do you want to request a feature or report a bug?
Bug
What did you do?
Added --cluster flag to my kubernetes ingress configuration. I tried both with and without --cluster.node flag and I tried running locally with --cluster on my Mac.
What did you expect to see?
Traefik starting in cluster mode.
What did you see instead?
Traefik crashing with a traceback. Tried this on both my kubernetes cluster and on my local OS X system with similar results.
Output of traefik version: (What version of Traefik are you using?)
1.6.5-alpine from docker on Kubernetes, 1.6.4 from Homebrew on OS X
What is your environment & configuration (arguments, toml, provider, platform, ...)?
On kubernetes:
kind: DaemonSet
apiVersion: extensions/v1beta1
metadata:
  name: traefik-ingress-controller
  namespace: kube-system
  labels:
    app: traefik-ingress-lb
spec:
  minReadySeconds: 5
  updateStrategy:
    type: RollingUpdate
  template:
    metadata:
      labels:
        app: traefik-ingress-lb
        name: traefik-ingress-lb
    spec:
      serviceAccountName: traefik-ingress-controller
      terminationGracePeriodSeconds: 60
      hostNetwork: true
      containers:
      - name: traefik-ingress
        image: traefik:1.6.5-alpine
        ports:
        - name: http
          containerPort: 80
          hostPort: 80
        - name: admin
          containerPort: 8080
        securityContext:
          privileged: true
        args:
        - --api
        - --ping
        - --cluster
        - --consul
        - --consul.endpoint=$(CONSUL_SERVICE_HOST):8500
        - --kubernetes
        - --logLevel=INFO

On OS X I just tried to run traefik --cluster
If applicable, please paste the log output in DEBUG level (--logLevel=DEBUG switch)
kubectl logs
$ kubectl -n kube-system logs traefik-ingress-controller-vx2fm
time="2018-07-12T21:08:47Z" level=info msg="Traefik version v1.6.5 built on 2018-07-10_03:54:03PM"
time="2018-07-12T21:08:47Z" level=info msg="\nStats collection is disabled.\nHelp us improve Traefik by turning this feature on :)\nMore details on: https://docs.traefik.io/basics
/#collected-data\n"
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x10 pc=0xd9867e]

goroutine 1 [running]:
github.com/containous/traefik/cluster.NewLeadership(0x23c3bc0, 0xc4202e8580, 0xc4202d4900, 0x0)
        /go/src/github.com/containous/traefik/cluster/leadership.go:35 +0x6e
github.com/containous/traefik/server.NewServer(0xc4200b2ea0, 0x0, 0x100, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
        /go/src/github.com/containous/traefik/server/server.go:126 +0x57c
main.runCmd(0xc4201e4000, 0x0, 0x0)
        /go/src/github.com/containous/traefik/cmd/traefik/traefik.go:180 +0x330
main.main.func1(0x4808101, 0xc4206149e0)
        /go/src/github.com/containous/traefik/cmd/traefik/traefik.go:54 +0x42
github.com/containous/traefik/vendor/github.com/containous/staert.(*Staert).Run(0xc42002e920, 0x23a4180, 0xc4206149e0)
        /go/src/github.com/containous/traefik/vendor/github.com/containous/staert/staert.go:79 +0x2e
main.main()
        /go/src/github.com/containous/traefik/cmd/traefik/traefik.go:142 +0x1b13

OS X output
$ traefik --cluster
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x10 pc=0x19a362e]

goroutine 1 [running]:
github.com/containous/traefik/cluster.NewLeadership(0x2fd25e0, 0xc4200df600, 0xc420620de0, 0x0)
        /private/tmp/traefik-20180615-52269-s43ora/src/github.com/containous/traefik/cluster/leadership.go:35 +0x6e
github.com/containous/traefik/server.NewServer(0xc4204aa350, 0x0, 0x100, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
        /private/tmp/traefik-20180615-52269-s43ora/src/github.com/containous/traefik/server/server.go:125 +0x57c
main.runCmd(0xc42020a000, 0x0, 0x0)
        /private/tmp/traefik-20180615-52269-s43ora/src/github.com/containous/traefik/cmd/traefik/traefik.go:180 +0x330
main.main.func1(0x2b2a8c0, 0xc420621ea0)
        /private/tmp/traefik-20180615-52269-s43ora/src/github.com/containous/traefik/cmd/traefik/traefik.go:54 +0x42
github.com/containous/traefik/vendor/github.com/containous/staert.(*Staert).Run(0xc4206206c0, 0xc42020a000, 0xc420621ea0)
        /private/tmp/traefik-20180615-52269-s43ora/src/github.com/containous/traefik/vendor/github.com/containous/staert/staert.go:84 +0x2e
main.main()
        /private/tmp/traefik-20180615-52269-s43ora/src/github.com/containous/traefik/cmd/traefik/traefik.go:142 +0x1b19

Closed by #3616.

@MadWombat Just to give you some clarification on what happened:
The --cluster flag was used in the past to ensure that traefik knew it was running in cluster mode. However, over the years, we have changed the code to automatically configure cluster mode if connected to a KV store. Therefore the flag is no longer needed.
The reason that there is no documentation or reference to that flag is that no user configuration is required. Just omit using that flag, and you should be fine.
Thanks!


Next release

Is there a timeline or plan for the next release of RubySaml? There hasn't been a release since 0.8.1 in February, and it is missing some key functionality, like security Settings and a metadata fetcher.
While using the :github directive in a Gemfile works, it isn't a solution for everyone, and a release bump that's easilly fetched from gem would be preferred, at least for me.
Thank you for the great gem, though.

@liamnoone ruby-saml 0.8.2 and 0.9 were released today!  0.8.2 fixes a security vulnerability, and 0.9 contains all pull requests from Feb 2014 to present day.
https://rubygems.org/gems/ruby-saml/versions/0.8.2
https://rubygems.org/gems/ruby-saml/versions/0.9


typo?

Hyper-V: Hyper-V VMs can be replicated every five minutes, o 30 seconds

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

ID: 7a8f2ca3-19b1-330d-8dd3-6dcdce38c8d4
Version Independent ID: 700335aa-c802-0314-1579-3493918f5000
Content: Azure Site Recovery: Frequently asked questions
Content Source: articles/site-recovery/site-recovery-faq.md
Service: site-recovery
GitHub Login: @rayne-wiselman
Microsoft Alias: raynew

Fixed. Fix should show up later today
#sign-off


BUG: Issue with website compilation for sphinxcontrib-jupyter=0.4.1

The following issue was found for sphinxcontrib-jupyter=0.4.1 with new template path
Exception occurred:
  File "/home/mmcky/anaconda3/lib/python3.7/site-packages/jinja2/loaders.py", line 408, in load
    raise TemplateNotFound(name)
jinja2.exceptions.TemplateNotFound: theme/templates/theme/templates/lectures-nbconvert.tpl
The full traceback has been saved in /tmp/sphinx-err-20tyw9qd.log, if you want to report the issue to the developers.

Moved to appropriate repository.
https://github.com/QuantEcon/sphinxcontrib-jupyter/issues/243


Any idea/ETA when templateRef will be implemented?

Hello,
Wondering when this solution will get implemented https://github.com/ngx-formly/ngx-formly/issues/2086#issuecomment-584847953?
I need to mix the form with angular components and do actions accordingly...
Thanks!

still not having the time :pray:, it can be achieved somehow using a custom field type with a CDK portal or just NgTemplateOutlet https://stackblitz.com/edit/angular-fvfcth

Thanks, @aitboudad ! This works perfectly until you need to pass template reference to a field in a repeating section.  Here is a forked repeating section example: https://stackblitz.com/edit/angular-xoamrw-zvjxrm
I have defined sample template in app.html:
<ng-template #formlyTemplate> Template content! </ng-template>
Passed template to field configuration:
      {
        key: 'investments',
        type: 'repeat',
        templateOptions: {
          template: this.formlyTemplate,
          addText: 'Add another investment',
        },

Added template to repeat section button:
      <button class="btn btn-primary" type="button" (click)="add()">{{ to.addText }}
      <ng-container [ngTemplateOutlet]="to.template"></ng-container>
      </button>

and it works like a charm!

But when I try to pass the template to the field in FieldArray, it fails:
        fieldArray: {
          fieldGroup: [
            {
              className: 'col-sm-4',
              type: 'input',
              key: 'investmentName',
              templateOptions: {
                template: this.formlyTemplate,
                label: 'Name of Investment:',
                required: true,
              },
            },

Any idea what I am missing?
What I am actually trying to do is to pass TemplateRef to Autocomplete, to avoid creating multiple autocomplete types in order to render options differently. But autocomplete field is in FieldArray :(

@rocktheroad
on the formArray field could you please try doing this,
              expressionProperties: {
                "templateOptions.template": "formState.template"
              }

on formstate before that do this
    this.options = {
      ...this.options,
      formState: {
        template: this.formlyTemplate
      }
    }

I'm not getting an error that way but because there is no autocomplete in the example to be used i'm not sure if it works or not.
@aitboudad it looks like there may be a bug when using the utils.clone function on cloning the formlyFields for the form array to use.  It is unable to clone the  TemplateRef I believe.  it is currently erroring out saying  "cannot read 'name' of undefined" but i'm not sure where name is coming from.

@kenisteward, that works. Actually, it is pretty close to what I had as a workaround. In autocomplete-type.component.ts I was checking first for the existence of template in templateOptions first, then in formState, and as a last resort I was providing default template.
The problem is when I have several different autocomplete types with different templates on the same form. Storing a template in formState limits you to one template, as it is shared between all fields.

@kenisteward, that works. Actually, it is pretty close to what I had as a workaround. In autocomplete-type.component.ts I was checking first for the existence of template in templateOptions first, then in formState, and as a last resort I was providing default template.
The problem is when I have several different autocomplete types with different templates on the same form. Storing a template in formState limits you to one template, as it is shared between all fields.

Isn't each autocomplete a different key? In the sense couldn't you just use a different key on formstate for each autocomplete ?
Or use an object to store all your templates and put each key in there.  I might be mis understanding your scenario so let me know!

@aitboudad it looks like there may be a bug when using the utils.clone function on cloning the formlyFields for the form array to use. It is unable to clone the TemplateRef I believe. it is currently erroring out saying "cannot read 'name' of undefined" but i'm not sure where name is coming from.

fixed, will be part of next release.

Any idea/ETA when templateRef will be implemented?

the new solution is available in https://github.com/ngx-formly/ngx-formly/releases/tag/v6.0.0-next.4, for usage check my comment https://github.com/ngx-formly/ngx-formly/issues/2086#issuecomment-584847953


indent problem

indent problem

Why close and reopen?

I accidentally pressed the wrong key.

Why repeat doing this?

I didn't see the pull request code merge into maser code . How to deal with it?

That depends on us. We need the project lead to confirm.

oh...

You don't have the rights to do this, unless you become committer. Please wait and keep this open.

ok


Method to disable Automatic Code Signing in Xcode 8

I see in the example CakeMania Cakefile there is way to enable automatic signing. I'm wondering, is there a way do set this to be manual?
I'm using fastlane's match to to handle my teams code signing. It seems when I build the project on CI the default value for Automatic Code signing is set.
I know one of the major configurations to set this occurs in the project.pbxproj file. In the PBXProject object's attributes there is a section of TargetAttributes.
TargetAttributes = {
    9EE7CAE93A712E88371C9275 = {
        ProvisioningStyle = Manual;
    };
};

In the CakeMania project.pbxproj this only includes the DevelopmentTeam.
Does xcake have any mechanism to set this? If not could you point be to the best section of code where I may try out some the changes and(hopefully) open a PR?

So right now no. From what I've read the target attribute is a new thing and xcake's underlying library xcodeproj as far as I'm aware does t support it yet.
Maybe @orta or @neonacho can confirm this

@jcampbell05 yes after opening this issue I searched through xcodeproj's issue and found something similar #384. It looks like xcodeproj doesn't have support to manage the projects TargetAttributes quite yet.
There is the update_project_codesigning fastlane plugin that should update the project configuration in a way that properly sets the Manual codesigning, but I haven't had any luck using it just after xcake creates my project.
I'll keep investigating to see what is possible, if I find something that works I'll open a PR for either xcake or for xcodeproj.

I'll close this for now as it seems like we can't do anything about it.


Bug: Command exception messages in home channel do not show full command name

For commands in groups, like emoji enlarge, the exception message in the home channel only show the individual command name (like enlarge):

An exception was thrown when executing a slash command
An exception was thrown when  used enlarge. Details are below. Additionally, an unknown exception occurred when trying to respond to the interaction: DSharpPlus.Exceptions.NotFoundException: Not found: NotFound

The full name should be shown instead. This was introduced from the D#+ command migration in #22 and was fixed somewhere else already (command counts maybe?).

Fixed in https://github.com/FloatingMilkshake/MechanicalMilkshake/commit/0abcd8b79f4aee5f419a0d43246ee2978332f5b4


[FLOC-2290] Don't use world-writeable permissions if dataset has been modified with files

Fixes: https://clusterhq.atlassian.net/browse/FLOC-2290
Merged 80f4df5c9b4a0dc310ec22618a243cf5a754aef8 (PR #1606) into master.

This includes unrelated changes.

This was stacked on a branch which has now been merged (#1627).

Merge when buildbot is happy.

The test failures were caused by Rackspace resource and api limits.
So I merged anyway.


pool removed as not compatible with epochs=1

@Mbompr @ihtiihti

Is there any point in keeping with Pool(processes = multiprocessing.cpu_count()) as pool: context?

I don't know @Mbompr
any thoughts @ihtiihti


Incorporate JPEG View as an alternative option vs Default Image Viewers

What feature or improvement do you think would benefit Files?
Yet another :D Incorporate JPEG View as the Default Image Viewer for Files. JPEG View is a beautiful, lightweight FOSS Image Viewing App for Windows. It's less than 5MB in size, so my hopes is that incorporating it into Files as its own default Viewer is easy enough to pull off. Ideally this could be in the options as a toggle between "default image viewer / JPEG View."
It'd be a nice option for all the FOSS enthusiasts such as myself and perhaps might make loading or opening images faster?
Requirements
This proposal will incorporate JPEG View's Image Viewer natively into Files, allowing users to opt for using JPEG View to open/view images vs Windows Image Viewer by toggling the option in the settings
Files Version
<IP_ADDRESS>
Windows Version
Windows 10
Comments
No response

Thanks for the feedback, Files can already preview jpgs what benefit does that have?

Are you refering to the preview pane?

I am unfamiliar with both Quicklook and SeerPro.
@yaira2 I am not referring to the preview pane, rather providing a built-in (the most lightweight) FOSS alternative for opening images for people who would like to opt-out of the Closed-Source alternatives / default image viewers that are built in to windows.
I was able to find Quicklook on Github and will explore it, I wasn't however able to find SeerPro's github, if you could provide the links I will gladly install them and try them out @Josh65-2201

Thank you for the response, our current approach is to leave the file previews to other applications as opposed to bundling extra software together with Files. The QuickLook integration is our recommendation if you're looking for a better way to preview files.


Most YAML diff changes are expected making the view messy. How to clean up?

Idea: Replace the input yaml run labels with PLACEHOLDER before the diff

Fixed


Add support for free-fall triggering to sensor api

It's my first Zephyr PR. Happy to hear suggestions on things I could add.

It's my first Zephyr PR. Happy to hear suggestions on things I should add.

Basically, this PR is addin a new trigger type, which makes perfectly sense. But no one is using it yet. So, I was expecting this PR to provide commits of driver/sample that makes use of it. But maybe they will come later on.

Ah, got it. Yes this is being used by a BMX160 driver that I'm working on, however for now it is a little half-baked and therefore I was intending to keep it out of tree for now at least.
I assume in general any commits should be more polished.

Hi @danb-swiftlabs  I am interested in the BMX160 driver you are working on. I am also trying to develop with this sensor. Would you be interested in sharing it? If so, would you be able to message me on the zephyr slack about it? I am @silt on slack.

@danb-swiftlabs Can you please share that BMX160 driver?


WX350 supported for the OpenMemories tweak

Does anyone know/can tell me if the Sony WX350 also is able to run this tweak?

Tweaking should already work. In pmca-gui, go to the tweaks tab and click "start tweaking".


Feature/scanning instrument controller

Review: By commit
Merge strategy: Merge (no squash)

Description
New module scanningInstrumentController added, as well as unit test, text documentation and updated release file.
Verification
Unit test added for the new module.
Documentation
Documentation created for the new module - no previous documentation affected.

@patkenneally, is there anything else you think I should address?


Enable Logging for Weights and Biases

I think it is easy to configure the finetuning scripts (this is the main focus for now given the NeurIPS challenge) with support for experiment tracking tools like Weights and Biases.
I have created a gist here with the relevant changes to the finetune/lora.py file (find "W&B" mention).
If this is of interest, I would love to open a PR.

@ayulockin How about collaborating on an integration for Fabric? I opened a feature request:
➡  https://github.com/Lightning-AI/lightning/issues/18397

...actually, a feature request already exists:
https://github.com/Lightning-AI/lightning/issues/17885
I closed the issue I opened earlier.

Thanks to @awaelchli  there's now a --logger_name option where you can choose --logger_name "wandb" for example. Let us know if there are any follow-up questions or issues.


Deprecate tertiary text color

The current tertiary text color has a contrast of 2.92 :1 on the base color (white), which does not meet any accessibility requirements. The usefulness of this color property is questionable.
The only place I remember it being used at the moment is in date picker, for the week day names and week numbers. That’s a poor design decision on my part.
I propose that we deprecate the tertiary text color, and remove it in the next major version of Lumo.
Technically, we could map the property by default to the secondary text color, to improve accessibility. Not sure if there’s a way to give any deprecation warning about this, if some custom CSS is using it?

Added this to the list of breaking changes. I don't think there is a way to give a warning without using a tool like Stylelint.

Well, we can at least mark it deprecated in the documentation and in code, to discourage its use.


feature: add disable auto reload config file user option

Did you check the docs?

[X] I have read all the LazyVim docs

Is your feature request related to a problem? Please describe.
I use neovim to edit LazyVim configs, but it's always let me restart neovim
Describe the solution you'd like
add a user option to disable this feature, like this
-- in init.lua
vim.g.disabled_auto_reload_configs = false

Describe alternatives you've considered
no
Additional context
No response

I'm not sure what exactly you want to achieve. Would you mind explaining with more detail please? You don't want to see the
# Config Change Detected. Reloading...

- **changed**: `lua/plugins/coding.lua`

notification if I understand correctly and you want to manually restart Neovim for the changes to take effect?

Thank you for your reply
You're right, I don't want to see this message, I'm used to manually restarting to reload configs

I'm not sure what exactly you want to achieve. Would you mind explaining with more detail please? You don't want to see the
# Config Change Detected. Reloading...

- **changed**: `lua/plugins/coding.lua`

notification if I understand correctly and you want to manually restart Neovim for the changes to take effect?

Thank you for your reply
You're right, I don't want to see this message, I'm used to manually restarting to reload configs

Add a change_detection key with the settings you want in your lazy.nvim spec (in config/lazy.lua). These are the defaults:
  change_detection = {
    -- automatically check for config file changes and reload the ui
    enabled = true,
    notify = true, -- get a notification when changes are found
  },

https://github.com/folke/lazy.nvim#-configuration

Add a change_detection key with the settings you want in your lazy.nvim spec (in config/lazy.lua). These are the defaults:
  change_detection = {
    -- automatically check for config file changes and reload the ui
    enabled = true,
    notify = true, -- get a notification when changes are found
  },

https://github.com/folke/lazy.nvim#-configuration

my problem is solved. Thank you very much

Plz bare in mind that if you put change_detection.enabled = false then you will also lose the ability for the Lazy UI to pick up changes when you make any change in your specs (Lazy will only pick the changes to install or clean a plugin after a restart). With just notify = false you won't see the notification any more.

Plz bare in mind that if you put change_detection.enabled = false then you will also lose the ability for the Lazy UI to pick up changes when you make any change in your specs (Lazy will only pick the changes to install or clean a plugin after a restart). With just notify = false you won't see the notification any more.

I remember it, now. thank you very much! 👍


Let me decide localization myself

Dear team,
Could you please let me stick with original en-us content?
I'm from Ukraine and your site pushes me to read Russian localized version which I don't want for multiple reasons.
The best option would be for you to massage geo-targeting the way Google search results do NOT filter out en-us content in favor of Russian.
Also let's change current "Read it in English" behavior to transfer me to original en-us page.
Currently "Read it in English" does something weird as the article itself being translated to English, but all the other content stays in Russian.
If you're not able to localize it to Ukrainian due to business reasons, then at least let me stick with original English one.
Thank you,

@AndriyGorda Thanks for your feedback. To better understand your issue, are you currently changing the language/local via: https://docs.microsoft.com/en-us/locale/?target=https://docs.microsoft.com/en-us/ ?

Hello Samir,
No, I don't and don't want to bother doing it every time I go to Microsoft site from Google search results.
That's exactly my request - I shall be given the ability to stick with English starting from Google search output for docs.microsoft.com and throughout my Microsoft sites experience.
What system gives me now is:

Only Russian localized docs.microsoft.com pages in Google search output (no English!)
No ability to stick with English throughout Microsoft pages

Duplicate of #462.


[release/1.24] repo: Release v1.24.9

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/retest

/docs

Ok, but then why isn't this file "empty" (all change-log comments removed).

its not empty - its just stripped of comments - they are only there as a guide for editors when the file is "active"
this can be a bit confusing - which is why i automated it some time ago - previously this process was very error prone


Merge finatra client USF senior team updates

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 7 committers have signed the CLA.:white_check_mark: locknic:x: dhensche:x: cacoco:x: murph:x: jenkins@bigbird:x: sdalo2:x: toropezajenkins@bigbird seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

🔨

@bryce-anderson I'm working on these.

We haven't given up on this and hope to have this work merged in soon. I'm going to close this PR for the time being but stay tuned!


DnsNameResolver.resolveAll(...) should not include duplicates

Motivation:
DnsNameResolver#resolveAll(String) may return duplicate results in the event that the original hostname DNS response includes an IP address X and a CNAME that ends up resolving the same IP address X. This behavior is inconsistent with the JDK’s resolver and is unexpected to retrun a List with duplicate entries from a resolveAll(..) call.
Modifications:

Filter out duplicates
Add unit test

Result:
More consistent and less suprising behavior

It seems like this is causing a regression on our side because DefaultDnsRawRecord does not compare the actual IP addresses. If a DNS server responds 3 different IP addresses for the same domain name, you'll get only one of them.

@trustin sounds like a bug DefaultDnsRawRecord. Do you want to do a PR ?

Let me dig a little bit more and send a PR.

@trustin fixes https://github.com/netty/netty/pull/9141.


TS-5022: fix silent exit problem

Linux build failed! See https://ci.trafficserver.apache.org/job/Github-Linux/1212/ for details.

FreeBSD build failed! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1318/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1319/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/1213/ for details.

Looks good.


unkown error

Fairly often an audiogram will fail to finish creating because of an "unkown error". The docker console gives this out:
Error: Can't set headers after they are sent.
    at ServerResponse.OutgoingMessage.setHeader (_http_outgoing.js:335:11)
    at ServerResponse.header (/home/audiogram/audiogram/node_modules/express/lib/response.js:719:10)
    at ServerResponse.send (/home/audiogram/audiogram/node_modules/express/lib/response.js:164:12)
    at module.exports (/home/audiogram/audiogram/server/error.js:17:9)
    at Layer.handle_error (/home/audiogram/audiogram/node_modules/express/lib/router/layer.js:71:5)
    at trim_prefix (/home/audiogram/audiogram/node_modules/express/lib/router/index.js:310:13)
    at /home/audiogram/audiogram/node_modules/express/lib/router/index.js:280:7
    at Function.process_params (/home/audiogram/audiogram/node_modules/express/lib/router/index.js:330:12)
    at next (/home/audiogram/audiogram/node_modules/express/lib/router/index.js:271:10)
    at Layer.handle_error (/home/audiogram/audiogram/node_modules/express/lib/router/layer.js:67:12)

Is there a specific sequence of events that reliably produces this error?

Not 100% reliable, but it seems to happen more often if the window/tab loses focus. It's hard to confirm though since I'm not monitoring it if it doesn't have focus.

Will close this for now, but if you can give me a sequence and configuration that always produces the error I'll investigate.

I think I know what caused this.  If an error occurred while checking the status of a pending job, it would send an error response and then throw an error, which trickled down to an Express error handler trying to re-send headers.  The most likely underlying error was a corrupted file or something - I've fixed this on the refactor branch so that you'll at least get a message describing the real error.

I've had another chance to play around with audiograms and I have a reliable way to create an unkown error
Anytime a process is Not Responding the audiogram stops processing and gives an unknown error

I have the same problem but it happens each time I want to render an audiogram, always.
It goes like Rendering frames 4%, 6%, 10%.. and bam! "uknown error"

I have the same problem. Any updates?


Generate Best Practice/Analysis Summary warning for Common Criteria Compliance.

In a recent SharePoint support case, a user that had recently upgraded SQL and SharePoint was seeing performance issues. It was eventually found that Common Criteria Compliance was the root cause. A warning in Nexus that this is enabled and can and will cause performance issues would be great. Note that we do call out performance issues in the public article:
https://learn.microsoft.com/en-us/sql/database-engine/configure-windows/common-criteria-compliance-enabled-server-configuration-option?view=sql-server-ver16

@troymoen1 this is now added. Thanks for filing


[Question] Fps uncapped, but still around 60?

So, this is a weird issue that I wouldn't attribute to the launcher but to the game - while unlocking fps does unlock fps and it reaches 120-144 in the main menu and sometimes 80-90 in game, it seems that everywhere in the open the game still orients itself around the 60 fps mark? My gpu usage is at around 50% (1440p) instead of being fully utilized. If I play on 4k, gpu usage is around 80-85%, with no decrease in fps.
I assumed the cpu (r9 5950x) to be the bottleneck, but locking cpu speed to 3800mhz or unlocking it to 4900mhz seems to make no difference either. It's as if the game simply waits to stay around 60 fps. Anyone else with the same issue, or a potential fix?

I have noticed the same thing.

Should be solved with <IP_ADDRESS> now. Let me know if it still happens.

Game decided to crap itself and bring me down to 0 fps every time I move my mouse after 16 hours of not having that issue, so I guess I'll have to do a fresh windows installation first. Will report back tomorrow.

Can confirm that with <IP_ADDRESS> my gpu utilization is now at 93% on average (1440p rx 6800 xt). Seems to be working fine.

Works, awesome! Now we just need to hope they fix the stuttering issues.


Cell barcode extraction from sequence files

As an HCA integration team member, I want to be able to incorporate DCP metadata into my integration analysis, even when I use only the count matrices.
** Acceptance Criteria / Definition of Done **

Given that I have a DCP project that contains sequence files (and maybe pooled analysis files)
When I run the script
Then I should get a table of barcodes mapped to fastq file and/or cell suspension

Depending on the dataset there are multiple cases.
We may have sequence files / analysis files.
Both can be per cell_suspension / pooled.
Analysis files could contain also the per cell sample_id / no sample_id.

Analysis files per CS
Analysis files pooled but sample_id provided
Analysis files pooled but seq files per CS
Only Seq files per CS

For all those cases, we can map the analysis files (and the corresponding cell barcodes) to a cell suspension, and then extract all the other relevant metadata for each cell.
In 1. we can directly extract the cell barcodes from each analysis file, and directly map the relevant CS.
In 2. we need to map the provided sample_id with the relevant CS ID that we have, and then we have cells mapped to CS.
In 3. we must extract the cell barcode from the fastq files and then map the corresponding cell ID to one specific CS.
In 4. it is similar to 3. but we don't have the list of cell barcodes that will be incorporated.
Datasets from the following bionetworks were investigated for their availability in this spreadsheet

Kidney
Gut
Eye
Lung

Column Type is one of the aforementioned 4 types of availability. In order to have the sample_id existence more investigation is needed.
Concerning the 3 and 4 types of availability.
Cell barcode extraction from fastq file can be done using the information that we have in our metadata schema (cell barcode). An investigation was done in this dataset (that has fastq files & analysis files while the size of fastq files is not extreme).
We have 5 different cell suspensions (samples) and we pooled our barcodes into 1 analysis file with 20008 cells in order to identify the cell barcodes of the analysis file.
In our investigation we found that we get multiple cell barcodes from each fastq file, because a large amount of cell barcodes is filtered out in the cellranger pipeline.

Post Tony/Gabby meeting actions:

[ ] create a kidney and a lung example of flattened metadata to the barcode level based on this template
[ ] execute the code for the 30+15 datasets of type 3,4 availability and extract the barcodes/CS in separate files (percise and ambigious)

Stats in spreadsheet were wrong.

Type
Number

1
1

3
30

4
15

more investigation
12

Not available
25

(if dataset is in type 1 and 3, we should show type 1 instead of 3. the opposite was done)
Updated stats:

Type
Number

1
20

2
5

3
18

4
15

partially 1
2

cannot unpool
22

No submission
12

Kidney example of cell metadata spreadsheet
The specific project was selected due to the already known list of desired metadata from the kidney bionetwork.
It is a type 1 project where it has 6 analysis files and 28 sequence files, all per Cell Suspension.
The authors mention 23,980 cells in their final count matrix, however, the raw, unfiltered and un-merged count matrices that are publicly available have 121,787 cells. Given that, we expect that the finaly merged objects will have an identifier before the barcode.
There are currently multiple ways to add this identifier (i.e. addition of a sample_id before the barcode NL2_R_TTTGTCAAGCGATTCT-1, 10X_087_AAACCTGTCCGAATGT-1, 1.1_AAACGAACAACGACAG-1, 0_AAACCCAAGCATTGAA-1, addition of a counter after the barcode AAACCCAAGCATTGAA-1, AAACCCAAGCATTGAA-2 or AAACCCAAGCGTTCAT-1_1).
What we can offer is a specific barcode, with all the corresponding metadata that we may have, and then it will be easy for the user to match the given identifier with one of all the sample metadata.
For this specific project, we extracted all the cell_barcodes from each count matrix (one for each of the 6 CS) into a csv file (R script), and using the DCP metadata spreadsheet, we extracted all the given metadata in this spreadsheet (jupyter notebook).
We have to note that the given project did not contain analysis_file metadata due to a known problem in a specific number of old datasets, and a fast curation was done for the purpose of this task.
Finally, it has to be stated that the procedure for now, cannot be fully automated, and a supervision on the script is needed. (different objects have different ways to extract barcodes, metadata needs to be traced back from files to donors and each project has a different experimental design)

Lung example is now live.
Bug: mouse specimens are pooled. Script to extract metadata needs to be updated otherwise metadata upwards this pooling are not filled.

Presentation of work until now

Created 2 CxG h5ad objects with DCP metadata (LungStromaEmphysema and KidneySexBasedTranscriptome), and a small tutorial on how to cast the file to an CxG instance.

Found a script that flattens metadata here. More intuitive than previous, but lacks the option to join pooled samples (i.e. donor_1||donor_2 in specimens).
Next steps:

Create a repo with current work done

[ ] Create a README.md that describes the problem and the solution
[ ] Create distinctive scripts with clear input-output arguments that:

Collects automatically the cell barcode information
Assigns the cell barcode information to a specific sample_ID
Flatten DCP metadata spreadsheeet
Merge the flatten spreadsheet with the sample_ID information

[ ] Provide clear instructions to run the scripts

Further steps

[ ] Investigate if Amnon's script (mentioned before) can include pooled cases
Automate the cell_barcode extraction for:

[ ] h5ad
[ ] Seurat (rds or RData)
[ ] SingleCellExperiment objects (rds or RData)
[ ] h5
[ ] tsv, csv count matrix


Notes

Initial comments added to SessionModel and client.js

@akrawchyk modified those two notes that you called out. Check them over and I'll merge it all into client? or master?

:+1: merge it into master


cleanup: remove the legacy folders (do-not-merge)

Description

We have made all the source code for the apps as a first class entity in the repo; thus we want to delete the old point-of-sale-app folder
We also have moved the anthos-baremetal-edge-deployment scripts to the anthos-samples repo; thus we also want to delete this folder
Any references to these folders have been already changed within this repo
Any references to these folders have been changed in the anthos-samples repo as well
We need to ensure cl/446868450 is merged for other content reference changes

:zap:  Two deployments have been created for the Point-of-Sale application. You may access and test them at: :zap:

MySQL DB backed deployment: <IP_ADDRESS>
Embedded H2 DB backed deployment: <IP_ADDRESS>

confirming that removing the regions tags is ok 👍

:zap:  Two deployments have been created for the Point-of-Sale application. You may access and test them at: :zap:

MySQL DB backed deployment: <IP_ADDRESS>
Embedded H2 DB backed deployment: <IP_ADDRESS>


Environment file with pip dependency file

It seems like the latest v2.1.0 has broken the processing of custom environment file with
  - pip:
    - -r file:requirements.txt

That now seems to look for it at a non-existing location:
  Installing pip dependencies: ...working... Ran pip subprocess with arguments:
  ['/usr/share/miniconda/envs/devito/bin/python', '-m', 'pip', 'install', '-U', '-r', '/tmp/condaenv.zimdqc7e.requirements.txt']
  Pip subprocess output:

  failed
  Warning: Pip subprocess error:
  ERROR: Could not open requirements file: [Errno 2] No such file or directory: 'requirements.txt'

Is this supposed to not be supported anymore and what's the proposed fix?

Yep, suppose that will happen when you specify python-version, which applies a patch. This was done to avoid doing multiple conda solves
We probably need to:

generate a patched file in-place rather than in /tmp to support this case
add a test

Will get to this when I get a chance... but that's not going to be until at this evening at the earliest.
Your quick bets are to:

don't use python-version

ensure your environment.yml specifies python (sed can help)

continue to use python-version

patch your -r  to points at an absolute path or
do a dedicated step of pip install -r requirements.txt

Thanks for the quick answer.

do a dedicated step of pip install -r requirements.txt

Yes that works but need yet another yaml file we trying to avoid, may have to do it in the meantime.

ensure your environment.yml specifies python

We'd like to avoid that too so will fall back to above.
Checking without python-version thanks.

Removing the python version did the trick (for some extremely lucky unknown reason our CI did not break with the default 3.9 we're not supposed to support).
Closing


Suggestion

You should make it actually useable by using 3rd party embed sources such as vidsrc.cc

Hi, I’m not considering including any streaming features at the moment, but thank you for the suggestion.


Remove istio

Codecov Report

Merging #16 into master will decrease coverage by 6.01%.
The diff coverage is 16.66%.

@@            Coverage Diff             @@
##           master      #16      +/-   ##
==========================================
- Coverage   70.83%   64.81%   -6.02%
==========================================
  Files           3        3
  Lines          48       54       +6
==========================================
+ Hits           34       35       +1
- Misses         10       15       +5
  Partials        4        4

Impacted Files
Coverage Δ

rest/handlers.go
55.81% <0%> (-7.35%)
:arrow_down:

rest/routes.go
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


[HTTP] Payloads are included in the responses to HEAD requests

Tried the following sample and the passthrough sample, and in both cases, the responses received for the HEAD requests contained response bodies as well.
Hello World sample
import ballerina.net.http;

@http:configuration {
	basePath: "/hello"
}
service<http> helloWorld {

    @http:resourceConfig {
	path: "/"
    }
    resource sayHello (http:Request req, http:Response res) {
        res.setStringPayload("Hello, World!");
        _ = res.send();
    }
}

request:
$ curl -v http://localhost:9090/hello -X HEAD
response:
< HTTP/1.1 200 OK
< Content-Type: text/plain
< Connection: Keep-Alive
< Content-Length: 13
<
* Connection #0 to host localhost left intact
Hello, World!

Closing this as per @shafreenAnfar 's comment


Incorrect s3 private endpoint used

Terraform Version Details
When using viya4-iac-aws to deploy in private or offline situations, the EKS creates but nodes fail to join the cluster.  Referencing AWS documentation, AWS specifies the use of "Gateway" S3 private endpoint for offline/private EKS deployments (found here).
Our IaC current deploys all private endpoints as "interface" type.
Terraform Variable File Details
No response
Steps to Reproduce
Use IaC in BYO scenario where the cluster API is private and has no outbound internet access (no NAT GW) from the VPC.
Expected Behavior
EKS will build and nodes can join the cluster.
Actual Behavior
EKS will build but nodes fail to join the cluster.
Additional Context
No response
References
No response
Code of Conduct

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

Upon further investigation, it appears this was not the cause of the issue.  S3 Gateway and interface should function the same, a Gateway type is a cheaper option.
Closing issue and cancelling associated PR.

This resurfaced from another deployed I was helping with.  So I decided to investigate and test again.
It appears private EKS does in fact need a Gateway type (not an interface type) S3 endpoint in order to function correctly.
Test scenarios:

VPC, subnets, security groups deployed by CloudFormation (with no endpoints).  IAC 5.4.0 used with BYON variables supplied.  IAC created the vpc endpoints (S3 type: interface).
VPC, subnets, security groups deployed by CloudFormation with endpoints based off of AWS provided template for private EKS. IAC 5.4.0 used with BYON variables supplied, mod applied to remove deployment of: NAT, IGW, and private_endpoints (changed count to 0 for all resources).

Test Results:

Nodes failed to join the cluster.
Nodes succeed joining the cluster.

While I can't find this explicitly in the AWS documentation, it is mentioned on the eksclt.io page and also specified in the AWS provided EKS private VPC template.
A S3 Gateway type has to be associated with a route table, so my work around at the moment would be to manually add a S3 gateway to the VPC and associate with the routetable.  You could also do like I did in my successful test scenario (scenario #2).
This may not be an issue in subscriptions with centralized VPC endpoints (hub and spoke) and that centralization has a S3 Gateway already.  In this case, it might be advantageous to not deploy the IAC created endpoints (so would require modification to terraform or terraform override file).

moving discussion to internal ticket and closing issue.

After further investigation/testing, we ultimately discovered that functionally a S3 VPC endpoint with either Gateway type OR interface type will work in an EKS air gap scenario. The difference is in implementation and cost. A gateway type requires a routable association whereas interfaces do not and Gateway types are free whereas interfaces have a cost associated.
See https://github.com/sassoftware/viya4-iac-aws/issues/272#issuecomment-1978945557 for further information.


Error when trying to unbond

Trying to unbond.  After submitting in MetaMask, I get a pop-up error: "Failed - There was an error submitting your transaction. See error message below for more details:
{
"receipt": {
"blockHash": "0xc1127e032c506dbbe8a9f5e88d07ed8e842b4d089f89d10b21c444c7067763b4",
"blockNumber": "a6fc71",
"contractAddress": null,
"cumulativeGasUsed": "442571",
"from": "0xaf00f59b6295be33ae1d5b12786b5f5a5be8dbbb",
"gasUsed": "63060",
"logs": [],
"logsBloom": "0x00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000",
"status": "0x0",
"to": "0x511bc4556d823ae99630ae8de28b9b80df90ea2e",
"transactionHash": "0xb3e94073e00d187e8482fbd4364dcf69f8d5aaaf53334f59cf4e1a6fba6d0b96",
"transactionIndex": "22"
},
"transaction": {
"blockHash": "0xc1127e032c506dbbe8a9f5e88d07ed8e842b4d089f89d10b21c444c7067763b4",
"blockNumber": "a6fc71",
"from": "0xaf00f59b6295be33ae1d5b12786b5f5a5be8dbbb",
"gas": "200b20",
"gasPrice": "c570bd7b3",
"hash": "0xb3e94073e00d187e8482fbd4364dcf69f8d5aaaf53334f59cf4e1a6fba6d0b96",
"input": "0x27de9e320000000000000000000000000000000000000000000000000000000000000000",
"nonce": "7a",
"r": "0x7bb14f7462607d070a1dd2a7ea42c30432f8488fb12d1e7fc2ee85ecf43a5372",
"s": "0x23610fb9595e981c3f7837809473ce3f0afe2b641f773fc158c27a43a5c3b177",
"to": "0x511bc4556d823ae99630ae8de28b9b80df90ea2e",
"transactionIndex": "22",
"v": "0x25",
"value": "0"
}
}
Could someone assist?  What am I doing wrong?  Thanks.

Hey, thanks for reporting. Which wallet are you using? Also, what are you entering when attempting to unbond? The reason I ask is because I'm seeing the following error in etherscan: "Fail with error 'unbond amount must be greater than 0k'"

Using MetaMask.  I've tried "0", i've tried the full amount, still getting the same errors.  Haven't tried an amount less than my total, maybe like 75% so I'll give that a shot.

Okay, looks like it worked this time.  Instead of full amount, I left a few LPT and it worked okay.  Thanks for the help Adam.

@madbum - Glad you got it working. That actually sounds like a bug. You should be able to unstake your entire stake by clicking "max". Thanks for reporting.


Missing EPs

EP 22, 23, 25, 26

https://www.tally.xyz/gov/ens

These are the PRs for all 4:
[EP 5.25]  Generated by the Agora bot.  It looks good to me, but it still needs the metadata added.
https://github.com/ensdomains/docs/pull/358
[EP 5.26] - Generated by the Agora bot.  Also looks good to me, but also still needs the metadata added.
https://github.com/ensdomains/docs/pull/359
This one is for [EP 5.23] and [EP 5.24] - Submitted by @estmcmxci, but shows a web pack build error.  Double check the paths.
https://github.com/ensdomains/docs/pull/347


Added support for widgets to occupy multiple cells in a grid

I added support for widgets to occupy multiple cells in a grid.
Check the example in examples/layout/grid.js to see better what I mean.

Thanks @mvaladas , I love it!

Glad you liked it. I've been having some fun with your widgets.
If something else comes up I'll let you know.


i69 code review feedback

Primary changes:

improve embedded comments
raise only non-fatal errors to avoid termination of the entire pipe run should a problem occur that is specific to a single object in the data source

Stripe Connector regression tests did not expose any issues.


"Too many fields" Error

Hello,
The plugin repeatedly gives me this error:

Plugin:obsidian-import-json:20962 [
{
"type": "FieldMismatch",
"code": "TooManyFields",
"message": "Too many fields: expected 1 fields but parsed 7",
"row": 0
},
{
"type": "FieldMismatch",
"code": "TooManyFields",
"message": "Too many fields: expected 1 fields but parsed 7",
"row": 1
}
]
convertCsv @ plugin:obsidian-import-json:20962
eval @ plugin:obsidian-import-json:21328
eval @ plugin:obsidian-import-json:49
__async @ plugin:obsidian-import-json:33
input5.onclick @ plugin:obsidian-import-json:21306
plugin:obsidian-import-json:20964 {
"delimiter": ",",
"linebreak": "\n",
"aborted": false,
"truncated": false,
"cursor": 1370,
"fields": [
"Question|Answer|References"
]
}
plugin:obsidian-import-json:21121 generateNotes('[object File]', 'undefined', [object Object] )

I'm using a pipe delimiter in this file and I don't think it's decoding my file correctly. I could be wrong. Does anyone have a solution to this?
Thanks

Oh, I just noticed in the last section of your log that it is using a comma as the delimiter.
You might have to explicitly set the delimiter in the plugin's settings.

Actually I don't have a field in which to enter the delimiter; it is supposed to be detected automatically.
I see that the decoder is reading the entire first line as a single field "Question|Answer|References".
I am using the Papaparse library to do the decoding. As a test can you check if putting spaces either side of the "|" in the first line causes the detection of separator to work properly?

Thank you, adding the pipe on each end solved the issue! I appreciate the quick response!


PowerToys crash due to disposed Timer

ℹ Computer information

Windows build number: 18363.959
PowerToys version: 0.20.0
PowerToy module: fancy zones

📝 Provide detailed reproduction steps (if any)
Don't have specific steps, was just moving Slack between zones using Win+Left and Win+Right buttons, and PowerToys crashed.
2020-08-06.txt
The only interesting detail - another monitor was connected after Windows started, though several hours before the incident. Usual setup, two 38402160 displays, newly connected one - 19201200.
❌ Actual result
PowerToys crashed

Have you tried upgrading to version 0.20.1? (PT Settings => check for updates). I think there's a bugfix for this issue in that version.

https://github.com/microsoft/PowerToys/releases/tag/v0.20.1
Duplicate of #5400


redirects from non-hash containing pypi.io and files.pythonhosted.org missing for newer package versions

Describe the bug
For the package snakemake-wrapper-utils, the versions 0.5.1 and 0.5.2 do not have the redirects from the simple and readable URLs without the hash value.
These are available for the earlier versions, for example: https://pypi.io/packages/source/s/snakemake-wrapper-utils/snakemake-wrapper-utils-0.5.0.tar.gz
Expected behavior
Have the following redirects for version 0.5.2 (and 0.5.1 accordingly):

https://pypi.io/packages/source/s/snakemake-wrapper-utils/snakemake-wrapper-utils-0.5.2.tar.gz
https://files.pythonhosted.org/packages/source/s/snakemake-wrapper-utils/snakemake-wrapper-utils-0.5.2.tar.gz

To Reproduce
Just check out the above links under Expected behaviour, which should optimally work.
My Platform

I think that this is independent of platform.
Additional context

We use these consistent URLs in the bioconda project, to keep yaml files for conda recipes readable and easy to maintain. For the here reported package, see the last working version right here:
https://github.com/bioconda/bioconda-recipes/blob/a5c1155190cc811934ff681dcd1276d84e160130/recipes/snakemake-wrapper-utils/meta.yaml#L8-L10
I can still find the respective readable link pattern (which is for example described in this stackoverflow question and its responses) for newly published versions of other packages, for example:
https://files.pythonhosted.org/packages/source/s/snakemake/snakemake-7.24.2.tar.gz
Thus, and because I didn't find any deprecation info for this in the docs or changelogs, I am assuming that this is not some deprecation of these readable URLs, but rather some kind of temporary hiccup with setting up these redirects. Probably something similar to (although probably with another root cause than) issue #11940.
Interestingly, the publication of the latter of the two package versions that currently don't have the redirects (0.5.2) coincides with a pypi incident report from January 10, 2023 (PyPI File Hosting Latencies and Timeouts). Thus, maybe snakemake-wrapper-utils is not the only package with this issue.
Generally, I guess that querying the JSON API for the download URL would be the official guidance (instead of using the somewhat legacy? readable URL redirects), but this guidance could do with a clear example on how to do this. I have found a code example on how to query pypi's JSON API on stackoverflow, so maybe something along those lines could be added to the docs.
As for the bioconda project, we could possibly automatically insert those into the conda recipes (along with the expected sha256 checksum) and this should be more robust and future-proof, right?

Many thanks for the swift response!
Just to confirm that this is intentional, the link will now contain the package name once in kebab-case and once in snake_case, right. Like this:
https://files.pythonhosted.org/packages/source/{ first_letter_of_package_name }/{ package-name-kebab-case }/{ package_name_snake_case }-{ version }.tar.gz
Also, would you consider using the JSON API to get the URL more future-proof than relying on these standardized URL redirects?

The url format is as documented at https://warehouse.pypa.io/api-reference/integration-guide.html#querying-pypi-for-package-urls:
/packages/{python_version}/{project_l}/{project_name}/{filename}
We do not do any handling of the filename, that is the unloaders decision.
Yes. Querying the JSON API is the official guidance

The redirect endpoint queries the JSON API, so this might serve as helpful reference:
https://github.com/pypi/conveyor/blob/230ebcdcda0f15072d2ed2ee0ce94613244cbc07/conveyor/views.py#L33-L104

The following info basically has nothing to do with pypi or its warehouse. But for others coming here with a similar problem, I want to nevertheless document the root cause of this problem in my case: It was a change in poetry build behaviour, which means that sdist files built by poetry will have all hyphens / dashes (-) in package names (where this is the standard) substituted by underscores (_):

merged pull request in poetry: https://github.com/python-poetry/poetry/pull/6621
merged pull request in peotry-core (the actual changes): https://github.com/python-poetry/poetry-core/pull/484

And this is actually a change to conform to the specs for sdist file names:
https://packaging.python.org/en/latest/specifications/binary-distribution-format/#escaping-and-unicode


Comparison Tool

Allow users to select multiple players and compare stats side by side.
(Make player comparisons with just graphs and leave other stats out for now?)
(Different color bar lines on same graph vs put data side by side?)

Tough. Maybe just allow to export to CSV would be easy. You paste on a spreadsheet and you get the Comparison.

Have a few Ideas. Prolly gonna make a special form for comparing players. But Low Pri for now


Fix/log clean

Codecov Report

Merging #26 into master will decrease coverage by 3.37%.
The diff coverage is 60%.

@@            Coverage Diff             @@
##           master      #26      +/-   ##
==========================================
- Coverage   90.51%   87.14%   -3.38%
==========================================
  Files          21       21
  Lines         348      350       +2
  Branches       25       26       +1
==========================================
- Hits          315      305      -10
- Misses         33       45      +12

Impacted Files
Coverage Δ

...wbaa/airlock/sts/api/directive/STSDirectives.scala
100% <ø> (ø)
:arrow_up:

.../com/ing/wbaa/airlock/sts/service/db/MariaDb.scala
80% <ø> (ø)
:arrow_up:

...a/airlock/sts/keycloak/KeycloakTokenVerifier.scala
94.11% <ø> (ø)
:arrow_up:

...n/scala/com/ing/wbaa/airlock/sts/api/UserApi.scala
88.88% <0%> (-5.23%)
:arrow_down:

.../scala/com/ing/wbaa/airlock/sts/api/AdminApi.scala
87.5% <100%> (ø)
:arrow_up:

.../wbaa/airlock/sts/service/UserTokenDbService.scala
83.78% <100%> (+0.45%)
:arrow_up:

...aa/airlock/sts/util/MaskingPatternJsonLayout.scala
0% <0%> (-91.67%)
:arrow_down:

Continue to review full report at Codecov.

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


Serialize value bypass authorization

Hi,
I hope it's me but I have  serialize field (serialize :point_of_sale_ids, Array).
When I add an authorization on this field and test it. Authorization is bypassed and I can updated this field.
Thanks for the feedback
Jerome

Please expand a bit. Can you provide the rules you are using?  Serialized values will certainly not work for query rewriting (with_permissions_to), but should work in filter rules for the controller and for permitted_to? etc.


screenspacecontroller support for cesium unreal

fullfill the feature from community

the globe pawn is support the full feature when georeference origin placement in true origin,in cartographic some feature is disabled. Use the quick add panel. It will add in proper place.

When is it online? It's urgent

expect

The operation is smooth，Thank you for sharing

What should I do when generating a solution and reporting error MSB307 is reported
ue 5.3
Visual Studio 2022
cmake 3.30

Thank you for fulfilling the GlobePawn！
Yet there are situations that will make Unreal Engine Crash.
It is hard to describe how to reproduce the crash situations.
It is likely to crash when zoom in and zoom out several times,  and  the viewport will be complete black, then UE will shut down.
The following is the log captured when UE crashes. Much appreciated if you could look into it and fix it.
UE5.3,CesiumforUnreal2.5.0
UnrealEditor_CesiumRuntime!CesiumGeometry::Plane::Plane()
UnrealEditor_CesiumRuntime!Cesium3DTilesSelection::createCullingVolume()
UnrealEditor_CesiumRuntime!Cesium3DTilesSelection::ViewState::create()
UnrealEditor_CesiumRuntime!ACesium3DTileset::CreateViewStateFromViewParameters() [E:\UnrealPack\CesiumforUE\UE_5.3\CesiumForUnreal\HostProject\Plugins\CesiumForUnreal\Source\CesiumRuntime\Private\Cesium3DTileset.cpp:1673]
UnrealEditor_CesiumRuntime!ACesium3DTileset::Tick() [E:\UnrealPack\CesiumforUE\UE_5.3\CesiumForUnreal\HostProject\Plugins\CesiumForUnreal\Source\CesiumRuntime\Private\Cesium3DTileset.cpp:2123]
UnrealEditor_Engine
UE5.3 CesiumforUnreal2.7.0
UnrealEditor_CesiumRuntime!Cesium3DTilesSelection::createCullingVolume()
UnrealEditor_CesiumRuntime!Cesium3DTilesSelection::ViewState::create()
UnrealEditor_CesiumRuntime!ACesium3DTileset::CreateViewStateFromViewParameters() [E:\UE53Projects\GlobePawn\Plugins\CesiumForUnreal\Source\CesiumRuntime\Private\Cesium3DTileset.cpp:1630]
UnrealEditor_CesiumRuntime!ACesium3DTileset::Tick() [E:\UE53Projects\GlobePawn\Plugins\CesiumForUnreal\Source\CesiumRuntime\Private\Cesium3DTileset.cpp:2082]

@jiangheng90  I hope Touch support can be added.

Hi @jiangheng90, sorry to see you close this. We've been slow to review it, but it's definitely a valuable contribution!

GlobePawn有BUG，用这个后，操作时UE编辑器运行窗口会突然黑屏，然后UE编辑器会崩溃（在编辑器运行后操作最多20分钟必出，该BUG概率挺高，最快可能1分钟就崩溃），UE5.3，在cesium unreal2.5，2.7版本都存在。换回cesium unreal自带的DynamicPawn再怎么操作都不会黑屏崩溃。

@kring I'm sorry for the pull request close.
It's been more than half a year since the end of development of this feature, and many of the technical details in it are not so familiar to me now.
Secondly, because of the development of this feature, some people found my social accounts under the social media and caused some trouble to my privacy, which I didn't want.
Right now, can u just make a feature branch and I make my pull request to this branch. it will take a long time to make this feature be all right and I think this is a better way for both of us to push it forward.

It's been more than half a year since the end of development of this feature, and many of the technical details in it are not so familiar to me now.

Understood. It's only been a couple of months since you opened the PR, but even that is too long.

Secondly, because of the development of this feature, some people found my social accounts and caused some trouble to my privacy, which I didn't want.

Oh no, really sorry that's happening to you. That sort of behavior definitely has no place in the Cesium community. Let us know if we can help in any way.

Right now, can u just make a feature branch and I make my pull request to this branch. it will take a long time to make this feature be all right and I think this is a better way for both of us to push it forward.

Ok, I created a branch called screen-space-controller in this repo. It's just a copy of main.

@kring I have all ready make a new pull request.

GlobePawn有BUG，用这个后，操作时UE编辑器运行窗口会突然黑屏，然后UE编辑器会崩溃（在编辑器运行后操作最多20分钟必出，该BUG概率挺高，最快可能1分钟就崩溃），UE5.3，在cesium unreal2.5，2.7版本都存在。换回cesium unreal自带的DynamicPawn再怎么操作都不会黑屏崩溃。

哥，最新的Cesium For Unreal插件怎么使用这个Pawn呀

GlobePawn有BUG，用这个后，操作时UE编辑器运行窗口会突然黑屏，然后UE编辑器会崩溃（在编辑器运行后操作最多20分钟必出，该BUG概率挺高，最快可能1分钟就崩溃），UE5.3，在cesium unreal2.5，2.7版本都存在。换回cesium unreal自带的DynamicPawn再怎么操作都不会黑屏崩溃。

哥，最新的Cesium For Unreal插件怎么使用这个Pawn呀

要修复这个Bug,请查看【Cesium for Unreal 解决GlobePawn致使UE崩溃的方法-哔哩哔哩】 https://b23.tv/OUdesOC


Str::singular('_data') returns unexpected result

Laravel Version: 6.2
PHP Version: 7.3.14
Database Driver & Version: irrelevant

Description:
Str::singular('foo_data') returns 'foo_datum' - this can make big problems eg. with route model bindings - using Route::resource('user-data', ...), the route parameter then unexpectedly becomes {user_datum}
Even if datum can be considered to be the singular form of data,  in this case I think it can be quite confusing. Str::singular('data') returns 'data', as one would expect.
Steps To Reproduce:
dd(Str::singular('foo_data'));

We're using the doctrine inflector class for this so you'll have to open an issue on their repo.


Improve block and thread calculations and invoke only if in range

This only applies to 1d parallel_for in CUDA. If this is acceptable to you all, a similar thing should be done where else it's applicable.

See #57. This is a problem in 1d also. It's actually not "wrong". It's just that if your N is greater than the threads per block and not divisible by the same, you pass indices to the kernel that are out of bounds.

Thanks @PhilipFackler eventually we will revisit this when doing performance testing, with the apps we are targeting.

Test this please

Also, are we missing the same for AMDGPU.jl?

Also, are we missing the same for AMDGPU.jl?

Yes, but I don't have a way to check that out locally


Feature Request: Support Verilog Language

You may choose not to do this, depending on how many people want it,thanks !

Thanks for the feedback. Yeah, I'll try to implement Verilog language support. Currently my time is limited but I'll try to find a JavaScript based parser (tokenizer) for Verilog, and do some experiments to make sure it works correctly.


CLI Packages

[ ] Package Support

[ ] Homebrew for Mac
[ ] Chocolatey for Windows
[ ] Cargo for Rustup
[ ] (descoped) apt-get for Linux

How does this relate to the other packaging work? Aka, pip, nuget, etc?


Making relative links work

I added the the baseFolder param to WebView.loadDataWithBaseURL in PreviewActivity.renderMarkdown.

Closed because the PinActivity changes should not be merged

Was going to comment about that right when you closed it :+1: let's continue discussion on the new PR :) changes look good so far.


add 3 packages

fix #90

they are widely used.
node >=12 has them.

maybe es6-promise need to defined in es-shim-like. remove it temporarily.


Can't run led.py

Getting the following error when trying to run led.py,
attributeerror: module 'neopixel' has no attribute 'adafruit_neopixel'
Though I have the rpi_ws281x library installed from here: https://github.com/jgarff/rpi_ws281x
And in neopixel.py it has the following code:
class Adafruit_NeoPixel(object):
def init(self, num, pin, freq_hz=800000, dma=10, invert=False,
brightness=255, channel=0, strip_type=ws.WS2811_STRIP_RGB):
so it obviously exists..? What am I doing wrong?

The installation wasn't in the correct place. had to move the neopixel.py file from the rpi_ws281x library into the actual python3.7 package list.

Glad you were able to solve it, is there anything I should update in the documentation to help others who may run into this?

A list of things I ran into when working through getting this to work properly that the documentation could use:

Raspberry Pi 4 is also compatible, and uses GPIO12 as the PWM output. I didn't need a logic converter, I just have a 5V 200W (40A) power supply.

Had a difficult time figuring out how to install NeoPixelBus on a headless installation of Raspbian. Looks like a UI (head..full?) is required, and to be downloaded from here: https://www.arduino.cc/en/Main/Software (Linux ARM 32 bits)

I needed to also install rpi_ws281x separately, https://github.com/jgarff/rpi_ws281x , though I did do this before the NeoPixelBus installation which may have it already installed. I then had to run the following to build it:

/rpi_ws281x $ sudo scons
/rpi_ws281x $ cd python
/rpi_ws281x/python $ sudo python3 setup.py build
/rpi_ws281x/python $ sudo mv /usr/local/lib/python3.7/dist-packages/neopixel.py /home/pi/Desktop
/rpi_ws281x/python $ sudo mv /home/pi/rpi_ws281x/build/lib.linux-armv7l-3.7/neopixel.py /usr/local/lib/python3.7/dist-packages/

What this does is builds the repo, then moves the correct neopixel.py file into the python distribution packages so it can be imported correctly in visualization.py.

The neopixelbus is only for Arduino, I actually edited that an hour or so ago because I realized I put it in the Raspberry pi section rather than the Arduino section.
I'll look a little more into the other things you've mentioned. I only have experience with the arduino side of things and don't own a raspberry pi so I can't be of too much help on that front but I'm open to any suggestions you may have that could improve documentation.

Ok so I helped a friend set one up. Here's some more stuff we think would be best to add to the documentation:
I feel like a good portion of people will not have an extra monitor to easily set this up (as it ALL can be done via some terminal), so a suggestion would be how to add ssh and install raspbian (Make sure you're adding the ssh files to the boot DRIVE on linux rather than the boot DIRECTORY) to the pi. Even just putting those links would be an immense help.
Second, this repo needs to be added to the documentation. It currently does not say it needs to be installed to run strandtest.py. There may be an issue with packages as I mentioned in my previous comment.

To summarize:

On line:

Connect a PWM GPIO pin on the Raspberry Pi to the data pin on the LED strip. If using the Raspberry Pi 2 or 3, then try Pin 18(GPIO5).
Append: If using the Raspberry Pi 4b, try *GPIO 13.
*Links to the GPIO pin DATASHEET

Add a section for the rpi_2812x GitHub installation.

Please add how to build it. Says it on the repo's readme but might be helpful.
Apparently it doesn't build correctly. So this will need to be added as an example:
/rpi_ws281x $ sudo scons
/rpi_ws281x $ cd python
/rpi_ws281x/python $ sudo python3 setup.py build
/rpi_ws281x/python $ sudo mv /usr/local/lib/python3.7/dist-packages/neopixel.py /home/pi/Desktop
/rpi_ws281x/python $ sudo mv /home/pi/rpi_ws281x/build/lib.linux-armv7l-3.7/neopixel.py /usr/local/lib/python3.7/dist-packages/

What I'm doing is moving the built python file into the python package list, or else an error appears when trying to run strandtest.py.

Add the raspbian OS and ssh install links. Include:

If using Linux to install OS, make sure you're adding the ssh files to the boot DRIVE rather than the boot DIRECTORY.


Output is garbled

Your environment

ruby -v: 2.7.5p203
debug: 1.4.0
rails <IP_ADDRESS>
PLATFORMS:  x86_64-darwin-21
MacBook Pro Monterey 12.0.1

Describe the bug
In Terminal, when I break, I always get a garbled output.
Here is what happens when I type 1 + 1 and hit enter:
16:26:00 web.1  | (rdbg) (rdbg) 1    # ruby(rdbg) 1     # ruby(rdbg) 1 +    # ruby(rdbg) 1 +     # ruby(rdbg) 1 + 1    # ruby  2
To Reproduce
Expected behavior
It should work like irb.

Could you describe more to reproduce?

May be a foreman issue.
Created a fresh rails app, added a new route & action and immediately called debugger.
I then added a few things around how I'm running the server in my main app:
I have a file ./bin/dev containing:
#!/usr/bin/env bash
foreman start -f Procfile.dev "$@"
Make this executable with sudo chmod u+x ./bin/dev
And a file Procfile.dev containing:
web: bin/rails server -p 3000
When I run Procfile.dev, I get the error above with the garbled debugger.
When I run bin/rails server -p 3000 on its own, the debugger is fine.
No worries if this isn't one you can help with, given that it seems to be caused by Foreman.

I need to understand foreman's TTY.
I heard another issue on foreman so I need to check it.

Sorry I don't have anything to add or assist.  But, my debug output is unreadable also.  There's no command line to interact with---it's just blank at the end of the break output.  I can attempt to type method names or variables in there, but the output of that is buried in so much junk it takes me 10 minutes to find where the variable output was printed.  Happened as soon as I switched over from Pry to Debug.  Rails 7, iterm, MacOS, zsh.

I am experiencing this as well.  Was this issue closed as just a recommendation to setup remote debugging?


Build shared and static lib with qmake

Fix #13

Please let me know if this also works for you - on both my Mac and Linux machines I need these flags in order to build both static and shared libs.
It would be great if you could update the 0.95 release with all these latest changes to the build tools.

Or maybe let's make a 0.96 release since changing an existing one is difficult.

@cbrnr Hi Clemens,
Just tested and it works very well. Let's keep it v0.95 and treat it as a 'minor bug fix'. I have already updated the windows binaries.

OK, but what happens when you download the 0.95 source? I assume you'd still get the old source. I'd really like to have these changes also in the source because my build instructions will be based on the source. Can you delete the release and make a new 0.95 with the current master?

@cbrnr Hi Clemens,
Seems even after I deleted the v0.95 and re-release it, it will still use the old source because the name tag already exists. So I just released a v0.96.

Ok cool. You probably could delete the tag and set it to the new position, but creating 0.96 is fine.


Fixed Prefix and Website

Changed Prefix in .snakes about from ! to .
https://github.com/python-discord/seasonalbot/issues/374
Relevant Issues

Closes #374
Description

Changed prefix from ! to .
Reasoning

The prefix used to be ! but the commands were using a . prefix
Did you:

[x] Join the Python Discord Community?
[x] If dependencies have been added or updated, run pipenv lock?
[x] Lint your code (pipenv run lint)?
[x] Set the PR to allow edits from contributors?

Please don't make 3 PRs for the same feature next time - just fix the first PR. Like kwzrd said, we could've helped with this if you didn't know how.

You’re Welcome :)


add small karts and custom gliders

add small karts. they should have a sm64js logo on it and the colours can be changed, along with adding some custom gliders

so you basically want mk8 and sm64js in one package?

ok yeah why not
Nintendo made super Mario Nintendo i beg you don't ban sm64js i have yet to see :(

do not

y is there nothing to be seen here

y is there nothing to be seen here

good idea

good idea

thnx

thnx


Linting

Coverage remained the same at 100.0% when pulling f5489a1eed911299617bc1ec9523e7320f6b57c2 on linting into 959328e7833b4588a52a9f5d145eaccd9d192ad1 on master.


[Select][joy] value is missing in onChange event

To have the same experience as the native select and Material UI's select:
<Select onChange={event => {
  event.target.value
}}>

Temporary solution
We could do something similar to Material UI's Select by applying the value to the event.taget:
if (onChange) {
  // Redefine target to allow name and value to be read.
  // This allows seamless integration with the most popular form libraries.
  // https://github.com/mui/material-ui/issues/13485#issuecomment-676048492
  // Clone the event to not override `target` of the original event.
  const nativeEvent = event.nativeEvent || event;
  const clonedEvent = new nativeEvent.constructor(nativeEvent.type, nativeEvent);

  Object.defineProperty(clonedEvent, 'target', {
    writable: true,
    value: { value: newValue, name },
  });
  onChange(clonedEvent, child);
}

Ideal solution
This should be implemented in Base UI

Is this issue duplicate of https://github.com/mui/material-ui/issues/36946

Is this issue duplicate of #36946 and #34414 ?

 I'm closing this.


Add Mule language

Description
Adds a new language: Mule
Mule applications are written using XML language but unfortunately they don't have a specific file extension and uses .xml. But they have at least 2 distinctive characteristics:

The root node is called <mule>
The default namespace for <mule> node is xmlns="http://www.mulesoft.org/schema/mule/core"

In order to distinguish this language from other XML file an heuristic was added with a pattern to match those characteristics.
As reference, this was a brief discussion about how to add it: https://github.com/github/linguist/discussions/5922
Checklist:

[x] I am adding a new language.

[x] The extension of the new language is used in hundreds of repositories on GitHub.com.

Search results for each extension:

https://github.com/search?utf8=✓&type=Code&ref=searchresults&q=extension%3Axml+mule+"http%3A%2F%2Fwww.mulesoft.org%2Fschema%2Fmule%2Fcore"

[x] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):

Not applicable, I created them specially for this contribution.

Sample license(s): MIT

[ ] I have included a syntax highlighting grammar: Not applicable, reuses the one for XML.
[x] I have updated the heuristics to distinguish my language from others using the same extension.

Hi Team!
I would like to know if there is anything else from my side that I need to do in order to get it reviewed and, eventually, merged and released. How often you usually do that?
Please let me know!
Thanks in advance!

@mdaloia This addition returns Mule for all repos with xml files. For example, for this repo https://github.com/ashwinsangem/spring4-mvc-gradle-xml-hello-world/tree/master/src/main/resources Mule is being returned.
Is this expected or something wrong. Asking because we have a failing test in scout. https://github.com/github/scout/commit/66ae072026f77cc3152974fbd93989d9d9a8f9ac/checks

Oh sheeet 🤦. I completely missed that Mule was a framework which means it doesn't qualify for inclusion in Linguist. I also missed that the heuristic didn't have an alternative 🤦🤦
I'm going to revert this PR as I need to make the release today.

Done. Thanks for bringing this up @vsvipul 🙇

@vsvipul / @lildude Sorry to hear that, clearly wasn't the intention
I thought that the heuristic declaration will differentiate it from other XML files as its regex looks for a <mule> node containing a specific attribute xmlns="http://www.mulesoft.org/schema/mule/core".
I asked on this discussion before making the PR.
Also there wasn't any test that failed, neither locally nor on CI.

I completely missed that Mule was a framework which means it doesn't qualify for inclusion in Linguist.

Mule is not a framework. You construct applications using a XML and those applications are then run in the Mule Runtime.
From: https://docs.mulesoft.com/mule-runtime/4.4/

Mule runtime engine (Mule) is a lightweight integration engine that runs Mule applications and supports domains and policies. Mule applications, domains, and policies share an XML DSL (domain-specific language).

Mule applications connect systems, services, APIs, and devices using API-led connectivity instead of point-to-point integrations. Mule applications provide functionality for message routing, data mapping, orchestration, reliability, security, and scalability.

It even has an SDK: https://docs.mulesoft.com/mule-sdk/1.1/
Also, how is Mule different from XProc which is also XML based? (Web  /  Article)
The only thing that I see is that it has a specific file extension.
Do you think that there is a way to refactor the contribution to make it work in the right way?
Thanks in advance

I also missed that the heuristic didn't have an alternative

@lildude Could you elaborate more on this? How is it specified?

I thought that the heuristic declaration will differentiate it from other XML files as its regex looks for a <mule> node containing a specific attribute xmlns="http://www.mulesoft.org/schema/mule/core".

So did I and my initial testing suggested it shouldn't cause any problems.

Mule is not a framework.

Wikipedia suggests it is.

Also, how is Mule different from XProc which is also XML based?

From a quick look…. the first sentence states it's an XML-based language, and it has a specific extension. That said, this isn't really a good comparison as XProc was added over 9 years ago when GitHub was first getting support for distinguishing languages so no thought or consideration was given to these sort of things, including the differentiation between data, prose, markup and programming languages that Linguist now uses to distinguish different languages.

I also missed that the heuristic didn't have an alternative

@lildude Could you elaborate more on this? How is it specified?
Yes. The heuristic you added only has a single "case":
- extensions: ['.xml']
  rules:
  - language: Mule
    pattern: '<mule\s[^>]*xmlns="http:\/\/www\.mulesoft\.org\/schema\/mule\/core"'

Heuristics are evaluated after the XML strategy…
https://github.com/github/linguist/blob/d7799da826e01acdb8f84694d33116dccaabe9c2/lib/linguist.rb#L62-L71
… which means all other .xml files which weren't caught by the earlier XML strategy or this heuristic would fall through to the classifier which isn't necessary. Adding a final -language: XML would prevent this:
- extensions: ['.xml']
  rules:
  - language: Mule
    pattern: '<mule\s[^>]*xmlns="http:\/\/www\.mulesoft\.org\/schema\/mule\/core"'
  - language: XML

I didn't get the chance to look closer yesterday, but now I do, I can see this is precisely why this causes a problem for other .xml. The number of Mule .xml sample files far outweighs the number of other .xml sample files and as such your files have a much heavier influence on training the classifier than the regular .xml samples.
We can see this when analysing the file @vsvipul highlighted:
$ LINGUIST_DEBUG=1 bundle exec bin/github-linguist tmp/spring4-mvc-gradle-xml-hello-world/src/main/resources/logback.xml --breakdown
tmp/spring4-mvc-gradle-xml-hello-world/src/main/resources/logback.xml: 26 lines (19 sloc)
  type:      Text
  mime type: application/xml
      Mule =   -736.148 +  -5.703 =   -741.851
       XML =   -775.102 +  -3.652 =   -778.755
  language:  Mule

$

The fact you see that breakdown indicates we've entered the classifier. Of course, we could add more legit .xml samples, but we shouldn't need to.
There's one other reason this has come to light: with this PR Mule is the only .xml language marked as a "programming" language. Normal XML is considered "data" and thus doesn't appear in the language statistics by default.

@lildude
First of all, thank you very much for all the comments and explanations! Really helpful and appreciatted!
I locally added the alternative case in the heuristics and updated the test as you mentioned and worked like a charm. In order to be sure I did the following steps (kind of a regression test):

Ran the failed example reported here (which failed as shown)
Ran the tests (ok, as it lacked of the non-Mule XML file tests)
Updated the heuristics test and ran the tests (failed, as expected as it didn't had the alternative case)
Added the alternative case and ran the tests (Ok)

Regarding your comments/concerns,

Mule is the only .xml language marked as a "programming" language. Normal XML is considered "data" and thus doesn't appear in the language statistics by default.

If the focus is strictly on the syntax then yes, Mule is XML using particular namespaces to develop applications.
I agree that XML per se is "data", but a XML that represents an application logic I think that fits as "programming".
Mule DSL is the Mule Language that is represented in the XML format, which lets you build you application logic.
If in the future Mule supports a custom syntax to define application logic, would it then be a candidate for "programming" categorization?

I don't think it should be considered a programming language.

I think that the line between what is and what is not a programming language is somewhat blurred.
Would it help to clarify what are the "requisites" for a language to be defined as "programming"?

Mule is the engine; XML is the language specifically used to configure/control it (the Mule "eco-system" is the specific domain). From what I can see, these XML files are not hand written and are used as data to describe various services and applications and the connections between them, as such are more befitting of "data" than "programming".

This isn't accurate. It isn't to describe and control the Runtime. Via the XML you can define your application logic using among other things control structures like an if (choice), for loops, data transformations, error handlers, batch processing, and more (https://docs.mulesoft.com/mule-runtime/4.4/about-components).
You can for example implement the endpoints of an OpenAPI definition and expose a HTTP/S listener to serve requests.
For more info you could see: https://docs.mulesoft.com/mule-runtime/4.4/mule-app-dev

these XML files are not hand written

There is a special IDE (Anypoint Studio, based on Eclipse) to help to assist developing Mule applications in a visual way, but it is technically possible (and sometimes maybe needed) to modify the XML by hand.
I hope that this information provides some clarification to determine which will be its categorization.
If something it is still unclear please let me know and I will happily try to clarify it.
Thanks in advance!

I agree that XML per se is "data", but a XML that represents an application logic I think that fits as "programming".

We can't classify projects based on how they're using languages, only what languages they're using. I agree that the distinction between "data" and "programming" is nebulous at best, particularly with regards to low-level formats generated by another program (e.g., Ninja). The best we can do is classify languages based on their most common real-world applications. For XML, that means general-purpose data serialisation.

Any additional thoughts @Alhadis?

IMHO, Mule should be added as an XML extension, but nothing more specific than that. Going forward, all "XML-based" formats should be added as XML (unless a dedicated grammar is used for highlighting, in which case a separate language entry is justified).


CosmosDiagnostics Bug: ITrace on large number of retries causes JSON depth exception

If there are hundreds of retries on an operation and a ToString() is done it will hit a JSON depth exception. Internal team hit it in the latest sync. The issue is the retry handler nests the ITrace instead of making all the retries at the same level.
To reproduce use the transport handler to inject a large number of 449s.
This is a regression that was introduced in 3.17.0 in PR #2097
There should be 2 separate fixes:

ITrace needs to be fixed to not nested retries which will prevent the JSON depth exception.
ITrace should have a max number of retries it records to avoid running out of memory.

The baselines do show the nesting:
https://github.com/Azure/azure-cosmos-dotnet-v3/blob/8bf3857ff0c6bbe5c4ce6197cc82aeca3c9e684d/Microsoft.Azure.Cosmos/tests/Microsoft.Azure.Cosmos.EmulatorTests/BaselineTest/TestBaseline/EndToEndTraceWriterBaselineTests.PointOperationsExceptionsAsync.xml#L276
I believe the nesting would be fixed by deleting the following line, but uncertain the reason it was added :
https://github.com/Azure/azure-cosmos-dotnet-v3/blob/8bf3857ff0c6bbe5c4ce6197cc82aeca3c9e684d/Microsoft.Azure.Cosmos/src/Handler/AbstractRetryHandler.cs#L25

I already have a fix for this; will submit once the latest sync is complete.