Add hardware embodied impacts

TL;DR: Currently CodeCarbon is focused on usage impacts related to energy consumption of the host. To assess impacts on a broader scope and in a multisteps way, as defined in Life Cycle Assessment studies, we need to add embodied impacts of the underlying hardware.
Description
When assessing the impacts of hardware (and thus software running on it), it is common sense to look at Life Cycle Assessment (LCA) methodologies. LCA is a multisteps and multicriteria methodology to compute environmental impacts.
Multicriteria includes impacts not only in terms of carbon emissions or energy consumption but abiotic resource depletion, water usage, biodiversity loss and so on.
Multisteps is about taking the whole life cycle into account, from resource extraction, manufacturing, transportation, usage and finally end-of-life. This is what this issue is about. We call embodied impacts what consists of the impacts before the usage part. For instance, the process of making a GPU emits greenhouse gases, and thus when you receive brand new hardware, its emissions are not zero and must be taken into account.
Why does it matter?
Embodied impacts account for hardware impacts and thus can help better answer the following types of questions:

What type of hardware should I use to run my models? CPU, GPU (or something else)?
Should I upgrade to next-generation hardware that is more power efficient?
How long should I keep a server? What "savings" in terms of impacts I am making by extending the lifetime of one or more years?

Introducing multisteps environmental impact assessment will make CodeCarbon even more precise regarding the impacts of software and AI.
Implementation
I propose to use the open source methodology and API we develop at Boavizta (project named BoaviztAPI). The API provides proxy functions to assess embodied impacts given a hardware specifications. So, for instance, given the CPU name, the quantity of RAM, the model of GPU, etc. we can compute embodied impacts.
BoaviztAPI resources:

Repository
Documentation
Swagger to test in your web browser
Datavizta educative frontend of the API.

The hosted API endpoint https://api.boavizta.org/ is today solely for testing and not really to be used in production or for heavy usage.
Implementation using static data
One proposition was to implement embodied impacts based on BoaviztAPI using static CSV or JSON files containing typical hardware configurations and associated impacts. This solution lacks the ability to automatically benefit from next update of BoaviztAPI, but can be considered to make a POC of the feature. Or it can also be considered as a backup solution.
Implementation using a dedicated API endpoint
The official endpoint is not designed to handle heavy traffic or production use cases (no SLA or uptime guarantee). We can mitigate that by either A) hosting an API endpoint dedicated to CodeCarbon users or B) asking the end user to host it on its infrastructure to use that feature.
While A) can be discussed, I found B) less relevant, as the feature will likely be used by very few people if done that way.
In addition, A) (and probably B, depending on the use case), lacks the possibility to run CodeCarbon without an Internet connection, and this can be an issue in some cases.
Implementation using the python SDK
Finally, we can consider using the official BoaviztAPI python SDK in offline mode, which wraps the API routes in python functions that can work just like the official and online version. This is possible because BoaviztAPI has been developed since the beginning to be a stateless API.
Thus, this solution needs testing on our side (Boavizta) to make sure it's reliable and can be used in all environments where CodeCarbon is already in use. Still, this feature can be considered in the beginning as an extra requirement so that it does not block the installation of CodeCarbon on incompatible systems/environments at the beginning. And then, when we are confident enough to push that features for all users.
Caveats
For this implementation to make sense in the context of CodeCarbon, we still need to finish up the modeling and implementation of GPUs inside BoaviztAPI, which is still a work in progress. (see related issue #65)
This issue is primarily focused on opening the discussion first and then probably support the first implementation of the feature. Happy to read your comments and suggestions! 🤗

Thanks !
A first implementation that could be done is to add this to the metadata area of the dashboard as it has the CPU and GPU model:


Add flashloan capability to the swap pool contract

TODO: Create a modifier for swap, add liquidity, remove liquidity so that the pool cannot be used during flashloan.


Add phasor_at_harmonic function

Description
This PR adds a convenience function, phasor_at_harmonic, which calculates, given the real component of phasor coordinates of a single exponential lifetime at a certain harmonic, the phasor coordinates at other harmonics. It is implemented as a numpy ufunc in Cython.
Release note
Summarize the changes in the code block below to be included in the
release notes:
Add phasor_at_harmonic function

Checklist

[x] The pull request title, summary, and description are concise.
[x] Related issues are linked in the description.
[x] New dependencies are explained.
[x] The source code and documentation can be distributed under the MIT license.
[x] The source code adheres to code standards.
[x] New classes, functions, and features are thoroughly tested.
[x] New, user-facing classes, functions, and features are documented.
[ ] New features are covered in tutorials.
[x] No files other than source code, documentation, and project settings are added to the repository.

Hi @bruno-pannunzio . This implements the phasor_at_harmonic convenience function we discussed by email.

Thanks for testing. I think you are right, there should be a sentence or two describing in little more detail.
What about the function name, phasor_at_harmonic? I couldn't come up with anything better...

I thought about the name of the function but also couldn't think of a better one without it being too long. I think we can use this and maybe add a check to see if the phasor is located at the semicircle and if not raise an error?

maybe add a check to see if the phasor is located at the semicircle and if not raise an error?

Note that the function only takes the real part of the phasor coordinates as an input. The imaginary part is implied, assumed to be on the semicircle.


Allow customizing icons for mini view

If one is not using a nerdfont, it's possible to customize the icons for the cmdline. However, this is not possible for the mini view, so, e.g., Errors  are prefixed with a "broken font" symbol.
An easier (and possibly preferable) approach would be to allow disabling icons altogether (icons = false).

I don't think I added any special icons to error messages etc. Are these messages coming from vim.notify?

Oh, neremind. I do add icons :)

Note to self: don't set icons = false; this will freeze Neovim hard ...

Hmm, this doesn't seem to work?

Hmm, I can't seem to get this to work. Would you mind telling me the exact line I need to put in the setup call?

Works, thank you!


Trigger configuration in Azure, certificate binding

I have my trigger working locally but not clear on how to configure the certificate binding to run in Azure. Others have pointed out the problem with hard-coding. I'm not sure how to hard-code these values. The answer to this is obvious to everyone who's deployed a kafka trigger, right??
I'm getting the error:
Confluent.Kafka: ssl.certificate.location failed: error:140DC002:SSL routines: SSL_CTX_use_certificate_chain_file:system lib.
Function code:
public static async Task Run( [KafkaTrigger ...
SslCaLocation = @"C:\Users... CaRoot.pem",
SslCertificateLocation = @"C:\Users\ PrivateCert.pem",
SslKeyLocation = @"C:\Users\ us_PrivateCert.pem ...)]
Azure TLS/SSL Settings:
Public key certificate: location = CurrentUserMy
Private key certificate:  hostname = us_PrivateCert.net
No TLS/SSL bindings set.
What do I use for these location values to run in Azure?
I have added the config setting WEBSITE_LOAD_CERTIFICATES = *

I got the answer by deploying a trigger that would output the trigger's context. Answer: C:\home\site\wwwroot\

See my previous comment


Install error, contents of /usr/local/bin/fn are "Not Found"

Running cat /usr/local/bin/fn results in Not Found.
This is after running the installer, which of course errors (with the above output) because it tries to execute fn.
WSL2
Distributor ID: Ubuntu
Description:    Ubuntu 22.04.5 LTS
Release:        22.04
Codename:       jammy

Hi, same here seems we can't run this cli on linux? I guess it's because of the last release version.
curl -LSs https://raw.githubusercontent.com/fnproject/cli/master/install | sh

Same issue here. Latest version doesn't have the binaries available for linux versions.

Copying to /usr/local/bin would need sudo permission. Are you running the command as root? If not, try out.
Otherwise, you can also follow the steps mentioned in: https://fnproject.io/tutorials/install/#DownloadFnCLI
In short, here are the steps:

Download fn binary for your OS from [https://github.com/fnproject/cli/releases/] using a program like wget
Move it to a folder which is already in your PATH, or include the folder in PATH
Rename the binary to fn


Could you update to latest mod_wsgi version?

[X] I read the conda-forge documentation and could not find the solution for my problem there.

Issue:
Could you please update this to the latest version of mod_wsgi (4.9.0)?

@jonathanmorgan not seen the bot has found problems, will have a look.


Add documentation for MomentDateAdapter usage

Bug, feature request, or proposal:
Feature Request
What is the expected behavior?
There should be at least a small guide (preferably official docs) about how to use the newly (in beta 11) introduced MomentDateAdapter for the datepicker.
What is the current behavior?
No docs at all.
What are the steps to reproduce?

Check release notes of beta 11 and be happy about these words:

datepicker: Add Moment.js adapter (#6860) (9545427)

Go to official docs, and find nothing about how to introduce the moment adapter.

What is the use-case or motivation for changing an existing behavior?
Every feature should be documented.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Beta 11
Is there anything else we should know?
NO

@emreavsar You just need extend "DateAdapter" class
date-adapter.ts

To use the adapter, you need to import MdMomentDateModule into your app module and you are good to go. You can use moment.Moment objects as ngModel for the date picker.
The problem that @angular/material-moment-adapter module has not been published to https://npmjs.com yet. I worked around that by ripping out the sources from https://github.com/angular/material2/tree/master/src/material-moment-adapter/adapter and pasting them into a folder in my app's source tree. Works well enough for now, but I'd love to switch to a published version.

@rkrzewski You can use the github builds: https://github.com/angular/material2-moment-adapter-builds. Just use npm install https://github.com/angular/material2-moment-adapter-builds --save

That worked like a charm @Yeradon, thanks!

Any example for using this?

+1 an example please

@yonatancohen @kwalski https://stackblitz.com/edit/angular-material-moment-adapter-example

The last 2 examples here are for moment adapter: https://material.angular.io/components/datepicker/examples

@mmalerba  is there a way to change the output from iso fromat to some custom format. I dont want datepicker to work with time zones as the service crops the time part and hence when that time is given back to server we get wrong dates based on timezones

UPDATE: It is available on npm use following command to install it:

npm i @angular/material-moment-adapter

@ankitraonka
i have the same issue , do you found any solution for this ?

@Moutaz-homsi The best solution would be to use ControlValueAccessor and make a custom input component, this will have you in control of input and output of the component.

@ankitraonka, that's solve my problem for this issue , Have a great day and thanks a lot

Full process for using Moment objects instead of Date objects.

npm i @angular/material-moment-adapter --save
npm i moment --save
Provide in your ngModule. import { MatMomentDateModule } from '@angular/material-moment-adapter';.
Your Datepickers will now automatically emit Moment objects

@Moutaz-homsi
https://stackoverflow.com/questions/47262449/how-to-change-output-format-of-material-2-date-picker

Thanks a lot @ankitraonka , this solved my issue , i repreduced the code here if anyone have the same problem : https://stackblitz.com/edit/angular-dlxnmx?file=app%2Fcva-date.component.ts

The DatePicker seems to be using Moment as dates just fine. But I have a model that has a Date as a property. I changed the type from Date to Moment but when my service retrieves data from the server the property of the model is just a string:
"2018-03-31T00:00:00"
Why is this not a Moment object? Am I missing something?

You can use amazing time picker (amazing-time-picker) for this purpose, and it is fully compatible with Angular 2-7 and Material design:
https://github.com/owsolutions/amazing-time-picker
Live demo:
https://owsolutions.github.io/amazing-time-picker
Thanks.


[ci] Restructure module_id check to save time

Description
At the moment, the module_id checks run with the in-depth lints. This makes sense but is also wasteful: it needs to compile all the tests that will later on be recompiled when running them. This takes about 30min in the CI.
Proposal: run the module_id check immediately after running the tests so that the binaries already exist.
Pros:

saves time
Cons:
easier to miss some tests
requires more changes in the azure pipeline.

@timothytrippel any thoughts?

I think this makes sense. I'm not sure I follow how this will be "easier to miss some tests"? You could also just run after the SW build step that builds all SW no?

I was thinking if we need to manually run this step after every test, we might forget to add it to some tests. But your suggestion seems good: we can just run it when we build every software item. I'll update my PR to implement that.

Ah got it. Thanks @pamaury !

@pamaury you mentioned you had a PR in flight for this... did this get addressed by now?

It has been merged #19510


Remove virtual hosting support

This removes support for virtual hosting.

The --default-host flag is now used mainly for parsing and sending HOST and hostname fields.
The default_host has been removed from modules.toml
The host field has been removed from modules.toml's [[module]] entries
The wagi.host feature has been removed from Bindle

At this point, when the server receives a request, it will treat hostnames as follows:

If a HOST header is set, it will assume that that is the correct host, or else...
If hyper provides a host, it will use that, or else...
It will attempt to use the value from --default-host, or else...
It will default to localhost:3000

Signed-off-by: Matt Butcher<EMAIL_ADDRESS>

Per a conversation with @thomastaylor312, I think I will rename the --default-host to just --host or --hostname, since it's not so much a "default" anymore.


Replace deprecated PyUnicode_* functions

https://www.python.org/dev/peps/pep-0623/
Starting with Python 3.9, some of CPython's PyUnicode_* functions are being deprecated. With coming versions, more functions will be deprecated and in Python 3.12, all the deprecated items are removed for good.
If I understand the implications of this correctly, PyO3 will need to change its usage of those deprecated functions to their modern equivalents before Python 3.12.
Information about the recommended replacements can be found in Python's C API documentation: https://docs.python.org/3/c-api/unicode.html#creating-and-accessing-unicode-strings. For example, PyUnicode_FromStringAndSize is deprecated in favor of PyUnicode_New.

Neeeevermind, sorry. PyUnicode_FromStringAndSize's usage is only deprecated when you're calling it with a NULL parameter. It's still the correct thing to use in all other circumstances.
Apart from PyUnicode_FromStringAndSize, there don't seem to be any usages of the to-be-deprecated functions in the codebase. A ripgrep search for each of the items mentioned in the PEP document yields nothing relevant.
Therefore this issue can probably be closed. Sorry for my hastiness

Neeeevermind, sorry. PyUnicode_FromStringAndSize's usage is only deprecated when you're calling it with a NULL parameter. It's still the correct thing to use in all other circumstances.
Apart from PyUnicode_FromStringAndSize, there don't seem to be any usages of the to-be-deprecated functions in the codebase. A ripgrep search for each of the items mentioned in the PEP document yields nothing relevant.
Therefore this issue can probably be closed. Sorry for my hastiness


Single Command Application example

Is there a way to use this to create a single command application?  I've gotten so used to using attributes to define arguments and options that I don't want to go back!
From https://symfony.com/doc/current/components/console/single_command_tool.html
#!/usr/bin/env php
<?php
require __DIR__.'/../../vendor/autoload.php';

use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\SingleCommandApplication;

(new SingleCommandApplication())
    ->setName('My Super Command') // Optional
    ->setVersion('1.0.0') // Optional
    ->addArgument('foo', InputArgument::OPTIONAL, 'The directory')
    ->addOption('bar', null, InputOption::VALUE_REQUIRED)
    ->setCode(function (InputInterface $input, OutputInterface $output) {
        $output->writeln("Success ");
        // output arguments and options
    })
    ->run();

I've gotten so used to using attributes to define arguments and options that I don't want to go back!

Hah!
I haven't really thought about this myself. Maybe something @chalasr could keep in mind when adding the attributes to symfony/console?
As for what you can do now, what about the alternate way to create a single command application? Something like:
#!/usr/bin/env php
<?php

require __DIR__.'/../../vendor/autoload.php';

$application = new Application('My Command', '1.0.0');

$application->addCommands([$command = new CommandWithZenstruckConsoleExtraAttributes()]);
$application->setDefaultCommand($command->getName());
$application->run();

Could this work?

Could this work?

Wouldn't that mean there would be two files for every command?  The shell and the class that uses it?
I have several simple utilities in various bundles, and I don't necessarily want to see all of those utilities with bin/console, e.g. bin/console mybundle:do-something, I'd rather have vendor/bin/mybundle do-something.
So I guess 2 classes would work, but better would be just one.  Any ideas on how we could do that?  Two classes in the same file?  Obviously not PSR compliant though.
If it is 2 classes, I guess the shell would be in /bin, and the command in /src/Command as it currently is.  So yes, that probably works.  I'll play around with it.
I already have a maker that creates the src/Command part, I could add a --single to create something in /bin.

Two classes in the same file?

I think that would work (both in the bin file).

I'm getting close, but getting the error
      Cannot add a required argument "msg" after an optional one "command".

If I remove the #[AsCommand()] attribute I get the folllowing, even though the name is defined.
     PHP Fatal error:  Uncaught Symfony\Component\Console\Exception\LogicException: The command defined in "CommandWithZenstruckConsoleExtraAttributes" cannot have an empty name. in /home/tac/ca/ac/vendor/symfony/console/Application.php:552
Stack trace:
#0 /home/tac/ca/ac/vendor/symfony/console/Application.php(522): Symfony\Component\Console\Application->add()

#!/usr/bin/env php
<?php
// bin/write-message.php

use Symfony\Component\Console\Attribute\AsCommand;
use Zenstruck\Console\Attribute\Argument;
use Zenstruck\Console\ConfigureWithAttributes;
use Zenstruck\Console\InvokableServiceCommand;
use Zenstruck\Console\RunsCommands;
use Zenstruck\Console\RunsProcesses;

$dir = __DIR__.'/../vendor/autoload.php';
assert(file_exists($dir), $dir);
include $dir;

#[AsCommand('app:write-message', 'Display a message')]
class CommandWithZenstruckConsoleExtraAttributes extends  InvokableServiceCommand
{
    use ConfigureWithAttributes, RunsCommands, RunsProcesses;

    public function __invoke(
        #[Argument(description: 'Message to display')] string        $msg,
    ) {
        $this->io()->writeln($msg);
    }
}

$application = new \Symfony\Component\Console\Application('My Command', '1.0.0');

$application->addCommands([$command = new CommandWithZenstruckConsoleExtraAttributes()]);
$application->setDefaultCommand($command->getName());
$application->run();

Hmm, my first thought is to not be extending InvokableServiceCommand but just the standard Command from symfony/console. Then have the command use the Invokable trait.
Does that change anything?

Oh, here's the issue:
-$application->setDefaultCommand($command->getName());
+$application->setDefaultCommand($command->getName(), isSingleCommand: true);

Yes, this runs!
#!/usr/bin/env php
<?php

use Symfony\Component\Console\Attribute\AsCommand;
use Symfony\Component\Console\Command\Command;
use Zenstruck\Console\Attribute\Argument;
use Zenstruck\Console\ConfigureWithAttributes;
use Zenstruck\Console\RunsCommands;
use Zenstruck\Console\RunsProcesses;
use Zenstruck\Console\Invokable;

include __DIR__.'/../vendor/autoload.php';

#[AsCommand('app:write-message', 'Display a message')]
class CommandWithZenstruckConsoleExtraAttributes extends  Command
{
    use ConfigureWithAttributes, RunsCommands, RunsProcesses;
    use Invokable;

    public function __invoke(
        #[Argument(description: 'Message to display')] string        $msg,
    ) {
        $this->io()->writeln($msg);
    }
}

$application = new \Symfony\Component\Console\Application('My Command', '1.0.0');

$application->addCommands([$command = new CommandWithZenstruckConsoleExtraAttributes()]);
$application->setDefaultCommand($command->getName(), isSingleCommand: true);
$application->run();

How do I inject services?  I'm trying replace a bash script that's a bunch of bin/console calls to Symfony commands, and I'd like to wrap them in a single command script.  But neither of the 2 injections here work:
    public function __construct(
        private KernelInterface $kernel,
        string $name = null)
    {
        parent::__construct($name);
    }

    public function __invoke(
        KernelInterface $kernel,
        #[Argument(description: 'Message to display')] string        $msg,
    ) {

        $this->io()->writeln("Message is " . $msg);
//        $this->runCommand($kernel, 'secrets:list');
    }

    private function runCommand(KernelInterface $kernel, string $cli)
    {
        $application = new Application($kernel);
        $command = $application->get($cli);
        $cliString = '-v';
        CommandRunner::from($application, $cliString)
            ->withOutput($this->io()->output()) // any OutputInterface
            ->run();
    }

How do I inject services?

Is this still in a single command app? If so, you won't have access to the container for DI.


Records do not scrape translations

Would be great to return the Übersetzungen section in an organized way

Hello,
this package does not claim to be complete. If you create a PR, I'll be happy to take a look at it. Until then I won't be able to extend its functionality.


under react native 0.48     Cannot read property 'Constants' of undefined

I'm using right now "react-native": "0.48.4" and I am testing "react-native-video": "^2.0.0" for an app I'm building. Is this error fixed in the newest version of React native? Or is there an old version of react-native-video that is stable enough for "react-native": "0.48.4"? @AntsWang

I have same issue. Can you help? please

@aronmoles downgrading to version "react-native-video": "1.2.0" fixed the problem.

what version of React-Native are you using ?

"react-native": "0.48.4",
"react-native-video": "1.2.0",


ST17H66 support

Is it possible to add support for the "new" BLE-Chips ?
It would be absolut fantastic.

These are dev boards with no predefined protocol to control LEDs, so nothing for me to add.


Azure Stack authentication based on Service rather then User

Hello
We are using Azure Stack and we would like to authenticate through service rather than a specific user with password.
In Read The Docs page showed an example of authentication only through user credentials, but no service.
We still attempted to do so, by creating application in Azure Cloud Admin portal (as AD -> App Registration option exists only in Cloud Panel, not in Azure Stack Panel)
We grant to the application all permissions to the Azure Stack trough portal and used ServicePrincipalCredentials class instead of UserPassCredentials
When we authorisation trough App - a credential object is successfully created with valid tokens and so on, however, it cant see anything, no subscriptions, no resource groups, no nothing.
If we authorise trough user personal credentials then everything works fine.
Code snippet:
CLIENT_USERNAME =<EMAIL_ADDRESS>CLIENT_PASSWORD = "<pass>"

applicationId = "<cba4f82f-6f83-48c5-blah-blah-blah>"
app_key = "<cXW/IH88+iiouB8d3EBlahBlahBlah>="
tenant_id = "<fbe03d47-17ee-48ad-blah-blah-blah>"
base_url = "https://management.local.azurestack.external"

# Getting Endpoints metadata
azure_stack_metadata = get_cloud_from_metadata_endpoint(base_url)
print "Azure Stack Metadata:"
print azure_stack_metadata

print "\n"
print "Getting Credentials"
# Doesnt work trough Service
credentials = ServicePrincipalCredentials(client_id=applicationId, secret=app_key, tenant=tenant_id,
                                          cloud_environment=azure_stack_metadata)

# It works trough user
# credentials = UserPassCredentials(CLIENT_USERNAME, CLIENT_PASSWORD, cloud_environment=azure_stack_metadata)
print "Credentials retrieved: "
print credentials
print "\n"

print "Getting Subscriptions"
sc = SubscriptionClient(credentials, base_url=azure_stack_metadata.endpoints.management)
for s in sc.subscriptions.list():
    subscription_id = str(s.subscription_id)
    print "Subscription ID: " + subscription_id
    print "Getting Resource group client"
    resource_client = ResourceManagementClient(credentials, subscription_id,
                                               base_url=azure_stack_metadata.endpoints.management)
    print "Listing Resource groups"
    for rsg in resource_client.resource_groups.list():
        print rsg
print "DONE"

Is there any way to authenticate through service with Azure Stack or only specific user credentials supported at this time?
Best Regards

@bganapa any recommendations here?

@Briksins It should have worked, though I have not tried with Python SDK+Service Principal before. I will get back to you.

Ok got it working, it was our fault, the ticket can be closed
The problem was in the setup of the admin panel.
Basically here is an instruction:

Go to Azure Cloud portal
Create an App in AD -> Register App
Grant Permissions to "Azure Stack" (Note: Azure Stack permissions are not listed, you have to specifically search it - not user-friendly to be honest)
Create a Secret Key for an App
Go to Azure Stack portal
Go to your Resource group and select required (or do for all of them)
in the selected resource group AMI permissions (where you have all users from AD) specifically search for your App name
Add it as an owner of RG

So our problem was that we didn't grant in Azure Stack AMI permission to the App to manage resource group.


Automatically publish prereleases with CI artifacts (built binaries)

LGTM.

I think time to test out on main branch.


wootility: 3.5.12 -> 3.6.16

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/wootility/versions.
meta.description for wootility is: A customization and management software for Wooting keyboards
meta.homepage for wootility is: https://wooting.io/wootility
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.6.16 in filename of file in /nix/store/kc2v81h17xpdbvrh6v8g46g7yf49pnak-wootility-3.6.16

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
wootility

Instructions to test this update (click to expand)

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

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/kc2v81h17xpdbvrh6v8g46g7yf49pnak-wootility-3.6.16
ls -la /nix/store/kc2v81h17xpdbvrh6v8g46g7yf49pnak-wootility-3.6.16/bin

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

1 package built:

wootility

Maintainer pings
cc @davidtwco for testing.

#178119


feat: add wasm32-unknown-unknown support and memvfs feature

Content:

Add a memory virtual file system isolated by filename, different from sqlite3's ':memory:' mode
Add the ability to make SQLite link to external memory allocator
Reimplement the functions used in libc (such as strcspn, qsort)

With these three features: linking sqlite3 to Rust's memory allocator, using an in-memory virtual file system, linking to a separate implementation of string functions and qsort functions, along with fine-tuned WASI headers, our implementation works under wasm32-unknown-unknown
Remind:
Time zone function temporarily missing

libsqlite3-sys/sqlite3/sqlite3.c cannot be touched (pristine from SQLite3 official almagation)
I know nothing about wasm platform so I may ask a dumb question: why headers in libsqlite3-sys/wasm-include are not provided by the compiler/platform ?

Wow, nice. I wonder if this would make us one of the -sys crates that has a story on wasm32-unknown-unknown.

libsqlite3-sys/sqlite3/sqlite3.c cannot be touched (pristine from SQLite3 official almagation)

Agreed 100%.

I know nothing about wasm platform so I may ask a dumb question: why headers in libsqlite3-sys/wasm-include are not provided by the compiler/platform ?

No, sadly. It's because there's no libc on the web. This has historically been a huge challenge for C code used by rust libraries on the web. Typically, C code uses the wasm32-unknown-emscripten target, which

Is there anywhere in the SQLite api where structures are passed or returned by value? Unfortunately, clang and rustc disagree about wasm32-unknown-unknown's ABI, and so it would be incompatible. Stuff like this is on the path to solving it, but that path is just starting: https://github.com/rust-lang/rust/pull/83763

A lot of the declarations in the header don't have definitions. What's the deal there?

We also should at least test building for this target in CI.

@passchaos This is very cool, do you intend to continue working on this PR to get it merged? If not, do you mind if others pick up the work?
Also, is the memvfs part critical, or can one just be restricted to the :memory: db? What is the advantage of having a separate memvfs, since one can have multiple distinct :memory: databases?

@trevyn Recently I am not dealing with WASM related things, welcome to continue this work; The reason for using a separate memvfs is that SQLite's ':memory:' is connnection-local, data can't be shared between database connections, my implementation allows different db connections to share data by db name, even if used from different threads (However, it seems that the latest SQLite version solves this problem, without confirming whether it can).

@passchaos Thank you!!
For reference, I have a very rough WIP of using a very small subset of the OpenBSD libc with clean licensing here: https://github.com/trevyn/rusqlite/tree/wasm32-unknown-unknown
Will submit a separate PR if I can get it working and address the rest of @thomcc 's reviews.


[BUG] KEEP Task shows "Keep: ERROR reading Keep stdout pipe"

Loading the KEEP task continually shows "Keep: ERROR reading Keep stdout pipe"on the log and does not stop.
10/08/2021 02:36:55 PM  Keep was not able to launch, please look at the Keep logs to diagnose.
10/08/2021 02:36:55 PM  Keep: ERROR reading Keep stdout pipe
10/08/2021 02:36:55 PM  Keep Termination complete.
10/08/2021 02:36:56 PM  Keep: ERROR reading Keep stdout pipe
10/08/2021 02:36:56 PM  Event: Error posting event to event queue 'PROBEMGR': Insufficient memory - event pool is full. You can increase it's size via the NOTES.INI setting EVENT_POOL_SIZE.
10/08/2021 02:36:56 PM  Keep: ERROR reading Keep stdout pipe
10/08/2021 02:36:56 PM  Event: Error posting event to event queue 'PROBEMGR': Insufficient memory - event pool is full. You can increase it's size via the NOTES.INI setting EVENT_POOL_SIZE.
10/08/2021 02:36:56 PM  Keep: ERROR reading Keep stdout pipe
10/08/2021 02:36:56 PM  Keep: ERROR reading Keep stdout pipe
10/08/2021 02:36:56 PM  Keep: ERROR reading Keep stdout pipe
10/08/2021 02:36:56 PM  Keep: ERROR reading Keep stdout pipe
...
KEEP was installed on a local Windows 10 Domino server R11.0.1 FP3 using
java -jar keepinstaller.jar -d="C:\Program Files\HCL\Domino\Data" -i="C:\Program Files\HCL\Domino\notes.ini" -k="C:\Program Files\HCL\KEEP" -p="C:\Program Files\HCL\Domino" -a

Oops wrong repo it seems.


ERROR: Plot axis not defined: y

Hi!
I created my genome dir and when trying to plot the blob by default, it asks to define the 'y axis'
Am I making some mistake?
The other plots went perfect!
my command:
blobtk plot -v blob -d genome_dir -o blob_plot.png
the error:
ERROR: Plot axis not defined: y
thanks!

It was my fault that I did not add the bam file for coverage.
I'm very sorry.

@rjchallis I think this error message could be more informative though?


Support Protobuf encoding

Implement protobuf encoder in monkey C

I'm not sure if this library is still being developed, but it would be great if we could get support for encoding messages.


Issue with changing endpoint of an old Teams App and Bot

I created a teams app for sending notifications from my application in 2022. I also had a proper bot handle in the Azure portal which was used to send proper notification messages to my Teams app using an endpoint that I declared in its settings. Recently, we have updated our application's URL and I need to update the endpoint of the bot as well. However, I'm unable to find the bot anywhere. It's neither in the Teams Developer's Portal nor in https://dev.botframework.com/bots. Due to this reason, my Teams App has stopped working. As per this link, "Web App Bot and Bot Channels Registration are deprecated but existing resources will continue to work". However, there is no mention of how to update the settings of an old, existing, working bot.

Could you please login to Azure portal and check under Bot Services section, you will be able to find your bot and on clicking on the bot, there is Configuration pane from where you can change your bot endpoint?

I checked the Azure Portal's Bot Services section, but no bots are listed over there. It's completely empty. I have the testing website and the Teams App + the bot is working fine over there (The production website is not working since we are updating the URL and hence want to update the bot endpoint). Even that is not listed here. So I'm completely unable to update any configuration of the bots of any website.

@rsgupta94 - Could you please confirm if you are able to see the app registration as part of your AAD tenant? If not, could you please raise a support ticket so that support team can check the issue with missing bot registration.
If you have a paid Azure subscription, visit the Azure Admin Center to open a ticket

@Prasad-MSFT Yes, all the app registrations are listed properly.

@rsgupta94 - Please raise a support ticket so that support team can check the issue with missing bot registration.
If you have a paid Azure subscription, visit the Azure Admin Center to open a ticket

We had created a support ticket earlier and we were redirected to customer support who instructed us to create an issue over here, which we did. It's going back and forth!

@rsgupta94 - Could you please send out an email to  Microsoft Teams Community Help and schedule a call, so that we can check the issue you are facing?

@Prasad-MSFT I have sent a mail to the email id you mentioned

Hi @rsgupta94 - We have responded to your mail. Please have a look and let us know about your availability.

Hi @Prasad-MSFT - We dropped an email yesterday requesting for a call today at 12 pm IST. However, we didn't get any reply from your end. Please drop me an email regarding your availability. We'll be available on weekdays, from 12 pm IST onwards.

Hi @rsgupta94 - For some reason we didn't get the mail and missed the instance. Please schedule a call on 17th April 12 PM IST.
Thanks!

Hi,
Sounds good! Please call us on 17th April at 12 PM IST.  Please add the following emails for the call - @., and @.
Thanks and Regards,
Rajdeep Sengupta
Tech Lead
CodeIAM (India) Private Limited
M : +91-7980956126
W: www.codeiam.comhttp://www.codeiam.com/

From: Prasad-MSFT @.>
Sent: 13 April 2023 17:31
To: MicrosoftDocs/msteams-docs @.>
Cc: Rajdeep Sen Gupta @.>; Mention @.>
Subject: Re: [MicrosoftDocs/msteams-docs] Issue with changing endpoint of an old Teams App and Bot (Issue #8344)
Hi @rsgupta94https://github.com/rsgupta94 - For some reason we didn't get the mail and missed the instance. Please schedule a call on 17th April 12 PM IST.
Thanks!
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/msteams-docs/issues/8344#issuecomment-1506844884, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOSPISOTE6J5HSK6ZPASCFTXA7TKBANCNFSM6AAAAAAWEZ5KCI.
You are receiving this because you were mentioned.Message ID: @.***>

@rsgupta94 - As discussed in the scheduled call earlier, we are checking this issue with engineering team and will inform you once we get any update.
In the meantime, could you please try raising a support request as below?

Hello everyone,
We are checking this issue with engineering team. Currently there is no update to share. We will inform you once we hear anything from engineering team.
Thanks!


[nrf noup] samples: mgmt: mcumgr: smp_svr: Add nrf5340dk-optimised sample

Adds a configuration to this sample that is optimised for testing/usage with the nRF5340 which enables the Bluetooth transport, enables most management groups, has a static partition manager file with secondary partitions on QSPI flash, configures MCUboot in overwrite only mode with network core update support

1. What intention behind that?

So that this sample can be used with any (nordic-based) board still, as the files in MCUboot will not be used

2. Why 3-rd party board files are added?

As above, only nordic-based boards are added

3. Should we remove original files from MCUboot's repository?

No

Would add that this is only need due to, yet again, partition manager

@nordicjm Can you rebase this?


[Addition]: Cluster of 9+2 Hotspots in Natural Reserve

Hotspot b58 Addresses
11zuXRPmNTnnCJeaH181xVc5J7FHinqeKd92W8hh11Zw4g6NPuB
11NU9ynz8YMEPHReq2aRLU1wDmwfHPGAFBVh1D9yEQ4VNX5N7MA
112FkXibR85Kh6e1xLee2hkCLgpyAFgTNiXvMJEuAB5w3spuWXVT
1123aVw3cjM74ahtoG4ybRwph3csWhFm4mvAANVDEut1Xww2poUt
112jEGHDbYs161vhEDUndeeKgAunWTsFE9428L37F8Br51gJ6XDE
11GZ2A6WMWzFFDuKS6uVNWWiKeRAXSCQpqdzNMJ3UVtSnAvoMhA
112qtnzahYcTwTbtUMf5QaMFcA5BhvXcfciq5TbMSgV9fuHZoH1B
11yZUueMM2Uvf4FfHCN5xC3u25Yi1ZASZd4GkMKuMLjbKh53UQw
11d94zakPd1d1qk4S5nuRjKdJT1wFnK1F6pvBHeG2taVBiFVx2Y
11LY4bWDvSgJeDdn6GGy3FEoMpgpRAGWP7MERcEU5CcMaTB7EDi
1129DtY3CinoeJAG29vKsTQy2GNzaEJDtdVDSJ9p1xbHfUa1G3uG
Discord Handle
Lomo#7404
Reason(s)
They are inside a Natural Reserve where there is no signal, also they only see each other.

And now example of real hotspots that are near:

As you can see, real ones get witnesses from sea because radio reflexion in water, but the ones from cluster have 0 of those witness
The supposed location of hotspots is in the middle of a natural reserve between mountains, where there are no buildings and signal would get lost due mountains

I would also pint out how this are has been used by other farmers before:

Thats why I added in the bttom of the list the 2 remaing hotspots that are atracting other farmers
Here you can check the other times farms got shut down
https://github.com/helium/denylist/issues/7227
https://github.com/helium/denylist/issues/5953
As a Helium hotspot owner this farming is turning very tiresome

You'll want to re-submit. Automation tools only picked up 4 hotspots that you submitted.
Did you modify the issue after submission?

You'll want to re-submit. Automation tools only picked up 4 hotspots that you submitted. Did you modify the issue after submission?

Yes, as I rise the issue more HS appeared


Single triangle BVH building crash

Building a BVH for a single triangle is not catched and triggers a crash inside the BVH building process.

Fixed by changing to libbvh


Synchronising the size-scaling of 3D models with multi-user capabilities

I finished the below tutorial, which enables synchronisation of 3D model movement and rotation between users.
What I need for my project is the ability to synchronise the size-scaling of 3D models as well.
I'm honestly already stuck on how to handle this.
If anybody has an idea, it would be immensely appreciated. Thank you.

Document Details

ID: c0833304-ecff-860a-aa97-098a7622c432
Version Independent ID: 404a0f3c-5b29-0e3c-adde-d9c71ada2502
Content: Sharing object movements with multiple users - Mixed Reality
Content Source: mixed-reality-docs/mr-dev-docs/develop/unity/tutorials/mr-learning-sharing-04.md
Product: mixed-reality
Technology: mr-dev
GitHub Login: @jessemcculloch
Microsoft Alias: vinnietieto

Solution: In the GenericNetSync.cs script, the synchronisation of the position and rotation is handled. Add localScale like this:
using Photon.Pun;
using UnityEngine;

namespace MRTK.Tutorials.MultiUserCapabilities
{
    public class GenericNetSync : MonoBehaviourPun, IPunObservable
    {
        [SerializeField] private bool isUser = default;

        private Camera mainCamera;

        private Vector3 networkLocalPosition;
        private Quaternion networkLocalRotation;
        private Vector3 networkLocalScale;

        private Vector3 startingLocalPosition;
        private Quaternion startingLocalRotation;
        private Vector3 startingLocalScale;

        void IPunObservable.OnPhotonSerializeView(PhotonStream stream, PhotonMessageInfo info)
        {
            if (stream.IsWriting)
            {
                stream.SendNext(transform.localPosition);
                stream.SendNext(transform.localRotation);
                stream.SendNext(transform.localScale);
            }
            else
            {
                networkLocalPosition = (Vector3) stream.ReceiveNext();
                networkLocalRotation = (Quaternion) stream.ReceiveNext();
                networkLocalScale = (Vector3) stream.ReceiveNext();
            }
        }

        private void Start()
        {
            mainCamera = Camera.main;

            if (isUser)
            {
                if (TableAnchor.Instance != null) transform.parent = FindObjectOfType<TableAnchor>().transform;

                if (photonView.IsMine) GenericNetworkManager.Instance.localUser = photonView;
            }

            var trans = transform;
            startingLocalPosition = trans.localPosition;
            startingLocalRotation = trans.localRotation;
            startingLocalScale = trans.localScale;

            networkLocalPosition = startingLocalPosition;
            networkLocalRotation = startingLocalRotation;
            networkLocalScale = startingLocalScale;
        }

        // private void FixedUpdate()
        private void Update()
        {
            if (!photonView.IsMine)
            {
                var trans = transform;
                trans.localPosition = networkLocalPosition;
                trans.localRotation = networkLocalRotation;
                trans.localScale = networkLocalScale;
            }

            if (photonView.IsMine && isUser)
            {
                var trans = transform;
                var mainCameraTransform = mainCamera.transform;
                trans.position = mainCameraTransform.position;
                trans.rotation = mainCameraTransform.rotation;
                trans.localScale = mainCameraTransform.localScale;
            }
        }
    }
}


Link friendly API Reference

/area documentation
Issue details
https://devfile.io/docs/devfile/2.1.0/user-guide/api-reference.html
Recently, I've been helping a few people to understand Devfile and its structure.
The API Reference is great for tool for helping people to understand  Devfile structure.
One problem that I was having with it is that I can't send links to a specific field.
It would be great if we could find a way how to add an anchor to each field  so we can send link to a specific field.

I'm hitting this issue more and more often :-(
@jc-berger do you when you will be able to look into this?
If you are busy with other stuff I might try looking into this.

@kadel thanks for bringing this back to my attention. Yes, I'm busy with other doc PRs and issues now. However, we did recently raise a PR for Antora changes to fix the 404 error for the API: https://github.com/devfile/docs/pull/121
Would any of this work be helpful for sending links to specific fields?

Would any of this work be helpful for sending links to specific fields?

I'm afraid that this won't help with that.
I haven't looked into the code yet. But if I remember correctly the API reference page is not generated by Antora. Something else generates it, and then it is embedded using iframe into the page. This is not ideal, but iframe is not the problem I have with it.
Problem is that the actual HTML page describing fields doesn't have anchors (<a name="xxx">) for fields. So it is impossible to direct someone to a specific field.

Got it, Tomas, thanks. I can redirect my focus to this issue if time permit, I have a planning meeting tomorrow.

This has been added in the landing page redesign

The new site design and especially the new API reference page are great!
It is much better, especially the examples on the right side of the api-referene page helps a lot with understanding devfile structure


Register html5 on GitFund

I want suggest register you project on GitFund. It will help raise  money for project development.

Goodness. That leaves a lot of questions unanswered -- who do you donate to when you donate to a registered project?

All donation go direct to project owner PayPal account. GitFund only registered donations on system & set up issue priority.

Aah. I'll pass, as there's little financial incentive that would motivate me.


Code optimization on hudi-common moudle

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
Code optimization on hudi-common moudle
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

hi， @yanghua ,can u take a look when freeeeeeeee?

@Trevor-zhang Please follow the contributing guidelines, e.g. the title of the PR.


test: adding test for accessing interchain account after closing channel

Description
closes: #703

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

Codecov Report

Merging #736 (dd63d6b) into main (7415da7) will increase coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #736      +/-   ##
==========================================
+ Coverage   79.55%   79.59%   +0.04%
==========================================
  Files         145      145
  Lines       10629    10646      +17
==========================================
+ Hits         8456     8474      +18
+ Misses       1759     1756       -3
- Partials      414      416       +2

Impacted Files
Coverage Δ

...interchain-accounts/controller/keeper/handshake.go
94.00% <0.00%> (-6.00%)
:arrow_down:

...ps/27-interchain-accounts/host/keeper/handshake.go
92.30% <0.00%> (-2.24%)
:arrow_down:

modules/apps/27-interchain-accounts/types/codec.go
79.54% <0.00%> (+2.62%)
:arrow_up:

...apps/27-interchain-accounts/host/keeper/account.go
100.00% <0.00%> (+23.07%)
:arrow_up:

...ules/apps/27-interchain-accounts/types/metadata.go
87.23% <0.00%> (+87.23%)
:arrow_up:


normaliseURL(u) encodes fewer items than (new URL(u)).href

Submitting test case patch as suggested in https://github.com/nuxt/nuxt.js/issues/8550

Hi @pdl thanks for making PR. Investigating added cases:
Input: http://foo.com/foo?u=https%3A%2F%2Fexample.com%2Fpath%3Fq%3Dstring
Normalized: http://foo.com/foo?u=https://example.com/path?q=string
new URL(normalized).searchParams.get('u'): https://example.com/path?q=string
Input: http://foo.com/foo?u=https%3A%2F%2Fexample.com%2Fpath%3Fone%3Dparam%26another%3Dparam%23fragment#realfragment
Normalized: http://foo.com/foo?u=https://example.com/path?one=param%26another=param%23fragment#realfragment
new URL(normalized).searchParams.get('u'): https://example.com/path?one=param&another=param#fragment
new URL(normalized).searchParams.hash: #realfragment
Input: http://foo.com/foo?x=%00%01%02%03%04%05%06%07%08%09%0A%0B%0C%0D%0E%0F%10%11%12%13%14%15%16%17%18%19%1A%1B%1C%1D%1E%1F%20!%22%23%24%25%26%27()*%2B%2C-.%2F0123456789%3A%3B%3C%3D%3E%3F%40ABCDEFGHIJKLMNOPQRSTUVWXYZ%5B%5C%5D%5E_%60abcdefghijklmnopqrstuvwxyz%7B%7C%7D~
Output: http://foo.com/foo?x=%00%01%02%03%04%05%06%07%08%09%0A%0B%0C%0D%0E%0F%10%11%12%13%14%15%16%17%18%19%1A%1B%1C%1D%1E%1F+!%22%23$%25%26'()*%2B,-./0123456789:;%3C=%3E?@ABCDEFGHIJKLMNOPQRSTUVWXYZ[%5C]^_``abcdefghijklmnopqrstuvwxyz{|}~
Please consider that there is still a difference(s) between vanilla new URL().href and normalizeURL. With normalizeURL, we parse input, decode all parts and properly encode back with maximum whatwg compatibility (if you check above, normalized results when parsed by URL, have proper props). encodeURIComponent encodes entire range while some charachters do not need to be encoded like , character in query values. Another reason for doing this encode(decode()) is to avoid double encoding.
Also there is an important difference that we are replacing space with + (for vue-router encoding compatibility that resolved nuxt#8550)

Hi @pdl thanks for making PR. Investigating added cases:
Input: http://foo.com/foo?u=https%3A%2F%2Fexample.com%2Fpath%3Fq%3Dstring
Normalized: http://foo.com/foo?u=https://example.com/path?q=string
new URL(normalized).searchParams.get('u'): https://example.com/path?q=string
Input: http://foo.com/foo?u=https%3A%2F%2Fexample.com%2Fpath%3Fone%3Dparam%26another%3Dparam%23fragment#realfragment
Normalized: http://foo.com/foo?u=https://example.com/path?one=param%26another=param%23fragment#realfragment
new URL(normalized).searchParams.get('u'): https://example.com/path?one=param&another=param#fragment
new URL(normalized).searchParams.hash: #realfragment
Input: http://foo.com/foo?x=%00%01%02%03%04%05%06%07%08%09%0A%0B%0C%0D%0E%0F%10%11%12%13%14%15%16%17%18%19%1A%1B%1C%1D%1E%1F%20!%22%23%24%25%26%27()*%2B%2C-.%2F0123456789%3A%3B%3C%3D%3E%3F%40ABCDEFGHIJKLMNOPQRSTUVWXYZ%5B%5C%5D%5E_%60abcdefghijklmnopqrstuvwxyz%7B%7C%7D~
Output: http://foo.com/foo?x=%00%01%02%03%04%05%06%07%08%09%0A%0B%0C%0D%0E%0F%10%11%12%13%14%15%16%17%18%19%1A%1B%1C%1D%1E%1F+!%22%23$%25%26'()*%2B,-./0123456789:;%3C=%3E?@ABCDEFGHIJKLMNOPQRSTUVWXYZ[%5C]^_``abcdefghijklmnopqrstuvwxyz{|}~
Please consider that there is still a difference(s) between vanilla new URL().href and normalizeURL. With normalizeURL, we parse input, decode all parts and properly encode back with maximum whatwg compatibility (if you check above, normalized results when parsed by URL, have proper props). encodeURIComponent encodes entire range while some charachters do not need to be encoded like , character in query values. Another reason for doing this encode(decode()) is to avoid double encoding.
Also there is an important difference that we are replacing space with + (for vue-router encoding compatibility that resolved nuxt#8550)


merge package from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>chore(scripts): cover@#overwrite c8 --temp-directory build/coverage ava && c8 report -r lcov --temp-directory build/coverage
chore(package): add main from template
chore(package): add nyc from template
docs(package): remove keyword null
chore(package): set $.ava.require='esm' as in template
chore(package): set $.ava.files='tests/-test.js,tests/-test.mjs' as in template
chore(package): set $.ava.extensions='js,mjs' as in template
chore(package): remove unknown value for main ({{main}})

Codecov Report

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

@@           Coverage Diff           @@
##           master     #289   +/-   ##
=======================================
  Coverage   95.01%   95.01%
=======================================
  Files           5        5
  Lines         281      281
  Branches       45       45
=======================================
  Hits          267      267
  Misses         13       13
  Partials        1        1

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 30dfd94...048d4c0. Read the comment docs.

Coverage remained the same at 46.505% when pulling 048d4c08d5a963a951537d1fda41d15887e22fbe on template-sync-1 into 30dfd941993f90d09a45f92b55f4e6bb031d6582 on master.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[WIP] Try to use shuffle_mode=appliance for load tests

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Load Tests Java 11 GBK Dataflow V2 Streaming

Run Load Tests Java 11 CoGBK Dataflow V2 Streaming


Scaling Issue

Microsoft PowerToys version
0.45
Running as admin

[X] Yes

Area(s) with issue?
FancyZones
Steps to reproduce
Set up zones. Change scaling to larger scaling in windows.  Windows now snap off screen despite rebooting. Zone editor is also showing scaling incorrectly. See screenshots. First screenshot is in the edit menu, note how my "fixed" version is actually a smaller ratio due to me having turned the scaling up.

But when you go in and edit, the actual scale is correct. See the following screenshot. I am using the bottom left configuration.

✔ Expected Behavior
Scaling the resolution should preserve zone ratios.
❌ Actual Behavior
Scaling the resolution ultimately breaks all preconfigurations of zones, and even the zone editor "preview" becomes mismatched with the actual configuration once manually fixed. Configurations that appear correct in the editor "preview" from my previous scaling actually scale windows bigger than the screen.
Other Software
Bugreport attached
PowerToysReport_2021-09-21-09-13-36.zip
.

Thank you for reporting. The layout you selected is a canvas layout, am I right? Canvas layouts are not supposed to be scaled as grid layouts. In future releases, it would be more clear if the layout fits the current resolution of the monitor, see #8528.

That appears to be the case.
There is very little indication that there are even variant methodologies
unless you're going in and choosing "create a new layout".
I simply took the shortest path, which is altering the ones that were
pre-made that was closest to what I had in mind - which appear to be Canvas
layouts.
While making clear the varients, types, etc, of zones will address my
particular issue, more clear labeling won't address the fact that Canvas
layouts will be more or less broken with scaling or resolution changes.
Thank you for your support.
On Tue, Sep 21, 2021 at 1:01 PM Seraphima Zykova @.***>
wrote:

Thank you for reporting. The layout you selected is a canvas layout, am I
right? Canvas layouts are not supposed to be scaled as grid layouts. In
future releases, it would be more clear if the layout fits the current
resolution of the monitor, see #8528
https://github.com/microsoft/PowerToys/issues/8528.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/13331#issuecomment-924178993,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGUJ2DDZEJPON7RMZ3ZYMP3UDC27LANCNFSM5EONJDPQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@cbtlr how often do you swap your resolution / scale your res?

Not often at all, but I had to delete all my previous canvases and remake
them - this time as grids that would scale.
Now I just need powertoys to remember my layout for wireless screens I
project to...
On Wed, Sep 29, 2021, 5:44 PM Clint Rutkas @.***> wrote:

@cbtlr https://github.com/cbtlr how often do you swap your resolution /
scale your res?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/13331#issuecomment-930567809,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGUJ2DBL5JGLOUDFCXRBTDDUEOCFTANCNFSM5EONJDPQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

If you are not overlapping zones, grid layouts are they way to go.


Add newline before subns footnote

Without this change, the subnamespace footnote looks like a slightly
misformatted part of the tree:
default
└── [s] sub
[s] indicates subnamespaces.

By adding a newline, there's a nice visual separation from the tree
while also emphasizing the footnote:
default
└── [s] sub

[s] indicates subnamespaces

I also removed the period since this isn't a full sentence. If there are
any conditions, a blank line will be printed between the subnamespace
and condition footnote.
Tested: ran kubectl hns tree as shown above.

/assign @GinnyJI
/assign @rjbez17

I think this can just go in :)
/hold cancel
On Tue, Oct 27, 2020 at 12:40 PM Yiqi Gao<EMAIL_ADDRESS>wrote:

@yiqigao217 commented on this pull request.
/lgtm
/hold
Feel free to remove the hold anyone!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes-sigs/multi-tenancy/pull/1235#pullrequestreview-517915589,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE43PZEHI6QKXIELYET3DMLSM3ZWVANCNFSM4S7ZK2KQ
.


empty message list

Hi
I'm trying to get a list of letters, according to the default example
Reads folders, sees them, but the number of letters in folders = 0
code
        $client = Client::account('price');
        $client->connect();

        //dump($client->isConnected());
        $folders = $client->getFolders();
        //dump($folders);
        //Loop through every Mailbox
        /** @var \Webklex\PHPIMAP\Folder $folder */
        foreach($folders as $folder){
            dump($folder->full_name);
            //Get all Messages of the current Mailbox $folder
            /** @var \Webklex\PHPIMAP\Support\MessageCollection $messages */
            $messages = $folder->messages()->limit(10,1)->get();
            dump($messages);
        }

result

emails in mailbox

version 2.4.0
php 8.1

i enable debug

TAG1 LOGIN "****" "*****" << CAPABILITY IMAP4rev1 CHILDREN UNSELECT LITERAL+ NAMESPACE XLIST BINARY UIDPLUS ENABLE ID IDLE MOVE << OK LOGIN Completed. >> TAG2 LIST "" "%" << LIST (\HasNoChildren \Unmarked \Drafts) "|" Drafts << LIST (\HasNoChildren \Marked \NoInferiors) "|" INBOX << LIST (\HasNoChildren \Unmarked) "|" Outbox << LIST (\HasNoChildren \Unmarked \Sent) "|" Sent << LIST (\HasNoChildren \Unmarked \Junk) "|" Spam << LIST (\HasNoChildren \Unmarked \Trash) "|" Trash << LIST (\HasNoChildren \Marked) "|" test << OK LIST Completed.

TAG3 SELECT "Drafts" << FLAGS (\Answered \Seen \Draft \Deleted $Forwarded) << 0 EXISTS << 0 RECENT << OK [PERMANENTFLAGS (\Answered \Seen \Draft \Flagged \Deleted $Forwarded *)] Limited << OK [UIDNEXT 1] Ok << OK [UIDVALIDITY<PHONE_NUMBER>] Ok << OK [READ-WRITE] SELECT Completed. >> TAG4 SEARCH << BAD Command syntax error. sc=PFbZO5C9DKo1_200915_6-c5d450314271

Hi @mihakot ,
thanks for your report. Please update to the latest version (>=4.0.0) and enable the debug mode inside your used config or by enabling it manually.
Config file: https://github.com/Webklex/php-imap/blob/master/src/config/imap.php#L152
Manual:
/** @var \Webklex\PHPIMAP\Client $client */
$client->getConnection()->enableDebug();

The query you are executing seem to miss a condition:
/** @var \Webklex\PHPIMAP\Support\MessageCollection $messages */
$messages = $folder->messages()->all()->limit(10,1)->get();

..if you don't have a condition, use WhereQuery::all(). A list of available search criteria can be found here: https://www.php-imap.com/api/query
Best regards & happy coding,

thanks. it worked!!


[CORE-2244] Update to Go 1.22.2 (#9917)

If these updates aren’t needed to close a vulnerability, then they shouldn’t be applied. Closing out.


feat: self emit option

I found a behavioral inconsistency between the libp2p-floodsub implementation and this implementation.
In libp2p-floodsub, when a peer A subscribes a topic, if that same peer A publishes for that topic, the topic is also emitted to self, which does not happen in this implementation. I added this behavior behind a flag, which defaults to false right now, in order to keep the behavior you previously added. I chose false for now, as you had tests that verified if the self was not emitted to self.
In libp2p-floodsub we currently do emit to self all the times. I also agree that there are several use cases, where we may not want that, and in this context, I will also add this option in libp2p-floodsub.
I will open an issue on libp2p/js-libp2p-pubsub for discussing which should be the default behavior from js-libp2p users, as I discussed this previously with @jacobheun . If you have no reason to not do this emit self by default, I would prefer to set it true for now in this PR, as we do in floodsub (will be a breaking change!). Otherwise, I will make it true by default from js-libp2p, until we decide if we should turn it into a false default.
NOTE: This PR also contains #39 commits, as I want to use this branch for testing the integration with js-ipfs. So, once #39 is merged, those 2 extra commits will be removed.

Out of curiosity, what is the use case for emitting to self?

Quoting @jacobheun

the advantage of emitting to yourself is that you don't have to handle your message, and then publish it. you just publish, and just maintain one handler
So it simplifies things a bit for consumers.

So, for example a real time dapp would benefit from publishing data, and it appears on the UI, through the same handler where it receives messages from the other peers.

Thats very curious

I am unsure of the actual intent for originally doing this, it's an assumption I made for the use case for users. I'd personally prefer to just take it out as part of moving pubsub out of experimental (alongside the gossipsub support release of libp2p), but there are potential impacts to end users, which is what this PR aims to avoid.
Since the config is changing we could remove it from floodsub and just be clear about this in the docs and release notes for libp2p.

I am unsure of the actual intent for originally doing this, it's an assumption I made for the use case for users. I'd personally prefer to just take it out as part of moving pubsub out of experimental (alongside the gossipsub support release of libp2p), but there are potential impacts to end users, which is what this PR aims to avoid.
Since the config is changing we could remove it from floodsub and just be clear about this in the docs and release notes for libp2p.

Might be worth reaching out to people whom are using the package to see if it causes a headache first? I know the go libp2p did a change that broke a fair bit of stuff a few weeks ago, happy to take this one slowly

So @jacobheun in your opinion, we default to false both here and in js-libp2p-floodsub, and allow on js-libp2p the users to set if to true, if they intend?
I think that taking into account the breaking change we will do in js-libp2p, it is fine to do that with a release note

Also, this affects js-ipfs tests, which expect the self emit to happen.

I think the right thing to do is to make it compatible with how it is now, as this PR proposes, and then try to move away from it after some community discussion. I'd rather have this in there and get Gossipsub out without needing to worry about this level of implementation detail.

makes sense to me - we'll wait for @wemeetagain review

Codecov Report

Merging #40 into master will decrease coverage by 2.02%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master      #40      +/-   ##
==========================================
- Coverage   85.45%   83.43%   -2.03%
==========================================
  Files           8        8
  Lines         495      507      +12
==========================================
  Hits          423      423
- Misses         72       84      +12

Impacted Files
Coverage Δ

src/index.js
81.13% <ø> (-0.48%)
:arrow_down:

src/pubsub.js
87.42% <50%> (-5.83%)
:arrow_down:

Continue to review full report at Codecov.

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

Codecov Report

Merging #40 into master will decrease coverage by 1.43%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master      #40      +/-   ##
==========================================
- Coverage   85.45%   84.01%   -1.44%
==========================================
  Files           8        8
  Lines         495      513      +18
==========================================
+ Hits          423      431       +8
- Misses         72       82      +10

Impacted Files
Coverage Δ

src/index.js
81.6% <ø> (ø)
:arrow_up:

src/pubsub.js
88.39% <50%> (-4.86%)
:arrow_down:

Continue to review full report at Codecov.

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

@wemeetagain @GregTheGreek
Rebased the PR after the message validation PR got merged. I think this is good to go
Merge #39 previously than this


Extend Intel NUC Getting Started for power button location

Minor update for my NUC model.
Starting simply -- my first doc commit as a balenista.

@balena-ci rebase

@balena-ci rebase


travis: gocyclo on unit test .go files?

does it makes sense to have gocyclo run on travis on our _test.go files?

@kaccardi We already do.
The only exception I can see here is the BAT tests.  As these tests are in a folder ignored by the go build system their currently ignored.  I'll update travis to check these files as well.
Have you come across some other test files that are not covered by static analysis.

@markdryan my question was poorly worded. I noticed that we do run gocyclo on our unit tests and it made me wonder if we should not do this. I wondered if we were making things hard on ourselves without much benefit.

it was decided that I should quit whining and deal with it.


fix: polish the logic when importing Jenkinsfiles

so that we prompt the user to create a new jenkins server

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign jstrachan
You can assign the PR to them by writing /assign @jstrachan in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jstrachan
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jstrachan]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Adding concurrency flag to GitLab CI stages

Issue
Fixes #130
Description
Testing instructions
Screenshots (optional)

@ficolo got some conflicts


read_block stability issue

[gcsfs version 0.2.1]
I'm reading a file located on google cloud storage in chunks (using read_block method). There is a stability issue that occurs while reading the file (I get the following error in ~10% of cases).
  File "</home/prd/workspace/microservices/airflow/venv/lib/python3.6/site-packages/decorator.py:decorator-gen-30>", line 2, in read_block
  File "/home/prd/workspace/microservices/airflow/venv/lib/python3.6/site-packages/gcsfs/core.py", line 52, in _tracemethod
    return f(self, *args, **kwargs)
  File "/home/prd/workspace/microservices/airflow/venv/lib/python3.6/site-packages/gcsfs/core.py", line 1168, in read_block
    bytes = read_block(f, offset, length, delimiter)
  File "/home/prd/workspace/microservices/airflow/venv/lib/python3.6/site-packages/gcsfs/utils.py", line 74, in read_block
    seek_delimiter(f, delimiter, 2**16)
  File "/home/prd/workspace/microservices/airflow/venv/lib/python3.6/site-packages/gcsfs/utils.py", line 25, in seek_delimiter
    current = file.read(blocksize)
  File "/home/prd/workspace/microservices/airflow/venv/lib/python3.6/site-packages/gcsfs/core.py", line 1529, in read
    self.loc - self.start + length]
TypeError: 'NoneType' object is not subscriptable

Did anyone encounter this type of issue?

Odd that you should be seeing this only intermittently...
Would you mind trying with the master version? It uses fsspec, which has more thoroughly-tested code for the various conditions.

I"ll give it a try, since it's a probabilistic issue hard to know if it's fixed.

@martindurant seems like the issue occurs also with the new version of gcsfs, any idea on why?

Can you please post the new traceback?

@martindurant yes,
  File "/home/prd/workspace/microservices/airflow/venv/lib/python3.6/site-packages/gcsfs/core.py", line 53, in _tracemethod
    return f(self, *args, **kwargs)
  File "/home/prd/workspace/microservices/airflow/venv/lib/python3.6/site-packages/gcsfs/core.py", line 1210, in read_block
    bytes = read_block(f, offset, length, delimiter)
  File "/home/prd/workspace/microservices/airflow/venv/lib/python3.6/site-packages/gcsfs/utils.py", line 79, in read_block
    seek_delimiter(f, delimiter, 2**16)
  File "/home/prd/workspace/microservices/airflow/venv/lib/python3.6/site-packages/gcsfs/utils.py", line 25, in seek_delimiter
    current = file.read(blocksize)
  File "/home/prd/workspace/microservices/airflow/venv/lib/python3.6/site-packages/gcsfs/core.py", line 1570, in read
    self._fetch(self.loc, self.loc + length)
  File "</home/prd/workspace/microservices/airflow/venv/lib/python3.6/site-packages/decorator.py:decorator-gen-37>", line 2, in _fetch
  File "/home/prd/workspace/microservices/airflow/venv/lib/python3.6/site-packages/gcsfs/core.py", line 53, in _tracemethod
    return f(self, *args, **kwargs)
  File "/home/prd/workspace/microservices/airflow/venv/lib/python3.6/site-packages/gcsfs/core.py", line 1553, in _fetch
    self.cache = self.cache + new
TypeError: can't concat NoneType to bytes```

This does not seem to be the new current version, since there is no reference to fsspec

Thanks, when you say new current version, are you referring to the version 0.2.3?

No, I mean directly installed from master:
pip install git+https://github.com/dask/gcsfs

(thought you would do well to install fsspec from master also, first)

let me try that, thanks for the help!

Are you planning on incorporating these changes in the next version?

Yes certainly. Because the move to fsspec is a big change, I didn't want to hurry a release.

I am facing the same error with gcsfs 0.2.3
  File "/usr/local/lib/python3.7/site-packages/gcsfs/core.py", line 1570, in read
    self._fetch(self.loc, self.loc + length)
  File "</usr/local/lib/python3.7/site-packages/decorator.py:decorator-gen-37>", line 2, in _fetch
  File "/usr/local/lib/python3.7/site-packages/gcsfs/core.py", line 53, in _tracemethod
    return f(self, *args, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/gcsfs/core.py", line 1553, in _fetch
    self.cache = self.cache + new
TypeError: can't concat NoneType to bytes

and with the latest gcsfs 0.3.0 and fsspec 0.4.1
  File "/usr/local/lib/python3.7/site-packages/fsspec/spec.py", line 1040, in read
    out = self.cache._fetch(self.loc, self.loc + length)
  File "/usr/local/lib/python3.7/site-packages/fsspec/core.py", line 473, in _fetch
    self.cache = new + self.cache
TypeError: unsupported operand type(s) for +: 'NoneType' and 'bytes'

It looks it just moved to fsspec. The problem occurs randomly.

So I suppose this means that the response contents were empty, but there was no error code. Can you try to capture one of these cases, and find out what the requests response object is like in this case? We don't explicitly do retries beyond the low-level ones requests does; we could configure those retries or handle our own.

I believe this is now fixed with the new release of fsspec


Add support for Python 2

This PR adds support for Python 2, and should fix https://github.com/facebookresearch/maskrcnn-benchmark/issues/7
I've successfully run tools/train.py and tools/test.py after these fixes, and training / inference succeeded without errors.
I'll run a quick schedule train/test cycle to verify that the numbers match the expected results.

Thanks for the review @rbgirshick !
I'm performing some full trainings to verify that accuracy is the same.
Training on the reduced scheduled gave expected results, as well as inference on pre-trained models.
Once I verify that this works as expected on the full training I'll merge it.

Here are the results for training on Mask R-CNN R-50 FPN 1x. Accuracies match the baselines, and it seems that the Python 2 runtime is a bit slower. I've run it twice.

backbone
type
lr sched
im / gpu
train mem(GB)
train time (s/iter)
total train time(hr)
inference time(s/im)
box AP
mask AP
model id

R-50-FPN
Mask
1x
2
5.2
0.4657
11.6
0.13757 + 0.034
37.7
34.3
6486235

R-50-FPN
Mask
1x
2
5.2
0.4670
11.6
0.14075 + 0.034
37.6
34.2
6486657

I'm merging this.


Extract useable places from the current Freecen1 Places collection

The Freecen1 Places collection contains place names and in many cases a location compiled over the years.

A rake task has been written to extract place names and their locations from the location.
Entries with no location have been eliminated.
Duplicates with the collection have been eliminated.
This left 2893 place names on the test server; there might be an additional 90 on the production system
The remainder were been ingested into my test server and 490 were added and 2403 were skipped
It would appear that some are the 1841 Hundreds.
It is likjely worth our while to add these place_names to Freecen2 Places collection

These have been added where needed


Make subject country and organization configurable for the membership services CAs

Description
This change externalizes the configuration for those properties in membersrvc.yaml.  The new properties are pki.ca.subject.country and pki.ca.subject.organization.  The new default values are "US" and "Hyperledger" respectively.
Motivation and Context

Fixes #1408
Prior to this change, membership services had hardcoded the country ("US") and organization ("IBM") used when generating new root CA files.
How Has This Been Tested?

Added unit tests for the new functionality which pass and ran through the existing behave tests with no related failures.  I also manually spun up membership services and checked the certificates to ensure that the configuration was working.
Checklist:

[X ] I have added a Signed-off-by
[ X] Either no new documentation is required by this change, OR I added new documentation
[ X] Either no new tests are required by this change, OR I added new tests
[ X] I have run goimports, go vet, and golint. I have cleaned up all valid errors and warnings in code I have added or modified. These tools may generate false positives.  The goal is clean, consistent, and readable code.

Signed-off-by: Gari Singh<EMAIL_ADDRESS>

good catch @christo4ferris

aside from my comment about possible missing newlines at the end of the yaml files, LGTM thanks @mastersingh24

LGTM. Thanks!


Add missing variables and format scripts

Proposed changes
Problem: The new variable used in NFR tests was missing from the workflow.
Solution: Add missing variables. Also adds shfmt to format the scripts.
Checklist
Before creating a PR, run through this checklist and mark each as complete.

[ ] I have read the CONTRIBUTING doc
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked that all unit tests pass after adding my changes
[ ] I have updated necessary documentation
[ ] I have rebased my branch onto main
[ ] I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

Release notes
If this PR introduces a change that affects users and needs to be mentioned in the release notes,
please add a brief note that summarizes the change.

NONE

pre-commit.ci rerun

pre-commit.ci autofix


revC1: add io port pinout to back silkscreen

my suggestion for silkscreen as described in  #119

Screenshot?

I think it's too busy (and non-symmetrical with the top); I would favor a sparser layout with a single character per pin. V for Vio, S for sense, 0-7 for the port pins, and the NC pins can just be left unmarked. The goal of putting the pinout on there isn't to tell people what the pins do (they should know that already), but just to remind existing users of which pin is which among the pins they already know are on there.

How about this:

Much better. Horizontal alignment seems a bit off, though. @whitequark, any opinions on Vio vs V for that pin? + could be another option.

I'd say V fits nicely.

Updated version using "V" instead of "Vio". I also aligned all the numbers and letters to the respective holes of the connectors.

Thanks!


Revert "Extend logs (#1431)"

Checklist

[x] Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
[x] Make sure mvn -P integration-test clean verify runs for the whole project and, if you touched any code for a service in the services folder, ensure it can be run with spring-boot:run

Description
Revert extend logs.

also approve #1454


Setting up chatbot/rules/Chat pannel for rules

I'll try and see if I can get this setup as mod, but it may require admin privilege's.  I'll follow up with you later if it does.
Steps on how to set up chat panel:
https://medium.com/the-emergence/twitch-chat-rules-what-you-need-to-know-57e19e5b6fdd#:~:text=You can find your channel,before writing their first message.

Ok, so setting up chat rules (a custom set of rules for the channel to educate new viewers on what behavior is expected in your chat) does require admin priv/can only be set up by the channel owner.
To do this go to:

Customize Channel (on the right side next to your name below the channel video)
In the menu on the lefthand side that shows up, select "Moderation"
Partway down the page under the "Channel Privileges" header will be "chat rules(private)" in which you can set up the rules that will show up for new viewers in a popup before they type anything.
One rule per line in the following format:

No homophobia, transphobia, sexism, racism, etc...


Add Retry button to failure page

Instead of having to go back and forward again on e.g. download errors, add a Retry button directly to the installation failure message page.

Example


initial swing at state easing

#62

Wow! looks very promising. Looking forward for that, thank you!

Great, thanks!


Compatibility with JGit (TeamCity)

When Write(string) is called on response - it adds "charset=utf-8" to the ContentType, so that raw HTTP header looks like "Content-Type: application/x-git-upload-pack-advertisement; charset=utf-8" and JGit treats that as incorrect. With explicit setting of the Charset everything works fine.

I confirm this solves the issue. I'm wondering if both this fix and the other one committed in the single other fork of the project are needed, though.

Other fork seems to address git protocol details correctness, not the problem with charset. In fact I checked this version with TeamCity 6.5.4 and msysgit 1.7.8 and both are working fine. So I didn't checked neighbor forks actually :)

Thanks a lot @ivan-danilov!


add option to overwrite existing content in importCSAR dialog

Signed-off-by: Tino Stadelmaier<EMAIL_ADDRESS>screenshot:

Can one of the admins verify this patch?

Minor comment not addressed yet, therefore not merged.


fix: missing theme_root on theme pull

pull_theme feature added in #57 is not using the theme_root config.
This causing the action to be stuck indefinitely after executing the shopify theme pull command if we use both pull_theme and a custom path for theme_root because the CLI is not able to find the theme and is waiting for user input.

Thank you for your contribution :D


menu Help color template doesn't work

A minor bug, in Help, color template doesn't work.
change "colors-tmpl": "0.1.0" to "^1.0.0" in package.json would work.

I wasn't able to replicate the problem, but no harm in updating :) It has been published, thanks for reporting.


Add a plopjs generator to add new redux constants/actions/reducers

[ ] redux constants
[ ] actions
[ ] reducers

Plop generators are in the generators/react-plop/templates/scripts/generators folder. To add new ones you can have a look to the existings generators ;)
@tcheymol you said that you created such generators on your project, would you mind to create a PR?


Increase timeout on onO365ConnectorCardAction

This error message is shown even the action is handled successfully.
How do I increase the timeout?

   "botbuilder": "^3.13.1",
    "botbuilder-teams": "^0.1.7"

connector.onO365ConnectorCardAction(o365CardActionHandler);

It seems Nodejs Async works
var o365CardActionAsyncHandler = async function(event, query, callback) {};
connector.onO365ConnectorCardAction(async o365CardActionAsyncHandler);

I found a solution to handle this issue for my case.
var o365CardActionHandler = function (event, query, callback) {
    bot.beginDialog(event.address, '/new_dialog', query.body, function (err, result) {
        callback(err, result);
    });
};


python312Packages.playwrightcapture: 1.24.8 -> 1.24.9

Diff: https://github.com/Lookyloo/PlaywrightCapture/compare/refs/tags/v1.24.8...v1.24.9
Changelog: https://github.com/Lookyloo/PlaywrightCapture/releases/tag/v1.24.9
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 312612 run on x86_64-linux 1

10 packages built:

lacus
lacus.dist
python311Packages.lacuscore
python311Packages.lacuscore.dist
python311Packages.playwrightcapture
python311Packages.playwrightcapture.dist
python312Packages.lacuscore
python312Packages.lacuscore.dist
python312Packages.playwrightcapture
python312Packages.playwrightcapture.dist


add retry for dict:incr in case of post event failed

Hi, I have the following problem when using Kong:

According to the function logic of updating the target of a service, in a node, the specific balance: targets: upstream_id will be expelled in on_upstream_event, which will execute invalidate_local (" balance: targets: upstream_id ") for all workers, and this function will call mlcache: delete (key) which deletes L2 and L1 caches. And the latter on_upstream_event -> get_upstream_by_id -> cache.get  show that If there is a value in L1, it will be returned directly. Otherwise, look for L2. If there is no value in L2, L3 will be called and  execute set shm_ set_ lru_。

Specifically, I have encountered a number of workers' inconsistency in the  balance: targets: upstream_id  cache. That is, some workers' L1 still caches old data, and at least one worker can read new data (combined with the implementation of lua-resty-mlcache: delete, this indicates that delete_shm is successful, but the problem may be caused by other workers' inability to remove the old data of L1 layer).
The following is the function call logic:
[kong] cache:invalidate(service_key,cluster) ---> [kong] self:invalidate_ local(key) ---> [lua-resty-mlcache] self.mlcache:delete(key) ---> [lua-resty-worker-event] post(channel_name, channel, data) ---> [lua-resty-worker-event] post_ event(source, event, data, unique) ---> [lua-resty-worker-event] dict:incr && dict:add
I think there may be problems in the following parts:
In lua-resty-worker-event's post_event function, dict: add has a retry mechanism(shm_retries), but dict: incr lacks a retry. If incr fails, it returns directly and interrupts post_event function.
In my problem, this will cause the invalidate event to not be inserted into the shm, so other workers cannot consume it, and other workers cannot remove the L1 layer data.
Refer to the implementation of dict: incr. When this function is called, a lock will be obtained. However, when there is high concurrency, other requests are also occupying the lock, resulting the dict:incr failure, thus interrupting post_event function and causes the invalidate event cannot be inserted into the shm, so that other workers cannot consume it.
implementation of dict: incr in lua-nginx-module

Since this function is designed as post event in shm, should we try to ensure the success rate of post? So I think we should also give dict:incr a few retries to avoid this to some extent like this:
https://github.com/Kong/lua-resty-worker-events/blob/master/lib/resty/worker/events.lua#L157
before:
_dict:add(KEY_LAST_ID, 0)
event_id, err = _dict:incr(KEY_LAST_ID, 1)
if err then return event_id, err end

after:
  _dict:add(KEY_LAST_ID, 0)
  retries = 0
  while not event_id and retries < 5 do
    event_id, err = _dict:incr(KEY_LAST_ID, 1)
    if err then
      return event_id, err
    elseif retries >= 5 then
      log(WARN, "worker-events: could not incr events-last-id to shm after ", retries,
              " tries. Payload size: ", #json, " bytes, ",
              "source: '", tostring(source), "', event: '", tostring(event))
    end
    retries = retries + 1
  end

Current status is that we're letting go of this library, in favour of event communications over unix sockets.

Current status is that we're letting go of this library, in favour of event communications over unix sockets.
@Tieske  Thank you for your reply. I looked at the implementation of the new library, and also did not provide the retry of event insertion (post_event && queue.put) to ensure successful insertion as much as possible. Do you want to leave the retry of failed event insertion to the method caller? In this case, should the relevant implementation of Kong also be adjusted?

I haven't been involved with the new lib. Maybe @chronolaw has ideas?

https://github.com/Kong/lua-resty-worker-events/blob/master/lib/resty/worker/events.lua#L291

Is this error in the log file?
@zjlooojoe
@Tieske

In lua-resty-events, we will report error if event publishing fails, then you can try to post it again.

If no out-of-memory error is reported, the retry code does not resolve it, it may be caused by another issue

Retrying makes sense to me, but I think the code is flawed. The goal is to retry both the dict:add and dict:incr operations if they fail. Both are done in a tight loop.

dict:add will on each retry expel a number of entries (30 iirc) from the LRU, until there is enough room to fit it in. So during the loop changes are made to improve the probability of success.
dict:incr will not change anything on a try. So I think the tight loop is bound to keep failing.

So can we add an exponential back-off to the dict:incr operation? that should improve the retry-success. (probably need to take care of yielding and non-yielding phases)


Is testing a reducer using the store an antipattern?

Is it considered an antipattern to test reducers using the full store instead of just the reducer itself? One of my actions is async using redux-thunk and most of my testing logic (a card game) is based on cards being played or not (this logic is handled in that thunk action) and the store's state.
If I test just the reducer instead of the entire store, the redux-thunk middleware doesn't work since I'm testing just the reducer. I don't feel like mocking out my entire store for testing purposes is going to be productive though, since I already have a fully functioning store (the real one) I can use instead.
More concretely: I would run my tests using store.dispatch(someActionCreator(someVariable)) and then assert on store.getState() instead of testing just the reducer (e.g. reducer(state, someActionCreator(someVariable)) and assert on the state result that is returned from that reducer).
It kind of feels okay to do this since all of the logic is contained in the reducer which the store exposes. I was just wondering if testing your reducers like this is a big no-no.

It is not an anti-pattern. But it is more of an integration test than a unit-test.
You will find integration tests can fail for many reasons, because they depend on a lot of interconnected code.
Mocking the store is likely worth it.

It sounds like you just want to test your thunks? Testing thunk actions in isolation is usually pretty straightforward. You can pretend to be redux-thunk in a test by just calling the action.
const dispatchSpy = createSpy() //from sinon, jasmine, whatever
myThunkAction(someVariable)(dispatchSpy, someState)

You can then assert that dispatch was called with your expected actions given some state.

There are no big no-nos; it is your code and you decide how to work with it. In general I would test action creators separately from reducers because they answer different questions. Even is you use Redux Thunk you can still test the reducer directly using the actual final actions it emits. And you can test the thunk action creator separately as well— either like this or with something like redux-mock-store. But of course you can also test everything together too, as noted correctly in https://github.com/reactjs/redux/issues/1530#issuecomment-198286324.


Added generated READMEs for datasets that were missing one.

This is it: we worked on a generator with Yacine @yjernite , and we generated dataset cards for all missing ones (161), with all the information we could gather from datasets repository, and using dummy_data to generate examples when possible.
Code is available here for the moment: https://github.com/madlag/datasets_readme_generator .
We will move it to a Hugging Face repository and to https://huggingface.co/datasets/card-creator/ later.

Looks like we need to trim the ones with too many configs, will look into it tomorrow!

Looks like we need to trim the ones with too many configs, will look into it tomorrow!


Issue when sending XLM

When I go to transfer my XLM to my wallet, I get the following error message showing below. The transaction fee was taken though.
Error
{
"type": "https://stellar.org/horizon-errors/transaction_failed",
"title": "Transaction Failed",
"status": 400,
"detail": "The transaction failed when submitted to the stellar network. The extras.result_codes field on this response contains further details.  Descriptions of each code can be found at: https://www.stellar.org/developers/learn/concepts/list-of-operations.html",
"instance": "horizon-live-001/jyEwd0dy8s-35802547",
"extras": {
"envelope_xdr": "AAAAADAenO1A0p0GtIvaR8rm9uO25+c4if7Mh3amQUw1ajtbAAAAZADaE1wAAAADAAAAAAAAAAAAAAABAAAAAAAAAAAAAAAAoNYbCUl5r6IPwUFsZYjCAnHzK2x5u+zOqB7XxTyelVQAAAAAO5e8wAAAAAAAAAABNWo7WwAAAEDpYhvpVIOJSYC9qtC803h3avb1GgE+fmq86F7bI9oSoJKiWfRui0qIxX0g5Mj8X0ulZFwuOWQeqPLxr7nJd/gH",
"result_codes": {
"transaction": "tx_failed",
"operations": [
"op_underfunded"
]
},
"result_xdr": "AAAAAAAAAGT/////AAAAAQAAAAAAAAAA/////gAAAAA="
}
}

It means you don't have enough lumens

I added a more detailed error message system.


Thieving: Pickpocketing stun timer over player (consider eating while stunned)

(A timer like those you see on trees and rocks)
• Place over player or the player's head.
• Timer shows stun duration until you can pickpocket again.
• This is not the same as the timer until you can move again(explained below)
• Eating while stunned will extend/pause the stun timer(beyond its visible effect), but only for your ability to pickpocket, not move.
• The duration it extends/pauses should be however long you spent eating while stunned.
• Eating while not stunned does not affect the pickpocket cooldown.
I'm sure with a little trial-and-error, you'll figure out the right timings.

I like this idea, I am going to take a look into implementing something along these lines in the coming days.

I like this idea, I am going to take a look into implementing something along these lines in the coming days.

added in #13552 as an infobox


document some of the initial steps

Signed-off-by: Felix Kronlage-Dammers<EMAIL_ADDRESS>

LGTM