Deployer failed complete deploying

Q
A

Issue Type
Question

Deployer Version
5

Local Machine OS
Windows-10

Remote Machine OS
Debian_8.7(Jessie)

Description
When trying to deploy, deployer creates the directories properly on remote machine but fails and throws errors as below:
Steps to reproduce
Content of deploy.php
 <?php
namespace Deployer;
require 'recipe/laravel.php';

// Configuration
set('keep_releases', 5);
set('ssh_type', 'native');
set('ssh_multiplexing', true);
set('branch', 'master');
set('repository', 'git@ip:root/~/temp/repo.git');

add('shared_files', [
    '.env'
]);
add('shared_dirs', [
    'storage'
]);

add('writable_dirs', [
    'bootstrap/cache',
    'storage',
    'storage/app',
    'storage/app/public',
    'storage/framework',
    'storage/framework/cache',
    'storage/framework/sessions',
    'storage/framework/views',
    'storage/logs',
]);

// Servers

server('production', 'ip')
    ->user('root')
    ->identityFile()
    ->configFile('~/.ssh/config')
    ->set('deploy_path', '/var/www/domain')
    ->pty(true);

// Tasks

task('upload:env', function () {
    upload('.env.production', '{{deploy_path}}/shared/.env');
})->desc('Environment setup');

desc('Restart PHP-FPM service');
task('php-fpm:restart', function () {
    // The user must have rights for restart service
    // /etc/sudoers: username ALL=NOPASSWD:/bin/systemctl restart php-fpm.service
    run('sudo systemctl restart php-fpm.service');
});
after('deploy:symlink', 'php-fpm:restart');

// [Optional] if deploy fails automatically unlock.
after('deploy:failed', 'deploy:unlock');

// Migrate database before symlink new release.
desc('Deploy your project');
task('deploy', [
    'deploy:prepare',
    'deploy:lock',
    'deploy:release',
    'deploy:update_code',
    'deploy:shared',
    'upload:env',
    'deploy:vendors',
    'deploy:writable',
    'artisan:migrate',
    'artisan:view:clear',
    'artisan:cache:clear',
    'artisan:config:cache',
    'artisan:route:cache',
    'artisan:optimize',
    'deploy:symlink',
    'deploy:unlock',
    'cleanup',
]);

before('deploy:symlink', 'artisan:migrate');

Output log

$ dep deploy -vvv
➤ Executing task deploy:prepare
[production] > echo $0
  SSH multiplexing initialization
mux_client_request_session: read from master failed: Connection reset by peer
Failed to connect to new control master
[production] < bash
[production] > if [ ! -d /var/www/domain ]; then mkdir -p /var/www/domain; fi
➤ Executing task deploy:failed
• done on [production]
✔ Ok [1ms]
➤ Executing task deploy:unlock
[production] > rm -f /var/www/domain/.dep/deploy.lock

  [RuntimeException]
  The command "ssh -A -o UserKnownHostsFile=/dev/null -o StrictHostKeyChecking=no -o ControlMaster=auto -o ControlPersist=5 -o ControlPath='~/.ssh/deployer_mux_root@remote_machine_ip:22' -F "C:\Users\EMKING/.ssh/config" -p "22" -i "C:\Users\

E
  MKING/.ssh/id_rsa" -t "root@remote_machine_ip" bash -s" failed.
  Exit Code: 255(Unknown error)
  Working directory: E:\User\meeveh
  Output:
  ================
  Error Output:
  ================
  Pseudo-terminal will not be allocated because stdin is not a terminal.
  mm_send_fd: sendmsg(2): Connection reset by peer
  mux_client_request_session: send fds failed

Exception trace:
 () at C:\Users\EMKING\AppData\Roaming\Composer\vendor\deployer\deployer\src\Server\Remote\NativeSsh.php:103
 Deployer\Server\Remote\NativeSsh->run() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\deployer\deployer\src\functions.php:318
 Deployer\run() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\deployer\deployer\recipe\deploy\lock.php:28
 Deployer\{closure}() at n/a:n/a

call_user_func() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\deployer\deployer\src\Task\Task.php:85
 Deployer\Task\Task->run() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\deployer\deployer\src\Executor\SeriesExecutor.php:40
 Deployer\Executor\SeriesExecutor->run() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\deployer\deployer\src\Console\TaskCommand.php:112
 Deployer\Console\TaskCommand->execute() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\symfony\console\Command\Command.php:262
 Symfony\Component\Console\Command\Command->run() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\symfony\console\Application.php:826
 Symfony\Component\Console\Application->doRunCommand() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\deployer\deployer\src\Console\Application.php:123
 Deployer\Console\Application->doRunCommand() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\symfony\console\Application.php:190
 Symfony\Component\Console\Application->doRun() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\symfony\console\Application.php:121
 Symfony\Component\Console\Application->run() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\deployer\deployer\src\Deployer.php:190
 Deployer\Deployer->run() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\deployer\deployer\bin\dep:120

deploy [-p|--parallel] [--no-hooks]

It's a bug in cygwin https://cygwin.com/ml/cygwin/2010-08/msg00072.html
Try bash on windows

@antonmedv Now i'm using open-ssh and it throws new error below:
> dep deploy -vvv
➤ Executing task deploy:prepare
[production] > echo $0
[production] < bash
[production] > if [ ! -d /var/www/domain ]; then mkdir -p /var/www/domain; fi
[production] > if [ ! -L /var/www/domain/current ] && [ -d /var/www/domain/current ]; then echo true; fi
[production] > cd /var/www/domain && if [ ! -d .dep ]; then mkdir .dep; fi
[production] > cd /var/www/domain && if [ ! -d releases ]; then mkdir releases; fi
[production] > cd /var/www/domain && if [ ! -d shared ]; then mkdir shared; fi
• done on [production]
✔ Ok [4s 912ms]
➤ Executing task deploy:lock
[production] > if [ -f /var/www/domain/.dep/deploy.lock ]; then echo 'true'; fi
[production] > touch /var/www/domain/.dep/deploy.lock
• done on [production]
✔ Ok [1s 633ms]
➤ Executing task deploy:release
[production] > cd /var/www/domain && (if [ -h release ]; then echo 'true'; fi)
[production] > cd /var/www/domain && ([ -d releases ] && [ "$(ls -A releases)" ] && echo "true" || echo "false")
[production] < true
[production] > cd /var/www/domain && (cd releases && ls -t -1 -d */)
[production] < 1/
[production] > cd /var/www/domain && (if [ -f .dep/releases ]; then echo "true"; fi)
[production] < true
[production] > cd /var/www/domain && (tail -n 15 .dep/releases)
[production] <<PHONE_NUMBER>2803,1
[production] <<PHONE_NUMBER>3338,1
[production] <<PHONE_NUMBER>4326,1
[production] <<PHONE_NUMBER>0357,1
[production] <<PHONE_NUMBER>4712,1
[production] > cd /var/www/domain && (if [ -d /var/www/domain/releases/2 ]; then echo 'true'; fi)
[production] > cd /var/www/domain && (date +"%Y%m%d%H%M%S")
[production] <<PHONE_NUMBER>5156
[production] > cd /var/www/domain && (echo '20170620095156,2' >> .dep/releases)
[production] > cd /var/www/domain && (mkdir /var/www/domain/releases/2)
[production] > cd /var/www/domain && (if [[ "$(man ln)" =~ "--relative" ]]; then echo "true"; fi)
[production] < true
[production] > cd /var/www/domain && (ln -nfs --relative /var/www/domain/releases/2 /var/www/domain/release)
➤ Executing task deploy:failed
• done on [production]
✔ Ok [1ms]
➤ Executing task deploy:unlock
[production] > rm -f /var/www/domain/.dep/deploy.lock
• done on [production]
✔ Ok [821ms]

  [RuntimeException]
  The command "ssh -A -o UserKnownHostsFile=/dev/null -o StrictHostKeyChecking=no -F "C:\Users\EMKING/.ssh/config" -p "22" -i "C:\Users\EMKING/.ssh/id_rsa" "root@remote_machine_ip" bash -s" failed.
  Exit Code: 255(Unknown error)
  Working directory: E:\User\meeveh
  Output:
  ================
  Error Output:
  ================
  Connection reset by remote_machine_ip port 22

Exception trace:
 () at C:\Users\EMKING\AppData\Roaming\Composer\vendor\deployer\deployer\src\Server\Remote\NativeSsh.php:103
 Deployer\Server\Remote\NativeSsh->run() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\deployer\deployer\src\functions.php:318
 Deployer\run() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\deployer\deployer\recipe\deploy\release.php:128
 Deployer\{closure}() at n/a:n/a
 call_user_func() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\deployer\deployer\src\Task\Task.php:85
 Deployer\Task\Task->run() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\deployer\deployer\src\Executor\SeriesExecutor.php:40
 Deployer\Executor\SeriesExecutor->run() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\deployer\deployer\src\Console\TaskCommand.php:103
 Deployer\Console\TaskCommand->execute() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\symfony\console\Command\Command.php:262
 Symfony\Component\Console\Command\Command->run() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\symfony\console\Application.php:826
 Symfony\Component\Console\Application->doRunCommand() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\deployer\deployer\src\Console\Application.php:123
 Deployer\Console\Application->doRunCommand() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\symfony\console\Application.php:190
 Symfony\Component\Console\Application->doRun() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\symfony\co

nsole\Application.php:121
 Symfony\Component\Console\Application->run() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\deployer\deployer\src\Deployer.php:190
 Deployer\Deployer->run() at C:\Users\EMKING\AppData\Roaming\Composer\vendor\deployer\deployer\bin\dep:120

deploy [-p|--parallel] [--no-hooks]

try this:
set('use_relative_symlinks', false);

@akoSalman fixed?)

I'm closing this issue as no activity for more then 2 weeks. I hope you found a solution (please, share with us).
If you still have any problems feel free to open a new issue.

In addition to @antonmedv's suggestion to use bash on windows (i.e. Windows Subsystem for Linux), which works, you can also make the Cygwin-caused mux_client_request_session problem go away by turning off ssh multiplexing.
i.e. in your deploy.php, change:
set('ssh_multiplexing', true);
to
set('ssh_multiplexing', false);
or if your version of deployer CLI supports it (I'm not seeing the -o option but since WSL is working I can't be bothered to dig) you can override config options from the command line, e.g.
dep deploy -o ssh_multiplexing=false ...

@antonmedv Thanks for saving my time. It worked.

This did the trick for me:
set('ssh_multiplexing', false);


Update package react@^17, <EMAIL_ADDRESS>to resolve conflict when run npm install.

Please update package react@^17, <EMAIL_ADDRESS>to resolve conflict when run npm install.

Any update on this? The react-native-svg version in the icons here is quite behind the updated version

+1 for this one.


Run Failure (Python Version)

Action Testing fails to run nGen
Current behavior
Failure to build and run nGen in Actions here:
https://github.com/NOAA-OWP/ngen/actions/runs/1027692345
Problem section:
-- Found Boost: /ngen/boost_1_72_0 (found suitable version "1.72.0", minimum required is "1.72.0")
1561
INFO NGEN_ACTIVATE_PYTHON not defined
1562
INFO Did not find 'PYTHON_INCLUDE_DIR' in ENV
1563
INFO Did not find 'PYTHON_LIBRARIES' in ENV
1564
CMake Error at /usr/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:165 (message):
1565
  Could NOT find Python: Found unsuitable version "3.6.8", but required is at
1566
  least "3.6.12" (found , found components: Interpreter)
1567
Call Stack (most recent call first):
1568
  /usr/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:456 (_FPHSA_FAILURE_MESSAGE)
1569
  /usr/share/cmake/Modules/FindPython/Support.cmake:2966 (find_package_handle_standard_args)
1570
  /usr/share/cmake/Modules/FindPython.cmake:393 (include)
1571
  CMakeLists.txt:130 (find_package)
1572

Expected behavior
nGen should build and run in GitHub Actions
Steps to replicate behavior (include URLs)

Run action

Screenshots

I can try and boost the version of python in the image, or run an update prior since it looks like just a minor version update with fix the issue.
However @robertbartel is that ENV something that just needs to be set in the action?

Not sure of best method with Docker, but I would recommend setting up a venv in the project root.

This has been fixed! Thanks!


IsExternalInit not generated correctly in multitargeting projects

Reproduction Steps
Make a project that targets netstandard2.1 and net8.0
<TargetFrameworks>netstandard2.1;net8.0</TargetFrameworks>

and add a record class to it with some members, or alternatively a class with required init properties.
Expected Behavior
IsExternalInits type should be included in the compiled assembly targeting netstandard2.1.
Actual Behavior
Compilation succeeds, but IsExternalInits type is not included in the compiled assembly targeting netstandard2.1.
System info
This section should contain useful info such as:

PolySharp NuGet version 1.14..1
Operating system version Pop!_OS (Ubuntu) 22.04
.NET SDK version 8.0.100
Runtime is Unity 2022.3.9f1 (implements net48 and netstandard2.1)

IsExternalInit must be emitted all the time, for all frameworks. Even if the framework already declares it.
The explanation of why is here: https://github.com/manuelroemer/IsExternalInit/issues/5

"IsExternalInit must be emitted all the time, for all frameworks."

This is not accurate. PolySharp generates the correct [TypeForwardedTo] in that case, instead of the polyfill. Not generating the polyfill is only a binary breaking changes if you don't generate the correct forwards, but PolySharp does that, so that's fine.

"Compilation succeeds, but IsExternalInits type is not included in the compiled assembly targeting netstandard2.1."

@Lordfirespeed I need more info from your end. First, I can't repro this:

Second, how is your project even compiling at all, if you're saying the .NET Standard 2.1 target doesn't have that polyfill?
Are you sure you're not looking at the wrong place or perhaps it was just VS not showing all generated files correctly?

@Sergio0694 You are quite right. Brilliant solution that I hadn't thought of so didn't check for. My apologies for spreading FUD.

All good, no worries! Just wanted to make sure there were no misunderstandings! 😊


Disabling all wireless features

Hello,
Is it possible to disable all the wireless terminal features?
I like the storage part of the mod but the wireless features are to OP for me.
Kind regards,

You can remove the recipes using a data pack or KubeJS.


pre existing values when adding public config

Why there are values when opening the dialog ?
They should be removed on mounting.

Screencast from 12-27-2023 11:09:32 AM.webm

They should be reset indeed.

It's not actually an issue.
what really happens if there's already public config will be rendered to user in dialog, so user can remove or update it.
what is shown in the video is the public config of my node.


Django version for install

I've been trying to install the app and am running into an issue about a Django version (in setup, I'm required to have Django version 3.0.7). I've tried to install this specific version with pip but I can't seem to get it. Of course, I can get a more recent version. Is there a particular reason for choosing this particular version or could we perhaps change this to (say)?
Django>=3.0.7

Hey @ben18785, which Django version you want for your required app??

Hey @ben18785, which Django version you want for your required app??

think this is resolved now?


