Some tests fail to reach test_main() on cc1352r1_launchxl

Describe the bug
The following tests crash prior to reaching test_main():

tests/crypto/rand32 (using prj_hw_random_xoroshiro.conf)

FATAL: ***** BUS FAULT *****
FATAL:   Imprecise data bus error
FATAL: r0/a1:  0x00000028  r1/a2:  0x200002d4  r2/a3:  0x00000000
FATAL: r3/a4:  0x00000000 r12/ip:  0x20001458 r14/lr:  0xfffffffd
FATAL:  xpsr:  0x6100000e
FATAL: Faulting instruction address (r15/pc): 0x00001a26
FATAL: >>> ZEPHYR FATAL ERROR 0: CPU exception
FATAL: Current thread: 0x00000000 (A^\)
FATAL: Halting system

tests/kernel/mem_protect/stackprot

FATAL: ***** BUS FAULT *****
FATAL:   Imprecise data bus error
FATAL: r0/a1:  0x00000028  r1/a2:  0x20000260  r2/a3:  0x00000000
FATAL: r3/a4:  0x00000000 r12/ip:  0x200018c8 r14/lr:  0xfffffffd
FATAL:  xpsr:  0x6100000e
FATAL: Faulting instruction address (r15/pc): 0x00001a8e
FATAL: >>> ZEPHYR FATAL ERROR 0: CPU exception
FATAL: Current thread: 0x00000000 (^])
wrong error type

tests/kernel/mem_protect/stack_random

FATAL: ***** BUS FAULT *****
FATAL:   Imprecise data bus error
FATAL: r0/a1:  0x00000028  r1/a2:  0x2000025c  r2/a3:  0x00000000
FATAL: r3/a4:  0x00000000 r12/ip:  0x200018b8 r14/lr:  0xfffffffd
FATAL:  xpsr:  0x6100000e
FATAL: Faulting instruction address (r15/pc): 0x0000103e
FATAL: >>> ZEPHYR FATAL ERROR 0: CPU exception
FATAL: Current thread: 0x00000000 (Y^R)
FATAL: Halting system

They all seem to crash while a call to entropy_cc13xx_cc26xx_get_entropy() is waiting on a semaphore:
k_sem_take(&data->sync, K_FOREVER);
To Reproduce
Steps to reproduce the behavior:
0. cd <test directory>
1. mkdir build; cd build
2. cmake .. -DBOARD=cc1352r1_launchxl
3. make flash
4. See error

Expected behavior
I expect the tests to reach test_main().
Environment (please complete the following information):

OS: Ubuntu 18.04
Toolchain: Zephyr SDK 0.10.3
Commit SHA tested: a4bd5b68354ffa8c6c5d33ee03e2c00837fc0fd3

Removed 2.1 milestone


Enable queuing in logstash and document index limits.

I've recently deployed a LME cluster onto a supported ubuntu OS and have experienced the following situation:
Winlogbeat will push all events including any historic events into the logstash container.
Logstash then pushes all of these events into elasticsearch, however if the push from elasticsearch fails then these events do not appear to be queued within logstash.
On my install, elasticsearch has set a 1000 shard limit. With each index using 4 shards and each day using a single index there is a maximum of 249 days worth of logging available before elasticsearch drops new events and the push from logstash to elasticsearch fails. In my specific case the cluster lasted a little under a week before running out of indexes due to the initial push of historic events.
The specific error in the logstash container logs is:
[WARN ][logstash.outputs.elasticsearch] Could not index event to Elasticsearch. {:status=>400, :action=>["index", {:_id=>nil, :_index=>"winlogbeat-12.03.2020", :_type=>"_doc", :routing=>nil}, #<LogStash::Event:0x2de12e62>], :response=>{"index"=>{"_index"=>"winlogbeat-12.03.2020", "_type"=>"_doc", "_id"=>nil, "status"=>400, "error"=>{"type"=>"validation_exception", "reason"=>"Validation Failed: 1: this action would add [4] total shards, but this cluster currently has [1000]/[1000] maximum shards open;"}}}}
As logstash does not appear to be queuing these events there does not appear to be any easy way to resend the dropped events now that I have deleted old indexes. Perhaps this is something specific to my install or I have missed something in the documentation however I was not aware that there was a limit to the number of days of event logs that the cluster would be able to handle.

@anderoo6 - is this issue resolved?
There is some instructions to up the shard count in Issue https://github.com/ukncsc/lme/issues/60
As stated in that issue, we are working on refining the deployments for larger organisations or for deployments with high data volumes.

My issue is resolved now I understand the behaviour, the method to re-index the old logs from Issue #60 is useful to know too.

Closed due to project archive


added previous/next-month method

Description

Issue Reference

Proposed Changes

Screenshots / GIFs / Videos

Checklist

[ ] Lint rules are passed correctly
[ ] Code builds correctly
[ ] Created or updated tests for the new code
[ ] All existing and new tests passed
[ ] Pull request title is concise and descriptive.
[ ] Pull request is linked to the related issue.

Let me know @RoshanDalami , if you need help resolving merge conflicts

@blackkspydo Need help to resolve conflict


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_e2mh_2njw7/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_e2mh_2njw7/clone/recipe/meta.yaml:19: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_e2mh_2njw7/clone/recipe/meta.yaml:29: missing_imports_or_run_test_py: Python packages require imports or a python test file in tests.
WARNING: /tmp/abs_e2mh_2njw7/clone/recipe/meta.yaml:35: missing_description: The recipe is missing a description
ERROR: /tmp/abs_e2mh_2njw7/clone/recipe/meta.yaml:35: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_e2mh_2njw7/clone/recipe/meta.yaml:35: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_e2mh_2njw7/clone/recipe/meta.yaml:35: missing_license_family: The recipe is missing the about/license_family key.
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_a4ro143drz/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_a4ro143drz/clone/recipe/meta.yaml:19: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_a4ro143drz/clone/recipe/meta.yaml:29: missing_imports_or_run_test_py: Python packages require imports or a python test file in tests.
WARNING: /tmp/abs_a4ro143drz/clone/recipe/meta.yaml:35: missing_description: The recipe is missing a description
ERROR: /tmp/abs_a4ro143drz/clone/recipe/meta.yaml:35: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_a4ro143drz/clone/recipe/meta.yaml:35: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_a4ro143drz/clone/recipe/meta.yaml:35: missing_license_family: The recipe is missing the about/license_family key.
Errors were found


Adding controller APIs to fetch brokers information

Description
Adding controller APIs to fetch brokers information
Per issue: https://github.com/apache/incubator-pinot/issues/5682
Sample output:
➜ curl localhost:9000/brokers | jq .
{
  "tenants": {
    "DefaultTenant": [
      "Broker_<IP_ADDRESS>_8000"
    ]
  },
  "tables": {
    "airlineStats_OFFLINE": [
      "Broker_<IP_ADDRESS>_8000"
    ],
    "airlineStats_REALTIME": [
      "Broker_<IP_ADDRESS>_8000"
    ]
  }
}

➜ curl localhost:9000/brokers/tenants | jq .
{
  "DefaultTenant": [
    "Broker_<IP_ADDRESS>_8000"
  ]
}

➜ curl localhost:9000/brokers/tenants/DefaultTenant | jq .
[
  "Broker_<IP_ADDRESS>_8000"
]

➜ curl localhost:9000/brokers/tables | jq .
{
  "airlineStats_OFFLINE": [
    "Broker_<IP_ADDRESS>_8000"
  ],
  "airlineStats_REALTIME": [
    "Broker_<IP_ADDRESS>_8000"
  ]
}

➜ curl localhost:9000/brokers/tables/airlineStats_OFFLINE | jq .
[
  "Broker_<IP_ADDRESS>_8000"
]

➜ curl localhost:9000/brokers/tables/airlineStats_REALTIME | jq .
[
  "Broker_<IP_ADDRESS>_8000"
]

➜ curl localhost:9000/brokers/tables/airlineStats | jq .
[
  "Broker_<IP_ADDRESS>_8000"
]

Release Notes
Adding Controller APIs to fetch brokers information:
// Return all the {state} brokers mapping of tenants/tables
GET /brokers?state={online|offline}

// Return all the {state} brokers mapping per tenants
GET /brokers/tenants?state={online|offline}

// Return {state} brokers for a given {tenant}, returns all brokers if state is not specified.
GET /brokers/tenants/{tenant}?state={online|offline}

// Return all the {state} brokers mapping per table
GET /brokers/tenants?state={online|offline}

// Return {state} brokers for a given {table}, returns all brokers if state is not specified.
GET /brokers/tables/{table}?state={online|offline}

In order to be consistent with other APIs, I would take the raw table name, and a query parameter of table type in the API (instead of tableName_OFFLINE. etc.)

Also, we don't allow admin to specify different broker tenant names for realtime and offline tables. I would combine the two in the output as well. Keeps it consistent with the table config settings.
Otherwise, you run the risk of an admin trying to add broker tenant as "myTenant_REALTIME" or "myTenant_OFFLINE" in the table config.

In order to be consistent with other APIs, I would take the raw table name, and a query parameter of table type in the API (instead of tableName_OFFLINE. etc.)

done

Also, we don't allow admin to specify different broker tenant names for realtime and offline tables. I would combine the two in the output as well. Keeps it consistent with the table config settings.
Otherwise, you run the risk of an admin trying to add broker tenant as "myTenant_REALTIME" or "myTenant_OFFLINE" in the table config.

Correct, broker tenant has no offline/realtime suffix also changed the behavior for tables to remove suffix.


Update version number in manifest.json

Just a minor thing, but I noticed that the version number in manifest.json has been on 0.10.3 for the past few releases:   "version": "0.10.3",
I went into the installed manifest.json to see if I needed to update the component, comparing it with the latest 1.1.1. Then I checked the version that come with 1.1.1 and it was 0.10.3. Perhaps I had the latest version all along?!! :)
Thanks for this very useful integration!

This will be updated in future releases.

Addressed in 1.1.2


Support SliceOp and SplitOp in DimAnalysis

SliceOp and SplitOp are found in the GPT2 model. So add them to DimAnalysis.

Jenkins Linux ppc64le Build #7714 [push] Split and Slice (#1866) ... started at 22:10

Jenkins Linux s390x Build #8663 [push] Split and Slice (#1866) ... started at 22:07

Jenkins Linux amd64 Build #8647 [push] Split and Slice (#1866) ... started at 21:07

Jenkins Linux amd64 Build #8647 [push] Split and Slice (#1866) ... passed after 1 hr 11 min

Jenkins Linux s390x Build #8663 [push] Split and Slice (#1866) ... passed after 1 hr 33 min

Jenkins Linux ppc64le Build #7714 [push] Split and Slice (#1866) ... passed after 1 hr 50 min


Add flannel error

quickstart version: beta-kubernetes:0.4.0-1.9.0-beta
dcos version:  1.10.4
I installed kubernetes with web ui,it seems every service in dcos thing is going on and  I can get three Ready nodes

[root@localhost ~]# kubectl get nodes
NAME                                   STATUS    ROLES     AGE       VERSION
kube-node-0-kubelet.kubernetes.mesos   Ready         23h       v1.9.0
kube-node-1-kubelet.kubernetes.mesos   Ready         23h       v1.9.0
kube-node-2-kubelet.kubernetes.mesos   Ready         23h       v1.9.0

Then I try to add flannel to cluster, but get an error

[root@localhost ~]# kubectl get pods --namespace=kube-system
NAME                        READY     STATUS             RESTARTS   AGE
heapster-7d7b7f4f87-j4j4j   1/1       Running            0          23h
kube-dns-6dc79b7c4-kgmc2    2/3       CrashLoopBackOff   555        23h
kube-flannel-ds-9mkw4       0/1       CrashLoopBackOff   4          2m
kube-flannel-ds-s7w6h       0/1       CrashLoopBackOff   4          2m
kube-flannel-ds-xncpc       0/1       CrashLoopBackOff   4          2m

Logs look like this

[root@localhost ~]# kubectl logs --namespace=kube-system kube-flannel-ds-9mkw4
I0201 02:24:56.214697       1 main.go:474] Determining IP address of default interface
I0201 02:24:56.218057       1 main.go:487] Using interface with name eth0 and address <IP_ADDRESS>
I0201 02:24:56.218141       1 main.go:504] Defaulting external address to interface address (<IP_ADDRESS>)
I0201 02:24:56.277590       1 kube.go:130] Waiting 10m0s for node controller to sync
I0201 02:24:56.277757       1 kube.go:283] Starting kube subnet manager
I0201 02:24:57.277999       1 kube.go:137] Node controller sync successful
I0201 02:24:57.278089       1 main.go:234] Created subnet manager: Kubernetes Subnet Manager - kube-node-1-kubelet.kubernetes.mesos
I0201 02:24:57.278107       1 main.go:237] Installing signal handlers
I0201 02:24:57.278338       1 main.go:352] Found network config - Backend type: vxlan
I0201 02:24:57.278490       1 vxlan.go:119] VXLAN config: VNI=1 Port=0 GBP=false DirectRouting=false
E0201 02:24:57.279419       1 main.go:279] Error registering network: failed to acquire lease: node "kube-node-1-kubelet.kubernetes.mesos" pod cidr not assigned
I0201 02:24:57.279476       1 main.go:332] Stopping shutdownHandler...

