requests.exceptions.ConnectionError - when importing data

When executing the import data command I get an connection error, but when I run the same command again, it passes. However, it appears that the application does read in the information from the data.txt.
I don't set VERBA_URL, VERBA_API_KEY
Command that I execute:
verba import --path "/home/snow/Documents/Projects/weaviate-verba/resources/" --model "gpt-3.5-turbo" --clear True
Traceback:
verba import --path "/home/snow/Documents/Projects/weaviate-verba/resources/" --model "gpt-3.5-turbo" --clear True

===================== Creating Document and Chunk class =====================
ℹ Setting up client
ℹ VERBA_URL environment variable not set. Using Weaviate Embedded
Started ./.verba/cache/weaviate-embedded: process ID 200093
{"action":"startup","default_vectorizer_module":"none","level":"info","msg":"the default vectorizer modules is set to \"none\", as a result all new schema classes without an explicit vectorizer setting, will use this vectorizer","time":"2023-09-08T00:11:07+02:00"}
{"action":"startup","auto_schema_enabled":true,"level":"info","msg":"auto schema enabled setting is set to \"true\"","time":"2023-09-08T00:11:07+02:00"}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"cache_d8Z8DfRI0d6H","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:11:07+02:00","took":140330}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"chunk_ZFmXy4onugDM","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:11:07+02:00","took":87042}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"document_0dHcUXJompMJ","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:11:07+02:00","took":90100}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"suggestion_aJSePokVcdxW","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:11:07+02:00","took":104632}
{"level":"warning","msg":"Multiple vector spaces are present, GraphQL Explore and REST API list objects endpoint module include params has been disabled as a result.","time":"2023-09-08T00:11:07+02:00"}
{"action":"grpc_startup","level":"info","msg":"grpc server listening at [::]:50051","time":"2023-09-08T00:11:07+02:00"}
{"action":"restapi_management","level":"info","msg":"Serving weaviate at http://<IP_ADDRESS>:6666","time":"2023-09-08T00:11:07+02:00"}
✔ Client connected to local Weaviate server
Document class already exists, do you want to overwrite it? (y/n): y
{"action":"hnsw_vector_cache_prefill","count":1000,"index_id":"document_at2uxArUgLG1","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:11:11+02:00","took":182906}
{"action":"hnsw_vector_cache_prefill","count":1000,"index_id":"chunk_L4xfAO6z3nSu","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:11:11+02:00","took":170259}
✔ 'Document' and 'Chunk' schemas created
{"action":"restapi_management","level":"info","msg":"Shutting down... ","time":"2023-09-08T00:11:11+02:00"}
{"action":"restapi_management","level":"info","msg":"Stopped serving weaviate at http://<IP_ADDRESS>:6666","time":"2023-09-08T00:11:11+02:00"}
ℹ Done

============================ Creating Cache class ============================
ℹ Setting up client
ℹ VERBA_URL environment variable not set. Using Weaviate Embedded
Started ./.verba/cache/weaviate-embedded: process ID 200235
{"action":"startup","default_vectorizer_module":"none","level":"info","msg":"the default vectorizer modules is set to \"none\", as a result all new schema classes without an explicit vectorizer setting, will use this vectorizer","time":"2023-09-08T00:11:11+02:00"}
{"action":"startup","auto_schema_enabled":true,"level":"info","msg":"auto schema enabled setting is set to \"true\"","time":"2023-09-08T00:11:11+02:00"}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"cache_d8Z8DfRI0d6H","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:11:11+02:00","took":59862}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"chunk_L4xfAO6z3nSu","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:11:11+02:00","took":38679}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"document_at2uxArUgLG1","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:11:11+02:00","took":38062}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"suggestion_aJSePokVcdxW","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:11:11+02:00","took":35334}
{"level":"warning","msg":"Multiple vector spaces are present, GraphQL Explore and REST API list objects endpoint module include params has been disabled as a result.","time":"2023-09-08T00:11:11+02:00"}
{"action":"grpc_startup","level":"info","msg":"grpc server listening at [::]:50051","time":"2023-09-08T00:11:11+02:00"}
{"action":"restapi_management","level":"info","msg":"Serving weaviate at http://<IP_ADDRESS>:6666","time":"2023-09-08T00:11:11+02:00"}
✔ Client connected to local Weaviate server
Cache class already exists, do you want to overwrite it? (y/n): y
{"action":"hnsw_vector_cache_prefill","count":1000,"index_id":"cache_n3X4UDflnGSR","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:11:13+02:00","took":108556}
✔ 'Cache' schema created
{"action":"restapi_management","level":"info","msg":"Shutting down... ","time":"2023-09-08T00:11:13+02:00"}
{"action":"restapi_management","level":"info","msg":"Stopped serving weaviate at http://<IP_ADDRESS>:6666","time":"2023-09-08T00:11:13+02:00"}
ℹ Done

========================= Creating Suggestion class =========================
ℹ Setting up client
ℹ VERBA_URL environment variable not set. Using Weaviate Embedded
Started ./.verba/cache/weaviate-embedded: process ID 200307
{"action":"startup","default_vectorizer_module":"none","level":"info","msg":"the default vectorizer modules is set to \"none\", as a result all new schema classes without an explicit vectorizer setting, will use this vectorizer","time":"2023-09-08T00:11:13+02:00"}
{"action":"startup","auto_schema_enabled":true,"level":"info","msg":"auto schema enabled setting is set to \"true\"","time":"2023-09-08T00:11:13+02:00"}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"cache_n3X4UDflnGSR","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:11:13+02:00","took":56962}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"chunk_L4xfAO6z3nSu","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:11:13+02:00","took":55732}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"document_at2uxArUgLG1","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:11:13+02:00","took":84016}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"suggestion_aJSePokVcdxW","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:11:13+02:00","took":60599}
{"level":"warning","msg":"Multiple vector spaces are present, GraphQL Explore and REST API list objects endpoint module include params has been disabled as a result.","time":"2023-09-08T00:11:13+02:00"}
{"action":"grpc_startup","level":"info","msg":"grpc server listening at [::]:50051","time":"2023-09-08T00:11:13+02:00"}
{"action":"restapi_management","level":"info","msg":"Serving weaviate at http://<IP_ADDRESS>:6666","time":"2023-09-08T00:11:13+02:00"}
✔ Client connected to local Weaviate server
Suggestion class already exists, do you want to overwrite it? (y/n): y
{"action":"hnsw_vector_cache_prefill","count":1000,"index_id":"suggestion_JvWuPHTJUNPN","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:11:14+02:00","took":199752}
✔ 'Suggestion' schema created
{"action":"restapi_management","level":"info","msg":"Shutting down... ","time":"2023-09-08T00:11:14+02:00"}
{"action":"restapi_management","level":"info","msg":"Stopped serving weaviate at http://<IP_ADDRESS>:6666","time":"2023-09-08T00:11:14+02:00"}
ℹ Done

