Request failed with status code 429

Cannot get response from the ChatGPT even with the paid Plus ChatGPT account:
{
  "level": "ERROR",
  "time": "2023-04-18T09:46:19.306Z",
  "name": "bot",
  "msg": "Request failed with status code 429",
  "pid": 1,
  "hostname": "2b5d342ab71e",
  "diff": 253,
  "err": {
    "name": "Error",
    "stack": "Error: Request failed with status code 429\n    at createError (/app/node_modules/axios/lib/core/createError.js:16:15)\n    at settle (/app/node_modules/axios/lib/core/settle.js:17:12)\n    at IncomingMessage.handleStreamEnd (/app/node_modules/axios/lib/adapters/http.js:322:11)\n    at IncomingMessage.emit (node:events:525:35)\n    at endReadableNT (node:internal/streams/readable:1358:12)\n    at processTicksAndRejections (node:internal/process/task_queues:83:21)",
    "config": {
      "transitional": {
        "silentJSONParsing": true,
        "forcedJSONParsing": true,
        "clarifyTimeoutError": false
      },
      "transformRequest": [
        null
      ],
      "transformResponse": [
        null
      ],
      "timeout": 0,
      "xsrfCookieName": "XSRF-TOKEN",
      "xsrfHeaderName": "X-XSRF-TOKEN",
      "maxContentLength": -1,
      "maxBodyLength": -1,
      "headers": {
        "Accept": "application/json, text/plain, */*",
        "Content-Type": "application/json",
        "User-Agent": "OpenAI/NodeJS/3.2.1",
        "Authorization": "Bearer **MY_TOKEN_API_KEY**",
        "Content-Length": 247
      },
      "method": "post",
      "data": "{\"messages\":[{\"role\":\"system\",\"content\":\"You are a helpful assistant named @chatgpt who provides succinct answers in Markdown format.\"},{\"role\":\"user\",\"content\":\"@chatgpt  now with OPENAI_MAX_TOKENS=100\"}],\"model\":\"gpt-3.5-turbo\",\"max_tokens\":100}",
      "url": "https://api.openai.com/v1/chat/completions"
    },
    "status": 429
  }
}

Try creating a new token api key with the new API account. Otherwise, please check if the API works for you without the bot. If it doesn't work, either, then it's obviously not a problem with the bot.

Do I maybe need the payed version for Mattermost?

Mattermost ist not involved here, at all. So the answer to that question is a clear: "no".
By "checking the API" I mean the OpenAI API calls. Basically just call the following code in isolation:
https://github.com/yGuy/chatgpt-mattermost-bot/blob/06f41d0216e56301131d092cac78a6357f627182/src/openai-thread-completion.js#L12
It's the only place where the error occurs and is completely isolate from mattermost, and if you look at the code it's also completely isolated from the bot, so it's just the Open AI API that can reasonably fail, here.

Sorry, yes, meant to write payed for OpenAI instead of Mattermost
.
Please excuse my ignorance but I am not sure how to run this script in isolation. I tried via node but that errors out because of missing dependencies, obviously. I guess I need to somehow run the createChatCompletion function in it's natural environment :-) so all the supporting libraries are found, but I don't know.

Definitely consider paying those few cents per month to OpenAI. It's well worth it (some would call it cheap to pay about 1$ per month for your whole team) and ensures proper operation.

Would going for a $ account be a test or do you know that's the issue?
Reason for asking is because this is for a non profit org, and I am putting my time in for free as well. Getting them to sign up and leave their financial data with OpenAI will need convincing and is probably not worth the effort. The alternative is that I pay out of my own pocket which I am not sure is a good idea either.

Would going for a $ account be a test or do you know that's the issue?

Don't know. Just try. I have a paid account and I have never seen this issue. The same holds true for others, it seems.
You are working for free but cannot afford to pay (much less than) 1 US Dollars for testing? Really?

Nah, of course $1 won't break the bank. It's more the uncontrollable nature of others surfing on my account :-)
I guess I'll change to a paid for account and see if this changes anything. I'll report back…

That was it, changed my account to a paid account and voila, ChatGPT is talking to me again.
Still strange that my KDE ChatGPT applet kept on working and the ChatBot with the same token stopped, but I got a working solution again.
Thanks for helping along…


[Bug]: AOI over 3 S1 SLC bursts not subsetting

Hi,
I have an AOI that spans 3 bursts and I was following the tutorial for the Golden Valley to test the code.

It seems that the AOI is ignored on the plots in some ways compared to the example

However the PS function is calculated over the 3 bursts

I am planning to cut that corner as it is not useful so I have only 2 bursts but just to let you know.
System and software version:

OS: Ubuntu 22.04
PyGMTSAR version: 2024.8.2.post3

Yes, I see—the bounding box of the selected burst covers the area fully. You can add a buffer like AOI.buffer(0.05) to expand your Area of Interest (AOI). This issue is technically related to GMTSAR binaries used in Sentinel-1 preprocessing. I plan to develop my own Sentinel-1 preprocessor to address this and other issues (including saving significant disk space) when I have the time for this project.


Stable/unbound/add tcp port + minor fixes

What this PR does / why we need it:

Updates stable/unbound to add explicit tcp support.
Removes spurious exec: from unbound readiness probe.
Wraps resources in with block to prevent rendering when empty
Unindents with blocks to left margin for readability
Increments chart version

Which issue this PR fixes:
N/A
Special notes for your reviewer:
None

/lgtm


Disallow bindings from other sources on optional claim binding.

So i want to bind the role from claim like this:
    [FromClaim(claimType: ClaimTypes.Role, false)]
    public string? Role { get; set; }

My goal is to do something role related if you have one. In the swagger document i got this:

I dont want to bind the role from quary or other source, but i want it to be optional (some users dont have any role), thus the request returns a 400.
I found this on the documantation:

Is there a way to disable higher priority binding soruces for this property? Link: https://fast-endpoints.com/docs/model-binding#from-user-claims

this looks like a unique requirement which can be achieved with a custom request binder like this:
public class MyRequest
{
    [JsonIgnore, Newtonsoft.Json.JsonIgnore]
    public string? Role { get; set; }

    public string Id { get; set; }
}

public class MyRequestBinder : RequestBinder<MyRequest>
{
    public override async ValueTask<MyRequest> BindAsync(BinderContext ctx, CancellationToken ct)
    {
        var req = await base.BindAsync(ctx, ct);
        req.Role = ctx.HttpContext.User.ClaimValue(ClaimTypes.Role);
        return req;
    }
}

public class MyEndpoint : Endpoint<MyRequest>
{
    public override void Configure()
    {
        Get("test");
        RequestBinder(new MyRequestBinder());
    }

    public override async Task HandleAsync(MyRequest r, CancellationToken c)
    {
        await SendAsync($"User Role: {r.Role}");
    }
}

or just read the user role in the handler method without putting a property on your request dto for the role:
public override async Task HandleAsync(MyRequest r, CancellationToken c)
{
    var userRole = User.ClaimValue(ClaimTypes.Role);

    if (userRole is null)
    {
        ThrowError("User doesn't have a role claim!");
    }
    else
    {
        await SendAsync($"user role is: {userRole}");
    }
}

this looks like a unique requirement which can be achieved with a custom request binder like this:
public class MyRequest
{
    [JsonIgnore]
    public string? Role { get; set; }

    public string Id { get; set; }
}

public class MyRequestBinder : RequestBinder<MyRequest>
{
    public override async ValueTask<MyRequest> BindAsync(BinderContext ctx, CancellationToken ct)
    {
        var req = await base.BindAsync(ctx, ct);
        req.Role = ctx.HttpContext.User.ClaimValue(ClaimTypes.Role);
        return req;
    }
}

public class MyEndpoint : Endpoint<MyRequest>
{
    public override void Configure()
    {
        Post("test");
        RequestBinder(new MyRequestBinder());
    }

    public override async Task HandleAsync(MyRequest r, CancellationToken c)
    {
        await SendAsync($"User Role: {r.Role}");
    }
}

Probably going to use something like this. Is there a way to implement my own attribute globaly?

with v5.1.0-beta1 you can now do the following:
public interface IHasRole
{
    string? Role { get; set; }
}

public class Request : IHasRole
{
    public string? Role { get; set; }
}

app.UseFastEndpoints(c => c.Binding.Modifier = (req, tReq, ctx, ct) =>
{
    if (req is IHasRole r)
    {
        r.Role = ctx.HttpContext.User.ClaimValue(ClaimTypes.Role) ?? "Guest";
    }
});

with v5.1.0-beta1 you can now do the following:
public interface IHasRole
{
    string? Role { get; set; }
}

public class Request : IHasRole
{
    public string? Role { get; set; }
}

app.UseFastEndpoints(c => c.Binding.Modifier = (req, tReq, ctx, ct) =>
{
    if (req is IHasRole r)
    {
        r.Role = ctx.HttpContext.User.ClaimValue(ClaimTypes.Role) ?? "Guest";
    }
});

I did almost the same thing with RequestBinder. I already started seaching a config to add request binder globally. Binding modifier is exactly what im looking for. Ty :)

Have one more question. Tried JsonIgnore and HideFromDocs on the Role property, but Swagger still shows it as query parameter. How to ignore it properly?  Haven t found any other Attribute yet.

since swagger is a different universe, you're gonna need a custom operation processor like this:
public class MyOperationProcessor : IOperationProcessor
{
    public bool Process(OperationProcessorContext ctx)
    {
        var apiDescription = ((AspNetCoreOperationProcessorContext)ctx).ApiDescription;
        var reqDtoType = apiDescription.ParameterDescriptions.FirstOrDefault()?.Type;
        var opParams = ctx.OperationDescription.Operation.Parameters;

        if (reqDtoType?.GetInterfaces().Contains(typeof(IHasRole)) is true)
        {
            foreach (var param in opParams.ToArray())
            {
                if (param.Name == nameof(IHasRole.Role))
                    opParams.Remove(param);
            }
        }
        return true;
    }
}

register it at startup like this:
builder.Services.AddSwaggerDoc(s => s.OperationProcessors.Add(new MyOperationProcessor()));


Use jsonapi-rails fork and add caching to V3 courses endpoint

Context
We forked jsonapi-rails because:

its caching mechanism is broken
the project appears to be abandoned, with the current maintainer unresponsive to open issues and PRs

Commit 359d794 of the fork provides a fix for the cache.
With the code change in place, we're able to add caching to V3 courses.
Changes proposed in this pull request

Use the DfE fork of jsonapi/rails
Do sub-object caching of courses in the index action of the V3 courses controller, keeping serialised JSON
course payloads in the cache and using them when rendering responses at
this endpoint.

Guidance to review

