Get-AuthenticodeSignature returns the wrong signer certificate

Steps to reproduce
Get-AuthenticodeSignature c:\Windows\System32\ApplyTrustOffline.exe

    Directory: C:\Windows\System32

SignerCertificate                         Status                    StatusMessage             Path
-----------------                         ------                    -------------             ----
84EC67B9AC9D7789BAB500503A7862173F432ADB  Valid                     Signature verified.       ApplyTrustOffline.exe

Expected behavior
Should return 458D803A5CF470DD3F01A475214938D97A5051E8 as evident by running sigcheck. This can also be verified by right-clicking the c:\Windows\System32\ApplyTrustOffline.exe, select 'Properties' and check the "Digital Signatures" tab.
sigcheck64.exe -i c:\Windows\System32\ApplyTrustOffline.exe

Sigcheck v2.70 - File version and signature viewer
Copyright (C) 2004-2018 Mark Russinovich
Sysinternals - www.sysinternals.com

c:\windows\system32\ApplyTrustOffline.exe:
        Verified:       Signed
        Signing date:   8:38 AM 11/1/2018
        Signing date:   8:38 AM 11/1/2018
        Catalog:        c:\windows\system32\ApplyTrustOffline.exe
        Signers:
           Microsoft Windows Publisher
                Cert Status:    Valid
                Valid Usage:    Protected Process Light Signing, NT5 Crypto, Code Signing
                Cert Issuer:    Microsoft Windows Production PCA 2011
                Serial Number:  33 00 00 01 A9 0F 2D 80 C9 A9 29 38 7C 00 00 00 00 01 A9
                Thumbprint:     458D803A5CF470DD3F01A475214938D97A5051E8
                Algorithm:      sha256RSA
                Valid from:     7:57 PM 6/6/2018
                Valid to:       7:57 PM 5/29/2019
           Microsoft Windows Production PCA 2011
                Cert Status:    Valid
                Valid Usage:    All
                Cert Issuer:    Microsoft Root Certificate Authority 2010
                Serial Number:  61 07 76 56 00 00 00 00 00 08
                Thumbprint:     580A6F4CC4E4B669B9EBDC1B2B3E087B80D0678D
                Algorithm:      sha256RSA
                Valid from:     7:41 PM 10/19/2011
                Valid to:       7:51 PM 10/19/2026
           Microsoft Root Certificate Authority 2010
                Cert Status:    Valid
                Valid Usage:    All
                Cert Issuer:    Microsoft Root Certificate Authority 2010
                Serial Number:  28 CC 3A 25 BF BA 44 AC 44 9A 9B 58 6B 43 39 AA
                Thumbprint:     3B1EFD3A66EA28B16697394703A72CA340A05BD5
                Algorithm:      sha256RSA
                Valid from:     10:57 PM 6/23/2010
                Valid to:       11:04 PM 6/23/2035
        Counter Signers:
           Microsoft Time-Stamp Service
                Cert Status:    Valid
                Valid Usage:    Timestamp Signing
                Cert Issuer:    Microsoft Time-Stamp PCA 2010
                Serial Number:  33 00 00 00 D1 98 CF E4 3C 99 25 64 84 00 00 00 00 00 D1
                Thumbprint:     4A5CCCD24EAFE96B11027A86975D51B976B654F1
                Algorithm:      sha256RSA
                Valid from:     9:26 PM 8/23/2018
                Valid to:       9:26 PM 11/23/2019
           Microsoft Time-Stamp PCA 2010
                Cert Status:    Valid
                Valid Usage:    All
                Cert Issuer:    Microsoft Root Certificate Authority 2010
                Serial Number:  61 09 81 2A 00 00 00 00 00 02
                Thumbprint:     2AA752FE64C49ABE82913C463529CF10FF2F04EE
                Algorithm:      sha256RSA
                Valid from:     10:36 PM 7/1/2010
                Valid to:       10:46 PM 7/1/2025
           Microsoft Root Certificate Authority 2010
                Cert Status:    Valid
                Valid Usage:    All
                Cert Issuer:    Microsoft Root Certificate Authority 2010
                Serial Number:  28 CC 3A 25 BF BA 44 AC 44 9A 9B 58 6B 43 39 AA
                Thumbprint:     3B1EFD3A66EA28B16697394703A72CA340A05BD5
                Algorithm:      sha256RSA
                Valid from:     10:57 PM 6/23/2010
                Valid to:       11:04 PM 6/23/2035
        Company:        Microsoft Corporation
        Description:    "ApplyTrustOffline.PROGRAM"
        Product:        Microsoft« Windows« Operating System
        Prod version:   10.0.17134.407
        File version:   10.0.17134.407 (WinBuild.160101.0800)
        MachineType:    64-bit

Note the first certificate in the chain has the thumbprint 458D803A5CF470DD3F01A475214938D97A5051E8.
Environment data

$PSVersionTable

Name                           Value
----                           -----
PSVersion                      6.2.0-preview.2
PSEdition                      Core
GitCommitId                    6.2.0-preview.2
OS                             Microsoft Windows 10.0.17134
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

The issue seems to be here: https://github.com/PowerShell/PowerShell/blob/a11810bf3328a400347b17c14eec968e7ebda241/src/System.Management.Automation/security/Authenticode.cs#L295
The WTGetSignatureInfo method is completely undocumented, but from running the code it seems to return a different certificate than the one the file is actually signed with.

@TravisEz13 Sigcheck and Windows properties verify the signature using a catalog as well. It seems that calling WTGetSignatureInfo (used in Powershell - undocumented) instead of CryptCATAdminAcquireContext2 (used everywhere else and documented) produces different results.
This can potentially lead to a security vulnerability when Powershell is used to verify files. Is this really intended behavior?
I looked for the signer certificate based on the thumbprint provided by WTGetSignatureInfo in the certificate store (both user and machine) and there is no certificate with the thumbprint 84EC67B9AC9D7789BAB500503A7862173F432ADB and it does not even produce a single result on Google. This is unexpected as Windows seems to have explicit trust on a certificate that is neither in the trust store or documented elsewhere.

I verified that Get-AuthenticodeSignature c:\Windows\System32\ApplyTrustOffline.exe is returning the correct signature. The file is catalog signed and catalog signature take precedence over embedded signatures in signing built-in Windows files.
(Get-AuthenticodeSignature c:\Windows\System32\ApplyTrustOffline.exe).SignatureType will give you where the cmdlet is getting the signature from.
Feel free to file a feature request to add a switch to force the cmdlet to use the embedded signature, but the default behavior needs to match Windows Authenticode behavior.

@TravisEz13
Then this behavior shall be described for the Get-AuthenticodeSignature cmdlet.
https://docs.microsoft.com/en-us/powershell/module/microsoft.powershell.security/get-authenticodesignature?view=powershell-7.1

Exactly, that's what the docs describe.  Thanks for clarifying that the docs already doc what you want documented.

Exactly, that's what the docs describe. Thanks for clarifying that the docs already doc what you want documented.

I think he wants the catalog behavior described, which is not in the documentation.

@Genbox This is standard Windows Authenticode behavior and implied.

@Genbox This is standard Windows Authenticode behavior and implied.
He's asking to say that it doesn't only look at the embedded signature, but Windows allows multiple ways to Authenticode sign.
Anyone can file a documentation issue.

How?

I created a PR for the docs here:  https://github.com/MicrosoftDocs/PowerShell-Docs/pull/7271

Hi here ist the workaround:
`$certificate = New-Object -TypeName System.Security.Cryptography.X509Certificates.X509Certificate
$certificate = [System.Security.Cryptography.X509Certificates.X509Certificate]::CreateFromSignedFile("c:\Windows\System32\ApplyTrustOffline.exe")
$certificate2 = New-Object -TypeName System.Security.Cryptography.X509Certificates.X509Certificate2 -ArgumentList $certificate`
I but it still would be nice to have an additional option for Get-AuthenticodeSignature cmdlet telling that embedded signature shall be checked.

@Denis1983 You could ask for a feature request to only get the embedded signature to Get-AuthenticodeSignature.  Perhaps -EmbeddedOnly switch would do this.  File an new feature request if you want this.


show Hz unit in info repr

error is gamma map again.
merge if happy.
basically you have Hz unit displayed on info print.

yeah looks good to me. Merged. Thanks @agramfort !


remove escapes in gdscript test output

Minor annoyance when working on https://github.com/godotengine/godot/pull/60609
Before:
  logged: "/home/nathan/workspace/public/godot/modules/gdscript/tests/scripts/runtime/features/await_without_coroutine.gd"
          "GDTEST_OK
>> WARNING
>> Line: 4
>> REDUNDANT_AWAIT
>> \"await\" keyword not needed in this case, because the expression isn't a coroutine nor a signal.
awaited
"

After:
  logged: /home/nathan/workspace/public/godot/modules/gdscript/tests/scripts/runtime/features/await_without_coroutine.gd
          GDTEST_OK
>> WARNING
>> Line: 4
>> REDUNDANT_AWAIT
>> "await" keyword not needed in this case, because the expression isn't a coroutine nor a signal.
awaited

Thanks!


Export ES and JVM Version as labels

Hi @vvanholl ,
As a Feature Request, it would be nice to have  the ES and JVM Version exported as labels.
java -version 2>&1 | grep openjdk
openjdk version "1.8.0_171"

Is that possible? Maybe at this metric here,
es_cluster_status{cluster="es-mon",} 0.0

to something like,
es_cluster_status{cluster="es-mon",version="6.3.1",jvm="1.8.0_171"} 0.0

Thanks!

I am not sure about the metric es_cluster_status but it would be definitely useful to have ES version in a label somewhere.


Fix #261: Remove writing to log file from process

Remove creating out.log when starting Oni - there isn't much interesting that goes in there, anyway.

FYI - @keforbes

Removing the log entirely is definitely the easiest solution. 😄  If the log file never really provides any benefit then this is probably the right course of action.


Create databases, tables, etc. using bal persist push command

Description:

We need to support Ballerina Persistent Layer on top of Ballerina SQL modules and support DB operations easily without writing any SQL statements. As a part of it, the CLI tool will be provided to generate Ballerina client code for the entity records defined in the Ballerina project.
The bal persist push command will create databases, tables, etc.
Ballerina Persist layer design doc: https://docs.google.com/document/d/1hDa9pTZYmT3GbGdUrFZCaOenGIsTc5P9/edit#heading=h.g59emsvn14w0

Proposal: https://github.com/ballerina-platform/ballerina-standard-library/issues/3310


Type inference on schema with passthrough()

Hi,
Vscode infers this code:
const test=z.object({id:z.string()}).passthrough()
type Typetest=z.infer<typeof test>

to be
type Typetest ={
id:string
}

Is this correct? I was thinking about using this pattern to have a converter for "id", so I can transform the id objects before I ship my data (which is a document including id) off to the db driver. I was thinking that Typetest should display "id" and then some..

Not sure what you're asking here. What do you think the type should be?

