Point cloud visualization plugin

🎉 New feature

Closes https://github.com/ignitionrobotics/ign-gazebo/issues/1156
Requires https://github.com/ignitionrobotics/ign-msgs/pull/210 (also needs to be merged forward)

Summary

Porting the point cloud visualization tool from osrf/lrauv.
This needs to target Garden because it requires per-point coloring of point clouds:

https://github.com/ignitionrobotics/ign-rendering/pull/494
https://github.com/ignitionrobotics/ign-gui/pull/318
https://github.com/ignitionrobotics/ign-msgs/pull/202

Features:

Display hundreds of thousands of points with different colors
Point positions in 3D come from a PointCloudPacked message
Optionally, users can choose a separate FloatV topic to get values for each point
Point size and color can be changed from the GUI
Multiple point clouds can be displayed side-by-side, just instantiate the plugin multiple times

Known issues:

https://github.com/ignitionrobotics/ign-rendering/issues/497

Future work:
There are many improvements that can be made to the plugin. Here are some improvements I can think of for the future:

Use a palette with more than 2 colors to have a richer gradient
Optionally take color from a channel on the point cloud, instead of another topic
Support a custom reference frame, so we can properly visualize LIDAR data - right now all points must be expressed in the world frame

Test it

See the added example and its instructions:

Here's an examples using it to display maritime science data for LRAUV:

Checklist

[x] Signed all commits for DCO
[x] Added tests
[ ] Added example and/or tutorial
[x] Updated documentation (as needed)
[x] Updated migration guide (as needed)
[x] codecheck passed (See contributing)
[x] All tests passed (See test coverage)
[x] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.
🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸

Thanks for the review, @mabelzhang . I've addressed your comments and added an example. This is now open for review.

Do we care about the doxygen warnings in the Bionic CI?

Ouch oh yeah we do, on it

Just pending green CI.

Ok, doxygen should be fixed with edecdb8 . Besides that:

Ubuntu's warnings should be fixed when #345 is forward-ported
The Helpers test failure on Homebrew looks like a new failure unrelated to this PR

Okay re Jenkins ones.
Are these ones on the GitHub Actions intermittent?
  	 23 - UNIT_Publisher_TEST (Failed)
  	 35 - INTEGRATION_ExamplesBuild_TEST (Failed)

UNIT_Publisher_TEST is about X, failing only on the PR Action
  [GUI] [Wrn] [Application.cc:674] [QT] QXcbConnection: Could not connect to display :1.0
  [GUI] [Err] [Application.cc:678] [QT] Could not connect to any X display.

INTEGRATION_ExamplesBuild_TEST is failing on both the PR and push Actions.

UNIT_Publisher_TEST is about X, failing only on the PR Action

Yup, these flaky failures due to X are all captured here:

https://github.com/ignitionrobotics/ign-gui/issues/58

It makes life very complicated as we have to check each test and see if the failure was due to X :cry:

Humm now I'm confused about the Helper test failure on Homebrew. It doesn't fail on any of our stable branches, but I can't see why this PR would affect it.

I'm looking at the test results again, a different part of the log, and I'm seeing this related to the ExamplesBuild test that was failing:
  /github/workspace/examples/standalone/point_cloud/point_cloud.cc:31:40: error: variable 'std::atomic<bool> g_terminatePub' has initializer but incomplete type
   static std::atomic<bool> g_terminatePub(false);
                                          ^
  CMakeFiles/point_cloud.dir/build.make:62: recipe for target 'CMakeFiles/point_cloud.dir/point_cloud.cc.o' failed
  make[3]: *** [CMakeFiles/point_cloud.dir/point_cloud.cc.o] Error 1
  CMakeFiles/Makefile2:67: recipe for target 'CMakeFiles/point_cloud.dir/all' failed
  make[2]: *** [CMakeFiles/point_cloud.dir/all] Error 2
  Makefile:83: recipe for target 'all' failed
  make[1]: *** [all] Error 2

I think this is a real error?

I think this is a real error?

Thanks for spotting, yes! On it

I'm confused about the Helper test failure on Homebrew. It doesn't fail on any of our stable branches, but I can't see why this PR would affect it.

I pushed some bumpers to see if it prevents the test from segfaulting in a8ab1c0. It feels completely unrelated to this PR, but since the test consistently fails on this branch and consistently passes on main (search for "helper" here), I can't test if it works targeting a PR at main :woman_shrugging:

I pushed some bumpers to see if it prevents the test from segfaulting in a8ab1c0.

It looks like they worked :pray:
Ok, I think we're in good shape now, phew. Thanks for being thorough!

I'm confused about the Helper test failure on Homebrew. It doesn't fail on any of our stable branches, but I can't see why this PR would affect it.

I pushed some bumpers to see if it prevents the test from segfaulting in a8ab1c0. It feels completely unrelated to this PR, but since the test consistently fails on this branch and consistently passes on main (search for "helper" here), I can't test if it works targeting a PR at main 🤷♀

sadly this test is failing on main, though I don't think it was caused by this PR: https://github.com/ignitionrobotics/ign-gui/issues/360

Support a custom reference frame, so we can properly visualize LIDAR data - right now all points must be expressed in the world frame

I am trying to visualize a point cloud from a camera sensor attached to a drone but the data is in body frame. Has this been implemented and can someone steer me to the right place to add this?

There should be a frame ID in the header... I would think. Line 48 in examples/standalone/point_cloud/point_cloud.cc in this PR seems to suggest it's been implemented.
There's usage example in the gz-sim integration tests e.g.
https://github.com/gazebosim/gz-sim/blob/gz-sim8/test/integration/environment_preload_system.cc
https://github.com/gazebosim/gz-sim/blob/gz-sim8/test/integration/environmental_sensor_system.cc
gz-sim environment system implementation is here
https://github.com/gazebosim/gz-sim/tree/gz-sim8/src/systems/environment_preload
https://github.com/gazebosim/gz-sim/tree/gz-sim8/src/systems/environmental_sensor_system
There's a tutorial for using the environment system, in a PR https://github.com/gazebosim/gz-sim/pull/1806

@mabelzhang thanks for the reply. I took a look at those files and it's not clear to me how to access reference frame information of PointCloud data within a plugin. When I search for "frame" or "model" in the src/plugins/ I don't see any other examples of converting between reference frames for sensor data attached to a moving model. The goal is to visualize obstacle detect and avoid on PX4. Thanks for the help!
Here's the model I'm using for the drone, you can see how the camera is attached
https://github.com/PX4/PX4-Autopilot/blob/main/Tools/simulation/gz/models/x500_depth/model.sdf
I have some example code where I am downsampling a camera point cloud and republishing it
https://github.com/dakejahl/PX4-Autopilot/commit/02ef5280a647ffc83d8de74f0830476f1fe3d02a
Here's a pic of a drone with a point cloud looking at a block some distance from the origin

Hey, okay, so first, we really shouldn't be starting a new topic in a merged PR. The more correct thing to do would be to open an issue and tag this PR as the relevant source code. I replied with links to the source code because they depend on code in this PR. But with the new details you posted, it should really be in a new issue ticket for better visibility.
Please keep that in mind and open a new ticket if you post new information.
@arjo129 can people set frame ID when visualizing point clouds? That would be part of the environment system right?

To be fair this has nothing to do with the environment system. We should open a ticket, I suspect point clouds don't follow frame_ids set :scream: .

Sorry about that! I figured posting here was the best way to get the attention of the person who knows! :grin:
Ticket here
https://github.com/gazebosim/gz-gui/issues/595


Manual Device Scan

What does this PR do?
This adds an snmp scan command that does a partial snmp walk of a device and sends that data to the backend.
Motivation
NDMII-2395: we want to be able to use this data to streamline the process of editing profiles.
QA
The only QA necessary is to verify that the agent snmp walk command still runs; while this PR adds all the code to support the scan command, it doesn't actually connect it to the CLI app (because there is additional backend work ongoing, so right now the scan command just gathers some data and sends it for the backend to discard as meaningless).
Additional Notes
Some backend work will be necessary before this functions completely (the backend needs to support the transmitted data).

/merge


Support for two weather stations

I have two weather stations, WS90 and WS68 that I'd like both to be accessible simultaneously in the plugin. Is there a way this can be done? I haven't been successful implementing two instances of your plugin. I've tried having both weather stations reporting to the same /data/report but that doesn't seem to work either.  I don't know how to directly view /data/report so I cannot tell if it's getting mangled when both stations are outputting to it.
Thanks for any suggestions.
Louis

Thanks for the helpful information. My setup is two separate gateways with one weather station each. I actually need wind data from both stations so scenario #2 won't work.
Can you detail how to run two instances of the plugin? It isn't obvious if it can be done from the homebridge ui and I'm not sure what files to duplicate/modify to make it happen. Thanks!

Did some research and I think I figured this out. Steps I took:

Duplicated the Ecowitt platform in the JSON config file.
On the duplicate made these changes:

Changed mac, port, and path to mach the second weather station

Made various changes to name overrides, hidden, etc. on duplicate

Created a unique username and port in _bridge

Restart Homebridge

Pair each instance in HomeKit.
Looks like success!
Did I miss anything?
Thanks for your help!

I think that should cover it.  Thanks for sharing your steps.  If you leave on debug logging, each plugin should output the data report it's receiving, which you can use to validate the expected weather station and its data is going to the right plugin.
Please let me know if you hit any more issues.  And if you feel you are getting value from this plugin, please consider giving this repo a star.


[Feature] Add support for operating systems that can not properly provide gpioHardwareRevision

I am running a PXE booted custom alpha build of Flatcar Container Linux on my Raspberry Pi 4 which is basically a 64bit Gentoo live operating system. This means, I am not installing anything and the Raspberry gets netbooted every reboot. In this particular setup the gpioHardwareRevision can not be properly determined:
In /proc/cpuinfo none of the keys Hardware,  Revision, Serial or Model are present.
core@localhost:~# cat /proc/cpuinfo

processor       : 0
BogoMIPS        : 108.00
Features        : fp asimd evtstrm crc32 cpuid
CPU implementer : 0x41
CPU architecture: 8
CPU variant     : 0x0
CPU part        : 0xd08
CPU revision    : 3

processor       : 1
BogoMIPS        : 108.00
Features        : fp asimd evtstrm crc32 cpuid
CPU implementer : 0x41
CPU architecture: 8
CPU variant     : 0x0
CPU part        : 0xd08
CPU revision    : 3

processor       : 2
BogoMIPS        : 108.00
Features        : fp asimd evtstrm crc32 cpuid
CPU implementer : 0x41
CPU architecture: 8
CPU variant     : 0x0
CPU part        : 0xd08
CPU revision    : 3

processor       : 3
BogoMIPS        : 108.00
Features        : fp asimd evtstrm crc32 cpuid
CPU implementer : 0x41
CPU architecture: 8
CPU variant     : 0x0
CPU part        : 0xd08
CPU revision    : 3

Also, from my understanding due to the nature of the netboot, the file /proc/device-tree/system/linux,revision is not present on the system. This results in rev always being 0 and pigpiod not starting properly.
Usecase
My idea is to run a Kubernetes Cluster of Raspberry Pi's while still being able to use the GPIOs but from within the cluster. This approach would allow me to run a sensor cluster while using all the benefits Kubernetes offers as a resource scheduler and manager.
Solution
I would propose as a solution to this issue, that users can specify an environment variable to manually set the value for those usecases. This would work both, for the systemd unit running the daemon or of pigpiod was started from command line. The environment variable could contain either a path to a file providing the missing information or just provide the information directly. I would prefer a path to a file.

