Error in command sitaxis

The code script for step 3 has an error:
az identity create -resource-group myResourceGroup --name myUserAssignedIdentity

The parameter resource-group only has one hyphen. Correct adding the missing one.

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

ID: 389c63ed-8360-d5cd-83e0-121f567f9308
Version Independent ID: 8d01970a-af78-85ab-c929-b9ade6934d0a
Content: Configure managed identities with Azure App Configuration
Content Source: articles/azure-app-configuration/overview-managed-identity.md
Service: azure-app-configuration
GitHub Login: @maud-lv
Microsoft Alias: malev

@warnov
Thanks for your feedback! We will investigate and update as appropriate.

@warnov Thanks for reporting this item! I have updated the doc and it will be published soon. We appreciate the feedback!

My pleasure!
On Mon, Apr 29, 2024, 00:39 MayankBargali-MSFT @.***>
wrote:

@warnov https://github.com/warnov Thanks for reporting this item! I
have updated the doc and it will be published soon. We appreciate the
feedback!
—
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/122035#issuecomment-2081923816,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOS62F2MSFIFBY6GVT4BOLY7XMH7AVCNFSM6AAAAABG4IGETWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBRHEZDGOBRGY
.
You are receiving this because you were mentioned.Message ID:
@.***>


Update configuration-page.md

Fixed format of code samples

Docs Build status updates of commit 148ff81:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/tabs/how-to/create-tab-pages/configuration-page.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


[analyzer] Incorrect functionality and documentation of invalid_dependency diagnostic

I went through the pub diagnostics recently but got confused by invalid_dependency.
The current description says:

The analyzer produces this diagnostic when a package under either dependencies or dev_dependencies isn’t a pub, git, or path based dependency.

But it does trigger on git and path dependencies, contradicting the description. Which makes sense to me, since as far as I understand, pub.dev doesn't allow path or git hosted dependencies.
Also it mentions producing the diagnostic on dev_dependencies, but I don't believe it does, and I don't believe it should.
Then it presents the following example using a (non-pub) hosted dependency and indicates it is triggered on this:

name: example
dependencies:
  transmogrify:
    hosted:
      name: transmogrify
      url: http://your-package-server.com
    version: ^1.4.0

However it looks like the diagnostic is not triggered by this. I'm not sure if non-pub.dev hosted sources are allowed, but as far as I can tell, they aren't?
Please let me know if my assumptions are correct, then I'd be happy to update the diagnostic to trigger on the non-pub.dev hosted dependency and update the messaging to match the desired/actual behavior.

Expand for proposed documentation
#### Description

The analyzer produces this diagnostic when a package under `dependencies`
isn't a `pub.dev` based dependency.

To learn more about the different types of dependency sources,
check out [Package dependencies](https://dart.dev/tools/pub/dependencies).

#### Example

The following code produces this diagnostic because the dependency on the
package `transmogrify` isn't a `pub.dev` based dependency:

```yaml
%uri="pubspec.yaml"
name: example
dependencies:
  transmogrify:
    [!hosted!]: http://your-package-server.com
    version: ^1.4.0
```

#### Common fixes

If you want to publish your package to `pub.dev`, then change the
dependency to one that that is published on `pub.dev`.

If you don't want to publish your package to `pub.dev`, then add a
`publish_to: none` entry to mark your package as not intended to
be published:

```yaml
%uri="pubspec.yaml"
name: example
publish_to: none
dependencies:
  transmogrify:
    hosted: http://your-package-server.com
    version: ^1.4.0
```

\cc @jonasfj or @sigurdm Could one of you verify if my understanding of how this diagnostic should work(/should be documented) is correct? I appreciate it!

Yes, I think you are right, and the text is broken.


opt interrupt running parquet/orc readers when scannode is finished

Proposed changes
VScanNode::get_next will check whether the ScanNode has reached limit condition, and send eos to TaskScheduler, and TaskScheduler will try to close ScanNode.
However, ScanNode must wait all running scanners finished, so even if ScanNode has reached limit condition, it can't be closed immediately.
This PR try to interrupt the running readers, and make ScanNode to end as soon as possible.

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

run buildall


Add retry logic on .getReceipt for network response "Unknown"

Problem
As a developer, I need to get the receipt of a transaction to validate the transaction was successful or to get data from the transaction that executed which is returned in the receipt.
If I ask for the receipt and the receipt returns a status of Success then do not retry to get the receipt of the transaction.
If I ask for the receipt and receipt returns "Unknown" retry attempting to get the receipt. Unknown means that the status of returning the receipt or not is unknown https://github.com/hashgraph/hedera-protobufs/blob/main/services/response_code.proto#L147.
Solution

Review the getReceipt logic that is implemented in the SDK
Retry getting the receipt on an "Unknown" status
Test the feature works as expected when "Unknown" is returned

Alternatives
No response

Hi @SimiHunjan, I have checked the codebase for retry logic on .getReceipt when the network response is "Unknown", and it is already present there. Additionally, I have tested it and verified that it works.
Please find here the checks for the status, as well as the execution flow here based on them.


Simplify / revamp search filters for odysee (and lbry.tv)

Most sites have at least some simple search filters that we should add back to Odysee. Channel vs videos would be one, and we can also add something to sort by newest/oldest (available today) and should be able to add a "last day/month/year/etc" param (switch between date/relevance) easily.
Also, unleash the infinite scroll!

#5677 and #5181 are related tickets that it may make sense to do at the same time. Please include these if it's efficient to do so.


PLS doesn't flag error in a relative import if it is found in another root directory

The analyzer incorrectly accepts relative imports in cases where the path is relative to a root but not to the importing file.
Consider the following case:
Directory structure:
src
|->module
|--->init.py
test.py
PYTHONPATH=src
test.py contains the following code:
from .module import Foo

This import should be flagged as invalid because the path ".module" should be relative to the test.py file. However, the code in PathResolverSnapshot.GetImportsFromRelativePath considers all of the root search directories when it performs its search. It therefore finds an import match when it shouldn't.
I think the problem is in PathResolverSnapshot.RootContains. It doesn't do anything to validate that the specified lastEdge starts in the same location as the root. If the relative path happens to match items within that root, the method returns true. If the relative path is zero length, all roots return true.
A proposed fix is to add the following piece of code to PathResolverSnapshot.RootContains before the while loop:
if (root != sourceEdge.Next.Start) {
    rootEdge = default;
    return false;
}

The problem isn't related to the multiple roots. With "python.autoComplete.extraPaths": ["D:\\Work\\Temp\\Issues\\509\\src"] and the following file structure:
├─ package
│  └─ test.py
└─ src
   └─ package
      └─ module
         └─ __init__.py

code from .module import * in test.py is valid.
But, from .module is invalid when test.py is in root.

We probably should have an exception for the working directory, cause it maybe a top directory of the module.


Karpenter Resource Helm Chart to Support SnapshotID

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

What is the outcome that you are trying to reach?
Let the Karpenter Resource helm chart to accept snpashot ID to support developing the feature Enhance pull speed for Large ML container Images with Bottlerocket

Describe the solution you would like

Describe alternatives you have considered

Additional context

Reopen the issue

Fixed in #41


Empty Page

Hi @MahdiKarimipour,
Thanks for using react-dynamic sitemap. I see you deleted the contents of this issue so I am assuming you fixed the issue yourself. If that's the case, could you please re-add all of the details of the ticket, and add a comment about how you fixed it, in case someone else has the same issue later on, they can use your ticket to fix their issue.
Thanks


Update release publishing workflow to generate universal wheels

1) Enable universal wheel creation by default.
This saves having to pass --universal when creating wheels:
http://python-packaging-user-guide.readthedocs.org/en/latest/distributing/#universal-wheels
2) Update the publishing instructions to use twine and generate wheels.
The official Python packaging guidelines recommend using twine:
http://python-packaging-user-guide.readthedocs.org/en/latest/distributing/#upload-your-distributions
This avoids sending passwords over HTTP (when using Python <2.7.9) and has a few additional advantages:
https://github.com/pypa/twine#why-should-i-use-this
These instructions now also result in a wheel distribution being generated and uploaded to PyPI alongside the source distribution, which offers several advantages over it for compatible versions of pip:
http://python-packaging-user-guide.readthedocs.org/en/latest/distributing/#wheels

Thanks!
I think it would be fine to add twine to requirements/dev.txt and then remove the instruction to install it. There is already a prerequisite in the developer docs to install these deps.

PR updated, thanks for the feedback :-)

ok, looks good

Thanks :-)


Support cgroupsv2 for bootstrapping cluster

Feature Request
Especially with Docker Desktop defaulting to cgroupsv2 in 4.3.0+, we really need to support cgroupsv2 by default. Otherwise no one will be able to bootstrap any type of cluster without running an older version of docker desktop or, for linux hosts, adding a kernel parameter in to downgrade to cgroupsv1
This is a TCE tracking issue for:

https://github.com/vmware-tanzu/tanzu-framework/issues/118

sorry this was pre-maturely closed by me :) re-opening.

Just an FYI that I did build with v0.17 (of framework) today and this worked great.

We are now on the bleeding edge of tanzu-framework, however, the image used for kind-based managed clusters still resolves from the BOM.
Keeping this issue open until we can verify what kind image will end up in the BOM.

cgroup V2 is really needed for people who run on Linux Desktops (Ubuntu, Mint, PopOS). Asking adopters to change kernel parameters on their desktop or to spin up a VM just to try TCE provides substantial adoption friction.
Is there a timeline for this feature to be released?

Assigned to myself. i'll own the validation here, once the BoM is produced.

Maybe raise the priority a little, this CVE affects TCE:
https://unit42.paloaltonetworks.com/cve-2022-0492-cgroups/

Maybe raise the priority a little, this CVE affects TCE: https://unit42.paloaltonetworks.com/cve-2022-0492-cgroups/

