Trying to run mpi job in pod

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
Description
What I am trying to achieve is, having multiple mpi jobs on one host. What works so far is to start one job with the flags --pid=host --net=host --ipc=host --userns=keep-id
To seperate the two jobs it would be nice to have seperated namespaces between the jobs. So I thought this would be a great usecase for a pod.
Since pods share the net and the ipc I created a pod like this:
podman pod create --share=pid --name dyna-00
And then tried to create containers in this pod using mpirun.
But unfortunatly this fails with the following error:
crun:
Error: cannot setns `/proc/47477/ns/pid`: Operation not permitted
write to sync socket: Broken pipe: OCI runtime permission denied error

runc:
Error: container_linux.go:345: starting container process caused "process_linux.go:430: container init caused \"rootfs_linux.go:58: mounting \\\"proc\\\" to rootfs \\\"/chome/poduser/.local/share/containers/storage/overlay/3c2065ca89c1efa71279086b3e45c7ffffe4a9e1e332f4e6f54e8d097f5593fb/merged\\\" at \\\"/proc\\\" caused \\\"operation not permitted\\\"\"": OCI runtime permission denied error
MPI Application rank 3 exited before MPI_Init() with status 126
mpirun: Broken pipe

Steps to reproduce the issue:

podman pod create --share=pid --name test

mpirun -hostfile hostfile podman run --runtime /usr/bin/crun --rm --env-host --userns=keep-id -d --pod test localhost/ls-dyna sleep 600

Describe the results you received:
crun:
Error: cannot setns `/proc/47477/ns/pid`: Operation not permitted
write to sync socket: Broken pipe: OCI runtime permission denied error

runc:
Error: container_linux.go:345: starting container process caused "process_linux.go:430: container init caused \"rootfs_linux.go:58: mounting \\\"proc\\\" to rootfs \\\"/chome/poduser/.local/share/containers/storage/overlay/3c2065ca89c1efa71279086b3e45c7ffffe4a9e1e332f4e6f54e8d097f5593fb/merged\\\" at \\\"/proc\\\" caused \\\"operation not permitted\\\"\"": OCI runtime permission denied error
MPI Application rank 3 exited before MPI_Init() with status 126
mpirun: Broken pipe

Describe the results you expected:
Creation of 24 containers that sleep for 600 seconds
Additional information you deem important (e.g. issue happens only occasionally):
When looking at the pod after the error happens, there are ~5 containers associated to the pod. But they are in status created and are not running
Output of podman version:
Version:            1.6.4
RemoteAPI Version:  1
Go Version:         go1.12.12
OS/Arch:            linux/amd64

Output of podman info --debug:
debug:
  compiler: gc
  git commit: ""
  go version: go1.12.12
  podman version: 1.6.4
host:
  BuildahVersion: 1.12.0-dev
  CgroupVersion: v1
  Conmon:
    package: conmon-2.0.6-1.module+el8.1.1+5259+bcdd613a.x86_64
    path: /usr/bin/conmon
    version: 'conmon version 2.0.6, commit: 6ffbb2ec70dbe5ba56e4bfde946fb04f19dd8bbf'
  Distribution:
    distribution: '"centos"'
    version: "8"
  IDMappings:
    gidmap:
    - container_id: 0
      host_id: 1002
      size: 1
    - container_id: 1
      host_id: 165536
      size: 65536
    uidmap:
    - container_id: 0
      host_id: 1002
      size: 1
    - container_id: 1
      host_id: 165536
      size: 65536
  MemFree:<PHONE_NUMBER>8
  MemTotal:<PHONE_NUMBER>08
  OCIRuntime:
    name: runc
    package: runc-1.0.0-60.rc8.module_el8.1.0+237+63e26edc.x86_64
    path: /usr/bin/runc
    version: 'runc version spec: 1.0.1-dev'
  SwapFree:<PHONE_NUMBER>
  SwapTotal:<PHONE_NUMBER>
  arch: amd64
  cpus: 48
  eventlogger: journald
  hostname: smtcac1903
  kernel: 4.18.0-147.3.1.el8_1.x86_64
  os: linux
  rootless: true
  slirp4netns:
    Executable: /bin/slirp4netns
    Package: slirp4netns-0.4.2-2.git21fdece.module+el8.1.1+5460+3ac089c3.x86_64
    Version: |-
      slirp4netns version 0.4.2+dev
      commit: 21fdece2737dc24ffa3f01a341b8a6854f8b13b4
  uptime: 1493h 26m 33.17s (Approximately 62.21 days)
registries:
  blocked: null
  insecure: null
  search: null
store:
  ConfigFile: /chome/poduser/.config/containers/storage.conf
  ContainerStore:
    number: 81
  GraphDriverName: overlay
  GraphOptions:
    overlay.mount_program:
      Executable: /bin/fuse-overlayfs
      Package: fuse-overlayfs-0.4.1-1.module_el8.1.0+237+63e26edc.x86_64
      Version: |-
        fuse-overlayfs: version 0.4.1
        FUSE library version 3.2.1
        using FUSE kernel interface version 7.26
  GraphRoot: /chome/poduser/.local/share/containers/storage
  GraphStatus:
    Backing Filesystem: xfs
    Native Overlay Diff: "false"
    Supports d_type: "true"
    Using metacopy: "false"
  ImageStore:
    number: 1
  RunRoot: /tmp/run-1002
  VolumePath: /chome/poduser/.local/share/containers/storage/volumes

Package info (e.g. output of rpm -q podman or apt list podman):
podman-1.6.4-2.module+el8.1.1+5363+bf8ff1af.x86_64

Additional environment details (AWS, VirtualBox, physical, etc.):
physical

Sorry @firefly-serenity This seems to have been lost i the noice.
@adrianreber @giuseppe PTAL

both crun and runc fail for the reason that we are trying to join a PID namespace and at the same time create a new user namespace.  The new user namespace doesn't own the PID namespace, so it cannot mount /proc.  We can't even workaround it from Podman as we cannot bind mount PID from the host.
We need to support --userns keep-id at the POD level so that it is not necessary to specify it for the container.
I will play with it and see if I can do anything from crun to handle this case.

I am still trying to figure out a way to address it in the OCI runtime.
I am going to close it as a dup of https://github.com/containers/libpod/issues/6153


UX: Secret Recovery Phrase Validation screen can't be shown on small devices

What were you trying to do
Validate a Secret recovery phrase on iPhone SE and iPod Touch

What happened
The view does not fit the screen

I will create two solutions here to test: 1) responsive resizing to try and work inside the screen with the existing UX 2) a different UX for smaller views and accessibility needs.

For an accessibility issue, where someone has jacked up their font-size we would have to used one of these two solutions (I think the second solution is a better UX):


Add tests for Failing status and for basic logs

/lgtm

/retest

/hold
Ok, recent changes has made impossible to get into the Failing state, all configurations are valid.

Added a new check that Replicas >= 0.
/hold cancel


Loop enhancements

While it is possible to loop N times using
- for-each: IT ${{= [...Array(${{N}}).keys()] }}
  then: ...

it would be convenient to have a simpler syntax, e.g.
- for-each:
  name: IT
  range: 1 ${{N}}
  then: ...

where range is in form ?<start> <end> ?<increment>.
Also, it would be nice to be able to break out of a loop.

Ouch, the above even breaks due to whitespaces in the  expression - please limit the String.split to 2 parts.

I think this is a bug in how the yaml scalar is parsed. The Parser.split() should not split a state expression (${{...}}) but right now it is too simplistic. I will work on that.