Why not put the missing information in the file /proc/cpuinfo?

@guymcswain If I remember correctly, /proc/ filesystem is a virtual one, the files in there aren't real and the filesystem as a whole is simply an interface to some of the information the kernel is holding in memory. You can't change /proc/cpuinfo unless you want to modify the kernel. If you could, your changes would be wiped out every time the file was updated by the kernel. So basically there two options, either I modify the kernel so that it prints the right information or pigpio offers a way to override the path where to get the information from.
Anyway, I was raising this request to open a discussion if that even possible or out of scope for pigpio.

https://www.raspberrypi.org/forums/viewtopic.php?t=232417&start=25
this thread contains a discussion on how to bind mount a fake /proc/cpuinfo

A solution was offered that does not require a change to the library.


whisper-auto-resize improvements and logging for all components

We implemented some improvements to the whisper-auto-resize script and added logging. Tested in production since many month.

Please rebase squash this merge, and add tests if you don't mind. The excess merge commits aren't necessary.
Also, this needs to be rebased against master, and can you split the changes up logically into individual commits? I'm going to close this PR as we can't merge this as it stands currently.


API client now passes the user's full name to ecommerce

This merge also requires a merge on ecommerce and edx-platform with tyler-jwt-fullname, so keep that in mind.
@clintonb @rlucioni FYI: @wedaly @jimabramson

Tests are required.

@clintonb Will do that now

Minor: Cleanup the commit message. We generally use a simple statement of the overall change as the title (e.g. Added support for passing full name via JWT). Additional information can be provided as a paragraph or list after the title.
Added support for passing full name via JWT

- User's full name can be passed to the API when using JWT authentication. This field is optional.
- Email is no longer required for JWT authentication.

Aside from the message change, :+1: . Please await a second approval from @rlucioni or @jimabramson before merging.

Oh, I actually not aware that Git messages could be multiline like that. I'll change it then force a push. Thanks @clintonb.

Once you get a clean build, :+1: .

@Nickersoft I'm surprised to see a major version bump on this change; was expecting 0.5.0.  Is there something backwards-incompatible?

@jimabramson My mistake. The constructor was changed in a backwards-incompatible manner. We can make it backwards-compatible if you'd like.

ah.   somehow i missed that.


[Enhancement] Add Generation to meow lookup

Description
This PR adds the generation information to the command meow lookup
Changes

adds a .gitignore
organizes imports of core props for index.js
adds  Generation to meow lookup

Example
$ meow loookup 10628
Kitty #10628
  Birthdate:    2017-12-01 23:17:05
  Owner:        0x88b8601E1679AbDe44332F0e3eFF32396fcDaA2D
  Generation:   5
  Busy Until:   4679125
  Genes:        0x5a12b318851a46e421ee18000310c67396e08146218e7318e343dcf639ae
  Matron Id:    10342
  Sire Id:      10258
  Siring With:  0
  URL:          https://www.cryptokitties.co/kitty/10628

Tests

[x] verified meow lookup returns expected output

Can you move the meow output back to alphabetical order?
Then it looks good and I'll merge. :)

@ricmoo: sure thing.

Sorry to bug you one last time, but Matron Id comes before Owner. :)
In the future the number of output parameters will get much longer, so it is important to be able to find the values you are looking for...

That was a brain fart there. I originally tried to sort based on a relative hierarchy -- so keys Matron Id and Sire Id would follow after one another -- but alphabetical works too.

LOL! No worries, it happens.
I will be adding cattributes, imageUrl and all that jazz too. My current version has auctions enabled, but needs more testing, and the cost of testing just got crazy.
I'm hoping to get it up tomorrow, along with auto-breed:
/home/ricmoo> meow --account account.json --auto-breed 1001 3021 5674 2566 100

Which will continuously look for random valid pairs between the kitty IDs whenever they aren't resting. It was running for over a day until the 0.015 birthing fee kicked in; then I stopped it.

Thanks again!

Cool!
The imageUrl seems to be https://storage.googleapis.com/ck-kitty-image/$contractAddress/$kittyId.svg and contractAddress is just 0x06012c8cf97bead5deae237070f9587f8e7a266d
How do you read cattributes?

Actually, that url only works for non-fancy cats; fancy cats need the API to figure out the image.
You have to hit their API for cattributes too. I've also got a script mining them into a CSV, just trying to figure out a better way to host them (prolly IPFS), so their servers don't get smashed by all the data-mining bots.

@ricmoo: I'm curious how cryptokittydex queries the cattributes.
I'd also like to add fetching your kitty Id's via your address using their API.


proxy: Use impl Trait to unbox some futures

Now that impl Trait is stable, we don't need to box as many futures. We still
need to box before spawning them on an executor, but the component futures no
longer require their own boxes.
Signed-off-by: Eliza Weisman<EMAIL_ADDRESS>

also: impl Trait! impl Trait! impl Trait!


[WB-9083] Dont assume executable for given entrypoints

Fixes WB-9083
Fixes #NNNN
Description
What does the PR do?
Testing
How was this PR tested?
Checklist

Name PR "[WB-NNNN][WB-MMMM] Add support for..." similar to entries in CHANGELOG.md
Include reference to internal ticket "Fixes WB-NNNN" (and github issue "Fixes #NNNN" if applicable)

Absolutely. This PR is solving that exact issue, a user passing in a .py extension and us assuming python rather than python3. So this starts by forcing users to specify the executable as much as possible.


Regular increase of disk usage

cloudsql-proxy seems to leak disk on the pod resource graph.
Is it expected behavior or, any wrong settings?
It is using to connect to Cloud SQL for PostgreSQL.

Hi @idubnori,
Could you post the settings you are using? Is your application correctly closing connections after they are used?

I don't think the cloudsql-proxy uses the disk much at all, so this is a
little weird. Have you configured something in the pod to write logs to a
local file or something?

Hi @kurtisvg @Carrotman42,
Thank you for your advices.
The configuration is almost based on GKE tutorial.
My container app stacks are ASP.NET Core + EF Core + npgsql. And Active connection count leak did not observe on Cloud SQL for PostgreSQL resource graph.
I belive my container does not output any local files or something to the container side, but the purple graph(my container) has the one cliff of disk usage increment. I will check it again...
The normal behavior is that cloudsql-proxy disk usage graph should be perfect flat?

apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  name: foo-api
  labels:
    app: foo-api
spec:
  template:
    metadata:
      labels:
        app: foo-api
    spec:
      containers:
        - name: foo-api
          image: asia.gcr.io/foo-project-id/foo.api:latest
          resources:
            requests:
              cpu: 200m
              memory: 128Mi
            limits:
              cpu: 1500m
              memory: 256Mi
          imagePullPolicy: Always
          ports:
            - containerPort: 80
          # The following environment variables will contain the database host,
          # user and password to connect to the PostgreSQL instance.
          env:
            # [START cloudsql_secrets]
            - name: POSTGRES_DB_USER
              valueFrom:
                secretKeyRef:
                  name: cloudsql-db-credentials
                  key: username
            - name: POSTGRES_DB_PASSWORD
              valueFrom:
                secretKeyRef:
                  name: cloudsql-db-credentials
                  key: password
            # [END cloudsql_secrets]
            - name: ASPNETCORE_ENVIRONMENT
              value: Development
            - name: ConnectionStrings__FooDatabase
              value: "Host=<IP_ADDRESS>;Port=5432;Username=$(POSTGRES_DB_USER);Password=$(POSTGRES_DB_PASSWORD);Database=Foo"
          readinessProbe:
            httpGet:
                path: /hc
                port: 80
            initialDelaySeconds: 5
            periodSeconds: 10

        # [START proxy_container]
        - name: cloudsql-proxy
          image: gcr.io/cloudsql-docker/gce-proxy:1.11
          foo: ["/cloud_sql_proxy",
                    "-instances=foo-project-id:asia-northeast1:foo-cloudsql-1=tcp:5432",
                    "-credential_file=/secrets/cloudsql/credentials.json"]
          resources:
            requests:
              cpu: 10m
              memory: 8Mi
            limits:
              cpu: 100m
              memory: 32Mi
          # [START cloudsql_security_context]
          securityContext:
            runAsUser: 2  # non-root user
            allowPrivilegeEscalation: false
          # [END cloudsql_security_context]
          volumeMounts:
            - name: cloudsql-instance-credentials
              mountPath: /secrets/cloudsql
              readOnly: true
        # [END proxy_container]
      # [START volumes]
      volumes:
        - name: cloudsql-instance-credentials
          secret:
            secretName: cloudsql-instance-credentials
      # [END volumes

Are you able to run df in the container to see where the space is going?

@hfwang Container image layer seems to increase.
Baseline
Filesystem           1K-blocks      Used Available Use% Mounted on
overlay               98868448   3419208  95432856   3% /
tmpfs                  1819160         0   1819160   0% /dev
tmpfs                  1819160         0   1819160   0% /sys/fs/cgroup
/dev/sda1             98868448   3419208  95432856   3% /dev/termination-log
tmpfs                  1819160         4   1819156   0% /secrets/cloudsql
/dev/sda1             98868448   3419208  95432856   3% /etc/resolv.conf
/dev/sda1             98868448   3419212  95432852   3% /etc/hostname
/dev/sda1             98868448   3419212  95432852   3% /etc/hosts
shm                      65536         8     65528   0% /dev/shm
tmpfs                  1819160        12   1819148   0% /var/run/secrets/kubernetes.io/serviceaccount
tmpfs                  1819160         0   1819160   0% /proc/kcore
tmpfs                  1819160         0   1819160   0% /proc/timer_list
tmpfs                  1819160         0   1819160   0% /sys/firmware

About 1min after
Filesystem           1K-blocks      Used Available Use% Mounted on
overlay               98868448   3419272  95432792   3% /
tmpfs                  1819160         0   1819160   0% /dev
tmpfs                  1819160         0   1819160   0% /sys/fs/cgroup
/dev/sda1             98868448   3419272  95432792   3% /dev/termination-log
tmpfs                  1819160         4   1819156   0% /secrets/cloudsql
/dev/sda1             98868448   3419272  95432792   3% /etc/resolv.conf
/dev/sda1             98868448   3419272  95432792   3% /etc/hostname
/dev/sda1             98868448   3419272  95432792   3% /etc/hosts
shm                      65536         8     65528   0% /dev/shm
tmpfs                  1819160        12   1819148   0% /var/run/secrets/kubernetes.io/serviceaccount
tmpfs                  1819160         0   1819160   0% /proc/kcore
tmpfs                  1819160         0   1819160   0% /proc/timer_list
tmpfs                  1819160         0   1819160   0% /sys/firmware

can you look in the contents of /dev/termination-log, it should have output when processes exit, and that should give you some insight into where the disk usage increase is coming from.

Closing this due to lack of response and unable to replicated on our end.


macOS 11.0.1 BigSur

The Widget on the notification center doesn't appear anymore with the new OS.

linking #58


MyPy tracking script

