"eventfd failed" error on SPOD daemonset pods

What happened:
About three hours ago all of our spod daemonset pods started to crash loop, all with similar error messages:
$ k logs spod-6zrg7
Defaulted container "security-profiles-operator" out of: security-profiles-operator, log-enricher, metrics, non-root-enabler (init)
runtime: eventfd failed with<PHONE_NUMBER>3709551615
fatal error: runtime: eventfd failed

What you expected to happen:
SPOD pods should run normally.
How to reproduce it (as minimally and precisely as possible):
I think perhaps related to a recent build?  The pattern I see is that all of our older SPOD pods, all running image gcr.io/k8s-staging-sp-operator/security-profiles-operator@sha256:a9a912f30dc62baa229d6db54b9e49dcb87f7b7d4633958f481435180f1d8057, are running fine.
Replicas coming up within the past three hours have a different image id and all are erroring.
Anything else we need to know?:

full log output from one spod
runtime: eventfd failed with<PHONE_NUMBER>3709551615
fatal error: runtime: eventfd failed
goroutine 1 gp=0xc0000061c0 m=0 mp=0x51ea4c0 [running, locked to thread]:
runtime.throw({0x31dece7?, 0xc0000061c0?})
runtime/panic.go:1067 +0x48 fp=0xc00009eb08 sp=0xc00009ead8 pc=0x477488
runtime.netpollinit()
runtime/netpoll_epoll.go:31 +0x156 fp=0xc00009eb78 sp=0xc00009eb08 pc=0x439c56
runtime.netpollGenericInit()
runtime/netpoll.go:224 +0x35 fp=0xc00009eb90 sp=0xc00009eb78 pc=0x439155
internal/poll.runtime_pollServerInit()
runtime/netpoll.go:215 +0xf fp=0xc00009eba0 sp=0xc00009eb90 pc=0x47654f
sync.(*Once).doSlow(0xffffffffffffff9c?, 0xc00009ec30?)
sync/once.go:76 +0xb4 fp=0xc00009ec00 sp=0xc00009eba0 pc=0x48d514
sync.(*Once).Do(...)
sync/once.go:67
internal/poll.(*pollDesc).init(0xc000148140, 0xc000148120)
internal/poll/fd_poll_runtime.go:39 +0x3c fp=0xc00009ec20 sp=0xc00009ec00 pc=0x5388bc
internal/poll.(*FD).Init(0xc000148120, {0x31ab7af?, 0x0?}, 0xb0?)
internal/poll/fd_unix.go:66 +0x45 fp=0xc00009ec40 sp=0xc00009ec20 pc=0x5397a5
os.newFile(0x3, {0xc000156000, 0x1b}, 0x1, 0x0)
os/file_unix.go:237 +0x165 fp=0xc00009ec80 sp=0xc00009ec40 pc=0x548ce5
os.openFileNolog({0xc000156000, 0x1b}, 0x0, 0x156000?)
os/file_unix.go:297 +0x192 fp=0xc00009ed18 sp=0xc00009ec80 pc=0x548f32
os.OpenFile({0xc000156000, 0x1b}, 0x0, 0x0)
os/file.go:385 +0x3e fp=0xc00009ed48 sp=0xc00009ed18 pc=0x5468fe
os.Open(...)
os/file.go:365
google.golang.org/protobuf/internal/detrand.binaryHash()
google.golang.org/protobuf@v1.35.1/internal/detrand/rand.go:46 +0x53 fp=0xc00009ee10 sp=0xc00009ed48 pc=0x139f8d3
google.golang.org/protobuf/internal/detrand.init()
google.golang.org/protobuf@v1.35.1/internal/detrand/rand.go:38 +0xf fp=0xc00009ee20 sp=0xc00009ee10 pc=0x139f86f
runtime.doInit1(0x50fe5e0)
runtime/proc.go:7290 +0xe8 fp=0xc00009ef50 sp=0xc00009ee20 pc=0x44f528
runtime.doInit(...)
runtime/proc.go:7257
runtime.main()
runtime/proc.go:254 +0x345 fp=0xc00009efe0 sp=0xc00009ef50 pc=0x440ce5
runtime.goexit({})
runtime/asm_amd64.s:1700 +0x1 fp=0xc00009efe8 sp=0xc00009efe0 pc=0x47ffe1
goroutine 2 gp=0xc000006c40 m=nil [force gc (idle)]:
runtime.gopark(0x0?, 0x0?, 0x0?, 0x0?, 0x0?)
runtime/proc.go:424 +0xce fp=0xc00008cfa8 sp=0xc00008cf88 pc=0x4775ae
runtime.goparkunlock(...)
runtime/proc.go:430
runtime.forcegchelper()
runtime/proc.go:337 +0xb3 fp=0xc00008cfe0 sp=0xc00008cfa8 pc=0x440f73
runtime.goexit({})
runtime/asm_amd64.s:1700 +0x1 fp=0xc00008cfe8 sp=0xc00008cfe0 pc=0x47ffe1
created by runtime.init.7 in goroutine 1
runtime/proc.go:325 +0x1a
goroutine 3 gp=0xc000007180 m=nil [GC sweep wait]:
runtime.gopark(0x0?, 0x0?, 0x0?, 0x0?, 0x0?)
runtime/proc.go:424 +0xce fp=0xc00008d780 sp=0xc00008d760 pc=0x4775ae
runtime.goparkunlock(...)
runtime/proc.go:430
runtime.bgsweep(0xc0000ba000)
runtime/mgcsweep.go:277 +0x94 fp=0xc00008d7c8 sp=0xc00008d780 pc=0x4288b4
runtime.gcenable.gowrap1()
runtime/mgc.go:203 +0x25 fp=0xc00008d7e0 sp=0xc00008d7c8 pc=0x41cfe5
runtime.goexit({})
runtime/asm_amd64.s:1700 +0x1 fp=0xc00008d7e8 sp=0xc00008d7e0 pc=0x47ffe1
created by runtime.gcenable in goroutine 1
runtime/mgc.go:203 +0x66
goroutine 4 gp=0xc000007340 m=nil [GC scavenge wait]:
runtime.gopark(0xc0000ba000?, 0x36c6dc8?, 0x1?, 0x0?, 0xc000007340?)
runtime/proc.go:424 +0xce fp=0xc00008df78 sp=0xc00008df58 pc=0x4775ae
runtime.goparkunlock(...)
runtime/proc.go:430
runtime.(*scavengerState).park(0x51e7c20)
runtime/mgcscavenge.go:425 +0x49 fp=0xc00008dfa8 sp=0xc00008df78 pc=0x4262e9
runtime.bgscavenge(0xc0000ba000)
runtime/mgcscavenge.go:653 +0x3c fp=0xc00008dfc8 sp=0xc00008dfa8 pc=0x42685c
runtime.gcenable.gowrap2()
runtime/mgc.go:204 +0x25 fp=0xc00008dfe0 sp=0xc00008dfc8 pc=0x41cf85
runtime.goexit({})
runtime/asm_amd64.s:1700 +0x1 fp=0xc00008dfe8 sp=0xc00008dfe0 pc=0x47ffe1
created by runtime.gcenable in goroutine 1
runtime/mgc.go:204 +0xa5
goroutine 18 gp=0xc000104700 m=nil [finalizer wait]:
runtime.gopark(0xc00008c648?, 0x412725?, 0xb0?, 0x1?, 0xc0000061c0?)
runtime/proc.go:424 +0xce fp=0xc00008c620 sp=0xc00008c600 pc=0x4775ae
runtime.runfinq()
runtime/mfinal.go:193 +0x107 fp=0xc00008c7e0 sp=0xc00008c620 pc=0x41c067
runtime.goexit({})
runtime/asm_amd64.s:1700 +0x1 fp=0xc00008c7e8 sp=0xc00008c7e0 pc=0x47ffe1
created by runtime.createfing in goroutine 1
runtime/mfinal.go:163 +0x3d

Environment:

Cloud provider or hardware configuration: AWS EKS
OS (e.g: cat /etc/os-release): Amazon Linux 2
Kernel (e.g. uname -a): 5.10.217-205.860.amzn2.x86_64
Others:

The staging image wasn't pushed in a while https://prow.k8s.io/job-history/gs/kubernetes-ci-logs/logs/post-security-profiles-operator-push-image.
It started to work again now. Maybe you want to try one of the latest images pushed today.

Thanks, will take a look there... the interesting thing is that I'm not specifying any particular image to use, just the 0.8.4 helm chart, but I think internally it is specifying to pull a latest for the spod container.  Will poke around on it some more...

I'm seeing this too -- it looks like pulling latest is still not helping?
 Normal   Pulled     7m10s                   kubelet            Successfully pulled image "gcr.io/k8s-staging-sp-operator/security-profiles-operator:latest" in 824ms (824ms including waiting)
  Normal   Started    7m9s (x3 over 7m29s)    kubelet            Started container security-profiles-operator
  Warning  BackOff    2m20s (x33 over 7m27s)  kubelet            Back-off restarting failed container security-profiles-operator in pod spod-vd8dd_security-profiles-operator(392ea67e-5c26-4fb3-85d7-9bd4be14b9ce)

@zestysoft we updated the seccomp profile for the spod, do you mind updating the deployment to match?
https://github.com/kubernetes-sigs/security-profiles-operator/blob/0102ed08538e6e16c4322d97c82680cdf33a0d62/deploy/operator.yaml#L2927

I think the larger problem is that we're using a Helm chart that is pinned to a specific version, but the values.yaml file is using latest by default:
https://github.com/kubernetes-sigs/security-profiles-operator/blob/4c0611e2b783e9751f332a9c466e74e57c4a7f0b/deploy/helm/values.yaml#L7
overriding this to v0.8.4 fixes this problem.

confirmed that overriding to v0.8.4 fixes the problem for now, thanks!  closing issue.


Allow disk shrinking.

Author Name: Ruben S. Montero (@rsmontero)
Original Redmine Issue: 5237, https://dev.opennebula.org/issues/5237
Original Date: 2017-07-13

As reported in #3395

I tried to shrink an image (live running VM) and got: [one.vm.diskresize] New disk size has to be greater than current one. Please, let me shrink disks too. Just give a big "WARNING" message that you should already have shrunk filesytems / paritions, etc. ... but let me do it. If you think it is too "dangerous" for most use cases a oned option of "ALLOW_DISK_RESIZE_SHRINKING" could be used to enable this. Workloads change, ONE should be able to accomodate, whether capacity should increase or decrease.

Original Redmine Comment
Author Name: Anton Todorov (Anton Todorov)
Original Date: 2017-10-02T19:09:26Z

+1
BR,
Anton


bug: k8s.apisix.apache.org/plugin-conifg-name annotation is not working on kubernetes ingress

Issue description
I configured an ingress according to the annotation guide on an ACK kubernetes. But It's not working at all.
My configuration as below:

apiVersion: apisix.apache.org/v2
kind: ApisixPluginConfig
metadata:
  name: echo-and-cors-apc
  namespace: test
