 This step only can take up to 2 minutes.
However, if I only type dep deploy:vendors, it is very fast (2 s)
Steps to reproduced
dep deploy-vvv
Content of deploy.php
<?php
namespace Deployer;
require 'recipe/symfony.php';
require 'vendor/deployer/recipes/cachetool.php';

//------------------------------
// Configuration
//------------------------------
set('ssh_type', 'native');
set('ssh_multiplexing', true);
set('repository', '***');
set('default_stage', '****-stage');
set('shared_files', ['app/config/parameters.yml.dist']);

add('shared_dirs', ['/web/uploads']);
add('writable_dirs', ['/web/uploads']);

//------------------------------
// Servers
//------------------------------

server('****-stage', '*.*.*.*')
    ->stage('stage')
    ->set('env', 'dev')
    ->set('composer_options', '{{composer_action}} --verbose --prefer-dist --no-progress --no-interaction --optimize-autoloader')
    ->set('clear_paths', ['web/config.php'])
    ->set('deploy_path', '/home/****/stage')
    ->set('cachetool', '/var/run/php/php7.1-fpm.sock')
    ->user('***')
    ->forwardAgent()
    ->set('branch', 'master');

server('****', '*.*.*.*')
    ->stage('prod')
    ->set('deploy_path', '/home/****/prod')
    ->user('***')
    ->forwardAgent()
    ->set('branch', 'master');

//------------------------------
// Tasks
//------------------------------

// If deploy fails automatically unlock.
after('deploy:failed', 'deploy:unlock');

// Migrate database before symlink new release.
before('deploy:symlink', 'database:migrate');

// Clear opcache
after('deploy:symlink', 'cachetool:clear:opcache');

task('upload-stage-files', function () {
    $deployPath = get('deploy_path');
    upload("web/uploads/", "/home/****/stage/shared/web/uploads/");
});

Output log

$ dep deploy -vvv
➤ Executing task deploy:prepare
[project-stage] > echo $0
SSH multiplexing initialization
[project-stage] < bash
[project-stage] > if [ ! -d /home/project/stage ]; then mkdir -p /home/project/stage; fi
[project-stage] > if [ ! -L /home/project/stage/current ] && [ -d /home/project/stage/current ]; then echo true; fi
[project-stage] > cd /home/project/stage && if [ ! -d .dep ]; then mkdir .dep; fi
[project-stage] > cd /home/project/stage && if [ ! -d releases ]; then mkdir releases; fi
[project-stage] > cd /home/project/stage && if [ ! -d shared ]; then mkdir shared; fi
• done on [project-stage]
✔ Ok [1s 669ms]
➤ Executing task deploy:lock
[project-stage] > if [ -f /home/project/stage/.dep/deploy.lock ]; then echo 'true'; fi
[project-stage] > touch /home/project/stage/.dep/deploy.lock
• done on [project-stage]
✔ Ok [323ms]
➤ Executing task deploy:release
[project-stage] > cd /home/project/stage && (if [ -h release ]; then echo 'true'; fi)
[project-stage] > cd /home/project/stage && ([ -d releases ] && [ "$(ls -A releases)" ] && echo "true" || echo "false")
[project-stage] < true
[project-stage] > cd /home/project/stage && (cd releases && ls -t -1 -d */)
[project-stage] < 47/
[project-stage] < 46/
[project-stage] < 45/
[project-stage] < 44/
[project-stage] < 43/
[project-stage] > cd /home/project/stage && (if [ -f .dep/releases ]; then echo "true"; fi)
[project-stage] < true
[project-stage] > cd /home/project/stage && (tail -n 15 .dep/releases)
[project-stage] <<PHONE_NUMBER>4610,34
[project-stage] <<PHONE_NUMBER>3746,35
[project-stage] <<PHONE_NUMBER>4409,36
[project-stage] <<PHONE_NUMBER>5420,37
[project-stage] <<PHONE_NUMBER>0548,38
[project-stage] <<PHONE_NUMBER>1306,38
[project-stage] <<PHONE_NUMBER>1918,39
[project-stage] <<PHONE_NUMBER>0749,40
[project-stage] <<PHONE_NUMBER>3000,41
[project-stage] <<PHONE_NUMBER>4846,42
[project-stage] <<PHONE_NUMBER>3817,43
[project-stage] <<PHONE_NUMBER>0016,44
[project-stage] <<PHONE_NUMBER>0528,45
[project-stage] <<PHONE_NUMBER>1617,46
[project-stage] <<PHONE_NUMBER>2042,47
[project-stage] > cd /home/project/stage && (if [ -d /home/project/stage/releases/48 ]; then echo 'true'; fi)
[project-stage] > cd /home/project/stage && (date +"%Y%m%d%H%M%S")
[project-stage] <<PHONE_NUMBER>3205
[project-stage] > cd /home/project/stage && (echo '20170529153205,48' >> .dep/releases)
[project-stage] > cd /home/project/stage && (mkdir /home/project/stage/releases/48)
[project-stage] > cd /home/project/stage && (if [[ "$(man ln)" =~ "--relative" ]]; then echo "true"; fi)
[project-stage] < true
[project-stage] > cd /home/project/stage && (ln -nfs --relative /home/project/stage/releases/48 /home/project/stage/release)
• done on [project-stage]
✔ Ok [1s 932ms]
➤ Executing task deploy:update_code
[project-stage] > which git
[project-stage] < /usr/bin/git
[project-stage] > /usr/bin/git version
[project-stage] < git version 2.1.4
[project-stage] > if [ -h /home/project/stage/release ]; then echo 'true'; fi
[project-stage] < true
[project-stage] > readlink /home/project/stage/release
[project-stage] < releases/48
[project-stage] > /usr/bin/git clone -b master --depth 1 --recursive -q git@github.com:project/project.git /home/project/stage/releases/48 2>&1
• done on [project-stage]
✔ Ok [3s 308ms]
➤ Executing task deploy:clear_paths
[project-stage] >  rm -rf /home/project/stage/releases/48/web/config.php
• done on [project-stage]
✔ Ok [165ms]
➤ Executing task deploy:create_cache_dir
[project-stage] > if [ -d "/home/project/stage/releases/48/app/cache" ]; then rm -rf /home/project/stage/releases/48/app/cache; fi
[project-stage] > mkdir -p /home/project/stage/releases/48/app/cache
[project-stage] > chmod -R g+w /home/project/stage/releases/48/app/cache
• done on [project-stage]
✔ Ok [533ms]
➤ Executing task deploy:shared
[project-stage] > if [ -d /home/project/stage/shared/app/logs ]; then echo 'true'; fi
[project-stage] < true
[project-stage] > rm -rf /home/project/stage/releases/48/app/logs
[project-stage] > mkdir -p dirname /home/project/stage/releases/48/app/logs
[project-stage] > ln -nfs --relative /home/project/stage/shared/app/logs /home/project/stage/releases/48/app/logs
[project-stage] > if [ -d /home/project/stage/shared//web/uploads ]; then echo 'true'; fi
[project-stage] < true
[project-stage] > rm -rf /home/project/stage/releases/48//web/uploads
[project-stage] > mkdir -p dirname /home/project/stage/releases/48//web/uploads
[project-stage] > ln -nfs --relative /home/project/stage/shared//web/uploads /home/project/stage/releases/48//web/uploads
[project-stage] > if [ -f $(echo /home/project/stage/releases/48/app/config/parameters.yml.dist) ]; then rm -rf /home/project/stage/releases/48/app/config/parameters.yml.dist; fi
[project-stage] > if [ ! -d $(echo /home/project/stage/releases/48/app/config) ]; then mkdir -p /home/project/stage/releases/48/app/config;fi
[project-stage] > mkdir -p /home/project/stage/shared/app/config
[project-stage] > touch /home/project/stage/shared/app/config/parameters.yml.dist
[project-stage] > ln -nfs --relative /home/project/stage/shared/app/config/parameters.yml.dist /home/project/stage/releases/48/app/config/parameters.yml.dist
• done on [project-stage]
✔ Ok [2s 287ms]
➤ Executing task deploy:assets
[project-stage] > find /home/project/stage/releases/48/web/css /home/project/stage/releases/48/web/images /home/project/stage/releases/48/web/js -exec touch -t<PHONE_NUMBER>32.12 {} ';' &> /dev/null || true
• done on [project-stage]
✔ Ok [215ms]
➤ Executing task deploy:vendors
[project-stage] > if hash composer 2>/dev/null; then echo 'true'; fi
[project-stage] < true
[project-stage] > which composer
[project-stage] < /usr/local/bin/composer
[project-stage] > which php
[project-stage] < /usr/bin/php
[project-stage] > cd /home/project/stage/releases/48 && SYMFONY_ENV=dev /usr/bin/php /usr/local/bin/composer install --verbose --prefer-dist --no-progress --no-interaction --optimize-autoloader
[project-stage] < ocramius/package-versions:  Generating version class...
[project-stage] < ocramius/package-versions: ...done generating version class
[project-stage] < Creating the "app/config/parameters.yml" file
[project-stage] <
[project-stage] <  // Clearing the cache for the dev environment with debug true
[project-stage] <
[project-stage] <  [OK] Cache for the "dev" environment (debug=true) was successfully cleared.
[project-stage] <
[project-stage] <
[project-stage] <  Trying to install assets as relative symbolic links.
[project-stage] <
[project-stage] <  --- --------------------- ------------------
[project-stage] <       Bundle                Method / Error
[project-stage] <  --- --------------------- ------------------
[project-stage] <   ✔   FrameworkBundle       relative symlink
[project-stage] <   ✔   MopaBootstrapBundle   relative symlink
[project-stage] <  --- --------------------- ------------------
[project-stage] <
[project-stage] <  [OK] All assets were successfully installed.
• done on [project-stage]
✔ Ok [112s 930ms]
➤ Executing task deploy:assets:install
[project-stage] > SYMFONY_ENV=dev /usr/bin/php /home/project/stage/releases/48/app/console assets:install --no-interaction --env=dev /home/project/stage/releases/48/web
[project-stage] <
[project-stage] <  Installing assets as hard copies.
[project-stage] <
[project-stage] <  --- --------------------- ----------------
[project-stage] <       Bundle                Method / Error
[project-stage] <  --- --------------------- ----------------
[project-stage] <   ✔   FrameworkBundle       copy
[project-stage] <   ✔   MopaBootstrapBundle   copy
[project-stage] <  --- --------------------- ----------------
[project-stage] <
[project-stage] <  ! [NOTE] Some assets were installed via copy. If you make changes to these assets you have to run this command again.
[project-stage] <
[project-stage] <  [OK] All assets were successfully installed.
• done on [project-stage]
✔ Ok [407ms]
➤ Executing task deploy:assetic:dump
• done on [project-stage]
✔ Ok [0ms]
➤ Executing task deploy:cache:warmup
[project-stage] > SYMFONY_ENV=dev /usr/bin/php /home/project/stage/releases/48/app/console cache:warmup --no-interaction --env=dev
[project-stage] <
[project-stage] <  // Warming up the cache for the dev environment with debug true
[project-stage] <
[project-stage] <  [OK] Cache for the "dev" environment (debug=true) was successfully warmed.
• done on [project-stage]
✔ Ok [3s 97ms]
➤ Executing task deploy:writable
[project-stage] > ps axo user,comm | grep -E '[a]pache|[h]ttpd|[_]www|[w]ww-data|[n]ginx' | grep -v root | head -1 | cut -d\  -f1
[project-stage] < www-data
[project-stage] > cd /home/project/stage/releases/48 && (mkdir -p app/cache app/logs /web/uploads)
[project-stage] > cd /home/project/stage/releases/48 && (chmod 2>&1; true)
[project-stage] < chmod: missing operand
[project-stage] < Try 'chmod --help' for more information.
[project-stage] > cd /home/project/stage/releases/48 && (if hash setfacl 2>/dev/null; then echo 'true'; fi)
[project-stage] < true
[project-stage] > cd /home/project/stage/releases/48 && (getfacl -p app/cache | grep "^user:www-data:.*w" | wc -l)
[project-stage] < 0
[project-stage] > cd /home/project/stage/releases/48 && (setfacl -RL -m u:"www-data":rwX -m u:whoami:rwX app/cache)
[project-stage] > cd /home/project/stage/releases/48 && (setfacl -dRL -m u:"www-data":rwX -m u:whoami:rwX app/cache)
[project-stage] > cd /home/project/stage/releases/48 && (getfacl -p app/logs | grep "^user:www-data:.*w" | wc -l)
[project-stage] < 1
[project-stage] > cd /home/project/stage/releases/48 && (getfacl -p /web/uploads | grep "^user:www-data:.*w" | wc -l)
[project-stage] < 1
• done on [project-stage]
✔ Ok [1s 500ms]
➤ Executing task database:migrate
[project-stage] > SYMFONY_ENV=dev /usr/bin/php /home/project/stage/releases/48/app/console doctrine:migrations:migrate --no-interaction --env=dev --allow-no-migration
[project-stage] <
[project-stage] <                     Application Migrations
[project-stage] <
[project-stage] <
[project-stage] < No migrations to execute.
• done on [project-stage]
✔ Ok [549ms]
➤ Executing task deploy:symlink
[project-stage] > if [[ "$(man mv)" =~ "--no-target-directory" ]]; then echo "true"; fi
[project-stage] < true
[project-stage] > mv -T /home/project/stage/release /home/project/stage/current
• done on [project-stage]
✔ Ok [356ms]
➤ Executing task cachetool:clear:opcache
[project-stage] > cd /home/project/stage/releases/48 && (if [ -e /home/project/stage/releases/48/cachetool.phar ]; then echo 'true'; fi)
[project-stage] > cd /home/project/stage/releases/48 && (curl -sO https://gordalina.github.io/cachetool/downloads/cachetool.phar)
[project-stage] > cd /home/project/stage/releases/48 && (/usr/bin/php cachetool.phar opcache:reset --fcgi=/var/run/php/php7.1-fpm.sock)
• done on [project-stage]
✔ Ok [1s 363ms]
➤ Executing task deploy:unlock
[project-stage] > rm -f /home/project/stage/.dep/deploy.lock
• done on [project-stage]
✔ Ok [160ms]
➤ Executing task cleanup
[project-stage] > rm -rf /home/project/stage/releases/43
[project-stage] > cd /home/project/stage && if [ -e release ]; then rm release; fi
[project-stage] > cd /home/project/stage && if [ -h release ]; then rm release; fi
• done on [project-stage]
✔ Ok [1s 326ms]
➤ Executing task success
✔ Ok [0ms]
Successfully deployed!
*

It's because composer installing deps. When you run dep deploy:vendors all deps already installed, so nothing goes not.
Make sure you use latest composer installed and it's uses cache.

Hello,
It is the latest composer version.
I've done a composer clear-cache and retried a new time dep deploy-vvv : my ~/.composer/cache folder went from 0 to 64M.
I've tried one las time dep deploy-vvv:
➤ Executing task deploy:vendors [project-stage] > if hash composer 2>/dev/null; then echo 'true'; fi [project-stage] < true [project-stage] > which composer [project-stage] < /usr/local/bin/composer [project-stage] > which php [project-stage] < /usr/bin/php [project-stage] > cd /home/project/stage/releases/54 && SYMFONY_ENV=dev /usr/bin/php /usr/local/bin/composer install --verbose --prefer-dist --no-progress --no-interaction --optimize-autoloader 	[project-stage] < ocramius/package-versions:  Generating version class... [project-stage] < ocramius/package-versions: ...done generating version class [project-stage] < Creating the "app/config/parameters.yml" file [project-stage] < [project-stage] <  // Clearing the cache for the dev environment with debug true [project-stage] < [project-stage] <  [OK] Cache for the "dev" environment (debug=true) was successfully cleared. [project-stage] < [project-stage] < [project-stage] <  Trying to install assets as relative symbolic links. [project-stage] < [project-stage] <  --- --------------------- ------------------ [project-stage] <       Bundle                Method / Error [project-stage] <  --- --------------------- ------------------ [project-stage] <   ✔   FrameworkBundle       relative symlink [project-stage] <   ✔   MopaBootstrapBundle   relative symlink [project-stage] <  --- --------------------- ------------------ [project-stage] < [project-stage] <  [OK] All assets were successfully installed. • done on [project-stage] ✔ Ok [113s 88ms]
How can i make Composer uses its cache when called via Deployer ?

Composer always uses it's cache. There isn't even an option to turn the composer option off. See this discussion.

I have done a composer clear-cache and retried a new time dep deploy-vvv : my ~/.composer/cache folder went from 0 to 64M.

So composer is already caching right? You should try to deploy again with a filled cache and not clear the cache afterwards. This time composer should use your cached packages and not download them again. This should result in a faster deploy:vendor step.

The ~/.composer/cache folder is filled with data (64M). But on dep deploy, composer still spends 2 minutes on the deploy:vendor step.
I did the clear-cache command only to check if composer was caching on a dep deploy command. It is : the cache is created in ~/.composer/cache. But it seems it is not reading it on futures dep deploy.

Are you sure that it's a problem of the package download of composer? Normally your log should contain something like this (this is taken from my local vm deploy):
[vagrant] > cd /var/www/html/project/releases/7 &&  /usr/bin/php /usr/bin/composer install --verbose --prefer-dist --no-progress --no-interaction --optimize-autoloader
[vagrant] < Loading composer repositories with package information
[vagrant] < Installing dependencies (including require-dev) from lock file
[vagrant] < Dependency resolution completed in 0.001 seconds
[vagrant] < Analyzed 190 packages to resolve dependencies
[vagrant] < Analyzed 482 rules to resolve dependencies
[vagrant] <   - Installing skyzyx/mimetypes (1.1.1)
[vagrant] <     Loading from cache
[vagrant] <     Extracting archive
[vagrant] <   - Installing apfelbox/php-file-download (v2.1)
[vagrant] <     Loading from cache
[vagrant] <     Extracting archive

As you can see, it tells you that it is using the cache for those two packages for example. If the package isn't cached, it would tell you something like download package. Your log on the other hand seems to not load a single package in the deploy:vendor step. So the problem for the delay has to lie somewhere else I think.

Also you can try something like this:
set('copy_dirs', ['vendor']);

Ow :) I just visited the issues with the same thought but actually there is a task about composer being slow ))
As for me, I have an offer - maybe we should check if composer.lock did not change since previous release (md5sum compare of {{current_path}} and {{deploy_path}}) if composer did not change?
Of course, we should have "composer_skip_unchanged" defaulting to false to avoid BC's.

