Unable to setup CLP in Debian OS

Hi Team,
I am trying to follow the steps mentioned in package-template, when I am trying to start clp, it is failing with this error. The system has enough memory. Can you please edit the readme.md and post all the required softwares.
Traceback (most recent call last):
  File "/root/clp/tools/packager/out/clp-package-ubuntu-focal-x86_64-v0.0.1/etc/../sbin/start-clp", line 462, in main
    start_queue(instance_id, clp_config)
  File "/root/clp/tools/packager/out/clp-package-ubuntu-focal-x86_64-v0.0.1/etc/../sbin/start-clp", line 271, in start_queue
    subprocess.run(cmd, stdout=subprocess.DEVNULL, check=True)
  File "/opt/conda/default/lib/python3.10/subprocess.py", line 526, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['docker', 'exec', '-it', 'clp-queue-209e', 'rabbitmqctl', 'wait', '/tmp/rabbitmq.pid']' returned non-zero exit status 137.

Hi mantrimahitha,
Sorry, I haven't seen that issue before. From my own tests, the container in your error message uses a maximum of about 200MB during startup.
Could you give us the following details about your system to help us debug?

Amount of RAM
Docker version
OS version
clp commit hash that you're building from

Thanks!

@kirkrodrigues  I met similar issue when running cmd "python3 start-clp"
here are my server info.
RAM: 64GB
Docker version 24.0.7
Operating System: CentOS Linux 7 (Core)
CPE OS Name: cpe:/o:centos:centos:7
Kernel: Linux 3.10.0-1160.49.1.el7.x86_64
Architecture: x86-64
clp last commit 2b884bd ( I believe so as the package tool pulls newest code from github)
error log:
Traceback (most recent call last):
File "start-clp", line 462, in main
start_queue(instance_id, clp_config)
File "start-clp", line 271, in start_queue
subprocess.run(cmd, stdout=subprocess.DEVNULL, check=True)
File "/usr/lib64/python3.6/subprocess.py", line 438, in run
output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['docker', 'exec', '-it', 'clp-queue-938f', 'rabbitmqctl', 'wait', '/tmp/rabbitmq.pid']' returned non-zero exit status 137.
And I consider the compress is seperate with clp process(Hope I'm wrong).  so I run  "python3 compress xxl-service.log"
and got below error. I'm heading to install using single node. the experience is like a blackbox touching by me. If we have better doc to explain the components( I see it trys to run docker images to support it) and install flow, it would be great. As the log info is limited.
docker: Error response from daemon: invalid mount config for type "bind": bind source path does not exist: /vdb/clp-package-ubuntu-focal-x86_64-v0.0.2/var/data/archives.
See 'docker run --help'.
Traceback (most recent call last):
File "compress", line 168, in
sys.exit(main(sys.argv))
File "compress", line 159, in main
subprocess.run(cmd, check=True)
File "/usr/lib64/python3.6/subprocess.py", line 438, in run
output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['docker', 'run', '-i', '--rm', '--network', 'host', '-w', '/opt/clp', '-u', '0:0', '--name', 'clp-compressor-502b', '--mount', 'type=bind,src=/vdb/clp-package-ubuntu-focal-x86_64-v0.0.2,dst=/opt/clp', '--mount', 'type=bind,src=/,dst=/mnt/logs,readonly', '--mount', 'type=bind,src=/vdb/clp-package-ubuntu-focal-x86_64-v0.0.2/var/data/somehostname.company.com,dst=/opt/clp/var/data', '--mount', 'type=bind,src=/vdb/clp-package-ubuntu-focal-x86_64-v0.0.2/var/log/somehostname.company.com,dst=/opt/clp/var/log', '--mount', 'type=bind,src=/vdb/clp-package-ubuntu-focal-x86_64-v0.0.2/var/data/archives,dst=/mnt/archive-output', 'ghcr.io/y-scope/clp/clp-execution-x86-ubuntu-focal:main', '/opt/clp/sbin/native/compress', '--config', '/opt/clp/var/log/.clp-compressor-502b-config.yml', '--remove-path-prefix', '/mnt/logs', '/mnt/logs/vdb/clp-package-ubuntu-focal-x86_64-v0.0.2/sbin/xxl-service.log']' returned non-zero exit status 125.

by downloading from https://github.com/y-scope/clp/releases/download/v0.0.2/clp-package-ubuntu-focal-x86_64-v0.0.2.tar.gz  and install in same type of centos server
it's not working
[root@hostname clp-package-ubuntu-focal-x86_64-v0.0.2]# sbin/start-clp
2023-11-16 09:06:18,253 [INFO] [clp] Starting database...
Unable to find image 'mariadb:10.6.4-focal' locally
10.6.4-focal: Pulling from library/mariadb
...
3cb89a1550ea: Pull complete
df9f153bd930: Pull complete
Digest: sha256:c014ba1efc5dbd711d0520c7762d57807f35549de3414eb31e942a420c8a2ed2
Status: Downloaded newer image for mariadb:10.6.4-focal
docker: Error response from daemon: failed to update store for object type *libnetwork.endpointCnt: Key not found in store.
2023-11-16 09:06:27,432 [ERROR] [clp] Failed to start CLP.
Traceback (most recent call last):
File "sbin/start-clp", line 459, in main
start_db(instance_id, clp_config, conf_dir)
File "sbin/start-clp", line 164, in start_db
subprocess.run(cmd, stdout=subprocess.DEVNULL, check=True)
File "/usr/lib64/python3.6/subprocess.py", line 438, in run
output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['docker', 'run', '-d', '--rm', '--name', 'clp-db-cc7f', '-e', 'MYSQL_ROOT_PASSWORD=in5K8oZ3F6o', '-e', 'MYSQL_USER=clp-user', '-e', 'MYSQL_PASSWORD=in5K8oZ3F6o', '-e', 'MYSQL_DATABASE=clp-db', '-u', '0:0', '--mount', 'type=bind,src=/root/clp-package-ubuntu-focal-x86_64-v0.0.2/etc/mysql/conf.d,dst=/etc/mysql/conf.d,readonly', '--mount', 'type=bind,src=/root/clp-package-ubuntu-focal-x86_64-v0.0.2/var/data/hostname.company.com/db,dst=/var/lib/mysql', '--mount', 'type=bind,src=/root/clp-package-ubuntu-focal-x86_64-v0.0.2/var/log/hostname.company.com/db,dst=/var/log/mysql', '-p', '<IP_ADDRESS>:3306:3306', 'mariadb:10.6.4-focal']' returned non-zero exit status 125.

I even upgrade python3 to 3.7 and retry,  it fails also.
2023-11-16 09:44:10,263 [INFO] [clp] Starting database...
2023-11-16 09:44:11,904 [INFO] [clp] Started database.
2023-11-16 09:44:11,904 [INFO] [clp] Creating database tables...
2023-11-16 09:44:12,617 [INFO] [clp] Created database tables.
2023-11-16 09:44:12,617 [INFO] [clp] Starting queue...
2023-11-16 09:44:16,752 [INFO] [clp] Stopping clp-db-0862...
2023-11-16 09:44:17,159 [INFO] [clp] Stopped clp-db-0862.
2023-11-16 09:44:17,179 [ERROR] [clp] Failed to start CLP.
Traceback (most recent call last):
File "sbin/start-clp", line 462, in main
start_queue(instance_id, clp_config)
File "sbin/start-clp", line 271, in start_queue
subprocess.run(cmd, stdout=subprocess.DEVNULL, check=True)
File "/usr/local/lib/python3.7/subprocess.py", line 468, in run
output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['docker', 'exec', '-it', 'clp-queue-0862', 'rabbitmqctl', 'wait', '/tmp/rabbitmq.pid']' returned non-zero exit status 137.

After chatting offline with @bossjie, several issues were identified which prevented the successful startup of CLP package.

Docker bug in CentOS 7 which prevents the MariaDB SQL database and rabbitmq container image from being pulled and started from docker-hub

solution: upgrade to a recent docker release such as docker-ce-24.0.7-1

Permission issues with rabbitmq container's log file when started with the root user

temporary solution: use another user to start CLP package
permanent solution: a PR will be pushed out to resolve the underlying permission issue


Velocity Integration

Describe the solution you'd like
Velocity Integration with automatic loadbalancing, service discovery and persistency. Clone of MultiPaper Master + Proxy but instead running MultiPaper Master as a Velocity Plugin.
Automatically switch the configuration values once switching from MultiPaper Master standalone to Velocity.
Make completely configurable.
Describe alternatives you've considered
None
Additional context
None

Postponing until #13 is completely finished and ready for 1.18.2 and 1.19.2


Facebook\Authentication\AccessToken

At first login, it was successful but when i try to refresh the page or click the Logout button, this is what i got..
Fatal error: Facebook::authenticate(): The script tried to execute a method or access a property of an incomplete object. Please ensure that the class definition "Facebook\Authentication\AccessToken" of the object you are trying to operate on was loaded before unserialize() gets called or provide a __autoload() function to load the class definition in C:*******\libraries\Facebook.php on line 213
on line 213 :
if (!$access_token OR $access_token->isExpired())
i dont know what seems to be the problem here.
im using xampp localhost..

I had the same problem. You need to register Facebook autoloader before loading session library.

@pawelkg Can you explain in detail. Even i am facing the same problem

@surajde16 Just load FB autoloader before load library. I can't give you detailed instruction, because I used this library in heavily modified version of CI 1.5.4 mixed with CI 2.2.x

@pawelkg  @surajde16  i tried adding require_once 'Facebook/autoload.php'; before class declaration in facebook library but it didn't work. Can you please tell me where to call the autoload file?

@cjogloc @pawelkg @surajde16 @gheesixz
You don't need to include the Facebook autoload manually. Please see my response to issue #23

@darkwhispering  composer autoload is already set to TRUE in config.php file. and i am already autoloading the facebook library in autoload.php still am getting the error.

@cjogloc What version of the library are you using? And what version of CodeIgniter?

@darkwhispering i using codeigniter 3.0.6 and library version 3.0.0 beta 2
from here https://github.com/darkwhispering/facebook-sdk-v4-codeigniter/tree/fb-sdk-v5-develop

Hi bro, can you tell me please? I Logged in success but not access token.  Please help me bro!
$user = $this->facebook->request('get', '/me?fields=id,name,email');
var_dump($user); exit;
Result
array (size=2)
'error' => int 0
'message' => string 'You must provide an access token.' (length=33)

Please include before of controller class and working like below :
<?php
defined('BASEPATH') OR exit('No direct script access allowed');
include(APPPATH."libraries/Facebook/src/autoload.php");

class Logiciellab extends CI_Controller {
---
}

@darkwhispering  @parbatpithiya  still not working.
If it is working for you. Can anyone give me the zip with folder structure (including SDK)?

@cjogloc I have been unable to replicate the issue so far. I have however not had time to do any extensive testing so far and I also only the latest version of the FB SDK in the example controller I include in this library.

Hi, me too was facing  the same issue for the last two days. For me the problem was instead of doing " Install the Facebook PHP SDK by navigating to your applications folder in the terminal and run Composer with composer install. " i did run the composer in base directory. Please check if you have done the same mistake,

Hi @code-freakz
If you have the vendor folder outside of the application folder, then you are not using the CodeIgniter composer_autoload setting or you have configured a custom path for composer autoload in your config file.
The default location the composer_autoload looks for the composer autoload file is in application/vendor/autoload.php as stated in the config file
/*
|--------------------------------------------------------------------------
| Composer auto-loading
|--------------------------------------------------------------------------
|
| Enabling this setting will tell CodeIgniter to look for a Composer
| package auto-loader script in application/vendor/autoload.php.
|
|	$config['composer_autoload'] = TRUE;
|
| Or if you have your vendor/ directory located somewhere else, you
| can opt to set a specific path as well:
|
|	$config['composer_autoload'] = '/path/to/vendor/autoload.php';
|
| For more information about Composer, please visit http://getcomposer.org/
|
| Note: This will NOT disable or override the CodeIgniter-specific
|	autoloading (application/config/autoload.php)
*/
$config['composer_autoload'] = TRUE;

The library loads fine for me using the default composer autoload and have the composer vendor folder in the application folder. I'm still trying to find the actual problem as it seems different solutions works for different people.

Original issue posted by @gheesixz should be resolved in the last release, v3.0.0 now available in the master branch


Fix potential out-of-bounds memory access in cpp starter

Hey, I noticed I introduced a bug in PR #113 . Specifically, the pointer returned by return &citytiles.back(); can become invalid if you add a bunch more citytiles to a city, and vector has to resize the underlying memory. The fix is to lookup the memory addresses after the vector is done being modified.
Let me know if there's anything you'd like me to change.

To test this PR, I manually added some assertions. Is there a place for automated API tests, or are we sticking to manual testing for now?
Do you want me to re-zip the starter packages?

I can handle the rezipping. We usually run npm test to test, if someone wants to, we'd be very happy to have a circleci integration :)


Integrate with the Find & Apply candidates API

### Trello card
Trello-1641
Context
The Find & Apply team are providing a candidates API that exposes candidate information updated since a given date/time. We want to query this API for the GiT API and sync up the id of their candidates with those matched in the CRM.
Initially we will only be matching back on email address and updating existing candidates (creating new candidates in the CRM that exist in Find & Apply but don't match back will come later).
We want to run the sync operation hourly and only in non-production environments. We will only be concerned with records updated after the first sync run (past records will be bulk-imported by another process/job).
Changes proposed in this pull request

Store find/apply last sync date in Redis

We need to keep track of the date that the Find & Apply API was last synced successfully (as we pass this as a parameter in the subsequent API call).
Update AppSettings to enable storing the last sync date.

Add find/apply API credentials

In order to access the Find & Apply API we will need the URL for the instance (sandbox or production) and a valid API key.

Support match back on email address only

The normal match back process involves matching the email plus 2/3 of first name, last name and date of birth.
As the Find & Apply API is only going to be exposing the email address of the candidates initially we need to support matching back on just that field.
Replicates the current match back logic (taking into account status, duplicate score and date modified) and picks the first with the correct email.

Add FindApplyId to Candidate model

When we sync with the Find & Apply API we will be storing their candidate id against the corresponding contact in the CRM; this adds the field mapping for the id.

Model response from find/apply API

Add the models that map to the response from the Find & Apply candidates API.

Add job to sync a find/apply candidate with the CRM

Add a job that takes a Find & Apply candidate model and matches it against a contact in the CRM. If there is a match the FindApplyId is set and the model is persisted back to the CRM.

Add job to sync with find/apply API

The job requests Find & Apply candidates that have been updated since the last sync date. If no sync has been performed yet it will use the current date/time (we will pre-fill past records separately). A job is then queued for each candidate to sync it with the CRM.
Flurl is used for making the HTTP requests.

Run find/apply sync job on a schedule

We want to sync with the Find & Apply API periodically; initially this will be hourly and only in our development and staging environments (we don't want this enabled in production yet).
Guidance to review
Best reviewed by-commit.

Yeah I dithered a bit on the naming but I think ultimately its the core Candidate model that should be renamed to Contact to better reflect the CRM domain... I see what you mean about Response, I'll perhaps change that to Result or similar to avoid confusion, but I'll do that as part of the next PR.


Plugin stopped working with Anki Version 23.10

Your plugin is the best and I use it for different multimedia card decks. But now Anki updated their software to 23.10, and the plugin stopped working. I hope you can take a look at this issue.

Sorry I didn't use Anki for a long time, afraid I can't carry it on
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: petecodes100 @.>
Sent: Thursday, November 9, 2023 6:42:07 PM
To: hssm/media-import @.>
Cc: Subscribed @.***>
Subject: [hssm/media-import] Plugin stopped working with Anki Version 23.10 (Issue #19)
Your plugin is the best and I use it for different multimedia card decks. But now Anki updated their software to 23.10, and the plugin stopped working. I hope you can take a look at this issue.
—
Reply to this email directly, view it on GitHubhttps://github.com/hssm/media-import/issues/19, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOVRMIAUJNNJRMPAGJM34VDYDSXP7AVCNFSM6AAAAAA7EKYU7OVHI2DSMVQWIX3LMV43ASLTON2WKOZRHE4DKMZUGQ3DGOA.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

I have published a fixed version of Media Import here:
https://ankiweb.net/shared/info/129299120

Thanks a lot! Now we can finally use the plugin again!
Am Mi., 24. Jan. 2024 um 14:26 Uhr schrieb Iksas @.***>:

I have published a fixed version of Media Import here:
https://ankiweb.net/shared/info/129299120
—
Reply to this email directly, view it on GitHub
https://github.com/hssm/media-import/issues/19#issuecomment-1908120593,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYRYP7QNM3AXRUKKTRH5DI3YQEDZHAVCNFSM6AAAAAA7EKYU7OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBYGEZDANJZGM
.
You are receiving this because you authored the thread.Message ID:
@.***>


Starts python instead of IPython

This plugin has served me well, but after upgrade (apt upgrade) and
PluginUpdate it starts python instead of IPython, hence nothing works.
I have tree different installations:
Windows 10
    Nvim 0.3.1
    Python 3.7.1
    IPython 7.2.0

    IronRepl starts Python 3.7.1

Windows 10 (Ubuntu in WSL)
    Nvim 0.3.1
    Python 3.6.7
    IPython 7.1.1

    IronRepl OK

Ubuntu 18.04 LTS
    Nvim 0.3.1
    Python 3.6.7 and Python 2.7.15rc1 (installed by system)
    IPython 7.2.1

    IronRepl starts Python 2.7.15rc1

    On this machine IronrRepl has served me well on Ubuntu 14.x up to this
    verson, but after apt upgrade (Neovim in ppa),  IronRepl failed.

Any remedy?

Thank you for your swift response.
THIS WORKED!!!
I have become very addicted to your plugin, its a must have.


support GaLore

Can repo support GaLore soon ?
https://github.com/jiaweizzhao/GaLore

Hello @batman-do
It looks like GaLore is a drop-in replacement for the PyTorch optimizer, meaning that you can take any script that does pretraining/fine-tuning and replacement the part that defines optimizer.

Hello @batman-do
It looks like GaLore is a drop-in replacement for the PyTorch optimizer, meaning that you can take any script that does pretraining/fine-tuning and replacement the part that defines optimizer.

Can u support fsdp + qlora soon , i stuck oom when max_seq_length large when fine-tune lora > 24gb with model 7b

Can repo support GaLore soon ?

In the works via #1192 !


Make Password Strength Checker

Is my password truly strong?
A tool that checks the strength of a password and gives feedback based on criteria such as length, use of symbols, numbers, and capitalization.

python?

Please @sumanth-0 Assign this issue to me.

@shiv-0101 only in python and within 100 lines guys. Lets make some interesting projects

@sumanth-0  Make Password Strength Checker #37


Record language being used by users

https://developer.chrome.com/extensions/i18n
The chrome i18n methods provide the ability to detect which language Streamus is being rendered in, or attempting to be rendered in.
There are two methods, I am unsure of their differences:
chrome.i18n.getAcceptLanguages
chrome.i18n.getUILanguage
It would be nice to keep track of these values by storing it either on the User object or on a new object which has a FK to the User object. This way I can tell which i18n files need the most attention.
If the client-side logic can be done by someone else then I'll happily add the necessary work to the server to persist the data.

Hi Sean, I wrote you a email on last Friday with Polish locale file. I would be glad if you could add it into extension or if you have any comments to it please write me back ;)
Have you seen this?
Gets the browser UI language of the browser. This is different from i18n.getAcceptLanguages which returns the preferred user languages.

Hey,
I know, I've got your e-mail. I'm just cleaning up my locale's file today and removing unused stuff / simplifying sentences. I'm also putting together a Google Docs to keep track of all the different translators.
And yeah, I did see it, I just don't understand what the difference is between preferred user language & language of the browser.

Ok, good :) there were some duplicated lines ^^
Well, I can't test it right now becouse I'm on my phone but maybe this getAccept get language from some browser settings and UI language is from OS? i will check it soon becouse I'm curious.

Language is now being recorded!


Add serde default attribute for fields

When annotated with #[serde(default)], a field gets a default value using
std:default::Default, and will not error when decoding if the field is missing
from the input.
Addresses #9

This is awesome! Unfortunately there is a conflict. Could you rebase?

Rebased.  The only change in the conflicting regions was for the type of the push argument of expand_derive_deserialize and expand_derive_serialize, which seems to have changed.
I'm using rustc 81bce5290 2015-02-16.


Add PPHA's file

created a branch, created a file and made a commit, and opened a pull request

finished the demo

finished the demo


Teams Messaging Chat is changing Shake emoticon to Sad emoticon, when other person replies on sent message thread.

Problem Statement:
Teams Messaging Chat is changing Shake emoticon to Sad emoticon, when other person replies on sent message thread.
Environment
Teams Version(copied from About Window) : You have Microsoft Teams Version <IP_ADDRESS>64 (64-bit). It was last updated on 02-08-2021.
Priority
Major
Steps To Reproduce:

Login with valid credentials as Person A.
Go to teams chat and compose a message containing text with Shake emoticon and send it to other person alias as B.

Actual Result:
When this person B replies on message sent by person A, Shake emoticon gets changed to Sad emoticon.
Expected Result:
Emoticon should not get changed even when other person is replies on same message thread, for more insights, please refer to screenshot
**Screenshot **

@parul001 - I am also able to repro this issue.
We will check internally and raise a bug for it, if needed.

@v-chetsh  Can I know why this issue is closed?. I mean if its reproducible or you need more logs or something else?

@v-chetsh I faced the same bug. What could be the right place to get this bug fixed. Because clearly, turning my expression of "sadness" to the expression of "joy" is not comfortable, specially if i'm empathizing with my grieving friend.
The links mentioned by the msfbbot are for asking questions or developing apps. However, I aim to address this with the best intent. Could you please help it get checked. Happy to provide more details to reproduce and experience the pain points.

@msftbot
Apologies that it's the wrong place. Sure thing. I'm not going to report it again, though. And yeah, I realize I'm talking to a bot, @msftbot) the closing comment doesn't help me to find where to turn to either. :(

@parul001 - We already have a internal bug raised for it but if you want to raise it from your end, you can go to "Help" in Microsoft Teams desktop client & can select "Report a problem" & provide relevant details in it.
Thanks!!


Accordion-clickable

All accordions should now be click to expand.
Any other additional actions (buttons, tab nav, etc.) should not modify the accordion, with the exception of badge icons -- these should only expand the panel on click (but not collapse).
Note that this should not work on the accordion content, i.e. clicking inside an expanded accordion should not collapse the accordion.

Codecov Report

Merging #119 (0c6eab9) into main (e9d248f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #119   +/-   ##
=======================================
  Coverage   93.22%   93.22%
=======================================
  Files         107      107
  Lines        2656     2656
=======================================
  Hits         2476     2476
  Misses        180      180

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 e9d248f...0c6eab9. Read the comment docs.

FYI I force pushed a rebase against main so I could see the report box on top.


Fixed GitHub Release

📲 What
After the build was successful the release to github was failing. Additionally it was not possible to test a release unless on the main branch.
🤔 Why
Being able to release software is crucial for people being able to access and use the tool!
Being able to make a pre-release allows us to be able to release software to more people for testing, plus it allows us to test that the pipeline is working in its entirety.
🛠 How
Changed the script performing the release to use the one form the module and not the script in the repo.
Added logic to the pipeline to be able to perform a pre-release by specifying a parameter to denote this request.
👀 Evidence
Pre-release and all artifcates are created

🕵 How to test
Notes for QA
✅ Acceptance criteria Checklist

[ ] Code peer reviewed?
[ ] Documentation has been updated to reflect the changes?
[ ] Passing all automated tests, including a successful deployment?
[ ] Passing any exploratory testing?
[ ] Rebased/merged with latest changes from development and re-tested?
[ ] Meeting the Coding Standards?

Nice one 🙂


Some types of fetches in tf.Session.run are not supported and it behaves strange in container

There're two issues I have found when giving different type of fetches in tf.Session.run().
The doc says that we can pass the nested list or tuple to run() but it doesn't work.
tf.Session.run(fetches, feed_dict=None, options=None, run_metadata=None)

Runs operations and evaluates tensors in fetches.

This method runs one "step" of TensorFlow computation, by running the necessary graph fragment to execute every Operation and evaluate every Tensor in fetches, substituting the values in feed_dict for the corresponding input values.

The fetches argument may be a single graph element, or an arbitrarily nested list, tuple, namedtuple, or dict containing graph elements at its leaves. A graph element can be one of the following types:

It's easy to test with this script.
import tensorflow as tf

a = tf.placeholder("float")
b = tf.placeholder("float")
add_op = tf.add(a, b)
mul_op = tf.mul(a, b)

output_array = [[add_op, mul_op]]

with tf.Session() as sess:
    output_array = sess.run(output_array, feed_dict={a: 1, b: 2})
    print(output_array)

And I found it works if I pass the array or dict. But what's strange is that the dict doesn't work in docker container which is also easy to reproduce by creating the tensorflow container to run the same script. You can find my test code and error log below.

Environment info
Operating System: Ubuntu 16.04
If installed from binary pip package, provide:
TensorFlow 0.8

Try a newer version of tf

I run the tensorflow/tensorflow:0.10.0rc0 and all list, nested list, dict and tuple work like a charm. The dict works because my local TensorFlow is 0.9.0 and the document seems really clear.
Thanks for your guide @vrv and this feature is really important for our inference services.
BTW, the tensorflow/tensorflow:latest is not "latest" at all.


Push metadata leads to memory exhaustion

This issue is imported from pivotal - Originaly created at Mar 1, 2019 by Bart Geesink
SURFconext currently has so many entities that a PUSH leads to a memory exhaustion:
Mar  1 13:15:29 papp6.prd.utr.surfconext.nl Apache-EBAPI: [Fri Mar 01 13:15:29.494441 2019] [proxy_fcgi:error] [pid 28763] [client <IP_ADDRESS>:37960] AH01071: Got error 'PHP message: PHP Fatal error:  Allowed memory size of 277872640 bytes exhausted (tried to allocate 9272 bytes) in /opt/openconext/OpenConext-engineblock-5.8.7/vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/PDOStatement.php on line 91\n'
The 277872640 bytes - 265MB, which is configured here:
https://github.com/OpenConext/OpenConext-engineblock/blob/master/src/OpenConext/EngineBlockBundle/Controller/Api/ConnectionsController.php#L77
See also https://www.pivotaltracker.com/story/show/164329554
This problem arises when a new IdP is added.
A workaround can be achieved by updating the memory limits in the php fpm pool configuration settings.
When you apply the above mentioned workaround, the IdP can be added.
If you undo the workaround (so removing the memory limit settings again from the fpm pool config settings) pushes are working normally again. The exhaustion will then happen again when you either remove the IdP again, or when you change the ACL settings again of that entity.
It seems that the amount of IdPs and SPs in SURFconext is the maximum that can be processed with 265 MB of memory.
Is it possible to make this process more efficient so it needs less RAM?

(bstrooband - Apr 15, 2019)

Speaking from past experience the best solution is to dismantle the ORM layer for this action. At this moment I have no well-considered opinion on running the action in a single transaction or not. It seems a logical thing to do in my opinion.   (Michiel Kodde - Apr 16, 2019)


feat: treat OutputInstance in template literal as rule violation too

OutputInstance should be treated the same as Output. Please check the tests.
Currently, this is "good code":
const myOutput: pulumi.OutputInstance<number> = pulumi.output([1, 2, 3])[0];
const myBadTemplate = `${myOutput}`

This PR would fix the above code into:
const myOutput: pulumi.OutputInstance<number> = pulumi.output([1, 2, 3])[0];
const myBadTemplate = pulumi.interpolate`${myOutput}`

Hi @blakegong !  Thank you so much for your contribution - my only request here would be to break these into two separate linting rules - I'm fine with duplicating existing code in order to make each individual linting failure more clear.  Once that's done, I'm happy to get this merged in :) Thanks again!

@kpitzen no problem! It's in a new no-output-instance-in-template-literal rule now. Please have a look!

This looks great @blakegong - thank you for your contribution!  I'll get it merged and a new version released. :)

FYI @blakegong - this is now published in version 0.2.0.  Thank you again!

Thank you @kpitzen!


Writing reference type fields

¿Is it possible to store values of type reference? I´ve been trying for a couple of days (while getting familiar with the crate), but all I can get is:

Writing a string instead
Getting a compiler error when trying to write a -non-serializable- ValueType::ReferenceValue containing the string version of the doc reference.

THNX in advance!

Hey, right now struct the Firestore serde serializer read them as strings, and writes them back also as a string. (You can work with them on the low-level doc level though).
It is not complicated to implement the same way as I implemented the Firestore Timestamps or Geo LatLng serializer, something like.
I never used them myself before, but since they're easy to implement I can do this later this week.

This is the PR:
https://github.com/abdolence/firestore-rs/pull/75

Ok, now it is merged and available in the master branch. You can use a reference to the git:
https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html#specifying-dependencies-from-git-repositories
This is an example of how to use it:
https://github.com/abdolence/firestore-rs/blob/master/examples/reference.rs
Let me know if you have questions or issues with this before I release this later on.
Thanks for the report.

THNX!!!!
I just tested it and it works. I´ve tried with a parent_path(...) reference,  as in your example, and also with a document in a subcollection by chainingparent_path(...).at(...).
I´ll be testing it more extensively all along the week, I´ll let you know about the results.
Kind regards

Released in
https://github.com/abdolence/firestore-rs/releases/tag/v0.27.0

I just tested it and it works. I´ve tried with a parent_path(...) reference, as in your example, and also with a document in a subcollection by chainingparent_path(...).at(...).

Sorry for possibly bumping this older issue, but @jordinuco, I was wondering if you could provide some information on how this is useful to you? I'm not seeing a way in the crate to access the data of a  referenced document. It gives the path, but there are not public methods to access the document by path. And, if using a sub-collection, I'm not seeing a way to reference the parent object from a resulting query that returns the child. Any help would be greatly appreciated.
Thanks!

Hey @bekriebel
Since the reference is just a string with absolute path to a document, then it fairly simple manipulation will give you path, collection and document IDs.
To make life easier, I added an auxiliary function in v0.39.1 and left an example here:
https://github.com/abdolence/firestore-rs/blob/cfdde89d23751ca0fba4968810050ffd0e526c19/examples/reference.rs#L69

Thanks, @abdolence. I did start to realize that the reference value doesn't have any special meaning, at least in this context. Rather than bother with the reference type, I'm just using an existing string value that maps to a document ID in my use case. I appreciate the helper function, though - that may be helpful for other uses.

yes, firestore references aren't much useful. The only useful thing I remember there was some kind of UI support in the console. They don't provide consistency or checks. They don't automatically return anything.


external-plugins/cherrypicker: include PR description as comment

This PR prepends the PR description to the comments list, allowing users to specify branches that they wish to cherrypick to when they first create the PR.
This change fixes the cherrypick chain functionality introduced in #136.

/cc @bradmwilliams


I receive an error about the bp attribute

Using React 17 and create-react-app with typescript template
i did npm install blueprint-css and imported using import "blueprint-css/dist/blueprint.min.css" in the index.tsx file
Property 'bp' does not exist on type 'DetailedHTMLProps<HTMLAttributes<HTMLDivElement>, HTMLDivElement>'.  TS2322

    4 |   return (
    5 |

  > 6 |

      |            ^
    7 |

    8 |   );
    9 | }

add the attribute via type augmentation
// in some declaration file you create, for example react.d.ts
import * as React from 'react'

declare module 'react' {
  interface HTMLAttributes<T> extends React.AriaAttributes, React.DOMAttributes<T> {
    bp?: string
  }
}

@steve-py96 thanks for the response, I'm going to leave this open to remind myself to add this as part of the docs.

@steve-py96 thanks for the response, I'm going to leave this open to remind myself to add this as part of the docs.


Version 0.2.2

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
584

Relevant Lines:
2099

💛  - Coveralls


The engine crashed during 3D scene opening

Godot version
v4.0.beta13.official [caacade56]
System information
Linux kernel 5.15.0-58-generic, x86_64, Ubuntu 22.04 // 00:02.0 VGA compatible controller: Intel Corporation GeminiLake [UHD Graphics 600] (rev 06) // OpenGL ES GLSL ES 3.20
Issue description

When I try in empty 3D scene to open 3D gltf-file via 'Instantiate Child Scene' then I get a crash with the following dump:
_================================================================
handle_crash: Program crashed with signal 11
Engine version: Godot Engine v4.0.beta13.official (caacade569eb7a541aaa7a8cdc3eedffca1422d9)
Dumping the backtrace. Please include this when reporting the bug to the project developer.
[1] /lib/x86_64-linux-gnu/libc.so.6(+0x42520) [0x7f088fa79520] (??:0)
[2] /home/iva/Godot_v4.0-beta13/Godot_v4.0-beta13_linux.x86_64() [0x335916b] (??:0)
[3] /home/iva/Godot_v4.0-beta13/Godot_v4.0-beta13_linux.x86_64() [0x224b897] (??:0)
[4] /home/iva/Godot_v4.0-beta13/Godot_v4.0-beta13_linux.x86_64() [0x21e8ef8] (??:0)
[5] /home/iva/Godot_v4.0-beta13/Godot_v4.0-beta13_linux.x86_64() [0x2f06561] (??:0)
[6] /home/iva/Godot_v4.0-beta13/Godot_v4.0-beta13_linux.x86_64() [0x22df983] (??:0)
[7] /home/iva/Godot_v4.0-beta13/Godot_v4.0-beta13_linux.x86_64() [0x1f4a223] (??:0)
[8] /home/iva/Godot_v4.0-beta13/Godot_v4.0-beta13_linux.x86_64() [0x442f788] (??:0)
[9] /home/iva/Godot_v4.0-beta13/Godot_v4.0-beta13_linux.x86_64() [0x426d4b4] (??:0)
[10] /home/iva/Godot_v4.0-beta13/Godot_v4.0-beta13_linux.x86_64() [0x49fa555] (??:0)
[11] /home/iva/Godot_v4.0-beta13/Godot_v4.0-beta13_linux.x86_64() [0x4463487] (??:0)
[12] /home/iva/Godot_v4.0-beta13/Godot_v4.0-beta13_linux.x86_64() [0x2aea4e8] (??:0)
[13] /home/iva/Godot_v4.0-beta13/Godot_v4.0-beta13_linux.x86_64() [0xe973b6] (??:0)
[14] /home/iva/Godot_v4.0-beta13/Godot_v4.0-beta13_linux.x86_64() [0xdcb2b3] (??:0)
[15] /lib/x86_64-linux-gnu/libc.so.6(+0x29d90) [0x7f088fa60d90] (??:0)
[16] /lib/x86_64-linux-gnu/libc.so.6(_libc_start_main+0x80) [0x7f088fa60e40] (??:0)
[17] /home/iva/Godot_v4.0-beta13/Godot_v4.0-beta13_linux.x86_64() [0xdea5ce] (??:0)
-- END OF BACKTRACE --
================================================================

The same issue if I just try to open the scene as "New Inherited Scene"

But if I try just open the scene via "Open Scene" it opens successfully

Steps to reproduce
Try to add gltf-file in project in different ways
Minimal reproduction project
Blank project

Please try to reproduce it on the recent (beta 17 version).

I have tried it on beta 17 version - now it is not crashed but nothing is visible - the scene becomes black and I can see the following records:
ERROR: Condition "index >= vc" is true. Returning: Ref()
at: generate_triangle_mesh (scene/resources/mesh.cpp:192)
ERROR: Condition "index >= vc" is true. Returning: Ref()
at: generate_triangle_mesh (scene/resources/mesh.cpp:192)
ERROR: Condition "index >= vc" is true. Returning: Ref()
at: generate_triangle_mesh (scene/resources/mesh.cpp:192)
ERROR: Condition "index >= vc" is true. Returning: Ref()
at: generate_triangle_mesh (scene/resources/mesh.cpp:192)
ERROR: Condition "index >= vc" is true. Returning: Ref()
at: generate_triangle_mesh (scene/resources/mesh.cpp:192)
...
And as I understand the number of these records is equal to number of 3D elements in instantiated scene.
Additionally I can see the following at the engine start:
_/snap/gnome-3-38-2004/119/usr/lib/x86_64-linux-gnu/libxkbcommon.so.0: undefined symbol: xkb_utf32_to_keysym
/snap/gnome-3-38-2004/119/usr/lib/x86_64-linux-gnu/libxkbcommon.so.0: undefined symbol: xkb_keymap_key_get_mods_for_level
OpenGL Renderer: Mesa Intel(R) UHD Graphics 600 (GLK 2)
Editing project: /home/iva/projects/FromDungeonWithLove/godot_project
Godot Engine v4.0.beta17.official.c40020513 - https://godotengine.org
/snap/gnome-3-38-2004/119/usr/lib/x86_64-linux-gnu/libxkbcommon.so.0: undefined symbol: xkb_utf32_to_keysym
/snap/gnome-3-38-2004/119/usr/lib/x86_64-linux-gnu/libxkbcommon.so.0: undefined symbol: xkb_keymap_key_get_mods_for_level
Vulkan API 1.3.224 - Using Vulkan Device #0: Intel - Intel(R) UHD Graphics 600 (GLK 2)
WARNING: Blend file import is enabled in the project settings, but no Blender path is configured in the editor settings. Blend files will not be imported.
at: editor_init (modules/gltf/register_types.cpp:70)
ERROR: Priority must be greater than 0.
at: set_collision_priority (servers/physics_3d/godot_collision_object_3d.h:170)
May be it impacts of the scene instantiating

Closing because @ruaivantso claims the originally reported crash is fixed, and they opened #73133 for the blank scene issue and say "As I understand it duplicates this issue"


Windows Compatibility and USB Camera Support

Thank you for sharing your great job !
I would like to ask if this project can be used on a Windows system. Additionally, does it support USB cameras?
Thank you for your time and assistance!

Yes it can be used in windows, and yes it supports USB cameras although if
it’s not the main camera you may have to update values in the code
On Mon, Dec 2, 2024 at 10:14 AM xwhkkk @.***> wrote:

Thank you for sharing your great job !
I would like to ask if this project can be used on a Windows system.
Additionally, does it support USB cameras?
Thank you for your time and assistance!
—
Reply to this email directly, view it on GitHub
https://github.com/small-cactus/handTrack/issues/2, or unsubscribe
https://github.com/notifications/unsubscribe-auth/A57SAQMSHRAXZSNZMMUDNVD2DR2NXAVCNFSM6AAAAABS3UTK7GVHI2DSMVQWIX3LMV43ASLTON2WKOZSG4YTEMRYGU4TINY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Yes it can be used in windows, and yes it supports USB cameras although if it’s not the main camera you may have to update values in the code
…
On Mon, Dec 2, 2024 at 10:14 AM xwhkkk @.> wrote: Thank you for sharing your great job ! I would like to ask if this project can be used on a Windows system. Additionally, does it support USB cameras? Thank you for your time and assistance! — Reply to this email directly, view it on GitHub <#2>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A57SAQMSHRAXZSNZMMUDNVD2DR2NXAVCNFSM6AAAAABS3UTK7GVHI2DSMVQWIX3LMV43ASLTON2WKOZSG4YTEMRYGU4TINY . You are receiving this because you are subscribed to this thread.Message ID: @.>

Thank you for your kind reply. When running the project, is it required to have the palm facing the camera for gesture recognition? Would it be possible to use the back of the hand instead ?

The back of your hand will work if you only use one hand, I’m not sure how
well it’ll work with 2 hands showing at once, the tap to click won’t work
if your fingers aren’t in clear view of the camera though.
And you’ll have to enter the screen size of your monitor in the code to use
it on windows, other than that I’m pretty sure it would work.
On Tue, Dec 3, 2024 at 12:54 AM xwhkkk @.***> wrote:

Yes it can be used in windows, and yes it supports USB cameras although if
it’s not the main camera you may have to update values in the code
… <#m_-4845581252566728955_>
On Mon, Dec 2, 2024 at 10:14 AM xwhkkk @.> wrote: Thank you for
sharing your great job ! I would like to ask if this project can be used on
a Windows system. Additionally, does it support USB cameras? Thank you for
your time and assistance! — Reply to this email directly, view it on GitHub
<#2 https://github.com/small-cactus/handTrack/issues/2>, or unsubscribe
https://github.com/notifications/unsubscribe-auth/A57SAQMSHRAXZSNZMMUDNVD2DR2NXAVCNFSM6AAAAABS3UTK7GVHI2DSMVQWIX3LMV43ASLTON2WKOZSG4YTEMRYGU4TINY
https://github.com/notifications/unsubscribe-auth/A57SAQMSHRAXZSNZMMUDNVD2DR2NXAVCNFSM6AAAAABS3UTK7GVHI2DSMVQWIX3LMV43ASLTON2WKOZSG4YTEMRYGU4TINY
. You are receiving this because you are subscribed to this thread.Message
ID: @.>
Thank you for your kind reply. When running the project, is it required to
have the palm facing the camera for gesture recognition? Would it be
possible to use the back of the hand instead ?
—
Reply to this email directly, view it on GitHub
https://github.com/small-cactus/handTrack/issues/2#issuecomment-2513624426,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A57SAQOYV5L6P56SPMVLMQD2DVBS7AVCNFSM6AAAAABS3UTK7GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJTGYZDINBSGY
.
You are receiving this because you commented.Message ID:
@.***>

The back of your hand will work if you only use one hand, I’m not sure how well it’ll work with 2 hands showing at once, the tap to click won’t work if your fingers aren’t in clear view of the camera though. And you’ll have to enter the screen size of your monitor in the code to use it on windows, other than that I’m pretty sure it would work.
…
On Tue, Dec 3, 2024 at 12:54 AM xwhkkk @.> wrote: Yes it can be used in windows, and yes it supports USB cameras although if it’s not the main camera you may have to update values in the code … <#m_-4845581252566728955_> On Mon, Dec 2, 2024 at 10:14 AM xwhkkk @.> wrote: Thank you for sharing your great job ! I would like to ask if this project can be used on a Windows system. Additionally, does it support USB cameras? Thank you for your time and assistance! — Reply to this email directly, view it on GitHub <#2 <#2>>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A57SAQMSHRAXZSNZMMUDNVD2DR2NXAVCNFSM6AAAAABS3UTK7GVHI2DSMVQWIX3LMV43ASLTON2WKOZSG4YTEMRYGU4TINY https://github.com/notifications/unsubscribe-auth/A57SAQMSHRAXZSNZMMUDNVD2DR2NXAVCNFSM6AAAAABS3UTK7GVHI2DSMVQWIX3LMV43ASLTON2WKOZSG4YTEMRYGU4TINY . You are receiving this because you are subscribed to this thread.Message ID: @.> Thank you for your kind reply. When running the project, is it required to have the palm facing the camera for gesture recognition? Would it be possible to use the back of the hand instead ? — Reply to this email directly, view it on GitHub <#2 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A57SAQOYV5L6P56SPMVLMQD2DVBS7AVCNFSM6AAAAABS3UTK7GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJTGYZDINBSGY . You are receiving this because you commented.Message ID: @.**>

Thank you! Just to confirm, is the screen size you mentioned the screen resolution (e.g., 1920x1080) or the physical size of the screen?

Thank you! Just to confirm, is the screen size you mentioned the screen resolution (e.g., 1920x1080) or the physical size of the screen?

Yeah it’s the screen resolution, I have it hard coded for M series MacBooks
right now so the cursor won’t scale completely normal on any other side.
On Tue, Dec 3, 2024 at 10:14 AM xwhkkk @.***> wrote:

Thank you! Just to confirm, is the screen size you mentioned the screen
resolution (e.g., 1920x1080) or the physical size of the screen?
—
Reply to this email directly, view it on GitHub
https://github.com/small-cactus/handTrack/issues/2#issuecomment-2514846363,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A57SAQKV34C6LHTAXTFADBT2DXDEVAVCNFSM6AAAAABS3UTK7GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJUHA2DMMZWGM
.
You are receiving this because you commented.Message ID:
@.***>

I have test it in the windows

The back of your hand will work if you only use one hand, I’m not sure how well it’ll work with 2 hands showing at once, the tap to click won’t work if your fingers aren’t in clear view of the camera though. And you’ll have to enter the screen size of your monitor in the code to use it on windows, other than that I’m pretty sure it would work.
…
On Tue, Dec 3, 2024 at 12:54 AM xwhkkk @.> wrote: Yes it can be used in windows, and yes it supports USB cameras although if it’s not the main camera you may have to update values in the code … <#m_-4845581252566728955_> On Mon, Dec 2, 2024 at 10:14 AM xwhkkk @.> wrote: Thank you for sharing your great job ! I would like to ask if this project can be used on a Windows system. Additionally, does it support USB cameras? Thank you for your time and assistance! — Reply to this email directly, view it on GitHub <#2 <#2>>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A57SAQMSHRAXZSNZMMUDNVD2DR2NXAVCNFSM6AAAAABS3UTK7GVHI2DSMVQWIX3LMV43ASLTON2WKOZSG4YTEMRYGU4TINY https://github.com/notifications/unsubscribe-auth/A57SAQMSHRAXZSNZMMUDNVD2DR2NXAVCNFSM6AAAAABS3UTK7GVHI2DSMVQWIX3LMV43ASLTON2WKOZSG4YTEMRYGU4TINY . You are receiving this because you are subscribed to this thread.Message ID: @.> Thank you for your kind reply. When running the project, is it required to have the palm facing the camera for gesture recognition? Would it be possible to use the back of the hand instead ? — Reply to this email directly, view it on GitHub <#2 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A57SAQOYV5L6P56SPMVLMQD2DVBS7AVCNFSM6AAAAABS3UTK7GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJTGYZDINBSGY . You are receiving this because you commented.Message ID: @.**>

Hi, I have tested this project on Windows, and it works with the back of the hand; however, the mouse movement in the vertical direction is reversed. Additionally, if I want to use this project on an external projection screen, are there any modifications I need to make to ensure it works correctly?
I appreciate your help!


Establishing a RoadEvent superclass and a WorkZoneEvent subclass.

Establishing a RoadEvent superclass and a WorkZoneEvent subclass
The RoadEvent object in version 3.1 of the Work Zone Data Specification comingles two objects, Road Events and Work Zone Events, in a single class.  Comingling different objects in a single class deviates from the well-established best practice of data normalization.  Normalization is important in data design because it reduces redundant data storage and exchange.
Normalization is important to the Work Zone Data Working group because it eliminates the unnecessary exchange of unneeded, or unwanted, data and enables the publication of basic road features like named roads or named roads with restrictions.  Without normalization the required fields prohibit the specification from being used to publish anything other than a Work Zone Feature or a detour related to a work-zone.  Making the specification flexible, enabling it to address nonwork-zone use-cases, maximizes its social benefit.
Normalizing the content of the Road Event object will create two classes; A RoadEvent class and a WorkZoneEvent class. (See model at end of post.)  Each object will only contain the properties necessary to exchange information about itself.  The RoadEvent class would contain 10 properties; Four of which are required. Similarly, the WorkZoneEvent would contain 25 properties;  Eleven of which are required. The WorkZoneEvent would inherit 10 RoadEvent properties and has 15 properties specific to a work zones.
With normalization, a data publisher needs only 4 properties to publish a basic road feature like a named road segment: event_type, source_id, names, and direction.  A data publisher needs 11 properties to publish a work-zones.
Positives

Inline with Objective 2 of the Working Group's Charter
A more flexible specification meets multiple use-cases
A more flexible specification may act as a gateway and encourage data publishers to use the WZD spec
Enables the publication of restrictions, and named road segments
Maximizes the social benefit of the specification

Negatives

Breaks backwards compatibility

Draft Normalized Class Model

This is a great idea! Seattle already uses our feed to communicate scheduled events that impact travel such as parades and runs. The necessary data overlaps that of work zones significantly and the end consumers are the same.
I suggest moving the following properties form the WorkZoneEvent to the RoadEvent since each RoadEvent can only represent one GeoJSON feature:
beginning_cross_street
ending_cross_street
beginning_milepost
ending_milepost
We may want to consider moving the spatial verification properties as well but I can envision cases where you would want the spatial verification tied to the WorkZoneEvent; such as when the spatial verification is being communicated by an automated device at the work zone.

Resolved in #209.


Issue emplacing component to entity

Hi,
I am getting the following error:
Assertion failed: (entity_traits::to_version(elem) == entity_traits::to_version(tombstone)), function assure_at_least, file sparse_set.hpp, line 210.
when I emplace a component to an entity that I know is alive (because I have printed out all the entities that have been "destroyed" by calling the method destroy and I didn't see that one on the list). The emplace happens within a view each and the entity that I am emplacing the component to is the current entity.
 I am assuming that this error message refers to a emplacing a component to an invalid/tombstoned entity, is that right? if so or not, what could be another situation when this message is shown?
P.S. I am not using the in-place delete functionality.
Thanks

Can I ask you what version of EnTT is it? I can't find this line of code anymore. 😅
Anyway, at a first glance, this is what replaced it:
ENTT_ASSERT(elem == null, "Slot not available");

It states that the set already contains the component. So, roughly speaking, you're adding a component to an entity that has an instance of the same type already.
You can try to assert before invoking emplace to confirm it.

So, roughly speaking, you're adding a component to an entity that has an instance of the same type already.

So, you are saying that the entity already has the same component added to it?

It looks so from the error, yeah. Hard to confirm 100% without a repro though.

Ok, that makes sense, I was able to find the issue with your explanation, thanks. Quick question, do you normally use emplace_or_replace. I feel that it's hard to keep track of all system dependencies and the components that each add, especially with a large set of systems.

Actually, I don't. The last version of EnTT offers the possibility of getting the storage from a view and this makes passing the registry around pointless. Moreover, the whole library is moving towards a model where you can have storage classes and views but no registry at all if you like.

Interesting, do you have a post or doc with more details of the model where EnTT is headed to?

I should but there is not actually. If you have any questions, feel free to drop them here and I'll be glad to answer them.


statistics don't match

the means indicated in the attached photos should match. the quartiles match.

This is the call i am making to get the aaeq summary table damage reduced:

This is the call to get the AAEQ damage with uncertainty value. I was doing the top line, but i see that there is a damage reduced call (the bottom red arrow). @rnugent3  Should I be using the damage reduced call here?

Looks like this is the same issue as #449

This "Mean" value should be the damage reduced call.

correct

Looks like this is the same issue as #449

I think that is right.


UF-225 Skipping unlock when RuntimeException is thrown in AbstractIOService.endBatch()

BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1251358
Description of problem:
AbstractIOService.java
    public void endBatch() {
        if ( !batchLockControl.isLocked() ) {
            throw new RuntimeException( "There is no batch process." );
        }

        if ( batchLockControl.getHoldCount() > 1 ) {
            batchLockControl.unlock();
            return;
        }

        if ( !fileSystems.isEmpty() ) {
            cleanupClosedFileSystems();
        }

        for ( final FileSystem fs : fileSystems ) {
            unsetBatchModeOn( fs );
        }

        batchLockControl.unlock();
    }

If an RuntimeException is thrown before "batchLockControl.unlock()", the lock is not released so subsequent threads will wait for the lock forever.
(for example)
RULE throwEx
CLASS org.uberfire.io.impl.AbstractIOService
METHOD unsetBatchModeOn
AT ENTRY
IF TRUE
  DO throw new RuntimeException("dummy");
ENDRULE

Solution:
In  order to reproduce this, I've created a new test and a fix for it.
However, even if the lock is released the exception of the FS would leave the UF in an inconsistent state. Anyway, this PR raises an exception if that happens (and releases the lock).

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

ok to test

'ok to test'

master: 00bde1a5e43f9dd19d617d87bebb5b448aee9244
0.7.x: ca8c804a93cae2b619c0d2ca4b35169977dfeb7d

@csadilek, @ederign could you please try again commenting with the trigger phrase (without the quotes)? There was a configuration error in the Jenkins job, which should be now hopefully fixed.

Ah, I see the the PR was just merged. So let's test this with the next PR :)

ok to test

Closed PRs are ignored, so the phrase will not work here anymore. After the fix, everyone with public membership (can be configured per user) in uberfire or droolsjbpm github org. units should be able to approve the PR + the build should be automatically triggered for them (without the need for the phrase).

+100 -- this problem has been around for a while. Thank you for fixing this Eder!


feat(message-input): DLT-1753 add support for dpm meeting pill

feat(message-input): DLT-1753 add support for dpm meeting pill

Obligatory GIF (super important!)

:hammer_and_wrench: Type Of Change

These types will increment the version number on release:

[x] Feature

:book: Jira Ticket
https://dialpad.atlassian.net/browse/DLT-1753
:book: Description

Added a new node view to support the functionality of DPM pill within the message input.
Refactored rich-text-editor to use $listeners so we don't have to manually output events from node-views in rich-text-editor itself. Clicking the button on the pill will trigger meeting-pill-close event.
Still need to do vue 3 changes. It will be a bit of additional work because of $listeners.

:bulb: Context
Since this component is quite product specific I've included it within message input recipe instead of rich text editor, and made a way to pass additional extensions to the rich-text-editor.
In order to get this to show up you need to set the value to  into the input (with i18n wrapped text in product of course). It will output the pill in the text as "/dpm"
The component will not invoke this immediately when typing /dpm, you have to receive the selected-command event and then set the input to display the node view as mentioned above.
There is some bug with the input losing focus right now, it's not related to this change, I am going to look into it.
:pencil: Checklist

For all PRs:

[x] I have ensured no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).
[x] I have reviewed my changes.
[x] I have added all relevant documentation.
[x] I have considered the performance impact of my change.

For all Vue changes:

[ ] I have added / updated unit tests.
[ ] I have made my changes in Vue 2 and Vue 3. Note: you may sync your changes from Vue 2 to Vue 3 (or vice versa) using the ./scripts/dialtone-vue-sync.sh script. Read docs here: Dialtone Vue Sync Script
[ ] I have validated components with a screen reader.
[ ] I have validated components keyboard navigation.

For all CSS changes:

[x] I have used design tokens whenever possible.
[x] I have considered how this change will behave on different screen sizes.
[x] I have visually validated my change in light and dark mode.
[x] I have used gap or flexbox properties for layout instead of margin whenever possible.

:crystal_ball: Next Steps
Let Bianca know to try it out in product when this is released.
:camera: Screenshots / GIFs

:link: Sources
https://tiptap.dev/docs/editor/guide/node-views/vue

How will this be used by the product? Would they need to add the meeting pill as an additional extension? In that case, does it need to be exported?

How will this be used by the product? Would they need to add the meeting pill as an additional extension? In that case, does it need to be exported?

No, it's just automatically used in the message_input (but not rich_text_editor)

qq: is there a way to disable typing when having the meeting pill?

hmm not really without also disabling the backspace to remove functionality. I will try and find a solution.

qq: is there a way to disable typing when having the meeting pill?

Added new prop called "preventTyping" that will prevent you from typing any more characters when true, but will still allow you to delete.

btw I'm not seeing preventTyping prop here

Right, I need to add it to message input also


Introduce Gemfile, ruby-version

Summary
Implement par of the discussion https://github.com/react-native-community/discussions-and-proposals/discussions/411, except the .nvmrc part, this includes:

Setting .ruby-version in the main project and also template/
Fixing the CocoaPods version with a project-level Gemfile and also template/Gemfile
Using all pod executions from bundle exec pod, using the determined version
Script to manage and update the ruby version

Changelog
[iOS] [Added] - Gemfile with CocoaPods 1.11 and ruby-version (2.7.4)
Test Plan
Build for iOS and run all CircleCI tests to see if nothing changed

Looks like the build failures may be related?
#!/bin/bash -eo pipefail
node ./scripts/publish-npm.js --dry-run
Setting Ruby version to: 2.7.0
sed: can't read : No such file or directory
Failed to update Ruby version
Failed to bump version number

Exited with code exit status 1
CircleCI received exit code 1

I'll take a look tomorrow, maybe change to set -xe so we can see where it failed... It's weird because
sed -i '' -e "s/^\(ruby '\)[^']*\('.*\)$/\1$VERSION\2/" Gemfile
sed -i '' -e "s/^\(ruby '\)[^']*\('.*\)$/\1$VERSION\2/" template/Gemfile

and both files were added in the commit, but I need to see if stuff was copied, maybe I forgot to add those files.

@fkgozali do you have more knowledge on this?
+ cd /root/react-native/scripts/..
...
+ sed -i '' -e 's/^\(ruby '\''\)[^'\'']*\('\''.*\)$/\12.7.0\2/' Gemfile

is the /root/react-native the checkout or something else? why wouldn't Gemfile be there?

also, is it linux or macOS? Because sed -i behaves different in both cases, so maybe that's the reason

@fkgozali do you have more knowledge on this?
+ cd /root/react-native/scripts/..
...
+ sed -i '' -e 's/^\(ruby '\''\)[^'\'']*\('\''.*\)$/\12.7.0\2/' Gemfile

