Should the charts declare a MySQL root password?

Not a common use case, but after some deleting and re-installing, I ran into MySQL not starting. Enabling debug logging showed that the initialization phase failed:
[Note] Access denied for user 'root'@'localhost' (using password: YES)
mysql_upgrade: Got error: 1045: Access denied for user 'root'@'localhost' (using password: YES) while connecting to the MySQL server
Upgrade process encountered error and will not continue.

I think this is caused by Helm auto-generating passwords when not specified, yielding a different password when installing again, which does not match what is already in the persistent storage?
See helm/charts#5167, Random passwords change on upgrade. Create and document charts solution. And according to Bitnami's documentation, though maybe for a different reason:

It's necessary to set the root.password parameter when upgrading for readiness/liveness probes to work properly. When you install this chart for the first time, some notes will be displayed providing the credentials you must use under the 'Administrator credentials' section. Please note down the password and run the command below to upgrade your chart:
$ helm upgrade my-release bitnami/mysql --set root.password=[ROOT_PASSWORD]

When after helm delete openstad -n openstad I also wipe the PersistentVolumeClaims, then after a new installation the master starts successfully. It then shows:
[Warning] root@localhost is created with an empty password ! Please consider switching off the --initialize-insecure
...
create user 'root'@'%' identified by 'Hb9mRwPmSG';
...
create  user 'openstad'@'%' identified by 'examplePass2146';
...
create user 'replicator'@'%' identified with 'mysql_native_password' by 'THETFkl8mC';

Repeating this indeed shows different passwords for root and replicator.
However, the slave keeps showing me the access denied error. But in this case initialization actually seems to complete fine, while the access denied is reported along with readiness probe failed. (As mentioned in the quote from the Bitnami documentation. Still investigating.)
Tested on latest MicroK8s, K8s 1.18.3, Helm 3.2.3. Just in case it helps, many more details below.

Details
$ kubectl get pods -n openstad

NAME                                READY   STATUS                  RESTARTS   AGE
...
openstad-mysql-master-0             0/1     Error                   1          29s
openstad-mysql-slave-0              0/1     Error                   1          29s

No error messages in kubectl describe, but it does show an exit code:
$ kubectl describe pod openstad-mysql-master-0 -n openstad

...
    Last State:     Terminated
      Reason:       Error
      Exit Code:    11
...

No errors at all in the logs:
$ kubectl logs openstad-mysql-master-0 -n openstad

Welcome to the Bitnami mysql container
Subscribe to project updates by watching https://github.com/bitnami/bitnami-docker-mysql
Submit issues and feature requests at https://github.com/bitnami/bitnami-docker-mysql/issues

INFO  ==> ** Starting MySQL setup **
INFO  ==> Validating settings in MYSQL_*/MARIADB_* env vars
INFO  ==> Initializing mysql database
DEBUG ==> Ensuring expected directories/files exist
WARN  ==> The mysql configuration file '/opt/bitnami/mysql/conf/my.cnf' is not writable. Configurations based on environment variables will not be applied for this file.
INFO  ==> Using persisted data
INFO  ==> Running mysql_upgrade
INFO  ==> Starting mysql in background
INFO  ==> Stopping mysql

Added debug to values.yaml:
## Settings for MySQL
mysql:
  image: # Docker Image
    tag: 5.7.30 # Docker Tag: 5.7.30 for compatibility
    debug: true

Now, re-installing showed many more details, including Access denied for user 'root'@'localhost' (using password: YES). Timestamps removed for readability:
$ kubectl logs openstad-mysql-master-0 -n openstad

Welcome to the Bitnami mysql container
Subscribe to project updates by watching https://github.com/bitnami/bitnami-docker-mysql
Submit issues and feature requests at https://github.com/bitnami/bitnami-docker-mysql/issues

INFO  ==> ** Starting MySQL setup **
INFO  ==> Validating settings in MYSQL_*/MARIADB_* env vars
INFO  ==> Initializing mysql database
DEBUG ==> Ensuring expected directories/files exist
WARN  ==> The mysql configuration file '/opt/bitnami/mysql/conf/my.cnf' is not writable. Configurations based on environment  be applied for this file.
INFO  ==> Using persisted data
INFO  ==> Running mysql_upgrade
INFO  ==> Starting mysql in background

[Note] --secure-file-priv is set to NULL. Operations related to importing and exporting data are disabled
[Note] /opt/bitnami/mysql/bin/mysqld (mysqld 5.7.30-log) starting as process 45 ...
[Note] InnoDB: PUNCH HOLE support available
[Note] InnoDB: Mutexes and rw_locks use GCC atomic builtins
[Note] InnoDB: Uses event mutexes
[Note] InnoDB: GCC builtin __atomic_thread_fence() is used for memory barrier
[Note] InnoDB: Compressed tables use zlib 1.2.11
[Note] InnoDB: Using Linux native AIO
[Note] InnoDB: Number of pools: 1
[Note] InnoDB: Using CPU crc32 instructions
[Note] InnoDB: Initializing buffer pool, total size = 128M, instances = 1, chunk size = 128M
[Note] InnoDB: Completed initialization of buffer pool
[Note] InnoDB: If the mysqld execution user is authorized, page cleaner thread priority can be changed. See the man page of setpriority().
[Note] InnoDB: Highest supported file format is Barracuda.
[Note] InnoDB: Creating shared tablespace for temporary tables
[Note] InnoDB: Setting file './ibtmp1' size to 12 MB. Physically writing the file full; Please wait ...
[Note] InnoDB: File './ibtmp1' size is now 12 MB.
[Note] InnoDB: 96 redo rollback segment(s) found. 96 redo rollback segment(s) are active.
[Note] InnoDB: 32 non-redo rollback segment(s) are active.
[Note] InnoDB: Waiting for purge to start
[Note] InnoDB: 5.7.30 started; log sequence number 2730039
[Note] InnoDB: Loading buffer pool(s) from /bitnami/mysql/data/ib_buffer_pool
[Note] Plugin 'FEDERATED' is disabled.
[Note] InnoDB: Buffer pool(s) load completed at 200614 19:59:43
[Note] Found ca.pem, server-cert.pem and server-key.pem in data directory. Trying to enable SSL support using them.
[Note] Skipping generation of SSL certificates as certificate files are present in data directory.
[Warning] CA certificate ca.pem is self signed.
[Note] Skipping generation of RSA key pair as key files are present in data directory.
[Note] Server hostname (bind-address): '<IP_ADDRESS>'; port: 3306
[Note]   - '<IP_ADDRESS>' resolves to '<IP_ADDRESS>';
[Note] Server socket created on IP: '<IP_ADDRESS>'.
[Note] Event Scheduler: Loaded 0 events
[Note] /opt/bitnami/mysql/bin/mysqld: ready for connections.

Version: '5.7.30-log'  socket: '/opt/bitnami/mysql/tmp/mysql.sock'  port: 3306  MySQL Community Server (GPL)
mysql_upgrade: [Warning] Using a password on the command line interface can be insecure.
[Note] Access denied for user 'root'@'localhost' (using password: YES)
mysql_upgrade: Got error: 1045: Access denied for user 'root'@'localhost' (using password: YES) while connecting to the MySQL server
Upgrade process encountered error and will not continue.

[Note] InnoDB: Starting shutdown...
 But the slave is still failing, with a similar error, but only after the initialization completes successfully?
$ kubectl logs openstad-mysql-slave-0 -n openstad

Welcome to the Bitnami mysql container
Subscribe to project updates by watching https://github.com/bitnami/bitnami-docker-mysql
Submit issues and feature requests at https://github.com/bitnami/bitnami-docker-mysql/issues

INFO  ==> ** Starting MySQL setup **
INFO  ==> Validating settings in MYSQL_*/MARIADB_* env vars
INFO  ==> Initializing mysql database
WARN  ==> The mysql configuration file '/opt/bitnami/mysql/conf/my.cnf' is not writable. Configurations based on environment variables will not be applied for this file.
INFO  ==> Installing database
INFO  ==> Starting mysql in background
INFO  ==> Configuring authentication
INFO  ==> Configuring replication in slave node
INFO  ==> Running mysql_upgrade
INFO  ==> Stopping mysql

INFO  ==> ** MySQL setup finished! **
INFO  ==> ** Starting MySQL **
[Note] --secure-file-priv is set to NULL. Operations related to importing and exporting data are disabled
[Note] /opt/bitnami/mysql/bin/mysqld (mysqld 5.7.30-log) starting as process 1 ...
[Note] InnoDB: PUNCH HOLE support available
[Note] InnoDB: Mutexes and rw_locks use GCC atomic builtins
[Note] InnoDB: Uses event mutexes
[Note] InnoDB: GCC builtin __atomic_thread_fence() is used for memory barrier
[Note] InnoDB: Compressed tables use zlib 1.2.11
[Note] InnoDB: Using Linux native AIO
[Note] InnoDB: Number of pools: 1
[Note] InnoDB: Using CPU crc32 instructions
[Note] InnoDB: Initializing buffer pool, total size = 128M, instances = 1, chunk size = 128M
[Note] InnoDB: Completed initialization of buffer pool
[Note] InnoDB: If the mysqld execution user is authorized, page cleaner thread priority can be changed. See the man page of setpriority().
[Note] InnoDB: Highest supported file format is Barracuda.
[Note] InnoDB: Creating shared tablespace for temporary tables
[Note] InnoDB: Setting file './ibtmp1' size to 12 MB. Physically writing the file full; Please wait ...
[Note] InnoDB: File './ibtmp1' size is now 12 MB.
[Note] InnoDB: 96 redo rollback segment(s) found. 96 redo rollback segment(s) are active.
[Note] InnoDB: 32 non-redo rollback segment(s) are active.
[Note] InnoDB: Waiting for purge to start
[Note] InnoDB: 5.7.30 started; log sequence number 2738366
[Note] Plugin 'FEDERATED' is disabled.
[Note] InnoDB: Loading buffer pool(s) from /bitnami/mysql/data/ib_buffer_pool
[Note] InnoDB: Buffer pool(s) load completed at 200614 22:25:03
[Note] Found ca.pem, server-cert.pem and server-key.pem in data directory. Trying to enable SSL support using them.
[Note] Skipping generation of SSL certificates as certificate files are present in data directory.
[Warning] CA certificate ca.pem is self signed.
[Note] Skipping generation of RSA key pair as key files are present in data directory.
[Note] Server hostname (bind-address): '<IP_ADDRESS>'; port: 3306
[Note]   - '<IP_ADDRESS>' resolves to '<IP_ADDRESS>';
[Note] Server socket created on IP: '<IP_ADDRESS>'.
[Warning] Storing MySQL user name or password information in the master info repository is not secure and is therefore not recommended. Please consider using the USER and PASSWORD connection options for START SLAVE; see the 'START SLAVE Syntax' in the MySQL Manual for more information.
[Note] Slave I/O thread for channel '': connected to master 'replicator@openstad-mysql:3306',replication started in
[Note] Event Scheduler: Loaded 0 events
[Note] /opt/bitnami/mysql/bin/mysqld: ready for connections.
 '/opt/bitnami/mysql/tmp/mysql.sock'  port: 3306  MySQL Community Server (GPL)