Changes made for this issue #24623
Description This scripting tool will allow MyPy coverage to be tracked in the project.
Verification Ran the script and verified that the output was correct.

Thanks for the PR, however I don't want to maintain a complicated custom script to do this. Feel free to open a new PR if you can simplify this.


/var/log/messages spam

Hi and thank you for this perfect project.
I am not sure in the reasons, but looks these messages appeared after this script.
Be honest I don't check messages very often.
Synology_enable_M2_volume v1.0.10
DS918+ DSM 7.2-64570-3
root@Server:~# tail -f /var/log/messages
2023-09-28T08:36:20+03:00 Server synostgd-disk[29430]: disk_monitor.c:289 The temperature[23] of /dev/nvme0n1 >= T2. (T1: -273, T2: -273)
2023-09-28T08:37:20+03:00 Server synostgd-disk[29608]: disk_monitor.c:289 The temperature[23] of /dev/nvme0n1 >= T2. (T1: -273, T2: -273)
2023-09-28T08:39:20+03:00 Server synostgd-disk[30025]: disk_monitor.c:289 The temperature[23] of /dev/nvme0n1 >= T2. (T1: -273, T2: -273)
2023-09-28T08:40:20+03:00 Server synostgd-disk[30203]: disk_monitor.c:289 The temperature[23] of /dev/nvme0n1 >= T2. (T1: -273, T2: -273)
2023-09-28T08:42:20+03:00 Server synostgd-disk[31038]: disk_monitor.c:289 The temperature[23] of /dev/nvme0n1 >= T2. (T1: -273, T2: -273)
2023-09-28T08:43:20+03:00 Server synostgd-disk[31267]: disk_monitor.c:289 The temperature[23] of /dev/nvme0n1 >= T2. (T1: -273, T2: -273)
2023-09-28T08:44:20+03:00 Server synostgd-disk[31641]: disk_monitor.c:289 The temperature[23] of /dev/nvme0n1 >= T2. (T1: -273, T2: -273)
2023-09-28T08:45:20+03:00 Server synostgd-disk[32206]: disk_monitor.c:289 The temperature[23] of /dev/nvme0n1 >= T2. (T1: -273, T2: -273)
2023-09-28T08:46:20+03:00 Server synostgd-disk[318]: disk_monitor.c:289 The temperature[23] of /dev/nvme0n1 >= T2. (T1: -273, T2: -273)
2023-09-28T08:47:20+03:00 Server synostgd-disk[908]: disk_monitor.c:289 The temperature[23] of /dev/nvme0n1 >= T2. (T1: -273, T2: -273)
2023-09-28T08:48:20+03:00 Server synostgd-disk[1486]: disk_monitor.c:289 The temperature[23] of /dev/nvme0n1 >= T2. (T1: -273, T2: -273)
2023-09-28T08:49:20+03:00 Server synostgd-disk[2047]: disk_monitor.c:289 The temperature[23] of /dev/nvme0n1 >= T2. (T1: -273, T2: -273)
2023-09-28T08:50:20+03:00 Server synostgd-disk[2630]: disk_monitor.c:289 The temperature[23] of /dev/nvme0n1 >= T2. (T1: -273, T2: -273)
2023-09-28T08:51:20+03:00 Server synostgd-disk[3199]: disk_monitor.c:289 The temperature[23] of /dev/nvme0n1 >= T2. (T1: -273, T2: -273)
2023-09-28T08:52:20+03:00 Server synostgd-disk[3770]: disk_monitor.c:289 The temperature[23] of /dev/nvme0n1 >= T2. (T1: -273, T2: -273)
2023-09-28T08:53:20+03:00 Server disk_monitor.c:289 The temperature[23] of /dev/nvme0n1 >= T2. (T1: -273, T2: -273)
2023-09-28T08:54:20+03:00 Server synostgd-disk[4947]: disk_monitor.c:289 The temperature[23] of /dev/nvme0n1 >= T2. (T1: -273, T2: -273)
2023-09-28T08:55:20+03:00 Server synostgd-disk[5519]: disk_monitor.c:289 The temperature[23] of /dev/nvme0n1 >= T2. (T1: -273, T2: -273)

I would be very grateful if you could advise me with the reason of this spam and how it could be fixed

I've never seen that before. But I've never looked for it before either.
I just checked /var/log/messages for my DS1821+ DSM 7.2-64570-2 and it doesn't have nvme temperature related log entries.
I also checked /var/log/messages for my DS720+ DSM 7.2.1-69057 and it doesn't have any disk_monitor log entries.
Plenty of people with a DS918+ just use the syno_hdd_db script and not the syno_enable_m2_volume script.
Do you want to:

Run syno_enable_m2_volume --restore
Reboot and run syno_hdd_db -n
Check your volume is still shown in storage manager.
Wait a couple of minutes and then check the log.