How can I fix this?

@wyflimago Why do you need to install flannel ?

Hi @wyfligamo! Currently, we support only the DC/OS overlay. We have been trying some other overlay networks, like Calico, but it will need more work, and so we'll leave it to post-GA.
Adding some more details, in this particular case, the error is:

E0201 02:24:57.279419       1 main.go:279] Error registering network: failed to acquire lease: node "kube-node-1-kubelet.kubernetes.mesos" pod cidr not assigned
This happens because we do not enable the --allocate-node-cidrs flag for the kube-controller-manager.

@wyfligamo feel free to comment but I'm going to close this as it's no bug. Thank you for reporting.

@hectorj2f  my kube-dns pod status is CrashLoopBackOff and kubedns container log says

E0202 01:21:34.174837       1 reflector.go:201] k8s.io/dns/pkg/dns/dns.go:147: Failed to list *v1.Endpoints: Get https://<IP_ADDRESS>:443/api/v1/endpoints?resourceVersion=0: dial tcp <IP_ADDRESS>:443: i/o timeout
E0202 01:21:34.175333       1 reflector.go:201] k8s.io/dns/pkg/dns/dns.go:150: Failed to list *v1.Service: Get https://<IP_ADDRESS>:443/api/v1/services?resourceVersion=0: dial tcp <IP_ADDRESS>:443: i/o timeout

I can't find any other pods except heapster

[root@localhost ~]# kubectl get pods --all-namespaces
NAMESPACE     NAME                        READY     STATUS             RESTARTS   AGE
kube-system   heapster-7d7b7f4f87-brp2l   1/1       Running            0          21h
kube-system   kube-dns-6dc79b7c4-dbw5g    1/3       CrashLoopBackOff   506        21h

so I wonder if I should add a network plugin like flannel

@pires I don't understand about DC/OS overlay, is that working as a kubernetes pods or a dcos/k8s service?
My k8s only start kube-dns and heapster pods, is that right? they can't communicate with api service, but I can get api response with browser outside the cluster

[root@localhost ~]# kubectl logs --namespace=kube-system heapster-7d7b7f4f87-brp2l
E0202 05:26:09.524966       1 reflector.go:190] k8s.io/heapster/metrics/heapster.go:322: Failed to list *v1.Pod: Get https://kubernetes.default/api/v1/pods?resourceVersion=0: dial tcp: i/o timeout

[root@localhost ~]# kubectl logs --namespace=kube-system kube-dns-6dc79b7c4-dbw5g kubedns
E0202 05:23:47.244936       1 reflector.go:201] k8s.io/dns/pkg/dns/dns.go:150: Failed to list *v1.Service: Get https://<IP_ADDRESS>:443/api/v1/services?resourceVersion=0: dial tcp <IP_ADDRESS>:443: i/o timeout
E0202 05:23:47.244981       1 reflector.go:201] k8s.io/dns/pkg/dns/dns.go:147: Failed to list *v1.Endpoints: Get https://<IP_ADDRESS>:443/api/v1/endpoints?resourceVersion=0: dial tcp <IP_ADDRESS>:443: i/o timeout
I0202 05:23:47.742922       1 dns.go:173] Waiting for services and endpoints to be initialized from apiserver...
I0202 05:23:48.242824       1 dns.go:173] Waiting for services and endpoints to be initialized from apiserver...
I0202 05:23:48.742850       1 dns.go:173] Waiting for services and endpoints to be initialized from apiserver...
I0202 05:23:49.242879       1 dns.go:173] Waiting for services and endpoints to be initialized from apiserver...

@wyfligamo https://docs.mesosphere.com/1.10/networking/virtual-networks/#dcos-overlay

@wyfligamo looking at your logs, your problem doesn't seem to be pod networking so the description of this issue was misleading.
Again, looking at your logs it seems that kube-dns can't connect to the Kubernetes API (<IP_ADDRESS>). Does your host network use any of <IP_ADDRESS>/8 or <IP_ADDRESS>/16 CIDRs?

Since this issue was initially about running flannel and that has been clarified to the user, I am moving the real issue to a new issue #58. Let's continue there.


write possibly emptied link to standard header

link is set to the empty string if it is too long and will be set in the extended header

Yep, this seems like the right fix to me.  Thanks!


Initial Dapr Shared Docs

Please follow this checklist before submitting:

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[x] Read the contribution guide
[x] Commands include options for Linux, MacOS, and Windows within codetabs
[x] New file and folder names are globally unique
[ ] Page references use shortcodes instead of markdown or URL links
[x] Images use HTML style and have alternative text
[ ] Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:
Description
This is an initial version of the Dapr Shared docs. Now that Dapr Shared is out of Sandbox, a new set of docs are needed for new users to understand how to use it and get started with Dapr Shared. I would appreciate feedback on the location of these docs and what other sections to include.
Issue reference

@salaboy - @msfussell suggested I update the diagrams and send you the slides in case you wanted to tweak. Here's the pptx link:
Suggested-Dapr-Share-diagrams.pptx

@salaboy - @msfussell suggested I update the diagrams and send you the slides in case you wanted to tweak. Here's the pptx link:
Suggested-Dapr-Share-diagrams.pptx

Thanks a lot for doing this @hhunter-ms ! the diagrams look amazing

@hhunter-ms there are a few things that I need to manually change.. but the rest looks good.. I will tackle these changes in the next few days.

@salaboy - Tell me when you have time to review these comments to that we can get the PR merged. Suggest getting changes to the Dapr-Shared repo at the same time

bump.. working on this now

Ok, quick update here.
To get this done, I needed to change the hello-kubernetes quickstart applications to support the SDK supported environment variables: https://github.com/dapr/quickstarts/issues/1029 and https://github.com/dapr/quickstarts/pull/1030
Once these changes are done, I am submitting a PR to the dapr-shared repository with a new tutorial expanding on the hello-kubernetes tutorial to use Dapr Shared.
Finally, I will align this PR to link to that tutorial and maybe use the diagrams shown there.

@msfussell @hhunter-ms I think the daemonset diagram is fine, as the note explicitly says one per cluster node..

quick grammar review

Thanks a lot @hhunter-ms !

@msfussell diagrams updated..
I tried to answer all your comments but can we please not change the terminology and use Deployment and DaemonSet strategies? I think we will confuse people

@msfussell diagrams updated.. I tried to answer all your comments but can we please not change the terminology and use Deployment and DaemonSet strategies? I think we will confuse people

@salaboy - Where is the source for the diagrams? And one ask on the diagrams is to add the Kubernetes logo to show the boundary of the cluster. Like this example https://docs.dapr.io/concepts/overview/#kubernetes

New diagrams here:
Dapr-Shared-diagrams.pptx based on what @hhunter-ms shared with me

Done

Blog: http://salaboy.com http://salaboy.wordpress.com

Github user: http://github.com/salaboy

Twitter: http://twitter.com/salaboy

Mauricio "Salaboy" Salatino -

On Thu, 27 Jun 2024 at 16:55, Hannah Hunter @.***>
wrote:

@.**** commented on this pull request.
@salaboy https://github.com/salaboy - could you "Ignore all" the
spelling error in the daemonset.png diagram to remove the red squiggly
lines? other than that, lgtm!
—
Reply to this email directly, view it on GitHub
https://github.com/dapr/docs/pull/4121#pullrequestreview-2145835699, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACCMXVCF4WMEAFQN7OYJG3ZJQYX3AVCNFSM6AAAAABG6LU66CVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDCNBVHAZTKNRZHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

@holopin-bot @salaboy Thanks Salaboy!


Wrong timestamp component in 32bit system

The exact type of time_t is platform dependent and therefore can't be assumed to be uint64_t all the time.
For example, on 32bit platforms, where time_t is defined as long (32bit), the compiler will generate the following warning, due to the arithmetic used in the macro:
ulid_struct.hpp:248:34: warning: shift count >= width of type [-Wshift-count-overflow]

Force time_t to be parsed as uint64_t.

Can you replace all references to time_t with uint64_t, with a comment with a link to this PR stating why. Also make the corresponding changes in ulid_struct.hh, this only makes changes in ulid_uint128.hh. The test failure is because benchmark is broken on mac, am looking into it.


Use "Twitter" keyboard style

Any reason we're not using this right now? Especially once #219 lands, it would make sense to me if we prefer the @ and # shortcuts over the return (which with the .twitter keyboard is still accessible in were the numbers are).
AKA: want to wait for #219, do it now, or is there a reason you didn't do this @rnystrom?

I find myself using the return pretty frequently, but let's give this a try instead.

