Config reader crash

Hi,
with the current version, when I copy&paste exactly the last config from "Need help configurating 3 Riing Plus 12 fans", having the same ports (1-3 used for pump and 2 fans, 4 and 5 free) and adapting to my hardware (intelcpu instead of amdcpu)
{
 "Profiles": [
    {
      "Name": "Default",
      "Guid": "458ad61c-e033-4c84-b27d-b57492716450",
      "Ports": [
        [9802, 8101, 1],
        [9802, 8101, 2],
        [9802, 8101, 3]
      ],
      "SpeedControllers": [
        {
          "Type": "PwmSpeedController",
          "Config": {
            "CurvePoints": [
              [15, 0],
              [30, 20],
              [40, 40],
              [50, 50],
              [60, 75],
              [65, 90],
              [70, 100]
            ],
            "Sensors": [
              "/intelcpu/0/temperature/0"
            ],
            "SensorMixFunction": "Maximum",
            "MinimumChange": 4,
            "MaximumChange": 8,
            "Trigger": {
              "AlwaysTrigger": {}
            }
          }
        }      ],
      "Effects": [
      ]
    }
  ],
  "ComputerStateProfiles": [
  ],
  "CriticalTemperature": {
    "/intelcpu/0/temperature/0": 70,
    "/intelcpu/0/temperature/1": 70,
    "/intelcpu/0/temperature/2": 70,
    "/intelcpu/0/temperature/3": 70,
    "/intelcpu/0/temperature/4": 70
  },
  "TemperatureTimerInterval": 250,
  "DeviceSpeedTimerInterval": 2500,
  "DeviceRgbTimerInterval": 1000
}

with the following hardware
Controllers
-------------------------------
Name: Riing Plus Controller
VendorId: 9802
ProductId: 8101
Ports:
        Id: 1
        Data: [PortId: 1, Speed: 35, Rpm: 0, Unknown: 255]
        Identifier: [9802, 8101, 1]

        Id: 2
        Data: [PortId: 2, Speed: 35, Rpm: 616, Unknown: 255]
        Identifier: [9802, 8101, 2]

        Id: 3
        Data: [PortId: 3, Speed: 35, Rpm: 620, Unknown: 255]
        Identifier: [9802, 8101, 3]

        Id: 4
        Data: [PortId: 4, Speed: 60, Rpm: 0, Unknown: 255]
        Identifier: [9802, 8101, 4]

        Id: 5
        Data: [PortId: 5, Speed: 60, Rpm: 0, Unknown: 255]
        Identifier: [9802, 8101, 5]

Available effect types:
Flow_Extreme, Flow_Fast, Flow_Normal, Flow_Slow, Spectrum_Extreme, Spectrum_Fast, Spectrum_Normal, Spectrum_Slow, Ripple_Extreme, Ripple_Fast, Ripple_Normal, Ripple_Slow, Blink_Extreme, Blink_Fast, Blink_Normal, Blink_Slow, Pulse_Extreme, Pulse_Fast, Pulse_Normal, Pulse_Slow, Wave_Extreme, Wave_Fast, Wave_Normal, Wave_Slow, ByLed, Full

Sensors
-------------------------------
Intel Core i7-4790K:
        /intelcpu/0/temperature/0:
                Name: CPU Core #1
                Value: 32
Intel Core i7-4790K:
        /intelcpu/0/temperature/1:
                Name: CPU Core #2
                Value: 34
Intel Core i7-4790K:
        /intelcpu/0/temperature/2:
                Name: CPU Core #3
                Value: 32
Intel Core i7-4790K:
        /intelcpu/0/temperature/3:
                Name: CPU Core #4
                Value: 32
Intel Core i7-4790K:
        /intelcpu/0/temperature/4:
                Name: CPU Package
                Value: 34
NVIDIA GeForce GTX 1080 Ti:
        /nvidiagpu/0/temperature/0:
                Name: GPU Core
                Value: 40
WDC WD20EFRX-68EUZN0:
        /hdd/0/temperature/0:
                Name: Temperature
                Value: 28
-------------------------------

the program crashes:
17:26:41.8214: ================================================================
17:26:41.8324: Initializing...
17:26:42.0394: Loading plugins...
17:26:42.0394: Loading assembly: TTController.Plugin.BlinkEffect [<IP_ADDRESS>]
17:26:42.0394: Loading assembly: TTController.Plugin.ByLedColorEffect [<IP_ADDRESS>]
17:26:42.0394: Loading assembly: TTController.Plugin.FlowEffect [<IP_ADDRESS>]
17:26:42.0394: Loading assembly: TTController.Plugin.FullColorEffect [<IP_ADDRESS>]
17:26:42.0394: Loading assembly: TTController.Plugin.PulseEffect [<IP_ADDRESS>]
17:26:42.0394: Loading assembly: TTController.Plugin.RawEffect [<IP_ADDRESS>]
17:26:42.0394: Loading assembly: TTController.Plugin.RippleEffect [<IP_ADDRESS>]
17:26:42.0394: Loading assembly: TTController.Plugin.SnakeEffect [<IP_ADDRESS>]
17:26:42.0394: Loading assembly: CSCore [<IP_ADDRESS>]
17:26:42.0394: Loading assembly: TTController.Plugin.SoundEffect [<IP_ADDRESS>]
17:26:42.0394: Loading assembly: TTController.Plugin.SpectrumEffect [<IP_ADDRESS>]
17:26:42.0394: Loading assembly: TTController.Plugin.TemperatureEffect [<IP_ADDRESS>]
17:26:42.0394: Loading assembly: TTController.Plugin.WaveEffect [<IP_ADDRESS>]
17:26:42.0394: Loading assembly: TTController.Plugin.PwmSpeedController [<IP_ADDRESS>]
17:26:42.0394: Loading assembly: TTController.Plugin.StaticSpeedController [<IP_ADDRESS>]
17:26:42.0394: Loading assembly: TTController.Plugin.AlwaysTrigger [<IP_ADDRESS>]
17:26:42.0394: Loading assembly: TTController.Plugin.LogicTrigger [<IP_ADDRESS>]
17:26:42.0394: Loading assembly: TTController.Plugin.OneTimeTrigger [<IP_ADDRESS>]
17:26:42.0394: Loading assembly: TTController.Plugin.ProcessTrigger [<IP_ADDRESS>]
17:26:42.0394: Loading assembly: TTController.Plugin.PulseTrigger [<IP_ADDRESS>]
17:26:42.0474: Creating DataCache...
17:26:42.0474: Creating Config Manager...
17:26:42.0474: Loading config...

The "AlwaysTrigger": {} line should be changed to "Type": "AlwaysTrigger"
So the full config would be:
{
 "Profiles": [
    {
      "Name": "Default",
      "Guid": "458ad61c-e033-4c84-b27d-b57492716450",
      "Ports": [
        [9802, 8101, 1],
        [9802, 8101, 2],
        [9802, 8101, 3]
      ],
      "SpeedControllers": [
        {
          "Type": "PwmSpeedController",
          "Config": {
            "CurvePoints": [
              [15, 0],
              [30, 20],
              [40, 40],
              [50, 50],
              [60, 75],
              [65, 90],
              [70, 100]
            ],
            "Sensors": [
              "/intelcpu/0/temperature/0"
            ],
            "SensorMixFunction": "Maximum",
            "MinimumChange": 4,
            "MaximumChange": 8,
            "Trigger": {
              "Type": "AlwaysTrigger"
            }
          }
        }      ],
      "Effects": []
    }
  ],
  "ComputerStateProfiles": [],
  "CriticalTemperature": {
    "/intelcpu/0/temperature/0": 70,
    "/intelcpu/0/temperature/1": 70,
    "/intelcpu/0/temperature/2": 70,
    "/intelcpu/0/temperature/3": 70,
    "/intelcpu/0/temperature/4": 70
  },
  "TemperatureTimerInterval": 250,
  "DeviceSpeedTimerInterval": 2500,
  "DeviceRgbTimerInterval": 1000
}

Thx for the quick reply, I assume it's the same for the Effects section of that issue... do you have an overview over accepted sets?
I tried the original one... and I assume that Type and/or Trigger need to be different, too?
      "Effects": [
        {
          "Type": "DefaultEffect",
          "Config": {
            "Type": "Pulse",
            "Speed": "Slow",
            "Colors": [
              [0, 0, 255],
              [0, 0, 255],
              [0, 0, 255],
              [0, 0, 255],
              [0, 0, 255],
              [0, 0, 255],
              [0, 0, 255],
              [0, 0, 255],
              [0, 0, 255],
              [0, 0, 255],
              [0, 0, 255],
              [0, 0, 255]
            ],
            "Trigger": {
              "OneTimeTrigger": {}
            }
          }
        }
      ]

Btw. this one from the main page leads also to crash of the config reader.
        {
          "Type": "RippleEffect",
          "Config": {
            "Length": 4,
            "Color": [
              [255, 0, 0]
            ],
            "TickInterval": 2,
            "Trigger": {
              "Type": "OneTimeTrigger"
            }
          }
        }

Recommendation: Skip the section and print the faulty statement if that's feasible.

Is it possible to exclude a single port from FanSpeed (like the liquid pump) but keep it for effects?

Thx for the quick reply, I assume it's the same for the Effects section of that issue... do you have an overview over accepted sets?
I tried the original one... and I assume that Type and/or Trigger need to be different, too?

Yes, the trigger format is wrong, but also the DefaultEffect no longer exists. Each of the default Thermaltake effects was rewritten as a separate plugin for more configuration options. If you want to use the controller builtin effects you can use RawEffect which uses one of the effects from the "Available effect types" list.

Btw. this one from the main page leads also to crash of the config reader.

Whoops, thanks for spotting this.
It should be "Color": [255, 0, 0],.

Recommendation: Skip the section and print the faulty statement if that's feasible.

Nicer logging of config errors is already added on develop branch, will be in the next release.

Is it possible to exclude a single port from FanSpeed (like the liquid pump) but keep it for effects?

Yea just create another profile.
Basing on your other issue it would look something like this:
{
  "Profiles": [
    {
      "Name": "Fans",
      "Guid": "5212f5c7-a69e-49fd-86c0-ef7412eea6f0",
      "Ports": [
        [9802, 8101, 2],
        [9802, 8101, 3]
      ],
      "SpeedControllers": [
        {
          "Type": "PwmSpeedController",
          "Config": {
            "CurvePoints": [
              [30, 20],
              [40, 25],
              [50, 40],
              [60, 75],
              [70, 100]
            ],
            "Sensors": ["/intelcpu/0/temperature/4"],
            "MinimumChange": 4,
            "MaximumChange": 8,
            "Trigger": {
              "Type": "AlwaysTrigger"
            }
          }
        }
      ],
      "Effects": [
        {
          "Type": "TemperatureEffect",
          "Config": {
            "Sensors": ["/intelcpu/0/temperature/4"],
            "ColorGradient": [
              [30, [0, 0, 255]],
              [60, [255, 0, 0]]
            ],
            "Trigger": {
              "Type": "AlwaysTrigger"
            }
          }
        }
      ]
    },
    {
      "Name": "Pump",
      "Guid": "c162d280-273d-4b65-a52a-d7a1ff1e41d1",
      "Ports": [
        [9802, 8101, 1]
      ],
      "SpeedControllers": [],
      "Effects": [
        {
          "Type": "TemperatureEffect",
          "Config": {
            "Sensors": ["/intelcpu/0/temperature/4"],
            "ColorGradient": [
              [30, [0, 0, 255]],
              [60, [255, 0, 0]]
            ],
            "Trigger": {
              "Type": "AlwaysTrigger"
            }
          }
        }
      ]
    }
  ],
  "ComputerStateProfiles": [
    {
      "StateType": "Shutdown",
      "Ports": [
        [9802, 8101, 2],
        [9802, 8101, 3]
      ],
      "Speed": 50
    }
  ],
  "CriticalTemperature": {
    "/intelcpu/0/temperature/4": 70
  },
  "PortConfig": []
}

Btw what pump do you have? Was its speed changing with the fans? In the "show hardware info" output you can see its RPM is 0, so I assumed it is not speed controlled.

What an outstanding support...
I recently replaced my 2014 Thermaltake Water 3.0 Extreme with Thermaltake Floe Dual Ring RGB 240 Premium, so I have the Floe Riing Water Pump (which as pump is running as a constant speed) and the two 120mm Riing Plus RGB fans for the cooling element.
The old software for the Water 3.0 was not updated anymore, it was really loud when the fans ran at 50+% and it started to ignore the config (or fought with the BIOS over control).
With the Floe, I had to notice that after a resuming from hibernation, TT software did not restore the original status, so I had to end and restart the software. Looking for a solution, I found your tool mentioned in the TT forums which is exactly the solution I am looking for. Nice, clean minimal service, running in the background and doing, what it should do.
Anything I need to add for resume after hibernation or does the service resets the controller properly?
The pump does not show any RPM in the TT software, only the fans do.

On what base do you create the GUIDs?

so I have the Floe Riing Water Pump (which as pump is running as a constant speed)

Well if the pump is at constant speed and is not speed controllable than there is no reason for 2 profiles. The service will still send commands to the controller to change the speed on pump port but if the pump does not understand that it needs to change the speed it will simply ignore it.
In the official software you can set device type for each port, and so you can force the controller to think that on a port that u have a pump connected there is a fan, that would mean the software is sending speed update commands to the pump, just like the service would with 1 profile.

Anything I need to add for resume after hibernation or does the service resets the controller properly?

Nope it should work correctly, ofc each setup is different so no guarantees.

On what base do you create the GUIDs?

I generate them here. But you could just change one number to a different one, they just have to be in this format and unique.
Btw you might want to add
    {
      "StateType": "Boot",
      "Ports": [
        [9802, 8101, 2],
        [9802, 8101, 3]
      ],
      "Speed": 50
    }

to the ComputerStateProfiles so when your pc is booting it will force the fans to a given speed. It overrides the default speed (and/or rgb) if there is no software controlling the fans. The service needs to run once to save the boot profile settings to the controller.

The TT software reports an error if one tries to convert the TT Floe Riing (Pump) into a Riing Plus (Fan) and does not allow it. That's why I asked how to exclude it here from the pwmspeed part completely.
Your program is provided for free and as is, so if it breaks, I just keep both parts :)
Ok, added and restarted the service. Thanks for that.
Guess a FAQ / examples section where every user can send you his config with a description (processor type/cores and what it does, as well as the version of TTController it was used with) could help new users understanding how to configure or just by grabbing the matching example and going from there.
Nevertheless, great piece of software and exactly what I needed. Thanks for providing it and supporting it.

The TT software reports an error if one tries to convert the TT Floe Riing (Pump) into a Riing Plus (Fan) and does not allow it.

Oh, thats interesting. I wonder how they detect the pump. What does the error say?

Guess a FAQ / examples section where every user can send you his config with a description (processor type/cores and what it does, as well as the version of TTController it was used with) could help new users understanding how to configure or just by grabbing the matching example and going from there.

Yea the plan is to create a configurator app to create the configs so you would not have to touch json at all, but I have not yet figured out a good way to do this.

It shows this.

Basically, if it should be more than a simple json editor that would need sets of effects to achieve and then the app would offer a form for each effect to fill in the values, as text, as graph (for example for the fan speed curve) or as other control item.
Perhaps you could that even with config snippets that are read by the config app and then the config app generates the input form for that snippet.

Could you stop the service then run this and post the output here?

Thanks, so nothing additional is returned for the pump port.
I think that the warning you see is just a no/low RPM warning. Since the pump does not report its RPM when you change the pump port to a fan it expects the RPM to be non 0. So the controller doesn't know that the pump is connected to that port.
Im gonna close this issue since we are getting off topic.
If you need more help with configuration please make a new issue.


Suggestion: be able to track the active publishednodes.json file version

In a scenario where an external component is managing the subscription items by modifying the publishednodes.json file it would be great to be able to track the version current in use by OPC Publisher.
A possible solution would be to add metadata to the publishednodes.json file with a version number, for example:
{
  "versionId": "myUniqueVersionId",
  "items": [
    {
        "EndpointUrl": "opc.tcp://host.docker.internal:49322/",
        "UseSecurity": false,
        "OpcNodes": []
     }
]

Ways to report the current configuration version (not mutual exclusive):

Header in the monitored items telemetry
Can be retrieved using a Direct Method
Updating the module twin reported properties

The version property is now part of 2.9.0-preview4. It is incremented on every change.


Jax fixes

Description of changes
Doing a pass to get jax are related libraries to a good place.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Deferring to the CUDA team, they are much closer to Jax.


Magic Mirror problem after update

Hello,
I need help, i don't have good skills in Javascript. My Magic Mirror don't startet after Update, the problem is one Modul.
The Modul is MMM-Holiday from cowboysdude.
"its a very good modul but after a update from the Magic Mirror 2 it's don't working.
My problem is the Magic Mirror don't start after the instellation of the modul.
Its not a synthax fail. The modul blocket the autostart and the manuel start."
Can you fix it ?  or give me a Tip please.
I don't want a other Holiday Modul.

yes, this module suffers from a missing library
see https://forum.magicmirror.builders/topic/15778/fix-for-black-screen-in-2-16-and-later

this issue should opened on the module github page as the author needs to fix the installation

please follow the suggestions from @sdetweil above. We don't support foreign modules here, so closing this.


fixes empty method

fixes empty method and make downloads work again

I notice the empty method when friendlyName is introduced.  However, seems Chasseurmic no longer maintain this repos?


NIM-20111

Fixes error when '&' is present as a value in any request param.

You did it @rakeshnpatel!
Thank you for signing the Anthem Opensource Software CLA.


PartNet-Mobility Dataset Collision Issue

System:

OS version: Ubuntu 20.04
Python version (if applicable): Python 3.8
SAPIEN version (pip freeze | grep sapien): 1.1.1
Environment: Desktop

Describe the bug
A clear and concise description of what the bug is.
When I load objects such as Ovens and Microwaves into a pybullet environment and place a cube inside those objects, there will be weird mesh collision issue: 1. the cube might penetrate the wall 2. the cube might be shot out by the partnet-mobility object. See the two gifs below.

To Reproduce
Steps to reproduce the behavior (use pastebin for code):

Load any Oven/Microwave into pybullet
Create a cube with proper xyz location and start stepping.

Expected behavior
A clear and concise description of what you expected to happen.
Weird physics.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

This Github repo is for the SAPIEN simulator.
You should probably examine the collision shapes generated by bullet and report any unexpected simulation behavior to them.

I can now provide a guess for this issue. Bullet is probably taking the convex hull of the collision shapes, and you can use their v-hacd module to do a decomposition. Please refer to their documentation for details.


The meaning of the features in the dataset

Hi,
firstly thank you very much for your work and your code. Now I have a question about the meaning of the features. Because I want to learn more about the relationship between images and parameters. So I want to ask can you provide more details of the dataset so that it is possible to learn about which parameter means what. Thank you so much!

Hi Whaleye,
I don't have a comprehensive list, but if you want to investigate the meaning of any particular parameter, I advise that you look into its function in the bikecad software! However, with some luck the parameters you are looking for are in this informal list: https://docs.google.com/spreadsheets/d/1-rOeT-bFhZWLiO0wm6uakQPihaCawp4cJq8QIEMpWcU/edit?usp=sharing
Hope this helps!

Sorry, I think I can not open this link. Can you please update it?


fix(s3-bucket): only bypass governance if object lock enabled

A bug was introduced during support for removing objects with governance settings that if object lock is not enabled the cleanup will fail, this fixes that problem.

:tada: This issue has been resolved in version 3.14.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


how to add group to a project

sorry,I used to use gitbucket to handler my project in my pc.
but, I cant find one way to add group to a project.
please someone could tell me how do it?

sorry,I used to use gitbucket to handler my project in my pc.
but, I cant find one way to add group to a project.
please someone could tell me how do it?

When you say project, do you mean a repository?

yes,I mean a repository.

I don't think it's possible to add a group to a repo, even at github.
so, did you mean add a repository to a group?
you can change the owner of a repo when you are creating one.

oh,group I mean In some program team many developer are in there.
not github's group .

I think what you would have to do is create a group called "Group A". Create a new repository "Project A" and make the "Group A" owner of the project. Then go to edit the "Group A" and add the developers you want. You can also add another group. If you have the developers already in a group e.g. "All developers", just add that group as a member. That should work.

If you have the developers already in a group e.g. "All developers", just add that group as a member. That should work.

This is exactly what I want to do! But I could not find a way to do that from current (3.3) GitBucket UI...
For example, I have three groups designer, developer, and manager.  In that case, how can I bind these groups to a repository ReposA without expanding the group members flatly ?
As for GitHub, they allow us to add collaborators to a repository on team basis. Also they have
organization on the top of that.

Close all questions. If you have any questions about GitBucket, please send them to the gitter room instead of issue according to the Guideline for Issues.


can't load package: package cmd/hello: cannot find package "."

What version of Go are you using (go version)?
go version go1.9 darwin/amd64
Does this issue reproduce with the latest release?
yes.
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/csj/git/go"
GORACE=""
GOROOT="/usr/local/Cellar/go/1.9/libexec"
GOTOOLDIR="/usr/local/Cellar/go/1.9/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/h8/glg18vq91h3b5x5nfpqnbcmw0000gn/T/go-build804657928=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"

What did you do?
I just do these:

Make diretory $GOPATH/src/cmd/hello
Make file $GOPATH/src/cmd/hello/main.go the content in the file:

package main

import (
	"fmt"
)

func main() {
	fmt.Println("hello world")
}

cd $GOPATH
go build cmd/hello

What did you expect to see?
to be build successfully.
What did you see instead?
can't load package: package cmd/hello: cannot find package "." in:
	/usr/local/Cellar/go/1.9/libexec/src/cmd/hello

What make you confused?
https://github.com/golang/go/blob/release-branch.go1.9/src/cmd/go/internal/load/pkg.go#L1735
why treat packages start with cmd/ && !contains '/' followed as "." ?

For better or for worse, this is documented at https://golang.org/cmd/go/#hdr-Description_of_package_lists: "Import paths beginning with "cmd/" only match source code in the Go repository."

Since this is a question which has been answered, I'll assume it's safe to now close this issue.


Replace service module to use systemd. Make masquing more simple.

A small pull request to replace service module and to use systemd module.

I had to revert this one, since the systemd module seems to be only present in newer ansible distributions
ERROR! no action detected in task. This often indicates a misspelled module name, or incorrect module path.
The error appears to have been in '/Users/wolfram/git/hailstorm/ansible/roles/layer1/tasks/prepare_host.yml': line 28, column 3, but may
be elsewhere in the file depending on the exact syntax problem.
The offending line appears to be:

name: disable & stop NetworkManager
^ here

The error appears to have been in '/Users/wolfram/git/hailstorm/ansible/roles/layer1/tasks/prepare_host.yml': line 28, column 3, but may
be elsewhere in the file depending on the exact syntax problem.
The offending line appears to be:

name: disable & stop NetworkManager
^ here


misleading next/previous buttons

These aren't doing anything.
Potential solution:

Hide button when links aren't present.
Better tree traversal for next/prev links (IDK how this is implemented)

example: https://docs.erpnext.com/docs/v12/user/manual/en/selling#1-topics


Simpler line in simple/core.cljs

It also makes the code like in the walkthrough:
https://github.com/Day8/re-frame/blob/master/docs/CodeWalkthrough.md#reg-sub

Good to get this consistent, thanks.


Test cargo miri target selection

This is a followup to #1525, adding a few test invocations with targets specified in the cargo arguments.

Thanks. :)
@bors r+

