Giving "messages" certain names can have side effects

Example: if you give a message the name "Update", this will currently actually create the same code as the "Execute every frame again" block.
void Update () { ... }

We should make sure that messages produce a different method name in the code when this occurs. (Like, add a number)

Accidentally closed issue because of submit which also merged history of source blockly github project.

This has been fixed now. The generated code will no longer use names which have some other meaning in Unity.


feat: kanon 0.3.0

Codecov Report

Merging #40 (b752d5d) into master (12a7a67) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #40   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           12        12
  Lines          372       372
=========================================
  Hits           372       372

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 12a7a67...b752d5d. Read the comment docs.


Refactor unit tests

https://trello.com/c/a6izDPez/930-separate-integration-tests-from-unit-tests-for-valid-schemas-3
An initial attempt at separating larger integration scenarios from unit tests.
I've focused on the main command specs which are extensive, some of these were good candidates for moving to integration specs and calling via the endpoints.
There's also some work done to DRY up common examples and build a common context for the new integration specs for the /v2/content endpoint

Thanks @thomasleese I think this is rebased and up to date now.


search history should not show empty list

If I don't have any past search queries, then don't show me a search history. If I have only 1 or a few past search queries, then show only those.

this was a part of the tests for search history - disabled it


[ios] fix bug uifont lead crash in iOS system less than 8.2

[ios] fix bug uifont lead crash in iOS system less than 8.2 and  add font fallback for   system less than 8.2

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: kfeagle:x: 齐山齐山 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


feat: auto save report in directory in bulk mode

If the browser crashes during a long bulk import, it is frustrating to lose the report which a status where it was...)

The report is saved if:

bulk process
the "Save locally as .docx is checked" AND the directory is selected AND permissions are given.


feat(i18n): support german

Hey again, @Leecason !
I completely redid the PR, now it has only one commit. I added the new translations for images (misunderstood what you meant last time, sorry about that).

Thank you!

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


🐛 Extend helm feature gates

What this PR does / why we need it:
This PR aims to extend the featureGates to the bootstrap provider and controlPlane provider.
Which issue(s) this PR fixes:
Fixes #633
@furkatgofurov7

Somehow intersects with https://github.com/kubernetes-sigs/cluster-api-operator/pull/536

/ok-to-test

Somehow intersects with #536

yeap, however it has a pending comment https://github.com/kubernetes-sigs/cluster-api-operator/pull/536#issuecomment-2488131270 that needs addressing. It would be great to join the efforts here and have single patch to address both concerns (in fact one is extended version of second)
cc @kahirokunn

Thank you very much!
{{- if or $controlPlaneVersion $.Values.configSecret.name $.Values.manager }}
I have added this conditional statement on my side.
Therefore, if you could stop modifying this file, I believe we can merge your changes with mine smoothly:
hack/charts/cluster-api-operator/templates/control-plane.yaml

Please rebase the PR @gcezaralmeida


[Bug]: Binary Download Fail & Parse Error in windows

Extension Version
0.1.11
PHP Binary
Local PHP
Operating System
Windows
What happened?
Problem
As soon as I installed extension and started working on project, I got notification: "Downloading binary for Laravel extension", after a minute, got notification: "Failed to download binary for Laravel extension" and got following error:
   2024-12-17 07:47:03.920 [error] Parse Error:
   Auth Data

  2024-12-17 07:47:03.920 [error]
  define('LARAVEL_START', microtime(true));

  require_once 'd:\Workspaces\SOME\tool/vendor/autoload.php';
  $app = require_once 'd:\Workspaces\SOME\tool/bootstrap/app.php';

  class VsCodeLaravel extends \Illuminate\Support\ServiceProvider
  {
      public function register()
      {
          //
      }

      public function boot()
      {
          if (method_exists($this->app['log'], 'setHandlers')) {
              $this->app['log']->setHandlers([new \Monolog\Handler\ProcessHandler()]);
          }
      }
  }

  $app->register(new VsCodeLaravel($app));
  $kernel = $app->make(Illuminate\Contracts\Console\Kernel::class);
  $kernel->bootstrap();

  echo '__VSCODE_LARAVEL_START_OUTPUT__';
   echo collect(app()->getBindings())     ->filter(fn ($binding) => ($binding['concrete'] ?? null) !== null)     ->flatMap(function ($binding, $key) {         $boundTo = new ReflectionFunction($binding['concrete']);          $closureClass = $boundTo->getClosureScopeClass();          return [             $key => [                 'uri' => $closureClass->getFileName(),                 'class' => $closureClass->getName(),                 'startLine' => $boundTo->getStartLine(),             ],         ];     })->toJson(); ;
  echo '__VSCODE_LARAVEL_END_OUTPUT__';

  exit(0);

Its same error for Auth Data, HTTP Routes, Paths, Middleware, Configs, Translation Namespaces, Eloquent Attributes and Relations :
2024-12-17 07:47:04.022 [error] Parse Error:
Paths

2024-12-17 07:47:04.022 [error]
define('LARAVEL_START', microtime(true));
...same...

Environment

PHP Version : PHP 8.2.12 (cli) (built: Oct 24 2023 21:15:15) (ZTS Visual C++ 2019 x64)
Extensions Installed: dsznajder.es7-react-js-snippets, github.copilot, eamodio.gitlens, xdebug.php-debug, bmewburn.vscode-intelephense-client, bradlc.vscode-tailwindcss, mechatroner.rainbow-csv
Using 'php artisan serve' to run project
Composer is installed and added in path
php is added in path
laravel cli is globally installed using composer

Mimimal Code Sample
No response

Encountered the same issue

Encountered the same issue on windows 11: php8.3.11

Same issue.

Same issue. Does that mean that the extension is currently unavailable for Windows?

Same issue in Windows 11

Same issue in Windows 10

Same issue on windows 11

same issues!

same


fixes #32 vuejs side effect warning

Documented here:
<EMAIL_ADDRESS>

To use this in your project on the fork you can as follows in your composer.json:
"repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/TappNetwork/honey"
        }
],

"lukeraymonddowning/honey": "dev-fix-side-effects-warning",

Then run composer update.


Live chat sample

Per https://github.com/ampproject/amphtml/issues/14520#issuecomment-380153213, would be great to demonstrate the capability to trigger a live chat feature.
Not sure if we can get credentials for an actual live chat (which would probably be a bad idea regardless, to have folks triggering a lot of live chats), but we could potentially make a live chat UI with an iframe signaling ["drop your code here"]

@kristoferbaxter just shared an excellent idea: to hook up an example with a chat bot. Sounds like it would be easy to do and would be a great way to demo the flexibility


No audio while streaming torrent in webtorrent

What version of WebTorrent Desktop?
Version is not defined, I am using webtorrent in Linux and downloaded today.
What operating system and version?
Ubuntu 16.04
What did you do?
I have added a movie and try to watch it while downloading using webtorrent but there is no audio, when I open it in VLC, the audio is perfectly comes.

Duplicate of https://github.com/feross/webtorrent-desktop/issues/350 Will be fixed in next release.

If you want Dolby surround sound (AC3) support in WebTorrent, please star this Chromium issue: https://bugs.chromium.org/p/chromium/issues/detail?id=611517


Interested on contributing deepbots-panda into deepworlds?

Hello @KelvinYang0320,
First of all, congratulation for this works, it is amazing!
I would like to ask you if you are interested on contributing deepbots-panda at deepworlds repository? Deepworlds repository aims to collect various robotics scenarios that can be used as testbeds for students and researchers. Collecting multiple scenarios in one place might be useful for DRL open-source community. You can contribute deepbots-panda by opening a PR in deepworlds and we provide you an extensive code review. Feel free to ask for help.
Thank you in advance

@ManosMagnus Thank you for your deepbots and your compliments. Your deepbots really benefits our Panda project. I am interested in contributing it at deepworlds repo. After doing deepbots-panda code refactoring, i will open a PR.

@ManosMagnus
It might take more time for me to deal with some bugs and refactoring before contributing deepbots-panda into deepworld since I found some issues when doing deepbots-panda code refactoring.


Make vanderlay-industries.js work

batr should be transparent if possible?
Something like:
const { standarda, ava, playwright, rollup } = require('batr')

Not sure how to make that work, but will figure it out.

Hasn't got it to work like this yet:
const { standarda, ava, playwright, rollup } = require('batr')

But since the dependencies gets installed, I can do stuff like:
const test = reguire('ava')

Will investigate some more.

https://stackoverflow.com/questions/60250163/var-module-require-or-const-module-require

Fixed this for Ava, but it doesn't work for ESM and import, so skipping this part. This way it will be similar in rollup.config and misc. tests.
https://stackoverflow.com/questions/67607939/node-js-how-to-transparently-require-and-export-submodules


Ban tokio sleep and tokio filesystem writes via semgrep

semgrep doesn't work on mac so this I haven't tested this (cc @dpc)

Semgrep does not display validation errors, reported upstream.
Check with semgrep scan --config .semgrep.all.yaml.
Fixes needed:
> git diff HEAD
diff --git a/.semgrep.all.yaml b/.semgrep.all.yaml
index bd2655fa4b..3e82c10766 100644
--- a/.semgrep.all.yaml
+++ b/.semgrep.all.yaml
@@ -13,8 +13,8 @@ rules:
     - rust
   message: Overwritting existing files is rarely a good idea. Use fedimint-core::util::write_overwrite if you are really sure you want to overwrite existing data or `fedimint-core::util::write_overwrite::write_new` if you don't. Alternatively use `fs::File::options` and express intention explicitly.
   pattern-either:
-    - tokio::fs::write
-    - std::fs::write
+    - pattern: tokio::fs::write
+    - pattern: std::fs::write
   severity: WARNING

 - id: ban-file-create
@@ -29,6 +29,6 @@ rules:
 - id: ban-tokio-sleep
   languages:
     - rust
-  message: `tokio::time::sleep` doesn't work in WASM. Use `fedimint_core::task::sleep` instead.
+  message: "`tokio::time::sleep` doesn't work in WASM. Use `fedimint_core::task::sleep` instead."
   pattern: tokio::time::sleep
   severity: WARNING

but unfortunately this reports bunch of existing instances.

converted to draft because of ^^. feel free to take this over @dpc because semgrep doesn't work on my machine so i can't test it.

@dpc could you try to finish this off? lots of problems with tokio::time recently ...

Moved to https://github.com/fedimint/fedimint/pull/2564

https://github.com/fedimint/fedimint/pull/2564 did this


Ark of the Covenant transfers cards and doesn't score them

didn't get monument

Did you see what the new Monument achievement criteria is?

Woah, no I didn't.  That is a huge change.

Indeed!


Plugin for WebdriverIO ("wdio" / wdio.conf.js)?

Coming over from https://github.com/eslint/eslint/pull/18005#discussion_r1458067768: it'd be great to have a built-in plugin for WebdriverIO. It's a test framework that has a wdio.conf.js file & is used by ESLint.
I'd be happy to contribute this. I don't personally use wdio but it'd be a good motivator for me to contribute to Knip. 😄