Well, I'm no Typescript expert so might be wrong. What I am saying is that I read the type as only having "id". Should I rather be reading it as a "contract" of at least having an "id", like an interface? I was thinking more in lines of T &{ id:string} , but T might be to generic here.. Again, no Typescript expert.


Add a Flask example app for the demo

We can create an examples directory in the project root folder.

Just pasting this piece as a test in case we don't have enough time to make the example app.
from protecc.vortex import Vortex  #make sure to change after package name change

vtx = Vortex()

vtx.SECRETS['test'] = 'ab'

@vtx.register_endpoint
def does_something(x):
    return x

does_something(vtx.SECRETS['test'])

I've actually got a test app lying around for this, but will need to fix #19 for it to work. I'll assign this issue to myself.


Created a GitHub actions workflow

Hi @rocketman12435 ,
I have created a GitHub actions bot which will give a personalised comment to anyone who submits a pull request to this repo, if you merge it.  See https://github.com/2010person/yourlostitems.github.io/pull/2 for an example.

Cool!

GITHUB token - https://docs.github.com/en/actions/security-guides/automatic-token-authentication
Pr url is the url of the pull request which it gets

What are the permissions needed for the TOKEN?

Ok will look at this

It should work if somebody submits another pupil request to this repo.


Private cluster creation

This project already creates nodegroups in a private subnets by default
The next step towards a private cluster is to disable public access to the cluster api.
Creating a private cluster requires to run Terraform from a client that has direct access to the VPC where the cluster resides. The reason for this is that Terraform needs access to the cluster api at certain stages in the process. If the Terraform client sits outside of the VPC it needs to contact the Cluster api through a public IP - which a private cluster does not have.
The Terraform client could be run from a VM within the same VPC as the cluster, from a peered VPC or from a VPC that is connected via VPN. Google Cloud Shell is also an option. In this scenario, the Terraform client will most typically run on a GCP VM.
Authentication to Terraform from a Google Cloud Platform VM adds an additional authentication option. We currently support only Service Account json files.
The Best Practice for GCP VMs to associate Service Accounts with the Compute VM. Any Service Account associated with Google Cloud Shell or a dedicated Google Compute VM will be recognized by the Google terraform provider.  There is no need to provide a Service Account json file.
We will enable and document this additional authentication mechanism in a first step.
In a second step we will add a switch to disable public access to the cluster api and document Best Practices.

feature added via https://github.com/sassoftware/viya4-iac-gcp/commit/c540765394082793652834fbadee71d0e473b675


Salt Design Onboarding

Create an entry point with Salt 101/library intro with all the unsful links/resources for consumers

@bhoppers2008 to share the link to the latest Salt 101 deck

Confluence page created. further work will be expanded / planned with knowledge share / community support.


Automated cherry pick of #4848

Cherry pick of #4848 on release-5.20.

#4848: MM-22199 Disable scrolling to channel on switch

/cc  @hmhealey

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Trying to auto merge this PR.

Pull Request successfully merged
SHA: 02032c6b60817beb662f6da0ad40e669225523f8


Fixing NotificationsList on Service Worker

Proposed Changes

Fixes #8006
Added Type Check for Service Worker and Push Manager
Added a try-catch block around the subscription status check logic within intialSubscriptionState function.

@ohcnetwork/care-fe-code-reviewers

Merge Checklist

[ ] Add specs that demonstrate bug / test a new feature.
[ ] Update product documentation.
[ ] Ensure that UI text is kept in I18n files.
[ ] Prep screenshot or demo video for changelog entry, and attach it to issue.
[ ] Request for Peer Reviews
[ ] Completion of QA

LGTM

Hey @bodhish, Since @vishwansh01 hasn’t responded to the requested changes on the PR, can I create a new PR for this issue if he remains inactive?

@vishwansh01 will you be working on this issue?

@bodhish yes I am working

@vishwansh01 fix the linting errors

@vishwansh01 let me know when its up for review.

@coderabbitai full review

closing in favor of https://github.com/ohcnetwork/care_fe/pull/8935


Wrong output order in github actions environment

Although the validator may not be fully complete yet I decided to replace poseidon2 already with poet in the automatic validation pipeline of data_ancient. Here's the first successful run after some configuration issues: https://github.com/poseidon-framework/data_ancient/runs/1403474318?check_suite_focus=true. The runtime should be much shorter now that the cache is filled.
The output should look like on my local system:
POSEIDON.yml file consistency:
50 Poseidon packages seem to be fine
JANNO file consistency:
4778 samples seem to be fine
BIBTEX file consistency:
120 literature references seem to be fine
JANNO-BIBTEX interaction:
The following papers lack BibTeX entries: BraceDiekmannNatureEcologyEvolution2019, WangBiorxiv2020, Yang2020Science, Yu2020Cell
==> Validation passed ✓

Instead it looks like this:
50 Poseidon packages seem to be fine
4778 samples seem to be fine
120 literature references seem to be fine
POSEIDON.yml file consistency:
JANNO file consistency:
BIBTEX file consistency:
JANNO-BIBTEX interaction:
The following papers lack BibTeX entries: BraceDiekmannNatureEcologyEvolution2019, WangBiorxiv2020, Yang2020Science, Yu2020Cell
==> Validation passed ✓

Why that?

 I'd need to look more carefully to understand this here.

Apparently it was the use of hPutStrLn which caused the issue. Some of the messages were returned to stderr, others to stdout. I guess that caused the rearrangement. Now everything goes to stdout and looks fine:
POSEIDON.yml file consistency:
50 Poseidon packages seem to be fine
JANNO file consistency:
4778 samples seem to be fine
BIBTEX file consistency:
120 literature references seem to be fine
JANNO-BIBTEX interaction:
The following papers lack BibTeX entries: BraceDiekmannNatureEcologyEvolution2019, WangBiorxiv2020, Yang2020Science, Yu2020Cell
==> Validation passed ✓

Total runtime with the cache is down to 2m 49s (without active cache 41m 55s), which is acceptable. Maybe we could provide poet/trident as a precompiled executable in the future.


Use style settings plugin for theming

Currently, the settings for this plugin is a bit confusing with what is retained in a theme and what is generally retained in terms of the toggles. It would be good to separate the theming from the plugin by delegating the theming to the style settings plugin. You can export and import themes into this plugin as well. This way, the settings for this plugin would be purely functional (automatic line numbering or not, automatic title or not etc.) and the styling would still be extensible and manageable but in a more common and understandable format.

What is confusing? I disagree. Why would it be a good idea to delegate some options to another plugin? That would mean that you require two plugins instead of one. You csn set everything in the settings tab

No worries! That makes sense. Thank you for considering it.
Personally I was confused about what information in the toggles is retained in a theme. For example, does the automatic icon showing toggle change if you change themes?

I'm always open for improvements, but for me it would not make sense to depend on an additional plugin.
Basically the color settings are just saved. Every other setting e.g. displaying line numbers or not, or the showing icon, are not saved. This has the reason that if you switch between the dark and light themes often, and one has the line numbers enabled and the other doesn't then it would be confusing. Alternative highlight colors are also not saved, from the same reasoning. It would mean that every alternative highlight color should be saved into the settings multiple times(for every theme you save). The toggle options are valid for every theme, that's why they are not saved.


deny user permission to modify distribution group in outlook web app

Hi -I want limit /to restrict users to modify the distribution in the outlook web app group please help me with the power shell command

@sachinoda, any sort of 'allow' or 'deny' in Exchange or Exchange Online always involves role-based access control (RBAC).
Does this topic help?
How to prevent users from creating and managing distribution groups in Office 365

@chrisda Thank you very much for sharing this explanation. @sachinoda Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Add render_model_add_block tag

OK, this works nicely to satisfy #3873, but would be even better if ClassyTags would let me declare the nodelist block "argument" as, optional. Anyone know if this is possible?

@ojii, is it possible to have a single, but optional block/nodelist defined?

Thats odd. The GitHub integration with Travis says this build failed, but if you click through, it shows all passing tests...

Travis, go home, you're drunk! :)

@mkoistinen can you revert the doc change regarding render_model_add? I have a fix for that
Or just leave it and I'll rebase my PR on top of this after the merge

@ojii, is it possible to have a single, but optional block/nodelist defined in an InclusionTag?

No, not without a heavy dose of dark magic. "endblock detection" happens at parse time, not at render time. Argument processing (including the one that would signal if it should scan for an endblock or not) happens at render time.

@yakky - done
@ojii, ok, thanks, this implementation will stand then.

Coverage increased (+0.04%) to 90.94% when pulling b7ab3a1cca448892b774dd5b198adcb87c7e8b69 on mkoistinen:mkoistinen-feature/render_model_add_block into 4a75faec8d1d0df8991b49be18505464850b7e5a on divio:develop.

Coverage increased (+0.04%) to 90.94% when pulling b7ab3a1cca448892b774dd5b198adcb87c7e8b69 on mkoistinen:mkoistinen-feature/render_model_add_block into 4a75faec8d1d0df8991b49be18505464850b7e5a on divio:develop.

Coverage increased (+0.04%) to 90.94% when pulling b7ab3a1cca448892b774dd5b198adcb87c7e8b69 on mkoistinen:mkoistinen-feature/render_model_add_block into 4a75faec8d1d0df8991b49be18505464850b7e5a on divio:develop.


Removing default style override in GitHub Sponsors badge

Overview
The purpose of this issue is to inform the community that we will be removing the default overridden style in the GitHub Sponsors badge. Based on information from our observability solution and searches on GitHub we do not believe this will impact any existing badges, but are posting this for transparency and awareness.
Action Steps
If you are currently using the GitHub Sponsors badge without the style=social query parameter in your badge url and would like to continue utilizing the social style for your GitHub Sponsors badge, then you will need to add the ?style=social query parameter to your badge url.
E.g. https://img.shields.io/github/sponsors/homebrew --> https://img.shields.io/github/sponsors/homebrew?style=social
If you already have the style query parameter then there is no action needed.
Background
Shields.io supports several different style options for our badges which can be controlled via the ?style=... query parameter on the badge urls. More information can be found under the styles section on the shields.io site - https://shields.io/#styles
The default stye is flat, so a badge requested with a url that does not explicitly specify the desired style with the style param will default to the flat style.
For example:
The default flat style can be observed with the npm version badge.
https://img.shields.io/npm/v/badge-maker

but can be changed to a different style with an explicit query param
https://img.shields.io/npm/v/badge-maker?style=social

However, there are a couple of badges that have overridden that default style for that specific badge to use a default other than flat.
This can be seen with the Twitter badges where the style utilized by default is social instead of flat
https://img.shields.io/twitter/follow/shields

but the style can of course still be set/overridden explicitly
https://img.shields.io/twitter/follow/shields?style=flat

We typically avoid badge-specific overrides of the default style, but there are a few such instances within our social category of badges, including the GitHub Sponsors badge, where this is done.
https://img.shields.io/github/sponsors/homebrew