:wave: @l0r3zz: You're right that this vulnerability may be applicable to bootstrap clusters. However, bootstrap clusters are just temporary clusters used to create the initial management-cluster. Additionally, you already have full admin access to the bootstrap cluster, so you can already run containers that arbitrarily bind to the host (which is viewed as the kind image.
Luckily, the clusters created by the bootstrap clusters (e.g. AWS, vSphere, etc), where workloads run, are not vulnerable to this CVE.
Here's an example test using TCE v0.10.0:
$ k get no -A -o wide | awk '{print $1}'
NAME
ip-10-0-20-154.us-west-2.compute.internal
ip-10-0-28-87.us-west-2.compute.internal

$ kubectl run --restart Never --rm -it --image=us-central1-docker.pkg.dev/twistlock-secresearch/public/can-ctr-escape-cve-2022-0492:latest test-for-cve-2022-0492
If you don't see a command prompt, try pressing enter.
Error attaching, falling back to logs: Internal error occurred: error attaching to container: container is in CONTAINER_EXITED state
[*] Testing whether CVE-2022-0492 can be exploited for container escape
[+] Contained: cannot escape via CVE-2022-0492

A few small notes:

unmanaged-clusters support cgroupsv2 today.

https://tanzucommunityedition.io/docs/latest/getting-started-unmanaged/

managemenet-cluster using Docker (local/CAPD) also gives you full admin access. Not to mention, it mounts the docker socket. For many reasons, far beyond security, these should be used for development and not real workloads.

Hope the above helps. We working to get cgroupsv2 working for local(CAPD) and bootstrap clusters in the next release.

Validated using:
$ tanzu mc version
v0.11.2

cgroups are v2:
$ mount | grep cgroup
cgroup2 on /sys/fs/cgroup type cgroup2 (rw,nosuid,nodev,noexec,relatime,nsdelegate,memory_recursiveprot)

BoM is:
$ tree ~/.config/tanzu/tkg/bom/
/home/josh/.config/tanzu/tkg/bom/
├── tkg-bom-v1.5.1-tf-v0.11.2.yaml
└── tkr-bom-v1.22.5+vmware.1-tkg.3-tf-v0.11.2.yaml

Cluster bootstrapping succeeded:

Summary:
cgroupsv2-based hosts will work in TCE v0.11.0


[New target candidate?] libui-ng for microcontrollers?

Hey there,
This is just a proposal / idea. Please feel free to close it at any moment in time.
In my spare time I investigate arduino, for various reasons. One is that I need
to be able to craft some micro-device that helps me automatically test cells
(such as bacteria and eukaryotic cells), where this mini-robot eventually
replaces the manual work from me (I hate pipetting and stuff so I want
robots to take over, and I need to eventually build my own one here).
libui-ng has as main candidates:

linux (via gtk 3 I think)
osx
windows

What if we were to add a fourth option, e. g. something for small embeddable
devices? E. g. microcontrollers?
I guess in the simplest scenario we could perhaps repurpose an existing
toolkit and just abstract this away. Unfortunately I know next to nothing
about this right now, so I am almost completely clueless - I can't even give
better ideas since I know so little.
On arduino we can use the Java IDEA there, and I suppose we could also
in principle use a GUI for arduino as-is. (People may wonder why but here
I am thinking more of a GUI similar to how web-interfaces run on a router,
you tend to get only a very limited subset of what you can do there; or
controller interfaces for TVs where people also do only a few tasks at
max).
Anyway - that's just a rough idea, kind of "extending libui use cases".
Please feel free to ignore and close, or discuss, then close. :)

linux (via gtk 3 I think)

Unix backend works anywhere you have Gtk, including mobile and embedded devices. Basically any SBC with video output and capable of booting a full operating system will work. Bare metal is not supported.
(For example the Raspberry Pi Zero would run your libui-ng apps written in Ruby, and it costs only 15 USD.)


Update process user input

Important
Please provide the requested parameters below to help us review your pull request.
Pull requests without a body or explanation will be rejected.
Refactor updateuserinput func to consolidate logic and args setttings into smaller units of work.
Issue ticket number and link
Closes #
Checklist before requesting a review

[x] I have performed a self-review of my code
[ ] If it is a core feature, I have added tests.
[x] I have run make check and there are no failures.

Type of change
Please delete options that are not relevant.
N/A

Figured I'd just start the PR so you could see the diff.

My pleasure!


Navigator rotates with animation when viewport.setRotation(90, true) is called

OSD Version 4.0.0
Reloading the page here shows that viewport.setRotation(90, true) animates the OSD viewer immediately without animation. However, the navigator animates and ignores the immediate property.

Looks like the code making the navigator mirror the main viewer's rotation needs to be updated to account for immediately:
https://github.com/openseadragon/openseadragon/blob/b45d500f1eeb3869caa807365f4cd5de03e074d8/src/navigator.js#L227-L241

Good catch!
@pearcetm Thank you for spotting the cause!
@robertjcolley Would you be up for making a patch?


[Rendering] Adaptive cards are not rendering

Target Platforms
NodeJS
SDK Version
1.4
Application Name
Microsoft Teams
Problem Description
Adaptive cards are not rendering in Channels, When adaptive cards or being posted in channel by Bot or webhooks the card is not being rendered for some users
Screenshots

Card JSON
{
            "type": "AdaptiveCard",
            "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
            "version": "1.3",
            "body": [
                {
                    "type": "TextBlock",
                    "text": "Please fill the form to get traceroute",
                    "wrap": true,
                    "size": "large",
                    "weight": "bolder",
                    "id": "title"
                },
                {
                    "type": "Input.Text",
                    "placeholder": "Enter an IP Address or Hostname",
                    "isMultiline": false,
                    "wrap": true,
                    "width": "stretch",
                    "maxLength": 100,
                    "id": "destinationIP",
                    "isRequired": true,
                    "errorMessage": "Please enter an IP Address or Hostname",
                    "label": "Destination Endpoint"
                },
                {
                    "type": "Input.ChoiceSet",
                    "value": "",
                    "choices": [
                      {
                          "title": "NAME",
                           "value": "URL"
                          },
                          {
                              "title": "NAME",
                              "value": "URL"
                              },
                              {
                                "title": "NAME",
                                  "value": "URL"
                                  },
                                  {
                                      "title": "NAME",
                                    "value": "URL"
                                    },
                                    {
                                        "title": "NAME",
                                        "value": "URL"
                                        }
                    ],
                    "placeholder": "Please select a value",
                    "id": "sourceDatacenter",
                    "isRequired": true,
                    "errorMessage": "Please select a source datacenter",
                    "label": "Source Datacenter"
                },
                {
                    "type": "ActionSet",
                    "actions": [
                        {
                            "type": "Action.Submit",
                            "title": "Submit",
                            "data": {
                                "postback_prop": true,
                                "postback_activity": "getTracerouteActivity"
                              },

                            "requiredInputs": [
                            "destinationIP",
                            "sourceDatacenter"
                            ]

                        }
                    ]
                }
            ]
          }

Sample Code Language
No response
Sample Code
No response

Confirmed this is happened for me as well , using the example card in https://learn.microsoft.com/en-us/adaptive-cards/authoring-cards/getting-started#example-card
{
    "attachments": [
        {
            "content": {
                "type": "AdaptiveCard",
                "version": "1.0",
                "body": [
                    {
                        "type": "TextBlock",
                        "text": "Here is a ninja cat"
                    },
                    {
                        "type": "Image",
                        "url": "http://adaptivecards.io/content/cats/1.png"
                    }
                ]
            },
            "contentType": "application/vnd.microsoft.card.adaptive"
        }
    ],
    "type": "message"
}

@DanielSmithGKGAB , confirmed this is working, thanks for the workaround

@DanielSmithGKGAB  Thanks for the solution ! I found the fix last week forgot to update this thread, @ringoc thanks for confirming. For those who are still facing this issue after trying out @DanielSmithGKGAB’s solution, Try this: Open the Run dialog box by pressing the Windows logo key +R. In the Run dialog box, enter %appdata%\Microsoft\Teams, and then select OK. Delete all files and folders in the %appdata%\Microsoft\Teams directory. Restart Teams.

Closing this thread as the fix is working


sst.aws.Nextjs: server.edge.viewerResponse.injection does nothing

following the provided example does not create a cloudfront function.
you can replicate the issue using a blank project as follows:
npx create-next-app@14 aws-nextjs
cd aws-nextjs
npx sst@latest init

# update sst.config.ts
/// <reference path="./.sst/platform/config.d.ts" />

export default $config({
  app(input) {
    return {
      name: "aws-nextjs",
      removal: input?.stage === "production" ? "retain" : "remove",
      home: "aws",
    };
  },
  async run() {
    new sst.aws.Nextjs("MyWeb", {
      server: {
        edge: {
          viewerResponse: {
            injection: `event.response.headers["x-foo"] = "bar";`
          }
        }
      }
    });
  },
});

npx sst deploy --stage=production

I'll take a look.

According to the CF functions docs you need an object to set the value of the header.
https://docs.aws.amazon.com/AmazonCloudFront/latest/DeveloperGuide/example_cloudfront_functions_add_cache_control_header_section.html
Your code should be:
export default $config({
  app(input) {
    return {
      name: "aws-nextjs",
      removal: input?.stage === "production" ? "retain" : "remove",
      home: "aws",
    };
  },
  async run() {
    new sst.aws.Nextjs("MyWeb", {
      server: {
        edge: {
          viewerResponse: {
            injection: `event.response.headers["x-foo"] = {value: 'bar'};`
          }
        }
      }
    });
  },
});

Hi @leoalves,
even though you are right, that's not the issue here.
Using the wrong cloudfront-js should still create a function which then fails on execution.
At least that's what happens when you try to do the same with viewerRequest injection instead.

Very odd. Retried with the latest version and still facing this issue. Also tried with different application names. Will try within another aws account and report back.

@jayair I think the function in your screenshot is the viewerRequest-function
I've tried with another aws account and can replicate the issue there as well. no viewerResponse-function is created and the x-foo-response-header is not set

It might be a bug. @fwang

Fixed in v3.3.41


Search  Country List

It will be great to have a search filter in the country list, when selecting countries as it is a really long list.
Thanks!

@KCastiglione Hey thanks for the idea, but I really kept it that way because of the usecase that I was looking for is quite simple.
May be you can fork it and create a pull request :)