spec:
  plugins:
  - name: echo
    enable: true
    config:
      before_body: "This is the preface"
      after_body: "This is the epilogue"
      headers:
        X-Foo: v1
        X-Foo2: v2
  - name: cors
    enable: true
---

apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  name: httpbin-demo-apisix
  namespace: test
  annotations:
    kubernetes.io/ingress.class: apisix
    k8s.apisix.apache.org/plugin-conifg-name: "echo-and-cors-apc"
spec:
  rules:
  - host: httpbin-demo.apisix.mysite.com
    http:
      paths:
      - backend:
          service:
            name: httpbin-demo
            port:
              number: 80
        path: /
        pathType: Prefix

After applying, I can get the configured PluginConfig and Route from  apisix_admin api.
The PluginConfig added:
{
        "createdIndex": 377,
        "value": {
          "name": "test_echo-and-cors-apc",
          "update_time":<PHONE_NUMBER>,
          "desc": "Created by apisix-ingress-controller, DO NOT modify it manually",
          "create_time":<PHONE_NUMBER>,
          "plugins": {
            "cors": {
              "allow_headers": "*",
              "allow_origins": "*",
              "expose_headers": "*",
              "max_age": 5,
              "allow_credential": false,
              "allow_methods": "*"
            },
            "echo": {
              "before_body": "This is the preface",
              "after_body": "This is the epilogue",
              "headers": {
                "X-Foo": "v1",
                "X-Foo2": "v2"
              }
            }
          },
          "id": "777a719e",
          "labels": {
            "managed-by": "apisix-ingress-controller"
          }
        },
        "modifiedIndex": 380,
        "key": "/apisix/plugin_configs/777a719e"
 }

The Route added:
{
        "createdIndex": 379,
        "value": {
          "name": "ing_test_httpbin-demo-apisix_e5d87ed",
          "priority": 0,
          "desc": "Created by apisix-ingress-controller, DO NOT modify it manually",
          "upstream_id": "29777be0",
          "host": "httpbin-demo.apisix.mysite.com",
          "update_time":<PHONE_NUMBER>,
          "status": 1,
          "create_time":<PHONE_NUMBER>,
          "uris": [
            "/",
            "/*"
          ],
          "id": "d6df95c8",
          "labels": {
            "managed-by": "apisix-ingress-controller"
          }
        },
        "modifiedIndex": 379,
        "key": "/apisix/routes/d6df95c8"
}

As we can see, no PluginConfig attached to Route.
And tests to http://httpbin-demo.apisix.mysite.com/get from browser also show no working.
Environment

your apisix-ingress-controller version (output of apisix-ingress-controller version --long):

helm.sh/chart:ingress-controller-0.10.1
app.kubernetes.io/version:1.5.0

your Kubernetes cluster version (output of kubectl version):

Client Version: version.Info{Major:"1", Minor:"25", GitVersion:"v1.25.2", GitCommit:"5835544ca568b757a8ecae5c153f317e5736700e", GitTreeState:"clean", BuildDate:"2022-09-21T14:33:49Z", GoVersion:"go1.19.1", Compiler:"gc", Platform:"linux/amd64"}
Kustomize Version: v4.5.7
Server Version: version.Info{Major:"1", Minor:"24+", GitVersion:"v1.24.6-aliyun.1", GitCommit:"be2c483aa02d6189c93f0b4ca6420c1d37d5c55c", GitTreeState:"clean", BuildDate:"2022-11-16T04:06:26Z", GoVersion:"go1.18.6", Compiler:"gc", Platform:"linux/amd64"}

if you run apisix-ingress-controller in Bare-metal environment, also show your OS version (uname -a):
nope

Minimal test code / Steps to reproduce

applly with below template：

apiVersion: apisix.apache.org/v2
kind: ApisixPluginConfig
metadata:
  name: echo-and-cors-apc
  namespace: test
spec:
  plugins:
  - name: echo
    enable: true
    config:
      before_body: "This is the preface"
      after_body: "This is the epilogue"
      headers:
        X-Foo: v1
        X-Foo2: v2
  - name: cors
    enable: true
---

apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  name: httpbin-demo-apisix
  namespace: test
  annotations:
    kubernetes.io/ingress.class: apisix
    k8s.apisix.apache.org/plugin-conifg-name: "echo-and-cors-apc"
spec:
  rules:
  - host: httpbin-demo.apisix.mysite.com
    http:
      paths:
      - backend:
          service:
            name: httpbin-demo
            port:
              number: 80
        path: /
        pathType: Prefix

Open http://httpbin-demo.apisix.mysite.com/get from browser.

Actual result
The PluginConfig has no effect. the output from browser network inspection keep the same.
Error log
Not noticed.
Expected result
The PluginConfig takes effect. http header added:  "X-Foo": "v1", "X-Foo2": "v2"
Thanks a lot for your excellent work ！
YufuLuo

Sorry for the wrong version info.
apisix-ingress-controller version is 1.5.1

I think you should check your config.
Should be plugin-config-name instead of plugin-conifg-name.

We have an almost identical e2e test case to cover this. Please refer to this one.
https://github.com/apache/apisix-ingress-controller/blob/master/test/e2e/suite-annotations/plugin_conifg.go#L78-L107

I think you should check your config. Should be plugin-config-name instead of plugin-conifg-name.

The Truth. Will be more careful.
thanks for your time.

I think we can add log warnings for unsupported or incorrect Annotations.


arg_parse.c: Support multiple files conversion

Hi, @danny-burrows.
This issue is for arg_parse.c file.
About suggestion
When I used the program, I ran the command below.

./img_to_txt -hw 100 01.jpg -hw 50 02.jpg

As you can guess, I just wanted to convert multiple files with different options separately.
But, the results generated by the program showed me that they have same spec (or scale) given by the very last option (in this case, -hw 50).
So, there is my suggestion I want you to consider.
Could I make a pull request about dealing with multiple files that have different options?
Expected changes

Delete a -- detecting condition
Reorganization the structure of arg_parse function

This is just a draft, so you don't have to take care of them deeply.

Thanks for reading, and I look forward to hearing from you.

Hello @TwoPair, I'd be more than open to accepting a pull request to fix this behavior. The original usage was modeled on a review of the common utility ls but I can absolutely see that having the option for different settings for each image would be of benefit.
Thank you for your interest and contribution, I look forward to hearing more on this. :)


[WIP] Protocol : Nonequilibrium Cycling

Description

A nonequilibrium cycling protocol supporting execution with OpenMM, using the protocol structure defined in gufe.
Motivation and context

This is one of the first protocols to be developed using the gufe structure, and is intended to surface deficiencies in that structure.  The end result will be a protocol that can be executed locally (with Protocol.execute()) as well as on cluster resources using a suitable Scheduler (e.g. an HPC scheduler like that being developed by OpenFE).
Immediate needs

[ ] docstrings on all methods required or defined

How has this been tested?

[ ] unit tests of protocol units
[ ] unit tests of NonEquilibriumCyclingProtocol
[ ] unit tests of NonEquilibriumCyclingResult

Change log

New features
^^^^^^^^^^^^
- `perses.protocols` submodule added; features `NonEquilibriumCyclingProtocol`

Suggestions for names other than Protocol: experiment, runner, workflow, pipeline

Ah, I think I'm still inclined to call this Protocol, since in the context of gufe this defines the procedure by which we execute the simulations necessary to calculate a \Delta G. A Protocol produces a ProtocolDAG from a set of input ChemicalSystems, an atom mapping (if necessary), and potentially any previous result that should be used as a starting point.
The ProtocolDAG is more akin to an individual experiment or workflow in this sense, while the Protocol it comes from is more akin to the instructions for performing it.

Thanks for the clarification! I still think it would be better to rename Protocol because if gufe is going to replace the perses API, "protocol" means something different in the context of free energy calculations, so its going to be confusing for perses users. However, it seems like I may be the only one that has a problem with the name, so I think its ok if we keep it as is and just make sure the class/name choice is well documented (which I'm sure you are planning to do anyway).

I don't have a strong opinion here, there are valid points from all sides. I guess that as long as we make this clear from the docstrings and existing documentation, this should be fine. We may even want to put something like "Not to be confused with... " as a first line in the docstring.
I wonder if aliasing the gufe classes names is a sane option for perses codebase. I don't really know enough about design patterns around this.

...because if gufe is going to replace the perses API, "protocol" means something different in the context of free energy calculations, so its going to be confusing for perses users.

@zhang-ivy can you define here what "protocol" means in the context of free energy calculations? I don't know that there's a universally-accepted definition in the field, but if there is strong consensus I'd like to know what it is so we can act accordingly.

When I hear "protocol" in the context of alchemical free energy calculations, I think of the alchemical protocol, which tells us how we should transform the WT (or lambda = 0 endstate) into the mutant (or lambda = 1 endstate) residue. In practice, this can be thought of as a list of lambda values. The protocols I've been using are linear (e.g. if there are 4 alchemical states, the protocol would be [0, 0.25, 0.5, 1.0] and 0 would be mean that the WT residue's nonbonded interactions are fully on and the mutant residue's nonbonded interactions are fully off, 0.25 would mean that the WT residue's nonbonded interactions are turned 75% on and the mutant nonbonded interactions are turned 25% on, and so on), but they can be non-linear with a varying number of alchemical states.
The alchemical protocol used is critical to the calculation because it determines how well our states overlap, which in turn affects the bias and error of the calculation.

@ijpulidos can we sync up on this early this week? I'd like to be in a position to run this protocol locally, ideally by 11/08. Is there anything blocking this for you?

Added example notebook here examples/NEQ_cycling_using_gufe_objects/NEQ_cycling_from_gufe_objs.ipynb it is messy, but a start on the notebook that we will want to show people how to use the code in the PR.

@dotsdl @ijpulidos : Does the object model for free energy results support confidence intervals or bootstrap samples for free energy estimates, or only standard errors?

For testing:

We want to run alchemical transformations on very simple small systems, such as a small molecule transformation in vacuum or a tiny solvent box.
@irfan noted he has a series of simple benzene transformations he could share
Failing that, we could go back to our simple transformations (e.g. mutation in alanine dipeptide, changing substituents of benzene)
Ideally, we would carry out a null transformation in vacuum (or solvent if needed) where we transform the molecule into the same molecule, but we don't map all atoms. For example, fluorobenzene to fluorobenzene:

[c:1]1(F)[c:2][c:3][c:4][c:5][c:6]1>>[c:1]1[c:2][c:3][c:4](F)[c:5][c:6]1

20 independent trajectories of 1 cycle each should generate 20 forward/reverse work pairs, and this should be enough to estimate a free energy.
We can keep the cycling trajectories short: 100 ps/segment (where a segment is one of eqA, neqAB, eqB, neqBA), so 400 ps/cycle

For production:

We likely want to recommend 100 independent trajectories of 1 cycle each, 1 ns/segment

* @irfan noted he has a series of simple benzene transformations he could share

The benzene transformations are in https://github.com/OpenFreeEnergy/openfe-benchmarks/blob/main/openfe_benchmarks/benzenes.py

I'm fixing up the settings bits to match latest gufe.