Motivation
The GitHub Sponsors badge was added relatively recently, and was added to our offering in the social category with the default style overridden to be social as well. This was done based on the initial discussion and request for the badge to look similar to the native sponsor button within the GitHub interface, and a general desire for our social style with the new badge.
However, there were some subsequent discussions and questions about the category (funding vs. social) and whether we truly want to override the default style for this badge. While there's still some ongoing discussion about the category, we have determined that we need to remove the default style override for the GitHub Sponsors badge.

Going to go ahead and close this as the default style override has already been removed and change deployed (as such a few of the example badges in the PR descriptions have styles that no longer match their associated text).
Will keep it pinned for a while longer though for awareness

think this has done its job as a pinned issue now


file with .njk extension

Hello,
how can i use snippets in njk files? I often use @eleventy
I yet configured my vscode with this setting:
"emmet.includeLanguages": { 		"njk": "html", 		"nunjucks": "html" 	}

@saiballo
It will have the same html skeletons right?
If that so, i will add support for the njk in next release.

Can you please check in the latest https://github.com/anburocky3/bootstrap5-snippets/releases/tag/v0.4.4
reopen, if problem persist.

yeah!!! works perfectly! thank you!


Add Pushbullet support

Add Pushbullet support.
Assignees:
Labels: feature

Proprietary integrations will not be included in this project. Sorry. You're welcome to post a link to your implementation here for others to find.

My apologies! This was opened by mistake. Meant to be on my fork!

all good.


Update stable Rust to 1.74

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Closes #39.

@conda-forge-admin please rerender

@xhochy do it seems to be failing on a subset of Mac cross-builds, do we need to do anything there?

I hope to fix this in https://github.com/conda-forge/rust-activation-feedstock/pull/42

Fixed by #42


Can I upload multiple docker compose files?

I have my application broken down into multiple docker compose files.  Any way I can upload them and visualize?

Thank you for this feedback.
You can upload multiple sequentially, but the application is designed to display only one Compose instance at once. This is a good edge case for us to consider though. I'll speak internally with the team about implementing this.
What exactly would be best for your use case? Would you want all services to be displayed at once across multiple Compose files. Or maybe the ability switch between which Compose instance your viewing?

Hey Josh, our team is iterating on your project at Codesmith and we are attempting to implement this feature. Have you given it any consideration yet? We have been thinking of making multiple tabs that allow you to quickly switch between opened compose-files. We could also attempt to show them on the same screen if that's something that people want.


currencylayer API and netbeans

Hello guys, am trying to build a currency converter desktop application using currencylayer API
but it wont just work, have used all the example but no result
can someone please post me a simple way of doing this.
thanks Guys

wrong repository I think :)


az network front-door routing-rule update removes caching configuration

Extension name (the extension in question)
front-door
Description of issue (in as much detail as possible)
I am using az network front-door routing-rule update to add a rules engine to a routing rule, using --rules-engine parameter, to an already existing routing-rule and the caching configuration is also updated to null.
What I am expecting is that the routing rule will only be updated with what was passed in parameters, leaving the caching configuration untouched.
az network front-door routing-rule update \
    --front-door-name "${FRONT_DOOR_NAME}" \
    --resource-group "${RESOURCE_GROUP_NAME}" \
    --name ${routing_rule} \
    --rules-engine ${RULES_ENGINE_NAME}

$ az --version
azure-cli                         2.22.1
Extensions:
front-door                        1.0.14

route to service team


Domain activation is Failed - [BRIEF DESCRIPTION]

App Id: dsa40otdropqh
Region: ap-southeast-1
Step: Domain activation
Status: Failed

Note: Do not include information that is sensitive in nature such as your domain name, company etc.

Issue/question
All retry attempts failed at domain activation stage
Error message
One or more aliases specified for the distribution includes an incorrectly configured DNS record that points to another CloudFront distribution. You must update the DNS record to correct the problem.
Note: Be sure to check the message for sensitive information.
Additional information
The DNS tracker showed me all our CNAME records points to the one distribution indicated under action > DNS records. I dont understand why is there a need to update DNS record.

Hi @jmhen, is this still an issue?
If  so, be sure to check the custom domains troubleshooting guide, specifically CNAMEAlreadyExists to see if that helps.


Update AboutUs details for viki0526

Codecov Report

Merging #21 (cea3b37) into master (fdde3e2) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #21   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Continue to review full report at Codecov.

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


Add pre-checks / raise proper quota limits

When submitting a job which fails for some reason we should report this error to the client. For example, if we request more than 10 qubits for simulation which should tell us that we have limited the system to 10, etc.

Requires proper error messange handling.


Remove toolset compiler workaround

It should not be needed with .NET SDK 9 preview 7.

@jaredpar @dotnet/razor-compiler for reviews, thanks


Missing StatusOK 200

I am currently working on a service broker and I noticed that the brokerapi isn't returning StatusOK 200 and the respond() function isn't public either. In other words there isn't a way to properly get StatusOK 200 from the brokerapi.
I currently need this functionality as described by the servicebrokerapi for #provisioning and the #binding when an instance/binding already exists and the requested parameters are identical to the existing instance/binding.
Any help or clarification on this matter would be appreciated.

Hey @mrgunior!
Thanks for writing. We write brokerapi in conjunction with our on-demand service broker, which interacts with CF. Interestingly and for some reason no one currently on the team knows, the on-demand broker doesn't follow spec, and our broker returns an [instance exists error](Use github.com/pivotal-cf/brokerapi/domain) error, which is handled in the broker api in the [error checking](Use github.com/pivotal-cf/brokerapi/domain).
In order to return the StatusOK 200, we thought of a couple of possibilities.
The provisionResponse could be extended to include a new property like InstanceAlreadyExists, and that could be checked after the broker returns. If it's true, return 200.
Or before the broker Provision gets called, there could be a call to the OSBAPI GetInstance endpoint. If the incoming ProvisionDetails match details from the result of GetInstance, return 200 and never ask the broker to provision.
Were you thinking of other options?
Also our team would need time to make the change as we have other things active at the moment. Is this something you'd want to PR?
Thanks
Winna

Get instance won't work, since according to the specs provision should be the one to do these checks. Get instance also doesn't really help, since not all parameters are set by the user from upfront. There needs to be a thorough check between the local and the remote instance and a correct code response should be returned.

Thanks for getting to us @mrgunior. It sounds like you have an approach in mind. Do you have time/capacity to PR the work and tests for this?

Merged this functionality in https://github.com/pivotal-cf/brokerapi/pull/77


RequirementMachine: miscellaneous fixes and improvements

Atoms are now uniqued in a new RewriteContext. For now, there is one RewriteContext per requirement machine, but shortly it will become global.
Also, there is a fix to the completion procedure; we were not considering all overlaps due to a logic error concerning the processing of deleted rules.
Finally, add some statistical counters and a small test case, and fixes for a couple of request evaluator cycles when the requirement machine is enabled (but there are still some validation test failures remaining, so I'm leaving it off by default).

@swift-ci Please test source compatibility

@swift-ci Please test source compatibility

@swift-ci Please test source compatibility

@swift-ci Please test source compatibility

@swift-ci Please test source compatibility


Check Sparkasse

DB

Seems like spam, if this is a genuine issue, reopen it and explain what you are asking in more detail.


Unpacking build cache entries sometimes takes too long

Android Gradle plugin runs a set of performance benchmark, where multiple builds are triggered and task duration is recorded.
For one of the tasks, enabling cacheability of artifact transforms in change https://android.googlesource.com/platform/tools/base/+/de9841d4c32a5456ab99f796037d8aed652e84ea, impacts the performance. More specifically, we run multiple iterations and min value did not change. However, max value did change substantially. Running the build with --debug -Dorg.gradle.caching.debug=true this is output for the iteration that takes too long:
2020-02-28T14:58:37.720+0000 [DEBUG] [org.gradle.caching.internal.origin.OriginMetadataFactory] Origin for task ':santa-tracker:dexBuilderDevelopmentDebug': {executionTime=1824, hostName=0ECC8A84F7A1, operatingSystem=Windows Server 2016, buildInvocationId=2xtaq6kjmfaplchdl4aftbftpi, creationTime=1582901503811, identity=:santa-tracker:dexBuilderDevelopmentDebug, type=org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.TaskExecution, userName=ContainerAdministrator, rootPath=C:\botcode\w\_tmp\cd1479c20f4e4e1a98af745913251845\tmp_gradle_src, gradleVersion=6.2.1}
2020-02-28T14:58:39.970+0000 [DEBUG] [org.gradle.internal.operations.DefaultBuildOperationExecutor] Completing Build operation 'Unpack build cache entry 4cef22464d5fcad6ef4d76c0e1c69825'
2020-02-28T14:58:39.971+0000 [DEBUG] [org.gradle.internal.operations.DefaultBuildOperationExecutor] Build operation 'Unpack build cache entry 4cef22464d5fcad6ef4d76c0e1c69825' completed

I have this and #16101 , making cache very slow, but only in remote machines I don't have access to do more thorough debugging...


[WIP] Port switch-to-configuration script to Python

Motivation for this change
Note that this has yet to be tested. Rather I am just posting this as a placeholder.
This is a port of the switch-to-configuration script from Perl to Python. The motivation for this change is to ease cross-compilation, which is documented to be quite difficult in the case of Perl (see #36675).
One tricky issue that this raises is the lack of sensible dbus bindings for Python. Each of the options has its share of issues:

dbus-python is largely unmaintained and binds the libdbus implementation, use of which is somewhat actively discouraged
pydbus uses gobject-introspection, which sadly appears to be very unfriendly to cross-compilation
QtDBus uses Qt, which is rather heavy-weight for this purpose.

Ideally I would like a small binding for sdbus but, that not being available, I have simply used dbus-python here.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

@Mic92 I think we can add a tiny builder in trivial-builders for building a Python script, that is also being checked with mypy and pylint. We may want to have the option for disabling those checks.

There is also pystemd: https://github.com/facebookincubator/pystemd that wraps around libsystemd,
but I don't know what the story of cross-compiling cython is at the moment. It should be lightweight at least.
cc @flokli

@FRidh I also have some other scripts flying around that can profit from that: https://github.com/NixOS/nixpkgs/blob/master/pkgs/misc/vim-plugins/update.py https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/tools/analysis/radare2/update.py

I also started rewriting setup-etc.pl at some point, but I think I lost the source code for that.

Pystemd would indeed be very nice for this usecase. @flokli and I have been experimenting with it and it seemed to work really well. I'll have a look at Cython's cross-compilation story. Because I'm not sure either.

What about using Jeepney for DBus? It is a pure Python library.

What about using Jeepney for DBus? It is a pure Python library.

I considered it but decided against it for now for concerns of correctness and verbosity.

The current head typechecks with mypy.

@Mic92, this now includes a port of setup-etc as well (although it might be wise to split this into a new PR)

I also think setup-etc should go to a new pull request. switch-to-configuration is already a lot of code to review.

Great work! Hopefully something can be done about gobject-introspection someday...

How does this related to gobject-introspection?

@Mic92 only in that the prefered python library uses it.

Are there lessons to be learned about using a scripting language, which might incline us to using something a bit more ... robust?

@GrahamcOfBorg I think thanks to mypy we have now best of both worlds: We have static type checking, but due its dynamic nature we can still use import pdb; pdb.set_trace() to understand environmental problems that are not covered by type systems. The switch from Perl to Python seems to attract way more code reviews then we used to have otherwise. But apart from that we should probably have not longer bike-shed discussions in this pull request as it disrupts the code review.

from IRC:
17:04 <Mic92> gchristensen: I guess you need to hurry up if you want a rust port :)
17:06 <gchristensen> Mic92: of hydra? :)
17:06 <Mic92> gchristensen: for activation scripts.
17:06 <gchristensen> aw
17:13 <gchristensen> Mic92: just not so convinced on porting from perl -> python, really. porting is dangerous I think, and I'd rather do it as few times as possible.
17:13 <gchristensen> especially when the dbus bindings are apparently  actively discouraged and not great.
17:14 <Mic92> gchristensen: To be honest I don't expect them to break.
17:14 <gchristensen> obviously not ;)
17:15 <Mic92> gchristensen: also if we really want, we can still shell-out busctl instead.
17:15 <Mic92> or systemctl for that matter
17:16 <gchristensen> that does address the dbus concern but not the general danger
17:16 <gchristensen> I would at least like @edolstra to assign someone to review it
17:17 <Mic92> gchristensen: this sounds like FUD too me. The responsibility is still contained. The systemd update that just went through changed way more things and had less reviews.
17:19 <gchristensen> I have been bitten by rewrites. This is the most critical juncture of nixos. I'm not saying no, I'm asking for great care

