[test] fix fuzz tests that might crash on duplicate settings params

Fuzzing magnificently managed to construct a valid settings byte frame that triggers an assert in test utility code. The assert ASSERT(result.second) validates that inserting test settings param into an unordered set is successful. The fuzz test generated a settings frame with duplicate settings param and tripped the ASSERT.
I take this as a feat of fuzzing.
Fixes: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=21318
Testing: corpus entry added
Signed-off-by: Asra Ali<EMAIL_ADDRESS>

Actually: Probably this test utility should not hold settings parmas in an unordered set. nghttp2 should handle (with an invalid argument failure) duplicate settings ID, so there's no need to prevent testing that codepath.


403 Forbidden

when i use the Sample app to send steam to easy darwin,then 403 happend
I/RtspClient: RTSP/1.0 403 Forbidden
Connection Failed : Error configure stream, access denied

sry, i find this is easydarwin error


fix(deps): update all non-major

This PR contains the following updates:

Package
Type
Update
Change

regex
dependencies
patch
1.10.1 -> 1.10.2

ring
dependencies
patch
0.17.4 -> 0.17.5

thiserror
dependencies
patch
1.0.49 -> 1.0.50

tracing (source)
dependencies
patch
0.1.39 -> 0.1.40

Release Notes

rust-lang/regex (regex)
v1.10.2
Compare Source
===================
This is a new patch release that fixes a search regression where incorrect
matches could be reported.
Bug fixes:

BUG #1110:
Revert broadening of reverse suffix literal optimization introduced in 1.10.1.

dtolnay/thiserror (thiserror)
v1.0.50
Compare Source

Improve diagnostic when a #[source], #[from], or #[transparant] attribute refers to a type that has no std::error::Error impl (#258, thanks @de-vri-es)

tokio-rs/tracing (tracing)
v0.1.40: tracing 0.1.40
Compare Source
This release fixes a potential stack use-after-free in the
Instrument::into_inner method. Only uses of this method are affected by this
bug.
Fixed

Use mem::ManuallyDrop instead of mem::forget in Instrument::into_inner
(#2765)

Thanks to @cramertj and @manishearth for finding and fixing this issue!

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

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

This PR has been generated by Renovate Bot.

:tada: This PR is included in version 1.2.3 :tada:
The release is available on:

GitHub release
v1.2.3

Your semantic-release bot :package::rocket:


Feature request: Nord theme

CyberChef currently supports multiple themes:

Solarized Dark
Solarized Light
Geocities
Default

Could we add support for Nord theme palette? (the palette is described in https://www.nordtheme.com/)

Can i be assigned this issue?


ReFrame tutorial: missing -pthread in A Multithreaded “Hello, World!”

The initial Multithreaded “Hello, World!” most likely need a "-pthread" compiler flag. It is at least required for g++ 9.3.0.
Without it I get:
cat stage/generic/default/builtin/HelloThreadedTest/rfm_HelloThreadedTest_build.err
/scratch/cc0LAYZw.o:hello_threads.cpp:function std::thread::thread<void (&)(int), int&, void>(void (&)(int), int&): error: undefined reference to 'pthread_create'
collect2: error: ld returned 1 exit status

@akesandgren I don't know if this is a "bug." The tutorial examples are not entirely generic and need to be slightly adapted to each system. We're testing them continuously on Daint, but of course, this doesn't make them generic. Nonetheless, adding -pthread also in the examples might be harmless.

Adding -pthread would make it more (linux) generic since that's what a standard gcc requires, but it might require a comment that non-gcc (xlc, pgcc for instance) will have problem with it.

Actually, I think the Cray wrappers do add the -pthread for you to keep things "simple" :-)


Edoxaban as preferred DOAC

NHSE Guidance indicates a preference that clinicians use edoxaban for AF. A user has requested a measure on OP to inform implementation
https://www.england.nhs.uk/wp-content/uploads/2022/01/B1279-national-procurement-for-DOACs-commissioning-recommendations-v1.pdf

Annex D of this letter indicates IIF indicators will be launched in support of this measure
https://www.england.nhs.uk/wp-content/uploads/2022/03/B1375_Letter-re-General-practice-contract-arrangements-in-2022-23_010322.pdf

https://github.com/ebmdatalab/openprescribing/pull/3470


faster diff

doing a diff in abapGit for abapGit itself takes a lot of time, since the file is ~18000 lines of code. Think the problem is in the amount of HTML to be transferred to the GUI, suggest displaying only eg. 100 lines surrounding a change, and omit code that is far away from changes.
Perhaps a feature to show full file diff.
And perhaps only show partial file if the file is more than 5000 lines

also see #210, it might solve the issue for abapGit


Change language for social sharing buttons

Keep link to provider (e.g. Flickr page hosting image) as the main link.
But the default suggested language for social media sharing needs to do a couple things:

Make user aware of CC Search as a product
Include reference to relevant CC social media account

So change to something like, "I found an image through CC search @creativecommons: [image link]”
Open to suggestions for tweaks!

#165

I am interested in this but I have some questions? Would you want this text to change the Plain Text and Rich Text output on the catalog? Or would you like the HTML one to be changed as well? Or am I completely lost on what you are looking for.
Looking forward to contributing on this,
Faris M.

This is regarding the social sharing buttons that are currently only live in the dev site. See https://github.com/creativecommons/cccatalog-frontend/issues/165

Ran into some small hiccups, getting it working on my local computer, (there were some issues with running it using npm both on my mac and my laptop using arch, but I got it working using docker, I will open an issue for this later). I also had to set the flag to enable social media sharing. Will attempt to solve this :)

@farouski CC does not practice assigning of issues so you should start working on it now and submit a PR as soon as you're ready.
Also since the Dockerfile does very little apart from npm install the only plausible reason for it to not work outside of Docker that I can think of is a Node version difference. Try seeing if your local Node version matches the one in the Dockerfile.

@dhruvkb I saw on the guidelines, to wait to be assigned. I have already completed this work, but I guess I might as well submit my PR right? I will also look into the npm differences. Thanks a lot for your help :)

@farouski If someone else is assigned to an issue or if it has the 'In progress' label, you should probably wait for the other person to either solve it or be inactive for atleast a week. But seeing as no one is assigned and that 'Good first issue' is meant to introduce new people such as yourself to the codebase, your PR is more than welcome.

#274 fixed this

Can we change "CC search" to "CC Search"? Small nit, but I'd like to make the Search more prominent in the text, esp since we're not linking back to the platform for this deploy.
Also, this text does not pop up for the FB option when I tested it. Should also read "I found an image through CC Search @creativecommons: https://www.flickr.com/photos/biodivlibrary/9240505252" for FB as well, currently just a link

Once the above changes have been made, we can push to prod and turn off feature flag.

I will attempt to see if I can get this done :)

Marked as in progress.

I'll look into this!

@brenoferreira the PR is just to fix the capitalization. The issue with the Facebook sharer not loading the sharer text is due to Facebook only looking at meta tags. I listed some possibilities to fix this in the PR, but don't know how to test modifications to the meta tags. Do you have a staging environment? Or do you know how I can force the meta tags to evaluate on my dev server?

@akowtko we use server side rendering to render the html, so there's no need to use vue-meta. You can see how we set meta tags here

@akowtko For the Facebook text, can you open a new issue for that?

@brenoferreira apologies for the delay. I filed a new issue here: https://github.com/creativecommons/cccatalog-frontend/issues/519.


Add current plan point as third parameter to attribute value

It can help to omit N+1 for cases, when requested inner rescord which is not a activerecord a association, but it can
have other associations to preload
Example:
class API::CurrentUserSerializer < ApplicationSerializer
  attribute :current_order, serializer: API::OrderSerializer, value: -> { |user,  _ctx, point|
     order = user.orders.draft.first
     associations = point.preloads
     ActiveRecord::Associations::Preloader.new(records: [order], associations: associations, available_records: [user]).()      if associations.present?
    ).()
    order
  }
end

We can't  omit N+1 in this case as all value blocks are serialized one by one and we will preload associations to each such value.
But having point link is not a bad idea, it can be still helpful for some scenarios

when we want to find parent resource
we know that current serializer always serializes single resource, so we can omit N+1 for nested resources


Kubernetes external ip for a loadbalancer service is in peding state

ISSUE TYPE

Bug Report
Other

COMPONENT NAME

Kubernetes
CLOUDSTACK VERSION

4.18
CONFIGURATION

Advance Networking
OS / ENVIRONMENT

Ubuntu 22.04
SUMMARY

Created a kubernetes cluster with shared network. Service as loadbalancer is not fetching external IP. External IP shows as pending. Also there is no event when I try to describe the service
STEPS TO REPRODUCE

EXPECTED RESULTS

ACTUAL RESULTS

Hi @Noelantogerorge
what is the network offering you have used for the shared network?
By default, there is no LB service  for the shared network  offering (DefaultSharedNetworkOffering)

If you look at the network offering DefaultNetworkOfferingforKubernetesService, there is LB service

Hi @kiranchavala  Thanks for the update...
I had created a custome networkoffering for shared network and enable all  supported service like defaultnetworkofferingforKubernetesService. So my custome networoffering is same as Kubernetes network offering. But still external ip is in pending state.
I tried with isolated network and it works fine. Issue with shared network.
We need some port to run on hostnetwork state so worker node NIC will be utilized. In that case we required to have a public IP so shared network is required.

This issue is reproducible in lab
@davidjumani  @weizhouapache  can you take a look

@Noelantogerorge cc @davidjumani it seems cloudstack-kubernetes-provider is not deployed for a shared network which must be causing this issue.
https://github.com/apache/cloudstack/blob/main/plugins/integrations/kubernetes-service/src/main/java/com/cloud/kubernetes/cluster/actionworkers/KubernetesClusterActionWorker.java#L565-L569
Let me check if that can be changed

@Noelantogerorge cc @davidjumani it seems cloudstack-kubernetes-provider is not deployed for a shared network which must be causing this issue. https://github.com/apache/cloudstack/blob/main/plugins/integrations/kubernetes-service/src/main/java/com/cloud/kubernetes/cluster/actionworkers/KubernetesClusterActionWorker.java#L565-L569 Let me check if that can be changed

ccm should have been deployed, the error happened when deploy a service with LoadBalancer. But LB is not supported in shared network so it failed when ccm tried to acquire a public ip.
@Noelantogerorge
Can you log into the control plane vm and get more information? For example,
sudo kubectl get services
sudo kunectl describe service

@weizhouapache I tried with the changed code to allow deploying provider for shared network.
After that though it tries to list/create loadbalancer but fails with the API call,
⇒  ./kubectl --kubeconfig ./kube.conf describe svc nginx-deployment
Name:                     nginx-deployment
Namespace:                default
Labels:                   app=nginx
Annotations:              <none>
Selector:                 app=nginx
Type:                     LoadBalancer
IP Family Policy:         SingleStack
IP Families:              IPv4
IP:                       <IP_ADDRESS>
IPs:                      <IP_ADDRESS>
Port:                     <unset>  80/TCP
TargetPort:               80/TCP
NodePort:                 <unset>  30495/TCP
Endpoints:
Session Affinity:         None
External Traffic Policy:  Cluster
Events:
  Type     Reason                  Age                   From                Message
  ----     ------                  ----                  ----                -------
  Normal   EnsuringLoadBalancer    2m40s (x11 over 32m)  service-controller  Ensuring load balancer
  Warning  SyncLoadBalancerFailed  2m10s (x11 over 32m)  service-controller  Error syncing load balancer: failed to ensure load balancer: error retrieving load balancer rules: Get http://<IP_ADDRESS>:8080/client/api?apiKey=KrMLbhjYaLWLqnPISgpqo-2XRXKx0MFuJwfcaf2kGuTVaaoTRYmlyQRTy_VuVBn5zXP2Pkb5cCnZxmYu80ycKw&command=listLoadBalancerRules&keyword=a2508be6b99c0462f9bdc09ba0ab46ac&listall=true&response=json&signature=5y5T5QbgcR9jhjHf5raodd%2F3FsA%3D: dial tcp <IP_ADDRESS>:8080: i/o timeout
