MSID: 91406 Version: 2 DOI: 2<IP_ADDRESS>0846

[PLACE MANUSCRIPT AND EDITOR DETAILS HERE WHEN AVAILABLE] See step 3
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=91406)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/91406/v2)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews and QC
Who can help: Production team

[ ] QC passed or QC not needed

Who can help: Editorial team

[ ] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>0846)
[ ] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[ ] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/91406/v2/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/91406)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/91406

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/91406

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/91406/v2/91406-v2.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/91406)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

Full text for version 4 not yet available.

Full text is now available ofr v4, but mECA package is not:
curl 'https://api.biorxiv.org/meca_index/elife/all/2<IP_ADDRESS>0846' | jq
{
  "status": [
    {
      "count": 1,
      "messages": "ok"
    }
  ],
  "results": [
    {
      "msid": "490846",
      "tdm_doi": "10.1101/2<IP_ADDRESS>0846",
      "ms_version": "3",
      "filedate": "2023-07-28",
      "tdm_path": "s3://transfers-elife/biorxiv_Current_Content/August_2023/14_Aug_23_Batch_1703/f37a934c-6d9b-1014-943a-815b7fa9e471.meca",
      "transfer_type": "api"
    }
  ]
}


:robot: triggering CI on branch 'release-next' after synching from upstream/main

Codecov Report

Merging #631 (d05bd6d) into release-next (d05bd6d) will not change coverage.
 Consider uploading reports for the commit 102a42e to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@              Coverage Diff              @@
##           release-next     #631   +/-   ##
=============================================
  Coverage         49.07%   49.07%
=============================================
  Files                96       96
  Lines             13099    13099
=============================================
  Hits               6428     6428
  Misses             6037     6037
  Partials            634      634

OCF Webhook is merging this PR


etcd cluster startup flakiness with TLS enabled

Reported by @matte21 on https://github.com/kubernetes/kubernetes/issues/81508#issuecomment-590646553:

I used to deploy a 3-member etcd v3.3.11 cluster on a Kubernetes 1.16 IKS cluster. The etcd cluster was used to store CRs defined via API aggregation, and was managed via coreOS's etcd operator v0.9.4.
I had successfully deployed the etcd cluster and accessed the API objects stored in it countless times on the very same IKS cluster, but one day, after deploying it, the weird behavior began. More precisely, the first member would always start up successfully, but the second member encountered an error and got stuck in failed state without being replaced. Its logs terminated with

2020-02-10 22:23:40.466759 W | etcdserver: could not get cluster response from https://the-etcd-cluster-qdwdv2ctn7.the-etcd-cluster.example-com.svc:2380: Get https://the-etcd-cluster-qdwdv2ctn7.the-etcd-cluster.example-com.svc:2380/members: EOF
2020-02-10 22:23:40.467349 C | etcdmain: cannot fetch cluster info from peer urls: could not retrieve cluster information from the given urls

where the-etcd-cluster-qdwdv2ctn7 is the first member of the etcd cluster. The third member's pod was never created. This behavior happened around 80% of the times. In the remaining 20% the cluster got up and running and operated correctly; the 1st and second members of the cluster were up and running immediately, while the third member failed to start, got replaced (too quickly for me to intercept its logs) and then started correctly. Both the 2nd and 3rd member logs contained messages like:

