Custom converter/adapter for @Option

Expected Behavior
You can put @Option(option="somename", converter=Some.class) on property and @Option infrastructure calls the converter for us.
Current Behavior (optional)
@Option can only set a value provided from the command line as-is.
command line
Context
For example, if a task option is Configuration, then it can be configured from CLI by name.
With plain setters/getters, it's easy to have an additional setter.
With a Property, it's clunky. We need an extra property and a way to wire the original property and the intermediate one.
A tempting solution is to have a convention on the original property to be the mapped intermediate value. However, this is not a solution since CLI args should have higher precedence over configurations.
Another reason is that it might be undesirable to expose the public setter for non-CLI contexts.

Related:

https://github.com/gradle/gradle/issues/29256


[OpenSearch Playground]OSD took a long time to start causing failure in OSD up checks

Is your feature request related to a problem?
After OSD is deployed, it took a long time to start causing failure in the FT test workflow to fail when it performs status checks before running FT. Often a manual retry is needed at this moment in order to finish the rest of tests
What solution would you like?

auto retry the status check

What alternatives have you considered?
Manually trigger retry after OSD is up
Do you have any additional context?
NA

We can add retry for now, but also good to investigate why it took so long to boot up.
Tried that curl -XGET --retry 15 --retry-delay 30 --retry-all-errors <url> would retry for no response from server error. Initially used 10 retries, and the last retry worked, thus we can use 15 retries for now.


xkb-switch-i3: 1.8.1 -> 1.8.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/xkb-switch-i3/versions.
meta.description for xkb-switch-i3 is: "Switch your X keyboard layouts from the command line(i3 edition)"
meta.homepage for xkb-switch-i3 is: "https://github.com/Zebradil/xkb-switch-i3"
meta.changelog for xkb-switch-i3 is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/07fs9izifpxrfadw0l6vjmnrffysjadw-xkb-switch-i3-1.8.5/bin/xkb-switch had a zero exit code or showed the expected version
0 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 1.8.5 with grep in /nix/store/07fs9izifpxrfadw0l6vjmnrffysjadw-xkb-switch-i3-1.8.5
found 1.8.5 in filename of file in /nix/store/07fs9izifpxrfadw0l6vjmnrffysjadw-xkb-switch-i3-1.8.5

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
0 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
xkb-switch-i3

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/07fs9izifpxrfadw0l6vjmnrffysjadw-xkb-switch-i3-1.8.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A xkb-switch-i3 https://github.com/r-ryantm/nixpkgs/archive/0349c745d1f615f2dc1c7ce78248c13af2bf5944.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/07fs9izifpxrfadw0l6vjmnrffysjadw-xkb-switch-i3-1.8.5
ls -la /nix/store/07fs9izifpxrfadw0l6vjmnrffysjadw-xkb-switch-i3-1.8.5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

xkb-switch-i3

Maintainer pings
cc @ewok for testing.

Result of nixpkgs-review pr 166759 run on x86_64-linux 1

1 package built:

xkb-switch-i3

Result of nixpkgs-review pr 166759 run on x86_64-linux 1

1 package built:

xkb-switch-i3


I dont understand how to do it

ummmm how do i use it???

Install OpenCV with Python3 bindings
Install any missing dependencies with Pip
Install pywin32, the Windows API bindings for Python.
Install the Nvidia CUDA toolkit if you have a compatible GPU or OpenCL if you don't
Then run the program with Python3 with python3 pine.py


Transactions are auto-committed

I am using Entity Framework code-first approach. I create transactions in the code, and they are never committed. Despite NOT having a commit - some of the transactions end up being saved to the database!
This ticket could be either an issue or a question. I am willing to understand if it's an expected behavior. If it is - then please share some comments on this. Otherwise if it's a bug then you might want to fix it in the next milestone.
Steps to reproduce
Consider following testcase:
[TestMethod]
public void Test1()
{
    int invoiceId = 1;

    Parallel.For(0, 30, i =>
    {
        var context1 = new ApplicationDbContext();
        var transaction1 = context1.Database.BeginTransaction();
        var invoice1 = new Invoice()
        {
            InvoiceId = (invoiceId + i).ToString()
        };
        context1.Invoices.Add(invoice1);
        context1.SaveChanges();

        //transaction1.Commit();
    });
}

There is no commit done, but, after you run the test several times - you will then see that new records appear in DB. There are total of 30 transactions created. Only 0 to 5 of them will be auto-committed, on average. I couldn't find any reasons why it's happening, so if it's working for you just fine - try increasing a number of transactions to 100 or running test multiple times.
Fix
A quick for this - is to dispose of DB context using a "using" block, like so:
[TestMethod]
public void Test2()
{
    int invoiceId = 3350;

    Parallel.For(0, 30, i =>
    {
        using (var context = new ApplicationDbContext())
        {
            var transaction = context.Database.BeginTransaction();
            var invoice = new OracleDatabaseService.Models.Invoice()
            {
                InvoiceId = (invoiceId + i).ToString(),
                ClientId = "3",
                ExternalSystemId = "0"
            };
            context.Invoices.Add(invoice);
            context.SaveChanges();

            //transaction.Commit();
        }
    });
}

This works fine, though, I think this should not be a fix, because sometimes you will get exceptions in the code, meaning that destructor will NOT be called and same data might end up in DB. So what I am looking forward to - is a proper fix, which will make sure that transaction is committed once you explicitly wish for it!
Sources
Following code should help you with issue recreation:
[System.ComponentModel.DataAnnotations.Schema.Table("Invoice")]
public class Invoice
{
	[Column("INVOICE_ID")]
	[MaxLength(100)]
	public string InvoiceId { get; set; }

	[Required]
	[Column("DATE_CREATED")]
	public DateTime DateCreated { get; set; }

	[Column("DATE_CHANGED")]
	public DateTime? DateChanged { get; set; }
}

and
public class ApplicationDbContext : DbContext
{

	public ApplicationDbContext() : base("ApplicationDbContext")
	{
		this.Configuration.LazyLoadingEnabled = false;
	}

	public DbSet<Invoice> Invoices { get; set; }

	protected override void OnModelCreating(DbModelBuilder modelBuilder)
	{
		Database.SetInitializer<ApplicationDbContext>(null);
		base.OnModelCreating(modelBuilder);

		string connectionSTring = this.Database.Connection.ConnectionString;
		var connectionStringBuilder = new OracleConnectionStringBuilder(connectionSTring);
		string schemaName = connectionStringBuilder.UserID.ToUpper();
		modelBuilder.HasDefaultSchema(schemaName);

		modelBuilder.Entity<Invoice>()
			.HasKey(pc => new { pc.InvoiceId });
	}
}

Further technical details
EF version: 6.2.0
Database Provider: Oracle.ManagedDataAccess.EntityFramework
Operating system: Windows 10 x64
IDE: Visual Studio 2017 15.7.5 Pro

@pavlexander Couple of things here. First, is is required that the DbContext is disposed. Note that an exception occurring inside the using statement (or other try/finally construct) will still result in the context instance being disposed.
Second, if the context is not disposed, then typically the DbConnection will not be disposed, which can mean that native resources are not correctly cleaned up. It seems possible from your example that this results in something going wrong in the Oracle ADO.NET provider, which then causes the transaction to be committed. You would need to follow up with Oracle on this since this is in their code, not EF or ADO.NET.


Turning Off the Utility Class Validator

Hey @billyma - the utility class suggestions provided by the validator aren't accurate. It is giving some questionable advice, so we should just disable it for now until we revisit this.

Issue is addressed with the PR above.


fix benchmark format reporting in buildkite

Unbreak this UI

Oops - Somehow I missed this PR until now. Thanks for fixing it and I'll be more careful with the CI!


Hello there. After flutter version 2.2.1 is upgraded, the packaging speed of android app. is much slower than before. Is there any way to troubleshoot this?

Steps to Reproduce

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

Expected results:
Actual results:

Logs

@yaoming04111
When opening an issue, please try to provide all or some of the information indicated in the 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 this issue, since no information has been provided.
Thank you.


Generate API Docs

What this PR does / why we need it:
Adds Scripts to generate API documentation for the site and makes sure it's up to date in CI.
Special notes for your reviewer:
Many API type definitions violated the generators validations so have been changed to follow convention.
Generate API documentation in site

/assign @simonswine

/close

/reopen

/test verify docs

/unassign
/assign @simonswine

Also the generated code looks a bit different for me can you address that, it should fix that:  npm notice created a lockfile as package-lock.json. You should commit this file.

/unassign
/assign @simonswine
@simonswine This should fix these issues, LMK if you still have problems

One difference with the committed output for me is:
	typechange: docs/generated/reference/output/node_modules/.bin/marked
	typechange: docs/generated/reference/output/node_modules/.bin/mime
	typechange: docs/generated/reference/output/node_modules/.bin/static

These became files instead of symlinks.

/unassign
/assign @simonswine
/assign @MattiasGees

/unassign @MattiasGees
Thanks @JoshVanL, I will rebase this

@simonswine Sorry didn't see your message, went ahead and did it

Thank you @JoshVanL this is a great improvement!
/approve
/lgtm


Heroes Unlimited 2E, lift calculation bug fix

Submission Checklist
Required

[ ] The pull request title clearly contains the name of the sheet I am editing.
[ ] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[ ] The pull request makes changes to files in only one sub-folder.
[ ] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[ ] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description
Corrected a bug where lift capacity was not double carry capacity with normal strength above 17.
Corrected a bug where lift capacity was being calculated as x400 instead of x500 of ps score with supernatural strength toggled on in settings.
Corrected a bug that kept an input field from correctly retaining text (somehow a class designation had migrated to type in that line).
Added a note about the update to the documentation tab.

Character Sheet Info Roll20 Internal Use only.


Finalise Validate trait

The Validate trait is used for checking the correctness of glTF JSON. The wrapper library assumes validation has succeeded as an invariant. This ensures classic bugs such as out-of-range indices are checked upfront on behalf of the user. The final design of the Validate trait needs to be agreed upon prior to the 1.0 release.
Currently implemented features:

Index range checks for all top-level glTF objects.
Index range checks for animation samplers.
Semantic name correctness.
Restrictions on parameter values.

Remaining features include:

Ability to terminate validation early (more important.)
Ability to opt-out of validation (more important.)
Ability to ignore 'less severe' validation errors (less important.)

I would like to be able to write the following:
struct Foo<'a> {
    bar: Bar<'a>,
    baz: i32.
}

impl<'a> Validate<'a> for Foo<'a> {
    fn validate<P, R>(&self, root: &Root<'a>, path: P, report: &mut R) -> Action
    where
        P: Fn() -> JsonPath,
        R: FnMut(Error),
    {
        // Exits early if return value is `Action::Stop`.
        self.bar.validate(root, || path.field("bar"), report)?;

        // The `report` callback may decide to stop iteration by returning `Action::Stop`.
        if self.baz == 42 {
            return report(Error::invalid_value(self.baz, format!("must not be `42`")));
        }

        // Continue the validation process
        Action::Continue
    }
}

where
enum Action {
    Continue,
    Stop,
}

In order for this to be possible in stable Rust, we must either create a macro similar to try! that expands to
if self.bar.validate(root, || path.field("bar"), report) == Action::Stop {
    return Action::Stop;
}

or wait for the Try trait to become stable and implement it on Action.
I'm not fond of the idea of using Result<(), ()> instead because it's neither idiomatic nor future proof.

https://github.com/alteous/gltf/commit/9f5dcb138796594ec05df34348babaa170d667d8 implements the option to choose the validation strategy.


Sims 2 UC: The problem after the family settled on the lot.

I have Sims 2 Ultimate Collection with DXVK 2.1 and NoErrorVisualization mod by Lazy Duchess, also added boolprop reparseshaders true cheat to userstartup.cheat file.
And now I will describe the following case: I created my family, settled it, and this is what happened:
https://ibb.co/r5Xpgn6
HP Pavilion Gaming Laptop 15:
Windows 10 Home (Russian) SL 64bit 22h2
Intel Core i5-8300H 2.3 GHz
Intel UHD Graphics 630
NVIDIA GeForce 1060 MaxQ
12GB RAM
What to do?

Does it only happen with DXVK? Does it look fine if you remove DXVK?

Please provide logs and make an apitrace. https://github.com/doitsujin/dxvk/wiki/Using-Apitrace

Here they are:
https://mega.nz/folder/bT4AyR6Y#C07L0g3QMa81n2c5aGIZDg

Does it only happen with DXVK? Does it look fine if you remove DXVK?

I tried to play without DXVK, but the game crashed while the family was settling in.

Instead of copying the apitrace dll file, can you try apitrace.exe trace -m Sims2EP9.exe with dxvk?

I think I've seen a visual issue like this before and it too happened on PCs with Intel graphics, on Windows.

@mrdeathjr28 do you by any chance dualboot Windows? I saw you have a machine with UHD graphics so it would be nice if you tested Sims 2 with Intel graphics on Windows

@mrdeathjr28 do you by any chance dualboot Windows? I saw you have a machine with UHD graphics so it would be nice if you tested Sims 2 with Intel graphics on Windows

I hate windows
But this images are with dxvk 2.3 and last driver from here:
https://launchpad.net/~ernstp/+archive/ubuntu/mesaaco

And graphicrulesmaker 2.30 x64 are used too: https://www.simsnetwork.com/tools/graphics-rules-maker

@Burinis Friendly ping. Could you do a recheck of this with dxvk 2.4 and latest graphics drivers

@Burinis Double friendly ping


Add S3 access and secret keys to localsettings

@gcapalbo @dannyroberts
Goes with: https://github.com/dimagi/commcare-hq/pull/8750

ooooh yeaaa. also i can make a request to get staging machines with at least 8GB of RAM?


MM-40127 : Fix team sidebar buttons screen reader hover

Summary
Fix screen reader does not read out buttons in team sidebar on hover.
Screen reader now say:

Team image: "{team name} Team Image"
Team initials: "{team name} Team Initials"
Plus icon: "Plus icon"

Can't fix team sidebar button on click because the focus is given by default to the post textbox after the page load.
So, screen reader read the post textbox aria-label.
Update team_icon.test.tsx.snap snapshot.
Ticket Link
Fix https://github.com/mattermost/mattermost-server/issues/19161
Jira: https://mattermost.atlassian.net/browse/MM-40127
Release Note
NONE

Hello @jbattistispiria,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

/update-branch

/e2e-test

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/188257

Thanks @jbattistispiria
Tested, looks good to merge.

Verified screen reader is reading out team buttons.
E2E report looks good, no PR related failures.


Naver TV download not working

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2022.06.29 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
South Korea
Provide a description that is worded well enough to be understood
Naver TV download not working
https://tv.naver.com/
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
C:\UTIL\yt-dlg>yt-dlp -vu https://tv.naver.com/v/27746776
[debug] Command-line config: ['-vu', 'https://tv.naver.com/v/27746776']
Type account password and press [Return]:
[debug] Encodings: locale cp949, fs utf-8, pref cp949, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.06.29 [9d339c4] (win32_exe)
[debug] Python 3.8.10 (CPython 64bit) - Windows-10-10.0.19044-SP0
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg N-107010-g0dcbe1c1aa-20220526 (setts), ffprobe N-107010-g0dcbe1c1aa-20220526
[debug] Optional libraries: Cryptodome-3.15.0, brotli-1.0.9, certifi-2022.06.15, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.3
[debug] Proxy map: {}

That's not a full verbose log and doesn't demonstrate the error

The correct spelling you're asked to type is -vU (a dash, little V, capital U).
With little U, you're setting username for log in!

Apologies @lunamoth, I assumed you had just only posted a partial verbose log (believe it or not, lots of people do that). Anyway, can you please do what Lesmiscore said and then post a new verbose log?

Sorry for the inconvenience. log again

C:\UTIL\yt-dlg>yt-dlp -vU https://tv.naver.com/v/27746776
[debug] Command-line config: ['-vU', 'https://tv.naver.com/v/27746776']
[debug] Encodings: locale cp949, fs utf-8, pref cp949, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.06.29 [9d339c4] (win32_exe)
[debug] Python 3.8.10 (CPython 64bit) - Windows-10-10.0.19044-SP0
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg N-107010-g0dcbe1c1aa-20220526 (setts), ffprobe N-107010-g0dcbe1c1aa-20220526
[debug] Optional libraries: Cryptodome-3.15.0, brotli-1.0.9, certifi-2022.06.15, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.3
[debug] Proxy map: {}
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
[debug] Downloading update_spec from https://github.com/yt-dlp/yt-dlp/releases/download/2022.06.29/update_spec
Latest version: 2022.06.29, Current version: 2022.06.29
yt-dlp is up to date (2022.06.29)
[debug] [Naver] Extracting URL: https://tv.naver.com/v/27746776
[Naver] 27746776: Downloading JSON metadata
[Naver] 27746776: Downloading JSON metadata
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id
[debug] Default format spec: bestvideo*+bestaudio/best
[info] 27746776: Downloading 1 format(s): avc1_1080P
[debug] Invoking http downloader on "http://b01-kr-cdn.vod.naver.net/navertv/c/read/v2/VOD_ALPHA/navertv_2022_07_04_308/551f73ff-fb76-11ec-ba5e-48df37269fd0.mp4?lsu_sa=6665b1fb21d462e637dbd58a6c75b6b32e0c3e68d50e5f6334076cc5871636e5f527aa316b75db06601735320e3ecb49610bfee0615069055294ed96784afd3f672bfe7a8f2bd590460cef1c94f0626d"
[download] Got server HTTP error: <urlopen error [Errno 11004] getaddrinfo failed>. Retrying (attempt 1 of 10) ...
[download] Got server HTTP error: <urlopen error [Errno 11004] getaddrinfo failed>. Retrying (attempt 2 of 10) ...
[download] Got server HTTP error: <urlopen error [Errno 11004] getaddrinfo failed>. Retrying (attempt 3 of 10) ...
[download] Got server HTTP error: <urlopen error [Errno 11004] getaddrinfo failed>. Retrying (attempt 4 of 10) ...
[download] Got server HTTP error: <urlopen error [Errno 11004] getaddrinfo failed>. Retrying (attempt 5 of 10) ...
[download] Got server HTTP error: <urlopen error [Errno 11004] getaddrinfo failed>. Retrying (attempt 6 of 10) ...
[download] Got server HTTP error: <urlopen error [Errno 11004] getaddrinfo failed>. Retrying (attempt 7 of 10) ...
[download] Got server HTTP error: <urlopen error [Errno 11004] getaddrinfo failed>. Retrying (attempt 8 of 10) ...
[download] Got server HTTP error: <urlopen error [Errno 11004] getaddrinfo failed>. Retrying (attempt 9 of 10) ...
[download] Got server HTTP error: <urlopen error [Errno 11004] getaddrinfo failed>. Retrying (attempt 10 of 10) ...
[download] Got server HTTP error: <urlopen error [Errno 11004] getaddrinfo failed>
ERROR: giving up after 10 retries
File "yt_dlp_main.py", line 17, in
File "yt_dlp_init.py", line 921, in main
File "yt_dlp_init_.py", line 913, in _real_main
File "yt_dlp\YoutubeDL.py", line 3249, in download
File "yt_dlp\YoutubeDL.py", line 3225, in wrapper
File "yt_dlp\YoutubeDL.py", line 1411, in extract_info
File "yt_dlp\YoutubeDL.py", line 1420, in wrapper
File "yt_dlp\YoutubeDL.py", line 1504, in __extract_info
File "yt_dlp\YoutubeDL.py", line 1561, in process_ie_result
File "yt_dlp\YoutubeDL.py", line 2630, in process_video_result
File "yt_dlp\YoutubeDL.py", line 3129, in process_info
File "yt_dlp\YoutubeDL.py", line 2829, in dl
File "yt_dlp\downloader\common.py", line 444, in download
File "yt_dlp\downloader\http.py", line 386, in real_download
File "yt_dlp\YoutubeDL.py", line 962, in report_error
File "yt_dlp\YoutubeDL.py", line 894, in trouble

