{"error": "listen tcp :8080: bind: address already in use"}

Describe the bug
I am getting an error 8080 bind in use but the ports i have changed to 9443 on both.
To Reproduce
Steps to reproduce the behavior:

Source v0.107.24
Sync v0.107.26

Expected behavior
Syncing Data
Log Files
FATAL	sync	sync/http.go:77	HTTP server ListenAndServe	{"error": "listen tcp :8080: bind: address already in use"}

Please provide your configuration of agh-sync

This is what i have:
version: "2.1" services:   adguardhome-sync:     image: ghcr.io/bakito/adguardhome-sync     network_mode: "host"     container_name: adguardhome-sync     command: run     environment:       ORIGIN_URL: 'http://IP:9443'       ORIGIN_USERNAME: 'XXXXX'       ORIGIN_PASSWORD: 'XXXXX'       REPLICA_URL: 'http://IP'       REPLICA_USERNAME: 'XXXXX'       REPLICA_PASSWORD: 'XXXXX'       # REPLICA1_AUTOSETUP: true # if true, AdGuardHome is automatically initialized.       # REPLICA1_INTERFACENAME: 'ens18' # use custom dhcp interface name       # REPLICA1_DHCPSERVERENABLED: true/false (optional) enables/disables the dhcp server on the replica       CRON: '*/10 * * * *' # run every 10 minutes       RUNONSTART: true       # Configure sync features; by default all features are enabled.       # FEATURES_GENERALSETTINGS: true       # FEATURES_QUERYLOGCONFIG: true       # FEATURES_STATSCONFIG: true       # FEATURES_CLIENTSETTINGS: true       # FEATURES_SERVICES: true       # FEATURES_FILTERS: true       # FEATURES_DHCP_SERVERCONFIG: true       # FEATURES_DHCP_STATICLEASES: true       # FEATURES_DNS_SERVERCONFIG: true       # FEATURES_DNS_ACCESSLISTS: true       # FEATURES_DNS_REWRITES: true     ports:       - 9443:9443     restart: unless-stopped
I have removed the IP, Username and password. I have tried with an without ports on the Orginal and Replica.

This is probably the API port of agh-sync which is started by default on port 8080.
You probably have already something listening o that port.
The port can be changed with env variable API_PORT.
set it to 0 to disable it

I get not allowed in Portainer if i use API_PORT 0.

Now i get the below:
Versions do not match
Add filter
Error syncing filters
2023-03-16T08:46:02.823Z	ERROR	sync	sync/sync.go:241	Error syncing filters	{"from": "xxxx:xx", "to": "xxxx:xxx", "error": "400 Bad Request(Couldn't fetch filter from url https://block.energized.pro/spark/formats/filter: got status code 404, want 200\n)"}
github.com/bakito/adguardhome-sync/pkg/sync.(*worker).syncTo
/go/src/app/pkg/sync/sync.go:241
github.com/bakito/adguardhome-sync/pkg/sync.(*worker).sync
/go/src/app/pkg/sync/sync.go:186
github.com/bakito/adguardhome-sync/pkg/sync.Sync.func2
/go/src/app/pkg/sync/sync.go:49
github.com/robfig/cron/v3.FuncJob.Run
/go/pkg/mod/github.com/robfig/cron/v3@v3.0.1/cron.go:136
github.com/robfig/cron/v3.(*Cron).startJob.func1
/go/pkg/mod/github.com/robfig/cron/v3@v3.0.1/cron.go:312

Hi @bakito
New Issues now:
ERROR	sync	sync/sync.go:263	Error syncing dns.
"error": "400 Bad Request(bad dhcpv4 configuration: dhcpv4: invalid IP is not an IPv4 address\n)"}
I do not have DHCP Setup so not sure why this is giving me this error.

Can you please enable debug logging and attach the whole debug log?
env variable:
LOG_LEVEL=debug

I'm having the same issue with syncing between nodes:
[custom-init] No custom files found, skipping...
[ls.io-init] done.
2023-03-28T07:11:45.046-0700    INFO    root    cmd/root.go:122 Using config file:/config/adguardhome-sync.yaml
2023-03-28T07:11:45.047-0700    INFO    sync    sync/sync.go:29 AdGuardHome sync        {"version": "0.4.13", "build": "2023-03-07T17:27:31Z"}
2023-03-28T07:11:45.047-0700    INFO    sync    sync/sync.go:55 Setup cronjob   {"cron": "*/10 * * * *", "next-execution": "2023-03-28T07:20:00.000-0700"}
2023-03-28T07:11:45.047-0700    INFO    sync    sync/http.go:52 Starting API server     {"port": 8080}
2023-03-28T07:11:45.047-0700    INFO    sync    sync/sync.go:65 Running sync on startup
2023-03-28T07:11:45.252-0700    INFO    sync    sync/sync.go:113        Connected to origin     {"from": "<IP_ADDRESS>:80", "version": "v0.107.26"}
2023-03-28T07:11:47.353-0700    INFO    sync    sync/sync.go:198        Start sync      {"from": "<IP_ADDRESS>:80", "to": "<IP_ADDRESS>"}
2023-03-28T07:11:47.443-0700    INFO    sync    sync/sync.go:206        Connected to replica    {"from": "<IP_ADDRESS>:80", "to": "<IP_ADDRESS>", "version": "v0.107.26"}
2023-03-28T07:11:48.096-0700    INFO    client  client/client.go:306    Update filter   {"host": "<IP_ADDRESS>", "url": "https://raw.githubusercontent.com/hoshsadiq/adblock-nocoin-list/master/hosts.txt", "whitelist": false, "enabled": false}
2023-03-28T07:11:48.194-0700    INFO    client  client/client.go:317    Refresh filter  {"host": "<IP_ADDRESS>", "whitelist": false}
2023-03-28T07:12:05.014-0700    INFO    sync    sync/sync.go:268        Sync done       {"from": "<IP_ADDRESS>:80", "to": "<IP_ADDRESS>"}
2023-03-28T07:12:05.014-0700    INFO    sync    sync/sync.go:198        Start sync      {"from": "<IP_ADDRESS>:80", "to": "<IP_ADDRESS>"}
2023-03-28T07:12:05.107-0700    INFO    sync    sync/sync.go:206        Connected to replica    {"from": "<IP_ADDRESS>:80", "to": "<IP_ADDRESS>", "version": "v0.107.26"}
2023-03-28T07:12:05.798-0700    INFO    client  client/client.go:306    Update filter   {"host": "<IP_ADDRESS>", "url": "https://raw.githubusercontent.com/hoshsadiq/adblock-nocoin-list/master/hosts.txt", "whitelist": false, "enabled": false}
2023-03-28T07:12:05.902-0700    INFO    client  client/client.go:317    Refresh filter  {"host": "<IP_ADDRESS>", "whitelist": false}
2023-03-28T07:12:23.872-0700    INFO    client  client/client.go:472    Set dhcp server config  {"host": "<IP_ADDRESS>"}
2023-03-28T07:12:23.963-0700    ERROR   sync    sync/sync.go:263        Error syncing dns       {"from": "<IP_ADDRESS>:80", "to": "<IP_ADDRESS>", "error": "400 Bad Request(bad dhcpv4 configuration: dhcpv4: invalid IP is not an IPv4 address\n)"}
github.com/bakito/adguardhome-sync/pkg/sync.(*worker).syncTo
        /home/bakito/go/src/github.com/bakito/adguardhome-sync/pkg/sync/sync.go:263
github.com/bakito/adguardhome-sync/pkg/sync.(*worker).sync
        /home/bakito/go/src/github.com/bakito/adguardhome-sync/pkg/sync/sync.go:186
github.com/bakito/adguardhome-sync/pkg/sync.Sync.func3
        /home/bakito/go/src/github.com/bakito/adguardhome-sync/pkg/sync/sync.go:66

I'm running Adguard Sync within Unraid and added the LOG_LEVEL=debug as an environmental variable. Does the above log include the debug logs or do they go to a file somewhere?

@WMT-IT with debug enabled you should see DEBUG scoped messages, where the requests and body contents are logged.
As there are no such logs, I assume debug log is not enabled. Perhaps the env var si not correctle set, or duplicated.