2020-02-11 14:11:03.413970 I | embed: rejected connection from "<IP_ADDRESS>:47446" (error "tls: "<IP_ADDRESS>" does not match any of DNSNames [".the-etcd-cluster.example-com.svc" ".the-etcd-cluster.example-com.svc.cluster.local"]", ServerName "the-etcd-cluster-nqkqzxqgzt.the-etcd-cluster.example-com^C I | embed: rejected connection from "<IP_ADDRESS>:47458" (error "tls: "<IP_ADDRESS>" does not match any of DNSNames [".the-etcd-cluster.example-com.svc" ".the-etcd-cluster.example-com.svc.cluster.local"]", ServerName "the-etcd-cluster-nqkqzxqgzt.the-etcd-cluster.example-com.svc", IPAddresses [], DNSNames [".the-etcd-cluster.example-com.svc" ".the-etcd-cluster.example-com.svc.cluster.local"])

I observed the issue only when TLS was enabled, I did many trials w/o TLS and the cluster started and operated perfectly fine.
I also tried many deployments with TLS on but different etcd versions. I observed the issue with
v3.2.0 and v3.4.3, never with v3.1.15, v3.1.16 and v3.1.20.
After some experiments without solving the issue I deleted my IKS cluster and got a new one with Kubernetes 1.15. I've never seen the issue again since then.
For completeness:
manifest of the etcd cluster: https://github.com/MikeSpreitzer/kube-examples/blob/add-kos/staging/kos/deploy/etcd-cluster/48-etcd.yaml
manifest of the etcd operator: https://github.com/MikeSpreitzer/kube-examples/blob/add-kos/staging/kos/deploy/etcd-operator/47-eo-theop.yaml

cc @MikeSpreitzer @hexfusion

@matte21 Can you provide any details about what certs etcd-operator generated in this case? Do you know what dns names were in the Subject Alternative Names?

The etcd cluster members used static TLS:
apiVersion: etcd.database.coreos.com/v1beta2
kind: EtcdCluster
metadata:
  name: the-etcd-cluster
  namespace: example-com
spec:
  size: 3
  version: "3.3.11"
  TLS:
    static:
      member:
        peerSecret: etcd-peer
        serverSecret: etcd-server
      operatorSecret: etcd-client
...

The certificates and the secrets carrying them where created by a deployment script. Follow the cnf for the three certs.
Peer:
[req]
req_extensions = v3_req
distinguished_name = req_distinguished_name
[req_distinguished_name]
[ v3_req ]
basicConstraints = CA:FALSE
keyUsage = nonRepudiation, digitalSignature, keyEncipherment
extendedKeyUsage = clientAuth, serverAuth
subjectAltName = @alt_names
[alt_names]
DNS.1 = *.the-etcd-cluster.example-com.svc
DNS.2 = *.the-etcd-cluster.example-com.svc.cluster.local

Server:
[req]
req_extensions = v3_req
distinguished_name = req_distinguished_name
[req_distinguished_name]
[ v3_req ]
basicConstraints = CA:FALSE
keyUsage = nonRepudiation, digitalSignature, keyEncipherment
extendedKeyUsage = clientAuth, serverAuth
subjectAltName = @alt_names
[alt_names]
DNS.1 = *.the-etcd-cluster.example-com.svc
DNS.2 = the-etcd-cluster-client.example-com.svc
DNS.3 = the-etcd-cluster-client
DNS.4 = localhost

Client:
[req]
req_extensions = v3_req
distinguished_name = req_distinguished_name
[req_distinguished_name]
[ v3_req ]
basicConstraints = CA:FALSE
keyUsage = nonRepudiation, digitalSignature, keyEncipherment
extendedKeyUsage = clientAuth, serverAuth

As an addition: after the second member of the etcd cluster got stuck in failed state, the etcd operator logs contained:

time="2020-02-11T13:57:33Z" level=info msg="Start reconciling" cluster-name=the-etcd-cluster cluster-namespace=example-com pkg=cluster
time="2020-02-11T13:57:33Z" level=info msg="running members: the-etcd-cluster-bxbxwtwvpf" cluster-name=the-etcd-cluster cluster-namespace=example-com pkg=cluster
time="2020-02-11T13:57:33Z" level=info msg="cluster membership: the-etcd-cluster-rnq8c9wtzz,the-etcd-cluster-bxbxwtwvpf" cluster-name=the-etcd-cluster cluster-namespace=example-com pkg=cluster
time="2020-02-11T13:57:33Z" level=info msg="Finish reconciling" cluster-name=the-etcd-cluster cluster-namespace=example-com pkg=cluster
time="2020-02-11T13:57:33Z" level=error msg="failed to reconcile: lost quorum" cluster-name=the-etcd-cluster cluster-namespace=example-com pkg=cluster

On @MikeSpreitzer 's prompt, here's a list of issues for whom the observed behavior was similar to the observed one in this issue:
https://github.com/coreos/etcd-operator/issues/1330
https://github.com/coreos/etcd-operator/issues/1300
https://github.com/coreos/etcd-operator/issues/1962
https://github.com/etcd-io/etcd/issues/8803
https://github.com/etcd-io/etcd/issues/8268
https://github.com/kubernetes/kops/issues/6024

Is there a solution for this issue?

@myazid not that I know of. Are you experiencing it? If so, could you post more details?

I saw this error in the logs
2020-03-22 23:32:33.713411 I | embed: rejected connection from "<IP_ADDRESS>:53494" (error "tls: \"<IP_ADDRESS>\" does not match any of DNSNames [\"*.orchestrator-etcd-cluster.default.svc\" \"*.orchestrator-etcd-cluster.default.svc.cluster.local\"]", ServerName "orchestrator-etcd-cluster-2q52srppbq.orchestrator-etcd-cluster.default.svc", IPAddresses [], DNSNames ["*.orchestrator-etcd-cluster.default.svc" "*.orchestrator-etcd-cluster.default.svc.cluster.local"])
ectd did a reverse lookup by IP and I test lookup by IP
nslookup <IP_ADDRESS>
<IP_ADDRESS>.in-addr.arpa    name = 10-244-1-3.orchestrator-etcd-cluster-client.default.svc.cluster.local.
domain is not in the DNSNames.
Server TLS define
CN: *.orchestrator-etcd-cluster.default.svc
SANs: *.orchestrator-etcd-cluster.default.svc.cluster.local, *.orchestrator-etcd-cluster-client.default.svc, *.orchestrator-etcd-cluster-client.default.svc.cluster.local, orchestrator-etcd-cluster-client.default.svc.cluster.local, localhost

Observation: I was experiencing the same problem. However, when I configured PVCs, the etcd pod would take significant time (~50s) to start after it had been spawned. In this case, the DNS name mismatch did not occur.
Hypothesis: Could it be the case that if the newly spawned pod is too quick in connecting to the existing pod, the DNS request is resolved by a cache that has not had time to pick up the recent IP addr/DNS name addition?


Supporting 4 line Winstar WEH displays

It turns out there are some Winstar WEH displays that support more than 2 lines specifically the Winstar WEH002004.  While these displays do use the ws0010 controller, they do NOT support the graphics mode unlike the WEH001602 displays which do.  So the luma.oled.display.ws0010 and the luma.oled.display.winstar_weh drivers will not work with them.  What they need is a driver that is largely the same as luma.lcd.display.hd44780 but with the initialization logic of the ws0010.  This should support any ws0010 powered device that needs to operate in character mode.
So my question is, would you prefer to see the character-mode functionality added to luma.oled.display.winstar_weh or the creation of a new driver perhaps named luma.oled.display.winstar_weh_character?
It is worth noting that I do not own a display to test this on.  However, there is someone requesting this functionality over in the pydPiper project.  I've already implemented a version of the needed driver within pydPiper and I can see if the user is willing to test the resulting luma driver once it is completed.

Perhaps a new driver.. is there anything more specific we could use than luma.oled.display.winstar_weh_character ?

Well, there are only a few displays using the ws0010 controller that I think would require the character-based approach but it's hard to tell exactly which ones need it due to the lack of specific, model-by-model documentation on winstar's website.  According to the ws0010 documentation I have, the graphics addressing is limited to 100x16 pixels (e.g. 20 chars x 2 lines) so I'd assume that any WEH display that advertises larger values than that are stuck to character mode.
As for naming, I could create a master class that supports everything (perhaps a modification of the existing weh driver) with subclasses that use the exact model number of the display with appropriate defaults set for the initialization parameters.


About swaylock executable location

Hi,
I've compiled your program on debian 11 bullseye but as you know swaylock is removed.
I have to modify the swaylock executable location from /usr/bin/swaylock to /usr/local/bin/swaylock in swaylockd.c and in Makefile because I compiled swaylock from the git repo sources and I have installed it here.
So what I suggest you is to put a control of the location of the executable of swaylock instead of the absolute location.
Best regards

It is configurable on build time – look at SWAYLOCK_PATH in Makefile.
I assume that you have used the prebuilt binary, right?

Yes it's my fault, sorry.
No, I compile from source.


Error Deserialize Invalid JSON primitive

Azure DevOps Extension you are using
Generate Release Notes Build Task
Where are you running it?
You can run the extensions in the cloud or on-premise

Azure DevOps Service (VSTS)

Version of Extension/Task
Last pubic
Expected behaviour and actual behaviour
tasklog_5.log
Steps to reproduce the problem
Process CD VSTS

If you enter the following URL, from the script in browser, as a user with suitable creds i.e. can login to the Azure DevOps organsiation
https://vsrm.dev.azure.com/lcouveignes//POC_Deploiement_Function_Azure/_apis/release/releases/48?api-version=3.0-preview
It should list some JSON, my guess is it will not and will give an error message.
Also try
https://vsrm.dev.azure.com/lcouveignes//POC_Deploiement_Function_Azure/_apis/release/releases?api-version=3.0-preview
This should show all the releases available.
It looks like an access problem.
Other option is to try my newer Cross platform version that uses newer API calls

his url is automatically automatically. I can not parameter.
How can I solve the problem of access?

The build/release agent is presumably running as a user in the build service accounts group. By default members of this group should have access to the API. So I am surprised you are seeing this problem.
A few things to check

who is the build agent running as, can they see this URL
do you get the same problems with the Microsoft hosted build agent

As you are asking question about uploading the results of this task, can I assume you are work?
if so can you provide details of the solution in this call in case it help others in the future

I work with azure devops online.
After nothing special about my Ci / CD

When it works with hosted but not with private it usually means

rights on the Agent PC
rights of the account running the agent
firewalls blocking calls

As per my standard practice I am closing this issue as I have heard nothing for a while. If this issue remains a problem please do not hesitate to re-open it


Add OpenAPI documentation for models

Codecov Report
Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (8078141) 20.45% compared to head (f653810) 20.45%.

Files
Patch %
Lines

pkg/model/backup_routine.go
0.00%
3 Missing and 2 partials :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #105   +/-   ##
=======================================
  Coverage   20.45%   20.45%
=======================================
  Files          28       28
  Lines        1325     1325
=======================================
  Hits          271      271
  Misses       1023     1023
  Partials       31       31

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


Duplicate of https://github.com/owncloud/ocis/issues/574 ?


Implemented AutoAssign functionality to NextJS Admin Dashboard

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

#26  👈
#25
main

This stack of pull requests is managed by Graphite.


FailedToRegisterForRemoveNotifications should be FailedToRegisterForRemoteNotifications

In the Note:
Note

Registering for remote notifications can fail in situations such as no network connection. You can choose to override the FailedToRegisterForRemoveNotifications method to handle registration failure.

It should be FailedToRegisterForRemoteNotifications.

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

ID: 92f390d9-cbd0-ab8c-6c8c-49df76d1b8e0
Version Independent ID: 15739c2f-afbb-c21d-b96f-fabe7060e410
Content: Send and receive push notifications with Azure Notification Hubs and Xamarin.Forms - Xamarin
Content Source: docs/xamarin-forms/data-cloud/azure-services/azure-notification-hub.md
Product: xamarin
Technology: xamarin-forms
GitHub Login: @profexorgeek
Microsoft Alias: jusjohns

Hi @c-lamont
Looks like this has since been fixed. Regardless, thanks for letting me know.


Individual Membership Subgroup Recommendation

At the Individual Membership Subgroup meeting on 2018-08-20 (#364), we came to a preliminary agreement that we have strong consensus that individual membership is important to the success of Node.js.
A unanimous vote to continue Individual Membership was supported by @dshaw @bnb @mhdawson @williamkapke @codeekage @chowdhurian.
In the coming weeks, we will continue our work defining the nature, scope and technical requirements for a revised implementation of Individual Membership.

#390 is the direct result of this initial motion to define the scope and technical requirements for a revised Individual Membership.
Closing this issue for now, please feel free to reopen when necessary! 💖


GMSA - missing information?

Basically I miss the complete installation instructions for CEP / CES. Information I can find is fragmented and sometimes outdated.
In the section Configuration is stated "You can also configure any user service account, MSA, or GMSA for CES to work."
We can get it to work with a service account, but of course we prefer a GMSA and then we get stuck with WS_E_ENDPOINT_ACCESS_DENIED .

Can GMSA also be used for CEP, or only for CES?
What extra configuration would be necessary (like logon as a batchjob/logon as a service/NTFS permissions etc)?

Also, the refences to the two basic installation articles (Certificate Enrollment Policy Web Service Guidance and
Certificate Enrollment Web Service Guidance) are helpful, but these are with a note "We're no longer updating this content regularly.". Unfortunately, the author has passed away and the feedback links on this page point to another dead end.
Any help greatly appreciated!

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

ID: 1a15cfbb-d54a-0533-edde-464554b94b7f
Version Independent ID: 4d47d383-956e-e580-505b-21ec55278984
Content: Configuring Certificate Enrollment Web Service for certificate key-based renewal on a custom port
Content Source: WindowsServerDocs/identity/solution-guides/certificate-enrollment-certificate-key-based-renewal.md
Product: windows-server
Technology: security
GitHub Login: @Deland-Han
Microsoft Alias: delhan

Hi, our engineer points out that the access denied error can be caused by various reasons like authentication failure. Please contact an expert or our support for help.
Please note as mentioned in the article, this setup is created for a specific requirement and has limited supportability.
#please-close

Ok, thanks. We solved this with help from MS support. Still, information present at [docs.]microsoft.com is fragmented and incomplete, making it hard to implement this without help.


Euler Angles Problem

When I tried to get angles as euler form just you did Euler_X is perfectly correct but euler_y and euler_z are not correct. Suddenly they jump to high degrees. What is the problem, thanks.

@ismailtoprak Hi! Could you please provide more information? Demo source code, used hardware, SDK and all other information, that can make it clear what is wrong?

@ismailtoprak  Hi , How did you overcome this issue I'm having the same problem?
using F1 with cube IDE
log.txt

please refer to the angular velocity  and linear linear acceleration values in log.txt


Create topics with Kubernetes job

Currently we create the Kafka topics with a bash script. Instead we need to do it with a Kubernetes job, which looks for the Kafka connection and the topics in a configMap and creates the topics if they aren't there.

Conceptually it's clear to me what we plan to do here. I'm unsure I understand who's responsible for scheduling the job

@lucapette
You mean if it's the responsability of airy create or the airy config apply command?

@lucapette At the moment we can deploy the job with helm, with the hook on install. That way it will run only once, when the chart is installed. The job will wait for the Kafka service to be available and will the create the topics.
In the future, we can also choose to create the job via the airy create command and wait for it to complete, before proceeding.
For facilitating the preview of the status with airy create (and helm instlall also), one option would be to schedule a status job which waits for all the prerequisites, provisioning and components to be there. When everything is created - it will complete. It can also read the provisioning checklist from a configMap if we want to get more granular and exact to the stage the process is in particular moment. Almost all the outputs which are generated on the fly, are written to a configMap from helm (random tokens and strings), so no need to output anything in particular from this job. It will have the sole purpose to check that all the components are deployed correctly.

Creating topics implemented with kubernetes job.


Minimized use of unsafe and a bit of changes to the api

I've also changed some of the types in the ibase module back to the originals generated by bindgen, i think it's better
to not change the autogenerated code to simplify moving to newer versions of the fbclient, and the module
could be autogenerated if we can make static linking work, what do you think?

I've also changed some of the types in the ibase module back to the originals generated by bindgen, i think it's better
to not change the autogenerated code to simplify moving to newer versions of the fbclient, and the module
could be autogenerated if we can make static linking work, what do you think?

I agree. I will make a small changes in the error.rs and the examples, but the PR are OK and we can accept then


install-chef-suse: Create admin node in DB

We expect that the admin node already exists in the database and the
mechanism for the nodes doesn't catch this case.
Needs to go together with https://github.com/crowbar/crowbar-core/pull/920

Gating is failing with
uninitialized constant Node (NameError)


Adding missed MIT license in a few files

Summary of the Pull Request
Adding missed MIT license in a few files.
PR Checklist

[X] Closes #12062
[X] CLA signed. If not, go over here and sign the CLA

@msftbot merge this in 5 minutes


:seedling: Fix grammer, spell, and punctuation issues in api description.

Summary
The PR aims to enhance the overall quality of the API description section.

/assign @oafisher

@xuezhaojun Thanks for making these updates, I've gone through all the descriptions and added my suggestions. I mostly removed passive voice as per IBM style, otherwise they mostly looked good.

@xuezhaojun Thanks for making these updates, I've gone through all the descriptions and added my suggestions. I mostly removed passive voice as per IBM style, otherwise they mostly looked good.

Thanks @oafischer, all suggestions are updated.

/assign @qiujian16
Improved the descriptions of CRDs.


Gradle 6.5 allocated large numbers of 'jar transforms' threads

Looks like these eventually get freed but wanted to check in that such large numbers of allocated threads weren't a surprise:
{noformat}
jstack 32507 | grep 'jar transforms' | wc -l
223
{noformat}
Expected Behavior
Not sure. That's why I'm asking.
Current Behavior
Large number of transform threads are allocated.
Context
N/A
Steps to Reproduce
Ran a build for the first time on Gradle 6.5.
Your Environment
Gradle 6.5

Since 2a1e74166bc82607e15de78002ef56582b34af0d, we've limited the number of threads that can be created to ~availableProcessors
This should probably be rolled into other worker limits, but I think this is very early in the process.


Upgrade to new sitespeed.io and Lighthouse 10

Hi @soulgalore
I do see this PR is failing
Is it related to the need of availability of chrome v122 or is there something else that i might be missing?

Hi @nachopv no I think it's failing because I need update the test. I'm gonna release this this week when I release the new sitespeed.io version.

Hi @soulgalore
The new version of lighthouse use Moto G Power by default.
Will we have the results for this device model?
Regards


Night CI 2022-11-22: Failed

action url: https://github.com/spidernet-io/egressgateway/actions/runs/3526672154

early ci bug, fixed


BotFrameworkAdapter.processActivity(): 500 ERROR - Error: Unknown

Versions
SDK v4.
NodeJS 10.15.2
Azure + MS Teams channel
Windows
Describe the bug
Bot gives HTTP Error 500 while calling any of my dialogs. Application works great on Emulator. I found that it calls twice the Bot method: async onTurn(turnContext).
I checked that https://github.com/Microsoft/botbuilder-js/issues/249 is not the case since I disconfigured QnA Maker and my LUIS quota is ok:

To Reproduce
Steps to reproduce the behavior:

MS Team Channel
Invoke any of the dialogs that prompts the user or shows more than 1 card
Bot throws this error twice:

[onTurnError]: Error: Unknown
BotFrameworkAdapter.processActivity(): 500 ERROR - Error: Unknown
(node:4468) UnhandledPromiseRejectionWarning: Error: Error: Unknown
at BotFrameworkAdapter.processActivity (D:\home\site\wwwroot\node_modules\botbuilder\lib\botFrameworkAdapter.js:469:19)
at process._tickCallback (internal/process/next_tick.js:68:7)
(node:4468) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 10)
My dependencies are:
"dependencies": {
"axios": "^0.18.0",
"botbuilder": "^4.2.0",
"botbuilder-ai": "^4.3.4",
"botbuilder-azure": "^4.3.4",
"botbuilder-dialogs": "^4.3.4",
"botframework-config": "^4.2.0",
"dotenv": "^6.1.0",
"fs": "0.0.1-security",
"request-promise": "^4.2.4",
"restify": "^7.2.3",
"sp-request": "^2.1.3",
"spsave": "^3.1.5"
},
I suspect that when the bot needs to show more than 5 cards one after the other this error is thrown. With less than 6 cards sometimes isn't reproduced. And the only channel that is failing with is MS Teams.
Cards could have images, or could be plain text.
I wonder how you would try to reproduce it. Do you want the Bot Id? Or one of the dialogs classes?
Expected behavior
This issue is not reproducible using local environment with same NodeJS version and the Emulator as is happening on Azure / MS Teams. I even tried the Web Channel as is not reproducible either.
[bug]

@textminer - What type of card are they? Are the cards being sent in the same message or separate messages?

They are separate thumbnail cards:
`function createThumbnailCardAnonymousCaseStudies(url, title, mimeType) {
var thumbnailImage = thumbnailImageIcon(mimeType);
return CardFactory.thumbnailCard(
    title,
    CardFactory.images([process.env.AZURE_STORAGE_HOST + "/icons/" + thumbnailImage]),
    CardFactory.actions([
        {
            type: 'openUrl',
            title: "File Link",
            value: url
        }
    ]),
);

}
for (var i = 0; i < sharepointFiles.length; i++) {
var card = globalEntities.createThumbnailCardAnonymousCaseStudies(longFolder + sharepointFiles[i],     sharepointFiles[i], globalEntities.getFileExtension(sharepointFiles[i]));
await turnContext.sendActivity({ attachments: [card] });
}`
And even doing multiple (more than 5 or 6):
await turnContext.sendActivity("some text");
the error is thrown.

@textminer - Just to make sure I'm understanding you correctly, are you saying the error occurs even when you send a text-only activity with no attachments? So the error has nothing to do with cards?

Exactly, the error is more related to a channel issue (with Teams), rather than which content I send.

I'm invoking same dialogs on Azure Web Chat and on Teams. First works perfect, second doesn't.

I enabled appinsights for the bot. I send one message to the bot using Teams that invokes a dialog. That is seen as two requests on the graph, whereas if I send same message using Azure Web Chat it's log correctly as only one request:

Inspecting App Insights transaction logs I found HTTP 429:

Inspecting the User Flow:

@textminer - I have successfully reproduced this in with the Node SDK. I cannot reproduce it in .NET, but it's unclear whether that's because the .NET SDK contains safeguards to prevent it or because of random happenstance. It's also unclear why this is only happening in Teams. As you might be able to guess, HTTP 429 means too many messages are being sent.
I tested in Facebook, GroupMe, Kik, Skype, Slack, Telegram, and Twilio. Kik and Slack seem to go pretty fast, but they still just send each message one after the other.  I wonder if it's a problem with the Teams connector.
My recommendation is to just send one message instead of many. Since you want to send cards, remember that one message can contain multiple attachments either as a list or a carousel. If you still want to send multiple messages you could try artificially waiting between messages, or you could switch to the .NET SDK.
Since there does seem to be a bug here, I am going to transfer this issue back to the SDK team.

@textminer - Actually, I've changed my mind. I just tried updating to botbuilder 4.3.4 and the issue stopped occurring. Perhaps the bug has already been fixed. Please update your NPM packages and see what happens.

Here's the pattern for every broken dialog transaction. It's seen duplicate below and ends with a HTTP 429 at the end:

I believe this must be related to, which this is not the cause, it's the consequence of:
https://docs.microsoft.com/en-us/microsoftteams/platform/resources/general/rate-limit

@textminer - Can you go ahead and tell me what botbuilder version you're using?

Finally, I could make the MS Teams channel work only if I introduce a sleep of 500 ms between every await stepContext.context.sendActivity() calls. This is definately related with specific Teams channel rate limits.
The botAdapter should we trapping the trace of the HTTP 429.
Could you please advise what that rate is?

@textminer - Are you seeing the posts where I'm asking you what botbuilder version you're using? You can use npm list to check the versions of all your packages.

Hi @v-kydela, I'm using botbuilder v4.3.4. Thanks for your support. Sure, I will workaround it adding a sleep time between cards of 500 ms.
+--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | --<EMAIL_ADDRESS>deduped | +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | --<EMAIL_ADDRESS>| |   --<EMAIL_ADDRESS>deduped | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | --<EMAIL_ADDRESS>| | |   --<EMAIL_ADDRESS>| | --<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>deduped
| | | +--<EMAIL_ADDRESS>deduped
| | | +--<EMAIL_ADDRESS>deduped
| | | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>deduped
| | | --<EMAIL_ADDRESS>| | |   +--<EMAIL_ADDRESS>| | |   --<EMAIL_ADDRESS>deduped
| | +--<EMAIL_ADDRESS>| | | --<EMAIL_ADDRESS>deduped | | +--<EMAIL_ADDRESS>deduped | | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>deduped | | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>deduped | | | --<EMAIL_ADDRESS>deduped

Hmm, that's interesting that you're still getting the issue in 4.3.4 and I'm not. In any case, I don't know that there's any specific rate you need to use, since the issue is so inconsistent. 500 ms is probably long enough, but my recommendation is still to just send fewer messages. Since you seem to be satisfied with your workaround, I will close this issue now. Thank you for posting!

Hey, we are somehow facing similar issue right now.
We are getting "429 Too many Requests".
This was working fine previously. Just started causing errors in 2 days.

Hi @dkbhadeshiya, the issue is not solved, you´ll need to slow down the load of messages that the bot answers. I made a workaround invoking a sleep function with 500+ ms among messages to stop receiving that error.
async sleep(millis) {
return new Promise(resolve => setTimeout(resolve, millis));
}

Hello! Is there any update in this issue, I am facing the same error in facebook channel. It was actually working just yesterday and now facebook won't send the bot the "name" of user who messaged the bot. Thanks!

hey @v-kydela , the bot was working fine now. To be clear, I didn't do any changes in the bot, but today it miraculously changed to its normal operations. I just have to ask if there was some changes in the bot channels at that time? If so, where can we have real time updates about these changes, (as some of it might cause major business impact). Thanks!
And in response to the Graph API question in stackoverflow. See this https://developers.facebook.com/docs/graph-api/reference/user/  where I just used the curl to get last name, first name and profile picture of the user. Thanks!


Brainstorming: Create a XCM dialect example.

There is an on-going discussion about implementing a common XCM dialects, letting the receiving systems to interpret all calls triggered by the Transact instructions.
As stated by Rob H the goal of having a common XCM dialect would work as a predictable and common API surface for remote systems with respect to:

Which operations they support
How to construct Xcm::Transact packets that can achieve those operations.

The idea of implementing this example on Trappist is not to define the base for developing standards, but more to provide an exploring platform based on a non production real use case.  Also as discussed with @KiChjang , this initiative might result in useful feedback for any XCM protocol change needed in regards to Dialects and versioning management.
We might focus on just one use case around our DEX pallet using the AMM functionalities. Something like:
Transact {
    payload: encode![Amm::Swap]
}

I see some prerequisites before we can start working on this:

Bring another parachain runtime besides the existing one on Trappist. We might create a parachains or runtimes folder and include have different runtimes with the same dependencies.
There is some work to be done on XCM v2 and v3 to include the logic for Dialects. Potentially there should be a version negotiations process implemented according to this comment.

Reading resources:

https://forum.polkadot.network/t/xcm-dialects-plugging-into-xcm-to-create-standardized-and-predictable-api-surfaces/384
https://github.com/paritytech/polkadot/issues/6013
https://github.com/paritytech/xcm-format/issues/30

Really interesting stuff, makes total sense to me ! However, re. the example you proposed, is this a "good example" ?
My understanding was that we want to avoid exposing operations that will lead to "chatty cross-chain interactions".
Just a thought .. let me know what you think.

Un-assigning @bidzyyys as he is currently busy with other tasks.

This approach was dismissed so there is no point on keeping this issue.


[BUG]Fps Decrease

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
[BUG]My roblox used to be over 60 FPS. After the recent update (Roblox Official FPS Unlocker), I am now at 30-40 FPS. Is there any way to fix this? Is this a roblox issue?

Without Bloxstrap, it is  way smoother for some reason,

Well you broke some setting.

Like what?

Well you broke some setting.

Like what?

Like what?

You didn't break anything it's more of a roblox issue.

Without Bloxstrap, it is way smoother for some reason,

In what way? What does your flag configuration look like?

Without Bloxstrap, it is way smoother for some reason,

In what way? What does your flag configuration look like?

FPS is smoother,

FPS is smoother.

https://github.com/pizzaboxer/bloxstrap/wiki/Uploading-your-Bloxstrap-configuration


vulkan-headers: build using ninja to fix gcc-14 compatibility

When vulkan-headers detects gcc-14 or above it enables c++ modules support. But make-mased backend does not support it and fails as:
$ nix build --impure --expr 'with import ./. {}; vulkan-headers.override {stdenv = gcc14Stdenv; }' -L

CMake Error in CMakeLists.txt:
  The target named "Vulkan-Module" has C++ sources that may use modules, but
  modules are not supported by this generator:
    Unix Makefiles
  Modules are supported only by Ninja, Ninja Multi-Config, and Visual Studio
  generators for VS 17.4 and newer.  See the cmake-cxxmodules(7) manual for
  details.  Use the CMAKE_CXX_SCAN_FOR_MODULES variable to enable or disable
  scanning.

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Added the comment
+  # Adding `ninja` here to enable Ninja backend. Otherwise on gcc-14 or
+  # later the build fails as:
+  #   modules are not supported by this generator: Unix Makefiles

*-darwin suffers from this issue on the current staging-next already.

But the darwin build seems bad anyway.  If I add ninja locally, it fails on #include <algorithm>:
source/include/vulkan/vulkan.hpp:11:10: fatal error: 'algorithm' file not found


[Feature request] More flexible MySQL datasource configuration

Currently the MySQL data storage is configured roughly here: https://github.com/openzipkin/zipkin/blob/d47e80433cdf97a78e92712f313ae607e6f61dec/zipkin-autoconfigure/storage-mysql/src/main/java/zipkin2/autoconfigure/storage/mysql/ZipkinMySQLStorageProperties.java#L89-L102
This doesn't give a whole lot of flexibility in the datasource setup. In my case I have a Galera cluster for which I can specify a JDBC url like this: jdbc:mariadb:failover://host1:3306,host2:3306,host3:3306/database?prop1=true This allows for failover in case one of the nodes is down.
Even if you use 'plain' MySQL there is still a host of failover configuration options: https://dev.mysql.com/doc/connector-j/8.0/en/connector-j-config-failover.html.
Can we perhaps add an optional property to ZipkinMySQLStorageProperties to specify the JDBC url directly and if it doesn't exist use the current implementation as a fallback?
Given a property jdbcUrl, kind of like this?
    StringBuilder url = new StringBuilder();
    if (getJdbcUrl() != null) {
      url.append(getJdbcUrl());
    } else {
      url.append("jdbc:mysql://");
      url.append(getHost()).append(":").append(getPort());
      url.append("/").append(getDb());
      url.append("?autoReconnect=true");
      url.append("&useSSL=").append(isUseSsl());
      url.append("&useUnicode=yes&characterEncoding=UTF-8");
    }

bear in mind this configuration is noted as not for production. even if we
add an env option for a full url you may still run into performance
problems due to the schema design which is intended for learning not prod.
On Fri, 14 Sep 2018, 03:57 Auke van Leeuwen<EMAIL_ADDRESS>wrote:

Currently the MySQL data storage is configured roughly here:
https://github.com/openzipkin/zipkin/blob/d47e80433cdf97a78e92712f313ae607e6f61dec/zipkin-autoconfigure/storage-mysql/src/main/java/zipkin2/autoconfigure/storage/mysql/ZipkinMySQLStorageProperties.java#L89-L102
This doesn't give a whole lot of flexibility in the datasource setup. In
my case I have a Galera cluster for which I can specify a JDBC url like
this:
jdbc:mariadb:failover://host1:3306,host2:3306,host3:3306/database?prop1=true
This allows for failover in case one of the nodes is down.
Even if you use 'plain' MySQL there is still a host of failover
configuration options:
https://dev.mysql.com/doc/connector-j/8.0/en/connector-j-config-failover.html
.
Can we perhaps add an optional property to ZipkinMySQLStorageProperties
to specify the JDBC url directly and if it doesn't exist use the current
implementation as a fallback?
Given a property jdbcUrl, kind of like this?
StringBuilder url = new StringBuilder();
if (getJdbcUrl() != null) {
  url.append(getJdbcUrl());
} else {
  url.append("jdbc:mysql://");
  url.append(getHost()).append(":").append(getPort());
  url.append("/").append(getDb());
  url.append("?autoReconnect=true");
  url.append("&useSSL=").append(isUseSsl());
  url.append("&useUnicode=yes&characterEncoding=UTF-8");
}

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openzipkin/zipkin/issues/2190, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAD610YTOTSYRo5rxq8S_czK0L3DU0uVks5uarikgaJpZM4WoGu5
.

Hmm I didn't really read** it like that. It sounded like if we really put a lot (but what is a lot ??) of data into it, the querying of certain traces and or dependencies would be slow. That is okay for now I think.
I would like to know if putting the data in, so storing the spans, would be going slower as well, thus possibly hampering the clients that are sending the data? (Brave, Sleuth)
Any ball park figure on what 'a lot of data' is?
** https://github.com/openzipkin/zipkin/tree/master/zipkin-storage talks about 'discouraged' for newer systems
** https://github.com/openzipkin/zipkin talks about "The following components are no longer encouraged, but exist to help aid transition to supported ones. "

I would classify this as use at your own risk as it is double-discouraged,
one for performance and the other for the schema.  If that's ok with you we can look about
adding the url option.
On Fri, Sep 14, 2018 at 4:57 PM Auke van Leeuwen<EMAIL_ADDRESS>wrote:

Hmm I didn't really read** it like that. It sounded like if we really put
a lot (but what is a lot ??) of data into it, the querying of certain
traces and or dependencies would be slow. That is okay for now I think.
I would like to know if putting the data in, so storing the spans,
would be going slower as well, thus possibly hampering the clients that are
sending the data? (Brave, Sleuth)
Any ball park figure on what 'a lot of data' is?
** https://github.com/openzipkin/zipkin/tree/master/zipkin-storage talks
about 'discouraged' for newer systems
** https://github.com/openzipkin/zipkin talks about "The following
components are no longer encouraged, but exist to help aid transition to
supported ones. "
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/openzipkin/zipkin/issues/2190#issuecomment-421281391,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAD613wPGxUcKx1h9nd28UP65oIpngj2ks5ua2-UgaJpZM4WoGu5
.


I should be able to select a fee rate when opening a new channel

Detailed Description
Add fee selection widget to open channel form

Context
New Channel UI

@JimmyMow do you have any thoughts about what these three configuration options  should map to? (slow, medium, fast)
We can set a per byte fee...
Or a target number of block confirmations...
We currently already have a way to query https://bitcoinfees.earn.com/api/v1/fees/recommended built into the app, which we use to determine the current per byte fee that we will apply when making on-chain transactions (I think we select the fastestFee option for that). The data from that api call looks like:
{
"fastestFee": 20,
"halfHourFee": 18,
"hourFee": 6
}

Should we just map these fees directly to the slow/medium/fast options that we want to present and apply as a per-byte fee?

Closed via https://github.com/LN-Zap/zap-desktop/pull/1598


3D editor freelook only recognizes the first key assigned to a shortcut

Godot version
4.0.dev (1aef3a42b)
System information
Fedora 34, GeForce GTX 1080 (NVIDIA 470.74)
Issue description
Related to https://github.com/godotengine/godot/issues/54460.
Following the merge of https://github.com/godotengine/godot/pull/51273, the editor only recognizes the first key assigned to a shortcut for freelook shortcuts (even if none of the keys are marked as physical).
Here, only W will be used to move forward while freelook is active:

Here, only Up will be used to move forward while freelook is active:

This does not occur with other shortcuts, such as the 2D editor's Always Show Grid shortcut:

Here, both # and L work as expected.
This is the part of the code that handles 3D editor freelook shortcuts: https://github.com/godotengine/godot/blob/1aef3a42b234b86c5db21d5a64509453e6bf145c/editor/plugins/node_3d_editor_plugin.cpp#L2606-L2623
Steps to reproduce
Open the Editor Settings, go to the Shortcuts tab and add a second shortcut for one of the freelook movement keys.
Minimal reproduction project
N/A

The is_shortcut_pressed method is defined specifically for the 3D editor. See the method below, this is what would need to be changed.
https://github.com/godotengine/godot/blob/1aef3a42b234b86c5db21d5a64509453e6bf145c/editor/plugins/node_3d_editor_plugin.cpp#L2557-L2575

Soudns like this is_shortcut_pressed method might not be needed at all? It might predate better shortcut handling and now it's counter productive.

@akien-mga From memory I believe this is because it is checking the shortcut event(s) without having an event available from _input(Ref<InputEvent>), etc, so it needs to check it "manually" using the Input singleton.
I think you are on the right track though - this could potentially be solved by putting this checking logic into the unhandled_key_input() method, and updating an instance variable called Vector3 freelook_velocity, then in the update_freelook method, just use this vector instead of the direction vector which is created locally.
This would work in my head and I've explained the logic a bit here, so I will mark this as a good beginner issue and see if anyone gets to it before I do 😄

Hmm ok, this is slightly more complex than I first though.
_input methods are no good because they include keypress echoing, So there is a short delay between when you first press a key, and the next event if you hold it down (the echo).
What is needed is kind of like putting Input.is_action_pressed() inside _process() in a Godot game. This is basically what is done now, but there is no shortcut equivalent of is_action_pressed(). There is no method in general inside Input to check if a certain InputEvent is currently pressed - there is only is_key_pressed().  (Side note: The input singleton only keeps a record of keys pressed by using event->get_keycode(), won't this also screw up if a physical keycode is used? Not sure what the interaction with physical keycodes is here. Related to #54460)
Another pitfall of the current implementation is that if you set the freelook shortcuts to include a modifier, they also won't work since it only checks if the keycode is pressed.
Maybe some method in the Input singleton needs to be added to check if a particular event is currently pressed? This would require storing every event that comes through though, and removing it from storage when it is released. Doesn't sound good.
Overall I'm not sure what to do with this - seems tricky. Could we somehow leverage the Input Actions system, since that has pressed-checking already implemented? Like, convert those shortcuts to internal, editor-only hidden actions? Not sure how that would work but I might look into it.


Importing files into R by 'looping' over file names

I have a folder with 500 matlab files, each containing 4 matrices. I would like to import them into R via readMat, but I need a loop which, for each file in turn:
(1) uploads it, giving it a name consistent with a pattern (say model1)
(2) saves the second and third of the matrices from the file as, say model1A, model1B
(3) removes the imported object containing 4 matrices, I will only need the two, saved as described above
(4) repeat, i.e. producing model2A, model2B, repeat, model3A, model3B…
I would like to end up with a list of 1000 matrices (model1A, model1B, model2A, …, model500B).
How do I deal with this? How to loop over a file name? Does this maybe call for using apply()?

This has nothing to do with crouton, only something you want to do inside crouton. I suggest you close this issue and ask the same question on stack overflow, if you can't find the answer with Google. This issue is wasting time to process, and distracting from real issues.


Disabled cursor mode doesn't work right when connected over RDP #1276

This is a new PR to fix the problem over Remote Desktop when Cursor Mode is Disabled.
https://github.com/glfw/glfw/issues/1276

Sorry, i did not see your reply, I share your concern about this change when not using remote desktop. I'll try what you suggest soon and come back to you.

I added an check if the current session in from remote desktop based on this article I found: https://learn.microsoft.com/en-us/windows/win32/termserv/detecting-the-terminal-services-environment
If not Remote Desktop, the original code should now execute and blankCursor is NULL has before.
Tested via Remote Desktop and locally on Windows 11 with the cursor exemple project.

Many thanks, a first glance that looks great. I'll test and review shortly.

Looking through the code I am think it would be more suitable if the cursor and isRemoteSession variable were in the  _glfw.win32 variable (part of the _GLFWlibraryWin32 struct) and initialized at startup. We only need one cursor for all windows, and hopefully only need to check for RDP at app startup. What are your thoughts?

Excellent suggestion, i did the modification to move the isRemoteSession and blankCursor in _GLFWlibraryWin32 and i moved the code to init them in win_init. I really new in the glfw code, so i'm not sure i moved everything at the right place. Just tell me if everything sounds good.

Looking at the terms of use for learn.microsoft.com it seems that we likely can't use their code directly as there is no suitable license for the code.
However, luckily RemoteFX has been deprecated and removed from all versions of windows, so we only need to use GetSystemMetrics(SM_REMOTESESSION) inline to detect a remote session.
In order to ensure no copyrighted code is in GLFW repository I can squash and merge this PR, so we just need to update to using the simpler check.
In completing this PR we will also need the steps as Contributing a bug fix namely:

Change log entry in README.md, describing the incorrect behavior
Credits entries for all authors of the bug fix

However if I can add these after merging.
Note that as yet I've not gone through rigorous testing and checking of the code, which I'll do once we have the RemoteFX change done.

The code for RemoteFX has been removed and the Readme.md updated. But i don't known where and how to set "Credits entries for all authors of the bug fix".

Thanks once again for this!
The credits are in CONTRIBUTORS.md, sorted alphabetically by last name if using a full name or by handle if using that. I'll modify the guide to make that clear. I can also add your credit after merging so don't worry about this (as your handle Hilderin or your name if you want).
Could you remove the function isCurrentRemoteSession() and simply use GetSystemMetrics(SM_REMOTESESSION) > 0? That would simplify things and remove any trace of the article code. Although I could do this it might cause the merge to label the squashed commit as mine, and you've done all the work so I'd prefer to retain you as author.

The function isCurrentRemoteSession has been removed.
Thanks to add Hilderin to the contributors for me.
I hope you have good vacation!
I'm here if you find problems later in your tests.

Having tested this over Remote Desktop I can confirm the solution works.
However the issue also appears to be present with a virtual machine without a remote desktop connection, so I am investigating why (possibly related to RemoteFX as this is an older VM, so I will check if updating resolves the issue).

Interesting, i did not test on a virtual machine. From what i read, the absolute mode should be used only on remote desktop. But with my tests, it is also used with other remote tools like Parsec. Maybe it is used for Virtual machine also and we should remove the isRemoteDesktop condition completely?

I've tried setting isRemoteDesktop to TRUE and this doesn't resolve the VM issue, which I now believe to be a separate problem. So we should keep this test.

I note that much of this code appears to be a copy-paste of #1279 by @Pokechu22 ?

Yes absolutely, i based my modifications on this PR, i fixed the merge problem and made some minor modifications. Maybe i should have reference this PR from 2018 somewhere, i'm pretty new to all this. Feel free to include @Pokechu22 in the contributors.

I'm fine with being added to the contributors without any other clarification (a link to my PR in the commit message or a co-authored-by tag would be nice, but if it ends up being difficult to add one then it doesn't really matter). I meant to rebase that PR a long time ago but never really got around to it (since I don't have a good setup for testing RDP issues anymore). Thanks for handling that for me :)

Many thanks all - I'll try to add a co-authored-by tag for @Pokechu22  and will definitely add them to the GLFW contributors.

I've submitted some change requests in my review. Once those are done I should be able to merge this PR, and then I will add the credits if you don't do so yourself.

I just pushed the requested modifications. I added @Pokechu22 and myself in the Contributors. I let you add the co-authored-by in the squash commit.

Many thanks for making these changes. I will re-test later on today  and then ask @elmindreda for a final review.

I've tested this updated version and as expected it works, enabling both disabled cursor and disabled cursor with raw mouse motion modes to work over a remote desktop connection.

Merged, many thanks to you both. I was able to get @Pokechu22 as a co-author in the commit and @hilderin has already added them to the CONTRIBUTORS.md file.

This fixes an issue I ran into with windows on both sides, and the underlying issue is a quirk with how RDP works on Windows. Whether or not it will affect Linux depends on a bunch of implementation details. It seems plausible that it could fix a similar issue with Linux but it depends on how e.g. xvnc behaves when the cursor is disabled. But it's worth trying at least.

okay, just one more question. do both devices need this? or only the host machine

Only the game/other program you're running needs it.

@Emeraldiaa what problem are you having / trying to solve?


Groups

How can we use this in groups? How would I retreive the Chat ID?
I can talk with the bot in a normal chat but It doesn't reply in a group or a supergroup.
Current code:
https://github.com/KennyNL/Clevergram

did you read this

No I didn't yet. Thanks man :)

Fixed, thanks!


Incomplete shutdown

2 Fixes:

Register all boundaries generated by source connections
Dispose DataChannel along with DataReceiver

Closes #1359

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


Some PermissionListPage bug fix and make Dockerfile more productive.

Make Dockerfile more productive.

Build all-in-one image:

docker build --target ALLINONE -t casbin/casdoor-all-in-one:latest .

Build standard image:

docker build --target STANDARD -t casbin/casdoor:latest .

Solve the problem that the PermissionList page jumped abnormally when clicking the permission name in the list.

Insert the owner into the url when jumping to the permissions details page.

Fix incorrect SignupApplication field value in initBuiltInUser.

Wrong SignupApplication value causes exceptions in multiple functions for the first admin user, such as uploading avatars, user detail pages, etc.

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


Npm Install Error

npm WARN ERESOLVE overriding peer dependency
npm WARN While resolving<EMAIL_ADDRESS>npm WARN Found: @linkurious/ogma@undefined
npm WARN node_modules/@linkurious/ogma
npm WARN   peer<EMAIL_ADDRESS>from the root project
npm WARN
npm WARN Could not resolve dependency:
npm WARN peer<EMAIL_ADDRESS>from the root project
npm ERR! code E404
npm ERR! 404 Not Found - GET https://registry.npmjs.org/@linkurious%2Fogma - Not found
npm ERR! 404
npm ERR! 404  '<EMAIL_ADDRESS>is not in this registry.
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

Hi! Ogma is a commercial library, do you have an API key?


RuntimeError: maximum recursion depth exceeded in cmp (generic information extractor, peekvids.com)

youtube-dl -v https://www.peekvids.com/watch?v=cAvbo9PSFJC                                                                                [945/1940]
[debug] System config: [u'--recode-video=mp4', u'-f', u'bestvideo+bestaudio/best']
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'https://www.peekvids.com/watch?v=cAvbo9PSFJC']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.10.07
[debug] Python version 2.7.10 - Darwin-17.0.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 3.3.4, ffprobe 3.3.4, rtmpdump 2.4
[debug] Proxy map: {'http': 'http://<IP_ADDRESS>:8118', 'https': 'http://<IP_ADDRESS>:8118'}
[generic] watch?v=cAvbo9PSFJC: Requesting header
WARNING: Falling back on generic information extractor.
[generic] watch?v=cAvbo9PSFJC: Downloading webpage
[generic] watch?v=cAvbo9PSFJC: Extracting information
[generic] watch?v=cAvbo9PSFJC: Downloading MPD manifest
Traceback (most recent call last):
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 162, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/usr/local/bin/youtube-dl/__main__.py", line 19, in <module>
  File "/usr/local/bin/youtube-dl/youtube_dl/__init__.py", line 465, in main
  File "/usr/local/bin/youtube-dl/youtube_dl/__init__.py", line 455, in _real_main
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1980, in download
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 784, in extract_info
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 434, in extract
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/generic.py", line 2890, in _real_extract
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 2175, in _parse_html5_media_entries
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 2145, in _media_formats
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 1745, in _extract_mpd_formats
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 532, in _download_webpage_handle
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 503, in _request_webpage
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2190, in urlopen
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 431, in open
    response = self._open(req, data)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 449, in _open
    '_open', req)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 409, in _call_chain
    result = func(*args)
  File "/usr/local/bin/youtube-dl/youtube_dl/utils.py", line 1086, in https_open
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 1194, in do_open
    h.request(req.get_method(), req.get_selector(), req.data, headers)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 296, in get_selector
    return self.__r_host
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
(...)
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 255, in __getattr__
    return getattr(self, attr)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 251, in __getattr__
    if attr[:12] == '_Request__r_':