Seems like a network issue. Confirm that you can access the video in browser from same IP
❯ yt-dlp -v https://tv.naver.com/v/27746776
[debug] Command-line config: ['--ignore-config', '-v', 'https://tv.naver.com/v/27746776']
[debug] Encodings: locale cp65001, fs utf-8, pref cp65001, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.06.29 [9d339c41e] (source)
[debug] Lazy loading extractors is disabled
[debug] Plugins: ['SamplePluginIE', 'SamplePluginPP']
[debug] Git HEAD: 385f7f389
[debug] Python 3.10.4 (CPython 64bit) - Windows-10-10.0.22000-SP0
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg N-106550-g072101bd52-20220410 (fdk,setts), ffprobe N-106624-g391ce570c8-20220415, phantomjs 2.1.1
[debug] Optional libraries: Cryptodome-3.14.1, brotli-1.0.9, certifi-2021.10.08, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.1
[debug] Proxy map: {}
[debug] [Naver] Extracting URL: https://tv.naver.com/v/27746776
[Naver] 27746776: Downloading JSON metadata
[Naver] 27746776: Downloading JSON metadata
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id
[debug] Default format spec: bestvideo*+bestaudio/best
[info] 27746776: Downloading 1 format(s): avc1_1080P
[debug] Invoking http downloader on "http://a01-g-cdn.vod.naver.net/navertv/c/read/v2/VOD_ALPHA/navertv_2022_07_04_308/551f73ff-fb76-11ec-ba5e-48df37269fd0.mp4?__gda__=1657018606_697971a488f5aa44be483f370018ccec"
[download] Resuming download at byte 16035264
[debug] File locking is not supported. Proceeding without locking
[download] Destination: 친구들 맞이하는 SON '동료들의 기대, 걱정이에요' [스포츠타임] [27746776].mp4
[download]  14.8% of 116.51MiB at    2.47MiB/s ETA 00:40

The problem was caused by blocking b01-kr-cdn.vod.naver.net in NextDNS. It wasn't a problem. Sorry to bother everyone.


symbol's function definition is void: org-journal-find-location

In the readme, the following capture template is recommended:
      ("j" "Journal entry" entry (function org-journal-find-location)
       "* %(format-time-string org-journal-time-format)%^{Title}\n%i%?")

However, there doesn't appear to be a function called org-journal-find-location.
What's the preferred way of using org-journal with org-capture?

But in the README right above the snippet you posted is also the org-journal-find-location function definition. In other words, you are supposed to write one.


Add new manipulation and analyze methods for collections.

Solve #13.
This will add the following methods:

column
first
last (additionally to first)
sort
filter
where
map
diff
intersect
unique
merge

The methods have been implemented according to the suggested API. If you want any change to these methods. Keep me posted.

Will update after I got my little girl to sleep ;)

Would using array_values instead of merging it with an empty array work?
The array_merge with the extra array is a little confusing to read, and I
wonder about performance?
On Fri, Oct 5, 2018, 11:48 AM Andreas Frömer<EMAIL_ADDRESS>wrote:

@icanhazstring commented on this pull request.
In src/AbstractCollection.php
https://github.com/ramsey/collection/pull/44#discussion_r223055272:

       $temp[] = $this->extractValue($item, $propertyOrMethod);

   }

   return $temp;

}

/**

* @inheritDoc

*/

public function first()
{

   if (empty($this->data)) {

       throw new OutOfBoundsException('Can\'t determine first item. Collection is empty');

   }

   $temp = array_merge([], $this->data);

Because of the nature of array_shift and array_pop which will alter the
target array. This will make sure I will lose the reference of the current
data array.
And because the key of the data list is not necessary numeric I just use
internal function to get the last or first item.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ramsey/collection/pull/44#discussion_r223055272, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEAm5lH_ohfG1XCVjKMBskEXkTLODjxaks5uh39ngaJpZM4XC_Mg
.

Yep. I just changed it to use the internal array pointer with current() and end() performance should be much better then.

@jmauerhan renamed the exception.
@ramsey if we are good you can merge it :)

Awesome. Thank you :blush:


not work for markdown

hello, i am using vim8 with Ubuntu 18.04.
following the wiki page to install Tagbar and markdown2ctags, i found that, it is NOT working.
can you provide some guides on this issue?
~/.vimrc setup:

 Plug 'majutsushi/tagbar'

let g:tagbar_type_markdown = {
     \ 'ctagstype': 'markdown',
     \ 'ctagsbin' : '~/.vim/markdown2ctags.py',
     \ 'ctagsargs' : '-f - --sort=yes',
     \ 'kinds' : [
       \ 's:sections',
       \ 'i:images'
     \ ],
     \ 'sro' : '|',
     \ 'kind2scope' : {
         \ 's' : 'section',
     \ },
     \ 'sort': 0,
 \ }

You're going to have to provide more information than "it's not working".  I can't work with that.    If you don't have a markdown plugin installed, then the filetype might not be setup correctly and tagbar will fail to make use of the settings in your .vimrc.  FWIW, I use this one.
BTW, you said:

following the wiki page to install Tagbar and markdown2ctags

What wiki page?  There's no wiki page for this in my project.  Are you following directions from somewhere else?

Note: if you don't have a Markdown plugin installed in your vim setup, then the default behavior is to associate .md files with Modula-2.  I'd expect TagBar not to work as expected in that case.

hi @jszakmeister ,

I follow this guide to install the tagbar for Markdown

https://github.com/majutsushi/tagbar/wiki#markdown

running the markdown2ctag.py in vim, i got the error below:
E488: 多余的尾部字符

but running in terminal, it works ok.

markdown-related plug-in list

"markdown syntax plugin
Plug 'godlygeek/tabular'
Plug 'plasticboy/vim-markdown'
Plug 'iamcco/markdown-preview.vim'
Plug 'shime/vim-livedown'

The reason is that, thie file markdown2ctags.py did not have suficent authority.

What do you mean?  If you were able to run markdown2ctags.py on the command line, then you had the required permissions already.

Following command resolves the issue. Thanks for your time!

What command?  You didn't post it. :-)

BTW, E488 issue still exists.

Again, you'll have to provide more information here for me to do anything.  I can't really do much with the error provided as I can't read the error message and it doesn't appear to point to anything specific.  Have you looked at the output of :messages in vim?  You can also start vim with debug logging and save that to a file:
vim -V9debug.log file.txt

Then reproduce the error and take a look at the logs to see where it's coming from.
Also, does it only happen in a specific document or does it happen all the time?  If it's a specific document, try running markdown2ctags.py on the command line against the file just to make sure it's not failing in some way.  If it is, I definitely want to know that.  If it's not, then the bug may be in TagBar (or some other plugin) and not in markdown2ctags.

Sorry to make you confused.
I use copy command to recreate the file markdown2ctags.py then it works.
In additional, i change the file authority with command:
chmod 777 markdown2ctags.py
thanks for your info on the trouble shooting tips!


Method descriptions - Issue #431

@larshp

the methods table(type = seoo_methods_r) contains a lot of data. It would be cool if a method can be added in the ABAP code, and the XML will not need to be updated. Currently it looks like a lot of the ABAP code will be replicated in the xml(eg. method parameters)

Maybe I could only put the descriptions in the XML?
Check what was the result of the XML with  in the example here:
https://github.com/EduardoCopat/abap-sandbox/blob/master/zec_b_class.clas.xml#L48

Descriptions only: yes
Implementation idea: When deserializing call SEO_INTERFACE_TYPEINFO_GET with version = inactive, and add the descriptions and call SET, dont know if it will work or not

Description: Is the impact in performance so high?
Idea: Why version inactive?

description: it is not due to performance; ABAP is moving towards source based editing, so I would like abapGit to store most of the data in the ABAP source code file
Idea: the ABAP source code is deserialized to inactive state
Idea 2: Alternatively find the place where the method descriptions is stored(table/function modules)

So only include the data from database table SEOCOMPOTX. It looks like the table can be updated directly, or alternatively function module LXE_OBJ_TEXT_WRITE_CLAS looks promising

Efforts so far: I tried only to put the "methods descriptions" in the UML, but when deserializing it does not work.
XML Example: https://github.com/EduardoCopat/abap-sandbox/commit/e6236ac0a3c0c6c297858129ca516cfe01c7c359#diff-c55e45c781a08dc936e43873ba4e948aR59
Might be the case that a direct update to the database table might be needed, but I'd like to avoid that.

Direct update on SEOCOMPOTX does not work. The class components (methods) were not created yet.
Latest try here: https://gist.github.com/EduardoCopat/18df63e5225039721c55ac19c0d4b6f7
Either we add more footprint in the XML or we don't save the descriptions.
P.S.: In ABAP 7.5, the method descriptions/help are in the source code: https://blogs.sap.com/2015/10/21/new-abap-doc-features-with-netweaver-75/

Can you put the code in a branch somewhere? It makes it easier to try out
I guess the update of the database table was successful? And something else removed the entries, or?

Can you put the code in a branch somewhere? It makes it easier to try out

https://github.com/EduardoCopat/abapGit/blob/method-description-simple-xml

I guess the update of the database table was successful? And something else removed the entries, or?

using MODIFY the entries were inserted, but probably something else override them (activation?)

seems to work for me, however I cannot figure out how function module SEO_CLASS_TYPEINFO_GET works, suggest just selecting directly from database table SEOCOMPOTX instead of calling the function module. This will also fix attribute + type descriptions.
Also it could be nice if the row is not included in the XML if the description is empty, and a check to only add the descriptions table to the xml if it contains rows

I will test it again then.. do not merge this request yet :)

@larshp I could make it work now using Function Module SEO_METHOD_MODIFY. Check an example XML here: https://github.com/EduardoCopat/abap-sandbox/blob/master/zec_d_class.clas.xml
Only the descriptions are added. Is it better?
Do note pull request yet, because I still need to somehow put this under test, do refactorings (e.g. only add if description is there).

will it also work for attribute and type descriptions?

will it also work for attribute and type descriptions?
No. Unfortunately only methods. We can open a new issue for this.

did a bit of testing, have one class with one method, however nothing is returned from SEO_CLASS_TYPEINFO_GET, which SAP version are you testing on? Perhaps it has something to do with ADT?
I am also afraid that SEO_METHOD_MODIFY will regenerate the ABAP class source, but have not been able to test it.

Did a quick and dirty solution, take a look at https://github.com/larshp/abapGit/pull/480
Comments for solution in PR 480:
a: bad since it updates the table directly
b: works with method + types + attributes descriptions
c: In the future I think the descriptions are going away, it will move towards ABAP Doc. So it might be okay not to have a perfect solution for these descriptions
d: This will not interfere with any ADT logic

a: Yes. I was trying to use APIs instead of direct SQL statements
b: That is better
c: indeed
d: I don't think it has to be with ADT. I still need to check the system version.
e: Yes
So, should we cancel this PR and merge #480  instead?

#480 instead: I think so. I would prefer the APIs but it looks like a lot of work, and it is only descriptions, which ABAP is moving away from
@sbcgua do you have an opinion on this?

Not anything certain. In term of implementation - I don't know those internals well. In terms of descriptions in general: yeah, it feels incomplete without them, but also those texts are not exposed to users so doesn't seems really critical to me. API vs INSERT: nothing to add, indeed for texts maybe this is OK to insert ... similar code for MSAG exists for example ... of course API would be better if works.

@EduardoCopat I think we should go ahead and merge #480, is this okay with you?

@sbcgua thanks for the inputs
@larshp absolutely. No problems at all.


FR: Last Hydrogen Output Area view type selected to be remembered across sessions

Hi, it'd be great if Atom could remember and recall last hydrogen output area view type selection. Currently, it always defaults to 1st one. Thanks.

Not a solution, but maybe people just prefer the second option as default.
This can be achieved by changing a single line of code in the following file:

...\packages\Hydrogen\lib\components\output-area.js

Search for the showHistory variable in the OutputArea class and change it to false:
class OutputArea extends React.Component<{ store: store }> {
  @observable
  showHistory: boolean = false;

Worked for me at least.


Getting current road

Hey !
Is there a way to get the Road object corresponding to the road a car is on ?? (it's in order to get the bouding box of this road and then make the car follow the path "manually" with a script that control the steering)

Hi!
Did you already take a look at the road_definition.py provided in the examples?

Ok, so as it turns out there is also a get_roads function in the BeamNGpy class you could use. The function returns a list of tuples that make up the road network. Each tuple consists of three points that make up the left-most point middle point and right-most point of the road.
I'm going to write a jupyter notebook to better visualize how the road network can be used and to give an example on how to use the return values of get_roads.

You can find the example in the dev branch. Hope that helps.


link to our github project in the about_page

i mean it is open source, why not add a link to it? :)


Flower_Classification_with_TFLite_Model_Maker: missing labels.txt

The Tutorial and the Notebook state that I need a labels.txt file that should be generated along with the model.tflite but is not:

After this simple 4 steps, we can now download the model and label files

https://github.com/tensorflow/examples/blob/master/lite/codelabs/flower_classification/ml/Flower_Classification_with_TFLite_Model_Maker.ipynb

Copy the TensorFlow Lite model model.tflite and label.txt that you trained earlier to assets

https://codelabs.developers.google.com/codelabs/recognize-flowers-with-tensorflow-on-android
I tried
model.export(export_dir='/content/export/tflite_quant', label_filename='labels.txt', tflite_filename='model_quant.tflite', quantization_config=config)
and it says INFO:tensorflow:Label file is inside the TFLite model with metadata. in the log.
So is this a new feature? Is the labels.txt no longer required? What changes need to be made?
I manually created the labels.txt for the app and it worked, but I am not sure if it works without.

Also
lite/codelabs/flower_classification/start/app/src/main/assets/
changed to
lite/codelabs/flower_classification/android/start/app/src/main/assets
in https://codelabs.developers.google.com/codelabs/recognize-flowers-with-tensorflow-on-android/#4

Perhaps this can help https://github.com/tensorflow/examples/tree/master/lite/examples/image_classification/android/models/src/main/assets

Perhaps this can help with the codelab approach https://github.com/tensorflow/examples/blob/master/lite/codelabs/flower_classification/android/finish/app/src/main/assets/labels.txt
I see there are multiple examples for implementing flowers recognition problem such as using codelab or ModelMaker approach.

@ymodak I know it is not difficult to write a labels.txt . I just would like to have a clarification if TF no longer creates this file when saving a tflite model. In this case the tutorial should be adjusted.

Thanks for the report I see it fails to generate labels file along with the tflite file.

labels are embedded in tflite metadata.
use model.export(export_dir='.', with_metadata=False) instead of model.export(export_dir='.') to get labels.txt
Check the source code for why, https://github.com/tensorflow/examples/blob/master/tensorflow_examples/lite/model_maker/core/task/custom_model.py#L130-L177

@freedomtan Thanks, in this case, the Colab notebook should be adjusted accordingly.

@natowi I think original the Jupyter notebook was correct. The behavior or model maker changed.

@freedomtan Thanks


(opinion) UDL usage should be simplified

This is purely an opinion, so take with a grain of salt :)
I find the English-like numeral suffixes to be a little too "cute" and not necessarily helpful to code readability at scale. Especially when considering that, while software is often written in English, is is often written by non-native English speakers. Getting fancy with English numerals with UDL suffixes like _nd vs _rd seems like a recipe for confusion. It probably also increases the risk of unnecessary name clashes with other libraries when a particular function wants to using namespace UDLs from several different places.
I'd recommend dropping all the English-y UDLs and just sticking to _tag.
replace:
template <char... D> constexpr index_t<D...> operator""_tag() { return {}; }
template <char... D> constexpr index_t<D...> operator""_st() { return {}; }
template <char... D> constexpr index_t<D...> operator""_nd() { return {}; }
template <char... D> constexpr index_t<D...> operator""_rd() { return {}; }
template <char... D> constexpr index_t<D...> operator""_th() { return {}; }

with just:
template <char... D> constexpr index_t<D...> operator""_tag() { return {}; }