Please can we have the return key back? I like using multiple paragraphs in my comments :-)
[I only just found the return key on the second keyboard page—hadn't even realised that was there!]

Yeah - you can use return on the numbers+more keyboard.


Bug: 403 response status code doesn't indicate success!!

v1.804
link used: https://www.youtube.com/watch?v=hSbBf4REAA8

Does it happen with other videos as well? I'll not be home until August 9th so I can't check it out right now

Yup, it does happen with some other age-restricted videos aswell.

I've released an update that should fix this, let me know if its still happening

Fixed!!

Fixed!!


Link to original project

Can you link to my original project? Thank you! https://github.com/ryanmcdermott/clean-code-javascript

In fact I have linked original project in my introduction:

And I will add the original link before the title later in order to clearer :)

Awesome thank you!


test: Cypress - added tag - @tag.AutoHeight

Description

Add tags to cases in cypress/e2e/Regression/ClientSide/DynamicHeight folder

Type of change

Chore (housekeeping or task changes that don't impact user perception)

Testing

/ok-to-test<EMAIL_ADDRESS>
How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.

[ ] Manual
[ ] Cypress

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/ok-to-test<EMAIL_ADDRESS>

/ok-to-test<EMAIL_ADDRESS>@tag.IDE"


revisiting blue tile casually before anything happens

elif saw.bluetile and (item.ladle_full == False):
    show bg museum_blue_p_rusty
    show posty happy
    show bluetile giddy
    p "Hey again, Blue!"
    bluetile "Hi Posty!"
    show posty concerned
    "You look away from the painting. You don't want to be caught in a trance again. You can sense the {color=#ffff00}{i}inspiration{/i}{/color} emanating from the framed piece."
    p "So what's so great about this piece?"
    bluetile "Isn't it obvious?"
    p suspicious "Well like, is it the imagery and subject matter people make with the color blue?"
    p "I mean, with this piece, Crayon Box is something of a pop art icon nowadays."
    bluetile "Oh of course! What people create with the lightest tints of cyan and an analogous color palette is incredible!"
    bluetile annoyed "My friend Red likes to think that all I am is a numbnuts obsessed with the color blue, like I'm its number one fan and sworn protector."
    p "Aren't you though?"
    bluetile giddy "Well yeah!"
    bluetile "But what they don't know is that I wouldn't be if everything blue wasn't so pretty!"
    p "OK."
    p "Well, talk to you later."
    bluetile giddy "Bye Posty! Come by again!"
    jump museum_blue


ContextReplacementPlugin is used to suppress critical dependency warnings caused by require.toUrl in buildModuleUrl

Fixes #6.
ContextReplacementPlugin

Related issue in Cesium repo: https://github.com/CesiumGS/cesium/issues/8401

If it's merged, probably, this tutorial should also be updated.

Thanks for this PR @ezze – it solved the issues I was having.. I hope it gets merged at some point.

Thanks for this PR @ezze – it solved the issues I was having.. I hope it gets merged at some point.


feat(styles): removed demo section, added tabbed code/demos

I've removed the demo section since only wtc has access to it. I've updated the code blocks to be in tabbed formatting with demo examples.

Thanks for all this @chanroyc
I made some changes here:

removed unused typography stylesheets, as they were conflicting with the docs' styling
re-formatted a lot of the docs into markdown tables, so that the table of contents on the right sidebar isn't as busy
adjusted code snippets with HTML syntax highlighting and opted to not use Pug syntax since it's React

The big ticket item I'm wondering about is the live React examples in the page. It's awesome to have those, no doubt. My concern is that we're duplicating stylesheets, which are subject to change. I tried pulling them from the /cli/templates/project/ folder, but we still have the conflicting Docusaurus className issues that you solved in your previous commit. I don't know of a solution for these duplicated stylesheets that's both maintainable and simple enough to implement.
I wonder if it's best to abstract these examples out into a Components page or something (à la Storybook), or at the very least just link to (or embed) CodePens. Looping in @ste-vg for ideas as well.

As discussed off-thread, I've reverted this PR to not have the React demos.
I've created a backup feature branch with all of those though, for when we have time to come up with a solution! https://github.com/wethegit/corgi/tree/feature/docs/flex-grid-demos


Event Description Missing?

I've found everything I've needed from windows events using winevt in python, except the eventid description.
For example, the description for eventid 1502 is:
"The Group Policy settings for the computer were processed successfully. New settings from %6 Group Policy objects were detected and applied."
Is the event description available through winevt?

Not at the moment. I recall looking into that previously and I believe you
obtain those by having to look up the event producing dll and calling some
function in it to get a description. Been a while.
On Wed, Mar 2, 2022, 10:02 AM panofish @.***> wrote:

I've found everything I've needed from windows events using winevt in
python, except the eventid description.
For example, the description for eventid 1502 is:
"The Group Policy settings for the computer were processed successfully.
New settings from %6 Group Policy objects were detected and applied."
Is the event description available through winevt?
—
Reply to this email directly, view it on GitHub
https://github.com/bannsec/winevt/issues/15, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB2HPYEKOVNJDAJH2CPEMT3U557HZANCNFSM5PXWU54A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


SQL Query Editor - Duplicate Tab Feature

SQL Query Editor

Duplicate editor from the context menu
It would be useful to have a Duplicate editor facility from an existing editors context menu.
This would be a quicker way to split queries against the same or different databases with the same table schema.
Regards,
Graham O'Riley

We have an existing feature request for the same: https://github.com/microsoft/azuredatastudio/issues/245
Closing as duplicate.


Feature request - week starting on Monday

Can we please add ability to display weeks in the calendar with Monday as the first day?
Thanks.

My calendar starts on monday, but I don't see any option to choose that. Maybe it's because I have it in spanish.

Mine starts on Sunday so I guess it is pulling this setting from the server?

I noticed this issue/feature was mentioned before (#1537).
Closing the issue.


Public URLs: Add a page to list/manage

Create a page at ui.rilldata.com/[org]/[project]/-/public-urls
List all Public URLs in a table
Include:

dashboard to which the URL applies
expiration time (if any)
a delete action

Figma mock
Ref: https://github.com/rilldata/rill/pull/5283

How do we get public urls atm? @ericpgreen2
I see rill public-url from the doc.

And rill share-url in the latest CLI build.

Rill version
rill version 0.47.5 (build commit: 5e3bb02b9 date: 2024-07-19T05:16:50Z)

How do we get public urls atm? @ericpgreen2

In our v47 release notes, check out the gif that shows how to create them: https://docs.rilldata.com/notes/0.47


direction plugin needed

could you please add this plugin to this build
https://github.com/abedi-ir/ckeditor5-direction

#60


[BEAM-1116] Support for new Timer API in Flink runner (streaming)

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

Coverage decreased (-0.003%) to 69.224% when pulling d8628be6a64307f613481ac680ff7e511e79f47c on JingsongLi:BEAM-1116-newTimerApiInFlink into 36ed3f6c54d35ac48241789fd77e2657d54c5831 on apache:master.

Run Flink RunableOnService

I stared a Flink Runner post-commit hook here: https://builds.apache.org/view/Beam/job/beam_PostCommit_Java_RunnableOnService_Flink/1674/

Thank you, it passed the test.

Perfect! Let's wait for an answer by @kennknowles to your question on the Jira Issa regarding how to extract the window from a namespace. Then I would merge this. 👍

Sorry for the delay! I missed the real comment between the bot noise on the JIRA. I answer there, but the basic thing is that namespaces are generalized beyond what user timers need. You only need WindowNamespace.

OK, I got it. User timers all come from StateNamespaces.window, so the namespace must be WindowNamespace.

Coverage decreased (-0.01%) to 69.217% when pulling ddbbb11f68245869e305e4005ee4303797e482cf on JingsongLi:BEAM-1116-newTimerApiInFlink into 36ed3f6c54d35ac48241789fd77e2657d54c5831 on apache:master.


Freeform search API: pagination

Is your feature request related to a problem? Please describe.
A freeform search API was introduced in https://github.com/project-zot/zot/issues/591 / https://github.com/project-zot/zot/pull/618
This API should support pagination.
Describe the solution you'd like
We should include a limit and an offset in order to request a specific page of a given size.
See also:
https://www.moesif.com/blog/technical/api-design/REST-API-Design-Filtering-Sorting-and-Pagination/#pagination
Additional context
This is needed for the zui functionality - in both the general search view, and for the free type search (to request only the first page with the most relevant results).

In review


WeightCalculatorWebFilter doesn't refresh the weights correctly for the routes on the fly which causes gateway to return CONTENT_NOT_FOUND

Describe the bug
We use the weight based routing in spring cloud gateway, the route definitions are sourced from the YAML file which is fetched from the remote location. The rationale for externalising the YAML is to have the routes refreshed on the fly(without server restart). Below is the route definition YML excerpt,
spring:
  cloud:
    gateway:
      routes:
      - id: route_def_75
        uri: https://host1.org
        predicates:
       - Path=/test
       - Method=GET
        - Weight=group1, 75
      - id: route_def_25
        uri: https://host2.org
        predicates:
        - Path=/test
        - Method=GET
        - Weight=group1, 25

We wanted to change the traffic distribution percentage from 75 - 25 to 60 - 40, and updated the route definitions YML with something like below
spring:
  cloud:
    gateway:
      routes:
      - id: route_def_60
        uri: https://host1.org
        predicates:
        - Path=/test
        - Method=GET
        - Weight=group1, 60
      - id: route_def_40
        uri: https://host2.org
        predicates:
        - Path=/test
        - Method=GET
        - Weight=group1, 40

As mentioned in the beginning, we have the mechanism to change routes on the fly(without server restart) by publishing RefreshRoutesEvent on spring's ApplicationContext(applicationContext.publishEvent(new RefreshRoutesEvent(this)).
Upon the refresh event, when requests to /test is fired, we could see the gateway is responding with 404 CONTENT_NOT_FOUND which is something we get when there is NO route matching existing for the request.
However, when the gateway server gets restarted, the issue will no longer exists & could see the traffic distribution as per the new percentages. This issue is only with the routes getting refreshed on the fly(without server restart)
We use spring-cloud-gateway-server version 3.1.2
Sample & steps to reproduce
This error is very easy to reproduce, just that we need to have the setup as pre-requisite where the routes can be refreshed on the fly. Here is one of the reference example
Once you have the setup ready,

Please have the routes defined with certain weights attached to them
Change the weights configuration & once the route refreshed successfully(note that it should be on the fly, without server restart).
Fire the requests to the path that is being configured in the routes, it is observed that the 404 CONTENT_NOT_FOUND is being returned by the gateway application

Initial triage and analysis
We have done the initial triage & found that in WeightPredicateFactory the routes map is not being refreshed successfully. Upon digging deeper, the group weights in WeightCalculatorWebFilter is not being updating properly and the same being updated in the exchange that is being used in WeightPredicateFactory

3.1.x is no longer supported. Is this still an issue in 4.1.x?

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


Some examples not working

This page has a bunch of examples on how to use maplibre-gl-inspect. Some are not working:

https://maplibre.org/maplibre-gl-inspect/geojson-source.html#3/40/-74.5
https://maplibre.org/maplibre-gl-inspect/query-params.html#3/40/-74.5
https://maplibre.org/maplibre-gl-inspect/custom-popup.html#2.36/43.77/-76.86

Yeah, this was inherited when we took over the repo. Feel free to open a PR to fix them.

Looking at the above, I think the first and the third are working as expected when setting the inspect to "on".
I agree the second one is not working as expected, probably due to the style that doesn't have a valid token for maptiler.
Am I missing any thing?


(docs) Update README with reference to hiera-eyaml-vault

This is a useful plugin we recently wrote for providing "encryption as a service" using the transit secrets engine from Vault... Seems sensible to mention it here.

Thanks for the PR!


Get current Row / Column in customer Aggregators

Does anyone know if it's possible to get the current row / column in a custom aggregator ?
I'm trying to add a different type of calculation as an aggregation function.

What do you get when you console.log(this)?

Ok bit more info If I try this example:
var successRate = function() {
  return function() {
    return {
      sumSuccesses: 0,
      sumTrials: 0,
      push: function(record) {
        if (!isNaN(parseFloat(record.successes))) {
          this.sumSuccesses += parseFloat(record.successes);
        }
        if (!isNaN(parseFloat(record[denom]))) {
          return this.sumTrials += parseFloat(record.trials);
        }
      },
      value: function() { return this.sumSuccesses / this.sumTrials; },
      format: function(x) { return x; },
      numInputs: 0
    };
  };
};

The value function doesn't seem to be able to get the current record only the aggregator function.
Btw returning this on the example listed  would just return the object with  sumSuccesses: 0,sumTrials: 0

Does the value function provide any arguments that you can catch?

Tried that and I get nothing :-(

OK, hm. I am trying to find the documentation.

The docs saty:

Aggregator functions receive 3 parameters: a PivotData object, a rowKey and a colKey

Can you use the row key to access the row from the data object in the parent scope?

I.e. you might catch all three parameters as described, and do something like data[rowKey] from within the value function.

My function is defined like:
                    value: function (data, rowKey, colKey) {
                        debugger;
                        return 0; },

And in the debugger I get:
data = undefined, rowKey = undefined, colKey = undefined

Hm, right. Seems contrary to the documentation.

I think the function def would look like:

var countUnique = function(attributeArray) {
var attribute = attributeArray[0];
return function(data, rowKey, colKey) {
return {
uniq: [],
push: function(record) {
var _ref;
if (_ref = record[attribute], __indexOf.call(this.uniq, _ref) < 0) {
this.uniq.push(record[attribute]);
}
},
value: function() { return this.uniq.length; },
format: function(x) { return x; },
numInputs: 1
};
};
}

but as you can see the value function still doesn't get passed those vars :-(

Ah, make sure you are passing in the attribute array.

Sorry I don't understand.

You also might need to pass in the data, rowKey, and colKey arguments to the return function.

Could you show an example ?

var successRate = function() {
  return function(data, rowKey, colKey) { // These arguments may be necessary
    return {
      sumSuccesses: 0,
      sumTrials: 0,
      push: function(record) {
        if (!isNaN(parseFloat(record.successes))) {
          this.sumSuccesses += parseFloat(record.successes);
        }
        if (!isNaN(parseFloat(record[denom]))) {
          return this.sumTrials += parseFloat(record.trials);
        }
      },
      value: function() { return this.sumSuccesses / this.sumTrials; },
      format: function(x) { return x; },
      numInputs: 0
    };
  };
};

I have posted some example code.

Thanks !

Bah rowKey and colKey are empty arrays !

Ah it's because I'm adding them as a closure so those vars only reference the on startup...

Right, I was thinking something like that was the case. Closures are confusing :-)

Yeah :-( looks like there is no way to find anything about the current record you are on when creating an aggregator function :-(.

rowKey and colKey are empty arrays when you fetch the grand total aggregator. Else they are a list of the values of the properties of the group by statements in the rows and cols.
To do what you want to do you need to dig a little deeper into pivotData data-structure.
Each cell in the pivottable is aggregator object that can be fetched via data.getAggregator(colKey,rowKey).  The cells that are empty have no aggregregator.  This should help.
var successRate = function(attributeArray) {
  var attribute = attributeArray[0];
  return function(data, rowKey, colKey) { // These arguments may be necessary
    return {
      sumSuccesses: 0,
      sumTrials: 0,
      push: function(record) {
        if (!isNaN(parseFloat(record.successes))) {
          this.sumSuccesses += parseFloat(record.successes);
        }
        if (!isNaN(parseFloat(record[denom]))) {
          return this.sumTrials += parseFloat(record.trials);
        }
      },
      value: function (data, rowKey, colKey) { // remember to handle them here too
      //get all possible columns for this dataset.
        var colkeys = data.getColKeys();
        var allRows =[];

        //loop through all possible columns
        colKeys.forEach(function(newColKey){
               var row_agg = data.getAggregator(rowKey,newColKey)
               if (row_agg.hasOwnProperty('value')){
               allRows.push(agg.value())
                } else {
                  allRows.push('Some sane default')
                  }
             })
       //Use the values in allRows in a calculation
        return this.sumSuccesses / this.sumTrials;
      },
      format: function(x) { return x; },
      numInputs: 0
    };
  };
};

Just tried this and I get:
Cannot read property 'getColKeys' of undefined.

Indeed the way to do this is via the data, rowKey, colKey parameters, which you need to capture in a closure.
The only built-in aggregator which uses this is the fractionOf aggregator and it's a bit of a mind-bender because it wraps other aggregators.
The key to making this work is not to have data, rowKey, colKey as parameters to the value function, so that the body of the value function access the objects in the closure.

But then they don't get updated.

I don't follow... There is one aggregator for each cell in the table, so they never get updated.

Oh, and value is only called once per aggregator.

What I mean is these values:
rowKey, colKey
are always empty arrays regardless of if I use a closure or not.

Yes, for the Grand Total aggregator, but not for the other ones.

Can you show an example? I'm always getting empty array.

Can you post an example of what you're trying so I can copy it and edit it?

Thank you for the help. I've managed to create my calculation without needing access to the current cell so I'll just leave it as is. Maybe just close this issue now.

Closing.,

OK. Well here's an example of it working ... just look in the console: https://jsfiddle.net/spd346y8/


az storage blob delete-batch not running

Describe the bug
I tried to "clean" a directory in my blob storage and the command does not finish so I stop it. There are 2 CSV files of around 1 GB, so I would expect a fast response. Previously I list those items and they are shown properly.
I tried different parameters including timeout and debug and nothing changed.
Command Name
az storage blob delete-batch
Errors:
Did not finish.
To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...

az storage blob delete-batch --account-name {} --account-key {} --auth-mode {} --source {} --pattern {} --verbose --dryrun --timeout {} --debug

Expected Behavior
Delete blobs in that directory.
Environment Summary
Linux-5.3.0-45-generic-x86_64-with-debian-buster-sid
Python 3.6.5

azure-cli 2.2.0 *

Extensions:
interactive 0.4.3

Additional Context

hi @Juliehzl could you pls have a look?

add to S169

I tried the command again in dryrun, and it finished in 2180 sec (a lot of time !)
az storage blob delete-batch \

--account-name $AZURE_STORAGE_ACCOUNT \
--account-key $AZURE_STORAGE_ACCESS_KEY \
--auth-mode key \
--source $container_name \
--pattern "$nuevo_trx*" \
--verbose \
--dryrun \
--timeout 30 \
--debug

Thank you.

@andres-ab Thanks. Was your issue resolved?

add to S171


Fix RPC SpendZerocoin

Fix RPC spendzerocoin:

Passing NULL into CBitcoinAddress constructor segfaults in CBase58Data::DecodeBase58Check

Changed
    CBitcoinAddress address = NULL;
to
    CBitcoinAddress address;

Am closing this @mfrankovi @reubenyap as it has been merged into Poramin's spend-to-3rd-party branch couple of weeks ago - thank you


[Proposal] X-GraphQL-Event-Stream header

This PR is to open a dialog; I'm not expecting it to be merged without some significant changes as I've kind of gone at the source with a hatchet... 😇
When a GraphQL server updates the GraphQL schema it would be nice if GraphiQL automatically reflected this. Over at PostGraphile we implement this with a text/event-stream from the server, which fires a change event when the schema changes. Event streams are perfect for this because they're server-sent events, you don't need to worry about receiving data from the client (because you can't) and they are much simpler and more widely supported than websockets.
I'm proposing that we make this a standard - if a GraphQL server supports updating its GraphQL schema it should create the header X-GraphQL-Event-Stream in response to every GraphQL request, passing as the value the relative or absolute URL to the text/event-stream event stream.
The client can then subscribe to the event stream (which can be consumed with EventSource), and will re-inspect the schema whenever a change event is received.
An example of the event-stream implementation in PostGraphile is here:
https://github.com/graphile/postgraphile/blob/master/src/postgraphile/http/setupServerSentEvents.js
https://github.com/graphile/postgraphile/blob/5fd2cf218654e03021ef06ffe8f7556a874906b4/src/postgraphile/http/createPostGraphileHttpRequestHandler.js#L286-L295
I've implemented this in GraphiQL.app in this PR. Though doing so doesn't require much code I went a bit further and made sure that the GraphiQL navigation stack is persisted and fixed some other issues also... At the time I wasn't thinking about sending a PR so it kind of snowballed.
If you're interested in any of this I'm happy to send the changes as separate PRs that can be reviewed individually. I understand reviewing this PR as-is is a lot of work!
PS: a lot of the code in here is MIT licensed by @calebmer; I've modified it to make it work in GraphiQL.app.
🙏 Thanks!

PPS: apologies that this PR came through before I'd written a description, I accidentally did Cmd-Enter 😳


Add Feature - Authentication by Google using OAuth 2.0

Hi @Naresh-chandanbatve,
I have identified that there is no option for user to Login using Google authentication.

Benefits :-

This is one of the most authentic way to authenticate user
It will be very convenient for regular users use who don't have account in GitHub/Gitlab/ Bitbucket
It is reliable and secure
Most of the users trust on google
It fast and responsive

Approach:
To add this feature, I propose the following steps:

To implement this feature we will use  Passport which is authentication middleware for Node.js
We will setup passport google oauth2 strategy
We will generate a OAuth client id
then verify the user using Google Account
Save the user data into database

Additional Information:
As a proud participant of GSSOC3, armed with my extensive experience in similar projects, I possess the unwavering confidence to skillfully incorporate this feature .
Thank you for considering my involvement in resolving this issue. I am eager to collaborate with the team to ensure the successful implementation of the necessary improvements.

Can i Work on this issue GSSOC contributor here.

We have already added google auth you can check by runnig the app in local server or in docker container
And dont forget to add api keya in the .env files

If there is a problem we will continue with this issue


issue #27 - setting up commands for the master, worker(s) and services (...

...REST server).
Commands we need to invoke :
$JAVA_HOME/bin/java org.apache.gearpump.cluster.main.Master -ip  -port
$JAVA_HOME/bin/java org.apache.gearpump.cluster.main.Worker -master :
$JAVA_HOME/bin/java org.apache.gearpump.cluster.main.Services -master :
I've extracted class AppConfig. Parameters are pulled from reference.conf but it is possible to override them with command line, like so :
$JAVA_HOME... configuration.yarn.container.count 4 configuration.yarn.container.vcores 2
(a little bit cumbersome)
Maybe we should create some kind of short alias name for cmdLine parameters that will be often overridden to make this more convenient. Will come back to this in the next iteration.
Please share if you have some other ideas.

I'll create another PR for issue #27


ROS2 Galactic Desktop driver installation error

Environment setup:

Jetson Xavier
JetPack 5.0.2 (ubuntu 20.04)
ROS2 Galactic Desktop (pull from dustynv's repo)

To reproduce the error:

git clone this repo with -b galactic
cd ros2_ouster_drivers
colcon build

--- stderr: ros2_ouster
CMake Warning (dev) at /usr/share/cmake-3.24/Modules/FindPackageHandleStandardArgs.cmake:438 (message):
  The package name passed to `find_package_handle_standard_args` (PCL_COMMON)
  does not match the name of the calling package (PCL).  This can lead to
  problems in calling code that expects `find_package` result variables
  (e.g., `_FOUND`) to follow a certain pattern.
Call Stack (most recent call first):
  /usr/lib/aarch64-linux-gnu/cmake/pcl/PCLConfig.cmake:616 (find_package_handle_standard_args)
  CMakeLists.txt:17 (find_package)
This warning is for project developers.  Use -Wno-dev to suppress it.

/tmp/ros2_ws/ros2_ouster_drivers/ros2_ouster/src/ouster_driver.cpp: In constructor ‘ros2_ouster::OusterDriver::OusterDriver(std::unique_ptr<ros2_ouster::SensorInterface>, const rclcpp::NodeOptions&)’:
/tmp/ros2_ws/ros2_ouster_drivers/ros2_ouster/src/ouster_driver.cpp:41:37: warning: ‘const rclcpp::ParameterValue& rclcpp_lifecycle::LifecycleNode::declare_parameter(const string&)’ is deprecated: declare_parameter() with only a name is deprecated and will be deleted in the future.\nIf you want to declare a parameter that won't change type without a default value use:\n`node->declare_parameter<ParameterT>(name)`, where e.g. ParameterT=int64_t.\n\nIf you want to declare a parameter that can dynamically change type use:\n```\nrcl_interfaces::msg::ParameterDescriptor descriptor;\ndescriptor.dynamic_typing = true;\nnode->declare_parameter(name, rclcpp::ParameterValue{}, descriptor);\n``` [-Wdeprecated-declarations]
   41 |   this->declare_parameter("lidar_ip");
      |                                     ^
In file included from /tmp/ros2_ws/ros2_ouster_drivers/ros2_ouster/include/ros2_ouster/interfaces/lifecycle_interface.hpp:20,
                 from /tmp/ros2_ws/ros2_ouster_drivers/ros2_ouster/src/ouster_driver.cpp:22:
/opt/ros/galactic/install/include/rclcpp_lifecycle/lifecycle_node.hpp:369:3: note: declared here
  369 |   declare_parameter(const std::string & name);
      |   ^~~~~~~~~~~~~~~~~
/tmp/ros2_ws/ros2_ouster_drivers/ros2_ouster/src/ouster_driver.cpp:42:40: warning: ‘const rclcpp::ParameterValue& rclcpp_lifecycle::LifecycleNode::declare_parameter(const string&)’ is deprecated: declare_parameter() with only a name is deprecated and will be deleted in the future.\nIf you want to declare a parameter that won't change type without a default value use:\n`node->declare_parameter<ParameterT>(name)`, where e.g. ParameterT=int64_t.\n\nIf you want to declare a parameter that can dynamically change type use:\n```\nrcl_interfaces::msg::ParameterDescriptor descriptor;\ndescriptor.dynamic_typing = true;\nnode->declare_parameter(name, rclcpp::ParameterValue{}, descriptor);\n``` [-Wdeprecated-declarations]
   42 |   this->declare_parameter("computer_ip");
      |                                        ^
In file included from /tmp/ros2_ws/ros2_ouster_drivers/ros2_ouster/include/ros2_ouster/interfaces/lifecycle_interface.hpp:20,
                 from /tmp/ros2_ws/ros2_ouster_drivers/ros2_ouster/src/ouster_driver.cpp:22:
/opt/ros/galactic/install/include/rclcpp_lifecycle/lifecycle_node.hpp:369:3: note: declared here
  369 |   declare_parameter(const std::string & name);
      |   ^~~~~~~~~~~~~~~~~
---
Finished <<< ros2_ouster [1min 37s]

Summary: 2 packages finished [1min 56s]
  1 package had stderr output: ros2_ouster

Sorry I am not so familiarize with the build procedure, could some one help on this? Great appreciate.

Finished <<< ros2_ouster [1min 37s]
Summary: 2 packages finished [1min 56s]
1 package had stderr output: ros2_ouster

Does this mean the installation is fine? those are just the warning nor error... Thanks!


Re: response := this('Runtime.evaluate',...) in Chrome.ahk

Regarding this line: https://github.com/thqby/ahk2_lib/blob/b72b2d06aa6ee0b55965227f68e97e9170128ee5/Chrome.ahk#L228C28-L228C28

Should this be response := this.Call('Runtime.evaluate', ...) ?
Why does response := this('Runtime.evaluate', ...) work as is?

https://www.autohotkey.com/docs/v2/misc/Functor.htm#User-Defined


Fix typo in filename

Thank you @jean. Merged via 0f41d8532

@all-contributors please add @jean for code.


Add customization to Channel Feed Announcements

There is no customization currently - would be nice to add.

how do I activate this?

Is there a way i can get notified when this feature will be stable and live?

I'll update this task when it's done. And also post in http://discord.couchbot.io.


Extension method in class referenced "using static" is not correctly resolved

(I'm finding this a bit hard to explain but hopefully the code below will illustrate it!) The problem is that if I have an extension method in a static class that is in a namespace different to the calling code then I need a "using" in order to access that extension method (eg. using Demo.MyExtensionsNamespace). This works fine but if I use "using static" then I hit problems (eg. using Demo.MyExtensionsNamespace.MyExtensionsClass).
Here is a complete example (available at deck.net/d57ba3d775510a4e93504fba7650b48c) -
namespace Demo1
{
    using static Demo2.Extensions;

    // It works if I replace the line above with this line
    //using Demo2;

    public static class Program
    {
        static void Main()
        {
            "Hi!".Test();
        }
    }
}

namespace Demo2
{
    public static class Extensions
    {
        public static void Test(this string source)
        {
            System.Console.WriteLine("In Test method");
        }
    }
}

This fails because the line "Hi!".Test(); is incorrectly translated into ("Hi!").test();.
If I replace the line using static Demo2.Extensions; with using Demo2; then it does work, the failing line is instead translated into Demo2.Extensions.Test("Hi!");.
Note that in both cases, the code compiles in Visual Studio but the broken case fails at runtime.

This seems to work as expected in repl.it, but dotnetfiddle.net didn't like using static at all.
Repl.it: https://repl.it/@FabricioMurta/issue-4056


TLS 1.2

Fixes Jenkins breaks due to https://github.com/dotnet/announcements/issues/77.

Merging this to unblock other PRs. Feel free to retroactively review.


Fix duplicate shift of image in 3D scene

Overview: What does this pull request change?
As illustrated in #3146, images move differently compared to VMObjects when changing the frame center of the camera.
This will fix #3146.

Motivation and Explanation: Why and how do your changes improve the library?
This bug originates because the points of the image will be transformed twice by the frame center.
Once by the 3DCamera's transform_points_pre_display and once by points_to_pixel_coords.
We removed the point shift by the points_to_pixel_coords.
We also wrote two unit tests for moving the camera in X and Y directions with both an image and a VMobject in the same scene.

Links to added or changed documentation pages
No documentation changed.

Further Information and Comments

This change will introduce a similar problem with the MovingScene, where images won't "move" if the frame center is changed, since the only transformation done to images by the MovingScene/Camera is inside points_to_pixel_coords.
Implementing a transform_points_pre_display for the MovingCamera that transforms the points using the frame center will correctly animate the image, but VMObjects will move twice the distance. We couldn't figure out why/where any additional transformation were applied to VMObjects (especially since they aren't applied in the ThreeDScene).
Any suggestions on why this happens in the MovingScene is welcome, so that we may include that fix as well.

Reviewer Checklist

[ ] The PR title is descriptive enough for the changelog, and the PR is labeled correctly
[ ] If applicable: newly added non-private functions and classes have a docstring including a short summary and a PARAMETERS section
[ ] If applicable: newly added functions and classes are tested

I'll convert this to a draft until the situation is a bit clearer.


[Bug]: analyze`s --backend arg error

Checklist

[X] I've searched for similar issues and couldn't find anything matching
[X] I've included steps to reproduce the behavior

Affected Components

[X] K8sGPT (CLI)
[ ] K8sGPT Operator

K8sGPT Version
No response
Kubernetes Version
No response
Host OS and its Version
No response
Steps to reproduce
2 problems.

set default backend as none openai and give -b openai -> use default backend not openai

> ./bin/k8sgpt analyze -f Service -n default -c -e -b openai
   0% |                                                                                | (0/8, 0 it/hr) [0s:0s]
Error: failed while calling AI provider localai: error, status code: 404, message: %!s(<nil>)

set any default backend and run analyze with -b arg -> only use default backend not provided arg. (i think this is bug)

> ./bin/k8sgpt auth default -p noopai
Default provider set to noopai
> ./bin/k8sgpt analyze -f Service -n default -e -c localai
 100% |███████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| (7/7, 2693 it/s)
AI Provider: noopai

Expected behaviour
ai provider should be provided --backend  even openai is given.
Actual behaviour
No response
Additional Information
No response

@VaibhavMalik4187 sure

It looks like this behaviour is intentional and was introduced by @AlexsJones in the commit cbe2fb4a4c160a0a24b3fb4602cae8e5eebd6aa0.
@AlexsJones, in https://github.com/k8sgpt-ai/k8sgpt/pull/427#discussion_r1194874035, I don't think that the code matches the reasoning. Like, in this case, the backend has been specified as openai and the default backend is also set. But the default backend is being used because it satisfies the following condition and sets the backend to use the default backend:
	if configAI.DefaultProvider != "" && backend == "openai" {
		backend = configAI.DefaultProvider
	}

Changing the default value of the backend flag to "" and updating the if condition to:
	if configAI.DefaultProvider != "" && backend == "" {
		backend = configAI.DefaultProvider
	}

Fixes the issue, I haven't investigated the side effects of this change yet. Would like to know your thoughts on this @AlexsJones, @JuHyung-Son. Thanks!


FadeOut uses wrong cache for ImageMobject

Description of bug / unexpected behavior

Repeated FadeIn and FadeOut of multiple ImageMobjects seems to cause FadeOut to use the wrong cache.
When the attached code was executed with the following command, the animation of the first FadeOut was played during the second FadeOut.
manim -pql <script>

On the other hand, the following command produces the correct animation
manim -pql --disable_caching <script>

Expected behavior

The same behavior regardless of whether the cache is enabled or disabled
How to reproduce the issue

Code for reproducing the problem
from manim import *

class FadeOutImage(Scene):
    def construct(self):
        data1 = np.zeros((100, 100))
        data1[:, 0:5] = 255
        data1[0:5, :] = 255
        data1[:, -6:-1] = 255
        data1[-6:-1, :] = 255
        data2 = np.zeros((100, 100))
        data2[:, 25:30] = 255
        data2[25:30, :] = 255
        data2[:, -31:-26] = 255
        data2[-31:-26, :] = 255
        data = [data1, data2]
        for i in range(2):
            image = ImageMobject(data[i])
            self.play(FadeIn(image))
            self.play(FadeOut(image))

Additional media files

Images/GIFs

Logs
Terminal output

PASTE HERE OR PROVIDE LINK TO https://pastebin.com/ OR SIMILAR

System specifications
System Details

OS (with version, e.g., Windows 10 v2004 or macOS 10.15 (Catalina)): macOS 13.0 (Ventura)
RAM:
Python version (python/py/python3 --version): 3.8.15
Installed modules (provide output from pip list):

PASTE HERE

LaTeX details

LaTeX distribution (e.g. TeX Live 2020):
Installed LaTeX packages:

FFMPEG
Output of ffmpeg -version:
ffmpeg version 5.1.2 Copyright (c) 2000-2022 the FFmpeg developers
built with Apple clang version 14.0.0 (clang-14<IP_ADDRESS>)
configuration: --prefix=/usr/local/Cellar/ffmpeg/5.1.2 --enable-shared --enable-pthreads --enable-version3 --cc=clang --host-cflags= --host-ldflags= --enable-ffplay --enable-gnutls --enable-gpl --enable-libaom --enable-libbluray --enable-libdav1d --enable-libmp3lame --enable-libopus --enable-librav1e --enable-librist --enable-librubberband --enable-libsnappy --enable-libsrt --enable-libtesseract --enable-libtheora --enable-libvidstab --enable-libvmaf --enable-libvorbis --enable-libvpx --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxml2 --enable-libxvid --enable-lzma --enable-libfontconfig --enable-libfreetype --enable-frei0r --enable-libass --enable-libopencore-amrnb --enable-libopencore-amrwb --enable-libopenjpeg --enable-libspeex --enable-libsoxr --enable-libzmq --enable-libzimg --disable-libjack --disable-indev=jack --enable-videotoolbox
libavutil      57. 28.100 / 57. 28.100
libavcodec     59. 37.100 / 59. 37.100
libavformat    59. 27.100 / 59. 27.100
libavdevice    59.  7.100 / 59.  7.100
libavfilter     8. 44.100 /  8. 44.100
libswscale      6.  7.100 /  6.  7.100
libswresample   4.  7.100 /  4.  7.100
libpostproc    56.  6.100 / 56.  6.100

Additional comments

Hello and thanks for the report !
I think this is a duplicate of #2924


nimble/host: Change host-stop procedure

With this PR host is sending in the raw Disconnect Command to all the active connections and then start graceful disconnect timeout 2s.
Host notifies that is stopped when
a) all the disconnection complete events arrived within 2s
b) disconnect timeout fires
btmon logs for a) scenario:
< HCI Command: Disconnect (0x01|0x0006) plen 3                                                                         #1191 110.578000
Handle: 3
Reason: Remote User Terminated Connection (0x13)

HCI Event: Command Status (0x0f) plen 4                                                                              #1192 110.578000
Disconnect (0x01|0x0006) ncmd 1
Status: Success (0x00)
< HCI Command: Disconnect (0x01|0x0006) plen 3                                                                         #1193 110.601500
Handle: 2
Reason: Remote User Terminated Connection (0x13)
HCI Event: Command Status (0x0f) plen 4                                                                              #1194 110.601500
Disconnect (0x01|0x0006) ncmd 1
Status: Success (0x00)
< HCI Command: Disconnect (0x01|0x0006) plen 3                                                                         #1195 110.624900
Handle: 1
Reason: Remote User Terminated Connection (0x13)
HCI Event: Command Status (0x0f) plen 4                                                                              #1196 110.624900
Disconnect (0x01|0x0006) ncmd 1
Status: Success (0x00)
HCI Event: Number of Completed Packets (0x13) plen 5                                                                 #1197 110.632700
Num handles: 1
Handle: 2
Count: 0
HCI Event: Disconnect Complete (0x05) plen 4                                                                         #1198 110.632700
Status: Success (0x00)
Handle: 3
Reason: Connection Terminated By Local Host (0x16)

HCI Event: Disconnect Complete (0x05) plen 4                                                                         #1201 110.695200
Status: Success (0x00)
Handle: 1
Reason: Connection Terminated By Local Host (0x16)
HCI Event: Disconnect Complete (0x05) plen 4                                                                         #1202 110.765500
Status: Success (0x00)
Handle: 2
Reason: Connection Terminated By Local Host (0x16)

@ccollins476ad Thanks, fixed


test: code

feat: add PostListSort
feat: apply sort
feat: remote unused
fix: format
test: code

Summary by CodeRabbit

New Features

Added sorting functionality to the post list view, allowing users to sort posts by creation date, update date, or randomly.

Refactor

Simplified route definitions by removing duplicated decorators.

Bug Fixes

Corrected the naming of the assets Blueprint to asset and updated route decorators accordingly.

@sourcery-ai review


CSP does not allow eval

Can we use another way other than calling eval?
This is not good for security reasons
https://github.com/webpack-contrib/script-loader/blob/master/addScript.js#L9
addScript.js:9 Uncaught EvalError: Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: "script-src 'self'".

    at eval (<anonymous>)
    at e.exports (addScript.js:9)
    at Object.659 (jquery.mask.js?872b:1)
    at __webpack_require__ (bootstrap 3d71110…:54)
    at Object.869 (vendors.js:32296)
    at __webpack_require__ (bootstrap 3d71110…:54)
    at e.20 (bootstrap 3d71110…:152)
    at bootstrap 3d71110…:152

source: https://scotthelme.co.uk/content-security-policy-an-introduction/

@cordoval your wan't eval on bundling step or in compiled code?

the trace i got in production after all the processing that webpack does, so if that is compiled there yes

@cordoval this loader use eval inside, your should search other solution for your case. Can your explain case why your use eval?

i don't use eval and eval is weak for security. This is very clear when you check what it CSP and how attackers can leverage eval() function used in script-loader to break security. My use case is just the plain usage of this script-loader. I am going to close it but this was a big warning.

I have the same issue.
I try to create an application using angular 2 and bootstrap.
Angular is using webpack for its build.
When script loader try to load the first javascript file I get this error:

Uncaught EvalError: Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: "script-src 'self'".

    at eval (<anonymous>)
    at webpackJsonp.../../../../script-loader/addScript.js.module.exports (addScript.js:9)
    at Object.../../../../script-loader/index.js!../../../../jquery/dist/jquery.slim.min.js (jquery.slim.min.js?5773:1)
    at __webpack_require__ (bootstrap c6feb7269e0a6fee60b5:54)
    at Object.2 (scripts.bundle.js:65)
    at __webpack_require__ (bootstrap c6feb7269e0a6fee60b5:54)
    at webpackJsonpCallback (bootstrap c6feb7269e0a6fee60b5:25)
    at scripts.bundle.js:1

Is there any way around it ?

the work around is not using it

I'm not sure I understand the work around.
According to security guidelines I cannot use CSP with 'unsafe-eval'
Does this mean I cannot use angular with additional scripts due to this script-loader issue ?

exactly, you need to use another thing

@cordoval can you propose what this to use?

I am not sure @hotrush , i don't use this anymore, i cut it off long time ago

I'm having this issue too.
I have a legacy minified .js we have to import. script-loader works fine in most cases, except when running in an iframe in an app in Chrome OS and we get the unsafe-eval error.
I've tried using the noParse option in Webpack but that breaks the legacy file for some reason.
Has anyone found workaround?


Unless user removes github repo for deleted project - the project's pipelines are unexpectedly recreated

If a user opts to delete a project, he/she must delete all builds, pipelines, etc. in the Openshift console, as these cannot at present be deleted in the openshift.io UI
However, if the user does not also delete the corresponding github repo for the project, the pipelines that had previously been deleted are recreated when a new project is created - this might surprise users as they have already deleted these pipelines
James S. mentioned that the root cause of this bug is that the jenkins org job doesn't get updated to remove the old project names; so they get reimported.

Related to:
https://github.com/fabric8io/fabric8-init-tenant/issues/75


Use initial transformation not functional?

I am trying to use a translation transform before using affine in two separate registration passes but the use initial transform setting seems not to anything since the program shows the result right away.
To break down the issue in simpler steps:

Rigid registration with Translation-->"Transform1" file
Rigid registration with Euler using initial transformation "Transform1" --> Immediate output without any computation done, just the application of Transform1

I have identical issues also when attempting a first Euler rigid registration followed by BSpline where there is a duplicated rigid transform without any computation. My current solution involves using the interpolated files from the first registration pass and feed them it to the second registration round, not ideal.
Thank you for releasing this by the way!

Hi @DSbiomedAU, usually this should work.
In the "Temporary directory for intermediate files" there should be an elastix.log file.
Could you post this here?
Maybe we can see what is going on.

Hi @tischi, I think I found the problem by looking at the log file as you suggested. I was saving the transformation files without a ".txt" file extension and elastix was throwing an error in the log about it. They are not saved in the temp folder btw.
After fixing my mistake I can see that the second transformation is now applied (e.g. first translation, second Euler) and it doesn't just apply the first transformation. I'm not sure it is actually using the first transformation file to initialize the registration but most likely it is working as intended.
log without .txt extension elastix.log
log with .txt extension elastix.log
Thanks! It has been bugging me for a while, now I will be able to apply multiple transforms in succession without resampling the images at each step. Maybe this could be another documentation improvement point? Users always find a way to make mistakes. ;)

Looking at my documentation I must admit I was underwhelmed ;-)
There is a lot of room for improvement and I will add everything you suggested!
Thanks a lot!
I just have to strike a balance in order not to rewrite the official elastix manual, which, by the way, is a fantastic read!

You make a good point about not rewriting the manual. ;-)
The documentation of the wrapper is quite good and got me going within few attempts, I hit those minor stumbling blocks when trying to use more advanced features.
I agree that the official manual is quite nice and clear!

thanks a lot I added some more documentation: https://github.com/embl-cba/elastixWrapper/commit/d41a655c4d60fb86f1189891465955f1ef94d1d7


Detect device disconnected

Detection of device disconnection. The connection monitoring thread runs in the background and checks the connection once per second.

I'm currently reviewing this PR

@hitnik this breaks the DHCP network modules because the util class is copied into those modules. Docker and psutil libraries are required so the network modules do not start. Could we take out content that requires those modules and place in a separate docker_util class?

@hitnik this breaks the DHCP network modules because the util class is copied into those modules. Docker and psutil libraries are required so the network modules do not start. Could we take out content that requires those modules and place in a separate docker_util class?

@jboddey  I noticed this behavior but didn't know the reason. I will move the function get_sys_interfaces to ip_control module.

Error with the latest commit:
Traceback (most recent call last):   File "framework/python/src/core/test_runner.py", line 100, in <module>     runner = TestRunner(config_file=args.config_file,   File "framework/python/src/core/test_runner.py", line 42, in __init__     self.test_run = Testrun(config_file=config_file,   File "/home/runner/work/testrun/testrun/framework/python/src/core/testrun.py", line 126, in __init__     self.start()   File "/home/runner/work/testrun/testrun/framework/python/src/core/testrun.py", line 332, in start     self.get_session().start()   File "/home/runner/work/testrun/testrun/framework/python/src/common/session.py", line 114, in start     self.reset()   File "/home/runner/work/testrun/testrun/framework/python/src/common/session.py", line 591, in reset     self._ifaces = util.get_sys_interfaces() AttributeError: module 'common.util' has no attribute 'get_sys_interfaces'

Error with the latest commit:
Traceback (most recent call last): File "framework/python/src/core/test_runner.py", line 100, in runner = TestRunner(config_file=args.config_file, File "framework/python/src/core/test_runner.py", line 42, in init self.test_run = Testrun(config_file=config_file, File "/home/runner/work/testrun/testrun/framework/python/src/core/testrun.py", line 126, in init self.start() File "/home/runner/work/testrun/testrun/framework/python/src/core/testrun.py", line 332, in start self.get_session().start() File "/home/runner/work/testrun/testrun/framework/python/src/common/session.py", line 114, in start self.reset() File "/home/runner/work/testrun/testrun/framework/python/src/common/session.py", line 591, in reset self._ifaces = util.get_sys_interfaces() AttributeError: module 'common.util' has no attribute 'get_sys_interfaces'

fixed

I am currently working on a way to automate the dhcp disconnect tests which requires the device to be disconnected which will conflict with this feature.  We will need need a way to pause the monitor thread during specific tests for both features to work.

@hitnik The util class still references the docker module which breaks the DHCP servers. Can the get_docker_host_by_name method be moved to a new docker_utils class?

@hitnik The util class still references the docker module which breaks the DHCP servers. Can the get_docker_host_by_name method be moved to a new docker_utils class?

I've removed this method.

There's a few overlaps with PR #616. Will wait until that one is merged to make this PR easier to review.

Need to get pylint working and I believe you have the incorrect email for a commit somewhere causing a CLA error.

@hitnik Are you still wanting to merge this branch or has the functionality been implemented elsewhere?


Consider removing lodash dependency

Doesn't bring much help considering it adds 18KB to bundle size.
Maybe we can remove everything we don't absolutely need ?

Moved to https://github.com/davidlj95/chrislb/discussions/329


Agility Plugin: Show a timer on the moving blocks in the agility pyramid

The number of times I've been pushed off the pyramid because I thought I had a second longer than I did to run past it is frustratingly high

I feel this could be an easy add, a small circular timer such as the one used in Tears of Guthix?

I believe you would need to be in range to see the last time it did its animation for the countdown to work properly.

I think you would only need to see it move once, then just start counting game ticks from there

I'd like to take a stab at this because I also feel this frustration
(Would be my first runelite contribution so it might take a while :) )

#8647

I created a plugin in the plugin-hub, for those still wanting this feature. https://runelite.net/plugin-hub/show/agility-pyramid-slider-block-timer


fix default cache size

How is this not merged? :8ball:

@geakw Please rebase master to your branch and push force it. I will merge it.

@geakw I've merged your commit


How do I track the same person in video

How do I track the same person in video

Use face descriptor matching. Check out the face recognition tutorials and examples.

Thank you very much！
When will the error be reported？
if (!params) {
  throw new Error('FaceRecognitionNet - load model before inference')
}

params will not be defined if you didn't call the networks load method before attempting to forward an image through the net.


Allow Java Harness to understand Fns the Java Runner produces.

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Yes, this grew out of trying to use the PULR, but I knew we needed for the JULR and will want to migrate the harness to these as well.

Thanks for the review, PTAL.

@kennknowles Any more thoughts on this?

LGTM; you'll need to rebase to resolve conflicts before you can clicky merge. Definitely useful to move forward here. Sorry for the delay.


offset y axis by value

Feature Proposal
My suggestion is that the offset becomes an number. Right now I try to change axis: Scale<CoreScaleOptions>, axis._startValue, axis._endValue, axis._valueRange but I am overwritten evertime something is drawed. Maybe there is way but it is very difficult. I need to offset the y-axis by a lot more than the standart true gives me.
y: {
offset: boolean
}
becomes
y: {
offset: number
}

Possible Implementation
Change offset to take a value.

Did you find any solution?
I have the same problem but for the X axis.

I found out that removing the x border line by drawboarder solve my problem. But if you want you can change the source code:
https://github.com/chartjs/Chart.js/blob/master/src/scales/scale.linearbase.js#L297 this is line that controls the offset I think. It is half of the normal space between each tick on y axis.


UTXO: Adapt input and output formats for more flexibility

This changes the format of transaction inputs and outputs to enable more
future extensibility. For outputs, the Destination type is introduced
to differentiate between various spending types such as P2SH, P2PK,
P2PKH and so on. It is an enum so it is easily extended by adding extra
variants. In inputs, there are two new fields: lock and witness.
These are binary data that are interpreted according to the output
destination.
TODO: Update utxo-tester to reflect the changes introduced here.

Agreed that you should add yourself to the authors at this point

Perfect Lukas!

LGTM. My only concern is how these new enum values are used with python so I'll be postponing merging this to my local node until I'm done with PP tests. Previously the values were used asu8. But looks good and I think you should add yourself to the authors section of lib.rs.

I am in the process of updating the utxo-tester to reflect the changes introduced here. I don't intend to merge this until it's ready and then merge both at the same time.

PR for utxo-tester is up https://github.com/mintlayer/utxo-tester/pull/2


footer enhancement

[x] Background color/image
[ ] Column Features - buzzwords maybe with some icon or descriptions.
[ ] Link to Research Data Management - which column should we put this link.

These points pending for @utnapischtim feedback.
changes needed

[x] Barrierefreiheit => Accessibility
[x] remove Repository column
[x] Accessibilityas a separate column
[x] repository.tugraz.at => Repository

background color/image: #fbfcff (please remove the #margin-divider above the footer)
link to Research Data Management -> Connected Services

(please remove the #margin-divider above the footer)

Remove the whole divider div or just the #margin-divider style?

(please remove the #margin-divider above the footer)

Remove the whole divider div or just the #margin-divider style?

the div and the style. when there is a color, then the border should not more necessary. but some margin-top should be necessary for the footer to increase the vertical space between the content and the footer.


Fix flaky signaling tests

Use channels instead of atomics for tests.
Fixes: #260

Ran this for a couple of minutes without failures. On curious thing I saw is that sometimes some of the tests take a long time (more than 1 seconds), but they still pass.

~/d/m/matchbox_signaling (fix-flaky-signaling-tests|✔) cargo stress
Compiling test binaries: cargo test --no-run --message-format=json
    Finished test [unoptimized + debuginfo] target(s) in 0.09s
Running test binary: "/home/johan/dev/matchbox/target/debug/deps/matchbox_signaling-a34762f6f0bb4182"
Running test binary: "/home/johan/dev/matchbox/target/debug/deps/full_mesh-cb8e8ac1c73fc567"
Running test binary: "/home/johan/dev/matchbox/target/debug/deps/client_server-9bfec17bb3fc2177"
Running test binary: "/home/johan/dev/matchbox/target/debug/examples/client_server"
Running test binary: "/home/johan/dev/matchbox/target/debug/examples/full_mesh"
1 runs so far, 0 failures, over 0s
6121 runs so far, 0 failures, over 5s
11270 runs so far, 0 failures, over 10s
14378 runs so far, 0 failures, over 15s
16718 runs so far, 0 failures, over 20s
18965 runs so far, 0 failures, over 25s
21279 runs so far, 0 failures, over 30s
23592 runs so far, 0 failures, over 35s
25947 runs so far, 0 failures, over 40s
28306 runs so far, 0 failures, over 45s
30634 runs so far, 0 failures, over 50s
32852 runs so far, 0 failures, over 55s
35133 runs so far, 0 failures, over 60s
37493 runs so far, 0 failures, over 65s
39847 runs so far, 0 failures, over 70s
42139 runs so far, 0 failures, over 75s
44311 runs so far, 0 failures, over 80s
46589 runs so far, 0 failures, over 85s
47279 runs so far, 0 failures, over 90s
47279 runs so far, 0 failures, over 95s
47279 runs so far, 0 failures, over 100s
47279 runs so far, 0 failures, over 105s
47279 runs so far, 0 failures, over 110s
47279 runs so far, 0 failures, over 115s
47279 runs so far, 0 failures, over 120s
47279 runs so far, 0 failures, over 125s
47279 runs so far, 0 failures, over 130s
47279 runs so far, 0 failures, over 135s
47279 runs so far, 0 failures, over 140s
47279 runs so far, 0 failures, over 145s
47279 runs so far, 0 failures, over 150s
49059 runs so far, 0 failures, over 155s
51503 runs so far, 0 failures, over 160s
53982 runs so far, 0 failures, over 165s
56456 runs so far, 0 failures, over 170s
58909 runs so far, 0 failures, over 175s

Uh, sometimes it hangs for even longer... maybe we still have some issue to fix. I still think this is better, though.


Start Gradle as lower priority process

There should an option to start gradle and it's daemon as a lower priority process to increase system responsiveness when running parallel tasks/threads.
I would argue that this should be the default.
Expected Behavior

Gradle should not hog all system resources at normal priority as a default. If the daemon is set to start with lower priority, very little performance is lost and system responsitivity increases a lot.
Current Behavior

Currently the wrapper starts gradle at normal priority, with or without the daemon. I would argue that the should be a choice, maybe in the wrapper setting to select default priority level for the gradle process that is started.
I would argue that the default for this should be low prio, but that is not as important as given the user the choice.
Context
My user environment and local VMs grid to a halt when I do heavy parallel compiles using gradle.
Running a parallel gradle build on my 8 thread/24GB/i7 developer machine makes Cinnamon (Linux Mint Window Manager) slow and my local guest virtual machine pretty much stand still.
Instead running: nice -n 15 ./gradlew --no-daemon --parallel build makes the user environment completely usable during the whole build and the performance loss is less than 10%.

Your Environment

Linux Mint, Ubuntu, Debian (all same results)

Given parallel workers and the general push towards parallel by default, this can only get worse. My MacBook is regularly a stuttering mess when I'm running builds, and I either have to limit the maximum workers or nice the process as @staffanf does. This'd be a great improvement.

An important thing to mention is that on Mac OS nice is broken (Apple knows about this) so for the meantime we can't even run nice -n 15 ... :(

If we want to change the default, 5.0 would be a good point to do that. We should do some measurements to see the impact on default performance vs default usability.

For 5.0, I'm going to add an option that you can configure in your user home. Once we have some positive experience with this, we can add auto-detection, e.g. based on whether Gradle is attached to a console.

Question: does --parallel make things worse? What about --daemon? I'm on Gradle 4.8.1 (not my choice). My machine is unusable when I run gradle and I am trying to narrow down what's causing it.

For anyone finding this, see docs:

org.gradle.priority=low|normal
--priority=low|normal


Error  executing the ejabberd installer

Hi,
when i'm trying to build the image from the Dockerfile, in the installation step. If use directly the image from the registry all works well, do you know where could be the problem? I need to build the image cause i need to add something to the config file.
Thank you in advance,
Regards

Hey, please post the output from your build log and from docker version.
Regards

of course! I'm using docker 1.4.1 And the log output is:
Step 12 : RUN wget -q -O /tmp/ejabberd-installer.run "http://www.process-one.net/downloads/downloads-action.php?file=/ejabberd/$EJABBERD_VERSION/ejabberd-$EJABBERD_VERSION-linux-x86_64-installer.run"     && chmod +x /tmp/ejabberd-installer.run     && /tmp/ejabberd-installer.run             --mode unattended             --prefix $EJABBERD_ROOT             --adminpw ejabberd     && rm -rf /tmp/*     && mkdir $EJABBERD_ROOT/ssl     && rm -rf $EJABBERD_ROOT/database/ejabberd@localhost
 ---> Running in 4ab3c7bfd691
INFO[0067] The command [/bin/sh -c wget -q -O /tmp/ejabberd-installer.run "http://www.process-one.net/downloads/downloads-action.php?file=/ejabberd/$EJABBERD_VERSION/ejabberd-$EJABBERD_VERSION-linux-x86_64-installer.run"     && chmod +x /tmp/ejabberd-installer.run     && /tmp/ejabberd-installer.run             --mode unattended             --prefix $EJABBERD_ROOT             --adminpw ejabberd     && rm -rf /tmp/*     && mkdir $EJABBERD_ROOT/ssl     && rm -rf $EJABBERD_ROOT/database/ejabberd@localhost] returned a non-zero code: 1

Thank you for your quick answer

I can reproduce this behaviour on an Ubuntu 12.04.5 System with Docker 1.4.1 and AUFS storage driver. You can try to build the image with the devicemapper storage driver => http://muehe.org/posts/switching-docker-from-aufs-to-devicemapper
But you don't need to build the entire image by yourself to use your own configuration files. Use this image as the base image and copy your configfiles => https://github.com/rroemhild/docker-ejabberd#using-docker-ejabberd-as-base-image
Regards,
Rafael

Ok, i used you image as my base image and all works well, thank you very much, if i need to compile the image for some reason i will do it as you say, thank you again, you save a lot of time with your work!


feat: impl BatchDelete

I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
This pr mainly implements BatchDelete on kv_store
Checklist

[ ]  I have written the necessary rustdoc comments.
[x]  I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)
#1249

The corresponding batch delete API needs to be added in metasrv client crate.

@Fengys123 Rebased, PTAL 😆


Mozilla Firefox

There's no Run button

Well, that's definitely not the right experience. Do you mind giving us some more information about the browser and operating system you're using? Specifically, I'd love to know:

Browser version
OS and version
At which DartPad URL you saw the issue


Specifying what to remove in command line?

I'm interested in removing everything in the System Applications universally safe to remove list via the command prompt in a way that doesn't require user interaction (i.e. a popup where I have to select each one individually). I notice that the command includes -WindowsApps "Select", and I'm wondering if there is a way to modify that to specify what should be removed, and if so, what the syntax would be. For instance, -WindowsApps "BioEnrollment CallingShellApp MicrosoftEdge MicrosoftEdgeDevToolsClient SecHealthUI ContentDeliveryManager ParentalControls SecureAssessmentBrowser XGpuEjectDialog XboxGameCallableUI"

The -WindowsApps parameter removes Provisioned Appx Packages, not System Apps.
The best way to incorporate what you're requesting is to do something similar to the -WindowsApps "Whitelist" feature where only System Applications added to that file are removed.
I had this enabled many versions ago and have been contemplating re-adding it (along with Whitelists for Capability Packages and Windows Features).
I will add this feature in an upcoming build.


v. 2.3.4 visual bug with Too Many Emotes

Whenever one tries to use an emote with Too Many Emotes, when you enter the 3rd person view, you will see a visual bug with floating arms. This does not happen if other players use emotes. Only yourself. This has only been an issue the past couple of days so one of the latest updates seems to have caused this issue.
https://imgur.com/a/4IV0OPF

Are you using any other mods alongside? As I showed in that image I sent earlier, that simply hasn't been my experience, so I'm having trouble tracing the issue.

Thjis is now fixed in 2.3.5. Thanks!


chore: change from yarn to pnpm

This changes the package manager for the dashboard dependencies over
for the reason that pnpm is a bit more uniformly supported and has
more features. pnpm also takes up less storage and is faster at
installing.
We can also scrap this idea and go with installing and using https://bun.sh/
instead of pnpm

TODO:

[ ] Write new tests or update the old ones to cover new functionality.
[ ] Update doc-strings where appropriate.
[ ] Update or write new documentation in packit/packit.dev.
[ ] ‹fill in›

Fixes
Related to
Merge before/after

RELEASE NOTES BEGIN
Change package manager from yarn to pnpm
RELEASE NOTES END

@lbarcziova WDYT of using something like this? I know I've talked a bit with Miro who brought up Bun as well

@Venefilyn I will have a look into those as I am not that familiar with them and let you know! Thanks for the ideas!

@Venefilyn I will have a look into those as I am not that familiar with them and let you know! Thanks for the ideas!

@lbarcziova did you get a chance to look at the current options out there? IIRC just yarn, npm, pnpm, bun

@Venefilyn yes, I looked into a few comparisons, and if the decision would be pnpm vs bun, for now, I would go with pnpm. I see that pnpm can have some advantages compared to yarn, I am just not sure how much benefit it would bring for our use case at the moment. But since you are more experienced in these technologies, if you think this would be worth the transition, we can go for it!

Primarily for now it would help with sharp as it doesn't work very well with yarn as far as I can see, and there are some issues with the different yarn lock file versions I don't like. This might need containers to be updated in base Packit repos but unsure
Anyway, I think it's ready for merge


Ringbuffer object not usable from C++ due to missing extern "C" declaration (linker error)

This may be a duplicate of
https://esp32.com/viewtopic.php?f=13&t=1301
but it seems not to be solved yet.
In
...Arduino\hardware\espressif\esp32\tools\sdk\include\freertos\freertos\ringbuf.h
extern "C"-Declaration is missing:
#ifdef __cplusplus
extern "C" {
#endif

It seems that it was fixed 2 weeks ago in the latest ESP-IDF.  See:
https://github.com/espressif/esp-idf/commit/0b48d5978e4d7173eb7e9932b93234aeb3f2283a
Try and refresh your ESP-IDF to the latest version available and see if that resolves the puzzle.

For any reason get.exe did not refresh my SDK-files.
After deleting the whole repository and cloning it anew, I have the latest files and the declaration is there.
Issue resolved.
Sorry for wasting your time. I should  have had a look into the files via Gitgub webgui.


Login/Setup Wizard

Multi-step wizard walking a user through registration

Sign-in
Import a repository


Support slide_left and slide_right for SSE2, AVX and AVX2

As green as a clover field \o/ @JohanMabille that's ready for review, and as usual, Neon part is for you, looks like https://developer.arm.com/architectures/instruction-sets/intrinsics/#q=vextq_s8 is the way to go there


Avoid showing StopIteration to beginners

Call to item() raises StopIteration on end of file; it will be a common error when mixing it with items(). Consider catching it and raising instead an exception more understandable by beginners.

Solved since sometime before 1.0.


Add on-comment annotation to match PipelineRun from a git provider comment

We now support the on-comment annotation to match a PipelineRuns on
pull requests.
This is only supported on pull_requests and on GitHub, Gitea and GitLab
providers.

Submitter Checklist

[ ] 📝 Please ensure your commit message is clear and informative. For guidance on crafting effective commit messages, refer to the How to write a git commit message guide. We prefer the commit message to be included in the PR body itself rather than a link to an external website (ie: Jira ticket).

[ ] ♽ Before submitting a PR, run make test lint to avoid unnecessary CI processing. For an even more efficient workflow, consider installing pre-commit and running pre-commit install in the root of this repository.

[ ] ✨ We use linters to maintain clean and consistent code. Please ensure you've run make lint before submitting a PR. Some linters offer a --fix mode, which can be executed with the command make fix-linters (ensure markdownlint and golangci-lint tools are installed first).

[ ] 📖 If you're introducing a user-facing feature or changing existing behavior, please ensure it's properly documented.

[ ] 🧪 While 100% coverage isn't a requirement, we encourage unit tests for any code changes where possible.

[ ] 🎁 If feasible, please check if an end-to-end test can be added. See README for more details.

[ ] 🔎 If there's any flakiness in the CI tests, don't necessarily ignore it. It's better to address the issue before merging, or provide a valid reason to bypass it if fixing isn't possible (e.g., token rate limitations).

/retest

added check for output in the latest git  push force for github/gitea as @savitaashture suggested

looks good to me
@piyush-garg PTAL

lgtm


Automater does not work when called from external directory

First of all, thank you for making this tool, it is a great tiemsaver.
I found a small limitation that may be already well known, but I feel it makes the tool a little less convenient for users.
First I have cloned the git repo to /opt/TekDefense-Automater.
If I run it from inside that directory, everything works:
$ python Automater.py google.com
[*] Checking http://www.fortiguard.com/ip_rep/index.php?data=google.com&lookup=Lookup
...

However, if I run it from outside that directory, for example in /tmp, the script doesn't output anything. No results, no errors.
$ cd /tmp
$ python /opt/TekDefense-Automater/Automater.py google.com
$

It would be nice if you could call the script from anywhere, for example with an alias.

I'm interested in this working from a bash script, from a different directory...I'm suspecting that it's because the called files by Automater.py are relative paths vs. absolute. The newest version returns this response, which makes me think I may be right on the paths:
$ python "/bin/automater.py" ae453cf9921e77e8c68e592d44d5839c
Unfortunately there is neither a tekdefense.xml file nor a sites.xml file that can be utilized for proper parsing.
At least one configuration XML file must be available for Automater to work properly.
Please see https://github.com/1aN0rmus/TekDefense-Automater for further instructions.

I've experimented with subroutines changing to the Automater directory, it works (like @ktneely showed above) but it feels...wrong.


Map false unused resources: EKS

Find all default resources in a EKS cluster that kor flags as unused.
List them
Create an issue to exclude them
Bonus: Open merge request with the exclusions
(example can be found here  https://github.com/yonahd/kor/blob/main/pkg/kor/serviceaccounts.go#L16)

Happy to work on this and get the initial scan.

Happy to work on this and get the initial scan.

Assigned. Thank you!

@pbr0ck3r are you available for contributing this or should I look to reassign it?

@pbr0ck3r are you available for contributing this or should I look to reassign it?

I should be able to do it in the next week.


Release v0.4.1

Signed-off-by: Rodny Molina<EMAIL_ADDRESS>

CI failure is due to a well-known issue with Travis setup.