xrdp-based client crashes when logged in with a new created user

Hello,
I'm experiencing an issue on xrdp-based client. The logs and the file contents are the same of what I described in the following issue ticket https://github.com/neutrinolabs/xrdp/issues/2572
I opened a new ticket because the issue is different. Here, I have a container where I created a first user in the Dockerfile. When I run the container with RDP port, and I use remmina by xrdp for connecting to the container, the access by RDP by this first user works correctly.
If I create a second user inside the container (so not defined at the beginning in the Dockerfile liek the first user), even if I assign the same groups of the first user, when I try to access by RDP by the second user, the rdp client crashes.
The logs I get are the same of the issue I reported above. I dont think it is related directly to GNOME session because I am able to access by the 1st user by RDP. I cannot do this by the 2nd user.
Why I have a different behavior if I try to access by a user that has been created when the system has been already configured?

There will be differences in the users causing the session to fail, for sure.
You've checked group memberships, which is a good start. Look at the following too:-

Home directory created with correct permissions and not on a read-only filesystem?
Home directory contents? One of your user directories may be populated from /etc/skel and the other may not be.
On SELinux systems, check the file contexts are correct on and in the home directory with ls -lZ. There's possibly an AppArmor equivalent of this, if you're using that.
For both users, generate a list of environment variables with env | sort and diff them to see what the differences are.

Another thing you can do is start a minimal session by installing xterm in your container, and adding an xterm command near the start of startwm.sh.  When you log in, you should get the xterm up, and you can look at the user environment. Press CTRL-D to exist the xterm and continue with starting GNOME.

Thank you for the answer @matt335672 My container is deployed as WSL and the first working user, called athena, in RDP has been defined and created in the Dockerfile. The second user, named apollo that is failing to access by xrdp is created after I install the WSL image, indeed when the container is run the first time, it asks me to create a user for starting.

The two users seem to have the same permissions:
athena

drwx------ athena users 4.0 KB Mon Mar 13 23:11:37 2023  ./
drwxr-xr-x root   root  4.0 KB Mon Mar 13 23:06:13 2023  ../
.rw-r--r-- athena users  11 KB Fri Mar 10 19:53:23 2023  .bash_aliases
.rw-r--r-- athena users  21 B  Thu Feb  2 07:38:44 2023  .bash_logout
.rw-r--r-- athena users  57 B  Thu Feb  2 07:38:44 2023  .bash_profile
.rw-r--r-- athena users 7.7 KB Tue Mar  7 21:50:01 2023  .bashrc
.rw-r--r-- athena users 172 B  Thu Feb  2 07:38:44 2023  .bashrc.pacnew
drwxr-xr-x athena users 4.0 KB Sun Mar 12 13:15:31 2023  .cache/
drwxrwxr-x athena users 4.0 KB Sun Mar 12 12:20:23 2023  .config/
.rw-r--r-- athena users  50 KB Fri Mar 10 19:53:23 2023  .face
.rw-r--r-- athena users   0 B  Fri Mar 10 19:53:23 2023  .flag-work-once
.rw-r--r-- athena users  83 B  Sun Mar  5 22:24:34 2023  .htb.conf
drwxr-xr-x athena users 4.0 KB Sun Mar 12 13:15:31 2023  .icons/
drwxr-xr-x athena users 4.0 KB Sun Mar 12 13:15:31 2023  .local/
drwxr-xr-x athena users 4.0 KB Sun Mar 12 13:16:18 2023  .mozilla/
drwxr-xr-x athena users 4.0 KB Sun Mar 12 13:15:31 2023  .tmux/
.rw-r--r-- athena users 3.7 KB Fri Mar 10 19:53:23 2023  .tmux.conf
drwxr-xr-x athena users 4.0 KB Sun Mar 12 13:15:31 2023  .vim_runtime/
.rw-r--r-- athena users 885 B  Fri Mar 10 19:53:23 2023  .viminfo
.rw-r--r-- athena users 349 B  Fri Mar 10 19:53:23 2023  .vimrc
drwxr-xr-x athena users 4.0 KB Sun Mar 12 13:16:19 2023  .vscode-oss/
.rwxrwxr-x athena users 890 B  Sun Mar 12 12:20:23 2023  .xinitrc*
.rw-r--r-- athena users  43 KB Mon Mar 13 23:11:37 2023  .zcompdump
.rw-r--r-- athena users 662 B  Sun Mar 12 13:18:34 2023  .zshrc

apollo
drwx------ apollo users 4.0 KB Mon Mar 13 23:06:31 2023  ./
drwxr-xr-x root   root  4.0 KB Mon Mar 13 23:06:13 2023  ../
.rw-r--r-- apollo users  11 KB Fri Mar 10 19:53:23 2023  .bash_aliases
.rw-r--r-- apollo users  21 B  Thu Feb  2 07:38:44 2023  .bash_logout
.rw-r--r-- apollo users  57 B  Thu Feb  2 07:38:44 2023  .bash_profile
.rw-r--r-- apollo users 7.7 KB Tue Mar  7 21:50:01 2023  .bashrc
.rw-r--r-- apollo users 172 B  Thu Feb  2 07:38:44 2023  .bashrc.pacnew
drwxr-xr-x apollo users 4.0 KB Sun Mar 12 13:15:31 2023  .cache/
drwxrwxr-x apollo users 4.0 KB Sun Mar 12 12:20:23 2023  .config/
.rw-r--r-- apollo users  50 KB Fri Mar 10 19:53:23 2023  .face
.rw-r--r-- apollo users   0 B  Fri Mar 10 19:53:23 2023  .flag-work-once
.rw-r--r-- apollo users  83 B  Sun Mar  5 22:24:34 2023  .htb.conf
drwxr-xr-x apollo users 4.0 KB Sun Mar 12 13:15:31 2023  .icons/
drwxr-xr-x apollo users 4.0 KB Sun Mar 12 13:15:31 2023  .local/
drwxr-xr-x apollo users 4.0 KB Sun Mar 12 13:16:18 2023  .mozilla/
drwxr-xr-x apollo users 4.0 KB Sun Mar 12 13:15:31 2023  .tmux/
.rw-r--r-- apollo users 3.7 KB Fri Mar 10 19:53:23 2023  .tmux.conf
drwxr-xr-x apollo users 4.0 KB Sun Mar 12 13:15:31 2023  .vim_runtime/
.rw-r--r-- apollo users 885 B  Fri Mar 10 19:53:23 2023  .viminfo
.rw-r--r-- apollo users 349 B  Fri Mar 10 19:53:23 2023  .vimrc
drwxr-xr-x apollo users 4.0 KB Sun Mar 12 13:16:19 2023  .vscode-oss/
.rwxrwxr-x apollo users 890 B  Sun Mar 12 12:20:23 2023  .xinitrc*
.rw-r--r-- apollo users  43 KB Mon Mar 13 23:06:31 2023  .zcompdump
.rw-r--r-- apollo users 662 B  Sun Mar 12 13:18:34 2023  .zshrc