shwstppr@shwstppr-ThinkPad:~/Downloads|

@weizhouapache I tried with the changed code to allow deploying provider for shared network. After that though it tries to list/create loadbalancer but fails with the API call,
⇒  ./kubectl --kubeconfig ./kube.conf describe svc nginx-deployment
Name:                     nginx-deployment
Namespace:                default
Labels:                   app=nginx
Annotations:              <none>
Selector:                 app=nginx
Type:                     LoadBalancer
IP Family Policy:         SingleStack
IP Families:              IPv4
IP:                       <IP_ADDRESS>
IPs:                      <IP_ADDRESS>
Port:                     <unset>  80/TCP
TargetPort:               80/TCP
NodePort:                 <unset>  30495/TCP
Endpoints:
Session Affinity:         None
External Traffic Policy:  Cluster
Events:
  Type     Reason                  Age                   From                Message
  ----     ------                  ----                  ----                -------
  Normal   EnsuringLoadBalancer    2m40s (x11 over 32m)  service-controller  Ensuring load balancer
  Warning  SyncLoadBalancerFailed  2m10s (x11 over 32m)  service-controller  Error syncing load balancer: failed to ensure load balancer: error retrieving load balancer rules: Get http://<IP_ADDRESS>:8080/client/api?apiKey=KrMLbhjYaLWLqnPISgpqo-2XRXKx0MFuJwfcaf2kGuTVaaoTRYmlyQRTy_VuVBn5zXP2Pkb5cCnZxmYu80ycKw&command=listLoadBalancerRules&keyword=a2508be6b99c0462f9bdc09ba0ab46ac&listall=true&response=json&signature=5y5T5QbgcR9jhjHf5raodd%2F3FsA%3D: dial tcp <IP_ADDRESS>:8080: i/o timeout

@shwstppr it looks like expected behaviour to me.
cc @davidjumani

@weizhouapache https://github.com/weizhouapache
My environment is like I need kubnertes cluster to have public IP. So that
when I deploy pod as host network node IP should be public network and I
need all traffic to route to node network namespace.
So I create a shared network with public IP and observer that load balancer
won't able to provide any external IP address
If this is a normal behaviour then how can we deploy kubernetes in
hostnetwork mode
Same scenario works in coud environment like AWS, AZURE, Digital Ocean etc
On Fri, Apr 21, 2023, 11:41 PM Wei Zhou @.***> wrote:

@weizhouapache https://github.com/weizhouapache I tried with the
changed code to allow deploying provider for shared network. After that
though it tries to list/create loadbalancer but fails with the API call,
⇒  ./kubectl --kubeconfig ./kube.conf describe svc nginx-deployment
Name:                     nginx-deployment
Namespace:                default
Labels:                   app=nginx
Annotations:
Selector:                 app=nginx
Type:                     LoadBalancer
IP Family Policy:         SingleStack
IP Families:              IPv4
IP:                       <IP_ADDRESS>
IPs:                      <IP_ADDRESS>
Port:                       80/TCP
TargetPort:               80/TCP
NodePort:                   30495/TCP
Endpoints:
Session Affinity:         None
External Traffic Policy:  Cluster
Events:
Type     Reason                  Age                   From                Message

Normal   EnsuringLoadBalancer    2m40s (x11 over 32m)  service-controller  Ensuring load balancer
Warning  SyncLoadBalancerFailed  2m10s (x11 over 32m)  service-controller  Error syncing load balancer: failed to ensure load balancer: error retrieving load balancer rules: Get http://<IP_ADDRESS>:8080/client/api?apiKey=KrMLbhjYaLWLqnPISgpqo-2XRXKx0MFuJwfcaf2kGuTVaaoTRYmlyQRTy_VuVBn5zXP2Pkb5cCnZxmYu80ycKw&command=listLoadBalancerRules&keyword=a2508be6b99c0462f9bdc09ba0ab46ac&listall=true&response=json&signature=5y5T5QbgcR9jhjHf5raodd%2F3FsA%3D: dial tcp <IP_ADDRESS>:8080: i/o timeout
@shwstppr https://github.com/shwstppr it looks like expected behaviour
to me.
cc @davidjumani https://github.com/davidjumani
—
Reply to this email directly, view it on GitHub
https://github.com/apache/cloudstack/issues/7431#issuecomment-1518177870,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6FZLRE6XMZRWLIA2MAQ433XCLEVJANCNFSM6AAAAAAW5C4UAM
.
You are receiving this because you were mentioned.Message ID:
@.***>

@weizhouapache https://github.com/weizhouapache My environment is like I need kubnertes cluster to have public IP. So that when I deploy pod as host network node IP should be public network and I need all traffic to route to node network namespace. So I create a shared network with public IP and observer that load balancer won't able to provide any external IP address If this is a normal behaviour then how can we deploy kubernetes in hostnetwork mode Same scenario works in coud environment like AWS, AZURE, Digital Ocean etc
…
On Fri, Apr 21, 2023, 11:41 PM Wei Zhou @.> wrote: @weizhouapache https://github.com/weizhouapache I tried with the changed code to allow deploying provider for shared network. After that though it tries to list/create loadbalancer but fails with the API call, ⇒ ./kubectl --kubeconfig ./kube.conf describe svc nginx-deployment Name: nginx-deployment Namespace: default Labels: app=nginx Annotations:  Selector: app=nginx Type: LoadBalancer IP Family Policy: SingleStack IP Families: IPv4 IP: <IP_ADDRESS> IPs: <IP_ADDRESS> Port:  80/TCP TargetPort: 80/TCP NodePort:  30495/TCP Endpoints: Session Affinity: None External Traffic Policy: Cluster Events: Type Reason Age From Message ---- ------ ---- ---- ------- Normal EnsuringLoadBalancer 2m40s (x11 over 32m) service-controller Ensuring load balancer Warning SyncLoadBalancerFailed 2m10s (x11 over 32m) service-controller Error syncing load balancer: failed to ensure load balancer: error retrieving load balancer rules: Get http://<IP_ADDRESS>:8080/client/api?apiKey=KrMLbhjYaLWLqnPISgpqo-2XRXKx0MFuJwfcaf2kGuTVaaoTRYmlyQRTy_VuVBn5zXP2Pkb5cCnZxmYu80ycKw&command=listLoadBalancerRules&keyword=a2508be6b99c0462f9bdc09ba0ab46ac&listall=true&response=json&signature=5y5T5QbgcR9jhjHf5raodd%2F3FsA%3D: dial tcp <IP_ADDRESS>:8080: i/o timeout @shwstppr https://github.com/shwstppr it looks like expected behaviour to me. cc @davidjumani https://github.com/davidjumani — Reply to this email directly, view it on GitHub <#7431 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A6FZLRE6XMZRWLIA2MAQ433XCLEVJANCNFSM6AAAAAAW5C4UAM . You are receiving this because you were mentioned.Message ID: @.>

@Noelantogerorge
As you mentioned, there is no issue with isolated network. In my opinion, you will have to use isolated network or vpc.
Can you describe more details of your requirements, better with an example ? let's discuss if there is a resolution.

Hi,
We are trying to deploy media server and its app server on kubernetes.
Media server works in host network mode. So basically there are lot of
ports used by media server these ports works through host server network.
If it behind any nat then it won't work.
I have the deployment yml file which works in cloud like AWS,Digital ocean.
If needed I can share the yml file.
On cloudstack we can create a isolated or vpc network for kubernetes
cluster and LB works perfectly. But as I mentioned if we deploy my pod it
works on host network and it get some isolated lan cidr. We need that to be
public IP.
So I created a shared network with public IP and start the cluster and
unfortunately LB fails
I tried adding a shared network to another NIC and see how it works but
that crash down the cluster. Is there any way how we can make nodes to have
public IP rather than isolated or vpc lan cidr.
On Sat, Apr 22, 2023, 1:04 AM Wei Zhou @.***> wrote:

@weizhouapache https://github.com/weizhouapache
https://github.com/weizhouapache My environment is like I need kubnertes
cluster to have public IP. So that when I deploy pod as host network node
IP should be public network and I need all traffic to route to node network
namespace. So I create a shared network with public IP and observer that
load balancer won't able to provide any external IP address If this is a
normal behaviour then how can we deploy kubernetes in hostnetwork mode Same
scenario works in coud environment like AWS, AZURE, Digital Ocean etc
… <#m_6023603284368853770_>
On Fri, Apr 21, 2023, 11:41 PM Wei Zhou @.> wrote: @weizhouapache
https://github.com/weizhouapache https://github.com/weizhouapache
https://github.com/weizhouapache I tried with the changed code to allow
deploying provider for shared network. After that though it tries to
list/create loadbalancer but fails with the API call, ⇒ ./kubectl
--kubeconfig ./kube.conf describe svc nginx-deployment Name:
nginx-deployment Namespace: default Labels: app=nginx Annotations:
Selector: app=nginx Type: LoadBalancer IP Family Policy: SingleStack IP
Families: IPv4 IP: <IP_ADDRESS> IPs: <IP_ADDRESS> Port: 80/TCP
TargetPort: 80/TCP NodePort: 30495/TCP Endpoints: Session Affinity: None
External Traffic Policy: Cluster Events: Type Reason Age From Message ----
------ ---- ---- ------- Normal EnsuringLoadBalancer 2m40s (x11 over 32m)
service-controller Ensuring load balancer Warning SyncLoadBalancerFailed
2m10s (x11 over 32m) service-controller Error syncing load balancer: failed
to ensure load balancer: error retrieving load balancer rules: Get
http://<IP_ADDRESS>:8080/client/api?apiKey=KrMLbhjYaLWLqnPISgpqo-2XRXKx0MFuJwfcaf2kGuTVaaoTRYmlyQRTy_VuVBn5zXP2Pkb5cCnZxmYu80ycKw&command=listLoadBalancerRules&keyword=a2508be6b99c0462f9bdc09ba0ab46ac&listall=true&response=json&signature=5y5T5QbgcR9jhjHf5raodd%2F3FsA%3D
http://<IP_ADDRESS>:8080/client/api?apiKey=KrMLbhjYaLWLqnPISgpqo-2XRXKx0MFuJwfcaf2kGuTVaaoTRYmlyQRTy_VuVBn5zXP2Pkb5cCnZxmYu80ycKw&command=listLoadBalancerRules&keyword=a2508be6b99c0462f9bdc09ba0ab46ac&listall=true&response=json&signature=5y5T5QbgcR9jhjHf5raodd%2F3FsA%3D:
dial tcp <IP_ADDRESS>:8080 http://<IP_ADDRESS>:8080: i/o timeout @shwstppr
https://github.com/shwstppr https://github.com/shwstppr
https://github.com/shwstppr it looks like expected behaviour to me. cc
@davidjumani https://github.com/davidjumani
https://github.com/davidjumani https://github.com/davidjumani — Reply to
this email directly, view it on GitHub <#7431 (comment)
https://github.com/apache/cloudstack/issues/7431#issuecomment-1518177870>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6FZLRE6XMZRWLIA2MAQ433XCLEVJANCNFSM6AAAAAAW5C4UAM
https://github.com/notifications/unsubscribe-auth/A6FZLRE6XMZRWLIA2MAQ433XCLEVJANCNFSM6AAAAAAW5C4UAM
. You are receiving this because you were mentioned.Message ID: @.>
@Noelantogerorge https://github.com/Noelantogerorge
As you mentioned, there is no issue with isolated network. In my opinion,
you will have to use isolated network or vpc.
Can you describe more details of your requirements, better with an example
? let's discuss if there is a resolution.
—
Reply to this email directly, view it on GitHub
https://github.com/apache/cloudstack/issues/7431#issuecomment-1518256554,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6FZLRFBVMHJTIJEMIWL4LLXCLOMXANCNFSM6AAAAAAW5C4UAM
.
You are receiving this because you were mentioned.Message ID:
@.***>

