feat: add asciidoc

closes #161
Ascii-Doc is available for js with npm now.
No need for the ruby-version any more.

[x] initialize ascii-doc
[x] create Action

In the stargate-repo is a action for generating the docs. The Docs are getting hosted on GH-Pages then. Do we have to activate something for usage of this feature within this repo?

Ascii-Doc is available for js with npm now.

oh nice 🎉

The Docs are getting hosted on GH-Pages

I can't remember 🤦♂ . I think it pushes to the gh-pages branch, then we might have to turn it on in settings

Gh-Pages needs to be enabled in the settings. Unfortunaltey I cant do it, so you have to enable it. I created a branch gh-pages which should be the base for the page I guess.

I think if Gh-Pages is enabled its mergeable

I enabled GH-Pages for the repo. In my opinion everything is set up and can be tested

Looks great 👍

I think the README is getting deployed, not the docs folder?

Ignore me, I think it is because the Action has not run yet 🤦♂

did I do something wrong? Seems like the action didnt run :(

oh it looks fine to me 🤔

Is running actions from a PR still possible though, because of the crypto-mining issue Github had?

yep it should still be possible, restrictions are usually on new contributors - I just tried to make a change to the docs folder and still no trigger 🤔

we should remember setting the branch back to main before merging

Everything looks fine, I cannot see why the Action is not running 🤷♂

Now something ran but seems like failing

Seems like it worked

It deployed 🎉 - I will change the trigger back

Thats actually an awesome feature 💥

For some reason it is not triggering on paths - let's merge and see if it solves the issue, otherwise we can have it run for any change on main, not as efficient but will still be fine


Making changes to make it compatible with cassandra-driver > 2.4

Few changes with the latest cassandra-driver.

Thanks for the pull request!


fetch-mock 12.0 + node:test issues

node v22.5.1
npm 10.8.2
fetch-mock 12.0.2
i was using fetch-mock v11 which was working great along with node:test, but i upgraded to v12 and after taking steps to migrate, my preexisting tests will now pass as long as i run one at a time via --test-only but will fail if i attempt to run any combination of multiple tests. any ideas what i am missing? am i leaving out some awaits or something? heres a copy of my test file. thanks.
import { strict as assert } from "node:assert";
import { describe, it } from "node:test";
import fetchMock from "fetch-mock";
import { FETCH_TIMEOUT_MS } from "../errors/TimeoutError";
import { fetcher, fetcherResult, fetcherTimeout } from "./core";

describe("core.ts", async () => {
  const FAKE_URL = "https://www.fake.com";

  describe("fetcher()", async () => {
    it("adds common properties to body", async () => {
      fetchMock.mockGlobal().route(FAKE_URL, 200);
      await fetcher(FAKE_URL, { body: {} });
      const lastCallArg = fetchMock.callHistory.lastCall()?.args[1] as {
        method: string;
        body: string;
      };
      const method = lastCallArg.method;
      const body = JSON.parse(lastCallArg.body);
      const ver = body.ver;
      assert.equal(method, "POST");
      assert.equal(ver, 1);
      fetchMock.unmockGlobal();
    });
  });

  describe("fetcherTimeout()", async () => {
    it("allows a quick endpoint to funciton unimpeded", async () => {
      const options = { body: { rand: Math.random() } };
      fetchMock
        .mockGlobal()
        .route(FAKE_URL, options, { delay: FETCH_TIMEOUT_MS * 0.1 });
      const response = await fetcherTimeout<Response>(FAKE_URL, options);
      const {
        body: { rand },
      } = (await fetcherResult<Response>(options, response)) as any;
      assert.equal(rand, options.body.rand);
      fetchMock.unmockGlobal();
    });
  });

  describe("fetcherResult()", async () => {
    it("returns JSON object", async () => {
      const options = { body: { foo: "bar" } };
      fetchMock.mockGlobal().route(FAKE_URL, { options });
      const response = await fetcher(FAKE_URL, options);
      const json = (await fetcherResult(options, response as Response)) as any;
      assert.deepEqual(json.options, options);
      fetchMock.unmockGlobal();
    });
  });
});

It makes sense that they'd only work one at a time as each one mocks and adds different routes to fetch independently; if they run in parallel then they are bound to interfere as they are all acting on a single fetch implementation.
Are you able to share your code from before the upgrade? I'm confused how they could have passed TBH

Closing due to inactivity

sorry about the delay. here is that same file from when i was using fetch-mock v11.1.5, and all tests can run and succeed.
import { strict as assert } from "node:assert";
import { describe, it } from "node:test";
import fetchMock from "fetch-mock";
import { FETCH_TIMEOUT_MS } from "../errors/TimeoutError";
import { fetcher, fetcherResult, fetcherTimeout } from "./core";

describe("core.ts", async () => {
  const FAKE_URL = "https://www.fake.com";

  describe("fetcher()", async () => {
    it("adds common properties to body", async () => {
      fetchMock.mock(FAKE_URL, { status: 200 });
      await fetcher(FAKE_URL, { body: {} });
      const lastCallArg = fetchMock.lastCall()?.[1] as any;
      const method = lastCallArg?.method;
      const body = JSON.parse(lastCallArg?.body);
      const ver = body.ver;
      assert.equal(method, "POST");
      assert.equal(ver, 1);
      fetchMock.restore();
    });
  });

  describe("fetcherTimeout()", async () => {
    it("allows a quick endpoint to funciton unimpeded", async () => {
      const options = { body: { rand: Math.random() } };
      fetchMock.mock(
        { url: FAKE_URL, delay: FETCH_TIMEOUT_MS * 0.5 },
        { status: 200, options },
      );
      const response = await fetcherTimeout<Response>(FAKE_URL, options);
      const json = await fetcherResult<Response>(options, response);
      assert.equal(json.options.body.rand, options.body.rand);
      fetchMock.restore();
    });
  });

  describe("fetcherResult()", async () => {
    it("returns JSON object", async () => {
      const options = { body: { foo: "bar" } };
      fetchMock.mock(FAKE_URL, { status: 200, options });
      const response = await fetcher(FAKE_URL, options);
      const json = (await fetcherResult(options, response as Response)) as any;
      assert.deepEqual(json.options, options);
      fetchMock.restore();
    });
  });
});

I see what's happening now. .restore() did a lot more than unmockGlobal().
I've been thinking of adding an equivalent method to restore() but didn't want to use the word "restore" as over the years different testing libraries have meant different things by "restore", so I don't want to confuse users.
I've gone with hardReset() which is now merged to main, but I'm having some issues with the build pipeline so releasing is a bit delayed

Now released as 12.2.0

thank you i appreciate it. is there something i should be doing differently to run these tests more "normal" for however you have written this library?

i have updated to 12.2.0, and replaced all instances of fetchMock.restore() with fetchMock.hardReset() and that allows the test file to run. however, in reality, i have multiple it statements per describe, and i have the same breaking problem as soon as there are multiple it statements. any ideas what im doing wrong?


Request: Unlimited Strength Security JRE

Hey @serge-rider not sure if this is already the case or not but can you include the unlimited strength security policy in dbeaver for the included windows JRE.
http://www.oracle.com/technetwork/java/javase/downloads/jce8-download-2133166.html
I'm running into an issue and my understanding is that because of our use of aes-256 java chokes as the default security policy doesn't include support for that.

JRE included in installer is already configured with unlimited security strength.
You can check C:\Program Files\DBeaver\jre\lib\security\local_policy.jar file, it contains default_local.policy file with the following contents:
// Country-specific policy file for countries with no limits on crypto strength.
grant {
    // There is no restriction to any algorithms.
    permission javax.crypto.CryptoAllPermission;
};


Better selecting

When you have a single selection, like a race or class, clicking a tile immediately selects it
When you have multi-selections, like a spell list, clicking a tile expands it and clicking it again selects it. Then clicking it a third time collapses it.

nooo

why did it include all the commits


Is it possible to set property of Text components like numberOfLines, ellipsizeMode  to Highlighter

Hi guys,
Thank you for simple and very useful Highlighter component.
Is it possible to set property of Text components like numberOfLines, ellipsizeMode  to Highlighter ?
I've added above properties to style but its not working.

Hey @dongawli,
You should be able to add these properties directly on Highlighter and they will be passed down to the wrapper <Text> node


Add server requirements in readme

Included in https://github.com/Frojd/Wagtail-Boilerplate/commit/d8d1c7ed8ba743a5853b7a43566558bda8cca83c


Move the module.exports up

Like everywhere else in the code.
The point is to know what the file exports without having to scroll down the entire length of the file.

I did it but it reduces consistency because it cannot be done for variables / classes that are exposed.
So you end up with some modules exposing at the top and some at the bottom.

Okay, I'll review everything tomorrow and find a solution for what you said... I'll merge the maximum now.


Bring up to date with json 5

see https://github.com/Vulnogram/Vulnogram/pull/48

As well as support for v5 format (required for future automation and pushes) it also changes a lot of other things.

test version https://vulnogram.github.io/cve5/#editor


[Question] Client Credentials Grant with scope

Hi folks,
I'm using the package as the "Client Credentials Grant" example from README.md and I'm trying to inform a specific scope called cbnpagamento_vendas, appearently without success, example:
use League\OAuth2\Client\Provider\GenericProvider;

$options = [
    'clientId' => 'xxx',
    'clientSecret' => 'xxx',
    'urlAuthorize' => 'https://apigw-h.bndes.gov.br/token',
    'urlAccessToken' => 'https://apigw-h.bndes.gov.br/token',
    'urlResourceOwnerDetails' => 'https://apigw-h.bndes.gov.br/cbn-fornecedor',
];

$prov = new GenericProvider($options);
$token = $prov->getAccessToken('client_credentials', ['scope' => ['cbnpagamento_vendas']]);
var_dump($token);

And I get:
class League\OAuth2\Client\Token\AccessToken#32 (5) {
  protected $accessToken =>
  string(36) "7b27b500-89a2-36b8-8f9c-xxx"
  protected $expires =>
  int(1551893968)
  protected $refreshToken =>
  NULL
  protected $resourceOwnerId =>
  NULL
  protected $values =>
  array(2) {
    'scope' =>
    string(28) "am_application_scope default"       <== HERE!
    'token_type' =>
    string(6) "Bearer"
  }
}

If I request the token without specifing the second parameter ['scope' => ['cbnpagamento_vendas']] I get the same result (same scopes "am_application_scope default"), so it seems to me that parameter doesn't make any difference.
What am I doing wrong?
Thank you!