RuntimeError: maximum recursion depth exceeded in cmp

This is a Python bug fixed in 2.7.12 (https://bugs.python.org/issue6500). You may want to use either official Python packages from https://www.python.org or third-party packages like Homebrew or Macports.


UNITY-211 - update generic hands to have prefix at root level

and update bone weights to reduce shard at back of hand
Pull Request Templates
Switch template by going to preview and clicking the link - note it not work if you've made any changes to the description.

default.md - for contributions to stable packages.
lightweight.md - for contributions to pre-release/preview packages use the lightweight merge request template. The quality threshold for reviewing is lower - it prioritizes a lean review process.
release.md - for release merge requests.

You are currently using: lightweight.md
Note: these links work by overwriting query parameters of the current url. If the current url contains any you may want to amend the url with &template=name.md instead of using the link. See query parameter docs for more information.
Summary
merge the changes from the branch breakingchanges/generichandmodel
Contributor Tasks
These tasks are for the merge request creator to tick off when creating a merge request.

[ ] Pair review with a member of the QA team.
[x] Add any release testing considerations to the MR for the next release.
[x] Consider any licensing/other legal implications for this MR e.g. notices required by any new libraries.
[ ] If this MR closes a Jira issue, make sure the fix version on the JIRA issue is set to the correct one.

Reviewer Tasks
Add any instructions or tasks for the reviewer such as specific test considerations before this can be merged.
Use emojis in review threads to communicate intent and help contributors.

[ ] Code reviewed.
[ ] Non-code assets e.g. Unity assets/scenes reviewed.
[ ] Checked and agree with release testing considerations added to MR for the next release.

Closes JIRA Issue
If this MR closes any JIRA issues list them below in the form Closes PROJECT-#
Closes UNITY-211

@Greg-UL
Test considerations:

Ghost hands and Outline hands work and look right in all three hand example scenes (standard, URP and HDRP)

make sure the new model looks the same / very similar to old one
hand work in all orientations and poses (check for shards on back of the hand when hand is visible from the side and hand is bend at the wrist -> these should be improved now)

hand scaling works

the standard tests we do for hand scaling pass (eg. changing the scale offset, changing fingertip offsets, enabling / disabling scale)

@Greg-UL Test considerations:

Ghost hands and Outline hands work and look right in all three hand example scenes (standard, URP and HDRP)

make sure the new model looks the same / very similar to old one
hand work in all orientations and poses (check for shards on back of the hand when hand is visible from the side and hand is bend at the wrist -> these should be improved now)

hand scaling works

the standard tests we do for hand scaling pass (eg. changing the scale offset, changing fingertip offsets, enabling / disabling scale)

@JuliaRonnebergerUL tests look good the main one I can think of that I can't see recorded is validating that the correct prefix has been applied in the hierarchy on adding the given hand models into a scene.

@Greg-UL

Also will these changes be effective on existing scenes or will this only work when a new model is introduced to a scene?

As I changed the prefabs, it should be effective in all existing scenes using the prefabs :) This works in our rigged hands example scenes, so it should also work for anyone else using the prefabs in their scenes