:pushpin: Commit 5d9d75fc1f2fc87f419913db7bf8a073b04f955d has been approved by RalfJung

:hourglass: Testing commit 5d9d75fc1f2fc87f419913db7bf8a073b04f955d with merge 9123f0e01ec32103a35ab9d51122ceee5ce7bcfb...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing 9123f0e01ec32103a35ab9d51122ceee5ce7bcfb to master...


how to infer and visualize?

hello, After the training is completed and the weight file is obtained, I want to learn how to infer and visualize 2D and 3D images. Can you provide the corresponding code for Pointers? Thank you.

The inference function has been updated in trainer.py of the test branch, and you can use python run.py -m inf to evaluate your network. Besides, we visualized the results in Jupyter Notebook, which is easy to use.

Thank you for your reply. In addition, I would like to ask you to see that you provided preprocessed data in the question of responding to a request for raw data from someone else on github. If there is a PI-CAI22 dataset, I do not see the preprocessing code for the dataset in the code. Could you please provide the dataset preprocessing code？


feat(deps): update all

Updates packages

let's name it @stacks/eslint-config

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Moving columns not possible if the rewrite is enabled

Hello,
In the preferences of a project, I can not set the order of columns if the url rewrite is enabled with nginx anyway (/project/8/column/34/move/down?csrf_token=......).
I get an error message : Access forbidden
Thx

What Nginx config are you using?

server {
        listen 80;
        server_name mydomain.com;
        return 301 https://$server_name$request_uri;  # enforce https

	access_log /var/log/nginx/kanboard.access.log;
	error_log /var/log/nginx/kanboard.error.log;

        location ~ \.php$ {
                fastcgi_pass    unix:/var/run/php5-fpm.sock;
                fastcgi_index   index.php;
                fastcgi_param   SCRIPT_FILENAME $document_root$fastcgi_script_na
me;
                include                 fastcgi_params;
        }

        location / {
                try_files $uri $uri/ index.php;
        }
}

server {
	listen		443 ssl spdy;
	server_name	mydomain.com;
	root		/var/www/kanboard/;
	index		index.php;

	access_log /var/log/nginx/kanboard.access.log;
	error_log /var/log/nginx/kanboard.error.log;

        ssl on;
	ssl_certificate /etc/nginx/certs/mydomain.com.crt;
	ssl_certificate_key /etc/nginx/certs/mydomain.com.key;

	location / {
    		try_files $uri $uri/ /index.php;
	}

	location ~ \.php$ {
    		try_files $uri =404;
    		fastcgi_split_path_info ^(.+\.php)(/.+)$;
   		fastcgi_pass unix:/var/run/php5-fpm.sock;
   		fastcgi_index index.php;
    		fastcgi_param   SCRIPT_FILENAME $document_root$fastcgi_script_na
me;
    		include fastcgi_params;
	}

	# Deny access to the directory data
	location ~* /data {
    		deny all;
    		return 404;
	}

	# Deny access to .htaccess
	location ~ /\.ht {
    		deny all;
    		return 404;
	}
}

Can confirm this problem, also running nginx with the config suggested in the documentation.

I was able to fix this issue by changing the
try_files $uri $uri/ /index.php;
to
try_files $uri $uri/ /index.php$is_args$args;
as also suggested in issue https://github.com/fguillot/kanboard/issues/1460

Works without problems thank you ;)


EfficientDet Model Help

Hi,
I'm currently developing a version of EfficientDet-D0 with an added BiFPN neck. Current AP[50:95] for this model is 31.5%, which is lower from the paper's reported value of 34.6%.
Are there any further improvements that can be made to this model, specifically when it comes to finetuning the config? Most of the parameters used are based off of the EfficientNet PR, so should any of these be changed that could better fit EfficientDet? Are there any tools that can be used for finetuning in mmdetection, or is it mostly guess and check? Any and all feedback would be greatly appreciated.

It is very difficult to reproduce the performance of EfficientDet. There are many important details or training tricks. The generalization ability of the model in other datasets is not very good (purely personal experience). I do not recommend trying this series of models.

It is very difficult to reproduce the performance of EfficientDet. There are many important details or training tricks. The generalization ability of the model in other datasets is not very good (purely personal experience). I do not recommend trying this series of models.


CI: Envoy warning on work-in-progress feature being used

Happened in https://github.com/cilium/cilium/actions/runs/7115820553/job/19372964949:
2023-12-06T14:30:15.009644153Z level=warning msg="[message 'envoy.extensions.bootstrap.internal_listener.v3.InternalListener' is contained in proto file 'envoy/extensions/bootstrap/internal_listener/v3/internal_listener.proto' marked as work-in-progress. API features marked as work-in-progress are not considered stable, are not covered by the threat model, are not supported by the security team, and are subject to breaking changes. Do not use this feature without understanding each of the previous points." subsys=envoy-misc threadID=637

Happened in many different tests.

Related to https://github.com/cilium/cilium/pull/29026

The internal_listener API becomes stable as part of v1.28.0, so I think this error log will be gone once cilium main is upgraded to envoy v1.28.x later (the current envoy version used in cilium main is v1.27.x)
https://github.com/envoyproxy/envoy/blob/719de95c12d55fb0b6ec7ec88e8953078248b856/api/envoy/extensions/bootstrap/internal_listener/v3/internal_listener.proto#L14

AFAIU, we'll be able to drop this warning exception from CI once we update Envoy. So let's keep this open to track that. (Please correct me if I'm mistaken.)

This is fixed as part of below PR, closing the issue.
https://github.com/cilium/cilium/pull/29820


WIP: Lcam 376 calc appeal contrib business logic

What
Link to story
Migration of business logic in the contribution.calc_appeal_contrib stored procedure to the new CCC service, along with associated tests. The business logic first validates request data to check we have all the information to proceed, then it checks through the appeal assessment results to determine if it PASSed or FAILed. It uses this to call off to MAAT API to get the amount of contribution that should be paid for the appeal, which it then checks against the current upfront contribution amount (again retrieved via MAAT API) to see if the data in MAAT DB needs updating. If they are different it then the data needs to be updated so we call off to MAAT API a third time to replace the old contribution entry with the updated data and return this updated contribution entry to MAAT. If they are the same then no update is required and we just return the existing contribution entry to MAAT. MAAT will then use this response to determine if the MATRIX processor needs to be called and so on.
Checklist
Before you ask people to review this PR:

[x] Tests should be passing: ./gradlew test
[x] Github should not be reporting conflicts; you should have recently run git rebase main.
[x] Avoid mixing whitespace changes with code changes in the same commit. These make diffs harder to read and conflicts more likely.
[x] You should have looked at the diff against main and ensured that nothing unexpected is included in your changes.
[x] You should have checked that the commit messages say why the change was made.

Please resolve the code smells.


[ADD FRONTEND] : Authentication UI Update

Frontend Summary 📝
/login , /register , /forgot-password
Development Approach 🚀
Authentication pages UI Update little bit.
Deployment Status
None
Select the open source program you are contributing to
Other
Code of Conduct

[X] I adhere to the CONTRIBUTING GUIDELINES of this project.

Hey @shuvadeepmondal👋! Thanks for raising this issue and demonstrating your interest in contributing to our project!🚀
We value your engagement and eagerly anticipate your contributions. Should you wish to get involved, please use the /assignme command to self-assign the issue.
This issue provides an excellent opportunity for us to enhance our project, and we're excited to have you join us! Please take a moment to review our CONTRIBUTING GUIDELINES before getting started.
If you have any questions or need assistance, feel free to reach out to us on LinkedIn.❤

/assignme

Hello @shuvadeepmondal! We'd like to inform you that the issue has been resolved and is now closed. We appreciate your understanding and look forward to your continued involvement with our repository!🤗
If you encounter any further issues, please don't hesitate to reach out to us on LinkedIn. Thank you for your support!❣
Happy coding!✨ We hope to see you again soon!❣


ADB error on Mac

I get the following error when running npm run start:android on mac.
reaction start -p android

events.js:183
      throw er; // Unhandled 'error' event
      ^

Error: spawn adb ENOENT
    at _errnoException (util.js:992:11)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:190:19)
    at onErrorNT (internal/child_process.js:372:16)
    at _combinedTickCallback (internal/process/next_tick.js:138:11)
    at process._tickCallback (internal/process/next_tick.js:180:9)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start:android: `reaction start -p android`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start:android script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

