without jQuery, templates('#...') doesn't work.

https://jsfiddle.net/gvLrdysz/1/
jsrender.templates("#foo").render({})

without jQuery, the above just returns "#foo"

as an aside: the templates function seems weirdly overloaded. for example, when jQuery is loaded, the following returns nothing:
jsrender.templates("#### octothorps rule! ####").render({}));

as an aside: the templates function seems weirdly overloaded. for example, when jQuery is loaded, the following returns nothing:
jsrender.templates("#### octothorps rule! ####").render({}));

With jQuery loaded, $.templates(markupOrSelector) will first attempt treat the string markupOrSelector as a jQuery selector. If the selector returns an element, it will use the content of the element as markup for the template. If the string is not a valid selector, or is a selector which does not match any element, it will return a template with markupOrSelector as markup.
If jQuery is not loaded, then jQuery selectors are not used, and the string is treated immediately as markup.
See https://www.jsviews.com/#compiletmpl@fromscriptblock
By far the most common use case in the browser is with jQuery. But for advanced scenarios, or on the server, see https://www.jsviews.com/#nojqueryapi
Obviously in this case jQuery selectors don't apply. You need to use HTML APIs.  as in https://www.jsviews.com/#download/pages-jsr
For your second point, with jQuery loaded,
jsrender.templates("#### octothorps rule! ####").render({});
will  give an error that jsrender is not defined.
But with:
var html = $.templates("#### octothorps rule! ####").render({});
it will indeed give html = "#### octothorps rule! ####"

With jQuery loaded, $.templates(markupOrSelector) will first attempt treat the string markupOrSelector as a jQuery selector. If the selector returns an element, it will use the content of the element as markup for the template. If the string is not a valid selector, or is a selector which does not match any element, it will return a template with markupOrSelector as markup.
If jQuery is not loaded, then jQuery selectors are not used, and the string is treated immediately as markup.
See https://www.jsviews.com/#compiletmpl@fromscriptblock
By far the most common use case in the browser is with jQuery. But for advanced scenarios, or on the server, see https://www.jsviews.com/#nojqueryapi
Obviously in this case jQuery selectors don't apply. You need to use HTML APIs.  as in https://www.jsviews.com/#download/pages-jsr
For your second point, with jQuery loaded,
jsrender.templates("#### octothorps rule! ####").render({});
will  give an error that jsrender is not defined.
But with:
var html = $.templates("#### octothorps rule! ####").render({});
it will indeed give html = "#### octothorps rule! ####"

ok. you might want to clarify that in the readme on github, because it states that

The jsrender namespace provides the same methods/APIs as with jQuery, so if jQuery is not present you can still use all the API examples,

and further down it says

then, somewhere in your script:
var tmpl = $.templates("#myTemplate");

but nowhere that i can see does it mention that the code there requires jQuery.

ok. you might want to clarify that in the readme on github, because it states that

The jsrender namespace provides the same methods/APIs as with jQuery, so if jQuery is not present you can still use all the API examples,

and further down it says

then, somewhere in your script:
var tmpl = $.templates("#myTemplate");

but nowhere that i can see does it mention that the code there requires jQuery.

Yes, it makes sense to add a note in the docs/readme, in order to make it clearer... I'll look at doing that....

Yes, it makes sense to add a note in the docs/readme, in order to make it clearer... I'll look at doing that....

On further thought, I agree with you that reading the docs may make people expect to be able to pass an ID selector for a script block to $.templates("#myTmpl") even when jQuery is not loaded.
In addition, it is convenient, as an API.
So I am adding support for it, as a feature improvement,  in the next update.
I will also update the docs to indicate that other jQuery selectors can be used too, but only if jQuery is loaded.
Thanks for calling this out!

On further thought, I agree with you that reading the docs may make people expect to be able to pass an ID selector for a script block to $.templates("#myTmpl") even when jQuery is not loaded.
In addition, it is convenient, as an API.
So I am adding support for it, as a feature improvement,  in the next update.
I will also update the docs to indicate that other jQuery selectors can be used too, but only if jQuery is loaded.
Thanks for calling this out!

This has been resolved in v1.0.11.  See  https://www.jsviews.com/#jsr-quickstart@nojquery and<EMAIL_ADDRESS>

This has been resolved in v1.0.11.  See  https://www.jsviews.com/#jsr-quickstart@nojquery and<EMAIL_ADDRESS>


Add ethers.js integration example

What changed? Why?
I added a reference and link to ethers.js integration example. There's only examples / tutorials for onchainkit and viem and projects like ours use ethers.js instead so I thought I'd share the results.
Notes to reviewers
Not sure if this is the best place for the link but I think it would be useful to others.
How has it been tested?
Yes it's live on https://anybody.gg

Review Error for wbnns @ 2024-10-18 07:50:40 UTC
User failed mfa authentication, see go/mfa-help

Hello, apologies, since this links directly to a Gist we decided it would be better if it was included on the site as an existing doc (rather than having to leave). If you'd be interested in making this contribution as a tutorial, please let us know!


Added Conan Support

@eyalbe4 I have done the requested changes. Can you please review it again?
Thanks.

Amazing work @elioengcomp!
Thank you so much for this 😄

Thank you! ;)


Select [suggestion]

I see that you updated code in the select component just few hours ago. I checked also the bootstrap-select, which you referred as inspiration and really like the Title parameter, when you want to display some different (for example shorter) text than Label in the Select once an option is selected. This is what I mean:

Hot Dog, Fries and a Soda
Burger, Shake and a Smile
Sugar, Spice and all things nice

Would you think about integrating that in your component? If not, I might try it on my own.
Thanks!

About the select I don't integrate the label, in fact I removed justified, all that because that way you could do things like that: https://jsfiddle.net/wffranco/nyrt4q01/5/
If I add that features, then need too much validations and html or component duplication, making a bigger and very complex component, so I prefer to make it simpler but more flexible.


Support masking

Networking Packet Management in Gaming
In online, the handling of packets plays a crucial role in maintaining smooth gameplay and efficient network communication. One concern is the scenario where a player sends a packet that is then echoed back to them unnecessarily. While this may not immediately disrupt gameplay, it can lead to inefficiencies and potentially impact the player experience over time.
Potential Issues

Redundant Data: Receiving echoed packets can result in redundant data processing and network usage.
Bandwidth Consumption: Unnecessary packet echoing can consume additional bandwidth without providing significant benefits.
Latency: Increased traffic from echoed packets may contribute to higher latency in the gameplay experience.

Recommendations
To optimize packet management and network efficiency, it is advisable to implement a masking mechanism in the networking logic. This mechanism would selectively filter out packets that do not require a response back to the originating player. By integrating this masking mechanism into the broadcast logic, developers can tailor the packet flow to prioritize relevant data transmission.
In essence, by masking specific packets intelligently, game developers can streamline network traffic, reduce unnecessary data processing, and ultimately enhance the overall gaming experience for players.

Solved by #63


⚠ AGEG website (prod) has degraded performance

In 20890ac, AGEG website (prod) (https://www.ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 2039 ms

Resolved: AGEG website (prod) performance has improved in ab1dfd0.


New line is not rendered on chrome 101

What version of GoTTY are you using (gotty --version)?
gotty version 1.0.1
What operating system and browser are you using?
Ubuntu 20
What did you do?
./gotty --port 8081 -w /bin/bash
What did you expect to see?
A simple bash terminal in the browser.
What did you see instead?
It seems that the new line is not rendered by the browser (using chrome v101), so the output becomes trashed (see the image below).
I wonder how no one except me faced this issue, since I did not find similar issues anywhere.
Please not that I've tested the same thing on Firefox 100 and everything is ok.

I just noticed that the extension Dark Reader was causing the problem, so it was my mistake.


Improve errors for parse failure

Also properly serializes error messages sent over the socket.
The extra quotes surrounding every output are gone aswell.

@nwolverson @epost This will most likely break your plugins as it removes the unneeded quotes around the output.
Before:
echo "complete fil Project" | psc-ide
"filter, filterM"

After:
echo "complete fil Project" | psc-ide
filter, filterM

I will wait with merging until I hear your thoughts on the matter.

 The more relentless, the better, for now. :) I'll clean up the elisp.

allriight great! Here we go :D


Crash after switching from ES6 -> ES5 Mode

Still working on repro steps:
---------------------------
Assertion Failed: Abort=Quit, Retry=Debug, Ignore=Continue
---------------------------
An unhandled exception is about to terminate the process:

No such interface supported (Exception from HRESULT: 0x80004002 (E_NOINTERFACE))