(Separately, I'm not sure the UDLs are all that helpful anyway; C++ made UDLs painful to use due to the need to always using namespace blah::literals; to access them; I'd generally just fallback to using tag<N> instead of N_tag anyway; it's one character longer, but cuts out an entire using statement.)

Hi Sean,
I took this change into consideration, and I think you're right! I removed the other literals, so that only the _tag literal remains. I like having the literals because you'd either have to do tag<0>() or use tag_v<0>, both of which are uglier than just writing 0_tag.
An example is seen here in this code in index-test-2.cpp
bool is_good(int a, int b, std::string c) {
    return a == 1 && b == 2 && c == "Hello, world!";
}

int main() {
    using namespace tuplet::literals;
    auto tup = tuplet::tuple {1, 2, std::string("Hello, world!")};

    static_assert(
        std::is_same_v<std::tuple_element_t<0, decltype(tup)>, int>,
        "Expected int& at element 0");
    static_assert(
        std::is_same_v<std::tuple_element_t<1, decltype(tup)>, int>,
        "Expected int& at element 1");
    static_assert(
        std::is_same_v<std::tuple_element_t<2, decltype(tup)>, std::string>,
        "Expected std::string& at element 2");

    auto& a = tup[0_tag];
    auto& b = tup[1_tag];
    auto& c = tup[2_tag];
    return !is_good(a, b, c);
}


io-homecontroll support

Describe you new feature you'd like
@rstrouse  Thanks for your amazing work on this project.
I wanted to get my new blinds with RTS and us this project. When I reached out to my local installers they told me that nowadays they only install IO motors. They installed last RTS motors years ago and they are not able to order RTS motors. Looks like io-homecontroll is inevitable.
I found this article that started reverse engineering the protocol. Looks like iown-homecontrol started building on top of that article and they are able to control the blinds.
Is there a way to add support for IO to this project ?

Unfortunately, not from me.  I do not have any IO home control motors and will never buy them.  I have no desire to install any product that has functionality reliant on external servers with no clear reason for it to be required.
While your installers are telling you this I see folks with new installs all the time on RTS.  However inevitable I don't have the desire or bandwidth to spend time on IO.  That will be for another Somfy user to figure out how to get the 3rd party booger hooks out of their blinds.

I have no desire to install any product that has functionality reliant on external servers with no clear reason for it to be required.

The same is true for me. I found out that IO should only be newer version of RTS with 2 way comunication and it is still just radio protocol. There is no need to use any of their hubs and connect IO motors to cloud.
Looks like the iown-homecontrol team was able to successfully reverse engineer the IO protocol and are able to control the blinds with ESP32.

@Velocet should be able to help you out there.  He gives some advice here on RTS protocol differences and has a grasp on IO protocols.
Just wait!  Cloud infrastructure and bandwidth isn't getting cheaper, these costs are growing exponentially.   When those costs start eating into their operational budgets it has to be offset with Annual Recurring Revenue (ARR) under subscription.  This has become so prevalent that there is a market for analyzing personal expenses to identify subscriptions that you may have forgotten about so you can churn out.
It is one thing if there are clear subscription benefits but it gets really insidious when service ends and features that you originally purchased go away or the product just stops working.  For me those products will remain on the shelf.  My box of retired hubs and hobbled equipment is full at this point.

I am going to close this but if anyone wants to strap this on feel free to use anything here.


refactor(dap)!: split dap and dap_ui integrations

This Pull Request makes 3 changes to the nvim-dap integration:

Splits the dap and dap_ui integrations. Rationale: there's no need for both plugins to share a integration. In fact, that's the only integration that contains multiple plugins.
Adds a missing highlight group to the nvim-dap integration
Removes a misguided comment about enabling the nvim-dap integration

I'm not sure why, but the integrations are not enabled by default. Did I forget something?

Need to add backwards compatibility because we're no longer 0ver

dap = { enabled = true, dap_ui = true }

should be auto converted to
dap = true, dap_ui = true

Something like this would work (Please verify:
if O.integrations.dap.enabled and O.integrations.dap.dap_ui then
  O.integrations.dap_ui = O.integrations.dap.dap_ui
end

Integrations are enabled by default if they're known to be well tested.

Failed on require("catppuccin").setup { integrations = { dap = true } }
Failed to run `config` for catppuccin

...chilly/code/git/catppuccin/lua/catppuccin/lib/mapper.lua:30: attempt to index field
'dap' (a boolean value)


Don't stop a full batch of destinations when one isn't found, log failed destination instead

see https://github.com/edwlook/node-google-distance/pull/13

Oh right, could you also add the destination that failed in your result element please ? Pretty useful to be able to identify which coordinate/placeId need a check


Handling SIGINT and sequential deployments

Environment
Device and OS: M2 Max, OSX Sonoma 14.4.1
App version: v0.33.0
Kubernetes distro being used: k3d (uds deploy k3d-core-slim-dev:0.21.1)
Other:
Steps to reproduce

zarf package deploy ... My app has an error that fails deployment "0 out of 1 expected pods are ready"
SIGINT (ctrl+c) to stop the zarf package deploy
(make updates, rebuild zarf package) and run zarf package deploy

Expected result
The interruption is handled and future zarf package deploy function.
Actual Result
My cluster has a helm CR with a status of pending-install Which prevents sequential zarf package deploy-ments of my package (once I've fixed the error)
Sequential deploys fail/retry with WARNING  Retrying (1/3) in 5s: unable to complete the helm chart install/upgrade: another operation                                                               (install/upgrade/rollback) is in progress
Visual Proof (screenshots, videos, text, etc)

Severity/Priority
Additional Context
workaround is to manually remove the helm CR and clean up the failed deployment.

This should be covered by #2505. It is good however to have a specific example of why this is needed.

@andrewg-xyz I am not sure there is a way Zarf can reliably guarantee cleanup of resources when a deployment is canceled with a SIGINT. Zarf uses the Helm SDK to cleanup resources on a best effort basis for failed upgrades and rollbacks, but it is risky for Zarf to start removing resources from the cluster outside of the Helm SDK on a failed deployment in my opinion.
I would recommend running zarf package remove to cleanup the failed deployment before attempting re-deployment

Thanks - I'm good to close with the context. I'll keep in mind as I interact with similar situations.


ValueError: Unknown primitive: mlprimitives.custom.timeseries_preprocessing.time_segments_aggregate

Orion version:latest version
Python version:3.7
Operating System:Google Colab

Description
I run the tutorial Tulog.ipynb on Colab, but an error occurs when running pipeline='tadgan.json'.

hi @twotigeres! Thanks for using Orion!
After you finish installing Orion via pip install orion-ml, please restart the runtime. From the top bar, click runtime -> restart runtime.
Let me know if you have any further questions!

hi @twotigeres! Thanks for using Orion!
After you finish installing Orion via pip install orion-ml, please restart the runtime. From the top bar, click runtime -> restart runtime.
Let me know if you have any further questions!

This worked for me, thanx


Context Canceled for mongo image in kitematic

Expected behavior
Create a mongo container from the docker hub in kitematic using the mongo image
Actual behavior
Error message saying context cancelled
Information about the Issue
Running the latest version of docker with the latest version of kitematic
Steps to reproduce the behavior

Open kitematic
Find Mongo image
Click "create"

I'm also having this issue, any update?

This problem has been solved by kitematic v0.17.7


spirv-opt: allow debug printf instrumentation for some mixed shader stage cases

This change modifies the behavior of the debug printf instrumentation code of spirv-opt to now allow for single modules containing mixed shader stages so long as entry points across different shader stage types do not call common functions.
The motivation for this PR is to allow for ray tracing pipeline scenarios where individual programs (raygens, closest hits, anyhits, intersections, miss programs) all built into one module can still access the debug printf functionality in SPIRV.
fixes #4892

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@greg-lunarg I'm currently using this little hlsl -> SPIRV example to test:
void commonFunction() {
  printf("Common function!\n");
}

[shader("raygeneration")]
void RayGen() {
    commonFunction();
}

[shader("miss")]
void MissProg(inout int tmp) {
    commonFunction();
}

That generates the following (unoptimized, un-inlined) SPV:
deviceCode_unoptimized_as.spv.txt
I was able to repro the issue by running that spv through spirv-opt using the "--inst-debug-printf" pass.
With this change, I now report a slightly different error with "--inst-debug-printf", which now detects that "void commonFunction" as being an issue. But then when "--inline-entry-points-exhaustive" is used before "--inst-debug-printf", we now go ahead and instrument the shader like before.

For some reason I'm getting an Assertion failed: def && "Definition is not registered."
@greg-lunarg any ideas why that might be?
I seem get this "definition is not registered" assertion on master without any of my changes though, whenever --inst-debug-printf is used--regardless of if I have mixed shader stages or not.

Please file the above problem as a separate issue. If you can show the problem without --inst-debug-printf, that would be best. Submit the minimal case that recreates the problem.

@greg-lunarg I can confirm that this is an issue on the master branch before my changes.
Simply use the these flags on the .spv file here deviceCode_unoptimized_as.spv.txt:
--inline-entry-points-exhaustive
--inst-debug-printf
Since this is a separate issue from my changes in the PR here, I'd still advocate to accept this PR if possible.

@greg-lunarg Let me know when you think this looks good.  I would like a test to be added.

This will need to be rebased for the tests to pass.

Should be rebased now.

@s-perron Sorry, I had a shadowed variable which was throwing an error on linux builds. should be fixed now.

@s-perron Ok, most checks are passing now, but I missed that I was failing a formatting check. Just pushed some changes to correct for formatting. Can we run tests again?

@s-perron I guess I missed a couple of spaces... More formatting changes pushed... Can we run tests again?

Nice! Tests seem to be passing now. :)
@greg-lunarg any next steps you recommend I take?

This seems to have gone stale. I will close it. Open a new PR when you have added the appropriate tests.


Problems with re intent with file upload.

Hi to all. First, here's my configuration with plupload:
runtimes: 'html5,flash,silverlight,html4',
url: "{{ oneup_uploader_endpoint('gallery') }}",
views: {
list: true,
thumbs: false,
active: 'list'
},
multipart_params : { "token" : token},
rename: true,
required_features: true,
unique_names: true,
max_retries: 5,
sortable: true,
buttons: {
browse: true,
start: true,
stop: false
},
filters: {
mime_types: [
{title: "Archivos de Imagen", extensions: "jpg,jpeg,png"}
],
prevent_duplicates: true
},
max_file_size: "25mb",
// Flash settings
flash_swf_url: 'http://www.plupload.com/examples/plupload/js/Moxie.swf',
// Silverlight settings
silverlight_xap_url: 'http://www.plupload.com/examples/plupload/js/Moxie.xap',
With the bundle, just :
oneup_uploader:
mappings:
gallery:
frontend: plupload
My problem is that sometimes the files are not being uploaded and the result of the "post" its a 200 code. Like you see, I have the max_retries option in 5. I have no idea where the problem is.
Regards,
Agustin

I'm sorry, but I have no idea given the information you provided. If the problem persists, can you create a new bare-bone Symfony project, and re-create the error, so I can take a look?

Hi there. Sorry for the late response. The problem was with APACHE and the max_input_time variable!
Thanks!
Agustin


Ignore: Merge 'PhaseEnable' changes in from the RPi master

Resolved conflicts, good to go.

Sorry - GitHub blunders!

Codecov Report

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

@@          Coverage Diff           @@
##           master    #620   +/-   ##
======================================
  Coverage    85.6%   85.6%
======================================
  Files          36      36
  Lines        7250    7250
======================================
  Hits         6206    6206
  Misses       1044    1044

Impacted Files
Coverage Δ

gpiozerocli/pinout.py
46.34% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8d93860...20cf2cf. Read the comment docs.


Trailing comma goes at the end of inline comment

I use include_trailing_comma so that isort adds a trailing comma at the end of from imports.
It usually works perfectly except when I have inline comments.
For instance, when applying isort to this import:
from project.moduleA.moduleB.moduleC.moduleD import (
    MyVeryUsefulClassA,  # some inline comment
)

it becomes:
from project.moduleA.moduleB.moduleC.moduleD import (
    MyVeryUsefulClassA  # some inline comment,
)

The trailing comma travels from the end of the imported class to the end of the inline comment.
Moreover, it makes it incompatible with black.
Thank you!

Below is the isort v4.3.21 configuration I use for black compatibility.
[isort]
multi_line_output=3
include_trailing_comma=True
force_grid_wrap=0
use_parentheses=True
line_length=88

This is fixed in the just released 5.0.0 version of isort: https://timothycrosley.github.io/isort/docs/major_releases/introducing_isort_5/
Thank you for reporting!


Crashed after multi-image inference.

FastPathology consistently crashed after multi-image inference. All the output files are saved, but it crashes right after this with no visible error print:
WARNING [48032] Test data was NOT found at C:/ProgramData/FAST/data/. Download the test data by running ./downloadTestData or fast.downloadTestDataIfNotExists() in Python.
OK
parsing
OK
Done
Running pipeline [=======================================================================================================================================================================================>  ] 98% | ETA 0.0 minsSaving Tensor data to C:/Users/henri/fastpathology/projects/2024-08-25-153819//results/.../.../heatmap
Writing timestamping..2024-08-25-163905
canceled..
done waiting
stopping pipeline
done
removing renderers..
done

When does it crash, after all images have been processed, or during?

@smistad, @SahPet Didn't we have this issue on Windows way back when the filename contained a space or misc characters (æ,ø,å)?
I think FP fails to save the final result in the project, if I recall correctly. I still think that is an issue in FP, but probably a year since I last tested this.


HIVE-27534: Fix the potential database connection leak in the HMS exchange partitions operation

In the HMSHandler.exchange_partitions method, it executes RawStore.openTransaction and performs some checks. If these checks fail, an exception is thrown without executing commitTransaction or rollbackTransaction. This can result in a leakage of the underlying database connection, as it is not returned to the thread pool. Consequently, all threads may be blocked at the point of obtaining a connection from the connection pool.
We should place all operations that come after RawStore.openTransaction inside a try-finally block.

Thanks for the patch @HonestManXin, there are few qtest failures can you please check: http://ci.hive.apache.org/blue/organizations/jenkins/hive-precommit/detail/PR-4518/2/tests


Added a new composite node: simple_parallel

Description
Added a composite node called simple parallel. It does the same thing like the one in Unreal Engine. This node allows you to execute multiple things at the same time, e.g moving while shooting. Nested simple parallel is also supported but not suggested
Addressed issues
N/A
Screen shots
Here's debug view of my own behavior tree in Beehave debugger:

@DarkAngelZT thank you for raising this - a few things on this:

would you mind attaching the UE docs on this? I am curious about the design decision of limiting it to exactly two child nodes
any new node should be covered by unit tests to ensure it works as expected. Please read the contribution guidelines for more information
the current v2.x version of beehave does not correctly test the debugger (since it is hard to test) - could you attach screenshots/gif of how the parallel node looks like inside the Beehave debugger?
We also would require documentation on this new composite node. So far, we only support selectors and sequence nodes. Perhaps we can introduce a new one for 'Parallel'

Generally an okay pull request - one more thing: "SimpleParallel" suggests that there could be different types of parallel, however, the architecture of Beehave V2 does not allow for other types (e.g. it assumes that always one node at a time is in RUNNING state). Could we perhaps just call it Parallel?

Thanks for reply. Here is the official explaination of simple parallel node on epic website:
https://dev.epicgames.com/documentation/en-us/unreal-engine/behavior-tree-in-unreal-engine---overview#concurrentbehaviors
So basically this node has concurrency features, but it still act like a regular composite node. Which makes it different from real parallel node. As you can see its state is only controlled by first child. So I think if we simply call it parallel woud cause some misunderstanding of its feature.
I will add icon and unit test later. This node is currently running in my project and I've wrote a python version for that few years ago for another engine, ant it working smoothly till now in some comercial projects, quality ensured ;)

@bitbrain  Unit test and docs are done, debugger screen shot has updated above. Please review

@DarkAngelZT the icon does not seem to display correctly in the debugger?

That's the same code in my current project, which doesn't have icon set. here's new one:


[BUG]: raw SQL fields selected from subqueries should not require manually-defined aliases

What version of drizzle-orm are you using?
0.32.1
What version of drizzle-kit are you using?
No response
Describe the Bug
GENERIC CASE: I ran into this while doing a simple subquery, generally of the form db.select().from(subquery).orderBy(something), where the subquery includes raw SQL fields.
SPECIFIC REASON: I'm using Postgres, and wanted to use a DISTINCT ON query, but have those results sorted by a different column from the distinct expression.  The most straightforward way to do this is as above, just using a subquery to get the final sort order.  (This is not an uncommon pattern for advanced querying using DISTINCT ON; see this StackOverflow question/discussion.)
I am effectively selecting the most recently created row belonging to each login session (most recent activity record for each session), and then ordering those based on when they were created (when the activity occurred).  (It's actually a bit more complex than that in practice, this is a slightly simplified example.)
The key being that I am not doing any complex selection/filtering on the outer query, it's just a re-ordering of the existing rows from the subquery.  See simplified example code below.
const distinctQuery = trx
  .selectDistinctOn([activitySchema.sessionId], {
    createdAt: activitySchema.createdAt,
    customerName: sql<string>`${activitySchema.user} ->> 'name'`,
    customerPhone: sql<string>`${activitySchema.user} ->> 'phone'`,
    // ... many other fields, both existing columns and derived values like above ...
  })
  .from(activitySchema)
  .where(and(whereClauses))
  .orderBy(activitySchema.sessionId, desc(activitySchema.createdAt))
  .as('distinct_on_query');
const results = await trx
  .select()
  .from(distinctQuery)
  .orderBy(desc(distinctQuery.createdAt));

This results in the following error:

Error: You tried to reference "customerName" field from a subquery, which is a raw SQL field, but it doesn't have an alias declared. Please add an alias to the field using ".as('alias')" method.

Expected behavior
I would expect a default alias to be created for such columns, using the names provided in the subquery's .select() object (in the example above, "customerName" and "customerPhone").
Environment & setup
No response

+1

Did you find a way to make this work? how do you add the alias?

I had the same problem when I used .select({ createdAt: max(table.createdAt) }).
The fix was adding .as('alias') like this:
.select({
  createdAt: max(table.createdAt).as("createdAt"),
})


Processing offer and initializing the worker has not finished in 10.0 seconds

Hi,
I got this error message 10 seconds after pressing the start button.
The browser and streamlit app are not running on the same machine but they all see each other.
Any idea?

streamlit-webrtc shows this message to warn possible performance flaw, but it does not indicate the cause.
If you don't have any performance problem, just ignore it.


badge overlays show on home page

the badge overlays are present on the home page for moderators and owners

Created a new template specifically for usage on "home.html". They shouldn't have overlays.


Output is pixelated

Was capturing https://github.com/vrm-c/UniVRM/tree/master/Tests/Models/Alicia_vrm-0.51 and I could not get the output to look detailed and smooth.
Rename vrm to glb.

You can increase resolution of baked image in TestScene/ViewportBaking/Camera by modyfiing parameter "Image Square Size" to something like 16k. But importing big textures in Godot is very slow right now. Right now one frame of 4k image with 16 frames is roughly equal to 256x256pixels. This is not enough for close viewing of impostor.

So using 16k textures will give me 1024X024 pixels?

Yes, You can also decrease a "Grid Size" parameter but it could not be enough for proper illusion. The formula is image_square_size/grid_size.

I also looked into rotating the object to fit the square area, but I didn't have bandwidth for it.

An inspection of the png shows that 1) it is missing dilation 2) The content area is very small.

Is it possible to use more sets of textures?

I think it is, but I am not sure if it's really a good idea. Those textures are few megs each so it won't be a great optimization. I tried to bake this model and i found out that import texture settings were kind strange. Here is the result with my settings:
https://streamable.com/t07jlk
I think its not bad for 256x256 frame size. Also please check if all textures are correct in impostor shader, it seems like normal map was a wrong file.

And here is 8k texture version with 512x512 frame size:
https://streamable.com/8pyy7v
Looks really nice but texture weights 14 mb. I think at this point its probably better to use some LOD version of real model.

Someone told me we should using texture arrays. https://docs.godotengine.org/en/stable/classes/class_texturearray.html

I didn't know that we could use texture arrays in shaders. I will look into this tomorrow. I am not really sure what improvements to expect with this change, probably Shader code will be simpler?
So we have mesh decimation and billboarding. The biggest problem is a lack of some kind of logic to automatically swap meshes by view distance. I think it would be simple with single MeshInstance, but not with MultimeshInstance. It would be nice if Godot have something like that integrated into its source code.

The main thing is mipmaps are allowed.

It seems that sometimes Texture Array is not imported properly after baking. Other than that  I think its working properly. On the other hand, I'm worried about two other issues. First of all, the constant GL_MAX_ARRAY_TEXTURE_LAYERS limits us to the maximum of 16x16 frames in opengles 3. The second problem is the lack of any support for Array Textures in opengles 2. I am not sure if we should go this way.

https://opengles.gpuinfo.org/displaycapability.php?name=GL_MAX_ARRAY_TEXTURE_LAYERS&esversion=3 Interesting numbers on the max.
I don't know how long opengles 2 will exist in Godot Engine.
https://github.com/godotengine/godot-proposals/issues/877

Its in a texture_array branch for now. According to this https://godotengine.org/article/abandoning-gles3-vulkan-and-gles2 GLES 2.0 will stay in Godot 4.0.

You can increase resolution of baked image in TestScene/ViewportBaking/Camera by modyfiing parameter "Image Square Size" to something like 16k. But importing big textures in Godot is very slow right now.

Did you disable Import Etc2 in the Project Settings?

No, I have not disabled this option. However, I don't see many alternatives. Same as in this article, textures use all 4 channels (RGBA) to save some memory. From what I understand, Import Etc does not support the alpha channel, and the other options are not supported by Android. In addition, it appears that Import Etc2 will not work on GLES 2.0, so the current implementation will not work in this case.  I should probably only use RGB channels or distinguish between GLES 2.0 and GLES 3.0 versions.

After a moment of thought, I disabled Import Etc2 for Baker. If someone needs the Android version, they will simply import the generated assets to another project.

The main advantage of using texture array is the ability to avoid texture sizes of atlasing. Even if each image in the array has a max size of 2k, that is a lot more than the limits of 256.

Are you able to update the texture array branch? I'm having a hard time upgrading the code. Wanted to test it.

Yes I'll do it. I'm just waiting for some minor corrections to the pull request because it introduces very big changes that would require re-modifying the TextureArray branch again.

Ok, I added TextureArray on master as another shader option. TextureArrays baker uses ugly workaround for importing as correct resource, so I cannot guarantee that it will work on all operating systems.

I'm unable to open the new baker ui on master https://github.com/wojtekpil/Godot-Octahedral-Impostors/commit/7e96db2ec35a9b6e8e53fe73a0ada5ded511a025

Yes please, read README file. It now works as a plugin. Just select any MeshInstance and click this button:

So I have to merge my meshes to one arraymesh?

I can't test it, but I'll open a new bug for the other issue.

I tried the test and it still seems like the imposter size is very pixelated. Is it possible to use multiple texture atlases? I assume the limit is one texture for each frame of 16x16, this can then be combined into the texture array.

For now I don't have plans for textures atlases. It's too expensive in VRAM memory. Right now 4k version costs about 60 mb of VRAM. It's a lot and for an LOD system it's probably too much already. We can go as low as 7.5 mb with some optimization like 2k textures color/alpha and 1k textures for everything else. But I see that your goal is to achieve better quality. In that case, I'd probably use some mesh optimizer right now.

I've already done the mesh optimizer / merging code. Wanted to develop this part of the code.
https://github.com/HungryProton/concept_graph/issues/44

Will close issue because many things have changed in repository and I need to re-evaluate the problem.


Enhancement in our motive section

Deliverables:

[ ] On hovering, the whole card content should be white
[ ] Make the icon size a bit smaller

Please assign this to me

Please assign this to me

text and icons are already white on hovering what else we have to change on hovering of card?

text and icons are already white on hovering what else we have to change on hovering of card?

@abhay313 Please pull the changes from the main branch.

@abhay313 Please pull the changes from the main branch.

Please check size of icons
size: 40px

size: 50px

size: 55px

size: 60px

Please check size of icons
size: 40px

size: 50px

size: 55px

size: 60px

@abhay313 can we please have a better icon for the third card. Rest looks good to go!

@abhay313 can we please have a better icon for the third card. Rest looks good to go!

@abhay313 40px is fine.
Give padding/margin on either side of the icon.
If you see the first-ever screenshot you pasted in this thread, there is equal padding/margin on left and right of the icon.

@abhay313 40px is fine.
Give padding/margin on either side of the icon.
If you see the first-ever screenshot you pasted in this thread, there is equal padding/margin on left and right of the icon.

Please check it once
size: 40px;
margin: 5px 10px 0 5px;

Please check it once
size: 40px;
margin: 5px 10px 0 5px;

do we need to change third icon ?

do we need to change third icon ?

do we need to change third icon ?

Yes.

do we need to change third icon ?

Yes.

Can you suggest anyone, please ?

Can you suggest anyone, please ?

https://fontawesome.com/icons/book-open?style=solid
https://fontawesome.com/icons/chalkboard-teacher?style=solid
https://fontawesome.com/icons/graduation-cap?style=solid
https://fontawesome.com/icons/book-reader?style=solid
anyone looks better?

https://fontawesome.com/icons/book-open?style=solid
https://fontawesome.com/icons/chalkboard-teacher?style=solid
https://fontawesome.com/icons/graduation-cap?style=solid
https://fontawesome.com/icons/book-reader?style=solid
anyone looks better?

Hey @abhay313 let's use this icon

Hey @abhay313 let's use this icon


Add DSC Rule: Resource module contains Examples folder, which contains sample configurations

There should be sample configurations present for every resource.

Charles. please send us a pull request and we will code review and integrate the change

Rule added and merged.


bazel build //modules/localization/rtk:rtk_localization_test出错

System information

**OS Platform and Distribution: Linux Ubuntu 23.04
**Apollo installed from: 软件包
**Apollo version: 8.0

Steps to reproduce the issue:
When I run the command:
bazel build //modules/localization/rtk:rtk_localization_test
and I encounter such an error:

How can I solve it?

System information

**OS Platform and Distribution: Linux Ubuntu 23.04
**Apollo installed from: 软件包
**Apollo version: 8.0

Steps to reproduce the issue:
When I run the command: bazel build //modules/localization/rtk:rtk_localization_test and I encounter such an error:  How can I solve it?

I think there is an error with the command, but when I use this command:
bazel test //modules/localization/rtk:rtk_localization_test
the same error occurred.

Because the bazel tool required for compilation has not been downloaded, please check whether the network in your container can stably connect to the external network. If so, it will be automatically downloaded after you enter the compilation command.

Because the bazel tool required for compilation has not been downloaded, please check whether the network in your container can stably connect to the external network. If so, it will be automatically downloaded after you enter the compilation command.
I'm testing the network using the ping command inside a docker container. The container is connected to the outside network, and I use bazel --version to check the bazel version is 5.3.1. But this error still occurs.

This may be caused by a mismatch between the current version of bazel and the specified toolchain, the easiest way is to remove the specified bazel jdk in two files：
vim /apollo_workspace/WORKSPACE
vim /opt/apollo/neo/packages/buildtool-dev/latest/data/templates/workspace.in // if you are using apollo 8.0
vim /opt/apollo/neo/packages/buildtool/latest/data/templates/workspace.in // if you are using apollo 9.0

remove the following content:
http_archive(
    name = "remotejdk11_linux",
    sha256 = "360626cc19063bc411bfed2914301b908a8f77a7919aaea007a977fa8fb3cde1",
    urls = [
        "https://apollo-pkg-beta.bj.bcebos.com/bazel/zulu11.37.17-ca-jdk11.0.6-linux_x64.tar.gz",
    ],
)

and try bazel test or buildtool test -p modules/localization, the problem should be fixed


league/flysystem-aws-s3-v3  需要 league/flysystem 2.0

league/flysystem-aws-s3-v3  需要 league/flysystem 2.0
而这个安装包里面版本是1.1
Problem 1
- Conclusion: don't install league/flysystem-aws-s3-v3 2.0.0
- Conclusion: don't install league/flysystem-aws-s3-v3 2.0.0-RC1
- Conclusion: don't install league/flysystem-aws-s3-v3 2.0.0-beta.3
- Conclusion: don't install league/flysystem-aws-s3-v3 2.0.0-beta.2
- league/flysystem-aws-s3-v3 2.0.0-alpha.4 requires league/flysystem 2.0.0-alpha.3 -> satisfiable by league/flysystem[2.0.0-alpha.3].
- league/flysystem-aws-s3-v3 2.0.0-beta.1 requires league/flysystem 2.0.0-alpha.3 -> satisfiable by league/flysystem[2.0.0-alpha.3].
- Conclusion: don't install league/flysystem 2.0.0-alpha.3
- Installation request for hyperf/filesystem (locked at v2.0.11, required as ^2.0) -> satisfiable by hyperf/filesystem[v2.0.11].
- league/flysystem-aws-s3-v3 2.0.0-alpha.1 requires league/flysystem 2.0.0-alpha.1 -> satisfiable by league/flysystem[2.0.0-alpha.1].
- league/flysystem-aws-s3-v3 2.0.0-alpha.2 requires league/flysystem 2.0.0-alpha.1 -> satisfiable by league/flysystem[2.0.0-alpha.1].
- Conclusion: don't install league/flysystem 2.0.0-alpha.1
- Conclusion: don't install league/flysystem-aws-s3-v3 2.x-dev|remove league/flysystem 1.1.3|install league/flysystem 2.0.0-alpha.1|install league/flysystem 2.0.0-alpha.3
- Conclusion: don't install league/flysystem-aws-s3-v3 2.x-dev|don't install league/flysystem 1.1.3|install league/flysystem 2.0.0-alpha.1|install league/flysystem 2.0.0-alpha.3
- Installation request for league/flysystem-aws-s3-v3 ^2.0 -> satisfiable by league/flysystem-aws-s3-v3[2.0.0, 2.0.0-RC1, 2.0.0-alpha.1, 2.0.0-alpha.2, 2.0.0-alpha.4, 2.0.0-beta.1, 2.0.0-beta.2, 2.0.0-beta.3, 2.x-dev].
- league/flysystem-aws-s3-v3 2.x-dev requires league/flysystem ^2.0.0 -> satisfiable by league/flysystem[2.0.0, 2.0.0-RC1, 2.0.0-alpha.1, 2.0.0-alpha.2, 2.0.0-alpha.3, 2.0.0-alpha.4, 2.0.0-beta.1, 2.0.0-beta.2, 2.0.0-beta.3, 2.x-dev].
- Can only install one of: league/flysystem[2.0.0, 1.1.3].
- Can only install one of: league/flysystem[2.0.0-RC1, 1.1.3].
- Can only install one of: league/flysystem[2.0.0-alpha.2, 1.1.3].
- Can only install one of: league/flysystem[2.0.0-alpha.4, 1.1.3].
- Can only install one of: league/flysystem[2.0.0-beta.1, 1.1.3].
- Can only install one of: league/flysystem[2.0.0-beta.2, 1.1.3].
- Can only install one of: league/flysystem[2.0.0-beta.3, 1.1.3].
- Can only install one of: league/flysystem[2.x-dev, 1.1.3].
- Installation request for league/flysystem (locked at 1.1.3) -> satisfiable by league/flysystem[1.1.3].
目前是不是支持2.0一下？

2.1版本支持了


Add configurable data plane log level

Add Nginx error log level to NginxProxy.
Problem: Users would like to assign a log level for the data plane.
Solution: Add Nginx error log level to NginxProxy which allows users to set the error log level for Nginx.
Testing: Unit tests and manual testing.
Manual testing included

Validating the enum values in the new ErrorLevel field in the NginxProxy resource.
Deploying NGF, setting up the cafe-app, deploying an NginxProxy resource, and testing each of the possible log levels and checking that the nginx conf (the main.conf file and running nginx -T) in the nginx container was updated correctly.
Checked that if the error level was not set in the NginxProxy resource, the nginx conf would be correctly updated back to the default of info log level.
Tested that if a log level was set to something "beneath" notice such as the error level, after making changes which would result in notice log levels being outputted (such as deleting and re-applying cafe.yaml in the cafe example), no notice log levels are outputted.
nginx -T correctly shows the main.conf configuration file being included in the nginx configuration.
ps aux |grep nginx when inside the nginx container correctly showed nginx-debug processes.
Deployed the cafe example and deleted it a few times and NGF still worked properly with no error logs in either container.

Closes #901
Checklist
Before creating a PR, run through this checklist and mark each as complete.

[x] I have read the CONTRIBUTING doc
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have checked that all unit tests pass after adding my changes
[ ] I have updated necessary documentation
[x] I have rebased my branch onto main
[x] I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

Release notes
If this PR introduces a change that affects users and needs to be mentioned in the release notes,
please add a brief note that summarizes the change.

Opening this and closing #2561 in order for the NGINX Plus image to be built correctly with the repo's certs.

Needed to squash and rebase so the commit trail is gone, no major changes to actual functionality or code since previous review. Only big changes has been adding the Helm schema to the chart's values.yaml and documentation changes.


question: --watch, services best practices in monorepos

Hey 👋! Have a few questions regarding when to pass the --watch flag into scripts. Specifically, I'm thinking of a monorepo setup where I'd like to have multiple packages watched. One package for a client scripts build, one for a styles build, one for a server build, etc etc. Something like this:
package.json wireit
"dev": {
	"command": "node --inspect \"./dist/server.js\"",
	"service": {
		"readyWhen": {
			"lineMatches": "Debugger listening on*."
		}
	},
	"dependencies": [
		"build-tsc",
		"../scripts:dev",
		"../styles:dev",
 		"../someother:dev"
	]
},

Should the dependencies (like "../scripts:dev") include --watch flags in their respective package.json files? Should they be listed as services, since they are watchers and therefore long running tasks? Per reading the docs, it looks to me like the way to go is to pass --watch from the topmost task in the call chain, but that doesn't seem to be working as I'd expect it to.

If you run a script with the --watch flag, then all transitive dependencies will also run in watch mode. The entire wireit process is either in watch mode or not.
A script should be a service if it is a long-running process that doesn't end by itself, such as a web server. A build step such as a compiler or bundler should not be a service.
Could you provide more information about your configuration, and more details of what's not working? One quick suggestion, looking at the one example above, is that you'll want to make sure you've defined a files and output array for all of your scripts to maximize the benefits of wireit, and in particular to ensure that --watch mode only re-builds the things that really need to be rebuilt. If a script doesn't have its files declared, then Wireit can't know which files that script consumes, and will assume that any change will require that script to re-build.

Thank you for the speedy response and for validating my suspicions that I've done something incorrectly. I do have files and output declared in the respective package.json files.
The context is that I'm attempting to run a server (via a service), a styles watcher for client styles, a client side scripts watcher that depends on a localized strings package (also needs watching), and a few other various and sundry moving parts. The server (or dev script in the snippet) is essentially dependent on everything. There are lots of other deps, and too many cross package deps to list out. May even need browsersync for some reloading of too but hoping wireit allows me to avoid that somehow ...
I'm curious about "transitive" dependencies. Does this mean something different from all dependencies of all dependencies (for lack of a better way to put it)? In terms of the snippet above, does the dev task, need to list out dependencies that are be dependencies of, for example, the "../scripts:dev" script?

Thank you for the speedy response and for validating my suspicions that I've done something incorrectly. I do have files and output declared in the respective package.json files.
The context is that I'm attempting to run a server (via a service), a styles watcher for client styles, a client side scripts watcher that depends on a localized strings package (also needs watching), and a few other various and sundry moving parts.

These watchers -- are the scripts doing their own intrinsic watching, or are you letting wireit's --watch mode do that? In general, what will work best with wireit is to write all your commands as normal build commands (that do a single invocation, with no intrinsic watching), and then let wireit manage the watching for everything. There are tradeoffs here, and we have some ongoing discussions about how to better support the intrinsic watch modes that many tools have (because they can be faster by caching things in memory etc.) but the approach of letting wireit own all of the watching will give you the most consistent and reliable behavior.

The server (or dev script in the snippet) is essentially dependent on everything. There are lots of other deps, and too many cross package deps to list out. May even need browsersync for some reloading of too but hoping wireit allows me to avoid that somehow ... your classic modernization of a large project.

Sounds like a great use case for wireit in general!

I'm curious about "transitive" dependencies. Does this mean something different from all dependencies of all dependencies (for lack of a better way to put it)?

Nope, I think we're on the same page with the definition. The dependencies of a script, and the dependencies of those dependencies, and so on...

In terms of the snippet above, does the dev task, need to list out dependencies that are be dependencies of, for example, the "../scripts:dev" script?

Nope, it doesn't need to list them out. Wireit understands the transitive dependencies automatically. If you run npm run dev --watch then the input files for ../scripts:dev will be monitored, along with the input files of all of ../scripts:dev's dependencies, and so on

Excellent, happy to hear that one top level --watch flag is the way to go. I'll revisit my project and ensure I'm doing things that way you suggest. Appreciate clarifications!


Livewire Charts added

Hey @coding-sunshine. Although the project definitely seems very promising I feel this more belongs to the awesome livewire list: https://github.com/imliam/awesome-livewire
We're looking for projects specifically bridging the gap between the different parts of the TALL stack. Thanks.


Added missing crypto import (fixes #340)

Summary
Added missing "crypto" module import so that it won't crash with Deno because of undefined randomUUID function.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Add API BruteForceRangeSearch()

Signed-off-by: yudong.cai<EMAIL_ADDRESS>

/kind improvement

/hold

/unhold

/lgtm

question, why

bruteforce has to be a interface?
even range search, why do we have a specific interface


print \t and \n not interpreted correctly

The Asteroid print function ignores the \t and \n characters.
print("hello\nworld").

prints
hello\nworld

fixed.


Can i show error use TextInputLayout error attribute ?

Hi, great package.
thanks a lot for creating this package.
but I have some problems, can I show error use TextInputLayout error attribute without custom using setOnErrorValidatorListiner?
your default use the edit text error attribe it's not match with my design UI.
thanks

Hi MashudiSudonym,
You can use error listener like this
sheenValidator.setOnErrorValidatorListener { list ->
    list.forEach {
         when(it.id){
             R.id.txt_username -> til_username.error = "Oops!... looks like your username is not valid"
             R.id.txt_password -> til_password.error = "Hmm... password is required"
                }
            }
        }

With you are solution.
can I use the default error text instead of custom like that?


Connection to redis is mandatory with 2.1.8?

It seems that the connection to redis is mandatory with the v2.8.x branch at least.
Running a simple php app/console cache:clear -n --no-warmup  ends up with an error:
 [Predis\Connection\ConnectionException]
  Connection refused [tcp://localhost:6379]

config:
snc_redis:
    # configure predis as client
    clients:
        default:
            type: predis
            alias: default
            dsn: redis://localhost
            options:
                read_write_timeout: -1

Any way to workaround that behavior ?

This is actually quite a problem when working with CI/CD that don't really need to access data but just build the sources

I'm experiencing the same problem.

This issue was not happening with 2.1.7?

@B-Galati happens also with 2.1.7 version

I cannot reproduce it on a fresh install. Could you setup a reproducer please?

app/config/config.yml
 17   doctrine:

 36     orm:
 37         auto_generate_proxy_classes: '%kernel.debug%'
 38         entity_managers:
 39             default:
 40                 naming_strategy: doctrine.orm.naming_strategy.underscore
 41                 auto_mapping: true
 42                 second_level_cache:
 43                     enabled: true
 44                     region_cache_driver:
 45                         type: service
 46                         id: snc_second_level_cache
 47                     region_lifetime: 86400

271 snc_redis:
272     clients:
273         default:
274             type: predis
275             alias: default
276             dsn: 'redis://%cache_dsn%'
277             logging: '%kernel.debug%'
278         doctrine:
279             type: predis
280             alias: doctrine
281             dsn: 'redis://%cache_dsn%'
282             logging: '%kernel.debug%'

284     doctrine:
285         metadata_cache:
286             client: doctrine
287             entity_manager: default
288             namespace: 'dmc:'
289         result_cache:
290             client: doctrine
291             entity_manager: default
292             namespace: 'drc:'
293         query_cache:
294             client: doctrine
295             entity_manager: default
296             namespace: 'dqc:'
297         second_level_cache:
298             client: doctrine
299             entity_manager: default
300             namespace: 'dslc:'

composer.json
109         "symfony-scripts": [
110             "Incenteev\\ParameterHandler\\ScriptHandler::buildParameters",
111             "Sensio\\Bundle\\DistributionBundle\\Composer\\ScriptHandler::clearCache",
112             "Sensio\\Bundle\\DistributionBundle\\Composer\\ScriptHandler::installAssets",
113             "Sensio\\Bundle\\DistributionBundle\\Composer\\ScriptHandler::installRequirementsFile",
114             "@php bin/console assetic:dump"
115         ],
116         "post-install-cmd": [
117             "@symfony-scripts"
118         ],

It fails when executing both "Sensio\Bundle\DistributionBundle\Composer\ScriptHandler::clearCache" and "Sensio\Bundle\DistributionBundle\Composer\ScriptHandler::installAssets", trying to connect to the default Redis connection set on localhost.

@maxgio92 Does it work if you remove doctrine metadata_cache config?
This one should be configured with cache.system instead of redis for better performances.
Same goes for the query cache
Check https://github.com/symfony/recipes/blob/master/doctrine/doctrine-bundle/1.6/config/packages/prod/doctrine.yaml out for reference.

Hi @B-Galati ,
this way I will not using Redis?
I would want to use Redis for scaling up since the application is containerized.

Sorry I don't understand what you are saying @maxgio92.
Metadata cache are information about your entities that don't need to be stored in Redis. They just need to be compiled and stored once in var/cache. This is far more efficient than using Redis in this case.
Does it work with what I've pointed out?

I took some time to try what @B-Galati suggested (desactivate doctrine cache).
On a fresh install with the following parameters i can reproduce the error :

@fredleger Thx! could you provide the full stack trace please? (By adding -v to app/console)
Does it happen with cache:clear command now or it's only with assetic?
What do you mean exactly by fresh install? app/console does not look like a fresh install :sweat_smile:

hi @B-Galati. Sorry it wasn't that clear. By fresh i was making reference to a new bare project not cutting edge code obviously :-) I have to stick on 2.8 for porting legacy project. no choice.
It was only related to assetic:dump after disabling the  metadata caching.
Now, adding the -v lead me to the answer actually. The redis extension was configured to be the default session storage in the config.yml. Once set to null the error message goes away. That is enough for me.
I guess a little entry in the documentation could help for people reaching the case.
Thanxs for clearing that out.

@fredleger You're welcome, for the doc I would be glad to review a PR from you or anybody that contributes to this issue.
Cheers!

@fredleger Could you please provide your working config.yml?

@maxgio92 Does it work if you remove doctrine metadata_cache config?
This one should be configured with cache.system instead of redis for better performances.
Same goes for the query cache
Check https://github.com/symfony/recipes/blob/master/doctrine/doctrine-bundle/1.6/config/packages/prod/doctrine.yaml out for reference.

@B-Galati I agree with you that cache.system can be used instead of redis for performance reasons, but what if one wants to use redis for storingmetadata_cache(@maxgio92 use case)?
The bundle's documentation https://github.com/snc/SncRedisBundle/blob/master/Resources/doc/index.md#complete-configuration-example explicitly shows that this is feasable.
So if ones build fails if using redis for metadata_cache(Connection refused exception) this must mean that this bundle is not lazy loading redis. Your proposed solution to use cache.system is just a workaround.

@B-Galati I agree with you that cache.system can be used instead of redis for performance reasons, but what if one wants to use redis for storingmetadata_cache(@maxgio92 use case)?

Lazy loading could not do anything for that as doctrine metadata cache is static and built when cache:warmup is ran. Basically the cache warmup would ask for redis connection anyway to store cache data in it. I don't see any valid use case but perhaps there are some.

The bundle's documentation https://github.com/snc/SncRedisBundle/blob/master/Resources/doc/index.md#complete-configuration-example explicitly shows that this is feasable.

Indeed. But that's not a good idea IMHO. We should update this example.

So if ones build fails if using redis for metadata_cache(Connection refused exception) this must mean that this bundle is not lazy loading redis. Your proposed solution to use cache.system is just a workaround.

No it's not, see my 1st answer.


multiple security

Hello! How can i use multiple security with AND condition, not OR

Hello @mat-twg
I believe this issue it's outside the scope of this package.
This should be handled by the official swagger package which this package relies on, as the only thing nestjs-redoc does it's to pass the Swagger data object generated by nest into redoc.

I would not be so sure, the swagger module does not render any logical separation between authorizations...