@ijpulidos it looks like the CI may be stuck? Also, running tests locally I now get:
> pytest --pdb perses/tests/protocols
============================================================================================= test session starts =============================================================================================
platform linux -- Python 3.9.16, pytest-7.2.2, pluggy-1.0.0
rootdir: /home/david/Library/choderalab/perses
plugins: anyio-3.6.2
collected 6 items

perses/tests/protocols/test_nonequilibrium_cycling.py E
>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>> captured log >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
INFO     utils.openeye:openeye.py:434 molecule benzene                      does not have unique atom names. Generating now...
INFO     utils.openeye:openeye.py:434 molecule toluene                      does not have unique atom names. Generating now...
INFO     relative_setup:relative_setup.py:167 			_hmass: 3 Da.

INFO     relative_setup:relative_setup.py:198 No receptor specified. Assuming non-complex phase.
INFO     relative_setup:relative_setup.py:209 	setting both molecule oemol titles to 'MOL'.
INFO     relative_setup:relative_setup.py:224 Created mdtraj topologies for both ligands.
INFO     relative_setup:relative_setup.py:492 Setting non bonded method to PME
INFO     relative_setup:relative_setup.py:251 set MonteCarloBarostat because pressure was specified as 1.0 atm atmospheres
INFO     relative_setup:relative_setup.py:288 PME tolerance: 0.00025
INFO     relative_setup:relative_setup.py:313 successfully created SystemGenerator to create ligand systems
INFO     relative_setup:relative_setup.py:315 executing SmallMoleculeSetProposalEngine...
INFO     relative_setup:relative_setup.py:329 instantiating FFAllAngleGeometryEngine...
INFO     relative_setup:relative_setup.py:737 Detected solvent...
INFO     relative_setup:relative_setup.py:739 assgning noCutoff to nonbonded_method
INFO     relative_setup:relative_setup.py:743 Removing barostat for vacuum phase
INFO     relative_setup:relative_setup.py:745 Setting nonbondedMethod to NoCutoff for vacuum phase
INFO     relative_setup:relative_setup.py:747 calling SystemGenerator to create ligand systems.
INFO     relative_setup:relative_setup.py:750 No complex or solvent leg, so performing topology proposal for vacuum leg
INFO     relative_setup:relative_setup.py:1055 	Skipping solvation of vacuum perturbation
INFO     relative_setup:relative_setup.py:1086 	parameterizing...
INFO     openmmforcefields.generators.template_generators:template_generators.py:280 Requested to generate parameters for residue <Residue 0 (MOL) of chain 0>
INFO     openmmforcefields.generators.template_generators:template_generators.py:1402 Generating a residue template for [H]c1c(c(c(c(c1[H])[H])[H])[H])[H] using openff-2.0.0
INFO     relative_setup:relative_setup.py:1088 	System parameterized
INFO     relative_setup:relative_setup.py:1095 Both trajectory_directory and trajectory_prefix need to be provided to save .pdb
INFO     proposal_generator:topology_proposal.py:2053 creating current receptor topology by removing small molecule from current topology...
INFO     proposal_generator:topology_proposal.py:2060 small molecule start index: 0
INFO     proposal_generator:topology_proposal.py:2061 small molecule has 12 atoms.
INFO     proposal_generator:topology_proposal.py:2065 old alchemical atom indices: range(0, 12)
INFO     proposal_generator:topology_proposal.py:2076 proposed mol detected with smiles [H]c1c(c(c(c(c1[H])[H])C([H])([H])[H])[H])[H] and logp_proposal of 0.0
INFO     proposal_generator:topology_proposal.py:2079 conducting proposal from [H]c1c(c(c(c(c1[H])[H])[H])[H])[H] to [H]c1c(c(c(c(c1[H])[H])C([H])([H])[H])[H])[H]...
INFO     proposal_generator:topology_proposal.py:2082 building new topology with proposed molecule and current receptor topology...
INFO     proposal_generator:topology_proposal.py:2303 	setting proposed oemol title to MOL
INFO     proposal_generator:topology_proposal.py:2305 	creating mol topology from oemol...
INFO     proposal_generator:topology_proposal.py:2308 	appending current receptor topology to new mol topology...
INFO     proposal_generator:topology_proposal.py:2087 new molecule has a start index of 0 and 15 atoms.
INFO     proposal_generator:topology_proposal.py:2090 proceeding to build the new system from the new topology...
INFO     openmmforcefields.generators.template_generators:template_generators.py:280 Requested to generate parameters for residue <Residue 0 (MOL) of chain 1>
INFO     openmmforcefields.generators.template_generators:template_generators.py:1402 Generating a residue template for [H]c1c(c(c(c(c1[H])[H])C([H])([H])[H])[H])[H] using openff-2.0.0
INFO     proposal_generator:topology_proposal.py:2099 determining atom map between old and new molecules...
INFO     proposal_generator:topology_proposal.py:2127 atom map is pre-determined as {0: 4, 1: 5, 2: 6, 3: 7, 4: 8, 5: 9, 6: 10, 7: 11, 8: 12, 9: 13, 11: 14}
INFO     proposal_generator:topology_proposal.py:2175 Proposed transformation would delete 2 atoms and create 5 atoms.
INFO     relative_setup:relative_setup.py:771 conducting geometry proposal...
INFO     geometry:geometry.py:218 propose: performing forward proposal
INFO     geometry:geometry.py:235 propose: unique new atoms detected; proceeding to _logp_propose...
INFO     geometry:geometry.py:371 Conducting forward proposal...
INFO     geometry:geometry.py:380 Computing proposal order with NetworkX...
 Since, there are also merge conflicts I think that's why the CI is stuck.

@dotsdl Oh, looking at your error traceback the problem must be related with how we inverted the mapping in the protocol but we haven't done so in the test fixtures. Should be an easy fix.

Now that we are doing the refactor, we can merge this in once tests pass (feel free to remove tests that don't make sense anymore, but I do what the new stuff tested)

Tests will fail with this one, I don't really know why they aren't being run. But I expect we would need to refactor some of the tests since the API for the RelativeFEPSetup object has changed.
Does it make sense that we try to merge this one as it is (after we resolve the merging conflicts) and work on fixing/refactoring the tests for it in a separate PR? Such that we can review these changes independently and it doesn't get too overwhelming (this is already a huge PR).

Oh right, I think tests are not running because of the merge conflict, @mikemhenry already mentioned that in the past.

We could just mark the failing tests to be skiped as old_api or similar. And fix the tests when we work on the refactor.

The activity on this protocol has been migrated to https://github.com/choderalab/feflow. Closing.


chore: setup pact in CI/CD

The diagram of the pipeline can be found at https://miro.com/app/board/uXjVNsON77A=/?moveToWidget=3458764586699482727&cot=14
The workspace-service branch is https://github.com/snyk/workspace-service/compare/main...feat/spike/pact-separate.
After publish but before verify:

After verify:
To do:

check if prodsec are happy with the solution
move from the team's pactflow to the general one

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: teodora-sandu:x: michelkaporinYou have signed the CLA already but the status is still pending? Let us recheck it.

Most of this was implemented in https://github.com/snyk/code-client-go/pull/55. This version is the diamond version but webhookless, which we shouldn't do because of what's explained in https://snyksec.atlassian.net/browse/IDE-296?focusedCommentId=998299.


Implement new error notifications design

https://openedx.atlassian.net/browse/MA-2412

@miankhalid, @BenjiLee, this is ready for an initial review.

Current coverage is 18.87% (diff: 1.33%)

Merging #866 into master will decrease coverage by 0.05%

@@             master       #866   diff @@
==========================================
  Files           469        472     +3
  Lines         19941      20009    +68
  Methods           0          0
  Messages          0          0
  Branches       2915       2927    +12
==========================================
+ Hits           3775       3776     +1
- Misses        15719      15786    +67
  Partials        447        447

Powered by Codecov. Last update 7b3d10f...3263608

The AlertDialogFragment has also been updated to include a new static
constructor that takes string resource IDs instead of the actual
resolved strings, in order to not have an unneeded dependency on the
Context in it's notification implementation.

Confusing double negative! but I don't know how else to word it.

Observation: Any snackbar that appears is not dismissable by swiping. Is this something that was left out intentionally ?

Yeah, that's something I left out initially since I was just working on the
core change. I'll add a change adding a CoordinatorLayout to the root of
all the layouts later.
On Mon, Oct 31, 2016 at 2:36 PM, Mi@N Kh@LiD <EMAIL_ADDRESS>wrote:

Observation: Any snackbar that appears is not dismissable by swiping. Is
this something that was left out intentionally ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/edx/edx-app-android/pull/866#issuecomment-257249996,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEDCjW73lRQhfhzDiDM8y-tbDiQGGaTEks5q5baZgaJpZM4KZln2
.

Discussion threads are not loading and snackbar + fullscreen error are appearing simultaneously:

I'm holding off on reviewing this PR until this PR #857 is ready for review (cuz the dependency over that PR is too high), since this PR doesn't seem to be upto mark with the acceptance criteria listed on the ticket.
p.s. It might be a good idea to mark this PR as WIP.

The persistent Snackbar errors on multiple screens were caused by the NavigationFragment, which is added by BaseFragmentActivity if the drawer layout is present, but blocked from opening by most implementations. I have removed error notifications from NavigationFragment in commit 6d359c090188ecbdf6e9ae5b320be09a470ab184, since there's no proper place for it to be destroyed (as Snackbars should be shown below the drawer).
I have also removed temporary debugging code that I had accidentally checked in, which was causing all the calls to be reported as failing in commit f46f510c2372adc277d1bd45de8f2588a7e4d93f.

@1zaman I'm not seeing any snackbars.

@miankhalid @1zaman Should we prioritize the review of https://github.com/edx/edx-app-android/pull/857 over this?

Should we prioritize the review of #857 over this?

+1 from me.

Yeah, I guess it would be better to get #857 merged before this, since then it will be easier to implement this in the Assessments module easily without major changes later.

I've rebased this PR successfully, and tests are passing too.
The issues mentioned above have been tested and I haven't been able to reproduce any. @BenjiLee can you give it a look too ?
I have found a minor issue in Discussion, where persistent connection error appears in the middle of screen. I'll fix that issue in a followup commit, but before that @verdanmahmood suggested to get the branch tested thoroughly by @MirzaMubasharBaig (which he'll be doing tomorrow hopefully).
So, I'll fix all the issues (if anything else surfaces) in one go.

I'm still seeing the old UI everywhere.

@farhan this PR is ready for a first pass.
p.s ignore this Commit # 2b033d6 (Offline support for Handouts and Announcements) infavor of this Commit # 1ce08f1 (Offline mode changes).

Snack bar also appearing with full screen blocker
Steps:

Fresh install the app and keep the internet available.
Do login in the app.
Choose some course from "My Courses" screen and go to its dashboard.
On "Course Dashboard" screen, disable the internet.
Now navigate to its Outline screen by pressing "Course" tab.
You will see full screen blocker as well as snack bar. Snack bar is not required here.

Offline full screen blocker not getting hide after successful reloading of page.

