Deprecate object mode

Historically "recent" (since early ~2014?!) Numba has had the object mode compilation path which will attempt to compile any python code into a compiled function. This compiled function treats all variables as Python objects and uses the Python C API to perform all operations on those objects. The performance of the compiled function is often no better, and can even be worse, than simply interpreting the original source in the CPython interpreter.
Object mode is accessed by either setting the numba.jit decorator kwarg forceobj to True or via a fall back mechanism where compilation of a numba.jit decorated function is first tried through nopython mode and if this fails object mode is used instead. The fall back can be confusing in that seemingly innocuous changes in user code lead to drastic performance changes as code which may have once compiled in nopython mode may switch to object mode silently. e.g.:
from numba import jit

@jit
def foo():
    l = []
    for x in range(10):
        l.append(x)
    return l

foo()

assert foo.nopython_signatures

@jit
def bar():
    l = []
    for x in range(10):
        l.append(x)
    return reversed(l) # innocuous change, but no reversed support in nopython mode

bar()

assert not bar.nopython_signatures

Another reason to remove the fall back is that it is confusing for the compiler engineers as it causes internal state problems that are really hard to debug and makes manipulating the compiler pipelines incredibly challenging.
Object mode itself is not without merit, there are some minor but important uses in the case of a with objmode: context managed block which permits a section of a nopython mode function to run in object mode. It is therefore not appropriate to completely remove this mode and compilation pipeline.
Further, it has long been considered best practice that the nopython kwarg in the numba.jit decorator is set to True and that any user effort spent should go into making code work in this mode as there's very little gain if it does not. The result is that, as Numba has evolved, the amount of use object mode gets in practice and it's general utility has decreased. It can be noted that there are some minor improvements available through the notion of loop-lifting, whereby a loop that can be compiled to not use the Python C API inside an object mode function is "lifted" out into a nopython mode function. The cases of this being used in practice are, however, rare and often a legacy from less-recent Numba whereby such behaviour was better accommodated/the use of @jit was recommended.
Proposed changes:

numba.jit will compile code in nopython mode by default.
numba.jit will compile in object mode only if forceobj=True is set as a kwarg in the decorator.
The fall back behaviour is removed entirely.
Numba internals ends up with two distinct pipelines, one for nopython mode and one for object mode.

Summary of benefits:

Less confusion for (especially new) users, i.e. numba.jit either manages to compile in nopython mode and benefit is immediately felt, or the user is told there is a compilation problem, this prevents reports of "numba.jit did nothing for performance".
Encourages users to pursue nopython mode for better performance.
Makes it much harder for users to create accidental performance regressions.
Less confusion for developers debugging/manipulating a state heavy compilation pipeline.

Proposed method for implementing changes:

Issue a deprecation warning to users using the numba.jit decorator with the value of the  nopython kwarg set to False (the default value at the time of writing). This message should convey that the default for the nopython kwarg will at some time in the future become True. Once the default is updated, object mode will still work, but users will have to opt-in to using it. Write an entry in the documentation about why this change is being made, explain to users how to adjust their code differentiating between code which can be compiled in nopython mode and code which cannot. Also explain why in general object mode is not going to yield good performance and encourage users to remove the decorator/fix their code to work under nopython=True. Link to documentation from warning. Fix Numba testing code to silence this warning.

Deprecate the "nopython falls back to object mode" behaviour, a warning can be issued from the use of the fall back path in the pipeline that describes that a) it is happening and b) if the user wants objmode to use forceobj=True kwarg. Relevant documentation can be provided to link to from the warning message text. Fix Numba test base to silence this warning.

Set the nopython kwarg default to True and make it so that it becomes an error to set it to any other value, the reason for not removing the kwarg is its extensive use in the wild (due to having to constantly spell it out at present and recommending this as best practice). The error message can refer to documentation about why this is so and also mention forceobj.

Split the nopython and object mode pipelines so there's a pure variant of each pipeline with no interference between the two, this makes it easier to debug and implement pipeline changes. The default is now to always compile in nopython mode, users wanting object mode can opt in with forceobj=True. This internal change should be transparent to the user base as the "fall back" behaviour would have been removed earlier from the user API entry points by virtue of the decorator kwarg default changes.

It may be most profitable to perform steps 1 and 2 simultaneously so that there is a loud and clear warning for all users that are relying on object mode/object mode fall back that they will have to change their code at some point in the future. This change being to either:

use forceobj=True
only decorate functions that can be compiled in nopython mode.

I should add... edits/ideas/comments are most welcome!

@stuartarchibald, with so many libraries implementing Numpy's API it is very common to write functions that work on many types of array-like objects, for example Numpy and Dask arrays. What would be the best way to decorate functions such that when receiving a Numpy array they are jitted but when receiving an array that Numba cannot handle they fallback on pure python? I am talking about parameter-type fallback, not line-by-line fallback as currently.
would it be possible to retain some opt-in fallback for when functions receive a type that they cannot handle? Or should users write overloads for their own functions, such that when the functions receives non-Numpy arrays they return a pure python function?

@luk-f-a please could you open a new issue about the above, and if possible please provide a couple of examples, thanks.

Not sure where this comment fits better here or in #4159. I, too, rather consider current fallback behavior to an object mode as an error in my code than a useful feature. Therefore, I would vote for:

The fall back behaviour is removed entirely.

On the other hand, my experience tells me that there is some (in many cases significant) difference in an idiomatic code that I write with numba in mind and without. In other words, if numba.njit does not work, then I’m not just deleting the decorator, but almost always completely rewriting the function in a different style. And this of course contradicts my first paragraph.
Of course at the moment it is only a superficial idea, but what if we have the ability to explicitly control the fallback behavior through new fallback kwarg? Like:
@nb.njit(..., fallback=None)  # raise if it can't compile to nopython

@nb.njit(..., fallback=some_callable)  # call some_callable if it can't compile to nopython

@nb.njit(..., fallback=nb.object_mode)  # current behavior -> fallback to object mode

where nb.object_mode is some singleton. What do you think?

Thanks @godaygo I've added a note to #4159 to make sure this comment is considered.

xref: https://github.com/numba/numba/issues/4159#issuecomment-501000264

#8702 deprecates the fall-back to object mode behaviour.


fix(VERSIONED_EXPORTS): Ensure dashboards and charts adhere to the VERSIONED_EXPORTS feature flag

SUMMARY
It seems like charts, databases etc. adhere to the VERSIONED_EXPORTS feature flag with regards to whether one can export said entity however dashboards and datasets don't. This PR remedies the issue.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

When VERSIONED_EXPORT is set to False
Before

After

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@imanollew I think your issue is being tracked at https://github.com/apache/superset/issues/26417


Revert "merging PR #471: Mention meritocracy when needed"

This change was using the DB, which broke a lot of stuff. Reverting until the DB gets fixed.
@rudehn when you do the database redo PR, would you mind un-reverting this? Thanks.

 Chaosbot is down right now. I'm not exactly sure what's happening, but I'll the tracebacks I've seen seem to be coming from the database code.

Looks like @amoffat got it working after all. Closing.


Unable get open mod menu

I installed your mod and activated it in the remix menu( had to restart the game to complete installation). Now when i double click on SimplifiedMoveset in remix mod menu, it say no menu available for this mod. I tried to play spearmaster campaign and found no change in beam climbing or anything. what am i missing?

This is a common issue (for all mods). Disable and enable all mods (you need to restart twice). If that does not help then check out the Section FAQ for other fixes.

I disabled all mods and enabled them couple of times now, restarted thrice.
when I restarted, one time I got the prompt saying "mods are changed and needs to update something", I clicked continue. It finished updating something. But still, double clicking on SimpliedMoveset in the remix menu says, this mod doesnt have remix ui.
My RW version is 1.9.15 and SimplifiedMoveset version is 2.5.3

There are other fixes. But these are some old files did not get updated correctly (RW v1.5 -> v1.9+). I assume that is not the case for you. If you are on Steam Deck or Linux there are specific things you have to do to use mods there.
You can try to disable all mods and only enable this one.
When you have to click continue then a mod got updated (Steam downloaded an update). This can also trigger the bug again. The problem is that the game can mess up the loading of mods. (If a mods fails to load for any reason it stops loading further mods.) The option menu is just a symptom. These mods are not loaded.

I disabled all other mods and kept only this, but the problem still remains.
I am on Linux

Check the FAQ and maybe the Rain World Discord server. Mods don't work on Linux by default. In the FAQ I posted a solution that someone posted in the comment section.

Nice!


Resource limits does not work

Describe the bug

deploy a pod and set the resources limit  cpu:200m.
run the command "while true" in that container
get the top info to see the cpu percent

Expected behavior
excepted : cpu 20%
but whatever i set to the limit ,cpu  percent  is always 100%.
Additional context
OS: centos 7
kernel ver: 3.10.0-957.10.1.el7.x86_64
k3s ver: 0.4

When booting up k3s it logs Disabling CPU quotas due to missing cpu.cfs_period_us.
Maybe it's related to this issue and helps.

I got into this while executing this https://kubernetes.io/docs/tasks/configure-pod-container/assign-memory-resource/
This creates the pod but expected OOMKilled on the next section is not happening:
kubectl apply -f https://k8s.io/examples/pods/resource/memory-request-limit-2.yaml --namespace=mem-example

....
kubectl get pod memory-demo-2 --namespace=mem-example

@mengyangGIT  I am not able to reproduce the issue with the latest k3s version, here are my steps:

start k3s server
install metrics server
run the following yaml:

apiVersion: v1
kind: Pod
metadata:
  name: cpu-demo
  namespace: cpu-example
spec:
  containers:
  - name: cpu-demo-ctr
    image: vish/stress
    resources:
      limits:
        cpu: "1"
      requests:
        cpu: "0.5"
    args:
    - -cpus
    - "2"

run kubectl top pods -n cpu-example

Result:
I can see that the limit is honored correctly:
✗ k top pods cpu-demo -n cpu-example
NAME       CPU(cores)   MEMORY(bytes)
cpu-demo   991m         1Mi

I also tried the example with exceeding the cpu limit:
apiVersion: v1
kind: Pod
metadata:
  name: cpu-demo-2
  namespace: cpu-example
spec:
  containers:
  - name: cpu-demo-ctr-2
    image: vish/stress
    resources:
      limits:
        cpu: "100"
      requests:
        cpu: "100"
    args:
    - -cpus
    - "2"

which requests 100 cpu, and the pod didn't start as expected:
k describe pods/cpu-demo-2 -n cpu-example
.....
Events:
  Type     Reason            Age                  From               Message
  ----     ------            ----                 ----               -------
  Warning  FailedScheduling  27s (x5 over 6m24s)  default-scheduler  0/1 nodes are available: 1 Insufficient cpu.

@joaovitor I was able to reproduce this case, the OOMkiller doesn't seem to be invoked, however I noticed that the container is not exceeding the memory limit configured for the pod running the stress command:
k top pod -n mem-example
NAME            CPU(cores)   MEMORY(bytes)
memory-demo-2   18m          99Mi

bash-4.3# ps -o pid,user,rss,vsz,comm ax
PID   USER     RSS  VSZ  COMMAND
    1 root        0  740 stress
    6 root      83m 250m stress
    7 root      196 6212 bash
   15 root        4 1520 ps

