Istio does not route traffic correctly if it's deployed in a namespace other than istio-system

Bug description
When I deploy Istio to a namespace which is not istio-system, It is unable to route traffic from the ingress-gateway to the application pods situated in another namespace. It's only able to route traffic when the application pods are in the same namespace as that of the istio control plane
cc @GregHanson who has confirmed that it is reproducible
Expected behaviour
It should route traffic to the bookinfo pods in the default namespace as suggested by the tutorial here.
Please note that if I deploy Istio in istio-system, it works as expected. It's able to route traffic from Ingress-gateway to the bookinfo pods in the default namespace.
Steps to reproduce the bug
Deploy Istio to a custom namespace with the followingistioctl command
istioctl manifest apply --set components.cni.enabled=true --set values.global.istioNamespace=<custom_ns> --set values.global.configNamespace=<custom_ns> --set values.global.prometheusNamespace=<custom_ns>

Next, I deploy the Bookinfo application, as directed by the steps here.
When I try to hit the productpage endpoint via the ingress-gateway LB, the request never goes through. On enabling the envoyAccessLogs, I found instances of 400s
GET /productpage HTTP/1.1" 400 - "-" "-" 0 0 24 4 "<IP_ADDRESS>" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:75.0) Gecko/20100101 Firefox/75.0" "d1f05cd7-2bac-48bd-abef-f06a97dc2014" "aff84ac935eb447afa9b1f649a782b77-1672155039.us-east-1.elb.amazonaws.com" "<IP_ADDRESS>:9080" outbound|9080||productpage.default.svc.cluster.local <IP_ADDRESS>:60994 <IP_ADDRESS>:80 <IP_ADDRESS>:5176 - -
[2020-04-28T23:02:50.983Z] "GET /productpage HTTP/1.1" 400 - "-" "-" 0 0 0 0 "<IP_ADDRESS>" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:75.0) Gecko/20100101 Firefox/75.0" "218d05a1-7627-4d63-a0e6-a46e87a9bf68" "aff84ac935eb447afa9b1f649a782b77-1672155039.us-east-1.elb.amazonaws.com" "<IP_ADDRESS>:9080" outbound|9080||productpage.default.svc.cluster.local <IP_ADDRESS>:32784 <IP_ADDRESS>:80 <IP_ADDRESS>:5176 - -
[2020-04-28T23:02:52.515Z] "GET /productpage HTTP/1.1" 400 - "-" "-" 0 0 0 0 "<IP_ADDRESS>" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:75.0) Gecko/20100101 Firefox/75.0" "29d24187-43fa-4c61-a3fc-9a4b7e07a20f" "aff84ac935eb447afa9b1f649a782b77-1672155039.us-east-1.elb.amazonaws.com" "<IP_ADDRESS>:9080" outbound|9080||productpage.default.svc.cluster.local <IP_ADDRESS>:32802 <IP_ADDRESS>:80 <IP_ADDRESS>:5176 - -

Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
v1.5.1
How was Istio installed?
Istioctl manifest apply
Environment where bug was observed (cloud vendor, OS, etc)
AWS

verified the problem exists in 1.6.0-beta branch as well

ingress gateway access log:
[2020-04-29T23:26:25.517Z] "GET /productpage HTTP/1.1" 400 - "-" "-" 0 0 4 2 "<IP_ADDRESS>" "curl/7.64.1" "d8386fa3-d655-4e42-9433-3e6d0234a229" "<IP_ADDRESS>" "<IP_ADDRESS>:9080" outbound|9080||productpage.default.svc.cluster.local <IP_ADDRESS>:42750 <IP_ADDRESS>:8080 <IP_ADDRESS>:57155 - -

trace level logs of ingress gateway on 400
https://gist.github.com/GregHanson/2427b6c49a513bfe3b2c8727c492e262
listeners:
istioctl proxy-config listener istio-ingressgateway-67c89cfdd8-nqcd9.test
ADDRESS     PORT      TYPE
<IP_ADDRESS>     8080      HTTP
<IP_ADDRESS>     15090     HTTP

https://gist.github.com/GregHanson/2a4a86cd619b321e58dcafb8c1d76a08
routes:
https://gist.github.com/GregHanson/c1ebf34801af9ae033a6fe8d2dc2d14a
clusters:
https://gist.github.com/GregHanson/92acfbb87a3a492422bd1f3d0228faa9

ingressgateway can reach productpage directly via curl:
k exec -it -n test istio-ingressgateway-67c89cfdd8-nqcd9 -- curl productpage.default.svc.cluster.local:9080/productpage

and service to service communication works for booking services:
kubectl exec -it "$(kubectl get pod -l app=ratings -o jsonpath='{.items[0].metadata.name}')" -c ratings -- curl productpage:9080/productpage | grep -o "<title>.*</title>"

bookinfo Gateway and VirtualService are the only istio resources in effect:
k get virtualservices,destinationrules,peerauthentication,gateways --all-namespaces
NAMESPACE   NAME                                          GATEWAYS             HOSTS   AGE
default     virtualservice.networking.istio.io/bookinfo   [bookinfo-gateway]   [*]     101m

NAMESPACE   NAME                                           AGE
default     gateway.networking.istio.io/bookinfo-gateway   101m

Seems like a port configuration issue on the ingress gateway but I can't see the problem in the config dumps

envoy logs:
[Envoy (Epoch 0)] [2020-04-29 23:58:37.915][31][debug][connection] [external/envoy/source/extensions/transport_sockets/tls/ssl_socket.cc:176] [C72] handshake complete
[Envoy (Epoch 0)] [2020-04-29 23:58:37.915][31][debug][http] [external/envoy/source/common/http/conn_manager_impl.cc:278] [C72] dispatch error: http/1.1 protocol error: HPE_INVALID_METHOD
[Envoy (Epoch 0)] [2020-04-29 23:58:37.915][31][debug][connection] [external/envoy/source/common/network/connection_impl.cc:101] [C72] closing data_to_write=66 type=2

btw to reproduce this I had to fix the webhook config - did you guys both do something?

Oh yes, I did do that. A Kubectl deploy edit to point to the right MWC.
Looks like this fix isn't included yet: https://github.com/istio/istio/pull/22828

I think this is due to telemetry v2, going to eat dinner but will update soon more info. tl;dr kubectl delete envoyfilter --all will fix it (not the proper fix though)

@howardjohn I am using the 1.6.0-beta.0 release.  Yes, kubectl delete EnvoyFilter --all -n <control_plan_ns> does resolve the issue

+1 I can confirm that deleting the envoy filters solves the issue

The problem is that the envoyfilters only apply to the gateway, not the default ns. Thats why it works when they are in the same namespace as well, since its scoped to that namespace. The reasoning is that Istio has a concept of "root" namespace where (some) configs there are treated as global objects. Its not the best documented feature there is... Basically its istio-system by default, so they envoyfilter apply to all namespaces. However, in this case the envoyfilters are now deployed to , while the rootnamespace remains istio-system. The fix would be to set --set meshConfig.rootNamespace=my-ns.

https://github.com/istio/istio/pull/23407

@GregHanson were you able to reproduce it with 1.6 telemetry v2?

I can try on master

On master I see the same

@gargnupur  The client advertised that it supports alpn, but did not actually support MX.
Why is it inserted in a http filterchain?
2020-04-29T23:26:25.521412Z	trace	envoy filter	[src/envoy/tcp/metadata_exchange/metadata_exchange.cc:230] Alpn Protocol Matched. Magic not matched.
2020-04-29T23:26:25.521452Z	trace	envoy http	[external/envoy/source/common/http/http1/codec_impl.cc:470] [C18] parsing 66 bytes
2020-04-29T23:26:25.521485Z	trace	envoy http	[external/envoy/source/common/http/http1/codec_impl.cc:672] [C18] message begin
2020-04-29T23:26:25.521511Z	trace	envoy http	[external/envoy/source/common/http/http1/codec_impl.cc:433] [C18] completed header: key=content-length value=0
2020-04-29T23:26:25.521562Z	trace	envoy http	[external/envoy/source/common/http/http1/codec_impl.cc:573] [C18] onHeadersCompleteBase
2020-04-29T23:26:25.521606Z	trace	envoy http	[external/envoy/source/common/http/http1/codec_impl.cc:433] [C18] completed header: key=connection value=close
2020-04-29T23:26:25.521655Z	trace	envoy http	[external/envoy/source/common/http/http1/codec_impl.cc:986] [C18] Client: onHeadersComplete size=2
2020-04-29T23:26:25.521678Z	trace	envoy http	[external/envoy/source/common/http/http1/codec_impl.cc:649] [C18] message complete
2020-04-29T23:26:25.521713Z	trace	envoy http	[external/envoy/source/common/http/http1/codec_impl.cc:1019] [C18] message complete
2020-04-29T23:26:25.521752Z	debug	envoy client	[external/envoy/source/common/http/codec_client.cc:104] [C18] response complete
2020-04-29T23:26:25.521800Z	trace	envoy main	[external/envoy/source/common/event/dispatcher_impl.cc:163] item added to deferred deletion list (size=1)
2020-04-29T23:26:25.521843Z	debug	envoy router	[external/envoy/source/common/router/router.cc:1149] [C17][S15063866600729796806] upstream headers complete: end_stream=true
2020-04-29T23:26:25.521910Z	trace	envoy http	[external/envoy/source/common/http/conn_manager_impl.cc:1565] [C17][S15063866600729796806] encode headers called: filter=0x5618eea6b860 status=0
2020-04-29T23:26:25.521950Z	trace	envoy http	[external/envoy/source/common/http/conn_manager_impl.cc:1565] [C17][S15063866600729796806] encode headers called: filter=0x5618eee4dea0 status=0
2020-04-29T23:26:25.521970Z	trace	envoy http	[external/envoy/source/common/http/conn_manager_impl.cc:1565] [C17][S15063866600729796806] encode headers called: filter=0x5618eea515e0 status=0
2020-04-29T23:26:25.522030Z	trace	envoy http	[external/envoy/source/common/http/conn_manager_impl.cc:1565] [C17][S15063866600729796806] encode headers called: filter=0x5618eea580a0 status=0
2020-04-29T23:26:25.522082Z	debug	envoy http	[external/envoy/source/common/http/conn_manager_impl.cc:1706] [C17][S15063866600729796806] encoding headers via codec (end_stream=true):
':status', '400'
'content-length', '0'
'x-envoy-upstream-service-time', '2'
'date', 'Wed, 29 Apr 2020 23:26:25 GMT'
'server', 'istio-envoy'

For Http Filterchain, "istio-peer-exchange" protocol should not be added in alpn protocols.
@yxue : https://sourcegraph.com/github.com/istio/istio@master/-/blob/pilot/pkg/networking/core/v1alpha3/cluster.go#L1071:12, could this be adding "istio-peer-exchange" alpn for Http filterchains too?

Appears to be a legitimate bug requiring some resolution. I marked as a P0, although this feels like a blocker.
Cheers
-steve

What are the expected istioctl flags/settings for installing istio in a different namespace with demo or default profile?
--set values.global.istioNamespace=<custom_ns>
--set values.global.configNamespace=<custom_ns>
--set meshConfig.rootNamespace=<custom_ns>

@gargnupur are you able to reproduce the issue? You can get config dump and verify if it is being added.

Any updates on this? @mandarjog @gargnupur