feat: add support for .mjs file output

Hey.
This PR adds outputting .mjs files if the build option platform=neutral.
Currently we can use platform=neutral to output the bundle as esm. The bundle extension however remains .js. This is fine when it is accompanied by a package.json which has "type": "module" set.
However, the `package.json' is explicitly not included in the final zip.
https://github.com/floydspace/serverless-esbuild/blob/master/src/pack.ts#L38
This means that lambdas run the code as CommonJs.
Changing the file type seems a more explicit option (rather than package the package.json)
Considerations:
-platform=neutral now outputs .mjs files
Notes on testing

How typing is currently set up, it is hard to change buildOptions in tests.  I have therefore casted when added to the overrides.  Keeping the variable typed should ensure the type safety.

First PR to serverless-esbuild so let me know if you require anything else.
Partially closes #315 (package.json is no longer required)
Update
Have updated so that .js is the default extension thereby making this a none breaking change.
If you wish to change the extension you can now use the new outputFileExtension which has some validation to keep the user right.

PR changed to be more configurable

Hi @codingnuclei , thank you for the careful PR. I think it's a necessary addition.
I'm happy to merge it, but please consider my comments about serverless error

Hi @codingnuclei , thank you for the careful PR. I think it's a necessary addition. I'm happy to merge it, but please consider my comments about serverless error

This might be me being silly - but I am not seeing any comments?

@floydspace updated with your comments resolved :)


Bug:  module not found

Describe the bug

Cannot find module 'src/createDomEvent' or its corresponding type declarations.(wrapperLike.d.ts)

To Reproduce

add "@vue/test-utils": "^2.0.0-rc.18" to vite project
run vue-tsc
Cannot find module 'src/createDomEvent' or its corresponding type declarations.(wrapperLike.d.ts)

Expected behavior

No error

Related information:

@vue/test-utils version: v2.0.0-rc.18
Vue version:  3.2.25
node version: 16
npm (or yarn) version: 1.22.4

Additional context

I want to make a pr, but I can't push my branch

Hi @JuoJuo
This is a duplicate of #936, #1206, #1239, and #1268. It was fixed in #1184.
This should be released soon. In the meantime, you can use skipLibCheck in your tsconfig.

thanks


v2.0 release binary for windows missing

https://github.com/deplinenoise/tundra/commit/469c9bfc3f1ff4b855f172cd874d0740734f292a
I was able to compile ok on ubuntu.  Tundra is really fast.
I was getting errors crosscompiling it for windows hence the request for the binary :-)

My crosscompile env is a little shot at the moment. Is there a reason you can't build it yourself on Windows using Visual Studio?

I've never used visual studio except for their command line tools from http://landinghub.visualstudio.com/visual-cpp-build-tools
I tried to execute msbuild Tundra.sln but that didn't work.

I recently did this and it was a little bit of pain but once I found the right packages it built just fine.
You can use the msys2 environment on Windows and then download the tool chain from here https://sourceforge.net/projects/mingwbuilds/files/host-windows/releases/4.8.0/64-bit/threads-win32/seh/
Unzip that, add x86_64-w64-mingw32\bin to your path and configure the environment for the windows build, then run make.
I'm using VS2015 right now and it's giving my build errors as well but I think this is just because it's looking for the wrong version of the VC++ compiler. We can override this like so:

msbuild /p:PlatformToolset=v140;Configuration=Release vs2012\Tundra.sln

Which built just fine. Just download master, unzip and run the above command line from within the Visual C++ x64 Native Build Tools Command Prompt.
Here's a link to the binaries
https://drive.google.com/open?id=0B5JWqqxHqeHeTXZvenRHZlVxalE
That's from my personal Google Drive so, it's might get throttled but it will work for now.

Also, @deplinenoise I was under the impression that older versions of Visual Studio lacked the necessary C++11 features to actually compile tundra. Was I wrong?

Thanks for putting up the binaries @leidegre

VS2012 is fine, and I expect 2015 to also work. At some point after the GDC craziness I'll put up an official windows build.

Official binaries and installers are now available for 2.01.


Tried to register two views with the same name RNCMaskedView, js engine: hermes

Failure Information (for bugs)
ERROR  Invariant Violation: Tried to register two views with the same name RNCMaskedView, js engine: hermes
ERROR  Invariant Violation: "main" has not been registered. This can happen if:

Metro (the local dev server) is run from the wrong folder. Check if Metro is running, stop it and restart it in the current project.
A module failed to load due to an error and AppRegistry.registerComponent wasn't called., js engine: hermes

Steps to Reproduce
Just run npm run start

Hello friend, did you get a solution? I have the same problem @wtricks

deleting node_modules/@react-native-community/masked-view folder worked for me since it's a duplicate of node_modules/@react-native-masked-view/masked-view
Originally posted by @erickwarui28 in https://github.com/creativetimofficial/soft-ui-react-native/issues/32#issuecomment-2408531150_


check_config: Additional information in log output to avoid confusion

Updated log file to say that the new versions was installed ONLY to this add-on.
This is because previously new users were fearing that it installed a new version to their production machine due to the log message; this PR is to improve the log to indicate that the new version was installed only to this add-on.

While I do see what you are trying to do, it makes this inconsistent.
Secondly, I do not agree on a personal level. Please point me to reference where users were experiencing this issue.
For example, it makes no sense to make to adjust it on this line, but not on this one:
https://github.com/home-assistant/hassio-addons/blob/8440f9c548741acca9db95e4016350190a517bc2/check_config/run.sh#L12
Maybe it makes more sense to add a logline right above pasted snippet that shows a message like:

"Don't worry, this temporary installation is not overwriting your current one."

Hello @SiliconAvatar,
When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es).
The commits that are missing a linked GitHub account are the following:

90e50a3e4f15ae138ad240071caaf9e11ad2c70d - This commit has something that looks like an email address (private@a.a). Maybe try linking that to GitHub?.

Unfortunately, we are unable to accept this pull request until this situation is corrected.
Here are your options:

If you had an email address set for the commit that simply wasn't linked to your GitHub account you can link that email now and it will retroactively apply to your commits. The simplest way to do this is to click the link to one of the above commits and look for a blue question mark in a blue circle in the top left. Hovering over that bubble will show you what email address you used. Clicking on that button will take you to your email address settings on GitHub. Just add the email address on that page and you're all set. GitHub has more information about this option in their help center.

If you didn't use an email address at all, it was an invalid email, or it's one you can't link to your GitHub, you will need to change the authorship information of the commit and your global Git settings so this doesn't happen again going forward. GitHub provides some great instructions on how to change your authorship information in their help center.

If you only made a single commit you should be able to rungit commit --amend --author="Author Name<EMAIL_ADDRESS>
(substituting Author Name and<EMAIL_ADDRESS>for your actual information) to set the authorship information.
If you made more than one commit and the commit with the missing authorship information is not the most recent one you have two options:

You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
You can use this script that GitHub provides to rewrite history. Please note: this should be used only if you are very confident in your abilities and understand its impacts.

Whichever method you choose, I will come by to re-check the pull request once you push the fixes to this branch.

We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community.
Thanks, I look forward to checking this PR again soon! :heart:

https://community.home-assistant.io/t/what-is-up-with-all-these-breaking-changes/103134/187
First example I could find. Regardless of examples, showing "Installed version XX" to users who not technical may not be the best thing, considering this is only installing it in the container to check config. I'm trying to keep new users in mind.

Maybe it makes more sense to add a logline right below the above pasted snippet that shows a message like:

"Don't worry, this temporary installation is not overwriting your current one."

I agree and actually like that better than my original commit; I've amended the PR.


feat: 51-amelioration adresse

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


Deploy Cronjob for Package Feed

Deploy Cronjob. Frankly this can probably run on the same box that the Grafana instance runs on.
Blocked by: https://github.com/vipyrsec/infrastructure/issues/9
Blocks deployment/MVP

https://github.com/vipyrsec/infrastructure/pull/20


Interoperability with existing code/systems

We discussed this offline today and I'm just capturing the discussion here.  @JoeyZwicker @jdoliner @sjezewski please correct me if I'm missing anything.
So sounds like there are two problems:

Some users have existing code and systems that expect to talk to a certain API, which usually is the S3 API.  We want to make it easy for these users to transition to using Pachyderm.  One idea was to implement a S3 API on top of pfs.
Some users want to use their data in multiple systems.  For instance, a user who uses pps for analytics might also want to use Postgres for quick queries.  So we want to make it easy to pump data from pfs into these external systems.  One idea was to have a pipeline for each of these external systems.  The pipeline would simply listen for new commits and pump new data into the external system whenever a commit comes in.

The first one is now represented by this issue: https://github.com/pachyderm/pachyderm/issues/257
The second one is described in several issues that are talking about egress pipelines / services / persistent services.


feat: #605 add show exchangekey list page

Add the basic html structure of the get exchangekey list page

The code looks alright but needs to be merged into the main and integrated with other parts to see the effect.


Fix permissions on Docker

When running NodeOS on Docker, no matter what user do you login, it gets UID & GID 0, having root permissions. We need to find and fix this.
Docker AppArmor template
disable filesystems mounting
by default, so it's needed to disable usage of AppArmor at all. A better
alternative would be to use a custom template that mimics the Docker default one
but allowing filesystems mounting. This could be easily crafted with
Bane.
Docker default seccomp profile accept setud,
but for some reason it's not working and instead they are a no-op

how do you check its uid and git inside nodeos?

Wth pstree and looking what processes tree it print, the one for root or the one of nodeos user. Dumb, but effective :-)

what does your tree for root and for nodeos print?

Im getting this:
~ > pstree
init
├── exclfs
└─┬ init
  ├─┬ nsh
  │ └── pstree
  └── nodeos-reverse-

Im getting this (on docker):

This is the tree for root user.

But it should be like this (on QEmu)?

Hum... that's not good, seems on QEmu we are having a regression, because it's too the list of root processes... In both cases they should be
~ > pstree
nsh
└ pstree

This seem more like a software error from logon than from docker or qemu

This seem more like a software error from logon than from docker or qemu

Docker has a problem, but it's true that logon was using setreuid() and changed it to setuid() because the docs says it's more secure. Maybe this could be the reason of the problem with QEmu... Problem here is about how to test it :-/

You say that it was using setreuid() but the source on here is using setreuid()
the usersfs pulls the dependencies on build so this could be the problem or am i wrong?

Yes, no matter what of them you use, they are from the same functions family. It's a problem on/with Docker.


Make SettingsTab respect keybindings

Description of the new feature/enhancement
Some keybindings should be respected when in the settings tab. At the very least, we should respect "closeTab".
Ideally, we'd respect all app-level keybindings. This includes tab management, mode switching (i.e always on top, focus mode, fullscreen, etc...), and command palette. In the end, we'll really have "support" for "all" keybindings because the architecture checks if there's a terminal that can actually handle the action.

@carlos-zamora - does tab-switcher works for you when you attempt to start from the settings tab? As it doesn't for me - I am stuck there and cannot escape :smile:

@carlos-zamora - we recently added keybinding handling to the TerminalPage - see the key-down handler there. So I would expect the routed event to propagate there if not handled by the settings tab.
For instance this is how it works when you issue a chord in the command palette or when focused on the searchbox.
The terminal control is a special case because we wanted to reverse the order of handling (I am still not sure why preview key down in the TerminalPage won't do the job).
However here I hoped it will work the same. But something seems to swallow the events.

@carlos-zamora - we recently added keybinding handling to the TerminalPage - see the key-down handler there. So I would expect the routed event to propagate there if not handled by the settings tab.
For instance this is how it works when you issue a chord in the command palette or when focused on the searchbox.
The terminal control is a special case because we wanted to reverse the order of handling (I am still not sure why preview key down in the TerminalPage won't do the job).
However here I hoped it will work the same. But something seems to swallow the events.

GAH
Okay, it was trivial enough to just hook the settings UI up to the action dispatcher.
diff --git a/src/cascadia/TerminalApp/TerminalPage.cpp b/src/cascadia/TerminalApp/TerminalPage.cpp
index f5d700e4e..a1c618702 100644
--- a/src/cascadia/TerminalApp/TerminalPage.cpp
+++ b/src/cascadia/TerminalApp/TerminalPage.cpp
@@ -2806,6 +2806,7 @@ namespace winrt::TerminalApp::implementation
             {
                 sui.SetHostingWindow(reinterpret_cast<uint64_t>(*_hostingHwnd));
             }
+            sui.PreviewKeyDown({ this, &TerminalPage::_KeyDownHandler });

             sui.OpenJson([weakThis{ get_weak() }](auto&& /*s*/, winrt::Microsoft::Terminal::Settings::Model::SettingsTarget e) {
                 if (auto page{ weakThis.get() })

That'll do the job. However, then we have the very real problem that we don't actually check if the control is valid when we do things like
    void TerminalPage::_HandleAdjustFontSize(const IInspectable& /*sender*/,
                                             const ActionEventArgs& args)
    {
        if (const auto& realArgs = args.ActionArgs().try_as<AdjustFontSizeArgs>())
        {
            const auto termControl = _GetActiveControl();
            termControl.AdjustFontSize(realArgs.Delta());
            args.Handled(true);
        }
    }

So we'll need to audit all the actions to make sure they're safe to use from the context of the SUI.

GAH
Okay, it was trivial enough to just hook the settings UI up to the action dispatcher.
diff --git a/src/cascadia/TerminalApp/TerminalPage.cpp b/src/cascadia/TerminalApp/TerminalPage.cpp
index f5d700e4e..a1c618702 100644
--- a/src/cascadia/TerminalApp/TerminalPage.cpp
+++ b/src/cascadia/TerminalApp/TerminalPage.cpp
@@ -2806,6 +2806,7 @@ namespace winrt::TerminalApp::implementation
             {
                 sui.SetHostingWindow(reinterpret_cast<uint64_t>(*_hostingHwnd));
             }
+            sui.PreviewKeyDown({ this, &TerminalPage::_KeyDownHandler });

             sui.OpenJson([weakThis{ get_weak() }](auto&& /*s*/, winrt::Microsoft::Terminal::Settings::Model::SettingsTarget e) {
                 if (auto page{ weakThis.get() })

That'll do the job. However, then we have the very real problem that we don't actually check if the control is valid when we do things like
    void TerminalPage::_HandleAdjustFontSize(const IInspectable& /*sender*/,
                                             const ActionEventArgs& args)
    {
        if (const auto& realArgs = args.ActionArgs().try_as<AdjustFontSizeArgs>())
        {
            const auto termControl = _GetActiveControl();
            termControl.AdjustFontSize(realArgs.Delta());
            args.Handled(true);
        }
    }

So we'll need to audit all the actions to make sure they're safe to use from the context of the SUI.

GAH
Okay, it was trivial enough to just hook the settings UI up to the action dispatcher.
diff --git a/src/cascadia/TerminalApp/TerminalPage.cpp b/src/cascadia/TerminalApp/TerminalPage.cpp
index f5d700e4e..a1c618702 100644
--- a/src/cascadia/TerminalApp/TerminalPage.cpp
+++ b/src/cascadia/TerminalApp/TerminalPage.cpp
@@ -2806,6 +2806,7 @@ namespace winrt::TerminalApp::implementation
             {
                 sui.SetHostingWindow(reinterpret_cast<uint64_t>(*_hostingHwnd));
             }
+            sui.PreviewKeyDown({ this, &TerminalPage::_KeyDownHandler });

             sui.OpenJson([weakThis{ get_weak() }](auto&& /*s*/, winrt::Microsoft::Terminal::Settings::Model::SettingsTarget e) {
                 if (auto page{ weakThis.get() })

That'll do the job. However, then we have the very real problem that we don't actually check if the control is valid when we do things like
    void TerminalPage::_HandleAdjustFontSize(const IInspectable& /*sender*/,
                                             const ActionEventArgs& args)
    {
        if (const auto& realArgs = args.ActionArgs().try_as<AdjustFontSizeArgs>())
        {
            const auto termControl = _GetActiveControl();
            termControl.AdjustFontSize(realArgs.Delta());
            args.Handled(true);
        }
    }

So we'll need to audit all the actions to make sure they're safe to use from the context of the SUI.

Or we can bind only switch tab. The way we did it in cmdpal? If this is OK I can try to do it.

GAH
Okay, it was trivial enough to just hook the settings UI up to the action dispatcher.
diff --git a/src/cascadia/TerminalApp/TerminalPage.cpp b/src/cascadia/TerminalApp/TerminalPage.cpp
index f5d700e4e..a1c618702 100644
--- a/src/cascadia/TerminalApp/TerminalPage.cpp
+++ b/src/cascadia/TerminalApp/TerminalPage.cpp
@@ -2806,6 +2806,7 @@ namespace winrt::TerminalApp::implementation
             {
                 sui.SetHostingWindow(reinterpret_cast<uint64_t>(*_hostingHwnd));
             }
+            sui.PreviewKeyDown({ this, &TerminalPage::_KeyDownHandler });

             sui.OpenJson([weakThis{ get_weak() }](auto&& /*s*/, winrt::Microsoft::Terminal::Settings::Model::SettingsTarget e) {
                 if (auto page{ weakThis.get() })

That'll do the job. However, then we have the very real problem that we don't actually check if the control is valid when we do things like
    void TerminalPage::_HandleAdjustFontSize(const IInspectable& /*sender*/,
                                             const ActionEventArgs& args)
    {
        if (const auto& realArgs = args.ActionArgs().try_as<AdjustFontSizeArgs>())
        {
            const auto termControl = _GetActiveControl();
            termControl.AdjustFontSize(realArgs.Delta());
            args.Handled(true);
        }
    }

So we'll need to audit all the actions to make sure they're safe to use from the context of the SUI.

Or we can bind only switch tab. The way we did it in cmdpal? If this is OK I can try to do it.

Here's my thought though - like real long term, I want to be able to have panes that don't host terminals in them. So eventually, we'll want this work to be done. Might as well do it now.
Plus, we definitely want SwitchToTab, but also probably NewTab, CloseTab... I'm not sure if that's it or now. May as well just go for it and do 'em all.

Here's my thought though - like real long term, I want to be able to have panes that don't host terminals in them. So eventually, we'll want this work to be done. Might as well do it now.
Plus, we definitely want SwitchToTab, but also probably NewTab, CloseTab... I'm not sure if that's it or now. May as well just go for it and do 'em all.

Here's my thought though - like real long term, I want to be able to have panes that don't host terminals in them. So eventually, we'll want this work to be done. Might as well do it now.
Plus, we definitely want SwitchToTab, but also probably NewTab, CloseTab... I'm not sure if that's it or now. May as well just go for it and do 'em all.

So let's do it now.
2 questions:

Is it a blocker for 1.6 preview?
Are we OK to push a fix for all bindings to 1.6 preview?

Here's my thought though - like real long term, I want to be able to have panes that don't host terminals in them. So eventually, we'll want this work to be done. Might as well do it now.
Plus, we definitely want SwitchToTab, but also probably NewTab, CloseTab... I'm not sure if that's it or now. May as well just go for it and do 'em all.

So let's do it now.
2 questions:

Is it a blocker for 1.6 preview?
Are we OK to push a fix for all bindings to 1.6 preview?

IMO yes and yes? @DHowett back me up on this decision though. I'm working through em all now, but we're running short on runway

IMO yes and yes? @DHowett back me up on this decision though. I'm working through em all now, but we're running short on runway

IMO yes and yes? @DHowett back me up on this decision though. I'm working through em all now, but we're running short on runway

OK.. there are hardly 7 lines that require a fix.

IMO yes and yes? @DHowett back me up on this decision though. I'm working through em all now, but we're running short on runway

OK.. there are hardly 7 lines that require a fix.

From @rashil2000 in #8987:

Environment
Windows build number: 10.0.19042.746
Windows Terminal version (if applicable): 1.6.10272.0

Steps to reproduce
Multiple ways. Open the Settings UI tab using your keybind (like Ctrl+Shift+,). Now use other keybindings, like toggleFocusMode (Shift+F11), or Command Pallete (Ctrl+Shift+P), or JSON file (Ctrl+,), or new tab (Ctrl+Shift+T).
Expected behavior
They should open their respective actions, like any other tab.
Actual behavior
None of these commands seem to work.
Nothing happens.
Context
I have set my launchmode to focus, so each WT sessions opens in focus mode. But when I open the Settings UI from there, I get stuck on that tab, because the toggleFocusMode keybind does not do anything, and I can't even open a new tab.

If we're nor going to have all keybindings work, then we'll definitely want to add

close window
toggleFocusMode
toggleFullscreen
command palette?

I agree about wanting to have at a minimum those four features. I disagree about the copy/paste not being required since there is a section to input the whitespace characters as well as paths for the profiles.
I personally am not a fan of the new Windows Settings/the newer Metro (Fluent design?) style. I still try to use Control Panel for as much as possible since the older one included more information without feeling as visually bloated as the newer one. The below screenshot has a ton of whitespace on the right hand side, on the sidebar and for the "Discard changes" and "Save" buttons while also requiring the user to scroll down to see all the options within the same section ("Appearance").
Is there an easy way to use something similar to VSCode's settings menu (I am fully aware that I have no idea how long it takes to make a UI so this may be the user asking for something unrealistic)?

I disagree about the copy/paste not being required since there is a section to input the whitespace characters as well as paths for the profiles.

Oh, we can't have the copy and paste keybindings that you've set for the Terminal work in the SUI, because of the way those keybindings are handled. Those two are keybindings specific to the TermControl (the control hosting the terminal content), that specifically tell it to use those keystrokes for copy/paste. We can overwrite those keystrokes, because we own the entire input to the TermControl.
Normal xaml TextBoxs throughout the SUI will still have copy/paste on ctrl+cctrl+v, but we can't overload those controls.

The below screenshot has a ton of whitespace on the right hand side, on the sidebar and for the "Discard changes" and "Save" buttons while also requiring the user to scroll down to see all the options within the same section ("Appearance").
(I am fully aware that I have no idea how long it takes to make a UI so this may be the user asking for something unrealistic.)

Trust me, the Settings app team hears this feedback constantly. We're just trying to be consistent with the rest of the platform, and that means following their choices.

Trust me, the Settings app team hears this feedback constantly. We're just trying to be consistent with the rest of the platform, and that means following their choices.

The system settings app tries to utilize the whitespace on the right by showing help links, extra useful info etc. Maybe the terminal can do that too...

@rashil2000 it's a good thing we're already tracking that in #9824

Oh, we can't have the copy and paste keybindings that you've set for the Terminal work in the SUI, because of the way those keybindings are handled. Those two are keybindings specific to the TermControl (the control hosting the terminal content), that specifically tell it to use those keystrokes for copy/paste. We can overwrite those keystrokes, because we own the entire input to the TermControl.
Normal xaml TextBoxs throughout the SUI will still have copy/paste on ctrl+c,ctrl+v, but we can't overload those controls.

Ahh, so it is the user-defined copy/past you all were talking about. I should've thought that through before commenting. Apologies.

Trust me, the Settings app team hears this feedback constantly. We're just trying to be consistent with the rest of the platform, and that means following their choices.

I forgot that Terminal was a UWP app as opposed to a traditional .exe like vscode and others are. I've noticed in Terminal, Settings and Feedback Hub, you cannot drag the size of sidebar. Is that something that is hard defined in the UWP design language or can that be implemented on an app-by-app basis?
Lastly (and out of scope of terminal), is there a way to give feedback to MSFT about the Settings app in general and asking them to not get rid of Control Panel in a future update?  (I don't know if they were planning on removing it in favor of Settings UWP completely.)

Terminal was a UWP app as opposed to a traditional .exe like vscode and others are.

That's not strictly true - it doesn't really make sense to think about apps as Win32 vs UWP apps anymore. The Terminal actually is a traditional exe, that just so happens to be using UWP XAML for its UI. Nowadays, win32 applications have access to basically the whole gamut of UWP APIs, so the line between the two is blurrier and blurrier.

I've noticed in Terminal, Settings and Feedback Hub, you cannot drag the size of sidebar. Is that something that is hard defined in the UWP design language or can that be implemented on an app-by-app basis?

I believe that's just a hardcoded part of the NavigationView in WinUI.

Lastly (and out of scope of terminal), is there a way to give feedback to MSFT about the Settings app in general and asking them to not get rid of Control Panel in a future update? (I don't know if they were planning on removing it in favor of Settings UWP completely.)

Frankly, I don't really know. Every team happens to have their own ways of doing things. Typically the Feedback hub is a good place to start. I don't know if UserVoice is still a thing, but I know we used that before we moved to being entirely on GitHub.


Privacy link shows in footer if posts have a tag named "Privacy"

Describe the bug
I have deleted the privacy.md file.
I have blog post that is tagged with the word "Privacy".
The privacy link shows in the footer.
To Reproduce
Steps to reproduce the behavior:

Delete privacy.md
Add a blog post with a tag "Privacy".
The privacy link shows in the footer.

Expected behavior
The privacy link should not show due to a post tagged with "Privacy".
Technical details:

Link to your GitHub project:
Wowchemy Version (from your go.mod):  v0.0.0-20210413192202-ab8d28988e0e
Hugo Version (run hugo version): 0.82.0
Browser/OS: all
Wowchemy Template (e.g. Academic): Academic

workaround
add a layouts/partials/site_footer.html and comment out block starting with {{ if or (site.GetPage "terms.md") (site.GetPage "privacy.md") }}

The Hugo code used is {{ with site.GetPage "privacy.md" }}. If you believe there to be a Hugo bug, consider filing it in the Hugo repo

Thanks. I can do that. But given that the call resides in the html that is part of this theme, it might worth considering using theme specific names for the terms and privacy files, e.g. wowchemy-privacy.md to avoid collisions with what might be common tags/categories in blogs.
Thanks for the great software.

Bug opened on hugo - https://github.com/gohugoio/hugo/issues/8494

Advice from response to Hugo issue I raised is to use {{ .getPage "/privacy.md" }} instead of {{  site.GetPage "privacy.md" }}  etc. I have tried this and it resolves the problem. Happy to make PR this change if you want?

This issue is there in the cookie_consent.html too. Instead of running site.GetPage just .GetPage is enough. Can this be changed in the cookie_consent.html and site_footer.html please? Thanks.

I still see this problem in my blog. The privacy link in the footer points to the unrelated /tag/privacy. Was the problem really fixed?
I'm using hugo v0.121.2 and
module github.com/HugoBlox/theme-academic-cv

go 1.15

require (
	github.com/HugoBlox/hugo-blox-builder/modules/blox-bootstrap/v5 v5.9.6
	github.com/HugoBlox/hugo-blox-builder/modules/blox-plugin-netlify v1.1.2-0.20231108141515-0478cf6921f9
	github.com/HugoBlox/hugo-blox-builder/modules/blox-plugin-reveal v1.1.2
)


MoneyTree from rubygems.org

Pull Request Test Coverage Report for Build 30

13 of 13 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 26:
0.0%

Covered Lines:
138

Relevant Lines:
138

💛  - Coveralls


Make the readme file more valuable

Created a much better readme layout as well as an example readme layout so that we can better inform people about what our repository is all about.

Fixes #3


Add a README

Would be great to have a readme to help people not familiar with Gamercade how to get started quickly.

Thanks I added the README.md file and the reference is
https://github.com/gamercade-io/zig-template


Support configuring default salt

Blog post: https://cs-syd.eu/posts/2021-09-11-json-vulnerability
The biggest problem exposed by this blog post is that any third-party code using hash (using the default salt) that has problems with hash collisions can have malicious input constructed beforehand. The -frandom-init-seed flag helps with this at the cost of a different seed every start up.
One quick fix that can help mitigate this is to allow reading an environment variable like HASHABLE_DEFAULT_SALT with unsafePerformIO that one can set differently for their application. This has the advantage that my application can hash things with a different salt than the default, publicly available one, so an attacker attacking my application can't reuse malicious values from attacking another application. It would also be difficult to construct malicious input if my application's default salt isn't shared, and my application doesn't easily expose the default salt in any way.
Related:

https://github.com/haskell-unordered-containers/unordered-containers/issues/319
https://github.com/haskell/aeson/issues/864

One quick fix that can help mitigate this is to allow reading an environment variable like HASHABLE_DEFAULT_SALT with unsafePerformIO that one can set differently for their application.

This is SO hacky that we shouldn't even merge the PR, just have people use the open PR if necessary. IMO

I disagree that it's "SO" hacky. Using environment variables to configure upstream behavior is not uncommon (SYSTEM_CERTIFICATE_PATH for x509-system or http_proxy for http-client). unsafePerformIO isn't terribly hacky if used correctly, in this case to do one-time IO setup actions.

wouldn't setting HASHABLE_RANDOM_SEED work?

@jappeace yes that's a (very short-term) workaround.

It doesn't give the ability for configuring it, but I think you don't want to configure it anyway, just let it be random on boot.

Yes, I mentioned that in the beginning

The -frandom-init-seed flag helps with this at the cost of a different seed every start up.

But in some situations, you don't want it to be random on boot, such as when you need deterministic tests.


WMS 5361 - Updates to GoldenGate Veridata Workshop

Hi,
Could you please approve this PR to republish GoldenGate workshop.
Thanks,
Anu
General requirements

Information in the workshop is adequate and updated
Code is correct and working
Links are correct
Make sure you enter the help email link in your manifest.json
Please make sure WMS URLs are updated as needed after your PR is approved

Checklist - Refer to the QA document for the complete list
Please confirm that the following is completed before submitting your PR

[ ] All filenames are lower case (including folders, images, files, etc.)
[ ] Filenames are descriptive
[ ] Your workshop folder structure should be similar to the one used in the sample workshop (https://github.com/oracle-livelabs/common/tree/main/sample-livelabs-templates/sample-workshop)
[ ] Are you using multiple versions (desktop/, sandbox/, tenancy/)? Make sure that each of them contains a manifest.json and an index.html
[ ] Image references in markdown contain an alternate text

Image names should be all lowercase, please rename them

Closing the PR per Anuradha's request.


Add ability to use components props in observe

Will be nice to have an access to components props (which is passed by parent) in observe. This feature can allow to perform some actions, get some stuff from "store" according to passed value, or even simply wrap up third-party component without creating the new one.
This proposal is inspired by react-redux package which allow to do same things.

you raise a very interesting question, and thanks for that! we never thought of supporting it in fn of:
observe(fn)(Component)

since it always expects the app instance. we will definitely think about it more.
but can you please share any existing use case for it?

one possible API:
observe(function (app, props$) {
  return Observable.of({});
})(RootComponent);

or make the second argument an object, and we can destruct the props$ out of it:
observe(function (app, { props$ }) {
  return Observable.of({});
})(RootComponent);

I don't fully understand what the use cases would be, but as for your proposed APIs, I think option 2 (passes the props$ observable as part of an extra object argument) is better and leads to a more clean and easy-to-maintain API in future.

@alexmiranda @fahad19 Just explaining without code: you often have cases when for example your data stored in store (or in some service in app), and component get key from parent. So it needs to take data from store, get the right data by key, and perform some actions, render, etc.
For now we can pass throw observe this data, and in render method get necessary data.
I guess the better way is in observe get the passed key, get data, and pass to component exact what it needs. So you don't do any additional actions in render method. Just display this data.
As I said before analogue of this approach is connect method from react-redux, which have an "ownProps" argument.
Sorry for the long answer :(

@JutIgor: I am looking into it right now, as I hit a use case myself with dynamic routing using frint-router-react.


Clearer date error messages

The error 'Date must be a real date' does not help users to fix the problem, as it is not very descriptive. This breaks down the reasons why a date might not be a real date to help users fix the problem and continue with the service.

Added some decision help suggestions: https://github.com/alphagov/govuk-design-system-backlog/issues/42#issuecomment-2213800580

Hi @tomyems, thanks for this contribution.
Speaking personally, the error messaging is somewhat ambiguous because the error the user has made is itself often ambiguous and cannot be programatically determined.
For example:

If I enter 31 06 2024, that is an invalid date, but you don't know whether I've typo'd the day of the month (I could have meant to type 30) or if the day is correct and it's actually the month that's entered incorrectly (it could be 05 or 07).
If I enter 29 02 2025, that is an invalid date, but you don't know whether I've entered the day wrong (could be 28), the month wrong (could be 01 or 03) or the year wrong (could have meant 2024).

In both examples above, only one or more than one field could have been entered incorrectly, but it's not possible to determine with any certainty which field or fields are wrong. For this reason it is generally expected that teams validate the date as a whole (as though it came from a single input) rather than validating each of the fields individually.
This doesn't apply if someone has left a field blank or entered the 42nd day of the 13th month, of course—in those cases it's obvious—but hopefully you see what I'm getting at.

Yes that section is the hardest to get to work - and for transparency I do not have research to support this.
For 31 06 2024, I believe 'Day must be a number between 1 and 30 for June' to be much more helpful that 'Date must be a real date'.
This is because it provides the context to the user about what is wrong with the date - both that 'June' could be wrong or the '31' could be wrong (though it is biased towards 'day'). This is particularly helpful for users who are less familiar with the numerals for dates (perhaps used to different calendar systems and so on). This is why both the 'day' and 'month' field should be highlighted.
You could try '31 June does not exist. For June, day must be a number between 1 and 30. For 31, month must be January, March, May, July, August, October, December.' - I've tried this several different ways/shorter - open to ideas about how to simplify it
[similar, but slightly more complex argument for 29 02 2025]
[P.S. can always drop the changes for this section and revert to 'Date must be a real date' - if not confident enough to support the changes, but happy with the others]


Set umask 0002 on macos for shared classes caches to work

Resolves https://github.com/AdoptOpenJDK/openjdk-tests/issues/1262
Signed-off-by<EMAIL_ADDRESS><EMAIL_ADDRESS>

Tests:
OSX: https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/1901/consoleFull
Linux: https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/1902/console
Windows: https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/1903/console


Add support for announcing and querying LBRY streams over BEP15 (BitTorrent Trackers)

Coverage decreased (-0.02%) to 70.106% when pulling 305991f36cf82d469fcc439bd4d29d8faa58a6cf on trackers into f05943ff79a0b4eb89442446fba48b3229e2c6b7 on master.


csv: Support Binary-typed columns in the CSV writer

Which issue does this PR close?
Closes #3291.
Are there any user-facing changes?
The API is not changed.
The docs currently say:

The writer does not support writing ListArray and StructArray.

So no documentation changes are required - this MR actually brings the code more in line with the existing docs.

IIUC you can't (in general) round-trip an arrow table through a CSV file.  So I suppose you mean the other way: round-tripping a CSV file through an arrow table and getting the exact same CSV back out again.  This MR doesn't touch the CSV reader, so in theory that kind of test should be unaffected.

IIUC you can't (in general) round-trip an arrow table through a CSV file

Where are you reading this, being able to read back what was written is table stakes for a data format.

you can't (in general) round-trip an arrow table through a CSV file

What I mean by this is that arrow is a more expressive format than CSV.
For example, you might have an arrow array containing Timestamp values.  When you convert to CSV they get stringified.  Then, when you convert back to arrow, your Timestamp array has turned into a String array.  Or, perhaps the CSV reader is clever and manages to guess that the strings in the CSV file are really timestamps!  In that case, Timestamp arrays will indeed roundtrip correctly; but a String column containing values which look like timestamps will fail to roundtrip (since it'll come back as a Timestamp column).
I'm no arrow expert so what I'm saying might be nonsense!  (eg. Maybe you guys are encoding the types in the CSV header or something?)  Naively, though, I would think that the relationship between arrow tables and CSV files is many-to-one; in others words, CSV is a lossy encoding for arrow tables.  Going in the other direction (roundtripping CSV files through arrow and back to CSV) should work perfectly, though.
Again, please let me know if I'm way off on this!

we should be able to read the CSV files written and get the same data back

The reader could look for things which look like hex-escapes, and convert the column to binary in that case, by parsing the escapes?  AFAICT there's nothing for this in libstd though, so we'd have to write a parser.

perhaps the CSV reader is clever

The reader is given a schema with which to interpret the encoded values - see here.

manages to guess that the strings in the CSV file are really timestamps

The CSV schema inference logic will do this, it has a load of regex expressions for this purpose

CSV is a lossy encoding for arrow tables

In general the arrow implementations try very hard to not be lossy, it is actually a source of non-trivial pain, just google for "allow_truncated_timestamps". In general we should strive very hard to not be a lossy encoding.

The reader could look for things which look like hex-escapes

Perhaps we could do what the python writer does and allow writing binary arrays so long as the content is valid UTF-8? Perhaps you could expand upon the use-case of encoding binary data in a non-binary format?

reader is given a schema

Aah, I see! You convert an arrow table into a CSV file + a schema, and then you can losslessly convert that pair back into the original arrow table. Got it!
And it sounds like you require that, if the writer succeeds in producing a CSV file, then the reader should produce the original arrow table when given the original schema (rather than, say, an error). Now I understand why you want some work on the reader side, to maintain this invariant.

binary data will not be correctly escaped

Oh! I assumed (but didn’t check) that this escaping was happening later in the writer code.

the use-case of encoding binary data in a non-binary format

I have a bunch of parquet files which happen to contain a few mostly-ascii columns. When making changes to the code which generates these files, I find it useful to eyeball the contents of the files. I do this by converting them to CSV. I’m sure there are better ways to do it…

then the reader should produce the original arrow table when given the original schema

Yup, in a lot of data systems the schema will be stored separately in some sort of catalog, and so this is a fairly important invariant

that this escaping was happening later in the writer code.

Not sure either, certainly worth a test or two

I’m sure there are better ways to do it…

You could use the pretty print module, I think it supports binary arrays (although I could be mistaken).

contents of the files

A cursory google would suggest that CSV files need not be entirely UTF-8. As such perhaps we could avoid escaping at all? Provided we can both read and write it, I don't see any issue.

Marking as a draft as I'm not sure there is consensus on how to move forward with this


feat: source project number from data resource

removes the variable for project number and sources this from the data resource instead to remove duplicate entry

Two things

we would also want the removal of the project number in our alternate terraform file main.tf.nospanner
This code also removes spanner from the main terraform file. Labs that do not include spanner is what main.tf.nospanner is for. However, this project does utilize spanner to showcase how Duet AI is integrated with spanner tooling as well, even if it is not really used for the actual "product."


Update azure-ad-ipv6-support.md

"How to find the IPv6 address used to sign-in to Azure AD" section added by Abizerh

@MicrosoftGuyJFlo Let me know if you finish your review for this PR. Then, we will proceed to review and merge the changes.


code cleanup: move admin config files from kalamu/kalamu to kalamu/cm…

…s-admin-bundle

Perfect @Ezar101 thanks for your help


How to build without downloading modules?

I'm trying to package Atom for a distribution I use called Exherbo Linux, which is a source-based distribution akin to Gentoo, and it's package manager uses a sandbox when building packages so that they do not mess with the system outside when not expected. This includes sandboxing the network, so that it can not contact the outside world and is only allowed to use any whitelisted IP addresses, or instead has to fetch dependencies outside of the compilation process.
Atom presents a problem here, because it's build script fetches dependencies while installing, which is not allowed in the sandbox.
To get the point, my question is: is there a supported way to fetch dependencies of the build script, disable any network related operations during compilation, and then build as normal?

...

...

To get the point, my question is: is there a supported way to fetch dependencies of the build script, disable any network related operations during compilation, and then build as normal?

Once the modules and bundled packages have been installed a network connection shouldn't be needed to build.
Modules are only downloaded when they are out of date so once they are synced, the network shouldn't be required.
So these steps should work:

Run script/bootstrap
Disable the network
Run script/build
Atom is built

Please let me know what issues you see, if any, with these steps, sorry for the delayed response.

This is almost right, but the issue now is that it still compiles with script/bootstrap. All resources need to be fetched, and then compiled in a later phase; this might help with understanding the environment being worked with.
Atom's script/bootstrap still compiles things instead of just downloading what is needed and then compiling with script/build

but the issue now is that it still compiles with script/bootstrap.

Yeah, npm, which is what Atom uses, compiles things as they are installed during bootstrap, I'm not sure there is a way around this currently.

Dang.

#3717 needs this as well, apparently

So, is there a related npm issue that allows building 'offline'? Would really love pointers into how this can happen...

Hello,
I would like to ask if there are any updates in regard to this. I am also for Exherbo, and I am wondering if there is any way to download all of the modules without having to download them one by one through npm.
Thanks!


Focus being stolen

First off: This is not talking about the Selection Focus, but the Screen Focus.
I believe this is related to the Linter portion of the Skyrim Syntax. Whenever there is an error in the .psc file, it will jump to that line and put it in the center of my screen. Great, incredibly helpful...until every time i modify the source code it steals focus from where i was, to try and show me the error i already knew about.
It would be great if you stored the last errors line #, and ensured that the # didn't change >2 lines before stealing focusing.
Reproducible steps: On a large .psc file (I was modifying a 2.7k line script), copy and paste an if statement on a variable, and change the variable name. Return to the top of the script to create the new variable for it, and do the same for the properties (Lazy coding ftw :>), but after pasting the property, the focus will be returned to the last error.
Also, are you taking feature requests? If so, i have one but not sure if it's possible to (cleanly) implement.

There is a setting, center_highlighted_line, that you can modify in order to disable the behavior you are talking about. Go to Preferences > Package Settings > SublimePapyrus > Settings - User and copy and paste the following:
"center_highlighted_line": false
What is the feature you are thinking about?

The thing is that i want to keep the feature, i just don't want it to occur every time i change something (Sort of a cache, only centering the highlighted line when it the line changes)
As for the feature, it appear user error was preventing what i wanted, and you can absolutely ignore that portion of the question!

Do you mean that the line that causes the error should not be brought to the center of the screen while you are editing that line? So that the line could be kept at the bottom or top of the screen without having the line being brought to the center if the linter finds an issue with the partially finished line?

No no, The current way it handles it is fine.
What happens when you go to edit a line at the top of the file, the linter finds the same error again, and centers the problematic line again, when i'm in the middle of fixing the issue (In the case this occured was a missing variable)

I think I now understand what you mean.

The upcoming version will include tracking of errors until the linter no longer finds errors. A previously encountered error (or what is assumed to be the same error based on the message and the line it is on) is no longer centered more than once unless the line it is on has changed by more than the value defined in a new user setting (default: 2 lines).
I still need to do some more testing (the update contains over 20 additions, changes, and fixes), update the readme, and merge the update branch into the master branch. The new version will hopefully be available by the end of this week.

The new version is out.


Integrate Eclipse SommR (Some/IP stack)

The Eclipse SommR project is going to provide a Some/IP implementation.
The SommR daemon should be preinstalled on the Leda distribution.

There is no sources available yet :( so maybe starting with RISC-V ?


question "For You" Tab

Just installed the extension and am I missing something or can't we add a column for the "For You" tab of regular twitter?

that never existed in old tweetdeck

that never existed in old tweetdeck

Yep I know, so not available then thanks


Angular 8

Would it be possible to get the peer dependencies updated if Typescript 3.x is supported so we can lose the peer dependency warnings on Angular 8?
Appears to work fine in the projects I have tried it with Angular 8 / Typescript 3.4.5.

:tada: This issue has been resolved in version 5.0.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


make sure the node_modules are installed before generating the tarball

Signed-off-by: Augustin Husson<EMAIL_ADDRESS>

:+1:


Add numbers and booleans to Table

Table currently has only strings - would be great to be able to add numeric and boolean columns as well.
Would really help with export/import to other applications.

I just tested this with a number and a boolean in my properties.  Changing a 3 to 4 in the table resulted in a number in the geojson, but changing a false to true resulted in the string "true" in the data.
The table is created using d3-metatable.  Unclear what the lift would be for adding a way to specify the type on properties.

Maybe as a part of this, but switching to a table component that would allow for highlight/copy/paste multiple rows and drag-to-fill à la spreadsheets would be a game changer.
I’m not referring to formulas or anything, just the text manipulation.
I would love to help, but I don’t know much in the way of JS or web development


Effective Supply is 1.5 mil... but it says tokens needed to pass is 3 mil... it should say not possible

See scenario 3... there should be a check to confirm that if its asking for more tokens than the effective supply then it's not possible

Still happening

@GriffGreen fixed, one dimension of the conviction voting period was not being considered into the "not possible" cap


Behaviour of RPermitExpirableSemaphore

few questions on RPermitExpirableSemaphore

what is the probability that permit id generated by  acquire() method returns same permit id for different or same key ? In otherwords is there a possiblity of collision of permit Ids in any scenario when acquiring the permit via this interface ?

What happens to the permit when redisson node on which the permit was taken has crashed after permit ? Does the permit expire ? Does the LockWatchDogTimeout come into play here as well ?

How does LockWatchDog Scheduler works ? Is this a single daemon thread or is there a TimerTask triggered for every lock request ? I am looking to understand implementation details of LockWatchDog . can we get a reference to the java class in Redisson Source code that is responsible for prolonging the expiry of keys

for 10^12 hashes probability is 2.93873588×10^−15. Basically zero.
that depends on leaseTimeout parameter defined during acquisition.
LockWatchDog Scheduler only applied to Lock object. https://github.com/redisson/redisson/blob/f7b3fbc64a422e625f974b23f1d5dfc73542ad1e/redisson/src/main/java/org/redisson/RedissonLock.java#L301

for 10^12 hashes probability is 2.93873588×10^−15. Basically zero.
-- is this true for when calculating for same key same mapname

it's true for overall amount of hashes.


headroom.js can't be used in angular

In angular.headroom.js  at last line the 'window.Headroom' is undefined

Are you using CommonJS or AMD modules? Check out my comment on #213

@sofichael please do not create duplicate issues. If you have any further comments, please add them to #264


Make BluFi (nimble) customizable broadcast (on master branch) (IDFGH-7388)

Raise the PR #8958 on the master branch

Thanks for your contribution.

sha=563d988668094c75660764f2f345e1735959d0ed


build: update peer dependencies to account for 15.1.0-next

superseded by https://github.com/angular/angular-cli/pull/24462


What does the check mark boxes mean in the "Customise Blocklist" button for the Windows10DebloaterGUI?

I got a question regarding what the check mark boxes when clicking on them in the "Customise Blocklist" button when running the Windows10DebloaterGUI. Does the checked boxes mean that it is the files you want to uninstall/delete? Or does the unchecked boxes represent the files you want to uninstall/delete?

https://github.com/Sycnex/Windows10Debloater#allowlist-and-blocklist

There may be some confusion, but when using the Allowlist/Blocklist, the checkmark means it is on the blocklist, and that it will be removed.

Hi! Please read the README before making an issue, or search already-existing issues for similar problems. As this is not a real issue, please close it.


fix: Dashboard Highlight card skeleton covers hovercard details

Description

This PR fixes the following bug:
When the stats are still loading on the page https://app.opensauced.pizza/javascript/dashboard/filter/recent?range=30, hovering on a contributor from the repositories table would lead to hovercard details being covered by the dashboard highlight card skeleton.
What type of PR is this? (check all applicable)

[ ] 🍕 Feature
[x] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents

Fixes #2193
Mobile & Desktop Screenshots/Recordings

Before:

After:

Added tests?

[ ] 👍 yes
[x] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[x] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

Thanks @nickytonline for letting me know the changes.


CI: adding a Windows CI job with MSVC + MKL + Intel Fortran (ifx)

The recent troubles with MSVC (e.g., gh-20840) and MKL (e.g., gh-20728) are a reminder that we really need a Windows CI job to cover this setup. Having one would prevent serious issues like those showing up after tagging a release.
We had a preliminary attempt in gh-16957, before getting rid of Azure. The plan there was to wait for Flang+conda-forge, but that still does not have a fixed timeline, so ifx + MKL is better because it can be done now. A new job should be on GitHub Actions. Intel maintains a repo with examples that is useful; this should be a decent start:
https://github.com/oneapi-src/oneapi-ci/blob/4313a81c1ed3dac47f0c431f735c55b6b569333b/.github/workflows/build_all.yml#L72
That doesn't cover MKL exactly, but it seems that's a small change: https://github.com/oneapi-src/oneapi-ci/issues/71.
@czgdp1807 would you be able to work on this?

Yes. I will take this up.


Look into potential uses of seqspec

Seqspec (https://github.com/IGVF/seqspec) is a machine-readable specification for libraries, a file format based on yaml, that enables sequence read annotation. The scope is broader than ours, including ATAC-Seq, WGS, SHARE-Seq, also not just Illumina.
We could explore the uses of seqspec in relation to HTSinfer, maybe compare the results in some known cases (e.g. detailed information from publication/sequencing kit), in terms of region_type, min_len and max_len.
The article can be found here: https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>3215v1.full

I'll look into the details further, just wanted to have it here so it doesn't get left out.


MINIFICPP-2343 Fix quoting when CMake calls generateVersion.bat

If the source or build directory paths contained whitespace, CMake could not generate the agent_version.cpp file because the call to generateVersion.bat failed.
The two path parameters simply had to be quoted, but CMake calls the COMMAND directly without stripping ""s, so I had to use PowerShell to convert the path to a short path (which looks like C:\Users\JaneDo~1\src\minifi).
I have also added more verbose logging to make it easier to diagnose issues which may arise in the future.
https://issues.apache.org/jira/browse/MINIFICPP-2343

Thank you for submitting a contribution to Apache NiFi - MiNiFi C++.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with MINIFICPP-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically main)?

[x] Is your initial contribution a single, squashed commit?

For code changes:

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file?
[ ] If applicable, have you updated the NOTICE file?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI results for build issues and submit an update to your PR as soon as possible.

I'd prefer not relying on short paths that are just the legacy of the DOS era, and may disappear over time. Would it work if we added an extra set of quotes around the generateVersion.bat script path?

I'd prefer not relying on short paths that are just the legacy of the DOS era, and may disappear over time. Would it work if we added an extra set of quotes around the generateVersion.bat script path?

I have tried a zillion ways to quote the command.  In every case, either CMake stripped the quotes and gave me a 'C:/Users/Ferenc' is not recognized as an internal or external command, operable program or batch file error, or it kept the quotes and then went and tried to find a command with a quote in its name, which of course failed.
If you have a specific suggestion for how to quote the command, I'll try it (again, probably), but most of the advice I have found online was "don't quote the command, quote everything else".  I'm not a fan of DOS file names, either, but this was the only thing that worked.


Fix typos

Hi, thank you for this! :smiley:
I hope to improve a bit the package during the week-end, will release this change then. :)

Fantastic! Thanks :+1:

Haven't got time to work on this during this week-end, but I published 2.0.3 with your fixes nevertheless, cheers! :)


image: Add the ImageTag type to return more details than ImageStreamTag

ImageStreamTag has confusing semantics and returns only when a status
tag is set. Since creating a v2 image api would require significantly
more changes and ImageTag will change semantics (behavior around GET)
of API calls, it is more appropriate to create a new, simpler type that
will gradually supplant the old one.
This type includes spec, status, and image optional fields, rather than
purely the image and exploded spec status from the ImageStreamTag. This
allows pure status and pure spec tags to work equally well, to simplify
update logic, and to allow history to be viewed from client side
commands.
Implementation in openshift/openshift-apiserver#51

/lgtm


[BugFix] Fix transaction report trigger redundant publish tasks

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Multiple tablets belonging to the same partition will generate multiple identical TPartitionVersionInfo, which will trigger multiple redundant publish tasks in BE.
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto backported to target branch

[ ] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

run starrocks_fe_unittest

[FE PR Coverage Check]
:disappointed: fail : 1 / 14 (07.14%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/TabletInvertedIndex.java
0
10
00.00%
[259, 261, 262, 263, 265, 266, 267, 268, 269, 271]

:large_blue_circle:
com/starrocks/leader/ReportHandler.java
1
4
25.00%
[1020, 1024, 1025]

run starrocks_admit_test


Checked the code in KeyValueUtils, there is a method getSerializedSize
  public static int getSerializedSize(Cell cell, boolean withTags) {
    if (withTags) {
      return cell.getSerializedSize();
    }
    if (cell instanceof ExtendedCell) {
      return ((ExtendedCell) cell).getSerializedSize(withTags);
    }
    return length(cell.getRowLength(), cell.getFamilyLength(), cell.getQualifierLength(),
      cell.getValueLength(), cell.getTagsLength(), withTags);
  }

It seems that if withTags is true, we will always call cell.getSerializedSize directly. I guess this is why we use getSerializedSize there.
So let's add some comments to say that, the byte array is not used for serializaing, as it will copy all the fields such as column family, qualifier, etc, so it is not suitable to call getSerializedSize here, we should calculate it with all the fields.
Thanks.

Thanks for the review @Apache9 , I have added the comment.

Thanks for the review @Apache9 , I have added the comment.

Forgot to push it? Seems the PR remains unchanged...

@Apache9 Yes, I pushed to the wrong repo.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  1s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
5m 36s
master passed

+1 :green_heart:
compile
0m 55s
master passed

+1 :green_heart:
checkstyle
0m 31s
master passed

+1 :green_heart:
spotbugs
0m 56s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
5m  7s
the patch passed

+1 :green_heart:
compile
0m 51s
the patch passed

+1 :green_heart:
javac
0m 51s
the patch passed

+1 :green_heart:
checkstyle
0m 29s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
26m 35s
Patch does not cause any errors with Hadoop 3.1.2 3.2.2 3.3.1.

+1 :green_heart:
spotbugs
1m  1s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 14s
The patch does not generate ASF License warnings.

61m 21s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3904/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3904

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile

uname
Linux 587f6a4aefbe 4.15.0-153-generic #160-Ubuntu SMP Thu Jul 29 06:54:29 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / dcd622d702

Default Java
AdoptOpenJDK-1.8.0_282-b08

Max. process+thread count
86 (vs. ulimit of 30000)

modules
C: hbase-common U: hbase-common

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3904/2/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Implement batch runner for postgresql.

In general it's almost copy/paste of MySqlBatchRunner class.
Slight modifications:

changed queries to postgresql syntax
UPDATE
SET
FROM (  )
slighly modified regular expression with self updates.
ColumAlias is optional group now. Alias is not generated every time.
When in TransactionScope it means we have open transaction, really, it is.
Postgre treats such transaction as distributed, that's why opening new transaction fails.

Is there any other Relevant Commit or Fork Concerning Postgre SQL?
I have downloaded this  Commit Source , Needed to Change syntax from
(schema.table ) to ("schema"."table") and also ( columnName) to ("columnName") .
But otherwise it worked fine for a batch update i needed. Too bad it didn't got merged into the master branch , since a Postgresql handler is missing.

@KivraS, I'm not sure I've got your point. You are saying that the generated query by PgSqlBatch runner is incorrect. Right?
It seems you're using TableNames and ColumnNames instead of tablenames, i.e. your schema is not lowercase. Right?
If so, I will make additional changes to handle such cases.

"dbo" is the Schema name, "Transport" is the Table name.
For example. Previously the Runner Generated this sql.
select * from dbo.Transport where Id = 450
If you run this code on pg Admin on Postgresql 9.3 you get

ERROR:  relation "dbo.transport" does not exist
LINE 1: select * from dbo.Transport where Id = 450

Correct syntax to work is
select * from dbo."Transport" where "Id" = 450
I have managed to use your PsqlBatchRunner by altering some lines.
For example
Line 122
deleteCommand.CommandText = sqlBuilder.ToString().Replace("[","").Replace("]", "");
To
deleteCommand.CommandText = sqlBuilder.ToString().Replace("[","\"").Replace("]", "\"");
And similarly for column names and Update queries.

@KivraS Okay. Clear. Will fix my code. Thanks.


adding support for NSMutableURLRequest

As the subject line suggests, this pull request adds support for NSMutableURLRequest as the content in a PM::WebScreen.  This allows for the addition of headers to the request.

Thanks @cavalryjim !

You are welcome. Hopefully, I can contribute more in the future.
Thank you for this awesome gem.
Regards,
James
Sent from my iPhone

On Oct 29, 2015, at 23:57, Jamon Holmgren<EMAIL_ADDRESS>wrote:
Thanks @cavalryjim !
—
Reply to this email directly or view it on GitHub.


Alternate names for exercises

Thanks for this really helpful project!
I am trying to port a workout dump that uses exercises from ExerciseDB. Since most of the exercises have a different name here, it is quite difficult to do so. I suggest we add a new key to each exercise called alternateNames: string[]. This will allow others users to port their own data to this dataset more easily.
I can make a PR for the same.

Hey, glad it's been helpful! sounds totally reasonable, My hope was that we could use a single canonical name but in the real world that's probably a little too much to ask for 😆 out of interest have you got some examples?

I have added link to exercise, most exercises have a different name as to what this database has.

I see, happy for you to push up a PR!

Hello. Sorry, I never was able to do it since it would take too much effort. Since then I have created a mapping of around 200 exercises from exerciseDb to this project for my own project.
I have written down some instructions here. Maybe it will be helpful to someone!


Issue with predict_many in bioimageio.core using Python 3.10.12

Hi,
I am using bioimageio.core==0.6.9 with Python 3.10.12. When I call predict_many from bioimageio.core, I encounter the following error:
if not isinstance(inputs, collections.abc.Mapping):
   sample_id = str(sample_id)
   if "{i}" not in sample_id and "{i:" not in sample_id:
       # Your logic here

The error occurs because collections.Mapping has been deprecated in Python 3.10+. However, the issue is resolved when I modify the code to use collections.abc.Mapping.
It looks like this change is necessary for compatibility with Python>3.10 and later versions.

thanks for reporting this issue! 👍


Fuzz the alu 64/32 instructions

This tests checks that all the alu instructions have the same output for jit and interpreted.

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

Comparison is base (179a0f9) 88.34% compared to head (fb0f0eb) 89.43%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #552      +/-   ##
==========================================
+ Coverage   88.34%   89.43%   +1.08%
==========================================
  Files          24       23       -1
  Lines       10282     9931     -351
==========================================
- Hits         9084     8882     -202
+ Misses       1198     1049     -149

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

Looks good in general. However, we already have a bunch of fuzzers all with different interfaces, capabilities, etc. Would be nice to have a more structured approach instead of many individual ad-hoc solutions.

Looks good in general. However, we already have a bunch of fuzzers all with different interfaces, capabilities, etc. Would be nice to have a more structured approach instead of many individual ad-hoc solutions.

What do you have in mind?
I can extend this test to include branch tests and load/store tests, which would cover almost all of the instructions. I also feel this would be a useful addition for any future arm64 jitter.

Is there any updates on the status of this PR? I was looking forward to using this fuzzer.

I believe that PR is really good to ensure the consistency between the jitter and the interpreter and I thought it would be nice to even cover the SBFv1 instructions. There are instructions that the compiler does not emit today, but that the RPBF crate supports, so having this test ensures everything works in tandem if we were to enable their emission.

@Lichtso @LucasSte I've updated the PR, now all instructions are exercised.


Ensure static IP tests use different IPs

They can run in parallel, which could flake if they both allocate the same address at the same time

Change LGTM, but the tests are falling over.

It helps when you adjust the expected values to match the changes you made. Re-pushed.

@mheon Have you seen my code to generate random IP addresses for tests at:
https://github.com/containers/libpod/blob/master/test/e2e/checkpoint_test.go#L19
If this would help I could move this to common_test.go and use it everywhere IP addresses are specified during testing.

Oh, that would also work well. Avoiding hardcoded addresses for tests will definitely help.

Oh, that would also work well. Avoiding hardcoded addresses for tests will definitely help.

#3663

Closing in favor of #3663