@andrewtch I don't think that Deployer needs to worry about your composer install speed at all, it's there to execute commands, not have specific implementation optimizations.

Won't deploy:vendors typically do nothing when run in isolation, because it's run in an existing directory that already has a complete vendors? I think copy_dirs is the mitigation for a full deployment. But it will still be much slower, because there is actual work to do.

In theory it could usually copy the vendor dir from the previous release and then run composer install, but not sure how much that would speed up.

please use blackfire profilling (or similar) to measure where your time is spent

You could also run it with -vvv and add the --profile flag to composer.

Ok, I got the point.
This happens when there is no uzip command (@jolipixel, are you using clean DO droplet by any chance?) and / or zip PHP extension.
Vendors pass on such server always takes around 3-4 minutes as composer fallbacks to git clone. Installing unzip and adding:
add('copy_dirs', ['vendor']);
before('deploy:vendors', 'deploy:copy_dirs');

reduced times for copy_dirs to ~5s and vendors to ~4s.

This happens when there is no uzip command (@jolipixel, are you using clean DO droplet by any chance?) and / or zip PHP extension.

It was exactly that !
Deployment time went from ~112s to ~11s after :
apt-get install unzip
apt-get install php7.1-zip

Nice catch. Thank you.

Added check in deploy:vendors task for unzip command.


add colormap endpoints

ref https://github.com/developmentseed/titiler/discussions/509

TODO

[x] ask for feedback
[ ] add tests

This new endpoint looks very useful. I would suggest to have a query param for number of classes at the endpoint of /colormap/{id}.
I can see current endpoint returns so many classes (44 No. what I can see in screenshot), but it would be nice if client side can request how many classes to be returned from titiler. I think number of classes parameter can be useful if want to use custom colormap to classify data with limited classes instead of linear legend.

@JinIgarashi thank for your feedback 🙏

(44 No. what I can see in screenshot)