Yes, both of the users take files from /etc/skel (for example .flag-work-once.

I don't use SELinux

They dont seem to have impacting differences:
athena

AUTOWORDLISTS=/usr/share/payloads/Auto_Wordlists
BFETCH_ART=cowsay '<3 Athena OS'
BFETCH_COLOR=/home/athena/.local/textart/color/icon/panes.textart
BFETCH_INFO=pfetch
DBUS_SESSION_BUS_ADDRESS=unix:path=/run/user/1001/bus
DEBUGINFOD_URLS=https://debuginfod.archlinux.org
DIRBIG=/usr/share/payloads/SecLists/Discovery/Web-Content/directory-list-2.3-big.txt
DIRMEDIUM=/usr/share/payloads/SecLists/Discovery/Web-Content/directory-list-2.3-medium.txt
DIRSMALL=/usr/share/payloads/SecLists/Discovery/Web-Content/directory-list-2.3-small.txt
DISPLAY=:0
EDITOR=/usr/bin/nvim
FUZZDB=/usr/share/payloads/FuzzDB
GOPATH=/home/athena/go
HISTCONTROL=ignoreboth:erasedups:ignorespace
HOME=/home/athena
HOSTTYPE=x86_64
LANG=en_US.UTF-8
LOGNAME=athena
MIMIKATZ=/usr/share/windows/mimikatz/
NAME=DESKTOP-LV4580H
OLDPWD=/home/apollo
PAGER=most
PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/usr/lib/wsl/lib:/mnt/c/Program Files (x86)/VMware/VMware Workstation/bin/:/mnt/c/Program Files/Oculus/Support/oculus-runtime:/mnt/c/Windows/system32:/mnt/c/Windows:/mnt/c/Windows/System32/Wbem:/mnt/c/Windows/System32/WindowsPowerShell/v1.0/:/mnt/c/Windows/System32/OpenSSH/:/mnt/c/Program Files (x86)/NVIDIA Corporation/PhysX/Common:/mnt/c/WINDOWS/system32:/mnt/c/WINDOWS:/mnt/c/WINDOWS/System32/Wbem:/mnt/c/WINDOWS/System32/WindowsPowerShell/v1.0/:/mnt/c/WINDOWS/System32/OpenSSH/:/mnt/c/Program Files/VSCodium/bin:/mnt/c/Program Files/NVIDIA Corporation/NVIDIA NvDLISR:/mnt/c/Program Files/dotnet/:/mnt/c/Program Files (x86)/GnuWin32/bin/:/mnt/c/Program Files (x86)/Windows Kits/10/bin/10.0.22621.0/x64/:/mnt/c/Program Files/Docker/Docker/resources/bin:/mnt/c/Users/Antonio/AppData/Local/Programs/Python/Python311/Scripts/:/mnt/c/Users/Antonio/AppData/Local/Programs/Python/Python311/:/mnt/c/Users/Antonio/AppData/Local/Microsoft/WindowsApps:/mnt/c/Users/Antonio/AppData/Local/GitHubDesktop/bin:/mnt/c/Users/Antonio/AppData/Local/Programs/MiKTeX/miktex/bin/x64/:/mnt/c/Program Files/qemu:/mnt/c/Users/Antonio/.dotnet/tools:/usr/lib/jvm/default/bin:/usr/bin/site_perl:/usr/bin/vendor_perl:/usr/bin/core_perl:/bin:/home/apollo/go/bin:/bin:/home/apollo/go/bin:/bin:/home/apollo/go/bin:/bin:/home/apollo/go/bin:/bin:/home/athena/go/bin:/bin:/home/athena/go/bin:/bin:/home/athena/go/bin:/bin:/home/athena/go/bin:/bin:/home/athena/go/bin
PAYLOADSALLTHETHINGS=/usr/share/payloads/PayloadsAllTheThings
PAYLOADS=/usr/share/payloads
POWERSPLOIT=/usr/share/windows/powersploit/
PROMPT_COMMAND=source ~/.zshrc
PULSE_SERVER=unix:/mnt/wslg/PulseServer
PWD=/home/athena
ROCKYOU=/usr/share/payloads/SecLists/Passwords/Leaked-Databases/rockyou.txt
SECLISTS=/usr/share/payloads/SecLists
SECURITYWORDLIST=/usr/share/payloads/Security-Wordlist
SHELL=/bin/zsh
SHLVL=2
TERM=xterm-256color
USER=athena
_=/usr/sbin/env
VISUAL=nano
WAYLAND_DISPLAY=wayland-0
WEBAPI_COMMON=/usr/share/payloads/SecLists/Discovery/Web-Content/api/api-endpoints.txt
WEBAPI_MAZEN=/usr/share/payloads/SecLists/Discovery/Web-Content/common-api-endpoints-mazen160.txt
WEBCOMMON=/usr/share/payloads/SecLists/Discovery/Web-Content/common.txt
WEBPARAM=/usr/share/payloads/SecLists/Discovery/Web-Content/burp-parameter-names.txt
WSL2_GUI_APPS_ENABLED=1
WSL_DISTRO_NAME=Athena
WSLENV=
WSL_INTEROP=/run/WSL/135_interop
XDG_RUNTIME_DIR=/run/user/1001/

apollo
AUTOWORDLISTS=/usr/share/payloads/Auto_Wordlists
BFETCH_ART=cowsay '<3 Athena OS'
BFETCH_COLOR=/home/apollo/.local/textart/color/icon/panes.textart
BFETCH_INFO=pfetch
DBUS_SESSION_BUS_ADDRESS=unix:path=/run/user/1001/bus
DEBUGINFOD_URLS=https://debuginfod.archlinux.org
DIRBIG=/usr/share/payloads/SecLists/Discovery/Web-Content/directory-list-2.3-big.txt
DIRMEDIUM=/usr/share/payloads/SecLists/Discovery/Web-Content/directory-list-2.3-medium.txt
DIRSMALL=/usr/share/payloads/SecLists/Discovery/Web-Content/directory-list-2.3-small.txt
DISPLAY=:0
EDITOR=/usr/bin/nvim
FUZZDB=/usr/share/payloads/FuzzDB
GOPATH=/home/apollo/go
HISTCONTROL=ignoreboth:erasedups:ignorespace
HOME=/home/apollo
HOSTTYPE=x86_64
LANG=en_US.UTF-8
LOGNAME=apollo
MIMIKATZ=/usr/share/windows/mimikatz/
NAME=DESKTOP-LV4580H
OLDPWD=/home/athena
PAGER=most
PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/usr/lib/wsl/lib:/mnt/c/Program Files (x86)/VMware/VMware Workstation/bin/:/mnt/c/Program Files/Oculus/Support/oculus-runtime:/mnt/c/Windows/system32:/mnt/c/Windows:/mnt/c/Windows/System32/Wbem:/mnt/c/Windows/System32/WindowsPowerShell/v1.0/:/mnt/c/Windows/System32/OpenSSH/:/mnt/c/Program Files (x86)/NVIDIA Corporation/PhysX/Common:/mnt/c/WINDOWS/system32:/mnt/c/WINDOWS:/mnt/c/WINDOWS/System32/Wbem:/mnt/c/WINDOWS/System32/WindowsPowerShell/v1.0/:/mnt/c/WINDOWS/System32/OpenSSH/:/mnt/c/Program Files/VSCodium/bin:/mnt/c/Program Files/NVIDIA Corporation/NVIDIA NvDLISR:/mnt/c/Program Files/dotnet/:/mnt/c/Program Files (x86)/GnuWin32/bin/:/mnt/c/Program Files (x86)/Windows Kits/10/bin/10.0.22621.0/x64/:/mnt/c/Program Files/Docker/Docker/resources/bin:/mnt/c/Users/Antonio/AppData/Local/Programs/Python/Python311/Scripts/:/mnt/c/Users/Antonio/AppData/Local/Programs/Python/Python311/:/mnt/c/Users/Antonio/AppData/Local/Microsoft/WindowsApps:/mnt/c/Users/Antonio/AppData/Local/GitHubDesktop/bin:/mnt/c/Users/Antonio/AppData/Local/Programs/MiKTeX/miktex/bin/x64/:/mnt/c/Program Files/qemu:/mnt/c/Users/Antonio/.dotnet/tools:/usr/lib/jvm/default/bin:/usr/bin/site_perl:/usr/bin/vendor_perl:/usr/bin/core_perl:/bin:/home/apollo/go/bin:/bin:/home/apollo/go/bin:/bin:/home/apollo/go/bin:/bin:/home/apollo/go/bin:/bin:/home/athena/go/bin:/bin:/home/athena/go/bin:/bin:/home/athena/go/bin:/bin:/home/athena/go/bin:/bin:/home/athena/go/bin:/bin:/home/athena/go/bin:/bin:/home/apollo/go/bin:/bin:/home/apollo/go/bin:/bin:/home/apollo/go/bin
PAYLOADSALLTHETHINGS=/usr/share/payloads/PayloadsAllTheThings
PAYLOADS=/usr/share/payloads
POWERSPLOIT=/usr/share/windows/powersploit/
PROMPT_COMMAND=source ~/.zshrc
PULSE_SERVER=unix:/mnt/wslg/PulseServer
PWD=/home/apollo
ROCKYOU=/usr/share/payloads/SecLists/Passwords/Leaked-Databases/rockyou.txt
SECLISTS=/usr/share/payloads/SecLists
SECURITYWORDLIST=/usr/share/payloads/Security-Wordlist
SHELL=/bin/zsh
SHLVL=3
TERM=xterm-256color
USER=apollo
_=/usr/sbin/env
VISUAL=nano
WAYLAND_DISPLAY=wayland-0
WEBAPI_COMMON=/usr/share/payloads/SecLists/Discovery/Web-Content/api/api-endpoints.txt
WEBAPI_MAZEN=/usr/share/payloads/SecLists/Discovery/Web-Content/common-api-endpoints-mazen160.txt
WEBCOMMON=/usr/share/payloads/SecLists/Discovery/Web-Content/common.txt
WEBPARAM=/usr/share/payloads/SecLists/Discovery/Web-Content/burp-parameter-names.txt
WSL2_GUI_APPS_ENABLED=1
WSL_DISTRO_NAME=Athena
WSLENV=
WSL_INTEROP=/run/WSL/135_interop
XDG_RUNTIME_DIR=/run/user/1001/

About xterm, I think I dont need it, because on WSL I already get a shell when I run WSL.

The point of xterm is you can see where the problem is.
 If you can't start the xterm, that is informative too.

The point of xterm is you can see where the problem is.
 If you can't start the xterm, that is informative too.

Ok, so your proposal is:

Creating a new container that has xterm already installed
Put xterm in starwm.sh
Run the container. When running, I open xterm
Then, I run a RDP connection to the container on the host machine
I should get some xterm output before going to GNOME

Is it right?
What in particular I should check on xterm? I will get some output automatically?

Put xterm in your .xinitrc before exec gnome-session.
Connect to the container over RDP.
You should get an RDP session with just an xterm. In there you can check the user is working properly, then press CTRL-D to run GNOME, or even try running gnome-session manually to see what errors you get.
Have I explained that well enough?

I'm able to reproduce better this issue. In practice it does not depend on the first or second user. It seems to depend on the Set Default User.
Indeed, in the WSL topic, you can install your WSL image by a DistroLauncher that is available by Microsoft: https://github.com/Microsoft/WSL-DistroLauncher
From here, you can get the Visual Studio project and the source code of files that are used for installing your WSL image.
One of these files manages the creation of the user after the installation. According to the source files, when the user is created, it is set as Default User. I noted that when it occurs, xrdp client does not work. Indeed I tested the creation of the user with SetDefaultUser() function and without it. In the first case, xrdp client does not work, in the second case, it works.
The source code of SetDefaultUser() is the following (DistroLauncher.cpp):
HRESULT SetDefaultUser(std::wstring_view userName)
{
    // Query the UID of the given user name and configure the distribution
    // to use this UID as the default.
    ULONG uid = DistributionInfo::QueryUid(userName);
    if (uid == UID_INVALID)
    {
        return E_INVALIDARG;
    }

    HRESULT hr = g_wslApi.WslConfigureDistribution(uid, WSL_DISTRIBUTION_FLAGS_DEFAULT);
    if (FAILED(hr))
    {
        return hr;
    }

    return hr;
}

I solved the issue by deleting the /run/user/1000 folder content after the creation of the user during the installation.
After this deletion, I can connect by xrdp client by the user.

Hello @matt335672 Just for improving the chance to fix similar issues. In a GNOME environment, xrdp client could crash at login time also if /run/user/100x directory contains wayland-0 and wayland-0.lock files. They are created in environments (like WSL) after the shutdown and the restart of WSL process, I'm not sure if it can occur also on docker containers. By removing them by rm -rf /run/user/100x/wayland-0*, gnome-shell is able again to find a DISPLAY and XRDP client starts to work again.

Here's another possibility for you:-
export XDG_SESSION_TYPE=x11
export GDK_BACKEND=x11

Found on the excellent Arch wiki:-
https://wiki.archlinux.org/title/GNOME


NOTICE file

Pull Request Description
What issue does this change request address?
Added NOTICE FILE
What are the significant changes in functionality due to this change request?
Added NOTICE FILE

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[x] 1. Review all computer code.
[x] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[x] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[x] 4. Automated Tests should pass.
[x] 5. If significant functionality is added, there must be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large tes.
[x] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[x] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[x] 8. If an analytic test is changed/added, the the analytic documentation must be updated/added.
[x] 9. If any test used as a basis for documentation examples have been changed, the associated documentation must be reviewed and assured the text matches the example.

pull request approved since just the addition of the notice file. You can merge when tests pass

MR approved


Grandient color support?

Hi there!
Congrats for this program, it's unbelievably awesome!!  It would be nice to have a gradient option on pen color selection, something like this:

Maybe have an animated gradient, where the line colors flows into the gradient colors...
Any thoughts on that?

Hi @aluferraz,
ppInk relies mainly on strokes from microsoft Ink library. This class does not offer gradient colors. also the  implementation of multiple colors will increase the complexity of the UI.
In order to provide such an effect, I would recommend you to use cliparts that could predefine some prefined shapes with gradients that can be then be animated.

Hello!
Thank you so much for replying! That's a perfect solution, I'll definitely try it out!! 😁
😁


Unique index on security_groups_vms

There should be
add_index  :security_groups_vms, [:security_group_id, :vm_id], :unique => true
but even cartridge is throwing exception, that means the real data will be probably corrupted too, so we will need some migration script along with this change, that will clean up the table

@carbonin I'm assigning this old ticket to you as it may be fixed by your work with the primary keys.


robot position and joint angles does not get updated

Hi,
I'm using (group =) moveit_commander.MoveGroupCommander("right_arm") to get the group. I'm able to access the group. But when I do group.get_current_joint_values() , I get all the values equal to zero, irrespective of the configuration of the robot. Also when I try to do group.get_current_pose().pose, I get the pose corresponding to the zero joint angles in any configuration of the arm. But the rviz moveit visualizer is working fine ( it is showing the present configuration of the robot).
Can you guide me how to resolve this issue?

Based on your description, it seems you're running into the remapping of /joint_states to /robot/joint_states that moveit_commander requires. See this thread of brr-users-group for more details:
rosrun moveit_commander moveit_commander_cmdline.py  joint_states:=/robot/joint_states


Add MRMR

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Before submitting (must do checklist)

[x] Did you read the contribution guide?
[x] Did you update the docs? We use Numpy format for all the methods and classes.
[x] Did you write any new necessary tests?
[x] Did you update the CHANGELOG?

Type of Change

[ ] Examples / docs / tutorials / contributors update
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Improvement (non-breaking change which improves an existing feature)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Proposed Changes

Related Issue

Closing issues

closes #419

Codecov Report

Merging #439 (7d32d2c) into master (5ef9223) will decrease coverage by 34.69%.
The diff coverage is 37.28%.

:exclamation: Current head 7d32d2c differs from pull request most recent head d3ac5aa. Consider uploading reports for the commit d3ac5aa to get more accurate results

@@             Coverage Diff             @@
##           master     #439       +/-   ##
===========================================
- Coverage   87.64%   52.94%   -34.70%
===========================================
  Files         112      114        +2
  Lines        5276     5334       +58
===========================================
- Hits         4624     2824     -1800
- Misses        652     2510     +1858

Impacted Files
Coverage Δ

etna/analysis/feature_selection/mrmr.py
32.35% <32.35%> (ø)

...transforms/feature_selection/feature_importance.py
43.66% <36.36%> (-56.34%)
:arrow_down:

etna/analysis/feature_selection/__init__.py
100.00% <100.00%> (ø)

etna/transforms/__init__.py
100.00% <100.00%> (ø)

etna/transforms/feature_selection/__init__.py
100.00% <100.00%> (ø)

etna/commands/__init__.py
0.00% <0.00%> (-100.00%)
:arrow_down:

etna/commands/backtest_command.py
0.00% <0.00%> (-96.43%)
:arrow_down:

etna/commands/forecast_command.py
0.00% <0.00%> (-92.00%)
:arrow_down:

etna/commands/__main__.py
0.00% <0.00%> (-87.50%)
:arrow_down:

etna/commands/resolvers.py
0.00% <0.00%> (-80.00%)
:arrow_down:

... and 69 more

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 5ef9223...d3ac5aa. Read the comment docs.


move tabulate to required packages

Resolves #137

The failing tests are caused by an apparent regression in sklearn v1.3.0, which I am reporting now.

Never mind, this was expected, see [the release notes](The sample_weight parameter now will be used in centroids initialization for cluster.KMeans, cluster.BisectingKMeans and cluster.MiniBatchKMeans. This change will break backward compatibility, since numbers generated from same random seeds will be different.). Will open a separate issue.

Rolling this into a single PR with v1.1.1


Encoding parsing doesn't give good error message for invalid encoding

From @ironpythonbot on December 9, 2014 16:35
#  -*- coding: nonsense -*-

print "Testing..."

Traceback (most recent call last):
File mscorlib, line unknown, in .ctor
File mscorlib, line unknown, in .ctor
TypeError: Value cannot be null.
Parameter name: encoding

Work Item Details
Original CodePlex Issue: Issue 1019
Status: Active
Reason Closed:  Unassigned
Assigned to:    Unassigned
Reported on:    Jul 17, 2006 at 5:38 PM
Updated on: Feb 22, 2013 at 2:17 AM
Updated by: jdhardy
Test:   test_codecs.py (test_cp1019)
Thanks: Snaury
Copied from original issue: IronLanguages/main#255

From @ironpythonbot on December 9, 2014 16:35
On 2007-11-01 04:48:22 UTC, sborde commented:
Its a negative scenario

From @ironpythonbot on December 9, 2014 16:35
On 2009-05-22 10:29:03 UTC, unknown user commented:
We now report "unknown encoding type"

From @ironpythonbot on December 9, 2014 16:35
On 2009-10-30 04:02:21 UTC, dfugate commented:
Not fixed. The correct error message is "SyntaxError: encoding problem: with BOM".

Current output is this:
SyntaxError: encoding problem: nonsense

From @ironpythonbot on December 9, 2014 16:35
On 2009-10-30 04:05:28 UTC, dfugate commented:
No dev added test and not fixed.

There is a different error message printed out on CPy 2.7
SyntaxError: encoding problem: nonsense


Fixed 'Return to top' button width on footer

Fixes DF-2208. Design wants the "Return to top" button to span the full width of the screen.
Changes

Fixed 'Return to top' button width on footer

Preview

:+1:


feat: add all hashing commands and complete encode/decode commands

I've implemented the methods for every hash type suggested by the readme, and also the rest of base64/encoding related methods :3

Looks great on a quick glance, I'll try it out locally later :)


Remove alias in pandas docs for numpy.random.randn

See the discussion here: https://github.com/pandas-dev/pandas/issues/28038#issuecomment-523439376
In pandas we have a header variable that we inject at the top of all documentation pages. One of the things there is randn = np.random.randn, which is a very bad idea that is only useful to confuse users.
With a quick grep I could just find a single case where that is used:
whatsnew/v0.10.0.rst:  In [58]: p4d = Panel4D(randn(2, 2, 5, 4),

We should  replace that case to use np.randon.randn explicitly, remove that line from the header in doc/source/conf.py and see in the doc builds if nothing fails after that change (this can be done locally or also in the CI).

Hi @Marc.
Is this for just testing?
I can infer that the issue is with using 'randn' as variable name not change np.random.randn to np.randon.randn.
Please clarify, I will send PR for this

that's code that shows in the docs, I don't think in that specific case the code is even run by Sphinx, since that Panel4D class doesn't exist anymore.
The issue is that randn = np.random.randn is a very bad idea, and we'll make that code public now (we run it in the docs, but we don't show it now).

Ok, I will send a PR and tag you.

@WuraolaOyewusi I think this was merged, feel free to close this issue.


Opt out of defaults for Configuration

These changes allow a user to create a Configuration to not allow defaults by passing in the "allow_defaults" parameter, which defaults to True to keep the rest of the code working just fine.  This can be a key in a JSON file grabbed by the .from_json() method, or directly as a kwarg into the creation of a Configuration.  In the event it's in both, a consistency check is performed.
It will raise a TensorForce exception if a user creates a Configuration that won't allow defaults and a default is attempted to be set on a parameter not already in the configuration.  Your logic of ignoring the default if the attribute exists still works, and won't raise the exception.
I included unit tests, but I wasn't sure your conventions, so rearrange as you see fit.
This flag could be added to the examples going forward if you wanted to make sure that all keys are always up to date.

Checks failing are due to agents not succeeding in tasks, nothing to do with this PR.  It'd be nice to make things a bit more deterministic to avoid these false alarms.

Thanks for the PR, can review tomorrow. Making tests deterministic is difficult because we found seeds to be always working on our machines not working on Travis.


Adds favicon and updates permalinks

👷 Deploy Preview for amp42 processing.

Name
Link

🔨 Latest commit
0fca87a7cecc4d91d7f4c7d2668b1a9c1e9f81b3

🔍 Latest deploy log
https://app.netlify.com/sites/amp42/deploys/645920df5d54d800081340c9

Looks good.


Surveying_dust_structure_via_GALEX_MIS_pt2

I have addressed all the comments in the tutorial made by @ttdu. I have not find any more specific information about MBM 15, so that remains unchanged. However, for the solution, I propose to look at the same target but with NUV images instead of FUV, and I use this to illustrate how these may serve as a preliminary estimation of dust size!


Typo in first paragraph of Development overview section

OPS Build status updates of commit cd4ca94:
:clock10: Preparing: average preparing time is 50 sec(s)

OPS Build status updates of commit cd4ca94:
:clock10: Incremental building: average incremental building time is 25 sec(s)

OPS Build status updates of commit cd4ca94:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/development-overview.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Improve thrust_util.h

uninitialized_allocator seems to be broken, because T may not be trivially default constructible.
construct_range and destroy_range should see if T is trivially constructible or trivially destructible.

💯


Should the optionalDependencies actually be peerDependencies marked as optional?

Environment
N/A
Reproduction
N/A
Describe the bug
Right now, all of these azure packages are loaded into my node_modules, and people that need it should probably explicitly install it.
Perhaps marking it as an optional peerDependency would be better?
https://docs.npmjs.com/cli/v9/configuring-npm/package-json#peerdependencies
https://docs.npmjs.com/cli/v9/configuring-npm/package-json#peerdependenciesmeta
Additional context
No response
Logs
No response

Hi. Latest release have them as optional peer.


SentencePiece

Are you going to use SentencePiece for BPE tokenization?
Thank you.

We aren't planning on changing our tokenizer. The scheme that we have works well for 102/103 languages, with the exception of Thai. (We probably should've used a modified version of SentencePiece in the first place, though).
However, for people who are training their own models on a new language, or just want to re-train from scratch with their own vocabulary, SentencePiece should work well.