41bb087 should fix the bug with parsing the spaces in a state pattern.
I agree, ${{= [...Array(${{N}}).keys()] }} is quite cumbersome when we want to use for-each to loop a fixed number of times.
What do you think about adding a javascript range method?
We could use ${{= range(${{N}) }} for [0,N), ${{= range(1,${{N}) }} for [1,N), or ${{= range(1,${{N}},5) }} to step by 5 each time?
I like the idea of adding the javascript method because it keeps the yaml syntax the same for the different for-each use cases but I do worry that it will be hard to discover unless documentation is improved. The mapping yaml for for-each:
- for-each:
    name: IT
    input: ${{=range(1, ${{N}} ) }}
  then:
  ...

If we want to expand the yaml syntax to be more declarative then what do you think about adding range as a possible child for input

for-each:
name: IT
input:
range:
start: 1 #default would be 0
stop: ${{N}}
step: 5 #default would be 1
then:
...

The Parser would convert the yaml `range` into the json array we already use internally. I am personally not a fan of the declarative approach but I agree it is worth considering.

*breaking out of a loop
We could add a check to the yaml mapping but then what should be the value of the check? One idea is we can use the javascript engine and check is a function that acts on the state and returns something falsey if the loop should abort.
- for-each:
    name: IT
    input: [ 0, 1, 2, 3, 4, 5]
    check: |
      (state) -> {
        return state.foo == "OK"
      }
  then:
  ...

This would let us extend the supported javascript method parameters to include a snapshot of signals and counters without having to create a descriptive language for all of the possible checks we might want to perform. Perhaps something like
- for-each:
  ...
  check: |
    (state,signals,counters) -> {
      if(signals[${{signal_name}}].count > 0){ #more than 0 are remaining to signal
         return true;                          #continue the loop
      } else {
         return false;                         #stop the loop
      }
    }

Using range in Javascript is not as concise as just range: 1 ${{N}} but good enough. User needs to know about existence of some syntax anyway - if you link to range function in the for-each docs section it is as good as a special syntax.
In the structured YAML I don't think you need to nest range under input - I'd put those at the same level.
I would only use break and/or break-if instead of `check (and invert the condition function).
Rather than enumerating state, signals and counters as the optional parameters you could create an ultimate JavaScipt object encompassing all qDup (lodash-style) - you wouldn't need to remember if signals comes before counters or afterwards, just use $.counters etc.

I like the idea of a wrapper similar to the Context we provide to Cmd.
Do you think it should be a static global or an argument to the break javascript function? If it's global it could be available in js check and in any ${{=...}} javascript evaluation.

I guess that a global variable is more convenient - it would not be necessary to type as an argument (guessing the position when there are other arguments) for the function.


When saving an entity containing a HasOne relationship, Voyager tries to write the related id into the wrong table.

Laravel Version: 5.5.*
Voyager Version: 1.0.0
PHP Version: 7.1
Database Driver & Version: mysql, utf8mb4, utf8mb4_unicode_ci

Description:
The entity cannot be saved since voyager is complaining about a missing field. That field is supposed to be looked up in the db table of the related, not the source object.
Example:
An Event has one Agenda. The agenda has a db column named event_id.
The Model classes are as follows:
App\Event.php
class Event extends Model
{
  public function agenda()
  {
    return $this->hasOne(Agenda::class);
  }
}

App\Agenda.php
class Agenda extends Model
{

  public function event()
  {
    return $this->belongsTo(Event::class);
  }

}

DB table for agendas:

DB table for events:

The BREAD config of Event looks like this:

Problem 1:
No dropdown is rendered (see screenshot)

Problem 2:
When trying to create a new event, an error is thrown:
"SQLSTATE[42S22]: Column not found: 1054 Unknown column 'event_id' in 'field list' (SQL: insert into events (title, event_id, updated_at, created_at) v ▶"
Problem 3:
The text describing the relation is wrong:

Steps To Reproduce:
Extract and run the attached project. The DB dump is inside the app's root folder. It is populated with three Agenda Entities. Go to Events in the Menu on the left and try to create a new one.
Expected result: The Form renders a dropdown, allowing the user to select an Agenda.
Actual result: "Agenda 1" - the name of the first Agenda Entity is rendered as plain text instead of a dropdown. When trying to save the Event an Exception is thrown.

Test Project:
voyager-relations-test.zip

vendor/tcg/voyager/src/Http/Controllers/Controller.php:
change:
         if ($row->type == 'relationship') {                 $row->field = @$options->column;             }
to:
            if ($row->type == 'relationship') {               if ($options->type == 'hasMany')  {                 break;               } else {                 $row->field = @$options->column;               }             }
Skips hasMany relationship fields. Not sure if there's a use-case where some update actually needs to happen on the related rows.

Running into the same issue, currently setting up these relationships without Edit, Add and Delete options, which avoids the issue.

@alonweinstein problem not in hasMany.
I and @artifacts have same problem with hasOne.
When add simple belongsTo - all works:

but if after that add hasOne:

After saving the user I get an error similar to the one above
Illuminate \ Database \ QueryException (42S22)
SQLSTATE[42S22]: Column not found: 1054 Unknown column 'user_id' in 'field list'
(SQL: update `users` set `password` = XXXXXXXXX, `updated_at` = 2017-11-17 22:06:16,
 `user_id` = where `id` = 1)

1 is just not supported, 3 was fixed in https://github.com/the-control-group/voyager/pull/2155.
If 2 is still a problem in the current Voyager version, please open a new issue as there were a lot fixes related to relationships.


CompletionProposal does disable <Alt+F4>

I have a form with a TBCEdior and open the CompletionProposal window. Now I press  to close my form - but nothing happens.
If I press  the CompletionProposal window closes, and I can press  to close my form.
I have an application with the WS_POPUP and WS_BORDER ... but I can use the <Alt+F4> button, while the popup is open.
Btw: Would be nice, if you would add the CS_DROPSHADOW style:
Params.WindowClass.Style := Params.WindowClass.Style or CS_DROPSHADOW;

... but until now, I didn't find a stable solution for this in my application.

I added cpoShowShadow option for completion proposal. Some like it, some don't.
I am not sure about that other part but now function keys goes back to editor. Does it help?

 still does not work on my end....

Now if I press , I get an AV in TBCEditorKeyboardHandler.ExecuteKeyDown in line 9.

Cool.
What do you mean by ? I know Alt+F4 but plain F4 is a mystery for me...

Sorry, my mistake. I ment <Alt+F4> for closing the application.

That seems to work.

Ok, you are not able to see the problem. I respect this.

I can't repeat the issue, so that's it.


fix issue #1934

Codecov Report
Base: 76.52% // Head: 76.52% // Decreases project coverage by -0.00% :warning:

Coverage data is based on head (3c242d5) compared to base (35623b5).
Patch coverage: 100.00% of modified lines in pull request are covered.

:exclamation: Current head 3c242d5 differs from pull request most recent head e4b08a9. Consider uploading reports for the commit e4b08a9 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2193      +/-   ##
==========================================
- Coverage   76.52%   76.52%   -0.01%
==========================================
  Files         580      580
  Lines       48180    48180
==========================================
- Hits        36872    36869       -3
- Misses       9289     9291       +2
- Partials     2019     2020       +1

Flag
Coverage Δ

go-1.15-386
76.56% <100.00%> (+0.04%)
:arrow_up:

go-1.15-amd64
76.55% <100.00%> (+0.01%)
:arrow_up:

go-1.16-386
76.55% <100.00%> (+0.03%)
:arrow_up:

go-1.16-amd64
76.56% <100.00%> (+<0.01%)
:arrow_up:

go-1.17-386
76.58% <100.00%> (+0.02%)
:arrow_up:

go-1.17-amd64
76.56% <100.00%> (+0.02%)
:arrow_up:

go-1.18-386
76.47% <100.00%> (+0.05%)
:arrow_up:

go-1.18-amd64
76.49% <100.00%> (+0.05%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

database/gdb/gdb_func.go
85.12% <100.00%> (ø)

os/gfsnotify/gfsnotify_watcher_loop.go
83.19% <0.00%> (-2.53%)
:arrow_down:

os/gfsnotify/gfsnotify_watcher.go
81.51% <0.00%> (-2.53%)
:arrow_down:

os/glog/glog_logger_rotate.go
64.70% <0.00%> (+1.35%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Create 0000-01-02-steve019-chen.md

Created my first file in slideshow project.

approve and merge

well done

Well done


Please fix terrain example!

Hi @rogerscg ! Would be great if you could fix the terrain example & put it in the demos so I & other people can test it out!
Thanks a bunch, brother! <3

@rogerscg : Hello?

Hey @Thundros , found a bit of time to work on the engine. I'm hoping to get to a terrain example soon. The way the engine is used/built is a bit different now.

@rogerscg : I would be honored to beta test it! <3 When are you going to have a terrain example?
MUCH thanks brother! <3

Hi @Thundros , I recently fixed the terrain example. It's a free-roam entity with a terrain map. It takes a bit to load since I'm also generating terrain textures on the client. Take a look when you get the chance.


Parser: push var for type declaration

Fixes #8611

It seems this conflicts with #4114 and #4106
I don't know what to do. Maybe reverting #4114 is good because #8611 will start working and #4106 will still give an error message, though not the ideal error message (but probably we can change the error message to say "undefined local variable blah blah").

I was trying to search for that issue, to bring it up against this change. I think being able to declare variables this way isn't necessary, and the previous behaviour was correct. In general, I don't think macros should be allowed to declare local variables.
In the absence of finding any past discussion though, I did approve this PR.

It does work if you assign to a variable, though. That's why I think it should work in all cases, or in none.

That's why I think it should work in all cases, or in none.

I agree.

Closing because this didn't work.


What version does this run on?

Hi, you can find this in the package.json.
Right now it is: 2020.2 or above.


How to refresh or reload page - use dom() function to run location.reload()

Hello Sir,
How to refresh page or reload page if data is invalid in form
Because here run it in loop so if 2 loop of data is invalid then  it will continue  with 3 loop until the loop completed

Thank You Sir ,
It is very useful


Zephyr support

We need to discuss how we want to provide Zephyr support for the examples. This might get quite complex with plenty of #ifdefs.
The current example compiles only for Portenta H7 qith errors for native_sim and qemu_x86. Nonetheless a first start.
Not ready for merging.

This PR needs to be also tested on hardware. Another important factor is how do we plan to integrate support for, e.g.,, QNX and other platforms in the future.

I think one issue on the can side here is, that acf-can-common actually ist NOT common, so I guess the examples would need to be reworked.  (Incidentally just today I was mulling over the idea using io_uring on Linux, which probably lead the the same mess and again would "un-commonize" that file).


CSharp missing "pragma" preproc

(Original issue 342 created by ananelson on 2008-06-28T14:31:54+00:00)
I think CSharp is missing a "pragma" preproc directive in the list of keywords on line 96 and 97 of the dotnet lexer.
I know zilch about CSharp, this came up in testing a DelegatingLexer of which CSharp was one of several delegated languages. 2 people independently emailed me to say that #pragma.. is a valid CSharp directive. Because pragma is not on the list of preproc directives, the preceding # is showing up as an error.

(Original issue was assigned to thatch)
(Original issue 342 last updated on 2008-07-26T16:45:58+00:00)
(Issue automaticaly closed due to status in Bitbucket: resolved)

(Original comment by gbrandl on 2008-07-26T13:40:19+00:00)
Now merged to main. Thanks!

(Original comment by thatch on 2008-07-26T04:45:22+00:00)
Confirmed, #pragma is valid in the 4th edition of ECMA 334.  Fix committed in [0abb01a39652], not yet merged.

(Original comment by ananelson on 2008-06-28T15:23:37+00:00)
I have just been informed that "pragma" and some others were added in CSharp version 2, so perhaps the current grammar is based on v 1?


WIP: Cni helm add master

This is a cherry-pick of https://github.com/istio/istio/pull/11877

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: john-a-joyce
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: rshriram
If they are not already assigned, you can assign the PR to them by writing /assign @rshriram in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/test e2e-simpleTests-cni

@john-a-joyce: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
14b4126abbd1cc4c44e61b59f8a2fad3957dcfb3
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@john-a-joyce: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
14b4126abbd1cc4c44e61b59f8a2fad3957dcfb3
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
14b4126abbd1cc4c44e61b59f8a2fad3957dcfb3
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

This is good to go,  but most of it will be picked up if a final sync is done from release 1.1 to master at release time.   I will leave it up to the branch owners if this should merge now as is or be reworked after the 1.1->master sync to pick up the few additional changes.

@john-a-joyce: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
14b4126abbd1cc4c44e61b59f8a2fad3957dcfb3
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
1b546c59d7dabac9d10d75d37538b9b1feb10906
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

This was merged via 1.1 mass merge to master


Are roadmarks available in opendrive standalone mode?

Hi, I'm using the opendrive standalone mode and from what I have read, there are no possibilities to displaying roadmarks with this mode.
Am I missing something? If no, I was wondering if it is a potential feature in future releases?
Thank you!

We are not rendering lanemakrs, but if you visualize the map using the nonrendering_mode script: https://github.com/carla-simulator/carla/blob/master/PythonAPI/examples/no_rendering_mode.py
That should offer you more information.
In the future, we will provide the functionality to render lane markings and other road features.


Problem to get bounding boxes of some countries

I tried to get bounding boxes of several countries, but problem occurred for the following countries: US, Russia, and New Zealand
My impression that this problem occurred because territories of these countries are crossing 180 degree of longitude, but is this problem of Nominatim itself, rather than this geopy library?
The results I got:
US: ['-14.7608358', '71.6048217', '-180', '180']
Russia: ['41.1850968', '82.0586232', '-180', '180']
New Zealand: ['-52.8213687', '-29.0303303', '-179.059153', '179.3643594']
There was no problem with following countries:
UK: ['49.674', '61.061', '-14.015517', '2.0919117']
Canada: ['41.6765556', '83.3362128', '-141.00275', '-52.3231981']

I found that this is a problem of Nominatim itself.
I found two issues in openstreetmap/Nominatim repository:
https://github.com/openstreetmap/Nominatim/issues/234
https://github.com/openstreetmap/Nominatim/issues/184
So, I will try to explore more by myself.
Thanks.


bareos: Mark as broken

Motivation for this change

Doesn't have a maintainer.
Doesn't work with our new glusterfs version.
bareos18 has changed from autotools to cmake so the derivation has to be
completely rewritten.

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc #68361

backported in d0d0a151751240bceb835c4b5b1ab84112d0daab


Power Toys Suggestion

Description of the new feature / enhancement

Enable hiding system tray icon - helps to look windows look a lot cleaner
Allow to change positioning of the PowerToys Run - So that its position can be changed from that fixed top to centre, or to bottom depending on the user.

Scenario when this would be used?

For first suggestion, it will be used when someone wants to simple make the system look cleaner and less bulky.
The later suggestion will used to make the search bar pop up at a specific place just for the user preference and convinience.

Supporting information
No response

Duplicate of #9526

Duplicate of #4576

Issue is a duplicate
Thank you for your issue! We've linked your report against another issue to centralize the thread.
Thanks for helping us to make PowerToys a better piece of software.


[BUG] Networks tab does not show all containers

I have the newest portainer.
Docker 1.12 from sock.
Networks tab -> bridge, shows me only two containers but I have 3 running in bridge network:

If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below.

Description
Briefly describe the problem you are having in a few paragraphs.
Steps to reproduce the issue:

Any other info e.g. Why do you consider this to be a bug? What did you expect to happen instead?
Technical details:

Portainer version:
Target Docker version (the host/cluster you manage):
Platform (windows/linux):
Command used to start Portainer (docker run -p 9000:9000 portainer/portainer):
Target Swarm version (if applicable):
Browser:

Portainer calls docker api
Do you see 2 or 3 containers with docker network inspect bridge ?
If you see 2, portainer works as expected !

I know Portainer calls docker api but still it uses bridge network to host ui.
As I expect docker network inspect bridge gives me 3 containers.
Portainer does not shows itself under network -> bridge tab.

I cannot reproduce this issue. @ghost Are you able to reproduce using the latest Portainer version (1.17.0)?


Setting relative temperature for climate

Hello to everyone,
working with climate element, you can read the actual temperature in label like this
Temp:  {{temperature}}°
also in label, you can easily add or subtract to that temperature, like {{temperature+1}} and works fine.
But seems not works when you try the same operation in Data JSON for the service "Set temperature", like this
{
"temperature" : "{{temperature+1}}"
}
Do I have wrote something wrong in code or is it an actual limitation?
The final goal is having buttons to increase or decrease the temperature for baby steps, starting to the actual.
Thank you in advance

Hi,
this is a limitation. The data json is currently static, so no variables are evaluated.
This may change, but not soon :/

Maybe you can implement what you want by creating a script in homeassistant and call that instead of the "Set temperature" service.

Maybe you can implement what you want by creating a script in homeassistant and call that instead of the "Set temperature" service.

good idea, thanks

I tried if it works as i would expect, and at least calling a script works fine.

(if you see all entities in the drop down and not only scripts - that is a bug that will be fixed with the next release. Until then you have to search...)

yes, I see the list, thanks


build(deps): Replace dependency winapi by windows-sys

While reviewing my dependencies, I noticed both winapi and windows-sys in my dependency tree. Both provide a similar functionality, but windows-sys seems to be actively maintained by Microsoft.
Are you open to replacing this depencency? If not, please say so.
Is a version range preferred for this dependency? I found that this version range works: version = ">= 0.32, <= 0.52". If so, I can add a minimal version CI build.

Is a version range preferred for this dependency? I found that this version range works: version = ">= 0.32, <= 0.52". If so, I can add a minimal version CI build.

Are you interested in the version range + dependabot combination? That way, updated dependency versions are less of a pain.

Are you interested in the version range + dependabot combination? That way, updated dependency versions are less of a pain.

Oh sorry, I missed your question. I'm not sure about version range but for automation, I've been using renovate in zbus and busd. I'd very much prefer keeping it consistent across projects I maintain.

Oh and btw, I appreciate that you took the time to add all the details to the commit log (apart from the PR description) but in future, please break the lines on 74 chars (the usual convention for git). Otherwise it's harder to read the log in some clients

.

just FYI, I enabled renovate for this repo as well now and it already created PRs for updating deps:
https://github.com/zeenix/xdg-home/pull/8
https://github.com/zeenix/xdg-home/pull/9


Add OIDC Login to kkpctl

Is your feature request related to a problem? Please describe.
I as an KKP User want to use kkpctl to login to KKP using my KKP OIDC Login
Describe the solution you'd like
I can use kkpctl oidc-login similar to kubectl oidc-login

This is implemented in c8a0058b8a2d91679c08031ffe37ab31a8d86a18


Update America/Caracas Windows display name offset

As of April 27, 2016, Venezuela (America/Caracas) time zone offset changed to GMT -04:00 see https://www.timeanddate.com/news/time/venezuela-change-timezone.html.

PR has gone stale, closing.


[Quarkus 2.5.0] Kafka SASL / SSL tests are failing

Needs investigating properly, but I don't think the custom configs to apply the necessary security bits to the Kafka broker are working. It may be related to the testcontainers version bump from 1.16.0 to 1.16.2.
I'm disabling the tests for now.

@jamesnetherton I think you are right. This is related to testcontainers 1.6.2 changes with
https://github.com/testcontainers/testcontainers-java/blob/4df9b900e79e4a3033ef0975abacf9064a0e7353/modules/kafka/src/main/java/org/testcontainers/containers/KafkaContainer.java#L76-L86
it set KAFKA_ADVERTISED_LISTENERS with BROKER:localhost:9092 explicitly. So Kafka SSL is not enabled when lauching the instance.
I think this is a bug in testcontainers.

It should be a simple workaround to override protected void configure() to set KAFKA_ADVERTISED_LISTENERS with SSL://localhost:9093,BROKER://localhost:9092. I've checked that it works on my local machine.

This should be fixed in quarkus-main branch.

I think we generally prefer marking issues as fixed only after they are merged in main. quarkus-main is an instable (is rebased regularly) branch and so the sha1 of the fix will change several times till it reaches main. I mean each fixed issue should contain a stable reference to the fix. This is important for both to be able to audit the fix at any time in the future and also to be able to port the fix to older branches.

OK, I get it. Let's reopen it.


While Destroying Security Hub module I ran into an error for deletion of S3 bucket

Describe the bug
While Destroying Security Hub module I ran into an error for S3 bucket that it cant be deleted due to sufficient amount of data present in bucket so either delete it manually and then TF will delete that config s3 bucket
To Reproduce
Steps to reproduce the behavior:

Destroy security HUB module
You will get the below error

Expected behavior
variable "force_destroy" {
  type        = bool
  description = "(Optional, Default:false ) A boolean that indicates all objects should be deleted from the bucket so that the bucket can be destroyed without error. These objects are not recoverable"
  default     = false
}

Screenshots

@vijay-stephen  This variable force_destroy can be added in our module and I know this has all the config data but if someone wants to delete it then we should have this variable added at least so that someone dont have to empty that s3 bucket manually  cc: @jamescrowley321  @tsaucier-sf


Population collapsing without dynamics

This code implements the evolutionary prisoner's dilemma from Kristian Lindgren.
The slight difference in this code is that the noise of a player's action (player plays the opposite action than what their strategy selected).
However, instead of displaying any dynamics, or Tit-for-Tat emerging as the winner over time, the population quickly dies and converges to one or two players playing strategy (1, 1).
Parameters
N = 1000 # number of agents
https://github.com/clemp/evolutionary-prisoners-dilemma/blob/7a96954f17f1e559975b7e426db837927bca50d6/model.py#L295
p_duplicate = pow(10, -5)
https://github.com/clemp/evolutionary-prisoners-dilemma/blob/7a96954f17f1e559975b7e426db837927bca50d6/model.py#L234
p_mutate = 2 * pow(10, -5)
https://github.com/clemp/evolutionary-prisoners-dilemma/blob/7a96954f17f1e559975b7e426db837927bca50d6/model.py#L236
p_split = pow(10, -5)
https://github.com/clemp/evolutionary-prisoners-dilemma/blob/7a96954f17f1e559975b7e426db837927bca50d6/model.py#L238
Results
iteration:  0  | population: 1000
{(1, 0): 247, (0, 1): 251, (1, 1): 255, (0, 0): 247}
iteration:  1  | population: 661
{(1, 0): 247, (1, 1): 255, (0, 1): 89, (0, 0): 70}
iteration:  2  | population: 399
{(1, 0): 142, (0, 1): 73, (1, 1): 154, (0, 0): 30}
iteration:  3  | population: 250
{(1, 0): 81, (0, 1): 46, (1, 1): 93, (0, 0): 30}
iteration:  4  | population: 153
{(0, 1): 26, (1, 1): 57, (1, 0): 49, (0, 0): 21}
iteration:  5  | population: 90
{(0, 1): 14, (1, 1): 32, (1, 0): 32, (0, 0): 12}
iteration:  6  | population: 52
{(1, 1): 18, (1, 0): 21, (0, 0): 8, (0, 1): 5}
iteration:  7  | population: 29
{(1, 1): 11, (1, 0): 12, (0, 1): 2, (0, 0): 4}
iteration:  8  | population: 15
{(1, 0): 8, (1, 1): 5, (0, 1): 1, (0, 0): 1}
iteration:  9  | population: 7
{(1, 0): 4, (1, 1): 3}
iteration:  10  | population: 3
{(1, 0): 2, (1, 1): 1}
iteration:  11  | population: 2
{(1, 1): 1, (1, 0): 1}
iteration:  12  | population: 2
{(1, 1): 2}

Help wanted
There's something odd about these results.
I'm not sure if my fitness function that decides whether an agent reproduces or dies is working as intended.
Logic leading to reproduction
https://github.com/clemp/evolutionary-prisoners-dilemma/blob/7a96954f17f1e559975b7e426db837927bca50d6/model.py#L219-L222
Logic leading to death
https://github.com/clemp/evolutionary-prisoners-dilemma/blob/7a96954f17f1e559975b7e426db837927bca50d6/model.py#L263-L267

Also I think the fitness calculation for each agent against the average is incorrect in the way I have it implemented here.
For example, in the first iteration, after each agent has played each other agent and the payouts have been accumulated to the agent, the current solution returns these values:
{
    'id': 0,
    'memory_length': 1,
    'genome': (0, 0),
    'fitness': 1530,
    'memory_history': {
        0: None,
        1: [...],
        2: [...],
        3: [...],
    ...}
}

The average fitness from this round is 2269.857
So then the adjusted fitness for this agent is calculated
https://github.com/clemp/evolutionary-prisoners-dilemma/blob/7a96954f17f1e559975b7e426db837927bca50d6/model.py#L216-L217
In this example fitness = 1530 - 2269.857 = -739.857
Then in the next lines to determine reproduction or death..
https://github.com/clemp/evolutionary-prisoners-dilemma/blob/7a96954f17f1e559975b7e426db837927bca50d6/model.py#L263-L269
d * abs(fitness) = 73.9857 which of course will always be greater than random.uniform(0, 1)
What needs to be modified so that the fitness value determining reproduction or death is a value bounded in [0, 1]?


clientv3: won't connect if any endpoint is down

How to reproduce: start one etcd server on 2379, then ETCDCTL_API=3 etcdctl put abc def. It will fail to connect to <IP_ADDRESS>:{2,3}2379 and never send the command.

Why not disable grpc.WithBlock ?  It seems grpc can handle connection issues like this automatically in the background if  grpc.WithBlock is disabled.

@westhood I think the code should confirm it can connect to some endpoint before returning a client, but non-blocking connect is a possible alternative.


How to export an ONNX with opset version = 13?

How to export an ONNX with opset version = 13? Currently, the silero_vad.onnx is opset version = 16.
Could you tell me how to get other opset version of the ONNX model?
Thanks

While the new release is fresh, we are able to export with certain other opsets.
If this is necessary, please provide the necessary opsets and some reasoning.

If this becomes relevant, please open a new issue.

I would like to downgrade to opset versions lower than 15 because, when deploying the model to Triton, the current Kubernetes setup only supports up to opset 15.This is the error I receive.Thanks
| silero_vad | 1 | UNAVAILABLE: Internal: onnx runtime error 1: Load model from /tmp/folderH4ekFf/1/model.onnx failed:/workspace/onnxruntime/onnxruntime/core/graph/model_load_utils.h:47 void onnxruntime::model_load_utils::ValidateOpsetForDomain(const std::unordered_map<std::__cxx11::basic_string<char>, int>&, const onnxruntime::logging::Logger&, bool, const string&, int) ONNX Runtime only *guarantees* support for models stamped with official released onnx opset versions. Opset 16 is |
| | | under development and support for this is limited. The operator schemas and or other functionality may change before next ONNX release and in this case ONNX Runtime will not guarantee backward compatibility. Current official support for domain ai.onnx is till opset 15. |

15 for Kubernetes and Triton.
13 for - ?
Are there any other opsets necessary?

I would like to downgrade to opset versions lower than 15 not necessarily 13.Thanks

Could you tell me how to get other opset version of the ONNX model? Thanks

https://github.com/snakers4/silero-vad/pull/573
We added a 16 kHz only ONNX model that was exported with opset 15.
Turns out the tool we were using to package all of the models was internally overriding the opset value and exporting with 16.
Opset 16 has support of if-statements, unlike earlier ones.
If this model would not work for you, please create another issue.


State for zwave climate without operation mode support is always unknown and cannot be overriden anymore

With the latest HA version I have the problem that the state of my Danfoss Zwave thermostats is always "unknown". Because of this, i.e. the persistence to Influx is not working because "unknown" states are not persisted.
I assume that the persistence isn't the bug itself, but rather that I cannot override the current operating mode (which I i.e. always want to either be "on" or "heat", as they cannot do anything else).
Using the customize config block I used to set
  operation_mode: heat
  operation_list:
    - heat

but since version 0.60 or 0.61 this doesn't work anymore. Setting operation_list still shows the value in the dropdown in the UI card, which is also preselected but in the climat panel as well as in the dev-state page of HA the state is still unknown. Overriding the current mode via operation_mode isn't working anymore.
As I said, this used to work and I didn't see any breaking changes. Was this renamed to some other property? I tried a couple of other combinations (operating|operation)_(mode|state): heat so far, but nothing's working.

These thermostats work fully automatically and I think that HA should display their state as 'auto' or 'heat'. The 'unknown' state may be confusing to the user.

I agree... ability to change presentation mode of operation mode (instead "unknown") should be restored definitely.
Additionally if thermostat doesn’t return temperature to hass, there is no need to show "current" field.
At the moment in the case of very popular Danfoss thermostats.. it looks really confusing:

Additionally if thermostat doesn’t return temperature to hass, there is no need to show "current" field.

Likely off topic for this issue but having a current_temperature template would be awesome. There are only nasty workarounds and a feature request.

Feature request from @bieniu looks really good. It will be superb if it was implemented.
Hovewer restoring operation_mode functionality doens't look very complicated (because it worked!).

Hovewer restoring operation_mode functionality doens't look very complicated (because it worked!).

Looking forward to your PR :) And I just wanted to mention the feature request as it's another topic and is not directly related to this issue.

@mlesn may I ask again how it worked for you?

I assumed that it worked on your initial post on this thread ;-)
I had hoped that z-wave developers will address this issue. Unfortunately nothing happened ;-(
Meanwhile someone wrote another issue concerning this problem.
Take a look at: https://github.com/home-assistant/home-assistant/issues/9478
Vote for change, maybe it helps.

Dear bot ;) workarounds in form of custom scripts exist, but the problem is still there in the latest version unfortunately.

Issue still exists.

Still exists in 0.81.1

theRadiator Thermostat should be moved to the water-heater component if possible

the Radiator Thermostat should be moved to the water-heater component if possible
indeed that kinda fits because of the limited functionality of the danfoss thermostat but feels kinda ... wrong?
are we loosing anything by moving to water heater from climate?

for version 0.89.1 I can confirm that the issue is still there. Danfoss popp and LC-13 radiator thermostats showing state of "unknown" instead of heating/idle (or something like that).


Implement hash for Digest

It's quite common to want to hash a given Digest, as a key in a dictionary for example.
This MR implements Hash for Digest

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 70.86%. Comparing base (57570dd) to head (b35d481).
Report is 34 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##             main     #221       +/-   ##
===========================================
+ Coverage   26.61%   70.86%   +44.24%
===========================================
  Files          24       26        +2
  Lines        2788     3147      +359
  Branches     1516     1663      +147
===========================================
+ Hits          742     2230     +1488
+ Misses        948      917       -31
+ Partials     1098        0     -1098

Sure! Closed


Add support for nodeSelector, tolerations & affinity in podTemplate

What this PR does / why we need it:
Adds support for nodeSelector and tolerations in podTemplate.spec.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #1770
Special notes for your reviewer:
I haven't added any validation for the actual values as I think that would be out-of-scope for cert-manager, I may be wrong though.
Release note:

Adds support for `nodeSelector` and `tolerations` in `podTemplate.spec`

/retest

/retest

/retest


Where to place ruby subscription request?

Quick question, I'm trying to get message rockets to work with my rails app and there is only one part I can't figure out- where I am supposed to put the ruby event handler for request subscriptions?
MessageRocket.on :subscription_request do |request|
  if request.channel == '/my-awesome-channel'
    request.permit
  else
    request.deny
  end
end

I tried putting it in my controller but the request turns out to be an ActionDispatch object that doesn't have a channel.  I also tried putting it in my background process, but the event never fires.  Thanks for any help.

I'd start by putting it in an initializer, you might want to factor it out into a service if it starts getting too complicated.


[build ERROR] publish_state : candidate expects 3 arguments, 4 provided

publish_state : candidate expects 3 arguments, 4 provided #777
home/tangshp/code/ros_ur_driver/src/Universal_Robots_ROS2_Driver/ur_controllers/src/scaled_joint_trajectory_controller.cpp: In member function ‘virtual controller_interface::return_type ur_controllers::ScaledJointTrajectoryController::update(const rclcpp::Time&, const rclcpp::Duration&)’:
/home/tangshp/code/ros_ur_driver/src/Universal_Robots_ROS2_Driver/ur_controllers/src/scaled_joint_trajectory_controller.cpp:274:16: error: no matching function for call to ‘ur_controllers::ScaledJointTrajectoryController::publish_state(const rclcpp::Time&, joint_trajectory_controller::JointTrajectoryController::JointTrajectoryPoint&, joint_trajectory_controller::JointTrajectoryController::JointTrajectoryPoint&, joint_trajectory_controller::JointTrajectoryController::JointTrajectoryPoint&)’
274 |   publish_state(time, state_desired, state_current, state_error);
|   ~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /home/tangshp/code/ros_ur_driver/src/Universal_Robots_ROS2_Driver/ur_controllers/include/ur_controllers/scaled_joint_trajectory_controller.hpp:41,
from /home/tangshp/code/ros_ur_driver/src/Universal_Robots_ROS2_Driver/ur_controllers/src/scaled_joint_trajectory_controller.cpp:41:
/home/tangshp/code/ros_ur_driver/install/joint_trajectory_controller/include/joint_trajectory_controller/joint_trajectory_controller/joint_trajectory_controller.hpp:257:8: note: candidate: ‘void joint_trajectory_controller::JointTrajectoryController::publish_state(const JointTrajectoryPoint&, const JointTrajectoryPoint&, const JointTrajectoryPoint&)’
257 |   void publish_state(
|        ^~~~~~~~~~~~~
/home/tangshp/code/ros_ur_driver/install/joint_trajectory_controller/include/joint_trajectory_controller/joint_trajectory_controller/joint_trajectory_controller.hpp:257:8: note:   candidate expects 3 arguments, 4 provided
gmake[2]: *** [CMakeFiles/ur_controllers.dir/build.make:76：CMakeFiles/ur_controllers.dir/src/scaled_joint_trajectory_controller.cpp.o] 错误 1
gmake[1]: *** [CMakeFiles/Makefile2:223：CMakeFiles/ur_controllers.dir/all] 错误 2
gmake: *** [Makefile:146：all] 错误 2
Failed   <<< ur_controllers [17.6s, exited with code 2]
Aborted  <<< admittance_controller [59.1s]
Summary: 35 packages finished [7min 24s]
1 package failed: ur_controllers
1 package aborted: admittance_controller
2 packages had stderr output: ur_client_library ur_controllers
4 packages not processed

Can you please provide more information on this error? In particular:

Which ros distribution are you using?
Were you following our instructions on building from source?
Did you build ros2_control and ros2_controllers from source?
Which branches are you on for all relevant packages?

This looks vaguely like the problem in #674, but without more information it is impossible to tell.

Can you please provide more information on this error? In particular:

Which ros distribution are you using?
Were you following our instructions on building from source?
Did you build ros2_control and ros2_controllers from source?
Which branches are you on for all relevant packages?

This looks vaguely like the problem in #674, but without more information it is impossible to tell.

Ros distribution is humble.
Follow the Build from source in the readme
Both ros2_control and ros2_controllers are in Universal_Robots_ROS2_Driver
main branch

Building for humble with ros2_control and ros2_controllers on main branch won't work, you need to go to their respective humble branches for that (just as you need to use the humble branch of this repo). Even more so, you should not need to build them from source at all (step 5 should not be necessary). Be aware that the README has the distribution-aware build steps on the corresponding branches, so the humble README lists the relevant steps for you.
Can you check if changing the repo branches fixes the build for you?

Thanks. The humble branch can be compiled successfully, but after using launch (ros2 launch ur_moveit_config ur_moveit.launch.py ur_type:=ur3), plan&execute always fails in rivz
file:///home/tangshp/%E5%9B%BE%E7%89%87/2023-08-23_08-42.png

What exactly is your setup for this:

Are you using a real robot or ursim? If so, did you setup the robot as described in our documentation? And is the program running on the robot when you try to move it?
Which part fails exactly? Does planning work and execution fail? Please provide the console error output for further investigation.
Are you able to move the robot without MoveIt? Can you e.g. test if rqt_joint_trajectory_controller works for manual motion?

Hi @Tangshp163,
since some time has passed since our reply I will close this issue. If it is still relevant, feel free to reopen it.


Patch to avoid conflicting "contrast" function name in OSL code gen

Fixes #1106

Add "contrast" to disallowed list as it exists in mx_funcs.h and colorcorrect has a "contrast" arg so will conflict.
@crydalch a small workaround to avoid a name conflict
@niklasharrysson if you think this is a good way to go then I assume all mx_funcs.h function names should be
added.

Hey @bernardkwok, yes this will solve the problem with the identifier name collision.
A consequence I want to mention though is that the conflicting shader parameter will now instead be named contrast1 when contrast is a reserved word.
An alternative would be to avoid polluting the namespace with these functions in mx_funcs.h, for example by prefixing them with "mx_", so the contrast function would be named mx_contrast instead. It seems mx_funcs.h is inconsistent in it's naming convention, where some functions are prefixed with mx_ and some are not. So maybe it's worth investigating if this could be changed.
That's a much large change though and for another PR. Your quick fix looks good for now!

Yes I noticed that the argument is called contrast1 and thus does not reflect the definition's input name anymore.
Going with mx_ does seem a better way though I'm not sure what that may break if anyone includes mx_funcs.h.
I would be curious if these could be removed if they become OSL intrinsics as another route, but it won't really rid the issue of
any include files in custom OSL distributions possibility having conflicts.
I've added a new issue for the generic problem: https://github.com/AcademySoftwareFoundation/MaterialX/issues/1110 for feedback.
Thanks.

Thanks for writing up this patch, @kwokcb, and thanks for merging it, @niklasharrysson!  My sense is that we shouldn't leave this workaround in 1.38.6, and should instead implement the rename of contrast to mx_contrast that Niklas suggests above, as this approach is more robust.  But having a patch to enable OSL builds to proceed in the meantime is definitely worthwhile.

Yes, @jstone-lucasfilm . This can be checked in against #1110 where all non-mx prefixed functions can be patched up and added to the "disallowed" list.

@kwokcb That approach is great for this initial patch, but for 1.38.6 I'd recommend we start moving in the other direction, removing these changes to the disallowed list and adding mx prefixes to any OSL functions that are likely to conflict (and eventually to all of them).


Beta markings, and misc javadoc scrub

Can one of the admins verify this patch?

for now, we're keeping all async operations that aren't on the path of implementing the GA-level sync operations as Beta. (So for example, deleteAsync is not beta)


Can you provide any feedback on operation?

Just wondering if this is working and if you can show what amount of data and control you have from this setup.
I'm interested in possibly using this for a INTELLIFLO VSF pump we just installed after the previous one failed.  Control of the pump would be nice but not  absolutely needed. I'd mainly want to know pump speed, power, and whether schedule is enabled/running. Ideally, I can figure out how to integrate your pentair.cpp intoESPHome.

It all works great and has been stable on my esp for more than a year. But
all functionality is read only at this point.
Pump speed and power are in there.
not sure what you mean about schedule enabled/running though.
On Mon, May 23, 2022 at 3:44 PM DK @.***> wrote:

Just wondering if this is working and if you can show what amount of data
and control you have from this setup.
I'm interested in possibly using this for a INTELLIFLO VSF pump we just
installed after the previous one failed. Control of the pump would be nice
but not absolutely needed. I'd mainly want to know pump speed, power, and
whether schedule is enabled/running. Ideally, I can figure out how to
integrate your pentair.cpp intoESPHome
https://esphome.io/custom/custom_component.html.
—
Reply to this email directly, view it on GitHub
https://github.com/jml3on/arduino-pentair/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABMS4FZB5JC5OLZCXHY77BTVLODUXANCNFSM5WV5AKSA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Allow iovecs with buffers having length over 32bit.

Did some quick tests in Eio and it fixes that case: See https://github.com/ocaml-multicore/eio/issues/335
There are probably more spots (outside iovec) where a (s)size_t is wrongly expressed as an Unsigned.UInt, ideally it should always be a Long.

Thank you.
size_t should be Unsigned.Size_t, because size_t can be defined differently by different implementations. I will update this patch in a follow-on commit.

76c713f952f70b947b7f8da8628f0d39511ec052


is there anyway to append the datepiker outside of the div

I am trying to initialize the datepiker outside the div but its position is not working right

The parent div is overflow hidden the datepiker is not working properly because its dynamically building in that div i want to show the textbox here  and the datepiker will show somewhere in footer but its alignment should not disturb.


Have you considered using sass + css modules as the main technical point

with-external-scoped-css is not perfect.
In a slightly larger system, doing so will add a lot of work.
I think style jsx is not a good idea, it can not and other open source UI framework very good integration.
Have you considered using sass + css modules as the main technical point

Imo, css-in-js is better for component styling. And even more, with tools like PreJSS you can write both - (any steroid) css and css-in-js.
Anyway, there is a problem with plain css for components styling - duplicates in next/head. With this fix released you can write your own wrapper to reduce boilerplate code for styles content injection. Take a look at my example here

@code4aman thanks for your tip! I have a similar problem. I'm using a ui kit that is built on top of css modules. I would like to use the ui kit in my next.js application. It seems like next.js does not support css modules though it isn't global. So practically, i'm thinking of exporting the ui kit into a .js and .css file that I'll use in the application, I'll need to live with the fact that the ui kit has to be imported globally for now. Your solution is great but does it mean that the entire css would be downloaded on every page load? I'm thinking of copying the css file into the static folder (during build process somehow) and then import it as a head <link ref='stylesheet'> tag pointing to the static file. What do you think?
Cheers! :)

@khankuan that's a default behavior and yes, this way your styles will be referenced on every page load. The real problem is, since we are component-oriented, do we reference only styles the current page uses? To import them on-demand, in per-component manner, we need that PR. And we need to somehow (my example wrapper is just a point to start with) reduce current boilerplate code for css injection.
If your 3rd party framework provides one large stylesheets file then your only way is to reference it in _document.js or maybe your own layout file.

@code4aman would your method allow browser chacing?

@khankuan yes, why not. CSS Modules will change hash only if content is changed, so it's safe to rebuild.

 I have a uikit that is based on css modules + sass + react. I tried overriding the webpack config but didn't manage to get it working. I then tried building my uikit into a umd lib with a css file. But I realized that it didn't work well with styled-jsx because it does not pass data-jsx prop to components, so I could not do layout styling in my application. Any help/advice would be appreciated :)

CSS is pretty confusing when it's come to choices. So, we won't change the default styled-jsx usage. Since it won't add any overhead if you haven't use it.
We'll keep trying to improve.


Fix golint errors

I wanted to enable golangci-lint for the test/ directory (which is currently not checked) and noticed that new errors were introduced to the rest of the code.  I have no idea how that was possible but now the linter is happy again.
@giuseppe @mheon @baude @TomSweeneyRedHat @rhatdan PTAL

Note that this PR is not enabling the linter for ./test. Something we can tackle later.

LGTM, but gating is angry

LGTM, but gating is angry

Should be happy now. Had some whitespaces in the commits :^)

Did we turn off branch protection? I have merge permission even though the PR isn't green yet

Did we turn off branch protection? I have merge permission even though the PR isn't green yet

I see it too, but  didn't notice before :thinking:

LGTM assuming happy tests

I assume these are flakes (for the record):
[+0925s] [Fail] Podman run with --sig-proxy [It] signals are not forwarded to container with sig-proxy false
[+0925s] /var/tmp/go/src/github.com/containers/libpod/test/e2e/run_signal_test.go:135
[+0925s]
[+0925s] [Fail] Podman run with --sig-proxy [It] signals are not forwarded to container with sig-proxy false
[+0925s] /var/tmp/go/src/github.com/containers/libpod/test/e2e/run_signal_test.go:135
[+0925s]
[+0925s] [Fail] Podman run with --sig-proxy [It] signals are not forwarded to container with sig-proxy false
[+0925s] /var/tmp/go/src/github.com/containers/libpod/test/e2e/run_signal_test.go:135
[+0925s]
[+0925s] [Fail] Podman network [It] podman network rm
[+0925s] /var/tmp/go/src/github.com/containers/libpod/test/e2e/network_test.go:134

I seem to be unable to restart the flaked test. Maybe someone else can try?

Seems like an UI issue here. Jobs are running but the UI isn't updating. @cevich is investigating.

/lgtm


[InnoSport] Add info about start and end of semesters

Describe the feature
Currently https://sport.innopolis.university/api/semester produces:
[
  {
    "id": 6,
    "name": "F20"
  }
]

and we can't calculate how many days to the end of semester.
Suggested solution
Just add start and end fields to the output of the endpoint.
Additional context
No response

Also propagate in our API so website will show "you have N days to farm sport hours"


模型训练中的数据增强出错

问题类型：模型训练
问题描述
====================
我用的是python生成的图片，但是有的能读取，有的读取不了。不知道为什么，检查过了，所有图片都能用Image读取显示。
2021-05-27 22:11:23,853-INFO: If regularizer of a Parameter has been set by 'fluid.ParamAttr' or 'fluid.WeightNormParamAttr' already. The Regularization[L2Decay, regularization_coeff=0.000100] in Optimizer will not take effect, and it will only be applied to other Parameters!
2021-05-27 22:11:24 [INFO]	Decompressing output/faster_rcnn_r50_fpn/pretrain/ResNet50_cos_pretrained.tar...
2021-05-27 22:11:27 [INFO]	Load pretrain weights from output/faster_rcnn_r50_fpn/pretrain/ResNet50_cos_pretrained.
2021-05-27 22:11:27,131-WARNING: output/faster_rcnn_r50_fpn/pretrain/ResNet50_cos_pretrained.pdparams not found, try to load model file saved with [ save_params, save_persistables, save_vars ]
2021-05-27 22:11:27 [INFO]	There are 265 varaibles in output/faster_rcnn_r50_fpn/pretrain/ResNet50_cos_pretrained are loaded.
2021-05-27 22:11:29 [INFO]	[TRAIN] Epoch=1/10, Step=2/120, loss=1.619201, loss_cls=0.876635, loss_bbox=2e-06, loss_rpn_cls=0.698067, loss_rpn_bbox=0.044497, lr=0.000837, time_each_step=0.66s, eta=0:16:34
2021-05-27 22:11:29 [INFO]	[TRAIN] Epoch=1/10, Step=4/120, loss=1.018859, loss_cls=0.264852, loss_bbox=4e-06, loss_rpn_cls=0.695395, loss_rpn_bbox=0.058608, lr=0.000843, time_each_step=0.49s, eta=0:12:6
2021-05-27 22:11:30 [INFO]	[TRAIN] Epoch=1/10, Step=6/120, loss=0.919157, loss_cls=0.036404, loss_bbox=0.002886, loss_rpn_cls=0.689783, loss_rpn_bbox=0.190083, lr=0.00085, time_each_step=0.41s, eta=0:10:6
2021-05-27 22:11:30 [INFO]	[TRAIN] Epoch=1/10, Step=8/120, loss=1.055311, loss_cls=0.061291, loss_bbox=0.020401, loss_rpn_cls=0.690866, loss_rpn_bbox=0.282754, lr=0.000857, time_each_step=0.37s, eta=0:9:5
2021-05-27 22:11:31 [INFO]	[TRAIN] Epoch=1/10, Step=10/120, loss=0.747305, loss_cls=0.031498, loss_bbox=0.002285, loss_rpn_cls=0.677515, loss_rpn_bbox=0.036007, lr=0.000863, time_each_step=0.36s, eta=0:8:49
2021-05-27 22:11:31 [INFO]	[TRAIN] Epoch=1/10, Step=12/120, loss=3.773873, loss_cls=0.051113, loss_bbox=0.009292, loss_rpn_cls=0.662133, loss_rpn_bbox=3.051336, lr=0.00087, time_each_step=0.35s, eta=0:8:33
2021-05-27 22:11:32 [INFO]	[TRAIN] Epoch=1/10, Step=14/120, loss=1.389426, loss_cls=0.214923, loss_bbox=0.041175, loss_rpn_cls=0.653619, loss_rpn_bbox=0.479709, lr=0.000877, time_each_step=0.33s, eta=0:8:3
2021-05-27 22:11:33 [INFO]	[TRAIN] Epoch=1/10, Step=16/120, loss=1.208307, loss_cls=0.122072, loss_bbox=0.03224, loss_rpn_cls=0.622, loss_rpn_bbox=0.431996, lr=0.000883, time_each_step=0.32s, eta=0:7:58
Exception in thread Thread-22:
Traceback (most recent call last):
File "/opt/conda/envs/python35-paddle120-env/lib/python3.7/threading.py", line 926, in _bootstrap_inner
self.run()
File "/opt/conda/envs/python35-paddle120-env/lib/python3.7/threading.py", line 870, in run
self._target(*self._args, **self._kwargs)
File "/opt/conda/envs/python35-paddle120-env/lib/python3.7/site-packages/paddlex/cv/datasets/dataset.py", line 85, in handle_worker
r = mapper(sample[0], sample[1], sample[2])
File "/opt/conda/envs/python35-paddle120-env/lib/python3.7/site-packages/paddlex/cv/transforms/det_transforms.py", line 149, in call
outputs = decode_image(im, im_info, label_info, input_channel)
File "/opt/conda/envs/python35-paddle120-env/lib/python3.7/site-packages/paddlex/cv/transforms/det_transforms.py", line 124, in decode_image
self.data_type = im.dtype
AttributeError: 'NoneType' object has no attribute 'dtype'

其中某张图片读取后为None，建议检查下所有图片是否能使用cv2.imread读取，并且读取后不为None

谢谢！我检查了一下，图片的起始为1，而json里的为0，所以读取失败，谢谢！！


Users: require student.lu.se email

Adding momentjs for Bootstrap Datetimepicker.
Adding validation for emails and a test for it.

Looks good +1


mongodb_meta (elasticsearch) index never gets created and update failed

I'm trying to index a single MongoDB collection called catalog.summary to Elasticsearch, this collection has just one document in it. I'm using mongo-connector 2.5.3 (upgraded to the latest).
Here's what I did.

Stopped mongo-connector
Deleted oplog.timestamp file
Deleted “catalog” elasticsearch index
Created “catalog” elasticsearch index with mappings

curl -u xyz: -XPUT https://xxxx.com -d '{"settings" : {"number_of_shards" : 1 }, "mappings" : {"summary" : { "properties" : {"RetailerName" : { "type" : "string", "index" : "not_analyzed" }, "BrandName" : { "type" : "string", "index" : "not_analyzed" }, "Categories" : { "type" : "string", "index" : "not_analyzed" } } } } }'

Started mongo-connector

sudo mongo-connector -m xyz.mongolab.com:64697 -t<EMAIL_ADDRESS>-d elastic_doc_manager -n catalog.summary --admin-username oplog-reader --password abc123 --auto-commit-interval=60

Updated the MongoDB document (catalog.summary) but mongo-connector fails to update the elasticsearch index.

I'm using hosted MongoDB cluster from MongoLab. I have changed the server names and other private info.
This is my log from mongo-connector.
The error above seems to be complaining that the document I'm updating doesn’t exist in elasticsearch index. I'm assuming at start up mongo-connector see if the destination is empty it should do a collection dump but i don't see that's happening from the log. Also, I wonder why it's not creating mongodb_meta index.
[ec2-user@ip-172-16-0-161 ~]$ cat mongo-connector.log
2015-02-25 17:54:28,995 [INFO] mongo_connector.connector:988 - Beginning Mongo Connector
2015-02-25 17:54:28,995 [INFO] mongo_connector.connector:116 - MongoConnector: Can't find oplog.timestamp, attempting to create an empty progress log
2015-02-25 17:54:29,330 [INFO] mongo_connector.connector:222 - MongoConnector: Empty oplog progress file.
2015-02-25 17:54:29,763 [INFO] mongo_connector.oplog_manager:87 - OplogThread: Initializing oplog thread
2015-02-25 17:54:29,835 [INFO] mongo_connector.connector:294 - MongoConnector: Starting connection thread MongoClient([u'xyz.mongolab.com:64696', u'xyz.mongolab.com:64697'])
2015-02-25 17:55:29,000 [INFO] elasticsearch:62 - POST http://xxxx:443/_refresh [status:200 request:0.005s]
2015-02-25 17:55:29,000 [INFO] elasticsearch.trace:72 - curl -XPOST 'http://localhost:9200/_refresh?pretty' -d ''
2015-02-25 17:55:34,610 [INFO] elasticsearch:62 - POST http://xxxx:443/_refresh [status:200 request:0.005s]
2015-02-25 17:55:34,610 [INFO] elasticsearch.trace:72 - curl -XPOST 'http://localhost:9200/_refresh?pretty' -d ''
2015-02-25 17:55:34,616 [WARNING] elasticsearch:81 - GET /catalog/summary/54e7c8a9bfede101744e4adb [status:404 request:0.005s]
2015-02-25 17:55:34,616 [ERROR] mongo_connector.oplog_manager:278 - Unable to process oplog document {u'h':<PHONE_NUMBER>426869494L, u'ts': Timestamp(1424886903, 1), u'o': {u'ItemId': 39, u'SeoKeywords': u'LectroFan - Fan Sound and White Noise Machine, White, Black Colors', u'RetailerName': u'Amazon', u'ImageUrl': u'http://ecx.images-amazon.com/images/I/81DBEz5lnjL.SL1500.jpg', u'BrandName': u'LectroSound', u'AffiliateMarketingVerbs': [u'Fan Sound', u'White Noise'], u'SeoDescription': u'LectroFan - Fan Sound and White Noise Machine, White', u'ListPrice': 64.950000000000003, u'Description': u"LectroFan's compact size makes it perfect for baby nurseries, day care centers, college dorms, apartments - or any sleep environment where unwanted, disturbing, or intrusive noises are a problem. The LectroFan has two ways to reduce unwanted noises, either fan sounds or white noise sounds. Each has 10 non repeating unique sound variations. Use it in your office or bedroom, or take it with you on your travels. The LectroFan uses a USB cord for power so it can be plugged into most computers if you're on the go. Unlike other sound machines, LectroFan's sounds are dynamically generated, they never repeat. Which means your or your baby's sleep wont be disturbed by the looping of short recordings which makes it one of the best sound therapy products on the market. In addition it has a multi-level volume control so you can adjust to any environment.Don't have a baby? That's OK, use it to: -Block out noisy neighbors-Drown out your partner snoring-Reduce tinnitus ringing or hissing-Sound proof a room for studying or reading-Sleep easier during the day-Keep conversations confidential at work or home", u'SellingPrice': 54.950000000000003, u'Properties': [{u'Name': u'ASIN', u'Value': u'B00E6D6LQY'}, {u'Name': u'UPC', u'Value': u'897392002060'}, {u'Name': u'Shipping Weight', u'Value': u'1 pounds'}, {u'Name': u'Product Dimensions', u'Value': u'4.4 x 4.4 x 2.2 inches ; 1 pounds'}], u'Categories': [u'Electronics', u'Fans'], u'Name': u'LectroFan Gold - Fan Sound and White Noise Black Machine, White', u'Summary': u'Small transportable size and great for travelPowered using USB. Can be plugged into computer or AC Adapter (included). AC adapter runs on 100 to 240 VAC10 non-repeating unique fan sounds & 10 non-repeating unique white noise soundsMultilevel adjustable volume and 60 minute sleep timerMulti-band parametric EQ speaker compensation', u'CostPrice': 3.1000000000000001, u'ImageAltTag': u'LectroFan - Fan Sound and White Noise Machine, White Color', u'Locale': u'en-Us', u'AffiliateDealTypes': [u'Placement', u'PayPerClick'], u'Currency': None, u'AltTag': u'LectroFan - Fan Sound and White Noise Machine, White', u'SeoTitle': u'LectroFan - Fan Sound and White Noise Machine, White', u'_id': ObjectId('54e7c8a9bfede101744e4adb')}, u'v': 2, u'ns': u'catalog.summary', u'o2': {u'_id': ObjectId('54e7c8a9bfede101744e4adb')}, u'op': u'u'}
Traceback (most recent call last):
File "/usr/lib/python2.6/site-packages/mongo_connector/oplog_manager.py", line 264, in run
ns, timestamp)
File "/usr/lib/python2.6/site-packages/mongo_connector/util.py", line 38, in wrapped
reraise(new_type, exc_value, exc_tb)
File "/usr/lib/python2.6/site-packages/mongo_connector/util.py", line 32, in wrapped
return f(*args, **kwargs)
File "/usr/lib/python2.6/site-packages/mongo_connector/doc_managers/elastic_doc_manager.py", line 121, in update
id=u(document_id))
File "/usr/lib/python2.6/site-packages/elasticsearch/client/utils.py", line 68, in _wrapped
return func(*args, params=params, **kwargs)
File "/usr/lib/python2.6/site-packages/elasticsearch/client/init.py", line 321, in get
params=params)
File "/usr/lib/python2.6/site-packages/elasticsearch/transport.py", line 307, in perform_request
status, headers, data = connection.perform_request(method, url, params, body, ignore=ignore, timeout=timeout)
File "/usr/lib/python2.6/site-packages/elasticsearch/connection/http_urllib3.py", line 86, in perform_request
self._raise_error(response.status, raw_data)
File "/usr/lib/python2.6/site-packages/elasticsearch/connection/base.py", line 102, in _raise_error
raise HTTP_EXCEPTIONS.get(status_code, TransportError)(status_code, error_message, additional_info)
OperationFailed: TransportError(404, u'{"_index":"catalog","_type":"summary","_id":"54e7c8a9bfede101744e4adb","found":false}')
[ec2-user@ip-172-16-0-161 ~]$

It was a user permission issue. The database user didn't have oplog privileges.


korean translation

new kor.json

I'll quote @ItsANameToo from #110 (except for the Korean translation):

Don't forget to add Bulgarian to the list of available languages in src/providers/settings-data/settings-data.ts 😄

oh plz add the config as well :)

in case it wasn't clear, you need to add Korean here:
https://github.com/ArkEcosystem/ark-mobile/blob/master/src/providers/settings-data/settings-data.ts#L23-L30
I will test once you've done that

i think its done now?

Looks good and all working! 👍

@infinitytn provide me with your ARK address or contact me on Ark slack (boldninja) if you don't want to share it publicly.

@boldninja AZHsCdMyw9p5k2LPbB5NWSigf5AvktX54U here we go


fix: automerge permissions with update version / disclaimer GH workflows

Description
This PR splits the approve and merge steps for the app version bump + update licences and disclaimer GH workflows. Previously with Mergify, the jobs used the default GH action bot to approve and tag with automerge. Since removing Mergify, we need to update the branch and automerge manually and ran into permissions issues with valora-bot trying to do too many things: https://github.com/valora-inc/wallet/actions/runs/3061163606
Now: valora-bot opens the PR, default GH bot approves, valora-bot merges.
Other changes
Use conventional commit format for the opened PRs
Tested
Example https://github.com/valora-inc/wallet/pull/2882
How others should test
N/A
Related issues
Relates RET-394
Backwards compatibility
Yes

Hey @MuckT since NA is mentioned in how other should test section ,we are skipping this PR
CC: @kathaypacific
Thanks.!


Unify kernel and kernel composition usage outside kernel manager

Components such as tuning runner and configuration manager now need to work with both kernels and compositions which leads to significant code duplication.
Standalone kernels can be wrapped into a composition so that only kernel manager has to directly work with both types of these structures. Other components will utilize only compositions.

Kernel compositions were refactored on v2 branch.


ARROW-2979: [GLib] Add operator functions in GArrowDecimal128

@kou Thank you for review.I fixed.
Please review again.

Codecov Report

Merging #2370 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2370      +/-   ##
==========================================
- Coverage   84.51%   84.49%   -0.03%
==========================================
  Files         293      293
  Lines       45329    45329
==========================================
- Hits        38312    38301      -11
- Misses       6990     6997       +7
- Partials       27       31       +4

Impacted Files
Coverage Δ

go/arrow/memory/memory_avx2_amd64.go
0% <0%> (-100%)
:arrow_down:

go/arrow/math/int64_avx2_amd64.go
0% <0%> (-100%)
:arrow_down:

go/arrow/math/uint64_avx2_amd64.go
0% <0%> (-100%)
:arrow_down:

go/arrow/math/float64_avx2_amd64.go
0% <0%> (-100%)
:arrow_down:

go/arrow/memory/memory_amd64.go
28.57% <0%> (-14.29%)
:arrow_down:

go/arrow/math/math_amd64.go
31.57% <0%> (-5.27%)
:arrow_down:

cpp/src/arrow/util/thread-pool-test.cc
98.91% <0%> (-0.55%)
:arrow_down:

go/arrow/math/uint64_amd64.go
33.33% <0%> (ø)
:arrow_up:

go/arrow/math/int64_amd64.go
33.33% <0%> (ø)
:arrow_up:

go/arrow/math/float64_amd64.go
33.33% <0%> (ø)
:arrow_up:

... and 4 more

Continue to review full report at Codecov.

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

rebased

Thanks!!
I'll work to address these points over this coming weekend.

Thanks!!!
I’ll read each commit.


Have a look at the list of commits for details.

Hi @mboes! Shall I look into this PR or was this an accident?

Will resubmit shortly. Sorry for the noise!
On 2 July 2017 at 19:42, Andrey Mokhov<EMAIL_ADDRESS>wrote:

Hi @mboes https://github.com/mboes! Shall I look into this PR or was
this an accident?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/snowleopard/hadrian/pull/335#issuecomment-312506046,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADInIyNCjtd30Cw3AkbmBXg2F8wgx0Eks5sJ9ahgaJpZM4OLoAv
.

@mboes No problem at all!

Resubmitted as #336


Implement PauliStringPhasorGate

Implements PauliStringPhasor in terms of a GateOperation on a new class PauliStringPhasorGate.
Mostly involved moving existing functions from the operation to the gate, and then having the operation call those methods under the hood.
Closes #1561, xref #4683
@tanujkhattar

Q1: Can we make PauliString a GateOperation ?

I didn't notice that it wasn't (it has a gate field). It would be nice if we could. I'll look into that separately. I don't think it would affect what we do in this PR, but would be good to verify.

Q2: Can we make PauliStringGateOperation a GateOperation or deprecate it after this change?

I think we should deprecate it. It doesn't appear to be a useful abstraction, it just provides some convenience functions that should probably be module-level, if anything. That said, we may want an interface ContainsPauliString that could be used for anything that has a pauli_string property.

@tanujkhattar Here's a slightly different approach where we don't break the inheritance from PauliStringGateOperation. https://github.com/quantumlib/Cirq/compare/master...daxfohl:gates-psp2?expand=1. This would be safer, I think, and follows the precedent of PauliString, in that it doesn't inherit GateOperation, but still provides a gate. (As I mentioned above, to me having a gate is the important thing; it doesn't matter whether it inherits GateOperation or not).
It's a little more redundant, since it has to do all the protocol forwarding. But safer since it doesn't break the hierarchy. I do see a couple places in the code where isinstance(PauliStringGateOperation) is used. So it's probably better to be safe here. If we want to deprecate PauliStringGateOperation later and change the hierarchy, we can do that separately. But I don't see a particular need to rush that.
(Ideally we could make PauliStringGateOperation inherit GateOperation and have the best of both worlds. But I can't think of a way to do that since it is not guaranteed to have a gate. Unless we give it a dummy NoOpGate if nothing else is provided.)
What do you think? Should we go with this alternate approach?

I went ahead and merged in the approach mentioned above, to ensure backwards compatibility with PauliStringGateOperation is preserved. It's a safer, smaller, simpler change. If we want to deprecate PauliStringGateOperation later we can revisit that as a separate effort.

Reverted this to bring back the hard break from PauliStringGateOperation as we'd discussed in cirq-sync noted above. @tanujkhattar


Google Map does not load for 3.0.0 and above

When I used version 3.0.0 and above it does not load Google Maps.
After investigation, I found that it is the issue with the latest release when you directly use
URLSessionProxyDelegate.enableAutomaticRegistration()
The quick solution is to use version less than 3.0.0

I have the same issue

Hi, nothing significant changed in URLSessionProxyDelegate in Pulse 3, so I'm not sure what might be the root cause. What OS version have you tested on? Is there anything in the logs/stacktraces that indicates what the issue might be?
The only change in URLSessionProxyDelegate was removing conditional compilation from:

Can you please try removing this code entirely from the file and see if that works?

@kean  - Thanks for the feedback.
Yes, I was surprised too when I found out this was causing the issue. I have tested in iOS 16.1, 16.2, and 16.3.
I removed URLSessionProxyDelegate integration and manually integrated using EventMonitor protocol and it's working perfectly.
Also, I did not find anything in the logs/stacktraces for this issue.

Hello @kean. I confirm, GTMSessionFetcher https://cocoapods.org/pods/GoogleAPIClientForREST breaks when using URLSessionProxyDelegate.enableAutomaticRegistration(). GTMSessionFetcher is used in all google services such as maps, disk, etc.

Potentially related: https://github.com/kean/Pulse/issues/141

GTMSessionFetcher has a lot of code that manages session creation and delegate. Does someone have a sample project where you could integrate Pulse and debug it?
P.S. I wish there was a simple way to observe URLSession without intercepting its delegate.

As requested, I provide a test case based on reading Google Drive.
https://github.com/ayuzhin/google-drive-demo.git
There are two buttons on the main screen:

Read items from Google Drive
Display Pulse console

If you don't use the URLSession global design by using URLSessionProxyDelegate via URLSessionProxyDelegate.enableAutomaticRegistration() call, then in the Xcode console you can see the phrase "Fetch of items from Google Drive is done."
If you uncomment the line URLSessionProxyDelegate.enableAutomaticRegistration() in the viewDidAppear method of the ViewController, then the response from the google service will never be received, and in some scenario it may lead to a deadlock in GTMSessionFetcher and the application hangs.
I hope this helps debug and fix a problem in your really interesting and useful library.
P. S. Yes, manual logging of network requests works, but it is not always convenient and informative, because not all modules can connect EventMonitor.

I added a workaround in version 3.4.3 where URLSessionProxyDelegate.enableAutomaticRegistration skips GTMSessionFetcher. Obviously, it's less than perfect. I'm not sure what else I could do.


Updated FAIL macros with two arguments to work with more complex items

Updated FAIL macros that take two arguments to have parentheses around the arguments before doing the comparison to ensure the macros work when more complex items are passed in (like ternary expressions).
I also added some unit tests to verify the changes I made.  Hopefully I added the unit tests in the right place.

released as v3.25. thanks!


Safety improvements to State

from_ptr is marked unsafe since it takes an arbitrary raw pointer and assumes it to be valid for future use.
error and arg_error are marked divergent since they use C longjmp internally.
get_alloc_fn is changed to not require the user to provision their own *mut *mut c_void.

It's worth noting that arg_error leaks the CString and in general it's infeasible for the library to prevent client code from leaking when it calls a divergent function.

Thanks.


Fix 0746294

Issue:
The Go Live playbook was ignoring the "Notify All Responders" parameter
Fix:
Add a condition to only send the email if "Notify All Responders" is checked

https://mantis.fortinet.com/bug_view_page.php?bug_id=0746294


Add autofix for "unused variable"

To start, we can just mimic autoflake, whose logic is relatively simple: https://github.com/PyCQA/autoflake/blob/60eeb31082bf5290341e84a008d5c832fb04fd13/autoflake.py#L664.

(Not "trivial" but could be a good first issue if anyone's interested given that the autoflake logic can mostly be ported over.)

I would take it!


Add Show instances for PatternRepeat, TextAlign, Composite

Did you have something else in mind for the show implementation?

Ok, I updated the Show instances for TextAlign and PatternRepeat, and Composite as well, and replaced the show calls with toString as a private helper in the where clause. By the way, I noticed that a bunch of the newer composite operations are not in Composite (e.g. multiply, screen, etc.). I could add those in if you want.

This looks great, thanks!

I could add those in if you want.

That would be good, if you don't mind.

Ok great, I'll add those as well :)

Just added the new blend modes (https://www.w3.org/TR/compositing-1/).

Fantastic, thank you!


how to short column like version?

Hi team,
How to short column like version one. look like there is no documentation in the tutorial file.

Tutorial for v2.0 is not finished yet, sorry. But if you can explain in more detail what do you want to achieve, maybe I can provide the answer.


autorun module

Is there a way to know which module is currently set as the autorun module (apart from checking them one by one)?

Hi, i think you are using the old autorun style. Did you try this - https://www.blackhillsinfosec.com/empire-resource-files-auto-runs-docx/? Then command autorun show will solve your issue.

@poopaapoopaa please let us know if the suggestion from @winnie22 does not work for you.


Put back assert and use names wihtout tildes for artificial tables

See https://github.com/samwebster/times-excel-reader/pull/28#discussion_r1118093699

@siddharth-krishna should we put the assert back already?

Or will this be irrelevant once we transition away from times_mapping.txt?

Yes, perhaps once we move to the TimesModelData class in #11 this is no longer an issue?


Time#toBarsBeatsSixteenths() Float Issue

In certain conditions, I believe the Time#toBarsBeatsSixteenths() function can output undesired results due to floating point math.
The specific conditions I can recreate with:

BPM of 100
a Time object occurring at 0:1:3
the output of that Time object's toBarsBeatsSixteenths() will yield 0:1:3.000

Reproduced in JS Bin

To be clear, your desired output is 0:1:3?
Does seem like a floating point math error, where the output for the sixteenths notes might have been something like 3.00001 which was truncated to 3.000.
Can you submit a PR if this specific formatting is important to your application?

@tambien that's right. And yes, it's a pretty petty issue -- regardless, I'll give the fix a shot. Thanks for checking it out.


Failed to parse manifest when packaging.

Greetings!
Problem

With version 1.37.0, I'm unable to cargo package a library of mine. Cargo failed with this error message.
~/c/eventstore-rs ❯❯❯ cargo package                                                                                master
   Packaging eventstore v0.4.2 (/home/yoeight/code/eventstore-rs)
   Verifying eventstore v0.4.2 (/home/yoeight/code/eventstore-rs)
error: failed to verify package tarball

Caused by:
  failed to parse manifest at `/home/yoeight/code/eventstore-rs/target/package/eventstore-0.4.2/Cargo.toml`

Caused by:
  can't find `integration` test, specify test.path

Note that I don't experience that issue if I use 1.36.0 version.
Steps

Clone https://github.com/YoEight/eventstore-rs. You might have to install openssl lib, I know it's libssl-dev on Ubuntu.
cargo build
cargo package (It will fail every time)

Possible Solution(s)

If you pass --no-verify, cargo package succeeds.
Notes
~/c/eventstore-rs ❯❯❯ cargo --version                                       master
cargo 1.37.0 (9edd08916 2019-08-02)

~/c/eventstore-rs ❯❯❯ rustc --version                                       master
rustc 1.37.0 (eae3437df 2019-08-13)

I experienced that issue on both Ubuntu 18.04LTS and 19.04. I didn't try on a different OS yet.
Thank you for your time.

Thanks for the report! I believe this is because tests are excluded from the package

@alexcrichton Thanks. By no longer excluding tests/**/*, it works. I will update my Cargo.toml accordingly. Do you know why that behaviour changed after 1.36 version?


Update CI host images to ubuntu 22.04

Signed-off-by: Patrik Stas<EMAIL_ADDRESS>

Codecov Report

Merging #751 (ade1a17) into main (66ee076) will decrease coverage by 24.77%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main     #751       +/-   ##
===========================================
- Coverage   54.73%   29.96%   -24.77%
===========================================
  Files         363      379       +16
  Lines       37122    30910     -6212
  Branches     8089     6299     -1790
===========================================
- Hits        20317     9262    -11055
- Misses      10832    18939     +8107
+ Partials     5973     2709     -3264

Flag
Coverage Δ

unittests-aries-vcx
29.96% <ø> (-24.67%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

 Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Catch local save lf config errors. Fixes #527

Description
Handles the error where local configuration cannot be saved, outputs a warning, debug logs the error message.
Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Fixes #527


Enderian not taking water damage.

This has happened on three new single player worlds. Enderian takes water damage at first, then after playing for a bit just stops being effected by the water.

Are you on the latest version of Origins (1.1.0, released yesterday)?


Change "Your shopping cart" to "Shopping cart" in backend sample data

because it's backend, we can't fix it in GH-2785

@Xymmer  Shouldn't this be done in the b2c-addon for the backend change 'Shopping Cart'? Because it shouldn't only be a fix for spa. Can you create a ticket for c3po please

Other than that backend 'issue', shouldn't we follow the wireframes? Another point to this is the cart # because I don't think the user needs to know the cart # before purchasing the item. I'm guessing this unique id for cart is going to be used for the order confirmation too.
---> Might be a bug or a feature, but the cart # at the beginning and after purchasing it is different by 1. For example, at the cart page it would be 0001, and in the confirmation after checking out, it would be 0002.


Document products endpoints

Adds the necessary information for the products endpoints,
including /products and /products/{product_id}, as well as
the product definition to the openapi spec.

@dplumb94 I added a component schema entry for the properties field in the product endpoint response


feat(viz): migrate to elk for layouting

Changes

migrates layouting library from dagre to elk
converts layouting function to promise, update store to use elk layout values
adds a uuid to branch steps

Would appreciate help with testing this branch, please. :)
NOTE: there should be a bit more spacing in vertical mode for the parent branch label (see screenshots), but this isn't a showstopper.
NOTE 2: Camel Routes are not rendering correctly, but I have tested and this is also happening on the main branch as well, so I will open a separate issue for this.
Screenshots
Repeated Set Body

apiVersion: camel.apache.org/v1alpha1
kind: Kamelet
metadata:
  annotations:
    camel.apache.org/kamelet.icon: cherry
  labels:
    camel.apache.org/kamelet.type: action
  name: eip-action
spec:
  definition:
    title: EIP Kamelet
  dependencies:
    - camel:core
    - camel:kamelet
  template:
    from:
      uri: kamelet:source
      steps:
        - choice:
            when:
              - simple: "{{?foo}}"
                steps:
                  - set-header:
                      name: bar
                      simple: foo
              - simple: "{{?foo2}}"
                steps:
                  - set-header:
                      name: bar
                      simple: foo2
              - simple: "{{?foo3}}"
                steps:
                  - set-header:
                      name: bar
                      simple: foo3
            otherwise:
              steps:
                - set-body:
                    simple: cheesecubez
        - to:
            uri: kamelet:sink

Action Sink

apiVersion: camel.apache.org/v1alpha1
kind: Kamelet
metadata:
  annotations:
    camel.apache.org/kamelet.icon: cherry
  labels:
    camel.apache.org/kamelet.type: sink
  name: eip-action-sink
spec:
  definition:
    title: EIP Kamelet
    description: Used to test all EIP we implement
    properties: {}
  dependencies:
    - camel:core
    - camel:kamelet
  template:
    from:
      uri: kamelet:source
      steps:
        - loop:
            constant: "3"
            copy: true
            steps:
              - delay:
                  expression:
                    simple: ${body}
                  async-delayed: true
        - choice:
            when:
              - simple: "{{?foo}}"
                steps:
                  - dynamic-router:
                      expression:
                        simple: ${body}
                  - set-header:
                      name: bar
                      simple: foo
              - simple: "{{?bar}}"
                steps:
                  - set-property:
                      name: property
                      simple: bar
                  - remove-property:
                      name: property
                  - marshal:
                      json:
                        library: Gson
              - simple: "{{?baz}}"
                steps:
                  - transform:
                      simple: baz
                  - aggregate:
                      correlation-expression:
                        simple: ${header.StockSymbol}
                      aggregation-strategy: myAggregatorStrategy
                      completion-size: 2
                  - log:
                      message: test
                      logging-level: INFO
                      log-name: yaml
            otherwise:
              steps:
                - set-body:
                    simple: cheesecubez
                - remove-header:
                    name: removeme
                - claim-check:
                    operation: Push
                    key: foo
                    filter: header:(foo|bar)

Evil EIP

apiVersion: camel.apache.org/v1alpha1
kind: Kamelet
metadata:
  annotations:
    camel.apache.org/kamelet.icon: cherry
  labels:
    camel.apache.org/kamelet.type: action
  name: eip-action
spec:
  definition:
    title: EIP Kamelet
    description: Used to test all EIP we implement
    properties: {}
  dependencies:
  - camel:core
  - camel:kamelet
  template:
    from:
      uri: kamelet:source
      steps:
      - loop:
          constant: '3'
          copy: true
          steps:
          - delay:
              expression:
                simple: ${body}
              async-delayed: true
      - choice:
          when:
          - simple: '{{?foo}}'
            steps:
            - dynamic-router:
                expression:
                  simple: ${body}
            - set-header:
                name: bar
                simple: foo
          - simple: '{{?bar}}'
            steps:
            - set-property:
                name: property
                simple: bar
            - remove-property:
                name: property
            - marshal:
                json:
                  library: Gson
          - simple: '{{?baz}}'
            steps:
            - transform:
                simple: baz
            - aggregate:
                correlation-expression:
                  simple: ${header.StockSymbol}
                aggregation-strategy: myAggregatorStrategy
                completion-size: 2
            - log:
                message: test
                logging-level: INFO
                log-name: yaml
          otherwise:
            steps:
            - set-body:
                simple: cheesecubez
            - remove-header:
                name: removeme
            - claim-check:
                operation: Push
                key: foo
                filter: header:(foo|bar)
      - filter:
          simple: '{{?foo}}'
          steps:
          - set-body:
              simple: abc
          - unmarshal:
              json:
                unmarshal-type-name: MyClass
          - circuit-breaker:
              description:
                text: Another one
                lang: eng
              steps:
              - enrich:
                  expression:
                    simple: ${body}
              - to:
                  uri: dropbox:put
                  parameters:
                    accessToken: '{{accessToken}}'
              - convert-body-to:
                  type: java.lang.String
                  charset: UTF8
              on-fallback:
                steps:
                - log:
                    message: test
                    logging-level: INFO
                    log-name: yaml
      - to:
          uri: kamelet:sink

Codecov Report

Merging #951 (ef67313) into main (458204a) will decrease coverage by 2.03%.
The diff coverage is 37.25%.

:exclamation: Current head ef67313 differs from pull request most recent head fdaba96. Consider uploading reports for the commit fdaba96 to get more accurate results

@@            Coverage Diff             @@
##             main     #951      +/-   ##
==========================================
- Coverage   41.79%   39.76%   -2.04%
==========================================
  Files          47       47
  Lines        1426     1441      +15
  Branches      284      288       +4
==========================================
- Hits          596      573      -23
- Misses        786      824      +38
  Partials       44       44

Impacted Files
Coverage Δ

src/components/VisualizationControls.tsx
80.00% <0.00%> (ø)

src/components/VisualizationStep.tsx
21.21% <0.00%> (-22.73%)
:arrow_down:

src/store/visualizationStore.tsx
23.52% <ø> (-11.77%)
:arrow_down:

src/components/Visualization.tsx
48.45% <5.55%> (-7.23%)
:arrow_down:

src/services/visualizationService.ts
68.88% <64.28%> (-6.31%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Ram memory increase when using Navigation Rail

When I change tab when using Navigation Rail on your example, my ram memory increase
The more I use Navigation Rail, the more ram memory increase
How to fix it? If I use this code for my program, it may cause slow due to over ram memory

Could you be more specific please: what program are you running exactly, are you ruining it as web or desktop, what process consumes memory, what OS are you on?


Nmap-error

This is the issue
They say first error we couldnt find gateway ip then they said error we couldnt find mac addres and then this happened.
"nmap: netutil.cc:3265: int route_dst_netlink(const sockaddr_storage*, route_nfo*, const char*, const sockaddr_storage*): Assertion `p != NULL' failed.\n"

Try - sudo apt install nmap


Different errors in explorer.

Different errors being generated on this explorer page:
https://explorer.stacks.co/txid/SPZW30K9VG6YCPYV4BX4V1FT0VJ66R1Q01W9DQ1W.city-collection-mia?chain=mainnet

Wasn't able to catch it as har file.

I'm moving to design so we can review these three versions and perhaps simplify.


[BIOIN-1832] Helper script that collects hompolymer locations in reference genome

Added a script that collects a sample of locations of homopolymers in a reference genome.
To be used for collecting error statistics in the future

@gilhornung , @doron-st - review much appreciated.


coinjoin dont work crashes

General Description

How To Reproduce?

Go to '...'
Click on '...'
Scroll down to '...'
See ERROR

Screenshots

Operating System

Logs

Insert Your Logs Here

Wasabi Version

Without knowing what version of Wasabi you are running, your operating system, and your recent log messages, there's no way to figure out what causes the crash. Could you provide this information?

This kind of useless issues must be closed immediately.