This error has disappeared with updates to Android.


Update monitors_test.py

Move from contrib to training

Can one of the admins verify this patch?

@tensorflow-jenkins test this please

@tensorflow-jenkins test this please


Add Murky

Murky is super useful for testing merkle tree validation. Would be great to have this added to soldeer.
https://github.com/dmfxyz/murky

gm, i wrote as well in one of the issues as well what It needs. meanwhile, you can use forge soldeer install murky~1.0 git@github.com:dmfxyz/murky.git, will install from git, it won't be fast but until then you can workaround with this


feat(build): add flag to optionally generate c witness calculator

Adding a config option that allows to generate the c witness calculator files.
Addresses #43

I have moved the witness generation test w.r.t cWitness under configs.test.ts, compiling the same circuit twice caused some issues in the test (i will dive deeper into it #45).


Scaling on Retina displays

I'm trying to understand what's happening with scaling and Retina displays (but not just them, even resizing windows is having some issues).
I noticed that you're assigning the scale coming from SDL (I double checked that value and it's correct) to the window.scale property, but then that property isn't used by anything. Where am I wrong?
https://github.com/openfl/lime/blob/master/lime/_backend/native/NativeRenderer.hx#L48

I think I fixed this with the following PRs:
https://github.com/openfl/lime/pull/609
https://github.com/openfl/openfl/pull/873

Ouch, I had an improvement in OpenFL before, but this appears to have not been committed somehow?!
We need to apply the window.scale to incoming stage size, mouse event coordinates, etc, on the OpenFL side. Otherwise, tweaking the sizes in Lime still results in all our mouse events (etc) being wrong. Let me take a look

I already added it to stage size and resize I think .. I don't know about mouse events as they seems to work but that might be because I'm tapping in the correct half of the screen :)

Okay, could you take a look at that change? I believe this is what I had working and tested before on iOS.
I'll try and take another look here, but it helps to get more feedback from others as well. Are you testing on a retina Mac laptop/desktop, or a device?

Here's an example from prime31 on gitter chat (uses latest changes):
https://vid.me/e/suFy

I've been testing on iOS devices. I have a Retina Macbook Pro as well where I can try this out. I'll have a look at it tomorrow if I can find some time during the weekend. I'll keep you posted.

@larsiusprime this may be related https://github.com/openfl/openfl/pull/828/files#diff-850c104eddc0405ea6d18a7b67318b35R1039 The change wasn't merged.

@mrcdk I tried but it looks it's not directly related to this. At least it's not part of the Retina problem.. might be for the resize, though. I'll have another look at it after I get Retina working right.
On Mac and iOS the stage and window size at startup is being set to exactly 2x what it should be (so the window goes off the screen and the stage is scaled 2x more than what it should be).
The problem is that it happens with just Lime as well, so it's something that has to be fixed on the Lime side.
@jgranick can you find me a sample using only Lime that draws a colored rectangle that fills the whole screen, with a line all along the border? I fear I'm messing up with matrices and stuff when I try on my own and eventually the window might be bigger than it should be and I can't see it as the SampleImage only draws a bitmap that is smaller than the screen.

Here's what I understand of what's going on between SDL and Retina displays.
On iOS the size of the window is exactly the size in points (not pixels) and the scale property that we calculate is the multiplier to obtain the size of the screen in pixels.
On OSX the size of the window is the size in pixels. The scale is being calculated the same way, so it ends up having a window that is double the size that it should actually be.
And of course all of the calculations involving the scale are wrong.
I don't know what's the SDL best practice, but we should either differentiate between iOS and Mac or find out how to get both the scale and window size with homogeneous values.
Any suggestions?

Reading around the web, it looks like SDL is working this way:
Window size, window position, display mode size, and mouse position are all in �points� (as iOS and OS X defines them) rather than pixels. If the mouse position isn�t in that coordinate space then there�s a bug somewhere.
SDL�s touch coordinates are always in the range of [0, 1]. On touch screen devices that�s relative to the SDL window�s position.

Here�s some more explanations:
https://developer.apple.com/library/mac/documentation/GraphicsAnimation/Conceptual/HighResolutionOSX/Introduction/Introduction.html
https://developer.apple.com/library/mac/documentation/GraphicsAnimation/Conceptual/HighResolutionOSX/Explained/Explained.html

The drawable size is in pixels rather than points when high-dpi mode is enabled on a retina display. The idea is to have significantly more pixel density while taking up the exact same amount of area and with content displaying at the same sizes and relative positions as when high-dpi mode isn�t enabled.

which suggests that the correct way to handle those things is to work in points just like they do.

I believe that we should be handling this in points now. On iOS, the window size, resize, and mouse coordinates are in points. By reading the scale value, we can extrapolate to pixels.
Does it seem like SDL is doing this wrong on OS X? Or that Apple handles this differently?

It looks like it's SDL that's wrong. SDL_GetRendererOutputSize() gives the size in pixels instead of points


[Event Request] codeunit 5940 ServContractManagement - CreateAllCreditLines

We would like to request an additional publisher in function CreateAllCreditLines so we can overrule the value of the OldWDate variable:
        OldWDate := CalcDate('<CM>', WDate);
        if Days <> 1 then
            Days := 1
        else begin
            for i := 1 to InvPeriod do
                OldWDate := CalcDate('<CM>', OldWDate) + 1;
            OldWDate := OldWDate - 1;
        end;

        // ##### START >>
        OnCreateAllCreditLinesOnAfterDetermineOldWDate(ServContract, InvPeriod, Days, WDate, VAR OldWDate);
        // ##### END <<

        if OldWDate >= PeriodStarts then begin
            if WDate < PeriodStarts then
                WDate := PeriodStarts;
            if OldWDate > PeriodEnds then

Many thanks.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


add support for Doctor (env checker/helper)

duplicate of https://github.com/flexn-io/renative/pull/1632


Expanded lock return codes

Please review @schilit and @roywant

What do you think of structuring these more like Android doc for "Returns"
Returns

Insufficient Authorization if this was that, false if it was something else.

Updated


look into openAPI and GRAPHQL spec

Look at #63 for an example implementation of the GraphQL library. We must use an OpenAPI / GraphQL integration as the fundamental first step for building any API for Ark.
Schema based API generation ensures that we can utilize automated code and test generation in the long run.


Edit text

I don't know what you think but, I've thought about whether it would be better if selected text (or text in a text field) should be pre-inserted in emacs or not. Another idea could be to provide a settings for emacs-everywhere to choose things like this.
The current alternative isn't that bad either, one must just select and copy the text, and then paste it on emacs, after it would be replaced because it was selected before insertion.

Hi @jaimehrubiks ! I think this issue is similar to #3. Thanks!

I didn´t see that issue! I´ll close then. Thanks.

By the way, I am really enjoying this software, it is so fun!. Thanks.

I’m  really happy you’re getting some use out of it! And I appreciate you getting involved and helping us out. Thank you!


How to Save Configuration

Is there any way to store that sa file and favourite drive list in this not because each time it restarts it asks for it again.

Hi,
It's not possible. You can host the bot in another service than heroku if you don't want it to restart every 24h

if a clone task need more than 24 hours to finish. it will never be done?


fixes before/after taxes discrepancy

Changes model sentence method to fix discrepancy between creation and showing of salary entries where one said before taxes and the other said after taxtes

Thanks so much for spotting this @TPei & for the fix!


Change ContextSingleton retention policy to RUNTIME

Hi,
First of all thanks for the great library!
Recently, I tried updating toothpick from 1.0.3 to 1.0.5 in my Android project and got:
Error:Scope Annotation toothpick.smoothie.annotations.ContextSingleton does not have RUNTIME retention policy. during compilation.
I checked and ContextSingleton from smoothie has runtime policy CLASS currently unlike e.g. Singleton from javax.inject or the custom scope in smoothie-sample which probably causes this error. This is probably due to the changes in #191
Could you help please?

Hi @zawadz88 ,
Thanks for the issue.
This class was meant to be there just as an example. However, the javadoc is not really clear and probably the class is not useful either.
Therefore, we will deprecate the class, change the retention and update the javadoc.
The class will be removed in a later release.
If you are using context singletons, please create the annotation yourself and bind it to the right scopes:
https://github.com/stephanenicolas/toothpick/wiki/Scope-Annotations#binding-a-scope-to-a-scope-annotation

I would add that ContextSingleton was really a RoboGuice thing. It looked really cool at that time as you could quickly inject a context. At Groupon, we used a TP ContextSingleton for our migration from RG to TP.
But, later, we just removed it. The issue is that when you @Inject Context, then you just don't know exactly which context you are injecting : an activity or the app context ?? And one can create a leak, not the other one, simply depending on where you inject your entity.
So, definitely ContextSingleton is more a simple example of a custom annotation but it should not be used in a production app. At Groupon, we ActivityScope, ServiceScope, and we never inject a context, always either the Application or an Activity or a Service, so that we have full control over what is injected. And we use the scope verification mechanism of TP to be sure we don't leak the activity.

Thanks for the clarification and sorry for the misclick ;)

I was searching how to inject the Application in a @Singleton is there any example on the documentation/demo?
I also don't find any documentation about Toothpick. I'm just a beginner with DI :/

If you install the smoothie application module in your app scope, then you can simply use @Inject Application app in this scope or any sub scope.
All the docs of TP is in the wiki in the repo... @HugoGresse

well, that what I thought but I can't seem to make it work in my @Singleton
public class MyApplication extends Application {

    @Override
    public void onCreate() {
        super.onCreate();

        Scope appScope = Toothpick.openScope(this);
        appScope.installModules(new SmoothieApplicationModule(this));
    }
}

@Singleton
public class ServiceProvider {

    @Inject
    protected SocialService                mSocialService;

    public ServiceProvider() {
    }

@Singleton
public class SocialService {

    @Inject
    public Application mApplication;

    public SocialService() {
        mChromeCustomTabsManager = new ChromeCustomTabsManager(mApplication);
    }

I can't achieve to inject the Application in the last SocialService singleton... Am I missing something? I'm using @Inject ServiceProvider in activities like this

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        Scope scope = Toothpick.openScopes(getApplication(), ServiceProvider.class, this);
        scope.installModules(new SmoothieActivityModule(this));
        super.onCreate(savedInstanceState);
        Toothpick.inject(this, scope);

@HugoGresse were you able to do it?

yep, that was fine. I've still some issue where the @Inject is done after constructor call which conduce me to delay some action. For example here. I've probably made something wrong but I am inject the scope on the class AFTER is instantation that maybe be this.

@HugoGresse there are 2 things you can do with TP, and I believe it's the same with dagger / Roboguice:

use a constructor with @Inject annotation, then all arguments will be created by TP (it needs to have a provider / or a binding or the classes to be annotated properly).

class Foo {
  Bar bar;