Now enable the internet and press 'Reload' button. Full blocker message keep appearing even the page gets reload successfully.

As mentioned on this wiki: https://openedx.atlassian.net/wiki/display/MA/Offline+Errors
"In the login activity, when the app detects that we are offline, an alert dialog is displayed that tells the user there is no connection. This can be dismissed either by hitting OK or getting the back button. It also goes away if we have a network connection again."
Dialog doesn't go away automatically if have a network connection back again.

Here are few issues which needs to be addressed.

Find Courses screen is according to new design like iOS. Its in both cases if user come on it after singing in or without signing in.
On My Courses screen, if courses are not cached, iOS is showing full screen blocker message rather than offline snack bar.
Load all the courses on My Courses screen, disable internet, dismiss the offline snack bar, navigate to some course dashboard and come back to My Courses screen,  you will not see the snack bar again (it happens on other screens as well) unlike iOS.
Disable internet and tap on some video to watch it. On Course unit screen you will see a snackbar which is unrequired and not appearing in iOS as well. Suggestion: This screen needs to be improved in case of non-downloaded videos, full screen blocker is a good option, otherwise it will be a bad user experience.

ErrorNotification, SnackbarErrorNotification, DialogErrorNotification & OverlayErrorNotification classes code indentation and java docs suppose to be improved at best as whole new design implementation revolves around it.
Indentation could be improved by just selection all code and hit formatting short cut key.

panel_offfline_access.xml layout and its resources should be removed as they are not getting used anywhere.

Getting offline error in top panel when try the change picture in offline mode.

@farhan All the issues that you reported have been fixed.
Following are the answers to some of the queries listed:

Dialog doesn't go away automatically if have a network connection back again.

Blocking errors i.e. Dialogs shouldn't go away automatically.

Find Courses screen is according to new design like iOS. Its in both cases if user come on it after singing in or without signing in.

Find Courses is out of scope for this PR.

Indentation could be improved by just selection all code and hit formatting short cut key.

Lets do this at the end when you've approved the PR.

panel_offline_access.xml layout and its resources

I can remove the functions you mentioned, but this layout is still being used in the VideoListActivity.java

Ready for another pass @farhan.

@farhan can you please make sure to give it another pass?

"My Courses" screen still shows snackbar rather than full screen message in case when no online or cache data is available to show. Looks like some exceptions in background.

App got crashed with following logs
05-26 15:11:40.842 9505-9505/org.edx.mobile E/AndroidRuntime: FATAL EXCEPTION: main
Process: org.edx.mobile, PID: 9505
java.lang.NullPointerException: Attempt to invoke virtual method 'android.content.res.AssetManager android.content.Context.getAssets()' on a null object reference
at org.edx.mobile.util.FileUtil.loadTextFileFromAssets(FileUtil.java:44)
at org.edx.mobile.util.WebViewUtil.getIntialWebviewBuffer(WebViewUtil.java:23)
at org.edx.mobile.view.CourseCombinedInfoFragment.populateAnnouncements(CourseCombinedInfoFragment.java:233)
at org.edx.mobile.view.CourseCombinedInfoFragment.access$100(CourseCombinedInfoFragment.java:46)
at org.edx.mobile.view.CourseCombinedInfoFragment$3.onResponse(CourseCombinedInfoFragment.java:213)
at org.edx.mobile.view.CourseCombinedInfoFragment$3.onResponse(CourseCombinedInfoFragment.java:208)
at org.edx.mobile.http.callback.ErrorHandlingOkCallback$1.run(ErrorHandlingOkCallback.java:285)
at android.os.Handler.handleCallback(Handler.java:739)
at android.os.Handler.dispatchMessage(Handler.java:95)
at android.os.Looper.loop(Looper.java:158)
at android.app.ActivityThread.main(ActivityThread.java:7224)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1230)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1120)
05-26 15:11:42.202 9505-9960/org.edx.mobile D/OkHttp: <-- 200 OK https://courses.edx.org/api/user/v1/accounts/Mkhalid (1624ms)
05-26 15:11:42.202 9505-9960/org.edx.mobile D/OkHttp: Allow: GET, PATCH, HEAD, OPTIONS
05-26 15:11:42.202 9505-9960/org.edx.mobile D/OkHttp: Content-Language: en
05-26 15:11:42.202 9505-9960/org.edx.mobile D/OkHttp: Content-Type: application/j

I can remove the functions you mentioned, but this layout is still being used in the VideoListActivity.java

I didn't find the usage of 'panel_offline_access.xml' in project.

@miankhalid i am done with second pass, need few things to address.

Ready for another pass @farhan

LGTM :+1:
Just last thing, verify the following case with requirement, do we need to fix it ?

Login to the app and go to 'My Courses' screen.
Disable internet access.
Choose some course and go to 'Course Outline' screen you will see the full screen error message here. Enable the internet and reload the contents.
Now go back to 'My Courses' screen, you will still see Offline Snackbar.

@farhan Fixed the last case you mentioned in 81bc08d. Final pass maybe?

@miankhalid  Snack bar is not hiding for 'Discussion topics' screen for rest required screens its working fine.
Steps:

Go to 'Discussion topics' screen of any course and disable internet, you will see offline snackbar on screen.
Now navigate to next screen by choosing any topic (or go to home screen)
Enable the internet
Come back to 'Discussion topics' screen, you will still see the offline snack bar.


Questionnaire for institutions

We should come up with a questionnaire for institutions to fill out, so that we can both learn about their needs and their information, and so that we can stay connected with them.
This could be as simple as just "what is your email address?", or it could be more complex and ask questions like: size of institution, type of institution, interested in research vs. education, etc.

This now exists!


Color for empty values

Hi,
if I load my data with unset value (null) and if I set a lengendColors such as:
legendColors: { min: "red",
max: "blue",
empty: "grey" }
the cell with null value are not filled properly.
But it works if in addStyle function I add an extra check (d.v === 0 || d.v === null)
such as:
if ((d.v === 0 || d.v === null) && options.legendColors !== null && options.legendColors.hasOwnProperty("empty"))
I hope I didn't misunderstood how lengendColors works.
Regards
Marc

@marcopovitch
I encountered this bug today and use this code to set the rest rect to grey or whatever color u want.
$(".graph>svg>svg>g>rect:not(.r1,.r2,.r3,.r4,.r5)").css("fill","#ededed")
Hope it's useful for u

If anyone is still interested on this, this was my solution.
css:
rect.r0 {
    fill: #ededed;
}


Hidden files

Is there an ability to show dot (hidden) files?

Yes, I want this feature too. Thanks.

Ok. I don't know the right atom packages archiecture right now to make a settings dialog for choose an option.
Open Settings -> Open config folder
navigate to ./.atom/packages/Remote-FTP/lib/ and open directory.js file.
Find "if (name == '.' || name == '..')" (line 113 for me) and change it to "if (name == '..')"
Restart atom and now you can see things were hidden! It's working. Tested for a 15 minutes.
Now I wanna to add config option and commit this little feature.

Windows is showing hidden files. How to do the same on OS X?

Duplicate of https://github.com/mgrenier/remote-ftp/issues/8

Open Settings, open Config folder.
packages/Remote-FTP/node_modules/ftp/lib/connection.js
There are 4 lines begins with cmd = 'LIST:
cmd = 'LIST';
cmd = 'LIST';
cmd = 'LIST ' + path;
cmd = 'LIST ' + path;
Change them:
cmd = 'LIST -al';
cmd = 'LIST -al';
cmd = 'LIST -al ' + path;
cmd = 'LIST -al ' + path;
It's working even with SFTP for me!
OR
On some FTP servers you can add .ftpaccess file to the root with contents
ListOptions "-la"

As @hatzopoulos said, it's a duplicate of #8 which is a feature that is somewhat planned.


Effort to render pydocs for Docs

Proposed changes
Similar to Go Docs, we need to push this repo to use and render pydocs for our code documentation. Also, we (as in I) need to make an effort going forward to have all docs in the repo before tagging the release. Doing this busy use versioned documentation.
Context
See above
Possible Implementation
See above
Other information
See above

@UniversalDevGenius is taking a look at this one for us! Thanks! Will be collaborating together on this.

Closing issue as the initial work is done.


In the compressed format, H represents 8 registers of 8 bytes each.

So the size needs to be H * 8 * 8

@pmsjt : Thanks for your contribution! The author, @, has been notified to review your proposed change.


⚠ Turnos 24x7 has degraded performance

In 7e9af1b, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 1003 ms

Resolved: Turnos 24x7 performance has improved in 3ae2fb3.


email list

no email list (email registration, creation of tickets, etc.)

It seems that logging to database is disabled. Please try go to Configuration -> Settings -> Email -> Email Settings and set Log sent emails to database to Yes.

no work!

Did you enable it? Please submit a ticket with your credentials from members area and we will check what's causing the issue.

access #5832


Update silencer-lib, silencer-plugin to 1.4.2

Updates

com.github.ghik:silencer-lib
com.github.ghik:silencer-plugin

from 1.4.1 to 1.4.2.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [{ groupId = "com.github.ghik" }]

labels: semver-patch

Codecov Report

Merging #72 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #72   +/-   ##
=======================================
  Coverage   51.63%   51.63%
=======================================
  Files          29       29
  Lines         244      244
=======================================
  Hits          126      126
  Misses        118      118

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 1296806...536747c. Read the comment docs.


⚠ Bofrost.es has degraded performance

In fbd9a07, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1460 ms

Resolved: Bofrost.es performance has improved in 7f995de after 8 minutes.


Update settings.yml

Making links in the about me section open in a new tab.

@foohyfooh Right! I'm pretty new with this type of thing. Thank you :)

@djangokillen can you add the {:target=_blank} that @foohyfooh suggested? I will merge then.

I added the feature https://github.com/DevTips/Artists-Theme/commit/6b1173dde6494c300ff18f937095e9ed0c1cbfea


fix(localizations): Add missing keys for pt-BR

add missing localization keys for sign-in actionLink in pt-BR
Type of change

[ ] 🐛 Bug fix
[x] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[ ] @clerk/clerk-js
[ ] @clerk/clerk-react
[ ] @clerk/nextjs
[ ] @clerk/remix
[ ] @clerk/types
[ ] @clerk/themes
[x] @clerk/localizations
[ ] @clerk/clerk-expo
[ ] @clerk/backend
[ ] @clerk/clerk-sdk-node
[ ] @clerk/shared
[ ] @clerk/fastify
[ ] @clerk/chrome-extension
[ ] gatsby-plugin-clerk
[ ] build/tooling/chore

Description

[ ] npm test runs as expected.
[ ] npm run build runs as expected.

Replaced by #1710

This PR has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.


how send content-type in header  use c++

i write c++/tls client use gprc , it work fine.  but when i use cdn, server can not recv request,
i found cf doc:
https://developers.cloudflare.com/support/network/understanding-cloudflare-grpc-support/
Requirements
Your gRPC endpoint must listen on port 443.
Your gRPC endpoint must support TLS and HTTP/2.
HTTP/2 must be advertised over ALPN.
Use application/grpc or application/grpc+<message type (for example: application/grpc+proto) for the Content-Type header of gRPC requests.