Thank you for the replay. Ive updated to 7.2.1 and while updating system have lost my nvme disk. Re-run syno_enable_m2_volume.sh helped me. I have some critical services runs on SSD, so Im sorry, but I`m not ready to experiments.
For now on 7.2.1 spam stays.

Christian Franke, the co-creator of smartctrl, says for smartctrl v6.6 to  try lease retry smartctrl --info /dev/nvme0 but DSM has smartctrl v6.5
I've never been able to get smartctrl --info to work with an NVMe drive. I believe the smartctrl version in DSM is too old.


GameLogTextPaths

Is there an example of how GameLogTextPaths should work. I used dnSpy to find the right line of code. But I do not know how to specify the path.

I guess it is a fairly undocumented feature. But you should  put the path of a text component that is treated as a "log" into it. This will simply make the plugin treat the texts on a per-line basis.


WAIT FOR REVIEW changed parents regulation of transcription by chromatin organization…

… fixes #11628

OK discussed with @ValWood


GUVNOR-2820: Enumerations are present even if they were deleted

See https://issues.jboss.org/browse/GUVNOR-2820
This PR adds a base class to invalidate the LRUDataModelOracleCache when certain files are deleted.

Part of an ensemble:
https://github.com/uberfire/uberfire/pull/597
https://github.com/droolsjbpm/kie-wb-common/pull/624
https://github.com/droolsjbpm/drools-wb/pull/375


Fix flake8 warnings

Fix flake8 warnings for both source and test files. The following warnings were ignored:

F403 'from utils import *' used; unable to detect undefined names

And for line too long warnings, some were fixed when the code readability was not affected negatively.

Thanks, @lucasmoura !


Please add validation data

Please add validation all the filed getting expected data fielding or not.
Please add validation all the filed getting expected data fielding or not and once all the rows complete with the information give a complete status.

Can you please explain what you are needing?

@purnachandarr I am closing this issue in an effort to clean up the issues page.
If you’re issue persists, please follow the guidelines in the README and submit a new issue using the template provided via the link.


[main] Fix - Update dependencies from dotnet/arcade

In the internal build it failed with the following error message:
Build FAILED.

D:\a\_work\_tool\dotnet\sdk\8.0.101\Microsoft.Common.CurrentVersion.targets(4911,5): error MSB3030: Could not copy the file "D:\a\_work\1\s\artifacts\bin\Microsoft.TryDotNet\Release\net8.0\Microsoft.TryDotNet.pdb" because it was not found. [D:\a\_work\1\s\src\Microsoft.TryDotNet.IntegrationTests\Microsoft.TryDotNet.IntegrationTests.csproj]
D:\a\_work\_tool\dotnet\sdk\8.0.101\Microsoft.Common.CurrentVersion.targets(4911,5): error MSB3030: Could not copy the file "D:\a\_work\1\s\artifacts\bin\Microsoft.TryDotNet.WasmRunner\Release\net8.0\Microsoft.TryDotNet.WasmRunner.pdb" because it was not found. [D:\a\_work\1\s\src\Microsoft.TryDotNet.IntegrationTests\Microsoft.TryDotNet.IntegrationTests.csproj]
    0 Warning(s)
    2 Error(s)

@ocallesp : do you know if we are building with embedded pdbs? That is the default AFAIK, in which case, I wouldn't expect those files to be present. We'd want to fix our code.
On a side note, did you want to publish this PR?

@ocallesp : do you know if we are building with embedded pdbs? That is the default AFAIK, in which case, I wouldn't expect those files to be present. We'd want to fix our code.
On a side note, did you want to publish this PR?

None of the projects are using embedded PDBs, so we are expecting the .pdb files to be generated.

Passed internally
https://dev.azure.com/dnceng/internal/_build/results?buildId=2386862&view=results


issue install from ROS2 package list

When I install the motion capture tracking with
sudo apt-get install ros-humble-motion-capture-tracking

Then I get the following error when running launch:
$ ros2 launch motion_capture_tracking launch.py
[INFO] [launch]: All log files can be found below /home/kimberly/.ros/log/2024-02-27-08-40-59-962626-pop-os-32712
[INFO] [launch]: Default logging verbosity is set to INFO
[INFO] [motion_capture_tracking_node-1]: process started with pid [32713]
[ERROR] [motion_capture_tracking_node-1]: process has died [pid 32713, exit code 127, cmd '/opt/ros/humble/lib/motion_capture_tracking/motion_capture_tracking_node --ros-args -r __node:=motion_capture_tracking --params-file /opt/ros/humble/share/motion_capture_tracking/config/cfg.yaml'].
[INFO] [rviz2-2]: process started with pid [32715]
[motion_capture_tracking_node-1] /opt/ros/humble/lib/motion_capture_tracking/motion_capture_tracking_node: error while loading shared libraries: libNatNet.so: cannot open shared object file: No such file or directory

This was found when trying to solve this crazyswarm2 issue https://github.com/IMRCLab/crazyswarm2/issues/432

Oh, that's super annoying:-/ Thanks for testing it out.
As workaround one could put libNatNet in the PATH (from https://github.com/whoenig/NatNetSDKCrossplatform/tree/main/lib/ubuntu). Of course, Optitrack doesn't provide source code or a static library, so I am not sure how to fix it besides disabling the "optitrack_closed_source" backend in the binary distribution.


Fix retraction script

I just noticed that our retraction logic was broken due to an import error from https://github.com/leap-stc/leap-data-management-utils. This is likely related to the refactor here

Ok this seems to work again.


:shield: travis: use python3.6

It's a try to fix encoding problem on printing HTML in random tour errors
https://api.travis-ci.com/v3/job/212575529/log.txt
'ascii' codec can't encode character u'\u20ac' in position 7: ordinal not in range(128)

I approve to merge it now


HTTPS Support

Is it possible to use this with SSL enabled, without having to load insecure content?
I cannot resolve https://strapdownjs.com/v/0.2/strapdown.js
If I try with https://raw.githubusercontent.com/arturadib/strapdown/gh-pages/v/0.2/strapdown.js it loads without a theme.

Either, self host or modify strapdown.js to match the new paths.


not have maven build project?

not have maven build project?

Maven link is now in the Readme documented
LG JUergen


Oscillations in overtones

I have no idea if this is expected behavior in a SST, but at least to me it was a surprise. I noticed this at first when analyzing raw audio material, which revealed oscillations in overtones much more than what my hear would tell. But it is also possible to reproduce the effect synthetically:
The following example shows a mix of sine waves with frequencies i * f_base. In each plot one more frequency is added; the left side shows the transform of the mix, the right side is the transform of only the last/added harmonic (just as a check how its transform looks in isolation):

Full reproduction code
import numpy as np
import matplotlib.pyplot as plt

from ssqueezepy import synsq_cwt_fwd

SAMPLE_RATE = 44100

def sine(freq, num_samples, amp=1.0):
    ts = np.arange(num_samples) / SAMPLE_RATE
    wave = amp * np.sin(2.0 * np.pi * freq * ts)
    return wave

def mix(*waves):
    return np.array(waves).sum(axis=0) / len(waves)

def plot(Tx, ax):
    xs = np.arange(Tx.shape[1] + 1)
    ys = np.arange(Tx.shape[0] + 1)
    img = ax.pcolormesh(xs, ys, np.abs(Tx))
    plt.colorbar(img, ax=ax)

def plot_overtone_series(num_overtones, freq=440.0, num_samples=SAMPLE_RATE // 4):
    waves = [
        sine(freq * i, num_samples)
        for i in range(1, num_overtones + 1)
    ]

    Tx_mix, *_ = synsq_cwt_fwd(mix(*waves), fs=1.0, nv=32)
    Tx_add, *_ = synsq_cwt_fwd(waves[-1], fs=1.0, nv=32)

    fig, axes = plt.subplots(1, 2, figsize=(18, 8))
    plot(Tx_mix, axes[0])
    plot(Tx_add, axes[1])
    fig.tight_layout()
    plt.savefig("/tmp/overtones_{}".format(num_overtones))
    plt.show()

if __name__ == "__main__":
    for i in range(1, 9):
        plot_overtone_series(i)

Observations:

Even though the sine waves have a constant instantaneous frequency, the SST plot shows relatively large oscillations -- each time in the highest harmonic. For instance here is a zoomed in version of the n = 4 case:

If I understand it correctly there are 32 "voices" per octave. The oscillation covers ~10 bins, which in musical terms would correspond to more than 3 semitones -- which seems a lot for a constant frequency signal.

The highest harmonic seems to cover up the harmonics in between. Perhaps this oscillation in the highest harmonic is actually an reassignment artifact of "stealing" energy from the lower harmonics?

Thanks for the example and code. As for the questions ... I have no idea. I haven't gotten around to synchrosqueezing at all yet, been wrestling CWT all this time. When I do get there, this should be a fine test case.

Finished with CWT; for now, my decision is to keep ssqueezepy's, with an additional test result on amplitude modulation coming up. It remains to be seen whether ssqueezepy's in particular confers benefits to synchrosqueezing or whether PyWavelets' can be used, but I've already ruled out Scipy's CWT. (ssqueezepy's not going anywhere; it's only about whether pywt's will be used at all)
Reproduced your plots; it indeed seems like a sort of interaction between nearby bands; try below. I'll comment further once I learn more; now moving to studying phase transform, then synchrosqueezing.

Modified code
import numpy as np
import matplotlib.pyplot as plt

from ssqueezepy import synsq_cwt_fwd

SAMPLE_RATE = 44100

def sine(freq, num_samples, amp=1.0):
    ts = np.arange(num_samples) / SAMPLE_RATE
    wave = amp * np.sin(2.0 * np.pi * freq * ts)
    return wave

def mix(*waves):
    return np.array(waves).sum(axis=0) / len(waves)

def plot(Tx, ax):
    xs = np.arange(Tx.shape[1] + 1)
    ys = np.arange(Tx.shape[0] + 1)
    ax.pcolormesh(xs, ys, np.abs(Tx))

def plot_overtone_series(mults, freq=440.0, num_samples=SAMPLE_RATE // 4):
    waves = [
        sine(freq * m, num_samples)
        for m in mults
    ]

    Tx_mix, *_ = synsq_cwt_fwd(mix(*waves), fs=1.0, nv=32)
    Tx_add, *_ = synsq_cwt_fwd(waves[-1], fs=1.0, nv=32)

    fig, axes = plt.subplots(1, 2, figsize=(12, 8))
    plot(Tx_mix, axes[0])
    plot(Tx_add, axes[1])
    fig.tight_layout()
    plt.show()

if __name__ == "__main__":
    plot_overtone_series([1, 2, 3, 4, 6, 8])
    plot_overtone_series([1, 2, 3, 4, 8])
    plot_overtone_series([1, 2, 3, 8])
    plot_overtone_series([1, 2, 8])
    plot_overtone_series([1, 8])

Confirming, this is indeed per insufficiently separated bands.
Effectiveness of synchrosqueezing and the modulation model is rooted in "separability conditions", which include that frequency bands are sufficiently separated - or more generally, frequency modulation and amplitude modulation are "sufficiently slow" relative to that of the carrier ("main signal"), and what qualifies as "slow enough" is controlled by a wavelet's time-frequency parameter (for Morlet, mu). Larger mu -> superior frequency resolution, poorer time resolution.
It's much clearer from CWT plots:

Code
import numpy as np
import matplotlib.pyplot as plt

from ssqueezepy import ssq_cwt, cwt

SAMPLE_RATE = 4100

def sine(freq, num_samples, amp=1.0):
    ts = np.arange(num_samples) / SAMPLE_RATE
    wave = amp * np.sin(2 * np.pi * freq * ts)
    return wave

def mix(*waves):
    return np.array(waves).sum(axis=0) / len(waves)

def imshow(Tx, yticks, ax):
    ax.imshow(np.abs(Tx), aspect='auto')
    locs = np.linspace(0, len(yticks) - 1, 8).astype('int32')
    yt = ["%.2f" % h for h in yticks[locs]]
    ax.set_yticks(locs)
    ax.set_yticklabels(yt[::-1])

def overtone_ssq_cwt(mults, freq=220.0, num_samples=SAMPLE_RATE//4, mu=6, ssq=1):
    waves = [
        sine(freq * m, num_samples)
        for m in mults
    ]
    kw = dict(nv=32, wavelet=('morlet', {'mu': mu}))
    if ssq:
        Tx_mix, yt_m, *_ = ssq_cwt(mix(*waves), fs=SAMPLE_RATE, **kw)
        Tx_add, yt_a, *_ = ssq_cwt(waves[-1],   fs=SAMPLE_RATE, **kw)
        Tx_mix, Tx_add = np.flipud(Tx_mix), np.flipud(Tx_add)
    else:
        Tx_mix, yt_m, *_ = cwt(mix(*waves), dt=1/SAMPLE_RATE, **kw)
        Tx_add, yt_a, *_ = cwt(waves[-1],   dt=1/SAMPLE_RATE, **kw)

    fig, axes = plt.subplots(1, 2, figsize=(12, 8), sharey=True)
    imshow(Tx_mix, yt_m, axes[0])
    imshow(Tx_add, yt_a, axes[1])

    fig.tight_layout()
    title = "{} -- mu={:.1f}".format("SSWT" if ssq else "CWT", mu)
    fig.suptitle(title, weight='bold', fontsize=15, y=1.01)
    plt.show()

if __name__ == "__main__":
    mults = [1, 2, 3, 4, 6, 8]
    overtone_ssq_cwt(mults, mu=16, ssq=1)
    for mu in (4, 7, 10, 13, 16):
        overtone_ssq_cwt(mults, mu=mu, ssq=0)

Note that in log-CWT "closeness" is measured in ratios, i.e. 1 is as close to 2 as 4 to 8, but it's more like 4 to 8.1 or so and worse for higher frequencies due to CWT's inherent time-frequency resolution:

For separating high frequencies, or frequencies distributed more linearly than exponentially, STFT is the better option. In fact, this is a very important advantage of STFT over CWT, among a few others, which I didn't acknowledge earlier; this said, I will be porting STFT and its synchrosqueezing for v0.6.0.

Indeed, comparing to the raw CWT is very insightful. I have done another side-by-side comparison, plots left-to-right are:

CWT mixed signal
SSWT mixed signal
CWT last overtone
SSWT last overtone

mu = 5.0

mu = 10.0

mu = 20.0

Basically at mu = 5, the CWT is so blurred in frequency, that there is no way for SSWT to separate the independent components properly. Interesting though that it manifests itself in oscillations. And fascinating to see that the SSWT completely "removes all blur" in case the components are well separated in CWT.
Since everything works as expected, we can probably close this issue.
Side question: Any particular reason why cwt and ssq_cwt are slightly inconsistent in their API? One taking fs the other one dt, and apparently different semantics on the y/scales/frequency axis? The np.flipud indicates they are flipped, which I applied here as well. But thought I can skip the explicit yt handling that you did -- apparently this is indeed needed, which is why in my plots there is an offset (which seems to increase with mu).

@bluenote10 This is a nice canonical illustrative example I prefer to keep open for others (feel free to unsubscribe if spammy).
Indeed, I've touched on why there's interference, but why's it manifest as oscillations? And mu does cause shifts. These'd make a nice Stack Exchange question, and I'd prefer if you were to ask there if possible (can just paste original post, I'll bring up the relevant points). I'll still respond a bit later, regardless.

Regarding dt and fs - I agree it is duplicative and actually entirely redundant, but it's how the toolbox did it and the upside is added flexibility to specifying parameters. But it's nothing final yet - I'm thinking of either going with a single parameter or making options consistent across cwt and ssq_cwt.

These'd make a nice Stack Exchange question, and I'd prefer if you were to ask there if possible

Sorry for the delay. I was hoping I could turn the question into something that shows more "research effort", but in the end I now basically asked it as is: [https://dsp.stackexchange.com/questions/71855/why-does-a-synchrosqueezed-wavelet-transform-show-oscillating-behavior](Why does a synchrosqueezed wavelet transform show oscillating behavior?)

And mu does cause shifts.

I guess I see why changing mu does, but I was more referring to the shift between the pure CWT and the synchrosqueezed CWT bins. Basically I would have expected that the semantics of the y-axis bins are the same, i.e., in the above plots the left and right hand side images "align".

@bluenote10 I'd panic on the slightly inconvenient timing of the post in light of what I have planned, but good luck to anyone in beating my answer to come; may take a few days though. And the question is fine; I'll address it, the mus, and everything else on this thread there.

@bluenote10 Bonus question that I just stumbled upon in tuning testing: take CWT of white noise with mu=5 and mu=20; why does latter have so much... 'nothingness' higher up? Not easy to answer - might not dedicate a Q&A to it but can share later if you're interested.

I keep making more work for myself with these discoveries - will v0.5.0 release before glaciers melt? Unclear.

Been busy with visuals and automatic scale setting (my previous comment is no longer applicable unless you do scales='log:naive'). Will get to your q after v0.5.0, which should be anyday now. The linked Q&A is also relevant to it.

Happy holidays.


Normalize the log format of restful-swagger

Coverage decreased (-0.04%) to 61.687% when pulling dd8b5d9546b53c733f658d83d7e53b47ce719b06 on humingcheng:master into 248c8d196b7357d9e568dce40b11d9ba2ae2ecd3 on ServiceComb:master.


Question added

Checklist:

[ x] I have read freeCodeCamp's contribution guidelines.
[x ] My pull request has a descriptive title (not a vague title like Update index.md)

Closes #XXXXX

@erickleon88
I pushed up a fix to resolve your syntax errors.
In the future, please be careful about double checking for correct syntax for these quiz question entries 👍


add ability to set how much memory you would like to run with and change memory on up

review @jsingle

VirtualBox does everything in MB, so we should too.

Looks good


fix: login expire limit

Fixed an issue where the number of days in RememberMe for login was incorrect in multiple languages.

Thank you, merging in!


refactor(model): improve == and hashCode methods

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 6 (0.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 24.555%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/src/model/date/date_range_item.dart
0
2
0.0%

lib/src/model/date/holiday.dart
0
4
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
262

Relevant Lines:
1067

💛  - Coveralls


[SPARK-11714][Mesos] Make Spark on Mesos honor port restrictions on coarse grain mode 2

Make mesos coarse grained scheduler accept port offers

Propagate them through an env var to the executor

Enhance startServiceOnPort utils method to take into consideration port restrictions when starting a servic

Refer to this link for build results (access rights to CI server needed):
https://ci.typesafe.com/job/ghprb-spark-multi-conf/135/

retest this please jenkins

Refer to this link for build results (access rights to CI server needed):
https://ci.typesafe.com/job/ghprb-spark-multi-conf/136/

Refer to this link for build results (access rights to CI server needed):
https://ci.typesafe.com/job/ghprb-spark-multi-conf/137/


Colors

Is there any way to set different colors for the navbar and sidebar?

Hi,
You can add your custom CSS class with .navbar to change the color in nav-bar.
For sidebar color options add your custom class with .pmd-sidebar to overwrite default color.

Ok thanks @sunil-digicorp ,
You guys might want to add some examples in the docs for doing this, considering material design is designed around different, bold colours. Maybe create some standard colour palettes already in the css, or different theme files...
https://material.io/guidelines/style/color.html#color-color-palette


Migrate template from existing repository to this repository

Copied the template from old repository into here
Updated template with new paths
Updated documentation

@srjames-ukho could you remove yourself directly from the code owner?

@srjames-ukho could you remove yourself directly from the code owner?

Ha, yeah.

Cool, probably be the same for the admin role. I think you're in there and get access via the team as well.

Cool, probably be the same for the admin role. I think you're in there and get access via the team as well.

I think I found what you meant, I removed myself.

Can anyone from @UKHO/calypso comment on this as LGTM?

Can someone from @UKHO/digital-delivery-capability give their blessing?

:blessing: @srjames-ukho

:blessing: @srjames-ukho

Thanks Saint Luke


Added the barriers to fail the task and workflows with large output

Also added a bug fix for FAILED_WITH_TERMINAL_ERROR Workflow output

Is there any document on the size limit of task output/input?


API and Job Spec Stability

Get to API stability for the Job, JobSpec, etc structs so that we can turn on persistence without being unable to upgrade nodes in future, and for the yaml/json versions too - IOW, change whatever we want to change before we commit to the API
ETA: 2023Q1

https://www.notion.so/pl-strflt/Job-Schema-521ba6cdc06b4bdb940dbb151c576882?pvs=4


smoketest: Fix test_vm_life_cycle secure migration tests

This fixes intermittently failing secure live VM migration tests on KVM.
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

How Has This Been Tested?
After fixing smoketest, ran the tests several times against two KVM CentOS7 monkeyboxes against which they passed.
Test secure VM migration ... === TestName: test_01_secure_vm_migration | Status : SUCCESS ===
ok
Test Non-secured VM Migration ... === TestName: test_02_unsecure_vm_migration | Status : SUCCESS ===
ok
Test destroy Virtual Machine ... === TestName: test_03_secured_to_nonsecured_vm_migration | Status : SUCCESS ===
ok
Test Non-secured VM Migration ... === TestName: test_04_nonsecured_to_secured_vm_migration | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 4 tests in 579.830s

OK

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
Testing
[ ] I have added tests to cover my changes.
[x] All relevant new and existing integration tests have passed.
 I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2132

Trillian test result (tid-2767)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 25442 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2715-t2767-kvm-centos7.zip
Intermitten failure detected: /marvin/tests/smoke/test_deploy_virtio_scsi_vm.py
Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermitten failure detected: /marvin/tests/smoke/test_vm_life_cycle.py
Intermitten failure detected: /marvin/tests/smoke/test_vpc_vpn.py
Smoke tests completed. 63 look OK, 4 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

ContextSuite context=TestDeployVirtioSCSIVM>:setup
Error
0.00
test_deploy_virtio_scsi_vm.py

test_03_vpc_privategw_restart_vpc_cleanup
Error
115.15
test_privategw_acl.py

test_01_secure_vm_migration
Error
67.49
test_vm_life_cycle.py

test_02_unsecure_vm_migration
Error
66.50
test_vm_life_cycle.py

test_03_secured_to_nonsecured_vm_migration
Error
67.50
test_vm_life_cycle.py

test_04_nonsecured_to_secured_vm_migration
Error
66.50
test_vm_life_cycle.py

test_08_migrate_vm
Error
15.86
test_vm_life_cycle.py

test_01_redundant_vpc_site2site_vpn
Failure
327.08
test_vpc_vpn.py

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2139

@blueorangutan test

@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-2777)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 18952 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2715-t2777-kvm-centos7.zip
Intermitten failure detected: /marvin/tests/smoke/test_deploy_virtio_scsi_vm.py
Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
Smoke tests completed. 65 look OK, 2 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

ContextSuite context=TestDeployVirtioSCSIVM>:setup
Error
0.00
test_deploy_virtio_scsi_vm.py

test_03_vpc_privategw_restart_vpc_cleanup
Error
102.00
test_privategw_acl.py

Merged based on test results and 2 lgtms.


Feature/12219 testprocess family members/12482 unit tests

Description
Adding unit tests for ExposureSubmissionTestResultFamilyMemberViewModel and ExposureSubmissionCoordinatorModelTests
Link to Jira
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-12482

What about testing those?

What about testing those?

That is something which @nickguendling worked on in https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-12304. I just added the unit tests related to 12421 and 12342


Auth request with security factor requirements does not allow authentication after adding factors from request screen

I'm running into a bug where the user cannot approve an authentication request if they have not previously set up a security factor and then attempt to set one up. This is using version 2.2.1 of the whitelabel sdk. I can repeat it using the following steps:

Successfully register/link the device
Do not set up any security factors
Send an auth request to the device
The Whitelabel fragment will pop up indicating that the user cannot authorize the request until they set up a security factor.
Tap on the link "tap here". This takes you to a screen to set up a factor.
Set up a pin code and hit the checkmark

This returns you to the request screen that does not acknowledge you have set up any factors. Hitting the "tap here" link again and going back does not update the request screen. Setting more than one security factor does not update the request screen.

Thanks for the feedback, we'll make sure we tackle this as well. I'll keep this issue up to date.

This has been fixed as of v2.3.0, released today. It will now automatically refresh of there has been a change in the factors set.


fpga_plugin: orchestration/orchestrated fixed in READMEs

Not touching "orchestration programmed". Fixing only instances where
this refers directly to the mode recognized by the webhook-deploy.sh
script.
Signed-off-by: Antti Kervinen<EMAIL_ADDRESS>

@mythi, small fix to a doc issue found in fpga kustomization.


Can't execute openocd using the flake

When I try to use the openocd bundled in the esp32-idf flake, I get the following error during execution
/nix/store/6zyhvqamxp5j6fqy2nx1m879klxninq7-openocd-esp32-esp-idf-v5.1/bin/openocd-unwrapped: error while loading shared libraries: libudev.so.1: cannot open shared object file: No such file or directory

Looking with ldd I can see that it has multiple unmet dependencies.
ldd /nix/store/6zyhvqamxp5j6fqy2nx1m879klxninq7-openocd-esp32-esp-idf-v5.1/bin/openocd-unwrapped
	linux-vdso.so.1 (0x00007ffff7fc8000)
	libudev.so.1 => not found
	libusb-1.0.so.0 => not found
	libm.so.6 => /nix/store/3n58xw4373jp0ljirf06d8077j15pc4j-glibc-2.37-8/lib/libm.so.6 (0x00007ffff7ee2000)
	libz.so.1 => not found
	libutil.so.1 => /nix/store/3n58xw4373jp0ljirf06d8077j15pc4j-glibc-2.37-8/lib/libutil.so.1 (0x00007ffff7edd000)
	libdl.so.2 => /nix/store/3n58xw4373jp0ljirf06d8077j15pc4j-glibc-2.37-8/lib/libdl.so.2 (0x00007ffff7ed6000)
	libc.so.6 => /nix/store/3n58xw4373jp0ljirf06d8077j15pc4j-glibc-2.37-8/lib/libc.so.6 (0x00007ffff7cf0000)
	/lib64/ld-linux-x86-64.so.2 => /nix/store/3n58xw4373jp0ljirf06d8077j15pc4j-glibc-2.37-8/lib64/ld-linux-x86-64.so.2 (0x00007ffff7fca000)

I'm totally new to nix, so I'm a bit baffled what the right way forward would be
I've started the flake with
nix --experimental-features 'nix-command flakes' develop github:mirrexagon/nixpkgs-esp-dev#esp32s3-idf

It seems that even though those dependencies are specificied in the targetPkgs for the FHSUserEnv used to run those tools, they don't get picked up. I'm not sure why they don't get picked up.

This got patched by #44.

Whoops, I missed closing this, thanks.

Thanks ^^


Investigate the "+30s" and "-10s" buttons not always triggering in Firefox

See #208, tour 3, step 5. In Firefox, these buttons don't always work, plainly

The problem here is that, when audio is playing, Firefox disregards the changes to the currentTime, overwriting them with the time that is playing currently. I don't really know how to set about fixing this


MenuItemManager now inherits from MPTT TreeManager

Fixes #54 .  MenuItemManager's get_queryset method also now replicates the ordering behavior from TreeManager's get_queryset

This isn't really the same. TreeManager.get_queryset returns an instance of TreeQuerySet and this returns a MenuUnCacheQuerySet which will break chaining custom queryset methods. The functionality need to either be combined or we need a better way to do the cache invalidation handled by  MenuUnCacheQuerySet.

Maybe combine these via
class MenuUnCacheQuerySet(TreeQuerySet):
...

MenuItemManager = TreeManager.from_queryset(MenuUnCacheQuerySet)()

based on https://docs.djangoproject.com/en/1.8/topics/db/managers/#from-queryset

That change works.  Have not found any evidence that we need to handle cache invalidation for queryset .delete or .update, however.  At least not in treenav.  It does look like there is at least one place where MPTT uses .update and one or two where it uses .delete.

Coveralls appears to fail because I removed three lines that had test coverage.

Weird. I like less code.
This is, however, backwards incompatible and needs to be noted that we are removing the .tree manager in the off and unlikely chance that someone was using it for customization in their project.
Let's start/add some release notes and :shipit:


fix for [BUG] wrong workflow summary for async scans #133

[BUG] wrong workflow summary for async scans #133
By submitting a PR to this repository, you agree to the terms within the Checkmarx Code of Conduct. Please see the contributing guidelines for how to create and submit a high-quality PR for this repo.
Description

Describe the purpose of this PR along with any background information and the impacts of the proposed change.

References

Include supporting link to GitHub Issue/PR number

Testing

Describe how this change was tested. Be specific about anything not tested and reasons why. If this solution has unit and/or integration testing, tests should be added for new functionality and existing tests should complete without errors.
Please include any manual steps for testing end-to-end or functionality not covered by unit/integration tests.

Checklist

[ ] I have added documentation for new/changed functionality in this PR (if applicable).
[ ] All active GitHub checks for tests, formatting, and security are passing
[ ] The correct base branch is being used

Thank you for your valuable contribution to this pull request. We appreciate your effort in addressing the outlined issue.
Currently, our team is diligently investigating and rigorously testing the proposed solution to ensure its compatibility, reliability, and performance within our codebase. This process is a crucial step in maintaining the overall quality of our software.
We kindly ask for your patience during this testing phase. Your input is crucial to the success of our project, and we are excited to collaborate with you to bring this enhancement to fruition. Rest assured, we will provide timely updates on the progress and outcome of our testing efforts.
Once again, thank you for your dedication to our project and for being an integral part of our development community.
Best regards,
Igor Lomba.

Hi @igorlombacx, do you have any update?

Hi @ArnaudLacroix
We reviewed it internally, and we decide to change inside our ast-cli. We will create the markdown report with a message that the scan is still running, that way you can have a link back to the scan in the scan summary.
Thanks.


Inspect TM Scopes is not designed for big code.

os: sierra
vscode: latest and greatest!
Take this code. When you try to inspect the tokenizer's scopes you'll end up with this:

I'm not sure if it's even worthy of getting investigated but well a simple box sizing can fix it I think

Verified, it might look better using the ellipsis character '…' over '...' though

@alexandrudima @Tyriar thanks a lot!


2024-12-18_05-47-34_master-next_amazon-kvs-webrtc-sdk

Automatically created.

Backport failed for kirkstone-next, because it was unable to cherry-pick the commit(s).
Please cherry-pick the changes locally and resolve any conflicts.
git fetch origin kirkstone-next
git worktree add -d .worktree/backport-10854-to-kirkstone-next origin/kirkstone-next
cd .worktree/backport-10854-to-kirkstone-next
git switch --create backport-10854-to-kirkstone-next
git cherry-pick -x 9feb5a856cc4348ed5dbd413ef0ca4a126d2f02c

Successfully created backport PR for scarthgap-next:

#10862

Successfully created backport PR for styhead-next:

#10863


Issue in running the train.py file

Hi @Grzego,
I am trying to run the train.py (python train.py) file on the same IAM Online handwriting dataset. But getting some error (using tf 1.15)

Thank you for your help!!

I found the issue is coming through this line L174 of the train.py file.
When I remove the initial_state argument and added the dtype then the epochs started running but after the training, the trained model is not performing.

Hi @Grzego,
I hope you are doing well!
I figure out the issue and resolved it (using TF 1.15.2 environment). The error on the above screenshot shows the shape of the window variable that is coming from the WindowLayer line L54 in the train.py file.
Actually, the tf.matmul function gives the different format of shaped output for different batch_size.
For (64, 1, None) * (None, None, 80) gives (64, 1, 80)
For (1, 1, None) * (None, None, 80) gives (None, 1, 80) it should be (1, 1, 80)
Here, 64 and 1 are the batch_size.
So, what I did is changed the shape of output after tf.matmul to (1, 1, 80) if it is (None, 1, 80)
window = tf.matmul(phi, self.sequence)
if window.get_shape().as_list() == [None, 1, 80]:
    window = tf.reshape(window, (1, 1, 80))

This resolved the issue, I faced in training the model. However thank you for this repository, I got many things to learn from it.


Export config - groups

TL;DR
Allow for multi-period export config to be explicit in the server.
For example, daily exports combined with 4 hour exports.
All of a clean mapping for a device to understand which export files to process to get complete coverage in the event of being offline for multiple days.
Currently, the understanding of the index.txt is rather opaque.
/kind enhancement
/assign

I can hold onto this for time being. We may not need "catch up" files after all if we get the right sort of flexibility into the api.

keep alive

After v1.5 we won't need the notion of catch-up files as the api can be called with multiple whole batches, so we probably don't need to support more complexity in grouping generated files by frequency. If servers find they need to do this for pre-v1.5 devices, some simple prefix conventions might be sufficient to distinguish.


Put manual injection on top.

Got several errors from customer when they play with bookinfo example. Turns out they did not install initializer, but try to use auto injection feature when deploying bookinfo.
So I moved the auto injection option down. If a customer installed initializer, either way works; if he/she does not, the first option (manual sidecar inject) will work as well.

Jenkins job istio.github.io/presubmit passed

Jenkins job istio.github.io/presubmit passed


can't install any extension directly and manually

Type: Feature Request
installing extensions
VS Code version: Code 1.95.3 (f1a4fb101478ce6ec82fe9627c43efbf9e98c813, 2024-11-13T14:50:04.152Z)
OS version: Windows_NT x64 10.0.19045
Modes:

Thanks for reaching VS Marketplace, please share error message or screenshot to investigate better.
-VS Marketplace Team

Closing the issue since there has been no updates on the information requested. If the issue still persists, please feel free to reopen or create a new issue.
Regards,
VS Marketplace Team


ResolveAll not throwing exception

This maybe a clarification of expected behavior, but I have a scenario where I have a registration done with a name-key. When I Resolve(key) I get an exception; this is expected because config that ctor depends on is bad. When I ResolveAll I get a T[0] and no exception.
There is only that single registration done so I expect 1 when ResolveAll works or an exception when ResolveAll fails.
In short this created an exception-less fail state when my app starts up.

Any feedback on this behavior and potential change?

This behavior is by design.

Is there a way to get these failures to come up the stack?


Proxy server default configuration

Is your feature request related to a problem? Please describe.
I use the postwoman-proxy to create a private proxy server, and I want to use the server url become the default configutarion. There are many files that need to be changed, may be create a common config.json to export the DEFAULT_PROXY_URL?

@NBTX

Also met the CORS problem.I have changed the config.json to fit my private postwoman.
#365

I think Access-Control-Allow-Origin is only set to allow requests from https://postwoman.io.

yep, the default config is that,but I have changed it to my server address.

You only need to make sure that the page you're accessing postwoman from is included in the allowedOrigins list. If you use the Postwoman app from postwoman.io with your proxy no additional configuration is required.

I will test it today and paste the result below.


Create .devcontainer/devcontainer.json

There is not enough context in this PR or what the purpose of this work is. We don't have any container related content in VCC at this time.


Fix Selnium test

There exists 2 rendered templates:

jinja2/basic.jinja
base.html

Sounds good, I will add that.
OTOH, if you already have the work ready, feel free to throw up a PR and I'll close this one.

Closing this PR in favor of #1343.


Semantic information for Database.denotations

Follow up of #822. Currently it's possible to resolve a few names in Database.sugars but not in Database.denotations. It would be nice if for example it was possible to attribute references in Denotation.info

One thing that I'd like to implement here is making Denotation.info strings parseable. For that, we'll need to introduce Type.Method that can express dependent method types. This is going to be a new AST node that will need a checkParent check that makes sure that it doesn't appear in normal programs.

Related https://github.com/scalacenter/scalafix/issues/254 scalafix ExplicitReturnTypes uses Denotation.info to insert the explicit types, in some cases the types are not fully qualified causing the rewrite to insert false names (e.g., Long when java.lang.Long should be inserted)


Remove the vertical skip in solutions that start with a displaystyle

Because of the vertical skip, starting a solution with a display style math doesn't look well. Once resolved, try to remove first paragraphs in appendix A, if they aren't necessary.

Also, make the vertical skips above and below display styles equal height and better aligned with the ones in the book.

Consider setting \abovedisplayskip, \belowdisplayskip, \abovedisplayshortskip, \belowdisplayshortskip lengths to 0pt or something else, either globally in the preamble, or locally for specific equations.


feature: add no-trunc flag for 'pouch ps'

Signed-off-by: Zou Rui<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
This pr adds no-trunc flag for 'pouch ps' command.
Ⅱ. Does this pull request fix one issue?

fixes #903
Ⅲ. Describe how you did it
add flag
Ⅳ. Describe how to verify it
[root@izj6c44nzox6sb97nzgmhyz ~]# pouch ps --no-trunc
Name   ID                                                                 Status        Created        Image                            Runtime
foo2   692c77587b38f60bbd91d986ec3703848d72aea5030e320d4988eb02aa3f9d48   Up 1 minute   1 minute ago   docker.io/library/redis:alpine   runc
foo<PHONE_NUMBER>6405ee64788bd108ef1de3d24dc3add73725891f4787d0f8e036f5   Up 1 minute   1 minute ago   docker.io/library/redis:alpine   runc
[root@izj6c44nzox6sb97nzgmhyz ~]# pouch ps --no-trunc -q
692c77587b38f60bbd91d986ec3703848d72aea5030e320d4988eb02aa3f9d48
18592900006405ee64788bd108ef1de3d24dc3add73725891f4787d0f8e036f5
[root@izj6c44nzox6sb97nzgmhyz ~]# pouch ps --no-trunc -a
Name   ID                                                                 Status         Created         Image                            Runtime
foo3   63fd6371f3d614bb1ecad2780972d5975ca1ab534ec280c5f7d8f4c7b2e9989d   created        2 minutes ago   docker.io/library/redis:alpine   runc
foo2   692c77587b38f60bbd91d986ec3703848d72aea5030e320d4988eb02aa3f9d48   Up 2 minutes   2 minutes ago   docker.io/library/redis:alpine   runc
foo<PHONE_NUMBER>6405ee64788bd108ef1de3d24dc3add73725891f4787d0f8e036f5   Up 2 minutes   2 minutes ago   docker.io/library/redis:alpine   runc

Ⅴ. Special notes for reviews

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@9e3e901). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master     #909   +/-   ##
=========================================
  Coverage          ?   13.66%
=========================================
  Files             ?      121
  Lines             ?     7669
  Branches          ?        0
=========================================
  Hits              ?     1048
  Misses            ?     6531
  Partials          ?       90

Impacted Files
Coverage Δ

cli/ps.go
0% <0%> (ø)

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 9e3e901...6be54b8. Read the comment docs.

@ZouRui89 Thanks for your work, we also should add some cli test case for the --no-trunc flag

PTAL. @HusterWan

LGTM


Screenshot does not appear in order on

You said in your description that you listed it from cheap to expensive, but the situation is different in the image :)
Thank you for your effort, it was a successful work!...

Thanks. I updated the screenshot. I think it will appear in few minutes :P


Should we support specific version installs of Steampipe via brew?

For example, after the release of v0.17.0, should you still be able to do something like this?
brew install<EMAIL_ADDRESS>

94fc00c6b4ace5859dc321bde5ca1a384b14ec7c

Completed https://github.com/turbot/steampipe/pull/2759. Now users will be able to install specific versions of steampipe using brew.


Use the Debug Console

Currently, the extension launches a terminal process whenever a launch configuratin gets launched.
However, a better option would be to let the launched process be visible - and allow to interact with - in the Debug Console REPL, similar to how Rebornix's Ruby Extension does.

You can interact with debug console by choosing a tab.
The disadvantage of using debug console primary is, we can not input to STDIN.
Do you want to make an option to use only debug console (and give up STDIN)?

+1 for this! I would give up STDIN like a shot if I could have my terminal back :)

Let me clear.
Now it supports "debug console" by choosing the tab.
Do you want to request further?

What I want is the output from rdbg to go to the debug console no matter which tab is selected. I want the terminal to always remain untouched. Personally I'm not fussed about STDIN in the debug console.

could you clear what is "output from rdbg"?

I mean the launching of the rdbg command and its output. I want that in the debug console instead of the terminal :)

Do you think you will be able to stop rdbg taking over the terminal? It's really annoying!

Ah, so you want:

Do not show terminal (rdbg invocation and messages).

All output should be printed on "OUTPUT" tab.

And you don't want to use STDIN anymore for debuggee.

Does it correct?
Again, I'm worry about (3).

I'm personally not bothered about 3 at all but it sounds like it could be possible anyway based on what the original poster said. There might be some code to look at in the link he posted.

Yes, introducing other options is possible.

Isn't this now supported in 0.1.0? By default vscode-rdbg now sends output to the debug console and you need to specify useTerminal: true in the launch.json to send output to a terminal. Or have I misunderstood what the enhancement request is for?

I've just tried the launch option again and it no longer steals the terminal so my problem is solved. Thank you very much :)

Great! Sorry I forget to tell you.


Suggestion for README & FAQ

A few suggestions that might improve readability / organization.  Consider:

[ ] Adding a (linked) table of contents to the README.md
[ ] Breaking the FAQ out into a separate FAQ.md doc in the repo, and just linking it from the README?

Damn I'm just now seeing this (a year later). These are good suggestions @cboettig and they've been implemented in the last month (more or less). More conceptual stuff is now in the site wiki, and  it's broken down by page rather than being in one mega-doc. Please keep adding suggestions etc in the future, I think this repo will be checked / used more frequently from here on.


Manually create waypoints with helm and k8s Gateway API

Describe the feature request
Docs claim that you can create manually a gateway. However, it only works for ingress gateway and not waypoints.
How to reproduce:
istioctl install -y --set profile=ambient --set meshConfig.accessLogFile=/dev/stdout --set components.ztunnel.values.logLevel=debug

kubectl get crd gateways.gateway.networking.k8s.io &> /dev/null || \
  { kubectl apply -f https://github.com/kubernetes-sigs/gateway-api/releases/download/v1.1.0/standard-install.yaml\; }

kubectl create ns httpbin

kubectl label namespace httpbin istio.io/dataplane-mode=ambient

kubectl -n httpbin apply -f https://raw.githubusercontent.com/solo-io/gloo-mesh-use-cases/main/policy-demo/httpbin.yaml
kubectl -n httpbin apply -f https://raw.githubusercontent.com/istio/istio/refs/heads/master/samples/sleep/sleep.yaml

kubectl label svc/httpbin -n httpbin  istio.io/use-waypoint=httpbin-waypoint

helm upgrade --install httpbin-waypoint istio/gateway -n httpbin -f - <<EOF
labels:
  istio.io/waypoint-for: all
podAnnotations:
  inject.istio.io/templates: "waypoint"
service:
  type: ClusterIP
  ports:
  - appProtocol: tcp
    name: status-port
    port: 15021
    protocol: TCP
    targetPort: 15021
  - appProtocol: hbone
    name: mesh
    port: 15008
    protocol: TCP
    targetPort: 15008
EOF

kubectl apply -f - <<EOF
apiVersion: gateway.networking.k8s.io/v1beta1
kind: Gateway
metadata:
  name: httpbin-waypoint
  namespace: httpbin
  labels:
    istio.io/waypoint-for: all
    istio.io/for-service-account: httpbin-waypoint # Trying this to see if solves the issue with the service account, but it seems that not
  annotations:
    istio.io/for-service-account: httpbin-waypoint # Trying this to see if solves the issue with the service account, but it seems that not
spec:
  addresses:
  - value: httpbin-waypoint.httpbin.svc.cluster.local
    type: Hostname
  gatewayClassName: istio-waypoint
  listeners:
  - name: mesh
    port: 15008
    protocol: HBONE
EOF

The ReplicaSet for the httpbin-waypoint throws an error about not able to inject the ServiceAccount:
│ Events:                                                                                │
│   Type     Reason        Age                   From                   Message                                                                                │
│   ----     ------        ----                  ----                   -------                                                                                │
│   Warning  FailedCreate  7m27s (x18 over 18m)  replicaset-controller  Error creating: admission webhook "object.sidecar-injector.istio.io" denied the request: failed to run injection template: template: inject:4:10: executing "injec │
│ t" at <.ServiceAccount>: can't evaluate field ServiceAccount in type *inject.SidecarTemplateData

And istiod throws another error about the same: not able to inject the ServiceAccount in the template:
error	Invalid template: template: inject:4:10: executing "inject" at <.ServiceAccount>: can't evaluate field ServiceAccount in type *inject.SidecarTemplateData

Describe alternatives you've considered
The alternative is to create the Waypoint applying directly the resources which is not the best approach since the template injection is the way Istio suggests as good practice.
Affected product area (please put an X in all that apply)
[ x ] Ambient
[ x ] Docs
[ ] Dual Stack
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane
Additional context

Hi @antonioberben can you explain why you need to manually create a waypoint? The doc you referenced only worked for ingress gateway and not intended for waypoint.

@linsun, I tried to manually deploy a waypoint using inject.istio.io/templates: "waypoint" too with no success.
The reason for this is to be able to customize resources. I see .Values.global.waypoint.resources available for this, but it is global.
My intention is to have one global waypoint that will be shared across many dormant "infrastructure" applications, one waypoint per "environment" namespace, and one waypoint per application with high traffic. But with .Values.global.waypoint.resources I cannot fine-tune waypoints.

having to use istioctl to do stuff is a non-starter for many people.  We absolutely need a declarative way to do this.

@johnthompson-ybor, this is slightly orthogonal I think. Istioctl isn't required to create a waypoint per se. The cli has some things which are convenient (like ability to label a namespace for using the waypoint) but in the end all it really does is create a gtwapi gateway resource which is pretty simple.
The issue being discussed here is primarily about some ability to manually create the deployment/service which implements that gateway, this part is a little trickier to manage though because the deployment in particular is quite different from the one you get by using helm to deploy an ingress gateway. Right now istiod has a controller which creates the implementing resources in response to the gtwapi gateway and that is basically the reasonable way to get a waypoint. Whether that gtwapi resource is created with istioctl, kubectl or something else isn't strictly germane.


Harmonise module name style and add link to cited papers and repo

looks good to me --- thank you!


Login modal design is broken and not closable at small screen.

Describe the bug
At small screen devices no way to close the login modal and design is also broken.
To Reproduce
Steps to reproduce the behavior:

You should not be logged in if you are logged in log out.
Go to any Challenge details page.
Open this page on mobile device which has resolution below 670px or resize your browser window below 670px.
Click on Download button.
See modal which is not closable on small devices and design is broken.

Expected behavior
The modal should be closable in small devices and design should be fixed show modal in the middle.
Screenshots
At mobile screen.

At medium tablet screen

Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome, Edge
Version: 102.0, 102.0.1245.41 (Edge)

Smartphone (please complete the following information):

Device: Redmi note 7s
OS: MIUI 12
Browser: Chrome
Version: 101.0

Additional context

Instead of relative positioning we should add fixed position so modal. we could center align modal using below css. we could achieve same css with tailwind utility classes.

position: fixed;
top: 50%;
left: 50%;
transform: translate(-50%, -50%);

Hello Paras,
Thanks for reporting this issue.
Would you like to work on this issue?

Yeah would love to help..


[in_app_purchase] fix README.md in plugins/packages/in_app_purchase/in_app_purchase_ios

issues fixed by this PR

in_app_purchase Mistake in formatting of README in in_app_purchase_ios package #87187

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Note that unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[ ] I signed the CLA.
[ ] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[ ] I updated CHANGELOG.md to add a description of the change.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@googlebot I signed it!

@googlebot I fixed it.

Thanks for the contribution! I didn't see this PR and I fixed the link in https://github.com/flutter/plugins/pull/4352.
It still needs to be fixed for the Android one if you're willing to send another PR.  🙂 https://github.com/flutter/plugins/blob/master/packages/in_app_purchase/in_app_purchase_android/README.md
Closing.


README update for typo fixes

Corrected one typo on the link,
Corrected descriptive word of the link.

Thanks @ofcyln!


fix: bump zwave-js and zwave-js-server

Fixes issues with the hybrid ESM/CJS packages that caused the bundle size to double and broke some CC-related logic in the driver.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 21.069%

Totals

Change from base Build<PHONE_NUMBER>3:
0.0%

Covered Lines:
3946

Relevant Lines:
19918

💛  - Coveralls


[BUG] more than 1 filter do not work well sometimes

Was a very dumb problem.
If the first condition was true, does not check the next... and that behaviour with all the iteration


Q: How to use tengo in a gameloop where Update() is called 60 times a second?

Hi, I'm trying to work out if my gameloop use-case is possible with Tengo and would like some advice.
I need to call into tengo code 60 times a second, persisting any state between calls. Kind of like a very fast REPL.
From what I've seen of Tengo you generally call Run() and let it run to completion. The REPL seems to be the exception, but it does a lot of work to keep state between lines (I don't need to recompile each line, I call the same function over and over again but need the state to persist between calls). Or perhaps there's a way to send messages/signals into an always-running tengo script, so I can trigger it to act at the appropriate time?
Is this gameloop use-case one that can work with Tengo? If so I'd appreciate any advice on the right approach. Thanks.

Maybe you want to look at Compiled. You can compile the script code once (Script.Compile) and use Compiled instance to run it multiple times. Between runs, you can access global variables using its Get and Set methods.
If that doesn't work for your use case, you can consider managing Compiler and VM instances directly. See how Script and Compiled does that.

I think for what I need the game logic script would have to run 'forever' and I'd have to sync it to the framerate with a blocking call into go code. Something like...
engine := import("engine")

someUpdateFunction := func() {
  // move some sprites around
  // sundry per-frame game logic
}

for {
   engine.WaitForNextVerticalBlank()
   someUpdateFunction()
}

@d5 Rather than ask abstract questions I've written a skeleton for a game loop. If you've got time could you tell me if you see any obvious problem with using Tengo this way? ('this way' being running the tengo script infinitely and blocking on the next frame. Also the way I've used tengo modules to simulate classes of sorts, is this poor form for tengo?)

package main

import (
	"context"
	"fmt"
	"image/color"
	"log"

	"github.com/d5/tengo/v2"
	"github.com/d5/tengo/v2/stdlib"

	"github.com/hajimehoshi/ebiten/v2"
	"github.com/hajimehoshi/ebiten/v2/ebitenutil"
)

var (
	tengoDoneWithFrameCh = make(chan int)
	tengoStartFrameCh    = make(chan int)
	frameNum             = 0
	playerModuleCode     = []byte(`