  @Inject
  public Foo(Bar bar) {
     bar.bar(); // <-- would work as long as TP knows how to create the instance of Bar
  }

use a @Inject annnotated method. These methods are executed after fields are initialized.

class Foo {
  @Inject Bar bar;

  @Inject
  public Foo() {
     bar.bar(); // <-- would fail because members are not initialized yet, see the factory code.
  }

 @Inject
 void init() {
     bar.bar(); // <-- would work
  }
}

Release 1.0.8 will solve this issue.


improper stratux shutdowns

It occurred to me that we might have to manage the proper shutdown of the rPi(x) as they get more popular and in the hands of people who aren't familiar with the requirement to do so.  It might also help if some sort of certification is in stratux' future.
Here are the first 3 possibilities I could think of:

Add a "shutdown stratux and system" option to the 'web page' interface.  It's purpose is to pass a "shutdown -h now" command to the kernel.  Further, stress in user training that this MUST be performed rather than just turning off the power to the "stratux box."
Leave the code as-is.  Emphasize training to newbs on how to ssh into the rPi(x) to pass the shutdown command.  Yeah, like that will work on a non-computer person.
Add a bit of hardware to the stratux system that has a few minutes of backup battery time, a current (12 or 5 vdc) sensor to the stratux power-in that toggles a gpio port that external power has been lost and to execute a graceful shutdown if the user allows it (could be an emergency situation and the user desperately needs the gps or ahrs functions, but regardless do a proper shutdown when backup battery is nearly empty.

Other power sensors could be utilized in the future.  Like if on ship's power, delay powering up the stratux system until after that massive current drain of the engine starter is over with.  Brownouts can really hurt systems like ours.  I mean this is getting rather sophisticated for a $130 box, but we might want to keep it in the back of our minds for the future.  A pass-through battery power conditioner would be just the thing...

Has been discussed before, don't want to open this conversation again. Make an app for it or something, should be a two liner.


2d auto segmentation

Hi,
I am testing 2d .
the auto segmentation runs, but doesn't give any outputs (with default parameters) ... ?
Any ideas ?
thanks
antho

I can share an image if that helps ?

If you're using the vit_b_lm then you need to set lower values for the thresholds:
pred_iou_thresh = 0.65, stability_score_thresh = 0.75

We are working on setting these thresholds automatically, see #337.

Thanks a lot,
using vit_b_lm and pred_iou_thresh = 0.65, stability_score_thresh = 0.75 as suggested :
see below :
I think it needs it might need pre filtering or some normalization step ?

Any ideas ?
thanks

Yes, I think the cells are too small for the current settings of the automated segmentation. I think it is more promising if you follow the approach outlined in #383. For this the automatic segmentation results don't really matter. As long as manual annotation either via point or bounding box prompts works well the refinement approach should work.


Add support for time-point specific overrides to petab.parameter_mapping

petab.parameter_mapping does not support time-point specific parameter overrides. To work with the current mapping system, these measurements should be split into groups by overrides and returning them as separate conditions.

Wouldn't it be better to avoid the generation of additional conditions and instead duplicate the observables? This would also avoid additional simulations.

Avoiding additional simulations is a good point. Besides that I don't see any (dis)advantages of the one over the other.

Wouldn't it be better to avoid the generation of additional conditions and instead duplicate the observables?

Update: This will work for observableParameters and noiseParameters. However, we also want to support dynamic parameter overrides. There resimulation is inevitable. For the output parameter we should avoid it though.

is there any example in the benchmark having this feature at the moment?

I'd propose to do that as a preprocessing step independently of the parameter mapping functions, creating a new set of tables.
For the replicate-observable case:
Something like flatten_timepoint_specific_output_overrides(petab_problem: petab.Problem) -> None.
This would:

[ ] check for observables with timepoint-specific observableParameters and noiseParameters
[ ] determine unique combinations of condition, preequilibration, observableParameters, noiseParameters, timepoint
[ ] create new observable {oldname}_{n} and noise model assignment rule in the SBML model for each of those combinations
[ ] remove or keep (?) original rules
[ ] replace observableId in the measurement table

is there any example in the benchmark having this feature at the moment?

I think so, but don't remember which one. should be simple to construct though.
For the implementation, anyways  a test case with a minimal(!) example should be added

There should be several examples, e.g. the Isensee model. The control condition is use in many experiments with identical time points, but the observable parameters differ.

Just need a short clarification @dweindl how to handle the case (if considered):

different noise parameters, but
same scaling

should it be also considered as a new replicate (e.g. obs1_1 and obs1_2)?

Yes, it should.

I came up with a solution, it works for the simple test case, but I am afraid it will not support all possible models/scenarios... Could you still @dweindl have a look? Maybe you can give me some guidance :smile:

If it works for the test case it should be fine. I thought I considered all relevant scenarios.


Issue #6 - Add EddieHubCommunity template files

We skipped using the template! But we can always merge it in!

@stemount can you give this a rebase when you have a moment?

hey @nhcarrigan thank you for the feedback, squashing/rebase would have tidied this up as you can see the mess of merge commits here. Thankfully there isn't LOADS of merge with "upstream" commits, and just 2.
thanks @rohan-kulkarni-25 managed to sneak in fixing the PNG thing in here too ;-) sneaky


[WIP] signals: initial commit

WIP

@xmudrii can you fix the content and then we can disable the tests and merge.
I will look at fixing the tests once we have this merged if that is okay!


Update examples to ES6

Pretty self-expatiatory. ES6 police 🚓

Hey Stefan,
Thanks for your PR!
However I have a question. Why using const over let in this context? I thought const would be for storing primary types that should stay in stone. But my vision of constants is maybe outdated due to years of trauma doing C++ at uni :-D

When we use const to declare something, it can not be reassigned again later on. In this case, nobody would want to reassign Vivus after being initially set. Consts are much more error proof in this case. Also Vivus object could be modified as objects declared with const are mutable. Makes sense or I might be a bit dumb?

You're completely right, and it's not the first time I see it. It's just me. I'm still not used to the idea of having an object in a const and call methods to it that will modify the object. To me, const is beyond locking the pointer, it's also about locking all the child values contained in that object. But I guess I'm an old grumpy cat that needs to evolve.


[Feature] New table strategy (Truncate+Insert)

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
Currently, the "Table" models delete and recreate the table, for most cases this is ok, but when you have an application granting and supervising permissions, they get lost on each execution, and granting them on the DBT can make the permissions not be aligned with the centralized permission assignation process.
Describe alternatives you've considered
The current workaround is an incremental model that executes a TRUNCATE as a preaction, but it isn't a good practice.
The proposed case will run as follows:

TRUNCATE table;
Create table xxx_tmp as SELECT
INSERT into table SELECT * from xxx_tmp

Who will this benefit?
Anyone that has a centralized permission process and wants to avoid users manually granting permissions.
It also will make the process transparent if a view depends on that table.
Also if that table has been modified for performance (index, distkey, etc) it doesn't need to change DBT configuration.
Are you interested in contributing this feature?
No response
Anything else?
No response

I would love to explore and work on this.

Thanks for raising this idea @alejandrofm !
It sounds like one key use case here is when there is a centralized permission process that is granting and supervising permissions.
In the scenario you are proposing, how is the table created initially? It is also being created and managed by a process outside of dbt?

In my particular case, I manage the permissions per schema, so after the table is created (by DBT or by a user) the process gives the proper permissions to the users that should be able to query that schema.
The first time we have some minutes where the table has no permissions (only the owner), but the idea is that this happens only the first time, if we don't delete the table it keeps the permissions with no downtime for other processes.

Just as our response to https://github.com/dbt-labs/dbt-core/issues/9056, rather than add new incremental strategies in dbt-core, we'd rather encourage the ecosystem through:
Rather than add new incremental strategies in dbt-core, we'd rather encourage the ecosystem through:

Enabling custom incremental strategies, and
Foster sharing that custom logic through the dbt Package Hub

So I'm going to close this as "not planned" in dbt-core favor of either a custom incremental strategies or custom materialization. See below for an example of a custom incremental strategy.
Example
This is a really quick example for demo purposes only and is completely untested, but you can create a custom incremental strategy named truncate_insert like this:
macros/truncate_insert.sql
{% macro get_incremental_truncate_insert_sql(arg_dict) %}

  {% do return(some_custom_macro_with_sql(arg_dict["target_relation"], arg_dict["temp_relation"], arg_dict["unique_key"], arg_dict["dest_columns"], arg_dict["incremental_predicates"])) %}

{% endmacro %}

{% macro some_custom_macro_with_sql(target_relation, temp_relation, unique_key, dest_columns, incremental_predicates) %}

    {%- set dest_cols_csv = get_quoted_csv(dest_columns | map(attribute="name")) -%}

    truncate {{ target_relation }};

