"Json::encode" different between version 2.0.38 and version 2.0.40

What steps will reproduce the problem?
build two project, example yii2-2.0.38 and yii2-2.0.40
Model.php
namespace App;

class Model extends \yii\base\Model implements \JsonSerializable
{
	protected $class;

	public $r;

	public function __construct(array $r)
	{
		parent::__construct([
			'r' => $r
		]);
	}

	public function jsonSerialize()
	{
		$class = $this->class;
		return array_map(function ($r) use ($class) {
			return new $class($r);
		}, $this->r);
	}
}

InterestTestTopicListVo.php
namespace App;

class InterestTestTopicListVo extends Model
{
	protected $class = InterestTestTopicVo::class;
}

InterestTestTopicVo.php
namespace App;

class InterestTestTopicVo extends \yii\base\Model
{
	public $id;
	public $question;
	public $answer;

	public function __construct($config = [])
	{
		parent::__construct([
			'id' => $config['id'],
			'question' => $config['question'],
			'answer' => new InterestTestTopicAnswerListVo($config['answer']),
		]);
	}
}

InterestTestTopicAnswerListVo.php
namespace App;

class InterestTestTopicAnswerListVo extends Model
{
	protected $class = InterestTestTopicAnswerVo::class;
}

InterestTestTopicAnswerVo.php
namespace App;

class InterestTestTopicAnswerVo extends \yii\base\Model
{
	public $no;
	public $text;
	public $score;
}

index.php
require __DIR__ . '/vendor/autoload.php';
require __DIR__ . '/vendor/yiisoft/yii2/Yii.php';

$array = [
	[
		'id' => 1,
		'question' => 'abc',
		'answer' => [
			[
				'no' => 'A',
				'text' => 'hahaha',
				'score' => 3
			],
			[
				'no' => 'B',
				'text' => 'hahaha',
				'score' => 5
			],
		]
	],
	[
		'id' => 2,
		'question' => 'abcdefg',
		'answer' => [
			[
				'no' => 'A',
				'text' => 'hahaha',
				'score' => 3
			],
			[
				'no' => 'B',
				'text' => 'hahaha',
				'score' => 5
			],
		]
	]
];

var_dump(\yii\helpers\Json::encode(new \App\InterestTestTopicListVo($array)));

What is the expected result?
in 2.0.38
string(227) "[{"id":1,"question":"abc","answer":[{"no":"A","text":"hahaha","score":3},{"no":"B","text":"hahaha","score":5}]},{"id":2,"question":"abcdefg","answer":[{"no":"A","text":"hahaha","score":3},{"no":"B","text":"hahaha","score":5}]}]"
in 2.0.40
string(227) "[{"id":1,"question":"abc","answer":[{"no":"A","text":"hahaha","score":3},{"no":"B","text":"hahaha","score":5}]},{"id":2,"question":"abcdefg","answer":[{"no":"A","text":"hahaha","score":3},{"no":"B","text":"hahaha","score":5}]}]"
What do you get instead?
in 2.0.38
string(227) "[{"id":1,"question":"abc","answer":[{"no":"A","text":"hahaha","score":3},{"no":"B","text":"hahaha","score":5}]},{"id":2,"question":"abcdefg","answer":[{"no":"A","text":"hahaha","score":3},{"no":"B","text":"hahaha","score":5}]}]"
in 2.0.40
string(239) "[{"id":1,"question":"abc","answer":{"r":[{"no":"A","text":"hahaha","score":3},{"no":"B","text":"hahaha","score":5}]}},{"id":2,"question":"abcdefg","answer":{"r":[{"no":"A","text":"hahaha","score":3},{"no":"B","text":"hahaha","score":5}]}}]"
Different in attribute 'answer', in 2.0.38 is a json array but in 2.0.40 is a json object.
Additional info

Q
A

Yii version
2.0.38, 2.0.40

PHP version
php7.3

Operating system
MacOS

Hmm, this may be because of https://github.com/yiisoft/yii2/pull/18437

Hmm, this may be because of https://github.com/yiisoft/yii2/pull/18437

@JimChenWYU Could you try with 2.0.37 and see if it is the same result as 2.0.40?

@JimChenWYU Could you try with 2.0.37 and see if it is the same result as 2.0.40?

@schmunk42 No, same as 2.0.38.

@schmunk42 No, same as 2.0.38.

I'm out of ideas how to fix that. Some wants Arrayable to take precedence, some wants JsonSerializable. There won't be an easy way out.
@JimChenWYU some semi-temp solution for you might be to add
public function toArray(array $fields = [], array $expand = [], $recursive = true)
{
    return parent::toArray($fields, $expand, false);
}

in InterestTestTopicVo class.

I'm out of ideas how to fix that. Some wants Arrayable to take precedence, some wants JsonSerializable. There won't be an easy way out.
@JimChenWYU some semi-temp solution for you might be to add
public function toArray(array $fields = [], array $expand = [], $recursive = true)
{
    return parent::toArray($fields, $expand, false);
}

in InterestTestTopicVo class.

@bizley Thanks. In fact, I had solved my problem. Just In Base 'Model.php' add
public function toArray(array $fields = [], array $expand = [], $recursive = true)
{
	return $this->jsonSerialize();
}

But I'm not sure it's a bug or not. If it's not a bug, it maybe was told in CHANGELOG.md or UPGRADE.md.

@bizley Thanks. In fact, I had solved my problem. Just In Base 'Model.php' add
public function toArray(array $fields = [], array $expand = [], $recursive = true)
{
	return $this->jsonSerialize();
}

But I'm not sure it's a bug or not. If it's not a bug, it maybe was told in CHANGELOG.md or UPGRADE.md.

Yes, it should be mentioned in the UPGRADE but somehow we neglected it when merging. I'll keep this open to remember to add it.

Yes, it should be mentioned in the UPGRADE but somehow we neglected it when merging. I'll keep this open to remember to add it.

@bizley Thanks more, I think your solution is better than me. :smiley:

@bizley Thanks more, I think your solution is better than me. :smiley:


Question about shared validation logic client side and plugin (server-side)

Hello.
From sparklexrm.com I read that:

Sparkle XRM is designed to enable you to write C# code and compile to JavaScript and Plugin Code at the same time. Write your validation logic once and use it when your users enter data into standard CRM forms, custom Webresources or called from Plugins when updating via Bulk Upload or Bulk Updates.

@ScottColson, Could you please provide guidance on how do we do it, that is:

In which project do I put my shared logic?
Should I still use the Script Class Item template when writing this shared code?
Can we also use SparkleXrm validation framework there?
Anything else we should know?

SparkleXrm looks a well-written framework, however really lacks documentation.

You can put your shared code in either the server or the client project and then include to the other using link rather than add in Visual Studio.
You can then use conditional code to include a different set of includes if need be:
#if SCRIPTSHARP

#else

#endif

Using this same technique you can share constant definitions or code that can be compiled both server and client - simple business logic and fetchxml query generation functions work well.
You can't use the Validation Data Binder at this time on the Server but that sounds like an interesting idea.
We are always looking for help with documentation.


Language Client: Implement signature help

Implement signature help functionality: https://github.com/Microsoft/language-server-protocol/blob/master/protocol.md#textDocument_signatureHelp

Based on what the language server protocol gives us for this, the signature help UI might need to be restyled.
The typescript language service gives us separator characters for each parameter. However, the language server just gives the function name, a description, and argument info. A better format for representing that might be a bulleted list, and the selected argument can be highlighted.


Custom variables in front matter

After adding custom variables, in this case image = 'path/my_filename.jpg' , in the front matter, and adding  {{ page.image }} to _default/page.html.twig, I get the following error
In PagesRender.php line 155:                                                                                [Cecil\Exception\Exception]                                                                                Template "site:_default/page.html.twig" line 26 (for page "another-page"): Neither the property "image" nor one of the methods "image()", "getimage()"/"isimage()"/"hasimage()" or "__call()" exist and have public access in class "Cecil\Collection\Page\Page".
Could the documentation include a section on how to make custom variables in the markdown file's front matter accessible to templates?

Hello @haraldgroven, as you can read in the Page Anatomy section of the documentation, the front matter must be a valid YAML:
image = 'path/my_filename.jpg' -> image: '/path/my_filename.jpg'.
Could you try this? Thanks.

Sorry for typo in image: 'path/my_filename.jpg'
I have checked that all markdown content pages are valid YAML according to yamllint. However, after adding {{ page.image }} to _default/page.html.twig cecil expects every single markdown page to have the image: key/value pair, and throws exception even for the 404 page.
[Cecil\Exception\Exception] Template "site:_default/page.html.twig" line 26 (for page "404"): Neither the property "image" nor one of the methods "image()", "getimage()"/"isimage()"/"hasimage()" or "__call()" exist and have public access in class "Cecil\Collection\Page\Page".

I see: you should add a control in your template.
Something like that:
{% if page.image is defined %}
<img src="{{ url(page.image) }}" alt="{{ page.title }}" />
{% endif %}


DiagnosticScopeValidatingInterceptor: Allow methods with no client calls

We're adding a protected virtual async method that doesn't make any client calls.  [ForwardsClientCalls] lets us say that nothing with its name should ever appear, but since no client calls are ever made the interceptor complains that no scopes were created.  We should add something like a [NoClientCalls] for niche scenarios like this.

Please reopen if there is still a need.


Remove Stripe internal type from signature

Replacing Chalk::Tools::RedactionUtils::RedactionDirectiveSpec with T.untyped in the signature of T::Props::Decorator#handle_redaction_option.
Motivation
Currently T::Utils.run_all_sig_blocks method call fails for every codebase other than Stripe's due to the Chalk::Tools::RedactionUtils::RedactionDirectiveSpec type in one of the signatures in sorbet-runtime.
Test plan

No tests, do you want me to add one?

Gentle bump to get this merged. Thanks


Reset flags when resetting an Arcade Body

This PR

Fixes a bug

Which bug does this fix?

If you call Body#reset(),

blocked and touching may be incorrect until the first physics step after reset
wasTouching may be incorrect until the second physics step after reset

👍


Merge branch 'release-2.1.0' into ldap_pagination

Signed-off-by: alexandrovas<EMAIL_ADDRESS>

Codecov Report

:exclamation: No coverage uploaded for pull request base (release-2.1.0@9bdaa9f). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             release-2.1.0       #2   +/-   ##
================================================
  Coverage                 ?   51.68%
================================================
  Files                    ?      276
  Lines                    ?    13628
  Branches                 ?     2017
================================================
  Hits                     ?     7044
  Misses                   ?     5893
  Partials                 ?      691