Is the spec sufficient?

Checklist

[ ] Make sure all information from the Trello card is in here
[ ] Attach to Trello card
[ ] Rebased master
[ ] Cleaned commit history
[ ] Tested by running locally

This looks good to me — wdyt @gpeng?


Fix for secondaryFiles inline

Implements fix for secondaryFiles parameter exposed by https://github.com/ljdursi/sga-annotate-docker

[x] Check that you pass the basic style checks and unit tests by running mvn clean install

Coverage decreased (-0.01%) to 73.387% when pulling 8399d5b763147b882eb11a5ec4a605ff8152fdd4 on feature/secondary_syntax into a691bf466d1572fcb06fb144c14b58c19881055b on develop.


Switch base.css styling to just use Bootstrap 4

Duplicate the current style but with the mobile-friendliness / ease of bootstrap

Bootstrap was not needed, excessive and just 👎


Add a vector to DashVectorHook method so mods can get the original

dash vector before changing it. This makes DashVectorHook better resemble the other hooks that require returning an output.
This also makes the modding api 0.0002% more efficient when dashing becasue the following code is no longer called twice in a row in the absence of a mod using this hook:
AffectedByGravity(false);
ResetHardLandingTimer();
if (dash_timer > DASH_TIME)
{
    FinishedDashing();
    return;
}

Gradow is ok with this breaking blackmoth so ok


[FIX] fixes to the dropdown buttons

This pull request includes a handful of improvements around the drop-down/launch buttons:

Improves the interactivity of the drop-down buttons: Version 1.0.0 introduced a gap between the hover area and the drop-down. When the mouse would move from the former to the latter (especially if slowly), the drop-down would disappear. You can see this behavior on the live site. cc @choldgraf since you did work in this area.
Adds alt text to the launch button logos
Adds a space between "Launch on" and the tool name - fixes https://github.com/executablebooks/sphinx-book-theme/issues/722 cc @rhugonnet

@choldgraf LGTM, but do cast your eye over the SBT dropdowns for "Launch on"; I didn't author this originally, so I'm not familiar with whether this is a regression or a long-standing bug.

Looks like it was introduced with the translation in https://github.com/executablebooks/sphinx-book-theme/commit/1656b772f78153e18bebde637c27482e30d4365c#diff-c18ac40355c51c6c6566efc355dfa5e7f002859040252df71ebc2d3a01da96bcL115.


Only show Involved projects if available & Added my profile

Hi @tshiamobhuda
Thanks for fixing the bug with the involved projects.
Dont forget to come say and hi tomorrow and grab a sticker


can't find variable: SockJS

npm run dev
ios 10 safari show this error

where does this error appear? any more info? line numbers? files? anything?

Closing in favor of #1108 since it has better description of the problem.


Several changes to match the new SeaClouds API.

This includes some tweaks and improvements, among with a overall code clean-up on the JS side. This PR requires #221 to be merged in order to work properly.

Current coverage is 36.54%

Merging #222 into master will increase coverage by +0.09% as of 9a9adef

@@            master    #222   diff @@
======================================
  Files          326     326
  Stmts        11926   11926
  Branches      1527    1527
  Methods          0       0
======================================
+ Hit           4348    4358    +10
+ Partial        440     435     -5
+ Missed        7138    7133     -5

Review entire Coverage Diff as of 9a9adef

Powered by Codecov. Updated on successful CI builds.

I'm going to close this until I fix some issues found during the Jan 27th demo.

@andreaturli could you reopen this PR? I pushed force this branch and now it doesn't allow me to reopen  it

Cant do that as well

Dont worry I just opened another one. It seems that if you force push a closed PR GH loses the reference to the branch or sth like this.
El 29 ene. 2016 6:30 p. m., Andrea Turli<EMAIL_ADDRESS>escribió:Cant do that as well
—Reply to this email directly or view it on GitHub.


Adding the --target flag to fal run

It would be very useful to be able to use fal with a dbt profile other than the default. For instance, I would want to be able to do dbt run --target dev followed by fal run --target dev where dev is not the default profile.
The current behavior is that fal does not recognize the --target flag, so I can't do fal run --target dev and I haven't found another way to get fal to recognize models that were run with a profile that is not the default.
For example, if I try doing dbt run --target dev followed by fal run, when fal runs I get the message:
Unable to do partial parsing because config vars, config profile, or config target have changed
and then when the script tries to retrieve the current model using a command like faldbt.ref(current_model), the script crashes with the error:
Exception: Could not get relation for 'model.<project>.<model>'
even though that model was successfully run by dbt.
Is there a way to get fal to recognize models run with a non-default dbt profile that I'm not aware of? Or would that require the addition of the --target flag to fal?
Thank you!

Hi @jakeluoma! Thanks for raising this issue! We indeed do not process the --target flag in fal yet. I'm looking into it now.

Hi @mkam039, now that the PR is merged, I tested it out (dbt run --target dev; fal run) and everything works as expected. Thank you very much for implementing this!

fantastic! closing the PR please  re-open if you have any further questions


Validate all phone numbers in the production db against Twilio's Lookup API

Description
We came across the following error in App Insights:
Caused by: Error type: NOT_A_NUMBER. The string supplied did not seem to be a phone number.
	at com.google.i18n.phonenumbers.PhoneNumberUtil.parseHelper(PhoneNumberUtil.java:3218)
	at com.google.i18n.phonenumbers.PhoneNumberUtil.parse(PhoneNumberUtil.java:3098)
	at com.google.i18n.phonenumbers.PhoneNumberUtil.parse(PhoneNumberUtil.java:3088)
	at gov.cdc.usds.simplereport.service.sms.SmsService.formatNumber(SmsService.java:60)
	at gov.cdc.usds.simplereport.service.sms.SmsService.sendToPerson(SmsService.java:50)
	at gov.cdc.usds.simplereport.service.sms.SmsService.sendToPatientLink(SmsService.java:45)
	at gov.cdc.usds.simplereport.service.sms.SmsService$$FastClassBySpringCGLIB$$f183b8bd.invoke(<generated>)

This seems to suggest that there may be patients in the database who have invalid phone numbers.
Validation
We can use Twilio's Lookup API to determine whether a given phone number is able to receive text messages. We should write a little script or Spring task to perform this validation and run it in a VM that can access the database, outputting the internal_id of phone numbers that are invalid.
Remediation
Once we have identified all invalid or un-textable numbers... what do we do with them?

@nullflux will recording the SIDs and then pinging twilio to find out what happened help with this? Or is there another way to find out a phone number is not valid other than trying to use it?

@aliciabeckett-gov Based on our latest convo w/ the Twilio folks, we can run a bulk job to validate all of the numbers in our db against Twilio

@nullflux - should we update this ticket to be "Validate all phone numbers in our DB"? Or at least validate which ones are eligible for texting? And also is this something we should do on all numbers that claim to be mobile?

@aliciabeckett-gov Done – what should we do with invalid entries?

@nickclyde do you happen to know the status of this ticket? I'm pretty sure this happened when we added the mobile/landline designation, but want to double-check

Hmmmm, I wasn't part of that work, so I'm out of the loop too :thinking: git blame says Nathan worked on it :smile: @nathancrtr do you know if we ever did this?


Readme typo

Last paragraph in Introduction, "pauppaal" may be "pyuppaal".

We have fixed this typo, thank you very much!


Post-Receive hook sends percent-encoded password

When specifying an URL with a username and password like<EMAIL_ADDRESS>as Post-Receive URL, the password is sent encoded to the server.
For example, if I have a @ in my password, the url is<EMAIL_ADDRESS>(the @ is encoded as %40). The Post-Receive hook sends bar%40 as password instead of<EMAIL_ADDRESS>

Ah this is a bug in Addressable that's been worked around in the latest Faraday.  As soon as 0.8 drops, I'll update github-services to use it.


[Feature] Use image of Ray head container as the default Ray Autoscaler container

Why are these changes needed?
Before the introduction of this PR, KubeRay used to determine the Ray Autoscaler sidecar container's image based on the rayVersion parameter.

Case 1: If this version was older than Ray 2.0.0, KubeRay defaulted to using Ray 2.0.0 for the sidecar container image.
Case 2: In other cases, it used the same image as the Ray head container. For more information, please refer to autoscalerSupportIsStable and getAutoscalerImage.

Case 1 is not necessary at this moment because KubeRay does not guarantee the support of a version older than Ray 2.0.0. In addition, there is no compatibility test for the version mismatch between Ray Autoscaler and Ray.
I also removed the description of the rayVersion behavior in https://github.com/ray-project/ray/pull/39345.