I've changed my code to inform the scopes key on the $options param when instantiating the provider, but nothing has changed, I still get the same scopes, ie:
$options = [
    'clientId' => 'jnKh1MAlfMZHqfpmdzGRrHtIAe0a',
    'clientSecret' => 'wC3rwel0nSjTMNmOfScynRHK7Rka',
    'urlAuthorize' => 'https://apigw-h.bndes.gov.br/token',
    'urlAccessToken' => 'https://apigw-h.bndes.gov.br/token',
    'urlResourceOwnerDetails' => 'https://apigw-h.bndes.gov.br/cbn-fornecedor',
    'scopes' => ['cbnpagamento_vendas'],
];

$provider = new GenericProvider($options);
$token = $provider->getAccessToken('client_credentials');
var_dump($token);

class League\OAuth2\Client\Token\AccessToken#32 (5) {
  protected $accessToken =>
  string(36) "4455c031-9b5c-3af5-af42-5b60cd74c7c1"
  protected $expires =>
  int(1551898135)
  protected $refreshToken =>
  NULL
  protected $resourceOwnerId =>
  NULL
  protected $values =>
  array(2) {
    'scope' =>
    string(28) "am_application_scope default"
    'token_type' =>
    string(6) "Bearer"
  }
}

I wonder, is there any way I could see the built URL, or a log of the request, like cURL...?

It looks like you’re doing everything correctly and the provider is returning the scope string you see. I’m not sure if we expose a way to access the response you receive, but the easiest way might be to set up a proxy like Charles and follow the instructions here to see the requests/responses: https://github.com/thephpleague/oauth2-client/blob/master/README.md#using-a-proxy

If you’re not familiar with Charles, you can get it here: https://www.charlesproxy.com

Hi there, thank you very much I'll try that.

I tryed to do that to use Uber API, but doesn't work because getAuthorizationUrl method calls getAuthorizationParameters method that do an implode whit the scope to send in the request.
This:
if (is_array($options['scope'])) {
     $separator = $this->getScopeSeparator();
     $options['scope'] = implode($separator, $options['scope']);
}

And the getAccessToken method doesn't do that because don't expect scopes.
I've only been able to do it right by setting the scope to string.
And then sending together:
$options = [
     'scope' => 'scope_1 scope_2 scope_3'
];
$provider->getAccessToken('client_credentials', $options);

Hi folks, I remember I ended up implementing by myself using curl() instead of trying to figure out what was happening inside the oauth2-client code. I'll try to review/resume my code, I suspect maybe there's a bug here, related to scopes?

I don't know if it's a bug, but needs an implementation to treat the scope when use it on getAccessToken method.

A possible solution is implement the scope on file src/Provider/AbstractProvider.php.
Before:
`public function getAccessToken($grant, array $options = [])
{
$grant = $this->verifyGrant($grant);
    $params = [
        'client_id'     => $this->clientId,
        'client_secret' => $this->clientSecret,
        'redirect_uri'  => $this->redirectUri
    ];`

After:
`public function getAccessToken($grant, array $options = [])
{
$grant = $this->verifyGrant($grant);
    $params = [
        'client_id'     => $this->clientId,
        'client_secret' => $this->clientSecret,
        'redirect_uri'  => $this->redirectUri,
        'scope' => $this->getDefaultScopes()
    ];`

@bampnet you should probably check that it's not already set.
        if (!isset($options['scope'])) {
          $options['scope'] = $this->getDefaultScopes();
        }

Better yet, this formats everything correctly.   $options = $this->getAuthorizationParameters($options);
Full method, which should resolve the issue @ezandonai has with needing to set the scope to be a string.
    public function getAccessToken($grant, array $options = [])
    {
        $grant = $this->verifyGrant($grant);

        $params = [
            'client_id'     => $this->clientId,
            'client_secret' => $this->clientSecret,
            'redirect_uri'  => $this->redirectUri,
        ];

        $options = $this->getAuthorizationParameters($options);
        $params   = $grant->prepareRequestParameters($params, $options);
        $request  = $this->getAccessTokenRequest($params);
        $response = $this->getParsedResponse($request);
        if (false === is_array($response)) {
            throw new UnexpectedValueException(
                'Invalid response received from Authorization Server. Expected JSON.'
            );
        }
        $prepared = $this->prepareAccessTokenResponse($response);
        $token    = $this->createAccessToken($prepared, $grant);

        return $token;
    }

All providers are different, and the GenericProvider may not work for all of them. It works for many cases, but not every provider follows the OAuth 2 spec. This is why we made league/oauth2-client extensible, so you can extend AbstractProvider to modify things according to the needs of the provider you're trying to use.
@ezandonai, you should check out stevenmaguire/oauth2-uber. It's a custom provider for Uber that handles the issues you described with the scope.
@imclean557, if scopes is passed in through the $options parameter, there's no need to also call $this->getAuthorizationParameters($options). This will have the result of adding additional query string parameters to the request that should only used for the authorization request. When the $options are prepared and sent to getAccessTokenRequest(), they are already added to the query string (if it is a GET request).
The issue @fmertins reported originally is that the scope am_application_scope default was set on the AccessToken object no matter what scopes he provided when calling getAccessToken(). The only way league/oauth2-client could do this is by setting the value we received from the OAuth 2 server. I don't believe there is a bug in the oauth2-client code that caused this. I think the problem was on the server's side. Perhaps it always sets am_application_scope default, if it receives no scope or a scope it doesn't understand, so perhaps it didn't understand cbnpagamento_vendas, or maybe this was misspelled.
These are just guesses, but I don't believe @fmertins's issue is related to the issue @ezandonai encountered with the Uber OAuth 2 provider or the problem that @imclean557 is trying to solve. Maybe I'm wrong?
I'll close this issue for now. If it is still a problem, please open a new issue, and we'll try to get more details.

Thanks for the information @ramsey. I'll need to look into it again more carefully, but from memory the problem was there was no way to pass the scope to getAccessToken() using HttpBasicAuthOptionProvider and client credentials grant.  Our server requires a scope when using basic auth.

@imclean557 I believe the $params passed to OptionProviderInterface::getAccessTokenOptions() has the scope in it, so you should be able to create your own option provider that implements OptionProviderInterface and call setOptionProvider() on the provider object to set it. You could probably achieve what you need by extending HttpBasicAuthOptionProvider and overriding getAccessTokenOptions(), adding the scope to the headers, as needed.

@ramsey,

You could probably achieve what you need by extending HttpBasicAuthOptionProvider and overriding getAccessTokenOptions(), adding the scope to the headers, as needed.

This looks like the solution in this case. I was trying to use the oauth2-client as "off the shelf" as possible but extending one class shouldn't be a problem.  Thanks.

While we provide the GenericProvider for convenience, oauth2-client isn't really intended to be used "off the shelf." Everyone's OAuth implementation is just a little different, as you've found. 😉


Creating card with Label

Hi
This extension is really what I need...except I can't figure how to add a label...
The RADME tells :

You can add a card with a Label by entering Label Name:Text.

But it never works...
I tried to find a way by editing the .kanbanfile but without success...
What is wrong ?
thanks in advance

Hi
I didn't see the "+" sign to add a tag.
Sorry.


Errors were encountered while processing: /var/cache/apt/archives/python2.7-minimal_2.7.9-2_amd64.deb

env

ubuntu 15.04 vivid 64bit
java 8
mesos 0.26
marathon=1.1.0-1.0.471.ubuntu1504

problem
I use the repo provided by mesosphere (deb http://repos.mesosphere.io/ubuntu/ vivid main).I run into this error when  installing marathon:
Errors were encountered while processing:
/var/cache/apt/archives/python2.7-minimal_2.7.9-2_amd64.deb
E: Sub-process /usr/bin/dpkg returned an error code (1)

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-4542. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.

@meichstedt  Thanks!


overload error

Hi! Thank you very much for your code, it helps me a lot.
However I have met the error: "Stim 300 IMU overload."
I have read the datasheet of the sensor, it said "If time between triggers is longer than 8ms, an overload may occur..." But I have checked that the sample frequence is 125hz. I have even tried to configurate the frequence to 250hz by the stim's software in windows, but it still doesn't work. I have also checked that the default configuration in code is same with the actual configuration, which is also setted up using the software in windows...
And the sensor works well in windows with the software.
Do you have any idea how to solve this problem?
Thank you very much!

Hi, it’s been a while since I have worked with the IMU, but I will try to help you out. First of I can’t remember running in to this before and don’t know if this error have been properly tested so there is a chance there is just a bug in the driver.
What the driver does is that just accumulate all the status bytes from the different sensors on the IMU into one and then check that status byte with respect to the table in section 6.4 in the datashet.
Assuming the driver is working as intended this means the IMU set the overflow bit in the status byte for one of the sensors.
The sentence from the datasheet you refer to is regarding the external triggering option, meaning an input pulse signal from the computer tells the IMU exactly when to perform measurements. So if the triggering is to sparse, accumulated gyro measurements within the IMU will overflow. I assume you are not operating in external triggering mode and instead just configure the imu to sample at 125 hz or 250 hz. Then that should not be a problem.
However I believe the overflow error can also be caused in normal operation (I.e not in external triggering mode), as I think it is raised whenever the accelerometer sensors measure above 10% of their specified range. I.e if you have a 5g version and shake the IMU really hard so it measures a acceleration of 5.5g I believe this will cause this error.
If this error occurs in every measurement even when the IMU is standing still, you could try to use minicom or some other serial monitor to do a single shot measurement like in Figure 10-20: “Example of response from a (SINGLE-SHOT MEASUREMENT) command with error flagging” from the datasheet. To see if the error is there as well.

Hi, thank you very much for your reply, it helps me a lot.
At first I thought it was something wrong in the driver, where I didn't configure correctly, that caused this problem. After reading your reply, I checked the Manufacturer supplied software in windows again, and have found out that the OVERLOAD problem also happens, which I didn't realize before.
After that I have been communicating with the support from the manufacture and found out that this is a hardware issue, I plan to return this unit back for further troubleshooting.
Thank you very much again for your help!

Hi, thank you for the update. Glad I could be of some help and hope the IMU will be fixed. I'll close this issue.


1318 Button behavior pre and post-Lollipop

This fixes the original bug report about Color None buttons displaying as default.
It also removes the Alpha changes on the Touch event for Lollipop and higher. This allows Material UI buttons to retain their ripple effect, and Classic buttons seem to do the correct thing on press as well (they don't pre-Lollipop, thus the Alpha changes to begin with).
After examining the LollipopUtil class, I'm not clear whether moving the changes Touch event into the util class is the right way to organize, since this is additional code that needs to be run for OLDER versions.
I'm submitting as is, and we can discuss.
Fixes #1318

Issues should be addressed. This is ready for review again.

I believe issues are resolved. Ready for review again.

Having thought about it a bit more, I'm not opposed to taking this as is. If people really want the rectangular style they can change the shape from default to rectangle, which is effectively what happens when a color other than COLOR_DEFAULT is used. The other benefit is that changing the color doesn't change the effective width/height of the button, which seems all-around bad semantics.

@jisqyv @halatmit Do either of you have any strong opinions one way or the other?

For the record, we decided that we were okay with the visual change since it is already in production, and that if people have trouble with the behavior they can switch to using the rectangle shape, which is the same as the old behavior.


Error: java.io.IOException - Expected a SETTINGS frame but was WINDOW_UPDATE during commit message generation

First of all, thank you for developing the AICommit plugin! It's been a really helpful tool, but I’ve recently encountered an issue.
When I try to generate a commit message, I get the following error:
java.lang.RuntimeException: java.io.IOException: Expected a SETTINGS frame but was WINDOW_UPDATE
I'm using GoLand 2024.2 on Ubuntu 22.04.
Interestingly, it works fine on Windows 10, but not on Ubuntu.

Hey @Trileon12, I'm glad you find it useful :smiley: . Could you please add the whole stack trace? Thanks :pray: .

Hey  @Blarc
Thanks for the quick reply! I'm not entirely sure what you mean by "stack trace" .  I'm not seeing any events in the Tail log when the error happens.
I’ve found a few more details while testing it out:

Right after I reload the IDE, the first commit message usually generates without any issues.
After that, the error starts popping up more consistently, but oddly enough, sometimes it works again even without a full IDE restart—though that’s less common.

Where do you see the error that you posted?

Huh, it seems this error has something to do with http2. A quick google suggests that the problem might be in the size of the data sent. Could you maybe test if this error only appears if the diff is really big?
Furthermore, can you please tell me which LLM client are you using? This might be a bug in langchain4j.

I think this error definitely doesn't depend on the diff size. I tried using it with a small diff (just one line) and got the same error.
I use the gpt-4o model.

Could you maybe try with the development version of the plugin from GitHub actions: link.
The plugin should now re-throw the exception and the whole stack trace will be logged by the IDE. This way I'll have more information and be able to open an issue in langchain4j.
You can follow these instructions to install the plugin from zip.

tail_log.txt
I've attached a file with the log for one request.

I've posted a question about how to force http/1.1 on langchain4j and will try to add this option if possible.
Please update, if anything changes in the meantime @Trileon12.
Thank you :pray: .

According to this reply from langchain4j, it is currently not possible to force http/1.1.

Hey Blarc, thanks for keeping me updated! As I understand it, there's currently no way to force HTTP/1.1, but I'll be happy to test it when this feature becomes available. If anything changes on my side, I'll let you know.
Thanks a lot for your work and support!


Npm build command

Found this when setting up a new dev environment. The npm install needs to reference the dev dependencies

That's odd.. It should work the same whether --only=dev is there or not.
This project only has dev dependencies.
Did you get an error without the option present?

Yes, errored on colors not being installed. Node 10 Mac OS X nvm.
On Tue, 24 Sep 2019, 14:32 Bryan Housel<EMAIL_ADDRESS>wrote:

That's odd.. It should work the same whether --only=dev is there or not.
This project only has dev dependencies.
Did you get an error without the option present?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/osmlab/osm-community-index/pull/272?email_source=notifications&email_token=AAAUKM5DSQM2RCSQCIWBILLQLIJFNA5CNFSM4IZ6B2GKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7OLNRI#issuecomment-534558405,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAUKM5HRLB4ZKX63VDB5LLQLIJFNANCNFSM4IZ6B2GA
.

Hmm... is your $NODE_ENV environment variable set to production ?
That's the only reason that I would think that npm install would skip installing the dev dependencies.  It normally installs them.
You can check this with a command like:
echo $NODE_ENV
(It should probably return nothing / not be set)

my $NODE_ENV is blank.
It's not a biggy, so happy if you close the issue if you are sure. My node experience is not extensive. Perhaps it is the npm version that matters?

It's not a biggy, so happy if you close the issue if you are sure.

Ok, yeah I'm going to close this, but I'll keep it in the back of my mind.  npm install really should install all of the dependencies you need to work with the code..


[Improvement] Don't just use sysdefault when there's one card only

With reference to this post, when CARD_INDEX and CARD_NAME are not specified, but there is only one card available (need to understand how to count those cards), we should behave as if CARD_INDEX was set to 0.

After long thinking about this feature, I believe it would be wrong to implement it. After all, all that is required on the user side is to add a CARD_INDEX=0 to the .env file in order to force the one card instead of using sysdefault, which remains the legitimate choice (IMO) when the user does not express a preference on the output device.


Make Select.ready() an async iterable

If we do this, the usage can be much more idiomatic:
    receiver1 = some_chan.get_receiver()
    timer = channel.Timer(30.0)
    select = Select(receiver1=receiver1, timer=timer)
    async for selected in select.ready():
        match selected.origin:
            case receiver1:
                process_data(selected.value)
            case timer:
                # something to do once every 30 seconds
                pass
            case _:
                raise Exception("Unknown origin")

So .ready() should return an async iterator itself, by just yielding for each ready message.
With this we also don't need to keep a reference to the Select instance around, we can just do async for selected in Select(...).ready(), or we could just add an utility function to make it even more compact, as 99% of the cases nobody will need to even know Select is a class: async for selected in select(...). This also completely removes the temptation of adding the option to dynamically change the Select instance inside the loop (like in #24).
This also sort of depends on #27, as it could make breaking from the loop even more natural, and cause data loss in that case.
Originally posted by @leandro-lucarella-frequenz in https://github.com/frequenz-floss/frequenz-channels-python/issues/27#issuecomment-1303299010

This will be probably dropped because of the issues mentioned in:

https://github.com/frequenz-floss/frequenz-channels-python/issues/27#issuecomment-1319252644.

I was just reading about the Select implementation in go again,  and realized that when there are multiple receivers ready,  it picks one at random,  and doesn't allow us to prioritize.
Now that I've thought about it further,  I think that's how it should be,  and is the true async way.  So maybe we should revisit the ideas in #27.
I feel it might be useful to look into what other languages do as well.
Does that make sense?

I am not sure why would we want to not allow the user to prioritize what to process first if there are multiple receivers ready. :thinking:
If it is some sort of safety (prevent starvation), maybe that should be an option?
async for receiver in select.one_by_one():
    if receiver is x:
        pass

In any case (I mean if we went with a one receiver at the time) I would still prefer to offer an async iterator interface rather than the less pythonic while loop.
We could decide which one should be default, but removing options from the user doesn't seem like a good idea to me.
About looking at other languages, sure!

OK, now that I went though your new comments in the PR I have a better feeling for the starving issue and I think I'm starting to lean towards reviving #27 too. I created a new issue about that:

#72


is it possible to add $MaxOpenFiles

I think the default limit is about 1000 open connection. Is it possible to add the setting to list?
$MaxOpenFiles
Thanks,
Steve K.

Sure, it's possible. Just send me a PR and I'll be happy to merge it.


tests: add more filestore tests

Add tests exhibiting issues on filestore.
Ticket
If your pull request is related to a Suricata ticket, please provide
the full URL to the ticket here so this pull request can monitor
changes to the ticket status:
Redmine ticket:

https://redmine.openinfosecfoundation.org/issues/7356
https://redmine.openinfosecfoundation.org/issues/7357

Looks like min-version: 8 is missing


[WIP][Debug]Tfmodels Debug

This pull request is solely for determining where the strange Travis/linter error is in the Tfmodels export support pull request. I will close this when I determine where the error is.

Codecov Report

Merging #46 into master will decrease coverage by 0.14%.
The diff coverage is 16.66%.

@@            Coverage Diff             @@
##           master      #46      +/-   ##
==========================================
- Coverage   77.08%   76.94%   -0.15%
==========================================
  Files          44       45       +1
  Lines        2540     2546       +6
==========================================
+ Hits         1958     1959       +1
- Misses        582      587       +5

Impacted Files
Coverage Δ

mlflow/tensorflow.py
0% <0%> (ø)

mlflow/cli.py
83.58% <100%> (+0.24%)
:arrow_up:

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 8994b14...e8fc716. Read the comment docs.

Turns out the problem was with prospector segfaulting when dealing with Tensorflow code. Fix is in #59.


Git actions - Release

https://andresalmiray.com/publishing-to-maven-central-using-apache-maven/

Alternatively: Something like CircleCI/TravisCI to do this
Ideally I would have something where I can decide on the version number / semantic versioning (diff jobs for patch/minor/major would be ok with me)

https://github.com/sdkman/sdkman-vendor-maven-plugin/blob/master/pom.xml

Via github-actions? publishing-packages-to-the-maven-central-repository


FileNotFoundError: [Errno 2] No such file or directory: '/src/png/c++.png'

When cpp files are included in repo it seems an error [FileNotFoundError: [Errno 2] No such file or directory: '/src/png/c++.png']. I think it will be work fine if rename the  '/src/png/c++-1.png' to '/src/png/c++.png'.
please check this pull request https://github.com/jayanta525/github-pages-directory-listing/pulls.
Thank you.

Same problem today. 🙏


[FEATURE] Add option to disable focus in certain filetypes

Hi, thank you for creating this lib, the truth has been very useful :slightly_smiling_face: but there is a small detail ... I would like to be able to have an option to disable focus on certain filetypes since I use some plugins like Vista.vim and when using focus the Vista view is messed up when I open multiple buffers at the same time (like NvimTree and Vista) :confused:

Hi there :)
Thanks for the positive feedback! Happy to hear that some people are finding focus to be useful.
I can absolutely look into adding this feature, it seems like a sensible request so it has my vote

Okay, so I have had some issues on my end confirming if this is working.
I also did some more testing and noticed sometimes when you have minimal screen real estate, and try to open too many windows it can sort of 'crush' the other windows in view. I think this is normal vim behaviour in that instance. E.g I tried opening NerdTree, NvimTree and CHADTree all at once and it ended up messing them up even with focus disabled.
I have pushed some commits to this branch, that should allow for defining custom filetypes. https://github.com/beauwilliams/focus.nvim/tree/cust_filetypes
Let me know what you think :)

So for example we open a new buffer, make a split, run :lua vim.o.winwidth=120 to resize our main window, open NvimTree open NERDTree and then switch (ctrl-w) back to our original split we can see that the NERDTree has shrunk due to the fact there is not enough width to accommodate all the windows
At least this is the behaviour I am finding on my end right now 🤔

Hi, really thank you very much for the improvements, I still had some problems with Vista when opening the same windows but I reduced the width to 60 and now they all work like a charm :slightly_smiling_face:

So in itself the changes work, it would only be to play a little with the width according to the size of the screen and the number of windows that will be opened, but that already by the user :thinking:

Okay wonderful, I think for the meantime we could go with these changes?
I can have another think about how we might deal with limited screen real estate but its fair to say that can be up to the user to reduce the size of focus in that case ☺