is the /root/react-native the checkout or something else? why wouldn't Gemfile be there?

Hmm it should be the checkout. I'm actually not sure here, you might need to add a bit of more logs (echo?) to debug further.
The iOS tests are always running on MacOS, e.g. https://app.circleci.com/pipelines/github/facebook/react-native/10546/workflows/27b085f4-3142-4453-9125-cbbaea93d3e0/jobs/219844

added some more safety, just updates from macOS... I think that's the sed -i differences between Linux and macOS, the Linux doesn't recognize the empty string argument as the extension (ie: no backup) and instead will consider the empty string as the file to be updated :-/
Since the Ruby part is only interesting for macOS, I'm ignoring updates from other OS.

@fkgozali seems that one is fixed, but still some errors. Could you check what are they about?
test_ios_unit_jsc - yarn test-ios and test_ios_unit_hermes both fail in a similar [CP-User] Generate Specs phase, this could be related, but hard to say without actual logs:
PhaseScriptExecution [CP-User]\ Generate\ Specs /Users/distiller/Library/Developer/Xcode/DerivedData/RNTesterPods-cbjldxeeenzmdtbkoznuxxultmjt/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/FBReactNativeSpec.build/Script-D50C3B18C67D0D1A8D380643D5AEF835.sh (in target 'FBReactNativeSpec' from project 'Pods')

