PowerShell 6.2 window often just closes immediately after launch

After upgrading to PowerShell 6.2, from 6.1.3, and removing PowerShell 6 Preview (6.2 RC 1), PowerShell often will not start.  The window appears, powershell looks like its starting, and then the window closes.    Several attempts are usually needed to get a window to successfully get to a prompt.
All the update/uninstall process was done at one time.  I have two different machines that seem to be exhibiting this issue, but I don't have any that are not.
Environment data
Name                           Value
----                           -----
PSVersion                      6.2.0
PSEdition                      Core
GitCommitId                    6.2.0
OS                             Microsoft Windows 10.0.17763
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

My other machine is running the most recent Windows 19H1 Insiders Preview.
I am also experiencing a lot of crashing in VS Code's PowerShell 2.0.1 extension with certain documents open, but at least PowerShell 6.2 usually opens fine in the 'integrated console' of the extension.  May or may not be related, but I am not seeing anyone posting on that topic either.
Let me know if there is any other data I can include.
Note I originally had trouble with earlier updates to the preview releases, see #8442.  I have not reviewed the condition there as in this case, PowerShell does usually open if I keep trying.

It may be related to VS Code or the PowerShell extension on VS Code.   After rebooting my system, I was able to immediately open PowerShell with no issue.  With VS Code open, non-administrative sessions of PowerShell refuse to stay open.  Even after closing VS Code, PowerShell seems to refuse reliably opening.

You could run PowerShell Core from cmd.exe and see an error message.

When I try to open PWSH from CMD at prompt, PowerShell starts fine.  Even with that session of PowerShell open, attempting to open it from a taskbar icon results in it not opening.  I also tried typing PWSH in an Explorer address bar and get same no-open result.
By not opening, this is what happens.   A window pops up, and has the header info:
PowerShell 6.2.0
Copyright (c) Microsoft Corporation. All rights reserved.

https://aka.ms/pscore6-docs
Type 'help' to get help.

The command prompt never appears, and after a short delay, the window closes.
If I leave the system for a while (minutes, maybe 10 minutes, but I do continue doing other tasks on the system) then opening PowerShell works fine.   I've been able to kind of repeat the pattern.  After PowerShell is opening fine, I close VS Code (if it was even open), wait a bit, reopen VS Code, wait a bit (maybe a minute) and then PowerShell fails opening again, and this remains again for some period of time.
I tried repeating all this on my 19H1 insiders machine last night, and it would not repeat past the first time.  The first time opening PowerShell failed, but every subsequent time worked, even after a reboot.

When I try to open PWSH from CMD at prompt,

Always?
If you open PowerShell Core home directory and double click on pwsh.exe - does it start well in any time?

When I try to open PWSH from CMD at prompt,

Always?

So far.   I would try a PWSH window, it would fail, I would open CMD, type PWSH, it started fine, try another PWSH window, it would fail.  EXIT the CMD instance of PWSH and try again, it would open just fine again.  I'd keep repeating, closing CMD window entirely, starting over, still PWSH at a CMD prompt seems to be always working.
Also, I will not completely swear, that even after an instance of a PWSH window successfully opens, attempting again a few minutes later and it won't open again, without any particular reason I can so far deduce.

If you open PowerShell Core home directory and double click on pwsh.exe - does it start well in any time?

Yes, it seems so.  This is weird though, because directly clicking on EXE file seems to result in the same no-open condition most the time, but clicking on the short-cut stored in the start menu folder seems more sporadic, often opening successfully, but definitely not always.

Windows OS keeps a window parameters for every (console) application. It seems the parameters is broken and we need clean up them.

I'm experiencing this too. If it helps, the following appears in the Event Viewer:
Source: .NET Runtime
Application: pwsh.exe
CoreCLR Version: 4.6.27317.3
Description: The process was terminated due to an internal error in the .NET Runtime at IP 00007FF93A6B298E (00007FF93A6B0000) with exit code 80131506.

Source: Application Error
Faulting application name: pwsh.exe, version: <IP_ADDRESS>, time stamp: 0x5c410542
Faulting module name: coreclr.dll, version: 4.6.27317.3, time stamp: 0x5c40be0f
Exception code: 0xc0000005
Fault offset: 0x000000000000298e
Faulting process id: 0x4f48
Faulting application start time: 0x01d4f30cd62ddac2
Faulting application path: C:\Program Files\PowerShell\6\pwsh.exe
Faulting module path: C:\Program Files\PowerShell\6\coreclr.dll
Report Id: 532d875c-683f-4640-bf93-310d5de00cb4
Faulting package full name:
Faulting package-relative application ID:

Just like @cpmcgrath, I can confirm that I see the same thing in the application event log immediately after a failure to launch:
(2 events)
Application: pwsh.exe
CoreCLR Version: 4.6.27317.3
Description: The process was terminated due to an internal error in the .NET Runtime at IP 00007FFFDDAB298E (00007FFFDDAB0000) with exit code 80131506.

Faulting application name: pwsh.exe, version: <IP_ADDRESS>, time stamp: 0x5c410542
Faulting module name: coreclr.dll, version: 4.6.27317.3, time stamp: 0x5c40be0f
Exception code: 0xc0000005
Fault offset: 0x000000000000298e
Faulting process id: 0xc60
Faulting application start time: 0x01d4f3a7d85fc982
Faulting application path: C:\Program Files\PowerShell\6\pwsh.exe
Faulting module path: C:\Program Files\PowerShell\6\coreclr.dll
Report Id: 2d4862a8-7e13-4eeb-8d60-4ec7e2e74f30
Faulting package full name:
Faulting package-relative application ID:

I am still seeing this corelated to opening VS Code with the PowerShell Preview Extension, but I suppose other applications that use .Net could set up this condition as well.

@msftrncs Could you please compile and run debug build to get more info? Also it would be nice to get repo steps from you so that we can reproduce the problem on clean system.

I am experiencing this same issue.
Source: Application Error
Faulting application name: pwsh.exe, version: <IP_ADDRESS>, time stamp: 0x5c410542
Faulting module name: coreclr.dll, version: 4.6.27317.3, time stamp: 0x5c40be0f
Exception code: 0xc0000005
Fault offset: 0x000000000000298e
Faulting process id: 0x4e64
Faulting application start time: 0x01d520380945a490
Faulting application path: C:\Program Files\PowerShell\6\pwsh.exe
Faulting module path: C:\Program Files\PowerShell\6\coreclr.dll
Report Id: 0fe828fd-a45e-4dc2-b7f0-77b07ecdba93
Faulting package full name:
Faulting package-relative application ID:

Source: .NET Runtime
Application: pwsh.exe
CoreCLR Version: 4.6.27317.3
Description: The process was terminated due to an internal error in the .NET Runtime at IP 00007FFC0240298E (00007FFC02400000) with exit code 80131506.

I've attached the information collected by Windows Error Reporting.
Report.zip
Observations:

I am running Windows v1903 (OS Build 18362.30)
I do have Visual Studio Code installed (v1.35.0) with the Powershell extension (v2019.5.0)
It seems to happen after a reboot
Profile load time does seem to take a while
It sometimes happens in Adminsitrative mode, and sometimes not
I have the postgit module installed as part of my profile, which is defined below:

Import-Module posh-git

# Customise the Prompt
$GitPromptSettings.DefaultPromptPrefix.Text = '$((Get-Date).ToShortTimeString()) ';
$GitPromptSettings.DefaultPromptPrefix.ForegroundColor = [ConsoleColor]::Magenta;
#$GitPromptSettings.DefaultPromptSuffix.Text = ' $(GitVersionForCurrentRepoPrefixed)`r`nλ ';
$GitPromptSettings.DefaultPromptSuffix.Text = '`r`nλ ';

@Antaris Thanks! Is your repo persistent? Can you repo with latest PowerShell Core build?

It is not persistent, more intermittent. I'll try and download the latest build and see if that fixes it.

@Antaris If you can please fork the repo and build - with debug mode you could get more information in exception.
@SteveL-MSFT  Should we be worried about 6.2 version?

Judging by faulting module name I have a feeling that is .NET problem. Similar thing happened before.
Meanwhile, here is a simple way to get a dump of the crashing process with all details that will help greatly to investigate and fix. @Antaris - you can try ProcDump utility:
Register as the Just-in-Time (AeDebug) debugger. Makes full dumps in c:\dumps.
C:\>procdump -ma -i c:\dumps

The next time PS crashes, the dump with detailed error info will be written to the specified folder.

@anmenaga
I managed to tricky this experience this morning. Fresh re-start of my laptop. The first time I opened PowerShell Core, it was fine. I then used Visual Studio Code to read my PS Core profile, and initiated a save without making alterations. I then opened PowerShell Core and it immediately closed.
Dump file is here:
https://drive.google.com/file/d/1KKeS3co8rE3w1xi3cFUPT21notKSmudT/view?usp=sharing
@iSazonov Sorry, I haven't had time to test this against the current commit, I'll try and get that done this week

It looks like the Access violation is coming from under TaskbarJumpList.CreateElevatedEntry.
@bergmeister , do you recall any stability issues with this function?
ExceptionAddress: 00007fff471f298e (coreclr!InteropSyncBlockInfo::GetRCWAndIncrementUseCount+0x000000000000000a)
   ExceptionCode: c0000005 (Access violation)
  ExceptionFlags: 00000001
NumberParameters: 2
   Parameter[0]:<PHONE_NUMBER>000000
   Parameter[1]:<PHONE_NUMBER>000018
Attempt to read from address<PHONE_NUMBER>000018

Partial stack:
coreclr!InteropSyncBlockInfo::GetRCWAndIncrementUseCount+0xa
coreclr!RCWHolder::Init+0x16
coreclr!ComObject::SupportsInterface+0x101
coreclr!ObjIsInstanceOf+0x482
coreclr!JITutil_ChkCastAny+0x95
coreclr!JITutil_ChkCastInterface+0x2e
Microsoft_PowerShell_ConsoleHost!Microsoft.PowerShell.TaskbarJumpList.CreateElevatedEntry(System.String)+0x386
Microsoft_PowerShell_ConsoleHost!Microsoft.PowerShell.ConsoleHost+<>c.<Start>b__4_0()+0x14

That would explain why this doesnt happen if I launch pwsh from the command line.

We have seen sporadic failures of it before and other people tried to improve the COM calls. In this case I rather suspect that the used Windows Insider build 19H1 might be a culprit as well.
I ported the original C++ code from Windows PowerShell to C# using the same COM calls and used the Windows Api pack source code for the COM interface definitions.
The good news is that this code only runs when pwsh starts interactively and that the jumplist menu registration needs to happen only once (there is code that checks that implicitly). Maybe we should put a try{}catch block around it and only log out the failure stack trace?

@bergmeister are you willing to take on this work?  Please :)

I can try, the WPF API will need something like an added boolean parameter (defaulting to false) to make the jumplist entry open the application in elevated mode. It will depend on how flexible the WPF guys are in terms of willing to accept the API change and how hard it is to get a PR merged but I think it will be race again time because .Net Core 3 wants to publish an RC around July (meaning that they are less likely to accept such changes after that time)....

