text:
Console window remains open when VSCode started from PowerShell

Tracked in VSCode as https://github.com/microsoft/vscode/issues/50308.
Steps to reproduce
(On Windows)

Open VSCode with code .
exit PowerShell

Expected behavior

PowerShell console window closes
VSCode stays open

Actual behavior

PowerShell window remains open
VSCode remains open
When VSCode window is closed, the PowerShell window is closed
If PowerShell runs code but does not open a new window (because one for the directory is already open) the issue does not occur

Environment data

Name                           Value
----                           -----
PSVersion                      7.0.0-rc.1
PSEdition                      Core
GitCommitId                    7.0.0-rc.1
OS                             Microsoft Windows 10.0.19535
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           1.1.0.1
WSManStackVersion              3.0

This is microsoft/vscode#66750.

PowerShell try to detect that application is GUI then run it async. Otherwise it run the application as console application in sync.

See #9970 for reference on IsWindowsApplication()

There's nothing PowerShell can do to prevent this. VSCode (<=1.41? to be confirmed) re-attaches to the console handle it receives from its parent. In so doing, it prevents the console from being closed when PowerShell exits because there's still one application attached to it.

authors: DHowett-MSFT iSazonov rjmholt


text:
Get-AzStorageContainer Cannot Return "PublicAccess" attribute if using AD-based context

Get-AzStorageContainer command utilizes a -context to authenticate and retrieve values for a storage account.  The 'PublicAccess' attribute only returns values if the -context supplied uses a storage account key.  If the context utilizes AD credentials it fails to return any values.
See below:

The Az module isn't maintained by the PowerShell team @JoelCBennett. You'll need to file this issue in the azure-powershell repo. 🙂

Thanks @vexx32.  Opened new issue there (link) and closing this one.

source: gharchive/issue

repo: PowerShell/PowerShell

license: MIT

license_type: permissive

license_source: github-api


text:
Get-Command fails with unexpected error when given a blank command-name argument

Note: This is definitely a minor bug that won't arise too often in the real world.
Passing an all-whitespace -Name argument to Get-Command results in unexpected error message Get-Command: Index was outside the bounds of the array.
Steps to reproduce
{ Get-Command -ea Stop -Name ' '  } | Should -Throw -ErrorId CommandNotFoundException

Expected behavior
The test should succeed; that is, the usual CommandNotFoundException should be reported.
Actual behavior
The test fails, because the error mentioned above is reported instead.
Expected an exception, with FullyQualifiedErrorId 'CommandNotFoundException' to be thrown,
but the FullyQualifiedErrorId was
'System.IndexOutOfRangeException,Microsoft.PowerShell.Commands.GetCommandCommand'

Environment data
PowerShell Core 7.1.0-preview.5

I'll try tackling this one.

This issue could benefit from the proposed enhancement in #10010 to have [ValidateNotNullOrEmpty()] have a settable property to check for whitespace in addition to null and empty.

I'd like to point out that when reproducing this error after doing Start-DevPowershell it gives a different error. This is because src\System.Management.Automation\engine\CommandSearcher.cs has the following Dbg.Assert(                 !string.IsNullOrEmpty(name),                 "Caller should verify name");.
My question is, does this assert belong here? Issue #10165 was closed because it was decided that variable names and alias names with whitespace should be possible. So should Get-Command return the whitespace alias if it exists?

Given the conclusion to that issue, I'd agree that that assertion shouldn't be there, yeah. 👍

Well I've done some digging and it looks like in CommandSearcher.cs at line 1540 _commandName is trimmed _commandName = _commandName.TrimEnd(Utils.Separators.PathSearchTrimEnd);. This is what ultimately causes the index out of range exception to be thrown at line 1102                if (_commandName[0] == '.' || _commandName[0] == '~' || _commandName[0] == '\\'). If I comment out the _commandName.TrimEnd(Utils.Separators.PathSearchTrimEnd); The behavior of Get-Command ' ' is as expected and everything else seems to work fine.
My next question is (and I realize this is asking a lot), does anyone have any ideas why the _commandName = _commandName.TrimEnd(Utils.Separators.PathSearchTrimEnd); is there?


text:
Get-Date -Date ("01-01-1970") -UFormat %s returns wrong time

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
Run the following command:
Get-Date -Date ("01-01-1970") -UFormat %s
Expected behavior
PS> Get-Date -Date ("01-01-1970") -UFormat %s
0

Actual behavior
Get-Date -Date ("01-01-1970") -UFormat %s
-3600

Error details
In Powershell version: 5.1.17763.2931 it correctly returns 0

I do not know why it returns +1 hour when getting epoc time, but all date conversions using epoc time will be inaccurate.

No Error produced by command.

Environment data
Name                           Value
----                           -----
PSVersion                      7.3.0
PSEdition                      Core
GitCommitId                    7.3.0
OS                             Microsoft Windows 10.0.17763
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           1.1.0.1
WSManStackVersion              3.0

Visuals
No response

What time zone are you in ?  The date is in local time.

@jhoneill, I think the crux of the issue here is that the behavior of -UFormat %s changed between Windows PowerShell and PowerShell Core.
While the change was technically breaking, it should be thought of as  bug fix, because the Windows PowerShell behavior is incorrect, whereas  PowerShell Core's new behavior is the correct one.
I've opened a docs issue:

https://github.com/MicrosoftDocs/PowerShell-Docs/issues/9483

@mklement0
OK there are two things.  And bluntly I have always thought that Get-Date's behaviour is odd.

Get-Date always treats  get-date "1/1/1970"  (or any other date) as local time.  and get-date "1/1/1970" -AsUTC   may return 31 December 1969 23:00:00 if your time zone is UTC +1 . It is necessary to pass a string with "z" to say the time given is