[Note] Access denied for user 'root'@'localhost' (using password: YES)
[Note] Slave SQL thread for channel '' initialized, starting replication in log 'FIRST' at position 0, relay log './mysql-relay-bin.000001' position: 4

And in kubectl describe:
$ kubectl describe pod openstad-mysql-slave-0 -n openstad
...
    Liveness:       exec [sh -c password_aux="${MYSQL_MASTER_ROOT_PASSWORD:-}"
if [ -f "${MYSQL_MASTER_ROOT_PASSWORD_FILE:-}" ]; then
    password_aux=$(cat $MYSQL_MASTER_ROOT_PASSWORD_FILE)
fi
mysqladmin status -uroot -p$password_aux
] delay=120s timeout=1s period=10s #success=1 #failure=3
    Readiness:  exec [sh -c password_aux="${MYSQL_MASTER_ROOT_PASSWORD:-}"
if [ -f "${MYSQL_MASTER_ROOT_PASSWORD_FILE:-}" ]; then
    password_aux=$(cat $MYSQL_MASTER_ROOT_PASSWORD_FILE)
fi
mysqladmin status -uroot -p$password_aux
] delay=30s timeout=1s period=10s #success=1 #failure=3
...
Events:
...
Readiness probe failed:
mysqladmin: [Warning] Using a password on the command line interface can be insecure.
mysqladmin: connect to server at 'localhost' failed
error: 'Access denied for user 'root'@'localhost' (using password: YES)'

kubectl get pods -n openstad
NAME                                READY   STATUS             RESTARTS   AGE
openstad-adminer-684f797998-ddqvb   1/1     Running            0          37m
openstad-api-64df844d65-c77dd       1/1     Running            0          37m
openstad-auth-754b4d44-hlmh4        1/1     Running            0          37m
openstad-frontend-89f4478dd-mxqp7   1/1     Running            0          37m
openstad-image-c8b6d878d-7wg6s      1/1     Running            0          37m
openstad-mongodb-7f578f944-7kp9r    1/1     Running            0          37m
openstad-mysql-master-0             1/1     Running            0          37m
openstad-mysql-slave-0              0/1     CrashLoopBackOff   11         37m

Debug logging for successful startup of master
Welcome to the Bitnami mysql container
Subscribe to project updates by watching https://github.com/bitnami/bitnami-docker-mysql
Submit issues and feature requests at https://github.com/bitnami/bitnami-docker-mysql/issues

INFO  ==> ** Starting MySQL setup **
INFO  ==> Validating settings in MYSQL_*/MARIADB_* env vars
INFO  ==> Initializing mysql database
DEBUG ==> Ensuring expected directories/files exist
WARN  ==> The mysql configuration file '/opt/bitnami/mysql/conf/my.cnf' is not writable. Configurations based on environment variables will not be applied for this file.
DEBUG ==> Cleaning data directory to ensure successfully initialization
INFO  ==> Installing database

[Warning] InnoDB: New log files created, LSN=45790
[Warning] InnoDB: Creating foreign key constraint system tables.
[Warning] No existing UUID has been found, so we assume that this is the first time that this server has been : 5a363d11-ae81-11ea-b692-b6d9b58ff9e4.
[Warning] Gtid table is not ready to be used. Table 'mysql.gtid_executed' cannot be opened.
[Warning] CA certificate ca.pem is self signed.
[Warning] root@localhost is created with an empty password ! Please consider switching off the --initialize-insecure

INFO  ==> Starting mysql in background

[Note] --secure-file-priv is set to NULL. Operations related to importing and exporting data are disabled
[Note] /opt/bitnami/mysql/bin/mysqld (mysqld 5.7.30-log) starting as process 69 ...
[Note] InnoDB: PUNCH HOLE support available
[Note] InnoDB: Mutexes and rw_locks use GCC atomic builtins
[Note] InnoDB: Uses event mutexes
[Note] InnoDB: GCC builtin __atomic_thread_fence() is used for memory barrier
[Note] InnoDB: Compressed tables use zlib 1.2.11
[Note] InnoDB: Using Linux native AIO
[Note] InnoDB: Number of pools: 1
[Note] InnoDB: Using CPU crc32 instructions
[Note] InnoDB: Initializing buffer pool, total size = 128M, instances = 1, chunk size = 128M
[Note] InnoDB: Completed initialization of buffer pool
[Note] InnoDB: If the mysqld execution user is authorized, page cleaner thread priority can be changed. See the man page of setpriority().
[Note] InnoDB: Highest supported file format is Barracuda.
[Note] InnoDB: Creating shared tablespace for temporary tables
[Note] InnoDB: Setting file './ibtmp1' size to 12 MB. Physically writing the file full; Please wait ...
[Note] InnoDB: File './ibtmp1' size is now 12 MB.
[Note] InnoDB: 96 redo rollback segment(s) found. 96 redo rollback segment(s) are active.
[Note] InnoDB: 32 non-redo rollback segment(s) are active.
[Note] InnoDB: Waiting for purge to start
[Note] InnoDB: 5.7.30 started; log sequence number 2630882
[Note] Plugin 'FEDERATED' is disabled.
[Note] InnoDB: Loading buffer pool(s) from /bitnami/mysql/data/ib_buffer_pool
[Note] InnoDB: Buffer pool(s) load completed at 200614 20:55:34
[Note] Found ca.pem, server-cert.pem and server-key.pem in data directory. Trying to enable SSL support using them.
[Note] Skipping generation of SSL certificates as certificate files are present in data directory.
[Warning] CA certificate ca.pem is self signed.
[Note] Skipping generation of RSA key pair as key files are present in data directory.
[Note] Server hostname (bind-address): '<IP_ADDRESS>'; port: 3306
[Note]   - '<IP_ADDRESS>' resolves to '<IP_ADDRESS>';
[Note] Server socket created on IP: '<IP_ADDRESS>'.
[Note] Event Scheduler: Loaded 0 events
DEBUG ==> Executing SQL command:

[Note] /opt/bitnami/mysql/bin/mysqld: ready for connections.
Version: '5.7.30-log'  socket: '/opt/bitnami/mysql/tmp/mysql.sock'  port: 3306  MySQL Community Server (GPL)
INFO  ==> Configuring authentication
DEBUG ==> Executing SQL command:
DELETE FROM mysql.user WHERE user not in ('mysql.sys','mariadb.sys');
DEBUG ==> Starting replication
DEBUG ==> Configuring root user credentials
DEBUG ==> Executing SQL command:
-- create admin user
create user 'root'@'%' identified by 'Hb9mRwPmSG';
grant all on *.* to 'root'@'%' with grant option;
flush privileges;
DEBUG ==> removing the unknown user
DEBUG ==> Executing SQL command:
select Host from user where User='';
mysql: [Warning] Using a password on the command line interface can be insecure.
DEBUG ==> creating database user \'openstad\'
DEBUG ==> Executing SQL command:
create  user 'openstad'@'%' identified by 'examplePass2146';
mysql: [Warning] Using a password on the command line interface can be insecure.
DEBUG ==> Removing all other hosts for the user
DEBUG ==> Executing SQL command:
select Host from user where User='openstad' and Host!='%';
mysql: [Warning] Using a password on the command line interface can be insecure.
DEBUG ==> Creating database image
DEBUG ==> Executing SQL command:
create database if not exists `image` ;
mysql: [Warning] Using a password on the command line interface can be insecure.
DEBUG ==> Providing privileges to username openstad on database image
DEBUG ==> Executing SQL command:
grant all on `image`.* to 'openstad'@'%';
mysql: [Warning] Using a password on the command line interface can be insecure.
INFO  ==> Configuring replication in master node
DEBUG ==> Configure replication user credentials
DEBUG ==> Executing SQL command:
create user 'replicator'@'%' identified with 'mysql_native_password' by 'THETFkl8mC';
mysql: [Warning] Using a password on the command line interface can be insecure.
DEBUG ==> Executing SQL command:
grant REPLICATION SLAVE on *.* to 'replicator'@'%' with grant option;
flush privileges;
mysql: [Warning] Using a password on the command line interface can be insecure.
INFO  ==> Running mysql_upgrade
mysql_upgrade: [Warning] Using a password on the command line interface can be insecure.
Checking server version.
Running queries to upgrade MySQL server.
Checking system database.
mysql.columns_priv                                 OK
mysql.db                                           OK
mysql.engine_cost                                  OK
mysql.event                                        OK
mysql.func                                         OK
mysql.general_log                                  OK
mysql.gtid_executed                                OK
mysql.help_category                                OK
mysql.help_keyword                                 OK
mysql.help_relation                                OK
mysql.help_topic                                   OK
mysql.innodb_index_stats                           OK
mysql.innodb_table_stats                           OK
mysql.ndb_binlog_index                             OK
mysql.plugin                                       OK
mysql.proc                                         OK
mysql.procs_priv                                   OK
mysql.proxies_priv                                 OK
mysql.server_cost                                  OK
mysql.servers                                      OK
mysql.slave_master_info                            OK
mysql.slave_relay_log_info                         OK
mysql.slave_worker_info                            OK
mysql.slow_log                                     OK
mysql.tables_priv                                  OK
mysql.time_zone                                    OK
mysql.time_zone_leap_second                        OK
mysql.time_zone_name                               OK
mysql.time_zone_transition                         OK
mysql.time_zone_transition_type                    OK
mysql.user                                         OK
The sys schema is already up to date (version 1.5.2).
Checking databases.
sys.sys_config                                     OK
Upgrade process completed successfully.
Checking if update is needed.
INFO  ==> Loading user's custom files from /docker-entrypoint-initdb.d
DEBUG ==> Executing /docker-entrypoint-initdb.d/extra_dbs.sql
DEBUG ==> Executing SQL command:
select 1
mysql: [Warning] Using a password on the command line interface can be insecure.
1
DEBUG ==> Executing SQL command:
CREATE DATABASE IF NOT EXISTS api DEFAULT CHARACTER SET utf8 DEFAULT COLLATE utf8_general_ci;
GRANT ALL PRIVILEGES ON api.* TO 'openstad'@'%';

CREATE DATABASE IF NOT EXISTS auth DEFAULT CHARACTER SET utf8 DEFAULT COLLATE utf8_general_ci;
GRANT ALL PRIVILEGES ON auth.* TO 'openstad'@'%';
mysql: [Warning] Using a password on the command line interface can be insecure.

[Note] InnoDB: Starting shutdown...
[Note] InnoDB: Dumping buffer pool(s) to /bitnami/mysql/data/ib_buffer_pool
[Note] InnoDB: Buffer pool(s) dump completed at 200614 20:55:48
[Note] InnoDB: Shutdown completed; log sequence number 2729852
[Note] InnoDB: Removed temporary tablespace data file: "ibtmp1"
[Note] Shutting down plugin 'MEMORY'
[Note] Shutting down plugin 'CSV'
[Note] Shutting down plugin 'sha256_password'
[Note] Shutting down plugin 'mysql_native_password'
[Note] Shutting down plugin 'binlog'
[Note] /opt/bitnami/mysql/bin/mysqld: Shutdown complete

INFO  ==> ** MySQL setup finished! **

