Getting Input/output error when trying to access the volume-mount from an application pod

Description

Getting Input/output error when trying to access the volume mount across the PODS of a Deployment.
$ kubectl  exec nginx-deployment-695f8d5d5d-p7zbg -- ls -lh /etc/nginx/conf.d
total 0
ls: reading directory '/etc/nginx/conf.d': Input/output error
command terminated with exit code 2
$ kubectl  exec nginx-deployment-695f8d5d5d-dcfwb -- ls -lh /etc/nginx/conf.d
total 0
ls: reading directory '/etc/nginx/conf.d': Input/output error
command terminated with exit code 2
Application PODS of the Deployment

NAME                                READY   STATUS    RESTARTS   AGE
nginx-deployment-695f8d5d5d-nfd5l   1/1     Running   0          4s
nginx-deployment-695f8d5d5d-x7cmk   1/1     Running   0          64s
nginx-deployment-695f8d5d5d-zl6np   1/1     Running   0          3m4s

kubectl  get deploy
NAME               READY   UP-TO-DATE   AVAILABLE   AGE
nginx-deployment   3/3     3            3           81m

kubectl  get pvc
NAME             STATUS   VOLUME                                     CAPACITY   ACCESS MODES   STORAGECLASS   AGE
demo-cstor-vol   Bound    pvc-0680ae85-1a83-4d9e-80c9-1a205f8a28bd   10Gi       RWO            cstor-csi      103m

$ kubectl  get nodes
NAME          STATUS   ROLES    AGE    VERSION
portworx-m1   Ready    master   265d   v1.19.2
portworx-w1   Ready    <none>   265d   v1.19.2
portworx-w2   Ready    <none>   265d   v1.19.2
portworx-w3   Ready    <none>   265d   v1.19.2

$ kubectl  get pods -n openebs --show-labels |grep cspc-operator
cspc-operator-6d796d7b8f-nv4jx                                    1/1     Running   1          2d      name=cspc-operator,openebs.io/component-name=cspc-operator,openebs.io/version=2.9.0,pod-template-hash=6d796d7b8f

Expected Behavior

The PVC should not be mounted across all the PODS of the deployment and few PODS should go in a Containercreating state.

Current Behavior

When the deployment yaml is applied, it creates 3 replicas which is fine as the replica count is 3 in the deployment yaml. But the PVC actually gets mounted across all the PODS and we are able to view the contents of the volume mount. However, this is only possible/stays for few minutes and later start giving as Input/Output error as described in the Description section and expected as well.

$ kubectl  exec nginx-deployment-695f8d5d5d-nfd5l  -- ls -lh /etc/nginx/conf.d
total 20K
-rw-r--r-- 1 root root  20 Jun 17 09:24 file
drwx------ 2 root root 16K Jun 17 09:23 lost+found

$ kubectl  exec nginx-deployment-695f8d5d5d-x7cmk -- ls -lh /etc/nginx/conf.d
total 20K
-rw-r--r-- 1 root root  20 Jun 17 09:24 file
drwx------ 2 root root 16K Jun 17 09:23 lost+found

Your Environment

kubectl get nodes:

$ kubectl  get nodes
NAME          STATUS   ROLES    AGE    VERSION
portworx-m1   Ready    master   265d   v1.19.2
portworx-w1   Ready    <none>   265d   v1.19.2
portworx-w2   Ready    <none>   265d   v1.19.2
portworx-w3   Ready    <none>   265d   v1.19.2

kubectl get pods --all-namespaces:

$ kubectl  get pods --all-namespaces
NAMESPACE     NAME                                                              READY   STATUS             RESTARTS   AGE
default       nginx-deployment-695f8d5d5d-dcfwb                                 1/1     Running            0          124m
default       nginx-deployment-695f8d5d5d-p7zbg                                 1/1     Running            0          124m
default       nginx-deployment-695f8d5d5d-xpgtr                                 1/1     Running            0          124m
kube-system   autopilot-7c4d5f49d6-jwb48                                        1/1     Running            15         45d
kube-system   coredns-f9fd979d6-2wx4p                                           1/1     Running            6          45d
kube-system   coredns-f9fd979d6-mk655                                           1/1     Running            6          45d
kube-system   etcd-portworx-m1                                                  1/1     Running            20         45d
kube-system   kube-apiserver-portworx-m1                                        1/1     Running            20         45d
kube-system   kube-controller-manager-portworx-m1                               1/1     Running            20         45d
kube-system   kube-proxy-4v7fr                                                  1/1     Running            7          45d
kube-system   kube-proxy-6h7zv                                                  1/1     Running            6          45d
kube-system   kube-proxy-7xwsl                                                  1/1     Running            6          45d
kube-system   kube-proxy-nlmlv                                                  1/1     Running            2          45d
kube-system   kube-router-jqpvz                                                 1/1     Running            8          45d
kube-system   kube-router-mxlkt                                                 1/1     Running            7          45d
kube-system   kube-router-wlmfk                                                 1/1     Running            9          45d
kube-system   kube-router-xzj84                                                 1/1     Running            2          45d
kube-system   kube-scheduler-portworx-m1                                        1/1     Running            20         45d
kube-system   px-lighthouse-777fbbf998-bw2x2                                    3/3     Running   5891       45d
kube-system   stork-7cfcb88895-cxzzn                                            1/1     Running            12         45d
kube-system   stork-7cfcb88895-tqh9t                                            1/1     Running            13         45d
kube-system   stork-7cfcb88895-v9lmh                                            1/1     Running            11         45d
kube-system   stork-scheduler-96d9f4cff-ck4gv                                   1/1     Running            8          45d
kube-system   stork-scheduler-96d9f4cff-j9ks2                                   1/1     Running            6          45d
kube-system   stork-scheduler-96d9f4cff-zvgsw                                   1/1     Running            7          45d
openebs       cspc-operator-6d796d7b8f-nv4jx                                    1/1     Running            1          47h
openebs       cstor-storage-5fxr-679d895884-gxf2k                               3/3     Running            3          46h
openebs       cstor-storage-jskf-bbcd8fcb5-kp9hv                                3/3     Running            3          46h
openebs       cstor-storage-r57t-5d47b8677b-brx7j                               3/3     Running            6          46h
openebs       cvc-operator-849dff65c-cbg2k                                      1/1     Running            2          47h
openebs       openebs-cstor-admission-server-5d868c64d4-xmmjn                   1/1     Running            1          47h
openebs       openebs-cstor-csi-controller-0                                    6/6     Running            6          47h
openebs       openebs-cstor-csi-node-f688q                                      2/2     Running            4          47h
openebs       openebs-cstor-csi-node-gtg67                                      2/2     Running            2          47h
openebs       openebs-cstor-csi-node-qd5p4                                      2/2     Running            2          47h
openebs       openebs-ndm-4rb4f                                                 1/1     Running            2          47h
openebs       openebs-ndm-cjzbg                                                 1/1     Running            3          47h
openebs       openebs-ndm-kr449                                                 1/1     Running            1          47h
openebs       openebs-ndm-operator-59ffcfc949-tcfb9                             1/1     Running            2          47h
openebs       pvc-0680ae85-1a83-4d9e-80c9-1a205f8a28bd-target-69487c985dzppvr   3/3     Running            0          142m

kubectl get services -n openebs:

openebs       cvc-operator-service                       ClusterIP   <IP_ADDRESS>    <none>        5757/TCP                              47h
openebs       openebs-cstor-admission-server             ClusterIP   <IP_ADDRESS>   <none>        443/TCP                               47h
openebs       pvc-0680ae85-1a83-4d9e-80c9-1a205f8a28bd   ClusterIP   <IP_ADDRESS>   <none>        3260/TCP,7777/TCP,6060/TCP,9500/TCP   144m

kubectl get sc:

$ kubectl  get sc
NAME                PROVISIONER                     RECLAIMPOLICY   VOLUMEBINDINGMODE   ALLOWVOLUMEEXPANSION   AGE
cstor-csi           cstor.csi.openebs.io            Delete          Immediate           true                   46h
mayastor-iscsi      io.openebs.csi-mayastor         Delete          Immediate           false                  217d
mayastor-nvmf       io.openebs.csi-mayastor         Delete          Immediate           false                  217d
mysql-sc            kubernetes.io/portworx-volume   Delete          Immediate           false                  264d
px-ha-sc            kubernetes.io/portworx-volume   Delete          Immediate           false                  264d
stork-snapshot-sc   stork-snapshot                  Delete          Immediate           false                  265d
wordpress-sc        kubernetes.io/portworx-volume   Delete          Immediate           false                  264d

kubectl get pv:

$ kubectl  get pv
NAME                                       CAPACITY   ACCESS MODES   RECLAIM POLICY   STATUS   CLAIM                    STORAGECLASS   REASON   AGE
pvc-0680ae85-1a83-4d9e-80c9-1a205f8a28bd   10Gi       RWO            Delete           Bound    default/demo-cstor-vol   cstor-csi               146m

kubectl get pvc:

$ kubectl  get pvc
NAME             STATUS   VOLUME                                     CAPACITY   ACCESS MODES   STORAGECLASS   AGE
demo-cstor-vol   Bound    pvc-0680ae85-1a83-4d9e-80c9-1a205f8a28bd   10Gi       RWO            cstor-csi      147m

OS (from /etc/os-release):

$ cat /etc/os-release
NAME="Ubuntu"
VERSION="18.04.4 LTS (Bionic Beaver)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 18.04.4 LTS"
VERSION_ID="18.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=bionic
UBUNTU_CODENAME=bionic

Kernel (from uname -a):

$ uname -a
Linux portworx-m1 5.4.0-74-generic #83~18.04.1-Ubuntu SMP Tue May 11 16:01:00 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Install tools:

Others:
TargetPOD-cStor-istgt.txt
targetpod-maya-volume-exporter.txt
targetpod-cstor-volume-mgmt.txt

Yamls for SC, PVC,Deployment

$ cat sc.yaml
kind: StorageClass
apiVersion: storage.k8s.io/v1
metadata:
  name: cstor-csi
provisioner: cstor.csi.openebs.io
allowVolumeExpansion: true
parameters:
  cas-type: cstor
  # cstorPoolCluster should have the name of the CSPC
  cstorPoolCluster: cstor-storage
  # replicaCount should be <= no. of CSPI
  replicaCount: "3"

$ cat pvc.yaml
kind: PersistentVolumeClaim
apiVersion: v1
metadata:
  name: demo-cstor-vol
spec:
  storageClassName: cstor-csi
  accessModes:
    - ReadWriteOnce
  resources:
    requests:
      storage: 10Gi

cat nginx.yaml
apiVersion: apps/v1
kind: Deployment
metadata:
  name: nginx-deployment