GraphQL schema docs search

This is a feature request and it's not related to any issue.
The feature is search option in graphql schema docs, i'm working on a very large graphql api, and schema docs get very large, so if i want to find a query or a mutation or ...etc i need to scroll through the whole list.
So it would awesome to have a search bar to surf through the docs.

Closing since this seems to be a duplicate of #2720, feel free to re-open if you feel that your request is in some way different. Please also update the OP with any differing mechanisms / requests to ensure it stays open!


Request for code for generating novel sequences

Thank you for your brilliant work!
I'm wondering whether the code for generating novel sequences within a protein family will be released.
Thank you,
Tong

Thanks for the compliment! This response is super late but in case it's useful -
We're not planning on updating the repo any time soon with an example of generation, but this method should work. xs and segment_sizes mean the same thing as the corresponding parameters of the embed method used in the scoring script.

Thank you! I will try this method.


Strange group resizing

Group resizing with livePreview is bugged, so I think livePreview should be disabled for this particular type of resize.
How to reproduce:

grapheditor / draw.io, create group with 2 rectangles
try to resize group by dragging left-top corner. Preview of resize does not resize rectangles but drags them. This looks ugly when decreasing size.

Live preview will be disabled for this case in the next release.


Shift should not only be for clicking

Closed by https://github.com/AlgebraicJulia/Semagrams.jl/commit/ccc10bccfea37dc10b177d81b214d28d35a5dca8