x/text: gotext not compatible with Go-1.20.1

What version of Go are you using (go version)?

$ go version

go version go1.20.1 darwin/amd64

Does this issue reproduce with the latest release?
Yes, the problem started by upgrading 1.19.x to 1.20.1
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/croessner/Library/Caches/go-build"
GOENV="/Users/croessner/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/croessner/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/croessner/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.20.1"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/croessner/GoLandProjects/authserv/go.mod"
GOWORK=""
CGO_CFLAGS="-O2 -g"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-O2 -g"
CGO_FFLAGS="-O2 -g"
CGO_LDFLAGS="-O2 -g"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/1p/ps03vl3150d7nsm9ynkxkbmc0000gn/T/go-build3048905333=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

After upgrading Go, I also did:

go get -u golang.org/x/text/cmd/gotext
go mod tidy
go mod vendor
Change into subfolder server, where the main go code lives.
Call go generate

What did you expect to see?
As I added new strings, I expected to see that texts for German and French would be missing and that the catalog.go file should have been updated.
What did you see instead?

/usr/local/go/src/runtime/debuglog.go:296:20: StringData not declared by package unsafe
/usr/local/go/src/runtime/heapdump.go:159:37: StringData not declared by package unsafe
/usr/local/go/src/runtime/heapdump.go:202:32: StringData not declared by package unsafe
/usr/local/go/src/runtime/heapdump.go:204:32: StringData not declared by package unsafe
/usr/local/go/src/runtime/proc.go:658:16: String not declared by package unsafe
/usr/local/go/src/runtime/runtime.go:115:33: StringData not declared by package unsafe
/usr/local/go/src/runtime/string.go:105:17: String not declared by package unsafe
/usr/local/go/src/runtime/string.go:115:16: String not declared by package unsafe
/usr/local/go/src/runtime/string.go:121:39: StringData not declared by package unsafe
/usr/local/go/src/runtime/string.go:163:16: String not declared by package unsafe
/usr/local/go/src/runtime/string.go:268:16: String not declared by package unsafe
/usr/local/go/src/runtime/type.go:465:16: String not declared by package unsafe
/usr/local/go/src/runtime/type.go:474:16: String not declared by package unsafe
/usr/local/go/src/internal/reflectlite/type.go:346:16: String not declared by package unsafe
/usr/local/go/src/internal/reflectlite/type.go:355:16: String not declared by package unsafe
/usr/local/go/src/strings/builder.go:48:16: String not declared by package unsafe
/usr/local/go/src/strings/builder.go:48:30: SliceData not declared by package unsafe
/usr/local/go/src/strings/clone.go:27:16: String not declared by package unsafe
/usr/local/go/src/net/netip/netip.go:343:28: cannot convert slice (variable of type []byte) to [4]byte
/usr/local/go/src/net/netip/netip.go:345:30: cannot convert slice (variable of type []byte) to [16]byte
/usr/local/go/src/net/netip/netip.go:1013:27: cannot convert b (variable of type []byte) to [4]byte
/usr/local/go/src/net/netip/netip.go:1016:29: cannot convert b (variable of type []byte) to [16]byte
/usr/local/go/src/net/netip/netip.go:1019:29: cannot convert b[:16] (value of type []byte) to [16]byte
/usr/local/go/src/reflect/type.go:532:16: String not declared by package unsafe
/usr/local/go/src/reflect/type.go:541:16: String not declared by package unsafe
/usr/local/go/src/crypto/x509/internal/macos/corefoundation.go:65:29: SliceData not declared by package unsafe
/usr/local/go/src/crypto/x509/internal/macos/corefoundation.go:76:29: StringData not declared by package unsafe
/usr/local/go/src/os/file.go:246:27: StringData not declared by package unsafe
gotext: extract failed: : : couldn't load packages due to errors: reflect, os, internal/reflectlite and 4 more
hydra.go:3: running "gotext": exit status 1