Feel free to work on this! I think it should be quite similar to the Ava, Jest and Mocha plugins.
Looks like the main objectives are the same: getting the entry files right and translate things like framework: "mocha" to @wdio/mocha-framework.

Closing in favor of https://github.com/webpro/knip/issues/483


Make Airnode logger utility more usable for other projects

The current logging utility in Airnode was built only with Airnode in mind. We’re starting using it in other tools (Airkeeper, Airseeker) and we’re running into some limitations.
I propose merging the LogOptions fields meta and additional into one field allowing to add any key/value pair instead of the prepared few in meta. That way it’s not bound to one project but reusable everywhere.

It would be better if we moved the logging utilities into a separate repo.

It would be better if we moved the logging utilities into a separate repo.

What would be the difference?
Why is this blocked?

What would be the difference?

The logging utilities should be a generic library. It's awkward to depend on a generic purpose library of a specific service (Airnode) in other services we use. I am fine with first implementing this in the Airnode, but we should still discuss this and create a new task for it if necessary.

Why is this blocked?

Until the discussion is over and we've decided where to implement this.

I am fine with first implementing this in the Airnode, but we should still discuss this and create a new task for it if necessary.

I agree, let's do that. Implement first under Airnode and potentially move it after.

Created https://github.com/api3dao/airnode/issues/1312.

I've started working on this and will resume after vacation. If this issue becomes a blocker and I am not back feel free to take over.

@kolenic-martin I tested this on AWS, seeing the logs metadata (coordinator ID, chainId and provider name being set correctly). Can you test on GCP and locally please?

Sure, no prob. I'll be off till Monday, is it okay to do it then? Or should
I squeeze this in sometime today?

Monday is fine.

Hi @Siegrift, sorry for the late reply but I actually tested this earlier in the week but I pushed back writing the report as I juggled other issues. The log metadata seems to be valid both in GCP and the local output. Sorry I didn't give an OK earlier.


DatasetFolder incorrectly specifies required folder structure

📚 Documentation

The documentation for DatasetFolder specifies the following required folder organization:
root/class_x/xxx.ext
root/class_x/xxy.ext
root/class_x/xxz.ext

root/class_y/123.ext
root/class_y/nsdf3.ext
root/class_y/asd932_.ext

However, the source code for DatasetFolder shows that make_dataset() uses os.walk() to discover the files in each class' folder. This means that any file within the class folder or its subfolders that passes is_valid_file() will be included. To me, this behavior was unexpected.
A more complete specification of the format could be:
root/class_x/xxx.ext
root/class_x/xxy.ext
root/class_x/[...]/xxz.ext

root/class_y/123.ext
root/class_y/nsdf3.ext
root/class_y/[...]/asd932_.ext

cc @pmeier

Hi @rjbruin. Indeed you are correct. I think the easiest way is to adjust the documentation as you proposed. Do you want to submit a PR? If so ping me within that, so I can review.

Sure, I can make the PR. See #3131.


publish the source code of this nice book

Sir, would you please push this book's LaTex source code to this repository. It would really nice that the source code is published. With the help of the community, the quality of this book would be greatly improved.

Maybe @ToverPomelo could help you!


Can't install xdebug extension for PHP 5.3.x

Hi,
I don't know if this is normal or not, but I cannot install the xdebug extension at PHP 5.3.x. I am not sure if this problem affects all 5.3.x builds or just some of them, but I tried with 5.3.29 and with 5.3.15 too just for testing purposes. It is also possible that my problem is not just the xdebug extension related. I run Ubuntu 14.04.2 64bit system.
I can install the xdebug extension (with the phpbrew ext install xdebug command) without any problem at PHP 5.6.x, 5.5.x and 5.4.x, but I cannot at 5.3.x. The terminal output I get is the following:
$ phpbrew switch php-5.3.29
$ phpbrew --debug ext install xdebug
Exception: Invalid package version: xdebug @
Thrown from phar:///usr/bin/phpbrew/PEARX/Channel.php at line 2:

    1<?php
>   2namespace PEARX; use PEARX\ChannelParser; use DOMDocument; use Exception; use PEARX\Core; class Channel { public $name; public $alias; public $summary; public $primary = array(); public $rest; public $cache; public $downloader; public $retry = 3; public $channelXml; public $scheme = 'http'; public $core; public function __construct($host, $options = array() ) { $this->core = new Core( $options ); $this->channelXml = $this->fetchChannelXml( $host ); $parser = new ChannelParser; $info = $parser->parse( $this->channelXml ); $this->name = $info->name; $this->summary = $info->summary; $this->alias = $info->alias; $this->primary = $info->primary; $this->rest = $info->rest; } public function getBaseUrl() { return $this->scheme . '://' . $this->name; } public function getRestBaseUrl($version = null) { if( $version && $this->primary[$version] ) return rtrim($this->primary[ $version ],'/'); return rtrim($this->primary[ $this->rest ],'/'); } public function fetchPackageReleaseXml($packageName, $version = 'stable') { $baseUrl = $this->getRestBaseUrl(); $url = "$baseUrl/r/" . strtolower($packageName); if( $version === 'stable' || $version === 'latest' || $version === 'beta' ) { $version = @file_get_contents($url . '/' . $version . '.txt' ); } if (! $version) { throw new Exception("Invalid package version: $packageName @ $version"); } $url = $url . '/' . $version . '.xml'; $xmlStr = $this->core->request($url); $xml = Utils::create_dom(); if( false === $xml->loadXml( $xmlStr ) ) { throw new Exception("Error in XMl document: $url"); } return $xml; } public function fetchChannelXml($host) { $xmlstr = null; $xmlstr = $this->core->cache ? $this->core->cache->get( $host ) : null; if( null !== $xmlstr ) return $xmlstr; $httpUrl = 'http://' . $host . '/channel.xml'; $httpsUrl = 'https://' . $host . '/channel.xml'; while( $this->retry-- ) { try { if( $xmlstr = $this->core->request($httpUrl) ) { $this->scheme = 'http'; break; } if( $xmlstr = $this->core->request( $httpsUrl ) ) { $this->scheme = 'https'; break; } } catch( Exception $e ) { fwrite( STDERR , "PEAR Channel discover failed: $host\n" ); fwrite( STDERR , $e->getMessage() . "\n" ); } } if( ! $xmlstr ) { throw new Exception('channel.xml fetch failed.'); } if( $this->cache ) { $this->cache->set($host, $xmlstr ); } return $xmlstr; } public function getCategories() { $baseUrl = $this->getRestBaseUrl(); $url = $baseUrl . '/c/categories.xml'; $xmlStr = $this->core->request($url); $xml = Utils::create_dom(); if( false === $xml->loadXml( $xmlStr ) ) { throw new Exception("Error in XMl document: $url"); } $list = array(); $nodes = $xml->getElementsByTagName('c'); foreach ($nodes as $node) { $link = $node->getAttribute("xlink:href"); $name = $node->nodeValue; $category = new Category( $this, $name , $link ); $list[] = $category; } return $list; } public function getPackages() { $packages = array(); foreach( $this->getCategories() as $category ) { $packages[ $category->name ] = $category->getPackages(); } return $packages; } public function findPackage($name) { foreach( $this->getCategories() as $category ) { $packages = $category->getPackages(); if( isset($packages[$name]) ) return $packages[ $name ]; } } }
PHP Notice:  Undefined offset: 2 in phar:///usr/bin/phpbrew/CLIFramework/ExceptionPrinter/DevelopmentExceptionPrinter.php on line 2

Notice: Undefined offset: 2 in phar:///usr/bin/phpbrew/CLIFramework/ExceptionPrinter/DevelopmentExceptionPrinter.php on line 2
    3

Trace:

    0) PEARX\Channel->fetchPackageReleaseXml('xdebug', 'stable')
        from phar:///usr/bin/phpbrew/PhpBrew/Extension/Provider/PeclProvider.php: 2

    1) PhpBrew\Extension\Provider\PeclProvider->buildPackageDownloadUrl('stable')
        from phar:///usr/bin/phpbrew/PhpBrew/Extension/ExtensionDownloader.php: 2

    2) PhpBrew\Extension\ExtensionDownloader->download(PhpBrew\Extension\Provider\PeclProvider, 'stable')
        from phar:///usr/bin/phpbrew/PhpBrew/Command/ExtensionCommand/InstallCommand.php: 2

    3) PhpBrew\Command\ExtensionCommand\InstallCommand->execute('xdebug')
PHP Notice:  Undefined index: file in phar:///usr/bin/phpbrew/CLIFramework/ExceptionPrinter/DevelopmentExceptionPrinter.php on line 2

Notice: Undefined index: file in phar:///usr/bin/phpbrew/CLIFramework/ExceptionPrinter/DevelopmentExceptionPrinter.php on line 2
PHP Notice:  Undefined index: line in phar:///usr/bin/phpbrew/CLIFramework/ExceptionPrinter/DevelopmentExceptionPrinter.php on line 2

Notice: Undefined index: line in phar:///usr/bin/phpbrew/CLIFramework/ExceptionPrinter/DevelopmentExceptionPrinter.php on line 2
        from : 0

    4) call_user_func_array([PhpBrew\Command\ExtensionCommand\InstallCommand, 'execute'], ['xdebug'])
        from phar:///usr/bin/phpbrew/CLIFramework/CommandBase.php: 2

    5) CLIFramework\CommandBase->executeWrapper(['xdebug'])
        from phar:///usr/bin/phpbrew/CLIFramework/Application.php: 2

    6) CLIFramework\Application->run(['/usr/bin/phpbrew', '--debug', 'ext', 'install', 'xdebug'])
        from phar:///usr/bin/phpbrew/CLIFramework/Application.php: 2

    7) CLIFramework\Application->runWithTry(['/usr/bin/phpbrew', '--debug', 'ext', 'install', 'xdebug'])
        from phar:///usr/bin/phpbrew/scripts/phpbrew-emb.php: 2

    8) require('phar:///usr/bin/phpbrew/scripts/phpbrew-emb.php')
        from /usr/bin/phpbrew: 6

My phpbrew version is the currently latest 1.19.4

Looks like the xdebug extension from version 2.3.x has problems to build with some older versions of php 5.3.x. The following was successful:
phpbrew ext install --pecl xdebug 2.2.7

Unless there is a patch for xdebug served by pecl repo, I don't think we can easily fix it. The error messages could be much better, though ^^

You can get a list of releases of given extension with phpbrew ext known <ext-name>:
phpbrew ext known xdebug

>>>
2.3.2, 2.3.1, 2.3.0, 2.2.7, 2.2.6, 2.2.5, 2.2.4, 2.2.3, 2.2.2, 2.2.1, 2.2.0,
2.2.0RC2, 2.2.0RC1, 2.1.4, 2.1.3, 2.1.2, 2.1.1, 2.1.1RC1, 2.1.0, 2.1.0RC1,
2.1.0beta3, 2.1.0beta2, 2.1.0beta1, 2.0.5, 2.0.4, 2.0.3, 2.0.2, 2.0.1, 2.0.0,
2.0.0RC4, 2.0.0RC3, 2.0.0RC2, 2.0.0RC1, 2.0.0beta6, 2.0.0beta5, 2.0.0beta4,
2.0.0beta3, 2.0.0beta2, 2.0.0beta1, 1.3.2, 1.3.1, 1.3.0, 1.3.0rc2, 1.3.0rc1,
1.2.0

