Question: fleet adapter not accepting ChargeBattery task

Hi just want to check how to populate Task for a charging job right now I have populated the task_type TYPE_CHARGE_BATTERY=3 as well as the Station field to the charging station. However when I  submit this task to rmf, rmf says that my fleet is not configured to accept ChargeBattery task.
What am I missing/got wrong here? Anything I'll have to configure in the ros parameter for fleet adapter?

The current implementation of the fleet adapters takes care of charging automatically. There isn't currently a way to explicitly demand that a robot perform a charging task.
We're working on a redesign of the task system. We can take this into account so that the new system supports explicit charging tasks, although I can't think of a reason that a user would need to explicitly ask for a charging since the task planner will make optimal decisions about when charging should occur.
Maybe what you really want is to set a finishing request which decides what the behavior of the robot will be once it has finished all of its assigned tasks. You can set the finishing request to charge the battery so that the robot automatically returns to it charger every time it is finished with its task queue.

I see okay I'll look for a work around for what we need right now. Cause what we need is to pass rmf waypoint name to our system so that our system uses the rmf waypoint name as the charging station.

If you can describe what kinds of information/messages from RMF would be helpful for your system, let us know and we can try to incorporate it into the battery charging workflow. E.g. Should the battery charging task publish a message like
robot_name: "MyRobot001"
charging_station: "charger_004"

when RMF wants "MyRobot001" to use "charger_004" to charge?

If you can describe what kinds of information/messages from RMF would be helpful for your system, let us know and we can try to incorporate it into the battery charging workflow. E.g. Should the battery charging task publish a message like
robot_name: "MyRobot001"
charging_station: "charger_004"

when RMF wants "MyRobot001" to use "charger_004" to charge?

Yes that looks sufficient to what we need and also the robot needs to know how to get there through PathRequest messages from rmf to our fleet driver

Just one more question if you guys happen to see this. We also have an issue that rmf will publish the PathRequest to the closes path first and so happens our ending point is closes the the first waypoint and once our robot reaches the first waypoint it returns to our ending point next.

Just one more question if you guys happen to see this. We also have an issue that rmf will publish the PathRequest to the closes path first and so happens our ending point is closes the the first waypoint and once our robot reaches the first waypoint it returns to our ending point next.

This is a byproduct of using the rmf_fleet_msgs API which has some known issues like what you're describing. Unfortunately there isn't a very good way to fix this behavior because the rmf_fleet_msgs API doesn't communicate enough information to the fleet adapter.
You can consider doing a more direct integration with the fleet adapter API to prevent this kind of issue. This fleet adapter template can help provide a baseline for integrating your fleet.


Within a new environment, the recompile button is disabled while the server_compile setting shows enabled

Problem
Within a new environment, the recompile button is disabled while the server_compile setting shows enabled.
API
A new environment does not have the server_compile setting set.
The backend response does not return this property and is therefor undefined.
web-console
On the configuration tab of the settings page, the server_compile setting is shown as enabled.
This is the default value. The default value is used for undefined settings.
The web-console never uses any of the settings. These settings have no effect on the web-console.
It is just data shown to the user.
Except for the server_compile property. It is used to disable the Recompile button.
Fix
Does the frontend need to use the default value for any settings it uses that are undefined?
Or does the backend need to return default values?

@bartv
Does the frontend need to use the default value for any settings it uses that are undefined?
Or does the backend need to return default values?

This should be implemented in the webconsole


Two Step Method for Adding a new Control Key

Context
Currently the add_public_key_to_msa method adds a control key to an msa. While this works, it requires three signatures:

Signature of a current control key holder
Signature of the new additional control key holder
Transaction signature of the action payee.

Signatures 1 and 2 can be difficult to coordinate. Proposed in the Community Call was adding method that enables separate chain interactions for those two signatures. This follows a pattern akin to the multisig pallet
For additional context and discussion see the Frequency Community Call on 2023-09-28: https://discord.com/channels/969001918460469250/969308206562164736/1157148627987333261
Additional Flow Proposal
(As transcribed by me, any errors are my own)

"Offer Control Step" Current control key holder submits an extrinsic that adds a key to a new "pending" state

via a signature with a possible alternative payee
Call should "prepay" the cost of the second call below

"Claim Control Step" The holder of the new key may then later submit a "prepaid" transaction that completes the action

As a prepaid transaction, the new control key does NOT need any tokens
The offer once claimed should remove the offer and the resulting MSA change should look no different than one that occurred via add_public_key_to_msa

Questions/Notes

This would NOT remove the prior method which may be better for some use cases
Any claim control call that gets into a block that fails must still remove the offer to prevent DOS issues
Do we need to prevent multiple calls in the same block to claim control? (Is this an issue on other "prepaid" calls?)
A way to remove an offer would be required
Do offers need to expire?
Should this be a different pallet? (I think no)

My understanding of what is being asked
Currently, most users are tokenless users who are onboarded by providers via custodial wallet. If a user wants to add a new key to their account it would require some coordination between the custodial wallet and the wallet where the new key is stored. For example, a UI that is integrated with both the custodial wallet and the wallet of the new key.
What is needed to be signed with the keys stored in custodial is the extrinsic add_public_key_to_msa
this extrinsic requires:
- Proof of ownership over the new key.
- Proof of ownership over the msa.
- A key that is funded to pay for the transaction.
We can decouple this coordination by allowing anyone to be able to PROPOSE a new key to be attached to an account.
The proposer is required to leave a deposit in addition to paying the transaction.
The owner of the MSA can later approve the proposal to add a new key free of cost.
Some thoughts
Some decisions to make:
who initiates the proposal the owner of the MSA or anyone?
If it is the owner who requests to add a new key it would require the owner to have a funded key which is not the case for tokenless users.
If anyone can request a key to be added to an account a user may accidentally add a key to an account that is not wanted. It would be easy for anyone to add a key making it difficult for the user to know which one to accept to add.
Potential stories WIP
A User wants to add a new key to an account that they do NOT own.
A user is requesting access to the MSA account

potential future features would be to also include permissions

A user wants to add a new key via a two-step claim and verification process.
A user can create a claim leaving a deposit that it wants to add a new key.
It claims that a new key should be added to an MSA and wants to add a key to it.
This claim provides proof that it has ownership over a key.
The owner or authorized provider can approve the key.
As a user I want to be able to add a new key to an MSA

the user provides the msa-id for which they want to associate the account
the user provides proof of key ownership
The user leaves a deposit for adding a key
note: giving authority to the service provider to accept a claim can have the risk of a service provider hijacking an account.

Possible ways to confirm:
The transaction is submitted by anyone
As a user of frequency who has Capacity,
I want to be able to submit and pay for an add-key claim on behalf of a user.

user providers proof of ownership over MSA by signing.
the user provides the key they are allowing to be added
-- Proof ownership of new key: part of the claim
--proof of MSA ownership

The transaction is submitted by the owner of the MSA
As an owner of MSA, I want to be able to submit transactions and verify an add key claim.

user submits transaction with a key associated with an MSA to prove ownership
The user submits the key that is being added

Closing as stale


Color4.FromHexString default alpha to 1

Patch to default alpha to 1 when a 7 char hex string is passed to Color4.FromHexString, to allow standard hex strings to be copied and pasted from color table websites, etc., more conveniently without having to add "ff" to the string each time.
When an empty or invalid string is passed, all 4 components still default to 0, in keeping with the original behavior.
Forum posts with more info on the use case for this are here: https://forum.babylonjs.com/t/default-alpha-to-1-for-color4-fromhexstring/25226/3

Fixed the whitespace errors.

Reopened:)


highlight plugin issues an event which throws an exception

This has been reported before, by me, so this is a duplicate.  This is a placeholder for a corresponding pull request.
    originalEmit.apply(this, args);

Could you please reference your original issue? thanks! :)


Expose the ability to set the branching priority in pywraplp

What language and solver does this apply to?
C++ Linear Solver
Describe the problem you are trying to solve.
I'd like to add branching priority to certain variables. C++ API:
https://developers.google.com/optimization/reference/linear_solver/linear_solver/MPVariable
Describe the solution you'd like
Something along the lines of:
var.branching_priority(priority)

Describe alternatives you've considered
No alternatives, I'm afraid.
Additional context
...

@lperron I see you edited my question. Just to be clear, I'd like this functionality to be available in Python.

But you write C++ api in your description? And you listed all solvers.
Please be precise.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le lun. 6 juil. 2020 à 11:17, snedeljkovic<EMAIL_ADDRESS>a
écrit :

@lperron https://github.com/lperron I see you edited my question. Just
to be clear, I'd like this functionality to be available in Python.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/2094#issuecomment-654116938,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3L5V373B4CXWKSEXVLR2GJB3ANCNFSM4OQ454OA
.

@lperron Sorry, my intention was to point you to the C++ implementation and suggest you expose that to Python.

Now it is clearer.
I pushed the feature on the master branch,

@lperron Thank you very much!

@lperron I actually need to set the branching priority before solving, The method you added is just a getter. Sorry if I wasn't clear enough (I did mention the ability to set the priority in the title).

I added both I believe.
Le lun. 6 juil. 2020 à 12:29, snedeljkovic<EMAIL_ADDRESS>a
écrit :

@lperron https://github.com/lperron I actually need to set the
branching priority before solving, The method you added is just a getter.
Sorry if I wasn't clear enough (I did mention the ability to set the
priority in the title).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/2094#issuecomment-654151277,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3PHTAXWZ6CWA32QW2DR2GRPFANCNFSM4OQ454OA
.

@lperron You are right. Thank you, again.


Split User and Sneeze Data Types in the Sneezeboard extension

In the external Sneezeboard extension, split out the User and Sneeze data type into separate data engines

Quick clarification: the new api will allow you to serve both users and sneezes from the same data provider engine but with a nicer api.
Moved to sneeze board repo.
Depends on #109


How to use in Angular 2

Hi
Please tell me can i use this package on angular 2+ if YES please tell me how to use?
if NO have any plane for angular 2+?

This is a React component, not an Angular one. Unless Angular 2 can render React components, I don't think this will be possible.
We have no plans to support Angular.

Please one more Question?
Can i use this component like javascript library in none React base project?
If Yes how do that?

No - this is a React component, so you can only use it if you are using React (or a similar React-like rendering engine).