Patch to work with Pandas 2.x

Improvement Description
The code of version 2023.9.0 has some conflict with Pandas 2.x
Current Behavior
There are test suite errors with Pandas 2.x
Proposed Behavior
In the Debian packaged version of q2-types we include a patch that fixes the issue.  Feel free to take over this patch.
Hope this helps, Andreas.

q2-types is now working with pandas 2.2.2. This update will be included in the upcoming 2024.5 release (coming later this month).


[Feature] Search modules implementation

Implement module search form

Thank you @klinux :+1:
@jcanizalez can you take a look for this PR?


Create sit.txt

Have added our engineering college .
Location : India
State : Karnataka

College website is :- Siddaganga institute of Technology
City : Tumkur

@nateshmbhat Pull request merged. Thank you!

Our college hasn't given Gmails to students with the above mentioned domain name. How can I register then?

@nateshmbhat Does the college give the students email in another domain? If yes, you need to register it here. If not, you need to request your free license for JetBrains tools at https://jetbrains.com/student with your ISIC card or with a photo ID which confirms that you are the student.

No the college doesn't provide us with any other email domain as such. I
have sent the request in the jetbrains site with a pic of the ID card. Let
me know if u need more documents to verify. I have college certificates,
github repositories, passport to verify my age etc.
On 21-Dec-2017 8:17 PM, "Philip Torchinsky"<EMAIL_ADDRESS>wrote:

@nateshmbhat https://github.com/nateshmbhat Does the college give the
students email in another domain? If yes, you need to register it here. If
not, you need to request your free license for JetBrains tools at
https://jetbrains.com/student with your ISIC card or with a photo ID
which confirms that you are the student.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/3828#issuecomment-353368802, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AWM5NmLItugtsLNZRDOtsLuYQBrw23yGks5tCm9-gaJpZM4RGmJA
.


survey-html-form: add autofocus parameter

Users can provide the id of an element to focus on when the
slide is displayed. While HTML5 has an autofocus parameter
it is not universal and not all browsers support it. This
solution should work across browsers and is more in keeping
with the jsPsych object style.
Issue #1062

Thanks @chrisbrickhouse! I agree that the autofocus attribute isn't reliable and calling .focus() seems to work better. Thanks also for showing how this parameter works in the plugin's example file!
Somewhere around line 71 in the plugin, I think there needs to be a check that an element with the ID specified by trial.autofocus exists on the page, otherwise line 72 will produce an error. Perhaps there should be a console warning if no element with this ID is found. What do you think? If you agree, feel free to update the pull request or I can merge as is and then make changes.

Perfect, thanks very much!


linux: further cleanup config after drop of 4.9

Description of changes
Continuation of #201845
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Hi, could this merge request get merged @vcunat ?