Thanks for the help. Ok then, I will use an older xdebug version.

Actually znode_op is defined here:
https://github.com/php/php-src/blob/master/Zend/zend_compile.h
But it is not available in PHP 5.3.29:
https://github.com/php/php-src/blob/php-5.3.29/Zend/zend_compile.h
The latest xdebug version, which does not use znode_op is 2.1.4
@marcioAlmada Are you sure it works with 2.2.7?
I'v tried to install xdebug 2.1.4, but encountered #522. So currently I can not tell if it works or not.

You other report seems very confuse... I think that phpbrew is just failing to force the redownload of extension source. Please try:
phpbrew ext install --redownload xdebug 2.2.7

Confirmed here. The following works for php 5.3.29:
phpbrew ext install --redownload xdebug 2.2.7

===> Downloading from http://pecl.php.net/get/xdebug-2.2.7.tgz
===> /home/marcio/.phpbrew/distfiles/xdebug-2.2.7.tgz downloaded.
===> Extracting to /home/marcio/.phpbrew/build/php-5.3.29/ext...
[ ] xdebug extension is already disabled.
===> Installing xdebug extension...
Log stored at: /home/marcio/.phpbrew/build/php-5.3.29/ext/xdebug/build.log
Changing directory to /home/marcio/.phpbrew/build/php-5.3.29/ext/xdebug
===> Phpize...
===> Configuring...
===> Building...
===> Running make all: make -C /home/marcio/.phpbrew/build/php-5.3.29/ext/xdebug  all  >> /home/marcio/.phpbrew/build/php-5.3.29/ext/xdebug/build.log 2>&1
===> Installing...
===> Running make install: make -C /home/marcio/.phpbrew/build/php-5.3.29/ext/xdebug  install  >> /home/marcio/.phpbrew/build/php-5.3.29/ext/xdebug/build.log 2>&1
===> Extension is installed.
===> Creating config file /home/marcio/.phpbrew/php/php-5.3.29/var/db/xdebug.ini.disabled
===> Enabling extension xdebug
[*] xdebug extension is enabled.
Done.

It looks like phpbrew is reusing the source of the given extension once it's downloaded. It should always redownload the source in case command asks to install a different version :sweat_smile: Please, use the --redownload flag for now while I don't push a fix.

Okay, did not expect this behaviour. I'v cleaned the sources but it did not help, because it extracted the old archive from distfiles. :D Thanks.

Xdebug 2.2.7 works for me too at PHP 5.3.29.
I did a little bit differently. Instead of the --redownload flag at first I purged the whole PHP version, and then reinstalled (compiled) it, but --redownload is much more elegant :smile:

@marcioAlmada  I'v updated #522, including your --redownload proposal.

:+1:


investigate: getState related all regenFns's metric show empty data plus getPreState regenFn job metrics

on a cursory glance, seems like queued getState seems to be never called (which is our entrypoint for starting any regen metric).
however will there is a getState in api/impl/beacon/state/index.ts that further does resolveStateId. need to dig a bit deeper into that to figure out what is going on.

seems like getPreState computation is never getting queued for computation and is getting resolved in cache only.
it does get called from processBlocksInEpoch but gets resolved using cache's only in regen/queued.ts and never further ahead.
here is an effort to look up any observation across last 30 days in a prater running node"

@dapplion  should these metrics panels be removed from regenFn Stats of the metrics dashboard? or should I aggregate all these metrics in a single widget (multiple targets), someday if something shows up we can dig deeper
atleast we can remove the crowding from regenFn Stats

@dapplion should these metrics panels be removed from regenFn Stats of the metrics dashboard? or should I aggregate all these metrics in a single widget (multiple targets), someday if something shows up we can dig deeper
atleast we can remove the crowding from regenFn Stats

Sounds good do a PR with screen captures and let's see how it looks. If you can make all panels by 2 per row format would be great, there are still 2 full row width panels.

get preState graphs are now showing up when the node is experiencing the sync failure, which means these graphs can be retained,

will collapse only getState graphs (apart from also resizing the full width graphs into half width)

@g11tech Why is the cache hit ratio negative sometimes?

because its an approximation as we skipped counting the queued but are deriving it from queued processing. so assume at sec 1 one gets 10 requests, which gets queued, on second 2 queued length processes those 10, so delta(total) - delta(queued processed) creates these artifacts.
the better solution will be to count queued in the same function scope as well do that there is no drift when we do delta(total) - delta (queued). let me know if you would want me to make this change?


iOS 8.1 no country name

In iOS 8.1. I can't get country name. Please help me to fix it. Thank you. Code as follow:
NBMetadataHelper.m
NSString *country = [[NSLocale currentLocale] displayNameForKey:NSLocaleIdentifier value:identifier];
// country is nil.

It seems a iOS system bug.

Maybe it will helps...
    NSLocale *localLocale = [NSLocale currentLocale];
    return [localLocale objectForKey:NSLocaleCountryCode];

or use telephone sim locale.

It's value is also nil in iOS 8.1.

Right now, I got a answer. In iOS 8.1, If the country's name is nil. Just return it's country code. It may be more kind to user.

OK~


Mount host /proc to propagate nginx perf improvement settings

This is an alternative approach to https://github.com/giantswarm/nginx-ingress-controller-app/pull/55 to have nginx performance improvement settings (which got moved from nginx init container https://github.com/giantswarm/kubernetes-nginx-ingress-controller/pull/94/files#diff-c83f79df1a69a7a302bc5c17c284f945L46 to host OS https://github.com/giantswarm/k8scloudconfig/pull/537) to be propagated to nginx Pod.

Closed in favor of https://github.com/giantswarm/nginx-ingress-controller-app/pull/61


Add up to date videos of latest tasks to the README

The top level readme should have more links to videos of our latest and greatest tasks 🙂

cc @hartikainen, @nkhosh


ISPN-15212 TOUCH with CompletionStages.performSequentially

https://issues.redhat.com/browse/ISPN-15212

@wburns review https://github.com/infinispan/infinispan/pull/11366

@wburns changed to false, indeed is the way it should work

@wburns changed to false, indeed is the way it should work

Did you notice my other suggestion to change to use a summing collector instead? This avoids any List allocation and is a bit less code.

@wburns I merged it and I replaced then locally by mistake

Integrated into main, thanks @karesti !


Update Request: FishSET-Map-Viewer (FishSET_Map_Viewer)

Please provide the tool name
FishSET-Map-Viewer (FishSET_Map_Viewer)
Description
Recommend removing this tool. It hasn't been updated in 5 years and the author is no longer with the agency. I took over development of the FishSET tool last year and we'll have something to onboard as soon as we switch to github enterprise, so soon.
Code of Conduct

[X] I agree to follow the FIT's Code of Conduct

@lpfeif thanks! I removed the tool from the toolbox webpage. What should be done with the repository? Perhaps it should be migrated to where the other fish set tools live? It could also be "archived" to make it clear that future contributions won't be made to it.

@k-doering-NOAA The new map viewer will be part of the full fishset package. It should already be full archived (built upon in our repository). This "map viewer" was just separated out because it was functional without the rest of the package. I can check with my developer to see if there's anything in it he wants.

Oh, I see, thanks for explaining! If the changes are already incorporated in the full fish set, it can probably just be deleted. Let me know when you hear back from your developer to double check.


[rutorrent] add chart

Hey gang!
Once again, I love your work, and the library chart makes it so easy to deploy a simple but very extensible chart!
Here's a new chart for rutorrent - it uses "crazymax"'s image, since the linuxserver.io image is deprecated.
Cheers!
D
(The README was a copy/paste from autobrr, but I understand that it's regenerated automatically from values.yaml and Chart.yaml, so I didn't change anything - I hope this was the correct approach!)

We already have a rtorrent-flood helm chart, maybe name this chart rtorrent-rutorrent ?

We already have a rtorrent-flood helm chart, maybe name this chart rtorrent-rutorrent ?

That's sensible, will do :)


Add buttons on top and bottom of blocks

Currently the button's blockhandler doesn't handle placing buttons on the top and bottom faces of a block (1.8 feature?) Add the handling for those two block faces.

Same for levers. Or was that already implemented in an earlier version of MC.

Levers should work. I tried that a few times

I submitted a pull-request for this issue.

The pull-request has been merged. Should we close this issue?
@madmaxoft


Training code

Hi, When do you think you will release the training code?

Hi, Thanks for your interests. We should be able to release the training code in about two weeks.

Wow, awesome!Also can this be run in cpu mode?
Sent from Outlook Mobile
On Wed, May 4, 2016 at 11:08 PM -0700, "chuanli11"<EMAIL_ADDRESS>wrote:
Hi, Thanks for your interests. We should be able to release the training code in about two weeks.

You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/chuanli11/MGANs/issues/3#issuecomment-217082794

How long does the training take and on what gpu out of interest?
Sent from Outlook Mobile
On Wed, May 4, 2016 at 11:08 PM -0700, "chuanli11"<EMAIL_ADDRESS>wrote:
Hi, Thanks for your interests. We should be able to release the training code in about two weeks.

You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/chuanli11/MGANs/issues/3#issuecomment-217082794

Hi,
Here are some preliminary figures: a Titan X takes about an hour to train a model. Generating training data takes another hour or so. Other cards such as GTX980 should also do the job.
The first release will not support cpu.

Hello! cool work!
"The first release will not support cpu."
But will the generated models (trained with GPU) be able to be used in CPU, or we need also GPU for sampling?
thx

Can I marry you?

Training code released now. Welcome to feedback!


October 2023 Patch Release

We're currently processing your upload.  This comment will be updated when the results are available.


Use service defined by user as default

Summary
If the user defines a QiskitRuntimeService, then that service will be treated as the default service in the program.
Details and comments
This is implemented by adding a static member QiskitRuntimeService.global_service. The primitive will use that last defined service as its service.
Fixes #923.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

9 of 9 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 74.118%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
2437

Relevant Lines:
3288

💛  - Coveralls


Correct copyright and Author Emails for Lisk - Closes #18

Closes #18

Made the change yourself instead of merging this perfectly good PR? -1


clean up rviz render window resize

I was trying to debug this rviz points resize issue that was first reported on ROS noetic, but has since now affected every ROS release since.
I was trying to dig in and figure out what was going on....havent fixed it yet but I have found a few things that could be improved going forward.
I can make another PR for the main ros2 branch as well with these same changes if wanted.

I can make another PR for the main ros2 branch as well with these same changes if wanted.

Yes, please retarget this PR to the ros2 branch to start with.  Once we land it there, we can consider backporting to galactic (and foxy, for that matter).

Yes, please retarget this PR to the ros2 branch

on it


Outdated documentation on registering custom input type

The documentation is outlined at https://vueformulate.com/guide/inputs/custom-inputs/#registering-an-input is showing the usage of
Vue.use(VueFormulate, {
  library: {
    autocomplete: {
      classification: 'text',
      component: 'MyFormulateAutocomplete'
    }
  }
})