It seems that gotext needs updates for Go 1.20.x

Duplicate of #58737

I followed the ticket #58737 , but the problem is not solved!
I have updated all modules and the problem persists. Please re-open this issue.

Please include a self contained reproducer for what you did, with specific versions ("updated" is also unclear).
Your original report says you ran "go generate" which is not very useful to us without your code.

You can find my code at https://gitlab.roessner-net.de/croessner/authserv/-/tree/features
I have one file, which is in server/hydra.go that uses translations. There is also the go generate line:
//go:generate gotext -srclang=en update -out=catalog.go -lang=en,de,fr
When I run "go generate" in this folder, the errors appear as shown in this issue.
You can see the go.mod and go.sum and vendor files. It should have all latest versions. Does this help a little bit? :-)

BTW, this is the example code I am using:
https://github.com/nikhilbhandari123/plugin-example
README has the steps to see the problem

@croessner did you find any workaround. I'm using Go-1.21 (from 1.19) and I have the same exact problem.


Wallet Conntection

What version of Wallet are you using?
0.0.1
What is happening?
I tried to connect vercel app for test transaction.
But when I clicked "connect" -> opened another tab with page contains "/popup.html#/request/connection" end point. Not in extension, I also tried to open fuel extension for check any requests, but didnt see, only main page with my address and balance.
So, Page without any button for signature

Next step - I open the html code for this page and tried to find any buttons, and I saw 4 buttons by xpath path : "//button"
And When I move to third button  - the buttons appeared
steps to reproduce

Download fuel extension
Register a new account
Receive tokens by faucet
Open https://fuels-wallet.vercel.app/#/dapp
Click on "Connect" button

How it should behave?
I was expecting an extension to open with a request to sign a message to connect to the site
What os are you seeing the problem on?
Windows Desktop
What browser?
Chrome
What is your browser version?
Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36

We have tested using the env, the pup opens as expected, and the buttons are shown correctly.


BUG: Should test_prim in jax2tf reraise exceptions caught from ConvertAndCompare?

Description
Otherwise it seems like this could hide some test failures.
In fact, this might already be the case, see related PR https://github.com/google/jax/pull/11747
What jax/jaxlib version are you using?
No response
Which accelerator(s) are you using?

[ ] CPU
[ ] GPU
[ ] TPU

Additional System Info
No response

Nice catch! This seems to have been fixed in 856d91d25138cf8cc2ea689371799e6c2b84b0d4


Does not support REM units in media queries

Like it says in the general.css, media queries like
@media screen and (min-width: 480px) {
    p {
        font-size: 10rem;
    }
    .test {
        background-color: red;
        border: 1px solid yellow;
        font-size: 3rem;
    }
}

are not supported. Does this mean in my stylesheet that if I use media queries based on REM units I will have to make a separate stylesheet for IE8 and there use PX units for my media queries?
Is there a way to tell REM.js to accept REM units in media queries perhaps?
Would love to be able to use REM.js with something like below.
@media screen and (min-width: 30rem) {
    p {
        font-size: 10rem;
    }
    .test {
        background-color: red;
        border: 1px solid yellow;
        font-size: 3rem;
    }
}

If that is not possibly I assume it was a wanted decision to not support REM units in media queries. If so I would like to hear why those have been left out, apparently with a purpose. Curious to what that purpose is. Perhaps that can help me understand better.
If you can just let me know if or why REM units in media queries are not supported that would be great.
Thank you for your time and help.

I have not checked the closed tickets. Duh!
Though reading the closed tickets and in particular the discussion about media queries I am not entirely sure I understand everything that has been said. MQ support was there and has been abandoned? If so where is the version of the REM polyfill that did support MQs?
In particular I don't understand this post: https://github.com/chuckcarpenter/REM-unit-polyfill/issues/60#issuecomment-135187895
What do you mean with "but the decision was made awhile ago to ignore MQ rules when the polyfill was kicked in"? So the "polyfill kicked in" means it was created(?) and then you decided to not have MQ support because it is only needed on desktop? What about those users that do not have their browser open full screen but half or a third? Make those scroll the horizontal scrollbar to see content?
So yeah if you could clarify to let me know if there is a version out there supporting REM units in MQs that would be great.
If by any chance you have an idea how to support MQs with REM units (separate stylesheet with MQ in PX for IE8 and lower?) in IE8 I am all ears.
Thank you.

@monolux the short version of it is that we decided not to support rem values in a MQ is because browsers that do not support rem values, also do not support MQs. Therein, it's not really within the scope of this polyfill to provide support for a MQ. There's no version of it that has MQ support, outside of ignoring values within them.

Thank you for your reply. Using Respond for media queries and this polyfill for rem values alongside Shiv, all working fine down to ie8 and lower possibly. Respond and Shiv are kinda so widely spread I thought this would close the circle and also make it support rem inside mqs. But hey great nonetheless and very useful!


Use DCMTK targets if present

We recently run into trouble with CTK and MITK when we finally switched most of our dependencies from the old file and directory based dependency handling of CMake pre v2.8.12 to the "modern" CMake target-based approach.
CMake cannot handle the same dependency in both modes in parallel, though, which conflicted with CTK and DCMTK. This patch checks if targets are present and then switch to the target-based approach for CTK's DCMTK dependency. Otherwise, keep the old behavior.

Closing. Superseded by https://github.com/commontk/CTK/pull/1224


Install basemap in Julia

I am trying to update my basemap from conda-forge in Julia since the basemap version in conda is v1.0.7. However, I am not sure how to update basemap from conda-forge in Julia environment. Can someone please help me do this?

It is not clear to me what you mean by Julia environment. How did you install Julia?
This command here is an example of how to start an environment with conda-forge's basemap and Julia:
conda create --name JULIA --channel conda-forge basemap basemap-data-hires julia

What I mean is, can I use conda-forge as a binary provider for Julia?
Let me give an example. If I have to install a conda package like basemap in Julia, i'd do:
using Conda
Conda.add("basemap")

The basemap package in v1.0.7, that is why I want to use conda-forge's basemap.

I have no idea of how how  Conda works in Julia and how the python package will be called there.
Sorry, I cannot help you with that. You are better off asking that in a Julialang forum or stackoverflow.
This feedstock is about the python basemap package for conda.


UI Fixes for Viewing S3 Imagery

Send Aashish some images with the level scaling problems.
Filtering of clouds/noData.

will require storing the level of NoData in the SiteImage endpoint for referencing

Double Buffering items
Add in some indicator that items are downloading.
Requiring toggling images when switching between S2 and WV

Scaling issues Currently:
It looks like the S2 imagery that has NODATA regions uses a value of 1 for are of nodata and happens to squash the rang eof the rest of the data in the view.  When using rio-tiler statistics and rio-tiler info the min/max values as well as the percentile_98 and percentile_2 are typically within ~0-255 range.
When looking at individual tiles it looks like any tiles that border a NODATA region will report a min of 1,1,1 for all bands where as the other tiles which will scale normally will report ranges from 128-240 or so.  So it appears the nodata is being baked in causing the shift in the view of the image.

Added in bits and custom scaling levels for S3 imagery.


I've installed manim, but can't get it to run the example scenes

I've installed manim, but can't get it to run the example scenes
python -m manim example_scenes.py SquareToCircle -pl
And I get the following back:
RuntimeWarning: 'manim.main' found in sys.modules after import of package 'manim', but prior to execution of 'manim.main'; this may result in unpredictable behaviour
warn(RuntimeWarning(msg))
Manim Community v0.14.0
Usage: python -m manim render [OPTIONS] FILE [SCENE_NAMES]...
Try 'python -m manim render --help' for help.
Error: No such option: -l
I'm running Python 3.10 on a Windows 11 OS.

Could please read the document or README first before you start using manim?
How to install and how to run it are both writen in the README file, why don't you read it but ask it here?