Perhaps for now to give a warning that if it doesn't work it's that the screen is very small and should reduce the focus width :thinking:
For example, right now I'm on a 1368x768 monitor but due to problems with the Nvidia drivers I have to boot without them and that leaves me with a resolution of 1024x768, which is really small and made me reduce the focus width a little more to so it works

Great idea! I will be sure to add this to the readme then. Let me get onto it right away.
Fingers crossed most users are not stuck with such a small resolution. But glad that there is still enough space for you there to make focus still useful 😁


hover on all row

When hovering over a cell, I want all cells in the corresponding row to exhibit a hover effect

?

Can I have the entire corresponding row highlight when I hover over a cell?

https://maxim-saplin.github.io/data_table_2/#/datatable2
It is the current default behaviour, as long as you don't have tap event handlers on individual cells

Thanks I was using old version of this package.


Set CSS class according inline style menu selection paragraph style

Fix https://github.com/eea/volto-slate/issues/109

@ksuess Please rebase

would you squash?


Is there existed some downloadable example/demo projects or tutorials?

I don't know where to start learning. I downloaded the engine, created an empty project, drew something on the map, run it and I see a black screen. I understand that probably the map needs to be drawn from the script, but I have no any examples, so it is very difficult for me to take the first steps.

You can find some examples here: https://github.com/cstoquer/pixelbox-examples/

Thank you!


Getting Error during the run docker command

I am getting below error after run following command, i don't know how to pass a flag, can you provide a example here.
command - docker run -p 9261:9261 frodenas/grafana-exporter
Error - time="2018-11-13T11:52:46Z" level=error msg="Flag grafana.uri is required" source="grafana_exporter.go:96"

You just need to pass the args after the docker image: docker run -p 9261:9261 frodenas/grafana-exporter <flags>. Flags are documented here.


Doesn't build with filepath 1.4

http://hackage.haskell.org/package/filepath
in the cabal file:
 filepath >= 1.3 && < 1.4,

I increased the upper bound on filepath and pushed a minor patch bump to hackage.

:+1: Thank you.


feat: advanced mode

Fix #290

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


Create ActionUnion

@ccheney @n8rzz what do you think of this change?

@n8rzz The static methods are private so you wouldn't test those. The only thing I was testing was to make sure the static methods in the reducer had the same action payload type as the action. With the union you now have to tell TypeScript what the payload type is.
https://github.com/codeBelt/typescript-hapi-react-hot-loader-example/blob/b4687279476a98a7563d1d7624047d0bd69dbc5c/src/stores/user/UserReducer.ts#L24


Binary Backward

Update the examples for binary backward documentation
24 ops completed, 1 op pending

All prs merged


Comment input field as almEditable element, fixes #722

Replaces textarea with contenteditable div, by almEditable directive
Fixes an index counter issue introduced by #731, that was breaking tests
Workflow fixes for the new element (content reset on various actions)
Optimized the way insertion into comment collection for a WI was being handled

Signed-off-by: Soumya Deb<EMAIL_ADDRESS>

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Following behavior I noticed when I open the detail page for the first time after page reload:

Comment Box is automatically focused. Do we need this behavior?
"Add a new comment..." is not a placeholder of the comment box. "Add a new comment..." should be a placeholder.

The comment field should not be focused by default.
Focusing on the comment field should remove the place holder and make it empty
Place holder should have a different text style as it's done in description
Use  tag instead of  for auto height adjust and don't specify any height for the element anywhere.

@debloper is your tab size is 4?  Can you change it to 2? And it is good if you install EditorConfig for VS Code.

I just went with the consistency of the particular files.
My own settings are at 2 space, but I don't wanna mix it up in a file where the whole file is 4 space indented.

@debloper unit tests are failing locally.

@nimishamukherjee @debloper to keep height in em or px We will ask UX Team and i think we should do this change in separate PR as this will going to change several places

@SMahil thanks, sounds good.

[test]


Search duration setting is stuck to 28 days

By default, the Search Duration setting is set to 28 days.  If I change it to, say 100 days, and then go back to the daily note outline side window, I can see that the search duration is still 28 days.  If I return to settings window for this plugin, I see that Search Duration has been reset to 28 days.

Sorry for the inconvenience.
I am not able to reproduce the problem at this time, so we need to find the cause of the problem.
First, could you please open the data.json of the DNO plugin with a text editor and check the "duration" field?
In my case, it looks like this:
  "duration": {
    "day": 28,
    "week": 12,
    "month": 6,
    "quarter": 8,
    "year": 4
  },

If the structure is not as described above, the duration setting may be corrupted for some reason.

In this case, please close Obsidian and do one of the following to fix the problem.

Rewrite the "duration" part of data.json as above.
Backup and remove data.json from the DNO plugin folder (this operation will recreate data.json anew).
Uninstall and reinstall the latest DNO plugin.
If any of the above seems to repair the problem (and allow duration changes), try using it for a while.

If you are still unable to change the duration in spite of the above duration structure, or if none of the above fixes the problem, please provide me with the following information.

Are other parts of DNO working fine except for the duration change?
If you rewrite the value of duration.day in data.json directly from 28 to 100, does it change the range to be displayed in DNO view?
Whether the core Daily Note plugin is enabled or not.
Whether you are using Periodic Notes plugin or not. And its version.
How you have changed your DNO settings from the default. (I would appreciate it if you could paste the contents of the data.json file after removing the privacy concerns).
Any other information that might help me reproduce the problem.

Well, I don't know what happened, but when I finally got back to this, the search function duration setting works as expected.  Since I reported this, I've updated many plugins, updated Windows, and rebooted more than once...  Huh.
Now that daily note search works for me, I've been using it, and I'm finding search restricted to daily notes very useful.  Thanks.
One suggestion / request from me is to add some shortcuts.  Keyboards shortcuts are much faster than mousing, and one thing great about Obsidian is how many plugins have them.

I am glad to hear that it seems to be working properly for now.
If you have further problems, please let me know.
You mean creating hotkeys, or rather, commands for various actions. I would like to consider this. Please let me know if there is any particular action that I should set up a command for.


Improve design capture page

Description
In this PR, I've improve the design of the capture page by highlighting the recommended modes. Ref: #7063

Checklist

[x] I have updated tests where applicable.
[ ] I have added an entry to the CHANGELOG.

it should be fine now :)


Two small modifications so to integrate with existing projects

In order to use this code in my project, I changed two things :

moved the "use namespace std" below the "namespace JSLike" scope

my compiler did not know bad_cast::__construct_from_string_literal so I replaced it with an undocumented bad_cast.

Hi Sébastien.  Thank you for the fixes.  I am inclined to accept the locally-scoped "use namespace std".  Looks great!
For the bad_cast fix, I am wondering if it would be possible to preserve the text message in the exception raised in case of a failed dynamic_cast?  In my 15min of free time this morning, I asked ChatGPT for an opinion.  This is the solution that it proposed:
class MyBadCastException : public std::bad_cast {
public:
MyBadCastException(const std::string& message) : m_message(message) {}
const char* what() const noexcept override {
return m_message.c_str();
}
private:
std::string m_message;
};
If you do something like this (and it works), I am inclined to merge it, probably over the weekend or when I have some time to test the proposed change with Visual Studio as well.
Thanks again.  I appreciate your kind help.
Virgil

Instead of creating a new exception type, I used the more general but still accurate std::logic_error.

I am not familiar with the syntax but this kind of check might be doable at compile time

Let me think on it.  From what I understand, logic_error is used to indicate issues with the logic of the program and is typically thrown by "sanity checks" or more formally, invariant condition checks.  Arguably, failed dynamic casts fall outside of this category.
If you prefer, please create a branch/PR that moves the "use namespace std" below the "namespace JSLike" scope, and I will merge it.  In that case, I will do a separate commit with the specialized bad_cast.  This will allow existing code that catches bad_cast to continue to function without changes.

If you prefer, please create a branch/PR that moves the "use namespace std" below the "namespace JSLike" scope, and I will merge it.

Done.

In that case, I will do a separate commit with the specialized bad_cast.

Yes please.

This will allow existing code that catches bad_cast to continue to function without changes.

Yes, I forgot about backward compatibility with existing user code. You are right not to change the API.


Adds Brazilian portuguese translation file.

Adds Brazilian Portuguese translation file. 65% complete. Missing translations will be added later.

Muito obrigado!
I'll reuse it to start the PT version.

@akazorg Thanks! Duly noted. I've translated another chunk this morning, will probably be able to finalize it tonight. I'll try to keep the capitalizations consistent with the English version. Although, and please feel free to disagree, in both English and French (and possibly all the romantic languages), capitalization and punctuation work a bit different than English.
I work building bilingual sites on a regular basis (French and English) and even though we have capitalized letters in English (e.g. Lorem Ipsum Dolor Sit Amet) when translating to French we always leave just the first word capitalized (e.g. Lorem ipsum dolor sit amet).
But maybe this is a particularity of the company I work for or the French language.

Hi @vpakg, absolutely about the differences between languages.
I meant for the capitalized headers and buttons. If we apply a rule and be consistent, the UI / UX will work perfectly.
I've also translated a part of PT (your PR help, ofc!), must be ready soon.

@vpakg I've just finished PT, if you want to reuse it, check here.

Hey, @marktopper and @akazorg thanks for the input! Realy sorry this is taking so long, I'm crazy busy these days. I'll get this done today for sure. If anything, ping me on Slack, I'm usually online there.
Cheers.

Reworked it, pushed patch-2 for review.

Looks great Vitor, will merge it.
Thank you!


overlap creator 623

Introducing an updated HTML layout featuring sophisticated overlapping design elements. This revision enhances visual engagement by incorporating layered images with text overlays, optimized for responsiveness and accessibility. The design leverages modern CSS techniques for a clean, aesthetically pleasing user interface, aiming to improve the overall user experience.

Heartfelt Thanks for Your Contribution to Our Open Source Project!
Dear @tejachalla123 (None),
I trust this message finds you in good health and high spirits. I am reaching out with a heart full of gratitude for your recent contribution to our open source project. Your dedication has not only elevated the functionality of our project but has also exemplified the collaborative spirit that lies at the core of thriving open source communities.
A heartfelt congratulations are in order for your exceptional work! Your commitment and expertise have played a pivotal role in the success of our project, and we consider ourselves truly fortunate to count you among our community members.
I must extend my sincere apologies for the delay in responding to your pull request. Your patience and understanding during this time are immensely appreciated. As our project continues to grow, we acknowledge the imperative need to enhance the quality and response time for contributions. To address this, we've kickstarted a call for maintainers and reviewers who can help us uphold the high standards of our project and ensure swift responses to pull requests.
Your involvement and insights would be invaluable in this initiative. If you, or someone you know, is interested in taking on a more active role in the project, please don't hesitate to reach out. We firmly believe that with a robust team of maintainers and reviewers, we can elevate both the quality of the project and the speed of our responses.
On an exciting note, I am thrilled to share that we have activated the sponsorship button for our project. This development allows supporters like yourself to contribute to the sustainability and growth of the project. Your ongoing support, whether through code contributions or sponsorship, is genuinely appreciated.
Once again, congratulations on your outstanding contribution, and thank you for being an indispensable part of our open source community. We eagerly anticipate your continued collaboration and shared success.
Best regards,
Ntem Kenyor K.
Lead
Kwickerhub