Where I can see that the correct usage now is actually the plugin approach:
export default function (formulateInstance) {
  formulateInstance.extend({
    components: {
      MyFormulateAutocomplete
    },
    library: {
      autocomplete: {
        classification: 'text',
        component: 'MyFormulateAutocomplete'
      }
    }
  })
}

Howdy! You do not have to write that as a plugin, most people choose to just register them directly during registration. The plugin approach is also valid, but the first example is not incorrect or deprecated.

Howdy @justin-schroeder! I regret not capturing more context in this issue, but I vaguely remembered that I spent hours troubleshooting why the first example is not working and ended up scouring through the innards of the documentation to make this work. Feel free to close if I got it wrong though. Thanks for maintaining this awesome library.

OK! Sorry you spent that time :( If this crops up again as an issue for you we'll take another look!


METRON-1296 Full Dev Fails to Deploy Index Templates

Fixed a bug that prevents the index templates from being installed.  This also ensures that the cause of an exception is logged.
Oops.  This should not have passed testing.   I reviewed the original PR and should have caught this.  I think a late change in the review process mucked things up.
To test this, spin up Full Dev and make sure that the index templates are installed.  Open up the Alerts UI and ensure that you can see alerts in the system.
Pull Request Checklist

[x] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[x] Has your PR been rebased against the latest commit within the target branch (typically master)?
[x] Have you included steps to reproduce the behavior or problem that is being changed or addressed?
[x] Have you included steps or a guide to how the change may be verified and tested manually?
[x] Have you ensured that the full suite of tests and checks have been executed in the root metron folder via:
[ ] Have you written or updated unit tests and or integration tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[x] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?

Worked for me when I ran it up in Full Dev.  I am able to see the templates and use the Alerts UI.
2017-11-03 22:22:41,294 - File['/var/lib/ambari-agent/tmp/bro_index.template'] {'content': StaticFile('bro_index.template'), 'mode': 0755}
2017-11-03 22:22:41,298 - Writing File['/var/lib/ambari-agent/tmp/bro_index.template'] because it doesn't exist
2017-11-03 22:22:41,299 - Changing permission for /var/lib/ambari-agent/tmp/bro_index.template from 644 to 755
2017-11-03 22:22:41,299 - File['/var/lib/ambari-agent/tmp/snort_index.template'] {'content': StaticFile('snort_index.template'), 'mode': 0755}
2017-11-03 22:22:41,300 - Writing File['/var/lib/ambari-agent/tmp/snort_index.template'] because it doesn't exist
2017-11-03 22:22:41,301 - Changing permission for /var/lib/ambari-agent/tmp/snort_index.template from 644 to 755
2017-11-03 22:22:41,301 - File['/var/lib/ambari-agent/tmp/yaf_index.template'] {'content': StaticFile('yaf_index.template'), 'mode': 0755}
2017-11-03 22:22:41,302 - Writing File['/var/lib/ambari-agent/tmp/yaf_index.template'] because it doesn't exist
2017-11-03 22:22:41,302 - Changing permission for /var/lib/ambari-agent/tmp/yaf_index.template from 644 to 755
2017-11-03 22:22:41,302 - File['/var/lib/ambari-agent/tmp/error_index.template'] {'content': StaticFile('error_index.template'), 'mode': 0755}
2017-11-03 22:22:41,303 - Writing File['/var/lib/ambari-agent/tmp/error_index.template'] because it doesn't exist
2017-11-03 22:22:41,303 - Changing permission for /var/lib/ambari-agent/tmp/error_index.template from 644 to 755
2017-11-03 22:22:41,303 - File['/var/lib/ambari-agent/tmp/meta_index.mapping'] {'content': StaticFile('meta_index.mapping'), 'mode': 0755}
2017-11-03 22:22:41,304 - Writing File['/var/lib/ambari-agent/tmp/meta_index.mapping'] because it doesn't exist
2017-11-03 22:22:41,305 - Changing permission for /var/lib/ambari-agent/tmp/meta_index.mapping from 644 to 755
2017-11-03 22:22:41,305 - Execute['curl -s -XPOST http://node1:9200/_template/bro_index -d @/var/lib/ambari-agent/tmp/bro_index.template'] {'logoutput': True}
{"acknowledged":true}2017-11-03 22:22:41,428 - Execute['curl -s -XPOST http://node1:9200/_template/snort_index -d @/var/lib/ambari-agent/tmp/snort_index.template'] {'logoutput': True}
{"acknowledged":true}2017-11-03 22:22:41,448 - Execute['curl -s -XPOST http://node1:9200/_template/yaf_index -d @/var/lib/ambari-agent/tmp/yaf_index.template'] {'logoutput': True}
{"acknowledged":true}2017-11-03 22:22:41,473 - Execute['curl -s -XPOST http://node1:9200/_template/error_index -d @/var/lib/ambari-agent/tmp/error_index.template'] {'logoutput': True}
{"acknowledged":true}2017-11-03 22:22:41,497 - Execute['curl -s -XPOST http://node1:9200/metaalert_index -d @/var/lib/ambari-agent/tmp/meta_index.mapping'] {'logoutput': True}
{"acknowledged":true}2017-11-03 22:22:41,832 - Setting Elasticsearch template installed to True
2017-11-03 22:22:41,832 - File['/usr/metron/0.4.2/config/zookeeper/../metron_elasticsearch_template_installed_flag_file'] {'owner': 'metron', 'content': 'This file created on: 2017-11-03 22:22:41', 'mode': 0755}
2017-11-03 22:22:41,833 - Writing File['/usr/metron/0.4.2/config/zookeeper/../metron_elasticsearch_template_installed_flag_file'] because it doesn't exist
2017-11-03 22:22:41,833 - Changing owner for /usr/metron/0.4.2/config/zookeeper/../metron_elasticsearch_template_installed_flag_file from 0 to metron
2017-11-03 22:22:41,833 - Changing permission for /usr/metron/0.4.2/config/zookeeper/../metron_elasticsearch_template_installed_flag_file from 644 to 755
2017-11-03 22:22:41,833 - Starting indexing

@anandsubbu Would you mind taking a look at this and double-checking the fix?

Hi @nickwallen , thank you for the fix. It is my bad, I should have added the line commands = IndexingCommands(params) before the try block in my previous fix :(.
I am under way validating and will post my observation. However, given that the indexing templates are a must have for the alerts UI to work properly, should we remove the try/expect block and allow the install to fail in the event it was not successfully installed? Or do you think we should WARN and move on?

However, given that the indexing templates are a must have for the alerts UI to work properly, should we remove the try/expect block and allow the install to fail in the event it was not successfully installed? Or do you think we should WARN and move on?

I think we should leave that as a separate discussion.  It's definitely worthy of a discussion, but I think we should just fix what we have in this PR.   Thanks for validating!

I'm +1 on the PR, but I strongly agree with Anand, this should not be a warn.  We should throw an exception here and fail the build.  That being said, despite it being in the vicinity of this PR, it's not the core value of the PR, so I've created METRON-1306 to track it.

+1.  I agree with Casey that this should fail instead of WARN.  If templates are not installed it puts us in a bad state and requires manual intervention that may not be obvious.

@anandsubbu agreed with your comments on the exception

@nickwallen , for the record full dev spun up fine with the fix. Thank you!


[Bug] deterministic substreams errors are not reported at the graph-node level

Bug report
A subgraph-powerered-subgraph that fails with a deterministic error is not properly handled by graph-node. It should detect the error, and report the subgraph as "failed" in a deterministic way. Instead, graph-node reports the subgraph to be healthy and continually retries the substream
graph-node: v0.32.0
firehose-ethereum: v1.4.10
Relevant log output
Nov 07 14:12:41.040 INFO Blockstreams connected, provider: substreams, deployment: QmT2nEwS9YWFzLE39Ujuv2oNVhgbctc8VUtB28RXb4wUbY, sgd: 351, subgraph_id: QmT2nEwS9YWFzLE39Ujuv2oNVhgbctc8VUtB28RXb4wUbY, component: SubstreamsBlockStream
Nov 07 14:12:41.043 INFO received err, provider: substreams, deployment: QmT2nEwS9YWFzLE39Ujuv2oNVhgbctc8VUtB28RXb4wUbY, sgd: 351, subgraph_id: QmT2nEwS9YWFzLE39Ujuv2oNVhgbctc8VUtB28RXb4wUbY, component: SubstreamsBlockStream
Nov 07 14:12:41.043 ERRO An error occurred while streaming blocks: status: Unknown, message: "rpc error: code = InvalidArgument desc = step new irr: handler step new: execute modules: applying executor results \"graph_out\": execute: maps wasm call: block 9974225: module \"graph_out\": wasm execution failed deterministically: panic in the wasm: \"called `Option::unwrap()` on a `None` value\" at /Users/mack/.cargo/registry/src/github.com-1ecc6299db9ec823/substreams-0.5.10/src/scalar.rs:404:25

IPFS hash
QmT2nEwS9YWFzLE39Ujuv2oNVhgbctc8VUtB28RXb4wUbY
Subgraph name or link to explorer
No response
Some information to help us out

[ ] Tick this box if this bug is caused by a regression found in the latest release.
[ ] Tick this box if this bug is specific to the hosted service.
[X] I have searched the issue tracker to make sure this issue is not a duplicate.

OS information
None

thanks @madumas!
@mangas checking if this is the currently expected behaviour, or if we have some error handling in place for substreams?

We currently don't have this information surfaced through the GRPC types. The current Error definition looks like this:
message Error {
  string module = 1;
  string reason = 2;
  repeated string logs = 3;
  // FailureLogsTruncated is a flag that tells you if you received all the logs or if they
  // were truncated because you logged too much (fixed limit currently is set to 128 KiB).
  bool logs_truncated = 4;
}

The correct way to handle this would be for a deterministic flag to be added so that the graph-node can treat it as such.

@mangas we don't need a flag: the condition is in the Error Code:
code = InvalidArgument
Invalid Argument is returned from Substreams when the error is "not due to the state of the system".
I'm not certain why you are getting "Status: Unknown" however...

The GRPC issue has been fixed in substreams, pulled to firehose-core
https://github.com/streamingfast/firehose-core/releases/tag/v0.2.4
Status field will show as InvalidArgument instead of Unknown.
Now, can graph-node be configured to consider "InvalidArgument" status code as a deterministic error ?

@sduchesneau is that the only error code which is indicative of a deterministic error?

todo: Investigate status


WIP: [CLOUD-1383] Add openshift.io/display-name and description annotation…

…s to all image stream tag definitions
Some files have multiple image Stream Tag definitions, I added the openshift/display-name var to the image stream tag definitions which refer to the application being built from the template. For example, for templates like eap64-amq-persistent-s2i.json, I added the openshift/display-name vars to the image stream tag definitions which were named ${APPLICATION_NAME}:latest but not to the ones named "jboss-eap64-openshift:1.4" or "jboss-amq-62:1.3". Is this the right way to do this?
*NOTE: My script cleaned up some typos and indentation(a couple of the datavirt files had duplicate env lists). I can put this in a seperate commit/PR if you think that is better.

Hey @kyguy, sorry it took me so long to get to this.  Wrt the extraneous changes, please remove them (or use OrderedDict in your script, assuming it's py).  They pollute the diff's with unrelated changes.
Regarding the display names, these should apply to the items in jboss-image-streams.json, not in the templates.

Since you're going to be in there, feel free to address CLOUD-1382 at the same time.

Put CLOUD-1382 and CLOUD-1383 changes together and updated the correct files. Let me know if the labels look appropriate

I put the indentation and spacing fixes in a separate commit as part of this MR, is that proper protocol? Should I put the format changes in a separate MR? I have some ideas about future format handling below, let me know what you think:

I have been using a Python script with the json library to make changes across the template files.
In order to keep the diffs clean I have been using the following arguments when parsing and writing to the template files.
data = OrderedDict() ... json.dump(data, f, indent=4, separators=(',', ': '))

The script corrects any inconsistencies among the files(indentations, spacing). It seems that my script occasionally comes across these inconsistencies and auto corrects them. For the sake of keeping the templates clean, I was thinking we declare a format standard for the template json files. For instance, we could declare that a proper indentation is 4 spaces and that there be a single space after separators(such as colons). What do you think?

If you think this is a good idea, I can add a cleaning script that people can run before they commit their changes. By doing this, we can reduce the formatting commits and keep the formatting of the files consistent.

I mimicked the format of the OpenShift example image-streams. Although the OpenShift example image-streams have the "openshift.io/display-name" label in both the annotations and tags, they don't seem to appear in the latest version of the OpenShift web console for the image-streams.  Maybe the "openshift.io/display-name" values (for the image-streams) are planned for a future release? Still investigating this though, I will let you know what I find

Not sure what you mean.  Looking at the Ruby image, the display-name is "Ruby" while the name is ruby.  We can't change the name of the image streams, as this is what is used to reference the image stream in the rest of OpenShift.  Looking at the console, it displays "Ruby" not "ruby."
It also appears that the description display is specific to the tag selected in the drop down.

Updated the with metadata annotations, so now the openshift.io/display-names are appearing in the catalogue, I also updated the openshift.io/display-names to match the descriptions better


Monitoring graphics (CPU, MEMORY, network, disk consumption, I/O, etc)

[x] Code
[ ] Doc
[ ] Tests

@tinova PRs to approve:

Code

I want to work on this issue. Getting some heads up/approval would be helpful. I am a Masters student trying to work on this open source project.


Custom hooks extension support

Hi everyone!
I was playing around with esbuild and encountered a slight problem with its Surface compatibility. I use Preact on my project to render some react components and I trigger that rendering inside my hooks. The problem is, esbuild requires .jsx extension to support JSX and Surface strictly looks for .hooks.js extension. I think adding a configuration option for hooks extension would be great to solve this case, just like you did for output dir.

PR Ready! https://github.com/surface-ui/surface/pull/483


Add API support for PSA vehicles

API documentation
https://developer.groupe-psa.io/webapi/b2c/overview/about/
API specifications
https://developer.groupe-psa.io/webapi/b2c/api-reference/specification/
Scope of this API
Supported vehicles by this API (100% electric):

DS 3 Crossback e-tense
Opel Corsa-e
Peugeot e-2008
Peugeot e-208

Supported vehicles by this API (Plug-in Hybrid):

DS 7 Crossback e-tense
Peugot 3008 Hybrid & Hybrid4
Peugot 508 Hybrid
Peugot 508 SW Hybrid
Opel Grandland X Hybrid4
Citroén C5 Aircross Hybrid4

Goal

PSA vehicles are configurable within evcc.yaml
PSA Vehicle SOC information could be fetched from PSA API and is usable within evcc

It would be nice to add this. However, I need live credentials for testing in order to implement this. Please get in contact<EMAIL_ADDRESS>if you're interested in helping.

It would be nice to add this. However, I need live credentials for testing in order to implement this. Please get in contact<EMAIL_ADDRESS>if you're interested in helping.

According to https://github.com/ioBroker/AdapterRequests/issues/194 there seems to be support via bluetooth-connected phones only which is not suitable. Closing as wontfix.

Need to reopen. There is https://developer.groupe-psa.com/inc/node/1896 which looks good. Would still need credentials for testing.
/cc @Alex-ander-s

Hallo andig,
ich könnte Unterstützung anbieten, habe eine Mail gesendet.
Grüße, Alexander

Ist leider nix angekommen. Benötigt werden User/Passwort sowie ClientID/Secret, siehe https://github.com/flobz/psa_car_controller.

Ping @alexan-d willst Du es nochmal versuchen (cpuidle@gmx.de)?

Hi andig, thx for your project. EVCC working great.
can you give some informations or template how to add a Peugeot208 or  another PSA car?
I'm using this on iobroker which working great just need username and password.
there is no server "flobz/psa_car_controller" needed
https://github.com/TA2k/ioBroker.psa
thx

You can check https://github.com/andig/evcc/blob/master/vehicle/psa.go#L50 For the required structure

Ich iobroker works without clientid/secret please open a new issue to investigate

ok, will do


Bonus guide: Fulcrum

What
Fulcrum instalation tutorial for Raspibolt users.
Why
Alternative to electrum because of superior performance.
How
Following guide stated in the text, reporting issues and adding neccesary stuff.
Needed

[x]  Add tor hidden service
[ ]  Review guide and formatting

Thanks so much for writing this guide. Fulcrum is an important guide to have as in some cases it seems the only indexer that works properly.
I wished we could have reviewed this PR earlier but we are all quite busy at the same time so it did not happen.. I'll try to give it a try in the coming couple of weeks! Hang in there! :)

Some suggestions for the guide.
Should we use "rpccookie" config? :
#rpccookie = /path/to/bitcoind/datadir/.cookie

make "fulcrum" user and add to "bitcoin" group

use SSL (port 50002) instead of 50001 (for more wallet compatibility)

Some suggestions for the guide.
* use "rpccookie" config :

#rpccookie = /path/to/bitcoind/datadir/.cookie
* make "fulcrum" user and add to "bitcoin" group

* use SSL (port 50002) instead of 50001 (for more wallet compatibility)

Good ideas! They all seem like very sensible suggestions to me :+1:
@DanielPX1, would you be keen to implement them? For the new user you could check how we usually do it by looking at the Electrs guide for example (or any other server program on the node as they all use a separate user).

Some suggestions for the guide.
* use "rpccookie" config :

#rpccookie = /path/to/bitcoind/datadir/.cookie
* make "fulcrum" user and add to "bitcoin" group

* use SSL (port 50002) instead of 50001 (for more wallet compatibility)

Good ideas! They all seem like very sensible suggestions to me 👍
@DanielPX1, would you be keen to implement them? For the new user you could check how we usually do it by looking at the Electrs guide for example (or any other server program on the node as they all use a separate user).

Sure! Great suggestions, I am not available until Monday, but when I am back for sure will work on that!

Some suggestions for the guide.
* use "rpccookie" config :

#rpccookie = /path/to/bitcoind/datadir/.cookie
* make "fulcrum" user and add to "bitcoin" group

* use SSL (port 50002) instead of 50001 (for more wallet compatibility)

Good ideas! They all seem like very sensible suggestions to me 👍
@DanielPX1, would you be keen to implement them? For the new user you could check how we usually do it by looking at the Electrs guide for example (or any other server program on the node as they all use a separate user).

@VajraOfIndra  @twofaktor
I have added the suggested fixes, tested on my device and no problems, would someone be able to check if all is ok?
All I am unsure of are permissions to fulcrum folder, which command would you suggest for making all files and subfiles executable. readable and writable? I personally used chmod 777 as I am not very familiar with them yet, but will change them as it is not secure to my understading

Some suggestions for the guide.
* use "rpccookie" config :

#rpccookie = /path/to/bitcoind/datadir/.cookie
* make "fulcrum" user and add to "bitcoin" group

* use SSL (port 50002) instead of 50001 (for more wallet compatibility)

Good ideas! They all seem like very sensible suggestions to me 👍
@DanielPX1, would you be keen to implement them? For the new user you could check how we usually do it by looking at the Electrs guide for example (or any other server program on the node as they all use a separate user).

@VajraOfIndra  @twofaktor
I have added the suggested fixes, tested on my device and no problems, would someone be able to check if all is ok?
All I am unsure of are permissions to fulcrum folder, which command would you suggest for making all files and subfiles executable,readable and writable? I personally used chmod 777 as I am not very familiar with them yet, but will change them as it is not secure to my understading

--> My proposals:
sudo chown -R fulcrum:fulcrum /data/fulcrum
&
Put only exact the necessary lines to include in bitcoin.conf for fulcrum work fine

I updated my suggestions!

I updated my suggestions @DanielPX1 !!!

Did not notice, will look into it!

txindex=1 is not necessary in bitcoin.conf, because fullcrum creates its own index database, anyway txindex=1 was already configured in Bitcoin Core section. I would deleted

is necessary to touch bitcoin.conf for fulcrum to work properly? what is de line/lines strictly necessaries?

Some suggestions for the guide.
* use "rpccookie" config :

#rpccookie = /path/to/bitcoind/datadir/.cookie
* make "fulcrum" user and add to "bitcoin" group

* use SSL (port 50002) instead of 50001 (for more wallet compatibility)

Good ideas! They all seem like very sensible suggestions to me 👍
@DanielPX1, would you be keen to implement them? For the new user you could check how we usually do it by looking at the Electrs guide for example (or any other server program on the node as they all use a separate user).

@VajraOfIndra  @twofaktor
I have added the suggested fixes, tested on my device and no problems, would someone be able to check if all is ok?
All I am unsure of are permissions to fulcrum folder, which command would you suggest for making all files and subfiles executable,readable and writable? I personally used chmod 777 as I am not very familiar with them yet, but will change them as it is not secure to my understading

--> My suggestions :
* `sudo chown -R fulcrum:fulcrum /data/fulcrum` instead of `sudo chown bitcoin:bitcoin /data/fulcrum/*`

* is necessary to touch bitcoin.conf for fulcrum to work properly? what is de line/lines strictly necessaries?

* is necessary this line?
  `sudo ufw allow 8332`
  never i has seen this line in Electrs section...

* delete line * Nginx of requirements --> we have `* Generate cert and key files for SSL` section

* in tor connect section: ` HiddenServicePort 50002 <IP_ADDRESS>:50002` instead of ` HiddenServicePort 50001 <IP_ADDRESS>:50001`

* this section is correct?? should be it 50002 port instead of 50001 to use SSL??
  `* Add following lines

### TOR
tor_hostname=xyz... .onion
tor_tcp_port=50001

$ sudo systemctl restart fulcrum.service
$ sudo ufw allow 50001 <--- why this line is necessary? previously we openned 50002 port to SSL connection... ---> * Allow port SSL: ` sudo ufw allow 50002`

* You should now be able to connect to your fulcrum server remotely via Tor using your hostname and port ### 50001` why???

Fixed permission command
To my understanding, rpcport and whitelist (used in electrum as well) should be specified but I might be wrong, deleted unnecessary lines and kept those so far. Deleted also rpcbind, apparently these are not neccesary, my fulcrum works without them.
It was not neccessary, thanks
Deleted
rest) Should be ok now?

I wrote directly in code this
For me, this lines are not necessary, rpcuser / rpcpassword never were mentioned in BTC core section and rpcport=8332 is by default integrated in Bitcoin code config

I wrote directly in code this
For me, this lines are not necessary, rpcuser / rpcpassword never were mentioned in BTC core section and rpcport=8332 is by default integrated in Bitcoin code config

Thanks for feedback, I will therefore remove them:) When comes to rpcuser/password, I was using it for programs outside raspibolt (Dojo etc.) and run into problems with cookie auth, so thought it might be nice to mention it. But no problem removing it as well

I wrote directly in code this
For me, this lines are not necessary, rpcuser / rpcpassword never were mentioned in BTC core section and rpcport=8332 is by default integrated in Bitcoin code config

Thanks for feedback, I will therefore remove them:) When comes to rpcuser/password, I was using it for programs outside raspibolt (Dojo etc.) and run into problems with cookie auth, so thought it might be nice to mention it. But no problem removing it as well