It seems a race condition in GUI.
@SteveL-MSFT It is not clear about 6.2 version -  should we fix it too ?

Without any data to back up my next statement, I seems this issue almost never happens if I open PowerShell with 'Run as Administrator'

@jlouros I've experienced both running non-elevated, and running as an Adminstrator

Does this happen for PS 7-preview1 as well? .Net Core 3 has improved COM interaction, it might be an issue of .Net Core 2.1.
Also: Maybe a similar improvement to the code similar to PR #7580 could help as well?

@bergmeister, I have had it happen with 7 preview as well, but much less often.   7 preview seems to give a CLR error dump, but it closes so quickly …  but I got it:

I also don't experience this nearly as much when running 'as administrator', as @jlouros mentions, but as @Antaris said, it does still ocassionally happen.

We could get more information with debug build (we ignore errors in release build in the code!)

I opened an API proposal in WPF to get a sign off first (the proposed change is non-breaking, so it should be OK) and included some low level implementation details. I have not looked at the WPF code itself in detail but it should not be too difficult (the hardest part might be the testing)
https://github.com/dotnet/wpf/issues/950

@bergmeister I think for now, perhaps the interim solution is to wrap it in try...catch so that pwsh at least starts.  We can take that fix to 6.2.x servicing.  The proper fix can come later.

@SteveL-MSFT Are you sure a fatal CLR exception can be caught?

@bergmeister I can't repro this myself.  I think the best option might be as you suggest to build a private and ask someone who can repro it to try it out.

I see that we ignore return codes in the code that can be bad.

Looking at the COM interface definitions, they are a bit fishy and certainly not what you usually would see.

