Error: No OpenCL Device Found

I tried to enable GPU support, but ran into an error.
Environment info
Operating System: Ubuntu 16.04
CPU: Intel(R) Xeon(R) CPU E5-2620 v3 @ 2.40GHz
C++/Python/R version: gcc 5.4.0
GPU: Nvidia Titan X
Error Message:
/home/deep/LightGBM/lightgbm: /usr/local/cuda/lib64/libOpenCL.so.1: no version information available (required by /home/deep/LightGBM/lightgbm)
/home/deep/LightGBM/lightgbm: /usr/local/cuda/lib64/libOpenCL.so.1: no version information available (required by /home/deep/LightGBM/lightgbm)
[LightGBM] [Info] Finished loading parameters
[LightGBM] [Info] Loading weights...
[LightGBM] [Info] Loading weights...
[LightGBM] [Info] Finished loading data in 0.092822 seconds
[LightGBM] [Info] Number of positive: 3716, number of negative: 3284
[LightGBM] [Info] This is the GPU trainer!!
[LightGBM] [Info] Total Bins 6143
[LightGBM] [Info] Number of data: 7000, number of used features: 28
Met Exceptions:
No OpenCL device found
Reproducible examples
examples/binary_classification
Steps to reproduce

clone the latest master branch
export LIBOPENCL=/usr/local/cuda/lib; export LIBOPENCL=/usr/local/cuda/lib64
follow the build procedure for GPU support
add device=gpu in examples/binary_classification/train.conf
lightgbm config=train.conf

does anyone know how to fix it? thanks!

I'm having the exact same problem. Did you manage to solve this problem?

So what ended up solving it for me was:
mkdir -p /etc/OpenCL/vendors && \     echo "libnvidia-opencl.so.1" > /etc/OpenCL/vendors/nvidia.icd
I got this from the gpu docker file. Solved my problem, hope it helps!

I face the same question:LightGBMError: No OpenCL device found while using jupyter notebook, I do the following procedure in jupyter notebook terminals:
git clone --recursive https://github.com/Microsoft/LightGBM
cd LightGBM
mkdir build
cd build
cmake -DUSE_GPU=1 -DOpenCL_LIBRARY=/usr/local/cuda-10.0/lib64/libOpenCL.so -DOpenCL_
INCLUDE_DIR=/usr/local/cuda-10.0/include/ ..
make -j$(nproc)
cd ..

cd python-packages/
python setup.py install --gpu --opencl-include-dir=/usr/local/cuda-10.0/include/

and every thing goes fine.
But after I import lightgbm in jupyter and use the
lgbres = lgb.LGBMRegressor(...)
lgbres.fit(train,y_train)