I know, but currently for Raspibolt none of the services needs rpcuser/rpcpassword yet

I wrote directly in code this
For me, this lines are not necessary, rpcuser / rpcpassword never were mentioned in BTC core section and rpcport=8332 is by default integrated in Bitcoin code config

Thanks for feedback, I will therefore remove them:) When comes to rpcuser/password, I was using it for programs outside raspibolt (Dojo etc.) and run into problems with cookie auth, so thought it might be nice to mention it. But no problem removing it as well

I know, but currently for Raspibolt none of the services needs rpcuser/rpcpassword yet

No prob, I understand, it is gone. Btw I figured out why i put $ sudo ufw allow 8332, without it being in place, samourai dojo cant receive blocks because of docker. I accidetanlly put it there thinking its for fulcrum

new suggestions in code comments! @DanielPX1 , revise please! thanks!

new suggestions in code comments! @DanielPX1 , revise please! thanks!

Hmmm I checked everything I could and do not see any of your comments, am I doing something wrong? Sorry I am still a noob when comes to this...

On top, in tab "Files changed" left to "Checks (0)" you can see the code and all my comments

Thanks for feedback! Will fix that

Alright @twofaktor @Stadicus, should be fixed

Hello guys, I'm not sure what's the correct configuration to API connect to BTC RPC Explorer of Fulcrum, I think that is necessary to change this lines of .env file conf of BTC RPC Explorer, because Fulcrum use directly SSL connection:
#BTCEXP_ADDRESS_API=(electrum|electrumx|blockchain.com|blockchair.com|blockcypher.com)
#BTCEXP_ELECTRUM_SERVERS=tls://electrumx.server.com:50002,tcp://<IP_ADDRESS>:50001,...
Do you know what's the correct configuration? @Stadicus ??
I think it is like that, but i'm not sure:
BTCEXP_ADDRESS_API=electrum
BTCEXP_ELECTRUM_SERVERS=tls://<IP_ADDRESS>:50002