============================ Starting data import ============================
ℹ Setting up client
ℹ VERBA_URL environment variable not set. Using Weaviate Embedded
Started ./.verba/cache/weaviate-embedded: process ID 200378
{"action":"startup","default_vectorizer_module":"none","level":"info","msg":"the default vectorizer modules is set to \"none\", as a result all new schema classes without an explicit vectorizer setting, will use this vectorizer","time":"2023-09-08T00:11:14+02:00"}
{"action":"startup","auto_schema_enabled":true,"level":"info","msg":"auto schema enabled setting is set to \"true\"","time":"2023-09-08T00:11:14+02:00"}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"cache_n3X4UDflnGSR","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:11:14+02:00","took":39691}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"chunk_L4xfAO6z3nSu","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:11:14+02:00","took":54810}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"document_at2uxArUgLG1","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:11:14+02:00","took":34729}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"suggestion_JvWuPHTJUNPN","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:11:14+02:00","took":48080}
{"level":"warning","msg":"Multiple vector spaces are present, GraphQL Explore and REST API list objects endpoint module include params has been disabled as a result.","time":"2023-09-08T00:11:14+02:00"}
{"action":"grpc_startup","level":"info","msg":"grpc server listening at [::]:50051","time":"2023-09-08T00:11:14+02:00"}
{"action":"restapi_management","level":"info","msg":"Serving weaviate at http://<IP_ADDRESS>:6666","time":"2023-09-08T00:11:14+02:00"}
✔ Client connected to local Weaviate server
ℹ All schemas available
ℹ Reading
/home/snow/Documents/Projects/weaviate-verba/resources/data.txt
✔ Loaded 1 files
ℹ Converted
/home/snow/Documents/Projects/weaviate-verba/resources/data.txt
✔ All 1 files successfully loaded
ℹ Starting splitting process
✔ Successful splitting (total 0)
ℹ (1/1) Importing document
/home/snow/Documents/Projects/weaviate-verba/resources/data.txt
✔ Imported all docs
✔ Imported all chunks
{"action":"restapi_management","level":"info","msg":"Shutting down... ","time":"2023-09-08T00:11:14+02:00"}
{"action":"restapi_management","level":"info","msg":"Stopped serving weaviate at http://<IP_ADDRESS>:6666","time":"2023-09-08T00:11:14+02:00"}
Exception in thread batchSizeRefresh:
Traceback (most recent call last):
  File "/home/snow/venv/weaviate-verba/lib/python3.10/site-packages/urllib3/connection.py", line 203, in _new_conn
    sock = connection.create_connection(
  File "/home/snow/venv/weaviate-verba/lib/python3.10/site-packages/urllib3/util/connection.py", line 85, in create_connection
    raise err
  File "/home/snow/venv/weaviate-verba/lib/python3.10/site-packages/urllib3/util/connection.py", line 73, in create_connection
    sock.connect(sa)
ConnectionRefusedError: [Errno 111] Connection refused

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

Traceback (most recent call last):
  File "/home/snow/venv/weaviate-verba/lib/python3.10/site-packages/urllib3/connectionpool.py", line 790, in urlopen
    response = self._make_request(
  File "/home/snow/venv/weaviate-verba/lib/python3.10/site-packages/urllib3/connectionpool.py", line 496, in _make_request
    conn.request(
  File "/home/snow/venv/weaviate-verba/lib/python3.10/site-packages/urllib3/connection.py", line 395, in request
    self.endheaders()
  File "/usr/lib/python3.10/http/client.py", line 1278, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/usr/lib/python3.10/http/client.py", line 1038, in _send_output
    self.send(msg)
  File "/usr/lib/python3.10/http/client.py", line 976, in send
    self.connect()
  File "/home/snow/venv/weaviate-verba/lib/python3.10/site-packages/urllib3/connection.py", line 243, in connect
    self.sock = self._new_conn()
  File "/home/snow/venv/weaviate-verba/lib/python3.10/site-packages/urllib3/connection.py", line 218, in _new_conn
    raise NewConnectionError(
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPConnection object at 0x7f5eadaec370>: Failed to establish a new connection: [Errno 111] Connection refused

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

Traceback (most recent call last):
  File "/home/snow/venv/weaviate-verba/lib/python3.10/site-packages/requests/adapters.py", line 486, in send
    resp = conn.urlopen(
  File "/home/snow/venv/weaviate-verba/lib/python3.10/site-packages/urllib3/connectionpool.py", line 844, in urlopen
    retries = retries.increment(
  File "/home/snow/venv/weaviate-verba/lib/python3.10/site-packages/urllib3/util/retry.py", line 515, in increment
    raise MaxRetryError(_pool, url, reason) from reason  # type: ignore[arg-type]
urllib3.exceptions.MaxRetryError: HTTPConnectionPool(host='localhost', port=6666): Max retries exceeded with url: /v1/nodes (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f5eadaec370>: Failed to establish a new connection: [Errno 111] Connection refused'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/snow/venv/weaviate-verba/lib/python3.10/site-packages/weaviate/cluster/cluster.py", line 62, in get_nodes_status
    response = self._connection.get(path=path)
  File "/home/snow/venv/weaviate-verba/lib/python3.10/site-packages/weaviate/connect/connection.py", line 539, in get
    return self._session.get(
  File "/home/snow/venv/weaviate-verba/lib/python3.10/site-packages/requests/sessions.py", line 602, in get
    return self.request("GET", url, **kwargs)
  File "/home/snow/venv/weaviate-verba/lib/python3.10/site-packages/requests/sessions.py", line 589, in request
    resp = self.send(prep, **send_kwargs)
  File "/home/snow/venv/weaviate-verba/lib/python3.10/site-packages/requests/sessions.py", line 703, in send
    r = adapter.send(request, **kwargs)
  File "/home/snow/venv/weaviate-verba/lib/python3.10/site-packages/requests/adapters.py", line 519, in send
    raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPConnectionPool(host='localhost', port=6666): Max retries exceeded with url: /v1/nodes (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f5eadaec370>: Failed to establish a new connection: [Errno 111] Connection refused'))

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

Traceback (most recent call last):
  File "/usr/lib/python3.10/threading.py", line 1016, in _bootstrap_inner
    self.run()
  File "/usr/lib/python3.10/threading.py", line 953, in run
    self._target(*self._args, **self._kwargs)
  File "/home/snow/venv/weaviate-verba/lib/python3.10/site-packages/weaviate/batch/crud_batch.py", line 442, in periodic_check
    status = cluster.get_nodes_status()
  File "/home/snow/venv/weaviate-verba/lib/python3.10/site-packages/weaviate/cluster/cluster.py", line 64, in get_nodes_status
    raise RequestsConnectionError(
requests.exceptions.ConnectionError: Get nodes status failed due to connection error

after simply repeating the command:
===================== Creating Document and Chunk class =====================
ℹ Setting up client
ℹ VERBA_URL environment variable not set. Using Weaviate Embedded
Started ./.verba/cache/weaviate-embedded: process ID 205223
{"action":"startup","default_vectorizer_module":"none","level":"info","msg":"the default vectorizer modules is set to \"none\", as a result all new schema classes without an explicit vectorizer setting, will use this vectorizer","time":"2023-09-08T00:14:46+02:00"}
{"action":"startup","auto_schema_enabled":true,"level":"info","msg":"auto schema enabled setting is set to \"true\"","time":"2023-09-08T00:14:46+02:00"}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"cache_n3X4UDflnGSR","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:14:46+02:00","took":192734}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"chunk_L4xfAO6z3nSu","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:14:46+02:00","took":180536}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"document_at2uxArUgLG1","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:14:46+02:00","took":261801}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"suggestion_JvWuPHTJUNPN","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:14:46+02:00","took":173080}
{"level":"warning","msg":"Multiple vector spaces are present, GraphQL Explore and REST API list objects endpoint module include params has been disabled as a result.","time":"2023-09-08T00:14:46+02:00"}
{"action":"grpc_startup","level":"info","msg":"grpc server listening at [::]:50051","time":"2023-09-08T00:14:46+02:00"}
{"action":"restapi_management","level":"info","msg":"Serving weaviate at http://<IP_ADDRESS>:6666","time":"2023-09-08T00:14:46+02:00"}
✔ Client connected to local Weaviate server
Document class already exists, do you want to overwrite it? (y/n): y
{"action":"hnsw_vector_cache_prefill","count":1000,"index_id":"document_4KF2RtSb7AzF","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:14:49+02:00","took":50649}
{"action":"hnsw_vector_cache_prefill","count":1000,"index_id":"chunk_afMWBLITi7Hg","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:14:49+02:00","took":36439}
✔ 'Document' and 'Chunk' schemas created
{"action":"restapi_management","level":"info","msg":"Shutting down... ","time":"2023-09-08T00:14:49+02:00"}
{"action":"restapi_management","level":"info","msg":"Stopped serving weaviate at http://<IP_ADDRESS>:6666","time":"2023-09-08T00:14:49+02:00"}
ℹ Done

============================ Creating Cache class ============================
ℹ Setting up client
ℹ VERBA_URL environment variable not set. Using Weaviate Embedded
Started ./.verba/cache/weaviate-embedded: process ID 205337
{"action":"startup","default_vectorizer_module":"none","level":"info","msg":"the default vectorizer modules is set to \"none\", as a result all new schema classes without an explicit vectorizer setting, will use this vectorizer","time":"2023-09-08T00:14:49+02:00"}
{"action":"startup","auto_schema_enabled":true,"level":"info","msg":"auto schema enabled setting is set to \"true\"","time":"2023-09-08T00:14:49+02:00"}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"cache_n3X4UDflnGSR","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:14:49+02:00","took":53770}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"chunk_afMWBLITi7Hg","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:14:49+02:00","took":49957}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"document_4KF2RtSb7AzF","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:14:49+02:00","took":48970}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"suggestion_JvWuPHTJUNPN","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:14:49+02:00","took":51621}
{"level":"warning","msg":"Multiple vector spaces are present, GraphQL Explore and REST API list objects endpoint module include params has been disabled as a result.","time":"2023-09-08T00:14:49+02:00"}
{"action":"grpc_startup","level":"info","msg":"grpc server listening at [::]:50051","time":"2023-09-08T00:14:49+02:00"}
{"action":"restapi_management","level":"info","msg":"Serving weaviate at http://<IP_ADDRESS>:6666","time":"2023-09-08T00:14:49+02:00"}
✔ Client connected to local Weaviate server
Cache class already exists, do you want to overwrite it? (y/n): y
{"action":"hnsw_vector_cache_prefill","count":1000,"index_id":"cache_dE7829wgUrT0","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:14:50+02:00","took":80944}
✔ 'Cache' schema created
{"action":"restapi_management","level":"info","msg":"Shutting down... ","time":"2023-09-08T00:14:50+02:00"}
{"action":"restapi_management","level":"info","msg":"Stopped serving weaviate at http://<IP_ADDRESS>:6666","time":"2023-09-08T00:14:50+02:00"}
ℹ Done

========================= Creating Suggestion class =========================
ℹ Setting up client
ℹ VERBA_URL environment variable not set. Using Weaviate Embedded
Started ./.verba/cache/weaviate-embedded: process ID 205405
{"action":"startup","default_vectorizer_module":"none","level":"info","msg":"the default vectorizer modules is set to \"none\", as a result all new schema classes without an explicit vectorizer setting, will use this vectorizer","time":"2023-09-08T00:14:50+02:00"}
{"action":"startup","auto_schema_enabled":true,"level":"info","msg":"auto schema enabled setting is set to \"true\"","time":"2023-09-08T00:14:50+02:00"}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"cache_dE7829wgUrT0","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:14:50+02:00","took":41406}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"chunk_afMWBLITi7Hg","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:14:50+02:00","took":49932}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"document_4KF2RtSb7AzF","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:14:50+02:00","took":47882}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"suggestion_JvWuPHTJUNPN","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:14:50+02:00","took":49368}
{"level":"warning","msg":"Multiple vector spaces are present, GraphQL Explore and REST API list objects endpoint module include params has been disabled as a result.","time":"2023-09-08T00:14:50+02:00"}
{"action":"grpc_startup","level":"info","msg":"grpc server listening at [::]:50051","time":"2023-09-08T00:14:50+02:00"}
{"action":"restapi_management","level":"info","msg":"Serving weaviate at http://<IP_ADDRESS>:6666","time":"2023-09-08T00:14:50+02:00"}
✔ Client connected to local Weaviate server
Suggestion class already exists, do you want to overwrite it? (y/n): y
{"action":"hnsw_vector_cache_prefill","count":1000,"index_id":"suggestion_nRDqcQr9MEAo","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:14:51+02:00","took":186550}
✔ 'Suggestion' schema created
{"action":"restapi_management","level":"info","msg":"Shutting down... ","time":"2023-09-08T00:14:51+02:00"}
{"action":"restapi_management","level":"info","msg":"Stopped serving weaviate at http://<IP_ADDRESS>:6666","time":"2023-09-08T00:14:51+02:00"}
ℹ Done

============================ Starting data import ============================
ℹ Setting up client
ℹ VERBA_URL environment variable not set. Using Weaviate Embedded
Started ./.verba/cache/weaviate-embedded: process ID 205474
{"action":"startup","default_vectorizer_module":"none","level":"info","msg":"the default vectorizer modules is set to \"none\", as a result all new schema classes without an explicit vectorizer setting, will use this vectorizer","time":"2023-09-08T00:14:51+02:00"}
{"action":"startup","auto_schema_enabled":true,"level":"info","msg":"auto schema enabled setting is set to \"true\"","time":"2023-09-08T00:14:51+02:00"}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"cache_dE7829wgUrT0","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:14:51+02:00","took":196427}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"chunk_afMWBLITi7Hg","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:14:51+02:00","took":272944}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"document_4KF2RtSb7AzF","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:14:51+02:00","took":226739}
{"action":"hnsw_vector_cache_prefill","count":3000,"index_id":"suggestion_nRDqcQr9MEAo","level":"info","limit":1000000000000,"msg":"prefilled vector cache","time":"2023-09-08T00:14:51+02:00","took":157232}
{"level":"warning","msg":"Multiple vector spaces are present, GraphQL Explore and REST API list objects endpoint module include params has been disabled as a result.","time":"2023-09-08T00:14:51+02:00"}
{"action":"grpc_startup","level":"info","msg":"grpc server listening at [::]:50051","time":"2023-09-08T00:14:51+02:00"}
{"action":"restapi_management","level":"info","msg":"Serving weaviate at http://<IP_ADDRESS>:6666","time":"2023-09-08T00:14:51+02:00"}
✔ Client connected to local Weaviate server
ℹ All schemas available
ℹ Reading
/home/snow/Documents/Projects/weaviate-verba/resources/data.txt
✔ Loaded 1 files
ℹ Converted
/home/snow/Documents/Projects/weaviate-verba/resources/data.txt
✔ All 1 files successfully loaded
ℹ Starting splitting process
✔ Successful splitting (total 0)
ℹ (1/1) Importing document
/home/snow/Documents/Projects/weaviate-verba/resources/data.txt
✔ Imported all docs
✔ Imported all chunks
{"action":"restapi_management","level":"info","msg":"Shutting down... ","time":"2023-09-08T00:14:52+02:00"}
{"action":"restapi_management","level":"info","msg":"Stopped serving weaviate at http://<IP_ADDRESS>:6666","time":"2023-09-08T00:14:52+02:00"}