Others looks unrelated:

test_docker_android is failing echo y | npx envinfo@latest
analyze_pr fails using apt repository https://deb.nodesource.com/node_14.x

test_ios_unit_jsc - yarn test-ios and test_ios_unit_hermes both fail in a similar [CP-User] Generate Specs phase, this could be related, but hard to say without actual logs:

These are unrelated, it was due to Xcode 13 upgrade, but we're not sure why yet.

test_ios_unit_jsc - yarn test-ios and test_ios_unit_hermes both fail in a similar [CP-User] Generate Specs phase, this could be related, but hard to say without actual logs:

These are unrelated, it was due to Xcode 13 upgrade, but we're not sure why yet.

some google says it's related to build order, I'll take a look if I can fix that in another PR.
As for the current approach regarding ruby-version + Gemfile, let me know what you guys think about that.

also cc @mikehardy for inputs

The other error should be fixed by https://github.com/facebook/react-native/pull/32309

Might I suggest also adding a bundler config to tell bundler to install the dependencies locally in the project? Again, to keep things independent of global installed deps:
.bundle/config:
---
BUNDLE_PATH: "vendor/bundle"

And then add /vendor to the .gitignore.

I'm not good enough with ruby to offer much, but I do worry if it doesn't work on Linux for android only template init build tests (I do this sometimes) or if there's no way to override (I use ruby 3.0.2 personally). Those are feature specification hopes not code comments though. I worry a little about future issue traffic from the ruby ignorant (like myself) about all things related to this, I fear it opens a support surface area for ruby distribution, upgrades, gem files etc), and I'm curious if it will be balanced by a decrease in issue count around incorrect ruby versions, or incorrect pod versions.

I'm not good enough with ruby to offer much, but I do worry if it doesn't work on Linux for android only template init build tests (I do this sometimes) or if there's no way to override (I use ruby 3.0.2 personally). Those are feature specification hopes not code comments though. I worry a little about future issue traffic from the ruby ignorant (like myself) about all things related to this, I fear it opens a support surface area for ruby distribution, upgrades, gem files etc), and I'm curious if it will be balanced by a decrease in issue count around incorrect ruby versions, or incorrect pod versions.

It will work on Android, but in android there is no need to pod install. Currently the usage of Gems is only to get CocoaPods. In the future we could isolate parts of that with platform blocks/groups, in that case I'd have to fix the sed call to be different on Linux (easy, but the current "fix" is simpler and more reliable, as no ruby/gems is used outside of macOS).
As for use another version of Ruby, you definitely can, in particularly if you create a new repo (react native init), you're free to change the versions there, but you may run into problems since it's not as tested.
We're sticking to the "latest stable", and it's the same as CircleCI (and likely other CI) uses.

Those are certainly reasonable responses. My comments were all non-quantitative, they were qualitative, so any reasonable defense can be thought of as sufficient I think. Works for me and we'll see I think :-)

@mikehardy the new version uses the sed_i function to allow it to run on Linux.
@liamjones it also uses .bundle/config
However, I noticed something: the _-prefixed files are not being replaced automatically as I thought, instead we'd need to convert them by changing https://github.com/react-native-community/cli/blob/master/packages/cli/src/tools/generator/copyProjectTemplateAndReplace.ts#L135-L150 so I need to send a patch there, if those are to be used and also fix scripts/run-ci-e2e-tests.js