System.InvalidCastException: No such interface supported (Exception from HRESULT: 0x80004002 (E_NOINTERFACE))

   at EnvDTE._DTE.get_Properties(String Category, String Page)

   at Microsoft.NodejsTools.SmartIndent.GetDesiredIndentation(ITextSnapshotLine line) in C:\src\nodejstools\Nodejs\Product\Nodejs\SmartIndent.cs:line 51

   at Microsoft.VisualStudio.Text.Editor.Implementation.SmartIndentationService.GetDesiredIndentation(ITextView textView, ITextSnapshotLine line)

   at Microsoft.VisualStudio.Text.Editor.Implementation.CaretElement.MapXCoordinate(ITextViewLine textLine, Double xCoordinate, Boolean userSpecifiedXCoordinate)

   at Microsoft.VisualStudio.Text.Editor.Implementation.CaretElement.MoveTo(ITextViewLine textLine, Double xCoordinate, Boolean captureHorizontalPosition)

   at Microsoft.VisualStudio.Text.Editor.Implementation.CaretElement.MoveTo(ITextView......

<truncated>
---------------------------
Abort   Retry   Ignore
---------------------------

Is this a dup of #1014? This also involved the smart indenter and a similar call stack if I recall, but I was seeing it in the interactive window instead.

Not sure - this brought down all of VS.

The crash location and local call stack are the same as #1014, so it's likely the exception is just not being handled in this case. I submitted a fix with the previous one with #1015.

Okay, let's try and get that fix in ASAP then because it's almost impossible to test the editor without it.

@mousetraps addressed comments on #1015 and merged in. Switching IntelliSense modes is working for me but please let me know if you are still seeing the issue.


I would like to have JavaScript ingestion parse spec

JavaScript.
JavaScript. JavaScript. JavaScript.
I would like to JavaScript data into Druid.

I :heart: JavaScript

Since we have javascript stop-gaps for other actionable items in druid, it makes sense to have a parse spec as well.

this exists now, see http://druid.io/docs/0.9.0-rc1/ingestion/data-formats.html


plugin: new: dffml-model-transformers

We created a new plugin, the GitHub repo is
https://github.com/dffml/dffml-model-transformers

(This is acctually a very old plugin, but you get the gist)


[FEA] Improve distributed api docs

Modulus Pull Request
Description
Improves API docs for the distributed API.
Checklist

[x] I am familiar with the Contributing Guidelines.
[x] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.
[ ] The CHANGELOG.md is up to date with these changes.
[x] An issue is linked to this pull request.

Dependencies

/blossom-ci

/blossom-ci


Fix codegen bug found via fuzzing

The problem was that Wasm instructions such as {i32,i64,f32,f64}.const, local.get and ref.null did not reset the InstrEncoder's last instruction thus not preventing invalid instruction fusion of i32.and and i32.eqz etc..


Expose total memory to the formatting directive

I would like to be able to see the memory usage expressed like 3.3G / 8.0G, which is not possible at the moment. This PR solves this.
I also noticed that there is no option to display the swap values. If you agree, I'd like to make a PR for that as well.
Note: Wiki page needs to be updated if this gets merged.

Thanks!
For sure, we might need #338 in order to avoid unneeded logic :)
Can you update the wiki?

Yeap, on it!


Array#pop and Array#shift should return nullables

Calling Array#pop and Array#shift on empty arrays returns undefined, so the return value of these functions should be annotated as nullable.

Thanks for the contribution and sorry for the delay! This looks good to me! I rebased this branch and switched ?T to T | void which is a little bit more precise. I also fixed the related Flow code. This change will cause lots of Flow errors in existing code, but it IS right so is probably worth it. I'll run it past the team, but I think they'll be onboard.
Thanks again!

Ok, so I've discussed this with the team and we're not going to take this PR for now. The main reason is that there is a lot of code like
while(arr.length > 0) {
  doSomething(arr.shift());
}

or
arr.length > 0 && doSomething(arr.pop());

If we took this PR as is, Flow would make Flow users check for undefined in the result of pop() and shift(), even if the user properly guarded against empty arrays.
Future work could result in a smarter Array type that carries a non-empty bit. Then we could refine the type of arrays with length checks and support the common JavaScript idioms. But for now, I'm going to close out this PR.
Thanks for your patience and understanding. Sorry this took so long to resolve!

@gabelevi If this is considered along with indexing, there are two arguments that could be made:

The general case is not an empty Array, but the absence of a value at the requested index.
In some theoretical context where the interpreter is multi-threaded, your examples are not "properly guarded against empty arrays", and the type check should be explicitly on the return value. All of these cases should be errors to be truly safe:

const foos: Array<number> = [1]
const foo: number = foos[1]

const bars: Array<number> = []
const bar: number = bars.pop()
const baz: number = bars.shift()

Particularly, I would like Flow to force me to write this:
const foos: Array<number> = [1]
const foo: number | void = foos[1]

const bars: Array<number> = []
const bar: number | void = bars.pop()
const baz: number | void = bars.shift()

I don't think your suggestion of a non-empty Array type would work for all of these cases, would it? Would it instead be possible to make this nullable behaviour opt-in for those who want Flow to enforce this safety?


ApiControllerAttribute's binding of complex parameter cannot be overridden.

I want to use the new [ApiController] attribute for the automatic '400 if model state is invalid behaviour', but still be explicit about the binding of parameters. For example:
[HttpGet]
public async Task<ActionResult> GetItems(
            CancellationToken cancel,
            [FromQuery]OffsetDateTime fromTime,
            [FromQuery]OffsetDateTime toTime)

I have a custom model binder for handling those NodaTime types, but they're receiving null as the model value, presumably because ASP.NET is still trying to read them from the body because of the ApiController attribute. If I remove the attribute it works again.
I had assumed that by being explicit about the binding source on the method, it'd override the  behaviour defined by a controller attribute, but this doesn't seem to be the case?
(As an aside, can I opt-in just the model state handling, but not the other behaviours of ApiController?)

Thanks for contacting us, @jamiewinder.
@pranavkm, does the ApiController convention take priority even in this case? Shouldn't custom model binders be respected for API controller actions?

@jamiewinder the FromQuery attributes should take precedence over auto inferred sources.
a) Have you tried setting LogLevel to Trace in your app? Usually it should give you hints about what's causing the issue.
b) A repro app would also help to dig in to this.
You could configure ApiBehaviorOptions.SuppressInferBindingSourcesForParameters to disable the binding source inference. That said, it would be great if we could get some of your feedback on why you'd like to disable it. We want the feature set in ApiController to be both intuitive and non-intrusive and intend on building further experiences (analyzers and code fixes) that assume these defaults.

a. I've tried calling the following on my IWebHostBuilder:
.ConfigureLogging(logging => logging.SetMinimumLevel(LogLevel.Trace))

Is that what you meant? It didn't seem to give me any extra information about model binding...?
b. I'll send one when I can.
I don't particular like the inference of where the parameters should come from, but I don't particularly mind so long as I can override it. In this case I'm using the NodaTime library rather than the BCL types, but we can see that complex types such as DateTimeOffset are being automatically treated as something very different to a OffsetDateTime when they are basically the same thing.
The ASP.NET team have wisely made it possible to extract a DateTimeOffset from the query or route, but to enforce the same behaviour on other types, just because they're 'complex' seems strange. I should add I'm still relatively new to ASP.NET Core so there may be a good reason for it.
Conversely, the behaviour of 'reject with a 400 Bad Request if any part of the model binding fails' seems obvious to me.
Again, I don't particularly mind if I can override it with an explicit FromQuery as I like to be an explicit as possible about this anyway.

Does this work:
.ConfigureLogging(factory =>
                {
                    factory
                        .AddConsole()
                        .AddDebug()
                       .SetMinimumLevel(LogLevel.Trace);
                })

I've set up a small reproduction here:
https://github.com/jamiewinder/AspNetCoreApiControllerIssue
Browsing to /api/withoutapi?time=2018-05-21T00:00:00Z should echo the time from the query.
Browsing to /api/withapi?time=2018-05-21T00:00:00Z will return a 400 with a model validation error
The code for the controllers is the same except for the ApiController attribute. I've also added a comment in Startup.cs around the ApiBehaviorOptions workaround you mentioned - it doesn't seem to have the expected effect?

With the logging setup as you suggested, I get the following:
To /withapi:
Microsoft.AspNetCore.Hosting.Internal.WebHost:Information: Request starting HTTP/1.1 GET http://localhost:44357/api/withapi?time=2018-05-21T00:00:00Z
Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker:Information: Route matched with {action = "Echo", controller = "WithApi"}. Executing action AspNetCoreApiControllerExample.Controllers.WithApiController.Echo (AspNetCoreApiControllerExample)
Microsoft.AspNetCore.Mvc.Infrastructure.ObjectResultExecutor:Information: Executing ObjectResult, writing value of type 'Microsoft.AspNetCore.Mvc.SerializableError'.
Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker:Information: Executed action AspNetCoreApiControllerExample.Controllers.WithApiController.Echo (AspNetCoreApiControllerExample) in 2.4333ms
Microsoft.AspNetCore.Hosting.Internal.WebHost:Information: Request finished in 11.4805ms 400 application/json; charset=utf-8

To /withoutapi:
Microsoft.AspNetCore.Hosting.Internal.WebHost:Information: Request starting HTTP/1.1 GET http://localhost:44357/api/withoutapi?time=2018-05-21T00:00:00Z
Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker:Information: Route matched with {action = "Echo", controller = "WithoutApi"}. Executing action AspNetCoreApiControllerExample.Controllers.WithoutApiController.Echo (AspNetCoreApiControllerExample)
Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker:Information: Executing action method AspNetCoreApiControllerExample.Controllers.WithoutApiController.Echo (AspNetCoreApiControllerExample) with arguments (2018-05-21T00:00:00Z) - Validation state: Valid
Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker:Information: Executed action method AspNetCoreApiControllerExample.Controllers.WithoutApiController.Echo (AspNetCoreApiControllerExample), returned result Microsoft.AspNetCore.Mvc.ObjectResult in 0.0611ms.
Microsoft.AspNetCore.Mvc.Infrastructure.ObjectResultExecutor:Information: Executing ObjectResult, writing value of type 'System.String'.
Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker:Information: Executed action AspNetCoreApiControllerExample.Controllers.WithoutApiController.Echo (AspNetCoreApiControllerExample) in 6.2025ms
Microsoft.AspNetCore.Hosting.Internal.WebHost:Information: Request finished in 13.0016ms 200 text/plain; charset=utf-8

Looks like you're running in to another side-effect of one of the issues we discovered late in with ApiControllers - https://github.com/aspnet/Mvc/issues/7609. Essentially ModelName gets lost in transition which results in the behavior you're seeing. I've filed an issue to port this to an upcoming patch.
In the meanwhile, here's a workaround you could use that should continue working once we'v addressed the issue. In your binder,
// Workaround for https://github.com/aspnet/Mvc/issues/7609. bindingContext.ModelName may be an empty string.
var modelName = bindingContext.ModelName;
if (string.IsNullOrEmpty(modelName) && bindingContext.IsTopLevelObject)
{
    modelName = bindingContext.ModelMetadata.ParameterName;
}

var val = bindingContext.ValueProvider.GetValue(modelName);

I'm going to close this as dup since we've already tracking the underlying issue (https://github.com/aspnet/Mvc/issues/7609 and https://github.com/aspnet/Mvc/issues/7804). Feel free to reopen if you have further questions.

I think this is still an issue in 2.1?

@jamiewinder unfortunately yes. We're addressing this in our upcoming 2.1.1 patch release.


Skip rendering sky if viewport is set to transparent background

In Godot 3, if we set the viewport to have a transparent background we would clear the environment used for that viewport skipping rendering things like the sky.
In Godot 4 we no longer do this as many settings in the environment still apply. However as we render the sky with alpha 0 (at least on Vulkan), the user was blissfully unaware the sky was being rendered and this took up a fair bit of resources.
This is because we're (ab)using the alpha for sub surface scattering.
The compatibility renderer doesn't do this and you do see the sky, which at least informs the user they may wish to reconfigure the background. This is something we may change in the future but we'll do that as a separate thing.
This PR resolves this issue by skipping rendering the sky if our viewport is set to transparent. We do still update the radiance map so you can have the sky contribute to ambient lighting.
Test project:
test-transparent-sky.zip

@clayjohn I've changed the logic so draw_sky and/or draw_sky_fog_only only are set to true if the background isn't transparent.
The radiance map will thus not be updated when the background is transparent unless:

the reflection source is set to background AND the background is set to sky,
the reflection source is set to sky,
or the ambient source is set to sky.

Hopefully that will give the right level of control.

Thanks!


EFSBackup runs into timeout when mounting EFS volumes

Hello
I am facing a timeout issue when applying the DataPipeline template for EFS Backups. Usually that effect suggests a wrong security group configuration, however, I manually launched an EC2 instance belonging to mySrcSecGroupID and myBackupSecGroupID and accessing both EFS volumes was OK.
Attaching the StdErr.log below.
Thanks,
Peter
--2016-09-22 08:52:24--  https://s3-us-west-2.amazonaws.com/XXXXXX/efs-backup.sh
Resolving s3-us-west-2.amazonaws.com (s3-us-west-2.amazonaws.com)... <IP_ADDRESS>
Connecting to s3-us-west-2.amazonaws.com (s3-us-west-2.amazonaws.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 2986 (2.9K) [application/x-sh]
Saving to: â€˜efs-backup.shâ€™
 0K ..                                                    100% 93.5M=0s

2016-09-22 08:52:24 (93.5 MB/s) - â€˜efs-backup.shâ€™ saved [2986/2986]
mount.nfs: Connection timed out
mount.nfs: Connection timed out
rm: cannot remove â€˜/tmp/efs-backup.logâ€™: No such file or directory

Thanks for your comment! Sorry to hear about the connection troubles. I'll take a look and let you know what I find.
-Dan

Thanks, that issue actually has resolved. Looks like I needed to give it a few hours after creating the EFS mount points before running it.
-Peter

Oh, awesome. Thanks for letting me know!


Show user's spotify playlists

Import one or all songs from playlist into database

Closed by pull request #99


Missing postgresql initial parameters

The postgresql.conf file should be included for reliable reproduction of this benchmark and validation of settings. Postgres is notorious for extremely conservative resource settings by default and does no inspection of the underlying platform so the settings in this file will define how the query planner represents the underlying system resources.

It's included in the postgresql archive (data folder). You can find it on the bottom of the blog post or here: https://s3.amazonaws.com/nosql-sample-data/postgresql-9.4.4.tar.bz2

Doh, I did indeed miss that final crucial section of the blog post. Thank you!


Vs code crashed reason clean exit code 0

Thanks for creating this issue! We figured it's missing some basic information, such as a version number, or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

Is everything okay now

Thanks for creating this issue! We figured it's missing some basic information, such as a version number, or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

Any steps to reproduce?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Can't single click select file entries in Safari

I've fixed this partially by re-focusing on blur which is occurring too fast, I think possibly because of this being a button and Safari having issues with focusing on those.
https://github.com/DustinBrett/daedalOS/commit/58dbc8238086ef073649eace3bde5c6a7a90aa96
This has fixed the selection issue but I will make a new issue for the fact I still can't drag now, single or multi.


Automatically add --manifest-path when -x and -C are combined

To preface: I'm not a cargo-watch user, but I've been in a discussion with one, helping them work around some issues where the sanest workaround is an out-of-tree build (TLDR: bind mounts in Docker on anything but Linux have longstanding performance issues, OP in the Reddit discussion cut their build time by something close to 20x).
What I proposed, as a solution to their issue, is a Dockerfile that looks like this:
RUN mkdir -p /build
WORKDIR `/src`
ENTRYPOINT cargo watch -C /build -x run --manifest-path=/src/Cargo.toml -- --your-arg

It seems reasonable to add that --manifest-path argument automatically if -C is present. Perhaps do not change the behavior of existing -x, but rather add a new flag, -X.

The current implementation of -C is actually aberrant, as it changes the workdir globally before doing anything. The intent was always to change it so the workdir was only changed for the command, so all the paths given on the command line and implied by defaults still make sense. As a result while the suggestion is appreciated I'm not going to introduce a new option to work around something that will go away soonish :)
Also off topic but I wouldn't recommend using both -x/-s and the -- command style, especially as complements to one another. It's an interesting pattern but only works incidentally, so it's probable it will break in the future, which is not great for scripts or commands baked into docker images.
Finally running cargo watch as PID 1 is not great at the moment, they might want to wrap a lightweight supervisor around it if they run into any issues! ...though I think ENTRYPOINT does this with a shell already in this syntax? I don't recall. Just something to keep in mind anyway.

As a result while the suggestion is appreciated I'm not going to introduce a new option to work around something that will go away soonish :)

It's a shame, since it seems like one of the very few possible workarounds for running cargo-watch in Docker with bind mounted source. Which isn't that unusual. But I understand trimming the cruft, and stuff that doesn't match the rest of the project.

Finally running cargo watch as PID 1 is not great at the moment, they might want to wrap a lightweight supervisor around it if they run into any issues! ...though I think ENTRYPOINT does this with a shell already in this syntax? I don't recall. Just something to keep in mind anyway.

Hah, good to know, I might pass it to OP in that Reddit thread.

I'll add some advice from this issue in the readme, that may help a bit.


Fix typo

a -> an

@changeworld : Thanks for your contribution! The author, @sethmanheim, has been notified to review your proposed change.

Merging typo fix.


chore: Find and replace '@platform' with 'atPlatform'

- What I did
Global replaced @platform with atPlatform
(and carefully went through each entry, I could not think of any problems that would result from these changes.)
- How I did it
- How to verify it
- Description for the changelog
chore: Find and replace '@platform' with 'atPlatform'

I will update the changelog and bump the version once the CI passes.


Add atuin

Description

PR created to add atuin sync bash history tool, requested in the issue https://github.com/alexellis/arkade/issues/903
Motivation and Context

[ ] I have raised an issue to propose this change, which has been given a label of design/approved by a maintainer (required)

How Has This Been Tested?

make e2e
=== RUN   Test_DownloadAtuin
--- PASS: Test_DownloadAtuin (0.00s)
=== RUN   Test_CheckTools/Download_of_atuin
=== PAUSE Test_CheckTools/Download_of_atuin
=== CONT  Test_CheckTools/Download_of_atuin
--- PASS: Test_CheckTools/Download_of_atuin (0.88s)

./hack/test-tool.sh atuin
+ ./arkade get atuin --arch x86_64 --os darwin --quiet
+ file /home/lbrdevcrypto/.arkade/bin/atuin
/home/lbrdevcrypto/.arkade/bin/atuin: Mach-O 64-bit x86_64 executable, flags:<NOUNDEFS|DYLDLINK|TWOLEVEL|PIE|HAS_TLV_DESCRIPTORS>
+ rm /home/lbrdevcrypto/.arkade/bin/atuin
+ echo

+ ./arkade get atuin --arch x86_64 --os linux --quiet
+ file /home/lbrdevcrypto/.arkade/bin/atuin
/home/lbrdevcrypto/.arkade/bin/atuin: ELF 64-bit LSB pie executable, x86-64, version 1 (SYSV), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, BuildID[sha1]=3244ac15a5e31ca1926a41befa0b1017c1048fa8, for GNU/Linux 3.2.0, stripped
+ rm /home/lbrdevcrypto/.arkade/bin/atuin
+ echo

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Documentation

[x] I have updated the list of tools in README.md if (required) with ./arkade get --format markdown
[ ] I have updated the list of apps in README.md if (required) with ./arkade install --help

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I've read the CONTRIBUTION guide
[x] I have signed-off my commits with git commit -s

[ ] I have tested this on arm, or have added code to prevent deployment

Hey thanks for the PR.
Please could you rebase all your commits down into one?
Here's how I do it: https://www.youtube.com/watch?v=8j0H6urZ-bU
Then - go ahead and do a final sign-off with git commit -s --amend
Alex

Sounds like the upstream project won't be adding Arm support any time soon, so we'll go ahead as is. Just run the rebase and ping me when done please.

Hi Alex,
I was able to follow your video (amazing tutorial) and rebase my commits.
If anything else needs to be done please let me know.
Regards.

Just for future reference please wrap the commit message as per - https://cbea.ms/git-commit/amp/ and use your real name in the sign off text via git config.


@theatre/r3f extension breaks with vite

the extension breaks and don't know why
https://stackblitz.com/edit/vitejs-vite-2f3xa3?file=src%2FApp.tsx&terminal=dev
any idea?

@MartinEmanuelMaldonado93 I am running into the same issue, how did you resolve it?

Yeah I've done the same and it works


security-setup script hangs on low entropy linux hosts due to /dev/random

/dev/random blocks in the following code segment when not enough true random entropy is available:
    try:
        with open('/dev/random') as ran:
            bytes = ran.read(size)

It may be worth migrating this to /dev/urandom to ensure we don't block, at the cost of lower quality randomness for the keys generated.
Users experiencing this issue will hang at the following part of the ./security-setup output:
----> Default ACL policy
set consul_default_acl_policy
=============== Marathon ===============
----> marathon framework authentication ```

Depending on the utilization, the conversation of /dev/random vs. /dev/urandom is likely incorrect. The issue typically isn't "lower quality", but how can those interfaces be attacked. Two good reads on the subject:
http://www.2uo.de/myths-about-urandom/
http://insanecoding.blogspot.com/2014/05/a-good-idea-with-bad-usage-devurandom.html
That said, I don't think we need these to be absolutely cryptographically secure, just highly unpredictable. Using /dev/urandom (or even Python PRNG) is likely sufficient for these purposes. Alternatively, we could take 256-bits as a seed from /dev/random, and run it through a more secure key expansion algorithm like arc4random:
https://github.com/rolandshoemaker/py-arc4random
(OpenBSD recently swapped the algorithm under their 'arc4random' implementation to use ChaCha20. http://arc4random.com/)

welp, I learned something about randomness sources today. I'll take care of this by grabbing values from os.urandom, since it'll work most everywhere. That's probably unpredictable enough to generate secret keys for.


Migrate tests mocks for Translator API v3

Modified the tests mocks to resolve using JSON instead of XML for correlation with the migration of Translator API v2 to v3.

Changes already merged into Microsoft/master in PR #322


Create "widgets-in-arenas" RFC

Rendered.

We haven't decided yet. Plan would be to make a formal decision next office hours.


Convert env-var string-lists to actual python-lists.

Allows for list input from environment-variables if correctly configured.
Previously always detected input as string-type

As in, I don't see how this check ever could have triggered before: https://github.com/NegatioN/ConfigArgParse/blob/db31e4de493be05afe2314c157bbcc96f9791d28/configargparse.py#L683

@bw2 I'm sorry to push you on this one, but what do you think about the addition?
We could really use it for an internal project if we're gonna keep using the library.

@NegatioN otherwise it looks great.
Thanks for implementing.

I would leave out nested lists and dicts - this adds too much complexity.
If people need this, they can parse it at the application level, or use yaml config files.

@bw2 made the changes you pointed out. And no problem! Appreciate the work you've put in to this as well. :)

Great, thanks!


Fixes for CUDA 12.4, METIS

Fixes #1564 #1568

Another part of code that fails with CUDA 12.4 is https://github.com/ginkgo-project/ginkgo/blob/develop/common/cuda_hip/multigrid/pgm_kernels.hpp.inc#L32-L48

When I applied the const_cast workarounds, I still can't build the full Ginkgo 1.7.0 as the Kokkos example now fails to build. I don't know if this is caused by CUDA 12.4 or Kokkos 4.1.00, though.
[1/3] Building CXX object examples/kokkos_assembly/CMakeFiles/kokkos-assembly.dir/kokkos_assembly.cpp.o
FAILED: examples/kokkos_assembly/CMakeFiles/kokkos-assembly.dir/kokkos_assembly.cpp.o
/usr/bin/c++ -DKOKKOS_DEPENDENCE -I/home/lahwaacz/ginkgo/build-cuda/include -I/home/lahwaacz/ginkgo/include -I/home/lahwaacz/ginkgo  -MD -MT examples/kokkos_assembly/CMakeFiles/kokkos-assembly.dir/kokkos_assembly.cpp.o -MF examples/kokkos_assembly/CMakeFiles/kokkos-assembly.dir/kokkos_assembly.cpp.o.d -o examples/kokkos_assembly/CMakeFiles/kokkos-assembly.dir/kokkos_assembly.cpp.o -c /home/lahwaacz/ginkgo/examples/kokkos_assembly/kokkos_assembly.cpp
/home/lahwaacz/ginkgo/examples/kokkos_assembly/kokkos_assembly.cpp: In lambda function:
/home/lahwaacz/ginkgo/examples/kokkos_assembly/kokkos_assembly.cpp:127:43: error: ‘abs’ is not a member of ‘Kokkos::Experimental’
  127 |             lsum += Kokkos::Experimental::abs(
      |                                           ^~~
/home/lahwaacz/ginkgo/examples/kokkos_assembly/kokkos_assembly.cpp:127:43: note: suggested alternatives:
In file included from /usr/include/c++/13.2.1/cstdlib:81,
                 from /usr/include/c++/13.2.1/ext/string_conversions.h:43,
                 from /usr/include/c++/13.2.1/bits/basic_string.h:4097,
                 from /usr/include/c++/13.2.1/string:54,
                 from /usr/include/c++/13.2.1/bits/locale_classes.h:40,
                 from /usr/include/c++/13.2.1/bits/ios_base.h:41,
                 from /usr/include/c++/13.2.1/ios:44,
                 from /usr/include/c++/13.2.1/ostream:40,
                 from /usr/include/c++/13.2.1/iostream:41,
                 from /home/lahwaacz/ginkgo/examples/kokkos_assembly/kokkos_assembly.cpp:33:
/usr/include/c++/13.2.1/bits/std_abs.h:56:3: note:   ‘abs’
   56 |   abs(long __i) { return __builtin_labs(__i); }
      |   ^~~
In file included from /usr/include/Kokkos_Complex.hpp:28,
                 from /usr/include/Kokkos_MathematicalSpecialFunctions.hpp:31,
                 from /usr/include/Kokkos_Core.hpp:54,
                 from /home/lahwaacz/ginkgo/examples/kokkos_assembly/kokkos_assembly.cpp:39:
/usr/include/c++/13.2.1/complex:887:5: note:   ‘std::abs’
  887 |     abs(const complex<_Tp>& __z) { return __complex_abs(__z.__rep()); }
      |     ^~~
/usr/include/Kokkos_Complex.hpp:677:33: note:   ‘Kokkos::abs’
  677 | KOKKOS_INLINE_FUNCTION RealType abs(const complex<RealType>& x) {
      |                                 ^~~
In file included from /home/lahwaacz/ginkgo/include/ginkgo/core/base/matrix_data.hpp:45,
                 from /home/lahwaacz/ginkgo/include/ginkgo/core/base/mtx_io.hpp:40,
                 from /home/lahwaacz/ginkgo/include/ginkgo/core/base/batch_multi_vector.hpp:45,
                 from /home/lahwaacz/ginkgo/include/ginkgo/core/base/batch_lin_op.hpp:43,
                 from /home/lahwaacz/ginkgo/include/ginkgo/ginkgo.hpp:43,
                 from /home/lahwaacz/ginkgo/examples/kokkos_assembly/kokkos_assembly.cpp:40:
/home/lahwaacz/ginkgo/include/ginkgo/core/base/range.hpp:727:49: note:   ‘gko::abs’
  727 | GKO_ENABLE_UNARY_RANGE_OPERATION(abs_operation, abs,
      |                                                 ^~~
/home/lahwaacz/ginkgo/include/ginkgo/core/base/range.hpp:648:9: note: in definition of macro ‘GKO_BIND_UNARY_RANGE_OPERATION_TO_OPERATOR’
  648 |         _operator_name(const range<Accessor>& operand)                       \
      |         ^~~~~~~~~~~~~~
/home/lahwaacz/ginkgo/include/ginkgo/core/base/range.hpp:727:1: note: in expansion of macro ‘GKO_ENABLE_UNARY_RANGE_OPERATION’
  727 | GKO_ENABLE_UNARY_RANGE_OPERATION(abs_operation, abs,
      | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /usr/include/c++/13.2.1/mutex:43,
                 from /usr/include/Serial/Kokkos_Serial.hpp:33,
                 from /usr/include/decl/Kokkos_Declare_SERIAL.hpp:21,
                 from /usr/include/KokkosCore_Config_DeclareBackend.hpp:22,
                 from /usr/include/Kokkos_Core.hpp:45:
/usr/include/c++/13.2.1/bits/chrono.h:457:7: note:   ‘std::chrono::abs’
  457 |       abs(duration<_Rep, _Period> __d)
      |       ^~~
In file included from /usr/include/c++/13.2.1/cstdlib:79:
/usr/include/stdlib.h:980:12: note:   ‘std::abs’
  980 | extern int abs (int __x) __THROW __attribute__ ((__const__)) __wur;
      |            ^~~
/usr/include/stdlib.h:980:12: note:   ‘std::abs’
/usr/include/stdlib.h:980:12: note:   ‘std::abs’
/usr/include/stdlib.h:980:12: note:   ‘std::abs’
/usr/include/stdlib.h:980:12: note:   ‘std::abs’
/home/lahwaacz/ginkgo/examples/kokkos_assembly/kokkos_assembly.cpp:129:39: error: ‘abs’ is not a member of ‘Kokkos::Experimental’
  129 |                 Kokkos::Experimental::abs(correct_u(xi)));
      |                                       ^~~
/home/lahwaacz/ginkgo/examples/kokkos_assembly/kokkos_assembly.cpp:129:39: note: suggested alternatives:
/usr/include/c++/13.2.1/bits/std_abs.h:56:3: note:   ‘abs’
   56 |   abs(long __i) { return __builtin_labs(__i); }
      |   ^~~
/usr/include/c++/13.2.1/complex:887:5: note:   ‘std::abs’
  887 |     abs(const complex<_Tp>& __z) { return __complex_abs(__z.__rep()); }
      |     ^~~
/usr/include/Kokkos_Complex.hpp:677:33: note:   ‘Kokkos::abs’
  677 | KOKKOS_INLINE_FUNCTION RealType abs(const complex<RealType>& x) {
      |                                 ^~~
/home/lahwaacz/ginkgo/include/ginkgo/core/base/range.hpp:727:49: note:   ‘gko::abs’
  727 | GKO_ENABLE_UNARY_RANGE_OPERATION(abs_operation, abs,
      |                                                 ^~~
/home/lahwaacz/ginkgo/include/ginkgo/core/base/range.hpp:648:9: note: in definition of macro ‘GKO_BIND_UNARY_RANGE_OPERATION_TO_OPERATOR’
  648 |         _operator_name(const range<Accessor>& operand)                       \
      |         ^~~~~~~~~~~~~~
/home/lahwaacz/ginkgo/include/ginkgo/core/base/range.hpp:727:1: note: in expansion of macro ‘GKO_ENABLE_UNARY_RANGE_OPERATION’
  727 | GKO_ENABLE_UNARY_RANGE_OPERATION(abs_operation, abs,
      | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/usr/include/c++/13.2.1/bits/chrono.h:457:7: note:   ‘std::chrono::abs’
  457 |       abs(duration<_Rep, _Period> __d)
      |       ^~~
/usr/include/stdlib.h:980:12: note:   ‘std::abs’
  980 | extern int abs (int __x) __THROW __attribute__ ((__const__)) __wur;
      |            ^~~
/usr/include/stdlib.h:980:12: note:   ‘std::abs’
/usr/include/stdlib.h:980:12: note:   ‘std::abs’
/usr/include/stdlib.h:980:12: note:   ‘std::abs’
/usr/include/stdlib.h:980:12: note:   ‘std::abs’

@lahwaacz Regarding the Kokkos example, yes that is due to changes in Kokkos 4.x. We are reworking the Kokkos example anway ATM, which will also fix this issue.

For info, this also came up in spack:

Sorry, I need to hold this up. From the comments, something like pgm also fails. I am compiling it with cuda 12.4 to ensure everything work

Yes, I didn't get around to fixing that one, should be an easy one as well, though.

Yes, it requires const_cast though.
Do you have any update from nvidia?
void compute_coarse_coo(std::shared_ptr<const DefaultExecutor> exec, ...
 {
     auto vals_it = as_device_type(vals);
     auto key_it =
-        thrust::make_zip_iterator(thrust::make_tuple(row_idxs, col_idxs));
+        thrust::make_zip_iterator(thrust::make_tuple(const_cast<IndexType*>(row_idxs), const_cast<IndexType*>(col_idxs)));

It's the last piece for compiling ginkgo with CUDA 12.4

The fix should be in the next release of CUDA https://github.com/NVIDIA/cccl/issues/1527

Nice! then probably guard this const_cast only with CUDA 12.4.


Process of collecting the dataset from publicly available information

Hi,
First of all, great work !
I just wanted to know how you are collecting the data files like attractions.csv or clean_flights_2022.csv etc. Our group wants to curate such a dataset for a different country, so we were wondering which websites or how you are procuring the data.

Hi,
Thanks for your interest in our work.
You could refer to Appendix A.3 of our paper, where we provide the details.
Feel free to contact us if you have further questions.
Best,
Jian

Hi,
Just another thing: How do you make the dataset for TravelPlanner's train/ val or test set? Do you manually make the natural language queries or get it from some website? And, do you generate the columns from the of the dataset by parsing the natural language query using GPT-4?
For train set, there is an "annotated_plan" column which is done manually as per your website. What is the use of "reference_information" column and how do you get that?

Hi,
For natural language query generation, please refer to Section 3.3 - Query Construction. We also provide codes here.
The "annotated_plan" is created by human annotators. We required the annotators to use tools for information gathering, and the information that ultimately contributes to the annotated plan is provided as "reference_information".


mariadb: 10.1.16 -> 10.1.17

Motivation for this change
Update / fix vulnerability / #18856
https://lwn.net/Vulnerabilities/700833/
Things done

[X] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] OS X
[ ] Linux

[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

PORT ME TO 16.09

In 16.09 as 0af39741ca.


inconsistent securityProfile implementation for privileged containers

Description
According to the cri-api code, for a privileged container the following rules apply:
// LinuxContainerSecurityContext holds linux security configuration that will be applied to a container.
message LinuxContainerSecurityContext {
    // Capabilities to add or drop.
    Capability capabilities = 1;
    // If set, run container in privileged mode.
    // Privileged mode is incompatible with the following options. If
    // privileged is set, the following features MAY have no effect:
    // 1. capabilities
    // 2. selinux_options
    // 4. seccomp
    // 5. apparmor
    //
    // Privileged mode implies the following specific options are applied:
    // 1. All capabilities are added.
    // 2. Sensitive paths, such as kernel module paths within sysfs, are not masked.
    // 3. Any sysfs and procfs mounts are mounted RW.
    // 4. AppArmor confinement is not applied.
    // 5. Seccomp restrictions are not applied.
    // 6. The device cgroup does not restrict access to any devices.
    // 7. All devices from the host's /dev are available within the container.
    // 8. SELinux restrictions are not applied (e.g. label=disabled).
    bool privileged = 2;

Notably:
    // 4. AppArmor confinement is not applied.
    // 5. Seccomp restrictions are not applied.

In practice, in code of containerd, seccomp and apparmor seem to be treated differently:

in generateSeccompSpecOpts the if privileged { is immediately checked, and the function returns nil, nil if so, no matter what was configured
in generateApparmorSpecOpts the if privileged { condition is only checked in the case runtime.SecurityProfile_RuntimeDefault branch; everywhere else the same behavior is kept as for a non-privileged container, this means that in case runtime.SecurityProfile_Localhost the provided profile will be applied even though the documentation says it shouldn't be

Steps to reproduce the issue
N/A
Describe the results you received and expected
Current behavior
Consider this matrix for the current behavior
                            |  privileged   |   non-privileged
================================================================
Seccomp    =Unconfined      |   Unconfined  |   Unconfined      |
           =RuntimeDefault  |   Unconfined  |   RuntimeDefault  |
           =Localhost       |   Unconfined  |   Localhost       |
================================================================
AppArmor   =Unconfined      |   Unconfined  |   Unconfined      |
           =RuntimeDefault  |   Unconfined  |   RuntimeDefault  |
           =Localhost       |   Localhost   |   Localhost       |

^ note the last row.
Behavior that would comply with specification
To comply with the spec the results should be
                            |  privileged   |   non-privileged
================================================================
Seccomp    =Unconfined      |   Unconfined  |   Unconfined      |
           =RuntimeDefault  |   Unconfined  |   RuntimeDefault  |
           =Localhost       |   Unconfined  |   Localhost       |
================================================================
AppArmor   =Unconfined      |   Unconfined  |   Unconfined      |
           =RuntimeDefault  |   Unconfined  |   RuntimeDefault  |
           =Localhost       |   Unconfined  |   Localhost       |

Alternative consistent behavior
The other option would be to align seccomp behavior with apparmor the other way around - still violating the spec, but being consistent. In that case, if Localhost type seccomp  profile is set, then it should be kept instead of reverting to unconfined:
                            |  privileged   |   non-privileged
================================================================
Seccomp    =Unconfined      |   Unconfined  |   Unconfined      |
           =RuntimeDefault  |   Unconfined  |   RuntimeDefault  |
           =Localhost       |   Localhost   |   Localhost       |
================================================================
AppArmor   =Unconfined      |   Unconfined  |   Unconfined      |
           =RuntimeDefault  |   Unconfined  |   RuntimeDefault  |
           =Localhost       |   Localhost   |   Localhost       |

Conclusion
From a security design perspective the last option makes the most sense actually - it stands to reason that users would want to use Localhost seccomp profiles on their privileged containers, just like they do with AppArmor.
There are a few workloads and applications which cannot live without privileged=true (eg.: they need write access to sysfs files for certain purposes) but other than a few narrow cases they do not need complete root access afforded by capabilities. For such workloads, many things should be forbidden despite being privileged and having the capabilities - eg. syscalls like setns or mount or access to host devices they do not need. AppArmor and seccomp are exactly for this purpose.
What version of containerd are you using?
containerd github.com/containerd/containerd v1.5.7 8686ededfc90076914c5238eb96c883ea093a8ba
Any other relevant information
No response
Show configuration if it is related to CRI plugin.
No response

/assign

By the way, an option that could be compatible with both the CRI spec and also support the increased security scenarios would be adding some sort of configuration flag to containerd to determine the behavior.
This option could be build time or runtime.
One idea is something like this (I am not familiar with how containerd is configured):
IgnoreAppArmorOnPrivilegedContainers:
  description: >-
    Specify what kinds of AppArmor settings should be ignored on privileged containers.
    According to the CRI specification, "AppArmor confinement is not applied" to privileged containers,
    so the CRI specification compliant choice is "IgnoreAll".
    Despite this, users and administrators may want to have AppArmor on privileged containers for extra security,
    so the behavior can be changed by providing "IgnoreAllExceptLocalhost"
    (to only apply the specified profile if its type is Localhost)
    or "IgnoreNone" (to always apply the specified profile).
  type: string
  enum: ["IgnoreAll", "IgnoreAllExceptLocalhost", "IgnoreNone"]
  default: "IgnoreAllExceptLocalhost"  # prefer backwards compatibility or should it be "IgnoreAll" to match CRI spec by default ?

(^ and a similar option for Seccomp)

deja-vu :-) https://github.com/containerd/cri/pull/281#discussion_r140793069 ..
was done this way because of the way it was originally implemented in dockershim.
I also prefer the Alternative "consistent" Behavior.  Would require a KEP, I think, to change the behavior of seccomp to support privileged with confinement.
If the behavior is consistent with test buckets between crio and containerd, we might want to "first" fix the CRI spec comments /docs to reflect current state. Then move forward with the KEP to move to privileged being confineable by specifying a localhost profile for the privileged container.

same with the enum.. I like it.. and that would need to be done through a KEP before we could change CRI defaults.  If you like we could work together to draft a KEP for this.


[Script]: Titanic Survival Prediction ML Model

Aim
Analyzing and predicting passenger survival on the Titanic using the CatBoost machine learning algorithm. The Titanic dataset contains passenger information such as age, gender, cabin class, fare, and more.
Details
The project involves preprocessing the dataset, handling missing values, converting categorical variables into numeric representations, and using the CatBoostClassifier to build a predictive model.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

/assign


enable cloudstack flc

Issue #, if available:
https://github.com/aws/eks-anywhere-internal/issues/1311
Description of changes:
Testing (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/lgtm

/approve


[SPARK-41548][CONNECT][TESTS] Disable ANSI mode in pyspark.sql.tests.connect.test_connect_functions

What changes were proposed in this pull request?
This PR proposes to disable ANSI mode in pyspark.sql.tests.connect.test_connect_functions temporarily.
This PR is sort of a followup of https://github.com/apache/spark/pull/38946 and https://github.com/apache/spark/pull/39071
Why are the changes needed?
There are failures in test_connect_functions with ANSI mode on (https://github.com/apache/spark/actions/runs/3709431687/jobs/6288067223). I tried to fix but they are tricky to fix because Spark Connect does not respect the runtime configuration at the server side.
It is also tricky to fix the test to pass in both ANSI mode on and off. Therefore, it disables temporarily to make other tests pass. Note that PySpark tests stop in the middle if one fails.
Does this PR introduce any user-facing change?
No, test-only.
How was this patch tested?
Manually verified that:
SPARK_ANSI_SQL_MODE=true ./python/run-tests --modules pyspark-connect -p 1

passes fine.

cc @xinrong-meng @zhengruifeng FYI


Document necessary secretsmanager permissions.

Supersedes #15.

I think in addition to the secrets_manager permissions, we also need to grant access to the KMS key being used. In our deployment we have the following policy:
{
  "Version": "2012-10-17",
  "Statement": [
    {
      "Effect": "Allow",
      "Action": [
        "kms:GenerateDataKey",
        "kms:Encrypt",
        "kms:Decrypt"
      ],
      "Resource": "<kms_key_arn>"
    }
  ]
}

While we're here, I wonder whether we should also update the elasticache permissions to recommend using the AWS-provided policy instead of a custom one (with arn arn:aws:iam::aws:policy/AmazonElastiCacheFullAccess). This additionally provides the service-linked role stuff that we ran into issues with.

@alext I thought those permissions were only required by Concourse? But I might be wrong.

If you use a custom KMS key you need to give kms:Decrypt access too: https://docs.aws.amazon.com/secretsmanager/latest/userguide/manage_retrieve-secret.html

@samcrang @bandesz I've drafted an alternate version of this update in the update-permissions-documentation-alternate branch. What do you think. If you're happy with that it can be cherry-picked/squashed into this branch...


Add test cases for index.js of context

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes
This PR covers several test cases for index.js of context.
Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the mailing list/slack team.
[x] I have run grunt to verify the unit tests pass
[x] I have added suitable unit tests to cover the new/changed functionality

Coverage increased (+0.03%) to 75.099% when pulling ca3da262da7ce343c02d5a8b1bf7dcb4e72454d8 on node-red-hitachi:context-test-index into 6bccdd015f347e6c3423a50fb02ac549c5a7e531 on node-red:master.


Replace Join Discord with Refresh Mod List, moved Discord server link to About

Quality of life change which makes leaving the mod loader on continuously feasible. By moving the Discord server link to the About tab, nothing is really lost here.
This could definitely be implemented better, however for now this works, I think.

Turns out this was useless thanks to a feature I honestly didn't even know existed


Appveyor build doesn't use generated identifier to tag packages

@CharliePoole spotted this in nunit/nunit-project-editor#16 - which I'd based off the cake script here.
The package names are built in a local variable at the start of the script, before the packageVersion var is modified in InitializeBuild. This then isn't reflected in the package names.

See https://ci.appveyor.com/project/CharliePoole/nunit-gui


Improve linter

Hello. I've come across an add-in which uses a similar VS Code like editor to AFE, and it has a much better linter (https://pages.store.office.com/addinsinstallpage.aspx?assetid=WA200001584&rs=es-ES&correlationId=2ca63351-445b-37ae-91ce-b498fcc38dea), even in the free non-AI version. For example it suggests replacing nested IF with IFS. There seems to be a lot of room in the AFE to improve on things like that.
Thanks!

By the way, not sure that's a good idea to replace nested IF with IFS...

By the way, not sure that's a good idea to replace nested IF with IFS...

Why not? (Honest, non-retorical Q.)

IFS evaluates all conditions before returns the result. Nested IF stops on first TRUE. From performance point of the the latest is more preferrable. Another point, IFS returns #N/A if no one condition met. IF returns FALSE. What is better depends on concrete scenario.

With last argument as TRUE we meet at least one condition. The question is in different behaviour if no one condition is met.

With last argument as TRUE we meet at least one condition. The question is in different behavior if no one condition is met.

Are these two not equivalent?

Okay, they return the same. My point was - with many conditions nested IF works faster, depends on priority of conditions.
In general, my impression AFE is died, as many other Garage projects. Perhaps it could be integrated into main Excel due to relatively high demand. Perhaps it will continue to survive with very limited or no support. Will see.

much better linter

Aside from the suggested formula rewrites, are there lint rules that stand out as being particularly useful? In AFE we do have comprehensive error and syntax checking.

In general, my impression AFE is died, as many other Garage projects.

We released our debugger in December! AFE is not dead, and Nick has be continually fixing reported bugs. However we do have limited bandwidth for new features based on current team priorities.

Aside from the suggested formula rewrites, are there lint rules that stand out as being particularly useful? In AFE we do have comprehensive error and syntax checking.

Take a look at these optimization rules. https://www.10studio.tech/docs/spreadsheet-optimizer#optimization-rules
I'd probably also suggest flagging INDIRECT and volatile functions due to their slowness, and suggest alternatives. There's probably room for SWITCH statements and other optimizations. But even automatically flattening IF statements would be a great help!
Also, do check out that guy's (several) addins, you'll find them very interesting, even if needing a few more minutes in the oven.

We released our debugger in December! AFE is not dead, and Nick has be continually fixing reported bugs. However we do have limited bandwidth for new features based on current team priorities.

It's great that you're continuing development, and I hope you get the resources needed to push forward.

This probably goes beyond a linter, but I wrote a macro that refactors formulas as LETs. Ie it takes each references and gives it a name in a LET. Then I F2-rename each. Super useful for reverse-engineering monster formulas.


Adding textfields to Alert View

Is it possible to add additional field to an Alert View? I tried but couldn't find how to apply, for example, the [UIAlertController addTextFieldWithConfigurationHandler] method.

RMUniversalAlert is really a lowest-common-denominator API polyfill for both UIAlertView and UIAlertController. I haven't added any polyfill functionality to bridge UIAlertController's addTextFieldWithConfigurationHandler: with UIAlertView's textFieldAtIndex: method. If you're targeting iOS8 and above - consider just using https://github.com/ryanmaxwell/UIAlertController-Blocks

I guess I will do that. Thanks a lot anyway for that great wrapper!

no prob!


Ensure StorageGRID collector works with password

Thanks to Dess on Discord for reporting!
22.08.1
They are getting the following error
"level":"warn","Poller":"gridname01","error":"invalid character 'I' looking for beginning of value => failed to unmarshal storage grid err","collector":"StorageGrid","object":"Tenant","caller":"poller/poller.go:665","time":"2023-11-07T15:25:29+01:00","message":"init collector-object"}
{"level":"warn","Poller":"gridname01","error":"invalid character 'I' looking for beginning of value => failed to unmarshal storage grid err","collector":"StorageGrid","object":"Prometheus","caller":"poller/poller.go:665","time":"2023-11-07T15:25:29+01:00","message":"init collector-object"}

with this config and a password that is like this %#8mdrUcPANDY#ei
gridname01:
    datacenter: "Location Here"
    addr: 10.x.x.x
    username: "user-name"
    password: "password"
    use_insecure_tls: true
    exporters:
      - prometheus1
    collectors:
      - StorageGrid

I could able to reproduce this issue locally in StorageGrid pollers, the culprit here is % character, which would error out while marshalling. Same error behaviour is not reproduced in case of Ontap pollers.

Use password as %#8mdrUcPANDY#ei (this is not actual password for that SG)
Tested with commit 166b0263
hardikl@hardikl-mac-0 harvest % ./bin/poller -p sg-tudor --promPort 18001
2024-02-13T13:35:02+05:30 INF poller/poller.go:215 > Init Poller=sg-tudor asup=false confPath=conf config=harvest.yml configPath=harvest.yml daemon=false debug=false homePath= hostname=hardikl-mac-0 logLevel=info logPath=/var/log/harvest/ profiling=0 promPort=18001 version="harvest version 24.02.1313-v23.11.0 (commit 166b0263) (build date 2024-02-13T13:30:28+0530) darwin/amd64"
2024-02-13T13:35:02+05:30 INF poller/poller.go:239 > started in foreground Poller=sg-tudor pid=3476
2024-02-13T13:35:06+05:30 WRN poller/poller.go:678 > init collector-object error="invalid character 'I' looking for beginning of value => failed to unmarshal storage grid err" Poller=sg-tudor collector=StorageGrid object=Tenant
2024-02-13T13:35:11+05:30 WRN poller/poller.go:678 > init collector-object error="invalid character 'I' looking for beginning of value => failed to unmarshal storage grid err" Poller=sg-tudor collector=StorageGrid object=Prometheus
2024-02-13T13:35:11+05:30 WRN poller/poller.go:306 > no collectors initialized, stopping Poller=sg-tudor
2024-02-13T13:35:11+05:30 INF poller/poller.go:531 > cleaning up and stopping [pid=3476] Poller=sg-tudor

Tested in main with commit febb62c5
hardikl@hardikl-mac-0 harvest % ./bin/poller -p sg-tudor --promPort 18001
2024-02-13T13:36:53+05:30 INF poller/poller.go:215 > Init Poller=sg-tudor asup=false confPath=conf config=harvest.yml configPath=harvest.yml daemon=false debug=false homePath= hostname=hardikl-mac-0 logLevel=info logPath=/var/log/harvest/ profiling=0 promPort=18001 version="harvest version 24.02.1313-v23.11.0 (commit febb62c5) (build date 2024-02-13T13:36:45+0530) darwin/amd64"
2024-02-13T13:36:53+05:30 INF poller/poller.go:239 > started in foreground Poller=sg-tudor pid=3845
2024-02-13T13:37:08+05:30 WRN poller/poller.go:687 > init collector-object error="auth failed" Poller=sg-tudor collector=StorageGrid object=Prometheus
2024-02-13T13:37:22+05:30 WRN poller/poller.go:687 > init collector-object error="auth failed" Poller=sg-tudor collector=StorageGrid object=Tenant
2024-02-13T13:37:22+05:30 WRN poller/poller.go:306 > no collectors initialized, stopping Poller=sg-tudor
2024-02-13T13:37:22+05:30 INF poller/poller.go:540 > cleaning up and stopping [pid=3845] Poller=sg-tudor


Unable to set some system properties

Gradle always prepends the D before the system property and assumes they are always key value pairs.
https://github.com/gradle/gradle/blob/master/subprojects/core/src/main/java/org/gradle/process/internal/JvmOptions.java#L107
https://github.com/gradle/gradle/blob/master/subprojects/core/src/main/java/org/gradle/process/internal/JvmOptions.java#L109
Some system properties aren't key value pairs, and some system properties don't start with -D but -XX.

-XX is VM arguments, they are not system properties.


Remove some parameters from config:scale

health move to another config file, not about scaling at all
nagle move to config:servers
gcInterval move to another config file
watchInterval move to another config file

gcInterval renamed to gc
watchInterval renamed to watch


Bump cchardet

Only cchardet==2.1.7 supports Python3.9
ref: https://github.com/PyYoshi/cChardet/releases/tag/2.1.7

Hi @q0w, thank you for your PR!
Please note that requirements.txt refers to development requirements. The CI pipeline validates Python 3.9, so there should be no compatibility issues with cchardet unless I am missing something.

Codecov Report

Merging #202 (00a6f21) into main (3dac548) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #202   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           48        48
  Lines         4384      4383    -1
=========================================
- Hits          4384      4383    -1

Impacted Files
Coverage Δ

blacksheep/server/controllers.py
100.00% <0.00%> (ø)

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 3dac548...00a6f21. Read the comment docs.

@RobertoPrevato so I tried install these requirements locally, and cchardet raises an exception

@RobertoPrevato so I tried install these requirements locally, and cchardet raises an exception

Maybe it depends on your platform? The build pipeline as you can see in the GitHub Workflows files, run and works for all tested versions. Anyway increasing the version of cchardet is perfectly fine. The setup.py file anyway is using ~=... which takes the greater minor version.

Thank You!


Implement RestGuildUser.Hierarchy

Added in https://github.com/Discord-Net-Labs/Discord.Net-Labs/commit/118de5dd1d3e40c3c12bf155e17dd8cfce714502


Rebuild bindings

You mentioned that verifications checks are split between JS and native code.  Is this function, and the others in that file, an example of what you are talking about?  Here is an example of a TS union type (Uint8Array/Buffer serialized key and PublicKey class instance) conversion for native use: Native Argument Validation Example


jupyter related packages updates 2024-01-20

Description of changes
Update jupyter-related packages at once to avoid building hundreds of packages over and over again.
This PR is based on the commits generated by the following command.
nix search nixpkgs#python311Packages 'ipython|jupyter|notebook' --json | jq -r 'keys | .[]' | cut -d . -f 3,4 | xargs -n 1 nix-update --commit

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[x] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

There are too many false failures this time due to the previous python-updates.
I suggest looking at the logs or using hydra-check.

BTW, we now know from https://github.com/NixOS/nixpkgs/issues/280539 that patch version updates can cause regressions in obscure parts of Jupyter that affect users. I'm not sure what we can do about this, since it seems to arise from the Jupyter team being insufficiently careful about semver constraints. I guess we could bump Jupyter stuff less frequently and/or apply some manual testing each time, as painful as that would be.

The jupyter package sometimes has security-related updates, as mentioned in the comments above.
So I don't want to update less often.
Also, regarding the version mismatch, I think we can prevent it to some extent by batch updates.
Until now, jupyter-related packages have been updated occasionally by r-ryantm or someone else.
This caused some packages to be older than expected upstream.
Also, before submitting this PR, I confirmed that the reported problem is not reproducible.
Now we only have to pay attention once a week, otherwise we would have to check packages several times a week, as the number of commits included in this PR indicates.

Hmmm, it's hard to see the log.
I'll split it into a couple of PRs.

Should I be looking at tearing off the jupyterlab bump on its own if there are complications here?

Doh I completely missed #283764

Sorry, I've been busy at work for a while.
I'll get on with this PR today.

Result of nixpkgs-review pr 282199 at ef37bad1 run on x86_64-linux 1

10 packages marked as broken and skipped:

piper-train
python311Packages.elegy
python311Packages.jupyter-contrib-nbextensions
python311Packages.kmapper
python311Packages.labelbox
python311Packages.openllm
python311Packages.scikit-tda
python311Packages.stytra
python311Packages.treex
python312Packages.nbdev

57 packages failed to build:

aitrack
coconut (python311Packages.coconut)
deface
livecaptions
monado
napari (python311Packages.napari)
obs-studio-plugins.obs-backgroundremoval
onnxruntime
onnxruntime.dev
opencomposite-helper
piper-phonemize
piper-tts
python311Packages.ax
python311Packages.bambi
python311Packages.cirq
python311Packages.dalle-mini
python311Packages.distrax
python311Packages.dm-haiku
python311Packages.fastembed
python311Packages.faster-whisper
python311Packages.ffcv
python311Packages.flax
python311Packages.glueviz
python311Packages.ibis-framework
python311Packages.ilua
python311Packages.insightface
python311Packages.jupyterlab-git
python311Packages.layoutparser
python311Packages.mmcv
python311Packages.nbdime
python311Packages.onnxruntime
python311Packages.pdfplumber
python311Packages.piper-phonemize
python311Packages.pydeck
python311Packages.pytorch-pfn-extras
python311Packages.qcodes-contrib-drivers
python311Packages.rlax
python311Packages.schemdraw
python311Packages.scikit-rf
streamlit (python311Packages.streamlit)
python311Packages.testbook
python311Packages.textacy
python311Packages.textnets
python311Packages.tf2onnx
python311Packages.vector
python311Packages.vqgan-jax
python311Packages.ziafont
python311Packages.ziamath
python312Packages.piper-phonemize
rocmPackages.mivisionx (rocmPackages.mivisionx-hip ,rocmPackages_5.mivisionx ,rocmPackages_5.mivisionx-hip)
rocmPackages.mivisionx-cpu (rocmPackages_5.mivisionx-cpu)
rocmPackages.mivisionx-opencl (rocmPackages_5.mivisionx-opencl)
sage
sageWithDoc
tts
whisper-ctranslate2
wolfram-notebook

132 packages built successfully:

cantor (libsForQt5.cantor ,plasma5Packages.cantor)
easyocr (python311Packages.easyocr)
ihaskell
inkcut
inkscape-extensions.inkcut
jupyter
jupyter-all
labplot
manim
nbqa
open-interpreter (python311Packages.open-interpreter)
opsdroid
python311Packages.altair
python311Packages.ansible-kernel
python311Packages.anywidget
python311Packages.arviz
python311Packages.atomman
python311Packages.augmax
python311Packages.bash-kernel
python311Packages.batchspawner
python311Packages.bqplot
python311Packages.bqscales
python311Packages.calysto
python311Packages.calysto-scheme
python311Packages.cirq-ft
python311Packages.comm
python311Packages.cufflinks
python311Packages.datashader
python311Packages.devito
python311Packages.dockerspawner
python311Packages.duckdb-engine
python311Packages.einops
python311Packages.encodec
python311Packages.experiment-utilities
python311Packages.fastai
python311Packages.folium
python311Packages.gradio
python311Packages.gradio-client
python311Packages.gradio-pdf
python311Packages.ipycanvas
python311Packages.ipydatawidgets
python311Packages.ipykernel
python311Packages.ipympl
python311Packages.ipynbname
python311Packages.ipyniivue
python311Packages.ipyparallel
python311Packages.ipytablewidgets
python311Packages.ipython-sql
python311Packages.ipyvue
python311Packages.ipyvuetify
python311Packages.ipywidgets
python311Packages.jupyter
python311Packages.jupyter-book
python311Packages.jupyter-c-kernel
python311Packages.jupyter-collaboration
python311Packages.jupyter-console
python311Packages.jupyter-contrib-core
python311Packages.jupyter-lsp
python311Packages.jupyter-nbextensions-configurator
python311Packages.jupyter-server
python311Packages.jupyter-server-fileid
python311Packages.jupyter-server-mathjax
python311Packages.jupyter-sphinx
python311Packages.jupyterhub
python311Packages.jupyterhub-ldapauthenticator
python311Packages.jupyterhub-systemdspawner
python311Packages.jupyterhub-tmpauthenticator
python311Packages.jupyterlab
python311Packages.jupyterlab-lsp
python311Packages.jupyterlab-server
python311Packages.jupytext
python311Packages.k-diffusion
python311Packages.lightparam
python311Packages.livelossplot
python311Packages.memestra
python311Packages.metakernel
python311Packages.mkdocs-jupyter
python311Packages.myst-nb
python311Packages.napari-console
python311Packages.nbclassic
python311Packages.nbconflux
python311Packages.nbconvert
python311Packages.nbdev
python311Packages.nbexec
python311Packages.nbsmoke
python311Packages.nbsphinx
python311Packages.nbval
python311Packages.nikola
python311Packages.nix-kernel
python311Packages.notebook
python311Packages.notebook-shim
python311Packages.notedown
python311Packages.oauthenticator
python311Packages.onnx
python311Packages.onnxconverter-common
python311Packages.onnxmltools
python311Packages.onnxruntime-tools
python311Packages.orange3
python311Packages.osmnx
python311Packages.phik
python311Packages.potentials
python311Packages.pweave
python311Packages.pygmo
python311Packages.pyls-memestra
python311Packages.pymc
python311Packages.pymoo
python311Packages.qcodes
python311Packages.qgrid
python311Packages.qtconsole
python311Packages.rotary-embedding-torch
python311Packages.skl2onnx
python311Packages.spacy
python311Packages.spacy-loggers
python311Packages.spacy-lookups-data
python311Packages.spacy-transformers
spyder (python311Packages.spyder)
python311Packages.spyder-kernels
python311Packages.thinc
python311Packages.tsfresh
python311Packages.unidic
python311Packages.unstructured-api-tools
python311Packages.vega
python311Packages.wasabi
python311Packages.weasel
python311Packages.xarray-einstats
python312Packages.comm
quarto
rocmPackages.migraphx (rocmPackages_5.migraphx)
rstudio
rstudio-server
rstudioServerWrapper
rstudioWrapper

It took too long time to check but there are no regressions.

Hi! It is broken for me
building '/nix/store/r31x88sqcn95d38bckszl0kjl218vgy0-python3.11-jupyter-server-2.12.5.drv'...
error: builder for '/nix/store/r31x88sqcn95d38bckszl0kjl218vgy0-python3.11-jupyter-server-2.12.5.drv' failed with exit code 1;
       last 10 log lines:
       > 1.00s teardown tests/auth/test_authorizer.py::test_authorized_requests[False-GET-/api/sessions-None]
       > 1.00s teardown tests/auth/test_authorizer.py::test_authorized_requests[True-POST-/api/kernels-None]
       > 1.00s teardown tests/auth/test_authorizer.py::test_authorized_requests[True-POST-/api/sessions-{"path": "foo", "type": "bar"}]
       > =========================== short test summary info ============================
       > SKIPPED [1] tests/test_terminal.py:161: Not yet working
       > SKIPPED [1] tests/test_terminal.py:198: Not yet working
       > SKIPPED [7] tests/conftest.py:42: Skipping this test because it's marked 'integration_test'. Run integration tests using the `--integration_tests` flag.
       > ERROR tests/auth/test_authorizer.py::test_authorized_requests[False-POST-/api/contents-{"type": "directory"}] - pytest.PytestUnraisableExceptionWarning: Exception ignored in: <socket.sock...
       > ====== 860 passed, 9 skipped, 10 deselected, 1 error in 69.61s (0:01:09) =======
       > /nix/store/bknngadwym46j65qs14ic2w79rpav888-stdenv-linux/setup: line 1582: pop_var_context: head of shell_variables not a function context
       For full logs, run 'nix log /nix/store/r31x88sqcn95d38bckszl0kjl218vgy0-python3.11-jupyter-server-2.12.5.drv'.

@spiage Thanks for letting me know.
I didn't see this failure in this PR, but I found it and fixed it in the following PR. https://github.com/NixOS/nixpkgs/pull/285959/commits/44b987c3ae55900bbfcc7e274ee3f7cf8e821151

Thank you!
I'll wait for it in master branch and let you know the result
=)

@natsukium I can't find this in nixpkgs and github show:

This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

This PR :) https://github.com/NixOS/nixpkgs/pull/285959


Fix invalid git.io/misspell link in Dockerfile

Fixes https://github.com/reviewdog/action-pyflakes/issues/22

Thanks!

Hi, @adrianlizarraga! We merged your PR to reviewdog! 🐶
Thank you for your contribution! ✨
We just invited you to join the @reviewdog organization on GitHub.
Accept the invite by visiting https://github.com/orgs/reviewdog/invitation.
By joining the team, you'll be a part of reviewdog community and can help the maintenance of reviewdog.
Thanks again!


Ng5 Migration

Hey @makimenko :)
This isn't an actual issue or anything with your example, more of a query:
Was just wondering if this example of yours would work the same if a migration to Angular 5 was done?
I was using something similar, by setting the TS compiler allowJS=true, and using a seperate JS file to load the ThreeJS library, bit since the migration on my project this doesn't exactly work as it used to with Ng4 👎
Cheers!

I got it to work! With minimal effort, I'd like to close this but I do have one query:
Why in your example don't you have the compiler allowJS set to true?

Hi @jarodsmk,
Thanks for hint.
Migrated to NG5 and allowJs settled to false.
https://github.com/makimenko/angular-three-examples/commit/9b8b2c115ac88f0d77afa7a65521d4a4398a543f
Tested -> workign well on my workstation.
Cheers!


feat: Support SHOW CREATE TABLE|EVENT|FUNCTION (MySQL specific)

Hello!
Thanks

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

38 of 60 (63.33%)  changed or added relevant lines in 3 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.3%) to 89.798%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/ast/mod.rs
8
17
47.06%

src/parser.rs
11
24
45.83%

Files with Coverage Reduction
New Missed Lines
%

src/ast/mod.rs
1
70.48%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
5607

Relevant Lines:
6244

💛  - Coveralls

Thanks @ovr !


[IPsec] Apply a patch to upgrade Python2 code in ovs ovs-monitor-ipsec

It is a workaround before upstream/ovs code update in ovs-monitor-ipsec.
#cat /var/log/antrea/openvswitch/ovs-monitor-ipsec.log
2020-08-06T10:38:36.455Z |  17 | ovs-monitor-ipsec | INFO | Tunnel stbed7-1-5b3e08 disappeared from OVSDB
2020-08-06T10:38:36.456Z |  18 | ovs-monitor-ipsec | INFO | Tunnel stbed7-2-b2a7f3 disappeared from OVSDB
2020-08-06T10:38:36.457Z |  19 | ovs-monitor-ipsec | INFO | Refreshing StrongSwan configuration
2020-08-06T10:38:36.573Z |  2  | daemon | INFO | pid 61 died, exit status 0, exiting
2020-08-06T10:38:53.25.Z |  0  | ovs-monitor-ipsec | INFO | Restarting StrongSwan
2020-08-06T10:38:55.201Z |  2  | reconnect | INFO | unix:/var/run/openvswitch/db.sock: connecting...
2020-08-06T10:38:55.202Z |  5  | reconnect | INFO | unix:/var/run/openvswitch/db.sock: connected
2020-08-06T10:38:55.284Z |  10 | ovs-monitor-ipsec | INFO | Tunnel stbed7-1-5b3e08 appeared in OVSDB
2020-08-06T10:38:55.285Z |  12 | ovs-monitor-ipsec | INFO | Tunnel stbed7-2-b2a7f3 appeared in OVSDB
2020-08-06T10:38:55.286Z |  14 | ovs-monitor-ipsec | INFO | Refreshing StrongSwan configuration

Thanks for the patch. But 2 things:

if you refer to a commit on your fork like this one, make sure you are never going to amend / remove that commit, or the reference will become invalid
I do not believe we should consider #1043 "fixed" until we make sure that the e2e tests for IPsec can detect such issues in the future

Edit: I also left a review comment for your OVS PR (openvswitch/ovs#331) after taking a look at the patch.

Make sense. I updated the commit link. The new one is on the branch which is not going to merge.
I removed "fixes" in this PR.

/test-all


Fixed #26165 -- Added some FAQs about CSRF protection.

https://code.djangoproject.com/ticket/26165

@apollo13, could you check this?

Thanks for the review, have updated for those comments.


Please add an option to tag an article

Thanks in advance for your tremendous support.
Now, that I found all untagged articles I like to tag articles as well.
I suggest doing something like
wallabag update <id> --addtag tag1,tag2,tag3
wallabag update <id> --deltag tag1,tag2,tag3

I think you can do it with 'tags' command.
24 мая 2021 г. 13:10:03 GMT+06:00, Dirk Deimeke @.***> пишет:

Thanks in advance for your tremendous support.
Now, that I found all untagged articles I like to tag articles as well.
I suggest doing something like
wallabag update <id> --addtag tag1,tag2,tag3
wallabag update <id> --deltag tag1,tag2,tag3

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/artur-shaik/wallabag-client/issues/11

--
Простите за краткость, создано в K-9 Mail.

Yes, that works.
Thanks a lot.


$state.params != $stateParams in resolve block

angular 1.3.15
ui-router 0.2.14
I'm not sure if this is expected behavior, but I found it kind of odd. That said, it is unclear to me what the difference is between the two.
		.state('auth.store', {
			url: '^/stores/:crock',
                        controller: 'StoresController',
			resolve: {
				store: ($state, $stateParams) => {
                                     $state.params.crock; // this is undefined
                                     $stateParams.crock; // this is defined
				}
			}
		})

The confusion here is that if I inject $state and $stateParams into StoresController, both of them have the value set correctly. I'd prefer to only have to inject one thing, so I pick $state since it would give me everything I need in one object. :smile:

The answer is that $state.params is a global, and $state's globals are only updated once the transition is finalized (since the transition could fail as a result of a resolve, which would leave your app in an inconsistent state). $stateParams, however, is a magical local injected value that represents the parameters received by just that state (and parent states).

:+1:
Definitely something to add to the documentation.

Updated the Wiki FAQ here.


Use rancher/mirrored-library-nginx 1.19.9-alpine

The image rancher/library-nginx:1.19.9-alpine does not exist.
Replaced with rancher/mirrored-library-nginx:1.19.9-alpine.
Grafana nginx image was not updated previously, it has now
been updated to use the new repo and version 1.19.9-alpine.
Issue:
https://github.com/rancher/rancher/issues/32010

@rmweir this can move forward.


Async http client change

Changes for async invocation of HTTP requests to read health of server instances using JDK11 java client (instead of JAX-RS HttpClient).

Note: I am not going to review this, I don't think I have the necessary domain knowledge.  If you feel you need me to review it, let me know.


How to override Upload scalar to MultipartFile?

I followed instructions in official doc. I created custom serializer which is called FileSerializer. It overrides Upload scalar to MultipartFile. But in advance, I get a null response when I use it to upload an image.
My custom serializer:
import 'package:built_value/serializer.dart';
import 'package:http/http.dart';
import 'package:http_parser/http_parser.dart';

class FileSerializer implements PrimitiveSerializer<MultipartFile> {
  @override
  MultipartFile deserialize(
    Serializers serializers,
    Object serialized, {
    FullType specifiedType = FullType.unspecified,
  }) {
    assert(serialized is List<int>,
        "FileSerializer expected 'List<int>' but got ${serialized.runtimeType}");
    return MultipartFile.fromBytes(
      'photo',
      serialized as List<int>,
      filename: '${DateTime.now().millisecondsSinceEpoch}.jpg',
      contentType: MediaType("image", "jpg"),
    );
  }

  @override
  Object serialize(
    Serializers serializers,
    MultipartFile file, {
    FullType specifiedType = FullType.unspecified,
  }) =>
      file;

  @override
  Iterable<Type> get types => [MultipartFile];

  @override
  String get wireName => "Upload";
}

uploadProfilePicture():
  Future<GUploadProfilePictureData> uploadProfilePicture(
      {required String userId, required http.MultipartFile file}) async {
    late final OperationResponse response;
    try {
      final uploadProfilePicture = GUploadProfilePictureReq((q) => q
        ..vars.userId = userId
        ..vars.file = file);
      response = await graphQlClient.request(uploadProfilePicture).first;
      return response.data;
    } catch (e) {
      print(response.graphqlErrors);
      throw Exception(e.toString());
    }
  }

And one last thing that bothers me: serialized as List<int> in the FileSerializer. This is the Uint8List which is gotten by calling readAsBytesSync () on a file. So, if we expect to serialize MultipartFile, why do we use serialized as List<int>? In this case, I assume that we actually serialize serialized as List<int> then use it to return MultipartFile. By the way, FileSerializer is an example that I found out in the ferry discord channel. I am not sure it is the correct approach to override MultipartFile

Any idea?

Hi @dokumanx , I have written an article in medium explaining how to do it: https://link.medium.com/gIJHjREi8hb

Hi @dokumanx , I have written an article in medium explaining how to do it: https://link.medium.com/gIJHjREi8hb

Hi, I have just checked it out. It seems detailed. I will try. Thanks for your reply.

yaw! Let me know if you achieve it or if I can help you

Hi @dokumanx , I have written an article in medium explaining how to do it: https://link.medium.com/gIJHjREi8hb

Hi, I have just checked it out. It seems detailed. I will try. Thanks for your reply.

yaw! Let me know if you achieve it or if I can help you

Closing this since it doesn't appear there is any issue.
Feel free to use the discord if you have other questions.


Use proper version string in tutorial templates

I am not sure if this is intention or a typo, but you have the r prefix in several of your templates even though you tell the reader not to put any prefixes in the version string.

"Do as I say, not as I do."
You seem to be doing some proofreading (if you could call it that) of my tutorials. I am thankful for your contributions. I wish for these tutorials to become a reliable resource for working with xbps-src online. I recognize that these highly technical works are more prone to errors. I've of course tried to limit these errors to a minimum, but having a second pair of eyes, which is willing to contribute, is certainly helpful.
If you'd like to contact me more directly, I am active on Libera Chat. You might (unsurprisingly) find me at #voidlinux and #xbps. Feel free to DM me.

You seem to be doing some proofreading (if you could call it that) of my tutorials.

Not really, I am currently working through your tutorials because I want to learn how to properly package software for Void, and the least I can do in return is fix the things I notice along the way.
In the past I could never wrap my head around packaging for Void because the only documentation we have is the manual, but the manual mainly a reference for people who already know what they are doing. Your tutorial is perfect starting point for people like me who already know how software works, but have never had to write actual OS packages and deal with all the different quirks.
Good documentation consists of four parts: tutorials, user guides, reference and explanation. Someone even came up with a fancy name for the concept: Diataxis. Of course depending on the kind of project not all of these are applicable, but the point stands. Your tutorial is really the piece that was missing from the documentation. I think it should be merged into the Void Packages documentation eventually.


Precompiled version

It would be nice if the funcs could use a precompiled regexp. For speed.

Do you know a way for using the precompiled regexp as Constant?

No, they're not constants, but I'm not sure why that is important. I would:

Keep the string constants as is today.
Create vars for the compiled regexps (just keep them unexported, I guess), create them in func init()
Use the vars from all the funcs.

Sounds good. Using the init() is a great idea.
So, I'll work for that soon. Thank you for giving a tip.

Thinking about it, you could just create them directly with the MustCompile func and drop the init.

@bep It is using the precompiled regexps now. : 8ba7634c336cdb5a9a6358683689eb05cd1c5a03


Dynamic Payment Gateway with Stripe and Razorpay for Streamlined Global and Localised Transactions

Current Situation
The old version of this website used Razorpay to process payments. It was good for Indian transactions because it supports all the major UPI apps (PhonePe, Paytm, Google Pe etc.). Recently, we expanded our platform to global audience and we've decided to migrate from Razorpay to Stripe for smooth processing of international payments and currencies. But, lately we started receiving feedback from our customers in India to integrate UPI for processing payments as that was the most preferred way to transact.
Desired Solution:
Implement a dynamic payment gateway solution that utilizes:

Select the payment gateway based on customer's geographic location.
Stripe for international transactions (outside India)
Razorpay for Indian transactions (UPI included)
This ensures a seamless user experience with localized payment options for all customers.

Benefits:

Enables acceptance of international payments through Stripe.
Provides Indian customers with the preferred UPI payment option via Razorpay.
Streamlines the checkout process by automatically routing users to the appropriate gateway based on their location.
Improves customer satisfaction by offering relevant payment methods.

Implementation Considerations:

Already implemented logic to determine the user's geographical location for selecting the currency.
Extend the logic to also select the appropriate payment gateway.
Select Stripe as the default gateway.
Select Stripe for Indian Customers only.

Hi, came across this amazing repository when playing around with Github search. Thanks a lot for making this open source, have learnt a lot by going through the code.
Coming to this issue, not sure if it is relevant or helpful, but sharing the links of another open source project which helps access payment ecosystems across 130+ countries, incase it is of any use

https://github.com/juspay/hyperswitch
https://docs.hyperswitch.io/hyperswitch-cloud/quickstart/migrate-from-stripe


Creating second environment in AZD causes hash conflict error on deployment

After successfully deploying one environment, I wanted to create a second environment with different settings.
> azd env new
> azd provision
Provisioning Azure resources (azd provision)
Provisioning Azure resources can take some time.

? Select an Azure Subscription to use: 95. ca-a
  (✓) Done: Retrieving locations...
? Select an Azure location to use:  9. (Asia Pacific) Japan East (japaneast)
Subscription: ca-a
Location: Japan East

  (✓) Done: Retrieving locations...
? Enter a value for the 'openAiResourceGroupLocation' infrastructure parameter:  2. (Asia Pacific) Japan East (japaneast)
? Save the value in the environment for future use Yes

  You can view detailed progress in the Azure Portal:
  https://portal.azure.com/#view/HubsExtension/DeploymentDetailsBlade/~/overview/id/%2Fsubscriptions%2F

  (✓) Done: Resource group: rg-pe-demo-different
  (✓) Done: App Service plan: plan-z75cxykbuw3co
  (✓) Done: Azure OpenAI: cog-z75cxykbuw3co
  (✓) Done: Form recognizer: cog-fr-z75cxykbuw3co
  (✓) Done: Storage account: stz75cxykbuw3co
  (✓) Done: Search service: gptkb-z75cxykbuw3co
  (✓) Done: App Service: app-backend-z75cxykbuw3co

ERROR: deployment failed: failing invoking action 'provision', error deploying infrastructure: deploying to subscription:

Deployment Error Details:
HashConflictOnDifferentRoleAssignmentIds: A hash conflict was encountered for the role Assignment ID. Please use a new Guid.

TraceID: ...

The error comes from Openai-role-user resource
A hash conflict was encountered for the role Assignment ID. Please use a new Guid. (Code: HashConflictOnDifferentRoleAssignmentIds)

@jongio FYI, might be related to another issue you reported

Related https://github.com/Azure/azure-cli/issues/23770

I don't understand why it would fail for this specific user, but not the others since they use the same logic to generate a GUID


[15721] Add/Drop Index

Group: Add Drop Index
Member: Xingyu Jin, Rong Huang, Ziheng Liao
Please check the following files:
/src/concurrency/lock_manager.cpp (lock manager, please check the whole file)
/src/include/concurrency/lock_manager.h (lock manager, please check the whole file)
/src/executor/create_executor.cpp (please check the function CreateIndex() )
/src/executor/drop_executor.cpp (please check the function DropIndex() )
/src/executor/*_executor.cpp (some other additions to various executors, no single functions. Please comment if you want to see those changes.)
/test/concurrency/lock_manager_test.cpp (tests for lock manager, please check the whole file)
/test/concurrency/testing_transaction_util.cpp (support functions for index create and drop)
/test/include/concurrency/testing_transaction_util.h (header for support functions for index create and drop)
/test/concurrency/transaction_index_test.cpp (tests for index create and drop, both in one thread and multiple threads)

Coverage increased (+0.08%) to 77.049% when pulling 781118b6b3995b866fad1635b60e15419f80a5da on alphalzh:master into 0251ba3b907a6dcdb1fba4d4de3b51ddc3dbc469 on cmu-db:master.

Replaced by #1364


[PostitTrampoline] Automatically enable when target version is >= 2

See https://github.com/bundler/bundler/issues/4853

@homu r+

:pushpin: Commit 82d5938 has been approved by indirect

:hourglass: Testing commit 82d5938 with merge a87522b...

:sunny: Test successful - status


Adopt LLVM rL314993 to fix application build error issues

Original patch:
https://reviews.llvm.org/rL314993

make test result after applying this patch:
********************
Testing Time: 438.64s
********************
Failing Tests (1):
    HCC :: Unit/DynamicTileStatic/test11.cpp