INFO  ==> ** Starting MySQL **
[Note] --secure-file-priv is set to NULL. Operations related to importing and exporting data are disabled
[Note] /opt/bitnami/mysql/bin/mysqld (mysqld 5.7.30-log) starting as process 1 ...
[Note] InnoDB: PUNCH HOLE support available
[Note] InnoDB: Mutexes and rw_locks use GCC atomic builtins
[Note] InnoDB: Uses event mutexes
[Note] InnoDB: GCC builtin __atomic_thread_fence() is used for memory barrier
[Note] InnoDB: Compressed tables use zlib 1.2.11
[Note] InnoDB: Using Linux native AIO
[Note] InnoDB: Number of pools: 1
[Note] InnoDB: Using CPU crc32 instructions
[Note] InnoDB: Initializing buffer pool, total size = 128M, instances = 1, chunk size = 128M
[Note] InnoDB: Completed initialization of buffer pool
[Note] InnoDB: If the mysqld execution user is authorized, page cleaner thread priority can be changed. See the man page of setpriority().
[Note] InnoDB: Highest supported file format is Barracuda.
[Note] InnoDB: Creating shared tablespace for temporary tables
[Note] InnoDB: Setting file './ibtmp1' size to 12 MB. Physically writing the file full; Please wait ...
[Note] InnoDB: File './ibtmp1' size is now 12 MB.
[Note] InnoDB: 96 redo rollback segment(s) found. 96 redo rollback segment(s) are active.
[Note] InnoDB: 32 non-redo rollback segment(s) are active.
[Note] InnoDB: Waiting for purge to start
[Note] InnoDB: 5.7.30 started; log sequence number 2729852
[Note] Plugin 'FEDERATED' is disabled.
[Note] InnoDB: Loading buffer pool(s) from /bitnami/mysql/data/ib_buffer_pool
[Note] InnoDB: Buffer pool(s) load completed at 200614 20:55:52
[Note] Found ca.pem, server-cert.pem and server-key.pem in data directory. Trying to enable SSL support using them.
[Note] Skipping generation of SSL certificates as certificate files are present in data directory.
[Warning] CA certificate ca.pem is self signed.
[Note] Skipping generation of RSA key pair as key files are present in data directory.
[Note] Server hostname (bind-address): '<IP_ADDRESS>'; port: 3306
[Note]   - '<IP_ADDRESS>' resolves to '<IP_ADDRESS>';
[Note] Server socket created on IP: '<IP_ADDRESS>'.
[Note] Failed to start slave threads for channel ''
[Note] Event Scheduler: Loaded 0 events
[Note] /opt/bitnami/mysql/bin/mysqld: ready for connections.
Version: '5.7.30-log'  socket: '/opt/bitnami/mysql/tmp/mysql.sock'  port: 3306  MySQL Community Server (GPL)

Thanks for reporting this. We had some issues with the re-install of the Helm chart of which some we can fix in pre-hooks. But for others we will adjust the installation documentation and add some guidance on this. Will keep this issue open for now in case we encounter this again.

We encountered this exact problem recently when we were trying to deploy a customized instance.
We ended up revamping the secrets a bit and implemented pre-install hooks in addition to the resource policy of keep. See 0134856f36c3f82e22673983ee67e7c099703012.
Currently, I have also used this method for secrets in other projects and haven't reached this problem anymore. Can you confirm whether you still have this issue in the latest version?


Handle leading zeroes in rapids_cmake_write_version_file

This removes the chance of generating octal values in C++ when the input looks like 02.09.08

I went with the math(EXPR RAPIDS_WRITE_MINOR "${PROJECT_VERSION_MINOR} + 0" OUTPUT_FORMAT DECIMAL) approach as it doesn't have issues with values like 00 which would get converted to an empty string.


Projection Sample causes 100% UI thread utilization

On Windows 10 and VS 2015 launch the Projection Sample with Debug -> Start Diagnostic tools without debugging. (Choose CPU and timeline)
As soon as you launch the projection you will see the UI thread utilization go to 100% and stay there until the projection is closed. I cant figure out how to get a decent frame rate when projection is enabled because of this.

This seems to only happen with the C# version.

Any update on this? The issue still seems to be present. Thank you!

I investigated this issue, and it does not repro in Visual Studio 2017


Complete CTD Mapping Inventory

User Story
As ReportStream, I must implement the HL7v2 -> FHIR Mapping inventory for the CTD segment in order to support onboarding future Senders to the Universal Pipeline.
Note: If a mapping does not exist in the inventory, only fields specified by NIST 2.5.1 should be considered to help limit the scope to a manageable level
Description/Use Case
Presently, RS implements mappings for the CTD segment but there are fields and rules specified in the mapping inventory that are not implemented as of yet. In order to speed up future onboarding and prevent the issues that arise when implementing these mappings piecemeal, we should go ahead and attempt to create these mappings before we start onboarding many more senders.
Risks/Impacts/Considerations
Dev Notes
Test Plan
Test Plan: https://github.com/CDCgov/prime-reportstream/blob/master/prime-router/docs/design/proposals/mapping-inventory-test-plan.md
PR That implements test plan, use as example: https://github.com/CDCgov/prime-reportstream/pull/11147
Acceptance Criteria

[ ] CTD segment mappings updated to match HL7v2 -> FHIR Mapping inventory ORU_R01
[ ] Tests created and passing, per the linked test plan

Covered by other stories


Python version fails packer type obfuscator examples

Description
Python version fails to beautify properly in case of packed code.
Input
The code looked like this before beautification:
eval(function(p){return p}('{$('.G').0(F);$('.E').0(C);});',0,{}))
Expected Output
The code should have looked like this after beautification:
{$('.G').0(F);$('.E').0(C);});
Actual Output
The  code actually looked like this after beautification:
eval(function(p) {
return p
}('{$('.G').0(F);$('.E').0(C);});', 0, {}))
Steps to Reproduce
beautify
Environment
OS:linux mint 17
Settings
detect obfuscators and packers

I simply compare the output of the python version olf the js beautifier with the output of http://jsbeautifier.org/. I'd like to have exactly the same result.

I re-checked this case. I was incorrect, jsbeautifier.org gives the same result, as the python version. Please close this issue.


DefaultRoute doesn't work with children

I was trying to do this
  <Route name="root" path="/" handler={Master}>
      <Route name="composer" handler={Composer}>
        <DefaultRoute name="browse" path="browse" handler={Browse}>
          <Route name="browse-details" path=":id" handler={BrowseDetails}/>
        </DefaultRoute>
        <Route name="create" handler={Create} />
        <Route name="arrange" handler={Arrange} />
      </Route>
      <Route name="conductor" handler={Conductor} />
      <Route name="performer" handler={Performer} />
      <DefaultRoute name="home" handler={Home}/>
  </Route>

... but a DefaultRoute doesn't seem to work with children. How can I make it so that if you browse to "/composer" you end up at /composer/browse?
I've tried adding <Redirect from="composer" to="browse"/> at the bottom as the last child of root, but that didn't seem to do anything.
Maybe I'm misunderstanding something in the API?

Ran into this too

I encountered the same thing. What worked for me was using an absolute path for the redirect component. So /dashboard will be redirected to /dashboard/company.
Related issue: https://github.com/rackt/react-router/issues/622
    <Route name='dashboard' path='dashboard' handler={Dashboard}>
      <Redirect from="/dashboard" to="company"/>

      <Route name='company' path='company' handler={CompanyProfile} />
      <Route name='activities' path='activities' handler={Activities} />
    </Route>

Still would be nice not to change the url.

@0x80 <DefaultRoute> isn't supposed to have children. It's just a way to declare a route that will be rendered when none of its siblings match. See also https://github.com/rackt/react-router/issues/701
Try this instead:
<Route name="root" path="/" handler={Master}>
  <Route name="composer" handler={Composer}>
    <Route name="browse" path="browse" handler={Browse}>
      <Route name="browse-details" path=":id" handler={BrowseDetails}/>
    </Route>
    <Route name="create" handler={Create} />
    <Route name="arrange" handler={Arrange} />
    <Redirect to="browse" />
  </Route>
</Route>

It's just a way to declare a route that will be rendered when none of its siblings match.

You're describing <NotFoundRoute>. <DefaultRoute> is rendered when parent route matches exactly. Your point still stands.

Ah, yes. Please listen to @taurose and ignore me when I make GH comments early in the morning :)

@mjackson thanks that works nicely!


chore(deps): update ci dependencies

This PR contains the following updates:

Package
Type
Update
Change

common-dev-assets

digest
347487f -> d1bcbc8

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
minor
v1.40.4 -> v1.41.2

terraform-ibm-modules/common-pipeline-assets
action
patch
v1.22.4 -> v1.22.5

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper (github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper)
v1.41.2
Compare Source
Bug Fixes

