[APL] H264: SPS header values set to zero, once iHD driver is loaded

There is issue in MediaSDK library for H264 stream.
If H264 stream has SPS header information with "bitstream_restriction_flag = 1" and  "max_dec_frame_buffering = 4", as H264 Spec it should use max_dec_frame_buffering, rather than using default dpzSize for frames calculation. Which is not happening currently.
Command Used:
./sample_decode h264 -hw -async 1 -vaapi -i ~/test_2880x960.h264 -n 10
Test Environment:
OS: Ubuntu 18.04
Hardware: APL
Latest MediaSDK Commit: 2e3dbfa7b30a1efa27e021f34dd6791f6e9447bc
Latest MediaDriver Commit: d7061b608ad4d55fccab00f8d80b852f20eab1d7 (20.1.pre)
Latest Libva Commit: ccf3b081ff3d2b0784cd66daf04bbc317277cf8b (2.8.0.pre1)

Debugging on MediaSDK side, found function call flow as below.
main (sample_decode.cpp) -->  CDecodingPipeline::Init (pipeline_decode.cpp) --> CDecodingPipeline::InitMfxParams (pipeline_decode.cpp) --> VideoDECODEH264::DecodeHeader (umc_h264_mfx_supplier.cpp) -->  CalculateDPBSize(umc_h264_task_supplier.h)
We get dpbSize = 16 at this point based on CodecProfile: 100, CodecLevel: 51
Later when  following function  "UMC::FillVideoParam" in "umc_h264_dec_mfx.cpp" invoked
par->mfx.MaxDecFrameBuffering =  seq->vui.bitstream_restriction_flag ? seq->vui.max_dec_frame_buffering : 0; <-- mfx.MaxDecFrameBuffering is assigned value 4 based on SPS Header inform from stream supplied (carplay_2880x960.h264)
(gdb) p par->mfx.MaxDecFrameBuffering
$6 = 4
Later on, CreateAllocator(pipeline_decode.cpp) is invoked calls --> CreateHWDevice(pipeline_decode.cpp) --> CreateVAAPIDevice (pipeline_decode.cpp) -->CVAAPIDeviceDRM(vaapi_device.cpp ) <-- This point (dri/iHD_drv_video.so) is loaded
libva info: VA-API version 1.8.0
libva info: User environment variable requested driver 'iHD'
libva info: Trying to open /usr/lib/x86_64-linux-gnu/dri/iHD_drv_video.so
libva info: Found init function __vaDriverInit_1_8
libva info: va_openDriver() returns 0
Again, it will invoke "CalculateDPBSize" function
When "VideoDECODEH264::QueryIOSurfInternal"  function in "mfx_h264_dec_decode.cpp" file invoked
if (par->mfx.MaxDecFrameBuffering && par->mfx.MaxDecFrameBuffering < dpbSize)
dpbSize = par->mfx.MaxDecFrameBuffering;
(gdb) p par->mfx.MaxDecFrameBuffering
$4 = 0
(gdb) p dpbSize
$5 = 16

FYI, @alariono

@guowillyy , could you find the right owner?

@dmitryermilov , Yan will help take a look this issue, thx Yan!

@VUNNYSODHI Could you please share your clip for debugging? Thanks.

We got recorded clips from our customer, which are covered under NDA and can't be shared publicly. Let us try to sync internally.
In essence, the stream header had the flag "bitstream_restriction_flag" set with "max_dec_frame_buffering==4", with a resolution of 2880x960 in h.264/AVC, as CodecProfile: 100, CodecLevel: 51.

Hi @wangyan-intel, you can use any H264 stream provided SPS header has "bitstream_restriction_flag" set and "max_dec_frame_buffering" with some value. I have tried some other 2880 clips issue is reproducible.
Thanks


lisp-modules: replace with an implementation based on github:uthar/nix-cl

Description of changes
Replace lisp-modules with implementation based on https://github.com/uthar/nix-cl.

Things done

Added Lisp section to Nixpkgs manual
Replaced lisp-modules with a new API. The interface consists of :

sbcl.pkgs
sbcl.withPackages
sbcl.buildASDFSystem
sbcl.pkgs.alexandria.overrideLispAttrs (oldAttrs: { ... })
sbcl.withOverrides (self: super: { ... })
~sbcl.override { packageOverrides = { ... }; }~
~sbcl.override { spec = { asdf = ...; pkg = ...; program = ...; flags = ...; faslExt = ...;  }; }~
also for abcl, ccl, clasp-common-lisp, clisp, cmucl_binary, ecl, mkcl

Added recurseIntoAttrs for just sbclPackages for now (maybe extend with other CLs in the future?)
Removed lispPackages and lispPackages_new
Updated clasp-common-lisp to release 2.2.0

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Left to discuss:

[ ] Flags is an array of strings
[ ] patches = [] defaults to empty anyway
[ ] checksummed vs latest

[X] I have used conventional prefixes in commits
[X] I have split the patch in to smaller commits
[X] I have Rewritten documentation
[X] I have changed v1.4 -> 1.4 in alexandria examples
[X] I have removed occurences of fetchTarball
[X] I have made it so packages.sqlite is removed before import
[X] I have fixed the weird alexantdria.overrideLispAttrs example
[X] I have Moved back ecl.pkg -> ecl in some places
[X] I have Removed the unnecessary gitignore
[X] I have removed extraneous setup-hook.nix
[X] I have documented that the previous approach didn't trust the upstream QL data
[X] I have switched to :nix-cl-user instead of :cl-user in builder

I should also document that the imported.nix file should not be edited by hand.

I should add self-links in lisp.section.md

Hmm, I wonder if it is a good idea to soften the breaking changes. I.e. move lispPackages to lispPackages-obsolete with a comment to remove in the next release. And maybe even have lispPackages_new-obsolete-layout? So that people can quickly get something-fixed and then migrate over a few months…

How about adding a warning like this?
--- a/pkgs/development/lisp-modules-new/lisp-packages.nix
+++ b/pkgs/development/lisp-modules-new/lisp-packages.nix
@@ -85,7 +85,8 @@ let
   #
   # Wrapper around stdenv.mkDerivation for building ASDF systems.
   #
-  build-asdf-system = makeOverridableLispPackage (
+  build-asdf-system = lib.warn "lispPackages_new is deprecated and will be removed in release 23.05"
+    (makeOverridableLispPackage (
     { pname,
       version,
       src ? null,
@@ -256,7 +257,7 @@ let
         (builtins.concatLists
           (lib.catAttrs "propagatedBuildInputs"
             (builtins.concatLists [[args] lispLibs nativeLibs javaLibs])));
-    })));
+    }))));

   # Build the set of lisp packages using `lisp`
   # These packages are defined manually for one reason or another:
--- a/pkgs/top-level/all-packages.nix
+++ b/pkgs/top-level/all-packages.nix
@@ -24220,8 +24220,9 @@ with pkgs;
     inherit clwrapper;
   };

