Long-process on web-server: delay of 15 min before getting new oauth token

Hi :wave:
We have web-server process which lasts ~1h25. We run it automatically at the beginning of each month but those two last months we had a delay issue of 15 min with Google before getting a new access token.
The process starts at 11pm and ends at 0h25 of the next day. At midnight, 1h after the process start, we have Google OAuth issues during 15 minutes, then everyting is going fine for the last 10 minutes:

And this is the error we have:

{ "error": { "code": 401, "message": "Request had invalid authentication credentials. Expected OAuth 2 access token, login cookie or other valid authentication credential. See https://developers.google.com/identity/sign-in/web/devconsole-project.", "errors": [ { "message": "Invalid Credentials", "domain": "global", "reason": "authError", "location": "Authorization", "locationType": "header" } ], "status": "UNAUTHENTICATED" } }

Code example
This is our ClientFactory which allows us to create Google Client and keep them in runtime cache. Each time we need a Client we call $clientFactory->getGoogleClient().
This also allows us to manually refresh the access token if it's expired, but I don't think this is the right way to do.
<?php

declare(strict_types=1);

namespace App\Admin\AnalyticsBundle\API;

use Google_Client;
use Psr\Log\LoggerInterface;

class ClientFactory implements ClientFactoryInterface
{
    private string $applicationName;
    private string $authConfig;
    private string $developerKey;
    private string $refreshToken;
    private ?LoggerInterface $logger;
    /** @var array<string, Google_Client> */
    private array $clients = [];

    public function __construct(string $applicationName, string $authConfig, string $developerKey, string $refreshToken, ?LoggerInterface $logger = null)
    {
        $this->applicationName = $applicationName;
        $this->authConfig      = $authConfig;
        $this->developerKey    = $developerKey;
        $this->refreshToken    = $refreshToken;
        $this->logger          = $logger;
    }

    public function getGoogleClient(array $options = []): Google_Client
    {
        $clientId = serialize($options);

        if (!\array_key_exists($clientId, $this->clients)) {
            $this->clients[$clientId] = $client = $this->createGoogleClient($options);
        }

        $client = $this->clients[$clientId];

        // Since we had issues with refreshing access token, we manually check+refresh it
        if ($client->isAccessTokenExpired()) {
            $client->fetchAccessTokenWithRefreshToken($client->getRefreshToken());
        }

        return $client;
    }

    /**
     * @param GoogleClient_Options $options
     */
    private function createGoogleClient(array $options): Google_Client
    {
        $client = new Google_Client([
            'use_objects' => true,
            'retry'       => [
                'max_delay' => 10, // in seconds
                'retries'   => 3,
            ],
        ]);

        if (null !== $this->logger) {
            $client->setLogger($this->logger);
        }

        $client->setAuthConfig($this->authConfig); // path to a .json which looks like `{"web": {"client_id": "....", "project_id": "...", ...}`
        $client->setAccessType('offline'); // as documented at https://developers.google.com/identity/protocols/oauth2/web-server#creatingclient
        $client->setApprovalPrompt('consent'); // as documented at https://developers.google.com/identity/protocols/oauth2/web-server#creatingclient
        $client->setApplicationName($this->applicationName); // our app name in API console
        $client->setDeveloperKey($this->developerKey); // AIzaSyDGr...........
        $client->setIncludeGrantedScopes(true); // as documented at https://developers.google.com/identity/protocols/oauth2/web-server#creatingclient
        $client->fetchAccessTokenWithRefreshToken($this->refreshToken); // a refresh token a previous developer got a log time ago

        $httpClient = new \GuzzleHttp\Client(array_merge(
            [
                'timeout' => 10,
            ],
            $options['guzzle'] ?? [],
            // Options that are normally handled by Google, in \Google_Client::createDefaultHttpClient() method.
            [
                'exceptions' => false,
                'base_uri'   => $client->getConfig('base_path'),
            ]
        ));
        $client->setHttpClient($httpClient);

        return $client;
    }
}

And we use it like this:
$gaData = (new Google_Service_Analytics($this->clientFactory->getGoogleClient()))->data_ga->get(...);

Environment details

OS: Debian GNU/Linux 10 (buster)
PHP version: 7.4.15
Package name and version<EMAIL_ADDRESS>

Do you know what happens? Is our implementation bad or there is really an issue with Google servers?
Thanks!

any update?

any update?

For google/apiclient v2.11.0 this is what works for me:
$client = new Google_Client(
    [
        'api_format_v2'  => true,
        'token_callback' => function ($cacheKey, $newAccessToken) {
            $token = array_merge(
                $client->getAccessToken(),
                [
                    'access_token' => $newAccessToken,
                    'expires_in'   => 3600,
                    'created'      => time(),
                ]
            );
            $client->setAccessToken($token);
        },
        'cache_config'   => [
            'lifetime' => 10
        ]
    ]
);

They broke it after some version, no idea if it is fixed yet. cache_config part is important, IIRC by default it is 3600(s) - it is what breaks the authentication after an hour. Using it in 4+ hours script.

I'm not sure what the use_objects option is supposed to do in your implementation, but I don't see anything else wrong in your implementation.
This behavior would indicate that the access token is expired (after an hour), but the auth library continues using for 15 minutes after it's expired, but then refreshes the token as expected.
The default behavior of the auth library cache is to refresh the token after 1500 seconds (25 minutes), so this behavior would only happen if a custom cache configuration (via cache_config was being provided to the client library). So I would suggest ensuring that the default cache lifetime is less than what you have here.
Another issue I see is that you are fetching a new access token ONE TIME by calling fetchAccessTokenWithRefreshToken, but this does not tell the client library how to refresh another once this one is expired. The way to do that would be to ensure that the file passed to setAuthConfig contains the refresh token in the refresh_token field of the JSON credentials (along with the client_id and client_secret associated with the refresh token).


Issue compiling

