All grpc-go performance benchmarks are broken

This has probably been happening for a while, but we haven't noticed
https://source.cloud.google.com/results/invocations/c6533106-62e0-4979-9712-c4b446c711a3/targets/github%2Fgrpc%2Freports%2F32core/tests
Looks like that the client are failing to connect to the server:
++ dirname tools/run_tests/performance/run_qps_driver.sh
+ cd tools/run_tests/performance/../../..
+ bins/opt/qps_json_driver '--scenarios_json={"scenarios": [{"name": "go_protobuf_sync_streaming_ping_pong_secure", "warmup_seconds": 5, "benchmark_seconds": 30, "num_servers": 1, "server_config": {"async_server_threads": 1, "channel_args": [{"str_value": "latency", "name": "grpc.optimization_target"}], "security_params": {"use_test_ca": true, "server_host_override": "foo.test.google.fr"}, "threads_per_cq": 0, "server_type": "SYNC_SERVER"}, "client_config": {"security_params": {"use_test_ca": true, "server_host_override": "foo.test.google.fr"}, "channel_args": [{"str_value": "latency", "name": "grpc.optimization_target"}], "async_client_threads": 1, "outstanding_rpcs_per_channel": 1, "rpc_type": "STREAMING", "payload_config": {"simple_params": {"resp_size": 0, "req_size": 0}}, "client_channels": 1, "threads_per_cq": 0, "load_params": {"closed_loop": {}}, "client_type": "SYNC_CLIENT", "histogram_params": {"max_possible":<PHONE_NUMBER>0.0, "resolution": 0.01}}, "num_clients": 1}]}' --scenario_result_file=scenario_result.json
RUNNING SCENARIO: go_protobuf_sync_streaming_ping_pong_secure
I0507 00:37:42.671391159   22301 driver.cc:272]              Starting server on grpc-kokoro-performance-server-32core:10600 (worker #0)
D0507 00:37:42.671556603   22301 ev_posix.cc:142]            Using polling engine: epollex
D0507 00:37:42.671586093   22301 dns_resolver.cc:339]        Using native dns resolver
D0507 00:37:42.671691649   22301 dns_resolver.cc:280]        Start resolving.
I0507 00:37:42.679975596   22301 subchannel.cc:646]          Connect failed: {"created":"@1525653462.679946278","description":"Failed to connect to remote host: OS Error","errno":111,"file":"src/core/lib/iomgr/tcp_client_posix.cc","file_line":201,"os_error":"Connection refused","syscall":"connect","target_address":"ipv4:<IP_ADDRESS>:10600"}
I0507 00:37:42.679994559   22301 subchannel.cc:470]          Subchannel 0x7fb3d0003600: Retry in 999 milliseconds
I0507 00:37:43.679066052   22303 subchannel.cc:425]          Failed to connect to channel, retrying
I0507 00:37:43.679603040   22301 subchannel.cc:608]          New connected subchannel at 0x560541c48e50 for subchannel 0x7fb3d0003600
I0507 00:37:43.682625926   22301 driver.cc:328]              Starting client on grpc-kokoro-performance-client-32core:10600 (worker #1)
D0507 00:37:43.682678275   22301 driver.cc:350]              Client 0 gets 1 channels
D0507 00:37:43.682702772   22301 dns_resolver.cc:280]        Start resolving.
I0507 00:37:43.684203971   22301 subchannel.cc:608]          New connected subchannel at 0x560541c5e830 for subchannel 0x7fb3d00054c0
D0507 00:37:43.709733259   22301 dns_resolver.cc:256]        In cooldown from last resolution (from 27 ms ago). Will resolve again in 973 ms
D0507 00:37:43.709763473   22301 dns_resolver.cc:280]        Start resolving.
E0507 00:37:43.709835913   22301 driver.cc:365]              Client 0 did not yield initial status
I0507 00:37:43.709844259   22301 driver.cc:371]              Initiating
E0507 00:37:43.709876897   22301 driver.cc:381]              Couldn't write mark to client 0
E0507 00:37:43.709886004   22301 driver.cc:387]              Couldn't get status from client 0
I0507 00:37:43.709888817   22301 driver.cc:392]              Warming up
I0507 00:37:48.710184251   22301 driver.cc:398]              Starting
E0507 00:37:48.710356631   22301 driver.cc:408]              Couldn't write mark to client 0
E0507 00:37:48.711196274   22301 driver.cc:420]              Couldn't get status from client 0
I0507 00:37:48.711212376   22301 driver.cc:425]              Running
I0507 00:38:18.710037728   22301 driver.cc:439]              Finishing clients
E0507 00:38:18.710106362   22301 driver.cc:443]              Couldn't write mark to client 0
E0507 00:38:18.710116014   22301 driver.cc:446]              Failed WritesDone for client 0
E0507 00:38:18.710125088   22301 driver.cc:464]              Couldn't get final status from client 0
E0507 00:38:18.710183130   22301 driver.cc:472]              Client 0 had an error Socket closed
I0507 00:38:18.710269327   22301 driver.cc:485]              Finishing servers
I0507 00:38:18.711943181   22301 driver.cc:499]              Received final status from server 0
I0507 00:38:18.712463240   22301 report.cc:82]               QPS: -nan
I0507 00:38:18.713033093   22301 report.cc:122]              QPS: -nan (-nan/server core)
I0507 00:38:18.713045165   22301 report.cc:127]              Latencies (50/90/95/99/99.9%-ile): 0.0/0.0/0.0/0.0/0.0 us
I0507 00:38:18.713048636   22301 report.cc:137]              Server system time: 0.00%
I0507 00:38:18.713051310   22301 report.cc:139]              Server user time:   0.00%
I0507 00:38:18.713053757   22301 report.cc:141]              Client system time: -nan%
I0507 00:38:18.713056199   22301 report.cc:143]              Client user time:   -nan%
I0507 00:38:18.713058857   22301 report.cc:148]              Server CPU usage: 0.00%
I0507 00:38:18.713061415   22301 report.cc:153]              Client Polls per Request: -nan
I0507 00:38:18.713063850   22301 report.cc:155]              Server Polls per Request: -nan
I0507 00:38:18.713066331   22301 report.cc:160]              Server Queries/CPU-sec: -nan
I0507 00:38:18.713068755   22301 report.cc:162]              Client Queries/CPU-sec: -nan

Also seeing:
2018-05-06 22:51:51,797 START: qps_json_driver.quit
2018-05-06 22:51:51,800 WAITING: ETA 36.0 sec; 0 queued, 1 jobs running, 1 complete, 1 failed (load 1.00)
2018-05-06 22:51:52,814 WAITING: ETA 37.1 sec; 0 queued, 1 jobs running, 1 complete, 1 failed (load 1.00)
2018-05-06 22:52:02,814 WAITING: ETA 47.1 sec; 0 queued, 1 jobs running, 1 complete, 1 failed (load 1.00)
2018-05-06 22:52:12,815 WAITING: ETA 57.1 sec; 0 queued, 1 jobs running, 1 complete, 1 failed (load 1.00)
2018-05-06 22:52:22,815 WAITING: ETA 67.1 sec; 0 queued, 1 jobs running, 1 complete, 1 failed (load 1.00)
2018-05-06 22:52:32,816 WAITING: ETA 77.1 sec; 0 queued, 1 jobs running, 1 complete, 1 failed (load 1.00)
2018-05-06 22:52:42,816 WAITING: ETA 87.1 sec; 0 queued, 1 jobs running, 1 complete, 1 failed (load 1.00)
2018-05-06 22:52:52,817 WAITING: ETA 97.1 sec; 0 queued, 1 jobs running, 1 complete, 1 failed (load 1.00)
2018-05-06 22:53:02,817 WAITING: ETA 107.1 sec; 0 queued, 1 jobs running, 1 complete, 1 failed (load 1.00)
2018-05-06 22:53:12,818 WAITING: ETA 117.1 sec; 0 queued, 1 jobs running, 1 complete, 1 failed (load 1.00)
2018-05-06 22:53:22,818 WAITING: ETA 127.1 sec; 0 queued, 1 jobs running, 1 complete, 1 failed (load 1.00)
2018-05-06 22:53:32,819 WAITING: ETA 137.1 sec; 0 queued, 1 jobs running, 1 complete, 1 failed (load 1.00)
2018-05-06 22:53:42,819 WAITING: ETA 147.1 sec; 0 queued, 1 jobs running, 1 complete, 1 failed (load 1.00)
2018-05-06 22:53:52,819 WAITING: ETA 157.1 sec; 0 queued, 1 jobs running, 1 complete, 1 failed (load 1.00)
2018-05-06 22:54:02,820 WAITING: ETA 167.1 sec; 0 queued, 1 jobs running, 1 complete, 1 failed (load 1.00)
2018-05-06 22:54:12,820 WAITING: ETA 177.1 sec; 0 queued, 1 jobs running, 1 complete, 1 failed (load 1.00)
2018-05-06 22:54:22,821 WAITING: ETA 187.1 sec; 0 queued, 1 jobs running, 1 complete, 1 failed (load 1.00)
2018-05-06 22:54:32,821 WAITING: ETA 197.1 sec; 0 queued, 1 jobs running, 1 complete, 1 failed (load 1.00)
2018-05-06 22:54:42,822 WAITING: ETA 207.1 sec; 0 queued, 1 jobs running, 1 complete, 1 failed (load 1.00)
2018-05-06 22:54:46,064 WAITING: ETA 210.3 sec; 0 queued, 1 jobs running, 1 complete, 1 failed (load 1.00)
2018-05-06 22:54:56,064 TIMEOUT: qps_json_driver.quit [pid=31775, time=184.3sec]

Also seeing these errors:
2018-05-06 22:54:56,065 2018-05-06 22:51:16,017 detected port server running version 20
2018-05-06 22:51:16,050 my port server is version 20
Port server started successfully
++ dirname tools/run_tests/performance/run_worker_go.sh
+ cd tools/run_tests/performance/../../..
++ pwd
+ export GOPATH=/home/kbuilder/performance_workspace/grpc/../gopath
+ GOPATH=/home/kbuilder/performance_workspace/grpc/../gopath
+ /home/kbuilder/performance_workspace/grpc/../gopath/bin/worker --driver_port=10600
FATAL: 2018/05/06 22:51:16 &{0xc420206300}.StreamingCall(_) = _, rpc error: code = Unavailable desc = all SubConns are in TransientFailure, latest connection error: connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:38561: getsockopt: connection refused"

2018-05-06 22:54:56,065 FAILED: qps_worker_go_1 [ret=1, pid=31766, time=220.3sec]

2018-05-06 22:54:56,065 2018-05-06 22:51:16,021 detected port server running version 20
2018-05-06 22:51:16,054 my port server is version 20
Port server started successfully
++ dirname tools/run_tests/performance/run_worker_go.sh
+ cd tools/run_tests/performance/../../..
++ pwd
+ export GOPATH=/home/kbuilder/performance_workspace/grpc/../gopath
+ GOPATH=/home/kbuilder/performance_workspace/grpc/../gopath
+ /home/kbuilder/performance_workspace/grpc/../gopath/bin/worker --driver_port=10600

2018-05-06 22:54:56,065 FAILED: qps_worker_go_2 [ret=124, pid=31767, time=220.3sec]

CC @matt-kwong

@menghanl @dfawley  can we get an update on this?