Flag
Coverage Δ

unittests
51.68% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Codecov Report

:exclamation: No coverage uploaded for pull request base (release-2.1.0@9bdaa9f). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             release-2.1.0       #2   +/-   ##
================================================
  Coverage                 ?   51.68%
================================================
  Files                    ?      276
  Lines                    ?    13628
  Branches                 ?     2017
================================================
  Hits                     ?     7044
  Misses                   ?     5893
  Partials                 ?      691

Flag
Coverage Δ

unittests
51.68% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.


Multiple Sprite3D being rendered in weird order

Systems tested:
http://pasted.co/ca4970b9
http://pasted.co/d1a10e38
Version: v2.1.2.stable.official
Issue description:
Placing a big Sprite3D in front of a small Sprite3D may render the big one in front of the small one, depending on the view angle.
Adding a TestCube between both sprites creates a curious side-effect, here the big sprite on the back is rendered in front of the small sprite, except where the latter overlaps the TestCube, where the small sprite is rendered as expected.
It happens on both the editor and when running.
Steps to reproduce:

Link to minimal example project:
Issue.tar.gz

Hm, not sure if you can change the sprite3d's material, but if you can, could you try to set it's "depth draw" to "always" or to "pre-pass alpha"?

Using a material override and using those settings seems to work.