fix: bug in account storage transaction

Closing in favor of https://github.com/bucketeer-io/bucketeer/pull/755


Azure breaking changes policy is deprecated without a publicly accessible alternative

A comment at the top of this guide suggests it is deprecated and refers to an internal MSFT URL.
https://github.com/Azure/azure-rest-api-specs/blob/main/documentation/Breaking changes guidelines.md

<deprecated. For internal contributor, please refer to https://aka.ms/AzBreakingChangesPolicy for the topic>

Is there a publicly available replacement?

@strefethen
Apologies for the late reply. Thanks for reaching out to us and reporting this issue. We are looking into this issue and we will provide an update.


More information about Aegis Libraries shoul be available

Monogatari works internally with both Artemis and Kayros, however there is no information about them and thus no way for anyone to take advantage of them.
Artemis is pretty much a replacement for jQuery so Monogatari could ditch jQuery from the releases and also provides a lot of other functions that could be useful for anyone.
Kayros provides a CSS framework so people could take advantage of its grid and simple components.

UP.
If Aegis Libraries are well documented, the file Monogatari/dist/js/vendor/jquery.min.js should be deleted for the "stable" v2.0
If the user wants to use Jquery, he should import it himself instead.


Enable extensibility using plugins

Here is an example of how I might want to extend Netbox - to generate DNS records.

for each IPAM address entry, add a "DNS name" field
use this data to export zone files (both forward and reverse)
this requires:

adding a new column to ipam_ipaddress
corresponding new data entry field in IP Address add/edit/import
a separate web area for other DNS operations (e.g. export zone file, adding other DNS records like MX and CNAME)
optional triggers for provisioning (e.g. queue list of DNS changes to push)

However, making this part of Netbox core might not suit how everyone wants to manage their DNS. So would it be better instead for Netbox to support such extensions as plugins, so they remain optional?
The issue of modifying the ipam_ipaddress table can be dealt with by having a separate linked table, but the DNS code would still need some way to hook into the UI. At simplest, the DNS plugin could ask for a "DNS" button to appear on the IP address page, and that would link to a separate page or pop-up.
I can see other possible plugins to:

Auto-populate serial number, interfaces and IP addresses from SNMP queries
Connect to iKVM, serial console; control power via PDU; control console and power via IPMI
Manage VMs via ESXi, Openstack, Ganeti etc
Configure monitoring
Collect interface stats, ARP tables, bridge forwarding tables
Record fibre strands and cables

These probably belong in separate projects, given the large number of different devices which need to be supported - but they will still want to integrate into the UI somehow.
P.S. If there is a discussion list please point me at it :-)

I'm going to close this for now in favor of the generic webhooks mechanism proposed in #81 .  If it makes sense to enable plugins in some other form, we can revisit this issue.

I think this should be revisited. I need to create a custom form to standardize our process of opening a new branch, this form would ask everything necessary to create the site, vlan group, vlans and devices for a router and switch.
I can do this easily enough with my own forms and the netbox API, but a plugin infrastructure would allow me to add my custom page into the netbox gui, with links to it from the navbar and with the netbox header and footer consistently applied to my custom forms.


Add “Submit your case study” link

Is this ready to go?

Yes
On Mon 22 Nov 2021 at 10:47, Pedro Duarte @.***> wrote:

Is this ready to go?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/radix-ui/website/pull/280#issuecomment-975393014, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGERUND6PPIQBNAMIMUICDUNINVJANCNFSM5IMYUMKQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

--
Cheers,
Colm
colmtuite.com
twitter.com/colmtuite

lfg
On Mon 22 Nov 2021 at 10:49, Colm Tuite @.***> wrote:

Yes
On Mon 22 Nov 2021 at 10:47, Pedro Duarte @.***>
wrote:

Is this ready to go?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/radix-ui/website/pull/280#issuecomment-975393014,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGERUND6PPIQBNAMIMUICDUNINVJANCNFSM5IMYUMKQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

--
Cheers,
Colm
colmtuite.com
twitter.com/colmtuite

--
Cheers,
Colm
colmtuite.com
twitter.com/colmtuite


TextInput padding not working on Android

Is this a bug report?
Yes. On android, TextInput padding not working when its parent has display: none/flex to show/hide the TextInput. It works fine when the TextInput will be mounted/unmounted.
Environment
Environment:
OS:  macOS Sierra 10.12.5
Node:  6.11.1
Yarn:  0.27.5
npm:  5.3.0
Watchman:  4.7.0
Xcode:  Xcode 8.3.2 Build version 8E2002
Android Studio:  2.3 AI-162.4069837
Packages: (wanted => installed)
react: 16.0.0-beta.5 => 16.0.0-beta.5
react-native: 0.49.3 => 0.49.3
Steps to Reproduce

Parent Component of TextInput has display: 'none' when component mounted
Click to some button to show the parent component having display from 'none' to 'flex'

Expected Behavior
Padding should work just the same on iOS, TextInput padding on iOS works always even parent is displayed or not(NOTE: not mounted/unmounted but using style display: 'none'/'flex')
Actual Behavior
On Android, padding not working on TextInput

This is still an issue on 0.53. The workaround is to use absolute positioning and a negative zIndex when you want to hide the input instead of display: none.

This is still a problem in 0.50. But if render TextInput by visibility condition it works fine.
const field = !invisible ? <TextInput placeholder="Write here" /> : null;
return (<View style={invisible ? { display: 'none' } : { display: 'flex' }}>{field}</View>);


No error validation prompts when creating a new slot with an existing name

OS: All
Build Version: 20230808.1
Regression: previous release (0.11.0)
Repro Steps:

Create a slot named "test1" for one function app.
F1 -> Azure Functions: Create Slot... -> Select the above function app -> Enter "test1".
Check whether an error validation prompts.

Expect:
An error validation prompts.

Actual:

No error verification prompts
Pop up "Successful created slot" notification.

More Info:

This issue doesn't reproduce when creating a new slot with an existing name on the portal.

This issue doesn't reproduce on the released Functions extension v1.11.0.

Fixed by #3800


How to use in a corporate environment with a security proxy?

Hello,
I would like to use this in a corporate environment, but for some reason, the container proxies the request to our internal security proxy and doesn't address the thanos query endpoint directly. I don't know why it is using the security proxy, because I actually specified the upstream as http://thanos-query.thanos.svc.cluster.local:9090.
I also tried to set the environment variables no_proxy, NO_PROXY, http_proxy, HTTP_PROXY, https_proxy and HTTPS_PROXY with no success.
Do you have any idea how to fix this?

Hmm, I'm quite surprised that prom-label-proxy didn't honor the proxy env variables because it doesn't specify any Transport meaning that it uses<EMAIL_ADDRESS>It uses HTTP proxies as directed by the environment variables HTTP_PROXY, HTTPS_PROXY and NO_PROXY (or the lowercase versions thereof).


error with logging onto my piratelite wallet

there's a login error saying "There was an error sending the transaction. The error was:
Server's latest block(589564) is behind ours(1373346)"
help would be appreciated thanks
jake

yes,indeed,same with me and many others ,it seems they have some problems .this is not good.I read many reports of this error but no reply .It seems pirate chain is not yet ready ,they can not deliver what they promise


optional macro parameter for adding Sync marker to returned Future type

fixes #77
Sponsored-by: 49nord GmbH

What is the maintainers' opinion on this PR?
Despite of the discussions in #77, I believe that it would be the most pragmatic approach to merge the feature implemented by this PR until the larger Rust community converges to how we should deal with Sync futures across libraries.

Hi. Is there any option to merge it?


Screenshots of UI changes:
After

@robyoung91 looks good 👍


npm package v3.1.0 does not includes last changes

The last version of the psd.js module does not include some features like artboards.
Is it possible to update the npm package?
Best regards.

Could you elaborate a bit more ?
After doing some testing with artboards, i can confirm that they do still work and parse as they should?

I have this structure for instance:
--- Artboard 1
------ group button
------------ Text
------------ Shape background

I parse each node using a recursive function and I always call for the method:
node.get('artboard') but I always got 'undefined'.
Maybe I'm not using this function correctly?

From what i can see, an artboard is a layer type by itself ( @meltingice will need to confirm this as i am not 100% sure as my coffeescript is shoddy at best ). so using .get() on it wont work, since that is for getting values or helpers on a layer itself.
Maybe this helps you out ?

After some more testing, i get inconsistent results getting artboard data using node.get(). i will mark this as an issue now as this is something that will need to be fixed soon. For now i can't come up with a solution for you, maybe @meltingice has any thoughts on this ?

Thanks for confirmation.
The weird thing is that Artboards don't work on npm but it work with the github repository.
Maybe the npm package is not up to date?

This is indeed true after comparing the 2. The artboard file is not present in the NPM package version.
@meltingice i do think this is rather urgent as artboards are an important feature, so could you take a look at updating the npm repo when you get the chance ?

Yeah, it looks like npm hasn't been updated since some PRs were merged. Updating the npm version now.

Okay<EMAIL_ADDRESS>is released. Let me know if that fixes your issues.

After installing version ~3.2.0 i can confirm that the artboard.coffee is now present.


Fix typo in pacman command

This fixes a potentially dangerous typo in the pacman command for installing ImageMagick. pacman -Sy somepackage should never be run without -u, as it could - worst case - lead to complete system breakage and the inability to boot.

Thank you!


Refactored examples by extracting common code on camera stuff

I've been playing with the examples and I've seen there's some common code which could be extracted from them.
I've created an ExampleGame struct with shared fields accross all examples (scene / cam info) and some methods to process the rotation / movement / setup of the camera.
Some thoughts:

I'm not sure whether examples/base_example.go is the correct path to put this new code. I can change it to whatever path you seem fit.
The example shapes was not refactored as I didn't understand how to make a tetra3d.INode which is the type of the camera in that example to a tetra3d.Camera which is what most of the other examples use.
The example orthographic was also not changed as it treated the camera differently than the rest.
As I refactored several examples I began to think that ExampleGame might be a good candidate to use as a starter point for any tetra3d game one starts? It has a camera, and basic movement. Maybe rename it BaseGame?

I think there's some more code (related to the debug options, quitting, full screen) in the examples which could be refactored. I might tackle that later.

oops, just realized I pushed this commit https://github.com/SolarLune/Tetra3d/pull/13/commits/d43b9df3a7c3be5db9522f08f1c47f6d5fe7de67 which was not intended for this PR. If you are interesed in the refactoring, I can create another PR with only those changes.

Whoops, sorry for the late reply - this sounds like a great idea that I really do like, but I also think that examples should be standalone so it's simple to read and understand. I could generally refactor the examples for clarity and simplicity, though... Thanks for the PR, though - really do appreciate it!


