Should give an error when it is a required field

Steps to Reproduce the Problem
Entity.ts
  @Column({ fieldName: 'gender', required: true })
  gender!: string;

Specifications

Version: 4.0.1-alpha.19

Fixed before


[Chore] add batch call to GET http://localhost:4280/api/favorites

Avoid making multiple calls to check if a Listing is favorited by the current user.

Hi can i get this?


Create landing page for applicant

As a program applicant
I want to see a landing page once logged in that displays in progress applications, submitted applications, and a new application button
so that I can have my options and information consolidated in one page.
AC:

[ ] new page that display in-progress applications, submitted applications, and a new application button
[ ] screenshot and share to Miro board for customer review

No longer relevant for RTP. Will create new cards for Intake.


UI Changes for configuring monitoring

When configuring monitoring for Azure Spring Cloud with https://github.com/microsoft/azure-spring-cloud-training/tree/master/03-configure-monitoring, the UI has changed. The screenshot on this page should be retaken.

Thani you @vermegi ! This has been updated in #145


Exclude comments with links in hide-low-quality-comments

https://github.com/refined-github/refined-github/issues/5257
Test URLs
https://github.com/ambarvm/ng-action-test/issues/1
Screenshot

I just realized the test I added in c400478 (#5363) was pointless

So we should drop it here

So we should drop it here

No need, I reverted it by accident in 47b8d70 (#5363), so it didn't make it into the PR in the end :smile:


Handle CTRL-/ on gVim.exe

Fixes https://github.com/vim/vim/issues/9895

putty the pesudo terminal on Windows handle CTRL-/ like below on WM_KEYUP. since Emacs is in trouble that can't undo with CTRL-/.
https://git.tartarus.org/?p=simon/putty.git;a=commitdiff;h=011542a9d589d48c92a315bb64e74dcfc8bc6258


How to remove/change sign-in account?

Hi,
Would you be so kind and add instructions to also, how to remove the sign-in account from the device's details on Teams Admin Center? Even I remove the device and provision it again, the same account is assigned automatically.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6ff0b460-9f13-fefe-37b1-b40d5062fb86
Version Independent ID: 6ff0b460-9f13-fefe-37b1-b40d5062fb86
Content: Remote sign in and sign out for Teams Android devices - Microsoft Teams
Content Source: Teams/devices/remote-sign-in-and-sign-out.md
Service: msteams
GitHub Login: @cazawideh
Microsoft Alias: czawideh

Hello @Petri-X
To understand better and try to reproduce, are you performing a Remote Sign-Out before removing the device? is the device being factory reset before re-provision?
Thanks

Hi @get-itips
When looking for this: Teams phone - Remote sign out
It does not give too much information where I should be. But I tested by select from User Phones view:

But I tried also when I have open the phone:

So do you know if there is some other place for sign-out as well?

Hello @Petri-X sorry for the delay
I understand, will check that. Have you seen my other question?

is the device being factory reset before re-provision?

Hi,
I'm not sure what role the factory reset plays on this? Let say your phone went broken, as the user account is on phone object in the Teams Admin Center.
. . .but in these days I have learnt something more :)
Intune (Endpoint Manager) have a role on this game as well. The phone device is marked on there, and in there I can actually see the link between phone device and my account. Sounds like there is a bug in TAC code, when you remove the phone devices from TAC, that does not remove phone device from Endpoint Manager.

when you remove the phone devices from TAC, that does not remove phone device from Endpoint Manager.

Hello @Petri-X I had information that removing the device from TAC does not unenroll the device from Intune and that is currently by design

I wish I could use "...by design..." statement at home as well, when I have forgot to clean, or made food :D
I believe more realistic answer would be: "someone had a bit rush, and this was forgot to do".
But perhaps this will be fixed in future.
Big thanks @get-itips to your effort again _o/

@get-itips
Just an extra note, normally CAP license does not includes the intune license. But because it accidentally sign-in with my own account, which had intune license, and because of that it appears to endpoint manager.
And yes, the factory reset was required on the device. My own account was still sign-in to that device, which I found from factory reset view. Need to say, it was quite well hided =)


Failed when with_velocity=False in nuscenes evaluation

https://github.com/open-mmlab/mmdetection3d/blob/c556d27444958476f18e81c0a268a42f7a92e7f9/mmdet3d/datasets/nuscenes_dataset.py#L548
If with_velocity=False, box3d.tensor will have shape [*, 7] which make velocity length less than 3. It seems that nuscenes Box doesn't support velocity with length less than 3.

We have not trained models without velocity prediction on nuScenes for long. I am not very sure of your situation from your description. Maybe you can try to set velocity to default value (like nan or 0) as nuScenes box does to avoid the potential problem.
If there exist minor bugs when with_velocity=False, PRs are definitely welcome. Thanks for your information.


Add missing links, convert some crefs into xrefs, add alt text to images

Overview
Some links were empty, possibly awaiting the existence of their corresponding docs. Now that those docs exist, I've updated the empty links I found.
I also converted a few crefs into xrefs, since it didn't appear that crefs work in our markdown.
I also added alt text to all our images that were missing them. This is important for the accessibility of our docs to ensure images have alt text for screen readers.

/azp run mrtk_docs

/azp run mrtk_docs


Add Zipkin optional module

Once #15 done, we could add a Maven module with Zipkin UI and Zipkin Server. Thus we could have a UI to browse collecting logs.
See https://dzone.com/articles/tracing-in-microservices-with-spring-cloud-sleuth
If Zipkin is not started, the application continue to work.

Close by #23


Bad interaction with SDL RenderPresent?

I'm experiencing a rather strange problem related to v4l2 frame reading: whenever ReadFrame() is called (or, more, precisely, the buffer enqueuing/dequeuing ioctls), SDL screen updates stop or happen at random, large intervals. Exact behavior seems to be different on different machines. I don't know if the problem is in V4L2, SDL, the GL stack, X11,  drm, or my code. I still didn't try to reimplement the example in C to bypass the Go packages, but it doesn't seem that Go is causing this.
Perhaps you could try the following example to see if you have any idea on what's happening here? This code should show a smooth color change in the SDL window, but fails to do so if cam.ReadFrame() is uncommented. The frame counter shows that ReadFrame() is not blocking, it's just the window image that's not being properly updated.
package main

import (
	"fmt"
	"github.com/blackjack/webcam"
	"github.com/veandco/go-sdl2/sdl"
	"os"
)

func main() {
	window, err := sdl.CreateWindow("test", sdl.WINDOWPOS_UNDEFINED,
		sdl.WINDOWPOS_UNDEFINED, 640, 480, sdl.WINDOW_SHOWN)
	if err != nil {
		fmt.Printf("can't create window: %s\n", err)
		os.Exit(1)
	}
	defer window.Destroy()

	renderer, err := sdl.CreateRenderer(window, -1, sdl.RENDERER_ACCELERATED)
	if err != nil {
		fmt.Printf("can't create renderer: %s\n", err)
		os.Exit(1)
	}
	defer renderer.Destroy()

	texture, err := renderer.CreateTexture(sdl.PIXELFORMAT_YUY2, sdl.TEXTUREACCESS_STREAMING, 640, 480)
	if err != nil {
		fmt.Printf("can't create texture: %s", err)
		os.Exit(1)
	}
	defer texture.Destroy()

	x := uint8(0)

	cam, err := webcam.Open("/dev/video0")
	if err != nil {
		fmt.Printf("can't open webcam: %s", err)
		os.Exit(1)
	}
	_, _, _, err = cam.SetImageFormat(0x56595559, 640, 480)
	if err != nil {
		fmt.Printf("can't set image format: %s", err)
		os.Exit(1)
	}
	cam.StartStreaming()

	for {
		cam.WaitForFrame(1)
		cam.ReadFrame()
		renderer.SetDrawColor(x, x, x, 255)
		x++
		renderer.Clear()
		renderer.Present()
		fmt.Printf("%03d\r", x)
	}
}

Duplicate of #5


[storage] Move name field to top level from spec

Signed-off-by: Boris Glimcher<EMAIL_ADDRESS>

ok, to break. merging. cyclic dependency. tested locally


Support Ubuntu 17.10

This PR has the necessary fixes to get the system to build + run on Ubuntu 17.10. There was a problem with getting C++11 to work correctly where you have to add compiler directives. From what I can tell online this seems to be a problem specific to g++5.4, so I limited the fix to just that version.
Note that I added the header to 'value_factory.h' and for some reason it changed every line.
https://www.youtube.com/watch?v=b0MFNM9oOoU

Coverage increased (+0.02%) to 76.888% when pulling 825a703d0163cf7c6e5288f0f76a2d328fb23dd7 on apavlo:jan2018-ubuntu17 into 2b649041042573b36d52998bf3e78eb76f2ee829 on cmu-db:master.


Normalize the config file to minimize further changes

This is the result of running the configlet fmt command.
See https://github.com/exercism/meta/issues/95

Blast it. See exercism/meta#95 for a great discussion about why I need to redo this later.


sensors status ideas

Piecing together sensor issues from logs is often more difficult than it needs to be. I was thinking of expanding sensors status to include the device id (decoded), rotation, internal/external, mapping between uORB numbering and CAL parameters, etc.
Then sensors status could be appended to the log preflight and postflight just as as perf and top are now.
Alternatively we publish a few of the key details that are missing and do more work on flight review to decode and summarize. A snapshot of the sensors status table can also be helpful for debugging sensor rotation issues.
@davids5 @priseborough @bkueng

Sensor status identification can definitely be improved and these are some good ideas. sensors status is a good place for that.
Adding that directly to the log has the disadvantage that it won't work on archs where logger runs on a different CPU, such as snappy (there might be others in future) - perf & top already have that problem, so we might have to find a more generic solution. Such as logger sending a shell command to the FMU and receiving its stdout.

Thanks, that's a good point.


Repomap - A Repomix/Mermaid/GenAI repo visualisation tool

I've used Repomix to make Repomap.
https://github.com/George5562/Repomap
It creates a Repomix XML, uses an LLM with Langchain to convert to mermaid so you can visualise your project as a flowchart.
You can specify the types of nodes (pages, components, functions etc) and edges/relationships (depends on, uses, renders etc).
Using the --add CLI you can add an enhancement "add nextAuth", and two additional LLM calls will create a suggested file structure, and then create an updated flowchart, now with possible future components added.

Hi, @George5562 !
Thank you for creating and sharing Repomap! This is a fantastic use case of Repomix's XML output.
The combination with LLM and Mermaid for visualization is really clever.
I haven't looked into the details yet, but I'm looking forward to exploring it more thoroughly.


Mirrored PR: update to node 20

Mirroring because I can't run workflows in education/autograding.
https://github.com/education/autograding/pull/288

any idea when this will be merged upstream?


Need a quick response -> How get post by link like this link: https://www.instagram.com/p/CClpx7qsYm-/ ???

Hi friends..
InstagramUserFeedRequest is not good for this.
so can i get post by link?

i used this method
    public static String getMediaIdFromURL(String mediaCode) {         String alphabet = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789-_";         long id = 0;         for (int i = 0; i < mediaCode.length(); i++) {             char c = mediaCode.charAt(i);             id = id * 64 + alphabet.indexOf(c);         }         return id + "";     }
argument is CClpx7qsYm-, i didnt bother but you can use regex for input argument 'https://www.instagram.com/p/CClpx7qsYm-/'

thanks but i mean get all the post by link not just Code or id.
the code you send is a part of code just for return id so what is next for get feed by id?

i used this method
`

public static String getMediaIdFromURL(String mediaCode) {
String alphabet = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789-_";
long id = 0;
for (int i = 0; i < mediaCode.length(); i++) {
char c = mediaCode.charAt(i);
id = id * 64 + alphabet.indexOf(c);
}
return id + "";
}

`
argument is string 'CClpx7qsYm-', i didnt bother but you can use regex for input argument 'https://www.instagram.com/p/CClpx7qsYm-/'

Thanks.. but i mean get all the post by link not just return id.
the code you send is for return id so what is next for get feed by id?

i got it, but i can't to get feed by mediaCode. If i solved it i'll put here)

Try with InstagramMediaInfoRequest to grab info for certain media
Make sure to pass in the media id (which is converted from short code from URL using method above).
Library does provide util methods to convert shortcode and back in InstagramCodeUtil but I think one is bugged there is a pull request opened with correct method for which you can use. #444

Addressed in rewrite or no longer relevant.


Update pipelines.md

Website needs bullet points to be 4 spaces to render correctly. See https://tekton.dev/docs/pipelines/pipelines/ for the issue.
/kind documentation
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Docs included if any changes are user facing
[x] Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been filled in or deleted (only if no user facing changes)

NONE

/retest
nice to see we are not the only ones with flakes :D

/test pull-tekton-pipeline-alpha-integration-tests

/lgtm


.NET 6/Fable 3.6 support

Pull request checklist¹¹
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[ ] Build (fake build or .\build.cmd) on local branch was successful

Pull request type

Please check the type of change your PR introduces:

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no api changes)
[x] Build related changes
[x] Documentation content changes
[ ] Other (please describe):

Does this introduce a breaking change?

[ ] Yes
[x] No

Other information

Started getting these errors with the new versions (.NET 6/Fable 3.6)

i hope it doesn't break in the old versions. also the ./build.cmd is erroring with this, probably it will pass on the CI

ty!

ye the log thing is still happening =( will check.

@Shmew this seems to be blocking us to update to the latest version of Fable because we depend on Feliz.Recoil (which depends on the library, as you know ;) ). What is the status of this PR?

Maybe it would be a good idea to fork these libs to Fable-compiler or somewhere else ?
@Shmew doesn't seem to be maintaining this one for quite some time

I am also stuck can't update to latest Fable. Might have to fork Feliz.Recoil. @Shmew hope you can merge this.


Feature/docker package

Switched from DATABASE_URL back to DB_URL, updated the documentation.
Also added in healthcheck and fixed a few little glitches

cc @cpcundill


Feature request: enable modification of data in tfrecord files

It would be nice to be able to append or remove samples to or from an existing set of samples saved in a tfrecords file. Otherwise the file has to be duplicated, which is unpractical for datasets >> 1TB.
Formulated in a more generalized way, this can also be seen as splitting and concatenating tfrecord files.

I suspect that this is a feature that is not practical to implement since TFRecords are a simple serial file format. Additionally, compressed gzip is not seekable, so the compression would  have to be chunked into some seekable block size. Probably, the best solution is to shard your data into multiple record files. @concretevitamin, could you comment?

@jhseu, i know you are giving a talk at the dev summit, but since you are talking about TFRecord in it, I thought you might have additional insight.

Removing records is pretty infeasible without rewriting. I'm not sure we'd want to design the format with that feature, because we're typically optimizing for sequential reads and writes.
Additionally, certain filesystems (e.g., GCS, HDFS) don't allow in-place modifications, so it wouldn't be generally useful.

Any chance that append will be implemented?
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/lib/io/tf_record.py#L91
@josh11b

For uncompressed files, you can concatenate them since there is no per-file header. Beyond that, I don't think we have immediate plans to implement append.  That comment you link to was written before we supported compression, which complicates the implementation of the feature.

For uncompressed files, you can concatenate them

How should we do this? Simply make another file and concat the files to one? How do we prevent compression?

@evolu8 you should be able concatenate with bash or windows cmd
https://ss64.com/bash/cat.html
https://ss64.com/nt/type.html

Any update on this issue?
Thanks.

Any updates? Can I append an existing TFRecord?

Any updates? Can I append an existing TFRecord?

Any update about this issue?

Any update?


Beta3 Router Will Leave.

Hi Guys,
I'm having to do this in my form that is embedded in more than one place because in one area transition hook is not needed and context is not available.  It's horrible and it seems like the old way was better?  Maybe I'm doing it wrong:
// above is asset form .....

export default AssetForm;

var addTransitionHook = function (Component) {
    return React.createClass({
        mixins: [ TransitionHook ],
        contextTypes: {
            router: React.PropTypes.object.isRequired
        },
        routerWillLeave: Component.routerWillLeave,
        render: function() {
            return <Component {...this.props} />
        }
    });
};

export var TransitionableAssetForm = addTransitionHook(AssetForm);

Thoughts?

Have you figured it out yet? I'm in the same boat

Still doing that horrible stuff above...

check out the Lifecycle mixin on master https://github.com/rackt/react-router/blob/master/docs/Lifecycle.md


Ouput Directory For Divergent Results

Type of change

Improvement (improvement to code, performance, etc)

Description

Several changes were made to the ledger utility tool in preparation for future updates, most notably: output files are now created in a new directory, compare tool no longer overwrites existing files and will throw an error instead, and improved usability and consistency for the outputDir flag

Since most of these changes have been deemed unnecessary, this PR will be closed and the remaining changes will be rolled into a new PR.


portaudio: apple-sdk migration

Migrated portaudio to the new implicit apple-sdk (references to Darwin frameworks have been removed)

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

@emilazy Should I use apple-sdk or apple-sdk_11 here (in lieu of #352129)?

Hmm maybe the musescore change can be decoupled from the portaudio change? Although they are related? It's just that changes to musescore alone don't need to go through staging, and not only that, I'm updating musescore in #353207 , so there's a small chance of merge conflicts.

There is never any reason to specify plain apple-sdk as a dependency; it is implicit in all builds. The 11.3 PR will rebuild every single package, so there is no concern there. Just add whatever is required for full functionality. I see no availability conditionals or SDK checks in PortAudio so it probably needs no SDK dependency at all. Since it depends on Qt, however, it will inherit the macOS 12 SDK anyway.
Grepping for __builtin_available in the MuseScore repository turns up a few __builtin_available(macOS 11.0, *) results, so it wants apple-sdk_11.

@doronbehar Works for me, I've just dropped the commit re MuseScore
@emilazy thanks for the wisdom 🙏🏻

PortAudio doesn’t seem to need any SDK override at all, per my previous comment. It can just be dropped entirely.

@doronbehar Works for me, I've just dropped the commit re MuseScore

But will it be required once this portaudio commit reaches master? Or perhaps I can cherry pick that lost musescore commit to master right now? Even without the change to portaudio?

No coupling at all. Both can happen at any point.

Oh, musescore does overrides… Then yes, they’ll both need adjusting, since those overrides are now pointless.

According to the docs, if I add another apple-sdk_* to the build inputs, the highest SDK is selected.
Would this patch do for musescore?
diff --git a/pkgs/applications/audio/musescore/default.nix b/pkgs/applications/audio/musescore/default.nix
index 6f28bf98a1b1..64fd9456da22 100644
--- a/pkgs/applications/audio/musescore/default.nix
+++ b/pkgs/applications/audio/musescore/default.nix
@@ -31,24 +31,14 @@
 , qtnetworkauth
 , qttools
 , nixosTests
-, darwin
+, apple-sdk_11
 }:

 let