Locally benchmark didn't work for me (could not build because of https://github.com/grpc/grpc/issues/15259).
The last data point was from Feb 15. Will find out if anything suspicious was changed during that time.

Seems fixed. Thanks!


Implement complete pipeline

Implement a pipeline that does the following:

Takes some configuration as input (e.g. seed, drop percentages, transforms, etc)
Computes similarity metrics
Trains networks on both models
Trains transfer attacks (on both?)
Computes attack success metrics
Stores metrics computed

It may be the case that portions of this pipeline are best separated from one another (or are in partially separate pipelines). For example, we will probably want to be training networks and attacks while implementing the similarity metrics

A general note for how we want this to work:

Several configs for different components of the experiment

Dataset config

Controls alterations between A and B, and input seed
We'll need to specify many datasets, while creating some (3-5?) with the same configuration but with a different seed
Open question: do we take sets of options (dropping, transform1, transform2) and extract combinations of alterations, or should we specify the experiments we want to perform in advance?
The datasets will be created separately in the similarity measurement pipeline and in the model + attack pipeline

Metric config

Which similarity measures we want to use
Initially constant across datasets. If we start looking into e.g. different labels or different datasets, we'll need to revisit that constancy (as some measures will no longer be appropriate or will only be appropriate under certain conditions - e.g. PAD requires same features, OT + OTDD will require gromov-wasserstein instead of wasserstein distance if features are not the same)

Model config

Unsure if required, but if we start relaxing the extent to which models are the same we'll need to look into this

We'll need functions for handling the configs above
We'll also need two scripts to handle everything:

A metrics script, that produces the datasets, computes the distance measures, and stores the results
A models script, that produces the datasets, farms out model training for each A and B dataset, then transfers an attack from A to B and from B to A, computes the attack success metrics, and stores the results

Since we'll have two sets of results stored, we'll also need to make sure we can join up the two results in a third script

With #29 this is mostly done. The main two tasks remaining are:

#32 replacing bash script generation with slurm (or something else appropriate for the computers we'll be using)
#17 and #18 to get the transfer attack training + success metrics set up. We'll need some results from model training before that stage can proceed

[from meeting]
we are running training on HPC
where are we running metric calculations?
where are we running attacks?

#17 and #18 are now done. #40 has been opened to deal with adding attack scripts to pipeline. We also want to work out where on the pipeline we are doing similarity metrics (as above)

Metris calculation location opened as #42

With #40 done, #42 is the last piece of pipeline work to be done. Will be looking at #26 before that

With #42 done, this PR is now finished. I've opened #57 to cover the need to actually use the pipeline to produce the final results


StyleGAN support

Is there an estimate for when StyleGAN will be added?

same question here.

Same question as well, especially interested in a comparison to already existing implementations.

Sorry for the delay:
There is a beta version of StyleGAN ongoing. But it still has some bugs. I will release it within a week or two.

@Molugan Is the pull request delayed by the release of StyleGAN2  by any chance?

PR incoming, you can start to work with the branch but I need to check that everything os ok.
https://github.com/facebookresearch/pytorch_GAN_zoo/pull/95


change hard coded domain

removed hard coded domain name at several places

Hi -- I've now manually merged your commits to master (I didn't want to pull that last change of author). Closing this PR but thank you so much for the code!

Thank you! I apologize for the accidental appending of the last commit.


ported thank you page, webpack loader error

also @kingsmil there's currently 2 git branches pushed that seem related to this:

port/SCSE-249
migration/SCSE-249

is the "port" branch of relevance?

@xJQx i bumped next from v13.0 to v13.4, and replaced next-transpile-modules with the new built-in equivalent.
docs: https://nextjs.org/docs/app/api-reference/next-config-js/transpilePackages
removed module is now deprecated: https://www.npmjs.com/package/next-transpile-modules

Merged as 1d6da5f.


Add Clock Multiplier & Divider

One last MR for 2023!
Adds a combined clock multiplier & divider script.  Pretty standard functionality; each output can have an independently-controlled modifier from [/12, /8, /6, /5, /4, /3, /2, x1, x2, x3, x4, x5, x6, x8, x12].  DIN accepts the incoming clock signal, which cannot be slower than 1Hz.  AIN accepts a digital reset signal.
Each output is a ~50% duty cycle square wave with a rate equal to the incoming clock speed multiplied by the clock modifier.
This script uses the same analog-to-digital input wrapper introduced in https://github.com/Allen-Synthesis/EuroPi/pull/311 to allow AIN to be used in a fashion similar to DIN with a callback on the rising and falling edges.

Rebased to latest traffic branch after rebasing that to latest main.


Fix a sample code in README.md

Hi.
This is a great library.
I found a mistake in the sample code in the README.md.
I would appreciate it if you could check this PR.

@joe-king-sh Thank you for spotting this! :)


This application requires Java Runtime Environment when open the java software jd-gui

This application requires Java Runtime Environment when open the java software jd-gui

According to this issue you need Oracle Java 8 or any of the OpenJDK 8 binaries to run jd-gui.


Is the data for SA and SSA different from that in Figure 4 of paper

Reference paper: FES2014 global ocean tide atlas: design and performance
The data in Figure 4 shows max of 0.11m for SA and 0.042m for SSA component, But the data in the sa.nc and ssa.nc files of this github dataset has a much smaller values (i.e., 0.043 and 1.78) with label of unit as 'cm'.
Could you please help check what might be the reason for the difference? Thank you very much.

The file included in the distribution is not the official version of FES2014, but a very old version FES99. You need to download the atlas from the AVISO website


samples/sensor/*: Build issue when board expose sensors defined on both I2C and SPI buses

Describe the bug
There are a few sensor samples which fail to build with 96b_argonkey:
samples/sensor/vl53l0x/
samples/sensor/hts221
samples/sensor/lps22hb
/<redacted>/zephyr-sdk/sdk-0.14.0/arm-zephyr-eabi/bin/../lib/gcc/arm-zephyr-eabi/10.3.0/../../../../arm-zephyr-eabi/bin/ld.bfd: zephyr/drivers/sensor/lsm6dsl/libdrivers__sensor__lsm6dsl.a(lsm6dsl.c.obj):(.rodata.lsm6dsl_config_0+0x4): undefined reference to `__device_dts_ord_83'

Root cause is:

Board dts defines lsm6dsl on SPI bus, while other sensors (hts221, vl53l0x, lps22hb) are defined on I2C bus
samples for I2C sensors enable both CONFIG_SENSOR and CONFIG_I2C
Due to CONFIG_SENSOR=y drivers/sensor/lsm6dsl.c is compiled, but compilation fails due to CONFIG_SPI not being enabled.

This is likely an effect of:
menuconfig LSM6DSL
	bool "LSM6DSL I2C/SPI accelerometer and gyroscope Chip"
	default y
+	depends on DT_HAS_ST_LSM6DSL_ENABLED
	depends on I2C || SPI

Ideally LSM6DSL activation should be a little bit more selective and check bus activation:
	depends on DT_HAS_ST_LSM6DSL_ENABLED
	depends on ((I2C && DT_LSM6DSL_ENABLED_ON_I2C_BUS) ||
                   (SPI && DT_LSM6DSL_ENABLED_ON_SPI_BUS))

@erwango I had worked up something similar to what you suggested in a PR:
https://github.com/zephyrproject-rtos/zephyr/pull/48048
depends on (I2C && $(dt_compat_on_bus,$(DT_COMPAT_ADI_ADXL345),i2c)) || \
		   (SPI && $(dt_compat_on_bus,$(DT_COMPAT_ADI_ADXL345),spi))

However @MaureenHelm asked to just update the board Kconfig.defconfig to enable the bus needed.  So will look for her comment on this to see what direction we should go.

@MaureenHelm asked to just update the board Kconfig.defconfig to enable the bus needed

If this is something like follows, this would make sense indeed:
config SPI
    default y if LSM6DSL

config I2C
   default y if VL53L0X || HTS221 || LPS22HB

depends on (I2C && $(dt_compat_on_bus,$(DT_COMPAT_ADI_ADXL345),i2c)) || \
		   (SPI && $(dt_compat_on_bus,$(DT_COMPAT_ADI_ADXL345),spi))

I really don't like this pattern.

If this is something like follows, this would make sense indeed:
config SPI
    default y if LSM6DSL

config I2C
   default y if VL53L0X || HTS221 || LPS22HB

I tried this before, but it created a circular dependency (LSM6DSL depends on SPI, and SPI depends on LSM6DSL). That's why I wound up using SENSOR instead of LSM6DSL.
config SPI
	default y if SENSOR

Edit: There would be a consequence on samples/sensors/*/prj.conf since enabling SENSOR would then automatically bring up buses. So CONFIG_I2C=y and CONFIG_SPI=y would have to be cleaned up. Another consequence is that in case of board has sensor on both bus. Enabling SENSOR would bring up both buses automatically. But this could be fine tuned using .overlays, so that is not a concern to me.

Agreed

@erwango will you work up a PR for this?

That's why I wound up using SENSOR instead of LSM6DSL.

config SPI
	default y if SENSOR

I'm not convinced by this solution. This enables bus irrespective of sensor devicetree status on the bus.
We end up in a situation where:

If I have sensors on one type of bus, bus is activated only if required based on devices device tree status.
If I have sensors splits on 2 types of bus, both bus are activated irrespective of devicetree status (as long as one sensor is enabled). If I want to enable sensors on one type of bus, I need to update device tree and manually edit board_config to disable the bus not needed.

So I kind of prefer solution proposed here

Dev meeting: Review the bus selection: select bus in drivers and remove board related code.
I'll have a try and propose a PR

I'll have a try and propose a PR

pushed a PR, will see how it goes through CI.


[chore] update prerequisites

I think this is missing changes to the collector chart's readme file.


Preformance issue with manual mode

Hello,
I'm trying to build complex POC using react-table with manual filtering/sorting/pagination
I use 100 rows table
I have 2 values filters with selectbox and input
I'm not actually filtering the table, everything is in manual mode
I'm seeing too big delays between I press button and letter appears to the input
Provide an example via Codesandbox! (required)
Example complex_table to reproduce the issue.
Steps To Reproduce (required)
just start typing inside any filter
Expected behavior (Recommended)
I expect that in manual mode filters not perform any inner operations and should just update state and it not sounds like expensive operation
Desktop (please complete the following information):

OS: ios
Browser: chrome, firefox
Version: latest

Additional context
maybe I forgot to memoize something but I checked a few times
issue disappers when I change generated rows count from 100 to 1-2
btw, in production mode build it works much faster but still visible

I can't really get your sandbox to work. Please check it out and either comment that it's fixed :)

What is the issue with it?
I'm able to successfully launch it in both Chrome and Firefox

Checked in incognito tabs, all running, but issue still exist, please reopen this


fix : infinite resize issue

This adds the fix for infinite resize issue of zdog container by adding box-sizing border box to the renderer element (canvas/svg) and wrapper div.

Recreated Issue here : https://codesandbox.io/s/gifted-violet-7x4n9q?file=/src/App.js

resolved in v1.0.12

closing now (again)

Demo : https://codesandbox.io/s/gifted-violet-7x4n9q?file=/src/App.js
closing now (again)


C#: Remove SphericalInterpolateWith from Transform3D

Removed from Core in https://github.com/godotengine/godot/pull/65131

Wanted to ping Tokage, but it's not a blocker for merge.

Thanks!


feat: add 'Dart' to the package section

Description
Added Dart to the Package section.
Note:
This PR solves #862
This will also include 'Flutter' projects since it uses Dart

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


EventChannel not being called every time

I am very new to flutter+dart framework. I am trying to understand how EventChannel works. I have set up EventChannel to capture the number of an incoming call.
On the android side, I have set up an BroadcastReceiver as follows.
public class CallEventHandler extends BroadcastReceiver implements EventChannel.StreamHandler {
private static final String TAG = "[SAMPLE]";
private static final int NUMBER_LEN = 10;

private EventChannel.EventSink eventSink = null;
private Activity activity = null;

public CallEventHandler(Activity activity) {
    this.activity = activity;
}
@Override
public void onListen(Object arguments, EventChannel.EventSink events) {
    Log.i(TAG, "[onListen] setting up events");
    eventSink = events;
}

@Override
public void onCancel(Object arguments) {
    Log.i(TAG, "[onCancel] cancel events");
    eventSink = null;
    activity = null;
}

@Override
public void onReceive(Context context, Intent intent) {
    try {
        String state = intent.getStringExtra(TelephonyManager.EXTRA_STATE);
        String incomingNumber = intent.getStringExtra(TelephonyManager.EXTRA_INCOMING_NUMBER);
        if(state.equals(TelephonyManager.EXTRA_STATE_RINGING)) {
            if(incomingNumber != null) {
                Log.i(TAG, "[CallEventHandler] Incoming number : " + incomingNumber);
                if(incomingNumber.length() > NUMBER_LEN) {
                    incomingNumber = incomingNumber.substring(incomingNumber.length() - NUMBER_LEN, incomingNumber.length());
                    Log.i(TAG, "[CallEventHandler] Incoming number after : " + incomingNumber);
                    if(activity != null) {
                        String finalIncomingNumber = incomingNumber;
                        activity.runOnUiThread(new Runnable() {
                            @Override
                            public void run() {
                                if(eventSink != null) {
                                    Log.i(TAG, "[CallEventHandler] HERESSSSS : " + finalIncomingNumber);
                                    eventSink.success(finalIncomingNumber);
                                }
                            }
                        });
                    }
                }
            }
        }
    } catch (Exception ex) {
        ex.printStackTrace();
    }
}

}
In the onReceive method, I am getting the incoming number and I am sending it to EventSink.
In my MainActivity I am setting up the CallEventHandler as follows:
private final String eventId = "SAMPLE_ID";
private CallEventHandler handler = new CallEventHandler(this);

@Override
public void onStart() {
    super.onStart();
    ...
    registerReceiver(handler, filter);
}

@Override
public void onStop() {
    super.onStop();
    unregisterReceiver(handler);
}

@Override
public void configureFlutterEngine(@NonNull  FlutterEngine flutterEngine) {
    super.configureFlutterEngine(flutterEngine);
    new EventChannel(flutterEngine.getDartExecutor().getBinaryMessenger(), eventId)
            .setStreamHandler(handler);
}

On the Flutter side, the code is as follows:
class EventHandler {
    static const String TAG = "[SAMPLE]";
    final String _eventId = "SAMPLE_ID";
    EventChannel? _evtChannel;
    Stream<String>? _evtStream;

    EventHandler() {
        debugPrint(TAG + " Setting up EventHandler");
        _evtChannel = EventChannel(_eventId);
        _evtStream = _evtChannel?.receiveBroadcastStream().distinct().map((dynamic
        event) => getString(event as String));
    }

    void startListening(void Function(String data)? onData) {
        debugPrint(TAG + " starting listening");
        _evtStream?.listen((data) {
            debugPrint(TAG + " In listening");
            onData!(data);
        });
    }
}

In my UI code, I have a StatefulWidget (MySamplePage) where I am registering my callback when the call is received
void initState() {
    widget.handler.startListening((incomingNumber) {
      debugPrint(_tag + " data : $incomingNumber");
      ...
    });
}

In my stateful home page build method, I initialize the handler in initState and added a route in build method
class _HomeScreenState extends State<HomeScreen> {
    @override
    void initState() {
        super.initState();
        debugPrint(_tag + "initState");
        _handler = EventHandler();
    }

    @override
    Widget build(BuildContext context) {
        return MaterialApp(
            routes: {
                '/caller': (context) => MySamplePage(
                    handler: _handler
                 ),
            },
            ...
        );
    }
}

