Support region topology for the ebs in tree provider

Version

Karpenter: v0.5.4

Kubernetes: v1.21.5
Expected Behavior

Karpenter creates nodes using volume topology when a PVC already exists for a pod
Actual Behavior

Karpenter can't create a new node due to wrong nodeSelector
Steps to Reproduce the Problem

Use the following steps and manifests.

Apply manifests first time (a node will be provisioned)
Scale STS to 0
Wait until the node deleted
Scale STS to 1
Get the error

So it looks like Karpenter can't provision nodes if a PVC already exists for a pod.
Manifests:
apiVersion: v1
kind: Service
metadata:
  name: nginx
  labels:
    app: nginx
spec:
  ports:
    - port: 80
      name: web
  clusterIP: None
  selector:
    app: nginx
---
apiVersion: apps/v1
kind: StatefulSet
metadata:
  name: web
spec:
  serviceName: "nginx"
  replicas: 2
  selector:
    matchLabels:
      app: nginx
  template:
    metadata:
      labels:
        app: nginx
    spec:
      nodeSelector:
        only-db: "true"
      tolerations:
        - key: "only-db"
          operator: Exists
          effect: NoSchedule
      containers:
        - name: nginx
          image: k8s.gcr.io/nginx-slim:0.8
          ports:
            - containerPort: 80
              name: web
          volumeMounts:
            - name: www
              mountPath: /usr/share/nginx/html
  volumeClaimTemplates:
    - metadata:
        name: www
      spec:
        accessModes: [ "ReadWriteOnce" ]
        resources:
          requests:
            storage: 1Gi

Resource Specs and Logs

Provisioner:
apiVersion: karpenter.sh/v1alpha5
kind: Provisioner
metadata:
  annotations:
    meta.helm.sh/release-name: smb-eu-central-1
    meta.helm.sh/release-namespace: smb-eu-central-1
  creationTimestamp: "2022-01-08T05:39:28Z"
  generation: 1
  labels:
    app.kubernetes.io/managed-by: Helm
  name: storage-db-smb-eu-central-1
  resourceVersion: "171015299"
  uid: 3e707cf2-a6a0-4b06-b8ae-5d8675722cbc
spec:
  kubeletConfiguration: {}
  labels:
    only-db: "true"
  limits: {}
  provider:
    apiVersion: extensions.karpenter.sh/v1alpha1
    instanceProfile: ad-eks-c-NodeInstanceProfile
    kind: AWS
    launchTemplate: ad-eks-db-lt
    securityGroupSelector:
      aws:eks:cluster-name: ad-eks-c
    subnetSelector:
      kubernetes.io/cluster/ad-eks-c: '*'
  requirements:
  - key: kubernetes.io/arch
    operator: In
    values:
    - amd64
  - key: karpenter.sh/capacity-type
    operator: In
    values:
    - on-demand
  taints:
  - effect: NoSchedule
    key: only-db
    value: "true"
  ttlSecondsAfterEmpty: 120
status:
  resources:
    cpu: "0"
    memory: "0"

Log:
tried provisioner/storage-db-smb-eu-central-1: invalid nodeSelector "topology.kubernetes.io/region", [eu-central-1] not in []	{"commit": "7e79a67", "pod": "kazoo-db/storage-db-smb-eu-central-1-0"}

Part of pod spec:
      nodeSelector:
        only-db: "true"
      tolerations:
      - effect: NoSchedule
        key: only-db
        value: "true"
      topologySpreadConstraints:
      - labelSelector:
          matchLabels:
            app: db
            release: smb-eu-central-1
        maxSkew: 1
        topologyKey: kubernetes.io/hostname
        whenUnsatisfiable: DoNotSchedule
      - labelSelector:
          matchLabels:
            app: db
            release: smb-eu-central-1
        maxSkew: 1
        topologyKey: topology.kubernetes.io/zone
        whenUnsatisfiable: DoNotSchedule

As I can see Karpenter tries to use some other nodeSelector for some reason.

It looks like the topology region label is getting applied to your volume somehow. Is it possible that this label is a property on your storage class for some reason?
Karpenter doesn't support (currently) the region label since it doesn't support multiple regions. We could always add noop support for it, but I'm curious how this is getting applied in the first place.

Yeah, my volumes have the following labels:
apiVersion: v1
kind: PersistentVolume
metadata:
  annotations:
    kubernetes.io/createdby: aws-ebs-dynamic-provisioner
    pv.kubernetes.io/bound-by-controller: "yes"
    pv.kubernetes.io/provisioned-by: kubernetes.io/aws-ebs
  creationTimestamp: "2021-12-27T08:10:47Z"
  finalizers:
  - kubernetes.io/pv-protection
  labels:
    topology.kubernetes.io/region: eu-central-1
    topology.kubernetes.io/zone: eu-central-1a
...

But the StorageClass doesn't have any unexpected settings:
allowVolumeExpansion: true
apiVersion: storage.k8s.io/v1
kind: StorageClass
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"storage.k8s.io/v1","kind":"StorageClass","metadata":{"annotations":{"storageclass.kubernetes.io/is-default-class":"true"},"name":"gp2"},"parameters":{"fsType":"ext4","type":"gp2"},"provisioner":"kubernetes.io/aws-ebs","volumeBindingMode":"WaitForFirstConsumer"}
    storageclass.kubernetes.io/is-default-class: "true"
  creationTimestamp: "2021-01-26T12:45:33Z"
  name: gp2
  resourceVersion: "76963236"
  uid: d2acc111-2227-4953-9c5d-ea09084b70a0
parameters:
  fsType: ext4
  type: gp2
provisioner: kubernetes.io/aws-ebs
reclaimPolicy: Delete
volumeBindingMode: WaitForFirstConsumer

I know of a couple of configurations that will cause this. I don't know what EKS use, but I am guessing it depends on versions.
First off, there is this KCM admission controller
Secondly, it is CCMs GetLabelsForVolume. I believe this is only used if you are using the in-tree EBS volume driver, which you are (the kubernetes.io/aws-ebs bit).
If you are using the AWS EBS CSI driver, I believe these labels would not be set.

Yeah, I've tested it.
When I use https://kubernetes.io/docs/concepts/storage/volumes/#awselasticblockstore
The labels and the nodeAffinity are set to volumes.
When I use AWS EBS CSI driver the volume looks like this and seems it will work as expected:
apiVersion: v1
kind: PersistentVolume
metadata:
  annotations:
    pv.kubernetes.io/provisioned-by: ebs.csi.aws.com
  creationTimestamp: "2022-01-08T08:41:35Z"
  finalizers:
  - kubernetes.io/pv-protection
  - external-attacher/ebs-csi-aws-com
  name: pvc-2986b2aa-d788-4549-a9b7-01bb8198dc48
  resourceVersion: "171094109"
  uid: f91cbd91-a115-4c3e-b39e-77d5dc6b62f4
spec:
  accessModes:
  - ReadWriteOnce
  capacity:
    storage: 4Gi
  claimRef:
    apiVersion: v1
    kind: PersistentVolumeClaim
    name: ebs-claim
    namespace: smb-eu-central-1
    resourceVersion: "171093369"
    uid: 2986b2aa-d788-4549-a9b7-01bb8198dc48
  csi:
    driver: ebs.csi.aws.com
    fsType: ext4
    volumeAttributes:
      storage.kubernetes.io/csiProvisionerIdentity:<PHONE_NUMBER>849-8081-ebs.csi.aws.com
    volumeHandle: vol-09c8e2e41f8bdba99
  nodeAffinity:
    required:
      nodeSelectorTerms:
      - matchExpressions:
        - key: topology.ebs.csi.aws.com/zone
          operator: In
          values:
          - eu-central-1a
  persistentVolumeReclaimPolicy: Delete
  storageClassName: ebs-sc
  volumeMode: Filesystem
status:
  phase: Bound

Confirmed.
It works with https://github.com/kubernetes-sigs/aws-ebs-csi-driver
But with in-tree aws ebs provisioner it doesn't work as expected.

How difficult is it for you to migrate to the out-of-tree provider? I think we likely want to support the in-tree in the short term, but perhaps not in the long term.