-  stdenv' = if stdenv.hostPlatform.isDarwin then darwin.apple_sdk_11_0.stdenv else stdenv;
-  # portaudio propagates Darwin frameworks. Rebuild it using the 11.0 stdenv
-  # from Qt and the 11.0 SDK frameworks.
-  portaudio' = if stdenv.hostPlatform.isDarwin then portaudio.override {
-    stdenv = stdenv';
-    inherit (darwin.apple_sdk_11_0.frameworks)
-      AudioUnit
-      AudioToolbox
-      CoreAudio
-      CoreServices
-      Carbon
-    ;
-  } else portaudio;
-in stdenv'.mkDerivation (finalAttrs: {
+  portaudio' = if stdenv.hostPlatform.isDarwin then portaudio.overrideAttrs(oldAttrs: {
+    buildInputs = oldAttrs.buildInputs ++ [ apple-sdk_11 ];
+  }) else portaudio;
+in stdenv.mkDerivation (finalAttrs: {
   pname = "musescore";
   version = "4.4.2";

@@ -145,7 +135,7 @@ in stdenv'.mkDerivation (finalAttrs: {
     alsa-lib
     qtwayland
   ] ++ lib.optionals stdenv.hostPlatform.isDarwin [
-    darwin.apple_sdk_11_0.frameworks.Cocoa
+    apple-sdk_11
   ];

   postInstall = ''

I add another apple-sdk_* to the build inputs, the highest SDK is selected.

Sounds like an unnecessary risk, why not simply .override it ?

@doronbehar  apple-sdk is coming from Darwin's stdenv itself now, as @emilazy pointed out,  it's an extraBuildInput according to this (hopefully I am looking at the right bit):
https://github.com/NixOS/nixpkgs/blob/2f45b7ffd7297dd721a747e8d8399abb84ccf1ab/pkgs/stdenv/darwin/default.nix
In other packages requiring apple-sdk_11 (i.e. games that require Metal), I add that to the build inputs, and it overrides apple-sdk, so I think this approach would work.
Tagging also @reckenrode as he may have more insights - what would be the idiomatic/recommended  way to override the implicit apple-sdk with apple-sdk_11 in derivations?
Sorry for the chaos, just trying to understand what the best pattern would be with

I apologize for the lack of clarity. I’m working on a documentation update for the new SDK, and I’ll add this situation. The recommended pattern is to add the SDK to your buildInputs. It is not necessary or recommended to override the one in the stdenv.
Unlike the old SDK pattern, there’s no risk from having multiple SDKs in your inputs. In fact, the SDK is designed with that situation in mind (because it turns out that it’s impossible to avoid in the general case). If there are multiple ones (from the stdenv, propagation, your inputs, etc), the highest version is chosen. The SDK uses a hook to do that.
https://github.com/NixOS/nixpkgs/blob/a86d06940e17a4c236d9ea3cefdd323cad362679/pkgs/by-name/ap/apple-sdk/setup-hooks/sdk-hook.sh
If you really need to ensure that derivations using your package as a dependency use at least a certain version, you can add it as a propagatedBuildInput. If your package is expected to be added as a nativeBuildInput, use depsTargetTargetPropagated instead.
However, other than compilers, it’s usually a bad idea to propagate the SDK. It becomes part of your package’s public API, making changing the propagated SDK version becomes a potentially breaking change. (The current version of the documentation suggests otherwise, but it is being revised to reflect this new guidance.)

Unlike the old SDK pattern, there’s no risk from having multiple SDKs in your inputs. In fact, the SDK is designed with that situation in mind (because it turns out that it’s impossible to avoid in the general case). If there are multiple ones (from the stdenv, propagation, your inputs, etc), the highest version is chosen.

So is it possible (as in the musescore case) to build a package using an old SDK and building the dependencies of it with the same old SDK?

You don't need to do anything fancy. Just use PortAudio as a normal dependency, no overrides, no propagation. The override stuff was hacking around flaws of the old pattern.

So is it possible (as in the musescore case) to build a package using an old SDK and building the dependencies of it with the same old SDK?

To elaborate on @emilazy’s comment, the legacy SDK had problems with propagated packages, which is what Qt did. It propagated a bunch of different frameworks. In the legacy, frameworks were normal packages. If you had CoreAudio 11.0 and CoreAudio 12.3, you’d have to versions of effectively the same framework. These would get mixed, and your build would break (usually with an error related to punctuation).
The solution for the legacy SDK was to do a bunch of hackery (using overrideSDK) to replace those propagated frameworks with the ones for the preferred SDK. Qt (for example) would still be built with whatever SDK it used, but you would get the frameworks you needed for your package. Usually. Unfortunately, there are ways to propagate packages that are invisible from Nix (e.g., setup hooks, interpolating a package’s path in a phase script, etc).
The new SDK works by defining the SDK as a sysroot at SDKROOT (and a path to all the SDK bits at DEVELOPER_DIR). There can only ever be one sysroot at that environment variable. If you add every single SDK to your buildInputs, their hooks will cause SDKROOT to point to the SDK with the highest version (probably 15.0). Both cc-wrapper and bintools-wrapper as well as some tooling understand this variable (e.g., CMake will pick it up and do the right thing).
That’s why there’s no risk from having multiple SDKs. The hooks resolve them to one SDK always. The issue with propagation caused by Qt (and other libraries) is being avoided by advising against it for all but language implementations that require it. musecore and portaudio can use whatever SDKs that are appropriate for them even if those are different versions. The goal is for packages not to have to care about SDKs except when they need to use a non-default one.

I'm now trying to rebuild musescore without the portaudio override, to see if it works.
SDK versions:
portaudio -> apple-sdk (implicit)
musescore ->  apple-sdk_11 (explicit)
Going to report here once it's done building.
diff --git a/pkgs/applications/audio/musescore/default.nix b/pkgs/applications/audio/musescore/default.nix
index 6f28bf98a1b1..100034b998e4 100644
--- a/pkgs/applications/audio/musescore/default.nix
+++ b/pkgs/applications/audio/musescore/default.nix
@@ -31,24 +31,10 @@
 , qtnetworkauth
 , qttools
 , nixosTests
-, darwin
+, apple-sdk_11
 }:

-let
-  stdenv' = if stdenv.hostPlatform.isDarwin then darwin.apple_sdk_11_0.stdenv else stdenv;
-  # portaudio propagates Darwin frameworks. Rebuild it using the 11.0 stdenv
-  # from Qt and the 11.0 SDK frameworks.
-  portaudio' = if stdenv.hostPlatform.isDarwin then portaudio.override {
-    stdenv = stdenv';
-    inherit (darwin.apple_sdk_11_0.frameworks)
-      AudioUnit
-      AudioToolbox
-      CoreAudio
-      CoreServices
-      Carbon
-    ;
-  } else portaudio;
-in stdenv'.mkDerivation (finalAttrs: {
+stdenv.mkDerivation (finalAttrs: {
   pname = "musescore";
   version = "4.4.2";

@@ -129,7 +115,7 @@ in stdenv'.mkDerivation (finalAttrs: {
     libpulseaudio
     libsndfile
     libvorbis
-    portaudio'
+    portaudio
     portmidi
     flac
     libopusenc
@@ -145,7 +131,7 @@ in stdenv'.mkDerivation (finalAttrs: {
     alsa-lib
     qtwayland
   ] ++ lib.optionals stdenv.hostPlatform.isDarwin [
-    darwin.apple_sdk_11_0.frameworks.Cocoa
+    apple-sdk_11
   ];

   postInstall = ''

Took ages, but I'm happy to report that removing the portaudio override from musescore, and migrating musescore to the apple-sdk pattern, yields a usable build.
@doronbehar please have a look at my latest commit - the portaudio override is not needed anymore
@emilazy @reckenrode thanks for input, really appreciate it


Add AV1 codec

tested and working with my arc a380

Hmm that help message for --codec seems wrong, oops


Introduce Master and Dev branch

As we start having more adoption and need to continue to do dev work, i propose having 2 main branches.  Dev and Master

Master -> Current public release.  This is where any hot fix would have to be integrated with

Release builds are build only from this branch.

Dev -> our constant, stable dev branch where we can continue to accept pull requests.

Nightly builds are build only from this branch.

@crutkas
we may have to choose a different name, since we are currently using dev/username/feature branches and because of that we can't create a dev branch.
We also have a lot of people that forked the project and are already used to the fact that master is actually the working branch.
We can call the current public release release or stable so we can start using this new approach with minimum effort (renaming the dev/*  branches would require to much work, especially because dev/enricogior/telemetry was promoted to be the branch for the release build).

We can also create a release branch for each release and name it  release-x.y.z

I don't want to create a branch for each release.  I'm fine with "Stable" and have our master be the working branch

looks like stable was added, can we close this out?


add logger to windows service shellout

This adds a logger to shellout when spawning a client run in the windows service so that any debug logging regarding process creation/destruction can be captured. This accompanies but can be deployed independently of https://github.com/chef/mixlib-shellout/pull/110

:+1:

/cc @chef/client-windows

:+1:


Can't disable restriction for C-N, C-P

Describe the bug
Cheers!
I have a problem with removing C-N and C-P from the restricted keys. It doesn't work as expected (by me)
Config options
I copy pasted default config table and simply removed C-N and C-P from the list
pastebin
To Reproduce
Steps to reproduce the behavior:
0. have the following remaps: vim.keymap.set("n", "<C-n>", "<cmd>cnext<CR>zz") vim.keymap.set("n", "<C-p>", "<cmd>cprev<CR>zz")

Go to config file for hardtime (e.g. hardtime.lua)
Remove C-N and C-P from the list of restricted_keys
Reload nvim, open telescope live grep window? type something, click C-Q to create qf window from the results.
Spam C-N and see it's blocked after couple clicks

Expected behavior
C-N and C-P should be removed from restricted keys, I should be able to navigate fast in qf window
Screenshots
none
Additional context
I can change value for say restriction_mode and it works after nvim reload, so config is applied.

Hi, the way you config is wrong. To remove <C-N> and <C-P> from restricted_keys you can simply set this:
 restricted_keys = {
    ["<C-N>"] = {},
    ["<C-P>"] = {},
 },

This is documented in README:

If the option is a table with a key = value pair, your value will overwrite the default if the key exists, and the pair will be appended to the default configuration if the key doesn't exist. You can set key = {} to remove the default key-value pair.

This approach make sure user don't have to copy paste all the default config just to make a little change!

Yep, I didn't understand that part, thnx for clarification! now it works.


How to exclude a broken type definition?

pg_v7 if broken for me. How can I ignore it?
My current workaround is to run rm flow-typed/npm/pg_v7.x.x.js || flow-typed create-stub pg (https://github.com/flowtype/flow-typed/wiki/CLI-Commands-and-Flags#create-stub) after running flow-typed install.
Regardless of what version is specified, it creates a stub called pg_vx.x.x.js, when what I want it to do is overwrite pg_v7.x.x.js.
I opened an issue at flow too (https://github.com/facebook/flow/issues/6256) in case it can be done there in the .flowconfig.

Not exactly what you are looking for, but you can replace the conflicting types with any while waiting for an update (that's what I've done on my side).

I think you'd want flow-typed create-stub pg@^7 or whatever.
And yes, you can just any conflicting types until the libdef gets fixed.

Closing for now, LMK if that doesn't address your question.


Update actions/upload-artifact and download-artifact to v4.

Update actions/upload-artifact and download-artifact to v4. v3 is going away and will break builds in Jan.

Ref: https://github.com/signalfx/signalfx-dotnet-tracing?tab=readme-ov-file#warning-deprecation-notice
I hope, I will merge only one PR here, with information that it is fully deprecated.


Chore: 301 generic action handler

Consolidate how we're interacting with the backend into a single action function: actionHandler()
Pass user_guid to the backend via an Authentication header
Remove User data fetching hardcoding in endpoints (Users.object.first())

Nice, and DRY and working a-ok  :)
I suspect the use of  JSON.stringify instead of FormData to submit a form might be problematic for future file upload requirements; so, perhaps implementing a FormData condition in actionHandler function might prove useful for use cases requiring FormData objects.
Something like:
"use server";
import { revalidatePath } from "next/cache";
import { getServerSession } from "next-auth";
import { authOptions } from<EMAIL_ADDRESS>
/**
 * Generic action handler that sends a request to the specified API endpoint
 * and returns the response as a JSON object.
 * @param apiUrl The relative URL of the API endpoint to send the request to.
 * @param method The HTTP method to use for the request (GET, POST, PUT, DELETE, PATCH).
 * @param pathToRevalidate The path of the data to revalidate after the request is complete.
 * @param options Optional data to include in the request body (example: body for POST, PUT, and PATCH requests, overriding cache control).
 * @param options Optional data to include in the request body (example: body for POST, PUT, and PATCH requests, overriding cache control).
 * @returns A Promise that resolves to the JSON response from the API endpoint, or an error object if the request fails.
 */
export async function actionHandler(
  endpoint: string,
  method: "GET" | "POST" | "PUT" | "DELETE" | "PATCH",
  pathToRevalidate: string,
  options: RequestInit = {},
) {
  try {
    const session = await getServerSession(authOptions);

    const defaultOptions: RequestInit = {
      cache: "no-store", // Default cache option
      method,
      headers: new Headers({
        Authorization: JSON.stringify({
          user_guid: session?.user?.user_guid,
        }),
      }),
    };

    let mergedOptions: RequestInit;

    if (method === "POST" && options.body instanceof FormData) {
      // If it's a POST request and the body is FormData, handle it separately
      mergedOptions = {
        ...defaultOptions,
        ...options,
        body: options.body,
      };
    } else {
      // For other methods or non-FormData POST requests, merge options normally
      mergedOptions = {
        ...defaultOptions,
        ...options,
      };
    }

    const response = await fetch(
      `${process.env.API_URL}${endpoint}`,
      mergedOptions,
    );

    if (!response.ok) {
      // Handle HTTP errors, e.g., response.status is not in the 200-299 range
      return { error: `HTTP error! Status: ${response.status}` };
    }

    const data = await response.json();
    revalidatePath(pathToRevalidate);
    return data;
  } catch (error: unknown) {
    // Handle any errors, including network issues
    if (error instanceof Error) {
      // eslint-disable-next-line no-console
      console.error(`An error occurred while fetching ${endpoint}:`, error);
      return {
        // eslint-disable-next-line no-console
        error: `An error occurred while fetching ${endpoint}: ${error.message}`,
      };
    } else {
      // eslint-disable-next-line no-console
      console.error(`An unknown error occurred while fetching ${endpoint}`);
      return {
        error: `An unknown error occurred while fetching ${endpoint}`,
      };
    }
  }
}

Is there a card for CSRF checks to ensure a valid token request.headers.get('Authorization'))["user_guid"] in each request?
Perhaps a Django middleware?

@shon-button good points. I'm going to merge this as-is since it has significant effects on development workflows. Then we can assess if we need follow up cards to address the potential file upload issue & CSRF check


Show the number of labels on an image

Is your feature request related to a problem? Please describe.
there is no way to find out how many labels are there on an image and how many of what kind.
Describe the solution you'd like
show the count of respective types of labels
Describe alternatives you've considered
we can also display using the different color the respective count of the labels.
Additional context
I'm working on this issue currently.

Waiting for the errors showing to get fixed as without those getting fixed the client side wont work.


Update install.md

Algoscan shutdown August 31st, 2023. (https://algoscan.app/curtain) No longer relevant.

 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.


K8SSAND-1733 ⁃ Remove PreserveUnknownFields from K8ssandraCluster CRD

What is missing?
The CassandraDatacenterTemplate type is declared with the kubebuilder annotation //+kubebuilder:pruning:PreserveUnknownFields, which leads to lighter touch OpenAPI validation on the type. This should be removed if possible.
Why do we need it?
Invalid fields can be added to a CR without triggering an error, which leads to a poor user experience when those fields then cause silent reconciliation failures.
See here for an example.
┆Issue is synchronized with this Jira Task by Unito
┆friendlyId: K8SSAND-1733
┆priority: Medium

I have started work to make this change, however it appears to trigger a NPE. I need to dig more deeply into why.

Hi @Miles-Garnsey, any updates on this ticket? Do you need help to get this done?

Hi @Miles-Garnsey, any updates on this ticket? Do you need help to get this done?
Yep, I'd been asking for a while but figured it was de-prioritised. I don't know how to move forward on it given the huge number of failures we get on removing the tag.

I found the culprit: if you remove the annotation, this patch gets applied and removes everything in the config.
We need to remove this patch completely. See the work I'm doing in #703.

I'm not sure why that would cause all of the unit test failures I observed, unit tests don't normally touch the kustomize patches?

The "integration tests" (aka testEnv tests) do, since they install all the CRDs. Other unit tests are OK on my end, which failures are you seeing?

I was seeing failures on my PR over here, the logs are all gone though. I think I should close that PR if you have this working over on your branch :)
https://github.com/k8ssandra/k8ssandra-operator/pull/653

At this point I'm confident you can close your PR. This will be fixed as part of #710, PR to follow shortly.


UNKNOWN_MEMBER_69 CherishOS

Removed Timeout of 90min.

Removed Timeout of 90min.

Why you making changes in server configuration sir?


feat: add IntBoolCast for Entity

Description

add IntBoolCast ('int-bool') for Entity

From https://github.com/codeigniter4/shield/pull/339
Checklist:

[x] Securely signed commits
[x] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[x] User guide updated
[x] Conforms to style guide

@rmileck #5944

Added the docs, and changed the handler name to int-bool.
Because json-arrayalready exists.


Assignment 9: k-fold cross-validation

Hello Prof. Zhe,
In assignment 9, we are using the k-fold cross-validation technique to find the average metrics over k-subsets. So, I feel we should average over the number of folds (k in this case) instead of the number of data points.
Current code:
objs_crossval = objs_crossval / float(len(self.data_y))

My suggestion:
objs_crossval = objs_crossval / float(self.crossval_fold)

This change will cause the metrics taken from 2-folds (default value passed to my_GA) to be averaged correctly.
Please let me know if this change is correct.
Thank you

You are supposed to average the obj even when the sub folds were not divided equally. Take this into consideration and find the correct solution when you sum the objs.

Understood. Let me check again on that.

Thank you Prof. Zhe. I have now performed a weighted average instead of assuming that the length of each fold is uniform.


chore: switch to dist.ipfs.tech

See https://github.com/protocol/bifrost-infra/issues/2018
Closes #54

Merging as this is relatively harmless + we want to whip it with 0.17-rc1 for testing.


FEATURE: Settings option to remove "origin/" from git commands via Terminal Window

via Terminal Window commands for operations like "Checkout Branch (via Terminal)" are including "origin/" in the command when that is not necessary.
git checkout origin/1.8/dev
instead of what I want
git checkout 1.8/dev
Describe the feature you'd like.
GitLen setting to remove origin from those commands?
apologies if this exists already, great product BTW!

Why do you want that removed? It avoid any possible ambiguity that might come from the specific usage.

I'm not the greatest expert in git but when I am using GitLens commands via Terminal in VS Code I get two different things.
PS C:\dev\project1> git checkout origin/1.8/dev
Note: checking out 'origin/1.8/dev'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at acdbda8f Merge pull request #271 from project1/1.8/commit123

versus the same command without "origin/"
PS C:\dev\project1> git checkout 1.8/dev
Switched to branch '1.8/dev'
Your branch is up to date with 'origin/1.8/dev'.

That what happens if the branch 1.8/dev exists.  If it's a new branch, then VS Code thinks the branch name is a GUID unless I take the "origin/" out of the git commands.

Right -- damn, sorry about that. Will get that fixed up!

I am closing this issue because it has not had recent activity and I believe this is behaving as designed.


fix(ui): paginator klass option rename and became optional

[x] I have reviewed my code in "Files changed" tab.
[x] I have re-read the issue and this PR fully resolves it.

btw, do we want to remove mt-3 in activities tab for candidates?

btw, do we want to remove mt-3 in activities tab for candidates?

I moved it to a paginator base class list


Possible Issues with SimpleTheme?

I've got an application which I've updated for A11P6. Currently the app runs with FluentTheme, however, I've tried it with the new SimpleTheme. I'm running Linux/Gnome.
Shown below are two screenshots, the first is with SimpleTheme. The second is the same view, but with the app running FluentTheme (it's current default).
Possible issue with SimpleTheme:
It seems to draw borders around some controls that FluentTheme does not. Ie. around Buttons and ProjectTree.
SimpleTheme:

FluentTheme:

Also, would ask: FluentTheme.DensityStyle seems to do nothing? Is this correct?
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
You can get the source to my app:
https://github.com/kuiperzone/AvantGarde
Main branch currently builds out of the box on A11P6.
Expected behavior
Switching themes should not generally affects apps or show unusual artifacts unless the apps are overriding in some way.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Fedora 37 / Gnome
Version 11P6

Additional context
Add any other context about the problem here.

SimpleTheme and FluentTheme are complete different themes. So differnt outcome is expected. Check also ControlCatalog for reference.
You can do the following to inspect the root cause:

Look up the original styles in source
Use DevTools [F12] to inspect the visual tree. You should be able to see the borders which are bugging you.

One more note: Denistiy styles only adjusts some margins and hights iirc. I also suggest to look it up in source.

SimpleTheme and FluentTheme are complete different themes. So differnt outcome is expected.

Thank you Tim.
Yes, I accept the reasoning here.


Alt (Right) does not adapt to key remapping

Microsoft PowerToys version
0.64.1
Installation method
GitHub
Running as admin
No
Area(s) with issue?
Keyboard Manager
Steps to reproduce
Dear Team,
Thank you for having created this awesome set of tools for an enhanced workflow.
I remapped my Ctrl(Left) to Alt(Left) and my Alt(Left) to Ctrl(Left), so I exchanged both of the keys.
All the functionalities work except the use of Alt(Right) for writing signs such as "@" or "€".
Before the swap, I was able to use Alt(Right) (which is, as you know, a combination of Alt(Right) and Ctrl(Left)) to write above signs without having to press any other key. After the swap, I cannot only use Alt(Right) to write the signs. Instead, I additionally need to press a Ctrl key to write the signs.
I am no expert but it seems to me that the system does not recognize the key change. It uses the physical key that was before mapped to Ctrl(Left) but cannot use this key because it is now mapped to Alt(Left). So it reads Alt(Right) + Alt(Left) and in this consequence does not write "@" or "€" but gives an error.
Would you be able to tell the system to adapt the other keys to this change so that Alt(Right) accesses Alt(right) + Ctrl(Left) instead of Alt(Right) + Alt(Left).
I would be very grateful if this could be changed.
Best regards,
R. Reuther
✔ Expected Behavior
I expect the use of Alt(Right) to always find the correct Ctrl(Left) key, even if the physical Ctrl(Left) key is priorly remapped to another virtual key.
❌ Actual Behavior
After remapping Ctrl(Left) to Alt(Left) and Alt(Left) to Ctrl(Left) the Alt(Right) key does not find the Ctrl(Left) key anymore and, in that consequence, cannot be used to write signs such as "@" or "€".
Other Software
No response

At first, this is expected by design. You remap the left Ctrl and the system applies that.

It uses the physical key that was before mapped to Ctrl(Left) but cannot use this key because it is now mapped to Alt(Left). So it reads Alt(Right) + Alt(Left) and in this consequence does not write "@" or "€" but gives an error.

..sums it up nicely. So, your request/expectation is basically to circumvent the basic logic. @crutkas would #14055 be a solution?
Otherwise, Maybe MS Keyboard an mouse Center or AutoHotkey can help you out.


FEAT: Disable Gradio Telemetry

Pull requests are disabled but see here:
https://github.com/arch-btw/inference/pull/1

Thanks for your feedback! Is there any problem with the pull request?

@arch-btw Thanks for your contribution! 🤗 I've created a pull request according to your changes #299.
Please let us know more details about pull requests being disabled.


238 contact area of three roll passes

Changed the Contact area calculation so that its more representative for three-roll rolling.
Also enhanced the contact_lines property to handle ThreeRollPasses correctly, as this is also required for coming changes in the Zouhar-Package.
Currently there is also a factor 0.999 necessary to correctly calculate the in_profile local_width - probably due to small numerical deviations.

Reason for these empty intersection lines (not really empty but just 1 instead of two points) is as @axtimhaus said that the bottom polygon lines aren't perfectly horizontal.
The adjusted test shows, that these Differences are only in the 10e-18 range though and of the profiles tested also only occur on the flat and oval - in_profile and not on the round one:
============================== 8 passed in 3.03s ==============================
PASSED [ 12%]Two lowest y-coords: [-0.013160254037844385313626993650, -0.013160254037844381844180041696] 	 diff: 0.000000000000000003469446951954
PASSED [ 25%]Two lowest y-coords: [-0.010033594466608928424955493597, -0.010033594466608926690232017620] 	 diff: 0.000000000000000001734723475977
PASSED [ 37%]Two lowest y-coords: [-0.009322056340506710073179341691, -0.009322056340506710073179341691] 	 diff: 0.000000000000000000000000000000
PASSED [ 50%]Two lowest y-coords: [-0.009322056340506710073179341691, -0.009322056340506710073179341691] 	 diff: 0.000000000000000000000000000000
PASSED [ 62%]Two lowest y-coords: [-0.009322056340506710073179341691, -0.009322056340506710073179341691] 	 diff: 0.000000000000000000000000000000
PASSED [ 75%]Two lowest y-coords: [-0.010033594466608928424955493597, -0.010033594466608926690232017620] 	 diff: 0.000000000000000001734723475977
PASSED [ 87%]Two lowest y-coords: [-0.009322056340506710073179341691, -0.009322056340506710073179341691] 	 diff: 0.000000000000000000000000000000
PASSED [100%]Two lowest y-coords: [-0.010033594466608928424955493597, -0.010033594466608926690232017620] 	 diff: 0.000000000000000001734723475977

The change to the local width/ heigth makes it work, but is still somewhat hacky - but i cant find where this numerical error actually stems from.


String protection not working when build.minify is set to true

Describe the bug
The protectedStrings option fails to find the wanted strings when build.minify is true. Removing minify: true fixes it.
Here's my electron.vite.config.js:
export default defineConfig({
  main: {
    plugins: [
      externalizeDepsPlugin(),
      bytecodePlugin({ protectedStrings: ['ABCDE', 'FGHIJ'] }),
    ],
    build: {
      rollupOptions: {
        input: {
          index: resolve(__dirname, 'src/main/index.js'),
          index2: resolve(__dirname, 'src/main/index2.js'),
          index3: resolve(__dirname, 'src/main/index3.js')
        }
      },
      minify: true //works without this
    }
  },
  preload: {
    plugins: [externalizeDepsPlugin(), bytecodePlugin()],
    build: {
      minify: true
    }
  },
  renderer: {
    resolve: {
      alias: {
        '@renderer': resolve('src/renderer/src')
      }
    },
    plugins: [react()],
    build: {
      minify: true
    }
  }
})

Enforcing the plugin ordering does not seem to help either:
plugins: [
      {
        ...bytecodePlugin({
          protectedStrings: ['ABCDE', 'FGHIJ']
        }),
        enforce: 'pre'
      },
      externalizeDepsPlugin()
    ],

Electron-Vite Version
2.0.0
Electron Version
28.2.4
Vite Version
5.1.4
Validations

[X] Follow the Code of Conduct.
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

@therealartti Thank you for your feedback. When minification is enabled, string protection has no effect. This is because string protection is based on character codes. However, modern minification tools, whether esbuild or terser, will restore the converted character codes, causing the protection to fail. electron-vite will throw a warning. In fact, minification has little effect on reducing bytecode size, so it is recommended not to enable minification when protecting strings.


IndexError: list index out of range

File "../guided_diffusion/unet.py", line 775, in forward
uemb, cal = self.highway_forward(c, [hs[3],hs[6],hs[9],hs[12]])
IndexError: list index out of range
how can I solve it?

check this issue: https://github.com/WuJunde/MedSegDiff/issues/8


Renamed brew tap to aspnet/dnx

Should be aspnet/dnx since the Homebrew repo was renamed from homebrew-k to homebrew-dnx

Hi @henkmollema, I'm your friendly neighborhood Microsoft Open Technologies, Inc. Pull Request Bot (You can call me MSOTBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft Open Technologies, Inc. and real humans are currently evaluating your PR.
TTYL, MSOTBOT;

@glennc @BrennanConroy do you know what this is supposed to be?

@moozzyk too.

aspnet/dnx is the new one, Glenn must have mistyped :D

This does look right to me though.

Merged https://github.com/aspnet/Home/pull/492 instead because it had some other small fixes.
@BrennanConroy got it :smile:

:+1:


Manual Eclipse build should kick off ant build

Since ant all is the canonical way to build the project, it seemed like it would be handy to be able to do that from within Eclipse. I added a launch configuration that will do just that upon a manual build. I didn't configure anything for automatic builds, because that seemed pretty slow.

The ant build is fully deprecated. We moved to gradle. This PR is not really relevant anymore...


fix/missing li

fix: wrap pagination links in li
chore: bump version

Description
A concise description of the proposed changes and/or a reference to the Github
issue.
Checklist

[ ] I added tests that cover my proposed changes.
[ ] I updated the documentation related to my changes (if applicable).

coverage: 98.99% (-1.0%) from 100.0% when pulling 65458e0462aa390e35aa42ee543944af45013114 on fix/missing-li into 797d9a6d202464f0f7342f7e404c7930c16a604d on main.


Log 'requestId' for intra-cluster log correlation

Well, first, thanks for this plugin, tried this locally in my dev environment and worked a treat, we're definitely going to use this in production.
However, what would be extra-awesome is if this plugin could allow the requesting client application to pass on a 'requestId' that correlates any ES logs with a unique requestID handle.  What we do is when a user interacts with our site, a unique requestId token is generated and bundled as a threadLocal, and use log4j's MDC concept so that all user logs for that requestId have that requestId in them, allowing very simple filtering to find all logs for a given request.
Now when a client app interacts with something like ES, it would be great for the ES api to be able to pass on this requestID in the headers so that the logging on the ES side also has the same thing.  This way all ES queries/interactions for a given user base action could be logged and correlated together.
ES has the X-Opaque-Id header which I had planned to look at as part of trying this myself, see https://github.com/elasticsearch/elasticsearch/issues/1202 but maybe this plugin is the best place to do it.
For reference, we're building up to something like Google's Dapper (http://research.google.com/pubs/pub36356.html) where by in a distributed system unique requests can be identified (we're sticking with a single 'span' concept though at this stage).
Love to have your comments, and thanks again for a great plugin.

As a workaround, you can repeat this id as a parameter on URL and it will be logged.


Remove bringup: true from web tests.

We should make this a blocking CI step ASAP.

We should make this a blocking CI step ASAP.

Blocking presubumits or blocking Mq?

Blocking presubmits, not MQ. MQ has the linux_web_engine builder which just builds the web engine artifacts. This runs the tests, but we should make sure that these actually run in presubmit and postsubmit. It's an important signal, especially for dart SDK rolls and skia rolls.

I believe we're deciding that all presubmits must pass; so we'll have to update Cocoon to record multiple stages before completing the MQG. Good thing for the new year.


Flow garden test

Flow garden testing script.

@AboudyKreidieh we will simultaneously write a test for this and visualizer_rllib; I will set that up.

@kanaadp @eugenevinitsky do you guys still want to merge this?

going to close for now. if we want to merge this feel free to reopen


Add desktop-full quickstart

Follow-up to:

https://github.com/ros-infrastructure/rep/pull/334
https://github.com/ros2/variants/pull/34

Now that ROS 2 has the desktop-full "batteries included" variant, users can easily see a working robot right when they first install ROS.

@Mergifyio backport rolling


impl From<Cow<'_, str>> for CompactStr

Super small; just something I noticed when I went to benchmark replacing smartstring with compact_str in one of my projects :)
Kudos on the awesome crate!  It turns out that, at least in imdb_async, I deal with a lot of strings that are 24 bytes in length.  Average performance seems to be approx. 10% faster with compact_str for that crate, presumably a result of increasing the allocation threshold from 23 to 24 bytes.

Thanks for the PR!
I'd be curious to hear how the migration from smartstring to compact_str was. I know there are still some APIs missing from CompactStr that are present in String.

For my use cases, at least, it's just been a matter of (along with adding this From impl) s/smartstring::alias::String/compact_str::CompactStr/g


Merge Filter and AdaptiveFilter enums in the public API

Right now the API for encoding is kinda awkward in that we have a separate Filter and AdaptiveFilter enums that are separately applied to the encoder, and the API does not communicate that the Filter setting is ignored if AdaptiveFilter::Adaptive is also set.
By contrast, the image crate exposes the Filter as a single enum already which lists all the individual filters plus Filter::Adaptive, which I think makes for a much nicer API.
There may be cases where adaptive filtering is not an option (e.g. in decoding or in per-row APIs), in which case we can keep that as a RowFilter or some such and ideally make that enum private.

I'm happy to try and apply this change unless somebody objects, since we have some semver-breaking API changes queued already.

I've been wanting to make this change. Would also be nice to make the enum non-exhaustive so other adaptive filter strategies could be added in the future.
Could you make the change is the same semver-breaking PR? That way we can look at the entire new API design in one place.


Add robots.txt to reject spider?

Just found that, the portal of some huginn instances can be found in Google, and the invitation code is the default try-huginn, that I successfully registered accounts on several of them, which is not very safe.
Although it is user's responsibility, that don't post huginn's URL address in public, and change invitation code, however, is it a good idea that considering adding a robot.txt option in .env, and the default state of which is reject spider, unless the user change it proactively?
Just an accidental discovery and immature little idea.

We already have a robots.txt in the public folder. It might be a good idea to uncomment the rule to prevent the indexing.


Automating Team Selection when using Spaceship/Sigh for an Apple Developer Account having access to Multiple Development Teams.

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I have read the Contribution Guidelines

Issue Description
Using sigh prompts for entering Apple ID (and password for 1st time), logging into which it waits for user input to select a Team on the basis of Numbered Bullets. I wanted to ask if we can automate this step in any possible way so that we don't have to provide a user input on run time when we are using the command.
✅ sigh output ✅
|           Summary for sigh 2.48.0           |
| adhoc                               | false |
| development                         | false |
| skip_install                        | false |
| force                               | false |
| ignore_profiles_with_different_name | false |
| skip_fetch_profiles                 | false |
| skip_certificate_verification       | false |
| platform                            | ios   |
[16:11:16]: To not be asked about this value, you can specify it using 'username'
[16:11:16]: Your Apple ID Username<EMAIL_ADDRESS>[16:11:23]: Starting login with user<EMAIL_ADDRESS>Multiple teams found on the Developer Portal, please enter the number of the team you want to use:

1AB123456 "TEAM-A" (In-House)
2AB123456 "TEAM-B" (In-House)
3AB123456 "TEAM-C" (In-House)
4AB123456 "TEAM-D" (Company/Organization)
5AB123456 "TEAM-E" (In-House)
6AB123456 "TEAM-F" (In-House)

Apart from that, when we use Spaceship commands inside irb same happens. We have to use fixed defined commands. Writing these commands require an interaction with the system. We want to automate this process. So is there any possible way of automating this?
 What is your preference for how this would be automated? Like would it just pick the first team, or would you specify the team in advance when calling into sigh from your Fastfile?
Thanks so much for writing in! 🚀

I don't know much about sigh, just that we can't pass TEAM ID as an argument with the command. If that can be done, then it would be great!
For spaceship though we have commands like,

<EMAIL_ADDRESS>"password")

where username and password can be passed as parameter, but for selecting Team we need to run the following kind of command which waits for a user interaction and doesn't process unless an input is provided,

Spaceship::Portal.select_team

If possible, I wanted to have the above option to have Team ID as passing parameter like,

Spaceship::Portal.select_team("1AB123456")

This could enable us to cater to multiple teams when we run our Continuous Integration setup where before building we can get the latest Provisioning Profile automatically before building the Xcode Project.
Reference: https://github.com/fastlane/fastlane/blob/master/spaceship/docs/DeveloperPortal.md
Thanks !

You can actually do this now with spaceship. If you look at this file:
https://github.com/fastlane/fastlane/blob/master/spaceship/lib/spaceship/portal/portal_client.rb
At line 59, there is a method team_id= that you can use to set the team_id.
So, you'd do this:
<EMAIL_ADDRESS>"password")
Spaceship::Portal.team_id = "1AB123456"