@Noelantogerorge
I assume only the app service is created with type=LoadBalancer.

Hi, We are trying to deploy media server and its app server on kubernetes. Media server works in host network mode. So basically there are lot of ports used by media server these ports works through host server network. If it behind any nat then it won't work.

If media servers are on an isolated network, you can use static nat or create port forwarding rule with a port range.
The app server can access the media servers via :

I have the deployment yml file which works in cloud like AWS,Digital ocean. If needed I can share the yml file.
@Noelantogerorge
can you share the yaml file and network/ip information of nodes/pods on one of the cloud provider ?

On cloudstack we can create a isolated or vpc network for kubernetes cluster and LB works perfectly. But as I mentioned if we deploy my pod it works on host network and it get some isolated lan cidr. We need that to be public IP. So I created a shared network with public IP and start the cluster and unfortunately LB fails
@Noelantogerorge
yes, as we have discussed, LB is not supported in cloudstack shared networks.
If the media servers and app servers must be in same kubernetes cluster, it will not work due to the LB failure.

I tried adding a shared network to another NIC and see how it works but that crash down the cluster. Is there any way how we can make nodes to have public IP rather than isolated or vpc lan cidr.

@weizhouapache
I assume only the app service is created with type=LoadBalancer.
No both media server and app server should be on loadbalancer
If media servers are on an isolated network, you can use static nat or create port forwarding rule with a port range.
The app server can access the media servers via publicIP:publicPort
No prot forwarding and NAT won't work for media server. It should be in host network namespace
can you share the yaml file and network/ip information of nodes/pods on one of the cloud provider ?
I will share you my yml files
yes, as we have discussed, LB is not supported in cloudstack shared networks.
If the media servers and app servers must be in same kubernetes cluster, it will not work due to the LB failure.
So cloudstack is not designed in such a way that kubernetes cluster node can get public IP. In AWS,Digital Ocean I had deployed this and it is working perfectly. Also I had deployed on a baremetal server using kubernetes and other service like metal LB etc.. Is there any other way or work around?

got it @Noelantogerorge
Cannot you create load balancer rules for media servers with host network mode ?
if not, can you share some kubernetes logs ? (e.g. kubectl describe svc )
the load balancer for app servers should work, I think.

@weizhouapache
I created 2 cluster 1 with shared network and other with isolated network. So my app servers and its associated pods works in isolated network with LoadBalancing where as media server works on shared network. I can connect media server with app server through node IP.
Need lot of testing... Thank you for the suppport....

@weizhouapache
Is it possible for you to add metric api to kubernetes cluster..

@Noelantogerorge
can you explain a bit more of your requirement ?

@weizhouapache
Ya, When ever I create a cluster there is no metric API available with it. With metric API we could get the pod and node utilization also it helps to deploy horizontal scaling of pods. I could get it by manually installing metric API's and doing some changes to make it available. But if a normal user spins up a kubernetes cluster it dosen't have these API so they will also have to install manually.
Hope you understand the requirement..

@Noelantogerorge yes, understood.
can you create a github issue for it ? it looks like a small change
can you also share what you have done to make it work ?

@weizhouapache Done, I had raised a github issue with my solution.
As I could able to solve my infra issue with alternative way I am closing this issue. Thank you @weizhouapache, @kiranchavala, @shwstppr... Thank you guys...


AttributeError: module 'jax.tree_util' has no attribute 'register_pytree_with_keys_class'

AttributeError: module 'jax.tree_util' has no attribute 'register_pytree_with_keys_class'
what version of jax and jaxlib works with this?

skip it, this works in google colab:
!pip install jax[tpu] -f https://storage.googleapis.com/jax-releases/libtpu_releases.html


Attempt to improve Russian localization

Keep in mind that I suck at electronics so I did as much as I could here. Someone who is actually good at electronics would do a much better job.

A lot of people use this app in Russia, so on their behalf, thank you! This is great. I hope to refine these over time so please don't worry about subtitles for now - it's just such a relief to have a human look them over instead of blindly trusting the robot!


Integrate into v2.0

This PR includes all of the previous Roames changes squashed into one commit and also following changes.

New REST API endpoints to request DAG information (by run_id)
Auth0 authentication for new RBAC UI
Auth0 authentication to REST API using Roames Access API
New RBAC UI changes to confirms to Roames needs

This PR is done on a separate branch forked off airflow/master (with V2.0 incubating). I did not do any changes to current roames-master branch as merging all V2 changes to that branch is a nightmare.


Changed examples to adhere to new links structure

links is now an array instead of an object.

This reflects the changes in Ericsson/eiffel#25.

Skipped a few files. Will fix.


when use govendor async_producer.go:177:12: undefined: RecordHeader

Versions
Please specify real version numbers or git SHAs, not just "Latest" since that changes fairly regularly.

Logs
go build
vendor/github.com/Shopify/sarama/async_producer.go:177:12: undefined: RecordHeader
When filing an issue please provide logs from Sarama and Kafka if at all
possible. You can set sarama.Logger to a log.Logger to capture Sarama debug
output.
Problem Description
I used govendor (v1.0.9) for my project, when use go build, it reported:
vendor/github.com/Shopify/sarama/async_producer.go:177:12: undefined: RecordHeader
but if I rename vendor to vendor.bak
it will use $GOPATH, then it will be ok, I'm confused

I think it's my problem


Variable query with SQLite datasource: error updating options: e.data[0] is undefined

Describe the bug
When creating a new variable as query with SQLite backend, I get an error.
To Reproduce
Steps to reproduce the behavior:

Go to variables
Click on new variable
Select query variable on SQLite datasource
Write a simple query like "select id from device" (that should return 4 rows)
Click outside of the query field in order to make the query request, an error appear (like in the screenshot)

Screenshots
If applicable, add screenshots to help explain your problem.

Versions (please complete the following information):

OS: Windows
Run via docker? no
Browser: firefox
Grafana Version v8.1.2 (103f8fa094) - enterprise unlicensed
Plugin Version 2.0.0
Plugin installed via grafana-cli or manually? grafana-cli

Additional context
Add any other context about the problem here. Please also attach the logs of Grafana!
Here is the query of select id from device:
POST http://localhost:3000/api/ds/query with body:
{
	"queries": [
		{
			"datasourceId": 2,
			"queryText": "select id from device",
			"rawQueryText": "select id from device",
			"timeColumns": []
		}
	]
}

and result
{
	"results": {
		"A": {
			"frames": [
				{
					"schema": {
						"name": "response",
						"refId": "A",
						"meta": {
							"executedQueryString": "select id from device"
						},
						"fields": [
							{
								"name": "id",
								"type": "string",
								"typeInfo": {
									"frame": "string",
									"nullable": true
								}
							}
						]
					},
					"data": {
						"values": [
							[
								"DESKTOP-87EKQJF_fsp-fnf-solution_dummy",
								"DESKTOP-87EKQJF_fsp-ftv_dummy",
								"FFS000-00000-CS000-0000",
								"FFS220-0129-CS220-1124-EX2"
							]
						]
					}
				}
			]
		}
	}
}

Here is the error stack:
from DataSource.ts (see the red line):
 import { DataSourceInstanceSettings, DataFrame, ScopedVars } from '@grafana/data';
 import { DataSourceWithBackend, getTemplateSrv } from '@grafana/runtime';
 import { MyDataSourceOptions, SQLiteQuery } from './types';

 export class DataSource extends DataSourceWithBackend<SQLiteQuery, MyDataSourceOptions> {
   templateSrv;

   constructor(instanceSettings: DataSourceInstanceSettings<MyDataSourceOptions>) {
     super(instanceSettings);

     this.templateSrv = getTemplateSrv();
   }

   applyTemplateVariables(query: SQLiteQuery, scopedVars: ScopedVars): SQLiteQuery {
     query.queryText = this.templateSrv.replace(query.rawQueryText, scopedVars);
     return query;
   }

   async metricFindQuery(query: string, options?: any) {
     if (!query) {
       return [];
     }
     const response = await this.query({
       targets: [
         {
           rawQueryText: query,
           queryText: query,
           timeColumns: [],
         },
       ],
     } as any).toPromise();

     if (response.error) {
       throw new Error(response.error.message);
     }

-    const data = response.data[0] as DataFrame;

     if (data.fields.length !== 1) {
       throw new Error(
         `Received more than one (${data.fields.length}) fields: ${data.fields.map(x => x.name).join(',')}`
       );
     }

     return data.fields[0].values.toArray().map(text => ({ text }));
   }
 }