(don't mind the corners, alpha was disabled on the material override)
Still, it is definitely not the behavior I expected, when I used the two Sprite3D, where one is in front of the other. I would expect that in case those were billboards, when the small sprite is overlapped by the big one at around the same angle...
If I change the diffuse parameters on the material to add alpha this is what happens:

The left one is a common Sprite3D, the one in the middle has Depth Draw set to Always, and the one on the right to Pre-Pass Alpha.

Hi, why was this closed? I have quite a few textures in the same scene. It's basically the body with different parts of an armor, a shield, a weapon, all in Sprite3D. And I'm still hitting this. Is there any way to set the order?
Thank you

Depth-sorting is inherently problematic. Turn on pre-pass alpha.

Thank you very much.
For future reference on anyone having a similar issue:
Click on the Sprite3D -> Flags -> Alpha Cut -> Select
Move your textures slightly back and forth on Z to set the order

Hi, why was this closed?

Hi, I closed this since I moved on to another engnie for that specific project, as I did not have the time to wait nor the knowledge to contribute a fix. This is a really hard problem to tackle, in general. In the shadow of a the new engine version (what included a new renderer) this issue was not really all that useful for the project anymore, and it made sense to just close it.
I do believe this problem still exists in 3.1, but in case it happens to affect anyone, I'd say it would make more sense to open a new issue for it.
Sorry for the late reply.

@dejaime I don't think it should have been closed, even if it was no longer useful to you.
@hikari-no-yume Any idea why this wouldn't just be enabled by default if the transparency option above it is enabled?

@gerrywastaken perhaos the pre-pass is somewhat expensive so Godot wants to avoid it unless we really need it?

The general limitation around transparency sorting is now documented on the 3D rendering limitations page.

I had actually just read that and honestly, it left me with more questions. :D

Enabling opaque prepass by default isn't a good idea due to the added rendering cost. Only pay for the features you actually need slightly_smiling_face

That makes sense. The only problem is that when your sprites disappear behind something like a massive plane mesh, it can take quite some time before you discover what is actually happening. It wasn't until lowering my camera after an hour (or perhaps longer) that it became clear that it was just the ordering that was making the sprite vanish. Some sort of warning on the hover text of the transparency option, or a warning when it's enabled would be very helpful. It won't help me anymore, now that I know, but for others, it could save a lot of time.

Either way, this should be addressed by godotengine/godot-proposals#3986 and its implementation #60704.

Thank you for the link!

@gerrywastaken maybe not, but this was a 2.1 bug. The minimal project does not work on current engine versions, the reproducing steps are not the same, and this might not even be an issue on 3.5 / 4.0 afaik. This to me means the issue was no longer relevant to the engine, except maybe for reference.
If someone were to face a similar problem and end up stumbling on this issue, this could give them some insight. But that's about it, they'd still be better off opening a new issue for their engine version, with new repro steps and example project.


Android Lint Reporter got undefined

We use https://github.com/worker8/AndroidLintReporter to comment lint reports on PR automatically, but it shows undefined as username (the second one in the above image).
Can I define the name or exclude the reporter?

Hi @seabornlee, thanks for sharing this issue.
I made an improvement on v2.4.4 to filter reviewers with undefined username. This should remove the empty row caused by AndroidLintReporter.
Could you give it a try and confirm if it works for you?
Thanks

It works now. Thanks very much!


Codelabs - Copy icon not present next to code snippet

Is there an existing issue for this?

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

Steps to reproduce
Adaptive Apps in Flutter and a few other codelabs do not not copy icon next to the sample code, some instances
https://codelabs.developers.google.com/codelabs/flutter-adaptive-app?hl=en#3
https://codelabs.developers.google.com/codelabs/flutter-adaptive-app?hl=en#6
Expected results
Copy icon along with dark/light mode toggle should be present
Actual results
Copy icon not present
Code sample
NA
Screenshots or Video
No response
Logs
Logs
[Paste your logs here]

Flutter Doctor output
NA

cc: @domesticmouse


[Flaky Test] managedresources #WaitUntilHealthy [It] should retry when the managed resource is not healthy yet

How to categorize this issue?

/area testing
/kind flake
Which test(s)/suite(s) are flaking:

managedresources #WaitUntilHealthy [It] should retry when the managed resource is not healthy yet
managedresources #WaitUntilHealthyAndNotProgressing() [It] should retry when the managed resource is not healthy yet or still progressing

CI link:
https://prow.gardener.cloud/view/gs/gardener-prow/pr-logs/pull/gardener_gardener/10180/pull-gardener-unit/1816009268977995776
https://prow.gardener.cloud/view/gs/gardener-prow/pr-logs/pull/gardener_gardener/10185/pull-gardener-unit/1816011125670547456
Reason for failure:
• [FAILED] [0.293 seconds]
managedresources #WaitUntilHealthy [It] should retry when the managed resource is not healthy yet
/home/prow/go/src/github.com/gardener/gardener/pkg/utils/managedresources/managedresources_test.go:458
  [FAILED] Expected
      <bool>: true
  to be false
  In [It] at: /home/prow/go/src/github.com/gardener/gardener/pkg/utils/managedresources/managedresources_test.go:562 @ 07/24/24 07:30:55.052

Anything else we need to know:
% stress -p 16 ./pkg/utils/managedresources/managedresources.test -ginkgo.focus "should retry when the managed resource is not healthy yet"

6m30s: 9700 runs so far, 7 failures (0.07%)
6m35s: 9828 runs so far, 7 failures (0.07%)
6m40s: 9956 runs so far, 7 failures (0.07%)
6m45s: 10068 runs so far, 7 failures (0.07%)

I assume what happens is that https://github.com/gardener/gardener/blob/297802f13a00a033b4891fa84ba2316f81423eb5/pkg/utils/managedresources/managedresources_test.go#L562 gets invoked before https://github.com/gardener/gardener/blob/297802f13a00a033b4891fa84ba2316f81423eb5/pkg/utils/managedresources/managedresources_test.go#L556.
There is no guarantee that after the execution of https://github.com/gardener/gardener/blob/297802f13a00a033b4891fa84ba2316f81423eb5/pkg/utils/managedresources/managedresources_test.go#L538-L554 in the separate go routine, https://github.com/gardener/gardener/blob/297802f13a00a033b4891fa84ba2316f81423eb5/pkg/utils/managedresources/managedresources_test.go#L556 will be executed first before https://github.com/gardener/gardener/blob/297802f13a00a033b4891fa84ba2316f81423eb5/pkg/utils/managedresources/managedresources_test.go#L562.
cc @dimityrmirchev


Another exception was thrown: RangeError: Value not in range: 1  ════════ Exception caught by gesture ═══════════════════════════════════════════════════════════════ Value not in range: 1 ════════════════════════════════════════════════════════════════════════════════════════════════════

Steps to Reproduce

Run flutter create bug.
Update the files as follows: ...
...

Expected results:
Actual results:

Logs

Another exception was thrown: RangeError: Value not in range: 1
════════ Exception caught by gesture ═══════════════════════════════════════════════════════════════
Value not in range: 1
════════════════════════════════════════════════════════════════════════════════════════════════════

Hi @guruprasadh-j,
When opening an issue, provide some or all information mentioned in issue template.

issue template
If you are looking for support, please check out our documentation
or consider asking a question on Stack Overflow:
- https://flutter.dev/
- https://api.flutter.dev/
- https://stackoverflow.com/questions/tagged/flutter?sort=frequent

If you have found a bug or if our documentation doesn't have an answer
to what you're looking for, then fill our the template below. Please read
our guide to filing a bug first: https://flutter.dev/docs/resources/bug-reports

Steps to Reproduce

Please tell us exactly how to reproduce the problem you are running into.

Please attach a small application (ideally just one main.dart file) that
reproduces the problem. You could use https://gist.github.com/ for this.

If the problem is with your application's rendering, then please attach
a screenshot and explain what the problem is.

1. ...
2. ...
3. ...

Please tell us which target platform(s) the problem occurs (Android / iOS / Web / macOS / Linux / Windows)
Which target OS version, for Web, browser, is the test system running?
Does the problem occur on emulator/simulator as well as on physical devices?

Target Platform:
Target OS version/browser:
Devices:

Run your application with `flutter run --verbose` and attach all the
log output below between the lines with the backticks. If there is an
exception, please see if the error message includes enough information
to explain how to solve the issue.

Run `flutter analyze` and attach any output of that command below.
If there are any analysis errors, try resolving them before filing this issue.

Finally, paste the output of running `flutter doctor -v` here.

Closing, as no information is provided.


IO / FMU: Guard mixers for thread safety

I have alleviated the issue on IO with this patch: https://github.com/PX4/Firmware/pull/6176
But it requires three more fixes:

[ ] IO true thread-safety, probably with enter / leave_critical_section()
[ ] FMU semaphore
[ ] UAVCAN semaphore
[ ] TAP semaphore

Will discuss with @davids5 best approach for IO.

The current state is stable, so I'm leaving this cleanup for v2


[Feature Request] Time played while using Collapse

A functionality I miss from other launchers is play-time tracking.
Given the large amount of time investment Hoyoverse games require in the long run, it would be great to know what is the total playtime of each game played thru Collapse.
(as HI3 and Genshin are quite old, there could be an option to add the playtime values you have from other "trackers" i.e. GOG Galaxy, Epic Games Store, Android and IOS app screen time)

I like the idea but for importing play time from other platforms seems quite complicated to do. What we can do instead is to manually input/edit play time themself

That's what I meant, sorry if my phrasing was not clear enough. The platforms were just a example of places where a player could manually get this information to insert it on collapse.

I like the idea but for importing play time from other platforms seems quite complicated to do. What we can do instead is to manually input/edit play time themself

We could, but I know for a fact that GOG has an API for querying the total playtime, thru a GET request, and Epic probably has that as well.
Leaving this here in case you're interested:
Link 1
Link 2


SSO protection for Documentation (viewers) in NextJS

Title of ticket: SSO protection for Documentation (viewers) in NextJS
Description
The actual content of documentation should be behind SSO too.  This is for view access of the content.  Any IDIR is sufficient.
Dependencies
Are there any dependencies?
DOD

[x] SSO implemented in NextJS for staticly generated documentation

Deployed to dev, https://foi-help-frontend-dev.apps.silver.devops.gov.bc.ca/help-articles
But need to get envs for test and tidy up OCP yaml.

@acoard-aot this is the error message I receive when trying to log in with my idir:

Definitely a bug, I think I know what it is, thank you for the screenshoot it really helped.

@acoard-aot Can you tag @arielleandrews & I when this is fixed so we can test it and moved to closed

@JHarrietha-AOT @arielleandrews please test prod, for some reason dev is down but prod should be working.
https://foi-help-frontend-prod.apps.silver.devops.gov.bc.ca/

@arielleandrews it works for me know, with the link above. Let me know if you want to check as well then we can move to closed

Will do!
On Mon, Aug 15, 2022 at 12:24 PM Jacklyn @.***> wrote:

@acoard-aot https://github.com/acoard-aot Can you tag @arielleandrews
https://github.com/arielleandrews & I when this is fixed so we can test
it and moved to closed
—
Reply to this email directly, view it on GitHub
https://github.com/bcgov/foi-flow/issues/1984#issuecomment-1215647338,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOTSLJSG6R3WWTEMTRSSZ6DVZKKOLANCNFSM5UPOLQ2A
.
You are receiving this because you were mentioned.Message ID:
@.***>


[CLI] Update CLI with Log command

Changes
Added Log command and Get Record command.

Submitter Checklist
These are the criteria that every PR should meet, please check them off as you review them:

[x] Has Docs included if any changes are user facing
[ ] Has Tests included if any functionality added or changed
[x] Tested your changes locally (if this is a code change)
[ ] Follows the commit message standard
[x] Meets the Tekton contributor standards (including functionality, content, code)
[ ] Has a kind label. You can add a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user-facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[ ] Release notes contain the string "action required" if the change requires additional action from users switching to the new release

Release Notes
Add List and Get command for Logs. Also added Get Record command.

Nice feature to have. We should revise code and merge it quicker. It had been one month since pr come out.

Opened https://github.com/tektoncd/results/issues/587 for improving UX

/lgtm

/lgtm


Publish LSP to npm

Publish a binary to npm as rescriptls. Support two channels stdio and node-ipc
VSCode client keeps using node-ipc channel

Related #354

Great stuff!
One thing I'd like to mull over a bit before going this route is that there might be a separate case for publishing just the analysis binary to npm, by itself. That would be because the analysis binary might in the future contain more general tooling, not just stuff for powering the LSP. Doc extraction is one example where it's possible that's the easiest way of distributing it.
This package will have the analysis bin too of course. So maybe there's a way to name this (and expose the analysis bin separately) so that they can be exposed via the same package, without it being fantastically confusing.. Let me think a bit. Any feedback welcome.

Perhaps only expose the part of the analysis binary which is for public consumption. Whiteout even leaking the existence of the binary (helps keeping freedom to move stuff between binary and ts).

If I understand correctly, as reported by users the main problem is that we don't publish an LSP like OCaml does to facilitate the setup in several editors. Just publishing the analysis binary does not resolve the reported issue.

yes accessing lsp server and related binaries each time when changing machine that doesn't have it mean setting up $PATH or changing editor configuration file (in this case Helix) over again. npm package definitely help.
Comment

One idea is to do what rust-analyzer does. We publish a single binary with subcommands.

rescript-toolchain lsp [args] - Start LSP
rescript-toolchain analysis [args] - Call OCaml analysis binary commands

Ready for review.

The first version should be 0.0.1 or 1.20?


Next.js with webpack 5 hangs on first compile (.next folder is empty)

Hey there, using a brand new create-next-app, using webpack 5, all dependencies up to date, following the tailwindcss-jit README then on first load (when .next folder is empty) the build never finishes.
You have to CTRL+C and then start again for it to work.
Example repo: https://github.com/vvo/tailwindcss-jit-hangs-webpack5-next.js
git clone git@github.com:vvo/tailwindcss-jit-hangs-webpack5-next.js.git
cd tailwindcss-jit-hangs-webpack5-next
yarn && yarn dev

Now try to load http://localhost:3000 and it will hang forever.
If you restart the server it will work.
To reproduce the hang, rm -rf .next.
Disabling webpack 5 solves this issue, using webpack 5 using regular tailwindcss works.
PS: On my real-world project, the build never finishes at all, first load or second load, I could not reproduce this behavior though 🤷

Can confirm the dev build never finishes on my project with Next.js + webpack 5, no matter how many times I restart the server.
With DEBUG=1, I can see that the JIT compilation is working:
Source path: .../index.css
Setting up new context...
Reading changed files: 0.605ms
Generate rules: 6.206ms
Build stylesheet: 0.255ms
Changed files:  4
Potential classes:  113
Active contexts:  1
Content match entries 38
JIT TOTAL: 185.58ms

Source path: .../index.css
Reading changed files: 0.004ms
Generate rules: 0.108ms
Build stylesheet: 0.003ms
Changed files:  0
Potential classes:  1
Active contexts:  1
Content match entries 38
JIT TOTAL: 6.254ms

I suspect is this because NODE_ENV is set to development. Either set it to production or set TAILWIND_MODE=build. We run a bunch of watchers in the background for watching template changes and by default we set those up if NODE_ENV is development or TAILWIND_MODE is watch.

That was actually the first thing I tried, but it didn't work. I patched this package to confirm that TAILWIND_MODE is being correctly set:

Btw, next build runs fine (and fast!). The problem is only in next dev.

@adamwathan NODE_ENV is development on my side yes because I use next dev.
The bug shows when next dev is used.
The bug is not present when not using webpack 5.

I am also seeing next hang when using webpack 5. Dropping to webpack 4 fixes the problem but it would be great to have both the tailwind JIT and webpack 5 filesystem caching running together.

I have opened an issue over on vercel/next.js for this, and it looks like there's already a pull request over on webpack/webpack to address the issue 🙌 Hopefully it won't be too long before a fix lands in Next.js, but for now you can either disable webpack 5, or use Next.js version 10.0.8. Will keep this open for now and will try to keep you updated 👍

@bradlc thanks for looking into this! I tried downgrading next.js: 10.0.7 works fine, but for some reason 10.0.8 hangs for me just like 10.0.9. Anyways, since it works with 10.0.7, the problem is not in this package.

The latest canary version of next (10.0.10-canary.7) resolves this issue 🙌 Thanks to @sokra for the fix!

Thanks @sokra @bradlc, I can confirm @tailwindcss/jit works with 10.0.10-canary.7 with webpack 5 in dev mode!

Awesome, going to close then — thanks @sokra!


[BUG] Azure Cosmos DB OLTP Spark 3 connector not dealing at all with large partition key support

Describe the bug
The Azure Cosmos DB OLTP Spark 3 connector neither provides the option to enable a large partition key on container creation nor enables it silently by default as Azure does (see Azure portal UI changed and no more provides the option to enable large partition key for your reference). This was very hard to spot and caused extremely bad performances.
To Reproduce
Steps to reproduce the behavior:

Create a Cosmos DB container through the Azure portal with default options
Verify on the portal that under the container's settings the label "Large partition key has been enabled" is shown
Verify that in the exported ARM template of the database you see {"kind": "Hash", "version": 2} under the container's partition key properties - ref Stack Overflow article to tell if a Cosmos DB uses a large partition key
Create a Cosmos DB container through the Azure Cosmos DB OLTP Spark 3 connector with default options
Verify on the portal that under the container's settings there's no label "Large partition key has been enabled"
Verify that in the exported ARM template of the database you just see {"kind": "Hash"} without any version field under the container's partition key properties.

Expected behavior
Either the Azure Cosmos DB OLTP Spark 3 connector provides an option to enable a large partition key on creation or enables it automatically like Azure. I wasn't able to find any reference either in the Catalog API or Configuration Reference docs. Also, I highly recommend to update the Live Migrate Azure Cosmos DB SQL API Containers data with Spark Connector and Azure Databricks article too as that was my starting point and did not mention anything about large partition keys (whereas I remind Azure automatically enables the support to large partition keys on new containers created through the portal).

@xinlian12 and @tvaron3 - please take a look at this.


Item context menu and item info not working

I can't seem to get item info or item context menu working. Every time I omni-click on an item either in my equipment slots, inventory, or stash, it just shows 'no strings for this search'. How do I get these features to work? I'm on v1.29.1 and all screen-check tests return positive.

The 'no strings' message indicates that the search-strings feature takes precedence, which means the tool cannot copy the item's information for some reason. After omni-clicking an item, try pasting your clipboard into a text-file or something to see what it contains. Are you running PoE as an administrator? There were people who had problems with the client not receiving inputs when running as admin.

The clipboard is empty when I paste. I tried running POE and UI in administrative mode, but it's having the same issue.

I guess manually copying works, right? Did you change anything in the omni-key settings? Because there is a workaround that splits the omni-key into two separate hotkeys, one for items and another one for anything screen-related. Can you copy something random into the clipboard, press the omni-key on an item, and then check if the clipboard gets cleared? If not, the script may not have access to the clipboard.
All that's left is checking everything between the client and the input. Is there some software that would have access to the clipboard? People also had issues with mouse/keyboard software and drivers. Maybe updating AHK helps. The cause of the problem is so simple, yet there isn't much I can do from here as I'm running out of ideas.

I didn't change anything for omni-key. The clipboard does get cleared after I press the omni-key on an item. I did manage to get the item context menu to show up if I manually hit ctrl-alt-C on an item.

I got the item info to work by clearing the omni-key (items) keybind. It was defaulted to ctrl+alt+c.


ui: add guide modal

@harunurhan is this the final?

@drjova no waiting for actual step images, will merge immediately when this comes


Missing sub-expression match with conditional expression

https://github.com/signalapp/Signal-Android/commit/c4a37e38aba926c2bef27e4fc00e3a4848ce46bd
DRAFT.IMAGE should have been matched with the conditional expression else part.

@tsantalis RefactoringMiner doesn't report the last sub-expression mapping:

Could you please verify?

I have replicated the problem here https://github.com/pouryafard75/TestCases/commit/929055779b332d08cfb9a0f78ffb53f884a1c73c


env: 'go' : No such file or directory

Receiving "env: 'go' : No such file or directory" when building with sudo ./build.sh or ./build.sh
Any reason, please?
Nai.

Please use the pre-built binaries from this release. (https://github.com/aungmawjj/icmpnet/releases/tag/v0.2)
There are binaries for Windows, Linux, and mac.
To build your own binary (eg. msgclient):
Please install golang first. (https://golang.org/doc/install)
go build -o ./bin ./cmd/msgclient


Fix #8 - Make share output valid hexadecimal

Not 100% sure if this actually works, need some code review for sure.
Also I doubt that this is backwards compatible with previously generated secrets, so that's going to be a problem...

Please provide a small test (before / after).


unable to generate refresh token

Prerequisites

[Y] Did you read FAQ section?
[Y] Did you test with the latest releases or commit ?

Description
no refresh token is generated
Steps to Reproduce

fill in all the details in config.ini
run
wait

Expected behavior: automatic generation of refresh token
Actual behavior: blank
Versions pixivutil20200524
You can get this information from executing PixivUtil2.py --help.
Latest version available in https://github.com/Nandaka/PixivUtil2/releases
Tried the method suggested earlier where I change my useragent to match my broswer i.e. Chrome/81.0.4044.138 bu this is still not solving the issue

I think I've seen a similar issue before.
The answer was pretty much like: if refresh token is not required for the actions you use, then it would not be obtained.
Do you have any problem downloading anything?

I am trying to download from my supported artist on Fanbox.
Without the refresh token, I just receive the OAuth failed message.

If the tool fails to obtain the token when it's needed, there should be error messages?
Do you mind post them?

Hi, sure, here is what's happening:
Trying to log in FANBOX with saved cookie
FANBOX Login successful.
Getting supported artists from https://api.fanbox.cc/plan.listSupporting
Found 5 supported artist(s)

displaying supported artist

Using OAuth to retrieve member info for: 1657441
103:pixiv ID、またはメールアドレス、パスワードが正しいかチェックしてください。
Failed to login using OAuth
press enter to exit.

https://github.com/Nandaka/PixivUtil2/issues/631
Might be similar to this one?
What do you have in your config for username and password? Are they empty or dummies? If so, can you try filling them with your account and password?
If not, and you are using "username" username or "email" username, can you try switching to the other one and try again?

Hi,
I have actually tried the method you linked and it used to work, up till fanbox changed their address. This has only become a problem since the change by fanbox.

Hmm. I have no problem with using fanbox though so far...
Just tried deleting the token from my config and download from fanbox, it had no problem retrieving refresh token at all.
Do you mind generate a new log file and post it?
(BTW, you didn't delete the artist number in one of your previous posts.)

Try to use your email address for the username? the error message state something about email/password is not correct.


refactor(ui): replace page components

Closes #14871
Closes #14708

Replace local Page components with the Clockface version
Remove unused logic in Templates page
Rename Navigation to UnauthenticatedApp
Update Settings and Load Data icons in navbar
Wrap entire app in Clockface AppWrapper
Standardize page titling via a utility (HTML title tag, not in the UI)
Move control of presentation mode to a single component (App)
Remove unused vestigial "Account" page (used to be where you could change your username)

Created this issue: https://github.com/influxdata/clockface/issues/258

[ ] CHANGELOG.md updated with a link to the PR (not the Issue)
[x] Well-formatted commit messages
[x] Rebased/mergeable
[x] Tests pass

Awesome.
Since this is a lot of markup/style changes, I did a run through of the app to see if I could find anything that looked weird. All seemed good to me except for one minor thing, the padding around the plot in the DE:

Ahh good catch, I'll fix that up before merging


NestJS SBOX_FATAL_MEMORY_EXCEEDED

Describe the bug
Most of the times, after the 'nest start' command, the page crashes and it shows the Error code: SBOX_FATAL_MEMORY_EXCEEDED. allready tried on diferents browsers and cleaning the cache and updating chrome.
Link to the blitz that caused the error
https://stackblitz.com/edit/nestjs-typescript-starter-c6jr5q?file=README.md
Steps to reproduce
As soon as you launch your project, after the nest start command.
Expected behavior
The page loading correctly
Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots

Platform

OS: Windows
Browser:  Chrome, Edge
Version: Chrome 114.0.5735.91

Additional context
No response

Hi @juanpaprz ! :wave:
Thanks for your report! I think you are facing the same issue as #970.
Could you share how much RAM you have?

Can you also share how much RAM you have on your machine? I believe we looked into this in the past and it has to do with something RAM related.

That confirms the theory that was found in this issue https://github.com/stackblitz/webcontainer-core/issues/970. 16GB RAM on Windows seems to cause issues, while it works perfectly fine on Linux with 16 GB RAM.

A workaround is to use Firefox which doesn't exhibit the same behaviour.

Since I hadn't been able to see the link to this repository with the container crashing, I posted https://github.com/stackblitz/core/issues/2631 which would be a duplicate of this.
I'm also on windows 11 with 16GB of RAM, on Microsoft Edge. My RAM utilization was as low as 68% when booting the container. Stopping the automatic "install and run" terminal worked for me, when I manually ran the pnpm install and then the pnpm run dev for my container. This is mine: https://stackblitz.com/edit/vitejs-vite-pumjms


Fix retries for aggregated cache

For aggregated steps, the "retries" field is increased each time a child is processed, which can prevent retrying jobs when there is an error.
See for example:

This cache has been updated 11 times because of its children, but it was a crash error from "dataset-size" last time. Since it already has 11 retries, backfill might not consider it to append a new job.
New logic to increase a retry:
Increase if there already exists an old cache for the same combination of kind+dataset+config+split and it has a different revision and:

job was failed or
it is a completed job (http_status=200 and progress=1.0)

This way, we should always have retries less than the limit = 3 https://github.com/huggingface/datasets-server/blob/main/libs/libcommon/src/libcommon/constants.py#L23
Question: Should I prepare a migration to set all cache records with retries > 3 to 0 again? so that the failed ones can be considered for the next backfill execution?  I would set 0 to those success results.

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

Comparison is base (829eac8) 92.38% compared to head (e3e1fff) 84.55%.
Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2247      +/-   ##
==========================================
- Coverage   92.38%   84.55%   -7.83%
==========================================
  Files          91       58      -33
  Lines        6225     1800    -4425
==========================================
- Hits         5751     1522    -4229
+ Misses        474      278     -196

Flag
Coverage Δ

jobs_mongodb_migration
85.89% <ø> (?)

services_search
80.57% <100.00%> (?)

services_worker
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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

What if we set it to 0 when http_status is 200 OK? We're just interested in incrementing if we have an error. The field could reflect that it only counts errors: failed_runs?

We're just interested in incrementing if we have an error. The field could reflect that it only counts errors: failed_runs?

It would be easier to understand. I can also change the field name and prepare a migration for it.


Feature/dynamicmemory revision

Works well enough for me during my testing of https://github.com/eProsima/ROS-RMW-Fast-RTPS-cpp/pull/51

Thanks for the help with this feature 👍


added keyboard Accessibility to dropdown links

directly related to #1740

Hi, sorry for the late review.
When you check for accessibility issue for these type of elements I'd suggest trying to navigate through the website with the keyboard first, and then use a Screenreader to dig more into its usability for screenreader users.
Here's some issues I found by just navigating via keyboard:

The button automatically expanded when the focus enter the element - this is not a desirable behavior.
Once the button is activated, there is no way of activating the links underneath using a keyboard, making these links completely inaccessible for keyboard users.
While the button is focused, the next "tab" action remove the focus but doesn't put the focus on the next element. At this point user does not no where the focus is on the page until the next "tab" action.

Here's the desirable behaviors for the toggle button with links underneath:

Either button or anchor element is OK as toggler since they are accessible by default (by means of keyboard/screen reader accessibility - not as a blanket rule :) )
aria-expanded attribute should be presented on the element to let (screen reader) user know the state is "expanded" or "collapsed"
When the button is focused: aria-expanded should be false (collapsed) and the sublinks should be hidden
When the button is activated (space or enter keyboard key, or click), aria-expanded should be set to false (expanded) and the sublinks should be displayed, the focus should move to the first link in the list
User should be able to navigate through the sublinks using tab keyboard
User should be able to Crt Tab to the button and active/deactivate the button to collapse/expand the list
The User should be able to tab through all the links within the list, after the last link the focus should be on the next button.

A bonus behavior: when the focus is on one of the links, "ESC" key should remove the links (or collapse the list) and place the focus back on the button.
Hope this is enough as a guide for you to start :)


Update free-programming-books-pl.md

added link to git cheatsheet
What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo
For resources
Description
Why is this valuable (or not)?
Git knowledge available in one place
How do we know it's really free?
It is publically available on GitHub
For book lists, is it a book? For course lists, is it a course? etc.
Cheatsheet
Checklist:

[x] Read our contributing guidelines
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction)

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

cheatsheets are mostly in English, this cheatsheet is in polish

not added yet

closed in favor of #6597


New version: eza-community.eza version 0.19.4

Pull request has been created with WinGet Releaser v2 :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-09-18 07:17:38.264 [FAIL] Installer failed security check. Url: https://github.com/eza-community/eza/releases/download/v0.19.4/eza.exe_x86_64-pc-windows-gnu.zip Result: 0x80004005

Automated error analysis suggests -1978335187 may mean Installer failed security check. Another script should retry this every 18 hours.
Automated error analysis suggests 80004005 may mean Resource missing
(Automated response - build 895.)

@wingetbot run


Add functionality to download steady state data

Add option to download steady state data for more in depth analysis off platform

Issue addressed in commit 8c3544 on the bcmd-web repo


Auto-update dependencies

Produced via:
dep ensure -update knative.dev/test-infra knative.dev/pkg knative.dev/serving
/assign shashwathi tanzeeb
/cc shashwathi tanzeeb

There was a breaking change.  Fixing.


Add release notes for v0.4.3

Signed-off-by: Logan Seeley<EMAIL_ADDRESS>

Closing in favor of #47


Change the timeout API?

Timeouts are confusing. @spacejam recently wrote an example that contains the following piece of code:
stream
    .read_to_end(&mut buf)
    .timeout(Duration::from_secs(5))
    .await?;

The problem here is that we need two ?s after .await and it's easy to forget that.
I think the confusing part is in that the .timeout() combinator looks like it just transforms the future in a similar vein to .map() or .and_then(), but it really does not!
Instead, .timeout() bubbles the result of the future so that its type becomes Result<Result<_, io::Error>, TimeoutError>.
Perhaps it would be less confusing if timeout() was a free-standing function in the future module rather than a method on the time::Timeout extension trait?
future::timeout(
    stream.read_to_end(&mut buf),
    Duration::from_secs(5),
)
.await??;

This timeout() function would stand alongside ready(), pending(), and maybe some other convenience functions in the future module.
Here's another idea. What if we had io::timeout() function that resolves to a Result<_, io::Error> instead of bubbling the results? Then we could write the following with a single ?:
io::timeout(
    stream.read_to_end(&mut buf),
    Duration::from_secs(5),
)
.await?;

Now it's also more obvious that we're setting a timeout for an I/O operation and not for an arbitrary future.
In addition to that, perhaps we could delete the whole time module? I'm not really a fan of it because it looks nothing like std::time and I generally dislike extension traits like Timeout.
Note that we already have a time-related function, task::sleep(), which is not placed in the time module so we probably shouldn't worry about grouping everything time-related into the time module. I think it's okay if we have io::timeout() and future::timeout().
Finally, here's a really conservative proposal. Let's remove the whole time module and only have io::timeout(). A more generic function for timeouts can then be left for later design work. I think I prefer this option the most.

What if we had io::timeout() function that resolves to a Result<_, io::Error> instead of bubbling the results?

In addition to the current variant? IMHO the current variant is useful on its own for non-IO futures too. You might have some more complicated user defined futures (which might or might not be around IO futures) that don't return a plain io::Error but some more complex error type (that might contain an io::Error variant somewhere though).
Removing the current variant would seem suboptimal to me.

Just as example I prefer timeouts that allow me to restart and retrieve original future on failure:
pub fn timed<F: Future>(job: F,  timeout: Duration) -> impl Future<Output = Result<F::Output, Expired<F, Timer>>>
And without any extra error from timer itself 😄

@DoumanAsh If fut is a Future, then &mut fut is also a Future, so you can pass a &mut fut to the timeout function and continue using fut if timeout occurred.
Unless I'm missing something, there's not much reason to return the original future since one can use a mutable borrow of the future. Do you agree with that assessment?

It is indeed true, that with &mut F you could get around it.
But it applies severe restriction on F, it must be Unpin always then, which is not true for async fn and it is not clear whether it will change (Boxing is work around, not a solution)

But it applies severe restriction on F, it must be Unpin always then

Is that not true also for the case when the timeout function returns the original future back to the user? If F has self-references and gets polled by the timeout function, and is then returned back, returning moves the future and therefore invalidates self-references.

The original issue has been resolved, and conversation has settled down. I'm going to go ahead and close this for now (:


[BACKPORT][v1.0.2] - Added hostname_domain_extension.

Backport issue: #2768
Pull request: #2790

DoD checklist

[ ] Changelog updated
[ ] COMPONENTS.md updated / doesn't need to be updated
[ ] Schema updated / doesn't need to be updated
[ ] Feature has automated tests
[ ] Automated tests passed (QA pipelines)

[ ] apply
[ ] upgrade

[ ] Idempotency tested
[ ] Documentation added / updated / doesn't need to be updated
[ ] All conversations in PR resolved
[ ] Solution meets requirements and is done according to design doc
[ ] Usage compliant with license
[ ] Backport tasks created / doesn't need to be backported

:heavy_check_mark: A hostname domain extension with basic validation has been added for the Azure provider


Allow duplicate names

Within guest/vendor list, cannot have same name, across guest/vendor list can have same name


[Question] Further token authentication API examples

Is there any further API examples? Would be great to see how this could be implemented with a token based authentication system.

There isn't a whole lot of public example code yet. You’ll find some hints in the configuration chapter of the user guide and in the API docs for invalidateConfiguration.
Although there isn’t currently time available for it, my hope to write up a cookbook of examples for the 1.0 release.
If you can pose a more specific question, I can help point you in the right direction.

The cookbook sounds great.
Question wise, how would you handle oAuth 2 with Siesta?

I’d first select a third-party lib such as p2/OAuth2 or dongri/OAuthSwift to handle the whole dance of acquiring an OAuth token. That's well outside the scope of Siesta.
Once you have a token, you’d do something like this:
class MyApi: Service
    {
    init()
        {
        ...

        configure { $0.config.headers["Authorization"] = authHeader }
        }

    var authHeader: String?
        {
        didSet
            {
            // Clear any cached data now that auth has changed
            wipeResources()

            // Force resources to recompute headers next time they’re fetched
            invalidateConfiguration()
            }
        }
    }

…and then when authentication succeeds:
myAPI.authToken = authTokenFromOAuthLib

…and for logout:
myAPI.authToken = nil

You could integrate OAuth more tightly with Siesta. For example, you could add special error handling hooks in your Siesta config to trigger a token refresh when you detect an OAuth error. However, I’d start with something as simple as the code above and see what other needs you have.

I added some of this info to the Siesta docs: http://bustoutsolutions.github.io/siesta/guide/security/
Closing this issue, but feel free to post more on Stack Overflow (with the siesta-swift tag) or here if you need more info.


map revenue for all track events

This should've been inside #8 but slipped. Will merge this before deploy!

LGTM


fixes remove picture button

Fixes #2787
Describe the changes you have made in this PR -
Previously, Profile picture is not removing, when button being clicked.
fixed this issue by removing the person image.
video of the change -

Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

@vedant-jain03 Please review it !

@VaibhavUpreti PTAL

Kindly follow the code of conduct from next time:

Comment on issue thread if you interested
Get assigned
Raise a PR

As you are first time contributor, we can consider this PR
Looping @tanmoysrt for review

@shauryag2002 fix the formatting of the html, the editor may have changed formatting, spacing before each line has been doubled

@tanmoysrt Is the converting jquery to vanilla JavaScript part is good or i have to fix that also?

@tanmoysrt Is the converting jquery to vanilla JavaScript part is good or i have to fix that also?

Pls do the formatting first, else the exact changes not showing !

@shauryag2002 pls only use vanilla js for your part. Converting other part from jquery from js is out of scope for this issue. IF you want to do the conversion you can raise separate issue and PR.

@tanmoysrt PTAL


Combine media queries

For those of us who prefer to write our media queries in the same file as the main selector, I think it'd be helpful to add something like gulp-combine-mq to the build tools. I'd be happy to make a PR.

Are you referring to your custom CSS or the framework itself?
For the framework it would be amazing to have related CSS all in the same file. Love the idea and a PR would really help tons of people!

I only mean to combine media queries when embedding them into the final email
Example of separate media queries when using Sass:
_partial.scss
.container {
  max-width: 600px;

  // this is usually done with a mixin
  @media (max-width: 620px) {
    max-width: 100%;
  }
}

.someColumn {
  width: 50%;
  @media (max-width: 620px) {
    width: 100%;
  }
}

app.css
.container {
  max-width: 600px;
}
@media (max-width: 620px) {
  .container {
    max-width: 100%;
  }
}

.someColumn {
  width: 50%;
}
@media (max-width: 620px) {
  .someColumn {
    width: 100%;
  }
}

embedded result in the email after running npm run build
<style>@media (max-width: 620px) { .container { max-width: 100%; } } @media (max-width: 620px) { .someColumn { width: 100%; } } </style>

Using something like the above mentioned gulp-combine-mq should solve this if it was added in the build task.

have related CSS all in the same file

@rafibomb I'm not sure I understand what you mean that. I can make a PR for this but it does nothing more than merge media queries.

I think a PR that only did that would be handy. It's a useful feature and it likely won't get added without one, right? :)