The current closure has python3 in it:
$ cat foo.nix
with import <nixpkgs> {};
  (nixos {
    fileSystems."/" = {
      device = "/dev/mmcblk1p2";
      fsType = "ext4";
    };
    boot.loader.grub.enable = true;
    boot.loader.grub.devices = ["/dev/sda"];
  }).toplevel
$ nix-build foo.nix
$ nix path-info -rS ./result | grep python
/nix/store/bwfygfcdvis9wd1c1v51xwnwhw1hx0a0-python3-3.6.6                                          96280592

I guess then we won't see any improvements soon. Rust is probably even harder to cross-compile at this point in the current stage. And it does not seem as if the pr author would rewrite it in c++.

Practically we will probably see python3 in most real-world system anyway and rust/go binaries would be even larger in size then larger then scripts.

$ cat nixos.nix
with import <nixpkgs> {};
  (nixos {
    fileSystems."/" = {
      device = "/dev/mmcblk1p2";
      fsType = "ext4";
    };
    boot.loader.grub.enable = true;
    boot.loader.grub.devices = ["/dev/sda"];
  }).toplevel

$ nix-build ./nixos.nix
/nix/store/yyy7hadsskm49nf8cfkx4ccm14ldkjqg-nixos-system-nixos-18.09.783.299814b385d

$ nix path-info -rS ./result | grep python; echo ''

$ PS1=$'\n'"\$ "

$ nix path-info -rS ./result | grep python | wc -l
0

I don't see python in the minimal closure right now. Here is the full path list: https://gist.github.com/grahamc/3c8f5321f52ad947f76e96a4248bdd5a

On latest master 0111c7d8515d229cc19b487bb0c2e62571413e3c
it is there: https://gist.github.com/Mic92/ce491dfca7fb516c686d30c817f94929

Mind running nix why-depends ./result /nix/store/bwfygfcdvis9wd1c1v51xwnwhw1hx0a0-python3-3.6.6  ?

grub -> zfs -> nfs -> python:
/nix/store/kmkahiy24rg0f7vhrp784qll18jynv0n-nixos-system-nixos-19.03.git.0111c7d
╚═══activate: …rapped..WRAPPER_PATH=/nix/store/3pv2x6rz4aq751cpy1xvp2w07z2m8mmy-system-path/bin:/nix/store/3pv2…
    => /nix/store/3pv2x6rz4aq751cpy1xvp2w07z2m8mmy-system-path
    ╚═══bin/grub-bios-setup -> /nix/store/m7mkpgh50qlb4ssyg6xzd4lrnfg6px7h-grub-2.02/bin/grub-bios-setup
        => /nix/store/m7mkpgh50qlb4ssyg6xzd4lrnfg6px7h-grub-2.02
        ╚═══bin/grub-bios-setup: …jc-lvm2-2.02.177/lib:/nix/store/n9z3wxqgmvzj4vmyn7xcdpj72jfk092m-zfs-user-0.7.11-lib/lib:/nix/st…
            => /nix/store/n9z3wxqgmvzj4vmyn7xcdpj72jfk092m-zfs-user-0.7.11-lib
            ╚═══lib/libzfs.so.2.0.0: …id.anongid.-u......../nix/store/8si5yad9jb76qzk520a2kgbv8fzb5pxx-nfs-utils-2.3.3/bin/exportfs...…
                => /nix/store/8si5yad9jb76qzk520a2kgbv8fzb5pxx-nfs-utils-2.3.3
                ╚═══bin/mountstats: …#!/nix/store/bwfygfcdvis9wd1c1v51xwnwhw1hx0a0-python3-3.6.6/bin/python.# -*- …
                    => /nix/store/bwfygfcdvis9wd1c1v51xwnwhw1hx0a0-python3-3.6.6

Hmm... Seems like a bug, since boot.loader.grub.zfsSupport defaults to false, grub probably shouldn't depend upon ZFS.

 with import <nixpkgs> {};
  (nixos {
    fileSystems."/" = {
      device = "/dev/mmcblk1p2";
      fsType = "ext4";
    };
    boot.loader.grub.enable = true;
    boot.loader.grub.zfsSupport = lib.mkForce false;
    boot.loader.grub.devices = ["/dev/sda"];
  }).toplevel

and
NIXPKGS_CONFIG= NIX_PATH=nixpkgs=/home/joerg/git/nixpkgs nix-build foo.nix

evaluate to the same.

Right, makes sense -- the option defaults to false and setting it to false again shouldn't change anything, and probably shouldn't depend on ZFS :)

FWIW, see #48611 regarding nfs-utils.

Alright, I will put this aside then. Thanks to everyone who reviewed!

The only Python script in NixOS is systemd-boot-builder.py, which kind of slipped in under the radar. I once started to rewrite it in Perl, but didn't finish it.

So that one script effectively increases closure sizes by 55 MiB.

@FRidh I just bisected to that very commit. Tough :/

@edolstra - OOC, is the primary concern closure size? Any thoughts on a more acceptable number (e.g. is a 5MB increase okay)?

@edolstra,j while I'm not attached to Python in particular, I really don't see the sense in using Perl over Python. If anything it seems to me that the residual Perl should be moved to Python, not the other way around. My reasons are three-fold:

Python has a sensible static analysis/typechecking story, which Perl (AFAIK) does lacks. I have been pretty impressed by how much mypy can catch.
Perl is, frankly, hell to cross-compile and very little can be done to fix this. The language and its ecosystem were simply developed with very little consideration for cross-compilation.
In 2018 it is much easier to find people to work in Python than Perl

A compiled language (Rust or C++) also would be fine here. Really, just anything but Perl.

@andrew-d No, the main concern is not doing gratuitous rewrites. The motivation (cross-compilation) is a weak one because NixOS does not support cross-compilation.
Also, I find Python a horrible language to maintain (worse than Perl, even). Rewriting NixOps in Python was the worst decision I ever made. Never again.

rust/go binaries would be even larger in size then scripts

A Rust binary would not be bigger than the 50+ MiB that Python adds to the closure size. All the current Perl scripts (switch-to-configuration.pl, setup-etc.pl, install-grub.pl, nixos-generate-config.pl, update-users-groups.pl, command-not-found.pl) could be replaced by a single Rust binary that shouldn't be bigger than a megabyte or two.

No, the main concern is not doing gratuitous rewrites. The motivation (cross-compilation) is a weak one because NixOS does not support cross-compilation.

I'm not sure what you mean by this; Both @Ericson2314 and I have put quite a bit of work into fixing cross-compilation in both Nixpkgs and NixOS. However, at this point I cross-compile NixOS routinely and use it to support a client project.

I give others credit for NixOS in particular :). But yeah as I understand it the only obstical remaining is Perl.
I don't love either but Python > Perl is also by far the more common opinion as others have said, and that should be taken into account.
I don't think Rust buys us much either. I'm a huge believer in static types, but if everything but the dbus part is shelling out, it's still fundamentally stringly typed. Hardening non-weakest links is a rather pointless exercise.
Still, if Rust is the best way for this to move forward, so be it. I haven't looked that much in the nixpkgs some of things, but in principle it should work very well.

Does that actually work at the moment? The shebang paths in switch-to-configuration and other scripts in 64-bit closures won't work on 32-bit systems either.
To be honest I don't think it's a use case that needs to be a hard requirement.

Rust would be soon usable w.r.t to cross compiling: https://github.com/NixOS/nixpkgs/pull/50866

They were in bash before, but where rewritten in perl because they were too slow according to the git history.

They were in bash before, but where rewritten in perl because they were too slow according to the git history.

It would be nice to rewrite stdenv builders from bash to perl.
Not only for speed but also for better portability to Windows

@bgamari Frankly, I see no alternative to perl (besides exotics like Red)

Splitting to bash on POSIX and PowerShell or VBS on Windows is obviously weird - every derication will have two versions
Lua (which could be embedded into Nix) requires GCC, it is tricky to compile it using MSVC.

(https://github.com/NixOS/nix/issues/1320 might be a more relevant issue to continue the topic)

@edolstra

Also, I find Python a horrible language to maintain (worse than Perl, even). Rewriting NixOps in Python was the worst decision I ever made. Never again.

This is unique experience. Can you write-up a separate post on "why"? Or is it more personal, so nothing to share?

This is unique experience. Can you write-up a separate post on "why"? Or is it more personal, so nothing to share?

If I had to guess, breaking changes in the standard library between major/minor releases.
Perl is really stable: you can take 25yo script and it will still run unchanged.

This is unique experience. Can you write-up a separate post on "why"? Or is it more personal, so nothing to share?

If I had to guess, breaking changes in the standard library between major/minor releases.
Perl is really stable: you can take 25yo script and it will still run unchanged.

Besides increasing minimal system closure size, switching to python would increase number of packets required to cross-compile a system closure: Python + some libs + deps would need to be cross-compilable.
I nice idea I think is to rewrite switch-to-configuration in C++ and build as i686 static executable.
Although it might put the readability even beyond perl, it would:

decrease minimal system closure size
allow to nix-rebuild boot; reboot (and nix-infect) i686 installations effectively upgrading them to x64_86. Currently switch-to-configuration in x64_86 closure requires 64-bit perl which does not run on i686, so nix-rebuild boot; reboot works only in one direction (x64_86 -> i686 using 32-bit perl which runs on both). switch-to-configuration in C++ would enable both directions.

Besides increasing minimal system closure size, switching to python would increase number of packets required to cross-compile a system closure: Python + some libs + deps would need to be cross-compilable.
I nice idea I think is to rewrite switch-to-configuration in C++ and build as i686 static executable.
Although it might put the readability even beyond perl, it would:

decrease minimal system closure size
allow to nix-rebuild boot; reboot (and nix-infect) i686 installations effectively upgrading them to x64_86. Currently switch-to-configuration in x64_86 closure requires 64-bit perl which does not run on i686, so nix-rebuild boot; reboot works only in one direction (x64_86 -> i686 using 32-bit perl which runs on both). switch-to-configuration in C++ would enable both directions.

@edolstra

Also, I find Python a horrible language to maintain (worse than Perl, even). Rewriting NixOps in Python was the worst decision I ever made. Never again.

This is unique experience. Can you write-up a separate post on "why"? Or is it more personal, so nothing to share?

That raises questions how the derision on test-driver perl->python rewrite had been passed and why @edolstra did not vetoed it (I agree that as embed language python is terrible)

@edolstra

Also, I find Python a horrible language to maintain (worse than Perl, even). Rewriting NixOps in Python was the worst decision I ever made. Never again.

This is unique experience. Can you write-up a separate post on "why"? Or is it more personal, so nothing to share?

That raises questions how the derision on test-driver perl->python rewrite had been passed and why @edolstra did not vetoed it (I agree that as embed language python is terrible)

@edolstra

Also, I find Python a horrible language to maintain (worse than Perl, even). Rewriting NixOps in Python was the worst decision I ever made. Never again.

This is unique experience. Can you write-up a separate post on "why"? Or is it more personal, so nothing to share?

That raises questions how the decision on test-driver perl->python rewrite had been passed and why @edolstra did not vetoed it (I agree that as embed language python is terrible + having tab-language within Nix string literals with non-trivial indent stripping and escape sequences is awful)

Because not many contributors know perl anymore.

@edolstra

Also, I find Python a horrible language to maintain (worse than Perl, even). Rewriting NixOps in Python was the worst decision I ever made. Never again.

This is unique experience. Can you write-up a separate post on "why"? Or is it more personal, so nothing to share?

That raises questions how the decision on test-driver perl->python rewrite had been passed and why @edolstra did not vetoed it (I agree that as embed language python is terrible + having tab-language within Nix string literals with non-trivial indent stripping and escape sequences is awful)

Because not many contributors know perl anymore.

Also note perl is probably worse when it comes to cross-compilation than python.

Also note perl is probably worse when it comes to cross-compilation than python.

Also note perl is probably worse when it comes to cross-compilation than python.

Yes, we depend on https://arsv.github.io/perl-cross/ which is not very reliable. The project is not very popular and they might stop supporting it at any moment. That's +1 for C++

Also note perl is probably worse when it comes to cross-compilation than python.

Yes, we depend on https://arsv.github.io/perl-cross/ which is not very reliable. The project is not very popular and they might stop supporting it at any moment. That's +1 for C++


unigram_df attribute of Nlpeda module only works if no null values in Dataframe

Is this the only way to do it or is there a possibility to maybe deal with null values and/or replace them, so that the unigram_df attribute still works without an error message?

yes, instead of dropping null rows, we can fill in with some other values, I will update in upcoming versions


Debugging related changes

Trying to fix #1542 (Test flake 17-cilium_policy-id-remove.sh). These changes just add to the debugging, no real fix yet. Still trying to figure out how to reproduce.

Thanks for the quick review!


Add New Connection does nothing

Hi,
sqltools Add New Connection does nothing for me, I can click all day. No error messages, nothing.
I tried reinstalling the extension. Are there any pre-requisites for it to work properly?
How do I figure out what's wrong?

I get the following error when trying to open the "Add New Connection" view:

An entirely empty tab titled "SQLTools Settings" is opened, but nothing else happens.

From Command Palette please run SQLTools Management: Show Output Channel Logs
Copy the content of that output channel and paste it here, or save it to a file and attach the file. Please note that email responses to GitHub will not transmit attachments. Please use the web UI.

Perhaps https://github.com/mtxr/vscode-sqltools/issues/593#issuecomment-1272446735 is a workaround?

@gjsjohnmurray

Hi,
I have the same issue. Here is my about:

And the results from Logs (only shows once):
[2022-10-25 08:18:12.867] [exthost] [info] ExtensionService#_doActivateExtension mtxr.sqltools, startup: true, activationEvent: '*'


Maven issue

When trying to package this using Maven I get the following errors in the DocumentDBSerDe.java:
[ERROR] /tmp/azure-documentdb-hadoop/src/com/microsoft/azure/documentdb/hive/DocumentDBSerDe.java:[16,32] package org.openx.data.jsonserde does not exist
[ERROR] /tmp/azure-documentdb-hadoop/src/com/microsoft/azure/documentdb/hive/DocumentDBSerDe.java:[17,37] package org.openx.data.jsonserde.json does not exist
Is there some reference to the jsonserde project not set correctly or am I doing something wrong?

The issue is that the DocumentDBSerDe.java has a dependency on the org.openx.data.jsonserde. This jsonserde is not published on maven, so you have to download the dependency from here https://github.com/rcongiu/Hive-JSON-Serde/releases. Afterwards, you can add the downloaded dependency to your maven local repository. This should solve your problem.

This is correct. I installed the Hive-JSON-Serde project to my local repository. But afterwards I had to add this to the pom.xml of the azure-documentdb-hadoop project to make it work:

org.openx.data
json-serde
1.3.6-SNAPSHOT

Glad to hear you got it working! If you have any other questions, feel free to post a comment.


upgrade to Cake 4

@SharpeRAD  any chances to get this one merged?
@AdaskoTheBeAsT  do you have a nuget of the fork available for sharing?

Can you fix the build?

hi I will take a look one more time - I had problems with solving build issues

Tks all of you for your time and effort.
From the logs seems the path passed to ReportUnit is not correct:
Results File: C:/projects/cake-powershell/build/results/tests/Cake.Powershell.xml
Passed!  - Failed:     0, Passed:    12, Skipped:     0, Total:    12, Duration: 4 s - Cake.Powershell.Tests.dll (net6.0)
Building report
ReportUnit v1.50.0. Report generator for the test-runner family.
http://reportunit.relevantcodes.com/
Copyright (c) 2015 Anshoo Arora (Relevant Codes)
Developers:  Anshoo Arora, Sandra Greenhalgh
[2024.07.03 00:34:21][Info] The file C:/projects/cake-powershell/build/results/tests\Cake.Powershell.xml contains Unknown test results
Also ReportUnit is no longer maintained since Dec 2018. It is replaced by extentreports-dotnet-cli.
I ll have a look on the fork.

@AdaskoTheBeAsT,   can you try my PR on your fork.
As defined in variables.cake, testResultsDir is a relative path
var buildResultDir = "./build/results";
var testResultsDir = buildResultDir + "/tests";
ReportUnit() takes absolute paths.
https://github.com/reportunit/reportunit/issues/34

@gsgou for me ReportUnit didn't worked - I changed report generation mechanism to use native xunit html report generation by xslt

Even better @AdaskoTheBeAsT . Thanks for your time.

@SharpeRAD @devlead checks pass now, can be merged!

Thank you for the quick merge @SharpeRAD. We ll be waiting for the nuget package to be published.


services: add custom fields support

Opened issues for the corresponding comments in this PR. Closing in favor of custom-fields branch.


Git Hook Config

hello there
please is it possible to add args of '--all-files' to the git hook config
detect-secrets is skipping files with ".txt" files
see here
  - repo: https://github.com/Yelp/detect-secrets
    rev: v1.3.0
    hooks:
      - id: detect-secrets
         args: ['--all-files', '*']

Hi @saintmalik, thank you for reporting this issue to us. We'll take a look at it in the next few days and let you know when we have updates about it.

Hi @saintmalik , I'm unable to reproduce this issue. Please see below logs for details.
Could you please provide detailed steps to reproduce (a minimal MVC would be ideal).

Terminal output with a sample .txt file ( private_key_2.txt ):

~/git/test-repo   master ✚  git status
On branch master
Changes to be committed:
  (use "git restore --staged <file>..."

Secret Type: Private Key
Location:    private_key_2.txt:1

Possible mitigations:
  - For information about putting your secrets in a safer place, please ask in
    #security
  - Mark false positives with an inline `pragma: allowlist secret`
    comment

If a secret has already been committed, visit
https://help.github.com/articles/removing-sensitive-data-from-a-repository

We're going to close this issue as it hasn't received any update in a very long time. Feel free to re-open it if you think it's still relevant.


JSON custom field being returned as a repr() string when using GraphQL

Environment

Nautobot version (Docker tag too if applicable):

v2.0.0b1 (yes, I know I need to upgrade, but this is where I found it)
1.6.1

Python version: Python 3.10.12
Database platform, version: docker.io/bitnami/postgresql:15.3.0-debian-11-r7
Middleware(s): Kubernetes / NGINX Ingress

Steps to Reproduce

Create a custom field test_json

Set type to JSON
Associate with device (though it can be any object)

Restart Nautobot server
Create/edit an object (device) with some JSON data in that custom field. For example use the following.{
  "a-number": 1,
  "a-string": "hello world!",
  "a-null": null,
  "a-list": [5, 6, 7],
  "a-dict": {"foo": "bar"},
  "a-bool1": true,
  "a-bool2": false
}

View the object via the Rest API, and observe the JSON data being returned:{
  "id": "7c5cefba-9bb7-4f72-aead-9402f5558ad5",
  ...,
  "custom_fields": {
    "test_json": {
      "a-number": 1,
      "a-string": "hello world!",
      "a-null": null,
      "a-list": [5, 6, 7],
      "a-dict": {
        "foo": "bar"
      },
      "a-bool1": true,
      "a-bool2": false
    }
  }
}

Use the GraphQL to execute the following query:query {
  devices {
    _custom_field_data
    cf_test_json
  }
}

Expected Behavior
I'd expect the GraphQL to return structured JSON data, such as:
     "cf_test_json": {"a-number": 1, "a-string": "hello world!", "a-null": null, "a-list": [5, 6, 7], "a-dict": {"foo": "bar"}, "a-bool1": true, "a-bool2": false}

Observed Behavior
Observe that the value returned for cf_test_json is python repr()
       "cf_test_json": "{'a-number': 1, 'a-string': 'hello world!', 'a-null': None, 'a-list': [5, 6, 7], 'a-dict': {'foo': 'bar'}, 'a-bool1': True, 'a-bool2': False}"

It's also worth noting that the value shown in _custom_field_data is actual JSON data, which is what I'd expect.
References:

thread in #nautobot on NTC slack
TY to @joewesch for reproducing the issue in 1.6.1

Thanks for the report! Confirmed this is still the behavior in 2.0


Extend --license-header to cover the generated ProxyCmdletDefinition.ps1

Currently, --license-header only applies the specified license header to .cs cmdlets under /generated/cmdlets/. In relation to #814, --license-header should also be extended to also set the provided license-header in ProxyCmdletDefinition.ps1.
Repro Steps

Run autorest --powershell --license-header:MICROSOFT_MIT_NO_VERSION --input-file:{ANY_FILE}  with any input file.
Inspect the generated module.

Only .cs files will have the specified license header.
Scripts under /generated/exports/ will have a hardcoded Apache license header - https://github.com/Azure/autorest.powershell/blob/59e8eb2020d0f70fabd5ef0cc203996b4d818f63/powershell/resources/psruntime/BuildTime/Cmdlets/ExportProxyCmdlet.cs#L86-L99

Environment
AutoRest core: 3.0.6306
AutoRest PowerShell: 2.1.402

Expected behavior
Scripts under /generated/exports/, including ProxyCmdletDefinition.ps1, should have the same license header as .cs files in /generated/cmdlets/ when --license-header:MICROSOFT_MIT_NO_VERSION is specified.

Add --pwsh-license-header to set license header for generated powershell code.
If --pwsh-license-header is not set, --license-header is applied.
And magic words for this setting include:
microsoft_mit_no_version
microsoft_apache_no_version
microsoft_apache_no_codegen
microsoft_mit_small_no_codegen


Sometimes the paste command doesn't work

The paste system doesn't work everytime.
There is nothing wrong with the copy command.
enhancement : do not clear content that will be paste in raw <input type="text"> or  <textarea> elements.

Solved & copy/cut/paste clear content system improved.


#147 Upgrade nextjs to latest version

I upgraded next to the latest version and tested the app afterwards. I didn't notice any issues and according to the docs most changes affect self-hosting, dev builds performance, dx, parallel/intercepted routes so I think we are safe to upgrade.
Looking at the next repo I did notice a couple of issues reported related to 14.1 - https://github.com/vercel/next.js/issues/61002 and https://github.com/vercel/next.js/issues/62727. However, both seem to be resolved in 14.1.4.

Hey @brunomenezes , I tested this one via the docker container and didn't notice any issues. The app works as expected.


ChromeSafariBrowser cannot open if default browser app does not support custom tabs on Android

[x] I have read the Get Started - Installation section
[x] I have read and done the Get Started - Setup Android section
[ ] I have read and done the Get Started - Setup iOS section
[x] I have already searched for the same problem

Environment

Technology
Version

Flutter version
2.10.5

Plugin version
5.4.3+7

Android version
6.0.0+

iOS version
N/A

Xcode version
N/A

Device information:
Description
Only on Android.
ChromeSafariBrowser cannot open if default browser app does not support custom tabs, due to limitation of PackageManager#queryIntentActivities since Android M/N.
note:
issue #391 looks like similar but probably not exactly the same.
Expected behavior:
Open with Chrome
Current behavior:
Nothing happens but output StackTrace.
See Stacktrace/Logcat section.
Steps to reproduce

Install browser app which does not support custom tabs such as Sleipnir
https://play.google.com/store/apps/details?id=jp.co.fenrir.android.sleipnir

Set installed browser as a default browser app by setting app

Try to open URL by ChromeSafariBrowser

Stacktrace/Logcat
[ERROR:flutter/lib/ui/ui_dart_state.cc(209)] Unhandled Exception: PlatformException(ChromeBrowserManager, ChromeCustomTabs is not available!, null, null)
#0      StandardMethodCodec.decodeEnvelope (package:flutter/src/services/message_codecs.dart:607:7)
#1      MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:177:18)
<asynchronous suspension>
#2      ChromeSafariBrowser.open (package:flutter_inappwebview/src/chrome_safari_browser/chrome_safari_browser.dart:112:5)
<asynchronous suspension>
#3      LoginPageState._openExternalBrowser (package:poddog2_app/ui/login/login_page.dart:253:5)
<asynchronous suspension>

Yes ! We need...

Fix merged now

This error seems to happen again after the fix from Android 13 devices... Did someone have the same issue?


zvm_cursor_style errors out every return

General information

konsole 21.08.1
- and, xterm-370
FreeBSD 13.0-RELEASE-p3
zinit
zsh 5.8
zvm 0.8.5

Problem description
zvm_cursor_style:33: failed to compile regex: trailing backslash ()
Reproduction steps

source  attached zshrc
will occur after hitting return (but not before, i.e. not right when terminal loads)

Expected behavior
error shown in prob. desc. occurs immediately after the command line.  There is no newline before it.  The error is itself followed by a newline.  There is no extraneous trailing newline occurring after command output.
Occurs regardless of whether ZVM_CURSOR_STYLE_ENABLED is set to true or false
zprofile.txt
zshrc.txt

The error is in if [[ $old_style =~ '\e\][0-9]+;.+\a' ]]; then in zsh-vi-mode.zsh:3047

I'm hitting this, too 😢

I solved this by simply removing this 3 line if block, currently on line 3075

Hi @klmanion,
Thanks for your reporting, may I know if this issue is still existing in using the latest version?
Thanks & Regards

@jeffreytse I am on freebsd and can verify that the issue still exists with the latest version,

Hi @anupamsr
Currently, I can't reproduce your issue on my local env. Could we have a co-debugging on your env?
Thanks & Regards

Hello! Well, apparently zsh on FreeBSD does not support PCRE by default, as in, the following shall fail:
setopt re_match_pcre

A practical approach to enable PCRE support is to reinstall via the ports tree with the respective flag enabled:
cd /usr/ports/shells/zsh

# enable the "pcre" option...
make config

# reinstall the shell...
make reinstall clean

Thus you can issue the setopt again:
setopt re_match_pcre

With the module in place the error messages is gone.
On macOS (homebrew) and Linux (DNF/Fedora), zsh supports PCRE by default, apparently this issue will only occur on FreeBSD using pre-compiled packages.

zvm_cursor_style:33: failed to compile regex: trailing backslash ()
i got same error on termux. Which line to delete or comment.

@juliekoubova I solved this by simply removing this 3 line if block, currently on line 3075

zvm_cursor_style:33: failed to compile regex: trailing backslash () i got same error on termux. Which line to delete or comment.

@juliekoubova I solved this by simply removing this 3 line if block, currently on line 3075

The three lines of if judgment code from line 3222 to line 3224.
But here comes the question, is it caused by regularity or something else?

@daiaji  thanks @otaviof  I tried compiling zsh from port using libpcre but still faced the same issue.

Hi @anupamsr, I'd love to help settle this issue, but currently I've not reproduced it in my local.

zvm_cursor_style:33: failed to compile regex: trailing backslash () i got same error on termux. Which line to delete or comment.

@juliekoubova I solved this by simply removing this 3 line if block, currently on line 3075

I also got this error in termux. I fixed it by just adding setopt re_match_pcre to .zshrc as @otaviof suggested. I did not have to recompile anything.
@anupamsr As for freebsd, you might try recompiling libpcre with different config options? That's the only relevant difference I can find between the termux version, the archlinux version (which works without the zsh option), and the port version.
For reference,
Archlinux
https://gitlab.archlinux.org/archlinux/packaging/packages/pcre/-/blob/8.45-4/PKGBUILD#L40
Termux-pacman
https://github.com/termux-pacman/termux-packages/blob/master/packages/pcre/build.sh#L13
FreeBSD
https://cgit.freebsd.org/ports/tree/devel/pcre/Makefile?id=56fcffd8655a1cc30676ff357af80be1225b4c34#n23
Hope that helps

I did not have this problem, but when I set -o GLOB_SUBST then I have almost the same error after every <CR>, <Up>, or <Down>: zvm_cursor_style:38: bad pattern: \e[0 q.  I also get zvm_update_repeat_commands:42: bad pattern: ^[[A

Thanks @rosetintedcheeks . I have stopped using freebsd now so I can't be of more help here.


Missing core-js dependencies

core-js dependencies are markes as dev in package.json, however
when I try https://github.com/react-toolbox/react-toolbox-example with react-toolbox 0.12.9
I'll get build error:
ERROR in ./~/react-toolbox/lib/utils/polyfills.js
Module not found: Error: Cannot resolve module 'core-js/fn/array/from' in /Users/finch/dev/mapilary/mapilary-widget-ng/node_modules/react-toolbox/lib/utils
 @ ./~/react-toolbox/lib/utils/polyfills.js 3:0-32

ERROR in ./~/react-toolbox/lib/utils/polyfills.js
Module not found: Error: Cannot resolve module 'core-js/fn/array/iterator' in /Users/finch/dev/mapilary/mapilary-widget-ng/node_modules/react-toolbox/lib/utils
 @ ./~/react-toolbox/lib/utils/polyfills.js 5:0-36

ERROR in ./~/react-toolbox/lib/utils/polyfills.js
Module not found: Error: Cannot resolve module 'core-js/fn/map' in /Users/finch/dev/mapilary/mapilary-widget-ng/node_modules/react-toolbox/lib/utils
 @ ./~/react-toolbox/lib/utils/polyfills.js 7:0-25

ERROR in ./~/react-toolbox/lib/utils/polyfills.js
Module not found: Error: Cannot resolve module 'core-js/fn/string/starts-with' in /Users/finch/dev/mapilary/mapilary-widget-ng/node_modules/react-toolbox/lib/utils
 @ ./~/react-toolbox/lib/utils/polyfills.js 9:0-40

ERROR in ./~/react-toolbox/lib/utils/polyfills.js
Module not found: Error: Cannot resolve module 'core-js/fn/symbol' in /Users/finch/dev/mapilary/mapilary-widget-ng/node_modules/react-toolbox/lib/utils
 @ ./~/react-toolbox/lib/utils/polyfills.js 11:0-28

I had to install core-js to resolve this issue.

Checking it this afternoon.
Thanks for reporting!

Apparently it's working for us. Maybe it's related to your config?
If anybody else has the same problem, please reopen!

@mauron85 Were you installing with a production flag?

I too am seeing this error :crying_cat_face:

Hi @Rillus ! Did you try to update to the latest version and ran npm install?

I seem to have resolved it, but couldn't pin point what fixed it specifically...
I updated my npm and node to the latest versions (I was quite a bit behind).
I also removed everything and started again! Either of those could be the key...
Thanks for the quick reply though, and thanks for some really nice work on this - everything looks and works how you'd hope!

Nice, thanks @Rillus !

$ node --version
v4.2.3
$ npm --version
2.14.7
$ npm list
....
├─┬<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>...
npm ERR! extraneous<EMAIL_ADDRESS>D:\Users\Choupignon\Mes documents\nTorrent\nTorrent\node_modules\eslint-plugin-react
npm ERR! extraneous<EMAIL_ADDRESS>D:\Users\Choupignon\Mes documents\nTorrent\nTorrent\node_modules\react-hot-loader
npm ERR! extraneous<EMAIL_ADDRESS>D:\Users\Choupignon\Mes documents\nTorrent\nTorrent\node_modules\webpack-dev-server
npm ERR! extraneous<EMAIL_ADDRESS>D:\Users\Choupignon\Mes documents\nTorrent\nTorrent\node_modules\babel-preset-stage-0

Getting same error tried update npm, npm update not working. Any idea? Thanks

@vinz243 what error are you referring to? that error you put there is due to globally installed deps

@nathanmarks i'm referring to this error:
ERROR in ./~/react-toolbox/lib/utils/polyfills.js
Module not found: Error: Cannot resolve module 'core-js/fn/array/from' in /Users/finch/dev/mapilary/mapilary-widget-ng/node_modules/react-toolbox/lib/utils
 @ ./~/react-toolbox/lib/utils/polyfills.js 3:0-32

ERROR in ./~/react-toolbox/lib/utils/polyfills.js
Module not found: Error: Cannot resolve module 'core-js/fn/array/iterator' in /Users/finch/dev/mapilary/mapilary-widget-ng/node_modules/react-toolbox/lib/utils
 @ ./~/react-toolbox/lib/utils/polyfills.js 5:0-36

ERROR in ./~/react-toolbox/lib/utils/polyfills.js
Module not found: Error: Cannot resolve module 'core-js/fn/map' in /Users/finch/dev/mapilary/mapilary-widget-ng/node_modules/react-toolbox/lib/utils
 @ ./~/react-toolbox/lib/utils/polyfills.js 7:0-25

ERROR in ./~/react-toolbox/lib/utils/polyfills.js
Module not found: Error: Cannot resolve module 'core-js/fn/string/starts-with' in /Users/finch/dev/mapilary/mapilary-widget-ng/node_modules/react-toolbox/lib/utils
 @ ./~/react-toolbox/lib/utils/polyfills.js 9:0-40

ERROR in ./~/react-toolbox/lib/utils/polyfills.js
Module not found: Error: Cannot resolve module 'core-js/fn/symbol' in /Users/finch/dev/mapilary/mapilary-widget-ng/node_modules/react-toolbox/lib/utils
 @

Is core-js/fn/*/ present in your node modules?

Found \node_modules\react\node_modules\fbjs\node_modules\core-js\fn so i
believe yes :D
Le mar. 8 déc. 2015 à 14:35, Nathan<EMAIL_ADDRESS>a écrit :

Is core-js/fn/*/ present in your node modules?
—
Reply to this email directly or view it on GitHub
https://github.com/react-toolbox/react-toolbox/issues/127#issuecomment-162880875
.

@vinz243 Did you rm -rf node_modules and npm install again?

I just tried but I get those Path Too Long errors
Most annoying thing about npm
Le Mar 8 Déc 2015 18:40, Nathan<EMAIL_ADDRESS>a écrit :

@vinz243 https://github.com/vinz243 Did you rm -rf node_modules and npm
install again?
—
Reply to this email directly or view it on GitHub
https://github.com/react-toolbox/react-toolbox/issues/127#issuecomment-162957033
.

@vinz243 I was getting this error too, and was on the exact same node / npm version as you.  I upgraded npm to 3.5.1 (npm update npm -g), and then rm -rf node_modules + npm install.  Problem gone.

@CacheControl  would want too but can't delete files :'(

@vinz243 Try delete your node_modules folder. Then run npm install core-js, and then do npm install.

Thanks! I copied my files to another folder except node_modules and ran npm install core-js and then npm install it worked!

Same error, trying use the Tab components. I installed react-toolbox in the last 24 hours. Is core-js not listed as a required package?
npm install --save core-js fixed it right away for me.

Had the same problem with missing core-js. I think it should either be in peerDependencies instead of devDependencies (IMO the better option), or the readme should state that npm@3 is required.


fix: package.json & .snyk to reduce vulnerabilities

The following vulnerabilities are fixed with an upgrade:

https://snyk.io/vuln/npm:minimatch:20160620

The following vulnerabilities are fixed with a Snyk patch:

https://snyk.io/vuln/npm:minimatch:20160620

Latest report for therealpiotrp/omnisharp-vscode:package.json:
https://snyk.io/test/github/therealpiotrp/omnisharp-vscode?targetFile=package.json

I will close this for now, since this is not on our current plan.


Crashes when using Playground Component - docz v0.13.7

Bug Report
Crashes when using the Playground component
To Reproduce
Just wrapping my components in
🚨 Whoops, we crashed 🚨
Passed an incorrect argument to a color function, please pass a string representation of a color.
Stack trace
in Styled(div) (created by right)
in right (created by Resizable)
in div (created by Resizer)
in Resizer (created by Resizable)
in span (created by Resizable)
in div (created by Resizable)
in Resizable (created by RenderBase)
in div (created by Context.Consumer)
in Styled(div) (created by RenderBase)
in div (created by LiveProvider)
in LiveProvider (created by RenderBase)
in RenderBase (created by Context.Consumer)
in ThemeConfig (created by Render)
in Render (created by BasePlayground)
in BasePlayground (created by Context.Consumer)
in Component (at text.mdx:19)
in div (created by MDXTag)
in MDXTag (created by Context.Consumer)
in Component (at text.mdx:14)
in MDXContent (created by AsyncComponent)

My mdx code:
---
name: Button
---

import { Playground } from 'docz'
import { Button } from './button'

# Button

## Basic usage

<Playground>
  <Button
    label='Click me'
  />
</Playground>

More Information:
I already see this issue: https://github.com/pedronauck/docz/issues/540
And I already install the docz-theme-default and still crash
EDIT: I did this example app to simulate the problem, take a look:
https://github.com/rodrigowpl/rn-docz-play
Expected behavior
Render without crashing
Environment

OS: OSX 10.14.2
Node/npm version: [e.g. Node 8/npm 6]
react-native 0.57.8
react 16.6.3
styled-components 4.1.3
docz 0.13.7

Same on my Windows, apart of it there missed some theme styles, so menu looks broken

thrown from
node_modules/polished/lib/color/parseToRgb.js


Add link for Willard Goosey's ClearView mode for StrongED

Willard Goosey has written a StrongED mode for ClearView:
https://www.riscosopen.org/forum/forums/11/topics/15378
It'd be nice to link this in the README.

Done!


Not compatible with NodeJS 6

node-gyp rebuild ( part of npm install nodehun ) fails on system with latest versions of NPM and NodeJS.

I am no longer maintaining this project.

sorry to hear that. thanks for the nodehun project!

I've fixed this in my fork here.
This should work:
npm install wulf/nodehun
I'll try to get in touch with @nathanjsweet.

fixed #32


Ostrich process hangs for larger calibrations

Ostrich seems to hang if we ask for larger number of model evaluations. Ex:
Calibrating on 100 model evaluations = 38 seconds.
Calibrating on 1000 model evaluations = 440 seconds.
Calibrating on 10000 model evaluations = Was still incomplete (no error, just running) after 16 hours.
This leads me to believe there is some sort of config limiting the duration of processes maybe? a sort of timeout?

Are you using progress=True ?

Synchronous mode
    The client sends the `Execute` request to the server and waits with open
    server connection, till the process is calculated and final response is
    returned back. This is useful for fast calculations which do not take
    longer then a couple of seconds (`Apache2 httpd server uses 300 seconds <https://httpd.apache.org/docs/2.4/mod/core.html#timeout>`_ as default value for ConnectionTimeout).

Asynchronous mode
    Client sends the `Execute` request with explicit request for asynchronous
    mode. If supported by the process (in PyWPS, we have a configuration for
    that), the server returns back `ProcessAccepted` response immediately with
    URL, where the client can regularly check for *process execution status*.

This should be better documented in our docs.

Hmmm I thought I did in all my notebooks, turns out this one did not. I'll relaunch with a larger size and keep you posted. Thanks!

I just get this error in the notebook when I try to run the process with Progress=True:
The save operation succeeded, but the notebook does not appear to be valid. The validation error was:
Notebook validation failed: {'version_major': 2, 'version_minor': 0, 'model_id': '817877db674542ad8586f18de20711ba'} is not valid under any of the given schemas:
{
"version_major": 2,
"version_minor": 0,
"model_id": "817877db674542ad8586f18de20711ba"
}

After running the code on the Ouranos JupyterLab instance (on pavics.ouranos.ca/jupyter) with progress=True, OSTRICH does the same thing where it does not crash but has been running for over 12 hours and still no response whereas I expected the code to take ~1h20 mins.

Ok. Could you confirm that if you run Raven on your own machine (from the terminal, no python wrapper or wps server), it works.

Hi @huard. I was testing Richard's setup on my machine. For a 4747-day simulation with different budgets for OSTRICH took:
budget 100 iterations: 10s
budget 1000 iterations: 78s
budget 10000 iterations: ~8348s (still running; estimate based on Progress-file; will update when it's done)
I will contact Shawn Matott (developer of Ostrich) if he has an idea why runtime is not linear.

OK, glad to see I'm not going crazy. thanks for the info!

Yeah, I'm sorry about that. I normally don't use such large budgets and never realized. I am guessing that it has something to do with the increased memory allocation of Ostrich to hold all the statistics etc of the previous runs. But let's seee what Shawn says. Just sent out the email with the runtime stats and the example setup. :)

Follow-up: It would seem that the process hanging also affects other birds that demand long run times.

There is a known issue with PyWPS queue management. I'm hoping to make some progress on this front over the next months.
@julemai Any news from Shawn ?

I think this has actually nothing to do with Ostrich or Raven. Didn't we find that it is actually hanging in the WPS?

I think the comment here refers to the Raven that includes DDS internally so we can accelerate the calibrations much faster and avoid this problem altogether.

Ok. James has implemented DDS functionality in Raven.
But:

it only works for a subset of parameters we want to calibrate
it is only faster if ALL data can be read at once (no NetCDF chunks)

It is a longer discussion if we really want to make use of this since all calibration settings would need to be divided into "Is Raven doing the calibration internally?" or "Is Ostrich doing the calibration?"
The runtime of ALL Raven runs and hence also calibration runs can be significantly improved when the input data (forcings) are aggregated from gridded to HRU-aggregates using aggregate-forcings-to-hrus as described here.

Thanks for the update. I suggest we close this issue here, since the PyWPS problem is described elsewhere.

@huard Can you link to the PyWPS issue for posterity please? then we can close this one. Thanks!

https://github.com/geopython/pywps/issues/600


Meetup Suggestion: Example Bitcoin Meetup

Thanks for making a suggestion for a new meetup.
Please note: Only suggest bitcoin-only meetups.
Meetups which promote any coin or token other than Bitcoin are not suitable for listing.
Altcoin's are only mentioned in the context of warnings or lessons learnt for bitcoin.
Required:

Name of Meetup: Rocket City Bitcoin Meetup
Location Of Meetup: Huntsville, AL
At least one of the following:

Meetup Page (or similar): https://meetu.ps/e/KF2G2/Dpqyd/i
Telegram Group (or similar):
Website:

Optional

Name of Organiser(s) & Twitter Handle:
Links showing public support: >>>  example
Tweet your commitment

As the organiser of [Rocket City Bitcoin] Meetup I can confirm that the meetup is bitcoin-only
-The meetup is bitcoin focused.
-'Altcoins' are scams and are not promoted in talks/presentations at our meetups.
#bitcoinonlymeetups @bitcoinonly_

Added Previously


default mean should not be from mean_var

Defining mean from mean_var makes sense only for Gaussian VAE, otherwise, it does not work. I propose to do this for CGaussian with DiagVar, but not for others. Calling  default mean() for UnitVar does not make any sense.

CGaussian with DiagVar and ScalarVar need to call the mapping to get either mean or variance, only UnitVar does not need to call the mapping to get the variance. As UnitVar, as I see it, is the special case here I can change it to this:
mean(p::CGaussian{T,UnitVar}, z::AbstractArray) = p.mapping(z)
variance(p::CGaussian{T,UnitVar}, z::AbstractArray) = fill!(similar(z, xlength(p)), 1)
mean_var(p::CGaussian{T,UnitVar}, z::AbstractArray) = (mean(p, z), variance(p, z))

the variance still needs the input z and the similar to create a CuArray if the model is on the GPU.

we will solve this by moving the UnitVar special case to SharedVarCGaussian. We will also rename like:
CGaussian -> CMeanVarGaussian
SharedVarCGaussian -> CMeanGaussian

closed by b27c26d653d6e31e788ea9ae48ff8f337e1d50f5


Replace hooks by those offered by latex2e 2020/10/01?

See pre-release note and especially texdoc lthooks.

There is \NewReversedHook, but that reverses the order of code added with distinct labels, not within a given label.

Nice catch and thank you very much! This corrected my misunderstanding of reversed hooks.
Quoting relative docs for \AddToHook in texdoc lthooks-doc,

If there already exists code under the ⟨label⟩ then the new ⟨code⟩ is appended to the existing one (even if this is a reversed hook).

Just curious, what is the reason for the prepending (if you happen to know; I know you're not the original maintainer)? I find the behavior counterintuitive. If one adds a sequence of code blocks, my expectation would be that they're executed in the order declared.

I don't know the exact reason.
Once a hook-like thing was provided, the request to specify insertion position, execution order and selective removal arose. (If the request was not loud enough, then the hook is not heavily used.) I think LaTeX's hook management does a good job. The only pity is, it's not released ten years ago.


Mojibake problem: A¯

Hi again,
Back again with some mojibake problems.
Using ftfy 5.7 and python 3 and ftfy.fix_text(x, uncurl_quotes=False).
U+0041  A       [Lu] LATIN CAPITAL LETTER A
U+00AF  ¯       [Sk] MACRON
SHOULD BE:
U+00EF  ï       [Ll] LATIN SMALL LETTER I WITH DIAERESIS

Examples:
LegaA¯a 2 Duel Saga
animatie geA¯nspireerde tekeningen
Is it possible to fix this?
PS:
I had the bright idea to do a full analysis of all my documents for mojibake. After manually shifting through all the data I can say that ftfy fixes most of mojibake problems I have in the documents, but I still have ~45 unique mojibake problems that I think might be solvable. Should I report them all in this style or is another type of reporting style preferable for this many?

I'd be happy to review examples of mojibake found in the wild!
However, to try to filter the reports a bit, ftfy is not going to be able to fix cases like this, where information has been lost that makes it impossible to be certain which character was the original one. And in general, it should be very reluctant to change common ASCII characters like "A".
In this case, the usual windows-1252 mojibake for Legaïa would have been LegaÃ¯a, but the tilde on the A was removed, which removes the ability to identify the specific intended character.


Update documentation

There is not such interface or class Cache in the library. However the example suggests the following:
val coroutines: Cache = CoroutinesCache(cacheParams: CacheParams, scope: CoroutineScope)
This should be changed to
val coroutines: CoroutinesCache = CoroutinesCache(cacheParams: CacheParams, scope: CoroutineScope)
or simply
 val cache = CoroutinesCache(cacheParams: CacheParams, scope: CoroutineScope)

Related to doc:
Provide the data from RxCache
We do not have RxCache here :)

Fixed it in last pull request


CLOUDSTACK-8666: Put host in Alert state only after alert.wait timeout

Instead of putting the host to Alert state immediately, the investigators should be allowed to run for some time based on alert.wait global config.
At the end of this interval if the host state still cannot be determined then put the host in Alert. Also updated some of the log messages.
Refer to the bug for the detailed description.
Since these scenarios are difficult to simulate, haven't written any tests. If anyone has suggestions on some tests please let me know.

LGTM

LGTM

Merging it as 2 LGTMs

wasn't this needing (unit)-tests?

@DaanHoogland Writing an unit test for changes in AgentManagerImpl.java is hard due to the way the method gets invoked. Let me know if you have any suggestion.
Fot the HighAvailabilityManagerImpl.java changes I have written a unit test, will send out the PR for that.

Hi @koushik-das
Perhaps refactoring the code in a way that its design is improved could make easy to add tests for it.

@wilderrodrigues Agree that refactoring the code might help with the tests. But that will be a bigger effort.


[BUG] Unable to remove tags in TagInput

Describe the bug
This occurred when we try to remove the tags added in the TagInput box.

Feature request: TagInput should also support the backspace key.

To Reproduce

Go to TagInput in the storybook.

Type the tag name and Enter. This will add the tag in the input box as expected.

But when we start clicking on the x doesn't remove the tag present in the box.

Expected behavior
On Clicking the x should remove the tag in the input box.
Screenshots

Additional information
No response

Proposed solution:
const removeTags = (indexToRemove: number) => {
    // index should be the second parameter, otherwise filtering won't work
    setTags([...tags.filter((_: string, index: number) => index !== indexToRemove)]);
};


src/rc/rc-status: fix 'started' output (-Wformat)

On 32-bit platforms (e.g. HPPA), GCC warns with:
../src/rc/rc-status.c:137:49: warning: format ‘%lld’ expects argument of type ‘long long int’, but argument 5 has type ‘time_t’ {aka ‘long int’} [-Wformat=]
  137 |                                                 "%02"PRId64":%02"PRId64":%02"PRId64" (%s)",
      |                                                 ^~~~~
  138 |                                                 diff_hours, diff_mins, diff_secs, start_count);
      |                                                                        ~~~~~~~~~
      |                                                                        |
      |                                                                        time_t {aka long int}

We expect the output to be 64-bit but time_t isn't always wide enough, so
let's cast to long long which is guaranteed to be.
Fixes: 25d5de8fd9698fed0fdbbd8949fd4089602b796d
See: https://forums.gentoo.org/viewtopic-t-1146117.html
Signed-off-by: Sam James<EMAIL_ADDRESS>

Shouldn't this be casting to (int64_t), or changing the format to use %lld as well?


Light state turn off  after 10s

After turn on the Light @Home Assistant Light turn off after 10s Refresh-Rate  (H6159 / H6163)
Also the brightness is still 40%.....
https://user-images.githubusercontent.com/76902841/115114460-d48bbf00-9f8f-11eb-8971-50868eed9de1.mp4

Same issue, turn on led strip, led stip turns on, after 5s ha entity turns off (looks like fails to get state), led strip stays on.

Hi, please head over to the support thread here. This is an issue in the Govee API, but we made a workaround possible until they fixed it.
It's described here in detail:
https://community.home-assistant.io/t/govee-integration/228516/454?u=laggat


SLO native splitted to native/table and native/query

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull request type

Please check the type of change your PR introduces:

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

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

Codecov Report
Attention: Patch coverage is 0% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 37.75%. Comparing base (f29bef1) to head (11e22fd).

Files
Patch %
Lines

query/do_options.go
0.00%
2 Missing :warning:

query/do_tx_options.go
0.00%
2 Missing :warning:

query/options.go
0.00%
2 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #1085       +/-   ##
===========================================
- Coverage   66.66%   37.75%   -28.91%
===========================================
  Files         282      281        -1
  Lines       27618    27488      -130
===========================================
- Hits        18412    10379     -8033
- Misses       8340    16703     +8363
+ Partials      866      406      -460

Flag
Coverage Δ

?

go-1.21.x
38.95% <0.00%> (-29.60%)
:arrow_down:

go-1.22.x
37.74% <0.00%> (-28.90%)
:arrow_down:

integration
?

macOS
38.95% <0.00%> (+1.19%)
:arrow_up:

ubuntu
37.74% <0.00%> (-0.03%)
:arrow_down:

unit
37.75% <0.00%> (-0.06%)
:arrow_down:

windows
?

ydb-22.5
?

ydb-23.1
?

ydb-23.2
?

ydb-23.3
?

ydb-24.1
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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


zerofree: install manpage and COPYING file

Motivation for this change
Install COPYING and manpage from Debian/Ubuntu.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

@moredread please resolve the merge conflict.

@moredread please resolve the merge conflict.

@SuperSandro2000 there is no official page for this package. Alternatively we could maybe add a ubuntu-manpages and debian-manpages derivation and link from those. I'm not sure though how large the man pages are that are provided by Debian and Ubuntu and if we want to add a large dependency only for the man page.
Maybe this is a larger issue... When I get around to it I'll write something up und Discourse.
Are you still interested in this PR?

@SuperSandro2000 there is no official page for this package. Alternatively we could maybe add a ubuntu-manpages and debian-manpages derivation and link from those. I'm not sure though how large the man pages are that are provided by Debian and Ubuntu and if we want to add a large dependency only for the man page.
Maybe this is a larger issue... When I get around to it I'll write something up und Discourse.
Are you still interested in this PR?

Are you still interested in this PR?

I am not interested in most PRs personally. If you think this is a good addition go for it.

Are you still interested in this PR?

I am not interested in most PRs personally. If you think this is a good addition go for it.

Alternately, you could try to work with upstream and get it to accept the manpage from Debian. We did that for other packages successfully in the past (example).

Alternately, you could try to work with upstream and get it to accept the manpage from Debian. We did that for other packages successfully in the past (example).

I've resolved the merge conflict.


shortened Plus Code to LatLng not working

i have this position "J8J7+257, Misserghin" that i want to covert to latlng but getting Invalid argument(s): Passed Open Location Code is not a valid code: J8J7+257, Misserghin
my code :
var plusCode = olc.PlusCode('J8J7+257, Misserghin');
var codeArea = plusCode.decode();
print(codeArea);

Did you try to parse olc.PlusCode('J8J7+257');? Misserghin is not part of a valid pluscode. Probably you will need to split your input strings.

Can you please validate the code with another library? I just tried a random site (https://www.dcode.fr/open-location-code) and it also gives me an validation error.

I'm really not sure about this one, but it seems like google maps does something different here, that is not conform with the plus code standard. Even when you enter your code on the official site https://plus.codes/map it will in fact truncate the last digit and show me a place in Greece. Maybe you can try out the official dart code from the official google repo https://github.com/google/open-location-code/ where I also ported the dart code from, and in case there are also problems, raise an issue there.

Okay i will, thank you for your time

@benouarAbdou could you solve the issue?

i didn't try actually , i switched to other problems i was working on on my repositories