The issue I am facing is that, when the widget is opened I am receiving the first incoming call, as expected. But if I make another call, then that second call is not captured by the stream. If I press the back button, and reopen the Widget everything works as expected, the first incoming call is printed in the console. I know the the Android code is sending the event from the onReceive method (The `HERESSSSS' line is printed every time), but the flutter stream is not getting the values. I am not sure what I am doing wrong here. Can anyone please help?
Thank you

Hi @sudipta1411,
This issue doesn't seem to describe a bug or a feature request. Please see https://flutter.dev/community#community-grid for resources and asking questions like this. You may also get some help if you post it on Stack Overflow with a minimal code sample that clearly describes your problem.
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and we will reopen it.
Thank you


Add Information and Logos to Training posts

Plase add all the logos to the training posts as featured image, and add the meta data (price, etc.) to the post template.  After this we will work on the formatting.

All the Meta Data is now Visible to the training posts


Add support for mandatory private endpoints

Is there an existing issue for this?

[X] I have searched the existing issues

Description
In our company, users need to create private endpoint on Storage Account to be able to access it. This PE needs to exist and have its dns zone propagated before you can create subresources like a container. A workaround is to wait 5 mins after PE creation but other solutions might be possible as well.
New or Affected Resource(s)/Data Source(s)
time_sleep
Potential Terraform Configuration
resource "time_sleep" "wait_5_min_containers" {
  create_duration = "5m"

  depends_on = [azurerm_private_endpoint.this]
}

References
No response

@on4tux Thanks for opening this issue, would we declare this time_sleep resource outside this module, and list the module instance in it's depends_on list? For this example, we can add such a time_sleep in it's main.tf file:
resource "time_sleep" "wait_5_min_containers" {
  create_duration = "5m"

  depends_on = [module.this]
}

@on4tux As suggested by @lonegunmanb, you could incorporate time_sleep into main.tf within the examples. At present, there are no plans to integrate a timer into the module.


Fix doller sign escaping issue

Description
Fix doller sign escaping issue in installation documentation.
Self Check:

[ ] Attached issue to pull request
[x] Changelog entry
[ ] Type annotations are present
[ ] Code is clear and sufficiently documented
[ ] No (preventable) type errors (check using make mypy or make mypy-diff)
[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Code is clear and sufficiently documented
[ ] Correct, in line with design

Processing this pull request

Merged into branches master in a74dcc42952e4bb5e0c6d376da27c1d6b08940c0


Test Configuration Button Always uses Test API Key, even if Live Environment is activated

Describe the bug
Test Configuration Button in Admin backend always uses Test API Key, even if the Live Environment is activated.
Versions
Shopware version: [e.g. <IP_ADDRESS>]
Plugin version: [e.g. 3.4.1]
To Reproduce
Steps to reproduce the behavior:

Go to the Admin Backend and into die Adyen Payments Configuration
Enter Valid Live API Key, Valid Merchant Account, valid live endpoint URL prefix and activate Live Environment toggle
Click on 'Test Configuration' Button
See error 'Unauthorised'

Expected behavior
I expected a success message
Screenshots
I think a description is plentiful in this case
Desktop (please complete the following information):

OS: [Tested on Linux CentOs 7, Linux CentOs 8, Windows 10 and Windows 11 and Mac OSX]
Browser [Tested in Chrome, Firefox, Safari and Microsoft Edge]
Version [Current Versions]

Smartphone (please complete the following information):
No Smartphone was tested
Additional context
Just took me almost an entire work day to figure out why the plugin kept responding with Unauthorized. So I backtracked the requests, to find out that your plugin always uses the Test API Key, regardless of Environment configuration.
It is an easy, I would have created a Pull Request, but I am not that familiar with Github, so I didn't bother.
Steps to fix:
https://github.com/Adyen/adyen-shopware6/blob/33d2d0dde3db02a75f222a68d202565faab28cd5/src/Controller/AdminController.php#L136
Change $client->setXApiKey($dataBag->get(ConfigurationService::BUNDLE_NAME . '.config.apiKeyTest'));
To: $client->setXApiKey($dataBag->get(ConfigurationService::BUNDLE_NAME . '.config.environment') ? ($dataBag->get(ConfigurationService::BUNDLE_NAME . '.config.apiKeyLive')) : ($dataBag->get(ConfigurationService::BUNDLE_NAME . '.config.apiKeyTest')));

Hi @Watschman thanks for spotting this! It is indeed a very easy fix, we will work on it


duke dataset link Invalid. Can you please update?

No description provided.

is this question have down?

@m250317460 try https://github.com/sxzrt/DukeMTMC-reID_evaluation


Add status id to films

AB#5851 Adding Status ID to Film and TV Season models, will be used to differentiate published and non published content for admins.

@brynyShift72 is this ready for review?

@brynyShift72 is this ready for review?

Yes should be ready, note that the original card says for pages and films, it turns out that pages + bundles will probably need some API changes and will be done later, this branch is just for Films + TV seasons :)


mobile buttons bug

Godot version
4.0
System information
android 6 j2 prime
Issue description
Thanks for solving the mobile black screen bug, but there's just one more bug that I can't press the buttons and some buttons stay off the screen and I can't press anything when I edit the project
Steps to reproduce
Thanks for solving the mobile black screen bug, but there's just one more bug that I can't press the buttons and some buttons stay off the screen and I can't press anything when I edit the project
Minimal reproduction project
Thanks for solving the mobile black screen bug, but there's just one more bug that I can't press the buttons and some buttons stay off the screen and I can't press anything when I edit the project

You're not following the template in any way, we can't do anything.


Passport NoImage Flags Disregarded

The incp flag for picture blob is disregarded by the GVIVE webservice for Passport Authentications. Pictures are returned regardless of whether the flag is turned on or off.
I will follow up with an official email to the team to sort this out. Until then I will artificially handle this within the wrapper to make the gem behave uniformly across various Identities if user decides to turn on the flags or vice versa through the gem.

I've logged the complaint officially with the GVIVE Team. Also pushed an artificial fix to the current problem as promised above. Commit changes here
Would rollback when issues have been fixed at the web service source.

GVIVE Team got the issue fixed. Artificial fix removed Commit


Fails to run with pyglet 1.3

Trying to run the environment with a fresh conda install produces the following error when you try to run the environment loop in the notebook:
NotImplementedError: abstract
This appears to be an issue with OpenAI gym and some breaking change in pyglet 1.3. See: https://github.com/openai/gym/issues/775/. Downgrading to pyglet 1.2.4 solves the issue.

@bgalbraith are you seeing this error in the notebook only?

@cinjon I tried it again from a clean conda env and ran the initial demo code from the notebook in the python shell. It appeared to work fine, so this seems to be restricted to running from within the notebook.


Security issue from Dependabot

The following is reported from Dependabot (NPM also flags this as critical):
JSONPath Plus Remote Code Execution (RCE) Vulnerability
<EMAIL_ADDRESS>requires<EMAIL_ADDRESS>via a transitive dependency on<EMAIL_ADDRESS><EMAIL_ADDRESS>requires<EMAIL_ADDRESS>via a transitive dependency on<EMAIL_ADDRESS>No patched version available for jsonpath-plus
Versions of the package jsonpath-plus before 10.0.0 are vulnerable to Remote Code Execution (RCE) due to improper input sanitization. An attacker can execute aribitrary code on the system by exploiting the unsafe default usage of vm in Node.

Duplicate of https://github.com/orval-labs/orval/issues/1661


validate_pct_encoding default argument is counterintuitive

TL;DR
The default CharSet const& cs in validate_pct_encoding is counterintuitive because reserved_chars_t{} would accept everything that should be rejected and reject everything that should be accepted.
Context
template<
    class CharSet = reserved_chars_t>
std::size_t
validate_pct_encoding(
    string_view s,
    error_code& ec,
    pct_decode_opts const& opt = {},
    CharSet const& cs = {}) noexcept;

The name cs is not clear about whether these are the chars that will be accepted or rejected. I've always assumed it was the rejected chars, because of the default parameter reserved_chars_t{}.
This assumption was wrong but never caused any problems because validate_pct_encoding is never called without a explicit Charset in Boost.URL.
Proposed solution
The default charset would be:
[](unsigned char ch) { return ch != '%'; };

or, if the function already handles c == %, then simply
[](unsigned char ch) { return true; };

Rationale
The parameter cs exists to handle the special cases of URI components that have requirements besides percent encoding. That is, it defines the characters that should be rejected besides % in a string because of the component grammar and despite percent-encoding. These are usually the chars that would be ambiguous as boundaries with other components.
In the general case, when decoding an arbitrary percent-encoded string, these extra URI requirements don't apply. The user should be able to decode anything without any arbitrary requirements of a specific URI component. In other words, anything but % should be accepted as non-encoded chars.
This would make validate_pct_encoding coherent with pct_decode_unchecked, which doesn't need a charset, since everything that's not % should be considered valid at this point. This also helps us note that simply negating reserved_chars_t would also include arbitrary constraints.

Yes this looks like we have a few problems in percent-encoding algorithms

rename cs to allowed

separate overloads

template<class CharSet>
std::size_t
validate_pct_encoding(
    string_view s,
    error_code& ec,
    CharSet const& allowed,
    pct_decode_opts const& opt = {}) noexcept;

BOOST_URL_DECL
std::size_t
validate_pct_encoding(
    string_view s,
    error_code& ec,
    pct_decode_opts const& opt = {}) noexcept;

maybe lift opt.plus_to_space into two loops in validate_pct_encoding since it can't change during the loop

add to validate_pct_encoding

    BOOST_ASSERT(! opt.plus_to_space || allowed('+'));

if plus_to_space == true should we require the allowed CharSet include +?

change cs to allowed in url.ipp

break pct_decode overload with CharSet into 2 functions (one with CharSet one without)

remove pct_decode overload that returns basic_string

remove pct_decode_to_value

remove pct_decode_unchecked returning const_string

remove default value for CharSet in pct_encode_bytes

remove CharSet default value from pct_encode overload

rename pct_encode which returns basic_string, to pct_encode_to_string

add pct_encode_append_to and pct_encode_assign_to

remove pct_encode_to_value that returns const_string


add bundle which <GEM> command

We already have bundle open <GEM>, but sometimes I just want the full path to an installed gem, instead of opening a new $EDITOR.
I think we should add a which command, that just prints the path of the installed gem.
Example:
$ bundle which racc
/home/user/.rbenv/versions/2.6.1/lib/ruby/gems/2.6.0/gems/racc-1.4.16

I very much agree with this feature request, since I've missed it myself too.

bundle path has this feature.
$ bundle info rails --path
/Users/c/.asdf/installs/ruby/2.6.5/lib/ruby/gems/2.6.0/gems/rails-<IP_ADDRESS>

RIght, I somehow managed to forget about that. Do you see value on a separate CLI command, or you'd rather close this?

I like the idea of a separate which command (I'd use it more), but not the amount of code required for it when we already have that functionality elsewhere.
Does bundler have any future plans for internal aliases, similar to git aliases? More specifically, it'd be cool if bundle which <gem> simply was an alias for bundle info <gem> --path.
I'd rather not have the which command if it requires duplication of existing code.

I don't personally think there is value in a separate command, but i could be completely wrong. My only suggestion is that i would more prefer the command to be bundle where instead, bundle which doesn't feel right IMO.

Unless I'm missing something there is already a bundle show command that lists the path to a gem?!
$ bundle show parser
/Users/me/.rbenv/versions/2.7.0/lib/ruby/gems/2.7.0/gems/parser-<IP_ADDRESS>


improve error reporting

Use a proper logging library instead of mindlessly spamming the console. The user doesn't need to read all that.
Validate configuration at startup. Especially whether the login is working or not.
Give a proper error message via anki connect if a card cannot be added.

done with 0.3


compile issue with last esp-idf

Hi,
I could not compile it with esp32 v5.x and I needed to use tiny midi usb which only included in that version
all the compiling issues are with the LovyanGFX library
I manage to compile with esp idf v4 but I can not use that version in my project
please update the current examples to the last esp idf or provide a screen example that compiles with the last esp idf
thanks

A quick fix is to retrieve a newer version of LovyanGFX. The example was based on an older version.
cd into components/LovyanGFX and do a git checkout master
Depending on your version of esp-idf, you might need to change the test on line 812 in LovyanGFX/src/lgfx/v1/platforms/esp32/common.cpp.
Using ESP-IDF v5.1.1-dirty, I needed to change the value from 5,1,0 to 5,1,2:

--- a/src/lgfx/v1/platforms/esp32/common.cpp
+++ b/src/lgfx/v1/platforms/esp32/common.cpp
@@ -806,7 +806,7 @@ namespace lgfx
         cmd_val |= (1 << 10); // ACK_VALUE (set NACK)
       }
 #if defined (CONFIG_IDF_TARGET_ESP32S3)
- #if (ESP_IDF_VERSION >= ESP_IDF_VERSION_VAL(5, 0, 2) && ESP_IDF_VERSION < ESP_IDF_VERSION_VAL(5, 1, 0))
+ #if (ESP_IDF_VERSION >= ESP_IDF_VERSION_VAL(5, 0, 2) && ESP_IDF_VERSION < ESP_IDF_VERSION_VAL(5, 1, 2))
       (&dev->comd[0])[index].val = cmd_val;
  #else
       (&dev->comd0)[index].val = cmd_val;


chore(main): release 2.5.3

:robot: I have created a release beep boop
2.5.3 (2024-07-01)
Bug Fixes

deps: update rust crate clap to v4.5.8 (e8f4683)
deps: update rust crate serde_json to v1.0.119 (1e31b2d)
Use transacted_at for filtering/sort (748ef00)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/philipcristiano/et/releases/tag/v2.5.3 :sunflower:


Global build options no longer global

Describe the bug
Deprecation flags not showing up in build for cFE/apps/PSP
To Reproduce
Prep with SIMULATION=native ENABLE_UNIT_TEST=true OMIT_DEPRECATED=true
make VERBOSE=true and see the DEPRECATED flags not applied to cFE/apps/PSP
Expected behavior
Global flags should be global
Code snips
None
System observed on:

Hardware: cFS Dev Server
OS: Ubuntu 18.04
Versions: Bundle (was testing fix to #484)

Additional context
Critical for CI testing...
Reporter Info
Jacob Hageman - NASA/GSFC

Diagnosis is that the recent change for #585 (flag propagation) needs to append rather than set.
Fix is pushed directly to "integration-candidate" branch (fairly trivial).  See commit above for review.

Diagnosis is that the recent change for #585 (flag propagation) needs to append rather than set.
Fix is pushed directly to "integration-candidate" branch (fairly trivial). See commit above for review.

Thanks for the quick fix.  Worked for me.  Linked issue directly to IC PR.


Unstacking the diagonals of a sequence of matrices raises ValueError: IndexVariable objects must be 1-dimensional

What happened?
In my work, I produced a sequence of covariance matrices for a 2-D quantity.  I wanted to extract the diagonal of the covariance matrices, then make that diagonal 2-D so I could plot it.
I could unstack the 2-D dimensions in the sequence of covariance matrices without issue.  I figured out how to extract the diagonal of the covariance matrices.  Unstacking the diagonal using the same procedure raised a ValueError.
What did you expect to happen?
I expected the sequence of one-dimensional diagonals to unstack into a sequence of two-dimensional fields so I could plot them with pcolormesh.
I can make this happen by unstacking the two dimensions (producing a 5-D DataArray) and extracting the diagonals from that, but I don't see a reason it shouldn't work in the other order.
Minimal Complete Verifiable Example
import numpy as np
import xarray

# Working:
test = xarray.DataArray(
    np.eye(12),
    dims=("dim0", "adj_dim0"),
    coords={
        "dim0_0": (("dim0",), np.repeat(np.arange(3), 4)),
        "dim0_1": (("dim0",), np.tile(np.arange(4), 3)),
    },
)
unstacked = test.set_index(dim0=["dim0_0", "dim0_1"]).unstack("dim0")
diag_index = xarray.DataArray(np.arange(test.shape[0]), dims=("diag",))
unstacked_diag = (
    test.isel(dim0=diag_index, adj_dim0=diag_index)
    .set_index(diag=["dim0_0", "dim0_1"])
    .unstack("diag")
)

# Not working:
test = xarray.DataArray(
    dims=("dim1", "dim0", "adj_dim0"),
    data=np.tile(np.eye(12), (2, 1, 1)),
    coords={
        "dim0": np.arange(12),
        "dim0_0": (("dim0",), np.repeat(np.arange(3), 4)),
        "dim0_1": (("dim0",), np.tile(np.arange(4), 3)),
        "adj_dim0": np.arange(12),
        "adj_dim0_0": (("adj_dim0",), np.repeat(np.arange(3), 4)),
        "adj_dim0_1": (("adj_dim0",), np.tile(np.arange(4), 3)),
    },
)
unstacked = test.set_index(
    dim0=["dim0_0", "dim0_1"], adj_dim0=["adj_dim0_0", "adj_dim0_1"]
).unstack(["dim0", "adj_dim0"])

diag_index0 = xarray.DataArray(np.arange(unstacked.shape[1]), dims=("diag_0",))
diag_index1 = xarray.DataArray(np.arange(unstacked.shape[2]), dims=("diag_1",))
unstacked_diag = unstacked.isel(
    dim0_0=diag_index0,
    dim0_1=diag_index1,
    adj_dim0_0=diag_index0,
    adj_dim0_1=diag_index1,
)

diag_index = xarray.DataArray(np.arange(test.shape[0]), dims=("diag",))
test.isel(dim0=diag_index, adj_dim0=diag_index).set_index(
    diag=["dim0_0", "dim0_1"]
).unstack("diag")

Relevant log output
Traceback (most recent call last):
  File "./test.py", line 50, in <module>
    ).unstack("diag")
  File "~/.conda/envs/plotting/lib/python3.10/site-packages/xarray/core/dataarray.py", line 2201, in unstack
    ds = self._to_temp_dataset().unstack(dim, fill_value, sparse)
  File "~/.conda/envs/plotting/lib/python3.10/site-packages/xarray/core/dataset.py", line 4214, in unstack
    result = result._unstack_once(dim, fill_value, sparse)
  File "~/.conda/envs/plotting/lib/python3.10/site-packages/xarray/core/dataset.py", line 4070, in _unstack_once
    variables[name] = var._unstack_once(
  File "~/.conda/envs/plotting/lib/python3.10/site-packages/xarray/core/variable.py", line 1690, in _unstack_once
    return self._replace(dims=new_dims, data=data)
  File "~/.conda/envs/plotting/lib/python3.10/site-packages/xarray/core/variable.py", line 978, in _replace
    return type(self)(dims, data, attrs, encoding, fastpath=True)
  File "~/.conda/envs/plotting/lib/python3.10/site-packages/xarray/core/variable.py", line 2668, in __init__
    raise ValueError(f"{type(self).__name__} objects must be 1-dimensional")
ValueError: IndexVariable objects must be 1-dimensional

Anything else we need to know?
 Maybe some side effect of the indexes refactor in #5692? I just run the example without inspecting the outputs so not sure if everything is right, though.

Just a tip: You don't need any stacking for that.
Just use an indexer with a new dim:
diag = xr.DataArray(np.arange(12), dims="diag")
test.isel(dim0=diag, adj_dim0=diag)

this should do the trick ;)

@DWesl have you tried running your code with Xarray's main branch? If yes, could you confirm whether or not it yields the expected output? Thanks!

Oh, right, you suggested that a bit ago.
When I checkout upstream/main in my local XArray repository root and run the example, it completes without error.  When I fix the example to use the correct dimension, the implicit print on the last line shows the nearly the same as unstacked_diag from a few lines earlier.
Still not sure what fixed this, but since it's working, I don't care.  Thank you!


uninitialized constant Chef::Provider::MysqlServiceSysvinit

Cookbook version
mysql (8.1.1)
Chef-client version
Chef: 12.15.19
Platform Details
CentOS Linux release 7.2.1511 (Core)
Scenario:
I try to manually set the provider.
Steps to Reproduce:
mysql_service node['data']['database']['name'] do
  port '3306'
  version '5.6'
  initial_root_password node['data']['database']['root_rand_p']
  Chef::Provider::MysqlServiceSysvinit
  action [:create, :start]
end

Expected Result:
Service created with correct provider.
Actual Result:
       Recipe Compile Error in /tmp/kitchen/cache/cookbooks/data_data/recipes/default.rb
       ================================================================================

       NameError
       ---------
       uninitialized constant Chef::Provider::MysqlServiceSysvinit

       Cookbook Trace:
       ---------------
         /tmp/kitchen/cache/cookbooks/data_data/recipes/database_setup.rb:34:in `block in from_file'
         /tmp/kitchen/cache/cookbooks/data_data/recipes/database_setup.rb:29:in `from_file'
         /tmp/kitchen/cache/cookbooks/compat_resource/files/lib/chef_compat/monkeypatches/chef/run_context.rb:347:in `load_recipe'
         /tmp/kitchen/cache/cookbooks/compat_resource/files/lib/chef_compat/monkeypatches/chef/run_context.rb:303:in `block in include_recipe'
         /tmp/kitchen/cache/cookbooks/compat_resource/files/lib/chef_compat/monkeypatches/chef/run_context.rb:302:in `each'
         /tmp/kitchen/cache/cookbooks/compat_resource/files/lib/chef_compat/monkeypatches/chef/run_context.rb:302:in `include_recipe'
         /tmp/kitchen/cache/cookbooks/data_data/recipes/default.rb:8:in `from_file'
         /tmp/kitchen/cache/cookbooks/compat_resource/files/lib/chef_compat/monkeypatches/chef/run_context.rb:347:in `load_recipe'

       Relevant File Content:
       ----------------------
       /tmp/kitchen/cache/cookbooks/data_data/recipes/database_setup.rb:

        27:  end
        28:
        29:  mysql_service node['data_data']['database']['name'] do
        30:    port '3306'
        31:    # package_name 'rh-mysql56-mysql-server'
        32:    version '5.6'
        33:    initial_root_password node['data_data']['database']['root_rand_p']
        34>>   Chef::Provider::MysqlServiceSysvinit
        35:    action [:create, :start]
        36:  end
        37:
        38:  # package %w(ruby-devel rubygems-devel)
        39:
        40:  # Install mysql2 gem
        41:  mysql2_chef_gem 'default' do
        42:    action :install
        43:  end

       Platform:
       ---------
       x86_64-linux

After looking over some of my own error logs on a different problem entirely, give this syntax a try:
mysql_service node['data']['database']['name'] do
  port '3306'
  version '5.6'
  initial_root_password node['data']['database']['root_rand_p']
  service_manager "systemd"
  action [:create, :start]
end

Perhaps this service_manager directive should be added to the supermarket documentation page?

You shouldn't need to specify the service manage. It's automatic unless you have a use case where you want to run sys-v on a systemd host. Just leave it out and things should work fine.


TiledMap can be loaded only in xml format and not json.

Basically this is a request for new features.
I would like to have the chance to load a map in json (.tmj) format from TILED. Right now only .tmx formats are loaded.
More i would like to have in the loader also a method to load a defined Tileset file (both formats as well) directly into a TiledMapTileSet.
Thanks a lot for your efforts.
Keep up the incredible good work.

Hi, I'll try to add it.

Hi @patton73, I'm not sure to fully undestand what you mean by having in the loader a method to load a defined Tileset file directly into a TiledMapTileSet. Isn’t it already the case with the TmxMapLoader, which loads a map with the associated tilesets from the given file?

Hi @Kevin-Garnier thanks for the question.
Basically is not the same. Right now you can only load a predefined Map (made with Tiled). So you can load the entire map with tilesets included. But what if i would like to generate a random map using those tilesets?
Right now you cannot do it. You can only load a complete map but not a complete tileset.
According to me this is a cool upgrade to the library.
At least make those methods public, so you can use them to load a fully functional tileset to generate a random map.
Thanks anyway for your effort.
For any question please ask.

But what if i would like to generate a random map using those tilesets? Right now you cannot do it.

@patton73 right now you can load an empty map references those tilesets and use them to generate your maps : map.getTileSets().getTileSet("awesome-tileset"); which is an OK workaround IMHO.
About JSON format, i'm not sure what the benefits using this format but assuming you get a tilemap/tileset from someone else, you could re-export them as XML from Tiled.
I can understand that it could be nice to have full support at some point but those are easily workaround-able until then.

Since in Tiled i can have tileset files outside the map.

@patton73 actually you can load TMX maps with libGDX which has separated TSX files, in this case tilesets are not embeded into the TMX map, it only references them. So your code doesn't need to know if tilesets are embeded or referenced by the tilemap, which could be a plus in term of assets organization for some people. That said, i can understand it could be more handy for some people to directly load TSX files, no doubt.

I've already done the json format on my fork, I can do that too

@Kevin-Garnier : I saw you opened a PR with some Tiled enhancements which is great.
Can you please also include a fix that the loader does not crash with properties of type file? It crashes in a castProperty method of the loader because it does not support that type.
Imo it should not crash and simply load it as a string (no cast necessary).
Also, imo text objects are also not loaded correctly. They don't contain any of the text prooerties that Tiled stores.
If you have time and motivation it would be great if you can add the support for this as well ☺

Sorry for the delayed response. Is the issue only with the JSON part, or does it also affect the XML part? If it's both, I think it would be best to create a new issue, as it may require more extensive modifications.

It is a general issue of the original loader. But seeing how slowly (never?) PR are merged I think we can simply ignore that. I have implemented a custom loader that overrides the problematic method and fixes it for my own needs.
Thanks anyway for your response 😊

It is a general issue of the original loader. But seeing how slowly (never?) PR are merged I think we can simply ignore that. I have implemented a custom loader that overrides the problematic method and fixes it for my own needs.
Thanks anyway for your response 😊

Please can you share the code since it seems like having a PR that closes the bug?
Thanks.


[Bug Report] v-popover not supporting camel case

Element UI version
1.2.9
OS/Browsers version
macOS 10.12.4/chrome 58.0.3029.81
Vue version
2.3.0
Reproduction Link
https://jsfiddle.net/4fpq3L5f/1/
Steps to reproduce
set the el-popover ref and v-popover under camel case naming convention will not trigger the pop over effect correctly.
It was easy to reproduce this issue and I just made a littile change from a jsfiddle link on official docs, you can check it here
What is Expected?
maybe make v-popover just like other directives so we can use it like v-popover="xxxxx" instead of v-popover:xxxx for consistency, and then it could be correctly compiled under camel case naming convention.
What is actually happening?
not sure about that but I was guessing that the browser turned v-popover:xxxx as an attribute to lower case so it throwed a type error when compiling.

For the reference element, it is recommended to use the named slot rather than binding a directive: https://jsfiddle.net/du9aa5ys/

yeah there are certain ways to avoid this issue but I think lots of people will just use a directive like the examples in official docs. so imho it would be easy to be trapped by this bug since javascript is heavy camelcase named in most circumstances.
Anyway its always good to let inexperienced developers know that this directive won't support camelcase.


Created markdown to build the models!

this was a mistake to exclude. Closes #8

Fixed, this should work for you @mdnestor


steven: flying addictions

I made some fixes, I know it can be wrong use of trigonometric functions, but looks nicer :)

What is the "format" step?

It's only placeholder code until minecraft like flying is implemented.
The format step checks the formatting of the code, if it is incorrect the build will fail.


Control plane failure modes for high-availability documentation

We likely need some brief documentation on what customers can expect in terms of the reliability of the control plane. We discussed the "majority" vs "less than majority" buckets of problems, would be great to have documentation that we can point to, in order to justify our reliability stance

We likely need some brief documentation on what customers can expect in terms of the reliability of the control plane. We discussed the "majority" vs "less than majority" buckets of problems, would be great to have documentation that we can point to, in order to justify our reliability stance

when speaking about "majority" is this about etcd's raft algorithm? k8s core doesn't have this requirement directly.
also, when / where was this discussed?

/kind feature
/triage needs-information
/sigs docs
(tagging with docs until owner is established, if ever)

It'd be good to understand the gaps: what should https://kubernetes.io/docs/setup/production-environment/tools/kubeadm/high-availability/ cover that it doesn't?

/close
the ticket has missing information; questions were not answered.
please update and re-open.

Hi all, really sorry for the delay on my elaboration of this issue!
The context is that my team is working on Kubernetes reliability (as part of a product) and we want to understand the failure modes of the control plane. I had a chat with Han Kang about this offline, and I wanted to amend the details of this issue with our conversation of what I think is missing, but I wanted to review the links you all sent first to see if I was missing something. @sftim thank you very much for sending it over!
The part I wanted the most is the expectations of restrictions when one or more nodes of the control plane are down. We're currently working with a setup that considers HA as three control plane nodes, so we were trying to understand what were the consequences of:

A single node being down
The majority of nodes
All of them (we assume cluster down, but just for completeness)

So what I was asking was "what Kubernetes customers can expect in case of failure of their control plane nodes".
Let me know if this makes sense, and sorry again for the delay

what you are talking about makes sense, @royalsflush
please include more detail in the OP post:
https://github.com/kubernetes/website/issues/43849#issue-1981863546
i don't mind us including more documentation about failures and recovery of the CP, as the documentation is lacking.
let's see what is actionable here.
/reopen

/sig architecture
/sig api-machinery
/remove-triage needs-information
Please revise (edit) the original issue description @royalsflush to explain what you want added to the documentation. You could write this as a user story or as a definition of done.

/assign
(I can take this, if y'all don't mind)

Thanks @logicalhan. These things are important.

/triage accepted
/priority important-longterm

I would add that we ideally ought to cover some of the less common situations too. I'll outline some below. What I hope is that someone carefully reading the docs can answer what the expected outcome is, without actually setting up a cluster or reading any source code.
“answer“ means working out if the expected behavior as seen by a client is: API usable; API unavailable / degraded; undefined behavior
Eg:

three control plane nodes (1 per zone); separate etcd hosts (1 per zone); full failure in exactly one zone; “perfect” client-side load balancing and retries
three control plane nodes (1 per zone); separate etcd hosts (1 per zone); etcd healthy but full API server failure in exactly one zone; “perfect” client-side load balancing and retries
even number of control plane nodes, of which all are healthy; separate etcd cluster has odd number of nodes and some (but fewer than half) have failed; “perfect” client-side load balancing and retries
even number of control plane nodes, only half of which all are healthy; separate etcd cluster  has odd number of nodes and some (but fewer than half) have failed; “perfect” client-side load balancing and retries
stacked 3-node control plane; each API server only speaks to local etcd; one etcd fully unavailable; “dumb” round-robin style load balancing without health checks

I'm sure we could think up more; maybe we even have a list already?

We can produce - and publish - docs without meeting this ideal; I've mentioned it so we understand where we'd like to end up.

These questions need not appear in the page; you could think of them as like unit tests for the docs. In other words, if a reviewer picks a question, can they - just by reading what's in the page - work out what the answer must be?
(we could even ask a large language AI model to help us check)

I dig the framing.

https://github.com/kubernetes/website/pull/43903 feels slightly relevant (only slightly, though)

+1 to covert upgrades and rollback.
in KEP PRRs we require "downgradability" of k8s features, but etcd by design does not support downgrade well, yet:
https://github.com/etcd-io/etcd/issues/15878#issuecomment-1567986308
kubeadm as a whole also does not support downgrades. it supports rollback, in case of component failure, but that may or may not work, depending on:

if it was a k8s component, hopefully all features, skews, etc properly guarantee downgrade
if it was etcd, who knows

#43903 feels slightly relevant (only slightly, though). I don't know how much we want to also cover upgrades and how they impact failure modes.

it's a bug in kubeadm's api-machinery usage and the etcd upgrade failure will trigger a rollback, unless the user workarounds it.
but since the rollback will restore an etcd with the same version, it will act as a component restart.

+1 @sftim , Can you reshare the docs for gaps

+1 @sftim , Can you reshare the docs for gaps

I don't understand what you'd like me to do here @kumarankit999. How would you know when I'd done what you're asking (can you frame it as a definition of done)?
If you mean https://github.com/kubernetes/website/issues/43849#issuecomment-1799444308, I was the person who asked the question, and I do not have the answer to it.


definitions.json - Missing neutral gender

The gender enumeration in the /core/definitions.json is restricted to two genders (male/female). There should be one gender-neutral option available.
Sticking to the CDC standard at least the third option "o - others" should be added.

Dear @lechnerc77
thank you for your contribution and sending the first pull request!
We are looking forward to other suggestions for improving the Global Insurance Ontology.


Add contracts to Defender through SDK

It seems that there is no POST/ method for contracts in the SDK docs.
https://www.api-docs.defender.openzeppelin.com/#defender-sdk-admin
Not sure if this is a doc issue or a missing endpoint, but it would be really useful to add the contracts programatically.
Thanks!

Hi @Ramarti there is a way:
  const client = new Defender(creds);
  const contract = await client.proposal.addContract({
    contractName: 'Test',
    contractPath: 'contracts/Test.sol',
    network: 'sepolia',
  });


Have dumpprivkey be able to export a version that can be used in paper wallet app

Either make paper wallet app use key as is or add a -p to dumpmasterwalletprivkey to make a output paper wallet app can use

Assignment request. @sherlockcoin

assigned

@aguycalled is this still an issue? might be related to the key dumping scenario that I talked about on discord.


Cannot find module 'escape-html'

Hi Dan,
Thanks for all update.
Updating my local version with sudo npm install -g localturk worked ok.
After upgrade below error has been :
module.js:544
    throw err;
    ^

Error: Cannot find module 'escape-html'
    at Function.Module._resolveFilename (module.js:542:15)
    at Function.Module._load (module.js:472:25)
    at Module.require (module.js:585:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/usr/local/lib/node_modules/localturk/node_modules/errorhandler/index.js:18:18)
    at Module._compile (module.js:641:30)
    at Object.Module._extensions..js (module.js:652:10)
    at Module.load (module.js:560:32)
    at tryModuleLoad (module.js:503:12)
    at Function.Module._load (module.js:495:3)

I've fixed it by running:
sudo apt-get install node-escape-html
Having said that, I've noticed that npm installed file points to ".js" file and not the new ".ts".
Regards.

Thanks for the heads up. I've added the undeclared dep in the linked commit.
The .js files are executed is intentional. The .ts files are only for development.


Would you like to package based on materia widget, which of the following is better and the performance will be higher

Steps to reproduce
Would you like to package based on materia widget, which of the following is better and the performance will be higher
Code sample

class CustomAppBar extends AppBar {
  CustomAppBar({
    super.key,
    super.leading = const Icon(Icons.arrow_back_ios_rounded),
    super.automaticallyImplyLeading,
    super.title,
    super.actions,
    super.flexibleSpace,
    super.bottom,
    super.elevation,
    super.scrolledUnderElevation,
    super.notificationPredicate,
    super.shadowColor,
    super.surfaceTintColor,
    super.shape,
    super.backgroundColor,
    super.foregroundColor,
    super.iconTheme,
    super.actionsIconTheme,
    super.primary,
    super.centerTitle = true,
    super.excludeHeaderSemantics,
    super.titleSpacing,
    super.toolbarOpacity,
    super.bottomOpacity,
    super.toolbarHeight,
    super.leadingWidth,
    super.toolbarTextStyle,
    super.titleTextStyle,
    super.systemOverlayStyle,
    super.forceMaterialTransparency,
    super.clipBehavior,
  });
}

class CustomAppBar extends AppBar {
  CustomAppBar({
    Key? key,
    Widget? leading,
    bool automaticallyImplyLeading = true,
    Widget? title,
    List<Widget>? actions,
    Widget? flexibleSpace,
    PreferredSizeWidget? bottom,
    double? elevation,
    double? scrolledUnderElevation,
    ScrollNotificationPredicate notificationPredicate = defaultScrollNotificationPredicate,
    Color? shadowColor,
    Color? surfaceTintColor,
    ShapeBorder? shape,
    Color? backgroundColor,
    Color? foregroundColor,
    IconThemeData? iconTheme,
    IconThemeData? actionsIconTheme,
    bool primary = true,
    bool? centerTitle,
    bool excludeHeaderSemantics = false,
    double? titleSpacing,
    double toolbarOpacity = 1.0,
    double bottomOpacity = 1.0,
    double? toolbarHeight,
    double? leadingWidth,
    TextStyle? toolbarTextStyle,
    TextStyle? titleTextStyle,
    SystemUiOverlayStyle? systemOverlayStyle,
    bool forceMaterialTransparency = false,
    Clip? clipBehavior,
  }) : super(
          key: key,
          leading: leading,
          automaticallyImplyLeading: automaticallyImplyLeading,
          title: title,
          actions: actions,
          flexibleSpace: flexibleSpace,
          bottom: bottom,
          elevation: elevation,
          scrolledUnderElevation: scrolledUnderElevation,
          notificationPredicate: notificationPredicate,
          shadowColor: shadowColor,
          surfaceTintColor: surfaceTintColor,
          shape: shape,
          backgroundColor: backgroundColor,
          foregroundColor: foregroundColor,
          iconTheme: iconTheme,
          actionsIconTheme: actionsIconTheme,
          primary: primary,
          centerTitle: centerTitle,
          excludeHeaderSemantics: excludeHeaderSemantics,
          titleSpacing: titleSpacing,
          toolbarOpacity: toolbarOpacity,
          bottomOpacity: bottomOpacity,
          toolbarHeight: toolbarHeight,
          leadingWidth: leadingWidth,
          toolbarTextStyle: toolbarTextStyle,
          titleTextStyle: titleTextStyle,
          systemOverlayStyle: systemOverlayStyle,
          forceMaterialTransparency: forceMaterialTransparency,
          clipBehavior: clipBehavior,
        );
}

What target platforms are you seeing this bug on?
Android
OS/Browser name and version | Device information
Android
Does the problem occur on emulator/simulator as well as on physical devices?
Yes
Logs
Logs
[Paste your logs here]

Flutter Doctor output

@stuartmorgan   @dam-ease      Can you help me? Thank you

Please see https://flutter.dev/community#community-grid for a list of resources where you can get advice on your own projects. Closing as this is neither a bug nor a feature request.
Also, do not ping random people in issues.


Feature: Add User Agent Information

It would be good to enhance things further to provide the option to have a header that gives the user/agent.

There was already a PR to add this and a conclusion that it is already possible: https://github.com/nautobot/pynautobot/pull/171#issuecomment-2104209674

My two cents: We actually ran into a case recently where having the client send a vendor-side user-agent would have been useful; specifically, being able to run reports to determine what version of pynautobot our customers were using; having that in the user-agent header would have been useful.

Closed via #207


Discovery Async

Description
Makes the same change as sync Discover, but discovery is done in a thread.

Fixes a bug where some Feature Flag requests didn't have headers.

All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[X] I have read the contribution guidelines.

General Guidelines and Best Practices

[X] Title of the pull request is clear and informative.
[X] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-appconfiguration-provider


Feature/allow mixture of bounded and unbounded fit parameters

No tests added (so far).
Fixes #931

We are currently using a workaround to get this feature to work. @cdrnet Any chance of getting this integrated the near future?


[Feature Request] Output Restricted by Context Free Grammar or JSON Schema

🚀 Feature
Pass in a grammar or JSON schema to restrict the output of generated tokens. This would make data extraction and potentially tool-usage use cases simpler to implement.
Motivation
Having the ability to constrain responses to a specified grammar or a JSON schema would unlock data extraction and function calling use cases.
Alternatives
Prompt-engineering isn't sufficient and not transferrable between models.
Fine-tuning would be a much heavier lift compared a grammar that could drive output.
Additional context
Some prior art to consider:
https://github.com/ggerganov/llama.cpp/pull/1773
https://huggingface.co/spaces/mishig/jsonformer
https://github.com/normal-computing/outlines
https://github.com/r2d4/rellm
Great project, I have been able to get highly performant and high quality response in a couple of hours of effort. Huge kudos to the MLC team and to Simon Willison for this project that got me started with his llm library:
https://github.com/simonw/llm-mlc

Probably duplicate of #499
It seems to me it's out of scope for this project to have native support for JSON schema.
We can easily apply a layer like outlines on top of a MLC LLM if the support for something like logits_processor is added


[ACR] Deprecate support for Azure Germany

Packages impacted by this PR

@azure/container-registry

Issues associated with this PR

Fix #24128

Describe the problem that is addressed by this PR
Azure Germany is deprecated. This PR marks the corresponding audience in KnownContainerRegistryAudience as deprecated.

API change check
API changes are not detected in this pull request.


Update versionlist files after packing new data

Not gonna do this in the standard tool, a plugin may be eventually used for this


Authentication system for elfeed-web

Hello!
It isn't critical, but would be very nice if elfeed web api had some kind of authentication. I don't know simple-httpd so I don't know how difficult is implement this. Do you think it is possible?
Thanks

It's definitely possible. HTTP basic access authentication is trivial
and I've implemented it on simple-httpd before. However, TLS isn't
something that's easy to retrofit onto simple-httpd, if you care about
that.
In this case, I think an access token would be more appropriate than a
username/password combo. Have elfeed-web generate a random token and add
it to a list, which gets persisted somehow (configuration,
customization?). Every POST/PUT/DELETE would require the token. Since
there's no TLS, maybe we'd even want something like HTOP to prevent
replay attacks (only an active MITM could mess things up).


DOC: Clarify that res-2 entity does not mean 2mm

@bbfrederick brought up the problem that, although the documentation does not state the above explicitly, it is the most plausible reading.
This PR adds a danger admonition to clarify the ambiguity.
Follow-up: templateflow/templateflow#92.
Resolves: #1847.

@bbfrederick I took the liberty of setting you as a reviewer for this PR, please let me know if you would like I keep doing so sparingly.
That said, given the small size of the changes, we might go ahead with it soon if you can't find a moment to check (or are on vacation, which is also highly likely at this point of the year).


Comment out update hooks that were failing.

I left the updates in to make it easier to see what happened
but i noticed from this deploy that update hooks weren't running
Fixes #6188

Hokay! :+1:


Update caffeine to 2.9.0 and tell dependabot to ignore 3.x as for jdk 11 only

Coverage remained the same at 93.333% when pulling 3fa29880b482a88f81eb87d1cc3eb77433595974 on hazendaz:master into 0b3e77b2e5f3a75d9ecada67482092689558a1e2 on mybatis:master.


fix(blockifier_reexecution): remove double error redirect

This change is

crates/blockifier_reexecution/src/state_reader/serde_utils.rs line 11 at r2 (raw file):
Previously, dorimedini-starkware wrote…
don't use super imports outside of test modules; use crate::...

Done. The compiler does this by default. Do you know if there's a way to change this behaviour?


Upgrade version of rpc starter to v5.4.0.

Pull Request Test Coverage Report for Build 230

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

Totals

Change from base Build 228:
0.0%

Covered Lines:
2033

Relevant Lines:
2990

💛  - Coveralls


Nested Sitemaps

Details
Describe The Bug
i have a ,multi language website and i need a sitemap schema like this

ar_sitemap
2.products_sitemap
3.categories_sitemap

en_sitemap
2.products_sitemap
3.categories_sitemap

the issue is when im trying to create nested configuration as explained in documentation, i came up with this configuration
 sitemap: {                     sitemaps:{                      en:{                          products:{                               resources:['/api/__sitemap__/urls/en/products'] }   ,         categories:{                               resources:['/api/__sitemap__/urls/en/categories'] }    } }             },
problem is that in the sitemap preview, it renders first level like en_sitemap.xml. but when i go deeper to the next level there is nothing like products_sitemap.xml and categories_sitemap.xml. i wanted to know that is there anything like this implemented in the package? or am i doing something wrong in the configuration?
Reproduction

System / Nuxt Info
`- Operating System: Windows_NT

Node Version:     v20.11.0
Nuxt Version:     3.10.1
CLI Version:      3.10.0
Nitro Version:    2.8.1
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools, app, css, link, postcss, modules, i18n, pwa, runtimeConfig, experimental, site, sitemap
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>nuxt-schema
<EMAIL_ADDRESS>Build Modules:    -
`

The sitemap protocol doesn't support multiple index sitemaps, you should use a flat structure.


ZeroDivisionError

Reading: 100.00 %Traceback (most recent call last):
  File "./monkeytest.py", line 193, in <module>
    main()
  File "./monkeytest.py", line 189, in main
    benchmark.print_result()
  File "./monkeytest.py", line 165, in print_result
    max=self.read_block_b / (1024 * 1024 * min(self.read_results)),
ZeroDivisionError: float division by zero

I see the same


trace API: fix missing/inconsistent doc of category items

Make OSSL_trace_set_channel.pod and openssl.pod consistent again with trace.h.

This pull request is ready to merge

Squashed and merged to 3.0 and master - thanks @levitte and @t8m


Feat: Has a plan to support case-sensitive with byKey=true?

We could distinguish between A and a by keydownEvent.key.
Has a plan to support option caseSensitive when byKey is true?
I'm pleasure to make a pr.

Hey @imcuttle that sounds like a good idea, I'm down for a PR!


Cost percentages incorrect when there are many plans in a single document

As per this issue:
https://github.com/opserver/Opserver/issues/139
Looks like the percentage is being calculated incorrectly because there are multiple queries / plans.

Caused because the estimated operator cost was being incorrectly calculated as 271, instead of 0.003. Already fixed by #1.


Document values that can be used with the Encoding parameter

The documentation for DoCmd.OutputTo says nothing about which values are valid for the Encoding parameter, only that it is of Variant data type.

@zspitz Hi, the types of formats are  MS-DOS, Unicode, or Unicode (UTF-8). The MS-DOS argument setting is available only for text files. If you leave this argument blank, Access outputs the data by using the Windows default encoding for text files and the default system encoding for HTML files. Have you tried asking this question on Stack Overflow? You can ask Access developers questions about your code and maybe somebody has more experience using the Encoding parameter.
Ask Access developers questions on Stack Overflow
Meanwhile, here's more info on character encoding. You could try other formats and report back to us what you've discovered!
Choose text encoding when you open and save files
Choosing & applying a character encoding.
I updated the topic, thanks for the suggestion.

@lindalu-MSFT Thanks for taking the time and effort to improve this; the documentation for this parameter is much better now.
However, I would suggest that describing the types of the formats still doesn't explain which values I need to pass in: is there an acOutputFormatEncoding with three different values; or am I supposed to pass it in as a string: the string "Unicode" if I want the Unicode option, or the string "UTF-8" if I want the UTF-8 option; or is there some sort of ISO standard naming of encodings that I should use with this method.
Passing the question off to StackOverflow seems a little odd to me, as any answer from StackOverflow will almost certainly be the product of trial and error, and will probably be incorrect in some detail. If nobody knows what is supposed to happen when passing in various values here (I imagine that if someone did know, it would already be part of the documentation), isn't it more appropriate to turn to someone at Microsoft who has access to the source code for the OutputTo method, and can describe what will happen for various values?


Add a separate LICENSE.md file with MIT License, and point to it from README.md.

Added an explicit license for the code, using the standard naming convention in github. I'm proposing the MIT license as that's a very common permissive license, and the license used by plotly and vue. I don't think there's any data actually in the repository, since it's all pulled dynamically from external sources, so there is no need to address that.

Hi @saufrecht. Thanks for suggesting this. We discussed this and @mustpax went ahead and put in an MIT license with #31. Good call on linking to it in the readme. Feel free to make or edit your PR for this, if you have a moment.

replacing with new pull request


Console Spam when looking at chisel and bits and beetroot plants

I am not sure about the beetroot.  It won't show growth.  All other vanilla plants are fine.  When I look at growth it errors and according to the log it errors because of chisel and bits.  If I look at chisel and bits things it errors out also.
Could I exclude chisel and bits from the probe so it won't show it?
Here is the log

Not sure about beetroot but the Chisel & Bits thing is as far as I can see a bug in Chisel & Bits. Their implementation of getPickBlock() doesn't seem to work server-side.

Is there a way to tell TheOneProbe to not show anything from Chisel and Bits?

I'm going to talk to the author of Chisel & bits about this and try to get this bug fixed. For the moment you cannot avoid it doing this I'm afraid

Here is a log without Chisel & Bits.  Just looking at the growth of beetroots.
http://pastebin.com/wsDJsJvL

Ok. I will check that out

The beetroot problem will be fixed next release


MM-37485 MM-37602 MM-37610 MM-37684 MM-37719 MM-37522 MM-37720  App crash, Sync issue fixes

Summary

Fixes app crash on opening channel header
Fixes app crash on opening permalink from permalink screen
Fixes showing duplicates channels while searching in "Jump to..."
Fixes auto following the thread on opening the thread view when there are unreads
Fixes updation of threads state (Read/Unread/Deleted) of already loaded threads
Fixes counter showing as blank due to threads sync issue
Doesn't show Follow Thread/Message when CRT is not enabled

Ticket Link
https://mattermost.atlassian.net/browse/MM-37485
https://mattermost.atlassian.net/browse/MM-37602
https://mattermost.atlassian.net/browse/MM-37610
https://mattermost.atlassian.net/browse/MM-37684
https://mattermost.atlassian.net/browse/MM-37719
https://mattermost.atlassian.net/browse/MM-37522
https://mattermost.atlassian.net/browse/MM-37720
Device Information
This PR was tested on: iOS 14.5
Release Note
NONE

Building app in separate branch.

Building app in separate branch.

Thanks @anurag6713
Seeing few issues

MM-37610 Verified tapping on the highlighted permalink reply post no longer crashes the app.
There is still an issue where the post is highlighted on the main channel view, not on the thread. Let me know if you want this filed separately, since the crash is fixed and this feels out of scope of that particular ticket.
permalink view

main channel, after jump to recent

MM-37719 Verified deleting doc and data no longer reduces number of items displayed on global thread. Seeing an issue where global threads item is not bolding on the channel sidebar after delete doc and data if unread threads are still present.

MM-37720 Verified long press menu the option to follow a thread is not available when CRT if off
 on the threads view. Option to follow message is still available on the main channel view for root and reply posts.

Thank you @jgilliam17.
Regarding the truth post getting highlighted, we have a ticket for it here https://mattermost.atlassian.net/browse/MM-37517

Thanks @anurag6713
I can reproduce 2. consistently - Threads item is not bolded on the channel sidebar after deleting documents and data until user taps on the Global Threads item again.

@jgilliam17 can you confirm what issues are actual blockers and what can be addressed separately (in the next release)?

cc// @anurag6713 @ashishbhate @enahum ^

@esethna I have already reviewed https://github.com/mattermost/mattermost-mobile/pull/5612#pullrequestreview-726599476

@esethna 3. is a blocker as it causes the app to crash

There is a follow up ticket https://mattermost.atlassian.net/browse/MM-37517
I discussed with Anurag, not a blocker. I'll create a ticket for it.

@jgilliam17 can you clarify the behaviour that causes the crash for (3), sorry I might be missing something based on your review above

Sure. Option to follow root post and replies is available on the main channel long press menu when CRT is off
steps:

toggle CRT off on daily
login on daily on the mobile app
view any channel
long press on any post (root or reply)
Observed: Follow Message option is available while CRT is off

https://mattermost.atlassian.net/browse/MM-37720
@jgilliam17 Can you test on the latest build? I have added that fix after the 1st build.

With CRT:

Without CRT:

Can you test on the latest build? I have added that fix after the 1st build.

Yes, give me a sec to find and install the latest app

@esethna @anurag6713 Please disregard 3. It's no longer reproducing. Fixed.

@enahum if all looks good can you please approve?
@jgilliam17 thanks, does that mean this PR is QA approved given the followup items?
@ashishbhate please help with a priority dev review :)

@esethna I won't until QA approves and there are no new changes

@esethna found a bug where CRT is disabled but the post options show Follow Message which is should not be present in the options menu.

@enahum we are checking in the index.ts and setting thread as null in case crt is not enabled. Agreed it's not clear and I will move the condition here for clarity.

@enahum we are checking in the index.ts and setting thread as null in case crt is not enabled. Agreed it's not clear and I will move the condition here for clarity.

@anurag6713 yes, so it later on.. so that is why I removed my comment and approved again

Cherry pick is scheduled.


Update doc building instructions and pin dependencies

This pins the dependencies for the docs to specific versions and adds pytest as a dependency which was missing but required when I did a build. It also moves the doc instructions to a .md so GitHub knows how to render it nicely in the browser. Tested out the requirements.txt with a fresh ubuntu:focal Docker image to verify that the required depedencies work.
Thanks for contributing to TVM!   Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers by @ them in the pull request thread.

@areusch can I get a review?

Thanks @driazati . Considering the fact that not everybody have a GPU machine in their env. It would be good to "promote" the instruction for building docs without running tutorials(which can actually be most of the cases).
This will also answer the common question "how do i build docs quickly on my local env in a few mins"
https://github.com/apache/tvm/blob/main/tests/scripts/task_sphinx_precheck.sh#L39 does that/


Add a :formatter option to Kino.DataTable.new/2

See the discusssion in #440.
Adds a :formatter option to Kino.DataTable.new/2 which is a 2-arity function passed the key (column name) and cell value.
The special key :__header__ is passed when formatting the column headings.

Hey @kipcole9! I like the idea, just a small note on the API and we can ship it :D

Ah and as for tests, please add one in test/kino/data_table_test.exs. For example here:
https://github.com/livebook-dev/kino/blob/93d042a7c72c77584004644391a52c7d2ef6f8ee/test/kino/data_table_test.exs#L249-L261
The data: [...] carries the already formatted values.

@jonatanklosko, thanks for the review and great suggestions. I've made the requested changes, tests are passing. Now I just need to add a new test as you proposed.

Test case added so I think this is now ready for the next review.

@kipcole9 perfect, last two nitpicks!

Fixed as requested. Let me know if there's anything else?


Support for 1v1 Display

@semerj continuing conversation for comparison here with reference to commits:
https://github.com/SFCrime/SFC-database/commit/9e84523bbc92923bcc77ec651fb384176024999b
https://github.com/SFCrime/SFC-viewer/commit/15ea4b2e285c7aee3413948cbf064da48037f2f6
I think we should probably try and hit the API twice because it just keeps things simple and the backend might want to be comparison agnostic. On the other hand, I think that keeping it in one model might help with linking/story telling. I'll think about the best way to do this.
As for the stories, yeah, that's one thing that really excites me.

I think start_date_1 and start_date_2 is the simplest way to do this...at least for now. The better decision is probably the former one, longer term but shorter term it's a lot more work and thinking. Will update soon.

When you get a chance, could you add a sample geojson to the repo so I can get crackin' on the viz?

just get pull both and make a shape, I added in some default dates for the model so it'll give it back to you just like it would be.


Masonry: MasonryInfinite for infinite fetching

Pulled fetching out from Masonry
MasornyInfinite is exported as the old Masonry
and should function the same
export MasonryBeta as it can be used with custom
fetching logic
This provides a graceful migration path.
Final migration will involve renaming
MasonryBeta/MasonryInfinite/Masrony
Existing Masonry integration will be testing MasonryInfinite to ensure
compatibility

Deploy preview for gestalt ready!
Built with commit ee2766ed15669d5598d435df5b4f08338d758a8e
https://deploy-preview-307--gestalt.netlify.com


Current docker image contains ~125 known vulnerabilities

The current docker image for version 1.1.0 contains 125 known vulnerabilities according to Jfrog's Xray. Almost all of these vulnerabilities come from the old base layer. Bumping the base image to 2.6-slim should mitigate almost all of these vulnerabilities.
What would you like to be added:
Use 2.6-slim as a base layer.

Closing as this issue is fixed in the latest release - hub.docker.com/repository/docker/splunk/fluentd-hec


[nl] update bar sorting

updating sorting bars in a bar chart to not have a default order if there is more than 1 sv because for some svpg, we want to show it in a specific order
e.g., we want age distribution to be in the order of the ranges
prod: https://screenshot.googleplex.com/BVyKS3ZWk9pW8vk
local: https://screenshot.googleplex.com/BPdhLpgGMfDx8tq

thanks for the review!


Debarcer group function does not check for empty regions

Debarcer group function fails with errors when it encounters a region bed with no reads in the bam.
I had assigned a small python fix for myself when I implemented this myself;

def checkReg(bam, region):
status = []
chrom = region.split(":")[0]
spos = int(region.split(":")[1].split("-")[0])
epos = int(region.split(":")[1].split("-")[1])
bam_file = pysam.AlignmentFile(bam, "rb")
reads = bam_file.fetch(chrom, spos, epos)
for read in reads:
if read:
status.append(True)
else:
status.append(False)
return (status)

before calling group and collapse for a region:

for region in part:
status = checkReg(bam, region)
if not status:
continue

Can something similar be introduced as a part of group?
Where it checks if the bam file has enough coverage before grouping

An assertion error is thrown during grouping when no UMIs are found in a genomic region
This assertion statement is in umi_network_collapse.py in code borrowed from UMI-tools
[2020-09-08 13:21:05] Grouping UMIs...
Counting UMIs...
Clustering UMIs...
Traceback (most recent call last):
  File "/u/rjovelin/debarcer_rjdev/debarcer/debarcer/debarcer.py", line 806, in <module>
    args.func(args)
  File "/u/rjovelin/debarcer_rjdev/debarcer/debarcer/debarcer.py", line 154, in group_umis
    umi_families, umi_groups, umi_positions, mapped_reads = get_umi_families(contig, region_start, region_end, bam_file, pos_threshold, dist_threshold, args.ignore, args.truncate, args.separator)
  File "/u/rjovelin/debarcer_rjdev/debarcer/debarcer/src/umi_error_correct.py", line 286, in get_umi_families
    umi_groups = cluster_umis(umis, counts, dist_threshold)
  File "/u/rjovelin/debarcer_rjdev/debarcer/debarcer/src/umi_error_correct.py", line 90, in cluster_umis
    umi_groups = clusterer(umis, counts, dist_threshold)
  File "/u/rjovelin/debarcer_rjdev/debarcer/debarcer/src/umi_network_collapse.py", line 216, in __call__
    assert len_umis, "No UMIs present(!)"
AssertionError: No UMIs present(!)

A couple of comments:

It is assumed users have identified regions of coverage before running debarcer
debarcer bed command can identify regions of coverage and generate a bed if such regions are unknown

The proposed logic loops over each read in the region, which is not efficient.
Instead I introduced a new parameter in the group and run commands, readcount and method count on pysam AlignmentFile object to count the number of mapped reads in a given genomic region given by contig, start and stop.
Grouping is done if the number of reads in region is greater than the parameter readcount. Default value is 0.
Collapse will throw a ValueError if outpuf files from grouping are not generated.


can't deploy in southeastasia

We're currently blocked rolling out the southeastasia RP.  When the RP VMs boot, we get the following repeatable error:

[stdout]
Red Hat Enterprise Linux 8 for x86_64 - BaseOS   47 MB/s |  15 MB     00:00
Red Hat Enterprise Linux 8 for x86_64 - AppStre  48 MB/s |  15 MB     00:00
Red Hat CodeReady Linux Builder for RHEL 8 x86_  12 MB/s | 2.9 MB     00:00
Microsoft Azure RPMs for Red Hat Enterprise Lin  24 kB/s | 1.4 kB     00:00
(try to add '--allowerasing' to command line to replace conflicting packages or '--skip-broken' to skip uninstallable packages or '--nobest' to use not only best candidate packages)
[stderr]
++ yum -y update -x WALinuxAgent
Error:
Problem: The operation would result in removing the following protected packages: systemd

https://github.com/Azure/ARO-RP/pull/534


Get current items selected

Hi,
I need to implement the vehicle routing problem but with additional constraint which is the bin packing.
So every time the algorithm checks if the next item is OK to add to current list of selected items, it should retrieve currently selected items and try to 3D bin pack them plus the new item. If they fit then it should say OK. Can I somehow get such information about currently selected list if items while trying to check if the new item fits?

Each job has a UserData object, you can add any data to this as you like:
From the WHATS_NEW document:
2017-06-08 new release v1.7.2

one can now set custom properties/user defined data (see pr: https://github.com/graphhopper/jsprit/pull/348)
For example, if one wants to enrich a service with an arbitrary number of custom properties, set a map like this:

Service one = Service.Builder.newInstance("s").setLocation(Location.newInstance("loc"))
    .setUserData(new HashMap<String, Object>()).build();

When creating your new constraint, you can then access all inserted jobs for the current iteration and thus their custom UserData.


manager runs on Mac, and other openpilot for PC fixes

Hmm, so this is actually broken. The C params API is using a different directory, guess it doesn't matter for the tests?

Test car models is timing out still!


RobloxGui/Modules variable usage

A few times CoreGui.RobloxGui was used when RobloxGui was already initialized.
TopbarConstants had an incorrect comment title.

This PR has conflicts, please resolve them.
Also, some CoreScripts use CoreGui.RobloxGui.Modules or RobloxGui.Modules when Modules is already defined and initalized. I do think you should fix this too.
Thanks for helping out :+1:

Done.

Looks like there are conflicts, will close for now but can look at if these issues still persist

I could have fixed the conflicts. Though, I'm sure this sort of PR isn't vital to your code so unless requested I will not resubmit it


AboutUs update

yea

Pull Request Test Coverage Report for Build 605

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

Totals

Change from base Build 604:
0.0%

Covered Lines:
2999

Relevant Lines:
3291

💛  - Coveralls


Add short note and link for h5z-zfp

Wasn't sure of best place to put. Added to end of section on the zfp file compression command. I adjusted the title of the section slightly so that it was a little more inclusive.

@markcmiller86 I agree that we should advertise H5Z-ZFP, but this doesn't seem the right place for it as the intent of this section is to document the zfp command-line utility.  H5Z-ZFP and other plugins (for ADIOS, Intel IPP, TTK, ...) are discussed in the overview section.  Admittedly we could do a better job highlighting these plugins elsewhere in the documentation.
I don't think we want to endorse only HDF5, so perhaps the best solution is to add a reference from the File Compressor section back to the overview section and mention that there is zfp support for doing file I/O through other I/O libraries.  Does that make sense?

I don't think we want to endorse only HDF5, so perhaps the best solution is to add a reference from the File Compressor section back to the overview section and mention that there is zfp support for doing file I/O through other I/O libraries. Does that make sense?

Sure. That makes perfect sense. I wasn't aware of those external references anywhere in these docs.

Codecov Report

Merging #68 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop      #68   +/-   ##
========================================
  Coverage    90.34%   90.34%
========================================
  Files           64       64
  Lines         3448     3448
========================================
  Hits          3115     3115
  Misses         333      333

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 ae26c99...fa22af2. Read the comment docs.

@lindstro Sorry for the delay. It wasn't clear to me if you wanted me to make the proposed change or someone else. I fiddled with the text a bit as you suggested putting a mention at top and bottom of the section.