@ellistarn
I already have moved to the ebs csi driver due to the issue.
So it is not a problem anymore. At least for me.
This issue was a good point to try an out-of-tree storage provider (:

Probably it would be convenient to update the documentation about this limitation to avoid new similar issues

FWIW I ran into this issue yesterday. Having support for the in-tree provisioner would be greatly appreciated! I will look into switching to the AWS EBS CSI driver, but I assume that will be too much churn right now.

We're running into an issue where the in-tree provisioner used the deprecated failure-domain.beta.kubernetes.io/{region,zone} labels, and so Karpenter will not select it:
2022-01-19T19:37:11.206Z	DEBUG	controller.selection	Could not schedule pod, matched 0/1 provisioners, tried provisioner/default: invalid nodeSelector "failure-domain.beta.kubernetes.io/region", [us-east-2] not in []	{"commit": "7e79a67", "pod": "cockroachdb-system/cockroachdb-1"}

It would be really helpful if Karpenter could support these labels for those of us still using in-tree provisioned volumes.

We're running into this same issue that prevents us from adopting Karpenter unfortunately. A workaround to support in-tree provisioner would be greatly appreciated!

I also got the same issue. Really wait for this feature to be supported soon. Thanks

hi, Folks. Looks like you should add v1.LabelFailureDomainBetaRegion to IgnoredLabels too because v1.LabelTopologyRegion - it's "topology.kubernetes.io/region" and i got error
invalid nodeSelector "failure-domain.beta.kubernetes.io/region", [us-east-2] not in []

until add v1.LabelFailureDomainBetaRegion (failure-domain.beta.kubernetes.io/region) to IgnoredLabels

I'm using v0.6.0 and have the exact same issue.

https://github.com/aws/karpenter/pull/1271/files


Information about tokeniser (use of root zone character, etc)

Hi!
I'd like to ask a quick question about your tokeniser.
Imagine that I have something like this (simplified) example:
$ORIGIN typefish.co.uk.

@                                   3600 IN A     <IP_ADDRESS>
a                                   3600 IN A     <IP_ADDRESS>
a.typefish.co.uk                    3600 IN A     <IP_ADDRESS>
a.typefish.co.uk.                   3600 IN A     <IP_ADDRESS>

We end up with output from named-compilezone similar to:
typefish.co.uk.                     3600 IN A     <IP_ADDRESS>
a.typefish.co.uk.                   3600 IN A     <IP_ADDRESS>
a.typefish.co.uk.a.typefish.co.uk.  3600 IN A     <IP_ADDRESS>
a.typefish.co.uk.                   3600 IN A     <IP_ADDRESS>

However, the output from your tokeniser is:
@                                   3600 IN A     <IP_ADDRESS>
a.typefish.co.uk.                   3600 IN A     <IP_ADDRESS>
a.typefish.co.uk.a.typefish.co.uk.  3600 IN A     <IP_ADDRESS>
a.typefish.co.uk.                   3600 IN A     <IP_ADDRESS>

Now, the code - and its tests, imply that the root zone is tokenised as @ as opposed to typefish.co.uk. - should this be the case?
(I personally think it should match named-compilezone)
I've changed the code on my machine to match the output of named-compilezone but I don't want to change any tests nor proceed further with any merging of code until I know why this has been done in this manner.
Regards,
David Weston

This is the line in question:
https://github.com/LTD-Beget/dns-zone-tokenizer/blob/master/src/LTDBeget/dns/record/Record.php#L115
if($this->globalOrigin && substr($this->tokens['NAME'], -1) !== '.' && $this->tokens['NAME'] !== '@') {
    if($this->globalOrigin === '.') {
        $this->tokens['NAME'] .= $this->globalOrigin;
    } else {
        $this->tokens['NAME'] .= '.'.$this->globalOrigin;
    }
}

Hi, I was wondering if someone was able to help me regarding this behaviour?


Setup event tracking

We can just integrate segment during a pre-release, but before a developer preview we need to remove any 3rd party scripts, and proxy events instead (similar to how the Browser works).
There is already a function setup which gets called during all the events we want, it just isn't integrated with Segment yet.

Moving this back to "in progress" since events not showing up in Segment debugger yet

See #111


refactor(code-snippet): changes code highlight folder location and md…

…x plugins

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (5268f1d) 81.53% compared to head (1b3face) 81.53%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #731   +/-   ##
=======================================
  Coverage   81.53%   81.53%
=======================================
  Files          31       31
  Lines         352      352
  Branches       45       45
=======================================
  Hits          287      287
  Misses         60       60
  Partials        5        5

Impacted Files
Coverage Δ

src/styles/code.tsx
100.00% <ø> (ø)

src/components/Components.tsx
58.33% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


command_name not set / update for 2.5

On task version 2.5.1
Traceback (most recent call last):
  File "/home/x/.local/bin/tw", line 11, in <module>
    sys.exit(main())
  File "/home/x/.local/lib/python2.7/site-packages/taskwarrior_capsules/cmdline.py", line 94, in main
    terminal=term,
  File "/home/x/.local/lib/python2.7/site-packages/taskwarrior_capsules/capsule.py", line 159, in execute
    self.validate(**kwargs)
  File "/home/x/.local/lib/python2.7/site-packages/taskwarrior_capsules/capsule.py", line 76, in validate
    self.command_name,
AttributeError: 'Blocks' object has no attribute 'command_name'

Probably related to:
/home/x/.local/lib/python2.7/site-packages/taskwarrior_capsules/capsule.py:79: UserWarning: Capsule 'add' is not compatible with version 2.5.1 of taskwarrior; minimum version: 2.2; maximum version 2.4.9999.

after I at least got it back to work with:
    def execute(
        self, variant, command_name, filter_args, extra_args, **kwargs
    ):
        self.command_name = command_name

Btw: is this project still alive ?

Btw: is this project still alive ?

No, I do not think it is. I would suggest checking out taskpirate.


uv installed python doesn't have os.pidfd_open

os.pidfd_open has been supported since python 3.9 and kernel 5.2, but despite this recent python versions pulled by uv don't support os.pidfd_open. It would be nice if either upstream provided builds with os.pidfd_open enabled, or if uv provided more sources for python builds, or if uv had the option to build python from source.
Related: https://github.com/indygreg/python-build-standalone/issues/193

Semi related https://github.com/astral-sh/uv/pull/8517

Adding the option to build python from source would be nice for developers looking for maximum performance from their python version as well. Using https://github.com/indygreg/python-build-standalone is much quicker to download and run for a one-off python execution, so ideal for the default uv behavior, but for somebody who wants to use the best python possible for his system and does not mind waiting a bit longer for it to build, the option for a source compiled python (like pyenv does) should be an improvement.

We don't have plans to build Python from source on the user's machine, there are other tools better suited for that. We're already investing a great deal into the standalone builds, which are pretty heavily optimized. I'd prefer to invest my time into weak symbols upstream as suggested by Greg in https://github.com/indygreg/python-build-standalone/issues/193#issuecomment-1751971155


Fix WorkerTaskExecutorThreadPool#isOverload is incorrect

Purpose of the pull request
When we submit a task into ThreadPool, if all core threads has been created(In most of times, the core threads are created.),   then the task will be submit into task queue first.
So we cannot use threadPoolExecutor.getQueue().size() > 0 to judge whether the thread pool is busy
Brief change log

Use WorkerTaskExecutorHolder to judge whether the worker is overload.

Verify this pull request
Test by UT

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

Project coverage is 40.61%. Comparing base (cb0b4fa) to head (4f338fe).

Additional details and impacted files
@@             Coverage Diff              @@
##                dev   #16027      +/-   ##
============================================
+ Coverage     40.58%   40.61%   +0.03%
- Complexity     5215     5222       +7
============================================
  Files          1380     1380
  Lines         46056    46027      -29
  Branches       4916     4892      -24
============================================
+ Hits          18692    18695       +3
+ Misses        25437    25408      -29
+ Partials       1927     1924       -3

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


Include External Schemas in connection tree view

Currently it appears that the tree view is only showing local schemas (The schemas you get back from the query "SELECT * FROM  svv_redshift_schemas")
It would be very helpful to include external schemas as well. External schemas are commonly used to expose data in other databases or the Glue Hive Meta store.
https://docs.aws.amazon.com/redshift/latest/dg/r_CREATE_EXTERNAL_SCHEMA.html
You can get the combined list of schemas with the query
SELECT * FROM  svv_all_schemas
Or you can just get the external schemas
SELECT * FROM  svv_external_schemas

Hi @pbayerle .
Sorry for my late reply.
I'm currently don't have much time to fix this issue. So, sorry again I can't tell you when to be resolved.
If you could help this issue, a PR is appreciated.


[BUG] build cuda_codegen of densenet161-fp16.onnx failed

🐛 build cuda_codegen of densenet161-fp16.onnx failed

Now I can generate densenet161-fp32 and densenet161-fp16  cudacode and have a correct output, but for fp16, I got a builded failed issue, and the message is :
error: more than one operator "+" matches these operands:
            built-in operator "arithmetic + arithmetic"
            function "operator+(const __half &, const __half &)"
            operand types are: double + half

According to my research, this was caused by that cuda does't implement operator overload for datatype half :
extern "C" __launch_bounds__(49) __global__ void BatchNormInference_half_half_half_half_half_half_cuda_BatchNormInference_1049(half* input0, half* input1, half* input2, half* input3, half* input4, half* output0)
{
    const int st = blockIdx.x * 7 * 7;
    const int c_id = blockIdx.x % 736;
    #pragma unroll 1
    for (int i = threadIdx.x; i < 7 * 7; i += blockDim.x)
    {
        output0[st + i] = (input1[c_id] + (input0[c_id] * (input2[st + i] - input3[c_id]) / sqrtf(1e-05 + input4[c_id])));
    }

}

It worked when I reconstruct the expression to below:
output0[st + i] = __hadd(input1[c_id] , __hdiv(__hmul(input0[c_id], __hsub(input2[st + i], input3[c_id])), sqrtf(__hadd(__float2half(1e-05), input4[c_id]))));

I'm now trying to figure it out.

Currently I rewrite the cuda::BatchNormNCHW::emit_function_body() to solve this problem, But I think there's another solution, to reference cutlass code, implement operator overload of half datatype, for example:
https://github1s.com/NVIDIA/cutlass/blob/HEAD/include/cutlass/half.h#L805
CUTLASS_HOST_DEVICE
half_t operator/(half_t const& lhs, half_t const& rhs) {
#if defined(__CUDA_ARCH__) && (__CUDA_ARCH__ >= 530)
  return half_t(__hdiv(lhs.to_half(), rhs.to_half()));
#else
  return half_t(float(lhs) / float(rhs));
#endif
}

CUTLASS_HOST_DEVICE
half_t& operator+=(half_t & lhs, half_t const& rhs) {
#if defined(__CUDA_ARCH__) && (__CUDA_ARCH__ >= 530)
  lhs = half_t(__hadd(lhs.to_half(), rhs.to_half()));
#else
  lhs = half_t(float(lhs) + float(rhs));
#endif
  return lhs;
}

CUTLASS_HOST_DEVICE
half_t& operator-=(half_t & lhs, half_t const& rhs) {
#if defined(__CUDA_ARCH__) && (__CUDA_ARCH__ >= 530)
  lhs = half_t(__hsub(lhs.to_half(), rhs.to_half()));
#else
  lhs = half_t(float(lhs) - float(rhs));
#endif
  return lhs;
}


Added trait for capturing screenshots.

First code review complete.

@crschoen I guess I rushed the interface extraction a forgot to clean up those comments. Everything should be fixed now.

Please add composer.lock to the .gitignore file.


[Refactor] Replaced old mascot_happy image with new light and dark mode images

Closes #5288
Changes
Background color has been modified because in dark mode with Configuration.Color.Semantic.dialogBackground (black) background, the centre ball of the logo does not show up.

Screen shots

Device
Light
Dark

iPhone SE 3

iPhone 14 Pro

iPad Pro 6

@eviltofu and this is not a refactor (have to fix the PR title and commit message)

@hboon updated file names, pr title, and commit message.


Experimental test runner (part 2)

This pull request continues from pull request #4, adding some major features to showrunner. The set of commits also contains some smaller fixes to Python 2.7 compatibility and examples/python3-urllib/run.py.
On-the-fly Certificate Generation
Add showrunner.gencert.gencert(cn="...") for generating a PEM encoded server certificate + private key pair and a PEM ca bundle against which the server certificate verifies. The function basically just wraps the openssl command and generates a server certificate for the given Common Name. To save time the private server key and the private CA key are generated once per process.
As an example you can generate cert.pem, cert.key and ca.pem like this:
from showrunner.gencert import gencert

def writefile(filename, data):
  with open(filename, "wb") as f:
      f.write(data)

cert, key, ca = gencert("localhost")
writefile("cert.pem", cert)
writefile("cert.key", key)
writefile("ca.pem", ca)

Now launching a HTTP server on localhost with the cert.pem and cert.key files allows clients to create a verified connection to it using the ca.pem CA file.
Helpers for Local Testing
Add helpers for writing local test servers: showrunner.testenv.local(ok_expected, cn, [callback=...]) launches a simple temporary TLS server on localhost (with keys generated by gencert).  After the test has been completed the temporary TLS server gets shut down.

ok_expected tells whether the connected clients should be able to verify the server certificate.

cn tells the Common Name that should be passed to gencert when generating the server certificates.

The optional callback argument can be passed in to modify how a created connection socket is used after a socket connection has been established. The default callback just does the TLS handshake, but is also used by the HTTPS bundle (described later) to imitate a webserver. For an example see showrunner.bundles.https.https_callback.

Test Bundles
Add the concept of test bundles and add a command line option -t/--test-bundle to the trytls tool for configuring the used test bundle.
A test bundle is a bundle of tests specialized for some protocol/situation ("just a plain TLS handshake", "a HTTPS server", ...). In practice a test bundle is just a named list of tests to be run. As an example see showrunner.bundles.handshakes.all_tests (described later).
The command line parameter -t (or --test-bundle) is used to tell which test suite should be run. The value should be an absolute Python import path, for example showrunner.bundles.https.all_tests for importing the test list all_tests from the showrunner.bundles.https module. For convenience the path can also be a relative one (i.e. starting with .) in which case the bundle is imported under the showrunner.bundles package meant for built-in bundles. Therefore showrunner.bundles.handshake.all_tests can be referred to as .handshake.all_tests.
The default --test-bundle value is .handshake.all_tests.
The support for absolute import paths is there because that allows people to write their own test bundles that don't have to be a part of the showrunner package. This in turn makes it easier to host the bundle code somewhere else than this repository.
Built-in Bundles
Add some example test bundles of varying quality:

.handshake.all_tests/showrunner.bundles.handshake.all_tests for testing plain TLS handshakes (on a local server and on the BadSSL servers). This is the default test bundle for the trytls tool.

.https.all_tests/showrunner.bundles.https.all_tests that is pretty much like the .handshake bundle but also sends a rudimentary HTTP response back to the client. Should help in testing HTTP clients.

.imap.all_tests/showrunner.bundles.imap.all_tests that currently just tells to connect to Gmail's IMAP servers.

Example
Testing the examples/python3-urllib/run.py stub against only the local tests in the builtin HTTPS bundle:
$ trytls -t .https.local_tests python3 examples/python3-urllib/run.py

Uh, please consider putting this epic documentation to showrunner/README.md


Update application-gateway-autoscaling-zone-redundant.md

@weixian-zhang : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Changed from "scale up or down" to "scale out or in".

#label:"aq-pr-triaged"

@weixian-zhang  This PR has a merge conflict, so this issue was fixed in a separate PR in the private repo. Please close this one.


Input/output (reading/writing files, databases, streams, csv, pathlib)

Basic

reading files:

open, close
binary vs text mode
writing and reading with iteration or with write_lines

reading and writing csv with the built-in csv module
paths with pathlib

creating paths
resolving paths
path mainpulation (suffix, appending, ...)

Advanced

streams with io module
context managers for resource management

understanding context managers
implementing context managers to handle custom resources

databases:

local databases with sqlite3 module
database querying with pandas


Improve Decoupling between S2L and Robot

Hi there. I'm working at a large government organization that has been evaluating Selenium2Library as our testing framework. After trying to write our own library on top of Selenium's WebDriver API for use in unittests, we discovered RF and S2L and were really impressed with S2L's modularity and with how easy it was to use S2L outside of Robot framework (i.e. with a Python unittest). Why would we want to do this? Well, to be honest, so far we've found that whatever you can do in a unittest you can do by writing keywords for your Robot test—but the design of the library is already so conducive to this that it's tempting to try to keep it that way, and we'd like to keep our options open for how we write and run our tests.
One thing we've started doing, in addition to running outside RF, is creating "page objects"--libraries that do functionality specific to our applications while using S2L to interact with Selenium. We're not subclassing S2L in our libraries (because we don't want to redefine all S2L's keywords (which would create a conflict) and expose them unless the user asks for them), and because our page objects are not conceptually inheriting from, but are more hiding S2L.
So, I noticed that one place in which the library is coupled to Robot is in keywords/_runonfailure.py. There is a call to robot.libraries.BuiltIn's class BuiltIn's run_keyword() method to run the default run-on-failure keyword. The result is that, after instantiating the Selenium2Library class (which is how we run the library outside RF), and running a unittest, on a failure, we get a warning that the "Capture Page Screenshot" keyword can't be found.
This also happens within Robot Framework, when our page object library is imported by a Robot test but S2L is not.
It seems to me my options are:

Just instantiate Selenium2Library with the keyword argument run_on_failure=None. But then I don't get to use the "Capture Page Screenshot" on failure.
If I want a screenshot on failure, I could define my own keyword, "Capture Page Screenshot", that simply calls capture_page_screenshot on my S2L instance.

OK, option 2 is great, but what if I want to allow some other S2L keyword to be called on failure? Now I've hardcoded "Capture Page Screenshot" into my library. Well, I could modify run_keyword() in my library so that if it gets the run-on-failure keyword it passes it along to the S2L library instead.
BUT now I'm sort of working around the coupling of a S2L (reasonable in itself) to RF, and I still don't have a solution that works outside RF. What if we added a "run_on_failure_method" argument that S2L would check first, then fall back to the keyword if it's specified.
I'd like to fork and open a pull request, but wanted to see if this strikes you as logical first.

Thanks for your input!
This is an extremely creative usage of the library, and I'm pretty impressed. I'd be willing to review any PRs you guys put in, but otherwise I can't really spend too much development effort following this path.
If you're interested just let me know and I'll reopen the issue.


CI Stack not working for ARM64 instances

Describe the bug

When deploying the Buildkite Elastic CI Stack using AWS CloudFormation on ARM64 instances like t4g.large with AMI ami-07d67603110c131fb, the Buildkite agent, Docker, and SSH keys from the secret bucket are not installed on the EC2 instance. Additionally, the IAM role does not seem to be correctly assigned.

Steps To Reproduce

Deploy the CI stack using AWS CloudFormation.
Specify ARM64 instances (t4g.large) and use AMI ami-07d67603110c131fb.
After the CI process completes and the EC2 instance is up, SSH into the instance.
Observe the absence of the Buildkite agent, Docker, and SSH keys.

Expected behavior

The EC2 instance spun up should have the Buildkite agent installed, Docker set up, and access to SSH keys from the secret bucket. The instance should also correctly assume the specified IAM role.

Actual behaviour

The EC2 instance lacks the Buildkite agent and Docker installation. SSH keys from the secret bucket and the IAM role are not correctly assigned.

Stack parameters (please complete the following information):

AWS Region: ap-south-1
Version v6.10.0

Additional context

This issue hinders the utilization of ARM64 instances in our CI pipeline, affecting the potential for performance improvements and cost optimization.

Hey @triarius, Well we were able to deploy it properly i think it issue only comes up when used it with this ami ami-07d67603110c131fb

Hi @oh-tarnished, can you tell us more about how you are using ami-07d67603110c131fb and why (compared with launching with a Buildkite AMI)? It looks like that AMI is just the base Amazon Linux 2023 image, which would lack the Buildkite Agent, Docker, etc.


IA-896 Integrate django-sql-dashboard

you can go to /explore to see it in action
New python deps you will need to rebuild your docker iaso container: docker-compose build iaso.
On the deployment side we will need to create a new user for each env/db and configure it.
https://user-images.githubusercontent.com/82500/140764755-c9ecff5f-8f31-495f-9375-b3a588b51d64.mp4

Really cool :-)