gulp-combine-mq was not updated for 5 years. So this is currently not an option.

There are postcss plugins and other packages that do it still I believe
On Sun, Dec 22, 2019, 6:07 AM Daniel Ruf<EMAIL_ADDRESS>wrote:

gulp-combine-mq was not updated for 5 years. So this is currently not an
option.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/foundation/foundation-emails/issues/519?email_source=notifications&email_token=AAQZCJLOD7TQCTHMDVMWYCTQZ5DAXA5CNFSM4CI5QWWKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHPNP6Y#issuecomment-568252411,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQZCJJ4TVXIYWVHH363SQ3QZ5DAXANCNFSM4CI5QWWA
.


Auto complete

Add auto complete
Currently the interaction type for auto completes are not handled.
This should be added in v2.0.0
Resources
https://discord.com/developers/docs/interactions/receiving-and-responding#interaction-object-interaction-type

Removed from milestone 2.0.0
This will wont be added yet, currently don't have enough time.


Re-work pooling to have a fixed memory cost

Earlier versions of pooling didn't re-use allocations within the construction of a FST. It only amortised allocations across different FSTs being built. As a result of this, its memory usage grew proportional to the number of distinct states in the FST (this can be several gigabytes when constructing large FSTs)
To address this shortcoming, this PR relies on the fact that frozen builderNode(s) can be re-used once they're evicted from the Registry. A further nice property of this PR's implementation is it bounds memory usage to be proportional to the size of the Registry (plus max length of unfinishedNodes). This allows users of Vellum to control memory behaviour more predictably, and seems to give a nice 20% CPU savings too.
Some stats from a micro-benchmark I've been using:
Before the PR (i.e. master) - time 42s, max total memory: 8.6GB
time ./vellum-upstream-master --expvar ':8081' set --sorted sample.input output.fst
./vellum-upstream-master --expvar ':8081' set --sorted sample.input output.fs  54.00s user 5.74s system 140% cpu 42.449 total