Hi @bakito
Sorry for the delay please see the below:
2023-04-05T13:08:01.854Z	DEBUG	client	client/client.go:140	do get	{"host": "IP_ADDRESS:9443", "method": "GET", "path": "/blocked_services/list", "username": "USER_ACCOUNT"} 2023-04-05T13:08:01.940Z	DEBUG	client	client/client.go:156	got response	{"host": "IP_ADDRESS:9443", "method": "GET", "path": "/blocked_services/list", "username": "USER_ACCOUNT", "status": 200, "body": "[]\n", "content-type": ["application/json"]} 2023-04-05T13:08:01.940Z	DEBUG	client	client/client.go:140	do get	{"host": "IP_ADDRESS:9443", "method": "GET", "path": "/clients", "username": "USER_ACCOUNT"} 2023-04-05T13:08:02.026Z	DEBUG	client	client/client.go:156	got response	{"host": "IP_ADDRESS:9443", "method": "GET", "path": "/clients", "username": "USER_ACCOUNT", "status": 200, "body": "{\"clients\":null,\"auto_clients\":[{\"whois_info\":{},\"name\":\"ip6-allnodes\",\"ip\":\"ff02::1\",\"source\":\"etc/hosts\"},{\"whois_info\":{},\"name\":\"ip6-allrouters\",\"ip\":\"ff02::2\",\"source\":\"etc/hosts\"},{\"whois_info\":{},\"name\":\"adguardhome-v5.local\",\"ip\":\"<IP_ADDRESS>\",\"source\":\"etc/hosts\"},{\"whois_info\":{},\"name\":\"\",\"ip\":\"IP_ADDRESS\",\"source\":\"ARP\"},{\"whois_info\":{},\"name\":\"\",\"ip\":\"IP_ADDRESS\",\"source\":\"ARP\"},{\"whois_info\":{},\"name\":\"\",\"ip\":\"IP_ADDRESS\",\"source\":\"ARP\"},{\"whois_info\":{},\"name\":\"localhost\",\"ip\":\"<IP_ADDRESS>\",\"source\":\"etc/hosts\"},{\"whois_info\":{},\"name\":\"localhost\",\"ip\":\"::1\",\"source\":\"etc/hosts\"}],\"supported_tags\":[\"device_audio\",\"device_camera\",\"device_gameconsole\",\"device_laptop\",\"device_nas\",\"device_other\",\"device_pc\",\"device_phone\",\"device_printer\",\"device_securityalarm\",\"device_tablet\",\"device_tv\",\"os_android\",\"os_ios\",\"os_linux\",\"os_macos\",\"os_other\",\"os_windows\",\"user_admin\",\"user_child\",\"user_regular\"]}\n", "content-type": ["application/json"]} 2023-04-05T13:08:02.026Z	DEBUG	client	client/client.go:140	do get	{"host": "IP_ADDRESS:9443", "method": "GET", "path": "/access/list", "username": "USER_ACCOUNT"} 2023-04-05T13:08:02.113Z	DEBUG	client	client/client.go:156	got response	{"host": "IP_ADDRESS:9443", "method": "GET", "path": "/access/list", "username": "USER_ACCOUNT", "status": 200, "body": "{\"allowed_clients\":[],\"disallowed_clients\":[],\"blocked_hosts\":[\"version.bind\",\"id.server\",\"hostname.bind\"]}\n", "content-type": ["application/json"]} 2023-04-05T13:08:02.113Z	DEBUG	client	client/client.go:140	do get	{"host": "IP_ADDRESS:9443", "method": "GET", "path": "/dns_info", "username": "USER_ACCOUNT"} 2023-04-05T13:08:02.199Z	DEBUG	client	client/client.go:156	got response	{"host": "IP_ADDRESS:9443", "method": "GET", "path": "/dns_info", "username": "USER_ACCOUNT", "status": 200, "body": "{\"upstream_dns\":[\"https://dns.quad9.net/dns-query\",\"https://dns.google/dns-query\",\"https://dns.cloudflare.com/dns-query\"],\"upstream_dns_file\":\"\",\"bootstrap_dns\":[\"<IP_ADDRESS>\",\"<IP_ADDRESS>\",\"2620:fe::10\",\"2620:fe::fe:10\"],\"protection_enabled\":true,\"ratelimit\":20,\"blocking_mode\":\"default\",\"edns_cs_enabled\":false,\"dnssec_enabled\":true,\"disable_ipv6\":false,\"upstream_mode\":\"\",\"cache_size\":4194304,\"cache_ttl_min\":0,\"cache_ttl_max\":0,\"cache_optimistic\":false,\"resolve_clients\":true,\"use_private_ptr_resolvers\":true,\"local_ptr_upstreams\":[],\"blocking_ipv4\":\"\",\"blocking_ipv6\":\"\",\"default_local_ptr_upstreams\":[\"IP_ADDRESS\",\"<IP_ADDRESS>\"]}\n", "content-type": ["application/json"]} 2023-04-05T13:08:02.199Z	DEBUG	client	client/client.go:140	do get	{"host": "IP_ADDRESS:9443", "method": "GET", "path": "/dhcp/status", "username": "USER_ACCOUNT"} 2023-04-05T13:08:02.286Z	DEBUG	client	client/client.go:156	got response	{"host": "IP_ADDRESS:9443", "method": "GET", "path": "/dhcp/status", "username": "USER_ACCOUNT", "status": 200, "body": "{\"interface_name\":\"\",\"v4\":{\"gateway_ip\":\"\",\"subnet_mask\":\"\",\"range_start\":\"\",\"range_end\":\"\",\"lease_duration\":0},\"v6\":{\"range_start\":\"\",\"lease_duration\":86400},\"leases\":[],\"static_leases\":[],\"enabled\":false}\n", "content-type": ["application/json"]} 2023-04-05T13:08:02.286Z	INFO	client	client/client.go:472	Set dhcp server config	{"host": "IP_ADDRESS:9443"} 2023-04-05T13:08:02.286Z	DEBUG	client	client/client.go:169	do post	{"host": "IP_ADDRESS:9443", "method": "POST", "path": "/dhcp/set_config", "username": "USER_ACCOUNT", "body": "{\"v4\":{\"gateway_ip\":\"\",\"subnet_mask\":\"\",\"range_start\":\"\",\"range_end\":\"\",\"lease_duration\":0},\"v6\":{\"range_start\":\"\",\"range_end\":\"\",\"lease_duration\":86400},\"interface_name\":\"0\",\"enabled\":false}"} 2023-04-05T13:08:02.372Z	DEBUG	client	client/client.go:179	got response	{"host": "IP_ADDRESS:9443", "method": "POST", "path": "/dhcp/set_config", "username": "USER_ACCOUNT", "status": 400, "body": "bad dhcpv4 configuration: dhcpv4: invalid IP is not an IPv4 address\n", "content-type": "text/plain; charset=utf-8"} 2023-04-05T13:08:02.373Z	ERROR	sync	sync/sync.go:263	Error syncing dns	{"from": "IP_ADDRESS:80", "to": "IP_ADDRESS:9443", "error": "400 Bad Request(bad dhcpv4 configuration: dhcpv4: invalid IP is not an IPv4 address\n)"} github.com/bakito/adguardhome-sync/pkg/sync.(*worker).syncTo 	/go/src/app/pkg/sync/sync.go:263 github.com/bakito/adguardhome-sync/pkg/sync.(*worker).sync 	/go/src/app/pkg/sync/sync.go:186 github.com/bakito/adguardhome-sync/pkg/sync.Sync.func2 	/go/src/app/pkg/sync/sync.go:49 github.com/robfig/cron/v3.FuncJob.Run 	/go/pkg/mod/github.com/robfig/cron/v3@v3.0.1/cron.go:136 github.com/robfig/cron/v3.(*Cron).startJob.func1 	/go/pkg/mod/github.com/robfig/cron/v3@v3.0.1/cron.go:312

@HPMSGEN8 do you have the sync features enabled? If so disable them if you do not have dhcp configured.

@HPMSGEN8 do you have the sync features enabled? If so disable them if you do not have dhcp configured.

Hi @bakito
DHCP is Disabled on both of the Instances.

Disable the adguardhome-sync features for dhcp.
FEATURES_DHCP_SERVERCONFIG: false
FEATURES_DHCP_STATICLEASES: false

Disable the adguardhome-sync features for dhcp.
FEATURES_DHCP_SERVERCONFIG: false
FEATURES_DHCP_STATICLEASES: false

Iist this an ENV ?

Yes, please check the readme

Hi @bakito
Thanks a lot for the help here. From the logs we look good now :) This is what it is meant to say once all is done i take it.
2023-04-10T05:33:02.149Z	INFO	sync	sync/sync.go:268	Sync done	{"from": "xxx.xxx.xxx.xxx:80", "to": "xxx.xxx.xxx.xxx:9443"}

Disable the adguardhome-sync features for dhcp.

This solved the issue for me. Thank you very much.


Switching from Linear to Logarithmic axes (WPF)

How to reproduce?
Create a CartesianGraph with the following xaml:
<UserControl x:Class="Ansys.Study.UI.ChartingModule.Views.ChartingView"
xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
xmlns:local="clr-namespace:Ansys.Study.UI.ChartingModule"
xmlns:wpf="clr-namespace:LiveCharts.Wpf;assembly=LiveCharts.Wpf"
mc:Ignorable="d"
d:DesignHeight="300"
d:DesignWidth="300">

<Grid.RowDefinitions>

</Grid.RowDefinitions>

<wpf:CartesianChart Grid.Row="1"
Series="{Binding SelectedChartData}"
LegendLocation="Top"
HorizontalAlignment="Stretch">
wpf:CartesianChart.AxisX
<wpf:Axis Title="X" />
</wpf:CartesianChart.AxisX>
wpf:CartesianChart.AxisY
<wpf:Axis Title="Y" />
</wpf:CartesianChart.AxisY>
wpf:CartesianChart.Style

</wpf:CartesianChart.Style>
</wpf:CartesianChart>

_
This is basically a data trigger bound to an Enum from a Combobox that switches the Y axis from Log to linear and vice versa. Give the graph some sort of series.
When selecting the Scale type with the Combobox, the Y axis does switch between Linear and Log, but after another switch or two, a NullReferenceException is thrown in Axis.UpdateTitle(). It appears that the TitleBlock holding the axis title control is being cleared unnecessarily?
Extra notes
Is there a cleaner way to do this in general, or can the bug simply be prevented in some way?
I am using

Live-Charts version
0.9.5

.Net Version
4.6

Windows
10

ChartingView.zip

Hi, thanks for the feedback.
I'll try to reproduce, well my first impression is why are you so rude with this control? 😟 😄
The truth is that many things are happening behind the scenes. when you change the axis.
As a work around that would be softer to process could be, to have always 2 axis, a regular Axis and your Log axis.
<wpf:CartesianChart>
        <wpf:CartesianChart.Series>
            <wpf:LineSeries Values="1,3,4" ScalesYAt="{Binding Cx}"></wpf:LineSeries>
        </wpf:CartesianChart.Series>
        <wpf:CartesianChart.AxisY>
            <wpf:Axis ShowLabels="{Binding RegularLabels}"></wpf:Axis>
            <wpf:LogarithmicAxis ShowLabels="{Binding LogLabels}"></wpf:LogarithmicAxis>
        </wpf:CartesianChart.AxisY>
    </wpf:CartesianChart>

I have the same issue. I tried it with chart.AxisY.Remove() by remove the linear axis and adding the logarithmic axis, and when switching the second time, I got a NullReferenceException.
So I tried your suggestion. I added two axis to my chart, one linear and the other one logarithmic.
                yAxis = new Axis()
                {
                    Title = yLabel
                };

                yAxisLogarithmic = new LogarithmicAxis()
                {
                    Title = "log",
                    Base = 10,
                    Separator = new LiveCharts.Wpf.Separator()
                    {
                        Stroke = Brushes.LightGray
                    }
                };

when I change it like this:
 if (menuItem.IsChecked)
            {
                chart.AxisY[0].ShowLabels = false;
                chart.AxisY[1].ShowLabels = true;
                foreach(Series series in chart.Series)
                {
                    series.ScalesYAt = 1;
                }
            }
            else
            {
                chart.AxisY[0].ShowLabels = true;
                chart.AxisY[1].ShowLabels = false;

                foreach (Series series in chart.Series)
                {
                    series.ScalesYAt = 0;
                }
            }

I get an exception, once I change the scale.
System.Windows.Media.Animation.AnimationException
HResult=0x80131501
Nachricht = Die Eigenschaft "Top" auf "System.Windows.Controls.TextBlock" kann nicht mithilfe von "System.Windows.Media.Animation.DoubleAnimation" animiert werden. Detaillierte Informationen finden Sie bei der inneren Ausnahme.
Quelle = PresentationCore
Stapelüberwachung:
bei System.Windows.Media.Animation.AnimationStorage.OnCurrentTimeInvalidated(Object sender, EventArgs args)
...
Innere Ausnahme 1:
InvalidOperationException: Von "System.Windows.Media.Animation.DoubleAnimation" kann der standardmäßige origin-Wert "NaN" nicht verwendet werden.
it says "the property "Top" in System.Windows.Controls.TextBlock can not be animated trough "System.Windows.Media.Animation.DoubleAnimation"


Support rootfs WindowsContainerResources.rootfs_size_in_bytes in CRI API