Is that what you are looking for?

@rishijoshi-ibm One thing you could do is to provide sigh an account that only has access to one team so that you don't get prompted. This is one of many solutions that you can do but I see this as a best practice in most situations (not sure about your application) as its a nice way to separate concerns and making sure that one account only has permissions to exactly what it needs.

Hey @rishijoshi-ibm, you can also specify your team ID with the FASTLANE_ITC_TEAM_ID environment variable. If you dont want to store this in your shell environment directly, you can also put it in a .env file in your project directory and just make sure to call require 'dotenv'; Dotenv.load at the top of your Fastfile. You can also prefix your fastlane command like so: FASTLANE_ITC_TEAM_ID=XXXXX fastlane ....
Let us know if that is helpful! 🐙 🚀

@mpirri I'm exactly looking for something like this. I put the commands you shared in a Ruby file and ran the Ruby file, but getting the following error:
in '<main>': undefined method 'team_id=' for Spaceship::Portal:Module (NoMethodError)
Fastlane Version shows:
fastlane 2.49.0
@joshdholtz I don't think that would be feasible. I had thought of that option earlier, but then I can't have separate account for each team. Still, thanks for writing in.
@ohayon Thanks for writing in. I'm currently trying to do something which doesn't depend on Fastfile. Some other solution like running the Spaceship commands and calling them via a Ruby file. Though, will try your solution once and let you know if it helps.