Right, I think we've given enough time for further feedback.

Ty :3


Default Name Incorrect with multiple periods

create a file called validate.test.js
right click -> duplicate file  or directory
expected ... default name is validate-copy.test.js or validate.test-copy.js
actual: validate-copy.test (drops the .js extension)

To add to this (since it's really the same issue)... filenames with spaces, the change to adding -copy, and the option to not add extensions seem to further confuse the code.  Ex.
MyUserscript v2.1.user.js the duplicate is suggested to name: MyUserscript v2-copy.1.user
FWIW VSCode has a new(?) option ( "explorer.incrementalNaming": "smart" ) that if you copy MyUserscript v2.1.user.js and paste it in the same folder, the duplicate is named MyUserscript v2.2.user.js (it tries to auto increment if it spots a number pattern).
Obviously the above is a new feature, since this extension was last updated, but if this extension does get updated, maybe tapping into the new logic would save building some custom logic.

Try this one.
https://marketplace.visualstudio.com/items?itemName=axelrindle.duplicate-file


Feature request: DockBarX module

Integrates DockBarX (a popular app dock) into the panel, similar to its integration with xfce-panel via https://github.com/TiZ-EX1/xfce4-dockbarx-plugin

Closing. I'll add a reference to this in #84


Request for server API script without sessions

So I could just send a simple request and get simple response in a free-form mode, without any additional context

This is pretty much what the example_flask.py script does. Is that what you're after?

Oh god really, it is, I'm sorry
But it will be even better if I could pass generation settings with inference post request or set them in a separate request. I know it's not a big deal, I can rewrite it myself, but it will be so much better for everyone if such script will be available from the box.

Thanks. Jumped here from oobabooga's, I really like your minimal dependency approach in contrast with enourmous blob of hundred dependencies mixed with conda, keep it up!

I posted it in another issue at some point, but I have a script which sounds like what you want here: https://gist.github.com/BlankParenthesis/4f490630b6307ec441364ab64f3ce900
It mimics ooba's text-generation-webui API so it works with tavern (the main difference being the streaming API is the same port as the regular API).
I tried to keep it simple, but I ended up having to make some changes for it to work nicely. Some of those workarounds make the generate function a bit more complex than I'd like so perhaps there's a nice way to do it, but this seems to work at least. The settings mismatch is also a bit ugly: by following the existing format it excludes exllama specific settings and ignores others.
There's no way to set the model using this, but adding a route for that should be simple enough if you want.


Add support for sidecar chance configs to cloudformation

Apparently this didn't get finished.

Confirmed this gets the dead letter chance bits working.


[PHP 7.4 Compatibility] Replace array_key_exists with property_exists during response parsing

The current version of the library uses array_key_exists on objects, which is deprecated now.
This PR replaces that call with property_exists which serves much the same purpose and works identically for dynamic objects such as those returned by json_decode.
In the first commit I used isset but I opted for property_exists as the former will not accept null values, whereas the latter will, like array_key_exists would.

@Werelds Hey, Thanks to you. I have the same issue #29 and i change this file manually and the problem is solved.

I've also tested this fix and it does work

LGTM!
Thanks for the PR @Werelds - sorry this one seems to have slipped through the net!


bug: HTTP API returns datetime strings without timezone info

Overview

We have several HTTP endpoints that return datetimes
The API serializes datetimes in JSON as ISO8061 strings
According to our own conventions, all datetimes should be treated as UTC times
According to ISO8061, datetimes without timezone info are to be treated as local time
Many of our datetimes are all coming back without timezone info

See Time zone designators in the Wikipedia article for ISO8061 for more info. This is a problem because if you pass a datetime string without timezone info into a spec-conforming date library (like Date in an ES2015 engine), it will treat it as local time and give you the wrong date.
Steps to reproduce
Try to hit an endpoint with datetimes, like GET /labware/calibrations. You'll get something back like:
{
  "id": "ed323db6ca1ddf197aeb20667c1a7a91c89cfb2f931f45079d483928da056812",
  "type": "LabwareCalibration",
  "attributes": {
    "calibrationData": {
      "offset": {
        "value": [ 0.0, 0.0, 0.0 ],
        "lastModified": "2020-08-17T21:18:49.832437"
      },
      ...
    },
    ...
  }
}

Notice that 2020-08-17T21:18:49.832437 does not end with any timezone info (e.g. Z or +00.00). If you were to try to construct a date on the client side, you would get the wrong time, because it would adhere to the ISO spec and create the date in local time:
> var date = new Date('2020-08-17T21:18:49.832437')
undefined
> date.toISOString()
'2020-08-18T02:18:49.832Z'

Root cause + fix
We create most of our datetimes in Python with datetime.utcnow, which creates a timezone-less naive datetime object set to the current UTC datetime.
According to the Python docs:

Warning: Because naive datetime objects are treated by many datetime methods as local times, it is preferred to use aware datetimes to represent times in UTC. As such, the recommended way to create an object representing the current time in UTC is by calling datetime.now(timezone.utc).

Accordingly, we need to change all instances of datetime.utcnow() to datetime.now(timezone.utc) so that we are creating datetimes that are aware that they represent time in UTC and will thus serialize to a dateime string including timezone info. You can try this out for yourself in the REPL
>>> from datetime import datetime, timezone
>>> datetime.utcnow().isoformat()
'2020-08-17T20:31:29.776284'
>>> datetime.now(timezone.utc).isoformat()
'2020-08-17T20:31:39.497474+00:00'

Dang. Good catch!


Merge 1.1.x tags

:tada: This PR is included in version 1.1.3 :tada:
The release is available on:

1.1.3
GitHub release

Your semantic-release bot :package::rocket:


Cocoapods: Document and automate deploying

There's a draft of the currently-followed steps here: https://raw.githubusercontent.com/jcanizales/grpc/deploy-cocoapods-instructions/src/objective-c/DEPLOY.md
@beriberikix has automated this kind of things in the past. He's found this flow to work for them: https://fuller.li/posts/automated-cocoapods-releases-with-ci/
@makdharma FYI :)

Pasting the content of DEPLOY.md.
Test latest version of BoringSSL

Go to boringssl's repo and copy the SHA of the last commit in master: https://boringssl.googlesource.com/boringssl/+/master
Change BoringSSL.podspec in the gRPC repo (link!!) this way:

instead of :tag => '...', use:
:commit => 'SHA you just copied'

Run gRPC ObjC tests
In the rare case that err_data.c needs to be updated, either:

copy the contents of /src/boringssl/err_data.c into the BoringSSL.podspec, or
regenerate err_data.c for your specific commit, following the instructions at the BoringSSL repo. You need to install Go.

Deploy new version of BoringSSL's pod.
So that the new release of gRPC uses the latest BoringSSL.