Bug Fixed

Description

Security Report: https://www.huntr.dev/bounties/9284a48c-7f56-4087-b623-09b2ff785a4f/

This PR is the patch contents for the above report.

javascript:alert(1)
javascript://%0aalert(1)

There are two cases of javascript scheme as above. But in parse-url() only the second case was validated. The reason was that the protocol was always parsed only based on the :// character. So I added exception handling for this part. After the addition, white space removal logic was added to the beginning of the protocol, which is the content of the report.
Additional

Added javascript scheme parsing function
Added logic to remove white space characters at the beginning of the protocol

Testing
const parser = require('../lib/index.js');

console.log("[+] Test Case - 1")
console.log(parser("javascript://google.com/%0aalert(1)"))

console.log("Test Case - 2")
console.log(parser("javascript:alert(1)"))

console.log("Test Case - 3")
console.log(parser("j\x61vascript:alert(1)"))

console.log("Test Case - 4")
console.log(parser("\x00javascript:alert(1)"))

[+] Test Case - 1
{
  protocols: [ 'javascript' ],
  protocol: 'javascript',
  port: null,
  resource: 'google.com',
  user: '',
  pathname: '/%0aalert(1)',
  hash: '',
  search: '',
  href: 'javascript://google.com/%0aalert(1)',
  query: [Object: null prototype] {}
}
Test Case - 2
{
  protocols: [ 'javascript' ],
  protocol: 'javascript',
  port: null,
  resource: '',
  user: '',
  pathname: '/alert(1)',
  hash: '',
  search: '',
  href: 'javascript:alert(1)',
  query: [Object: null prototype] {}
}
Test Case - 3
{
  protocols: [ 'javascript' ],
  protocol: 'javascript',
  port: null,
  resource: '',
  user: '',
  pathname: '/alert(1)',
  hash: '',
  search: '',
  href: 'javascript:alert(1)',
  query: [Object: null prototype] {}
}
Test Case - 4
{
  protocols: [ 'javascript' ],
  protocol: 'javascript',
  port: null,
  resource: '',
  user: '',
  pathname: '/alert(1)',
  hash: '',
  search: '',
  href: 'javascript:alert(1)',
  query: [Object: null prototype] {}
}

Thank you for reply.
Are you saying that the test code should be the same as the default?
나의 iPhone에서 보냄

19:53, Ionică Bizău @.***> 작성:

@IonicaBizau requested changes on this pull request.
Thank you for the fix! However, please keep the previous test checks that were in place.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.


Fix QR code generation on initial request

In case of initial QR code wasn't scanned withing defined TTL new QR
code has to be generated. For that initial request has to be sent again
to get new ref, otherwise the same QR code will be generated over and
over again and user could not link it in the Whastapp mobile app, as it
fails with QR code outdated error.

Oh, didn't noticed change in pom.xml to be honest, and then test failed because of unsupported Java 16 so I've upgraded .github/workflows/maven.yml but in fact I've missed change in pom.xml (don't ask me how it happened). I'll revert it.
Btw, any clue why test are failing now? I've wanted to run that locally, and created .test/config.properties but I'm not sure what values I should use for contact and github_token

Oh, didn't noticed change in pom.xml to be honest, and then test failed because of unsupported Java 17 on Github actions, so I've upgraded .github/workflows/maven.yml but in fact I've missed change in pom.xml (don't ask me how it happened). I'll revert it.
Btw, any clue why test are failing now? I've wanted to run that locally, and created .test/config.propertiesfile but I'm not sure what values I should use for contact and github_token. Is there any documentation for that?

So the CI of this project is kind of intrigued. You can run the tests easily locally, all it takes is you scanning the qr code.  The problem is that you can't really do the same on a remote server. The first idea I had was to create a queue system and just make the CI tests wait for someone to scan the qr in order to make the tests work, but that is kind of hard to justify privacy wise. Instead, I designed a system that allows myself to upload my WhatsApp keys as secrets to GitHub and make all ci events run with that. I'm currently working on the MD version, so this is not going to be necessary anymore and because of that I haven't uploaded the CI credentials. That's why the tests are failing. If you want to run the tests locally, add to your config a contact entry where you specify the name of one of your WhatsApp contacts that will be used for testing purposes


Ruby on Rails Group Capstone: Kanban Board Setup

Project Link

Project Kanban Link: https://github.com/users/PraisesPJMT/projects/9
Repository Link: https://github.com/PraisesPJMT/FoodPad-App

Team Members
Here are the team members working on this project.

Member 1: Paul Onchera (@Paul-js-hub)
Member 2: Praises Tula (@PraisesPJMT)

Team Tasks
This is how project tasks are distributed to team members.

Group Task: Tasks #2, #3, #10
Member 1 (@Paul-js-hub): Tasks #4, #6, #8
Member 2 (@PraisesPJMT): Tasks #1, #5, #7, #9

Happy Reviewing!!!!

Hi team @PraisesPJMT, @Paul-js-hub 🙋♀,
Excellent work! 💯
You have done a great job creating a Kanban Board for the Recipe App.
Your feature is complete!
Congratulations!!!🎊 🥳
Highlights 🌞
✔  Board has To Do, In Progress, In Review, and Done columns
✔  Tasks have assigned members with points and a descriptive title
✔  Tasks are distributed in a fair way
Status: Approved ✔

Cheers 🥂  and Happy coding!!! 👯
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me @KDlamini  in your question so I can receive the notification.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Thanks you so much @KDlamini for the amazing review and feedback!


Zoom URL parameter