:( :(
Thank you for permanent replying.

Hi! Probably this will help https://github.com/larixsource/angular-react-dates

Thank you a lot.
This is exactly i need.
On Apr 13, 2017 00:15, "Felipe Hernández Lagos"<EMAIL_ADDRESS>wrote:

Hi! Probably this will help https://github.com/larixsource/angular-react-
dates http://url
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/airbnb/react-dates/issues/369#issuecomment-293686952,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABiW8nku28TszREWU5Ya7W9NxppoQLq8ks5rvSn2gaJpZM4Mae_a
.

Hi! Probably this will help https://github.com/larixsource/angular-react-dates

how modify this angular code beacuse i need to add some custom methods and ui?????????????????

Just use https://ng-bootstrap.github.io/#/components/datepicker/overview by import {NgbDate, NgbCalendar} from '@ng-bootstrap/ng-bootstrap';


Consumer already closed with system topic

Describe the bug
Unexpected log output by logger org.apache.pulsar.client.impl.MultiTopicsConsumerImpl, thread: pulsar-client-internal-45-1
Receive operation failed on consumer ConsumerBase{subscription='multiTopicsReader-98cff5df66', consumerName='6a4ac', topic='persistent://c/mot/__change_events-partition-2'} - Retrying later
java.util.concurrent.CompletionException: org.apache.pulsar.client.api.PulsarClientException$AlreadyClosedException: Consumer already closed
	at java.util.concurrent.CompletableFuture.encodeThrowable(CompletableFuture.java:292) ~[?:1.8.0_181]
	at java.util.concurrent.CompletableFuture.completeThrowable(CompletableFuture.java:308) ~[?:1.8.0_181]
	at java.util.concurrent.CompletableFuture.uniAccept(CompletableFuture.java:647) ~[?:1.8.0_181]
	at java.util.concurrent.CompletableFuture.uniAcceptStage(CompletableFuture.java:669) ~[?:1.8.0_181]
	at java.util.concurrent.CompletableFuture.thenAccept(CompletableFuture.java:1997) ~[?:1.8.0_181]
	at org.apache.pulsar.client.impl.MultiTopicsConsumerImpl.receiveMessageFromConsumer(MultiTopicsConsumerImpl.java:248) ~[org.apache.pulsar-pulsar-client-original-2.8.1.jar:2.8.1]
	at org.apache.pulsar.client.impl.MultiTopicsConsumerImpl.lambda$receiveMessageFromConsumer$9(MultiTopicsConsumerImpl.java:274) ~[org.apache.pulsar-pulsar-client-original-2.8.1.jar:2.8.1]
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) [?:1.8.0_181]
	at java.util.concurrent.FutureTask.run(FutureTask.java:266) [?:1.8.0_181]
	at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:180) [?:1.8.0_181]
	at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:293) [?:1.8.0_181]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) [?:1.8.0_181]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) [?:1.8.0_181]
	at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30) [io.netty-netty-common-4.1.66.Final.jar:4.1.66.Final]
	at java.lang.Thread.run(Thread.java:748) [?:1.8.0_181]
Caused by: org.apache.pulsar.client.api.PulsarClientException$AlreadyClosedException: Consumer already closed
	at org.apache.pulsar.client.impl.ConsumerBase.verifyConsumerState(ConsumerBase.java:754) ~[org.apache.pulsar-pulsar-client-original-2.8.1.jar:2.8.1]
	at org.apache.pulsar.client.impl.ConsumerBase.receiveAsync(ConsumerBase.java:162) ~[org.apache.pulsar-pulsar-client-original-2.8.1.jar:2.8.1]
	... 10 more

Screenshots

Additional context

pulsar version: 2.8.1
OS: CentOS Linux release 7.9.2009 (Core)
kernel: 3.10.0-1160.45.1.el7.x86_64

cc @codelipenghui

@Shawyeok Could you give us more detail ?

also experiencing something similar related to systemTopics:
{"msg":"org.apache.pulsar.client.impl.ConsumerImpl - [persistent://tenant/default/__change_events-partition-2][multiTopicsReader-f2ae775e21] Failed getLastMessageId command","thread":"pulsar-io-4-3","timestamp":"07:38:28.863"}
"msg":"org.apache.pulsar.client.impl.ClientCnx - [id: 0x9ad7ffd4, L:/<IP_ADDRESS>:56568 - R:pulsar-broker-3.pulsar-broker.cynet.svc.cluster.local/<IP_ADDRESS>:6650] Received error from server: Failed to read last entry of the compacted Ledger Incorrect parameter input","severity":"WARN"
the broker cannot recover this exception unless I force delete the systemtopic.
the broker gets into that state sporadically.
pulsar broker version: 2.8.2

@mattisonchao @yuvalgut It is duplicate with #12024, and fixed by #12044


Use the LinuxAdmin gem instead of miqnet.sh for system power operations

Power operations don't belong in miqnet.sh.
Allows us to remove stuff from miqnet.sh here: https://github.com/ManageIQ/manageiq-appliance/pull/26
@Fryguy @jrafanie

@gtanzillo @Fryguy please review /cc @jrafanie


Test if docs can be built

Add a travis job for docs. This will make sure we can catch if a PR is likely to break docs

Current coverage is 94.52% (diff: 100%)

Merging #96 into master will not change coverage

@@             master        #96   diff @@
==========================================
  Files             9          9
  Lines           658        658
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            622        622
  Misses           36         36
  Partials          0          0

Powered by Codecov. Last update c147f66...a2eb6c3

you have a conflict. I guess now that you are fixing the conflict you can remove all unnecessary codecov commands. Up to you though. this look good to me after fixing the conflict.

Ack @mbohlool fixing it while i fix the conflicts in this PR

@mbohlool - rebased. Let's get this in please.

Great. Thanks.


Difference to UISplitViewController

I'm curious and I developed some iPhone Apps a few years ago. Currently I plan to work on a new project and need something like the PKRevealController. But one question I came up with is: What are the benefits of using PKRevealController instead of the UISplitViewController, which can nowadays also be used on IPhones?
This is more of question than a real issue, I am sorry for that, but I didnt find a proper place to ask. On StackOverflow there does not seem to be a real discussion about this and it might be useful if the PKRevealController documentation might state the difference to UISplitViewController.

The quick answer is that they do different things. One shows the shelf with an arbitrary view controller. The UISplitViewController is designed around a master-detail data refinement process. Yes, you can use it like PKRevealController. But it is different and may not be what you want from a UI.
Anon,
Andrew


Fail to build nextjs app since 20240407.1.0

Description
Hi,
I just don't get what is going on since a week and the new runner image 20240407.1.0 with ubuntu-22.04 but I am no longer able to build my nextjs app...
Error looks around the locate detection :

Please note we add no error with runner image 20240324.2.0 and the same build that pass with this image, no longer build with the new image. No change on our side. (I rerun an old build to be sure the problem was the runner image)
Platforms affected

[ ] Azure DevOps
[ ] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[X] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[ ] macOS 13
[ ] macOS 13 Arm64
[ ] macOS 14
[ ] macOS 14 Arm64
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
20240407.1.0
Is it regression?
20240324.2.0
Expected behavior
I expect my nextjs app build
Actual behavior

Repro steps
I will try to isolate the issue but I have no idea for now what is going on and how an update of runner image could break a nextjs build.

Unfortunately it is a private repository. I am working on a repro step right now

After hours isolating the problem I was able to confirm that the problem was linked to the tricks in the nextjs doc to redirect our default locale to /en.
So we end up with a default locale in our getStaticProps, which up until now hasn't caused any problems because the result of the locale negotiation is that we end up with the default locale.
It seems that the problem is linked to formatjs and i18n and the way of determining whether a locale is valid or not.
I imagine it must be based on a system call and that's why we got the error on the new image and on a new docker image.
Sorry for the inconvenience!


Added some error handling ideas