    insert into {{ target_relation }} ({{ dest_cols_csv }})
    (
        select {{ dest_cols_csv }}
        from {{ temp_relation }}
    )

{% endmacro %}

And then use it like this:
models/my_model.sql
{{ config(
    materialized="incremental",
    incremental_strategy="truncate_insert",
) }}

select 1 as id

@alejandrofm I think using a custom incremental strategy like above would achieve all the benefits you described, but please let me know if there's something unaddressed.
The only thing is that your desired strategy wouldn't be a built-in strategy within dbt and it would need to be added to each dbt project (via macros or the dbt package ecosystem).

I can do that, in fact, now I am using an incremental with truncate as a preaction, so, it's in line with that and easy to share with all internal teams.
I thought more people could benefit from something like this out of the box.
But if I'm the only one, I can go for a custom one

I thought more people could benefit from something like this out of the box.

We're not planning on adding more strategies out of the box, but we do want to encourage sharing between folks that have the same need! 🎁
The way to do that is to:

Create a dbt package and publish it to the dbt Package Hub
Add your custom strategy to the package
Then others can install and use that strategy like this


Automatically enable JMX and pick an open port.

New Configuration

gobblin.yarn.app.master.jmx.enabled (optional)  If set to true, JMX will be enabled on the app_master on a randomly picked port.
gobblin.yarn.container.jmx.enabled (optional) If set to true, JMX will be enabled on the containers each with a randomly picked port.
gobblin.yarn.jmx.configuration (optional) If set, the value will be used when specifying the jmx configuration for containers or app_master which have jmx enabled.  The token $JMX_PORT will be replaced with a randomly selected port. Default: -Dcom.sun.management.jmxremote -Dcom.sun.management.jmxremote.authenticate=false -Dcom.sun.management.jmxremote.ssl=false -Djava.rmi.server.hostname=<IP_ADDRESS> -Dcom.sun.management.jmxremote.port=$JMX_PORT -Dcom.sun.management.jmxremote.rmi.port=$JMX_PORT

New Logging
JVM input arguments are logged on app_master and container start.  These can be used to determine the JMX port for the
2016-02-22 09:31:13 UTC INFO  [main] gobblin.yarn.GobblinWorkUnitRunner  - JVM Input Arguments: -Dcom.sun.management.jmxremote -Dcom.sun.management.jmxremote.authenticate=false -Dcom.sun.management.jmxremote.ssl=false -Djava.rmi.server.hostname=<IP_ADDRESS> -Dcom.sun.management.jmxremote.port=56298 -Dcom.sun.management.jmxremote.rmi.port=56298 -Xmx1216M

Usage
With the default JMX configuration, to connect an app_master or container for which JMX is enabled:

Check the logs to find the JXM port by specified in the line starting with: JVM Input Arguments:
Open an ssh tunnel: ssh -nNT -L <JMX_PORT>:localhost:<JMX_PORT> <USER>@<HOST>
Connect to localhost:<JMX_PORT> with VisualVM or a similar tool.

I was thinking about maybe doing this a different way.  What if we recognized certain "tokens" in the app_master and container jvm arguments and replaced them when creating the java command.  The first token would be "$PORT_X" where X is a number > 0.  We would identify all such tokens and replace them with a random port.  All instances of the token which share the same X would have the same port.  This would be much more generic as I could then use YourKit, JMX, etc with the app_master and containers.

@kadaan In my experience, using random ports is not a good idea. Applications connecting to Gobblin with JMX will have a hard time figuring out the port.
I would suggest that user provides the port, and a port selection policy: either fail if the port is not available (default) or look for an available port in case of failure.

For this reason, I have added logging of the JVM input arguments when the app_master and container start. That way it is easy to figure out what the port will be.
That said, maybe the “token” should be $PORT_<REQUESTED_PORT> like $PORT_10001.  If port 10001 is open we take it.  If not we try 10002.  And so on.  It will still be “random” in Yarn because there are multiple container on the same node, so the JVM input argument logging will still be needed.
As a longer term item, it would be really nice to be able to “classify” the ports and expose them via the admin UI.  That way I could see a list of nodes, see what containers exist, see the “open” ports, etc.


Allow for fast accumulation selection for FP8 GEMM

In this PR, issue#6168  is addressed by introducing a custom gradients(forward mode autodiff) for fp8 dot_general. This PR is closely related to FLAX PR-6599@reedwm @kaixih @burmako

Thank you for the heads up! Also cc @GleasonK

Codecov Report

Merging #3416 (1ee5f34) into main (8db32ae) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #3416      +/-   ##
==========================================
+ Coverage   83.59%   83.61%   +0.02%
==========================================
  Files          56       56
  Lines        6773     6782       +9
==========================================
+ Hits         5662     5671       +9
  Misses       1111     1111

Files
Coverage Δ

flax/linen/fp8_ops.py
100.00% <100.00%> (ø)

@wenscarl - sorry I was about to merge but there's a fresh merge conflict - would you mind resolving it and I can merge?

As for https://github.com/google/flax/pull/3416#issuecomment-1791485046, can you also rebase your change against the latest? I think the upstream has done some format cleanup. @wenscarl

@levskaya Can you take a look if the merge can be triggered?