Add Alert: More replica running than desired for some time

See https://git.speag.com/oSparc/osparc-infra/-/issues/incident/21

Let's not do it until we face this issue more often.


Can you have two ismatch calls using "and" operator?

This will not return anything.  Am I doing something wrong or is this not part of the syntax?
Filter = search.ismatch('"Bob"', 'Name') and search.ismatch('"D"', 'Address')

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

ID: 952c30d2-a5ca-40b1-2caf-26285d261131
Version Independent ID: a8f93577-22fe-0bfb-f87d-d6e368ea4fee
Content: OData full-text search function reference - Azure Cognitive Search
Content Source: articles/search/search-query-odata-full-text-search-functions.md
Service: cognitive-search
GitHub Login: @brjohnstmsft
Microsoft Alias: brjohnst

@Pryorite Yes, it should be possible to use search.ismatch in this way. I recommend you create a question on StackOverflow and tag it with azure-cognitive-search. There is automation that dispatches such questions to engineers on the Azure Cognitive Search team. Please include more details in your StackOverflow question, such as whether you're using the REST API or SDK (and if so, which language), and examples of which documents you expect to match the queries.
Since this is more of a support question and not a docs issue per se, I recommend we close this.

@brjohnstmsft - I think I was looking at it wrong.  Seems the ismatch matches full works not contains.  As in the Address part would have to be Drive or Dr to get anything back on the and condition.  Does that sound about right to you?

@Pryorite StackOverflow really is your best bet for getting help troubleshooting this.

I'm closing this issue as I mentioned above the ismatch matches whole words within fields.


Update README to reference Doxx project

The project README includes a Using Templates section at the bottom that references dynamically generated pages (DGPs) and shows an example. Recently, I wanted to learn more about DGPs, @vipulgupta2048 referred me to the Doxx documentation. So, for future learners this PR adds a reference to the Doxx project README, which contains useful information about DGPs.
The PR also updates the example use of a template to select a DGP, which contained outdated variable names.

@balena-ci retest

@balena-ci I self-certify!


VERIFY-LINUX-DISK-SETUP - RedHat RHEL 7-LVM 7.6.2019060521

No UUID filter from dmesg
 python VERIFY-LINUX-DISK-SETUP.py
2.7.5 (default, May 20 2019, 12:21:26)
[GCC 4.8.5 20150623 (Red Hat 4.8.5-36)]
Traceback (most recent call last):
  File "VERIFY-LINUX-DISK-SETUP.py", line 194, in <module>
    RunTest()
  File "VERIFY-LINUX-DISK-SETUP.py", line 183, in RunTest
    VerifyUUID()
  File "VERIFY-LINUX-DISK-SETUP.py", line 102, in VerifyUUID
    uuid_from_demesg = uuid_from_demesg.replace("\\x2d", "-")
AttributeError: 'int' object has no attribute 'replace'

@v-sijliu Please take over this.


update CONUS404 datasets

update on-prem/cloud hourly
update on-prem daily
update on-prem monthly
(daily and monthly have not been pushed to s3 yet)

When do we plan to be making updates to the datasets? I think if we update and change the data in a significant way (such as bias-correction), we should give it a new name. We don't want people unknowingly pulling data that has been processed in a different  way without modifying their workflows. I don't know why we have needed to make updates so far, so it would be good to understand if we are expecting to make more changes and what they are.

I'm wondering how to best communicate changes in datasets with our customers, especially since we are not likely to keep copies of the old versions readily available.
Seem like the options are:

change the name of the actual dataset in intake, which will cause notebooks to break, and they will have to then figure out what the name of the latest datasets is (e.g. conus404_hourly_202209 => conus404_hourly_202211 or something)
keep the datasets names the same and change a version number or description in the Intake Metadata.   Then notebooks don't break, but the results might change (if we change to a bias-corrected version, for example)

Since we don't keep the older versions (since they're huge) I'm somewhat neutral on whether we update the dataset name with a date or keep the original dataset name and just update metadata about the version. It would provide consistency if the dataset name did not change and would minimize breaking notebooks. How to alert users to potential changes in the data would be a challenge.
Regardless we need to figure out a place where the changes are detailed for each dataset so those that are interested are able to see the changes that have been made to a given dataset.

@rsignell-usgs @pnorton-usgs When do we plan to be making updates to the datasets? I think if we update and change the data in a significant way (such as bias-correction), we should give it a new name. We don't want people unknowingly pulling data that has been processed in a different way without modifying their workflows. I don't know why we have needed to make updates so far, so it would be good to understand if we are expecting to make more changes and what they are.

So far the changes that have occurred have mostly been in response to fixes applied by the NCAR folks to our source datasets. There will be changes as errors are discovered and also when the data are extended in time. As noted in my previous reply I think we need to have a change log for each dataset to track the changes that are made to the dataset. The datasets are generally too large to be able to keep prior versions so the change log would be critical to providing information about the updates that have been applied.
Products like the bias-corrected model output will be completely new datasets that we can add to the catalog and maintain their own change log.


[BEAM-4594] Skip test_pardo_state_only for Flink PVR.

Skip test_pardo_state_only for Flink PVR.

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---

Run Python Flink PortableValidatesRunner

@angoenka how do I trigger the Flink PVR tests? Run Python Flink PortableValidatesRunner does not seem to do anything.

@angoenka the build did trigger, but not on the merged source.

Thanks for identifying that. I will look into it.


out_http: ReadTimeout resulting in an invalid json in following chunks

Check CONTRIBUTING guideline first and here is the list to help us investigate the problem.
Describe the bug
I use out_http to output json arrays over HTTP. When the server is slow to respond, and ReadTimeout occurs, fluentd retries sending the chunk, but the JSON format is invalid (looks like the end of the buffer is trimmed).
To Reproduce
Send any log over http and make the server not respond for at least read_timeout seconds.
Expected behavior
Fluentd should retry the exact same chunk, or at least not slice the chunk in an arbitrary position.
Your Environment

Fluentd version 1.11.2
Operating system: Debian container running on MacOS

Your Configuration
<match **>
        @type http

        endpoint "http://localhost:8080" # local test server
        open_timeout 2
        read_timeout 10

        content_type "application/json"
        retryable_response_codes [503]

        json_array true
        <format>
            @type json
            add_newline false
        </format>

        <buffer>
            # Interval for flushing event chunks
            flush_interval 5s

            # The number of threads to flush/write chunks in parallel
            flush_thread_count 2

            # The max size of each chunks: events will be written into chunks until
            # the size of chunks become this size
            chunk_limit_size 16mb
        </buffer>
    </match>

Your Error Log
Example ReadTimeout error:
[warn]: #0 failed to flush the buffer. retry_time=275 next_retry_seconds=2020-09-29 10:01:02 +0000 chunk="5b070cb025df79d5cd538adb86617115" error_class=Net::ReadTimeout error="Net::ReadTimeout"

Additional context
Example for an invalid JSON output (note the json array closing AFTER the payload is trimmed):
[{"worker=0","service":"fluentd","tag":"fluent.info"},"sourcetype":"fluentd","agent_timestamp":"2020-09-29T09:44:27Z","hostname":]

(the actual payload is larger, ±14kb)

Patch: https://github.com/fluent/fluentd/pull/3144


Support -q in nerdctl push

# snd = sudo nerdctl
$ snd push xxx/alpine
INFO[0000] pushing as a reduced-platform image (application/vnd.docker.distribution.manifest.list.v2+json, sha256:b8c628ebeabe71102d6d889930d80a21ce2593c3ac17461b7a875220f2d876ab)
index-sha256:b8c628ebeabe71102d6d889930d80a21ce2593c3ac17461b7a875220f2d876ab:    done           |++++++++++++++++++++++++++++++++++++++|
manifest-sha256:93d5a28ff72d288d69b5997b8ba47396d2cbb62a72b5d87cd3351094b5d578a0: done           |++++++++++++++++++++++++++++++++++++++|
config-sha256:042a816809aac8d0f7d7cacac7965782ee2ecac3f21bcf9f24b1de1a7387b769:   done           |++++++++++++++++++++++++++++++++++++++|
elapsed: 0.2 s                                                                    total:  2.3 Ki (11.3 KiB/s)
$ snd push -q xxx/alpine
INFO[0000] pushing as a reduced-platform image (application/vnd.docker.distribution.manifest.list.v2+json, sha256:b8c628ebeabe71102d6d889930d80a21ce2593c3ac17461b7a875220f2d876ab)

As a comparison, in docker:
# d = docker
$ d push xxx/alpine
Using default tag: latest
The push refers to repository [docker.io/xxx/alpine]
7cd52847ad77: Mounted from library/alpine
latest: digest: sha256:e2e16842c9b54d985bf1ef9242a313f36b856181f188de21313820e177002501 size: 528
$ d push -q xxx/alpine
docker.io/xxx/alpine:latest

As a comparison, in docker:

Can we make the outputs same?

Can we make the outputs same?

Updated to print ref iff -q is given, same as docker:
$ d push -q xxxxxx/alpine
docker.io/xxxxxx/alpine:latest

$ snd push -q xxxxxx/alpine
INFO[0000] pushing as a reduced-platform image (application/vnd.docker.distribution.manifest.list.v2+json, sha256:b8c628ebeabe71102d6d889930d80a21ce2593c3ac17461b7a875220f2d876ab)
docker.io/xxxxxx/alpine:latest

$ d push xxxxxx/alpine
Using default tag: latest
The push refers to repository [docker.io/xxxxxx/alpine]
7cd52847ad77: Layer already exists
latest: digest: sha256:e2e16842c9b54d985bf1ef9242a313f36b856181f188de21313820e177002501 size: 528

$ snd push xxxxxx/alpine
INFO[0000] pushing as a reduced-platform image (application/vnd.docker.distribution.manifest.list.v2+json, sha256:b8c628ebeabe71102d6d889930d80a21ce2593c3ac17461b7a875220f2d876ab)
index-sha256:b8c628ebeabe71102d6d889930d80a21ce2593c3ac17461b7a875220f2d876ab:    done           |++++++++++++++++++++++++++++++++++++++|
manifest-sha256:93d5a28ff72d288d69b5997b8ba47396d2cbb62a72b5d87cd3351094b5d578a0: done           |++++++++++++++++++++++++++++++++++++++|
config-sha256:042a816809aac8d0f7d7cacac7965782ee2ecac3f21bcf9f24b1de1a7387b769:   done           |++++++++++++++++++++++++++++++++++++++|
elapsed: 0.3 s

