chore: add nestjs-external example

Related issue (if exists)
bundle node_modules is much harder than external node_modules, since we needs to deal with lots packages which not support bundle, so we provide another example for NestJS users who don't need bundle node_modules
Summary

🤖 Generated by Copilot at a4c3c1e
No summary available (Limit exceeded: required to process 76420 tokens, but only 50000 are allowed per call)

Walkthrough

🤖 Generated by Copilot at a4c3c1e
No walkthrough available (Limit exceeded: required to process 76420 tokens, but only 50000 are allowed per call)

cc @zackarychapple @antoinepairet might find this useful


Erro in ramp_wall.py

File "examples/terran/ramp_wall.py", line 42, in on_step
depot_placement_positions = self.main_base_ramp.corner_depots
File "C:\Users\ederq\Anaconda3\lib\site-packages\sc2\bot_ai.py", line 80, in main_base_ramp
key=(lambda r: self.start_location.distance_to(r.top_center))
python 3.7

Same problem here. I wonder why it's no longer finding the ramp's corners.

Hi all,
I had that issue and did two things:

not important: you need to retain the original folder for the maps StarCraft II\Maps\Ladder2018Season2
use the 4.9 fix branch because there is the nasty ValueError: 3794 is not a valid AbilityId

"""
pip3 install --user pipenv
pip3 install --upgrade<EMAIL_ADDRESS>and now rampWall works!


more-logs-2 Log HttpErrorHandler exception message too

Codecov Report
Base: 69.79% // Head: 69.51% // Decreases project coverage by -0.28% :warning:

Coverage data is based on head (884b6f1) compared to base (c71ad16).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop      #28      +/-   ##
=============================================
- Coverage      69.79%   69.51%   -0.29%
  Complexity       158      158
=============================================
  Files             28       28
  Lines            490      492       +2
=============================================
  Hits             342      342
- Misses           148      150       +2

Impacted Files
Coverage Δ

src/Application/Handlers/HttpErrorHandler.php
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Ability to cancel a long running task

Sometimes I task takes too long or it has been invoked with the wrong parameters.
If I try to cancel it with Ctrl+C under Linux, the taks is cancelled, but SBT is also killed.  Restarting SBT implies spending an additional minute in startup time.
How can I abort a long running process and keep SBT alive?

Duplicate of #228


[BugFix] solve memory leakage during autoregressive decoding

Description
By not keeping only the logprobs of selected actions but all logprobs in the buffer of the decoding strategies, memory issues might occur. This is now fixed by adding an additional parameter only_store_selected_logp to the base decoding strategy, which defaults to True. Only exception is when we require the computation of the entropy (as in PPO).
This fix should be backward compatible. We can still pass all logprobs to the get_log_likelihood function along with the actions, in which case this functions gathers the correct logprobs (like before).
Motivation and Context
see above
Types of changes
What types of changes does your code introduce? Remove all that do not apply:

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds core functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (update in the documentation)
[ ] Example (update in the folder of examples)

Checklist
Go over all the following points, and put an x in all the boxes that apply.
If you are unsure about any of these, don't hesitate to ask. We are here to help!

[ ] My change requires a change to the documentation.
[ ] I have updated the tests accordingly (required for a bug fix or a new feature).
[ ] I have updated the documentation accordingly.

Codecov Report
Attention: Patch coverage is 92.85714% with 1 lines in your changes are missing coverage. Please review.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files
Coverage Δ

rl4co/__init__.py
100.00% <100.00%> (ø)

rl4co/models/common/constructive/base.py
85.13% <ø> (ø)

rl4co/models/rl/ppo/ppo.py
84.00% <ø> (ø)

rl4co/models/zoo/ptrnet/decoder.py
96.25% <100.00%> (+0.09%)
:arrow_up:

rl4co/utils/decoding.py
91.12% <90.90%> (-0.16%)
:arrow_down:

... and 1 file with indirect coverage changes

Results on a notebook, single 3090, training POMO with 100k samples/epoch,  bs 64, on  TSP 100
Old / New

Time: 3:02 / 2:58
GPU load: 61-68% / 59-65%
VRAM: 5.5GB /  3.4GB

The training time and load improve slightly (some percentage points) and VRAM usage is decreased by around 40%! :rocket:  Will merge

Codecov Report
Attention: Patch coverage is 92.85714% with 1 lines in your changes are missing coverage. Please review.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files
Coverage Δ

rl4co/__init__.py
100.00% <100.00%> (ø)

rl4co/models/common/constructive/base.py
85.13% <ø> (ø)

rl4co/models/rl/ppo/ppo.py
84.00% <ø> (ø)

rl4co/models/zoo/ptrnet/decoder.py
96.25% <100.00%> (+0.09%)
:arrow_up:

rl4co/utils/decoding.py
91.12% <90.90%> (-0.16%)
:arrow_down:

... and 3 files with indirect coverage changes

Thanks @LTluttmann! Awesome work. I didn't give a review in time but based on the test results as shown by @fedebotu , I believe this is a really good updating! 🙌


LCS for 3 strings

Aim
Hey @Ayush7614
I would like to add LCS 3 strings code.
Please assign me.
Thanks
Details
It a very important from competitive point of view.
Programming language

[ ] C
[x] C++
[ ] Java
[ ] Python

Mention your language

In c++

Can I go ahead ?

Shall I try in python ? @Amit366


Handling the toupper function that can be used in filter expression

Added functionality to handle case-sensitive filters. I've added two tests (14 and 15) that illustrate the requirement.

@maciejpieprzyk Thanks for all your contribution to this package. I have approved all pending PRs. Please forgive my absense alll this months. I want to discuss with you co-ownership of this package with you. Please write to me at<EMAIL_ADDRESS>


Cast icon / device list not appearing

I'm seeing the same issue for for all the iOS sample apps--the cast icon does not appear on initial launch. Setting the app ID to use the default receiver app (app id = "kGCKMediaDefaultReceiverApplicationID") does not fix this. The app should prompt to view available devices and allow connecting to devices if found. The readme indicates updating the app ID field but this alone isn't enough to make the sample app functional.

Are you using double-quotes when you set the app id? You shouldn't, if you are; it should read like:
[ChromecastDeviceController sharedInstance].applicationID = kGCKMediaDefaultReceiverApplicationID;

No, but the original code in the sample app looks like: [ChromecastDeviceController sharedInstance].applicationID = @"4F8B3483";
Updating the app ID to kGCKMediaDefaultReceiverApplicationID results in failure to build: "Use of undeclared identifier 'kGCKMediaDefaultReceiverApplicationID' ".

After resetting the Chromecast a few times and verifying that devices are communicating across the same network, it's seeing my Chromecast. The issue must have been somewhere in my hardware setup rather than the code.


Wav2Vec2ASRBundle source code not found

🐛 Describe the bug
When clicking the [Source] link on the Wav2Vec2ASRBundle , it will show "404 not found".
Versions
N/A

oh this is a tricky one.
Jekyll by default ignores all the directory starts with _. So underscored modules are excluded.
We need to add _wav2vec2 and _tts.
https://github.com/pytorch/audio/blob/gh-pages/_config.yml#L15

Close by #1997


Add and test templated local connection parameters

refs #1022, #1116

Turns out the only connection parameter of ansible.plugins.connection.local is connection pipelining, for which Mitogen does its own thing.


[BUG] AccessViolationException in finalizer thread

Description
After upgrading from v2.80.4-preview.9 to v2.88.0, I am now getting an uncatchable AccessViolationException in the finalizer thread, as described in the comment to #1817.
Code
The code is an ASP.NET MVC 5  application targeting .NET Framework 4.8. It uses QuestPDF v2022.6.3 to generate a PDF file. The only remotely complicated part of the code is for adding rounded corners to an image:
public class RoundedImage : IDynamicComponent<int>
{
    public RoundedImage([NotNull] string imagePath)
    {
        ImagePath = imagePath;
    }

    [NotNull]
    public string ImagePath { get; }

    int IDynamicComponent<int>.State { get; set; }

    [NotNull]
    private static SKBitmap LoadImage([NotNull] string imagePath)
    {
        using var stream = File.OpenRead(imagePath);
        return SKBitmap.Decode(stream);
    }

    private static float CalculateImageHeight([NotNull] string imagePath, float width)
    {
        using var image = LoadImage(imagePath);
        return width * image.Height / image.Width;
    }

    /// <inheritdoc />
    public DynamicComponentComposeResult Compose(DynamicContext context)
    {
        string imagePath = ImagePath;
        if (string.IsNullOrEmpty(imagePath) || !File.Exists(imagePath))
        {
            return new();
        }

        var content = context.CreateElement(container =>
        {
            var width = context.AvailableSize.Width;
            var imageHeight = CalculateImageHeight(imagePath, width);

            container.MaxHeight(imageHeight).Canvas((canvas, size) =>
            {
                canvas.Save();
                var canvasRect = SKRect.Create(0, 0, size.Width, size.Height);
                using SKPath path = new();
                path.AddRoundRect(canvasRect, 15f, 15f);
                canvas.ClipPath(path, antialias: true);
                using var image = LoadImage(imagePath);
                var imageRect = SKRect.Create(0, 0, image.Width, image.Height);
                canvas.DrawBitmap(image, imageRect, canvasRect);
                canvas.Restore();
            });
        });

        return new() { Content = content };
    }
}

The code works as expected, and the PDF is generated correctly.
Expected Behavior
No exception should be thrown in the finalizer thread.
Actual Behavior
An uncatchable exception is thrown in the finalizer thread.
Application: w3wp.exe
Framework Version: v4.0.30319
Description: The process was terminated due to an unhandled exception.
Exception Info: System.AccessViolationException
   at SkiaSharp.Internals.PlatformLock+NonAlertableWin32Lock.EnterCriticalSection(IntPtr)
   at SkiaSharp.Internals.PlatformLock+NonAlertableWin32Lock.EnterWriteLock()
   at SkiaSharp.HandleDictionary.DeregisterHandle(IntPtr, SkiaSharp.SKObject)
   at SkiaSharp.SKObject.DeregisterHandle(IntPtr, SkiaSharp.SKObject)
   at SkiaSharp.SKObject.set_Handle(IntPtr)
   at SkiaSharp.SKNativeObject.Dispose(Boolean)
   at SkiaSharp.SKNativeObject.Finalize()

Basic Information

Version with issue:  2.88.0
Last known good version: v2.80.4-preview.9
IDE:  Visual Studio 2022
Platform Target Frameworks:

Windows Classic:  Windows 11

Detailed IDE/OS information (click to expand)

Microsoft Visual Studio Professional 2022
Version 17.2.6
VisualStudio.17.Release/17.2.6+32630.192
Microsoft .NET Framework
Version 4.8.04161

Installed Version: Professional

.NET Core Debugging with WSL   1.0
.NET Core Debugging with WSL

ADL Tools Service Provider   1.0
This package contains services used by Data Lake tools

ASA Service Provider   1.0

ASP.NET and Web Tools 2019   17.2.393.26812
ASP.NET and Web Tools 2019

ASP.NET Web Frameworks and Tools 2012   17.2.393.26812
For additional information, visit https://www.asp.net/

Azure App Service Tools v3.0.0   17.2.393.26812
Azure App Service Tools v3.0.0

Azure Data Lake Tools for Visual Studio   2.6.5000.0
Microsoft Azure Data Lake Tools for Visual Studio

Azure Functions and Web Jobs Tools   17.2.393.26812
Azure Functions and Web Jobs Tools

Azure Stream Analytics Tools for Visual Studio   2.6.5000.0
Microsoft Azure Stream Analytics Tools for Visual Studio

Bundler & Minifier   2.9.3
Adds support for bundling and minifying JavaScript, CSS and HTML files in any project.

C# Tools   4.2.0-4.22281.5+8d3180e5f00d42f0f0295165f756f368f0cbfa44
C# components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.

Common Azure Tools   1.10
Provides common services for use by Azure Mobile Services and Microsoft Azure Tools.

Dotfuscator Community Edition   6.5.0+76d4669002
PreEmptive Protection - Dotfuscator CE

Extensibility Message Bus   1.2.6 (master@34d6af2)
Provides common messaging-based MEF services for loosely coupled Visual Studio extension components communication and integration.

File Icons   2.7
Adds icons for files that are not recognized by Solution Explorer

GhostDoc   22.2.22190.0
Generate XML Comments from your code, maintain clean and up-to-date documentation, produce help documentation in multiple formats, use intelligent source code Spell Checker in Visual Studio.

Microsoft Azure Hive Query Language Service   2.6.5000.0
Language service for Hive query

Microsoft Azure Stream Analytics Language Service   2.6.5000.0
Language service for Azure Stream Analytics

Microsoft Azure Tools for Visual Studio   2.9
Support for Azure Cloud Services projects

Microsoft JVM Debugger   1.0
Provides support for connecting the Visual Studio debugger to JDWP compatible Java Virtual Machines

Mono Debugging for Visual Studio   17.2.20 (482eb2a)
Support for debugging Mono processes with Visual Studio.

NuGet Package Manager   6.2.1
NuGet Package Manager in Visual Studio. For more information about NuGet, visit https://docs.nuget.org/

Razor (ASP.NET Core)   <IP_ADDRESS>8101+885a343b00bcab620a90c1550c37dafd730ce984
Provides languages services for ASP.NET Core Razor.

Sandcastle Help File Builder   SHFB
Visual Studio integration for the Sandcastle Help File Builder.
https://GitHub.com/EWSoftware/SHFB

SQL Server Data Tools   17.0.62204.01010
Microsoft SQL Server Data Tools

Syntax Visualizer   1.0
An extension for visualizing Roslyn SyntaxTrees.

ToolWindowHostedEditor   1.0
Hosting json editor into a tool window

TypeScript Tools   17.0.10418.2001
TypeScript Tools for Microsoft Visual Studio

Visual Basic Tools   4.2.0-4.22281.5+8d3180e5f00d42f0f0295165f756f368f0cbfa44
Visual Basic components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.

Visual F# Tools   17.1.0-beta.22329.1+702b8e77f5fbfe21e6743324c1750503e02f182d
Microsoft Visual F# Tools

Visual Studio IntelliCode   2.2
AI-assisted development for Visual Studio.

VisualStudio.DeviceLog   1.0
Information about my package

VisualStudio.Mac   1.0
Mac Extension for Visual Studio

Web Compiler   1.14.8
Compiler for LESS, Sass and CoffeeScript files

Xamarin   <IP_ADDRESS> (d17-2@2f6c881)
Visual Studio extension to enable development for Xamarin.iOS and Xamarin.Android.

Xamarin Designer   <IP_ADDRESS> (remotes/origin/d17-2@197e1a0b7)
Visual Studio extension to enable Xamarin Designer tools in Visual Studio.

Xamarin Templates   17.2.15 (2e3b60e)
Templates for building iOS, Android, and Windows apps with Xamarin and Xamarin.Forms.

Xamarin.Android SDK   <IP_ADDRESS> (d17-2/4e061b7)
Xamarin.Android Reference Assemblies and MSBuild support.
    Mono: dffa5ab
    Java.Interop: xamarin/java.interop/d17-2@9760f0a9
    ProGuard: Guardsquare/proguard/v7.0.1@912d149
    SQLite: xamarin/sqlite/3.38.2@7b1e016
    Xamarin.Android Tools: xamarin/xamarin-android-tools/d17-2@fc3c2ac

Xamarin.iOS and Xamarin.Mac SDK   <IP_ADDRESS> (96b3edb6d)
Xamarin.iOS and Xamarin.Mac Reference Assemblies and MSBuild support.

May also be related to https://github.com/mono/SkiaSharp/issues/2181?


bug: api.getStickerSet returns true

const stickerSetName = `my_name`
const stickerSet = await ctx.api.getStickerSet(stickerSetName)

// stickerSet === true

This behavior is for valid and invalid set names - it returns true in any case.

Did you enable webhookReplies for this method?

I cannot reproduce this.
The following code works for me.
import { Api } from<EMAIL_ADDRESS>
const api = new Api("<token>");

const stickerSetName = "senya_animated"
const stickerSet = await api.getStickerSet(stickerSetName)

console.log(stickerSet.stickers.length); // output: 24

Are you running on webhooks and did you enable canUseWebhookReply for this method?

Yes, on webhooks.
And thanks for your quick reply! I disabled canUseWebhookReply for this method, and voila!
canUseWebhookReply: (method) => method !== 'getStickerSet'

Glad it's fixed! :)
Also beware of more pitfalls with webhook replies: https://grammy.dev/guide/deployment-types.html#webhook-reply
Closing.


Crash presenting JSQLocationMediaItem

Sometimes I have a crash when presenting a JSQLocationMediaItem. It is not consistent and I can't seem to always reproduce it, but it happens quite frequently.
The crash is at JSQLocationMediaItem.m, specifically inside the
createMapViewSnapshotForLocation:(CLLocation *)location
                        coordinateRegion:(MKCoordinateRegion)region
                   withCompletionHandler:(JSQLocationMediaItemCompletionBlock)completion

method, at:
 MKAnnotationView *pin = [[MKPinAnnotationView alloc] initWithAnnotation:nil reuseIdentifier:nil]

Stack trace:
UIKit 0x0000000186a4e08c -[_UIViewAdditiveAnimationAction runActionForKey:object:arguments:] + 636
QuartzCore 0x00000001860b7e94 CA::Layer::set_bounds() + 404
QuartzCore 0x00000001860b7c74 -[CALayer setBounds:] + 100
MapKit 0x0000000183e959b8 -[MKLayer setBounds:] + 52
UIKit 0x00000001867d5418 -[UIView(Geometry) setBounds:] + 324
MapKit 0x0000000183e957a0 -[MKPinAnnotationView initWithAnnotation:reuseIdentifier:] + 156
JSQMessagesViewController 0x00000001007d3c2c 0x00000001007cc000 + 31788
libdispatch.dylib 0x0000000193ce9994 _dispatch_call_block_and_release + 20
libdispatch.dylib 0x0000000193ce9954 _dispatch_client_callout + 12
libdispatch.dylib 0x0000000193cf6780 _dispatch_root_queue_drain + 1844
libdispatch.dylib 0x0000000193cf7c4c _dispatch_worker_thread3 + 104
libsystem_pthread.dylib 0x0000000193ec922c _pthread_wqthread + 812
libsystem_pthread.dylib 0x0000000193ec8ef0 start_wqthread + 0
I just wanted to see if it happened with anyone else, or if anyone can enlighten me :/

Yes, I´m also seeing it. Can´t reproduce it though.

Same here. It's extremely hard to reproduce too, it happens sporadically.

@alezoffoli - please update the issue after reviewing the contribution guidelines :smile:

I changed in code line
[snapShotter startWithQueue:dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0) to
[snapShotter startWithQueue:dispatch_get_main_queue()
And it seems fixed.

Hello everyone!
I'm sorry to inform the community that I'm officially deprecating this project. 😢 Please read my blog post for details:
http://www.jessesquires.com/blog/officially-deprecating-jsqmessagesviewcontroller/
Thus, I'm closing all issues and pull requests and making the necessary updates to formally deprecate the library. I'm sorry if this is unexpected or disappointing. Please know that this was an extremely difficult decision to make. I'd like to thank everyone here for contributing and making this project so great. It was a fun 4 years. 😊
Thanks for understanding,
— jsq


difficulty of training Universal model with fmax is null

Hello! Thank you so much for publishing such a great code!
Thanks to you, I'm enjoying my voice conversion!
I'm currently using hifigan as a vocoder for voice conversion, and I'm trying to train it to create a Universal model with different parameters, but I'm having trouble.
I need some advice. Here is the config.

{
"resblock": "1",
"num_gpus": 0,
"batch_size": 12,
"learning_rate": 0.0002,
"adam_b1": 0.8,
"adam_b2": 0.99,
"lr_decay": 0.999,
"seed": 1234,
"upsample_rates": [5,5,3,2,2],
"upsample_kernel_sizes": [11,11,7,4,4],
"upsample_initial_channel": 512,
"resblock_kernel_sizes": [3,7,11],
"resblock_dilation_sizes": [[1,3,5], [1,3,5], [1,3,5]],
"discriminator_periods": [2,3,5,7,11,17,23],

"segment_size": 9600,
"num_mels": 80,
"n_fft": 2048,
"hop_size": 300,
"win_size": 1200,

"sampling_rate": 24000,

"fmin": 0,
"fmax": null,
"fmax_for_loss": null,

"num_workers": 4,

"dist_config": {
    "dist_backend": "nccl",
    "dist_url": "tcp://localhost:54321",
    "world_size": 1
}

}

An image of the change in LOSS is also attached below.
The result was no different from training LJSpeech by itself.
Here are some other things we tried.

Prepared mel without audio normalization: no change in results
Trained using the distributed Universal config: Almost the same result here
Finetuning using the distributed universal weights: almost the same result.
Training with mel without dividing audio by max_wav: Worse

Dataset used

JSUT
JVS
LibriTTS
LJSpeech
VCTK
I train with a mixture of all of these.

The reason is that I set fmax to null (i.e., 12000, which is half of 24000).
I don't want to change the other parameters of the stft as much as possible because they are set by the VC.
I'm sorry for the length of this article, and thank you for your patience.

Hello! Thank you so much for publishing such a great code!
Thanks to you, I'm enjoying my voice conversion!
I'm currently using hifigan as a vocoder for voice conversion, and I'm trying to train it to create a Universal model with different parameters, but I'm having trouble.
I need some advice. Here is the config.

{
"resblock": "1",
"num_gpus": 0,
"batch_size": 12,
"learning_rate": 0.0002,
"adam_b1": 0.8,
"adam_b2": 0.99,
"lr_decay": 0.999,
"seed": 1234,
"upsample_rates": [5,5,3,2,2],
"upsample_kernel_sizes": [11,11,7,4,4],
"upsample_initial_channel": 512,
"resblock_kernel_sizes": [3,7,11],
"resblock_dilation_sizes": [[1,3,5], [1,3,5], [1,3,5]],
"discriminator_periods": [2,3,5,7,11,17,23],

"segment_size": 9600,
"num_mels": 80,
"n_fft": 2048,
"hop_size": 300,
"win_size": 1200,

"sampling_rate": 24000,

"fmin": 0,
"fmax": null,
"fmax_for_loss": null,

"num_workers": 4,

"dist_config": {
    "dist_backend": "nccl",
    "dist_url": "tcp://localhost:54321",
    "world_size": 1
}

}

An image of the change in LOSS is also attached below.
The result was no different from training LJSpeech by itself.
Here are some other things we tried.
* Prepared mel without audio normalization: no change in results

* Trained using the distributed Universal config: Almost the same result here

* Finetuning using the distributed universal weights: almost the same result.

* Training with mel without dividing audio by max_wav: Worse

Dataset used
* JSUT

* JVS

* LibriTTS

* LJSpeech

* VCTK
  I train with a mixture of all of these.

The reason is that I set fmax to null (i.e., 12000, which is half of 24000).
I don't want to change the other parameters of the stft as much as possible because they are set by the VC.
I'm sorry for the length of this article, and thank you for your patience.

So what is your trouble?

Thank you for your reply.
And I apologize for not clarifying my question.
So, what I'm trying to ask here is.
I can't get mel loss to go to 0.2 (not enough quality for inference).
I would like to know how I can achieve that.
Is it a bad parameter?
Or is it a problem with the data?

@YutoNishimura-v2 I think the authors of this repository trained a universal vocoder for several million iterations, since the data was about 1000 hours (judging by this comment #1).
Your fmax is automatically converted to half the sample rate of your audio when you set it to zero.
I would train up to 1 million iterations and only then draw conclusions.

As for the data, we are using Libri, VCTK, and LJSpeech, which the author says is sufficient!
I thought I had failed because the slope looked almost zero when I looked at the val, but after listening to you, I think I still need to train
I'll let it train for 1M and if it doesn't work, I'll ask again!
Thank you very much.

@Alexey322
Hello.
I have just taken your advice and am training for 1M, but even after 50K (1/20 of 1M) of training, the validation loss is not even 0.0001 lower.
In general, I think that the loss tends to decrease more easily in the early stage of learning. Therefore, I don't think that it will decrease after 1M.
So, if there is any history left, can you show me the initial change in loss when you ran 1M before?
Honestly, I feel that there is something fundamentally wrong, not just an iter problem.
Thank you very much.

2048

Do you know how to set the config if my sample_rate is 16k ?

@Alexey322 Hello.
I have just taken your advice and am training for 1M, but even after 50K (1/20 of 1M) of training, the validation loss is not even 0.0001 lower.
In general, I think that the loss tends to decrease more easily in the early stage of learning. Therefore, I don't think that it will decrease after 1M.
So, if there is any history left, can you show me the initial change in loss when you ran 1M before?
Honestly, I feel that there is something fundamentally wrong, not just an iter problem.
Thank you very much.

Any tips or updates here for Universal Hifigan, which is better between fmax=null or fmax=8000 by default?

Anyone knows why fmax≠fmax_for_loss?


MINOR: Add DEBUG level logs for successful/failed authentications wit…

I believe that it'll be useful for debugging purposes to know which IP addresses are consistently failing authentication.
Applications that consistently fail authentication can introduce CPU pressure to the broker and as such, I think it is useful for Ops teams to have the needed information to quickly block.
I've left these logs on DEBUG level since I presume that most common cases won't require to know this information. I'm sure we shouldn't log successful auths in INFO but I'm hesitant about unsuccessful auths
cc @rajinisivaram @ijuma @rondagostino

I think failed authentications should be info given that they're pretty expensive for people using TLS which is very common.

Related JIRA for server side logging: https://issues.apache.org/jira/browse/KAFKA-5810


Cross-Site Request Forgery (CSRF)

How is Use HTTP verbs in a RESTful way a cross-site request forgery problem?
The second statement in there is correct:
Do not use GET requests to alter the state of resources

I can't think of a way to perform a CSRF attack with a PUT or DELETE request unless CORS settings are completely unrestricted. I agree that the first part of the sentence is not important. Freel free to submit a PR to improve that item.


Pagination buttons should be before image controls in source order

When tabbing through the elements in the viewer, you arrive at the image controls before the pagination controls. This should be changed in line with the new visual ordering of these elements after the A/B test in #4550

@davidpmccormick @gestchild This is no longer a problem, correct?


Added 'const' to setRobotStateFromPoint declaration parameters

issue: https://github.com/moveit/moveit2/issues/879
I made the integer parameter in the setRobotStateFromPoint function declaration a constant.

Thanks for the PR but I think you'd need to update the function definition too in order to make this work. Given the size of the PR and how relevant this change is, I'll close the PR for now. If you'd like to contribute to moveit2 in a meaningful way, I'd recommend picking up one of the good first issues


Add file that describes creation and evolution of the galaxies

There should be a way to create and evolve systems using some kind of external file that stores information about their propeties.
It could possibly be a new language or a JSON/YAML with some amount of paramateres.

Done in 119be16a92c0c479c8b92c36e584bcba2acc19fb


interpolate texcoords

interpolate texcoords
obj loader parses vt, mtllib and usemtl
obj exporter stores vt, mtllib and usemtl

Seems to work reasonably.

Thank you for the UV update. Might be improved further by marking UV seams as border


Move to PostgreSQL Flexible Server in Azure

https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/postgresql_flexible_server
https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/postgresql_flexible_server_configuration
https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/postgresql_flexible_server_database

Completed


mobile and tablet menus

Description
WIP
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Change Status

[ ] Complete, tested, ready to review and merge
[ ] Complete, but not tested (may need new tests)
[ ] Incomplete/work-in-progress, PR is for discussion/feedback

How Has This Been Tested?

[ ] Test A
[ ] Test B

Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] My code has been reviewed by at least one peer
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] There are no merge conflicts

I'm working on building a form for creating a property, but I have pulled in this branch and made some adjustments, changed position absolute to flexbox for positioning so it should be a little more flexible even if we adjust the nav dimensions in the future, I'm gonna go ahead merge this and I'll submit my PR in a little bit


Test: unit test for FormatTimeMillis

Added unit-tests for FormatTimeMillis func
ref: https://github.com/cectc/dbpack/issues/70
Ⅰ. Describe what this PR did
Adding unit-tests for FormatTimeMillis func
Ⅱ. Does this pull request fix one issue?
I am not sure, but i think yes
Ⅲ. Why don't you add test cases (unit test/integration test)?
tests added
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews
this PR is ok for you, i will continue for others, let me know if you have any comments for PR

Thank you for your pr, hope you can learn more through this project。

i wrote unit tests for func FormatDate(tsMillis uint64) string

pls check the ci results, and use time.FixedZone fix time zone issue.

pls check the ci results, and use time.FixedZone fix time zone issue.

I used UTC() for fix time zone issue, wdyt? will it work for you?

it's ok

Codecov Report

Merging #75 (2c8149f) into dev (f798737) will increase coverage by 0.06%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              dev      #75      +/-   ##
==========================================
+ Coverage   31.83%   31.90%   +0.06%
==========================================
  Files          47       47
  Lines        5776     5776
==========================================
+ Hits         1839     1843       +4
+ Misses       3749     3745       -4
  Partials      188      188

Impacted Files
Coverage Δ

pkg/misc/time.go
50.00% <100.00%> (+50.00%)
:arrow_up:

Continue to review full report at Codecov.

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

CurrentTimeMillis() and CurrentTimeNano() should set time zone too

LGTM

btw, when i make test in repo, i got some test fails github.com/cectc/dbpack/pkg/optimize , how can i fix it for my local repo?

git remote add upstream  git@github.com:CECTC/dbpack.git
git fetch upstream
git pull --rebase upstream dev
git push -f

did not fix for me,
--- FAIL: TestOptimizeInsert (0.00s)
panic: permission denied [recovered]
        panic: permission denied

goroutine 22 [running]:
testing.tRunner.func1.2({0x101e69e60, 0x102c5b068})
        /usr/local/go/src/testing/testing.go:1389 +0x1c8
testing.tRunner.func1()
        /usr/local/go/src/testing/testing.go:1392 +0x384
panic({0x101e69e60, 0x102c5b068})
        /usr/local/go/src/runtime/panic.go:838 +0x204
github.com/agiledragon/gomonkey/v2.modifyBinary(0x101a21ae0, {0x140000401c8, 0x18, 0xc7e5043d10886077?})
        /Users/tanryberdi/go/pkg/mod/github.com/agiledragon/gomonkey/v2@v2.7.0/modify_binary_darwin.go:9 +0xa4
github.com/agiledragon/gomonkey/v2.replace(0x101a21ae0, 0x1400052da90?)
        /Users/tanryberdi/go/pkg/mod/github.com/agiledragon/gomonkey/v2@v2.7.0/patch.go:255 +0x8c
github.com/agiledragon/gomonkey/v2.(*Patches).ApplyCore(0x1400052da78, {0x140004cb080?, 0x1400059e148?, 0x101fb7880?}, {0x140004cb080?, 0x14000371980?, 0x14000136000?})
        /Users/tanryberdi/go/pkg/mod/github.com/agiledragon/gomonkey/v2@v2.7.0/patch.go:217 +0xe8
github.com/agiledragon/gomonkey/v2.(*Patches).ApplyMethodFunc(0x14000085a88?, {0x101e6bf60, 0x0}, {0x101ad6ddc, 0xc}, {0x101e1dca0, 0x101fb7880})
        /Users/tanryberdi/go/pkg/mod/github.com/agiledragon/gomonkey/v2@v2.7.0/patch.go:100 +0x11c
github.com/agiledragon/gomonkey/v2.ApplyMethodFunc(...)
        /Users/tanryberdi/go/pkg/mod/github.com/agiledragon/gomonkey/v2@v2.7.0/patch.go:32
github.com/cectc/dbpack/pkg/optimize.TestOptimizeInsert(0x14000503ba0)
        /Users/tanryberdi/work/open-source/dbpack/pkg/optimize/optimizer_test.go:95 +0x260
testing.tRunner(0x14000503ba0, 0x101fb7888)
        /usr/local/go/src/testing/testing.go:1439 +0x110
created by testing.(*T).Run
        /usr/local/go/src/testing/testing.go:1486 +0x300

cd `go env GOPATH`
git clone https://github.com/eisenxp/macos-golink-wrapper.git
mv `go env GOTOOLDIR`/link `go env GOTOOLDIR`/original_link
cp `go env GOPATH`/macos-golink-wrapper/link  `go env GOTOOLDIR`/link
chmod +x `go env GOTOOLDIR`/link

cd `go env GOPATH`
git clone https://github.com/eisenxp/macos-golink-wrapper.git
mv `go env GOTOOLDIR`/link `go env GOTOOLDIR`/original_link
cp `go env GOPATH`/macos-golink-wrapper/link  `go env GOTOOLDIR`/link
chmod +x `go env GOTOOLDIR`/link

did not work one more time, i got same fails ((

you can ignore it, that is not your problem, this problem may occur on mac

you can ignore it, that is not your problem, that problem may occur on mac

got it

LGTM

you can ignore it, that is not your problem, that problem may occur on mac

got it
add -gcflags=all=-l  can solve this problem, like this:

	@go test -gcflags=all=-l -short ${PKG_LIST}


Skip download if video is already downloaded?

So let's say I have a playlist with 3 videos that I want to download as MP3, I'll just do :
'youtube-dl [playlist link here]  --playlist-start 1 --extract-audio --audio-format mp3'
Ok, so I download the playlist, but then I add 3 new videos, and I want to download them too. If I execute the same command as before, it will download ALL the videos in the playlist, even the ones that were already downloaded before.
How can I make it so that it doesn't download the playlist videos that are already downloaded?

Use --download-archive.

I tried to use it, but it seems to require an argument (the list of IDs of all downloaded videos)??

It requires the filename to know where to store and look for the downloaded videos.

Thank you!!


Small bug in filter template

Hi,
When you filter by "blocked" in the log, it also makes a checkmark at "OK - forwarded".
Regards
Hyper

:man_facepalming: :sweat_smile:

Fixed on v4.0.5.


TypeError: make() missing 1 required positional argument: 'arg2'

Good evening. I switched from the gnuradio branch to the new gnuradio branch, uninstalled and then installed the latest.
While performing a quick test (on same machine the older gnuradio flowgraph ran on) I now encounter this upon starting and wondered if it's just me or if anyone else is having this issue,
`[INFO] [B200] Detected Device: B205mini
[INFO] [B200] Operating over USB 3.
[INFO] [B200] Initialize CODEC control...
[INFO] [B200] Initialize Radio control...
[INFO] [B200] Performing register loopback test...
[INFO] [B200] Register loopback test passed
[INFO] [B200] Setting master clock rate selection to 'automatic'.
[INFO] [B200] Asking for clock rate 16.000000 MHz...
[INFO] [B200] Actually got clock rate 16.000000 MHz.
[INFO] [B200] Asking for clock rate 30.720000 MHz...
[INFO] [B200] Actually got clock rate 30.720000 MHz.
[INFO] [MULTI_USRP]     1) catch time transition at pps edge
[INFO] [MULTI_USRP]     2) set times next pps (synchronously)
Traceback (most recent call last):
File "/tmp/drone_id_demod_test.py", line 357, in
main()
File "/tmp/drone_id_demod_test.py", line 335, in main
tb = top_block_cls()
File "/tmp/drone_id_demod_test.py", line 222, in init
self.droneid_extractor_0 = droneid.extractor(samp_rate / decimation, )
TypeError: make() missing 1 required positional argument: 'arg2'

Done (return code 1)
`

I uninstalled gr-droneid update branch, went back to just the gr-droneid branch and all is well after making the /tmp/bursts directory. Not sure what's causing the difference in the update branch.

No red that I can see, graph builds, but maybe somehow there’s a missing variable even though the blocks are not red.

Looks like arg2 should be a float. When looking at the generated .py file there’s nothing after the , so no arg2 that is. Just seeing what might be expected there.

Got it, the extractor on the flow graph had no correlator value, but then again no red. I put a zero in and the flow graph will run now.


initWithModelToJSONDictionary

Hi, i saw in example in the readme file a function named initWithModelToJSONDictionary. Where is it? is it deprecated?

Nope that should be fine.

Perhaps you need to use an equivalent with the error output though?

I do not have initWithModelToJSONDictionary

is there an update that i missed?

Just upgrade JSONModel to the latest version - 1.4.0

Thanks, that was right. I was convinced I had the latest version :S


Gateway - add $UMA to bridge

I propose we add $UMA, UMA's native protocol token. We have a lot  of cool things we're doing with them in store, and would be great to show their community some love before we announce. The token contract is 0x04fa0d235c4abf4bcf4787af4cf447de572ef828.
https://www.coingecko.com/en/coins/uma

#256


italian language support for home routing

I've routed overseas languages with trailing slash  /en/ /it/ so that header menu gets instantly translated too

Sumimasen, any problem merging this?

👍🏻


Debian agent installation failed due to sudo: command not found

Describe what happened:
Hi, we currently use Datadog as our observability tool.
We run our apps in docker containers and deploy them to Google cloudrun (Fully Managed and I'm aware that the AMP feature is not supported). We wanted to try to install the agent in the dockerfile to collect traces (that is, if the agent is supported that way).
We use debian:bullseye-slim in a base image for our docker builds. Each application pulls the base image and runs separately as a cloudrun service. See below for code snippet.
Dockerfile.base:
FROM debian:bullseye-slim as audiowaveform-builder
RUN apt-get update -y \
    && apt-get install -y --no-install-recommends \
        apt-transport-https \
        ca-certificates \
        cmake \
        curl \
...

Dockerfile for one application:
FROM base image url
ENV NODE_ENV=production

# install Datadog agent to collect traces
RUN DD_AGENT_MAJOR_VERSION=7 DD_API_KEY=<emitted> DD_SITE="datadoghq.com" bash -c "$(curl -L https://s3.amazonaws.com/dd-agent/scripts/install_script.sh)"

Describe what you expected:
I expect the docker build to succeed with Datadog agent successfully installed.
Steps to reproduce the issue:
We use nx build for docker build. When I run it, I get the following error (See screenshot).
I read the install script and it seems like it removes the $sudo_cmd prefix if it's empty or the output is command not found, but the error suggested that it hit the else statement in the next line with the $sudo_cmd prefix again, which I believe exits the script.

Not sure if this is a known issue or if it's the debian version. Would love to get some help on this. Thank you!
Additional environment details (Operating System, Cloud provider, etc):
I ran the install script for mac OsX on my computer and it succeeded as expected. And I can open the agent gui.

Does installing sudo in the Docker image work around the issue?

Sorry this issue may be stale. Installing sudo requires access to the container shell because it requires a password or root permission and it's just not the best practice when running docker containers. Also, unfortunately we use cloudrun fully managed service for our infrastructure and it doesn't give us the access to the container shell. We are pulling in OpenTelemetry to solve our tracing data in cloudrun. Closing this issue as there's not much Datadog can do at this point.

I was at the Dash conference and discovered that now Datadog allows the agent to be installed with the containerized application on CloudRun. I missed the demo so am not sure how exactly it works but that's a pretty exciting improvement.


fail to run python3 -c "import lmcache_vllm.vllm as vllm"

Describe the bug
I start a container with image lmcache_vllm:lmcache-0.1.3. when i execute next command, some error happend
 # python3 -c "import lmcache_vllm.vllm as vllm"
Using /root/.cache/torch_extensions/py310_cu121 as PyTorch extensions root...
Emitting ninja build file /root/.cache/torch_extensions/py310_cu121/torchac_backend/build.ninja...
Building extension module torchac_backend...
Allowing ninja to set a default number of workers... (overridable by setting the environment variable MAX_JOBS=N)
ninja: no work to do.
Loading extension module torchac_backend...
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ModuleNotFoundError: No module named 'lmcache_vllm.vllm'

I have tried different version, 0.1.0, 0.1.3 image, all the same error. Can any body hello? thank you so mush!
To Reproduce
Steps to reproduce the behavior:
docker run -itd --name lmcache --privileged --gpus all  --shm-size=128g lmcache_vllm:lmcache-0.1.3 bash
docker exec -it lmcache bash
python3 -c "import lmcache_vllm.vllm as vllm"
Using /root/.cache/torch_extensions/py310_cu121 as PyTorch extensions root...
Emitting ninja build file /root/.cache/torch_extensions/py310_cu121/torchac_backend/build.ninja...
Building extension module torchac_backend...
Allowing ninja to set a default number of workers... (overridable by setting the environment variable MAX_JOBS=N)
ninja: no work to do.
Loading extension module torchac_backend...
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ModuleNotFoundError: No module named 'lmcache_vllm.vllm'

Hi, could you share the command you used to install LMCache?


cstddef not found

What is the expected behavior

Hi, I am trying to replace the clfft to the rocm fft lib but my original code is written by c (not cpp), after gcc compiling, I got the error cstddef not found in header file, I wonder is there a circumvent this?
What actually happens

How to reproduce

Environment

Hardware
description

GPU
device string

CPU
device string

Software
version

ROCK
v0.0

ROCR
v0.0

HCC
v0.0

Library
v0.0

rocFFT requires a C++ compiler.
If you use g++ instead of gcc, things should build successfully.
If for some reason it's difficult to change the compiler name, you could work around it by adding -x c++ at compile time and -lstdc++ at link time to try to convince gcc to build and link like g++ would.


Test if content exists in ListingBody, for addon Dropdownmenu

When you use addon dropdownmenu, you can add some blocks into a dropdown menu.
If you add some blocks and you try to access a non published content in private mode, an error is thrown because ListingBody expects a content to check if there's some items inside.
With this fix, ListingBody could work without 'content'

@giuliaghisini changelog please!

@giuliaghisini changelog please!

sorry! done


How to specify a component dependency on board

From condition, we could know following condition attributes are supported.

Question is how to specify a component dependency on board? There is no board related attribute there.

@tcsunhao this is a feature gap that we should IMHO add.
PROPOSAL
Add to conditions element a way to filter for boards with:

Attribute
Description

Bvendor
Specifies board vendor

Bname
Specifies board name

Brevision
Specifies board revision

We may even support the extension points in the conditions ?
@jkrech : what do you think ?

@fred-r, I think this needs to be reviewed as part of specifying: https://github.com/Open-CMSIS-Pack/Open-CMSIS-Pack-Spec/issues/27 but I do agree that we need to support the extension of attributes used to filter components for a given context.

example:
  <require Bvendor="ARM" Bname="V2M-MPS2-SSE-200" Brevision="B"

added 'Bvendor', 'Bname', 'Brevision' attribute to conditions element to allow board specific filtering to Open-CMSIS-Spec


Report Grids: Survey Instance 'Issued On' field is reporting the Survey Run 'Issued On' value

Description
When exporting/viewing the report grid, the survey section in the UI displays the correct information
Waltz Version
No response
Steps to Reproduce

...
Expected Result
No response
Actual Result
No response

Confirmed this is still an issue.


AZCopy logs missing summary when executed via Storage Explorer

Which version of the AzCopy was used?
Both Azcopy 10.6.0 and 10.7.0
Storage Explorer 1.16.0
Which platform are you using? (ex: Windows, Mac, Linux)
Windows 10
What command did you run?
azcopy copy [https://BLOB URL] [C:\Temp] --output-type=json --cancel-from-stdin --overwrite=prompt --check-md5 FailIfDifferent --from-to=BlobLocal --recursive
(Command copied from the Storage Explorer log file)
What problem was encountered?
When using Storage Explorer to copy files/folders, the azcopy log does not show the Diagnostic Stats or Job Summary at the end of the log.
When running the identical azcopy command from a command prompt, the log file shows both the Diagnostic Stats and Job Summary.
How can we reproduce the problem in the simplest way?
Using Azure Storage Explorer

Select any file or folder located in a BLOB container.
Copy the file/folder to a local path
Open the log file
Observe that Diagnostic Stats and Job Summary information are missing from the bottom of the log.

Using command line:

Copy the "Job-command" line (line 5) from the above log file.
Execute azcopy using the same "Job-command" parameters from the log file.
Open the new log file
Observe that Diagnostic Stats and Job Summary information exists in the bottom of the log.

Have you found a mitigation/solution?
No.
Have attempted to replace the azcopy_[operatingsystem].exe binary that Storage Explorer installs, with the native AZcopy.exe downloaded fresh, just in-case Storage Explorer is shipped with a different azcopy binary.
Unfortunately there was no change to the result.

Thanks @phrak for reporting this issue! I was able to repro it.


replace old explorer url with the new one and remove edit option #1766

fixes #1766

Good job mate. After fixing the LGTM issue, I think we can approve & merge.

From my side all good :+1:

@yilmazbahadir LGTM issues fixed.


Test code review body

🎉 Snyk hasn't found any issues so far.
✅ security/snyk check is completed. No issues were found. (View Details)


Implemented background analysis after edit/return to idle state

Update submodule sarif-pattern-matcher to latest
Implemented background analysis after user updated code/text content in current editor window.
Some refactoring

// Copyright (c) Microsoft. All rights reserved.

remove this file, since we don't have any global suppression

Refers to: src/Sarif.Sarifer/GlobalSuppressions.cs:1 in cb08985. [](commit_id = cb089855b2e21d97d0bcd25415f0a1f4617c2731, deletion_comment = False)

// Copyright (c) Microsoft. All rights reserved.

remove this file, since we don't have any global suppression

Refers to: src/Sarif.Sarifer/GlobalSuppressions.cs:1 in cb08985. [](commit_id = cb089855b2e21d97d0bcd25415f0a1f4617c2731, deletion_comment = False)

// Copyright (c) Microsoft. All rights reserved.

done

In reply to: 764660882 [](ancestors = 764660882)

Refers to: src/Sarif.Sarifer/GlobalSuppressions.cs:1 in cb08985. [](commit_id = cb089855b2e21d97d0bcd25415f0a1f4617c2731, deletion_comment = False)

// Copyright (c) Microsoft. All rights reserved.

done

In reply to: 764660882 [](ancestors = 764660882)

Refers to: src/Sarif.Sarifer/GlobalSuppressions.cs:1 in cb08985. [](commit_id = cb089855b2e21d97d0bcd25415f0a1f4617c2731, deletion_comment = False)


getUrlForService can return incorrect URL

From my understanding, a URL is registered in the nginx proxy for each service running, something like http://myservicename.dev.<IP_ADDRESS>.xip.io.
When getUrlForService is invoked, it returns something of the same form.
The problem arises when we change, for example, xip for nip. nginx can have services registered with xip URLs, and getUrlForService will return nip URLs which won't work.
As I see it there are two possible solutions here:

Make nginx match only on the <servicename>.<navyname> part of the url
Make getUrlForService look up the url it has registered with nginx for a given service, and return that, rather than assume it's used xip or nip.

Why would nginx still have services registered with xip rather than nip if you've changed the domain?
I agree it would be better to lookup the URL from the nginx hosts file rather than just doing simple string concatenation, but it should always return the correct URL.
After changing the domain from xip to nip, you should do a navy refresh-config to make sure that all of the services launched are up to date.

We haven't moved everything over yet because we're still testing the release candidates, so some people have a mix of Navy 2 and 3 running globally and project locally.
So yes, it's only a problem if we have a mix of versions. But if we ever need to change this again then it would be good to have a robust system that won't break in the changeover period.
Hopefully then it could also not be considered to be a breaking change.

Also if you do the following:
navy launch a and then export NAVY_EXTERNAL_SUBDOMAIN=foo.com; navy launch b, service a will now return a foo.com domain when doing navy url a even though it's virtual host in nginx has been configured for nip. Really navy url needs to use the container environment variables as its source of truth.
Possibly this logic needs to be broken out into a getUrlForService and createUrlForService functions.

Solved by #59


Architecture.md

mitex's design of spliting spec and parser/lexer is very clever and impressive. It however make it harder for people to understand what's happening. It would be better we can have a doc explaining what will happen when you type #mi(...)

I'm reading mitex's code currently and maybe I will write one.

The existing snapshots of lexer, parser, and converter would help you understand correctly, by simulating execution in your mind and compared with the snapshots.
btw, you can also make your owned testpoint in bug_playground functions:
https://github.com/mitex-rs/mitex/blob/8e5045ac836e31f4c0b7e185052a1145e2d8b809/crates/mitex-parser/tests/ast.rs#L41-L47
For example you write a snapshot assertion:
/// Convenient function to launch/debug a test case
#[test]
fn bug_playground() {
  assert_debug_snapshot!(parse(r#"\frac{ a }{ b }"#), @"");
}

You can generate the snapshot fastly by cargo insta test --accept. You can also launch debugging by code lenses over bug_playground function to dive in execution:


Label Studio is unable to open my import .mp4 videos

As said above, for some reason, whenever I try to import .mp4 video for annotations, label studio lets me go through the entire process, but then is unable to load the video.

@MelinaSimp can you please share any additional information you have about your setup, including Label Studio version, your labeling configuration, and a link to sample data. There are a number of reasons that you might be experiencing this, including using an unsupported file type or CORS issues.
It's unlikely that this is a bug in Label Studio, and I encourage you to join our community slack where you'll be able to get deeper support. Thanks!

This is also happened to me. I believe that Label Studio does not accept mp4 files with codec hev1. I tried to change the codec to a avc1 and it worked.

That's good to know. Unfortunately, Label Studio doesn't have any control over which video codecs are available on the browser, or how to install new codecs if you're using one that isn't supported.


[SPARK-31034][CORE] ShuffleBlockFetcherIterator should always create request for last block group

What changes were proposed in this pull request?

This is a bug fix of #27280. This PR fix the bug where ShuffleBlockFetcherIterator may forget to create request for the last block group.
Why are the changes needed?

When (all blocks).sum < targetRemoteRequestSize and (all blocks).length > maxBlocksInFlightPerAddress and (last block group).size < maxBlocksInFlightPerAddress,
ShuffleBlockFetcherIterator will not create a request for the last group. Thus, it will lost data for the reduce task.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Updated test.

cc @cloud-fan @xuanyuanking

Test build #119285 has started for PR 27786 at commit fc36eb1.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24025/
Test PASSed.

Test build #119285 has finished for PR 27786 at commit fc36eb1.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119285/
Test FAILed.

Test build #119287 has started for PR 27786 at commit 28df4ab.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24027/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24034/
Test PASSed.

Test build #119294 has started for PR 27786 at commit 778be33.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24035/
Test PASSed.

Test build #119294 has finished for PR 27786 at commit 778be33.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119294/
Test FAILed.

Test build #119287 has finished for PR 27786 at commit 28df4ab.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119287/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24048/
Test PASSed.

Test build #119308 has started for PR 27786 at commit 6766c7c.

Test build #119319 has started for PR 27786 at commit dc94365.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24057/
Test PASSed.

Test build #119308 has finished for PR 27786 at commit 6766c7c.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119308/
Test FAILed.

Test build #119319 has finished for PR 27786 at commit dc94365.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119319/
Test PASSed.

thanks, merging to master/3.0!

thanks all!


How to set the tab toString as default ( not text only )

Wondering if there's something I'm missing as I would like to display toString by default ?

Thanks for any info

You would need to write a plugin to move it to the front (The first tab is always selected by default)
Or if you wanted toString to always be at the front you could extend the plugin and do it there. Either way the body of the parse should look something like this:
if (!$var instanceof Api\Invoice) {
    return;
}

$rep = $o->getRepresentation('tostring');

if ($rep) {
    $o->replaceRepresentation($rep, 0);
}

Thanks for the speedy reply <3

I'll go ahead and close it then, comment back if you have more trouble


Is testnet parameter working?

I keep getting binance.exceptions.BinanceAPIException: APIError(code=-2015): Invalid API-key, IP, or permissions for action. error when trying to use the testnet by setting the testnet parameter to True client = await AsyncClient.create(api_key, api_secret, testnet=True)
I'm using the api key and secret given at the bottom of testnet.binancefuture.com and it works correctly when I connect to biance API with the same key and secret using CCXT library for example so I know the key is correct and functional. Meanwhile I don't get an error when I connect to the live trading verison.
Has anyone else had any issues? I really can't see what I am doing wrong.

I got binance.exceptions.BinanceAPIException: APIError(code=0): Invalid JSON error message from Binance:

This is how I do it:

I have a boolean test, that I switch on / off to TRUE when I want to use the test
I have this code:
if(test):
self.api_key = "OwRVV9y4FLFfOzhihszzDMqfW7YRnBLRS1aFH3cGySjqTiczxSH2mFJ2d9axYOQI"
self.api_secret = "wiJwfRu7bZKl96OIJkGFK8EVtiiTZ5gdlE4QKxFsyr59V6pUUnvEi9aG2TZ9z5xG"
# This is required to change the API_URL endpoint
self.client.API_URL = 'https://testnet.binance.vision/api'

I hope it helps
Gonzalo Murillo
www.mrbot.es


Making it easier to get started

Added a README!

This is awesome - thanks Zen! I'll add connection deets to Heroku Postgres soon.


Document IP Address Overlaps in AKS Troubleshooting Documentation

#40618 Document IP Address Overlaps in AKS Troubleshooting Documentation
Creating section for possible issues due to IP address range overlapping
This was created to satisfy the issue here - https://github.com/MicrosoftDocs/azure-docs/issues/40618

@izzymsft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@jnoller I have submitted the pull request

@jnoller, when you have a moment, please could you take a look? Thanks

@sauryadas Could you review this pull request? Thank you.

@jnoller @sauryadas @georgewallace - Could one of you let us know whether this pull request from 2019 can be merged into the repo?

@mlearned

@jluk  @palma21  can you take a look?

@izzymsft Could you respond to the reviewer's questions that were added to the PR? Thanks!

@palma21 Thanks for your review.

@izzymsft Do you want to respond to the reviewer's questions that were added to the PR?

@izzymsft Your open pull request and issue from October 2019 await your response in this PR. Shall we close the PR? Thanks!

@izzymsft Your open pull request and issue from October 2019 await your response in this PR. Shall we close the PR? Thanks!

@ktoliver  #please-close ,  close do not merge

@ktoliver  #please-close ,  close do not merge

we will create a new work item and take the commentary here to link to

we will create a new work item and take the commentary here to link to

@mlearned Thanks!

@mlearned Thanks!


Added Script To Collect Data From Serial Monitor And Save It To Firebase

Added Script To Collect Data From Serial Monitor And Save It To Firebase Database

Add your name to contributor's list 😊

Done 😊

Thnaks For contributing @MadhavBahlMD  😊

I love to contribute in open source projects 😊 will see if there are more
scripts i can make.
On 11 Oct 2017 14:52, "Khushal Sharma"<EMAIL_ADDRESS>wrote:

Thnaks For contributing @MadhavBahlMD https://github.com/madhavbahlmd 😊
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Logan1x/Python-Scripts/pull/25#issuecomment-335749957,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AY94ulhFn4NgVr1asxrG2pA42qCKZtl8ks5srIjRgaJpZM4P1EYz
.


The Anatomy of an Editorial Feature

An overview of the new EditorialFeature component including the motivation for the current implementation, how features touch different artsy domains, steps to create a feature, and context and history of features in the publishing domain.
This was originally drafted to be a doc in the codebase, but it seemed like it could be interesting to outside observers, so have decided to post as a blog entry as well.

@dblandin ok to merge? This will render nothing on the artist page until we enable the test in Force.

Unless there are other comments, I believe this is now ready to merge.

OOh actually going to add the hold back so i can insert some images into this post, will update and plan to publish on 3/5.

Cool, thanks for the status update @eessex!


add XML docs to methods in the templates

Description
Added XML docs to methods in the templates; removed all warnings as well so the generated code compiles with TreatWarningsAsErrors set to true.
Fixes #4079
Checklist

Is this feature complete?

[X] Yes. Ready to ship.
[ ] No. Follow-up changes expected.

Are you including unit tests for the changes and scenario tests if relevant?

[ ] Yes
[X] No

Did you add public API?

[ ] Yes

If yes, did you have an API Review for it?

[ ] Yes
[ ] No

Did you add <remarks /> and <code /> elements on your triple slash comments?

[ ] Yes
[ ] No

[X] No

Does the change make any security assumptions or guarantees?

[ ] Yes

If yes, have you done a threat model and had a security review?

[ ] Yes
[ ] No

[X] No

Does the change require an update in our Aspire docs?

[ ] Yes

Link to aspire-docs issue:

[X] No

Microsoft Reviewers: Open in CodeFlow

@dotnet-policy-service agree
Sent from my iPad

On Aug 25, 2024, at 21:41, dotnet-policy-service[bot] @.***> wrote:
@dotnet-policy-service agree

@DamianEdwards @eerhardt can you review?

@boutquin are you still planning to work on this one?

@boutquin are you still planning to work on this one?

@DamianEdwards Yes, got a bit busy with work--will resume work on this ASAP. Thanks for the gently nod! :)


[finatra-swagger] bump versions

Twitter libs:  22.2.0
logback-classic:  1.2.11
byte-buddy: 1.12.8
swaggerUIVersion: 4.5.2

[info] DocsControllerTest: [info] - sampleController: docs endpoint should return 307 (78 milliseconds) [info] - sampleController: docs endpoint should return 200 from full URL (40 milliseconds) [info] - sampleController: /swagger.json should be returned (101 milliseconds) [info] - Startup and be healthy (25 milliseconds) [info] Run completed in 2 seconds, 571 milliseconds. [info] Total number of tests run: 4 [info] Suites: completed 1, aborted 0 [info] Tests: succeeded 4, failed 0, canceled 0, ignored 0, pending 0 [info] All tests passed.


eval EMDB

Exciting work. I have a problem with the error in the picture when I evaluate on EMDB(1). Do you know how to solve it? (Both 3DPW and RICH are runnable)

Hi @xiaoyudanaa , as mentioned at this line in the evaluation script, the current code requires EMDB to be downloaded and stored at ./dataset/. Please visit the official website to register and download the EMDB dataset to run the current evaluation. Sorry for the confusion!

Hi @yohanshin , would you be willing to upload the implementation of RTE, ROE, and ERVE for testing the EMDB-2?

Hi @yohanshin @xiaoyudanaa, how did you successfully evaluate WHAM on RICH? I try hard to make sure that I have strictly followed the setup instructions provided in the README file, and indeed successfully evaluate WHAM on 3DPW But when it comes to RICH, I encountered two main problems as follows:
1.

2.

The first one can be partly tackled by setting cfg.FLIP_EVAL = False, but I want to know what this parameter means exactly. The second may be due to the miss of smplx models, and I have noticed that you haven't mentioned this in your README file :)

Hi @yohanshin @xiaoyudanaa, how did you successfully evaluate WHAM on RICH? I try hard to make sure that I have strictly followed the setup instructions provided in the README file, and indeed successfully evaluate WHAM on 3DPW But when it comes to RICH, I encountered two main problems as follows: 1.  2.  The first one can be partly tackled by setting cfg.FLIP_EVAL = False, but I want to know what this parameter means exactly. The second may be due to the miss of smplx models, and I have noticed that you haven't mentioned this in your README file :)

Hi, thanks for you experience sharing. To the second problem, you should download the THREE SMPLX_GENDER.npz files from the link provided by smplx.


Blank names allowed in user creation

Does not crash the server, but it's be nice to throw an appropriate message.

Fixed in 0.6.3 - To be closed when 0.6.3 is published


add arena_vector to replace pmr::vector

arena_vector is better than pmr::vector,
if the arena_vector is the last object in arena block, and the block still has space to extend, use it.

回头把测试迁移一下就 ok 了。

give up arena_vector try, pmr::vector is good enough.


Move accounts settings to seperate file

I want to be able to add my chatterino settings to a VCS like git without too much hassle.
When the accounts are in a separate file I can just ignore that file.

Duplicate of #1099


Fix of GH action

Fix of GH action

https://github.com/kyma-project/application-connector-manager/pull/179


[bug] Got "No matching key, history:<PHONE_NUMBER>234" while disconnecting after refresh the page using wallet connect

Is there an existing issue for this?

[X] I have searched the existing issues

RainbowKit Version
1.0.4
wagmi Version
1.3.4
Current Behavior
After connecting via WalletConnect & refreshing the page, it throws this error when i try to disconnect

Able to disconnect before refreshing the page
Any idea to at least catch this error and handle it softly? i tried using a try catch when calling the disconnect() but the page still crashes
Expected Behavior
Able to disconnect without throwing an error
Steps To Reproduce

Connect via WalletConnect
Refresh the page
Disconnect after refresh

Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
No response
Anything else?
No response

Running into this as well
Also running into No matching key. keychain <some alphanumeric value here>
Full stack trace:
Error: No matching key. keychain: <some alphanumeric value here>
  at this.get(./node_modules/@walletconnect/core/dist/index.es.js:1:16678)
  at this.decode(./node_modules/@walletconnect/core/dist/index.es.js:1:18736)
  at this.decode(./node_modules/@walletconnect/core/dist/index.es.js:1:18798)
  at <anonymous>(./node_modules/@walletconnect/core/dist/index.es.js:1:47319)
  at ReflectApply(./node_modules/@walletconnect/core/dist/index.es.js:1:47537)
  at EventEmitter.prototype.emit(./node_modules/events/events.js:158:1)
  at index_es_Ot.onMessageEvent(./node_modules/@walletconnect/core/dist/index.es.js:1:36928)

This seems to be coming from WalletConnect, but more specifically it seems to be caused by how @walletconnect/ethereum-provider is used (based on this comment)?

The latest version of Wagmi includes upgraded versions of WalletConnect that seem to be more stable. Let me know if you're still seeing this problem.

Yes @DanielSinclair , still getting this issue with latest rainbowkit 1.0,7 and wagmi 1.3.9

Yes @DanielSinclair , still getting this issue with latest rainbowkit 1.0,7 and wagmi 1.3.9

I am also getting same error with latest version

I tried clearing my local storage of react app and it worked for me.

are there any updates on this issue? if not could some one tell me where I could at least catch the error?

facing same issues aswell ! any update on this ?

same error here w/ the latest
"@rainbow-me/rainbowkit": "^1.0.8",
"wagmi": "^1.3.10"

Hi everyone this was discussed here. And it seems to be an issue related to the wallet implementation side.


High water stress index

The water stress factor is reaching values of almost 1 (high stress) for an experiment in Nigeria

During those days the soil water content is in wilting point, why?

Those layers have soil water contents (SW) below wilting point (LL):


_TEXT_SECTION_NAME_2 on ARM Cortex-R

Describe the bug
ARM platforms define a _TEXT_SECTION_NAME_2 which ends up in the final binary.
This isn't right. We just want one text section, if stuff needs to be aligned internally we can do that within the text section itself.
Delete the exception for _TEXT_SECTION_NAME_2 in scripts/sanitycheck.

As far as I could see this has little to do with alignment @andrewboie , it is rather related to the code relocation work done in #10892
I wonder if we could simply rename the initial TEXT Section that contains the vector table to something else.

I wonder if we could simply rename the initial TEXT Section that contains the vector table to something else.

@wentongwu do you think we can do the above?

I wonder if we could simply rename the initial TEXT Section that contains the vector table to something else.

@wentongwu do you think we can do the above?

@ioannisg I think we can rename this section, maybe vector_table_section.

I wonder if we could simply rename the initial TEXT Section that contains the vector table to something else.

@wentongwu do you think we can do the above?

@ioannisg I think we can rename this section, maybe _vector_table_section.

@wentongwu @andrewboie  I am ok with that. Actually, at least 2 other architectures, Risc-V and Nios2 have a similar thing, that is, several sections at the beginning of ROM (vector, reset exception section) followed by the text section.
This means that __image_rom_start and __image_text_start will be different. The whole thing seems trivial; I'll send a PR and we can take the discussion there. I would like to make sure we are not breaking anything by doing that.

@ioannisg sounds right -- what we really have here is a special rodata section immediately before the actual program text.
Thinking out loud: could we just reverse the order of the text and rodata sections? So what we have now is:
vector table and other special rom
text section
.ARM.extab (more special ROM)
.ARM.exidx (more special ROM)
rodata section

Could we just do
rodata section (containing vector table + extab/exidx + zephyr rodata)
text section

@ioannisg I will look into this on Monday.


Automated Package Upgrade

:tada: This PR is included in version 8.1.15 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[fix-#14174] sudo.enable = false   not work

Purpose of the pull request
this close : #14174

#14174

Brief change log
Verify this pull request
This pull request is code cleanup without any test coverage.

@zhuangchong @zhongjiajie PTAL

You can compare the changes of this pr #13860
if not add this judge ,The task has failed when it is submitted to the worker.

Codecov Report

:exclamation: No coverage uploaded for pull request base (3.0.6-prepare@a626b4c). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head 751b7e6 differs from pull request most recent head 598f51d. Consider uploading reports for the commit 598f51d to get more accurate results

@@               Coverage Diff                @@
##             3.0.6-prepare   #14176   +/-   ##
================================================
  Coverage                 ?   40.32%
  Complexity               ?     4643
================================================
  Files                    ?      889
  Lines                    ?    35252
  Branches                 ?     3901
================================================
  Hits                     ?    14214
  Misses                   ?    19649
  Partials                 ?     1389

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

SonarCloud Quality Gate failed.
  25 Bugs   14 Vulnerabilities   4 Security Hotspots   805 Code Smells
 35.5% Coverage  5.8% Duplication

BTW, the sonar result seems not right. :eyes:


Can't get sublimetextwebinspector working

I wonder if its ok to post a question here about getting sublimetextwebinspector working. I've been trying for several days. If this is the wrong place to post such a question could someone point me to the right place?  The sublime text forum have been down the whole time. Are they ever up?
If its OK to ask, whenever I to try click on "Start Google Chrome with remote debug port 9222" I get the following error message.
[Errno 13] Permission denied
[cmd: ['/Applications/Google Chrome.app', '--remote-debugging-port=9222', '--profile-directory=Default','']]
[dir: /Users/caseymandell/Dropbox/Documents/sublimeProjects/Experiments/src/scripts]
[path: /usr/bin:/bin:/usr/sbin:/sbin]
[Finished]
I have no real expertise in these matters at all. From searching the web I've gleaned that one of the first things to do is see if my browser will run  http://<IP_ADDRESS>:9222/json.  None of my browsers, (Firefox, Aurora, Chrome, Canary or Safari) will run this url which must be telling, but after quite a bit of searching, I can't find out what it means or what to do about it. If there's any other information I need to furnish please tell me what it is and I'll gladly furnish it. I'm dying to use the debugger. My swi.sublime-settings file follows. (I'm on a Mac.)
{
"chrome_path":{
//"osx" : "/Applications/Google Chrome Canary.app --remote-debugging-port=9222 --no-first-run --no-default-browser-check --user-data-dir=$(mktemp -d -t 'chrome-remote_data_dir'",
//"osx": "/usr/sbin/Google Chrome Canary.app",
"osx":"/Applications/Google Chrome.app",
//"osx" : "Applications/Google Chrome.app/Contents/MacOS/Google Chrome",
"windows": "C:\Program Files\Google\Chrome\chrome.exe",
"linux": "/usr/bin/google-chrome"
},
"chrome_profile": "Default",
"chrome_remote_port": "9222",
"breakpoint_scope": "swi.breakpoint",
"current_line_scope": "swi.current",
"interactive_scope": "mcol_0088CCFF.settings",
"stack_layout": {
"cols": [0.0, 0.6, 1.0],
"rows": [0.0, 0.7, 1.0],
"cells": [[0, 0, 2, 1], [0, 1, 1, 2], [1, 1, 2, 2]]
},
"console_layout": {
"cols": [0.0, 0.6, 1.0],
"rows": [0.0, 0.7, 1.0],
"cells": [[0, 0, 2, 1], [0, 1, 1, 2], [1, 1, 2, 2]]
},
"reload_on_start": false,
"reload_on_save": true,
"reload_timeout": 0,
"user_agent": "",
"set_script_source": false,
"open_stack_current_in_new_tab": false,
"breaks": {}
}

My resolution for this was to get the settings file setup correctly. #74 shows my resolution for this.

@casemanx you seem to have commented out the path to the chrome executable.
In OSX applications with the '.app' extension are actually just folders. The actual executable is in a specific location in that folder.
So your line
"osx":"/Applications/Google Chrome.app"
should actually be
"osx": "/Applications/Google Chrome.app/Contents/MacOS/Google Chrome"
so that is actually points at the chrome executable.

Thanks you very much for your reply! I'm super busy right now, but I'll
give it a try.
Sincerely,
Casey
Daniel Weber wrote:

@casemanx https://github.com/casemanx you seem to have commented out
the path to the chrome executable.
In OSX applications with the '.app' extension are actually just
folders. The actual executable is in a specific location in that folder.
So your line
|"osx":"/Applications/Google Chrome.app"|
should actually be
|"osx": "/Applications/Google Chrome.app/Contents/MacOS/Google Chrome"|
so that is actually points at the chrome executable.

Reply to this email directly or view it on GitHub
https://github.com/sokolovstas/SublimeWebInspector/issues/78#issuecomment-70850213.

Since there were no more comments I'm going to assume this solved the problem.


Bandito makes monkey sounds

Description
title. prolly just placeholder, but still a bit weird
Reproduction

pet bandito
ook ook
Screenshots

Additional context

Although it sounds like a monkey, the noise Bandito makes is in fact, a ferret's call :)
See https://youtu.be/th3_zv-5UVQ?si=xM2k2eVrcNKnYYUH&t=205

hooray for ferrets

Peak scrungle content


Async overloads for AnsiConsole Prompt/Ask/Confirm.

@patriksvensson ptal.

Please upvote :+1: this pull request if you are interested in it.

@patriksvensson ptal.

@tmds Sorry for taking time with this; currently a bit behind on the backlog. Will try to take a look at this tonight.

This is very useful. Would love to see it merged.

I'll take a look at this @tmds

Dear @spectreconsole/maintainers, what's our policy on unit test coverage when adding Async overloads to existing functionality.
The code in this PR is clear, well structured, and high-quality. However, it doesn't contain any unit tests.
We have extensive unit test coverage of prompts here: test/Spectre.Console.Tests/Unit/Prompts, however they exercise the current (non-async) calls.
I don't think it makes sense to duplicate the 100-something existing unit tests for these async extensions, however equally I don't want to merge this until I've checked it's ok 'as-is' without tests.
What are your thoughts?

This is very useful. Would love to see it merged.

@mbursill Do you have a code example of how you'd like to use the async overloads, out of interest?

@tmds - what is the usage for an async prompt? I need some help understanding why a synchronous user prompt should be extended to be async. I'm not saying there isn't a need, but I need a working code snippet illustrating the motivation behind this PR please.

@tmds - what is the usage for an async prompt? I need some help understanding why a synchronous user prompt should be extended to be async. I'm not saying there isn't a need, but I need a working code snippet illustrating the motivation behind this PR please.

The CancellationToken I'm passing is the one from https://github.com/dotnet/command-line-api that informs me the user has requested the app to terminate via Ctrl+C. As part of the termination, I want some things to get cleaned-up/Disposed.

@tmds ok thanks for replying after so much time. Apologies, the widgets aren't my usual area of expertise and the other maintainers are quite busy to ask. I noticed your PR which looked like a good quality contribution and taken it on myself to review/merge. I think I understand the usage now. I do plan to get this one over the line.

No worries, @FrankRay78. Thanks for picking this up!

@FrankRay78 Unit tests are preferred so we'll know if we introduce regressions. But it's your call.

We've been discussing with @FrankRay78 and I've been experimenting with an alternative approach to this issue. I think the async APIs actually don't make sense but the sync APIs need cancellation support. I've been working on an obsolete-async-prompt-apis branch where I have added cancellation support for all the prompt APIs.
@tmds Would you mind checking out this branch, build from source and see if this approach solves your problem? If you need help building from source I can assist.

I think the async APIs actually don't make sense but the sync APIs need cancellation support.

Can you explain why?
Because the CancellationToken gets triggered asynchronously, I don't think a sync implementation can do something "better" than an async one. I think the only difference is whether the caller gets non-blocking async behavior or whether the thread blocks when there is no input.

Because the CancellationToken gets triggered asynchronously, I don't think a sync implementation can do something "better" than an async one. I think the only difference is whether the caller gets non-blocking async behavior or whether the thread blocks when there is no input.

You're right, I'll scrap my branch and start over with only adding cancellation support to the synchronous APIs (which is still needed) but without touching the asynchronous APIs which are fine.

I don't think it makes sense to duplicate the 100-something existing unit tests for these async extensions, however equally I don't want to merge this until I've checked it's ok 'as-is' without tests.
What are your thoughts?

[!NOTE]
The tests that matter for this pull request are in src/Tests/Spectre.Console.Tests/Unit/AnsiConsoleTests.Prompt.cs instead of src/Tests/Spectre.Console.Tests/Unit/Prompts/*.cs and their count is closer to 3 than to 100. 😉

What about modifying the existing tests like this, from fact to theory with both sync and async paths? I think it's still pretty readable.
Before:
[Fact]
public void Should_Return_Correct_DateTime_When_Asked_US_Culture()
{
    // Given
    var console = new TestConsole().EmitAnsiSequences();
    console.Input.PushTextWithEnter("2/1/1998");

    // When
    var dateTime = console.Ask<DateTime>(string.Empty, CultureInfo.GetCultureInfo("en-US"));

    // Then
    dateTime.ShouldBe(new DateTime(1998, 2, 1));
}

After:
[Theory]
[InlineData(true)]
[InlineData(false)]
public async Task Should_Return_Correct_DateTime_When_Asked_US_Culture(bool async)
{
    // Given
    var console = new TestConsole().EmitAnsiSequences();
    console.Input.PushTextWithEnter("2/1/1998");

    // When
    DateTime dateTime;
    if (async)
        dateTime = await console.AskAsync<DateTime>(string.Empty, CultureInfo.GetCultureInfo("en-US"));
    else
        dateTime = console.Ask<DateTime>(string.Empty, CultureInfo.GetCultureInfo("en-US"));

    // Then
    dateTime.ShouldBe(new DateTime(1998, 2, 1));
}

@tmds - unfortunately, @0xced has been called away.  So I'm going to put my name down on this, with an intention of getting the PR over the line.  I'm about to pull down the branch and take a look. Short of more significant prompt refactoring @0xced had in mind, it looks like a few unit test changes is all that's required for this PR.  Do you agree, and if so, are you able to help with them? I can review and merge, when appropriate.

This is now top of my list to get over the line @tmds.
I'll take a look at what unit test coverage we need, then it should be a trivial matter of updating your branch and pushing.
Then we can welcome you into the Spectre.Console git commit history 👏

hi @FrankRay78 , sorry for the unresponsiveness, it was a very busy time for me. If you can provide some guidance on the unit test coverage, we can try to get this merged by the end of next week.

I'm traveling, but I'd wager the attributes for AOT that were added might be at fault. I don't have a computer but I'm happy to answer any questions or review

Hello @phil-scott-78, do you know if as a maintainer, whether I should be able to resolve the conflict of another users fork/branch, then force push it back? Or if the user will always need to do that themselves? The contribution is fully reviewed and good to merge.

@FrankRay78, I've done it in the past and I've had maintainers fix my stuff, but the submitting user needs to enable it on their end.
At the end of the day, I always feel like as long as the user's original commit is still in the history plus a maintainer commit with the needed touch up, then it's all good.

the submitting user needs to enable it on their end.

afaik it is enabled by default and you should be able to push to my branch.
I've resolved the conflicts by merging in the main branch.
I assume you're going to squash this PR when merging. If not, let me know if you want me to clean up the commit history.

Thanks for your help, @FrankRay78!


Accordian Menu [EN] Feedback for: /tools/starter-kit/index.html

Not an issue as such, but many menus for sites contain unordered lists within unordered lists in the menu with an accordion effect so that menu items can be hidden beneath main categories.
Having an example of such a menu in the web starter kit will help a lot of people develop their site much quicker.
I look forward to new versions of the web starter kit

Please file this in the Polymer Starter Kit repo
https://github.com/polymerelements/polymer-starter-kit


Fix spec time-bomb

late_generator_spec (now overdue_user_notification_list_spec)
stopped running on 1st March, because we weren't creating tasks for the
test period window, so the factories were creating tasks with a
period_month last_month, hence creating an expiring test.
Create overdue user spec tasks with an explicit period_month, and also
fix dates to Jan 2019 in the tasks factory such that it's harder to
accidentally create expiring tests.

Yes, ummed and ahhed over it, because neither solution is nice. It is safe but leaves one feeling mildly deflated. Perhaps I should have ameliorated that feeling by offering a :current trait.

Yep, I was thinking the same thing. A current trait would probably be handy as the code to set that up is relatively non-trivial (at least to do in a way that is concise and easy to follow). My plan was to drop that in at some point when it seemed necessary or appropriate.

Here you go. #291

(while I still have the context, best to not waste it)


Parts of my configuration.yaml are ignored (mqtt). How the heck can that be??

Description of the issue
I am just migrating from standalone z2m to the add-on.
I copied my previous config and it definitely works because it finds the devices in my network.
However, it totally ignores what I have for MQTT server and always wants to connect to core-mosquitto:1883:

$ head /config/zigbee2mqtt/configuration.yaml
homeassistant: {}
mqtt:
  base_topic: zigbee2mqtt
  server: mqtt://<IP_ADDRESS>:1884
  reject_unauthorized: false
  user: z2m
  password: *********
serial:

How on earth can that be? Again, the file is definitely correct because it successfully connects to my ser2net port which I have configured in the same file...
Addon version
1.37.1-1
Platform
Core: 2024.3.3
Supervisor: 2024.05.1
Operating System: 12.2
Frontend: 20240307.0
Logs of the issue (if applicable)
No response

Facing the same issue.
From log:
[2024-05-23 08:49:27] info: 	z2m: Zigbee: disabling joining new devices.
[2024-05-23 08:49:27] info: 	z2m: Connecting to MQTT server at mqtt://core-mosquitto:1883
[2024-05-23 08:49:27] error: 	z2m: MQTT error: Connection refused: Not authorized
[2024-05-23 08:49:27] error: 	z2m: MQTT failed to connect, exiting...
[2024-05-23 08:49:27] info: 	z2m: Stopping zigbee-herdsman...
But the configuration got:
mqtt:
base_topic: zigbee2mqtt,
server: "core-mosquitto-test"
user: "a"
password: "a"

Home Assistant addon MQTT settings are taken from the addon settings page

I am facing a similar issue when changing the base_topic from zigbee2mqtt to zigbee2mqtt_garage in the addon settings. Z2M is completely ignoring it and sending the messages to the standard topic zigbee2mqtt.
This is the MQTT section in the settings of the addon:
mqtt:
  base_topic: zigbee2mqtt_garage

facing the same issue since upgrade to 1.38
everything was fine before in 1.37, I also have a renamed base_topic, since the upgrade it just dont care.
is there an override somewhere?

Trying to understand why such settings from the addon are not working, @Koenkk is it ok to not have base_topic listed here in schema validation ? https://github.com/zigbee2mqtt/hassio-zigbee2mqtt/blob/master/zigbee2mqtt/config.json#L57

@JSteunou the base_topic can be added indeed (since it needs to be configurable before z2m starts). Could you make a PR for that?


feat(appmetadata): metadata will be in root

Coverage remained the same at 52.557% when pulling 206bf27af4554f4d2d25ab38751ea9cb12009038 on appnewfields into 9fb6ca990ece248afd5e6975991eb8615b2c63f2 on master.

Coverage remained the same at 52.557% when pulling 206bf27af4554f4d2d25ab38751ea9cb12009038 on appnewfields into 9fb6ca990ece248afd5e6975991eb8615b2c63f2 on master.


Where Will the Ball Fall

Problem Link
https://leetcode.com/problems/where-will-the-ball-fall/
Coding platform
Leetcode
Programming language

[ ] C
[x] C++
[x] Java
[ ] Python

Additional context (OPTIONAL)
I want to solve it in Java

I would like to solve this in C++

Hi @Nivedita967
I would like to solve this issue in Java Or Python. Could you please assign it to me?

Sure @abhinav5481 Go for Python.


Crash at "ready" function.

Godot version
4.0 beta 14
System information
Windows 11, MacOS also
Issue description
Callstack:

CrashHandlerException: Program crashed
Engine version: Godot Engine v4.0.beta.custom_build (4db3716d8d9bbc95c031818b190002602558b0f2)
Dumping the backtrace. Please include this when reporting the bug to the project developer.
[0] <couldn't map PC to fn name>
[1] <couldn't map PC to fn name>
[2] GDScriptFunction::call (C:\Users\xx\Documents\godot\modules\gdscript\gdscript_vm.cpp:2000)
[3] GDScriptInstance::callp (C:\Users\xx\Documents\godot\modules\gdscript\gdscript.cpp:1838)
[4] Node::_notification (C:\Users\xx\Documents\godot\scene\main\node.cpp:155)
[5] Object::notification (C:\Users\xx\Documents\godot\core\object\object.cpp:792)
[6] Node::_propagate_ready (C:\Users\xx\Documents\godot\scene\main\node.cpp:189)
[7] Node::_propagate_ready (C:\Users\xx\Documents\godot\scene\main\node.cpp:179)
[8] Node::_propagate_ready (C:\Users\xx\Documents\godot\scene\main\node.cpp:179)
[9] Node::_propagate_ready (C:\Users\xx\Documents\godot\scene\main\node.cpp:179)
[10] Node::_set_tree (C:\Users\xx\Documents\godot\scene\main\node.cpp:2595)
[11] SceneTree::initialize (C:\Users\xx\Documents\godot\scene\main\scene_tree.cpp:411)
[12] OS_Windows::run (C:\Users\xx\Documents\godot\platform\windows\os_windows.cpp:1288)
[13] widechar_main (C:\Users\xx\Documents\godot\platform\windows\godot_windows.cpp:181)
[14] _main (C:\Users\xx\Documents\godot\platform\windows\godot_windows.cpp:205)
[15] main (C:\Users\xx\Documents\godot\platform\windows\godot_windows.cpp:217)
[16] __scrt_common_main_seh (D:\a_work\1\s\src\vctools\crt\vcstartup\src\startup\exe_common.inl:288)
[17] <couldn't map PC to fn name>
-- END OF BACKTRACE --

These crashes are not caused by GDExtension codes from my test.
Steps to reproduce
Just launch game.
Minimal reproduction project
mini_project.zip

Related? https://github.com/godotengine/godot/issues/71483

Related? #71483

I'm not sure. When I comment "_ready" function in my original project. It will crash for similar reason.

@RicardRC  I can still reproduce it at beta 16. Maybe it is not same issue with #71483

I was working at beta 12. Because my project has alot of tasks. So I have little time to fix it sorry.

From my test. First crash happened when I call a method from gdtextension. And this method has a parameter which type is "Object*", I modify it:

Crash passed. And Then I meat another crash which is very simular with first crash. In gdscript I called a method:

which parameter is a custom "Resource" data. I'm a little confused.
@Chaosus @RicardRC  Is it caused by by my wrong parameter type?

From my test. This issue appears after godot: "7502e1dc0da15334f88911a802fa6ba919b6c83c" and godot-cpp: "992d85e6f845d798ab26237fe52d1aa6461452c5" submitions. Mutex in object is not initailized maybe. I haven't figer out why does  Ref reference sent back a object with wrong mutex.

From my test. This issue appears after godot: "7502e1dc0da15334f88911a802fa6ba919b6c83c" and godot-cpp: "992d85e6f845d798ab26237fe52d1aa6461452c5" submitions. Mutex in object is not initailized maybe. I haven't figer out why does Ref reference sent back a object with wrong mutex. @BastiaanOlij Would you please help me for this issue if you have time?

I reset these changes back. But crash still there.

When I change all parameters type of C++ functions which will be called at gdscript "lifetime functions" (_ready, _process and so on) from Object*, Ref to Variant. Crashes are gone.:>. But I think there has something wrong  with passing parameters from gdscript to C++ function.

Can you still reproduce this in 4.0.3 and 4.1-beta3 or later?

Sorry to replay later. It was fixed at godot-cpp repor


Ir emitter stopped working on Lenovo X1 gen 9.

Output of linux-enable-ir-emitter boot status
× linux-enable-ir-emitter.service - enable the infrared emitter
     Loaded: loaded (/lib/systemd/system/linux-enable-ir-emitter.service; enabled; vendor preset: enabled)
     Active: failed (Result: exit-code) since Tue 2023-01-03 08:24:15 CET; 1min 2s ago
    Process: 13024 ExecStartPre=/sbin/modprobe uvcvideo (code=exited, status=0/SUCCESS)
    Process: 13025 ExecStart=/usr/bin/linux-enable-ir-emitter run (code=exited, status=1/FAILURE)
   Main PID: 13025 (code=exited, status=1/FAILURE)
        CPU: 31ms

Jan 03 08:24:15 steffen-popos systemd[1]: Starting enable the infrared emitter...
Jan 03 08:24:15 steffen-popos linux-enable-ir-emitter[13025]: ERROR: Impossible to execute the driver at /etc/linux-enable-ir-emitter/video2.
Jan 03 08:24:15 steffen-popos systemd[1]: linux-enable-ir-emitter.service: Main process exited, code=exited, status=1/FAILURE
Jan 03 08:24:15 steffen-popos systemd[1]: linux-enable-ir-emitter.service: Failed with result 'exit-code'.

Output of cat /etc/linux-enable-ir-emitter/*
device=/dev/video2
unit=14
selector=6
size=9
control0=1
control1=3
control2=2
control3=0
control4=0
control5=0
control6=0
control7=0

Give more information if you have

Ir emitting have been working perfectly till it suddenly dident. I had just done a firmware update when it stopped working, but after downgrading the firmware the problem seemed to remain.
Additional info

Distro: pop_OS
Version: 22.04
Version of linux-enable-ir-emitter: linux-enable-ir-emitter 4.1.5
How did you install linux-enable-ir-emitter:  ppa

Did you try to reconfigure the camera? Can you paste the outputs of sudo linux-enable-ir-emitter -v -d /dev/video2 configure
And also, ensure that the infrared camera still on /dev/video2

I don't know what fixed the problem, but it seem to be working as expected now


Owin broke Crowbar

As mentioned in issue #3, Owin breaks Crowbar. The stacktrace from that issue is pretty clear that Owin is the cause. I spent sometime trying to figure out why and hoped to find a fix. So far things look grim. The problem arises in the following code from System.Web's HttpResponse:
public NameValueCollection Headers
{
  get
  {
    if (!(this._wr is IIS7WorkerRequest))
      throw new PlatformNotSupportedException(SR.GetString("Requires_Iis_Integrated_Mode"));
    if (this._headers == null)
      this._headers = new HttpHeaderCollection(this._wr, this, 16);
    return (NameValueCollection) this._headers;
  }
}

In order for Owin to create its context, it needs to grab the response headers. The problem is HttpResponse won't allow access to the headers if the current HttpWorkerRequest isn't of type IIS7WorkerRequest. Crowbar is calling HttpRuntime.ProcessRequest programmatically and passing in it's own custom HttpWorkerRequest, CrowbarHttpWorker. But CrobarHttpWorker inherits from SimpleWorkerRequest, not IIS7WorkerRequest and so the exception in the code snippet above gets thrown. It would have been an easy enough fix to simply have CrowbarHttpWorker inherit from IIS7WorkerRequest but IIS7WorkerRequest is sealed and internal. I've looked at other projects similar to Crowbar like CasiniDev but they use the same idea of calling HttpRuntime.ProcessRequest with a custom HttpWorkerRequest so we can assume they won't work with Owin either.
Owin is not required when using MVC5 but Microsoft is promoting Owin. They rewrote FormsAuthentication as an Owin Middleware and all of their security modules are written as Owin Middleware. I understand you're busy so I'd love to help out on getting Crowbar working with Owin. I'm fresh out of ideas but if you have any then please let me know and I'll take care of researching and spiking them out. Oh, and great tool by the way :-)

Any update on this? Where are we with MVC 5?

MVC 5 support is currently not being worked on.