A change in the behaviour of %s. After V 5.   (Get-Date -uformat %s  gives different results on 5 and 7 if local time isn't UTC)

2 means in PS 5 get-date "1/1/1970"  -uformat %s   gets the date as local and subtracts 1/1/1970 00:00 LOCAL - zero, the answer the OP expected, but only correct if Local time is UTC in January.
and in PS 7 get-date "1/1/1970"  -uformat %s   gets the date as local and subtracts 1/1/1970 00:00 UTC  - giving the time offset during January (no DST in the northern hemisphere, with DST in the southern) which is correct, but not what the OP expected.
Times (and Date-times) which do not specify offsets from UTC - they tell you the time with some locally selected offset from UTC applied. "9:30" is meaningless unless I say it is 9:30 London, 9:30 New York, or 9:30 Sydney.   And the difference between 9:30 in those different places is not constant year round.  Saying the Unix Epoch is "Midnight 1st of Jan 1970" without saying where allows up +/- 12 hours of error.

@jhoneill

I have always thought that Get-Date's behaviour is odd.

Get-Date is clearly to blame for the broken behavior of -UFormat %s in Windows PowerShell: it uses the wrong reference timestamp: it should be using midnight of 1 Jan 1970 UTC, but mistakenly uses whatever midnight of 1 Jan 1970 happens to be in the local machine's local time zone.

Apart from that, Get-Date is just the messenger for the inherent limitations of the [datetime] .NET type (System.DateTime)

Only [datetime] instances whose .Kind property value is Utc are unambiguous.
Both the Local and the Unspecified kind are inherently ambiguous, the latter more so than the former.

Local is unambiguous, unless external context is provided, i.e. the UTC offset implied by host machine's time zone.

Unspecified - as returned by Get-Date 1970-01-01, for instance - is by design unambiguous, and it is situationally interpreted as either Local (if .ToUniversalTime() is invoked on it) or as Utc (if .ToLocalTime() is invoked on it).

Ultimately, it is best to avoid [datetime] altogether, in the interest of the - always unambiguous - [datetimeoffset] (System.DateTimeOffset).
Unfortunately, PowerShell currently lacks proper support for[datetimeoffset]:

#18255
#13312
#3172
#13598

Thank you for your clarification. Very well explained. 👍

authors: KnutssonDevelopment jhoneill mklement0


text:
unicode variable name garbled

1 I cannot determine the source of the problem. I suspect that it is related to the powershell engine.
Please help find the cause and change the issue title.
2 Looks unrelated to psreadline.
3
note:
There is a tab before each line of code.
Delete the tab in front of each line of code, the garbled disappears.
Only affect the paste code in the console, in the ps1 script, there is no such problem.
Steps to reproduce
note:
There is a tab before each line of code.
Delete the tab in front of each line of code, the garbled disappears.
Only affect the paste code in the console, in the ps1 script, there is no such problem.
	$t1, $t2 = top -b -d 2 -n 2 | grep Cpu
	[decimal]$t11 = $t1.replace('id,', '乂').split('乂')[0].replace('ni,', '乂').split('乂')[1]
	[decimal]$t12 = $t2.replace('id,', '乂').split('乂')[0].replace('ni,', '乂').split('乂')[1]
	$t11
	$t12
	$aaaa = ($t11 + $t12) / 2
	$aaaa

	$cpu空闲值百分比_实时_所有核心 = ($t11 + $t12) / 2
	$cpu空闲值百分比_实时_所有核心

return

Environment data
putty 0.70
putty 0.71
xshell 5 build 1249
powershell 6.1.3 on centos 76
powershell 6.2.0 on centos 76
reporter
By the chinese powershell study & research group. QQ group No. is 183173532

Testing on Windows, PS 6.2:
Works fine in VS Code's integrated console.
If I change Windows Console font to 'NSimSun', the characters appear to display properly,
At no time do the commands fail, even though the characters are not displayed.
It would appear, from the number of boxes displayed, its an issue of the input encoding not matching, possibly because the 'tab' character upsets the decoding of the paste, and its probably specific to putty or centos, and not to PowerShell.

[root@centos76 ~]# echo $LANG
zh_CN.UTF-8

[root@centos76 ~]# pwsh
PowerShell 6.2.0
Copyright (c) Microsoft Corporation. All rights reserved.

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

PS /root> $OutputEncoding

Preamble          :
BodyName          : utf-8
EncodingName      : Unicode (UTF-8)
HeaderName        : utf-8
WebName           : utf-8

实时 ----> 6 block

Steps to reproduce2
$日期对象 = Get-Date
$日期格式 = New-Object System.Globalization.CultureInfo "en-us"
$日期格式.DateTimeFormat.ShortTimePattern
$日期格式.DateTimeFormat.LongDatePattern
$日期对象.ToString($日期格式)

write-host '-------------'
$中文日期格式 = New-Object System.Globalization.CultureInfo "zh-cn"

remove this line  --->$日期对象 = Get-Date,the garbled disappears


text:
Enhance Remove-Item to work with OneDrive

PR Summary
Before the fix only enumeration (Get-ChildItem) worked on OneDrive.
Now Remove-Item correctly removes folders and files on OneDrive too.

Enhance IsReparsePointLikeSymlink() method so that exclude non named surrogates from reparse points. As result PowerShell recurse into such reparse points which is a OneDrive entities.
Use updated IsReparsePointLikeSymlink() method in RemoveDirectoryInfoItem() and NameString() methods
Add new tests and a test hook to emulate old PowerShell behavior and test new PowerShell behavior on OneDrive.
Add small perf improvement in Dir() and IsReparsePointLikeSymlink() methods to exclude extra p/invoke in most scenarios.

PR Context
Replace #14860
Related #9246
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[ ] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[x] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[ ] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

We need reviews from some other contributors here. I'm hoping some of the reviewers I've added can provide a review, since I know less about filesystem interactions.

/azp list

source: gharchive/pull-request

authors: TravisEz13 iSazonov rjmholt


text:
Improve handling of long named pipe server names on non-Windows hosts

PR Summary
Fixes handling on tmp dirs or process names that exceed the 108 character limit of a unix domain socket on non-Windows hosts. This PR ensures that the trimming of socket names take into account:

The dir path length it is created in
The null char at the end of the path
It always has enough space for unique attributes of the proc and skips creating if it doesn't

PR Context
Fixes https://github.com/PowerShell/PowerShell/issues/16994
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[ ] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[ ] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[ ] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

Adding @SeeminglyScience  since I believe the VSCode extension uses this for process attach debugging.  Also, the extension uses custom named pipes, but I don't think this change affects that.

@jborean93  Thanks for looking into this. I am still troubled by this trimming approach (nothing to do with your changes, but the original code is troubling). I will mark this both for WG-Security group review and committee review.

Sounds like a great idea, I do prefer just failing implicitly if it exceeds the path as it will simplify the code in general so if that's the prefer approach based on the WG then I'm happy to remove all of the trimming code.

Looks like the limit will have to be set to 103 (+ NULL char = 104) as that's the limit on macOS. If WG-Security does decide to continue trimming the length then that value would need to be updated in the PR to handle this.

@WG-Security  The group discussed this and feel this is not a security issue since connecting with Enter-PSHostProcess is broken if the namedpipe pathName is trimmed.  However, the API will allow connection to truncated pipe NamePaths.
The working group recommends:

Named pipe names should not be arbitrarily trimmed.
This pipe name creation code should be re-written to create the smallest name possible, while still having acceptable uniqueness.  If name/path is too long then throw and error (for PowerShell simply do not start a named pipe listener).
This will be a breaking change, but can be mitigated by having Enter-PSHostProcess check for both old and new naming conventions when attempting to connect to a local PowerShell hosted process.

I'm totally happy with the outcome, would be great to simplify the code.

This pipe name creation code should be re-written to create the smallest name possible, while still having acceptable uniqueness. If name/path is too long then throw and error (for PowerShell simply do not start a named pipe listener).

This makes sense but brings up the next question, what is considered unique. Right now the pipe is essentially /tmp/CoreFxPipe_PSHost.{start_time}.{pid}.None.{proc_name}. Can we safely remove any of those parts (Culture/None would be one of them) but the process name will always lead to some variable length potentially exceeding the max length.
A potential option is to hash the same name format and just have /tmp/CoreFxPipe_{hash}. This allows pwsh to have a name as long as it wants for uniqueness with a fixed pipe name length. The only variable part here would be if someone has changed their TMPDIR env var to something other than /tmp. If this is desired the question would be what algorithm to use, here is what they would look like

Algorithm
Value
Length

None
/tmp/CoreFxPipe_PSHost.D837E104.23097.None.pwsh
47

MD5
/tmp/CoreFxPipe_f970e2767d0cfe75876ea857f92e319b
48

SHA1
/tmp/CoreFxPipe_a9993e364706816aba3e25717850c26c9cd0d89d
59

SHA256
/tmp/CoreFxPipe_ba7816bf8f01cfea414140de5dae2223b00361a396177a9cb410ff61f20015ad
80

MD5 while close may not be workable in FIPS enabled hosts + may not be secure enough for what pwsh is trying to avoid. SHA1 is only slightly longer but I'm also unsure if it's ok to use in this type of scenario. SHA256 should be unique enough to avoid collisions but is a bit longer. Considering the variability comes from the TMPDIR path maybe that's an acceptable compromise.
Technically we could even shorten this even more by defining a rooted path as the pipe name, say /tmp/PS_{hash} to avoid the CoreFxPipe_ prefix if we wanted to eek out a few more characters available for a custom TMPDIR. The disadvantages I see this this are:

I'm unsure how long .NET supported a rooted path pipe name so older pwsh versions may not work
A client would also need to pass in a rooted path for the pipe name when connecting to it, today they just need to specify the PSHost_... part
Does the extra complexity really offset the advantages here?

Another question that comes up is whether the same change should happen on Windows. Should it also have the same setup or continue being what it does today?

Yes, good question about uniqueness.  We did talk about hashing, and that is a possibility.  However, I feel that process start time ticks is probably good enough to prevent spoofing.  Also it would be nice to keep some human readable part of the pipe name.
In any case (sorry I forgot to mention this) but please hold off any changes until the committee has reviewed the proposal.
The @WG-Security committee feels we need to keep backwards compatibility (connecting to older PowerShell Core as well as WindowsPowerShell versions).  But forward compatibility can be broken.

In any case (sorry I forgot to mention this) but please hold off any changes until the committee has reviewed the proposal.

No worries, will wait to hear back when a clearer path is decided.

@PowerShell/powershell-committee reviewed this, we are ok with using a hash for the pipename and keeping it as short as possible, but an identifier for PowerShell (like pwsh) so that Get-PSHostProcessInfo can discover pwsh named pipes to connect to.  We agree to retain backwards compatibility (so a newer PS7 client and discover older host processes) and not forward compatibility.

authors: PaulHigin SteveL-MSFT jborean93


text:
SqlServerDatabaseMail

Details of the scenario you tried and the problem that is occurring
sqlServerDatabaseMail Logon failure: the user has not been granted the requested logon type at this computer. I am installing and configuring SQL Server. During the install i am setting up Database mail but get security errors. The user i am installing with is in  a domain group that has admin access on the server and in SQL. The sql install all completes except the database mail.
Verbose logs showing the problem
Logon failure: the user has not been granted the requested logon type at this computer
Suggested solution to the issue
The DSC configuration that is used to reproduce the issue (as detailed as possible)
# insert configuration here

           SqlServerDatabaseMail 'EnableDatabaseMail'
           {
                Ensure               = 'Present'
                ServerName           = $Node.NodeName
                InstanceName         = $Node.InstanceName
                AccountName          = $Node.AccountName
                ProfileName          = $Node.ProfileName
                EmailAddress         = $Node.EmailAddress
                ReplyToAddress       = $Node.EmailAddress
                DisplayName          = $Node.MailServerName
                MailServerName       = $Node.MailServerName
                Description          = $Node.Description
                LoggingLevel         = $Node.LoggingLevel
                TcpPort              = $Node.TcpPort
                PsDscRunAsCredential = $Credential

            }

#### SQL Server edition and version the target node is running
sql 2016

#### SQL Server PowerShell modules present on the target node
Name                             Version Path
----                             ------- ----
xSQLServer                       9.1.0.0 C:\Program Files\WindowsPowerShell\Modules\xSQLServer\9.1.0.0\xSQLServer.psd1
MSFT_SqlAG                       0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlAG\MSFT_SqlAG.psm1
MSFT_SqlAGDatabase               0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlAGDatabase\MSFT_SqlAGDatabase.psm1
MSFT_SqlAgentOperator            0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlAgentOperator\MSFT_SqlAgentOper...
MSFT_SqlAGListener               0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlAGListener\MSFT_SqlAGListener.psm1
MSFT_SqlAGReplica                0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlAGReplica\MSFT_SqlAGReplica.psm1
MSFT_SqlAlias                    0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlAlias\MSFT_SqlAlias.psm1
MSFT_SqlAlwaysOnService          0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlAlwaysOnService\MSFT_SqlAlwaysO...
MSFT_SqlDatabase                 0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlDatabase\MSFT_SqlDatabase.psm1
MSFT_SqlDatabaseDefaultLocation  0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlDatabaseDefaultLocation\MSFT_Sq...
MSFT_SqlDatabaseOwner            0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlDatabaseOwner\MSFT_SqlDatabaseO...
MSFT_SqlDatabasePermission       0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlDatabasePermission\MSFT_SqlData...
MSFT_SqlDatabaseRecoveryModel    0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlDatabaseRecoveryModel\MSFT_SqlD...
MSFT_SqlDatabaseRole             0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlDatabaseRole\MSFT_SqlDatabaseRo...
MSFT_SqlRS                       0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlRS\MSFT_SqlRS.psm1
MSFT_SqlRSSetup                  0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlRSSetup\MSFT_SqlRSSetup.psm1
MSFT_SqlScript                   0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlScript\MSFT_SqlScript.psm1
MSFT_SqlScriptQuery              0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlScriptQuery\MSFT_SqlScriptQuery...
MSFT_SqlServerConfiguration      0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlServerConfiguration\MSFT_SqlSer...
MSFT_SqlServerDatabaseMail       0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlServerDatabaseMail\MSFT_SqlServ...
MSFT_SqlServerEndpoint           0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlServerEndpoint\MSFT_SqlServerEn...
MSFT_SqlServerEndpointPermission 0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlServerEndpointPermission\MSFT_S...
MSFT_SqlServerEndpointState      0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlServerEndpointState\MSFT_SqlSer...
MSFT_SqlServerLogin              0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlServerLogin\MSFT_SqlServerLogin...
MSFT_SqlServerMaxDop             0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlServerMaxDop\MSFT_SqlServerMaxD...
MSFT_SqlServerMemory             0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlServerMemory\MSFT_SqlServerMemo...
MSFT_SqlServerNetwork            0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlServerNetwork\MSFT_SqlServerNet...
MSFT_SqlServerPermission         0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlServerPermission\MSFT_SqlServer...
MSFT_SqlServerReplication        0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlServerReplication\MSFT_SqlServe...
MSFT_SqlServerRole               0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlServerRole\MSFT_SqlServerRole.psm1
MSFT_SqlServerSecureConnection   0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlServerSecureConnection\MSFT_Sql...
MSFT_SqlServiceAccount           0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlServiceAccount\MSFT_SqlServiceA...
MSFT_SqlSetup                    0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlSetup\MSFT_SqlSetup.psm1
MSFT_SqlWaitForAG                0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlWaitForAG\MSFT_SqlWaitForAG.psm1
MSFT_SqlWindowsFirewall          0.0     C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\SqlServerDscOLD\12.5.0.0\DSCResources\MSFT_SqlWindowsFirewall\MSFT_SqlWindows...

#### The operating system the target node is running

    Server 2016

#### Version and build of PowerShell the target node is running
5.1.14393.2969

#### Version of the DSC module that was used ('dev' if using current dev branch)

It must be that the user in the PsDscRunAsCredential = $Credential are not allowed to log on locally (in Windows, not SQL) on the node? I'm guessing the resource SqlServerDatabaseMail  does not even start?
Can you verify in the security event log that the account name in the $Credential is not allowed to logon locally?

Yes i added the account to the local backup operators to allow for logon privileges. Maybe you can help with this new problem. The database mail seems to complete successfully but after it is complete the database mail is not set up. Here are the logs
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] Connecting to SQL Server instance 'MyServer\MSSQLSERVER'.
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] Information: PowerShell module SqlServer not found, trying to use older SQLPS module.
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] Importing PowerShell module 'SQLPS' with version '1.0' from path 'M:\Program Files (x86)\Microso
ft SQL Server\130\Tools\PowerShell\Modules\SQLPS\SQLPS.psd1'.
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] Connected to SQL instance 'MyServer'.
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] SQL Server Database Mail is enabled. Database Mail XPs are set to 1.
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] Account name 'SQLMAIL' was not found.
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] Determines if the Database Mail is in the desired state.
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] String value for property AccountName does not match. Current state is '' and desired state is '
SQLMAIL'.
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] String value for property EmailAddress does not match. Current state is '' and desired state is
'Service.myservice.ab.ca'.
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] String value for property MailServerName does not match. Current state is '' and desired state i
s 'MyMailRelay'.
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] String value for property ProfileName does not match. Current state is '' and desired state is '
SQLMAIL'.
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] String value for property Ensure does not match. Current state is 'Absent' and desired state is
'Present'.
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] String value for property ReplyToAddress does not match. Current state is '' and desired state i
s 'Service.myservice.ab.ca'.
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] UInt16 value for property TcpPort does not match. Current state is '' and desired state is '25'.
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] String value for property DisplayName does not match. Current state is '' and desired state is '
MyMailRelay'.
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] String value for property Description does not match. Current state is '' and desired state is '
SQL Server Database Mail'.
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] String value for property LoggingLevel does not match. Current state is '' and desired state is
'Normal'.
VERBOSE: [MyServer]: LCM:  [ End    Test     ]  [[SqlServerDatabaseMail]EnableDatabaseMail]  in 3.7660 seconds.
VERBOSE: [MyServer]: LCM:  [ Start  Set      ]  [[SqlServerDatabaseMail]EnableDatabaseMail]
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] Connecting to SQL Server instance 'MyServer\MSSQLSERVER'.
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] Found PowerShell module Sqlps already imported in the session.
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] Connected to SQL instance 'MyServer'.
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] Changing the SQL Server Database Mail logging level to 'Normal' (value '1').
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] Creating the mail account 'SQLMAIL'.
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] Creating a public default profile 'SQLMAIL'.
VERBOSE: [MyServer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] Configure the SQL Agent to use Database Mail.
VERBOSE: [MyServer]: LCM:  [ End    Set      ]  [[SqlServerDatabaseMail]EnableDatabaseMail]  in 0.9060 seconds.
VERBOSE: [MyServer]: LCM:  [ End    Resource ]  [[SqlServerDatabaseMail]EnableDatabaseMail]