  But beyond that  I'm seeing this error in internal CI for Fp8Test.test_fp8_dot_general_injection:
jax.errors.SimplifiedTraceback: For simplicity, JAX has removed its internal frames from the traceback of the following exception. Set JAX_TRACEBACK_FILTERING=off to include these.

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/build/work/74f3597f42da87446d9a71de4bf7594fa9c1/google3/runfiles/google3/third_party/py/flax/tests/linen/linen_test.py", line 940, in test_fp8_dot_general_injection
    output2a, output2b = run(True, expected_shapes_new)
  File "/build/work/74f3597f42da87446d9a71de4bf7594fa9c1/google3/runfiles/google3/third_party/py/flax/tests/linen/linen_test.py", line 909, in run
    y, initial_vars = p.init_with_output(init_key, x)
  File "/build/work/74f3597f42da87446d9a71de4bf7594fa9c1/google3/runfiles/google3/third_party/py/flax/linen/linear.py", line 167, in __call__
    out = dot_general(
  File "/build/work/74f3597f42da87446d9a71de4bf7594fa9c1/google3/runfiles/google3/third_party/py/flax/linen/fp8_ops.py", line 196, in __call__
    y_qdq = dot_general_with_precision(x_qdq, k_qdq, dimension_numbers)  # type: ignore
AttributeError: No JVP defined for custom_jvp function dot_general_with_precision using defjvp.

@levskaya Could you please review this once more? We have resolved the issue with the failed test, which occurred due to the accidental deletion of a decorator in the previous commit. All tests are now passing successfully.

@kaixih - should be merged now!  sorry for the delay!


Prefix all our environment variables with BEYLA_

Maybe the exception should be the OTEL standard variables.
Consider also prefix discovery-related env variables with something like BEYLA_DISCOVERY_. So this would be something like BEYLA_DISCOVERY_OPEN_PORT or BEYLA_DISCOVERY_EXECUTABLE_NAME

Duplicate of https://github.com/grafana/beyla/issues/213


Source Code App Crash

is there anyone? who can solve the error of this application. Because this application crashed when click on any button, both emulator and mobile..

give camera permission for the application

what kind of IDE you started to develop this app?

Android studio
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: Rendelnalangan<EMAIL_ADDRESS>Sent: Tuesday, January 14, 2020 8:39:45 PM
To: YahyaOdeh/HealthWatcher<EMAIL_ADDRESS>Cc: Yahya Odeh<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [YahyaOdeh/HealthWatcher] Source Code App Crash (#4)
what kind of IDE you started to develop this app?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/YahyaOdeh/HealthWatcher/issues/4?email_source=notifications&email_token=AHBDKM6VFJ2N2Z434UFF6PTQ5YIIDA5CNFSM4IOCAATKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEI53L7A#issuecomment-574338556, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHBDKM6O7R7TQPNN6QFYLL3Q5YIIDANCNFSM4IOCAATA.


[OOBE]Add What's New page - SCOOBE

Summary of the Pull Request
What is this about:
Adds the Second Chance Out Of the Box Experience.
What is included in the PR:

Adds a "What's New?" page to OOBE, containing the release notes for the 5 latest releases.
The release notes are download from GitHub releases and the markdown is rendered.
Adds a detection mechanism to show the "What's New?" page when PowerToys starts in a different version than the previous time it ran.

While loading:

In case of an error:

How does someone test / validate:
Open OOBE and check the page.
Make sure there's no file named "oobe_last_version_run.json" on %LOCALAPPDATA%\Microsoft\PowerToys or that the version saved inside is different than the one you're running. In this case, the "What's New?" page should appear when you start PowerToys.
Quality Checklist

[x] Linked issue: #14536
[ ] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[x] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[x] Binaries: Any new files are added to WXS / YML

[x] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Hi @niels9001 ,
I've used a generic icon for the What's New page for now. (Used the settings icon)
Could you please do a Fluent Icon for it as well? :)
Regarding style, I don't have much of an aesthetic eye, but I think the Segoe UI Symbol for GiftboxOpen F133 could be a good base.

But it's up to you, really. Feel free to disregard this suggestion.
Thank you, in advance.

In this case, is the content of SCOOBE localizable?

Not in this case, since we're using the release notes from the GitHub release page.

In this case, is the content of SCOOBE localizable?

Not in this case, since we're using the release notes from the GitHub release page.

In this case, is the content of SCOOBE localizable?

Not in this case, since we're using the release notes from the GitHub release page.

I bet we'll get some issues about this. @crutkas Any ideas?

@jaimecbernardo Awesome work, I'll have a loot at the icons and PR it against this branch.
Side-question: will .pngs/.gifs rendered as well if they are part of the GitHub release notes?

Can we move WhatsNew page in the nav area down like we have in the settings window with oobe button. (WhatsNew page isn't a module.)
Btw I like the gift box symbol idea.

Is it possible to hide the installer hash?

Is it possible to hide the installer hash?

I think a simple regex can do this.
https://regex101.com/r/bfGeP0/1
But @crutkas will need to make sure that the release note for regex is in the same format.
The release note for 0.55 is ## Installer Hash\r\n\r\n<hash>
while other versions is ## Installer SHA256 Hash\r\n\r\n<hash>

Is it possible to hide the installer hash?

I think a simple regex can do this.
https://regex101.com/r/bfGeP0/1
But @crutkas will need to make sure that the release note for regex is in the same format.
The release note for 0.55 is ## Installer Hash\r\n\r\n<hash>
while other versions is ## Installer SHA256 Hash\r\n\r\n<hash>
As the result, the 0.55's hash won't match this.

Another way is adding the remarks in markdown like
<!-- hide in scoobe -->
## Installer SHA256 Hash

<hash>
<!-- end hide in scoobe -->

the remarks won't show in the release notes on github.com, but will be sent in API and we can replace this with regex.

I like the second idea. Technically we could also do soemthing like
<! - - show in scoobe
Additional information/images
-->

@jaimecbernardo Awesome work, I'll have a loot at the icons and PR it against this branch.
Side-question: will .pngs/.gifs rendered as well if they are part of the GitHub release notes?

@niels9001
Thanks a lot. I've pointed to another repo with images in notes and those were rendered, both pngs and gifs (though gifs took a bit more time to render).

Can we move WhatsNew page in the nav area down like we have in the settings window with oobe button. (WhatsNew page isn't a module.)

@htcfreek
I'll give it a try. It'll have to be a bit fake, though, since I think Navigation View doesn't really allow that out of the box.

@jaimecbernardo Does the contributor links in the release notes work from What's New page?

Every link seems to be working, so those should work as well. Relative links won't work, though.

Is it possible to hide the installer hash?

@Aaron-Junker , I think I'll go with the removing known formats for now, as a "Minimum like-able feature" mindset. I think improving a model makes sense in the future, but it'll have to be designed in coordination with @crutkas .

Can we move WhatsNew page in the nav area down like we have in the settings window with oobe button. (WhatsNew page isn't a module.)

@htcfreek
I'll give it a try. It'll have to be a bit fake, though, since I think Navigation View doesn't really allow that out of the box.

@jaimecbernardo
Nav pane has a footer area we can use.

@jaimecbernardo @htcfreek I can pick up the NavigationView.Footer work as well. I think the proposal is have something like was defined in the spec:

I'll do a PR against this branch to move the NavigationViewItem to the footer.
Another question: do you think it would make sense to open SCOOBE when clicking on this button as well?

@jaimecbernardo @htcfreek I can pick up the NavigationView.Footer work as well. I think the proposal is have something like was defined in the spec:

I'll do a PR against this branch to move the NavigationViewItem to the footer.

Yes. That's the proposal.

Another question: do you think it would make sense to open SCOOBE when clicking on this button as well?

Only having a second link I think. And have in mind that what's new page doesn't make sense if an update is available.

@jaimecbernardo @htcfreek I can pick up the NavigationView.Footer work as well. I think the proposal is have something like was defined in the spec:

Sounds great @niels9001 . Thank you! Let me know if you get issues connecting it around. I'm not sure the kind of tricks that will be needed for that.

Another question: do you think it would make sense to open SCOOBE when clicking on this button as well?

I think it doesn't make much sense. Release notes on GitHub are sort of the "official" place for release notes. While "What's new?" right now just gets the Release notes, it's possible that we do something a bit more accessible in the future.

Added a regex to remove installer hashes from the release notes in this commit: https://github.com/microsoft/PowerToys/pull/16478/commits/8072fd71ec21f407e246621ea10c3fe5eec52b6d

@niels9001 , How do you feel about merging this in for now? Review seems to be done and icon PR can be done against main afterwards. Thank you.

@niels9001 , How do you feel about merging this in for now? Review seems to be done and icon PR can be done against main afterwards. Thank you.

Yes, go ahead :)

What about adding a link to network troubleshooter when there's no connection. It's just that many other MSFT products do this.

What about adding a link to network troubleshooter when there's no connection. It's just that many other MSFT products do this.

I don't think we should overcomplicating things.


chore(FDS-182) Nivo version bump

This PR should resolve some lockfile issues that were present and causing issues in our CI install. It also should give us a warning-free installation, which we want to pass along to our consumers.
Dependencies

This bumps all Nivo packages to 0.67.0
We also alias react-spring to @react-spring/web since Nivo uses it, but the full platform flavor throws warnings about missing platform dependencies, which we do not support. Making sure we are only using the web platform package gets rid of those warnings for our package consumers.

We may need to just stick with link:../ aliasing for our packages so services like Vercel can deploy without having to try to get packages from our package source.


[Bug]: RangeError Maximum call stack size exceeded

Which variants of Embla Carousel are you using?

[ ] embla-carousel (Core)
[X] embla-carousel-react
[ ] embla-carousel-vue
[ ] embla-carousel-svelte
[ ] embla-carousel-autoplay
[ ] embla-carousel-auto-scroll
[ ] embla-carousel-solid
[ ] embla-carousel-auto-height
[ ] embla-carousel-class-names
[ ] embla-carousel-fade
[ ] embla-carousel-docs (Documentation)
[ ] embla-carousel-docs (Generator)

Steps to reproduce
Was not able to reproduce.
Expected Behavior
No error.
Additional Context
Started happening after upgrading from 8.3.1 to 8.4.0.
The issue is happening in updateAndRemoveAccumulatedTime:

Looking at the diff between 8.3.1 and 8.4.0 it seems that this function was introduced in the new release: https://github.com/davidjerleke/embla-carousel/compare/v8.3.1...v8.4.0#diff-8cdac3333b1ac4c0d5bb15d6b666ca387664c871d960fbce4f43704207715deeR47-R51
What browsers are you seeing the problem on?
Chrome, Safari, Microsoft Edge
Version
8.4.0
CodeSandbox
No response
Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues
[X] I agree to follow this project's Contributing Guidelines for bug reports

@jperezr21 thanks for your bug report. Did this happen in production or when your development watch was running?

In production.

@jperezr21 let me know if you find a consistent way of reproducing the problem. Thanks.

Thank you @davidjerleke!

@jperezr21 a bug fix has been released in v8.5.1. Try upgrading and let me know if the problem goes away.


Finetuning code on LIBERO

Thank you for your excellent work!
In the "OpenVLA Fine-Tuning Results" section, results for "Octo fine-tuned" are provided. Is there any code, configuration, and rollout scripts available for fine-tuning Octo with LIBERO data?
Thank you for your time and assistance.

I guess you only need to modify the path to dataset they have converted in the LoRA fine-tuning script. It works for me.

But I am also curious about the fine-tuning configuration to reproduce the results on LIBERO.

For Octo hyperparameters, I just used the official Octo finetuning script, running it on the LIBERO dataset (Octo and OpenVLA use the same data loader, so you don't need to change the format)

Thank you all for your helpful responses and detailed insights!
I’ll try the same as well.


Added my CardUI

read the instructions carefully and then add your card ui with proper formating ok

can you please explain that what mistake I have done?


Update GPU operator to v22.9.0

Summary
Update GPU operator to version v22.9.0, fixing support for 1.25

Testing:
# microk8s kubectl version
WARNING: This version information is deprecated and will be replaced with the output from kubectl version --short.  Use --output=yaml|json to get the full version.
Client Version: version.Info{Major:"1", Minor:"25", GitVersion:"v1.25.2", GitCommit:"5835544ca568b757a8ecae5c153f317e5736700e", GitTreeState:"clean", BuildDate:"2022-09-29T07:02:27Z", GoVersion:"go1.19", Compiler:"gc", Platform:"linux/amd64"}
Kustomize Version: v4.5.7
Server Version: version.Info{Major:"1", Minor:"25", GitVersion:"v1.25.2", GitCommit:"5835544ca568b757a8ecae5c153f317e5736700e", GitTreeState:"clean", BuildDate:"2022-09-29T07:03:08Z", GoVersion:"go1.19", Compiler:"gc", Platform:"linux/amd64"}

# microk8s kubectl get pod -A
NAMESPACE                NAME                                                          READY   STATUS      RESTARTS   AGE
kube-system              calico-node-hqf6l                                             1/1     Running     0          15m
kube-system              calico-kube-controllers-86cc584896-572xc                      1/1     Running     0          15m
kube-system              coredns-d489fb88-qsv7z                                        1/1     Running     0          14m
gpu-operator-resources   gpu-operator-node-feature-discovery-worker-j4bsn              1/1     Running     0          13m
gpu-operator-resources   gpu-operator-5dc6b8989b-6x8gc                                 1/1     Running     0          13m
gpu-operator-resources   gpu-operator-node-feature-discovery-master-65c9bd48c4-9tqtw   1/1     Running     0          13m
gpu-operator-resources   nvidia-driver-daemonset-82rww                                 1/1     Running     0          12m
gpu-operator-resources   nvidia-container-toolkit-daemonset-vsdk4                      1/1     Running     0          12m
gpu-operator-resources   nvidia-cuda-validator-xw7rh                                   0/1     Completed   0          6m59s
gpu-operator-resources   gpu-feature-discovery-x4mgv                                   1/1     Running     0          12m
gpu-operator-resources   nvidia-device-plugin-daemonset-chljh                          1/1     Running     0          12m
gpu-operator-resources   nvidia-device-plugin-validator-vhxr5                          0/1     Completed   0          6m37s
gpu-operator-resources   nvidia-operator-validator-txn7c                               1/1     Running     0          12m
gpu-operator-resources   nvidia-dcgm-exporter-vsj2m                                    1/1     Running     0          12m

# microk8s kubectl -n gpu-operator-resources logs -lapp=nvidia-operator-validator
Defaulted container "nvidia-operator-validator" out of: nvidia-operator-validator, driver-validation (init), toolkit-validation (init), cuda-validation (init), plugin-validation (init)
all validations are successful

Images used:
# microk8s ctr image ls -q | grep nvcr | grep -v sha256:
nvcr.io/nvidia/cloud-native/gpu-operator-validator:v22.9.0
nvcr.io/nvidia/cloud-native/k8s-driver-manager:v0.4.2
nvcr.io/nvidia/driver:515.65.01-ubuntu20.04
nvcr.io/nvidia/gpu-feature-discovery:v0.6.2-ubi8
nvcr.io/nvidia/gpu-operator:v22.9.0
nvcr.io/nvidia/k8s-device-plugin:v0.12.3-ubi8
nvcr.io/nvidia/k8s/container-toolkit:v1.11.0-ubuntu20.04
nvcr.io/nvidia/k8s/dcgm-exporter:3.0.4-3.0.0-ubuntu20.04

We need to back port this to 1.25


Option to easily delete files

Hello,
Do you plan to add an option to be able to directly delete files from the interface?
Thanks in advance.

Nice suggestion, maybe I could add a delete button to each item in the recent uploads list

Done!


"Edit" button on code examples

Based on #35188, this PR adds the "Edit" button on code examples, clicking which will open up stackblitz editor in new tab with the code example in mind.
Here's a video:
https://user-images.githubusercontent.com/47742487/147886526-f8ed5544-d675-49b4-a6d6-20d211a7fc53.mp4

Hey, thanks for the PR, but this has many unrelated changes.

Can you specify the unrelated changes? So I can rectify them

Sorry, I don't have the time to clarify everything. For starters you are not running the lint scripts locally.
Secondly, you are loading the SDK from unpkg. You should use a real CDN and pin the version. And finally, you have hardcoded stuff.

Changed it from unkpg to jsdelivr
I ran the lint scripts and fixed the issue with EOF on _clipboard-js.scss
If you have time, can you specify what you mean by "hard-coded"?

Hi, I will close this PR and make a new one with new changes. I am having some issues in git locally

New PR here #35644


Can't use a fully qualified filename to load config

Scenario/Intent
Load a config from a file which is not in the same path that the CLI using oneview-golang.
Environment Details

OneView SDK Version: v9.1.0
OneView Appliance Version:  9.00.00-0493574
Golang Version: v1.22.1
Platform: Ubuntu 24.04

Steps to Reproduce
Use LoadConfigFile function with a value of configFile parameter equal to /etc/whatever.json
Expected Result
Configuration object loaded from file /etc/whatever.json
Actual Result
Open file error because de LoadConfigFile function prepends the configFile with the path of Caller.

Hi @pyfontan
We went through your PR and understood the issue you are facing. But your PR doesn't seem to solve the issue as we tried running it.
Please let us know if you need any further help on this.
Thanks,
Alisha

Hi @alisha-k-kalladassery
I created the new LoadConfigFilePath  in order to be able to load a config file with a full path.
I didn't want to break the operation of the initial function LoadConfigFile
Hope it helps.
Thanks
Pierre-yves

Thanks @pyfontan .
We checked it and it is working fine. We will approve it and if workflows run fine, We can merge the same.
Thanks
Alisha


fix Volume migration across pools sometimes loses volume

Description

When secondary storage usage is> 90%, VOLUME migration across primary storage will cause the migration to fail and lose VOLUME

3958

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)

Screenshots (if appropriate):
How Has This Been Tested?

code looks good, let's get this tested and applied to 4.13 !

@div8cn can you rebase this commit on 4.13 so we can apply it there. We'll fwd merge it afterwards

@blueorangutan package

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

Packaging result: ✔centos7 ✔debian. JID-1189

closing in favour of #4043


docmaker fails to correctly reset current directory after loading data files

Describe the bug
When processing data files, Docmaker tries to change the working directory to the project directory. This is done to allow dynamic (js) data files to be able to simply open files (fs.readFile etc.) relative to the project directory.
https://github.com/BlameButton/docmaker/blob/8516b9161278e85c053fd81c10b2f1c5c7428f32/src/data.ts#L45-L46
After processing all data files Docmaker tries to reset the path back to its original setting, but it appears getCwd() does not return a full path but a relative path (e.g. foo instead of /home/<user>/Documents/foo) which makes the call to chdir fail.
https://github.com/BlameButton/docmaker/blob/8516b9161278e85c053fd81c10b2f1c5c7428f32/src/data.ts#L52
✖  error     Error: ENOENT: no such file or directory, chdir '/path/to/foo' -> 'foo'

To Reproduce
Steps to reproduce the behavior:

Run docmaker with the configuration below
See error

Expected behavior
It should correctly reset the current directory after loading all data files.
Configuration
docmaker.yaml:
layout: "layout.html"
pages:
  - "*.md"
data:
  - "data.js"

page.md:
# Test
- This is a test

layout.html:
{{ content }}

data.js:
return { foo: "bar" };

Desktop (please complete the following information)

OS / version: Ubuntu 20.04
Docmaker version: 0.1.4

Would adding a basename call help?

uhh no, I don't think so. Somehow getCwd() is returning a relative path, we should probably convert that to a absolute path first


Add obake to global pinning

obake defines a run-export, but it isn't part of the global pinning, and thus new versions don't get migrated as obake gets updated. This leads to situations like https://github.com/conda-forge/pyaudi-feedstock/pull/34 where
- Encountered problems while solving:
-   - package audi-1.9.2-h6e2fe03_0 requires obake >=0.7.0,<0.8.0a0, but none of the providers can be installed

This is since audi gets the run-export (in this case on an old version that hasn't been rebuilt for new libabseil), and so the requirements cannot be satisfied

@bluescarni, I want to add obake to the global pinning, as we keep running into issues with not having it pinned. Could you specify the expected level of ABI-stability? Presumably a 0.8.1 would be compatible with 0.8.0 (though I don't see any patch releases in the history either). So I guess we go with minor-level for now...?

hi @h-vetinari and sorry for the late reply.
Yes ABI is expected to be stable across patch versions. I want to transition obake to a semantic versioning scheme at one point in the future, but even under semantic versioning ABI compatibility would be guaranteed only across patch releases.


Ruby require_relative support

This follows up on issue #326. require_relative links are clickable with this PR but do not always work. For example, the first link on this page tries to resolve a path to a page dedicated to a Ruby gem, but the other two require_relative links work fine. On many other pages such as this one, the links just throw a 404 error in the console.

Thanks man, looks good. Will be part of the upcoming release next week.


Remove single iteration loop

This loop is intentionally designed to only ever execute, which is non-obvious
on first glance. Changing the code to be clearer that the entire data structure
isn't being iterated.
This will also fix importing SPIRV-Tools into Chromium, since their style
checkers are failing the build due to this iterate once construct.

LGTM but it seems that #3513 will solve this issue.

Superseded by #3513


How to setting sitemap.xml path?

As the title suggests, I want to hide sitemaps because I don't want to expose how many records I have.
Maybe add sitemapPrefix to support the multiple sitemaps option.
sitemap: {
    sitemapPrefix: '/go/to',
    sitemaps: {
      posts: {
         // ...
      },
      pages: {
        // ...
      },
    },
  }

result:
https://<IP_ADDRESS>/go/to/sitemap_index.xml

I don't think this is something that will be supported as hiding the amount of records you have is redundant, anyone can query google to see how many indexed URLs exist for a particular domain.


⚠ Minecraft has degraded performance

In 2aac587, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1172 ms

Resolved: Minecraft performance has improved in 1e503e4.


Error while updating property 'src' of a view managed by: RCTPDFViewAndroid

This error is generated when 'react-native-pdf-view' try to open an invalid src file
Please, provide a handler to catch errors like invalid path, or damaged file!
Cheers.

+1


.gometalinter.json enable all doesn't enable all linters

It appears .gometalinter.json enable all doesn't actually enable all linters. My .gometalinter.json has "EnableAll": true, but optional linters like dupl and lll are not run.
Here's my .gometalinter.json and debug output from running with the JSON file with "EnableAll": true, enable-all-json-debug.log, and with the --enable-all flag, enable-all-flag-debug.log. In each case I used gometalinter ./... to test.

.gometalinter.json
enable-all-json-debug.log
enable-all-flag-debug.log

You can see in enable-all-flag-debug.log the dupl linter was enabled because it found duplicate code. No duplicate code was found in enable-all-json-debug.log. I pulled latest to make sure I've got the latest commits. My local repo is on https://github.com/alecthomas/gometalinter/commit/276fcc3b5021666bb60e55519d53c0eb4b36b85d.
I see --enable-all triggers the action enableAllAction, which enables all default linters (https://github.com/alecthomas/gometalinter/blob/master/main.go#L136). I don't see anything based on Config.EnableAll that would trigger a call to enableAllAction. The only reference I see is in a call to replaceWithMegacheck (https://github.com/alecthomas/gometalinter/blob/master/main.go#L384).
If this issue is not caused by operator error, then I have an idea how to implement the behavior and could submit a quick PR for feedback.

I met the same problem.


helm: add service monitor scrape interval config

This commit was related to the service monitor scrape interval option.
I made a change to allow options to be injected via the values.yaml file.
fix: #1633

@lambdanis Thanks Review, I will change it

The CI failures seem to be a CI bug: https://github.com/cilium/tetragon/issues/1646


chore: refactor atem device to state handler SOFIE-2490

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Other information:

I think this is a good time to remove the (already deprecated) makeReady action.

Sure

Should the _initialized property be kept around? Is it significantly different from not being connected?

I am not sure. It has ended up solely being a guard for ensuring that init is called. But perhaps this shouldn't care about that and should assume that it will be? I suppose it is part of the contract of the Device interface that it will be called appropriately.

I like the approach taken here with the unit tests. I think we should be a bit careful with testing external libraries unnecessarily though. i.e. most of what diffStates outputs comes from the atem-state library. Not saying it's too much here either though.

Yeah, I almost did mock atem-state for this file but didnt. I think there is value in some tests, to ensure that things do line up fully, but I suppose the full flow ones in atem.spec.ts cover that.
With #289 I think this file could have more purpose, and could definitely benefit from mocking for some of the tests. That PR should really have some more added here, with the focus being checking the 'diff' parameter being passed to atem-state. I also think it would be appropriate to check the output (in less detail) to ensure that things did/didnt diff as intended. While that will be testing atem-state, it would also be verifying our interpretation of what should happen


Does not report invalid relative links when invalid routing rules are provided

Test Steps
Provide routing rules with regex that do not match any of the invalid relative links in the config file. Then run textlint-rule-no-dead-relative-link.
Expected Results
See report from textlint-rule-no-dead-relative-link stating that the relative link is invalid.
Actual Results
The report is empty with no invalid relative links.

Released in v1.1.1


json_get_real() not accept an interger

Given an input json string:  {"a":1,"b":-2}
In case of trying to call
json_get_real(root, "b", dVal) will returned
"error: b is not a real type"
currently, I can only work around by
if( json_get_real(root, "b", dVal) ){
b = dVal;
}
else if( json_get_integer(root, "B", nVal) ){
b = (float)nVal;
}
since b is real in most of the time, but input json might encode -2.0 just only to be -2,
Does it resonable to allow json_get_real() to work in case of interger?

please just ignore it, I forget json_get_real() is my wrapper function, json_real_value() work for both real and integer input

@alanywlee So you can close the issue.


Fix broken GraphAddCallNode creation.

Somehow a call to GraphAddCallNode with a missing argument got merged and is for some reason only inconsistently breaking builds. This PR adds the missing argument and should resolve build issues.

@tqchen we should probably try to merge this soon to fix builds.

@areusch already rasied https://github.com/apache/tvm/pull/7982


Fixed incorrect initial noise sampling

The precedence of the operators made the square apply to the number of frames instead of the coefficients. The square is correctly applied after the averaging in the original matlab script. This does not make a noticeable difference when the number of initial frames is small (because the noise profile is continuously updated anyway), however it will effectively make the noise profile near 0 when it is larger.

LGTM


use rxjava inner ofType

see http://reactivex.io/RxJava/javadoc/rx/Observable.html#ofType(java.lang.Class)

Can one of the admins verify this patch?

This looks like a good change, thanks! Merging now :D


IE11 CSS calc() 'Unspecified error'

I get an 'Unspecified error' in setPropertyValue, line 2338 when setting property width with value calc(1899.27650582391 + 20.723494176090014px).
I added a try/catch and it copes, but seems to be an obscure IE11 bug here... (like the comment above)?
/* A try/catch is used for IE<=8, which throws an error when "invalid" CSS values are set, e.g. a negative width.
 Try/catch is avoided for other browsers since it incurs a performance overhead. */
if (IE <= 8) {
	try {
		element.style[propertyName] = propertyValue;
	} catch (error) {
		if (Velocity.debug) {
			console.log("Browser does not support [" + propertyValue + "] for [" + propertyName + "]");
		}
	}
	/* SVG elements have their dimensional properties (width, height, x, y, cx, etc.) applied directly as attributes instead of as styles. */
	/* Note: IE8 does not support SVG elements, so it's okay that we skip it for SVG animation. */
} else {
	var data = Data(element);

	if (data && data.isSVG && CSS.Names.SVGAttribute(property)) {
		/* Note: For SVG attributes, vendor-prefixed property names are never used. */
		/* Note: Not all CSS properties can be animated via attributes, but the browser won't throw an error for unsupported properties. */
		element.setAttribute(property, propertyValue);
	} else {
// >>>> NEW CODE
		try {
			element.style[propertyName] = propertyValue;
		} catch (error) {
			if (Velocity.debug) {
				console.log("Browser does not support [" + propertyValue + "] for [" + propertyName + "]");
			}
		}
// >>>> END NEW CODE
//		element.style[propertyName] = propertyValue;
	}
}

A JSFiddle would be useful - but looking at that calc line, it's missing a unit at the front of it - got a feeling you're not using ForceFeeding, and it's grabbing a pixel value but not appending "px" to it...
I'm not really doing anything to v1 while concentrating on v2 development, but if that's right then it's not relevant in v2 simply due to differences in how it works...

Sorry, what do you mean by 'ForceFeeding'?

http://velocityjs.org/#forcefeeding