Done (y)

Hello guys, I'm not sure what's the correct configuration to API connect to BTC RPC Explorer of Fulcrum, I think that is necessary to change this lines of .env file conf of BTC RPC Explorer, because Fulcrum uses directly SSL connection: #ssl = <IP_ADDRESS>:50002
#BTCEXP_ADDRESS_API=(electrum|electrumx|blockchain.com|blockchair.com|blockcypher.com) #BTCEXP_ELECTRUM_SERVERS=tls://electrumx.server.com:50002,tcp://<IP_ADDRESS>:50001,...
Do you know what's the correct configuration? @Stadicus ??
I think it is like that, but i'm not sure:
BTCEXP_ADDRESS_API=electrum BTCEXP_ELECTRUM_SERVERS=tls://<IP_ADDRESS>:50002

Good point. I'll check once Fulcrum has fully synced.
I had to stop the sync after 3.5 days, as I configured it for an 8GB Pi instead of the required 4GB Pi... During that time, Fulcrum synced up to 64%. Unfortunately, even with the 6 minutes allowed for shutdown, Fulcrum did not shut down properly on a system unit restart and corrupted the database. So I have to start synching from start.

Wow, the two settings for 4GB and 8GB really make a huge difference. On the 4GB optimizations, I'm already at 62% after 17 hours (compared to 64% with the wrong 8GB settings after 3.5 days)

Wow, the two settings for 4GB and 8GB really make a huge difference. On the 4GB optimizations, I'm already at 62% after 17 hours (compared to 64% with the wrong 8GB settings after 3.5 days)

Yeah its really a difference, I am syncing BTC core on my 4GB node, so tommorow should test out the guide myself as well

Unfortunately, my 4GB Pi crashed during the Fulcrum sync, and the database got corrupted again. I restarted the sync with all other services disabled, I hope it will go through this time.
The reason also might be that the Pi is only cooled passively. Not an issue, usually, but it gets pretty hot during the intense sync...

Unfortunately, my 4GB Pi crashed during the Fulcrum sync, and the database got corrupted again. I restarted the sync with all other services disabled, I hope it will go through this time.
The reason also might be that the Pi is only cooled passively. Not an issue, usually, but it gets pretty hot during the intense sync...

hmmm that is strange. I am going to try it out now, so we will see. My raspi is cooled actively

Unfortunately, my 4GB Pi crashed during the Fulcrum sync, and the database got corrupted again. I restarted the sync with all other services disabled, I hope it will go through this time.
The reason also might be that the Pi is only cooled passively. Not an issue, usually, but it gets pretty hot during the intense sync...

 When I was using swapfile - 10GB allocation was great to sync with no issues and then switch it back to default once done. However at 2GB,4GB swap I had corruption issues same as you.
I am now testing out the guide and syncing fulcrum. I allocated 10GB for zramswap. So we will see (I use rpi 4GB).
It is possible that different amount (lower) may be enough for 8GB RAM version. Not sure yet. Will give feedback soon, right now I am at 40%

The Pi was running very hot, around 80 degrees celsius. I added active cooling now and started syncing again, with the same settings.

Ok, active cooling did not help, the sync crashed at 57%. As zram simply creates a compressed volume within the regular RAM, its size can't be increased. As I'm wary to just up the regular swap file size, I'll test with the following conservative Fulcrum config first:
db_mem = 256.0
db_max_open_files = 20
fast-sync = 0
db_use_fsync = true

Ok, active cooling did not help, the sync crashed at 57%. As zram simply creates a compressed volume within the regular RAM, its size can't be increased. As I'm wary to just up the regular swap file size, I'll test with the following conservative Fulcrum config first:
db_mem = 256.0
db_max_open_files = 20
fast-sync = 0
db_use_fsync = true

If this works, I hope to be able to use fast-sync with a small value of 256 as mentioned in cculianu/Fulcrum#96 (comment).
Note, @openoms mentioned a few weeks ago that Fulcrum simply might not be ready for 4GB systems: rootzoll/raspiblitz#2924 (comment). I wonder if that's still the case, or if we actually can get it to work.

I have set zram to 10GB and right now I am 84% into sync with guide settings, wanna try it?

I have set zram-swap to 10GB

Interesting. I'm not sure what this actually does, though. Zram creates a swap partition within the RAM, compressing it to make it (virtually) bigger. If you only have 4GB RAM, it cannot possibly create a 10GB partition within that. Does it then automatically use the SSD swap partition as well? Or do I have a basic misunderstanding here?

My very conservative settings seem to work so far as well, but it's pretty slow. I'm at 64% (the previous breaking point) after two days, but with very few resources used:

@Stadicus me neither, It is just what I know from experimenting with swap. I have experienced crashes on 4GB system with lower amounts of it. However I noticed that with lower amounts of swap, I had 300mb of RAM available and kept decreasing until crash. Now with 10GB swap I had stable ~1GB free RAM everytime I checked my system. Does not zram-swap behave as a normal swap?
Swap is a space on a disk that is used when the amount of physical RAM memory is full. When a Linux system runs out of RAM, inactive pages are moved from the RAM to the swap space. Swap space can take the form of either a dedicated swap partition or a swap file - wikipedia
I have today successfully sinced fulcrum with this setting

We have a new tested and possibly a final version of the guide. Could someone proceed to review it? @VajraOfIndra @Stadicus @kristapsk??

The Fulcrum config specified is pretty volatile... :) that makes it hard to test. I'm currently syncing with the previous config, which uses

db_use_fsync = true --> now no longer explicitly enabled
db_max_open_files= 60 --> now 200

Did you test this specific configuration on a 4GB RPi, @DanielPX1?

The Fulcrum config specified is pretty volatile... :) that makes it hard to test. I'm currently syncing with the previous config, which uses

db_use_fsync = true --> now no longer enabled (default: false)
db_max_open_files= 60 --> now 200

Any special reason to disable fsync, @DanielPX1? It seems like a good option for low-resource systems using ext4 fs:
# RocksDB use "fsync" - 'db_use_fsync' - DEFAULT: false
#
# This boolean value, if true, tells the rocksdb library to use fsync() calls as
# opposed to fdatasync(). fsync() is slower but works around some potential
# issues in older Linux kernels and the ext4 filesystem.
#
# From the rocksdb documentation:
#
#  " By default, writes to stable storage use fdatasync (on platforms where this
#    function is available). If this option is true, fsync is used instead.
#
#    fsync and fdatasync are equally safe for our purposes and fdatasync is
#    faster, so it is rarely necessary to set this option. It is provided as a
#    workaround for kernel/filesystem bugs, such as one that affected datasync
#    with ext4 in kernel versions prior to 3.7. " - rocksdb/options.h
#
# db_use_fsync = false

https://github.com/cculianu/Fulcrum/blob/master/doc/fulcrum-example-config.conf#L666
Did you test this specific configuration on a 4GB RPi?

I caught Raspiblitz's references script to changes of values and other things, and the before comment "db_use_fsync = false" --> "so it is rarely necessary to set this option" made me believes that it wasn't necessary.
but anyway if you think it's necessary to keep both, let me know and I put it, no problem.

The Fulcrum config specified is pretty volatile... :) that makes it hard to test. I'm currently syncing with the previous config, which uses

db_use_fsync = true --> now no longer enabled (default: false)
db_max_open_files= 60 --> now 200

Any special reason to disable fsync, @DanielPX1? It seems like a good option for low-resource systems using ext4 fs:
# RocksDB use "fsync" - 'db_use_fsync' - DEFAULT: false
#
# This boolean value, if true, tells the rocksdb library to use fsync() calls as
# opposed to fdatasync(). fsync() is slower but works around some potential
# issues in older Linux kernels and the ext4 filesystem.
#
# From the rocksdb documentation:
#
#  " By default, writes to stable storage use fdatasync (on platforms where this
#    function is available). If this option is true, fsync is used instead.
#
#    fsync and fdatasync are equally safe for our purposes and fdatasync is
#    faster, so it is rarely necessary to set this option. It is provided as a
#    workaround for kernel/filesystem bugs, such as one that affected datasync
#    with ext4 in kernel versions prior to 3.7. " - rocksdb/options.h
#
# db_use_fsync = false