Not sure if the logs should still be generated/printed if -q is given (right not didn't change it)


Add Python 3 support

It's all in the title. :D
Maybe @ASalvail could you have a quick look at this one whenever you have time.
Edit: closes #89

This makes #89 Invalid.

Not familiar with the code base, but with the comments addressed, the changes look fine to me.

Coverage decreased (-0.05%) to 96.047% when pulling 59c4d0c4de4cfd689297db914fbaa47c753375fa on MarcCote:add_python3_support into 0e6283c959485c89cee385540bf891939b61c106 on SMART-Lab:master.

Coverage decreased (-0.3%) to 95.867% when pulling cc987285ba5b2004e99b94e5e61786089a66f08b on MarcCote:add_python3_support into 845a8402b8d8f9d22cc6136a312ab4d28362bd29 on SMART-Lab:master.

Thank you @ASalvail for your comments.

@mgermain I added a unit test for the --commandsFile option. Make sure you check https://github.com/SMART-Lab/smartdispatch/pull/106/files#diff-f3e1285c65572b36c9b0f21b0734c260

Coverage decreased (-0.09%) to 95.331% when pulling c85fb5f19df8abd52e2837b71634fd657ceb65e0 on MarcCote:add_python3_support into 99859d2e36f7c2532a344c0a05e985ba03fd9fdc on SMART-Lab:master.

Coverage decreased (-0.1%) to 95.29% when pulling 6eb7f8efa6fc9dc76651908ee4d52952f7337e31 on MarcCote:add_python3_support into c33726e79d728dc1f6ecfcde9de3a2046c64e362 on SMART-Lab:master.

See PR #161


Re-add interactivity warning

Did #429 not address this PR?

Did #429 not address this PR?

This issue you mean? :) And yes!


Navbar fixes

Not the most elegant solution to 453 but will avoid breaking changes for now.
Issues were:

Content wasn't pushed down automatically
When rendered at 1280px both text and icon wasn't displaying for <Navbar.Action>

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update documentation for sce_to_anndata to include note about conversion of string annotations to categoricals

Originally noted in https://github.com/AlexsLemonade/sc-data-integration/pull/42#issuecomment-1187703050, the conversion to AnnData was converting some of the metadata columns to categorical factors and not all. The reason for this is based on the presence of duplicate gene names or duplicate entries in the column. We should add a note in the documentation for that function about how columns are chosen to be converted.

Closed by #120


description in guides should update

In chapter Production, webpack.dev.js file.
+ const { merge } = require('webpack-merge');
+ const common = require('./webpack.common.js');
+
+ module.exports = merge(common, {
+   mode: 'development',
+   devtool: 'inline-source-map',
+   devServer: {
+     contentBase: './dist',
+   },
+ });

property 'contentBase' should be replaced by 'static'

yes, some of the guides are still outdated with older versions of webpack and webpack-dev-server, we need to update them. Any help is appericiated.


Google Vertex AI PaLM Support