@barbieri https://www.youtube.com/watch?v=AbSehcT19u0 :laughing:

@barbieri https://www.youtube.com/watch?v=AbSehcT19u0 😆

exactly! :-D

@barbieri @mikehardy thanks for the inputs. To be honest, I have little context on Ruby/Gemfile setup, but I did find myself struggling to make sure my env was correct, usually around random Ruby failures. So from that angle alone, IMO "any signals to indicate that something is misconfigured in your system" is very valuable. I understand the potential risks of us getting asked non-React Native questions like Ruby stuffs, but perhaps it's gonna be just in the short term (we'll see).
So I'll import this PR and get some more inputs from our team at FB, and if there's no concern, we'll merge it. I see this as an incremental improvement though, so we can always tweak things as needed and make things more robust.

@barbieri https://github.com/facebook/react-native/commit/34b7d22f3d33792ffd86092d6c39564969c027d8 just landed, which should fix the iOS tests. Could you rebase this PR to pick it up so we can be sure your PR doesn't regress anything else? After that I'll import

@fkgozali rebase done

Thanks for the updates. I've validated this vs our internal script (had to bring back validate_env, but that's about it). I'm gonna land this for now so we can iterate further as needed. Thanks again for building this out!

Btw, there's an internal test failure (I'm looking at it) related to the _bundle dir in the new app template. In the meantime, it would be useful to validate that this flow works with the new app template as well. You can try the .tar.gz artifact with the npx react-native init CLI to create a brand new app: https://app.circleci.com/pipelines/github/facebook/react-native/10566/workflows/99007ae2-648e-4389-a218-4052da0eda03/jobs/220064/artifacts