Thanks a lot! We'll look into this

This issue should be fixed with the new 0.3.0 version 🚀  Thanks a lot!


issue123 changes from base repo

Is there an issue this PR needs to be linked to?

Did you run the full comparison like this?

It can reveal minor differences that can be ironed out like this blank line. Also this would prevent the IntentModifier typo.
Please rerun this way.


fix encapsulated item with corresponding ReturnType

@elgonzo
I created a pr, and hope to fix issue #505

Just to make clear, i am not the author nor any of the maintainers of this project, nor associated with any of them. Like you, i am just a user. Therefore, don't take my word for gospel ;-)
And since i have speak with no authority regarding HAP, take from my review suggestions what you like. Or don't take away anything. I won't take no offense if you don't and it's all fine by me. Also, i am only able to visually check the code, so my apologies if i overlook something. Anyways, getting on with the business...
Your pull request is more than solid, imo. The only worthwhile improvement i can see is a code duplication situation that could be avoided by slightly restructuring the code.
There is a third place where ReturnType values are being handled in pretty much the same manner as your GetEncapsulatedHtml method. It's in the nested Tools class:
https://github.com/zzzprojects/html-agility-pack/blob/c12580b2025144d618d3397dec1fd81132e3f822/src/HtmlAgilityPack.Shared/HtmlNode.Encapsulator.cs#L608-L632
To avoid this kind of code duplication, i would put GetEncapsulatedHtml as an internal static method inside the nested Tools class. And also rename it, because what exactly is "encapsulated HTML" supposed to mean? ;-)
Something like this:
    internal static class Tools
    {
        ...

        internal static string GetHtmlForEncapsulation(HtmlNode node, ReturnType returnType)
        {
            switch (returnType)
            {
                case ReturnType.InnerText:
                    return node.InnerText;
                case ReturnType.InnerHtml:
                    return node.InnerHtml;
                case ReturnType.OuterHtml:
                    return node.OuterHtml;
                default:
                    throw new Exception("Unhandled ReturnType: " + returnType.ToString());
            };
        }

        ...

I also switched the method parameters around to maintain the same style/pattern as the other methods in the nested Tools class. (There is no particular reason i didn't make it an extension method. If you feel like it better being an extension method, make it so. It's fine with me either way.)
That would then lead to the two call sites of your GetEncapsulatedHtml methods to be changed from
innerHtmlDocument.LoadHtml(GetEncapsulatedHtml(xPathAttribute.NodeReturnType, htmlNode));

into
innerHtmlDocument.LoadHtml(Tools.GetHtmlForEncapsulation(htmlNode, xPathAttribute.NodeReturnType));

(or node instead of htmlNode at the second call site, respectively)
and this code in the GetNodesValuesBasedOnXPathReturnType method in the nested Tools class
https://github.com/zzzprojects/html-agility-pack/blob/c12580b2025144d618d3397dec1fd81132e3f822/src/HtmlAgilityPack.Shared/HtmlNode.Encapsulator.cs#L658-L690
would be simplified into
        foreach (HtmlNode node in htmlNodeCollection)
        {
            result.Add(
                Convert.ChangeType(
                    GetHtmlForEncapsulation(node, xPathAttribute.NodeReturnType),
                    listGenericType
                )
            );
        }

A closing side note that's a bit tangential and unrelated to your pull request, but i got a bit of a stomach pain regarding the EncapsulatorTests.cs test file. No, your test is fine as far as i can tell. It's more regarding the rest. (My advance apologies to the HAP team, as i will express a strong opinion.) if you look at EncapsulatorTests.cs, the test cases it contains (except your addition) are extremely poorly conceived, and in my honest opinion should just be eliminated (and ideally replaced with actual meaningful, robust and reliable tests). They don't test anything other than the GetEncapsulatedData method returning something that isn't null. There are no tests whether GetEncapsulatedData does actually has populated anything in the object instance it returned. But worse -- and more importantly -- is these tests being utterly unreliable, because they rely on external data from 3rd-party web sites the maintainers of HtmlAgilityPack have exactly zero (let me repeat: zero) control over. Yeah, those 3rd-party web sites one day might just want to "modernize" their web site design/layout, and the tests in EncapsulatorTests.cs might just like that completely fall apart. These are not sane tests.

Hello @rwecho ,
What is your thought about @elgonzo recommendation?
Let me know when this branch will be ready to merge.
Best Regards,
Jon

@elgonzo, thank you for your advice. You have been patient and a powerful mentor. I have taken your suggestion and refactored the other two duplications. I believe it is functioning correctly now, so there is no need to make further changes, thanks to your timely correction and support. I will commit the changes again. If there are any areas that can be further improved, please let me know. Thank you once again, and I appreciate HAP team as well.

@JonathanMagnan  I improved the code structure by elgonzo advice.

@JonathanMagnan please approve this PR if there's nothing that needs to be changed about this topic.

Hello @rwecho
Thank you again for your contribution.
Your code has been released in the v1.11.50
Best Regards,
Jon


[stream] Add metrics

Not planned.
Rely on upper layers to track how much is being sent in aggregate


Test/docs quick start

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 79.91%. Comparing base (5937b88) to head (34a966e).
Report is 44 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #259      +/-   ##
==========================================
+ Coverage   75.75%   79.91%   +4.16%
==========================================
  Files          70       65       -5
  Lines        4615     4864     +249
==========================================
+ Hits         3496     3887     +391
+ Misses       1119      977     -142

Flag
Coverage Δ

python-3.11
79.91% <ø> (+4.24%)
:arrow_up:

python-3.9
?

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Contact onAccepted Handler does not provide contact as parameter

If I register an on accepted handler on a contact c the callback function does not get the contact data as stated in the api documentation (https://github.com/amazon-connect/amazon-connect-streams/blob/master/Documentation.md#contactonaccepted)
Code example:
c.onAccepted(function(contact) {
   console.log(contact);
});

This logs null to the console.
Calling the same callback log function with the onConnecting function (https://github.com/amazon-connect/amazon-connect-streams/blob/master/Documentation.md#contactonconnecting) returns the contact data correctly. Both function signatures are similar.
Not sure if other contact event handlers are affected too.``

You can access the contact event from within the contact event - There should only be one contact with multiple subscriptions.
https://github.com/amazon-connect/amazon-connect-streams/blob/master/cheat-sheet.md
var c;
connect.contact(function (contact) {
    c = contact;

    c.onAccepted(function(ignorethis) {
       console.log(c);
    });
});

Some of the subscription handlers have the contact in the callback attribute and some do not.

@tgxn Thanks for your answer. I already have a way to access the contact in the on accepted handler, that's not my biggest issue.
Due to the documentation bug we lost an hour and nearly implemented a bug in our library, because we tested for some other events that worked. (New users should bump into that)
Would be good to have the right documentation or to implement the handlers in a consistent way. Maybe it's only a bug in the function that calls the event handler.
Do you know which other handlers do not provide the contact in the callback function?

This issue has been resolved since June 25th: https://github.com/amazon-connect/amazon-connect-streams/commit/2b3342b5ada4b3eb1756b096c3a090caefbc9eea#diff-e1d62d8c4965edf0e532dfa19da0891fR641. Please upgrade to the latest version of streams to get this fix.


Variance formulae corrections

Obviously the results of using the equation would change e.g. in the table.


fix: parameters when creating credentials without revocation

When attempting to create some credentials without any kind of revocation configuration, under JS I've noticed that it was not properly handling some optional parameters for creating and processing credentials:

In create_credential, revocationConfiguration was sending a default CredRevInfoStruct, that library was not interpreting as null so it was throwing an error about missing tails file path
Also in create_credential, JS wrapper was not accepting an empty  revocationStatusList. In addition, once solved, Rust part was throwing an 'Invalid object handle' error when trying to load the object as it was mandatory
In process_credential, JS wrapper was not accepting an empty revocationRegistryDefinition

I'm not sure if the solutions implemented in this PR are completely correct, even if they seem to work fine. So please @whalelephant and @blu3beri (as the authors of this revocation stuff in Rust and FFI wrapper in JS) let me know about any suggestion you may have (or feel free to add some commits to this PR or even completely disregard it if it's too bad! :stuck_out_tongue: )

@blu3beri just to make a bigger controversy, I've just added some fixes for optional parameters in CredentialEntryStruct:

timestamp: I had to default it to -1 because otherwise anoncreds would take it as valid
rev_state: Defaulted to 0 as other cases in this PR

So basically the question is how can we define the structures in order to accept these values, or how to find an elegant way to define them.


Calling UICollectionViewCell.canBecomeFocused from within DTCollectionViewDelegate.canFocusItemAt can create an infinite recursion.

Saw an issue where the delegate routine for canFocus could be called recursively, until a crash. Issue was mis-filed over in the DTTableViewManager project (sorry!). https://github.com/DenHeadless/DTTableViewManager/issues/64
When calling the cell's canBecomeFocused routine, it may call the delegate's canFocusItemAt routuine (the routine where this change is). To avoid this, we're removing the call to canBecomeFocused here.

Codecov Report

Merging #31 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master      #31   +/-   ##
=======================================
  Coverage   85.71%   85.71%
=======================================
  Files          16       16
  Lines         889      889
=======================================
  Hits          762      762
  Misses        127      127

I've released this fix in 6.4.1 release. Thanks for contribution!


Pre-Install Procedures

Please add the following at the very beginning of the pre-install procedures
In newer versions of Windows, Group Policy settings for Microsoft Defender are reverted back.
To prevent this, before changing them:
Open Resource Monitor (type resmon.exe in the search box)
Overview
Find MsMpEng.exe in the list
Right-click > Suspend Process

thanks this did help


LwIP memory leak - I posted this to closed issue 1829 so I'm posting it again here (IDFGH-6517)

Environment
Arduino 1.8.16, ESP32 Dev Module

Operating System:     Windows
(Windows only) environment type:
Using an IDE?: Arduino 1.8.16
Power Supply:         USB

Problem Description
LwIP is loosing memory for each TCP connection that arrives to TCP server.
Expected Behavior
Whole heap memory to be regained after 2 minutes.
Actual Behavior
104 bytes of heap gets lost.
Steps to reproduce
Run the code.
Code to reproduce this issue
#include <WiFi.h>
#include <lwip/sockets.h>

void setup () {
  Serial.begin (115200);
  WiFi.begin ("ZTE_C8151C", "*****");
  WiFi.mode (WIFI_STA);
  while (WiFi.localIP ().toString () == "<IP_ADDRESS>") { delay (1000); Serial.printf ("   .\n"); } // wait until we get IP from router
  Serial.printf ("Got IP: %s\n", (char *) WiFi.localIP ().toString ().c_str ());

  // display free memmory twice a minute: freeHeap to check memory leaks, max memory block to check memory fragmentation
  xTaskCreate ([] (void * pvParameters) {
                                          while (true) {
                                            Serial.printf ("--- Free heap: %6lu --- max memory block: %6lu ---\n", (unsigned long) ESP.getFreeHeap (), (unsigned long) heap_caps_get_largest_free_block (MALLOC_CAP_DEFAULT));
                                            delay (30000);
                                          }
                                        }, "memoryTest", 2048, NULL, 1, NULL);

  // a very basic TCP (web) server
  int listeningSocket = -1;
  listeningSocket = socket (PF_INET, SOCK_STREAM, 0);
  if (listeningSocket == -1) {
    Serial.printf ("(listener) socket error: %i\n", errno);
  } else {
    // make address reusable - so we won't have to wait a few minutes in case server will be restarted
    int flag = 1;
    if (setsockopt (listeningSocket, SOL_SOCKET, SO_REUSEADDR, &flag, sizeof (flag)) == -1) {
      Serial.printf ("(listener) setsockopt error: %i\n", errno);
    } else {
     // bind listening socket to IP address and port
     struct sockaddr_in serverAddress;
     memset (&serverAddress, 0, sizeof (struct sockaddr_in));
     serverAddress.sin_family = AF_INET;
     serverAddress.sin_addr.s_addr = inet_addr ("<IP_ADDRESS>");
     serverAddress.sin_port = htons (80);
     if (bind (listeningSocket, (struct sockaddr *) &serverAddress, sizeof (serverAddress)) == -1) {
      Serial.printf ("(listener) bind error: %i\n", errno);
     } else {
       // mark socket as listening socket
       #define BACKLOG 8
       if (listen (listeningSocket, BACKLOG) == -1) {
        Serial.printf ("(listener) listen error: %i\n", errno);
       } else {

        // listener is ready for accepting connections
        Serial.printf ("A very basic TCP (web) server is waiting for new connections ...\n");
        while (true) {

            int connectingSocket;
            struct sockaddr_in connectingAddress;
            socklen_t connectingAddressSize = sizeof (connectingAddress);
            connectingSocket = accept (listeningSocket, (struct sockaddr *) &connectingAddress, &connectingAddressSize);
            if (connectingSocket == -1) {
              Serial.printf ("accept error: %i\n", errno);
            } else {
              Serial.printf ("New connection arrived, sending 404 reply ...\n");
              // don't even read the request, just reply with error 404 and break the connection
              char httpReply [] = "HTTP/1.1 404 Not found\r\nContent-Length:20\r\n\r\nPage does not exist.";
              if (send (connectingSocket, httpReply, strlen (httpReply), 0) == -1) {
                Serial.printf ("send error: %i\n", errno);
              }
              close (connectingSocket);
            }

        } // while accepting connections - for ever

       } // listen
     } // bind
    } // setsockopt
    close (listeningSocket); // never executes
  } // socket

}

void loop () {

}

Debug Logs
.
.
.
Got IP: <IP_ADDRESS>
--- Free heap: 280712 --- max memory block: 113792 ---
A very basic TCP (web) server is waiting for new connections ...
--- Free heap: 279876 --- max memory block: 113792 ---
--- Free heap: 279876 --- max memory block: 113792 ---
--- Free heap: 279876 --- max memory block: 113792 ---
--- Free heap: 279736 --- max memory block: 113792 ---
--- Free heap: 279736 --- max memory block: 113792 ---
--- Free heap: 279736 --- max memory block: 113792 ---
--- Free heap: 279736 --- max memory block: 113792 ---
--- Free heap: 279736 --- max memory block: 113792 ---
New connection arrived, sending 404 reply ...
--- Free heap: 279248 --- max memory block: 113792 ---
--- Free heap: 279248 --- max memory block: 113792 ---
--- Free heap: 279248 --- max memory block: 113792 ---
--- Free heap: 279248 --- max memory block: 113792 ---
--- Free heap: 279492 --- max memory block: 113792 --- <- lost 244 bytes
--- Free heap: 279492 --- max memory block: 113792 ---
--- Free heap: 279492 --- max memory block: 113792 ---
--- Free heap: 279492 --- max memory block: 113792 ---
--- Free heap: 279492 --- max memory block: 113792 ---
New connection arrived, sending 404 reply ...
--- Free heap: 279144 --- max memory block: 113792 ---
--- Free heap: 279144 --- max memory block: 113792 ---
--- Free heap: 279144 --- max memory block: 113792 ---
--- Free heap: 279144 --- max memory block: 113792 ---
--- Free heap: 279388 --- max memory block: 113792 --- <- lost 104 bytes
--- Free heap: 279388 --- max memory block: 113792 ---
--- Free heap: 279388 --- max memory block: 113792 ---
--- Free heap: 277680 --- max memory block: 113792 ---
--- Free heap: 279388 --- max memory block: 113792 ---
--- Free heap: 279388 --- max memory block: 113792 ---
--- Free heap: 279388 --- max memory block: 113792 ---
--- Free heap: 279388 --- max memory block: 113792 ---
New connection arrived, sending 404 reply ...
--- Free heap: 279044 --- max memory block: 113792 ---
--- Free heap: 279044 --- max memory block: 113792 ---
--- Free heap: 279044 --- max memory block: 113792 ---
--- Free heap: 279044 --- max memory block: 113792 ---
--- Free heap: 279284 --- max memory block: 113792 --- <- lost 104 bytes
--- Free heap: 279284 --- max memory block: 113792 ---

@BojanJurca Which IDF version you used.

ESP_IDF_VERSION_MAJOR 3
ESP_IDF_VERSION_MINOR 3
ESP_IDF_VERSION_PATCH 5

@BojanJurca I think you can check this https://github.com/espressif/esp-idf/issues/7916#issuecomment-975232010.

This is it! Thank you. Once all 10 sockets are being used there is no memory leak any more for new connections. It is perfectly fine for me. I'm closing this issue.


Automatic Server URL Discovery

The client should retrieve a server instance from: https://beam.pro/api/v1/interactive/hosts and use that instead of requiring the developer to retrieve one.
I'd like to look for a lightweight HTTP library to include here as most are large and seem wasteful for just one HTTP request.

Is this different then PR 36?

#36 Resolves this, I didn't remember to include the fixes messages to automatically close this. Closing it now :)


Question, not an issue

I have an actor called Party Treasure that is set to the PF1e loot sheet. All PCs have Observer permission to it.
I have one PC who is like an assistant and I want them to be responsible for moving loot from defeated enemies/treasure parcels/etc.
The problem is, I can't figure out a way for a player to be able to drag an item from a defeated enemy loot sheet to the party treasure. And if they use the loot item button, it just goes to their character sheet.

Is there a way I can make this work?

You could enable "dragging" and drag it on another sheet. However, it will duplicate the item and you'll have to remove it from the loot sheet. There is unfortunately no other way. You could also take it as a character and drop it on the party treasure but this means 2x more effort.


"Open File Location" button on measurements

The lack of this button is frustrating even though I know exactly where these files are present. This addition is an important part of keeping Enlighten usable without needing to consult the manual.

Looking at that screenshot, my first thought is that the ▼ is being wasted as a button, and should instead pull-down a menu of less-frequently used options related to the measurement (such as that you describe).

Yes I keep clicking it thinking there will be a context menu


Compress main README.md png

Compress artwork/readme/apps.png

Compressed image has visible artifacts:

What tool did you use?

Oh I didn't notice that. I used TinyJPG's extension.

Should I close this PR?

Feel free to close :). Thanks!
If you will have compressed png without artifacts, you can reopen this PR.


How to delete folder?

Hello guys, I use meteor uploads for image storage and need to delete folder when I'm deleting images. Any ideas?

You need to implement custom function for it, as according to the documentation. You need to implement a custom server call I'm affraid.


Add Celery Flower service for monitoring and management

Description
This PR addresses the issue of adding Celery Flower to the project, as outlined in issue #19. The changes include adding the necessary configurations and components for implementing Celery Flower, which allows for better monitoring and management of Celery tasks.
Changes Made

Added Celery Flower service and configurations to the project
Implemented necessary changes to ensure compatibility with Celery Flower
Updated README for accessing the Celery Flower dashboard

Testing Done

Verified the functionality of Celery Flower locally with and without docker

Related Issue
Resolves #19

@asacristani Can you please review this Pull Request? Thanks.


Joints Angle Exceed Limit with tool_move

Steps to reproduce:
from modules.drawXarm import DrawXarm
import config

port = config.xarm1_port
drawbot = DrawXarm(port)

drawbot.go_position_draw()
drawbot.tool_move('black')
drawbot.tool_move('blue')
drawbot.tool_move('red')
drawbot.tool_move('green')
drawbot.tool_move('undefined_colour')

Expected result:
Rotates the tool head
Actual result:
Controller Error: Joints Angle Exceed Limit

Closing this issue as we are now using roll and pitch for pen change (#47) since it is much faster than tool rotation.


SAP HANA APL - Step 2: Run Recommendation APL function

The SQL statement has two quotes too much.
SET SCHEMA "MOVIELENS";
SET SESSION 'APL_CACHE_SCHEMA' = 'MOVIELENS_USER';
Improvement:
SET SESSION 'APL_CACHE_SCHEMA' = 'MOVIELENS_USER';
Best regards

Thanks for you feedback @nahorgebre
This issue is now fixed


Add errors for multiple tagHelperPrefix directives in the same file

Currently Razor will use the (lexical) first @tagHelperPrefix in each file and will report an error for each subsequent one. This allows views to override an imported @tagHelperPrefix but not to specify it multiple times in the same file.
After some discussion, it's likely that this semantic should be applied to more of our directives or at least applied at a different layer than it is today. I'm logging this issue to follow up with implementing this validation since we're temporarily adding support for the taghelper directives without preserving the current behavior.

05cc4123a75150dd84a352bebde7f805b2f9ae97


In settings page, some fields(eCom Store,OMS) are appearing in English after switching language to Japanese

Current behavior
Currently, On the BOPIS app's setting page, some of the field titles (like eCom Store, OMS) are appearing in English instead of Japanese after switching the language to Japanese.
Expected behavior
eComStore and OMS should be in the Japanese language as well
Steps to reproduce the issue

Can you handle fixing this bug by yourself?

[ ] YES
[x] NO

Environment details

Browser:
OS:
Code Version:

Additional information

@itaru-tokuda can you please help with this?

@appurvajain Any update on this

@Dhiraj1405 Should we close this as settings page has changed and we have multiple pages on which static text translations are missing?


Update subgraphs templates

:clipboard: Add associated issues, tickets, docs URL here.
Overview
Describe what your Pull Request is about in a few sentences.
This updates the subgraphs and the command line used to generate the subgraph.yaml file
Changes
Describe your changes and implementation choices. More details make PRs easier to review.

Update the tool

Testing
Describe how to test your new feature/bug fix and if possible, a step by step guide on how to demo this.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 79.52%. Comparing base (0e11b39) to head (a3f90b1).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #258   +/-   ##
========================================
  Coverage    79.52%   79.52%
========================================
  Files           41       41
  Lines         3287     3287
========================================
  Hits          2614     2614
  Misses         511      511
  Partials       162      162

Flag
Coverage Δ

cannon-go-tests
79.52% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


feat: 6770 new column type switch

Description

Enable New Column type switch in table

Fixes #6770
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Tested locally.
Added test cases.

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:green_circle: Total coverage has increased
// Code coverage diff between base branch:release and head branch: FEATURE/6770-new-column-type-switch

Status
File
% Stmts
% Branch
% Funcs
% Lines

:green_circle:
total
55.14 (0.01)
37.2 (0.03)
34.02 (0.05)
55.64 (0)

:green_circle:
app/client/src/selectors/commentsSelectors.ts
85.25 (1.64)
64.71 (2.95)
73.33 (0)
90.59 (2.35)

:green_circle:
app/client/src/widgets/TableWidget/component/TableStyledWrappers.tsx
87.06 (0.16)
55.22 (0.33)
79.63 (0)
87.06 (0.16)

:red_circle:
app/client/src/widgets/TableWidget/component/TableUtilities.tsx
37.57 (-0.38)
24.79 (0.9)
14.29 (-1.93)
35.93 (-0.25)

:green_circle:
app/client/src/widgets/TableWidget/widget/derived.js
65.74 (7.09)
62.79 (3.03)
50 (4.76)
63.82 (5.18)

:green_circle:
app/client/src/widgets/TableWidget/widget/getTableColumns.tsx
56 (25.57)
23.08 (23.08)
33.33 (33.33)
50 (30)

:red_circle:
app/client/src/widgets/TableWidget/widget/index.tsx
29.03 (-0.85)
9.77 (-0.03)
29.09 (-0.91)
28.57 (-0.97)

:red_circle:
app/client/src/widgets/TableWidget/widget/propertyConfig.ts
90 (-5.74)
100 (0)
86.05 (-7.28)
90 (-5.74)

:green_circle:
app/client/src/widgets/TableWidget/widget/propertyUtils.ts
25.56 (1.13)
8 (0)
20 (0)
24.19 (1.24)

@SatishGandham Actually that is another ticket but since we need some common code to be used we are merging that branch here.

Closing this PR. Feature will be reopened with TableV2.


Add mtimeUpdate parameter

Currently, when copying a file from src to dest with grunt-contrib-copy the last modified time of the file is not updated.
It would be nice to have the ability to force updating this value as to play nice with grunt:newer. See: https://github.com/tschaub/grunt-newer/issues/84
This PR introduces a new option to, mtimeUpdate which, if set to true, will force the copied file's mtime to be set to the current time.

Any reason for closing without discussion?

Hey @MeoMix could you rebase this PR and add documentation for the options under docs/ ?

Sure. I'll try and take a look at it this weekend to ensure it's still necessary/useful. Thanks!

It took me a lot of time to notice that the mtime was not modified and I wanted to know why the Makefile was running the task multiple times. It is quite sad to see that a PR existed to fix this but was not merged more than 4 years later.


[Performance] Investigate performance regression to app open time (reported by e2e system)

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

What performance issue do we need to solve?
The e2e performance test have reported a performance regression:

Name
Duration

App start TTI
1265.941 ms → 1922.656 ms (+656.715 ms, +51.9%) 🔴🔴

App start contentAppeared_To_screenTTI
540.925 ms → 1235.091 ms (+694.166 ms, +128.3%) 🔴🔴

The regressions are testing from commit https://github.com/Expensify/App/commit/a6ff319010c4839a20216b31c2c4d213abcbf218 against version https://github.com/Expensify/App/releases/tag/9.0.43-4
In between those changes must be a regression
What is the impact of this on end-users?
App start time is delayed by ~600ms
List any benchmarks that show the severity of the issue
See e2e tests
Proposed solution (if any)
None yet. We need to find the faulty commit first.
List any benchmarks after implementing the changes to show impacts of the proposed solution (if any)
E2e test results should improve
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Version Number:
Reproducible in staging?:
Reproducible in production?:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:
View all open jobs on Upwork

please assign to me

This is very likely being fixed here:

https://github.com/Expensify/App/pull/50143

Draft PR here, should be ready for review soon

@hannojg @luacmartins Were we able to retest this?


Resolution issue

Hi, it works pretty well, thank you.  However I noticed the final resolution has one row and column less than expected (e.g. 256 in the source will give 511 in the result).  It's a problem to measure PSNR and SSIM.  Is it possible to fix that?  THX

I had this issue as well. I was using an image that was 2560x1440 and the end result was 5119×2879.

I had this issue as well.

I  have modified code ,the problem is
   heightgridLR = np.linspace(0,heightLR-1,heightLR)     widthgridLR = np.linspace(0,widthLR-1,widthLR)     bilinearinterp = interpolate.interp2d(widthgridLR, heightgridLR, grayorigin, kind='linear')     heightgridHR = np.linspace(0,heightLR-1,heightLR*2-1)     widthgridHR = np.linspace(0,widthLR-1,widthLR*2-1)     upscaledLR = bilinearinterp(widthgridHR,heightgridHR)
heightgridLR = np.linspace(0,heightLR,heightLR)
widthgridLR = np.linspace(0,widthLR,widthLR)
bilinearinterp = interpolate.interp2d(widthgridLR, heightgridLR, grayorigin, kind='linear')
heightgridHR = np.linspace(0,heightLR,heightLR2)
widthgridHR = np.linspace(0,widthLR,widthLR2)
upscaledLR = bilinearinterp(widthgridHR, heightgridHR)
the result is right ,but the PSNR is 34,53,but the paper is 35.9 for 2x，Can someone point out some problem?

@excllent123 Thank you for the effort! Quote from the paper:

Bilinear upscaling by a factor of 2 in each axis. There are four types of pixels, denoted by P1-P4, corresponding to the four kernels that are applied during the bilinear interpolation.

So all the P1s from the upscaled image should be identical to the original pixels from the LR image. So that's why the code omits the rightmost column and the bottom row. If you print out the pixel value of the LR image and the upscaled image, you can see that every other pixel of the upscaled image is identical to the original LR.

My suggestion is to pad extra column and row at the right and bottom, so that it still follows that P1-P4 order.
Modified upscaled
[[0.94117647 0.94313725 0.94509804 ... 0.93529412 0.93333333 0.93333333]
 [0.94117647 0.94313725 0.94509804 ... 0.93529412 0.93333333 0.93333333]
 [0.94117647 0.94313725 0.94509804 ... 0.93529412 0.93333333 0.93333333]
 ...
 [0.59215686 0.61176471 0.63137255 ... 0.66960784 0.65686275 0.65686275]
 [0.57254902 0.58235294 0.59215686 ... 0.64313725 0.61568627 0.61568627]
 [0.57254902 0.58235294 0.59215686 ... 0.64313725 0.61568627 0.61568627]]

I pushed a testing version to the dev branch. Try it and tell me if it fits your needs!
@excllent123 Just a guess, try modifying the threshold for angle, strength, coherence in hashkey.py may generate a better PSNR. The original paper didn't mention the actual threshold, so the current threshold is just a trial and error result from me.

@movehand Thanks for you kindly help.

@excllent123 Could you upload the filter on MEGA? I think I'm doing something wrong and Python is only using 1 core of my CPU, making it extremely long and painful to train.

@excllent123 Thank you for the feedback!
Please refer to the contribution page for further information. Basically, a pull request will do.
I haven't get into surveying the actual implementation of CT blending yet, sorry!

@excllent123 If you are interested in blending, take a look at this project. Hope it helps!


ERROR: Failed building wheel for mmcv-full

Hi, I meet 'AssertionError: MMCV==1.3.1 is used but incompatible. Please install mmcv>=1.2.4, <=1.3'.
So I run 'pip install mmcv-full==1.3.0 -f https://download.openmmlab.com/mmcv/dist/cu102/torch1.8.1/index.html' to down my mmcv version.
But i meet error 'ERROR: Failed building wheel for mmcv-full'
And after that i can install all the version of mmcv ~._.~
Please help me. Thanks.

Please use the error template. You can use the newest MMDetection, which should allow MMCV=1.3.1. And you need to check whether the MMCV is compatible with your environment.


Trying to get an invite for eta-lang.slack.com leads to an error on hamsterpad.com

I tried it on Mac OS X El Capitan with both Safari & Chrome and when I click the 'Get My Invite' button, it'll take me to a form where I can create a HamsterPad account.
Can you share the details of your OS/Browser and which region you're accessing from?
Is anyone else able to reproduce this?

when I click the 'Get My Invite' button, it'll take me to a form where I can create a HamsterPad account

For me too. I didn't capture in the GIF the account creation. After you create an account you then get the error:

I am using an hamsterpad account made with my @gmail.com address... could that be the problem?

We have lots of people on the Slack channel right now that don't have email addresses in those domains. We'll try removing that setting and see if that helps.
@Jyothsnasrinivas Can you take a look?

I removed the domains. @filippovitale can you please check now? Make sure you are signing up with the same email you used to sign up for Slack.
https://www.hamsterpad.com/chat/eta-lang

Sorry @Jyothsnasrinivas , same problem.
BTW you removed the @typelead.com and @rediff.com domains. It looks like that right now only email addresses from @google.com are allowed to create an account for: https://eta-lang.slack.com/

Ok. @filippovitale Are you using the same email for Slack and Hamsterpad? And, are you currently on other Slack channels?

Yes, and yes

Ok. @filippovitale I'll notify the Hamsterpad guys regarding this. Others are able to join without any issues. Meanwhile, can you please personal message me your email on Gitter? I'll send you a direct invite to the Eta Slack Channel.

Currently we are unable to reproduce this. Since we manually added you, I'm closing this issue.


Use of hard-coded passwords, MD5, and HTTP

I am a security researcher, who is looking for security smells in Puppet scripts.
I noticed two instances of hard-coded passwords, which are against the best practices
recommended by Common Weakness Enumeration (CWE) [https://cwe.mitre.org/data/definitions/259.html] and also by other security practitioners.
I have added hiera support to mitigate this smell. Feedback is welcome.
Pull request: https://github.com/biemond/biemond-wildfly/pull/249
Here is where I noticed hard-coded passwords: https://github.com/biemond/biemond-wildfly/blob/v0.5.x/manifests/params.pp

I also noticed instances of MD5 uses in one of the Puppet scripts. MD5 is vulnerable to attacks, and should be avoided. The Common Weakness Enumeration organization recommends against usage of weak cryptographic algorithms such as MD5. Reff: https://cwe.mitre.org/data/definitions/327.html.
I suggest the use of SHA512 , which is more secure. Any feedback is appreciated.
Here is where I noticed MD5 use: https://github.com/biemond/biemond-wildfly/blob/master/manifests/config/user.pp

I also found instances where the HTTP protocol is used instead of HTTPS (HTTP with TLS). According to the Common Weakness Enumeration organization this is a security weakness (https://cwe.mitre.org/data/definitions/319.html).  I was wondering why HTTP is used? Is it because of lack of tool support?
I am trying to find out if developers are forced to adopt bad practices due to lack of tool support when it comes to the HTTPS protocol. Maybe it is due to dependency on a resource that uses HTTP?
Any feedback is appreciated.
Source: https://github.com/biemond/biemond-wildfly/blob/master/manifests/init.pp

There is a pull request out there:
https://github.com/biemond/biemond-wildfly/pull/223
that addresses setting up JBoss/Wildfly with HTTPS.

Excellent. Thanks for the feedback. I am closing the issue, but please re-open if further discussion is necessary.


[RR Request]: ToolJet v1.0

Open Source Project name
ToolJet
What is your project?
ToolJet is a low-code platform built on ReactJS and NestJS that can be used to build and deploy internal tools and business applications. The drag and drop application builder can be used to compose the user interface of applications and to set up events such as on click. ToolJet can connect to databases, cloud storages, API endpoints, GraphQL endpoints, and products you love such as Airtable, Google Sheets, Slack, etc.
Version
1.0
Date
February 8, 2020
Description of breaking changes
Here are the changes since the launch of beta ( v0.5.3 ) in June 2021:

Ported ToolJet server from Ruby to Node.js.
Ability to extend ToolJet using plugins.
Comandline tool ( https://www.npmjs.com/package/@tooljet/cli ) to bootstrap plugins and add dependencies.
Started with 7 integrations and now we have 20+ datasource integrations.
Started with 15 widgets and now there are 35+ UI widgets.
Ability to run JavaScript snippets.
Redesigned the application builder and dashboard for better usability.
Auto-save for changes in editor and version control for apps to support multiple environments.
Supports keyboard shortcuts for canvas actions like undo, redo, delete, clone, and more.
Permissions based on user groups.
Realtime collaboration features to communicate with your team members.
Support for deploying on more platforms ( Azure AKS, Google Cloud Run, and more ).
Support for SSO. Google & GitHub SSO on the community edition. Okta, AzureAD, and more on enterprise edition.
Audit logs for enterprise edition users

GitHub Repo
https://github.com/ToolJet/ToolJet/releases/tag/v1.0
Website
https://tooljet.com/
Link to changelog
https://github.com/ToolJet/ToolJet/releases/tag/v1.0
Social media
https://twitter.com/ToolJet/status/1490970429841838080
Anything else to add?
https://blog.tooljet.com/announcing-tooljet-v1/

Hey @mishmanners
Not sure if this is right time to make corrections but I have mistakenly added the like to release in GitHub Repo.
Here's the correct link to repository:
https://github.com/ToolJet/ToolJet

Congrats on being featured in the February 2022 Release Radar 🥳

Read the blog here
Feel free to retweet Mish's tweet featuring you


MWPW-160360 - Decorate standalone text with body default styles

When standalone content is not decorated according to its format or a text style override, the decorate text function will add default body styling like it does with other empty elements.
This will apply default styles to text in consonant blocks that use the shared text decorate functionality.
Resolves: MWPW-160360
Test URLs:

Before: https://main--milo--adobecom.hlx.page/drafts/sarchibeque/format-text?martech=off

After: https://sartxi-mwpw-160360-text--milo--adobecom.hlx.page/drafts/sarchibeque/format-text?martech=off

Before: https://main--milo--adobecom.hlx.page/drafts/sarchibeque/standalone-text?martech=off

After: https://sartxi-mwpw-160360-text--milo--adobecom.hlx.page/drafts/sarchibeque/standalone-text?martech=off

I see units tests and also nala tests are failing. Can you have a look at that @Sartxi ?
For the failing nala tests you might try to purge the cached assets.

I tried purging the cache but it didn't seem to work :(


TNO-2075: Goodbye nest, and each feature to have its own store

That was more work than expected..

Ripping out models that were associated with NEST
Changing all findContent() to findContentWithElasticSearch
Add filter store for each feature to maintain state independent of each other

Still need to:

store returned content so page doesn't have to refetch each time (making another ticket)
publish tno-core

@Fosol - just need to publish but this is ready for review


webmention references in some tests missing?

I'm not sure if it's me not understanding the spec, or whether there's an issue with the tests, but a good number of them seem to be missing their rel="webmention" (or whatever) links. For example, tests 1 & 2. Other tests appear fine.
I looked at them in various browsers, & downloaded them with curl, just in case it was some bizarre browser artifact, but got a consistent confusion.
Apologies in advance if this is me not understanding the tests.
I attach a zip of the test files I downloaded (using windows curl), just in case it's an artifact that only appears in certain circumstances.
rocks.zip

That's the point of the test! ;-)
Try checking the HTTP headers.

There's no link rel="webmention" in some of the headers! I've uploaded what I downloaded.
Ok, I've obviously misunderstood what's meant to be happening. Apologies for wasting your time. :-(

I don't see any example in your comment above. Here's what test 1 returns in the HTTP headers:
HTTP/1.1 200 OK
Server: nginx/1.14.2
Date: Sun, 16 Jun 2019 05:13:58 GMT
Content-Type: text/html; charset=UTF-8
Transfer-Encoding: chunked
Connection: keep-alive
Link: </test/1/webmention>; rel=webmention
Access-Control-Allow-Origin: *
Access-Control-Allow-Methods: GET, POST

You're right, I failed to post the HTML page I downloaded, so have now posted a zip of them all.
I'm clearly attempt to dig up the reference up the wrong way. I wasn't looking at the HTTP headers, just the HTML. Thanks for responding. I'll go work this out.

Sorry for wasting your time.

no problem! Glad you figured it out!


Font locating/loading problem on Archlinux [lime 2.4.9, openfl 3.1.3]

I'm having a weird problem. I tried using a simple OpenFL TextField (that's all the program does) compiled either into Neko or C++ and there was no text rendered. It was on Linux. I then tried on Windows and on Android, and on those OSes it worked, I could see the text rendered. My Linux is an Archlinux. I thought that lime would be tested on Ubuntu as it's the most widely used Linux distribution, and thus tried my simple program on Ubuntu 15.04... and it the text showed up correctly.
I began to wonder about font directories. It seems the default font used by TextField is Times New Roman, and the font directory used by lime is /usr/share/fonts/truetype on Linux (seen here). On Archlinux, the main fonts and Times new Roman are located in /usr/share/fonts/TTF, while on Ubuntu the main fonts are located in/usr/share/fonts/truetype. I thought that it was the problem. However, on Ubuntu, there is no Times New Roman. The font just isn't there, but the program still runs fine and shows the text in the TextField. On the other hand, on Archlinux, I tried making a /usr/share/fonts/truetype directory with all the fonts from /usr/share/fonts/TTF inside, but still no text in the TextField...
When manually loading a font located in the assets directory it always works however. The problem is with the default font only.
What could go wrong other than locating the font ?

see https://github.com/openfl/lime/issues/392 ?

Perhaps we need to consider fonts as a special case. Instead of a system directory, we need a list of system paths. Either that, or we use a smarter system of detecting where the fonts are :wink:
https://github.com/openfl/lime/blob/master/lime/system/System.hx#L34
goes to:
https://github.com/openfl/lime/blob/master/project/src/backend/sdl/SDLSystem.cpp#L132
In OpenFL, the default font is actually "_serif" (similar to "_sans" or "_typewriter"), a special font name that looks for a default serif font for the system:
https://github.com/openfl/openfl/blob/a8c738938a339a77efb0e8b82edb9e40c26aafe3/openfl/_internal/renderer/cairo/CairoTextField.hx#L217
On Linux, we're looking for "FreeSans.ttf" at the moment.

It seems like FreeSans.ttf isn't on Arch by default. I found no font that were installed on both systems right from the start :disappointed: ... I came across the "fc-match" shell utility. It's installed on both systems and fetches the best available font according to a pattern (for example : fc-match serif returns DejaVuSerif.ttf: "DejaVu Serif" "Book"). But that will probably just add unnecessary complexity, and creating a process using sys.io.Process will make a console pop on Windows...
Fonts can really be a mess on Linux. Perhaps the best solution would be to simply change the font directory and extend the font list ?

No tff font on Archlinux by default (and no default install too).
package: https://www.archlinux.org/packages/extra/any/ttf-freefont/
file: /usr/share/fonts/TTF/FreeSans.ttf

Is that package regularly installed?
Perhaps using "TTF" instead of "truetype" in the font path would do it, or perhaps we should use "/usr/share/fonts" only for the font system path, and append those to the font name we're looking for (search "truetype/freefont/FreeSerif.ttf", "TTF/FreeSans.ttf", etc)
Actually, now that I say it out loud, that seems just like what we need, what do you think?

Moving the TTF/truetype part of the path is a good idea, but unfortunately, I fear that ttf-freefont isn't regularly installed, with all I have installed (and I can install loads of packages sometimes) I don't have it installed. Some fonts seem to be installed with Xorg, but they are PCF fonts and not TrueType fonts. Other than that, each graphical interface installs its own choice of fonts...

I wonder if we should consider bundling some free fonts
Do you have other default sans or serif fonts installed, like Arial, or other common faces?

ttf-freefont is not regularly installed. ttf-dejavu is most popular for the default ttf font. But 6 or 7 other fonts could replace it (ttf-dejavu, ttf-freefont, ttf-liberation, ttf-bitstream-vera, ttf-linux-libertine, ttf-droid, ttf-ubuntu-font-family, ttf-oxygen).

I just realized I was worried about a console popping up on Windows, when the issue is on Linux... Silly me. Using sys.io.Process and fc-match it should be possible to get a usable font on absolutely any Linux distribution. If you type fc-list | grep $(fc-match | cut -d: -f1) | cut -d: -f1 in a shell prompt, you should get the path to a font.

I believe Vera Sans and Liberation sans are some good free fonts, for what it's worth.


Move solr import class to sidekiq worker

Create a sidekiq worker that either implements this class: app/services/solr_data_import_service.rb or just contains that code so we can push this to an async process when admins release submissions.

Should create a separate queue for this.


Automated cherry pick of #10445

Cherry pick of #10445 on release-7.0.
/cc  @julmondragon
NONE

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Pull Request successfully merged
SHA: 2201e9695f25ba02ede2080da23e61cca98589cd


Tree View Sample: Ability to display a badge

Hello,
In the tree-view-sample there's a nice example of how to create e.g. a Refresh button on the tree view.
I'd like to add a badge with a number indicating total count of the items that are stored in my tree.
Similar thing is visible on the "Changes" tree view for VCS:

I'm wondering whether it's possible for me to have that too.
I'd appreciate any help.
CC: @sandy081
EDIT: Just found that in the Git extension it's called countBadge:
https://github.com/Microsoft/vscode/blob/master/extensions/git/package.json#L828
Is it exposed somewhere?
Thank you all for the tremendous work!

@kubawolanin No Badges are not supported in Tree views.
Please file any issues or feature requests here


Fix issue #580

Telescope configuration can now be saved without raising an exception. However, an exception is raised in case the folder exists.

I've implemented the write_dir function to Karabo & added it to the telescope write_to_file function.
The only thing I'm not really happy about is that firstly it not a write-to-file (since it's a dir) function (however, changing the API is out of scope for this PR I think) and secondly that we don't do any checks if the overwrite=True flag is set. I think just checking if it ends with a .tm should be sufficient? If you agree, we would have to change some code (each call of Telescope.write_to_file would need a .tm ending)

I've also changed line 475/476 in telescope.py to:
if not os.path.exists(tmp_dir):
    self.write_to_file(tmp_dir)

However, we could also do:
self.write_to_file(tmp_dir, overwrite=True)

instead to get the Note assumption mentioned in the docstring. I think this would be a better choice?


Chinese naming support

When I used the /warps add command, I found that I could only set the name of the teleport I wanted to set to English. Could you add support for Chinese naming for this mod?

The latest discovery, I found that by adding ' ' to the command to add Chinese named warp. For example as shown below:

The warp can be created successfully, so is there a way to create a Chinese-named warp with the ' ' symbol removed?

from what i gather, the warp should be added with the quotes automatically removed:


Add support for privileged mode

- What I did
#1129 was closed with https://github.com/moby/swarmkit/pull/1129#issuecomment-243273764

I think this was a pull request before its time. There will come a day when we add --privileged to swarmkit, but today is not that day.
Closing. Feel free to reopen later on (or right now), if you disagree.

Opening this one for checking if now ~six~ seven years later is right time to go forward with this one.
First step to implement #1030 / https://github.com/moby/moby/issues/24862
- How I did it

Added new option to ContainerSpec
Generated protos
Copied relevant parts from #1129
Added test case.

- How to test it
swarmctl service create --privileged
- Description for the changelog

@dperny PTAL

I think this might be useful as a stopgap for people waiting for apparmor support, see #41371

Having this in the next release of moby/moby would be really helpful and to me this looks simple enough to be merged.

Adding my voice to those who want this merged

@thaJeztah PTAL

Judging from the am amount of upvotes, this feels like something to be addressed. Does this PR make sense for the project?

I think it's worth considering; there are a number of things I personally have been looking into plumbing recently:

--userns=host
--priviledged
seccomp-related options
tmpfs without noexec

As the keen-eyed may have noticed, the road to 23.0.0 has been painful and has sucked up most of the bandwidth of the project. I hope that we'll have a lot more energy to focus on Swarm in general once the release hangover and follow-up patch releases are over, myself.

@neersighted I created this PR with hope that it would be early enough for 23.0.0 (as it would a lot of people happy) but because that was not the case we most probably should set as a target to get whole lost mentioned on https://github.com/moby/moby/issues/25303 to be implemented on swarm mode on next major version as after all it is not that much code which is missing.

@neersighted i must say your comment makes me really happy. Take your time. Such a huge Release after such a long time must be really painful.
I think I speak for everyone interested in swarm when I say that we really appreciate the new energy around swarm. The last thing we want is you guys burning out on this. Let us know how we, the community, can help.

I'm really happy with all this swarm come back! Looking forward to the coming features.

@olljanat mentioned maintainers. #1030
@thaJeztah mentioned he is not a maintainer. #3036
Who is a maintainer? Who is responsible? Who is controlling the fate of Docker Swarm?
It doesn't look like Mirantis is doing much besides blog posts for their paid service.
How to get this seemingly ready pull request merged? What is blocking?

@bluepuma77 people in this thread (@neersighted ) are from Mirantis - so they are on our side :)
As mentioned above, this was a huge release. Let's not stress people out by asking too much right out of the start.
Also, there is a lot of work being done on the technical side for swarm (CI pipelines have been modernized, networking fixes over at moby/moby, PRs are starting to get merged) which no one in their right mind would do if they were to abandon the project.

@thaJeztah and I discussed this PR a bit the other day, but not in depth. There are still some concerns about merging it (everything from design principles to practical -- Swarm was built to be least privilege and to try to abstract away the coupled-to-the-implementation sins of the CLI).
I hope to discuss this in general with the Moby maintainers in a week or two -- step one is getting the SwarmKit backlog (for Mirantis) formally triaged which I am working on. Step two is getting out of the 23.0.0 release hangover which will require at least one point release, maybe two. Step three is compiling the knowledge we have of the lay of the land and historical context of some of the FRs/PRs so we can have a coherent discussion with the other maintainers and not waste people's time.
In any case, asking for updates/pinging here won't make things move faster, and we'll post back here (you might see we do that a lot in moby/moby) with a summary of what we discuss in the maintainer's call.
There's no guarantees this is merged yet (and that is not up to any one person), but time/nothing better becoming available is a strong argument (if weak technically), and it does deserve to be discussed & addressed at the very least.

@neersighted For me it's also about security.
For services that need to access the Docker socket (like reverse proxy Traefik) it makes sense to use something like docker-socket-proxy to make the access more secure, but that needs --privileged to be able to run.
Therefore it would be great to provide --privileged as a Docker Swarm option that can manually be enabled, to improve the overall security of Docker Swarm.

PTAL @neersighted

FYI, I see that #3152 was merged so looks that those who are making decisions want to implement it first.
I know that it is not same than privileged mode but together with cap-add which was added couple of versions ago you can already solve many use cases.

Indeed, that is the intent (to allow more workloads without the sledgehammer). We also discussed this recently in a Moby maintainer's call, but honestly not a lot moved. @dperny do you want to weigh in on if it's time to drop the 'purity' of SwarmKit and stop waiting for capabilities, and potentially allow this?

@dperny do you want to weigh in on if it's time to drop the 'purity' of SwarmKit and stop waiting for capabilities, and potentially allow this?

FYI, this PR have been here for a while so I did close + reopen to trigger CI again. Looks to be still 🟢 so even that is not blocked...

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@48dd893). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #3072   +/-   ##
=========================================
  Coverage          ?   61.86%
=========================================
  Files             ?      155
  Lines             ?    31140
  Branches          ?        0
=========================================
  Hits              ?    19264
  Misses            ?    10331
  Partials          ?     1545

@thaJeztah and I briefly discussed this PR a bit the other day. There are still some concerns about merging it (everything from design principles to practical -- Swarm was built to be least privilege and to try to abstract away the coupled-to-the-implementation sins of the CLI).

@neersighted @thaJeztah @dperny compromise proposal. What it if we add new --swarm-privileged-enabled flag to Docker daemon which defaults to false and allow  this only on nodes where it have been purposely enabled?

Knock knock!

Any news on this? I think that a lot of swarm user are waiting for such feature. Without it some application are just not implementable in swarm.

Closing because Docker maintainers looks to be more favor of getting rid of --privileged than adding support for it to Swarm: https://github.com/moby/moby/issues/47663
#3152 already added support for Seccomp and AppArmor profiles and I just created rebased version of device support in #3186


Dbt core version

Description
Bump dbt-core version to 1.7
resolves #880 #909
Integration tests
Adapter to test:

postgres
snowflake
bigquery
redshift
duckdb
athena

Python version to test:

3.8
3.9
3.10

just to help in testing i have the following error now
`
There's someone who started a PR
Traceback (most recent call last):
File "/usr/local/bin/dbt-fal", line 8, in
sys.exit(cli())
File "/usr/local/lib/python3.10/site-packages/fal/dbt/cli/cli.py", line 17, in cli
_cli(argv)
File "/usr/local/lib/python3.10/site-packages/fal/dbt/telemetry/telemetry.py", line 384, in wrapper
return func(*func_args, **func_kwargs)
File "/usr/local/lib/python3.10/site-packages/fal/dbt/cli/cli.py", line 38, in _cli
fal_run(parsed)
File "/usr/local/lib/python3.10/site-packages/fal/dbt/cli/fal_runner.py", line 43, in fal_run
faldbt = create_fal_dbt(args)
File "/usr/local/lib/python3.10/site-packages/fal/dbt/cli/fal_runner.py", line 20, in create_fal_dbt
return FalDbt(
File "/usr/local/lib/python3.10/site-packages/fal/dbt/utils.py", line 42, in init
super().init(*args, **kwargs)
File "/usr/local/lib/python3.10/site-packages/fal/dbt/integration/project.py", line 491, in init
self.scripts_dir = parse.get_scripts_dir(self.project_dir, args_vars)
File "/usr/local/lib/python3.10/site-packages/fal/dbt/integration/parse.py", line 144, in get_scripts_dir
vars = get_vars_dict(project_dir, args_vars)
File "/usr/local/lib/python3.10/site-packages/fal/dbt/integration/parse.py", line 102, in get_vars_dict
project_contract = load_dbt_project_contract(project_dir)
File "/usr/local/lib/python3.10/site-packages/fal/dbt/integration/parse.py", line 50, in load_dbt_project_contract
partial_project = Project.partial_load(project_dir)
AttributeError: type object 'Project' has no attribute 'partial_load'
`


Ephemeral deploy tweaks

PR Template:
Describe your changes

Ignore local/auto-generated bonfire config
Remove venv in deploy script
Use local config
Add 'Content-Type: application/json' header to example request

Checklist

[ ] Are the agreed upon acceptance criteria fulfilled?

[ ] Was the 4-eye-principle applied? (async PR review, pairing, ensembling)

[ ] Do your changes have passing automated tests and sufficient observability?

[x] Are the work steps you introduced repeatable by others, either through automation or documentation?

[ ] If automation is possible but not done due to other constraints, a ticket to the tech debt sprint is added
[ ] An SOP (Standard Operating Procedure) was created

[ ] The Changes were automatically built, tested, and  - if needed, behind a feature flag - deployed to our production environment. (Please check this when the new deployment is done and you could verify it.)

[ ] Are the agreed upon coding/architectural practices applied?

[ ] Are security needs fullfilled? (e.g. no internal URL)

[ ] Is the corresponding Ticket in the right state? (should be on "review" now, put to done when this change made it to production)

[ ] For changes to the public API / code dependencies: Was the whole team (or a sufficient amount of ppl) able to review?

LGTM - thanks for the updates to the script @coderbydesign


[Dot] [MFMA] Support FP16 output of MFMA dot

This PR adds cast of output tensor to requested data type.

@micmelesse Can you try this PR to see if it resolve the issue in test_core?

Yes, It is fixing the issue


Add a payload verifier for DDR3 and multiple test payloads.

The second commit reduces the verification time of one million ACT/PRE loop from ~24 seconds to ~14 seconds. Still incredibly slow compared to the C++ version, but I think it might be good enough for now.


Enough metadata for codegen?

Hi there!
Looking at the database x86data.js and I was wondering if the file has enough information to generate a proper x86/x64 code generator? (assuming that the /0, ib, /r...etc. have to be "handcoded")... as it looks like you are using it for asmjit (for the generate-XXX.js), I believe that it should be ok, but just want to be sure!
Thanks!

Yes, it can.
However, if you check out the generate-x86.js tool it's pretty complicated to aggregate x86 data in a way to be used by encoder / decoder (the generate tool reads asmjit's DB and patches it on-the-fly). It most likely depends on how fast the encoder / decoder you want and how you want to deal with ambiguities (like which instruction to prefer - modrm/mormr, etc).
As AsmJit is a high performance encoder I still use custom opcode and encoding fields in its instruction database that I created manually in the past - these will be generated by generate-x86.js tool in the future, just haven't had time yet to fix the generator.
Just explore it and tell me what you miss. Asmdb was designed to be a universal database.
Also, you can check the data online here (it's not official yet, work in progress):
https://kobalicek.com/asmgrid

Just explore it and tell me what you miss. Asmdb was designed to be a universal database.

That's amazing, exactly what I have been looking for. I like a lot the idea that the database is encoding things like side effects (RW on registers, flags...etc.) as it makes it well prepared for JIT/register allocator scenarios, instruction analysis/rescheduling...etc.

However, if you check out the generate-x86.js tool it's pretty complicated to aggregate x86 data in a way to be used by encoder / decoder (the generate tool reads asmjit's DB and patches it on-the-fly). It most likely depends on how fast the encoder / decoder you want and how you want to deal with ambiguities (like which instruction to prefer - modrm/modmr, etc).

No problem. I'm evaluating the idea to build a code gen in C#, so I will mostly work from the asmdb data.
Will get back here once I have started to use it to give some feedback/PR if necessary.
Thanks a lot for your work!

Let me know if you face any issues. You can optionally join asmjit/asmjit gitter chat if you need to get some quick info about something.


Migrate to babel@7

Been working on it at /vikr01/electron-react-boilerplate/tree/feature/babel-7.
When running commands from the root directory, everything works fine. When running from the app directory, babel is not finding its config.
Issue seems to be related to babel believing the project is a monorepo due to app/package.json. Switched .babelrc to babel.config.js and still no luck.


Wallet creation cancel

Codecov Report

Merging #1239 (6a29e7d) into master (408751f) will increase coverage by 39.33%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #1239       +/-   ##
===========================================
+ Coverage   37.13%   76.46%   +39.33%
===========================================
  Files         422       37      -385
  Lines        9910     1194     -8716
  Branches     1802      262     -1540
===========================================
- Hits         3680      913     -2767
+ Misses       5932      261     -5671
+ Partials      298       20      -278

Impacted Files
Coverage Δ

src/index.ts
100.00% <0.00%> (ø)

src/blockchains/index.ts
100.00% <0.00%> (ø)

src/components/accounts/ImportMnemonic/index.ts

src/components/accounts/Balance/Balance.tsx

src/components/common/Logo/index.ts

src/settings/actions.ts

src/app/layout/NotificationBar/index.ts

src/transaction/CreateTx/ToField/ToField.tsx

src/transaction/CreateTx/FormFieldWrapper.tsx

src/icons/Warning.tsx

... and 377 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Delete share request and check existing share request

Whenever we create a share request, it cannot be deleted. The API for deletion of share_object does not exist. We need to create a new API that checks if a share object contains shared items and if it does not, then delete the item from the RDS shares table. To be able to use the API we also need to add a button in the frontend, for example next to "Submit"

@dlpzx Hello. May I know more detail about this issue?  I want to take this. Thanks!

@dlpzx Thanks for your comments. In a word, the goal of this feat. is to add a "delete" functionality of share object which allows user to delete share request which does not have shared items. But if the share request has shared items, user need to delete shared items first. Am I right?


updating dependecy libphonenumber-js from ^1.9.6 to ^1.10.12

reason:
doesn't get the right country.
the fix of this library was introduced on this commit:

https://gitlab.com/catamphetamine/libphonenumber-js/-/commit/6743a40c17a685698f656c634dfb9ab1124f47c9

Before:
https://user-images.githubusercontent.com/30967304/184786576-8ab32cb1-5e35-4685-a7c4-586b841f5eb7.mp4
After:
https://user-images.githubusercontent.com/30967304/184788070-4f1c80be-2eb4-447e-af14-bfa960e5567a.mov

Thanks @FauricioValencia!

released along with 6.0.4 and 5.12.0


Support PID / PID file to attach tracepoints to bitcoind

With https://github.com/bitcoin/bitcoin/pull/26593, we will only be able to attach to a bitcoind instance via a PID. The peer-observer-extractor should support this:

either, pass a PID via command line
or, read a bitcoind PID file

Note that the peer-observer-extractor might need to be restarted after bitcoind is restarted as the PID changes.

While implementing PID file reading in e.g. systemd, I noticed it's probably cleaner to support this in the extractor itself. Doing this in #64


Remove default server.cer

Depends on
https://github.com/ManageIQ/manageiq-appliance/pull/119
What
https://access.redhat.com/security/cve/CVE-2016-4457
The custom certificate will get generate at first server start-up by initialize_appliance.sh. Merge only after ManageIQ/manageiq-appliance#119.

Adding fine/no until the ManageIQ/manageiq-appliance#119 is merged. Then, we can backport.


Chat Room

I've been spending a bit of time playing around and deciphering your code so far and it'd be a great to have a Slack or similar chat set up so we can coordinate (and I can ask questions 😄). Let me know if you'd like me to set something up.

Okay, I made a discord server open for anyone to join: https://discord.gg/wjKqZCV. I'll do my best to be helpful.


x/build: darwin-arm64 builders will be unavailable during maintenance period

The darwin builders hosted on host-darwin-arm64-11 and host-darwin-arm64-12 will be unavailable while they are being migrated to a different location. This maintenance is scheduled for 2021-01-17 between 20:00 and 22:00 UTC.
@golang/release

The migration has been completed.


polars generic dataset

Signed-off-by: wmoreiraa<EMAIL_ADDRESS>Description

Add polars.GenericDataSet as discussed in #95
Development notes

Similar to pandas.GenericDataSet, changes made:

Add entry "write_mode" to be able to use formats that polars doesnt provide a write method (e.g.: delta, excel)
Write file permission changed to "wb" from "w".
Added more tests cases for different file formats.

Checklist

[x] Updated the documentation to reflect the code changes
[x] Added a description of this change in the relevant RELEASE.md file
[x] Added tests to cover my changes

Hi @wmoreiraa, thanks a lot for your pull request! 🙌🏽
I didn't look at the code yet, but a few pipelines are failing because some lines are not covered by tests.
On the other hand, I'm spotting a dependencies issue on Python 3.10. pip keeps backtracking until it fails to build an old version of NumPy. On another pull request from a few hours ago it was working and there were some dependency changes introduced here, so it might be related.

Hi @astrojuanlu ! Yeah! Working on it right now!

Ive pushed a commit, but it will still failt at  pip install -r test_requirements.txt and I dont know how to proceed:

If you check: pip freeze from PR few hours ago , pandas and numpy version are not following pandas=~1.3 and its probably the numpy version (1.22.4 doesnt work with pyspark)

@wmoreiraa this is looking really really great. T
angentially I've been thinking of how to help users work with Delta lake without needing Spark and in my research I realised the Pythonic solution uses a rust library anyway! So this is actually our answer to that question as well.
We probably don't need to do it in this PR, but we should look to add the scan_* readers to the scope later on so that users can read things lazily too 💪

@datajoely I think I finally did it... couldn't find another way 🤣


workingServices

Uncommented the default services to be deployed by default

@prasadker,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Please check and merge.


No blank lines before }