I've never seen anyone add [MethodImpl(MethodImplOptions.InternalCall, MethodCodeType = MethodCodeType.Runtime)] to COM interop code
You are returning HResult structs (I assume structs, didn't look for the definition), but without [PreserveSig] attributes the interop layer will add another output HRESULT to the argument list so you expect the function returning two distinct HRESULTs, which most likely isn't true and may be responsible for some of the crashes
returning HResult wrapper structs is actually not valid for the COM calling convention. Desktop Framework had a long standing bug which allowed it, but .NET Core "fixed" this bug causing such code to crash. They reverted the fix for compatibility reasons. See dotnet/coreclr#23974 for a technical discussion of the calling convention problem.

I don't know which .NET Core versions have problems with HRESULT-struct-returning COM methods, but it certainly will be one cause for your crashes if you are running on the "wrong" .NET Core version. Forgetting to annotate methods with [PreserveSig] probably is another cause of crashes. Anyways, your interop code looks strange enough that it probably should be reviewed. I can try taking a look if you don't have anyone with COM interop experience around.

@weltkante  Interesting, I took the interface definitions from the original Windows API pack (see e.g. here) and was not aware of this. I am not an expert in this area but if you think your changes make it better, please submit a PR.

@Antaris  @jlouros @msftrncs @cpmcgrath
In PR #9896, I added a global catch around it and added logging to got more information out
Please download the MSI from the build of that PR.
If you are not familiar with the system just use the direct link to the build here and click on the button on the top right, and then click on Artifacts and then the artifacts submenu. This will download you a zip and in there is the MSI, which is a custom version of PowerShell 7 from the lastest commit in master.

Please report if it fixes it and if not, please provide the log messages that get printed to the PowerShell console. This is just a test build and not intended for any supported use.

@bergmeister I've installed that Preview7 MSI and will get back to you as I try and recreate the issue, and run it for a couple of days as my main shell 👍

I see that we ignore return codes in the code that can be bad.

@iSazonov where do you see that? I just did a review of the COM interface declarations as well as the calling CreateElevatedEntry method and couldn't find anything obviously wrong, at least in the methods which are actually used. It looks like all methods declaring PreserveSig are checking their return codes (methods returning void and not declaring PreserveSig have their return code checked by the interop layer).
Anyways, not checking return codes cannot lead to AccessViolationException in coreclr. If the stacktrace from @anmenaga above is representative for the issue then it could very well be another bug in the coreclr interop layer (there already have been a few)
If your investigation doesn't lead anywhere it might be worth having someone from coreclr having a look at the dump. The stacktrace being in internal coreclr code really doesn't look very much like calling into wrongly coded interop declarations. In particular when in the coreclr RCWHolder helper class the line m_pSB->GetInteropInfoNoCreate()->GetRCWAndIncrementUseCount(); returns NULL for the middle call of GetInteropInfoNoCreate then it didn't really get very far yet. Because that is what happened in the dump.

I means
                if (hResult < 0)
                {
                    Debug.Fail($"BeginList on ICustomDestinationList failed with HResult '{hResult}'.");
                    return;
                }

In release build we ignore fail results.

@iSazonov ah ok, this doesn't really ignore the failure result, it just doesn't give out diagnostics. It still returns immediately, meaning the attempt to create the JumpList entry is aborted without doing any more COM calls. Doing a return certainly can't lead to the kind of crash we're seeing here.

I just had a look at WPF's implementation and its code checks that it is in STA apartment state. WPF threads are all in STA so I am not sure if this check is due to WPF or if STA is more suited for the COM calls that we make (PowerShell Core, even 7, is in MTA mode due to .Net Core historically not supporting STA):
https://github.com/dotnet/wpf/blob/ae1790531c3b993b56eba8b1f0dd395a3ed7de75/src/Microsoft.DotNet.Wpf/src/PresentationFramework/System/Windows/Shell/JumpList.cs#L564

Usually COM handles the apartment for you, if you aren't on STA then CoCreateInstance will create a dedicated STA thread for the COM objects if they are registered as only running on STA.

@Antaris Thanks. P.S.: If you see a message of the format Exception '{exception}' with stack trace {exception.StackTrace} successfully caught. please let us know because that would mean that the catch statement was able to catch the fatal CLR error (which I doubt at the moment).

@bergmeister I haven't been able to replicate the issue in the last few days. I'll keep at it, as I use PowerShell constantly.
I have also not encountered any exceptions.
For reference, my spec machine is:
Dell XPS 15 9570
Intel Core i7-8750H - 6 Core
32GB RAM

That is good but have you seen a log in the console saying
Exception '{exception}' with stack trace {exception.StackTrace} successfully caught.
Only if you've seen it then it would be the proof that the fatal CLR error could be caught.

I can reproduce this bug every time if I try to run PowerShell. I will describe in 2 different Scenario, in the first one, it is working without any problems on the other hand with the second one not.
If needed i can provide more logs or outputs.
My Hardware :
Lenovo Y520
Intel I7 7700HQ ( 2.8Ghz )
16GB Ram
Samsung 512 970Pro SSD
WesternDigital WD10SPCX 1TB HDD
Nvidia 1050TI GPU 4GB
My Software :
OS :
M$ Windows 10 - Version 1903 - OsBuild 18917.1000
VS Code :
Version: 1.36.0-insider (system setup)
Commit: 68a7e5bc437b38d0281df0756997a25da2a2900c
Date: 2019-06-18T18:40:22.519Z
Electron: 4.2.4
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.31-electron.0
OS: Windows_NT x64 10.0.18917
PowerShell:
$PSVersionTable
Name                           Value

PSVersion                      7.0.0-preview.1
PSEdition                      Core
GitCommitId                    7.0.0-preview.1
OS                             Microsoft Windows 10.0.18917
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0
Scenarios :
Scenario1 :
Steps :
1-try to right click a folder and select PowerShell preview 7 -> open here
Result :
works perfectly without any problem
Scenario2 :
Steps :
1-open a VS Code in an pipenv ( virtualenv) folder ( Right click a folder/directory and select Open with Code Insider )
2-wait until it initialize virtualenv
3-try to right click same folder and select PowerShell preview 7 -> open here ( or try to run it via shortcut )
Result :
Gives this error  and close :
Internal CLR error. (0x80131506)
at Microsoft.PowerShell.TaskbarJumpList.CreateElevatedEntry(System.String)
at Microsoft.PowerShell.ConsoleHost+<>c.b__4_0()
at System.Threading.Tasks.Task.InnerInvoke()
at System.Threading.Tasks.Task+<>c.<.cctor>b__274_0(System.Object)
at System.Threading.ExecutionContext.RunFromThreadPoolDispatchLoop(System.Threading.Thread, System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object)
at System.Threading.Tasks.Task.ExecuteWithThreadLocal(System.Threading.Tasks.Task ByRef, System.Threading.Thread)
at System.Threading.Tasks.Task.ExecuteEntryUnsafe(System.Threading.Thread)
at System.Threading.Tasks.Task.ExecuteFromThreadPool(System.Threading.Thread)
at System.Threading.ThreadPoolWorkQueue.Dispatch()
at System.Threading._ThreadPoolWaitCallback.PerformWaitCallback()

@usta Can you please describe what you mean by 'pipenv (virtualenv)', I do not use Python very much. Since you say you can always reproduce, please read my comment below, install the custom build and please report if it fixes it:
https://github.com/PowerShell/PowerShell/issues/9295#issuecomment-502053584

@usta Can you please describe what you mean by 'pipenv (virtualenv)', I do not use Python very much. Since you say you can always reproduce, please read my comment below, install the custom build and please report if it fixes it:
#9295 (comment)

It looks like this one have fixed that bug ( if I have hit the same problem I will mention it here ) thanks

I cannot seem to reproduce it with the custom build, but it was pretty rare with 7 preview 1, yet 6.2 does it very regularly.   There might be some usefulness in applying a patch to a custom build of 6.2, or a series of builds from 6.2.  Just adding the debug/try/catch in might have changed something such a timing related resource lock up or something.

cc @TravisEz13 @adityapatwardhan we should consider taking @bergmeister's change to 6.2.2

@bergmeister can we reopen this bug please ?
Because i have hit same bug today again  (with the one you have mentioned ( the custom build ) )
(or maybe a different one)
GetStartupInfo
CoCreateInstance
BeginList
SetValue
Commit
CoCreateInstance
AddObject
Internal CLR error. (0x80131506)
at Microsoft.PowerShell.TaskbarJumpList.CreateElevatedEntry(System.String)
at Microsoft.PowerShell.ConsoleHost+<>c.b__4_0()
at System.Threading.Tasks.Task.InnerInvoke()
at System.Threading.Tasks.Task+<>c.<.cctor>b__274_0(System.Object)
at System.Threading.ExecutionContext.RunFromThreadPoolDispatchLoop(System.Threading.Thread, System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object)
at System.Threading.Tasks.Task.ExecuteWithThreadLocal(System.Threading.Tasks.Task ByRef, System.Threading.Thread)
at System.Threading.Tasks.Task.ExecuteEntryUnsafe(System.Threading.Thread)
at System.Threading.Tasks.Task.ExecuteFromThreadPool(System.Threading.Thread)
at System.Threading.ThreadPoolWorkQueue.Dispatch()
at System.Threading._ThreadPoolWaitCallback.PerformWaitCallback()
PS C:\Windows\System32> $PSVersionTable
Name                           Value

PSVersion                      7.0.0-preview2.25651
PSEdition                      Core
GitCommitId                    7.0.0-preview2.25651
OS                             Microsoft Windows 10.0.18922
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

@usta Yes, I agree, we should re-open (I do not have the rights though but will notify maintainers). Thank you very much for reporting and providing the log, now we know that the problem happens on this line (should it happen again but with a different log where AddObject is not the last line before the crash let us know though):
https://github.com/PowerShell/PowerShell/blob/86a1697da95974495fc8f08bc58932d0b4020603/src/Microsoft.PowerShell.ConsoleHost/WindowsTaskbarJumpList/TaskbarJumpList.cs#L89-L90
@daxian-dbw @SteveL-MSFT @iSazonov  This means that despite initial positive reports that PR #9928 is not able to catch the fatal CLR error that has been reported multiple times (all bug reports reported the same error). Whilst there was still some value in doing a global catch for something that is not essential, the problem still persists... With the additional information I could try to look into seeing if I code more defensively to avoid getting to this point where this issue happens (which I suspect to be a bug in the coreclr itself, is it worthwhile opening an issue there?). Otherwise I can only think of disabling this feature via e.g. an environment variable (or storing some info if the jumplist has been populated once already since it persists afterwards, not sure if it persists after Windows updates though...) Let me know what you think/prefer.
On a related note, I opened the following issue and PR in WPF to add the required APIs to radically simplify the code/responsibility in this repo going forward but so far the PR has not been reviewed and the issue has been tagged with .Net 5....:
https://github.com/dotnet/wpf/issues/950
https://github.com/dotnet/wpf/pull/996

@bergmeister now i tried again and again to reproduce same error.
At the end i may reproduce it but this time it give anohter log:
GetStartupInfo
CoCreateInstance
BeginList
SetValue
Commit
CoCreateInstance
AddObject
Exception 'System.Runtime.InteropServices.InvalidComObjectException: COM object that has been separated from its underlying RCW cannot be used.
at System.StubHelpers.InterfaceMarshaler.ConvertToNative(Object objSrc, IntPtr itfMT, IntPtr classMT, Int32 flags)
at Microsoft.PowerShell.ComInterfaces.ICustomDestinationList.AddUserTasks(IObjectArray poa)
at Microsoft.PowerShell.TaskbarJumpList.CreateElevatedEntry(String title)
at Microsoft.PowerShell.ConsoleHost.<>c.b__4_0()' with stack trace    at System.StubHelpers.InterfaceMarshaler.ConvertToNative(Object objSrc, IntPtr itfMT, IntPtr classMT, Int32 flags)
at Microsoft.PowerShell.ComInterfaces.ICustomDestinationList.AddUserTasks(IObjectArray poa)
at Microsoft.PowerShell.TaskbarJumpList.CreateElevatedEntry(String title)
at Microsoft.PowerShell.ConsoleHost.<>c.b__4_0() successfully caught.
PS C:\Users\usta>

If needed I will be glad to test another build or try to provide other logs ( which might need to overcome this bug )

@bergmeister
I am not even a m$ technology programmer but just because of curiosity isn't this line also needs an extra check before passing to AddUserTasks ?
( https://github.com/PowerShell/PowerShell/blob/86a1697da95974495fc8f08bc58932d0b4020603/src/Microsoft.PowerShell.ConsoleHost/WindowsTaskbarJumpList/TaskbarJumpList.cs#L87 )
I mean isn't that might be better something like this :
hResult = pShortCutCollection.AddObject((IShellLinkW)nativePropertyStore);
if (hResult < 0)
{
pShortCutCollection.Clear();
Debug.Fail($"Couldnt add nativePropertyStore to  Collection :  HResult '{hResult}'.");
return;
}

Is it possible to detect if the jump list already exists (persists), and skip the steps to build it in that case?

@usta no, AddObject is declared as returning void and thus the interop layer will do the check (and throw an exception)
@msftrncs I don't know of any managed API which exposes reading of jumplists, so I suspect that the native API doesn't even support it, but I didn't check. In any way, just checking for existence is wrong, it would cause jumplists to become outdated if one of their properties changed.
In either way if you can't fix the problem here (and I seriously doubt adding try/catch for a coreclr panic is going to work nor a good idea) you should probably raise an issue with coreclr for further examination. You do have a dump to examine after all. The scenario in the dump certainly looks like a coreclr bug to me.

I opened an issue with all the details in the CoreClr repo, maybe they can help us. I had a look at the code and there is defensive way of querying an API if the jumplist has already been created, we only get the maximum number of available slots available and already return early if there is not enough space for the jumplist, all we could do is cache the jumplist internally or have something like a feature flag to skip the problematic jumplist creation code (once the jumplist entry has been created, it persists).
https://github.com/dotnet/coreclr/issues/25502

@bergmeister I have tried latest daily build and it looks like in this build i couldnt reproduce bug.
Whether it is fixed or the reason if that bug gone.
PS C:\Users\usta> echo $PSVersionTable
Name                           Value

PSVersion                      7.0.0-dailypreview2.26796
PSEdition                      Core
GitCommitId                    7.0.0-dailypreview2.26796
OS                             Microsoft Windows 10.0.18922
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Everyone: The CoreClr team has investigated the issue and the problem is that the COM APIs that are being called are strictly STA only (PS Core currently still operates in MTA until #7216 is resolved) and the coreclr did not provide any warnings/errors. Therefore I've pushed changes to create the JumpList in an STA thread, which should hopefully be the final resolution.
Please re-test with the latest build of PR #9896

Everyone: The fix for this issue has been backported to the recent release of 6.2.2, please update and provide feedback if it fixed it. Users of the preview will have to wait for 7.0.0-preview.2
https://github.com/PowerShell/PowerShell/releases/tag/v6.2.2
cc @msftrncs @usta @Antaris @cpmcgrath @jlouros


overflow:auto as an option?

I'd prefer that the scrollbars only appear when there is enough content to overflow. Would it be possible to enable this?
Thanks!

Yeah. I'll see if I have some time soon to fix this.
For now, to work around it, add a child that takes 100% height and provides its own scrollbars when needed.

I can whip up a crude PR if it helps?

Sorry, have been really busy. A PR would help. We could probably just hardcode it to be auto?

Done, I'll close this.
Thanks, really liking react-sidebar!


[BUG] Instructions for archiving, signing and notarizing the app needs to be updated

Describe the bug
The instructions for archiving, signing and notarizing the app does not match the steps in the latest version of Xcode.
To Reproduce
Try to perform steps 2-10 in the guide in Xcode 15.

Thanks for catching this, we will update the wiki page!

Documentation has been updated.
https://github.com/IBM/mac-ibm-notifications/wiki/Archive,-sign-and-notarize-the-app


Folder command?

When I use the command folder, liko12 freeze.
Is it a shortcut to open the folder in windows ?
\AppData\Roaming\LOVE\liko12\data

Strange issue, it's caused by the löve framework. What I do is ask love about the appdata folder path, join it with the current directory in liko12 terminal, then open it by openning the url "file://" + finalPath
could you provide some extra info about your platform (os, cpu, gpu, etc..), I will be available to help you at sunday

yes, it's a shortcut, you can do it manually, it's for lazy people who don't know how to find the appdata folder.

Thank you for your answer, I have found the problem, it is me. My graphic card bug a lot lately and will die soon I am sure. I deactivated my graphic card and it work fine.


dotnet 8 support in isolated mode

The selected version '.NET 7 Isolated' will deprecate on 5/14/2024. Please consider updating it soon.Learn more

Thanks for the remainder, @lkurzyniec , will certainly take care of this.

Seriously? This needs to be high prio! .Net 7 is a Maintenance release that many cannot use.

Hello @mronnblom-ith , to clarify: .NET versions are backward-compatible.
So nothing prevents you to inject the current DfMon Isolated package into your .NET 8 projects (in which case it is .NET 8 that will be actually used, of course).

Ok, that is reasonable. It is probably the AspNet Core integration incompatibility that is causing my problems then, not the .Net version.

I'm using standalone isolated version, and it's not working with .NET 8 Isolated .NET version of Function App.

@lkurzyniec , can you please provide more details? How exactly is it not working?

I have a working version on .NET 7 Isolated.

When I changed to .NET 8 Isolated, it stops working.

Error details
Encountered an error (InternalServerError) from host runtime.

@lkurzyniec , I confirm there seems to be an issue like that on a Linux platform. When deploying an empty .NET Isolated project targeting .NET 7 and then changing .NET version to 8 in the instance's settings, the app fails to work until you change TargetFramework to net8.0 in CSPROJ file and redeploy.
On a Windows instance (the one which you'd get by deploying the included ARM template) this does not happen though - DfMon Isolated works fine.

When creating it from scratch in .NET 8 Isolated, the problem is the same.

Anyway, this 6.5.0-beta2 package should work (as it targets .NET 8).
Can you please check?

It's working 🚀
Now I can wait for a stable release of the package, and then I will close that issue.
Thx for that 💪🏼

It would be nice to bump Azure.Identity as version 1.10.4 is vulnerable.

@scale-tone here is PR: https://github.com/microsoft/DurableFunctionsMonitor/pull/179

Thanks a lot for your PR, @lkurzyniec !
Typical trouble with massive package bump-ups is that it often breaks things, sometimes in very unexpected places. E.g. here is an example of Microsoft.IdentityModel.Protocols.OpenIdConnect.
And sadly here in DfMon, with all its incarnations, all these numerous ways of running it, all various platforms and pricing tiers, we do not have a 100% reliable auto-tests.  Please, give me some time for that...

Got it. I didn't think that upgrading the version could be so risky and painful. I thought it is a matter of building the projects.

@scale-tone any ETA for a new stable version?

Done


Fix underscore replacement error for heatmap

This has been temporarily fixed for heatmap with pull request #51
The same issue was fixed for scatterplot tab previously.


Use a custom function in xpath

Hi,
I would like to use a custom function in xpath, but I don't see any clear documentation on how to do that.
Let's say that I want to use a custom match for regular expressions. How can I write one?
I looked at StaticContext, but it is not clear for me.
Thanks,
Alex.

I have prepared very small example with one custom user function called "factorialize":
    const staticContext = xpath.createStaticContext();

    staticContext.signatures	= {};
    var cXSDecimal = require('xpath2.js/lib/types/schema/simple/atomic/XSDecimal');

   // add our custom function to staticContext
    staticContext.functions["factorialize"]	= (oValue) =>{

        var num = global.Math.round(oValue[0].textContent);
        var result = num;
        if (num === 0 || num === 1)
          return 1;
        while (num > 1) {
          num--;
          result *= num;
        }

        return new cXSDecimal(result);
    };

    // add parameters to this function
    staticContext.signatures["factorialize"]	=  [[cXSDecimal, '?']];

    const parser = new DOMParser();
    const document = parser.parseFromString('<data ><some>5</some></data>', "application/xml");

    let result = xpath.evaluate ("factorialize(data/some)",document ,staticContext);
    console.log(result[0]);  // output to console is 120


[HC] Corrects typo

Summary
Corrects typo.
Documentation checklist

[ ] The documentation style guide has been adhered to.

Closes https://github.com/cloudflare/cloudflare-docs/issues/17587


Unable to re-enable Persistent Search Bar

I disabled the persistent search bar to have more room on my home screen, but ended up wanting it back. Tried to Re-Enable persistent search bar, and the launcher restarts without adding it back to the homescreen.

Thanks for reporting, I will fix soon!!

It is fixed?

Can confirm that on my device (OnePlus One, running Android 7.1.1) it is possible to disable and then again enable the persistent search bar.


chore: allow tailing both stdout and stderr & log when actor shuts down

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#589  👈 (View in Graphite)
#588
#587
#586
#585
#584
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Remove version: FutureScholars.Paperlib version 2.1.2

All urls in manifest result in 404s, no longer available
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/GeoffreyChen777/paperlib/releases/download/release-electron-2.1.2/Paperlib_2.1.2.exe
Response status code does not indicate success: 404 (Not Found).
(Automated message - build 727)


CLOSE added to all cells in a tableView

I use the ExpandableLabel in a tableview, when I press on MORE in one of the CELL then all my cells have the red CLOSE in them, even those that didn't need to expand

I can't reproduce this problem. Have a look at the demo project, especially the ViewController class. You need to track the open/close state of your cells in your code and set it on the ExpandableLabel when dequeueing reusable cells.


[Question]The disk has already been force-formatted. Current file system is ext4, You can format it manually.

I found that my system does not seem to recognize my hard disk, and the prompt is as follows:
The disk has already been force-formatted. Current file system is ext4, You can format it manually.

Hi @983183947,
That's weird.
Even if the disk is formatted, we should detect it.
Could you provide the support bundle or log of the harvester-node-disk-manager?
And what harvester version you use?


Ignite Thin Client - connection timeout exception

Deadlock in Perper Fabric caused by looped continuous listeners on "streams" cache (from main Program and also from the launched Services - Re-Invoke and BindOutput). Deadlock caused socket to become unresponsive and lead to timeout in Perper Functions.

Fixed in 6fe754d


feat(providers): adding Solana fungible price support

Description
This PR adds Solana support for the fungibles pricing endpoint by using the SolScan Token price API endpoint for the SOL native token and SolScan Token Metadata endpoint for the SPL tokens.
How Has This Been Tested?

A new integration test was implemented.

Due Diligence

[ ] Breaking change
[ ] Requires a documentation update
[ ] Requires a e2e/integration test update

As the further optimization caching of the get_token_info method response will be implemented as a follow-up to save on SolScan provider calls since we can cache the tokens metadata for 24 hours (I don't think metadata changes often) and for 1-2 minutes for the price calls.


Change exp to a peerDependency

[x] move exp to peerDeps

closes #23

@jpdriver can you relax the semver constraint to e.g. >48 and add a note to installation instructions to install exp as well?


Documentation request

Hi. Would you consider adding documentation on how to install cm-rgb from the github repo? (Currently you only point out the possibility of installing via pip, but I was curious about the new temp monitor feature)

Hi,
I'll add some info as it's pretty easy.

Clone repository
Enter the directory
Run pip install .

Folder contains setup.py
This should be sufficient, however since I don't have dev branch it may be sometimes unstable.
If you have previously installed cm-rgb try following:

Clone repository
Enter the directory
Run for example ./scripts/cm-rgb-monitor --show-temp --temp-source k10temp-pci-00c3/Tdie --print-values --interval=1

Let me know if this helped you.

Easier than I could have thought. Thanks for pointing the path out. You changed the option names you had originally documented on the readme, though. Whereas the readme gives the following line cm-rgb-monitor --cpu-color=#FFA500 --bg-color=#00FFFF --show-sensor --low-color=#00FF00 --high-color=#FF0000, now the option name is ---show-temp and --temp-low-color and --temp-high-color, so that the command that actually runs is /cm-rgb-monitor --cpu-color=#FFA500 --bg-color=#00FFFF --show-temp --temp-low-color=#00FF00 --temp-high-color=#FF0000

You are right, I was hesitant to update the readme before I have build published as most people install from pip and wont have this function yet.
In the mean time I've added rainbow and cycle support for the ring if you are interested :)
I've also set default interval to 0.02 as this does not flicker in my opinion.
Could you try this interval and give me your opinion ?

I've pulled the newest commit and I'm running cm-rgb-monitor right now. Yes, I do notice less flickering, but now the AMD logo is a blinking light red. CPU temperature is being shown by the fan itself, right?

Oh, and I'm glad the rainbow is back. I couldn't restore it through a Win10 VM and had already come to terms with the fact that I would never see it again.

I've added wiki page https://github.com/gfduszynski/cm-rgb/wiki/Installation-&-Configuration


AttributeError: 'WSGIRequestHandler' object has no attribute 'environ'

Got this
INFO:zerotest.record.http_recorder:recorder service start, store http record to 'octocat.data'
INFO:werkzeug: * Running on http://<IP_ADDRESS>:7000/ (Press CTRL+C to quit)
----------------------------------------
Exception happened during processing of request from ('<IP_ADDRESS>', 43451)
Traceback (most recent call last):
  File "/usr/lib/python2.7/SocketServer.py", line 295, in _handle_request_noblock
    self.process_request(request, client_address)
  File "/usr/lib/python2.7/SocketServer.py", line 321, in process_request
    self.finish_request(request, client_address)
  File "/usr/lib/python2.7/SocketServer.py", line 334, in finish_request
    self.RequestHandlerClass(request, client_address, self)
  File "/usr/lib/python2.7/SocketServer.py", line 655, in __init__
    self.handle()
  File "/usr/local/lib/python2.7/dist-packages/werkzeug/serving.py", line 215, in handle
    rv = BaseHTTPRequestHandler.handle(self)
  File "/usr/lib/python2.7/BaseHTTPServer.py", line 340, in handle
    self.handle_one_request()
  File "/usr/local/lib/python2.7/dist-packages/werkzeug/serving.py", line 249, in handle_one_request
    elif self.parse_request():
  File "/usr/lib/python2.7/BaseHTTPServer.py", line 286, in parse_request
    self.send_error(400, "Bad request syntax (%r)" % requestline)
  File "/usr/lib/python2.7/BaseHTTPServer.py", line 364, in send_error
    self.log_error("code %d, message %s", code, message)
  File "/usr/local/lib/python2.7/dist-packages/werkzeug/serving.py", line 271, in log_error
    self.log('error', *args)
  File "/usr/local/lib/python2.7/dist-packages/werkzeug/serving.py", line 277, in log
    _log(type, '%s - - [%s] %s\n' % (self.address_string(),
  File "/usr/local/lib/python2.7/dist-packages/werkzeug/serving.py", line 265, in address_string
    return self.environ['REMOTE_ADDR']
AttributeError: 'WSGIRequestHandler' object has no attribute 'environ'

for curl "https://localhost:7000/api/2/storelist/10117" --insecure
Any ideas what is that about?

The address localhost:7000 is a HTTP local proxy, can you try curl "http://localhost:7000/api/2/storelist/10117" and paste result?

here it is:
curl: (35) Unknown SSL protocol error in connection to localhost:7000
with the same error

Seems like a werkzeug bug, you send https request to http proxy server, werkzeug cannot parse request and return 400 # this line    self.send_error(400, "Bad request syntax (%r)" % requestline) , werkzeug try log the error but request environ is not init.
Are you sure change the https scheme to http, and see this error again?

Sorry, I missed this part, you are right it happens only for https


The corpus file used to be evaluated

Hi,
When I use the scripts to evaluate the performance, I find there is a corpus file needed by the code. I download the files from the 20news homepage, but the result is not the same as the result file. Could you share the corpus file？
Best

Same issue here
I'm using Jey Han Lau's wiki corpus and his script here https://github.com/jhlau/topic_interpretability/blob/master/run-oc.sh
Note that he used 20 words sliding window in his script.
I did a run on the 2nd group of topic here and got results that are substantially different than yours @akashgit
https://github.com/akashgit/autoencoding_vi_for_topic_models/blob/master/coherence_from_paper_script/AVTIM_50
[0.07] ( 0.07; ) apartment woman neighbor jesus armenians tear daughter soldier hide afraid
[0.26] ( 0.26; ) nhl hockey wings rangers montreal calgary leafs winnipeg angeles detroit
[0.18] ( 0.18; ) bike amp helmet kit honda turbo gear brake rear engine
[0.14] ( 0.14; ) privacy conduct security enforcement electronic policy encryption states americans agency
[0.19] ( 0.19; ) gun crime criminal assault weapon handgun batf violent abuse firearm
[0.07] ( 0.07; ) myers president stephanopoulos decision meeting congress february package secretary community
[0.17] ( 0.17; ) hitter pitcher baseball defensive braves player bike pitch fan career
[0.23] ( 0.23; ) season nhl hockey league player coach puck pitcher team playoff
[0.14] ( 0.14; ) bike honda turbo helmet gear dealer saturn rear engine amp
[0.10] ( 0.10; ) homicide weapon firearm handgun vancouver knife minority shall gun militia
[0.08] ( 0.08; ) cancer firearm committee volume amendment handgun health states united patient
[0.11] ( 0.11; ) windows gateway swap port window printer modem setup hd mouse
[0.23] ( 0.23; ) muslims islam jews genocide israel jewish muslim islamic jew turks
[0.21] ( 0.21; ) armenian armenia armenians village muslim turks israel lebanon azerbaijan turkish
[0.02] ( 0.02; ) entry output file winner variable oname buf abuse char io
[0.23] ( 0.23; ) god jesus christian revelation moral doctrine resurrection principle faith christianity
[0.15] ( 0.15; ) motherboard meg hd mb mw slot simm ram adapter port
[0.09] ( 0.09; ) azerbaijan armenian armenians apartment neighbor troops father soviet town hide
[0.15] ( 0.15; ) xterm menu client font swap workstation resource server unix directory
[0.06] ( 0.06; ) db mov bh cs dos byte ax connector hd adapter
[0.02] ( 0.02; ) db det mov tor bh que wm mw pit byte
[0.04] ( 0.04; ) entry buf output variable io oname winner char stream printf
[0.11] ( 0.11; ) anonymous probe privacy mission lunar electronic cipher satellite solar launch
[0.12] ( 0.12; ) motherboard mhz shipping mw printer simm slot adapter meg quadra
[0.15] ( 0.15; ) encryption enforcement serial chip encrypt wiretap escrow clipper agency device
[0.17] ( 0.17; ) jesus marriage prophet christ islam god scripture marry verse prophecy
[0.19] ( 0.19; ) bike manual helmet brake honda turbo gear engine rear ford
[0.23] ( 0.23; ) belief god existence faith christ christian doctrine truth teaching christianity
[0.19] ( 0.19; ) vga dos printer isa adapter pc windows ram scsus scsi
[0.22] ( 0.22; ) scsus scsi ide controller bio mb interface floppy ram rom
[0.22] ( 0.22; ) braves playoff season nhl puck hockey leafs wings cup coach
[0.06] ( 0.06; ) administration president stephanopoulos secret island congress russia government libertarian escrow
[0.11] ( 0.11; ) font xt toolkit colormap vendor export server xterm widget directory
[0.17] ( 0.17; ) turks israel israeli armenia turkish village arab armenian genocide greek
[0.10] ( 0.10; ) workstation wiring anonymous null server xterm directory platform ftp toolkit
[0.08] ( 0.08; ) tor det nhl hockey pit que winnipeg league leafs detroit
[0.23] ( 0.23; ) doctrine jesus god christ revelation scripture heaven atheist christian religious
[0.18] ( 0.18; ) season defensive playoff puck flyers coach league braves team penalty
[0.22] ( 0.22; ) pitch hitter player pitcher score baseball season defensive team braves
[0.04] ( 0.04; ) pt rg eus pd pp calgary philadelphia winnipeg detroit bhj
[0.15] ( 0.15; ) turkish armenian turks village genocide jews muslim murder israel greek
[0.13] ( 0.13; ) gun criminal knife crime gang cop batf violent insurance weapon
[0.13] ( 0.13; ) privacy enforcement encryption security americans device secure escrow rsa conversation
[0.22] ( 0.22; ) windows font binary os pc rom vga cache server hardware
[0.16] ( 0.16; ) connector ide scsi scsus quadra motherboard hd mb meg isa
[0.19] ( 0.19; ) motherboard quadra slot mhz hd meg adapter processor simm ide
[0.22] ( 0.22; ) ide controller scsi scsus floppy isa mb meg motherboard connector
[0.20] ( 0.20; ) jesus faith belief verse god scripture passage satan eternal interpretation
[0.22] ( 0.22; ) ram cache swap mb pc mac windows vga os scsi
[0.08] ( 0.08; ) anonymous wiring privacy wire outlet protocol unix nec ripem ftp
Average Topic Coherence = 0.148
Median Topic Coherence = 0.151

BTW, I don't really question the validity of the results from the paper, I really enjoyed reading it. Just want to make sure we are all using the same method to evaluate ;)

If you run topic model on 20News, the reference corpus to compute topic coherence is 20News accordingly. You'll get confused result if you use different corpus to get topic coherence.

Hi,
I used the 2nd group of topic extracted from coherence_from_paper_script/AVTIM_50, and then calculated the topic coherence score using Jey Han Lau's wiki corpus. What I got likes:
[0.01] ( 0.01; ) apartment woman neighbor jesus armenians tear daughter soldier hide afraid
[0.00] ( 0.00; ) nhl hockey wings rangers montreal calgary leafs winnipeg angeles detroit
[0.00] ( 0.00; ) bike amp helmet kit honda turbo gear brake rear engine
[0.02] ( 0.02; ) privacy conduct security enforcement electronic policy encryption states americans agency
[0.01] ( 0.01; ) gun crime criminal assault weapon handgun batf violent abuse firearm
[0.00] ( 0.00; ) myers president stephanopoulos decision meeting congress february package secretary community
[0.02] ( 0.02; ) hitter pitcher baseball defensive braves player bike pitch fan career
[0.06] ( 0.06; ) season nhl hockey league player coach puck pitcher team playoff
[0.00] ( 0.00; ) bike honda turbo helmet gear dealer saturn rear engine amp
[0.00] ( 0.00; ) homicide weapon firearm handgun vancouver knife minority shall gun militia
...
[0.02] ( 0.02; ) windows font binary os pc rom vga cache server hardware
[0.00] ( 0.00; ) connector ide scsi scsus quadra motherboard hd mb meg isa
[0.00] ( 0.00; ) motherboard quadra slot mhz hd meg adapter processor simm ide
[0.00] ( 0.00; ) ide controller scsi scsus floppy isa mb meg motherboard connector
[0.00] ( 0.00; ) jesus faith belief verse god scripture passage satan eternal interpretation
[0.00] ( 0.00; ) ram cache swap mb pc mac windows vga os scsi
[0.00] ( 0.00; ) anonymous wiring privacy wire outlet protocol unix nec ripem ftp
==========================================================================
Average Topic Coherence = 0.008
Median Topic Coherence = 0.000
@dingran How did you run the topic coherence script, and was the same result as reported in the paper obtained when using 20news as the reference corpus?
What I got when using 20news as the reference corpus:
[0.03] ( 0.03; ) apartment woman neighbor jesus armenians tear daughter soldier hide afraid
[0.00] ( 0.00; ) nhl hockey wings rangers montreal calgary leafs winnipeg angeles detroit
[0.08] ( 0.08; ) bike amp helmet kit honda turbo gear brake rear engine
[0.08] ( 0.08; ) privacy conduct security enforcement electronic policy encryption states americans agency
[0.11] ( 0.11; ) gun crime criminal assault weapon handgun batf violent abuse firearm
[0.02] ( 0.02; ) myers president stephanopoulos decision meeting congress february package secretary community
...
[0.04] ( 0.04; ) connector ide scsi scsus quadra motherboard hd mb meg isa
[0.05] ( 0.05; ) motherboard quadra slot mhz hd meg adapter processor simm ide
[0.11] ( 0.11; ) ide controller scsi scsus floppy isa mb meg motherboard connector
[0.06] ( 0.06; ) jesus faith belief verse god scripture passage satan eternal interpretation
[0.07] ( 0.07; ) ram cache swap mb pc mac windows vga os scsi
[0.04] ( 0.04; ) anonymous wiring privacy wire outlet protocol unix nec ripem ftp
==========================================================================
Average Topic Coherence = 0.062
Median Topic Coherence = 0.064

the corpus files are here: autoencoding_vi_for_topic_models/data/20news_clean/

If you run topic model on 20News, the reference corpus to compute topic coherence is 20News accordingly. You'll get confused result if you use different corpus to get topic coherence.

I am also curious which corpus refrence they have used.
Based on your comment, you mean for evaluating 20 news group data set they have used 20 news group data set as the external corpus?
I thought its not valid to do like this. Am I missing something here?

Hi,
I used the 2nd group of topic extracted from coherence_from_paper_script/AVTIM_50, and then calculated the topic coherence score using Jey Han Lau's wiki corpus. What I got likes:
[0.01] ( 0.01; ) apartment woman neighbor jesus armenians tear daughter soldier hide afraid
[0.00] ( 0.00; ) nhl hockey wings rangers montreal calgary leafs winnipeg angeles detroit
[0.00] ( 0.00; ) bike amp helmet kit honda turbo gear brake rear engine
[0.02] ( 0.02; ) privacy conduct security enforcement electronic policy encryption states americans agency
[0.01] ( 0.01; ) gun crime criminal assault weapon handgun batf violent abuse firearm
[0.00] ( 0.00; ) myers president stephanopoulos decision meeting congress february package secretary community
[0.02] ( 0.02; ) hitter pitcher baseball defensive braves player bike pitch fan career
[0.06] ( 0.06; ) season nhl hockey league player coach puck pitcher team playoff
[0.00] ( 0.00; ) bike honda turbo helmet gear dealer saturn rear engine amp
[0.00] ( 0.00; ) homicide weapon firearm handgun vancouver knife minority shall gun militia
...
[0.02] ( 0.02; ) windows font binary os pc rom vga cache server hardware
[0.00] ( 0.00; ) connector ide scsi scsus quadra motherboard hd mb meg isa
[0.00] ( 0.00; ) motherboard quadra slot mhz hd meg adapter processor simm ide
[0.00] ( 0.00; ) ide controller scsi scsus floppy isa mb meg motherboard connector
[0.00] ( 0.00; ) jesus faith belief verse god scripture passage satan eternal interpretation
[0.00] ( 0.00; ) ram cache swap mb pc mac windows vga os scsi
[0.00] ( 0.00; ) anonymous wiring privacy wire outlet protocol unix nec ripem ftp
==========================================================================
Average Topic Coherence = 0.008
Median Topic Coherence = 0.000
@dingran How did you run the topic coherence script, and was the same result as reported in the paper obtained when using 20news as the reference corpus?
What I got when using 20news as the reference corpus:
[0.03] ( 0.03; ) apartment woman neighbor jesus armenians tear daughter soldier hide afraid
[0.00] ( 0.00; ) nhl hockey wings rangers montreal calgary leafs winnipeg angeles detroit
[0.08] ( 0.08; ) bike amp helmet kit honda turbo gear brake rear engine
[0.08] ( 0.08; ) privacy conduct security enforcement electronic policy encryption states americans agency
[0.11] ( 0.11; ) gun crime criminal assault weapon handgun batf violent abuse firearm
[0.02] ( 0.02; ) myers president stephanopoulos decision meeting congress february package secretary community
...
[0.04] ( 0.04; ) connector ide scsi scsus quadra motherboard hd mb meg isa
[0.05] ( 0.05; ) motherboard quadra slot mhz hd meg adapter processor simm ide
[0.11] ( 0.11; ) ide controller scsi scsus floppy isa mb meg motherboard connector
[0.06] ( 0.06; ) jesus faith belief verse god scripture passage satan eternal interpretation
[0.07] ( 0.07; ) ram cache swap mb pc mac windows vga os scsi
[0.04] ( 0.04; ) anonymous wiring privacy wire outlet protocol unix nec ripem ftp
==========================================================================
Average Topic Coherence = 0.062
Median Topic Coherence = 0.064

Hi,
Could you share your 20 newsgroup that you have used as the external source?
Thanks~

UPDATE: we recently found that the TC numbers in the paper are slightly under reported due to the way the TC script works. please make sure that you set the window size to -1 (whole document) if you are using the same script as me.

since there is a bit of confusion regarding the reference corpus: if you dont use the same reference corpus as mentioned in the paper the scores will not be the same. (thanks for mentioning this @AEGISEDGE )
i will update the scores (using -1) here soon

Could you share the link that you have used for the coherence?
I have used this link for coherence: https://github.com/jhlau/topic_interpretability/blob/master/run-oc.sh and the result is not the same. Could you share what am I missing here?
What do you mean by setting the document=-1? as the whole corpus in the shared link is in one file.

UPDATE: we recently found that the TC numbers in the paper are slightly under reported due to the way the TC script works. please make sure that you set the window size to -1 (whole document) if you are using the same script as me.

since there is a bit of confusion regarding the reference corpus: if you dont use the same reference corpus as mentioned in the paper the scores will not be the same. (thanks for mentioning this @AEGISEDGE )
i will update the scores (using -1) here soon

Could you share the link that you have used for the coherence?
I have used this link for coherence: https://github.com/jhlau/topic_interpretability/blob/master/run-oc.sh and the result is not the same. Could you share what am I missing here?
What do you mean by setting the document=-1? as the whole corpus in the shared link is in one file.

@akashgit
document = -1 means that, if two words appeared in a document, they have a coappearence, which is used in NPMI counting.
Modify the window_size=0 instead. https://github.com/jhlau/topic_interpretability/blob/b7a7cdc556840cc959252085bc80d1e63031473b/ComputeWordCount.py#L26

UPDATE: we recently found that the TC numbers in the paper are slightly under reported due to the way the TC script works. please make sure that you set the window size to -1 (whole document) if you are using the same script as me.

since there is a bit of confusion regarding the reference corpus: if you dont use the same reference corpus as mentioned in the paper the scores will not be the same. (thanks for mentioning this @AEGISEDGE )
i will update the scores (using -1) here soon

Could you share the link that you have used for the coherence?
I have used this link for coherence: https://github.com/jhlau/topic_interpretability/blob/master/run-oc.sh and the result is not the same. Could you share what am I missing here?
What do you mean by setting the document=-1? as the whole corpus in the shared link is in one file.

document = -1 means that, if two words appeared in a document, they have a coappearence, which is used in NPMI counting.
Modify the window_size=0 instead. https://github.com/jhlau/topic_interpretability/blob/b7a7cdc556840cc959252085bc80d1e63031473b/ComputeWordCount.py#L26
I got the tokens of 20NG by the vocabulary and npy files. Since the whole document is used for coappearence, word orders are unnecessary.

Thanks, Yongfeiyan, for the quick reply.
You mean I have to set the window_size = 0?
Also, Im not sure how can I get the tokens with .npy fileand vocab. Do you mind if you share the code with me?
Also, Im using the same code as you. That source code already shared the wiki and news as the corpus refrence. Im not sure if that news is 20 newsgroup or other news.
Thanks~

The .npy file at https://github.com/akashgit/autoencoding_vi_for_topic_models/tree/master/data/20news_clean are BOW format of share D x V, D is the total number of documents, V is the vocab size.
for each row,
from itertools import chain
list(chain(*[[vocab[i]]*v for i, v in enumerate(row)])  # yields the document corresponding to row in npy file.

I'm sorry that the codes I wrote last year were missing.
When using window_size=0 with 20NG, the topic coherence should be sound.

from itertools import chain
list(chain(*[[vocab[i]]*v for i, v in enumerate(row)])

Sorry Actually I have not worked with pkl files and having kind of difficulty getting it run.
Does it should be like this:

c = np.load('test.txt.npy', encoding='latin1')
print(c)
with open('vocab.pkl', 'rb') as file:
    arr = pickle.load(file)
    print(list(chain(*[[arr[i]] * v for i, v in enumerate(c)])))

Thank you again for taking the time.


New release of react router v4 coming in a day or two in november?

I was just wondering, in the latest alpha release of november there was a note that you should wait before upgrading because a new release would come in a day or two. What is the status of that? Should I just use alpha.6 if I want to try out the react router v4?

It's hard work to get the abstraction right :) We've been working very hard on v4 and should have a beta release this week.

great thanks for the info


Tile making recipe to reduce tile size

Erica,
We tried different approach to dump data in geojson based of the fclass tags from OSM data.  We tried to make tiles smaller and effecient with this reciepe.  You are suggesting this for inclusion in the "recipe" section of the README?

if it helps people why not . some of them can help and make it better .

this approach is best to reduce the tile size but merging Tile-Join takes ages it runs sometimes for 24 hours and use only single core on 48 core machine. how we can use tile join to use all the cores ?


refactor: Fix route-observer null logs in some situations

I've commented on #517 about the null logs, so I've decided to refactor the route observer code to fix those logs (not the issue itself).

Thanks!
LGTM


switch to Azure.Identity to support conditional access.

Summary of the changes

Introduce a new constructor for ServiceEndpoint
Switch to use Azure.Identity

Azure SDK team asked us to support Azure.Identity in function extension. See issue https://github.com/Azure/azure-functions-signalrservice-extension/issues/238.
The key point is getting the TokenCredential from an IConfiguration section. See example from Event Hub binding.
Although there is no request for SDK to support getting the TokenCredential from an IConfiguration section, I think it would be better to implement it in the SDK side. Considering we have multiple endpoints which can share the same managed identities as discussed with @terencefan , we should design a new configuration style for this. @vicancy

Although there is no request for SDK to support getting the TokenCredential from an IConfiguration section, I think it would be better to implement it in the SDK side. Considering we have multiple endpoints which can share the same managed identities as discussed with @terencefan , we should design a new configuration style for this. @vicancy

What's preventing our current configuration from supporting TokenCredential? Looks like we need some ConfigurationSetup?

Yeah, at least we need a function like
ServiceEndpoint FromConfigurationSection(IConfiguration configuration)


Gulp build return error when choose RTL

Hi there, I wanna build RTL website using the new version of semantic-ui.
After completing npm install semantic-ui processing, and going to make gulp build order, I have the following message:

Task 'build rtl' is not in your gulpfile
Please check the documentation for proper gulpfile formatting

What I have to do ??

Hi, it looks like a small typo in the task was causing it not to find the task. The task name is build-rtl not build rtl
You can run the RTL build task manually for now until patch comes out
gulp build-rtl

It will also be fixed in 1.9.3 to work with gulp build when rtl: true. Will be releasing shortly

Thanx for quick reply. It's work.


Fixes for casting\truncation

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


⚠ Bofrost.es has degraded performance

In 3ed8b0c, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1609 ms

Resolved: Bofrost.es performance has improved in 69347a0 after 38 minutes.


.js Learn Recursion Step 48 - accepted solutions don't match text instructions

Describe the Issue
I have found an inconsistency with the text vs accepted solutions on this step. The instructions read:
...
In your callStack array, add the following string: "a(): returns 'freeCodeCamp ' + b()". This represents the function call and the code that will be executed.
Note: Since the string you’re adding includes double quotation marks, wrap it in single quotation marks (') or backticks (`).
...
As it is written, my understanding is that we need to add the given string, including the double quotes, to the array. In order to do that we need to add single quotes  or backticks to the outside. I have tried several variations of single and double quotes-- what is accepted and isn't is detailed in the code section below.
Additionally, there seems to be a typo in the error text (shown in screenshot).
Affected Page
https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures-v8/learn-recursion-by-building-a-decimal-to-binary-converter/step-48
Your code
const callStack = [
'"a(): returns 'freeCodeCamp ' + b()"'
];
//This does not pass

const callStack = [
`"a(): returns 'freeCodeCamp ' + b()"`
];
//This passes

const callStack = [
"a(): returns 'freeCodeCamp ' + b()"
];
//This passes too

const callStack = [
'a(): returns "freeCodeCamp " + b()'
];
//This also passes

Expected behavior
Given the "Note" I would expect the solution to be adding the entire string to the array, including the double quotes on the outside, then adding either single quotes or backticks on the outside. As shown in the code section this solution is accepted for backticks but not single quotes.
Another accepted solution is to copy/paste the text as written, with the double quotes but without any enclosing single quotes or backticks. This seems to make the "Note" unnecessary, but is also confusing to me as I am learning. With this variety of accepted solutions I don't understand what the correct formatting is.
Screenshots

System
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Additional context
No response

I think the note is just totally unnecessary in this case and could be removed. If you copy/paste the string as given, including the double quotes, it's correct.
If we want to keep the note to learn something about alternating quotes with strings, we could just the string to:
a(): returns "freeCodeCamp " + b()

Then it would contain double quotes and the Note is applicable
Note: Since the string you're adding includes double quotation marks, wrap it in single quotation marks (') or backticks (`).

Although, is it a bit odd for a given string not to be wrapped in quotes already, which is the definition of a string?

The easiest thing here is update the instructions, switching "single quotes" with "double quotes" in the text.

built an app to get paid for this PR
https://www.n0va-io.com/discover/freeCodeCamp/freeCodeCamp


I can't compile PXGoogleDirections with Xcode 11.5 because it generated with older version of Xcode

Hello,
I can't compile PXGoogleDirections with Xcode 11.5 because it generated with older version of Xcode
error :
Module compiled with Swift 5.1.3 cannot be imported by the Swift 5.2.4 compiler: /Users/dev/Desktop/iviflo/Ios/SmartFlo/PXGoogleDirections.framework/Modules/PXGoogleDirections.swiftmodule/arm64-apple-ios.swiftmodule

also I can't use  #  pod 'PXGoogleDirections' with   pod 'GoogleMaps' together in my pod file so I use pod 'GoogleMaps' and i use the source code all class of PXGoogleDirections in my project.


fix: stash working tree should work when everything/nothing is staged

Stash working tree is running 3 commands so there is some edge cases which we should take into account:

If everything is staged don't do anything
If nothing is staged don't try to save index in the stash and then later pop it (either errors or pops non-related stashes)

Addresses: https://github.com/altsem/gitu/pull/100#issuecomment-2025832482

Trying to do some digging if there's a better way to handle all this.
There's multiple places where things might fail. What if there's an ongoing merge etc?
If only git had a git stash push --worktree :man_facepalming:.
The git stash pop 1 is scary because it assumes nothing went wrong.
One idea is to return an error code from run_cmd and react to it.
I found this function in the git source code:
https://github.com/git/git/blob/d6fd04375f9196f8b203d442f235bd96a1a068cc/builtin/stash.c#L1288

The C function does something like this I believe (123 would be the pid):
$ git diff --name-only | GIT_INDEX_FILE=.git/index.stash.123 git update-index --ignore-skip-worktree-entries --add --remove --stdin

And then it writes a "stash commit" somehow after.
Although doing this results in not just the diffed file being stage.
$ GIT_INDEX_FILE=.git/index.stash.123 git status
On branch fix/stash-working-tree-edge-cases
Your branch is up to date with 'golden-expiriensu/fix/stash-working-tree-edge-cases'.

Changes to be committed:
  (use "git restore --staged <file>..." to unstage)
	deleted:    .cliffignore
	deleted:    .envrc
	deleted:    .github/dependabot.yml
	deleted:    .github/workflows/cachix.yml
...

Git creates this .git/index.stash.123 file temporarily.

Maybe just changing the state.run_cmd() function to actually return an error if the command failed is the way to go. :thinking:
That way we'd just break execution in the stash_worktree fn.
Would not have to modify anything of what you already wrote in this case I think.

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

Project coverage is 88.41%. Comparing base (699b133) to head (55c2042).
Report is 2 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #103      +/-   ##
==========================================
+ Coverage   88.21%   88.41%   +0.20%
==========================================
  Files          41       41
  Lines        3885     3954      +69
==========================================
+ Hits         3427     3496      +69
  Misses        458      458

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

Looked good! :)


No way to create a custom propagator

There is an API which allows you to pass a custom propagator as an option, but the propagator is meant to convert back and forth between a SpanContext and a carrier. Unfortunately SpanContext is private, so there's no way to actually use the API without completely reimplementing SpanContext (or importing it from the library internals). While both are technically easy, neither seems like a good idea.

Actually creating a custom span context is even iffier than I supposed, because the tracer uses private fields of the span context!! : (


Service Bus UX Study Actions -  Review Builder design

Following issues came in UX  study regarding to builder

having split builder is not usual . Since top level builder do not have any .build() method on it. And we get a sub builder from a builder
Connection sharing is not clear from this design.

Based on discussion with Jonathan and users and after analyzing pro and con for all other options, we have decided to not change builder API for now.
More details here
https://gist.github.com/hemanttanwar/4fb17b3a9dd5a1dd70655debe9873cdb


[1.4] [Quality setting not being used]

Expected Behavior
Set 1080p as video quality → 1080p video download
Current Behavior
360p is always being downloaded regardless of quality value. I understand 360p is fallback but from video metadata I can see it supports [ '144p', '240p', '360p', '480p', '720p', '1080p' ].
Failure Information (for bugs)
No failure, video downloads successfully but not in desired quality.
Video tried:
id: 'KQyXqYUdc_0',
title: 'Rublev vs Sinner in WILD atmosphere; Tsitsipas, Zverev Feature | Monte Carlo 2022 Highlights Day 5',
Checklist

[X] I am running the latest version
[X] I checked the documentation and found no answer
[X] I checked to make sure that this issue has not already been filed
[X] I have provided sufficient information

The library is choosing the 360p resolution because it is the only one available with audio, if you want higher quality videos you'll have to download video and audio separately and then merge them. So just set the type to video instead of videoandaudio and it'll download the desired quality but no audio.


Latest code

Very impressed by your work, I notice that better results are reported in the archive version of your paper. Could you release the latest code that gives the better results? Much appreciated!

The code in this repo corresponds to the results of feat* in the updated paper, where the code can reproduce those results.

Thanks a lot!


ResolveAsync does not iterate through all servers

Repro code below, check the comment for where it throws vs it doesn't.
List<IPAddress> lDnsServers = new List<IPAddress>();
lDnsServers.Add(IPAddress.Parse("<IP_ADDRESS>")); // non existent
lDnsServers.Add(IPAddress.Parse("<IP_ADDRESS>"));
DnsClient lClient = new DnsClient(lDnsServers, 5000);
lClient.IsTcpEnabled = true;
var resolver = new DnsStubResolver(lClient);
var records1 = resolver.Resolve<MxRecord>(lDomain, RecordType.Mx); // this will NOT throw and get the result from <IP_ADDRESS>
var records2 = await resolver.ResolveAsync<MxRecord>(lDomain, RecordType.Mx).ConfigureAwait(false); // this will throw on the non existent server and never query <IP_ADDRESS>

See https://github.com/Netatwork-de/ARSoft.Tools.Net/commit/8bc92454c9bc6764f4759681b3f9305c6337b627 for a fix.


Any examples of basic IGListKit in Objective-C?

Wondering if theres any basic examples of working IGListKit in Objective-C.

@farhansyed9 there aren't, but I've gotten this question so many times that I think there needs to be. Let's add a new example VC, section controllers, and cells using ObJC!

Wondering too...

@farhansyed9 @liuxz656 @rnystrom
I have just uploaded  an ObjC demo to my repository.  Maybe it is helpful.

@rnystrom Did we want to have a single project (Examples-iOS) with both ObjC and Swift, or to create a new pack (Examples-iOS-ObjC) to keep them separate?

@Sherlouk nah single ObjC VC+SC+cell in the current Examples-iOS is fine I think. Don't want to make the examples too complicated. Thoughts?

@rnystrom   Actually, it's totally identical. 😀  And I will PR it ASAP.

fixed in #446


updated snowflake setup docs to account for our new okta login workflow

fixes #348

any idea why the changes from this PR are not reflected here: https://cagov.github.io/data-infrastructure/setup/
I re-ran the job and included a docs build. Also cleared my cache. @JamesSLogan


What is the /v1/users/:id PUT api good for when you have v1/users/:id PATCH?

Why do you need the user-controller replace function and the /v1/users/:id PUT api when you have the user-controller update function and the /v1/users/:id PATCH api?
Is this just for the example use-case (some people like to use PUT and some like to use PATCH), or is there some deeper reasoning behind this decision, like supporting some official api standard (i.e. OAuth)?

It's just for example, they have different behaviors and you don't need both. Here is a good explanation: https://stackoverflow.com/a/28558968/4914054

Thanks


How cloud I use SnapKit in Objc Project???

Hi guys ;)
I create new project with CocoaPods and of course use SnapKit in it.
and I want use third party class TreeView ,this class write by Objc.
My Q is How could I use SnapKit in class TreeView ??? Thanks ;)

I think you may have an easier time with the Masonry project if you need the same functionality as SnapKit but for an Objective-C project, but your mileage may vary and while it is possible perhaps to twist SnapKit enough to work as you wish Masonry may still be your best bet (SnapKit is Masonry’s Swift successor).
Sent from my iPhone

On 20 May 2018, at 12:08, hopy<EMAIL_ADDRESS>wrote:
Hi guys ;)
I create new project with CocoaPods and of course use SnapKit in it.
and I want use third party class TreeView ,this class write by Objc.
My Q is How cloud I use SnapKit in class TreeView ???
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks you @Panajev

for reference https://github.com/thisfin/DigitalClock/blob/1b428ede298a720019b01467edacf87837560851/DigitalClock/DigitalClockView.m


mockRequest with omitted body is ignored

Mocking this:
    mockRequest({
      method: 'post',
      url: m.stringContaining('/api/spending/v4/search'),
      response: {
        status,
        body: { transactions: fixture },
      },
    })

I expected it to work, but was completely ignored:
Request error ([Mappersmith Test] No match found for "POST /api/spending/v4/search" (body: "{"categories":["categoryId"],"order":"DESC","sort":"DATE","limit":100}"), check your mock definition) [spending/v4/search] 400
It was like mappersmith test said to me: "So you have a verb and URI that is in my list of mocked resources. Alright, cool. But... wait a minute - you are also sending a body? Eww!
Adding body: m.anything(), to the options makes the mock go through.
I expected that the default when not passing body at all would be to default to body: m.anything(). The current behaviour feels a bit too strict.
However, IF I have had specified a body in my mock, and it doesn't match, then the error is good and the strictness is warranted.

Solved in #272


block params as number?

{{#path 1 2 3 as |4|}} text {{4}} {{/path}}

Should we not throw an error if the first char in the variable name is not a letter?:
{{#path 1 2 3 as |a4|}} text {{a4}} {{/path}}

There has never been a requirement that we follow javascript id patterns, so {{4}} is just fine in the language. What is the argument for changing this?

It conflicts with {{4}} which is already supported by handlebars as a number literal.

I don't believe this to be the case from my test compilation, it resolves
the value (or helper) "4". Can you put together a fiddle showing the
opposite if you are seeing different behavior?
On Wed, Mar 16, 2016 at 10:15 AM Steven Vachon<EMAIL_ADDRESS>wrote:

It conflicts with {{4}} which is already supported by handlebars as a
number literal.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/wycats/handlebars.js/issues/1199#issuecomment-197377101

It conflicts semantically and doesn't work; no actual error. Because of this, it should probably throw when being parsed.
console.log( require("handlebars").compile("{{#each list  as |value 4|}} {{value}} {{4}} {{/each}}")({list:[0]}) )

produces 0 0. {{4}} doesn't work as might be expected.
An example of something similar that does throw a parse error:
{{#each list  as ||}} {{/each}}

This is operating as expected, {{4}} is resolving the second block parameter. Without the block parameter it would resolve to context['4'] so both behaviors are consistent. There is some level of oddity as numbers are treated as literals in any other position, but I'm not willing to break a use case that has been around for this long because someone chooses a bad variable name.


PSR Namespaced Autoloading through Composer

Every new package these days user Composer to take care of autoloading.
Once we have loads of unit tests in place we should start to remove all the old fashioned includes and requires everywhere, and rename the classes to be PSR-4 compliant, allowing them to be autoloaded by Composer without the need to include the class files.

Closing this, lack of time to do this on my own and apparently not too much public interest to get them added. Thanks a lot for the PR though, should I ever have the time to go back to this I will use it as a base to get started! (Shameless copy and paste)


Does it work with high voltage inverters?

I presume this does not work with SUN-10K-SG01HP3-EU-AM2 ?
That's a high voltage inverter, serial no. starts with 23xxx, so presumably it's not protocol Version 5, but 4? Or another?
Futhermore the sensors/registers are different for the HV inverters...

Hi,
first I have to ask where are you coming from? Are you a "first time user" or you are already using implementation from the @StephanJoubert? Cause theoretically this integration should work for all the same devices.
That being sad, it's really tested only for the device I own. So it has to be tried to be really sure.
On the other hand if definition file for your device is missing but you have the registry list i could try to add it. ;)
Wish I could help you more.
Have fun.

I'm trying to use the implementation from the @StephanJoubert, but that does not work for me either.
Over in his repository there is a discussion about that, and someone has supplied an incomplete custom_parameters.yaml for the high voltage inverters (everything related to time of use is missing,  e.g.)
But I can't get that far, as the integration does not connect to the wifi dongle at all, the serial number of the inverter starts with 23xx, the serial number of the dongle starts with 29xx - both are not listed as working with protocol V5 in the readme from https://github.com/StephanJoubert. There is an omnious remark to try v4, but it's not explained anywhere how to do this.
The inverter is registered with Solarman Business App, and all values show up there.

Inverter information
Inverter serial number 2312122xxx
Firmware version (main)
Firmware version (slave)

Device information
Device serial number  2908548xxx
Firmware version LSW3_15_FFFF_1.0.9E

I'll try tomorrow ...

I don't have the register list directly, but the definitions can be found in
https://github.com/kellerza/sunsynk/blob/main/src/sunsynk/definitions/three_phase_common.py and https://github.com/kellerza/sunsynk/blob/main/src/sunsynk/definitions/three_phase_hv.py combined.

And there is someting here:

https://github.com/kbialek/deye-inverter-mqtt/blob/main/docs/metric_group_deye_sg01hp3.md
https://github.com/kbialek/deye-inverter-mqtt/blob/main/docs/metric_group_deye_sg01hp3_battery.md

https://github.com/kbialek/deye-inverter-mqtt/blob/main/docs/metric_group_deye_sg01hp3_bms.md
https://github.com/kbialek/deye-inverter-mqtt/blob/main/docs/metric_group_deye_sg01hp3_ups.md
https://github.com/kbialek/deye-inverter-mqtt/blob/main/docs/metric_group_settings.md

Actually I found the register list
MODBUSRTU三相高压储能通信规约V104（高压）(3)(1)1111_SG01-HP3-AM2.pdf

Great, I'll try to integrate it as soon as I can but we still need to find out if there is even any communication happening (cause of that V5 or V4 matter).

@bsafh I added file for your device in: b53e902

tested it, using "main" in the version dialog (HACS download): protocol 5 works!
But ... config flow is ... not commentet, so I have no idea what to fill in:

I managed to get around this by just copying the file deye_sg01hp3.yaml over custom_parameters.yaml (which was alread configured from the now overwritten config flow from @StephanJoubert 's version.
It does recognize the inverter :-)
And it does read the new definitions.
But it does not contain any "Time of use" information, neither the switcg for that nor the timetable itself. And there are no switches (e..g switch priority_load on and off - oops, priority_load is not even there?)

Time of Use is not read as far as I know in any of the inverter definition files yet.
Instead of reconfiguring, try remove and then add device again. I'll look into why configuring flow is missing the config values description.
But I'm glad that it at least tries to read any values, that's a good sign.

So I advise to you really re add the device.

how do I get rid of the old devices?
For the switches I have found this:
deye-modbus.zip

It's really not about the registry addresses but about implementation in the integration. There are not entities for it. I do want to eventually add support for as much things as possible but my focus was currently on the connection stability.
Anyway for removing the device:

Hi i use this integration for a HV 50KW Deye too, works very well. Some Entitys you have to set a other Scale is in the Documentation. I will attached this new Version what i get from Deye since 2 Weeks.
【b】30240301001287-使用说明书-储能三相-sun-(25-50)k-sg01hp3-eu-bm4-德业英文-封面彩印-封面封底157g-内页70g铜版纸-142.5x210mm-ver2.2.pdf

Hi there, thanks for your comment. I was trying to adjust the scaling according to the previus doc but is possible I missed some of them so if that's the case let me know which one and I'll correct them. 😉

for Example

Battery Voltage LV 0.01 - Battery Voltage  HV 0.1

You can find it in the Manual on the 5t Row

Also what i add to your Config is the Gen Port, because a many useres from my side need this for the Micro Grid Input.

Hi, thanks for your comment, I'll look into that. 😉
Have fun!

Hi guys, would be some of you willing to track for me different numeric states for "* Device Relay Status" (register: 0x0228) entity?
What i saw so far was 6 and 15, any idea what they could mean? Cause they do not match states from my LV device and not even match the modbus doc for HV device...
Would be highly appreciated as this register is used for detecting Power Outages.
Thanks and have fun!

for Example

Battery Voltage LV 0.01 - Battery Voltage  HV 0.1

You can find it in the Manual on the 5t Row
Also what i add to your Config is the Gen Port, because a many useres from my side need this for the Micro Grid Input.

I added entities for SmartLoad/Generator/Microinverter power and voltage in feat: Add SmartLoad/Gen voltage and power entities
Naming of the entities are still subjects to change as those entities could have meaning in three different contexts...

All the necessary configurations is now possible using this integration (for SG04LP3 and SG01HP3).
But there is still plenty (not so crucial) to implement. So if let me know guys if forget any important ones.
 I proly never saw something implemented so poorly... That being said setting of those configurables sometimes don't work very smoothly. I will try to improve on that in the future. 😉
It will be all part of next release which is scheduled on: 24.07.24
Cheers!


Unexpected horizontal line is rendered on  component in Safari v15.0

Describe the bug
 component is rendered with unexpected horizontal line at the top in Safari v15.0 browser.
To Reproduce
Steps to reproduce the behavior:

Go to https://nova-ui.solarwinds.io/bits/release_v11.8.x/components/PanelComponent.html#example in Safari v15.0
Review any  usage example

Expected behavior
 component is clearly rendered with no unexpected elements
Screenshots

Environment (please complete the following information):

OS: MacOS Big Sur v.11.6
Browser: Safari
Browser Version: Version 15.0 (166<IP_ADDRESS>.4, 16612)

Additional context
Assumption is that current issue is caused by 'background' style of underlying '.nui-scroll-shadows' class.
Issue stops reproducing when disabling this style in browser 'Elements' tool:

Hello,
Has anybody had a chance to look at this issue?
Regards!

Hi, @gtereshkov, Nova currently doesn't have teams actively developing on it. I believe team Bravo may be accepting contributions for features/bugfixes. cc: @mzknksh

right now nova is in maintenence mode due to work on OO, so we're encouraging contributions which we can help check and release. (I would just recommend researching different option than just removing class that misbehaves in safari altogether)


[FATAL] ORA-01034: ORACLE not available

Any ideas why?
Also, how can i increase shared pool size &  shared pool size ?
➜  18.4.0 git:(master) ✗ docker run -it --privileged oracle18c-xe

ORACLE PASSWORD FOR SYS AND SYSTEM: oracle
Specify a password to be used for database accounts. Oracle recommends that the password entered should be at least 8 characters in length, contain at least 1 uppercase character, 1 lower case character and 1 digit [0-9]. Note that the same password will be used for SYS, SYSTEM and PDBADMIN accounts:
Confirm the password:
Configuring Oracle Listener.
Listener configuration succeeded.
Configuring Oracle Database XE.
[WARNING] [DBT-11205] Specified shared pool size does not meet the recommended minimum size requirement. This might cause database creation to fail.
   ACTION: Specify at least (1,888 MB) for shared pool size.

Database configuration failed. Check logs under '/opt/oracle/cfgtoollogs/dbca'.
mv: cannot stat '/opt/oracle/product/18c/dbhomeXE/dbs/spfilemedssb01r1.ora': No such file or directory
mv: cannot stat '/opt/oracle/product/18c/dbhomeXE/dbs/orapwmedssb01r1': No such file or directory
The Oracle base remains unchanged with value /opt/oracle
#####################################
########### E R R O R ###############
DATABASE SETUP WAS NOT SUCCESSFUL!
Please check output for further info!
########### E R R O R ###############
#####################################
The following output is now a tail of the alert.log:
tail: cannot open '/opt/oracle/diag/rdbms/*/*/trace/alert*.log' for reading: No such file or directory
tail: no files remaining

Can you please share docker info and also how much memory and CPUs you have given Docker?

Same issue installing database-xe-18c-1.0-1.x86_64.rpm into linux docker instance.  64GB Ram and 24 core threadripper 2970 cpu.

Please open a new issue as this one is already closed.


Expose remaining lambda config

Closes #680.

Per private slack, will test this further against tools/mailer, etc. to harden.

Alright @kapilt, I've run through the README tutorials for both mailer and sentry. Seeing same behavior on both master and here.


Possible to add option to throw AssertionError if buffer leak is detected?

The page on reference counted objects (https://netty.io/wiki/reference-counted-objects.html) mentions that it is good practice to run tests at PARANOID leak detection level, and then fail the build if the test logs contain "LEAK:".
I was wondering if it could make sense to add another option for the leak detection level, that would cause ResourceLeakDetector to throw an AssertionError if a leak is detected? It obviously wouldn't make sense when running outside tests, but it would be convenient if the tests would fail when resources leak, rather than having to scan the test logs for "LEAK:" after running the tests.
Would this be possible, or is it a bad idea due to e.g. the errors potentially being swallowed when they kill internal Netty threads?

@srdo I am not sure if this would actually work as it will be processed by the GC threads. That said you can easily test this by implementing your own ResourceLeakDetectorFactory which will provide a ResourceLeakDetector that will not log but do throw the error as you suggested.
Let me know how this works out.

@srdo ping

@normanmaurer Sorry, haven't had time yet to work on this. As far as I can tell, the log lines are coming from the event loop threads (thrown when a reference counted object is allocated), so the difficulty would be in getting the exception from there to the thread running a given unit test.
I'll hopefully get some time soon to look at this.

@normanmaurer I had a chance to try this out, by throwing an AssertionError from https://github.com/netty/netty/blob/a8549b138d2dcf64539fe4707c18fb1a3003e06f/common/src/main/java/io/netty/util/ResourceLeakDetector.java#L310 and https://github.com/netty/netty/blob/a8549b138d2dcf64539fe4707c18fb1a3003e06f/common/src/main/java/io/netty/util/ResourceLeakDetector.java#L321
I modified one of our ChannelInboundHandlers to leak a ByteBuf, and set the leak detection level to paranoid.  In some cases, the leak is detected when allocating a buffer in the pipeline, and gets caught by the exceptionCaught method in one of our handlers (which is our case kills the JVM). In others, the leak is detected when allocating a buffer for sending a message, which throws the exception to the client's ChannelFutureListener. Finally, if I remove the exceptionCaught method from our handlers, the exception can end up in https://github.com/netty/netty/blob/4.1/transport/src/main/java/io/netty/channel/DefaultChannelPipeline.java#L1186, which just logs it.
In the cases where the leak is detected, it causes our tests to fail, because the message handling wasn't allowed to finish properly, so the tests time out waiting for messages to be received. I'd expect something similar to happen for other people's tests, as long as those tests check that messages are not lost.
I think throwing an error in addition to logging when a leak is detected could be somewhat useful. It does come with the same caveats as the regular leak detection, and also potentially makes tests containing leaks flaky until you fix the leak. If you think it makes sense to provide a new detection level to throw an error in addition to logging when leaks are found, I'm happy to open a PR.

That said you can easily test this by implementing your own ResourceLeakDetectorFactory which will provide a ResourceLeakDetector that will not log but do throw the error as you suggested.

I'd also like to add automatic detection in all integration tests.
@normanmaurer I can see that ByteBuf for example uses the default instance, how can I link my implementation of ResourceLeakDetectorFactory?
static final ResourceLeakDetector<ByteBuf> leakDetector =
            ResourceLeakDetectorFactory.instance().newResourceLeakDetector(ByteBuf.class);

Also it does not look like there's an option to plug into other ResourceLeakDetectors?


⚠ ClamAV has degraded performance

In dcd1462, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 95 ms

Resolved: ClamAV performance has improved in 8d431b4.


Add Thermodynamix by Xcemines

Song Name
Thermodynamix
Artist Name
Xcemines
Source
Youtube
Youtube Link
XoY1sQMFoguP1L5r
Direct File Link
No response
Song ID
770520
Start Offset [ms]
93
Extra
Created from Auto Nong.
Created for level: 73667628

reject
I'm sorry the video isn't available anymore


Pseudo pre-compiling

Instead of building templates at runtime, what about parsing a JSON object that looks something like what @justinbmeyer mentioned:
can.stache([["start","h1"],["attrStart","class"],["attrValue","bar"],["attrEnd"],["close"]])

You're stating that can now parses the templates correctly in the related stealjs issue.
I'd love include those parsed templates in my amd build layers, as opposed to simply wrapping them in can.stache() calls. Is this now possible? I've been toying around with it a bit, but I haven't got it working yet. The rendering parts of can are fairly complex to get a full understanding of how they work.

This is implemented now.

Any news on the slower performance that this gives? We've discussed this once on gitter...

Can you remind me what you mean by  "slower performance"?

I ran some tests in the past, and intermediate compiled templates actually performed worse at runtime than on the fly compiliation of stache templates.


feat(partner center sell): add partner-center-sell service to project

PR summary
We are adding a new service that we plan to bring to the terraform provider
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

Current vs new behavior
We are adding a new service that we plan to bring to the terraform provider
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

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

Please add bgazsiatibm and balazs-marjan to the writers, they connected their accounts to their IBM account.

Evidence for example tests:

Evidence for integration:

:tada: This PR is included in version 0.67.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket: