rz gate in OpenQASM 2 maybe has the wrong matrix representation

Environment

Qiskit Terra version: 0.25.1
Python version: 3.10.12
Operating system: Ubuntu 22.04.1 LTS
Qiskit version: 0.44.1

What is happening?
The rz gate seems to have the wrong matrix representation when imported from an OpenQASM 2 file. In OpenQASM 2, the rz gate is defined to be rz(phi) = u1(phi) = diag(1, exp(i*phi). Specifically, it is defined in the "qelib1.inc" header file as: gate rz(phi) a { u1(phi) a; }, which suggests it should have the same matrix representation as u1.
However, it seems to have the matrix representation rz(phi) = diag(exp(-i*phi/2), exp(i*phi/2)) instead, differing by a (global) phase of -phi/2. I'm aware that the original paper says rz is equivalent to u1 "up to a global phase", but the code in the header file suggests that they should actually be identical.
Note that the definition which differs from u1 by a global phase of -i*phi/2 is used in OpenQASM 3. In particular, in the "stdgates.inc" header file, rz is defined as: gate rz(lambda) a { gphase(-lambda/2); U(0, 0, lambda) a; }. The presence of the gphase(-lambda/2) here seems to suggest that the matrix representation of OpenQASM 2's version of rz should not have this global phase.
(The u1 gate is defined as gate u1(lambda) q { U(0,0,lambda) q; } in both header files.)
How can we reproduce the issue?
The code below outputs the matrices for rz, controlled-rz, u1, and controlled-u1, respectively.
from qiskit.circuit import QuantumCircuit
from qiskit import quantum_info

s1 = """
OPENQASM 2.0;
include "qelib1.inc";
qreg q[1];
rz(pi/2) q[0];
"""
c1 = QuantumCircuit.from_qasm_str(s1)
print(quantum_info.Operator(c1).data)

s2 = """
OPENQASM 2.0;
include "qelib1.inc";
qreg q[2];
rz(pi/4) q[0];
cx q[1],q[0];
rz(-pi/4) q[0];
cx q[1],q[0];
"""
c2 = QuantumCircuit.from_qasm_str(s2)
print(quantum_info.Operator(c2).data)

s3 = """
OPENQASM 2.0;
include "qelib1.inc";
qreg q[1];
u1(pi/2) q[0];
"""
c3 = QuantumCircuit.from_qasm_str(s3)
print(quantum_info.Operator(c3).data)

s4 = """
OPENQASM 2.0;
include "qelib1.inc";
qreg q[2];
u1(pi/4) q[0];
cx q[1],q[0];
u1(-pi/4) q[0];
cx q[1],q[0];
"""
c4 = QuantumCircuit.from_qasm_str(s4)
print(quantum_info.Operator(c4).data)

What should happen?
My expected output is:
[[1.        +0.j         0.        +0.j]
 [0.        +0.j         0.        +1.j]

[[1.        +0.j         0.        +0.j         0.        +0.j  0.        +0.j        ]
 [0.        +0.j         1.        +0.j         0.        +0.j  0.        +0.j        ]
 [0.        +0.j         0.        +0.j         1.        +0.j  0.        +0.j        ]
 [0.        +0.j         0.        +0.j         0.        +0.j  0.        +1.j]]

[[1.        +0.j         0.        +0.j]
 [0.        +0.j         0.        +1.j]

[[1.        +0.j         0.        +0.j         0.        +0.j  0.        +0.j        ]
 [0.        +0.j         1.        +0.j         0.        +0.j  0.        +0.j        ]
 [0.        +0.j         0.        +0.j         1.        +0.j  0.        +0.j        ]
 [0.        +0.j         0.        +0.j         0.        +0.j  0.        +1.j]]

The actual output is:
[[0.70710678-0.70710678j 0.        +0.j        ]
 [0.        +0.j         0.70710678+0.70710678j]]

[[1.        +0.j         0.        +0.j         0.        +0.j  0.        +0.j        ]
 [0.        +0.j         1.        +0.j         0.        +0.j  0.        +0.j        ]
 [0.        +0.j         0.        +0.j         0.70710678-0.70710678j  0.        +0.j        ]
 [0.        +0.j         0.        +0.j         0.        +0.j  0.70710678+0.70710678j]]

[[1.000000e+00+0.j 0.000000e+00+0.j]
 [0.000000e+00+0.j 6.123234e-17+1.j]]

[[1.        +0.j         0.        +0.j         0.        +0.j  0.        +0.j        ]
 [0.        +0.j         1.        +0.j         0.        +0.j  0.        +0.j        ]
 [0.        +0.j         0.        +0.j         0.70710678-0.70710678j  0.        +0.j        ]
 [0.        +0.j         0.        +0.j         0.        +0.j  0.70710678+0.70710678j]]

What's strange is that the controlled-u1 gate's matrix doesn't even appear consistent with the u1 gate's matrix.
Any suggestions?
No response

OpenQASM 2 doesn't have any way of representing a global phase (unlike OpenQASM 3), so there's no possibility of the operations being defined any other way.  It's not really valid as a serialisation language for the exact unitary form of operations, only complete programs with measurements (i.e. it's not really suitable to use OQ2 to represent just one operator that should later be controlled in a larger circuit).  Under that description, I think what we do is a valid interpretation of the spec, filling in some undefined behaviour (the global phase of the matrix representation of a circuit) by using our defined RZ phase convention if the user specified that, and the U1/P phase convention if that was specified.
With your last circuit - that circuit doesn't appear to faithfully implement a controlled U1 gate to me, but the matrix that comes out from the input circuit does look correct.  From the circuit you've written, I'd expect to see something that looks the same as your c2 circuit, because we can make the substitution $U_1(\theta) = a R_z(\theta)$ for whatever scalar $a$ that conjugates under the action of the adjoint, then you can bring the scalar terms to the front of the multiplication and see that they immediately cancel.

It's not really valid as a serialisation language for the exact matrix form of operations

I should be clear that this is purely my interpretation of OpenQASM 2, not an official position of the language: I didn't have any part of writing its spec.  I got to that opinion mostly on the basis of there not being a way to define a global phase in the language, and from my understanding of discussions during the design of OpenQASM 3 (which I was more involved in).

Thinking about workarounds somebody might try in order to have this work more like pyZX wants: Qiskit's OpenQASM 2 parser actually allows for a lot of customisation over how exactly gates are interpreted and built on the Qiskit side.  A user could specify that they want rz gates to be brought into Qiskit actually using the PhaseGate class (as opposed to RZGate).  You do that like this:
from qiskit import qasm2
from qiskit.circuit.library import PhaseGate
from qiskit.quantum_info import Operator

prog = """
OPENQASM 2.0;
include "qelib1.inc";
qreg q[1];
rz(pi/2) q[0];
"""

qc = qasm2.loads(prog, custom_instructions=[qasm2.CustomInstruction("rz", 1, 1, PhaseGate)])
print(Operator(qc).data)

array([[1.000000e+00+0.j, 0.000000e+00+0.j],
       [0.000000e+00+0.j, 6.123234e-17+1.j]])

which I think is what you wanted (you can also inspect qc to see it used a PhaseGate instead of an RZGate).

QuantumCircuit.from_qasm_str is actually a now-legacy wrapper around the main importer logic that lives in qiskit.qasm2.  The full documentation on the parsing infrastructure for OpenQASM 2 is here: https://qiskit.org/documentation/apidoc/qasm2.html#parsing-api.

I've just opened PR #10801 to add some more documentation around how Qiskit interprets these phase conventions, and how they can be overridden.

Thanks very much, Jake!
It's not really about "what pyZX wants", but what its users expect. Based on existing filed issues, it seems there's a mismatch. I think it would be clearer if pyzx was changed to internally use u1 explicitly. In any case, you've given me the info I need to work on a fix in pyzx itself.

No worries, let us know if we can help more / you find anything else.  Qiskit and OpenQASM 2 were developed in quite an intertwined manner at the time, so it's quite likely there's other places where Qiskit has some behaviour that was also the "spirit" of OpenQASM 2 even if not the specification.
On "what pyZX wants" - sorry if it wasn't clear, I mostly just meant that the OQ2 file pyZX wrote out was assuming it'd be read with a different phase convention for the gates than what Qiskit does by default.  I think both Qiskit's and pyZX's readings of the spec are totally valid here, and now we're aware there's an ambiguity that different libraries are resolving in different ways, we can be sure to document our conventions and how to override them better.


Messenger recipe constantly 17% CPU usage in Franz 5

Upgraded to Franz 5 yesterday and I noticed a higher CPU usage than the older client.
Expected Behavior
Not eat a constant 17-20% of cpu.
Current Behavior
Constantly requires at least 17%
Screenshots (if appropriate):

Possible Solution

Steps to Reproduce (for bugs)

Add a Facebook Messenger service
Login to service
Check CPU usage
Disable and it will go down.

Context

High CPU usage, drains battery and makes the device heat up.
Your Environment

Franz Version used: 5.0.0-beta.10.7
Operating System and version: macOS High Sierra 10.13

connected with #15

Same behaviour here, the Facebook tab is really CPU expensive.

Rambox also having the same issue: https://github.com/saenzramiro/rambox/issues/366

Wow it really puzzles me. I have added quite a lot of services which usually use round 0.1-1% each and sometimes go to 10-15% which I would still consider ok as it's short time. Messenger however is going batshit crazy with it's CPU usage!
I did some tests disabling all recipe code, polling, ... and it doesn't improve anything. I will spend some time profiling the service to figure out what to do!

Can confirm the update

@rwkyyy great to hear!
Can we do a little crowd-test to make sure this is actually helping on a larger scale?
In ~/Library/Application Support/Franz/recipes/messenger, update the index.js with:
module.exports = Franz => class Messenger extends Franz {
  overrideUserAgent() {
    return 'Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/61.0.3163.100 Safari/537.36';
  }
};

then reload Franz and take a look your activity monitor.

Even better 😊

Have a high CPU load with two Slack Services actice

@ridculy I don't think the service is the problem. my cpu usage is the same as in my previous reply, and I have two slack channels logged.
Most probabbly something else is causing the CPU load (from Franz).

It's the "Support with Processing Power"

@ridculy What happens on your machine when disable "Support with Processing Power"?

Without I disable "Support with Processing Power" everything is fine.
When I Enabling it :

I don't know where the Monero mining setting is as I couldn't find it in the UI, but I can no longer use Franz as it cuts my laptops battery life in half. You really shouldn't, by default, drain users batteries to mine a silly cryptocurrency -- especially since I can't even find a way to disable it. This is what viruses do.
Back to using a couple separate clients.

it isn‘t on by default. Go to settings -> account. if you see pricing options it‘s not activated.

At least it's not supposed to be. What would explain the CPU usage then?
I don't have any pricing options in my settings. Never did.

Maybe related to #117? As mentioned there, performance is atrocious unless I disable Facebook messenger!
@adlk Although your suggestion above to override the user agent does seem to improve it a little bit…

@rikur you are on Franz 5? If yes, does the Franz logo in your sidebar has a little ⭐ ?
@danielbayley it could be. At my machine, CPU usage is now down to ~3-5% for messenger but I don't have a dedicated GPU in my MacBook Pro. At this point I suspect that the availability of a GPU could be a reason here.

@adlk I apologize for my harsh language earlier on.. I had installed a "franz 5 beta" from a community package -- however, turns out the maintainer had messed up build and it was still running franz 4.x
The actual Franz 5 is a lot more CPU friendly. Thank you.

Due to the messenger recipe update there are almost no complaints. I'm closing this for now. If anything comes up, let me know and I'm happy to reopen this.

Is Franz actually mining a cryptocurrency in the background?

When I enable ICQ service it jumps from 6-8% to 20-25%
I also have a few gmails, FB, WhatsApp, telegram, keep and calendar.
In general, why would it take 6-8% when no messages are being sent or received

This is really unprofessional, Sry but you get no information when installing this, that there will be mining. I really liked this tool, but first of all i wont pay 30€ per Year for a tool which is just using some APIs. I would be willing to support but not at this high price, why is there is no lifetime membership (at max 20€).
I will drop this tool now because of bad behavior against your customers or at least against me. Some people seem to have just money bills in their heads.

Please reopen. My cooling fan is driving my colleagues crazy while running franz :(

same here, when fb-messenger is open with a group-chat, cpu jumps to 80%

Issue is still there, please reopen and investigate

@AurelienVernay same here

CPU Load is still crazy high

Franz is now unusable. It is now at 150% + while being in the background and not using it.

I don't have facebook messanger and franz cpu is crazy high
What do I use:
Notion.so, Todoist, Whatsapp, Slack, Telegram, Gmail, github, Google calendar

I've thrown in the towel now and switched to Rambox. It's open source and does not use nearly as much CPU as franz does. Can recommend 👍


Add reason code to medication

Fixes Feedback on g6pd release 'Add reason code to details screens and do UI adjustments incase of long text.
Type
Bug fix | Feature
Checklist

[x] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[x] I have added any strings visible on UI components to the strings.xml file
[ ] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[x] I have built and run the fhircore app to verify my change fixes the issue and/or does not break the app

does this relate to an existing issue?

It was a feedback on g6pd app. Does not have an issue.


Update kpod inspect to use the new container state

kpod inspect now uses the new libpod container state
and closely matches the output of docker inspect
some aspects of it are still WIP as the libpod container state
is still being worked on
Signed-off-by: umohnani8<EMAIL_ADDRESS>

@rhatdan @mheon @baude PTAL.
There is quite a bit of information that is not available yet, will add those later.
I have tried to match this as closely as possible to docker inspect.
If there are fields that we will definitely not be needing, let me know and I will take them out.
Also please double check that I am fetching the correct data for each field, and if I am overlooking any.
docker inspect does not omit empty fields, so I have done the same for kpod. What is our preference, omit or not?
Also I have no clue why ctrCreate in sql_internal_state.go is messed up with the spacing, will look into that later.

I think we need to have separate ContainerInfo structs in libpod and cmd/kpod. There's a lot of stuff in the existing docker-based structs that makes no sense in libpod and will clutter the struct for people trying to use the API. The cmd/kpod structs can include the missing information, which we can store as labels or container artifacts

@rhatdan I think we need to restructure this so we have different inspect structs coming out of libpod and 'kpod inspect' - libpod doesn't know about a lot of things that should be in the kpod inspect struct (what mounts were user specified volume mounts, what namespaces were specified as host or another container, etc). CRI-O and other libpod API users won't know or care about these because they don't have a kpod CLI. We should output only the things libpod knows about from libpod, and then we can save the kpod metadata separately and use that for kpod inspect

Made changes based on reviews. Will make further changes based on @rhatdan response to having inspect be broken in two, 1 for kpod and the other for the libpod api.

@mheon I agree.  I think kpod create/run should save this information on their own, and leave to libpod stuff that it knows about.  Therefore cri-o inspect would return less information then kpod run/kpod inspect.
Some of the information that kpod inspect of a cri-o container, could probably reconstructed.

@rhatdan I talked with @umohnani8 and we agreed on an interface for storing metadata. We'll need such a thing for CRI-O anyways, so we'll build it now and use it to store information from kpod run (CLI arguments) so we can reconstruct the rest of what we need.

Rebased. Separated the inspect info between libpod and kpod. Got information for some more of the fields. Will wait for stats as inspect shows some stats info as well.
@mheon @rhatdan PTAL

Getting some wierd failures in CI which might be flakes or environment issues. @umohnani8 this doesn't vendor anything new or change existing vendors, right?

@mheon no it doesn't, but I think I know why tests are failing.

Tests are green. LGTM.
@rhatdan PTAL

@umohnani8 Rebase needed.

rebased

@rhatdan is this good to merge just so we have a working inspect?

@umohnani8 we need to continue working on this and move some of this config into libpod so we can handle data from the containers image.  But for now this LGTM.
@mheon @TomSweeneyRedHat @baude PTAL

@rhatdan yup will get back to it once we figure out how to get all the data together.

LGTM
In the future we'll need to work out a better way to inspect CRI-O containers that don't have inspect artifacts, though we are also discussing API changes that will put all the relevant data in libpod so every container will have it.

That can wait until later, though, as we don't have a major libpod consumer except kpod now
@rh-atomic-bot r+

Looks like a flake
@rh-atomic-bot retry

Seeing a lot of what look like SELinux related flakes in papr

@rhatdan @mheon is this okay to merge, even though homu is failing?

I think it should be fine as tests are green. LGTM. Merging.


Swift tests passing

:tada: This PR is included in version 3.7.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Set an environmental variable indentifying the instance kind

Having an environment variable telling if the machine is a worker or the scheduler or the webserver might be very useful for post-deployment scripts and other specific behaviors. Provides a possible step towards solving #62.

Done with /etc/environment, let's see how it goes.


Tree component

Do you have any plans to implement the components shown in the following diagram.

No, no plans for that. There are too many issues with using the tree web components like that.

No... But using the new ItemTemplate, you could develop it yourself.
https://preview.fluentui-blazor.net/TreeView#withitems


catch braced block without argument

https://civet.dev/playground/?code=dHJ5IGEgY2F0Y2ggeyBjIH0%3D
try a catch { c }
--- actual
try {
  a;
} catch ({ c }) {}
--- expected (or similar)
try {
  a;
} catch(e) {
  c;
}

Originally posted by @STRd6 in https://github.com/DanielXMoore/Civet/issues/399#issuecomment-1437453463

Now that I'm looking at this more closely it may be correct as is. :|

Seems correct.

I think it's a bit awkward for JavaScript compatibility.  Here's a more real example:
function parse(x) {
  try {
    JSON.parse(x)
  } catch {
    defaultObject
  }
}

Admittedly I'm relying on implicit returns here, so there's some Civet buy-in.  On the other hand, it's nice to match exception objects without having to wrap in parens...
Perhaps we could have the best of both worlds like so?

catch !OpenBrace expr block works as usual; block is optional
catch &OpenBrace expr block requires block.
catch block is the third thing to try, and block is optional here (as in CoffeeScript)

The idea is that it's not useful to destructure without having a block, because bindings are local.  Of course, I say that, but we could have
try
  foo
catch {@foo}

That doesn't compile correctly currently... but that's a bug.

I think the correct fix will be an esCompat flag to address this as well as: https://github.com/DanielXMoore/Civet/issues/667 and non-parenthesized single argument arrow functions.

OK, added to #270


Use KS5 built-in auth for Meetup demo

No need for these custom routes, they're all handled within KS5 already.

Approved by @timleslie


Can't build docker container

When trying to run docker build . I get a permission error:
Step 6/7 : FROM htdvisser/protocontainer:20180821
pull access denied for htdvisser/protocontainer, repository does not exist or may require 'docker login': denied: requested access to the resource is denied

I'm going to close this issue. We're not going to address it anymore, as V2 is going to shut down in December.


0.9.1+ support?

Is it coming?

Accounts merge is now compatible the latest version of Meteor. https://atmospherejs.com/mikael/accounts-merge


Don’t expose polyfill to global when in CJS/AMD

Closes #93.
Also, I’ve removed forceRegisterDialog and registerDialog methods since they’re already defined on dialogPolyfill object which is exported to window. I suppose that’s OK?

I signed it!

LGTM


Block Chrome NextDNS Default DoH

add chromium.dns.nextdns.io to block default NextDNS DoH in Chrome

sorry to hitchhiking the issue, but #3


Add plist structure definition identifier

This pull request adds support for the plistStructureDefinitionIdentifier definition identifier. The plistStructureDefinitionIdentifier attribute appears to be quite common.

Thank you so much for the PR @dnicolson 😃  Can you add the compiled grammar files now?

@pedrovgs if you merge this I can merge the others one by one.


⚠ Slack has degraded performance

In af41a09, Slack (https://status.slack.com/api/v2.0.0/current) experienced degraded performance:

HTTP code: 200
Response time: 110 ms

Resolved: Slack performance has improved in 26c66ba.


[BUG] code bug

In benchmarks/python/mem_monitor.py
def get_memory_info(handle):
    mem_info = pynvml.nvmlDeviceGetMemoryInfo(handle)
    total = round(mem_info.total / 1024 / 1024 / 1024, 2)
    used = round(mem_info.used / 1024 / 1024 / 1024, 2)
    free = round(mem_info.used / 1024 / 1024 / 1024, 2)
    return total, used, free

For 'free' in line 5, should be 'mem_info.free'

Thank you for report. We will fix it ASAP.


Bump library version

Bump version to enable publishing of a new library version

Why? Shall we do a new release soon?

I think we need a new version since we fixed this bug and we had major python dependencies update.
Sure we need to discuss the strategy of bumping versions.


Using your data for my website – thanks!

COVID-19 Query Tool uses this repository indirectly via BigQuery.
Thanks for your efforts!

Very nice! Can we add it to our list of example usages?
I also read this from your site:

The dataset has some data with names of state/province left blank. It would appear the blank/missing names are used to show totals.

While this is true, you may prefer to use the column aggregation_level to tell whether the data is aggregated by country, subregion1 or subregion2.

Sure, you can add it to the list.

While this is true, you may prefer to use the column aggregation_level to tell whether the data is aggregated by country, subregion1 or subregion2.

Thanks, I already use this column. The Skip state/province data setting found under 'advanced options' uses it in queries as a filter to skip subregion1 and subregion2 data when required.


Enable Numa support for Power (ppc64le) architecture

Fixes #124
Signed-off-by<EMAIL_ADDRESS>

Coverage remained the same at 81.057% when pulling b2aa0225ac2468809e818dcc5f1bad5c95fa4507 on bpradipt:master into ad66e4caf8259a0324c487b748bc3413938a7938 on intel:master.

@devimc Can you take a look?

@bpradipt You will need to vendor in govmm into Kata for fixing the ppc issue.

@bpradipt You will need to vendor in govmm into Kata for fixing the ppc issue.

thanks @amshinde. I created a PR to vendor in the latest govmm


mkv file does not work with editor

mkv file does not work with editor. only with mp4?

that would indeed not work, an oversight by me. i don't think browsers support mkv still so it wouldn't be able to play it anyway


Fix openshift custom host role

Add role rule routes.route.openshift.io/custom-host to support OCP. Does not affect k8s eventhough it is "openshift" related role.
Tested on:

k8s 1.14.10
OCP 4.2

An alternative option would be maintain operator metadata different for k8s and OCP "flavors".
The motivation for this change is that when creating an Ingress with an IngressRule that has the 'host' field set in an OpenShift environment the following error is shown:
{"level":"error","ts":1585080246.091675,"logger":"controller_apicast","msg":"Requeuing request...","error":"ingresses.extensions \"apicast-apicast-stage\" is invalid: spec.rules[0]: Forbidden: you do not have permission to set host fields in ingress rules

Looking at OpenShift code it seems it has some sort of custom code that includes an Admission Controller for Kubernetes Ingress objects:
https://github.com/openshift/origin/blob/v3.9.0/pkg/ingress/admission/ingress_admission.go#L106
There also seems to be this check for OpenShift routes itself:
https://github.com/openshift/origin/blob/v3.9.0/pkg/route/registry/route/strategy.go#L104
We had a similar problem with Zync some time ago.
What are the implications of adding that element from a non-admin user point of view? Can we check them?
I seem to remember this permission is not added to the list of default permissions that is given to a non-admin user, which meant that a non-admin user could not deploy that role anymore.
Are we also sure that on k8s native the addition to that permission is not a problem? have this been tried with OLM or a non-admin user? I also seem to remember that permissions problems were not reproduced when deploying the role.yaml as an admin. For example, if you tried to add a non-existing verb for a given Kind/APIGroup and you deployed as admin I seem to remember K8s/OpenShift didn't complain but at the moment this was done as a non-admin the problem surfaced.

I answer to myself a little bit of the comments I left regarding:
We had a similar problem with Zync some time ago.

What are the implications of adding that element from a non-admin user point of view? Can we check them?
I seem to remember this permission is not added to the list of default permissions that is given to a non-admin user, which meant that a non-admin user could not deploy that role anymore.

Are we also sure that on k8s native the addition to that permission is not a problem? have this been tried with OLM or a non-admin user? I also seem to remember that permissions problems were not reproduced when deploying the role.yaml as an admin. For example, if you tried to add a non-existing verb for a given Kind/APIGroup and you deployed as admin I seem to remember K8s/OpenShift didn't complain but at the moment this was done as a non-admin the problem surfaced.

The problem I was referring to was related to UPDATING the host name and not creating it.
The issue with allowing update (we are not giving that permission at the moment, not even in this PR) was that it allows you to "steal" routes from other elements. See this issue for more detail:
https://github.com/openshift/origin/pull/18177#issuecomment-360660024. So, maybe the route/custom-host for create IS given to default non-admin users but not the update one. Can we check that?
Can we still check what happens when you deploy the role.yaml including the OpenShift reference (routes/custom-host create) as a non-admin user in a K8s native cluster?

Tested reconcilliation and added required role rules.

Are we also sure that on k8s native the addition to that permission is not a problem? have this been tried with OLM or a non-admin user? I also seem to remember that permissions problems were not reproduced when deploying the role.yaml as an admin. For example, if you tried to add a non-existing verb for a given Kind/APIGroup and you deployed as admin I seem to remember K8s/OpenShift didn't complain but at the moment this was done as a non-admin the problem surfaced.

I've tried the following example in an OpenShift cluster:
The following role can be deployed as a non-admin user:
kind: Role
apiVersion: rbac.authorization.k8s.io/v1
metadata:
  name: simplerole
rules:
- apiGroups:
  - ""
  resources:
  - pods
  verbs:
  - create
  - delete
  - get
  - list
  - patch
  - update
  - watch

But, when adding a non existing APIGroup/Kind into the role an error is received when the user tries to create it:
msoriano@localhost:~/go/src/github.com/3scale/testscaf2/deploy$ cat simplerole.yaml
kind: Role
apiVersion: rbac.authorization.k8s.io/v1
metadata:
  name: simplerole
rules:
- apiGroups:
  - ""
  resources:
  - pods
  verbs:
  - create
  - delete
  - get
  - list
  - patch
  - update
  - watch
- apiGroups:
  - myapigroup.com
  resources:
  - mykind
  verbs:
  - create

msoriano@localhost:~/go/src/github.com/3scale/testscaf2/deploy$ oc create -f simplerole.yaml
Error from server (Forbidden): error when creating "simplerole.yaml": roles.rbac.authorization.k8s.io "simplerole" is forbidden: user "anonadminuser" (groups=["system:authenticated:oauth" "system:authenticated"]) is attempting to grant RBAC permissions not currently held:
{APIGroups:["myapigroup.com"], Resources:["mykind"], Verbs:["create"]}

However, if the user that tries to create the role is a cluster-admin, NO error is shown even when the APIGroup/Kind does not exist in the cluster, what confirms what I thought:

msoriano@localhost:~/go/src/github.com/3scale/testscaf2/deploy$ oc create -f simplerole.yaml
role.rbac.authorization.k8s.io/simplerole created

Some findings:
Operators can only be deployed by cluster admin  users.  https://access.redhat.com/documentation/en-us/openshift_container_platform/4.2/html/operators/olm-understanding-operatorhub
Namespace admin users should be able to deploy apicast instances
The new role definition in this PR can be deployed using OLM. Tested in OCP 4.2

As you commented, to deploy operator.yaml you have to be cluster-admin so I understand we can go forward with including the OpenShift permissions in a single file, even deploying in K8s where the API/Kind of those permissions does not exist, because it is ignored when you are cluster admin as I showed in OpenShift case


Add odo ssh command

[kind/Feature]
Which functionality do you think we should add?
odo ssh command would allow a user to directly log into the currently active (or the appropriate one based on component, application and project flags) pod to be able to examine its state without having to first hunt it down.
Why is this needed?
Not needed but would be nice for odo to help user diagnose issues / check that everything is as expected.

Agreed. Not sure if ssh would be better than rsh, but feature should exist. This command exists in oc

@jorgemoralespou @metacosm
The command exists in oc but let's hope that we can access this through the OpenShift golang library.

The command exists in oc but let's hope that we can access this through the OpenShift golang library.

It shouldn't be a problem to implement this. It is just executing /bin/bash or other shell and redirecting input and output to local console. It should be pretty similar to what we already do for copying files.

/lifecycle frozen
IMO we should (eventually) implement this into odo debug subset of commands.

@metacosm I am closing this issue because odo exec seems to serve the purpose described in this issue. Feel free to reopen it if I misunderstood the issue and you think otherwise.


What is the "effect" and "p-adj" and "Significant" column in Fulco_interactions_wScore.txt

Hi
Thank you for your wonderful paper.
When I check you Zenodo directory (https://zenodo.org/records/7351632), I find three columns ("effect" and "p-adj" and "Significant") in the file Fulco_interactions_wScore.txt. How can we understand this kind of statistical results? is it from your STARE tools?
Thanks !

Hey,
Thank you for your interest!
These columns come from the Fulco et al. paper 2019 (https://doi.org/10.1038/s41588-019-0538-0), after intersecting them with our enhancer-gene interactions so that we could add the columns with the score of the different ABC-scoring versions. That means that it only has the entries for which we had a matching enhancer-gene interaction from the ABC-score. The same is true for the of Gasperini and Schraivogel.
Best wishes,
Dennis


validate input shapes

Fix: https://github.com/triton-inference-server/server/issues/7171
Validate input shapes at core level
Testing: https://github.com/triton-inference-server/server/pull/7195

Do not merge as seeing failures in L0_cuda_shared_memory and L0_shared_memory

Please don't merge a PR before the corresponding tests or related PRs are also passing and ready to merge. It looks like the testing is still a WIP: https://github.com/triton-inference-server/server/pull/7195.


coreir.neg unsupported in interpreter

Unsupported node: neg_inst has operation name: coreir.neg
Assertion failed: (false), function updateNodeValues, file interpret.cpp, line 1160.

Steps to reproduce:
git clone -b test-coreir-neg https://github.com/leonardt/pycoreir
cd pycoreir
pytest -s

Also, coreir.udiv, coreir.urem, coreir.sdiv, coreir.srem, and coreir.smod

coreir.neg is fixed in #477

@dillonhuff Do you want me to open up a separate issue for coreir.udiv, coreir.urem, coreir.sdiv, coreir.srem, and coreir.smod? Or we can just keep this one open for them

@leonardt it would be good if you could open a separate issue for those.
Is implementation of them needed now?

@dillonhuff not needed now by me, they just failed in my unit tests for the primitives (since they weren't there), but I can wait to merge those tests in until the implementation is ready (and needed)


🐛 Prevent gc/quota workqueue backlogs

Summary
Run the initial monitor syncing in a goroutine to avoid any stalling here to block a worker.
Related issue(s)
Maybe #1878 #2665

https://github.com/kcp-dev/kubernetes/blob/f48a2e575b3bf96221060778f57689972c3709b1/pkg/controller/resourcequota/resource_quota_controller_patch.go#L80 for quota (and similarly for gc) is blocking and never ends because the ctx we use with resourceQuotaController.UpdateMonitors(ctx, c.dynamicDiscoverySharedInformerFactory.ServerPreferredResources) has no timeout. We could consider giving it one?

@sttts interested in your thoughts

/lgtm
Adding a timeout on the context sounds good to me; is there really a benefit to leaving it off?

Not really, upstream's is 30 seconds IIRC. I guess I thought we wouldn't have an issue since we don't have API aggregation...

/approve


Updated to Python3

Hi,
I have updated the syntax for Python 3

I made some significant changes to the codebase, and bumped it up to Python3 compatibility. Thanks for the pr though.


Can I use functional api with vuex?

Hi,
I am really eager to try this new api and especially the typescript integration. But can I use this api with vuex? If so could anybody provide an example? :slightly_smiling_face:
Thanks!

Hi,
I am really eager to try this new api and especially the typescript integration. But can I use this api with vuex? If so could anybody provide an example? 🙂
Thanks!

see comment https://github.com/vuejs/vue-function-api/issues/8#issuecomment-506143686

Actually, after splitting states into hooks, I removed vuex from my project dependencies. 🤪

Hi, everyone, I created a vue-hooks repository and package.
It contains commonly used hooks (e.g: vuex / vue-router).
Documentation has not been written yet, welcome everyone to contribute! 😄

@SilentDepth how are you sharing sharing vue-function-api state between components?
care to share an example? please

@gianko Just like this:
/* hook-msg.js */

import {value as binding} from 'vue-function-api'

const msg = binding('hello world')

export default function useMsg () {
  return {
    msg,
  }
}

/* comp-a.vue */

import useMsg from './hook-msg'

export default {
  setup () {
    const {msg} = useMsg()

    return {
      msg,
    }
  },
}

/* comp-b.vue */

import {computed} from 'vue-function-api'
import useMsg from './hook-msg'

export default {
  setup () {
    const {msg} = useMsg()

    return {
      msgUpperCased: computed(() => msg.value.toUpperCase()),
    }
  },
}

Thanks!... I was getting an error about using vue function api before calling the use plugin.

I used the trick in the #8 answer but built on it a bit by making a small vuex plugin to do the wrapping of my getters.
Feels kind of hacky, but then all getters can be unpacked in one line
/* store-plugin.js */
import { computed } from 'vue-function-api'

let getters = {};

const myPlugin = store => {
  Object.keys(store.getters)
    .forEach(key => (getters[key] = computed(() => store.getters[key])));
}

export {
  myPlugin,
  getters
};

Then the computed getters can be accessed with:
/* some-vue-file.vue */
import { getters } from './store-plugin.vue';

const { someGetter, someOtherGetter } = getters;

What do you think about https://github.com/zerobias/effector as a state manager?
Vue-function-api is the great thing with jsx =)
I thinking vuex doesn't need anymore =D
See:
// @app/core/hooks/use-value
import { value } from "vue-function-api"

export const useValue = (initialValue, lazy) => {
  lazy = !!lazy
  const state = value(initialValue)
  const dispatch = (newValue) => {
    if (!lazy) {
      state.value = newValue
    } else if (state.value !== newValue) {
      state.value = newValue
    }
  }
  const reset = () => dispatch(initialValue)
  return [state, dispatch, reset]
}

// @app/core/hooks/use-reducer
import { useValue } from "./use-value"

export const useReducer = (reducer, initialValue, lazy) => {
  const [state, set, reset] = useValue(initialValue, lazy)
  const dispatch = (payload) => {
    const reducerResult = reducer(state.value, payload)
    set(reducerResult)
  }
  return [state, dispatch, reset]
}

Example:
import { useReducer } from "@app/core/hooks/use-reducer"

export const ExampleCounter = {
  setup() {
    const initialValue = 0

    function reducer(value, payload) {
      switch (payload.type) {
        case "increment":
          return value + 1
        case "decrement":
          return value - 1
        default:
          throw new Error()
      }
    }

    const [count, updateCount, reset] = useReducer(reducer, initialValue)

    const increment = () => updateCount({ type: "increment" })
    const decrement = () => updateCount({ type: "decrement" })

    // No needs, we have reset state function already
    // const reset = () => updateCount({ type: "reset" })

    return {
      count,
      increment,
      decrement,
      reset,
    }
  },

  render(h) {
    return (
      <div>
        Count: {this.counter.toString()}
        <button vOn:click_prevent={this.increment}>+</button>
        <button vOn:click_prevent={this.decrement}>-</button>
        <button vOn:click_prevent={this.reset}>reset</button>
      </div>
    )
  },
}

PS:
Example uses jsx - vue requires the next babel plugins for that
{
  plugins: [
    "@vue/babel-plugin-transform-vue-jsx",
    "@vue/babel-sugar-v-on",
  ]
}

Duplicate of #8

@SilentDepth i presume this example is not still relevant?
I tried to do the equivalent using ref and got an error Uncaught Error: [vue-composition-api] must call Vue.use(plugin) before using any function.

@gianko Just like this:
/* hook-msg.js */

import {value as binding} from 'vue-function-api'

const msg = binding('hello world')

export default function useMsg () {
  return {
    msg,
  }
}

/* comp-a.vue */

import useMsg from './hook-msg'

export default {
  setup () {
    const {msg} = useMsg()

    return {
      msg,
    }
  },
}

/* comp-b.vue */

import {computed} from 'vue-function-api'
import useMsg from './hook-msg'

export default {
  setup () {
    const {msg} = useMsg()

    return {
      msgUpperCased: computed(() => msg.value.toUpperCase()),
    }
  },
}

@blowsie You need to call Vue.use() first, before any composition api being used.
The missing entry file of my example:
/* main.js */

import Vue from 'vue'
import './use-composition-api'
import App from './app.vue'

new Vue({
  render: h => h(App),
})

/* use-composition-api.js */

import Vue from 'vue'
import CompositionApi from '@vue/composition-api'

Vue.use(CompositionApi)


Consider the pixels per point

Closes https://github.com/emilk/egui/issues/3453
[x] I have followed the instructions in the PR template

I'm fairly new to egui.
I read the code, but I didn't follow the approach mentioned in https://github.com/emilk/egui/issues/3453#issuecomment-2208255433.
I believe this is an easier way to achieve that, though I'm not certain if it's the best method.

I get really nice svg with this patch.

It's interesting that some of the indents are tabs while others are 4-spaces.
Would you like me to fix that?


Update devicelab_staging_config.star to use ios 14.4 devices.

Update devices from ios 14.3->14.4

abandoning this for a different PR: https://github.com/flutter/infra/pull/368


[Swift 4.2 Migration] 3Sum and 4Sum

Checklist

[x ] I've looked at the contribution guidelines.
[ x] This pull request is complete and ready for review.

Description

Checked for compatibility with Swift 4.2/Xcode 10.1. No changes needed.

Thanks for the update 😄

#748 for reference


Table stickyHeader doesn't work with height: 100%

Table stickHeader functionality doesn't work when the TableContainer height is set to "100%" or "flex: 1"

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
When you set the TableContainer height to 100% or flex: 1 the sticky functionality doesn't seem to work.
Expected Behavior 🤔
Sticky functionality would work based on the TableContainer's height when restricted by flex or height as a percentage value. I rarely set scroll heights to px values b/c it's usually dependant on the browser window size.
Steps to Reproduce 🕹
https://codesandbox.io/s/material-demo-forked-6knzs?file=/demo.tsx
(this is a slightly modified version of the sample code from the docs).
Notice the table rows overflow the paper. I would have expected an overflow-y property on TableContainer to make that scroll, and then the sticky functionality to keep the headers at the top of the scrolled window.
Context 🔦
I'm trying to show a couple tables at nearly full page height with vertical scrolling. I'd like the headers to stay attached to the top of my scroll window.
Your Environment 🌎

`npx @material-ui/envinfo`
Tested on Chrome: 89.0.4389.90
  System:
    OS: macOS 10.15.7
  Binaries:
    Node: 15.3.0 - /usr/local/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 7.0.14 - /usr/local/bin/npm
  Browsers:
    Chrome: 89.0.4389.90
    Edge: 83.0.478.61
    Firefox: 84.0.2
    Safari: 14.0.3
  npmPackages:
    @material-ui/core: ^4.11.0 => 4.11.3
    @material-ui/data-grid: ^4.0.0-alpha.8 => 4.0.0-alpha.20
    @material-ui/icons: ^4.9.1 => 4.11.2
    @material-ui/lab: ^4.0.0-alpha.56 => 4.0.0-alpha.57
    @material-ui/pickers: ^3.2.10 => 3.2.10
    @material-ui/styles:  4.11.3
    @material-ui/system:  4.11.3
    @material-ui/types:  5.1.0
    @material-ui/utils:  4.11.2
    @types/react: ^16.9.53 => 16.9.55
    react: ^17.0.1 => 17.0.1
    react-dom: ^17.0.1 => 17.0.1
    typescript: ^4.0.3 => 4.0.5

@ntbosscher You have to make it overflow correctly.


[v9] issue: checkbox checked value always 1 in POST

I want to submit a form from a block (not in Dashboard) with a checkbox, e.g.
echo $form->checkbox('agree', 1, '0');
echo $form->label('agree', t('I agree'));

and I read the POST value in a block controller:
$agree = $data['agree'];

but that $agree value is always 1 regardless whether the checkbox is selected or not. This exact code works in the Dashboard when I save forms with checkboxes. But it doesn’t work in the front end.
doing $agree = isset($data['agree']); has no effect

Can anyone else reproduce this behavior?

A textual '0' string is a non-empty string. As a boolean it is true.

it doesn't matter what I use. I tried
$form->checkbox('agree', 1, '0');
$form->checkbox('agree', 1, 0);
$form->checkbox('agree', 1);

Regardless of being checked or not,
If I do $form->checkbox('agree', 1); the post value is 1.
If I do $form->checkbox('agree', 1, 0); the post value is 1.
If I do $form->checkbox('agree', 1, '0'); the post value is 1.
If I do $form->checkbox('agree', 0); the post value is 0.
If I do $form->checkbox('agree', 0, 0); the post value is 0.
If I do  the post value is undefined.

 I've just spent 2 days looking for it. Then when I thought I've exhausted all options, I've raised the issue.
As it turned out, I forgot I was using ajax to send the form, and the JS was taking the $('#agree').val(); instead of $('#agree').is(':checked') ? 1 : 0; . Sorry for false alarm. Feel so stupid...


Attributes giving arbitrary/irrational values

tenet_oceanic:
  name: Oceanic Tenet
  description: Spend time in Water. Avoid leaving the water.
  triggers:
    - trigger: interval
      interval: 10000
  actions:
    cast:
    - class: CheckBlock
      allowed: water
      source_location: feet
      actions:
        - class: ModifyAttribute
          attribute: deity_favour
          bypass_undo: true
          value: "x+0.6"
      fail:
        - class: CheckRequirements
          requirements:
            - attributes:
              - attribute: deity_favour
                min: 1
          actions:
            - class: ModifyAttribute
              attribute: deity_favour
              bypass_undo: true
              value: "x-0.2"
  parameters:
    target: self

when using the above spell, the player should gain 0.6 attribute for every 10 seconds spent in water, and lose 0.2 attribute every 10 seconds out of water, but it seems that it's not adding/removing exactly 0.2 or 0.6 and over time it winds up with these strange values. The 0.6 and 0.2 are the only things able to change/alter this attribute right now

Are these not just floating point rounding errors?
The underlying mathematics is done on double-values, which can only
represent a subset of rational values.
Therefore, operations using numbers that it cannot represent exactly,
will introduce some rounding errors.
Depending on how you use them, these may accumulate over time.
See:
https://en.wikipedia.org/wiki/Floating-point_arithmetic#Representable_numbers,_conversion_and_rounding

I've encountered these before when using operations that may result in irrational values like these, such as multiplication by decimals/floating point. But the part that's baffling me here is the only functions/operations/values are completely rational/normal numbers (I'm only incrementing by adding or subtracting 0.6 or 0.2 exactly) so I can't understand where the spell is gaining or losing these specific extra numbers. For example this triggers every 10 seconds, so it would be expected that after 10 seconds out of water the Attribute would decrease from 10.0 to 9.8 but instead it will fall to something like 9.79999999 or 9.80000001 etc. It's strange because there's no reason the function should need to round anything, if that's the error i'm encountering


Deterministic sort

Changelog entry
...
Changelog category

Not for changelog (changelog entry is not required)

Additional information
...

https://github.com/orgs/ydb-platform/projects/27/views/1?pane=issue&itemId=53107691


Fix to file path for notebook quickstart

Describe changes
When running zenml go for the guided quickstart as instructed in Readme, I get following:
FileNotFoundError: [Errno 2] No such file or directory: '~/.../zenml_tutorial/notebooks'
notebooks path does not get created in files download so I have removed from Jupyter notebook call.
Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[ ] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] I have based my new branch on develop and the open PR is targeting develop. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.
[ ] If my changes require changes to the dashboard, these changes are communicated/requested.

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

@coderabbitai review


POC of moving NRF51_unified to common sources [former nRF52840 ONLY port]

Description
This is POC of moving NRF51 unified targets to common source for all Nordic's MCU. In directory of this port (./TARGET_NRF5_SDK13 were placed two nRF5 SDK versions: 11   and 13). HAL implementations are unified (where possible)
Changes:
renamed dir. ./sdk -> ./sdk13
copied dir. NRF5/sdk -> ./sdk11
copied dir. NRF5/TARGET_NRF51822_UNIFIED -> ./TARGET_NRF51822_UNIFIED (only non end-target files)
moved dir. NRF5/TARGET_NRF51822_UNIFIED/TARGET_NRF51_DK -> ./TARGET_NRF51822_UNIFIED/TARGET_NRF51_DK
moved dir. NRF5/TARGET_NRF51822_UNIFIED/TARGET_NRF51_DONGLE -> ./TARGET_NRF51822_UNIFIED/TARGET_NRF51_DONGLE
Added backward compatibility code to API implementation of gpio, pwm, serial, port, spi
Added backward compatibility code to BLE implementation of GAP.
Modified MCU_NRF51_UNIFIED target description in order to use moved files.
Modified NRF52840_DK targets description (small adaptation for using proper sdk version).
Ideas which needs to be discussed

MCU_NRF51_UNIFIED has two 3-party child target orphan:

DELTA_DFCM_NNN50
TY51822R3
Is it acceptable to move it with nordic targets?

After applying this PR renamed port directory form TARGET_NRF5_SDK13 to TARGET_NRF5X or other more generic name. The second possibility is to join/merge TARGET_NRF5_SDK13 together with TARGET_NRF5 sources.
Changes from PR are possible to be applied even before #4132 will be merged in ARM mbed-os (need only rebase).

Status
POC

FYI @pan-, @anangl

https://github.com/ARMmbed/mbed-os/pull/4245 introduce combined NRF5 + NRF52840.

https://github.com/ARMmbed/mbed-os/pull/4245 introduce combined NRF5 + NRF52840.


Fixed Issue : Problem with Menu in Edit Template

Fixed Issue #23, #24

Updated PR. Separated Commit According to Issue


feat: Add support for FindInMap in serverless app plugin

Description of changes:

Add FindInMap action
Support intrinsics in serverless app plugin to resolve ApplicationId and SemanticVersion under Location

Description of how you validated changes:
Added unit tests and end to end tests
Checklist:

[x] Write/update tests
[x] make pr passes

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

Please fix the Travis error


Update runtime info in dataset's status during binding

Ⅰ. Describe what this PR does
Dataset's status stores runtime information(i.e. Dataset.Status.Runtimes[]) to indicate which kind of Runtime is binding to the Dataset. However, now the code updates Dataset.Status.Runtimes[] in the sync loop, which is not appropriate for such information.
This PR fixes this by moving the related code logic from the sync loop to the point when Fluid updates the Dataset's phase to bound.
ReferenceDataset doesn't need code changes because Fluid now copies physical dataset's status into the reference dataset.
Ⅱ. Does this pull request fix one issue?

NONE
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

/test fluid-e2e


Spanish translation missing

I would like to add a Spanish translation to this app.
To solve this issue:

comment below requesting the assignment and wait my reply;
fork the project and create a new branch, then checkout to the new branch;
work on the new translation (in alternative you can use the translation editor as described here):

create the folder value-es under app/src/main/res;
create in the new folder the file strings.xml copied from the default strings.xml;
translate each string to Spanish;

commit your changes (please use the convention of gitmoji.dev as other commits in this project);
open a pull request with the main branch of this repo as target.

Hi Lorenzo. Im a Native Spanish Freelance Translator and Localization Specialist English🔄Spanish. Over 3 years of experience. I will love to help you😊

Hi @Quadropo, thank you for reaching me!
There is a further step to do (I'm sorry, I forgot to add it before):

[ ] I accept the Code of Conduct.
[ ] I accept the Contribution Guidelines.

Please answer to this message pasting the two lines above with checkmarks checked as sign of acceptance of Code of Conduct and Contribution Guidelines 🙂

Can you pls check your Twitter DMs

I accept the Code of Conduct.
I accept the Contribution Guidelines.


Add gRPC service registration feature in spring-boot-autoconfigure

Hello, I've added gRPC service registration feature in spring-boot-autoconfigure
Motivation:

Makes it easy to configure gRPC server in Spring Boot applications

Modifications:

Add GrpcServiceRegistrationBean.java
Add configureGrpcServices method in ArmeriaConfigurationUtil.java then call it in ArmeriaAutoConfiguration.java and ArmeriaReactiveWebServerFactory.java

Result:

It can be configured as follows

@Bean
public GrpcServiceRegistrationBean helloService() {
    return new GrpcServiceRegistrationBean()
            .setServiceName("helloService")
            .setService(new GrpcServiceBuilder()
                                .addService(new HelloService())
                                .supportedSerializationFormats(
                                        GrpcSerializationFormats.values())
                                .enableUnframedRequests(true)
                                .build())
            .setDecorators(LoggingService.newDecorator())
            .setExampleRequests(List.of(ExampleRequest.of(HelloServiceGrpc.SERVICE_NAME,
                                                          "Hello",
                                                          HelloRequest.newBuilder()
                                                                      .setName("Armeria")
                                                                      .build())));
}

Thank you for your comment!
updated.
The test failed in AppVeyor, but it seems to have nothing to do with the last push, right?

it seems to have nothing to do with the last push, right?

Yeah, just an issue with AppVeyor.

@matsumana Would you mind resolving the conflict and addressing @minwoox's comments?

Updated!

Thanks, @matsumana !

Thank you!

is it possible to config with the DocServiceBuilder?


Error during Ubuntu 20.04 build

I am getting that error when I run Ubuntu-build-20.04.sh

Found the issue

can you tell me how to fix this


The demo doesn't work with other model

Hi Nickolay,
I imported this project, built the apk file and run it on a Pie Android device. Everything was fine with the default kaldi model included in this respo. I further replaced this model by my own developed model (for Vietnamese), rebuilt the apk file and run on the same device. After the demo initialized if i touch  "Recognize File"  or "Recognize Microphone" button the demo will disappear (like core dump or something else). My model was trained using CNN+TDNN config with Kaldi version-5.5, the total size at final is 100M. So do you have any suggestion or idea about this issue?

Hi.
Please check logcat output, it might contain too much details.
100M is a bit on a large side, try to train model with less parameters, maybe a bit less accurate, there is a tradeoff between size and accuracy.
Also, make sure you created proper HCLr and Gr fsts with lookahead branch.

Thanks Nickolay,

"Please check logcat output, it might contain some additional details": The demo just disappeared without any log or bug.
"100M is a bit on a large side, try to train model with less parameters, maybe a bit less accurate, there is a tradeoff between size and accuracy.": I am now trying to train other model with smaller size. But the question is that how much size should be to be suitable. Is that should be less than 50M?
"Also, make sure you created proper HCLr and Gr fsts with lookahead branch.": I created HCLr and Gr fsts with lookahead branch.

Do you think that my CNN-TDNN model has some components which are not supported in Kaldi V5.2
My main componets as following: idct-layer, linear-component, batchnorm-component, combine-feature-maps-layer, conv-relu-batchnorm-layer, tdnnf-layer, prefinal-layer, output-layer

"Please check logcat output, it might contain some additional details": The demo just disappeared without any log or bug.

There should be some messages. Please check again. You can get full logcat with adb logcat
.> But the question is that how much size should be to be suitable. Is that should be less than 50M?
The current models are about 13Mb, mini librispeech with dimension 768 per layer should work. The RT ratio on the desktop should be less than 0.1xRT, then it will run realtime on the phone. We will optimize and compress models later, for now you can start with very tiny models.

My main componets as following: idct-layer, linear-component, batchnorm-component, combine-feature-maps-layer, conv-relu-batchnorm-layer, tdnnf-layer, prefinal-layer, output-layer

I don't think it is a problem, logcat should show.

Hi Nickolay,
The only line i got from logcat was:
2019-12-11 16:38:54.979 8172-8723/org.kaldi.demo A/libc: Fatal signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x0 in tid 8723 (AsyncTask #2), pid 8172 (org.kaldi.demo)
I also rechecked kaldi versions, it seems that the TDNN-F is supported from Version 5.4, but the included aar was built with Kaldi version 5.2. I am also trying to rebuild aar  (https://github.com/alphacep/kaldi-android) with the last version of Kaldi but there are still some bugs

There must be earlier lines. At least where it unpacks models.

included aar was built with Kaldi version 5.2

Not true, aar is built with latest kaldi. All released models are tdnn-f.

logcat.txt
The attached file is the logcat that i got. It seems that the issue relates to the size of model?

Do you see extra lines in the logcat now or still nothing?

I would first try with the smaller size.

Still nothing,

I would first try with the smaller size.

Yes, i am trying retrain my model with small size. This the only choice for now that i can do

I still got the same problem with smaller model (14M). I traced back and found that the crashing happens at lines 130, 254 in KaldiActivity.java
I uploaded my demo model at: https://drive.google.com/file/d/1CGpALf2Gp-s-waPLBq6dbI_Dfk9NOAjz/view?usp=sharing
Could you please help me to test it.

I see your Gr.fst is standard fst, can you try with arpa fst created with mkgraph_lookahead_arpa? It doesn't handle standard fsts automatically yet.

Thank you a lot,
You are right, my model worked with with mkgraph_lookahead_arpa!


install_requires  not work?

install_requires=['python-docx', 'pymysql']
pip install Mysql2docx error
requires will not auto install?

Hi @icecooly. This is the repository for the code which powers https://pypi.org/. In the future, you may want to look for help in the following places:

for issues with pip: https://github.com/pypa/pip
for issues with setuptools: https://github.com/pypa/setuptools

I took a quick look at your setup.py and it looks like you have the line import Mysql2docx before you call setup(). This is making your setup.py try to import docx before it can be installed. You can safely remove this line from your setup.py.
In addition, it looks like your distribution is missing the README.md file that it's attempting to use for your long_description.
If you fix these two issues, your package should be installable. You can test this at any time locally by running python setup.py install.

thank you very much


Jroach/add repo skelton via generator

Repo setup with basic info, and included the code for d2l-navigation-band
Once this is approved and merged, I'll move on to referencing this version in BSI (and removing the BSI version).

Thanks @svanherk - I just quickly addressed your concerns. Might as well do it while it's fresh 😄


Blazor sample with code snippets

Add a tab in the Blazor Samples so that the source code can be seen.

The main idea is that with docfx I generate the html pages with the c# code. And in the deployed Blazor I display in an iframe the code html. with a relative path.

I made a powershell script
https://github.com/Mapsui/Mapsui/blob/master/Scripts/BuildSamplesDocumentation.ps1
that adds the .md files from there the .md files generate the html files of the code. in mapsui.com/codesamples/animatedbussample.html for example

To get it prettier we might need to replace docfx with some other tool. A tool that takes a C# file as input and convert is into a html code snippet. Guess there are libraries for that, mostly javascript I fear.

oh, there is an html <code> tag. I guess everyone knew that already. So, simply pasting the file inside a <code> tag would already work. And then there are some css libraries that can add syntax highlighting. I scanning this blog: https://www.codiga.io/blog/display-code-snippets-in-html/


Failed to call service media_player/media_play. 16: Invalid State

Hi,
i'm new to HA but i've been through a few installations of HA and a few resets of the hue box.  i'm on a PI4 now but, each installation, i've received the following error while trying to turn of the sync setting:
Failed to call service media_player/media_play. 16: Invalid State
i am able to change inputs...
any thoughts would be helpful
thanks

I am not sure what is going on.
Could you provide a bit more detailed information on how or what you are doing?
For example are you using the card in the dashboard, using device automations, calling a service or something else?
(and just to be sure, I assume the box is working fine with the normal sync app?)
It would also help if you could temporarily enable logging the configuration.yaml by adding the section below (or adding just the lines mentioning huesyncbox if the logging section already exits) and then reproduce the issue. After that you can disable the logging again.
logger:
  default: info
  logs:
    custom_components.huesyncbox: debug
    aiohuesyncbox: debug


New Doc Theme: Furo

#Changes
New Theme: https://pradyunsg.me/furo/customisation/

I tried building this branch on readthedocs but it failed: https://readthedocs.org/projects/cumulusci/builds/12117586/
I think we need a new requirements file with just furo in it, so that I can point readthedocs to it.


feat(app): Show in progress modal during wifi connect request

overview
closes #2383:

changelog

feat(app): Show in progress modal during wifi connect request

review requests
The new card is still behind a feature flag. To view new card:
make -C app dev OT_APP_DEV_INTERNAL__MANAGE_ROBOT_CONNECTION__NEW_CARD=1
New connection card functionality is coming soon, to test this PR attempt a connect/reconnect to a wifi network using the still visible older ConnectivityCard

[ ] Robot Connection card is inactive while robot is actively trying to connect to a network
[ ] Spinner modal displays over new ConnectionCard with the message:
'Attempting to connect to network [network name]'

Closing in favor of tackling in progress in #2504


Unauth login lost after adding storage feature for auth users

Do you want to request a feature or report a bug?
bug
What is the current behavior?
i enable auth with unauth login. then enable storage with Auth user access only. => unauth logins no longer work
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.
Steps to reproduce the issue:

amplify add auth

? Allow unauthenticated logins? (Provides scoped down permissions that you can control via AWS IAM) Yes

amplify add storage

? Who should have access: Auth users only

At this point, unauth access is no longer available.
What is the expected behavior?
shouldnt lose unauth login just because i add stoge feature for auth users
Additional environment details

OS version (ie Windows 10 build X, macOS Sierra 10.12.6, etc.) mac

Output of amplify --version: 0.1.15

Did this work in previous versions?: Y/N dunno

This issue is fixed in the latest version of the CLI. Please feel free to comment on this thread if you're still facing this issue.


REST-only library

A customer has requested whether or not we could create REST-only library for cocoa. The aim would be to minimise footprint and, in particular, minimise the external dependencies. How easy would it be to do that?

Relates to #949

The basics of making a REST call from a Cocoa application (e.g. an iOS app) is simple and well understood by most developers. There are a few ways it can be done using Apple's foundation SDK (some for foreground execution and others for background via OS daemon), plus there are numerous third party libraries upon which some developers choose to depend upon as a layer on top of Apple's offerings.
What this customer is really looking for, as far as I understand, is help with publishing encrypted messages to Ably via our REST API. This really just needs some help with the encryption and Ably payload packaging, ready to go on the wire.
Therefore, given that the actual process of making the REST calls doesn't really need solving, all we are looking to help with is the Ably specific message packing. If a new library was just offering support in that area then, in addition, it could be of use to assist with MQTT, SSE and other applications where Ably's 'native' REST API is not being used to talk to Ably.
I think this may be able to be solved by producing a new pure Swift library that just offers synchronous methods to assist with these operations. I think we should also keep an eye on how it might interoperate with Combine - this may require no work at all but worth prototyping with.

Speaking to @paddybyers just now about this and he pointed out that, in respect of adding support for decoding/unwrapping inbound Ably messages, beyond utility for SSE and MQTT applications there is also the valid use case of using the REST API to retrieve message history.

The aim would be to minimise footprint and, in particular, minimise the external dependencies. How easy would it be to do that?

How many people have asked for this?
And why are dependencies an issue with Cocoa i.e. I assume this is for server-side use.
It would be good for us to track how many people need this and why before we do anything about it.

Personally I don't think there's sufficient demand and/or value in this to create a REST-only library. I suggest closing this issue. If @kavalerov you want to track this as a possible future feature, then go ahead, but there doesn't need to be an issue in this repo.

I am tracking this already in productboard (https://ably.productboard.com/feature-board/1237731-feature-organization/features/5898832/detail)

So ok ot close


Synchronize package.json and package-lock.json

Describe the bug
There is no way to install dependencies using npm ci
To Reproduce
Steps to reproduce the behavior:

Fork project
Open project in terminal
Enter command: npm ci
Error: npm ci can only install packages when your package.json and package-lock.json or npm-shrinkwrap.json are in sync. Please update your lock file with npm install before continuing.

Expected behavior
All the dependencies are installed.
Additional context
When upgrading to the new version 2.2.0, we forgot to update package-lock.json. The package-lock.json contains old dependencies, for example "@symbiotejs/symbiote": 2.0.4 and "typescript": "5.0.4", instead of "5.3.3".

#79


Fix example for multiple actions

The current example for the ; delimiter in ts-action does not work. We needed multiple actions feature in our code and I had almost pushed a PR for a new resume! operator when I found out it is already there :)

Argh sorry it has no test and went under unnoticed.


Doesn't work with WSL2

Just installed the latest insider build 18970 with Ubuntu 18.04 on WSL2 with .zsh and running wslgit.exe from PowerShell just opens a new bash session. Please see screenshot below

Edit: tried to set Windows environment WSLGIT_USE_INTERACTIVE_SHELL variable to 0 and it didn't works the same way

Thanks for reporting this issue.
Does manually running
wsl bash -c "git --version"

from cmd or PowerShell still work with WSL2, or were there any changes to the command line interface?
You may also try an unreleased alpha build of wslgit, which changed/unified the invocation of git.

Manually running wsl bash -c "git --version" in PowerShell gives me a version of git installed in wsl and returns back to PowerShell. I ended up installing git on windows, but I will try to use the alpha build when I have time.

I'm on 191025 and wslgit works for me. My mount points are on / and not /mnt/c. I've cloned a repository into my home directory, which is then shared to\\wsl$\Ubuntu-18.04\home\me\something. I've then linked /mnt/c/www/something => /home/me/www/something and C:\www\something => \\wsl$\Ubuntu-18.04\home\me\something.
Now I can use PHPStorm with the project directory  C:\www\something (it doesn't allow paths with an '$') and C:\Tools\wslgit\wslgit.exe as git executable. For PHPStorm's git test to work, I also needed to link /mnt/C/Users => /C/Users.
Maybe wslgit could offer some options to configure those paths. So creating symlinks won't be necessary. Also a way to use a non-default distro.

@phox4ever I'm not sure that this is possible like you described.
I created a symlink:
ln -s /home/me/www/something /mnt/c/www/something  which creates a file with name something in C:\www\something
Then I tried:
mklink /d C:\www\something \\wsl$\Ubuntu-18.04\home\me\www\something and it complains that the file with the name something already exists.
Can you please describe the exact commands you used to create these mappings?

@stevium My mount root is '/', not '/mnt/'. Also, I was using wslgit version 0.8.0. The latest version uses a different method of handling paths. I'm currently running PhpStorm within WSL2 and x410 which works straightforward (until you put your Laptop to sleep...).
My /etc/wsl.conf:
#enable extra metadata options by default
[automount]
enabled = true
root = /
options = "metadata,umask=22,fmask=11"
mountFsTab = false

@stevium Ah I see now, having the root mounted on / will not create a /mnt/c/... symlink in C:\... directory and wslgit mappings would still  work. Nice idea! Sorry for not noticing at first.
Yes, now when wslgit is updated we don't need this mapping anymore.  I tried the develop branch, and looks pretty functional.
Thanks for helping!

@stevium Version 0.9 is using wslpath, which cannot handle UNC paths, symlinks to UNC paths and network drives. But I'm thinking of wrapping wslpath in a bash script which overrides certain Windows paths to local Unix paths on the fly. Or is there an option coming up to create those path mappings for wslgit?

I only had problems with 0.9 when I wanted to use it with IntelliJ. That's why I decided to try the develop branch. And it seems to be handling UNC paths (\\wsl$\...) pretty fine. I was able to execute all the essential git commands, and even use Intellij on a large git project with a lot of submodules.
Maybe I'm missing something, but I don't see that these mappings are necessary anymore...
What exact command is failing for you?

Not sure about IntelliJ, but for PhpStorm, using UNC paths directly is not possible. https://youtrack.jetbrains.com/issue/IDEA-123273. I haven't tried with version 2020 yet though. So even if wslgit can handle \wsl$, PhpStorm still needs a Symlink. Not sure if this works also.

Most of the UNC issues were resolved with 2020.1 release: https://youtrack.jetbrains.com/issue/IDEA-197573 I've tried Intellij Idea, Rider, and Clion 2020.1 all of them supporting wsl2 and wslgit. So I would expect PhpStorm also to support it.

I've tested it with PhpStorm 2020.1 and I can confirm it works with the develop branch! Thank you.


Https server hand on close_fn

Moving my investigation here https://github.com/espressif/esp-idf/issues/11738
Summary: If there are multiple threads doing job when https server is running there is a chance it will hang. So far last thing that gets called is close_fn callback. It is reproduced if I put load on other thread when https call is made
It hangs at
https://github.com/esp-rs/esp-idf-svc/blob/cf0b85c7d92c7382cd87d5abb6b3a1bede70de6f/src/http/server.rs#L447
Note: Not sure why websocket close handler is even called when I open Https endpoint
I think I found where it locks, but not sure why it would lock.
pub struct Api {
    ...
    ws_listeners: Arc<Mutex<HashMap<i32, EspHttpWsDetachedSender>>>,
}

            ...
            .ws_handler::<_, WsError>("/ws", move |con| {
                log::info!("My ws handler is called");
                if con.is_new() {
                    let session_id = con.session();
                    log::info!("Ws connection open: {session_id}");
                    let detached_sender = con.create_detached_sender().unwrap();
                    ws_listeners
                        .lock()
                        .unwrap()
                        .insert(session_id, detached_sender);

                    return Ok(());
                }

                if con.is_closed() {
                    let session_id = con.session();
                    log::info!("Ws connection closed: {session_id}");
                    ws_listeners.lock().unwrap().remove(&session_id);
                    log::info!("Ws connection closed unlocked");
                    return Ok(());
                }
                Ok(())
            })?;

            ...

            let mut listeners = self.ws_listeners.lock().unwrap();
            for (_, sender) in listeners.iter_mut() {
                sender
                    .send(embedded_svc::ws::FrameType::Text(false), frame.as_bytes())
                    .map(|_| log::info!("send: {}", frame))
                    .map_err(|e| log::error!("Error sending ws frame: {:?}", e));
            }

Stuck at ws_listeners.lock().unwrap().remove(&session_id);. I'm not sure how it managed to deadlock tho

Has anyone used websockets where messages are sent out from another thread? Still not sure why it locks, but I wonder how I should change the logic for it to avoid locks. I could use try_lock, but then I risk of going into bad state

Thanks for explanation!
Now I use try_lock for ws_listeners.lock().unwrap().remove(&session_id); it is mostly called from http call so it doesn't remove anything anyway. Hope this will be fine until a more mature async HTTP server comes along.
A while back I implemented whole async websocket stack
smol = "1.2"
async-tungstenite = { version = "0.19", features = ["async-tls", "async-std-runtime"] }
futures = "0.3"
tungstenite = { version = "0.18", features = ["rustls-tls-native-roots"] }
async-tls = "0.12.0"
rustls = "0.20.8"

It was tricky and I even had to make https://github.com/briansmith/ring compile for esp32s3. In the end I got async WS working, but rustls was just too resource-heavy for the chip, so I dropped the idea.


Pictures

Remove the pictures of the main page (pictures from articles) and replace with the new one.
Old pictures > https://i.goopics.net/Gda3v.png
New pictures >

I have this result, it's clean 💪

Nice, could you add a timer which
automatically scroll pictures ?

A yes good idea

I have deployed the changes, what do you think ?

It's look good, I like it, good job.
I close this issue ✅


Support editing the currency list

While it would be better to support all possible currency tags the trade system supports, a simpler option would be to make it possible to edit the currency list that can be used as prices, for example I'd like to set exalted-shard as price for my chaos to buy shards without messing with the code manually.
Another option, custom currency tags might be typed directly from UI, if it's not going to mess up the current simplicity.
Supported tags:
https://www.pathofexile.com/trade/about
http://currency.poe.trade/tags

This is a nice idea, the tags could be read from a config file, which could be edited to enable/disable more currency types.


fake commit

Thanks for contributing!
chnges made are not valid

I'm not sure what this is, so I'm closing it.


Fix issue with unit test

Description
The unit test fails to run because it the fs module is mocked out and not loading the template correctly.
Done When

[x] The unit test if fixed and running;
[x] SonarCloud is happy;
[x] Merge in dependabot functionality.

All fixed.


Music Album and Genre Classes

For this feature I did the following

Created the music album and genre files
Created the music album and genre classes
Created the initialize methods for the music album and genre classes

Hi @Chimoneg27
Status: APPROVED 🟢
Your project is complete! There is nothing else to say other than... it's time to merge it: ship it:
Congratulations! 🎉 👏 👏 💯
Highlights!
• Followed Gitflow ✔
• Good commit messages ✔
• Linters are passing ✔
• Descriptive Pull request ✔


WASAPI multi-channel microphone input

Support multi-channel microphone input by downmixing to stereo

Enhanced the WASAPI audio driver to support microphones with any number of channels.
This modification ensures compatibility with a broader range of multi-channel microphones, allowing for more versatile input handling in engine.
Regarding “WASAPI: unsupported channel count in microphone!” error.

Related issues
This is a pull that I created in the Godot.
I tried compiling and testing the four channel microphone on my laptop, and it turned out to be error free. This enables the successful use of multi-channel microphones in Godot.
I made the assumption that the microphone might have a potential configuration with odd channels (e.g., Left, Right, Center), although such configurations are uncommon (in some professional audio equipment). The goal was to ensure that the microphone is at least usable within Godot, regardless of the channel count. The WASAPI documentation itself doesn't explicitly specify how multi-channel microphones should be handled, so I aimed for a more general solution to accommodate various configurations.
Thank you for reviewing !

This needs to be rebased, aside from that, do you have people who we can speak to to test the robustness of this?

Yeah I just wanted to get as many testers I can speak on this as possible, other than that it just needs to be squashed down to one PR and I don't see a reason it wouldn't be merged quick.


Issue with manual sorting in the roadmap

As a roadmap manager, I'd like to sort entries in my roadmap so I can set and see work sequence and priorities. Manual sorting works in "Now" column, however in "Later" or "Next" my changes do not save and entries are sorted in a random order.

I use ClearFlask on:

[x] Self-hosted (Own domain)
[ ] Cloud (clearflask.com)
[ ] Haven't tried (yet)

My financial contributions to ClearFlask:

[x] I use a free Plan
[ ] Thinking about it
[ ] Monthly / Yearly Plan
[ ] Lifetime Plan

I was able to reproduce it. I wrote a fix here. I will close this ticket once I verify the fix is working. It should make it to release 1.7.4 in a couple of days.

There is a caveat: you need to manually re-order the items to fix the issue. Sorry for the inconvenience.


Examples should link to absolute module path

The examples currently use a relative path to the parent submodule. Here's one instance in the KMS example
It would be good to use remote absolute paths that users could follow along without downloading the repo first.

@dtest the examples are just to show different scenarios how the module can be used .
you dont need to download the module you can also set the source to
git
module "cloud_spanner" {
  source = "git::https://github.com/GoogleCloudPlatform/terraform-google-cloud-spanner"`
  ...
  ...

or registry once it available there
module "cloud_spanner" {
  source  = "terraform-google-modules/cloud-spanner/google"
  ...
  ...

Thanks @anaik91 . The reason I opened this is that I think the examples should use the absolute options, rather than the remote. That way people who are maybe not as familiar with modules don't hit a wall when trying to use them.
Do we have a sense of when/if the module will be on the registry?


Update looping-strategies-matrix-repeat-and-parallelism.md

Updated indentation for maxConcurrency

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65c0db6bb16a011189852bb2--harness-developer.netlify.app


Use actions/github-pages to publish docs

Uses the official actions/github-pages action to publish docs to GitHub Pages. This should remove the need for gh-pages branch

At present this does not fit with the skeleton documentation model.
Skeleton keeps track of multiple versions of documentation and has a Docs Cleanup job to remove documentation on deleted branches. In future we could support the same features with actions/github-pages by fetching and mutating the documentation artifact. For now we will continue to use peaceiris/actions-gh-pages, in future this approach may be deprecated - review at this point.

Also, this PR broke the existing gh-pages deployment, I had to "unpublish", then change the gh-pages root and put it back to restore it.

Also, this PR broke the existing gh-pages deployment, I had to "unpublish", then change the gh-pages root and put it back to restore it.

Oops, sorry about that. Presumably the work of actions/configure-pages


using local db for DSN value

Hi, when I do the folllowing;
conn, err := gorm.Open(sqlite.New(sqlite.Config{
		DSN:        "file:local.db?parseTime=true",
		DriverName: "libsql",
	}), &gorm.Config{})

A file appears called "file:local.db?parseTime=true" and I don't think this is the intention with sqlite?
New to this driver

Hmmm interesting. I spent an hour this morning trying to replicate this and couldn't. My file was always created as "local.db". Any chance you could get a minimal repo for me or gist?
Have you tried this with the standard sqlite driver from Gorm? I doubt this driver is specifically causing the issue given how lightweight it is but I will certainly look into it.

Hi, im closing this as I'm archving the repo since the original change has been merged in. Reach out to me on twitter if you have any issues


finetune model error

When trying to fine tune model on custom dataset with 15 classes. I get
Traceback (most recent call last):
  File "tools/train.py", line 98, in <module>
    main()
  File "tools/train.py", line 78, in main
    train_dataset = get_dataset(cfg.data.train)
  File "/media/bubbles/fecf5b15-5a64-477b-8192-f8508a986ffe/ai/ryan/mmdetection/mmdet/datasets/utils.py", line 102, in get_dataset
    assert len(img_prefixes) == num_dset
AssertionError

I have changed the model like this and i am doing load_from --
   import torch
pretrained_weights  = torch.load('checkpoints/faster_rcnn_r50_fpn_1x_20181010-3d1b3351.pth')

num_class = 15
pretrained_weights['state_dict']['bbox_head.fc_cls.weight'].resize_(num_class, 1024)
pretrained_weights['state_dict']['bbox_head.fc_cls.bias'].resize_(num_class)
pretrained_weights['state_dict']['bbox_head.fc_reg.weight'].resize_(num_class*4, 1024)
pretrained_weights['state_dict']['bbox_head.fc_reg.bias'].resize_(num_class*4)

torch.save(pretrained_weights, "checkpoints/faster_rcnn_r50_fpn_1x_%d.pth"%num_class)

Fixed this


v0.6.1 fails inside docker

So, this is a bit vague, I'm happy to try to figure out more info if you need it, but mailcatcher fails on 0.6.1, but works on 0.6.0.
I'm running inside the stock dockerhub ruby image:
root@90276bbd1bfe:~/mailcatcher# mailcatcher --smtp-port 4025 --http-port 4081 --ip <IP_ADDRESS>
Starting MailCatcher
~~> ERROR: Something's using port 4025. Are you already running MailCatcher?

Nothing is running on those ports. 0.6.0 works as expected, and 0.6.1 binds to localhost fine. It looks like to only change was to eventmachine, so it might be their problem. I'm not sure what the resolution might be, but let me know if you need additional info/testing.

Hmm, the problem is more complicated than this, I think. Hold please.

Total PEBCAK, sorry about that.


Logging format configuration is not respected

There's a mix of log formats used by chef browser and the user's log configuration doesn't seem to be respected.
Using this config:
[logging]
format=console
level=info

[Unit]
Description = chefbrowser service
After = network-online.target

[Service]
Type = simple
ExecStart = /path/to/chefbrowser --config /path/to/chefbrowser.ini
User = chefbrowser
Group = chefbrowser
StandardOutput = append:/var/log/chefbrowser/chefbrowser.log
StandardError = append:/var/log/chefbrowser/console.log
ExecReload = /bin/kill -HUP $MAINPID
SuccessExitStatus = 3
Restart = on-failure

[Install]
WantedBy = multi-user.target

We see all of the following in /var/log/chefbrowser/chefbrowser.log:
2023-11-23T02:10:47.167Z	info	registering API routes
2023-11-23T02:10:47.167Z	info	registering UI routes
2023-11-23T02:10:47.167Z	info	starting web server on <IP_ADDRESS>:8080

⇨ http server started on [::]:8080

{"time":"2023-11-23T02:11:04.040430545Z","id":"","remote_ip":"<IP_ADDRESS>","host":"chef-browser.example.com","method":"GET","uri":"/ui/nodes/foo.example.com","user_agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36","status":200,"error":"","latency":36831518,"latency_human":"36.831518ms","bytes_in":0,"bytes_out":315284}

Partially described in https://github.com/labstack/echo/issues/1240


SqlPackage/SSDT table rebuild losing Change Tracking CHANGETABLE data

There are various scenarios where sqlpackage generates a table rebuild script e.g. adding a column in the middle of a table.
However, when this is done, all Change Tracking data for the table held in the CHANGETABLE is lost. This is dangerous in production environments where certain data that should be synchronised is also lost due to it never being recognised as changed.
It seems that SSDT/SqlPackage and Change Tracking are completely incompatible.
Steps to Reproduce:

Enable Change Tracking on database + table
Insert record into table
In SSDT add column in middle of the table
Publish database via SSDT/SqlPackage

Did this occur in prior versions?  If not - which version(s) did it work in?
As far as I'm aware this has never worked.
(DacFx/SqlPackage)

Hi @ssreerama,
Just to clarify, this issue is present for ANY table rebuild scenario, not specifically the new column insertion. Is there an exhaustive list of these scenarios that require a table rebuild documented anywhere?

We didn't have that scenario documented anywhere, but I totally understand this table rebuild happens for some specific scenarios like columnstore index changes, table column updates etc.


fixed bug in cloudformation template preventing extra_json from being…

… provided to chef cookbooks

See: https://github.com/awslabs/cfncluster/pull/83
For comments on topic.
Closed that pr as requested.

Can you also update for the ComputeFleet - https://github.com/clausman/cfncluster/blob/master/cloudformation/cfncluster.cfn.json#L2616

Whoops, missed that one, nice eye.

Looks good. Will be pulling into 1.1.0.


Newest version not on Pypi

Newest version of Pypi is 12.4a1, not the newest release 12.4

@camilonova committed a new commit bumping the version number, and committed the Git tag v12.4. Any member of Jazzband can commit new commits, however, we can't yet release anything on PyPI since we're still waiting on Jazzband roadies to set this up. Unfortunately, since he pushed a new Git tag, this is difficult to reverse.
Here are the relevant issues that we're waiting on until we can release the next version on PyPI:

https://github.com/jazzband/sorl-thumbnail/issues/507
https://github.com/jazzband/roadies/issues/69

Please go upvote them.


[FEATURE] DlSmartSearch - omit specific operators from the suggestion window

new prop called Omit-suggestions
is an array of values of the operators and any string
If the value exists in omit-suggestions then we will not display it in the suggestion menu

@fadiDL say we omit "OR" and then someone just types it without suggestion. What do we expect would happen?


Docker

is there any plan to move it to Docker?

Hi @idirouhab,
We're planning to write an article about how to dockerise your Golang project for our recently published blog. Our idea is to dockerise (in parallel) our GQL repo.
Anyway, thanks a lot for your suggestion.
Cheers,
Joan


Factorialize a Number

Challenge Factorialize a Number has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.71 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
Can someone please explain why this won't let me pass?  All of the scenarios are correct when I check them.
My code:
  var poop = 1;
function factorialize(num) {

  for (var i=1; i <= num; i++){
     poop*=i;
  }return poop;}

factorialize(20);

@jdavis4989 thanks for the issue. The tests are run sequentially right after each other so any global variables changed will change with each test run. Put your global variable within your function and it should pass. If you get stuck or have questions with a challenge in the future, please first ask questions to the Help Room. Happy coding!

That did it!
Thanks Eric!


Make decision choices about combat state

There are some useful things to know about combat

is the user in combat?
is the user currently executing a skill and is unable to perform any other operation?
is the user out of ammo? (cancel packet)
is the user out of mana? (cancel packe)

Where to store this information? Combat extensions? Game? Combat extension already references game so there would be no harm in storing it in game and using it from combat extensions?

If the bot is going to use combat, it should target Combat extensions anyway.
If the bot is not going to use combat, there is no reason to know combat information.
Thus it looks to me putting the information into combat extensions does not hurt. That's the decision.


nagios_host_context does not affect name when nagios_hostname_type=host

If I set nagios_hostname_type to unit, the nagios_host_context allows me to identify a site's units by the context - if I set nagios_hostname_type to host, that nagios_host_context is ignored and the hostname itself is all that is used.  This means sites with cryptic hostnames (i.e. 99% of sites) wind up being hard to identify when we aggregate checks from multiple sites.

Imported from Launchpad using lp2gh.

date created: 2017-08-17T22:19:57Z

owner: xavpaice

assignee: xavpaice

the launchpad url

(by xavpaice)
https://code.launchpad.net/~xavpaice/nrpe-charm/+git/nrpe-charm/+merge/329238

(by mthaddon)
Have confirmed this change is in cs:nrpe. Marking as Fix Released


Update yew example: use trunk, change stdweb to web_sys

Upstream, Yew has moved to the web_sys away from stdweb. This PR updates the Yew example to do the same. It also removes use of cargo-web which has fallen out of use in favor of trunk.

Thanks for your contribution! Sorry it took us so long to get back to this repo.


Checking that sampling rate and sig_time match

There could be a warning given to the user

Wom't fix for now


🐛 [Stream Firestore to BigQuery 0.1.48] Export no longer works

Hello everyone,
the Firestore export to BigQuery no longer works for me since version 0.1.48.
No more data is transferred.
When I set up a new customer, the dataset Firestore_Export is still created, but no export tables are created. With the previous versions, everything worked without any problems.
The basic configuration settings are the same.
Is this a known problem?
Best regards
Marco

When I set up a new customer, the dataset Firestore_Export is still created, but no export tables are created. With the previous versions, everything worked without any problems.

Could you please elaborate more? Do you mean when you install a new instance of the extension, the export table isn't created hence no changes are recorded?

same, no export tables created and for me it says processing for hours

@j05u3 thanks for sharing your issue, we're looking into it.

Hi @j05u3 and @GD-MarcoBug, thank you for reporting this issue. Could you please share the configuration settings of your installed Stream Firestore to BigQuery extensions? Feel free to redact any sensitive information.

Also wondering if you installed a new instance of the extension or updated from a previous version?

Hi @pr-Mais and @jauntybrain,
first of all, many thanks for the quick support.
The problem occurs both with a new instance and when updating to the latest version.
The following is a basic configuration, for new instances, it gets stuck in processing (last screenshot):

For your information. The problem also occurs with 0.1.44.
Since 20240412

I found that when I tried to stream from firestore in project A to bigquery in project B, I had to manually add the function's service account permissions in project B. The extension would then be able to add the dataset and table in bigquery. However, the extension would be stuck on processing until I manually added the required routines in BigQuery too.

@joeframpton what are those "the required routines in BigQuery " and how to add them? eager to make it work again 😀

@GD-MarcoBug do you know if the extension works on versions before 0.1.44?

Investigating this now, testing a fix. Thanks for your patience!

Same issue here. Creation of the tables does not work.

@joeframpton what are those "the required routines in BigQuery " and how to add them? eager to make it work again 😀

Hi, so the workaround I managed to implement (which autoinstalled the routined for me - alternatively you can install them following this: https://cloud.google.com/bigquery/docs/samples/bigquery-create-routine) was the following:

Run the fs-bq-import-collection script across projects (although it failed too - Issue raised here. This created the dataset and table in bigquery in the second project
Imported a schema using fs-bq-schema-views to the bigquery in the second project
Installed this extension to set up the stream across projects - and then manually added the service account from this extension to have BigQuery Data Editor & Cloud Datastore User in the second project
Run the fs-bq-import-collection script to import the data to bigquery in the original project
Set up the service account from the second project to have permissions in the first project
In Big Query in the second project, access the data from the original project's big query and merge into the data that's being streamed from step 3

That's what worked for me! There might be superfluous steps in there but I hope this is helpful

:wave: eagerly waiting for the fix, thanks for working on it!, in the meantime, is possible to run import script multiple times to update existing / updated documents? I'm running it but it skips over existing updated documents in my collections. I'm effectively looking for an "upsert" mechanic to hold me over until the extension is fixed.
I've also removed the extension and reinstalled the last known good extension version (0.1.42), but no updates are making it through to BigQuery.
Thanks, any help will be appreciated.

Today hopefully!

<EMAIL_ADDRESS>
I still see same problem

Runtime Status: Processing
Configuring BigQuery Sync.

table not created and errors in console: Unhandled error ApiError: Not found: Table

Strange, it had fixed it for me. I will investigate further.

I believe it could be an issue with the onConfigure logic but I won't get a chance to look at it and release until Monday, most likely. If it's urgent you could try installing a fresh instance of the extension as a workaround.

It may be intended behaviour not to create a table onConfigure actually, you might need to install a fresh instance to create the table

i was creating a new extension installation for 0.1.49, not just refreshing already existing one
or what do you mean "install a fresh instance to create the table"?

In that case I'll try and reproduce it again tomorrow  and see what's going wrong. Must be something else up with it, as the RC worked yesterday. Thanks for your patience

I'm having trouble reproducing the bug today with 0.1.49, do you have any additional logs?

@olegdater I don't believe you can have multiple firestore fields in a comma separated list, for
"Firestore Document field name for BigQuery SQL Time Partitioning field option (Optional)"
Our regex validation will need to be changed to reflect this, right now that field has no validation. I will open a new issue to track this.

thanks @cabljac ! confirmed, it now works 🎉