engine := import("engine")
rand := import("rand")
newVec2 := import("vec2")

move := func(self, spriteIdx) {
	return func() {
		self.pos.add(self.delta)
		if self.pos.x < 0 || self.pos.x > 320 {
			self.delta.x=-self.delta.x
		}
		if self.pos.y < 0 || self.pos.y > 240 {
			self.delta.y=-self.delta.y
		}
		engine.sprites[spriteIdx].move(self.pos.x, self.pos.y)
	}
}

export func(spriteIdx, x,y) {
	self := {
		pos: newVec2(x,y),
		delta: newVec2(5,5)
	}
	self.move = move(self, spriteIdx)
	return self
}
`)
	vec2ModuleCode = []byte(`
add := func(self) {
	return func(otherVec2) {
		self.x += otherVec2.x
		self.y += otherVec2.y
	}
}

export func(x,y) {
	vec2 := {
		x: float(x),
		y: float(y)
	}
	vec2.add = add(vec2)
	return vec2
}
`)
	script = []byte(
		`
fmt := import("fmt")
rand := import("rand")
engine := import("engine")

newPlayer := import("player")

p1 := newPlayer(0, rand.float()*320, rand.float()*240)
p2 := newPlayer(1, rand.float()*320, rand.float()*240)

for {
	p1.move()
	p2.move()
	fmt.println("frame", " ", p1, " ", p2)
	engine.WaitForNextFrame()
}