This research was done as part of issue [#20].
(https://github.com/CDLUC3/dmsp_backend_prototype/issues/20)
I made some updates as examples of how we can handle errors:

For graphQL requests:
-- Updated lib/graphql/client.ts with adding an errorLink to the client instance, and intercepting specific error types or codes to handle them differently if we want.
-- Updated lib/graphql/server/queries/dmpDataQueries.ts to 1) Map the error code to a user friendly message using a hash like utils/userFriendlyErrorMessage, log the error to the new utils/logger.ts, and throw an error that will be caught by the "try-catch" in the client- or server-side component. logger.ts can be used directly in client- and server-components as well.
-- Updated dmps/[...slug]/page.tsx to throw the returned error in the 'try-catch'
-- Installed pino and @elastic/ecs-pino-format to output formatted logs
Added app/not-found.tsx to handle 404 errors
Added app/error.tsx to handle server-side errors. Client components will most likely display returned errors in the page using useState().

To test:

Start up app
navigate to http://localhost:3001/dmps/10.48321/D1SP4H
Look at the stack trace, and you will see the log: {"log.level":"error","@timestamp":"2024-05-06T23:13:42.224Z","process.pid":4920,"host.hostname":"CDL-JSHIN-9M","ecs.version":"8.10.0","loggerPath":"getDMSP","message":"Error fetching item from DynamoDB"}

Will go ahead and merge my changes, since I went over the changes with the team.


Where to download torchvision0.5.1 .whl files

Hi,
I would like to download the torchvision 0.5.1 version but I cannot find a source to download the .whl file.
It is not in pypi.org nor in pytorch.org, nor anaconda.org.
I have proxy issues on my computer so I can not use pip command, I need to download the .whl file first.
Can you help me by giving an address of the 0.5.1 version? or some other hint to solve this? Thank you.

Hi,
The 0.5.1 version of torchvision (which has been renamed to 0.6.0) has not been officially released yet, and is in testing phase.
It will be made available next week in the official pytorch channels
I'm closing the issue as such, but let us know if you have further questions


Updated readme with minor changes

Coverage remained the same at 75.342% when pulling 395d7f7e1156d21f0449fe2a47340cbbe9fdd15c on ashish-billore:patch-1 into 9130daea9e95a17bf69110abe9616168ee425dce on contiv:master.


Use TaskGroup to ensure all primary / worker tasks are cancelled on error and panic

Put all JobHandle of primary, worker and consensus into a TaskGroup, so if one task returns an error or panics, the whole group will be cancelled.
If we want to stop the primary or worker, we can just drop the TaskManager associated with the TaskGroup.
For cluster tests, I'm assuming JobHandles will be finish on their own, i.e. checking where a JobHandle is_finished() is equivalent to checking if it has been cancelled.

Another potential refactor is to simplify shutdown handling: instead of each task implement logic to handle ReconfigureNotification::Shutdown, we may be able to handle it in one place to shutdown all tasks.

Sorry for the late review! This LGTM modulo a rebase,

Another potential refactor is to simplify shutdown handling: instead of each task implement logic to handle ReconfigureNotification::Shutdown, we may be able to handle it in one place to shutdown all tasks.
I think the goal of the ReconfigureNotification::Shutdown is to allow more ad-hoc graceful shutdown logic than killing the task.

Makes sense.


CRD managed pods not shown in the UI

What kind of request is this (question/bug/enhancement/feature request):
Enhancement
Steps to reproduce (least amount of steps as possible):
Launch pods managed by a CRD and go to the Workloads page. Can use this example from Elasticsearch (https://www.elastic.co/guide/en/cloud-on-k8s/current/k8s-quickstart.html) by running:
$ kubectl apply -f https://download.elastic.co/downloads/eck/0.9.0/all-in-one.yaml
$ cat <<EOF | kubectl apply -f -
apiVersion: elasticsearch.k8s.elastic.co/v1alpha1
kind: Elasticsearch
metadata:
  name: quickstart
spec:
  version: 7.2.0
  nodes:
  - nodeCount: 1
    config:
      node.master: true
      node.data: true
      node.ingest: true
EOF

Result:
It appears the UI or API will filter out all pods that are not managed by ReplicaSets, DaemonSets, StatefulSets, or unmanaged. If a pod is managed by a CRD, it is not displayed in the UI. The above example will set the following in the pod:
  ownerReferences:
  - apiVersion: elasticsearch.k8s.elastic.co/v1alpha1
    blockOwnerDeletion: true
    controller: true
    kind: Elasticsearch
    name: quickstart
    uid: 59253a84-ba36-11e9-a46f-0af107b47750

Other details that may be helpful:
After running above YAML and also launching an unmanaged pod, see this in CLI:
$ kubectl get pods
NAME                       READY   STATUS    RESTARTS   AGE
quickstart-es-cd28s99bh4   1/1     Running   0          22m
ubuntu-pod                 1/1     Running   0          17m

But see this in UI:

For showing these pods in the UI, would be good to note them as CRD managed and possibly give information on which CRD is managing the pod.
This issue may be somewhat related - https://github.com/rancher/rancher/issues/13323
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
v2.2.7
Installation option (single install/HA):
N/A

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):
Custom

Machine type (cloud/VM/metal) and specifications (CPU/memory):
AWS, t3a.medium

Kubernetes version (use kubectl version):
v1.14.5

Docker version (use docker version):
N/A

Previously fixed in https://github.com/rancher/rancher/issues/15804

Actually, this stills happening.
Deploy etcd-operator
Build a example cluster.
Pods not shown in UI.
How to reproduce:
kubectl create -f https://coreos.com/operators/etcd/latest/deployment.yaml
kubectl create -f https://coreos.com/operators/etcd/latest/example-etcd-cluster.yaml

@superseb So this appears to be a regression/bug and not a feature request, correct?

The same think with official jenkins-operator CRD:
https://github.com/jenkinsci/kubernetes-operator

@SkySonR I tested this on latest rancher, but I can see "jenkins-operator" listed under workloads.
I followed the intstallation instructions here:
https://github.com/jenkinsci/kubernetes-operator/blob/master/documentation/installation.md

@prachidamle That operator is a k8s deployment and shows up fine inthe UI. Believe it will watch for the creation a jenkins crd. Question is, if you create a jenkins crd and the jenkins-operator creates pods, do those pods show in the UI?
Or try out my elasticsearch example in the original post as a test case.

@dnoland1 I tried out the elasticsearch example in the original post. Here is what I see:

I see the "quickstart-es-xyz" pod in the UI in the flat list Or in the "Group by node" view.
I guess since this is not a pod that belongs to a "workload" in Rancher (having type Deployments, DaemonSets, StatefulSets, or CronJobs), it is not listed on "Group By Workload" view. But it is listed in the pods list.

@prachidamle What if you just deploy a simple ubuntu pod using YAML as I did above (see ubuntu-pod in screen capture)? That ubuntu-pod does not belong to a deployment, daemonset, statefulset, or cronjob, but is still displayed in the Group by Namespace/Workload view. Can try this YAML:
apiVersion: v1
kind: Pod
metadata:
  name: ubuntu-pod
spec:
  containers:
  - name: "ubuntu-pod"
    image: "ubuntu:latest"
    command:
      - /bin/sleep
      - "10000"

@dnoland1 I tried that out, so looks like UI is listing the unmanaged pod on the Group by Namespace/Workload view - even if API is not actually listing that unmanaged pod under workloads.
If you see the /workloads API list then the ubuntu unmanaged pod is not present in that output. It is however listed under /pods.
So looks like UI is listing that pod under workloads - if we also want to show the CRD backed pods, we probably need a similar shift on the UI display applied.

@dnoland1 Looks like we set workloadId on the Pod even if there is no workload owning it - that causes UI to not show in the "group by workload" view since associated workload cannot be found.
The fix here is not setting workloadId on the Pod managed by a CRD so that it shows up just like the standalone unmanaged pod.

I see the same issue happening. I have tested with https://www.arangodb.com/docs/3.5/tutorials-kubernetes.html
The pods in Kubernetes dashboard:

... they does not appear on Rancher UI:

I can confirm @averri 's findings. Using ArangoDB Operator to deploy an ArangoCluster works fine, with the exception, that these pods are not shown in the ui.

The bug is reproduced in Rancher:v2.3.3
The steps are the same as the issue provides
Results:

the pods show in the UI in the flat list or in the "Group by node" view, but not in the group by workload or group by namespace/workload view

the bug fix is validated in master-head 509b961a4
The steps are the same as the issue provides
Results:

now we can see the pod under all four views

workloadId is set to NULL


Maintain field value when widget.format_value() is empty

Fixes https://github.com/wagtail/wagtail-condensedinlinepanel/issues/59

I can confirm that this fixes the issue I reported. @kaedroho, could you please review this when you have a chance?

Good catch, thanks!


predicting zero objects always

I am clearly observing decrease in train loss and test loss while training but when I'm running evaluate.py it  just prints ground truth , i have put some print statements and realized that for each image the predict function in evaluate is giving list of zero size implying it has detected zero objects. Help me fix this issue

hello, anybody here ?
Can anybody fix this issue on the basis of such descriptions ?

I have run the model on a random rbg image to check whether it's giving any output from the evaluate.py . It has given the same empty array as output from the predict function . Is everything alright with evaluate.py?

me too

Me too. I am using my own dataset which has 2 classes.  Still predice zero objects.
I did not modify any value in config.py except __C.TRAIN.ANNOT_PATH and __C.YOLO.CLASSES. I used yolov3_coco_demo as my initial weight and the loss on the beginning was 2716. Is the loss normal?

I got this problem, then change self.trainable: True in evaluate.py. In my case, it's solved


[Bug] missing csrf token when serving temporal ui under a sub path

What are you really trying to do?

Describe the bug
Error happens when we try to "Terminate" a workflow on the UI.
The error message is:
"missing csrf token in request header"
Minimal Reproduction

build an image following the directions here: https://github.com/temporalio/ui-server/tree/main/docker#serve-ui-under-a-sub-path
start the temporal ui server
visit the ui page
select a running workflow, go to its details, click the "Terminate" button
the termination would fail with  a message: "missing csrf token in request header".

Environment/Versions

OS and processor: [e.g. M1 Mac, x86 Windows, Linux]
Temporal Version: [e.g. 1.14.0?] and/or SDK version
Are you using Docker or Kubernetes or building Temporal from source?

Additional context

still not able to repro the issue using via both HTTP and HTTPS
Do your UI and API URLs match? ie:

What version of the UI you are running?

Actually is your Auth enabled? testing with auth disabled as then CSRF token is checked

still no repro on v2.9.1. What is your UI version?

@feedmeapples
I think the UI version should be 2.8.x。I built it from master branch when the official release is 2.8.
Auth enabled? No, I did not set such a setting, and I visit the UI without login.
The API url is correct: with the sub path.
What's more, If I forward the web ui service from remote machine to my local machine, and visit the ui from localhost:8080/my-sub-path, then it works.

any chance you could try the same on v2.9.1? as i couldn't repro the issue on this version and there have been some recent changes in CSRF/auth processing

afaik this could be an issue on older versions of the UI, however as of the latest version couldn't repro. Closing unless there is a response


scripting: Add displays to create and config.

attempting to script a windows vm with a display

I would like to thread through the hardware qemu property also

+1 for this feature.
If this is working, I'd appreciate an example usage in apple script format So I can test it.


🐛 [firebase_dynamic_links] Define custom short suffix of dynamic link #11000

I saw related issues: 1182 and 11000 but does really give me a solution...
I'd simply need to custom the suffix of short link, as we can do on the web console:

I've also found this: https://pypi.org/project/firebase-dynamic-links-generator/#description
which uses customPrefix but it doesn't seem to work.
Any suggestion?
Thanks in advance

This is not possible from Dynamic Link API, and Firebase has stated it will only fix critical or security issues for this service as it will be shutdown on August 25, 2025: firebase.google.com/support/dynamic-links-faq


[IT-1124] Create account for iAtlas project

I think this is the way to go for projects like iAtlas... What determines the dependencies that get build into it? Like logging and GuardDuty, etc.

nothing at this moment @ahayden but you are welcome to make suggestions, in issue IT-1101, on what you think should be setup with every new account.


Update writing-mathematical-expressions.md: Unsupported Math Blocks in  Blocks

This issue is about displaying math blocks inside of the details blocks (and potentially other HTML blocks) regardless of the type of math block. This can include blocks such as
$$
\begin{pmatrix}
  0 & 1 \\
  2 & 3
\end{pmatrix}
$$

or blocks written like this:
```math
\begin{pmatrix}
  0 & 1 \\
  2 & 3
\end{pmatrix}
```

This issue is currently blocking the use of math blocks inside of details blocks, as well as other HTML blocks. This limitation prevents users from taking full advantage of these blocks, and should be noted in documentation. To ensure users are not held back, we must actively search for a potential solution to this issue so that math blocks can be displayed in these blocks.

Why:
Closes ISSUE

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[x] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

This issue is about displaying math blocks inside of the details blocks (and potentially other HTML blocks) regardless of the type of math block. This can include blocks such as
$$
\begin{pmatrix}
  0 & 1 \\
  2 & 3
\end{pmatrix}
$$

or blocks written like this:
```math
\begin{pmatrix}
  0 & 1 \\
  2 & 3
\end{pmatrix}
```

This issue is currently blocking the use of math blocks inside of details blocks, as well as other HTML blocks. This limitation prevents users from taking full advantage of these blocks, and should be noted in documentation. To ensure users are not held back, we must actively search for a potential solution to this issue so that math blocks can be displayed in these blocks.

Why:
Closes ISSUE

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[x] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

@ALernety Thanks so much for opening a PR! I'll get this triaged for review :zap:

Hi @ALernety, thanks for the PR! It appears to me that matrices render correctly IF you include a line of whitespace after the HTML tag.
For example,
<details>
<summary>Matrix</summary>

```math
\begin{pmatrix}
  0 & 1 \\
  2 & 3
\end{pmatrix}
```

</details>

Appears to render correctly:

Matrix
\begin{pmatrix}
  0 & 1 \\
  2 & 3
\end{pmatrix}

I wonder if you could check whether this works for your use case too, and update your PR to reflect this if so? ✨

It doesn't look like any updates have been on this for a while, so I'm going to go ahead and close it out 💛
Feel free to ping me here to reopen if it continues to be an issue.

Ty for your help I'm just a little busy have 😳been on GitHub today
On Tue, Sep 5, 2023, 2:14 PM Courtney Wilson @.***>
wrote:

Closed #23234 https://github.com/github/docs/pull/23234.
—
Reply to this email directly, view it on GitHub
https://github.com/github/docs/pull/23234#event-10286669470, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BCJXHM65L3DV6E2JXX22B5DXY5TY5ANCNFSM6AAAAAAT4RR4A4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Warning in Home Assistant

I´m using this great integration in home assistant and I'm thrilled with it. Unfortunately I have the following warning message in my system and hope someone can fix it.
""Protokolldetails (WARNUNG)
Logger: homeassistant.helpers.frame
Quelle: helpers/frame.py:184
Erstmals aufgetreten: 21. September 2024 um 20:01:43 (1 Vorkommnisse)
Zuletzt protokolliert: 21. September 2024 um 20:01:43
Detected that custom integration 'eta' calls async_forward_entry_setup for integration, eta with title: ETA at xx.xx.xx.x and entry_id: 01J30X5E9QTG27SG6F04N1D1S0, which is deprecated and will stop working in Home Assistant 2025.6, await async_forward_entry_setups instead at custom_components/eta/init.py, line 22: hass.async_create_task(, please create a bug report at https://github.com/nigl/homeassistant_eta_integration/issues""
Thanks in advance. Best regards redsun8

Push; has still not been fixed


Fix cron.repo_health_check schedule example in config cheat sheet

Fixes #10274.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@e96f354). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #10324   +/-   ##
=========================================
  Coverage          ?   43.69%
=========================================
  Files             ?      585
  Lines             ?    81015
  Branches          ?        0
=========================================
  Hits              ?    35398
  Misses            ?    41230
  Partials          ?     4387

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e96f354...ce71586. Read the comment docs.


fix: update railway deploy button

It looks like they changed the url to /new/template?template=
Fixes #35

Thanks @jsjoeio. Testing now!

LGTM! Is this ready to go @jsjoeio? I see it is marked as a draft

Whoops, I think so! Meant to mark as ready :D


[FLINK-11235] fix thread lack when elasticsearch transport client init failed

What is the purpose of the change
elasticsearch transport sink init steps
1, create client thread
2, try to check every host:port
3, if each host:port is unreachable, while throw RuntimeException
but, because of throw RuntimeException, the client can not close, so causing thread leak
TransportClient transportClient = new PreBuiltTransportClient(settings);
for (TransportAddress transport : ElasticsearchUtils.convertInetSocketAddresses(transportAddresses)){
    transportClient.addTransportAddress(transport);
}

// verify that we actually are connected to a cluster
if (transportClient.connectedNodes().isEmpty()) {
    throw new RuntimeException("Elasticsearch client is not connected to any Elasticsearch nodes!");
}

return transportClient;

thread leak

Brief change log
close the client before throw RuntimeException
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not documented)

Happy New Year, :v:

here is a reproducible example
StreamExecutionEnvironment env = StreamExecutionEnvironment.getExecutionEnvironment();
env.enableCheckpointing(5000L);

// env.setRestartStrategy(RestartStrategies.fixedDelayRestart(10000, 600L));

DataStreamSource<String> source = env.addSource(new SourceFunction<String>() {
    @Override
    public void run(SourceContext<String> ctx) throws Exception {
        for (int i = 0; i < 100000000; i++) {
            Thread.sleep(1000);
            ctx.collect("data " + i);
        }
    }

    @Override
    public void cancel() {

    }
});

Map<String, String> config = new HashMap<>();
config.put("cluster.name", "elasticsearch");
config.put("bulk.flush.max.actions", "3");

List<InetSocketAddress> transportAddresses = new ArrayList<>();
transportAddresses.add(new InetSocketAddress(InetAddress.getByName("<IP_ADDRESS>"), 9700));

ElasticsearchSink<String> essink = new ElasticsearchSink<>(config, transportAddresses, new ElasticsearchSinkFunction<String>() {
    @Override
    public void process(String element, RuntimeContext ctx, RequestIndexer indexer) {
        HashMap<String, String> data = Maps.newHashMap();
        data.put("key", element);
        indexer.add(Requests.indexRequest().index("index").type("type").source(data));

    }
});

source.addSink(essink);
env.execute();


[Sample Onboard Request] Large Scale Notification Bot

Owner:
@yiqing-zhao
ID:
large-scale-notification
Title:
Large Scale Notification Bot
Description:
This sample demonstrates the architecture of a Teams notfication bot app created by Teams Toolkit to send individual chat messages to a large number of users in a tenant.
Onboard Date:
ASAP
Tags:
You can give your samples a set of tags so that users can easily locate your sample from a group of samples. Those tags can be:

TS
Bot
Azure Durable Functions
Notification
Large user scale
You can customize the tag you used for your sample, the recommended number of tags is less than 5.

Time:
30 mins to run
Configuration:
Manual configurations required
Thumbnail:
https://github.com/OfficeDev/TeamsFx-Samples/blob/yiqingzhao/largescale-notification/large-scale-notification/assets/thumbnail.png
(Optional) Demo GIF:
No GIF
(Source code in this Repo) Source Code PR:
https://github.com/OfficeDev/TeamsFx-Samples/pull/1014
(Source code NOT in this Repo) Sample URL:
The URL if your sample code resides in another sample repo. It's better to use a tag instead of a branch for stablility.
E.g., "https://github.com/xxx/xxx/tree/v1.0/xxx".
(Optional) Minimum Teams Toolkit Version:
V5
(Optional) Maximum Teams Toolkit Version:
Teams Toolkit version requirement to run the sample - maximum version

PR is verified.

The sample passed PM review and passed test.

next step: Add to sample gallery.

Onboarding PR: #1054


high cpu usage + glitching of scrolling

I have 1000 childs (single sprite) in scrollBox.content
Having 100 items is ok.
But If I have 1000 items, the scrolling cause high cpu usage (almost 100%) + very laggy.
How to solve this problem?
I want to have long text lists.

1000 items should be okay based on the type of objects. Perf is very individual for a project. I would recommend staying away from pixi.text (use bitmaptext instead), and if needed, cull items when they are no longer in view to improve performance (ie, set visible = false when they're offscreen). Masks in pixi are a bit more expensive than not having masks, but it should be able to handle 1000 items.


Safari (Mobile on Iphone) has multiple graphical quirks

See attached image:

Note that the text is only half visible (including the required number of achievements to win).
The score and forecast is also cut off.  The score pile isn't displaying in two columns like the screenshots after the recent scoring spacing change.

Put another way, I don't have the experience necessary to get things fixed up for Safari. If we want to make progress on this, I think we need a dedicated developer to help with this. Lordmcfuzz would be a good candidate but he's been super busy lately.

Some of the latest changes should fix some of these issues. I'm going to close this and we can file new issues for specific problems.


Develop proper facial expressions handler – Stint 4

Task description
This is the follow-up of #88
We would like to:

[x] Integrate the new face expression received from rezzonico
[x] Use transitions
[ ] Check the impact of ergocubEmotions on the cpu while for example the realsense is streaming the data
[ ] Get rid of the face-expression service on ergocub-head (cc @davidelasagna)
[ ]  ...

Definition of done
The above list is completed
cc @pattacini

Just received the draft expressions from the designers.

Moved up to the current sprint as @martinaxgloria will start today.

Hi @martinaxgloria
Please, outline here below how you aim to address the further development required to deal with the transitions, both in terms of the handling of conf files and the necessary C++ objects.

Starting from this organization of the conf file which is the one currently used:
ergoCubEmotions/
├── config.ini
└── expressions
    ├── images
    │   ├── exp_img_0.png
    │   └── exp_img_1.png
    │   └── ...
    └── videos
        ├── exp_vid_0.mp4
        └── exp_vid_1.mp4
        └── ...

it could be an idea to add another folder simply called transitions (?) where to store the corresponding videos and it could be organized like:
|
└── transitions
        ├── angry-happy.mp4
        ├── neutral-happy.mp4
        ├── shy-angry.mp4
        ├── angry-shy.mp4
        └── angry-neutral.mp4
        └── ...

The aim of keeping the name of the emotions instead of exp_vid_#.mp4 is because it's easier to map the transition between two images/videos from the code side, but I'm open to better ideas.
The config.ini file, instead, is currently like this:
[general]
num_expressions 4

[expression_0]
name happy
type image
file images/exp_img_0.png

[expression_1]
name shy
type image
file images/exp_img_1.png

...

From this, we can add a section for each transition with name, type and file sections too, to maintain the same structure.
Let me know if it could work or it's better to move to another direction.
cc @Nicogene @pattacini

We need to manage transitions appropriately inside our code, hence we cannot get away simply with image and video types. Thereby, we're required to have a third type, like transition.
Even better, we can have in general num_expressions and num_transitions, so that we'll have:
[general]
num_expressions 4
num_transitions 10

[expression_0]
//...

[transition_0]
//...

An item of the set transition doesn't need a type attribute (it'll always be a video). Instead, it requires two specific attributes:

source, pointing to the corresponding expression, e.g., expression_0.
destination, pointing to the corresponding expression, e.g., expression_1.

When reading up the conf file, we need to:

check if the source and destination point to available expressions.
build a further "sort of map" for transitions linking sources to destinations.

Give this structure, the code will follow.
Finally, regarding the file structure, I think we can either stick to the images and videos sub-directories or you can add up the transitions sub-directory, as you like.

Even better, we can have in general num_expressions and num_transitions

Of course, you're right!

An item of the set transition doesn't need a type attribute (it'll always be a video). Instead, it requires two specific attributes:

source, pointing to the corresponding expression, e.g., expression_0.
destination, pointing to the corresponding expression, e.g., expression_1.

You're right about the type attribute. Regarding the source and destination, I was thinking to retrieve them directly from code by storing the command set during the previous iteration and the current setting (this is the reason why I would like to keep the name of the file like angry-happy, so it's easier to map them using the ResourceFinder methods like findFile). In this way, it's not necessary to build another "sort of map" from source to destination.
Do you think this is not a good approach?

We had a meetup w/ Rezzonico yesterday and we decided that:

We will send them a sample of our OLED.
They will send us the second batch of expressions+transitions.

We will send them a sample of our OLED.

https://github.com/icub-tech-iit/tickets/issues/2827
https://github.com/icub-tech-iit/tickets/issues/2826

I pushed on my fork the changes to the module in order to include the transitions from one "static" video to the other one. I've put aside the images for now (they are still present in the configuration files but as comments) and I worked with videos only.
Tomorrow, I will put here a brief video to show the results.
cc @pattacini @Nicogene

Cool!
However, consider that image->transition->image will be most likely our standard pipeline, rather than video->transition->video.

Yes, in fact they can be both displayed in the module but since the images we had are very different in quality and resolution with respect to the video received in the past days from Rezzonico, I preferred to have a good graphic rendering. Moreover, I wanted to test the video->transition->video which should be the most "tricky" case.
For the sake of completeness, tomorrow I will do the tests with both cases.