Update Wit-bindgen, related tools and make tests pass

This does a few things:

switches the default build mode to use released versions of tools
updates the submodules to point main projects
Updates the versions of wit-bindgen, wasmtools and wasm-adapters
Fixes all the samples and testing apps to use the generation changes in wit-bindgen
updates some of the testing projects to update tools and nuget packages properly, prior to this the tools didn't update when values were changed locally
add some docs for contributing and debugging

doh, I had a wrong deployment of emsdk on my path. I've been able to reproduce this and will get a fix up shortly


I2CDeviceStatus added in read and write method of I2CDevice.cpp

Changes in I2CDevice.cpp file, add_read method modified, status is given as input parameter. Status is stored in read status queue.
In case of write method, add_write_status is called, to add status, which gets stored in write status queue.
Checks added , 1. queue size of read buf and read status buf should be same. 2. Count and buf size should be equal.
Test cases of Mock I2CDevice.
Modified test cases for Sensor.cpp and Device.cpp according to I2CDevice latest changes.For Device.cpp and Sensor.cpp following error was seen, "fatal error {Unknown expression after the reported line}" , after changing count from 254 to number of bytes of data in read method, this issue was resolved. Couldn't recreate this issue.

This PR passes clang-tidy and clang-format checks.
For records-keeping:
This project is licensed under Apache License v2.0 for any software, and Solderpad Hardware License v2.1 for any hardware - do you agree that your contributions to this project will be under these licenses, too? - Yes
Were any of these contributions also part of work you did for an employer or a client? - No
Does this work include, or is it based on, any third-party work which you did not create? - No


Installing Remove Games 4GB error

During setup errors occur. (see screenshot)

Deactivated the menu step for ocs menu..
I guess I'm missing something (user packages?)

manually creating Remove-Game-Entries folder in the right place fixed that problem.

It's my mistake. I had forgotten to properly rename a directory for EAB WHDLoad Games OCS Menu package.
It's fixed now and will be part of v1.2.0-BETA3


[KOGITO-8666] - Update deps to podman 4.4.x

See:https://issues.redhat.com/browse/KOGITO-8666
Signed-off-by: desmax74<EMAIL_ADDRESS>
Description of the change:
Motivation for the change:
Checklist

[ ] Add or Modify a unit test for your change
[ ] Have you verified that tall the tests are passing?

Closing since we are moving to https://github.com/kiegroup/kogito-serverless-operator


AL: Go! does not set platform in app.json

1. Describe the bug
AL: Go! does not set the platform property according to the target platform.
2. To Reproduce
Steps to reproduce the behavior:

Hit F1, then AL: Go!.
Choose the latest target platform version, e.g. 12.0.
Open the app.json and check the platform property.

3. Expected behavior
The platform property should be set dynamically, just as the application and runtime already are.
See also example from standard code: version <IP_ADDRESS> is used.
4. Actual behavior
App.json:
  "platform": "<IP_ADDRESS>",

Or maybe there is a good reason to choose 1 as the minimum value?
The docs are totally unclear about which value to choose (the Symbol link does not point to the right content, it seems):

5. Versions:

AL Language: 12.5.914975

It seems the docs article above is supposed to forward to this link: https://learn.microsoft.com/en-us/dynamics365/business-central/application/system/module/system. I think I know now how to fix the docs ... However, the AL: Go! issue is still open ;-)

We have internally discussed this issue and have decided not to accept this issue to be fixed at the moment to prioritize other issues. Thank you for raising this concern.
If you do need the newer version of the platform, then you can specify that.


CB-11091 added plugins release blog post

It might be helpful to mention that cordova-plugin-compat is replacing the PermissionHelper.java file that was duplicated in all the plugin repos. There were a few plugin authors who have already copied that file from the core plugins and they should probably update to using cordova-plugin-compat.

LGTM.


refactor(applevideotoolbox): recreate decoder when kVTVideoDecoderMal…

…functionErr
Signed-off-by: pingkai<EMAIL_ADDRESS>