When trying to compile on Debian I get the following error:
Compiling xidlehook v0.6.0 (file:///home/otis/Developer/xidlehook)
error[E0658]: `crate` in paths is experimental (see issue #45477)
 --> src/x11api.rs:1:5
  |
1 | use crate::MyError;
  |     ^^^^^

error: aborting due to previous error

For more information about this error, try `rustc --explain E0658`.
error: Could not compile `xidlehook`.

To learn more, run the command again with --verbose

cargo version: 1.28.0
rustc version: 1.28.0

Update rust to 1.30 :)

Awesome, sorry I should have closed this earlier.


Add tips document(qrsh-inherit and rsync-localdir), and etc.

Changed document(/docs/) (changes are below).
Removed previously forgotten wd related entries from overview.
Added a chapter called Tips.
Added section "How to execute commands on a running compute node with qsub".
Added section "How to periodically rsync SGE_LOCALDIR to HOME etc.".

I found the fix.
Please wait until the fix is finished.

To delete the existing branch and create a new one.


change study subregion

Description of the issue
Right now, subregion is a subclass of spatial region and has only one child, study subregion.
study subregion is defined as: A study subregion is a subregion of a study region.
but has no relation to study region, neither through inheritance nor axiom.
Are subregion and study subregion important for the ontology?
If the changes from Issue #1745 will be implemented, study subregion has to be edited as well.
This is a little bit confusing, I offer a solution below but I am not sure, since subregion is defined as a spatial region that is part of a spatial region, so theoretically, a subregion could be bearer of the study region role as well, which would make the distinction obsolete. This may be related to the seemingly circular definition of subregion.
Ideas of solution
Change study subregion into a role.
study subregion role
definition: The study subregion role is the role of a subregion that is under investigation and is part of a spatial region with the study region role.
'subregion' and 'part of' 'spatial region' and 'has role' some 'study region role'
Workflow checklist

[ ] I discussed the issue with someone else than me before working on a solution
[ ] I already read the latest version of the workflow for this repository
[ ] The goal of this ontology is clear to me

I am aware that

[ ] every entry in the ontology should have a definition
[ ] classes should arise from concepts rather than from words

I think having those role is a good idea. However, I think we should add these additionally and not instead of the existing classes. We should keep then the existing classes as equivalent classes.


Make sure the stop_app() when exiting python does not fail

Task linked: QF-1771 Improvements in logging and error prevention during the upload files step


Cleanup k8s.go file of maya repo

Is this a BUG REPORT or FEATURE REQUEST?
FEATURE REQUEST
Why is this required ?
It is a good practice to remove commented code. It has been seen that the commented
code present in k8s.og at https://github.com/openebs/maya/blob/master/orchprovider/k8s/v1/k8s.go
is no longer relevant.
How to do ?

Remove the commented code in this k8s.go file
Remove the unused imports in this k8s.go file
Run make to test for any compilation errors

I'll take this, how can I be assigned to this issue?  Can I assign myself?

Sure @Aerisu !!!
Raise a PR and we assign this issue to you.

I'm having issues cloning the maya project to an IDE (IntelliJ).  Not all the project files appear.  Is there another IDE you would recommend?

At the moment I'm making my PR changes in Terminal.

Cool, Visual code is better to use.

@Aerisu @prateekpandey14 I am doing it.

@AmitKumarDas My apologies, I have been unable to complete this, busy schedule for the next few days.

Fixed as part of https://github.com/openebs/maya/pull/136


Update DeveloperGuide.md: Driver Features

Codecov Report

Merging #97 (cd0454d) into master (7730cc9) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #97   +/-   ##
=========================================
  Coverage     42.15%   42.15%
  Complexity      413      413
=========================================
  Files           127      127
  Lines          2223     2223
  Branches        263      263
=========================================
  Hits            937      937
  Misses         1237     1237
  Partials         49       49

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 7730cc9...cd0454d. Read the comment docs.


Use random MQTT client ID in example sketch

Many users load the sketch having only changed WiFi SSID and password. When multiple users attempt to connect to the same broker using same client ID, they get rejected. Currently the chances of connecting to broker.mqtt-dashboard.com using "ESP8266Client" ID are fairly slim. This change adds a random number to the client ID, increasing chances of connection for new users of this library.

I had the same error and only solved when I read your comment.
Thanks!!

Good idea... will add to the other example sketches as well


test(influxdb): testing table generation and transformations

Tests for cursor and table generation.
This tests uses the reads.Reader by mocking Store and cursors to return test-specific data.
At the moment I am not checking table keys, but only data.
I cannot get any data, because, when the transformation starts processing, it tries to add columns that already exist, because they are part of the group key.
If I remove the group key from the input data, then I get a panic because a point must have at least 2 tags.

[ ] Rebased/mergeable
[ ] Tests pass
[ ] http/swagger.yml updated (if modified Go structs or API)
[ ] Sign CLA (if not already signed)

@stuartcarnie I pushed another version of the code.
I was trying to implement GroupRead for the mockStore and I discovered it could be a bit painful and that, in the end, I could end up mocking too much. In the end I decided to leave the tests as they were in their last update.
The original purpose of this test was to create a test case for when a groupBy gets pushed down to storage and causes the MultiShardArrayCursor error (https://github.com/influxdata/flux/issues/321), this tests cannot generate that error at the moment, because GroupRead is not implemented.
However, @aanthony1243 is working on end-to-end tests for storage and I think that that is the proper way to cause the aforementioned error in a test.
We can decide either to merge this code if it is a relevant contribution, or not if it can be directly covered by end-to-end tests.

As said above, this PR overlaps with end-to-end tests functionality. I close the PR, for now.


Go-to-definition for use of C++20 concept

In the following code example:
template <typename T>
concept [[Fooable]] = requires (T t) { t.foo(); };

template <typename T> requires F^ooable<T>
void bar(T t) {
  t.foo();
}

go-to-definition at the indicated location (which is a use of a C++20 concept name) should navigate to the indicated definition of the concept.

This was fixed in llvm/llvm-project@cbcd07a4815f546739fb1374a7f24f35e5352e17


(1.7.10) in cheat mode, clicking item when inventory is full crashes

as it stated in the above
version: NotEnoughItems-1.7.10-<IP_ADDRESS>-universal.jar

You're using an ancient version of NEI.. <IP_ADDRESS> is the latest for 1.7.10, you can get it here: http://www.chickenbones.craftsaddle.org/Pages/links.html

Get the same error with 118 too.
https://github.com/Chicken-Bones/NotEnoughItems/issues/227

this bug is duplicated so many times on here,  would be nice if they could be merged.
Never the less,   i wanted to post here on the latest one  that i was able to fix this by disabling Buildcraft-compact  and leaving buildcraft enabled,  Compact is a compatibility layer,  and as far as i can see it adds minimal features (if any? haven't found it yet) I've had no issues with it disabled,  and no more crashes,  I'm willing to loose whatever compact provides to not crash every 30 minutes


VSCode extension emits error "Got error while processing diagnostics: : cannot satisfy -package-id base-<IP_ADDRESS>     (use -v for more information)"

I recently installed haskell-ide-engine, but I get the following error when I start up vscode and open a haskell file:
Got error while processing diagnostics: <command line>: cannot satisfy -package-id base-<IP_ADDRESS>
    (use -v for more information)

What GHC version your project uses? base-<IP_ADDRESS> is bundled with GHC 8.0.2 and HIE doesn't support it.

I want to use GHC 8.6.5, which is the version of GHC I have installed on my system. The project I'm having issues with is here - https://github.com/sdiehl/kaleidoscope

Update: The problem appears even in very simple projects, such as ones created with stack new. For example, I tried loading Data.Char, and got this error:
Could not load module ‘Data.Char’
It is a member of the hidden package ‘base-<IP_ADDRESS>’.
You can run ‘:set -package base’ to expose it.

If I clone that repo and say stack build on the command line, I get an error.  So the project does not build out of the box, so cannot work in hie.
$ stack build
No packages found in snapshot which provide a "llvm-config" executable, which is a build-tool dependency of llvm-hs
llvm-hs     > configure

--  While building package llvm-hs-<IP_ADDRESS> using:
      /tmp/stack14812/llvm-hs-<IP_ADDRESS>/.stack-work/dist/x86_64-linux-tinfo6/Cabal-<IP_ADDRESS>/setup/setup --builddir=.stack-work/dist/x86_64-linux-tinfo6/Cabal-<IP_ADDRESS> configure --user --package-db=clear --package-db=global --package-db=/home/alanz/.stack/snapshots/x86_64-linux-tinfo6/64c75b15ef9ff77a94fea2b04f6f1c62d7431c4ecc3716689f8642d0939ec98f/8.0.2/pkgdb --libdir=/home/alanz/.stack/snapshots/x86_64-linux-tinfo6/64c75b15ef9ff77a94fea2b04f6f1c62d7431c4ecc3716689f8642d0939ec98f/8.0.2/lib --bindir=/home/alanz/.stack/snapshots/x86_64-linux-tinfo6/64c75b15ef9ff77a94fea2b04f6f1c62d7431c4ecc3716689f8642d0939ec98f/8.0.2/bin --datadir=/home/alanz/.stack/snapshots/x86_64-linux-tinfo6/64c75b15ef9ff77a94fea2b04f6f1c62d7431c4ecc3716689f8642d0939ec98f/8.0.2/share --libexecdir=/home/alanz/.stack/snapshots/x86_64-linux-tinfo6/64c75b15ef9ff77a94fea2b04f6f1c62d7431c4ecc3716689f8642d0939ec98f/8.0.2/libexec --sysconfdir=/home/alanz/.stack/snapshots/x86_64-linux-tinfo6/64c75b15ef9ff77a94fea2b04f6f1c62d7431c4ecc3716689f8642d0939ec98f/8.0.2/etc --docdir=/home/alanz/.stack/snapshots/x86_64-linux-tinfo6/64c75b15ef9ff77a94fea2b04f6f1c62d7431c4ecc3716689f8642d0939ec98f/8.0.2/doc/llvm-hs-<IP_ADDRESS> --htmldir=/home/alanz/.stack/snapshots/x86_64-linux-tinfo6/64c75b15ef9ff77a94fea2b04f6f1c62d7431c4ecc3716689f8642d0939ec98f/8.0.2/doc/llvm-hs-<IP_ADDRESS> --haddockdir=/home/alanz/.stack/snapshots/x86_64-linux-tinfo6/64c75b15ef9ff77a94fea2b04f6f1c62d7431c4ecc3716689f8642d0939ec98f/8.0.2/doc/llvm-hs-<IP_ADDRESS> --dependency=Cabal=Cabal-<IP_ADDRESS> --dependency=array=array-<IP_ADDRESS> --dependency=base=base-<IP_ADDRESS> --dependency=bytestring=bytestring-<IP_ADDRESS> --dependency=containers=containers-<IP_ADDRESS> --dependency=llvm-hs-pure=llvm-hs-pure-<IP_ADDRESS>-1MCgdEfQEIH8UDePaGXGzF --dependency=mtl=mtl-2.2.1-BLKBelFsPB3BoFeSWSOYj6 --dependency=parsec=parsec-3.1.11-4cWJ4kq7tXA22vdnbgv3it --dependency=template-haskell=template-haskell-<IP_ADDRESS> --dependency=transformers=transformers-<IP_ADDRESS> --dependency=transformers-compat=transformers-compat-<IP_ADDRESS>-2tm9ddmq5Qk3kF2emBuYBK --dependency=utf8-string=utf8-string-<IP_ADDRESS>-1DpjxLeTvGg7ttZZojxJR5 -f-debug -f-semigroups -fshared-llvm --exact-configuration
    Process exited with code: ExitFailure 1
    Logs have been written to: /home/alanz/mysrc/github/sdiehl/kaleidoscope/.stack-work/logs/llvm-hs-<IP_ADDRESS>.log

    [1 of 2] Compiling Main             ( /tmp/stack14812/llvm-hs-<IP_ADDRESS>/Setup.hs, /tmp/stack14812/llvm-hs-<IP_ADDRESS>/.stack-work/dist/x86_64-linux-tinfo6/Cabal-<IP_ADDRESS>/setup/Main.o )
    [2 of 2] Compiling StackSetupShim   ( /home/alanz/.stack/setup-exe-src/setup-shim-mPHDZzAJ.hs, /tmp/stack14812/llvm-hs-<IP_ADDRESS>/.stack-work/dist/x86_64-linux-tinfo6/Cabal-<IP_ADDRESS>/setup/StackSetupShim.o )
    Linking /tmp/stack14812/llvm-hs-<IP_ADDRESS>/.stack-work/dist/x86_64-linux-tinfo6/Cabal-<IP_ADDRESS>/setup/setup ...
    setup: The program 'llvm-config' version ==4.0.* is required but it could not
    be found.
Progress 1/2

The project requires llvm as a dependency, however that is a separate issue. The problem occurs even with new projects, as mentioned above - the only difference is that it takes issue with a different version of base

Let's stick with example project to avoid additional issues with kaleidoscope.
@jantonioperez

How did you install HIE?
What is the value of resolver field in stack.yaml of your example project?
Which hie executables you have in PATH?

@jantonioperez I got it to start building with stack.yaml:
resolver: lts-14.15
packages:
- '.'

and kaleidoscope.cabal
...
executable chapter2
  default-language:    Haskell2010
  main-is:             Main.hs
  other-modules:
    Parser
    Syntax
    Lexer
  build-depends:
      base                 >= 4.7 && <5
    , haskeline            >= <IP_ADDRESS>
    , llvm-hs              >= 8.0
    , llvm-hs-pure         >= 8.0
    , parsec               >= 3.1
    , mtl                  >= 2.1.3
    , transformers         >= <IP_ADDRESS> && < 0.6
  hs-source-dirs:      src/chapter2
...

Give that a try, I wasn't able to fully build it to the end though since I didn't have the llvm-8 libs installed on my machine

Cabal <IP_ADDRESS>, GHC 8.6.5 (pinned).
VS Code
Version: 1.40.2 (system setup)
Commit: f359dd69833dd8800b54d458f6d37ab7c78df520
Date: 2019-11-25T14:54:45.096Z
Electron: 6.1.5
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Windows_NT x64 10.0.17763

When I create a pure cabal project, I receive the same error message:
Got error while processing diagnostics: readCreateProcess: C:\Users\rofu01\dev\haskell-ide-engine\.stack-work\install\f2ceea48\bin\cabal-helper-wrapper.exe "--with-ghc=ghc" "--with-ghc-pkg=ghc-pkg" "--with-cabal=cabal" "v1-style" "C:\\Users\\rofu01\\dev\\_x" "C:\\Users\\rofu01\\dev\\_x\\dist-newstyle/build\\x86_64-windows\\ghc-8.6.5\\x-<IP_ADDRESS>" "package-db-stack" "flags" "compiler-version" "ghc-merged-pkg-options" "config-flags" "non-default-config-flags" "ghc-src-options" "ghc-pkg-options" "ghc-lang-options" "ghc-options" "source-dirs" "entrypoints" "needs-build-output" (exit 1): failed


DO NOT MERGE - Add Knowledge base widget to support page

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
The focus of this PR is to add a new script source for helpkit.so in next.config.js and include the Script component from Next.js in the Support page for integrating Helpkit widget.
Detailed summary

Added *.helpkit.so to script-src in next.config.js
Imported Script component in Support page
Included Helpkit widget script in Support page

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@kien-ngo can I just get a preview URL for this? (DON'T NEED TO MERGE YET)


Some questions about arm builds

When we use the arm environment to compile and complete the use phase, it is found that the handler's initcontainer fails at startup
unsupported configuration: unable to find any emulator to server 'x86_64' architecture

It is found that the shell script started here is fixed to x86_64
cmd/virt-launcher/node-labeller/node-labeller.sh
virsh domcapabilities --machine q35 --arch x86_64 --virttype $VIRTTYPE > /var/lib/kubevirt-node-labeller/virsh_domcapabilities.xml

cp -r /usr/share/libvirt/cpu_map /var/lib/kubevirt-node-labeller

virsh domcapabilities --machine q35 --arch x86_64 --virttype $VIRTTYPE | virsh hypervisor-cpu-baseline --features /dev/stdin --machine q35 --arch x86_64 --virttype $VIRTTYPE > /var/lib/kubevirt-node-labeller/supported_features.xml
virsh capabilities > /var/lib/kubevirt-node-labeller/capabilities.xml

What is the reason for not adapting to the arm architecture here?
Is it possible to add different parameters to judge different schema types?

Hi, nodelabeller is only works on x86_64, please refer to the commit 72adfff47cd0d7207eb2cb69cf6f64dbe795b0ef.
Would you like to share how you start your kubevirt cluster on Arm server? By the way, we only support kubevirt works on Arm64 rather than Arm32.

Ok, I will do it. By the way, if you want to try compile kubeVirt on Arm64 and run it, Here is a fast way.
# cd to kubevirt source code
$ wget https://github.com/kubevirt/kubevirtci/pull/784.patch
$ git am 784.patch
$ export KUBEVIRT_PROVIDER=kind-k8s-1.23
$ export KUBEVIRT_NUM_NODES=2
$ export KIND_NODE_IMAGE=kindest/node:v1.23.6
$ make cluster-up && make cluster-sync

I built it in the following way
docker run -dp 5000:5000 --restart=always --name registry registry

BUILD_ARCH=aarch64

DOCKER_PREFIX=<IP_ADDRESS>:5000/kubevirt DOCKER_TAG=armfix make bazel-build-images

DOCKER_PREFIX=<IP_ADDRESS>:5000/kubevirt DOCKER_TAG=armfix make push


improving performance when converting DuckDB's results to pandas

we got some competition 😁: https://github.com/iqmo-org/magic_duckdb/blob/main/notebooks/benchmarking.ipynb
sqlalchemy is adding a lot of overhead when converting DuckDB results to pandas, the fix is simple, we should use DuckDB's native .df() method and bypass sqlalchemy.

I was just recently comparing timings between DuckDB directly via the Python API compared with using via a JupySQL %%sql cell, and noticed a considerable performance drop using JupySQL. In this case I wasn't using Pandas conversion, so it seems like there are performance issues with just using bare ResultSets.
The idea in #470 to make ResultSets lazy is good. But I also wonder if it could be good to provide more options to users to avoid the need for using ResultSets as much as possible. I like how magic_python has done this, by allowing users to specify the result type as any format that DuckDB can export to (Pandas, Arrow, Polars), or ask for a DuckDB relation back. The option to return a DuckDB relation is nice because then it enables workflows that make use of the DuckDB Python relational API.  Personally, when using DuckDB in a notebook, of the two, I'm always going to want a DuckDB relation over a ResultSet.

hi @ned2, thanks a lot for your feedback!
We inherited ResultSet from ipython-sql so my default thought was to make it better. but I see your point that in many cases, users will convert it to another format anyway. I'll ensure we take your suggestions into account, I'm still unsure what the best API so I'll keep you in the loop for feedback!

no worries, glad it's helpful!
oh and another I just thought of in favour of making the DuckDB relation available as a result type is that it's already lazy


win-15 is low on disk space for Temp directory.

Type of Request
bug
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
win-15 is low on disk space for Temp directory.

Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

Removing old files and directories to in Temp folder to free up disk space.


[Improve][Connector-V2] Fake supports column configuration

Purpose of this pull request
issue : #7492
Does this PR introduce any user-facing change?
How was this patch tested?
Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

@Hisoka-X @wuchunfu Also fix dead link,  merge as soon as possible if ci passes

@Hisoka-X @wuchunfu Also fix dead link, merge as soon as possible if ci passes

Can you submit a separate PR to fix this dead link?

@Hisoka-X @wuchunfu Also fix dead link, merge as soon as possible if ci passes

@corgy-w Can you submit a separate PR to fix this dead link?

get

@corgy-w Please resolve the conflict, thanks


Add better handling of network timeouts

Haven't seen as much of this, since I'm moving towards APIs


How to Inject the reducer in Only Smart Component Only without Using routes

I'm not sure if it's the way to do it, but here's the solution I've come up with.
I'm injecting the reducer in my smart component's constructor method like this:
import React, { Component, PropTypes } from 'react';
import reducer from 'path-to-reducer';
import { injectReducer } from 'path-to-inject-reducer';

export default class SmartComponent extends Component {
  static contextTypes = {
    store: PropTypes.object
  }

  constructor (props, context) {
    super(props);

    // function provided by the starter kit
    injectReducer(context.store, { key: 'key', reducer });
  }

  render () {
    // ...
  }
}

Hope this helps! :)


Avoid cloning String in marker evaluation

Summary
A small TODO that I found interesting. See: https://github.com/astral-sh/pubgrub/pull/35.

Tagging as #performance lol.


How to change credentials for a SP publish dataset?

How to change credentials for a SP publish dataset?
I'm trying to change the credentials for a datasource on a dataset published using SP connection and gives me an error...
It gives me error using PowerBI Actions and the UI... what I'm missing here?
thank you,
Miguel

That really depends on your situation and what you are trying todo could you elaborate more?

I  have a dataset that have two connections and I need to add SQL credentials using basic or auth but both failed... cannot change...
I have to manually take ownership with my account and everything works again...

Is the setup of your pipeline correct?

For changes the SPN when used needs to be owner and should have the correct API permissions and the correct permissions in Power BI


Cut a new release

I made a bunch of package updates, including a warning on a vulnerable package (string.js).  I'm not sure how releases get cut with this package since it doesn't appear to be automated in the travis config.  How do we release this thing?
FYI @GoogleCloudPlatform/node-team @GoogleCloudPlatform/yoshi-nodejs

Just release 2.3.5, this is what I did:
git checkout master
git pull
git checkout -b 2.3.5
rm -rf node_modules
npm install
gcloud components update
gcloud config set project my-project-id
export GCLOUD_PROJECT=my-project-id
export GOOGLE_APPLICATION_CREDENTIALS=/path/to/my/key.json
npm test

looked good so far. I updated package.json to 2.3.5 and added a 2.3.5 section to CHANGELOG.md.
git add -A
git commit -m "2.3.5"
git push -u origin 2.3.5

once PR is merged
git checkout master
git pull
npm run build # Just to be safe
npm publish . --tag latest
npm run release # Uploads binaries to GCS bucket

What's the status of synth_tool replacing this library?

Thank you :) I think? the only thing left for us is generate-scaffolding.  @kinwa91  - can you come up with a list of things we're still using repo-tools for, and create tracking issues in the appropriate repos?  It would be great to have a list of (n) things we need to do to move completely off.

Tracking work here: https://github.com/googleapis/google-cloud-node/issues/2868


Add "Gateway" to links

Please change the references in the text, add "Gateway" (added in bold)
Setup Azure Log Analytics Gateway(formerly known as OMS Gateway) to act as proxy or hub:
Azure Log Analytics Agent Gateway

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

ID: 0f520ab6-2fc9-4978-553c-6beff2ec3d9c
Version Independent ID: 840eb4e6-2871-6812-a64c-8e0b8d6c772a
Content: Onboard devices without Internet access to Microsoft Defender ATP - Windows security
Content Source: windows/security/threat-protection/microsoft-defender-atp/onboard-offline-machines.md
Product: w10
Technology: windows
GitHub Login: @mjcaparas
Microsoft Alias: macapara

Closing via #8157


Ensure arity of result callbacks is consistent.

Per https://github.com/balderdashy/waterline/issues/1043#issuecomment-109657906, waterline can at times not be super friendly with the Async package.  Particularly to play well with Async.waterfall, it's important that results callbacks are always called with the same number of arguments, whether or not there were any actual results.  I think this primarily affects findOne and possibly create, since otherwise an empty result is typically [] (as in find and update).
Here's one example where this occurs for findOne: https://github.com/balderdashy/waterline/blob/master/lib/waterline/query/finders/basic.js#L145

upvote - now on line 155: https://github.com/balderdashy/waterline/blob/master/lib/waterline/query/finders/basic.js#L155


Colon-pair details in REPL look buggy

https://docs.perl6.org/type/Pair says "Colon pairs can be chained without a comma to create a List of Pairs. Depending on context you may have to be explicit when assigning colon lists." - and follows with examples, which led me to experiments, that raise more questions. I would think all 5 of these should say the same thing but instead give 3 different answers. Does the doc need an addition to describe this? Is it a problem with Rakudo Star 2018.10?
$ perl6
To exit type 'exit' or '^D'
> (:a1:b2:c3)
(a1 => True b2 => 1 c3 => 1)
> (:a1,:b2,:c3)
(a1 => True b2 => True c3 => True)
> :a1,:b2,:c3
(a1 => True b2 => True c3 => True)
> :a1:b2:c3
a1 => True
> :a1 :b2 :c3
a1 => True

Maybe this is an issue for perl6/doc and not roast? Should I transfer it?

Yeah.  Afaik, the colonpairs are currently working by design (with which I don't always agree, but that's another matter).


refactor cec test data into fixtures

pvlib python pull request guidelines

[ ] Closes issue #xxxx
[x] I am familiar with the contributing guidelines.
[x] Fully tested. Added and/or modified tests to ensure correct behavior for all reasonable inputs. Tests (usually) must pass on the TravisCI and Appveyor testing services.
[x] Updates entries to docs/sphinx/source/api.rst for API changes.
[x] Adds description and name entries in the appropriate docs/sphinx/source/whatsnew file for all changes.
[x] Code quality and style is sufficient. Passes LGTM and SticklerCI checks.
[x] New code is fully documented. Includes sphinx/numpydoc compliant docstrings and comments in the code where necessary.
[x] Pull request is nearly complete and ready for detailed review.

Description
Prepares for https://github.com/pvlib/pvlib-python/pull/767.
In particular: try to remove all calls to retrieve_sam for all tests that were directly/indirectly calling it to retrieve CEC module/inverter data. Added specific tests for retrieve_sam() instead.

@wholmgren Maybe we can discuss this PR integration before https://github.com/pvlib/pvlib-python/pull/767.
For easier review, I would recommend going commit-by-commit. Stickler CI errors about sam_data are not really my fault, but I could move that fixture to conftest.py. I just tried to minimize changes in this PR, which already got bigger than expected. :sweat_smile:

@wholmgren Whoops. Amended the fix for the unused parameters.
I have seen that the sam_data parameter is also unused in other functions, but I did not touch them so I would rather not change that in this PR, unless to prefer so. Same with moving sam_data to conftest.py. Or would you rather do it now?
About the unused parameters, maybe it would make sense to add a lint check in the test suite (i.e.: flake8). I tend to do that with my projects and it is very useful. Also, unlike Stickler CI, you can run it locally as well. :blush:

Moved both sam_data and pvsyst_module_params fixtures to conftest.py to make Stickler CI happy.

Thanks! Test failures are unrelated, so ok with me to merge.

I'll look at it this evening.


"Save as" filename bug: "Save as" truncates filename if no extension is added in "File name:" textbox

When using "File -> Save As" to save a label in either PascalVOC or YOLO format, if no extension is added to the filename, the tool truncates the filename to 44 characters then adds the extension. This behavior happens whether or not any bounding boxes have been drawn. Once the appropriate file extension is added to the name, the label file is saved with the proper name.
I'm using the 1.7.0 release executable for Windows that's available for download from your github.io page.

OS: Windows 10 Pro 64bit, 10.0.16299
Release version: 1.7.0

I've attached a screenshot to try and illustrate this bug.

I was able to recreate the issue when using the v1.7.0 binary as you mentioned, but not when I run the latest code... So in the meantime I recommend running/building from source. I personally run it from Anaconda, which was a pretty painless setup.
Side-note: Are you using the 'Save as' operation for every image? If you set a default save directory (clicking the "Change Save Dir" button), then simply clicking the Save button will save the annotation file in that dir...
Side-note2: Having parenthesis and spaces in your filenames may cause some issues down the road

Bump @Shappenny

Thanks for looking into it. It's a pretty obscure edge case, so I'll just
work around it for now.
Side-note: No, just for images with no labels in order to create an empty
label file. The "Save" button and "Ctrl-S" do nothing for me unless there's
at least one label drawn on the image, even with the save dir set.
Side-note2: Yeah, though the code creating those file names is legacy and
it hasn't been a priority to change it.
Thanks,
Sean
On Sat, Jun 23, 2018, 00:38 vdalv<EMAIL_ADDRESS>wrote:

I was able to recreate the issue when using the v1.7.0 binary as you
mentioned, but not when I run the latest code... So in the meantime I
recommend running/building from source
https://github.com/tzutalin/labelImg#windows. I personally run it from
Anaconda, which was a pretty painless setup
https://github.com/tzutalin/labelImg#windows--anaconda.
Side-note: Are you using the 'Save as' operation for every image? If you
set a default save directory (clicking the "Change Save Dir" button), then
simply clicking the Save button will save the annotation file in that dir...
Side-note2: Having parenthesis and spaces in your filenames may cause some
issues down the road
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tzutalin/labelImg/issues/317#issuecomment-399647403,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGbCB7Codpxrwxb7sSMt24pUKC5Oxcbxks5t_fBYgaJpZM4Uy_Ue
.

You're welcome.
Ahh, thanks for the tip! I didn't really know what that button does, but
that'll be much easier to use.
Thanks,
Sean
On Thu, Jun 28, 2018, 00:26 vdalv<EMAIL_ADDRESS>wrote:

Thanks for bringing it to our attention anyhow, I'll be sure to look out
for it in the next release.
With regards to creating an empty label file, the "Verify Image" button
will allow you to do that. I've been meaning to update the docs with that
info for a while now, should get around to it soon.
Best,
-Vlad
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tzutalin/labelImg/issues/317#issuecomment-400938742,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGbCBwKjzF12RrFoROycw39ltew0yFNOks5uBIURgaJpZM4Uy_Ue
.


CCACHE support

r? @Manishearth

@bors-servo r=manishearth

:pushpin: Commit c3dea7d has been approved by manishearth

:hourglass: Testing commit c3dea7d with merge fb7a443...

:sunny: Test successful - travis


Add Global Privacy Control branding

I think we should add some Global Privacy Control (GPC) branding to OptMeowt. Maybe, the GPC logo at the bottom of the popup. Feel free to go ahead if anyone wants to ...

I added the main Global Privacy Control logo to OptMeowt at the bottom of the popup in the last commit to the main branch. I figured it was a small enough change we could just pop it right in there. Here's how it looks:

We could also drop in some more branding on the settings page, but let's discuss this in tomorrow's meeting.

I added the main Global Privacy Control logo to OptMeowt at the bottom of the popup in the last commit to the main branch. I figured it was a small enough change we could just pop it right in there. Here's how it looks:

We could also drop in some more branding on the settings page, but let's discuss this in tomorrow's meeting.

Very nice, @kalicki1! Can you move it a bit lower. Otherwise, it looks like this is related to the Dark Mode.
Also, maybe center it or move it a bit more to the side and have the logo link to globalprivacycontrol.com.

Very nice, @kalicki1! Can you move it a bit lower. Otherwise, it looks like this is related to the Dark Mode.
Also, maybe center it or move it a bit more to the side and have the logo link to globalprivacycontrol.com.


Create a diagram outlining how the Stationary Nodes are set up

This should describe/illustrate the following:

How the stationary nodes are connected (the concept of the Pi 4 supervisor system and the IO board experiment system)
What sensors are available
How the experiment flow should go (From experiment submission on the portal --> communicate with on-site server/manager --> deploy docker application on experiment system --> experiment output returns back to cloud storage)

Please leave a comment on this issue if anything is missing here!

I've included a diagram of the current layout of how the stationary nodes are connected and what sensors are available. My diagram was uploaded into the DISCOVER Teams group, under General > Stationary Nodes. This will eventually need to be expanded upon when we have the other components mapped out (described in step 3 in the above comment).


Bugfix for Required Column Attribute and German Translation

bug in display of required for Exasol
show DBeaver and Version in Connection Attributes in Exasol
Workaround for different handling of result set and table meta data in Exasol JDBC Driver V6
german translation for dbeaver.core and exasol (more to come)

A few notes:

There is a bug in plugins/org.jkiss.dbeaver.lang/.classpath diff. You can just remove org.jkiss.dbeaver.lang plugin from build, it is not used.
To provide client information it is better to use another approach. As I understood Exasol uses special connection properties (clientname & clientversion) to identify client. Oracle does the same. Check OracleDataSource.getInternalConnectionProperties method. This is a standard way to identify client while patching connection URL doesn't look very good.
Additionally DBUtils.getClientApplicationName allows to redefine client name in preferences (in some cases users want to mimic some other database client while working in DBeaver).

add provide client information:
with the exasol jdbc driver this can only be done in the url.  The properties clientname and clientversion can only be set in the url.
I've verified this with Exasol Support. They plan to allow setting these parameters as properties in the future.

Ok, I see.
I'll pull this patch. Thanks!
I'd still recommend to improve client application name obtain (see DBUtils.getClientApplicationName source).
If client name is overridden in preferences then you need to use it instead of real DBeaver product name.

OK Serge,
I will change the application name asap.
As the DBUtils.getClientApplicationName always retrieves the name including version number I will have to fill the clientversion field with some other strings. Otherwise "Unkown" will be show in the session information.
Regards
Charly


feat(rust): add timer to close secure channel

Current behavior
Currently, when to establish a secure channel, two workers are created: decrypt and encrypt. However, when the initiator shuts down its connection without explicitly removing the workers, they remain active on the responder side. This can lead to issues such as performance degradation or memory consumption over time.
Proposed changes
A timer to shut down the workers when a message is not received

@davide-baldo

I like it! The direction is excellent!
The only note I have is that using a boolean is somewhat fragile because you don't have any guarantee about when a task is actually run. For example, if some piece of the code misbehaves and blocks an asynchronous thread, every task that is using that thread will be suspended until the piece of code returns. In this case, the secure channel may be idle for the duration of the suspension (which could be minutes) and not detect it.
The most common solution would be to store the current timestamp (less performant) instead of storing the boolean. But being a simple timeout, even if we are not super precise, I think it's ok, the goal is to clean up resources, and it's doing that.

Hi @davide-baldo !
I'm not sure if it's suitable, but the security channel takes twice as long to close.

Hey @Alef-gabriel , do you plan to get back to this PR?


Fixed the jumping to function definition using 'Ctrl+LMB' and the 'Lookup Symbol' button.

Fixes #72353.
Fixes #73167.
I found the cause of bug #73167. The problem is that when we click on the Lookup Symbol button in the context menu, the method String CodeEdit::get_text_for_symbol_lookup() returns position of cursor which is located on the Lookup Symbol and wrong for code parsing.

Such a bug found in the 3.5.1.stable too.

Since we're at RC stage, and #72789 fixed the relatively minor #72353, I'm thinking the safest path might be to revert #72789 for now (and reopen #72353), and then keep this new PR to redo the relevant changes and make sure they're regression free for 4.1. WDYT?

@akien-mga ok.

Done, you can rebase and redo the relevant changes to fix #72353 + the regressions.

It seems to me that this is not a solution for #73167.

Some other related PRs: #68311 #69473

cc @Paulb23

Thanks!

I'm facing an issue regarding lookup symbol and autocomplete suggestions not working on referenced children nodes.
Could this be caused by this PR?
https://github.com/godotengine/godot/issues/82258

@nagidev No. I checked this for the previous commit, the result is the same. The reason is not in this PR.


failed to connect with Hardware API

I got following error when create product starting from today. it used to work before at 7/30
Here is the error output from the tool
SurfaceDevCenterManager v<IP_ADDRESS>

Create Option
Unhandled Exception:
System.Net.Http.HttpRequestException: An error occurred while sending the request. ---> System.Net.WebException: The underlying connection was closed: Could not establish trust relationship for the SSL/TLS secure channel. ---> System.Security.Authentication.AuthenticationException: The remote certificate is invalid according to the validation procedure.
at System.Net.TlsStream.EndWrite(IAsyncResult asyncResult)
at System.Net.PooledStream.EndWrite(IAsyncResult asyncResult)
at System.Net.ConnectStream.WriteHeadersCallback(IAsyncResult ar)
--- End of inner exception stack trace ---
at System.Net.HttpWebRequest.EndGetRequestStream(IAsyncResult asyncResult, TransportContext& context)
at System.Net.Http.HttpClientHandler.GetRequestStreamCallback(IAsyncResult ar)
--- End of inner exception stack trace ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at SurfaceDevCenterManager.Utility.AuthorizationHandler.d__5.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
at SurfaceDevCenterManager.DevCenterApi.DevCenterHandler.d__10.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at SurfaceDevCenterManager.DevCenterApi.DevCenterHandler.d__111.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at SurfaceDevCenterManager.DevCenterApi.DevCenterHandler.<HdcPost>d__131.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at SurfaceDevCenterManager.DevCenterApi.DevCenterHandler.d__15.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
at SurfaceDevCenterManager.Program.d__6.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at SurfaceDevCenterManager.Program.Main(String[] args)
Last Command:
ERROR (DevCenterErrorDetails)
Code:
HttpCode:0
Message:
Correlation Id: c6db4f17-9ba4-4d86-b319-f9c0397c1d15
Request Id:     6a71aa9f-5f8b-4bf3-83d2-b9e58d5ee80a

What I should do to correct this error?
Thanks
Herbert

This was due to an expired certificate on the hdc server.


tools/update-mlx.sh with mlx-generated/utils.cpp

I'am working on cholesky_inv from mlx-swift. it needs mlx v0.17.0. so I tried to update the submodules in cmlx. it started fine and update-mlx then successfully build the project. Then when I tried to run examples, it failed like

libc++abi: terminating due to uncaught exception of type std::runtime_error: [metal::Device] Unable to > > build metal library from source
program_source:3:9: error: unknown type name 'bfloat'; did you mean 'float'?
typedef bfloat bfloat16_t;
^~~~~~
float
program_source:5:23: error: redefinition of 'abs'
/System/Library/PrivateFrameworks/GPUCompiler.framework/Versions/32023/Libraries/lib/clang/32023.335/include/metal/metal_math:4594:18: note: previous definition is here
METAL_FUNC float abs(float x)
^

I checked the the utils.cpp in mlx-generated.  it gives starting with
namespace mlx::core::metal {

const char* utils() {
  return R"preamble(
using namespace metal;
typedef bfloat bfloat16_t;
namespace metal {
METAL_FUNC bfloat16_t abs(bfloat16_t x) { return static_cast<bfloat16_t>(__metal_fabs(static_cast<float>(x), __METAL_MAYBE_FAST_MATH__)); }

It is the same to the jit/utils.cpp when I build mlx-c.
I tried update-mlx on main branch from mlx-swift and results are the same, which is far from the original  mlx-generatred/utils.cpp in main branch.

However, when I replaced  the generated file with  original mlx-generated/utils.cpp in v0.17.0 case, it seems working well.

Yes, this requires a manual edit, see:

https://github.com/ml-explore/mlx-swift/blob/main/tools/update-mlx.sh#L19

I fount it may led by #https://github.com/ml-explore/mlx/issues/1170

Yeah, the update to mlx-c and mlx-swift are a bit involved -- each time there is something new.  You may want to just wait for us to update them.  We are getting ready now, so maybe next week.

I used the newest mlx and updated mlx-c  ( escaping from #https://github.com/ml-explore/mlx/commit/dba2bd1105a0e88ff0c9717d6c0382894afee435). Metal_version=3.0 and the problem solved. #https://github.com/AaronGe88inTHU/mlx-swift/tree/v0.17.2 is my implement.


adds circle ci context

Codecov Report

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

@@           Coverage Diff           @@
##           master       #4   +/-   ##
=======================================
  Coverage   77.60%   77.60%
=======================================
  Files           4        4
  Lines         125      125
  Branches        4        4
=======================================
  Hits           97       97
  Misses         26       26
  Partials        2        2

Flag
Coverage Δ

#backend
77.60% <ø> (ø)


reduces logo size

#7
Description

Changes the main page logo to the one created by the Batch 4 developers;
Creates a new title and subtitle for our application;
Uses a smaller image of only 80 KB.

Additional Information

[X] I have read the contributing docs (if this is your first contribution)
[X] This is not a duplicate of any existing pull request

Your ENS/address: 0xeA2f0BB0144fbff64E6f6F492B999674f9393882

@carletex @BillSchumacher made the changes you proposed! Thanks guys

Great stuf @gugasanchez
Pushed a little tweak here: https://github.com/BuidlGuidl/batch4.buidlguidl.com/pull/27/commits/736ec66c29bd53764d9471c9f6cad3b9e79a1596
BTW, you don't need to close & open a new PR when changes are requested. You can just push to the PR's branch and everything will be updated!


Use one-time angular bindings for view data that doesn't change

https://docs.angularjs.org/guide/expression
I think this should significantly improve performance for when we're displaying a lot of Pokémon cards at once, since angular won't have to recalculate all of them every time we make a change.

👍


PLT-572 Check alpha-equivalence of programs in conformance tests

refactored into using the golden test function instead of a custom IsTest instance. This is cleaner especially when we add more types of tests down the road.
the expected to fail tests are now marked with their filepath, not their names.
the UPLC evaluation tests now test alpha-equivalence of programs instead of comparing them textually. This hasn't fixed the failing Agda churchZero test yet because the Agda evaluator is actually returning a not alpha-equivalent program to the expected one. It seems the Agda evaluator has a bug relating to the off-by-one debruijn discrepancy we have in plutus-core.
with this merged once we fix the bug churchZero should be fixed. If not, there's a bug in checking the alpha-equivalence and we should fix it.

Pre-submit checklist:

Branch

[x] Tests are provided (if possible)
[x] Commit sequence broadly makes sense
[x] Key commits have useful messages
[x] Relevant tickets are mentioned in commit messages
[x] Formatting, PNG optimization, etc. are updated

PR

[ ] (For external contributions) Corresponding issue exists and is linked in the description
[x] Targeting master unless this is a cherry-pick backport
[x] Self-reviewed the diff
[x] Useful pull request description
[x] Reviewer requested

Programs with De Bruijn induces are not compared modulo alpha, since the syntax is supposed to be modulo alpha
In the implementation De Bruijn indices start with 1, which is weird and must be what's causing the failure. I've complained about that before and others too

In the implementation De Bruijn indices start with 1, which is weird and must be what's causing the failure. I've complained about that before and others too

Unfortunately we're pretty stuck with this since it's now immortalized in the binary format :|


Add Op class support for multi input and multi output function. 2.…

… Add DRR duplicate TransposeOp merge testing code

PR types

Othres
PR changes

Othres
Description

Add Op class support for multi input and multi output function.
Add DRR duplicate TransposeOp merge testing code

Add transferOP in createOption func
Add output tensor judgment for graphs in the match method
Delete VELOG in the match method


Fix parsing of unquoted NaN values in HashCache file

Include the filename in the error message if parsing fails.
Replace unquoted NaN strings with null.

I roughly tried to follow the similar fix added in commit https://github.com/RupertAvery/DiffusionToolkit/commit/c75a4489d4b53cd114f41dbed6c14d3bf13fad27.
fixes #155

Thanks for this. I'll check it out

Thanks, please check if the merge went well.


Settings Page

Added Settings Page with CRUDs for 'lucro' and 'especialidades'
Fix bug that leaves the page when reloading and the page needed authentication

Added this page:


refactor: update code coverage

Additionally, move custom attribute to Core service

:tada: This PR is included in version 0.25.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


React todo list app

Built the app with React
Updated readme file
Deployed the app on netlify.com

Hi @AFakhriddinov ,

Your project is complete! There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights
✔ Good commit history
✔Linters are passing
✔Very professional README
✔Correct Github flow
✔Descriptive PR
Optional suggestions
[Optional] In future please remember to merge every approved pull request to your dev branch before working on the next milestone. This is necessary as the pull request for each milestone should only contain the changes related to that particular milestone and not the changes from previous milestones. Thank you 👍
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.


Added .ConfigureAwait(false) to every await call

Just started playing with Roslyn, made a rewriter that adds .ConfigureAwait(false) (or changes it into true in some classes/methods, if you ask nicely) to every await expression without one and found your issue https://github.com/TimoHeiten/zer0mqXt/issues/30 that asks exactly for that, so I ran it on your core project and commited the results. Hope it didnt miss anything.

Cool - thank you very much for your contribution!
LGTM - I give the "main-tainer" also a chance to have a look at it as well.
I am a bit of a sucker for everything about Roslyn - would you mind sharing your project with us - I would love to have a look at it.

ConfigureAwait was implemented.


Expose CKMS point counts

This commit exposes the total number of points seen by a CKMS
datastructure through a function called count(). This removes
the need for the client to compute and store this information.
Signed-off-by: Brian L. Troutwine<EMAIL_ADDRESS>

r+
love that you updated the tests, too!


b-modal"er" (b-toaster)

Clear and concise description of the problem
After building out useToast, the effect of b-toaster was shown. In b-toaster, you're able to programmatically create toasts. We're able now to programmatically show/hide modals from a global perspective, however, what if you were able to create modals? Going off of this idea, https://vueuse.org/core/createTemplatePromise/ could possibly be used. You could possibly make this create method be a promise to be resolved, this could prove a playground for being able to create "confirm-modals" as shown here https://bootstrap-vue.org/docs/components/modal#modal-message-boxes up until this idea, I thought triggering confirm modals wasn't going to be added. Combined with https://github.com/bootstrap-vue-next/bootstrap-vue-next/issues/1653 confirm, create modals could be quite simple
Suggested solution
I suppose a similar backport of useToast could be added, to make it so in addition to just creating toasts they could be also used to show/hide toasts as well. Combining these codebases into one.
Important
"no-stacking" capabilities could be a challenging addition, when you consider orchestrating open modals that were user-initiated, with orchestrating modals in the global stack. Maybe not challenging, but it could be weird

If the value of useToast was instead made a provideable value instead of a "global" value, this would work better. That way the value will display as undefined when you try to mutate the state with useToast. If useToast simply did inject() with a default of undefined, then you can test if the parentData === undefined to safely ignore execution of functions without a global context, or throw errors


Adds template for Sonatype Nexus service

Changes

Adds Sonatype Nexus service template, both for x86 and arm.

Docs

https://github.com/coollabsio/documentation-coolify/pull/99

Thanks for the PR. I will test and fix it.


#161456525] Update badges

what does this PR do?
Implement changes proposed by LFA
Description of task accomplished?
update travis badge
How should this app be manually tested?
Clone the repo to your local machine
run the app by running run.py
test the app by using pytest
What are the relevant Pivotal tracker stories?
#161456525

Pull Request Test Coverage Report for Build 95

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

Totals

Change from base Build 93:
0.0%

Covered Lines:
241

Relevant Lines:
264

💛  - Coveralls


[WIP] Request toolbar layout after drawable loads

This applies https://github.com/facebook/react-native/pull/13876 in order to test it on CI against master. This is only temporay PR.

Ok all green. #18376 looks really good from what I can see


test: log parse error with line context

Fail with context when failing to parse spec:
Error: Unexpected file format in test-error.txt around

  | # Broken Spec
  |
  | b
  |

Related to https://github.com/lezer-parser/lezer/issues/25

Thanks!

Can we cut a release with this change and the other ones recently introduced?
Working of master seems not a good mid-term solution.

Sorry for the slow turnaround—was on holiday. I've released a 0.9.1 with the new patches.

:+1:


Fix typo

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

@bleblancMSFT - PR has been copy edited and is ready for final review.
Thank you.
cc: @kenwith


Generate NOTICE.txt and dependencies.asciidoc

add script to generate NOTICE.txt and dependencies.asciidoc
generate NOTICE.txt and dependencies.asciidoc
check they are up-to-date in ci

closes #197

Thank you for this, LGTM, aside from the name of the extension

@stuartnelson3 I actually just run make but the license wasn't updated. Can you ensure to also include the update into the regular make and build process?
It looks like you only hoooked up the check that the license is up to date with Jenkins.


Cluster /{key}/docs endpoint for shareable Swagger UI.

add /{key}/docs for Modules. If there is no openapi_spec method or the module is not found, we see an error like so:

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#590  👈
#589
#588
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rohinb2 and the rest of your teammates on  Graphite

Merge activity

Mar 14, 5:10 PM EDT: @rohinb2 started a stack merge that includes this pull request via Graphite.


Add Social icons to the footer with hover effects [Fixes #22]

Fixes Issue
Fixes #22
Changes proposed
Add Social icons to the footer with hover effects

Code of Conduct

[x] By submitting this pull request, I confirm I've read and complied with the CoC 🖖

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

okay

okay

You don't have to close the pr
You can simply push the code on the same branch, and the code will automatically be reflected here.

okay

You don't have to close the pr You can simply push the code on the same branch, and the code will automatically be reflected here.

Okay


webpack use @babel/plugin-syntax-dynamic-import get error

Bug report

What is the current behavior?
when I use @babel/plugin-syntax-dynamic-import to import some pack,expamle

If the current behavior is a bug, please provide the steps to reproduce.

What is the expected behavior?

Other relevant information:
webpack version:  4.29.5
Node.js version: 10.15.1
Operating System:  chorme
Additional tools: babel @babel/plugin-syntax-dynamic-import

when I use webpack 4.8.0 , this bug is fixed

What is your question?

Not enough information to identify the problem


gh-13340: Allow adding a parent field to an index with no fields

Addresses #13340 . It relaxes the previous rule that adding a parent field to an existing index is always forbidden, allowing it in the case where no fields have been created. In such a case it will be safe, and this enables the trivial case of opening and closing a new empty index twice to work in a sensible way.

hm maybe this is not safe? If one creates a doc block filled with empty documents? I'm not sure ...

I'll push since there don't seem to be any concerns raised. If we later want to make the index metadata a first-class file on its own we can always do that.

Would it be possible to back-port this fix to v.9.11? I did hit this issue as well while trying to make use of #12829.

I don't think we can backport to 9.11 since it has already been released, but I will backport to 9.x and then it should get released with the next. There was some talk about a 9.11.1, so maybe it would be in that?

That would be nice, thanks. @jpountz @benwtrent fyi.

@msokolov it might be worth doing a 9.11.1 release to fix this particular bug and maybe another other one: https://github.com/apache/lucene/pull/13475
I didn't know about this outstanding bug (parent field with empty index) when I originally did the 9.11 release, I should have investigated.

sigh, I would have backported right away except for some reason I thought the parent-field enforcement was only on 10.x not 9.x


HADOOP-18320. Fixes typos in Delegation Tokens documentation.

Description of PR
Noticed some typos while reading through the documentation, this PR fixes those.
How was this patch tested?
Not required

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m  7s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+0 :ok:
markdownlint
0m  0s

markdownlint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
41m  3s

trunk passed

+1 :green_heart:
mvnsite
0m 54s

trunk passed

+1 :green_heart:
shadedclient
64m 19s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 35s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 38s

the patch passed

+1 :green_heart:
shadedclient
22m 45s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
asflicense
0m 43s

The patch does not generate ASF License warnings.

91m 24s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4499/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/4499

Optional Tests
dupname asflicense mvnsite codespell detsecrets markdownlint

uname
Linux c7ba420d0b17 4.15.0-175-generic #184-Ubuntu SMP Thu Mar 24 17:48:36 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 86ef9d01dde1d44ad674d98ca8440a2fdfc5047c

Max. process+thread count
607 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-aws U: hadoop-tools/hadoop-aws

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4499/1/console

versions
git=2.25.1 maven=3.6.3

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

thanks for taking a look @mukund-thakur, have made the suggested changes.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 48s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+0 :ok:
markdownlint
0m  0s

markdownlint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
41m 17s

trunk passed

+1 :green_heart:
mvnsite
0m 41s

trunk passed

+1 :green_heart:
shadedclient
64m 27s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 30s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 32s

the patch passed

+1 :green_heart:
shadedclient
22m 48s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
asflicense
0m 34s

The patch does not generate ASF License warnings.

90m 34s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4499/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/4499

Optional Tests
dupname asflicense mvnsite codespell detsecrets markdownlint

uname
Linux c0b8666f4e9e 4.15.0-200-generic #211-Ubuntu SMP Thu Nov 24 18:16:04 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 736cedb4adc37822ef2f1ece48e4e4f4cca0d8c0

Max. process+thread count
568 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-aws U: hadoop-tools/hadoop-aws

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4499/3/console

versions
git=2.25.1 maven=3.6.3

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.


Fix priority issues with Z-Moves

@Marty-D

Well, no existing Z-Moves have priority, but future-proofing is always good. Neat test too! :+1:


chore: bdd-nh-1583822580 to 0.0.1

chore: Promote bdd-nh-1583822580 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
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

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


Player Body Direction

Valve's interaction system has a pretty handy Player class which has a few methods I'd love to see exposed in VRTK in some way.
Though at this point in time the main method I'm interested is Player.bodyDirectionGuess:
var bodyDirection = playAreaTransform.forward;
if (hmd)
{
	Vector3 direction = Vector3.ProjectOnPlane(hmd.forward, trackingOriginTransform.up);
	if (Vector3.Dot(hmd.up, trackingOriginTransform.up) < 0.0f)
	{
		// The HMD is upside-down. Either
		// -The player is bending over backwards
		// -The player is bent over looking through their legs
		direction = -direction;
	}
	return direction;
}
return trackingOriginTransform.forward;

This is a pretty easy thing to implement ourselves but it seems like it would be a useful addition to the toolkit.
We have some classes which offer similar functionality (such as VRTK_MoveInPlace, and I believe VRTK_UIPointer as well) but those are pretty specialized unfortunately.
With that said, I'm wondering if having a similar "Player Utilities" class might be useful? It could be something that piggy backs off of VRTK_DeviceFinder and offers some basic methods such as this, or perhaps it could be apart of  VRTK_DeviceFinder directly (though I'm not sure if that would really make sense)?

Agreed

What would be the benefit to know if the headset is upside down?

This is a snippet from SteamVR, but it's presumably so that your forward vector is still "forward" if you're leaning very far forwards / backwards (which isn't all that common, but is a perfectly valid edge case).

Linked to https://github.com/thestonefox/VRTK/issues/1522


Fonts: Group in Subfolders + Add Preview Images

[ ] Group fonts in subfolders to reduce quantity of fonts per folder.
[ ] Create fonts preview images.

Grouping fonts
There are too many fonts in the image-fonts/ and bmf-fonts/ directories. They should grouped in folders by the first letter of the font name. The problem is that many font files are just named as numbers.
For the BMF fonts, it should be possible to extract their name from the font metadata, with some dedicated tool.
Many image fonts from the Guldkrans’s collection were originally named with numbers, apparently, so I'll have to find another criteria to sort them in subfolders.
Fonts previews
I need to find (or create) a tool to automatically create an image preview of each BMF font, possibly using the font metadata to add its name to the image too, along with a specimen of the font characters, digits and symbols.

BMF Tools
http://bmf.php5.cz/?page=support
BMF Specs
http://bmf.php5.cz/?page=format
https://www.cosmigo.com/promotion/docs/onlinehelp/bitmapFonts.htm


Running test with Travis

I'm trying to deploy my app through Travis CI, but it fails the test phase because of the error:
> swagger project test
sh: 1: swagger: not found
npm ERR! Test failed.  See above for more details.

Is there a way to run the tests without having the swagger module installed on the remote machine?

Is the swagger module itself listed as a dependency in your package.json (and not only e.g. the swagger-express-mw module) ?
If I remember correctly I had this one too when deploying to Travis CI and could fix it by adding the swagger module (as a locally installed dependency ... and thus in the PATH) in my package.json.

@arne-at-daten-und-bass but that seems to be an overkill. I don't want to install the whole swagger and all its dependencies just to run the tests

Ok @bolein, I (personally) don’t see that point, but of course I do not know your project .. so never mind. Then, I guess, you can try installing the swagger-cli module only, but I suppose this (alone) probably will not work.
Independent of swagger-node capabilities you might best go for „standard“  #npm features (like configuring a test script in the scripts section of your package.json, e.g. node /path/to/test.js) or Travis CI features (like configuring the before_script and script stage according to your needs with default shell commands).
Anyway, for everybody else coming across this issue and deciding to go for swagger project test, keep in mind, that there is still an open issue with its exit code:
#288


How to sort a list by the latest version

Hi,
I'm parsing Yum repositories and I need to find the latest version for a package.
How can I implement this kind of sort?
repository
  .packages
  .iter()
  .sorted_by(|a, b| ...)
  .unique_by(|pkg| &pkg.name)
  .map(|pkg| YumUpdate {
    name: pkg.name.clone(),
    version: pkg.versions.first().unwrap().version.clone(),
  })

I tried Version::parse but I can't find a comparator to find the greatest/latest semver version. Thanks!
Full code here: https://github.com/navarroaxel/archlinux-checkupdates-external/blob/main/crates/yum/src/fetch.rs#L29

Version implements PartialOrd and Ord, so you can just compare them using normal comparison methods.  There is a detailed description of ordering in the docs.


https://www.ebi.ac.uk/chebi/searchId.do?chebiId=CHEBI%3A141070

This entry is incorrect. Note molecular formula and the Monoisotopic Mass

Now fixed. Thanks


ftp: Add test for FTP bounce attack detection

Ticket
Redmine ticket: https://redmine.openinfosecfoundation.org/issues/6087
Rebase of #1234

Merged in #1284


[EPIC] Allow Execution of queued transactions

Goals
Allow users to Execute txns even if a previous execution hasn't completed yet.
Mindmap: https://kinopio.club/allow-execution-of-all-txns-K_KHlteY0qCYu7HgJt4PG
First of all – research
Estimation may be failing for queued transactions because of the nonce. ⚠

[ ] Check, and if it's the case, find a workaround
[ ] Test with a few txns
[ ] Maybe query the Safe contract directly to fetch the nonce – Core SDK has getNonce – if already updated, enable execution right away

UI

[ ] Show a warning message that the tx may fail/gas can be wasted
[ ] Make the button not green
[ ] Show a prompt

Screens

Figma:

Links
Problem statement: https://github.com/gnosis/safe-pm/issues/27

Estimation will fail if a signed tx has a nonce in future. 👎
However, if a tx doesn’t have any signatures yet, estimation will work. So we could save the estimation when creating a tx. 👍

the design for warning modal


[FEATURE] Implement an API that returns a single Post

Implement an API that returns single Group Post by post id

Hi @mdryankov, we also would like to use this API for https://github.com/rokwire/illinois-app/issues/2354 so I have assigned this over to @roberlander2 so you can focus on cleanup.


Updates for new sagain api, s-c-build 3.1 and boot 2.6

Fixes gh-196
Fixes gh-238
Fixes gh-194

200 files...? LGTM ;)

Yeah, upgrading build did they. I'd really rest template Sagan client and Sagan updater.


feat: Add markers for node type

This PR adds markers that distinguish leaf/terminal nodes from branch/internal nodes. The markers can be enabled/disabled and their icons changed in a new node_markers config section.
It also removes an extra column of whitespace on the right borders of the left and middle popup panes.
Screenshots
Selecting a branch node:

Showing a preview of a leaf node:

Alternate: with space after branch node marker. This takes more space, but the markers are better aligned:

Disabling node markers works as expected (also the selected cursorline on the left and middle panels now extends to the panel border):

Hey wow! Very cool man!
I was actually trying to implement something like this but by adding spaces till the end, was facing issues in that approach while calculating string length. Because unicode character count as four characters. But using virtual text is brilliant idea!!

Thanks!
The one drawback is that the virtual text can cut off the last two characters of a label, such as display:focus_rang in the example. I tried setting wraplines for to the window width minus some number but it didn't seem to work. Since it's only two columns, I don't think it's a huge issue. What do you think?

Tried them out, noticed that when the text is exactly as long as the window width, a few characters get hidden under the virtual text. Its not a big issue in my opinion anyways.


driver: nrfx_uarte: Handle RXTO ISR only if it's expected

When initializing the UARTE driver, a check is present for the case where UARTE RX was left enabled by a bootloader.
If it's enabled, the code will call the STOPRX task and wait in loop for one of the RXTO or ERROR event flags to be set.
The check for ERROR event was added in
https://github.com/spanceac/zephyr/commit/e729e6db10763b87bc02e6508b4aef9d76945f61 with the assumption that if an ERROR event is present, an RXTO event might not come.
But an RXTO event triggered by the STOPRX task still might come after the initialization phase and the RXTO ISR will be triggered. If the RXTO interrupts triggers, depending on its timing, it can cause problems.
For example, if an UARTE API user registered an UARTE events handler callback and enabled RX, it will receive invalid UART_RX_BUF_RELEASED and UART_RX_DISABLED events.
This commit attempts to fix this issue by adding a flag to determine if the RXTO interrupt is expected and decide whether to handle it or not.

I think that we got it wrong in the first place (I mean https://github.com/spanceac/zephyr/commit/e729e6db10763b87bc02e6508b4aef9d76945f61). According to the documentation RXTO will always come event after ERROR occurred and in all those years we never seen a case where it didn't. It is more likely that we falsely assumed that if RXSTARTED event is set during the initialization it means that RX is active but it maybe that RXSTARTED  and ERROR events are not handled by the bootlooder driver but RXTO is.
It would be better to rework initialization and trigger STOPRX and wait for RXTO with timeout and if it timeouts we can assume that RX is stopped.

@nordic-krch I think we can assume that the bootloader might do unexpected things, like triggering the STOPRX task without clearing RXSTARTED flag and/or ERROR flag. When the initialization process starts we can assume that the UART RX is in an unknown state regardless of the RXSTARTED flag. Since we don't know the real state, I think the safest way is to try to stop RX. But I don't know if an RXTO event is still generated if UART is already stopped and we trigger the STOPRX task. I will do a test and see

I have checked, and if I call the STOPRX task and UART is already stopped, the RXTO never comes. So we need to trigger the STOPRX task and wait for RXTO with a timeout. From my observations of how long it takes for RXTO to come and reading the documentation, it seems that the timeout should be 4x the time it takes to receive a byte at the current baudrate. So the code should read the baudrate register and derive the timeout from there. I can do it like this if we agree upon the idea

I created an alternative solution which simply removes the potential STOPRX task from the init of the UARTE peripheral. It is not up to the application to do cleanup from a failing bootloader, the bootloader must leave the UARTE in an idle state with events cleared.

@bjarki-andreasen @anangl ping?

IMO this should be closed as #75703 solves that issue.

Closing because alternative solution https://github.com/zephyrproject-rtos/zephyr/pull/75703 was merged


gtest submodule won't initialize correctly

The one in the main arrayfire repo appears to work fine though
Submodule 'arrayfire' (https://github.com/arrayfire/arrayfire) registered for path 'arrayfire'
Cloning into 'arrayfire'...
remote: Counting objects: 23410, done.
remote: Compressing objects: 100% (46/46), done.
remote: Total 23410 (delta 11), reused 0 (delta 0), pack-reused 23364
Receiving objects: 100% (23410/23410), 5.23 MiB | 2.23 MiB/s, done.
Resolving deltas: 100% (16296/16296), done.
Checking connectivity... done.
Submodule path 'arrayfire': checked out '86426db07af709f8880b4e86a34eb354e9ceedac'
Submodule 'assets' (https://github.com/arrayfire/assets) registered for path 'assets'
Submodule 'test/data' (https://github.com/arrayfire/arrayfire_data) registered for path 'test/data'
Submodule 'test/gtest' (http://git.chromium.org/external/googletest.git) registered for path 'test/gtest'
Cloning into 'assets'...
remote: Counting objects: 106, done.
remote: Compressing objects: 100% (5/5), done.
remote: Total 106 (delta 8), reused 4 (delta 4), pack-reused 97
Receiving objects: 100% (106/106), 15.53 MiB | 4.27 MiB/s, done.
Resolving deltas: 100% (15/15), done.
Checking connectivity... done.
Submodule path 'arrayfire/assets': checked out '77ba8d3c8dc399c49da7413774b71985ce989a09'
Cloning into 'test/data'...
remote: Counting objects: 671, done.
remote: Compressing objects: 100% (9/9), done.
remote: Total 671 (delta 0), reused 0 (delta 0), pack-reused 662
Receiving objects: 100% (671/671), 10.05 MiB | 4.29 MiB/s, done.
Resolving deltas: 100% (272/272), done.
Checking connectivity... done.
Submodule path 'arrayfire/test/data': checked out '8b4a85eb89118028b4da524f0d734969baa4b0f2'
Cloning into 'test/gtest'...
fatal: http://git.chromium.org/external/googletest.git/info/refs not valid: is this a git repository?
Clone of 'http://git.chromium.org/external/googletest.git' into submodule path 'test/gtest' failed
Failed to recurse into submodule path 'arrayfire'
[kelvin@batman007 arrayfire_rust]$ git submodule update --init --recursive
Cloning into 'test/gtest'...
fatal: http://git.chromium.org/external/googletest.git/info/refs not valid: is this a git repository?
Clone of 'http://git.chromium.org/external/googletest.git' into submodule path 'test/gtest' failed
Failed to recurse into submodule path 'arrayfire'

This is now fixed. The link for the google test submodule had to be updated in arrayfire. Updated the submodule tag to fix it here.


Patch arm compiler

Adds on to the install_arduino.sh script to auto-install the correct compiler if you're running this on aarch64 (the robots).

Tested on Infantry and Sentry, works.


Problem of connection with my web browser

Dear Shinyproxy developers,
I have developped a shiny application and I want to use your great tool Shinyproxy to easily manage users and groups. But maybe there is one thing that I misunderstood, but I cannot access the application on my web browser.
Here my application.yml file :

shiny:
proxy:
title: Open Analytics Shiny Proxy
logo-url: http://www.openanalytics.eu/sites/www.openanalytics.eu/themes/oa/logo.png
landing-page: /
heartbeat-rate: 10000
heartbeat-timeout: 60000
port: 8085
authentication: none
admin-groups: admin
# Example: 'simple' authentication configuration
users:
- name: avelt
password: test
groups: admin
# Example: 'ldap' authentication configuration
ldap:
url: ldap://ldap.forumsys.com:389/dc=example,dc=com
user-dn-pattern: uid={0}
group-search-base:
group-search-filter: (uniqueMember={0})
manager-dn: cn=read-only-admin,dc=example,dc=com
manager-password: password
# Docker configuration
docker:
cert-path: /home/none
url: http://<IP_ADDRESS>:2375
port-range-start: 20000
apps:

name: 01_hello
display-name: Hello Application
description: Application which demonstrates the basics of a Shiny app
docker-cmd: ["R", "-e", "shinyproxy::run_01_hello()"]
docker-image: openanalytics/shinyproxy-demo
groups: admin

logging:
file:
shinyproxy.log

For now, I don't use authentication with LDAP, I prefer to walk things one by one. I work on a server whose address is of type : http: //<IP_ADDRESS>: 2375. I use the 2375 port, which seems to be the default port of docker.
I have built your image in my docker.
sudo docker images :

REPOSITORY                      TAG                 IMAGE ID            CREATED             VIRTUAL SIZE
greatshinyapp                   latest              debe57f07ec7        17 hours ago        2.172 GB
hello-world                     latest              690d80202531        9 days ago          1.848 kB
openanalytics/shinyproxy-demo   latest              59a414c9b6e5        11 days ago         828 MB
thinkr/rfull                    latest              26f8217e95a3        8 months ago        2.018 GB

I chose port 8085 for shinyproxy because my 8080 port is already used.
Now, I launched :
java -jar /path/to/shinyproxy-1.0.2.jar
All seem to work fine, I have the following lines :

...
2017-11-30 10:18:19.081  INFO 46695 --- [           main] o.s.w.s.handler.SimpleUrlHandlerMapping  : Mapped URL path [/**/favicon.ico] onto handler of type [class org.springframework.web.servlet.resource.ResourceHttpRequestHandler]
2017-11-30 10:18:19.918  INFO 46695 --- [           main] o.s.l.c.support.AbstractContextSource    : Property 'userDn' not set - anonymous context will be used for read-write operations
2017-11-30 10:18:20.203  INFO 46695 --- [           main] o.s.j.e.a.AnnotationMBeanExporter        : Registering beans for JMX exposure on startup
2017-11-30 10:18:20.390  INFO 46695 --- [           main] b.c.e.u.UndertowEmbeddedServletContainer : Undertow started on port(s) 8085 (http)

Ok, I open my web browser and I type :
http://<IP_ADDRESS>:8085/
But I have "ERR_CONNECTION_REFUSED" in Google chrome and "The connection failed" in Firefox. I don't understand the problem. I use shiny server on my port 3838 and when I type :
http://<IP_ADDRESS>:3838/greatshinyapp
I can access to my application.
Can you see the problem ? I used the false address ?
Thank you.
Best,
Amandine

Hi @a-velt, quick sanity check: did you open port 8085 in your firewall?

Hi,
Yes, I typed :
iptables -A INPUT -m state --state NEW -p tcp --dport 8085 -j ACCEPT
And then :
java -jar /path/to/shinyproxy-1.0.2.jar
Same message :

2017-12-01 09:08:35.431  INFO 8483 --- [           main] o.s.l.c.support.AbstractContextSource    : Property 'userDn' not set - anonymous context will be used for read-write operations
2017-12-01 09:08:35.599  INFO 8483 --- [           main] o.s.j.e.a.AnnotationMBeanExporter        : Registering beans for JMX exposure on startup
2017-12-01 09:08:35.672  INFO 8483 --- [           main] b.c.e.u.UndertowEmbeddedServletContainer : Undertow started on port(s) 8085 (http)
2017-12-01 09:08:35.675  INFO 8483 --- [           main] eu.openanalytics.ShinyProxyApplication   : Started ShinyProxyApplication in 4.978 seconds (JVM running for 5.403)

But when I go on  :
http://<IP_ADDRESS>:8085/
I have the error "The connection failed" ...

Hi @a-velt ,
I strongly suspect this is a network or firewall problem.
Does it work if you go on the server and use http://localhost:8085/ ?

Ok, you're right, I stopped my shiny-server and start shinyproxy on 3838 port, and it's ok, I have the shinyproxy interface, but the page is blank, your application does not display anything, is it normal? in any case I have no errors. Thank you for your help !

If authentication is set to none, you should see all apps listed.
In other cases, you could indeed see an empty list, if the authorization of the apps (i.e. the groups attribute) does not match any group memberships of the logged-in user.


Taskbar options first column hovering

i removed some items from the taskbar menu, and the folders option is hovering:

how can i make it so it's at bottom and not so high up?
here is the full taskbar menu:

i'm going to remove File Explorer and Goto because i don't need them.

huh, that fixed it. idk then


Atom change the theme on his own

Description
I just updated my atom software and whatever I do to change the theme, it change back to Atom Dark UI/Syntax theme instead of One Dark.
Steps to Reproduce

open atom with the last version
change the theme to One Dark
wait a few seconds

Expected behavior: Keep the theme I want
Actual behavior: Change the theme on his own
Reproduces how often: I tried to change 5 times and failed 5 times
Versions
Atom    : 1.24.1
Electron: 1.6.16
Chrome  : 56.0.2924.87
Node    : 7.4.0
apm  : 1.18.12
npm  : 3.10.10
OS : Windows 10 Familly 1709
Additional Information
Happened just after I updated on 1.24.1

Thanks for the report - I was unable to reproduce with Atom 1.24.1 on Windows 10.  It looks like you removed the Prerequisites section, are you able to reproduce in safe mode (atom --safe) or have you tried any of the other debugging advice?

Indeed, I remove the Prerequisites section because I didn't have the time when a post the report. But yes I tried it in safe mode and the problem is the same. I'll give you more details if I can when I'll be back at my place.

Okay so after I renamed my .atom to .atom-backup (as written in debugging section) it seems to work the good way. It's weird x) I'll install my config again step by step to see if there is something wrong and I'll come back to you if needed.

Thanks for reporting back!


[matplotplusplus] update to 2023-02-17

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[x] The "supports" clause reflects platforms that may be fixed by this new version
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

<!-- If this PR adds a new port, please uncomment and fill out this checklist:

- [ ] Changes comply with the [maintainer guide](https://github.com/microsoft/vcpkg-docs/blob/main/vcpkg/contributing/maintainer-guide.md)
- [ ] The name of the port matches an existing name for this component on https://repology.org/ if possible, and/or is strongly associated with that component on search engines.
- [ ] Optional dependencies are resolved in exactly one way. For example, if the component is built with CMake, all `find_package` calls are REQUIRED, are satisfied by `vcpkg.json`'s declared dependencies, or disabled with [CMAKE_DISABLE_FIND_PACKAGE_Xxx](https://cmake.org/cmake/help/latest/variable/CMAKE_DISABLE_FIND_PACKAGE_PackageName.html)
- [ ] The versioning scheme in `vcpkg.json` matches what upstream says.
- [ ] The license declaration in `vcpkg.json` matches what upstream says.
- [ ] The installed as the "copyright" file matches what upstream says.
- [ ] The source code of the component installed comes from an authoritative source.
- [ ] The generated "usage text" is accurate. See [adding-usage](https://github.com/microsoft/vcpkg-docs/blob/main/vcpkg/examples/adding-usage.md) for context.
- [ ] The version database is fixed by rerunning `./vcpkg x-add-version --all` and committing the result.
- [ ] Only one version is in the new port's versions file.
- [ ] Only one version is added to each modified port's versions file.

@aminya  Please add port-version in vcpkg.json.

Note: I will be converting your PR to draft status. When you respond, please revert to "ready for review". That way, I can be aware that you've responded since you can't modify the tags.

@aminya Please add port-version in vcpkg.json.

port-version is 0 here. So it is not needed! Actually, if I add it, vcpkg format-manifest will remove it.

May fix #26347.

https://github.com/microsoft/vcpkg/pull/31796#issuecomment-1576482876 Should also test feature on different platform.

@jiayuehua Do you want to upstream your OpenGL changes? The last time I asked, OpenGL was considered experimental. (Its installation works for me on Linux without issues).

As #31796 (comment) point out, This pr should also test feature on different platform.

I encountered the same problem when testing the feature again. After fixing this issue, please revert to "ready for review". Thanks.

@Adela0814 @dg0yt I merged all the patches upstream. Please approve this.

@Adela0814 @dg0yt I merged all the patches upstream. Please approve this. It seems vcpkg is falling behind the upstream.

I have rerun the failed CI triplet, and we need to wait for the CI to finish running.

@aminya  I encountered an error while testing the usage:
Usage:
    find_package(Matplot++ CONFIG REQUIRED)
    # note: 1 additional targets are not displayed.
    target_link_libraries(CMakeTest PRIVATE Matplot++::cimg std::filesystem Matplot++::matplot Matplot++::nodesoup)

Error:
CMake Error at E:/5/vcpkg/installed/x64-windows/share/matplot++/Matplot++Targets.cmake:68 (set_target_properties):
  The link interface of target "Matplot++::cimg" contains:

    ZLIB::ZLIB

  but the target was not found.  Possible reasons include:

    * There is a typo in the target name.
    * A find_package call is missing for an IMPORTED target.
    * An ALIAS target is missing.

This target is optional. @Adela0814

This target is optional. @Adela0814 You should enable its feature

I tested it with all features enabled.
.\vcpkg.exe install matplotplusplus[*]:x64-windows.

All feature test pass on the following triplets:

x64-windows
x64-windows-static
x86-windows

Usage test pass on x64-windows.

/azp run

@aminya  Please fix the conflicts, thanks.


French pages not updating with latest changes

Changes from #1948 (crc-covlib) have not been updated on the ORE Website when using the French URL (ouvert).
https://code.ouvert.canada.ca/en/open-source-codes.html
Is Github Pages still updating for the French Repo?  Or maybe just need to restart the build.
https://github.com/canada-ca/ore-ero-fr
The syncs are working and the update is in the code, but the French site is not updated.

I think it's related to the deprecation of the GitHub Pages build setup. Might need an explicit build action now. I'll take a look


Facing pairing issues for BCM test of Multi Fabrics testcases

Summary:
Facing Pairing issue for BCM tests
Steps to reproduce:

TH_CR1 starts a commissioning process with DUT_CE and DUT_CE is commissioned by TH_CR1
TH_CR1 opens a commissioning window on DUT_CE
TH_CR2 starts a commissioning process with DUT_CE and DUT_CE is commissioned by TH_CR2
if we try to open commissioning process with DUT_CE for 3rd Commissioner (TH_CR3) it is showing FAILURE response
Effected test TC-MF-1.18
Testplan screenshot:

Execution logs:

exe_log.txt

Tested issue has been resolved


Dropdown unit tests should cover the actions of the triggerButton

Feature Request
Problem description
We need unit test for the Dropdown that will check the correct behavior of the triggerButton (clicking on it should open the Dropdown). There was an issue in a recent PR https://github.com/stardust-ui/react/pull/1299 that broke this, but the unit tests didn't caught this.

    it('is "true" when opened by trigger button click', () => {
      const { getItems, clickOnTriggerButton } = renderDropdown()

      clickOnTriggerButton()

      expect(getItems()).toHaveLength(items.length)
    })

    it('is "false" when closed by trigger button click', () => {
      const { clickOnTriggerButton, getItems } = renderDropdown({
        defaultOpen: true,
      })

      clickOnTriggerButton()

      expect(getItems()).toHaveLength(0)
    })


Small path error

In the  Guide - Run a CKB Dev Blockchain seciton 4 Change the PoW Function to Eaglesong, there is a line:

Replace the miner.workers section in the specs/dev.toml file with the following:

The specs/dev.toml here probably should be ckb-miner.toml

Good catch. Indeed, you have to change both ckb-miner.toml and specs/dev.toml.

Could you please send a PR to fix this?

Could you please send a PR to fix this?

The PR is here: #118
But I am not sure about it. There is no miner.workers section in the efault specs/dev.toml file generated by the ckb init command.
Any suggesstion for the PR? Thanks!


Remove "re" filetype from Regex syntax

Since Atom doesn't seem to have a way to manage priorities, I'm suggesting to remove the filetype here since it conflicts with Facebook's Reason language and I don't think people actually write files just containing a regex.
Would fix facebook/reason#234.

This would be much appreciated, as long as people don't actually currently use the .re suffix.

No requirement for re file types. Just pushed 2.24.0 with this change.

Thank you @gandm !

Thanks so much!


Max Character

Directions
Given a string, return the character that is most commonly used in the string. Make use of functions and methods where necessory
You can code this with any language of you choice!
Examples
maxChar("abccccccd") === "c"
maxChar("apple 1231111") === "1"

please assign me

Assing me @xaster-Kies

Done @BertinAm

Thanks @xaster-Kies

please assign me

Done @ngigordon

November/Day 8/Python/Max_Character.py

e6ccd04fe0053dd9de789f67ab0c888ae3af80fe

thats my pull request for the max character


Config Routes for DiscoveryClient routing

It is more secure to config a list like spring zuul that only matcher can route to the backend service.  Is it any plan to do this or  user have to write a custom Filter?
zuul:
  routes:
    xxx-service: /xxx-service/**

I'm afraid I don't quite understand your question. Can you explain some more?

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.