-  lispPackages = recurseIntoAttrs (quicklispPackages //
-    (lispPackagesFor (wrapLisp sbcl)));
+  lispPackages = builtins.mapAttrs
+    (n: v: lib.warn "lispPackages is deprecated and will be removed in release 23.05" v)
+    (recurseIntoAttrs (quicklispPackages // (lispPackagesFor (wrapLisp sbcl))));

   quicklispPackagesFor = clwrapper: callPackage ../development/lisp-modules/quicklisp-to-nix.nix {
     inherit clwrapper;

Reachable warnings in Nixpkgs? Nah, forbidden

Ah, ok (-:
Hmm, moving the current attributes will still break setups. I was thinking, maybe it's better to include a deprecation notice in the next release notes, plus a comment in code, without renaming? Later, remove the current attributes in the second next release, and only then add the new version (so park this pull request until two next releases).

I think instant-replacement is not too nice either way

Ok, so let's sum up:

rename the two current lisp-modules and lisp-modules-new directories
rename their attributes with obsolete to make it visible that they are old
add the new thing

What about the warning using lib.warn?

Also, stumpwm and nyxt - should they use the old stuff for now, or come from the new sbclPackages ?

Perhaps something like the last 2 commits?

I think lib.warn for things reachable by attribute name is, even when not breaking the checks outright, discouraged. The renaming + source code comments about dropping soon should be enough.
A release note entry is welcome but not absolutely indispensable, I think.
If Nyxt and StumpWM work with new stuff, let's roll forward. CLPM, maybe leave back in a working state, and hopefully when it's next updated, it's migrated. Or maybe it gets into Quicklisp…

Hm wait, you want to say that we can just add this with no renaming (but add a lot of deprecation comments, and a deprecation comment in release notes)? Hmmm. Tempting.

Hm wait, you want to say that we can just add this with no renaming (but add a lot of deprecation comments, and a deprecation comment in release notes)? Hmmm. Tempting.

Yes it is possible, because the only new attribute is sbclPackages, otherwise the existing CL implementation attributes ( sbcl, ... ) are enhanced. (well there is the wrapLisp function, which conflicts with the current one, and is documented in the manual section. But we can rename the old one.)

My imagination: rename all the attributes to «-obsolete» versions, add aliases (forbidden to use from inside Nixpkgs; maybe we can add lib.warn there?), add release notes item about incompatible-but-better interface and migration period.
This is probably over-engineering, but I cannot convince myself of that fact without external help.

 I didn't know that

Such as lispPackages = lib.warn "lispPackages was replaced" lispPackages_obsolete in aliases.nix. This would evolve into lispPackages = throw "lispPackages was removed" in the next release.

Sounds good!

Are there any features of the current implementation that the new one doesn't support?

I do use esoteric weirdness where you can extract the knowledge how --eval/--load arguments differ between the implementations, it shouldn't take long to fix, but nominally it is a feature difference.

I do use esoteric weirdness where you can extract the knowledge how --eval/--load arguments differ between the implementations, it shouldn't take long to fix, but nominally it is a feature difference.

I was using --eval/--load flags before, but now switched to passing Lisp code on stdin - primarily this was used to compile and load ASDF

How to unblock ofborg-eval?

does this block merge? https://gist.github.com/GrahamcOfBorg/60be9656aa1258286c965f080d52612f

does this block merge? gist.github.com/GrahamcOfBorg/60be9656aa1258286c965f080d52612f

Yes, it completely blocks this PR.

So we can put throws but not warnings in aliases? You know what, I am fine with throws…
(They say how to fix them quickly, after all)

But adding throws would make the current stuff unusable already. How about keeping the old stuff as it is? Only keeping the deprecation notice in release notes, and then adding the throws in 23.11 ?

But adding throws would make the current stuff unusable already. How about keeping the old stuff as it is? Only keeping the deprecation notice in release notes, and then adding the throws in 23.11 ?

23.05: deprecate in Release notes and code comments
23.11: throw and rename for those inattentive
24.05: truly drop
?
Even better in my opinion…

Sounds good

woohoo!

This broke eval:
       error: attribute 'mysql-client' missing

       at pkgs/development/lisp-modules/ql.nix:59:22:

           58|     cl-mysql = super.cl-mysql.overrideLispAttrs (o: {
           59|       nativeLibs = [ pkgs.mysql-client ];
             |                      ^
           60|     });

Broke eval in a way that OfBorg doesn't catch? Ouch.

Broke eval in a way that OfBorg doesn't catch? Ouch.

Ofborg currently doesn't check whether aliases are used.

For reference, this PR adds over 4k builds of sbclPackages.* for x86_64-linux.

(which, all together, are smaller than one big build)

I would just re-wrapLisp, why doesn't it work?

Yeah, just re-wrap, until someone figures out how to "keep" the attributes between overrides

Meta: What's the right place to ask questions about the nix-cl code now that it's upstream?

I would say nixpkgs GitHub issues... or maybe Discourse/Matrix

I spent half an hour and couldn't get it working with wrapLisp. Got into a mess with trying to re-wrap the wrapped package, or not finding a way to unwrap the old wrapper out of the way, and not getting the desired results.
Maybe I'm missing something obvious?

Try like this:
kpg@e6330:~/scratch/nixpkgs (lisp2)
$ nix repl
Welcome to Nix 2.11.1. Type :? for help.

nix-repl> :l .
Added 18655 variables.

nix-repl> sbcl2 = sbcl.overrideAttrs (oa: { version = "6.6.6"; })

nix-repl> sbcl3 = wrapLisp { pkg = sbcl2; faslExt = "fasl"; }

nix-repl> sbcl3.withPackages (ps: [ ps.alexandria ])
«derivation /nix/store/mpvhd9cbrmg903hgk8ba29ph97i5lip5-sbcl-with-packages.drv»

Try like this:

Thanks for the working example! I had goofed.

Maybe it is a good question if we want to set hydraPlatforms to [] outside a known-used set (similar to the set of stuff in the old lispPackages I guess). As far as I understand, dependencies of things built on Hydra will be available as paths in binary cache even though they won't have job names, so no need to bother too much with tracking closures there.

They will be available, though in case of some failures it's easier to deal with them for builds that (also) have their own jobs.

Hey @Uthar ; thank you for this change, I am not very familiar with the maintainers of the Lisp ecosystem, could I invite you to create a NixOS team @NixOS/lisp so that release engineers (or any contributor) can ping you if needed?

(which, all together, are smaller than one big build)

It still adds overhead to the hydra evaluator where build time of the derivations does not matter, only count and complexity to evaluate them.

Also space.  But even cheap builds if you have lots of them (say, over staging* rebuilds).  The overall farm performance doesn't seem to be capable to exceed around 2 builds per second average, so that's at least half an hour of time on every rebuild.

I also noticed that clasp-common-lisp won't build on arm. How to disable it, so that it runs only on x86_64 ?

Ensure correct meta.platforms or meta.broken, ideally.

Maybe it is a good question if we want to set hydraPlatforms to [] outside a known-used set

Sounds good, I can take this task. I'll create a patch that sets an empty hydraPlatforms based on an allow-list of packages, starting from what was available in lispPackages

Thanks

Should we also remove recurseIntoAttrs from lispPackages and lispPackages_new, now that sbclPackages is the preferred option?

Indeed!

I also noticed that clasp-common-lisp won't build on arm. How to disable it, so that it runs only on x86_64 ?

Hmm. https://github.com/clasp-developers/clasp/blob/main/src/core/corePackage.cc#L1294 says no upstream support, so just set the platforms, I think.

could I invite you to create a NixOS team @NixOS/lisp

Sounds good, created the org and added @lukego @7c6f434c (Hope that's fine with you).

Fine with me. Maybe also @hraban ? Anyone else interested?

If I may also please ask you to send a PR in https://github.com/NixOS/nixpkgs/blob/master/maintainers/maintainer-list.nix to add your team and perhaps add some way to insert meta.maintainers = teams.lisp; to all lisp modules or packages which you think are in your scope, that would make it extremely awesome.

How would I go about overriding the packages in 22.11 to enable installation of StumpWM? Unsure how to override a whole ecosystem?

I think you can just import the package from another checkout


Docs in readthedocs.io are broken

the style sheet of http://django-categories.readthedocs.io/en/latest/ seems not loading properly.

@coordt any ideas or thoughts on this? I took a quick look and I didn't see anything obvious.

Properly documentation for urls are missing too.


Refactor and improve CLI code

Description
The ank CLI code is currently all located in the cli_command.rs module with now more than 1000 lines of code.
The CLI shall be spitted into smaller modules for better maintainability.
Additionally the design documentation of the CLI must be updated regarding structural view and the tags of the requirements. Currently there are tags for features, e.g., GetWorkload, instead of components.
Goals
Improve maintainability of the CLI
Final result

Summary
To be filled when the final solution is sketched.
Tasks

[x] Refactor the code
[x] Update SwDD

The PR with the implementation has been reviewed and merged into main.


Helper for prefixing all struct fields with pub

I often write a struct to later find I need all the fields to be pub, and usually at that point it's ... a sad time now. I could learn to use fancy multi-cursor input in my editor of choice and type pub once, but it would be nice if there was a helper to pub prefix all the fields of a struct. 😄

I think it would be nice if rust-analyzer could provide an assist to place multiple cursors at the beginning of each field. This will help your use case and more. But I remember something about LSP doesn't work with multi-cursors :(


Update the API

Remove the core* prefix from exported symbols.
Do not export PostgresBackend. This is an implementation detail.
Replace the create function with a static method in the Kernel: withPostgres.
Kernel.initialize now takes a LogContext instead of a MixedContext as it doesn't make sense for it to accept a Context object.
Update jellyfish-types to v2 and use the new JsonSchema type.

Change-type: major
Signed-off-by: Carol Schulze<EMAIL_ADDRESS>

@balena-ci I self-certify!


Support for multiple homes

I have 2 Panasonic aircon units in my house. I also have 1 in another house (parents).
Comfort Cloud supports these being allocated to different homes.
The plug-in puts them all, including the one not in my house, into my home in the Home app.
is multi-home support possible?

Multi home is not possible. The only thing that comes to my mind is to add the possibility of exclude one or more AC from adding to HomeKit. So in 1 house there would be all minus 1 and in the second all minus two. Only in this way on one account can you make in two different houses there are different ACs. But for this, you need to have a separate homebridge in each house and a separate account for Apple Homekit.

@Zaphod8 Option to exclude devices added in 1.9.0 beta. Can you install and test?

Seems to work ok. The log message is a little obscure though.
"[1/14/2024, 7:02:06 PM] [Homebridge Panasonic AC Platform] Removing accessory 'Mum - Aircon' (Cxxxxxxxxx) because it does not exist on the Comfort Cloud account (anymore?).”
Maybe something like:
"[1/14/2024, 7:02:06 PM] [Homebridge Panasonic AC Platform] Removing accessory 'Mum - Aircon' (Cxxxxxxxxx) because it has been manually excluded.”
Comfort Cloud allows for multiple homes and lists each aircon under the appropriate home. So there must be some meta data associated with each accessory identifying which home it belongs to?
Would it be more logical to provide an option to enter a home name and only load accessories associated with that home? If no home is specified just load them all?
Thanks for all the work supporting this.

On 14 Jan 2024, at 15:35, mkz212 @.***> wrote:
@Zaphod8 https://github.com/Zaphod8 Option to exclude devices added in 1.9.0 beta. Can you install and test?
—
Reply to this email directly, view it on GitHub https://github.com/homebridge-panasonic-ac-platform/homebridge-panasonic-ac-platform/issues/155#issuecomment-1890844692, or unsubscribe https://github.com/notifications/unsubscribe-auth/BFIUTEW6M2DIFE4KNJXJJDLYONOADAVCNFSM6AAAAABBZDCDMSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJQHA2DINRZGI.
You are receiving this because you were mentioned.

@Zaphod8
I corrected the log - it will be in the next beta. There is one function for removing old devices, when excluded, and when you remove it from CC account, so log for this operation is also one.
As for the option with giving the name of the house, thanks for the suggestion but:

first of all, I think that the exclude devices option is enough, usually all ACs from a given account are added anyway.
the exclude devices option is more accurate than choosing a house
in reponse API(as I quick checked it) I do not see that there is an assignment to a given house

This is the main screen from Comfort Cloud. Somehow it knows which house the accessories are in. That must be in the data it receives somewhere. CamOn 14 Jan 2024, at 19:58, mkz212 @.***> wrote:
@Zaphod8
I corrected the log - it will be in the next beta. There is one function for removing old devices, when excluded, and when you remove it from CC account, so log for this operation is also one.
As for the option with giving the name of the house, thanks for the suggestion but:
first of all, I think that the exclude devices option is enough, usually all ACs from a given account are added anyway.
the exclude devices option is more accurate than choosing a house
in reponse API(as I quick checked it) I do not see that there is an assignment to a given house
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

@Zaphod8 You can make a Pull Request with a solution to it. As for me, it is not needed.

Agree. I suspect there are few people who operate across multiple houses. On 14 Jan 2024, at 22:46, mkz212 @.***> wrote:
@Zaphod8 You can make a Pull Request with a solution to it. As for me, it is not needed.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>


[benchmark] invalid timestamp computation for multi-pass rendering

To repro:

start the benchmark application

set the following options:

VS: AluBound
PS: MemBound
LOD_0
Low_Precision
Interleaved
Depth Test
400 fullscreen quads
Texture

When "Single Renderpass" is checked, I get 10 FPS. When "Single  Renderpass" is unchecked, I get 106 FPS. But I do not really get 106 FPS, only the debug UI tells me that. Moving the UI around shows we are in the same FPS range.
Seems like the timestamp used doesn't work well with the multiple commands we generate.

I was not able to reproduce the issue with the latest commit merged (https://github.com/google/bigwheels/pull/357)
When "Single Renderpass" is checked, it renders around 513 FPS.

On the other hand, when "Single Renderpass" is unchecked, it renders around 4 FPS ( < 513 FPS).

Let me know if I set up something wrong to reproduce the issue.

@Keenuts , do you have an update?

Hello!
Sorry didn't get back.
Yes, can reproduce at 354933480c284862902ac4f26eb2af543e47852d
The issue is not the value, but the fact that the value seems wrong.
See the video:
When the FPS counter shows 90 fps, and I have a few quads, I do have 90 FPS, at least from the UI feel.
When I add 400 quads, FPS drops (which is expected), to something like 9FPS. But ticking single renderpass makes the FPS counter jump to 90 FPS again, while it's clear I am still at 9 fps from the UI feel.
issue.webm

A bit hard to see in the video :/ The recording rate is not that high.. But if you try on your device you might observe the same behavior: the UI doesn't lag behind the mouse at 100 FPS, as expected, but at 9, it does. And when it marks 100FPS with single renderpass, the UI lags behind heavily, as we were at 9FPS.

A bit hard to see in the video :/ The recording rate is not that high.. But if you try on your device you might observe the same behavior: the UI doesn't lag behind the mouse at 100 FPS, as expected, but at 9, it does. And when it marks 100FPS with single renderpass, the UI lags behind heavily, as we were at 9FPS.

I tried both on my laptop and desktop with latest commit (https://github.com/google/bigwheels/pull/365), but I couldn't repro it. Could you let us know your GPU and CPU? Were you in release or debug? and were you in Linux or Windows? thanks.

Desktop, linux, AMD Radeon Pro WX 3200.
The key ingredient to observe it is to put the number of texture fullscreen quads high enough, but not too high.
The rest can be set to default, or 1
Default settings, 1 sphere, 512 fullscreen texture quads, single renderpass checked: 100 FPS
Default settings, 1 sphere, 512 fullscreen texture quads, single renderpass unchecked: 8 FPS
To check the actual present values, I used mangofps (feels more correct than my feeling moving the UI window around 🙃)
With all settings, the counter is a bit off, but not that far, which makes sense. But with the texture quads (not color, not noise, just texture), the counter value goes wild.
Reproduced today at d3c53566510959e08117661476404b33da5d7749

Thanks for the update @Keenuts . Also all my tests were done on windows PCs, but I have a Linux laptop with Intel(R) UHD Graphics 620 (WHL GT2), so I will test on this Linux laptop.

@wangra-google , @Keenuts . I was able to reproduce the issue in my Linux laptop, so, I will start investigating what is going on.

@Keenuts @wangra-google . I believe I found the issue and fixed it. Please take a look of this PR that solves the issue (https://github.com/google/bigwheels/pull/375) and check on your side. I tested on my Linux machine and it works as expected now.


Error upgrading DART SDK through flutter via android studio

Steps to Reproduce
F:\flutter\bin>F:\flutter\bin\flutter upgrade

Logs
Checking Dart SDK version...
VERBOSE: [10/9/2018 9:02:45 PM] Initialize Transcript
Transcript started, output file is C:\Users\Andy\Documents\WindowsPowerShell\Transcripts\PSLOG_09-10-2018.txt
Downloading Dart SDK from Flutter engine 74625aed323d04f2add0410e84038d250f51b616...
Unzipping Dart SDK...
Expand-Archive : A parameter cannot be found that matches parameter name 'DestinationPath'.
At F:\flutter\bin\internal\update_dart_sdk.ps1:65 char:32
+     Expand-Archive $dartSdkZip -DestinationPath $cachePath

-->
I edited update_dart_sdk.ps1 replacing -DestinationPath with -OutputPath
    #Expand-Archive $dartSdkZip -DestinationPath $cachePath
	Expand-Archive $dartSdkZip -OutputPath $cachePath

F:\flutter\bin>F:\flutter\bin\flutter doctor -v
[√] Flutter (Channel beta, v0.9.4, on Microsoft Windows [Version 10.0.17134.285], locale en-US)
    • Flutter version 0.9.4 at F:\flutter
    • Framework revision f37c235c32 (2 weeks ago), 2018-09-25 17:45:40 -0400
    • Engine revision 74625aed32
    • Dart version 2.1.0-dev.5.0.flutter-a2eb050044

[√] Android toolchain - develop for Android devices (Android SDK 28.0.3)
    • Android SDK at C:\Users\Andy\AppData\Local\Android\sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
    • All Android licenses accepted.

[√] Android Studio (version 3.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 29.0.2
    • Dart plugin version 181.5540.11
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)

[√] Connected devices (1 available)
    • Android SDK built for x86 • emulator-5554 • android-x86 • Android 7.0 (API 24) (emulator)

• No issues found!

F:\flutter\bin>

The title says

via Android Studio

steps to reproduce says

F:\flutter\bin>F:\flutter\bin\flutter upgrade

What are the actual steps to reproduce?

Yes I should have been more clear.
Android Studio 3.2
Build #AI-181.55<IP_ADDRESS>14246, built on September 17, 2018
JRE: 1.8.0_152-release-1136-b06 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Windows 10 10.0
PS C:\Users\Andy> $psversiontable
Name                           Value

PSVersion                      5.1.17134.228
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.17134.228
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
While in android studio, I received a notice that there was a flutter upgrade available.  In terminal pane I executed the following:
Microsoft Windows [Version 10.0.17134.345]
(c) 2018 Microsoft Corporation. All rights reserved.
C:\Users\Andy\Documents\Android\flutter\flutter_firebase\flutter_firebase>f:
F:>cd flutter
F:\flutter>cd bin
F:\flutter\bin>
F:\flutter\bin>flutter upgrade
As noted the resolution was to update the  update_dart_sdk.ps1 file in /bin/internal by
Changing line 65:
            Expand-Archive $dartSdkZip -DestinationPath $cachePath

To:
            Expand-Archive $dartSdkZip -OutputPath $cachePath

Cheers
Andy ( Andre ) Paradis
From: Günter Zöchbauer<EMAIL_ADDRESS>Sent: Wednesday, October 10, 2018 1:03 AM
To: flutter/flutter<EMAIL_ADDRESS>Cc: andyp60<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [flutter/flutter] Error upgrading DART SDK through flutter via android studio (#22896)
The title says
via Android Studio
steps to reproduce says
F:\flutter\bin>F:\flutter\bin\flutter upgrade
What are the actual steps to reproduce?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/flutter/flutter/issues/22896#issuecomment-428460453 , or mute the thread https://github.com/notifications/unsubscribe-auth/AK1KETNAIxZZ3q5Ypq7pmHzWTkJCqBTeks5ujZuMgaJpZM4XUgFO .  https://github.com/notifications/beacon/AK1KEbR40iYFqJFWesFyngVqn4lcpoobks5ujZuMgaJpZM4XUgFO.gif

and flutter upgrade worked without errors, right?

Correct flutter upgrade worked without errors.
Cheers
Andy ( Andre ) Paradis
From: Günter Zöchbauer<EMAIL_ADDRESS>Sent: Wednesday, October 10, 2018 9:32 AM
To: flutter/flutter<EMAIL_ADDRESS>Cc: andyp60<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [flutter/flutter] Error upgrading DART SDK through flutter via android studio (#22896)
and flutter upgrade worked without errors, right?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/flutter/flutter/issues/22896#issuecomment-428618655 , or mute the thread https://github.com/notifications/unsubscribe-auth/AK1KEWY_kVYdIv3NX9N2_m9ENAD_ybI8ks5ujhLngaJpZM4XUgFO .  https://github.com/notifications/beacon/AK1KEZvPscBdFDOosxYqn3iahu1Gx3gXks5ujhLngaJpZM4XUgFO.gif

@devoncarew

I just found this problem recently. After some investigation, it's because Pscx module in powershell. So,
If you install Pscx module in powershell, it will overwrite alias for command Expand-Archive to use from Pscx instead of from Powershell. Two command work the same but Pscx use -OutputPath however Powershell use -DestinationPath.
For quick fix just change:
https://github.com/flutter/flutter/blob/a226c0f0d9f5d699a3d706fd9060edf6da45b13b/bin/internal/update_dart_sdk.ps1#L65
from
Expand-Archive $dartSdkZip -DestinationPath $cachePath
to
Expand-Archive $dartSdkZip -OutputPath $cachePath
Or it should specific which module to use as suggested in PowerShell issue#52
Microsoft.PowerShell.Archive\Expand-Archive $dartSdkZip -DestinationPath $cachePath


[feat] Have shadows even when disabling title bar on windows.

Describe the problem
I have implemented my own title bar, with maximizing, minimizing and close buttons. However, to disable the original buttons, I must pass
    "windows": [
      {
        "decorations": false,
      }

in the config. This disables all decorations, including the shadows. I only want to disable the title bar, but keep the shadows.
Describe the solution you'd like
I would like to have a setting to only disable the title bar, but keep the shadows (and maybe even the rounded corners on win11).
Alternatives considered
No response
Additional context
Title bar disabled but shadows missing:

Title bar enabled, but shadows are here:

use https://github.com/tauri-apps/window-shadows


[Snyk] Security upgrade golang from 1.22.5-bookworm to 1.22.6-bookworm

Snyk has created this PR to fix 2 vulnerabilities in the dockerfile dependencies of this project.
Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Snyk changed the following file(s):

build/Dockerfile

We recommend upgrading to golang:1.22.6-bookworm, as this image has only 89 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Out-of-bounds Read SNYK-DEBIAN12-CURL-7575306
  514

Out-of-bounds Read SNYK-DEBIAN12-CURL-7575306
  514

Out-of-bounds Read SNYK-DEBIAN12-CURL-7575306
  514

Link Following SNYK-DEBIAN12-GIT-6846203
  500

Link Following SNYK-DEBIAN12-GIT-6846203
  500

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

Already updated #201


Use webgl for bevy_ggrs example

We switched to webgpu by accident when upgrading to Bevy 0.11.
In Bevy 0.11 webgl2 is still default, but not when disabling default features.
It's not working properly everywhere yet, so switch back to webgl2, at least until it's default in bevy.

I noticed this too


Windows Agent does not start after reboot

To reproduce:

install 6.0.0-rc.1 on Windows 2012R2
reboot the machine

Expected result:

The Datadog Agent starts automatically after the reboot

Actual result:

The Datadog Agent is stopped after the reboot. The service is displayed as stopped.

Fixed with #1317 (service is defined as "delayed start")


path error in README.md

To Build the Field-Campaigner, it is mentioned to run the commands in directory /osm-reporter/deployment which I think should be /field-campaigner/deployment.
MakeFile has project Id osmreporter which needs to be replaced by field-campaigner(Please correct me if I am wrong).
@smit1678 What do you think about it?

👍  thanks


[Security] Update to safe version

This removes any potential for exploitation of CVE-2021-44228.

Where did you get the 2.15.1 version from? I can't build that stuff now
The maven repository seems to only have 2.15.0 and 2.16.0 (sse https://repo1.maven.org/maven2/org/apache/logging/log4j/log4j-core/2.15.0/ )

https://logging.apache.org/log4j/2.x/ says that

Log4j 2.15.1 has been released solely to disable access to JNDI by default. The CVE noted below was fixed in the 2.15.0 release. 2.15.1 is NOT a required upgrade but users may choose to use it to have confidence that JNDI will not be abused.

I just now realized that this has not yet been released...
But I hope that the release will happen soon. Otherwise I'll bump this to 2.16.0

Interesting thanks
Seems dependabot also means we should use 2.16.0


Odd, unaligned useless menu item that doesn't work on clicking

OS: Windows 10 22H2
Also, the Terminal menu item seems to be missing. Is this a Windows 10-specific issue?

This issue has been fixed in the portable version build 100
https://nilesoft.org/download/

Maybe close the issue?

I am having the same problem. I also have installed the Terminal (1.15.3466.0) application from Microsoft and I am running the latest version of Shell

OS: Windows 11 21H2 build 22000.1516

I am having the same problem. I also have installed the Terminal (1.15.3466.0) application from Microsoft and I am running the latest version of Shell

OS: Windows 11 21H2 build 22000.1516

Hi, just like @moudey said, you can fix this in the following way (or at least how I fixed it):

Run Shell as admin > Unregister.
Download latest portable from https://nilesoft.org/download/
Unzip the portable (Step 2) and copy all these files to the directory %programfiles%\Nilesoft\Shell
Replace when prompted.
Close out of all windows.
Run Shell as admin > Register.

Alternatively,

Uninstall Shell completely.
Download latest portable from https://nilesoft.org/download/
Run 'shell.exe' as admin > Register.

It worked. Thank you


[Windows/CMake] Enable SVD op and test.

@tensorflow-jenkins test this please.

@tensorflow-jenkins test this please

I am using windows 7 with keras and python 3.4. I would like to help with the tensorflow port.  I have models that currently train and test in theano, would be happy to help get tensorflow working and can validate it against theano for correctness.
How can I get started?

Woohoo! Looks like only failure was a flaky test on mac. Merging.

@mrry, @isaacgerg  do you have any specific instructions for Isaac about contributing on Windows, other thanthe generic instructions here: https://github.com/tensorflow/tensorflow/blob/master/CONTRIBUTING.md

@isaacgerg instructions how to get a windows build are here: https://github.com/tensorflow/tensorflow/blob/master/tensorflow/contrib/cmake/README.md
Keras should work ok, I ran the Keras example directory with a tensorflow backend under windows+gpu some time ago and it was happy.

@isaacgerg +1 to what @guschmue said. Currently we're working to get the Windows build at parity with what the Linux version supports. Our number-one priority is to get all of the kernels building, and you can see a list of the ones we're currently unable to build here. You can also see a list of tests that don't currently pass here. Any contributions to make these work would be most valuable!
Please feel free to open an issue if you run into any problems.


Issue 24 add sdk usage header

Motivation
Add feature #24
Checklist

[x ] Code follows coding conventions held in this repo
[x ] Automated tests have been added
[ x] Tests are passing
[ ] Docu has been updated (if applicable)
[x] Temporary settings (e.g. project ID used during development and testing) have been reverted to defaults

How to test
Run test 'CorrectSdkVersionHeaderAdded'

Thx @yrmartinez for the PR! @JanLenoch will have a look at it soon

Great @yrmartinez, thanks!
If you want to contribute to other hacktoberfest issue, feel free to pick any of the hacktoberfest labeled issues.

@yrmartinez - I have just refactored the tests to be placed on the placed on the right place: #28

Hey @yrmartinez,
we would like to thank you for contributing to Kentico's open-source project. As a sign of our appreciation, we would like to send you some Kentico Developer Community SWAG!
Please fill out your postal address if you are interested.
This comment was automatically generated. If you've submitted more than one pull request, it's OK to fill out the form just once.
Kentico Developer Community
If there’s anything we can do to help, please don’t hesitate to reach out to us at<EMAIL_ADDRESS>


Calling custom REST services

We can't get the toolkit working with a custom REST service we have. It seems to be building the URL incorrectly. The service is named "GetDirectDebitTransactions". Some details:
The code snippet:
using (var eftTypesTask = sfConnector.ExecuteRestApi("GetDirectDebitTransactions", "?StartDate=01/05/2015&EndDate=30/06/2015"))
Instead of going to this address: https://cs6.salesforce.com/services/apexrest/GetDirectDebitTransactions
It goes to the following: https://cs6.salesforce.com/services/data/v32.0/https://cs6.salesforce.com/services/apexrest/GetDirectDebitTransactions
Any ideas? Thanks.

This has been fixed as part of this pull request: https://github.com/developerforce/Force.com-Toolkit-for-NET/pull/131
It should make it into the next NuGet package.

Wade,Do you know when the next NuGet package is scheduled to be released? I was going to do a post about it and highlight some of the bigger fixes/enhancements from this latest round you have been on. And on behalf of Salesforce Developers, thank you for all you have done to support this release.Best,SaraOn August 26, 2015 at 3:51 PM Wade Wegner<EMAIL_ADDRESS>wrote:This has been fixed as part of this pull request: #131It should make it into the next NuGet package.—Reply to this email directly or view it on GitHub.  Sara Morgan NettlesIndependent Software Developer and Professional AuthorMy Blog: www.saramorgan.netMy Linked In Profile: https://www.linkedin.com/in/sarahasnolimits

I'm training @dcarroll on how to publish  and he's agreed to publish next. Dave, think you could do this today and publish as 1.1.1?

easy peasy, is everything we want in there in there?
On Wed, Aug 26, 2015 at 2:00 PM, Wade Wegner<EMAIL_ADDRESS>wrote:

I'm training @dcarroll https://github.com/dcarroll on how to publish
and he's agreed to publish next. Dave, think you could do this today and
publish as 1.1.1?
—
Reply to this email directly or view it on GitHub
https://github.com/developerforce/Force.com-Toolkit-for-NET/issues/130#issuecomment-135168975
.

--
Thanks,
Dave

I'd like to get https://github.com/developerforce/Force.com-Toolkit-for-NET/pull/121 in there too. Let's make a call later today.

Ok, if that's the case, tomorrow is when I can push to nuget.
On Wed, Aug 26, 2015 at 2:14 PM, Wade Wegner<EMAIL_ADDRESS>wrote:

I'd like to get #121
https://github.com/developerforce/Force.com-Toolkit-for-NET/pull/121 in
there too. Let's make a call later today.
—
Reply to this email directly or view it on GitHub
https://github.com/developerforce/Force.com-Toolkit-for-NET/issues/130#issuecomment-135172217
.

--
Thanks,
Dave

I"m not sure if this is intentional but the nuGet package that was published yesterday did not have the most up todate DLL in it. Are you still looking at doing another push up to nuGet.

The repo always will be slightly ahead of the NuGet. What specifically do you see missing?
(I'm not by a PC to easily check. I tested last night but it's possible I missed something. Hope not!)

IT appears that Nuget Package v1.1.0 only has the DLL version 1.0.0 so it is missing all the recent changes. including #131 Which is the one we need.

The assembly will always stay version 1.0.0 so we never force people to deal with assembly binding redirects.
I haven't pushed an update to NuGet since committing the changes you reference. We can do that today.

@dcarroll just pushed 1.1.1 with these updates. @bluephlame please give it a try!


cmd/netsetgo: Enforce pid argument

If we call netsetgo without arguments, it returns an error:
ERROR - no such process
It fails because Veth.MoveToNetworkNamespace calls
netlink.LinkSetNsPid, which needs a pid that belongs to another netns in
order to set a veth interface inside the new netns.
So, before doing any setup, check if a pid was passed as argument of
netsetgo.
Signed-off-by: Marcos Paulo de Souza<EMAIL_ADDRESS>

Thanks @marcosps!
Any chance you'd be able to add a test for this as well?
Maybe something to https://github.com/teddyking/netsetgo/blob/master/integration_test.go along the lines of
Context("when an invalid pid is provided", func() {
    It("exits -1", func() {
        ...
    })
})

Sure, will try to do so in a few hours!

@teddyking I'm struggling to install this concourse ci in my Ubuntu, but don't worry, it won't take the whole weekend (that's what I expect at least...)

Oh yeah that can be a bit tricky. You should also be able to just run the tests by installing ginkgo and running ginkgo -r from the root of the netsetgo repo. Although note that you'll need to be the root user and this will actually add/remove network interfaces from your machine (should be fairly safe but just FYI).
No worries if you can't get it working, I can add a test in later.

@teddyking tricky yes, but I made it work :)
I hope this now fulfills your expectations!

Awesome, looks good to me.
Thanks!
Ed


json file in the contacts directory

Hi,thank you for the great work!!!
For a new object, how to get the json file in the contacts directory

Hello,
For new objects, you need to train the 2D affordance prediction model on ContactDB. The code for training the model is here: https://github.com/priyankamandikal/graff/tree/main/affordance-pred
For inference on a new object, you can back-project the predicted 2D contact points to 3D using the depth map and save it in the json file.

Hello,thank for your answer. we want to do a comparative experiment with GRAFF. Can you provide the mesh and json files of unseen objects on the 3DNet dataset? Or the code using the depth map to get the json file from 2D contact points?      We are in a hurry to submit our paper, and time is extremely tight. If you can help us, we would be very grateful! @priyankamandikal

I've added supplementary code to evaluate on 3dnet and added some samples from a few categories. You can find the resources in the 3dnet branch. Note that this is only a sample subset. To evaluate on more meshes, please train the affordance prediction model and process the outputs accordinly.

Thank you so much!!!!!!!!!!!! @priyankamandikal


The question about run GA-DDPG in real world

Dear Liruiw:
I have a question about run GA-DDPG in real world.
You used the depth heuristic in test_realworld_ros_final.py, but I can't find the function self.graspnet.compute_grasps_score()  in the function that calculates the quality of the grasp in termination_heuristics(). Can I use the depth_termination_heuristics(depth_img, mask_img) function in utils to calculate whether the grasp is possible or not?

Yes, that heuristics should still be valid if the sensor still perceives depth at that range. Otherwise, you might want to use kinematics to compute the depth regions assuming the object is static.

I checked the paper and found that self.graspnet.compute_grasps_score() is the grasp evaluator of the 6-dof graspnet network, but I am not very familiar with this part of the network and I did not find this part of the code, can you share this function.

You can check this.


Distinct and OrderBy Date

A simple query as follow works great:
await db
    .select(lf.fn.distinct(table.id))
    .from(table)
    .orderBy(table.id, lf.Order.ASC)
    .limit(10)
    .exec();

or even by an string field:
await db
    .select(lf.fn.distinct(table.id))
    .from(table)
    .orderBy(table.title, lf.Order.ASC)
    .limit(10)
    .exec();

But ordering by date doesn't work in my case:
await db
    .select(lf.fn.distinct(table.id))
    .from(table)
    .orderBy(table.created, lf.Order.ASC)
    .limit(10)
    .exec();

By not working I mean it gives me a result that doesn't change if I modify the order to DESC or ASC. It just returns a static result as if the date created is zero or the same number for all rows. Checked manually and each row does in fact have a different value.
Removing the distinct function fixes the problem.
If you need any additional information, please ask.

By not working I mean it gives me a result that doesn't change if I modify the order to DESC or ASC. It just returns a static result as if the date modified is zero or the same number for all rows.

Additionally, when in a join, the distinct also breaks the order by when ordered by a field in the table that is not used in the distinct:
await db
   .select(lf.fn.distinct(table.id))
   .from(table, secondTable)
   .where(secondTable.parentId.eq(table.id))
   .orderBy(secondTable.id, lf.Order.ASC)
   .limit(10)
   .exec();
```TS

This will yield the same result as when ordering by `created` in the main table.

By not working I mean it gives me a result that doesn't change if I modify the order to DESC or ASC. It just returns a static result as if the date created is zero or the same number for all rows.

Could you post the code that registers the db schema? Can you verify that

created is registered as lf.Type.DATE_TIME?
when you populate these fields you are actually adding JS Date objects and not something else?

Removing the distinct function fixes the problem.

That sounds pretty interesting. Do you need to use distinct() though, if table.id is already unique (I assume?). Either way, does an equivalent query in SQLlite returns the results you expect?

Well, I just dug a little deeper into this, and here is what I have found:

If there is a distinct function involved, sorting can only be done by an indexed field. Fields without index just won't work for sorting. This needs to be documented.

If there is a distinct function involved in a multi-table query, sorting only works on the primary key of the table involved in the distinct function and won't work for any other field, indexed or not.

https://codepen.io/s-flhti/pen/mdEebgP


Options to send outbound HTTP requests via a HTTP proxy

To mirror a HTTPS-hosted repository, I recently had the need to send outbound HTTP requests from gitea via a HTTP proxy server, but found that gitea does not seem to come with a configuration option to define such a proxy.
I imagine we could have options like this (prefixed with OUTBOUND to avoid confusion with options concerning inbound requests):
[server]
OUTBOUND_HTTP_PROXY=https://<IP_ADDRESS>
OUTBOUND_HTTPS_PROXY=https://<IP_ADDRESS>

export environment variables

Thanks, this worked fro me. Shall we close this issue?

Yes, I think it can be closed as no need to reinvent wheel if OS already provides such functionality

Actually, it's not the OS providing this, but golang itself ;)

in Gitea case yes, GoLang provides that but other utilities will use it also so it's system-wide setting :)


Added pre-commit hook

This PR adds a git pre-commit hook that automatically formats all dart files

Looks like using https://github.com/hyiso/lint_staged is working (almost).
I say almost because I don't fully understand how it works with partially committed files and/or files outside the main lib/ directory (that is, the files in the example/push_notifications directory).
Let me explain:
For local development, I'm changing the files chatbox.dart, conversationlist.dart, and in the example main.dart, to point to the local server and use the correct app id.
When I make some changes to those files, I only partially stash those files, leaving out the parts for local development.
So, I tried to make some random changes to those files, and the ones in the lib/ directory looked like they worked correctly, but if I tried to stage the main.dart file in the example, lint_staged errored out with the error:
✔ Prepared lint_staged                    53ms
✔ Hide unstaged changes                   4ms
✔ Run tasks for staged files              9832ms
✔ Apply modifications                     7ms
✔ Restore unstaged changes                11ms
✔ Revert because of errors                73ms
✔ Cleanup temporary files                 21ms
✗ lint_staged failed due to a git error.
  Any lost modifications can be restored from a git stash:
    > git stash list
    stash@{0}: automatic lint_staged backup
    > git stash apply --index stash@{0}

Despite what the output says, there's nothing in the stash, and I cannot find a way to make this more verbose, for understanding what went wrong.
Side note:
Now you see in the pubspec.yaml file the following:
lint_staged:
  'lib/**.dart': dart format --fix && dart fix --apply

This is because formatting the files in the lib directory seems to work.
My tests, however were made with the following part in pubspec.yaml:
lint_staged:
  '**/*.dart': dart format --fix && dart fix --apply

I also tried:
lint_staged:
  'lib/**.dart': dart format --fix && dart fix --apply
  'example/push_notifications/lib/**.dart': dart format --fix && dart fix --apply

with the same results.

OK, I managed to get verbose output from lint_staged:
⠋ Restore unstaged changes...
lit_staged:git - git apply -v --whitespace=nowarn --recount --unidiff-zero .git/lint_staged_unstaged.patch
Checking patch example/push_notifications/lib/main.dart...
error: while searching for:
    final session = Session(appId: 'YOUR_APP_ID', enablePushNotifications: true);

error: patch failed: example/push_notifications/lib/main.dart:37
error: example/push_notifications/lib/main.dart: patch does not apply
lint_staged:workflow - Error while restoring changes:
lint_staged:workflow - Retrying with 3-way merge
lit_staged:git - git apply -v --whitespace=nowarn --recount --unidiff-zero --3way .git/lint_staged_unstaged.patch
Checking patch example/push_notifications/lib/main.dart...
error: while searching for:
    final session = Session(appId: 'YOUR_APP_ID', enablePushNotifications: true);

error: patch failed: example/push_notifications/lib/main.dart:37
Falling back to three-way merge...
Applied patch to 'example/push_notifications/lib/main.dart' with conflicts.
Applied patch example/push_notifications/lib/main.dart cleanly.
U example/push_notifications/lib/main.dart
lint_staged:workflow - Error while restoring unstaged changes using 3-way merge:
lint_staged:workflow - ProcessException: git apply -v --whitespace=nowarn --recount --unidiff-zero --3way .git/lint_staged_unstaged.patch
Checking patch example/push_notifications/lib/main.dart...
error: while searching for:
    final session = Session(appId: 'YOUR_APP_ID', enablePushNotifications: true);

error: patch failed: example/push_notifications/lib/main.dart:37
Falling back to three-way merge...
Applied patch to 'example/push_notifications/lib/main.dart' with conflicts.
Applied patch example/push_notifications/lib/main.dart cleanly.
U example/push_notifications/lib/main.dart
  Command: git apply -v --whitespace=nowarn --recount --unidiff-zero --3way .git/lint_staged_unstaged.patch
lint_staged:workflow - #0      Git.run (package:lint_staged/src/git.dart:55:7)
<asynchronous suspension>
#1      Workflow.resotreUnstagedChanges (package:lint_staged/src/workflow.dart:232:9)
<asynchronous suspension>
#2      runAll (package:lint_staged/src/run.dart:122:5)
<asynchronous suspension>
#3      lintStaged (package:lint_staged/lint_staged.dart:26:17)
<asynchronous suspension>
#4      main (file:///home/fri/.pub-cache/hosted/pub.dev/lint_staged-0.4.0/bin/lint_staged.dart:25:18)
<asynchronous suspension>

The error has to do with the fact that the file has been only partially staged, and the line
    final session = Session(appId: 'YOUR_APP_ID', enablePushNotifications: true);

does not exist in my working copy.
FWIW I originally chose the Nodejs version of lint_staged because it explicitly supports partially staged files.

Update to this whole lint_staged saga:
I opened a bug in the lint_staged repo, to see if they can fix it ( https://github.com/hyiso/lint_staged/issues/8 ).
I tried the same test with the JS version of lint_staged, and it fails in the same way, so it's safe to assume that what I'm doing is unsupported, and the current implementation of lint_staged is good enough for us.
@vickz84259 @moubaraks If that's OK for you, I propose to merge this PR.


perf: warn the usage of RENOVATE_GIT_AUTHOR

I have checked and decide to warn user's usage of RENOVATE_GIT_AUTHOR when enable a app bot.
The changes have impacts:

warn users it will override the git author of prs if RENOVATE_GIT_AUTHOR is set manually when enable a app bot.
do not warn users when use RENOVATE_TOKEN.

Related #400
Related #436

On Gitlab, it seems there is no needs to warn. It essentially equals the second impact on GitHub.

Thanks!


Broken build - 4.6 - behat

Failures:
edit-block-element.feature
element-editor.feature
https://travis-ci.com/github/silverstripe/silverstripe-elemental/jobs/504443871
Both graphql3 and graphql4 jobs are failing with the same error, so it's not graphql

Pretty sure the issue is there was a recent commit (I cannot remember which module it went into) that updating reading textarea content with chrome 91 which travis recently switched to
4 is green and last ran 2 days
I've triggered a rebuild on 4.6, which last ran 10 days ago - it should go green

Went green on re-run
https://travis-ci.com/github/silverstripe/silverstripe-elemental/builds/226489826
Closing issue


Updated German translations (Fixes #2132)

Change the word "Encoding" to "Kodierung".
Fixes #2132

Please add "(Fixes #2132)" to commit message that the issue will automatically closed.

@emanuelschuetze commit message changed.


Please invite me to the GitHub Community Organization

Name
Daniel Johnson
Discord Username (if applicable)
Daniel Johnson#7184
Additional Context
I first found Eddie on a livestream with RealToughCandy, he really helped demystify Open Source and make it feel like a welcoming community! I'm really hoping to improve my coding skills through collaboration, as well as meeting new coders of course.

Welcome to the amazing EddieHub community, @danjohnson77, where we believe Collaboration 1st, Code 2nd! 😄🎉👋


ccc

 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.:x: geotest-git sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Unable to load model from "data/levels/level.rgs"

After fixing build error in https://github.com/rg3dengine/rg3d-tutorials/pull/3, I noticed when running the rpg tutorial, this runtime error:
[ERROR]: Unable to load model from "data/levels/level.rgs"! Reason Visit(FieldDoesNotExist("Id"))
thread 'main' panicked at 'called Result::unwrap() on an Err value: Some(Visit(FieldDoesNotExist("Id")))', src\level
\mod.rs:19:14

I refactored particle systems in https://github.com/rg3dengine/rg3d/commit/a931fcf01eb4e2a8b859ab598e8dec5d7d2970c8 and in particular I replaced manual Visit trait implementation to proc-macro's implementations and I completely forgot that there are scenes with particle systems that will be affected. I'll fix this asap.

Fixed in https://github.com/rg3dengine/rg3d/commit/1001d80f07d35a814942b6fff3a9488618932307


Refactor

Codecov Report

Merging #611 (c14613b) into main (9df73ad) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #611   +/-   ##
=======================================
  Coverage   84.83%   84.84%
=======================================
  Files         300      300
  Lines       26938    26941    +3
=======================================
+ Hits        22854    22857    +3
  Misses       4084     4084

Impacted Files
Coverage Δ

lib/ffi/src/cps/stack.rs
98.82% <100.00%> (+0.04%)
:arrow_up:

lib/ffi/src/future/from_closure.rs
91.11% <100.00%> (ø)

Continue to review full report at Codecov.

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


CI: add sanitizers and valgrind-memcheck build configs

Description
Enabled sanitizers: address,undefined,leak,integer.
For Valgrind, it's only memcheck tool enabled.

Perfect PR, except for one little thing.


Add lien to production host project - to protect from accidental deletion during cloud build run

Issue: certain essential projects during day 3 operations like the production host project should have a lien on them.
For example if the project has a partially manually completed partner interconnect like #249 where there are partner side configuration events that are out-of-band of terraform - deleting the host project will break the interconnect.
Requirements:

gcloud add
gcloud delete
terraform version of the glcoud add/delete command (n/a as of now)

https://cloud.google.com/resource-manager/docs/project-liens#placing_a_lien_on_a_project

20240406: Closing issue during retrofit/rebase of this TEF V1 based/modified repo to TEF V4 standards
This issue may participate in the LZ refactor after rebase
Query on all issues related to the older V1 version via the tag
https://github.com/GoogleCloudPlatform/pbmm-on-gcp-onboarding/labels/2024-pre-tef-v4


az monitor log-analytics workspace get-schema | broken link in command description

Describe the bug
Request to engineering
Please replace the broken link https://docs.microsoft.com/en-us/rest/api/loganalytics/workspace-schema/get showing in the short-summary of  az monitor log-analytics workspace get-schema.

This link throws a 404

Related command
az monitor log-analytics workspace get-schema
Errors
404
Issue script & Debug output
.
Expected behavior
.
Environment Summary
Azure CLI 2.67.0
Additional context
No response

Thank you for opening this issue, we will look into it.

Closing GitHub issue as it is a duplicate of https://github.com/Azure/azure-cli/issues/29527.


Review k8s resources requested by each WMCore service

Impact of the new feature
Every single WMCore service running in CMSWEB
Is your feature request related to a problem? Please describe.
Yes, it started with this problem with the stepchain parentage cherrypy thread:
https://github.com/dmwm/WMCore/issues/11253
Now is a good moment to review how much resources (cpu and memory) each of the WMCore services is using, both in average and peaks, and update all their YAML files in the CMSKubernetes repository, e.g.:
https://github.com/dmwm/CMSKubernetes/blob/master/kubernetes/cmsweb/services/reqmgr2.yaml#L99-L105
Describe the solution you'd like
Based on the CMSWEB grafana dashboard(s), e.g. [1], evaluate the amount of resources used - and their peaks - by WMCore services, e.g.:

reqmgr2
reqmgr2-tasks
reqmgr2ms-***
reqmon
reqmon-tasks
t0_reqmon
t0_reqmon-tasks
workqueue
workqueue-tasks

and update every single service yaml with a more realistic resource requirements and limits. Here is an example yaml file:
https://github.com/dmwm/CMSKubernetes/blob/master/kubernetes/cmsweb/services/reqmgr2.yaml#L99-L105
Describe alternatives you've considered
None
Additional context
[1]
https://monit-grafana.cern.ch/goto/Lm2_r0W4z?orgId=11

As discussed over mattermost with Imran, resource requirements for reqmgr2-tasks service have been updated to
        resources:
          requests:
            memory: "1Gi"
            cpu: "400m"
          limits:
            memory: "5Gi"
            cpu: "1000m"

however, if the POD gets restarted or a new tag gets deployed, we will lose these new configurations that are yet to be persisted in the CMSKubernetes repository.
So, we need to deal with this issue as high priority, targetting it to be done by Sept 2nd.

Here is a CMSKubernetes PR correcting all of the WMCore resource requirements based on the current load in the system (which I consider to be a high load, with exception to global workqueue that has a low amount of GQEs)

Closed by https://github.com/dmwm/CMSKubernetes/pull/1201#event-7315913335


Add prettier, and eslint configs

There are no Prettier and ESLint configurations, which may lead to inconsistent coding styles and potential errors being missed.

Set up a .prettierrc file with rules for consistent code formatting, such as tab width, line endings, etc.
Set up an .eslintrc file to define linting rules, ensuring the code follows best practices and catches potential errors early on.

Hi @Neha, could you please assign this issue to me? I’d love to work on it. Thanks!

Hi @Neha, could you please assign this issue to me? I’d love to work on it. Thanks!

@giolaq ^

Hi @shrijit37 any updates on these tasks? Hacktoberfest is finishing on 31st, do you need any help? Or we can release it for other people to pick up


What about the ancestors (or ascendants) of a component? How would it be?

Hello man!
First of all, great compilation, congratulations for your efforts. I also really like styleguides, rules, like yours.
I was thinking of a case today which it may be interesting of debating it with you. What about a case where my component is like, a ribbon. You know, in the corner of an element. I was thinking in some markup like this:

New

Etc...

Lorem ipsum dolor sit amet.

In this case, the ribbon-container is an ancestor/ascendant of the component, and not a descendant. How would you suggest it to be marked up? ribbon-container? ribbon__container? I never read too much about BEM and this kind of suggestions so if you know something about it, it would be interesting listening to you.
Happy to contribute somehow. My best wishes. Regards, Gio. Oh, by the way, I'm also a brazilian, so if you'd prefer to discuss it in portuguese, feel free to do that.

Hi :smile: How I would do it:

Calling .ribbon__container is not ideal because the container is not a descendant of .ribbon. I would call it .ribbon-container as well.

Keep .ribbon-container in the same file as the rest of the .ribbon definition. Something like ribbons.sass, for example.

Hello. That's exactly what I did. :smile:
What you think of including this case to your repo?


Roles and formats - potential generic solution

We set a boolean for "Displayable" and "Downloadable"
We have a field on the Asset that points to the preferred downloadable and preferred displayable.
We use the preferred displayable in the viewer
Downloads default to the preferred downloadable
We offer the other downloadables as optional formats on the download page
Maybe the viewer could also offer the option of switching to other displayables.
Existing legacy tilt roles are preserved but used only as extra metadata "hints" for users that want them

@andybak for info, we are now displaying the same formats in the API as we are on the downloads page.


Not working with react 17

npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/little-state-machine-devtools
npm ERR!   little-state-machine-devtools@"*" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

Here's a PR for this:
https://github.com/bluebill1049/little-state-machine-dev-tools/pull/55


Related content using embeddings

I'm writing a TIL about this as I go.

The first time this ran successfully took this long:

If I got the code right it should be a lot faster on subsequent runs, since it will only calculate similarities for stuff related to the latest added entries.

https://til.simonwillison.net/tils now has the new tables:

https://til.simonwillison.net/tils/similarities?_facet=id looks good.

Here's the new SQL query: https://til.simonwillison.net/tils?sql=select
++til.topic%2C+til.slug%2C+til.title%2C+til.created
from+til
join+similarities+on+til.path+%3D+similarities.other_id
where+similarities.id+%3D+'python_pyproject.md'
order+by+similarities.score+desc+limit+10
select
  til.topic, til.slug, til.title, til.created
from til
join similarities on til.path = similarities.other_id
where similarities.id = 'python_pyproject.md'
order by similarities.score desc limit 10

Didn't quite work:
Skipped 446 rows that already existed
[{"id": "svg_dynamic-line-chart.md", "other_id": "observable-plot_wider-tooltip-areas.md", "score": 0.7923009914460658}]
error: Could not access 'HEAD~10'
Error: Must specify entries or --all

OK, the second time that ran it took 8s, which is as hoped for:

Here's the TIL with the full write-up: https://til.simonwillison.net/llms/openai-embeddings-related-content

The related content for the new article looks good:

Related

sqlite Related content with SQLite FTS and a Datasette template function - 2022-07-31
python Calculating embeddings with gtr-t5-large in Python - 2023-01-31
datasette Crawling Datasette with Datasette - 2022-02-27
sqlite Copy tables between SQLite databases - 2023-04-03
mastodon Export a Mastodon timeline to SQLite - 2022-11-04
datasette Scraping Reddit and writing data to the Datasette write API - 2023-03-13
sqlite Comparing two training datasets using sqlite-utils - 2023-05-23
shot-scraper Social media cards generated with shot-scraper - 2023-04-29
sphinx Adding Sphinx autodoc to a project, and configuring Read The Docs to build it - 2021-08-10
sqlite Replicating SQLite with rqlite - 2020-12-28


facepiles: handle missing profile pictures

i noticed just now that the default profile picture is a blank white square, which doesn't work for facepiles very well, since most themes (including mine) have white backgrounds. e.g. there's a like and a mention here, but you wouldn't know it :P

here's the like markup. note the lack of u-photo in the p-author.

  http://localhost:3000/users/ryan#likes/7

    http://localhost:3000/users/ryan

  likes this.

starting on this one. not sure which/whose placeholder profile picture to use. maybe gravatar's?

aha. gravatar lets you choose between different defaults, https://en.gravatar.com/site/implement/images/#default-image , and wordpress passes through that option and lets you choose in /w/wp-admin/options-discussion.php.
the catch is, that choice is used for both normally rendered comments and facepile avatars. ideally, i'd like a blank default avatar for comments, but a visible placeholder avatar for facepiles. hrm.

actually, disregard that. the HTML in my initial report shows that there's no  at all.


Improved robustness of texture/image export

This fixes #15 division by zero in a clean way

This seems to be what a pull request does.
Merging with dev branch.


Can I suppress Python 2-style model generation?

While Python 3 models are the ones being imported from the namespace, yet it seems that VS Code Pylance is looking for Python 2 models to give popup documentation and parameter names (at least when developing and with source code in the path). Python 2 models only have kwargs argument, which is burdensome. Since our generated SDK is for use in Python 3 only, can I suppress python 2-style model generation?

Any recommendation in a way of doing that programmatically?

I don't sadly. Let me look into why pylance is not looking at the python3 models instead, that seems to be more the root of the problem. In the meantime, you'd have to manually delete that file everytime you generate

ok @niander, talked to my manager, and we do want to add a flag to not generate py2 files

Understood. An easy way of trying to reproduce this is just generate the code in your current VS Code workspace, create an empty python file for local debug and import the models.


Add link to Project Updates page

Let's add the link to the Virtual Outreach event (September 15, 2022) recording by Manatee County
https://www.youtube.com/watch?v=H_kvWPZzJ7Y

Added in f4d6994


add customstatus instructions

@Evilfishbone , can I get a review/merge on this whenever you have some time?  Thanks!


Expose 'list_apps' on HTTP Server

Summary: Exposes the 'list_apps' action on the HTTP Server so that it can be used remotely.

(y)


Also support build sanity check on <3.11

/cc @mattip

Thanks, LGTM. Maybe add a comment that pytest swallows the chained error, apparently since it occurs when importing conftest.py very early in pytest startup


Speed upgrade bug

In this PR:

Fixed bug where missile speed upgrade would stop upgrading after first purchase

LGTM


Add support for Sonar analysis builder

Supports running standalone Sonar analysis as a build step.
https://wiki.jenkins-ci.org/display/JENKINS/Sonar+plugin
Change-Id: I86f8999dd951bf185c3b3e945605b5e860d52b03

@michalvanco @dkatona please check this out if it makes sense, so we can start using it internally and I can push the patch upstream as well.

Also merging with the latest master

Now I've rebased the unmerged patches onto the upstream master.


Export with template

Given:

A series of tasks, with tags (blocked by #8)
A per-tag template
A per-tag file path template

I want to be able to click "Export" and get all of the current information saved to a file.
An example template might be:
Ellen Dash: {{ time }} {{ task }}
{{ notes }}

And an example file path template might be:
/home/puppy/work/{{ tag }}/%y-%m-update.txt

If you have tasks with tags "Foo" and "Bar", this means if I exported things today I'd get:

/home/puppy/work/Foo/2024-09-update.txt containing all things tagged Foo
/home/puppy/work/Bar/2024-09-update.txt containing all things tagged Bar

brain dump:

and notably, the way i'm thinking of making STAYFOCUSED work is:

each task can have tags assigned to it
each tag can be assigned a template file
you can perform an export-to-file of everything associate with that template file

and the export-to-file would apply the template to each task with that tag, ensure there's a blank line between each one, and save it to a specified file
so you could have e.g. a 'Fractal' tag with this template:
{{ elapsed_time }} {{ description }}
{{ notes }}

and the result might look something like:
3h Some example task
Some notes about "Some example task"

2.25h Another task
Notes about "another task"

and maybe an "export time log" option that just uses a template of {{ elapsed_time }} {{ description }}, so with those same tasks you'd get:
3h Some example tasks
2.25h Another task

if whatever templating thing i use has filters, i'll probably ensure there's one for padding strings with spaces to a certain length, so you could e.g. make sure elapsed_time is always formatted with enough spaces to fill 6 characters (enough to fit up to 99.99h -- if you need more than that on one task you should probably break the task up, and you just don't need more than two decimal places of precision, and i'd argue you only need that for .25 and .75).


Improve documentation on ranges

Could you please update the range documentation to show the options you provided on #184 ?
There are no mentions to negative ranges, for example.

I've updated the range syntax page with new feature mentioned in #184. Thanks for point that out.

There are no references to negative numbers...


Add: bctv 16

Channel Name
bctv 16
Alternative Names (optional)
Burnsville Community Television 16
Network (optional)
No response
Owners (optional)
No response
Country
US
Subdivision (optional)
US-MN
City (optional)
Burnsville
Broadcast Area
s/US-MN
Languages
eng
Categories (optional)
legislative
NSFW
FALSE
Launched (optional)
No response
Closed (optional)
No response
Replaced By (optional)
No response
Website (optional)
https://burnsvillemn.gov/484/Burnsville-Community-Media
Logo
https://connect.telvue.com/dynamicthumb/W1siZiIsImVmNTc1NDMwLWNmMTQtMDEzMy1iMjUwLTAwMjU5MGYwMDg1Yy9zdHJlYW1zL3RodW1iLTE1MDc1NzM0ODJfODA3NTYuanBnIl0sWyJwIiwidGh1bWIiLCI2NDB4NjQwIl1d?sha=bd048435754f73be
Notes
No response

https://iptv-org.github.io/channel?id=BCTV16.us


Add support for Serilog

http://serilog.net/
Because that's perhaps the logs framework that @momo3038 should have added ;)

Nice one!
Thanks again


Can't wait on a wrapped future

I create a simple wrapped future in the attached file. It uses a sleep(1) to ensure that the lambda function doesn't complete too quickly to see the probem. The problem is that the wait() call completes, before the wrapped future finishes, thus the code prints:
finish
I am done
instead of:
I am done
finish
The way things stand, there seems to be no way to wait on a wrapped future.

I thought I'd attached that code. Apparently the web interface tricked me.

The code does the correct thing. The problem is that async(run) returns a future<future<void>>, so that async(run).wait() will just wait for the outer future to become ready. The outer future however becomes ready when run() has executed to completion. In order to achieve the effect you are looking for you'd need to change the code to:
async(run).get().wait();

alternatively (forcing implicit unwrapping):
future<void> f = async(run); f.wait();

I hope that explains things.


Make Function submission checks optional

Motivation
Currently when users submit functions/source/sinks, the function worker does some pretty extensive checks for java functions. This checks involve loading the user defined classes. Since function-worker is privileged code, but function itself is non-privileged, it could lead to situations where a misconfigured/malignant user-code causes catastrophic cluster failures.
This pr adds a switch to make these checks optional.
Modifications
Describe the modifications you've done.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes / no)
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

@jerrypeng have made the changes.

@srkukarni the PR in general looks fine since we really need to not load JARs in the worker/broker for security purposes.  I just have a few comments. However, the current validation logic is very complicated, hard to follow, and some methods are poorly named.  We should in the future, refactor the validation code.
Can we also have some unit/integration tests to test switching on and off the validation?

Closing this in favor of https://github.com/apache/pulsar/pull/4281