** It goes wrong. Showing:
~/anaconda3/lib/python3.6/site-packages/lightgbm/basic.py in _safe_call(ret)
44         The return value from C API calls.
45     """
---> 46     if ret != 0:
47         raise LightGBMError(decode_string(_LIB.LGBM_GetLastError()))
48
LightGBMError: No OpenCL device found
Could someone help me ? Thanks

I got the same error. This is the solution I follow:
https://www.kaggle.com/kirankunapuli/ieee-fraud-lightgbm-with-gpu

solved my problem too, thanks


Added get_all_schedule_bw_dates & get_next_n_schedule

Added two methods get_all_schedule_bw_dates  and  get_next_n_schedule .
These methods are very useful while using AWS cron schedule. I have added these as I needed this while using AWS personally and I think it will help other people too.
For example...

When we need to verify all executions of some lambda function on AWS based on a cron schedule between some date , we can use get_all_schedule_bw_dates  and compare whether the execution was done or not.

from_dt = datetime.datetime(2021, 8, 7, 8, 30, 57, tzinfo=datetime.timezone.utc)
to_date = datetime.datetime(2021, 8, 7, 11, 30, 57, tzinfo=datetime.timezone.utc)

AWSCron.get_all_schedule_bw_dates(from_dt, to_date, '0/23 * * * ? *')

# Outputs

[datetime.datetime(2021, 8, 7, 8, 46, tzinfo=datetime.timezone.utc),
 datetime.datetime(2021, 8, 7, 9, 0, tzinfo=datetime.timezone.utc),
 datetime.datetime(2021, 8, 7, 9, 23, tzinfo=datetime.timezone.utc),
 datetime.datetime(2021, 8, 7, 9, 46, tzinfo=datetime.timezone.utc),
 datetime.datetime(2021, 8, 7, 10, 0, tzinfo=datetime.timezone.utc),
 datetime.datetime(2021, 8, 7, 10, 23, tzinfo=datetime.timezone.utc),
 datetime.datetime(2021, 8, 7, 10, 46, tzinfo=datetime.timezone.utc),
 datetime.datetime(2021, 8, 7, 11, 0, tzinfo=datetime.timezone.utc),
 datetime.datetime(2021, 8, 7, 11, 23, tzinfo=datetime.timezone.utc)]

Sometimes we have to predict next few(say 5) executions based on a cron pattern. In this case we can use get_next_n_schedule  and pass n according to our requirements.

from_dt = datetime.datetime(2021, 8, 7, 8, 30, 57, tzinfo=datetime.timezone.utc)

AWSCron.get_next_n_schedule(10, from_dt, '0/23 * * * ? *')

#Outputs
[datetime.datetime(2021, 8, 7, 8, 46, tzinfo=datetime.timezone.utc),
 datetime.datetime(2021, 8, 7, 9, 0, tzinfo=datetime.timezone.utc),
 datetime.datetime(2021, 8, 7, 9, 23, tzinfo=datetime.timezone.utc),
 datetime.datetime(2021, 8, 7, 9, 46, tzinfo=datetime.timezone.utc),
 datetime.datetime(2021, 8, 7, 10, 0, tzinfo=datetime.timezone.utc),
 datetime.datetime(2021, 8, 7, 10, 23, tzinfo=datetime.timezone.utc),
 datetime.datetime(2021, 8, 7, 10, 46, tzinfo=datetime.timezone.utc),
 datetime.datetime(2021, 8, 7, 11, 0, tzinfo=datetime.timezone.utc),
 datetime.datetime(2021, 8, 7, 11, 23, tzinfo=datetime.timezone.utc),
 datetime.datetime(2021, 8, 7, 11, 46, tzinfo=datetime.timezone.utc)]

Fixed README typo.
I was literally scratching my head while trying to use this library after installing from PyPy.
Had to finally get through the source code to understand and use the correct method.

Updated README with new methods usage.


Xaml and other resources are not embeded into Microsoft.PowerShell.GraphicalHost and Microsoft.PowerShell.Activities.dll

Steps to reproduce

Build full PS in Github.
Open the generated Microsoft.PowerShell.Activities.dll and Microsoft.PowerShell.GraphicalHost.dll in ILSpy.exe.
Open the Microsoft.PowerShell.Activities.dll and Microsoft.PowerShell.GraphicalHost.dll from the GAC.
Compare the items in "Resources" folder that are displayed in ILSpy.exe.

Expected behavior
You will see for each of the DLLs from GAC, there is a "*.g.resources" item. Those resources are expected to be embedded in the DLLs produced in Github.
Actual behavior
Those resources items are missing from the DLLs produced in Github.
Environment data
PS:122> dotnet --info
.NET Command Line Tools (1.0.0-preview2-003119)
Product Information:
Version:            1.0.0-preview2-003119
Commit SHA-1 hash:  0708fe095e
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.10586
OS Platform: Windows
RID:         win10-x64

Been testing my XAML tools in PowerShell on Linux today and finding that I can't instantiate a XML NodeReader Object.

@vors Graphical host and WorkFlows was removed from the repo.


Conda publish as a github action

Once we have an action publishing to PyPi (https://github.com/PixarAnimationStudios/OpenTimelineIO/issues/988), we should create an action to publish to Conda-forge.
For reference - https://valerioviperino.me/conda/for-pipeline-tds

@jminor @meshula is this still an open issue? I'd like to try to work on this. Saw your presentation at tech talk and with Evan Goldberg. I'd be newbie to the project.

I don't think anyone's done any work on getting OTIO available via conda ~ so if you'd like to take it on, that would be welcome!

I'm curious, should the conda package provide both the python bindings and the C++ library or it should be two separate packages? How is this normally handled in conda?

It's inconsistent, and even when C++ libraries are available, the packaging is often incomplete. For example, boost python is in conda, and the C++ linking libraries are distributed, but the headers are not, the last time I checked.
As with pypi, I recommend that we don't attempt to distribute the C++ bindings via conda, and that if people need them, they should build from a corresponding tag on this github repo.


[Linux] rnv new fails on "No project found"

Describe the bug
Cannot use rnv new to create a new project, it fails on "No project found" in the newly created directory.
To Reproduce

Run rnv new
Use any template

Expected behavior
It should create a new project..
Screenshots

Desktop

OS: Ubuntu 22.04
Node Version: v16.16.0
RNV Version: 0.36.1

@TheDechev thanks for reporting this issue, is this still reproduced on latest version?

I can't reproduce it on version 0.37.1 (OS: Ubuntu 22.04.2, Node v18.17.0). Please update version and try again

Closing as solved


Bad hooks not failing the plan properly. The plan also won't work again until cleanup is done.

https://github.com/konveyor/mig-controller/pull/518
https://bugzilla.redhat.com/show_bug.cgi?id=1829390
An alternate failure scenario was discovered and is handled here in the PR. Added an owner label to correlate jobs/configmaps to a specific migmigration so that failed jobs won't prevent a rerun from working correctly.

Closed by https://github.com/konveyor/mig-controller/pull/518


Can't connect to electrs

I have an electrum rust server running on my raspiblitz (latest release). I was able to get my Wasabi wallet to connect to it, so I know it's up and running, but Sparrow can't connect. I've toggled SSL on/off and used the tor proxy localhost:9050 with Tor browser open and also toggling it off. The log below has me testing a few different combinations but the most common one was SSL toggled on and using the Tor proxy and it would return a "can't connect" or "socket closed" exception.
2021-10-14 16:37:44,530 ERROR [JavaFX Application Thread] c.s.s.p.ServerPreferencesController [null:-1] Connection error
com.sparrowwallet.sparrow.net.TorServerAlreadyBoundException: Tor server already bound
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.TorService$1.call(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.TorService$1.call(Unknown Source)
at javafx.graphics/javafx.concurrent.Task$TaskCallable.call(Unknown Source)
at java.base/java.util.concurrent.FutureTask.run(Unknown Source)
at javafx.graphics/javafx.concurrent.Service.lambda$executeTask$6(Unknown Source)
at java.base/java.security.AccessController.doPrivileged(Unknown Source)
at javafx.graphics/javafx.concurrent.Service.lambda$executeTask$7(Unknown Source)
at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
at java.base/java.lang.Thread.run(Unknown Source)
Caused by: net.freehaven.tor.control.TorControlError: Error reply: Unable to set option: Failed to bind one of the listener ports.
at com.github.JesusMcCloud.jtorctl@1.5/net.freehaven.tor.control.TorControlConnection.sendAndWaitForResponse(Unknown Source)
at com.github.JesusMcCloud.jtorctl@1.5/net.freehaven.tor.control.TorControlConnection.setConf(Unknown Source)
at com.github.JesusMcCloud.jtorctl@1.5/net.freehaven.tor.control.TorControlConnection.setConf(Unknown Source)
at netlayer.jpms@0.6.8/org.berndpruenster.netlayer.tor.TorController.enableNetwork(Unknown Source)
at netlayer.jpms@0.6.8/org.berndpruenster.netlayer.tor.NativeTor.(Unknown Source)
at netlayer.jpms@0.6.8/org.berndpruenster.netlayer.tor.NativeTor.(Unknown Source)
at netlayer.jpms@0.6.8/org.berndpruenster.netlayer.tor.NativeTor.(Unknown Source)
... 10 common frames omitted
2021-10-14 16:39:56,062 ERROR [ElectrumServerReadThread] c.s.s.n.TcpTransport [null:-1] Error opening socket inputstream
java.net.SocketException: Socket is closed
at java.base/sun.security.ssl.SSLSocketImpl.getInputStream(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.TcpTransport.readInputLoop(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.ElectrumServer$ReadRunnable.run(Unknown Source)
at java.base/java.lang.Thread.run(Unknown Source)
2021-10-14 16:40:03,556 ERROR [ElectrumServerReadThread] c.s.s.n.TcpTransport [null:-1] Error opening socket inputstream
java.net.SocketException: Socket is closed
at java.base/sun.security.ssl.SSLSocketImpl.getInputStream(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.TcpTransport.readInputLoop(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.ElectrumServer$ReadRunnable.run(Unknown Source)
at java.base/java.lang.Thread.run(Unknown Source)
2021-10-14 16:40:50,632 ERROR [ElectrumServerReadThread] c.s.s.n.TcpTransport [null:-1] Error opening socket inputstream
java.net.SocketException: Socket is closed
at java.base/sun.security.ssl.SSLSocketImpl.getInputStream(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.TcpTransport.readInputLoop(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.ElectrumServer$ReadRunnable.run(Unknown Source)
at java.base/java.lang.Thread.run(Unknown Source)
2021-10-14 16:40:50,635 ERROR [ElectrumServerReadThread] c.s.s.n.TcpTransport [null:-1] Error opening socket inputstream
java.net.SocketException: Socket is closed
at java.base/sun.security.ssl.SSLSocketImpl.getInputStream(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.TcpTransport.readInputLoop(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.ElectrumServer$ReadRunnable.run(Unknown Source)
at java.base/java.lang.Thread.run(Unknown Source)
2021-10-14 18:55:51,463 ERROR [ElectrumServerReadThread] c.s.s.n.TcpTransport [null:-1] Error opening socket inputstream
java.net.SocketException: Socket is closed
at java.base/sun.security.ssl.SSLSocketImpl.getInputStream(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.TcpTransport.readInputLoop(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.ElectrumServer$ReadRunnable.run(Unknown Source)
at java.base/java.lang.Thread.run(Unknown Source)
2021-10-14 18:56:06,719 ERROR [JavaFX Application Thread] c.s.s.p.ServerPreferencesController [null:-1] Connection error
com.sparrowwallet.sparrow.net.ElectrumServerRpcException: Error getting server version
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.SimpleElectrumServerRpc.getServerVersion(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.ElectrumServer.getServerVersion(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.ElectrumServer$ConnectionService$1.call(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.ElectrumServer$ConnectionService$1.call(Unknown Source)
at javafx.graphics/javafx.concurrent.Task$TaskCallable.call(Unknown Source)
at java.base/java.util.concurrent.FutureTask.run(Unknown Source)
at javafx.graphics/javafx.concurrent.Service.lambda$executeTask$6(Unknown Source)
at java.base/java.security.AccessController.doPrivileged(Unknown Source)
at javafx.graphics/javafx.concurrent.Service.lambda$executeTask$7(Unknown Source)
at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
at java.base/java.lang.Thread.run(Unknown Source)
Caused by: com.sparrowwallet.sparrow.net.ServerException: Retries exhausted
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.RetryLogic.getResult(Unknown Source)
... 12 common frames omitted
Caused by: java.lang.IllegalStateException: I/O error during a request processing
at simple.json.rpc.client@1.0/com.github.arteam.simplejsonrpc.client.builder.RequestBuilder.executeRequest(Unknown Source)
at simple.json.rpc.client@1.0/com.github.arteam.simplejsonrpc.client.builder.RequestBuilder.executeAndConvert(Unknown Source)
at simple.json.rpc.client@1.0/com.github.arteam.simplejsonrpc.client.builder.RequestBuilder.execute(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.SimpleElectrumServerRpc.lambda$getServerVersion$1(Unknown Source)
... 13 common frames omitted
Caused by: java.io.IOException: Error reading response: Could not connect to server at tcp://e6hlbtx64xsjo4ybutr7gbtr4r3fgrivopucwnh22luuwrv4say3klyd.onion:50002
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.TcpTransport.readResponse(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.TcpTransport.pass(Unknown Source)
... 17 common frames omitted
Caused by: java.io.IOException: Could not connect to server at tcp://e6hlbtx64xsjo4ybutr7gbtr4r3fgrivopucwnh22luuwrv4say3klyd.onion:50002
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.TcpTransport.readInputStream(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.TcpTransport.readInputLoop(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.ElectrumServer$ReadRunnable.run(Unknown Source)
... 1 common frames omitted
2021-10-14 18:56:09,912 ERROR [ElectrumServerReadThread] c.s.s.n.TcpTransport [null:-1] Error opening socket inputstream
java.net.SocketException: Socket is closed
at java.base/sun.security.ssl.SSLSocketImpl.getInputStream(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.TcpTransport.readInputLoop(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.ElectrumServer$ReadRunnable.run(Unknown Source)
at java.base/java.lang.Thread.run(Unknown Source)
2021-10-14 18:58:14,700 ERROR [ElectrumServerReadThread] c.s.s.n.TcpTransport [null:-1] Error opening socket inputstream
java.net.SocketException: Socket is closed
at java.base/sun.security.ssl.SSLSocketImpl.getInputStream(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.TcpTransport.readInputLoop(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.ElectrumServer$ReadRunnable.run(Unknown Source)
at java.base/java.lang.Thread.run(Unknown Source)
2021-10-14 18:58:19,596 ERROR [ElectrumServerReadThread] c.s.s.n.TcpTransport [null:-1] Error opening socket inputstream
java.net.SocketException: Socket is closed
at java.base/sun.security.ssl.SSLSocketImpl.getInputStream(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.TcpTransport.readInputLoop(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.ElectrumServer$ReadRunnable.run(Unknown Source)
at java.base/java.lang.Thread.run(Unknown Source)
2021-10-14 18:58:57,934 ERROR [ElectrumServerReadThread] c.s.s.n.TcpTransport [null:-1] Error opening socket inputstream
java.net.SocketException: Socket is closed
at java.base/sun.security.ssl.SSLSocketImpl.getInputStream(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.TcpTransport.readInputLoop(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.ElectrumServer$ReadRunnable.run(Unknown Source)
at java.base/java.lang.Thread.run(Unknown Source)
2021-10-14 19:00:22,613 ERROR [ElectrumServerReadThread] c.s.s.n.TcpTransport [null:-1] Error opening socket inputstream
java.net.SocketException: Socket is closed
at java.base/sun.security.ssl.SSLSocketImpl.getInputStream(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.TcpTransport.readInputLoop(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.ElectrumServer$ReadRunnable.run(Unknown Source)
at java.base/java.lang.Thread.run(Unknown Source)
2021-10-14 19:03:09,795 ERROR [ElectrumServerReadThread] c.s.s.n.TcpTransport [null:-1] Error opening socket inputstream
java.net.SocketException: Socket is closed
at java.base/sun.security.ssl.SSLSocketImpl.getInputStream(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.TcpTransport.readInputLoop(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.ElectrumServer$ReadRunnable.run(Unknown Source)
at java.base/java.lang.Thread.run(Unknown Source)
2021-10-14 19:06:05,198 ERROR [ElectrumServerReadThread] c.s.s.n.TcpTransport [null:-1] Error opening socket inputstream
java.net.SocketException: Socket is closed
at java.base/java.net.Socket.getInputStream(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.TcpTransport.readInputLoop(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.ElectrumServer$ReadRunnable.run(Unknown Source)
at java.base/java.lang.Thread.run(Unknown Source)
2021-10-14 19:07:16,787 ERROR [ElectrumServerReadThread] c.s.s.n.ElectrumServer [null:-1] Uncaught error in ConnectionService
java.lang.NullPointerException: Cannot read field "in" because "this.proxy" is null
at com.github.ravn.jsocks@1.0/com.runjva.sourceforge.jsocks.protocol.SocksSocket.getInputStream(Unknown Source)
at netlayer.jpms@0.6.8/org.berndpruenster.netlayer.tor.TorSocket.getInputStream(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.TcpTransport.readInputLoop(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.ElectrumServer$ReadRunnable.run(Unknown Source)
at java.base/java.lang.Thread.run(Unknown Source)
2021-10-14 19:08:33,762 ERROR [Thread-16] c.s.s.n.VersionCheckService [null:-1] Error retrieving version check file
java.net.SocketException: Connection refused: connect
at java.base/java.net.SocksSocketImpl.connect(Unknown Source)
at java.base/java.net.Socket.connect(Unknown Source)
at java.base/java.net.Socket.connect(Unknown Source)
at java.base/sun.net.NetworkClient.doConnect(Unknown Source)
at java.base/sun.net.www.http.HttpClient.openServer(Unknown Source)
at java.base/sun.net.www.http.HttpClient.openServer(Unknown Source)
at java.base/sun.net.www.protocol.https.HttpsClient.(Unknown Source)
at java.base/sun.net.www.protocol.https.HttpsClient.New(Unknown Source)
at java.base/sun.net.www.protocol.https.AbstractDelegateHttpsURLConnection.getNewHttpClient(Unknown Source)
at java.base/sun.net.www.protocol.http.HttpURLConnection.plainConnect0(Unknown Source)
at java.base/sun.net.www.protocol.http.HttpURLConnection.plainConnect(Unknown Source)
at java.base/sun.net.www.protocol.https.AbstractDelegateHttpsURLConnection.connect(Unknown Source)
at java.base/sun.net.www.protocol.http.HttpURLConnection.getInputStream0(Unknown Source)
at java.base/sun.net.www.protocol.http.HttpURLConnection.getInputStream(Unknown Source)
at java.base/sun.net.www.protocol.https.HttpsURLConnectionImpl.getInputStream(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.VersionCheckService.getVersionCheck(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.VersionCheckService$1.call(Unknown Source)
at com.sparrowwallet.sparrow/com.sparrowwallet.sparrow.net.VersionCheckService$1.call(Unknown Source)
at javafx.graphics/javafx.concurrent.Task$TaskCallable.call(Unknown Source)
at java.base/java.util.concurrent.FutureTask.run(Unknown Source)
at javafx.graphics/javafx.concurrent.Service.lambda$executeTask$6(Unknown Source)
at java.base/java.security.AccessController.doPrivileged(Unknown Source)
at javafx.graphics/javafx.concurrent.Service.lambda$executeTask$7(Unknown Source)
at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
at java.base/java.lang.Thread.run(Unknown Source)

A few points here:

Wasabi doesn't use Electrs AFAIK
SSL should probably be off if you are connecting with Tor
The Tor Browser proxy runs on port 9150
If SSL is off for the Tor address, the correct port to use for the onion URL is 50001.


Adoption of DSGP 1

DSGP 1 (#21), is now on Pending status. This is the issue to track the acceptance process of the proposal.
TODO:

Change the DSGP 1 status to Approved.
Apply the changes in our parser. This would mean a breaking change. However, we can do it.

Also, this issue has a precedent at #18.

#25 should close this in the future.


Error while installing tt-flash : "Maturin failed"

I am trying to setup an n300 card locally.
I encountered the following error while installing tt-flash.
(.venv) TT:~$ git clone https://github.com/tenstorrent/tt-flash.git
Cloning into 'tt-flash'...
remote: Enumerating objects: 322, done.
remote: Counting objects: 100% (78/78), done.
remote: Compressing objects: 100% (30/30), done.
remote: Total 322 (delta 65), reused 48 (delta 48), pack-reused 244 (from 1)
Receiving objects: 100% (322/322), 105.60 KiB | 2.58 MiB/s, done.
Resolving deltas: 100% (193/193), done.
(.venv) TT:~$ cd tt-flash
(.venv) TT:~/tt-flash$ pip install .
Looking in indexes: https://pypi.org/simple, https://download.pytorch.org/whl/cpu
Processing /home/tt-flash
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
Collecting pyluwen<EMAIL_ADDRESS>(from tt-flash==3.1.0)
  Cloning https://github.com/tenstorrent/luwen.git (to revision v0.4.6) to /tmp/pip-install-rclcb706/pyluwen_a2550f41c4f74e1b87db80e3b7d2539a
  Running command git clone --filter=blob:none --quiet https://github.com/tenstorrent/luwen.git /tmp/pip-install-rclcb706/pyluwen_a2550f41c4f74e1b87db80e3b7d2539a
  Running command git checkout -q 5cf4cc6545b7d9d92f167417faf39822ed88dbf7
  Resolved https://github.com/tenstorrent/luwen.git to commit 5cf4cc6545b7d9d92f167417faf39822ed88dbf7
  Running command git submodule update --init --recursive -q
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... error
  error: subprocess-exited-with-error

  × Preparing metadata (pyproject.toml) did not run successfully.
  │ exit code: 1
  ╰─> [12 lines of output]
          Updating crates.io index
           Locking 191 packages to latest compatible versions
            Adding bitfield-struct v0.6.2 (available: v0.10.0)
            Adding memmap2 v0.7.1 (available: v0.9.5)
            Adding nix v0.26.4 (available: v0.29.0)
            Adding pyo3 v0.19.2 (available: v0.23.3)
            Adding thiserror v1.0.69 (available: v2.0.9)
      💥 maturin failed
        Caused by: `project.version` field is required in pyproject.toml unless it is present in the `project.dynamic` list
      Error running maturin: Command '['maturin', 'pep517', 'write-dist-info', '--metadata-directory', '/tmp/pip-modern-metadata-1oimamkk', '--interpreter', '/home/.venv/bin/python3']' returned non-zero exit status 1.
      Checking for Rust toolchain....
      Running `maturin pep517 write-dist-info --metadata-directory /tmp/pip-modern-metadata-1oimamkk --interpreter /home/.venv/bin/python3`
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.
(.venv) TT:~/tt-flash$

Device: n300
OS: ubuntu-20.04
Please let me know if I missed something.

I can replicate this on Ubuntu-22, I suspect it's the maturin dependency and how it's defined. I can reproduce with the following:
git clone https://github.com/tenstorrent/tt-flash
cd tt-flash

sudo apt install python3.10-venv

python3 -m venv venv
source venv/bin/activate

pip install .

I can also reproduce using pip install git+https://github.com/tenstorrent/tt-flash.git as referenced in the README.md

Looks like the fix is started in luwen
While waiting for how Tenstorrent wants to fix it, you can change this in pyproject.toml to drop the version and just use tip:
'
"pyluwen<EMAIL_ADDRESS>to
"pyluwen @ git+https://github.com/tenstorrent/luwen.git#subdirectory=crates/pyluwen",
`
While including other changes, it seems to work.


Release v1.0.1

Reviewers: @fractaltheory
Changes

Fix for refresh method
Removes S3 release steps

TODOs:

[ ] +1
[x] Updated README
[x] Updated CHANGELOG

How To Test

Smoke test scooch demo page

Just need to update the build file and we're golden 👍


Fix glitch filter for win7 d3d

Thanks to @rtalon83 I found that we just cant use texture2D inside one of if branches.
Win7 and Win8.1 d3d doesnt like that.

Fixes #190

Example: https://www.pixiplayground.com/#/edit/kvw_c9yo_sM5bsgHmQEwb

Yo! @finscn make sure that you know this stuff

Example tested in:
Windows 7
Chrome Versión 74.0.3729.131 (Build oficial) (64 bits)
NVIDIA GeForce 550 Ti
It's OK!

@rtalon83 released patch in 3.0.2


Reviewing the Spanish translation YAML file

Review of the Spanish i18n YAML file.
This PR tries to normalize Spanish translation strings by using consistent verbs, nouns, and expressions over the whole site. The Castilian Spanish variety has been used. Some decisions have been made regarding ambiguities related to gender and context, namely:

Proxies: it wasn't clear whether proxies meant representatives or agents or some sort of computer-related term. I kept proxy as it is, usually understood in Spanish as the computational term
Your vs My: Since its use was not consistent, I kept my as in My Works
Usted vs tu: all strings now assume the treatment of respect in Spanish, which is gender neutral
Cargar vs subir: to mean upload, both words work just fine. However, in some cases it sounds more natural to say cargar or descargar (download), in others subir or bajar (download). I tried to make it sound solid and natural
Agregar vs Añadir: both mean to add, but Añadir sounds more in line with the Castilian variety
Clitics: in some cases, like You do not have sufficient privileges to edit them., is not clear whether edit refers to a masculine or feminine word in Spanish. Since it is necessary to add the clitic that encodes the gender, I assumed masculine when not specified or unable to see, since in Spanish the masculine also works as the general gender neutral form in most cases.

Wow, this is awesome! Thank you, @versae!
Would you mind signing a Contributor License Agreement for the Samvera (formerly "Hydra") community? You can read more about our CLA processes here:
https://wiki.duraspace.org/display/hydra/Samvera+Community+Intellectual+Property+Licensing+and+Ownership
The process itself is fairly simple -- you just need to print out the "Hydra Project iCLA," sign it, and send it to<EMAIL_ADDRESS>I can turn that around pretty quickly once it's been sent. Let me know if you have any questions about this process, and I'm happy to help.

Cool. Sent.


Unable to move item (drag and drop) inside TreeView in WinUI3 Desktop

Describe the bug
We are able to move items around (drag and drop) inside a TreeView in our UWP application.
However, once we switched over to WinUI3 Desktop, we are no longer able to move items around (drag and drop) inside the TreeView.
Steps to reproduce the bug

Clone the WinUI3 Problems DragDropTreeView repository.
Go to the DragDropTreeViewWinUI folder.
Open the DragDropTreeViewWinUI solution in Visual Studio 2019 Preview.
Build and run with Debug x64.
Attempt to drag an item (not the root node) in the TreeView onto another location in the TreeView.
You should see the "not permitted" symbol (circle with slash) as you are dragging over. See Screenshot#1 and Screenshot#2.

Expected behavior
We expect to see the drag indicator change to "Move" and be able to drop items back onto the TreeView.
On the right side of the application, there is the TreeView Actions Log that tracks the drag and drop events to verify what is happening.
We expect to see the same behavior we saw in UWP. See Screenshot#3 and Screenshot#4
Build and run the DragDropTreeViewUWP solution inside the DragDropTreeViewUWP folder to see expected behavior.
Screenshots

Screenshot#1 - Current Behavior

Screenshot#2 - Current Behavior End Result

Screenshot#3 - Expected Behavior

Screenshot#4 - Expected Behavior End Result
NuGet package version:
[Microsoft.VCRTForwarders.140 1.0.6]
[Microsoft.WinUI 3.0.0-preview1.200515.3]
Targeting:
Target: Universal Windows
Target version: Windows 10, version 1809 (10.0; Build 17763)
Min version: Windows 10, version 1809 (10.0; Build 17763)

Windows 10 version
Saw the problem?

Insider Build (xxxxx)

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)
Yes

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Mobile

Xbox

Surface Hub

IoT

Additional context
When we debugged the solution, we noticed that the treeView1_DragItemsCompleted (MainPage.xaml.cs Line 72) does get called.
However, the drop was not allowed so the dropped node is the same as the dragged node.

I've checked in fixes for a few different drag&drop issues in WinUI3, and they'll be in the next preview.


Reference master document one level down

NCL ERROR MESSAGE: (0)  info:   ++++++++++++++++++++++++++++++++++++++++++
NCL ERROR MESSAGE: (0)  info:   MyDiag_test_ta.ncl (var: ta)
NCL ERROR MESSAGE: (0)  info:   ++++++++++++++++++++++++++++++++++++++++++
NCL ERROR MESSAGE: fatal:asciiread: Unable to open input file (/group_workspaces/jasmin/ncas_cms/valeriu/anaconda2/lib/python2.7/site-packages/ESMValTool-1.1.1.dev1344+g8500eaa.d20171030-py2.7.egg/doc/MASTER_authors-refs-acknow.txt)
which is correct since now the MASTER document lives at:
/group_workspaces/jasmin/ncas_cms/valeriu/anaconda2/lib/python2.7/site-packages/ESMValTool-1.1.1.dev1344+g8500eaa.d20171030-py2.7.egg/esmvaltool/doc/MASTER_authors-refs-acknow.txt
ncl.interface: in_refs = (/"/group_workspaces/jasmin/ncas_cms/valeriu/anaconda2/lib/python2.7/site-packages/ESMValTool-1.1.1.dev1344+g8500eaa.d20171030-py2.7.egg/doc/MASTER_authors-refs-acknow.txt"/)
FIX: in main.py l.299
# master references-acknowledgements file (hard coded)
in_refs = os.path.join(esmvaltool_root, 'doc',
'MASTER_authors-refs-acknow.txt')
change to
# master references-acknowledgements file (hard coded)
in_refs = os.path.join(esmvaltool_root, 'esmvaltool/doc',
                       'MASTER_authors-refs-acknow.txt')

Ah, my bad.
As per #128, the duplicate docs folder in ESMValtool was removed, along with the duplicate MASTER_authors-refs-acknow.txt
I changed the reference to the "normal" doc folder. That however does not exist in the package if you install it. I'll adjust the package to match :-)

Cheers, Niels! Has this been merged into REFACTORING_backend, coz I just downloaded a fresh copy of the branch and am still seeing the error :(
Cheers,
V

He @valeriupredoi, not yet, will do, thanks for the reminder.

Cool, man, no rush, I thought I was doing something wrong and asked just in case :)


Typescript version

Hey @smakosh 👋 ,
I have a suggestion: a typescript version of your boilerplate.
it will be on another branch, so anyone can use the js or ts version.
what do you think?

Yes I was planning to do it, feel free to kick start it & I'll contribute when I can! Great suggestion!
Thank you

✔
thank you.

Added here: https://github.com/smakosh/CRA-boilerplate/tree/typescript


Update final-form version.

The final form version being out of date here causes the transpiled distributable to depend on Symbol since that was very recently fixed in final-form.

Will update deps before next release. Thanks.


Setting /calico/v1/Ready missing from master/2.0 docs

We are installing Calico 2.0 dockerless and had a hard time figuring out why calico/node worked but dockerless didn't. Finally we found the answer in the 1.6 docs: https://github.com/projectcalico/calico/blob/d83a2af83aa81f39982adca49bd750a15ff315a0/v1.6/getting-started/bare-metal/bare-metal.md#initialising-the-etcd-database
Please add this key-set to 2.0/master docs, as things will not work without.

@rtoma - the docs have been updated to discuss creating a "node" resource for each felix instance that you run - this will initialize the database.
I think prior to our 1.0.0 RC1 this was not the case, so it's possible our docs were ahead of our binary release.
Could you check that the doc makes sense and that using the 1.0.0 RC1 calicoctl resolves this issue.


Issues with importing with React Native 0.41

While I know this library is no longer maintained, I'm using a library which uses this library as a dependancy.
To update the package to work with RN 0.41, I had to update the imports like this:
"RCTBridge.h" to <React/RCTBridge.h>
I did this for all the imports with RCT are a prefix in all the .m files and the iOS app compiles successfully. The issue is when I import the library I get an error. After looking through this repo's code, I can see that React Native basically isn't importing WebViewBridgeManager properly from index.ios.js. What can I do to fix this? Thanks.

See #206


storage: close sockets on error

Continuing from #530
Fixes #523

I signed it!

Why don't merge?

Needs tests!

@ryanseys tests are ready if you want to take a glance. The feature implementation is the same.
@jgeewax / @robertdimarco this is the last outstanding PR before the next release is ready.

Going to merge since the implementation was reviewed previously.


Add ConvTasNet model

Replacing #893 to commit to master branch directly.
See #897 for the overview of ConvTasNet work.

i prefer how the code looked with sequential? but let's go with @cpuhrsch recommendation of avoiding using sequential as currently done here.

The point is not about preference, but about the flexibility. Not using Sequential container will allow users to change just the controls flow in forward method. This is better because if you use pertained weight, the structure of the model is encoded in the binary file. In this case if you want to change the execution flow in forward, you need to perform some kind of model surgery over the binary file. It is not possible but cumbersome. Not using Sequential container removes this burden.


Track batch updates in object and explicit state

Changes in this pull request
Batch updates are complicated b/c its unknown when the update block will actually execute. When executing the block, we want to collect inserts/deletes/reloads/moves (item and section). Allow mutations to happen synchronously outside of the update block.
Tracking state will also help with the auto-diff where we need to allow re-entrant updates.
Peeling off a chunk from #494
Issue fixed: #288
Checklist

[x] All tests pass. Demo project builds and runs.
[x] I added tests, an experiment, or detailed why my change isn't tested.
[x] I added an entry to the CHANGELOG.md for any breaking changes, enhancements, or bug fixes.

Importing for faster CI + internal review


All Files Become Identical

Hey, so I've encountered this bug twice now and it's really devastating. I'll be working on a document, I won't be able to publish to github, will switch to another document to see if it still works, and find that that other document has changed to be identical to the first. I'm not sure if this happens all at once or if switching to a new document triggers this, but all of them will have suddenly become identical with some one document that I was working on. I've lost a lot of work this way. I began syncing everything with Drive after it happened the first time, but since opening a new file prompts it to update to Drive, this bug erased my work there as well.
After this happened most recently, I deleted all my files, yet Stackedit kept doing this when I tried to import older drive files (making them copies of the default Hello! file). The first time I fixed this problem by clearing the browser cache: now this is not working. It seems to occur with files with a fair amount of comments on them; I wonder if this is the cause?
For what it's worth, I'm using Stackedit on the Android Chrome app.

Closing this. Please open a new ticket if the problem still exists in the v5 (https://stackedit.io/app).


Updated Embed.php to use correct DocTypes

I have forked and updated the Embed.php file to use correct DocTypes and includes use case that was also missing.
Tried to create pull request, but not able, possibly disabled :)
See fork here: https://github.com/StevenWillett/metabase-php
Hope this helps others.

Hey Steven, thanks for your contribution.
Let's try to figure out how to do the pull request so it keeps your attribution. I haven't disabled anything and can't seem to find an option about it. How did you try to do it?

I figured it out. Pull requested and merged :)


python311Packages.xattr: 1.0.0 -> 1.1.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.xattr is: Python wrapper for extended filesystem attributes
meta.homepage for python311Packages.xattr is: https://github.com/xattr/xattr
meta.changelog for python311Packages.xattr is: https://github.com/xattr/xattr/blob/v1.1.0/CHANGES.txt
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.1.0 with grep in /nix/store/6jcfxhgx90rbfhvlpazphgnjms32ph6h-python3.11-xattr-1.1.0
found 1.1.0 in filename of file in /nix/store/6jcfxhgx90rbfhvlpazphgnjms32ph6h-python3.11-xattr-1.1.0

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

6 total rebuild path(s)

6 package rebuild(s)

First fifty rebuilds by attrpath
mergerfs-tools
pifpaf
python311Packages.swift
python311Packages.xattr
python312Packages.swift
python312Packages.xattr

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/6jcfxhgx90rbfhvlpazphgnjms32ph6h-python3.11-xattr-1.1.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.xattr https://github.com/r-ryantm/nixpkgs/archive/401284660925a278c7ed55908839180e9db12f86.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/401284660925a278c7ed55908839180e9db12f86#python311Packages.xattr

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/6jcfxhgx90rbfhvlpazphgnjms32ph6h-python3.11-xattr-1.1.0
ls -la /nix/store/6jcfxhgx90rbfhvlpazphgnjms32ph6h-python3.11-xattr-1.1.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package marked as broken and skipped:

python312Packages.swift

5 packages built:

mergerfs-tools
pifpaf
python311Packages.swift
python311Packages.xattr
python312Packages.xattr

Maintainer pings

I tried enabling tests but they fail with OSError: [Errno 95] Operation not supported on ext4 and something else on ZFS (which is probably related to normalization).

Result of nixpkgs-review pr 285864 run on aarch64-darwin 1

2 packages marked as broken and skipped:

python312Packages.swift
python312Packages.swift.dist

14 packages built:

pifpaf
pifpaf.dist
poetry
poetry.dist
poetryPlugins.poetry-audit-plugin
poetryPlugins.poetry-audit-plugin.dist
poetryPlugins.poetry-plugin-up
poetryPlugins.poetry-plugin-up.dist
python311Packages.swift
python311Packages.swift.dist
python311Packages.xattr
python311Packages.xattr.dist
python312Packages.xattr
python312Packages.xattr.dist


connection.console.log don't output anything

I want to show some message in the vscode editor's debug console or Output windows. Sometimes I can use console.log to print message directly. But the message are gone after I package the extension to VSIX.
I have tried all methods to show notifications to user,
connection.window.showInformationMessage("Extension Notification")  // works but it show as a pop window.
connection.console.log("Extension Notification") // dose not work. (sometimes work sometimes not)
connection.sendNotification(), Client show errors with message "Language Server is not ready yet".
Is there any way to show log to users's output area or console area?

I have found the output message is in another channel 😭 and vscode have multiple output area.
First define a customer output channel,
let clientOptions: LanguageClientOptions = {
outputChannelName: "my channel"
}
Then let the output channel show,
client.outputChannel.show();


Add InlineAutocomplete component

Writeup adapted from github/memex#10580[^hubber-only]:
Component Description
A new InlineAutocomplete component that provides support for inline completion suggestions (for example, emojis) in textarea and input elements.
Screen recording of the component being used to mention a suggested user in a multiline text box:
https://user-images.githubusercontent.com/2294248/176513250-b3b50a5f-8dcd-4591-96fe-0116b227d72b.mov
The component is similar in some ways to the existing Autocomplete component, but this is a somewhat different pattern in that, when a suggestion is accepted, the value is inserted into the text (hence 'inline') rather than replacing the content. This pattern is closer to an IDE-style autocomplete, where the existing Primer component uses more of a 'search suggestions' pattern.
Supported Features

Stateless: suggestion visibility is fully controlled by the parent component through events and props
Can wrap any input or textarea element as well as PRC Input and TextArea components
Suggestions have a default rendering, but can be fully customized (ie, to display user avatars)
Supports multiple trigger types so that different suggestion types can be combined (ie, to support both user mentions and emojis in the same input)
Predefined hooks provide reusable, mixable behavior for the most common types of suggestions: user mentions, issue references, and emojis
Keyboard control: select items with arrow keys, apply items with Tab or Enter
Mouse control: click to select items (maintains focus on the input)
Undo history is preserved when applying suggestions
Caret position is preserved when applying suggestions

Additional Work
While adding this component, I also added some new hooks to abstract out some of the underlying logic:
useCombobox hook
The useCombobox hook provides a React-compatible wrapper around the @github/combobox-nav class. This provides keyboard control and accessible roles for comboboxes (lists of items that can be selected to apply to an input). The hook provides support for comboboxes that can be opened and closed like this one, and calls an easy to use event callback when an option is selected.
useSyntheticChange hook
When a suggestion is applied, we need to emit a change event from the underlying input. It should look to the API consumer as though the input was typed in - they shouldn't have to add any additional event handlers to react to suggestion application. In addition, we want to preserve the undo stack and caret position - if we have a separate event handler and just update the input's value, the caret position will be reset.
So this hook provides an easy way to simulate a change that is as close as possible to the real thing. The change event preserves the undo stack and caret position and calls the onChange handler that is already bound to the underlying input.
Background / Context
In projects (beta), we are using this component to provide emoji suggestions across the entire application and also to provide support for multiple types of inline suggestions in the Markdown editor.
This component is a dependency of the MarkdownEditor[^hubber-only] (AKA CommentBox[^hubber-only]) component as it provides the inline suggestions for # references, @ mentions, and emojis (:). Rather than open one giant PR, I wanted to split this work out into a separate task.
🚧 IMO, this component should be added as experimental status, just like the Markdown editor itself.
Closes github/memex#10580[^hubber-only]. There is not (at this time) a Primer tracking issue for this component.
Risks
New Dependencies
Two new external dependencies are introduced:

@github/combobox-nav: See the useCombobox hook note above.

[ ] Note: Due to the way this library uses the navigator global, it's incompatible with SSR so it doesn't work with the docs. I have an open PR to fix this.

@koddsson/textarea-caret: Detects the position of a character index in a textarea or input element. This is used to position the suggestions list and is already used in GitHub for the existing Markdown editor.

Accessibility
This component dynamically adds and removes a combobox as the user is typing. A combobox that is always visible is accessible, but one that is added and removed is not - the suggestions will likely never be read by a screen reader. This issue is already present in GitHub with the existing Markdown editor. When we have the resources to solve this, we should do so by fixing the @github/combobox-nav library so the fix will cascade down to the existing Markdown editor in addition to this component.
Storybook error
The story works in Storybook, but there is a console error being thrown that I have not been able to fix. I'm not familiar with Slots and could use some help here if anyone has any thoughts. I can't figure out where data-source might be getting applied. Is this a Storybook-specific issue?

Warning: Invalid prop data-source supplied to React.Fragment. React.Fragment can only have key and children props.
at Slots
at UserSuggestion
at SuggestionListItem
at ul
at StyledComponent
at List__ListBox
...

Merge checklist

[x] Added/updated tests (working on this - it's tested as part of MarkdownEditor in projects beta; I need to extract those tests and make them specific to this component)
[x] Added/updated documentation
[x] Tested in Chrome
[x] Tested in Firefox
[x] Tested in Safari
[x] Tested in Edge

Need to merge and update combobox-nav library to fix docs issue:

[x] https://github.com/github/combobox-nav/pull/53

[^hubber-only]: :octocat: Link only accessible to GitHub employees

Could we move this work to drafts/ to start? We can move it back to the main bundle once we're ready to mature InlineAutocomplete to an "Alpha" status

I don't mind this, but what's the difference between "Draft" and "Experimental"? Would we be able to start transitioning projects beta (memex) to use the Primer component if the Primer component is a draft?

@iansan5653 I would love to do some work on the screen reader support on this component. Currently if I activate the menu, because the screen reader is not aware that the menu appears, it's not announced by the screen reader and if the user presses down, because the menu is not associated with the input, the screenreader hijacks the down arrow and items cannot be selected.
Give me some time to discuss this in the Accessibility team but we might be able to resolve this by using aria-activedesendant
I'll have to double check if it's compatible with this usecase.

@owenniblock thanks for the accessibility notes. I am 100% on board with fixing the accessibility issues.
At the moment, I'm using the combobox-nav tool (GitHub-owned) to bind keyboard shortcuts and accessible roles. This is the same pattern we are using on the existing Rails Markdown editor across dotcom. I know that the problem also exists in the Rails editor
If I understand correctly from reading the docs, it seems that auto-complete-element is specifically designed for pulling autocomplete results from a server endpoint, which is not something we are necessarily doing here. Under the hood, the auto-complete-element is still dependent on combobox-nav.
I think the solution to this problem may lie in improving that combobox-nav utility, which would allow the changes to cascade down to the whole site. I'm not 100% sure though - we may also have to change some things on this React end to avoid unmounting/remounting the menu?

Just for reference, expand below to see the (simplified) HTML when the menu is closed. If we can figure out exactly what needs to change, we can then easily figure out whether the fix needs to be done inside combobox-nav or within this component (or both).

View HTML
When the list is not visible:
Inline Autocomplete Demo

<textarea
  aria-required="false"
  aria-invalid="false"
  id="react-aria3864888683-1"
  aria-describedby="react-aria3864888683-1-caption"
></textarea>

Try typing '@' to show user suggestions.

When the list is visible:
Inline Autocomplete Demo

<textarea
  aria-required="false"
  aria-invalid="false"
  id="input-id"
  aria-describedby="input-caption"
  role="combobox"
  aria-controls="combobox-7750"
  aria-expanded="true"
  aria-autocomplete="list"
  aria-haspopup="listbox"
></textarea>

Try typing '@' to show user suggestions.

      Monalisa Octocat

monalisa

      Primer

primer

      GitHub

github

Oh, it's my bad! It is actually setting it, but I had blurred the input (after disabling hiding the menu) to copy the HTML and that must have cleared it. When I focus the input and press down the activedescendant is set.

@owenniblock I've updated the component to (hopefully) make it work well with screen readers. I think based on my testing it should actually work better than the Markdown editor that is in production now.
Here's a list of decisions I made:

Set the role=combobox only when suggestions are available. This is because the combobox role is technically incompatible with multi-line inputs, so if apply it all the time then the input will not look multiline. By dynamically changing it when the menu appears, we can be correct as much of the time as possible while still providing useful roles.
Don't set aria-activedescendant when the suggestions first appear, even though the first item will be applied if Enter is pressed. This is because setting the activedescendant makes it look like the input is not focused anymore, which stops the screen reader from reading out the typed characters. It's also really disruptive to the user.
Insert an aria-live element with a status message that indicates when suggestions are available, how many there are, what the first one is, and how to apply it. This is assertive to prevent VoiceOver from reading the input label when the role changes, which it does try to do. assertive works well here since we don't want to wait until the user stops typing to read it.

I would appreciate any testing/feedback you can give me! I'd like to make this as ergonomic as possible for screen reader users.
Note - I do recommend testing this in a tab outside the Storybook with the "Open in new tab" button because VoiceOver may double-read aria-live regions inside iframes (it's a bug).

There are enough nuances with these changes that I think we should get a screen reader user to check them. Some of these decisions may be disruptive / confusing for screen reader users but I can't comment on that personally because I am not a screen reader user myself.
@inkblotty is there anything specific I need to do to engage James on a PR like this? Or shall I just add him as an Assignee and explain what we need?

@owenniblock Go ahead and ping him in Slack 👍

FWIW I did discuss this with James (Slack thread 🧵) already so he should already have some context - but I definitely agree with running the final product by him. I wish there was a way to deploy a staging Storybook from this PR to make it easier to share

Here's another Slack thread with @jscholes and @ichelsea about this component's accessibility. To summarize here:

The feedback message (the aria-live message that indicates when suggestions are available) is too long. @jscholes suggested changing it:- 3 autocomplete suggestions available. The first is "monalisa"; press Enter or Tab to insert it.
+ 3 autocomplete suggestions; monalisa is highlighted. Press Enter to insert.

We discussed the use of Tab to insert a suggestions. It's not ideal to hijack tab-navigation behavior but this may be expected functionality when used by software developers who are used to IDE suggestions (and are used to the current state of the Markdown editor). We'd like to eventually introduce a user setting to control this.
When you press Down Arrow one time, the activedescendant should be the first item, not the second. This is despite the first item getting inserted by default when you press Enter. @ichelsea linked to some documentation describing this exact scenario. In short, we should style the default-insert element differently than the currently active descendant.

I'll address these items today.

We're waiting for these pieces to come together, and once they do I'll do a behavioral test to be sure the requested changes all work as expected. 👍

Okay, should finally be ready now! Just to summarize, the accessibility fixes since the last review include:

Shortened the status message
Made Tab hijacking optional and disabled by default. When enabled, the status message indicates that Tab will insert
When suggestions appear, the first item in the list is highlighted using a lighter background to indicate that it is default, but not selected. When the user presses the down arrow, the first item becomes selected, with a darker background.

I see the failing "Check for changeset" action. Should I add a changeset to this PR considering it's targeting draft status? If so, how do I do that?

Thanks @andrialexandrou for taking a look at this!

I did notice that the SR announcement of each item read both the handle and the display name. This seems preferable to reading only handle or display name. Just taking note here so we're all on the same page.

Agreed, this is intentional. Just noting here that this HTML is controlled by the consumer (not the component) so the component doesn't have much control over how the items are read.

I think we should be good to go with this now!

Is this empty file intentional? https://github.com/primer/react/pull/2157/files#diff-e5c61c2ea86f2684b47c752adc87f089e3c2c06efb57b23deea6158fea599114

Is this empty file intentional?

Yes, it doesn't compile without it. I guess every folder in that directory needs an index.d.ts file.


Incorrect result with CBC

I run code from your example
const {Blowfish} = require('javascript-blowfish');
const bf = new Blowfish("key", "cbc");

// Encrypt and encode to base64
const encrypted = bf.encrypt("secret message", "cbcvecto");
const decrypted = bf.decrypt(encrypted, "cbcvecto");

console.log(decrypted);

Result of output is secret m]8⑿餸
Node version 12.16.1
Ubuntu 18.04.1

There is a change suggested in #5 but even with that change I can't seem to have a correct result with CBC.

I figured it out. It's because when you encrypted the message, the output is in binary. Since the input accepts strings, the binary is converted and the Unicode is messed up. Try this example, it works for me:
var bf = new Blowfish("key"); var encrypted = bf.base64Encode(bf.encrypt("secret message")); console.log(encrypted); var encrypted = bf.base64Decode(encrypted); var decrypted = bf.decrypt(encrypted); console.log(decrypted)

I figured it out. It's because when you encrypted the message, the output is in binary. Since the input accepts strings, the binary is converted and the Unicode is messed up. Try this example, it works for me:
var bf = new Blowfish("key"); var encrypted = bf.base64Encode(bf.encrypt("secret message")); console.log(encrypted); var encrypted = bf.base64Decode(encrypted); var decrypted = bf.decrypt(encrypted); console.log(decrypted)


fix(docs): build infinite loop

Describe the PR
Wrapping the setInterval in the SFC setup in a inBrowser check solves the infinite loop, see this comment in the vitepress issue queue. I was able to build the docs successfully locally without an infinite loop.
Addresses #1702.
PR checklist

What kind of change does this PR introduce? (check at least one)

[x] Bugfix :bug: - fix(...)
[ ] Feature - feat(...)
[ ] ARIA accessibility - fix(...)
[ ] Documentation update - docs(...)
[ ] Other (please describe)

The PR fulfills these requirements:

[x] Pull request title and all commits follow the Conventional Commits convention or has an override in this pull request body This is very important, as the CHANGELOG is generated from these messages, and determines the next version type. Pull requests that do not follow conventional commits or do not have an override will be denied

Thanks for the quick review. I will have a look.

@VividLemon I think you only need one of the two options you described:
instead of
if (inBrowser) {
  setInterval(() => {
    firstRef.value.body = `${Math.random()}`
  }, 1000)
}

Both the onMounted variant
onMounted(() => {
  setInterval(() => {
    firstRef.value.body = `${Math.random()}`
  }, 1000)
})

or  the vueuse useIntervalFn works:
useIntervalFn(() => {
  firstRef.value.body = `${Math.random()}`
}, 1000)

useIntervalFn has a built-in check for isClient, so an onMounted hook is not needed.
In both cases the docs build successfully. Which version do you prefer?

@VividLemon I think you only need one of the two options you described:
instead of
if (inBrowser) {
  setInterval(() => {
    firstRef.value.body = `${Math.random()}`
  }, 1000)
}

Both the onMounted variant
onMounted(() => {
  setInterval(() => {
    firstRef.value.body = `${Math.random()}`
  }, 1000)
})

or the vueuse useIntervalFn works:
useIntervalFn(() => {
  firstRef.value.body = `${Math.random()}`
}, 1000)

useIntervalFn has a built-in check for isClient, so an onMounted hook is not needed.
In both cases the docs build successfully. Which version do you prefer?

Just onMounted is fine

Thanks. Changed it to onMounted.


Button doesn't show up in Blender 3.0+

This plugin seems to not work in Blender 3.0+
The button just doesn't show up on rigs
Here are screenshots :

Are you sure you're looking at the properties of a generated rig? (I.e. the object called "rig" not "metarig"). The button showed up for me in Blender 3.0.0 and it worked like a charm!

Hi, same for me the addon doesn't work in Blender 3.3.1


Manually add licenses

About this issue
Hey there,
In older versions of the library I think it was possible to manually add licenses if I remember correctly. Is this still possible?
The use case is copy pasted source code not imported as a gradle dependency.

Yes you can still add libraries via the config folder like shown here in your project:

https://github.com/mikepenz/AboutLibraries/tree/develop/config/libraries
https://github.com/mikepenz/AboutLibraries/blob/develop/app/build.gradle#L94


Add outer gesture detector to increase mini-fab size to 48x48

This is one potential solution to the issue of certain hit targets being too small.  The gesture detector in the material button takes it's size from the parent constraints, there isn't a way to make the hit test area larger without also increasing the size of material.  By adding an outer gesture detector on padding we increase the hit test area without needing to modify Material or InkResponse.
For the mini-fab, by adding 4.0 of padding on all sides we increase the hit test region to 48x48.
Fixes https://github.com/flutter/flutter/issues/16625

Note: I confirmed that this works locally by increasing the padding to an absurdly large size so my fingers could hit it without pressing the button.

@Hixie & @goderbauer thoughts on this approach?

This looks like the approach I've discussed with @Hixie before.

Placing excludeSemantics: true on the outerPadding gesture detector seems to get the behavior we want, with the semantic bounds become the entire area.


Memory consumption issue/question

I would like to compute the sparse dictionary using ConvBPDNDictLearn over a collection of images, each of 720x480 resolution. Following your example in 'demo_cbpdndl_gry_2.py' , I was able to run on a small collection of images (about 10 720x480 images work). However, if I try 100 images, my computer freezes up.
I have two questions. First, is there a way to handle large numbers of images efficiently? Second, does it make sense to call ConvBPDNDictLearn in a loop, where I hand it 10 images at a time?

I would like to compute the sparse dictionary using ConvBPDNDictLearn over a collection of images, each of 720x480 resolution. Following your example in 'demo_cbpdndl_gry_2.py' , I was able to run on a small collection of images (about 10 720x480 images work). However, if I try 100 images, my computer freezes up.

As you've noticed, ConvBPDNDictLearn needs a lot of memory when applied to a large set of training images. It's worth making sure that your input array is single precision, which at least saves a factor of two in memory requirements compared with double precision. (I've just pushed a fix to a bug that was causing the demo scripts to operate in double precision.)

I have two questions. First, is there a way to handle large numbers of images efficiently?

At the moment, nothing directly supported in SPORCO, I'm afraid.

Second, does it make sense to call ConvBPDNDictLearn in a loop, where I hand it 10 images at a time?

Yes, that's a reasonable strategy. Making it work effectively will depend on exactly how you set it up though. Feel free to contact me via email if you'd like to discuss this further.


Configuration for how transactions are signed (Function Call Access Key vs Wallet integration)

Is your feature request related to a problem? Please describe.
The usecase of my DApp there is only single interaction with the Near chain. For the purpose of this, it makes more sense to make the singing of the transaction by using the wallet Full Access key instead of creating Function Call access key dedicated to the DApp and the smart contract.
The current implementation of most wallet-selector modules don't have a stable way of achieving single sign transaction without it creating function call access key.
Describe the solution you'd like
Easiest place to put this would be maybe in the modal-ui right next to the contractId. Or maybe it can be implicitly deducted when the contractId is se to undefined/null.
Describe alternatives you've considered

Providing empty string '' in the contractId field
Providing empty array for methods

Both of these behave differently on different wallet integrations. Not reliable.
Acceptance criteria

The DApp has the ability to define desired way of interacting with Near

Prefer Function Call access key
Prefer individual wallet integrations

Hi @toteto , thanks for raising an issue, we're taking a look on it.

Hi @toteto , the fix is applied, and it will be included with the next release.

Thanks @DamirSQA , will check it out when I get the release!

Yes, the fix is included the release.

@DamirSQA can you please let me know how this issue is related to the fix you provided via https://github.com/near/wallet-selector/pull/698?
Also reading through the release notes it seems that this issue is not resolved.

Hey, @toteto this issue was closed by the fact that now you can signIn to some wallets by setting the contractId to an empty string.
Just to give you more context on the topic of signIn: At the moment there is no correct way to signIn with wallet selector without creating a FunctionCall Access Key, based on the Standard here sign-in means creating a FAK. Some wallets allow you sign-in by passing the contractId as an empty string but that should not be the case because in those wallets signIn is doing two different things. The plan is for wallet selector to add a new way to signIn without adding a FunctionCall Access Key by following the signMessage NEP which is still in progress, in wallet selector this would ideally be done by using the verifyOwner function which we will need to update once everything is finalized regarding the mentioned NEP. If we have misunderstood something about this issue please let us know.

Thanks for the explanation. Hope the official support for signIn without FAK woudn't take so long. Do you have a issue that is tracking this, would be helpful maybe to link it here so people can navigate to it and track it.

You're welcome, you can have a look on this PR.


Enhancement: HamburgerButtonInfo Indicator

It would be nice to have an indicator on the HamburgerButtonInfo. For example, to show the number of unread messages. When only the icon is showing, it could overlay the upper right corner of the SymbolIcon. In the wide visual state, it could be to the right of the text.

good idea. please implement it.

nice feature

Hmm, I like it. Like the badge on a live tile. Do you have a visual example?

Closed, create a new if you get interested in this again.


[pump.io] Real 'share' activities

Currently, sharing a Pump.io post with AndStatus does not create a 'share' activity. Instead, it creates a new post, without a title if there was one (which then is lost), with a total loss of formatting, and with a weird reference to the original user (in the post text itself), and a non-clickable-link to the original post.
I think creating a real 'share' activity would be much, much better, and also, really easier, code-wise. Creating a 'share' activity is really simple:
"verb": "share",   "object": {"id": "whateverID" }
I understand that AndStatus supports several networks, and the current way is necessary to share something from Pump to the other networks, but it would be great if that new "cross-post-share" was posted only to the other networks, and to Pump, a real 'share' activity was created instead.
Cheers! o/

Existing "Share" feature doesn't make any assumptions about a target, this is why it is in a pure text format. See https://github.com/andstatus/andstatus/issues/112
Regarding Titles (Subject) support we have this ticket #150

So this ticket is for a specific case, when a source and a target are in the same Social Network of the "Pump.io" type.
And you know, we do have this feature implemented already :-)
It is called "Repost" ("Retweet" for Twitter). It was implemented long time ago.
Please check, I just "Shared" your post in terms of "Pump.io": https://identi.ca/jankusanagi/comment/cJdi4cGWQT-Z9Rn3mjr5Bw
Regarding terminology, Android has system wide meaning of "Share" action already, and AndStatus implements it in way, expected for an Android User. So using "Repost" term for Pump.io "Share" action is good enough?!
BTW, I remember complaining about "Share" action implementation in Pump.io: my followers don't see Shared message by default (by do see Favorited message...). Now I understand that maybe I have to put some recipients to the Share action in order for it to work as expected by GNU Social and Twitter users?!

So using "Repost" term for Pump.io "Share" action is good enough?!

Not really. Any user familiar with Pump.io will be used to "Share" in all other clients, so this is very misleading.
I'd call your "Share" somethink like "Copy post", "Clone post" or "Forward post", and call the other thing "Share". BTW that other "repost" option is sharing things to "public", but that's a different issue: https://github.com/andstatus/andstatus/issues/271
Note that the Pump.io "Share" action is not posting again (so, no RE-post at all), it means passing along a reference to the original post (therefore, Share).

BTW, I remember complaining about "Share" action implementation in Pump.io: my followers don't see Shared message by default (but do see Favorited message...). Now I understand that maybe I have to put some recipients to the Share action in order for it to work as expected by GNU Social and Twitter users?

Your followers see the post you share, that's worked perfectly fine all the time. Addressing for the share is not public however, but that's different. I don't think you should specify any addressing when sharing (I mean when creating a real share activity, not those repost things).

"Your followers see the post you share, that's worked perfectly fine all the time”
@jankusanagi Please read https://github.com/e14n/pump.io/issues/848 and explain, how it works, where followers can see it...
I wrote that ticket three years ago and still think that "Share” doesn't work in Pump.io as nobody replied to me.

C/P from there:
You explained a lot of technical data that I don't think is necessary here ;)
The simple test: share someone else's note, go to your feed, see the shared note.
I've done this a million times, and it works. By default, sharing is done Cc: Followers, so looking at someone's feed via the webUI while logged out will NOT show those shares, but they're there for logged in followers to see.

Now I inspected, how pump.io and AndStatus really work:

Pump.io doesn't add "by default"/automatically my Followers to a list of recipients of the shared object. I have to do this explicitly. Id of a Users's Followers group is: https://<user's host>/api/user/<user's nickname>/followers
e.g. for<EMAIL_ADDRESS>https://identi.ca/api/user/andstatus/followers

AndStatus creates "real 'share' activities" of Pump.io for years.

What is left to be done now to be more accustomed Pump.io User's friendly:

Rename "Reblog" action to "Share via Pump.io" ("Share" already has another meaning in AndStatus and in Android in general for millions of users...)
Add a User's Followers "collection" to "cc" field (before AndStatus didn't use that field at all)

Hey, thanks for working on this =)
However...

Pump.io doesn't add "by default"/automatically my Followers to a list of recipients of the shared object. I have to do this explicitly.

That's not the case. If you do NOT specify anything in the To and Cc fields, the server will take care of it, defaulting to "Cc: Followers".
Cheers!

@jankusanagi AndStatus puts "public" collection into "to" property ( https://www.w3.org/TR/activitystreams-vocabulary/#dfn-to ) meaning that a Note is publicly shared. As I understand this is not enough for my followers to see this publicly shared Note in their streams... (I think this is Pump.io bug), so, as a workaround, I have to add followers explicitly to "cc".
PS: Now I'm looking at ActivityStreams 2.0 specs and trying to use never terminology in our implementation (taking into account that pump.io is not compliant to that spec yet... https://identi.ca/cwebber/note/dqzGLzjRQdGw7TNutig4qw )

Similarly, when I create public ”Note”, I put Public collection into "to" property.
Do I have to add Followers as another recipient in order for my followers to see this ”Note”?

As I understand this is not enough for my followers to see this publicly shared Note in their streams/timelines... (I think this is Pump.io bug), so, as a workaround, I have to add followers explicitly to "cc".

No, that's not correct. Just adding "Public" to either "To" or "Cc" fields is more than enough for your followers to receive the activity. Any activity. This certainly works perfectly in Dianara. There is no Pump.io bug here.
But if you specifically add anything to either of those fields, then the server will not take care of adding anything else, it will all be up to you.
It's generally better to just leave the server do its thing, really. The only exception being when creating new content (mostly, activities with "post" verb, to create "note", "image", "video", etc. objects).

@jankusanagi "No, that's not correct. Just adding "Public" to either "To" or "Cc" fields is more than enough for your followers to receive the activity."
This means that this whole discussion is about changing a name of the Context menu item. Everything else is already OK.
Could you check now this with current released version? (v.28.01) Please use "Repost" context menu item to "Share" it.

Well, there is also the issue of "share" activities being posted "To: Public" instead of leaving the server decide (current server default is sharing "Cc: Followers"). It's not a bug per se, but I'd say it's better to leave the server do the addressing.
I'll give it a try when I can ;)
Thanks!

@jankusanagi I relied on your experience and made this change also:
"share" activity now doesn't have any recipients

@yvolk thanks for looking into this! :)

The fix is available since v.29.00. Currently in the Open Beta Testing Channel: https://play.google.com/apps/testing/org.andstatus.app


Add toml docs

Codecov Report

Merging #325 (4ea176c) into master (78b8303) will decrease coverage by 1.18%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #325      +/-   ##
==========================================
- Coverage   97.68%   96.49%   -1.19%
==========================================
  Files          23       23
  Lines        1854     1854
==========================================
- Hits         1811     1789      -22
- Misses         43       65      +22

Impacted Files
Coverage Δ

robocop/checkers/misc.py
92.30% <0.00%> (-5.77%)
:arrow_down:

robocop/checkers/comments.py
96.00% <0.00%> (-4.00%)
:arrow_down:

robocop/checkers/documentation.py
96.00% <0.00%> (-4.00%)
:arrow_down:

robocop/checkers/errors.py
94.73% <0.00%> (-2.64%)
:arrow_down:

robocop/utils/utils.py
96.29% <0.00%> (-2.47%)
:arrow_down:

robocop/checkers/__init__.py
90.80% <0.00%> (-2.30%)
:arrow_down:

robocop/checkers/naming.py
97.14% <0.00%> (-1.72%)
:arrow_down:

robocop/checkers/spacing.py
96.92% <0.00%> (-1.54%)
:arrow_down:

robocop/checkers/lengths.py
98.71% <0.00%> (-1.29%)
:arrow_down:

Continue to review full report at Codecov.

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


improve the dir generation so we have multple test outputs also when …

Improve the test directory in the target so it has a new copy also when only re-compiling the SW.

Will look at it ASAP. I CLOSE THE PR


auto row_height for table doesn't seem to be working

Hi, I'm trying to make my table's cells' height dynamic. After looking into the docs and PRs, I've tried both:
class DummyTableScreen < PM::TableScreen
  title "Your title here"
  stylesheet DummyTableScreenStylesheet
  row_height :auto, estimated: 44
...

and
  def table_data
    [{
      row_height: :auto,
      estimated_row_height: 50.0,
     cells: @texts.map do |text|
       {
        cell_class: FliessCell,
        style: { title: text },
      }
    end
    }]
  end

but maybe I'm doing something wrong somewhere else? This is what the simulator looks like:

Here is my cell's class and stylesheet:
class FliessCell < PM::TableViewCell
  def on_load
    apply_style :fliess_cell

    content = find(self.contentView)
    @title = content.append! UITextView, :fliess_cell_title
  end

  def title=(value)
    @title.text = value
  end
  def title
    @title
  end
end

module FliessCellStylesheet
  def fliess_cell_height
    40
  end

  def fliess_cell(st)
    st.frame = {l: 5, t: 100, w: 80, h: fliess_cell_height}
    st.background_color = color.light_gray
  end

  def fliess_cell_title(st)
    st.frame = {l: 10, fr: 0, centered: :vertical, h: 20}
    st.font = font.medium
    st.color = color.black
    st.scroll_enabled = false
  end
end

I'm not sure where the problem is so I'm really thankful for any help. Thanks!

@andersennl Hey! So, the docs should be updated to say this, but automatic row height only works with AutoLayout. That means that RMQ's frame-based height mechanism won't work.
There are a few other options, none ideal. The easiest, in my opinion, is to construct the cell, measure its height, and then throw it away.
    def tableView(table_view, heightForRowAtIndexPath: index_path)
      cell = tableView(table_view, cellForRowAtIndexPath: index_path)
      cell.frame.size.height
    end

Note this can cause some undesirable side-effects if you are relying on the cell's on_load (or on_cell_reused on the table) for anything else. This is because you're dequeuing a cell, setting it up, measuring its height, and then throwing it away.

Hey @jamonholmgren thanks for your response. May I ask, how exactly would you advise to use the height form inside the heightForRowAtIndexPath method? If I understand you correctly, you just grab the height from heightForRowAtIndexPath and pass it to another cell? I mean, until now I've used the table_data method to build the cells. Would you dynamically grab the cell's height from heightForRowAtIndexPath and assign it to the cells being built in table_data?
My table class basically just fetches some data in the on_load method, and then reloads the table in order to display the fetched data.
Thanks a lot!

Hey @andersennl ! You are overriding a UIKit delegate method, tableView:heightForRowAtIndexPath:. In regular ProMotion tables we provide this for you and just grab the height property of your cell hash, but in my example you're doing it and constructing the cell and measuring it to provide to UIKit the height.

@andersennl would recommend that you take a look at Motion Kit, which currently supports AutoLayout (though there are rumors that it's coming to RMQ eventually). Feel free to check out my demo projecthttps://github.com/jbender/motion-autolayout-cell-demo for an example of AutoLayout for table cells.

Thanks to both of you for your suggestions.


Unexpected error fetching porscheconnect data: 'nickName'

Version of the custom_component

Configuration
None... Using integration menu (login successful but error related to the Nickname)

Describe the bug
Error while loading the integration... Integration not loaded correctly.  Removing/adding the integration did not fix the issue.
Debug log

-------------------------------------------------------------------
porscheconnect
Version: 0.0.8
This is a custom integration!
If you have any issues with this you need to open an issue here:
https://github.com/cjne/ha-porscheconnect/issues
-------------------------------------------------------------------

2022-07-23 17:26:48 DEBUG (MainThread) [custom_components.porscheconnect] Saving tokens
2022-07-23 17:26:48 DEBUG (MainThread) [custom_components.porscheconnect] Init new data update coordinator
2022-07-23 17:26:50 INFO (MainThread) [custom_components.porscheconnect] User is authorized for vehicle vin WP1XXXXXXXX
2022-07-23 17:26:50 WARNING (MainThread) [homeassistant.helpers.frame] Detected integration that uses deprecated `async_get_registry` to access device registry, use async_get instead. Please report issue to the custom component author for tahoma using this method at custom_components/tahoma/__init__.py, line 141: device_registry = await dr.async_get_registry(hass)
2022-07-23 17:26:50 DEBUG (MainThread) [custom_components.porscheconnect] Fetching initial data for vehicle WP1XXXXXXXX
2022-07-23 17:26:50 ERROR (MainThread) [custom_components.porscheconnect] Unexpected error fetching porscheconnect data: 'nickName'
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 205, in _async_refresh
    self.data = await self._async_update_data()
  File "/config/custom_components/porscheconnect/__init__.py", line 187, in _async_update_data
    vehicle["name"] = summary["nickName"] or summary["modelDescription"]
KeyError: 'nickName'
2022-07-23 17:26:50 DEBUG (MainThread) [custom_components.porscheconnect] Finished fetching porscheconnect data in 2.574 seconds (success: False)

Fixed with new release...


Effective Code splitting for libraries?

Question:
With Laravel Mix, is it possible to code-split not only the external libraries into an vendor.js file with the .extract()-method, but also splitting single libs and load them only if necessary?
Example: We have a huge Vue application (online shop), and on one page new users can register an account. There we're using Algolias places.js. This is a huge lib - around 40kb gzipped. We want to load it only when needed on the register page, not on all pages in the vendor.js-file.
From the webpack side, this seems to be possible: https://webpack.js.org/guides/code-splitting/#dynamic-imports
It would be very good to have the same option in Laravel Mix.

You can definitely route split.
In your Vue Router file, import like this:
const AlgoliaComponent = () => import(/* webpackChunkName: "AlgoliaComponent" */ 'components/AlgoliaComponent')
Make sure you have a .babelrc file with
{
	"plugins": ["@babel/plugin-syntax-dynamic-import"]
}

in it, and install that in your package.json.
You'll also need to be on laravel-mix v4.

Thanks for the answer, but we're not using Vue Router, we have a setup with inline-templates, which are filled by Blade, in combination with regular Vue Components.

Good info to know. You may want to update your original post.
Since it seems like you're mostly doing server side routing, I don't think this is actually an issue, right? You could have an Algolia component that only gets imported in Vue on the Blade template file that matches whichever route you need it for.


[AMBARI-23034] Create HDP 2.6 to HDP 3.0 express upgrade pack

What changes were proposed in this pull request?
Create HDP 2.6 to HDP 3.0 express upgrade pack
How was this patch tested?
manual test
EU is failing on component restarts, will open separate jiras

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/706/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/708/
Test FAILed.
Test FAILured.


Issue with file types

I am having problem with file types. I want to accept *.lic files, and I can not get it to work... Here is my code:
import { FileUploader } from "react-drag-drop-files";

const fileTypes = ['lic'];

<FileUploader
  label="Drop your license (*.lic) file here"
  handleChange={handleChange}
  name="file"
  types={fileTypes}
/>

When I drag and drop my file DWX-J13M-DRCI-JQR1.lic the component would say error File type/size error, Hovered on types!.
What am I doing wrong?

Since my LIC files are basically XML files with LIC extension, I have tried the following:
import { FileUploader } from "react-drag-drop-files";
const fileTypes = ['xml'];

<FileUploader
  label="Drop your license (*.lic) file here"
  handleChange={handleChange}
  name="file"
  types={fileTypes}
/>

If I rename my LIC file to XML and drag and drop, then it is accepted. How do I achieve that my users can upload *.lic file

Can anyone help?

Figure out a solution? Running into similar issues with wav files

Hi @PrimozRome , really sorry for late (so late actually) reply. the global types not supported at the moment but we could add this in the future


Split bill - Can't select more than few users under specific Gmail account login

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

Action Performed:

Login with Gmail account with a lot of chats (always reproduced with<EMAIL_ADDRESS>Try to Split bill with 8 users

Expected Result:
The user should be able to select Split bill users without any issues
Actual Result:
As soon as a few users are selected - all the others disappear, the search also does not work.
Only for specific Gmail accounts
Workaround:
Unknown
Platforms:

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

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/65618b61-cb06-4b5e-8a76-332c1de07e56
https://github.com/Expensify/App/assets/78819774/dcc2f393-c10d-49c5-b738-5f57cdd22d27
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Yes looking 👀

Yikes this flow seems super broken...

Crashing here:
https://github.com/Expensify/App/blob/f1b37bb8f78952266dc32096b7d55cf99f0fc29b/src/libs/OptionsListUtils.js#L745-L748
Last touched here:
https://github.com/Expensify/App/pull/19515
possibly related to this one: https://github.com/Expensify/App/pull/20371/files
Still looking...

Fix incoming. I think we will maybe want to look into fixing this in Web-E based on the conversation here: https://expensify.slack.com/archives/C07J32337/p1687199748385919?thread_ts=1687190209.326479&cid=C07J32337 cc @Beamanator

After the crash is fixed there is an issue related to duplicate accounts showing up. Caused by report options and personal details options not having login fields at all here:
https://github.com/Expensify/App/blob/55e3cb9ddadde1e7ca3c60d1586343a84ffc9754/src/libs/OptionsListUtils.js#L730-L734
I am going to presume it's because we removed them in Auth/Web-Expensify without first migrating the App code, but will continue to help investigate tomorrow if required!
@Beamanator @puneetlath I think whatever we did in Auth/Web is really a "web deploy blocker" in that once it goes to production it will break the current App because the code expects login to exist on all these fields. I tested this issue on staging app + production web and it doesn't exist there.
Probably a safer rollout would be to update the App code to work for all cases then introduce changes in Auth and Web-Expensify.

I'm also seeing the crashes in dev, and my own account multiple times in the result list when searching for a chat.

This is still a bug, but I'm removing the blocker label as per this comment.

@lanitochka17 can we get a retest on this one? I think it's been fixed maybe?

This seems to have been fixed, I can't reproduce it on staging.

@marcaaron Hello, still reproducible, in search dialog a lot of old chats, but in Split bill - no. Checked with<EMAIL_ADDRESS>“problematic” account
https://github.com/Expensify/App/assets/78819774/a6cc22f9-097e-43ed-8705-b7db06acb1a8

Can you sign out and sign back in to see if the issue persists? I am suspecting this is due to some bad data stored.


sql auto-completion doesn't work with $$

Description
if use $$ in the sql text, then autocompletion doesn't work on ctrl+space

but

it also works in the function editor

I rechecked all the settings, but nothing changes
DBeaver Version
24.2.1
Operating System
Win10
Database and driver
pgsql 14.6
Steps to reproduce
No response
Additional context
No response

Thank you for your report.

Connected to #35669


Named form actions don't URI decode

Describe the bug
I've been testing out our new app using the latest version of sveltekit on AWS using CloudFront (w/ origin request policies) and Lambda Function URLs. When using named form actions I found that CloudFront will return a 400 because the / is Invalid. After encoding to %2F SvelteKit doesn't recognize it.
I've reached out to AWS already, they suggested the encoding mentioned above. I've followed up asking for documentation, will report back if they have any. I can submit a PR to sveltekit-adapter-aws to re-encode the leading / as a workaround, but doesn't feel like the right solution (cc @MikeBild).
Are other CDN provider running into this? Should decoding be handled by SvelteKit or the adaptor? Should there be a option the changes the prefix to something else that doesn't need URI encoding? Should a notice be added to the docs about AWS CloudFront?
Reproduction
Blocked by AWS CloudFront, but works locally
<form method="POST" action="?/anything">
...
</form>

AWS suggestion, Sveltekit doesn't recognize, falls back to default
<form method="POST" action="?%2Fanything">
...
</form>

Currently live example: https://qa.datastream.org/en-CA/login?/username
Logs
No response
System Info
Unable to run within AWS Lambda (arm64, nodejs18.x, esm)

Severity
serious, but I can work around it
Additional Information
No response

Something like this has come up before in #7098, but I have not read everything carefully.

Thanks, I'll give it a read.

I'm going to close, as it is a duplicate of https://github.com/sveltejs/kit/issues/7098. I'll comment there any new feedback there that I get from AWS.


rpiboot-unstable: 2018-03-27 -> 2020-02-17

Motivation for this change
Upstream added support for RPi 4, fixed a few bugs.
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

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

Very sorry about the delay in reviewing! To be honest, I don't actually have a NixOS system or Raspberry Pi handy anymore to test this change. @dezgeg @tavyc would either of you be up to take over maintaining this package?
Also note that #88174 appears to bump rpiboot-unstable's version as well.

Very sorry about the delay in reviewing! To be honest, I don't actually have a NixOS system or Raspberry Pi handy anymore to test this change. @dezgeg @tavyc would either of you be up to take over maintaining this package?

Yes, someone with a RPi at hand would be really useful for testing this. :)
On the other hand: If you got some other Linux System (maybe even WSL2) with Nix installed you can at least test compilation and invocation with nix-shell -p nixpkgs-review --run "nixpkgs-review 88174".

Also note that #88174 appears to bump rpiboot-unstable's version as well.

Don't worry, this will resolve automagically. ;)

/marvin opt-in
/status needs-reviewer

/status needs-reviewer

I think this pull request can be closed as the version in master seems to be more recent.

Closed by 271a9a012b7497984b236f1d136e58386677cf7d


Hue can't finish the query (hive on spark)

Run the query using hive on spark in HUE.After the query's result has been already displayed on the dashboard,the application's status is still running, I have to kill the applicationID manually to release Hadoop cluster computing resource. How to close the application automatically.THX.

Which Hive do you use? IIRC there is a patch for HiveServer2 to fix this issue.

Thanks for your replay.
we use Hive 1.1.0,CDH version is 5.12.1.


fix(iroh): Remove superflious info log

Description
Not every relay connection created needs to be logged on info level.
Breaking Changes

Notes & open questions

Change checklist

[x] Self-review.
[x] Documentation updates following the style guide, if relevant.
[x] Tests if relevant.
[x] All breaking changes documented.

Upon reflection, maybe I'd like this to be an event.  Not sure yet.


[ENH] Add Autocomplete for Spells and Macros

In the editor there is currently a text box to type in a spell name.  Plan is to change this to a completing edit box that auto completes known spells and current macro names.

For Patrons, tab in the box will show the spell and variable list.


Export Data Feature

The issue is that the digitizers are having to open a SQLite browser and through this pull up the 'specimen' table and export the day's work as a csv file to the N drive/specific folder.
We wish to reduced the amount of steps the digitization staff need to perform to complete a day's work.
Relevance
The process for moving the digitization effort from a workstation to the N drive is needlessly complicated and could potentially be a source for mistakes.
Effort required
It is a difficult task, yet Fedor has done work on this and made a proof of concept.
What is the expected acceptable result.
An implemented mechanism that enables the digitizer to trigger the creation of an export file. This file will still need to be transferred manually to the N drive export location (at least in this iteration of the app).
How to approach it?
Obtain all records from the specimen table by digitizer logged in name and the work date (should be the current day).
Write this to a csv file.
What would be the challenges
I see the challenges as decisions around the workflow, like : What are the criteria for a single data export?
Should the data be removed from the specimen table afterwards?
How should the export feature be implemented in the app?
Should the digitizer be able to export data made by other digitizers, and should there be an option for selecting other dates than "today"?
Potential risks
The current very hands on approach to data export does give the digitizer an opportunity to review the digitized data, that would be lost by a more automated process, unless a digest of sorts was made available to the staff upon triggering the auto export mechanism (this in itself is a large topic).
Tests
Using a well defined dataset for testing should adequately highlight any issues. A suite of predefined tests could be deployed.
Documentation
Updates to the app user manual would be needed.

[x] Ability to export table as spreadsheet file for importing into Specify
[x] Allow for marking exported records so they will not be exported a second time see #131

Though they are ticked off, the features:
Ability to export table as spreadsheet file for importing into Specify, and
Allow for marking exported records so they will not be exported a second time
... are implemented but made invisible for the moment.