spec:
  selector:
    matchLabels:
      app: nginx
  replicas: 3 # tells deployment to run 2 pods matching the template
  template:
    metadata:
      labels:
        app: nginx
    spec:
      containers:
      - name: nginx
        image: nginx:1.14.2
        ports:
        - containerPort: 80
        volumeMounts:
        - mountPath: /etc/nginx/conf.d
          name: volumenginx
      volumes:
        - name: volumenginx
          persistentVolumeClaim:
            claimName: demo-cstor-vol

when did the regression started? And, the testcase should be in E2E suite.
And, why the EIO error is coming? Is this the ext4 behavior?


Update for SDL v6.1

Use menu and choice set managers

The link to DarkSky is broken in the README

The Supporting Files folder in the Xcode directory is red.

Works well. I think the 1000 line SmartDeviceLinkService.m file should be broken into smaller, more readable files.

@NicoleYarroch fixed those weirdness things.
I don't disagree about breaking it apart...that'll just take a lot more time.

Should we add the Pod folder to the .gitignore and add instructions to the README on how to install the SDL library via CocoaPods?

🤷🏻♂ We could. I've just been leaving it as is for now.

Have you tested the app with Manticore? The primary graphic images are barely visible due to the image color being almost identical to the background color. It it possible to template the primary graphic image so it looks good when Manticore is updated to Core 5.0?


Update external go packages - abandoned

This PR contains the following updates:

Package
Type
Update
Change

github.com/prometheus/client_golang
require
minor
v1.8.0 -> v1.14.0

github.com/prometheus/common
require
minor
v0.15.0 -> v0.37.0

Configuration
📅 Schedule: Branch creation - "before 7am on Monday" (UTC), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Renovate will not automatically rebase this PR, because other commits have been found.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

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

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: go.sum
Command failed: go get -d -t ./...
go: downloading gopkg.in/alecthomas/kingpin.v2 v2.2.6
go: downloading github.com/alecthomas/template v0.0.0-20190718012654-fb15b899a751
go: downloading github.com/alecthomas/units v0.0.0-20201120081800-1786d5ef83d4
github.com/lunarway/snyk_exporter imports
	github.com/prometheus/common/log: cannot find module providing package github.com/prometheus/common/log

prometheus/common has removed their custom log package and moved to use the golang default. We should probably change the code to not use the log package from prometheus.

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

Autoclosing Skipped
This PR has been flagged for autoclosing. However, it is being skipped due to the branch being already modified. Please close/delete it manually or report a bug if you think this is in error.

The project is no longer maintained. Closing.


Correct repo information for internal-plugin-board

Also, npm still thinks @ciscospark/plugin-board exists still, and uses this broken link too

Thanks for the submission! We've actually got a separate set of changes the should merge sometime this week that addresses this in a more holistic fashion (all the modules have this issue at the moment).


Ability to run individual couchdb-harness tests

Annoying that you currently can't without directly invoking the couchdb-harness executable.

There is probably a way to do this by either modifying the scripts or modifying https://github.com/pouchdb/couchdb-harness itself.


can you update the readme ?

hello
can you update the readme with more details
as an example of file ( argements ) commends , etc ... ?

Hello
I have updated the readme. Bare in mind that main.cpp is only a dummy example.
If you believe anything needs any further clarification, please notify me and i ll update it.
Sorry for the late response and thank u for taking an interest into my work


Fixed get user playlists

See https://trello.com/c/npV2D4TE

@ld357 I found out we need the userId otherwise, it would always use our playlists for other users


Search: Improve swagger documentation

Since we generate most of the code, there are a lot of descriptions in swagger that need to be improved. They either aren't clear (e.g. almost all enum value description), or just wrong (e.g. PatternAnalyzer.Pattern description says it defaults to whitespace when, actually, it's non-whitespace, i.e. \w+).
This will positively improve all languages' SDKs and means we don't have to override all models (harder for some languages than others, but still a lot of manual work meant to be done by generators).
/cc @brjohnstmsft

We'll track indivudual items in Azure/azure-rest-api-specs, though I think at some point a top-down review would help to improve docs not only for REST APIs, but the client SDK doc comments as well.


Firefox Portable

Are there any plans to make Shadowfox Updater compatible with Firefox Portable?
Maybe by entering the profile path manually?
Download: https://portableapps.com/apps/internet/firefox_portable
I tried t2YU2m8l83's solution from https://github.com/overdodactyl/ShadowFox/issues/127 but it didn't work on my side. Had to copy the files myself into the chrome folder.

Creating an internal_UUIDs.txt file in the folder Shadowfox_customization and manually importing the extensionUUID and internalUUID for certain addons like uBlock and CookieAutoDelete also won't work. Is there something I'm missing or misinterpreting?

Are there any plans to make Shadowfox Updater compatible with Firefox Portable? Maybe by entering the profile path manually?

@SrKomodo would best be able to answer this question, but I am surprised the solution in #127 didn't work for you.  Are you sure the your files are located in the correct location?

Creating an internal_UUIDs.txt file in the folder Shadowfox_customization and manually importing the extensionUUID and internalUUID for certain addons like uBlock and CookieAutoDelete also won't work. Is there something I'm missing or misinterpreting?

The UUIDs need to be placed in the actual css.  When using the updater, the UUIDs are transferred from internal_UUIDs.txt into their respective locations within userContent.css, thus without the updater, you would have to do this part manually.  internal_UUIDs.txt is simply a way to store your UUIDs in one convenient location and not lose them between updates.
As an example, for uBlock Origin, the UUID would have to be placed in two locations:
https://github.com/overdodactyl/ShadowFox/blob/c0b767cbf13dcb77e0ea6ff0b501a25888013008/userContent.css#L2216-L2220
https://github.com/overdodactyl/ShadowFox/blob/c0b767cbf13dcb77e0ea6ff0b501a25888013008/userContent.css#L1608-L1612

The solution in #127 should work, if you dont have a profiles.ini file just create one and have it point to where the chrome folder should be.
As i said before, if for example your chrome folder is in Firefox/Profiles/foo/ then profiles.ini should look like this:
[Foo]
Path=Profiles/foo
IsRelative=1

and the file should be placed on Firefox/

Now I got it to work. Looks like I've put the profiles.ini file in the wrong directory to begin with. Thanks for the help!


ORACLE Incorrect DDL generated w/ auto incrementing

When creating DDL for Tables in Oracle containing columns with IDENTITY attributes, the DDL ist created like this:
CREATE TABLE XXX
(
  COLUMN_NAME_SK NUMBER(18,0) DEFAULT "SCHEMA"."ISEQ$$_380866".nextval
)

instead of:
CREATE TABLE XXX
(
  COLUMN_NAME_SK NUMBER(18,0) GENERATED BY DEFAULT AS IDENTITY MINVALUE 1 MAXVALUE<PHONE_NUMBER>999999999999999999 INCREMENT BY 1 START WITH 1 CACHE 20 NOORDER  NOCYCLE  NOT NULL ENABLE
)

Identity-Columns in Oracle are available since version 12c.
For any testing / questions / examples do not hesitate to contact me.
Thanks
Marcel

DBeaver uses Oracle functions to generate DDL (DBMS_METADATA.GET_DDL)
And it is a bit different from what you have posted.
Maybe you don't have permissions to call this system function?

You are completly right. The strange SQL is generated by DBEAVER, when DMBS_METADATA does not work / is not available. Using the Owner of that object returns the correct DDL. Same behaviour as in SQL-Developer.
So, just Close this issue.


Bigtable: add enhanced stub for bigtable table admin client

This will be used #3658 to add new callables that can't be autogenerated.

@kevinsi4508 please take a look

Removed all unused imports & whitespace

@pongad can you take a look when you have a moment?


please share me option about this

Describe the bug
A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

SQLTools Version [e.g. v0.17.x]
VS Code Version: [e.g 1.x.y]
OS: [e.g. Windows, Linux, Mac]
Driver:

[ ] PostgreSQL/Redshift
[ ] MySQL/MariaDB
[ ] MSSQL/Azure
[ ] SQLite
[ ] Other? Which...

Database version: [e.g. MySQL v5.6, PostgreSQL vX...]

Additional context
Add any other context about the problem here.

https://github.com/mtxr/vscode-sqltools/issues/1301#issuecomment-2045193154


separate New Contact Us page front end

Related Issue
-separate New Contact Us page front end
Closes: #239
Describe the changes you've made
Give a clear description what modifications you have made
Type of change
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Mention any unusual behaviour of your code (Write NA if not)
Any unusual behaviour of your code
Additional Info (optional)
Any additional information you want to give
Checklist:

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly whereever it was hard to understand.
[x] My changes generate no new warnings.
[ ] Any dependent changes have been merged and published in downstream modules.

@ankitapuri please merge this

These are some unnecessary comments which needs to be deleted

These are some unnecessary comments which needs to be deleted

@ankitapuri that will be automatically removed when render with django that's not a problem

@ankitapuri I had commented out that csrf token and removed that message.txt 4kb that was actually unnecessary

Great @arpit456jain 🎉


Lazy loading with forFeature

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [X] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

export class NgrxDataModule {
  static forRoot(config: NgrxDataModuleConfig): ModuleWithProviders {
    return {
      ngModule: NgrxDataModule,
      providers: [
        EntityEffects,
        {
          provide: ENTITY_METADATA_TOKEN,
          multi: true,
          useValue: config.entityMetadata ? config.entityMetadata : []
        },
        {
          provide: ENTITY_COLLECTION_META_REDUCERS,
          useValue: config.entityCollectionMetaReducers
            ? config.entityCollectionMetaReducers
            : []
        },
        {
          provide: PLURAL_NAMES_TOKEN,
          multi: true,
          useValue: config.pluralNames
        }
      ]
    };
  }

Any log messages given by the failure

Expected/desired behavior

forFeature option exists to enable use of ngrx in a lazy-loaded store configuration, i.e.,
FeatureModule
imports: [
 ...,
 NgrxDataModule.forFeature({
 entityMetadata: entityMetadata
 })
]

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)

Versions