There has been a bug with Grafana v8, which was fixed in v2.0.2 of the plugin (see link.
Can you install a newer plugin version? Here is how to do it: link


persist overrideConfig in chat_message table to support multi tenancy

In order to support multi tenancy in flowise, meaning that a single flow may server multiple customers we can deliver the overrideConfig parameter to the prediction/internal-prediction APIs, so you can overload additional variable of customer_id (for example, or you can give whatever alias you would like to).
But with the current implementation and in order to support the predictions analytics/logs you won't be able to differ such chat_messages among your different tenants/customers.
Hence, I added this overrideConfig to be persisted which will bring us the ability to QUERY by your given variable (e.g. customer_id)
So for example, if your API would called with such overrideConfig:

You can later query that watching the final result persisted on DB:

@HenryHengZJ @Jaredude FYI

It's failing CI Build, but aside from that, perhaps the overrideConfig column should take advantage of database specific data types? Sqlite doesn't have a json data type, but MySQL does and Postgres supports the jsonb data tyle. These data types are more optimized for json querying than having to convert the text to json to use json querying.

Hi @niztal,

you can pass in your custom customer_id value as chatId
Then you can query by chatId to identify your tenants/customers
In our roadmap we will be adding a new table to store customerInfo which will have relationship with chatId. This will require your tenants/customers to enter their info before they are able to enter the embedChat that you shared.
Is this what you're trying to achieve?

It's failing CI Build, but aside from that, perhaps the overrideConfig column should take advantage of database specific data types? Sqlite doesn't have a json data type, but MySQL does and Postgres supports the jsonb data tyle. These data types are more optimized for json querying than having to convert the text to json to use json querying.

It's just a string column data type, as you can see on my commit I've made sure to handle all db types (postgres, MySQL, SQLite) .
I don't have postgres besides that I've tested it on MySQL and SQLite and it worked fine.
I'll fix the ci failures.
Thanks @Jaredude

Hi @niztal,  you can pass in your custom customer_id value as chatId Then you can query by chatId to identify your tenants/customers
In our roadmap we will be adding a new table to store customerInfo which will have relationship with chatId. This will require your tenants/customers to enter their info before they are able to enter the embedChat that you shared.
Is this what you're trying to achieve?

Mainly yes, it sounds like a workaround, but for my use case I can use chatId.
Just one verification, the chatId can be overriden as part of the overrideConfig or just pass chatId as is to the prediction API?
Do you have eta regarding the customerInfo table?
@chungyau97
Thanks

This is my example last time:
{
    "chatId": "842a4db3-82b0-4d99-b213-b0df56ac540a",
    "question": "What's Jane?",
    "apiType": "embedded",
}

More Information:
Just think of chatId as a new chat started by your user
If userA started the chat, you can just pass in chatId: userA
One thing to make your chatId more unique is by combining the userId_chatflowId, this will prevent userA message get mix between different chatflow
Probably between 27Feb-5Mar, if everything went smooth.

@chungyau97 Thanks for the input on this.
@niztal I found a bug where if you query the chatmessages API endpoint by chatId, the result is always an empty aray. #1716 Resolves that issue in case you encounter it too

@chungyau97 Thanks for the input on this.
@niztal I found a bug where if you query the chatmessages API endpoint by chatId, the result is always an empty aray. #1716 Resolves that issue in case you encounter it too

Thanks @chungyau97 @Jaredude
I will close this PR.


Added ASCII-only output support

Dear Mr. Mateusz Pusz,
Much of C++ template programming with C++20 features is still a bit overwhelming for me at this point in time.
I do not fully understand how the library is written, but I can rely on what I already know about it and my intuition to understand certain parts. I think I get the gist of it.
I am using this library for SPICE circuit generation using custom templates. I have generator classes that are responsible for rendering the custom templates into SPICE circuit text.
Initializing my generator:
RCCircuitGenerator generator {5ns, 2ms, 5V, 1ms, 10kR, 10nF};
The formatting string for the line starting with "RLOAD" (see below) looks like this:
RLOAD 1 2 {:%Q%q}
The following SPICE RC circuit text would be generated:
RC_CIRCUIT
.TRAN 5ns 2ms
VSRC 1 0 PULSE 0 5V 0 0 0 1ms
RLOAD 1 2 10kΩ
CCHARGE 2 0 10nF
.END

However, the SPICE simulator library I am using, ngspice, expect ASCII encoding. The simulation crashes when this circuit is passed to the library.
With the changes I have provided, when the formatting string for the line starting with "RLOAD" looks like this:
RLOAD 1 2 {:%Q%a}
Then the following SPICE RC circuit text would be generated:
RC_CIRCUIT
.TRAN 5ns 2ms
VSRC 1 0 PULSE 0 5V 0 0 0 1ms
RLOAD 1 2 10kohm
CCHARGE 2 0 10nF
.END

And the SPICE simulation then works.
I hope that my modifications adhere to the style and philosophy of this library as closely as possible.
Please let me know of any modifications you would like to me perform first if you would like to merge this pull request eventually.
Thank you very much.
Ramzi Sabra

Hi, first of all, BIG thanks for the contribution! I am aware of ASCII-only need in the library as you can see in #30 but was delaying it because of priorities and waiting for gcc-10 to become mainstream (probably after Ubuntu 20.4 is released) as NTTP support in gcc-9 is really poor. The latter is caused by the fact that I do not like to repeat all symbols for units and prefixes if they are the same for both encodings. In my opinion, we should implement the feature by modifying/extending/wrapping basic_fixed_string class with a two-argument constructor that will allow providing the second ASCII-only version if it differs. In such a case the syntax will probably look as follows:
struct ohm : named_unit<ohm, {"Ω", "ohm"}, prefix> {};

with all other cases left as they are now.
Also, I am not the fun of %a instead of %q, because in such case one needs to always provide a long format string for ASCII output. I would prefer something like {:A} that will print the symbol in ASCII with a default space between or in your case {:A%q%q}. What do you think about it?
At last, we need to provide an extension to override default unit symbols with custom ones anyway (for example for internationalization), so maybe ASCII-only support should also be handled that way rather than extending the basic design?
Everyone, any feedback on it is greatly appreciated :smiley:

Following #76 and the time.format table alternatively we could provide 'A' modifier that will be working with %q. So you will type something like: {:%Q%Aq} (yeah I know it is longer to type :smiley:). Unfortunately, we will miss the short form for a default syntax with space, but we will be more compatible with std::chrono.

I added A modifier to the 'units-format-spec' docs.

It is great to see some real work being done with units ;)  I also think that ascii output should be the default rather than unicode, since it has the widest coverage and is compatible with widely used file formats. As soon as you write out a Ω by default ... you are likely to hit compatibility issues.

Hi, first of all, BIG thanks for the contribution!
Thank you for the warm welcome!

I am aware of ASCII-only need in the library as you can see in #30 but was delaying it because of priorities and waiting for gcc-10 to become mainstream (probably after Ubuntu 20.4 is released) as NTTP support in gcc-9 is really poor.
Oh, I see!

The latter is caused by the fact that I do not like to repeat all symbols for units and prefixes if they are the same for both encodings.
As I was implementing these changes, I thought about if this were a more generic solution that used arrays but realized soon that I was going overboard with the over-engineering. Then I reread your requirements, that you would want each unit to have two symbols and thought that there was probably a good reason to have exactly two symbols for each unit.

In my opinion, we should implement the feature by modifying/extending/wrapping basic_fixed_string class with a two-argument constructor that will allow providing the second ASCII-only version if it differs. In such a case the syntax will probably look as follows:
struct ohm : named_unit<ohm, {"Ω", "ohm"}, prefix> {};

with all other cases left as they are now.

And this gave me a clue on how to implement the changes much more elegantly than having to modify every single unit and add a redundant symbol for the vast majority of the units already implemented (since most already use ASCII characters).

Also, I am not the fun of %a instead of %q, because in such case one needs to always provide a long format string for ASCII output. I would prefer something like {:A} that will print the symbol in ASCII with a default space between or in your case {:A%Q%q}. What do you think about it?

Following #76 and the time.format table alternatively we could provide 'A' modifier that will be working with %q. So you will type something like: {:%Q%Aq} (yeah I know it is longer to type smiley). Unfortunately, we will miss the short form for a default syntax with space, but we will be more compatible with std::chrono.

I went with this. A qualifier 'A' before 'q' that indicates that the unit symbol should be ASCII.

At last, we need to provide an extension to override default unit symbols with custom ones anyway (for example for internationalization), so maybe ASCII-only support should also be handled that way rather than extending the basic design?

Yes, I saw an issue related to that. This is currently over my head without the necessary discussion needed for me to find my way through, so I would much prefer if we had a good discussion on how we should go about this. I'd love to give it a go after the discussion has reached consensus on the proper way to do this.
I have implemented the changes you have recommended, and it is now a much neater solution. However, in order not to pollute the commits with one that changes many files then another that changes them back, I decided to just create a new branch and issue a pull request from that. Therefore, I will be closing this pull request in order to open the other one.
Thank you again!

It is great to see some real work being done with units ;) I also think that ascii output should be the default rather than unicode, since it has the widest coverage and is compatible with widely used file formats. As soon as you write out a Ω by default ... you are likely to hit compatibility issues.

Thanks! You make a good point.

As soon as you write out a Ω by default ... you are likely to hit compatibility issues.

On the other hand how to be compatible if someone can print alternative as 1 ohm, 1 Ohm, 1 OHM, 1 R? Ω is the official symbol. If you are not using it than many people will have different ideas what should be the replacement ;-)

continuing https://github.com/mpusz/units/pull/77#issuecomment-600032707

As soon as you write out a Ω by default ... you are likely to hit compatibility issues.

On the other hand how to be compatible if someone can print alternative as 1 ohm, 1 Ohm, 1 OHM, 1 R? Ω is the official symbol. If you are not using it than many people will have different ideas what should be the replacement ;-)

Not really a great problem in practise, so long as it "runs everywhere"
1 ohm, 1 Ohm, 1 OHM, 1 R? Ω
All these communicate the same information , that we have a resistance in ohms, so I would argue that for basic functionality, exact format isnt important, but should be stable. Once up and running output can bee customised to what you want.
Use case
I am currently testing out mbed online compiler https://os.mbed.com/docs/mbed-os/v5.15/introduction/index.html . Nice Framework, since it aims to be compatible with C++ standard while being a standalone system on microcontroller, so std::cout is directed to serial port
( FWIW board I am using is a https://os.mbed.com/platforms/ST-Nucleo-F746ZG/ ) so USB serial port in this case.
Now for debugging , I have USB serial port. I am testing out magnetometer, (old HMC5883). Mbed allows importing libs ( eg quan-trunk from github ). Unfortunately can't use mpusz/units since c++2a isnt supported yet and also need to disable exceptions and rtti FWIW, so using my quan library.  Sample output sent to my terminal. Here is the source snippet that sends the output
  for (;;) {
        if ( mag_data_ready()) {
            quan::three_d::vect<quan::magnetic_flux_density::uT> values;
            if(mag_read(values)) {
                std::cout << "mag = " << values << '\n';
            } else {
                std::cout << "mag read failed\n";
            }
        } else {
            std::cout << "...\n";
            ThisThread::sleep_for(10U);
        }
    }

So exact same code I would use hosted. Compared to most microcontroller frameworks I have used, this is really impressive and it just works out of the box with all my overloaded stream out functions , without having to search through documentation to find some special switch to get output right. For getting up and running, this is really important. Once up and running, sure output can be customised to your needs , but default should be "what runs anywhere"
Sample output from above in my terminal :
mag = [35.04 uT, 1.825 uT, -20.367 uT]
mag = [35.04 uT, 1.825 uT, -20.367 uT]
mag = [35.113 uT, 1.898 uT, -20.367 uT]
mag = [35.04 uT, 1.825 uT, -20.367 uT]
mag = [35.186 uT, 1.825 uT, -20.294 uT]
mag = [35.04 uT, 1.825 uT, -20.44 uT]
mag = [35.113 uT, 1.825 uT, -20.44 uT]
mag = [35.113 uT, 1.898 uT, -20.44 uT]
mag = [35.04 uT, 1.752 uT, -20.513 uT]
mag = [35.04 uT, 1.898 uT, -20.44 uT]
mag = [35.113 uT, 1.898 uT, -20.513 uT]
...
mag = [35.04 uT, 1.825 uT, -20.44 uT]

...... etc
IOT is primary area of use for units library.

@kwikius, and what's the problem of sending Ω over serial?

IOT is primary area of use for units library.

That's arguable. There is a whole lot of scientific and engineering software which can benefit from it. I wouldn't call any of them "primary".

However, the SPICE simulator library I am using, ngspice, expect ASCII encoding.

In my opinion, it's a bug/lack of functionality in ngspice and it shouldn't define the design of C++ standard library. I understand the need for ASCII-only mode and localization, but standard symbols should be the default even if it requires Unicode.

SI specification is standardized by the ISO which is the same organization that standardizes C++ and our physical units library. The official SI specs define Ω to be the symbol of resistance and that is why we have to use it by default too. Of course, as I already pointed out, I agree that we need ASCII-only output too but it should not be the default mode of the text output.

@kwikius, and what's the problem with sending Ω over serial?

LOL. Doesnt even work in official  compiler (at least in my browser :)  ....
https://godbolt.org/z/K7BoSF
ASM generation compiler returned: 0
:9:17: warning: multi-character character constant [-Wmultichar]
9 |    std::cout << 'Ω' <<  '\n';

  |                 ^~~

:10:18: warning: multi-character character constant [-Wmultichar]
10 |    printf("%c\n",'Ω');
  |                  ^~~

Execution build compiler returned: 0
Program returned: 0
R
R
52905
�

IOT is primary area of use for units library.

That's arguable. There is a whole lot of scientific and engineering software which can benefit from units library. I wouldn't call any of areas "primary".

"Arguable " For sure ;)  This is an old issue. Engineering == IOT FWIW
1000's of embedded system per mainframe. ;)

However, the SPICE simulator library I am using, ngspice, expect ASCII encoding.

In my opinion, it's a bug/lack of functionality in ngspice and it shouldn't define the design of C++ standard library. I understand the need for ASCII-only mode and localization, but standard symbols should be the default even if it requires Unicode.

NGSPICE has this exactly right. It is designed to run anywhere on any system, anywhere for free, thus giving anybody the same chance, rather than locking out those that cant afford latest proprietary equipment (think Microsoft WORD for classic example of lockin, that people cant open old files by default), hence NGSPICE sticks to ascii characters only in source files, hence its ease of use and longevity.
@mpusz. SI spectication is not a software specification. You need to find the intersection with c++ basic source char set
https://bytes.com/topic/c/answers/698502-basic-source-character-set
Anyway meanwhile I can use my own units lib. Pleased that it works out of the box in the above application :)  and will be happy to try out mpusz/units in emebedded if and when it works there.

Doesnt even work in official compiler (at least in my browser :) ....

