torch.nn.MSELoss displays incorrectly

Hi, I realize you guys are trying to establish your own conventions.
But I think this particular convention of expecting all losses to sum across the batch dim is too surprising for regular pytorch users... Especially considering that it creates a silent error, this is the worst kind!
Examples where this sum-reduction convention is problematic:

If you pass torch.nn.MSELoss() to the eval_losses dictionary it will incorrectly be divided by the batch size, because it is assumed to use sum reductions.
If you consider certain other regression loss functions like R^2 (e.g. torcheval.metrics.R2Score) which is expected to have an interpretable range between 0-1 (but also uses mean reductions) it becomes much more confusing. It will return very small values without any warning indicating the model is very very bad.
At high resolutions the Lp and H1 losses could explode causing NaN gradients and weights.
Neural Operators are resolution invariant but Lp and H1 losses are not. This makes it difficult to compare such loss values across different resolutions (which is a primary use case of the library).

Proposed Solution(s):
Minimal Solution: For the Lp and H1 etc... losses just divide the output by the size of the batch dimension (inside the loss function). This will at least fix problems 1 & 2.
More General Solution: Alternatively you could just do what Pytorch itself does (e.g. torch.nn.L1Loss and torch.nn.NLLLoss) and adopt the mean reduction convention for loss functions. After all the purpose of Neural Operators is resolution invariance isn't it? Well regular Lp and H1 etc... losses are not resolution invariant. So why not adopt mean reduction?

Thank you @profPlum for the detailed issue - we simplified the API of the losses in https://github.com/neuraloperator/neuraloperator/pull/486
In short: to support arbitrary inputs (and not just regular grids), we allow users to optionally provide custom quadratures. In the absence of that quadrature, we assume a regular grid over a domain of a given measure (by default, now set to 1) but that can be specified by users (e.g. 2*pi for Navier-Stokes). This allows the loss to converge to the integral as we refine the discretization.
So by default, the loss ends up averaging over the spatial dimensions (dividing by 1/measure), and the user can choose whether to reduce over channels and batch via sum or mean, as in PyTorch. We are also averaging over the batch dimension in the trainer (not the loss directly), to allow us to efficiently compute averages over multiple mini-batches in a distributed manner (e.g. for the validation set).
With the new changes, the API should be clearer and default to what you would expect. Let us know if you have any other feedback or suggestions!

Awesome glad to see the update!

I agree on principle but then it becomes trickier to accumulate results over multiple devices and mini-batches (e.g. for a large validation set).
One option can instantiate a distributed meter handler that will properly accumulate averages and assume that the batch size of the input corresponds to the actual dimension average over (that assumption may not always hold, e.g. if a model augments the batch dim by, e.g. mirroring inputs).

The mean of means is not in general equal to the global mean, we are not guaranteed to have the same mini-batch size in each device/node using DDP, especially during evaluation, where we care mostly about accuracy. We'd need to track the mini-batch size in each device and accumulate these properly through an incremental mean handler.

The mean of means is not in general equal to the global mean, we are not guaranteed to have the same mini-batch size in each device/node using DDP, especially during evaluation, where we care mostly about accuracy. We'd need to track the mini-batch size in each device and accumulate these properly through an incremental mean handler.

Hmm that sounds like a strange customization option you support. I’m not sure how many people use it. But even so I think it is possible to overcome. You could multiply each metric by the batch size then divide by total data set size or something. That is my current hacky workaround.


ansible: Compatibility for Ubuntu on Raspberry pi 3-4

Many thanks for the PR @mickkael, sorry this hasn't been addressed sooner but do you mind opening a new PR at https://github.com/rancher/k3s-ansible and closing this one? The contrib/ansible stuff has been moved there for easier maintenance.

Closing as the ansible code has been removed from this project, please re-open this PR at https://github.com/rancher/k3s-ansible @mickkael


[kirkstone] Complete Milestone 17

Merged to other branches as well:
53d3b8f59..bfb09373a  master -> master-next


Prevent webhooks with 4xx response to be retried

AC

[ ] Only 5xx responses should trigger webhooks to be retried
[ ] Update docs -> describe how webhooks retry model works

Docs: https://github.com/saleor/saleor-docs/pull/1012


Click [decimal/int32/enum] DataGrid Header crash when publish aot.

Describe the bug
When I click the datagrid column header to sort which display decimal/enum/int32 field, the application will crash.
It works normal when not aot mode.
It works normal when dotnet run with PublishAot=true
It works normal when column data type is string/double
To Reproduce
I create a simple demo to reproduce this bug:

create a avalonia data grid demo.
create a TestModel which has decimal/int32/enum fields.
create a DataGrid to display TestModel data.
add <PublishAot>true</PublishAot> to csproj.
publish the application with NativeAot:dotnet publish -c Release
execute the published application:.\bin\Release\net8.0\win-x64\publish\AvaloniaDataGrid.exe
Click header to sort.

I have create a repo to reproduce that. https://github.com/dameng324/AvaloniaDataGridDemo
Expected behavior
do not crash when
Screenshots
you can simple download the code and reproduce it.
Environment

OS: Windows
Avalonia-Version: 11.0.6
dotnet sdk version: 8.0.100

Additional context
I have tried many types. result:

type
dotnet publish aot and run manually

int32
crash

decimal
crash

enum
crash

double
normal

string
normal

@workgroupengineering  Thanks, It works. Hope fixing it with a more elegant way in the future.


Add Dolby Surround (0040) to model parameters for VSX-1130

Thanks for the PR.
The VSX-1130 is closely related to the VSX-930 for which there is already an override definition containing the Dolby Surround extra listening mode, and two other modes. Does this override work on your VSX-1130? If it does, then a cleaner change would be to update the regex for the VSX-930 override definition to match the VSX-1130 as well.

I would say not close enough.
I got video info via your component and I assume this will disable querying the video info 'PARAM_ENABLED_FUNCTIONS_NO_VIDEO'

Ok, tested that PARAM_ENABLED_FUNCTIONS: PARAM_ENABLED_FUNCTIONS_NO_VIDEO and I lost all my video info, so that is not compatible with VSX1130
Added extended stereo and advanced stereo to sound modes as well.

I'm confused that your VSX-1130 loses video info with PARAM_ENABLED_FUNCTIONS: PARAM_ENABLED_FUNCTIONS_NO_VIDEO included.
What type of extra video information do you get from your VSX-1130? I understood that the VSX-1130 specs are almost identical to the VSX-930, so I am surprised with your test results.


Document the text_enabled param for plugins

I think the Django CMS docs could use a little more information about the text_enabled attribute, which is available on CMSPlugins.
It's a very cool attribute/setting, but it took me quite some time to find out that I was looking for this when I was developing a plugin for my work.

Hello @jeroenpeters1986,
I've moved your ticket to https://github.com/divio/djangocms-text-ckeditor/issues/478.
The text_enabled attribute is not used by the cms but rather by the text plugin.

@czpython that is allright, but the docs live in this repo, right?

@jeroenpeters1986 the main cms docs yes but this is a feature unique to the text plugin.

@czpython okay :-) My contribution would be to the docs, that's why I opened it here


Oops! Something went wrong. mapit.js

hey, guys about the problem of Oops! Something went wrong
https://s15.postimg.org/eozvu0c63/Screen_Shot_2016_09_27_at_2_37_26_PM.png
just you have to edit in file "jquery.mapit.js"
HENT I REMOVE < FROM SCRIPT CUZ IT DOESN'T APPEAR IN PREVIEW
document.write('script type="text/javascript" src="https://maps.googleapis.com/maps/api/js?sensor=false"></script');
with
document.write('script type="text/javascript" src="https://maps.googleapis.com/maps/api/js?sensor=false&key=YOU_KEY_API&callback=initMap"></script');
That's all i hope it help you guys ^_^

Issues related to Google Maps API and its key are resolved at version 0.3.0. Also setup instructions are up to date.


False negatives found via new 'reverse flipped' journals article

I was reading 'Matthias, L.; Jahn, N.; Laakso, M. The Two-Way Street of Open Access Journal Publishing: Flip It and Reverse It. Publications 2019, 7, 23. https://doi.org/10.3390/publications7020023'
and have seen that for some of the journals that have reverse flipped, the 'new' publisher has additionally paywalled all the old content that was made OA at the previous publisher/platform e.g. Redalyc or Scielo.
An example is the journal Actas Urológicas Españolas
Content is solely behind a paywall at Elsevier from 2012 onwards.
However content is OA at Scielo from 2003 to 2011.
Elsevier also make the 2003 to 2011 content available behind a paywall and it is where the DOI resolves to(!).
e.g. Vol35 issue 10 paywalled here: https://www.sciencedirect.com/journal/actas-urologicas-espanolas/vol/35/issue/10
Vol35 issue 10 open access here: http://scielo.isciii.es/scielo.php?script=sci_issuetoc&pid=0210-480620110010&lng=es&nrm=iso
Hundreds of articles to link-up. There may be many more such in the 152 journals identified by Matthias et al

https://support.unpaywall.org/public/tickets/26c380d0d1addbf6d9e032d04adc3ca596fa2f0fdbdb3bf609a23214886ba529


iPad: Some block games is empty

Describe the bug
iPad: Some block games is empty
Steps to reproduce the behavior
1.Navigate to https://arcade.makecode.com/beta#
2.Open every block games
Actual behavior
Some block games is empty.(Falling Duck, Space Destroyer, Level Up, Memory, Catch )

Additional context
1.OS: iPad mini4
2.arcade version: 0.18.15
3.Microsoft MakeCode version: 6.1.28

@Sheila111 can you please check live as well and see if this is a new bug in beta or not?

@abchatra Doesn't repro in live, so I think it's a new bug.

I can reproduce this bug whenever I have a Teams meeting running in the background. Only the on start block loads whenever I try to load any of the block games.
HW: Surface Book 2
OS: Windows 20.04
Arcade Version: 1.0.16
MakeCode Version: 6.1.42

@riknoll can you take a look. @aznhassan has a repro.

Took me a while to repro this, but was finally able to on a very old macbook. I think this is definitely perf related (will investigate)

Something timing out?


run tmux automatically if it's not running

For simplicity of use, i think it makes sense to automatically ensure tmux is running

+1

Thanks for the suggestion, but teamocil is a tool to use within tmux. It makes the assumption that tmux is running and I don’t want teamocil to have the task of launching tmux if it’s not.

You can simply do it like this:
    tmux attach || tmux new-session -d "teamocil sample" \; attach

I wraps this in a shell function to save keystokes:
ttt ()
{
    tmux -L bg attach || tmux -L bg new-session -d "teamocil sample" \; attach
}

@lins05 Thanks for the tip!


[ENH] Support multi-echo datasets in reports

These changes will report all echo times in cases of multi-echo datasets, and will eliminate a bug where additional echo times were reported as additional runs (e.g., four runs of a three echo dataset would be reported as 12 runs with one echo time).

LGTM, thanks!


29 11 mi trial process updates

Explain actions for for Cloud Team (on-call person):

how to create and hand of instance
how to delete instance

@daxmc99 I included your feedback - anything else we should update?


Combined get/list of Sandbox configs with instances

Changes:

Add sandboxesapi.SandboxWithConfig, which contains the two structures
Add sandboxesapi.Get and sandboxesapi.List, which return SandboxWithConfig
Update tests to use the new methods, GetInstanceRequest, GetConfigRequest, ListInstanceRequest, and ListConfigRequest are now tested implicitly through the combined methods

Poslední commit failnul kvůli tools intall, spouštím znovu.


[iOS] Config Amplify for Multi Environments

Describe the bug
Currently we are using amplify iOS SDK(1.26.1) working fine with staging environment. But we have requirement to setup preprod and prod environment.
So I am trying fetch the json amplifyconfiguration.json file based on the environment and passing amplify configuration and no error throwing during the first-time initialization.
similarly we have awsconfiguration.json file based on environment I am trying to pass JSONValue and no error throwing during the first-time initialization.
After renamed amplify and aws configuration file names then only the issue is happening , but without renaming the file how to handle for multiple environment's like stage, preprod and prod
PROBLEM But when I am trying to login facebook, google, apple sigin , email and phone number app is crashing.
Here is code:
private func initAmplify() {
    do {
#if DEBUG
        Amplify.Logging.logLevel = .verbose
#endif
        let awsCognitoAuthPlugin = AWSCognitoAuthPlugin()
        let awsConfiguration: JSONValue = getAwsJsonConfiguration(fileName: "awsconfiguration.preprod")
        try awsCognitoAuthPlugin.configure(using: awsConfiguration)
        try Amplify.add(plugin: awsCognitoAuthPlugin)
        let amplifyJsonPath = getAmplifyJsonFilePath()
        try Amplify.add(plugin: AWSCognitoAuthPlugin())
        if amplifyJsonPath != nil {
            let amplifyConfig = try AmplifyConfiguration(configurationFile: URL(fileURLWithPath: amplifyJsonPath!))
            try Amplify.configure(amplifyConfig)
        }
    } catch {
        helpers.logger?.error("Failed initializing Amplify", error: error, attributes: nil)
    }
}

func getAwsJsonConfiguration(fileName: String) -> JSONValue {
    if let path: String = Bundle.main.path(forResource: fileName, ofType: "json") {
        if let data = NSData(contentsOfFile: path) {
            if let rawString = JSON(data).rawString() {
                do {
                    let jsonValue = try JSONDecoder().decode(JSONValue.self, from: rawString.data(using: .utf8)!)
                    return jsonValue
                } catch {
                    return [:]
                }
            }
        }
    }
    return [:]
}

func getAmplifyJsonFilePath() -> URL? {
    guard let file = Bundle.main.url(forResource: "amplifyconfiguration.preprod", withExtension: ".json") else {
        return nil
    }
    return file
}

Steps To Reproduce

Lunch iOS app
Call initAmplify() didFinishLaunchingWithOptions from Appdelegate.
Dynamically configure AWSCognitoAuthPlugin with JSONValue
Get the Json file path from bundle and pass to Amplify configuration.

Expected behavior
It should work with all accounts(socials, email & iPhone number) when do to configure dynamically.
Amplify Framework Version
1.26.2
Amplify Categories
Auth
Dependency manager
Cocoapods
Swift version
5.0
CLI version
9.1.0
Xcode version
13.4
Relevant log output
No response
Is this a regression?
No
Regression additional context
No response
Device
Iphone 13
iOS Version
15.5
Specific to simulators
Iphone 13 Pro
Additional context
Below attached error screenshots for reference:

Hi @AdiPomelo, thanks for opening this issue.
Dynamic configuration like you're doing here is supported and should work. Let's try to figure out what's going on here. Can you please post the contents of the four configuration files you're using? Please make sure to strip out any confidential information. Once we know what those look like, we can try to reproduce what you're experiencing.
Thanks!

@atierian Thank you for quick reply.
Here is amplify json file looks like.
amplifyconfiguration.json
{
    "UserAgent": "aws-amplify-cli/2.0",
    "Version": "1.0",
    "auth": {
        "plugins": {
            "awsCognitoAuthPlugin": {
                "UserAgent": "aws-amplify/cli",
                "Version": "0.1.0",
                "IdentityManager": {
                    "Default": {}
                },
                "CredentialsProvider": {
                    "CognitoIdentity": {
                        "Default": {
                            "PoolId": "XXXXXXX",
                            "Region": "ap-southeast-1"
                        }
                    }
                },
                "CognitoUserPool": {
                    "Default": {
                        "PoolId": "XXXXXXXX",
                        "AppClientId": "XXXXXXXX",
                        "Region": "ap-southeast-1"
                    }
                },
                "Auth": {
                    "Default": {
                        "OAuth": {
                            "WebDomain": "XXXXXXXXX",
                            "AppClientId": "XXXXXXXXXX",
                            "SignInRedirectURI": "XXXX://auth/login",
                            "SignOutRedirectURI": "XXXXX://auth/logout",
                            "Scopes": [
                                "phone",
                                "email",
                                "openid",
                                "profile",
                                "aws.cognito.signin.user.admin"
                            ]
                        },
                        "authenticationFlowType": "CUSTOM_AUTH"
                    }
                }
            }
        }
    }
}

awsconfiguration.json
{
    "UserAgent": "aws-amplify/cli",
    "Version": "0.1.0",
    "IdentityManager": {
        "Default": {}
    },
    "CredentialsProvider": {
        "CognitoIdentity": {
            "Default": {
                "PoolId": "XXXXXXX",
                "Region": "ap-southeast-1"
            }
        }
    },
    "CognitoUserPool": {
        "Default": {
            "PoolId": "XXXXXX",
            "AppClientId": "XXXXXXXXX",
            "Region": "ap-southeast-1"
        }
    },
    "Auth": {
        "Default": {
            "OAuth": {
                "WebDomain": "auth.pre-production.XXX.co",
                "AppClientId": "XXXXXXXXXX",
                "SignInRedirectURI": "XXXX://auth/login",
                "SignOutRedirectURI": "XXXXX://auth/logout",
                "Scopes": [
                    "phone",
                    "email",
                    "openid",
                    "profile",
                    "aws.cognito.signin.user.admin"
                ]
            },
            "authenticationFlowType": "CUSTOM_AUTH"
        }
    }
}

Below mentioned key values will be changed based on environment.
CredentialsProvider -> CognitoIdentity -> Default -> PoolId
Auth -> Default -> OAuth ->  WebDomain
Auth -> Default -> OAuth ->  AppClientId
CognitoUserPool -> PoolId
CognitoUserPool -> AppClientId

@AdiPomelo I created an example of how to support multiple build variants previously which I have now shared here on GitHub.

BetaProd

Normally Amplify and the AWS SDK just have a single .json file for each one. You also would not want to include the staging and preprod configuration files in the prod build. It would reveal those endpoints in your app bundle. The sample project shows how you can use Build Schemes and Xcode configuration files to support build variants. Individual developers could even create their own named environments.
See the section on Cloud Configuration. You can create a folder for each build variant you want to support and place the .json files in those folders with the default names. Once you have this set up you can just change the selected Build Scheme in Xcode to build and run the app for the environment you want. Since you can set a unique App ID for each build you can have each app installed on a device or simulator concurrently with a different cloud configuration. And the tool linked in that repo will give you way to easily create app icons for the staging and prepared environments.
You could also set up an environment for integration testing which you run with CI/CD. Having isolated resources for this purpose would make your build automation more reliable and prevent interference with staging and preprod.
Let us know if this helps simplify your setup.

@brennanMKE Thank you for quick replay with sample project. I will try the solution suggested by you and let you know.

@atierian Any update ?

You don't need to use awsconfiguration.json when configuring Amplify.

Hey @AdiPomelo, just checking in. Did the snippet above resolve your issue?

@atierian  No.

Sorry to hear that. Is the error you're experiencing the same that you outlined in the original issue?

@atierian  Yes

Based on the information you provided and using the altered code snippet for configuring Amplify that I posted above, I am not able to reproduce the issue you're experiencing here. If you're still experiencing this issue, I suggest isolating it down to a small reproducible example that you can share so we can test it.

Closing due to inactivity.


Working directory should be project root when project entrypoint is used.

It's not uncommon for a Node.js application to list a file under ./lib as the main file for the project, but the working directory should almost universally be the project root.  At the very least, there should be an option to use the project root instead of the active file's directory.  (To be honest, even in those cases I suspect that using the project root as the working directory would be okay.)

👍  That's currently a big problem for me, as I use external modules that are build on that assumption that CWD/PWD is the project root.


ADFS on Server 2019 + Azure Gov

Update
Sorry about that - this error isn't reproducible in a Azure commercial tenant. It appears to be failing during picking up the tenant cert for Azure Gov (in case anyone gets a similar error) but it might be misconfiguration. Will update when I know more!
Original comment
Has anyone been able to successfully deploy Azure AD Connect Health with AD FS on Server 2019? I am getting a reproducible cryptographic unhandled exception on fresh installs of Server 2019 (fully patched) in various environments with the current version of AdHealthAdfsAgentSetup (<IP_ADDRESS>). This behavior doesn't exist on AD FS 2016. I've also tried importing the default AD FS 2016 cipher suites  (HKEY_LOCAL_MACHINE\SYSTEM\CurrentControlSet\Control\Cryptography\Configuration\Local\SSL) and completely disabling the firewall and IE ESC.
If Server 2019 is fully supported, please add any required configuration changes and update these references:

Supported operation system versions
AdHealthAdfsAgentSetup System Requirements

Expected behavior:
On Server 2019, Test-AzureADConnectHealthConnectivity -Role adfs returns:
PS C:\> Test-AzureADConnectHealthConnectivity -Role adfs
Test-AzureADConnectHealthConnectivity's execution in details are as follows:
Starting Test-AzureADConnectHealthConnectivity ...

Connectivity Test Step 1 of 3: Testing dependent service endpoints begins ...
AAD CDN connectivity is skipped.
Connecting to endpoint https://login.microsoftonline.com
Endpoint validation for https://login.microsoftonline.com is Successful.
Connecting to endpoint https://login.windows.net
Endpoint validation for https://login.windows.net is Successful.
Connecting to endpoint https://policykeyservice.dc.ad.msft.net/clientregistrationmanager.svc
Endpoint validation for https://policykeyservice.dc.ad.msft.net/clientregistrationmanager.svc is Successful.
Connecting to endpoint https://policykeyservice.dc.ad.msft.net/policymanager.svc
Endpoint validation for https://policykeyservice.dc.ad.msft.net/policymanager.svc is Successful.
Connectivity Test Step 1 of 3 - Testing dependent service endpoints completed successfully.

Connectivity Test Step 2 of 3 - Blob data upload procedure begins ...
Tenant Id is successfully collected during agent registration.
Connectivity Test Step 2 of 3 - Blob data upload procedure completed successfully.

Connectivity Test Step 3 of 3 - EventHub data upload procedure begins ...
Tenant Id is successfully collected during agent registration.
Connectivity Test Step 3 of 3 - EventHub data upload procedure completed successfully.

Test-AzureADConnectHealthConnectivity completed successfully...

Actual behavior:
On Server 2019, Test-AzureADConnectHealthConnectivity -Role adfs returns:
PS C:\> Test-AzureADConnectHealthConnectivity -Role adfs
Test-AzureADConnectHealthConnectivity's execution in details are as follows:
Starting Test-AzureADConnectHealthConnectivity ...

Connectivity Test Step 1 of 3: Testing dependent service endpoints begins ...
AAD CDN connectivity is skipped.
Connecting to endpoint https://login.microsoftonline.com
Endpoint validation for https://login.microsoftonline.com is Successful.
Connecting to endpoint https://login.windows.net
Endpoint validation for https://login.windows.net is Successful.
Connecting to endpoint https://policykeyservice.dc.ad.msft.net/clientregistrationmanager.svc
Endpoint validation for https://policykeyservice.dc.ad.msft.net/clientregistrationmanager.svc is Successful.
Connecting to endpoint https://policykeyservice.dc.ad.msft.net/policymanager.svc
Endpoint validation for https://policykeyservice.dc.ad.msft.net/policymanager.svc is Successful.
Connectivity Test Step 1 of 3 - Testing dependent service endpoints completed successfully.

Connectivity Test Step 2 of 3 - Blob data upload procedure begins ...
Unhandled exception occurred: System.Security.Cryptography.CryptographicException: The parameter is incorrect.

   at System.Security.Cryptography.ProtectedData.Unprotect(Byte[] encryptedData, Byte[] optionalEntropy, DataProtectionScope scope)
   at Microsoft.Identity.Health.Common.Clients.PowerShell.ConfigurationModule.TestAzureADConnectHealthConnectivity.LoadIdentityInfo()
   at Microsoft.Identity.Health.Common.Clients.PowerShell.ConfigurationModule.TestAzureADConnectHealthConnectivity.TestInsightServiceDataUploadProcedure()
   at Microsoft.Identity.Health.Common.Clients.PowerShell.ConfigurationModule.TestAzureADConnectHealthConnectivity.ProcessRecord()

Steps to reproduce:

Deploy Server 2019 from ISO
Complete all Windows Updates (2019-10 Cumulative Update at time of writing)
Validate all pre-existing Azure AD Connect Health Agent Installation requirements
Install latest version of Azure AD Connect Health Agent for ADFS (<IP_ADDRESS> at time of writing)
Execute Test-AzureADConnectHealthConnectivity -Role adfs

Originally posted by @milesgratz in https://github.com/MicrosoftDocs/azure-docs/issues/33795#issuecomment-543277488

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

ID: 1b4b9ec9-9ce6-dcff-b557-2816d40459b8
Version Independent ID: b2f56dc2-506e-4ff0-fec5-b7241f7fc4fc
Content: Azure Active Directory Connect Health FAQ - Azure
Content Source: articles/active-directory/hybrid/reference-connect-health-faq.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

This still appears to be an issue in Azure Gov on Server 2019. I'm getting the same unhandled crypto exception as @milesgratz described in #33795.

@john-crouch Thank you for your query . However , we use github channel for improvements in documentation. We will try to see if we can find something related to your issue and provide you an update. But in all probability this is an issue which may require deeper troubleshooting and we may request you to engage with Microsoft support on this. I will update this thread after reviewing and checking all the details related to the old issue you have linked.

@john-crouch Thank you for your patience with us and apologies for the late reply on this. I tried to find more information but was not successful . It would be better to work with the support team to find a solution for this issue to understand more. Please send an email to us on azcommunity[at]microsoft[dot]com referencing this thread and a subject line "ATTN Shashi" along with your azure subscription ID . I will help you with an one-time free support engagement for resolving this issue. We will now close this issue.
Thank you.


Found an issue when renaming overwrites existing file

I tried to rename a number of .epub files. Seven to be precise.
In the course of renaming, I changed one existing file to another name and gave a new file that previous name. So essentially I had something like (condensed to two files):
ABC -> ABC1
BCD -> ABC

It complained that the renaming of BCD to ABC would overwrite existing files.
There were two issues with that:

it didn't include all renames performed
it wrongly concluded that the renaming of BCD to ABC would overwrite the file which -- in the same renaming session -- had been reassigned the name ABC1

Here's the actual output:
$ renamer *.epub

The following replacements overwrite existing files:
Циолковский Константин. На Луне - royallib.com.epub -> На Луне (Константин Циолковский).epub

Execute these renames?: Yes

Thanks for the report! That doesn't surprise me because the existing files check only checks the state of things before the renames

This also causes the problem where --undo doesn't undo the affected files because it doesn't realize the conflict.


Test Windows installer with activities

Waiting to hear back from user on their operating system version (so that we can match).
User is receiving errors (related to teaching activity files) when attempting to install.

@oliver-phet those error messages would seem to suggest they copied the installer onto a computer using a USB drive, then tried to access the copied files after the USB drive was disconnected...or something.

@KatieWoe @oliver-phet I don't know if QA ever tests installing from a CD (I know I never did), which it looks like this user is doing?

It could have been a DVD or USB drive.

@phet-steele I've never done it.

Win 10 and Win 7 off internet seems fine.  I did run into some brief trouble by getting a network error when trying to download the installer on Win 7 chrome.  Using Firefox worked fine, and I believe this an issue with chrome or its settings.

@oliver-phet Just tested the installer on Win 8.1 and I didn't see any problems there either.  Let me know if you get more information about how their set up.

operating system is windows 7
Destination folder is in C:
Downloaded setup was at E:

@oliver-phet Just downloaded the installer on to a usb and used it to install on Win 7.  The drive was E: and it installed into C: with the same file path.  Didn't encounter any problems.  Sorry that's not very much use.

Ok, it is sounding like the users .exe file is corrupted. I'm going to have them re-download the installer. Closing for now.


[FLINK-18965][sql-client] Exclude hadoop-hdfs transitive dependency f…

…or flink-sql-client

What is the purpose of the change
Fix nightly test failures of ExecutionContextTest
Brief change log

Exclude hadoop-hdfs from hive connector dependency

Verifying this change
Existing tests & locally verified tests with Hadoop-2.8.3
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? NA

Do you think we should exclude hadoop in flink-connector-hive?

Maybe we should consider arranging the dependencies of flink-connector-hive.
I'll merge this PR first.


Upgraded to StructureMap v3

I've updated the project to use StructureMap v3, with the following changes:

Updated SolrNetRegistry.cs (added constructor dependencies to @nadeemc's pull request #144)
Added StructureMap as a NuGet reference (required a .NET 4.0 upgrade for SM dependent projects)

Note that tests won't pass if run under .NET 2.0 (I had to change Gallio.Host.exe.config on my PC but there may be a better solution)

To fix the compilation problem I have to set RuntimeVersion = "v4.0.30319" in the Gallio.Run command in the build.fsx, but I'm not sure whether this is a good idea or not. Should I proceed?

Completed


Base .value on .peek()

The only difference between .value and .peek() is that .value should set up a subscription, whereas .peek() shouldn't. This means .value can be based on .peek(). Both were looking already pretty similar in their implementation.

Since this is slightly larger, we should base the decision on performance. My fear is that we are doubling the function calls for every .value access.

The implementation in main is shortened a bit so this PR is redundant.


Next.js port v2

This ports the website from CreateReactApp to Next.js except this time it actually compiles

This PR fixes the compilation problems discussed in #1  but did not address the Netlify ones. After this PR, I installed the Netlify Next.js plugin and then re-deployed the entire website to get it working properly.


Negative values are no longer allowed for Fair Market Value Input

#631
Negative symbol is no longer allowed in the FairMarketValueInput
Changelog:

Prevent keypress of -
Paste should also detect if there's a - in the clipboard
Tooltip should be shown notifying the user negative values are not allowed

 New Credit Transfer price field | Update price per credit field so that negative values are not permissable |2|


Return "unknown" if show version info is disabled in Grafana

Description
We disabled displaying the version info because of ridiculous security Resasons.
Checklist

[ x ] The patch has appropriate test coverage
[ x ] The patch follows the style guidelines of this project
[ x ] The patch has appropriate comments, particularly in hard-to-understand areas
[ x ] The documentation was updated corresponding to the patch
[ x ] I have performed a self-review of this patch

Good Point! @amotl  I updated the request.

Note that there is atleast one API section which relies on Grafana version information to figure out whether the interfaces are supported.
Example :  https://github.com/panodata/grafana-client/blob/4bd145a78832a225ef83e304342f954ff6fba86c/grafana_client/elements/datasource.py#L177
Be aware that simply returning None might result in unexpected exceptions in these places until they are tweaked to handle this possibility.

Yes I know. I also have to work with lib elements where it's required. What else do you recommend? A Version not available Exception?

@schmiddim
Most of the version checks I've noticed seem to be related to deprecation warnings, and not for progressive upgradation of the interface. The version checks can be made insensitive to None versions where you find them (they already are in many places).
If grafana version isn't available, I think it's fair that the user creating that situation will need to take responsibility to deal with deprecation. So it should not be a problem to make that change, in my opinion.
The problem is going to be in going through and finding all of them. If @amotl is fine with this solution in principle, then I would recommend fixing it as and when you hit such a problem.

@chintal would be awesome. I also have to work with Library Elements and will fix things when I hit them.

@amotl
Allowing the user to set the version manually is probably fine. I do have these thoughts about it, though :

Setting the version manually should not be mandatory - even if the server does not report its version. If it isn't set, we simply don't check for deprecation. Documentation should also reflect this, and users should be discouraged from setting the version manually unless they know they need it.
Unless a clear use case is understood, a user should not be allowed / able to override the version reported by grafana (if it is available).
As a corollary to 2, if user code sets a grafana version, and grafana also reports its version, and grafana-client uses the grafana reported version silently, then all of a sudden the user code is running against a grafana / grafana-client version different from what it explicitly expects. This will lead to painful debugging triggered asynchronously when either grafana server configuration changes, or server version changes.

On the whole, my opinion is that it's a layer of complexity that creates as many problems as it solves, so whichever way you decide to go is fine.
In the long term, I wonder if instead of asking for permission (checking for deprecation against the grafana version), would it be better to ask for forgiveness instead? If there is a GrafanaServerError with the signature that deprecation would cause, then in the error message could we pack in relevant notes about deprecation of that endpoint? Such a change would need a more significant development effort, though, and I don't think you should do it without an integration test suite already in place.

Hi. Apologies that I lost a bit track of staying involved with this, and I am still a bit swamped. Do you all agree this patch should be merged as-is, and included into the next release?
If so, I will probably try to converge it soon. To do that, I would like to get a proper acknowledgment from the community (you!). Thank you very much in advance.

I think it can be merged. The change would not interfere with any currently working code.
In addition, changes should be made in other files where version is required. It can be probably be done separately as well, since that code probably isn't working under the no-version-available condition now anyway.
For example, at https://github.com/grafana-toolbox/grafana-client/blob/4bd145a78832a225ef83e304342f954ff6fba86c/grafana_client/elements/datasource.py#L177
if Version(self.api.version) > VERSION_10_2_2:
            raise NotImplementedError("Deprecated since Grafana 10.2.3")

can be changed to something like:
if self.api.version and Version(self.api.version) > VERSION_10_2_2:
            raise NotImplementedError("Deprecated since Grafana 10.2.3")

Hey will this MR be merged in the next time? I ran into the same Problem with version Exception

Hi again. Sorry so much for the delay. I will merge as-is, and run a release afterwards.

grafana-client 4.1.2, just published to PyPI, includes your improvement, @schmiddim. Thanks again!


Add a .NET client library reference

documentation: add a .net client library reference
updates wiki documentation

Please change the commit title from  Added a .NET client library reference to libraries-and-tools.md: add .NET client so this can pass the CI commit title check. After that it should be good to merge. Thanks!

@CodyBatt Ping?


Emergency trigger to use transient local qos

This makes the emergency topic subscriber transient local.

We don't actually publish on the emergency topic from the lift supervisor. A long time ago the intention was to do so, but it was later decided that the lift states are not necessarily a good source of truth for this, so now it's up to system integrators to decide when to publish emergency notices.
If anything we should delete that publisher from the lift supervisor.


feat: Plugin System Refactor

I now have most functionality back, I think I just need to reimplement metamethods for glam types, I.e. add sub mul

We wait with bated breath. God speed. :D

We wait with bated breath. God speed. :D

It's soooooo close, I can't wait to get this out :D

Okay that's all the functionality complete, I still need to fix up tests, and clean up / make docs but I will move this into staging


Improve behavior of image columns

Don't show anything if there is no value
Show a thumbnail with a popover, avoid auto row height

https://user-images.githubusercontent.com/2109932/217227612-a283bf18-dcb3-43f9-a84d-2a2d18e54b65.mov

Is there a way to have more control over the image size, or should there?
Anyway this solution works for me in that sense as not sure what the best approach would be, could depend on different use cases...
Found a bug though, on the 2nd item the popover was looking really big to me and had a scroll bar:
https://user-images.githubusercontent.com/10789765/217327117-6e7f5299-5408-4d1a-9ec6-ad622e7704c4.mov

Is there a way to have more control over the image size, or should there?

You can always create a custom component. I believe this is a more sensible default as it just behaves the way text behaves, i.e. just take the available space. The previous solution uses getRowHeight: () => 'auto', which had the side effect that it resized the rows on other columns as well, which is not the intended behavior.

Found a bug though, on the 2nd item the popover was looking really big to me and had a scroll bar:

I limited the size of the popover image. I also made sure cell alignment still works


all these packages miss proper install targets... released Debians are pretty empty.

sorry, I was wrong... all good.


A teacher can provide their teacher reference number

So that we can pay a teacher their teacher student loan re-imbursement payment we need to make sure they are really a teacher. Collecting their TRN is one of the ways we will achieve this.
At this stage we are validating the presence of the TRN and normalising it to digits only before saving it to the database.
At the moment we think that storing only digits is the right thing to do based on our understanding of TRNs, the DQT uses only digits to sign teacher in. We have contacted a subject matter expert at DfE to gain more understanding.
We think we may need to encrypt the value in the database at some point, but this work does not include that.
https://trello.com/c/01yTwht6

We had a response back from someone at DfE, they weren't the right person but we have the name of someone who may be, we'll close this PR while we see what they say about TRNs.


build: add a dependency target for non-stdlib builds

When building without the standard library, we need to create the dependency
target manually to track the dependencies internally.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please test and merge


pull request for e2e test at 01/13/2017 09:26:06

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


device.LaunchApp() never resolves

Description
device.LaunchApp()  promise never resolves.
    console.log('*******Launching app');
    await device.LaunchApp({
      newInstance:true,
      permissions: {notifications: 'YES', microphone: 'YES'},
      launchArgs: {
        detoxPrintBusyIdleResources: 'YES',
      },
    });
    console.log('*******Launched App!');

*******Launched App! never prints.  Lots of the identical following line is printed in the log (note no reason: null):
[com.wix.Detox:EarlGreyStatistics] App State -> busy (null)

[x] I have tested this issue on the latest Detox release and it still reproduces

Reproduction
Our tests have worked for a while. We are having trouble pinning down why they started failing.

Execute tests via [test.js] node_modules/.bin/mocha --opts tests/mocha.opts --configuration ios.sim.debug.xcode11 --loglevel trace --debug-synchronization 2 --grep :android: --invert --record-logs failing --take-screenshots failing --artifacts-location "./build/testSummary/" --use-custom-logger "true" "build"
await device.LaunchApp() in before() hook of mocha test
Our app launches and sits on the landing page (login prompt)
Notice the Promise never resolves/rejects.  Mocha timeout hit

Expected behavior
We expected a reason in the log for why synchronization mechanism is not detecting the state where tests can continue
Environment (please complete the following information):

Detox: 16.2.0
React Native: 0.60.6
Node: v12.14.1
Device: Simulator - iPhone 11 Pro Max (13.4)
Xcode: Xcode 11.4, Build version 11E146
iOS: 13.4
macOS: 10.15.4

Logs
package.json::detox.configurations:
      "ios.sim.debug.xcode11": {
        "binaryPath": "ios/build/Build/Products/Debug-iphonesimulator/OUR.app",
        "build": "export CODE_SIGNING_REQUIRED=NO && export RCT_NO_LAUNCH_PACKAGER=true && xcodebuild -workspace ios/OUR.xcworkspace -UseModernBuildSystem=NO -scheme 'Our' -sdk iphonesimulator -configuration Debug -derivedDataPath ios/build",
        "type": "ios.simulator",
        "name": "iPhone 11 Pro Max"
      },

Std Out/Err:
detox[44722] TRACE: [ArtifactsManager.js/LIFECYCLE] artifactsManager.onLaunchApp({
  bundleId: 'com.OURAPP,
  deviceId: 'removed,
  launchArgs: {
    detoxServer: 'ws://localhost:52266',
    detoxSessionId: 'removed',
    detoxPrintBusyIdleResources: 'YES'
  },
  pid: 44722
})
detox[44666] TRACE: [AsyncWebSocket.js/WEBSOCKET_SEND] {"type":"isReady","params":{},"messageId":-1000}
detox[44666] TRACE: [DetoxServer.js/MESSAGE] role=tester action=isReady (sessionId=1f781c4b-3b62-f054-49e3-c483d6fbcbfc)
detox[44666] DEBUG: [DetoxServer.js/CANNOT_FORWARD] role=testee not connected, cannot fw action (sessionId=1f781c4b-3b62-f054-49e3-c483d6fbcbfc)
detox[44666] DEBUG: [DetoxServer.js/LOGIN] role=testee, sessionId=1f781c4b-3b62-f054-49e3-c483d6fbcbfc
detox[44666] DEBUG: [DetoxServer.js/LOGIN_SUCCESS] role=testee, sessionId=1f781c4b-3b62-f054-49e3-c483d6fbcbfc
1) "before all" hook <-- std err, failure

If you are experiencing a timeout in your test

[x] I have followed the instructions under Lower-level Idling Resources Debug and am providing the log below:

Note the sheer number of entries in 1 ms:
2020-04-15 09:53:29.181 REMOVED[44442:ee0a4] [com.wix.Detox:EarlGreyStatistics] App State -> busy (null)
2020-04-15 09:53:29.182 REMOVED[44442:ee0a4] [com.wix.Detox:EarlGreyStatistics] App State -> busy (null)
2020-04-15 09:53:29.182 REMOVED[44442:ee0a4] [com.wix.Detox:EarlGreyStatistics] App State -> busy (null)
2020-04-15 09:53:29.182 REMOVED[44442:ee0a4] [com.wix.Detox:EarlGreyStatistics] App State -> busy (null)
2020-04-15 09:53:29.182 REMOVED[44442:ee0a4] [com.wix.Detox:EarlGreyStatistics] App State -> busy (null)
2020-04-15 09:53:29.182 REMOVED[44442:ee0a4] [com.wix.Detox:EarlGreyStatistics] App State -> busy (null)
2020-04-15 09:53:29.182 REMOVED[44442:ee0a4] [com.wix.Detox:EarlGreyStatistics] App State -> busy (null)
2020-04-15 09:53:29.182 REMOVED[44442:ee0a4] [com.wix.Detox:EarlGreyStatistics] App State -> busy (null)
2020-04-15 09:53:29.182 REMOVED[44442:ee0a4] [com.wix.Detox:EarlGreyStatistics] App State -> busy (null)
2020-04-15 09:53:29.182 REMOVED[44442:ee0a4] [com.wix.Detox:EarlGreyStatistics] App State -> busy (null)
2020-04-15 09:53:29.182 REMOVED[44442:ee0a4] [com.wix.Detox:EarlGreyStatistics] App State -> busy (null)
2020-04-15 09:53:29.182 REMOVED[44442:ee0a4] [com.wix.Detox:EarlGreyStatistics] App State -> busy (null)
2020-04-15 09:53:29.182 REMOVED[44442:ee0a4] [com.wix.Detox:EarlGreyStatistics] App State -> busy (null)
2020-04-15 09:53:29.182 REMOVED[44442:ee0a4] [com.wix.Detox:EarlGreyStatistics] App State -> busy (null)
2020-04-15 09:53:29.182 REMOVED[44442:ee0a4] [com.wix.Detox:EarlGreyStatistics] App State -> busy (null)
2020-04-15 09:53:29.183 REMOVED[44442:ee0a4] [com.wix.Detox:EarlGreyStatistics] App State -> busy (null)

Device and verbose Detox logs

[x] I have run my tests using the --loglevel trace argument
Provided std out/err trace logs above

Thanks in advance for your help, please let me know if there is anything else I can provide.

Closing. My Detox node module was out of date. Long story.


Fix hot reload

in ReadMe file :

node_env is required in start mode but is missing in example (probably in watch too)
node_env is described as default value as developpement which is a typo

Otherwise, it works great either from webpack generated index or from an external host 👍

@c3dr0x I'm quite happy about that :)
Still have to test that now, creating a dev-server by project is useless, can now use focus-dev-server command.
Also not sure about dev server on external host with subdomain, using DEV_SERVER_SUBDOMAIN. Could you test it tomorrow ?


typo Fixed typos in SHOW-ROUTINE-LOAD.md

Problem summary
Fixed typos in SHOW-ROUTINE-LOAD.md
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[x] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Project lookup fails when using a user for project-location

Hi Peter,
I have a project that belongs to a personal user (me).
I don't have an organization.
I want to link issues in several public repos from my personal user to the user project.
The project is the new beta version of projects.
The project lookup fails because the code is requesting an organization (a GET on 'https://api.github.com/orgs/tijlleenders') where I would expect it to skip that step as I provided a user for project-location:
Run peter-evans/create-or-update-project-card@v2
##[debug]Inputs: {
##[debug]  token: '***',
##[debug]  projectLocation: 'tijlleenders',
##[debug]  projectNumber: 1,
##[debug]  projectName: 'ZinZen_project',
##[debug]  columnName: 'Backlog',
##[debug]  repository: 'tijlleenders/ZinZen',
##[debug]  issueNumber: 198
##[debug]}
##[debug]RequestError [HttpError]: Not Found
##[debug]    at /home/runner/work/_actions/peter-evans/create-or-update-project-card/v2/dist/index.js:4041:23
##[debug]    at processTicksAndRejections (node:internal/process/task_queues:96:5) {
##[debug]  status: 404,
##[debug]  headers: {
##[debug]    'access-control-allow-origin': '*',
##[debug]    'access-control-expose-headers': 'ETag, Link, Location, Retry-After, X-GitHub-OTP, X-RateLimit-Limit, X-RateLimit-Remaining, X-RateLimit-Used, X-RateLimit-Resource, X-RateLimit-Reset, X-OAuth-Scopes, X-Accepted-OAuth-Scopes, X-Poll-Interval, X-GitHub-Media-Type, X-GitHub-SSO, X-GitHub-Request-Id, Deprecation, Sunset',
##[debug]    connection: 'close',
##[debug]    'content-encoding': 'gzip',
##[debug]    'content-security-policy': "default-src 'none'",
##[debug]    'content-type': 'application/json; charset=utf-8',
##[debug]    date: 'Fri, 08 Apr 2022 07:14:08 GMT',
##[debug]    'github-authentication-token-expiration': '2022-04-15 05:05:48 UTC',
##[debug]    'referrer-policy': 'origin-when-cross-origin, strict-origin-when-cross-origin',
##[debug]    server: 'GitHub.com',
##[debug]    'strict-transport-security': 'max-age=3153[60](https://github.com/tijlleenders/ZinZen/runs/5881148031?check_suite_focus=true#step:2:60)00; includeSubdomains; preload',
##[debug]    'transfer-encoding': 'chunked',
##[debug]    vary: 'Accept-Encoding, Accept, X-Requested-With',
##[debug]    'x-accepted-oauth-scopes': 'admin:org, read:org, repo, user, write:org',
##[debug]    'x-content-type-options': 'nosniff',
##[debug]    'x-frame-options': 'deny',
##[debug]    'x-github-media-type': 'github.v3; format=json',
##[debug]    'x-github-request-id': '0601:11C3:4520CE:C2809E:[62](https://github.com/tijlleenders/ZinZen/runs/5881148031?check_suite_focus=true#step:2:62)4FE0C0',
##[debug]    'x-oauth-scopes': 'admin:org, repo',
##[debug]    'x-ratelimit-limit': '5000',
##[debug]    'x-ratelimit-remaining': '4997',
##[debug]    'x-ratelimit-reset': '1[64](https://github.com/tijlleenders/ZinZen/runs/5881148031?check_suite_focus=true#step:2:64)94055[67](https://github.com/tijlleenders/ZinZen/runs/5881148031?check_suite_focus=true#step:2:67)',
##[debug]    'x-ratelimit-resource': 'core',
##[debug]    'x-ratelimit-used': '3',
##[debug]    'x-xss-protection': '0'
##[debug]  },
##[debug]  request: {
##[debug]    method: 'GET',
##[debug]    url: 'https://api.github.com/orgs/tijlleenders',
##[debug]    headers: {
##[debug]      accept: 'application/vnd.github.v3+json',
##[debug]      'user-agent': 'octokit-core.js/3.4.0 Node.js/16.13.0 (linux; x64)',
##[debug]      authorization: 'token [REDACTED]'
##[debug]    },
##[debug]    request: { agent: [Agent], hook: [Function: bound bound register] }
##[debug]  },
##[debug]  documentation_url: 'https://docs.github.com/rest/reference/orgs#get-an-organization'
##[debug]}
##[debug]Projects list: []
##[debug]Projects: []
##[debug]Project: undefined
##[debug]'No project matching the supplied inputs found.'
##[debug]Node Action run completed with exit code 1
##[debug]Finishing: Create or Update Project Card

Workflow file input:
on:
  issues:
    types: [opened]
jobs:
  createCard:
    runs-on: ubuntu-latest
    steps:
      - name: Create or Update Project Card
        uses: peter-evans/create-or-update-project-card@v2
        with:
          token: ${{ secrets.ADD_ISSUE_TO_PROJECT_PAT2 }}
          project-location: ${{ github.actor }}
          project-number: 1
          project-name: ZinZen_project
          repository: ${{ github.repository }}
          issue-number: ${{ github.event.issue.number }}
          column-name: Backlog

Some progress:
Using a BETA project seems to be problematic.
I created a normal project.
Only the normal project shows up in the GET result on 'https://api.github.com/users/tijlleenders/projects'.

Conclusion (maybe include in the README?):
Don't use this action for a BETA project belonging to a user. *
*As long as GitHub API does not show it in the GET result on 'https://api.github.com/users/INSERT_USER_HERE/projects' even though the project is set to visibility public.

I have the same problem but with a org owned beta project 🤔
only the normal projects are shown in the GET results


drivers: dp: swdp_bitbang: Update SWD clock calculation

This patch updates the SWD clock calculation to the latest behavior of DAPLink.
I'm doing this patch because I had trouble getting the frequencies right otherwise.
You can find the reference implementation here:
https://github.com/ARMmbed/DAPLink/blob/main/source/daplink/cmsis-dap/DAP.c

@fabiobaltieri @henrikbrixandersen @mniestroj @tmon-nordic @jfischer-no Could you have a look?

@maxd-nordic check the twister errors
In file included from /__w/zephyr/zephyr/drivers/dp/swdp_bitbang.c:24:
/__w/zephyr/zephyr/drivers/dp/swdp_ll_pin.h:9:10: fatal error: hal/nrf_gpio.h: No such file or directory

@jfischer-no I regrouped the changes and added back the line before the return. Could you please have another look?


Cyclical dependency issue

There is a cyclical dependcy when compiling the daemon with ignite in this module.

This issue still exsits and was only being caught in a dot visualizer.
if err := depinject.InjectDebug(
		depinject.FileVisualizer("/home/evan/work/cosmos-daemon/output.dot"),
		depinject.Configs(
			app.AppConfig(),
			depinject.Supply(
				log.NewNopLogger(),
			),
			depinject.Provide(
				ProvideClientContext,
				ProvideKeyring,
			),
		),
		&txConfigOpts,
		&autoCliOpts,
		&moduleBasicManager,
		&clientCtx,
	); err != nil {
		panic(err)
	}

 Panic: cyclic dependency: github.com/unigrid-project/cosmos-ugdmint/x/ugdmint/module.ProvideModule -> github.com/unigrid-project/cosmos-ugdmint/x/ugdmint/module.ProvideModule

goroutine 1 [running]:
github.com/unigrid-project/pax/cmd/paxd/cmd.NewRootCmd()
        /home/evan/work/cosmos-daemon/cmd/paxd/cmd/root.go:64 +0xbf2
main.main()
        /home/evan/work/cosmos-daemon/cmd/paxd/main.go:14 +0x1f
: exit status 2

Just pushed a branch with the proper BeginBlock added.
https://github.com/unigrid-project/cosmos-ugdmint/tree/eg_fixupdatemerge


[Bug] Local Activities fail to deserialize at gRPC if return type is error

Describe the bug
I'm getting error in temporal-web while trying to view workflow that contains local activities without return parameters.
To Reproduce
Steps to reproduce the behavior:

I took an example from the Go getting started guide and changed it to use Local Activities.

After I run this sample, I get “13 INTERNAL: Failed to parse server response” while viewing the workflow inside of temporal-web

I also get this error inside of temporal-web docker container

Expected behavior
I can view such workflows without getting an error
Desktop

OS: Windows
Browser: Chrome
Temporal Web Version: 1.5.2 (Docker)
Temporal Version: 1.6.0 (Docker)

Additional details

I also noticed that as soon as I add at least an empty return result to the activities, it starts to open without any error

2. I have started a discussion in community

for quick fix: Addressing this in sdk-go
additional fix to prevent similar cases in future: Filing a bug to @grpc/grpc-js


Steam engine clearing items

Thanks for reporting an issue to Teckle, an Elytra project!
When reporting an issue to an Elytra project, please make sure to include:
The version of the mod you are using 6.340 Full
Your Minecraft version 1.12.2
Your Forge version <IP_ADDRESS>5
When assembling my airships my engines clears their inventories, not just of fuel but of the iron bucket as well. If the steam engines worked like a Vanilla furnace with fuel that would be great, even if it's a separate mod addon

has this issue been solved yet? im having the same issue.

has this issue been solved yet? im having the same issue.

Not to my knowledge

Not to my knowledge


Update: Make the visualization toggle a full component

Partials aren't recommended for use anymore so we should make this a component instead. It looks and behaves the same.

@jkusa gave the verbal approval


Typos and missing links in page : Big Picture

Hi,
In https://sap.github.io/btp-solution-diagrams/docs/solution_diagr_intro/big_picture/
Under the TAM image, "Technical an detailed description of architectures" is missing a "d".
The starter kit can be obtained here. - missing link.
You can access the starter kit here. - missing link.
Thanks
Abdelkader

Hi @Kaderde , I just fixed the mentioned typo and missing links. Thx for letting us know...
best,
Fabian

Hi Fabian,
appreciate your quick response,
Now it looks good :)
Best,
Abdelkader


Add metrics for Codespaces usage

Based on the discovery in #8 .
We want to know how many Codespaces there are in the OpenSAFELY GitHub organisation.
This ticket is to:

Add some code into the Metrics repo to poll the GitHub API every hour.
Add a very basic graph in Grafana to display this data (i.e number of Codespaces over time)

If we can easily record additional information about the Codespaces, like Owner/Repo/State, that might be worth considering too.
The API call needed is something like:
gh api \
  -H "Accept: application/vnd.github+json" \
  -H "X-GitHub-Api-Version: 2022-11-28" \
  /orgs/opensafely/codespaces

I'd like to read my interpretation of the issue back to you, @lucyb. I think we want to count the number of active Codespaces in the opensafely organization, every hour. I think we want to use this information to answer the question: "Are Codespaces being used within the opensafely organization?" Consequently, we'd be scanning a timeseries to determine whether the count was zero or whether the count was greater than zero. That is, how much greater than zero doesn't matter.
We don't want to know, for example, when each Codespace was created, suspended, and deleted, and hence know how long each Codespace was active. We don't want to be able, for example, to associate each Codespace with a repo and a range of commits.

The token needs the admin:org scope to use the endpoint in the API call. However, there's an example response in the docs.

@lucyb and I had a chat about this issue last week. We agreed that for each Codespace, Metrics should record:

the user (owner.login)
the repo (repository.name)
when it was created (created_at)
when it was last used (last_used_at)

Metrics should record these data on the current daily schedule. We appreciate that doing so will mean that Metrics will miss data for codespaces that are created and deleted within a day.
With these data, we will derive the number of users that are developing their study code in Codespaces, over time. We hope this number is non-zero (someone is using a Codespace 🤞🏻) and is similar to the rate at which new studies are approved, albeit with a lag. For example, if a new study is approved every week for four weeks, then we hope that the number of users that are developing their study code in Codespaces will (eventually) increase to four. Knowing the user and repo will help us in the observation stage of the initiative: We will know who to ask about what, when we want to know about the experience of developing study code in Codespaces.
It would be useful to derive the distribution of time deltas between when a Codespace was created and when it was last used, as the distribution could help us calibrate our usage policy. For example, if the peak of the distribution was consistently low, then we could infer an ephemeral pattern of use. The current maximum retention period of 14 days would appropriate. However, if the peak of the distribution approached 14 days, then we should reevaluate the current maximum retention period, or at least our communication of it, to prevent users from loosing their work.
It would be useful to derive the distribution of time deltas between when a repo was created and when the associated Codespace was last used. We think this distribution will have positive skew -- that is, a large number of small deltas -- as this would demonstrate that new study code is being edited in Codespaces. However, we're very interested in repos to the right of the distribution, as these would demonstrate that old study code is being edited in Codespaces. These studies may be larger, more complex, and depend on older versions of our tools, and may help us address any challenges associated with developing study code in Codespaces sooner rather than later.

Just to note that one of our pilot users is using the template for a repo in their own  github account not the opensafely org so will be missing from these stats.
Until the service is fully opened back up we might find more instances of researchers trying to get a head start on projects that are still in the approvals process.


Feat Request: Path Bookmarks

Please describe the problem you're trying to solve
hello, I would like to see a way to track personal bookmarks for paths similar to ranger.
I see that there are a few 'built in' ones accessible by using gd, gh, and gt to take you to standard locations.
Describe the solution you'd like
I would like to see this expanded to include personal 'bookmarks' without the requirement of modifying the keymaps file.
I would like to see a keymap that can add the current dir as a book mark and a way to remove 'bookmarks'

https://github.com/DreamMaoMao/bookmarks-persistence.yazi
are you mean this?
also there are other choice in here:

closing as already exists


Dataset Details - additional metadata request

An internal tester would want to filter datasets by age group (e.g. "adult" vs "adolescent").
As well, they would like dataset file format and size to be provided in under Details. But these requests may be solved by providing data previews https://github.com/NIAID-Data-Ecosystem/nde-portal/issues/70

File size is stored now, but it's not populated in many of the sources.  File format is similarly sparsely covered, though we could extract that based off the file extension.


High Priority: Set API electionId to 5000

Due to an issue with the civic info api, we need to set the electionId to 5000 instead of leaving it blank as is the current standard.  After the election, we will need to remove this Id.

@lbirdpew I think it makes the most sense to set 5000 as the default id and still allow individual embeds to override that if they choose, for example to pull from a testing election. Let me know if you disagree.

@alexkonrad That works for me.

Election ID is now set to 5000 by default.


Revert "Revert "Capture CoreCLR ServiceHub dependencies in VSIX" (#39…

Reverts https://github.com/dotnet/aspnetcore-tooling/pull/3972, the original PR of which was originally reverted due to RPS issues that seem to have been figured out. (Validation insertion: https://devdiv.visualstudio.com/DevDiv/_git/VS/pullrequest/340366?_a=overview)
cc @genlu

Judging by the NuGet errors we're still having problems with that service?

@NTaylorMullen here's the RPS regression, which seems to be unrelated to this change after all
https://devdiv.visualstudio.com/DevDiv/_git/VS/pullrequest/339011

Looks like the NuGet failure was flaky, I re-ran the failing task and it's passing. Merging in


Docker Container Grow

Ciao,
I have created a container based on your image one year ago. Yesterday the container take 16 GByte on my Docker server.
I deleted and recreated container and after this it take 50 MByte. Today, after few days, it take 190 MByte.
Are you aware of this issue and can you helpme to fix?
Many thanks in advance.
Marco

I'm closing this issue due to inactivity. I will repoen it if it's necessary.


Add X2Y2 marketplace

@tp-x2y2 @liutaurasjanuskas @cryptopunks2 @blackalpacaa @shermiexyz Please review this PR

This marketplace is already integrated by DappRadar team: https://dappradar.com/nft/marketplaces/protocol/ethereum
Also, this code will not work as it doesn't have no Javascript code.


fix shorthand for bc/borderColor

I think bc stands for "borderColor', right?
It stopped working for me after updated takeout today, maybe after<EMAIL_ADDRESS>

You'd want @shorthands/v2 which has the updated values.

@natew do you know why it stopped working after updating to<EMAIL_ADDRESS>

It was always bc => backgroundColor before, it never should have been bc => borderColor until v2. Not sure what could have happened there


Documentation Improvements

When working on the upgrade to the latest Unirest I noticed a few small things that would be helpful if they were documented in the user guide. Some of these were talked about in issues, but not added to the guide.

Missing mapError
Missing mapBody
GenericType - while the example shows it being used ( https://kong.github.io/unirest-java/#responses ) it is missing javadoc and a description within the guide.
Metrics section:

Spelling : Context inforation
Grammar: In it’s simplest form it might like this --> In it’s simplest form an example may be:

Excellent upgrade, the client code is now much simpler! Thanks!

should be better. the config in the old system was all over the place, so I made it clear that everything is now in config()


Dragon does not support xi / xmi files

Dragon does not support xi / xmi files, at least from what I have read in the code, those file types aren't mentioned anywhere.

neither does logos

Seems like it does?
https://iphonedev.wiki/index.php/Logos#File_Extensions_for_Logos


Files not actually being deleted

Describe the bug
I have noticed that when deleting a file it does remove the json file but leaves the .data file behind which allows me to still view the file by adding it's extension even though everything should be deleted
To Reproduce
Steps to reproduce the behavior:

Go to Files
Select Files
Click "Delete Selected Files"

Expected behavior
The files should be fully deleted (the json and the data file)
Screenshots

Environment:

Browser: Firefox
Convos Version: 7.06
Docker Container

That was a silly oversight 😞
I can't seem to reproduce that you can actually access the files by using the extension though, but either way the deletion is fixed in "main" now.


Bedrock Cross-Region support

Description
The latest versions of Claude and Llama models now support Cross-Region inferences, which require adding the region prefix to the model identifier. This fix enables the use of this feature by adapting our model handling to include the regional prefix.
Fixes # (issue)
n/a
New Package?
Did I fill in the tool.llamahub section in the pyproject.toml and provide a detailed README.md for my new integration or package?

[ ] Yes
[ x] No

Version Bump?
Did I bump the version in the pyproject.toml file of the package I am updating? (Except for the llama-index-core package)

[ x] Yes
[ ] No

Type of Change

[ x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Your pull-request will likely not be merged unless it is covered by some form of impactful unit testing.

[ ] I added new unit tests to cover this change
[ x] I believe this change is already covered by existing unit tests

Suggested Checklist:

[x ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added Google Colab support for the newly added notebooks.
[x ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I ran make format; make lint to appease the lint gods

Should we also update bedrock converse?

I don't know what is bedrock converse, I checked the code in my project and now I can use the Claude 3.5 V2 without problem

@logan-markewich  can U approve this MR?
For next week I will make a code refactoring for simplify the library maintenance.

@csotomon bedrock converse is amazons preferred API. We have class exactly for it (BedrockConverse) -- it's much easier to use/maintain imo.
Anyways, can add it there myself

@csotomon bedrock converse is amazons preferred API. We have class exactly for it (BedrockConverse) -- it's much easier to use/maintain imo.
Anyways, can add it there myself

Perfect, I can help with  the BedrockConverse package and I will do it this weekend, but can U accept this pull request to update this package, I am using it in my projects.


[#165851677] Warn about cflinuxfs2 on org, space and app pages

What
Adds highly-visible warnings to our admin panel when tenant apps are still on cflinuxfs2.
Right now this hardcodes the GUID of cflinuxfs2 in our prod and prod-lon environments. This will be reverted once the migration to cflinuxfs3 is complete. But it may be easier to test this if we stop hardcoding the GUIDs… and that would help solve another issue: right now the application pages say nothing about the stack in use, and so aren't useful for tenants telling if things are on cflinuxfs3 yet.
The visuals are based upon https://design-system.service.gov.uk/components/warning-text/ and https://design-system.service.gov.uk/components/error-summary/.
Why
Tenants with lots of orgs, spaces and apps but limited CF knowledge have found it difficult to identify and coordinate what needed upgrading. This PR would turn paas-admin into a real-time checklist of what still needs upgrading.
How to review

~Review https://github.com/alphagov/paas-tech-docs/pull/212, wait for it to go live, and check that is accessible at https://docs.cloud.service.gov.uk/guidance.html#upgrading-from-cflinuxfs2;~ done;
Code review (recognising this is only going to be in the codebase short-term.)

Who can review
Not @46bit
Simulated screenshots
These are done manually but convey the idea:

🐝 Do not merge! The performing the upgrade yourself links go nowhere yet! 🐝 If you want to be lovely, write a public Google Doc or something they can link to…

The docs this links to are now live: https://docs.cloud.service.gov.uk/guidance.html#upgrading-from-cflinuxfs2
Ready for final review

Looks good to me - will raise a separate PR to update the stub API so it doesn't break following the new API calls.


Is there any way to render overlay on the pathTraced Scene?

Hi Previous Version it can draw overlay over the pathTraced Scene like code below
(ex. area light example that use TransformControls in past)
renderer.autoClear = false;
quad.render(renderer);
renderer.render(overlayScene, camera);
renderer.autoClear = true;

but somehow it can't render overlay
rasterized Scene it can render

but when it turns pathTraced Scene overlay cube disappear

is there something i missed?
thanks to read this issue

Currently if you'd like to render after the fact with depth you'll have to render to depth first and then draw the opaque objects on top of that. The pathtraced view has never provided a depth buffer state after rendering.

(ex. area light example that use TransformControls in past)

These controls never interacted with the depth buffer as far as I remember. Can you provide a small working example of what you were able to do in previous examples of the project in a previous verison?

I want do same in the 0.0.20 version of three-gpu-pathtracer  /areaLight example TransformControls
Gizmos drawn on the view
i think it's not related depth buffer I know it's draw over rendered view without clear

i open pr for example sharing
https://github.com/gkjohnson/three-gpu-pathtracer/pull/640

The transform controls don't check the depth buffer as far as I remember so the depth state wouldn't have ever really mattered while your box is test depth. But if you clear depth in your PR the objects will render just fine:
renderer.autoClear = false;
renderer.clearDepth();
renderer.render(testMesh, activeCamera);
renderer.autoClear = true;

We could change the path tracer full screen quad writing to disable writing and checking depth but it won't necessarily make the state of the depth buffer predictable. Initially it may have the depth state of the scene. But once the path tracer is rendering it will be empty:
this._quad = new FullScreenQuad( new ClampedInterpolationMaterial( {
	depthTest: false,
	depthWrite: false,

	// ...
} ) );

If you want to always use the depth state of the scene for rendering gizmos you can render the depth of the scene after the pathtracer contents are copied to the canvas you can do so like so:

renderer.autoClear = false;

// render depth of the scene
renderer.clearDepth();
scene.overrideMaterial = depthMat;
renderer.render( scene, activeCamera );
scene.overrideMaterial = null;

// render the gizmos / real time interactive objects
renderer.render( testMesh, activeCamera );

renderer.autoClear = true;

If you'd like to make a PR for adjusting the path tracer quad or a small example showing how to render objects on top of the path traced scene that would be appreciated!

Thanks for answers
I think make a example is better than adjusting quad if it don't causes a bug
I'll make PR for small example until this week


Game support

Hello, I would like to know if you're going to support the new Game class?

@maasencioh I'm not really developing this myself anymore, but I will accept pull-requests and contributions, even collaborators or new maintainers.
Please feel free to make pull-requests


Codespaces Pre-cached Image List

There does not appear to be any definitive list of which images are pre-cached in Github Codespaces so as to save a download. The universal images readme used to have a note that the latest version was precached.
This info should be added to the README to the file, or, if keeping this agnostic, link to a GitHub article or otherwise that lists what images are pre-cached in codespaces.

Uploading ironSource Exchange_2024-03-03_19_50_00_2024-03-04_19_50_00_by_au.csv…

Uploading ironSource Exchange_2024-03-03_19_50_00_2024-03-04_19_50_00_by_au.csv…

@Mohammedalduhamshi uh what?

Hi 👋
Thanks for starting this conversation. Looping in @craiglpeters (PM for GitHub Codespaces), should we create such a doc for Codespaces and link it to the corresponding READMEs?

Perhaps it would be sensible to list the images that are precached for Codespaces here? https://github.com/github/codespaces-host-images/blob/main/README.md cc: @osortega

@osortega @craiglpeters any update on this or is there a way to discover it within the codespace itself with docker-from-docker or the like?


Analyzing SG 147

Is there anything different I need to do to analyze spacegroup 147, which has broken time reversal symmetry? My code looks something like:
has_tr = false
D = 3
symeigsd, lgd = read_symdata(calcname; sgnum=sgnum, D=D, dir="output/")
# --- fix singular photonic symmetry content at Γ, ω=0 ---
lgs = littlegroups(sgnum, Val(D))
fixup_gamma_symmetry!(symeigsd, lgs["Γ"])
lgirsd = pick_lgirreps(lgd)
brs = bandreps(sgnum, timereversal=has_tr)
# --- analyze connectivity and topology of symmetry data ---
summaries = analyze_symmetry_data(symeigsd, lgirsd, brs)

but I get the following error:
ERROR: Could not find irrep label A₂⁺A₃⁺
Stacktrace:
 [1] error(s::String)
   @ Base ./error.jl:33
 [2] find_permutation(lgirsd::Dict{String, Vector{LGIrrep{3}}}, klabs::Vector{String}, irlabs::Vector{String})
   @ MPBUtils ~/.julia/dev/MPBUtils/src/read_utils.jl:297
 [3] find_permutation
   @ ~/.julia/dev/MPBUtils/src/read_utils.jl:282 [inlined]
 [4] _default_permutation
   @ ~/.julia/dev/MPBUtils/src/read_utils.jl:308 [inlined]
 [5] analyze_symmetry_data(symeigsd::Dict{String, Vector{Vector{ComplexF64}}}, lgirsd::Dict{String, Vector{LGIrrep{3}}}, brs::BandRepSet; multiplicities_kwargs::Base.Iterators.Pairs{Union{}, Union{}, Tuple{}, NamedTuple{(), Tuple{}}})
   @ MPBUtils ~/.julia/dev/MPBUtils/src/read_utils.jl:89
 [6] analyze_symmetry_data(symeigsd::Dict{String, Vector{Vector{ComplexF64}}}, lgirsd::Dict{String, Vector{LGIrrep{3}}}, brs::BandRepSet)
   @ MPBUtils ~/.julia/dev/MPBUtils/src/read_utils.jl:83
 [7] checkhinge2(sgnum::Int64, calcname::String; topoindex::Vector{Int64})
   @ Main /mnt/c/Users/Sam/Documents/MIT/TopPhC/photonic-topo-sym_mpb-ctl/process_symeigs.jl:258
 [8] checkhinge2(sgnum::Int64, calcname::String)
   @ Main /mnt/c/Users/Sam/Documents/MIT/TopPhC/photonic-topo-sym_mpb-ctl/process_symeigs.jl:234
 [9] top-level scope
   @ REPL[8]:1

I think there's two issues:

pick_lgirreps should be given a timereversal=has_tr keyword argument; that's why you're seeing the "doubled-up" irrep A₂⁺A₃⁺ (which cannot exist without time-reversal). So pick_lgirreps(lgd) should become pick_lgirreps(lgd; timereversal=has_tr).
We're missing some documentation for pick_lgirreps: the main function it has it to return lgirreps in the particular sorting of lgd's operations (and implicitly, it also converts the operations in the returned LGIrreps to a primitive setting if the kwarg isprimitive is true (the default)).
The line lgs = littlegroups(sgnum, Val(D)) is subtly wrong, because it returns the little group operations in a conventional setting. For it to be passed to fixup_gamma_symmetry! it should be in the primitive setting (since that's what the symeigsd values were calculated for). In fact, because the symeigsd values were calculated for a particular operator sorting, which is the operator sorting of lgd, it'd be better to just pass lgd["Γ"] to fixup_gamma_symmetry!.

In summary, something like the following would be better:
timereversal = false
D = 3
symeigsd, lgd = read_symdata(calcname; sgnum=sgnum, D=D, dir="output/")
# --- fix singular photonic symmetry content at Γ, ω=0 ---
fixup_gamma_symmetry!(symeigsd, lgd["Γ"])
lgirsd = pick_lgirreps(lgd; timereversall)
brs = bandreps(sgnum, timereversal)
# --- analyze connectivity and topology of symmetry data ---
summaries = analyze_symmetry_data(symeigsd, lgirsd, brs)

If you see a way to improve the current README.md to make this implicitly obvious/avoid this pitfall, it'd be great with a PR.

Btw., if you would: can you confirm that the new methods (i.e. analyze_symmetry_data) and the old methods give the same answers for your SG147 calculations?
I haven't done any explicit comparisons myself.
... we really should add some tests to this package and set up CI...

Thanks for the help! It runs now, but it gives different answer versus my old method.
So the old code is this:
"""Check symmetry eigenvalues to see if the bands allow for a hinge state"""
function checkhinge(sgnum, calcname; topoindex=[0, 6])
    # --- setup info ---
    has_tr = false

    # --- hilbert/ebr bases ---
    sb, brs = compatibility_basis(sgnum, 3; timereversal=has_tr)
    B = matrix(brs, true)
    F = smith(B)

    # --- load and process data ---
    bandirsd, lgirsd = extract_individual_multiplicities(
                            calcname,
                            timereversal=has_tr,
                            dir = "output/",
                            atol=2e-2,
                            D=3)
    length(lgirsd) ≠ length(sb.klabs) && error("missing k-point data")

    # extract the _potentially_ separable symmetry vectors `ns` and their band-ranges `bands`
    bands = nothing
    ns = nothing
    try
        bands, ns = extract_candidate_symmetryvectors(bandirsd, lgirsd, brs)
    catch e
        if e == ErrorException("found no isolable band candidates")
            @warn "found no isolable band candidates"
            return nothing
        else
            throw(e)
        end
    end

    # --- find which band combinations are in {BS} and then test their topology ---
    band′ = 0:0
    n′ = similar(first(ns))
    is_bs = true # at every new iteration, `is_bs` effectively means `was_prev_iter_bs`
    for (band, n) in zip(bands, ns)
        band′, n′, is_bs

        band′ = is_bs ? band : (minimum(band′):maximum(band))
        n′    = is_bs ? n : n′ + n

        # test if `n′` is in {BS} or not
        is_bs = if first(band′) == 1                        # singular bands
            is_transverse_bandstruct(n′, sb, lgirsd["Γ"], F)
        else                                                # regular bands
            isbandstruct(n′, F)
        end
        if !is_bs
            #println("   ", band′, " not a band structure")
            continue
        end

        # calculate topological index of `bands′`
        ν,_ = if first(band′) == 1
            indicators_singular(n′, sb, lgirsd["Γ"], B)
        else
            indicators(n′, F)
        end
        println((band′, ν))
        if ν == topoindex
            # We have a winner - possibly; although maybe we should also impose a check
            # on `band′``
            return band′
        end
    end

    return nothing
end

and it prints
(1:6, [0, 6])
and returns 1:6
But the new code
function checkhinge2(sgnum, calcname; topoindex=[0, 6])
    timereversal = false
    D = 3
    symeigsd, lgd = read_symdata(calcname; sgnum=sgnum, D=D, dir="output/")
    # --- fix singular photonic symmetry content at Γ, ω=0 ---
    fixup_gamma_symmetry!(symeigsd, lgd["Γ"])
    lgirsd = pick_lgirreps(lgd; timereversal)
    brs = bandreps(sgnum; timereversal=timereversal)
    # --- analyze connectivity and topology of symmetry data ---
    summaries = analyze_symmetry_data(symeigsd, lgirsd, brs)
    return summaries
end

and the summaries variable contains:
julia> summaries[1]
7-band BandSummary:
 bands:      1:7
 n:          2A₂⁺+2A₂⁻+2A₃⁺+A₃⁻, -Γ₁⁺+Γ₁⁻+2Γ₂⁺+2Γ₂⁻+Γ₃⁺+2Γ₃⁻, 3H₁+2H₂+2H₃, 3K₁+2K₂+2K₃, 3L₁⁺+4L₁⁻, 3M₁⁺+4M₁⁻
 topology:   nontrivial
 indicators: (1,8) ∈ Z₂×Z₁₂

julia> summaries[2]
1-band BandSummary:
 bands:      8:8
 n:          A₂⁺, Γ₂⁻, H₃, K₃, L₁⁻, M₁⁺
 topology:   nontrivial
 indicators: (1,4) ∈ Z₂×Z₁₂

Hmmm. Can you upload/paste the associated data files i need to reproduce this? Definitely want to make sure the old/new approaches agree.

Okay, this took me too long, but I figured the issue; the keyword atol was not being set explicitly in your "new" variations, and so it fell back to the default value (1e-2). In your "old" variation, you explicitly set atol = 2e-2.
Somehow, this difference is enough to make resolving the A irrep at the 6th and 7th band troublesome:

For atol = 1e-2, the irreps at the 6th and 7th band cannot be individually resolved individually at the desired tolerance. Instead, it resolves to A₁⁻+A₃⁺ over bands 6 and 7 in combination.
For atol = 2e-2, the irreps now can be resolved individually, resolving to A₃⁺ for band 6 and to A₁⁻ for band 7.

The reason the atol = 1e-2 case doesn't resolve is just that the accuracy of the symmetry eigenvalue calculation is a bit low. Specifically, the translation from symmetry eigenvalues to irreps gives the answer in floating point weights. It is this answer that is checked against atol. For the data you sent me, the answer for band 6 is 1.915e-5A₁⁺+0.014683763455326574A₁⁻+2.07e-5A₂⁺+0.0001876A₂⁻+0.984A₃⁺+0.000182A₃⁻. Note that the coefficient of A₃⁺ differs by more than 1e-2 from 1; hence the issue.
Long story short - the discrepancy is due to a different choice of atol and a relatively low resolution calculation of the symmetry eigenvalues. So there shouldn't be anything fundamentally wrong here.

For good measure, I get the following summaries at atol = 2-e2 (seems unchanged if I change the value higher):
julia> summaries
3-element Vector{BandSummary}:
 6-band (nontrivial): [A₂⁺+2A₂⁻+2A₃⁺+A₃⁻, -Γ₁⁺+Γ₁⁻+Γ₂⁺+2Γ₂⁻+Γ₃⁺+2Γ₃⁻, 2H₁+2H₂+2H₃, 2K₁+2K₂+2K₃, 3L₁⁺+3L₁⁻, 3M₁⁺+3M₁⁻]
 1-band (nontrivial): [A₂⁺, Γ₂⁺, H₁, K₁, L₁⁻, M₁⁻]
 1-band (nontrivial): [A₂⁺, Γ₂⁻, H₃, K₃, L₁⁻, M₁⁺]

julia> getproperty.(summaries, :indicators)
3-element Vector{Vector{Int64}}:
 [0, 6]
 [1, 2]
 [1, 4]

I pushed https://github.com/thchr/MPBUtils.jl/commit/9fafd40753e9cfa1281e2e326f62a2923d48324f to increase the default atol to 2e-2 and also extend some of the associated doc-strings to hopefully better clarify this.

Ohh interesting, thanks so much for looking into this! Do you think atol=2e-2 should generally work moving forward, or do we need to consider tuning it further to avoid missing possible cases? Or going back to higher resolution calculations?The numerical discrepancy of ~0.016 for A₃⁺ seems rather large.


Update mime to v2.2.0

A bunch of people depend on the mime files capability, though. Do you know of a way to mitigate this?

I'm trying to dig up some relevant content and figure out who I have to at

The most recent time we punted this https://github.com/jfhbrook/node-ecstatic/issues/212 and I think it links to most of the relevant issues.
I don't actually know how popular the mime files format is, or what it would take to replace it. Maybe we can find some people involved in the mime module we're using to comment on why they removed it?
/cc @dotnetCarpenter @mk-pmb might be relevant to your interests? no obligation to comment etc

It should also be noted that a bunch of the tests are failing :point_down:

Yeah, I just wanted to get the ball rolling on this again. I can look into fixing up the PR if there is interest.

A bunch of people depend on the mime files capability, though. Do you know of a way to mitigate this?

It’s a breaking change for sure. Bumping the major version (per semver) would prevent those depending on this functionality from suffering any breakage.

It prevents immediate breakage but also doesn't supply a migration path, which is still pretty unfortunate.

I have an unstable branch where my migration strategy is to provide a hook for users to supply their own MIME lookup function, individual for each ecstatic instance. IIRC the old mime package had the problem of keeping a global database for itself, so all modules customizeing "their" MIME would also customize all other modules that use it.
Bonus features I'd like to see soon[TM]:

Have a per-ecstatic-instance default MIME type option: This string is used if the MIME function returned something false-y.
In case the MIME function returns something that looks like a then-able, defer serving until it results are in, with reasonable timeout. (late false-y result still means default.)

Thenable's a good idea. Otherwise I guess we'd expect the v2 API, include a basic wrapper for the v1 API and drop cli support?
That could be a reasonable compromise.

we'd expect the v2 API

In which way "expect"? We'd use the MIME v2 API for the default lookup of course.
If users want to customize that lookup in any way, they can provide a function that looks up a MIME type by filename. If we're generous we can supply additional context about the FTP session as a 2nd argument. If they want to use the MIME module under the hood, its API is their problem.

Sorry for the mixum. I meant the ecstatic instance, there's no FTP here. :-)

Even better compromise: If the user MIME function yields a false-y result, that means ecstatic shall figure out on its own, using its default strategy (currently: mime v2).
This way, a user function for some few specials could be as easy as
function (filename) {
  return ({
    txt: 'text/comma-separated-values',
    ico: 'image/bitmap',
  })[filename.match(/\.(\w+)$|/)[1]];
}

oh I gotcha, we'd do mineLookupFunction: ((fname) => mime2.getType(fname)) or some such as a default and let people set that themselves. I like it!
I think as far as accepting thenables, I think that can be done in a separate PR.
@mathiasbynens if this proposal makes sense to you, and I'm not missing something, and you want to implement this rq (it shouldn't be too bad, the options parsing is a bit involved but easy to copy), I'd merge it.

As the default for mineLookupFunction I'd use either null (means ecstatic shall not even try to use the custom func) or a function that always returns a false-y value (e.g. (() => {})). If we're going to expose our default lookup mechanism, we should so so via an explicit API, not a config default.

without a default it would break, no? There's no behavior difference here.

Yes, behavior is the same, it's just ninja protection. Avoiding to expose a useful MIME handler is just help people not be overly "clever" by depending on an accidential API feature.

Is the key blocker here the regression of the ability to load a custom .types file (i.e. the .load() system being removed from mime 2.x)?
The mime-db project (from where mime gets its JSON type files) contains some code for parsing (via regex) and writing the JSON format that mime-db and mime use:
https://github.com/jshttp/mime-db/blob/master/scripts/fetch-apache.js
There are two conceivable approaches (so far :wink:):

add this sort of code and continue to handle .types file references (converting the them into the JSON mime needs at runtime)
ship a script which converts .types to these .json files and then passing those .json file references in via the command line

My preference would be for option 1 as the command line "signature" and expectations would be the same, but I can understand the motivations behind option 2.
Thoughts?

There's also

Lose the built in cli capability of parsing .types files, but add the option of supplying a custom mime lookup function which may or may not leverage mime@1

which was suggested by mk-pmb, who iirc is the most vocal person around this sort of functionality, so it works for me.

This would be good to support WASM, as it still doesn't serve in Firefox with this server as far as I can tell.

Closing in favor of #240


CrudService and Controllers update method should use Map<String,String>

This will needed to avoid creating more DTOs for the update methods.

After some thought, this is not necessary. It would just make things more complicated, like validation etc.

Actually this will needed...


Neither Lint nor Ace Linters plugins recognise Javascript private class fields

The Lint and Ace Linters plugins show an error when javascript private class fields are used. For example, according to the plugins, Line 2 and 5 in the below code are erroneous.
class Foo {
  #bar;

  constructor () {
    this.#bar = 12;
  }

}

Report it to plugin repo


Associate project for every organisation fetched

:nut_and_bolt: Description: What code changed, and why?
When a organisation is migrated from the chef server to automate, an IAM project needs to be linked with it.
So added the call to project create service to create the project with the name of serverId_orgID, every time the organisation is stored in the database the project is created with the name having all the policies as editors, viewer, owner.
:chains: Related Resources
https://github.com/chef/automate/issues/6380
:+1: Definition of Done
Project is created and stored in database at the time of migration
:athletic_shoe: How to Build and Test the Change
Steps to build:
rebuild components/infra-proxy-service
rebuild components/automate-gateway

Steps to test:

Ensure Chef Infra Server is running. If it is not start it with start_chef_server
Add the chef-server test environment credentials and extract the SERVER IDs

Request
This is a get request.
curl -sSkH "api-token: $(get_admin_token)" 'https://a2-dev.test/api/v0/infra/servers/SERVER_ID/infraserverorgs'
Response
{
    "orgs": [
        {
            "id": "test",
            "name": "test",
            "admin_user": "",
            "credential_id": "",
            "server_id": "serverId",
            "projects": ["serverID_test"]
        }
    ]
}

:white_check_mark: Checklist
All PRs must tick these:

[x] I have read the CONTRIBUTING document.
[x] All commits signed-off for the Developer Certification of Origin.

With occasional exceptions, all PRs from Progress employees must tick these:

[x] Is the code clear? (complicated code or lots of comments--subdivide and use well-named methods, meaningful variable names, etc.)
[x] Consistency checked? (user notifications, user prompts, visual patterns, code patterns, variable names)
[x] Repeated code blocks eliminated? (adapt and reuse existing components, blocks, functions, etc.)
[x] Spelling, grammar, typos checked? (at a minimum use make spell in any component directory)
[x] Code well-formatted? (indents, line breaks, etc. improve rather than hinder readability)

All PRs from Progress employees should tick these if appropriate:

[ ] Tests added/updated? (all new code needs new tests)
[ ] Docs added/updated? (all customer-facing changes)

Please add a note next to any checkbox above if you are NOT ticking it.
:camera: Screenshots, if applicable

a


2019-05-16: Remove duplicate lines of a file preserving their order in Linux

Feel free to write any comments you have for the post Remove duplicate lines of a file preserving their order in Linux

One of the best references is the original book on "new awk" by the language authors. The original paperback was quite expensive, but a PDF is now freely available.
https://archive.org/download/pdfy-MgN0H1joIoDVoIC7/The_AWK_Programming_Language.pdf

@orachas Thank you! I will add it to the post :+1:

That's a nice way to remove duplicated lines! I've used hashes to count occurrences in scripts and other programs, so it feels actually quite natural to use - I didn't know that you could set that up with awk so easily.
From my intuition, I'd suppose that using the awk approach might run into memory issues with very large files as it will keep a lot of content from the file in memory, right?
The cat | sort | sort | cut approach seems like something that might have the same problem. sort probably needs to take the whole file into consideration before being able to sort it. So that might suffer from an even bigger memory consumption than the awk solution...

@Narigo Thank you! :smiley:

From my intuition, I'd suppose that using the awk approach might run into memory issues with very large files as it will keep a lot of content from the file in memory, right?

Yes, you are correct.

The cat | sort | sort | cut approach seems like something that might have the same problem.

I am not familiar with sort's internals but it seems that it does use intermediate temporary files. Check this comment thread on the HN post: https://news.ycombinator.com/item?id=20038605 for more details.

Wow!.. Excellent mastery of awk. I raise my hat to you sir! :)

In FreeBSD, which uses onetrueawk, i had to change the command to this, for it to work:
awk '! visited[$0]++' your_file > deduplicated_file
note the blank after the "!"

@mjakob-gh Thank you for the note, I will add this note in the post asap!

This was really helpful to me, thanks!!

I can't test it right now but I believe that that cut -f2- will not preserve consecutive spaces at the same line. Can someone try it please?
Anyway, excellent posts on bash you have in your blog. Will start to follow it asap.

Great, straightforward explanation of the awk command! Thanks @iridakos!

Mirrors are the un-dead giveaway, you know.
Happy Halloween
Vektor

Hi again iridakos
The main reason I am here is to thank you for the deduplication script.
I found it on " https://www.linux.org/threads/remove-duplicate-lines-from-files-keeping-the-original-order-one-liner-explained.23541/ " .
I have used it several time after doing a search where the items were all jumbled up.
Thank you for the cool stuff.
Vektor


Making Posts accessable on every page

I may be wrong, but Ghost only has the complete list of posts on the main page. By making it accessible everywhere it allows you to make great content pages and landing pages and have the root of your blog on a different page. Right now the solution for this is to grab the rss feed and parse it with javascript which is annoying and probably not necessary.
This is also great for having complete glossaries on each blog post like so: http://vidrack.com/blog/

Hi collinglass,
are you talking about having a list of the blog posts accessible on all pages (e.g. to display "top posts" in the side bar)?
I think that's a good idea, but as Ghost doesn't have that feature either, it's something that will have a place further down the list of features.