What is the problem you're trying to solve
A new field rootfs_size_in_bytes is added to WindowsContainerResources in CRI. Containerd does not support it yet [2]. Creating this issue to track the effort of supporting rootfs_size_in_bytes.
[1] https://github.com/kubernetes/kubernetes/blob/df4ca2dd9e9fc072f68b1c7b9fd1efdb7a67f593/staging/src/k8s.io/cri-api/pkg/apis/runtime/v1/api.proto#L915
[2] https://github.com/containerd/containerd/blob/bfde58e3cdc0d16b24342886db319a2a2b0964b9/pkg/cri/opts/spec_windows.go#L179
Describe the solution you'd like
WindowsContainerResources.rootfs_size_in_bytes needs to be passed to runtimespec.Spec in [1]
[1] https://github.com/containerd/containerd/blob/bfde58e3cdc0d16b24342886db319a2a2b0964b9/pkg/cri/opts/spec_windows.go#L179
Additional context
No response

I'm sure @dcantah knows who will add this work for you

Hey! We actually do make use of this field already, just in the snapshotter and not in the shim via checking the runtime spec field (we could do either afaict, don't think it matters too much). https://github.com/containerd/containerd/blob/main/pkg/cri/server/container_create_windows.go#L152

Sweet! @ruiwen-zhao - I'm going to close since its resolved but please comment if you need anything else.

sounds good, thanks!


fix quick start examples

Fixed the code examples for react and svelte. They mention vueHook instead of the framework specific hook

It's merged. thanks a lot


"/ is not writable" message emitted on read-only bundle invocations

If bundler is run in a production environment (i.e., on an immutable filesystem and with $HOME unset), it helpfully prints out the following warning on every invocation (even those like bundle exec which should never write to the filesystem)
`/` is not writable.
Bundler will use `/tmp/bundler/home/username' as your home directory temporarily.

Setting $HOME to point to a writable temporary directory suppresses the message. I verified (using strace) that nothing is ever opened in that temporary directory.
I have two issues with this log message:

Warning that / isn't writable on an action that will never write seems extraneous.
Errors should be printed to stderr instead of stdout. There is never an acceptable care to print errors to stdout. Yes, I'm aware that I can export $BUNDLE_ERROR_ON_STDERR but does that really need to be opt-in for relatively-new warnings? Nobody could possibly have been relying on the broken behavior of writing errors to stdout for a warning that didn't even exist two minor versions ago.

This looks closely related to #5371 (which was closed but only for the case where $HOME is unset) and to have been proximally caused by #5385.
It would be swell if bundler (a) would not attempt to write when running bundle exec, and (b) would write this warning to stderr if it really feels the need to emit it.
% bundle version
Bundler version 1.16.3 (2018-07-17 commit 12e73cab7)
% bundle exec ruby --version
ruby 2.3.7p456 (2018-03-28 revision 63024) [x86_64-linux-gnu]

Let me know if you'd prefer that I re-open #5371 in favor of opening a new issue

Errors should be printed to stderr instead of stdout.
This is slated to come out in Bundler 2.

I'd love an option to turn this off, always looks like something is broken ... and it's not even using that home dir :(

FYI one workaround is to use ruby -rbundler/setup instead of bundle exec

I'd love for this to be hidden, if somebody points the way I can help getting it fixed.

I had this issue when running bundler within a docker container as a user from the host system (i.e. started with docker's --user flag), in a dev environment.
Re-exporting the HOME environment variable as the app's root dir (i.e. /app), rather than / suppressed this message. :)

Hi All,
In the apache error log I am getting below exception while running the command "bundle exec rake deploy":
/ is not writable.
Bundler will use /tmp/bundler/home/unknown' as your home directory temporarily. /home/ers/.rvm/rubies/ruby-2.4.0/lib/ruby/gems/2.4.0/gems/bundler-2.0.2/lib/bundler/spec_set.rb:87:in block in materialize': Could not find rake-13.0.1 in any of the sources (Bundler::GemNotFound)
My rake version is 13.0.1 and the command is working fine when I am running it from the CLI mode but when i am calling it from php getting this exception.

Hi All,
My all task is running fine but in the log file, I am still getting:
/ is not writable.
Bundler will use `/tmp/bundler/home/unknown' as your home directory temporarily.
Can you please help me. how to set a different directory rather than using /.


Trap

Trap, big hips, in swimsuit

Yes

Closed due to irrelevant comment. Please provide additional information to reopen the issue.


How to decode transaction_trace.act.data v1.0.0

I tried to upgrade fill-pg to v1.0.0, but found that transaction_trace.act.data is different, and I don't know how to decode it.
Take this for example:
cleos get transaction e96c84c377dde0940095c7c18b6b809ff09a7cd8ded8e01bb70b13d225416912 (this is on Kylin, not mainnet):
which returns:
      "actions": [{
          "account": "blocktivity1",
          "name": "push",
          "authorization": [{
              "actor": "blocktivity1",
              "permission": "push"
            }
          ],
          "data": {
            "block_num": 175269600,
            "timestamp": "2021-07-07T11:12:05",
            "transactions": 1058,
            "actions": 1195,
            "cpu_usage_us": 679178,
            "net_usage_words": 20831
          },
          "hex_data": "e066720a00000000058ce5602204000000000000ab040000000000000a5d0a00000000005f51000000000000"
        }
      ],

Now, if I call curl -H 'content-type:application/json' -d '{"code":"blocktivity1","action":"push","binargs":"5c7845303636373230413030303030303030303538434535363032323034303030303030303030303030414230343030303030303030303030303041354430413030303030303030303035463531303030303030303030303030"}' https://kylin.eosn.io/v1/chain/abi_bin_to_json | jq,
it's decoded correctly.
But in postgres, when I run select block_num, lower(id), (a).receipt.receiver, (a).receipt.global_sequence, (a).act.account, (a).act.name, (a).act.data from (select block_num, id, unnest(action_traces) as a from chain.transaction_trace where block_num = 175277249 and lower(id) = 'e96c84c377dde0940095c7c18b6b809ff09a7cd8ded8e01bb70b13d225416912') as t order by (a).receipt.global_sequence;
the returned data is:
block_num       | 175277249
lower           | e96c84c377dde0940095c7c18b6b809ff09a7cd8ded8e01bb70b13d225416912
receiver        | blocktivity1
global_sequence | 591515500
account         | blocktivity1
name            | push
data            | \x5c7845303636373230413030303030303030303538434535363032323034303030303030303030303030414230343030303030303030303030303041354430413030303030303030303035463531303030303030303030303030

Question is: how do I decode the returned data?

Figured it out, I need to unhex this thing first.
In Golang, something like
d, _ := hex.DecodeString(s)
println(string(d))

d is \xE066720A00000000058CE5602204000000000000AB040000000000000A5D0A00000000005F51000000000000


[COR-656] Test issue for CoreDB team

Hello, I am a test OSS user and I'm opening this issue.
COR-656

test


KAFKA-16448: Handle fatal user exception during processing error

This PR is part of KAFKA-16448 which aims to bring a ProcessingExceptionHandler to Kafka Streams in order to deal with exceptions that occur during processing.
This PR catch the exceptions thrown while handling a processing exception
Jira: https://issues.apache.org/jira/browse/KAFKA-16448.
Contributors
@Dabz
@sebastienviale
@loicgreffier

@sebastienviale -- The PR has merge conflicts and needs to be rebased.

Thanks for the PR. Merged to trunk and cherry-picked to 3.9 branch.


Re-enable running indexstoreDB tests part of Swift pull request preset

This reverts commit df31c7265b80ec1abfd34171f2c007988f5300cf.
rdar://59327164

@swift-ci please smoke test


Can't declare an exchange of a non built-in type

Given a RabbitMQ node with the message deduplication plugin enabled, this should declare an exchange:
rabbitmqadmin declare exchange --name dedup --type x-message-deduplication --arguments '{"x-cache-size": 100}'

However, this command performs PUT /api/exchanges/%2F/dedup with the following content:
{"name":"dedup","type":{"plugin":"x-message-deduplication"},"durable":true,"auto_delete":false,"arguments":{"x-cache-size":100}}

This leads to the following function_clause being logged by RabbitMQ:
Declare exchange error: {'EXIT',
    {function_clause,
        [{rabbit_registry,binary_to_type,
             [#{<<"plugin">> => <<"x-message-deduplication">>}],
             [{file,"rabbit_registry.erl"},{line,48}]},
         {rabbit_exchange,check_type,1,
             [{file,"rabbit_exchange.erl"},{line,149}]},
         {rabbit_channel,handle_method,6,
             [{file,"rabbit_channel.erl"},{line,2566}]}]}}

Thank you for the details. This will also affect the delayed message exchange plugin.


🚧 🎁 code generation

Code generation POC
This is currently the POC for what the generated code will look like.
single entry point
The default use case if for users to have their credentials stored in a .env file or in the environment. Entities can be instantiated directly
const user = new User();

or accessed through an sdk instance
const sdk = new SDK();
const user = sdk.entities.user();

Alternatively, configuration options can be passed to an entity
const user = new User( { apiKey: "ak_1" } );

or passed to an SDK instance, which will set up all entities with that configuration
const sdk = new SDK( { apiKey: "ak_1" } );
const user = sdk.entities.user();
const myAccount = sdk.entities.myAccount();

entity methods
Api methods are accessed through entity instances, rather than an api instance
const user = new User();
user.get();
user.create();
user.update();
user.delete();
user.list();

Entities can also have methods that return foreign keys
user.groups() // returns paginated response of PolicyGroup objects

renames
Each entity has the knowledge to deserialise itself from the api.
An entity can have a dictionary called renames, which contains the api name and the name on the entity.
E.g. for a User entity
public readonly _renames: { [key: string]: string } = {
    is_totp_enabled: "twoFactorAuthentication",
    is_gtc_accepted: "termsAccepted",
    is_marketing_accepted: "marketingAccepted",
    groups: "groupIds",
};

An entity can also have a dictionary which contains the key of every foreign key. This stores an instance of the foreign key and whether it is an array of foreign keys.
E.g. for a User entity
public readonly _foreignKeys: { [key: string]: { [key: string]: EntityBase | boolean } } = {
    loginHistory: {
        type: new LoginHistory(),
        array: true,
    }
};

This is a workaround for typescripts lack of reflection.
static imports
A requirement for the new style SDKs is to organise generated code by tag, to match documentation, but keep imports static so changes to tags do not break peoples code.
The generated folder has an index.ts that exports all the generated entities. The SDK folder then has an entities folder that has an index.ts that exports all the generated classes.
where to look?
example of an entity with renames and foreign keys src/sdk/generatedPOC/accounts/user/user.ts
_fromApi method handles renames src/sdk/common/entityBase.ts
everything in src/sdk/generatedPOC will be generated

Codecov Report

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

@@            Coverage Diff            @@
##             master     #235   +/-   ##
=========================================
  Coverage          ?   75.72%
=========================================
  Files             ?       88
  Lines             ?     2488
  Branches          ?      468
=========================================
  Hits              ?     1884
  Misses            ?      603
  Partials          ?        1

Impacted Files
Coverage Δ

src/sdk/generatedPOC/index.ts
100% <100%> (ø)

...generatedPOC/accounts/loginHistory/loginHistory.ts
100% <100%> (ø)

src/index.ts
100% <100%> (ø)

src/sdk/entities/index.ts
100% <100%> (ø)

src/sdk/generatedPOC/accounts/user/user.ts
33.33% <33.33%> (ø)

src/sdk/client/client.ts
40% <40%> (ø)

...k/generatedPOC/accounts/policyGroup/policyGroup.ts
43.33% <43.33%> (ø)

src/sdk/client/sdkApiBase.ts
45.38% <45.38%> (ø)

src/sdk/common/entityBase.ts
52.17% <52.17%> (ø)

src/sdk/sdk.ts
58.82% <58.82%> (ø)

... and 3 more

Continue to review full report at Codecov.

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


Add Akash to LF-Edge lanscape

Add logo and metadata for Akash

Akash Newtwork
Akash Network is a Decentralized Compute Marketplace and Deployment Platform
**Logo will need to be built
cat- decentralized
homepage_url: 'https://akash.network/'
logo:
crunchbase: 'https://www.crunchbase.com/organization/akash-network'


Document token_ops.py

#996 added a script but the script hasn't been documented.  This issue keeps track of documented, either in README or somewhere reachable from README.

I will do this

This has been done.


Add sample feature flagging system to next.js template

Changes

Add a service class for retrieving flags from evidently
Add example consuming the flags
Updated docs
tests

Context for reviewers

This is followup / related work alongside last week's platform infra changes.

Testing

I'd love some help testing this out against the platform infra.

I meant to just add one inline comment in response to a thread, not sure why it posted an entire review with what seems to be outdated comments. Sorry!


Reduce/restrict scope of the project?

node-core-utils has's been used to land PRs on some WGs from time to time, as well as some projects like llnode. None of those projects really need to follow nodejs/node landing flow though. Based on some comments on https://github.com/nodejs/node-core-utils/pull/469, I'm wondering if we should reduce the scope of this project to only land pull requests on nodejs/node, instead of aiming to land any pull requests on any projects in the org. This should help us reduce some complexity in the code base (such as multiple CI providers) as well as reduce frustration from trying to use it on other repositories (which always need some workaround) and allow us to filter out feature requests that are not targeted at nodejs/node (such as https://github.com/nodejs/node-core-utils/issues/373).
We would need to update some documentations to clarify that, and doing so could be considered a breaking change to the project.

I personally would be in favor of demarcating scope limits along these lines  for the same reasons, as well as to help ensure that decisions around what features we do and don't add (and workarounds) are more clearly enumerated.

I would suggest reopen this.
Currently I have to maintain a fork of this to make life easier.
executing git node metadata https://github.com/nodejs/node-gyp/pull/2285
will resolve to
✖  Failed to get collaborator info from nodejs/node-gyp/README.md
Error: Couldn't find ### TSC (Technical Steering Committee) in the README
    at parseCollaborators (/usr/local/share/.config/yarn/global/node_modules/node-core-utils/lib/collaborators.js:83:11)
    at getCollaborators (/usr/local/share/.config/yarn/global/node_modules/node-core-utils/lib/collaborators.js:61:21)
    at processTicksAndRejections (node:internal/process/task_queues:93:5)
    at async PRData.getCollaborators (/usr/local/share/.config/yarn/global/node_modules/node-core-utils/lib/pr_data.js:77:26)
    at async Promise.all (index 0)
    at async PRData.getAll (/usr/local/share/.config/yarn/global/node_modules/node-core-utils/lib/pr_data.js:61:5)
    at async getMetadata (/usr/local/share/.config/yarn/global/node_modules/node-core-utils/components/metadata.js:20:3)

fixing #373 really need this.
also cc @richardlau @rvagg

Yeah, I do all of node-gyp's metadata manually, that's still in my muscle memory from old-timey nodejs/node commit merging, but it does get pretty tedious and is probably one of the reasons I delay a lot of merging. Does this just need someone to do the work to enable this for alternative repos while not exposing that complexity to the common use-case of nodejs/node maintenance? That's the approach I took with branch-diff and changelog-maker -- they're made for nodejs/node by default (there's some URL inference in there if there's incomplete information) but they're also perfectly usable for any other repo if you have everything in order.


[BACKLOG-15784] Increase performance of data services for DET

Master PR: https://github.com/pentaho/pentaho-kettle/pull/3797
<IP_ADDRESS> PR: https://github.com/pentaho/pentaho-kettle/pull/3820
This change is required for this PR: https://github.com/pentaho/pdi-dataservice-server-plugin/pull/249

@dkincade @mdamour1976

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -f engine/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/src/org/pentaho/di/base/AbstractMeta.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.base.AbstractMeta

Branch Change: -.85%:small_red_triangle_down:
Complexity Change: -.68%:small_red_triangle_down:
Instruction Change: -.37%:small_red_triangle_down:
Line Change: -.56%:small_red_triangle_down:
Method Change: -.63%:small_red_triangle_down:

org.pentaho.di.core.auth.core.impl.DefaultAuthenticationConsumerFactory

Branch Change:  + 7.14%
Complexity Change:  + 7.69%

org.pentaho.di.job.entries.evalfilesmetrics.JobEntryEvalFilesMetrics

Branch Change: -.39%:small_red_triangle_down:
Complexity Change: -.51%:small_red_triangle_down:
Instruction Change: -.08%:small_red_triangle_down:

org.pentaho.di.job.entries.getpop.MailConnectionMeta

Branch Change: -1.00%:small_red_triangle_down:
Complexity Change: -1.41%:small_red_triangle_down:
Instruction Change: -.30%:small_red_triangle_down:

org.pentaho.di.job.entries.sql.JobEntrySQL

Instruction Change:  + 3.28%
Line Change:  + 3.87%

org.pentaho.di.job.entries.waitforsql.JobEntryWaitForSQL

Branch Change: -.66%:small_red_triangle_down:
Complexity Change: -.93%:small_red_triangle_down:
Instruction Change: -.13%:small_red_triangle_down:

org.pentaho.di.trans.steps.checksum.CheckSumMeta

Instruction Change:  + .11%

org.pentaho.di.trans.steps.combinationlookup.CombinationLookupMeta

Instruction Change:  + .04%

org.pentaho.di.trans.steps.datagrid.DataGridMeta

Instruction Change: -.09%:small_red_triangle_down:

org.pentaho.di.trans.steps.dbproc.DBProcMeta

Instruction Change: -.11%:small_red_triangle_down:

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change:  + 1.04%
Complexity Change:  + 1.05%
Instruction Change:  + .14%
Line Change:  + .37%

org.pentaho.di.trans.steps.ifnull.IfNullMeta

Branch Change: -2.00%:small_red_triangle_down:
Complexity Change: -1.89%:small_red_triangle_down:
Instruction Change: -.17%:small_red_triangle_down:

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change: -.06%:small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -1.18%:small_red_triangle_down:
Complexity Change: -1.35%:small_red_triangle_down:

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change:  + .07%

org.pentaho.di.trans.steps.setvalueconstant.SetValueConstantMeta

Instruction Change: -.14%:small_red_triangle_down:

org.pentaho.di.trans.steps.sort.SortRowsMeta

Instruction Change: -.08%:small_red_triangle_down:

org.pentaho.di.trans.steps.symmetriccrypto.symmetriccryptotrans.SymmetricCryptoTransMeta

Branch Change: -3.33%:small_red_triangle_down:
Complexity Change: -1.85%:small_red_triangle_down:
Instruction Change: -.28%:small_red_triangle_down:

org.pentaho.di.trans.steps.systemdata.SystemDataMeta

Branch Change:  + 2.04%
Complexity Change:  + 2.04%
Instruction Change:  + .32%
Line Change:  + .81%

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change: -.91%:small_red_triangle_down:
Complexity Change: -.82%:small_red_triangle_down:

org.pentaho.di.trans.steps.uniquerows.UniqueRowsMeta

Instruction Change: -.18%:small_red_triangle_down:

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change:  + 1.79%
Complexity Change:  + 1.19%
Instruction Change:  + .07%


Change the default changelog entry for release sync

TODO:

[x] Update the docs
[x] Inform via comments in packit-service

RELEASE NOTES BEGIN
As discussed and agreed on in https://pagure.io/packaging-committee/issue/1339,
we changed the default behaviour for changelog entry generation to comply with the packaging guidelines. The default changelog entry will be - Update to version <version>.
Users can still configure the behaviour using custom commands in
changelog-entry action or using copy_upstream_release_description config option.
RELEASE NOTES END

recheck


Set permissions for codeql workflow caller

Current dependencies on/for this PR:

main

PR #231   👈

This comment was auto-generated by Graphite.


kmymoney: 5.0.4 -> 5.0.5

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/kmymoney/versions.
meta.description for kmymoney is: '"Personal finance manager for KDE"'.
meta.homepage for kmymoney is: '"https://kmymoney.org/"

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
/bin/sh passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 5.0.5 with grep in /nix/store/jwvzyxg2bi3cd483xpd3jlpv9gz7k426-kmymoney-5.0.5
directory tree listing: https://gist.github.com/9de9739d94e7db366a3cb0e9b4030584
du listing: https://gist.github.com/d6eb8765ea863ed108aa70b8cff2ffd2

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)
1 package rebuild(s)
1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
kmymoney

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/jwvzyxg2bi3cd483xpd3jlpv9gz7k426-kmymoney-5.0.5 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A kmymoney https://github.com/r-ryantm/nixpkgs/archive/db7a7d626a905bd36abca9397d6a27e669d60b2b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/jwvzyxg2bi3cd483xpd3jlpv9gz7k426-kmymoney-5.0.5
ls -la /nix/store/jwvzyxg2bi3cd483xpd3jlpv9gz7k426-kmymoney-5.0.5/bin

Thanks @mmahut! :tada:


🚧Replace UI control events

Complete The Item Below

[X] I have updated the title without removing the 🚧 emoji.

Description
Replace all UI control events that use EventHandler delegate to either Action or Func
Acceptance Criteria

[ ] ControlBase class EventHandler<T> types replaced with other equivalent types

Note This will take care of the Button and Label controls automatically.

ToDo Items

[X] Change type labels added to this issue.  Refer to the Change Type Labels section below.
[X] Priority label added to this issue.  Refer to the Priority Type Labels section below.
[X] Issue linked to the correct project (if applicable).
[X] Issue linked to the correct milestone (if applicable).
[x] Draft pull request created and linked to this issue (only required with code changes).

Issue Dependencies
No response
Related Work
No response
Additional Information:
Change Type Labels

Change Type
Label

Bug Fixes
🐛bug

Breaking Changes
🧨breaking changes

New Feature
✨new feature

Workflow Changes
workflow

Code Doc Changes
🗒documentation/code

Product Doc Changes
📝documentation/product

Priority Type Labels

Priority Type
Label

Low Priority
low priority

Medium Priority
medium priority

High Priority
high priority

Code of Conduct

[X] I agree to follow this project's Code of Conduct.

After further research, it was discovered that this will give no value to how it is being used.
Issue closed without changes


Bug: Merge schemas with required properties

When using ajv-merge-patch module...
Given schema1
{
  type: 'object',
  properties: {
     properties: {
        foo: { type: "string" }
      },
  },
  required: ["foo"]
}

And given schema2
{
  $merge: {
    source: schema1,
    with: {
      type: 'object',
      properties: {
        bar: { type: "string" }
      },
      required: ["baz"]
    }
  }
}

I expect foo and bar to be required in schema2.
Instead, only bar is required.

$merge follow the rules of JSON-merge-patch spec - arrays are not supposed to be extended by it.


⚠ Bofrost.de has degraded performance

In 2540a89, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 919 ms

Resolved: Bofrost.de performance has improved in 123a9ec after 1 hour, 10 minutes.


[Bug] test-runner not running with yarn 3, it doesnt show any output in terminal

Describe the bug
run test-storybook
no output in terminal, and it laggy when you try to cancel process (ctrl + c)
Expected behavior
tests run
Environment

OS: MacOS
Node.js version: 16
Yarn version: 3

React 17
Storybook version 6.5

Hey @AleksanderSenkevich could you please share a reproduction? We've been using the test runner with yarn 3 without issues so far.

I have the same issue. I'll try and see if I can create a reproduction that I can share with you.

@yannickbuntsma Interesting issue. I am experiencing the same thing. It seems like your example is also experiencing a memory leak.
Running this via the CLI got me thinking to use the --verbose option, but it doesn't output anything. That makes me wonder that the issue arrises before the runner has the chance to run any of the tests.
I'll try to investigate it further, but it seems like a compatibility issue with one of the packages or tools.


Add a missing bullet point in the HACS installation section

This pull request adds a missing bullet point in the HACS installation section.

coverage: 100.0%. remained the same
when pulling 2e1a29f9adafd68aaeb42ae29f5c731c83fbabd7 on add_missing_bullet_point_in_hacs_installation
into 23983872e58ff32ab181897ce55a9425df7db0ec on master.


Tour spanning multiple components

Hi @NMilicic , I was wondering if this is possible, as I did not see it in the docs.
I have a tour that spans a parent component and its child component. For e.g. Step 1 is anchored to a div in the parent component - on clicking "Next", Step 2 (anchored to a div in the child component) should be shown - can a tour like this be implemented using this package?
Thank you for your help!

Am interested in this as well. This is why I gave up using intro.js


Ensure PodDisruptionBudget is configured properly

I started a 3-node cluster, introduced an problematic parameter value in values.yaml, ran helm upgrade..., and the first node went down and failed to start properly. With 1 node down, the statefulset then brought down the 2nd node (which also failed in the same way). I expected only 1 node to be taken down due to this in values.yaml:
statefulset:
  budget:
    maxUnavailable: 1

The value above gets used in the PodDisruptionBudget. I also see a warning in the logs:
52m         Warning   CalculateExpectedPodCountFailed   poddisruptionbudget/redpanda   Failed to calculate the number of expected pods: statefulsets.apps does not implement
the scale subresource

This was due to not having readiness checks. This should be solved. Please reopen if this problem is encountered again.


Added support for homogenous multiplication (#140)

Added support for multiplication of matrices with vectors with homogenous coordinates

Need to add doc for this.


Create a standard format for writing design decision posts

arc42 suggests using a format called Architectural Decision Record that looks like this:

Title: These documents have names that are short noun phrases.
Context: This section describes the forces at play, including technological, political, social, and project local. These forces are probably in tension, and should be called out as such. The language in this section is value-neutral. It is simply describing facts.
Decision: This section describes our response to these forces. It is stated in full sentences, with active voice. "We will …"
Status: A decision may be "proposed" if the project stakeholders haven't agreed with it yet, or "accepted" once it is agreed. If a later ADR changes or reverses a decision, it may be marked as "deprecated" or "superseded" with a reference to its replacement.
Consequences: This section describes the resulting context, after applying the decision. All consequences should be listed here, not just the "positive" ones. A particular decision may have positive, negative, and neutral consequences, but all of them affect the team and project in the future.

I'll make a post about this in the community decisions section and then I'll make a template for it.
https://adr.github.io/madr/

Decided on a slight modification of this text.


forgive, gata

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[f,g]

/trunk merge


Feature Request: Replace the hard coded default secrets with autogenerated ones

There is already a level of autogeneration of secrets, but there are still db passwords hardcoded in the configurations.
It seems to be easier to describe how to extract those from the cluster like in the gitlab manual and have all of those autogenerated

Yeah makes sense.

Ok been looking into this. Since we use either the existing MariaDB Chart or and external DB we can't generate random strings here, since we don't control the mechanism that would set the password.
What we could do is set the defaults to empty.


Makefile doesn't install m-concurrent.h

It looks that the actual Makefile doesn't install this recent header (others?). This hits the latest 0.2.0 release to it is buggy: maybe it is better to release a new minor release to fix this (IMHO).

You are right (I have checked all other headers). It has been fixed in master and a new release 0.2.1 has been made.
Thanks


[editor]: Support ttf in preview

Please add support for .ttf files to the editor preview.
Generated by adamraichu.zip-viewer

(Duplicate of #95.)


Problem adding manifest link in header

This is probably because of some module used, but hopefully someone can help me out.
I am making a web app and I need to link in a manifest file.
On a vanilla meteor I can do:
<head>
  <title>test</title>
  <meta name="mobile-web-app-capable" content="yes">
  <link rel="manifest" href="/manifest.json">
</head>

When I do the same however on a fresh meteor-starter, for some reason the meta name="mobile-web-app-capable" and the link rel="manifest" are removed from the <head> section.
When I add a link rel="icon" however, it is not removed, which is the expected behavior.
So what makes those head tags disappear? Should I set somewhere what head tags are whitelisted?
Right now I use this workaround, but it is ugly:
Template.masterLayout.onRendered(function() {
  $('head').append('<link rel="manifest" href="/manifest.json">');
  $('head').append('<meta name="mobile-web-app-capable" content="yes">');
});

Are you doing this in client/index.html file? If not try edit <head> tag in that file.

Yes, I do it in client/index.html.
Just try it on a fresh install. You'll see these tags are ignores.

Tags are in the markup returned from the server (you can see it with "View Page Source" on chrome) but disappear when inspected with chrome dev tools. Strange but it looks for meteor or chrome dev tools issue.

You're right.
I just checked and I can confirm this.
This is really strange. In Firefox I see exactly the same behavior. So probably it's not the dev tools.
Another strange thing is that I see the <title>tag twice in de header. Maybe it's related.

I'm getting a strange behavior too on using:

In the console, it reads:
"Manifest parsing error: Line: 1, column: 1, Unexpected token."
So i tried removing the manifest file and it threw the same error regardless of the file being present or not.
Then I open up /manifest.json from the dev tools, and it seems meteor is responding to /manifest.json with the application template itself.
So far I have no idea why I'm not able to load the manifest.json, I just started learning meteor last week. Am i missing out something?

The package manuelschoebel:ms-seo is messing with this. I found in my own project it is removing the link from the header before the browser gets it.

I'm not using this package but just had similar, if not the same, issue where link rel='manifest' gets removed. As JProgrammer had mentioned, the issue was caused by manuelschoebel:ms-seo package. To fix the issue, you can try to use this option: https://github.com/DerMambo/ms-seo#using-ignore

The solution here is to add the manifest details to the default config for the package.
  // set default config for SEO package
    return SEO.config({
        title: "",
        meta: {
            title: "",
            description:
                "",
            site_name: "",
            viewport: "",
        },
        og: {
            title: "",
            site_name: "",
        },
      link: [
        {
          home: "",
        },
        {
          rel: "manifest", href: "/manifest.webmanifest"
        },
      ]
    });


Github allowed tags list?

Does anyone happen to have a Github allowed tags+attrs list? I am doing something like this:
const marked = require('marked');
marked.setOptions({ sanitize: true });
const clean = marked(markdown);

However this won't allow for ANY html. The alternative is manually sanitize it with this package:
const marked = require('marked');
const sanitize = require('sanitize-html');
const dirty = marked(markdown);
const clean = sanitize(dirty, { allowedTags: ??, allowedAttributes: ?? });

But I'd somehow need to have a list of default allowedTags and allowedAttributes from Github. I could do some github testing, but preferably I'd use a list from someone who did this before not to repeat this work.

Never mind, I changed the way I'm looking at this and found this package based on sanitize-html:
const marked = require('marked');
const Renderer = require('marked-sanitized')(marked.Renderer);
marked.setOptions({ renderer: new marked.Renderer() });
const clean = marked(markdown);


[mc_log_ui] Fix various issues (animations, saving, plot limits, default style, etc)

This PR fixes multiple issues with mc_log_ui and in particular the animated plots:
Major:
Allow to generate animated plots with missing data.
Previously, trying to plot an animation with non-continous data (e.g. repeatedly adding/removing a task) would crash. Looking at the original PR (https://github.com/jrl-umi3218/mc_rtc/pull/13), it seems that the intent was to first generate a time range corresponding to valid data, and then generating a live plot for this time range. This is however not very intuitive, and still not crash-proof in case there are multiple curves displayed.
The new implementation:

Looks for the first/last non-nan data amongst all displayed curves to automatically find the suitable time-range for the animation
Fixes the computation of plot limits, notably for XY plots (previously only the left axis was taken into account). In this case, the x axis is shared between the left/right plots (its limits are the min/max value of all left and right curves). The left/right y axes can be scaled independently.
The axes-locking feature still works as expected
Margins between the curves and the figure axes is now handled consistently in all cases.

An example is worth a thousand words, here is an animated plot of walking showing the measured ZMP and feet limits, along with the right swing foot height. Note that the plot starts at t=4s automatically (there is no data before that as the walking hasn't yet started), and that the green curve (swing foot) is not continuous.
https://user-images.githubusercontent.com/67139/108188332-cddcf900-7152-11eb-95b0-015425c25206.mp4
Another example for XY plots (not very meaningful here, showing simultaneously a joint against another joint and the x vs y components of the CoM):
https://user-images.githubusercontent.com/67139/108188776-522f7c00-7153-11eb-854f-fb4aa8a033e3.mp4

[update] Add sliding windows: 56ad31d implements sliding windows, aka "only show the last N seconds" of an animation

https://user-images.githubusercontent.com/67139/108198247-bdcb1680-715e-11eb-9682-591ae5e90778.mp4
Minor:

Prevent RPY from flipping between 0 and 180deg. This uses the atan(tan(x)) trick to remap the angle between ]-PI, PI[
Saving animations:

Add .mp4 extension by default (makes matplotlib crash if saving as anything else than mp4)
Prevent saving empty animation with a dialog error message
Remember the last used save directory (during the current session)

Use a color set without invisible yellow color by default (can still be changed from the style menu)
Double default line size

@mmurooka Can you check that this fixes the issues you were experiencing?

@mmurooka I just added the ability to have sliding window in animated plots as well. You can optionally let it find the best fit for the plot limits online, or fix the axes. Do you mind trying on the use-cases you had in mind for your paper, and see how it does?

@mmurooka Can you check that this fixes the issues you were experiencing?

@arntanguy, thank you for the great update!
I have confirmed that this PR works correctly on my side (animations that could not be displayed before due to a NaN error can now be displayed, etc.).
The sliding window animation is also great. This time I prepared a dedicated script for my video, but it seems that mc_log_ui can be used from the next time.

bd22677 implements a progress bar display and live pre-visualization while saving animations.
The actual implementation is a bit tricky, as matplotlib's FuncAnimation.save(...) function blocks the GUI event loop while saving. However for every frame to render it calls our own rendering function animate(self, frame), so I ended up manually calling QApplication.processEvents() there which ensures that the GUI remains responsive. As we don't want any side effect from the user using the GUI while it is saving, the progress bar is displayed as a modal window on top and cannot be closed during the saving process. Note that it is impossible to cancel the saving process once it has been started: the rendering is managed by matplolib's save function and I couldn't find a way to ask it to stop (and running it in a thread is not an option).
I'll look into the range/rotation continuity issue next week.

@mehdi-benallegue 4f31cd7 fixes the issue you were having about the custom plot labels not being conserved when starting/stopping/saving an animation.

Can you double check the range issue?

You are right and this change is not behaving as intended. I retrieved it from an older plotting script that I used to use to solve this joint-angle flipping behavior. But upon investigation, this does indeed not work properly in the general case. I've reverted the change 5f978d4. As I don't have a quick clean implementation ready at hand, I think this issue can be left for a future PR.
@gergondet If you agree with the latest changes, I believe we can go ahead and merge this.

Good Job !


Head Master seems to have a bug on the connection list page

There is no way to edit the connections? The "pencil" and "trash" icons are missing from each row.

Can't reproduce... I feel like someone reported a similar issue recently and might have found a solution. Re-installing flask-admin to >=1.4.0 (from memory)?

This is resolved. It seems that as root (on Mac), I had flask-admin at 1.2.0. Whenever I did pip freeze | grep -i flask-admin, even when sudo'd, I only saw 1.4.0.
Anyway, I uninstalled 1.2.0 and installed 1.4.0 and it works.

FYI @askeys


Shape file opens in 1.4, but not in 1.6 or 1.7

This issue was imported from CodePlex
mlago wrote 2014-10-07 at 18:56
Just to report that I can open the attached shape file by using Visual Studio 2010 and the DotSpatial 1.4.
However, when I try to use the versions 1.6 or 1.7, the function "FeatureSet.Open" throw an exception.
Thanks, Marcelo.
jany_ wrote 2015-03-06 at 13:03
Hey Marcelo,
using Visual Studio 2013 and DotSpatial changeset 75590 I opened your file without an exception by callingFeatureSet.Open(@"d:\Downloads\Observation_Station_dfs0_files_list.shp");What exactly did you do to produce the exception?
mbayles wrote 2015-06-09 at 20:21
If you are still having problems with this issue can you upload the files again?
mlago wrote 2015-06-10 at 15:15
Dear mbayles,
I have tried again by replacing the 1.4 dll files by the 1.7 dll files, and reproduced the error. The exception message is:
The type initializer for 'DotSpatial.Data.DbaseLocalRegistry' threw an exception.
The shape files are attached again.
Thanks, Marcelo.
mbayles wrote 2015-06-10 at 19:51
Can you email one of the files directly to me? Codeplex is having some kind of issue with my computer and I can't download the attachments.
mlago wrote 2015-06-10 at 20:40
What is your email? or just send me an empty email to my email address if you prefer.
mbayles wrote 2015-06-10 at 21:22
I got CodePlex to give me the files. I had no trouble using FeatureSet.Open to open your files. Have you tried downloading DotSpatial again? Maybe one of the files is corrupted.
mlago wrote 2015-06-10 at 22:15
I am using ".Net Framework 4" inside Visual Studio 2010. Do you think this matters?
mbayles wrote 2015-06-10 at 23:05
I just tried it with .Net Framework 4 and Visual Studio 2010 and it worked just fine. Have you done anything else to your program?
mlago wrote 2015-06-11 at 00:16
No luck after following the steps:

I downloaded again the 1.7 release of DotSpatial. 2) I copied/replaced the dlls into the VS project folder. 3) I deleted the reference to the dlls in the project and add them again pointing to the new copied dlls4) Rebuild solution. After this, the windows
application runs, but throws the exception while trying to open this shape file and also other shape files. So, it does not seem to be a problem of this particular shape file.6) Copy back the 1.4 version dll files and rebuild solution. After this, shape files
are opened without problem.

If you want to take a closer look, we can try to share screens by using Skype (ID= marcelo.e.lago)
Thanks, Marcelo.
mbayles wrote 2015-06-11 at 01:12
I can't have Skype on this computer. However, can you send me your project? You can use
https://bft.usu.edu/ and my email is
<EMAIL_ADDRESS>mbayles wrote 2015-06-11 at 18:59
I opened your project and the file loaded just fine. I made sure that the 1.7 dlls you included were the ones referenced. I'm out of ideas about why you can't get it to work on your end.
mlago wrote 2015-06-11 at 21:02
I can send you a WebEx meeting invitation if you still want to see my screen ...
mbayles wrote 2015-06-11 at 22:43
Sure that might help.

Works in 1.9


Move source files around

Given that most source editing is the manually written code in Google.Api.Gax, it makes sense for those files to be at the top-level,
and all the generated code be more nested.
We could potentially move the generated folder to be a peer of src, but that would be a little odd.
(The content of the source files have not changed.)

Or we could use two top-level directories, "generated" and "hand-coded (or whatever)"?
This more cleanly separates the two, although I'm not sure it's better.

Yes, I'd considered that - but I think I prefer this approach. Aside from anything else, the location of the files in src mirrors the location of tests in test - where obviously there aren't any tests for generated code.

LGTM


IridiumSBD issues and improvements

if enabled (SYS_COMPANION 419200) and the driver fails to start (bad wiring, etc) the mavlink module will hard fault somewhere in ioctl code

needs settings for when to transmit

I'm thinking of adding a bitmask for transmitting only if: armed, in air, main datalink lost, etc

@bartosz-wawrzacz FYI

Hey, this issue has been closed because the label status/STALE is set and there were no updates for 30 days. Feel free to reopen this issue if you deem it appropriate.
(This is an automated comment from GitMate.io.)

@dagar is this addressed somehow by @acfloria PR?


Decision Service cannot be recognized in output columns of decision table

It relates to https://github.com/apache/incubator-kie-drools/issues/6058

Decision Service cannot be identified inside the input/output column of decision table.
DMN Validator returns DECISION_TABLE_ANALYSIS_ERROR: Skipped Decision Table Analysis of table 'PayersByCountry' because: Name of function is not instance of NameRefNode!FunctionInvocationNode{CountryAD.CountryAD(Input)}
The same decision as FEEL or Context can be recognised DS functions without any issues.

@dmitry-buykin thank you for reporting the tickets, we are currently checking if both repositories are affected. Just FYI we may close one of your tickets accordingly if only one repository will need the fix to be opened for it.


ev6 stop&go bug

Describe the bug
Hello.
I'm using the Sunny Pilot well.
my car is kia ev6 with hda2 / 2022
However, if you restart after stopping, there will be a bug.
"Can Bus Disconnected:Likely Faluty Cable"
I thought it was a cable problem, so I replaced all the cables
Occasionally occurs when restarting after stopping.
What should I do?
I'm sorry to ask for your help every time.
Thank you.
Which car does this affect?
ev6 with hda
Provide a route where the issue occurs
I don't know
openpilot version
relase-c3/staging-c3/dev-c3
Additional info
No response

This is an upstream issue that is also affecting some HKG CAN-FD models. comma is actively fixing this issue. Once the fix is implemented in upstream, it will be merged in sunnypilot. Feel free to track the issue linked below for updates to this issue:

https://github.com/commaai/panda/issues/1452

thank you
have a nice day :)


Filter many-to-many relationships

Hello
I am having troubles with filtering many-to-many relationships. Is there a code snippet somewhere to understand how this should be done?
Many thanks in advance.

@mrjoes could you please share some documentation on this?

Possibly here, for colums.


KYLIN-4013 Only show the cubes under one model

Can one of the admins verify this patch?

Pull Request Test Coverage Report for Build 4647

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.01%) to 28.216%