You are doing it wrong. Ω is a multi-byte sequence (exact bytes will depend on encoding) and cannot be a narrow character literal. You can print it as string without any issues. Out of the box it will give out UTF-8, which should be fine in the majority of cases.

Doesnt even work in official compiler (at least in my browser :) ....

You are doing it wrong.

The default should not be so easy to get wrong.
Ω is a multi-byte sequence (exact bytes will depend on encoding) and cannot be a narrow character literal. You can print it as string without any issues. Out of the box it will give out UTF-8, which should be fine in the majority of cases.
You can print as string but you are in a world of pain if you are writing to a LCD display ...
But I dont think that mpusz/units is really targetting embedded, so I guess that is OK.

NGSPICE has this exactly right. It is designed to run anywhere on any system, anywhere for free, thus giving anybody the same chance, rather than locking out those that cant afford latest proprietary equipment (think Microsoft WORD for classic example of lockin, that people cant open old files by default), hence NGSPICE sticks to ascii characters only in source files, hence its ease of use and longevity.

No, ngspice is a SPICE simulator. SPICE was released 47 years ago, in 1973. It evolved probably only up till the 80's / 90's and that's that. ASCII-only support in SPICE simulators is not a "right" feature, it's the only way to go. You're not going to find a SPICE circuit text file with a Unicode symbol, I can assure you that.
As for the rest, I can't see what this has to do with owning proprietary equipment and such.
Feel free to use fmt::format with the A modifier (%Aq) to have ASCII-only support on embedded and let us know how it goes.

Firstly, thankyou for your work on ASCII support. I will certainly try it out soon.

NGSPICE has this exactly right. It is designed to run anywhere on any system, anywhere for free, thus giving anybody the same chance, rather than locking out those that cant afford latest proprietary equipment (think Microsoft WORD for classic example of lockin, that people cant open old files by default), hence NGSPICE sticks to ascii characters only in source files, hence its ease of use and longevity.

[ ...]. You're not going to find a SPICE circuit text file with a Unicode symbol, I can assure you that.

Absolutely and that is how it should be , backward compatible, just like C++
There are very good reasons to limit source code to ASCII ONLY
There are periodic attempts to try to allowing pretty characters in source code. They are fraught with problems and not very successful so far. eg(https://en.wikipedia.org/wiki/Fortress_(programming_language))  C++ is nearly as old as Spice and should make every effort to stay backward compatible too.

As for the rest, I can't see what this has to do with owning proprietary equipment and such.

Feel free to use fmt::format with the A modifier (%Aq) to have ASCII-only support on embedded and let us know how it goes.

As regards using the library in embedded. Having multibyte as the default makes this library less useful, if I now have to include the extra bloat both of multibyte and of ascii functionality in my embedded system, where both ROM and RAM are expensive luxuries.
Anyway It is not important. I have ny own quantities libray which works very well on even very small systems and old compilers including Arduino :
https://github.com/kwikius/ArduIMU/blob/devel/ArduIMU/main_loop.cpp
https://www.sparkfun.com/products/retired/11055
Feel free to try that out and let us know how that goes too ;)

@kwikius I appreciate your engineering and physical units experience a lot. It is great that despite having your own library you make efforts to help us, contribute, and share your feedback. Thank you for this!
Regarding the Unicode support as default, such behavior was explicitly requested by both the Study Group 16 (SG16) (as they scope on Unicode) and LEWGI (as they want to be consistent with ISO 80000) in the ISO C++ Committee. As this library is going to be a subject for standardization this is why we have it that way. It does not mean that the ISO C++ Committee will not change its mind (actually it happens quite often :wink:) but I wouldn't expect it in this particular subject.
Because of the above, I would like to cut the discussion on the default mode of the text output. The only thing that we might continue discussing is if there should be an alternative way to set the mode to ASCII (other than %Aq modifier) but I really do not like globals to keep state :wink:.

OK. WG21 has often had strange ideas, and we must remember that it doesnt well represent the community, but usually language experts, so I will continue to make my case, but will leave this one now then :)
Looking at the actual output in both acii non asscii. So simple grammar I had assumed
NUMBER '  ' UNIT_STRING
where UNIT_STRING dooesnt include whitespace
doesnt work now?
Does the UNIT_STRING now allow arbitrary number of spaces in it?
If so that makes it very hard to parse, doesnt it?
It is all getting incredibly complicated to work with in my opinion!

Yes, I think that is side effect of https://github.com/mpusz/units/pull/78#discussion_r398539502. It seems we have to again choose if we want to be standard compliant or good from the engineering point of view :disappointed:

Just for reference here is a snippet of the official ISO 80000:

Opened #86 as reminder to discuss this with ISO at the next meeting.

Just for reference here is a snippet of the official ISO 80000:

It might be really hard to parse indeed. On the other hand, it is THE standard on physical units and we should strive to be consistent.

I don't see spaces though in those unit_strings. I am assuming the ' . ' is one multibyte character there? It is spaces I have an issue with. Note that e.g std::cin will parse till a space AFAIR?

I am assuming the ' . ' is one multibyte character there?

I don't think it is one character, but these can easily be some fancy Unicode spaces, e.g. 'NARROW NO-BREAK SPACE' (U+202F).

I am assuming the ' . ' is one multibyte character there?

I thought the same initially and that is why I tried to highlight this part of the text character by character in Adobe Reader. The cursor stopped 3 times...

But I would be more than happy for you to prove me wrong on this one because I really dislike those additional spaces in the text output too.

In line with https://github.com/mpusz/units/issues/72#issuecomment-590811331
how about making conforming output verbose and explicit:
fmt::print("{:%ISO8000Q %ISO8000q}", 9.8q_m_per_s2)

I do not know how it relates the comment you quoted above, but in a very special way I really like your suggestion... :stuck_out_tongue:

I do not know how it relates the comment you quoted above, but in a very special way I really like your suggestion... stuck_out_tongue

:)
It is a serious suggestion ( except maybe it doesnt fit the grammar)
A humble but strong feature of using strongly typed quantities is that they provide very good in code documentation, so the above is related in that it too provides source code documentation
const Energy auto E = total_energy(p, m, si::si2019::speed_of_light<>);

says "I specifically want the si2019 speed of light as opposed to the latest version"
fmt::print("{:%ISO80000Q %ISO80000q}", 9.8q_m_per_s2)

says explicitly and unequivocally , "This is a std::quantity and I specifically want this output to conform to ISO80000 both in numeric format and unit format".
Now you can have global output format state for those of use that prefer brevity, since you can override it ;)
The software house that needs ISO80000 can review the code and tick the box ISO8000 output. It is documented right there in the source code.

says "I specifically want the si2019 speed of light as opposed to the latest version"

says explicitly and unequivocally , "This is a std::quantity and I specifically want this output to conform to ISO80000 both in numeric format and unit format".

No, both cases are totally different. In the first case, we have to be explicit because C++ does not provide us an ABI-safe way to specify a "latest version". There is no C++ language feature that will allow us to do that.
The second case does not have anything in common with ABI stability and the proposed modifier has totally different meaning there. It is just a formatting switch. Right now the default is the one conforming to the ISO specs. If the ISO C++ Committee will agree that the default should be different we will change it. But for now, we have to obey the rules.


tikv-ctl: add diff feature to check difference between two dbs

@zhangjinpeng1987 @BusyJay PTAL

PTAL @BusyJay

LGTM

PTAL @zhangjinpeng1987

LGTM


version column empty

on the #files download page, the version column in the table just displays -.

looks like that version column is meant for the binary wheel target python version, but that is not obvious if there is no wheel.


Fix problem with only updating ghost paths between elements in the matching view

This was an unintended side-effect of b237fce63c2da6b0f50d97ecfd2df0c732ba4eb8

@jeffbaumes, let me know if this fixes the behavior you were seeing

Not sure why this made a Travis build fail, but LGTM, it fixes the issue I was seeing.

I think it's an old image testing issue that's cropped up periodically—I think we first saw it sometime around the time #290 was merged.


enable changing of expected_proxies_count

@jeredding commented on Wed Apr 22 2020

Overview

We should add the ability to adjust expected_proxies_count as it would be helpful to avoid dealing with json blobs (see the docs]
Possible Implementation

Testing

Context

Your Environment

@jeredding commented on Wed Apr 22 2020
I was able to use the raw API + PATCH to do this for now.

@marclop commented on Thu Jul 16 2020
@jeredding sorry for the delay here, from your wording that you used the PATCH method it seems that you used this API: https://www.elastic.co/guide/en/cloud-enterprise/2.5/update-proxies-settings.html. Is that correct? If so we could implement it.

This has been addressed by https://github.com/elastic/ecctl/issues/484


Cannot initialize Gauge label values with exception

When trying to initialise label values for a Gauge like so (following examples in Readme):
from prometheus_client import Gauge

sample_gauge = Gauge(name='example', documentation='Some doc', labelnames=['one', 'two', 'three'])

sample_gauge.labels(["1", "2", "3"])
sample_gauge.set(1)

Running this snippet will throw the following exception in any case:
Traceback (most recent call last):
  File "*/sample.py", line 5, in <module>
    sample_gauge.labels(["1", "2", "3"])
  File "*/Library/Caches/pypoetry/virtualenvs/checkm8-MuaFxrMT-py3.9/lib/python3.9/site-packages/prometheus_client/metrics.py", line 164, in labels
    raise ValueError('Incorrect label count')
ValueError: Incorrect label count

It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.

It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.

@brian-brazil then why is this not mentioned anywhere, and people are allowed to even open Github Issues in the first place? For an essential library used by thousands the documentation and the attitude of maintainers is a joke. If you don't provide a proper answer like a workaround or a patch I am forking.

@brian-brazil then why is this not mentioned anywhere, and people are allowed to even open Github Issues in the first place? For an essential library used by thousands the documentation and the attitude of maintainers is a joke. If you don't provide a proper answer like a workaround or a patch I am forking.

What you're asking is a usage question that is already covered by documentation, and accordingly the mailing list is the best place for that. Github issues are for bugs and feature requests, which this is not.

What you're asking is a usage question that is already covered by documentation, and accordingly the mailing list is the best place for that. Github issues are for bugs and feature requests, which this is not.

What usage question? This is clearly a bug, following the documentation. I am forking.

What usage question? This is clearly a bug, following the documentation. I am forking.

@DaniruKun - I understand this issue is closed by I did struggle with this a little today and then came across this issue and was able to make it work.
https://github.com/prometheus/client_python/issues/38
My code sample:
g = Gauge(name='my_sample_metric', documentation='Description of gauge', labelnames=["label1", "label2"])
g.labels(label1="foo", label2="bar").set(random.randint(0, 500))


Enhance fuse-overlayfs instructions.

The fuse-overlayfs package provided by Ubuntu up to 19.10,  is not recent enough and causes errors on buildah commit, for instance.
Adjust the rootless tutorial to point this out and to provide more detailed instructions on how to obtain fuse-overlayfs and configure it for use by libpod.

By the way, I wonder if project kubic could include fuse-overlayfs on their ubuntu packaging...

By the way, I wonder if project kubic could include fuse-overlayfs on their ubuntu packaging...

@lsm5 PTAL. https://build.opensuse.org/project/monitor/devel:kubic:libcontainers:stable?arch_x86_64=1&blocked=1&broken=1&building=1&defaults=0&deleting=1&dispatching=1&failed=1&finished=1&locked=1&repo_xUbuntu_19_10=1&scheduled=1&signing=1&succeeded=1&unresolvable=1 doesn't list fuse-overlayfs for 19.10

By the way, I wonder if project kubic could include fuse-overlayfs on their ubuntu packaging...

@lsm5 PTAL. https://build.opensuse.org/project/monitor/devel:kubic:libcontainers:stable?arch_x86_64=1&blocked=1&broken=1&building=1&defaults=0&deleting=1&dispatching=1&failed=1&finished=1&locked=1&repo_xUbuntu_19_10=1&scheduled=1&signing=1&succeeded=1&unresolvable=1 doesn't list fuse-overlayfs for 19.10