fix(runtime-core): hydration should not be performed during the HMR reload process(fix #7706)

Close #7706.
During HMR reload, the div element is unmounted. However, in SSR mode, the mountComponent process does not remount the div element but hydrates it instead, causing the issue of the div element is deleted.

Thank you for your contribution! We have created a new PR to resolve issue #7706, making this PR redundant. Additionally, this PR contains incorrect modifications and lacks necessary test cases. Thank you for your understanding!


Reconfigure generated code to take patch customization classes

Right now, we just generate our clients and operation groups without extension points for customization classes (i.e., here).
To take users customization inputs, we want to make our final client / operation groups a combination of customization (if users have added customization classes) and generated code, with the customziation classes taking precedence. Here is the rough design that we've come up with. We want all of our clients + operation groups to function this way now, as a combination of handwritten customizations + generated.
If you'll notice, we include imports in the if TYPE_CHECKING section, this is so intellisense knows what's happening (for more info, see here). Thus, the import serializer will also need to be updated so it can serialize this import section.
We should also add tests here for common scenarios of how we want to modify generated code. I've included a couple here, please add more based off of your own personal experience working on webpubsub, and any more examples you can think of. We want good testing here, since we expect SDK authors to make use of this to customize

already implemented


Temp Tables not deleted in BulkInsertOrUpdate

This is the configuration I am using for the transaction. Notice that I have set UseTempDb to true in the attempt to tell SQL to drop the tables when the operation is complete or if any exception occurs.

I wrapped the BulkOperation in a transaction as I saw has been suggested in other related issues.

I have tracked the events related to one instance of this issue: the temporary table is created in my DB, the operation goes on, and at the end this table (TransportDocumentTempxxxx)
I enabled audit on my SQL server and this are the queries I believe are issued by the BulkInsertOrUpdate

I was wondering why the temp table is not being deleted after the operation is complete.
Is there any evident issue in how I'm calling the BulkOperation?
thank you,
Enrico

There is a note about UseTempDB in the ReadMe:

UseTempDB when set then BulkOperation has to be inside Transaction.

So even setting it won't have any effect.
Instead table is created ad regular, just with sufix 'Temp' in the name.
Still after operation is finished it executes Drop of that table.
Last query in your log result show the Drop command.
Try to run that sql command manually and see what happens.

There was some improvement made with this, check v7.0.4


CLI: task and generator events are not being logged

this has been fixed locally, creating this issue for tracking

should be fixed


ERFNet-CULane-PyTorch test erro

could you provide the torch and cuda version of the newest code?thanks a lot.
@cardwing

i've got my answer.torch == 1.3.0 and cudatoolkit == 9.2


2DSE: Add spiral mode for revolving shapes

Revolve shape is fantastic for round skirting but it will hardly work with upward or downward moving slopes. It would be great to have a vertical spiral movement option (perhaps the unused Y axis of the global pivot is useful for this).

Edit: Yes I noticed the stair&wall were misaligned, I fixed it now. 😋

The stair mode works:

The smooth mode works:

Well that solved that problem! 😁


Noncontiguous indices in ParNCMesh for quadratic prism mesh

I have been attempting to use prismatic meshes with ex6p, and have generated some meshes that appear to trigger issues within the ParNCMesh structures. These meshes are generated with gmsh, and are of a cylindrical cavity. Using these meshes with ex1p they run just fine, and using a single rank on 6p they are also fine, so the issue appears to be isolated to ParNCMesh and ParFiniteElementSpace when calling the ParNCMesh. These meshes are available on the branch cavity-prism-par-ncmesh-bug. This has a sister issue https://github.com/mfem/mfem/issues/3671 that I am also working on.
The mesh is p2_prism_ref1.msh, and uses a slightly finer mesh size on the extruded face. Running the command make -j6 ex6p && mpirun -np 2 ex6p -m ../../p2_prism_ref1.msh results in
Verification failed: (key >= 0) is false:
 --> entity not found.
 ... in function: const NCMesh::MeshId &mfem::NCMesh::NCList::LookUp(int, int *) const
 ... in file: /Users/hughcars/AWS/mfem/mesh/ncmesh.cpp:3455

coming from const MeshId &id = pncmesh->GetNCList(ri.entity).LookUp(ri.index); in NeighborRowMessage::Encode(int rank). Stepping through here, the issue appears to be that the conformal face indices are not contiguous, namely printing the entries of inv_index to terminal as they are added shows:
R0 inv_index[15] 36
R0 inv_index[16] 40
R0 inv_index[18] 44
R0 inv_index[19] 48

where R0 is rank 0, and the lack of 17 in the table causes the miss when 17 is requested later. confirming indices used to build the inv_index list appear to not match those requested later. Using the same mesh in ex1p and visualizing in Glvis looks just fine, so I have reason to believe the conformal mesh itself is ok, just the construction of the ncmesh from it is wrong.
Not had time to diagnose it more closely yet, as the other issue appears more serious, but any input would be appreciated.
Update: this appears to be possible to trigger using the other meshes too, in particular a linear mesh with a second order solution:
make ex6p -j6 && mpirun -np 2 ex6p -m ../../data/p1_prism.msh -o 2

will trigger what appears ostensibly the same error, missing the 5th key. This mesh is 8 triangular prisms stacked into a cuboid. Going deeper it appears to be ParFiniteElementSpace::UnpackDof that is returning these incorrect indices from the mixed faces or var-order space branch of the faces. So it is incorrectly deciphering a regular face index on prisms.

Running with make ex6p -j6 && mpirun -np 2 ex6p -m ../../data/p2_prism.msh -o 2 -h1 appears to adapt fine, but running without -h1 namely using the RT smoothed flux space, triggers errors of missing indices in the inv_index of NCMesh::NCList::LookUp(int, int*). So the issue appears to be related explicitly to RT meshes on prism elements of order 2 and greater. The failing index 5 appears to be generated in ParFiniteElementSpace::UnpackDof.

Closing this in favour of https://github.com/mfem/mfem/issues/3691 as these all appear related.


fix for travis build problems

currently the travis build is checking the master branch instead of the new commits it should be checking; this will hopefully fix the problem.

this passes on my branch but fails here, likely due to the broken build #126; going to merge here and merge #127 - hopefully this fixes everything.


proxyed streams have no sound when rtmp is used

Which version are you using?
v0.18.2
Which operating system are you using?

[ x ] Linux amd64 standard
[ ] Linux amd64 Docker
[ ] Linux arm64 standard
[ ] Linux arm64 Docker
[ ] Linux arm7 standard
[ ] Linux arm7 Docker
[ ] Linux arm6 standard
[ ] Linux arm6 Docker
[ ] Windows amd64 standard
[ ] Windows amd64 Docker (WSL backend)
[ ] macOS amd64 standard
[ ] macOS amd64 Docker
[ ] Other (please describe)

Describe the issue
proxyed streams have no sound when rtmp is used
Describe how to replicate the issue

start the server
publish with obs on the remote server (rtmp is used)
setup rtmp remote url as "source:" in config
read the stream with whatever you want

Did you attach the server logs?

no
Did you attach a network dump?

no

FIXED AS LAST UPDATE (0.18.5)


Add healthcheck scripts without modifying role

It would be nice if there were a way to add site-specific healthcheck scripts without having to modify the role.
What would be a good way to accomplish that? And if should work with ansible-pull too..

To clarify (I hope):
The problem here is to install custom scripts to /etc/nhc/scripts without having to modify this role.
The role can already define custom scripts, it just can't install them.
Personally I don't have a problem with storing the custom checks in this role but yeah some way to fetch files from the install node and installing them in specific locations? Perhaps a more generic role rather than just for NHC could be made?

Yes, exactly.
Would it work to store extra scripts in some configurable folder, then copy them when running ansible in push mode, and for pull mode one could copy them to the install node and then get_url them?

I think it would work, two tasks is enough?

Copying files to /etc/nhc/scripts and /var/www/nhc/ is the same, one could set the path where they are stored differently with group_vars .
ansible-pull get_urls based on a list defined in group_vars

This role copies a list of files https://github.com/CSC-IT-Center-for-Science/ansible-role-nrpe-plugins

Do we still think a general "ansible-role-copy-files-here-and-there" is a good idea?
If so, can I get some role name suggestions?
Do you have any more other use cases right now besides NHC? Maybe we could make an ansible-role-nhc-plugins to make it clear what the role is doing.

I don't think I'll make a "copy files here and there" role.
IMO the best way to distribute these custom plugins would be with a package manager.
Anyway started working on https://github.com/CSC-IT-Center-for-Science/ansible-role-nhc-scripts

Remaining task: add the role to FGCI playbooks. Doing that in December to give sites time to update their fgci-ansible.

Added to FGCI playbooks.


Document what's happening with derives on i2c::Error

Thanks! That will help next time I stumble across this and wonder why the derive is cfg'd out.


[video_player_web] Add cookie support (2/3)

Part 2 of #2882 - see main description there.

Add included cookies, if any, to the document when initializing playback
Bump pubspec

Achieved in #3671!


Would love a working example.

Would love a working example as reference.
Been trying to get mine code working but it doesn't seem to work show anything in page.
It shows up the tag of
tag of <adsbygoogle/>
Console error:
[Vue warn]: Unknown custom element: <adsbygoogle> - did you register the component correctly? For recursive components, make sure to provide the "name" option.

but nothing to show for it.
Thanks!

This question is available on Nuxt.js community (#c3)

There is a simple example under example directory


Visual Studio 2015 not supported?

build log
https://dl.dropboxusercontent.com/u/12679384/web/rxcpp.txt
A lot of errors:
69>------ Skipped Build: Project: ALL_BUILD, Configuration: Debug Win32 ------
69>Project not selected to build for this solution configuration
========== Build: 2 succeeded, 65 failed, 0 up-to-date, 2 skipped ==========

Thank you for bringing this to my attention!
vs2015 is supported, but I probably have not tested against update #3.
What update are you using?

latest VS Update3
rxcpp from master

painful to track down, but working again now. Thanks!

thanks!


Fix incompatibility with vim-sleuth

When vim-sleuth plugin is used, selecting a file to edit from fuzzyy results in error E930: Cannot use :redir inside execute(), e.g.
Error detected while processing function <SNR>153_MenuFilter[28]..<SNR>153_GeneralPopupCallback[17]..<SNR>61_Select[5]..BufRead Autocommands for "*"..function <SNR>115_AutoInit[1]..<SNR>115_Init[15]..<SNR
>115_DetectDeclared[21]..<SNR>115_ModelineOptions[2]..<SNR>115_Capture:
line    1:
E930: Cannot use :redir inside execute()
Error detected while processing function <SNR>153_MenuFilter[28]..<SNR>153_GeneralPopupCallback[17]..<SNR>61_Select[5]..BufRead Autocommands for "*"..function <SNR>115_AutoInit[1]..<SNR>115_Init[15]..<SNR
>115_DetectDeclared:
line   21:
E712: Argument of extend() must be a List or Dictionary

Fix using :execute rather than execute(), seems to work fine, can't find anything in the commit history to suggest the function was needed.
Alternative solution would be to use execute('silent! edit ' .. path)

Thanks for merging :-)
I have a few other suggestions, which I can add to separate PRs or wrap up together, but maybe you could let me know what you think first...

Use --smart-case by default with rg and ag, this seems a logical expectation for a "fuzzy" finder to me
Remove --columnoption from rg and ag, unnecessary as already applied via use of--vimgrep` option
Increase ASYNC_STEP from 1,000  to 10,000 to improve performance, or at least perceived performance
Rename ag.vim to grep.vim, it covers multiple grep style programs and is triggered via :FuzzyGrep command
Allow "grep" command to be overridden somehow, possibly via a single g:fuzzyy_grep_command variable?
Allow grep command to be customised

btw, allowing something like g:fuzzyy_grep_command suggests something like g:fuzzyy_files_command would also be required for consistency. This doesn't seem unreasonable to me, allow Fuzzyy try to do the right thing by default for both grep and files, but if you have custom requirements (I do, I use a global .rgignore file), also allow users to override the defaults.

FYI, started another PR here: https://github.com/Donaldttt/fuzzyy/pull/49


Could you provide the link to the paper?

Hi,
I was wondering if the paper corresponding to this data is published yet?
Sorry, I accidentally closed the issue.
Best wishes,
Pejvak.

Hi, not sure if still needed, but in case here's the link to the paper!
https://www.nature.com/articles/s41586-019-0879-y
Best wishes,
Giulia


Re-add selfbot support

Summary
Recently all Discord frameworks removed support for selfbots. I want it back!
What is the feature request for?
The core library
The Problem
The problem is that when doing client.run(TOKEN, bot=False) the bot keyword no longer works because all frameworks decided to remove support for selfbots. I know selfbots are not permitted by TOS but come on, just re-add support.
The Ideal Solution
Re-adding support
The Current Solution
Once again, re-adding support
Additional Context
N/A

Self-botting is against Discord's Terms of Services. Which is why every library has removed that. It will not be added back, as, once again, it is against Discord's Terms of Service.

"I know selfbots are not permitted by TOS but come on, just re-add support."
Whatever, maybe I'll make my own fork.

In order to be listed among Discord API Community Resources and have a Verified and Partnered support server, we need to abide by the TOS.


Is there any implementation to run  parallelly two or multiple independent keywords in a single test case in robot framework?

Unfortunately not. This is a recognized missing feature, see #1791 and #1792, but due to the shared internal state implementing it isn't possible at the moment. Hopefully internal changes in RF 4.0 will make that easier in the future, but I cannot give any estimation when this feature would be supported. At the moment the way to run keywords in parallel is handling that on library level.

Notice also that this issue tracker is for bug reports and concrete enhancement requests. Questions should be sent to other forums such as mailing lists or Slack.

We use Robotframework in my organization and we need support to run ROBOT keywords to run in parallel with multithreading or any other way in a single test case.


Configuration problems running aoc-dev.

Hi!
I'm trying to set this up under MacOS. After forking this, installing cabal and Haskell, installing homebrew, installing llvm, setting paths appropriately, if I run cabal run aoc-dev I get the following error:
app/AOC.hs:1:1: error:
    File name does not match module name:
    Saw: ‘Main’
    Expected: ‘AOC’
  |
1 | {-# LANGUAGE OverloadedStrings              #-}
  | ^

(This is almost certainly something beginner-y, still trying to figure out how stack works. I played around with a few random "put a module incantation" somewhere attempts, but nothing I tried so far helped.)

As an update, I 'fixed' this issue by moving app/AOC.hs to app/Main.hs, but I'm not sure this is the best possible solution.


Integrate AWS pipeline

Introduce uploading user requests to AWS pipeline for multispeaker without deepgram.

Any idea what this error is?
~/chaincode/yt2btc  ( payload_params )  > tstbtc '~/Downloads/ajonas-taproot-channel.mp3' 'chaincode-labs/podcast' -t 'Taproot Channels with Elle and Olli' -c 'podcast' --diarize
/usr/local/lib/python3.11/site-packages/whisper/timing.py:57: NumbaDeprecationWarning: The 'nopython' keyword argument was not supplied to the 'numba.jit' decorator. The implicit default value for this argument is currently False, but it will be changed to True in Numba 0.59.0. See https://numba.readthedocs.io/en/stable/reference/deprecation.html#deprecation-of-object-mode-fall-back-behaviour-when-using-jit for details.
  @numba.jit

Ah it's just a warning in one of the libraries whisper uses. I don't think we should worry about it.

Closing this as we are not currently planning to make this integration. Additionally, if we decide to proceed in the future, the codebase has been refactored, so the approach may differ.


Parquet Page Index Reader Assumes Consecutive Offsets

Describe the bug

The parquet page index reading logic assumes that the index is contiguous, and in the case of the offset index it assumes it is in order.
The specification strongly suggests this is a valid assumption, however, it never explicitly specifies it. We should probably be defensive and assume that there is a writer out there that doesn't do this.
To Reproduce

Expected behavior

Additional context

label_issue.py automatically added labels {'parquet'} from #4151


WIP: Chef 14 support

Signed-off-by: JJ Asghar<EMAIL_ADDRESS>

To test this you'll need to do something like the following:
git clone https://github.com/chef-partners/knife-oraclecloud.git
cd knife-oraclecloud
bundle install
bundle exec knife oraclecloud server list

That should list the machines with Chef 14.


Fix for issue #3

Disallow rdflib 6.1.* because it has 40+ default prefixes pre-installed.
Reported issue to rdflib community w/ https://github.com/RDFLib/rdflib/issues/1676

How do we communicate this to folks?   Take LinkML as an example -- if it imports 6.1.1, any json-ld output will have 40+ namespaces that were not intended.   The good news, I guess, is it sort of forces folks to commit to a lot of namespace prefixes.   I'm not sure, however, that folks are 100% ready to do that yet.
If we don't put it into the shim, I'm going to have to go back over a number of projects and exclude 6.1.1 individually -- they don't pass unit tests the way things are right now.
I guess that, IF the rdflib people want to stick to this approach, we can fatten up the shim to allow a parameter that, when set, overrides the Graph constructor and undoes all the work.

On Wed, Jan 19, 2022 at 2:16 PM Harold Solbrig @.***>
wrote:

How do we communicate this to folks? Take LinkML as an example -- if it
imports 6.1.1, any json-ld output will have 40+ namespaces that were not
intended. The good news, I guess, is it sort of forces folks to commit to a
lot of namespace prefixes. I'm not sure, however, that folks are 100% ready
to do that yet.

Let's ask some others!
For me this is a minor aesthetic concern, and also linkml to jsonld is not
pulley production ready yet - and it looks like the rdflib folks are on
this https://github.com/RDFLib/rdflib/issues/1676

If we don't put it into the shim, I'm going to have to go back over a
number of projects and exclude 6.1.1 individually -- they don't pass unit
tests the way things are right now.

ah, that is a concern - so I guess the tests are doing a strict string
level comparison? or a json structure level comparison? at the rdf level,
the results are identical, yep?
But maybe the appropriate place to put the !=6.1 restriction is in those
downstream projects, and keep rdflib_shim more flexible? I know that's more
work...

I guess that, IF the rdflib people want to stick to this approach, we can
fatten up the shim to allow a parameter that, when set, overrides the
Graph constructor and undoes all the work.

I don't think they do, see:
https://github.com/RDFLib/rdflib/issues/1679
hopefully it will be fixed soon

—
Reply to this email directly, view it on GitHub
https://github.com/hsolbrig/rdflib-shim/pull/4#issuecomment-1016922865,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAMMOJOXAN435SLQDVOVHTUW4Z5BANCNFSM5MFICEJQ
.
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.
You are receiving this because your review was requested.Message ID:
@.***>


middleware/error cleaner log message for "no next middleware"

When zones are requested, but they are not available the log message just says there is no middleware next. This should be clearer for the end user. Aka "No middleware was configured to process the request" or something.

Fixed with #462


Stop process and exit with error code 1 on fail

Description
detox-server info 12:36:32: server listening on localhost:50682...
(node:39340) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 2): Error: app binary not found at '/Users/admin/builds/999529a4/0/tipsi_frontend/tipsi-react-app/ios/build/Build/Products/Release-iphonesimulator/tipsi.app', did you build it?
(node:39340) DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Currently, detox doesn't fail. This freezes our CI server. For now, I always need to check that tests have successfully started.
Steps to Reproduce
Run detox with incorrect binary path
Node, Device, Xcode and macOS Versions
No matter

How do you run detox ? What command do you type?
I'm running detox test -c ios.sim.release on Node 8.1.2, with a wrong path in my config binaryPath: wrong/path and it fails instantly.

@rotemmiz good question.
I have next command:
"test:ios-detox": "INSTRUMENT=detox tape -r ./__tests__/e2e/setup ./__tests__/e2e/runner",

Setup contains next files:
require('babel-register')({
  plugins: [['tipsi-flavors', { env: 'APP_ID' }]],
  ignore: /node_modules\/(?!tipsi_api)/,
  cache: false,
})
require('babel-polyfill')

and
import ignore from 'require-ignore'

ignore.install(['png'])

Runner contains next files:
// runnerDetox.js
import tape from 'tape'
import tapDiff from 'tap-diff'
import detox from 'detox'
import pkg from '../../../package.json'
import runTestSuites from '../suites'

function waitTapeFinish() {
  return new Promise(resolve => tape.onFinish(resolve))
}

export default async function runnerDetox() {
  await detox.init(pkg.detox)

  tape.createStream()
    .pipe(tapDiff())
    .pipe(process.stdout)

  runTestSuites()

  await waitTapeFinish()

  await detox.cleanup()

  process.exit(0)
}

So, suites - it's only an Array<string> with filenames of test suites

For my concern I had to add "--exit" in mocha.opts to get a proper exit at the end of successful tests.

Controlling the process lifecycle and exit codes is the job of a test runner, Detox is not a test runner. Test runners can either exit on the first error or continue running all tests and report on an error at the end, If we deliberately call exit(1) inside detox, it will break the test runner's lifecycle, and exit anyway.

Any idea why "something" is still open? For example jest (and I guess mocha is doing the same) are not exiting automatically because there is still something runing. Adding things like --exit is forcing the runner to exit, but it should be natural...


On macOS automatically switch to new default sound output

Is your feature request related to a problem? Please describe.
When I connect my Bluetooth headphones I expect spotify-player to switch to them automatically.
Describe the solution you'd like
When I connect my Bluetooth headphones I expect spotify-player to switch to them automatically.
Describe alternatives you've considered
Keep existing behavior. Right now I have to close spotify-player and open it again after connecting to BT headphones (old way). Another way would be to use RestartIntegratedClient, which I just discovered.
Additional context
Once again, thanks for the spotify-player!

RestartIntegratedClient is one way that I usually use when connecting to a new bluetooth device. The command doesn't work sometimes in the first try (maybe because it takes time to retrieve the newly connected bluetooth) and I need to re-run the command.
I don't know if it's possible to listen to new bluetooth device event and implement the automatically switching logic. Will take a look.

I didn't even expect this project to support wireless devices lol.
Seems like device_type = "wireless" doesn't do anything.

I didn't even expect this project to support wireless devices lol.

Not sure what you mean by supporting wireless devices. device_type only changes how the client is displayed in the official application. A list of options for device_type can be found in https://github.com/librespot-org/librespot/wiki/Options

I didn't even expect this project to support wireless devices lol.

Not sure what you mean by supporting wireless devices. device_type only changes how the client is displayed in the official application. A list of options for device_type can be found in https://github.com/librespot-org/librespot/wiki/Options

How to setup wireless connection?

I'll assume you mean hearing sound from a wireless device (bluetooth headphone). If you install the app with streaming feature, it should use the default output device on startup (in this case the bluetooth headphone), so no need to setup anything.

Yep the problem what u got is, that u don't have probably connected properly wireless hw to your Mac or u don't have Spotify daemon or u don't have opened Spotify (the official one) + Spotify_player.
I got m1 with the newest OS and everything works correctly.
brew info spotifyd
brew services list | grep spotifyd
this should indicates if there is even chance to connect into spotify

@aome510 from what I've gathered this is the issue with audio backends.
Spotifyd folks mention librespot as being the issue.
Librespot folks mention RustAudio/rodio as source of the problem:
https://github.com/librespot-org/librespot/issues/547
I wonder if we could implement some workaround - detecting if default output device changes and automatically restarting streaming service (RestartIntegratedClient) - or if it is worth in the first place.

This seems to be handled automatically now with new librespot v0.5.0 release


Add License File to Repository

Task

Choose an appropriate open-source license.
Add the LICENSE file to the root of the repository.
Update README.md to include a reference to the license.

@soham0005 Please assign this task to me with all the required labels attached to it.

Hello @rahulgithub-web , this issue is already being raised, do consider raising another issue or contributing to other issue.
Happy Coding!


using constant instead of utf8 string in heron package

As #1598 mentioned, this PR is trying to use constant instead of UTF-8 string on Java side in heron package. I've just modify the code in heron package, the contrib package is not included. If so, I can modify them as well. I'm not sure if I've found all the UTF-8 strings in heron package. Please let me know if I have any omissions.
The key changes are following:

remove import java.nio.charset.Charset and add java.nio.charset.StandardCharsets
replace Charset.forName("UTF-8") to StandardCharsets.UTF_8

There are four classes included:

heron/spi/utils/SchedulerUtils.java
heron/scheduler/aurora/AuroraScheduler.java
heron/spi/utils/ShellUtils.java
heron/integration_test/common/HdfsHelper.java

👍


25 detect sensitive fields

Updates for detecting sensitive fields: https://github.com/SuffolkLITLab/RateMyPDF/issues/25

Oh, didn't expect it would publish the package on a PR branch, especially w/ failing tests. I used 0.3.0a2 as the version, so doubt anyone will grab this. Not sure what the status of 0.3.0 is... Seems like we're in some sort of alpha still?
Code in docx_wrangling & pdf_wrangling is causing the mypy failures. Didn't make any changes to those files. Seems like the changes are from the pdf_context_extract merge... Any ideas if those are crucial and need fixing? I'm not familiar w/ mypy but I can dig into it if it's a priority.

That GH action is a bit misleading, nothing is published until you make a GH tag. The action still runs to make sure it can package correctly, but it doesn't publish the new version (you can see that here). You can check https://pypi.org/project/formfyxer/#history to see that there's not a new version there. IMO 0.3.0a2 is fine, it still need more testing to make sure it can be used as a dependency without issue (i.e. the problems we had when redeploying RateMyPDF were all issues with FormFyxer).
It'd be nice if you could look into the mypy issues, but if you can't, I can fix them this weekend. Those issues pop up because our version of mypy increments, and it gets better at finding mismatching types that could cause issues.

So mypy on main and #137 is fixed. Looking a bit closer, I don't think this PR needs #137 though? @codestronger, can you confirm that and if so, drop or revert https://github.com/SuffolkLITLab/FormFyxer/pull/134/commits/2c01ae08bf010dea6b45e857a7bfb39110f578af and https://github.com/SuffolkLITLab/FormFyxer/pull/134/commits/9bfea3389e16bf25a9c8b312202a11dacf84965e from this PR, i.e. https://git-scm.com/book/en/v2/Git-Tools-Rewriting-History?

So mypy on main and #137 is fixed. Looking a bit closer, I don't think this PR needs #137 though? @codestronger, can you confirm that and if so, drop or revert 2c01ae0 and 9bfea33 from this PR, i.e. https://git-scm.com/book/en/v2/Git-Tools-Rewriting-History?

Sounds good. I can rebase this PR on the current main. @nonprofittechy merged the pdf_context_extract branch back in Feb and it's been live as far as I'm aware. Others might have pulled it down too. Was it safe to rewrite it out of the history? At minimum, we should merge the new version of the pdf_context_extract before deploying my changes, so that we don't lose any functionality from a deploy.

I don't remember why I merged this branch in February, unfortunately.

I don't remember why I merged this branch in February, unfortunately.

@BryceStevenWilley I'll proceed with the plan that assumes pdf_context_extract was an accidental merge. We discussed the various downstream users of FormFyxer and as far as we can tell, no one is depending on the features in that branch.
I also plan to officially take the 0.3.0 version number for this branch. Will also review the dependency updates I made since that was done to bring in everything being used by the code that was deployed. Now that we've separated out pdf_context_extract, maybe we don't need all of those yet.

Diff looks much better to me! +1 for changing to 0.3.01, and +1 for merging.

Will also review the dependency updates I made since that was done to bring in everything being used by the code that was deployed

I think those are still necessary, those dependencies started being used in an earlier commit I think.
Footnotes

We were using alphas because this package has been notoriously difficult to use as a dependency, and without proper testing it can cause issues. I think these changes will be tested enough at this point. ↩

Ah, okay! I'll leave the dependencies alone then. In any case, we'll need them in the future.


Exact coverage calculation fails in the self.coverage_and_or_pdf_scan method

When  exact_coverage_calculation is set to true, an error occurs because the number of pdf_evals obtained from  get_multiresolution_evals does not correspond to a fixed number of pixels associated with a given nside in HEALPix. This results in an error,  as seen.

can you use a workaround for now? I have this function fixed in a version on my local machine, but it is not pushed yet


Background-Attachment: Fixed on block-level elements

I've been working with Backstretch for the past two days and I really enjoy the plugin. It's well documented and works well.
There's one thing I haven't been able to master: getting Backstretch to be a fixed position on block-level elements. It's fixed when I use it as a body background, but not when I use it on block level elements. I've attempted this solution (https://github.com/srobbin/jquery-backstretch/pull/93) to no avail.
I've put together a JSFiddle with my test code here: http://jsfiddle.net/pixelsmith/bC4EF/
I'm working with the latest version of Backstretch and jQuery.
Any help would be greatly appreciated,
A.

I do not think this is possible.. I have been wanting to do this for a long time, hell i have been trying to do it all day but it is simply not possible... I have tried everything from different positioning to methods i didnt even know exist in web development like getting each element of a parent and manipulating its margin with js so it does not change its position from the top and has z-index of 0, everything else beneath it has z-index of 1. It actually worked but broke when you refresh the page at a certian scroll level, among other bugs. I really need a framework for these kinds of things because if microsoft wants us to quit java and start working on progressive web apps, they should make stuff like this happen, although i dont think it is even possible to do with java in a mobile app unless you work with excess libraries.

@bobbyschultz
I tried a similar method with jQuery, doesnt seem to work out for me. Have you tested that on a server and under different conditions?


GPT whants to send image but cant

GPT thinks the links are correct but they just return "Image not found" or something like that. Is there any other way GPT could send images? Thanks for your help!

GPT can't send images and will make stuff up when it doesn't know what to say. It's just how completion models work.


Fix mjData comment on pluginstate

It seems there is no npluginstate, just nplugin, I checked the code, seems to be what it means.

This PR is not correct. npluginstate is defined here

Thanks for the info! Will look into what I can do on my end.


Is there any way to extract all edges crossed by the geodesic path through the mesh?

Hi, thanks so much for the awesome work and code you've shared! It's been really helpful for my project.
I was wondering if there is a function or a way to retrieve the (extrinsic) edges that the geodesic path intersects on the original mesh.
Currently, I am calling the following code:
std::unique_ptr<FlipEdgeNetwork> edgeNetwork;
edgeNetwork->paths;
for (auto& epPtr : edgeNetwork->paths) {
  FlipEdgePath& path = *epPtr;
  std::vector<Halfedge> heList = path.getHalfedgeList();
}

However, it seems that the heList contains information about the half-edges of the intrinsic triangulation, rather than the extrinsic half-edges of the original mesh.
Could you please suggest a solution or function that could help me retrieve the extrinsic edge information efficiently? I would greatly appreciate any guidance you could provide!

Oh, I found the function std::vector<std::vector<SurfacePoint>> FlipEdgeNetwork::getPathPolyline(), which provides the correspondence btw the geodesic path and the extrinsic mesh elements.
By using the type information in the SurfacePoint class, I can retrieve all intersected edges on the original mesh.
Great work and excellent correspondence!


how to build wasm from rust with shared-memory support

I tried to a wasm module from C and Rust with shared-memory support, and was able to build it from C with  below cmd and options.
/opt/wasi-sdk/bin/clang     \
    --target=wasm32-wasi -v -O0 -z stack-size=32768 -Wl,--initial-memory=65536 \
    --sysroot=/opt/wasi-sdk/share/wasi-sysroot  \
    -Wl,--strip-all,--no-entry -nostdlib -pthread \
    -Wl,--shared-memory,--max-memory=131072      \
    -Wl,--export=foo\
    -Wl,--export=__heap_base,--export=__data_end    \
    -Wl,--export=__wasm_call_ctors  \
    -Wl,--allow-undefined \
    -o ${WASM_FILE_NAME} main.c

But I got some problem to build it from rust.
I converted the above cmd options to rustflags as blow, but it looks cargo doesn't know codegen options "sysroot", "-pthread", and "-nostdlib".
Can you suggest how to set these flags for rust?
[build]
rustflags = [
  "-C", "link-arg=-zstack-size=32768",
  "-C", "link-arg=--initial-memory=65536",
  "-C", "link-arg=--strip-all",
  "-C", "link-arg=--no-entry",
  "-C", "sysroot=/opt/wasi-sdk/share/wasi-sysroot",
  "-C", "-pthread",
  "-C", "-nostdlib",
  "-C", "link-arg=--shared-memory",
  "-C", "link-arg=--max-memory=131072",
  "-C", "link-arg=--export=foo",
  "-C", "link-arg=--export=__heap_base",
  "-C", "link-arg=--export=__data_end",
  "-C", "link-arg=--export=__wasm_call_ctors",
  "-C", "link-arg=--allow-undefined",
]

Hi, maybe link-arg=--shared-memory and some others are enough to enabled the shared memory, could you try removing the following flags:
  "-C", "link-arg=--no-entry",
  "-C", "sysroot=/opt/wasi-sdk/share/wasi-sysroot",
  "-C", "-pthread",
  "-C", "-nostdlib",

Hi, thanks for your suggestion, but this actually gives another error,
  = note: rust-lld: error: --shared-memory is disallowed by std-056ca9b4df39b1a5.std.078b28f0-cgu.0.rcgu.o because it was not compiled with 'atomics' or 'bulk-memory' features.
          rust-lld: error: 'atomics' feature must be used in order to use shared memory

it looks link-arg=--shared-memory has to be used together with multi-thread enabled, but not sure how to add -pthread flag to rustflag.
"-C", "-pthread", or "-C", "link-arg=-pthread", is not allowed with error error: unknown codegen option: `-pthread`  or = note: rust-lld: error: unknown argument: -pthread

Could you try adding: "-C", "target-feature=+atomics,+mutable-globals,+bulk-memory"?
Refer to https://github.com/Amanieu/parking_lot/issues/308


What if I do not have access to a GPU?

Use Google Colab as a stand-in for toy experiments first. If the idea works well and need more stable computational resources we will discuss with you and give you extra assistance then.


chore: fix wiki link in README

The link does correctly redirect, but makes more sense to have the current final URL

Thanks, but I'll leave it as it is for now because it needs an overhaul anyway.


Did a lot of hard work on installation, finally the editor keeps crashing soon i open/create a new file on arch linux. I am using 64bit arch linux with kernel 4.4.5-1

Prerequisites

[ ] Can you reproduce the problem in safe mode?
[ ] Are you running the latest version of Atom?
[ ] Did you check the debugging guide?
[ ] Did you check the FAQs on Discuss?
[ ] Are you reporting to the correct repository?
[ ] Did you perform a cursory search to see if your bug or enhancement is already reported?

For more information on how to write a good bug report or enhancement request, see the CONTRIBUTING guide.
Description
[Description of the bug or feature]
Steps to Reproduce

[First Step]
[Second Step]
[and so on...]

Expected behavior: [What you expected to happen]
Actual behavior: [What actually happened]
Versions
You can get this information from executing atom --version and apm --version at the command line. Also, please include the OS and what version of the OS you're running.

This is most probably #11198.

Thanks @nackd.

thanks @nackd


Crypter window not scrollable

Can't scroll the crypto details when they overflow (they're just hidden).

fixed


Correct typos

Correct typos：挂在->挂载

/lgtm
/approve
This file is still in sync. Typo introduced very recently.


Crashed on Windows 11

Windows Version: 22621.963
My App crashes when I run it on Windows 11 with the error:
Error waiting for a debug connection: The log reader stopped unexpectedly, or never started.

Error launching application on Windows.

This does not happen on Windows 10. When I remove the added lines in /windows/runner/main.cpp the App works (but of course, without this package). When I execute the compiled .exe of my App, It works fine on Windows 10 but instantly closes on Windows 11.

With "the added lines" I mean the 2 lines required for the package. This happens on every Windows 11 PC I own. I even tried reinstalling Windows but that didn't help either. You can find my repo with the problem here.

Ok, after many hours debugging I figured it out. First, create a new Flutter project and see if it works in the new project. If it doesn, the the issue is within your project. I cleared the cache flutter clean, deleted the windows model and created a new one flutter create -platform=windows .  but nothing worked. I then copied the pubspec.yaml from the new project into the one that was not working, saved it so flutter could run the new pubspec.yaml code. I then returned back to the old pubspec.yaml and it worked =]

any updates on this?

Happening to me as well. Probably an issue with Windows 11
bitsdojo_window: ^0.1.5

With "the added lines" I mean the 2 lines required for the package. This happens on every Windows 11 PC I own. I even tried reinstalling Windows but that didn't help either. You can find my repo with the problem here.

Cloned your repo and tried running the app. Seems like there is a crash related to the flutter_acrylic package. Removing these lines related to the flutter_acrylic package made the app run without any error.
await Window.setEffect(
    effect: WindowEffect.transparent,
    color: darkMode! ? const Color(0xCC222222) : const Color(0xCCDDDDDD),
  )

Happening to me as well. Probably an issue with Windows 11
bitsdojo_window: ^0.1.5

Tried it on my Windows 11 machine but everything ran ok. Do you have a sample repo that I could try?

OS information：
版本        Windows 11 家庭中文版 (Chinese Home)
版本        21H2
安装日期        2022/8/6
操作系统版本        22000.1936
体验        Windows 功能体验包 1000.22001.1000.0

any update on this?

any update on this?

I think a windows update and/or package update fixed the issue for me. It doesn't crash anymore.

Idk why but I am facing this issue right now, I am on windows 11 canary version

Yes, I tried that. But now when I removed this complete package implementation, the app is still not opening

Just made a new project, release build worked. Then I copied my main project's dependencies and built, didn't open.
pubspec.yaml
name: test
description: "A new Flutter project."
# The following line prevents the package from being accidentally published to
# pub.dev using `flutter pub publish`. This is preferred for private packages.
publish_to: 'none' # Remove this line if you wish to publish to pub.dev

# The following defines the version and build number for your application.
# A version number is three numbers separated by dots, like 1.2.43
# followed by an optional build number separated by a +.
# Both the version and the builder number may be overridden in flutter
# build by specifying --build-name and --build-number, respectively.
# In Android, build-name is used as versionName while build-number used as versionCode.
# Read more about Android versioning at https://developer.android.com/studio/publish/versioning
# In iOS, build-name is used as CFBundleShortVersionString while build-number is used as CFBundleVersion.
# Read more about iOS versioning at
# https://developer.apple.com/library/archive/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html
# In Windows, build-name is used as the major, minor, and patch parts
# of the product and file versions while build-number is used as the build suffix.
version: 1.0.0+1

environment:
  sdk: '>=3.4.1 <4.0.0'

# Dependencies specify other packages that your package needs in order to work.
# To automatically upgrade your package dependencies to the latest versions
# consider running `flutter pub upgrade --major-versions`. Alternatively,
# dependencies can be manually updated by changing the version numbers below to
# the latest version available on pub.dev. To see which dependencies have newer
# versions available, run `flutter pub outdated`.
dependencies:
  flutter:
    sdk: flutter

  # The following adds the Cupertino Icons font to your application.
  # Use with the CupertinoIcons class for iOS style icons.
  cupertino_icons: ^1.0.6
  logger: ^2.3.0
  get: ^4.6.6
  shared_preferences: ^2.2.3
  google_fonts: ^6.2.1
  dio: ^5.4.3+1
  hive: ^2.2.3
  hive_flutter: ^1.1.0
  flutter_svg: ^2.0.10+1
  cached_network_image: ^3.3.1
  intl: ^0.19.0
  expandable: ^5.0.1
  media_kit: ^1.1.10+1
  media_kit_libs_video: ^1.0.4
  media_kit_video: ^1.2.4
  flutter_audio_waveforms: ^1.2.1+8
  audioplayers: ^6.0.0
  http: ^1.2.1
  stomp_dart_client: ^2.0.0
  emoji_picker_flutter: ^2.2.0
  styled_text: ^8.1.0
  easy_rich_text: ^2.1.0
  flutter_markdown: ^0.7.2+1
  url_launcher: ^6.3.0
  enough_giphy: ^0.2.1
  file_picker: ^8.0.5
  desktop_drop: ^0.4.4
  video_thumbnail: ^0.5.3

dev_dependencies:
  flutter_test:
    sdk: flutter

  # The "flutter_lints" package below contains a set of recommended lints to
  # encourage good coding practices. The lint set provided by the package is
  # activated in the `analysis_options.yaml` file located at the root of your
  # package. See that file for information about deactivating specific lint
  # rules and activating additional ones.
  flutter_lints: ^3.0.0

# For information on the generic Dart part of this file, see the
# following page: https://dart.dev/tools/pub/pubspec

# The following section is specific to Flutter packages.
flutter:

  # The following line ensures that the Material Icons font is
  # included with your application, so that you can use the icons in
  # the material Icons class.
  uses-material-design: true

  # To add assets to your application, add an assets section, like this:
  # assets:
  #   - images/a_dot_burr.jpeg
  #   - images/a_dot_ham.jpeg

  # An image asset can refer to one or more resolution-specific "variants", see
  # https://flutter.dev/assets-and-images/#resolution-aware

  # For details regarding adding assets from package dependencies, see
  # https://flutter.dev/assets-and-images/#from-packages

  # To add custom fonts to your application, add a fonts section here,
  # in this "flutter" section. Each entry in this list should have a
  # "family" key with the font family name, and a "fonts" key with a
  # list giving the asset and other descriptors for the font. For
  # example:
  # fonts:
  #   - family: Schyler
  #     fonts:
  #       - asset: fonts/Schyler-Regular.ttf
  #       - asset: fonts/Schyler-Italic.ttf
  #         style: italic
  #   - family: Trajan Pro
  #     fonts:
  #       - asset: fonts/TrajanPro.ttf
  #       - asset: fonts/TrajanPro_Bold.ttf
  #         weight: 700
  #
  # For details regarding fonts from package dependencies,
  # see https://flutter.dev/custom-fonts/#from-packages

If this doesn't work, try using a different computer or use a virtual machine and see if it works there.

ok

oh my god, the culprit was media_kit, removing that made it work.
I literally just wasted an entire day on this.


Rundeck integration into pan-fca

What Changes are taking place?
This change incorporates rundeck into the pan-fca container.
The rundeck implementation is generic to the master branch (It does not have any influence on the current processes and procedures that are described in the documentation)
This PR does not contain the ported Example Files (Azure and AWS) into the Rundeck projects.
Rundeck allows the user to plan ansible and terraform jobs, execute them in a controlled way reducing the probability of a human mistake.

FYI @ancoleman

Let's see some corresponding documentation to go along with this.

Check out https://github.com/PaloAltoNetworks/pan-fca/pull/25/files#diff-bdb5863a000379042f6eafc28e016124 for the documentation.

The problem is this document only explains how to turn on rundeck. If you could provide more documentation on how rundeck is integrated with the framework, and how to customize it for the user's needs.

Acknowledged

@mdrobniu we need a documentation like this https://github.com/PaloAltoNetworks/pan-fca/tree/master/FCA_Lab.
But we talked already about this.

@mdrobniu is there any update from your side? If not we will close this PR

This is still being processed.

@mdrobniu we have to close this case then we have to clean up the stuff. You can create an new one when everything is ready


Showing token unit price instead of the total amount changed

I find this number 👆 is useless.
And the UI is missing the very important token unit price. @colourfreak

In the #2273, I explained what this should be. Maybe not clear enough.
I conducted 3 tests using usabilityhub:

https://app.usabilityhub.com/tests/86ec06814d57/results/1f78709fe6f1

https://app.usabilityhub.com/tests/6076f465a67c/results/590f397ceee6

https://app.usabilityhub.com/tests/bcdef3f68e73/results/420f2f243df1

and the result is that they understand if the number you marked is the ‘24H Price change’ of the ‘how much your
tokens are worth in USD’ (5% x 1500 USD). This is also the same as Delta app, Zerion App, Blockfolio. So the winner was 2)
@zhangzhongnan928 @hboon
The trigger to open AlphaWallet and check wallet tab should be: tell me how much money I gained/lost in last 24h. There are other tools to check the market prices. And if you are looking for that data, open a token and you will find numbers like ETH price.
So if 1 ETH = 1000 USD and the ** ETH price raised by 5%** in last 24H then it should like this:

make sense

@vladyslav-iosdev Please apply the change from https://github.com/AlphaWallet/alpha-wallet-ios/issues/2894#issuecomment-876282955

@colourfreak will do

Can we reopen or create a new issue? Too hard to track orherwisr

Can we reopen or create a new issue? Too hard to track orherwisr

https://github.com/AlphaWallet/alpha-wallet-ios/issues/2894#issuecomment-915222932


Add tree-grid-column-hiding sample

We have added this per:
https://github.com/IgniteUI/igniteui-xplat-examples/blob/master/samples/grids/tree-grid/column-hiding-toolbar.json


"Uncaught ReferenceError: proc is not defined" after closing the test window (test passed)

/Users/Omar/.atom/packages/mocha-test-runner/node_modules/process-tree/index.coffee:41
Hide Stack Trace
ReferenceError: proc is not defined
at /Users/Omar/.atom/packages/mocha-test-runner/node_modules/process-tree/index.coffee:41:47
at ChildProcess. (/Users/Omar/.atom/packages/mocha-test-runner/node_modules/process-tree/index.coffee:38:4)
at emitTwo (events.js:92:20)
at ChildProcess.emit (events.js:172:7)
at maybeClose (internal/child_process.js:818:16)
at Socket. (internal/child_process.js:319:11)
at emitOne (events.js:77:13)
at Socket.emit (events.js:169:7)
at Pipe._onclose (net.js:469:12)

Same for me.

Bump.

Thanks for the report. It seems to come from process-tree itself, introduced in #56. This particular line of code is odd, I can't see proc defined anywhere.
name: info.name ? info.Name ? info.comm ? proc.cmd

@og1 @mattdell @huwmartin what OS do you run?
ps: also confused because the stack trace above points to CoffeeScript but this module publishes the compiled JS code. Any thoughts there?

Stack trace is generated using the sourcemap probably.

I have OSX 10.11.4. NodeJS 6.1.0. The previous version did not have this bug and always occurs after pressing escape to close the test window. Looks like it might have something to do with the events that are triggered when Pipe._onclose (net.js:469:12) fires.

+1

+1

I'm seeing the same issue, I'm running OSX 10.11.5 and Node 6.2.0.

+1

Same here. Like @rprieto I suspect a typo in process-tree, have made an issue for it.
Just on a hunch I changed proc.cmd to info.cmd on the offending line and now things work (well, don't give a loud error).

I've published<EMAIL_ADDRESS>which should fix this issue.

Sorry, problem remains the same. Tried uninstalling mocha-test-runner and reinstalling it but that did not work either. Replacing the process-tree files in the mocha-test-runner package manually did work so I guess all that is needed is a new build/update of mocha-test-runner.

That's because atom-mocha-test-runner has a dependency on "process-tree": "1.0.0" -- no semver cleverness there. So uninstalling/reinstalling is not going to give you the fixed 1.0.2 version, until package.json is changed.

True, but even adding process-tree to package.json does not help because it will add it to the node modules and leave the older version of process-tree inside mocha-test-runner.

I will update process-tree and publish a new version :)

Thanks Chris!

v0.5.1 has been published can you please update and see if it fixes your issue?

Yes it does! Great!


fix: change outputs

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Feature/fk block enable request

Enables users to set a custom variable BlockRemoteSetAvailableRequests to block incoming ChangeAvailability requests that set the CS / a connector operative (cf. comments in code).
(Choice as "custom variable" is to allow users to set this at runtime via the OCPP interface)

Will be implemented differently, closing this PR.


Accept enumerables of byte[] to ToMessage.

Currently ToMessage accepts IObservables and converts them to a list that is packed into a message when the source stream closes. It would be convenient to have an overload for IObservable<List> or e.g. IObservable<Tuple<T, T, T>> so that zipped streams can be directly passed to the ToMessage operator.

@RoboDoig If the inputs into Zip are observables each containing a single byte[] this could be done by replacing Zip with Merge. Similarly, if the observable type is a list of byte[] you can flatten it with Merge before ToMessage.
Are there potentially other cases not covered by these two options?

The thought behind this was that in many cases the frame order must be preserved going into ToMessage which a Merge doesn't guarantee. You're right though that you can just generate a list with the required ordering and then flatten it with Merge. Closing the issue.


Optimize Explorer for SEO

Problem
Request from Brady Gentile: I think we should optimize hashscan for SEO
Solution
To be investigated.
Alternatives
No response

Closing this since a first round of modifications on the HTML meta tags have fixed the most prominent issue reported.
A google search request for 'hashscan' now returns hashscan.io in 1st position with the following summary:
HashScan is a ledger explorer for the Hedera network and allows you to explore and search the network for transactions, accounts, tokens and contracts.
Let's create specific enhancements for potential further SEO optimizations.


Commands: cannot identify all context from service container

At design-time, we relied on the presence of DbContextOptions to figure our which types of context have been added in startup via AddDbContext<T>(). With recent changes, this means only one type of DbContextOptions is ever present.
Repro
serviceCollection
    .AddEntityFrameworkSqlite()
    .AddDbContext<Context1>(getOptions1)
    .AddDbContext<Context2>(getOptions2);

Then make a call to DbContextOperations.GetContextTypes();
Expected
DbContextOperations.GetContextTypes() should return Context1 and Context2
Actual
DbContextOperations.GetContextTypes() only returns Context1

cc @ajcvickers

@ajcvickers I told @natemcmaster to unblock himself by replacing the call to TryAddSingleton<DbContextOptions>() inside AddDbContext<TContext>() with AddSingleton<DbContextOptions>() so that all DbContextOptions<TContext> get registered as DbContextOptions.
AFAICS the only observable changes from what we have now should be:

GetService<DbContextOptions>() will return the last one registered instead of the first one.
GetServices<DbContextOptions>() should return them all (which is what migrations discovery logic needs)

Can you think of any other drawback?

One drawback I've found is https://github.com/aspnet/EntityFramework/issues/4994. We should put https://github.com/aspnet/EntityFramework/pull/5017 in RC2

@divega @natemcmaster For people using AddDbContext in a normal way there should be not drawbacks. #4994 still exists in either case, but as Diego said the context type that gets the wrong connection string will be switched.
The only drawback I am aware of (and the reason it was a TryAdd) is that if somebody else is registering DbContextOptions outside of AddDbContext and then calls AddDbContext then we will clobber what they registered. We make an effort not to do that in our AddXxx methods, but if we have to then I don't think it is a big deal.

Thanks @ajcvickers. I agree it is not a big deal.
I think @natemcmaster hit #4994 almost immediately and that is what he means. I agree it is a shame that the improved exception are not in RC2. We have seen a few customers hitting it as well. Good XML docs would help too.

Fixed in #5183


Cartesian quantitative helper

Pull request #80 introduced encodingChannelCovariateCartesian(), which was logically equivalent to encodingChannelCovariate() from pull request #30 but additionally restricted the output to either "x" or "y", which helps quickly rule out other irrelevant channels when the goal is to solve spatial questions about layout. This pull request now introduces encodingChannelQuantitativeCartesian(), the equivalent counterpart for identifying quantitative encodings which matter for layout in Cartesian space.

The better intended implementation arrived in pull request #396, but it was helpful to get a lightweight version in place here first.


On github I read : "Chrome Hosted Apps" in Supported Input Manifests but...

But with a "generic"/example Chrome Hosted app like the one shown here
https://developer.chrome.com/apps/first_app
I mean with inside manifest.json
{
  "name": "Hello World!",
  "description": "My first Chrome App.",
  "version": "0.1",
  "manifest_version": 2,
  "app": {
    "background": {
      "scripts": ["background.js"]
    }
  },
  "icons": { "16": "calculator-16.png", "128": "calculator-128.png" }
}

it seems not work.  I get a

ERROR: Could open the manifest file.

(I am trying manifoldjs -m <dir_where_manifest_is>)
There is another option/argument for Chrome Hosted app ?

yep so you'll still need to pass in the URL, esiecially since chrome doesn't have it in the above manifest.  So try manifoldjs http://www.mysite.com -m manifest.json

Thanks, but...
I tried with my app and got an Error
So I tried with Google "hello world" sample:
https://github.com/GoogleChrome/chrome-app-samples/tree/master/samples/hello-world
and same result:
ERROR: Invalid manifest format.
Arfy

I believe the manifest you mention is for a packaged app, not a hosted app and it's missing the launch URL. For example:
{
  "name": "Hello World!",
  "description": "My first Chrome App.",
  "version": "0.1",
  "manifest_version": 2,
  "app": {
    "launch": {
       "web_url": "http://yourdomain.com/hello"
    }
  },
  "icons": { "16": "calculator-16.png", "128": "calculator-128.png" }
}

Hi,
This mean ManifoldJS may only "convert" hosted app ?

Correct. ManifoldJS generates hosted apps.

Ok thanks... =)


how to create webview

Please speak English (English only), this is the language everybody of us can speak and write.
Please take a moment to search that an issue doesn't already exist.
Please make sure Golang, GCC is installed correctly before installing RobotGo.

Please give all relevant information below for bug reports, incomplete details will be handled as an invalid report.

You MUST delete the content above including this line before posting, otherwise your issue will be invalid.

Robotgo version (or commit ref):
Go version:
Gcc version:
Operating system and bit:
Resolution:
Can you reproduce the bug at Examples:

[ ] Yes (provide example code)
[ ] No
[ ] Not relevant

Provide example code:

Log gist:

Description
...

This isn't robotgo issue, you should move to https://github.com/webview/webview


Setting Vulkan pipeline names from entry point names.

This should match what is in tracy:

Wow, nice!


Add privacy manifest for Apple

https://github.com/isar/isar/issues/1601
Related to this. I was wondering if this is something that can be added to the isar-community builds for 3.x and 4.x.

This tool can be used to generate the manifest:
https://wemakeapps.net/manifest-maker

This tool can be used to generate the manifest:
https://wemakeapps.net/manifest-maker

Tks for the link @frederikstonge .
Please check both PRs, tell me if you have anything to add to this issue

Seems to be good, but I'm not sure why the build is failing.

The build fails for another reason.
I don't think we should have reasons for all three items. Only what Isar actually uses (even that we are no sure if Isar or limbdbx uses disk space checks)

Alright! :) It looks good then! Thank you for being so responsive!

Create a new file named "PrivacyInfo.xcprivacy" in your Xcode project.

Add the following key-value structure to the file, replacing the placeholders with your app's specific reasons:
NSPrivacyAccessedAPITypes

NSPrivacyAccessedAPIType
NSPrivacyAccessedAPICategoryFileTimestamp
NSPrivacyAccessedAPITypeReasons

YOUR_REASON_CODE

In Xcode, navigate to your app target and enable the "Privacy Info" capability.

Link the PrivacyInfo.xcprivacy file to your target. You can do this by dragging the file into the "Privacy Frameworks" section of your target's general settings.
By following these steps, you'll provide the necessary API reason descriptions without modifying the info.plist file.

https://stackoverflow.com/questions/78163859/itms-91053-missing-api-declaration-privacy

Create a new file named "PrivacyInfo.xcprivacy" in your Xcode project.
Add the following key-value structure to the file, replacing the placeholders with your app's specific reasons:

  <key>NSPrivacyAccessedAPITypes</key>
<array>
 <dict>
   <key>NSPrivacyAccessedAPIType</key>
   <string>NSPrivacyAccessedAPICategoryFileTimestamp</string>
   <key>NSPrivacyAccessedAPITypeReasons</key>
   <array>
     <string>YOUR_REASON_CODE</string>
   </array>
 </dict>
 </array>

In Xcode, navigate to your app target and enable the "Privacy Info" capability.
Link the PrivacyInfo.xcprivacy file to your target. You can do this by dragging the file into the "Privacy Frameworks" section of your target's general settings.

https://stackoverflow.com/questions/78163859/itms-91053-missing-api-declaration-privacy

In case you missed @micodev , we already have 2 PRs related to this issue

Any idea when this will be merged?

Any idea when this will be merged?

I will suggest that @mrclauss merge this along with a new release (maybe 3.1.6+1 ?)

Any idea when this will be merged?

I will suggest that @mrclauss merge #64 and #65 with a new release (maybe 3.1.6+1 ?)

And also for v4 since this is what I'm using. Also, on a side note, I have started a discussion here :
https://github.com/isar-community/isar/discussions/67
If you could give me your input about my question, I would really appreciate. Thank you

Hi guys, any idea when a new release can be expected? Thanks in advance and thank you for keeping this project alive.

This is already contained in versions 3.1.7/4.0.3

You need to use the Isar-Community version as described in the quickstart on Isar-Community.dev
Am 9. Mai 2024 09:38:38 MESZ schrieb "Draško Sarić" @.***>:

Thanks for quick response. But, I can only see 3.1.0+1 version on
pub.dev
--
Reply to this email directly or view it on GitHub:
https://github.com/isar-community/isar/issues/63#issuecomment-2102124503
You are receiving this because you modified the open/close state.
Message ID: @.***>

--
Martin Clauss
Meine Contactulater Card - All meine Kontaktinfo immer aktuell und vollständig:
https://contactulater.app/me/mc

Thanks! Again. congrats for team's hard work.


Surround Ssh key with double quotes in the yaml?

So multi line doesn't insert a space into the key when we write out the yaml?

I think it's worthwhile, since it's common for keys to be formatted with new lines.


feat(plugins/worker): support SharedWorker (resolve #2093)

Resolve #2093
Uses test.concurrent to test shared worker operation. It's still an experimental API. May there's a better way to do this test (or maybe add test for the compiled worker file instead of E2E test). 🤔

Don't know why appveyor failed. Seems like an unrelated failure.

@fi3ework Could you please update your PR by merging the latest main and maybe fixing every conflicts and failing tests? :slightly_smiling_face:
AppVeyor and CircleCI are removed and only GitHub Actions will run via latest main

@Shinigami92 Rebased. No conflict and test passed.

@patak-js Please take a look at this PR, maybe give some advice on the new test case. 😃

@patak-js Appended ?sharedworker to the doc where ?worker appears.

Thanks for the work here @fi3ework. Evan added the contribution welcome to this one in the issue, so we can merge it once we get enough approvals 👍🏼

Looks like jest v27 breaks the test. Looking into it

@antfu CI should be fixed in #3583.


Python: Finish filesystem bindings

Description
The PyArrow filesystem bindings are only partially implemented: https://github.com/delta-io/delta-rs/blob/main/python/deltalake/fs.py
Use Case
We will need the rest of the methods implemented for the PyArrow writer (write_deltalake()) to work.
Related Issue(s)
Blocks: #542

In addition, it probably makes sense to also add in support for Windows and "older" Linux. See https://github.com/delta-io/delta-rs/blob/d54e952026c83fd15c4d5a8b6c07339405751d48/rust/src/storage/file/rename.rs#L96-L114

I am working on this one.

Update: current plan is to use object-store-rs (soon to be moved into arrow-rs) and then wrap that as a PyArrow filesystem for the writer.
So blocked on #610 for now.

We can now pass ObjectStores into PyArrow writer. But we need to be able to provide PyArrow / fsspec filesystems into DeltaTable.
fs = S3Filesystem()
dt = DeltaTable("s3://path/to/table", filesystem=fs)
dt.to_pyarrow_table() # Should use fs
write_deltalake(dt, pa.table({'x': [1,2,3]})) # should use fs
write_deltalake("s3://path/to/table", pa.table({'x': [1,2,3]}), filesystem=fs)  # should use fs

Is there any ETA until this is done?

@wjones127 can you clarify what is still needed for this? I can work on it if it's still relevant

I am also waiting for this to be implemented:
https://github.com/delta-io/delta-rs/blob/d98a0c22efb98368c429610e89acf9bd1e45e299/python/deltalake/writer.py#L184

We aren't working on this, and might never implement this. Copying from another discussion where I explained why we didn't peruse this:

We don't write the log files with PyArrow, and that's the really important part to get right. If you mess that up, you mess up atomicity of transactions. The particular operation we need is an atomic replace if not exist or copy if not exist. That is how we write the log file to commit a transaction. ObjectStore has those, but I don't think fsspec has them.
I had considered just having two filesystems, letting the user pass a PyArrow / fsspec one for the data files and keep using object store for the log files. But then you have to keep the configuration of the two in sync. That seemed like more trouble than it was worth.


Fix closing backticks indentation on bug report

Pre-Submission Checklist

[x] Your pull request targets the staging branch of freeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[x] Closes currently open issue (replace XXXX with an issue no): Closes #12838

Description
As described in the issue, the closing backticks for a codeblock are indented, which makes them display as code.
It appears that there's an issue with dedent. I tried adding newlines before it both as lines that were empty in the editor and as \n, but it did not help.
I switched the code from dedent to good old string concatenation, which works fine.

@systimotic you just beat me into this... damn you are fast!

@systimotic Nice work! :blush:
Template strings and dedent are a clean solution, but since this works - why not stick to this method? :ok_hand:

@raisedadead I kind of already knew the solution when I reported it five hours ago, and had planned to do it now 😛 It was very fast, yeah, but that's not always the case for me 🙂
@Greenheart Thanks! Yeah, that was my thinking as well 👍

:shipit:


porter stemmer: string index out of range

see the following stackoverflow post

For future reference, I copy/paste your question here:

I have a set of pickled text documents which I would like to stem using nltk's PorterStemmer. For reasons specific to my project, I would like to do the stemming inside of a django app view.
However, when stemming the documents inside the django view, I receive an IndexError: string index out of range exception from PorterStemmer().stem() for the string 'oed'. As a result, running the following:
# xkcd_project/search/views.py
from nltk.stem.porter import PorterStemmer

def get_results(request):
    s = PorterStemmer()
    s.stem('oed')
    return render(request, 'list.html')

raises the mentioned error:
Traceback (most recent call last):
  File "//anaconda/envs/xkcd/lib/python2.7/site-packages/django/core/handlers/exception.py", line 39, in inner
    response = get_response(request)
  File "//anaconda/envs/xkcd/lib/python2.7/site-packages/django/core/handlers/base.py", line 187, in _get_response
    response = self.process_exception_by_middleware(e, request)
  File "//anaconda/envs/xkcd/lib/python2.7/site-packages/django/core/handlers/base.py", line 185, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/Users/jkarimi91/Projects/xkcd_search/xkcd_project/search/views.py", line 15, in get_results
    s.stem('oed')
  File "//anaconda/envs/xkcd/lib/python2.7/site-packages/nltk/stem/porter.py", line 665, in stem
    stem = self._step1b(stem)
  File "//anaconda/envs/xkcd/lib/python2.7/site-packages/nltk/stem/porter.py", line 376, in _step1b
    lambda stem: (self._measure(stem) == 1 and
  File "//anaconda/envs/xkcd/lib/python2.7/site-packages/nltk/stem/porter.py", line 258, in _apply_rule_list
    if suffix == '*d' and self._ends_double_consonant(word):
  File "//anaconda/envs/xkcd/lib/python2.7/site-packages/nltk/stem/porter.py", line 214, in _ends_double_consonant
    word[-1] == word[-2] and
IndexError: string index out of range

Now what is really odd is running the same stemmer on the same string outside django (be it a seperate python file or an interactive python console) produces no error. In other words:
# test.py
from nltk.stem.porter import PorterStemmer
s = PorterStemmer()
print s.stem('oed')

followed by:
python test.py
# successfully prints 'o'

what is causing this issue?

I have found that this issue is specific to nltk version 3.2.2. Originally, I ran test.py using ipython not python, as stated above. Somehow, I was able to access the ipython installation in my root environment //anaconda/bin/ipython even though I had not specified ipython in my currently activated virtual environment //anaconda/envs/xkcd/bin/. As a result, ipython must have been using the nltk installtion defined in my root environment as well which runs version 3.2.0.
To clarify, I have discovered that the PorterStemmer fails to stem the string 'oed' in nltk version 3.2.2 but not in nltk version 3.2.0. Why I have no idea.

Hey,
Sorry for this (problem).I mean I never use github, it was
accidentally happened. I don't know what I just trigger!
On Jan 7, 2017 11:47 PM, "jkarimi91"<EMAIL_ADDRESS>wrote:

I have found that this issue is specific to nltk version 3.2.2.
Originally, I ran test.py using ipython not python, as stated above.
Somehow, I was able to access the ipython installation in my root
environment //anaconda/bin/ipython even though I had not specified
ipython in my currently activated virtual environment
//anaconda/envs/xkcd/bin/. As a result, ipython must have been using the
nltk installtion defined in my root environment as well which runs version
3.2.0.
To clarify, I have discovered that the PorterStemmer fails to stem the
string 'oed' in nltk version 3.2.2 but not in nltk version 3.2.0. Why I
have no idea.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nltk/nltk/issues/1581#issuecomment-271100268, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AVTBBiywlg5c81StFrrcNOsyuF610y9uks5rP9bLgaJpZM4LdV66
.

Thanks @jkarimi91, @fievelk, @ExplodingCabbage

Hi, I encountered the exact same issue today. Could you please suggest how I could get a fix to this? Should I update any packages?

Hi @santoshbs. You can either use the master version of NLTK or release 3.2.1 to get rid of the bug; it only exists in version 3.2.2.

@ExplodingCabbage I think you are referring to the develop branch (not master). It's easy to get confused I guess :)

@fievelk you are quite right. Sorry, yes: you can either use the develop branch or 3.2.2 to get rid of the bug.

@fievelk you are quite right. Sorry, yes: you can either use the develop branch or 3.2.1 to get rid of the bug.

Thanks so much for the pointer.


Missing step to create Data access policy in purview

Hello guys, I was strucking to enable "Data access policy" in purview but thanks to the Microsoft support agent we found out that 1 required step was missing in your article => user must be added manually in "Policy authors" section.
QED : even if my user was "owner" on the purview resource (+owner at subscription level too), this manual step to add the user in policy authors is required ,  if not added the button "create new policy" is greyed out (and no specific explanation is displayed)
Here is the support request ID for your reference :<PHONE_NUMBER>002617

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

ID: 2bd209fb-231f-5c9e-bee8-3967f58d1f3e
Version Independent ID: da10959a-2228-9a50-1670-ee491ebba83b
Content: Data access policy provisioning for Azure Storage - Azure Purview
Content Source: articles/purview/how-to-access-policies-storage.md
Service: purview
Sub-service: purview-data-policies
GitHub Login: @ePpnqeqR
Microsoft Alias: vlrodrig

Hi. Thanks for giving Azure Purview data policies a try. That step is already in the documentation. Please see the permissions section of the document

indeed I didn't see that, my bad, I'm closing the issue. Thanks

@rabbyn Even though it was your bad, actually I am glad you posted the question here. I had the same issue - and after reading your post I came to know that even if you are an owner of the resource, you need to have Policy Author role assigned on that source in Purview. Thank you for sharing your mistake.


bug: variable of type PDOStatement used in string concatenation

https://github.com/Usbac/wolff-framework/blob/b3402ddb994ff53533185287d5a8892f81dd2c07/src/Core/DB.php#L334-L336
https://github.com/Usbac/wolff-framework/blob/b3402ddb994ff53533185287d5a8892f81dd2c07/src/Core/DB.php#L363-L365
https://github.com/Usbac/wolff-framework/blob/b3402ddb994ff53533185287d5a8892f81dd2c07/src/Core/DB.php#L432-L434

Thank you so much for pointing that out, it has been fixed. 😄  04108ecc052328758b7cae8e3e8f0adf573f52e2


IP Planner Tool does not display the available ranges /28 - /30 correctly

Description
If I use the "Planner" tool in the Tools -> Planner tab and select any Virtual Network, the address ranges with the mask /28 - /30 do not correctly display the number of available ranges. For example, my vNET has a range of /20 (i.e., 4096 IP addresses). The/28 range in the planner shows "MASK: 28 | (1/16 Used)". But I can create up to 256 subnets in Azure, not 16. The same problem then continues for ranges /29 and 30.
Screenshot

Hi @petrschonfeld, I'll start working on a fix for this right away. Thanks for bringing this to my attention!

Hi there @petrschonfeld, this should no be addressed in #201. Please restart your Azure IPAM App Service, allow it to pull the updated containers, and give it a test for me.
I wrote some automation to test various CIDR's against the IP Planner code, so I should be able to troubleshoot these things much faster in the future.
Have a great one!

Hi @DCMattyG, thanks for a pretty much quick fix. It works like a charm. 👍

Wonderful to hear, so happy we could get that fixed quickly for you!

Hi @pschonfeld, apologies for the delayed response on this...
I've re-written the Planner code from scratch to address this issue. The next release will include all of the fixes and I'll make sure to ping you once that code is available. Really appreciate you flagging these bugs and helping us make the product even better!

Hey there @pschonfeld, I've completely re-written the planner code and with #230 this should now be fixed (hopefully forever)!
If you've used the standard deployment script you can simply restart you Azure App Service and it will pull the latest containers down which contain the fixes.
Please let me know how everything is looking now!

Good afternoon @pschonfeld, just checking in that the fix got you where you need to be. Please let me know if were good to close this issue out (again).
Thanks so much!

Hi @pschonfeld, as this bug has been fixed for quite some time and I haven't heard back from you, I'm going to close this issue for the time being.
If for any reason you're still having issues with this, please feel free to re-open this issue or file a new one and I'll make sure you get taken care of right away.
Thanks so much for reporting this and I hope all is working much better for you now!


Write unit test(s) for Spark functions

Description
There are many functions for Spark, and most of them are created via a macro. However, not all of them have unit test coverage. Create additional unit tests based on similar test conditions from the exiting Spark API test cases.
I have been mirror the docstring tests from the PySpark API for reference.

Hi, would the scope for this be the following path - core/src/functions/mod.rs
I can see quite a number of tests from other modules like column.rs.

Hi @lexara-prime-ai  Welcome! Great question!
I have been trying to mirror the doctests from pyspark 3.5. And all the column unit tests in the docs are technically using the function col. So when I was writing some of the tests I just lumped them into functions. Was this correct? Hindsight probably not, but I think I was just in the zone of writing tests and kept moving forward.
Might be worth while to move those tests into column.rs

Thanks @sjrusso8! The explanation makes a lot of sense. Appreciate the groundwork you've laid with the existing test cases, they've been incredibly helpful as I piece everything together. I'll do my best to contribute and assist as much as possible.

Hi @sjrusso8 hope you're doing well.
Quick question on how the isnotnull function has been implemented.
I assumed it would be the exact opposite of the isnull function implementation.
However, the function that gets generated by the generate_functions! macro requires an extra argument.
Moving the "definition" from here
// functions that require exactly two col arguments
generate_functions!(
    two_cols: nvl,
    nullif,
    isnotnull,
    ifnull,

to here
// functions that require a single col argument
generate_functions!(
    one_col: isnan,
    isnull,
    isnotnull, // <---
    sqrt,

allows compilation and the test passes...
res -> RecordBatch { schema: Schema { fields: [Field { name: "a", data_type: Int64, nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} }, Field { name: "r1", data_type: Boolean, nullable: false, dict_id: 0, dict_is_ordered: false, metadata: {} }], metadata: {} }, columns: [PrimitiveArray<Int64>
[
  null,
  1,
], BooleanArray
[
  false,
  true,
]], row_count: 2 }
res -> RecordBatch { schema: Schema { fields: [Field { name: "a", data_type: Int64, nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} }, Field { name: "r1", data_type: Boolean, nullable: false, dict_id: 0, dict_is_ordered: false, metadata: {} }], metadata: {} }, columns: [PrimitiveArray<Int64>
[
  null,
  1,
], BooleanArray
[
  true,
  false,
]], row_count: 2 }
test functions::tests::test_func_is_not_null ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 117 filtered out; finished in 0.11s

INITIAL COMPILER DIAGNOSTIC
  --> core/src/functions/mod.rs:1180:32
     |
1180 |             .select([col("a"), isnotnull("a").alias("r1")])
     |                                ^^^^^^^^^-----
     |                                         ||
     |                                         |expected all arguments to be this `&str` type because they need to match the type of this parameter
     |                                         an argument of type `&str` is missing
     |
note: function defined here
    --> core/src/functions/mod.rs:468:5
     |
294  |             pub fn $func_name<T: expressions::ToExpr>(col1: T, col2: T) -> Column
     |                               -                       -------  ------- this parameter needs to match the `&str` type of `col1`
     |                               |                       |
     |                               |                       `col2` needs to match the `&str` type of this parameter
     |                               `col1` and `col2` all reference this parameter T
...
468  |     isnotnull,
     |     ^^^^^^^^^
help: provide the argument
     |
1180 |             .select([col("a"), isnotnull("a", /* &str */).alias("r1")])

@lexara-prime-ai I see what you mean. It should be only one input argument for the function, so your change is correct. Probably was not caught because of the lack of unit tests :)
I'm getting the list of arguments from the docs here -> https://spark.apache.org/docs/latest/api/python/reference/pyspark.sql/functions.html

Awesome!
Thanks for the confirmation! I'll make a note of the subtle change as part of the commit message for keeping track.

Column code coverage was added, keeping this open for now to track other function coverage

@sjrusso8 was looking through other modules as well. Are there any areas that you'd consider high priority so I can start from there?

Ok just went through the list, I can take a look at the other issues in the meantime.