`)
)

//// Sprite

type Sprite struct {
	X, Y     float64
	Image    *ebiten.Image
	DrawOpts ebiten.DrawImageOptions
}

func NewSprite() *Sprite {
	s := &Sprite{}
	s.setup()
	return s
}

func (s *Sprite) setup() *Sprite {
	s.Image = ebiten.NewImage(32, 32)
	s.Image.Fill(color.White)
	return s
}

func (s *Sprite) Draw(screen *ebiten.Image) {
	s.DrawOpts.GeoM.Reset()
	s.DrawOpts.GeoM.Translate(-16, -16)
	s.DrawOpts.GeoM.Translate(s.X, s.Y)
	screen.DrawImage(s.Image, &s.DrawOpts)
}

func (s *Sprite) GetTengoObject() tengo.Object {
	return &tengo.Map{
		Value: map[string]tengo.Object{
			"move": &tengo.UserFunction{Name: "move", Value: s.move},
		},
	}
}

func (s *Sprite) move(args ...tengo.Object) (ret tengo.Object, err error) {
	if len(args) != 2 {
		return nil, tengo.ErrWrongNumArguments
	}

	var ok bool

	if s.X, ok = tengo.ToFloat64(args[0]); !ok {
		return nil, tengo.ErrInvalidArgumentType{Name: "x", Expected: "float64", Found: args[0].TypeName()}
	}

	if s.Y, ok = tengo.ToFloat64(args[1]); !ok {
		return nil, tengo.ErrInvalidArgumentType{Name: "y", Expected: "float64", Found: args[0].TypeName()}
	}

	return tengo.UndefinedValue, nil
}

//// Game

type Game struct {
	Sprites       []*Sprite
	contextCancel func()
}

func (g *Game) Update() error {
	tengoStartFrameCh <- frameNum // tell tengo to start a new frame
	<-tengoDoneWithFrameCh        // wait for tengo to finish that frame
	return nil
}

func (g *Game) Draw(screen *ebiten.Image) {
	for _, sprite := range g.Sprites {
		sprite.Draw(screen)
	}
	ebitenutil.DebugPrint(screen, fmt.Sprintf("[ %d ]", frameNum))
}

func (g *Game) Layout(outsideWidth, outsideHeight int) (screenWidth, screenHeight int) {
	return 320, 240
}

func (g *Game) waitForNextFrame(args ...tengo.Object) (ret tengo.Object, err error) {
	if frameNum > 0 { // otherwise deadlock. someone has to go first.
		tengoDoneWithFrameCh <- frameNum // inform outer that we're done with the previous frame
	}
	frameNum++
	fmt.Println(frameNum)
	if frameNum == 2000 {
		g.contextCancel()
	}
	<-tengoStartFrameCh // wait for the new frame before letting tengo continue
	return tengo.UndefinedValue, nil
}

//// main

func main() {
	ctx, cancel := context.WithCancel(context.Background())

	game := &Game{
		Sprites: []*Sprite{
			NewSprite(),
			NewSprite(),
		},
		contextCancel: cancel,
	}

	engineModule := &tengo.BuiltinModule{
		Attrs: map[string]tengo.Object{
			"WaitForNextFrame": &tengo.UserFunction{
				Name:  "WaitForNextFrame",
				Value: game.waitForNextFrame,
			},
			"sprites": &tengo.Array{
				Value: []tengo.Object{
					game.Sprites[0].GetTengoObject(),
					game.Sprites[1].GetTengoObject(),
				},
			},
		},
	}

	script := tengo.NewScript(script)

	moduleMap := stdlib.GetModuleMap(stdlib.AllModuleNames()...)
	moduleMap.AddSourceModule("player", playerModuleCode)
	moduleMap.AddSourceModule("vec2", vec2ModuleCode)
	moduleMap.Add("engine", engineModule)
	script.SetImports(moduleMap)

	go func() { // run the script in its own goroutine
		compiled, err := script.RunContext(ctx)
		if err != nil {
			panic(err)
		}
		fmt.Println(compiled.GetAll())
	}()

	ebiten.SetWindowSize(640, 480)
	ebiten.SetWindowTitle("Hello, World!")

	if err := ebiten.RunGame(game); err != nil {
		log.Fatal(err)
	}
}

I don't know about the module function implementations, but structure-wise, they look good to me. Once the script is compiled into bytecode, it's just a VM code loop that runs the bytecode in the same Go runtime.
Btw, I used to be a game-dev myself for a while, and this is really interesting.


Revert "Add resource requests/limits"

Reverts almighty/keycloak#60

@alexeykazakov what did it happen with this PR ?

it was a problem with the cpu, right ?

Correct.


Add the installaiton docs on README at usage part on  on the docs part

Install the package:

unique_and_permanent_device_identifier:
git:
url: https://github.com/enesakbal/Device-Identifier-Manager.git
ref: main

Hi @feyroozecode
Thanks for your attention. I recently added installation to readme.


Beta

This PR includes:

improvements to type-safety and structure for url-loader
update monorepo infrastructure (prettier, eslint, add tsconfig paths)
apply updated formatting and lint rules
rewrite createCloudinaryUrl types to remove zod dependency in build output

Description

Issue Ticket Number
Fixes #<ISSUE_NUMBER>

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Fix or improve the documentation
[ ] This change requires a documentation update

Checklist

[ ] I have followed the contributing guidelines of this project as mentioned in CONTRIBUTING.md
[ ] I have created an issue ticket for this PR
[ ] I have checked to ensure there aren't other open Pull Requests for the same update/change?
[ ] I have performed a self-review of my own code
[ ] I have run tests locally to ensure they all pass
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes needed to the documentation

Tried upgrading to the beta version in the NExt.js project and got this error. unsure if this is a correction that is required in Next.js or if we're missing something from the restructure


Update Parcel to a stable release

Parcel has had its first release candidate released; this includes changes such as changing the JavaScript compiler (actually in beta 3).
After an initial test, the project as-in doesn't build.

Usage of the ORM currently has a lot of circular dependencies (where two entities depend on each other). The JS changes in Parcel seem to fail when building: Cannot access 'Community' before initialization

https://github.com/parcel-bundler/parcel/pull/6694 might help.

Parcel 2.0.0 has now been released, but the app still doesn't build.

I've managed to reproduce the Parcel problem and open a PR: https://github.com/parcel-bundler/parcel/pull/7293.

In order to move this forward, we should remove the decorators and use EntitySchema instead.
https://github.com/PREreview/prereview/issues/398#issuecomment-915031796 needs to be addressed though, as there is some code reliant on the use of decorators.

I'm currently struggling to get EntitySchema to work with Parcel's scope hoisting as classes are renamed, whereas the ORM relies on them being unchanged (so they're different at runtime). This wasn't a problem with the decorators as they were processed by TSC before Parcel would try to change them(?). Options are probably either to disable Parcel's scope hoisting/optimisation or define them explicitly.

define them explicitly

This didn't work, the types seemingly allow for both a class and a name to be set, but the latter is ignored and the name of the class itself continues to be used.
I also looked at using types rather than classes, which works but you have to declare types everywhere (since you're just referring to them by a string).
Feels like the most pragmatic approach now is to just stop Parcel from mangling the names.

hey gordon shumway :D. (nice avatar)
i saw your issues at parcel to use parcel v2.
I am using typeorm with sqlite in an electron app.
everything worked with parcel v2 beta3.1 but not with the final version.
I got it working by:

deactivating scopeHoisting and optimizations.
to avoid circular deps in entities - import you entities as with "type" and use relations as strings, e.g.

import type { person } from '../entities/person'

@Entity({ name: 'family' })
export class Family {
  @PrimaryGeneratedColumn()
  id: number

  // One family holds multiple patients
  @OneToMany('Person', 'family', { eager: true })
  patients: Person[]

  @CreateDateColumn()
  createdAt: Date

  @UpdateDateColumn()
  updatedAt: Date
}

PS: in the beta i needed to set a specific entity name, because parcels scopeHosting transformed the name of the class.


Combine all EAN barcodes...

Hi Guys, I'm asking if is possible to combine all the EAN codes together in search. Basically, I'd like to scan EAN_13, EAN_8 (reader should figures out if is 13 or 8). (one barcode at a time). Is this possible with quagga ?
Thank you.

Hey @quantaxproj ! i am working on this from last two month's and yes you can use ean_reader. but the chances of getting false positives is more. and set half-sample to false. i think this will work.

Thanks @lakshman2891 .
Did you mean this configuration? Because in this way it recnognizes only EAN_13 not EAN_8

Hey @quantaxproj , i am talking about scanning EAN-13 bar code type. when scanning EAN-13, there is a chances of reading it wrongly, like instead of 13 characters- it will read 8 characters.

@quantaxproj sorry for sound! are you talking about printing the barcode type in the console or webpage??

@Iakshman2891 My goal is to create a web app that uses quagga. The user launch quagga, without select ean type..., I need only ean13 and ean8, quagga should recognize the ean type and print resulta in webpage. The user scans one barcode at a time, and only ean13 or ean8 types. The goal is make this without use select optino box Ean (for 13) or ean_8 (for 8)...

@quantaxproj i understand now. if you use ean_8_reader and ean_reader in your decoder config file. this will work. but recognizing the ean_13 bar code types provide lot of false positives.


Enable docker config provider if docker.sock exists

What does this PR do?
enable both the docker and the kubelet config provider if the agent runs ok k8s and we can find the docker socket.
Motivation
Some users still configure autodiscovery with docker labels directly in the image, so we need to support both annotations and labels in k8s.
Additional Notes
I made it a new file after trying to optionally patch the existing one, which ended up being too brittle and would break at the next formatting change. Happy to change that if someone has an idea to make it robust.

Codecov Report

Merging #2544 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2544   +/-   ##
=======================================
  Coverage   57.08%   57.08%
=======================================
  Files         350      350
  Lines       19388    19388
=======================================
  Hits        11067    11067
+ Misses       7491     7490    -1
- Partials      830      831    +1

Impacted Files
Coverage Δ

pkg/logs/input/listener/tailer.go
90.47% <0%> (-4.77%)
:arrow_down:

pkg/forwarder/domain_forwarder.go
92.3% <0%> (+1.92%)
:arrow_up:

Hi, when would we able to merge this PR? It's blocking some logs related work. Thanks!

Thanks for the ping @tmichelet, merging it today.


Make size changing animation less funky

The current animation for changing size (e.g. "1234" to "12345") is a bit funky when the animation is slowed down. Investigate how to improve this behavior.

I believe that this is fixed by the latest levenshtein commit.