root@pop-os:/sys/fs/cgroup/memory/kubepods/burstable# cat pod51a6febe-4d87-4c7c-beff-5ead07df2da5/cad6693e3974a359b4ea0ef193a4998bce376e45a1fdcacc671e9643bcab1096/memory.limit_in_bytes
104857600
root@pop-os:/sys/fs/cgroup/memory/kubepods/burstable# cat pod51a6febe-4d87-4c7c-beff-5ead07df2da5/cad6693e3974a359b4ea0ef193a4998bce376e45a1fdcacc671e9643bcab1096/memory.usage_in_bytes
103915520

I was able to see the OOMkiller being invoked in a rke cluster with the same yaml file
cc @erikwilson

@joaovitor The issue is happening because the swap is enabled on the system, If the swap is enabled then the OOMkiller will not be triggered until there is no memory left in the swap.

Closing as it is expected behavior when swap is enabled

Cpu limits not work because the flag hasCFS checkCgroups  return is false
I found that in kernel 3.10.0-x, the cpu subsystems in /proc/{pid}/cgroup is cpuacct,cpu while in /sys/fs/cgroup it is cpu,cpuacct  (out of order) , which make k3s find wrong path of cpu.cfs_period_us.

The fix is not working for me:
k3d running k3s:
k3d cluster create 1-20 --image rancher/k3s:v1.20.5-rc1-k3s1

logs:
docker ps
CONTAINER ID        IMAGE                          COMMAND                  CREATED             STATUS              PORTS                             NAMES
cad43f091333        rancher/k3s:v1.20.5-rc1-k3s1   "/bin/k3s server --t…"   28 minutes ago      Up 28 minutes                                         k3d-1-20-server-0

docker logs cad43f091333
...
time="2021-03-27T20:56:01.397056042Z" level=warning msg="Disabling CPU quotas due to missing cpu.cfs_period_us"
...

docker exec cad43f091333 k3s --version
k3s version v1.20.5-rc1+k3s1 (355fff30)
go version go1.15.10

@pkoltermann can you confirm that it does not work when run outside of docker? I suspect docker may not be presenting all the correct cgroups to enable nested resource limits.

@brandond You are right, if I run it on the host machine it works. The question is how to make it work in docker?

I would probably take this question to the k3s issue tracker.

@galal-hussein I can only confirm that after disabling swap (it gets re-enabled after reboot in my case) and restarting k3s service the memory limits started working as expected. On Ubuntu I did the following.
Turn off all swaps
swapoff –a
Restart k3
systemctl restart k3s.service
After that the memory 'thirsty' pod was being restarted each time it reached the memory limit.


Loadable error when trying to use GLTF model

Hello, first, thank you for such an awesome starter for Next! I'm currently running into an issue when trying to load a GLTF component generated from GLTFJSX. I'm trying to use this model currently and import it similar to the default Sphere. However, upon loading I keep getting this error no matter the model I try to load:

Any ideas on what might be causing this? You can see my current code here. I was able to load the model on a Gatsby R3F starter which leads me to thinking this is NextJS specific. Any help is greatly appreciated! 🙏

Hello! Because you are loading using a dynamic component it's overriding the error.
I fixed your example without using dynamic component, this is the working index.js :
import { Suspense } from 'react'
import useStore from '@/helpers/store'
import Model from '@/components/canvas/Guitar'

const Guitar = () => {
  return (
    <Suspense fallback={null}>
      <ambientLight />
      <Model />
    </Suspense>
  )
}
const Page = () => {
  useStore.setState({ title: 'Sphere' })
  return (
    <>
      <Guitar r3f />
    </>
  )
}

export default Page

After that, it was possible to debug and it would just notify that you missed a Suspense since you're loading some data. So here is your example fixed with a dynamic component for better performances :

 import dynamic from 'next/dynamic'

const Model = dynamic(() => import('@/components/canvas/Guitar'), {
  ssr: false,
})

const Guitar = () => {
  return (
    <Suspense fallback={null}>
      <ambientLight />
      <Model />
    </Suspense>
  )
}
const Page = () => {
  useStore.setState({ title: 'Sphere' })
  return (
    <>
      <Guitar r3f />
    </>
  )
}

export default Page