Files with Coverage Reduction
New Missed Lines
%

core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java
4
77.81%

Totals

Change from base Build 4633:
0.01%

Covered Lines:
23257

Relevant Lines:
82424

💛  - Coveralls

Codecov Report

Merging #707 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #707      +/-   ##
============================================
- Coverage     25.74%   25.73%   -0.01%
+ Complexity     6014     6012       -2
============================================
  Files          1385     1385
  Lines         82424    82424
  Branches      11557    11557
============================================
- Hits          21216    21210       -6
- Misses        59167    59170       +3
- Partials       2041     2044       +3

Impacted Files
Coverage Δ
Complexity Δ

...he/kylin/dict/lookup/cache/RocksDBLookupTable.java
72.97% <0%> (-5.41%)
6% <0%> (-1%)

...rg/apache/kylin/cube/inmemcubing/MemDiskStore.java
69.3% <0%> (-0.92%)
7% <0%> (ø)

...a/org/apache/kylin/dict/Number2BytesConverter.java
81.74% <0%> (-0.8%)
17% <0%> (-1%)

Continue to review full report at Codecov.

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


Does the image have to be on a website?
I saw according to the example that a link to the url is needed.
Is it possible to send an image located in a network folder?

I'm looking for the same thing
Not successfuly on local netwrok :/