in fact it returns 256 values, because all rio-tiler colormaps are composed of values between 0 and 255 (because that's how GDAL colormap are)
We could add a step parameter to only select few values but user would need to understand that for colormaps (e.g qualitative colormaps) this would give really weird results

Also, it might be useful if it can return color as hsl format in addition to rgba and hex formats. If color format is following maplibre style spec, it can be easier to adapt for maplibre/mapbox.

Well the more I think about this, the more I think we should only return value as RGBA tuple, not event as HEX string. It seems quite trivial do transform the RGBA values to hex or hsl in client application instead of adding more complexity in the titiler endpoint

This new endpoint looks very useful. I would suggest to have a query param for number of classes at the endpoint of /colormap/{id}.

same as https://github.com/developmentseed/titiler/pull/796#issuecomment-1999296498, this would be pretty trivial to do in the client application directly instead of adding complexity in titiler endpoint IMO

Q: we have the tilematrixet endpoints with a /tileMatrixSet prefix (defined by OGC), so should the colormap endpoints be prefixed with colorMaps (instead of colormaps as defined now)?
## This PR

/colormaps
/colormaps/{colormapId}

## Proposed

/colorMaps
/colorMaps/{colormapId}


Sets COPY_PHASE_STRIP = YES in the release configuration

This change adds debug symbol stripping in the Release configuration.
(The commit message is not really informative about the rationale for this – I claim there due to momentary lapse of reason that this was to make things work better with Carthage, which is not really relevant.)

Ah, that might've been a mess-up on my part. This needs to be set in the xcconfigs, though. It's currently defined in Base.xcconfig.

@zwaldowski ok, moved it to release xcconfig now. It should still remain in base I think, only the release configuration should override?


Can't update zoner AV

Error running update command:
╰$ docker run --name=zoner malice/zoner update
Updating Zoner...
time="2018-08-06T09:27:12Z" level=fatal msg="exit status 1" category=av path= plugin=zoner

So something is wrong with the Zoner update servers.  When building this image you get: ERROR: Update failed - server encountered an error: "Cannot query SQL database"  So there is a problem with zoner itself, which is why i disabled this plugin in MaliceIO https://github.com/maliceio/malice/blob/master/plugins/plugins.toml#L223


Configuration issue

  MIDJOURNEY_API_SECRET: "123123"
  # => Midjourney API Address, eg: `"ws://...."`, or `"ws://host.docker.internal:8092/ws"`
  MIDJOURNEY_API_URL: "ws://localhost:8092/ws"

Do you need to set up the service locally by yourself? Are there any sample projects?

that's a good question, the answer is yes. @itxtl
You can implement a midjourney service yourself to accomplish this.
The implementation cost is not high, but limited by the user agreement, there may be some risks in public and private implementations.

Do you have any reference projects? Or do we need to use direct HTTP connection? Sorry, I'm not very familiar with Go language.


popular developer searcher added

[x] Popular

[ ] Recently Joined

Closes #30

@DannyFeliz may you please confirm me please if the 'forked' property on users.json is working and also if it is right to use in the Developers view.

@DannyFeliz may you please confirm me please if the 'forked' property on users.json is working and also if it is right to use in the Developers view.

I will change the data source soon with all the necessary information. I was talking with @rmariuzzo and he created a new repository for the data and recently he added the repositories data, so, let's leave it like this for now until we fix the that.

The last thing,
The grid system broke with some specific width: e.g.

I have tried many things, and the only one which works for me was a fixed height, but don't like use it, if you can help me I will thanks you.
@DannyFeliz @larizzatg

Let's wait for @larizzatg to take a look

Ok, I have fixed the columns display in this commit 6fff5b0b0b5ed56c2442be03f3a4564008e5aa51
And we are ready to code review. 👯

I was talking with @rmariuzzo and he created a new repository for the data and recently he added the repositories data, so, let's leave it like this for now until we fix the that.

The properties exported by https://github.com/developersdo/opensource-data for a user are listed here.

confirm me please if the 'forked' property on users.json is working

@ljcordero it is not for now.


no YAeHMOP support on Mac ARM for rdkit 2023.05

Version 2023.5 seems to be compiled without YAeHMOP support, 2022.05 has it on same machine.
Error message comes from
from rdkit import Chem
from rdkit.Chem import rdDetermineBonds
mol = Chem.MolFromXYZBlock("2\n\nH 0.0 0.0 0.0\nH 0.74 0.0 0.0\n")
rdDetermineBonds.DetermineConnectivity(mol,useHueckel=True)

Thanks for reporting this. I can repo this.
I can see the YAeHMOP build running in the build logs.  -DRDK_BUILD_YAEHMOP_SUPPORT=ON is also set. From Release_2023_09_3, rdkit throws the error:
In [1]: from rdkit import Chem
   ...: from rdkit.Chem import rdDetermineBonds
   ...: mol = Chem.MolFromXYZBlock("2\n\nH 0.0 0.0 0.0\nH 0.74 0.0 0.0\n")
   ...: rdDetermineBonds.DetermineConnectivity(mol,useHueckel=True)
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In[1], line 4
      2 from rdkit.Chem import rdDetermineBonds
      3 mol = Chem.MolFromXYZBlock("2\n\nH 0.0 0.0 0.0\nH 0.74 0.0 0.0\n")
----> 4 rdDetermineBonds.DetermineConnectivity(mol,useHueckel=True)

ValueError: The RDKit was not compiled with YAeHMOP support

Seems like there was some restructuring of YAeHMOP in
https://github.com/rdkit/rdkit/compare/Release_2023_09_2...Release_2023_09_3
@greglandrum can you help?

This is due to a mistake I made when attempting to make the YAeHMOP support optional for DetermineConnectivity.
I'll fix it for the 2024.03.2 release.
Thanks for pointing out the problem @juius

This should be fixed in the new release.


Fixes for Julia 0.6

There are still some deprecation warnings (generated by the tests, not the code in src). This requires https://github.com/JuliaLang/julia/pull/20573 to pass tests on 0.6, but makes sense even without it.

See also #20. Might be cleaner to take that one for the similar and then I can rebase this. CC @pabloferz.

I missed that you already had something here. You can bring the change in #20 if you want and I'll close that one.

No, I like this a little better. Let's merge it.

Rebased.

I don't see any need to wait for resolution on JuliaLang/julia#20573, so I'll merge and tag a new release.

@alsam, you might consider configuring attobot, it makes releases much easier.


sorting is failing

Sorting is not working at all, this is the error at Beth terminal:
Traceback (most recent call last):
  File "/Users/lonny/.pyenv/versions/3.12.0/lib/python3.12/site-packages/vars_gridview/scripts/run.py", line 314, in _sort_widgets
    self.image_mosaic.sort_rect_widgets(method)
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'sort_rect_widgets'
Traceback (most recent call last):
  File "/Users/lonny/.pyenv/versions/3.12.0/lib/python3.12/site-packages/vars_gridview/scripts/run.py", line 314, in _sort_widgets
    self.image_mosaic.sort_rect_widgets(method)
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'sort_rect_widgets'
Traceback (most recent call last):
  File "/Users/lonny/.pyenv/versions/3.12.0/lib/python3.12/site-packages/vars_gridview/scripts/run.py", line 314, in _sort_widgets
    self.image_mosaic.sort_rect_widgets(method)
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'sort_rect_widgets'
Traceback (most recent call last):
  File "/Users/lonny/.pyenv/versions/3.12.0/lib/python3.12/site-packages/vars_gridview/scripts/run.py", line 314, in _sort_widgets
    self.image_mosaic.sort_rect_widgets(method)
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'sort_rect_widgets'
Traceback (most recent call last):
  File "/Users/lonny/.pyenv/versions/3.12.0/lib/python3.12/site-packages/vars_gridview/scripts/run.py", line 314, in _sort_widgets
    self.image_mosaic.sort_rect_widgets(method)
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'sort_rect_widgets'

Hi @lonnylundsten, this error seems to indicate that the image mosaic (grid) hasn't been created yet. Are you trying to sort before loading any results? I just tested this prior to running any query and got the same error -- I'll put in a patch that avoids this error and instead shows a warning dialog. But in any case, sorting won't do anything before you have results loaded. Sorting after the results are loaded seems to work as normal on my end, so any more information to reproduce this would be appreciated.

I just pushed out v0.4.1 which adds a guard (and warning dialog) against sorting before the grid is loaded.

I'm pretty sure the grid was loaded. Will double check again as soon as this dive loads.

It's working now. I'll close the ticket!


Change SQL schema to have linking tables that allow multiple children per tutor_session and multiple children per tutor_sesion while maintaining only one child per semester_tutoring_obj

Done
[t:15]


feat(cheatcodes): mockCall with bytes4 data

Motivation
Closes #9213

Solution

Add overloads for mock cheatcodes with bytes4 data arg to pass the selector like token.balanceOf.selector instead of abi.encodeWithSelector

looks good, I'll push a change to make the same for vm.mockCallRevert sigs and then merge if OK with you @yash-atreya

Woot, thank you all!


Delete cookies on v2

I read the new docs but can't found how to delete a cookie.
before v2 I can easily cookie.remove('cookie').
Thanks.

It is still there in v2. My bad I forgot about it on the doc. Now it's there.


Added anime category class

This contributes to #42

Codecov Report

Merging #69 into main will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #69   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           92        94    +2
  Lines          795       795
=========================================
  Hits           795       795

Impacted Files
Coverage Δ

qual_id/categories/anime.py
100.00% <100.00%> (ø)

qual_id/pattern.py
100.00% <100.00%> (ø)

test/categories/test_anime.py
100.00% <100.00%> (ø)

test/utils/category_helper.py
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

Hey @gremlin97 , thanks for sorting out those failing tests. I've mentioned this to other people who've contributed, but in CONTRIBUTING.md, where I've requested that spaces and special characters not be included, what I mean is that I would like values to be single words. Some of your contributions here are clearly fine, like 'naruto', but others that are just multiple words combined are not. Would you be able to still add this category, but only use single words? Let me know your thoughts 🙂

Yeah Sure :smiley: . Now that I think about it having complex concatenated strings definitely beats the purpose of qual-id :sweat_smile: .

Great contribution @gremlin97 ! Thanks a lot! 🎉  🙂

Thanks for the help :tada:.  Looking forward to contribute in the future :smile:.


[fix][android] allow bundling to be disabled for release

It was not previously possible to disable the bundling of assets for android when bundling in release mode. That meant that it would always fall back to true, because setting bundleInRelease to false will fall through to targetName.toLowerCase().contains("release") (which would be true).
Closes #18892 where I tried this earlier but had problems with the builds that I couldn't understand and seemed unrelated.
Test Plan:
I've been running this patch using patch-package for a while, without any dramas.
It can be tested pretty easily. On current master, set bundleIn<variant>Release to be false. The bundle will still happen. Switch to this branch and make the same change and the bundle will be disabled.
Release Notes:
[ANDROID] [BUGFIX] [react.gradle] - Allow disabling of bundling for release builds

How good react-native master builds are failing too.

Ah, seems like we actually made the same change on master already. Thanks for your PR and sorry for not getting back to you earlier, but at least it is fixed :)