Here the result! Sorry for the quality of the video but the videos we have now are overly bright 😅
https://user-images.githubusercontent.com/114698424/232729774-8efef1aa-6d5a-41c4-8956-b4a450bfc33a.mp4
cc @Nicogene @pattacini @maggia80

Awesome!

The vademecum: vademecum_oled_ergocub.docx.

@martinaxgloria, I think you can open up the corresponding PR and link it to this issue.

PR opened here:

https://github.com/icub-tech-iit/ergocub-software/pull/94

cc @Nicogene @pattacini

Did you already check some borderline conditions such as:

num_transitions is higher than the actual number of inputs.
missing [transition_#] group.
missing videos and/or files.
wrong values for source and/or destination fields.
unallowed transition requested via RPC.

num_transitions is only used to fill the corresponding std::map in order to associate the "source" and the "destination" emotions.
it is related to the first answer.
for both images and videos, if the reading results in an empty file, it returns an error message.
what do you mean? I didn't get it
transitions are not required through RPC, the setEmotion service works by passing the expression to be set (for example happy) and inside the code there is a thread that handles the transition between the previous command and the current one.

When you process num_transitions from the conf file, you ought to parse the conf file to get the information. In this case, if num_transitions is for example 5 and you have only [transition_#] with # in {0..3}, you must raise an error.
Likewise, if num_transitions is 5 but the conf file contains only [transition_#] with # in {0, 1, 3, 4}, you must raise an error.

odd values for source and/or destination fields.

A transition has one or both of its fields source and/or destination pointing to non-existing expressions. In this case, you must raise an error.

unallowed transition requested via RPC.

 This cannot happen.

Ok, I have to handle this case since I didn't take into account this possibility.

I'd extend this check also to num_expressions then.

For now, if we are in this case, nothing happens. But of course, I can raise an error when it happens.

👍🏻 yep, it'd be a good addition to make the SW more robust.

When you process num_transitions from the conf file, you ought to parse the conf file to get the information. In this case, if num_transitions is for example 5 and you have only [transition_#] with # in {0..3}, you must raise an error at configuration time.

I was reading again this comment and I have a doubt: if num_transitions is higher than the number of the actual inputs (for example, num_transitions = 4 and the inputs are actually 3), it happens that transitionMap (which is the std::map that takes care of the transition files) will be filled in the first 3 let's say "positions" and the 4th will be empty.
Instead, if the opposite case occurs, for example num_transitions = 3 and inputs = 4, the transitionMap will be fully filled but one transition remains out since all the map positions are taken.
Based on that, should we handle both cases?
cc @pattacini @Nicogene

if num_transitions is higher than the number of the actual inputs (for example, num_transitions = 4 and the inputs are actually 3), it happens that transitionMap (which is the std::map that takes care of the transition files) will be filled in the first 3 let's say "positions" and the 4th will be empty.

This is clearly an error: you have to print out a message and quit the program.

Instead, if the opposite case occurs, for example num_transitions = 3 and inputs = 4, the transitionMap will be fully filled with the first 3 inputs but one transition remains out since all the map positions are taken.

This is not an error. The only weirdness is that the conf file contains more transitions than those specified in [general]. That's ok and it may happen during debugging.

This is clearly an error: you have to print out a message and quit the program.

Clear now, thanks!

Just as a general remark, we're required to handle both cases below:

Expression_1 → Expression_2
Expression_1 → Transition_12 → Expression_2

When you process num_transitions from the conf file, you ought to parse the conf file to get the information. In this case, if num_transitions is for example 5 and you have only [transition_#] with # in {0..3}, you must raise an error at configuration time.
Likewise, if num_transitions is 5 but the conf file contains only [transition_#] with # in {0, 1, 3, 4} (i.e., transition 2 is missing), you must raise an error.

About this, instead, I thought to put together these two cases (because it's the same problem but in the first case it happens in the middle, while in the other at the end) and raise an error with the number of the corresponding expression/transition that is missing and then quit the program. What do you think about?
cc @Nicogene @pattacini

Agreed 👍🏻
It's the same case.
You will have to make a for cycle from 0 to num_transitions-1 and during each iteration i you'll quit if you don't find the corresponding transitions_i.

Today I did a test in order to check the impact of ergoCubEmotions while the realsense was streaming data.
I post here the yarpview of both rgb and depth of the realsense while the module is running:

and the performances of the Jetson Xavier by running jtop: asyou can see, the cores are not saturated when the module and the realsense streaming run at the same time.

cc @pattacini @Nicogene

Let's head to the last 2 points:

Regarding the official delivery, I would:

advertise the functionality in Teams (we have a dedicated channel for ergoCub)
in the announcement, give availability for in-person explanation/support

Let's head to the last 2 points:

For the first point, I will ask @davidelasagna when he's available to do it.

About the official delivery, before doing it I would also add the yarpmanager application on ergocub laptopo in order to make it ready to use(obviously testing that that app works) and I would add some example of change of expressions in the readme using the video we already got. I can add those points to the issue.
Moreover we have to see after removing the service if we can avoid the display the nvidia symbol when the module is not running

I update the README.md with a video sample of the module in action.

Today, @davidelasagna and I did some tests on ergoCub in order to understand how to deal with the ergoCubEmotion module when the robot starts. After some attempts, in the end, we:

kept the service that was running on the ergocub-head so far, but we modified it so that only one expression (the "neutral" one) is shown until the module is launched;
since the service runs at the start, I modified the module by removing the piece of code that did the same thing;
changed the background with the same image mentioned in the previous point, so that if the module is closed or crashes, an expression continues to be displayed;
for the same reason as before, we clean up the desktop;
tried to launch the entire system from the beginning and tried to show the facial expressions successfully.

Now, the last step before the official release is to try to launch the module from the yarpmanager since I didn't have time today, but the application file is already implemented.
I also opened a PR in draft since the last tests have to be done.

https://github.com/icub-tech-iit/ergocub-software/pull/98

cc @Nicogene @pattacini

Nicely done!

since the service runs at the start, I modified the module by removing the piece of code that did the same thing;

We shouldn't adapt the module to the service. I'm in favor of leaving it as there wouldn't be a service running underneath. There's absolutely no problem in spotting an image switch when we launch the module.

changed the background with the same image mentioned in the previous point, so that if the module is closed or crashes, an expression continues to be displayed;

Don't know if this is desirable. If we close the module or the module itself crashes, there's no problem in seeing the desktop.
Also, in my understanding, the service is still required only for shadowing any pop-up messages while the module is running.

Nope, the service shadows the pop-up windows when the module is not running, so at the start-up.
After that, when the module is running, it takes control of the display in spite of the service and then it's the module that takes care of shadowing the unwanted messages that appear on the screen.

Ok sorry, I didn't catch this point.
Then, I reckon we no longer need the service.

After some tests on ergoCub and the integration of the application in yarpmanager, the PR

https://github.com/icub-tech-iit/ergocub-software/pull/98

is now ready for the review and for the official release.

Hi @martinaxgloria
Once the PR is merged, please fix the cluster XML file in robots-configuration@master.
Finally, we can announce that the module is available.

Once the PR is merged, please fix the cluster XML file in robots-configuration@master.

See:

https://github.com/robotology/robots-configuration/pull/500#pullrequestreview-1405738864

Very last point:

Announcing the delivery on MS Teams.

Announcement done! 📣


Fix grammar of email and add missing link.

@mondiaz & @kaileychen check text please!

@caesar2164  screenshot is no longer accurate.
👍  based on the "remove hyphen" link that takes me to the "files changed" page at https://github.com/Stanford-Online/openedx-themes/pull/79/commits/7e37b27dbd73e549e40823f483ce03c7f8608a3e.


Grafana "Outbound Deployments" count seems off

Looking at the Grafana dashboard for a deployment, it shows 216 outbound deployments:

However, this particular deployment only connects to 2 other services (deployments), totalling 18 pods.
So either the reported number is off, or what the number actually means is mislabeled. Is it the number of active outbound connections?

Thanks for the report @bourquep. This is unexpected. That stat is based on the following Prometheus query (templating via Grafana):
count(request_total{namespace="$namespace", deployment="$deployment", dst_deployment=~"$outbound", direction="outbound"})

I suspect this may be related to #705. In your deployment dashboard, browse to the Grafana Dashboard Settings in the upper-right, then click Variables. Browse through the four variables, note the Preview of values section at the bottom. If you see a variable with a lot of values, or an empty value, that's likely the culprit.
Also helpful in investigating these types of issues, connect directly to Prometheus to run queries:
$ kubectl -n conduit port-forward $(kubectl --namespace=conduit get po --selector=conduit.io/control-plane-component=prometheus -o jsonpath='{.items[*].metadata.name}') 9090:9090 &
$ open localhost:9090


Use #ffffff instead of #fff for foreground color as the short form seems to mess up.

Hey @rauchg,
This PR fixes https://github.com/zeit/hyperterm/issues/177 and https://github.com/zeit/hyperterm/issues/199, which looks like a dupe of the former.
It seems that for some reason, using #fff as the hex value for foreground correctly makes the text white, but highlighted texts are in a barely readable gray color. Using the full form #ffffff or white instead correctly makes both text and highlighted text white.
Before:

After:

Please check it out, thanks!

Amazing. How did you find this?
My only concern is that folks will definitely try the short form and get disappointed. Maybe we should convert them when we're about to pass foregroundColor to hterm in the term.js Component?
Thank you so much!

Confirmed that it works. Miraculous.

Works for me too! Would have never guessed this was the issue, nice find :+1:
Could use the color module here to parse the color and always use the long form.
import Color from 'color';

const longFormHex = Color('#333').hexString();

Really sweet, specially since it seems that we'll be using that module for the other PR. I haven't checked, but I hope the color module is minimal (we're very sensitive to code size for startup time).

I was getting annoyed with the grey color when using it for path
autocompletion, so I tried to track down the source of that color to no
avail. At one point I experimented with changing foreground and background
colors and I used 'white' instead of '#fff', and noticed this behavior.
I agree with using the Color module, I'll update this PR or make a new
one tomorrow if you want to merge this before then.
Thanks for the feedback and for your work on this awesome project!
El jul. 17, 2016 23:49, "Guillermo Rauch"<EMAIL_ADDRESS>escribió:

Amazing. How did you find this?
My only concern is that folks will definitely try the short form and get
disappointed. Maybe we should convert them when we're about to pass
foregroundColor to hterm in the term.js Component?
Thank you so much!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@rauchg I ran webpack --json > stats.json and dropped that into webpack visualizer
In it's current form it amounts to 0.68% of the project at 10.5k

React (for comparison) takes up 41% at 636k

Awesome :)

This seems like a underlying issue elsewhere, doesn't it?

@devinus definitely. That's why I want to post-process the color rather than having the user be surprised when they use a shorthand form.

@rauchg Totally, but what I meant is should we figure out why the shorthand form is causing this?

@devinus sure, intrigued about that as well :D

@rauchg @devinus I submitted a PR with the problem and solution in https://github.com/zeit/hyperterm/pull/272. It would make this PR obsolete.

Thanks!


Allow omitting platform dependencies from published POM and module metadata

When configuring a publication to publish resolved dependency versions, it would be great to have an option to exclude platform dependencies to be published in the generated POM and Gradle module metadata, in particular if the platform is a local subproject that is not intended to be published.
Here's a branch of the junit5 project:
https://github.com/junit-team/junit5/commit/8fdd62d1e07000ffa002fdfa9e9c3bdf33de3134
It defines a dependencies subproject that uses the java-platform plugin and uses the maven-publish plugin to publish other subprojects where :dependencies is used as a platform dependency. The MavenPublication is configured to publish the resolved versions.
Expected Behavior
Since we don't want to publish the documentation subproject, the generated POMs should not reference it.
Current Behavior
The POM contains a dependencyManagement block that imports the documentation BOM.
Context
Hacky workaround: remove dependencyManagement from the generated POM manually.
pom.withXml {
	val root = asNode()
	val nodes = root["dependencyManagement"] as NodeList
	if (nodes.isNotEmpty()) {
		root.remove(nodes.first() as Node)
	}
}

If we implement this, I think it should become a generic "filter out this dependency" thing, because there's no difference between a platform dependency and any other dependency in this context: removing it may break consumers, so it's a "use this at your own risk" problem.

We should document the current solution in the user manual and maybe as a sample.

Thanks @jjohannes for the solution. This is very helpful.
For anyone else who runs into this, our team had a slight extension to this issue where we were trying to use platform constraints for custom variants and source sets. The fromResolutionResult() declaration appears to try to resolve versions using the default configurations ("java-api" -> compileClasspath and "java-runtime" -> runtimeClasspath), regardless of the component configuration. This was not succeeding for us since our variants do not use the standard resolvable configurations (e.g., our "java-client" variant will use the "javaClientCompileClasspath" instead). We were able to address this by selecting the appropriate configurations:
publishing {
    publications {
        create<MavenPublication>("javaClient") {
            artifactId = "my-project-java-client"
            from(javaClientComponent)
            versionMapping {
                usage(Usage.JAVA_API) {
                    fromResolutionOf(javaClientSourceSet.compileClasspathConfigurationName)
                }
                usage(Usage.JAVA_RUNTIME) {
                    fromResolutionOf(javaClientSourceSet.runtimeClasspathConfigurationName)
                }
            }
        }
    }
}

Can gradle provide native support for "dependency filter" when publishing? Meanwhile, thank @jjohannes for the solution. It really saved me a lot of time.

If you want to remove all the dependencyManagement you can do:
publishing {
    publications {
        mavenJava(MavenPublication) {
            from components.java

            pom {
                withXml {
                    Node pomNode = asNode()
                    pomNode.remove(pomNode.dependencyManagement)
                }
            }

        }
    }
}

Also possible to be more selective, like this:
 withXml {
	  Node pomNode = asNode()
	  pomNode.dependencyManagement.dependencies.dependency.each() { node ->
		  logger.warn("${node.groupId.text()}")
		  if(node.groupId.text()?.startsWith("org.springframework")) {
			  node.parent().remove(node)
		  }
	  }
  }

@luisalves00 Your solution will create a discrepancy between the POM file and the Gradle Module Metadata, so it should not be used as such.

@jjohannes is the versionMapping / usage stuff really required? For our use-case where I'd like to locally run a CLI app with plugins in the classpath for development, but the CLI app should not depend on any plugins, I'm just doing this (requires Gradle 8.4 for dependencyScope):
configurations.dependencyScope("pluginClasspath")
configurations["runtimeClasspath"].extendsFrom(configurations["pluginClasspath"])

dependencies {
    "pluginClasspath"(platform(project(":plugins:commands")))
}

@sschuberth everything in the publishing block is only needed if you care about publishing versions of third-party dependencies as part of the metadata of your own components. If your use case does not involve publishing, you do not need that.

If your use case does not involve publishing, you do not need that.

It does involve publishing, though not third-party dependencies, but just my own project's artifacts.

In that case, you also do not need the versionMapping configuration code. It only concerns the versions of already published third-party components. The versions of your own components are always added to the metadata.


Multiple links in message - Slack sanitization

Proposed changes:

This PR is part of Slack Sanitization #4418 . PR #4475 taken care only one link instead of converting
multiple links if present.

Status (please check what you already did):

[X] made PR ready for code review
[ ] added some tests for the functionality
[ ] updated the documentation
[X] updated the changelog
[ ] reformat files using black (please check Readme for instructions)

@tmbo can you let me know where i need to do that?

@vigneshp826 you could add examples here: https://github.com/RasaHQ/rasa/blob/master/tests/core/test_channels.py#L464

@vigneshp826 are you still planning on finishing this PR?

@erohmensing little bit busy in my BAU activities. Will try t complete next week. Sorry for the delay

Okay, just wanted to check it wasn't going stale.
The mechanism for changelog has changed since, please remove your changelog file and follow these instructions


ailab, azuredatastudio, winfile file

Added files

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

:+1:


fix(ProtoBuff): component evidence should be optional, istead of repeated

non-breaking fix of #422
in contrast to #425

merging , as agreed in CoreWorkingGroup meeting on 2024-09-02


Support gradle task caching

Expected Behavior
Detekt Gradle task should pick cached task result and use it for the same inputs.
Current Behavior
On each run detektCheck it executes checking the code for the same input.
Steps to Reproduce (for bugs)
Run several times ./gradlew detektCheck on any project with enabled Gradle caching support, observe X actionable tasks: X executed
Context
Will speed up local and CI detekt check.
Your Environment

Version used: 1.0.0.RC6-3
Operating System and version: Gentoo linux
Link to your project: private project

This should also be handled by #771.

Closing this as #1079 was merged


Clustered postgresql charm does not update its relations when the Primary unit changes.

Steps to reproduce

Currently, we're using the postgresql charm (number of units: 3, channel: 14/stable, rev: 351) in relation with the landscape-server charm. Everything works fine, the postgresql unit Primary changes. From then on, the related application crashes, as it is now connected to a read-only PostgreSQL database. Note the error storm.database.ReadOnlySqlTransaction: cannot execute UPDATE in a read-only transaction below:
Feb 20 08:16:08 landscape-2 message-server-1[1914]: Got error while handling message\nTraceback (most recent call last):\n  File \"/usr/lib/python3.10/threading.py\", line 1016, in _bootstrap_inner\n    self.run()\n  File \"/usr/lib/python3.10/threading.py\", line 953, in run\n    self._target(*self._args, **self._kwargs)\n  File \"/usr/lib/python3/dist-packages/twisted/_threads/_threadworker.py\", line 47, in work\n    task()\n  File \"/usr/lib/python3/dist-packages/twisted/_threads/_team.py\", line 182, in doWork\n    task()\n--- <exception caught here> ---\n  File \"/usr/lib/python3/dist-packages/twisted/python/threadpool.py\", line 244, in inContext\n    result = inContext.theWork()  # type: ignore[attr-defined]\n  File \"/usr/lib/python3/dist-packages/twisted/python/threadpool.py\", line 260, in <lambda>\n    inContext.theWork = lambda: context.call(  # type: ignore[attr-defined]\n  File \"/usr/lib/python3/dist-packages/twisted/python/context.py\", line 117, in callWithContext\n    return self.currentContext().callWithContext(ctx, func, *args, **kw)\n  File \"/usr/lib/python3/dist-packages/twisted/python/context.py\", line 82, in callWithContext\n    return func(*args, **kw)\n  File \"/usr/lib/python3/dist-packages/storm/twisted/transact.py\", line 78, in _wrap\n    result = function(*args, **kwargs)\n  File \"/opt/canonical/landscape/canonical/landscape/message/resource.py\", line 136, in _handle_request\n    result = self._message_system.dispatch(request)\n  File \"/opt/canonical/landscape/canonical/message/system.py\", line 73, in dispatch\n    return message_api.run()\n  File \"/opt/canonical/landscape/canonical/landscape/message/apis.py\", line 265, in run\n    self.transaction.commit()\n  File \"/usr/lib/python3/dist-packages/transaction/_manager.py\", line 257, in commit\n    return self.manager.commit()\n  File \"/usr/lib/python3/dist-packages/transaction/_manager.py\", line 135, in commit\n    return self.get().commit()\n  File \"/usr/lib/python3/dist-packages/transaction/_transaction.py\", line 282, in commit\n    reraise(t, v, tb)\n  File \"/usr/lib/python3/dist-packages/transaction/_compat.py\", line 45, in reraise\n    raise value\n  File \"/usr/lib/python3/dist-packages/transaction/_transaction.py\", line 273, in commit\n    self._commitResources()\n  File \"/usr/lib/python3/dist-packages/transaction/_transaction.py\", line 465, in _commitResources\n    reraise(t, v, tb)\n  File \"/usr/lib/python3/dist-packages/transaction/_compat.py\", line 45, in reraise\n    raise value\n  File \"/usr/lib/python3/dist-packages/transaction/_transaction.py\", line 437, in _commitResources\n    rm.tpc_begin(self)\n  File \"/usr/lib/python3/dist-packages/storm/zope/zstorm.py\", line 298, in tpc_begin\n    self._store.flush()\n  File \"/usr/lib/python3/dist-packages/storm/store.py\", line 511, in flush\n    self._flush_one(obj_info)\n  File \"/usr/lib/python3/dist-packages/storm/store.py\", line 568, in _flush_one\n    self._connection.execute(expr, noresult=True)\n  File \"/usr/lib/python3/dist-packages/storm/databases/postgres.py\", line 306, in execute\n    return Connection.execute(self, statement, params, noresult)\n  File \"/usr/lib/python3/dist-packages/storm/database.py\", line 323, in execute\n    raw_cursor = self.raw_execute(statement, params)\n  File \"/usr/lib/python3/dist-packages/storm/database.py\", line 457, in raw_execute\n    self._run_execution(raw_cursor, args, params, statement)\n  File \"/usr/lib/python3/dist-packages/storm/database.py\", line 471, in _run_execution\n    self._check_disconnect(raw_cursor.execute, *args)\n  File \"/usr/lib/python3/dist-packages/storm/database.py\", line 537, in _check_disconnect\n    return function(*args, **kwargs)\n  File \"/usr/lib/python3/dist-packages/storm/database.py\", line 186, in wrapper\n    with wrap_exceptions(self._database):\n  File \"/usr/lib/python3.10/contextlib.py\", line 153, in __exit__\n    self.gen.throw(typ, value, traceback)\n  File \"/usr/lib/python3/dist-packages/storm/exceptions.py\", line 184, in wrap_exceptions\n    six.raise_from(wrapped.with_traceback(tb), e)\n  File \"<string>\", line 3, in raise_from\n    \n  File \"/usr/lib/python3/dist-packages/storm/exceptions.py\", line 165, in wrap_exceptions\n    yield\n  File \"/usr/lib/python3/dist-packages/storm/database.py\", line 187, in wrapper\n    return attr(*args, **kwargs)\nstorm.database.ReadOnlySqlTransaction: cannot execute UPDATE in a read-only transaction\n\n
Feb 20 08:16:08 landscape-2 message-server-1[1914]: \"<IP_ADDRESS>\" - - [20/Feb/2024:08:16:08 +0000] \"POST /message-system HTTP/1.1\" 500 - \"-\" \"landscape-client/18.01-0ubuntu13\"

Or in one of its other services:
Package Search daemon.
-search 2024/02/20 08:28:36 Starting up, revision
-search 2024/02/20 08:28:36 pq: cannot set transaction read-write mode during recovery
search.service: Main process exited, code=exited, status=1/FAILURE
search.service: Failed with result 'exit-code'.
search.service: Consumed 1.150s CPU time.

After some investigation, we've seen the following landscape-server relation data:
juju show-unit -m landscape-maas landscape-server/0
landscape-server/0:
  ...
  - relation-id: 10
    endpoint: db
    related-endpoint: db-admin
    application-data: {}
    related-units:
      landscape-postgresql/0:
        in-scope: true
        data:
          allowed-subnets: <IP_ADDRESS>/32,<IP_ADDRESS>/32,<IP_ADDRESS>/32
          allowed-units: landscape-server/0 landscape-server/1 landscape-server/2
          database: landscape-server
          egress-subnets: <IP_ADDRESS>/32
          host: <IP_ADDRESS>
          ingress-address: <IP_ADDRESS>
          master: dbname=landscape-server host=<IP_ADDRESS> password=removed
            port=5432 user=relation-10
          password: removed
          port: "5432"
          private-address: <IP_ADDRESS>
          schema_password: removed
          schema_user: relation-10
          standbys: dbname=landscape-server host=<IP_ADDRESS> password=removed
            port=5432 user=relation-10,dbname=landscape-server host=<IP_ADDRESS> password=removed
            port=5432 user=relation-10
          state: master
          user: relation-10
          version: "14.9"
      landscape-postgresql/1:
        in-scope: true
        data:
          egress-subnets: <IP_ADDRESS>/32
          ingress-address: <IP_ADDRESS>
          private-address: <IP_ADDRESS>
      landscape-postgresql/2:
        in-scope: true
        data:
          egress-subnets: <IP_ADDRESS>/32
          ingress-address: <IP_ADDRESS>
          private-address: <IP_ADDRESS>

As it can be seen from the relation data, the master is: "master: dbname=landscape-server host=<IP_ADDRESS>". Though, it is not. If we check the model, we see this:
juju status -m landscape-maas landscape-postgresql
Model           Controller            Cloud/Region  Version  SLA          Timestamp
landscape-maas  juju_maas_controller  maas_cloud    3.3.1    unsupported  08:40:49Z

App                   Version  Status  Scale  Charm             Channel    Rev  Exposed  Message
landscape-client               active      3  landscape-client  stable      69  no       Client registered!
landscape-postgresql  14.9     active      3  postgresql        14/stable  351  no
ubuntu-advantage               active      3  ubuntu-advantage  stable      71  no       Attached (esm-infra,livepatch)

Unit                     Workload  Agent  Machine  Public address  Ports     Message
landscape-postgresql/0*  active    idle   5        <IP_ADDRESS>    5432/tcp
  landscape-client/1     active    idle            <IP_ADDRESS>              Client registered!
  ubuntu-advantage/2     active    idle            <IP_ADDRESS>              Attached (esm-infra,livepatch)
landscape-postgresql/1   active    idle   7        <IP_ADDRESS>    5432/tcp  Primary
  landscape-client/7     active    idle            <IP_ADDRESS>              Client registered!
  ubuntu-advantage/7     active    idle            <IP_ADDRESS>              Attached (esm-infra,livepatch)
landscape-postgresql/2   active    idle   2        <IP_ADDRESS>    5432/tcp
  landscape-client/0*    active    idle            <IP_ADDRESS>              Client registered!
  ubuntu-advantage/0*    active    idle            <IP_ADDRESS>              Attached (esm-infra,livepatch)

Machine  State    Address       Inst id         Base          AZ     Message
2        started  <IP_ADDRESS>  landscapesql-1<EMAIL_ADDRESS>zone1  Deployed
5        started  <IP_ADDRESS>  landscapesql-2<EMAIL_ADDRESS>zone2  Deployed
7        started  <IP_ADDRESS>  landscapesql-3<EMAIL_ADDRESS>zone3  Deployed

Offer             Application       Charm             Rev  Connected  Endpoint               Interface             Role
landscape-server  landscape-server  landscape-server  107  1/1        application-dashboard  register-application  requirer

Checking the landscape-server units, we can see that they were configured to use a Secondary PostgreSQL server:
cat /etc/landscape/service.conf
[stores]
user = landscape
password = removed
host = <IP_ADDRESS>:5432

This causes the Landscape server to crash, and result in "System errors" when accessing its dashboard, and clients failing to register to it.
Expected behavior
In case a new Primary is elected, the postgresql charm should update its relations with the "master", so the requirer charms can update themselves accordingly.
Actual behavior

Relation data remains the same, as seen above.
Versions

Operating system: Ubuntu 22.04.4 LTS

Juju CLI: 3.3.1-genericlinux-amd64

Juju agent: 3.3.1

Charm revision: channel 14/stable, revision 351

LXD: 5.0.3
Log output

Juju debug log:

Additional context

Hi, @claudiubelu, I was able to replicate locally on revision 351, but not on the latest 14/stable (rev. 363). There were issues with juju secrets on 351 that I believe are the root cause for this. Can you retry with rev. 363?

Dear @claudiubelu the revision 363 is now in 14/stable.
I am resolving this issue due to lack of your reply on the message above.
Feel free to reopen the issue if you are still experiencing this issue.
Thank you for the bugreport!


Docker machine created hosts unable to deploy docker/ucp by default

When creating hosts on AWS with Docker Machine, a swarm-port conflict occurs when running:
docker run --rm -it -v /var/run/docker.sock:/var/run/docker.sock --name ucp docker/ucp install -i
You have to specify a new --swarm-port to continue.
--swarm-port 2377
I would think that Docker Machine users would want to be able to deploy UCP using Docker Machine with default settings.
MacBook-Pro-2:DockerCon home$ docker run --rm -it         --name ucp         -v /var/run/docker.sock:/var/run/docker.sock         docker/ucp         join --replica -i
Please enter the URL to your UCP server: https://ec2-54-153-117-90.us-west-1.compute.amazonaws.com
UCP server https://ec2-54-153-117-90.us-west-1.compute.amazonaws.com
Subject: ucp
Issuer: UCP Client Root CA
SHA1 Fingerprint=FD:AD:0B:AE:DA:D2:87:32:15:B2:B0:FB:57:4E:09:77:CE:F7:87:9A
Do you want to trust this server and proceed with the join? (y/n): y
Please enter your UCP Admin username: admin
Please enter your UCP Admin password:
INFO[0014] Pulling required images... (this may take a while)
WARN[0037] None of the hostnames we'll be using in the UCP certificates [datacenter-ucp-replica <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS>] contain a domain component.  Your generated certs may fail TLS validation unless you only use one of these shortnames or IPs to connect.  You can use the --san flag to add more aliases

You may enter additional aliases (SANs) now or press enter to proceed with the above list.
Additional aliases:
INFO[0000] This engine will join UCP and advertise itself with host address <IP_ADDRESS> - If this is incorrect, please specify an alternative address with the '--host-address' flag
INFO[0000] Verifying your system is compatible with UCP
FATA[0002] The following required ports are already in use on your host - [2376].  You may specify an alternative port number to 2376 with the --swarm-port argument.

This can be remedied by reassigning the swarm port
MacBook-Pro-2:DockerCon home$ docker run --rm -it         --name ucp         -v /var/run/docker.sock:/var/run/docker.sock         docker/ucp         join --replica -i --swarm-port 2377

What's the error message say?

@nathanleclaire updated original post.

mm @ehazlett @dhiltgen why do you all claim :2376 instead of :3376 for the swarm master port?

Hi @xacaxulu, this is a product design of the UCP team due to fact that Swarm is meant to imitate the Docker Engine endpoint as closely as possible.  As you noted, the workaround is to use --swarm-port if you need an alternative port (which will usually be true in Machine's case).


chore(ja): remove the description about the memory optimizations

Same as #932.
cc: @woodyZootopia or @JohnTitor

Thanks!


Kotlin?

I would like to report a bug, the bug is usage of kotlin. Can you please make this in java?

Sorry, no.


feat: Faster ci test setup

Description
This PR speeds up the CI tests, by using setup-micromamba to install the python environment.
This speeds up the time it takes to install conda/mamba and setup the conda-environment. This is especially fast when the environment is cached:
Linux: 3 min ->  11 sec
Windows: 12 min -> 2.5 min
For the black formatting: 60 sec -> 11 sec
It also replaces the cancel-previous job with the github-action built-in concurrency specification which does the same.
Finally, it adds a .test_duration file so pytest-split can minimize test time by spreading the tests across jobs in an optimal way. This cuts the test time by a thrid.
QC

[x] The PR contains a test case for the changes or the changes are already covered by an existing test case.
[x] The documentation (docs/) is updated to reflect the changes or this is not necessary (e.g. if the change does neither modify the language nor the behavior or functionalities of Snakemake).

@johanneskoester. This is ready for review.


always return the same

I have my server running, when i enter to <IP_ADDRESS> i can see
OpenOCR is running!
Need docs?
same with localhost.
But when i send requests like this:
curl -X POST -H "Content-Type: application/json" -d '{"img_url":"http://bit.ly/ocrimage","engine":"tesseract"}' http://<IP_ADDRESS>:8080
the response is:
OpenOCR is running!
Need docs?
am i doing anything wrong?
Thanks

You're missing the endpoint!
Try:
curl -X POST -H "Content-Type: application/json" -d '{"img_url":"http://bit.ly/ocrimage","engine":"tesseract"}' http://<IP_ADDRESS>:8080/ocr

Does that work?


Style/asset landing page

Fixes linting error in asset landing page

Oh yeah, no problem at all. I'll close this out.


Should we change billing info to shipping info? If billing is handled by PayPal, shipping still needs to be handled by the vendor. It also can be different order-by-order.

Shipping is handled by pay pal as well, it's provided back to us after the checkout and I store it in orders


Missing docs: Can we update the documentation with regards to JSON credentials loading not from a file reference?

Hi folks,
I'd like to send a pull request to update the project's README.md with instructions and examples on how to load credentials not by specifying a file path to a JSON file but rather using the credentials key. This isn't documented publicly anywhere and was only discovered by scouring the issue queue and a google search leading to it.

Hi @lirantal ,
Thanks for calling this out!! While I see why this is confusing and not well documented, I don't think the README is the right place for this information. Could you add this sample to the storage constructor file instead? https://github.com/googleapis/nodejs-storage/blob/784bd720c3392d13f39463442e7758c9a34f2d0c/src/storage.ts#L620-L652
There are a few examples there already.
Thanks again for the contribution!!

Happy to extend a hand and help.
I'm not sure why the README isn't the proper place though. I would say the current README update in this PR is a good elaborate one, where-as a code example snippet on the docblock will be nice but would need to be short for brevity. Realistically, no one is going to go into that file and read it, nor shouldn't they be expected to.
I'll shoot an update to that file in a new PR but am hoping we can land this one too with the hope of easing the frustration for developers.

@shaffeeullah see the docblock PR here: https://github.com/googleapis/nodejs-storage/pull/2124

@lirantal Thanks for making this change!! I think the README will get a little hard to parse through if we start adding code snippets directly to it. Keeping this in the docstring allows people to view it if they're looking at the API-level documentation. It won't be necessary for them to look at the code file directly. This could potentially be a good sample to go in the bottom section of the README. I'll discuss that option with the larger team when everyone is back after the holidays. Thanks again!


fill in queriedBy fields for results from resolver cache

otherwise, callers like ResolveThenIdentify2 will get the wrong answer for WasKBAssertion()

CI fail is a flake in TeamRekey uggg but not related


Nested table renders before parent table

If you use HTML to nest a table within the cell of another table then the nested table renders physically before the parent table
Repro
| title | nested table |
| :---: | :---: |
| Title |

Tag

 |

Rendered by GitHub

title
nested table

Title
Tag

Rendered by inlyne

@nicoburns I've tried to get nested tables to work on this branch https://github.com/trimental/inlyne/tree/nested-tables. Looks like I'm 80% there to getting it working however the the gap between the nested table and the other row is the maximum height of the table, even when the table isn't under min width conditions. Any suggestions on this?

@trimental Just wanted to note that I'm planning on getting to this, but I've just started a new job so I'm pretty busy at the moment.

@trimental The issue seems to be caused by setting size.width to length(bounds.0) in table.rs when bounds.0 is f32::MAX. This forces the size to be exactly f32::MAX rather than it being a limit. In addition to this, it seems to cause an overflow issue (presumably because the inner table is sizing as f32::MAX wide, and the outer table then has to add the widths of the other columns to that!). Changing width to max_width fixes the issue, although I think you might want to consider adopting Taffy's AvailableSpace enum (and perhaps Size struct) in inlyne's functions to make this more explicit.


Fix formatting for multi-line object destructuring

This fixes the rule insertSpaceAfterOpeningAndBeforeClosingNonemptyBraces: false for multiline object destructurings.
Input
const {
  a,
  b,
} = { a: 1, b: 2 };

Formatted output (actual)
const {a,
  b,} = {a: 1, b: 2};

Formatted output (expected)
const {
  a,
  b,
} = {a: 1, b: 2};

@KingHenne,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@KingHenne, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

/cc @SaschaNaz

I'm not a reviewer so tagging me won't help accepting this PR. LGTM anyway.

@SaschaNaz Just wanted to make sure I didn't misunderstand your intentions when you added this feature. So thanks for the "LGTM".


#[payable] attribute added

To prevent contract clients to accidentally attach deposit to methods which doesn't consume value (in case of Functional with deposit it means loss of tokens).
By default we don't allow contract methods to accept deposits. Method must be opted-it for it explicitly.
Discussion
https://commonwealth.im/near/proposal/discussion/329-payable-functions

btw, link to travis is broken
https://travis-ci.com/github/nearprotocol/near-sdk-rs/jobs/299961767

@nearmax
something broke:
➜  near-bindgen git:(fckt/payable) git push
ERROR: Permission to nearprotocol/near-bindgen.git denied to fckt.
fatal: Could not read from remote repository.
Please make sure you have the correct access rights

➜ near-bindgen git:(fckt/payable) git push
ERROR: Permission to nearprotocol/near-bindgen.git denied to fckt.
fatal: Could not read from remote repository.

It was renamed to near-sdk-rs

@nearmax
something broke:
➜ near-bindgen git:(fckt/payable) git push
ERROR: Permission to nearprotocol/near-bindgen.git denied to fckt.
fatal: Could not read from remote repository.
Please make sure you have the correct access rights

We moved it to a different organization and renamed it: https://github.com/near/near-sdk-rs


FocusTrap: Fix outline focus

Description
Locate the problem within the MudFocusTrap component. You can typically find the relevant code in the Razor component file.
In the Razor component file, navigate to the specific line (e.g., line 22) where the MudFocusTrap component is defined using the  tag.
Within this  element, you'll want to add an existing CSS class called "outline-none." This class is commonly available in MudBlazor and is used to remove unwanted outlines or focus styles.
How Has This Been Tested?
To test the problem:
Open a web browser that uses the Chromium engine, such as Google Chrome or Microsoft Edge.
Load the web page or web application that contains the MudFocusTrap component you want to test.
Open the dialog or window that contains the MudFocusTrap component. Ensure that the dialog is visible on the screen.
Within the content of the dialog, click on an area where there are no other interactive components, such as buttons or text input fields.
Press any key on the keyboard. Notice that an unwanted focus or outline may become visible around the MudFocusTrap component.
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.
Dialog - MudBlazor - Google Chrome 2023-11-30 10-10-48.zip

Hi.
Thanks for PR.
Sorry that it took so long to notice the PR...
Could you post a before and after picture or a gif?

Hello.
Here are the videos
Before: https://github.com/MudBlazor/MudBlazor/assets/79289665/e197d41d-5847-4943-8e09-7d5f7a2f2f93
After: https://github.com/MudBlazor/MudBlazor/assets/79289665/fa4ab9b6-325b-477a-b8c1-e8b385fce841

I wonder if the focus trap is part of the cause of #4283

it seems to be the solution to this problem

@henon  I'm not sure, but I'm testing this case

I found the cause of this problem; it's not in the FocusTrap but in the style applied to the MudButton. I will try to solve this problem in the best way possible and push a new update.

@LiZzeira that would be kind of you. Just open another PR please, I am merging this one.

@henon @danielchalmers
I made the correction regarding the problem with the focus on the button

New PR open
https://github.com/MudBlazor/MudBlazor/pull/8575


AdoptOpenJdkRemoteBinary doesn't respect libc

AdoptOpenJdkRemoteBinary doesn't respect the libC on Linux, for example if you run you build on Alpine it should add the parameter for MUSL libC to the REST Url which would be supported by the api.adoptjdk.net
Expected Behavior
If the build is running on Alpine Linux the correct SDK should be fetched.
Current Behavior
The libC parameter isn't set -> defaulting to GlibC
Context
Running a Gradle build on an Alpine Container with enabled JVM Auto Provisioning

AdoptOpenJdkRemoteBinary will be removed in Gradle 8.0, so getting it fixed will most likely not happen.
However, it's being replaced by Toolchain Resolver plugins and example of which is the Foojay Toolchains Plugin.
And there there is already an open issue for handling this problem, so I will be closing this one.


Problem with deselecting the input value

I am using 2 different instances of ng2-completer, one with datasource from an api and one from an array. In both situations, I emit the selected options but when I clean the input I either get an exception ( Cannot read property 'originalObject' of null) or nothing is emited.

Hi,
can you add an example or a plunker?

Actually I have added overrideSuggested as true and it seems to work fine

Ok, please close the issue if all is working fine 😄


Class FileCache contains 5 abstract methods and must therefore be declared abstract or implement the remaining methods

Describe the bug
Downloaded the current code in master branch. After setting things up(copy over from an existing instance) and when I attempt to just go to the site, it records this error in the log.
PHP Fatal error:  Class FileCache contains 5 abstract methods and must therefore be declared abstract or implement the remaining methods (CacheInterface::get, CacheInterface::set, CacheInterface::delete, ...) in rss-bridge-test/caches/FileCache.php on line 3
I've tried the Sqlitecache as well and it also returned the same error.
Running PHP 8.0
To Reproduce
Steps to reproduce the behavior:

Download current code in master
Set it up on a server
Go to the site
See error

Expected behavior
The site loads correctly
Screenshots
N/A
Desktop (please complete the following information):

OS: macOS
Browser Safari
Version 17.1.2

sounds like mixing old and new code.
your caches/FileCache.php probably is an older version.

Sure enough that was it. I kept comparing the code locally and didn't see a problem, but clearly should have been looking at it on the server.
Thanks!