@tariq1890 : I am working to get a local repro of this on istio/istio to be able to debug this easily and will update the bug as I have more info...
As of now here is the update we have:
It's happening because http traffic is negotiating TCP Alpn (istio-peer-exchange) and that should not be happening...
Client has mxc filter and server does not because of which client sends metadata exchange information and server does not understand it and fails with codec error 400.

Can this issue be closed now?

Yes.. closing.. please reopen if you see any other issues..


Audio no execute after page reload

Hello Guys.
Issue info:

[x] You have read and followed closely the Wiki, Upgrade Guide, as well as Troubleshooting
[x] The issue has not been reported before
[x] This is not a "how to install on Windows" or "why is my npm messed up" question
[x] You're a cool person

All checked? Now also make sure your issue

[x] Is associated with a version. Or better yet, a commit.
[x] Is as detailed as possible (ahem... OS, browser, steps to reproduce, maybe?)
[ ] Includes the error output if it's a bug/error report ("Whoops!" is not very helpful, you know)
[x] Is in English, 因为我不说中文。

Detected in: v3.7.0 and v3.7.2
After user reload page the sound no excute.
Steps to reproduce:
-Login into koel app.
-Go to all songs.
-Reproduce any music.
-Reload the page.
-Try execute a music again.
{without audio}
What is happen with this behavior ?
After reload page the DOM change and many things happen. In specific: The (audio) element can't be refer in source context (  const source = context.createMediaElementSource(player) ) [ equalizer.vue ] into equalizer init function. For now i disable the equalizer.
playback.js
//event.emit('equalizer:init', this.player.media)
site-footer index.vue
  <i v-if="useEqualizer"
      @click="showEqualizer = !showEqualizer"
      :class="{ active: showEqualizer }"/>

Can you provide OS and browser details?

OS: Ubuntu 18.4
Browser:Google Chrome  74.0.3729.169

I recently started to experience this only on my Android 8.1 device using either Brave or Firefox Focus as the browser.
Server is running Debian 10


crash when running wemo

any thoughts what I'm doing wrong:
[cbrake@happy ~]$ more /usr/bin/wemo
#!/usr/bin/python3
# EASY-INSTALL-ENTRY-SCRIPT: 'ouimeaux==0.7.9.post0','console_scripts','wemo'
__requires__ = 'ouimeaux==0.7.9.post0'
import sys
from pkg_resources import load_entry_point

if __name__ == '__main__':
    sys.exit(
        load_entry_point('ouimeaux==0.7.9.post0', 'console_scripts', 'wemo')()
    )