it seem to use  Content-Type header  in client , but i use
grpc::ClientContext context;
// Setting custom metadata to be sent to the server
context.AddMetadata("content-type", "application/grpc");

it not work!!!
who can tell me, how to solve this question?

gRPC automatically sends that content-type header, it's not something you need to do in your application.
I'm not sure why your CDN setup isn't working, but I don't think it's because of the content-type header.


Ben - feedback

Overall really nice job! I enjoyed learning about your project! I've provided some feedback below keeping with the recommended points in Homework 2, hope development is going well!
Design documents

DDS: clearly explained. Will the user be able to input preprocessed FASTQ or BAM files?
SRS: related to the above, it seems like the package will perform QC and then some basic analysis. If I have an already-QC'ed file, will I have the option to just run that through the basic analysis?
WBS: I really like how you have manageable tasks delineated so it is easy to edit and move through. Definitely something I am working on as well!
WBS: As you work through your prototype, it might be helpful to add more detail to the steps needed to implement the prototype in your WBS

Biological question

It may be a good idea to include a link to a HiChIP review or easily-accessible summary for those less familiar with the technique. I found a paper with a nice summary and figure here (https://academic.oup.com/bioinformatics/article/36/12/3625/5811234). Are there other tools that do similar things to what you are proposing? What are their advantages and disadvantages? Very cool technique!

Use of tools and resources discussed in class

I liked how you used the drawio tool. It was a clear way of communicating your project parameters and I am going to try to add one to my project as well

Feasibility of project

I like how you have stepwise attainable goals for the project (QC) and also a stretch goal (basic HiChIP analysis) if you have time. I would be interested to see how these goals evolve in the coming weeks as you continue learning and get deeper into the project

Progress

There appears to have been progress made on the goals in the last week, particularly around adding more functions and unit tests

Other

I really liked how you included clear installation instructions!
I recently switched from conda to mamba and highly recommend. Kelly has a nice summary in the Slack which I copied here as well

if you don’t already have a conda distribution installed, I highly recommended install the mambaforge distribution instead, as it comes with mamba already and you can just use mamba for everything rather than conda. https://github.com/conda-forge/miniforge#mambaforge
if you do already have a conda distro installed (e.g. anaconda3, miniconda3) and you don’t have any environments that you would be sad to lose, you can uninstall it (i.e. just delete the path to it) [I used the instructions here https://docs.anaconda.com/anaconda/install/uninstall/] and install mambaforge instead.

Thanks for the well-written feedback!
yes, preprocessed FASTQ or BAM files can also be used. There will be instructions on how to use those added
pre-QCed fastq/bam files should still undergo the library's QC to ensure no errors will happen :) but the QC is integrated into the script already, so there is no extra step for QC.
I can certainly add some information for current HiChIP analysis methods that other people have been using. So far there hasn't been other groups trying to do HiChIP and phasing together i think.
Mamba is something I am trying out as well :)


Results of cublas and sgemm_v3.cu differ

Dear @Liu-xiandong , the matrix multiplication results of the source code of sgemm_v3.cu differs from than of cublas. How can we solve it? Thanks.

Dear @Liu-xiandong , the matrix multiplication results of the source code of sgemm_v3.cu differs from than of cublas. How can we solve it? Thanks.

There seems to be some bugs at the moment, I will fix it next week.

@chaoming0625 Hello，the  bug is fixed. Welcome to use it.

Excellent!


make sure line_exists is not reported as error on versions where it does not exist

https://blogs.sap.com/2013/06/22/abap-news-for-release-740-new-internal-table-functions/
line_exists is from 7.40, SP02 and up

looking at the code, this is done, closing


The experience for leaving an organization seems to have changed

The experience for leaving an organization seems to have changed, and the documentation should be updated. In particular, the screenshots did not match my experience at all.

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

ID: b448cfd0-b673-7d04-047b-71aa62bebc27
Version Independent ID: 75118bff-94de-c9f9-7666-d2a6f4253b11
Content: Leave an organization as a guest user - Azure Active Directory
Content Source: articles/active-directory/external-identities/leave-the-organization.md
Service: active-directory
Sub-service: b2b
GitHub Login: @msmimart
Microsoft Alias: mimart

@elygre , Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will get back to you shortly.

I am having the same issue.  I want to be able to remove my account from an older directory.

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. The requested updates have not been made since the creation of this issue, and the timeline for resolution may vary based on resourcing, so we’ve created an internal work item to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary. #please-close


tapd/universe: fix potential concurrency issues related to concurrent SMT tree insertion

Related to https://github.com/lightninglabs/taproot-assets/issues/320 and https://github.com/lightninglabs/taproot-assets/issues/324
Since going testnet production, we've seen some issues with the default Universe server related to concurrent operations causing read/write hazards/anomalies.
We've seen instances on the prod server for certain assets where all the leaves are in place, but the universe root is actually incorrect. Here's an example:

Leaves: https://testnet.universe.lightning.finance/v1/taproot-assets/universe/leaves/asset-id/83f3564d453ca736368fb71a6989ae8bc547d5dbac5a9d3e4f983fe43040dc29
Root: https://testnet.universe.lightning.finance/v1/taproot-assets/universe/roots/asset-id/83f3564d453ca736368fb71a6989ae8bc547d5dbac5a9d3e4f983fe43040dc29

In this example we have ~100 leaves, but the root commits to 11, as if there were only 11 collectibles. This then results in an invalid proof being sent out, as the root doesn't match the leaf when inserted into the tree. We do always read the root from disk in order to generate an inclusion proof, but if the proof is invalid, then this will be as well.
We have a few options here:

Create a system for fine grained tree locking to ensure concurrent safe MS-SMT access.
Use PG advisory/table locking to lock all relevant tables for these operations.
Add internal consistnecy checks to ensure the server never serves incorrect data.
Update the default postgres concurrency settings.

I think the last one may be the best options, we we've seen this elsewhere when we want to do things like log the set of universe events. AFAICT, the default is actually read uncommitted, which can introduce a lot of issues: https://github.com/jackc/pgx/blob/c7733fe52efc445f13fe9dd3e1b829587400dfb2/stdlib/sql.go#L376-L389

Related issue in pgx: https://github.com/jackc/pgx/issues/1040

Another option is to just slap a read/write mutex on it for now as well.

Subissue idea
itest to assert sql and postgres methods match

Was fixed by #334


BLD: (DOC) fix LaTeX package expdlist bug causing blank lines in PDF

The way the line is broken in case of a long item label is buggy and
this shows in case of nested description lists.
We fix this, but the we then need a hack around Sphinx automatically
inserted \leavevmode which would again cause an extra blank line
after the long term label.
This suppresses about 2500 extra blank lines in PDF output...
closes #8652

This actually fixes also another bug, consider the empty Attributes section
\begin{quote}\begin{description}
\item[{Attributes}] \leavevmode
\end{description}\end{quote}

arising from the Classes in scipy.spatial (near end of subsection Nearest-neighbor Queries).
If we reduce the expdlist left margin parameter 60pt to for example 15pt so that Attributes is considered wide label, then PDF compilation halts with an error.
! Extra }, or forgotten \endgroup.
<recently read> \egroup

l.154946 \item[{Attributes}]
                             \leavevmode
? R
OK, entering \nonstopmode...

With the current PR, this problem does not show.

well, actually that last bug is due to the \\ inserted by expdlist in its buggy patch of LaTeX core \@item, because this is inside a table.

How about just removing the expdlist stuff completely?
The screenshot in #8652 looks fine.

@pv Indeed, that is clearly the simplest. I don't know if a modern and maintained LaTeX list package exists to achieve expdlist functionality. I do find that it is neat when the item contents go on next line after label if the latter is long one. Whereas LaTeX per default always starts contents at end of label, which can be short or long.
Package enumitem is widely-used, I looked quickly but could not conclude immediately how to use it here. Besides, scipy needs 'maxlistdepth' set-up in conf.py, and this is turned off by Sphinx when it detects enumitem. Because one can use enumitem own macros, but right now I have forgotten the details.

I think generally the less customization done in the preamble with tolerable output the better, so that we don't have to start fixing things when changes are made in sphinx.
However, as said also above, it's likely the pdf output is used by less people. However, if it's fun to fix then why not :)

@pv about removing expdlist completely see https://github.com/scipy/scipy/issues/8655#issuecomment-377962581 screenshot

Well, I'll leave the fine tuning to you, but least hackery in preamble is preferable.

The Sphinx style file fulllineitems environment, used for function signatures, actually contains in its macros \py@itemnewline, \pysiglinewithargsret, \pysigline a method to make list with labels on their own line. It would be nice for rendering of parameters by numpydoc, so the hackery will be moved upstream to LaTeX writer.
At https://github.com/jfbu/scipy/tree/no_expdl I suppress completely expdlist, then I use postprocess.py to malax the parameter lists to get similar output as for function signatures, i.e. the label is on its own.
For example:

But this might be a bit fragile, as it could be broken easily by numpydoc/Sphinx changes. Hence I am not doing a PR for that. However the first commit can be cherry-picked, it simply removes expdlist completely, using thus the LaTeX default way of doing things.

Was part of c881fdea7a7ef7c518b7


Request for opensearch_monitor resource

What resource do you need?
Support Opensearch Provider
https://registry.terraform.io/providers/opensearch-project/opensearch/latest/docs
Terraform Resource Names:

opensearch_monitor
opensearch_anomaly_detection
opensearch_dashboard_tenant
opensearch_dashboard_object

What is your use case?
Deploy monitoring logs resources and visualisations directly with the Crossplane Provider.

I don't believe this would fall into the scope of the provider-aws I believe you'd want to look at this provider https://github.com/dkb-bank/provider-opensearch

@blakeromano is correct, this provider is limited to the AWS resources only.
I'll close this issue for now.


Terminal lessh command

I am using pygments to do source highlighting on the command line.  I created an alias for lessh, so less that does source code highlighting. Sure, I could use a text editor like nano or vim, but I don't want there to be a chance that I'll edit the source file.
Bash Shell
I use a dark terminal environment, which works for "native" style.  Choose whichever style you prefer.
This uses the less environment variable LESSOPEN to pre-process the input file(s):
alias lessh='LESSOPEN="| pygmentize -f 256 -O style=native %s" less -M -R '

Default style version:
alias lessh='LESSOPEN="| pygmentize %s" less -M -R '

You could set the LESSOPEN variable globally and use less, but this will pipe every single file through pygmentize.  That will make less run slower because it will always run files through pygmentize.
Fish Shell
The fish shell equivalents is to create functions, funced lessh is used to create the function.  Test it out.  Then, save it to the fish shell with the funcsave lessh.
function lessh
       LESSOPEN="| pygmentize -f 256 -O style=native %s" less -M -R $argv
end

Note: I also made cath, which is basically alias cath="pygmentize".  Which doesn't really work like cat, so it does not concatenate the files and highlight. It just outputs a file to the screen highlighted.  I should choose a different alias for this.
I thought someone else might be able to benefit from this.  Could this be added to the documentation as a way to use pygments? (GNU Source-highlight documented a something similar, which I was inspired by.)