Hey Isaiah, quick wishlist item (if it doesn't already exist!).
It would be super nice if there was a way to specify the default zoom level via a URL parameter when using the https://jspaint.app/#load: syntax. When I open some very high resolution art pieces by my wife they are super zoomed in instead of "Zoomed to window". I don't know if changing the default behavior is something you would consider, but I think that would be a very nice touch if you have the time!
Thank you for your continual support of this amazing project!

Hello, I have already implemented it but the parameter I used is "zoom", it is pending to merge https://github.com/1j01/jspaint/pull/317on


Generate BigQuery DataExchange API

This PR does not publish the libraries (for various reasons).

Added the label to skip the prefix requirement - merging...


Why you need to make divisor 16 when deploying model to mmdeploy

I am now reviewing the code of fisher pruning, and find something interesting (https://github.com/open-mmlab/mmrazor/blob/main/configs/pruning/mmdet/group_fisher/retinanet/group_fisher_act_deploy_retinanet_r50_fpn_1x_coco.py#L60).
Here you gave a divisor 16 when deploying,  why you need to do that?

This is because of the characteristics of hardware, like GPU. These hardwares run faster if the dimension of a tensor/matrix is divisible by a constant, like 8, 16, or 32.

@LKJacky
yup. i knew that, and this is what i want.  did you just append all zero weight convolutions at the end of the channels?

We have two reasons to do that when deploying rather than before finetuning:

Padding when deploying is easier than padding before finetuning. Because finetuning needs to consider the initialization of the padding values. When deploying, we only need to pad zero.
We guess different hardware may need different divisors. Padding when deploying is more flexible.

You absolutely can pad before finetune, and we also provide the choice for you, please refer to source code

appreciate for your reply. i need some time to read  the code.


Double or Triple click an inactive pane too quickly cause it to not select accordingly

Type: Bug
I use 2 panes side by side, and say if the left pane is in focus and I try to double click on a word to select it on the right pane, then if I manually double click it works, but if I use Logitech Mouse's double click action then it would appear to be selected first then quickly get deselected (same for triple click action).
It seems to be that the selection worked initially then there's a delayed focus event which causes selection to be lost, as using the action to select stuff in an already focused pane works fine.
VS Code version: Code 1.92.0 (Universal) (b1c0a14de1414fcdaa400695b4db1c0799bc3124, 2024-07-31T23:26:45.634Z)
OS version: Darwin arm64 23.5.0
Modes:

System Info

Item
Value

CPUs
Apple M3 Max (16 x 2400)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabledwebgpu: enabledwebnn: disabled_off

Load (avg)
2, 3, 3

Memory (System)
64.00GB (0.28GB free)

Process Argv
--crash-reporter-id a82f9e15-b4dd-4142-9761-a3d9487fba5d

Screen Reader
no

VM
0%

Extensions (21)

Extension
Author (truncated)
Version

vscode-neovim
asv
1.18.7

dart-import-sorter
azi
0.3.3

ccs-flutter-color
cir
1.2.5

dart-code
Dar
3.94.0

flutter
Dar
3.94.0

flutter-auto-import
dav
0.0.1

vscode-eslint
dba
3.0.10

es7-react-js-snippets
dsz
4.4.3

gdscript-formatter-linter
Edd
1.3.1

prettier-vscode
esb
10.4.0

vscode-expo-tools
exp
1.5.0

vscode-yarn
gam
2.1.0

godot-tools
gee
2.1.0

vscode-react-typescript
inf
1.3.1

dart-import
lua
0.3.1

import-sorter
mik
3.3.1

vscode-react-native
msj
1.13.0

auto-semicolon-vscode
mya
1.5.2

tsimporter
pmn
2.0.1

autoimport
ste
1.5.4

vscode-gradle
vsc
3.16.2

(2 theme extensions excluded)

A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492cf:30256860
vscoreces:30445986
vscod805cf:30301675
binariesv615:30325510
vsaa593:30376534
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
962ge761:30959799
f8hc8238:30694864
pythongtdpath:30769146
welcomedialogc:30910334
pythonnoceb:30805159
asynctok:30898717
pythonregdiag2:30936856
pythonmypyd1:30879173
h48ei257:31000450
pythontbext0:30879054
accentitlementsc:30995553
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
dsvsc021:30996838
945dj816:31013170
pythoncenvpt:31062603
a69g1124:31058053
dvdeprecation:31068756
dwnewjupyter:31046869
newcmakeconfigv2:31071590
impr_priority:31102340
nativerepl1:31104043
refactort:31108082
ccpltc:31103427
pythonrstrctxtcf:31103194
wkspc-onlycs-c:31106320

I do not think that a change in VS Code would be possible here to make your custom mouse logic work. We implement mouse events over standard DOM APIs and so maybe this custom mouse logic does not result in the right set of events.
Does it reproduce on https://microsoft.github.io/monaco-editor/playground.html?source=v0.50.0#example-creating-the-editor-hello-world in a browser?

I do not know what kind of delay we need to remove.

I am closing this as unplanned given it involves a special mouse, which maybe makes it then worse for other people.


[KIECLOUD-467] - Prepare RHDM and RHPAM 7.9.x branches for 7.9.1 release

Signed-off-by: spolti<EMAIL_ADDRESS>See: https://issues.redhat.com/browse/KIECLOUD-467
Thanks for submitting your Pull Request!
Please make sure your PR meets the following requirements:

[x] Pull Request title is properly formatted: [RHDM-XYZ] Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket
[x] Attached commits represent units of work and are properly formatted
[x] You have read and agreed to the Developer Certificate of Origin (DCO) (see CONTRIBUTING.md)
[x] Every commit contains Signed-off-by: Your Name<EMAIL_ADDRESS>- use git commit -s

@mramendi f.y.i.


Autobuy stops buying health at 300k

The autobuy stops at energy shields 2, which is not enough to sustain through the night

Yeah this should probably be increased after a certain level. We're dying quick now. :skull: :skull: :skull:

Intended
https://www.allthefoxes.me/img/12-KwMk6URX.png

well spotted

It should still upgrade though, now that we are wormholing up people are dropping left and right

I've never dropped below max for more than a tick or two

Hello? This is SteamDatabase, not steamSummerMinigame.


chore: check exact? is fixed by leanprover/std4#368

Adds a test for an exact? failure reported on zulip

[x] depends on: https://github.com/leanprover/std4/pull/368

bors d+

bors merge


[SVCS-287] Make iFrame scrollable for IE11

Ticket
https://openscience.atlassian.net/browse/SVCS-287
Purpose
To make the Iframe scrollable on IE 11
Changes
Adds more js attributes for Iframe.
Side effects
None that I know of.
QA Notes
Only for IE11
Deployment Notes
Nothing special.

I forgot to mention that the scroll bar is at the end of the iframe so user still need to scroll all the way down. This is more of a QA/Prod question.

I've fixed the failing test locally. Merged.

🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉


Add setup/post cmd

Init
go mod init github.com/himkt/dobato-go
CLI (cobra + cobra-cli)

https://cobra.dev/#commands
https://zenn.dev/tama8021/articles/22_0627_go_cobra_cli
https://simple-minds-think-alike.moritamorie.com/entry/add-flags-to-cobra-app

Home directory

https://text.baldanders.info/golang/no-need-go-homedir/

Stdin

https://stackoverflow.com/questions/20895552/how-can-i-read-from-standard-input-in-the-console

Path util

Join: https://pkg.go.dev/path#example-Join
existence check: https://stackoverflow.com/questions/12518876/how-to-check-if-a-file-exists-in-go

File I/O

Write string to file: https://gobyexample.com/writing-files
Read line from file:

https://stackoverflow.com/questions/8757389/reading-a-file-line-by-line-in-go
https://qiita.com/ikawaha/items/28186d965780fab5533d

HTTP

https://pkg.go.dev/net/http
Json: https://qiita.com/howmuch515/items/65094d18292ff1c4408c

Suggestions are applied. Thank you for the reviews. :waiwai:


Add some infrastructure for invalid CASE message testing.

Adds some basic infrastructure for this and a test that tests a
SigmaR1 with an empty payload.
Problem
Hard to test what happens with CASE if you send invalid messages right now.
Change overview
Add some test infrastructure for this.
Testing
Basic "send an invalid message" test added using the new infrastructure.

@saurabhst @mspang @andy31415 @Damian-Nordic ?

@bzbarsky-apple - conflicts on this one. Commenting before stale bot has a chance to close  it.


unity crushes when i try the clip

everytime i try to use the clip, unity just crushes and closes,what should I do?

Hi!
What version of Realtime CSG and Unity are you using? Are you on windows/ mac?
Is it just the clip mode that's giving you problems? Does it crash all the time? Or just sometimes?

Also, please try the latest version at:
https://realtimecsg.com/updates/
Your problem may have already been fixed

And please let me know if a newer version fixes this for you!
Thanks!

Do you still have problems with the latest version?


fix: race condition creation module in graph in transformRequest

Description
Potential fix for https://github.com/vitejs/vite/issues/13030
Related to https://github.com/vitejs/vite/pull/8533
There could be a race condition if on the middle of processing a request, something changes in the environment (user code, optimized dep invalidation) that causes resolveId to return two different ids for the same URL.
We already had the module or the resolve result, and in loadAndTransform we were doing a new resolveId from the url after call load. This PR, avoids re-resolving by passing the resolved result if we already did it before load so we ensure the module node we add to the graph is the one we are currently transforming.

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

/ecosystem-ci run


2.0-wip: .input-prepend and .input-append are broken inside of a .form-inline

Since .input-prepend/append's styles are set to block, it breaks when used in inline forms:

The expected result should look something like:

Resolved this in my latest push, but there's one trick: you have to put the .add-on and input on the same line without space between them to work. Looks good in Chrome, will confirm later in other browsers.
Thanks!

I had to add the following CSS to fix the vertical alignment:
/* Tested in Safari 5.1.2 and Firefox 9.0.1 */
.form-inline {
  .input-prepend, .input-append {
    margin-bottom: 0;
    vertical-align: middle;
  }
}

This workaround seems broken in the latest chrome beta 17, but is working in firefox 10.

@mlegenhausen is correct, I have the same issue in chrome (dev build 18), but ok in ff10

I can confirm this is broken in last Chrome (17.0.963.46), even with .add_on and input in the same line. Working in last FF (10.0).

Does someone have a workaround? I tried experimenting with
.form-inline .input-prepend {
white-space: nowrap;
}
but while this causes rendering on a single line this causes Chrome to miscalculate the size of the input-prepend div by the size of the add-on span.

@ginkel you can add
-webkit-padding-end: 27px;

to fix the miscalculation

Confirm bug in Chromium 16.0.912.77.
@huug seems to be this fix doesn't work (maybe because of Chromium, instead of Chrome), - miscalculation remains.

Really, this solved the problem of Chome 16.0.912.77 and maintained compatibility with FF 10.0.2.
.input-prepend {
white-space: nowrap;
-webkit-padding-end: 27px;
}

@markdotto  - Is there an example of what you mean by "you have to put the .add-on and input on the same line"? I'm having trouble getting this to work.


Add Appcache deprecation

Docs Build status updates of commit e3fbe4a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/web-platform/site-impacting-changes.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 6759503:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/web-platform/site-impacting-changes.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit a486d09:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/web-platform/site-impacting-changes.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

#sign-off offline and merged.


A no shrink setting.

Some context: As you know, I am working on a complicated strategy with slow tests associated with it. This means that shrinking takes a long time (several hours), and that when we speed up shrinking we just make things more complicated to match. A better strategy may fix this situation (suggestions welcome), but that is the context I am coming from.
Currently if the tests pass it takes ~1 min to run, but if it fails then all we get on CI is a time out.
No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.
I wonder whether we could have an opt in setting for the proptest macro that would skip shrinking and just print the seed. The idea is that CI could run the test in this fail-fast mode, then if we want to shrink the example that CI found we could put the seed in the local regressions file.

There is a work around, and the more I think about this the less it makes sense as having first class support. It would be nice to see what seed failed the CI run, but it is not going to be easy to figure out what to do with it, like which file to add it too.


On macOS local camera frames have wrong reported resolution

Describe the bug
C++ SDK 2.5.2 incorrectly reports the width and height of local camera preview frames in handle_frame, causing the plugin to display only part of the frame if the width and height are lower than the actual buffer size.
Expected Behavior
The local camera preview is displayed correctly and fully.
Specifications

OS: macOS
Version:

comms-sdk-unreal develop branch
C++ SDK 2.5.2

Important to note this issue is macOS specific.

Will be fixed in 1.1.0-beta.7


Bumped Support Library to 25.3.1

As reported in #784, I updated supportLibraryVersion from 25.3.0 to 25.3.1.
I used this local build with my project instead of release v2.3.1 and it builds correctly.
I run the :bottom-bar tests locally and all of them passed.

Coverage remained the same at 63.273% when pulling c6a264badb10bd62479532f492265057faae2b75 on mateiradu:master into 711fcafb35d1ba0bb1fb6a7b646110afa9824088 on roughike:master.


[FLINK-18579][FLINK-15728][jdbc] Introduce FieldNamedPreparedStatement to support fields are bound multiple times in update statement

What is the purpose of the change
Currently, the update statement from JdbcDialect is always broken, because fields will be bound multiple times and this is not supported in the Jdbc source/sinks.
We introudced a FieldNamedPreparedStatement support fields are bound multiple times in update statement. Users can use :name to reference a value of name field, instead of using ?. So that, users can reference a same field in the statement multiple times easily. For example, an update statement will be:
UPDATE `tbl` SET `id` = :id, `name` = :name, `email` = :email, `ts` = :ts,
`field1` = :field1, `field_2` = :field_2
WHERE `id` = :id

Brief change log

The first commit 164b4a280cbd1083b87cf0cc887efacd8afeea67 remove the deprecated classes, this can reduce the amount of changes.
Introduce FieldNamedPreparedStatement, the implementation is simply mapping the field index to the parameter indexes and forwarding to invoke the underlying PreparedStatement.
Update the new RowData-based source/sink implementations to use the FieldNamedPreparedStatement.
We keep the old source/sink implementation as it is, because this requires a bigger refactoring and the current interface JdbcStatementBuilder conflicts with the FieldNamedPreparedStatement interface. So old source/sink connectors still have the problem of update statement.

Verifying this change

Added FieldNamedPreparedStatementImplTest to test the field name binding.
Updated JdbcDynamicTableSinkITCase#testUpsert to use a smaller batching size which can reproduce the problem.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

cc @leonardBang , appreciate if you can have a look when you have time.

Thanks for the reviewing @leonardBang . I have addressed the review comments. I have to rebased the branch to resovle the compile problem.

Thanks @wuchong for the update, Postgres dialect tests failed because of it does use "`" as quote identifier,  please fix it and others looks good to me.

LGTM, +1 to merge


fix: fix preview.proxy does not work when server.proxy is empty

7414

Thanks for the contribution @LIUYANG132315. I've created #7604 which should fix the test errors here. Closing as the issue is fixed.


Always includes blank on select if use_required_attribute = true

I can't remove the blank option from the select tags when use_required_attribute is being used. I tried to remove it locally:
<%= f.input :activity, as: :select, include_blank: false %>

and also changing the global configuration:
Formtastic::FormBuilder.include_blank_for_select_by_default = false

Unfortunately, none of these approaches worked as expected. The only whey to make it work is turning off the usage of the required attribute:
#Formtastic::FormBuilder.use_required_attribute = true

Using

Rails 4.0.0
Formtastic 2.2.1

@justinfrench I just hit this as well.
t.input :foo, as: :select, label: "Foo Type", include_blank: false, allow_blank: false, collection: {"Option 1" => "1", "Option 2" => "2"}, input_html: {required: true} still renders a blank option in the select. Removing required: true makes the blank option then disappear.

@robbieshepherd could you try to contribute a failing test case?

I've figured it out. It took me a while as I started looking at this from the wrong end of the problem. This is deliberate behaviour in Rails. In fact, in Rails 5 (which I'm not using yet), this would actually raise an exception. See rails/rails@077540738460d44c1463b3e52d2acb1fed9bc766. They have taken this decision to comply with the HTML 5 spec. It does make some sense as the browser should guard you from clicking submit with an blank value selected but this doesn't help users on older browsers. I guess the only way to work around this is to leave the blank value in but select the first item in the collection by default.


[BUG] Odd slow for app? Cant find logs

Describe the bug
Hi
Currently we have an odd issue as the app it self is really slow, i was thinking it was the host itself but i have other sites working well,
I was trying to troubleshoot but cant seem to find the logs of php or php-fpm any ideas where i can find it?
which Branch / PHP Version are you using ?
php8.1
Thank you

Hi, you could check the logs of the php container itself with docker logs .

Thanks for the reply, at the end it seems that it was an issue with the app itself


[ACS][JobRouter] GA Version

Data Plane API - Pull Request

API Info: The Basics
Most of the information about your service should be captured in the issue that serves as your API Spec engagement record.

Link to API Spec engagement record issue:

Is this review for (select one):

[ ] a private preview
[ ] a public preview
[x] GA release

Change Scope
This section will help us focus on the specific parts of your API that are new or have been modified. Please share a link to the design document for the new APIs, a link to the previous API Spec document (if applicable), and the root paths that have been updated.

Design Document:
Previous API Spec Doc:
Updated paths:

Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.
Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
Swagger-Suppression-Process
to get approval.
❔Got questions? Need additional info?? We are here to help!

 Contact us!
The Azure API Review Board is dedicated to helping you create amazing APIs. You can read about our mission and learn more about our process on our wiki.

💬 Teams Channel
💌 email

Click here for links to tools, specs, guidelines & other good stuff
Tooling

Open API validation tools were run on this PR.   Root cause is https://github.com/Azure/typespec-azure/issues/3833.

@anuchandy this is ready to be merged


Patch creation seems to fail

Describe the bug
When I run the example command, the code executes perfectly, but when I attempt to access the generated patch, it doesn't seem to have been created.
Steps/commands/code to Reproduce
docker run --rm -it -v /var/run/docker.sock:/var/run/docker.sock -v $(pwd)/keys.cfg:/app/keys.cfg sweagent/swe-agent-run:latest python run.py --image_name=sweagent/swe-agent:latest --model_name gpt4-turbo --data_path https://github.com/pvlib/pvlib-python/issues/1603 --config_file config/default_from_url.yaml --skip_existing=False

sudo docker run --rm -it -v /var/run/docker.sock:/var/run/docker.sock -v $(pwd)/keys.cfg:/app/keys.cfg sweagent/swe-agent-run:latest python run.py --image_name=sweagent/swe-agent:latest --model_name gpt4o --data_path https://github.com/pvlib/pvlib-python/issues/1603 --config_file config/default_from_url.yaml --skip_existing=False

Error message/results
INFO     Found submission:
         diff --git a/pvlib/modelchain.py b/pvlib/modelchain.py
         index 8456aac..4116d9b 100644
         --- a/pvlib/modelchain.py
         +++ b/pvlib/modelchain.py
         @@ -646,6 +646,10 @@ class ModelChain:
                  params = calcparams_model_function(self.results.effective_irradiance,
                                                     self.results.cell_temperature,
                                                     unwrap=False)
         +        # Check for zero effective irradiance and handle it
         +        if self.results.effective_irradiance <= 0:
         +            self.results.dc = pd.DataFrame({'p_mp': 0})
         +            return self
                  self.results.diode_params = tuple(itertools.starmap(
                      _make_diode_params, params))
                  self.results.dc = tuple(itertools.starmap(

INFO     Trajectory saved to
         trajectories/root/gpt4-turbo__pvlib__pvlib-python__default_from_url__t-0.00__p-0.95__c-3.00__install-1/pvlib__pvlib-python-i1603.tr
         aj
INFO     Saved predictions to
         trajectories/root/gpt4-turbo__pvlib__pvlib-python__default_from_url__t-0.00__p-0.95__c-3.00__install-1/all_preds.jsonl
╭──────────────────────────── 🎉 Submission successful 🎉 ────────────────────────────╮
│ SWE-agent has produced a patch that it believes will solve the issue you submitted! │
│ Use the code snippet below to inspect or apply it!                                  │
╰─────────────────────────────────────────────────────────────────────────────────────╯

 # The patch has been saved to your local filesystem at:
 PATCH_FILE_PATH='/app/trajectories/root/gpt4-turbo__pvlib__pvlib-python__default_from_url__t-0.00__p-0.95__c-3.00__install-1/patches/pvlib
 pvlib-python-i1603.patch'
 # Inspect it:
 cat "${PATCH_FILE_PATH}"
 # Apply it to a local repository:
 cd <your local repo root>
 git apply "${PATCH_FILE_PATH}"

INFO     Beginning environment shutdown...
INFO     Agent container stopped
chris@DESKTOP-LOKPOGS:~$ echo $PATCH_FILE_PATH
/app/trajectories/root/gpt-4o-2024-05-13__pvlib__pvlib-python__default_from_url__t-0.00__p-0.95__c-3.00__install-1/patche pvlib__pvlib-python-i1603.patch
chris@DESKTOP-LOKPOGS:~$ cat "${PATCH_FILE_PATH}"
cat: '/app/trajectories/root/gpt-4o-2024-05-13__pvlib__pvlib-python__default_from_url__t-0.00__p-0.95__c-3.00__install-1/patche'$'\n'' pvlib__pvlib-python-i1603.patch': No such file or directory
chris@DESKTOP-LOKPOGS:~$ cat $PATCH_FILE_PATH
cat: /app/trajectories/root/gpt-4o-2024-05-13__pvlib__pvlib-python__default_from_url__t-0.00__p-0.95__c-3.00__install-1/patche: No such file or directory
cat: pvlib__pvlib-python-i1603.patch: No such file or directory

example #2:
INFO     🎬 ACTION (primary)
         rm reproduce.py

INFO     🤖 MODEL INPUT
         Your command ran successfully and did not produce any output.
         (Open file: /pvlib__pvlib-python/pvlib/tools.py)
         (Current directory: /pvlib__pvlib-python)
         bash-$
DEBUG    input_tokens=16,660, output_tokens=28, instance_cost=2.01, cost=0.08
DEBUG    total_tokens_sent=394,801, total_tokens_received=2,369, total_cost=2.01, total_api_calls=25
INFO     💭 THOUGHT (primary)
         The `reproduce.py` file has been successfully removed. Now, let's submit our changes to the code base.

INFO     🎬 ACTION (primary)
         submit

INFO     Found submission:
         diff --git a/pvlib/tools.py b/pvlib/tools.py
         index c8d4d6e..2bc2c1a 100644
         --- a/pvlib/tools.py
         +++ b/pvlib/tools.py
         @@ -337,6 +337,8 @@ def _build_args(keys, input_dict, dict_name):
          # Modified: November, 2020 by C. W. Hansen, to add atol and change exit
          # criteria
          def _golden_sect_DataFrame(params, lower, upper, func, atol=1e-8):
         +    if np.all(upper == lower):
         +        max_val = upper
              """
              Vectorized golden section search for finding maximum of a function of a
              single variable.

INFO     Trajectory saved to
         trajectories/root/gpt4o__pvlib__pvlib-python__default_from_url__t-0.00__p-0.95__c-3.00__install-1/pvlib__pvlib-python-i1603.traj
INFO     Saved predictions to
         trajectories/root/gpt4o__pvlib__pvlib-python__default_from_url__t-0.00__p-0.95__c-3.00__install-1/all_preds.jsonl
╭──────────────────────────── 🎉 Submission successful 🎉 ────────────────────────────╮
│ SWE-agent has produced a patch that it believes will solve the issue you submitted! │
│ Use the code snippet below to inspect or apply it!                                  │
╰─────────────────────────────────────────────────────────────────────────────────────╯

 # The patch has been saved to your local filesystem at:
 PATCH_FILE_PATH='/app/trajectories/root/gpt4o__pvlib__pvlib-python__default_from_url__t-0.00__p-0.95__c-3.00__install-1/patches/pvlib__pvl
 -python-i1603.patch'
 # Inspect it:
 cat "${PATCH_FILE_PATH}"
 # Apply it to a local repository:
 cd <your local repo root>
 git apply "${PATCH_FILE_PATH}"

INFO     Beginning environment shutdown...
INFO     Agent container stopped
chris@DESKTOP-LOKPOGS:~$  PATCH_FILE_PATH='/app/trajectories/root/gpt4o__pvlib__pvlib-python__default_from_url__t-0.00__p-0.95__c-3.00__install-1/patches/pvlib__pvl
 -python-i1603.patch'
 # Inspect it:
 cat "${PATCH_FILE_PATH}"
cat: '/app/trajectories/root/gpt4o__pvlib__pvlib-python__default_from_url__t-0.00__p-0.95__c-3.00__install-1/patches/pvlib__pvl'$'\n'' -python-i1603.patch': No such file or directory
chris@DESKTOP-LOKPOGS:~$

System Information
Ubuntu 22.04.5 running in WSL2
Checklist

[X] I'm running with the latest docker container/on the latest development version (this means running both git pull and docker pull sweagent/swe-agent:latest)
[X] I have copied the full command/code that I ran (as text, not as screenshot!)
[X] If applicable: I have copied the full log file/error message that was the result (as text, not as screenshot!)
[X] I have enclosed code/log messages in triple backticks (docs) and clicked "Preview" to make sure it's displayed correctly.

Hi, this is a known issue. The patch file is created within your docker container rather than on your host machine. SWE-agent 1.0 (to be released soon) won't have this issue anymore, but for now, please use the local installation of swe-agent.


refactor: remove unused code

Description:
Remove unused code introduced in #1576

How Has This Been Tested?
Manually

Types of changes:
Refactor

Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[x] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage increased (+0.002%) to 19.099% when pulling b093ef9f5f4678846cacd08e376ffa531ff29551 on korhaliv:refactor/channel-code-clean into 0c04fc432bf5d581bf2e5829a62e834dcd10505d on LN-Zap:next.

Coverage increased (+0.002%) to 19.099% when pulling b093ef9f5f4678846cacd08e376ffa531ff29551 on korhaliv:refactor/channel-code-clean into 0c04fc432bf5d581bf2e5829a62e834dcd10505d on LN-Zap:next.