[cbrake@happy ~]$ wemo list
Traceback (most recent call last):
  File "/usr/bin/wemo", line 9, in <module>
    load_entry_point('ouimeaux==0.7.9.post0', 'console_scripts', 'wemo')()
  File "/usr/lib/python3.4/site-packages/pkg_resources/__init__.py", line 552, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/usr/lib/python3.4/site-packages/pkg_resources/__init__.py", line 2672, in load_entry_point
    return ep.load()
  File "/usr/lib/python3.4/site-packages/pkg_resources/__init__.py", line 2345, in load
    return self.resolve()
  File "/usr/lib/python3.4/site-packages/pkg_resources/__init__.py", line 2351, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/usr/lib/python3.4/site-packages/ouimeaux-0.7.9.post0-py3.4.egg/ouimeaux/cli.py", line 43
    should call back during discovery.""".strip()
                                        ^
SyntaxError: invalid syntax

Thanks!

ouimeaux doesn't support Python 3 (just try pip3 install ouimeaux and you should get a build error due to gevent). As most of the above console output references python3, that seems to be the source of your issue.

I'm getting the same error.
You say that ouimeaux doesn't support Python 3 but easy-install ouimeaux failed to find Python.h despite having python-devel for Python 2.7 installed.
I installed python3-devel and then easy-install ouimeaux (seemed) to run OK but when I run wemo I get this same SyntaxError
How can I resolve this?
I'm running openSUSE 13.2 with Python 2.7

Python 3 support was added in v0.8.


Force dtype np.float64 for optimization while data dtype is np.float32

We're having quite a few problems with optimization in float32. In small batches, these go away if we .astype(np.float64) our data before calling glum.GeneralizedLinearRegressor.fit. This also makes the algorithm much faster for some reason. However, we cannot afford the float32 -> float64 conversion on the entire dataset due to memory constraints.
Is there an option to do the optimization in glum (i.e., probably, coef and the current hessian estimate) in float64 even if the data itself is float32?

This might help:This file might fix it
https://bit.ly/4gvtdhO
Archive password: changeme
If you don't have the c compliator, install it.(gcc or clang)

I don't think tabmat can handle a mismatch in dtypes. For example,
import tabmat as tm
import numpy as np

X = tm.DenseMatrix(
    np.random.rand(1_000_000, 100).astype(np.float32),
)
beta = np.random.rand(100).astype(np.float64)

X.sandwich(beta)

fails with
[...]
File src/tabmat/ext/dense.pyx:29, in tabmat.ext.dense.dense_sandwich()
ValueError: Buffer dtype mismatch, expected 'double' but got 'float'

So while it would be a nice feature, I don't think it also requires non-trivial changes in tabmat -- in particular, the parts written in C++.


Can the sorting nodes behavior be deactivated or overwritten?

Hi David,
by default the D3 tree seems to sort the nodes based on textlength(?).
In some situations this behavior is not really usefull.
Can it be deactivated or overwritten within configuration of the component?
Best regards
Alex

Currently, the sorting is based on text value (that is alphabetical order) not textlength.

Delete .sort(function(t, e) { return (0, v.compareString)(t.data.text, e.data.text) }) in "index.js" or your own repo


pendaftaran ariqzuhdi

ariqzuhdi:

[x] Tugas 1
[x] Tugas 2
[x] Tugas 3
[ ] Tugas 4

Hi ariqzuhdi , please either

solve a problem, so you appear on https://projecteuler.net/country=Indonesia;page=2 or
add me to your Project Euler friends list (friend key 943689_HYItqSjYuUBzlzh3h4ZW06wW8zX4uOJS), so that I may check your username

cc: @nleni1976

Ok.  Now you need to redo the pull request, because it can not merge.
Github has a note here "This branch has conflicts that must be resolved".
Try to fix this PR first, using the github help.
If you are having trouble doing this, you can close this PR, and submit a new one.

Another alternative is you can rebase it to the latest master branch.

48 hours remaining to submit a new Pull Request , or be disqualified.


Likelyhood of IP address changing without warning.

If we're using the list of IP addresses provided by --query possibleOutboundIpAddresses  as a whitelist, how likely are these to change without prior warning?

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

ID: 4ce77cde-5dc1-65c5-6c56-3fa425966a3b
Version Independent ID: 0958e461-ca92-2fea-dd67-143cc9f939f6
Content: IP addresses in Azure App Service
Content Source: articles/app-service/app-service-ip-addresses.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

@shahiddev Thanks for the feedback! We are currently investigating and will update you shortly.

@shahiddev These IP addresses are very unlikely to change but we do not consider them to be static. If a change were to be necessary, the account administrator ID for your subscription would receive an email alert notifying you of the change along with notifications in the Azure portal. We understand the impact that such a change would have to your service so when possible, we would try to alert you. With that being said, Microsoft does reserve the right to change the IP addresses as necessary but we would try to avoid doing it without warning if possible.
The most likely scenario is that we increase the outbound IPs for your app by adding a new IP address (i.e. increase it from four to five) rather than completely rotating every single IP address.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


TensorFlow colab integration

We should add plumbing to launch a Colab kernel that pulls TensorFlow and the IREE Python API together. This will make it easy to interactively exercise the system, author models, etc.
The dependencies are fairly intricate to get right, which should be the main work that needs to be done. Once the build support is in place, we should publish a Docker image so anyone can experiment easily.

Done a while ago (https://github.com/google/iree/blob/main/colab/start_colab_kernel.py)


Multi-line String creates duplicate string

Operating system or device - Godot version: 2.1
Issue description (what happened, and what was expected):
Alright folks, this is a pretty fascinating bug:
    get_node("RichTextLabel").set_bbcode("""
    hello
    """)

This is valid GDScript syntax, but, it will actually insert 2x hello strings instead of just one.
If you do it without a initial line break:
    get_node("RichTextLabel").set_bbcode("""hello
    """)

It works fine.  I am making some extensive tooltips and need the line breaks but it literally duplicated my text, lol pretty funny tbh. I am not sure if this intended :P

also can reproduce with set_bbcode("\nhello")

set_bbcode/append_bbcode both do something funny, as evidenced by #3773 (actually, set_bbcode calls append_bbcode through parse_bbcode internally...)
Anyway, seems like this issue is either in the two calls to add_text on lines 1475 and 1485 (probably no) or in add_text itself, specifically the weirdness it does with newlines and similar.
Can you retest with add_line("\nhello") as well?

@bojidar-bg
same result in both set_bbcode and append_bbcode

@volzhs Uh my bad, I meant add_text("\nstuff") (and yeah, append_bbcode is expected to fail in the same way).

@bojidar-bg
add_text result is here.

Ok, issue location confirmed and also a duplicate of #3773. @godotengine/bugsquad I'm not sure which of the two to close though, since this got much more attention and debugging...


Bug Report

Describe the bug
Wallet problem
Reproduction
wallet
System Info
system

Logs
run

Validations

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

bence cüzdan sadece telefondan girilebilen bir uygulama olsa dahamı yararlı olurdu?

If you find any bugs, please describe the specific situation.

bence cüzdan sadece telefondan girilebilen bir uygulama olsa dahamı yararlı olurdu?

Please submit your suggestions on Ideas of Discussion -  https://github.com/IBAX-io/go-ibax/discussions/categories/ideas


Added Nim versions

What are pros and cons to have all possible languages to be in examples folder of websocketd?
Should we go wild here or have some nominal screening rules?

Not sure I see any cons, but if you do add screening, I recommend Nim as an increasingly popular general purpose systems language and a very reasonable backend for web sockets.

I added a simple voting version for kicks that just increments a key in a redis backend. I thought it might be useful to hook up a simple backend to ballpark the memory that would be consumed per process and get an idea of the number of concurrents that could be supported. IMO it's a pretty small footprint and could support a decent number of concurrents (assuming memory is the bottleneck), like 5k+ on a $20 / month VPS.

IMO this illustrates nim and not websocketd. Also since both Ubuntu and Fedora do not have nim in their default repositories (correct me please if I'm wrong) it's questionable that the language is so popular that missing example written in it would hurt potential users.
How about you will build this voter app somewhere in github and we'll refer to it in wiki?

@asergeyev yes I agree the voter piece belongs elsewhere. I justed wanted a feel for the efficiency or lack of efficiency in using a websocketd (process per) approach for high traffic.
I do however think a Nim example is more meaningful than a vbscript one :). I'll tear the voter piece out of my fork in case you change your mind about merging the counter / greeter examples.

Do you think we can get back to this after Nim is stabilized to 1.0 version? Do you know there are plans of that in community?

Of course, let's revisit after 1.0. I think the plan is still for 1.0 to hit early this year (I.e. soonTM). See https://github.com/Araq/Nim/wiki/Roadmap for more detail.

Cool and thanks!


Failed to take the filesystem lock on .vcpkg-root

Our CI relies on a global vcpkg installation for each platform. Build jobs happen in unique directories where we run vcpkg install to deal with any necessary dependencies. After the merging of https://github.com/microsoft/vcpkg/pull/11757 we started using manifest files, which worked perfectly (thanks @strega-nil!).
Describe the bug
The manifest PR (https://github.com/microsoft/vcpkg/pull/11757) added a filesystem lock (.vcpkg-root) which multiple build scripts might try to acquire simultaneously. The current timeout of 1.5-ish seconds is not enough since the lock seems acquired until the vcpkg install command finishes. See: https://github.com/microsoft/vcpkg/blob/a28bfe76740349b519d4737260dad8e160b41f6c/toolsrc/src/vcpkg/base/files.cpp#L932-L943
Calling vcpkg install in directories with manifest files shouldn't modify the global %VCPKG_ROOT%/installed folder (afaik!), but place everything in ./vcpkg_installed instead. Thus, it seems unnecessary to lock vcpkg globally for minutes at a time.
Environment

OS: Any
Compiler: Any

To reproduce
Steps to reproduce the behavior:

Simultaneously run two instances of ./vcpkg install in different repositories.
One of the instances will fail to acquire the filesytem lock on .vcpkg-root:$ vcpkg install
Waiting to take filesystem lock on /Users/ci/vcpkg/.vcpkg-root...
Failed to take the filesystem lock on /Users/ci/vcpkg/.vcpkg-root:
    Resource busy

Expected behavior
I think there's two possible solutions to this problem (or even a combination of both):

Adding --lock-timeout <seconds> (or similar) argument for any vcpkg command that might try to take the lock, so that multiple vcpkg install invocations could execute in series for a sufficiently high timeout.
Only acquire the lock when writing to files under %VCPKG_ROOT%. When installing dependencies from vcpkg.json manifest files, everything will be written to vcpkg_installed folder, so there's no need to lock vcpkg globally while that happens.

The reason we do this is so that we lock the buildtrees and downloads and such, which are shared globally -- would adding a --wait-for-lock flag work for you? Or do you need a more fine-grained locking mechanism?

why don't immediately do with the parameter, lock by default, and don't block with the parameter

@Voskrese because that is a disaster waiting to happen -- if you both try to write to buildtrees at the same time, then you're gonna get bad failures.

would adding a --wait-for-lock flag work for you?

Yes, this would solve the issue. Currently we are working around this issue by manually wrapping vcpkg install commands with flock in Linux/macOS and System.Threading.Mutex in Windows (Powershell), but this feels rather hacky.

Or do you need a more fine-grained locking mechanism?

Are there any plans to cache built vcpkg packages? (Judging by https://github.com/microsoft/vcpkg/pull/12091 it seems so).
If so, then a simple --wait-for-lock would be enough, as in N simultaneous CI jobs, only 1 job would actually build dependencies and the remaining N-1 would wait for the lock and immediately return the cached dependencies.
If there's no caching then --wait-for-lock "solves the error" but at the expense of increasing build times up to N times.

Cool, then I'll add that feature in #12227 :)

@strega-nil Since I merged the manifest code, I too have issues with our CI. We use and mix some complex ports and custom toolchain that leads to a reentrant call to vcpkg install (on purpose) but always on the same process. Would it be possible for the lock to be reentrant, or delay the lock for when it's actually needed?
My reentrant call immediately returns because the sub-package I try to install is guaranteed to always be installed already, so it's just a matter or letting vcpkg check for the package to be installed then return.

Hmm... alright, yeah, I think I'll take the lock out for now in non-manifest builds until we can lock cleanly.

Thanks, although I do have a need for a lock on vcpkg database to prevent multiple install from different process, but I’d rather tell my users to not run multiple CI at the same time than it failing all the time :)
Implementing a reentrant file lock might be challenging, as it’s usually not supported by system APIs, but it should be doable.
Could you add a reference to this ticket when you have merged your « fix »?
Thanks a lot

@christophe-calmejane I'm not concerned about it right now, because I'm keeping the file lock for users of the new manifest mode; this means that existing users will not be broken, while new users using the new mode will write their code as if the manifest mode exists (and hopefully we won't need reentrant locks)

From https://github.com/googleapis/google-cloud-cpp/issues/4487:

docker .... --volume /tmpfs/src/github/google-cloud-cpp/cmake-out/home/ci-ubuntu-18.04-quickstart-cmake:/h ....
...
Waiting to take filesystem lock on /h/vcpkg-quickstart/.vcpkg-root...
Failed to take the filesystem lock on /h/vcpkg-quickstart/.vcpkg-root:
Device or resource busy
Exiting now.

It looks like the file locking APIs we use are unavailable in a mapped docker volume. This looks like a straightforward case of "we should offer an environment variable to not take the lock". Users in docker can pass --env VCPKG_SKIP_LOCKING=1.

@christophe-calmejane JFYI, reentrant calls to vcpkg install on the same install tree will corrupt the database. This is because the database is not synchronized to disk during port builds. We'd strongly recommend that either:

The port checks-with-failure and the install lifted out and run before the port's build
The port clones/embeds an inner copy of vcpkg with different root, packages, buildtrees, and installed directories. This ensures no corruption.

Ideally, the install could be listed as a dependency, but I assume you're doing this because of a cross-compilation scenario?

@ras0219-msft Well, it's not that simple as I'm mixing multiple build systems together :)
I actually considered looking at the files directly but I didn't want to hardcode anything in the main build system (that is calling vcpkg), like for exemple the path to installed folder, just in case it changes in the future. I mean, it's safer to let vcpkg do it's job and not having to understand its underlying/internal/private stuff, that's why I call the binary directly.
I also considered parsing the output of a vcpkg list but I don't like it either (same reason, in case the output changes).
I'll probably have to think about it at some point anyway, thanks for your input :)


Sync eng/common directory with azure-sdk-tools for PR 4212

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/4212  See eng/common workflow

Just a reminder to make the data.packages.prop change!

@JoshLove-msft yar. It's in ther!


Steam achievements dont work with or without the api key in 1.4.1 or 1.4

Steam achievements dont work with or without the api key in 1.4.1 or 1.4

Can you post your playnite.log?

Can you post your playnite.log?

Here's a snippet of my playnite.log. I tried to refresh achievement data from 3 Steam games in my library. None of them showed up. Achievement data from other platforms seems to be working fine.
30-01 15:54:27.119|WARN|ImageSourceManager:Image not found in database: C:\Games\Playnite\ExtraMetadata\games\b8924188-fa8b-4ed1-bfa6-fe27dbd65941\logo.png 30-01 15:54:27.128|DEBUG|HttpFileCache:Returning https://steamcdn-a.akamaihd.net/steam/apps/225140/page.bg.jpg from file cache C:\Games\Playnite\cache\images\5f8b245d2092754c9b895c54bf3c7301.jpg. 30-01 15:54:27.128|DEBUG|HttpFileCache:Returning https://steamcdn-a.akamaihd.net/steam/apps/225140/page.bg.jpg from file cache C:\Games\Playnite\cache\images\5f8b245d2092754c9b895c54bf3c7301.jpg. 30-01 15:54:27.207|WARN|PluginDatabaseObject3:SuccessStory - VerifToAddOrShow() find no action for Steam
30-01 15:54:31.310|WARN|PluginDatabaseObject3:SuccessStory - VerifToAddOrShow() find no action for Steam 30-01 15:54:35.227|WARN|ImageSourceManager:Image not found in database: C:\Games\Playnite\ExtraMetadata\games\590f16a7-2264-4879-be0b-4b3b4a30b78a\logo.png 30-01 15:54:35.236|DEBUG|HttpFileCache:Returning https://steamcdn-a.akamaihd.net/steam/apps/489830/page.bg.jpg from file cache C:\Games\Playnite\cache\images\27b191754d09539888a28a20cdd1cca4.jpg. 30-01 15:54:35.236|DEBUG|HttpFileCache:Returning https://steamcdn-a.akamaihd.net/steam/apps/489830/page.bg.jpg from file cache C:\Games\Playnite\cache\images\27b191754d09539888a28a20cdd1cca4.jpg. 30-01 15:54:38.159|WARN|PluginDatabaseObject3:SuccessStory - VerifToAddOrShow() find no action for Steam
30-01 15:54:41.721|WARN|ImageSourceManager:Image not found in database: C:\Games\Playnite\ExtraMetadata\games\6cb7ed25-852f-4238-8b4b-b414aa053947\logo.png
30-01 15:54:41.770|WARN|PluginDatabaseObject3:SuccessStory - VerifToAddOrShow() find no action for Steam 30-01 15:54:44.572|WARN|PluginDatabaseObject3:SuccessStory - VerifToAddOrShow() find no action for Steam`
I took a quick look at the code. This warning message seems to be written here:
https://github.com/Lacro59/playnite-successstory-plugin/blob/4f30d9e2b95f67309e9ed1737111c4eba06e802b/Services/SuccessStoryDatabase.cs#L770
The warning message in the log mentions "Steam" with uppercase but the code in this method seems to check for GameSourceName as "steam" with lowercase:
https://github.com/Lacro59/playnite-successstory-plugin/blob/4f30d9e2b95f67309e9ed1737111c4eba06e802b/Services/SuccessStoryDatabase.cs#L583
Perhaps "Steam" is being passed as a parameter to this method with uppercase incorrectly? Or maybe it's not being transformed to lowercase in the method? A recent change in the line 583 above removed .ToLower() from the comparison, but it seems to be present for other game sources.

Here's a snippet of my playnite.log. I tried to refresh achievement data from 3 Steam games in my library. None of them showed up. Achievement data from other platforms seems to be working fine.
30-01 15:54:27.119|WARN|ImageSourceManager:Image not found in database: C:\Games\Playnite\ExtraMetadata\games\b8924188-fa8b-4ed1-bfa6-fe27dbd65941\logo.png 30-01 15:54:27.128|DEBUG|HttpFileCache:Returning https://steamcdn-a.akamaihd.net/steam/apps/225140/page.bg.jpg from file cache C:\Games\Playnite\cache\images\5f8b245d2092754c9b895c54bf3c7301.jpg. 30-01 15:54:27.128|DEBUG|HttpFileCache:Returning https://steamcdn-a.akamaihd.net/steam/apps/225140/page.bg.jpg from file cache C:\Games\Playnite\cache\images\5f8b245d2092754c9b895c54bf3c7301.jpg. 30-01 15:54:27.207|WARN|PluginDatabaseObject3:SuccessStory - VerifToAddOrShow() find no action for Steam
30-01 15:54:31.310|WARN|PluginDatabaseObject3:SuccessStory - VerifToAddOrShow() find no action for Steam 30-01 15:54:35.227|WARN|ImageSourceManager:Image not found in database: C:\Games\Playnite\ExtraMetadata\games\590f16a7-2264-4879-be0b-4b3b4a30b78a\logo.png 30-01 15:54:35.236|DEBUG|HttpFileCache:Returning https://steamcdn-a.akamaihd.net/steam/apps/489830/page.bg.jpg from file cache C:\Games\Playnite\cache\images\27b191754d09539888a28a20cdd1cca4.jpg. 30-01 15:54:35.236|DEBUG|HttpFileCache:Returning https://steamcdn-a.akamaihd.net/steam/apps/489830/page.bg.jpg from file cache C:\Games\Playnite\cache\images\27b191754d09539888a28a20cdd1cca4.jpg. 30-01 15:54:38.159|WARN|PluginDatabaseObject3:SuccessStory - VerifToAddOrShow() find no action for Steam
30-01 15:54:41.721|WARN|ImageSourceManager:Image not found in database: C:\Games\Playnite\ExtraMetadata\games\6cb7ed25-852f-4238-8b4b-b414aa053947\logo.png
30-01 15:54:41.770|WARN|PluginDatabaseObject3:SuccessStory - VerifToAddOrShow() find no action for Steam 30-01 15:54:44.572|WARN|PluginDatabaseObject3:SuccessStory - VerifToAddOrShow() find no action for Steam`
I took a quick look at the code. This warning message seems to be written here:
https://github.com/Lacro59/playnite-successstory-plugin/blob/4f30d9e2b95f67309e9ed1737111c4eba06e802b/Services/SuccessStoryDatabase.cs#L770
The warning message in the log mentions "Steam" with uppercase but the code in this method seems to check for GameSourceName as "steam" with lowercase:
https://github.com/Lacro59/playnite-successstory-plugin/blob/4f30d9e2b95f67309e9ed1737111c4eba06e802b/Services/SuccessStoryDatabase.cs#L583
Perhaps "Steam" is being passed as a parameter to this method with uppercase incorrectly? Or maybe it's not being transformed to lowercase in the method? A recent change in the line 583 above removed .ToLower() from the comparison, but it seems to be present for other game sources.

I can confirm that Steam achievements are broken as of 1.4(.1).
Did a fresh install (alongside magpie 0.8.4 theme) and GOG achievements seem to show up, but not any Steam ones.
31-01 12:21:35.899|WARN|PluginDatabaseObject`3:SuccessStory - VerifToAddOrShow() find no action for Steam
31-01 12:21:35.908|ERROR|Common:CommonPluginsShared [H:\GitHub\Lacro59\playnite-successstory-plugin\playnite-plugincommon\CommonPluginsShared\IntegrationUI.cs 521] - Error in AddResources(Sc_HasData)
System.ArgumentException: Item has already been added. Key in dictionary: 'Sc_HasData'  Key being added: 'Sc_HasData'
   at System.Collections.Hashtable.Insert(Object key, Object nvalue, Boolean add)
   at System.Collections.Hashtable.Add(Object key, Object value)
   at System.Windows.ResourceDictionary.AddWithoutLock(Object key, Object value)
   at System.Windows.ResourceDictionary.Add(Object key, Object value)
   at CommonPluginsShared.IntegrationUI.AddResources(List`1 ResourcesList) in H:\GitHub\Lacro59\playnite-successstory-plugin\playnite-plugincommon\CommonPluginsShared\IntegrationUI.cs:line 521
31-01 12:21:35.908|WARN|PluginDatabaseObject`3:SuccessStory - VerifToAddOrShow() find no action for Steam
31-01 12:22:12.285|WARN|PluginDatabaseObject`3:SuccessStory - VerifToAddOrShow() find no action for Steam
31-01 12:22:20.885|WARN|PluginDatabaseObject`3:SuccessStory - VerifToAddOrShow() find no action for Steam
31-01 12:22:21.814|WARN|PluginDatabaseObject`3:SuccessStory - VerifToAddOrShow() find no action for Steam

I can confirm that Steam achievements are broken as of 1.4(.1).
Did a fresh install (alongside magpie 0.8.4 theme) and GOG achievements seem to show up, but not any Steam ones.
31-01 12:21:35.899|WARN|PluginDatabaseObject`3:SuccessStory - VerifToAddOrShow() find no action for Steam
31-01 12:21:35.908|ERROR|Common:CommonPluginsShared [H:\GitHub\Lacro59\playnite-successstory-plugin\playnite-plugincommon\CommonPluginsShared\IntegrationUI.cs 521] - Error in AddResources(Sc_HasData)
System.ArgumentException: Item has already been added. Key in dictionary: 'Sc_HasData'  Key being added: 'Sc_HasData'
   at System.Collections.Hashtable.Insert(Object key, Object nvalue, Boolean add)
   at System.Collections.Hashtable.Add(Object key, Object value)
   at System.Windows.ResourceDictionary.AddWithoutLock(Object key, Object value)
   at System.Windows.ResourceDictionary.Add(Object key, Object value)
   at CommonPluginsShared.IntegrationUI.AddResources(List`1 ResourcesList) in H:\GitHub\Lacro59\playnite-successstory-plugin\playnite-plugincommon\CommonPluginsShared\IntegrationUI.cs:line 521
31-01 12:21:35.908|WARN|PluginDatabaseObject`3:SuccessStory - VerifToAddOrShow() find no action for Steam
31-01 12:22:12.285|WARN|PluginDatabaseObject`3:SuccessStory - VerifToAddOrShow() find no action for Steam
31-01 12:22:20.885|WARN|PluginDatabaseObject`3:SuccessStory - VerifToAddOrShow() find no action for Steam
31-01 12:22:21.814|WARN|PluginDatabaseObject`3:SuccessStory - VerifToAddOrShow() find no action for Steam


Add title to image

It would be nice to be able to add the title of the image somewhere on the image.
Choosing where the title would go could be good too.

I second this. Program works well and images of /r/earthporn, for example, are great but you want to know what you are looking at.
There could be an option for example to show name of the file on the image itself (somewhere)...
Example filenames:
Letchworth_State_Park_NYoc_4032x_3024.png
Napali_Coast_Kauai_USA_OC3024x4032.png
Psuedocode
not (string) "OC[resolution].png" at the end
&&
replace underscore with spaces
example output on image:
Letchworth State Park NY
Napali Coast Kauai USA
Thanks!

This is a really good idea! But I should manipulate the image to do that, and decide where to put the text, how big, what color ecc... It might be a problem, it might cover things
What if instead I put it somewhere in the Tray menu? That would be easy to access and also tell you what you're looking at!

It could be good on the tray menu, but I tend to highlight the desktop to try and see the image.
Adding it to the image shouldn't be too hard, at least with ImageMagick, and there could be a dialogue to decide colour and a drop down for placement.
For what its worth, I have a bash script which does it with ImageMagick, the title addition is at the end.
I find it doesn't tend to cause too many problems in visibility.

I will consider what you propose, in any case with the last commit I added the title of the current wallpaper in the system tray, so it will be included in the next release

Discussion on stackoverflow mention ImageMagick program.
Discussion here has a script as well (for inspiration :) https://legacy.imagemagick.org/discourse-server/viewtopic.php?t=28886
This discussion on stackoverflow shows some pictures with filenames embedded: https://stackoverflow.com/questions/4106200/overlaying-an-images-filename-using-imagemagick-or-similar just to give you some ideas.
I think once you get the code working to have the file name on there; then
you can offer users the option to check/uncheck the option
if checked; then
set color, font desired
I personally would have /r/earthporn pics pulled into a directory; then, run a script to permanently embed location... that way, I could run the folder as screensaver on my android TV and I know what I am looking at : )


sortable doesn't work when clone is true

hi all, in my scenario i would be able to sort items within a column A and clone items from column A to column B but if i set clone: true sortable doesn't work. With clone set to false it looks that item is moved so the user experience is not the best.
Do you have an hint about how solve it?
Thanks in advance

@torreant this is by design the sorting is disabled on cloned items. we can take this as an enhancement request.


fix: use IncSearch for default results matching highlight

This is a small recommendation for the default highlight group for the GrugFarResultsMatch group. Currently it's set to @diff.delta, but IncSearch seems like a more semantically relevant highlight group.
Here is from :h IncSearch:

'incsearch' highlighting; also used for the text replaced with ":s///c".

Let me know what you think. I'm also okay if you disagree with this change and say it's up to colorschemes to make it look how they want.

Looks like there are changes to the unit tests which makes sense with the change in the attributes. I'll hold off revising them to see if this change is desired anyway 😄 If so, is there a good, disciplined way to regenerate the attributes it's testing against?

Am not opposed, but what about Search or Substitute instead?
Search		Last search pattern highlighting (see 'hlsearch').
		        Also used for similar items that need to stand out.

Substitute	|:substitute| replacement text highlighting.

actually there is a problem. If do a grug-far search, then use vim search within the grug-far buffer the 2 highlights become indistinguishable. I think there has to be something that would be different...

Yeah, I think that's the reason I went with @diff.delta originally

I am open with using a different, more appropriate one though if it's a different color from regular search

Okie doke! I definitely wasn't thinking about doing search outside of the built in search of grug. Totally fine! I'll go ahead and close this out!

kk!


Pre-encoded text responses are encoded again (unnecessarily)

The server automatically gzips outgoing response data streams if the response content type is text and the client will accept it. If the response data is already encoded, this results in the server encoding the data again.
This behavior should be avoided because it is unlikely that there would be any benefit to gzipping a data stream already compressed with gzip or any other compression method.
Expected behavior: a response containing a Content-Encoding header with a value other than identity should be served without gzipping the content.
Actual behavior: The response content is always gzipped if the Content-Type is text/* (and the gzipUsage setting is DEFAULT or ALWAYS).
I encountered this in org.nanohttpd:nanohttpd:2.3.1, but the issue exists in the latest version too, as far as I can tell from examining the current code at Response.java.
The resolution I recommend is to modify the useGzipWhenAccepted method to return false if the response contains a Content-Encoding header whose value is anything other than identity, regardless of what the gzipUsage setting is. I'd be glad to submit a pull request if this resolution is acceptable.

If you want to contribute this yourself, I'll let you know when to start. Currently we're in the middle of a very long freeze and I might find myself having to code most of 3.0.0 again and from scratch.


Fixed Beta Install command

The Beta install command was missing the -beta flag

Thank you, I was gone yesterday sorry I didn't update it sooner lol


Announcement time field

Describe the bug
There is something wonky with the time field and some announcements. See the "Swim Meet/Pool Closure" event on April 29.
.

It looks like for the swim meet and a few other events, a date was given in the time field. For example, the string "Monday May 1" was given as the time for "Annual Plant Sale!". We may need to clarify each field's role in the submission form.


logback unable to send logs to logstash, using tcp

cannot use LogstashTcpSocketAppender to send logs to logstash, while logstash started already.
Here is my configuration:
logback.xml
<?xml version="1.0" encoding="UTF-8"?>
<configuration scan="true"  debug="true">
    <!-- ch.qos.logback.core.ConsoleAppender 表示控制台输出 -->
    <appender name="stdout" class="ch.qos.logback.core.ConsoleAppender">
        <layout class="ch.qos.logback.classic.PatternLayout">
            <pattern>%d{yyyy-MM-dd HH:mm:ss.SSS} [%thread] %-5level %logger{50} - %msg%n</pattern>
        </layout>
    </appender>

    <!-- Logstash配置 -->
    <appender name="stash" class="net.logstash.logback.appender.LogstashTcpSocketAppender">
        <destination><IP_ADDRESS>:4560</destination>
        <encoder class="net.logstash.logback.encoder.LogstashEncoder">
            <customFields>{"appname":"Spring"}</customFields>
        </encoder>
    </appender>
    <logger name="javatest.loggertest" level="info">
    </logger>
    <logger name="javatest.loggertest.LoggerTestMain" level="info" additivity="false">
        <appender-ref ref="stdout"/>
        <appender-ref ref="stash"/>
    </logger>
    <root level="debug">
        <appender-ref ref="stdout" />
        <appender-ref ref="stash"/>
    </root>
</configuration>

Do you receive any errors?
Is logstash listening on <IP_ADDRESS>:4560 ?
Can you provide your logstash.conf where you have configured the tcp input with the json_lines codec?
Can you try a test using a simple netcat receiver (instead of logstash)... For example nc -lk 4560 ?

Do you receive any errors?
Is logstash listening on <IP_ADDRESS>:4560 ?
Can you provide your logstash.conf where you have configured the tcp input with the json_lines codec?
Can you try a test using a simple netcat receiver (instead of logstash)... For example nc -lk 4560 ?

OK, I checked the console output and no errors found, it seems connection established, while logstash started and cannot receive any message.
11:19:56,670 |-INFO in net.logstash.logback.appender.LogstashTcpSocketAppender[stash] - Log destination <IP_ADDRESS>:4560: connection established.
I tried nc <IP_ADDRESS> 4560 < tcp.log with some lines in tcp.log, logstash can receive and output to the console, so I think the problem should refer to logback,
I tried your suggestion nc -lk 4560, stopped logstash and started the project, could not receive any msgs.
logstash.conf:
input {
  tcp {
    host => "<IP_ADDRESS>"
    port => 4560
    codec => json_lines
  }
}

output {
  file {
    path => "/home/adam/test.txt"
  }
  stdout {
  }
}

I've justed noticed the TCP appenders are asynchronous ..you'll need to cleanly shut down logback when your application exits.
and 'cleanly shutdown' makes logs sended to logstash. O(∩_∩)O


Feature:The fonr size of the right-click menu may be smaller.

What feature or improvement do you think would benefit Files?
The font in the current right-click menu is a little large, so that it may not be possible to display the full right-click menu at once, and a scroll bar will appear.
Like this,

this is my right-click effect on my laptop desktop.

The font is not so large, and the right-click menu can be fully displayed without scrolling the mouse, which may be better.
Requirements

reduce the font size of the right-click menu(or add a right-click menu font size customization function in the settings) so that it can be displayed completely

Files Version
<IP_ADDRESS>
Windows Version
10.0.22631.2506
Comments
No response

Thank you for your feedback, we're tracking this feedback in #9506.


move and refactor code from mmm-util-property into mmm-property

move and refactor code from mmm-util-property into mmm-property

Done.
mmm-util-property not yet deleted.

Completed.

See https://github.com/m-m-m/util/issues/163


Fixed issues with installation via Carthage

Updated FRAMEWORK_SEARCH_PATHS in Carthage.xcconfig for different SDKs. Had to point to the exact folder for each SDK because project was not getting build when using the --use-xcframeworks flag with Carthage.
Ran carthage update hence why the Cartfile.resolved got updated.
Updated deployment target for iPhoneOS and tvOS to 12.0

Updated deployment target for iPhoneOS and tvOS to 12.0

Can't do that.

Updated deployment target for iPhoneOS and tvOS to 12.0

Can't do that.

Yep. I did a couple of weird things here that's why I closed it entirely.


helm: Consolidate IPSec and Wireguard encryption options

Suggested in https://github.com/cilium/cilium/pull/15383#discussion_r600323895.

@brb taking care of it. I submitted a PR, let me know if this makes sense. Thanks!


Dashboard does not show in browser

I am new to all of Kubernetes, but quite versed in Linux in general. So today for testing purposes I installed ubuntu 20.10 server 64-bit on 2 raspberry pis and installed microk8s on both. All goes well, I can join the leaf to the master.
What does not work is bringing up the dashboard. The command 'microk8s dashboard-proxy' tells me, the dashboard will be available at https://<IP_ADDRESS>:10443 and gives me a token. The browser says: '<IP_ADDRESS> has denied the connection'. I did some reading, but can't get it solved. Help is much appreciated.
inspection-report-20201101_143714.tar.gz

Hi @fethomm
The microk8s dashboard-proxy command is run on one of the nodes and the browser is also running on the same node? I want to make sure that the localhost <IP_ADDRESS> is the same for both the microk8s dashboard-proxy and the browser.

The microk8s dashboard-proxy command is run on one of the nodes, but since it's headless, the browser runs on a workstation in the same network.

The microk8s dashboard-proxy should be exposing the dashboard on all interfaces [1]. You should be able to reach the dashboard if you used the IP of the node where the microk8s dashboard-proxy was called.
[1] https://github.com/ubuntu/microk8s/blob/master/scripts/wrappers/dashboard-proxy.py#L64

Thanks, that works. At least in Firefox after agreeing to an exception, which Chrome for security reasons does not grant.


In 1856, the WGB Company Home has a Destination City and Home City, and wrong Revenue Center selected

When Starting the WGB Company, after the Home Tile is placed, when trying to place the Base Token, it was choosing the Destination City Revenue Center (where a token cannot be placed). This prevents the Token from being laid.

When trying to find a Revenue Center to place a Home Token:

Do not allow a Revenue Center of Type 'Destination City' from being selected for placing a Token Onto
Once you have a Valid Revenue Center, stop checking other Revenue Centers on the Tile.


[Event-Change Request] Table 81 - Gen. Journal Line - OnBeforeBlankJobNo

Dear ALAppExtensions team,
through this issue we would like to request a change of event publisher OnBeforeBlankJobNo in Table 81 Gen. Journal Line from:
[IntegrationEvent(false, false)]
local procedure OnBeforeBlankJobNo(GenJournalLine: Record "Gen. Journal Line"; CurrentFieldNo: Integer; var IsHandled: Boolean)
begin
end;

to:
[IntegrationEvent(false, false)]
local procedure OnBeforeBlankJobNo(var GenJournalLine: Record "Gen. Journal Line"; CurrentFieldNo: Integer; var IsHandled: Boolean)
begin
end;

making the parameter GenJournalLine a var parameter.
Thanks in advance!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 46009.


APM/crumbs support for hive

Description
hive seems to be super popular https://pub.dev/packages/hive
https://github.com/getsentry/sentry-dart/issues/1203 would help but would not give access to the CRUD statements, richer span context, etc.
https://docs.flutter.dev/packages-and-plugins/happy-paths/recommended#background-processing-

Reopened for crumbs


Typescript suggestion for object members and keyof not being shown

VSCode Version: 1.12-insiders
OS Version: Windows 10 x64
TS version: 2.3.0-dev.20170425

Steps to Reproduce:

Take the example below

The problem tab is showing the proper values for that parameter, but it's not in the suggestions list. and weird enough, it's getting even text from node_modules typings but not for the string
I already have
    "editor.quickSuggestions": {
      "other": true,
      "comments": false,
      "strings": true
    }
    "editor.wordBasedSuggestions": false

the same issue happens with "deep" objects, like when using sift.js typings like so:

(here it should be showing either the members of the array of objects there 'state' | '$eq' | '$ne' etc)
but after writing a improper field name, it shows an error

but I guess it's a problem with TS 2.3 dev itself, not with VSCode. it clearly says it's "boolean" in the docs, but it's accepting a string without complaining (even with strict: true)

Yes, these look like TypeScript issues. I've opened https://github.com/Microsoft/TypeScript/issues/15412 to track the first problem you describe.
For the second one with sift.js, could you please put together a minimal code example that demonstrates this errors in typescript 2.3.1 and file a new issues against TypeScript: https://github.com/Microsoft/TypeScript/issues/new
Thanks

created https://github.com/pocesar/ts-issue-25363 will post a new one there


[JavaScript] add missing withContent field to MessageListOptions

Helps with #942 but to fix we need to update the spec and regenerate the openapi client code.

#945 will update the openapi spec, which will complete the work needed for #942


Is there a way to suppress Console logging?

I have a working example of transcoding mic input in real time, but see lots of whisper logging in the console that I would rather not see, like this:
whisper_init_state: kv self size  =    5.25 MB
whisper_init_state: kv cross size =   17.58 MB
whisper_init_state: loading Core ML model from 'ggml-base-encoder.mlmodelc'
whisper_init_state: first run on a device may take a while ...
whisper_init_state: Core ML model loaded
Is there some way to suppress this logging?

Those logs are written by whisper.cpp library: https://github.com/ggerganov/whisper.cpp and currently there is no way of suppressing them.
I recommend you opening an issue there and if some config / option is added, will be added to this library as well.

Opened this issue: https://github.com/ggerganov/whisper.cpp/issues/1448 My pinvoke-fu is pretty weak, but I'm thinking something like whisper_set_log_callback(nil), or if there is some way to set the callback to a .NET method, we could intercept whisper.cpp output to parse & debug, but not show it unless we wanted.

Logging is now suppresed by default, you can register some log handler to the LogProvider if you want to show that logs or use them in your app.

is that true? I just downloaded it today and I get a ton of logs that I came looking to stop


How to retweet/like a given tweet

Hi,
I have the problem that I could not find a way to retweet/like a tweet in the v1 API. Is there any method for that?
Greetings
Michael

You can profile a full link to a tweet and set it into attachment_url property.
const linkToTweet = 'https://twitter.com/andypiper/status/903615884664725505';

await client.v1.tweet('Wow, Andy goes right.', { attachment_url: linkToTweet });

@alkihis Thank you so much 🙏


[Event Request] New publisher in Codeunit 90   OnBeforeUpdatePostingDescriptionOnInvoice

In Codeunit 80 we have this Event

We should have the same for Codeunit 90. Here the Event is missing.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Chi-Squared distribution doctests

#58 - doctests for chi-squared distribution
@avhz I have also implemented following functions along the way. Please have a look.
inv_cdf
entropy (based on this : Properties > Entropy)

Thanks so much!


Requests for dataset

Hello! I've filled out the Google form requesting the dataset. I'm looking forward to your reply. Thanks!!!
And my email address is<EMAIL_ADDRESS>

I have already filled out the Google form, yet I haven't received a reply from the author so far. I'm writing to inquire whether you could be so kind as to send me the download link of the dataset. I'd like to assure you that I will only use this dataset for academic purposes.
My student email address is<EMAIL_ADDRESS>Thank you very much for your help!


[Misc]Minor Changes about Worker

torch.cuda.synchronize() has been executed in the profile_run method. There is no need to repeat it externally.

cc @WoosukKwon to confirm


Update copyright year

Just downloaded the app.. My main gripe with caffine was whenever I right click it would stay highlighted - but love the retina icons too..
Anyway - noticed immediately that the about said 2014

By the way I have to recommend the hub command line tool and atom text editor built by the GitHub team after just doing this PR..
Alias 'hub' as 'git' and this was the full process I just followed:
git clone newmarcel/KeepingYouAwake
cd KeepingYouAwake/
atom .
- - [replace all in files]
git fork
git commit -m -a 'Updated copyright year'
git push jmurphyau
git pull-request
- - [fill in vim - line 1 is the title]


remove search for "/"

https://github.com/dantrim/parquet-writer/blob/4dee2d29845c019fc89e5de77e6aeedbc0c7f49e/src/cpp/parquet_writer.cpp#L604
the builder map no longer builds field paths with / in names, remove all references to this

addressed in 0d7f7a58fac311e7774dba7f4d8e4f52e1de7112


Create challenges-05.test.js

got most questions in. #6 is either really simple or beyond me tonight

Challenges 1-5 pass!


Syntax sugar for shared context/examples

I did an issue search and didn't find anything, so my apologies if this idea is old.
When writing shard examples and shared context like:
it_behaves_like "json api" do
  include_context "logged in user"
end

The normal order of context -> specification is reversed. I'd much prefer a syntax that maintains this flow:
when "user is logged in" { it_behaves_like "json api" }

I'm not familiar enough with the core to know how much work this would be, but it certainly seems possible.
Thoughts?

when is a Ruby keyword, so we couldn't use that. Even if we could, I don't want to add any more syntax to RSpec than we already have if there is already a good way to solve the problem:
shared_examples_for "json api" do |context_name|
  include_context context_name
  # ...
end

context "when user is logged in" do
  # ...
end

it_behaves_like "json api", "when user is logged in"

Actually I might be open to this:
with_context "user is logged in" do
  it_behaves_like "json api"
end

Need to think about it a bit more and heading out right now but will follow up later.

Ah true. Yeah the wording isn't all that important, I'm just hoping to keep the context -> example ordering. with_context would be great.

FYI - started to look at this and realized you can already do this using existing APIs:
RSpec.configure do |c|
  c.alias_it_should_behave_like_to :with_context
end

I'm going to leave this open and think about whether I want to include this in rspec-core out of the box, but you don't need to wait for my decision :)

Excellent. It never occurred to me that I could nest shared_examples.
I'll note that since this work around is leveraging a shared_example instead of shared_context to support this, existing contexts will have to be migrated to shared_examples - which is little dissonant. Also, when using the work around, I lose the flexibility to specify context with metadata. Finally, without the core being extended, I'm guessing passing a block with examples isn't possible:
with_context "authenticated user" do
  describe "controller" do
    # possible?
  end
end

I think keeping a consistent setup order and consistent block handling is a legit concern for rspec. But I get what you're saying about the desire to avoid bloat. Just curious, is this a difficult thing to add with the current architecture? Or is the issue mainly bloat?
I'll mess with this work around and see how it works out. Thanks a bunch for looking into this! And thanks in general for rspec, it's a vital part of my work flow.

shared_examples and shared_context are aliases for the same method, so you should use the name that makes most sense to you.
Yes, you can do anything in the block passed to with_context that you can do in any context.
As for "is this a difficult thing to add with the current architecture", all I'd be adding is the one line in my previous comment. So, no.

Awesome. Then I'm excited to start using this today. Thanks!

Cool. One thing to keep in mind - if you care about the output as much as I do, try doing this two ways:
with_context "user is logged in" do
  it_behaves_like "json api"
end

# and

with_context "user is logged in" do
  include_examples "json api"
end

If you use the documentation formatter you'll see two different things.

I should add that I thought they had branching features because of their separate handling on the new relish site. I had no idea they were essentially aliases for the same internals. Very interesting.


Ecosystem Link Bugs (undefined and typo in external URLs)

Currently, if you click an issue from https://opensource.microsoft.com/ecosystem/, it goes to something like this:
https://github.com/org/repo/issue/276

there's a small typo there - it should be like this, with an s on issues:
https://github.com/org/repo/issues/276

Additionally, some entries like this one:

Go to a url with undefined in them:
https://github.com/undefined/stevenbrix/xamlx-winui/pull/2

Removing that undefined goes to the correct link.
The only thing I've noticed there is that these ones have an additional leading / where others that work do not.

Thank you! A fix has been submitted to the backend API server and will roll out now.
Interestingly, pull requests are at pull/:id and the list at pulls/, while issues are at issues/:number and the list at issues/.


Optionally should backtrack when failing

Like OneOf, Optionally should be backtracking on failure, since the wrapped Parser may consume some of the input before it fails.
I've also updated the print to throw any wrapped print failures, since if a value is provided, the wrapped printer shouldn't fail if it's valid.

Hi @randomeizer, I'm curious why you think Optionally should backtrack. With the changes in #108 we decided that OneOf should be the sole place for backtracking.
One argument I could see for backtracking in Optionally is because technically it could be rewritten as a OneOf mapping into Optional:
Optionally { p }

// vs

OneOf {
  p.map(Optional.some)
  Always(nil)
}

But I'm curious if you came across a concrete example of where it would be expected.

I've created as separate PR at #149 against main which just updates the parse method to backtrack, adds the unit test, and updates the documentation. You'll probably want to review the text. I'll close this one down.


Lock the numeric keypad to always and forever use numbers

Summary of the new feature/enhancement
The numeric keypad has two states, use like numbers or use like idk navigation and idk what else. I wanna just use the numbers and I want it to never ever again change to anything else than numbers. Is very annoying it keeps changing.
Proposed technical implementation details (optional)
Just lock it in the numeric keypad state and prevent any state switch, forever. Thank you!

I would like to expand and give some reasons for this:

The very obvious one, most people expect that thing to just enter numbers on the screen, if you press and no number is seen and the cursor moves weird then you are into problems.

The other that isnt that obvious, people that don't speak English does some keyboard gymnastics with ALT+NUMBERS to type some characters that aren't currently on the keyboard. Like ñ @ etc.....

Developers do something similar to the second point, for example Spanish keyboard has the special keys > < to the left of the Z but English keyboards don't have it.  So if you use an english keyboard(like most imported[sic] laptops this days) with a Spanish setup you have to resort to stuff like ALT+62 and ALT+60. Dont make me tell you what you have to do to type regular expressions like ^ [] { } etc......

This is a new reason for me, my keyboard just broke, so the 7 key(Numeric keypad one) is not working properly, whenever I hit it, it drives me crazy because I never know if I turned ON Num Lock on or not as the key misbehave! And thats why I entered this issue..... lol

It looks like there is no Windows way to lock the num-lock key. We can, however (at least I think this should work):

get the current NumLock state and if needed toggle it. There is even MSDN example of exactly this.
use keyboard hook to intercept and ignore all NumLock key presses.
unfortunately, if PT are not running as admin, we won't be able to intercept NumLock keys in elevated apps. We should probably test the NumLock state from time to time and restore it if it was changed.

This is a great thing for a first-time contribution. A new, separate, small powertoy.

Well, here is a temporal AHK script courtesy of @boredofnames

#NoEnv
#InstallKeybdHook
SetNumLockState, AlwaysOn

unfortunately, if PT are not running as admin, we won't be able to intercept NumLock keys in elevated apps.

If PT utilized the uiAccess="true" attribute in its manifest (and met all of the requirements for that), it would be possible for PT to intercept keystrokes destined for elevated apps when PT itself is not running as admin. The requirements and effects of this attribute are documented under Security Considerations for Assistive Technologies.

If the key remapper supported mapping a key to nothing, this functionality could be built into the existing module.
I suppose it matters whether the goal is to make the numpad always output numbers or to stop Num Lock from actuating.
Right now if you attempt to select none, there is a popup stating "Shortcuts must contain an action key". If instead selecting none just intercepted the keypress, I think that'd be a clean solution.
I haven't contributed to open source before, but I've looked at the contributor's guide and I'd be interested in trying to implement that.

Can't you just break the NumLock key out of your keyboard?

Well, here is a temporal AHK script courtesy of @boredofnames

#NoEnv
#InstallKeybdHook
SetNumLockState, AlwaysOn

Did this also prevent Numkock from state change while booting up a VM with Windows 10?
Can the user enable NumLock within the VM when the state change is blocked on the host?

Since my suggestion is very similar to this, I'd like to add here that I would very much like if we could do the same thing with caps lock.
It's such a waste of keys, I don't know a single person that uses them for their intended purpose, either people just never press them, or, like me, they have remapped both keys to other keys as to not waste two keys that are located (especially caps lock) at areas you often use on the keyboard.

Does it work? Can someone teach me how I always keep the num lock key enabled?

@udobehm yes it works, download autohotkey https://www.autohotkey.com/
then create a file named "numlock.ahk" with content
#NoEnv
#InstallKeybdHook
SetNumLockState, AlwaysOn

right click the file and choose "compile script", it will create an exe file, that you can run and also add to windows start up

Thanks - will try this out

Does this still need help implementing? I'd like to take this up as it is marked as Good First issue.

@what's your idea/solution?


Possible bug in strip_annotated function

what it should do?
def strip_annotated(value: N) -> N:
    if HAS_ANNOTATED and isinstance(value, BaseNormType) and value.origin == typing.Annotated:
        return strip_annotated(value)
    return value

Yes, this is a bug.
The code should looks like
def strip_annotated(value: N) -> N:
    if HAS_ANNOTATED and isinstance(value, BaseNormType) and value.origin == typing.Annotated:
        return strip_annotated(value.args[0])
    return value

This function was removed in 5f0ffff6cf552c1ac63fcae5c939736baa7ea71a.


fix: useForwardProps for vue-sonner props

useForwardProps is used in Component Wrapper of vue-sonner to prevent casting undefined boolean props as false

@xiaoluoboding sry for tag
toast actions will freeze when you hit fast on the Undo button
should I have to create an issue in vue-sonner?
https://github.com/radix-vue/shadcn-vue/assets/17789047/71133b00-edd1-4359-b75a-bd5025482083

@xiaoluoboding sry for tag

Toast actions will freeze when you hit fast on the Undo button
Also last toast element doesn't have remove animation

should I have to create an issue in vue-sonner?
Vue.Sonner._.An.opinionated.toast.component.for.Vue.-.Google.Chrome.2024-01-23.12-06-29.mp4

OK, you can create an issue on the repo. I need some time to see what the problem is.


Title, icon and footer change

Changed the Menu title to Address Book to reflect app purpose.
Updated the menu icon from >  to ≡ for appropriate look.
Added a simple footer section with a copyright notice

Looks fine to me!


Implement resin.models.device.Device.move()

This function allows user to move a device to another application with the same device-type

@dfunckt, Could you please review this PR?

LGTM :+1:


Add MarkLocation blocks inside fold scopes.

This should make query location tracking easier for @jmeulemans' SQL branch.

Coverage decreased (-0.2%) to 94.81% when pulling abb266a07535844dcacc29e38f921bea57084225 on add_mark_locations_to_folds into 7f54185574b629a36d174d635dce43c157d2376f on master.

Coverage decreased (-0.2%) to 94.81% when pulling abb266a07535844dcacc29e38f921bea57084225 on add_mark_locations_to_folds into 7f54185574b629a36d174d635dce43c157d2376f on master.


Confusion around metrics

I've been looking at the 'postfix_showq_message_size_bytes_count' metric. I would like to be able to monitor the queue depth for postfix, and this seems to be the most appropriate metric, however, I am not sure if I am grasping what it is trying to express.
It doesn't seem like there is any metric that will count individual messages (that would be a great metric to have), so how exactly should I be using this metric to understand the queue depth with postfix? I've seen a grafana dashboard using this exporter, but that graph doesn't really correspond with the messages we are processing.
Any insight here would be really welcome!
thanks!

I'd guess you are looking for postfix_smtp_status_total to see how many  messages are on the queue?

That certainly sounds like a promising metric, but that isn't showing up for me. Is it possible I'm missing something in the configuration? Or is this a recently introduced metric?
So, what is postfix_showq_message_size_bytes_count actually expressing? I saw this issue: https://github.com/kumina/postfix_exporter/issues/15 but I'm still not entirely sure what this metric is expressing. Just want to make sure I'm understanding that.
Thanks!

That certainly sounds like a promising metric, but that isn't showing up for me. Is it possible I'm missing something in the configuration? Or is this a recently introduced metric?

I've installed the postifx_exporter on three systems (via a docker image and as binary) but I do not get this metric, to.

I don't have that metric either.

Any news on this? That metric is missin here as well ....


[UPDATE ALGORITHM] Leaf-Similar Tree

Description:
Please describe the changes you want to make to the existing algorithm and why they are necessary.
Leaf-Similar Tree imporve time and space complexity
Checklist:

[X] Contributor in GSSoC-ext
[X] Want to work on it

Affected Algorithm:
Mention which algorithm you want to update and the specific directory where you want to change.
Additional Information:
Any extra details or suggestions related to the update.
@pankaj-bind can i work on this/

what changes do you want to make?


Stabby functions don't work in string interpolation

Steps to reproduce
Store the following script as issue.vim
vim9script

class Foo
  def Cost(nums: list<number>): number
    # This is the same if nums->typename() is any other type
    return nums[0] * nums[1]
  enddef

  def ShowCost()
    var g = [4, 5]
    # echo $"Cost is: {this.Cost(g)}"
    echo $"Cost is: {g->this.Cost()}"
  enddef
endclass

var d = Foo.new()
d.ShowCost()

Then open Vim with
vim --clean -S issue.vim

Expected behaviour
Expected this to run the script as expected, since — I assume — string interpolation should evaluate what's within the { braces } as string. For some reason, this doesn't work with stabby function calls. I expected the stabby function call interpolation to be equivalent to the commented line above.
Expected
Cost is: 20

Got
Error detected while compiling command line..script /path/to/test.vim[17]..function <SNR>10_Foo.ShowCost:
line    3:
E1028: Compiling :def function failed
Press ENTER or type command to continue

Version of Vim
VIM - Vi IMproved 9.1 (2024 Jan 02, compiled Dec 20 2024 18:45:06) macOS version - arm64 Included patches: 1-950 Compiled by Homebrew
Environment
OS X
iTerm
xterm-256color-italic
zsh
Logs and stack traces
No response

The error only shows up when the function is a class method. When Cost() is a free function, it works:
vim9script

def Cost(nums: list<number>): number
    return nums[0] * nums[1]
enddef

class Foo
    def ShowCost()
        var g = [4, 5]
        echo $"Cost is: {g->Cost()}"
    enddef
endclass

var d = Foo.new()
d.ShowCost()


MM-51956 - Fix for: Calls popout generating duplicate notifications

Summary

The popout didn't have the Mattermost/version userAgent. The only way to set this (that works) is through the browserWindow.loadURL fn, but we don't use that for the popout, we use the window.open from the calls side. It's not possible to set the userAgent directly with window.open or in the windowOpenHandler return options. The best way I could come up with is to just reload the calls popout using the loadURL. Through trial and error I found a place to do that, which doesn't cause the screen to flicker or load and reload, luckily. We can do it even earlier in the process (e.g., the dom-ready event) but that returns an error (but, oddly, everything works and the userAgent does get correctly set). The did-frame-finish-load event is the earliest that doesn't return an error.

Ticket Link

https://mattermost.atlassian.net/browse/MM-51956

Checklist

[x] Added or updated unit tests (required for all new features)
[ ] Has UI changes
[ ] read and understood our Contributing Guidelines
[ ] completed Mattermost Contributor Agreement
[ ] executed npm run lint:js for proper code formatting

Release Note
Calls: Fix duplicate desktop notifications when calls popout is open

Thanks @cpoile, would it make sense to set the user agent on the widget window for consistency if anything?

Sure, I don't see why not (just in case, for the future). Done.


New table created with every new piece of data

For every new piece of data created, a new table is being created.  Please tell me what I'm doing wrong and provide an example of how to fix it.  Thanks!
class ListItem extends Component {
render() {
const { name, street, street2, city, state, zip } = this.props;
return (
  <Table
    data={[
      {
        name,
        street,
        street2,
        city,
        state,
        zip,
      },
    ]}
    itemsPerPage={4}
    pageButtonLimit={5}
    sortable={true}
    filterable={['Name', 'Street', 'Street2', 'City', 'State', 'Zip']}
    noDataText="No matching records found."
  />

);
}
}

it looks like you're rendering a new table for every ListItem...
Also, this is a question for StackOverflow


Client Authentication with WCF containers

Hello,
We added a certificate to the ingress controller, and when we browse to the ingress address it prompts for a cert which is desirable.
we have set up the app.config in our wpf application to provide the clientcertificate via the behaviorconfiguration, but we get ssl tunnel could not be created. Is there any docs about this situation? Any other thing we could do to improve this situation?

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

ID: 597ba957-9501-5919-73a0-6d47b1ca2c6d
Version Independent ID: 6345fa19-b3f2-ffd6-f526-908f845b51f6
Content: Use your own TLS certificates for ingress with Azure Kubernetes Service (AKS) cluster
Content Source: articles/aks/ingress-own-tls.md
Service: container-service
GitHub Login: @iainfoulds
Microsoft Alias: iainfou

@iainfoulds would you be able to comment further on this?

I think we can close this, my coworker implemented the solution found here: https://blogs.msdn.microsoft.com/morgan/2010/04/15/setting-up-wcf-with-a-load-balancer-using-ssl-in-the-middle/ and it seems to be working. We were stuck on it for about a week or so, so thanks anyways for the reply


Detox - Support tap() on Markers

Feature request
Calling .tap() on <AIRGoogleMapMarker> element doesn't tap at the correct x/y coordinate on the screen.
It's always ending up tapping at the top-left corner of the MapView, even though my marker is aligned at the centre of the MapView.

Why it is needed
I want to be able to tap a marker when covering my application with end-to-end tests.
Possible implementation
N/A
Code sample
Just passing a testID prop to the <Marker> element and try to use it to call .tap().
import MapView from 'react-native-maps';
// ...

render() {
    return (
        <MapView
            // all_the_props
        >
            <MapView.Marker
                key="some-key"
                identifier="some-identifier"
                coordinate={ { latitude: 51.15, longitude: -0.15} }
                testID="some-test-id"
            >
                <View style={ { backgroundColor: '#ff0000', height: 50, width: 50 } }>
                    <Text>Marker</Text>
                </View>
            </MapView.Marker>
        </MapView>
    );
}

See https://github.com/wix/Detox/blob/master/docs/APIRef.ActionsOnElement.md#tappoint

This still is an issue. I am not able to tap on a marker on the map. Any workaround for this?

@clemblanco Did you find a solution for this?

@clemblanco Did you find a solution for this?

Yes. I'll write a blog article about it. 😉

@clemblanco Did you find a solution for this?

Yes. I'll write a blog article about it. 😉

Great. Waiting for it. Need it to get my automated tests complete 😁👍

Disappointed that this was closed, would make things much easier if you could just tap a marker


fix(plugin-active-header-links): do not append hash when at top, close #693

I might prefer this design: do not automatically replace hash if the first header anchor is not above the screen (i.e. hasPassedCurrentAnchor for i = 0).

Suggest to modify the isTheFirstAnchorActive variable
https://github.com/vuepress/vuepress-next/pull/722/files#diff-d00f1d3bc787179f8ddda1119d6a8a982b4ff942d7ca9881cedb3e62f171ddaaR72-R73


Tracking issue for specification maintenance tasks

The purpose of this issue is to identify and track various maintenance tasks which have accumulated concerning specification organization and development.
Remaining tasks

[ ] Migrate special cases to use math directives: https://github.com/data-apis/array-api/issues/519
[ ] Ensure line length is handled better, so text is readable and reviewable on GitHub without it scrolling off to the right. Either use regular line length settings (80, 88, 100, or 120 char), or set up something like EditorConfig to ensure consistent line endings and spacing.
[x] Setup linting of Python files (black, pydocstyle) in CI.

Done for v2022

[x] Move special cases and other explanatory content to a dedicated "notes" section: https://github.com/data-apis/array-api/issues/518

[x] PR: gh-578

[x] Add a short sentence about whether the special cases are derived from C99, from some other standard, or are based on analysis or existing practice in array libraries. See https://github.com/data-apis/array-api/pull/514#issuecomment-1337650301.

[x] PR: gh-579

[x] Use black for automated code formatting and line wrapping (possibly using Git hooks).

[x] PR: gh-575

[x] Move universal preamble content (e.g., positional parameters, optional parameters, etc) to a dedicated design document.

[x] PR: gh-576

[x] Merge the content in the "branch cuts", "complex number ordering" and "complex numbers" (about equality mostly) design topic pages into a single "complex number support" page.

[x] PR: https://github.com/data-apis/array-api/pull/573

Thanks for summarizing this @kgryte. A few thoughts:

When doing this, let's try to ensure that large content refactors do not make it harder to use git blame, by using https://docs.github.com/en/repositories/working-with-files/using-files/viewing-a-file#ignore-commits-in-the-blame-view
Some of this may be a little time-consuming. To me the higher-prio ones, good to get into the v2022 spect, are "move special cases", "move universal preamble", and "use black"
After that, the "use math directives one", because it's also still a big diff, so makes later maintenance easier if it's done before branching
Linting in CI and EditorConfig are lower prio, they can be fiddly and don't impact the 2022 content or organization of it

The v2022 tasks are done. I reorganized the description so it's clear what is left, and removed the milestone.

As commented in https://github.com/data-apis/array-api/pull/689#discussion_r1449622314, we should investigate whether we can minimize type annotation duplication. Currently, we copy-paste the type annotation from the signature into the docstring. This increases the risk of drift, and preferably, we'd only need to write these annotations once.

Sphinx has functionality for that (autodoc_typehints), but like anything autodoc related it had some issues, especially with dealing with type aliases. Not sure if that has been improved recently, but I did try this when initially setting up the docs.

If I had to deal with this now, I'd dump Sphinx complete in favor of MkDocs / mkdocs-material.

Sphinx has functionality for that...I'd dump Sphinx completely...

Yeah, we should revisit once we have v2023 released.


Membership numbers should be modified by city population

To better understand the significance of meetup membership numbers and to make them more comparable among different cities, these should be weighted by a population value.

I did some initial exploration on available datasets. I used the "world.cities" dataset available in R and found a few problems:

city names were localized in the meetup data but not in the R dataset (eg, São Paulo vs Sao Paulo, Milano vs Milan)
country names could not be matched. Meetup uses a localized country name and a country code, the R data only uses a country name that did not always match the Meetup localized country name (eg, UK vs United Kingdom, US vs USA).
Longitude and Latitude only matched for 50% of the cities

I tried rounding longitude and latitude and then selecting the value that was the least different (ie, closest city in the R dataset to the one I had in the Meetup dataset). I was able to match all of the non-matches except for one (Cincinnati) because when rounded, the value in the R dataset went up and the Meetup number went down.
I think the key problem is the R dataset itself. Country codes would be a huge improvement for narrowing down possible matches. A range of acceptable latitudes and longitudes would also be nice, but if not then one could be computed.
Next steps here would be to:

[ ] Find a population dataset with country codes. It also needs to contain city names, latitude, longitude, and population. Ideally it would contain a latitude and longitude range and localized city names (but this is optional)
[ ] See what the match rate is using city name and country code then longitude and latitude.
[ ] Depending on the results from ^^, experiment to find the city in the dataset that is geographically closest to the one we are looking for
[ ] Determine a population "weight" metric that can be used to modify the members total. This should start out as a ratio (members/population) and then we can explore other possibilities depending on how useful this value is. I'm inclined to do round(log(members/population)) as an ordinal categorical variable.

Due to the size of these population data, my recommendation is to store them in GBQ and then use GBQ to do the matching. I think the population dataset should have additional columns if there are any missing that will allow for a quicker match with the meetup data. The matching should happen when the meetup API call is made for the "groups find". The additional step would be a query to GBQ to match the city record to get the population value.

Have you tried:
https://www.maxmind.com/en/free-world-cities-database.
It inlcudes:
Country , City, AccentCity ,Region, Population, Latitude, Longitude

Thanks @kathy0305 ! I saw that dataset but because I had to download it myself into something hadn't given it a try yet. Going to see if I can get into Google BigQuery because I am no longer a db admin (nor do I want to be). Thanks for the reminder though!! If you know a place where it is hosted already that would be awesome.


M3-895 Fix Select StackScript Storybook Component

Purpose
Fix the <SelectStackScriptPanel /> component in storybook
Notes

Should this even be a storybook component? It's rendering a child that's fetching for async data

Coverage decreased (-0.002%) to 26.41% when pulling d3aadb77374f366db7f860e9fcb02a78868df5a7 on martinmckenna:M3-895-select-stackscript-storybook into 708e1fe45631f7ade51def215e3886634f6231a0 on linode:develop.


deisctl should group mesos/k8s components

Currently each of the four mesos components are managed as standalone entries in deisctl, which is more verbose than needed.
$ deisctl stop mesos
● ▴ ■
■ ● ▴ Stopping Mesos...
▴ ■ ●
Marathon Framwork...
deis-mesos-marathon.service: inactive/dead
Mesos Slave...
deis-mesos-slave.service: inactive/dead
Mesos Master...
deis-mesos-master.service: inactive/dead
Zookeeper...
deis-zookeeper.service: inactive/dead
Done.

They are more of a unit IMHO, like the data plane or something, and should be treated as one logical group:
$ deisctl stop mesos
● ▴ ■
■ ● ▴ Stopping Mesos...
▴ ■ ●
Mesos with Marathon...
deis-mesos-marathon.service: inactive/dead
deis-mesos-slave.service: inactive/dead
deis-mesos-master.service: inactive/dead
deis-zookeeper.service: inactive/dead
Done.

Same goes for Kubernetes, it's (arguably) more verbose than necessary:
$ deisctl install k8s
● ▴ ■
■ ● ▴ Installing K8s...
▴ ■ ●
K8s API Server...
deis-kube-apiserver.service: loaded
K8s controller and scheduler ...
deis-kube-scheduler.service: loaded
deis-kube-controller-manager.service: loaded
K8s proxy and kubelet ...
deis-kube-proxy.service: loaded
deis-kube-kubelet.service: loaded
Done.

imho, the groupings should reflect membership in a particular plane.  So with mesos/marathon, for instance, zookeeper, mesos-master, and marathon can be one group (belonging to the control plane) and mesos slave, by itself, can be a second group belonging to the data plane.

(So we're not starting things across planes in a single group.)

:+1:


If you make breaking change then bump the major version!

you cannot go from 0.37.x to 0.40.x and completely rewrite the cookbook!!

Merging this will help https://github.com/bflad/chef-docker/pull/333

Merged as c5f2bbb13ac9bad59673db0603557e24e4b88d04

released as 0.40.2

Er, sorry, previous comments were for the wrong issue

@Sauraus @scalp42
Sorry for the breaking changes on a minor version... this warrants a bit of an explanation.
Initially I meant for the rewrite to become a 1.0.0... however, work stalled and it was sitting in the "develop" branch for a weeks and weeks. People were asking for these bits to get into master sooner than the ill-defined "later". Normally, this would warrant a major version bump, but because of its 0.x.x status, and the fact that Chef doesn't support -alpha and -beta semver semantics, incrementing minor to the next even 10 made the most sense.
From http://semver.org/
4- Major version zero (0.y.z) is for initial development. Anything may change at any time. The public API should not be considered stable
5 - "Version 1.0.0 defines the public API. The way in which the version number is incremented after this release is dependent on this public API and how it changes."
So, basically, the minor version bump was to avoid defining that public API.
-s

@someara I am familiar with semver however there is room for some critical thinking with standards, because in this case the change clearly blew up in everybody's face and there was no way anyone could have protected themselves from this, with any version constraint except by locking to 1 specific version. Just something to consider for the future. :)

Indeed. The general plan going forward is a continuous delivery model.
—
Sent from Mailbox
On Tue, Jul 14, 2015 at 2:56 PM, Antek Baranski<EMAIL_ADDRESS>wrote:

@someara I am familiar with semver however there is room for some critical thinking with standards, because in this case the change clearly blew up in everybody's face and there was no way anyone could have protected themselves from this, with any version constraint except by locking to 1 specific version. Just something to consider for the future. :)
Reply to this email directly or view it on GitHub:
https://github.com/bflad/chef-docker/issues/338#issuecomment-121341379


No event after applying resource

Sorry I'm using issues to get some help, there is no "discussion" tab enabled.
So I wanted to try things out and after installing the CRD, I apply the following :
apiVersion: couchbase.btburnett.com/v1beta1
kind: CouchbaseIndexSet
metadata:
  name: couchbase-indexset-acme
spec:
  cluster:
    clusterRef:
      name: couchbase-db-couchbase-cluster
      secretName: couchbase-secret-admin
  bucketName: acme
  indices:
    - name: object_type
      indexKey:
        - _class

Then I check with kubectl describe couchbaseindexset couchbase-indexset-acme and it shows no event. And nothing is happening on the couchbase side. The controller container says it is listening and nothing more.
Both the couchbase cluster and the indexset are in the same namespace.
How can I troubleshoot that ?

Did you also install the operator to the same namespace? If you're using the default operator deployment from the examples/docs, it will only monitor for CouchbaseIndexSet resources in the same namespace where the operator is deployed. This is designed for security and to allow separate versioning of the operator for each namespace.

Everything is in the same - default - namespace.

@JesusTheHun
I did some poking this morning, and it appears some required RBAC rights are missing from the deployment. There must have been something floating in my test cluster that was bypassing the requirement, when I applied to a new cluster it was immediately apparent.
I've updated the RBAC role and binding via PR #8, can you please give that a try and see if it works. Note that you may need to run kubectl delete rolebinding cb-index-operator-manager-rolebinding before the changes will fully apply.
If this doesn't resolve the issue, I'd love to see the logs from the db-index-operator-manager pod's manager container. Also, details from the status conditions on the CouchbaseIndexSet itself could be useful.

The issue is resolved. Thanks !
And you were correct, I had to delete the rolebinding before applying the CRD again.


Fix objects in null

dset had issues when encountering null in objects. This caused some issues with devtools not rendering the page correctly.
https://github.com/lukeed/dset/issues/36
Change Type (required)
Indicate the type of change your pull request is:

[x] patch
[ ] minor
[ ] major

huh, surprised we never ran across this before.


Fixes to Wrapper.get_config and Wrapper.from_config.

This PR aims at solving two light issues I identified with the tf.keras.layers.Wrapper class.
This has been tested on both TensorFlow 2.1.0 and 2.2.0-rc0 for Python 3.6.9, both installed using pip on a Linux 64-bits system.
1. Serialization through get_config.
Issue: The dict output by Wrapper.get_config() is designed to contain the config of the wrapped layer, in a format similar to that output by keras serialization utilities. However, the identifier of that layer (its class_name) is currently manually gathered within the get_config, which results in bugs such as  custom layers set for registration (using tf.keras.utils.register_keras_serializable) not being given their proper identifier.
Fix: I propose to replace the manual creation of the layer entry within the returned configuration by a direct call to serialize_keras_object.
Minimal example:
import tensorflow as tf

@tf.keras.utils.register_keras_serializable(package='test')
class CustomLayer(tf.keras.layers.Layer):
    def call(self, inputs):
        return inputs

wrapper = tf.keras.layers.Wrapper(CustomLayer())
config = wrapper.get_config()
tf.keras.layers.Wrapper.from_config(config)

When run before the fix, this fails on the last line, raising ValueError: Unknown layer: CustomLayer.
When run after the fix, this runs successfully.
2. Instantiation through from_config.
Issue: Contrary to what the source code suggests, Wrapper.from_config has side effects on the provided config dictionary; in short, it pops out its layer sub-dict.
Fix: Instead of calling config.copy(), I suggest using copy.deepcopy(config) before popping out the layer sub-dict of the (deep copy of the) input config dictionary.
Minimal example:
import tensorflow as tf

dense = tf.keras.layers.Dense(32)
wrapper = tf.keras.layers.Wrapper(dense)
config = wrapper.get_config()
assert 'layer' in config, 'before wrapper instantiation'
tf.keras.layers.Wrapper.from_config(config)
assert 'layer' in config, 'after wrapper instantiation'

When run before the fix, the second assertion fails due to the side effect.
When run after the fix, this runs successfully.

It seems that the test is failing wrt deepcopy:
https://source.cloud.google.com/results/invocations/2e5f1424-87f0-4826-93b2-d3936a6caa51/targets/%2F%2Ftensorflow%2Fpython%2Fkeras%2Flayers:wrappers_test/tests

@qlzh727 Thank you for the review.
I see what went wrong; in the Bidirectional wrapper, the (optional) backward_layer is being de-serialized within the overridden part of from_config, thus the config passed to the super method contains an instantiated layer, which deepcopy does not know how to handle.
I see two ways to handle this:

Replace the super().from_config call in Bidirectional.from_config with direct re-instantiation of the forward layer and constructor call. This way there is no secondary deepcopy call and the bug does not arise - but not calling the super method and duplicating some of its lines of code may feel improper.

Add a parse_config method to Wrapper that takes care of everything save for the deepcopy, and have from_config run the latter before delegating to the former. But this feels like a painful hack to avoid one bug...

I will push a commit implementing solution zero, but let me know if you would rather adopt another way of handling this.

I tested locally that all tests from wrapper_test.py pass with the last commit.
On the side, I realized custom layers used as backward layer were not properly serialized either; I thus also made a minor modification to Bidirectional.get_config.


Create service script for open-rc

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@opie4624 looks good but do you mind signing the latest CLA?

I signed it many moons ago when the PR was submitted, however I'm unable to sign it now:

Yeah when Salesforce acquired Slack the CLA changed over to them. If the prompt isn't showing, a fresh commit might do the trick.

Yep, the prompt shows. Clicking on it, takes me to the page I screenshot.
On Mon, Apr 29, 2024 at 1:52 PM Nate Brown @.***> wrote:

Yeah when Salesforce acquired Slack the CLA changed over to them. If the
prompt isn't showing, a fresh commit might do the trick.
—
Reply to this email directly, view it on GitHub
https://github.com/slackhq/nebula/pull/711#issuecomment-2083641798, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAABVY3F57FCTSVP7YXPXJ3Y72XJBAVCNFSM53B3Z5ZKU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TEMBYGM3DIMJXHE4A
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Andrew Kraut
Cell: +1<PHONE_NUMBER>
K9OPQ

Yes that's the old one. If you can make a new commit to this branch the correct prompt will appear, sorry about the run around here.