(I also added a light since it's not by default you would have seen a black screen otherwise)

Hello! Because you are loading using a dynamic component it's overriding the error.
I fixed your example without using dynamic component, this is the working index.js :
import { Suspense } from 'react'
import useStore from '@/helpers/store'
import Model from '@/components/canvas/Guitar'

const Guitar = () => {
  return (
    <Suspense fallback={null}>
      <ambientLight />
      <Model />
    </Suspense>
  )
}
const Page = () => {
  useStore.setState({ title: 'Sphere' })
  return (
    <>
      <Guitar r3f />
    </>
  )
}

export default Page

After that, it was possible to debug and it would just notify that you missed a Suspense since you're loading some data. So here is your example fixed with a dynamic component for better performances :

 import dynamic from 'next/dynamic'

const Model = dynamic(() => import('@/components/canvas/Guitar'), {
  ssr: false,
})

const Guitar = () => {
  return (
    <Suspense fallback={null}>
      <ambientLight />
      <Model />
    </Suspense>
  )
}
const Page = () => {
  useStore.setState({ title: 'Sphere' })
  return (
    <>
      <Guitar r3f />
    </>
  )
}

export default Page

(I also added a light since it's not by default you would have seen a black screen otherwise)

Woot 🎉 this works great now! Thank you so much for the detailed answer! 🤗

Woot 🎉 this works great now! Thank you so much for the detailed answer! 🤗


Overly strict subject requirements

This arises from #6504 which talks about our checking that a commonName or subjectAltName is set on a cert.
The check for a CN or SANs might be wrong in the general case. RFC 5280 says that:

The subject name MAY be carried in the subject field and/or the subjectAltName extension.

(Important caveat: issuers MUST have a non-empty subject field, but we don't really check that)
I believe what we want is to enforce that any kind of subject name is present in the certificate such that it can be identified. Checking specifically for commonName is thus incorrect; the user could also reasonably set any other field in the subject.
🐛 We should consider widening the check to allow for any kind of subject field to be set, not just the CN. While a cert without a CN or any subjectAltNames might not be common, it does seem to be valid.
Furthermore, our literalSubject check currently enforces that the literalSubject contains a CN.
🐛 I can't think of a reason why we should enforce that a CN is present in literalSubject, for the same reason as in the bug above: a subject can be perfectly valid without a CN. We should check that the subject is non-empty if literalsubject is provided, but I can't see a reason to enforce that any particular field is set.
/kind bug

/remove-lifecycle rotten


Improve AesGcmSiv Aead performance by optimizing classpath check

Motivation:
Aead implementation for AesGcmSiv checks if GCMParameterSpec class exists in classpath by invoking Class#forName method whenever data gets encrypted or decrypted.
A benchmark on AesGcmSiv claims that Class#forName accounts for 4-5% of the CPU cycles consumed to encrypt and decrypt data and 12% or more throughput gain can be obtained by optimizing away the Class#forName invocation.
Modification:
Cache the result of the classpath check in AesGcmSiv
BM Results:

BM Code

Run on:

MacBook Pro (16-inch, 2021) Apple M1 Pro 32GB, MacOS Monterey version 12.6.2
Used Conscrypt custom build based on 3051cda to utilize AES-NI on Apple Silicon.

Before
# JMH version: 1.35
# VM version: JDK 11.0.17, OpenJDK 64-Bit Server VM, 11.0.17+8
# VM invoker: /Users/ks-yim/.sdkman/candidates/java/11.0.17-tem/bin/java
# VM options: -Dfile.encoding=UTF-8 -Djava.io.tmpdir=/Users/ks-yim/IdeaProjects/tink-bm/build/tmp/jmh -
Duser.country=KR -Duser.language=en -Duser.variant
# Blackhole mode: full + dont-inline hint (auto-detected, use -Djmh.blackhole.autoDetect=false to disable)
# Warmup: 1 iterations, 10 s each
# Measurement: 1 iterations, 10 s each
# Timeout: 10 min per iteration
# Threads: 8 threads, will synchronize iterations
# Benchmark mode: Throughput, ops/time
# Benchmark: com.example.tink.bm.AesGcmSivBenchmark.conscrypt_encrypt

Benchmark                                             Mode  Cnt       Score   Error  Units
AesGcmSivBenchmark.bc_decrypt                        thrpt       140397.856          ops/s
AesGcmSivBenchmark.bc_encrypt                        thrpt       161178.303          ops/s
AesGcmSivBenchmark.conscrypt_decrypt                 thrpt       487751.715          ops/s
AesGcmSivBenchmark.conscrypt_encrypt                 thrpt       462492.430          ops/s

After
Benchmark                                             Mode  Cnt       Score   Error  Units
AesGcmSivBenchmark.bc_decrypt                        thrpt       168118.773          ops/s
AesGcmSivBenchmark.bc_encrypt                        thrpt       172307.366          ops/s
AesGcmSivBenchmark.conscrypt_decrypt                 thrpt       546720.978          ops/s
AesGcmSivBenchmark.conscrypt_encrypt                 thrpt       551282.859          ops/s

CPU Profiling Result:

Before

Encryption

Decryption

After

Encryption

Decryption

Hi @ks-yim, could you please open this PR in https://github.com/tink-crypto/tink-java?
We are splitting and moving our repos to https://github.com/tink-crypto.

@morambro Ah, haven't noticed the change.
I will move the PR to the new repo, thanks!

Thanks!

Closing in favor of https://github.com/tink-crypto/tink-java/pull/5


use of a variable like ${X} instead of $X

NOTE: After opening the PR, please un-check and re-check the "Allow edits from maintainers" box so that maintainers can work on your patch and speed up the review process. This is a temporary workaround to address a known issue with GitHub.>
Please delete this note before submitting the pull request.

This change is

/lgtm
We don't seem to be consistent about variable conventions, and this one appears elsewhere, so OK to change. Do not see where this script is called in the published docs, but since it lives in the repo for folks to find it's OK.


Request for Greater Zoom Level

If I look on the Rainviewer website, it seems to allow greater zoom levels than the weather-radar-card does.  Any chance to allow zooming in closer to the target area?

+1

great job, same request ❤

temporary solution:
user can change the maximum zoom value by editing the file: waether-radar-card.js.
the file is located inside the folder: www/community.
change the entry: const maxZoom = 10;\n to const maxZoom = XX;\n (I set the value to 20)
delete the waether-card.js.gz file, reload the resources and empty the cache.
this change is canceled at the next card update


Update CNI-Genie repo URL

This is a Feature Request

What would you like to be added

In content/en/docs/concepts/cluster-administration/networking.md,

update the link URL from https://github.com/Huawei-PaaS/CNI-Genie
to https://github.com/cni-genie/CNI-Genie
update the link URL from https://github.com/Huawei-PaaS/CNI-Genie/blob/master/docs/multiple-cni-plugins/README.md#what-cni-genie-feature-1-multiple-cni-plugins-enables
to https://github.com/cni-genie/CNI-Genie/blob/master/docs/multiple-cni-plugins/README.md#what-cni-genie-feature-1-multiple-cni-plugins-enables
update the link URL from https://github.com/coreos/flannel#flannel
to https://github.com/flannel-io/flannel#flannel
update the link URL from https://docs.projectcalico.org
to https://projectcalico.docs.tigera.io/about/about-calico
update the link URL from https://www.weave.works/products/weave-net/
to https://www.weave.works/oss/net/
update the link URL from https://github.com/Huawei-PaaS/CNI-Genie/blob/master/docs/multiple-ips/README.md#feature-2-extension-cni-genie-multiple-ip-addresses-per-pod
to https://github.com/cni-genie/CNI-Genie/blob/master/docs/multiple-ips/README.md#feature-2-extension-cni-genie-multiple-ip-addresses-per-pod

Why is this needed
https://github.com/Huawei-PaaS/CNI-Genie redirects to https://github.com/huawei-cloudnative/CNI-Genie ,
which is a fork of https://github.com/cni-genie/CNI-Genie and currently 62 commits behind cni-genie:master.

The latest commit of https://github.com/cni-genie/CNI-Genie : on 29 Oct 2021
The latest commit of https://github.com/huawei-cloudnative/CNI-Genie : on 12 Feb 2019

and for other CNIs, it would be better to use the final URLs than using redirecting URLs.
Comments

Page to update: https://kubernetes.io/docs/concepts/cluster-administration/networking/#cni-genie-from-huawei

/assign

/triage accepted
/kind cleanup

/retitle Fix incorrect / outdated URIs in list of network plugins
/language en
/priority backlog


improve: allow customizing clusterDomain

Hi deployKF team,
I think you missed this config. In my case, I have many clusters with different cluster domains and I want to config in my custom-values.yaml file. Perhaps someone else need this in future, so I create this pull request.
I hope this is useful!

@cyanwind23 Thanks for your suggestion.
First, I would like to know the situation which is causing your cluster domain to not be cluster.local, given this is such a widespread standard?
Also, there are a few big problems with the PR:

You can't update the files under the upstream/ folders under kubeflow-tools, you have to use kustomize patches.
I think there are at least a few hard-coded cases that assume cluster.local with no way to change it (e.g. in controllers), but I could be wrong. (A big one was the profile-controller & kfam, but I think this was resolved in Kubeflow 1.8, released yesterday)

If you are very interested in getting custom cluster domains to work, I would greatly appreciate your help in testing and seeing what breaks when you have a non-default domain, but some of it will be slightly non-obvious.

Hi @thesuperzapper,

First, I would like to know the situation which is causing your cluster domain to not be cluster.local, given this is such a widespread standard?

When I installed K8s cluster, I changed my cluster domain to prepare for cross-cluster communication if necessary. I know changing the default value cluster.local is not a common case. Also, It might not be a good thing to do until now, as mentioned in this discussion. But I think my situation is not the only one. I saw some people did. I don't know what their purpose is, even just simply to identify the cluster such as cluster.dev, cluster.stag or cluster.prod, etc., maybe.

You can't update the files under the upstream/ folders under kubeflow-tools, you have to use kustomize patches.

Sorry for my mistake, I will check it.

I think there are at least a few hard-coded cases that assume cluster.local with no way to change it (e.g. in controllers), but I could be wrong.

Maybe you're right. I also think so although I don't know if there is a 3rd party app hard-coded value cluster.local in the code. However, in my opinion, K8s allow us to change cluster domain value therefore the K8s components themselves must support that (not hard-coded), and 3rd party apps also should do so.
Thank you for your attention to this matter.

I have fixed my code to not change upstream manifests and tested on my cluster with customized clusterDomain and it works.
I noticed this line pipelines-profile-controller/sync.py#L241, cluster.local is hard-coded but it can be overwritten by istio-authorization-config.yaml#L40


segvault when ratbagd crashes with just the Logitech Receiver attached with the mouse being disabled

Information

ratbagd version (ratbagd --version): 0.13-1+git1582-dba202a~ubuntu18.04.1
Distribution: Linux Mint 19.1
Kernel version (ex. uname -srmo): Linux 5.6.1-050601-generic x86_64 GNU/Linux

Describe the bug
When the Logitech receiver is attached, but the mouse is shut off, ratbagd crashes with segvault
To Reproduce

Ensure Logitech Receiver attached
Ensure mouse off
Run ratbagd:sudo ratbagd
ratbag error: Failed to get HID++1.0 device for Logitech USB Receiver
ratbag error: libratbag bug: Logitech USB Receiver: invalid number of profiles (0)
[1]    32158 segmentation fault  sudo ratbagd

Logs
$ ratbagd --verbose=raw
Initializing libratbag
ratbag debug: New device: Logitech USB Receiver
ratbag debug: Using data directory '/usr/share/libratbag'
ratbag debug: device assigned driver hidpp10
ratbag debug: hidraw info: bus 0x03 vendor 0x46d product 0xffffc531
ratbag debug: Logitech USB Receiver is device '/dev/hidraw0'.
ratbag debug: Parsing HID report descriptor
ratbag debug: Parsing HID report descriptor
ratbag debug: hidraw info: bus 0x03 vendor 0x46d product 0x102a
ratbag debug: hidraw info: bus 0x03 vendor 0x46d product 0xffffc531
ratbag debug: Logitech USB Receiver is device '/dev/hidraw1'.
ratbag debug: Parsing HID report descriptor
ratbag debug: Parsing HID report descriptor
ratbag debug: - HID report ID 01
ratbag debug: - HID report ID 03
ratbag debug: - HID report ID 04
ratbag debug: - HID report ID 10
ratbag debug: - HID report ID 11
ratbag raw: Fetching individual features (0x1)
ratbag raw: hidpp10 tx:  10 | 01 | 81 | 01 | 00 00 00
ratbag raw: hidpp write:  10 01 81 01 00 00 00
ratbag raw: hidpp read:   10 01 8f 81 01 09 00
ratbag raw:     HID++ error from the device (1): ERR_RESOURCE_ERROR (09)
ratbag raw: hidpp10 rx:  10 | 01 | 8f | 81 | 01 09 00
ratbag raw: Fetching HID++ notifications (00)
ratbag raw: hidpp10 tx:  10 | 01 | 81 | 00 | 00 00 00
ratbag raw: hidpp write:  10 01 81 00 00 00 00
ratbag raw: hidpp read:   10 01 8f 81 00 09 00
ratbag raw:     HID++ error from the device (1): ERR_RESOURCE_ERROR (09)
ratbag raw: hidpp10 rx:  10 | 01 | 8f | 81 | 00 09 00
ratbag raw: Fetching current resolution (0x63)
ratbag raw: hidpp10 tx:  10 | 01 | 83 | 63 | 00 00 00
ratbag raw: hidpp write:  10 01 83 63 00 00 00
ratbag raw: hidpp read:   10 01 8f 83 63 09 00
ratbag raw:     HID++ error from the device (1): ERR_RESOURCE_ERROR (09)
ratbag raw: hidpp10 rx:  10 | 01 | 8f | 83 | 63 09 00
ratbag raw: Fetching LED status (0x51)
ratbag raw: hidpp10 tx:  10 | 01 | 81 | 51 | 00 00 00
ratbag raw: hidpp write:  10 01 81 51 00 00 00
ratbag raw: hidpp read:   10 01 8f 81 51 09 00
ratbag raw:     HID++ error from the device (1): ERR_RESOURCE_ERROR (09)
ratbag raw: hidpp10 rx:  10 | 01 | 8f | 81 | 51 09 00
ratbag raw: Fetching USB refresh rate (0x64)
ratbag raw: hidpp10 tx:  10 | 01 | 81 | 64 | 00 00 00
ratbag raw: hidpp write:  10 01 81 64 00 00 00
ratbag raw: hidpp read:   10 01 8f 81 64 09 00
ratbag raw:     HID++ error from the device (1): ERR_RESOURCE_ERROR (09)
ratbag raw: hidpp10 rx:  10 | 01 | 8f | 81 | 64 09 00
ratbag raw: Fetching optical sensor settings (0x61)
ratbag raw: hidpp10 tx:  10 | 01 | 81 | 61 | 00 00 00
ratbag raw: hidpp write:  10 01 81 61 00 00 00
ratbag raw: hidpp read:   10 01 8f 81 61 09 00
ratbag raw:     HID++ error from the device (1): ERR_RESOURCE_ERROR (09)
ratbag raw: hidpp10 rx:  10 | 01 | 8f | 81 | 61 09 00
ratbag raw: Fetching current profile (0xf)
ratbag raw: hidpp10 tx:  10 | 01 | 81 | 0f | 00 00 00
ratbag raw: hidpp write:  10 01 81 0f 00 00 00
ratbag raw: hidpp read:   10 01 8f 81 0f 09 00
ratbag raw:     HID++ error from the device (1): ERR_RESOURCE_ERROR (09)
ratbag raw: hidpp10 rx:  10 | 01 | 8f | 81 | 0f 09 00
ratbag error: Failed to get HID++1.0 device for Logitech USB Receiver
ratbag error: libratbag bug: Logitech USB Receiver: invalid number of profiles (0)
[1]    13788 segmentation fault  sudo ratbagd --verbose=raw

Duplicate: #889.
This has actually been fixed! :)


can't preload self-referencing association

What version of Go are you using (go version)?
go version go1.11.1 darwin/amd64
Which database and its version are you using?
PostgreSQL 10.4 on x86_64-apple-darwin17.5.0, compiled by Apple LLVM version 9.1.0 (clang-9<IP_ADDRESS>), 64-bit
code

type Category struct {
	ID uint `gorm:"primary_key"`
	Name string `gorm:"size:50"`
	Parent *Category `gorm:"ForeignKey:ParentID"`
	ParentID uint
}

var testCategory []Category
db.Debug().Preload("Parent").Where("id = ?", 12).Find(&testCategory)
fmt.Println(testCategory)

I get result print as {12 高效办公 <nil> 16}。。I can't get parent information，but it exist，ParentID is 16.  the generated sql is
SELECT * FROM "si_category"  WHERE (id = 12)

SELECT * FROM "si_category"  WHERE ("parent_id" IN (12))

for second sql what i expect is SELECT * FROM "si_category"  WHERE ("parent_id" IN (16))
is there something wrong?

You want to use
db.Debug().Preload("Parent", "`parent_id` = ?", 12).Find(&testCategory)

I believe your field tag may be incorrect
Parent *Category `gorm:"foreignkey:ID;association_foreignkey:ParentID"`


Add UI equation to calculate tower height from h_tower, fix #471

Adjust group box widths to match O&M Cost form.
Fix alignment issues

Should the target branch on this be patch?

Shoot! I meant to branch this off of patch -- thanks for catching that, @brtietz.

Replace with branch off patch PR #475


Feature Request: pulseaudio-dlna

Issue description
Please provide nix package for pulseaudio-dlna.
https://github.com/masmu/pulseaudio-dlna
This would allow chromecast support on nixos.
Steps to reproduce
Technical details

system: "x86_64-linux"
host os: Linux 4.9.78, NixOS, 17.09.2905.c1d9aff56e0 (Hummingbird)
multi-user?: yes
sandbox: no
version: nix-env (Nix) 1.11.16
channels(root): "nixos-17.09.2905.c1d9aff56e0"
channels(poelzi): "nixos-17.09-17.09.2964.3e349a2b98b"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs

This looks cool, ill try to build it

built it here https://github.com/NixOS/nixpkgs/pull/35245
I need to test it on my own box though

@poelzi works on my box. this is a really cool piece of software thanks for the suggestion

thank you very much for packaging it. very speedy :) :+1:


General housekeeping PR

allow limiting the number of threads in the InlineJobExecutor
apply a limit of 2 in the lt_ctx fixture. Let's see if this fixes long test runtimes on Windows
remove some deprecated np.{float,bool} usage

Contributor Checklist:

[ ] I have added or updated my entry in the creators.json file
[ ] I have added a changelog entry for my contribution
[ ] I have added/updated documentation for all user-facing changes
[ ] I have added/updated test cases
[ ] I have included the rebuilt production build of the client (only if changes were made to the GUI)

Reviewer Checklist:

[x] /azp run libertem.libertem-data passed

/azp run libertem.libertem-data

/azp run libertem.libertem-data

/azp run libertem.libertem-data

Thx, merging!


update dependencies, development conveniences

As I pointed out here, I will only close this PR and push my local changes to master. Thanks!


Positioning of Circular Progress Bar in the Sign Up Screen

Describe the feature you'd like

On clicking the sign-up button the circular progress bar appears on the top left corner but for better UX it should replace the text inside Sign up button.
Screenshots

Additional context

Your views on this @iamareebjamal.
Would you like to work on the issue?

Yes

Duplicate of #647


ui: API to change the icon

SetIcon(icons []image.Image)

BTW, I found GLFW's API to set icon is empty on macOS

Possible blocker: https://github.com/go-gl/glfw/issues/209

Sigh... Edge doesn't work with data-url for a favicon https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/7480412/

On second thought, as Ebiten games should work in an iframe, changing favicon doesn't make sense.


3MF export leaves open edges

After exporting a model as 3MF like so,
rpi_mount.plate.export_3mf(
    "4Max_Pro_RPi_Touchscreen_Mount_Plate.3mf",
    tolerance=1e-3,
    angular_tolerance=0.1,
    unit=Unit.MILLIMETER,
)

I opened this file in PrusaSlicer and noticed it had a warning about the model having 9720 open edges, and 50 shells when it should have 1:

When exporting the exact same model as STL, this warning is not present, and I see 1 shell as I would expect:

The 3MF exporter has be totally rewritten to use the 3MF Consortium's Lib3MF which doesn't allow non manifold meshes. See: https://build123d.readthedocs.io/en/latest/import_export.html#d-mesh-export


natgw resource type now broken

this morning i ran -t natgw to capture some resources needed for some other resource type but was not captured.
i just ran it again and now a routing table says it cannot find a specific gateway id.
i know i downloaded it coz i still have the file in a backup directory for one of the other resources.
after copying the files across, the plan was generated

-t natgw - there is no such type, so if it worked at all it's by accident.
I've added it for you now and looked at the possible dependancies.

still getting the error
Error: Reference to undeclared resource

  on aws_route_table__rtb-RTB.tf line 12, in resource "aws_route_table" "rtb-RTB":
  12:       gateway_id                 = aws_internet_gateway.igw-IGW.id

A managed resource "aws_internet_gateway" "igw-IGW" has not been declared in the root module.

also, the previous run pull

eip

my bad - did u know -t igw works too (and plans without errors!)
anyways -t ngw has that error above and fails to generate a plan

committed change to catch those stray internet gateways at validation stage


fix: bump elixirls

Related to https://github.com/elixir-tools/elixir-tools.nvim/issues/196

I think there is a test that needs updated

I think there is a test that needs updated

The truth is that I am not very sure how it can be related to my change. The error seems more related to something about "next-ls" doesn't it? https://github.com/avillen/elixir-tools.nvim/blob/main/tests/utils_spec.lua#L45-L50


Field label component

Feature/component description
Create a separate label component for your input fields: <vscode-label>Label<vscode-label>.
Use case
Fields that haven't been supported yet by the toolkit, can then be developed ourselves and still use the same label styles without the need to duplicate the styles.

Great idea! We have this in the soon-to-be-released Figma toolkit. Agreed that it should be added here as well.

Hi @estruyf,
Back with a final update: I'm very sorry to say that the toolkit is being deprecated and all active development will be coming to a close.
There was an announcement last week where you can learn more details and leave any questions or comments you may have.
Beyond that, thank you so much filing this issue and apologies for never getting around to addressing it. It means a lot that you contributed to the improvement of this project. I wish you all the best in your future VS Code extension endeavors!


Fix annotation conflicts

Please merge https://github.com/ibissource/iaf/pull/4165 before this PR.


process: sync commits from Aspect-internal silo

Commit range https://github.com/aspect-build/silo/compare/580f85e4ccbdb3124c75d39c220c0e686667acb2..e27563aedb24f14a85ed2072f23c0f995877997d

I think I'll clobber this with #712


Demo Website No Longer Functional

Hello,
Apologies if this is not the correct means of reporting this issue.
The demo site at this time appears to be down, I’m receiving a HTTP 404 error:
https://passfault.appspot.com/password_strength.html
The following website also shows the site is down:
http://www.isitdownrightnow.com/passfault.appspot.com.html
I realize that you are probably aware of this and apologize if I’m reporting the obvious to you. Thanks for your time.

Thanks @hc970 I was not aware of this.  I'll look into it right away!

Oh.  I know what it is.  The app-engine setup I've been using is an older deprecated version and they've finally pulled the plug on it.  I need to resetup the page.  Thanks for the heads up!

I've got passfault deployed to an upgraded location: https://passfault-hrd.appspot.com and got DNS for http://passfault.com  pointing to the right place.  I need to wait a few days to change the the https://passfault.appspot... domain

Hello Cam,
Many thanks for resolving this issue so quickly. I use Passfault a lot and it's great to have it back again. I donated a small amount to OWASP last week and hope to give more sometime in the near future.
If you wish me to close this issue as resolved, I will do so. Keep up the excellent work. Thanks again.

Great to hear! I've put a lot into passfault and it's great to hear that it's being useful.
I'd like to keep this open until I fix the appspot domain, since many people have linked directly to it.


Optional regex for sonarr removed with sync

Is there a way to sync the optional preferred preferences for sonarr? I have now manually added the regex for Optional Ignore DoVi for WEBDL, but this gets removed from sonarr when I run the sync.
Maybe this can be added like it is done for radarr?

At the moment, there is no way to sync optional preferred. It's on my list but it's complicated to implement. Right now the best workaround is to create a new release profile using the same tag you use for TV. Add your optional preferred to that list, and it won't get overwritten by Trash Updater. Since it shares the same tag, it will supplement your series searches all the same.
Hopefully this works for you in the meantime.

Thanks, that workaround is easily usable and allows me to continue to use your tool!


Replace Deprecated Exceptions

Summary
Replaced dbt-core's deprecated exceptions with new exceptions
Description
Replaced the following old exceptions with new exceptions

Old
New

RuntimeException
DbtRuntimeError

DatabaseException
DbtDatabaseError

raise_compiler_error()
CompilationError

Test Results
Successfully ran all functional tests
Changelog

[x] Added a summary of what this PR accomplishes to CHANGELOG.md

Related Issue
#117

Ran the unit tests and it seems like the FailedToConnectException was removed entirely. Updated to new exception to get all unit tests passing.


Add Containers topology

In the default view, nodes from the same container instance (docker ID) are merged
In the grouped view, nodes from the same container image (docker image ID) are merged

Closes #91.
/cc @tomwilkie

Coverage increased (+0.37%) to 62.66% when pulling b493383ece5fb79670639682a14cb61f3bd23ea9 on add-containers-topology into 957d5253a126a60e70a13ae0d7c0ed5ab2323da4 on master.

Can you rebase and squash this into one please?  We don't need to introduce extra merge commits, or commits like "Works on my machine".  Also need to resolve why my two csets are appearing on your branch; probably go away if you rebase.

Ah I see you already got rid of the merge.

Coverage increased (+0.44%) to 62.72% when pulling 00a623a1162d3b785c86757d531eaf2fe43f3112 on add-containers-topology into 957d5253a126a60e70a13ae0d7c0ed5ab2323da4 on master.

LGTM


Closes #58: libmexclass build sporadically fails on macOS and win64 GitHub Actions Runner

Bug Description
libmexclass fails to build when find_package(MATLAB REQUIRED) is unable to detect the Matlab Version Number and instead sets Matlab_VERSION_STRING to "unknown". Because FindMatlab failed to detect the version, any variables that are dependent on on the version number are also not set, such as Matlab_DATAARRAY_LIBRARY and Matlab_ENGINE_LIBRARY. libmexclass links against the Matlab_DATAARRAY_LIBRARY, so if this variable is not set, the libmexclass build fails at link-time.
Fix Description
Added a new macro called MatlabConfigure that does the following:

Call find_package(MATLAB REQUIRED)
If Matlab_VERSION_STRING does not equal  "unknown", return.
If Matlab_VERSION_STRING equals  "unknown", derive the version number from VersionInfo.xml located within Matlab_ROOT_DIR.
Set the variables necessary (Matlab_DATAARRAY_LIBRARY, Matlab_ENGINE_LIBRARY, etc.) for building libmexclass.

Testing
To simulate this build failure on all platforms, I temporarily added a macro called MatlabUnsetVariables which set Matlab_VERSION_STRING to "unknown" and unset the necessary Matlab_* variables. This macro was invoked after find_package(MATLAB REQUIRED) to ensure the "unknown" Matlab_VERSION_STRING codepath was run.
All three jobs passed.

+1


Consider using the register crate

Currently, this crate has lots of pairs of getters and setters. The register crate is for defining MMIO registers, and this might help to reduce the number of lines.

It seems that the crate cannot specify in a way like: "Offset x from the start of the nth element of the array.".


Added RFC cron job

Added RFC Cron job which will iterate over existing referendas and post a link to them in the open Pull Requests.
This resolves polkadot-fellows/RFCs#57
Every time it iterates, it looks for referendas that have been submitted after the time of the last run, to stop itself from posting duplicated comments.
It is set to run every day at 12, and it can also be triggered manually.

The messages that it will post will be in the following style:

Voting for this referenda is ongoing.
Vote for it here


Orca invalid characters issue

Hey Picovoice team. I’m a big fan of your mission and I’m enjoying playing with the python sdk for the orca beta, but I’ve been encountering issues with certain common characters resulting in utf-8 decoding exceptions being thrown.
there may be more, but so far I’ve noticed that:
Any integers, even when wrapped in strings, - “ I need 10 bananas”
Colons - ‘:’
Semi-colons - ‘;’
Dashes - ‘-‘
Underscores - ‘_’
Newlines - ‘\n’ or ‘\n’
And - I believe, but need to double check, quotation marks, even when wrapped in single quotes - I.e 'Susan said "hi" to Bob'
(Ignore the fancy quotes, I’m typing this on a phone, they don’t exist in my actual inputs)
Will all cause orca.synthesize_to_file() to throw invalid Unicode decoding errors. This happens in python programs, your cli, a cli I wrote, and in the python interpreter itself.
Is this a known limitation? Am I doing something wrong? Looking forward to hearing from you, I’d love to be able to use this as a solution for my project.

We released a patch for the python version that includes a few fixes (fix error reporting, support for hyphens ("-"), better pronunciations). You might want to give it a try.
We are always happy to see people building on top of our products. Feel free to create a fork of the orca repo and link it here.

+1 on this.
I'm building an end-to-end voice assistant that dispatches voice commands gathered via Cheetah to ChatGPT (or compatible models) and uses Orca to render the response as audio.
Special punctuation characters aren't a big deal. Since I already have a text transcript, I can easily replace hyphens/underscores with spaces, colons/semicolons/newlines with periods etc.
But numbers are quite a problem - a response is quite likely to contain digits/numbers in some form.
A workaround for now may be to just pre-process the string via a regex that replaces all digits with their num2words representation before feeding it to Orca. But this would only work for Python - and it assumes that the client knows the target language.
Are there any plans to also handle numbers? If so, then I can wait for a new release. Otherwise I may go for the num2words+regex workaround in my implementation. I could also prepare a PR that pre-processes numbers in the text before Orca processes it, but my num2words workaround would only work for the Python bindings then.

Hi @blacklight ,
thanks for the feedback. We are working on a release that will support handling more types of inputs, including numbers. Keep an eye open for our next release.


Scrollbar styling (globally or util class)

We currently have (.scrollbar-none) to completely hide the scrollbar, but it would be nice to also have some better styling (possibly globally, but  at least opt-in).
Styling the scrollbar is setting:

::-webkit-scrollbar
::-webkit-scrollbar-thumb
::-webkit-scrollbar-track

on Chrome/Safari, and

scrollbar-color
scrollbar-width

on Firefox.
See docs for reference and additional properties

https://developer.mozilla.org/en-US/docs/Web/CSS/CSS_scrollbars_styling
https://css-tricks.com/almanac/properties/s/scrollbar/

I have an old REPL that can also be used as reference.
.scrollContainer {
	scrollbar-color: var(--scrollcolor) var(--scrollbackground);
}
.scrollContainer::-webkit-scrollbar {
	height: 0.5rem;
	width: 0.5rem;
	border-radius: 1rem;
	background: var(--scrollbackground);
}

.scrollContainer::-webkit-scrollbar-thumb {
	border-radius: 1rem;
	background: var(--scrollcolor);
}

.scrollContainer::-webkit-scrollbar-track {
	border-radius: 1rem;
	background: var(--scrollbackground);
}

Appears Skeleton sets this globally with CSS variables (and I've been working on #66 and determining which variables to expose), so this will be helpful for reference.

Would like to see if others do this globally (it didn't appear shadcn/ui does, but that's not surprising based on how they implement things).


COMP: Make ITK implementation make_unique_for_overwrite and bit_cast unconditional

Addresses MacOS/XCode C++20 compilation errors saying:

itkMakeUniqueForOverwrite.h:76:14: error: no member named 'make_unique_for_overwrite' in namespace 'std'

Reported by Mario Emmenlauer (@emmenlau), issue https://github.com/InsightSoftwareConsortium/ITK/issues/3855 "Apple Clang 14.0.0 (current latest) does not support std::make_unique_for_overwrite".
`

@N-Dekker could this patch please be based on the release branch?

could this patch please be based on the release branch?

@thewtex I just enabled "Allow edits and access to secrets by maintainers", does that help you?

I just rebased it to release branch.

I just rebased it to release branch.

Thanks @dzenanz Still wondering what it means. Because the PR should still be merged to the master, right?
Why not first merge it to the master, and then cherry-pick from master to release? Just curious!

I assume the plan is to merge this patch to release too, either via a PR or directly by Matt.

/azp run ITK.macOS


[GHSA-mq26-g339-26xf] When installing a package from a Mercurial VCS URL  (ie ...

Updates

Affected products
Severity
Summary

Comments
Adding patched version so dependabot can start patching.

Many thanks!


Question about SECP256K1 curve in go

Hello, mike, I found it shows "package NIST256" in ROM_SECP256K1_64.go line 22, I wonder if it should be "package SECP256K1"?

Correct. Thanks. Sorry for delay

Thanks.


Test LRO with sleep time

@mkoshurnikov don't make changes to the PR while I run the tests several times.

I'm sorry. Previous tests were successful.

Closing as this PR was just for testing purposes.


[Bug] App oppened in fullscreen mode

Platform

Device name: Motorola Moto G7 Power (ocean)
Device type (phone/TV): phone
Android version: 14
App version: 0.2
Script version: 3.2.4

Describe the bug
After update, the app it goes beyond the camera cut-out and is completely full screen. Please make the app behave like this only in landscape mode.
TOO YOU CAN'T SEE THE STATUS BAR WHEN RUNNING.
Screenshots/Videos

Partially corrected

Fixed in 0.3


Give Vouchers and GUI Doesn't work

Config: https://pastebin.com/ZvnnU5Zx
Server: Spigot 1.8
Plugin Version: 1.8.1

Stale.


Checks for "provided" before assigning $apolloProvider to it

I was having seeing the same error discussed in Issue 532. This adds a check for provided.
I ran yarn build before pushing. This was my first time contributing to an NPM package so, if running build was wrong let me know and I'll re-submit without building.

It's already fixed in #533.

if running build was wrong let me know and I'll re-submit without building.

Yes it's better without building, otherwise it almost always creates conflicts.


Whenn I Call sui move test, occurs error

The test file
#[test_only]
module hello_move::hello_move_tests {
    // uncomment this line to import the module
    // use hello_move::hello_move;

    const ENotImplemented: u64 = 0;

    #[test]
    fun test_hello_move() {
        // pass

    }

    #[test, expected_failure(abort_code = hello_move::hello_move_tests::ENotImplemented)]
    fun test_hello_move_fail() {
        abort ENotImplemented
    }
}

Error
When I call sui move test, occurs error

what should I do, please help

Move.toml
[package]
name = "hello_move"
edition = "2024.beta" # edition = "legacy" to use legacy (pre-2024) Move
# license = ""           # e.g., "MIT", "GPL", "Apache 2.0"
# authors = ["..."]      # e.g., ["Joe Smith (joesmith@noemail.com)", "John Snow (johnsnow@noemail.com)"]

[dependencies]
Sui = { git = "https://github.com/MystenLabs/sui.git", subdir = "crates/sui-framework/packages/sui-framework", rev = "framework/testnet" }

# For remote import, use the `{ git = "...", subdir = "...", rev = "..." }`.

# MyRemotePackage = { git = "https://some.remote/host.git", subdir = "remote/path", rev = "main" }

# For local dependencies use `local = path`. Path is relative to the package root
# Local = { local = "../path/to" }

# To resolve a version conflict and force a specific version for dependency
# override use `override = true`
# Override = { local = "../conflicting/version", override = true }

[addresses]
hello_move = "0x0"

# Named addresses will be accessible in Move as `@name`. They're also exported:
# for example, `std = "0x1"` is exported by the Standard Library.
# alice = "0xA11CE"

[dev-dependencies]
# The dev-dependencies section allows overriding dependencies for `--test` and
# `--dev` modes. You can introduce test-only dependencies here.
# Local = { local = "../path/to/dev-build" }

[dev-addresses]
# The dev-addresses section allows overwriting named addresses for the `--test`
# and `--dev` modes.
# alice = "0xB0B"

This looks like you might be running an old version of the Sui CLI. If you install the newest version (https://docs.sui.io/guides/developer/getting-started/sui-install#install-homebrew) that should fix your issues.
If after updating your CLI you're still having this issue, can you tell me what version of the CLI you're running? You can get this information with sui -V.

@tzakian Thanks for your help.But when I tring to upgrade, it shows the SUI is up-to-date

Use brew install sui to upgrade. Sui CLI is now at v1.27.

@stefan-mysten  Thanks a lot. It's working. And it's also normal to run sui move test.
Thanks again for your help. @tzakian @stefan-mysten 👍

Happy to hear it's working now!


Frontier [testing]: build conda env

Build the microconda env on Frontier login nodes

Check pt-caney dev requirements
Need to use AMD ROCm bindings w/ pytorch

Current conda env is also available via https://github.com/nasa-nccs-hpda/ilab-benchmarking-toolkit/tree/develop

Check:
Torch version: 2.2.1+rocm5.7
CUDA available: True
CUDA version: None
HIP  version: 5.7.31921-d1770ee1b
current device: 0
device arch name: AMD Instinct MI250X
BF16 support: True
NCCL available: True
NCCL version: (2, 17, 1)
ROCR_VISIIBLE_DEVICES: 0,1,2,3,4,5,6,7
HIP_VISIBLE_DEVICES: None
CUDA_VISIBLE_DEVICES: None


[Mingw] Bug fix in Filesystem

Fix recent changes to Filesystem::open where internal buffers of istream/ostream would never get deleted on MINGW.

Note that while this correctly fixes memory leaks, there are still issues with UTF-8 files. I'm trying to find the root cause of that.

Wow, this is getting complicated.
Travis tests are failing on this PR: https://travis-ci.org/OpenImageIO/oiio/builds/111006447
How is this issue addressed with other software?
Is there anything in C++11 (or 17?) or Boost that does a proper combination of UTF-8 and streams and resource management in a portable way?

I should probably have looked into this more deeply when the first code review came around.
OK, back to first principles.
The problem is that (on Windows? or everywhere?) the std ifstream/ofstream constructors don't take UTF-8 filenames. So we created a Filesystem::open() wrapper that on Unix/Linux will just open the stream as usual but on Windows converts the filename from UTF-8 to UTF-16 and then uses a version of stream.open() that accepts wstring.
Right so far?
But then it turns out that if you're on Windows but using MinGW rather than MSVS, that doesn't exist? So another strategy was necessary?
But now it's morphed into new classes and resource management and totally different calling sequences on the other side (the clients of Filesystem). This is making me suspect that maybe this is the wrong approach.
Briefly looking at http://www.cplusplus.com/reference/fstream/ifstream/, I'm noticing that ifstream is actually typedef to a template instantiation, std::basic_ifstream.
What if, instead of all this, we used a simpler approach:

In filesystem.h, define OIIO::ifstream, which for most platforms simply typedefs to std::ifstream, but on MinGW is std::basic_ifstream<wchar_t>.

Thus, the new versions of open() can be reverted, and just the old version can have minor modifications with #if for the MinGW case. On MinGW, will a std::basic_ifstream<wchar_t> properly open a UTF-16 filename?

The PNM and PSD cases directly manage their own streams, so for those the conversion is trivial -- just switch to OIIO::ifstream.

OpenEXR is the only hard case, because we have to pass a stream to the IlmImf API. But wait, we have our own OpenEXRInputStream wrapper. So maybe it's easy there, too.

(Repeat the same logic for output streams.)
I'm just sketching this out, not sure if it will work (and have no way to test myself on MinGW). But it sounds like if we can make something like this work, it might be less intrusive overall.
Am I even remotely understanding this all?

Your understanding of the problem is right, the matter is only related to compilation under Windows with MingW/GCC.
According to http://stackoverflow.com/questions/6524821/opening-stream-via-function the only way to fix UTF8 filenames on Windows when compiling with GCC is to use __gnu_cxx::stdio_filebuf.
The problem is, according to this link the destructor of istream and ostream does not destroy nor performs any operations on the associated stream buffer object.
So to be able to use this solution we have to keep a istream around and a stdio_filebuf around.
The usage of the Filesystem::open should stay the same across different platforms, hence passing another argument is not really wise.
Even if we were to do the following:
#ifdef FILESYSTEM_USE_STDIO_FILEBUF
typedef std::istream if stream
#else
typedef std::if stream if stream
#endif

```;;
It would not work because we would still have to keep the stdio_filebuf around, hence the Wrapper.

Another solution come to my mind:

We could do keep the typedef on anything else than MingW, but on MingW, we inherit std::istream so that the *__gnu_cxx::stdio_filebuf* is actually a member of the class, hence it would be deleted when the istream would be deleted. We would still need to add a *close()* function because it is used at some places across OpenImageIO and does not exist in the API of std::istream.

Do you prefer this solution ? (less intrusive) but more hackish to understand

-------

Anyway, the reason why the problem is not *fully* solved yet, is because the *_wsopen_s* function used in *open_fstream_impl* is failing, returning errno = ERRNOENT (meaning *No such file or directory*) whereas  the file actually exists. It only fails when using filepaths with weird characters, such as "€ é è", eg.:
`C:\Users\MysuperUsér\Vidéos\Mes vidéos de vacances\t€t€.mov"`

I found the bug: it was in the StrUtil::utf8_to_utf16 function. An extra 0 character was appended to the wide string, making the _wsopen_s function actually fail.
I have found this function extremely useful to check that a string is correctly UTF-8 encoded:
// We check that strings contained in StringParam are UTF-8 encoded
// See http://openfx.sourceforge.net/Documentation/1.4/ofxProgrammingReference.html#kOfxParamTypeString
//
// From http://stackoverflow.com/questions/1031645/how-to-detect-utf-8-in-plain-c
static bool is_utf8(const char * string)
{
  if(!string)
    return 0;

  const unsigned char * bytes = (const unsigned char *)string;
  while(*bytes)
  {
    if( (// ASCII
         // use bytes[0] <= 0x7F to allow ASCII control characters
         bytes[0] == 0x09 ||
         bytes[0] == 0x0A ||
         bytes[0] == 0x0D ||
         (0x20 <= bytes[0] && bytes[0] <= 0x7E)
         )
       ) {
      bytes += 1;
      continue;
    }

    if( (// non-overlong 2-byte
         (0xC2 <= bytes[0] && bytes[0] <= 0xDF) &&
         (0x80 <= bytes[1] && bytes[1] <= 0xBF)
         )
       ) {
      bytes += 2;
      continue;
    }

    if( (// excluding overlongs
         bytes[0] == 0xE0 &&
         (0xA0 <= bytes[1] && bytes[1] <= 0xBF) &&
         (0x80 <= bytes[2] && bytes[2] <= 0xBF)
         ) ||
       (// straight 3-byte
        ((0xE1 <= bytes[0] && bytes[0] <= 0xEC) ||
         bytes[0] == 0xEE ||
         bytes[0] == 0xEF) &&
        (0x80 <= bytes[1] && bytes[1] <= 0xBF) &&
        (0x80 <= bytes[2] && bytes[2] <= 0xBF)
        ) ||
       (// excluding surrogates
        bytes[0] == 0xED &&
        (0x80 <= bytes[1] && bytes[1] <= 0x9F) &&
        (0x80 <= bytes[2] && bytes[2] <= 0xBF)
        )
       ) {
      bytes += 3;
      continue;
    }

    if( (// planes 1-3
         bytes[0] == 0xF0 &&
         (0x90 <= bytes[1] && bytes[1] <= 0xBF) &&
         (0x80 <= bytes[2] && bytes[2] <= 0xBF) &&
         (0x80 <= bytes[3] && bytes[3] <= 0xBF)
         ) ||
       (// planes 4-15
        (0xF1 <= bytes[0] && bytes[0] <= 0xF3) &&
        (0x80 <= bytes[1] && bytes[1] <= 0xBF) &&
        (0x80 <= bytes[2] && bytes[2] <= 0xBF) &&
        (0x80 <= bytes[3] && bytes[3] <= 0xBF)
        ) ||
       (// plane 16
        bytes[0] == 0xF4 &&
        (0x80 <= bytes[1] && bytes[1] <= 0x8F) &&
        (0x80 <= bytes[2] && bytes[2] <= 0xBF) &&
        (0x80 <= bytes[3] && bytes[3] <= 0xBF)
        )
       ) {
      bytes += 4;
      continue;
    }

    return 0;
  }

  return 1;
}

I recommend using it in debug mode whenever calling StrUtil::utf8_to_utf16

I made a attempt in the branch inherit-fstream in my fork  to just typedef and inherit basic_istream/basic_ostream on Mingw. This had to introduce a new header to separate the MingW stuff (fstream_mingw.h)
I'm going to run a few tests and modifications on it before it is ready

I took a quick look at your fork, and I liked what I saw. I think this is a much more straightforward approach, revert back almost completely to what we had before, with some extra stuff just for that one platform, emulating all the old simple call signatures.
I urge you to close/abandon this PR and instead start a new one with the inherit-fstream work (when you feel it's ready).

Will do. Note that I stripped a few functions off the API of ifstream/ofstream because some are C++11.
Anyway everything that is used withing OIIO is implemented in those classes.

This lack of UTF8 handling for MingW also goes beyond OpenImageIO and is present in OpenEXR too, in the function Imf::isOpenExrFile because they use standard ifstream.
I'll provide a patch to the OpenEXR guys.
I've run into issues also with every boost::filesystem functions if we give them a std::string instead of a std::wstring. I'm trying to figure out where it is failing in boost right now.

Aight: taken from boost::filesystem::path.hpp:

    //  For Windows, wchar_t strings do not undergo conversion. char strings
    //  are converted using the "ANSI" or "OEM" code pages, as determined by
    //  the AreFileApisANSI() function, or, if a conversion argument is given,
    //  using a conversion object modeled on std::wstring_convert.

Meaning there is no actual conversion from UTF-8 to UTF-16 done when they convert char* to wchar_t* internally.
They do not use MultiByteToWideChar (CP_UTF8,... to convert strings as done in StrUtils::utf8_to_utf16.
So every call to boost::filesystem on Windows has to be prepended with a call to StrUtils::utf8_to_utf16
In the end it's not really a bug in libstdc++ it's just that it's an undefined standard that MSVC imposed, so no patching of libstdc++ would be right. Only at the library level - say OpenImageIO - we can know whether we want to convert from UTF8 to UTF16 or any other encoding.
I'll fix this in filesystem.cpp and also make the same fixes to OpenEXR...

Quick update: after applying the same fixes to OpenEXR it now works like a charm. I'll re-make a new pull-request and also make one to OpenEXR


Publish artifacts for Scala 2.12

I am in the process of cross-compiling some of my libraries for Scala 2.12. Would it be possible to publish Circe's artifacts for 2.12?

Unfortunately Cats is not yet published for 2.12 (if I remember correctly this is primarily an issue with test library versions). As soon as that's out I'll publish 2.12 artifacts for circe.

Great, thanks!

ticket on the availability of Cats for 2.12: https://github.com/typelevel/cats/issues/1285

@tindzk I've got #417 lined up and ready to go as soon as Cats 0.8.0 is published, and if you really wanted you could build that branch locally now after publishing this Cats branch (which should get merged today, I hope) locally.

That's great news! Thanks a lot.

Cats 0.8.0 is published

@SethTisue Yep. I got bogged down with a couple of iteratee.io changes yesterday, but the circe release will be out today.

Would be great to see 0.6.0-RC1 for 2.12 final!

it appears to me that 0.6.0 was published for 2.12 and this ticket can be closed?

@SethTisue Yep, thanks.


The two versions of depth camera data look the same

I changed renderSensorType in the AppMain.cpp to show the short/long depth map. but they are looked the same.

how can I switch the version of depth?


Building on Windows

I'm following the linux example changing names where necessary. I have gotten to the glue stage.
"bin/godot.windows.tools.64 --generate-mono-glue modules/mono/glue"
I'm getting a whole bunch of errrs and warnings.
C:\godot> bin\godot.windows.tools.64.exe --generate-mono-glue modules/mono/glue
Couldn't open C:\godot\bin/godot.windows.tools.64.pck
Couldn't open godot.windows.tools.64.pck
Couldn't load/find C:/godot/project.godot or project.binary
Couldn't load/find C://project.godot or project.binary
Activated GL 3.3 contextOpenGL ES 3.0 Renderer: AMD Radeon HD 6800 Series
GLES3: max ubo light: 409
GLES3: max ubo reflections: 455, ubo size: 144
WARNING: not found: gdnative/singletons
Initializing mono...
Mono: Runtime initialized
Mono: EVERYTHING OK
WARNING: BindingsGenerator::_populate_object_type_interfaces: Name of method Popup is ambiguous with the name of its class Popup. Renaming method to Popup_
At: modules\mono\editor\bindings_generator.cpp:1640
WARNING: BindingsGenerator::_populate_object_type_interfaces: Name of method Reference is ambiguous with the name of its class Reference. Renaming method to Reference_
At: modules\mono\editor\bindings_generator.cpp:1640
Generating ARVRAnchor...
Generating ARVRCamera...
Generating ARVRController...
*** lots more generating goes here ***
Generating _Thread...
ERROR: No loader found for resource: res://modules/mono/glue
At: core\io\resource_loader.cpp:184
ERROR: Failed loading scene: res://modules/mono/glue
At: main\main.cpp:1515
Mono: Runtime cleanup...
debugger-agent: Unable to listen on 5576
WARNING: ObjectDB::cleanup: ObjectDB Instances still exist!
At: core\object.cpp:1858
If I ignore all those and build:
"scons p=windows target=release_debug tools=yes"
I build and run the application fine.
The problem comes the moment I try and add a c# script.
The whole app hangs on 'Generating solution' and 'building GodotSharp solution', I get no errors in the debug window.

The errors/warnings at the end are normal. How long do you wait for it to build solutions?

The problem comes the moment I try and add a c# script.
The whole app hangs on 'Generating solution' and 'building GodotSharp solution', I get no errors in the debug window.

I have problems with it hanging in Windows too. The binary of build of Godot with GodotSharp itself is fine, but when in the editor hitting F6 does the same as what you've described. Linux works fine though. I'm rebuilding with the latest Godot and GodotSharp right now and will report back if I can get a csharp solution to compile.

I've left it alone for about an hour while I had dinner, no change.
I can also reproduce the F5 problem. If I kill the process on script creation and reopen godot the script is sometimes there. if I then click run, it hangs.

No idea why this happens on Windows. However, blocking the main thread while building is not a good idea in the first place. The reason for it to be this way is because EditorNode's build callback is not asynchronous, so I guess that needs to be changed.

So is it a fair assumption that a windows build is a no-go for now ?

@nobnob2015 I think so, I'm still having the same issue where it's just stuck on "Building Godot Sharp solution..." indefinitely.
There are no errors in the x64 native tools command prompt either.

Alright, the problem is that we redirect stdout and stderr from the msbuild process and if the buffer is full it will block trying to write to it. I'll fix it tonight.

Does this still happen after 03294a8?

Unfortunately it's still happening as of the latest commit.

Seems to build and compile/run fine for me now. Will play some more with it this evening.

Problem now is that none of the code I add ever gets hit, it's like the script doesn't get loaded. Breakpoints don't get hit either (should I open a new bug for this)

Yes, and include an example project if possible.

There was nothing special about the project. I just create a new project in godot, add a 2d node, attach a c# script to that node. No code in the ready function gets called or stopped on using breakpoints.

I finally got it to compile. :) But, it doesn't run any of the code. :(
Here's a minimal project with only ready and process methods overridden. Neither call their code.
GodotMonoExample.zip

What might be relevant is that if I open the basic project in visual studio, it tries to reference GodotSharpEditor.dll which does not exist.

Ok I was able to get the node to use the c# script. What you need to do is, after you add a node, name it something unique then when attaching a script make sure the script has the same name as the node that you just created (the class name doesn't matter as long as the script name and node name are the same).
Now to figure out inheritance ... :)

@NathanWarden Your example works fine for me? Could it be that you are using a build with mono_glue=no?
@nobnob2015 Yes, the file name must be the same as the class name. I'll add a warning later to make this is less confusing.

@neikeq I'll double check. I make my build for this via a bat file, so I'll run the commands manually this time just to be sure since bat files do tend to be far more finicky than Linux shell scripts.

@neikeq Yep, it was my batch file didn't execute the rest of the second build command. In other words, everything is good here after I change the slashes for the AssemblyInfo paths :)

@NathanWarden Are you able to debug any of the code?

No, but I'm not sure debugging is even implemented. If you need some sort of feedback from your code you can always use: GD.print(whatever);
I've played quite a bit already with GodotSharp on Linux, but I haven't tried using breakpoints or anything like that. I'm pretty sure Godot would have to actually attach itself to the debugger, and not just open the IDE by double clicking the file. I'm sure @neikeq could clarify whether or not this should be possible yet. I'm going to make a wild guess and say it isn't.
Regardless, I'm happy it's working on Windows. Thanks @neikeq for your help :)

The plan is to create debugging extensions for VS Code, MonoDevelop and Visual Studio.
If you don't want to wait, the Mono Debug extension for VS Code is working fine for me, it just requires some manual configuration.

That sounds great, would you be able to share that configuration

This is my launch.json:
{
    "version": "0.2.0",
    "configurations": [
        {
            "name": "Attach",
            "type": "mono",
            "request": "attach",
            "address": "localhost",
            "port": 23685
        }
    ]
}

What did you have to do within godot to get it to start debugging on port 23685, I can't find any settings that specify debug ports

I have also tried changing port to 6007. I then run the game, attach the debugger in VScode.
None of the breakpoints get hit but if I stop the game the debugger automatically stops in VSCode  so it seems there is some sort of a connection.

I added some project settings in 4d9ec0a to configure some of the debugger agent options, specially mono/debugger_agent/port. I fixed the editor as well, which was already listening on that port.


Enable HSTS for Nginx (firezone#889)

Pull Request Test Coverage Report for Build 30cc7d25520e0c7c3fde6c62697c79876a7daae4-PR-10

25 of 34 (73.53%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 69.444%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

apps/fz_http/lib/fz_http_web/live/notifications_live/badge.ex
1
2
50.0%

apps/fz_http/lib/fz_http/notifications.ex
18
20
90.0%

apps/fz_http/lib/fz_http/events.ex
0
6
0.0%

Totals

Change from base Build 2c9528ae46552e3ea917848a2a14c6e280c3febf:
0.1%

Covered Lines:
1025

Relevant Lines:
1476

💛  - Coveralls


Addition of Low Level Design of Snake and Ladders Game

I would like to add low level design of snake and ladder game in java which is very useful for the programmers who want to get hands on practice on OOPS concepts and loops.
Kindly assign me with this issue :)

THIS REPO IS EXCLUDED FROM HACKTOBERFEST  please contribute to other repo's


FP32 model performance is better than quant model using TFLite on raspi 3 for mobilenetv2

Hi,
I have checked out the the tensorflow master and built the tflite library on raspi3. When I use the label_image.cpp and pass in the  mobilenet_v2_1.0_224.tflite and execute , its performance is better than mobilenet_v2_1.0_224_quant.tflite model .  Ideally quant model performance should better than FP32. Below are the numbers that are observed
For FP32 -332 ms
For quant  model - 538 ms.
Is this a known issue? Could you check the same on your side and clarify.
Note: I do not see the same issue with mobilenetv1_1.0_224.tflite model. In mobilenetv1 quant model performs better than FP32.
Thank you,
Praveen.

Adding @petewarden for raspberry pi performance.


chore(deps): update terraform-module

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/key-protect-all-inclusive/ibm (source)
module
patch
4.11.4 -> 4.11.8

terraform-ibm-modules/landing-zone-vpc/ibm (source)
module
patch
7.18.0 -> 7.18.1

Release Notes

terraform-ibm-modules/terraform-ibm-kms-all-inclusive (terraform-ibm-modules/key-protect-all-inclusive/ibm)
v4.11.8
Compare Source
Bug Fixes

deps: update terraform terraform-ibm-modules/key-protect/ibm to v2.7.1 (#482) (5f058fa)

v4.11.7
Compare Source
Bug Fixes

deps: bump provider version of DA solution to 1.65.1 to pickup fix for provider bug(#481) (d7138b8)

v4.11.6
Compare Source
Bug Fixes

deps: upgrade module required provider to at least 1.65.0 (#480) (f1bea48)

v4.11.5
Compare Source
Bug Fixes

new input variable rotation_interval_month for deployable solution (#479) * added new input variable rotation_interval_month to the deployable architecture solution, default to 12 months (0a72f66)

terraform-ibm-modules/terraform-ibm-landing-zone-vpc (terraform-ibm-modules/landing-zone-vpc/ibm)
v7.18.1
Compare Source
Bug Fixes

added fix to ensure vpc_data is always output after initial apply (#781) (069d418)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

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

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

SKIPPING UPGRADE TEST
The changes in the VPC module in the example is causing some drift in the CBR zone module, which is the only error caught in upgrade test:
  Address: module.cbr_zone.ibm_cbr_zone.cbr_zone
  Actions: [update]
  DIFF:
    Before:
   {"addresses":"SECURE_VALUE_HIDDEN_HASH:-7012914b1b53d9c4909a57bb2e2d711469d54c2101aa486ed8d63f72"}
    After:
   {"addresses":"SECURE_VALUE_HIDDEN_HASH:-679c69692eb65886bccceb6243dffc5190d457afbf2e27c323355462"}

The "addresses" field is determined from the VPC module, which is not part of this MySQL module, and is only used in the example for the test.
I don't believe this upgrade issue has anything to do with the MySQL module, just something happening in the example, so I'll skip the upgrade test.

/run pipeline

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


DashCanvas bugs

If widgets are resized vertically such that the dashCanvas either gains or loses a vertical scroll bar, the margins do not update to accommodate the bar until the window is either refreshed or resized.
If the margin property of DashCanvasModel is made small ([7, 7] or less), a very slight horizontal scroll bar is always present.

Problem 1 is reproducible on React-Grid-Layout's demo: https://react-grid-layout.github.io/react-grid-layout/examples/1-basic.html so seems to be an issue with the underlying component. Might be able to submit a bug report


Uncaught error with _openssl.so with certain commands

Description
Full error and command here:
$ az webapp deployment user set --user-name example --password foobar
/home/justin/lib/azure-cli/local/lib/python2.7/site-packages/cryptography/hazmat/bindings/_openssl.so: cannot enable executable stack as shared object requires: Invalid argument
Traceback (most recent call last):
  File "/home/justin/lib/azure-cli/local/lib/python2.7/site-packages/azure/cli/main.py", line 36, in main
    cmd_result = APPLICATION.execute(args)
  File "/home/justin/lib/azure-cli/local/lib/python2.7/site-packages/azure/cli/core/application.py", line 176, in execute
    self.configuration.load_params(command)
  File "/home/justin/lib/azure-cli/local/lib/python2.7/site-packages/azure/cli/core/application.py", line 50, in load_params
    commands.load_params(command)
  File "/home/justin/lib/azure-cli/local/lib/python2.7/site-packages/azure/cli/core/commands/__init__.py", line 391, in load_params
    command_table[command].load_arguments()
  File "/home/justin/lib/azure-cli/local/lib/python2.7/site-packages/azure/cli/core/commands/__init__.py", line 335, in load_arguments
    self.arguments.update(self.arguments_loader())
  File "/home/justin/lib/azure-cli/local/lib/python2.7/site-packages/azure/cli/core/commands/__init__.py", line 653, in arguments_loader
    return extract_args_from_signature(get_op_handler(operation), no_wait_param=no_wait_param)
  File "/home/justin/lib/azure-cli/local/lib/python2.7/site-packages/azure/cli/core/commands/__init__.py", line 537, in get_op_handler
    op = import_module(mod_to_import)
  File "/usr/lib/python2.7/importlib/__init__.py", line 37, in import_module
    __import__(name)
  File "/home/justin/lib/azure-cli/local/lib/python2.7/site-packages/azure/cli/command_modules/appservice/custom.py", line 12, in <module>
    import OpenSSL.crypto
  File "/home/justin/lib/azure-cli/local/lib/python2.7/site-packages/OpenSSL/__init__.py", line 8, in <module>
    from OpenSSL import crypto, SSL
  File "/home/justin/lib/azure-cli/local/lib/python2.7/site-packages/OpenSSL/crypto.py", line 16, in <module>
    from OpenSSL._util import (
  File "/home/justin/lib/azure-cli/local/lib/python2.7/site-packages/OpenSSL/_util.py", line 6, in <module>
    from cryptography.hazmat.bindings.openssl.binding import Binding
  File "/home/justin/lib/azure-cli/local/lib/python2.7/site-packages/cryptography/hazmat/bindings/openssl/binding.py", line 13, in <module>
    from cryptography.hazmat.bindings._openssl import ffi, lib
ImportError: /home/justin/lib/azure-cli/local/lib/python2.7/site-packages/cryptography/hazmat/bindings/_openssl.so: cannot enable executable stack as shared object requires: Invalid argument

Environment summary
Install Method: How did you install the CLI? (e.g. pip, interactive script, apt-get, Docker, MSI, nightly)
I followed the windows install steps here:
https://docs.microsoft.com/en-us/cli/azure/install-azure-cli?view=azure-cli-latest
CLI Version: What version of the CLI and modules are installed? (Use az --version)
21:03:15:justin:/mnt/e/code/justinmchase-example$ az -v
azure-cli (2.0.18)

acr (2.0.12)
acs (2.0.16)
appservice (0.1.17)
backup (1.0.1)
batch (3.1.4)
billing (0.1.5)
cdn (0.0.9)
cloud (2.0.8)
cognitiveservices (0.1.8)
command-modules-nspkg (2.0.1)
component (2.0.7)
configure (2.0.11)
consumption (0.1.5)
container (0.1.11)
core (2.0.17)
cosmosdb (0.1.13)
dla (0.0.12)
dls (0.0.14)
eventgrid (0.1.4)
extension (0.0.3)
feedback (2.0.6)
find (0.2.7)
interactive (0.3.10)
iot (0.1.12)
keyvault (2.0.11)
lab (0.0.11)
monitor (0.0.10)
network (2.0.15)
nspkg (3.0.1)
profile (2.0.13)
rdbms (0.0.7)
redis (0.2.9)
resource (2.0.15)
role (2.0.12)
servicefabric (0.0.4)
sql (2.0.12)
storage (2.0.16)
vm (2.0.15)

Python location '/home/justin/lib/azure-cli/bin/python'
Extensions directory '/home/justin/.azure/cliextensions'

Python (Linux) 2.7.6 (default, Oct 26 2016, 20:30:19)
[GCC 4.8.4]

OS Version: What OS and version are you using?
Windows 10
Shell Type: What shell are you using? (e.g. bash, cmd.exe, Bash on Windows)
Bash on Ubuntu on Windows

Based on some other solutions to this I've seen I got a little further:
$ sudo apt-get install execstack
$ sudo execstack -c /home/justin/lib/azure-cli/local/lib/python2.7/site-packages/cryptography/hazmat/bindings/_openssl.so
$ az webapp deployment user set --user-name justin --password foobar
Operation failed with status: 'Bad Request'. Details: 400 Client Error: Bad Request for url: https://management.azure.com/providers/Microsoft.Web/publishingUsers/web?api-version=2016-03-01

I am wondering if I have something wrong with my openssl install or python? I installed python27 much earlier than this for use with nodejs. Which version of python should be used by this sdk?

I tried this:
pip install --upgrade --user 'cryptography==1.9'

But no dice.

Have you seen https://github.com/Azure/azure-cli/issues/4154?
I created a PR to fix this in Bash on Windows with the apt-get install https://github.com/Azure/azure-cli/pull/4251/files.
Also, how did you install as looking at the install path /home/justin/lib/azure-cli/bin/python, it's not installed with apt-get which is recommended on Bash on Windows.

Ah, that is a very helpful clue.
The first time I tried to install I ran this via GitBash:
curl -L https://aka.ms/InstallAzureCli | bash

After struggling with that for a while, I was informed that that wasn't supported and I should be using Bash on Windows + apt-get instead. So it looks like I have some cruft in my environment that I need to clean out that I wasn't aware was bleeding over. I'll do that and re-update in Bash on Windows.
Regarding that PR you linked, thanks again. I'm ok with running that manually for now and look forward to it rolling out in the official package update. This should unblock me, if I hit any other issues I'll open separate tickets thanks.


chore(master): release 0.2.2

:robot: I have created a release beep boop
0.2.2 (2024-07-20)
What's Changed

Fix compatibility with synfony console 4-5 by @roxblnfk in https://github.com/php-internal/dload/pull/9

Full Changelog: https://github.com/php-internal/dload/compare/0.2.1...0.2.2

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/php-internal/dload/releases/tag/0.2.2 :sunflower:


[ec2] Account for IMDSv2 config flag when requesting EC2 tags

What does this PR do?
When fetchin ec2 tags/IAM profile from ec2 medatada endpoint only
uses token/IMDSv2 when specified in config.
Motivation
Fix a missing change that should have been included in #6033.
Additional Notes
N/A.
Describe your test plan
N/A.

I will test the docker image ASAP on EC2.


configPracticeLobby breaking while handling options

Calling Dota2.configPracticeLobby like this:
Dota2.configPracticeLobby(""+Dota2.Lobby.lobby_id, {
                        "server_region": dota2.ServerRegion.EUROPE,
                        });

Causes the following error to pop up:
/somepathhere/node-dota2/handlers/helper.js:72
        type = possibleOptions[option];
                              ^

TypeError: Cannot read property 'server_region' of undefined
    at Object.Dota2._parseOptions (/somepathhere/node-dota2/handlers/helper.js:72:31)
    at Dota2Client.Dota2.Dota2Client.configPracticeLobby (/somepathhere/node-dota2/handlers/lobbies.js:69:25)
    at Dota2Client.<anonymous> (/somepathhere/node-dota2/test/myTest:118:27)
    at emitMany (events.js:108:13)
    at Dota2Client.emit (events.js:182:7)
    at Dota2Client.onChatMessage (/somepathhere/node-dota2/handlers/chat.js:141:10)
    at SteamGameCoordinator.fromGC (/somepathhere/node-dota2/index.js:71:38)
    at emitThree (events.js:97:13)
    at SteamGameCoordinator.emit (events.js:175:7)
    at SteamGameCoordinator.handlers.(anonymous function) (/somepathhere/node-dota2/node_modules/steam/lib/handlers/game_coordinator/index.js:107:10)

This is just because in lobbies.js under the Dota2.Dota2Client.prototype.configPracticeLobby definition _parseOptions is called without the Dota2._lobbyOptions argument as the second argument (as it should be and as it is called a few lines above).
I'm submiting a pull request with the simply addition so that this doesn't stump anyone else. :)

Are you on dev or master branch. I should have merged dev into master ages ago. :(

Working with dev but it's broken on branches I believe.

Ok great. Looking forward to PR.


Once a subscription is created, the plugin should post the subscription ID and info to the channel

Once a subscription is created, the plugin should post the subscription ID and info to the channel. This is typical of other integrations. This could either be a post in the channel by the bot or just an ephemeral post to the user creating the subscription.
Currently nothing happens after a subscription save.
This serves to provide feedback that the subscription was created successfully and also shows the ID in case I want to go make an edit right after saving.
note: This is somewhat lower impact on SN as subscriptions are shown in the RHS as well as the list command.

@manojmalik20 just posting for visibility that this is not resolved by https://github.com/Brightscout/mattermost-plugin-servicenow/pull/121 Keeping issue open.
To be explicit I don't see this being a blocking issue.

@DHaussermann fixed in PR https://github.com/mattermost/mattermost-plugin-servicenow/pull/169. Please verify

@mickmister @DHaussermann Can we close this issue?

Can we close this issue?

@raghavaggarwal2308 It looks like the PR is still open


Form of data

I want to know that data return in test.py is in which form?

The formatting is quite arbitrary, each sample separate by dashes lines, then 3 lines with: sample number, EEG channels, AUX data.
It's straightforward to change that to whatever you need by customizing the printData function, it's really just an example to show the most basic way to acquire data -- and I realize that the test script is somewhat duplicated, ./test_log.py and ./scripts/test.py.


exposing getFeatures api for ol.source.VectorTile with MVT format

Seems getFeatures api is not exposed to ol.source.VectorTile with MVT format. How can I get this functionality?

ol.source.VectorTile is optimized for rendering. It has feature access through ol.Map#forEachFeatureAtPixel only. If you want access to all features, you'd have to hook into the tileLoadFunction and grab the features in the same pass you use ol.VectorTile#setFeatures. This way you'll gather the features tile by tile.


e management

Environment

Operating System:
Vim version :version:
NERDTree version git rev-parse --short HEAD:
NERDTree settings applied in your vimrc, if any:

Process

Current Result

Expected Result

Screenshot(s)
Possible Fix

No info. Closing.

Errrrmmm.  I have NO idea how this ticket got created.  I’m hoping my kid mashed some keys when I was away, and that this isn’t the first sign of a split personality disorder.  :/


Add cross product matrix tooling

This adds scripts, docs and samples supporting dynamic, cross-product matrix generation for azure pipeline jobs.
It aims to replicate the cross-product matrix functionality in github actions,
but also adds some additional features like sparse matrix generation, cross-product includes and excludes, and (TODO) programmable matrix filters.
This functionality is made possible by the ability for the azure pipelines yaml to take a dynamic variable as an input
for a job matrix definition (see the code sample at the bottom of the linked section).
See the README.md file for more details on the config file syntax and usage, as well as implementation details.
The tests (test-matrix-functions.tests.ps1) contain a lot of detail on expected data structures at various processing stages. The -test-matrix-functions.ps1` file could perhaps be split up or use some more organization, so let me know if it's hard to navigate.
Example:
{
  "displayNames": {
    "/p:UseProjectReferenceToAzureClients=true": "UseProjectRef"
  },
  "matrix": {
    "operatingSystem": [
      "windows-2019",
      "ubuntu-18.04",
      "macOS-10.15"
    ],
    "framework": [
      "net461",
      "netcoreapp2.1",
      "net50"
    ]
  },
  "include": [
    {
      "operatingSystem": "windows-2019",
      "framework": "net461",
      "additionalTestArguments": "/p:UseProjectReferenceToAzureClients=true"
    }
  ],
  "exclude": [
    {
      "operatingSystem": "windows-2019",
      "framework": "net461"
    }
  ]
}

Sparse matrix job generation in a pipeline: https://dev.azure.com/azure-sdk/playground/_build/results?buildId=643038&view=results

Related discussion: https://github.com/microsoft/azure-pipelines-yaml/issues/20

Copying this PR over from https://github.com/Azure/azure-sdk-tools/pull/1254, since we want to land it in net before the shared tools repo (easier iteration).
@weshaggard Most of your comments from there have been addressed. Two updates:

Having the matrix variable name be dynamic may be redundant since it's scoped by job name in the variable lookup. I have made job name dynamic based on the matrix parameter name.
There aren't any sorting guarantees with the JSON cmdlets, so if we want "as on disk" sorting, we'll need to implement our own JSON converter with ordered dictionary, or switch the matrix config to be array based. I don't think the former would be too bad.

/check-enforcer evaluate

/check-enforcer evaluate


fix-forms-bugs

Fix all bugs on the Find Your Shrek doc that are related to general and admin forms.
Implement changes in FYA Change Log
Apply high fidelity to forms
closes #189

Fixed bugs from NPO:

No spacing between image and submit button once image uploaded
Copy to include with the Box Number image section: "In order to confirm your box, please include an image of the box number in the upper left hand corner of the box."
For launch method, can the methods please be: Given to someone, Launched at a location


Pan on x-axis doesn't work

Hi, I have a very wide div, basically I am making a Family Hierarchy so it will get as wider as possible.
However, for a very wide Div, I cannot pan on x-axis.
CodeSandbox
The only it Pan on X-axis works is if its zoomed-in and then you can pan left and right.
The styles are a bit off here but I guess its irrelevant for this issue.

+1. Any solutions found so far?

Up

Issue is related to the .react-transform-wrapper width. You need to control it with max-width so it will fit in the screen.
Solution:
.react-transform-wrapper {
   max-width: calc(100vw - 80px);
}


AKS control plane cost

As explained on https://learn.microsoft.com/en-us/azure/aks/concepts-clusters-workloads?source=recommendations#control-plane
"control plane is provided at no cost as a managed Azure resource abstracted from the user. You only pay for the nodes attached to the AKS cluster"
However, going over to AKS Cost Managment, system pool vmss (MC_ resource group) incurs charges according to VM / Storage used for virtual machine scale set. Even with user pools stopped.
Aren't system pool virtual machine scale sets considered "control plane"?  If not, shouldn't that be documented at this point in the documentation?
Later in the doc, at https://learn.microsoft.com/en-us/azure/aks/concepts-clusters-workloads?source=recommendations#nodes-and-node-pools, where user workloads are discussed, there's no mention of system pool either.
Since even with b4ms vmss + 99.95% SLA, cost is approx. 200$/m, for control plane, w/o any user workloads, maybe some clarifications could be made? So the cost structure is clearer?

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

ID: a445f8cc-6581-ad43-1dba-719a94eda81a
Version Independent ID: 31280746-d8de-91b4-d430-91693c2e869d
Content: Concepts - Kubernetes basics for Azure Kubernetes Services (AKS) - Azure Kubernetes Service
Content Source: articles/aks/concepts-clusters-workloads.md
Service: container-service
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@spiroskon
Thanks for your feedback! We will investigate and update as appropriate.

@spiroskon
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.

@MGoedtel
Can you please check and add your comments on this doc update request as applicable.

Thank you @spiroskon for the feedback!
We've added an item to our backlog and will prioritize accordingly.
#please-close


Added interface and initializer for locale aware validators

Hello,
I've added an interface for validators that are aware of a locale. This allows to distinguish between normal validators and validators which behavior changes with different locales.
I also added an initializer which uses the interface to pass the locale from MvcTranslator to the those validators. This allows to pass the locale to all validators by configuration, just by adding the initializer to ValidatorPluginManager:
return [
  'validators' => [
        'initializers' => [
            Zend\Validator\Service\LocaleAwareValidatorInitializer::class
        ]
    ]
];

In a second PR I'll add this interface to the i18n validators.

I've noticed that we could unify this initializer and could also use it for i18n view helpers and filters.

Could you provide more information on what problem / use case this addition solves?

This PR makes it possible to pass the config-defined locale to all validators as default. Currently you need to pass it every time when calling `ValidatorPluginManager::get()``

:-1: for initializer, it is a source of uncertainty for a whole application.
if you only use ValidatorPluginManager::get() in factories, as you should, it is not really a problem since you have access to the application environment there.

Tentatively scheduling for 2.10.0 (as it is a new feature). I'm on the fence, as I do not like encouraging the usage of initializers, and, as noted, this sort of feature may better work within the actual factories for validators. (Where that falls down is invokable validators; this sort of feature could also be enabled via delegator factories, however, which would address that particular use case.)

@weierophinney: This is why I've made initializers optional. The framework user has to define that in their config to use that feature.
As an alternative I could provide an trait for factories, to get the locale and pass it to the validator. But that wouldn't make setting the locale optional and would change the default behavior of the framework.

Rebased to develop and set zendframework/zend-servicemanager to minimum 2.7.6. Changes require the ContainerInterface.

I'm choosing not to merge this at this time. Thanks for the effort you put into it, @fabiang .
The reason I'm choosing not to merge is because we're removing i18n features from the core zend-validator library for version 3 (see the zend-validator v3 RFC for details).
Any feature like this would need to be in a package that ships localized validators (e.g., zend-i18n, or a future zend-i18n-validator package).