Ask Adam Langley<EMAIL_ADDRESS>(CC: Matt Braithwaite<EMAIL_ADDRESS>to please add a new tag for Cocoapods in the BoringSSL repo.
Example email:

Hi guys, we're ready for the next gRPC release.

I've tested it against BoringSSL commit 232127d (Fold EC_GROUP...), and would need a version_for_cocoapods_2.0 tag to be added.

Thanks a lot!

Update the podspec's version number & commit
pod trunk push BoringSSL.podspec --use-libraries --allow-warnings

Test gRPC.podspec

In your personal fork, create a branch and edit gRPC.podspec.template (link!!!!) to point to the new version number.
./tools/buildgen/generate_projects.sh
commit and send to the release branch

"pod spec lint gRPC.podspec" will fail now, because the podspec points to a tag that doesn't exist in the repo.
We don't want to add the tag until after testing, though, so:

Add a tag like this: git tag release-0_13_0-objectivec-0.13.0 and push it to your personal fork of the grpc repo on GitHub, pointing to the release branch
Temporarily modify gRPC.podspec to point to your personal fork, instead of github.com/grpc/grpc
pod spec lint gRPC.podspec --verbose --use-libraries
If linting fails, this is a way to debug:

enter "script" in your terminal, so the output of pod spec lint will be captured in a file
repeat the lint command with --fail-fast --no-clean, so you can open the generated projects that had the failures
enter "exit" to exit the subshell

After fixing any listing errors, run gRPC ObjC tests to make sure they pass (as local development might differ from what the linter checks).

Deploy gRPC.podspec

If any change was done to fix problems, commit and merge them into the release branch of the main repo.
Recreate the version tag at the latest commit in the release branch.
Push the version tag to the main grpc repo
pod trunk push gRPC.podspec --use-libraries --allow-warnings


[winpr,assert] eliminate c++ warnings

Eliminate cast warnings when used in C++ code

@hardening an approving review would be great, then we can merge this ;)


Missing text in files and unable to type text in new files

VSCode Version: Code - Insiders 1.19.0-insider (432b53bc3a263fda8db6ecca83d1ce89ede04bf2, 2017-11-06T06:22:36.757Z)
OS Version: Linux x64 4.13.0-16-generic
Extensions:

Extension
Author (truncated)
Version

vscode-markdownlint
Dav
0.11.0

clojure
avl
0.9.1

scala-lsp
dra
0.2.2

code-runner
for
0.8.3

Steps to Reproduce:

Reproduces without extensions: Yes/No

closing my recent issue #37728 In favor of this already established issue.

https://github.com/Microsoft/vscode/issues/37698


[SPARK-22473][TEST] Replace deprecated AsyncAssertions.Waiter and methods of java.sql.Date

What changes were proposed in this pull request?
In spark-sql module tests there are deprecations warnings caused by the usage of deprecated methods of java.sql.Date and the usage of the deprecated AsyncAssertions.Waiter class.
This PR replace the deprecated methods of java.sql.Date with non-deprecated ones (using Calendar where needed). It replaces also the deprecated org.scalatest.concurrent.AsyncAssertions.Waiter with org.scalatest.concurrent.Waiters._.
How was this patch tested?
existing UTs

Test build #83600 has started for PR 19696 at commit 3b45663.

Test build #83600 has finished for PR 19696 at commit 3b45663.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83600/
Test PASSed.

Merged to master


deps: bump karpenter-core to v0.32.5

Fixes #N/A
Description
Bumps karpenter-core to the latest version.
How was this change tested?
N/A (its the same commit, new tag)
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.355%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
6175

Relevant Lines:
7498

💛  - Coveralls


fix(tabbar): make Wave's theme apply to TabBar component

What
Make Wave's theme apply to TabBar component
Media
{/* When passing a margin to TabBar it doesn't apply it correctly */}
{/* mb={3} should equal 24px (Spaces[3]), it is 16px instead */}
<TabBar mb={3}>
  {['Tab One', 'Tab Two'].map(it => (
    <TabBar.Link
        key={it}
        href="#"
        onClick={e => {
            e.preventDefault();
            setSelected(it);
        }}
        selected={selected === it}
    >
        {it}
    </TabBar.Link>
  ))}
</TabBar>
{/* Here's a correct margin for comparison */}
<Box width='100%' mb={3} >The margin above is 16px, but should be 24px</Box>
<Box width='100%' >The margin above is 24px</Box>

Check it out in this CodeSandbox
Why
All Wave components should have Wave's theme applied to them

How

Pass our theme as a styled component attribute in TabBar

Hi @martimalek , I know this is already merged, just a little comment from what I noticed recently, and I see in your comment above TabBar that you noticed it too, maybe we should promote more to use the Spaces[value] that we have exported from Wave, rather than just numbers alone for margins, paddings, etc., because:

as we see, values of spacing returned from Spaces[value] and just using [value] itself, are not the same
I think the main point of having the Spaces components, was to let it manage all spacing, when using Wave

So I think it would make sense to also include using Spaces, when defining spacings of in our other components within the library
wdyt?


Rule for redundant curly braces too strict

This line gives the warning Redundant curly braces around a hash parameter.
index({ ssn: 1 }, { unique: true })
but the curly braces are not redundant here. It is obviously the correct way to define an index for a Mongoid model http://mongoid.org/en/mongoid/docs/indexing.html
If we do not use curly braces it does not work, because the two hashes merge into one:
index(ssn: 1, unique: true)

It's only the last argument that's being flagged.
 ~$ rubocop -v
0.32.1
 ~$ rubocop --auto-correct /tmp/2022.rb
Inspecting 1 file
C

Offenses:

/tmp/2022.rb:1:19: C: [Corrected] Redundant curly braces around a hash parameter.
index({ ssn: 1 }, { unique: true })
                  ^^^^^^^^^^^^^^^^

1 file inspected, 1 offense detected, 1 offense corrected
 ~$ cat /tmp/2022.rb
index({ ssn: 1 }, unique: true)

OK, so this was a false alarm.


[CSGO] Tmux resume delay while hibernating (fix)

I've noticed a long time ago that when you attach to any tmux sessions that are being used by csgo there will be a delay on input in that session. I figured out that this is caused by the default configuration files setting sv_hibernate_when_empty  to 1,  which may annoy other people who haven't figured this out yet. You can also modify how many milliseconds to sleep per frame by using sv_hibernate_ms, if users want to keep the server in hibernation if there are no active players, but still be able to interact with console normally they should set sv_hibernate_ms to 5. I think by default lgsm-default.cfg for csgo should include sv_hibernate_when_empty 0 for ease of use.

Hi,
Server hibernation by default is a good thing to keep in my opinion to save some CPU power in a lot of cases.
But, sv_hibernate_ms sounds like a good idea.
However "sv_hibernate_ms x # of milliseconds to sleep per frame while hibernating" > what the hell do they mean ? Does it hibernate more if you increase the value ? Is it useless under a certain value ? Is it freezing over a certain value ?

It looks like it reduces the amount of ticks per second that the server handles as you increase the latency. If you set sv_hibernate_when_empty to 1 and sv_hibernate_ms to 0 it seems like it uses the same amount of CPU as if you had sv_hibernate_when_empty set to 0. So you have to assign a value to sv_hibernate_ms in order to decrease the amount of ticks that are updated per second, which should in turn lower CPU usage but it's almost negligible how much it actually saves. It seems like even adding an entire 100ms between each tick update, the process uses only about 1% less of CPU usage compared to having sv_hibernate_when_empty set to 0.

OK, so that's what i thought.
Do you know what's the default sv_hibernate_ms then ? So that we can lower it accordingly.

The default sv_hibernate_ms is 20

And between sv_hibernate_ms 5 and sv_hibernate_ms 20, console still doesn't work ?

Having it set to sv_hibernate_ms 5 allows acceptable interaction with the console. Anything higher is extremely annoying.

Well
https://github.com/dgibbs64/linuxgsm/commit/83492ed57cf665f762162d9fb1c2994d1cdc7df9
Should do the trick then. :)

I'm noticing that csgo config is quite empty. Do you think about anything else that would be worth adding ?

It might also be worth adding host_info_show 2 to allow game-trackers to query the server and host_players_show 2 to show what players are playing on that server, as well as host_name_store 1 to show the host name in queries as well. Most community servers will usually want to have this ability by default. There are quite a few other things that could also be included, but it would require input from the user during installation.

Found some doc about it.
I'll add those with a comment.
https://www.gametracker.com/games/csgo/forum.php?thread=91691

That looks good, it's sad that official documentation from valve is usually either outdated or hidden somewhere on an obscure page and people have to spend alot of time to figure things out.

From what i know, there is no actual official documentation from valve. It's a wiki, so anyone can write it, as i did for Rust, because when i needed it, i found information in it to be quite poor.
https://developer.valvesoftware.com/wiki/Rust_Dedicated_Server


Closes: #1

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

The CLA has been signed.

6


[WIP] ServiceManager v2 + v3 compatibility

@weierophinney please review this PR, thanks!

@ezimuel Looks great!
Please add the .travis.yml changes as seen in the relevant zend-barcode PR so that we can validate that the code works properly with both v2 and v3 versions of the libraries going forward. After we get the results from that change, we'll know for certain if it's ready!

@ezimuel Why did you remove all PECL_INSTALL_* environment variables in .travis.yml`?

@ezimuel @weierophinney status ?

Merged with #68 / #69 (which were rebased).