@rishijoshi-ibm Whoop!  I meant to type this instead:
Spaceship::Portal.client.team_id = "1AB123456"

Sorry about the confusion! 🚀

@mpirri Thanks a lot..!! This really helped me. I'm able to use this method as expected.

@mpirri Should this work with Spaceship::Tunes.client.team_id = "1AB123456" as well?


[Service Connector] az containerapp connection create: Enable Key Vault Reference in Container Apps

Related command

Description

This PR is to enable key vault reference during creating a connection for container apps with service connector.
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Service Connector


Labels for deeplab

I am wondering what labels were used for the deeplab model?
It outputs some 0 but mostly 9 when no object is in the image which corresponds to background and chair in the official deeplab model.
Were different labels used for this model?

@roflcoopter
I used the following items.
https://github.com/tensorflow/models/blob/master/research/deeplab/g3doc/quantize.md


Add: The Official Raspberry Pi Project Book

A 204 pages magazine about Raspberry Pi made by the makers of the official Raspberry pi magazine.
Note from impressum page 3: Except where otherwise noted, content in this magazine is licensed under a Creative Commons Attribution-NonCommercial-ShareAlike 3.0 Unported (CC BY-NC-SA 3.0). ISSN: 2051-9982.

As noted by Travis, for links directly to a pdf you should add the (PDF) notation.
This is a great addition, thanks!

@eshellman oh thanks, just read that in the contributing guide - PR update.


FTUE  Story: Update imagery to include new logo everywhere

Bug Description

Update the FTUE page to show the new Web Stories logo when showing the Gutenberg block.
Right now, it has the old web stories logo. Need to switch to the new one.
Expected Behaviour

Steps to Reproduce

Screenshots

Additional Context

Plugin Version:
Operating System:
Browser:

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

Have you cleared the cache? We‘ve updated this to the new colored logo already.

Screenshot of 1.0.0-rc.1:

Sorry, this is for @samitron7 in the FTUE story. Just filed a quick bug during the review meeting. I'll update to specify.

@o-fernandez Looking at https://stories-new-wordpress-amp.pantheonsite.io/stories/tips-to-make-the-most-of-web-stories/ I don't see the old logo anywhere. Was this resolved? This is kinda blocking work on the FTUE story integration into the plugin, so would be good to know.

Looks fixed! There was one item (gutenberg block screencast) that was not yet resolved, but it looks fixed now. :) We can close this out.


Is support different request instances retry different count?

Is support different request instances retry different count?

eg:
The request instance "https://mock.codes/xxx" retry 2 count.
Another request instance "https://mock.codes/yyy" retry 3 count.

@yfming93, please, put your sample code here.
If you want to have different count on different requests you have to create multiple Dio instances. Each of them will have it's own RetryInterceptor. And then you choose which dio instance to use

It should be possible, as I remember. Just pass retry count for a specific request.


AppIcon not optimized for Retina Display

System tray icon is low quality.

Yea, I tried fixing that but could only get it to use the low-res AppIcon.

I tested a way, and it worked! If you turn the WakaTime image into an SF Symbol and import it as Symbol Image Set, then it will display properly.
Here I used an asset from my project for testing.

#26

Released in v0.1.0 just now:
https://github.com/wakatime/macos-wakatime/releases/tag/v0.1.0-alpha.1


moving from wiki to real repo

as i already mentioned in the #2  maybe we should consider moving the wiki to the real repo,
then the starting "README.md" could be just the sidebar of the wiki.
what do you thing about this @holzkohlengrill

We would not have any permanent sidebar in this case which is in my opinion important (mainly usability and convenience reasons).
Another option would be using a plain repo in combination with a github pages supported framework like Jekyll. That's for sure a matter of taste to use it or not.

Closed; Currently no ideal solution.


Issue with the accuracy of the data being retrieved

1.when we use the connector to retrieve the data from the portal for the compliance state of resources or policies we end up having issues with the number and state in the portal not matching in the portal we have 165 resources compliant in the portal but in power bi it only shows 152 and we can see that power bi is trying to pull all the data it's just missing the 13 resources CC Applied Policies ---- missing resources

Power BI
2. when pulling tag meta data to one of our dashboards we see the resource group as tagged in the portal but not in the power bi dashboard

3. we also see the same issue as number 2 when we try to look at resources where it's tagged in the portal but missing in the power bi dashboard / report the resource shows up but in the tag box it says missing

Hi @mdelpnc01, Thank you for reaching out. Regarding your issue, could please confirm that you are leveraging the solution we are providing and not only part of it? Thank you!

Hi @mdelpnc01 we're not allowed to support customized versions, if you have problems with the published version let us know.