Reference: hwchase17/langchain#4532
Although there is an open issue for the PaLM API integration (#1129), these APIs are different and should be tracked separately.
I would be willing to work on this one as well and coordinate with @Jflick58 on the Python side.

Would love to see this integrated. I’m curious how it will perform given the examples parameter which explicitly defines expected outputs to the API. I still need to test this bit myself.
I just integrated PaLM 2 into my project here: https://github.com/danny-avila/chatgpt-clone
My chief complaint is that the moderation filter is pretty high. It will false flag certain topics consistently. Also despite its ability to translate well, it will refuse to answer messages without any English.

@afirstenberg  let's do it. Want to connect on the langchain discord?

I am also eager to work on this integration. I would love to hear more on this as soon as possible.
It certainly will be a nice feature to introduce to langchain ecosystem.

For those interested, the first integration (text) was just merged into main with #1273. Expect Chat and Embedding support soon.

For those interested, the first integration (text) was just merged into main with #1273. Expect Chat and Embedding support soon.

Awesome work!!!

These are now available as of versions 0.0.80 and 0.0.82.

Thank you so much @afirstenberg for the Google Palm support for langchainjs.


[zh] translate count-contributions docs

What type of PR is this?
/kind documentation

What this PR does / why we need it:
translate  docs
1./i18n/zh/docusaurus-plugin-content-docs/current/contributor/count-contributions.md
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:

/lgtm
/approve


Move Yammer bubble higher on the page

Hi team,
Can we request the Yammer Note bubble is moved to a more prominent position on the page i.e. underneath other Notes?
The page is quite long vertically so the Yammer note may go unnoticed as it did with a recent customer interaction I had. I am referring to
"Note For recommendations on Yammer IP addresses and URLs, see this blog post."
Thanks

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

ID: 077406d8-bff1-496e-3ea2-b38558fd9355
Version Independent ID: 57db3dec-2250-2dee-fd1a-ec6656911798
Content: Office 365 URLs and IP address ranges
Content Source: Enterprise/urls-and-ip-address-ranges.md
Service: o365-administration
GitHub Login: @JoeDavies-MSFT
Microsoft Alias: josephd

@dab-ms Based on our automated publishing system for the lists of endpoints, I can't insert the Skype note between the sections of lists. It needs to remain where it is. Thanks for your question.


New Crowdin translations

oh, i'll remember to set the branch to autodelete after this is merged

i'm gonna wipe this one and update it to go to master


Input: Improve state focus ring colours

Input: Improve state focus ring colours
This update improves the Input/Select focus ring colors to support all
of our main UI states, including error, success, and warning.
The default Input story was enhanced for better UI testing.
Input

Select

Resolves: https://github.com/helpscout/hsds-react/issues/571

Pull Request Test Coverage Report for Build 2088

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 2084:
0.0%

Covered Lines:
7806

Relevant Lines:
7806

💛  - Coveralls

totally removing Flow from HSDS 💯
That's the plan! One file at a time...


Deprecation [DEP0040] on dependency

Node is emitting a deprecation warning:
[DEP0040] DeprecationWarning: The punycode module is deprecated. Please use a userland alternative instead.
Its origin is<EMAIL_ADDRESS>node-fetch has breaking changes from 2.7 to 3.0 and drops support for very old node versions.

Based on https://github.com/lquixada/cross-fetch/issues/177#issuecomment-2358005899, cross-fetch might be unnecessary
Temporary solution could be https://github.com/lquixada/cross-fetch/issues/177#issuecomment-2172981911


pod remote stuck

Hi
I am trying to create app with github tracking with command pod remote app stfoo/app but it outputs message Username for 'https://github.com':  and blocks terminal, so I can't input any symbols or even interrupt pod with ctrl+c.
After it in pod list I see this:
root@stfoo:~# pod list

    name      status      port      restarts      uptime      memory      CPU

    book      BROKEN      ????

Hmm this is because you didn't add your machine's ssh key to your GitHub account. Follow this: https://help.github.com/articles/generating-ssh-keys/


clicking on a legend item should enable disable a line in a line chart

this could be achieved through a react legend and trigger a re-render


Support curved tracks

KiCad may not support curved tracks, but other EDA tools do (e.g. Eagle/Fusion 360 Electronics via GenericJSON).
Recommend adding optional fields to the 'tracks' object in the pcbdata struct to support rendering curved traces.

Kicad also supports them in nightly. I'll add this soon.

I've got this working on my fork with Eagle/Fusion.  I assumed the same structure as the 'arc' drawing object (start, radius, startangle, endangle).  The changes to render.js to make that work turned out to be trivial (which was good, since I'm not versed in javascript)
Do you know enough about the Kicad changes to define the revised format for tracks?  If so, I can try to conform to that and submit a PR for the update to render.js and the genericJSON schema (I'll leave the actual Kicad parser code to you)

Yes, arcs will be defined exactly like you describe. They are not accessible from python yet but I sent a MR for that
https://gitlab.com/kicad/code/kicad/-/merge_requests/701

Submitted PR #230


a way to build without -Werror

Currently it's hard-coded in mafiaBuild.

You can pass ghc-options to mafia after the optparse-application argument passing
e.g.
mafia build -- --ghc-option=-w
mafia build -- --ghc-option=-Wwarn


Benchmarking and modernization

This pull request started off as me doing some benchmark testing along with minor improvements and then turned into a whole modernization thing. I'm happy to break it into smaller PRs or whatever you like, but I thought this might be just mergeable as-is and save us both some work :)
Summary:

Latest ruby for dev, all latest MINOR versions for test.
Use pessimistic version constraints on all dependencies. A few forks had already done something similar to get the latest and greatest ox.
Require ox of at least 2.6.0. This is known to work on modern macOS and includes a lot of fixes/improvements. See CHANGELOG.
Minor cleanup and improvements to benchmark/benchmark.rb and run on modern hardware.
Fix a few deprecations.

+1 this would be amazing

@cloocher Anything I can do to help this along?

merge


Clean up formatting of playbooks

Split up the one liner so its a bit more readable.
Signed-off-by: Charles Short<EMAIL_ADDRESS>

@zulcss please resolve conflicts.

@zulcss conflicts.


Unit tests are failing on Solaris

Describe the bug
There are some tests failing and all this take hours to complete.
To Reproduce
Steps to reproduce the behavior:

build
make check
1 0 N N

Scheduler is PID 25631 on port 35710.

Running IPP compliance tests...
Performing 4.1-requests.test: PASS
Performing 4.2-cups-printer-ops.test: PASS
Performing 4.3-job-ops.test: PASS
Performing 4.4-subscription-ops.test: FAIL
Performing ipp-2.1.test: PASS

Running command tests...
Performing 5.1-lpadmin.sh: PASS
Waiting for jobs to complete...7...
Performing 5.2-lpc.sh: PASS
Waiting for jobs to complete...7...
Performing 5.3-lpq.sh: PASS
Waiting for jobs to complete...7...
Performing 5.4-lpstat.sh: PASS
Waiting for jobs to complete...7...
Performing 5.5-lp.sh: PASS
Waiting for jobs to complete...33...
Performing 5.6-lpr.sh: PASS
Waiting for jobs to complete...56...53...52...49...
Performing 5.7-lprm.sh: PASS
Waiting for jobs to complete...49...

FAIL: 4 job control files were not purged.
PASS: Printer 'Test1' correctly produced 0 page(s).
FAIL: Printer 'Test2' produced 0 page(s), expected 23.
FAIL: Printer 'Test3' produced 1 page(s), expected 2.
FAIL: 174 requests logged, expected 176.
PASS: CUPS-Get-Default not logged.
PASS: 0 emergency messages.
PASS: 0 alert messages.
PASS: 0 critical messages.
FAIL: 93 error messages, expected 33.

Log files zipped and attached. Let me know if you need more info.
Expected behavior
The check target passes usually. On Linux the tests finishes with PASSes in minutes.
System Information:

SunOS 5.11 <IP_ADDRESS>.0 i86pc
CUPS-2.4.6

Thank you,
Martin
amd64.zip

Hi,
I'm not able to reproduce the issue here on Fedora 37, but I found following errors in your error_log:
E [22/Jun/2023:14:13:10.889646 -0100] [Job 1] Stopping job because the scheduler could not open the output file.

It comes from this place:
1168           if (!strcmp(job->printer->device_uri, "file:/dev/null") ||
1169               !strcmp(job->printer->device_uri, "file:///dev/null"))
1170             job->print_pipes[1] = -1;
1171           else
1172           {
1173             if (!strncmp(job->printer->device_uri, "file:/dev/", 10))
1174               job->print_pipes[1] = open(job->printer->device_uri + 5,
1175                                          O_WRONLY | O_EXCL);
1176             else if (!strncmp(job->printer->device_uri, "file:///dev/", 12))
1177               job->print_pipes[1] = open(job->printer->device_uri + 7,
1178                                          O_WRONLY | O_EXCL);
1179             else if (!strncmp(job->printer->device_uri, "file:///", 8))
1180               job->print_pipes[1] = open(job->printer->device_uri + 7,
1181                                          O_WRONLY | O_CREAT | O_TRUNC, 0600);
1182             else
1183               job->print_pipes[1] = open(job->printer->device_uri + 5,
1184                                          O_WRONLY | O_CREAT | O_TRUNC, 0600);
1185
1186             if (job->print_pipes[1] < 0)
1187             {
1188               abort_message = "Stopping job because the scheduler could not "
1189                               "open the output file.";
1190
1191               goto abort_job;
1192             }

which means this print queue tries to put output into a file, and the file is supposed to be in /tmp:
D [22/Jun/2023:14:13:10.889468 -0100] [Job 1] envp[28]="DEVICE_URI=file:/tmp/Test1"

This could be one reason of test failures and why it takes so long, because the test keeps waiting for them to finish (however by default they stay in queue due default stop-printer policy) and the job history is not cleared due dangling jobs.
Subscription failure looks like a race condition, where the subscription #3 is released, but it is still seen in memory, so creating a new subscription hits the limit - I will look into the subscriptions, however the printing job issues look like specific for your OS...

Under normal circumstance we run the tests this way:
CUPS_TESTBASE=somelocaldirectory make check
But that doesn't make any difference.
The race condition definitely is something specific to hw and/or Solaris. I tried to run the same on Ubuntu seeing it finish under 3 minutes without failures. Let me know if you need any assistance, please.
Thank you,
m.

@l1gi I'm sorry, I've tried to find a SunOS 5.11 VM to try this, but no luck.
Would you mind investigating it, since you have such machine?

Hi Zdenek,
it seems version 2.4.7 resolved that issue. Closing.
Thank you,
Martin


2.0.0 Release Checklist

[x] Remove usages of async (#47 )
[x] Drop support for Python < 3.5 (#45 )
[ ] Update docs and add "Migrating to v2" section
[ ] Force default api_version to 2018-02-08
[ ] Setup new CI

Could we get the git version pushed to pypi soon? I can install the package locally with pip install -e . , but still fails when I try to install it with pip. This is pretty inconvenient for deploying apps with heroku. Thanks!

@Koeng101 We have published version 2.0.0rc1 to pypi. Just tried installing with pip install -I shippo==2.0.0rc1 and it worked.
If you were installing this version and getting an error, can you give more info on the error you are encountering?
Thanks!

@malcolmrebughini How safe is this version to switch to? We're upgrading to Python 3.7, and of course shippo 1 doesn't work. We're only using it for address validation and for tracking. Are there breaking changes?

@goblinJoel I would say it is fairly safe, usage for address and tracking hasn't been touched (other than the change to the async parameter). There's a breaking change to config that is explained in https://github.com/goshippo/shippo-python-client#migrating-to-v2
I cannot make promises that there won't be breaking changes between 2.0.0rc1 and the final release, but if more breaking changes appear they will be documented in the readme.

Thanks for the response! I checked the diff between the releases, and I noticed that note about the config change, so combined with what you're saying, it sounds like it should be a pretty straightforward update.

Thanks for providing this release candidate! I'm looking forward to seeing updates, and we can definitely use the release candidate in the meantime.

Version 2.0.0 has been released today. No new breaking changes other than the ones in 2.0.0rc1


DCOS-46146 - Re-enable test_ipv6 and test_if_overlay_ok

High-level description
Re-enable flakey tests test_ipv6 and test_if_overlay_ok as Docker has been upgraded DCOS-38144
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

DCOS-46146 networking integration test instability, to be fixed by upgrading Docker to version 17.12.x

Related tickets (optional)
Other tickets related to this change:

DCOS-38144 Update Docker on all TeamCity agents to 18.09.1-ce

Checklist for all PRs

[x] Added a comprehensible changelog entry to CHANGES.md or explain why this is not a user-facing change: N/A re-enabled test
[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain here: N/A this is a test.
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

@mesosphere-mergebot label Ready For Review

Please @jbarrick-mesosphere can you track down the failures?

Please @jbarrick-mesosphere can you track down the failures?

Not sure what happened there, but I re-ran and it seems to be better now.


I did the task Split StartGameMessage into

startcomputergame and starthumangame. Alex
check it pls

Unfortunately these changes were made obsolete when I refactored the login flow.

Unfortunately these changes were made obsolete when I refactored the login flow.


Hide Add Project Button from people who are not logged in

Fixes Issue #161

Could you cherrypick 5100f63 into a new branch forked from master? I think there were a few reasons that the edit title branch was not getting merged, but the commits are included in this pr.  Could you also make sure that the backend enforces this? server/api/project/index.js should allow only authenticated api calls to create projects.

@vkr96 like Kiana said this branch has too many commits in it. New branch's generally should come off of master. This should be a simple change that doesn't need all the other changes


Add environment override "TRUSTED" to skip chromium sandbox

This PR adds an optional environment variable TRUSTED which can be used to skip puppeteer/chromium's sandbox usage, it defaults to false. Relates to https://github.com/jef/nvidia-snatcher/issues/114 but our approach is slightly different, i.e. this change does not force the insecure setting unless set by the user.
Edit: linking to sample Docker usage using an image pushed to the DockerHub registry: https://github.com/andrewmackrodt/dockerfiles/tree/master/nvidia-snatcher#docker

I really like this! Although, I think TRUSTED could be used in a lot of different contexts. Do you mind if we prefix this environment variable with BROWSER_ or something similar?
Thanks!

Do you mind if we prefix this environment variable with BROWSER_ or something similar?

Sure, I've added the BROWSER_ prefix. The variable on the config object has stayed as isTrusted as I feel Config.browser.isTrusted is verbose enough.

Config.browser.isTrusted is verbose enough.

Agreed! Sorry, I wasn't clear there, but it all looks good from here!
Thank you!!


Create conda-forge adlfs-feedstock

Following on from https://github.com/dask/adlfs/issues/22 I would be interested in creating and maintaining adlfs-feedstock (along with @hayesgb and @martindurant) to build the package on conda-forge.
I've done this for one package (https://github.com/conda-forge/xskillscore-feedstock) and my normal procedure is to release on pypi then use the tar.gz in the meta.yaml.
Following some discussion here https://github.com/dask/adlfs/issues/22#issuecomment-586803021 I haven't had the the need to automate this build. I simply change a couple of lines after the package is released on pypi.

I was just about to ask what the plans for a conda-forge release are.

Yes, @raybellwaves , please go ahead and tag us on the PR. For a pure-python package like this, it should be straight-forward.

Very much WIP https://github.com/conda-forge/staged-recipes/pull/11831

Closed via https://github.com/conda-forge/staged-recipes/pull/11831


fix(material/select): adds cdkFocusTrap to attempt to make options focusable

Attempts to fix bug in Angular Components Select Component where the options in the listbox are not accessible/focusable on mobile. When swiping right to access the listbox the focus moves to the next focusable item rather than into the listbox of options. This fix adds cdkFocusTrap and cdkFocusAutoCapture to attempt to remedy that.
Fixes b/285945157

A long time ago in https://github.com/angular/components/pull/6856 we switched away from having focus on the options because it was problematic for VoiceOver on Safari.

A long time ago in #6856 we switched away from having focus on the options because it was problematic for VoiceOver on Safari.

Thanks for the context/info @crisbeto! Any chance you can add a dev-app link so I can test out/try to find some other solutions?

Closing this PR as this looks to be a bug outside of Angular Components' scope.


bump version dep for sl4j-log4j to 1.7.21

The kafka Client Library: https://mvnrepository.com/artifact/org.apache.kafka/kafka-clients/<IP_ADDRESS>
depends on slf4j 1.7.21. The same is the case for logstash-input-kafka
5.1. This PR upgrades the output plugin to reflect this.

here is the input's correct dependency: https://github.com/logstash-plugins/logstash-input-kafka/blob/5.x/logstash-input-kafka.gemspec

@suyograo when you get the chance, mind reviewing this?

LGTM


Typescript error - differs from already included file name<EMAIL_ADDRESS>only in casing

I'm getting the following error:
TS1149: File name<EMAIL_ADDRESS>differs from already included file name<EMAIL_ADDRESS>only in casing.

Imported via "Koa" from file<EMAIL_ADDRESS>with packageId '<EMAIL_ADDRESS>
Replacing Koa with koa in<EMAIL_ADDRESS>fixes this error. I have forceConsistentCasingInFileNames enabled in my tsconfig.

We stumbled across the same issue and made a PR https://github.com/koa-better-modules/joi-router/pull/9

v12.0.0 is here!


pt.1 of @de labels and descriptions to main classes

Added German labels and descriptions to a first set of classes.

thanks!


Roll Flutter Engine from d44462a42d73 to c59074f2569a (2 revisions)

https://github.com/flutter/engine/compare/d44462a42d73...c59074f2569a
2024-04-05<EMAIL_ADDRESS>Roll Dart SDK from 0ac840ba1f0b to 31ddd6924103 (5 revisions) (flutter/engine#51924)
2024-04-04<EMAIL_ADDRESS>Manual Dart roll to 0ac840ba1 and update frontend server snapshot filename (flutter/engine#51921)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC matanl@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


feat(bidi): implement UserContexts

This PR implements https://w3c.github.io/webdriver-bidi/#user-contexts for WebDriver BiDi protocol in Puppeteer.

User context should be passed to browsingContext.create in BrowserContext.newPage method

@sadym-chromium I think this is mostly ready but I will go over the change to see if there could be any flakiness


#184752079 Ch integrate coveralls

What does this PR do?

It set an environment in which  test will be run and  Coveralls code coverage service will be used

Description of Task to be completed?
This is to ensure we are properly informed just how well our code base is tested which in turn provides quality assurance for the product
How should this be manually tested?

Edit config.yml in the folder .circleci

Any background context you want to provide?
Units tests will be run on Circleci
What are the relevant pivotal tracker stories?

#184752079

this is on circleci but even in local I had issues with module not found when running the tests
Note: I checked the path is correct in the allroutes.js file and server.js file


add support for automated rendering

The keys are already there ensuring that version tags are automatically added. What we need on top of this is CI to render the quarto documents: https://forem.julialang.org/kellertuer/render-quarto-tutorials-in-documenterjl-with-github-actions-3fo


New asyncio communication in python

Updates the new asyncio routines for worker to worker communication with ucx-py, and fixes synchronization issues in multithreaded execution (C++).
Run blazingSQL in a thread, and all communication in the main thread's event loop.

@felipeblazing With this version I get pretty far in the E2E tests, until I hit the following error
Traceback (most recent call last):
  File "/autofs/nccs-svm1_sw/summit/.swci/0-core/opt/spack/20180914/linux-rhel7-ppc64le/gcc-4.8.5/python-3.7.0-ei3mpdncii74xsn55t5kxpuc46i3oezn/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/autofs/nccs-svm1_sw/summit/.swci/0-core/opt/spack/20180914/linux-rhel7-ppc64le/gcc-4.8.5/python-3.7.0-ei3mpdncii74xsn55t5kxpuc46i3oezn/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/gpfs/alpine/stf006/scratch/glaser/rapids/blazingsql/tests/BlazingSQLTest/EndToEndTests/allE2ETest.py", line 282, in <module>
    result, error_msgs = main()
  File "/gpfs/alpine/stf006/scratch/glaser/rapids/blazingsql/tests/BlazingSQLTest/EndToEndTests/allE2ETest.py", line 137, in main
    groupByTest.main(dask_client, drill, dir_data_file, bc, nRals)
  File "/gpfs/alpine/stf006/scratch/glaser/rapids/blazingsql/tests/BlazingSQLTest/EndToEndTests/groupByTest.py", line 171, in main
    executionTest()
  File "/gpfs/alpine/stf006/scratch/glaser/rapids/blazingsql/tests/BlazingSQLTest/EndToEndTests/groupByTest.py", line 164, in executionTest
    fileSchemaType,
  File "/gpfs/alpine/stf006/scratch/glaser/rapids/blazingsql/tests/BlazingSQLTest/Runner/runTest.py", line 1499, in run_query
    result_gdf = result_gdf.compute()
  File "/gpfs/alpine/world-shared/bif128/rapids-env/lib/python3.7/site-packages/dask/base.py", line 167, in compute
    (result,) = compute(self, traverse=False, **kwargs)
  File "/gpfs/alpine/world-shared/bif128/rapids-env/lib/python3.7/site-packages/dask/base.py", line 447, in compute
    results = schedule(dsk, keys, **kwargs)
  File "/gpfs/alpine/world-shared/bif128/rapids-env/lib/python3.7/site-packages/distributed-2.20.0+8.ga8a7586-py3.7.egg/distributed/client.py", line 2683, in get
    results = self.gather(packed, asynchronous=asynchronous, direct=direct)
  File "/gpfs/alpine/world-shared/bif128/rapids-env/lib/python3.7/site-packages/distributed-2.20.0+8.ga8a7586-py3.7.egg/distributed/client.py", line 1983, in gather
    asynchronous=asynchronous,
  File "/gpfs/alpine/world-shared/bif128/rapids-env/lib/python3.7/site-packages/distributed-2.20.0+8.ga8a7586-py3.7.egg/distributed/client.py", line 833, in sync
    self.loop, func, *args, callback_timeout=callback_timeout, **kwargs
  File "/gpfs/alpine/world-shared/bif128/rapids-env/lib/python3.7/site-packages/distributed-2.20.0+8.ga8a7586-py3.7.egg/distributed/utils.py", line 339, in sync
    raise exc.with_traceback(tb)
  File "/gpfs/alpine/world-shared/bif128/rapids-env/lib/python3.7/site-packages/distributed-2.20.0+8.ga8a7586-py3.7.egg/distributed/utils.py", line 323, in f
    result[0] = yield future
  File "/gpfs/alpine/world-shared/bif128/rapids-env/lib/python3.7/site-packages/tornado-6.0.4-py3.7-linux-ppc64le.egg/tornado/gen.py", line 735, in run
    value = future.result()
  File "/gpfs/alpine/world-shared/bif128/rapids-env/lib/python3.7/site-packages/distributed-2.20.0+8.ga8a7586-py3.7.egg/distributed/client.py", line 1842, in _gather
    raise exception.with_traceback(traceback)
  File "/gpfs/alpine/world-shared/bif128/rapids-env/lib/python3.7/site-packages/dask/dataframe/utils.py", line 671, in check_meta
    "%s" % ((" in `%s`" % funcname if funcname else ""), errmsg)
ValueError: Metadata mismatch found in `from_delayed`.

Partition type: `cudf.core.dataframe.DataFrame`
+-------------------+-------+----------+
| Column            | Found | Expected |
+-------------------+-------+----------+
| count($3)         | -     | int64    |
| count(l_orderkey) | int64 | -        |
+-------------------+-------+----------+

Any ideas what could cause this message?
Regardless of whether we prefer the asyncio loop implementation over the PeriodicCallback one, I fixed a few urgent issues in the way synchronization is done between kernels. I believe the current way of synchronization is problematic and causes deadlocks and/or race conditions. I had to modify some of the methods of CacheMachine and their use in the Batch* kernels, see d36cf5a8e20657c012094222eca4ab4fafc79a2a. I recommend that you take a look.
If we decide to go forward with the asyncio implementation, there are a few things that can and should be improved, namely the support for multiple concurrent queries. I think this will require including the query id in the names of of the synchronization events.

Note, this branch requires the rma branch of ucx-py https://github.com/rapidsai/ucx-py/pull/551

After fixing a synchronization issue in the ORDER BY processing, the branch runs at least up to the above error (3/3 times tested on 6 GPUs). I also had to use allocator='managed', for there seems to be some issue with registering regular zero-copy (?) memory with the Mellanox driver (or a consequence of a dangling pointer?). In the last attempt it failed in the 'simple strings' test TEST_02, in the BatchOrderProcessing kernel with an out of bounds access to a vector, presumably in libcudf:
In SortAndSample kernel batch 0 for Logical_SortAndSample(sort0=[$0], sort1=[$1], dir0=[ASC], dir1=[ASC], fetch=[50]). What: vector::_M_range_check: __n (which is 1) >= this->size() (which is 0)|||||


Add explanation of options for ExternalLoginSSOSessionProvider

In some examples of custom policies, ExternalLoginSSOSessionProvider has metadata item AlwaysFetchClaimsFromProvider, and also PersistedClaims, but these are not explained on the documentation page.
    <TechnicalProfile Id="SM-SocialLogin">
      <DisplayName>Session Mananagement Provider</DisplayName>
      <Protocol Name="Proprietary" Handler="Web.TPEngine.SSO.ExternalLoginSSOSessionProvider, Web.TPEngine, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null" />
      <Metadata>
        <Item Key="AlwaysFetchClaimsFromProvider">true</Item>
      </Metadata>
      <PersistedClaims>
        <PersistedClaim ClaimTypeReferenceId="AlternativeSecurityId" />
      </PersistedClaims>
    </TechnicalProfile>

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

ID: 8a542986-5eda-2cdd-180a-ecdd748b6ef6
Version Independent ID: d535e4db-5f33-ab9b-9196-6a1e0c7a551b
Content: Single sign-on session management using custom policies - Azure AD B2C
Content Source: articles/active-directory-b2c/custom-policy-reference-sso.md
Service: active-directory
Sub-service: b2c
GitHub Login: @mmacy
Microsoft Alias: marsma

Close issue because necessary metadata was not added by Firefox when navigating from https://docs.microsoft.com/en-us/azure/active-directory-b2c/custom-policy-reference-sso

@sgdesmet Thanks for the update! 👍

@mmacy, it would be great if you can share your insights on this.

Thank you for the updated documentation!

Why was this closed? There's still no documentation for the first example "AlwaysFetchClaimsFromProvider".

Note that this closed 2.5 years ago.
It looks like it closed automatically when I merged https://github.com/MicrosoftDocs/azure-docs-pr/pull/105814. The two PRs might have been linked.


Removing extra backticks

What is the purpose of this change? What does it change?
To remove a couple of extra backticks that could interfere with users who copy/paste the commands
Was the change discussed in an issue?
fixes #???
No
How to test changes?
Look at the rendered markdown

/retest

/approve


Update python to use 3.8

Summary

Update python to 3.8
update circleci aws s3 orb to the latest release available

Ticket Link
Jira: https://mattermost.atlassian.net/browse/DOPS-557

Thanks Carlos! This specific change requires us to use python 3.7 - is it possible to use that version? Thank you! :)

@justinegeffen sure, updated to 3.8 because i saw the preview env was using that
but will move everything to 3.7

Yeah, this is a new change related to how our conf.py file is structured and how it uses resources to build. We do currently use a different version of python (3.8 I think) but both the preview and build servers will need to be on 3.7 going forward to support the new conf.py settings. From what I understand, 3.7 is the only version that supports multi-threaded build processes (which is what we're implementing).

@justinegeffen PTAL
move to python 3.7


Separate Host Capabilities.

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
I'm wanting something that allows the mp4 automator to be on a separate host, so that sonarr, radarr, and the downloader, can all be on light hosts, rather than requiring beefier specs for encoding.
Describe the solution you'd like
A clear and concise description of what you want to happen.
I drew up an example of what it would be like in a flow... The flow itself is slightly incomplete, but it's just an idea. In this design, an agent would be present on the sonarr and radarr host(s) and would receive the environment variables and ship off that information into a remote host via NFS/SMB that would then create a file output. The service on the remote target system would be waiting for files to drop in its queue, then read the files and give that information to mp4automator to encode/unzip and then once done ship that information back off to sonarr/radarr.

I've started working on this, but I'm having conceptual problems, like when sonarr is told to manually import something, it waits for mp4automator to finish the ffmpeg task, but in this concept, sonarr would start the agent, and the agent would fulfill it's task and be done, but no file would be ready for import, as only the information was sent off to the queue.
Additional context
Add any other context or screenshots about the feature request here.
I guess I'm curious if I'm overthinking this or reinventing the wheel. Ultimately, I just want mp4 automator to be on its own host with its own resources. Does this already exist?
Thanks! :)

Sounds like you kind of just want tdarr

Thanks, I will look into it.

Have you seen this?
https://github.com/joshuaboniface/rffmpeg


Allow arbitrary config mixins during test genconf

@cmaloney This is Ready for Review

@cmaloney This is ready for review again

🚢