Expected Behavior
Run ktlintCheck should successed. There is not new line before {
Observed Behavior
Charter.kt:14:1: Unexpected newline before "{"
Steps to Reproduce
interface Charter
{
    fun method(test: Test)

Your Environment

Version of ktlint used: 9.2.1
Version of Gradle: 6.4.1
Operating System and version: Windows 10

Please tell me what a name have this rule if I want to disable it in .editorconfig

You can see the rule names after adding verbose.set(true) in the ktlint-gradle-plugin configuration, but the rule name is curly-spacing in this case.
Actually the rule behaves correctly, there is a newline before lbrace, which is not allowed according to the styleguide

@romtsn Great! Thank you. Can you tell me how can I disable Redundant curly braces?

@romtsn Hello! Can you tell me how can I disable Redundant curly braces rule ? I have problem with --verbose (I'm sure, this is problem with my environment) and I think this is my last rule which I need to disable.

It's a string-template rule. You can always look in this repo for the error you're getting and associate it with the rule.


New PR to test SignCLA at 2017-11-08 10:33:36.08203165 -0800 PST m=+95.237436256

Body of PR

unsigned-fail-staging: test succeeded


Improve diagnostics for C-style parameter declaration with structural type

Description
New Swift programmers of a C-like language background or polyglot programmers often find themselves mistakenly using the C-style parameter declaration, e.g.
func cStyle(X x) {}

Currently there's a diagnostic emitted for this case:
DiagnosticSpec(message: "expected ':' in parameter", fixIts: ["insert ':'"])

Undeniably this is the most unsurprising diagnostic for anyone except for the beginners.
Notwithstanding, when the type is structural,
func cStyleWithStructuralType(Array.Index i) {}

As structural types such as Array.Index cannot be identifiers, the parser should not recognize Array.Index as an identifier. However, the diagnostics currently emitted are suboptimal,
DiagnosticSpec(message: "expected ':' and type in parameter", fixIts: ["insert ':'"])
DiagnosticSpec(message: "unexpected code '.Index i' in parameter clause")

Applying the fix-it "insert ':'" will result in an undesirable outcome,
func test(Array: <#type#>.Index i)

This issue is also applicable to closure parameters and enum case parameters,
{ (Array.Index i) in }

case cStyleWithStructuralType(Array.Index i)

We should improve diagnostics for the situations above by emitting diagnostics like these,
DiagnosticSpec(message: "expected ':' and type in parameter", fixIts: ["insert ':'"])
DiagnosticSpec(message: "'i' must precede 'Array.Index'", fixIts: ["move 'i' in front of 'Array.index'"])

Synced to Apple’s issue tracker as rdar://132963346