Related issue number

Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[x] Unit tests
[x] Manual tests (I can reproduce https://github.com/ray-project/ray/pull/39345.)
[ ] This PR is not tested :(

The code looks good, but doesn't KubeRay guarantee compatibility with the last Ray 1.x version?

Yes, KubeRay offers compatibility with Ray 1.13.0 for certain features. However, the compatibility mentioned above involves using Ray 1.x for the Ray container and Ray 2.0.0 for the Ray Autoscaler sidecar container. If I recall correctly, no one has yet tested or tried this combination.

Makes sense, thanks!


Select appearance diaog: Allow alternative sorting

Options:

Alphabetical up/down
Date up/down
display subfolder names as part of apperance
...?

Set priority to high because it will help to fix some bugs.


Right solution and not pass the FCC.

Challenge Diff Two Arrays has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/52.0.2743.116 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

function diff(arr1, arr2) {
  var newArr = arr2.concat(arr1);
  diffArr = [];
	for(var i = 0 ; i < newArr.length ; i++){
		if(arr2.indexOf(newArr[i])=== -1 || arr1.indexOf(newArr[i])=== -1){
			diffArr.push(newArr[i]);
		}
	}

  return diffArr;
}

diff(["andesite", "grass", "dirt", "pink wool", "dead shrub"], ["diorite", "andesite", "grass", "dirt", "dead shrub"]);

I try all the check points . and it pass , but CTRL+ENTER didn't pass.

Please use the Help chat room for getting challenge related help. Happy Coding!


CSS Fixes

I'm back!
I've added some quick css fixes and the notification changes. Notifications is still not done? I think, don't remember if I did a PR or not.

Looks like the linter caught you:
/home/travis/build/cheshire137/overwatch-team-comps/app/assets/javascripts/components/notification.jsx
  14:31  error  Unexpected whitespace before property props                   no-whitespace-before-property
  35:1   error  Expected exception block, space or tab after '//' in comment  spaced-comment


bad-precaching-response - url - manifest.webmanifest - 404

How do i get rid of bad-precaching-response for manifest.webmanifest?

Here is my repo and deployment

@lovkyndig try adding registerWebManifestInRouteRules: true

Thank you very much! Thats is something I want to try out.


Target correct variable to fix issue

Warning: I don't have access to Windows and so I haven't tested this change on anything other than Mac.

Tested and works on every operating system. Merging.
(Closes #9)


Error item expanable viewHolder

I used animation for sub items when I clicked on the parent items.
But when I collase the parent item then the next parent item doesn't receive the onClick event anymore!
If you don't use the animation for the sub item, then everything works properly.
Can you give me a solution to fix this problem?
Thank all

@ngocjx Were you able to resolve this issue?


update argument from nBits to fpSize to accomodate new datamol version

As discussed in #16, datamol switched from using GetMorganFingerprintAsBitVect to GetMorganGenerator in version 0.12.5. This PR update the argument from nBits to fpSize  and set a minimum version for datamol.
Changelogs

Update default argument from nBits to fpSize.
Fix datamol version to above 0.12.5.

Checklist:

[x] Was this PR discussed in an issue? It is recommended to first discuss a new feature into a GitHub issue before opening a PR.
[x] Add tests to cover the fixed bug(s) or the newly introduced feature(s) (if appropriate).
[x] Update the API documentation if a new function is added, or an existing one is deleted.
[x] Write concise and explanatory changelogs above.
[x] If possible, assign one of the following labels to the PR: feature, fix or test (or ask a maintainer to do it for you).

discussion related to that PR

@cwognum Done!

Thanks again @tsa87 !
Unfortunately, the test cases are failing. Could you look into this? And one more comment after a second review on my end: Could you also update pyproject.toml?

Thanks @cwognum! I've updated the argument for the failing test and pyproject.toml for datamol version.

@tsa87 The failing CICD is not your mistake. It's due to https://github.com/datamol-io/splito/pull/17.
The workaround would be to change the CICD doc workflow to upgrade mkdocs-jupyter like I've done here in polaris. I would do it myself, but don't have write access. Sorry about that!

Correction: I was actually able to do it myself! Thanks for the work, @tsa87 ! 😄

Awesome @cwognum great to hear the issue was resolved!


Expose hasPrimaryKey method in Table object

This adds a new method to the Table object to allow checking for the existence of a primary key. This mirrors the existing hasForeignKey function on the object.

@dereuromark I guess the ideal way to add a test for this would be to do something like:
public function testHasPrimaryKey() {
    $mock = $this->createMock(AbstractAdapter::class);
    $mock->expects($this->once())->method('hasPrimaryKey')->with('test', 'col', null)->toReturn(true);
    $table = new Table('test', [], $mock);
    $this->assertTrue($table->hasPrimaryKey('col'));
}

The adapters themselves have tests for hasPrimaryKey so probably don't need to replicate that logic here. Thoughts?


chore(release): v0.11.0 🎉

:robot: I have created a release beep boop
0.11.0 (2024-12-02)
Features

refs: support direct declarations of Refs (9de3ca8)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ehmpathy/sql-dao-generator/releases/tag/v0.11.0 :sunflower:


[release/0.11] Cherry-pick testing fixes from v0.10.0-rc.9 and update containerd version used

This PR makes the following changes:

https://github.com/microsoft/hcsshim/pull/1752 (not exact cherry-pick. Makes only ci.yml changes)
https://github.com/microsoft/hcsshim/pull/1820 (exact cherry-pick)
https://github.com/microsoft/hcsshim/pull/1799 (exact cherry-pick)
Update containerd tag in hcsshim branch to v1.6.23 + update google.golang.org/grpc v1.50.1 to exactly match what containerd v1.6.23 is using. Confirmed by checking there are no unnecessary changes to protobuf/grpc etc in upstream containerd if these hcsshim changes are vendored in)

The only tests failing are the symlink failures which was automatically fixed when we moved hcsshim/main to containerd/1.7 . Should we skip this failing test for hcsshim/release/0.11?

cc @helsaawy @anmaxvl @katiewasnothere @kevpar

I skipped the symlink tests in #1891 as well (FOCUS="[^(TestContainerdRestart|TestContainerSymlinkVolumes)]" make cri-integration), so we might as well do that here too

Just to add - bc58c27 this commit that this PR has can be removed too as the TODO comment has made it to containerd v1.6.22 and updating containerd tag to v1.6.23 will cover this. @helsaawy is this ok to do?

I'd prefer that commit makes it in, since the gotestsum integration is pretty handy for looking at failures, but we can have another PR that upgrades to 1.6.23 and removes the skip?

Just to add - bc58c27 this commit that this PR has can be removed too as the TODO comment has made it to containerd v1.6.22 and updating containerd tag to v1.6.23 will cover this. @helsaawy is this ok to do?

I'd prefer that commit makes it in, since the gotestsum integration is pretty handy for looking at failures, but we can have another PR that upgrades to 1.6.23 and removes the skip?

Sure I can do that. The last commit has work to upgrade the containerd tag to v1.6.23 - do you want me to break it out to a separate PR? There are some failures due to go version if the version is not upgraded. Is it ok if I just add the change to remove TestConvert and add change to skip TestSymLink as an additional commit to this PR so that we have all the tests passing?

All the tests are now passing


cannot find tags for 1.7.x

Description
Not sure this is a bug or intentional. There are two releases in the v1.7.x range on the releases page, but no tags for them.
$ git checkout main
Already on 'main'
Your branch is up to date with 'upstream/main'.
$ git tag
...
...
v1.6.0-beta.3
v1.6.0-beta.4
v1.6.0-beta.5
v1.6.0-rc.0
v1.6.0-rc.1
v1.6.0-rc.2
v1.6.0-rc.3
v1.6.0-rc.4
v1.6.1
v1.6.10
v1.6.2
v1.6.3
v1.6.4
v1.6.5
v1.6.6
v1.6.7
v1.6.8
v1.6.9
v1.7.0-beta.0
$ git checkout v1.7.2
error: pathspec 'v1.7.2' did not match any file(s) known to git

Steps to reproduce the issue
See above
Describe the results you received and expected
Tags match releases for v1.7.x as for all prior
What version of containerd are you using?
main
Any other relevant information
No response
Show configuration if it is related to CRI plugin.
No response

Can you try git fetch --all?
$ git fetch --all
...
$ git tag | tail
v1.7.0-beta.1
v1.7.0-beta.2
v1.7.0-beta.3
v1.7.0-beta.4
v1.7.0-rc.0
v1.7.0-rc.1
v1.7.0-rc.2
v1.7.0-rc.3
v1.7.1
v1.7.2

Uh oh. Did I forget to fetch? That would be a big 🤦♂
I'll check when back at the laptop.

Hello, do the tags have an incidence on the release of debian packages ? Because 1.7.x versions are not available on repository.
Install doc :

https://github.com/containerd/containerd/blob/main/docs/getting-started.md#option-2-from-apt-get-or-dnf
https://docs.docker.com/engine/install/debian/#install-using-the-repository

apt-cache madison containerd.io
containerd.io |   1.6.21-1 | https://download.docker.com/linux/debian bullseye/stable amd64 Packages
containerd.io |   1.6.20-1 | https://download.docker.com/linux/debian bullseye/stable amd64 Packages
containerd.io |   1.6.19-1 | https://download.docker.com/linux/debian bullseye/stable amd64 Packages
...

Oh serious PEBKAC. I am sorry for the distraction.


Allow string keys for method kwarg splats

Attempting to fix: https://github.com/rspec/rspec-support/issues/554
The change expands the matching criteria for kwargs from x.is_a?(Symbol) to x.is_a?(Symbol) || x.is_a?(String).  This now properly mirrors the behavior in the ruby docs.
https://docs.ruby-lang.org/en/3.0/syntax/calling_methods_rdoc.html#label-Hash+to+Keyword+Arguments+Conversion
Elsewhere tests were asserting that args with string keys were treated as optional arguments.  I have not tested if this is true for ruby prior to ruby 3, but this is NOT true for ruby 3, and I've adjusted those tests.
Testing
bundle exec rspec spec/rspec/support/method_signature_verifier_spec.rb
...
Finished in 0.03491 seconds (files took 0.09141 seconds to load)
268 examples, 0 failures

Running script/run_build fails with a permissions error during rspec, so I don't have a clean test pass of the whole repo.

Thank you for this 👏

Released in 3.12.2 thank you!


Update android-intune.md

Added steps to add storage permissions tasks.

@JohanFreelancer9 Please copy review

@Dansimp Please merge this PR. Thanks


docs: Add building-gdp.md

Start from building-gdp-ivi9.md
Signed-off-by: Gianpaolo Macario<EMAIL_ADDRESS>

LGTM, merging


hide tournament button when there is no tournaments

@AndrewJakubowicz sorry for the stange structure of tournament collection . for a students, that url returns an object like this:
 class1: [ tournaments1, tournament2 .. ],
 class2: [ tournaments1, tournament2...]
...
}```
so i need to unpack it.
this structure previously works only for the `mainTournament` page, so i arrange it as a complex josn. and this time i don't want to make server code changes so i reuse this weird structure :joy:


Small but useful functions for codeunit 10

The following functions were added to COD10:

IsLeapYear
IsNumeric
NewLine

This pull request addressed the requests made in #21.


[Do Not Merge] Only deploy PSPs if cluster has it available to support 1.25+ clusters

[ ] Pending merge of https://github.com/rancher/helm-project-operator/pull/25

Related Issue: https://github.com/rancher/rancher/issues/39846

Tested in 1.25 in https://github.com/aiyengar2/helm-project-operator/actions/runs/3709293142; successfully passes e2e CI with this fix.
Fails without this fix https://github.com/aiyengar2/helm-project-operator/actions/runs/3709173918


Fixed dispatching to correct method and removed StackOverflowError

Mistake in article causes a StackOverflowError

Hi,
Thanks for submitting. I'm on vacation and try to answer until eastern.
Best Merlin

Of course, we all do that :-)

@kabutz I implemented tests and fixed the problem as suggested. I'll close this PR okay?


Extra underscores when replacing words inside markdown italics

When the extension replaces words inside markdown italics, it adds unnecessary extra underscores.
To reproduce, open a markdown file and write:
_This texts is in italics._
Then, replace texts with text.

A similar problem happens with links, so this is probably related to #267.
This used to work fine before the SDK update.

I'm on V0.24.0, and the issue is still there.  I also use a markdown plugin "Markdown All in One" v3.6.2 by Yu Zhang.
@luisherranz , do you also use a markdown plugin?  I'm curious if it's an interaction between the markdown plugin and this one.

It just does it, it's not about the extensions.


RFC: change mapping to chain mapper

I would like to change internal mapping driver to DriverChain. It would be more suitable for separate extensions, each could register own mapper, or for modular system.
Snippet from doc: https://www.doctrine-project.org/projects/doctrine-orm/en/2.6/reference/advanced-configuration.html#multiple-metadata-sources
$chain = new DriverChain();
$chain->addDriver($xmlDriver, 'Doctrine\Tests\Models\Company');
$chain->addDriver($yamlDriver, 'Doctrine\Tests\ORM\Mapping');

cc @uestla @jiripudil @MartkCz @aleswita

@f3l1x Couldn't we do same thing in Apitte?

Sure. Good idea.

Will be implemented in v0.6.


add taggedTemplateGrammar support in .languagebabel

Hi,
I noticed this great feature: https://github.com/gandm/language-babel/issues/242#issuecomment-256016824
but I also noticed it is missing in https://github.com/gandm/language-babel/blob/8ba30a7548e177595cff7211a88e1e62281221e0/lib/transpiler.coffee#L7
so it's not possible yet to define such .languagebabel :
{
  "taggedTemplateGrammar": ["GLSL:source.glsl"]
}

No, your correct, it's an Atom wide settin stored in your editor's config.cson, as it's not possible (well, not easily) to do it on a project basis. The reason is the array of tags/scopes defined in taggedTemplateGrammar actually creates a new grammar file, prefixed ttl, that is loaded by language-babel. If this were created on a project basis then opening files in different projects within the same editor would overwrite the grammar that other files were using.
What scenario would you envisage using this setting in one project but not in another?

oh ok, I thought it was linked to this project!
Actually it's more that I was thinking to "provide a basic config" for people forking / using my project. But maybe I should just document on how you can enable the syntax enhancement ;)
So I guess it's not possible to define it, I'll have to tell people to manually paste this in the conf?

I'll have to think about this. I guess I could read a .languagebabel  property and merge it into the config.cson property, as after the first merge the grammar wouldn't be recreated. However even a merge isn't trivial as it is just a string array, so it makes comparisons somewhat imprecise.

I can't think of a simple way to ensure that deleted or changed templates definitions inside a project are removed/updated from the global config. I think i'll have to stick with what I have now. I.e. user controls it.

ok sure, no worries ;) thanks


Fix: Table TextColumn rowIndex pagination support

Current implementation of ->rowIndex() on Table TextColumn does not take pagination into consideration.

We can't release breaking changes to rowIndex() but the docs can be updated and remove that method.

The correct rowIndex() is in v3


Fix fatal error getEnableDefaultEntities #87

Fix fatal error getEnableDefaultEntities:
Added a check to see if that function still exist (exist no more in dev-master opf zfcUser(zfcUser v2))
This check make the module compatible with zfcUser v1 and the dev-master(v2) untill now.

added 2 nexw commits today.
Please merge them.
Thez first is to disbale the previous (wrong fix) the second one fixes the issue

@Danielss89
Any progress on this PR?

I'm going to close this PR as we shouldn't update for ZfcUser 2.0 yet. It's still a WIP and could change. This module is for ZfcUser 1.x only.

Ok it's a WIP but this change doen't create issue with zfcUser.
It just add support for the latest WIP version's of fcUser
So only extra options


timer fixed

Related Issue
-made the timer to the left corner and also don't display the timer at the score page.
Fixes: #344
Describe the changes you've made
made the timer to the left also made it responsive
Checklist:

[x ] My code follows the style guidelines of this project.
[x ] I have performed a self-review of my own code.
[ x] I have commented my code, particularly in hard-to-understand areas.
[x ] I have made corresponding changes to the documentation.
[x ] My changes generate no new warnings.

Screenshots

Original
Updated

original screenshot
updated screenshot

Is the timer not visible on score page? @03amir

yes. not showing on score page


Fix dataset tags

benchmark is not a part of https://huggingface.co/api/datasets-tags-by-type anymore which raises an issue when loading DatasetTags.
Means all existing versions of huggingface_hub are broken (+ CI is also failing). Since it's not a very used feature anyway, I just made a quick fix here. Would be good to think about removing this part completely.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Thanks for spotting this (yay tests)!
To deprecate the benchmark field in the future, we could allow tags to be queried in DatasetFilter and then I can insert benchmark:{benchmark_name} in all relevant datasets

for other dataset tag types, in case you see anything broken in the wild / in the community in the next few days... hit me up! (or post here)

Currently we have the following datasets affected:

(69) https://huggingface.co/datasets?search=benchmark-raft
(19) https://huggingface.co/datasets?sort=downloads&search=benchmark-gem

So not too many, although this might also affect @abhishekkrthakur's project to host ML competitions on the Hub. Do you use the benchmark field to filter submissions in your leaderboards @abhishekkrthakur?
I think it shouldn't be too hard to update these to the expected schema, but my main question is: what's the best way to filter a dataset with for a custom tag? AFAICT the current API for DatasetFilter is restricted to the "official" tags, but perhaps there's a nice workaround?

I'll open an internal PR to add benchmark: officially, will be cleaner
For custom tags (you mean in tags:?), yes the search would/should work, like for models: e.g. https://huggingface.co/api/datasets?filter=pokemon gives you two datasets

internal PR

I'll open an internal PR to add benchmark: officially, will be cleaner
For custom tags (you mean in tags:?), yes the search would/should work, like for models: e.g. https://huggingface.co/api/datasets?filter=pokemon gives you two datasets

Great, sounds good. Thanks for the pointer about tags - that certainly works in the API but what I was referring to is performing the same filter directly in hfh
I'll open a separate issue for that :)

@lewtun are you referring about making exactly the same search in huggingface_hub without using the DatasetFilter object ?
Yes, that's totally doable. Both ways will call https://huggingface.co/api/datasets?filter=language:fr :
list_datasets(filter="language:fr")
list_datasets(filter=DatasetFilter(language="fr"))

In general DatasetFilter doesn't provide you any new feature. For what I understand, it is a class to "help" you explore which filter you might want to use.

for other dataset tag types, in case you see anything broken in the wild / in the community in the next few days... hit me up! (or post here)

I haven't seen any other broken tag anyway in the past few days. So we should be fine with just adding benchmark: right?
@Wauplin Was there any other tag in the huggingface_hub tests apart from the benchmark ones?

Wauplin Was there any other tag in the huggingface_hub tests apart from the benchmark ones? (i think not, looking at the code)

I think "tasks_ids" is currently broken as well. Also related to the revamped ?

test_filter_datasets_by_task_ids currently fails in the CI so I planned to skip it.

Ok thanks. Will do

I think we can close this PR without merging right ? The fix for "benchmark" is now live in production.
@julien-c @lewtun  Please re-open if there is still something missing :)


1.0?

I'm trying to get all the glob-stream dependencies to 1.0 before I release the next major.  Is there anything stopping this module from 1.0?  Can it be bumped?

done!


Enable LSM303DLHC on stm32f3_disco and add sample

Add sample application for LSM303DLHC and enable it on stm32f3_disco board.
Depends on PR #6420.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #6441   +/-   ##
=======================================
  Coverage   53.06%   53.06%
=======================================
  Files         430      430
  Lines       41071    41071
  Branches     7911     7911
=======================================
  Hits        21795    21795
  Misses      16052    16052
  Partials     3224     3224

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 42e94c7...701e197. Read the comment docs.

LGTM

@dbkinder Thanks for the review. I addressed all your comments.

Rebased to resolve conflicts.

@nashif  Updated to resolve conflicts. This PR depends on PR #6420. I believe that we could merge it if everyone agrees.

recheck

@MaureenHelm all comments addressed.

@MaureenHelm Comments addressed.


Extract hexagon icons from Age 1 cards

We already have a set of ImageMagick commands which will extract a hexagon icon from a card, and place it inside the corresponding *_hexagon_icon_border.png template. See https://github.com/micahstairs/bga-innovation/blob/main/misc/imagemagick_commands/hexagon_icons.txt. This file contains a sample command for each of the 4 possible locations.
Let's take this set of commands one step further by doing the following:

Place the commands in a .sh script instead of a .txt file so that we can easily execute all of the commands at once.
Extract the hexagon locations into well-named shell variables (e.g. TOP_LEFT).
Extract the border files into well-named shell variables (e.g. RED_BORDER).
Write commands to extract the hexagons from each of the 15 cards from Age 1
Name the generated images by their numeric order in this this image. For example, the Pottery icon should be extracted into a file named 000.png, Tools should be 001.png, and Mysticism should be 014.png.

NOTE: Before you begin working on this, please assign this issue to yourself. If you do not do this, then there's the possibility that 2 people will end up doing the same task, which is not a good use of time.

I'm going to close this because I think we should use InageMagick to automate the hexagon icon extraction.

I'm reopening this issue now that it's ready to be worked on!


"Current Containers" should give more info

If possible, the Current list should also list the world seed or save filename if used, and the Difficulty/Mode.
I suggest these (plus the MOTD, difficulty, mode etc) are stored as labels against the container so as to be able to easily retrieve them (and to identify which containers are 'owned' by this manager)

Just wanted to note that TYPE also needs to be shown since it's not apparent this is a Forge server

Good point, and Version would be good as well.  In fact, if a Forge server, there should be a list of modules displayed (as clients may also need to have them installed) - possibly as a popup to conserve screen space?

Forge containers at least present the "mod pack" download option, but I do like the idea of listing them out to help describe the server.

Commit 9cdb018928406d98f268593c42e27be800f33834 added server type and version

Left to do, we have:

[ ] difficulty / mode
[ ] motd
[ ] maybe a popup of modules / plugins

Anything else?

World seed number (if it was specified, or if it was pulled from the World ZIP file).
World name (from ZIP file if uploaded or taken from library)
Possibly world filename, though the name from the level.dat in the Zip file is probably sufficient
Agree that modules should be a popup.
Maybe have whitelist and mods as popups if they are defined as well?

Should also indicate if the 'public' flag is set.
Possibly use a different coloured header to the section depending on public/private, or on Minecraft server type vanilla/bukkit/forge?

Yes, I was thinking that too. I might throw in a simple filtering of the container list to quickly see just public ones while logged in.


Plotting error due to no cartopy projection

The Iris sample data files atlantic_profiles and hybrid_height produce the following error when you try and plot them in the Cube Explorer:
None-type object has no attribute 'as_cartopy_projection'

This is happening because the horizontal coordinates in these sample datasets have no coordinate system associated with them. This means that when you call as_cartopy_projection it tries to get the projection from the horizontal coordinates (which will be None), so it returns that error.
Looking at the code for iris.plot.default_projection I think you could avoid this by using default_projection rather than as_cartopy_projection in this instance.


Suggestion: provide better documentation

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[X] feature request
[ ] support request

I suggest there is better documentation available for ng-formly for ng2+.
As it is really not clear what are the possible type values, and specifically are file inputs supported? I found examples for angularjs-formly but nothing for ng2+.
What is the motivation / use case for changing the behavior?
Reduce the learning curve for using the framework.
Please tell us about your environment:
Mac OS X Sierra

Angular version: 4.3.6

Browser:
Chrome

I'm also having the same difficulty.
I'm thinking of writing some tutorials after I finish my study of this tool.

There is absolutely no documentation. :(

it's in our priority now but don't expect anything soon, we have a limited time for open source projects.
in the meantime there are a plenty of examples in our demo and https://ng2.angular-formly.com that cover the most use cases.
For ui templates we support bootstrap and recently we added material support too you may check the source code to have an idea of how to create you own ui templates or override the existing ones see https://github.com/formly-js/ng-formly/issues/531#issuecomment-337715530.
Also if you have any questions just left us an issue :wink:

http://angular-formly.com for angular.js can give you a quick intro

I recommend that you prioritize documentation before adding any more features. That way you will be able to get more people to contribute to the project :)

@jayasrinagrale we're pretty close, the next step is #543

I'm struggling with documentation, it's hard to understand all the examples if you dont know what means each property of the configuration.

@byhoratiss see http://formly-js.github.io/ng-formly/docs/interfaces/FormlyFieldConfig.html, each property has now a small description, we'll add an example for each one.

https://www.youtube.com/watch?v=WiljtPGZtW8

added two section in the docs (Custom Templates and Properties and Options) + a bunch of examples.

close in favor of #820, that's the only part that prevent us to release V2 any kind of help is welcomed!


Make docs.openservicemesh.io point to the latest released version and not unreleased version

It would make a lot more sense to have docs.openservicemesh.io mirror the latest release (currently v0.10), and not docs from the main branch which is both unreleased and untested. When users visit the website, they should be trying the latest release and not code/docs from main.
This change proposes the following:

Make docs.openservicemesh.io always point to the latest release (currently v0.10). The release dropdown menu on the website should display the latest release as v0.10 (latest).
When a newer release is available, the latest tag in the version dropdown must be updated, and docs.openservicemesh.io must point to the newer release, e.g. v0.11.
Documentation from main should not be published on the docs.openservicemesh.io domain, but should be available to view on the osm-docs.netlfify.app domain, e.g. https://unreleased-main--osm-docs.netlify.app/ for testing purposes.

Note to @flynnduism - OSM wants to change the default branch used by the website to be the latest release branch instead of "main". I've noted what I think we need to consider in order to accomplish this - please add comments - thanks.

Need to review https://github.com/openservicemesh/osm-docs/pull/205 to ensure it does what's expected.
Need to ensure documented/newly-updated steps are correct
Need to ensure https://docs.openservicemesh.io points to the latest release instead of to main but also ensure that the latest release pages all work without the "release" element to the URL.
Need to ensure Netlify configs are correct


Webpack-chunks written to root dir

I followed the instructions from the readme and I get this error:
Unhandled rejection Error: EACCES: permission denied, open '/webpack-chunks.json'
I am running webpack-dev-server --config webpack.client.config.js. It turns out that webpack_config.output.path get overwritten to /.
webpack.client.config.js:
const clientConfig = client_configuration(configuration, settings);
console.log('client config', clientConfig.output.path);

export default clientConfig;

chunk.js
function chunk_info_file_path(webpack_configuration) {
  console.log('chunk location', webpack_configuration.output.path)

output:
client config /home/myproject/dist/public
chunk location /

When I run webpack --config webpack.client.config.js I have no issue.

Is this error thrown during webpack development build?
Post your webpack development configuration then.

Ok, so you get this error when running just the single webpack-dev-server --config ... command but this error is not present when you run just the single webpack command.
That's an unexpected behaviour.
You can share a project so that I could reproduce and possibly fix the issue.

Or maybe I can do it without a sample project.
This bug could mean that webpack-dev-server alters the configuration after it is run.
What I could do is simply clone the configuration here:
https://github.com/halt-hammerzeit/universal-webpack/blob/master/source/client configuration.js#L18
I'll try it and release a new version.

Try<EMAIL_ADDRESS>

That fixed it! Thanks for your great support!


New get Pod logs API

Add new API to get pod logs:
GET   /jobs/{job-name}/pods/{podUid}/logs
RESP:
{
   "locations": [
      {"name": "stdout", "uri": "https://nodeIP/podUid/stdout.0?token=string"},
      {"name": "stderr"," uri": "https://nodeIP/podUid/stderr?token=string"}
   ]
}

Coverage decreased (-0.1%) to 34.27% when pulling b5189710175083c7594ba89b8d33b80980ca095c on binyli/log-rest into d434a3c096be72bc49073f176e403d303dd8df9c on master.


Fri AVX ops

This change is

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

#511
#496
#495
#494
#493
#492
#491
#490
#489
#563  👈
#488
#486
#531
#530
#529
#528
#527
#526
#548
dev

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

Join @spapinistarkware and the rest of your teammates on  Graphite

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.23%.  Consider uploading reports for the commit 61ee9b8 to get more accurate results

Additional details and impacted files
@@                       Coverage Diff                       @@
##           spapini/03-15-avx_quotients     #563      +/-   ##
===============================================================
+ Coverage                        93.11%   94.23%   +1.12%
===============================================================
  Files                               62       68       +6
  Lines                             8049    10239    +2190
  Branches                          8049    10239    +2190
===============================================================
+ Hits                              7495     9649    +2154
- Misses                             493      520      +27
- Partials                            61       70       +9

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Minor: Improve library docs to mention TreeNode, ExprSimplifier, PruningPredicate and cp_solver

Which issue does this PR close?
N/A
Rationale for this change
While working on some other documentation improvements this morning I realized that some of the cool features of DataFusion like TreeNode, ExprSimplifier, PruningPredicate and cp_solver were not mentioned on the doc / architecture page which might make them harder to find
What changes are included in this PR?
Add doc links to these features in the main crate library page
Are these changes tested?
Existing doc tests
Are there any user-facing changes?
Just docs, no code change

Thank you @wjones127


Allow injecting coreos-cargo.eclass into portage-stable ebuilds

Allow injecting coreos-cargo.eclass into portage-stable ebuilds
Ebuilds in portage-stable that use cargo.eclass don't cross-compile cleanly. Arm64 builds produce host binaries. Modify coreos-cargo.eclass slightly to allow it to be sourced from cros hooks, which allows us to override src_unpack cleanly. This is equivalent to injecting coreos-cargo.eclass into unmodified ebuilds.
Tested that with this change zram-generator and thin-provisioning-tools now produce arm64 binaries. This resolves the following build_image warning from arm64 builds:
[2024-03-21T03:45:55.114Z] sys-block/thin-provisioning-tools-1.0.10 is missing libraries:
[2024-03-21T03:45:55.114Z] 	x86_64: libc.so.6
[2024-03-21T03:45:55.114Z] 	x86_64: ld-linux-x86-64.so.2
[2024-03-21T03:45:55.114Z] 	x86_64: libgcc_s.so.1
[2024-03-21T03:45:55.114Z] 	x86_64: libm.so.6

How to use
[ describe what reviewers need to do in order to validate this PR ]
Testing done
$ emerge-arm64-usr thin-provisioning-tools
$ file /build/arm64-usr/usr/sbin/pdata_tools
/build/arm64-usr/usr/sbin/pdata_tools: ELF 64-bit LSB pie executable, ARM aarch64, version 1 (SYSV), dynamically linked, interpreter /lib/ld-linux-aarch64.so.1, for GNU/Linux 3.7.0, stripped

[Describe the testing you have done before submitting this PR. Please include both the commands you issued as well as the output you got.]

[ ] Changelog entries added in the respective changelog/ directory (user-facing change, bug fix, security fix, update)
[ ] Inspected CI output for image differences: /boot and /usr size, packages, list files for any missing binaries, kernel modules, config files, kernel modules, etc.

TODO: figure out how to upstream something like this to gentoo.


add space to ensure styles work correctly

✨ Codu Pull Request 💻
Pull Request details

Small bugfix - Bookmark and heart icons were missing color

Add a space to the template strings

Any Breaking changes

None

I think this was fixed. I don't know if you or @CarolinaCobo got there first.


You don't consider that?


How to serialize search request

I can get serialize search request from the search response like below.
var response = client.Search<T>(s => s.Query(...));
var serializedRequest = Encoding.UTF8.GetString(response.ConnectionStatus.Request);

This works when theSearch() does not throw an exception. For cases when this method throws an exception, is there a way to get the serialized JSON request? My point being, I should always be able to get the JSON request irrespective of whether a search request succeeds or fails.

Posted the issue a bit too early I guess :)
I looked through some of the test code to find that this is possible as under:
var sd = new SearchDescriptor<T>().Query(...);
try
{
    var response = client.Search<T>(s => sd);
}
finally
{
    var serializedRequest = Encoding.UTF8.GetString(client.Serializer.Serialize(sd));
}

Hope this is the right way. Let me know if it is not.

I think this ties into https://github.com/elastic/elasticsearch-net/issues/1332
if we wrap all exceptions you would always be able to do e.ConnectionStatus
closing for now untill we create an overal design ticket.

This is annoying, NEST library should have something that spits out the JSON from request.
However this worked for me:
SearchDescriptor<T> sd = new SearchDescriptor<T>().Query(...);

using (MemoryStream mStream = new MemoryStream()) {
	client.Serializer.Serialize(sd, mStream);
	Console.WriteLine(Encoding.ASCII.GetString(mStream.ToArray()));
}

BTW I am using NEST version <IP_ADDRESS>.

@apatel-gpsw You should use Encoding.UTF8 instead of Encoding.ASCII.
NEST already has several ways to get the JSON for the request:
var search = new SearchDescriptor<object>().MatchAll();

var jsonString = client.Serializer.SerializeToString(search);
var jsonBytes = client.Serializer.SerializeToBytes(search);

And if you want to capture requests and responses for logging purposes, you can use .DisableDirectStreaming() along with OnRequestCompleted()
var pool = new SingleNodeConnectionPool(new Uri("http://localhost:9200"));
var connectionSettings = new ConnectionSettings(pool)
	.DisableDirectStreaming()
        .PrettyJson()
	.OnRequestCompleted(response =>
		{
			// log out the request
			if (response.RequestBodyInBytes != null)
			{
				Console.WriteLine(
					$"{response.HttpMethod} {response.Uri} \n" +
					$"{Encoding.UTF8.GetString(response.RequestBodyInBytes)}");
			}
			else
			{
				Console.WriteLine($"{response.HttpMethod} {response.Uri}");
			}

			// log out the response
			if (response.ResponseBodyInBytes != null)
			{
				Console.WriteLine($"Status: {response.HttpStatusCode}\n" +
						 $"{Encoding.UTF8.GetString(response.ResponseBodyInBytes)}\n" +
						 $"{new string('-', 30)}\n");
			}
			else
			{
				Console.WriteLine($"Status: {response.HttpStatusCode}\n" +
						 $"{new string('-', 30)}\n");
			}
		})
	.DefaultIndex("examples");

var client = new ElasticClient(connectionSettings);

var search = client.Search<object>(s => s.MatchAll());

A shortcut to specifying .DisableDirectStreaming(), .PrettyJson() and .OnRequestCompleted() is to use .EnableDebugMode()
var pool = new SingleNodeConnectionPool(new Uri("http://localhost:9200"));
var connectionSettings = new ConnectionSettings(pool)
	.EnableDebugMode(response =>
		{
		// log out the request
		if (response.RequestBodyInBytes != null)
			{
				Console.WriteLine(
					$"{response.HttpMethod} {response.Uri} \n" +
					$"{Encoding.UTF8.GetString(response.RequestBodyInBytes)}");
			}
			else
			{
				Console.WriteLine($"{response.HttpMethod} {response.Uri}");
			}

		// log out the response
		if (response.ResponseBodyInBytes != null)
			{
				Console.WriteLine($"Status: {response.HttpStatusCode}\n" +
						 $"{Encoding.UTF8.GetString(response.ResponseBodyInBytes)}\n" +
						 $"{new string('-', 30)}\n");
			}
			else
			{
				Console.WriteLine($"Status: {response.HttpStatusCode}\n" +
						 $"{new string('-', 30)}\n");
			}
		})
	.DefaultIndex("examples");

var client = new ElasticClient(connectionSettings);


feat: include item mimeType

include mimeType

Good, is it still required that dependent projects need to alter their expected nocks as well?

probably... we can make it breaking...

Good, is it still required that dependent projects need to alter their expected nocks as well?

probably... we can make it breaking...

I think the tests will break as soon as they reference the new dependency


feat: add --check flag to the update command

CLOSES #48
FIXES #48
Description
This PR adds the functionality to check for updates before updating them through --check flag.

Hi @hanshal101, Thank you for the PR. Can you please sign off your commits?

also, can you please add yourself to CONTRIBUTORS file as mentioned here?
Just make a dedicated commit to do so
https://github.com/buildsafedev/bsf?tab=readme-ov-file#contributing


experimental docker install on ubuntu 14.04 failed with package conflict

First,  follow this(to make sure install aufs support)
Update your package manager.
$ sudo apt-get update
Install the recommended package.
$ sudo apt-get install linux-image-extra-$(uname -r)
Then
When followed this:
https://github.com/docker/docker/tree/master/experimental
curl -sSL https://experimental.docker.com/ | sh
it failed with
Building dependency tree...
Reading state information...
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:
The following packages have unmet dependencies:
docker-engine : Depends: init-system-helpers (>= 1.18~) but 1.14 is to be installed
Depends: libdevmapper1.02.1 (>= 2:1.02.90) but 2:1.02.77-6ubuntu2 is to be installed
Recommends: aufs-tools but it is not going to be installed
Recommends: cgroupfs-mount but it is not installable or
cgroup-lite but it is not going to be installed
Recommends: git but it is not going to be installed
Recommends: yubico-piv-tool (>= 1.1.0~) but it is not going to be installed
E: Unable to correct problems, you have held broken packages.

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

ping @thaJeztah

From http://packages.ubuntu.com/search?suite=default&section=all&arch=any&keywords=init-system-helpers&searchon=names
it seems ubuntu 14.04 LTS not allow initi-system-helpers to install 1.14 newer.
So the experimental guide is not clear what os version should use. And requrements need.
For me, I did not know what to use

This was test against with #18247

Thanks @HackToday, I think I've seen this happen on regular builds as well, but not sure why the dependencies fail on some systems. I think one of the other issues had some more info, I'll try to find that one.

@HackToday I think this is a duplicate of  https://github.com/docker/docker/issues/17063, and see this comment for a possible solution; https://github.com/docker/docker/pull/13680#issuecomment-122369631
Can you try if that helps?

Hi @thaJeztah  tried the ubuntu 15.04, and it have new init-helper related packages.
For you mentioned  #13680 (comment)  I not try it yet.  My whole point is do we have related such comments in docker official guide(whether user guide or install guide) Could you give me the link  ? Since when I try experimental build , I would refer docker official link and instrcutions

@HackToday we do mention purging the old packages here; https://docs.docker.com/engine/installation/ubuntulinux/#update-your-apt-sources (see step 9). Perhaps we should add it to the https://get.docker.com / https://experimental.docker.com scripts as well, but we're trying to encourage people to use the manual install procedure

Hi @thaJeztah  I think two difference things:

for experiemental guide, it is the only one given
https://github.com/docker/docker/tree/master/experimental
And it not said how to configure the apt source for experimental build .

And that experimental used script  curl -sSL https://experimental.docker.com/ | sh
it seems not need manual configure apt source steps.

Second. if you mean configure like this (notice experimental)
echo deb https://apt.dockerproject.org/repo debian-jessie experimental
I could try if it works. If you mean this, it would be confused to users and inconsistent.
in one point you asked use script to install, and thought users know how to configure experimental apt source, and ask them to better use apt-get install ways.

And for ubuntu 15.04, I can use this curl -sSL https://experimental.docker.com/ | sh directly. No issues.
But ubuntu 14.04 not work.

The latest experimental script worked. so close it now


Update semanticdb-scalac to 4.4.8

Updates org.scalameta:semanticdb-scalac from 4.4.0 to 4.4.8.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "semanticdb-scalac" } ]

labels: library-update, semver-patch

Superseded by #112.


Loading state

closes #51

This is for mobile too?

https://deploy-preview-53--bitcoinsearch.netlify.app/

Yes the loading screen is visible for all screens as long as there is a loading state (e.g search, filter, page change etc)


Embiggen font sizes and logo

Make the ALAO logo big enough to read what ALAO actually stands for
Increase font size of navigation menu on top and bottom

I started to work on this in the branch 'embiggen-main-logo'; it turns out that the light-text version of the logo has been cropped so that part of "Ohio" is missing. I'm working on getting a full copy to replace it.


Pipeline router tests

Description
This draft PR introduces integration tests for the new router features introduced in Data Prepper 2.0.
It builds on the work done in #1949. Since that PR is still pending, I've made this a draft PR.
Issues Resolved
N/A
Check List

[ ] New functionality includes testing.
[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #1988 (d7377c1) into main (ec8df69) will increase coverage by 0.07%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #1988      +/-   ##
============================================
+ Coverage     93.83%   93.91%   +0.07%
- Complexity     1535     1537       +2
============================================
  Files           197      197
  Lines          4561     4565       +4
  Branches        367      367
============================================
+ Hits           4280     4287       +7
+ Misses          198      196       -2
+ Partials         83       82       -1

Impacted Files
Coverage Δ

...dataprepper/pipeline/server/DataPrepperServer.java
100.00% <0.00%> (ø)

...taprepper/expression/ParseTreeCoercionService.java
100.00% <0.00%> (ø)

...opensearch/dataprepper/pipeline/ProcessWorker.java
90.38% <0.00%> (+5.76%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Now that #1949 is complete, this PR is ready for review.


Feature/fix sampler madqn

What?
Fixes multiple trainers, for different agents with different architectures for madqn
Why?
So that mava can be used for hierarchical reinforcement learning
How?
Implements @DriesSmit code in MADQN

I see I made my comments on the ddpg changes, seems like that shouldn't be in this PR, but the same goes for PPO.
Other than the minor variable/method name changes, all looks good


fix: respect frozen/locked inputs for pixi list

This action runs the following sequnce

pixi install ...
pixi info ...
pixi list ...

Running pixi list without --frozen or --locked will try to update dependencies and lockfile. I think these options need to be passed through to pixi list  to keep the environment established by a locked or frozen install from changing.

pixi list updating your lockfile shouldn't happen when running with the default settings of setup-pixi, right? Then, by default you are using pixi install --locked and if the lockfile differs, it should crash even before pixi list would be run?
But I can see this being relevant for --frozen (and in general for soundness).
We should add a deprecation warning for pixi<0.14.0 like here:
https://github.com/prefix-dev/setup-pixi/blob/92e1dd3a46d4cb599eccd780b6a485a7deb9dab0/src/main.ts#L85

oops missclicked

pixi list updating your lockfile shouldn't happen when running with the default settings of setup-pixi, right? Then, by default you are using pixi install --locked and if the lockfile differs, it should crash even before pixi list would be run?

But I can see this being relevant for --frozen (and in general for soundness).

Yes --frozen is where I saw this https://github.com/MODFLOW-USGS/modflow6/pull/1713#issuecomment-2048333478.
I understand the difference between that and --locked now.


Xcode installation is incomplete; a full installation is necessary for iOS development.

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
flutter doctor

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.7.12, on macOS 13.2.1 22D68 darwin-x64, locale zh-Hans-CN)
[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[!] Xcode - develop for iOS and macOS
    ✗ Xcode installation is incomplete; a full installation is necessary for iOS development.
      Download at: https://developer.apple.com/xcode/download/
      Or install Xcode via the App Store.
      Once installed, run:
        sudo xcode-select --switch /Applications/Xcode.app/Contents/Developer
        sudo xcodebuild -runFirstLaunch
[✓] Chrome - develop for the web
[!] Android Studio (version 2021.2.1)
    ✗ Unable to find bundled Java version.
[✓] IntelliJ IDEA Ultimate Edition (version 2021.2.1)
[✓] VS Code (version 1.77.3)
[✓] Connected device (2 available)
[✓] HTTP Host Availability

! Doctor found issues in 3 categories.

How to install xcode?
The xcode is Split the contents of the package  and  I  install the xcode and command-line-tool of version 14.3 ,but There's still a problem
I gave feedback on this problem last year in Twitter, and it is the same this year. The version and system are upgraded. so what happend? Does this problem belong to flutter or xcode?
Expected results
I hope this problem can be solved
Actual results
I hope this problem can be solved
Code sample
No Code
Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
No log
Flutter Doctor output
flutter doctor

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.7.12, on macOS 13.2.1 22D68 darwin-x64, locale zh-Hans-CN)
[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[!] Xcode - develop for iOS and macOS
    ✗ Xcode installation is incomplete; a full installation is necessary for iOS development.
      Download at: https://developer.apple.com/xcode/download/
      Or install Xcode via the App Store.
      Once installed, run:
        sudo xcode-select --switch /Applications/Xcode.app/Contents/Developer
        sudo xcodebuild -runFirstLaunch
[✓] Chrome - develop for the web
[!] Android Studio (version 2021.2.1)
    ✗ Unable to find bundled Java version.
[✓] IntelliJ IDEA Ultimate Edition (version 2021.2.1)
[✓] VS Code (version 1.77.3)
[✓] Connected device (2 available)
[✓] HTTP Host Availability

! Doctor found issues in 3 categories.

How to install xcode?
The xcode is Split the contents of the package  and  I  install the xcode and command-line-tool of version 14.3 ,but There's still a problem
I gave feedback on this problem last year in Twitter, and it is the same this year. The version and system are upgraded. so what happend? Does this problem belong to flutter or xcode?

Xcode installation is incomplete; a full installation is necessary for iOS development.
Download at: https://developer.apple.com/xcode/download/
Or install Xcode via the App Store.
Once installed, run:
sudo xcode-select --switch /Applications/Xcode.app/Contents/Developer
sudo xcodebuild -runFirstLaunch

@zhaohongbo910
Did you follow the steps mentioned as above ? If so and after xcode complete installation, do you see the tool getting installed ? Or after running the steps above and then running flutter doctor -v, you still see above output ?


Rename GreetingsProps -> GreetingProps

          Though I did manage to name it "GreetingsProps" for "Greeting" (non-plural) which I should fix...

Originally posted by @NickGerleman in https://github.com/facebook/react-native-website/issues/3492#issuecomment-1382650656

Fixed by referenced  PR. 🙂


Generating image: Argument ($hiddenLayers) must be of type array null given

I'm trying to generate a graph
./vendor/bin/deptrac  --formatter=graphviz-image --output=./deptrac.png

and experiencing such error:
Warning: Undefined array key "hidden_layers" in phar:///my_path/vendor/qossmic/deptrac-shim/deptrac/src/Supportive/OutputFormatter/Configuration/ConfigurationGraphViz.php on line 13

Warning: Undefined array key "groups" in phar:///my_path/vendor/qossmic/deptrac-shim/deptrac/src/Supportive/OutputFormatter/Configuration/ConfigurationGraphViz.php on line 13

Warning: Undefined array key "point_to_groups" in phar:///my_path/vendor/qossmic/deptrac-shim/deptrac/src/Supportive/OutputFormatter/Configuration/ConfigurationGraphViz.php on line 13

 [ERROR]

         Output formatter graphviz-image threw an Exception:

         Message:
         Qossmic\Deptrac\Supportive\OutputFormatter\Configuration\ConfigurationG
         raphViz::__construct(): Argument #1 ($hiddenLayers) must be of type
         array, null given, called in
         phar:///my_path/vendor/qossmic/deptrac-shim/deptr
         ac/src/Supportive/OutputFormatter/Configuration/ConfigurationGraphViz.p
         hp on line 13

I'm at the newest version of Deptrac.
Here's the config:
parameters:
  paths:
    - ./src
  exclude_files:
    - 'Kernel.php'
  layers:
    - name: Application
      collectors:
        - { type: directory, regex: src/Application/.* }

    - name: Domain
      collectors:
        - { type: directory, regex: src/Domain/.* }

    - name: Infrastructure
      collectors:
        - { type: directory, regex: src/Infrastructure/.* }

  ruleset:
    Application:
      - Domain
    Domain: ~
    Infrastructure:
      - Application
      - Domain

looks like we need a release that contains #1251

@see https://github.com/qossmic/deptrac/issues/1243


Request for spanish School Colegio San José, Málaga

Official website:
https://fundacionloyola.com/sanjose/
I'm requesting JetBrains for 1st and 2nd course of DAM - Desarollo de Aplicaciones Multiplataforma (Multiplatform Application Development)

@MilawWork13 I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host fundacionloyola.edu.es not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


PicCap UI not working on 3.4

Hi,
Carrying on from some of the Discord chat over the weekend - now that hyperion-webos is fixed for 3.4.0 / the M16_DVB board.
While the UI loads, root status never loads, nor is the UI able to save or load configuration, or start/stop the service.

The only functioning button is the "Reboot TV" button.
"board_type": "M16_DVB",
"hardware_id": "HE_DTV_W16N_AFADABAA",
"product_id": "49UH750V-ZA",
"core_os_release": "3.4.0-5908",
"webos_manufacturing_version": "05.60.25"

WebOS homebrew 0.4.0 installed, root reported as "ok" in homebrew.

Note: service is launched when called and elevate-service is properly executed, but calls still error out with Denied method call for category {}...
We probably need some extra patching in /var/palm/ls2-dev.

From a newbie prospective, i can confirm that the issue is strictly ui related. I took a look to the script.js file and tried the various calls and they are all working correctly:
luna-send -n 1 -f luna://org.webosbrew.piccap.service/isRoot '{}'
{
    "rootStatus": true,
    "returnValue": true,
    "backmsg": "Running as root!"
}

luna-send -n 1 -f luna://org.webosbrew.piccap.service/isRunning '{}'
{
    "returnValue": true,
    "isRunning": true
}

still, from the ui, the situation is what @Informatic already described.
For now i started the service using
luna-send -n 1 -f luna://org.webosbrew.piccap.service/getSettings '{}'
to get the json config. Modified as needed a set with
luna-send -n 1 -f luna://org.webosbrew.piccap.service/setSettings '{"returnValue": true, "backend": "l ibdile_vt", "port": 19400, "width": 360, "height": 180, "captureUI": true, "ip": "<IP_ADDRESS>", "fps": 0, "backmsg": "S ettings got successfully!", "captureVideo": true, "autostart": true}'
then with
luna-send -n 1 -f luna://org.webosbrew.piccap.service/start '{}'
the service started with no problem
as said, the ui is still stuck with "Root status: Loading status..." and with the buttons not working returning the message @Informatic  already reported so that will definetly need a fix.

@PaoloVallesi Thanks so much for the info on starting it via luna-send. I managed to get it working, but the backlights only display white light, which dims and brightens according to the scene.
I have an Nvidia Shield with the Hyperion Grabber installed that works perfectly well with colour.
Any ideas why colour isn't working with the LG implementation?

Hello.
My WebOS 3.4 have same bug.
I am send this settings, but service  not start
# luna-send -n 1 -f luna://org.webosbrew.piccap.service/setSettings '{"backend": "l ibdile_vt", "port": 19400, "width
": 360, "height": 180, "captureUI": true, "ip": "<IP_ADDRESS>", "fps": 0, "backmsg": "S ettings got successfully!", "c
aptureVideo": true, "autostart": true}'
{
    "returnValue": true,
    "backmsg": "Settings set successfully!"
}

Then I am try to start, I am get error
# luna-send -n 1 -f luna://org.webosbrew.piccap.service/start '{}'
{
    "returnValue": true,
    "backmsg": "ERROR: Capture main init failed!"

For me there was a small mistake in  "backend" section (only a whitespace to much).
I've added a fix for me in "with" so probably you should edit this.
This one worked for me:
luna-send -n 1 -f luna://org.webosbrew.piccap.service/setSettings '{     "returnValue": true,     "backend": "libdile_vt",     "port": 19400,     "width": 320,     "height": 240,     "captureUI": true,     "ip": "XX.XX.XX.XX",     "fps": 0,     "backmsg": "Settings got successfully!",     "captureVideo": true,     "autostart": false }'
And to Start and Stop :
luna-send -n 1 -f luna://org.webosbrew.piccap.service/start '{}'
luna-send -n 1 -f luna://org.webosbrew.piccap.service/stop '{}'

For me there was a small mistake in "backend" section (only a whitespace to much). I've added a fix for me in "width" so probably you should edit this.
This one worked for me:
luna-send -n 1 -f luna://org.webosbrew.piccap.service/setSettings '{
    "returnValue": true,
    "backend": "libdile_vt",
    "port": 19400,
    "width": 320,
    "height": 240,
    "captureUI": true,
    "ip": "XX.XX.XX.XX",
    "fps": 0,
    "backmsg": "Settings got successfully!",
    "captureVideo": true,
    "autostart": false
}'

And to Start and Stop :
luna-send -n 1 -f luna://org.webosbrew.piccap.service/start '{}'

luna-send -n 1 -f luna://org.webosbrew.piccap.service/stop '{}'

Super - it is work.
Thanks

Hi guys,
are you staring these commands

from SSH or how?
If i start them from SSH i get no answer like...

For me there was a small mistake in "backend" section (only a whitespace to much). I've added a fix for me in "width" so probably you should edit this.
This one worked for me:
luna-send -n 1 -f luna://org.webosbrew.piccap.service/setSettings '{
    "returnValue": true,
    "backend": "libdile_vt",
    "port": 19400,
    "width": 320,
    "height": 240,
    "captureUI": true,
    "ip": "XX.XX.XX.XX",
    "fps": 0,
    "backmsg": "Settings got successfully!",
    "captureVideo": true,
    "autostart": false
}'

And to Start and Stop :
luna-send -n 1 -f luna://org.webosbrew.piccap.service/start '{}'

luna-send -n 1 -f luna://org.webosbrew.piccap.service/stop '{}'

Super - it is work. Thanks

Tried this didnt work for me
Kept getting this error
root@LGwebOSTV:~# luna-send -n 1 -f luna://org.webosbrew.piccap.service/start '{
}'
{
    "errorCode": -1,
    "returnValue": false,
    "errorText": "Message status unknown."
}

root@LGwebOSTV:~# grep -h -E '"(hardware_id|core_os_release|product_id|webos_man
ufacturing_version|board_type)"' /var/run/nyx/*
    "board_type": "M16_DVB",
    "hardware_id": "HE_DTV_W16N_AFADABAA",
    "product_id": "65UH652T-TA",
    "core_os_release": "3.4.0-5908",
    "webos_manufacturing_version": "05.60.25",

Please check out latest version (currently 0.4.1) this is tested working on webOS 3.4.


Visual improvements to orgs using D

dont hyphenate the uses
removed duplicate "hiring"

Can we change the text at the bottom?
"Is your organization missing from this page? Please create an issue or open a pull request."
Also, what order are the companies in? E.g. Rust goes with alphabetical.

- max. dims: 200x100 (resize with e.g. `mogrify -resize 200x100 image.png`)

Why? On high-DPI screens (mobile devices) the extra resolution will not go to waste.

Most logos are simple shapes which do not suffer in file size much at bigger resolutions.

- if you need commas, use `$(COMMA)`

Is this still needed after the move to `$0`?

Can we change the text at the bottom?

Sure. Thanks & done

Also, what order are the companies in? E.g. Rust goes with alphabetical.

Initially it was based on where I had content with similar height, but yes sorting alphabetically is fair to all. I will submit another PR ;-)

Why? On high-DPI screens (mobile devices) the extra resolution will not go to waste.

It sounded like a reasonable cut-off for me, but we can of course change that.

Currently the same images are used for the front page too.
the "real" size is 150x75 at the moment

if you need commas, use $(COMMA)

Yes, we use $0 only for the FA_QUOTES.

Yep, there should not be restrictions on logo size. In any case we should have the original in the repo, and maybe a scaled down version if necessary (see the OS logos for the download page).
In addition to benefiting high-DPI displays, a future site redesign might require logos of different resolution.

Yep, there should not be restrictions on logo size. In any case we should have the original in the repo, and maybe a scaled down version if necessary

Okay I added a better explanation.

(see the OS logos for the download page).

Aren't they also all less than 50x50 at the moment?

Aren't they also all less than 50x50 at the moment?

Ah yes, the higher res ones + conversion commands are in the git history.


add routing for archived global25/50 pages

dynamic data comes from gtm

@B77Mills PR is failing :)


Math magicians Milestone 4

Math magicians Milestone 4: navigation
In this milestone we:

Add a navbar to our project
Add nav functionality using react-router-dom
Add styling to our app.

Hello @ambrose-kibet 👋,
This is an additional review as a result of the second opinion request.
Looks like your project is already approved. Good job @ambrose-kibet  ✊
Keep rocking 🚀!


Controller Error

I upgraded CI version 2.0.3 to 2.1 And I Am Getting The Following Error
Fatal error: Call to undefined method CI_Loader::initialize() in /var/www/<folder_name>/system/core/Controller.php on line 51
I Check The Controller Coded I Have Written That Is Instead Of Class Name I Used "__construct" and extends "CI_Controller"
It Was Working Fine When It Was On Version 2.0.3 And Problem Arises After Version Upgrade.
I Am Using ckeditor.

It could be the double-slash in the path to the file.

I've seen weird caching issues when uploading files with ftp where I have to open and re-save files before the changes take. It's strange, I know, but it's happened to me more than once on shared hosting servers.
@rahulbalakrishna can you verify that there is a function initialize() at line 145 of /var/www/system/core/Loader.php
https://github.com/EllisLab/CodeIgniter/blob/master/system/core/Loader.php#L145

Any One Please Help Me Out.......... :(

I checked in the Loader.php and there is a function initialize() at line 145 and also checked whether for double-slash but there are no double slash in url...

Why don't you back to 2.0.3?

My Seniors Want It To Be Updated To Latest Version Of Ci.....And They Want The Site Up In Couple..............


feat: lwgenerate for aws controltower module

Summary
This change adds support for lacework-aws-cloudtrail-controltower module in the lwgenerate package and new CLI command.
The ControlTower module can be generated with the cmd:
lacework generate cloud-account aws controltower
And basic options provided in prompt input:

s3_bucket_arn
sns_topic_arn
log_archive_account
audit_account

Generation without prompt is supported with the --noninteractive flag and basic options provided as flags:
lacework generate cloud-account aws controltower --s3_bucket_arn arn:aws:s3:example --sns_topic_arn arn:aws:sns:example --audit_account AWSAdministratorAccess:us-east-1 --log_archive_account LogProfile:us-east-2
--noninteractive

This change supports an object type input org_account_mapping. The flag input supports a json string. Eg:
lacework generate cloud-account aws controltower --org_account_mapping '{"default_lacework_account":"main","mapping": [{ "aws_accounts": ["123456789011"], "lacework_account": "sub-account-1"}]}'

How did you test this change?

Test

cli/cmd/generate_aws_controltower_test.go
integration/aws_controltower_generation_test.go
lwgenerate/aws_controltower/aws_controltower_test.go

Run

[x] lacework generate cloud-account aws controltower prompt with basic options
[x] lacework generate cloud-account aws controltowe --noninteractive ... noninteractive with flags
[x] lacework generate cloud-account aws controltower prompt with advanced options

Issue
https://lacework.atlassian.net/browse/GROW-1526

lacework generate cloud-account aws controltower --noninteractive succeeds and generates an invalid main.tf leading to the following Terraform plan errors.
│ Error: Missing required argument
│
│   on main.tf line 22, in module "lacework_aws_controltower":
│   22: module "lacework_aws_controltower" {
│
│ The argument "sns_topic_arn" is required, but no definition was found.
╵
╷
│ Error: Missing required argument
│
│   on main.tf line 22, in module "lacework_aws_controltower":
│   22: module "lacework_aws_controltower" {
│
│ The argument "s3_bucket_arn" is required, but no definition was found.


Cleanup lint issues in Go code, improve error handling

This PR fixes a whole lot of general linting issues, adds extra error handling, fixes some typos and replaces the navdata generator with a more generic, recursive implementation. genref also received a new kube_deps.go to prevent a  go mod tidy from removing all the k8s dependencies.
If desired, I can break this down into individual PRs per app.

@xrstf Thank you for this huge effort. Please consider split the PR on a per-tool basis so we can quickly review them and get them in.

Split this PR into

https://github.com/kubernetes-sigs/reference-docs/pull/329
https://github.com/kubernetes-sigs/reference-docs/pull/330
https://github.com/kubernetes-sigs/reference-docs/pull/331
https://github.com/kubernetes-sigs/reference-docs/pull/332
https://github.com/kubernetes-sigs/reference-docs/pull/333


Make TLS validate mode configurable

In case when middleware is behind the firewall for the clients, one can use SSH port-forwarding to connect to it and to PuppetDB for nodes discovery.
However, if there is certificate mismatch, then mco client will hang and just won't work.
It would be cool to have OpenSSL::SSL::VERIFY_PEER in https://github.com/choria-io/mcollective-choria/blob/master/lib/mcollective/util/choria.rb configurable, so you can force it to VERIFY_NONE.

I am not really keen on this to be honest, the moment this toggle exist everyone will just set it and take the "easy" route to deployment - leading to no-one being secure.
We had this situation for years with mcollective where literally no-one deployed a secure mcollective setup because it was all too hard.  One of the major innovations and achievements of choria as regard to the deployment story is by making security easy and ubiquitous.
The middleware is a plain TCP connection - no SSL - it starts TLS once connected and what I would suggest is making a certificate with common names matching where you will connect, I believe this will work for the tcp port forward scenario too.

Right, that's what I thought too. It could still be useful to make it configurable in a way, so maybe higher verification modes can be used then.
Feel free to close then and thanks for quick feedback!

In reality it is configurable. You can turn off TLS completely and even choria protocol level PKI security - not sure about verification specifically but you might as well turn TLS off if you don’t use verification.
The way it’s adjustable is by taking the burden of packaging on yourself. You have to recompile the choria package and carry a tiny patch to the client - set 1 variable.
I recognise security isn’t always for everyone but I consider the official release merely a “distribution” of choria and others can exist for example for internal use. I make all the tools for having your own distribution available to you but it needs to be clear that your unique distribution isn’t something I could support. For example on my very large multi 100k node deploy I run my own distribution because I want to tweak some defaults and compile in proprietary plugins.
That’s the approach I am taking for the moment might change later but that’s where we are at.
We have made all the security stuff - paths to certs etc - settable soecifically not by puppet only magic so they should you be in a bind with cert names you can with relative easy adjust configuration to use certs with appropriate common names etc - happy to help you find those settings.


Add github ci config

Signed-off-by: Manoranjith<EMAIL_ADDRESS>

This looks fine, but the workflow is not triggered in this repository.

I was initially working on this from a fork on my repository. Hence, closing this before it is merged, in favor of #75.


Added Pause Button next to Timer

On clicking pause button it opens Pause menu with Resume, Menu and Quit buttons

Could we change the location of pause button? @pspatel04


Session#id is not always an integer even in Sessions endpoint

When Service sessions are enabled it can also be an UUID

@JohannesRabauer
Released in 1.2.3


Unpublishable assets

Hi!
I've installed the latest release (1.1.2), but when I publish the assets as it is in the installation guide, artisan says:
Can't locate path: <vendor/amidesfahani/filament-tinyeditor/src/../vendor/tinymce/tinymce>
Although I've checked, and in my vendor I have the path specified.

test again


Reactions should be put into last comment

And not the issue

Similar to the function "add_reaction_to_issue" in _github_utilities.py we need a function "add_reaction_to_last_comment_of_issue". In this function, the reaction is added to the last comment of an issue, in case it has comments. Otherwise, "add_reaction_to_issue" is called. In _terminal.py we should then call "add_reaction_to_last_comment_of_issue" instead of "add_reaction_to_issue".
git-bob solve

Your first response was incomplete. We need to edit two files for this: _github_utilities.py and _terminal.py . Try again!
git-bob solve

This message was generated by git-bob 0.1.2, an experimental AI-based assistant. It can make mistakes and has limitations. Check its messages carefully.
Comment by git-bob:

Great idea, @haesleinhuepf! I'll start by implementing the function `add_reaction_to_last_comment_of_issue` in _github_utilities.py. This function will check if the issue has comments, and if so, add the reaction to the latest one; otherwise, it will fall back to `add_reaction_to_issue`. I'll also update _terminal.py to use this new function instead. I'll submit a PR soon!

it obviously works now


Custom error extension prefix

Backport of https://github.com/knative-sandbox/eventing-kafka-broker/pull/2657

/cherry-pick release-v1.5

/assign @pierDipi
This is ready for review

  *[2022-09-19T13:14:36Z]NoClusters: No clusters in pool are ready to be claimed

/retest

/retest

/retest

/retest

/test 46-test-conformance-aws-ocp-46

/test 46-test-reconciler-aws-ocp-46

  *[2022-09-20T03:32:28Z]NoClusters: No clusters in pool are ready to be claimed

this is really annoying ...


Warning about multiple output types

Using the newest nightly:

warning: due to multiple output types requested, the explicitly specified output file name will be adapted for each output type

Suppressed in 7406ed2e41ae8f66a79fa01e6a3af080f8c7ae7b.


JWT UserInfo response ignored

The code at OidcProfileCreator.java, line 155 only processes information from the getUserInfo accessor, but the parse method called above explicitly sets the claimsSet property to null if the content is a JWT. I think there should be an else statement there that also checks the getUserInfoJWT accessor.

Indeed, it might be some missing use case. For which OIDC provider did you encounter the issue?
Would you mind submitting a pull request for that?

I'll try to get a PR up today.
I encountered this while playing around with the settings of a private Keycloak instance. The application kept breaking when I enabled JWT. After a bit of digging I noticed that claims were missing from the profile.