hmm, fuse-overlayfs is present on debian sid, I'll try and rebuild that for ubuntu.

@TomSweeneyRedHat, adjusted per your feedback.
One thing I haven't verified is if 0.7.6 is really the minimum version, but since it's the currently released version and it works (as opposed to 0.3.1 from Ubuntu which doesn't).
If you'd prefer I report an earlier version as required, I can modify the instructions accordingly.

I'd say stick with 0.76 if that's a released stable version and will work going forward.

@lsm5, would be nice if there was a version of fuse-overlayfs pre-packaged for Ubuntu 18.04 as well, since it's the current LTS and lots of Ubuntu derivatives are based on it.

@lsm5, would be nice if there was a version of fuse-overlayfs pre-packaged for Ubuntu 18.04 as well, since it's the current LTS and lots of Ubuntu derivatives are based on it.

ya, i'll try and send one to obs now

available for ubuntu and debian now: https://build.opensuse.org/package/show/devel:kubic:libcontainers:stable/fuse-overlayfs .. RE: 18.04 seems like the build env can't find some debhelper packages. I'll check with OBS people tomorrow about this and update this thread. HTH.

Since we now have the correct version available for Ubuntu, do we still need this PR?

Since we now have the correct version available for Ubuntu, do we still need this PR?

let's wait till I hear from OBS guys about 18.04 build setup.

So, fuse3 (libfuse3-dev is a dep for fuse-overlayfs) isn't available for ubuntu 18.04 and building it appears to be non-trivial. I hope we can go ahead with this PR and not worry about 18.04 since a new LTS should be coming out in 2 months.
But in case you have buildable sources for fuse3 on 18.04, I can add them to OBS. Let me know..

/lgtm
/approve


Make wrapito agnostic of testing framework

:tophat: What?

Make wrapito agnostic of testing framework, i.e it will work with jest, vitest and other test frameworks if needed.
:thinking: Why?

Because we don't want to maintain 2 separate libraries
We all could benefits from the improvements in wrapito-vitest

:test_tube: How has this been tested? / :boom: How will I know if this breaks?

All tests must pass
closes #54
Migration steps
1.- Uninstall wrapito and install wrapito-vitests (changing all imports from wrapito to wrapito-vitest)
2.- Change all post/get/patch methods to uppercase
3.- Change in "src/setupTests.js" assertions with matchers
Initial tests in ACMOFy
First iteration

Expect API compatible
ACMOFy

BeeGs

We need to change the matcher to use .toUpperCase for the method, to keep consistency with regular wrapito

We need to change the matcher to use .toUpperCase for the method, to keep consistency with regular wrapito

My two cents on this matter is that it shouldn't matter if you use put or PUT to declare a mock. wrapito should be smart enough to allow both.

Makes sense to me, just changed the types and made the matcher use uppercase to verify methods like it was in wrapito


[Suggestion] add Browser tips to game UI

If this is still relevant, how about adding the "Browser tips" bit from http://steamcommunity.com/groups/MSG2015/discussions/0/598198356167623323/ on the game UI.
At least the part about minimizing or switching tabs.
The gray space next to the Easter button button could be a good place.

We're pretty limited on space unfortunately...
What I might do instead is add a link to that thread there.

Going to solve this at the same time as #169.


Remove custom handling of up, down, PgUp and PgDown

Right now up, down, PgUp and PgDown are bound to move to previous / next step, which is incredibly frustrating since it changes default keyboard behaviour, which is to scroll down actual contents of a step using a keyboard. More than that, it seems that there are currently no way at all to scroll contents of a step using a keyboard, which forces the user to use mouse for no good reason. Please consider removing these keybindings.
Noticed this on https://codelabs.developers.google.com/codelabs/ng2-dart/index.html, but I'm pretty sure this would apply to other codelabs as well, thus reporting the issue here.

Fixed by #29

I will need some time to propagate this to codelabs.dgc site, but it'll be there soon.

Hey @ddrone, I updated the site. How does  https://codelabs.developers.google.com/codelabs/ng2-dart/ look now?
It seem one still has to click on the content to be able to scroll up and down, but it's a step forward.

Now it works much better, thanks!


'modinfo' error

Catchable fatal error: Object of class stdClass could not be converted to string on line 46
line 46 :
same things for : MOTD detail :

Try converting it to a string like so:
echo $response['modinfo']->text;
and
echo $response['description_raw']->text;


Server side validation support?

Hi 👋
I'm curious and I couldn't see anything in the documentation but is it possible to pass back specific field errors from the server for example "Email address is already taken"
Ideally, I'd like a way of returning that error so that the appropriate field.error gets updated.
Apologies if it is in the documentation and I missed in (Great job on that documentation though, it's really easy to follow)

Do you use SolidJS or Qwik?

Here's an example using name and on the formAction$  I want to return that the email is already taken and assign it to the correct property:
import { component$, useStylesScoped$, useSignal } from<EMAIL_ADDRESS>import { routeLoader$, z } from<EMAIL_ADDRESS>import {
  formAction$,
  useForm,
  zodForm$,
  type InitialValues,
} from "@modular-forms/qwik";
import ContactStyles from "./contact.css?inline";

const contactSchema = z.object({
  name: z.string().min(5),
  message: z.string().min(5),
});

type ContactForm = z.input<typeof contactSchema>;

export const useFormLoader = routeLoader$<InitialValues<ContactForm>>(() => ({
  name: "",
  message: "",
}));

export const useFormAction = formAction$<ContactForm>((values) => {
  // Return name already taken error
}, zodForm$(contactSchema));

export default component$(() => {
  useStylesScoped$(ContactStyles);
  const formVisible = useSignal(false);
  const [, { Form, Field }] = useForm<ContactForm>({
    loader: useFormLoader(),
    action: useFormAction(),
    validate: zodForm$(contactSchema),
  });

  return (

Contact

      <button onClick$={() => (formVisible.value = true)}>Contact Me</button>

      {formVisible.value && (
        <Form>
          <Field name="name">
            {(field, props) => (
              <>
                Your name
                <input {...props} type="text" />
                {field.error &&
{field.error}
}
              </>
            )}
          </Field>
          <Field name="message">
            {(field, props) => (
              <>
                Your name
                <textarea {...props} />
                {field.error &&
{field.error}
}
              </>
            )}
          </Field>
          <button>Send</button>
        </Form>
      )}

  );
});

Yes what you have in mind is possible. Do you want the error message to appear at a field or in general, e.g. at the very end of the form? Do you want to use Zod for validation or validate this part manually in formAction$?

I'd like for it to appear at a specific field. i.e. the name field.error I don't mind validating manually in formAction$ though if Zod could be used though to validate that for me it would be awesome.

In formAction$ you could do the following:
export const useFormAction = formAction$<ContactForm>((values) => {
  if (names.includes(name)) {
    return {
      errors: { name: 'Name is already taken' }
    };
  }
}, zodForm$(contactSchema));

And with Zod, I would probably use refine:
const contactSchema = z.object({
  name: z.string().min(5).refine((name) => names.includes(name), 'Name is already taken'),
  message: z.string().min(5),
});

You may find these guides interesting:
https://modularforms.dev/guides/handle-submission
https://modularforms.dev/guides/enhanced-forms
https://zod.dev/?id=refine

Thanks for the reference material. I had hoped returning that error in the formAction would automatically update the appropriate field error in the form but that doesn't seem to have happened. Do I need to do something else?

It works for me. Have you updated to the latest version? https://github.com/fabian-hiller/modular-forms/releases
export const useFormAction = formAction$<LoginForm>(async (values) => {
  return {
    errors: {
      email: 'taken!!', // <--- this shows below the email field
    },
    status: 'success',
    message: 'login failed',
  };
}, zodForm$(loginSchema));

Whoops, I haven't. That would explain it. Thanks!


Remove command listing from bad command replies

Make the bad command replies static, and have them point to a LISTCOMMANDS command, which is baked into asock itself.

Also, this will let most message string fmts and errors be generated once, instead of on the fly.


spaceship - Get only 'live' apps from itc

What's the most efficient way to get the list of live apps from the spaceship apis? There seems to be one method live_version which could be possibly used to get the live version but I am not sure if this is the efficient way because calling live_version would iterate through list of all application objects and call network API (reference).

Calling live_version doesn't query all application objects from my understanding, but instead sends this request https://github.com/fastlane/fastlane/blob/master/spaceship/lib/spaceship/tunes/tunes_client.rb#L316-L332

It doesn't query all applications. My code looks like this.
all_apps = Spaceship::Tunes::Application.all
live_apps = all_apps.select { |app|
  app.live_version != nil
}

This will send request in a loop. Is this the correct way?

Hey @RishabhTayal that is what I would say to do for now. I don't believe there is a way to get just all the live versions from iTC directly sadly :( .

@ohayon for my apps I am getting exception in here when I access the live_version.
Here is the all_apps array in my case.
all_apps
<Spaceship::Tunes::Application
	apple_id="1252772653",
	name="Github Release Tracker",
	vendor_id="1498506458",
	bundle_id="com.rtayal.GHReleases",
	last_modified=1499117755000,
	issues_count=0,
	app_icon_preview_url="https://is4-ssl.mzstatic.com/image/thumb/Purple117/v4/9b/c9/db/9bc9db4c-4e8e-3ebf-7998-571b72352c8a/pr_source.png/340x340bb-80.png",
	version_sets=[<Spaceship::Tunes::VersionSet
		type="APP",
		application=<Spaceship::Tunes::Application
		#<Object ...>>,
		platform="ios">]>
<Spaceship::Tunes::Application
	apple_id="1170208387",
	name="Cotton Calculator",
	vendor_id="com.rtayal.cotton",
	bundle_id="com.rtayal.Cotton",
	last_modified=1504295706000,
	issues_count=1,
	app_icon_preview_url="https://is4-ssl.mzstatic.com/image/thumb/Purple22/v4/5d/28/cf/5d28cf5d-2d2b-8ea8-5c29-ca39a6bf4d5e/mzl.okcddume.png/340x340bb-80.png",
	version_sets=[<Spaceship::Tunes::VersionSet
		type="APP",
		application=<Spaceship::Tunes::Application
		#<Object ...>>,
		platform="ios">]>
<Spaceship::Tunes::Application
	apple_id="1142324557",
	name="Joke A Day",
	vendor_id="com.rtayal.Jokes",
	bundle_id="com.rtayal.Jokes",
	last_modified=1479315967000,
	issues_count=0,
	app_icon_preview_url="https://is4-ssl.mzstatic.com/image/thumb/Purple62/v4/66/65/17/6665173d-1de6-4c7a-57dd-287bee41790d/pr_source.png/340x340bb-80.png",
	version_sets=[<Spaceship::Tunes::VersionSet
		type="APP",
		application=<Spaceship::Tunes::Application
		#<Object ...>>,
		platform="ios">]>
<Spaceship::Tunes::Application
	apple_id="898275446",
	name="GoCandid - Add fun and motion to your pictures",
	vendor_id="com.rtayal.ping",
	bundle_id="com.rtayal.gocandid",
	last_modified=1504295706000,
	issues_count=0,
	app_icon_preview_url="https://is1-ssl.mzstatic.com/image/thumb/Purple5/v4/73/12/ba/7312baeb-bd85-6368-f5a3-c22a92b03edc/mzl.royirmmi.png/340x340bb-80.png",
	version_sets=[<Spaceship::Tunes::VersionSet
		type="APP",
		application=<Spaceship::Tunes::Application
		#<Object ...>>,
		platform="ios">]>
<Spaceship::Tunes::Application
	apple_id="1239136676",
	name="DaringFireball",
	vendor_id="1495341340",
	bundle_id="com.rtayal.Daring-Fireball",
	last_modified=1495341346000,
	issues_count=0,
	app_icon_preview_url=nil,
	version_sets=[<Spaceship::Tunes::VersionSet
		type="APP",
		application=<Spaceship::Tunes::Application
		#<Object ...>>,
		platform="ios">]>
<Spaceship::Tunes::Application
	apple_id="1166260530",
	name="Translate - translate in different languages",
	vendor_id="com.rtayal.Translate",
	bundle_id="com.rtayal.Translate",
	last_modified=1483974069000,
	issues_count=0,
	app_icon_preview_url="https://is3-ssl.mzstatic.com/image/thumb/Purple71/v4/c9/f1/e4/c9f1e419-876f-13fc-a931-1389d7fbdb18/mzl.yehkaxah.png/340x340bb-80.png",
	version_sets=[<Spaceship::Tunes::VersionSet
		type="APP",
		application=<Spaceship::Tunes::Application
		#<Object ...>>,
		platform="ios">]>
<Spaceship::Tunes::Application
	apple_id="1061690782",
	name="TVExample",
	vendor_id="tvexample",
	bundle_id="com.rtayal.tvexample",
	last_modified=1451536222000,
	issues_count=0,
	app_icon_preview_url=nil,
	version_sets=[<Spaceship::Tunes::VersionSet
		type="APP",
		application=<Spaceship::Tunes::Application
		#<Object ...>>,
		platform="appletvos">]>
<Spaceship::Tunes::Application
	apple_id="943609004",
	name="ParseBoard - Dashboard for Parse",
	vendor_id="com.rtayal.parse",
	bundle_id="com.rtayal.ParseBoard",
	last_modified=1479492037000,
	issues_count=0,
	app_icon_preview_url="https://is3-ssl.mzstatic.com/image/thumb/Purple4/v4/84/2e/2a/842e2ab7-2d6f-b207-0f1f-51d6b3174932/mzl.dikyhfsh.png/340x340bb-80.png",
	version_sets=[<Spaceship::Tunes::VersionSet
		type="APP",
		application=<Spaceship::Tunes::Application
		#<Object ...>>,
		platform="ios">]>
<Spaceship::Tunes::Application
	apple_id="979972793",
	name="Jump Jump Jump Game",
	vendor_id="com.rtayal.jump",
	bundle_id="com.rtayal.jump",
	last_modified=1492889468000,
	issues_count=0,
	app_icon_preview_url="https://is1-ssl.mzstatic.com/image/thumb/Purple111/v4/12/48/db/1248dbc7-9348-3f93-40f1-3715a05cd4f9/mzm.pwqrwici.png/340x340bb-80.png",
	version_sets=[<Spaceship::Tunes::VersionSet
		type="APP",
		application=<Spaceship::Tunes::Application
		#<Object ...>>,
		platform="ios">]>
<Spaceship::Tunes::Application
	apple_id="1066002152",
	name="EmailCollector",
	vendor_id="com.rtayal.emailcollector",
	bundle_id="com.rtayal.emailcollector",
	last_modified=1504297078000,
	issues_count=0,
	app_icon_preview_url="https://is1-ssl.mzstatic.com/image/thumb/Purple69/v4/6a/3f/d3/6a3fd387-cf4a-c2f4-5d10-93cdbf4ef50c/pr_source.png/340x340bb-80.png",
	version_sets=[<Spaceship::Tunes::VersionSet
		type="APP",
		application=<Spaceship::Tunes::Application
		#<Object ...>>,
		platform="ios">]>
<Spaceship::Tunes::Application
	apple_id="955870904",
	name="test game parakeet",
	vendor_id="com.testgame",
	bundle_id="com.jjustind.purrocket2",
	last_modified=1494882099000,
	issues_count=0,
	app_icon_preview_url="https://is4-ssl.mzstatic.com/image/thumb/Purple127/v4/bf/5c/cc/bf5ccc6b-4685-191b-c899-e7967feb73e2/pr_source.jpg/340x340bb-80.png",
	version_sets=[<Spaceship::Tunes::VersionSet
		type="APP",
		application=<Spaceship::Tunes::Application
		#<Object ...>>,
		platform="ios">]>
<Spaceship::Tunes::Application
	apple_id="875395391",
	name="vCinity - Chat Without Internet",
	vendor_id="com.rtayal.chatapp",
	bundle_id="com.rtayal.ChatApp",
	last_modified=1505057915000,
	issues_count=1,
	app_icon_preview_url="https://is3-ssl.mzstatic.com/image/thumb/Purple30/v4/b2/5a/f1/b25af1cd-13d1-1ebc-bf74-196693942107/mzl.cmpqhutn.png/340x340bb-80.png",
	version_sets=[<Spaceship::Tunes::VersionSet
		type="APP",
		application=<Spaceship::Tunes::Application
		#<Object ...>>,
		platform="ios">]>
<Spaceship::Tunes::Application
	apple_id="957426073",
	name="RepairShift Test",
	vendor_id="com.rtayal.repairshift",
	bundle_id="com.rtayal.RepairShift2",
	last_modified=1492772859000,
	issues_count=1,
	app_icon_preview_url="https://is1-ssl.mzstatic.com/image/thumb/Purple122/v4/f2/6d/8b/f26d8bad-5503-57f7-fad6-2b68375a56bc/contsched.cvcfjqdy.png/340x340bb-80.png",
	version_sets=[<Spaceship::Tunes::VersionSet
		type="APP",
		application=<Spaceship::Tunes::Application
		#<Object ...>>,
		platform="ios">]>
<Spaceship::Tunes::Application
	apple_id="1158085895",
	name="Scratch `em",
	vendor_id="com.rtayal.scratchitstickers",
	bundle_id="com.rtayal.ScratchItStickers",
	last_modified=1480699375000,
	issues_count=0,
	app_icon_preview_url="https://is5-ssl.mzstatic.com/image/thumb/Purple62/v4/87/91/2f/87912fdc-814a-2e60-4bb5-8ba6c0e4ff8e/pr_source.png/340x340bb-80.png",
	version_sets=[<Spaceship::Tunes::VersionSet
		type="APP",
		application=<Spaceship::Tunes::Application
		#<Object ...>>,
		platform="ios">]>
<Spaceship::Tunes::Application
	apple_id="1281668115",
	name="Review Monitor - Manage App Reviews",
	vendor_id="io.rtayal.ReviewMonitor",
	bundle_id="io.rtayal.ReviewMonitor",
	last_modified=1505168041000,
	issues_count=0,
	app_icon_preview_url=nil,
	version_sets=[<Spaceship::Tunes::VersionSet
		type="APP",
		application=<Spaceship::Tunes::Application
		#<Object ...>>,
		platform="ios">]>
<Spaceship::Tunes::Application
	apple_id="950713207",
	name="ttes",
	vendor_id="asdfdsa",
	bundle_id="com.joeldwyerTumbleCat2",
	last_modified=1448303541000,
	issues_count=0,
	app_icon_preview_url=nil,
	version_sets=[<Spaceship::Tunes::VersionSet
		type="APP",
		application=<Spaceship::Tunes::Application
		#<Object ...>>,
		platform="ios">]>
<Spaceship::Tunes::Application
	apple_id="1239119564",
	name="Pods_DaringFireball",
	vendor_id="1495328632",
	bundle_id="org.cocoapods.Pods-DaringFireball",
	last_modified=1495328638000,
	issues_count=0,
	app_icon_preview_url=nil,
	version_sets=[<Spaceship::Tunes::VersionSet
		type="APP",
		application=<Spaceship::Tunes::Application
		#<Object ...>>,
		platform="ios">]>
<Spaceship::Tunes::Application
	apple_id="1074388907",
	name="Trafficking",
	vendor_id="com.rtayal.Trafficking",
	bundle_id="com.rtayal.TraffickingPolice1",
	last_modified=1478887978000,
	issues_count=0,
	app_icon_preview_url="https://is4-ssl.mzstatic.com/image/thumb/Purple49/v4/2d/18/89/2d1889a3-1a68-9844-86fa-6b8e63e01863/mzl.zbdolorm.png/340x340bb-80.png",
	version_sets=[<Spaceship::Tunes::VersionSet
		type="APP",
		application=<Spaceship::Tunes::Application
		#<Object ...>>,
		platform="ios">]>
<Spaceship::Tunes::Application
	apple_id="1185165692",
	name="TempMessageApp",
	vendor_id="1481343968",
	bundle_id="com.rtayal.MessageGame",
	last_modified=1481929502000,
	issues_count=0,
	app_icon_preview_url=nil,
	version_sets=[<Spaceship::Tunes::VersionSet
		type="APP",
		application=<Spaceship::Tunes::Application
		#<Object ...>>,
		platform="ios">]>
<Spaceship::Tunes::Application
	apple_id="853205272",
	name="FilterVilla - Apply Filters To All Your Photos",
	vendor_id="filtervilla",
	bundle_id="com.rtayal.FilterVilla",
	last_modified=1448303553000,
	issues_count=0,
	app_icon_preview_url="https://is2-ssl.mzstatic.com/image/thumb/Purple/v4/e3/ea/bc/e3eabcea-407f-935c-7ab2-9f3e6ba50e1f/mzl.bfgpblom.png/340x340bb-80.png",
	version_sets=[<Spaceship::Tunes::VersionSet
		type="APP",
		application=<Spaceship::Tunes::Application
		#<Object ...>>,
		platform="ios">]>

Hey @RishabhTayal, would you mind sharing the entire output from the command line? Would be helpful to see all of the context.

@ohayon How can I get the entire output if I am running spaceship from irb?

@ohayon Any update here?

Ah, apologies, I didnt realize that you were using irb. But you should see a stack trace in irb right?

Any special parameter I need to pass while running irb? By default I don't see the stack trace.

@ohayon This is the complete error I get in the stack trace.
Error
Spaceship::Client::UnexpectedResponse: {"data"=>nil, "messages"=>{"warn"=>nil, "info"=>nil, "error"=>["Unexpected Error"]}, "statusCode"=>"ERROR"}
	from /Users/rtayal/.rvm/gems/ruby-2.4.0/bundler/gems/fastlane-b50b0af82a14/spaceship/lib/spaceship/client.rb:588:in `parse_response'
	from /Users/rtayal/.rvm/gems/ruby-2.4.0/bundler/gems/fastlane-b50b0af82a14/spaceship/lib/spaceship/tunes/tunes_client.rb:332:in `app_version'
	from /Users/rtayal/.rvm/gems/ruby-2.4.0/bundler/gems/fastlane-b50b0af82a14/spaceship/lib/spaceship/tunes/app_version.rb:267:in `find'
	from /Users/rtayal/.rvm/gems/ruby-2.4.0/bundler/gems/fastlane-b50b0af82a14/spaceship/lib/spaceship/tunes/application.rb:107:in `live_version'
	from (irb):6:in `block in irb_binding'
	from (irb):5:in `select'
	from (irb):5
	from /Users/rtayal/.rvm/rubies/ruby-2.4.0/bin/irb:11:in `<top (required)>'
	from /Users/rtayal/.rvm/gems/ruby-2.4.0/gems/bundler-1.15.4/lib/bundler/cli/exec.rb:74:in `load'
	from /Users/rtayal/.rvm/gems/ruby-2.4.0/gems/bundler-1.15.4/lib/bundler/cli/exec.rb:74:in `kernel_load'
	from /Users/rtayal/.rvm/gems/ruby-2.4.0/gems/bundler-1.15.4/lib/bundler/cli/exec.rb:27:in `run'
	from /Users/rtayal/.rvm/gems/ruby-2.4.0/gems/bundler-1.15.4/lib/bundler/cli.rb:362:in `exec'
	from /Users/rtayal/.rvm/gems/ruby-2.4.0/gems/bundler-1.15.4/lib/bundler/vendor/thor/lib/thor/command.rb:27:in `run'
	from /Users/rtayal/.rvm/gems/ruby-2.4.0/gems/bundler-1.15.4/lib/bundler/vendor/thor/lib/thor/invocation.rb:126:in `invoke_command'
	from /Users/rtayal/.rvm/gems/ruby-2.4.0/gems/bundler-1.15.4/lib/bundler/vendor/thor/lib/thor.rb:387:in `dispatch'
	from /Users/rtayal/.rvm/gems/ruby-2.4.0/gems/bundler-1.15.4/lib/bundler/cli.rb:22:in `dispatch'
	from /Users/rtayal/.rvm/gems/ruby-2.4.0/gems/bundler-1.15.4/lib/bundler/vendor/thor/lib/thor/base.rb:466:in `start'
	from /Users/rtayal/.rvm/gems/ruby-2.4.0/gems/bundler-1.15.4/lib/bundler/cli.rb:13:in `start'
	from /Users/rtayal/.rvm/gems/ruby-2.4.0/gems/bundler-1.15.4/exe/bundle:30:in `block in <top (required)>'
	from /Users/rtayal/.rvm/gems/ruby-2.4.0/gems/bundler-1.15.4/lib/bundler/friendly_errors.rb:121:in `with_friendly_errors'
	from /Users/rtayal/.rvm/gems/ruby-2.4.0/gems/bundler-1.15.4/exe/bundle:22:in `<top (required)>'
	from /Users/rtayal/.rvm/gems/ruby-2.4.0/bin/bundle:22:in `load'
	from /Users/rtayal/.rvm/gems/ruby-2.4.0/bin/bundle:22:in `<main>'
	from /Users/rtayal/.rvm/gems/ruby-2.4.0/bin/ruby_executable_hooks:15:in `eval'
	from /Users/rtayal/.rvm/gems/ruby-2.4.0/bin/ruby_executable_hooks:15:in `<main>'

This seems like an issue with a particular app in my iTunes account. The app is returned in the 	Spaceship::Tunes::Application.all list, but the app is not showing up in the iTunes connect dashboard portal website. I think there was some corrupted data in the Apple database which is causing this error. Nothing wrong with spaceship. I have contacted Apple regarding this issue and expecting to hear back from them in couple of days.

Thanks @RishabhTayal ! Going to close this issue for now! Let us know if you are still running into issues! 🚀

Update on this issue:
Apple responded to me saying I deleted this particular app 2 years back that's why it's not showing up in the iTC dashboard.
I am not sure how you can delete the app from App Store. Is there a feature to delete the app from itc? Even if there is, why they are returning this app in the api response when called from spaceship.

Yeah thats pretty bizarre that they are doing that. I didnt realize you could even delete an app from iTC hahaha.


Requirements / outcome visualisation document for the MIT AI Demo project

Description
Prepare a requirements documents, that outlines the following
Goals

[ ] Define the Functional Requirements for building an AI agent for the MIT Demo
[ ] Define the Outcome visualisation, defining personas and how the end user will be interacting / using the product
[ ] Define the success criteria for the MIT Demo

Expected Outcome
An end-to-end demo of a Whatsapp based conversation where the user is able to discover and book stay, event tickets at Yellowstone national park and also be able to discover and book EV chargers during his drive there.
Acceptance Criteria

[ ] Functional Requirements are clearly defined
[ ] Outcome visualisation should be defined
[ ] Success criteria to be clearly defined

Implementation Details

For whatsapp api, we will be using wati https://www.wati.io/
Bot will be build using nodejs
Required catalogs will be implemented using strapi bpp

Mockups / Wireframes
Mockups / wireframes if required will be provided

[Please note that the below section of the ticket has to be in the format as mentioned as it is key to enabling proper listing of the project. Please only choose the options mentioned under the headings wherever applicable.]
Product Name
AI Demo
Project Name
This project is for the MIT Demo, planned for april 9th
Tech Skills Needed:
AI, Nodejs

Requirements are currently captured in the document. https://docs.google.com/document/d/1iW433h0A5gfw8dfchSsAic_qd5DEvOmyqDmmUubvbh8/edit#heading=h.qfaskyl5y0hz. its better if we move the doc to confluence under
https://fide-official.atlassian.net/wiki/spaces/EC/pages/290029569/Beckn+-+AI

@ravi-prakash-v  Please validate this if the requirement is accurately captured


Realm.remove many

realm.removeMany implemented "Fixes #60".

Please link to the issues this resolves with "Fixes #xyz".

Please link to the issues this resolves with "Fixes #xyz".

OK, thanks. Done.

also we should snapshot before iteration

Note that snapshotting is only possible for object collections -  I don't think dart supports results of primitives yet, but when you do, you won't be able to snapshot those.

blocked by https://github.com/realm/realm-core/issues/5154


Maybe -w should not spawn Code as a child, when it is not running yet

OS X
This is more of a discussion item mixed with a few issues. This also only applies to when Code is not already running.
There are many repercussions to spawning Code as a direct child of the cli code and we should maybe stay away from it and always spawn as an independent process.

 What happened was that that SIGINT caused by Ctrl C propagated to all our process structure and killed most, except the renderer and its children. After the Cancel, the renderer and children became orphan processes.

I wonder if all would be fixed by also using detached: true here.

Moving to @Tyriar who does the spawn from cli.ts

Well, let me keep it here for me to discuss with Mr. Moreno.

I think we are good for GA atm.

Note the only reason I removed detached: true was to prevent the cli from terminating. It looks like that's now handled by the exit variable.

Pushed a change to always detach. This should fix the issues reported by Joao. Needs verification on all platforms.


Showing "No Value" for all rows of a table when table is a 'table alternate' view

https://dev.facebase.org/data/record/#1/isa:dataset/RID=14290
See the "File" nested table at the end of the page, as shown here in the screen shot.

The ERMrest query for that nested table does return tuples. You can see it in the developer tools network response. It can also be validated with cURL.
curl 'https://dev.facebase.org/ermrest/catalog/1/attributegroup/M:=isa:file_compact/left(id)=(isa:file:id)/left(dataset)=(isa:dataset:accession)/accession=FB00000879/$M/F1:=left(id)=(isa:file:id)/$M/id;M:=array(M:*),F1:=array(F1:*)@sort(id)?limit=26'
The only relevant changes are really the software stack (ie, chaise, ermrestjs, ermest) to the best of our knowledge. We have been working on ACLs, but that seems to be ruled out as a culprit here since ERMrest does return the tuples. There may have been some work on annotations within the catalog but not related to this table to our knowledge.

@robes can you review and close the issue?


Typo in your install command

[Enter feedback here]
One of the commands on your page has a typo in Step 4 of "Install SQL Server"  instructions.
Thank you.
Quickstart: Install SQL Server and create a database on Ubuntu
Step 4:
sudo apt-get update
- sudo apt-get install -y mssql-server
+ sudo apt-get install -y mysql-server

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

ID: 5634d572-0020-808b-693a-13addc8861d9
Version Independent ID: 136bab7d-d4d1-1e82-ba89-be2020bc784b
Content: Ubuntu: Install SQL Server on Linux - SQL Server
Content Source: docs/linux/quickstart-install-connect-ubuntu.md
Service: sql
Sub-service: linux
GitHub Login: @VanMSFT
Microsoft Alias: vanto

@Aarav568 -- thank you for catching this.  I am updating the article.

There's no typo here. SQL Server is mssql-server. If you're looking for MySQL documentation, please reference https://learn.microsoft.com/en-us/azure/mysql/.

#please-close


Check rope's patch compatibility with rope v0.9.4

From pierre.raybaut on 2012-05-24T11:42:28Z
See:
spyderlib/rope_patch.py
We have to fix this issue before processing issue #1046 .
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1047

From pierre.raybaut on 2012-05-24T09:44:04Z
Labels: -Type-Defect -Priority-Medium Type-Task Priority-Critical MS-v2.1 Cat-Editor

From pierre.raybaut on 2012-05-24T10:17:46Z
This issue was closed by revision 3de3dbb8201c .
Status: Fixed


Fix footnote markdown syntax

Add a semicolon ':' to footnote references, let's see if 'jekyll' understands this markdown syntax as github does.
For example, here is a footnote[^1].
[^1]: This should render just fine in the browser.

Now footnotes should work also in Jekyll. Thanks!

Ah it works great! Thanks 👍


[WEIRD] when enable redux-devtools #28

https://github.com/diegoddox/react-redux-toastr/issues/28

Please create and example project and provide steps to reproduce the issue.

Related to https://github.com/gaearon/redux-devtools/issues/304. Feel free to discuss it there.


Data tool layout improvements: mobile spacing & scrollable sidebar

This PR improves the data-tool layout:

Constrain the display to the windows' dimensions
Allow the sidebar to scroll
Fix mobile display

Screenshots:

Closing in favour of #25 which depended on these changes


Provide "jump to the entry with index n" for tag list

Newsboat version (copy the output of newsboat -v or the first line of git show):
2.19.0
Config file (copy from ~/.newsboat/config or ~/.config/newsboat/config):
N/a.
Steps to reproduce the issue:
In the tag list, it is not possible to jump to an entry by simply inputting the index number, like in the feed or article list. You need to input an ":" before. Is it possible to provide this feature here?
Other info you think is relevant:
In the documentation, it says:
"Jump to the entry with the index  (usually seen at the left side of the list). This currently works for the feed list and the article list."

Makes sense. Thanks for the suggestion!


[Bug] Doesn't work - can't use pose

Describe the bug
Can't use pose. Click Generate, then send to txt2image. Nothing happens
To Reproduce
Steps to reproduce the behavior:
Import a pose. Click Generate, then send to txt2image. Nothing happens
Expected behavior
Expected to be able to use the pose
Screenshots
Default UI repros issue.
Environment (please complete the following information):
latest Automatic1111 / default config
Additional context
Would be great to export as JSON, this would temporarily mitigate the issue. And also support DW OpenPose. Thanks!

I've tried it and it simply can't read ControlNet Units

[https://github.com/nonnonstop/sd-webui-3d-open-pose-editor](https://github.com/AUTOMATIC1111/stable-diffusion-webui/issues/8624)
try :

I tried this and for some reason it caused 3d Openpose to fail to load. (I did EXACTLY this) so I commented out the opes.add_option above, and added

And it works! When you click "send text 2 img" it...does! :)


Added map function to Cheerio

Im making a parser for TEI, a academic xml mark up languaje and i find my self doing a lot of eachs that were intented to be maps, so i added. Let me know i you need some kind of test to apply the merge.
Daniel

Thanks for adding this. What's with Change cheerio-soupselect path to point to my branch?

That was suppose to be en an other branch, return it to normal.
I need to made a change to cheerio-soupselect to accept tags like teiHeader. The regex didn't accept capital letters. Also submit a patch for that already, but for now i will need to use my repo as dependency.

Thanks. Okay do you mind opening an issue on cheerio-soupselect regarding this? In the meantime I'll test this addition.


Cleans Up Redux Store

Solution description
Cleaned up the redux store.  Added dummy data into redux to be used anywhere in the app

Excited to see the dummy data implemented.


Avoid mutating string literals

Includes [#1338]
Adds # frozen_string_literal: true to all *.rb files
Replaced string literal mutation in as many places as possible

Needs #1340 for green build

@tomgilligan hey, I pushed to your PR.
Reverted the change of string concatenation instead of mutation. Rails methods like content_tag return mutable strings, so all code paths can rely on them. We should just keep doing the same thing in all code paths.
It could be a massive change for user code that actually relies on these mutable strings.


Update dialogue_manager.gd

This seems to be a typo that's been there for a while, not sure why it wasn't really causing me issues before.

That type was on purpose but I really can't remember the reason. Making it a proper response type does make more sense anyhow.

I was specifically having a crash where it subsequently was looking for a "character" on the object after so yeah.

i dont know what to do