Nice work @barbieri :+1:
I do have one question though (Please correct me if I'm wrong): These changes are primarily for working with the react-native repo itself (e.g. running e2e tests, CI, etc), correct?
If yes, can we - by any chance - keep the basic app template clean and minimal without the overhead of additional Ruby/Gemfile/Bundler files (i.e. undo the changes inside of the template/ directory)?
There already is a clearly written section about the requirement to install CocoaPods in the official React Native environment setup guide (cc @thymikee).
I'm quoting one section:

You can use a Ruby Version manager, however we recommend that you use the standard Ruby available on macOS unless you know what you're doing.

To me it sounds like everything should still work for someone who set up their environment following the guide. I don't have an M1 mac available to test right now (in https://github.com/react-native-community/discussions-and-proposals/discussions/411 you mentioned "ffi on arm64/m1") - Are there any issues with it and the previous template?

Btw, there's an internal test failure (I'm looking at it) related to the _bundle dir in the new app template. In the meantime, it would be useful to validate that this flow works with the new app template as well. You can try the .tar.gz artifact with the npx react-native init CLI to create a brand new app: https://app.circleci.com/pipelines/github/facebook/react-native/10566/workflows/99007ae2-648e-4389-a218-4052da0eda03/jobs/220064/artifacts

I'll take a look what's the exact problem, but I know I need to modify the cli project to convert the new _-prefixed files to ..