The resource are not reporting any problem, can you see any error in the SQL server logs or event log that can give you a clue to why it didn’t work?

No i do not see any errors. I assume that i should be able to see the database mail setup in the GUI

I assume that yes. If you run the configuration again, does it say it already exist? If so it did work, and not sure why you are not seeing it in the GUI :thinking:

i ran it again and it does think its there i am puzzled
vERBOSE: [MyComputer]: LCM:  [ Start  Resource ]  [[SqlServerDatabaseMail]EnableDatabaseMail]
VERBOSE: [MyComputer]: LCM:  [ Start  Test     ]  [[SqlServerDatabaseMail]EnableDatabaseMail]
VERBOSE: [MyComputer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] Connecting to SQL Server instance 'MyComputer\MSSQLSERVER'.
VERBOSE: [MyComputer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] Information: PowerShell module SqlServer not found, trying to use older SQLPS module.
VERBOSE: [MyComputer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] Importing PowerShell module 'SQLPS' with version '1.0' from path 'M:\Program Files (x86)\Microso
ft SQL Server\130\Tools\PowerShell\Modules\SQLPS\SQLPS.psd1'.
VERBOSE: [MyComputer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] Connected to SQL instance 'MyComputer'.
VERBOSE: [MyComputer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] SQL Server Database Mail is enabled. Database Mail XPs are set to 1.
VERBOSE: [MyComputer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] Account name 'SQLMAIL' was found, returning the current state of the Database Mail configuration
.
VERBOSE: [MyComputer]:                            [[SqlServerDatabaseMail]EnableDatabaseMail] Determines if the Database Mail is in the desired state.
VERBOSE: [MyComputer]: LCM:  [ End    Test     ]  [[SqlServerDatabaseMail]EnableDatabaseMail]  in 2.4120 seconds.
VERBOSE: [MyComputer]: LCM:  [ Skip   Set      ]  [[SqlServerDatabaseMail]EnableDatabaseMail]
VERBOSE: [MyComputer]: LCM:  [ End    Resource ]  [[SqlServerDatabaseMail]EnableDatabaseMail]

Maybe there is a bug, something more that need to be configured, that makes it not show in the GUI. :thinking: Please comment here when you find out the cause.

well lol i do not think it will that easy to figure out

I have a question about logic. i have the script working to install the base sql. I want the option for the user to install reporting services. I have all the module info set up and it creates the info in the mof file. I want to ask the person if they want reporting services installed and they will answer yes or no. How do. How do i prevent the install of reporting services if they say no. I do not think if statement works in the config part of the script.

A DSC configuration is the desired state you want that node to have. If another node needs another desired state then that is another configuration.
The logic you are looking for is some another tool outside of DSC that builds the configurations. There are no interaction with the use once the configuration is compiled. The compiled configuration is the desired state.
I suggest you ask these questions to the community in the DSC Slack channel which can provide you with more real world suggestions. See there how to join https://github.com/PowerShell/DscResources/blob/master/CommunityCalls/2019-08-01/agenda.md
Also please open a new issue if you have questions that is not part of this issue. Since the initial question was answered I will close this issue.

repo: PowerShell/SqlServerDsc


text:
Support unordered list of links under RELATED LINKS

Summary of the new feature / enhancement
The 0.14 version of platyps requires that links under the H2 RELATED LINKS each be on a separate line. Rendering breaks if you forget to put a blank line between them.
The new version of PlatyPS should be able to read/parse the list of links whether they are listed on separate lines or they are collected in an unordered markdown list, regardless of the schema version of the markdown. (E.g. support either "list" format in 0.14 docs and new markdown docs)
All commands that write markdown should write the list of links as an unordered markdown list.
Proposed technical implementation details (optional)
No response

Fixed in https://github.com/PowerShell/platyPS/commit/27d5ffe4090f146b64c7d91453d6683c68c8ee53

repo: PowerShell/platyPS

license: mit

license_source: bigquery


text:
xSQLServerSetup references Setup ProcessID but doesnt capture it

In the possible variables there is a PID Variable being used that is meant to be used to capture the process for the SQL Setup.exe
However the PID variable doesn't get set properly and because of this causes the Set method to close early which then invokes the test method straight away causing it to error out

Oddly on a further run when the SQL media is extracted to the source folder as specified in #40 this runs and does correctly grab the process ID
However I do still think that the dependency on #40 could be made to be more dynamic and allow ISO mounts instead of being hardcoded still.

Facing the same issue - the process is launched however the Test fails soon after.

This will be resolved when PR #123 is merged.

authors: anwather johlju kilasuit

repo: PowerShell/xSQLServer


text:
Added a preloader

Description

Added a Responsive Preloader
Issue 43

Affected Dependencies
NA
How has this been tested?
Ran it on local server and ensured that it didn't fail the tests.
GIF Before

GIF After

Checklist

[x] I have followed the Contribution Guidelines.
[x] My changes do not edit/add any unrequired files.
[x] My changes are covered by tests.

@Praful932 @maneprajakta done with changes

@cryptofox17 We have deployed your PR, The preloader loads in later..

@Praful932 I am not really sure why that's happening. The content of the site is loading first even though I added an event listener to the window object and ensure that preloader is displayed first and it has a z index of 99.
On the local server it seems to be ok, I have also attached the gifs in the description.
Could you tell me where I could have made the mistake?

@cryptofox17 Reach out in the discord project channel, also update your branch with the latest upstream master

@cryptofox17 Any updates?

@Praful932 yeah, I tried multiple approaches but I feel like the script tag gets executed much later so the preloader is displayed later.
So I have initially made the preloader container visible and based on the cookies I set it's display to none. I'm not sure if that would work.
But I will update the PR Tomorrow so you can review it

@Praful932 i am not sure why but there are conflicts in my PR and my fork is also a lot of commits ahead. Should i make a new PR instead?

@Praful932 i am not sure why but there are conflicts in my PR and my fork is also a lot of commits ahead. Should i make a new PR instead?

If the issue persists, Yes please

@Praful932 since this PR has a lot of conflicts I decided to make a new one


text:
Looping issue

Looping Problem

issue solved


text:
Update short description and logo

Questions
Answers

Description?
Update short description and logo.

Type?
improvement

BC breaks?
no

Deprecations?
no

Fixed ticket?
https://github.com/PrestaShop/PrestaShop/issues/15858

How to test?
Please check that short description and logo are updated.

Thank you @Julievrz

authors: Julievrz PierreRambaud

repo: PrestaShop/statsstock


text:
Mobile responsiveness

in mobile :-
search icon is hide
not good for user experince

@PrinceSinghhub  please assign this issue to me. I want to improve that

repo: PrinceSinghhub/Check-Weather-App


text:
core\framework\op_kernel.cc:1192] Resource exhausted: OOM when allocating tensor with shape[64,256,16,16]

core\framework\op_kernel.cc:1192] Resource exhausted: OOM when allocating tensor with shape[64,256,16,16]
When I  train it??

Please give more details about your experiment environment.
Maybe you can scale down the batch size.

Caused by op 'decoder_6/deconv2/conv2d_transpose', defined at:
File "D:/work/gansgood/DNA-GAN-master/train.py", line 133, in
main()
File "D:/work/gansgood/DNA-GAN-master/train.py", line 128, in main
DNA_GAN = Model(args.attributes, is_train=True)
File "D:\work\gansgood\DNA-GAN-master\model.py", line 26, in init
self.build_model()
File "D:\work\gansgood\DNA-GAN-master\model.py", line 224, in build_model
self.Bxs = [self.joiner('decoder', enc_B) for enc_B in self.enc_Bxs]
File "D:\work\gansgood\DNA-GAN-master\model.py", line 224, in
self.Bxs = [self.joiner('decoder', enc_B) for enc_B in self.enc_Bxs]
File "D:\work\gansgood\DNA-GAN-master\model.py", line 174, in joiner
strides=[1,2,2,1])
File "D:\work\gansgood\DNA-GAN-master\model.py", line 94, in make_deconv_bn
output_shape=out_shape, strides=strides, padding='SAME'

gtx1070

I have tested it in different linux machines but no windows machine. I sorry for being not able to reproduce that. Maybe you can modify the dataset.py and change the batch size to a smaller number (e.g., 16) and start to train with respect to no more than 2 attributes.

repo: Prinsphield/DNA-GAN


text:
Scrollr inside a div

Hi there, i wonder if it is possible to use Scrollr just for a div and not the body, something like $('myElement').skrollr.init();
thanks.

+1 For this feature request.

Did anyone try the changes in IE 8 and more important on mobile? And "it seems to work" is not what I mean. For example APIs like setScrollTop should also work.

Why not just mark this feature in docs as experimental or IE9+ or even breaksSetScrollTop :) (or something similar)?

+1 for this feature,
since in some platforms like Microsoft SharePoint, disables the default scrolling behavior, and append the whole page content in a scrollable div.

authors: RomanGotsiy anasnakawa loopingGIT

repo: Prinzhorn/skrollr


text:
Implement result downloads from Hub

See May 10th discussion notes.

Not required.

repo: PrivateAIM/node-result-service


text:
tic-tac-toe button and score

I want to add new match button to the tic-tac-toe game and would like to add the feature so it keeps track of the total number of wins , losses and ties .
Please assign me this issue under JWOC 2024.

@madhurima316  progress?

Do I need to add colours and edit fonts for the button and the score section?

Then I may need another day.

@madhurima316 make a pr tomorrow with all thee changes.

can you please guide , how to set the restart functionality.


text:
Use min-height on .pages_background instead of height

This will ensure that if the content is larger than the screen, the user can scroll to reach the button, but will at least be able to see all information on the page.

Thanks @Neppord !

authors: Neppord objarni

repo: ProAgileAB/TDDStats


text:
docs: update README.md with Anexia Webhook Provider details

Here I add a readme that details on how to use the webhook as a sidecar

Pull Request Test Coverage Report for Build 8981680065
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 59.926%

Totals

Change from base Build 8980878193:
0.0%

Covered Lines:
323

Relevant Lines:
539

💛  - Coveralls

authors: ProbstenHias coveralls


text:
Working on different Computers

Hello,
I'm currently working on the same project from different computers with Git.
As the whole project, I also copy the update.nupdproj with Git, and this seems to be a problem.
A path string is stored inside the update.nupdproj, and because i'm sometimes on another computer, it cannot find this local path.
It's not a big problem, i just edited the update.nupdproj myself, so it fits the path, but I think you could improve there a little bit.

Nochmals in Deutsch, da mein Englisch nicht das Beste ist.
Ich arbeite momentan an einem Projekt mit mehreren Arbeitsplätzen, wobei mit Git gearbeitet wird, um immer die aktuellste Version auf jedem Rechner zu haben (des Sources).
Nun hatte ich damals auf Rechner 1 das nUpdate Projekt erstellt und die update.nupdproj Datei einfach auch mit Git synchronisiert, damit ich Updates auch vom zweiten Rechner aus hochladen kann.
Leider scheint dies nicht so einfach zu gehen.
Ich habe herausgefunden, dass in der update.nupdproj ein lokaler Pfad als String gespeichert ist (der Datei-eigene Pfad) der natürlich auf einem anderen Rechner nicht mehr unbedingt stimmt.
Ich konnte das Problem umgehen, in dem ich die Datei mit einem Texteditor geöffnet und von Hand den Pfad geändert habe, so dass es danach stimmte.
Dennoch denke ich, dass man hier eventuell etwas ändern könnte, auch wenn es nicht direkt notwendig ist.
Greetings KillTrot

Nevermind...
You already implemented an import function...
Keeping my eyes open,
KillTrot

Hi KillTrot,
yes, that's indeed a problem in version 3.x. I need to admit that the project management is kind of messed up there (also due to some other facts). This will be improved/changed in version 4.x which is currently under development (see the develop-branch for more).
What you can do for now (and what you just found :D) is: Open nUpdate Administration, choose "Import/Export project", choose the "Export"-option and then follow the steps. This will create a .ZIP-file which you can import using nUpdate Administration on the other computer. Then everything should be fixed/done automatically.
I know this is not very convenient, but currently the alternative to doing it manually when simply copying the project file.
In v4 the projects will have a completely different structure and functionality that is suitable for source control and will allow you to simply synchronize the project data itself without having to adjust anything.
The reason why the paths etc. are hardcoded relies on a very early version of nUpdate (Administration) and due to time pressure I did not really improve it before the official release became public.
Sorry for that, but thanks for expressing your improvement proposal. I understand the problem and have already thought deeply about fixing it. The solution will be a project management that is based on the one of Visual Studio.
Best,
Dominic

authors: KillTrot ProgTrade

repo: ProgTrade/nUpdate


text:
Fix auto3dseg swin template infer bug and sigmoid bug

Fix #257
Fix swin infer bug which only run on cuda 0.

/integration-test

repo: Project-MONAI/research-contributions


text:
Template Manager Block crash with copying and loading same templates with gadget

Building Gadgets version
1.0.0-Beta
Minecraft Version
1.17.1
Modpack & Version
No response
Do you have optifine installed?
No
Describe the issue
game crashes with recursion and stack overflow from copying template to same template
Steps to reproduce
get two identical templates
attempt to copy one of them to the other template/gadget
Expected behaviour
not to crash
Screenshots
No response
Log files
No response
Additional information
No response
Information

[X] I have provided as much information as possible

it also crashes when saving over an existing template