After PR: time 33s, max total memory: 23MB
time ./vellum-custom-pooling --expvar ':8081' set --sorted sample.input output.fst
./vellum-custom-pooling --expvar ':8081' set --sorted sample.input output.fst  47.51s user 1.19s system 146% cpu 33.132 total

wow -- very cool!  It's gonna take a bit to absorb what you did here, but the #'s are really exciting!
One other thing is we actually do code reviews over in another system, called Gerrit (same as what android, chrome, golang, libreoffice and others use).
It's running here... http://review.couchbase.org/ and here's the vellum drill down:  http://review.couchbase.org/#/q/project:vellum
You'd have to become a user of that (with a bit of a one time pain-in-the-ass setup steps, imho).

@steveyen cheers! I created http://review.couchbase.org/#/c/98887/ on Gerrit.
Might help if I explain the thought process a bit - the code relies on the lifecycle below for builder nodes.
+-------------------------+                            +-----------------------+
|    Unfinished Nodes     |      Transfer once         |        Registry       |
|(not frozen BuilderNodes)|-----BuilderNode is ------->| (frozen BuilderNodes) |
+-------------------------+         frozen             +-----------------------+
             ^                                                     |
             |                                                     |
             |                                                   Put()
             | Get() on        +-------------------+             when
             +-new char--------| BuilderNode Pool  |<-----------evicted
                               +-------------------+