I like the idea, and can confirm the lessh alias also works on ZSH, as I have added it to my personal setup :).
I can create a PR, just would like to have approval from one of the maintainers first, if adding this to the documentation sounds like a good idea.

Closing. Fixed by merging PR #2425.


Update the Edit button

In the edit screen, the "edit" button should be renamed into "Save"
The button should be disabled (grayed out and unclickable) if no change has been done

Current state :

I'm not able to assign this issue to myself. I will work on it after complete my in progress tickets.

PR #120


Loop option missing from API Documentation

Hi, I just wanted to note that the github instructions DO mention the loop option, however the API docs DO NOT mention the loop option.
https://ssense.github.io/vue-carousel/api/
Thanks for such a great component, and all that you do!

Yeah that has been in the plan to add that to official docs for weeks now and it keeps getting pushed back, thanks for reminding me!

I've created #262 to cover the doc updates, going to close this in favor of that. Feel free to make any comments/suggestions there!


tooltip for class comments

As a user I want to see the complente class comment I found right in Algernon in order to avoid switching to another program.
Original Description
so that the user can identify class comments in search results easier . Idea was to show a tooltip on the right side of the result list
Acceptance Criteria

[ ] A class comment in the search results can be expanded to be fully read in Algernon.

For inspiration you might want to look into Squeak's Autocompletion.
It provides a detail view for results using Ctrl+right.


[zuul][tempest] Re-enable the sg_core integration test

Depends-On: https://review.opendev.org/c/openstack/telemetry-tempest-plugin/+/909459
Depends-On: https://github.com/openstack-k8s-operators/telemetry-operator/pull/284

Once this passes, we can merge it into the main PR for the tempest job.

check-rdo

The opendev patch is merged in master now. I resolved the merge conflict, that was here and zuul is pointing to master now. If this passes CI, then I think we can merge this.

/retest
Memcached tried to pull an invalid image.
A bump for openstack-k8s-operators was just merged https://github.com/openstack-k8s-operators/telemetry-operator/pull/332 Lets see if this fixed the CI.

/approve
/lgtm


clear marker state machine

Fix one more bug discovered during local activity stress test.
I run test workflow that receive 1000 signals, each signal generate 100 local activities. As the workflow making progress, the perf degraded that it takes longer and longer to process each signal. (initially, it took about 10ms to process 100 local activities.)
We use marker state machine to generate decision, we are not drive marker state machine to completed state by history events. Once marker decision is sent, the markerStateMachine is considered as completed.
For SideEffect/Version markers, when the history event is applied, there is no marker decision state machine yet because we preload those marker events.
For local activity, when we apply the history event, we use it to create the marker state machine, there is no other event to drive it to completed state.
So for marker state machine, the sent state should be treated as the completed state and it should be removed once it is in sent state.

Coverage increased (+0.05%) to 67.172% when pulling 9af06669d2f0677cef3c71c3eb276a429975176f on yiminc:clear_marker_state_machine into 53b7af92027b72f16fe9127460faaf9ea3fc1faf on uber-go:master.


Fix "active time" acquisition

Active time could be defined as the time spent by a user to complete a single action using the UI.
Actions are:

ROI creation
Clinical annotation of existing ROIs

Time for each action is now acquired using two timestamps:

creation_start_date acquired client side
creation_date acquired server side

Since client's and server's clock could be not syncronized, these timestamps could lead to an incorrect "active time", sometimes producing even negative values if server's time is behind client's.
To fix this problem, the best thing to do is probably to move the acquisition of the "active time" client side by measuring it with something like the performance.now() method https://developer.mozilla.org/en-US/docs/Web/API/Performance/now
The acquired measure should replace the creation_start_date timestamp with an active_time float field used to express the elapsed time in milliseconds or seconds.

We could also add a further timestamp, acquired client-side, to register when the activity involving the usage of a specific tool is completed (which means that the acquired data is submitted to the server).
The timestamps and times collected for each action should become:

creation_date the server-side timestamp automatically acquired when an object is saved into the DB
action_start_time which will replace creation_start_date
action_complete_time acquired client-side, will register when the action is completed by clicking the save button to push the data on the server
action_duration the time spent by the user to perform and complete the task (a little bit redundant because is calculated as action_complete_time - action_start_time)


Login Process Validator PDD Added

Login Process Validator PDD Added

5 Points are assured. No Errors found by AI system, you must be really good in writing uipath code. An expert need to review your code and it will be done before next sunrise :)


Add hover effects and improve UI

Is your feature request related to a problem? Please describe.
The UI is not so good and I think there should be hover effects on the cards
Describe the solution you'd like
I will improve the UI and add hover effects
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

I want to do this task under SSOC 2024

Duplicate #61


ImportError: cannot import name 'load_json' from 'json2parquet' on windows

 from json2parquet import load_json, ingest_data, write_parquet, write_parquet_dataset   File "directory/file_name.py", line 5, in <module>     from json2parquet import load_json, ingest_data, write_parquet, write_parquet_dataset ImportError: cannot import name 'load_json' from 'json2parquet'
I have installed successfully json2parquet, but unable to import these function's.

Hmm, not super familiar with using Python on windows.  How did you install the library?

@andrewgross thanks for your reply,

Hmm, not super familiar with using Python on windows. How did you install the library?

as mentioned in Readme Document. using pip

Hmm, I am not sure why its having issues.  The name of your file when you are importing is not json2parquet is it?  I often make that mistake and trip up pythons import rules.


Store GPS coordinates of user

Describe the solution you'd like
I have a client requirement to store the GPS (longitude/latitude) coordinates when clocking in/out to verify location. I am thinking it it would be an "Active" button to pull the coordinates and store in the database.

Have you already seen the geosearch plugin?
https://github.com/saltcorn/geosearch

@christianhugoch I did notice that. But I thought it was more of a points on a map display view. I installed and see the "use current location" - looks great and didn't know that was an option. Thank you!


Appstudio update test-component-pac-msql

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-msql-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
init

✅ Succeeded
13 seconds
clone-repository

✅ Succeeded
25 seconds
build-container

✅ Succeeded
8 seconds
deprecated-base-image-check

✅ Succeeded
10 seconds
clair-scan

✅ Succeeded
9 seconds
inspect-image

✅ Succeeded
46 seconds
clamav-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
9 seconds
label-check

✅ Succeeded
8 seconds
show-sbom

✅ Succeeded
5 seconds
show-summary


TokenizedBuffer#bufferRangeForScopeAtPosition appears to be using incorrect position

I run into this with https://github.com/blakeembrey/atom-dash/issues/26 and solved it using https://github.com/blakeembrey/atom-dash/commit/750f84560f62e824b87fbd2012699d50dd3446e2. It seems a method are using the wrong position object. I suspect TokenizedBuffer#bufferRangeForScopeAtPosition is not calculating the position correctly.
The issue starts with https://github.com/atom/atom/blob/278e988cca2bba2b0f5f13d1833786ac59cb989e/src/text-editor.coffee#L2565-L2566 and ends up using https://github.com/atom/atom/blob/4ff3b6d5efd213f491b82317316bdbffd39efbdf/src/tokenized-buffer.coffee#L478. As it passes cursor buffer position, if you have tabs enabled it will never get the position back correctly and it ends up being off by however much your tab spacing is.
Edit: Here's an image in action:

toggle-quotes also doesn't work for tab indented lines.
Looks like bufferRangeForScopeAtPosition returns row in buffer coordinates, but col in screen coordinates.
See also my comments on https://github.com/atom/toggle-quotes/issues/27.

This is a duplicate of #9648 and is now fixed.

This appears to still be an issue, and is the underlying cause of https://github.com/atom/toggle-quotes/issues/27 (refs https://github.com/atom/toggle-quotes/issues/27#issuecomment-156734463).
It seems that this method still returns screen columns, which totally throws off the returned range.

I have reproduced this issue also. Definitely an issue.

Any updates on this one?


1.c - Add initial testing

User story:
As a developer, I want to refactor the test suite from our previous Chirp! CLI application. That is, keep all those test cases around that still apply to the new Razor Page application. Likely, these are some unit tests and perhaps a few suitable integration tests. Due to their dependency on CLI parameters and output, the previous end-to-end tests likely do not apply anymore, so that our test is up to date.
Add some new API tests. These should send GET requests to the endpoints for public and private timelines (/ and / respectively) and assert that certain information is in the HTML text of the response in the HTTP body.
Acceptance criteria:

Our current test is up to date.
Add new API tests, that send GET requests to the endpoint for public and private timelenes and assert certain information in the HTML text of the response in the HTTP body.

Other:
Add some new API tests. For example:

https://github.com/itu-bdsa/lecture_notes/blob/main/sessions/session_05/data/dump.sql)
a response body for a GET request send to / shall contain a cheep written by Helge and the content of a cheep shall be Hello, BDSA students! Similarly, a response body for a GET request send to /Rasmus shall contain a cheep written by Rasmus with the content Hej, velkommen til kurset.

---> pseudo code added + folder added , so essentially tests are ready to go.
due to internal server error 500, TA's said they need to figure out a fix for this, since all were having this issue.
So we will move to close and then further complete the tests when entering week 6's assignements.


Assertion t >= 0 && t < n_classes failed.

Sorry to bother you, I have encontered a probelm: Assertion t >= 0 && t < n_classes failed when I try to use your model to my datasets, I have changed the num_class in my datasets, but I still face the problem, can u help me? here is my change:
self.classes = list(range(2))
self.image_classes = calc_classes_per_image(self.masks, 2, cache_file)

Did you also specify num_classes in the arguments for the model?

Hi，Did you end up getting that fixed?

Yes, it's the label's number in the dataset, I have changed the number, it works!
盛星
@.***

------------------ 原始邮件 ------------------
发件人:                                                                                "YuvalNirkin/hyperseg"                                                                                @.>;
发送时间: 2023年12月24日(星期天) 下午2:59
@.>;
<EMAIL_ADDRESS>主题: Re: [YuvalNirkin/hyperseg] Assertion t >= 0 && t < n_classes failed. (Issue #26)
Hi，Did you end up getting that fixed?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


webui - corrections to be coherent with APIv2

@cyril1er This pull request can not be merged. Please have a look


chore: update production mode usage of webpack in readme for dist folder

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[x] Other, please describe: This pr updates the Development vs. Production Mode section in which webpack usage for version 4+ is added.

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] It's submitted to the dev branch for v2.x (or to a previous version branch), not the master branch
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[ ] All tests are passing: https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#development-setup
[ ] New/updated tests are included

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

chore: update production mode usage of webpack

@czf1998 Please stop spamming copy paste messages or we will have to block your account. Thank you.


[Bug] App Crash: IllegalStateException