https://github.com/cculianu/Fulcrum/blob/master/doc/fulcrum-example-config.conf#L666
Did you test this specific configuration on a 4GB RPi?

I caught a new Raspiblitz's references script to changes of values and other things, and the before comment "db_use_fsync = false" --> "so it is rarely necessary to set this option" made me believes that it wasn't necessary.
But anyway if you think it's necessary to keep both at the above values, let me know and we put it, no problem.

tbh I am not sure how big of difference it makes... Worked for me being off every time so far and as @twofaktor stated, it appeared unneccesary to have it turned on

Unfortunately, my 4GB Pi crashed again at 65%, this seems to be the exact point-in-time when the sync becomes too much for it to handle...
I don't want to be the blocker here, and maybe it's just my Pi. But I'm personally not convinced that the current 4GB config will work well in the wild.

Unfortunately, my 4GB Pi crashed again at 65%, this seems to be the exact point-in-time when the sync becomes too much for it to handle...
I don't want to be the blocker here, and maybe it's just my Pi. But I'm personally not convinced that the current 4GB config will work well in the wild.

@Stadicus can you please try the setting with 10GB zram and let me know? I think it should not make these issues for you raspi

try the setting with 10GB zram and let me know?

I tried that before, to no avail. Until today, I was not able to sync Fulcrum on my 4GB Pi, it crashed at 85% the latest. But That does not need to be a blocker if it works for your and there ain't any issues with 8GB Pis.

If you see issues popping up with 4GB Pis going forward, maybe we can add a disclaimer/warning at the beginning.

Did someone try this section?
https://raspibolt.org/guide/bonus/bitcoin/fulcrum.html#enable-fulcrum-api-connection-to-btc-rpc-explorer


Rename-Many: allow the user to escape the pattern with a leading back tick

Currently, the user can escape a pattern by using the esc, function. But this is quite clumsy to use because you can't just invoke the function, you have to warp it in brackets before doing so, otherwise powershell will take esc as though it were part of the pattern. So instead of this:

... -Pattern $(esc('blah'))

the user should now be able to do this:

... -Pattern '`blah'

Can't use a back tick because that has a special meaning inside powershell string, so use a tilde instead.

Can't use a back tick because that has a special meaning inside powershell string, so use a tilde instead.


Doubled inherited icons of items inside a folder

Describe the bug
Doubled inherited icons of items inside a folder
To Reproduce
Steps to reproduce the behavior:

Create a folder
Create notes inside that folder
Set an icon for the folder
Inherit this icon to the notes
Change the icon of the folder
Inherited icons of the notes aren't updated + "Inherit" option becomes available again
Inherit icons again, you will get double icons

NOTE: I'm not able to reproduce it although I'm 100% sure that's how it happened
Expected behavior
To update the inherited icon when the parent folder icon is updated
To remove the inherited icon when the parent folder icon is removed
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Also happened to me. Set the icon of a subfolder, than set the inherit icon of the higher folder. I am not able to remove all the icons after it happened:

I also have the same issue, help fix it.

Could you please provide some extra detailed steps on how I can reproduce this? Because at the moment I cannot reproduce this issue.

I do following this:

Create a folder/note inside the folder that already set the Inherit Icon
After you create, reload the window again by using cmd+P.
After the Obsidian window re-open, you will see the icon is duplicate in the folder that inside the Inherit Icon folder

This should be fixed in 2.2.0

@FlorianWoelki  Thanks for the update but after I update to version 2.2.1 still have the duplicate problem as the video here.
https://github.com/FlorianWoelki/obsidian-icon-folder/assets/38547838/76d86473-76eb-4226-90cc-24ae87fad81a

I've pushed another fix. This will be probably released within the next days.
In theory, directories are not supported anymore to have inherited icons. There was a smaller issue with renaming/creating directories.

@FlorianWoelki
Thanks man, looking forward to it. Cheer!

@FlorianWoelki
Is another fix have been publish and ready for update on Obsidian?

@FlorianWoelki Is another fix have been publish and ready for update on Obsidian?

Nope, it will be released within this week.


BUG: Setting export.$cancel = true of one test cancels all subsequent tests

Setting the exports.$cancel variable works fine in the VSCode, but when I execute the same tests from CLI, as soon as the first test in the sequence encounters the exports.$cancel=true condition, all subsequent test will be skipped as well (even the ones with no dependencies).
I created a demo here:
https://github.com/alekdavis/httpyac-extras/blob/main/Tests/Bugs/cli_cancel_bug.http
If you run all test in the file from VSCode, it will execute tests A, C and D, but if you run all tests via CLI, it will only execute test A. I used these command-line options:
httpyac send -a -o short Tests\Bugs\cli_cancel_bug.http

And my output is this:
---------------------

=== Test A ===

GET https://httpbin.org/json
=> 200 (368 ms, 612 B)

1 requests processed (1 succeeded, 0 failed)

when I call up the file, it actually looks correct at first.

Which CLI version are you currently using? We could try using a docker run command to test my app without your environment:
docker run -it -v ${PWD}:/data ghcr.io/anweber/httpyac:latest send -a -o short ./Tests/Bugs/cli_cancel_bug.http

Ah, cool. One or more of the versions (npm, node, or httpyac) must have been out of date. I should've recorded, but I'm not a user of node or npm outside of httpyac, so I just learned about the commands to check versions and run an update. I updated all three and now it looks good. Thank you.
While we're on this topic, if I update the VSCode extension, would it automatically update the CLI package? Or are they totally independent and I need to make sure to update them individually?

The versions are independent. I have noticed that someone has set up httpyac in homebrew so that the updates are automatically managed with homebrew.

I see, thanks. I assumed that the CLI was a dependency of the VSCode extension and expected a VSCode update to also update the CLI version. We do not use Homebrew (it looks like a MacOS/Linux thing and we're mostly a MSFT shop), but I will keep in mind that we need to monitor the CLI release announcements and make sure we update the CLI periodically.


Prepare release of SDK 1.8.3

The changes to the "LotResult" section in view-templates/28.efx were reverted in commit ac1df69.


Enzyme pullback for array output

Possible with https://enzymead.github.io/Enzyme.jl/stable/pullbacks/

Done


Update documentation

This pull request updates the documentation with an example of how to use the new deepQuery method.

coverage: 100.0%. remained the same
when pulling c341367b17862b253c3f88383d04afef8668e111 on update_documentation
into efd2c9bb0ee22b422f30da8e3082e05604a4d25d on master.


Add Laravel Kit

Laravel Kit

https://github.com/tarequemdhanif/laravel-kit

Not enough stars: https://github.com/chiraggude/awesome-laravel/blob/master/CONTRIBUTING.md


Adds documentation, updates to 2018 edition for error handling, updates dependency versions

Hello,
This pull request makes the following changes:

Adds LICENSE file to the main repository
Adds documentation, including summary of crate for doc.rs
Updates the syntax to Rust 2018 edition by replacing use of try!() macro with ? operation
Updates dependencies to most recent releases of byteorder and i2cdev

Adds native functions for dealing with traits deprecated by i2cdev

Moves main.rs to examples/ directory as simple.rs and loops over call
Fixes clippy suggested lints
Updates CI from TravisCI (which now has limited open source build minutes) to Github Actions

I've tested this implementation as building on my x86 machine using rustc 1.52.0-nightly and on an Odroid-C4 ARM64 board attached to an Adafruit BMP280 module over the /dev/i2c-1 connection. Please let me know if you have any questions or concerns!

@natemara Happy to help :) Please take a look at the comment regarding the API change for String to PathBuf when you have a moment-- I just want to make sure we're on the same page.

@quietlychris thank you for your contributions, version 0.4.0 has been released!


Fails assistant when it can't determine the task type

When the assistant can't create a task for a given family, it should not just
die. This catches the errors caused by unknown task families and logs messaeges
rather than just dying. The scheduler is informed of the failure and the
assistant can continue requesting other jobs.
As a future improvement, the scheduler could also use the failure report to
avoid giving the assistant these types of jobs.

LGTM!

LGTM


Leaking Implementation in Public API

A few APIs are leaking implementation classes into public API.
https://github.com/Azure/azure-sdk-for-java/blob/master/sdk/search/azure-search-documents/src/main/java/com/azure/search/documents/util/SearchPagedResponse.java#L43
https://github.com/Azure/azure-sdk-for-java/blob/master/sdk/search/azure-search-documents/src/main/java/com/azure/search/documents/util/SuggestPagedResponse.java#L43

How did this get past the checkstyle checks?

How did this get past the checkstyle checks?

It appears that the custom rule doesn't extract generic types, since these leaks are SimpleResponse<T> they bypass the check.

Good catch - we need to update our checkstyle rule to catch this! And Sima, this needs to be resolved prior to shipping GA.

Thanks for catching this! Does this count as a breaking change? I have to write an email for approval, right?

yes, it is a breaking change to API.

Fixed in #12504


Implementing default locale

I've followed the guidelines for setting up a locale depending on a routing parameter.
The problem I have that if a locale is given in the url, I have to inject it inside every route. This has to be done after the LocalizationMiddleware. Since the config and routes are already loaded I found no way to set the current locale as the default locale for every route.
However by a simple hack this can be done automatically:
<?php

namespace App\Core\Router;

class FastRouteRouter extends \Zend\Expressive\Router\FastRouteRouter
{
    public function generateUri($name, array $substitutions = [])
    {
        if (!array_key_exists('locale', $substitutions)) {
            $substitutions['locale'] = substr(\Locale::getDefault(), 0, 2);
        }

        return parent::generateUri($name, $substitutions);
    }
}

Would this be useful to implement this directly or add it to the cookbook?

They seem to want all the locale stuff in the cookbook. If they ever merge this pull request, you can take out the substr() as the locale logic will account for language-country locales. (But not script - sorry Serbians!)

@Ocramius I'm setting it with middleware first:
class LocalizationMiddleware
{
    public function __invoke(ServerRequestInterface $request, ResponseInterface $response, callable $next = null)
    {
        $locale = $request->getAttribute(
            'locale',
            Locale::acceptFromHttp($request->getServerParams()['HTTP_ACCEPT_LANGUAGE'])
        );
        Locale::setDefault($locale);

        if ($request->getUri()->getPath() === '/') {
            return new RedirectResponse('/' . substr(\Locale::getDefault(), 0, 2));
        }

        return $next($request, $response);
    }
}

Thanx for the advice on the delegator factory. I'll try that.

Locale::setDefault($locale);

Argh, we should really kill that stuff :-\

As discussed on IRC, it's better to inject the locale as request attribute and not rely on a static global.

@xtreamwayz
could you document that in the cookbook? Or is that already included in the recipies?

I'm figuring out a solution right now. The cookbook sets the static default locale and hasn't any locale detection.
What I have so far is something ugly but it's working: https://gist.github.com/xtreamwayz/1df70d2c072021633405a31e48f9138c