Codecov Report

Merging #628 (47ff0aa) into develop (6e81ff5) will increase coverage by 0.10%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #628      +/-   ##
===========================================
+ Coverage    66.90%   67.01%   +0.10%
===========================================
  Files          230      230
  Lines        12883    12883
===========================================
+ Hits          8620     8634      +14
+ Misses        4263     4249      -14

Impacted Files
Coverage Δ

framework/demuxer/avFormatDemuxer.cpp
79.48% <0.00%> (-2.06%)
:arrow_down:

framework/data_source/curl/curl_data_source.cpp
83.58% <0.00%> (-1.50%)
:arrow_down:

mediaPlayer/SMPMessageControllerListener.cpp
74.23% <0.00%> (-0.24%)
:arrow_down:

mediaPlayer/media_packet_queue.cpp
73.33% <0.00%> (ø)

mediaPlayer/SuperMediaPlayer.cpp
73.58% <0.00%> (+0.51%)
:arrow_up:

framework/demuxer/play_list/HLSStream.cpp
65.00% <0.00%> (+0.83%)
:arrow_up:

framework/demuxer/play_list/SegmentList.cpp
80.24% <0.00%> (+1.23%)
:arrow_up:

framework/demuxer/play_list/SegmentTracker.cpp
76.03% <0.00%> (+2.06%)
:arrow_up:

mediaPlayer/player_msg_control.cpp
76.80% <0.00%> (+3.20%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #628 (47ff0aa) into develop (6e81ff5) will increase coverage by 0.10%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #628      +/-   ##
===========================================
+ Coverage    66.90%   67.01%   +0.10%
===========================================
  Files          230      230
  Lines        12883    12883
===========================================
+ Hits          8620     8634      +14
+ Misses        4263     4249      -14

Impacted Files
Coverage Δ

framework/demuxer/avFormatDemuxer.cpp
79.48% <0.00%> (-2.06%)
:arrow_down:

framework/data_source/curl/curl_data_source.cpp
83.58% <0.00%> (-1.50%)
:arrow_down:

mediaPlayer/SMPMessageControllerListener.cpp
74.23% <0.00%> (-0.24%)
:arrow_down:

mediaPlayer/media_packet_queue.cpp
73.33% <0.00%> (ø)

mediaPlayer/SuperMediaPlayer.cpp
73.58% <0.00%> (+0.51%)
:arrow_up:

framework/demuxer/play_list/HLSStream.cpp
65.00% <0.00%> (+0.83%)
:arrow_up:

framework/demuxer/play_list/SegmentList.cpp
80.24% <0.00%> (+1.23%)
:arrow_up:

framework/demuxer/play_list/SegmentTracker.cpp
76.03% <0.00%> (+2.06%)
:arrow_up:

mediaPlayer/player_msg_control.cpp
76.80% <0.00%> (+3.20%)
:arrow_up:

Continue to review full report at Codecov.

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


579 add annotation to the task configuration entries for bibstransformer

Codecov Report

Merging #592 (cdb728e) into main (27efd8c) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

:exclamation: Current head cdb728e differs from pull request most recent head e167ef6. Consider uploading reports for the commit e167ef6 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #592      +/-   ##
==========================================
- Coverage   69.77%   69.77%   -0.01%
==========================================
  Files          97       97
  Lines       10422    10421       -1
  Branches     1243     1243
==========================================
- Hits         7272     7271       -1
  Misses       2865     2865
  Partials      285      285

Flag
Coverage Δ

unittests
69.77% <100.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...igration_tools/migration_tasks/bibs_transformer.py
67.30% <100.00%> (-0.62%)
:arrow_down:

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

@fontanka16 I'm going to go ahead and merge these two PRs and create an issue to fix the files description in both and deal with it later.


Add draft 2020 geo_lookup and standardize transitional

Replacing with lookup created using draft Bytes shapefiles.
#108

@mgraber Let's wait on this till we get finalized bytes shapefiles?

closing, in favor of #214


Refactor Client

Hi, @doorbash

Please provide additional param to "close" client connection - "leaveRoom". Default value must be false (maybe useful while android lifecycle pause/rotate - only disconnect). If developer manully call "close(true)" - additional action is leave from joined room. http://prntscr.com/mm3zme . My use case is singleton client and I am providing state - reset client http://prntscr.com/mm3veu. So I have "connect" and "stop" functions. It is useful when user call more times dialog create/connect game - show dialog (connect to server: create room or get available rooms), close dialog (disconnect from server with leave room).
Please Client "connect" function change to public and remove from constructor. I have some overhead for using "connect" function now.
Please add new function "setListener" to Client class

Thanks for help.

Hi @daimonkor I could do these things specially number 2 and 3 in your list but I tried to make it look more like the official Colyseus clients. see colyseus-unity3d/.../Client.cs and colyseus.js/.../Client.ts for example.
You can always fork the repo and change it the way you like.
Thanks for your suggestions.

Hi, @doorbash What do you think about my list number 1? I mean singleton with "connect" and "stop" functions?

I don't know much Kotlin but I see you're extending Client class and invoking supper class with reflection. You can remove room field in your class and use client.rooms instead to make it support multiple rooms. You should clone the repository and do num2 and 3, then you can call Client.connect and add setListener without using Reflection.

@doorbash ok. thanks

@doorbash What do you think about URL encoding room connection options http://prntscr.com/mm8nll . I got crush when using invalid uri symbols. http://prntscr.com/mm8o8v http://prntscr.com/mm8orl
I don`t know how it is valid, maybe auto encode to UTF-8 while send message and auto decode at server

ok I think that's a bug and I have to fix that. nice catch @daimonkor  👍

@doorbash Thanks a lot, the same bug found at C# and js versions

No i've just tested js version. it's fine with using whitespace in options

ws://<IP_ADDRESS>:3300/j5uc9XzVX?colyseusid=xKVTeTs-u&create=true&name=Room 17/02/2019 16:18:49&password=12121213213&requestId=3

Yes I know using whitespace character in client.options , client.id and room.options cause a java.net.URISyntaxException

@doorbash I am trying create like private rooms

@daimonkor Just released v1.2 to resolve this. please check if it is fixed, thanks

@doorbash I created fork from your repo. I think I will create pull request today.


JOSS review: Clean up interface to work in light mode and fit within Notebook

As pointed out by @jhagerer in https://github.com/openjournals/joss-reviews/issues/6873#issuecomment-2299852874, the interface doesn't render effectively in a light mode themed Jupyter instance, and is squished within Jupyter Notebook cells (as opposed to Jupyter Lab.)
Copying the example image over here for reference:

Hey @WarmCyan , in case to speed up the review process, an easy quick win could also be achieved as follows: Just change the documentation by telling that you need to activate dark theme for Jupyter Lab. You can refer to the following link:
https://stackoverflow.com/a/46219796/776847

As it turned out solving the theming issue was much easier than making it fit in Jupyter Notebook. The interface should now be usable in both light-theme and dark-theme Jupyter Lab:

Note that this required an update to our underlying ipyanchorviz package, so your environment will need to be updated with pip install --upgrade ipyanchorviz
I wasn't able to find a good solution for making it fit correctly inside a Notebook cell -- the space is heavily constrained and there's not an easy way of detecting or overriding this. For now, I'm going to update the documentation to state that this is intended for use only in Jupyter Lab

I can confirm it is working. You can close the ticket.

I can confirm that this issue is solved now. Please close.


Add ALB for DPS QA web application

This change introduces an application load balancer and related resources to permit access to the DPS QA web application.

CI: IaC Security warnings found!

CI: IaC Security warnings found!

CI: IaC Security warnings found!

CI: IaC Security warnings found!


Ping

Hi good job.
I'm making a connector with Laravel 6 to take products from odoo, but in some case the server is down, how can make with Laravel request send a ping to odoo server and validate is online??

Hey @yashiroiori ,
There is not ping method in the package but you may try with any of the following methods:
// If not logged in
$version = $odoo->version();

// Once you're logged in
$userId= $this->odoo->getUid();
or
$can = $odoo->can('read', 'res.partner');

Although they are not meant for ping purposes, they are the lighter actions and you could achieve what you need.

Hey @yashiroiori ,
There is not ping method in the package but you may try with any of the following methods:
// If not logged in
$version = $odoo->version();

// Once you're logged in
$userId= $this->odoo->getUid();
or
$can = $odoo->can('read', 'res.partner');

Although they are not meant for ping purposes, they are the lighter actions and you could achieve what you need.

Tankh you very much


Verbose log with -vU or -v ?

Checklist

[X] I'm asking a question and not reporting a bug/feature request
[X] I've looked through the README
[X] I've read the guidelines for opening an issue
[X] I've searched the bugtracker for similar questions including closed ones

Question
Why are we instructed to submit the verbose log using the -vU rather than the -v option? The reason I ask is that the former will not show much of the download log that gives much information  about the problems encountered.
Verbose log
No response

in case you have not downloaded the latest version of yt-dlp the -Uv command updates it, but if you have already downloaded the latest version just use the -v option, at least that's how I see it.

-v is fine if u know what u r doing. -U helps avoid the common mistake of being out of date


Use manage_db.sh init to create the database on first boot

The create_db.sh script was removed from 22.05 and is replaced with manage_db.sh init.

job-init-db.yaml also needs to be updated.

They are the same file. It was renamed in 4e33306cd04fcdbc1d7d311c09c52bb688275607


open with choose popup does not show when touch print button in the webpage

[x] I have read the Getting Started section
[x] I have already searched for the same problem

Environment

Technology
Version

Flutter version
3.22.0

Plugin version
flutter_inappwebview: ^5.6.0+2

Android version
android 9 and newer

iOS version

macOS version

Xcode version

Google Chrome version
128.0.6613.88 , it works well when loading the page on chrome browser in the same phone

Device information:
Description
Expected behavior:
when touch the print button on the webpage, the choose popup shoud be displayed
Current behavior:
nothing happen
Steps to reproduce
i tested with sample web page 'https://davbfr.github.io/dart_pdf/'
if you access the page on webview, you can see print button on the left bottom
touch the print button, there is no choose popup

This
Than that
Then

Images
Stacktrace/Logcat

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."


Kibana plugin banner

Hello guys,
Is there a way to install sort of info banner on ElastAlert page ?
Will explain:
We noticed that if somebody creates an alert without any configuration inside (just a name) - all alerting stops and sometime elastalert even crashes. So I would like to get people to pay attention to it.
Cheers.

Hi,
Banner is just a info that somebody can ignore.
It would be better to add validation but there is many cases that can go wrong e.g.

like you said empty rules
invalid yaml structure
invalid rule
and so one

I don't have plan to do that.
If you need this banner you could fork this repo, modify some html, build and use it ;)
Best regards,
Mateusz


Support command line install from ~/Applications

Allows /usr/local/bin/keybase to be linked from ~/Applications/Keybase.app.
Also cleaned up some logging and added blurb about removing the Keybase app when someone does a keybase uninstall.

@patrickxb ?


Character limit on tokens & environment variables

Describe the bug
I noticed that there are hard limits on character counts for tokens and environment variables. If you put anything over 704 characters either to a variable in environment variables or directly in the token field, client just removes it without any warning.
To Reproduce
Steps to reproduce the behavior:

Go to 'Manage Environments' section.
Put in a value longer than 704 characters
Close the environments window and reopen it again.
The value is removed by the client.

Expected behavior
Longer values can be saved.
Screenshots
https://drive.google.com/file/d/1-DxZGBm14u869Xbx01EaGQSBKOMzyTic/view
Desktop (please complete the following information):

OS: [ubuntu linux 20.04]
Installation Method: [snapd]
App Version [2020.3.0]

Installed the latest version from the repo instead of snapd and I couldn't replicate this bug.
Closing since I am guessing this is either fixed in the latest version or its something related with snapd sandboxing.


BCrypt InvalidHash when resetting passwords

I'm new to Rails, coming from a PHP + Python background. I'm setting up a development environment on my machine based on a production database dump.
I can't log in to my dev environment because every user sign in fails with a BCrypt InvalidHash error.
I checked the users.encrypted_password hash against my password on an online BCrypt website--it matches correctly.
I wrote a quick script to reset all the passwords to see if there's some weird configuration difference between my Fedora machine and the prod Heroku config:
User.find_each do |user|
  user.password = 'password'
  user.save
end

But that's still failing with the same InvalidHash error, stack trace below:
/home/eric/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/bcrypt-3.1.11/lib/bcrypt/password.rb:60:in `initialize': invalid hash (BCrypt::Errors::InvalidHash)
	from /home/eric/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/bcrypt-3.1.11/lib/bcrypt/password.rb:46:in `new'
	from /home/eric/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/bcrypt-3.1.11/lib/bcrypt/password.rb:46:in `create'
	from /home/eric/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/clearance-1.16.1/lib/clearance/password_strategies/bcrypt.rb:28:in `password='
	from /home/eric/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/clearance-1.16.1/lib/clearance/user.rb:111:in `password='
	from db/reset_passwords.rb:9:in `block in <top (required)>'

clearance.rb:
Clearance.configure do |config|
  config.rotate_csrf_on_sign_in = true
  config.password_strategy = Clearance::PasswordStrategies::BCrypt
end

What am I doing wrong? It seems like my dev environment is missing some kind of config.

:man-facepalming::skin-tone-2:
https://github.com/codahale/bcrypt-ruby/issues/170
Clearance had brought in bcrypt-3.1.11, which doesn't work on Fedora 28+. Updating to bcrypt-3.1.12 fixes the InvalidHashes.


Devloper's Bussiness Webapp

https://github.com/Kritika30032002/ReactCreations/assets/114663382/8b144777-861b-4945-aff9-ac21910402c3
I am participating in Codepeak and KWoC.

in which program you want this to be considered?

Okay So consider it for Codepeak

@Kritika30032002 Can I work on this issue under Codepeak-23?

Its done already.. I think

Ohk


Feedback: Outlay

This is the place to leave your feedback and questions about the following data element:

Outlay

The feedback period for Outlay closes on 4/29/2015.
The proposed definition of Outlay: http://fedspendingtransparency.github.io/whitepapers/outlay/
Federal Spending Collaboration home page: http://fedspendingtransparency.github.io/

AGA established a work group charged with responding to inquiries by the federal government, regarding implementation of the DATA Act. The following response is being submitted on behalf of AGA's work group, which is comprised of state, local, tribal and university practitioners from across the country.
The proposed definition of "outlay" does not address a number of issues applicable to entities receiving federal funds, including:
•	whether a cash disbursement includes the use/disbursement of certain funds, such as program income by the recipient or payments made in advance of when the activity related to the award occurs;
•	whether a federal agency’s "outlay" includes program income collected and later disbursed by the recipient;
•	whether a federal agency should report an "outlay" once cash is disbursed or wait until cash is disbursed by the recipient at the time the activity related to the award occurs; and
•	when an "outlay" occurs for pre-issuance states, which draw Federal funds in advance of need, in accordance with a Treasury State Agreement (TSA).
The proposed definition of ""outlay"" also demonstrates the importance of listing all federal financial management definitions in one place. For example, "outlay" and "Determining Federal awards expended," as provided for in OMB's “Uniform Administrative Requirements, Cost Principles and Audit Requirements for Federal Awards" (Uniform Guidance), are closely related and will have an impact on entities receiving federal funds. Among other things, the Uniform Guidance addresses the issue we raised, above, dealing with the receipt or use of program income:
Excerpt from the Uniform Guidance:
§ 200.502 Basis for determining Federal awards expended.
(a) Determining Federal awards expended. The determination of when a Federal award is expended should be based on when the activity related to the Federal award occurs. Generally, the activity pertains to events that require the non-Federal entity to comply with Federal statutes, regulations, and the terms and conditions of Federal awards, such as: expenditure/expense transactions associated with awards including grants, cost-reimbursement contracts under the FAR, compacts with Indian Tribes, cooperative agreements, and direct appropriations; the disbursement of funds to subrecipients; the use of loan proceeds under loan and loan guarantee programs; the receipt of property; the receipt of surplus property; the receipt or use of program income; the distribution or use of food commodities; the disbursement of amounts entitling the non-Federal entity to an interest subsidy; and the period when insurance is in force.

@HelenaSims  outlines details that I rarely come into contact with at the operational level on the Federal side of the ledger so to speak.  I'd interpret "expended" phase of the transaction as equivalent to "work performed and invoiced" but not necessarily paid.  It is important to keep a wall between "expenditure" and "outlay" otherwise the user base will get confused very quickly.  I'm hazy at the moment regarding the specific SGL codes that associate to outlay but I know that the "paid" and "unpaid" description is key to performing a compilation or reconciliation.  Our FAQ should keep it simple - that an outlay is the same as "payment" against an obligation, and some call payment a "disbursement".

The white paper on the data element makes many pertinent observations as to definitions -- among them that outlays can occur immediately or with a lag -- and desirable recommendations regarding business processes.  Where we are going to encounter the real problem is -- at least in my neck of the woods -- the granularity of outlay information compared to obligations.  Not all systems seem to keep transactional level payment data -- or maybe the roadblock is limited reporting features.  I believe this is what the 2nd paragraph of the Recommended Business Process was alluding to.  If so, the stated need to "design processes to combine information from multiple systems and/or modify the source system" is not something that is going to happen absent a huge investment of time and re-engineering -- perhaps more than we can afford dollar-wise or can accept as a delay time-wise to make the compliance deadlines or milestones.  While not a desirable approach in terms of ultimate accuracy, perhaps as an interim measure there could be consideration of crude (but easily explained) "transformation" pro-ration techniques to allocate a consolidated payment against multiple obligation lines.  Maybe this is not a problem for other Federal agencies.....

Thank you for your feedback on the data element Outlay. We will consider your feedback as we continually assess how we can improve data definitions. You may still contribute the discussion for other data elements. Check http://fedspendingtransparency.github.io/dataelements/ to find data elements with open feedback pages.


User-programmable shaders for DisplayObjects/Bitmaps

As the foundation for this has basically been introduced to lime "next" ( https://github.com/openfl/lime/issues/47 ), it would be great to have a simple to use interface in OpenFL that lets users use their own fragment shaders on a DisplayObject level.
Vertex shaders would be nice, too, but IMO optional as we're mostly talking about 2D here.
I am talking about a level of simplicity like this:
var shaderEffect :ShaderEffect = new ShaderEffect("assets/shaders/myEffect.frag");
shaderEffect.awesomeConfigurationOption = true;
myDisplayObject.shaderEffect = shaderEffect;

I am unsure if this is possible on a DisplayObject level, but it should be possible on a Bitmap level.

Has any work been done by now that would make implementing shader effects like these easier?

Pinging @mrcdk

https://github.com/openfl/openfl/pull/697

There we go :)

Great to see that!
I hope it can be mergen in eventually :)

Seems like @mrcdk stopped working on that.
Too bad, it seemed very promising!

I haven't, it's done. There are a few problems. The first one is that the
whole structure of the lib has changed (at least the parts that my
implementation cares about) so merging it will be a nightmare. The other
problem is that it needs a few fixes that I committed and were reverted
because they were failing in a particular case. I'm waiting for them to be
merged back.
So, once that last problem is solved, I'll see if I have time to make the
PR mergable again.
El 25/8/2015 14:52, "TheSHEEEP"<EMAIL_ADDRESS>escribió:

Seems like @mrcdk https://github.com/mrcdk stopped working on that.
Too bad, it seemed very promising!
—
Reply to this email directly or view it on GitHub
https://github.com/openfl/openfl/issues/303#issuecomment-134575334.

Hey @mrcdk, happy to help facilitate stuff. What are your pain points at the moment? I'm happy to break this down into a series of steps that we can try to push through in the next few weeks and see if Joshua, et al. can help figure it out.

FYI, the original #697 was closed, and the reincarnation #874 was merged. Perhaps we should close this issue now?


Christmas Entry - Regression based model

New model and feature set. Results in 'regressor_results.csv' labelled as 'Facies'

Hi George. Thanks for this. I'm afraid this one only managed 0.345. The feature engineering, especially the creative pandas tricks, are interesting. I wonder if the regressor model (which I haven't used myself) is even easier to overtrain than the classifier? Did you try these features with the classifier? Feel free to send another over. And merry Christmas!


Update Setup

I'm going to make some updates to the setup file as it still has Greg's name and information in at the moment

okay cool


Add features as described in Issue 37

Issue Link: https://github.com/PhonePe/mantis/issues/37

Flag for listing org

python launch.py list --list-orgs
python launch.py list -l

List domains for org

python launch.py list --list-domains --org "org1" --org "org2" --tlds # list TLDs for multiple orgs
python launch.py list --list-domains --org "org1" --org "org2" --subs # list subdomains for multiple orgs
python launch.py list --list-domains --org "org1" --org "org2" --tlds --subs # list TLDs + subdomains for multiple orgs

python launch.py list -d -o "org1" -o "org2" -t # list TLDs for multiple orgs
python launch.py list -d -o "org1" -o "org2" -s # list subdomains for multiple orgs
python launch.py list -d -o "org1" -o "org2" -t -s # list TLDs + subdomains for multiple orgs

Time based filter

python launch.py list -tds -o "org1" -a 2024-09-04 -b 2024-09-05 # list TLDs + subdomains for org1 in timeframe [2024-09-04, 2024-09-05]

[!NOTE]
-a/--after and -b/--before filters can be used individually as well

Checklist:

[ ] Merge https://github.com/PhonePe/mantis/pull/49
[ ] Merge main branch into dmdhrumilmistry:issue-37

Hi @dmdhrumilmistry
Thanks for raising the pr. I will review this and request changes if required any.

Hi @dmdhrumilmistry Thanks for raising the pr. I will review this and request changes if required any.

Sure. Sg!

@dmdhrumilmistry Thanks for the PR. This looks very good. I tested it and everything works as expected.
Quick question, any reason to use tds in list -tds -o "org1" -a 2024-09-04 -b 2024-09-05 instead of tlds?
Also, when i run mantis list -tds -o "sample-org" -a 2024-01-01 -b 2024-02-02, the time based filter works but I get all the subdomains. When i use tds my assumption was that I'll get only TLDs.

@dmdhrumilmistry Thanks for the PR. This looks very good. I tested it and everything works as expected.
Quick question, any reason for using tds keyword in list -tds -o "org1" -a 2024-09-04 -b 2024-09-05 instead of tlds?
Also, when i run mantis list -tds -o "sample-org" -a 2024-01-01 -b 2024-02-02, the time based filter works but I get all the subdomains. When I use tds, my assumption was that I'll get only TLDs.

sg, actually tds are different flags -t -d -s. I've just added it for demonstration that they can be clubbed together as well (-tds)

Flag
Description

t/tlds
includes tlds in response

s/subs
includes subdomains in response

Makes sense.
@Prateek-Thakare I tested the functionality. It's good to go. Can you go through the code and merge the PR?

Hi @dmdhrumilmistry,
Can you please remove the commits for the Ansible feature from this PR and re-raise it?
Thanks,
Prateek

Hi @dmdhrumilmistry, Can you please remove the commits for the Ansible feature from this PR and re-raise it?
Thanks, Prateek

Hey @Prateek-Thakare,
I've removed ansible files. LMK if any other changes are required

@Prateek-Thakare / @0xbharath thanks for the merge 🙌. Can someone add hacktoberfest-accepted label to this and https://github.com/PhonePe/mantis/pull/49 PRs?

Thanks for the contribution. I have added the labels to the PRs. Looking forward to more such contributions.


E: Package 'libvirt-bin' has no installation candidate

Ubuntu 20.04, i get this error "E: Package 'libvirt-bin' has no installation candidate" following the installation guide from here https://github.com/dahlia-os/documentation/blob/master/how to run dahila/How To Run DahilaOS EN.md#ubuntu

that is correct instead of libvirt-bin use libvirt-daemon-system libvirt-clients
updated command
sudo apt-get install qemu-kvm qemu virt-manager virt-viewer libvirt-daemon-system libvirt-clients
-hexaone

ok, thanks :D


Issue #27 Upgrade jgroups config to be compatible with jgroups 5

Also updated tcnative version
Thanks for submitting your Pull Request!
Please make sure your PR meets the following requirements:

[ ] Pull Request title is properly formatted: [CLOUD-XYA] Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket
[ ] Attached commits represent units of work and are properly formatted
[ ] You have read and agreed to the Developer Certificate of Origin (DCO) (see CONTRIBUTING.md)
[ ] Every commit contains Signed-off-by: Your Name<EMAIL_ADDRESS>- use git commit -s

@brusdev updated.


Missing file "resources/growth_scores_gene_sets.gmt"

Hi,
I think a file is missing here, either it was not added to the repo or it was moved and its
location was not updated.
The issue occurs when running
wot gene_set_scores --matrix <matrixfile>

In wot/commands/gene_set_scores.py, on line 55,
the gene_sets variable defaults to 'wot/resources/growth_scores_gene_sets.gmt'.
However, that file is not present, which triggers a FileNotFoundError at runtime.
Thanks !

Added the file.


White stripe ghost

Hi. I am using the Swiper on horizontal mode and had no issues on iPhone, iPad and Androids Tablets.  However, on Android phones, I get this strange behaviour:
Load one, two, three, four slides going back(swipe left) and forward(swipe right) no issue. However from slide 15 onward, I get a flicker white stripe on the direction I am navigating that fills 10% to 50% of the screen for a fraction of second. Now when I do back (left) works fine till the 8th slide from there below, the same white stripe glitch, now on the left, I turn back right again and it work fine 8 slides forward....
Is this a memory issue? I have tried with different memory free and it always starts at slide 15
Thanks for any clue, and thanks for the script!

I wonder if you can easily modify the example of @manuelpaulo for freemode. I have a slideshow with freemode enabled and lots of images (50+). Instead of preloading all the images a lazy loading solution would be great that appends additional images at the end.
I experimented with onSetWrapperTransform instead of onSlideChangeEnd but it looks not trivial to compute the correct solution if there are images with different widths.
Any ideas? :)


Patch version <IP_ADDRESS> breaking changes (API & behavioral)

Driver version
Upgrade from <IP_ADDRESS> to <IP_ADDRESS>
Redshift version
PostgreSQL 8.0.2 on i686-pc-linux-gnu, compiled by GCC gcc (GCC) 3.4.2 20041017 (Red Hat 3.4.2-6.fc3), Redshift 1.0.50708
Client Operating System
Ubuntu 22.04.2 LTS
JAVA/JVM version
openjdk 11.0.17 2022-10-18
OpenJDK Runtime Environment Temurin-11.0.17+8 (build 11.0.17+8)
OpenJDK 64-Bit Server VM Temurin-11.0.17+8 (build 11.0.17+8, mixed mode)
Problem description
When upgrading from the driver version <IP_ADDRESS> to <IP_ADDRESS>, there was a breaking change: the method RedshiftProperties#parseUrl() changed signature from (String, java.util.Properties) to (String, RedshiftProperties) (see https://github.com/aws/amazon-redshift-jdbc-driver/commit/7c807cd2cf1ba7ee5116d2d610f2daf324df3767#diff-9cac02d84c6529eb62db852fd9a951573b50ca9ac28640aeafbe34257f96a6a0R568).
These change was breaking but even after applying the required changes, I stumbled upon another issue.
There were some changes related to the case insensitivity, but those changes are not consistent with the java.sql.Connection methods:
    package my.packagename.name;

    import static org.assertj.core.api.Assertions.assertThat;

    import com.zaxxer.hikari.HikariConfig;
    import com.zaxxer.hikari.HikariDataSource;
    import com.amazon.redshift.util.RedshiftProperties;
    import java.sql.Connection;
    import java.sql.SQLException;
    import org.junit.jupiter.api.Test;

    @Test
    void testRedshiftDriver() throws SQLException {
        final String connectionString = RedshiftTestConstants.TEST_CONNECTION_STRING;

        final String expectedAppName = "an elegant and edgy name";
        final RedshiftProperties prop = new RedshiftProperties();
        prop.setProperty(RedshiftProperty.APPLICATION_NAME.getName(), expectedAppName);
        prop.setProperty(RedshiftProperty.PASSWORD.getName(), RedshiftTestConstants.getRedshiftPassword());

        final HikariConfig config = new HikariConfig();
        config.setJdbcUrl(connectionString);
        config.setDataSourceProperties(prop);
        final HikariDataSource dataSource = new HikariDataSource(config);

        final Connection connection = dataSource.getConnection();

        assertThat(RedshiftProperty.APPLICATION_NAME.getName())
                .isEqualTo("applicationname"); // mismatch with following statement
        assertThat(connection.getClientInfo())
                .hasSize(1)
                .containsEntry(
                        "ApplicationName", // mismatch with previous statement
                        "an elegant and edgy name");

        assertThat(connection.getClientInfo(RedshiftProperty.APPLICATION_NAME.getName()))
                .isEqualTo("an elegant and edgy name"); // failure, actual is null

    }

I was surprised to see such a behavioral change (in addition to the API change) in a patch version.
Also, what is the correct way from now on to retrieve connection info through java.sql.Connection.getClientInfo ?
Thanks in advance.

@NathanEckert thanks for submitting this issue.
Could you please elaborate how/why are you calling the parseUrl() method?

So I realized following-up on that question that RedshiftProperties#parseUrl() is in a util package and that we should maybe not call it direclty but use it through the more public API Driver#parseUrl(), however this method has the same API breaking change.
Our usage is do analyze the provided connection string to disallow certain properties that our application will override to avoid to confuse users:
	private static void rejectApplicationFromConnectionString(final String connectionString) {
		final Properties parameters = Driver.parseURL(connectionString, new Properties());
		if (parameters.containsKey(RedshiftProperty.APPLICATION_NAME.getName())) {
			throw new IllegalStateException("Redshift Application cannot be defined in the URL");
		}
	}

Hello @bhvkshah,
Any update on this issue ?

Hi @NathanEckert sorry for the delay! I'll get back to you soon as I have an update on this!

Hello @bhvkshah,
Any update on this issue ?

Thanks for the suggestion, I was able to upgrade !


docs nav parents no longer auto-expand when you interact with them

previously when you interacted with a parent it auto-expanded and displayed that menus children
that no longer seems to be the case
may have been impacted by https://github.com/pulumi/pulumi-hugo/pull/2848

you have to interact a number of times to get this to break it seems


Walter's activity: Complete a research project

At the close of the mission One Small Step (or the beginning of the mission The Old Neighborhood), you are encouraged to speak with the members of Constellation present at the Lodge. During the conversation with Walter he states: "I'm a fan of self reliance...", following which you are given two Activities: "Craft or modify an item" and "Complete a research project". The latter cannot be completed or removed if taken during an NG+ once all research has been completed.
This is minor, to be sure, but seems to be a bug, even if just for those of us who detest lingering, uncomplete-able tasks in our mission lists. I'm hoping this team agrees, and might see if there is a means to clear that activity when all research has previously been completed. Thank you for your work and consideration.

Game Version
<IP_ADDRESS>
Language
English (EN)
New Game
Yes
Reproduction Steps

Complete all research
Start NG+ into 'default' universe
In initial dialogue with Sarah, choose: [Repeat Main Quest] I want to join Constellation...
Speak with Walter through two dialogue choices until he says, "I'm a fan of self reliance, so I encourage you to make use of the tools we have to build what you need."
You will then receive both Activities, "Craft or modify an item" and "Complete a research project"

I have a fix for this.

You, Pickysaurus... you and this team are legends. Thank you =)


Prevent Xcode from crashing because of a bad branch config in manifest file

An assert would cause Xcode to crash if there is bad configuration in the manifest i.e. using the url to a pull request as if it was a branch.
Now instead we guard and throw a StringError for a more graceful exit.
rdar://60879046

@swift-ci please smoke test


Error using ReactDiffView in Remix App

Summary
I am running into an error when trying to use this library
import ReactDiffViewer from "react-diff-viewer-continued";

export const DiffExpandView = () => {
	return (
		<>
			<ReactDiffViewer
				oldValue={"sourceVersion"}
				newValue={"newVersion"}
				splitView={true}
				// linesOffset={5}
			/>
		</>
	);
}

Here is the error I get:
Error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: object.

Context
I am working with a remix-react app.
Any idea how to fix this? This component would really fit my current needs so I would love to be able to use it

@ObaidAshiq  I have updated it to remove the curly braces, but it still doesn't work
import ReactDiffViewer from "react-diff-viewer-continued";

export const DiffExpandView = () => {
	return (
		<>
			<ReactDiffViewer
				oldValue="source Version" <-- removed curly braces
				newValue="new Version"  <-- removed curly braces
				splitView={true}
				// linesOffset={5}
			/>
		</>
	);
}

I assume it has something to do with SSR somehow, but not quite sure what that would be. I'll have to boot it in a Remix app to see what effect it has.

This also happens with a basic InertiaJS + Vite using SSR https://inertia-rails.netlify.app/guide/. The component fails during SSR, and then renders correctly on the client.
Pretty tired at the moment to create a basic reproduction app, but hoping to do one soon.

Somehow browser and node is loading module differently. In node(server), if I print
import ReactDiffViewer from "react-diff-viewer-continued";
console.log(ReactDiffViewer)

I get
{
  LineNumberPrefix: { LEFT: 'L', RIGHT: 'R' },
  DiffMethod: [Getter],
  default: [class DiffViewer extends Component] {
    defaultProps: {
      oldValue: '',
      newValue: '',
      splitView: true,
      highlightLines: [],
      disableWordDiff: false,
      compareMethod: 'diffChars',
      styles: {},
      hideLineNumbers: false,
      hideMarkers: false,
      extraLinesSurroundingDiff: 3,
      showDiffOnly: true,
      useDarkTheme: false,
      linesOffset: 0,
      nonce: ''
    }
  }
}

but in browser
class extends React.Component {
      constructor(props) {
        super(props);
        this.resetCodeBlocks = () => {
          if (this.state.expandedBlocks.length > 0) {
            this.setState({
   …

I was able to fix it using <Suspense> and
const ReactDiffViewer = lazy(() => import('react-diff-viewer-continued'));

ReactDiffViewer needs to be wrapped by <Suspense>. It will prevent load of diff viewer on server side

I fixed my issue by doing:
import RDV from 'react-diff-viewer-continued'
let ReactDiffViewer
if(typeof window == 'undefined') ReactDiffViewer = RDV.default
else ReactDiffViewer = RDV

how to fix?

Same issue here.
I've used a variant of @SeungheonOh's workaround to solve the issue:
import RDV from 'react-diff-viewer-continued';

let ReactDiffViewer;

if (typeof RDV.default !== 'undefined') {
  ReactDiffViewer = RDV.default;
} else {
  ReactDiffViewer = RDV;
}


Discovery overlay for keyboard shortcuts and settings

I'd like an overlay that appears over the application with short-cuts, 'cheat codes', and settings in them so we can expand the UI features without adding more chrome to the basic model.
A little similar in spirit to ? in Gmail/Chromebook.
Lets start by when you press Ctrl + ? you get an overlay over the both 85% of the window with a list of the currently supported keyboard shortcuts.
Long term, it would be nice to have the overlay have the potential of interaction, so lets not choose anything that blocks that.
@kasperpeulen

Done by @kasperpeulen.


Crash if one permission granted and then grant second permission when try to launch camera

There is next code in onRequestPermissionsResult() for camera:
if ((grantResults.isNotEmpty() && grantResults[0] == PackageManager.PERMISSION_GRANTED
&& grantResults[1] == PackageManager.PERMISSION_GRANTED)) {
onPermissionsGranted(requestCode)
}
But in grantResults[] will be only one object and java.lang.ArrayIndexOutOfBoundsException: length=1; index=1 will be thrown.
Simple fix is check is grantResults[] has only one object.

Hi @Podox2
Thank you for reporting the issue.
I will fix it soon.


files missing in download

Hello and thanks for this tool!
I have some troubles in downloading from google drive.
I launch the following statement:
skicka download "myfolder\mysubfolder1" "c:\temp\test"
but skicka ends with error...it returns me something like:
skicka: my main folder\my sub folder\my file: skipping download of duplicate file on Drive
and I have a list of these errors also if the listed files/folders not included in my download statement...
Moreover I guess that some special character or blank space in the name/path can bring the download of the file be skipped
is it possible to fix that?
thanks, bye
marco

or at least modify the skicka command I launch to not considre those characters....

Hi,
Please paste the exact command used and the exact error messages given by skicka. Thanks!


exit with non-zero status if not all freetype deps are all builtin or not

Otherwise script building Godot may assume the build succeeded

Thanks!


Add Descriptor field

fixes #28
few questions:

How hack/* is generated? (the CRD spec and examples) ?
How the readme fields-table is generated  ?
Is the version field  going to be used by the controller (e.g to rollout apps) ?

Apologies for the delay. I think this change is desireable and this is the best time to make it :) Thanks!

I've updated the sample and documentation.
Would be nice to have them generated, at least the hack/install.yaml in the futur

/lgtm

/approve


Create account

Can ve create insta account using instagrapi

I am writing an automated account creation process

Check account - api
Check email - api
Send mail - api
Confirm email belonging-get registration code - api
Failed to invoke the creation interface.  /api/v1/accounts/create/ -400 Bad Request;   feedback_required

Version number: 2<IP_ADDRESS>.75
Do you have a partner to talk to? I am writing an automated account creation process

Check account - api
Check email - api
Send mail - api
Confirm email belonging-get registration code - api
Failed to invoke the creation interface. /api/v1/accounts/create/ -400 Bad Request; feedback_required

ins version number: 2<IP_ADDRESS>.75
Do you have a partner to talk to?
My email address is<EMAIL_ADDRESS>

I am writing an automated account creation process

Check account - api
Check email - api
Send mail - api
Confirm email belonging-get registration code - api
Failed to invoke the creation interface.    /api/v1/accounts/create/ -400 Bad Request;     feedback_required

ins version number: 2<IP_ADDRESS>.75
Do you have a partner to talk to?
My email address is<EMAIL_ADDRESS>

For me it is working.

@sunmasters What kind of agent did you use

@Nishant87555 How to use it? Can I see the code?

It is very difficult to create a valid account via Email, but with SMS using telephone APIs it is much easier. I won't go into details but the steps are as follows:

Create Phone Device:
url: 'api/v1/launcher/mobileconfig/',
method: 'POST',
form: {
"bool_opt_policy": '0',
"mobileconfigsessionless": '',
"api_version": '3',
"unit_type": '1';
"query_hash": 'b75d780f1c4cb7ae635c0c24e5dad1cb8d8d0a8c97a412f11978fb61d981d352'
"ts": current_timestamp,
"device_id": uuid
"fetch_type": 'ASYNC_FULL',
}

Check Valid Phone Number:
url: '/api/v1/accounts/check_phone_number/',
form: {
"phone_id": phone_id,
"login_nonce_map": '{}',
"phone_number": phone_number,
"_csrftoken": csrf_token,
"guid": device_uuid,
"device_id": device_id,
"prefill_shown": 'False',
}

Send SMS Code:
url: '/api/v1/accounts/send_signup_sms_code/',
form: {
"phone_id": phone_id,
"phone_number": phone_number,
"_csrftoken": csrf_token,
"guid": phone_number,
"device_id": device_id,
"android_build_type": 'release',
"waterfall_id": guid,
}

Validate SMS Code:
url: '/api/v1/accounts/validate_signup_sms_code/',
form: {
"verification_code": verification_code,
"phone_number": phone_number,
"_csrftoken": csrf_token,
"guid": device_uuid,
"device_id": device_id,
"waterfall_id": guid
}

Finish Sign Up
url: '/api/v1/accounts/create_validated/',
form: {
"is_secondary_account_creation": 'false',
"jazoest": encryptor_jazoest,
"tos_version": 'row',
"suggestedUsername": '',
"verification_code": verification_code,
"logged_in_user_id": logged_in_user_id,
"sn_result": sn_result,
"do_not_auto_login_if_credentials_match": 'true',
"phone_id": phone_id,
"enc_password": encrypted_password,
"phone_number": phoneNumber,
"username": username,
"first_name": first_name,
"day": birth_day
"adid": '00000000-0000-0000-0000-000000000000',
"guid": device_uuid,
"year": year,
"device_id": device_id,
"_uuid": device_uuid,
"month": month,
"sn_nonce": sn_nonce,
"force_sign_up_code": '',
"logged_in_user_authorization_token":logged_in_user_authorization_token,
"waterfall_id": guid_waterfall_id,
"qs_stamp": '',
"has_sms_consent": 'true',
"one_tap_opt_in": 'true',
}

@davidrodma bro have you faced this issue that instagram stopped sending otps? i am not sure the reason but yesterday it was working fine and now its not sending otps on any number i give
Already using good residential rotating proxies and numbers are also new

@davidrodma bro have you faced this issue that instagram stopped sending otps? i am not sure the reason but yesterday it was working fine and now its not sending otps on any number i give Already using good residential rotating proxies and numbers are also new

It is maybe IP problem. Try mobile proxy

I have tried my phone internet bro @sunmasters  and a mobile proxy too
Also tried 2 providers residential proxies , didn't get the OTP

Is number are temporary? Sms?

Yes those are virtual numbers but i have tried it with my personal number too , 2 of my personal numbers which are 3-4 years old

I just tried, I got code. Maybe you are getting messages to WhatsApp. Either your device or setup has issues

Not sure whats the issue here, yesterday with same script it was working , even logs says the same thing that otp sent request was successfull
2024-12-21 17:27:08,083 - instagram_signup - DEBUG - private_request 200: https://i.instagram.com/api/v1/accounts/send_signup_sms_code/ ({"autoconf_nonce_sent":false,"tos_version":"row","age_required":true,"status":"ok"})
2024-12-21 17:27:08,083 - instagram_signup - DEBUG - private_request 200: https://i.instagram.com/api/v1/accounts/send_signup_sms_code/ ({"autoconf_nonce_sent":false,"tos_version":"row","age_required":true,"status":"ok"})
2024-12-21 17:27:08,083 - private_request - INFO - None [200] POST https://i.instagram.com/api/v1/accounts/send_signup_sms_code/ (3<IP_ADDRESS>.89, samsung SM-A546B)
Do you have working script for signup through the instagrapi using mobile phone?

Not sure whats the issue here, yesterday with same script it was working , even logs says the same thing that otp sent request was successfull 2024-12-21 17:27:08,083 - instagram_signup - DEBUG - private_request 200: https://i.instagram.com/api/v1/accounts/send_signup_sms_code/ ({"autoconf_nonce_sent":false,"tos_version":"row","age_required":true,"status":"ok"}) 2024-12-21 17:27:08,083 - instagram_signup - DEBUG - private_request 200: https://i.instagram.com/api/v1/accounts/send_signup_sms_code/ ({"autoconf_nonce_sent":false,"tos_version":"row","age_required":true,"status":"ok"}) 2024-12-21 17:27:08,083 - private_request - INFO - None [200] POST https://i.instagram.com/api/v1/accounts/send_signup_sms_code/ (3<IP_ADDRESS>.89, samsung SM-A546B)
Do you have working script for signup through the instagrapi using mobile phone?

I had. But not tested. For me email signup is enough

Email signup always gives me a Instagram User which is blocked or generally taken as suspicious user, are others not facing issue on this?

Email signup always gives me a Instagram User which is blocked or generally taken as suspicious user, are others not facing issue on this?

No, my script works good. Maybe it's API problem.
Sms/web/app works for me

Is it possible we can connect or you can share the script you are using bro? @sunmasters

No, my script works good. Maybe it's API problem.

Hello @sunmasters! Can you share your script?


RuntimeError: cannot release un-acquired lock

I'm getting the above error using Python 2.7 in a multi-threaded application. Anyone know how to fix this?

Thanks for reporting this.  I assume you're working with fastcache 1.0.2?  How many threads were you running?  Can you provide an example that fails?  That would make it easier to track down the error since the multi-threads tests in the test suite pass.

Yeah, 1.0.2. Let me try to think of a simple example that fails.

I am running into this problem with Sympy + Dask all the time and it's almost impossible to reproduce.
I would love to help to debug this, but don't know where to start.

@basnijholt to debug this we need some kind of test that has a decent probability of failing.  As a starting point, we could run the multithreading test for long times (~30 min) and see if this error is triggered.  For an immediate fix to your problem: if you remove fastcache, sympy should use the lru cache from the standard library.  That was rewritten in C in 3.5 so there shouldn't be a performance penalty if you are using 3.5 or above

Excellent!  What python version is this?

concurrent.futures is only available in Python 3.

Indeed, I use Python 3.6.

We're using Sympy 1.2 and have encountered this issue and it's somewhat of a showstopper for us due to random crashing.  I've looked at the code a bit but I'm not very good at using the Python capi, especially the threading side.  I don't quite understand this around src/_lrucache.c:33:
// global variables for rlock - only modified when holding lock
static long rlock_owner = 0;
static unsigned long rlock_count = 0;

static int
rlock_acquire(PyThread_type_lock lock)
{
    long tid;
    PyLockStatus r;

    tid = PyThread_get_thread_ident();
    if (rlock_count > 0 && tid == rlock_owner) {
        unsigned long count = rlock_count + 1;
        if (count <= rlock_count) {
            PyErr_SetString(PyExc_OverflowError,
                            "Internal lock count overflowed");
            return -1;
        }
        rlock_count = count;
        return 1;
    }
    ... more lines ...

and:
static int
rlock_release(PyThread_type_lock lock)
{
    long tid = PyThread_get_thread_ident();

    if (rlock_count == 0 || rlock_owner != tid) {
        PyErr_SetString(PyExc_RuntimeError,
                        "cannot release un-acquired lock");
        return -1;
    }
    ... more lines ...

From the code it looks like the rlock_owner and rlock_count are used as an re-entrant lock mechanism.  If that's the case, they seem like they are global variables instead of having a pair of variables per lock.  Can multiple threads change those variables at the same time?  In particular, how would it work when two threads enter this function in the following sequence:
Thread1: acquire lock 1, (set rlock_count = 1 and  rlock_owner=thread1_tid)
Thread1: do things in python (may switch context and run another thread)
Thread2: acquire lock 2, (set count = 1, and rlock_owner=thread2_tid?)
Thread2: do things in python (switch context again)
Thread1: tries to unlock lock 1 (would raise the exception since rlock_owner is not thread2_tid?)

Is that scenario possible?


Update troubleshooting.md

Add description as to what 'sticked on revision' means, as the nf-core documentation is generally more accessible by users (than the nextflow docs which is more developer focused), even if it's a nextflow error.
Suggested for nf-core website here: https://github.com/nf-core/eager/pull/386#issuecomment-616419347

Looks good! Is this a new Nextflow feature? I don't remember seeing this message before.
"Sticked on revision" could maybe do with rewording a little...

Nope, been there ever since I've been using Nextflow (from 19.04 I think). I guess you're too much of a nextflow master to encounter these errors ;)

That or I haven't really been running any analysis since about v0.32 😆


request: add line: tag to ctags output

A popular vim addon tagbar that uses ctags doesn't support reading line numbers from the standard position of ctags, only when it's put in the line: field.
It would be nice if dscanner output is compatible with tagbar, as the output is much better than the patched for D ctags.

The format of the ctags output should be pretty easy for you to modify. In the mean time, you can get an outline of a source file by using the --outline option.


Flatiron refactor

As per commit messages.
Only one API change was made: Monitor no longer needs options.forever - when options.max is not set, forever acts just like options.forever was specified (I'm fine with leaving this commit out tho).
Please help with testing it.

@mmalecki I'm seeing failed tests here: https://gist.github.com/1383636

@indexzero Can you try rm -rf ~/.forever && killall node and applying https://gist.github.com/3441cfe6a310a4987592 (patch -p1 < monitor.patch in forever source tree)?
This patch is included in #163.

A little bit better, but still one failing test. https://gist.github.com/1383636

Cherry-picked. Win!


Adding Travis-CI integration

Added the required .travis.yml configuration file to build on Linux using sufficiently new versions of gcc or clang. You can enable this in GitHub integrations and services to run CI tests on all commits/PRs.
This can also be extended to get code coverage numbers from the tests run.
Please let me know if there are more changes required from my side.
Thanks.

Great ! Travis is definitely something we need for iod.
Just one little thing before I merge this PR: Could you remove the build/.gitignore and instead create the build dir in the travis script ?


Handle types in the plugin spec

The current spec definition is:
type PluginSpec struct {
	Files      []string `yaml:",flow"`
	Labels     []string `yaml:",flow"`
	PluginName string   `yaml:"plugin-name"`
	PreInstall string   `yaml:"pre-install"`
}

We need to define other types like:

yaml
bash
helm

it should be something like:
type PluginSpec struct {
	Yaml ---> yaml files
        Bash --> bash files
        Helm --> helm files

	Labels     []string `yaml:",flow"`
	PluginName string   `yaml:"plugin-name"`
	PreInstall string   `yaml:"pre-install"`

}

yaml:
plugin-name: Argo
pre-install: ""
yaml:
  - "https://raw.githubusercontent.com/argoproj/argo/stable/manifests/namespace-install.yaml"
bash:
  - bashfile
  - bashfile2
helm:
  - helmfile1
  - helmfile2

I can take this. Do you think it would make sense to restrict type to just one kind at a time in a spec file? Eg: Specifying both yaml and bash in a single spec file should not be supported.

I can take this.

Great.

Do you think it would make sense to restrict type to just one kind at a time in a spec file?

For the moment I'd leave the possibility to specify more types.

What's the purpose of pre-install? What kinda instruction would go here?

I was thinking about a generic shell script needed to configure or prepare the environment.
But I haven't decided yet. Any idea is welcome

oh, I thought it would be some other plugin that needs to be installed. Be it bash or another plugin, I can see some potential for conflict. Imagine two plugins doing a pre-install of two different version a tool. I am not overly worried about this at the moment but at some point when we run into problems, we can revisit it.

Let's remove pre-install for the moment, we will see for the future if we need it

Split into two issues #41 #50


server: move UpdateDefaultNic to vm work job queue

Description

While remove secondary nic from a Running vm, if update the default nic to the secondary nic before the nic is removed, the vm will not have default nic (and cannot be started) when both operations are completed.
It is because UpdateDefaultNic api is not handled as a vm work job (AddNicToVMCmd and RemoveNicFromVMCmd are), it is processed before nic is removed. The result is that secondary nic becomes default nic and got removed.

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[X] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Screenshots (if appropriate):
How Has This Been Tested?

remove secondary nic from a Running vm
update default nic to the secondary nic before step 1 is finished.
got an exception  as below.

@weizhouapache I saw this and #4019. seems good work but I am postponing reviewing till quiet times, and not bother you with my style purism now. Also I have no time for testing. marking 4.15 if you don't mind. feel free to change if you feel these are critical.

@weizhouapache I saw this and #4019. seems good work but I am postponing reviewing till quiet times, and not bother you with my style purism now. Also I have no time for testing. marking 4.15 if you don't mind. feel free to change if you feel these are critical.

@DaanHoogland The issues are not blocker. you can review them afterwards.
for now, we need to work together to make 4.14 (and 4.13.2) live, which has highest priority I think.

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos7 ✔debian. JID-1307

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✖centos7 ✖debian. JID-1365

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✖centos7 ✖debian. JID-1520

@ustcweizhou can you check build failure?

@blueorangutan package

@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos7 ✔debian. JID-1619

@blueorangutan test

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-2222)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 46550 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4020-t2222-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_routers_network_ops.py
Intermittent failure detected: /marvin/tests/smoke/test_vm_life_cycle.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Smoke tests completed. 81 look OK, 2 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

ContextSuite context=Test01DeployVM>:setup
Error
0.00
test_vm_life_cycle.py

ContextSuite context=Test02VMLifeCycle>:setup
Error
0.00
test_vm_life_cycle.py

ContextSuite context=Test03SecuredVmMigration>:setup
Error
0.00
test_vm_life_cycle.py

test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL
Failure
347.36
test_vpc_redundant.py

@borisstoyanov can you (have someone) test this? seems straight forward.

@blueorangutan test

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-2268)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 36058 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4020-t2268-kvm-centos7.zip
Smoke tests completed. 83 look OK, 0 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

@weizhouapache there are conflicts. also do you want this on 4.14 or master only?

@weizhouapache there are conflicts. also do you want this on 4.14 or master only?

@DaanHoogland rebased with latest commit in 4.14 branch.
it would be better to merge it into 4.14 and master both

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos7 ✔debian. JID-1734

@blueorangutan test

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos7 ✔centos8 ✔debian. JID-1838

@blueorangutan test

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-2599)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 51064 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4020-t2599-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_supported_versions.py
Intermittent failure detected: /marvin/tests/smoke/test_outofbandmanagement.py
Intermittent failure detected: /marvin/tests/smoke/test_routers_network_ops.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Intermittent failure detected: /marvin/tests/smoke/test_hostha_kvm.py
Smoke tests completed. 83 look OK, 2 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL
Failure
440.94
test_vpc_redundant.py

test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers
Error
346.95
test_vpc_redundant.py

test_04_rvpc_network_garbage_collector_nics
Error
3934.72
test_vpc_redundant.py

test_hostha_kvm_host_fencing
Error
103.41
test_hostha_kvm.py


Fix/4759/replace unsafe methods

Closes #GH-4759
Referring to Strict Mode - remove UNSAFE_componentWillMount & UNSAFE_componentWillReceiveProps,  also update some devDependencies - react, react-dom, react-test-renderer 18.2+ versions.

Closing due to side-effects in the getDerivedStateFromProps method. UNSAFE_componentWillReceiveProps should be replaced with componentDidUpdate, but it also will affect a lot of logic in other class methods.