Added a hue filter and removed it and then left the current image without saving and in a crashed.
Device: Pixel 5 (google - redfin), SDK: 34 (14), App: 2.3.3-alpha03 (90)java.lang.IllegalStateException: item can't be savedat y0.k.e(SourceFile:1)at n.n.a(SourceFile:38)at p0.i0.a(SourceFile:1)at p0.u.b(SourceFile:49)at p0.v.k(SourceFile:58)at p0.v.j(SourceFile:6)at p0.c2.g0(SourceFile:173)at androidx.compose.ui.platform.a1.doFrame(Unknown Source:6)at androidx.compose.ui.platform.y0.doFrame(SourceFile:48)at android.view.Choreographer$CallbackRecord.run(Choreographer.java:1337)at android.view.Choreographer$CallbackRecord.run(Choreographer.java:1348)at android.view.Choreographer.doCallbacks(Choreographer.java:952)at android.view.Choreographer.doFrame(Choreographer.java:878)at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:1322)at android.os.Handler.handleCallback(Handler.java:958)at android.os.Handler.dispatchMessage(Handler.java:99)at android.os.Looper.loopOnce(Looper.java:205)at android.os.Looper.loop(Looper.java:294)at android.app.ActivityThread.main(ActivityThread.java:8177)at java.lang.reflect.Method.invoke(Native Method)at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:552)at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:971)Suppressed: sb.h: [p0.k1@7605bae, androidx.compose.ui.platform.x1@68a3d4f, r1{Cancelling}@1a45cdc, z0@56ec1e5]

Left it and closed the app?

Left it and closed the app?
Yes, I exited out of editing the image without saving and the app crashed. I'll try to recreate this issue with the new version.  But I believe that's all I did.

Left it and closed the app?

Yes, I exited out of editing the image without saving and the app crashed. I'll try to recreate this issue with the new version.  But I believe that's all I did.
now to see if it depends on what image. I'll update this comment if I do it again.
Cannot replicate in 2.3.3-beta01 (91)
Suggest closing ticket.

I think this is some kind of rare library glitch, I'll close it for now and then we'll see


#155 fix ignored tests in branch 48 otp

connect to #155

@vaa25 fixed

PASSED

@AndriyBaibak time tracking: 15 minutes spent

@alex-anakin done

@IgorMaksymov see this comment

@alex-anakin please, reviev ASAP, its blocking 3 PR's

PASSED

@AndriyBaibak time tracking: 10 minutes spent


[Feature Request] Long tap to toggle wakeword detection

Checklist

[X] I have filled out the template to the best of my ability.
[X] This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
[X] This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.
I don't always want the device listening for a wakeword as sometimes there are false positives - for example when I am watching a show or movie, or if I'm having an important call. There is an existing switch exposed in Home Assistant of course, and that works, but sometimes I want to be able to toggle detection on the device itself.
Describe the solution you'd like
I would like to be able to do a longer tap on the onju which will toggle wakeword detection, and have there be visual feedback of the toggle coming from the LEDs.
Describe alternatives you've considered
I have a semi-working version of this in a fork, however the visual feedback when the toggle disables detection does not seem to work properly (the LEDs all flash red). I think it is something wrong with the way I have configured it. If this sounds like a good feature that people want, I can make a PR.
Additional context
Are all of these fields really required? Anyway, I was just going to make a PR but wanted to check if this wasn't too much 'feature creep', and if others had recommendations for further enhancements to the feature request.

What's wrong with the physical mute switch already built in?

Nothing the matter with it really - I just find that I never use it. Doing a brief touch on the top is far more convenient :)
On Thu, Apr 18, 2024, at 6:09 PM, Jeremy Herbison wrote:

What's wrong with the physical mute switch already built in?
—
Reply to this email directly, view it on GitHub https://github.com/tetele/onju-voice-satellite/issues/41#issuecomment-2065411648, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQFMUVWT6B5ZUYP3Y7P6JNLY6BAB5AVCNFSM6AAAAABGOBGS32VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRVGQYTCNRUHA.
You are receiving this because you authored the thread.Message ID: @.***>

You already have two methods of toggling the wake word - one physical and one virtual. They both need to be on for wake word to be running.
You can automate on the virtual one - "when a movie starts, turn off wake word". You can also expose it and do "hey Nabu, turn off use wake word" or another custom command. You can also tap the top and say your custom command (turn on/off the wake word.
I feel like long tapping for this purpose would be a bit of an overkill.

All good points! I guess it comes down to personal preference, as I have found that I never use the physical mute switch, and haven't gotten around to automating the toggling that much yet. For now I will keep using the long tap in my fork since I like it :)


[Host.Outbox.Sql] Abort publishing of unknown message types

OutboxSendingTask will immediately abort sending a message when the MessageType fails to resolve.
Changes:

MessageType resolution moved out of SqlOutboxRepository and into OutboxSendTask and OutboxForwardingPublishInterceptor.
When OutboxSendTask fails to resolve the 'MessageType', an error is logged and the message is flagged as 'DeliveryAborted'.
When OutboxSendTask fails to resolve the associated bus. The affected messages are also immediately aborted (previously, n attempts would be made before doing the same).

@EtherZa 2 minor comments added.
Could you please add a note about how a missing message type will be handled to the outbox docs?
Then I am good to merge.
Thanks for your contribution.

@EtherZa any chance you could increase the coverage on SqlOutboxRepository around the Abort methods and remove the unused variable flagged by Sonar?

@zarusz I have resolved the Sonar complaint and pushed the boundary again to get the requested coverage on SqlOutboxRepository by introducing TestContainers -I am assuming that you are keen to move in this direction as per your comment in 277.
Please let me know if you are happy with the approach and I will squash/sign the commit.

@EtherZa, I appreciate you pushing this over the finish line despite project deadlines!
Yes, happy to merge. Please squash and I am going to release a minor release soon.
Thanks!

Can you also resolve the conflict, I just merged another PR @EtherZa ?

@EtherZa, I appreciate you pushing this over the finish line despite project deadlines!
Yes, happy to merge. Please squash and I am going to release a minor release soon.
Thanks!

No worries. I've rebased and pushed. All should be good to go now :)


Create Market: Replace "trading fee" with "taker fee" and re-define "maker fee" to be independent

Currently we have:
trading fee: % of total value of trade (ie 2%)
maker fee: % of trading fee (ie 50%)
We are simplifying this paradigm so that a taker fee and maker fee can be specified more independently of each other, and dropping the concept of trading fee from the ui. In the market creation process, the fee inputs should be:
taker fee: % of total trade value (ie 2%)
maker fee: % of total trade value but not higher that 50% of taker fee (ie 1%)
The only limitation is that the maker fee cannot exceed half of the taker fee. Ex: if taker fee is 3% maker fee can be between 0 and 1.5%.
These fees will also be passed into augurjs for the createMarket functionality, so this issue is dependent on this one: https://github.com/AugurProject/augur.js/issues/85
And the related UI work: https://github.com/AugurProject/augur-ui-react-components/issues/36

PRs #675 + AURC #42 resolve this issue -- closing.


Apply fixes from StyleCI

This pull request applies code style fixes from an analysis carried out by StyleCI.

For more information, click here.

StyleCI 的规则改了吗 🤔


do a little bit more loggin in run_test.sh

r? @cjb
A logging change for the convenience of https://github.com/keybase/client/pull/3902. Waiting for CI to confirm that it logs Go version 1.7.

Doesn't really need review, mostly putting up to watch the CI run.

Failure in TestProvisionGPGSwitchToSign. Hmm. I'm going to declare that "officially not the fault of this echo statement" and rashly charge ahead.

f6211bc15d65d63245ec76c0c3255e3916ae160e

...does GitHub delete my test runs after I close a PR?


basic support for parsing Doxygen \param and \returns commands

Bug/issue #, if applicable: rdar://69835334
Summary
This PR introduces a new parser mode, adding support for a minimal set of Doxygen commands, to facilitate transitioning from alternate documentation systems into systems that use Swift-Markdown, such as Swift-DocC.
The parsing is gated behind a new parse option, currently named parseMinimalDoxygen (name open for bikeshedding). When both this and parseBlockDirectives are given, the block-directive parser will start looking for a specific set of Doxygen commands. This PR adds support for two basic commands: \param and \returns (though the latter will also recognize \return and \result, as those are synonyms in Doxygen itself).
To facilitate testing, this PR also extends the markdown-tool dump-tree command with an --experimental-parse-doxygen-commands flag which turns on the behavior.
I also fixed two other issues in the block directive parser in the course of this PR:

If a document begins with a code fence, the contents of that code fence were incorrectly scanned for block directives.
If a line run in a document or block directive included a line that didn't start on the beginning of the line (e.g. when a directive is written on one line), the source location was incorrectly given as the beginning of the line.

Dependencies
This requires another PR to integrate with Swift-DocC, which is yet to be written.
Testing
As the new functionality is gated behind a new ParseOption, testing should also ensure that existing functionality is not changed without the new option set.
Steps:

Write a Markdown file with Doxygen command content in it. See the tests added in this PR for examples.
swift run markdown-tool dump-tree --parse-block-directives --experimental-parse-doxygen-commands test.md
Inspect the result to ensure that it looks valid.
Repeat with --source-locations and ensure that the source locations for command descriptions look valid.

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran the ./bin/test script and it succeeded
[x] Updated documentation if necessary

@swift-ci Please test

@swift-ci Please test

@franklinsch Would you mind taking another look? Your comment reminded me that i hadn't added support in the markup formatter 😅

Looks great, thanks!


(Question) Pencil Touch vs Finger Touch

Hola hola,
We've had issues handling the Pencil vs Finger scenario for iPads. There is the slightest delay when sending touch info from Swift to JS.
Does this library address that issue?
Thanks in advance.

As far as I know, we haven't tried the pencil yet. Let us know if you are experiencing any issues with your use case.

I've tried the pencil a lot - tested the drawing example using the pencil and it didn't seem to have any of these problems. We treat the pencil as a regular touch event with pressure.

I think the question was about distinguishing between the skin and the Apple Pencil - so that your palm doesn't trigger a drawing while writing with the Apple Pencil?


Consolidate text to allow copy/paste

One huge benefit of text-based screenshots over PNGs is the ability to select / copy contents and paste them elsewhere.
ansisvg generates a dedicated <text> element for each character, and if text is selected / copied from the rendered SVG then the copied selection includes a whitespace after each character.
example: if "WORD" is selected/copied, then the clipboard contents are "W O R D"
ansisvg could detect consecutive characters that have the same style attribute and consolidate them into a single <text> element to facilitate copy/pasting.

Yes that would be great and i've tried to do it a couple of times but have failed to produce output that looks good in all major browsers and svg viewers. If i remember correctly it was tricky come up with a compatible way to position characters exactly when not positioning them one by one. Will try to remember more exactly what the problem was.

The issue here is probably that the calculated positions must exactly match the positions of the rendered characters. When the position is explicitly given for each single character (like it is now with dedicated <text> elements), then any deviation from the spacing in the rendered font is just between two adjacent characters, so mostly negligible. But if we render whole words per <text> element then any deviation will accumulate, so the space between words will appear either much too small or much too large. I believe it can be fixed with tweaking the settings for fontsize and charboxsize arguments, but I'm not sure if it's 1) working for all viewers and 2) if it's possible to come up with "working" default combinations of fontsize / charboxsize. Probably also a kind of depending on the currently used font. :sweat_smile:

How about a "experimental" option that just consolidates <text> elements if they share the style attribute, and does nothing else. The correct combination of fontname, fontsize and charboxsize is then up to the user.

Mm could try that. I usually try things by generating some svg output with ansisvg and then manually mess around with it.
Now i got some vague memory that whitespace or possibly some special unicode like drawing-ish characters was hard to get to fill up space correctly somehow.

Thanks for the quick implementation! A short test revealed that the output indeed looks the same as before (at least with my custom fontfamily/fontsize settings), and the copy/pasting now works as intended. Only thing missing is now that the linefeeds disappear from the original content. (I suppose we can't just put a LF into a tspan at the end of each line?)
I also see some instances where colors are not transferred (esp. when using kitty's copy_ansi_to_clipboard instead of just piping) but that's probably not related to this particular feature and needs some deeper investigation.

Thanks for the quick implementation! A short test revealed that the output indeed looks the same as before (at least with my custom fontfamily/fontsize settings), and the copy/pasting now works as intended. Only thing missing is now that the linefeeds disappear from the original content. (I suppose we can't just put a LF into a tspan at the end of each line?)

👍  yeap with #7 it should look the same, no consolidating tricks it needed, just change to tspan and some other things. Haven't look deep into preserving new lines yet, could possibly be just to keep/recreate literal \n somehow.
Maybe can merge #7 and do line literals separetly?

I also see some instances where colors are not transferred (esp. when using kitty's copy_ansi_to_clipboard instead of just piping) but that's probably not related to this particular feature and needs some deeper investigation.

That i didn't even consider. I'm mostly testing with iterm2 which seems to not copy styling at all. If it do support copying colors i would guess it should just work as changing styling in svg will require use of separate text or tspan elements.

Yes, but if consolidating was somehow an option it would greatly reduce file size as well as CPU load esp. for tools like Inkscape. However I understand that it'd be opening a whole different can of worms. Just going to tspan gives the better bang for the buck.

Mm maybe look into that later once we get something working.

Or translate to
 in worst case (not sure if SVG supports that, however)

No
 in svg unfortunately. Did some tests and i get a feeling literal new \n gets collapsed into a whitespace, so one probably has to generate <text> elements per line or possibly use html entity like
 etc.

Note this is about picking up the colors from the console content and rendering them in SVG, so completely unrelated to the "copying" issue.

Doh of course :)

I have a few custom keybindings, Shift-Ctrl-x to copy with esc sequences (kitty copy_ansi_to_clipboard), F1 to run clipboard through ansi2html (Python) and open it with Firefox, F2 to run it through ansi2html and save it to $HOME/Pictures, F3 to run it through ansisvg and save it to Pictures.
The thing is, ansi2html always gets the colors right, regardless where it gets its input from - but ansisvg misses some of them if it gets its input from the clipboard, not directly from the pipe. But I have to investigate that a bit deeper before I file a new issue.

Huh that is strange. Any pattern what styling it missing? If you could dump the input sent via clipboard vs pipe to a file somehow that would be great for debugging. Maybe can temporary change the clipboard program to shell scrip that  redirects to a file?

I opened a different issue to keep this one on topic.

@patrislav1 could you give #7 a try again? pushed a version that has a text elements per line, seem to fix new line issue

Rebased and merge on top of the cowsay fixes also. Let me know how it works out!


Aimbotz map not showing skins

Hi. I enter Aimbotz and some gun doesn't have skin such as: USP-S, Five-Seven. Can u fix this?

+1, I presume its to do with a teamcheck.

In aimbotz it may seem like you are on ct side, but the server thinks you are on t side (its super weird)
I recommend you use another map that isnt aimbotz to test skins (crashz' Crosshair Generator v3 might be a good one)

Let's hope daniel fix this :)

You are on T side with CT sleeves and gloves.

It shouldn't as it doesn't even display real skins.
that's because this map messes up with team id settings.


Property 'parent' in type 'SuperTabsComponent' is not assignable to the same property in base type 'NavigationContainer'.

Im using supertabs in my application and it was working fine but im upgrading my ionic 3 app to ionic 4. but im getting the following error
ERROR in node_modules/ionic2-super-tabs/dist/components/super-tabs.d.ts(153,5): error TS2416: Property 'parent' in type 'SuperTabsComponent' is not assignable to the same property in base type 'NavigationContainer'.   Type 'NavControllerBase' is not assignable to type 'NavController'.     Types of property 'config' are incompatible.       Type 'import("E:/Work/Projects/COJ/ConsumerApp/node_modules/ionic-angular/umd/config/config").Config' is not assignable to type 'import("E:/Work/Projects/COJ/ConsumerApp/node_modules/ionic-angular/config/config").Config'.         Types have separate declarations of a private property '_c'.
The type script versions that i am using is mentioned below
+--<EMAIL_ADDRESS>| --<EMAIL_ADDRESS>|   --<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>| --<EMAIL_ADDRESS>|   --<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>| --<EMAIL_ADDRESS>--<EMAIL_ADDRESS>Isn't this supporting for ionic 4?
Thanks in advance for the solutions.

This library is not compatible with Ionic 4 yet. Please watch the following branch for updates: https://github.com/zyra/ionic2-super-tabs/tree/master


Launch from VS 2017, debug js,ts and cs files

Hi, nice template, but there's a question:
How can I launch project based on this template from VS 2017 with debugging?
Something like here https://github.com/aspnet/JavaScriptServices
Here I can press IIS button and get app launched in debug mode, so i can put breakpoints etc.
Thanks.

Hi @kk4fun
Yes, I debug using Chrome inside VS2017

Hmm, so how can i do it? If i just run from VS IIS it just launch server, not client part.

In VS2017, click the black arrow on the Start button inside visual studio and switch the Web browser to chrome

Ofcource i tried it :)
it just launched server part http://localhost:5000/index.html
but no client in VS debug mode :(
client in template on 8080 port
Question is , how can i do it ?

You would debug the client-side files in Chrome as they are loaded and executed in Chrome. Have you tried that? BR


npm install not working - Could not resolve dependency

I'm getting errors when trying to install the packages which is related to the @twilio/paste packages.
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>I got a couple of these.

Same here
see the image

install with npm install --legacy-peer-deps

seems npm install is working with this Pull Request

Use  yarn no npm


[INTEL MK] Updating MKL CPU CI build and test.

Setting  KMP_BLOCKTIME to 0  to  workaround the effects of an oversubscription of OpenMP threads caused by executing multiple tests concurrently.

@gunan I added comments as requested. Thanks.


How to download latest build?

https://ci.appveyor.com/project/avast-tl/retdec/branch/master/artifacts
is empty
"The build job does not contain any artifacts."

@s3rvac thanks:)
no for windows-32bit executables ???
(for test with 32bit application)

In TeamCity on Windows, we only build RetDec to run on 64b Windows. If you still use 32b Windows, you have the following two options:

Download the latest stable build (v3.2). We provide both 32b and 64b builds for stable releases.
Build and install RetDec from sources (requires several dependencies).

As for your other question, RetDec currently only supports decompilation of 32b applications (support for decompilation of 64b applications is under development). You can use both the 64b and 32b versions of RetDec to decompile 32b applications. Thus, the choice between a 64b and 32b version of RetDec only depends on your operating system (64b Windows can run both 64b and 32b builds of RetDec, while 32b Windows can run only the 32b build of RetDec).


[AutoGen Studio] Add Friendly Title to Sessions

Currently, sessions in autogenstudio have the associated workflow title and a session id.
This issue will explore methods to generate a friendly name for each session

Allow edition of session names
Stretch

track conversation in UI, if there are more than 2 user messages,
Either use first message as friendly name or generate friendlyname using llm.
Update session data descriiption field in db

First time contributor here, can I take this?


"/docker-entrypoint.sh": permission denied

Error response from daemon: Cannot start container fce4fd332592b29a99ea41da6f065b188b5c7549d472e384f0193d017a5dd6e7: [8] System error: exec: "/docker-entrypoint.sh": permission denied

docker version, docker info, uname -a? (and I assume percona:latest and you did a docker pull percona again). Also what is your docker run line?

+1 having this same issue.

I am unable to replicate this. What does the yaml file have for the wcdevmysql service? When I run the following, it starts up just  fine.
docker run -d --name some-percona -e MYSQL_ROOT_PASSWORD=secret percona

I'm using docker-compose and docker-compose up using the files from https://github.com/docker-library/percona/tree/301bc073bee3e7e89fd96edd324581aabe3b3b73/5.5

@wreiske Looks like this might be a Windows problem with execute permissions.
Why are you having docker-compose build this image instead of using the percona image directly from Docker Hub?

I'm assuming that based on your comment that you have docker-compose "using the files from https://github.com/docker-library/percona/tree/301bc073bee3e7e89fd96edd324581aabe3b3b73/5.5".

I'm using the following config now which is working for me.
    percona:
    image: percona
    hostname: wc.mysql

I wanted to use a custom build Dockerfile to setup custom permissions and copy settings like a custom my.cnf, i'm not sure how to do that with using an image.

You should be able to use a simple Dockerfile with something like "FROM
percona" and then copy the relevant configuration into "/etc/mysql/conf.d"
using either "RUN echo ... > /etc/mysql/conf.d/some-file.cnf" or "COPY
some-file.cnf /etc/mysql/conf.d/".  Combine that with automated builds (
https://docs.docker.com/docker-hub/builds/) and repository links (
https://docs.docker.com/docker-hub/builds/#repository-links) and it's
reasonably easy to have an up-to-date image built "FROM" this one with
custom configuration.

Awesome! I'd say my issue is resolved, not sure about the first user's post though. Thanks guys.

It using same permission level for file that file have on host machine. If you will set that it is executable on host, it will work fine.


[Bug]: Listener port not starting interminently.

Description
We Have two services wired into a single HTTP listener instance in a single ballerina package.
When running with two services we can see the listener start intermittently.
When not starting happens As per debugging [1] not get invoked at the error state. Please refer to the ballerina module[2] used to debug.
[1] https://github.com/ballerina-platform/module-ballerina-http/blob/master/native/src/main/java/io/ballerina/stdlib/http/api/service/endpoint/Start.java#L50
[2] - https://github.com/wso2/apk/tree/main-abandoned/runtime/runtime-domain-service/ballerina
Steps to Reproduce
No response
Affected Version(s)
No response
OS, DB, other environment details and versions
No response
Related area
-> Runtime
Related issue(s) (optional)
No response
Suggested label(s) (optional)
BUG
Suggested assignee(s) (optional)
@TharmiganK @shafreenAnfar

@warunalakshitha As per the offline discussion, this issue is not there when we use dynamic listeners and start the listeners manually inside the init() function.
The issue is there when we used the service declaration like this :
service / on ep0 {
    resource function get health() returns http:Ok {
        ...
    }
}

For service declarations, the start() is called from runtime right? Can you check this from runtime point-of-view?

We will check from runtime side.