It detects the locale if it's missing.
Uses the request attribute to store it.
Reuses the locale from the route result in the UrlHelper (the ugly hack).

I still need to hook it up with a translation lib and twig. Once I have it completely working I'll update the cookbook.

I've got translations working with Twig and Symfony Translator. It should be almost the same for the Zend translator since they have almost the same api.
I've updated my gist: https://gist.github.com/xtreamwayz/1df70d2c072021633405a31e48f9138c
If you guys want to review it before I transfer it to the cookbook.
/cc @Ocramius @RalfEggert

This is merged with #351.


Web view crash with profile.spall

Thanks for making this tool!
I had a go using spall_native_auto.h but the file it generates crashes the web view. Seems to happen even when I don't initialise any threads for profiling using spall_auto_thread_init().
I've attached the file. Let me know if you'd like anything more to reproduce it.

profile.zip

Oops, my error message there needs a little work!
The native auto header only works with the native version of Spall, it's a different file format, and some of the things it does to compress the file like parse the traced executable and debug info for symbols, can't be done in-browser.
If you'd like to try auto-tracing, there is another version of the auto-tracer that'll work for the web client here: https://github.com/colrdavidson/spall-web/tree/master/examples/auto_tracing
If your trace is too big, and you'd prefer to mess with the native version of auto-tracing instead, that's on itch.io here: https://gravitymoth.itch.io/spall

Ah, thanks for the clarification. I'll take a look at the other options.


Added logs intake HTTP endpoint

What does this PR do?
Added the default prefix for logs HTTP endpoint
Motivation
Release HTTP
Additional Notes
Anything else we should know when reviewing?

already merged in 6.13


Add: New Resource

It is not a feature!!
I just wanted to add a resource and let me know if it is under the guidelines of this repository.
Visit the link here for curated list of GIFs & Avatars for GitHub.
I didn't want to make a PR without confirming.
@rzashakeri
Please assign me this issue if I can add this gif repo link.

It is not a feature!!
I just wanted to add a resource and let me know if it is under the guidelines of this repository.
Visit the link here for curated list of GIFs & Avatars for GitHub.
I didn't want to make a PR without confirming.
@rzashakeri
Please assign me this issue if I can add this gif repo link.

Hello, it is a very good repository, you can add at least two of the gifs that you think are the best as examples, because if you want to add them all, the page will be too heavy. 💙

Okay, I will make the changes.
Of course, I was not going to add all the GIFs.

Okay, I will make the changes.
Of course, I was not going to add all the GIFs.

Thanks 💙🙏


fix: make sure that the __import_unsupported patch is applied in Cloudflare as well

this is needed to resolve https://github.com/opennextjs/opennextjs-cloudflare/issues/177

@dario-piotrowicz would be nice to link to the issue on the cloudflare adapter in the PR description.
Thanks!

LGTM Thanks. Just resolve this https://github.com/opennextjs/opennextjs-aws/actions/runs/12351796922/job/34467482276?pr=678

done 🙂👍


Update reference for Style/FloatDivision

Follow https://github.com/rubocop-hq/rubocop/pull/9186#issuecomment-740405392.
This PR updates the reference for Style/FloatDivision.
The new link is a comprehensive reference that includes the old link address, so this PR is replacing the URL rather than adding new link.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[ ] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[ ] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Thanks!


itemstack issue

Everytime i click on the item or whenever the pagination gets updated the "test" lore keeps getting added in a loop that doenst stop.
Also the itemstack it returns is the one that gets modified and not the value "listing.item" by itself.  This never happend with the old menu api i used
private fun calculateAndUpdateEntries() {
pagination.items.clear()
    for (listing in listings) {
        pagination.addItem(Icon(listing.item).appendLore("test").onClick {
            if (listing.createdBy == player.uniqueId) {
                //
            } else {
                //
            }
        })
    }

    pagination.update()
}

Please describe the issue cleanly. What are reproduce steps, outputs, and expected outputs? Never assume that people know what you know. Fix format of the code block. In a nutshell, be attentive if you're asking for help.

Nmv it was my mistake i was retrieving the itemstack without cloning it first


How to pass a 256bit salt?

Thanks for your work. I want to pass a 256bit salt but the doc shows a string salt example. So can i pass a 256bit salt to this?
I tried to pass a buffer and it gave this error:
[[31,20,20,20],[0,0,1,0],[[221,100,1629384601603,false],[221],[223,30,1629384601610,false]],1271]]
ERROR  [Error: Exception in HostFunction: Malformed calls from JS: field sizes are different.

I realise I need to convert a key to utf-8!

Yes, you'll have to implement it both for Android and iOS.
If you do implement it, open to accepting PRs to merge the ability to use bytes instead of string


Basics for import from git UI

Ready for review.

:+1:
We'll probably need to tweak the api a bit to make the variable names the same e.g id vs locale
but that can happen in a later PR

@miltontony thanks for the review :)

We'll probably need to tweak the api a bit to make the variable names the same e.g id vs locale

Yep, have a plan for that, maybe best explained IRL.


Pagetop menus not working for new page

I'm submitting a ...
[ x ] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Do you want to request a feature or report a bug?
Bug

What is the current behavior?
ba-Pagetop menus are not getting displayed on new pages.

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via
https://plnkr.co or similar (you can use this template as a starting point: http://plnkr.co/edit/tpl:AvJOMERrnz94ekVua0u5).

What is the expected behavior?
ba-PageTop should display users, notification and message menu on new pages, please see details below.

What is the motivation / use case for changing the behavior?

Please tell us about your environment:

Angular version: 2..2.3
Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

Hello All,
I have created a new page based on the documentation and everything seems working fine except the menu on top.
for example when I navigate to my new page and I click on "Message" "Notification" or "User Area", the drop down menu class will not show up.
When I navigate to any of the pre existing page the Pagetop menu also stops working.
I have to reload the page to the dashboard view in order for the menus to reappear but again, once I navigate to the new page, the menu stops working again.
Interestingly i'm able to collapse the sidebar with no issues from the new page, only the user, notifications and messages are not displaying.
I can't seem to find any problem in the console window and there are no errors reported.
This happen on IE, Edge, Chrome and Opera.
I have tried multiple times, recreated the page, downloaded the repo from scratch but the behavior remain.
does anyone faced this issue or know how to trace it further?
Thanks a lot.

I have narrowed it down further to be caused by importing any module from the 'PrimeNG' Library,
for example adding the 'SplitButtonModule' will immediately trigger this behavior.
please note that I didn't require any PrimeNG CSS yet on my module, but certainly PrimeNG modules have CSS when importing their module.
did anyone successfully integrated PrimeNG with the project or know a quick workaround?
Thanks.

I had exactly the same issue. Any of the primeng module load will somehow trigger bootstrap dropdown toggle twice (one from vendor and one from the chunk). It removing line entry 324 "Dropdown" from the webpack.common.js appears to be working ok.. (temporary fix). Hope someone can explain better why this is the case and what the correct fix for this is. Thanks

@martinkoo1 Thanks! Commenting out the loader for the bootstrap dropdown in webpack.common.js also resolved the issue for me. My guess is also that we're ending up with multiple bootstrap instances conflicting silently in some webpack chunks between primeng & ng2-bootstrap imports.
Collapse: "exports-loader?Collapse!bootstrap/js/dist/collapse",
// Dropdown: "exports-loader?Dropdown!bootstrap/js/dist/dropdown",
Modal: "exports-loader?Modal!bootstrap/js/dist/modal",

Interestingly enough, the bootstrap buttons on the bundled demo page are all unaffected. It was just in the baPageTop I was seeing the profile/settings/logout menu not appearing. Also note that with this "fix", the bootstrap button page continues to function properly at a glance.

I'm running into this issue now unfortunately... I cannot use PrimeNg's auto-complete feature because of the interference with Ng2 in some capacity. But I also cannot track down the correct line to comment out, now that Webpack.common.js isn't readily available in the current version.


How save client?

I'm making an Android app!
Everything works, but only when the application closes and reopens Client becomes null.
But this is understandable, as I understand it, in order to save the session, you need to save the client in a third-party database (file).
But how to do it if the Client object is not serialized

The app isn't supposed to save anything between restarts. All needed data is saved by TDLib and the app just needs to provide the same database and files directory in setTdlibParameters to the newly created Client after restart. See https://core.telegram.org/tdlib/getting-started for more details.

Thahk you


some hlinks to a diagram within description raise Malformed link exception

In IFE model (https://github.com/DSD-DBS/Capella-IFE-sample) there are hlinks within descriptions that lead to diagrams. Some of those are simply hlink://_.... but some have a trailing "/". Those with trailing slash trigger the below exception:
File "templates/common_macros.html.j2", line 189, in template
    <p>{{ obj.description | safe }}</p>
    ^^^^^^^^^^^^^^^^^^^^^^^^^
  File ".../capella_model_explorer/backend/explorer.py", line 106, in __finalize
    return capellambse.helpers.replace_hlinks(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "...python3.11/site-packages/capellambse/helpers.py", line 571, in replace_hlinks
    return process_html_fragments(markup, cb)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "...python3.11/site-packages/capellambse/helpers.py", line 616, in process_html_fragments
    node_callback(node)
  File "...python3.11/site-packages/capellambse/helpers.py", line 558, in cb
    obj = model.by_uuid(target)
          ^^^^^^^^^^^^^^^^^^^^^
  File "...python3.11/site-packages/capellambse/model/_model.py", line 379, in by_uuid
    return _obj.ModelElement.from_model(self, self._loader[uuid])
                                              ~~~~~~~~~~~~^^^^^^
  File "...python3.11/site-packages/capellambse/loader/core.py", line 1227, in __getitem__
    return self.follow_link(None, key)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "...python3.11/site-packages/capellambse/loader/core.py", line 1131, in follow_link
    raise ValueError(f"Malformed link: {link!r}")
ValueError: Malformed link: '_xXSwEKv6EeSTJvLsyJitsw/'

This was fixed in #472 already, I just hadn't made a new release with that patch yet. I've fixed that now, v0.6.10 is out 🙂


fix-spec,Source: default to Source0 and Source

Fixes #536
With this change, we default to Source0 and Source, which should improve UX for newcommers if they use Source in their spec files.
Everyone is still able to set their Source\d line their config.

Are you okay with this change? Is the benefit worth it?

Are you okay with this change?

Not sure I understand your question. If I wasn't I'd not approved it.

Is the benefit worth it?

wut?

rpm build problem:
DEBUG util.py:593:  Error:
DEBUG util.py:593:   Problem: conflicting requests
DEBUG util.py:593:    - nothing provides python3.8dist(marshmallow) < 3 needed by rebase-helper-0.18.0-1.fc32.noarch

Are you okay with this change?

Not sure I understand your question. If I wasn't I'd not approved it.

Is the benefit worth it?

wut?

The point I'm trying to make is that we are adding more complexity to the code for sake of (small or not small?) UX improvement. Anyone can easily set spec_source_id: Source in their packit config and be sure all's good.
But, UX > our maintainer sanity, so... Let's keep this.