Mention any other details that might be useful
Looks like the forFeature branch is on the roadmap, but not yet available per ngrx-data.module.ts
Did I read that correctly and if so, do you have a sense of the priority level?
Love the idea, but it seems incompatible with a larger app/ nx/ xplat enterprise library approach if it depends on a singular store for the app (which means I won't be able to use it).  Let me know if I can help if it just needs some hands...I know your current schedule's pretty packed

Thanks! We'll be in touch soon.

It's on the roadmap. Should be simple because there are no extra services to provide. It's a matter of creating an options argument to .forFeature that registers additional metadata, etc.
You can do that easily today by injecting registry services (such as the EntityDefinitionService) and registering the feature pieces (such as the EntityMetadata for your feature).

Is there any plan to build this into a library? Thank you.


Front End Updates to the Homepage

Commits

the Hero component from tailwindcss has been added to the homepage
logos found on all current pages are functional and all lead back to the homepage
links in the navigation are all functional and go to it's designated routes

Current Bug:

the mobile menu is not closing / hiding as it should

Solution

partnering with a tutor on Slack to tackle the issue

Did a pull request without secondary approval so that I can share with Slack tutor for assistance in resolving bug.


int to signed byte

min(127, max(-127, int(self.x * speed))) & 255

the lower boundary should be -128.
min(127, max(-128, int(self.x * speed))) & 255

Just realized this was your code - noticed it at https://github.com/quangthanh010290/keyboard_mouse_emulate_on_raspberry and commented there.
You wouldn't have any ideas what would be needed to send absolute coordinates. Can that be done while emulating a mouse or would you need to emulate some other device class?

Yes, the full range of a signed byte is [-128;127] however the HID descriptor for the mouse input has the coordinate range of [-127;127]. I followed some example HID descriptors from the devices I have in this regard. I guess that the reason for using symmetrical values is to have the max mouse speed consistent in both directions.
As for the absolute movement it is indeed not possible with a HID mouse interface. Trackpads typically send absolute values but this would not help, since the OS would still translate those into relative motion of the mouse pointer. My bet is that the best results you could get via HID emulation is to use touch screen input. However there are limitations there as well. The OS will potentially treat touches differently than mouse clicks, it is not possible to emulate right or middle mouse button, the mouse pointer will not move, and some OSes will not support external HID touch screen at all (likely iOS, Android and Mac OS X).

thanks for the clarification. I noticed the -127/127 now when you mentioned it.
I got somewhat ok results scripting mouse movement on IOS using relative motion only. (once i found the equivalent to acceleration setting and turned it off ). I do however notice that some motions is missed. I think i might send to fast. or out of sync or something. Adding a 0.06 delay gives ~98% accuracy. 0.1 gives 100% - but that's a bit slow.
Do you think i should try to only send data in fixed intervals. Que them and send every 0.01-0.02 sec?
or maybe wait for a report request from the host?
Maybe i should try increasing the min/max to also, to allow bigger cross screen jumps in one report.


How to configure/customise  Kubectl-api --enable-admission-plugins

HI,
Can you please help me how to customise kubeapi-server --enable-admission-plugins. By default few admission plugins are set to  deny.

At the moment, the kube-apiserver job does not offer any mechanism for fine-grained control over the admission controls. The only things that are currently optional are SecurityContextDeny (through allow_privileged) and DenyEscalatingExec (through deny_escalating_exec).
I've been considering opening these up to be configured on a per-controller basis (with our manifest applying the current defaults), much the same way we do with feature gates. Hopefully that or some other mechanism will give you the flexibility you want.

It is possible to do with 0.24


Update AdvancedPayroll.md

Docs Build status updates of commit fd9ec96:
:warning: Validation status: warnings

File
Status
Preview URL
Details

dynamics-gp/payroll/AdvancedPayroll.md
:warning:Warning
View
Details

dynamics-gp/payroll/AdvancedPayroll.md

Line 362: [Warning] Invalid file link:(~/payroll/media/APPAYCOS.jpg).
Line 649: [Warning] Invalid file link:(~/payroll/media/APLABA.jpg).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Report ScrollNotification depth

ScrollNotifications now reports the number of scrollable widgets that have already received the notification. The overscroll indicator now only responds if it's the first.
Fixes #4167

But please add a test.


Provide wrapper around glib-compile-resources

It can be useful to embed resources using glib-compile-resources in build.rs, then using include_bytes! to include it in the executable. (Equivalent to using glib-compile-resources --generate-source in C).
This requires a certain amount of boilerplate, so it could be helpful for gtk-rs to provide a wrapper for it, unless there's already something I'm unaware of.

As a bunch of people already asked on IRC on how to do that, having a helper along with an example would be nice.

As a bunch of people already asked on IRC on how to do that, having a helper along with an example would be nice.

It should produce a cargo:rerun-if-changed= line for every file listed in the .gresource.xml file. Though that's a bit awkward since it would require an xml parser.

It should produce a cargo:rerun-if-changed= line for every file listed in the .gresource.xml file. Though that's a bit awkward since it would require an xml parser.

Well, if you really want that kind of integration a build system like meson on top of cargo is more suitable I guess.

Well, if you really want that kind of integration a build system like meson on top of cargo is more suitable I guess.

Nevermind depending on an xml parser; glib-compile-resources --generate-dependencies outputs a newline separated list of dependencies.

Nevermind depending on an xml parser; glib-compile-resources --generate-dependencies outputs a newline separated list of dependencies.


Autosummary

Kicking the tires on #271 from a branch that has OpenEye license and has #281 merged in

Okay, things are mostly building fine and the logic in the template also ignores the three new foo_tests "modules." But the new openff.interchange.testing module isn't included, probably since I don't know how these files were generated. Here I'd expect the .testing module to show up:


fix: clean openshift config map

Add optional to:
- LAUNCHER_BACKEND_URL
- LAUNCHER_KEYCLOAK_CLIENT_ID
- LAUNCHER_TRACKER_SEGMENT_TOKEN
- LAUNCHER_SENTRY_DSN

Don't forget to sync with the template in the Openshift templates repository, there is a script that does that


No backend folder and AxiosError: Network Error

After seemingly running the setup successfully and not getting any errors, I still don't have a "backend" folder as seen in the demonstration videos.
Furthermore, probably as a result, when opening the run file, I get the AxiosError: Network Error.
Do I need to run the setup as admin? Because when I do, I then get this error:
ERROR: Could not open requirements file: [Errno 2] No such file or directory: 'requirements.txt'
I looked all former issues and tried to implement their solutions but still couldn't manage to solve my problem.
Please bear with me, as i am kind of a layman when it comes to CS.

Hi,
No worries we'll have a look together.
Regarding the axios error, it is normal it is failing if the backend is not started!
From what I understand your zip might be incomplete since you have neither the requirements.txt and the backend folder.
Can you redownload the zip folder, extract in a new folder, and screenshot the result please ?

So I did as you said and redownloaded and extracted the regular windows release into a folder.

It looks 1:1 as it did before with no backend folder but with the requirements.txt. That is why the error:
ERROR: Could not open requirements file: [Errno 2] No such file or directory: 'requirements.txt'
was weird when the file was in fact in the folder.

So i found the solution and it is actually kind of funny.
The solution was to install Microsoft Visual Studio. lol


As a user, I want to use the metriq platform for domains in quantum information outside of benchmarking, so that I can use the metriq structure for other problems

Possibly related: https://github.com/unitaryfund/metriq-app/issues/186

I recognize this issue as the statement of a user story, from very early in the development process, but, by now, I think Metriq has demonstrated its flexibility in accommodating data besides "results." I think that's sufficient, to consider this user story to be addressed.


File input

For cases where my dataset is not available online and I don’t want to upload it, it would be awesome to have https://observablehq.com/@mbostock/file-input, ideally with at least text, JSON, and CSV variants. It would also be great to have an option to enable or disable the URL input.

A newer version, which I’d like to expose as Inputs.file: https://observablehq.com/@mbostock/localfile


add validate max stake amount

@huyp26102000 need add fallback for NaN case.
maybe as similar as below
const userCurrentStake = stakeInfo?.stakedValue / 10 ** 12 || 0;


Fixed parsing error in metaData.Duration

ffmpeg doesn't respect the culture of the client. Therefor parsing the duration of for example 00:05.31 on a german machine resulted in 531 seconds instead of 5 seconds and 310 ms. Fixed it by using InvariantCulture where the decimal point is always a point and not a comma.

thank you! i will look into all issues this week and merge all viable pullrequests :smiley:


Casting method group to delegate fails when instance is PSObject

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
For every object, this is true:
$InputObject.ToString -as [Func[String]]

But if I put that code in a function:
filter ConvertTo-String {
    [CmdletBinding()]
    param([Parameter(ValueFromPipeline)]$InputObject)
    if ($InputObject.ToString -as [Func[String]]) {
        $PSStyle.Foreground.Yellow + $InputObject.ToString()
    }
}

It's literally never true.
I had to add this:
    elseif ($InputObject.psobject.BaseObject.ToString -as [Func[String]]) {
        $PSStyle.Foreground.Cyan + $InputObject.ToString()
    }

The weird thing is that $InputObject.ToString.GetType() -eq $InputObject.psobject.BaseObject.ToString.GetType() so I don't know why one is castable and the other is not.
For what it's worth, my original goal was to write some "duck typing" where I check for "ToPsMetadata" and "FromPsMetadata" and assume the object implements IPSMetadataSerializable:
public interface IPsMetadataSerializable {
    string ToPsMetadata();
    void FromPsMetadata(string Metadata);
}

Expected behavior
`$Object.ToString -as [Func[String]]` should always return the delegate

Actual behavior
If the `$Object` is a PSObject, the PSMethodInfo can't be cast to the delegate

Error details
No response
Environment data
PSVersion                      7.2.4
PSEdition                      Core
GitCommitId                    7.2.4
OS                             Microsoft Windows 10.0.22621
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

Basically when PSGetMemberBinder calls PSGetMemberBinder.CloneMemberInfo it's not unwrapping psobjects. So the instance type "doesn't match" when creating the delegate with MethodInfo.CreateDelegate
Binder disassembly for when wrapped in a pso:

Binder disassembly for when not wrapped in a pso:

Notably, the former does not unwrap self (what I named the instance var) and just passes it as is. It should either unwrap, or take the instance field from the source object.


build(deps): bump controller-runtime to v0.14.1

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:

Project is to be archived - closing all issues and PRs
see https://github.com/kubernetes/org/issues/4122 and https://groups.google.com/d/msgid/kubernetes-sig-multicluster/9f8d81d1-07d1-4985-a7bf-d76197deb971n%40googlegroups.com
for details
/close


Documentation to include delete operation examples

Note: for reported bugs, please fill in the following details. bug reports without detailed steps on how to reproduce will be automatically closed.

Dragonboat version
v.3.3.4
Expected behavior
The code examples and/or go doc should describe the details to properly remove entries from Raft logs and the underlying persistence stores.
Actual behavior
Example code and go doc demonstrate the addition of entries but it is unclear on how to perform deletions.
Steps to reproduce the behavior
Request is relating to documentation, specifically for the OnDiskStateMachine.

@brianburnett In your email, you mentioned that

However, for my purposes I need to be able to support ephemeral keys and expire entries after a period of time.

My understanding is you want some of your data entries in OnDiskStateMachine to be expired after a period of time.
Raft log is an append only log of all operations you proposed for your OnDiskStateMachine, it can be compacted meaning you can request to have no longer required log entries (those already applied into your OnDiskStateMachine) to be removed, but there is no way to remove certain selected Raft log entries.
To support the feature you mentioned, you need to make a proposal to request your OnDiskStateMachine to ignore and remove all entries that carry certain timestamp. Your OnDiskStateMachine code need to implement such logic.
Please have a look at the section 2 of the Raft paper, it explains how Raft logs and OnDiskStateMachine work together as the Replicated State Machine.

Yes, I needed to better understand how to manage the removal of entries before opening up a ticket on implementing a mechanism to expire them. That feature would be similar to other Raft implementations such as RedisRaft's Expire or etcd's Lease.
For this issue I was hoping to see if it was possible to add some sample code that follows your existing examples.
Such as with your ondisk example code.  If the commands (GET, PUT) could be extended to include DELETE, what would be the proper way to call nh.SyncPropose() on an existing key?
                                if rt == PUT {
					kv := &KVData{
						Key: key,
						Val: val,
					}
					data, err := json.Marshal(kv)
					if err != nil {
						panic(err)
					}
					_, err = nh.SyncPropose(ctx, cs, data)
					if err != nil {
						fmt.Fprintf(os.Stderr, "SyncPropose returned error %v\n", err)
					}
				} else if rt == DELETE {
                                       // key exists from prior PUT
                                       ...
                                        _, err = nh.SyncPropose(ctx, cs, ???)
                                       ...
                                } else {
					result, err := nh.SyncRead(ctx, exampleClusterID, []byte(key))
					if err != nil {
						fmt.Fprintf(os.Stderr, "SyncRead returned error %v\n", err)
					} else {
						fmt.Fprintf(os.Stdout, "query key: %s, result: %s\n", key, result)
					}
				}```

Thanks!

For the replication state machine (RSM) model, PUT and DELETE are nothing different from each other, they both require changing the state of the (RSM) and thus they should be implemented as proposals.
I'd like to keep the exchanges as simple as possible, maybe I can add some comments in the example to indicate that PUT and DELETE should both be implemented as proposals.
Please feel free to open a new issue for the auto expiring feature.


QLoRA + FSDP

Is it possible to run QLoRA finetuning on more than a single device?  I don't see any configs for QLoRA other than for single_device.
If not, what are the gating issues?  More generally, what methods need to be implemented for custom tensor types (e.g., NF4) in order to compose with FSDP or other distributed training methods?

Hi @jeromeku thanks for creating the issue. We don't yet support QLoRA on multiple devices, but this is a high priority and we're hoping to have this working very soon! There is active work going on to enable this, you can see the PR #909.

Closing it since #909 was merged. You will need pytorch nightlies to run it. Config here: https://github.com/pytorch/torchtune/blob/6f37d15b2c99d49ca926173455569aa6f8e24d9d/recipes/configs/llama3/70B_full.yaml#L9
Please feel free to reopen the issue if you still have questions. Thanks! :)


au TerminalOpen * call setbufvar(...) does not work

Describe the bug
As described in documentation :h options-in-terminal:
au TerminalOpen * call setbufvar(+expand('<abuf>'), '&colorcolumn', 123)
should be run, but it does not get executed when switching to terminal buffer later.
Both setbufvar(...) and setwinvar(...) don't work.
Or au TerminalOpen does not work.
Tested in both terminal & GTK3 GUI.
To Reproduce
Detailed steps to reproduce the behavior:

Run vim --clean or gvim --clean
:au TerminalOpen * call setbufvar(+expand('<abuf>'), '&colorcolumn', 123)
:terminal ++hidden
:bnext
^W N
:set colorcolumn?
Observe setting is not set.

Expected behavior
Settings are set and autocommands are run.
Environment (please complete the following information):

Vim version 8.2.2489
OS: Manjaro KDE
Terminal: Konsole and GTK3 GUI.

The example is wrong, and it's my fault, sorry.  'colorcolumn' is a window-local option; it doesn't make sense to set it with setbufvar().  But at the time, I had difficulties finding a good example of a useful buffer-local option for a terminal buffer.  I think I found one: 'termwinscroll'.  So, this patch would be improve the example:
diff --git a/runtime/doc/terminal.txt b/runtime/doc/terminal.txt
index 5885cb197..87cbe6575 100644
--- a/runtime/doc/terminal.txt
+++ b/runtime/doc/terminal.txt
@@ -130,7 +130,7 @@ options specifically for the terminal window and buffer.  Example: >
 There is also the |TerminalOpen| event, but this may be triggered for a hidden
 terminal, and the current window and buffer may not be for the new terminal.
 You need to use <abuf>, which is set to the terminal buffer.  Example: >
-    au TerminalOpen * call setbufvar(+expand('<abuf>'), '&colorcolumn', 123)
+    au TerminalOpen * call setbufvar(+expand('<abuf>'), '&termwinscroll', 1234)

 Mouse events (click and drag) are passed to the terminal.  Mouse move events
 are only passed when Vim itself is receiving them.  For a terminal that is

Now, regarding your issue, you can't use setbufvar(), because 'colorcolumn' is not a buffer-local option.  You should use setwinvar() (or maybe settabwinvar()); but you can't, because at the time TerminalOpen is fired, your hidden terminal buffer has no window yet.  You need to delay the setting until the window has been created.  This should work:
au TerminalOpen * exe printf('au BufWinEnter <buffer=%d> ++once setl cc=%d', +expand('<abuf>'), 123)
terminal ++hidden

Note that the + prefix trick to coerce a string into a number only works in Vim script legacy.  In Vim9, you need str2nr():
vim9
au TerminalOpen * exe printf('au BufWinEnter <buffer=%d> ++once setl cc=%d', expand('<abuf>')->str2nr(), 123)
terminal ++hidden

Maybe we should include 2 examples; one for a buffer-local option, and another one for a window-local option:
diff --git a/runtime/doc/terminal.txt b/runtime/doc/terminal.txt
index 5885cb197..0d0825009 100644
--- a/runtime/doc/terminal.txt
+++ b/runtime/doc/terminal.txt
@@ -130,7 +130,14 @@ options specifically for the terminal window and buffer.  Example: >
 There is also the |TerminalOpen| event, but this may be triggered for a hidden
 terminal, and the current window and buffer may not be for the new terminal.
 You need to use <abuf>, which is set to the terminal buffer.  Example: >
-    au TerminalOpen * call setbufvar(+expand('<abuf>'), '&colorcolumn', 123)
+    au TerminalOpen * call setbufvar(+expand('<abuf>'), '&termwinscroll', 1234)
+For a window-local option, you need to delay the setting until the terminal
+window has been created: >
+    au TerminalOpen * exe printf(
+        \    'au BufWinEnter <buffer=%d> ++once '
+        \ .. 'setlocal colorcolumn=%d',
+        \ expand('<abuf>')->str2nr(), 123
+        \ )

 Mouse events (click and drag) are passed to the terminal.  Mouse move events
 are only passed when Vim itself is receiving them.  For a terminal that is

This should work:

You can check that it works with this shell command:
vim -Nu NONE -S <(cat <<'EOF'
    au TerminalOpen * exe printf('au BufWinEnter <buffer=%d> ++once setl cc=%d', +expand('<abuf>'), 123)
    terminal ++hidden
    sil bn
    call feedkeys("\<c-w>N", 'nxt')
    setl cc?
EOF
)

Maybe we should include 2 examples

Let's be consistent, if we use str2nr() in the second example, we should also use it in the first one:
diff --git a/runtime/doc/terminal.txt b/runtime/doc/terminal.txt
index 5885cb197..105917767 100644
--- a/runtime/doc/terminal.txt
+++ b/runtime/doc/terminal.txt
@@ -130,7 +130,14 @@ options specifically for the terminal window and buffer.  Example: >
 There is also the |TerminalOpen| event, but this may be triggered for a hidden
 terminal, and the current window and buffer may not be for the new terminal.
 You need to use <abuf>, which is set to the terminal buffer.  Example: >
-    au TerminalOpen * call setbufvar(+expand('<abuf>'), '&colorcolumn', 123)
+    au TerminalOpen * call setbufvar(expand('<abuf>')->str2nr(), '&termwinscroll', 1234)
+For a window-local option, you need to delay the setting until the terminal
+window has been created: >
+    au TerminalOpen * exe printf(
+        \    'au BufWinEnter <buffer=%d> ++once '
+        \ .. 'setlocal colorcolumn=%d',
+        \ expand('<abuf>')->str2nr(), 123
+        \ )

 Mouse events (click and drag) are passed to the terminal.  Mouse move events
 are only passed when Vim itself is receiving them.  For a terminal that is

Looks like this was a documentation problem.  I'll include the second example, that will fix it.
The example is a bit complicated, but will be helpful for someone who wants to do this.

@lacygoill yes, this works, thank you.

However, then this autocmd does not run when starting terminal without ++hidden, but according to documentation, it should run.
This can be checked with:
vim -Nu NONE -S <(cat <<'EOF'
    au TerminalOpen * exe printf('au BufWinEnter <buffer=%d> ++once setl cc=%d', +expand('<abuf>'), 123)
    terminal
    call feedkeys("\<c-w>N", 'nxt')
    setl cc?
EOF
)

However, then this autocmd does not run when starting terminal without ++hidden, but according to documentation, it should run.
This can be checked with:
vim -Nu NONE -S <(cat <<'EOF'
    au TerminalOpen * exe printf('au BufWinEnter <buffer=%d> ++once setl cc=%d', +expand('<abuf>'), 123)
    terminal
    call feedkeys("\<c-w>N", 'nxt')
    setl cc?
EOF
)

@brammool  @lacygoill  I've created related issue #7816 for this.


add jemalloc

https://github.com/jemalloc/jemalloc-cmake
set(jemalloc_URL https://github.com/jemalloc/jemalloc-cmake/archive/jemalloc-cmake.4.3.1.tar.gz)
set(jemalloc_HASH SHA256=f9be9a05fe906deb5c1c8ca818071a7d2e27d66fd87f5ba9a7bf3750bcedeaf0)

https://github.com/tensorflow/tensorflow/blob/7f84d88d39f236e5c0cea492a2248782e696c972/tensorflow/contrib/cmake/external/jemalloc.cmake#L18-L19

Please create a hunter-packages/jemalloc-cmake fork.  I will try to use the upstream release as is first.

Done: https://github.com/hunter-packages/jemalloc-cmake

Please create hunter-packages/jemalloc
Background
The jemalloc-cmake fork seems to be inactive and more or less windows only.  Based on discussions in [jemalloc issues](https://github.com/jemalloc/jemalloc/issues/303#issuecomment-343670287) it seems that the long term plan is to switch to CMake, and there are more recent CMake branches in the jemalloc repository than what is currently in jemalloc-cmake.  I *think* the best approach for this effort is to create a jemalloc fork to hold any required changes.

See: https://github.com/jemalloc/jemalloc/issues/303#issuecomment-343670287

Done: https://github.com/hunter-packages/jemalloc

PR: https://github.com/ruslo/hunter/pull/1187

CMake is a WIP.  There is an older jemalloc-cmake branch which only builds on MSVC, and there are more recent CMake forks of jemalloc that don't build on MSVC.  We could use the current default/stable autoconf scheme for compatible platforms and use jemalloc-cmake for MSVC, but it is probably better to wait until CMake is adopted officially (this is in the works).  This package is optional for tensorflow, so I'm going to close this issue for now.

This package is optional for tensorflow, so I'm going to close this issue for now.

I guess notes will be useful if anybody will try to add it again, reopen.


BeHe writes right-to-left in some editors!

Cannot understan why, but using BeHe keyboard with Jota Text Editor (v. 0.2.38 from Google Play) I get the text written from right to left. The inserting point (cursor) is correctly on the left side of the screen, but every letter I type get moved toward the right side.
The problem does not exists with other editors or other context.
Using BeHe Keyboard 1.0.5 from F-Droid, into an Android 6.0 chinese phone.

Strange enough though. I get the same effect in Jota, but can't explain why. I never encountered such a problem before. I'm currently working on making the keyboard height adjustable, but I'll take a look into this issue too. Thanks for reporting!


Connector + ConnectButton

https://github.com/ant-design/ant-design-web3/discussions/60#discussioncomment-7516828

[ ] #87
[ ] Connector logic
[ ] support select chains
[ ] support user profile
[ ] support menu
[ ] support tooltip

我领取一下
support menu
support tooltip
这两个吧，可能要周末才能开搞


APL 2001 labs have been updated.

Note
Replace issue name LAB00:QUICK_DESCRIPTION, for example "LAB01: My new issue" (or same name as linked Issue)

Related Issue
Link related Github Issue 🢂 Fixes # . (Include issue number after #)
Checklist
Mark completed with "x" between brackets, "[x]", or checking the box once the PR is created:

[ ] Has related GitHub Issue 💥 Create Issue 📝
[ ] Tested it
[ ] Read the PR collaboration guide 👓 Collaboration Guide

Changes proposed in this pull request:

[heart]         Abdullah Samim (Axianseu) reacted to your message:

From: Luiz Macedo @.>
Sent: Friday, October 13, 2023 6:55:57 PM
To: MicrosoftLearning/implement-security-through-pipeline-using-devops @.>
Cc: Abdullah Samim (Axianseu) @.>; Mention @.>
Subject: Re: [MicrosoftLearning/implement-security-through-pipeline-using-devops] APL 2001 labs have been updated. (PR #20)
@LuizMacedo approved this pull request.
Thank you so much @v-asamimhttps://github.com/v-asamim! Great suggestions!
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftLearning/implement-security-through-pipeline-using-devops/pull/20#pullrequestreview-1677162991, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A2WJCUHBVI5L5YW32SLV53LX7GFD3AVCNFSM6AAAAAA5YZOGTKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTMNZXGE3DEOJZGE.
You are receiving this because you were mentioned.Message ID: @.***>


matrixzulipbridge: init at 0.4.1

Description of changes
Fixes #337806
Closes #285702.

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] 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

[ ] Fits CONTRIBUTING.md.

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

Hi, welcome, and thank you for contributing! Since this seems to be your first derivation in the repo, I've added the "First Time Contribution" label so reviewers will know to give you extra help.
Stuff you should do:

Run an interactive rebase on your branch and "squash" the commits together. When done, use git push --force-with-leaseto update. See CONTRIBUTING.md
Make sure you're keeping up with the master branch. The CONTRIBUTING link above will tell you how.

Nice to have:

As someone who has used fetchPyPi in derivations here, it's generally considered better to fetchFromGitHub and build from source. Source builds will run unit tests while PyPi packages often exclude the tests. Here's an example: langchain-aws

And expect many little change requests from reviewers this first time out.

P.S. When there's a packaging request, please include a comment (or edit the description) to include it. In this case:
Closes #337806

Note to reviewers: He already has an entry in MAINTAINERS.

While you're here, please squash your commits (@natsukium will ask you to do the same thing). The standard is one complete commit per update. Run an interactive rebase on your branch and "squash" the commits together. When done, use git push --force-with-leaseto update. See CONTRIBUTING.md

@rrcomtech Just checking in: how are you doing?

Thank you for checking in :) I will work tomorrow on this again, was a busy week 😄

Result of nixpkgs-review pr 337953 run on aarch64-darwin 1

4 packages built:

python312Packages.matrix-zulip-bridge
python312Packages.matrix-zulip-bridge.dist
python312Packages.zulip-emoji-mapping
python312Packages.zulip-emoji-mapping.dist

Result of nixpkgs-review pr 337953 run on x86_64-linux 1

4 packages built:

python312Packages.matrix-zulip-bridge
python312Packages.matrix-zulip-bridge.dist
python312Packages.zulip-emoji-mapping
python312Packages.zulip-emoji-mapping.dist

Result of nixpkgs-review pr 337953 run on x86_64-darwin 1

4 packages built:

python312Packages.matrix-zulip-bridge
python312Packages.matrix-zulip-bridge.dist
python312Packages.zulip-emoji-mapping
python312Packages.zulip-emoji-mapping.dist

It's ready to be merged! Thank you for hanging in there.
Here's what happens next:

ofborg will label it with approvals: 1
@natsukium may choose to review it as well, but is really busy right now
One of the committers doing merges should come along and merge this fairly soon

If it doesn't merge within a few days, please ping me and I'll see if we can get one of the committers on it.

Thank you Sarah :D

Thank you Sarah :D

You're welcome! Please incorporate the review changes from @Aleksanaa (and ping me if you need help figuring something out)

Result of nixpkgs-review pr 337953 run on x86_64-linux 1

4 packages built:

matrix-zulip-bridge
matrix-zulip-bridge.dist
zulip-emoji-mapping
zulip-emoji-mapping.dist

Result of nixpkgs-review pr 337953 run on aarch64-darwin 1

4 packages built:

matrix-zulip-bridge
matrix-zulip-bridge.dist
zulip-emoji-mapping
zulip-emoji-mapping.dist

Result of nixpkgs-review pr 337953 run on aarch64-darwin 1

4 packages built:

matrix-zulip-bridge
matrix-zulip-bridge.dist
zulip-emoji-mapping
zulip-emoji-mapping.dist

Result of nixpkgs-review pr 337953 run on aarch64-darwin 1

4 packages built:

matrix-zulip-bridge
matrix-zulip-bridge.dist
python312Packages.zulip-emoji-mapping
python312Packages.zulip-emoji-mapping.dist

Result of nixpkgs-review pr 337953 run on x86_64-linux 1

4 packages built:

matrix-zulip-bridge
matrix-zulip-bridge.dist
python312Packages.zulip-emoji-mapping
python312Packages.zulip-emoji-mapping.dist


Unpredictable lighting in customizer

When you go to the customiser at /tezos/claim-bot - bot sometimes seems greyish, sometimes white-ish. I'm sure this has something to do with environment lighting.

@harsh242 or @bhaskarSingh please take a look.

Yup, high chance that it could be due to lighting, because from what I recall initial colors are set to #ffffff 🤔

@manangouhari is this fixed? (It likely is, confirming)

@manangouhari is this fixed? (It likely is, confirming)

It's not fixed yet.

The reason why sometimes, we are facing with the greyish bot, is because we are using environmental lighting and the file size is too big i.e. 1.58 MB & may not be able to load sometimes
Possible solutions:

Remove the environmental lighting and use our own lighting setup
Compress the hdri file : File Available here

@aayushsaini1 what do you suggest?
Cases:

With
<Environment files="royal_esplanade_1k.hdr" />

With environment bg set to true.
<Environment background={true} files="royal_esplanade_1k.hdr" />

Without Environment

Using HDRi w/o showing it is better as the above HDRi background is irrelevant.
But if we can compress our current HDRi, (which will ruin it's quality I guess), then we may be able to use it with visible background.

Can you try compressing the file to the lowest? I have mentioned the file in the above comment.

file size is around 500kb.
royal_esplanade_1k_compressed_50ppi.zip
Note: Compressing the HDRi also affects the lighting.

Old File :

Old File :

New File


"show dialog" functions should allow setting an AnimationStyle

PopupMenuButton allows setting an AnimationStyle to specify its duration & curve in either direction.
It would be great if showAdaptiveDialog() and similar functions also supported this type of customization.

Looks like @nploi isn't showing up as an assignee suggestion… will adding an @ comment here do it?

@nploi maybe you have to comment in this issue in order to show up under "assignees"…

yeah, i will support with this ticket. @nate-thegrate


Environment variables in watch path

Hello, I'm trying to use environment variables in watch path, but they are not recognized.
From the logs:
2024-11-16 22:23:26 INFO The path '[env:USERPROFILE]\Downloads' does not exists, so the watch was not created.
2024-11-16 22:24:16 INFO The path '%USERPROFILE%\Downloads' does not exists, so the watch was not created.

Are environment variables in watch path supported?

The watch path does not support environment variables.
It could be something I can look at implementing, though.


Fix two compilation issues on master-next

This doesn't fix all breakages on master-next, but it gets us closer.

@swift-ci Please test

⛵


⚠ Bofrost.nl has degraded performance

In c639de4, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1285 ms

Resolved: Bofrost.nl performance has improved in 8757001 after 6 minutes.


Return undefined with no newline characters

Closes https://github.com/sindresorhus/detect-newline/issues/4

Actually, it should return null, not undefined.

Why null? undefined, in this case, interops with things better (E.g. default params). I'm open to fixing it though, just curious on your own reasoning here.

https://github.com/loverajoel/jstips#05---differences-between-undefined-and-null
When you mention default params I'm honestly not sure. Happy to discuss it. The convention has always been that you return null when you explicitly want to tell the user there was no value. undefined is implicit. Like a function that doesn't return, variable not yet defined, argument to function not supplied.
Found this one on Stack Overflow:

null: absence of value for a variable; undefined: absence of variable itself;

And:

You might consider undefined to represent a system-level, unexpected, or error-like absence of value and null to represent program-level, normal, or expected absence of value. - JavaScript:The Definitive Guide

@sindresorhus Updated. Sorry that took so long, default parameters will be interesting going forward since they only support undefined, but I agree with the reasoning for null - should make it consistent instead of trying to support a single use-case.


Kubefirst should warn on new version

What would you like to be added:
When I run the kubefirst command, the CLI should warn me if there is a newer version available so I can upgrade as it may fix issues I have or add features that will make my life even better.
Why is this needed:
Maybe not needed at each run, and not sure the mechanism we want to use if we implement that, but it would need to check the actual version, the latest available, and if different, mention it to the user.

we should warn on the creation of mgmt clusters, otherwise we should allow them to stay pinned to the version of their mgmt cluster

Closing. Duplicate of https://github.com/kubefirst/kubefirst/issues/546.


Support for animated GIFS (Android)

This library uses Glide which handles gifs with the asGif method. The current request listener expects a <String, GlideDrawable> and will not support gifs. A new request listener should be formed depending on the mimetype of the image being pulled.
private String getMimeType(String url) {

        String type = null;

        String extension = MimeTypeMap.getFileExtensionFromUrl(url);

        if (extension != null) {

            type = MimeTypeMap.getSingleton().getMimeTypeFromExtension(extension);

        }

        return type;

    }

If it returns a gif, the RequestListener should create a request with <String, GifDrawable> and use the asGif method when loading into the imageview.
example:
import com.bumptech.glide.load.resource.gif.GifDrawable;

private RequestListener<String, GifDrawable> requestListener;

Thank for your suggestions. GIF are now supported in module version 1.0.8 and 2.0.8.

:1st_place_medal:


Updated emojis

This PR updates the list of supported emojis by running included tool.
798 new emojis is not a joke!

Would you mind drafting a new release with these changes?

Thank you @KazaiMazai . I published 0.4.1.

@safx Could you please publish this on CocoaPods?


appveyor.yml: split {build,test}_scripts to avoid exit code masking.

Red cross from Travis is because I've been canceling build, and [expected] red-cross from Appveyour is because #3133.

Hmm... I tried "dismiss review" button, and it didn't generate e-mail notification, so I'm going to abstain from it in the future. It's ready to go, do look now :-)


Added Airdrop highlight function to the lootable container feature.

Added Airdrop container Id to KnownTemplateId.cs
Added airdrop container Id to the container checking logic in LootableContainers.cs

This is PR is for issue #425
https://github.com/sailro/EscapeFromTarkov-Trainer/issues/425

Thanks for the PR!. One last request: could you restore the file scope namespace instead of using legacy style ? (I mean using namespace EFT.Trainer.Features; instead of namespace EFT.Trainer.Features { ... }), because it's messing up with the diff.
Thank you!


Implemented Windows-specific fix

This fixes a Windows pathing issue regarding Miserlou/Zappa#251.
All of the tests pass, with the exception of one: ERROR: test_upload_remove_s3 (tests.tests.TestZappa)
This is due to random filesystem permissions on my Windows 10 box, so if someone else has a Windows box and can run the tests, it would be nice to double tap.
Here is the test case I was using to find the fix:
import os
base = __file__.rsplit(os.sep, 1)[0]

try:
    print("test: using a redundant join method.")
    test = ''.join(os.path.join([base, os.sep, 'foo', os.sep, 'bar.py']))
    print("success: {0}".format(test))
except Exception as e:
    print("failure: known error: {0}!".format(e))

try:
    print("test: using regular join method.")
    test = "".join([base, os.sep, 'foo', os.sep, 'bar.py'])
    print("success: {0}".format(test))
except Exception as e:
    print("failure: join() error: {0}!".format(e))

try:
    print("test: using concatenation.")
    test = os.path.join([base, os.sep, 'foo', os.sep, 'bar.py'])
    print("success: {0}".format(test))
except Exception as e:
    print("failure: os.path.join() error: {0}".format(e))

try:
    print("test: using os.path.join() without casting list.")
    test = os.path.join(base, [os.sep, 'foo', os.sep, 'bar.py'])
    print("success: {0}".format(test))
except Exception as e:
    print("failure: os.path.join() error: {0}".format(e))

print("done.")

If you want to hear about the in-depth reason, I'm more than willing to go into as much detail as you'd like.
:dromedary_camel:

Coverage remained the same at 86.111% when pulling 9962bfef0d4203f5a8bc0d1f072b9b1d552718ac on mynameismevin:master into<PHONE_NUMBER>a19b1a723ea780929bc77cce11f011 on Miserlou:master.

Excellent, thank you! Certainly a bit of a strange error, thanks for fixing!

Welcome! :)

would this fix have made it into "pip install" ?

I think that line just got fixed in master recently, maybe try that? New version will be published soon

Ok great, master worked fine with  pip install --upgrade git+git://github.com/Miserlou/Zappa@master


Add public header support for .inc files

Public header files do not support .inc files. Therefore, the gradle native plugins
'cpp-library', 'c-library' does not export .inc files when publishing libraries.
Steps to Reproduce (for bugs)
Create a c++ library project with a .inc header file in the public header directory.

Publish the library locally.
The exported public header files do not contain the .inc files.

Your Environment
Gradle - 6.3
OS - Mac OS X 10.14.6
Compiler - Clang

You could manually include the files in the publishing zip for the headers to work around your issue. However, the zip task for the header is somewhat internal so it's not ideal.

I ended up writing a task that overwrites the cpp-headers.zip file. Would be nice if the header extension can be configurable.

I agree, I will consider it as part of the Nokee plugins.


[Keyvault] Fix warning on unresolved dependencies

This PR fixes the rollup script to avoid the below warning

/azp run js - keyvault - ci


[web] Introduce a default other component

Introduce a default auto-generated other component that contains files
that are uncovered by the rest of the components.

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 2

Complexity decreasing per file
==============================
+ web/server/codechecker_server/api/report_server.py  -9

See the complete overview on Codacy


Watch battery level widget on phone

[proposed Label] enhancement
Any chance the phone battery level could be added to any of the widgets? Or even a new widget..
A notification when the phone battery level went below a certain level would be awesome as well.
I of course do realise I could install a separate app for this but thought it would be a nice feature to have in the app. I like to run my phone/watch as lean as I can...
Many thanks for your hard work on this awesome app :)

Hi. Would you welcome a pull request for this feature?
Decided to learn some Android dev and this seems like a good thing to try (I'm an iOS dev usually)

Hi, if you like, you can provide a change.
Thank you for your support!
Michael

HI Michael, my pull request is ready to add but I don't have permission to push changes.. Do I need to create a fork?  (I've only worked in private repo's before)

I added you as contributor: https://github.com/pachi81/GlucoDataHandler/invitations
But please do not merge to branch 1.3!
Use your own one, so I can check.

Done :)


Minimap SaveKey may return null for dynamic overlays

@biship reported in commit e211d66a7b9b8cb23ca191aec436623497ee0e4e
Using this commit, saves workfine until I changed a minimap setting (toggled the zone or viewport setting). Now I can't save. I am using dev branch, latest commit: 4343ac7
But it's possibly this regex change that introduced the bug.
To reproduce, load a game, save (saves fine), change one or some of the minimap settings, & you get this error on next save.
An exception was thrown during saving to "C:\Users\nick\AppData\LocalLow\Ludeon Studios\RimWorld\Saves\Colonistites3.rws": System.ArgumentNullException: Argument cannot be null. Parameter name: input at System.Text.RegularExpressions.Regex.Replace (System.String input, System.String replacement, Int32 count, Int32 startat) [0x00000] in :0 at System.Text.RegularExpressions.Regex.Replace (System.String input, System.String replacement) [0x00000] in :0 at CommunityCoreLibrary.MiniMap.MiniMapOverlay.get_SaveKey () [0x00000] in :0 at CommunityCoreLibrary.MiniMap.MiniMapController.ExposeDataSave () [0x00000] in :0 at CommunityCoreLibrary.MiniMap.MiniMapController.ExposeData () [0x00000] in :0 at Verse.Scribe_Deep.LookDeep[MapComponent] (Verse.MapComponent& target, Boolean saveDestroyedThings, System.String label, System.Object[] ctorArgs) [0x00000] in :0 at Verse.Scribe_Collections.LookList[MapComponent] (System.Collections.Generic.List1& list, Boolean saveDestroyedThings, System.String label, LookMode lookMode, System.Object[] ctorArgs) [0x00000] in :0
at Verse.Scribe_Collections.LookListMapComponent [0x00000] in :0
at Verse.Map.ExposeComponents () [0x00000] in :0
at Verse.Map.ExposeData () [0x00000] in :0
at Verse.Scribe_Deep.LookDeepMap [0x00000] in :0
at Verse.Scribe_Deep.LookDeepMap [0x00000] in :0
at Verse.Game.ExposeData () [0x00000] in :0
at Verse.Scribe_Deep.LookDeepGame [0x00000] in :0
at Verse.Scribe_Deep.LookDeepGame [0x00000] in :0
at Verse.GameDataSaveLoader.m__814 () [0x00000] in :0
at Verse.SafeSaver.DoSave (System.String fullPath, System.String documentElementName, System.Action saveAction) [0x00000] in :0
(Filename: C:/buildslave/unity/build/artifacts/generated/common/runtime/UnityEngineDebugBindings.gen.cpp Line: 37)
Exception while saving map: System.ArgumentNullException: Argument cannot be null.
Parameter name: input
at System.Text.RegularExpressions.Regex.Replace (System.String input, System.String replacement, Int32 count, Int32 startat) [0x00000] in :0
at System.Text.RegularExpressions.Regex.Replace (System.String input, System.String replacement) [0x00000] in :0
at CommunityCoreLibrary.MiniMap.MiniMapOverlay.get_SaveKey () [0x00000] in :0
at CommunityCoreLibrary.MiniMap.MiniMapController.ExposeDataSave () [0x00000] in :0
at CommunityCoreLibrary.MiniMap.MiniMapController.ExposeData () [0x00000] in :0
at Verse.Scribe_Deep.LookDeepMapComponent [0x00000] in :0
at Verse.Scribe_Collections.LookListMapComponent [0x00000] in :0
at Verse.Scribe_Collections.LookListMapComponent [0x00000] in :0
at Verse.Map.ExposeComponents () [0x00000] in :0
at Verse.Map.ExposeData () [0x00000] in :0
at Verse.Scribe_Deep.LookDeepMap [0x00000] in :0
at Verse.Scribe_Deep.LookDeepMap [0x00000] in :0
at Verse.Game.ExposeData () [0x00000] in :0
at Verse.Scribe_Deep.LookDeepGame [0x00000] in :0
at Verse.Scribe_Deep.LookDeepGame [0x00000] in :0
at Verse.GameDataSaveLoader.m__814 () [0x00000] in :0
at Verse.SafeSaver.DoSave (System.String fullPath, System.String documentElementName, System.Action saveAction) [0x00000] in :0

@biship:
Can you provide a list of the area names?  The regex is removing "non-word characters" so it can create a "valid save key".  I didn't think about what would happen if there were no valid characters in the label.

I only have a few areas in the game. Home, Outside & Shop. All English text. I pull all Area_ from the save. There seems to be some inbuilt game hidden areas too.
`

Area_Named_Outside
Area_Home
Area_Named_Shop
Area_Home
Area_Named_Outside
`
Sorry for posting on the commit. I didn't want to open a issue and then have it closed being told the dev branch is only for testing!

No worries, I have it fixed now.  :)


Use os.platform to determine End-of-line

This patch adds a check to see if os.platform is available. If so, it will be used to determine end-of-line, otherwise it will default to \r\n as it is the default for 2.5.2.

hi @visionmedia,
thanks for merging my pull request. however, i have a question about the tweak you made after the merge.
the reason why i did

var eol = os.platform === undefined ? '\r\n' : ('win32' === os.platform() ? '\r\n' : '\n');

instead of

var eol = 'win32' == os.platform() ? '\r\n' : '\n';

is that node 0.4 doesn't have os.platform function. if you are planning to obsolete node 0.4 with express 2.x then i think that will work fine, but if you are still planning to keep node 0.4 support for express 2.x, then i think the code should check for availability of os.platform.

oh right forgot about 0.4.x

ok, then that sounds good for me :)


ldc checkPhase fails

ldc checkPhase fails
See build log
1600:  ... fail_compilation/testhtml2.html -O (-d -dw)
1600:  ... fail_compilation/testhtml3.html -O (-d -dw)
1600:  ... fail_compilation/testhtml.html -O (-d -dw)
1600: make: Leaving directory '/build/ldc-1.15.0-src/tests/d2/dmd-testsuite'
4/4 Test #1600: dmd-testsuite ....................   Passed  193.09 sec

The following tests passed:
        clean-dmd-testsuite-debug
        clean-dmd-testsuite
        dmd-testsuite-debug
        dmd-testsuite

100% tests passed, 0 tests failed out of 4

Total Test time (real) = 193.14 sec
Test project /build/ldc-1.15.0-src/build
    Start 1: build-ldc2-unittest
1/2 Test #1: build-ldc2-unittest ..............   Passed   40.02 sec
    Start 2: ldc2-unittest
2/2 Test #2: ldc2-unittest ....................Child aborted***Exception:   0.03 sec
Fatal error in EH code: DWARF header has unexpected format 2

50% tests passed, 1 tests failed out of 2

Total Test time (real) =  40.08 sec

The following tests FAILED:
          2 - ldc2-unittest (Child aborted)
Errors while running CTest
builder for '/nix/store/hxzygqhkhg4kqs2wdcmim65lp20vwbq3-ldc-1.15.0.drv' failed with exit code 8

It seems to have been disabled on darwin but now fails on all platforms.
According to @ThomasMader this was introduced in https://github.com/NixOS/nixpkgs/pull/56744.
See https://github.com/NixOS/nixpkgs/pull/61222#issuecomment-491443085 for more details.
Interestingly enough the same failure was seen with  LLVM 7 in https://github.com/ldc-developers/ldc/issues/2954
cc @matthewbauer

I think I was wrong. I guess the problem is the switch to llvm 7 from llvm 5 as a default.
I will try it out and follow up with a PR if everything works.

That was the problem this issue can be closed with https://github.com/NixOS/nixpkgs/pull/61354.


Upgrade support for validation tests

Add support in our current testing framework to allow the ability to tests upgrades from one version to another.

Framework and tests were tested and merged. So closing it.

List of PRs for this task:
https://github.com/rancher/validation-tests/pull/122
https://github.com/rancher/validation-tests/pull/116


Hi, it seems due to the incorrect model and the batch size. Please try LoftQ/Llama-2-7b-hf-4bit-64rank and total batch size equal to 64. Meanwhile, I have uploaded the new version training script

I can reproduce the perplexity of 5.60 now. Though it's slightly higher than the reported 5.24, I think it is acceptable.


Slash character in Heading creates invalid link fragment

Describe the bug
If a level 2 heading contains a slash (/) character, Create-TOC will generate an invalid link fragment in the generated Table of Contents.
To Reproduce

Open an existing document or create a new one that has at least one Level 2 heading that looks like this: ## Items / Todos
Activate 'Create Table of Contents'
The Table of Contents contains a link fragment that includes the slash character.
Markdown Linters will warn with a yellow under-squiggle stating 'MD051/link-fragments: Link  fragments should be valid'.
Clicking the rendered link (or using CTRL + Click in Markdown Page) does not correctly highlight and scroll the focus to the intended heading.

Expected behavior
When a Table of Contents is generated, all link fragments should be valid Markdown and upon clicking will scroll the markdown page to the correct heading.
Screenshots
none.
Environment

OS: Windows 11, 64-bit (10.0.22631)
VS Code: 1.91.0
Electron: 29.4.0
Chromium: 122.0.06261.156
Node: 20.9.0
V8: 12.2.281.27-electron.0
Extension version: v0.3.1 (pre-release)

Additional context
Tested in standard heading style only.

Fixed with PR #20


--max-downloads generates warning even when the limit was not exceeded

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2023.02.17 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
When using the option --max-downloads <limit> if the limit is reached but not exceeded, yt-dlp will output

[info] Maximum number of downloads reached, stopping due to --max-downloads
Aborting remaining downloads

and exit with status 101; Even though there is no "remaining downloads" to abort.
For instance --max-downloads 1 will always produce the above output and return 101 even when doing a single download.
I would expect to get the message and 101 status only if there actually was strictly more downloads to be done than the set limit.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['https://www.youtube.com/watch?v=aqz-KE-bpKQ', '--max-downloads', '1', '-vU', '-f', 'ba']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2023.02.17 [a0a7c01] (pip)
[debug] Python 3.10.6 (CPython x86_64 64bit) - Linux-5.15.0-48-generic-x86_64-with-glibc2.35 (OpenSSL 3.0.2 15 Mar 2022, glibc 2.35)
[debug] exe versions: ffmpeg 4.4.2 (setts), ffprobe 4.4.2, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.11.0, brotli-1.0.9, certifi-2020.06.20, mutagen-1.45.1, pyxattr-0.7.2, secretstorage-3.3.1, sqlite3-2.6.0, websockets-10.3
[debug] Proxy map: {}
[debug] Loaded 1782 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2023.02.17, Current version: 2023.02.17
yt-dlp is up to date (2023.02.17)
[youtube] Extracting URL: https://www.youtube.com/watch?v=aqz-KE-bpKQ
[youtube] aqz-KE-bpKQ: Downloading webpage
[youtube] aqz-KE-bpKQ: Downloading android player API JSON
[debug] Loading youtube-nsig.11e3a4ec from cache
[debug] [youtube] Decrypted nsig syXCvH2ENKRebldK => Oicnk5NWqIkYFg
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, vcodec:vp9.2, channels, acodec, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), channels, acodec, lang, proto, filesize, fs_approx, tbr, vbr, abr, asr, vext, aext, hasaud, id
[info] aqz-KE-bpKQ: Downloading 1 format(s): 258
[debug] Invoking http downloader on "..."
[download] Destination: Big Buck Bunny 60fps 4K - Official Blender Foundation Short Film [aqz-KE-bpKQ].m4a
[download] 100% of   29.34MiB in 00:00:01 at 22.58MiB/s
[FixupM4a] Correcting container of "Big Buck Bunny 60fps 4K - Official Blender Foundation Short Film [aqz-KE-bpKQ].m4a"
[debug] ffmpeg command line: ffmpeg -y -loglevel repeat+info -i 'file:Big Buck Bunny 60fps 4K - Official Blender Foundation Short Film [aqz-KE-bpKQ].m4a' -map 0 -dn -ignore_unknown -c copy -f mp4 -movflags +faststart 'file:Big Buck Bunny 60fps 4K - Official Blender Foundation Short Film [aqz-KE-bpKQ].temp.m4a'
[info] Maximum number of downloads reached, stopping due to --max-downloads
Aborting remaining downloads

Why is that a problem? The code does not check if there is something else in queue once the limit has reached. This allows us to skip extraction of the next item completely

It is a problem because if I set a max download limit at N, I probably do not care if it downloaded N-1 or N files: in both case all files have been downloaded.
With the current behavior if I set a limit at N, I get the same behavior whether N or N+1 files should be downloaded: in both cases get I get a warning message and a 101 exit status. But in one case all files have been downloaded and in the other case some files were not downloaded.
Also looking at https://github.com/yt-dlp/yt-dlp/issues/4262#issuecomment-1173133105

101 = Download cancelled by --max-downloads etc

One would assume that getting 101 means at least one download was cancelled.

However, it is not possible to say reliably whether there are videos left to download without attempting to run extraction on the next URL. That seems like a waste
Can you elaborate your use-case?

I would not mind yt-dlp reporting 101 if there are more URLs to run extraction on.
But in my use cases, yt-dlp does not have more URLs to run extraction on.
For instance when downloading a youtube playlist, yt-dlp knowns the number of items before the first video is downloaded:

download] Downloading playlist: XYZ
[youtube:tab] Playlist XYZ: Downloading 4 items of 4
[download] Downloading item 1 of 4
[youtube] Extracting URL: https://www.youtube.com/watch?v=xyz1

yet at the end it still reports 101

download] Downloading item 4 of 4
[youtube] Extracting URL: https://www.youtube.com/watch?v=xyz4
[youtube] xyz4: Downloading webpage
[youtube] xyz4: Downloading android player API JSON
[info] xyz4: Downloading 1 format(s): 251
[download] Destination: xyz4.webm
[download] 100% of   13.00MiB in 00:00:13 at 1.00MiB/s
[info] Maximum number of downloads reached, stopping due to --max-downloads
Aborting remaining downloads

My issue is that I use yt-dlp through a wrapper script that detects 101 status code as an error and I cannot just ignore it, because I'd like to know if there was actually more video to be downloaded (or more urls left to be extracted)

I'll look into what can be done

Also having an issue here. I'm trying to download one random song from a playlist with YoutubeDL.extract_info passing in --max-downloads:1. However, since this immediately throws an exception after downloading that song, I'm unable to retrieve the metadata from the video.

Adding in my thoughts here: The behavior I was expecting from --max-downloads was that there wouldn't be any error at all, let alone when the number of downloads matches the provided number. I just wanted to download the first thing at the URL and if there's anything else, that's fine, I don't need any error telling me about it.


STATUS_STACK_BUFFER_OVERRUN on Application::start

When I build and run the following code
use winrt::*;

import!(
    dependencies
        os
    types
        windows::ui::xaml::*
);

fn main() -> Result<()> {
    use windows::ui::xaml::{Application, ApplicationInitializationCallback};
    Application::start(ApplicationInitializationCallback::new(|_| Ok(())))
}

using rustc 1.44 stable toolchain
PS > rustc +stable -vV
rustc 1.44.0 (49cae5576 2020-06-01)
binary: rustc
commit-hash: 49cae55760da0a43428eba73abcb659bb70cf2e4
commit-date: 2020-06-01
host: x86_64-pc-windows-msvc
release: 1.44.0
LLVM version: 9.0

with winrt-rs 0.7,
[dependencies]
winrt = "0.7"

the application exit with code 0xc0000409, STATUS_STACK_BUFFER_OVERRUN.
PS > cargo run
......
    Finished dev [unoptimized + debuginfo] target(s) in 55.54s
     Running `target\debug\first-uwp.exe`
error: process didn't exit successfully: `target\debug\first-uwp.exe` (exit code: 0xc0000409, STATUS_STACK_BUFFER_OVERRUN)

This is not expected because executing a UWP program (written in C++/WinRT) gives exit code 0. Is it a problem with delegate/closure in winrt-rs?

Thanks for reporting this issue. There are a few things to note here. Firstly, the equivalent code in a C++ app also crashes:
using namespace winrt;
using namespace Windows::UI::Xaml;

int main()
{
    Application::Start([](auto&&) { });
}

The problem is not with the language projection (either C++ or Rust) but with the usage of the Xaml API. The callback provided to the Application::Start method is required to create an Application-derived class, but in the example (both Rust and here in C++) the callback does nothing. I believe Application::Start also requires a packaged app and not merely a desktop app (you can use Xaml Islands for that). So there's a bug in the Xaml application code because the Xaml API is being used in a way that is not supported.
On the other hand, we believe that WinRT APIs should map to 100% safe Rust. Thus if you find a bug where you can violate Rust's guarantees in safe code, that is a bug in the backing implementation. In this case we need to work with the Xaml team to ensure that this usage error is reported via a WinRT error rather than simply crashing. We should open a bug with the Xaml team here:
https://github.com/microsoft/microsoft-ui-xaml/

@kennykerr Thanks for the informative feedback. I am able to reproduce a similar crash with C++/WinRT Console Application. Once I changed the linker option /SUBSYSTEM to /SUBSYSTEM:WINDOWS, the C++/WinRT app did not crash any more. It should be the same case with Rust/WinRT.
Now that the issue is irrelavent to winrt-rs, please just close this issue.

The issue has been reported to winui project, see https://github.com/microsoft/microsoft-ui-xaml/issues/2632.

I had a closer look and it turns out that the failure in Xaml is as a result of a fail-fast operation, which is guaranteed to avoid unwinding and immediately terminate the process in a safe way. I spoke with @rylev and we have concluded that this does not violate Rust's safety principles. There may well still be a usability issue here in that the developer experience isn't all that great, but that's a matter for the Xaml team to consider.

I'm getting this error value despite setting #![windows_subsystem = "windows"] at the top of my main.rs. My goal was to play around with the new XAML UI example, but it seems unrelated to that.

@ketsuban Did you try this example? Xaml is not very forgiving if you try to use it without holding it just right.

That is exactly the code I'm using. The only difference is I didn't call my struct MyApp and I'm a little less free with wildcards (I checked, it made no difference).

And you used register.cmd to register the app and launched it from the Start menu? That particular example cannot be launched directly from cargo run.

Ah, I didn't try running from the Start menu. Do you know what needs to happen for it to work from cargo run?

For in-box Xaml I think this requires Xaml islands - basically a bunch more setup code. For WinUI it might be simpler (or not) but I haven't tried that either.
@riverar may have an example to share.

Okay, running from the Start menu isn't working either - a window appears with the logo file in it, then closes.

Debugging such registered apps is hard. If you can get something working with Xaml islands or WinUI in a regular executable it should be a lot easier to debug what might be going wrong on your system. Sorry I can't be more help - I don't know much about Xaml itself beyond the type system support to get the API callable in Rust.

Nevermind - I'd forgotten to change the executable name in the AppxManifest.xml file because my project isn't called xaml_app!

I got nerd sniped sidetracked into what turned out to be a Yak Shave of significant proportions. Partway through, I still managed to release cargo-uwp, a Cargo custom command that sets up a UWP-ready Cargo package, alongside a build script that—among other things—keeps the AppxManifest.xml in sync with the artifacts.
The documentation is still a bit thin, but there is a short section on debugging a registered application. Maybe that is of help to you, @ketsuban.


Issues aren't auto added to project board when they're created

Describe the bug
Problem is described at: https://github.community/t/new-issues-not-being-auto-added-to-project-board/904/60).
Fortunately there's a solution at the same page: https://github.community/t/new-issues-not-being-auto-added-to-project-board/904/60
I've implemented it. Will it work? hmmm

Confirmed - it works.
Note that I need to leave a special card in the project.


Set up Python package structure and a couple fixes

Signed-off-by: terrytangyuan<EMAIL_ADDRESS>
What this PR does / why we need it:
Fixes #18. Fixed a few typos in README. Also fixed the following exception:
---> 13 class FTRebuildStatus(ConsensusStatus):
     14     ABORT = 2

/usr/local/Cellar/python/3.6.5_1/Frameworks/Python.framework/Versions/3.6/lib/python3.6/enum.py in __prepare__(metacls, cls, bases)
    117         enum_dict = _EnumDict()
    118         # inherit previous flags and _generate_next_value_ function
--> 119         member_type, first_enum = metacls._get_mixins_(bases)
    120         if first_enum is not None:
    121             enum_dict['_generate_next_value_'] = getattr(first_enum, '_generate_next_value_', None)

/usr/local/Cellar/python/3.6.5_1/Frameworks/Python.framework/Versions/3.6/lib/python3.6/enum.py in _get_mixins_(bases)
    434                     issubclass(base, Enum) and
    435                     base._member_names_):
--> 436                 raise TypeError("Cannot extend enumerations")
    437         # base is now the last base in bases
    438         if not issubclass(base, Enum):

TypeError: Cannot extend enumerations

Which issue(s) this PR is related to (optional, link to 3rd issue(s)):
Fixes #
Reference to #

Special notes for your reviewer:
/cc @zw0610 @gaocegege @ddysher @pendoragon

Release note:

NONE

/cc @zw0610 @gaocegege @ddysher @pendoragon

/lgtm


Galaxy Workflows

We are exploring the addition of Galaxy Workflows into the Dockstore eco-system.

What do we need to know about ?

Does Galaxy have a concept

What would an interface class look like?

┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Fix Versions: Dockstore 1.8
┆Sprint: Seabright Sprint 12 Liner
┆Issue Number: DOCK-181
┆Epic: Galaxy-Dockstore Integration

➤ Melaina Legaspi commented:
[~accountid:5ad7afeb1db05e2a66fa947b] remember to remind Seabright/Dockstore to create more specific followup tickets for this issue


feat(gapic-common): The gRPC service stub supports default timeouts

Part of the alternative to #639

We made the decision to do #639 instead. This PR will no longer be necessary.


Updated manage-runas-account.md

Get-AzSubscription cmdlet returns the subscription ID, name and tenant for subscriptions that the account can access. The $SubscriptionName variable would end up being null with the actual code (line 169):
$SubscriptionName = $subscription.Subscription.Name
Also change the current context to use the specified subscription. it's better (and safer) to explicitly define the context before creating the Service Principal (as the default for Role and Scope parameters are "Contributor" and the current subscription).
See #55110

#sign-off


add stakeholders to review datadog-agent integration

What does this PR do?
Motivation
Add agent-core, agent-integrations, and prodsec as datadog-agent integration command reviewers.
Additional Notes
N/A

No problemo, thanks boss 🙇

Hi, can I get a ping, please? 🙇

Thanks for the ping, merging 👍


[R][BACKPORT-2.3] update package description

What changes were proposed in this pull request?
#23852
doesn't port cleanly to 2.3. we need this in branch-2.4 and branch-2.3

Test build #102580 has started for PR 23861 at commit fe224de.

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/testing-k8s-prb-make-spark-distribution-unified/8143/
Test PASSed.

Test build #102580 has finished for PR 23861 at commit fe224de.

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/102580/
Test PASSed.

Merged to branch-2.3.


Make the reboot tracker catch failed coordinators, too.

Scope & Purpose
Make the reboot tracker catch failed coordinators, too.
Previously the reboot tracker was invoked only when a DB server failed or was restarted, and when a coordinator was restarted.
Now it will also act if a coordinator just fails (without restart).

[x] :hankey: Bugfix
[x] :pizza: New feature
[ ] :hammer: Refactoring
[x] :book: CHANGELOG entry made
[x] :muscle: The behavior in this PR was manually tested
[ ] :computer: The behavior change can be verified via automatic tests

Backports:

[x] Backports required for: 3.5 (https://github.com/arangodb/arangodb/pull/12636), 3.6 (https://github.com/arangodb/arangodb/pull/12606), 3.7

Testing & Verification

[x] This change is a trivial rework / code cleanup without any test coverage.

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/11877/

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/11878/

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/11883/

Tests blue


Add time series for throughput

add time series for read and write throughput

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/11292/
Test FAILed.

@calvinjia Please let me know if i am missing any metrics. Also JIRA mentioned the scalability concern, which is why I first experimented with Histogram, but I am not sure a sampling-based approach is really what we want.

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/Alluxio-PR-Builder/11294/
Test PASSed.

@calvinjia any feedback?

alluxio-bot, merge this please.


Update exporter/distro README

More clearly describe whether to use exporter or distro.
Update Python versions to 3.8

API change check
API changes are not detected in this pull request.


Update build_lambda.sh

The latest version of amazonlinux doesn't have epel as a repo, yumdownloader isn't installed and python27-pip isn't a package anymore. The proposed changes work correctly

 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.

If it helps - I can confirm that making these changes enabled the build_lambda.sh to work correctly.
However, the changes made in #44 are also required to make the output lambda also work.

Sorry about the delays here - this is handled by https://github.com/upsidetravel/bucket-antivirus-function/pull/52


Timeframe is now restricted?

I want to get the DJIA values for 2020 in order to reproduce it with a Principal Component Analysis following this notebook using alpha_vantage. Yet it seems that if the notebook author was available to get the data for the whole year he was interested in, when I run the data it seems that I only have two months:
"""
Download the all-time DJIA dataset
"""
from alpha_vantage.timeseries import TimeSeries

# Update your Alpha Vantage API key here...
ALPHA_VANTAGE_API_KEY = 'MYKEY'

from pprint import pprint
ts = TimeSeries(key=ALPHA_VANTAGE_API_KEY, output_format='pandas')
df, meta_data = ts.get_intraday(symbol='DIA',interval='1min', outputsize='full')
pprint(df.info())

Returns:
<class 'pandas.core.frame.DataFrame'>
DatetimeIndex: 5351 entries, 2021-02-03 20:00:00 to 2021-01-21 04:02:00
Data columns (total 5 columns):
 #   Column     Non-Null Count  Dtype
---  ------     --------------  -----
 0   1. open    5351 non-null   float64
 1   2. high    5351 non-null   float64
 2   3. low     5351 non-null   float64
 3   4. close   5351 non-null   float64
 4   5. volume  5351 non-null   float64
dtypes: float64(5)
memory usage: 250.8 KB

Has alpha_vantage reduced the available data? Can I extend the timeframe? If it is not possible, how can I get the data for the DJIA otherwise?

@antoinecomp, this seems to be server-side behavior that is beyond the scope of this Python library. Could you please relay you message/questions to<EMAIL_ADDRESS>