Nice work @barbieri 👍
I do have one question though (Please correct me if I'm wrong): These changes are primarily for working with the react-native repo itself (e.g. running e2e tests, CI, etc), correct? If yes, can we - by any chance - keep the basic app template clean and minimal without the overhead of additional Ruby/Gemfile/Bundler files (i.e. undo the changes inside of the template/ directory)?

Yes, they work (at least with the CI and @fkgozali internal tests as well. The idea of template/ is to make it easier, not more complex, by bringing some well known versions, if you reduce the variations you're likely to reduce the number of issues. For example CocoaPods 1.11 works much better than 1.10 for newer Apple hardware (M1). Even CircleCI recommends that: https://circleci.com/docs/2.0/testing-ios/#images-using-xcode-117-and-later

As a result of the macOS system Ruby (2.6.3) becoming increasingly incompatible with various gems (especially those which require native extensions), Xcode 11.7 and later images default to Ruby 2.7 via chruby. Defaulting to Ruby 2.7 allows for greater compatibility and reliability with gems moving forward. Common gems, such as Fastlane, run without any issues in Ruby 2.7.

There already is a clearly written section about the requirement to install CocoaPods in the official React Native environment setup guide (cc @thymikee).

That doesn't state the recommended version, more than that, it will install global in the system, which may cause issues with people working in multiple projects (common if you have to do maintenance in one project while creating a new one).

I'm quoting one section:

You can use a Ruby Version manager, however we recommend that you use the standard Ruby available on macOS unless you know what you're doing.

To me it sounds like everything should still work for someone who set up their environment following the guide.

It seems it will still work, the Gemfile and .ruby-version are not strictly required, you cam gem install yourself, but if you call via bundle exec it will first check the version. Of course you're supposed to use the "tested version", but you can just ignore, remove the file or do whatever you want. Same for .ruby-version, that will help to get the proper environment setup, but if you do not have rbenv or rvm, the environment won't be changed.

I don't have an M1 mac available to test right now (in react-native-community/discussions-and-proposals#411 you mentioned "ffi on arm64/m1") - Are there any issues with it and the previous template?

To make sure I cleaned everything and did a new setup: there is no real problem, seems ruby 2.6.3 DOES WORK with ffi_c.bundle IF AND ONLY IF you never messed up with gems and rosetta2.
What happened (at least to me and some colleagues) is that many tutorials to work around M1 limitations did recommend to execute the processes in Rosetta environment. If you did that, the files may be of an incorrect architecture:
$ find /Library/Ruby/Gems -name 'ffi_c.bundle' | xargs file
/Library/Ruby/Gems/2.6.0/extensions/universal-darwin-20/2.6.0/ffi-1.15.4/ffi_c.bundle: Mach-O 64-bit bundle arm64
/Library/Ruby/Gems/2.6.0/gems/ffi-1.15.4/ext/ffi_c/ffi_c.bundle:                       Mach-O 64-bit bundle arm64
/Library/Ruby/Gems/2.6.0/gems/ffi-1.15.4/lib/ffi_c.bundle:                             Mach-O 64-bit bundle arm64

So these are correct. If any of the ffi_c.bundle is x86_64, then people may have issues with that. The simplest "fix" is to uninstall everything and install again without Rosetta2: sudo gem uninstall -Iax

I'll take a look what's the exact problem, but I know I need to modify the cli project to convert the new _-prefixed files to ..

The problem for FB internal was the command used to move _ to .. It used find -H * ... which works fine for files, but failed on dirs. I had to modify it to find -H . ...

Any reason vendor/bundle hasn't been added into template _gitignore ?

Any reason vendor/bundle hasn't been added into template _gitignore ?

hum... I forgot, nice catch :-)

@moox @barbieri if either of you makes a PR to address that, can you tag me in it so that we don't forget to cherry pick for 0.67?

@kelset what branch should the PR be pointed at?

main, and then we cherry pick to 0.67 branch

Now that this is out and live, it immediately causes me a problem:
/Users/mike/.rvm/gems/ruby-3.1.0/gems/bundler-2.3.5/lib/bundler/definition.rb:426:in `validate_ruby!': Your Ruby version is 3.1.0, but your Gemfile specified 2.7.4 (Bundler::RubyVersionMismatch)
        from /Users/mike/.rvm/gems/ruby-3.1.0/gems/bundler-2.3.5/lib/bundler/definition.rb:401:in `validate_runtime!'

I'm running current stable version of ruby, and I try to force everything to use that, so I do not have, and do not want ruby-2.7.4 on my system. I understand the desire for react-native to have a known version to work with, but having it force the issue at a step where I cannot intercept it (during npx react-native init) in order to modify the Gemfile or anything is not something I want.
For now I have to install the old ruby just to run scripted compile tests that start with react-native init I guess
But is there a way to make this optional? Or make it accept any version above the minimum, like >=2.7.4 ?

I can use --skip-install on the npx react-native init line, then sed the Gemfile to have ruby '>= 2.7.4' then do yarn && pushd ios && pod install && popd and it works.
Having the >= for ruby version by default seems like it would allow for perhaps some problems but would also work better for the reasonable case where 2.7.4 had long ago been abandoned by the user

I can use --skip-install on the npx react-native init line, then sed the Gemfile to have ruby '>= 2.7.4' then do yarn && pushd ios && pod install && popd and it works.
Having the >= for ruby version by default seems like it would allow for perhaps some problems but would also work better for the reasonable case where 2.7.4 had long ago been abandoned by the user

maybe doesn't help due the lockfile, but since you're running it, have a try if that changes (Gemfile.lock).

I am facing trouble due to this change, below are my doubts.

When I run bundle exec pod install, I get Your Ruby version is 2.6.8, but your Gemfile specified 2.7.4. So should I change system level Ruby version?
https://reactnative.dev/docs/environment-setup the docs still says to install global cocoapods. No doc related to these changes.
Is these changes ported to npx react-native, ie react-native-cli?
Why do we include the Gemfile.lock file in the template? Should not the end user just generate it?
Should we bundle install before bundle exec pod install?

@ravirajn22 if you install rbenv or rvm that can use the .ruby-version file to ensure the required version of ruby is available for the project without affecting your system version

I want to try the New Architecture
But my Ruby version was lower than 2.7.5 so I installed rbenv, configured... and now
ruby -v returns:
ruby 2.7.5p203 (2021-11-24 revision f69aeb8314) [arm64-darwin21]

But RCT_NEW_ARCH_ENABLED=1 bundle exec pod install returns:
Your Ruby version is 3.1.2, but your Gemfile specified 2.7.5

Why? How to use uby 2.7.5 here?

@Bardiamist I don't know if it helps, but what happens if you try running RCT_NEW_ARCH_ENABLED=1 pod install? I reckon (based on my experience with rbenv) that what might happen is that it might complain that cocoapods is not installed (basically rbenv makes you install cocoapods per each version of ruby) - but at that point, after installing it from that same folder (so that it will install it for 2.7.5) it should work?

@Bardiamist I don't know if it helps, but what happens if you try running RCT_NEW_ARCH_ENABLED=1 pod install? I reckon (based on my experience with rbenv) that what might happen is that it might complain that cocoapods is not installed (basically rbenv makes you install cocoapods per each version of ruby) - but at that point, after installing it from that same folder (so that it will install it for 2.7.5) it should work?

Ok, thanks, understand more now. Yes RCT_NEW_ARCH_ENABLED=1 pod install works because I has cocoapods installed by sudo gem install cocoapods.
But idea was to run bundle install for to install cocoapods. Look it installed cocoapods by Ruby 2.7.5 and now RCT_NEW_ARCH_ENABLED=1 bundle exec pod install also works.

I have to admit I still dislike the whole bundle / Gemfile / .rubyversion thing. I do this every time now:

npm_config_yes=true npx @react-native-community/cli init rnfbdemo --skip-install --version=0.69.4
cd rnfbdemo

# New versions of react-native include annoying Ruby stuff that forces use of old rubies. Obliterate.
if [ -f Gemfile ]; then
  rm -f Gemfile* .ruby*
fi

# Now run our initial dependency install
yarn


Add some client tests that are run across all clients

List of client functionality, that is tested equally across all clients.
Maybe each client has a list of the functionality that is implemented and should work, and ones that are not implemented are not tested.
use same "simple" test scripts and expected outputs on each one...

Can write them one in any lang as they are system tests and use the client via it's binary and a script, which in turn starts the anna processes.
So, could use simple-test written in rust to do the same test across all client implementations


[ANDROID] Deep link under WebVew not worked

Bug description: Im tryin to open other application from WebView but it not works it say Cant open bankId:// url

To Reproduce:

Expected behavior:

Screenshots/Videos:

Environment:

OS: Andorid
OS version: Latest
react-native version:
react-native-webview version:

I am having the same issue, did you found any solution for this?

Nope, instead of web view I started use web browser for Android, it solved my issue. @mouhsnimohamed

@abgaryanharutyun did you get the same error in IOS? for me it wasn't! here is what I got
ANDROID
errorDomain:  undefined
errorDesc:  ERR_UNKNOWN_URL_SCHEME
errorCode:  -10
IOS
errorDesc: unsupported URL
errorCode: -1002
errorDomain: NSURLErrorDomain

I finally fix this by allowing intent in originwhitelist prop
originWhitelist={['https://', 'intent://']}

@mouhsnimohamed now it worked correctly on IOS, issue related only for Andorid, but I solved the issue, for andorid i just opening browser instead of WebView


[vcpkg ci pipeline] suggest vcpkg_cmake -> vcpkg_cmake migration

I have now moved the text into the Details so that the message is not so long. The PR is now ready.

funktions

It's getting funky in here! :D

It's getting funky in here! :D

That is just a german language party ;) (https://dict.leo.org/german-english/function)

It's getting funky in here! :D

That is just a german language party ;) (https://dict.leo.org/german-english/function)

🎉🎉🎉🎉🎉🎉🎉🎈🎈🎈🎈🎈🎈🥳🥳🥳🥳🥳🥳🎈🎈🎈🎈🎈🎉🎉🎉🎉🎉

Dropping reviewed: This is good to go but I'm going to make sure other maintainers are OK with it (~11AM pacific tomorrow time)

This should perhaps be backed by adding the deprecated functions to https://github.com/microsoft/vcpkg/blob/master/docs/maintainers/maintainer-guide.md#avoid-deprecated-helper-functions.

Thanks for your help!

This should perhaps be backed by adding the deprecated functions to https://github.com/microsoft/vcpkg/blob/master/docs/maintainers/maintainer-guide.md#avoid-deprecated-helper-functions.

I agree. @autoantwort I added this to my pile for today but if you get to it before me that would be great.

I just saw this in action here and it seems to look at some older commits in the PR, because its recommendation is off?

@autoantwort Can you take a look at @wrobelda 's report?

Another example where it did the wrong thing: https://github.com/microsoft/vcpkg/pull/20207

I will look at it now

Fix in #20218


Version command added

This PR will add the version command for tk8

@niki-1905 we don't want the version to be hard-coded but rather it should have an option to accept the version number dynamically at build time

@arush-sal can you add a Makefile and pass the versions as ldflags , let me see I can get some time to do that.

@arush-sal  Okay. I will work on that.

@frezbo , PR for Makefile is already there https://github.com/kubernauts/tk8/pull/9

@niki-1905 , please add commit message :)