This isn’t customized it is just the original ccodashboard file from github nothing was added or subtracted from the .mez file it is all straight from github
Thank you,
Matt Delisio
Cloud Hosting Assurance, Developer
Security and Enterprise Response
Enterprise Technology & Security
PNC Financial Services Group
From: Jordi Suñé @.>
Sent: Tuesday, May 17, 2022 12:57 PM
To: Azure/ccodashboard @.>
Cc: Delisio, Matthew @.>; Mention @.>
Subject: EXTERNAL: Re: [Azure/ccodashboard] Issue with the accuracy of the data being retrieved (Issue #227)
** This email has been received from outside the organization – Think before clicking on links, opening attachments, or responding. **
Hi @mdelpnc01https://urldefense.com/v3/__https:/github.com/mdelpnc01__;!!INViTUAdUeDF!zXQcPC_hca-aV7Og-j3__X5fWD_2Mc5sR5SJ6DomTNJXmIGognLMcqMq-dof9rkVtem5_WYeySaNkWloGW3EgS8DtD0$ we're not allowed to support customized versions, if you have problems with the published version let us know.
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/Azure/ccodashboard/issues/227*issuecomment-1129102506__;Iw!!INViTUAdUeDF!zXQcPC_hca-aV7Og-j3__X5fWD_2Mc5sR5SJ6DomTNJXmIGognLMcqMq-dof9rkVtem5_WYeySaNkWloGW3ELb-oaKk$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/AVISOR35K5ZQVHMXOOMBZXDVKPFT7ANCNFSM5RI53JMA__;!!INViTUAdUeDF!zXQcPC_hca-aV7Og-j3__X5fWD_2Mc5sR5SJ6DomTNJXmIGognLMcqMq-dof9rkVtem5_WYeySaNkWloGW3EkVAp5-U$.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>The contents of this email are the property of PNC. If it was not addressed to you, you have no legal right to read it. If you think you received it in error, please notify the sender. Do not forward or copy without permission of the sender. This message may be considered a commercial electronic message under Canadian law or this message may contain an advertisement of a product or service and thus may constitute a commercial electronic mail message under US law. You may unsubscribe at any time from receiving commercial electronic messages from PNC at https://cloud.e.pnc.com/globalunsub
PNC, 249 Fifth Avenue, Pittsburgh, PA 15222; pnc.com


Add menu to open Folder/Finder to log files

Signed-off-by: Wayne Witzel III<EMAIL_ADDRESS>Which issue(s) this PR fixes

Fixes #1871

I agree, open a new issue for that. That behavior has no impact on opening the folder.

I agree, open a new issue for that. That behavior has no impact on opening the folder.


[backport] haskell shellFor: Fix hoogle

(cherry picked from commit 1c07ee79255bb3fe8a7eb382cd5a8348e0b85893)

Motivation for this change
backport of #80845 to fix #83783 in stable release-20.03 as well.
fixes #83783
Things done

rebuilt my nix-shell environment based on this PR
tested my code in the new environment
observed that during build of the environment, a hoogle DB gets created
successfully tried the hoogle command

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

ping @cdepillabout

Looks good, thanks!


updating commercial listing for MetricFire

Signed-off-by: Lindsey Rogerson<EMAIL_ADDRESS>

Thanks!


Trying to get in touch regarding a security issue

Hello 👋
I run a security community that finds and fixes vulnerabilities in OSS. A researcher (@jeffreygaor) has found a potential issue, which I would be eager to share with you.
Could you add a SECURITY.md file with an e-mail address for me to send further details to? GitHub recommends a security policy to ensure issues are responsibly disclosed, and it would help direct researchers in the future.
Looking forward to hearing from you 👍
(cc @huntr-helper)

Hi Pavlos, thanks for your help mate 🙏


Fix warning in clang 3.5 (-Wabsolute-value)

Travis builds with clang 3.4, so the warning didn't appear there.

Thanks!


Remove lodash dependency

While working on adding a new grammar I wanted to use the usual tree-viz and I noticed it was failing. I looked deeper and saw that it was failing because it couldn't find lodash.

IMO adding lodash as a dependency is not right, as it forces consumers of this library to have this dependency, and all that just for a trimEnd function.
In this PR I'm removing lodash and replacing it with a custom trimEnd implementation.

I am not a big fan of reinventing the wheel. Obviously importing whole lodash just for trimEnd isn't great, but what about using https://www.npmjs.com/package/lodash.trimend?

I really don't think we need any dependency for 3 lines of code.

Next method we wanna use we say the same :D But also fine for me right now - nevertheless using dependencies shouldn't be a great show stopper.
Though, it's fine for me to not use any dependency for this.


fix: Some editting bugs in Lu editor

fixes #2141

Incorrect syntax highlight for entity definition. For example:

Wrong place of inserting '-' in auto-completion of utterance

trigger error of lebeling experience

Can you record a gif or screenshot? that would be very nice.

Can you record a gif or screenshot? that would be very nice.

Sure. You can see the gif now.


github workflow: enable workflow approval

Allow to run GitHub workflows for Pull Requests that have the ok-to-test label.
Fixes #116

@jmhbnz done


Missing dll

Compiling JavaScriptViewEngine for .NETCoreApp,Version=v1.0
Compilation succeeded.
0 Warning(s)
0 Error(s)
Time elapsed 00:00:04.3719335
Expect required library hostpolicy.dll to be present in [C:\Projects\JavaScriptV
iewEngine\src\JavaScriptViewEngine\bin\Debug\netcoreapp1.0]

This may be because of an invalid .NET Core FX configuration in the directory.

What is wrong here?
Is this supposed to run under rc2?

This is an RC2 project. I'm not entirely sure what is wrong your setup. I have ran this repo on multiple machines with a clean configuration. I suspect your issue is a dotnet-cli one.

Well... checked out and run. Mac OSX, dotnet 1.0.0-preview1-002702

Any update on this?

You can close it. I'm no longer on this.


Add in a guide to making a custom debian package.

Signed-off-by: Chris Lalancette<EMAIL_ADDRESS>Based on the instructions in https://discourse.ros.org/t/how-to-create-debian-package-for-ros-2/19673/5

This pull request has been mentioned on ROS Discourse. There might be relevant details there:
https://discourse.ros.org/t/how-to-create-debian-package-for-ros-2/19673/6

@mergifyio backport galactic foxy dashing


Adding an option to compute title for each screen

This allows things like opening a detail view (in master-detail)
with the detail navbar title populated with the name of the item the user is
currently editing.
For example:
getTitle={(comp)=> comp.item ? comp.item.get('title') : "" }

Once the scene is navigated to, the navbar will use this specific
getTitle function, which will return a different title, based on
the navigation parameters that were used in order to navigate to it.

Sorry, it has conflicts now, please re-submit...

Sure, should be good now.


BUG: Check that result is not None before indexing

Fix #1178

:+1:

Coverage increased (+0.02%) to 79.3% when pulling ce7c5b513849f5eba6e7a60d9ee00cf3bb023ad2 on adamrp:verify_code-issue-1178 into da407f650ce67619aaabf61d620e382f39621c22 on biocore:master.

Should be good to go


Not recomputing in Jupyter notebook with bokeh 0.12

I am going through the examples folder, and noticed some weird behavior. When running the nyc_taxi example, I am not getting any dynamic recomputation when I zoom into the plot after the header "10-million-point datashaded plots: interactive". It zooms fine, there are no errors that I can find, but it never re-computes to get finer resolution.
This problem is occurring when I run using a notebook server on a remote computer. I also installed bokeh and datashader on my laptop and everything works fine there. It turns out my laptop has bokeh 0.11.1, while the remote computer has bokeh 0.12.0. In both cases, I have datashader version 0.3.0
I downgraded bokeh on the remote computer, and now it works correctly.

Yes, we are preparing a new release, hopefully by tomorrow, to address this incompatibility.  Meanwhile, the git version should have the correct behavior with Bokeh 0.12.

@kgullikson88  conda install -c bokeh datashader and you should get the updated version which plays nicer with bokeh 0.12.0


vcpkg-tool: Simplify installtion on nanoserver like

Describe the bug
Installation of vcpkg prove to be quite difficult on nanoserver like system.
Environment

OS: Windows nanoserver (eg. mcr.microsoft.com/dotnet/sdk:7.0)
Compiler: none

To Reproduce
Steps to reproduce the behavior:
> cat Dockerfile
# escape=`
FROM mcr.microsoft.com/dotnet/sdk:7.0

WORKDIR C:/dev/

ARG VCPKG_VERSION=2023.07.21
ADD https://github.com/microsoft/vcpkg/archive/refs/tags/$VCPKG_VERSION.zip C:\TEMP\vcpkg.zip

RUN tar -xf C:\TEMP\vcpkg.zip `
    && ren vcpkg-%VCPKG_VERSION% vcpkg `
    && del /f /q c:\TEMP\*.*;

SHELL ["pwsh", "-Command", "$ErrorActionPreference = 'Stop'; $ProgressPreference = 'SilentlyContinue';"]

RUN .\vcpkg\bootstrap-vcpkg.bat -disableMetrics;

Gives:
> docker build .
Sending build context to Docker daemon  2.048kB
Step 1/7 : FROM mcr.microsoft.com/dotnet/sdk:7.0
 ---> 43067ec72d68
Step 2/7 : WORKDIR C:/dev/
 ---> Running in 44765abf2793
Removing intermediate container 44765abf2793
 ---> 863d49ebd1c4
Step 3/7 : ARG VCPKG_VERSION=2023.07.21
 ---> Running in 2c95eca3b986
Removing intermediate container 2c95eca3b986
 ---> 1b6f0b11bef4
Step 4/7 : ADD https://github.com/microsoft/vcpkg/archive/refs/tags/$VCPKG_VERSION.zip C:\TEMP\vcpkg.zip
Downloading  7.571MB

 ---> c957800f783c
Step 5/7 : RUN tar -xf C:\TEMP\vcpkg.zip     && ren vcpkg-%VCPKG_VERSION% vcpkg     && del /f /q c:\TEMP\*.*;
 ---> Running in e2d020f011a6
Removing intermediate container e2d020f011a6
 ---> d5511b253da3
Step 6/7 : SHELL ["pwsh", "-Command", "$ErrorActionPreference = 'Stop'; $ProgressPreference = 'SilentlyContinue';"]
 ---> Running in 2c4d887da6a1
Removing intermediate container 2c4d887da6a1
 ---> 14b149981886
Step 7/7 : RUN .\vcpkg\bootstrap-vcpkg.bat -disableMetrics;
 ---> Running in 31e9ab80d53d
'powershell.exe' is not recognized as an internal or external command,
operable program or batch file.

Since:
> docker run --rm mcr.microsoft.com/dotnet/sdk:7.0 powershell --version
docker: Error response from daemon: container 615360a3654bbe128045fa96770d163dc6c9079b9e86e6d42357eb9e6ee6d10e encountered an error during hcs::System::CreateProcess: failure in a Windows system call: The system cannot find the file specified. (0x2)
[Event Detail:  Provider: 00000000-0000-0000-0000-000000000000]
[Event Detail:  Provider: 00000000-0000-0000-0000-000000000000]
[Event Detail: onecore\vm\compute\management\orchestration\vmhostedcontainer\processmanagement.cpp(173)\vmcomputeagent.exe!00007FF606A8A1DB: (caller: 00007FF606A3E19A) Exception(1) tid(390) 80070002 The system cannot find the file specified.
    CallContext:[\Bridge_ProcessMessage\VmHostedContainer_ExecuteProcess]
 Provider: 00000000-0000-0000-0000-000000000000].

Expected behavior
Installation should gracefully success since pwsh is available.
Failure logs
See above
I could not determine what is missing for tls12-download.exe to execute on such system.
Additional context
Current workaround:
# escape=`
FROM mcr.microsoft.com/dotnet/sdk:7.0

# Cannot update path, need hackish solution:
# https://github.com/PowerShell/PowerShell-Docker/issues/236
ENV PATH="C:\Windows\system32;C:\Program Files\dotnet;C:\Program Files\powershell;C:\Program Files\MinGit\cmd;C:\dev\vcpkg"

ARG VCPKG_VERSION=2023.07.21
# https://github.com/microsoft/vcpkg/blob/2023.07.21/scripts/bootstrap.ps1
# Naming convention taken from recent https://github.com/microsoft/vcpkg/blob/2023.11.20/scripts/bootstrap.ps1
ARG VCPKG_TOOL_RELEASE_TAG=2023-07-19

WORKDIR C:/dev/

RUN `
    pwsh -command "Invoke-WebRequest -OutFile C:\dev\vcpkg.zip https://github.com/microsoft/vcpkg/archive/refs/tags/%VCPKG_VERSION%.zip" `
    # Extract:
    && tar -xf C:\dev\vcpkg.zip `
    && del /q c:\dev\vcpkg.zip `
    # Remove version from folder name:
    && ren vcpkg-%VCPKG_VERSION% vcpkg `
    # Cannot use the default `curl` does not seems to download anything (302 issue?)
    # && curl -o vcpkg/vcpkg.exe https://github.com/microsoft/vcpkg-tool/releases/download/%VCPKG_TOOL_RELEASE_TAG%/vcpkg.exe
    # Same goes for the tool `tls12-download.exe` seems to be missing a dll ?
    # Use Invoke-WebRequest  instead:
    && pwsh -command "Invoke-WebRequest -OutFile vcpkg/vcpkg.exe https://github.com/microsoft/vcpkg-tool/releases/download/%VCPKG_TOOL_RELEASE_TAG%/vcpkg.exe";

@malaterre

'powershell.exe' is not recognized as an internal or external command,
operable program or batch file.

Is the powershell in the docker available?


who do it have 20 detects and in the old version it had 10 ?

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

its really depending on the features you choose. if you want a fud stealer, we recommend you, wait for rosev2

also, rose is slightly getting bigger, so also getting registered to virus banks, but i am rewriting the main part of the actual stealer right now, so they might go down soon.

probably because of the xmr miner the detections raise


SliderMarks::Custom is unusable

Hello,
I'm not sure if that's intended, but there is no way to create an instance of SliderMarks::Custom because it requires a Vec<SliderMark> and SliderMark has only private fields without any other way to instantiate it. I had a situation where the automatically generated slider marks were too close together and I only wanted to render every second mark, and I had to result to hiding every second slider mark with CSS, which isn't ideal:
.marks div:nth-child(even) {
  display: none;
}

If you want me to, I'll create a PR with a simple constructor for SliderMark.

This is perfect, thank you! I don't think a new function is required. It's already a fairly specialized use case, and it's not a particularly difficult struct to create anyway.
Also, thank you in general for building leptonic. I wouldn't have been able to build a nice looking website without it!


Research Tendermint / Cosmos more

Resources
Tendermint website- https://tendermint.com/
Tendermint GitHub - https://github.com/tendermint/tendermint
Cosmos website - https://cosmos.network/
Cosmos GitHub - https://github.com/cosmos
Cosmos SDK - https://github.com/cosmos/cosmos-sdk
Cosmos roadmap - https://cosmos.network/roadmap
Discoveries

Jack Zampolin is now at Tendermint/Cosmos

Complete.


Speed up .dt accessor by preserving Index objects.

[ ] Closes #xxxx
[ ] Tests added
[ ] User visible changes (including notable bug fixes) are documented in whats-new.rst
[ ] New functions/methods are listed in api.rst

Yes, 20-30% on my machine:
       before           after         ratio
     [91f14c9b]       [1eb74149]
     <v2023.04.2>       <speedup-dt-accesor>
-     1.17±0.04ms      1.02±0.04ms     0.87  accessors.DateTimeAccessor.time_dayofyear('standard')
-     1.25±0.07ms         976±30μs     0.78  accessors.DateTimeAccessor.time_year('standard')
-      3.90±0.1ms      2.68±0.05ms     0.69  accessors.DateTimeAccessor.time_year('noleap')
-     4.75±0.07ms      3.25±0.04ms     0.68  accessors.DateTimeAccessor.time_dayofyear('noleap')


Overriding plugin config with theme.colors: {...} is not working

Describe the bug
I am working on my tailwind.config.js file that incorporates the flowbite plugin.
Using the colors object in  tailwind.config.js, I was expecting usages of gray-300 in my app to be the value I provided, but instead it is picked up as the value in the plugin.
To Reproduce

Copy the config below into a project using the tailwind.css dependency
Use the text-gray-300 class on an element
Observe the computed value for the style
Here is the unpkg of the plugin<EMAIL_ADDRESS>
const defaultTheme = require('tailwindcss/defaultTheme');

/** @type {import('tailwindcss/tailwind-config').TailwindConfig} */
module.exports = {
  content: ['./src/**/*.{js,ts,jsx,tsx}', './node_modules/flowbite/**/*.js'],
  theme: {
    colors: {
      transparent: 'transparent',
      current: 'currentColor',
      white: '#fff',
      gray: {
        100: '#B1B3B399',
        200: '#B1B3B3CC',
        300: '#B1B3B3',
      },
    },
  },
  plugins: [require('flowbite/plugin')],
};

Expected behavior
The configuration for tailwind.config.js should take priority of the a plugin configuration.
Actual behavior
The plugin configuration takes priority over the tailwind.config.js theme.
Screenshots
** Wrong Computed Value (equates to #d1d5db, from Flowbite plugin): **

Desktop (please complete the following information):

OS: MacOS
Browser: Chrome
Version: Latest

Hey @whanso,
Makes perfect sense. I'll look into this for the v1.4.0 update.


$item.TypeName should be $item.Type

https://github.com/Microsoft/ReportingServicesTools/blob/034f4b91910fdf7ddb4a99181735f4cbea82cd56/ReportingServicesTools/Functions/CatalogItems/Out-RsFolderContent.ps1#L106
This does not work due to the fact the property name is Type and not TypeName according to the https://msdn.microsoft.com/en-us/library/reportservice2005.catalogitem.type.aspx

Yes it works.  The link you specified is for older RS2005, but newer RS2010 has TypeName property.
https://msdn.microsoft.com/en-us/library/reportservice2010.catalogitem.typename.aspx


[Hardware][NV] Add support for ModelOpt static scaling checkpoints.

This change adds support for a new quantization class that has the
ability to load ModelOpt checkpoints through HF. It can be invoked via -
llm = LLM(model=model_path, quantization="modelopt")
The checkpoint needs a hf_quant_config.json for loading the right
quantiztion format. To workaround the safetensors format limitation,
the quantized weights are loaded in int8 and then a
view(torch.float8_e4m3fn) converts it to fp8.
A test will be added once ModelOpt publishes the FP8 checkpoints to HF Hub.

 PR Checklist (Click to Expand)
Thank you for your contribution to vLLM! Before submitting the pull request, please ensure the PR meets the following criteria. This helps vLLM maintain the code quality and improve the efficiency of the review process.
PR Title and Classification
Only specific types of PRs will be reviewed. The PR title is prefixed appropriately to indicate the type of change. Please use one of the following:

[Bugfix] for bug fixes.
[CI/Build] for build or continuous integration improvements.
[Doc] for documentation fixes and improvements.
[Model] for adding a new model or improving an existing model. Model name should appear in the title.
[Frontend] For changes on the vLLM frontend (e.g., OpenAI API server, LLM class, etc.)
[Kernel] for changes affecting CUDA kernels or other compute kernels.
[Core] for changes in the core vLLM logic (e.g., LLMEngine, AsyncLLMEngine, Scheduler, etc.)
[Hardware][Vendor] for hardware-specific changes. Vendor name should appear in the prefix (e.g., [Hardware][AMD]).
[Misc] for PRs that do not fit the above categories. Please use this sparingly.

Note: If the PR spans more than one category, please include all relevant prefixes.
Code Quality
The PR need to meet the following code quality standards:

We adhere to Google Python style guide and Google C++ style guide.
Pass all linter checks. Please use format.sh to format your code.
The code need to be well-documented to ensure future contributors can easily understand the code.
Include sufficient tests to ensure the project to stay correct and robust. This includes both unit tests and integration tests.
Please add documentation to docs/source/ if the PR modifies the user-facing behaviors of vLLM. It helps vLLM user understand and utilize the new features or changes.

Notes for Large Changes
Please keep the changes as concise as possible. For major architectural changes (>500 LOC excluding kernel/data/config/test), we would expect a GitHub issue (RFC) discussing the technical design and justification. Otherwise, we will tag it with rfc-required and might not go through the PR.
What to Expect for the Reviews
The goal of the vLLM team is to be a transparent reviewing machine. We would like to make the review process transparent and efficient and make sure no contributor feel confused or frustrated. However, the vLLM team is small, so we need to prioritize some PRs over others. Here is what you can expect from the review process:

 After the PR is submitted, the PR will be assigned to a reviewer. Every reviewer will pick up the PRs based on their expertise and availability.
 After the PR is assigned, the reviewer will provide status update every 2-3 days. If the PR is not reviewed within 7 days, please feel free to ping the reviewer or the vLLM team.
 After the review, the reviewer will put an  action-required label on the PR if there are changes required. The contributor should address the comments and ping the reviewer to re-review the PR.
 Please respond to all comments within a reasonable time frame. If a comment isn't clear or you disagree with a suggestion, feel free to ask for clarification or discuss the suggestion.

Thank You
 Finally, thank you for taking the time to read these guidelines and for your interest in contributing to vLLM. Your contributions make vLLM a great tool for everyone!

With the utilities that I made for fp8, all of the core functionality of fp8 is now refactored out of fp8.py.
So, this PR should not need to touch fp8.py. Instead, you can use the factored out utils withModelOptConfig and ModelOptFp8LinearMethod that mirror the logic we used for AutoFP8

Utilities are in and this PR should now be unblocked

@robertgshaw2-neuralmagic @simon-mo @youkaichao Please review 🙏

@dsikka going to review

/ready


Audio cover preview support

Please describe the problem you're trying to solve
Without the proposed feature, the current state of audio file navigation in the application is not as intuitive or visually engaging as it could be. Users are only able to view metadata, which may not provide enough visual cues to quickly and easily identify specific audio files. This can lead to a less efficient and enjoyable user experience, especially for those with large collections of audio files with cover art.
Would you be willing to contribute this feature?

[X] Yes, I'll give it a shot

Describe the solution you'd like
The proposed solution involves checking if an audio file has a cover using the lofty library in Rust. If a cover is present, it will be used for the preview. If not, the program will fall back to previewing the audio's metadata. Here is a sample code snippet demonstrating this:
use lofty::file::TaggedFileExt;
use lofty::probe::Probe;

fn audio_has_cover<P: AsRef<Path>>(path: P) -> bool {
    return match Probe::open(path) {
        Ok(p) => {
            let tagged_file = match p.read().ok() {
                Some(file) => file,
                None => return false,
            };
            let tag = match tagged_file.primary_tag() {
                Some(tag) => tag,
                None => return false,
            };
            tag.picture_count() > 0
        }
        Err(_) => false,
    };
}

If the audio file has a cover, the following ffmpeg command can be used to generate the thumbnail:
[
    "/usr/bin/ffmpeg",
    "-i",
    audio,
    "-an",
    "-vcodec",
    "copy",
    thumbnail_path,
]

Additional context
No response

Hi, https://github.com/Sonico98/exifaudio.yazi is exactly what you're looking for:

You can find more awesome plugins at https://yazi-rs.github.io/docs/resources


Pre-training

Do the experimental results obtained in Table 2 of the paper use a pre-trained model? Using pre-training works well on the COVID-19 dataset, but the other one is not good.If not used, the opposite experiment is obtained.

And my GPU is a 2-card TESLA V100 32G.Looking forward to your recovery.

The performance may have something with the GPU used and the hyperparameters may be different. So you can choose what is best for your setup.


systemd error: Default-Start contains no runlevels, aborting.

I've a Ubuntu 16.04 with systemd.
The package has this in the recipe:
poise_service "node_exporter" do
    command "/some/command"
    user user
end

But I get this error when deploying:
 update-rc.d: error: node_exporter Default-Start contains no runlevels, aborting.

Did I miss anything in the recipe? The same recipe works fine on a Ubuntu 14.04 system. Thanks!

poise-service 1.4.2 according to my Berksfile.lock

Solved, ugh.
There were old files in /etc/sv/, /etc/service/ and /etc/init.d/ from a previous version. Removing those solved the problem.
Thanks :)