@cpojer I think it was my previous PR (#18892) where this got merged in to master. All good 😄

Hey! Is there an equivalent of this issue for iOS?


Cannot add reservation with website

Describe the bug
After installing the IPAM solution on 30-11-2022 I added a space and in that space I added two blocks.
If I then select the space and then the first block, I would like to add a reservation.
This does not work.
To Reproduce
Steps to reproduce the behavior:

Go to Configure in the menu
Click on the first space in the list
Cllick on the first block in the list that appears
Click on the three dots to open the menu and select 'Reservations'

You then get a screen like this:

It is not possible to add a reservation and it looks like some screens are mixed up.
Expected behavior
I would like to be able to add a reservation after the steps I listed above.
Desktop (please complete the following information):

OS: Windows 11
Browser: Microsoft Edge
Version 107.0.1418.56 (Official build) (64-bit)

Smartphone (please complete the following information):
Not used
Additional context
No

Never mind, I see in the docs that it is not yet supported...


Update from apps failed even when theire finished.

11:00:40 - Updating VSCodium from 1.63.2 to 1.65.2...
11:00:44 - ##########   WINGET UPGRADE PROCESS STARTS FOR APPLICATION ID 'VSCodium.VSCodium'   ##########
-\
No applicable update found.
-
Found VSCodium [VSCodium.VSCodium] Version 1.65.2
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://github.com/VSCodium/vscodium/releases/download/1.65.2/VSCodiumUserSetup-x64-1.65.2.exe
-\|/-\|/-\
▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒  1024 KB / 75.2 MB
█▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒  5.00 MB / 75.2 MB
█████▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒  15.0 MB / 75.2 MB
█████████▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒  24.0 MB / 75.2 MB
█████████████▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒  34.0 MB / 75.2 MB
█████████████████▒▒▒▒▒▒▒▒▒▒▒▒▒  43.0 MB / 75.2 MB
█████████████████████▒▒▒▒▒▒▒▒▒  53.0 MB / 75.2 MB
█████████████████████████▒▒▒▒▒  63.0 MB / 75.2 MB
████████████████████████████▒▒  71.2 MB / 75.2 MB
██████████████████████████████  75.2 MB / 75.2 MB
Successfully verified installer hash
Starting package install...
-\|/-\|/-\|/-\|/-\|/-\|/-\|/-\|/-\|/-\|/-\|/-\|/-\|/-\|/-\|/-\|/-\|/-\|/-\|/-\|/-\|/-\|/-\|/-\|/-\|/-\|/-\|/-\|/-\|
Successfully installed
11:01:38 - ##########   WINGET UPGRADE PROCESS FINISHED FOR APPLICATION ID 'VSCodium.VSCodium'   ##########
11:01:38 - VSCodium update failed.
11:01:42 - No new update.
11:01:42 - End of process!

Hi. It might be related to your app. No applicable update found. is probably indicating that Winget doesn't know how to handle the app. As a workaround, we put a winget -install behaviour.
What I would suggest is trying to uninstall your app manually, then install the 1.63.2 version with Winget, and try to update with WAU.

All right thanks!
But it still got the error when I try updating winget-autoupdate.

is your app installed from EXE or MSI ?

is your app installed from EXE or MSI ?

Thats a good question. I think its MSI since I prefer them.
But if i installed it over Winget they use exe

Maybe Winget doens't how to handle your app when installed as msi :(

Maybe Winget doesn't how to handle your app when installed as msi :(

:(
But how can winget detect if an app is already installed?
Its over the same api that the old uninstall panel from control center is using.
And there you have the msi as well right?

i'm sorry but i'm note a Winget dev. So maybe open a case on Winget repo 🤷♂

Currently having the same issue with VLC. But it's related to Winget or the way VLC is detected (even VLC dev problem maybe)

Currently having the same issue with VLC. But it's related to Winget or the way VLC is detected (even VLC dev problem maybe)

https://github.com/microsoft/winget-pkgs/issues/58397


List all lowering implementation in developer docs

With the growing number of implemented lowering and specializing overloads that can be in different source files, it becomes harder to find the definition location of the lowering of a supported nopython function.  This patch adds a autogeneration script that runs at documentation generation to add a section in the developer doc with a full listing of all lowering implementation.  In addition to help finding definition location of lowering implementation, this will also:

show implemented nopython features and the supported signatures
show how the lowering is implemented via source listing
help understand the specialization/generialization for potential refactoring.

When Siu first showed me this, it seemed kind of crazy, but it actually becomes oddly compelling to browse through and learn how to write new lowerings.  We could put it in another section besides the developer docs, but I think the idea is interesting.

Is there an online demo of what this outputs? :-)

Ok, so I ran the script. I agree the list of signatures for each implemented function is an interesting read, but I really don't like the inclusion of source code. It makes the page massive and difficult to navigate accross. I think it's enough to write the source files and line numbers beside each signature.
Also, you can make hypertext links to the source code on Github, for example "numba/targets/tupleobj.py lines 77-80" is at https://github.com/numba/numba/blob/master/numba/targets/tupleobj.py#L77-L80 (so the pattern appears to be https://github.com/numba/numba/blob/master/{filepath}#L{firstlineno}-L{lastlineno})

I like the idea of hyperlink to GH.  Should be possible to make it point exactly to the commit from which the doc is generated.

yeah, I have found the source links in the numpy docs to be invaluable.

Also, hopefully not including the source code will reduce the processing time (although the clean way to solve this may be to write a Sphinx extension).

[Current coverage][cc-pull] is 87.35%

Merging [#1890][cc-pull] into [master][cc-base-branch] will decrease coverage by -0.08%

@@             master      #1890   diff @@
==========================================
  Files           292        293     +1
  Lines         52207      52285    +78
  Methods           0          0
  Messages          0          0
  Branches       5397       5405     +8
==========================================
  Hits          45673      45673
- Misses         5610       5688    +78
  Partials        924        924

Powered by Codecov. Last updated by [dc8a633...97c1d9c][cc-compare]
[cc-base-branch]: https://codecov.io/gh/numba/numba/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/numba/numba/compare/dc8a633b257a264d134d2701d16005681329e7c2...97c1d9c7a90a4b49e59f313fa2033897bc448da9
[cc-pull]: https://codecov.io/gh/numba/numba/pull/1890?src=pr

I think this is good enough now, we can fine-tune later. @seibert, what do you think?

Yeah, with the switch to github links, I'm happy with this.


Landmark ids

Hey, great work! I was going through the repository but cannot find the what the ids corresponding to each of the landmarks are. I want the coordinates of some of the specific landmarks, say NoseTip, etc. Can someone help me with this.

hi, i did it myself, drawing the ids next to the points and recording a video of my face. what are the specific landmarks that you want?


iot-installer produced ISO fails to update "opening repo: opendir(/run/install/repo/ostree/repo): No such file or directory"

Describe the bug
Systems installed with an iot-installer produced iso, cannot be updated:
rpm-ostree upgrade
error: While pulling fedora/rawhide/x86_64/iot: opening repo: opendir(/run/install/repo/ostree/repo): No such file or directory

Environment

Fedora Koji

To Reproduce
Steps to reproduce the behavior:

Download a recent Fedora IoT Iso created with osbuild-composer:
https://koji.fedoraproject.org/koji/packageinfo?packageID=36296

Install a VM and attempt an update.

rpm-ostree upgrade
error: While pulling fedora/rawhide/x86_64/iot: opening repo: opendir(/run/install/repo/ostree/repo): No such file or directory

cat /etc/ostree/remotes.d/fedora.conf
[remote "fedora"]
url=file:///run/install/repo/ostree/repo

Adding a new remote and rebasing to 'fedora-iot' worked to update:
ostree remote add fedora-iot --contenturl=mirrorlist=https://ostree.fedoraproject.org/iot/mirrorlist --set=gpgkeypath=/etc/pki/rpm-gpg/ fedora-iot

rpm-ostree rebase -b fedora-iot:fedora/rawhide/x86_64/iot

This section of the fedora iot kickstarts shows how it was dealt with previously.

I wonder if we need need a blueprint option to specify the ostree remote OTA (Over The Air) update

@paulwhalen @nullr0ute I've been working on this PR (coming a bit later) by replicating the %post section in the kickstart through osbuild and osbuild-composer.
Currently the kickstart always sets the ostree remote to the Fedora upstream one after install. I would like to set the ostree remote to the one supplied during image build time after install.
Is that OK or must it be the Fedora upstream?

I wonder if we need need a blueprint option to specify the ostree remote OTA (Over The Air) update

What is the "remote OTA"?

So, after speaking with @ondrejbudai there's two ways I can go with the PR.

Include an RPM which contains the ostree remote information.
Write the ostree remote information in the kickstart stage's %post as the current kickstart does.

I've built 2, but would prefer 1. :)

@paulwhalen, @nullr0ute, can you please have a look at the solutions that Simon proposed and give us a guidance. We would love to close this asap to finish our preparations for Fedora 38. ;)

I would agree with @supakeen , #1 would be preferable and can be used as needed.

No action required on our side, the IoT team will ship the remotes in an RPM.

Once the rpm is shipped, we'd probably want to remove the hard-coded remote in the iot-raw-image (and add the rpm to our package set):
https://github.com/osbuild/osbuild-composer/blob/0e4a5b34b214a6496b22f808fae402f88bebe6a2/internal/distro/fedora/images.go#L397-L402
Should we open a separate issue for this?


Datasets for testing

I will create different sets of data for testing as we address the open issues

[x] Dataset 1: A small set of flattened images and metadata, all placed in a numpy array.  It assumes one object per image.  This is what currently works with the notebook example.  Train/test will be the same dataset.
[x] Dataset 1_full: A larger version of Dataset 1.  Will be split up into train/test datasets
[x] Dataset 2: The same as Dataset 1, but metadata will be separated and saved in a json file
[x] Dataset 2_full: The same as Dataset 1_full, but metadata will be separated and saved in a json file
[x] Dataset 3: A small set of flattened images, with separated json metadata.  It will account for multiple objects per image. Train/test will be the same dataset
[x] Dataset 3_full: A larger version of Dataset 3_full, split into train/test

Datasets are stored on HAL under /home/shared/hsc/DC2/test_data/

Datasets are stored on HAL under /home/shared/hsc/DC2/test_data/


Fixup existing secret clients to use new file-token-server

Please add to https://github.com/edgexfoundry/edgex-go/projects/13
Changes:

Modify any existing secret clients to use the new go-mod-core-security that supports per-service tokens (#1717)
Ensure that in the Dockerized implementation that secret-clients are mouting the required secrets (PKI CA and token file)
Pass in an appropriate VAULT_TOKEN_URL

Closing and replacing with #1930


chore(package): update webpack-cli to version 3.0.8

Closes #107

Pull Request Test Coverage Report for Build 494

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

Totals

Change from base Build 490:
0.0%

Covered Lines:
77

Relevant Lines:
77

💛  - Coveralls


docs(getting-started): adding clarifications for easily missed steps

Just some clarifications and making sure the instructions on adding the detail component are clearer.

Okay, I gonna squash + rebase your commits + some changes, on latest master once I've merge https://github.com/angular/router/pull/122

Sounds good. Thanks.


doc:  fix errors checked by markdownlint

Signed-off-by: Michael Wan<EMAIL_ADDRESS>Ⅰ. Describe what this PR did
remove trailing space:  %s/\s\+$//e
replace tab to 4 spaces: %s/\t/    /g
ignore errors:
MD002 - First header should be a top level header
MD003 - Header style
MD004 - Unordered list style
MD013 - Line length
MD024 - Multiple headers with the same content
MD026 - Trailing punctuation in header
MD036 - Emphasis used instead of a header
MD038 - Spaces inside code span elements
add tags:
MD029 - Ordered list item prefix :   -t or
Ⅱ. Does this pull request fix one issue?
Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #974 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #974   +/-   ##
=======================================
  Coverage   12.94%   12.94%
=======================================
  Files         123      123
  Lines        8314     8314
=======================================
  Hits         1076     1076
  Misses       7142     7142
  Partials       96       96

Continue to review full report at Codecov.

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

LGTM. Thanks a lot for your work. This is very important for the document. @HusterWan


Look up characters from user's drawing

Some open source tools that can do this may already exist. For example:

https://github.com/gugray/HanziLookupJS [JavaScript]

Out of scope


json schema validation and autocomplete not working for non default helm value files

When using non default values files like values-dev.yaml there is no schema validation or autocomplete functionality.
Would be great to be able to activate the plugin functionalities for custom file patterns.

Hello @paologentili ,
JSON schema provider is currently provided for 2 patterns:

<chart_root_dir>/values.yaml: aggregation for chart
<chart_root_dir>/*-values.yaml: aggregation for packaged chart

I'm considering removing the second pattern (*-values.yaml) because it is too much linked to naming conventions in my company.
The JSON schema for packaged chart would however still be generated even if it is not applied on any file by default.
In IntelliJ, JSON schema mappings provide the ability to select any JSON schema to validate a JSON / YAML file.
In bottom-left corner, you just need to click on "No JSON schema" to display a list of available JSON schemas.
JSON schemas generated by helm-values plugin are shown in that list.\

Once JSON schema is selected for the file, configuration is persisted for your project.
Auto-completion and validation of values is then available in the file.
JSON schema mappings can be modified in project settings.
I'm considering this IntelliJ feature as sufficient to support non-default values files.
I don't plan to add custom file patterns in plugin configuration..

thanks. using the json schema mapping is good enough for my use case


tls: first record does not look like a TLS handshake when connecting over ssl

@freeformz @evanphx @bmizerany @uggedal @benburkert
Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.7.5 darwin
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/workspace/GO_CODE"
GORACE=""
GOROOT="/usr/local/Cellar/go/1.7.5/libexec"
GOTOOLDIR="/usr/local/Cellar/go/1.7.5/libexec/pkg/tool/darwin_amd64"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/5g/xv46lbz53b73lpr3wjfqc9h00000gn/T/go-build665777701=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"
What did you do?
I am trying to connect to kafka server which is enabled with ssl. I have created the tls configuration with necessary certs. But it fails with the error tls: first record does not look like a TLS handshake.
The same thing works with python client but not with go client.
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.
What did you expect to see?
A successful handshake.
What did you see instead?
tls: first record does not look like a TLS handshake.

Maybe your Kafka server is doing STARTTLS instead of TLS straight away.
In any case, there's not enough information in this bug report to help. This is almost certainly a configuration problem of some sort.

Why did 5 of us get randomly ping'd at the start of this issue...?

Evan, good question. I'm going to close this.


Add minTime and maxTime properties to TimePicker

Hi ,  I suggest adding two poperties, minTime and maxTime. the input value would be checked if are in the range ( if it's greater than minTime or it's less than maxTime or it's in between minTime and maxTime properties) after filling the input; If it is not in range the function prop onError would be trigged and the input highlighted in red. Thk you.

Duplicate of #519


Update networking.md

There might be a typo in "one peers" which could be corrected to "once peers".
Thanks.
Motivation

Description

Closes #issue_number
Steps to test or reproduce

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi there!  Thanks for noticing that and making the PR.  We use conventional commits and the title of this PR will become the commit message.  Please update it to the conventional commit standard.  I would recommend something like docs: fix typo in networking.md

:tada: This PR is included in v1.14.0 :tada:


summarize_by_time - corr

corr does not appear to be a valid agg_function
For example:
df \
    .groupby("category_2") \
    .summarize_by_time(
        date_column='order_date',
        value_column= ['total_price'],
        freq = "MS",
        agg_func = ['corr']
    )

Will generate the error:
AttributeError: 'corr' is not a valid function for 'DatetimeIndexResamplerGroupby' object
I think simply modifying the docstring here:
        - "sum": Sum of values
        - "mean": Mean of values
        - "median": Median of values
        - "min": Minimum of values
        - "max": Maximum of values
        - "std": Standard deviation of values
        - "var": Variance of values
        - "first": First value in group
        - "last": Last value in group
        - "count": Count of values
        - "nunique": Number of unique values
        - "corr": Correlation between values <- Just remove

is the simplest solution. I am not entirely sure what the intended use for corr here was anyway, was it for comparing to features/covariates or was it meant to compare from t1 to t2 to t3 ...
Regardless should just tweak the docstring for now.
In addition, the function as currently written includes a 'kind' parameter, this defaults to 'timestamp', but that it will work for 'period' is also not specified. This should be included in the docstring.

Thanks for pointing this out. I'm wondering if there should be a functionality to return corr or summarizations that account for multiple columns. This is the same thing we are running into with augment_rolling.


Why root-relative EPE is higher than absolute EPE (Table.1 in the paper)

Hi,
Thanks for awesome work! I just have one confusion that I want to clear.
In Table.1 of your paper, you provide quantitative results (mm) of evaluations. I am confused why RR-V/RR-J are higher than MPVPE/MPJPE? As per my understanding, it should be like: PA-MPJPE < RR-MPJPE < absolute MPJPE.
Thank you.

Thank you for raising this issue,
We hypothesis that this behavior may closely related to the choice of the root joint. We have selected the 0-th point to align the vertices and joints. However, since the 0th joint is on the edge of the mesh model, using the coordinates of an edge point as the offset for the entire mesh model could potentially amplify the error.
I suggest experimenting with the 9-th joint (MCP joint) for root-relative MPJPE computation to see if the error decreases, which would help verify our hypothesis.
Best,

The error does decrease if I select MCP point. Thanks for clarifying this


Update mapsinR26May2023.qmd

@aporter121 use "quarto publish gh-pages" in the terminal before committing, seems to fix any issues

See: https://nhsengland.github.io/nhs-r-reporting/tutorials/R_drop_in_session_26_may_Maps_in_R/mapsinR26May2023.html


⚠ Bofrost.es has degraded performance

In 77c3019, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1615 ms

Resolved: Bofrost.es performance has improved in c75586a after 6 minutes.


Incorrect message on home page.

Level 2 is completed still it is showing wrong message on home page.

Verified this issue on dev app v1.9.5(338)


Warn when key, ref or slot are declared in props

  <demo key="key1"></demo>

//demo components
<template>

test demo

</template>

The key is a special attribute, along with ref and slot. You can not use them as prop names.
However I think Vue should give a warning when you try to declare them as props.

@simplesmiler  thanks


Fix: remove dead rpc nodes, added teia node

We got a report from a user that was unable to mint a piece in discord, they were using a broken rpc node in the settings. Switching to another node fixed it.

So this removes custom?

I vote for keeping custom.

I'm sorry, I thought that was a placeholder. How can I use the custom option? :/


This page is missing from the navigation

This page is missing from the navigation on the left

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

ID: cac22961-da69-7b22-6dff-f7bc5ee82ed3
Version Independent ID: dd4ed11e-a0f9-4e97-48ae-8bd94f510c07
Content: How the SharePoint Migration Tool works
Content Source: migration/how-the-sharepoint-migration-tool-works.md
Product: sharepoint-server-itpro
GitHub Login: @JoanneHendrickson
Microsoft Alias: jhendr

@officedocsbot assign @scanum

@danmyhre Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

@officedocsbot close


Add case to handle no previous release tag in create changelog

Description
The changelog is created using a previous release tag as reference point. In case no tag exists, the script failed.
Changes proposed in this pull request:

Script uses hard coded sha to use as reference point in case no previous release tag exists to create the changelog
Add comments from NM repo to the script for explanation of the steps

Related issue(s)

Closed, as problem was handled by adding a tag 0.0


Box symbols in prompt

Prerequisites

[x] I have read and understand the CONTRIBUTING guide
[x] I looked for duplicate issues before submitting this one

Description
Tried running in Windows Terminal and in Powershell directly, the symbols in many themes are not appearing correctly.
Environment

Oh my Posh version: 3.21.0
Theme: powerline
Operating System: Windows 10 2004
Shell: Powershell 7.0.3
Terminal: Windows Terminal
Font: Cascadia Code PL  2009.022

Steps to Reproduce

Run Get-PoshThemes

Expected behavior: No box symbols
Actual behavior:

@AakashM you have two options, use a nerd font on which the icons have been based (and working), or override the icons with icons which are available in Cascadia Code (but to be honest, switching font is the most straightforward option).

Thanks! Switching to nerd font has solved the issue.

 I made sure to also add that context here.


add annotations

As user might need to add some annotations for each pod, this PR is going to fix it.

I didn't add it the random generation process, while the annotation could be a long array and it seems to be a special case, while if someone needs it, they can simply add the annotation to the randomly generated json and run the generator again in preset mode!

But we can add it if you think people might need it.
For our special case, based on our RQs that we defined, we are not going to use random generation.


Fix return bomb from farming

fixes #24

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.4%) to 85.047%

Files with Coverage Reduction
New Missed Lines
%

contracts/ERC20Farmable.sol
1
75.27%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
130

Relevant Lines:
144

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.2%) to 85.238%

Files with Coverage Reduction
New Missed Lines
%

contracts/ERC20Farmable.sol
1
75.28%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
128

Relevant Lines:
142

💛  - Coveralls


Failed to define custom ZapiPerf collector file (other than defaults.yaml)

Describe the bug
When I try to change the collector configuration file for ZapiPerf like describe at the end of the section (change to not use the default.yaml) : https://github.com/NetApp/harvest/blob/main/cmd/collectors/zapiperf/README.md#harvest-configuration-file
I got the error : "cannot unmarshal !!map into string"
Environment
Provide accurate information about the environment to help us reproduce the issue.

Harvest version: harvest version 21.05.4-1 (commit 8976370) (build date 2021-08-02T11:24:55+0200) linux/amd64
Command line arguments used: ./bin/harvest start --foreground --verbose--collectors ZapiPerf --objects SystemNode DC-01
OS: Debian 10.9
Install method: dpkg install from manual package compilation
ONTAP Version: CDOT 9.5

To Reproduce
Steps to reproduce the behavior:
Create a file limited.yaml in folder /opt/harvest/conf/zapiperf/ :
root@XXX:/opt/harvest# ll conf/zapiperf/
total 16
drwxr-xr-x 3 harvest harvest 4096 juil.  1 22:33 7mode
drwxr-xr-x 4 harvest harvest 4096 août   6 16:59 cdot
-rw-r--r-- 1 harvest harvest 1798 août   2 11:24 default.yaml
-rw-r--r-- 1 harvest harvest   95 août   6 15:44 limited.yaml

With the following content:
root@XXX:/opt/harvest# cat conf/zapiperf/limited.yaml
objects:
  SystemNode:             system_node.yaml
  HostAdapter:            hostadapter.yaml

And change the harvest.yml on the application root folder :
root@XXX:/opt/harvest# cat harvest.yml
...

Defaults:
  collectors:
    - ZapiPerf
  use_insecure_tls: true

Pollers:
  DC-01:
    datacenter: Lab
    addr: 192.168.xxx.xxx
    auth_style: basic_auth
    username: admin
    password: xxxxxxx
    exporters:
      - my_influxdb
    collectors:
      - ZapiPerf:
        - limited.yaml

Got the error when a try to launch with command :
root@XXX:/opt/harvest# ./bin/harvest start --foreground --trace --collectors ZapiPerf --objects SystemNode DC-01
error unmarshalling config file=[./harvest.yml] yaml: unmarshal errors:
  line 25: cannot unmarshal !!map into string
config [./harvest.yml]: yaml: unmarshal errors:
  line 25: cannot unmarshal !!map into string

Line 25 corresponds to:
      - ZapiPerf:

thanks for the detailed report @albinpopote this feature was accidentally omitted with the transition from Python to Go.


HEAD build wants bazel >= 3.7.4 and <4.0.0 which are unavailable

System information

OS Platform and Distribution: Debian 10
TensorFlow installed from (source or binary): source
TensorFlow version: git HEAD
Python version: 3.7
Bazel version (if compiling from source): 4.0.0
GCC/Compiler version (if compiling from source): 8.3

Describe the problem
Nightly builds of Tensorflow want Bazel >= 3.7.4 so at Linaro CI we fetched 3.7.4 and built TF. Worked fine.
Upstream dropped Bazel 3.7.4 release so now 3.7.2 and 4.0.0 are available on their Github releases page. We moved to 4.0.0 and then TF complains that it is too new.
So at the moment Tensorflow nightly is not buildable on our CI.
Provide the exact sequence of commands / steps that you executed before running into the problem
Fetch bazel 4.0.0, build and install. Then build Tensorflow.
Any other info / logs
00:50:24.871     spawn ./configure
00:50:24.871     You have bazel 4.0.0- (@non-git) installed.
00:50:24.871     Please downgrade your bazel installation to version 3.99.0 or lower to build TensorFlow! To downgrade: download the installer for the old version (from https://github.com/bazelbuild/bazel/releases) then run the installer.
Whole log: https://ci.linaro.org/job/ldcg-python-tensorflow-nightly/nodes=docker-buster-arm64-ldcg/116/console

Coworker pointed me to 3.7.2 being minimal.


Why Bootstrap forces display: x on .visible-{size} classes?

I don't quite understand why Bootstrap does this, why not have a simple .visible-{size} class which won't apply the display: block/inline/inline-block? Why have 3 separate .visible classes for each display type?

which won't apply the display: block/inline/inline-block?

I don't quite know what you mean by that.
Anyway, there are 3 sets of classes because we have no way of magically knowing which display value (block/inline/inline-block) is the right one to apply in any given situation. So the developer, who is in a position to know the appropriate value, needs to select the appropriate corresponding class.
In v4, we've reworked the visibility classes so that only .hidden-* classes exist, thus sidestepping the problem (see http://v4-alpha.getbootstrap.com/migration/#responsive-utilities ) since they never declare any non-none display values. When the viewport size is such the element should be shown, the .hidden-* class's media query simply no longer applies, allowing the original display value to apply, thus showing the element.

Closing since Bootstrap v3 is in maintenance mode and no longer accepting new features (such as hypothetical additional visible/hidden classes) anyway.


broken of validates_xxx_of

Hi, after upgrade reform from 2.0.0.rc1 to 2.0.0 , the active_model's validations help methods dose not work, like validates_numericality_of, validates_length_of. in Reform::Form::ActiveModel,  only some basic mehtods were delegate to the Validator which include ActiveModel::Validations, but the validates_xxx_of are not.  Will they come back ?
I add an initializer file to fix it in our code
Reform::Form.instance_eval do
  class << self
    delegates :validator, :validates_absence_of, :validates_acceptance_of, :validates_confirmation_of,
                          :validates_exclusion_of, :validates_format_of, :validates_inclusion_of, :validates_length_of,
                          :validates_size_of, :validates_numericality_of, :validates_presence_of
  end
end

They are still there, but  we don't include ActiveModel::Validations into the form itself anymore and only delegate their DSL.
Here is where this is done: https://github.com/apotonick/reform/blob/master/lib/reform/form/active_model/validations.rb#L24
You can add your delegations there, if you want and I merge it. But before that, I'd like to know, why do you use this form and not the validates validator: .. form?

legacy codes.... when rails introduce the new validates way, they did not deprecate the validates_xx_of, so we keep it, and move it from model the reform. I think i will change my code to the validates form, and using my own initializer file as a workaround for now. thanks.
and I think  we should metion this in the CHANGES.md

Maybe you're right!


RAINCATCH-536: Add support for refactored fh-wfm-message / fh-wfm-message-angular modules.

closing as won't do


vueAdmin-template as a component

Hi guys,
Hope you are all well !
I am trying to create a chrome extension, based on this template https://github.com/ALiangLiang/vue-webpack-chrome-extension-template#/, where I would like to inject a sidepanel (https://github.com/raychar/vue-cd-sidebar) and vueAdmin-template as a vue component.
But I always get an error
vue.esm.js:591 [Vue warn]: Failed to mount component: template or render function not defined.

found in

---> <VueAdminLite>
       <App> at src/content/App.vue
         <Root>

here is my playground:
File ./content/components/admin-lite/index.js
import Vue from 'vue'

import 'normalize.css/normalize.css'// A modern alternative to CSS resets

import ElementUI from 'element-ui'
import 'element-ui/lib/theme-chalk/index.css'
import locale from 'element-ui/lib/locale/lang/en' // lang i18n

import<EMAIL_ADDRESS>// global css

import Content from './root'
import router from './router'
import store from './store'

import '@/content/components/admin-lite/icons' // icon
import '@/content/components/admin-lite/permission' // permission control

Vue.use(ElementUI, { locale })

Vue.config.productionTip = false

router.beforeEach(function (to, from, next) {
  console.log('router.beforeEach.to: ', to)
  console.log('router.beforeEach.from: ', from)
  next()
})

router.afterEach(function (a) {
  console.log('router.afterEach: ', a)
})

const VueAdminLite = new Vue({
  el: '#admin',
  router,
  store,
  render: h => h(Content)
})

/*
// This is your plugin object. It can be exported to be used anywhere.
const VueAdminLite = {
  // The install method is all that needs to exist on the plugin object.
  // It takes the global Vue object as well as user-defined options.
  install (Vue, options) {
    Vue.component('VueAdminLite',
      new Vue({
        el: '#admin',
        router,
        store,
        render: h => h(Content)
      })
    )

    // We call Vue.mixin() here to inject functionality into all components.
    // Vue.mixin({
      // Anything added to a mixin will be injected into all components.
      // In this case, the mounted() method runs when the component is added to the DOM.
    //   mounted () {
    //     console.log('VueAdminLite is Mounted!')
    //   }
    // })
  }
}
*/

// Automatic installation if Vue has been added to the global scope.
// if (typeof window !== 'undefined' && window.Vue) {
//   window.Vue.use(VueAdminLite, {debug: true})
// }

export default VueAdminLite

File: ./content/App.vue:
<template>

    <button :class="['snk-button', menuState ? 'snk-button-hidden' : '']"
            @click="openMenu">SNK</button>

              <vue-admin-lite></vue-admin-lite>

</template>


Icon buttons in datatable are misaligned

Rows in a datatable are 48px height, but because it's hardcoded like that, when we put some other things than just text, the alignment is therefore wrong.
Here's an example with icons:

The spec has an example with icons:

Can we get a codpen example that reproduces the issue please? Seeing the exact markup structure being used may help find a robust solution.

Sorry, seems I forgot to comment.
Creating an example was not hard. Here's one: http://codepen.io/anon/pen/mVqrVR

So quick possible patch here:

Change .mdl-data-table td .mdl-data-table__select to no left rule. Vertical-align middle. And no absolute position.
Change .mdl-data-table td to vertical-align middle and padding to also have 12px on the bottom.
Change header checkbox to have no extra rules.

See this demo for output.
@sgomes and @surma Seem like something we can drop in real fast? Not sure about direct spec alignment, but it gets things functional.
Or should we just hold off and handle this in 2.x?

@Garbee My vote goes to MDL 1.1

Alrighty, I'll work on fixing up a PR for this real quick.
FTR there are BC concerns changing this stuff so heavily... but... ugh, whatever.


Prepare for 2.0 release

@paf31 I know we talked about this before, but is there a reason not to provide an instance like this, rather than making every type with a Functor instance implement its own using imapF?
I thought one of the advantages of disallowing orphans is it made instances like this perfectly acceptable. Also, it's not like there's a case here where there's a more efficient implementation for imap for existing Functors - unlike using rmap for the Functor instance for Bifunctors, etc.

There is still an issue with potentially overlapping instances though, although I think in this case, parametricity forces any two instances to be equivalent.

Changing this will require a lot of other libraries to be updated, which wouldn't need to be updated otherwise.

That's fine since I'm bumping everything anyway :wink: (see IRC, I think it needs to happen for this cycle at least).

No wait! There is another issue. This instance matches every type constructor, whether it's a functor or not.

Right you are!


feat: reset password feature

Changes
Closes #667

Checklist

[x] I followed contributing guidelines

@mmularski
Thanks for your contribution! :)
Is "No matching customer for the email..." message returned from Shopware 6 API or it is somehow validated on Shopware PWA end? A common practice is to always return a successful resetting message in a way "If your user account really exists then check your email for the link to reset a password.". No matter if an account exists or not.

@rmakara
It is Shopware API response.

After a successful operation, modal just closes and email is sending. I was modeling on login operation.

After successful invocation (no API error) please hide email input and button, the display below message and let the user close the modal by himself. When he'll get this feedback on the action he knows what happened and what should he expect next :)
After this, PR is ready as everything else looks 👌
Message:

You should receive a link in a few moments. Please open that link to reset your password.

@patzick fixed. Now after a successful operation, modal looks like below


Command palette and search box visual tweaks

Tweaked command palette and search box to better match the Windows 11 Fluent design:

Now styled like a normal flyout, with acrylic, shadow and border
TextBox now uses default style
Tweaked button and key chord designs
Adjusted spacing

There is another design for the palette suggested here.
Which one would be better?

Wow I friggin love it. I'm offline for the weekend already, but lemme ping the team on Monday.

I, too, love this! Holy wow!

I somewhat prefer the first design, as it is consistent with the search box and what VSCode does. Giving the controls some space affords us some visual distance between the shadow, tab content and search.
How does the list look in tab switcher mode? (by default, Ctrl+Tab will trigger it; keep holding down Ctrl to see the switcher)

Ah, and how does this look in "commandline" mode? If you backspace the implicit >m it will change the UI up a bit.

The tab switcher has a weird space at the top, I'll try to fix it.

A couple more ideas with the other design:

This is gorgeous, I love it so much

I think this is waiting on the fix for the tab switcher top gap, right? I'm happy to review it still! I love it so much.

I fixed the spacing, but there is still a crash happening sometimes when typing and deleting text inside the command palette. It can be easily replicated by typing >, backspace, >.

I fixed the spacing, but there is still a crash happening sometimes when typing and deleting text inside the command palette. It can be easily replicated by typing >, backspace, >.

I'll try debugging this more later today. Wild that it doesn't even like give a useful stack trace - it just explodes 🤔

The issue happens when MinHeight in _filteredActionsView has values from 5 to 13. There's a connection to the negative padding, but I'm not sure why a key combination causes this.

The issue happens when MinHeight in _filteredActionsView has values from 5 to 13. There's a connection to the negative padding, but I'm not sure why a key combination causes this.

I suspect it has nothing to do with the key combo, but more about the showing/hiding/showing of certain UI elements. XAML can be finicky like that sometimes. At this point, we might just need to mess with the padding and general spacing to try and work around this.

That was because of a very small issue: when the list is empty, it still takes up space (6px), when it should be collapsed.  I made it 8px to look more balanced, but I'll revert that.


graph: remove Has method from Graph and Multigraph

Please take a look.
Closes #601.

Absolutely.
Note also that I've just cleaned up a couple of cases that I missed (and deleted a // +build ignore'd file that was written against the pre-iterator API (and I presumably left accidentally while debugging).

I should add. Having the changes here, I have tested gorgonia to see if it breaks. It doesn't. I suspect that few people actually use g.Has(id) outside gonum/graph since usually people are either using our routines or doing g.Nodes() and graph traversal, which together imply that g.Has(id) is true.

PTAL


fix: release version in scripts/makefile-snippets/makefile-release.mk

What this PR does / why we need it:
fix: release version in scripts/makefile-snippets/makefile-release.mk
Release note:
NONE

/retest

/retest

/lgtm

/retest


KDF option not supported on pass for iOS or android Password Store

Pretty much summarizes it. Using a yubikey with KDF on these apps always results in "incorrect PIN" error and decrements the PIN retry counter. Disabling KDF allows PIN entry to work.
Tested error on both Pass for iOS beta (adding yubikey support) and Password Store for android. Fix tested on pass for iOS only.
Recommend adding a note on the KDF section showing this risk with KDF setups, and the disadvantages of having it off. It is possible for someone to use this guide with NFC Yubikeys, aiming to use PGP on their phones, and not notice the source of the problem.
Can send a pull request with the relevant changes.

Thank you for the PR!


Improvement on changeset listing

I'm submitting a feature request.
Brief Description
When the user access a changeset with a direct link, like: https://osmcha.mapbox.com/changesets/50845434/, instead of listing the last changesets of any place in the world in the left sidebar, make a query using the bbox of the current changeset.
What is the motivation / use case for this feature?
Add more useful information for the user. But if it would request a lot of time to be implemented, I believe it's not a priority.

This is a nice idea @willemarcel .
There are a couple of problems with the changeset list. The user doesn't clearly have an idea what the list is about, unless he applies a filter.
How do we communicate to the user that the list you are seeing on left is the bbox query or whatever the current context is.?

Good question... one idea: change the filter button colour to a darker one when a filter is active.
When querying by the changeset bbox, add a message in the sidebar: "Filtering by current changeset bbox" (maybe the text is longer than ideal...)


YouTube 403 forbidden error on audio urls

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

Did something change in the YouTube audio formats, or are they returned differently than before? After merging https://github.com/ytdl-org/youtube-dl/commit/e1b3fa242cb94eb9dc949ab3f2cace91c46f11bf into my codebase, I am getting (somewhat) random failures, but still very repeatable. I wasn't getting these errors with the previous version of ytdl, but there was a new error (https://github.com/ytdl-org/youtube-dl/issues/32842#issuecomment-2247729123) so had to update.
Plex is giving me 406 (unacceptable) errors when I try to upload the audio.
2024-07-26 20:31:01,164 (1e38) :  ERROR (logkit:22) - 7: Error uploading media: (406) not_acceptable; http://<IP_ADDRESS>:32400/library/metadata/7/themes?url=https%3A%2F%2Frr2---sn-vgqsknll.googlevideo.com%2Fvideoplayback%3Fsparams%3Dexpire%252Cei%252Cip%252Cid%252Citag%252Csource%252Crequiressl%252Cxpc%252Cbui%252Cspc%252Cvprv%252Csvpuc%252Cmime%252Cns%252Crqh%252Cgir%252Cclen%252Cdur%252Clmt%26ei%3DdwekZoOfFMiL2_gPr9aZ8QY%26ip%3D<ip_address>%26clen%3D2952380%26spc%3DNO7bAUSfQKp7wA51RtQGKsRLBRf5mthsur--MRW5fbsriZPkNA1qre4UysJp%26id%3Do-AAoY5UDjTl1nbz2ILx_0-6f07PC6a8wTiWVAlOzqqb7U%26keepalive%3Dyes%26svpuc%3D1%26gir%3Dyes%26xpc%3DEgVo2aDSNQ%253D%253D%26requiressl%3Dyes%26source%3Dyoutube%26mv%3Du%26sig%3DAJfQdSswRQIhAMf8bjXRaGzgiRSBkOqQ_D39zP9NvzqawnrrT5DUi7FiAiB5sA67GgEuGddyT9dBkO93JerPVjcdWgiGzv499zHq4A%253D%253D%26dur%3D185.852%26ns%3DsfVMzFqD1xnov-0SyNvSksYQ%26vprv%3D1%26lsig%3DAGtxev0wRQIhALDSyv0QtzedMTElYBf8wrYFLVu9AVKMUkS73YdG0JmCAiB1MAf-1V11K54VOSB7cgbzRiqKJlpJmgEwfMoqD0sIog%253D%253D%26lsparams%3Dmh%252Cmm%252Cmn%252Cms%252Cmv%252Cmvi%252Cpl%26lmt%3D1381172018924357%26c%3DWEB%26sefc%3D1%26expire%3D1722047447%26mime%3Daudio%252Fmp4%26fvip%3D1%26rqh%3D1%26itag%3D140%26mm%3D31%252C29%26mn%3Dsn-vgqsknll%252Csn-vgqsrnsr%26mh%3DRH%26n%3DByy7QEICFdfW5A%26mt%3D1722025513%26bui%3DAXc671J3DF8IaVyrVeX7P2gjoWIta83r8-VJE-frLLoYb-Qu34nQWYMq_tCvsDUUawSv_qeOTvQLUkSo%26pl%3D23%26ms%3Dau%252Crdu%26mvi%3D2 <title>Not Acceptable</title>
406 Not Acceptable

Here is my code for getting the audio url: https://github.com/LizardByte/Themerr-plex/blob/c948caa73991e05cebe8b24b88bce5ccb6bede4c/Contents/Code/youtube_dl_helper.py#L133-L161
I tested locally, and I'm still getting audio urls, but could be in formats that aren't supported by Plex. Last update for Plex was July 9th, so nothing on their end changed in the last few days.
Any ideas?

I dug a little deeper and the issue if that the url provided by youtube-dl is not accessible and gives a 403 error. This seems to be very random.
https://github.com/LizardByte/Themerr-plex/actions/runs/10119441078/job/27987941216?pr=486#step:8:276
This seems like a new development since the commit I linked above. Could youtube-dl validate that the url it gives is accessible before returning it? Possibly with some retry logic? I guess it may be too complex to validate all the format URLs.

Could you post the original youtube url you're fetching so we could see what youtube-dl is returning/do some debugger-work?

@aiur-adept sure, it can occur on seemingly any video. Here's two examples:

https://www.youtube.com/watch?v=uXZd_W5B7N0
https://www.youtube.com/watch?v=CIvUnfNiUXI

Ah yeah, I've just seen the behaviour. It failed with 403 then after about 10 minutes it worked. Weird that they're sending a 403 when it's likely a rate limit. I'm not sure how to solve this as having the process wait for 10 minutes on a 403 isn't really a good solution.

If you are post-processing the download URLs, you could try using the equivalent back-up URL (specified in its query parameters) on unexpected 403, or 4xx generally. This has so far been deemed too hard in yt-dl[p], as it would need a custom downloader; also AFAIK no-one has conclusively shown that the backup URL work better than the original one.
The algorithm for generating the backup URL is in an old issue (also at yt-dlp) which I expect I can find if you can't.

I tried a small retry loop in my code as well, and it might have helped... still testing though.
I also tried not using a cookie file after seeing this: https://github.com/yt-dlp/yt-dlp/issues/10046 ... that also may have helped... still testing that too.

Weird that they're sending a 403 when it's likely a rate limit.

From what I can see this is kind of standard for google products.
@dirf, I am not using youtube-dl to do the downloading, and I'm handling that on my own. I did see this https://github.com/yt-dlp/yt-dlp/issues/4100#issuecomment-1159457298 I'm not sure I 100% understand though.

What's the algorithm for generating the backup url?

Probably this PR is the best source.

No luck: https://github.com/LizardByte/Themerr-plex/pull/486/commits/81df3d28c70213bb262ca1cff32c8f99140824b1

Don't want to start a new thread on an issue others are already reporting, but I'm confused as to which build I should download.
I just ran into the '403 forbidden error' when attempting to download a youtube audio file (m4a). My youtube-dl was out-of-date (2023) as I hadn't used in a while so I downloaded the latest daily @ https://github.com/ytdl-patched/youtube-dl (7/30), and I still get the error. I can't recall why I had that particular page bookmarked, but I realize there are multiple forks of this program and I honestly have no idea what I'm doing!
Assuming this error has been fixed (or even if not), I would be grateful if someone could point me to the best place to get updates going forward.  Thx

if someone could point me to the best place to get updates going forward.

https://github.com/ytdl-org/ytdl-nightly/releases

I found that the issue was at least partially because of m4a chosen as the best audio format. If I force it with 'bestaudio[ext=webm]' then previously 403 error'd URLs download with no issues. Still not a good solution because there's a few URLs with no webm format.

@JstaClarkey thank you for that information. I'm also using m4a. I guess I will need to incorporate ffmpeg to solve this.

This very much looks like the poToken experiment.
yt-dlp solved this by retrying the web api when the experiment is detected, since rollout is not yet at 100%.
In addition, yt-dlp switched away from the web client. As afaik. youtube-dl only support the web client, this will be more difficult to backport.

Yes, I've made a WIP that tests for poToken and so it could be made to retry.
The multiple client thing is conceptually straightforward (ie, instead of a single result to be traversed there is a list of results) but also brings in some related catch-up issues like extractor args.

@dirkf I'll be happy to test when it's ready. My CI tests about 30 videos, so it's pretty easy to hit this error.

My initial testing gives somewhat strange results. Getting the title from each item in a playlist brings up the poToken warning quite regularly. Downloading item 1 of the list works fine regardless of whether the warning appears.
Similar results with the two videos above.

Do you have a live branch for this?

Continued in #32905.


Add real support for RetryParsers?

For retry parser to work, the prompt needs to be available. I don't see how this is hooked up in the current implementation of  LLMChain. The LLMChain must provide the prompt and the corresponding generation to the appropriate output parser (I.e. the llm code must invoke parse_with_prompt). The real output parser could either use the base implementation of parse_with_prompt (i.e. simply call parse) or override it.
WIP: Needs tests, unsure if schema changes affects consumers(don't think a lot of people are subclassing output_parser). If my assessment is wrong, I will drop the work

@baskaryan I am not sure if RetryOutputParsers are properly supported. I had to do these changes to do retries with the booleanparser used by the llm filter chain (Please see https://github.com/langchain-ai/langchain/issues/11408). Please let me know if the changes make sense?

Any update on this PR? @sudranga - perhaps you need to tag someone to review it?

@baskaryan Let me know if these changes look appropriate.

Any update @sudranga @baskaryan ?

@timxieICN I don't have anything to add to this PR. @baskaryan What are your thoughts?

Currently we cannot really use RetryParser. It expects to use parse_with_prompt instead of parse. Not easy to add it to a given chain. Any news on this PR?

@louisoutin I don't know. For some reason, there's no interest in merging this PR.  If one of the repo owners agrees that this PR is useful, i will resolve the branch conflicts appropriately.

@sudranga one naive question, new to LangChain here. I was going over the documentation of Retry Parsers, how to collect the bad_response in the code. When i add the parser to a LLM Chain and if the output from the llm is not structured i was getting the RuntimeError from the parser.
To implement the RetryOutputParser you need the bad_response/partial_completion from the previous chain to pass right? How do i get that, in the documentation they initialised the bad_response instead of collecting from the chain directly

@hwchase17 @baskaryan I'm not sure what you want to do with this PR? If you think the PR is appropriate, i can resolve the conflicts.

Apologies for the slow review! Pr has some merge conflicts, happy to re-review if you'd like to resolve

@baskaryan Please check.

@baskaryan Do you have any concerns?

@baskaryan I redid the pull request(to address merge conflicts) because i assumed you would take a look. Now, there is a merge conflict again.
This particular topic of retry parsers seems to be of interest to folks. It takes time and effort to produce a PR.  I'm disappointed that this effort has been of no value towards improving langchain.
@hwchase17 ^^

Any updates on this?

@sudranga thanks for putting in the effort here, and sorry for the delay in reviewing. At this point in time, I am going to close this PR. we are considering how to best support retries for output parsers and will likely involve a larger refactor. I very much apologize for the delay in review and poor communication here


refactor: rename nav schematic to navigation

Our docs often describe the nav schematic as navigation schematics,
and it's not quite clear what the actual schematic name is. To avoid
this confusion, we just name it navigation, but keep support for
the old nav abbreviation.

@IgorMinar I made it so that nav is still supported. Just navigation is now supported, and the default/primary name. Does that make sense to you?

@devversion Ok that sound great!


Unable to compile bevy_wgpu for WASM

Bevy 0.3.0
So I wanted to run a simple WASM app, but there's a problem with bevy_wgpu. When I compiled without it, I got this error in browser console: "panicked at 'Resource does not exist alloc::boxed::Box<dyn bevy_render::renderer::render_resource_context::RenderResourceContext>". And it makes sense, because renderer uses wgpu backend, but then I added bevy_wgpu and compilation failed with many errors related to web_sys (for example "failed to resolve: could not find GpuShaderModuleDescriptor in web_sys").
So is there any way to run bevy with wgpu on WASM?

Currently I'm not aware of anyone getting bevy/wgpu/web working entirely yet, as takes a few more steps to align branches of wgpu with firefox's current support for webgpu https://github.com/gfx-rs/wgpu-rs#run-examples-on-the-web-wasm32-unknown-unknown
While waiting for webgpu and browser support to stabilize a bit more, the current path is to use a webgl backend. https://github.com/bevyengine/bevy/pull/613 adds WebGL2 backend to bevy and examples of it is showcased in https://mrk.sed.pl/bevy-showcase/

Yup @memoryruins is right. That being said I'd love to get WebGPU support working!

There is also interest in targeting WebGL in wgpu https://github.com/gfx-rs/wgpu/issues/115#issuecomment-722136678

While waiting for webgpu and browser support to stabilize a bit more, the current path is to use a webgl backend. #613 adds WebGL2 backend to bevy and examples of it is showcased in https://mrk.sed.pl/bevy-showcase/

bevy_webgl2 is available as external plugin now: https://github.com/mrk-its/bevy_webgl2
It is still lacking documentation, but there are working examples and simple shell script  for building them

This is resolved now; wgpu just works as of 0.6.


[WIP] Migrate get link modal to be pure and use Redux

Summary
Migrate get link modal to be pure and use Redux.
Ticket Link
mattermost-server/issues/7679
Checklist

[ ] Added or updated unit tests (required for all new features)

@saturninoabril Here is the PR. I'm stuck at getting the tests passed because of some unsupported DOM API by jsdom, namely queryCommandSupported and execCommand.
Related issues on jsdom:

https://github.com/tmpvar/jsdom/issues/1539
https://github.com/tmpvar/jsdom/issues/1742

Another finding is using the Object.defineProperty on document doesn't seem to work because of the Unforgeable properties in document that makes it not configurable as mentioned in this issue

@tkbky I've found a workaround, on mocking like:
const supportedCommands = ['copy'];

Object.defineProperty(document, 'queryCommandSupported', {
  value: (cmd) => supportedCommands.includes(cmd)
});

Object.defineProperty(document, 'execCommand', {
  value: (cmd) => supportedCommands.includes(cmd)
});

Also added/modified some tests.

@saturninoabril That looks great! Thanks for making the PR way better. 😄

Awesome, thanks @tkbky


Unique-Key constraint configurable

There is not jet a possibility to create containers with a unique-key-constraint.
Because of that it would be great if one could specify those. Maybe one could add an attribute to the Property that should be unique, int the Item that implaments the IItem interface.
Then when a container is created, the type it is created for is checked for this attribute.

@VeryCautious  I was just publishing version 2.1.7 when I realized that the UniqueKeyAttribute was private. It wouldn't have been exposed to any consumers, not sure how I missed that in the code review - but I need to fix that first. LOL

This is in version 2.1.8:
https://www.nuget.org/packages/IEvangelist.Azure.CosmosRepository/2.1.8
Should be live soon! Thanks again


idea: unify with predicate

(tests
  (def h (clojure.java.io/file "src/"))
  (type h) := java.io.File
  (datafy h)
  := #:user.datafy-fs{:name "src",
                      :lastModifiedTime _,
                      :size _,
                      :isDirectory ::folder,
                      :children ~seq?})                ; here

as well as {:children [~(instance? java.io.File %) & _]}

https://github.com/nubank/matcher-combinators


Custom Action fix

The plugin name in the provided Custom Action is not matching the plugin name in the manifest file -> Custom Action is not working
PR from my personal account since I cannot create a branch in this repository

@davidrapin @aminerhanemi @w8r not allowed to add reviewers for this PR: can you review it?

@andre91a done!


DBZ-5035 pass sink config props to db history adapter

do you mean regarding referencing the variable or in general? I can add a
constant at the class level.
another option is to pass the entire config to the history adapter instead
of limiting it to the history related settings.
WDYT?
On Mon, Jun 20, 2022, 13:54 Jiri Pechanec @.***> wrote:

@.**** commented on this pull request.
In
debezium-core/src/main/java/io/debezium/relational/HistorizedRelationalDatabaseConnectorConfig.java
https://github.com/debezium/debezium/pull/3626#discussion_r901532081:

@@ -101,7 +101,7 @@ public DatabaseHistory getDatabaseHistory() {
}

     // Do not remove the prefix from the subset of config properties ...

   Configuration dbHistoryConfig = config.subset(DatabaseHistory.CONFIGURATION_FIELD_PREFIX_STRING, false)

   Configuration dbHistoryConfig = config.filter(k -> k.startsWith(DatabaseHistory.CONFIGURATION_FIELD_PREFIX_STRING) || k.startsWith("debezium.sink."))

 Unfortunately the fix will not be that easy then as
server refs (even indirect) should not leak into the core.
—
Reply to this email directly, view it on GitHub
https://github.com/debezium/debezium/pull/3626#discussion_r901532081,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANAS5XAVDTL4UKEHXHGLZTVQBEUTANCNFSM5ZGTXVMQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Implemented via https://github.com/debezium/debezium/pull/3797


every upgrade destroys my settings

VSCode Version: Code 1.18.1 (929bacba01ef658b873545e26034d1a8067445e9, 2017-11-16T18:32:36.026Z)
OS Version: Windows_NT ia32 10.0.16299
Extensions:

Extension
Author (truncated)
Version

Ionide-fsharp
Ion
3.14.2

MagicPython
mag
1.0.12

vscode-azurefunctions
ms-
0.2.0

python
ms-
0.8.0

vscode-ai
ms-
0.1.4

azure-account
ms-
0.2.2

csharp
ms-
1.13.1

azure-hdinsight
msh
0.0.9

vscode-docker
Pet
0.0.22

python
tht
0.2.3

usql-vscode-ext
usq
0.2.7

Steps to Reproduce:
1.add configuration setting for CNTK to settings.json and launch.json for a python project. work cleanly
2.do any upgrade to VS code  lose all settings that worked for CNTK and have to redefine it each and every time. it is driving me insane

Reproduces without extensions: Yes/No

@jimwill3 Couple of questions

What are CNTK settings?
Do you loose all settings or just settings from an extension?


Python Client 1.0.0 suffers connection unavailable after service restarted

If I maintain an open channel connection and try to request after service restarted, the first attempt will always failed with error
<_Rendezvous of RPC that terminated with (StatusCode.UNAVAILABLE ...

to reproduce this, I modify greeter-client.py to request 2 times after service restarted
diff --git a/examples/python/helloworld/greeter_client.py b/examples/python/helloworld/greeter_client.py
index 44d42c1..9f6234a 100644
--- a/examples/python/helloworld/greeter_client.py
+++ b/examples/python/helloworld/greeter_client.py
@@ -31,7 +31,7 @@

 from __future__ import print_function

-import grpc
+import grpc, time

 import helloworld_pb2

@@ -41,7 +41,17 @@ def run():
   stub = helloworld_pb2.GreeterStub(channel)
   response = stub.SayHello(helloworld_pb2.HelloRequest(name='you'))
   print("Greeter client received: " + response.message)
-
+  print("Sleeping for 5 sec..., please restart service now")
+  time.sleep(5)
+  print("Service should be up by now, try to request again")
+  try:
+      response = stub.SayHello(helloworld_pb2.HelloRequest(name='you'))
+      print("Success on 1st attemp after service restart: " + response.message)
+  except Exception as ex:
+      print("Failed with error %r", ex)
+      print("Retry now ...")
+      response = stub.SayHello(helloworld_pb2.HelloRequest(name='you'))
+      print("Success on 2nd attempt after service restart: " + response.message)

here the output
Greeter client received: Hello, you!
Sleeping for 5 sec..., please restart service now
Service should be up by now, try to request again
Failed with error %r <_Rendezvous of RPC that terminated with (StatusCode.UNAVAILABLE, {"created":"@1475751516.250124000","description":"EOF","file":"src/core/lib/iomgr/tcp_posix.c","file_line":235,"grpc_status":14})>
Retry now ...
Success on 2nd attempt after service restart: Hello, you!

:+1:

Just updated everything and the problem seem still there... very annoying

Maybe this makes sense in a wider context (upcoming load balancing features, or for clients of stateful servers). But it does not make sense in our narrow context ☺ At the moment, every time we make a request we have to check the return code and retry (once) if the error is "service unavailable". Perhaps there could be a way to label a channel as being OK to reconnect over (so long as the disconnection happened while there were no outstanding requests, and so long as the first reconnection attempt succeeds) for naive clients like ours?

This problem boils down to the c core, aka I can reproduce it with the other wrapped languages.

However, there is probably no small fix for this at the moment. Without doing periodic health checks, there is no way for the client to know that its connection with the original server is no longer good. It needs that first failure to go through the process of remaking the connection.
Good news is that this will be fix once retry support is integrated in to the c core. That is an ongoing project and will be accomplished within the next three months

@ncteisen is the retry feature already implemented?

Not yet but it being very actively worked on! See #10684 to follow the development

thanks

Proper retry support is still a WIP, but we added a temporary workaround a while back where all channels will poll connectivity state, so this should not longer be an issue.

@kpayson64 what version is needed to have the workaround? I don't see a mention in releases.

@ijl
#11154 Was intended to address this issue, so anything 1.3.x or later should be able to reconnect.

@kpayson64 thanks.