@arush-sal mind looking at https://github.com/goreleaser/goreleaser for releasing and integrating with a CI?

I was too late :-) I just finished my take on the version cmd file.

I would get the latest tag using this command during the build git describe --tags and then use that as part of the bin stage in the Makefile

@lhaig IMHO fetching the latest tag using git tag -l|head -n 1 or git describe --tag --abbrev=0 would make more sense as compared to git describe --tags as that way we won't have the additional hash value attached with the version.

@niki-1905 Can you rebase your changes as that will resolve the conflict.

@frezbo Thank you for the suggestion to goreleaser. It looks promising and we will definitely look into the it.

@niki-1905 Closing this PR as of now, please create another once you have all the changes done.


Thead Invariant Violation Dynamic Data

Hi,
I have a table where I don't know how many columns there will be until the table is rendered.  I receive the following error:
Uncaught Invariant Violation: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.
    at invariant (http://localhost/static/js/1.chunk.js:45401:19)
    at createFiberFromTypeAndProps (http://localhost/static/js/1.chunk.js:55995:15)
    at createFiberFromElement (http://localhost/static/js/1.chunk.js:56015:19)
    at reconcileSingleElement (http://localhost/static/js/1.chunk.js:58382:27)
    at reconcileChildFibers (http://localhost/static/js/1.chunk.js:58442:39)
    at reconcileChildren (http://localhost/static/js/1.chunk.js:60306:32)
    at updateHostComponent (http://localhost/static/js/1.chunk.js:60790:7)

The above error occurred in the
 component:
    in thead (created by Thead)
    in Thead (at Table.js:139)

My code is:

{headerGroups.map((h, i) => {

                {i}

        })

        }

          {page.map(
            (row, i) => {
              console.log(row)
              return prepareRow(row) || (

                  {row.cells.map(cell => {
                    return (
                      {cell.render('Cell')}
                    )
                  })}

              )}
          )}

I am using this along with react-table.  The suggestions on the ReadMe for dynamic header titles seems related to my issue, but it hasn't provided me with any clues.

I think I can spot a problem up front after glancing at your code above; check this section where the error occurs:

{headerGroups.map((h, i) => {

      {i}

  })
}

Since you use { } around your map function, those contents (
...
) aren't actually getting returned. Second, I don't think you intend to create a row for each header, just a header cell for each within a single row. So you would need to change it up to something like this:

    {headerGroups.map((h, i) =>
      {i}
    )}

That way, the contents of your map function will be returned. I also added a key prop since that's preferable any time you have a map.
Now, as for making it work correctly with this library and dynamic headers, the thing you'll need to do is change another key prop on the main
 component whenever the columns would be changing--that change lets React know to recreate the component from scratch, which will ensure this library rebuilds the structure of the table headers.
You could do something simple like this, since your headers are just a list of strings inside headerGroups:

That merely creates a string key out of the list of headers, so that it will change when they change. I just quickly typed out all of the above, however, so it might be missing something in your use case.

@jasonphillips Thank you very much - these insights helped me to resolve my issue - had been banging my head for a while.


+[AZSCloudStorageAccount accountFromConnectionString:error:]: unrecognized selector sent to class

I just updated to whatever the latest version is via cocoapods and I'm getting this error when trying to instantiate an AZSCloudStorageAccount object via swift:
ex:
let account = try! AZSCloudStorageAccount(fromConnectionString: "DefaultEndpointsProtocol=https;AccountName=\(accountName!);AccountKey=\(accountKey!)")
this worked fine before the update minus the try! since the call now throws.

Thank you for your message.  We are looking into it and will get back to you.

I'm still seeing this, however I'm only noticing this on one of my machines. If I build the project on a different machine it works fine, but not my primary one.

Deleting the project contents in the DerivedData folder fixed this, for anyone else that might be running into this.

@vtcajones Thank you so much for figuring this out! 👍


add sql db related content to dashboard and readme

Adding content and documentation links to support SQL DB public preview

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/jupyter/serverInstance.ts
4
75.5%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
29238

Relevant Lines:
64159

💛  - Coveralls


Error when loading tilemaps in Chrome browser on mobile devices

Hi team, I used your package to render and large tilemap with animated sprites in Flame and it works well on desktop! However, there is an issue when loading on a mobile device.
Here is the the stacktrack:
at Object.throw_ [as throw] (errors.dart:297:3)
at _engine.SurfaceCanvas.__.drawAtlas (canvas.dart:451:5)
at sprite_batch.SpriteBatch.new.render (sprite_batch.dart:378:13)
at orthogonal_tile_layer.OrthogonalTileLayer.new.render (tile_layer.dart:145:16)
at renderable_tile_map.RenderableTiledMap.new.render (renderable_tile_map.dart:414:12)
at image_batch_compiler.ImageBatchCompiler.new.compileMapLayer (image_batch_compiler.dart:29:12)

There is also an inconsistent crash on Chrome in profile mode.
Error: TypeError: null: type 'Null' is not a subtype of type 'NativeUint8List'
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 297:3  throw_
dart-sdk/lib/_internal/js_shared/lib/rti.dart 1385:3                         _failedAsCheck
dart-sdk/lib/_internal/js_shared/lib/rti.dart 1363:3                         _generalAsCheckImplementation
dart-sdk/lib/_internal/js_shared/lib/js_util_patch.dart 110:3                callMethod$
lib/_engine/engine/canvaskit/canvaskit_api.dart 1175:7                       SkImageExtension.readPixels
lib/_engine/engine/canvaskit/picture.dart 108:38                             toImageSync
packages/flame_tiled_utils/src/image_batch_compiler.dart 39:26               compileMapLayer

There is nothing special in library's code to work with web inspector or in profile mode. Example application runs fine:

Same for profile mode...
I just upgraded library to support latest Flame version - try to check if the bug still exists.


[FEAT]: Add Redis cache

Description
Practice what to cache:

Determine key
Determine TTL
Benchmark & compare

Additional Information
No response

Key can use vary decorators
TTL defaults to 15 minutes, defined in settings.py

Does not work yet due to auth


Refactored benchmark tests

Summary

Refactored kernel benchmarking tests to save data to a single CSV (benchmark/data/all_benchmark_data.csv) that contains more complete information on how the benchmarking test was setup:

kernel_name (e.g. swiglu)
kernel_provider (e.g. liger or huggingface)
kernel_operation_mode (e.g. full, forward, backward)
metric_name (e.g. speed, memory)
metric_unit (e.g. MB)
x_name
x_label (e.g. hidden size)
x_value
y_value_50 (median)
y_value_20 (20th percentile)
y_value_80 (80th percnetile)
extra_benchmark_config_str (e.g. {""B"": 32, ""T"": 512, ""D"": 768, ""dtype"": ""torch.float32""})
gpu_name (e.g. NVIDIA A100-SXM4-80GB)
timestamp
liger_version (e.g. 0.2.1)

Removed existing image files to not clutter up the git history per previous discussion (https://github.com/linkedin/Liger-Kernel/pull/54)
Added sample jupyter notebook that can be used to filter and plot the benchmark data
Added makefile command to run all benchmarks: make run-benchmarks
Don't overwrite existing data by default if same benchmark. Same benchmark means same kernel_name, kernel_provider, kernel_operation_mode, metric_name, x_label, gpu_name, and extra_benchmark_config_str

Testing Done

Ran all benchmarks and spot-checked values with existing benchmark data

Hardware Type: A100
[x] run make test to ensure correctness
[x] run make checkstyle to ensure code style
[x] run make test-convergence to ensure convergence

can we add a readme in benchmark/ to elaborate how to run these?

Added instructions to contributing.md

Taking the raw results from triton.testing.do_bench here, we already get the unrounded values. I think the For example you can see in the CSV:
cross_entropy,liger,forward,speed,ms,V,vocab size,8192,0.8101439476013184,0.7565760016441345,0.9144319891929626,"{""B"": 8, ""T"": 2048}",NVIDIA A100-SXM4-80GB,2024-09-03 15:31:39,0.2.1

cc @ByronHsu @austin362667

ah i see! you are writing the csv on your own


Merge Barteguiden-Server with Barteguiden-MarkedsWeb

Todo: Rename repo to Barteguiden

Just tell me when, and I'll rename the repo.


Search based launcher v3 (from f-droid) doesn't see the Google Drive app

The older search based launcher on f-droid didn't have this problem. I use a Nexus 5 running stock 5.1.1 LMY48M. I type 'dr' and it auto launches a separate app, Drive Autosync. Can't get it to start Drive.

I tried toggling off autostart, and then I could type in drive and see it, but then it launched docs, which in turn had a sidebar option to go to Drive, so perhaps Search Based Launcher isn't seeing all the distinct apps.

Strange. Have tried searching for it in list of all activities?

"Drive" sometimes shows up as an app, but immediately launches "Docs", it flashes the name change as soon as I tap Drive, and Docs shows up in the history. Since Docs has an option in the sidebar to get to Drive, it seems like Search Based Launcher isn't seeing both apps and is preferring Docs.

With the update to 45 (from 39) that I just got, this issue seems to be fixed. I'll try to mark the issue closed. Thanks!


no need to get the status of installed packages in this role

Ansible will make sure that the required package(s) will be installed, or throw an error in case of a failure. If run in check mode, the role will report "changed:" instead of missing package(s) and "ok:" in case the package(s) is/are installed.

Solved in pull request #11 .


Remove extra semicolons

https://github.com/realm/realm-cocoa/issues/7221#issue-867086126

@JongchulShin Could you please fill out the CLA.

@ericjordanmossman OK. I filled out the CLA.