I can send an image that is on the Internet (url: "https://www.etcetera
I can send an image from my local network (url: "http://<IP_ADDRESS>/VRAX/{PHOTO.JPG
But how can I send an image that is on the raspberry where my HomeAssistant is running?

image has to be accessable via the internet
use the www folder (https://YOUR_PUBLIC_IP/local/image.jpeg)


showing type error between numeric float and double variable

Hello,
I think this extension is really great, but the type error hovering below bothers me a little:

Here, b is defined as a double variable.
This incompatibility might be caused by regarding numerical values as a float type in pure python.
I would appreciate it if you would take it into account.
Thank you in advance.

Hi,
These errors are from pyright configuration in pyrightconfig.json or a [tool.pyright] section in pyproject.toml. Since this extension does not take into account most of the possible settings, this extension will now no longer use options for pyright until they can be implemented correctly. Should be fixed in 0.3.2.


Support for packing/unpacking

Are you planning to add support for packing/unpacking?

It is something that I've always want to do. And also RPC support. But from the experience of last year, I don't think I'll have time to do it soon. So patches are welcome : )

Have you had any opportunity to look at packing/unpacking? Size difference is quite substantial for some objects.


improve UX by using file when creating k8s secrets in samples

Closes #64

Thanks for taking my PR !


Support for Distil-Whisper

Hey @guillaumekln! Thanks for this fantastic resource. We're looking at supporting the Distil-Whisper checkpoints in faster-whisper.
The checkpoints are fairly easy to convert: we just pin the number of decoder layers to 2 always, and load the 32 encoder layers.
For inference, we found a chunk length of 15-20s to be optimal for WER performance of the distilled model, see Table 23 of the paper:

Would you be open to a PR allowing the user to specify the chunk length and also the maximum generation length? This would enable full support of Distil-Whisper in faster-whisper!

This would be amazing, one step closer to something that feels realtime*ish

Is the distilled large-v2 model still multilingual or does it lose that attribute due to how distilling was done?

Is the distilled large-v2 model still multilingual or does it lose that attribute due to how distilling was done?

It was trained on English audio only so it most probably lost its multilingual capabilities.

Is the distilled large-v2 model still multilingual or does it lose that attribute due to how distilling was done?

From their repo: "Note: Distil-Whisper is currently only available for English speech recognition. Multilingual support will be provided soon."

Waiting for a multilingual model for this task. Looking forward to it

Subscribing for updates!

can't wait to test it, this will be awesome.

hi @sanchit-gandhi. FYI, @guillaumekln's account seems inactive since September, which correlates with the moment he moved from his former company. I don't know if he plans to continue maintaining this repo or if other users such as OpenNMT devs (cc @vince62s @homink @nguyendc-systran) have ownership on the repo or forked it. Maybe consider forking it yourself under HF's Github namespace?

Hi, I confirm that I'm no longer actively maintaining this repo but other people can still make it move forward. Please ping @nguyendc-systran to merge changes in faster-whisper. For anything related to CTranslate2, please ping @vince62s.

Hi, I confirm that I'm no longer actively maintaining this repo but other people can still make it move forward. Please ping @nguyendc-systran to merge changes in faster-whisper. For anything related to CTranslate2, please ping @vince62s.

Great job on Ctranslate2 and faster-whisper, glad I came across it awhile ago now...and good luck in the future.

FYI, distil-whisper should now be supported by CTranslate2: https://github.com/OpenNMT/CTranslate2/releases/tag/v3.21.0
We "just" need to adapt faster-whisper in order to have faster-distil-whisper :)

FYI, I made a PR: https://github.com/guillaumekln/faster-whisper/pull/557 to make it compatible with distil-whisper, hope it helps.


Cannot display CJK properly on linux.

OS: linux(archlinux)
godot version: 1.1-stable and 2.0-alpha-r1660.889d21e
Firstly, I'm not a native speaker of English , So I'm sorry if my English is hard to understand.
I know that the IME on linux is not supported yet.
So I tried to copy and paste japanese word テスト(which means test) to Label , Button , Script editor , etc.
But both versions of godot display it as \u30c6\u30b9\u30c8.
I read some forum and issuie, and imported the font for everyday use.
Then godot uses the specified font for Latin characters.
But, CJK characters are not yet displayed properly.

Hi Taro! I'm not exactly sure why the input is converted to those
characters, can test I guess. Copy the katakana from your mail to godot
works for me though on Ubuntu (if using a Japanese font)
We never implemented IME simply because we don't know how Asian users
utilize IME natively, so it's really difficult for us to do it.
Version 2.1 (next after current) is planned to have proper IME, but will
definitely need help with this
On Sat, Sep 19, 2015 at 3:34 AM, taro-yamada<EMAIL_ADDRESS>wrote:

OS: linux(archlinux)
godot version: 1.1-stable and 2.0-alpha-r1660.889d21e
Firstly, I'm not a native speaker of English , So I'm sorry if my English
is hard to understand.
I know that the IME on linux is not supported yet.
So I tried to copy and paste japanese word テスト(which means test) to Label
, Button , Script editor , etc.
But both versions of godot display it as \u30c6\u30b9\u30c8.
I read some forum and issuie, and imported the font for everyday use.
Then godot uses the specified font for Latin characters.
But, CJK characters are not yet displayed properly.
—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/issues/2491.

--
OkamStudio

Thank you for your prompt reply.
I tried on Ubuntu 15.04 64bit and godot1.1-stable  but the same behavior occured for me ;-(
Would you tell me which version of ubuntu/godot are you using?

I can confirm that there are issues not only with CJK characters, but also with the copy-pasting of any non-ascii character. For example I can input "ç" in the editor without issues, and it's displayed properly. But if I try to copy-paste it from Firefox into the editor then there is no output. And with CJK characters I get the same \u30c6, etc. as above. When copy-pasting from other applications, I guess different behaviours: copy-paste from Libreoffice (GTK, like Firefox) does not work for "ç", and adds a "?" for CJK characters; copy-paste from Kate (Qt) works for "ç" (Latin extended), but no output for CJK...
I tried with various fonts (Free Mono, Google Droid Sans). I tried Google Droid Japanse but got an editor crash when selecting "Unicode" in the font importer.

Fixed by 3e8eb39 \o/


Mixin transformer crashes when type variables on a method differ

class A {
  foo<T>() {}
}

class B {
  foo() {}
}

class C = A with B;
class D = B with A;
class E extends A with B {}
class F extends B with A {}

Any of the last four lines will cause a crash with this assert:
'package:kernel/transformations/mixin_full_resolution.dart': Failed assertion: line 203 pos 18: 'src.typeParameters.length == dst.typeParameters.length': is not true.

When run without asserts, the subsequent code indexing into the array crashes:
RangeError (index): Invalid value: Valid value range is empty: 0
#0      List.[] (dart:core/runtime/libgrowable_array.dart:142:60)
#1      MixinFullResolution.transformClass (package:kernel/transformations/mixin_full_resolution.dart:205:31)

Might be related to https://github.com/dart-lang/sdk/issues/32313 which is also a crash during mixin transformation caused by a missing type parameter.

I believe this is fixed after CL 100580 landed.


Working set refreshed when selecting working set item

I am using Brackets 1.5 on Windows 10, but I think this affects all versions.
When user clicks/selects an item in the working set, the working set is refreshed.  I don't think that the working set should be refreshed in this case.
I believe this is happening because of line 779 (in the function drop()) in WorkingSetView.js.  The line is postDropCleanup(); but I think the line should be postDropCleanup(true);
The working set is also refreshed when user closes/middle clicks an item in the working set.  I think this is caused by line 770 in WorkingSetView.js.

We refresh it by design. Does it cause any problems when the working set is refreshed?

@abose It is only causing a small issue for me but I didn't think the working set needs to be refreshed every time you click on/select another item.
The problem for me is that I am working on an extension that uses WorkingSetView addIconProvider().  This works fine except every time another item is clicked on/selected there is a small visual glitch/delay when the icon is redrawn.  You may not be able to notice it on a faster computer but on mine you can.  The extension uses uses a NodeDomain exec() within the icon provider callback which I know is causing a delay but I'm not sure how to do it differently since I need information that I can only get from a NodeDomain (a file's mode).
As for the working set being refreshed when you close/middle click on an item, that is not my main concern but I noticed that also so I thought I would mention it.

I think I may have found a solution for my problem, working on it now, will report back.

Closing as designed.


Refactor Zeus' core process management

This replaces the core of Zeus' process management (the processtree package)
with goals of:

Substantially narrowing the interface of the package to make it
easier to reason about.
Breaking process management down into separately testable units.
Resolving some of the race conditions (and awkward mitigations for races)
in the current code.

I believe I realized a few other benefits in the process:

Improving cleanup so that Zeus terminates its processes before exiting.
Better propagating potentially fatal errors in process management.
Removed global state from the status chart package.

TODO:

Some manual QA
Release an internal candidate for deeper beta testing
Consider whether there's a way to make the node and process packages internal to processtree.
Update design docs.

Closing for now, given the major issues with Zeus' current file monitoring system I'm going to have to rethink a bit.


Investigate NBN Fibre Upgrade for Strata Managed Buildings

https://www.nbnco.com.au/residential/upgrades/more-fibre-for-buildings

This is now included in the new fields and shown on map


Update Readme

Update readme with a proper description of the project, images, technologies used, team members and links to code of conduct & contributing guidelines.

Closing this, see here!


Where applicable, update Windows pools used to azsdk-pool-mms-win-2022-general and rename vmImage to the windows-20xx format

Where applicable, update Windows pools used to azsdk-pool-mms-win-2022-general and rename vmImage to the windows-20xx format.
This discussion explains why we chose given vmImage format:
Mike Harder: 1ES Hosted Pool image name changes
posted in Azure SDK / Engineering System 🛠 at Wednesday, February 15, 2023 12:10 PM
Changes made in the PR

search-and-replace all occurrences of windows-2019 and windows2019 to corresponding 2022 variant
search-and-replace azsdk-pool-mms-win-2019-general with azsdk-pool-mms-win-2022-general
added azsdk-pool-mms-win-2022-general as pool name to docindex.yml files, as it was absent from there
search and replace MMS2019 and MMS2022 to windows-2022

Context
For further context, please see:

https://github.com/Azure/azure-sdk-tools/issues/3407

@benbp @mikeharder need an approval on this one. Albeit because the changelog is large, it probably should be reviewed by a subject matter expert.

Closing this PR. Per my chat with @mikeharder, this PR is not necessary, as the files having the obsolete pool that needs to be updated are not actually used anywhere; they are just copied over.


Add a license please

The repo does not have any license which restricts anyone from even looking at the code (though the AI training companies will probably steal the code anyway)

Sure, I've added a license. This is only a fun side project, and very little code. I'm not very worried about people distributing or using this code.

closing

resolved

Thank you 🙂


[Enhancement⚡] Add #[inline] for PutMessageResult methods

Before Creating the Enhancement Request

[X] I have confirmed that this should be classified as an enhancement rather than a bug/feature.

Summary
Add #[inline] for PutMessageResult methods
Motivation

Describe the Solution You'd Like
No response
Describe Alternatives You've Considered
No response
Additional Context
No response

Hi @mxsm
I came across this issue and would like to work on resolving it. Could you please assign it to me?

Hi @mxsm I came across this issue and would like to work on resolving it. Could you please assign it to me?

@nakul-py  sure!


Add retry logic for pod deletion and change the cleanup of stale containers to cleanup of containers in terminal statuses

Tested on a test containerized cluster:
In test cluster current namespace there are 4 pods of finished executions
% kubectl get po|grep hxxxxx6
fc-dep-hxxxxx6-500792                                  1/1     Running                      0          5d2h
fc-dep-hxxxxx6-501170                                  1/1     Running                      0          2d19h
fc-dep-hxxxxx6-501382                                  1/1     Running                      0          28h
fc-dep-hxxxxx6-501399                                  1/1     Running                      0          23h
fc-dep-hxxxxx6-501504                                  1/1     Running                      0          38s
fc-dep-hxxxxx6-501505                                  1/1     Running                      0          31s
MySQL [azkaban_hxxxxx_k8s]> select status from execution_flows where exec_id in (500792,501170,501382,501399);
+--------+
| status |
+--------+
|     60 |
|     60 |
|     50 |
|     60 |
+--------+
confirmed deletion (status code 200) in web server log
azkaban-webserver.log.2022-04-30:2022/04/30 20:25:17.546 +0000  INFO [KubernetesContainerizedImpl] [azk-container-cleanup] [Azkaban] ExecId: 501399, Action: Pod Deletion, Pod Name: fc-dep-hxxxxx6-501399, Status: 200
azkaban-webserver.log.2022-04-30:2022/04/30 20:25:17.578 +0000  INFO [KubernetesContainerizedImpl] [azk-container-cleanup] [Azkaban] ExecId: 501382, Action: Pod Deletion, Pod Name: fc-dep-hxxxxx6-501382, Status: 200
azkaban-webserver.log.2022-04-30:2022/04/30 20:25:17.597 +0000  INFO [KubernetesContainerizedImpl] [azk-container-cleanup] [Azkaban] ExecId: 501170, Action: Pod Deletion, Pod Name: fc-dep-hxxxxx6-501170, Status: 200
azkaban-webserver.log.2022-04-30:2022/04/30 20:25:17.612 +0000  INFO [KubernetesContainerizedImpl] [azk-container-cleanup] [Azkaban] ExecId: 500792, Action: Pod Deletion, Pod Name: fc-dep-hxxxxx6-500792, Status: 200
double checked in cli:
kubectl get po|grep hxxxxx6
fc-dep-hxxxxx6-501504                                  1/1     Running                      0          7m54s

Pull Request Test Coverage Report for Build 6825

22 of 49 (44.9%)  changed or added relevant lines in 2 files are covered.
90 unchanged lines in 5 files lost coverage.
Overall coverage increased (+0.02%) to 45.3%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

azkaban-common/src/main/java/azkaban/executor/container/ContainerCleanupManager.java
22
27
81.48%

azkaban-common/src/main/java/azkaban/executor/container/KubernetesContainerizedImpl.java
0
22
0.0%

Files with Coverage Reduction
New Missed Lines
%

azkaban-common/src/main/java/azkaban/user/UserUtils.java
1
74.19%

azkaban-common/src/main/java/azkaban/executor/container/ContainerCleanupManager.java
2
56.91%

azkaban-common/src/main/java/azkaban/executor/ExecutableFlow.java
8
93.72%

az-core/src/main/java/azkaban/Constants.java
10
21.43%

az-hadoop-jobtype-plugin/src/main/java/azkaban/jobtype/HadoopPigJob.java
69
0%

Totals

Change from base Build 6820:
0.02%

Covered Lines:
18077

Relevant Lines:
39905

💛  - Coveralls


Remove dotnet org from dn-bot-all-orgs-test-r

Release Note Category

[ ] Feature changes/additions
[ ] Bug fixes
[ ] Internal Infrastructure Improvements

Release Note Description
Remove dotnet org from dn-bot-all-orgs-test-r

PR https://dev.azure.com/dnceng/internal/_git/dotnet-helix-service/pullrequest/33744

What was the requirement for this work? Should we have created new tokens instead? Is this going to break Build Analysis (because it needs to read from both dnceng and dnceng-public)?
/cc @AlitzelMendez

it is not going to break build analysisbuild Analysis  uses dnceng and dnceng-public and both are still on the secret, the only one removed was dotnet
it was removed because the dotnet wasn't no longer visible by dn-bot so the secret was not able to be rotated if we wanted to add this org

this was rolled out last week. Closing this


trained with faster rcnn and backbone resnet152?

Hi,
Thanks for the excellent code base.
can i get the feature trained with faster rcnn and backbone resnet152?
I'm waiting for your reply.

Hi,
I only transferred the weights from original bottom-up model that was trained with resnet101. I'm not planning to train any other architectures.


Use healthcheck command preferred by MinIO

Description

This PR updates the healthcheck used by the MinIO container to mc which is the preferred healthcheck tool.
Testing Instructions

Check out the PR.
Start the catalog containers with just catalog/up.
Wait for a few seconds.
Run just dc ps to see the status of the Docker Compose services.
The MinIO service s3 should show the status as healthy.

Checklist

[x] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.
[ ] I ran the DAG documentation generator (ov just catalog/generate-docs for catalog
PRs) or the media properties generator (ov just catalog/generate-docs media-props
for the catalog or ov just api/generate-docs for the API) where applicable.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

I was trying to understand MinIO better and was casually reading their docs and GitHub issues. There I came across a discussion that healthchecks were failing because curl couldn't be found. In that it was recommended to use mc.


How to deploy onnx models in C++?

Given that your preditc module contains convert_to_onnx features and I need to use C++ for inference deployment, how do I do that？

sorry, i don't know how to do…… maybe you can add the decode code to onnx, which will help


Linter Integration

I've been working on a Linter for the package in my free time. The problem that I ran into was that the item data needed to be decoupled from the autocompletion code in order to use it in both the linter provider and autocompletion provider. With the Linter nearly completed, the autocompletion provider now needs to be modified in order to make use of the new JSON data, but this brings us to a crossroads.
The linter is written in TypeScript, so the question is whether or not I should take this opportunity to just port everything to it. All of the code being TypeScript vs. some components being written in CoffeeScript and others in TypeScript.
Worth noting that I chose TypeScript for the linter because of the atom-typescript package.

Sorry, I didn't get an email about you creating this issue for some reason. A Linter sounds like a great feature. Can I see the code so far? I'm pretty happy to have it all ported but I want to see what it's like first. The atom-typescript package does look like it's worth using. I'm happy to help but I'm in another country on holiday at the moment.

I decided just to rewrite the linter in CoffeeScript. I'll update this or just create a pull request once the rewrite is complete.
The autocompletion provider will still need to be rewritten to create its snippets from shared JSON data files at runtime, but that can wait until the linter is merged.

I can't wait to see it.

grammars/poe.cson
styles/poe.less
Are you fine with this rewrite of the grammar? The rewrite initially began with the goal to hopefully using the data from it to enable partial updates in the linter, but that didn't pan out. Partial updates will still be supported, but in another way.
The updated grammar is oriented more towards highlighting valid tokens. The names have all changed to what is more typical in Atom, which gives us automatic syntax theme support. The linter will cover the cases where the user has used an invalid format or given invalid values, so we don't need to care about that here anymore.

Yeah that looks good. It's much clearer than my version.

This probably wont be merged for Path of Exile 2.5, which we'll have patch notes for today, but I'm keeping relevant changes synchronized.
I detoured back into TypeScript. The original problem was that the public type definitions for the Atom API were both out of date and incomplete, with pretty much all of the new event-driven stuff being missing, but I've been working on just outright replacing them. My repository has an example as to how some of the TypeScript code would look.
The Linter itself is incomplete and will be stitched together from both the CoffeeScript source and the old TypeScript source. Right now it's just a skeleton that logs events.


Shader no workie

Describe your bug here.
this shader does not work on my windows vista pc

Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
no.


Disallow script tags

As pointed out in this discussion in the mailing list, it is possible to inject arbitrary Javascript in Elm using elm-html by defining a script tag.
import Html (..)
import Html.Attributes (..)

script : String -> Html
script string =
  node "script" [] [text string]

main =
  script "alert('Hello World');"

There may be value to using script tags to load Javascript files, but the fundamental problem is that this violates the integrity of the Elm code. This is especially a problem if Elm components are intended to be shared and then one can suddenly inject Javascript into your code and do all sorts of nasty things.
If Javascript is really necessary and ports are not a viable option, then one should use Native modules which at the very least can be checked and flagged. This is uncheckable and unflaggable.

It ought to be fairly easy to add a runtime check to node to ensure the first argument is not "script". That's problematic in (1) it's runtime, and not even immediate runtime if someone makes a script tag later (2) it's easy for an attacker to fork the library and remove the check. I'm more interested in Richard's concern:

It's concerning that you could get this from someone else's library. The idea that you could get undocumebted side effects by incorporating Html values from third-party code into your own components is a bit scary.

Does this indicate a vulnerability in the compiler?

I am not sure if banning the script node really prevents third-party libraries from the possibility of having undocumented side effects. It will still be possible to have arbitrary JS code placed in the Native/ directory of the package executed by the package's Elm code.

I agree, we should in some way "escape" script to make sure that doesn't go through.
As for forking the library, that should be a general thing as in, if you want to use an element that requires, say, elm-html 2.0 and it really modified that elm-html 2.0, there should be some way to check that. But that's more of a general thing. That's handled by the package manager that will get the appropriate versions from github and will tell you in the elm-package.json.
I am concerned about the undocumented side effects. We should totally flag any library that uses native modules with a tag on the package site that says uses native modules or something. With this you can shift blame by saying, "look, we warned you". And that tag suddenly comes with a warning : "this may cause runtime crashes or have unchecked code".
But in general, we should see Elm as being a sandbox. Like, it lives in its own little world with its own rules and its own security features and communicates with the outside world only via ports (and whatever the standard library allows you to do).

One fairly clean solution is to simply strip them out just before handing things off to virtualdom.
There has to be a final translation step where elm-html calls a virtualdom function to create the element in question, and in such a case it could simply do something like:
if (elem.tag.match(/^script$/i) {
  console.warn("elm-html forbids instantiating <script> tags, so the following Html element was ignored:", elem);
} else {
  // proceed as normal
}

Alternatively, I haven't checked how virtualdom handles this, but certainly you can already get a runtime exception from manually calling document.createElement("$$$") (or any other invalid tag name), so there must be some existing precedent for what happens when you try to use an invalid string; presumably "script" could be treated as invalid in the same way that "$$$" is.