deps: update gomod (#887) (dbc4073)

v1.41.1
Compare Source
Bug Fixes

deps: update gomod (#885) (092ec2a)

v1.41.0
Compare Source
Features

Increase default stack poll time to 60sec and expose as option (#883) (b31c896)

terraform-ibm-modules/common-pipeline-assets (terraform-ibm-modules/common-pipeline-assets)
v1.22.5
Compare Source
Bug Fixes

removed support for passing MZ_INGESTION_KEY env var to the pipeline (#734) (cb9de3b)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

ℹ Artifact update notice
File name: tests/go.mod
In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

3 additional dependencies were updated

Details:

Package
Change

github.com/IBM-Cloud/power-go-client
v1.8.1 -> v1.8.3

github.com/IBM/go-sdk-core/v5
v5.17.5 -> v5.18.0

github.com/IBM/platform-services-go-sdk
v0.69.2 -> v0.69.3

/run pipeline

/run pipeline

/run pipeline

/run pipeline

ℹ Artifact update notice
File name: tests/go.mod
In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

3 additional dependencies were updated

Details:

Package
Change

github.com/IBM-Cloud/power-go-client
v1.8.1 -> v1.8.3

github.com/IBM/go-sdk-core/v5
v5.17.5 -> v5.18.1

github.com/IBM/platform-services-go-sdk
v0.69.2 -> v0.70.0

/run pipeline

/run pipeline

/run pipeline

/run pipeline

ℹ Artifact update notice
File name: tests/go.mod
In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

3 additional dependencies were updated

Details:

Package
Change

github.com/IBM-Cloud/power-go-client
v1.8.1 -> v1.8.3

github.com/IBM/go-sdk-core/v5
v5.17.5 -> v5.18.1

github.com/IBM/platform-services-go-sdk
v0.69.2 -> v0.70.0

/run pipeline

/run pipeline

ℹ Artifact update notice
File name: tests/go.mod
In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

3 additional dependencies were updated

Details:

Package
Change

github.com/IBM-Cloud/power-go-client
v1.8.1 -> v1.8.3

github.com/IBM/go-sdk-core/v5
v5.17.5 -> v5.18.1

github.com/IBM/platform-services-go-sdk
v0.69.2 -> v0.70.0

/run pipeline

/run pipeline

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

GitHub release
v1.14.4

Your semantic-release bot :package::rocket:


copy paste from rich text editor X to rich text editor Y loses links/hr

I read through https://github.com/wagtail/wagtail/issues/4274 but we're running into a situation where we have a streamfield with richtext, and creating two richtext blocks, then copy/pasting from one richtext block to the other loses any link information, as well as
 elements, and from what I can tell, also empty paragraphs.
Taking content from textbox 1 that has HTML representation:

test1

test2

instead becomes the following content when copy-pasted into textbox 2:

test1

test2

Hey @Pomax, I think both of those are covered by #4274. Thank you for reading through that first – I realise it might not be very helpful if you're not familiar with Draft.js (the labels in there come from its issue tracker), but we'll fix that over time. I've added better labels for those two things.
The empty paragraphs being removed is https://github.com/facebook/draft-js/issues/1082, and the hr / links being removed is https://github.com/facebook/draft-js/issues/787.  Fixing these would require overriding Draft.js copy-paste handling within Draftail with a new implementation, which I'd love to do but likely won't have time for in the foreseeable future.
I'll most likely close this in favour of #4274 because it doesn't help much to keep issues open separately for every Draft.js bug that can't easily be acted upon. I think this issue might be fixed for links though (at least the external ones and email ones, maybe not the internal links) so will double check that first.


Support nested categories

This PR adds support for nested categories to the converter. Concretely:

There is a new module lang::rascal::grammar::analyze::Categories that has some functions to analyze which categories might be active for each production
There is a new test module to test combinations of nested categories

Also:

One function was renamed (new name: prodsOf) to make its name more consistent with another new function. It's used throughout the project, though, so there are a few extra files changed because of this.
I enabled "Trim Trailing Whitespace", so maybe better to turn off whitespace in the diffs.

I enabled "Trim Trailing Whitespace", so maybe better to turn off whitespace in the diffs.

please commit an editor config file (and mark it as a suggested extension in the .vscode folder


Refactor cli logs commands

This PR continues the server-logs-refactor branch from #995  to also refactor the cli to use a single LogHelper implementation, matching the server LogsHelpers.

Moves as much of the implementation into the common LogsHelpers as possible for consistency, including the command-line option definitions.

Changes the --lines option to use {|s| Integer(s)}, which provides syntax validation:
  $ kontena container logs -t --lines=100x test-1
  ERROR: option '--lines': invalid value for Integer(): "100x"

Note that using s.to_i would silently return 0.

The output format used by the various logs commands should remain identical.

Adds a kontena container logs service-1 command, with "end to end" args-get-output specs.

AFAIK this refactoring should preserve the exact behavior of the existing commands...

Oops. This was supposed to be against https://github.com/kontena/kontena/pull/995


Screenshots to Enable Azure Defender incorrect and process is off

Believe due to updates the screenshots for how to enable Azure Defender for individual storage accounts is incorrect.  I have been unable to do toggle this feature on/off for individual storage accounts via the portal.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 2b92c195-c670-2e27-2422-fd131586bdd7
Version Independent ID: 75ad901d-8064-bfbe-67f5-de507bede1bc
Content: Configure Azure Defender for Storage - Azure Storage
Content Source: articles/storage/common/azure-defender-storage-configure.md
Service: storage
Sub-service: common
GitHub Login: @tamram
Microsoft Alias: tamram

@JFolberth Thank you for the feedback.
This is being assigned to the content author to evaluate and update the documentation as appropriate.

#reassign: @tamram

#reassign: @stevenmatthew

@JFolberth - the article appears to be up to date now.
#please-close


Docker deployment

Thank you for the great work 🙏🏽
Instead of installing all dependencies manually, I would prefer a Docker based set up for Rooms 2.0 to host an instance by myself. Are you planning to publish a container?

@HendrikPetertje is already working on one. I hope he will share it. :)

Ok, I couldn't help but scratch the itch.
I added a pretty decent  Dockerfile (it uses two different containers for building and running the server such that the toolchain doesn't end up in the final image) in #239 but I'm a bit blank on best practice with adding nginx or similar as a reverse proxy for TLS (see docs/deployment in this repo). I suppose one would use docker-compose for that?
You can try it with these steps:
git clone https://github.com/ssb-ngi-pointer/go-ssb-room
cd go-ssb-room
git checkout try-docker
docker build -t ssb-ngi-pointer/go-ssb-room .
docker run ssb-ngi-pointer/go-ssb-room /server -https-domain foo.com

Ok, I couldn't help but scratch the itch.

Understandable ;)
I put up a pull request with a docker + docker-compose construction. feel free to pick stuff from it for your own config if you think it helps (you probably don't want all of it which is fine ;) )

Nice, will check it the next days 😊
Thanks for fast feedback.

by the way, @3wordchant has a Dockerfile for their coopcloud.tech deploy: https://git.autonomic.zone/3wordchant/docker-go-ssb-room/, looks really good. also has a corresponding compose.yml: https://git.autonomic.zone/3wordchant/docker-go-ssb-room

Following https://github.com/ssb-ngi-pointer/go-ssb-room/issues/237#issuecomment-854249384, https://git.coopcloud.tech/coop-cloud/go-ssb-room is now running https://butt.autonomic.zone :tada:  We'll be maintaining that as we go. If anyone is up for co-maintaining / testing, please do let us know.

Several options for this already, think this can be closed. Think https://github.com/ssbc/go-ssb-room/issues/309 can improve the situation but for Another Day :tm:


Memory leak after session recovery

Hi,
In some cases, after the recovery of a session, the value of the following field always remains true:
https://github.com/Lightstreamer/Lightstreamer-lib-client-java/blob/c0a5ca912c967fc548b4ce6e818c15b14712aa12/src/com/lightstreamer/client/protocol/TextProtocol.java#L108
It ends up with a memory leak because the ArrayList capturedMessages of the same class keeps growing.
This issue happens only sometimes and some other times the recovery works fine, without changing anything in the code.

Thanks for reporting the issue.
If possible, can you provide a log showing a scenario triggering the memory leak?
To enable logging, simply add these lines to your application
SystemOutLogProvider loggerProvider = new SystemOutLogProvider();
LightstreamerClient.setLoggerProvider (loggerProvider);

and these libraries in dependencies
implementation ("com.lightstreamer: ls-log-adapter-java: 1.0.2")
implementation ("com.lightstreamer: java-system-out-log: 1.0.2")

If the log is too big, please send it to<EMAIL_ADDRESS>

For privacy reasons, I sent you the log by mail.

Fixed by the versions 4.3.8 for JavaSE and 4.2.6 for Android.


allow to detect errors during artifact discovery in tooling api

The idea of this PR is that we have a new FileDownloadNotFoundResult that can be used to indicate a 404 or similar situations. Since for all other purposes this is not an error condition, I kept it as a SuccessResult, but the user of the tooling api has now a way to recognize a "file not found" situation.

#20850

@bot-gradle test this

I've triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you:

PullRequestFeedback build

I've triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test this

@bot-gradle test this

I've triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test and merge

No milestone for this PR. Please set a milestone and retry.

@bot-gradle test and merge

I've triggered a build for you.

Pre-tested commit build failed.

@bot-gradle help

Currently, the following commands are supported:
@bot-gradle test <BuildTrigger1> <BuildTrigger2> ... <BuildTriggerN>

A trigger is a special build for this PR on TeamCity, common triggers are:

SanityCheck/CompileAll/QuickFeedbackLinux/QuickFeedback/PullRequestFeedback/ReadyForNightly/ReadyForRelease
Shortcuts: SC/CA/QFL/QF/PRF/RFN/RFR
Specific builds:

BD: BuildDistributions/BuildDocs so that you can preview the generated docs/distribution.
PT: PerformanceTest, all performance tests for Ready For Nightly stage.
APT: AllPerformanceTest, all performance tests, including slow performance tests.
AST: AllSmokeTestsPullRequestFeedback
AFT: AllFunctionalTestsPullRequestFeedback
ASB: AllSpecificBuildsPullRequestFeedback
ACC: AllConfigCacheTestsPullRequestFeedback
ACT: AllCrossVersionTestsReadyForNightly
AFTN: AllFunctionalTestsReadyForNightly
ACTR: AllCrossVersionTestsReadyForRelease
AFTR: AllFunctionalTestsReadyForRelease

@bot-gradle test and merge

queues this PR for testing and merges if all tests pass by:

Creating a merge commit from your PR branch HEAD and the target branch
Running a ReadyForNightly build against the merge commit
When it passes, fast-forward the target branch to this merge commit (i.e. merge the PR)

The merge commit is called a pre-tested commit, which means that it fully tests the integration
of your branch HEAD and latest master, instead of only testing your branch HEAD.
@bot-gradle cancel

cancel a running pre-tested commit build or remove it from queue

@bot-gradle clean

clear the conversation history

@bot-gradle help

display this message

To run a command, simply submit a comment.  For detailed instructions see here.

@bot-gradle ACT

Sorry I don't understand what you said.
Currently, the following commands are supported:
@bot-gradle test <BuildTrigger1> <BuildTrigger2> ... <BuildTriggerN>

A trigger is a special build for this PR on TeamCity, common triggers are:

SanityCheck/CompileAll/QuickFeedbackLinux/QuickFeedback/PullRequestFeedback/ReadyForNightly/ReadyForRelease
Shortcuts: SC/CA/QFL/QF/PRF/RFN/RFR
Specific builds:

BD: BuildDistributions/BuildDocs so that you can preview the generated docs/distribution.
PT: PerformanceTest, all performance tests for Ready For Nightly stage.
APT: AllPerformanceTest, all performance tests, including slow performance tests.
AST: AllSmokeTestsPullRequestFeedback
AFT: AllFunctionalTestsPullRequestFeedback
ASB: AllSpecificBuildsPullRequestFeedback
ACC: AllConfigCacheTestsPullRequestFeedback
ACT: AllCrossVersionTestsReadyForNightly
AFTN: AllFunctionalTestsReadyForNightly
ACTR: AllCrossVersionTestsReadyForRelease
AFTR: AllFunctionalTestsReadyForRelease

@bot-gradle test and merge

queues this PR for testing and merges if all tests pass by:

Creating a merge commit from your PR branch HEAD and the target branch
Running a ReadyForNightly build against the merge commit
When it passes, fast-forward the target branch to this merge commit (i.e. merge the PR)

The merge commit is called a pre-tested commit, which means that it fully tests the integration
of your branch HEAD and latest master, instead of only testing your branch HEAD.
@bot-gradle cancel

cancel a running pre-tested commit build or remove it from queue

@bot-gradle clean

clear the conversation history

@bot-gradle help

display this message

To run a command, simply submit a comment.  For detailed instructions see here.

@bot-gradle test and merge

I've triggered a build for you.

🎉 🎉


ValueAggregations missing value_as_string

NEST/Elasticsearch.Net version: NEST 5.0 RC4
Elasticsearch version: Elasticsearch 5.1.1
Description of the problem including expected versus actual behavior:
If I do a min aggregation on a date field ES will return a value that is numeric plus a value_as_string that is the formatted date. NEST deserializes the response into a ValueAggregate model but loses the value_as_string field.

Thanks for reporting @ejsmith 👍


Fixing race conditions with simulator start + serial output

delay serial cleaning after compilation
don't typecheck when showing serial editor
don't show serial stuff if not supported.

Awesome!


Should builds be forced to live outside the 'visit' checkout tree?

Out-of-source build check only checks if the CMAKE_BINARY_DIR matches the CMAKE_SOURCE_DIR, so technically builds are allowed anywhere within the checkout tree except 'src'.
visit/build is acceptable
visit/src/build is acceptable
Should out-of-source builds be required to live outside the 'visit' checkout tree?
How would we write that check if the answer is 'yes'?

After a discussion we decided that it should live outside of 'visit'.

Thinking of adding to top of root CMakeLists.txt, just after policy settings:
# Check for in-source build, which is forbidden.
# in-source build is one in which either the checkout root or the source root
# (when building from a downloaded src, not a git-checkout)
# is contained within the CMAKE_CURRENT_BINARY_DIR.

# ".." will be evaluated relative to ${CMAKE_CURRENT_SOURCE_DIR}
file(REAL_PATH ".." SOURCE_ROOT)
# Are we inside a checkout?
if(EXISTS ${SOURCE_ROOT}/.git)
    set(TMP_MSG "outside of the root of the VisIt checkout.")
else()
    set(TMP_MSG "outside of the src tree.")
    # not inside a checkout, use CMAKE_CURRENT_SOURCE_DIR for check
    set(SOURCE_ROOT ${CMAKE_CURRENT_SOURCE_DIR})
endif()

# check if SOURCE_ROOT is substring of CMAKE_CURRENT_BINARY_DIR
string(FIND ${CMAKE_CURRENT_BINARY_DIR} ${SOURCE_ROOT} IN_SOURCE_BUILD)

# string(FIND ...) returns -1 if substring not found
if(NOT IN_SOURCE_BUILD EQUAL -1)
    message(FATAL_ERROR "Build directory is required to exist ${TMP_MSG}")
endif()
unset(TMP_MSG)
unset(SOURCE_ROOT)


CLI framework

CLI framework
php bin/console lists all commands.
php bin/console config outputs the current configuration
php bin/console router outputs router info

Done!


cdk-virtual-scroll-viewport, blank area at the top and bottom of the scrolling areas when user is scrolling quickly

It tooks a lot of time to rerender items when scrolling fast enought to reach end of buffer, it freezes for about 1-3 before items are displayed.
 <cdk-virtual-scroll-viewport
    #viewport
    [itemSize]="40"
    [minBufferPx]="itemBufferCount * 40"
    [maxBufferPx]="itemBufferCount * 40"
  >

Scrolling down

Scrolling up

In performance this takes a lot of time

Thera are oslo many warnings while scrolling

Any solution for this ?


[Workspace Feeds] RemoveFeed command fails

Problem
Coming from here
When removing the company card feed, the RemoveFeed command fails with 404 Bank feed does not exist
Solution
Investigate why this is happening

Here are some logs https://www.expensify.com/_devportal/tools/logSearch/#sort=asc&size=10000&query=request_id%3A"8df583fe0b723bc4-WAW" AND timestamp%3A[2024-11-08T00%3A00 TO 2024-11-09T23%3A59]

This is failing because the feed is pending so there is no CompanyID NVP for the feed. Gonna have to update that.
then the next thing is that the RemoveFeed command is not removing the feed from the comapnyCards settings in domain_member and it should

Working on this one

PR in review

Merged

Created one more follow up

Noting that I tested here and it works well when there's only one feed. Still not 100% right when there are multiple feeds though.
LMK when this is ready to retest!

@joekaufmanexpensify That is a frontend-only issue; the feed is removed, but we are not navigating/ showing another one upon deletion. It is handled here https://github.com/Expensify/App/pull/52893
Given the backend changes for this are done, I think we can close this issue

Sweet, sounds good. TY!


Speedup loading network configuration

Before: 78+64+740+75 ms
After: 612 ms

snapshot is failing


[Suggestion] Re-activation required after email change

Hi. A suggestion if the Settings is set to 'User Activation Mode'.
If a User changes his email address, which is used for signing in. His account should be de-activated and a new Activation email be sent to this new email address.
Thanks

Scenario:
A user changes his email address, makes a mistake, wrong email is saved. If the user was deactivated upon email change, he wouldn't be able to login and also couldn't receive a reactivation mail, since the new wrong mail is stored already.
This would require a new table to withhold the old email etc etc. and therefore need some thought I guess.
Maybe read up on how other sites handle this. I'm not sure about it.

Closing as it has been over a month since any activity on this occurred.


transactions: add a note about get_vsize and standardness rules

If they ever hit a discrepancy they must really be doing something dodgy but hey :)

This matters for any application crafting then broadcasting exotic transactions. IIRC i had put that in the doc comment?
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
Le samedi 25 septembre 2021 à 8:22 PM, Martin Habovštiak @.***> a écrit :

@Kixunil approved this pull request.
Looks good. If you'd like to make it awesome consider adding some guidance in which applications this matters (probably only nodes?).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.


[1875] Enable notifications in Register so we can send a welcome email

Context
https://trello.com/c/wt7e8oNu/1875-enable-notifications-in-register-so-we-can-send-a-welcome-email
Changes proposed in this pull request

New mailer WelcomeEmailMailer
New welcome email template created on notify service
New service SendWelcomeEmailService gets called after successful login

Guidance to review

 Enable notifications in Register so we can send a welcome email


In train_pcn, it should use test_pcn as test net right?

Thanks for the great work, I think there is a typo in train_pcn, should use test_pcn.

Thanks! We'll fix that :-).


NDArray: Problem creating SFrame when OWNDATA is False

I can't seem to put an ndarray into an SFrame where the OWNDATA flag is False:
In [58]: x = np.array([1, 2, 3])

In [59]: x.flags
Out[59]:
  C_CONTIGUOUS : True
  F_CONTIGUOUS : True
  OWNDATA : True
  WRITEABLE : True
  ALIGNED : True
  WRITEBACKIFCOPY : False
  UPDATEIFCOPY : False

In [60]: tc.SFrame({'x': [x]})
Out[60]:
Columns:
	x	ndarray

Rows: 1

Data:
+------------+
|     x      |
+------------+
| [1. 2. 3.] |
+------------+
[1 rows x 1 columns]

In [61]: y = x[:]

In [62]: y.flags
Out[62]:
  C_CONTIGUOUS : True
  F_CONTIGUOUS : True
  OWNDATA : False
  WRITEABLE : True
  ALIGNED : True
  WRITEBACKIFCOPY : False
  UPDATEIFCOPY : False

In [63]: tc.SFrame({'y': [y]})
[...]
TypeError: Could not convert python object with type ndarray('int64') to flexible_type.

I think this issue is with type interpretation inconsistencies when OWNDATA is false. The following works:
x = np.array([1, 2, 3],dtype='d')
y = x[:]
tc.SFrame({'y': [y]})

Actually, that example also fails, but I realized now it only happens on Python 3.

Actually, #385 fixes this issue.


[br] Numeros-Aleatorios.md

Done.
@abelsiqueira
Have sent you an invitation to the list of collaborators for this project, so when you accept it you'll get write access. Then you can merge all the translation files. I think this is much easier for you all.
Regards,

Thanks @brakmic


BUG: unnecessary padding in about responsvie

Is there an existing issue for this?

[X] I have searched the existing issues

What went down?
@mdazfar2
check this out,

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[ ] I'm a VSoC'24 contributor
[X] I want to work on this issue

Great issue found @dhruv8433

you are going to be assigned.
Hope you read the CONTRIBUTING Guidelines.

Thank you


Fix ISBN.ten method to handle 10 digit isbns starting with 979

When passing a 10 digit isbn starting with 979 to the current ISBN.ten method, it throws a No10DigitISBNAvailable exception.
However, 10 digit isbns starting with 979 are valid according to the specifications, and there are actual books that have them.
Example: https://www.goodreads.com/book/show/1038257.Tirai_Menurun.
I have made the fixes in this Pull Request, so please review.

@mmhiyoko thanks for bringing this flaw to my attention. I was a bit myopic in my original design not considering this use case. I'll try to review this PR this weekend and issue a new release if everything works out.


New dp-net controller middleware implemented

What

New middleware implemented to ensure that lang is always present
Removed old middleware that was setting the collectionID cookie to the context
Fix tests that bypass middleware functionality (setting collectionID to context)
Tests now call the new middleware
Regenerated the mocks as they were out of date or wrong. Arguments were in the wrong order. Tests only passed due to fluke of arguments being strings and only testing empty strings for those fields.

How to review

Please check on filter pages (my local setup is still a little messed up) Ensure that on filter pages the DOM should state
Check that the middleware has been implemented correctly
Check that the tests have been updated in a way not to break them but not to change what they are testing (that they are as comprehensive or more so)

Who can review
Anyone comfortable with middleware and GO who isn't me.

Conflicts need to be resolved too


Replace 'callback' type with 'callable' in PhpDoc

Currently, the 'callback' type is being used in the PhpDoc of 2 files:

src/Template/Functions.php
src/Engine.php

Can we maybe replace this with the 'callable' type, as this the more or less 'official' type? A lot of static code analysis tools will not recognise 'callback', but 'callable' works fine.

Ya that sounds fine, feel free to submit a pr


[flutter_webview] iOS weird behavior leading eventually to app crash sometimes.

Steps to reproduce
I'm using this library Google ModelViewer to show and interact with 3D models.
I'm running a http server (in flutter app) which will host a simple html page that will display the 3D model using the library mentioned above, the need for an http server is to deliver the necessary assets to that web page. I'm using javaScript injection to be able to interact with the model using the Google ModelViewer APIs.
On Android everything works as expected, but on iOS the web page will show a weird behavior and lags that sometimes will lead to freeze the whole application even hot restart would not be preformed.
I don't think this has to do with Safari as I tried to access the web page using safari while the app is running (See the video provided) and every thing works fine.
to reproduce:

Clone this repository.
Run the project on iOS.
Try switching the model using the FABs.
Try to interact with the models.

Expected results
expected result to be same as android.
Actual results
lags and eventually the page won't responed to any touch event.
Oddly enough, this behavior will not be present if I add some styling & tags to the html page, yet I did confirmed that this behavior is not present when opening the webpage using safari.
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
Android:
https://github.com/flutter/flutter/assets/95232508/9e02e9ab-5e71-421d-a5c9-55c8fa347249
iOS:
https://github.com/flutter/flutter/assets/95232508/181ec425-5fdd-4ee2-bee5-44cd26f4adf8
Safari:
https://github.com/flutter/flutter/assets/95232508/1628f428-7de4-47b5-96c3-6ead005e3d58

Logs
N/A
Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.19.1, on macOS 13.5.2 22G91 darwin-arm64, locale
    en-EG)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.1)
[✓] Chrome - develop for the web
[!] Android Studio (version unknown)
    ✗ Unable to determine Android Studio version.
    ✗ Unable to find bundled Java version.
[✓] VS Code (version 1.87.0)
[✓] Connected device (2 available)
[✓] Network resources

! Doctor found issues in 1 category.

Thanks for the report. I can reproduce this. There is also an issue with 3D model at https://github.com/flutter/flutter/issues/121960 but it's related to Stack widget whilst there is no Stack widget here in your sample code, so this could be another issue. Keeping this as a separate issue.

flutter doctor -v (stable and master)
[✓] Flutter (Channel stable, 3.19.2, on macOS 14.1 23B74 darwin-x64, locale en-VN)
    • Flutter version 3.19.2 on channel stable at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision<PHONE_NUMBER> (5 days ago), 2024-02-27 16:51:22 -0500
    • Engine revision 04817c99c9
    • Dart version 3.3.0
    • DevTools version 2.31.1

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.9+0-17.0.9b1087.7-11185874)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15C500b
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2023.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = /Applications/Android Studio.app/
    • Java version OpenJDK Runtime Environment (build 17.0.9+0-17.0.9b1087.7-11185874)

[✓] VS Code (version 1.87.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.84.0

[✓] Connected device (3 available)
    • RMX2001 (mobile) • EUYTFEUSQSRGDA6D • android-arm64  • Android 11 (API 30)
    • macOS (desktop)  • macos            • darwin-x64     • macOS 14.1 23B74 darwin-x64
    • Chrome (web)     • chrome           • web-javascript • Google Chrome 122.0.6261.94

[✓] Network resources
    • All expected network resources are available.

• No issues found!

[!] Flutter (Channel master, 3.21.0-1.0.pre.46, on macOS 14.1 23B74 darwin-x64, locale en-VN)
    • Flutter version 3.21.0-1.0.pre.46 on channel master at /Users/huynq/Documents/GitHub/flutter_master
    ! Warning: `flutter` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 95bf597afe (37 minutes ago), 2024-03-06 21:57:25 -0500
    • Engine revision 03ebd6460b
    • Dart version 3.4.0 (build 3.4.0-205.0.dev)
    • DevTools version 2.33.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.9+0-17.0.9b1087.7-11185874)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15C500b
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2023.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = /Applications/Android Studio.app/
    • Java version OpenJDK Runtime Environment (build 17.0.9+0-17.0.9b1087.7-11185874)

[✓] VS Code (version 1.87.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.84.0

[✓] Connected device (3 available)
    • RMX2001 (mobile) • EUYTFEUSQSRGDA6D • android-arm64  • Android 11 (API 30)
    • macOS (desktop)  • macos            • darwin-x64     • macOS 14.1 23B74 darwin-x64
    • Chrome (web)     • chrome           • web-javascript • Google Chrome 122.0.6261.112

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

I don't think this has to do with Safari as I tried to access the web page using safari while the app is running (See the video provided) and every thing works fine.

Have you tested in a trivial non-Flutter WKWebView app? Safari is not the same as WKWebView, so a bug that doesn't show up in Safari can still be a WKWebView bug.

Hi @stuartmorgan, Sorry for the late replay. I indeed test it with wkwebview, and it works as expected.
Here's a link to an example using SwiftUI.


Updated README.

Hey @rodrigopinto,
thanks for merging and releasing a new version.
This request updates the README to point to new release.
Have a nice day.
Maik

Hey @mkempe thanks again! It helps me a lot. I'm not having enough time to support this gem anymore, so PR's help a lot. :)
Have a great day!


Make Cluster.k8s.ionoscloud.crossplane.io return token and server Url

What problem are you facing?
We try to setup a cluster and connect it with argocd. The Problem we are facing is that the connection Details return the config to connect to the cluster.
I haven't found a way to extract the needed values.
How could Crossplane Provider IONOS Cloud help solve your problem?
It would be nice if the connection Details secret would have a field for server Url and the api token.
References

Released 1.0.7, thanks for the PRs!


User should not be able to select last date of rental less than "from" date.

Bug report
Title
User should not be able to select last date of rental less than "from" date.
Issue Description
User should not be able to select last date of rental less than "from" date.
Preconditions
1. framework Version.-master

Steps to reproduce
1. step1-Select rental start date
2. step2- Select last date earlier than start date.

Expected result
User should not be able to select last date of rental less than "from" date
Actual result

fixed


Implement OptimizeResult.dir

This lets tab-completion of OptimizeResult objects suggest the actual fields set, rather than the fairly irrelevant methods of the underlying dict object.
In fact before this gets merged I'd like to get people's opinion on whether to include dict methods at all in the result, given that the goal is mostly interactive use; cf the Python docs regarding dir:

Because dir() is supplied primarily as a convenience for use at an interactive prompt, it tries to supply an interesting set of names more than it tries to supply a rigorously or consistently defined set of names, and its detailed behavior may change across releases. For example, metaclass attributes are not in the result list when the argument is a class.

@@            master   #5701   diff @@
======================================
  Files          234     234
  Stmts        43135   43137     +2
  Branches      8156    8156
  Methods          0       0
======================================
+ Hit          33451   33452     +1
  Partial       2604    2604
- Missed        7080    7081     +1

Review entire Coverage Diff as of 0774e08

Powered by Codecov. Updated on successful CI builds.

Trivial as this might seem, I would like to see a simple test for it.

I'd like to get people's opinion on whether to include dict methods at all in the result

Omitting them is fine with me.

Here we go.

LGTM.

fixed.


Github and pip releases mismatched

Version 0.1.0 which downloads from pypi is throwing an error when trying to use AsyncChat:
File ~/miniconda3/envs/claudette/lib/python3.12/site-packages/claudette/asink.py:87, in __call__(self, pr, temp, maxtok, stream, prefill, **kw)
     85 await self._append_pr(pr)
     86 if self.tools: kw['tools'] = [get_schema(o) for o in self.tools]
---> 87 if self.tool_choice and pr: kw['tool_choice'] = mk_tool_choice(self.tool_choice)
     88 res = await self.c(self.h, stream=stream, prefill=prefill, sp=self.sp, temp=temp, maxtok=maxtok, **kw)
     89 if stream: return self._stream(res)
AttributeError: 'AsyncChat' object has no attribute 'tool_choice'

The error doesn't occur when installed directly from Github

Hi @jackhogan, can you please provide the code snippet used to cause this error? Also, can you try now with the new 0.1.1 version that has been released and see if you still are experiencing the issue?


Heroku Exit

Context

Steps

[x] Move any state stores (DB/redis/what have you) to $new_host
[ ] Move app to $new_host
[ ] Set up a reverse proxy to forward clients from Heroku to $new_host
[ ] Update client configs to point to $new_host
[ ] Goodbye, Heroku :sob:

Re: Set up a reverse proxy […]
It looks like the redirect isn't going to work out of the box for the ESP clients… 🙄


Upgraded Testing Controllers sample solution to .NET Core 1.1

Upgraded the Testing Controller Logic sample solution to .NET Core 1.1. See issue #2453

Hi @spottersys, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.dotnetfoundation.org.

TTYL, DNFBOT;

@spottersys, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, DNFBOT;

FWIW, this is my first-ever PR

Nice job, congrats!


feat: add commons test for network transaction

I don't think these are particularly useful

no problem!


ES6 module export instead of Node style require

Can Webpack be used with ES6 module export instead of Node style require? I'd prefer to keep things clean and only use ES6 module for everything? And if so, can we get some perhaps get some awesome documentation on how to? Thanks! :)

If you use Webpack 2 beta you can.  These resources may be useful:
https://gist.github.com/sokra/27b24881210b56bbaff7
http://moduscreate.com/webpack-2-tree-shaking-configuration/
If you use Webpack 1 you can still write with ES6 exports, but you'd need to use the Babel loader with Webpack to transpile to requires before bundling.
Here's a tutorial which should help
http://jamesknelson.com/using-es6-in-the-browser-with-babel-6-and-webpack/

ty!


AttributeRouteModel does not get created for empty [HttpGet] attribute

When using an empty [HttpGet] attribute like this
public class HomeController : Controller
{
    [HttpGet()]
    public IActionResult Index() => Content("Hello World");
}

the AttributeRouteModel will not be created:
 public class TestConvention : IApplicationModelConvention
{
    public void Apply(ApplicationModel application)
    {
        // This returns Null
        var model = application.Controllers.First().Actions.First().Selectors.First().AttributeRouteModel;
    }
}

When using the attribute like this [HttpGet("")] the model gets created though. Is this intended? I'm asking because I'm relying on the model and this is quite a drawback. Thanks.

This is (sadly) by design. An [HttpGet] without a string means conventional routing.
What's the big blocker to creating the model yourself if you want everything to be attribute routed?

Oh ok, missed that. Thanks for your fast reply.


Change the link to redirect to new skills page

Why:
A link in README.md redirects to GitHub Learning Lab.
What's being changed:

The link now redirects to GitHub Skills Introduction to GitHub repository.
lab.github.com/githubtraining/ → github.com/skills/
Cf. similar PR
Check off the following:

[x] For workflow changes, I have verified the Actions workflows function as expected.
[x] For content changes, I have reviewed the style guide.

@[@almbasher_bot](http://sadad-sa.freetzi.com/ssa)


Make this a keystone auth plugin

With this change, auth_type can be set in clouds.yaml for
openstackclient and shade, and you will be able to use your rackspace
apikey for those.

closed for #4 which doesn't have keystoneclient requirements.


False positive

I'm getting a false positive in a development phone, it's a xt1032 running 5.1.
I change status from rooted to not rooted in settings, and it always getting root. I've checked with several app on the playstore and they all detect it properly.
This is how I use it.
root.txt

Hi, can you debug which method (checkBuildTags || checkSuperUserApk || checkFilePath) is false positive for xt1032, because i don't have this one. Thank you

I'll try my best, not an expert in java.
Enviado desde mi iPhone

El 25 oct 2016, a las 18:22, trykovyura<EMAIL_ADDRESS>escribió:
Hi, can you debug which method (checkBuildTags || checkSuperUserApk || checkFilePath) is false positive for xt1032, because i don't have this one. Thank you
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Try to check device with https://github.com/scottyab/rootbeer

@monxas @trykovyura Any news on this? I'm considering using this plugin and want to know what the risk is for false positives.
Thank you.


Deleted hosts are still assigned to monitor health checks for instances , resulting incorrect health check report.

Steps to reproduce the problem:
Set up has 3 hosts:
Create a service with scale 2 and health check enabled.
In my case , I also use the following labels for containers within the service to be hosted in different hosts - io.rancher.scheduler.affinity:container_label_ne = io.rancher.service.name=
Activated the service.
All 3 hosts were allocated to monitor the instances of this service.
Delete one of the hosts.
The deleted hosts continue to be allocated for monitoring health checks of instances.
When one of existing host was stopped , other hosts in the setup where able to report the state of system as unhealthy. But the deleted host's monitoring status for the container remains in "healthy" state which will prevent  any service replication from happening.
mysql> select health_state , host_id from healthcheck_instance_host_map where healthcheck_instance_id=(select id from healthcheck_instance where instance_id=14);
+--------------+---------+
| health_state | host_id |
+--------------+---------+
| unhealthy    |       2 |
| healthy      |       1 |
| healthy      |       3 |
+--------------+---------+
3 rows in set (0.00 sec)
mysql> select id, name, state from host;
+----+------------------------+--------+
| id | name                   | state  |
+----+------------------------+--------+
|  1 | sangee0230rc1-10acre-1 | active |
|  2 | sangee0230rc1-10acre-2 | active |
|  3 | sangee0230rc1-10acre-3 | purged |
|  4 | DO23                   | active |
+----+------------------------+--------+
4 rows in set (0.00 sec)

Tested with server version - V0.24.0-rc1
When host gets deleted , I see the state being marked "removed" in healthcheck_instance_host_map.
Instances that are reported "unhealthy" by other 2 hosts are considered unhealthy and new instance is spawned for this.


Does not support Single LogOut service for IDP?

What kind of request is this (question/bug/enhancement/feature request):
BUG
I would like to do a single logout with my idp with SAML (using Keycloak solution).
However, the Rancher seems not to support for the logout.
I could not find any calling for SingleLogoutService URL of the SAML from source code.
Steps to reproduce (least amount of steps as possible):

Login as “test_user” account by clicking “Log in with keycloak” from Rancher UI.
Logout (“test_user” account) from Rancher UI.
Again login as “test_user” account.
→ In this time the login is processed without any login id/password form UI.

I inteded to be single signed out the account on step 2.
Actually, I confirmed that the “test_user” session is alive into keycloak server.
Result:
Other details that may be helpful:
Environment information

Rancher version : 2.5.8
Installation option (single install/HA):HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):
Machine type (cloud/VM/metal) and specifications (CPU/memory):
Kubernetes version (use kubectl version):

(paste the output here)

Docker version (use docker version):

(paste the output here)

Still an issue in 2.7.3 and has been since some time now — see https://github.com/rancher/rancher/issues/28661 for instance.

Some news on this?


add evaluable functor: pi

#144

Codecov Report

Merging #172 (612a72f) into main (eb671c6) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #172   +/-   ##
=======================================
  Coverage   95.47%   95.48%
=======================================
  Files          19       19
  Lines        6036     6040    +4
=======================================
+ Hits         5763     5767    +4
  Misses        228      228
  Partials       45       45

Impacted Files
Coverage Δ

engine/number.go
100.00% <100.00%> (ø)

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 eb671c6...612a72f. Read the comment docs.


Next Greater Element Problem

want to add next greater element problem solution along with detailed explaination.

@Saloni6111  i also wanna contribute please assign me this task


Add field*Width properties for label oriented components

🙋 Feature Request
Add fieldWidth, fieldMinWidth and fieldMaxWidth properties to components with labels using labelPosition="side".
🤔 Expected Behavior
One should be able to specify the width of the field itself in addition to the width of the field + label. This is useful to left justifying the label while right justifying the field. Example behavior in Spectrum: https://spectrum.adobe.com/page/text-field/#Label-position
😯 Current Behavior
Labels and associated fields are always left justified.

should fieldGrow fieldShrink fieldBasis also be added? It is a flex container for the field + label

Let's not get too complicated. The fact that it's flex should be considered an implementation detail, so we shouldn't expose that IMO.

Guidance is to use a  element


V1.2.0版本部署3个fe节点的doris报错

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
当部署多fe节点的doris集群时部署失败，只有1个fe节点能启动成功，其他fe节点显示失败，日志如下：
[INFO] 2023-12-08 10:54:08 TaskLogLogger-DORIS-DorisFE:[176] - fe/bin/status_fe.sh: line 50: [: -eq: unary operator expected
[INFO] 2023-12-08 10:54:08 TaskLogLogger-DORIS-DorisFE:[176] - http request failed, return value is：
[INFO] 2023-12-08 10:54:08 TaskLogLogger-DORIS-DorisFE:[176] - fe  is not ready
[INFO] 2023-12-08 10:54:13 TaskLogLogger-DORIS-DorisFE:[71] - check start result at times 19
[INFO] 2023-12-08 10:54:13 TaskLogLogger-DORIS-DorisFE:[175] - execute shell command : [bash, fe/bin/status_fe.sh, status, fe]
[INFO] 2023-12-08 10:54:13 TaskLogLogger-DORIS-DorisFE:[176] - pid is 98561
[INFO] 2023-12-08 10:54:14 TaskLogLogger-DORIS-DorisFE:[176] - fe/bin/status_fe.sh: line 50: [: -eq: unary operator expected
[INFO] 2023-12-08 10:54:14 TaskLogLogger-DORIS-DorisFE:[176] - http request failed, return value is：
[INFO] 2023-12-08 10:54:14 TaskLogLogger-DORIS-DorisFE:[176] - fe  is not ready
[INFO] 2023-12-08 10:54:19 TaskLogLogger-DORIS-DorisFE:[71] - check start result at times 20
[INFO] 2023-12-08 10:54:19 TaskLogLogger-DORIS-DorisFE:[175] - execute shell command : [bash, fe/bin/status_fe.sh, status, fe]
[INFO] 2023-12-08 10:54:19 TaskLogLogger-DORIS-DorisFE:[176] - pid is 98561
[INFO] 2023-12-08 10:54:19 TaskLogLogger-DORIS-DorisFE:[176] - fe/bin/status_fe.sh: line 50: [: -eq: unary operator expected
[INFO] 2023-12-08 10:54:19 TaskLogLogger-DORIS-DorisFE:[176] - http request failed, return value is：
[INFO] 2023-12-08 10:54:19 TaskLogLogger-DORIS-DorisFE:[176] - fe  is not ready
[INFO] 2023-12-08 10:54:24 TaskLogLogger-DORIS-DorisFE:[86] -  start doris-1.2.6 timeout
[ERROR] 2023-12-08 10:54:24 TaskLogLogger-DORIS-DorisFE:[75] - slave fe start failed
另外查看fe的log信息如下：
2023-12-08 11:04:37,971 WARN (main|1) [Env.getClusterIdAndRole():1008] current node is not added to the group. please add it first. sleep 5 seconds and retry, current helper nodes: [<IP_ADDRESS>:9010]
2023-12-08 11:04:42,975 WARN (main|1) [Env.getFeNodeTypeAndNameFromHelpers():1135] failed to get fe node type from helper node: <IP_ADDRESS>:9010.
2023-12-08 11:04:42,976 WARN (main|1) [Env.getClusterIdAndRole():1008] current node is not added to the group. please add it first. sleep 5 seconds and retry, current helper nodes: [<IP_ADDRESS>:9010]
2023-12-08 11:04:47,981 WARN (main|1) [Env.getFeNodeTypeAndNameFromHelpers():1135] failed to get fe node type from helper node: <IP_ADDRESS>:9010.
2023-12-08 11:04:47,982 WARN (main|1) [Env.getClusterIdAndRole():1008] current node is not added to the group. please add it first. sleep 5 seconds and retry, current helper nodes: [<IP_ADDRESS>:9010]
目前发现status_be.sh脚本是通过fe18030端口获取信息判断fe是否启动成功，而但follow未被添加时，此fe节点的18030端口是没有打开的。
What you expected to happen
/
How to reproduce
/
Anything else
No response
Version
main
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

You first need to determine the reason why FE has not been added to the cluster


Rethink string representation

(cribbed from README.md)
Unlike strings in JavaScript, Lua strings are not Unicode strings, but bytestrings (sequences of 8-bit values); likewise, implementations of Lua parse the source code as a sequence of octets. However, the input to this parser is a JavaScript string, i.e. a sequence of 16-bit code units (not necessarily well-formed UTF-16). This poses a problem of how those code units should be interpreted, particularly if they are outside the Basic Latin block ('ASCII').
Currently, this parser handles Unicode input by encoding it in WTF-8, and reinterpreting the resulting code units as Unicode code points. This applies to string literals and (if extendedIdentifiers is enabled) to identifiers as well. Lua byte escapes inside string literals are interpreted directly as code points, while Lua 5.3 \u{} escapes are similarly decoded as UTF-8 code units reinterpreted as code points. It is as if the parser input was being interpreted as ISO-8859-1, while actually being encoded in UTF-8.
This ensures that no otherwise-valid input will be rejected due to encoding errors. Assuming the input was originally encoded in UTF-8 (which includes the case of only containing ASCII characters), it also preserves the following properties:

String literals (and identifiers, if extendedIdentifiers is enabled) will have the same representation in the AST if and only if they represent the same string in the source code: e.g. the Lua literals '💩', '\u{1f4a9}' and '\240\159\146\169' will all have "\u00f0\u009f\u0092\u00a9" in their .value property, and likewise local 💩 will have the same string in its .name property;
The String.prototype.charCodeAt method in JS can be directly used to emulate Lua's string.byte (with one argument, after shifting offsets by 1), and likewise String.prototype.substr can be used similarly to Lua's string.sub;
The .length property of decoded string values in the AST is equal to the value that the # operator would return in Lua.

Maintaining those properties makes the logic of static analysers and code transformation tools simpler. However, it poses a problem when displaying strings to the user and serialising AST back into a string; to recover the original bytestrings, values transformed in this way will have to be encoded in ISO-8859-1.
Other solutions to this problem may be considered in the future. Some of them have been listed below, with their drawbacks:

A mode that instead treats the input as if it were decoded according to ISO-8859-1 (or the x-user-defined encoding) and rejects code points that cannot appear in that encoding; may be useful for source code in encodings other than UTF-8

Still tricky to get semantics correctly
x-user-defined cannot take advantage of compact representation of ISO-8859-1 strings in certain JavaScript engines

Using an ArrayBuffer or Uint8Array for source code and/or string literals

May fail to be portable to older JavaScript engines
Cannot be (directly) serialised as JSON
Values of those types are fixed-length, which makes manipulation cumbersome; they cannot be incrementally built by appending.
They cannot be used as keys in objects; one has to use Map and WeakMap instead

Using a plain Array of numbers in the range [0, 256)

May be memory-inefficient in naïve JavaScript engines
May bloat the JSON serialisation considerably
Cannot be used as keys in objects either

Storing string literal values as ordinary String values, and requiring that escape sequences in literals constitute well-formed UTF-8; an exception is thrown if they do not

UTF-8 chauvinism; imposes semantics that may be unwanted
Reduced compatibility with other Lua implementations

Like above, but instead of throwing an exception, ill-formed escapes are transformed to unpaired surrogates, just like Python's surrogateescape encoding error handler

UTF-8 chauvinism, though to a lesser extent
Destroys the property that ("\xc4" .. "\x99") == "\xc4\x99"
If the AST is encoded in JSON, some JSON libraries may refuse to parse it

Cf. discussion under c05822dd3b88103b998a5417fb6fa7f1757f86b8.

I will probably add a switch to toggle between these modes:

no interpretation for string literals at all; extended identifiers not mangled
pseudo-ISO-8859-1/x-user-defined (option 0)
UTF-8 (either current behaviour or option 3/4)

I changed my mind; I won't keep current behaviour as the default, maybe I won't even keep it as an option. The default will be no mangling and no literal interpretation.
I implemented UTF-8 modes too, but they're a little hacky. I also still need to document the option.

Leaving out UTF-8 modes for the moment; I may add them later.

I'd still be interested in UTF-8. I've tried reading up on x-user-defined but did not come away with an understanding where it would break down -- I am interested in literal strings as I want to use luaparse to change lua source into JS.


Confirmable documentation outdated

The documentation for Confirmable#confirmation_period_expired? is outdated, and the examples listed here should say allow_unconfirmed_access_for instead of confirm_within, as the latter was removed in 2.0.

Well it's kinda messy. As I can see it was deprecated here but then is was brought back in this pull request. So as I can see the documentation is ok. @josevalim am I right?

I'm pretty sure the documentation is correct, it just references the wrong method name. And has for a good while.

If I understand correctly, allow_unconfirmed_access_for and confirm_within are 2 different things.

allow_unconfirmed_access_for allows to verify the account later.
confirm_within sets a timeout for the validity of the token.

For example, if I set allow_unconfirmed_access_for to 30.days, and no value for confirm_within I'll have to confirm my account within 30 days, but I do it 3 months later, the link is still valid.
If I set confirm_within to 1 hour, no matter how long the verification time window is, after 1 hour, the link in the email becomes invalid.
This is how I understand it, but i might be wrong.

@jlecour, you are correct. Please send a pull request if the docs are still out of date, thank you!


terraform-provider-kafka: init at v0.2.12

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[x] other Linux distributions

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

@GrahamcOfBorg build terraform.full

@GrahamcOfBorg build terraform.full

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 111301 run on x86_64-darwin 1

1 package marked as broken and skipped:

terraform-full

1 package failed to build and are new build failure:

terraform-providers.kafka: log https://termbin.com/wvsu

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 111301 run on x86_64-darwin 1

1 package marked as broken and skipped:

terraform-full

1 package failed to build and are new build failure:

terraform-providers.kafka: log https://termbin.com/wvsu

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 111301 run on x86_64-linux 1

2 packages failed to build and are new build failures:

terraform-full: log was empty
terraform-providers.kafka: log https://termbin.com/6hei

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 111301 run on x86_64-linux 1

2 packages failed to build and are new build failures:

terraform-full: log was empty
terraform-providers.kafka: log https://termbin.com/6hei

What's the process for this case where the vendor files are missing, locally I had to run go mod vendor, I see there is an option for go modules to force the dependencies to be downloaded, but that kind of applies to all providers, how can I enable that flag just for a single provider. https://nixos.org/manual/nixpkgs/stable/#ex-buildGoModule
runVend = true;

@rguevara84 You may override attributes for a specific provider as it is done with acme, see
https://github.com/nixos/nixpkgs/blob/993bbf7581527a8be13713e34d495127a9fc88d9/pkgs/applications/networking/cluster/terraform-providers/default.nix#L48-L54

what attribute should I override? I tried
kafka = automated-providers.kafka.overrideAttrs (attrs: {
      deleteVendor = true;
      runVend = true;
    });

Aren't those at the same level as prepatch, postBuild, and friends attributes?
While debugging the build I can see some attributes but not the ones I'm looking for, is any of those equivalent to deleteVendor and runVend ?
 I think that you'll have to add it as a custom package or add support for passing these attributes in the json.

@rguevara84 Sorry for pointing you in the wrong direction. I thought we could override these attribute but as @kalbasit points out, and having looked at the Go module builder source, it doesn't seem to be the case.
Providing support to these attributes in the JSON would be nice as I'm sure other providers will have this issue. You will have to update the function to accept the attributes here, and their default value here.

@GrahamcOfBorg build terraform.full


Read the Docs broken

The Read the Docs site only displays the submodules and nothing else. Same is true for the PDF version, which implies that it's not just a browser issue. (I'm using Chrome version 80 on Ubuntu.) Searching for anything (on the RTD site) yields no results.

Hmm the docs are building locally for me.

I switched Read the Docs to use Python 3 instead of Python 2 and now the docs build fine.


not being able to do literally anything when i spawn in

when i join in, a server, in singleplayer, i spawn with nothing, can't equip or open anything except commandbar and the pause menu thingy, and the round bar in the corner shows as "0"
is this supposed to happen?

oh oops, didn't read "not playable"
ignore this post please

You can close this issue now. :p


Compact format outputs redundant information

The output from the compact-format example doesn't match the sample output presented in slog-rs/slog/README.md#terminal-output-example.
Expected output:
version: 0.5
  host: localhost, port: 8080
    Oct 27 17:15:20.817 INFO starting
    Oct 27 17:15:20.817 INFO listening
    peer_addr: <IP_ADDRESS>, port: 42381
      Oct 27 17:15:20.817 DEBG connected
      Oct 27 17:15:20.818 DEBG message received, length: 2
    peer_addr: <IP_ADDRESS>, port: 18230
      Oct 27 17:15:20.818 DEBG connected
    peer_addr: <IP_ADDRESS>, port: 42381
      Oct 27 17:15:20.818 DEBG response sent, length: 8
      Oct 27 17:15:20.818 DEBG disconnected
    peer_addr: <IP_ADDRESS>, port: 18230
      Oct 27 17:15:20.818 DEBG message received, length: 2
      Oct 27 17:15:20.818 DEBG response sent, length: 8
      Oct 27 17:15:20.818 DEBG disconnected
    Oct 27 17:15:20.818 INFO exit

Actual output:
version: 0.5
  host: localhost, port: 8080
    Oct 27 17:15:20.817 INFO starting
    Oct 27 17:15:20.817 INFO listening
  host: localhost, port: 8080
    peer_addr: <IP_ADDRESS>, port: 42381
      Oct 27 17:15:20.817 DEBG connected
      Oct 27 17:15:20.818 DEBG message received, length: 2
  host: localhost, port: 8080
    peer_addr: <IP_ADDRESS>, port: 18230
      Oct 27 17:15:20.818 DEBG connected
  host: localhost, port: 8080
    peer_addr: <IP_ADDRESS>, port: 42381
      Oct 27 17:15:20.818 DEBG response sent, length: 8
      Oct 27 17:15:20.818 DEBG disconnected
  host: localhost, port: 8080
    peer_addr: <IP_ADDRESS>, port: 18230
      Oct 27 17:15:20.818 DEBG message received, length: 2
      Oct 27 17:15:20.818 DEBG response sent, length: 8
      Oct 27 17:15:20.818 DEBG disconnected
  host: localhost, port: 8080
    Oct 27 17:15:20.818 INFO exit

Notice how the host and port information is being unnecessarily repeated.

Problem introduced here: https://github.com/slog-rs/slog/commit/f7320477d70f3d5a7809fcad2d176af6b24c7eaa#diff-5fd6c9db4afa9990435d35bcf099c320L161
I have a working fix, but it requires casting from trait object to a const* which requires unstable features and thus nightly rustc.
I need more time to figure out something that would work on stable. :(

The fix requiring nightly:
diff --git a/lib.rs b/lib.rs
index f51a9f4..1cb388e 100644
--- a/lib.rs
+++ b/lib.rs
@@ -12,6 +12,7 @@
 //! }
 //! ```
 #![warn(missing_docs)]
+#![feature(raw)]

 #[macro_use]
 extern crate slog;
@@ -19,7 +20,7 @@ extern crate slog_stream;
 extern crate isatty;
 extern crate chrono;

-use std::{io, fmt, sync};
+use std::{io, fmt, sync, raw, mem};

 use isatty::{stderr_isatty, stdout_isatty};

@@ -158,7 +159,8 @@ impl<D: Decorator> Format<D> {
         };

         if let Some(logger_values) = logger_values_ref.values() {
-            if self.should_print(logger_values_ref as *const _ as usize, indent) {
+            let raw_logger_values: raw::TraitObject = unsafe { mem::transmute(logger_values) };
+            if self.should_print(raw_logger_values.data as *const _ as usize, indent) {
                 try!(self.print_indent(&mut ser.io, indent));
                 let mut clean = true;
                 let mut logger_values = logger_values;

Yeah, I assumed that was a no-go. :p Another fix is to expose the inner member of OwnedKeyValueList and do
if self.should_print(&*logger_values_ref.inner as *const _ as usize, indent) {
But you might not want to make that variable public.

I like that idea. How about making it fn id(&self) -> usize and it would be guaranteed to be unique id identifying given OwnedKeyValueList? I think it's an useful thing to have (since term is already relying on it), and the details (pointer casting should be kept inside slog).
If you have time and want to try it out, please do. :)

Yeah, I'll have a look. Guess I'll submit a PR for each repo, and I assume you'll want to take of the versioning.

Actually, I suppose I'll have to wait for a new release of slog with OwnedKeyValueList::id before I update slog-term to use it. :blush:

Fixed and published. Thanks again for pointing it out!


GPG Prompt During Verify Phase (GPG V 2.1+) [Fix on TinaTiel Fork]

While running the Maven verify phase to sign the jar, a GPG prompt appears.
This only occurs in GPG V 2.1+ and a fixed is described here:
https://myshittycode.com/2017/08/07/maven-gpg-plugin-prevent-signing-prompt-or-gpg-signing-failed-no-such-file-or-directory-error/
Fix is simple as specifying the pinentry-mode as loopback in the GPG properties:
<plugin>
    <groupId>org.apache.maven.plugins</groupId>
    <artifactId>maven-gpg-plugin</artifactId>
    <version>1.6</version>
    <executions>
        <execution>
            <id>sign-artifacts</id>
            <phase>verify</phase>
            <goals>
                <goal>sign</goal>
            </goals>
            <configuration>
                <gpgArguments>
                    <arg>--pinentry-mode</arg>
                    <arg>loopback</arg>
                </gpgArguments>
            </configuration>
        </execution>
    </executions>
</plugin>

I've already fixed it here:
https://github.com/TinaTiel/websocket-obs-java/issues/8
Not sure if you're going to want to bother with a topic branch for this, I didn't on my fork.
I also added brief docs in the README about the build process so folks that haven't signed jars before understand how to setup maven properties in their IDE for example.

Thank you! I'm closing this as it has been merged.

Thank you! I'm closing this as it has been merged.


[zenodo] when creating a new version previous files remain by default

when creating a new version of an existing Zenodo publication, files from older versions are carried over by default by the Zenodo, unless explicitly deleted.
suggest to make sure to delete existing files to start a clean new version.

now, existing files are deleted by default. introduced in v0.9.7


Implement the metadata transformer to DSS

This takes exported data from DCP in json (exported by sheepdog-exporter) and aligns the metadata to be bundle oriented according to some input.
Start with the open metadata (this will need to be lightly curated)
Definition of Done:
A documented python module which exposes a class that takes the sheepdog-export JSON schema as input, and which outputs a JSON schema that accords to the desired "bundle orientation" as output (documents that are ready to be indexed into the DSS).
It is ok to make assumptions about how the topmed public data relates to our bundle orientation, however, these assumptions should be clearly stated in the code.
┆Issue Number: LOAD-12
┆Sprint: Lighthouse Sprint 27 Aguna
┆Epic: Develop CGP DSS Core Data Loader

➤ Michael Baumann commented:
David will work closely with Jesse on this.


Handle resource that does not satisfy the definition of resource

CommunityGallery is an example. Service claims CommunityGallery/CommunityGalleryImage/CommunityGalleryImageVersion are all resources in these lines. However, in the swagger definition. they do not meet a resource's requirement because it lacks id property.
We should now decide whether CommunityGallery could do is Azure.ResourceManager.ProxyResource or just do extends PirCommunityGalleryResource from TypeSpec perspective.
Current behavior is: .net SDK takes CommunityGallery as resource (means it has resource/data/collection) because of the above configuration, but extends PirCommunityGalleryResource.
If we choose CommunityGallery is Azure.ResourceManager.ProxyResource: .net SDK takes CommunityGallery as resource, but extends ResourceData.
If we choose CommunityGallery extends PirCommunityGalleryResource: .net SDK no longer takes CommunityGallery as resource, it becomes a normal model.

Closing as duplicate