Before attempting to click load again which causes stack overflow
[23:33:23] [Render thread/FATAL] (Minecraft) Error executing task on Client
 java.util.ConcurrentModificationException: null
	at java.util.WeakHashMap$HashIterator.nextEntry(WeakHashMap.java:811) ~[?:?]
	at java.util.WeakHashMap$KeyIterator.next(WeakHashMap.java:844) ~[?:?]
	at com.direwolf20.buildinggadgets.client.cache.CacheTemplateProvider.notifyListeners(CacheTemplateProvider.java:118) ~[main/:?]
	at com.direwolf20.buildinggadgets.client.cache.CacheTemplateProvider.setTemplate(CacheTemplateProvider.java:60) ~[main/:?]
	at com.direwolf20.buildinggadgets.common.network.bidirection.SplitPacketUpdateTemplate$Client.lambda$receive$0(SplitPacketUpdateTemplate.java:79) ~[main/:?]
	at net.minecraft.util.thread.BlockableEventLoop.doRunTask(BlockableEventLoop.java:151) [minecraft-1.17.1-project-@-mapped-loom.mappings-layered.1045691763.minecraft.1.17.1-v2.jar:?]
	at net.minecraft.util.thread.ReentrantBlockableEventLoop.doRunTask(ReentrantBlockableEventLoop.java:23) [minecraft-1.17.1-project-@-mapped-loom.mappings-layered.1045691763.minecraft.1.17.1-v2.jar:?]
	at net.minecraft.util.thread.BlockableEventLoop.pollTask(BlockableEventLoop.java:125) [minecraft-1.17.1-project-@-mapped-loom.mappings-layered.1045691763.minecraft.1.17.1-v2.jar:?]
	at net.minecraft.util.thread.BlockableEventLoop.runAllTasks(BlockableEventLoop.java:110) [minecraft-1.17.1-project-@-mapped-loom.mappings-layered.1045691763.minecraft.1.17.1-v2.jar:?]
	at net.minecraft.client.Minecraft.runTick(Minecraft.java:1075) [minecraft-1.17.1-project-@-mapped-loom.mappings-layered.1045691763.minecraft.1.17.1-v2.jar:?]
	at net.minecraft.client.Minecraft.run(Minecraft.java:728) [minecraft-1.17.1-project-@-mapped-loom.mappings-layered.1045691763.minecraft.1.17.1-v2.jar:?]
	at net.minecraft.client.main.Main.main(Main.java:217) [minecraft-1.17.1-project-@-mapped-loom.mappings-layered.1045691763.minecraft.1.17.1-v2.jar:?]
	at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:?]
	at jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:78) ~[?:?]
	at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:?]
	at java.lang.reflect.Method.invoke(Method.java:567) ~[?:?]
	at net.fabricmc.loader.game.MinecraftGameProvider.launch(MinecraftGameProvider.java:234) [fabric-loader-0.11.7.jar:?]
	at net.fabricmc.loader.launch.knot.Knot.launch(Knot.java:153) [fabric-loader-0.11.7.jar:?]
	at net.fabricmc.loader.launch.knot.KnotClient.main(KnotClient.java:28) [fabric-loader-0.11.7.jar:?]
	at net.fabricmc.devlaunchinjector.Main.main(Main.java:86) [dev-launch-injector-0.2.1+build.8.jar:?]```

repo: ProjectET/BuildingGadgets-Fabric


text:
Added unit test for USDC + rounding error (ERC20 tokens with less than 18 decimals)

Motivation
In #197 the rounding issue with partial fills was solved, with the limitation that the sale price is only allowed to have a certain amount of decimal places for the maths to safely check out. At the time of authoring that PR, it was still unclear how ERC20 tokens with less than 18 decimals are affected by the limitations laid out in #197.
Solution
This PR adds an additional unit test with a mocked USDC ERC20 token, to prove the eligibility of the solution as well as the further limitations.
Result
Prices for orders paid in ERC20 tokens with 6 decimals (like USDC) must not have more than two decimals, when used together with seaport-js' fees feature and partial fills (i.e. 1.23 per NFT is ok, 1.234 may break for certain fee-basisPoints values).
When not using fees, this limitation doesn't apply, i.e. you can use the full range of six decimals safely.

Codecov Report
Patch and project coverage have no change.

Comparison is base (ee2ba66) 93.13% compared to head (282b255) 93.13%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #198   +/-   ##
=======================================
  Coverage   93.13%   93.13%
=======================================
  Files          16       16
  Lines        3786     3786
  Branches      333      333
=======================================
  Hits         3526     3526
  Misses        250      250
  Partials       10       10

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

authors: codecov-commenter xtools-at

repo: ProjectOpenSea/seaport-js


text:
Datepicker position

Hello, after update the position of datepicker is not center of the screen as it was before, and that is problem for me because I have it in sidenav. Is it possible to center it as it was before update? Thanks

@marianharbist you can use container property to display datepicker in center, refer our doc

authors: dharmeshpipariya marianharbist


text:
Invalid email causes crash

Sylius: v1.7.0
SyliusPayumStripeCheckoutSessionPlugin: v1.1

Occurs when the billing page allows an invalid email to be input, such as "test@test". This causes the Stripe API to reject the payment, which in turn causes the crash.
Email Entered

Error In Log
request.CRITICAL: Uncaught PHP Exception Stripe\Exception\InvalidRequestException: "Invalid email address: test@test" at /var/www/sylius/vendor/stripe/stripe-php/lib/Exception/ApiErrorException.php line 38 {"exception":"[object] (Stripe\\Exception\\InvalidRequestException(code: 0): Invalid email address: test@test at /var/www/sylius/vendor/stripe/stripe-php/lib/Exception/ApiErrorException.php:38)"} []
Error Message In Browser

Is there a way to mitigate this. Such as using regex to validate an email like Stripe would. Or on error, return to the payment page with an error message?

Hello @ad3m3r5,
Do you change the default validator of Sylius ? The email is validated against the EmailValidator of Symfony from what I remember. Maybe their is some alterations there causing later the exception you pointed out.

I haven't touched any validation stuff. I thought it was kinda odd that it didn't pick that issue up.

Based on the symfony docs, there are 3 modes for the emailValidator (strict, loose, html5). All three seem as though they would catch something like "test@test", since it does not contain a valid domain name.
https://symfony.com/doc/current/reference/constraints/Email.html#mode

Here is the validator used : https://github.com/Sylius/Sylius/blob/master/src/Sylius/Bundle/CustomerBundle/Resources/config/validation/Customer.xml
It seems that the strict mode allow test@test, maybe changing to another can fit your need.

Thanks. I'll be testing out some regex patterns.
https://symfony.com/doc/current/reference/constraints/Regex.html


text:
Покупатель знакомится с описанием книги

Как "покупатель", я могу ознакомиться с детальным описанием книги, чтобы принять решение о её покупке.
Основной сценарий

Покупатель выбирает книги из списка найденных книг.
Сайт показывает детальное описание.

Добавить поле 'Price' к книге.
Почистить HTML.

repo: ProstoArtyom/bookStore

license: Unlicense


text:
Prowl Manual/Guides

Is your feature request related to a problem? Please describe.
I am having trouble keeping up with all the new changes and how they work. It would be nice to have somewhere we can internally document Prowl and all the features we are adding.
eg.

Command Line arguments for prowl #151 (including headless mode and it's CLI args),
how to create a gameObject,
how to create a shader,
how to use the UI.

Describe the solution you'd like
inside of the Prowl repository I can make a manual/ or doc/ or guides/ directory with .md files that deploy as a VitePress site and have it automatically publish any pushes to development and main (with separate sites for each branch)
Describe alternatives you've considered
We are already considering using docfx for API docs but a manual with guides documenting non-API related things would be nice.
Additional context
VitePress site Currently at https://prowl.pages.dev

Oh we also currently have some gitbook docs, but it looks like they are unused atm

This is the same as the create docs one :P there's no docs yet and won't be for a little while. I would also rather the docs be in another repo.
Can you move this to be a comment to the other issue?

authors: PaperPrototype michaelsakharov

repo: ProwlEngine/Prowl


text:
🛑 Service - Customer Portal is down

In 26d0e77, Service - Customer Portal (https://my.proxeuse.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Service - Customer Portal is back up in ef4c5d4 after 45 minutes.

repo: Proxeuse/status


text:
can u show us the stress test result data ？

i saw this : “Stress testing shows that ProxyKit is approximately 8% slower than nginx for simple forwarding on linux”.
can u show the data about the stress testing ?

Yeah I need to back that up with some substance. I'll leave this open to follow up on. In the mean time I'm going to remove that statement until further proof because, on reflection, it shouldn't be there without it.

thanks for reply，and i will keep an eye on

Never got around to doing a formal benchmark and because of #230 I won't. However since Microsoft are building one in a similar fashion (kestrel -> middleware -> httpclient) for their Azure partners, it lends credence to the fact it'll be fast enough.

authors: damianh witskeeper

repo: ProxyKit/ProxyKit


text:
Moving Platforms

Anyway someone can provide a simple platformer with one moving platform?

halfway pseudocode until I make something else
import { System, deg2rad } from 'detect-collisions';

const physics = new System();
const platformProps = { pos: { x: 100, y: 200 }, width: 100, height: 50, speed: 1 };
const platform = physics.createBox(platform.pos, platform.width, platformProps.height);

const canvas = document.createElement('canvas');
const context = canvas.getContext('2d');
document.body.appendElement(canvas);

setInterval(() => {
  const degrees =  (platform.x <= platformProps.minx) ? 0 : 180;
  platform.angle = deg2rad(degrees);
  platform.move(platform.speed);
  physics.draw(context);
}, 1000 / 60);

something like this (not tested) should give you a moving platform
allow me to also point you to my other library that may be useful for your case:
https://github.com/prozi/oneforall
in the case you'd use it you would

create Scene
create GameObject/Sprite/Animator/Container
create for above a body (PolygonBody)
bind to its update$ function like

gameObject.update$.pipe(takeUntil(gameObject.destroy$))
  .subscribe((deltaTime: number) => {
    // the example 1 interval content
  });

this would further ogranize your code and make sure when you destroy gameObject its body and children are destroyed properly too
have fun!

Thank you for the reply, but this doesn't solve my problem, probably because I didn't word my question correctly.
I have a player and a platform body, both using a Box. The player moves and jumps around collides with other Boxes in my scene. But when it comes to a moving platform the player does not move with the platform. How can I keep the player stuck/on to the platform so when the platform moves the player moves with it? And let's assume the platform is wide so the player can move on the platform while the platform is moving.

@gregmax17 this assumes gravity = physics.
there are currently no plan to do it in library
you may do this fairly easy yourself
you should check if player collides with platform
if yes move player along x axis the same amount as platform

since neither friction nor gravity is going to be added to the library because of philosophy that it should not be a full physics engine but a base for your own - closing sorry

repo: Prozi/detect-collisions


text:
Compatibility with Flux

I think it would be really useful to allow interoperability with things like Flux, Zygote, etc...
I think much of the benefit of this package would be applicable to other projects if a SimpleChain could be used without attaching a loss or optimizer or anything else, but simply used for its allocation-free forward and backward pass.  So you should be able to do something like:
using SimpleChains, Flux

model = SimpleChain(8, TurboDense(SimpleChains.tanh, 8))
p = SimpleChains.init_params(model)
Flux.train!(
    # 'loss' function, returns the value to be minimized
    (x, y) -> Flux.Losses.mse(model(x, p), y),
    # Parameters to be optimized over
    p,
    # dataset
    [(randn(8,1), randn(8,1))],
    # optimizer
    Flux.Optimise.ADAM()
)

While we would still deal with the overhead of Zygote, Flux's optimizers, etc.... we would at least be able to eliminate our model's allocation burden, which may helpful for many users.

If I'm not mistaken, the definitions in https://github.com/PumasAI/SimpleChains.jl/blob/main/src/chain_rules.jl should be more than enough for this purpose. For example, your code snippet already works if I change p -> Flux.params([p]). More complex model configurations are also possible:
using Flux: mse # only required for the loss function. If you're making your own layers in a library, just write:
using Functors # for interop with Flux's module system
using Optimisers, Zygote # training loop essentials sans Flux
using SimpleChains

# Example drop-in layer that will work wherever a Flux model is expected.
# Zygote and Optimisers will let you train wrt. the params vector directly,
# but bundling state and behaviour opens up the possiblility of working with higher-level libraries like FastAI.jl.
struct WrappedSimpleChain{M,P}
  model::M
  params::P
end
@functor WrappedSimpleChain

(m::WrappedSimpleChain)(x) = m.model(x, m.params)

loss(m, x, y) = mse(m(x), y)

let
  sc = SimpleChain(8, TurboDense(identity, 8))
  p = SimpleChains.init_params(sc)
  model = WrappedSimpleChain(sc, p)
  opt_state = Optimisers.setup(Optimisers.ADAM(), model)

  x = randn(Float32, 8, 1)
  y = 0.75x
  for i in 1:10
    grads, = gradient(model) do m
      loss(m, x, y)
    end
    opt_state, model = Optimisers.update!(opt_state, model, grads)
    @info i, loss(model, x, y)
  end
end

So not only does the compatibility appear to be there, but it looks pretty future-proof as well!

Note that SimpleChains memory management is not threadsafe, and thus requires manual management.
The train_*! methods manage memory manually so they can multithread safely.
But simple calls/gradient calculations are (a) not multithreaded and (b) manually batching and calling with Threads.@threads, Threads.@spawn, or Polyester.@batch will give corrupted/wrong results.
You can work around this via having one SimpleChain per task.

authors: ToucheSir chriselrod staticfloat


text:
Unwanted text

The text at the top right keeps appearing while I enable and disable the cruise control is there anyway to remove the top right text?
Edit update: Even if I'm not in a vehicle and I push the keybind for Cruise control it still shows the text in the top right.

Go to client.lua line 56 and 85 and delete the - sign. If that doesnt work then maybe its your chat script problem.

It was a + sign but either way removing it from both lines as fixed the issue thanks for the quick reply and help!

authors: Puntzi WhosHaci


text:
iOS TitleView Routing.RegisterRoute issue

Thanks for the iOS TitleView workaround!
Unfortunately, this only appears to work / apply(?) to pages registered in AppShell.xaml, not for subpages that are manually registered via route in MauiProgram.cs (Routing.RegisterRoute("routingpage", typeof(RoutingPage));)
Minimal example: https://github.com/MiSteiner/MauiTitleViewBug

DemoPage registered in AppShell.xaml works perfectly (open via Flyout)

RoutingPage registered in MauiProgram.cs (open via Button on MainPage)

custom TitleView is shown with a delay after the page transition is finished & has the wrong size

Would it be possible to fix this in the workaround?

@pictos

@PureWeen, this doesn't look to be the large title feature, I guess you marked me because of that. Anyway more than happy to take a look on the TitleView during this week and see what I find

@pictos oh whoops :-p
ignore me. I'll take this one.

authors: MiSteiner PureWeen pictos

repo: PureWeen/ShanedlerSamples


text:
QuadTree: Back with something faster than a List (Benchmarks?)

Currently QuadTrees use a List in the background to store the quads, this is probably very slow.
Would be good to benchmark, possibly against RTree2D as a reference implementation?

Hmmm. Had a couple of goes at this expecting there to be a few easy wins, but not so much.

Current benchmarks look like this:

I suspect the work here really is to make the underlying operations faster. Things like approximate Vertex comparison and line intersection tests and things like that.

Update: The initial assertion here that QuadTrees are backed by Lists is wrong. The implementation uses lists in a bunch of different ways, but the actual tree is a nested ...tree ...structure made up of case objects and case classes.

repo: PurpleKingdomGames/indigo


text:
Callables whose return value is ignored

def foo(callback):
    callback()

There's a few ways to type this function:
def foo(callback: Callable[[], None]) -> None: ...
def foo(callback: Callable[[], Any]) -> None: ...
def foo(callback: Callable[[], object]) -> None: ...

Callable[[], None] is not great, because it doesn't allow you to pass a callback that does return something; that will also work at runtime, and the return value is ignored. When I learned about this, I changed my callbacks to use Callable[[], Any] instead. But Callable[[], object] would be even better.
We could check for this, but does it belong to this tool? This lint isn't really stub-specific.

We could check for this, but does it belong to this tool? This lint isn't really stub-specific.

I'm starting to wonder if we should split the tool in two: one flake8 plugin for checks which only apply for stubs, another which has typing-related checks for .pyi files and .py files alike.

We could also try to get some of the non-pyi-specific checks into flake8/pyflakes(?).

We could also try to get some of the non-pyi-specific checks accepted into flake8/pyflakes(?).

We could try, but I think they would probably be rejected. I think the checks we have here are probably too typing-specific.
There are other plugins that specialise in providing checks for annotations, e.g. https://github.com/sco1/flake8-annotations. The checks https://github.com/sco1/flake8-annotations currently does are pretty basic, though, imo.

authors: Akuli AlexWaygood srittau


text:
PyPI rejects package published from default CI.yml with unsupported platform tag errors

Using Maturin v1.2.3 and a stock CI.yml created by maturin new, my project builds successfully for all platform/architecture targets. But trying to publish all the compiled wheels to PyPI yields the following messages for at least a couple Linux architectures:
400 Binary wheel 'vtracer-0.6.5-cp310-cp310-linux_i686.whl' has an unsupported platform tag 'linux_i686'.
Failures so far included linux_i686 and linux_aarch64.
I think it might be possible that PyPI would accept these wheels if they were labeled manylinux2014_i686 and manylinux2014_aarch64, but I'm not sure how to change the CI.yml to generate those wheels.
Plus, if the default CI.yml is failing for me, maybe it's stopping some other projects from publishing too.
Here's an example Github Action run:  https://github.com/etjones/vtracer/actions/runs/6124812960/job/16625644076

https://github.com/etjones/vtracer/blob/06f52dc3859b76a608858f5b05d338b4837a21fa/cmdapp/pyproject.toml#L27
You have disabled manylinux compatibility in pyproject.toml.

Thanks so much! That did the trick

authors: etjones messense


text:
New argument compute_on_cpu

What does this PR do?
Rework of #851
Fixes #848
General solution that allows all metrics where the metric state is a list to store the state on CPU instead of GPU through a new keyword argument compute_on_cpu=True/False (default False). This can beneficial in situations where the user do not have that much VRAM. Consequence of enabling this feature is that code in compute will be executed on CPU (code in update will still be executed on GPU).
Fixes #866
Related as the MAP metric currently moves all metric states to CPU for faster computations. As the issue points out this can overload the CPU. Feature in this PR essentially also fixes this by making it an option to move to CPU by setting MeanAveragePrecision(compute_on_cpu=True).
Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

A few questions:

If I'm reading code correctly, self.device will still be gpu with this during compute? I'm wondering if this will break some metrics.
The implicit assumption here is that a List state is only appended during update as otherwise we'll get device mismatches, right?
This doesn't help with other metrics that can occupy significant memory (e.g. binned curve metrics) on GPU as those are allocated as fixed size tensors. In that case though, I guess, a user can manually place them on CPU and transfer data to CPU before update. Is that what we would be recommending people?

@maximsch2 I think the device should indicate the input and output device of a metric. E.g. even if the metic states are on cpu, it would require the devices of the inputs to be cuda and handle the device-transfer logic accordingly and the output would also be on gpu.

@SkafteNicki mind resolve conflicts :rabbit:
@justusschock mind review? :fish:

@SkafteNicki @justusschock pls ^^ :rabbit:

seems there is a DDP issue... :/

authors: Borda SkafteNicki justusschock maximsch2

repo: PyTorchLightning/metrics


text:
Decouple "Convert to Lightning" from the Lite page

📚 Documentation
I'd like to suggest extracting "Convert to Lightning" from the Lite page. https://pytorch-lightning.readthedocs.io/en/latest/starter/lightning_lite.html
I'm having a look at the Lite doc page again, and I found it a bit confusing that the doc page has both of the following parts in a single page:

Pure PyTorch to Lightning Lite (convert-to-lightninglite)
Lightning Lite to Lightning (convert-to-lightning)

I think existing pure PyTorch users (who are interested in migrating their code to Lite) will get distracted by "Lightning Lite to Lightning" part because they visit the page to see what Lite is capable of and how to use Lite, but not to migrate their Lite code to Lightning. Putting both of the sections in a single page attracts some visibility for Lightning, but it can also be distracting.
What do you think? cc @borda @carmocca @justusschock @awaelchli @aribornstein

I agree with you 100% i think we should split it up as this page is quite overwhelming. Lets a have a simple example of how to organize the code and what it gets you and then a second page explaining how it works and how far you can take it.

@tchaton STEPPING STONE TO LIGHTNING is also a bad title we should make it clear exactly what lite does and why it's a stepping stone. This asumes you already know lightning in which case you probably aren't going to be reading about lite

I think it was added to shine some light on Distributed Training Pitfalls and how LightningModule and Trainer mitigate those issues. If someone is converting their code to Lite, we should show them it's not the ultimate tool and has its limitations, in case they want something better in a long run.

As @rohitgr7 shared, this was meant to indicate to the first glance that Lite is not the end of the journey and contains lot of limitations.

@rohitgr7 @tchaton I see your point, but I'm still in favor of moving the section to a different page. Lite is already very new to PyTorch users, and mixing another new thing (=Lightning) there can be confusing. I think only mentioning that there are limitations in Lite and linking to another page explaining "Lite to Lightning" would suffice in that case IMHO, but the decision is yours.

@awaelchli @carmocca what are your thoughts ?

I generally agree with Akihiro

As discussed with @kaushikb11 via Slack, let's keep the page as is for the reason we want to push not only Lite but also Lightning to pure PyTorch users.
Thanks @kaushikb11 for start discussing this :)

authors: akihironitta aribornstein carmocca rohitgr7 tchaton

repo: PyTorchLightning/pytorch-lightning


text:
Fix parameters and docs in metrics

What does this PR do?

Fixes #2377
Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together? Otherwise, we ask you to create a separate PR for every change.
[x] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?
[x] Did you verify new and existing tests pass locally with your changes?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Hello @rohitgr7! Thanks for opening this PR.

In the file pytorch_lightning/metrics/functional/classification.py:

Line 885:1: W293 blank line contains whitespace

Do see the Hitchhiker's guide to code style

just curious, shall be these default taken from annotation/typing?

@Borda sphinx does that anyway and writes it in the signature. So if we write the defaults in the docstring also, we risk it getting outdated if someone changes the default in the signature but forgets to update the docstring or vice versa. On the other hand, it is a preference thing, so I don't know. I think it should be decided which way we go and then consequently be done everywhere. Since docs are still changing a lot, I wouldn't worry too much about it.

@awaelchli exactly that was my point that we shall not duplicate it and rather use the automatic if possible...
so you recommend dropping this PR, eventually remove all defaults from docstrings?

Yes, I think more arguments speak for the automatic method. Also, annotations like "default: None" are not really helpful since they can lead to special behaviours that need to be explained, otherwhise it's not very informative. My view is that simplicity and consistency leads to more clarity.


text:
Integrate Tracy for better profiling support?

Right now I use perf and hotspot for basic profiling. That works well enough, but does not really provide much detail over time - only aggregates at the end of a run. Perhaps we could try to integrate Tracy, which can be used to output many more metrics on a pretty neat dashboard. This guide could be helpful for some initial experimentation: https://luxeengine.com/integrating-tracy-profiler-in-cpp/.

There's also a nice post here: https://developer.codeplay.com/products/computecpp/ce/2.3.0/guides/computecpp-profiler/using-the-tracy-profiler-with-computecpp.

And the interactive demo: https://tracy.nereid.pl/.

repo: PyVRP/PyVRP


text:
Add a tutorial

This is very much a moving target but shows how a conference could be scheduled with the tool as is.
I think it highlights  a couple of things that can be removed from the data structures (we're duplicating things with the rooms which I'm not sure are necessary) but hopefully this tutorial helps with our discussion going forward. :)

Coverage increased (+0.006%) to 98.636% when pulling b320630aae067a229e60d8c5fb3f08a429e50a26 on 6-tutorial into 86d52345da3e6fe8103778381b256aacca450f09 on master.

Coverage increased (+1.4%) to 100.0% when pulling 6ad2a4e21ddd073ffaa9c9b8fc0bea4151644eaf on 6-tutorial into 86d52345da3e6fe8103778381b256aacca450f09 on master.

Coverage increased (+1.4%) to 100.0% when pulling 33477eac2eac14f6c1206a378c196c977eb21d47 on 6-tutorial into 86d52345da3e6fe8103778381b256aacca450f09 on master.

I hope you don't mind me butting in?

Mind?!! I'm delighted! Welcome aboard.

Coverage increased (+1.4%) to 100.0% when pulling 99c8358299b3b0f797e5ac64c81a5284e7399313 on 6-tutorial into 86d52345da3e6fe8103778381b256aacca450f09 on master.

Thanks for the comments @alexwlchan: very helpful!.
For some of your questions about the data setup this is all based on the current setup in the library. I think I'd suggest we remove the Room class: and replace this with simply using a string in slot.rooms:
So that would replace the initial setup of this Tutorial with:
>>> from conference_scheduler.resources import Room, Slot, Event

>>> talk_slots  = [Slot(room='Big', starts_at='15-Sep-2016 09:30', duration=30, session="A", capacity=200),
...                Slot(room='Big', starts_at='15-Sep-2016 10:00', duration=30, session="A", capacity=200),
...                Slot(room='Small', starts_at='15-Sep-2016 09:30', duration=30, session="B", capacity=50),
...                Slot(room='Small', starts_at='15-Sep-2016 10:00', duration=30, session="B", capacity=50),
...                Slot(room='Big', starts_at='15-Sep-2016 12:30', duration=30, session="C", capacity=200),
...                Slot(room='Big', starts_at='15-Sep-2016 13:00', duration=30, session="C", capacity=200),
...                Slot(room='Small', starts_at='15-Sep-2016 12:30', duration=30, session="D", capacity=50),
...                Slot(room='Small', starts_at='15-Sep-2016 13:00', duration=30, session="D", capacity=50),
...                Slot(room='Big', starts_at='16-Sep-2016 09:30', duration=30, session="E", capacity=50),
...                Slot(room='Big', starts_at='16-Sep-2016 10:00', duration=30, session="E", capacity=50),
...                Slot(room='Big', starts_at='16-Sep-2016 12:30', duration=30, session="F", capacity=50),
...                Slot(room='Big', starts_at='16-Sep-2016 13:00', duration=30, session="F", capacity=50)]
>>> workshop_slots = [Slot(room='Small', starts_at='16-Sep-2016 09:30', duration=60, session="G", capacity=50),
...                   Slot(room='Small', starts_at='16-Sep-2016 13:00', duration=60, session="H", capacity=50)]
>>> outside_slots = [Slot(room='outside', starts_at='16-Sep-2016 12:30', duration=90, session="I", capacity=1000),
...                  Slot(room='outside', starts_at='16-Sep-2016 13:00', duration=90, session="J", capacity=1000)]
>>> slots = talk_slots + workshop_slots + outside_slots

And everything else works just fine (the room is actually just decorative, the session is used by the algorithm underneath).
What do you think @meatballs ?

Yeah, let's get rid. We can always add it (or something like it) back if we need a more friendly interface.
I was actually starting to think about how it might not be a 'room' anyway and, at the very least, needed renaming to 'venue' or something else.

Yeah, let's get rid. We can always add it (or something like it) back if we need a more friendly interface.
I was actually starting to think about how it might not be a 'room' anyway and, at the very least, needed renaming to 'venue' or something else.

Ok cool, I'll pick that up in this PR?

Coverage remained the same at 100.0% when pulling 1569cb001c42444bdfb39260a752b6ead4a0f239 on 6-tutorial into ac27c72921546c7bafd3296cfc4366d680126f0f on master.

I've made those changes in this one: also changes rooms to venue.

authors: coveralls drvinceknight meatballs

repo: PyconUK/ConferenceScheduler


text:
replace terminal talk with fuzz testing talk

I've renamed the file too, and updated the schedule link

Thanks @tomviner -- unfortunately, this broke rendering the schedule, so I've fixed in #180.

authors: inglesp tomviner


text:
Support custom labels in TOC.

When the Table of Contents Extension is used in conjunction with the Attribute Lists Extension  and a data-toc-label attribute is defined on a header, the content of the  data-toc-label attribute is now used as the content of the Table of Contents item for that header.

Interesting workaround for sane tocs with complex headers.

Yeah, I've had the idea bouncing around for a while, but was thinking of using the title attribute. I wasn't crazy about it though as there is a legitimate use for titles and they are potentially visible to end users (and screen readers). Today it occurred to me that we could use a data-* attribute. And then I had it working with only one line of code. Writing the documentation was the hardest part.
We should probably still properly fix #660, although this would serve as a workaround in the meantime. Note sure if there is anything else we could do for #699 though.

The last commit (537f418) just added a another (unrelated) feature to TOC:
A new toc_tokens attribute has been added to the Markdown class, which contains the raw tokens used to build the Table of Contents. Users can use this to build their own custom Table of Contents rather than needing to parse the HTML available on the toc attribute of the Markdown class.

authors: facelessuser waylan

repo: Python-Markdown/markdown


text:
Added twitter topic modeling and sentiment analysis

Description
A short summary of what is included in your Pull Request.
Fixes #31
Replace issue_no in the above line, with the issue related to this PR.
Type of change
Choosing one or more options from the following as per the nature of your Pull request.

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Documentation Update

Checklist:
Please tick all the boxes that are fulfilled by your Pull Request.

[X] I have named my files and folder, according to this project's guidelines.
[X] My code follows the style guidelines of this project.
[X] My Pull Request has a descriptive title. (not a vague title like Update index.md)
[X] I have commented on my code, particularly in hard-to-understand areas.
[X] I have created a helpful and easy to understand README.md, according to the given README_TEMPLATE.
[X] I have included a requirements.txt file (if external libraries are required.)
[X] My changes do not produce any warnings.
[X] I have added a working sample/screenshot of the script.

Nice Job

Thank you for the opportunity!


text:
现在这个项目还可以使用吗

我爬取结果报 INFO 后面一串网址

一直正常可用呀？我一直用的没问题，麻烦发下错误截图吧，谢谢。

authors: Germey Masterofmystery


text:
Need proper documentation for Authentication and Authorization

Would you please provide proper documentation for Authentication and Authorization. How to protect route in controller and so on that we get normally in NestJS

@sayedcsekuet Yes! I will supply this docs. In the meanwhile, if you have ideas for more things to improve, i would be happy to hear

@itay-dar-lmnd Thanks for your reply, It Would be great if you provide routing related documentation also

authors: itay-dar-lmnd sayedcsekuet

repo: PythonNest/PyNest


text:
🛑 QAInsights is down

In c744dff, QAInsights (https://qainsights.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: QAInsights is back up in ba37790.

repo: QAInsights/UpTime


text:
pullrequest

Issue Number
fixes #

Describe the changes you've made
A clear and concise description of what you have done to successfully close your assigned issue. Any new files? or anything you feel to let us know!
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
A clear and concise description of it.
Additional context (OPTIONAL)
Add any other context or screenshots about the feature request here.
Test plan (OPTIONAL)
A good test plan should give instructions that someone else can easily follow.
How someone can test your code?
Checklist

[ ] I have added my name in the contributors list at the end of README.md file.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] The title of my pull request is a short description of the requested changes.
[ ] Open Source Program names(OPTIONAL: If you participated in any open-source program then please mention the program name here)

@Anushka-Gamad what is this?

authors: Anushka-Gamad QAZIMAAZARSHAD

repo: QAZIMAAZARSHAD/Income-Tax-Calcultor


text:
docker目录qa-community-rust中，entrypoiny.sh文件名错误，正确应该是entrypoint.sh

https://github.com/QUANTAXIS/QUANTAXIS/tree/master/docker/qa-community-rust中，entrypoiny.sh文件名与Dockerfile文件中的对应的copy指令行文件名entrypoint.sh不符，会导致build镜像报错

fixed

repo: QUANTAXIS/QUANTAXIS


text:
#175 initial implementation of Read API V2

Description
This PR provides implementation for the read part of the Spark DataSource API V2. Once merged it should close the issue #175.
At the moment the following items are pending:

Filter push down which should be ported from DataSource V1 implementation
Sampling push down
Limit push down
Tests for QbeastDataSource and dependent classes.

This PR starts as a draft to make it possible to review it ASAP, the pending items mentioned above
will be added to the same PR later as soon they are ready.
Type of change
This change is internal process of migrating the Qbeast code to the Spark DataSource API V2. After the changes are applied the library should be backward compatible modulo it requires version of Spark 3.+.
Checklist:

Implement the Read API V2.
Port filter push down from the DataSource V1 implementation.
Implement sampling push down.
Implement limit push down.
Fix QbeastDataSource tests.
Create tests for Read API V2 implementation.
Probably remove Read API V1 implementation.

How Has This Been Tested? (Optional)
Since the change is internal migration process it should be enough to use existing high level integration tests and to add unit tests for the new code.

This is a work in progress, but we should consider already supporting columnar reading. Is it really necessary/desirable to create single files for each class, even if they are very small? I know it is a requirement in Java but in Scala? I can see the benefits of both approaches; I'm wondering what our is sweat_smile

IMHO the project is mainly in Scala, so I would prefer to use Scala as much as possible to avoid mixing the languages. Regarding the code style I tried to follow the "official" recommendation https://docs.scala-lang.org/style/files.html. Honestly I think that Java and Scala use too fine grained approach, so if you believe that the official strategy is not optimal, please let me know, it is easy to place those classes in a single file. Thank you.

Just one more comment: I am not sure if we are using the V2 implementation right now.
To make a fallback to V1 for reading, we made a rule on the Spark Plan that removes the calls to DataSourceV2Relation and replaces it to a LogicalRelation. Could you remove this rule and run the test?
It is placed here: https://github.com/Qbeast-io/qbeast-spark/blob/cb19cd40e229be9d9783d18de09822c10712ffbd/src/main/scala/io/qbeast/spark/internal/rules/QbeastAnalysis.scala#L17
And initialised on the SessionExtension: https://github.com/Qbeast-io/qbeast-spark/blob/cb19cd40e229be9d9783d18de09822c10712ffbd/src/main/scala/io/qbeast/spark/internal/QbeastSparkSessionExtension.scala#LL20C4-L20C4

Do you think we can close this PR for the moment? @cugni

Hello! Is this PR relevant at the moment? Should we close it and pause until other priorities are resolved? @alexeiakimov

I have no idea when we continue working on this, probably the codebase will be different, so let's close it.

Closing until the codebase is stable to continue the work.

repo: Qbeast-io/qbeast-spark


text:
升级android studio 3.0 正式版本后RePlugin项目构建编译失败

问题详细描述 Detailed description of the problem
复现问题步骤 Steps to reproduce the problem

gradlew -p demo1 rpInstallPluginDebug

其它重要信息 Other important information
F:\MyAsProject\RePlugin\Code\RePlugin_Sample>gradlew -p demo1 rpInstallPluginDebug
FAILURE: Build failed with an exception.

What went wrong:
A problem occurred configuring root project 'RePlugin_Sample'.

Could not resolve all files for configuration ':classpath'.
Could not find com.android.tools.build:gradle:3.0.0.
Searched in the following locations:
file:/C:/Users/jy/.m2/repository/com/android/tools/build/gradle/3.0.0/gradle-3.0.0.pom
file:/C:/Users/jy/.m2/repository/com/android/tools/build/gradle/3.0.0/gradle-3.0.0.jar
https://jcenter.bintray.com/com/android/tools/build/gradle/3.0.0/gradle-3.0.0.pom
https://jcenter.bintray.com/com/android/tools/build/gradle/3.0.0/gradle-3.0.0.jar
https://repo1.maven.org/maven2/com/android/tools/build/gradle/3.0.0/gradle-3.0.0.pom
https://repo1.maven.org/maven2/com/android/tools/build/gradle/3.0.0/gradle-3.0.0.jar
Required by:
project :
Could not find com.android.tools.build:gradle:3.0.0.
Searched in the following locations:
file:/C:/Users/jy/.m2/repository/com/android/tools/build/gradle/3.0.0/gradle-3.0.0.pom
file:/C:/Users/jy/.m2/repository/com/android/tools/build/gradle/3.0.0/gradle-3.0.0.jar
https://jcenter.bintray.com/com/android/tools/build/gradle/3.0.0/gradle-3.0.0.pom
https://jcenter.bintray.com/com/android/tools/build/gradle/3.0.0/gradle-3.0.0.jar
https://repo1.maven.org/maven2/com/android/tools/build/gradle/3.0.0/gradle-3.0.0.pom
https://repo1.maven.org/maven2/com/android/tools/build/gradle/3.0.0/gradle-3.0.0.jar
Required by:
project : > com.qihoo360.replugin:replugin-host-gradle:2.2.2
project : > com.qihoo360.replugin:replugin-plugin-gradle:2.2.2

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

Get more help at https://help.gradle.org

BUILD FAILED in 5s
Android API Version：
Android 手机型号&ROM(Phone model & ROM):
Logcat上下文 Logcat context

Could not find com.android.tools.build:gradle:3.0.0.
看起来像你自己电脑的网络问题

在你项目的gradle中，加入google()

authors: CaptainJno a3349384 exhange


text:
approve:add the function to generate a pdf file ?

how to export a pdf or doc file ? this function is very useful, i hope  you can develop it,thanks

Thanks @o624366705
Can you tell me what you are going to do it?

Provide User's Manual and Operating instructions and Requirement document  as required for Most of the government or enterprise customers,they need a static file. that funny-looking !

I'm a backend developer. I'm not familiar with vue,So I want someone to provide it

Maybe we can generate pdf on the fly by https://github.com/alvarcarto/url-to-pdf-api
For example:
https://url-to-pdf-api.herokuapp.com/api/render?url=https%3A%2F%2Fdocsify.js.org%2F%23%2Fmarkdown

thanks a lot,it likes to be useful

@QingWei-Li Does it look good in PDF?

authors: QingWei-Li iamchathu o624366705

repo: QingWei-Li/docsify


text:
Qubits are not reset correctly when running multiple circuits in the same job on a real backend

Information

qiskit-ibm-provider version: 0.8.0
Python version: 3.12.0
Operating system: Windows 11

What is the current behavior?
I'm creating two circuits on one qubit each:

circuit 1 adds a single X gate (no measurement)

circuit 2 has just a measurement

Run these circuits in a list on a single job on a real IBM backend (I used 'ibm-osaka', but It was reproduced on other systems)
job = backend.run([circ1, circ2], shots=100)

The resulting counts for the second circuit should be all "0" since it only does a measurement, but they are 50% "1" and 50% "0"
Total counts (for both circuits) are: [{'0': 100}, {'0': 52, '1': 48}]

This happens consistently on every job with this configuration.
I tried adding the circuits in alternation [circuit1, circuit2, circuit1, circuit2] and the 50/50 measurement only happens at odd alternations.
I also tried transpiling the circuits against the real backend to make sure the same qubit is used on both circuits. Same result.
Steps to reproduce the problem
Run the following circuit
from qiskit import QuantumCircuit, transpile
from qiskit_ibm_provider import IBMProvider

provider = IBMProvider()
backend = provider.get_backend('ibm_osaka')

circ1 = QuantumCircuit(1, 1)
circ1.x(0)
display(circ1.draw('mpl'))

circ2 = QuantumCircuit(1, 1)
circ2.measure(0, 0)
display(circ2.draw('mpl'))

job = backend.run([circ1, circ2], shots=100)
results = job.result()
counts = results.get_counts()
print("\nTotal counts are:", counts)

What is the expected behavior?
The returned counts should be [{'0': 100}, {'0': 100}]
Suggested solutions

Issue has been resolved - https://github.com/Qiskit/qiskit-ibm-runtime/issues/1310#issuecomment-1889210897

repo: Qiskit/qiskit-ibm-provider


text:
line_length in text drawer needs a default

Informations

Qiskit Terra version: master
Python version:
Operating system:

What is the current behavior?
Large circuits are scrambled when using the text-based circuit drawer.

vs

Steps to reproduce the problem
Make a large circuit and draw it.
What is the expected behavior?
Suggested solutions
Have the text-based circuit wrap according to the size of the terminal or Jupyter notebook cell width, and split like the Matplotlib version does.

Did you try setting line_length option up?
On Thu, Nov 1, 2018, 6:17 PM Paul Nation <notifications@github.com wrote:

Informations

Qiskit Terra version: master
Python version:
Operating system:

What is the current behavior?
Large circuits are scrambled when using the text-based circuit drawer.
[image: image]
https://user-images.githubusercontent.com/1249193/47883034-ed28ae80-de01-11e8-9aa9-7f51a9a5ec56.png
vs
[image: screen shot 2018-11-01 at 6 16 35 pm]
https://user-images.githubusercontent.com/1249193/47883114-48f33780-de02-11e8-9c5d-9448bf45364c.png
Steps to reproduce the problem
Make a large circuit and draw it.
What is the expected behavior? Suggested solutions
Have the text-based circuit wrap according to the size of the terminal or
Jupyter notebook cell width, and split like the Matplotlib version does.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Qiskit/qiskit-terra/issues/1192, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAuy5Wz6l1LNQW1ZBXn5XnT_8eNXu_fIks5uq3MCgaJpZM4YJvmZ
.

That works, but why is it not set to some default value, say 80, that works for any circuit?

I think there are ways to know the dimension of the console that the code is running. However, I don't know how portable they are.
I have the feeling that 80 is very short for most of the modern scenarios.
Please, rename the issue to reflect the new enhancement request. Something like "line_length in text drawer needs a default", or similar.

There is a stdlib call in the shutil module: https://docs.python.org/3/library/shutil.html#querying-the-size-of-the-output-terminal. If it can't figure it out it falls back to assuming an 80x24 terminal. This is what I used when adding a __str__ method to the QuantumCircuit class in #911.

Let's use the tag type:bug for things that are broken. That is, they raise unexpected exceptions or/and the functionality that used to work now is not working. The type:enhancement tag is for things that work, but you prefer them to work differently. Or for additional funtionality that does not exists right now.

@mtreinish it seems very stright forward. Let's go for it. If I implemented in the visualizer, would yo remove it from the __str__?

Yes, if the text drawer automatically sizes the output by default there would be no reason to specify that for __str__ in #911

This issue is solved in PR #1200 .

authors: 1ucian0 mtreinish nonhermitian

repo: Qiskit/qiskit-terra


text:
Typo in U3Gate Documentation Example

Information

Qiskit Terra version: 0.16.0
Python version: 3.8.1
Operating system: Windows 10

Originally mentioned in https://github.com/Qiskit/qiskit/issues/1055
What is the current behavior?
An instance of pi is not defined correctly in the code under the examples section of the U3Gate documentation.

Steps to reproduce the problem
The problem can be found here under the examples section:
https://qiskit.org/documentation/stubs/qiskit.circuit.library.U3Gate.html
What is the expected behavior?
Pi should display as its correct symbol rather than "pi"
Suggested solutions
Add the missing backslash in front of "pi" in the lambda input variable of the example

I have fixed this issue and will have a pull request for the updated code ready soon.