With this structure in place, we're guaranteed that worst case we'd allocate at most  Size(Registry) + Max(len(InputKey)) buildeNodes; and further, we can re-use the builder nodes once they're evicted from the Registry.
Few things to note about the implementation:

We don't need to pool transitions and builderNodes separately, because builderNodes contain a slice of transitions. So as long as we reset the transitions slice to have zero length before re-using a builderNode, we get transitions pooling for free once we start pooling builder nodes.
builderNode storing []transition instead of []*transition. Mainly to avoid unnecessary GC pointer chasing. The transition type only has primitives so we can get away with this quite easily
Intrusive linked list for the pool - I modified the builderNode structure to have a next *builderNode field, to allow the pooling of nodes in a single-y linked list. We only need to operate on the head to insert/remove which makes the operations quite cheap. I tried experimenting with a couple of other strategies (arena/slice allocating) but found this to be better performing and have the simplest code

(thanks -- leaving a pointer to the gerrit change review request - http://review.couchbase.org/#/c/98887/)

(oh heh -- I saw @prateek left the link already.  I must have been too awed by the ascii diagram to not have seen it on the first read!)

Hey @prateek , Great work on this, I've merged your gerrit change (http://review.couchbase.org/#/c/98887/), so that should be available on the master branch of vellum. Feel free to close this PR. Cheers!

@abhinavdangeti @steveyen ta!


Hi Samuel.  I am getting an error when I try to run the code for the first time.

I am new to slim 4.
I have bought all of Daniel's books and wanted to learn within his repo.
I have had a torrid time of it trying to get it to work.
Then I found your codebase.
I am blown away by the detail I find.
Phew...  Amazing.
May I ask a question:  I added the following scripts to composer.json.
"start": "php -S localhost:8080 -t public/",
"dump": "composer dump-autoload -o",
I have set up the database and have tables inserted with seed:minimal.
I have set up config/env/env.php
I have xampp running on my machine for the database and it is running.  I then run the "start" script to get up and running.
However, when I put in credentials the system crashes.
I.e. in phpstorm I get the following error Script php -S localhost:8080 -t public/ handling the start event returned with error code -1073740791

and the server stops dead (per attached).
I am sure I have done everything correctly but there are a lot of moving parts.
Possibly you have seen above before.
PS I am really excited to dig into your code and use it to help me learn slim 4 in a manner that aligns with Daniel's 3 books I have bought.  Again.  Awesome work.  Thank you for the leg up to help me to understand what is happening within the piping of the framework.

Hi Mark
I'm glad if the documentation and this project can be helpful in your journey of learning PHP with Slim.
If you have XAMPP up and running, you don't need the built-in PHP web server.
You start the web server and database in the XAMPP control panel, place your project in c:\xampp\htdocs and you should be able to access the application via localhost/your-app-name.
Try it again without php -S localhost:8080 and let me know how it looks.
In case you installed the project using composer create-project like mentioned in the installation guide, it installed the v0.4 which is pretty outdated. I will release a new version soon but in the meantime you can fork the project or install it again via the git clone feature implemented in PHPStorm:

If you did that, forget what I said.

Hi Samuel.  Some feedback.
A while ago I set up an ability to run projects from my home server directly onto the internet.
I use no-ip.com and their DUC application.
I also use Certbot to make sure that the "https" padlock (or whatever their new switchy icon is now) is running.
I was able to fire up your incredible application and play around with it.
Phew, I am blown away by the depth and skill I see (all using the latest slim 4 framework).
May I switch to a quick question.  I see that you are not using twig but create pages in another method (using php and js I think) that has me stumped.
My goal, if this is possible, will be to try to use your framework and try to create a basic user login system to a landing page (dashboard) and a bunch of other internal pages only accessible to authorised.  Doing this, I hope, will help me to understand your approach, and learn from what you are doing.  I think this will take me many months to get right.  BUT, having your stable example of the slim skeleton (optiz) as a guide I believe I will be able to focus on learning how to get to my project goal without spending most of my time getting the framework (dependencies) to work.  Hopefully, I will also see how you are using the DI container and autowiring the different features into it.  I will also need to cross reference to your excellent notes and also the Daniel's slim 4 books.  VERY STEEP LEARNING CURVE ahead for me, but at least it is focused and in a tested and working environment.

I did try to run the localhost/projectname in my browser working on Xampp in localhost but did not have much success. Strange.

This is strange indeed! Running it on your home server with your config is awesome for a dev/prod environment but you should still be able to serve it locally for development easily. If you want I can help you debug the issue with XAMPP. Have you enabled the apache & mysql module and placed the project in the xampp/htdocs folder?

Thank you for the very positive feedback! This project has been my full-time job the last 3 years and I'm happy that it finally comes to an end now and if you can find value in it, that's double mission accomplished for me.
Have you heard of Xdebug? That tool is an absolute must have.

I see that you are not using twig but create pages in another method (using php and js I think) that has me stumped. Is your project a SPA?

You're right, I chose to not use twig but a PHP template renderer which is way more lightweight, native and doesn't require a custom syntax.
I have written a chapter about how the front end is rendered, which library is used and it also addresses the SPA question (spoiler: I'm not a fan of SPAs). Documentation: Template Rendering.
I must say, I'm not really a front-end guy. The dependency that front-end frameworks bring, the overhead for the client and the fact that a new one is trending each month/year or the complexity of tools like web pack scare me away.
I want code that is performant, durable and works for a long time without a lot of maintenance, even if it might take a little longer to write. That's why I chose to write everything javascript in pure vanilla js which has come a long way and I'm hopeful that it will continue to evolve into a more mature language with maybe one day native type safety (type annotations work great for now).

My goal, if this is possible, will be to try to use your framework and try to create a basic user login system to a landing page (dashboard) and a bunch of other internal pages only accessible to authorised
Hopefully, I will also see and finally figure out (with your clear examples) how you are using the DI container and autowiring the different features into it

That is a great goal and the true way to learning things. You could strip down the example project, take everything logic out and build your app with the resources available and you'll learn a ton.
I've tried to address most aspects of building such an app including DI, authentication and authorization in the documentation (table of content in the sidebar).
I wrote the documentation precisely for people like you, trying to uncover a lot of "behind the scenes" things that aren't being explained properly in most guides that just "expect" you to have this knowledge already.
If anything is a bit unclear or you find things that could be clarified, explained better please don't hesitate to write me. Here in the issues or via email<EMAIL_ADDRESS>You're feedback is very valuable to me.
And if you have other questions in your learning process, you're also welcome to write.

Thank you very much.  I did figure out the localhost/myproject issue.  I was putting in https://localhost/myprojectname.  When I left off the s it worked perfectly.  I agree fully with your advice about front end complexity.  I have spent loads of time trying to learn / unravel this and would far better not be dragged into the next fad.  I will definitely take you up on your offer to contact you with specific questions.  Again, thank you very much.  Kind regards Mark.


docs: JX-340: Updates doc titles and descriptions

Updates rhoas CLI doc titles and descriptions, and descriptions of UI-equivalent Getting Started docs

@jbyrne-redhat Will you please do a sanity check for all of the updates, and I'll merge if you're happy? Thanks :)


[Proposal] Streamlining docker images

Motivation
The modern eBPF probe is gaining prominence, prompting our decision to adopt it as the default driver. This shift offers significant advantages to our distribution system. By moving to the eBPF probe, we eliminate the need to include the full driver-building toolchain in the default Falco distribution. Users who prefer alternative setups can still opt for a different container image. Consequently, this allows us to transition the Falco default image to a "no-driver"/"distroless" configuration, streamlining our deployment and enhancing system simplicity.
As a result of this decision, we have re-audited all Docker images, gathered feedback, and are now proposing to streamline the Falco Docker images to better meet current needs and enhance their functionality. This proposal outlines the desired state of these improvements.
Feature
New supported images

Image Name
Tag (aliases)
Description

falcosecurity/falco
x.y.z (latest)
Distroless image without driver building toolchain support, based on the latest released tar.gz of Falco.  No tools or falcoctl not included.

falcosecurity/falco
x.y.z-debian  (latest-debian)
Debian-based image without driver building toolchain support, based on the latest released Deb of Falco (similar to the old falcosecurity/falco-no-driver image). May include some tools (ie. jq, curl), but not falcoctl.

falcosecurity/falco-driver-loader
x.y.z  (latest)
Based on falcosecurity/falco:x.y.z-debian (see above) plus the driver building toolchain support and also shipping the latest version of falcoctl. Recommended only when modern eBPF is not supported.

falcosecurity/falco-driver-loader
x.y.z-buster  (latest)
Similar to falcosecurity/falco-driver-loader (see above) but based on a legacy Debian image (i.e. buster ).  Recommended only for old kernel versions

Deprecated images

Image Name
Reason

falcosecurity/falco-distroless
Deprecated in favor of falcosecurity/falco:x.y.z

falcosecurity/falco-no-driver
Deprecated in favor of falcosecurity/falco:x.y.z-debian (basically the same image with a new name)

Other legacy or almost not-used images included in https://hub.docker.com/u/falcosecurity may be deprecated, too.
Alternatives
We have already evaluated different alternatives, but no valid alternatives emerged (or were discarded early).
A still valid alternative would be not to implement some image variants (i.e., -debian and -buster) or add others variants. This, however, may reduce the Falco compatibility range.
Additional context
This needs to be shipped before Falco 1.0 as per our roadmap.
Tentatively for
/milestone 0.38.0
Probably for 0.39.0

I think providing an image that does not include Falcoctl would be a good idea especially for folks trying to run Falco in a more secure environment or that are concerned about attack surface. I created a PR to create a distroless Dockerfile that does not include falcoctl. I have been testing this image in a dev environment with great results over the past week.
I was motivated to modify the existing distroless image as I was seeing several vulnerabilities introduced by falcoctl. I am also running falco in a way that does not require I ever interact with falcoctl. Not sure if anyone else has a similar use case but either way maybe this PR can help them out.

Hey @killerkenobi
as per my proposal, the new falcosecurity/falco will be distroless and not include falcoctl :point_down:

Distroless image without driver building toolchain support, based on the latest released tar.gz of Falco. No tools or falcoctl not included.

Also, please share :point_down: your use case :point_down:  anyway, it may be useful for someone! :pray:

am also running falco in a way that does not require I ever interact with falcoctl. Not sure if anyone else has a similar use case but either way maybe this PR can help them out.

@leogr that's great to hear and will be very helpful. I am currently running my modified distroless falco image in a Kubernetes environment that restricts access to the internet so Falcoctl would not be allowed to call out and get rule updates. The environment is pretty locked down due to compliance requirements. I am also using a custom rule set that is tailor made for my environment so I have no need to update the Falco-provided rule sets using Falcoctl.
In addition, I have to build my custom falco image using a CI/CD pipeline with strict vulnerability scanning requirements. Falcoctl was constantly failing this pipeline with vulnerabilities in various Go packages. Once I removed Falcoctl the image has been building and working as expected. The distroless image without falcoctl is also only ~50MB where the upstream falco distroless image is ~110MB if I remember correctly.

cc @falcosecurity/falcoctl-maintainers for visibility

/assign
/assign @FedeDP
/assign @LucaGuerra

I wanted also to highlight that we might want to ship multiple versions of the falco-driver-loader image (ie: the one with the toolchains), because eg: right now ubuntu24.04 ships a glibc version of 2.38 while our falco-driver-loader master image ships with 2.36. This breaks the kmod and bpf drivers build on it.
While this is not a major disruption since on newer kernels we will default at the modern-bpf probe, i think that we should still ship different versions of the falco-driver-loader image:

falco-driver-loader:x.y.z-buster
falco-driver-loader:x.y.z-bookworm
falco-driver-loader:x.y.z-trixie (ie: current debian testing based)

And so on, as new debian versions come out.

I wanted also to highlight that we might want to ship multiple versions of the falco-driver-loader image (ie: the one with the toolchains), because eg: right now ubuntu24.04 ships a glibc version of 2.38 while our falco-driver-loader master image ships with 2.36. This breaks the kmod and bpf drivers build on it. While this is not a major disruption since on newer kernels we will default at the modern-bpf probe, i think that we should still ship different versions of the falco-driver-loader image:

falco-driver-loader:x.y.z-buster
falco-driver-loader:x.y.z-bookworm
falco-driver-loader:x.y.z-trixie (ie: current debian testing based)

Debian trixie ships with correct glibc version:
/lib/x86_64-linux-gnu/libc.so.6
GNU C Library (Debian GLIBC 2.38-10) stable release version 2.38.

And so on, as new debian versions come out.

I totally agree. I guess it wouldn't be hard to maintain these variants, and they are useful also for testing purposes.

Do we also want to deprecate the slim image?

Do we also want to deprecate the slim image?

Yes, IMO.
The slim is just an alias to the falco-driver-loader image, so we should just use it instead.
https://github.com/falcosecurity/falco/blob/0bf7458f3decc72ee611f36e577b06c4a002b90f/.github/workflows/reusable_publish_docker.yaml#L92-L94

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

/remove-lifecycle stale

As discussed with @FedeDP we are moving this for
/milestone 0.40
In order to have more time to test and enhance the current WIP implementation.

@leogr: The provided milestone is not valid for this repository. Milestones in this repository: [0.39.0, 0.40.0, 1.0.0, TBD]
Use /milestone clear to clear the milestone.

In response to this:

As discussed with @FedeDP we are moving this for
/milestone 0.40
In order to have more time to test and enhance the current WIP implementation.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

/milestone 0.40.0


unresolved import?

Okay so I swear I'm not going crazy, but my compiler is giving me an unresolved import for Encode and Decode when I try
use tokio_core::io::{Decode, Encode};

then - "No Decode in tokio_core::io.
With my Cargo.toml
[dependencies]
futures = "0.1"
tokio-core = "0.1"

env_logger = "0.3"

I've had a lot of recent blockers, but this one takes the cake... and then some. Any idea what I could be doing wrong? That one-liner is literally what I had my main.rs comprised of (along with an extern crate tokio_core), just to make sure I wasn't losing it.
I also tried cleaning / removing Cargo.lock etc.

Also, wanted to add that I can import everything except Encode and Decode.

Oh one thing which may clarify this is that Encode and Decode haven't been published to crates.io just yet, they're only on the master branch. If you depend on the git versions though it should all work out!

Oh thanks, figured it was... like that.

Ok, closing!


Allow moving neighbouring thumbs to accommodate new thumb values (for issue #55)

This only addresses the first use case in issue #55.
I don't really understand the second use case: why would you want to allow breaking the min/max/step invariants in this case? I don't see any other situations where it is allowed to break these invariants.
I choose to augment the API to implement this feature by overloading Thumb::setValue with a version taking a "push" flag. This might not be the best approach, especially if use case 2 is needed. It might be better to have differently named methods, or to replace the flag with an enum.
Please let me know what you think. :)

I like the feature. Please consider accepting the pull-request.


Can I fix the width of each box

Can I fix the width of each box，For example, the width of a box is 200px

I fixed the width of the grid =>
each column gets a portion of that.
1000 max width and 5 columns equals 200 pixel each (and some loss to margin)


Vulnz times out without any log

We recently switched to Vulnz to make the dependency check more efficient by caching it and using the cached files within the maven plugin. This worked great until Februar 5th. Tried to execute it in AWS as well as from local machine and it always runs without any output. After an hour or so I cancel it. In the past it worked within 15min. As nothing changed in the system and lib I'm wondering if it is somehow API related?
I'm building it via gradlew and execute the jar file

unable to reproduce with the information given. We have the cache updating here https://github.com/dependency-check/DependencyCheck_Builder/actions/runs/7900362303/job/21561722360 and these jobs have not failed.

not sure if you figured things out? Could be an issue with a proxy?


Typo in doc block

I figured I might as well just fix it.

@mitchjam Thanks. :)  Unfortunately, that part was generated by webpack and has nothing to do with my code. I'm still glad you're offering to help anyway. ;)


Adding product features for PhysicalInfra Overview page

Add config into product features to display the overview page for Physical providers.
Related to: https://github.com/ManageIQ/manageiq-ui-classic/pull/4372

@agrare


MAINT: object is an implicit base for all classes

Reference issue
None.
What does this implement/fix?
In Python 3, object is an implicit base for all classes. No need to add it explicitly.

Thanks @DimitriPapadopoulos !


Clearing Ops - GRG Canvas Area Tool model and "Canvas Area GRG Script" not referencing tools

There was a problem with windows not finding the path to the toolboxes.
I was able to get it working on my machine by changing the first line of code (commented out) to the second.
#toolboxPath = os.path.dirname(os.path.dirname(arcpy.env.workspace)) + "\toolboxes\ClearingOperations.tbx"
toolboxPath = os.path.dirname(arcpy.env.workspace) + "\toolboxes\ClearingOperations.tbx"
#arcpy.ImportToolbox(toolboxPath, "VillageClearing")
arcpy.ImportToolbox(toolboxPath, "ClearingOperations")
...........................
...........................
Also fixed references to the .lyr file at the end of the script.
#symbologyPath = os.path.abspath(os.path.join(os.path.dirname(arcpy.env.workspace), os.pardir)) + "\data\Layers\GRG.lyr"
symbologyPath = os.path.dirname(arcpy.env.workspace) + "\data\Layers\GRG.lyr"

@jscott11 can you check it this is 'fixed' or not?

@mfunk this is pretty much the same problem Joe was having with #185 and #186 . I'm hoping to have a fix submitted for this today, and that'll take care of all 3 of these.

@jscott11 excellent. thanks for the info!

@jfrygeo as the author of this issue, can you verify?


Add callback for handling of  tags

I'd rather go for an approach that allows customization for any HTML tag

This makes a lot more sense

Can you change the logic to provide the tag name as a parameter into the callback and invoke that method for any tag?

Let me see what I can do 👍

Thanks for your contribution! It's been merged into 1.1.0 with some minor fixes.


Blob Simulator

This adds a methos simulate() to the Blob generator allowing you to pass in a Dataset object and it will build a Blob generator that generates samples from the same Gaussian distribution as the Dataset object.

Thanks for the review @Boorinio!


lima - failed to RunSimpleContainer - Sequoia 15.0.1 - ddev v1.23.5

Preliminary checklist

[x] I am using the latest stable version of DDEV (see upgrade guide)
[x] I have searched existing issues
[x] I have checked the troubleshooting guide
[x] I have run ddev debug test to include output below

Output of ddev debug test
Expand `ddev debug test` diagnostic information
#5 [web 2/8] RUN (groupadd --gid 844858721 "user" || groupadd "puser" || true) && (useradd  -l -m -s "/bin/bash" --gid "puser" --comment '' --uid 421654122 "puser" || useradd  -l -m -s "/bin/bash" --gid "puser" --comment '' "puser" || useradd  -l -m -s "/bin/bash" --gid "844858721" --comment '' "puser" || useradd -l -m -s "/bin/bash" --comment '' puser )
#5 0.097 useradd warning: puser uid 421654122 outside of the UID_MIN 1000 and UID_MAX 60000 range.
#5 DONE 0.1s

#6 [web 3/8] RUN mkdir -p /home/puser && chown pbabin /home/puser && chmod 600 /home/puser/.pgpass
#6 0.097 chown: changing ownership of '/home/puser': Invalid argument
#6 ERROR: process "/bin/bash -c mkdir -p /home/$username && chown $username /home/$username && chmod 600 /home/$username/.pgpass" did not complete successfully: exit code: 1

uid=421654122(puser) gid=844858721 groups=858925591,12(everyone),62(netaccounts),79(_appserverusr),80(admin),81(_appserveradm),98(_lpadmin),703(com.apple.sharepoint.group.3),702(com.apple.sharepoint.group.2),701(com.apple.sharepoint.group.1),33(_appstore),100(_lpoperator),204(_developer),250(_analyticsusers),395(com.apple.access_ftp),398(com.apple.access_screensharing),399(com.apple.access_ssh),400(com.apple.access_remote_ae)

Expected Behavior
DDEV Start works without chown issues
Actual Behavior
ddev start results in
failed to RunSimpleContainer to chown volumes: container run failed with exit code 1, output=chown: changing ownership of '/var/lib/mysql': Invalid argument
chown: changing ownership of '/mnt/ddev-global-cache/mkcert': Invalid argument
chown: changing ownership of '/mnt/ddev-global-cache': Invalid argument
Steps To Reproduce
Mac Sequoia 15.0.1
Lima
Anything else?
Worked with rflay in Discord to identify this issue and the pertinent pieces
OrbStack - Works without an issue
Colima - Works without an issue
Rancher Desktop works without an issue
Lima - does not work
Rfay wondered if it was a Sequoia issue . . .I do not believe so.  I ran into this same issue the last time I tried switching from Colima to Lima on a M1 Mac on Sonoma

Thanks so much for this and for checking in the various places. I guess it must be related to the Linux kernel being used in these different contexts.
From DDEV's perspective I think this is controlled by GetContainerUIDGid(): https://github.com/ddev/ddev/blob/e5bcf28e824b4f4e7636e270258afb43305f2a1c/pkg/util/utils.go#L167-L211
For testing, I think the uid/gid you failed with from your mac (altered slightly) were 521674292/855425821
Discord discussion: https://discordapp.com/channels/664580571770388500/1299022664257503292

I think this must be related to the build-time kernel or something. We add the matching username/uid/gid at build time.
On Docker Desktop, I can add this to .ddev/web-build/Dockerfile and it works fine:
RUN useradd -u 999999 junk

However, if I try sudo -s and then
# useradd -u 99999999 j3
useradd warning: j3's uid 99999999 outside of the UID_MIN 1000 and UID_MAX 60000 range.

I imagine that's because Debian itself rejects the uid based on /etc/login.defs.

Asked in Lima slack, https://cloud-native.slack.com/archives/C043N6ZFV9S/p1729789346868649

This fails only on Lima, only with huge UIDs (enterprise environments) and the Lima devs have no idea about it.
I don't think we'll chase it. Switch to OrbStack or Colima or Rancher Desktop or Docker Desktop.
Thanks for reporting it!


[Feature Request] Multilingual support

Hi, thank you for sharing the wonderful project. I'd like to evaluate my Korean RAG applications, there's no RAG evaluation framework supporting multilingual. Could you add this feature for non english developers? If you guide me how to add codes to support multilingual, I'll contribute to it.
Thanks.

Hi @sociengineer,
Thank you for your interest in our project! ARES recently has gone under new development and now supports multilingual evaluations, including for Korean RAG applications. You would need to swap out the ARES judge used to ensure it can appropriately handle multilingual content, a process made flexible with the use of vLLM, refer here. For more information view our quick start guide, and if you need further guidance on how to make these modifications within ARES, we'd be happy to help.


🧑💻 : Enhancing generate gradient page

:red_circle: Title :  Enhancing generate gradient page
:red_circle: Enhancement Aim : To improve the style of the page
:red_circle: Changes :

Include the color scale.
Include RGBA values along with hex values
Encoding the css code in code block.
List shades of different colors.

Screenshots 📷

✅ Details to Include When Taking the Issue:
Name : Arveti Likhitha
Participant Role (Specify the Open Source Program name, e.g., GSSOC, Hacktoberfest, etc.): gssoc-ext, hacktoberfest-accepted

Happy Contributing! 🚀
Wishing you all the best on your open source journey. Enjoy! 😎

I am not able to run the application because of the following error. Could you help me in fixing it?


Fix/arm disarm

added command id 400 in mavric to receive arm/disarm command from qgroundcontrol 2.8.0

Hi @alexcherpi , any progress on this?

@alexcherpi I accept it as it is because we need it for a project, and opened a follow-up issue #200


RFC: Add STUN enable/disable, STUN host&port

/etc/init.d/miniupnpd already has variable support for these additions to the miniupnpd packages.
This adds configuration support into Luci for those items.

Closing as I have made a mess of this pull and know better with regards to submitting. Apologies.


ICE trickling functionality

@tomaszdurka @vogdb
Here's a simple ICE trickling demo:
https://webrtc.github.io/samples/src/content/peerconnection/trickle-ice/
And the source code:
https://webrtc.github.io/samples/src/content/peerconnection/trickle-ice/js/main.js

Very nice. Thanks. I thought that we want to implement this functionality in a new project/repo that depends on janus-gateway-js cause janus-gateway-js is a core without WebRTC stuff.

I think that the ICE trickling functionality is inseparable from createOffer/createAnswer logic of Janus. Also I think that ICE is a subpart of createOffer/createAnswer logic and hence we should start from createOffer/createAnswer logic first. @njam wdyt?

I think I agree, that we should add ICE functionality to this client. @zazabe what are your thoughts? You know the topic better.

Agree, ICE and createOffer/createAnswer are about the RTCPeerConnection, should be implemented in the WebRTC specific lib in a new repo, as you said @vogdb

We would need a cross-browser RTCPeerConnection for that. The smaller, the better.

https://github.com/js-platform/node-webrtc. No. Too huge. Provides the whole webrtc functionality. Has dependencies like python
https://github.com/nickdesaulniers/node-rtc-peer-connection. Raw. Not ready.
https://github.com/otalk/RTCPeerConnection. Claims to be tiny. It's tiny only for npm usage. It's bundle file worth more than 200kb. Unacceptable for browser usage.

Don't know what to use. Seems that it's better to make a restriction to use only modern browsers and not to use any shims/wrappers.

Maybe there are some useful parts here: https://simplewebrtc.com/ ("Need More Control?"):

Signalmaster — Simple node.js/socket.io powered signaling server.
RTCPeerConnection — Wraps/normalizes peer connections.
getUserMedia — Wraps/normalizes getting access to user's camera/mic.
attachMediaStream — Normalizes adding streams to video tags and handles muting/mirroring in one simple API.
webrtcsupport — For checking browser capability and grabbing appropriate constructors for stuff like RTCPeerConnection and RTCSessionDescription as well as checking support for data channels, etc.
getScreenMedia — Extends getUserMedia but for getting access to screen media for screensharing.
mediastream-gain — Gives you a simple audio input volume controller.
hark — Detect who's speaking using the Web Audio API.
WildEmitter — Slick, minimalist EventEmitter with support for wildcards.

Work still in progress. Pushing here for the information.

[Current coverage][cc-pull] is 94.15%

Merging [#24][cc-pull] into [master][cc-base-branch] will increase coverage by 3.03%

@@           master        #24   diff @@
========================================
  Files          12         12
  Lines         462        479    +17
  Methods         0        123   +123
  Branches       58         58
========================================
+ Hits          421        451    +30
  Misses         28         28
+ Partials       13          0    -13

Powered by Codecov. Last updated by [fe4922b...59b1a6a][cc-compare]
[cc-base-branch]: https://codecov.io/gh/cargomedia/janus-gateway-js/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/cargomedia/janus-gateway-js/compare/fe4922bc8689aabc83cdd3dfcf69b4be43c0a2e6...59b1a6aa91fde9466ad69c0021f96d220896073e
[cc-pull]: https://codecov.io/gh/cargomedia/janus-gateway-js/pull/24?src=pr

The progress here is very good. The only remaining thing is rtp-broadcast-plugin now. Lets split the changes here on separate minor pull requests? Otherwise it will be hard to review I am afraid. @tomaszdurka @njam

Could you discuss with Michel what's the best way of finishing this up and using it in SK?
I discussed with Tomasz that we maybe should first use the code in cm-janus, because the integration might be simpler. But if this is already done SK is also an option. wdyt?

We should try it in SK if it has sufficient functionality.

I will split this PR on multiple PR


How to do “Matching to Optimise HOTA”

It is easy to calculate hota metrics after detection/gt match. But it is quite confusing about matching rule.
From the paper, hota is calculated independently under different iou threshold for 2d bbox tracking. And the Eq15 is to do Hungarian algorithm on Amax add weighted iou.
But it is some kind of Amax multiply iou in this repo.
It is hard to understand why it's guaranteed to meet maximum hota on all possible detection/gt match, both on paper and code.
For a tiny example on iou 0.5 threshold:
data['gt_ids'] = [np.array([0])] * 9 + [np.array([0,1])]

data['tracker_ids'] = [np.array([0])] * 10

data['similarity_scores'] = [np.array([[1]])] * 9 + [np.array([0.49,0.51]).reshape(2,1)]

data['num_tracker_ids'] = 1

data['num_gt_ids'] = 2

In the example, the code will discard last frame detection under iou 0.5 threshold, DetA 0.75 AssA 0.818 HOTA 0.783
But just let last frame detection to match gt id 1:  DetA 0.909 AssA 0.746 HOTA 0.8237
Just do linear_sum_assignment(-similarity) to reproduce.

I agree with this - how come there is such a huge discrepancy between the method used in the paper and the one in the repo? I would appreciate some more clarity on this, as it is quite a crucial part of the metric

I agree. I'm currently trying to understand what's going on in the paper by cross-referencing it to the implementation in this repo and it's been very difficult to make sense of any of it, especially because i started with the assumption that the implementation should be equivalent to the description in the paper.
It would be very helpful to at least have comments in the source code that note the parts that are different from the paper.


"divided by zero" issue of Noah3.9 has been fixed.

That sounds like a good idea since we are touching the original physics.


Function to add or update locations

There's a very popular store near me which isn't on the map, and I was hoping you could add a way to suggest new locations, either in the "Not Found" results box, or by clicking (or long-tapping for mobiles) on the map.

Hi,
yes, I'm working to make this possible.
I received a lot of feedback about this issue.
I hope to release it in the next few days

Awesome! Great work, by the way!

Can you check it today?
I removed the filter, so now you should search/see your place on the map.
It will be a blue marker, because I did not receive any information about the time spent or the populartimes for these places.
Let me know,
thank you

Thank you for the feedback.
From your screenshot I can see that some buttons are missing from the UI.
I think that something is being cached: can you try to refersh/remove the cache? Or can you try to visit the website instead of using the app?
This morning I released a new version of UI/UX.
Thank you

I cleared the cache on my phone's browser and everything looks good now. Thanks!

Hooray!
I'm still working on it, but in these days I'll monitor this feature.
I'm gonna close this issue, feel free to reopen.

Thank you for your precious feedback


Fix some bugs

Now dp should be the third dimension of parallel ranks
One missing argument

Now dp should be the third dimension of parallel ranks

Could you please elaborate on this?

One missing argument

Where is it 🤔

It is quite straightforward by looking at the definition of rank matrix
https://github.com/huggingface/nanotron/blob/ff3c7746577948743da08c4868aca46cbc0c110b/src/nanotron/parallel/context.py#L69-L76

Missing is_expert_sharded variable would cause referenced before defined error

https://github.com/huggingface/nanotron/blob/ff3c7746577948743da08c4868aca46cbc0c110b/src/nanotron/serialize/weights.py#L72-L95