IncompatibleAssign: Explicit cast between string pointer to enum-backing type pointer required

@jhendrixMSFT: Follow-up to https://github.com/Azure/autorest.go/issues/638
Usecase:
x-ms-parameterized-host is used with enum, declaration:
{
   "x-ms-parameterized-host": {
      "hostTemplate": "{geography}.atlas.microsoft.com",
      "parameters": [
        {
          "$ref": "#/parameters/GeographicResourceLocation"
        }
      ]
    },
   "parameters":  {
    "GeographicResourceLocation": {
      "name": "geography",
      "description": "This parameter specifies where the Azure Maps Creator resource is located.  Valid values are us and eu.",
      "in": "path",
      "required": true,
      "type": "string",
      "default": "us",
      "enum": [
        "us",
        "eu"
      ],
      "x-ms-enum": {
        "name": "GeographicResourceLocation",
        "modelAsString": true,
        "values": [
          {
            "value": "us",
            "description": "Used to access an Azure Maps Creator resource in the United States"
          },
          {
            "value": "eu",
            "description": "Used to access an Azure Maps Creator resource in Europe"
          }
        ]
      },
      "x-ms-parameter-location": "client"
    }
  }
}

Source:
https://github.com/propheel/azure-rest-api-specs/blob/dev-maps-Microsoft.Maps-2.0-versioning-test/specification/maps/data-plane/Common/preview/1.0/common.json
https://github.com/propheel/azure-rest-api-specs/blob/dev-maps-Microsoft.Maps-2.0-versioning-test/specification/maps/data-plane/Render/preview/2.0/render.json
Autorest invocation:
autorest --go --track2 --use=<EMAIL_ADDRESS>specification/maps/data-plane/Creator/readme.md --go-sdk-folder=../azure-sdk-for-go --multiapi
AutoRest code generation utility [cli version: 3.2.3; node: v14.17.0, max-memory: 2048 MB]
(C) 2018 Microsoft Corporation.
https://aka.ms/autorest
NOTE: AutoRest core version selected from configuration: 3.4.3.
   Loading AutoRest core      '/home/ambientlight/.autorest/@autorest_core@3.4.3/node_modules/@autorest/core/dist' (3.4.3)
INFORMATION: > Loading AutoRest extension '@autorest/go' (4.0.0-preview.22->4.0.0-preview.22)
INFORMATION: > Loading AutoRest extension '@autorest/modelerfour' (4.19.2->4.19.2)
Processing batch task - {"tag":"2.0-preview"} .
INFORMATION: > Loading AutoRest extension '@autorest/go' (4.0.0-preview.22->4.0.0-preview.22)
INFORMATION: > Loading AutoRest extension '@autorest/modelerfour' (4.19.2->4.19.2)

WARNING (PreCheck/CheckDuplicateSchemas): Checking for duplicate schemas, this could take a (long) while.  Run with --verbose for more detail.
[9.28 s] Generation Complete

Golang: 1.16.5
Platform: WSL2/Ubuntu 20.04
Autorest generates a valid constants.go#L140 however go complains on the assignment between string pointer and enum-backing type (Geography) pointer, unless explicit type cast is used.
func newConnection(geography *Geography, cred azcore.Credential, options *connectionOptions) *connection {
	if options == nil {
		options = &connectionOptions{}
	}
	// .. reducted
	hostURL := "https://{geography}.atlas.microsoft.com"
	if geography == nil {
		defaultValue := "us"

		// cannot use &defaultValue (value of type *string) as *Geography value in assignmentcompilerIncompatibleAssign
		geography = &defaultValue
	}

	// cannot use *geography (variable of type Geography) as string value in argument to strings.ReplaceAll(IncompatibleAssign)
	hostURL = strings.ReplaceAll(hostURL, "{geography}", *geography)
	return &connection{u: hostURL, p: azcore.NewPipeline(options.HTTPClient, policies...)}
}

Reference: connection.go
Should this be resolved by the addition of explicit casts?:
geography = ((*Geography)(&defaultValue))

hostURL = strings.ReplaceAll(hostURL, "{geography}", (string)(*geography))

To reproduce:
git clone -b experiment-scripts https://github.com/ambientlight/azure-rest-api-specs.git
cd azure-rest-api-specs
./gen_go.sh

Also occurring for:
https://github.com/ambientlight/azure-rest-api-specs/blob/5547eda058fa62078e929d746429f1c722755b93/specification/maps/data-plane/Route/preview/1.0/route.json#L303-L334
In
https://github.com/ambientlight/azure-sdk-for-go/blob/d97925c0dc464aa445862e5de620e41b80d389d5/services/preview/maps/1.0/route/route_client.go#L141

Fixed in generator version 4.0.0-preview.23


Two-way references and forms don't work together

Describe the current behavior
When we try to submit a form bound to a table which uses a two-way reference, we are answered an error. The devtools tells us it is a 403 error.
Steps to reproduce
You may download this Grist, import it and try to use the form: https://grist.incubateur.anct.gouv.fr/o/docs/api/docs/77NEs3HP29ttXX5hQuzhV8/download?template=false&nohistory=true
Otherwise, follow these steps:

Create a new document
Create a Towns table, with a name column and fill some data
Create a Contacts table, with a name column (text) and a town column (reference to the Town table)
Configure the town column and click on Add two-way reference and configure the two-way references with the Towns table
Make a new form, publish it, try to submit data using this form

We this error message:

There was an error submitting your form. Please try again.

Using the devtools, we see this error in response to the request:

Blocked by table update access rules

Describe the expected behavior
We may either:

at least display an intelligible error message, telling why the data could not be submitted, and guide how to fix that for the admin;
or prevent the case to occur
or better: have the form submitted

Where have you encountered this bug?

[ ] On docs.getgrist.com
[x] On a self-hosted instance

Instance information (when self-hosting only)

Grist instance:

Version: 1.2.0
URL (if it's OK for you to share it): https://grist.incubateur.anct.gouv.fr/o/docs/77NEs3HP29tt/forms-2-ways-references/p/1
Installation mode: kubernetes
Architecture: multi-workers

Browser name, version and platforms on which you could reproduce the bug: Firefox 128 ESR

I believe this issue translates to a low-level behavior of access rules, and a question whether it should be relaxed in general or if we need an exception specific to two-way references.
When you change a reference column C that's part of a two-way reference, that triggers another change to the reverse column R in the target table, where in fact multiple cells may get changed to stay in sync with the change in C. In general, forms rely on access rules to restrict an anonymous user's ability to only add a record. In particular, they don't allow anonymous users to change any cells in the target table.
There is a similar situation with formulas: a change to a cell that a user is allowed to edit may lead to calculated update to another cell that the user is not allowed to edit. We have an exception for that -- we call such actions "indirect" actions, and those bypass access rules.
So I see two ways to go about addressing this issue:

Consider the updates to a reverse reference column to be "indirect" actions. There is a slightly wider class of such triggered updates (similar to formulas but not the same). To consider them all indirect, you'd wrap the two lines at https://github.com/gristlabs/grist-core/blob/3f49c60e89d9f6bc9f56d610561f8289fdd73d19/sandbox/grist/useractions.py#L526 into self.indirect_actions() (and two other similar lines elsewhere).

This would have a more general effect: even without forms, if you permit someone to edit C, but not to edit R (its reverse column), they would be allowed to change R indirectly by changing C. Currently, they would need permission to change both.

We could keep the more general behavior strict (if R is a reverse column of C, then to be allowed to change either one, you need to give the user permission to edit both). Then we could address the forms issue by adding that second permission whenever we see that there is a two-way reference, and update it when two-way reference columns are added or removed.

I am leaning toward (1) as the more expected behavior generally. But any relaxing of access rules is a bit hard -- we need to think carefully of unexpected effects and unexpected use-cases that might rely on the current behavior. @paulfitz, what do you think?


New methods CellId.rangeMin and CellId.rangeMax

Hey @mast! Thanks for contributing again.
Looks great. We will need to update the README and TypeScript definitions accordingly, but I can do this.

Hey @mast! Thanks for contributing again.
Looks great. We will need to update the README and TypeScript definitions accordingly, but I can do this.


Rapid send() calls leads to dropped inputs and undefined behavior

move_relative(10,0)
move_relative(10,0)
move_relative(10,0)
move_relative(-30,0)

I expect this to result in no change to cursor position, but it moves the cursor to the left. Sending multiple inputs quickly without sleeps in between causes erratic behavior, for example, continuously moving the cursor left 100 times then right 100 times in a loop will not only move the cursor erratically but also lock out manual inputs.
I don't know if this is caused by the input buffer being overwritten, the mouse's polling rate, some limitation in the interception driver, or python. Is this an issue in C++?
On a related note, it's probably not a good idea to filter mouse movements and re-send() them. Curious if you've ever encountered this issue.

Hi @WHalcyon,
this doesnt have anything to do with interception, python, or any input buffers being overwritten.
This is simply due to a feature called Enhanced pointer precision that is enabled by default. It helps make mouse movement feel more 'natural' for a real human and changes the behavior of the mouse.
You can disable this setting by opening your windows mouse settings -> additional mouse options -> Pointer options and from there you disable it. When you turn it off you will see the results you initially expected, but its gonna feel terrible to use your mouse manually. As far as Im aware this setting only affects relative mouse movements, not absolute mouse movements.
If you need high precision relative movements, I'm sure you can find a way to disable the pointer precision setting programmatically.

On a related note, it's probably not a good idea to filter mouse movements and re-send() them

Why isnt that a good idea?

 It'll take a while to get used to.
Thank you. This has been causing me a ton of headaches and I've been using absolute coordinates because I assumed it was some kind of timing issue. I suspect this is going to resolve a ton of bugs I've had lol.

Update: It is possible to do it using SystemParametersInfo with SPI_SETMOUSE. Need to inquire whether that is an expensive opertion / whether the OS can properly handle the parameters swiftly changing - in which case we could just set the mouse acceleration to 0 before the relative movement.
Also, probably worth implementing the human movement using bezier curves using relative movements too as that is what it is natively when we capture our own mouse movements.

As it turns out this issue was screwing with my debug but doesn't affect raw input, so the problem is sadly application specific. Definitely a useful setting to check though.
I believe relative/absolute mode is device dependent, though I've yet to find a mouse that uses absolute. Unless you're keen on your inputs blending in, I prefer using whichever mode suits the use case since relying on getcursorpos is a pain.

The functionality is added now, see here.
Calls to move_to that use a bezier curve for human movement will automatically use relative movement now as well.


Check forks and apply patchs if needed

https://github.com/dwdyer/reportng/network

@juherr
From what I have seen, the below commits may need to be back-ported

https://github.com/ptsiakos77/reportng/commit/0a458d73871dfd88bcfbdef4a051e024dd4c0296
https://github.com/ptsiakos77/reportng/commit/aeb30697b4949983c14cdb3cdd33d3594c29b173

There are other commits, that I am not sure if we should be porting because it talks about including defect data into the test reports.

Ok :+1:

@juherr and @krmahadevan : Instead of checking all the forks, I recommend we check only the open pull request and merge if needed. What do you say?

Vimal
If we do that we are not going to be able to get all the changes that have
already been committed in the forks.  We also would need to take a look at
open PRs as well.￼
Thanks & Regards
Krishnan Mahadevan
"All the desirable things in life are either illegal, expensive, fattening
or in love with someone else!"
My Scribblings @ http://wakened-cognition.blogspot.com/
My Technical Scribbings @ http://rationaleemotions.wordpress.com/


If $9 is only for an hour or less, I think it's a scam

It didn't even let me build a complete project before all my tokens were used up. Why didn't I subscribe to the Claude plan?

check this https://github.com/stackblitz/bolt.new/issues/218#issuecomment-2400464667

Hey, @JKing-8! As we have been working on optimizing the tokens usage and accidentally triggered the limit on the accounts that weren't close to the limit. We've reverted the change so this should now be resolved and you should have your correct limit back. I'll close the issue, but in case a problem persist, please open a new one! Thank you for being an early adopter and for your patience as we're ironing out the experience on bolt!


Investigate cert-manager on dev00

Thread - https://gigantic.slack.com/archives/C5PDDBBDG/p1680087786534269
Summary:

SA for cert-manager on dev00 points to thunder role and not dev00.
cert-manager CM user-values points to dev00, but keeps on reverting to thunder role.

Findings:

Helm shows an issue with post-upgrade hook failure.

15              Tue Apr  4 12:08:08 2023        superseded      cert-manager-app-2.20.3 1.8.2           Rollback to 13
16              Tue Apr  4 14:30:27 2023        failed          cert-manager-app-2.20.3 1.8.2           Upgrade "cert-manager" failed: post-upgrade hooks failed: timed out waiting for the condition
17              Tue Apr  4 14:32:27 2023        superseded      cert-manager-app-2.20.3 1.8.2           Rollback to 15
18              Mon Apr 24 16:22:17 2023        failed          cert-manager-app-2.20.3 1.8.2           Upgrade "cert-manager" failed: post-upgrade hooks failed: timed out waiting for the condition
19              Mon Apr 24 16:24:18 2023        deployed        cert-manager-app-2.20.3 1.8.2           Rollback to 17

Upon further investigation by team cabbage, there's a failed job:
4m59s       Warning   FailedCreate             job/cert-manager-startupapicheck                                            Error creating: pods "cert-manager-startupapicheck-" is forbidden: error looking up service account kube-system/cert-manager-startupapicheck: serviceaccount "cert-manager-startupapicheck" not found

puru@giantswarm (gs-thunder-dev00-clientcert) ~ [0|1]> kubectl get sa cert-manager-startupapicheck -n kube-system
Error from server (NotFound): serviceaccounts "cert-manager-startupapicheck" not found

puru@giantswarm (gs-thunder-dev00-clientcert) ~> kubectl get jobs cert-manager-startupapicheck -n kube-system
NAME                           COMPLETIONS   DURATION   AGE
cert-manager-startupapicheck   0/1                      13h

puru@giantswarm (gs-thunder-dev00-clientcert) ~> kubectl describe job cert-manager-startupapicheck -n kube-system
Name:             cert-manager-startupapicheck
Namespace:        kube-system
Selector:         controller-uid=b394ca45-8e58-44ee-a1e0-c2b901da1077
Labels:           app=cert-manager
                  app.kubernetes.io/component=startupapicheck
                  app.kubernetes.io/instance=cert-manager
                  app.kubernetes.io/managed-by=Helm
                  app.kubernetes.io/name=cert-manager
                  application.giantswarm.io/team=team-cabbage
                  giantswarm.io/service-type=managed
                  helm.sh/chart=cert-manager-app-2.20.3
Annotations:      helm.sh/hook: post-install
                  helm.sh/hook-delete-policy: before-hook-creation,hook-succeeded
                  helm.sh/hook-weight: 1
Parallelism:      1
Completions:      1
Completion Mode:  NonIndexed
Pods Statuses:    0 Active / 0 Succeeded / 0 Failed
Pod Template:
  Labels:           app=cert-manager
                    app.kubernetes.io/component=startupapicheck
                    app.kubernetes.io/instance=cert-manager
                    app.kubernetes.io/managed-by=Helm
                    app.kubernetes.io/name=cert-manager
                    application.giantswarm.io/team=team-cabbage
                    controller-uid=b394ca45-8e58-44ee-a1e0-c2b901da1077
                    giantswarm.io/service-type=managed
                    helm.sh/chart=cert-manager-app-2.20.3
                    job-name=cert-manager-startupapicheck
  Service Account:  cert-manager-startupapicheck
  Containers:
   cert-manager-app:
    Image:      docker.io/giantswarm/cert-manager-ctl:v1.8.2
    Port:       <none>
    Host Port:  <none>
    Args:
      check
      api
      --wait=40s
    Requests:
      cpu:        10m
      memory:     32Mi
    Environment:  <none>
    Mounts:       <none>
  Volumes:        <none>
Events:
  Type     Reason           Age                  From            Message
  ----     ------           ----                 ----            -------
  Warning  PolicyViolation  29m (x14 over 13h)   kyverno-scan    policy require-requests-limits/autogen-validate-resources fail: validation error: CPU and memory resource requests and limits are required. rule autogen-validate-resources failed at path /spec/template/spec/containers/0/resources/limits/
  Warning  FailedCreate     99s (x140 over 13h)  job-controller  Error creating: pods "cert-manager-startupapicheck-" is forbidden: error looking up service account kube-system/cert-manager-startupapicheck: serviceaccount "cert-manager-startupapicheck" not found

Investigation completed; opening an issue on Cabbage board to upgrade cert-manager to upstream version.


Generate variables.tf for devops recipe

This is fixed by https://github.com/GoogleCloudPlatform/healthcare-data-protection-suite/pull/908


Correct diego_brain <-> CC Uploader entry

Corrections are to:

Port
App Layer Protocol
Security and Authentication

These should apply for the docs for:

2.0
2.1
2.2
2.3
and any future 2.x

[#160208021]

Thanks, @selzoc !


1.19 support

Minecraft 1.19 was released and I should probably update

open to PRs? might have a crack at it over the weekend

Yeah, feel free!
I'm away over the weekend and I didn't have any particular plans on when I wanted to get this done anyway

Released as v1.5.0+1.19!


Failed to fetch when installing

When i try to do the following step:
install all necessary piston dependencies
The following error comes up: (when running apt-get update)
root@piston:/# echo 'source /opt/.profile' >> /opt/.bashrc
root@piston:/# echo 'export HOME=/opt' >> /opt/.profile
root@piston:/# echo 'export TERM=linux' >> /opt/.profile
root@piston:/# echo 'export PATH=$PATH:/opt/.local/bin' >> /opt/.profile
root@piston:/# export HOME=/opt
root@piston:/# export TERM=linux
root@piston:/# sed -i 's//root//opt/' /etc/passwd
root@piston:/# sed -i \

's/http:\/\/archive.ubuntu.com\/ubuntu/http:\/\/mirror.math.princeton.edu\/pub\/ubuntu/' \
/etc/apt/sources.list

root@piston:/# apt-get update
Err:1 http://security.ubuntu.com/ubuntu bionic-security InRelease
Temporary failure resolving 'security.ubuntu.com'
Err:2 http://mirror.math.princeton.edu/pub/ubuntu bionic InRelease
Temporary failure resolving 'mirror.math.princeton.edu'
Err:3 http://mirror.math.princeton.edu/pub/ubuntu bionic-updates InRelease
Temporary failure resolving 'mirror.math.princeton.edu'
Reading package lists... Done
W: Failed to fetch http://mirror.math.princeton.edu/pub/ubuntu/dists/bionic/InRelease  Temporary failure resolving 'mirror.math.princeton.edu'
W: Failed to fetch http://mirror.math.princeton.edu/pub/ubuntu/dists/bionic-updates/InRelease  Temporary failure resolving 'mirror.math.princeton.edu'
W: Failed to fetch http://security.ubuntu.com/ubuntu/dists/bionic-security/InRelease  Temporary failure resolving 'security.ubuntu.com'
Im on a debian VPS, can someone help me out?

i think the shift of this issue is to do a test install for debian. will get back to this when i do.

i think the shift of this issue is to do a test install for debian. will get back to this when i do.

Issue is no networking is configured, see: https://wiki.debian.org/LXC#Networking for more information

Issue is no networking is configured, see: https://wiki.debian.org/LXC#Networking for more information

closing in favor of #75


replace jteeuwen/go-bindata with shuLhan/go-bindata

Prerequisites
N/A
Description
Replace github.com/jteeuwen/go-bindata/ with https://github.com/shuLhan/go-bindata
As per the discussion in this issue; https://github.com/jteeuwen/go-bindata/issues/165 /jteeuwen/go-bindata appears like an abandoned project.
And also from that discussion, it seems like the community is coalescing around https://github.com/shuLhan/go-bindata which is also actively maintained.
Of course jteeuwen/go-bindata still works!
Steps to Reproduce
Describe the steps required to reproduce the issue if applicable.
Love Up?
Please consider donating https://opencollective.com/apex-up/donate to help the project.
Just have a general question? Join us on Slack https://apex-slackin.herokuapp.com/

hmm seems to work fine, any benefit to switching? not worried about the maintenance part since it's already functional, but if it does a better job with something LMK!

I dont think there's any benefit to switching apart from the maintenance part.


Export works incorrect after deleting filters and search query

Steps for reproduce:
1. Use some filter or search for your entities list.
2. Delete your filter or search query after looking of new filtering results.
3. Do the Export operation for this list and look on the results.
Result:
The export results contains data from previous filtering operation.
Expected:
The export results should contains data from the current list view.

Just accidentally bumped into this issue, quite annoying indeed.
+1

Fixed by #835

I still got incorrect result when filters were deleted. This also happened when I sorted records. Data in list view was sorted properly, but the exported result (CSV) was sorted by ID (descending). Am I missing anything here?

@re1naldo: if I understand your scenario correctly, when removing a filter your exported data is still containing (or missing) the previously filtered data?
I just tried all scenarios (i.e. data is either added, removed or both after removing a filter) and the exported CSV was always containing the correct records.
You do have a case though about the sorting order which is not fully respected by the export feature. My suggestion to you is to open a new issue and it would be nice if you could also work on a solution!

After using ng-admin from master branch, the filtering feature works fine now. I will open different issue regarding data sorting.


Fixes #385 Private Sets

Ensures that if a source property was used in constructor that it will only be reapplied with an explicit resolver.

Solid work, thank you @JMPSequeira!

This is now available in the latest pre-release package.


Data type updates

For all data types except public health: Data, Guidelines and Services links and pages. Only Genomics & transcriptomics are complete though, the rest of the data types have skeletons.

I'm having a quick look at this now.. I'll see if I can fix the homepage problems and squash the commits a little.


Missing components.

I need some help figuring out which components to add to this project, so I'd really like some inputs.
I'll add issues for each components listed here to check on the progress, and so anyone can claim them.

Hey what about cards?


Have to importprivkey twice for it to work

As title says, importprivkey has to be ran twice from console in order for it to properly import. Tested on Windows 10 wallet, not Linux but can test that later.

It worked first try here on Linux, and a -HEAD build.
I don't have a Windows setup handy. Have you tried waiting a bit before issuing the command a second time? I get a callback a few ms later.


[RDKDEV-262] DeviceIdentification: syncup from metro version

Copyright scan failure
Commit: 476426b0dd1b56ac51ab75441164a4f752369e19
Report detail: https://gist.github.com/0aad0256851cbeb4e97e00820eb1c43f

@mhughesacn I have updated the NOTICE file as per your comment even though Copyright scan is failing.

Hi @pkumbh631: The team have pointed out that this is not the right way to use github/pull reqs.  They ask that you make the NOTICE change on the first pull request as a second commit (i.e. reopen pull req 2621) and close this one.  I will have to clear the copyright fail, which I'll do as soon as I see it. Thanks!


Profile page added

Fixes #68
Changes: Profile page added containing change password, change email and other details
Screenshots for the change:

@oindrilabhadra78 resolve the conflicts.

@somenath1435 can't we have better ui design for the profile page. Its not looking prominent to me.


currentSrc

Problem getting VAST plugin to work with youtube plugin
currentSrc is used by the videojs-ads plugin several times to keep track of the preroll video.
Which means that the youtube plugin doesn't implement the currentSrc api method.
original issue: https://github.com/theonion/videojs-vast-plugin/issues/15#issuecomment-41570950

@AramZS  : is you have  try use VAST plugins and. Youtube plugins work together?


yarn fails with "undefined exported symbol"

I need to bump the kotlin tree-sitter version (I'm working on adding support to Cursorless). I just cloned this for the first time, along with a fresh clone of emscripten (which I did activate) and I'm getting the following build failure:
~/projects/vscode-parse-tree$ yarn
yarn install v1.22.22
$ make web-tree-sitter
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100 2902k    0 2902k    0     0  3340k      0 --:--:-- --:--:-- --:--:-- 3340k
emcc: error: undefined exported symbol: "__ZNSt3__212basic_stringIcNS_11char_traitsIcEENS_9allocatorIcEEE9__grow_byEmmmmmm" [-Wundefined] [-Werror]
make: *** [Makefile:59: vendor/web-tree-sitter/README.md] Error 1
error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

I'm doing all this on WSL with an LTS installation of node/npm and a fresh installation of yarn. I see a similar closed bug but I'm not sure what the fix is: https://github.com/cursorless-dev/vscode-parse-tree/issues/30

Some more investigation indicates that I can reproduce this within a separate tree-sitter clone with ./script/build-wasm, but only if I check out the commit listed in in tree-sitter-version. tree-sitter builds fine at HEAD.
If I update tree-sitter-version here, I hit issues later in the build:
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack at ChildProcess.<anonymous> (/home/jwstout/.nvm/versions/node/v20.14.0/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:209:23)
gyp ERR! System Linux <IP_ADDRESS>-microsoft-standard-WSL2
gyp ERR! command "/home/jwstout/.nvm/versions/node/v20.14.0/bin/node" "/home/jwstout/.nvm/versions/node/v20.14.0/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /home/jwstout/projects/vscode-parse-tree/node_modules/tree-sitter-clojure

I updated to the latest Ubuntu LTS (22.04.04, Jammy) on WSL, which includes a later Python version (3.10), and the build completes, but only if I update tree-sitter-version (otherwise it fails with the originally reported error). So I guess we could bump the tree-sitter-version and close this out? Not sure what testing needs to be done.

 I did a little more research to understand why I wasn't able to compile with the current tree-sitter-version. It turns out that each tree-sitter version is only guaranteed to work with a specific emscripten version. If you look at the tree-sitter repository at the pinned version, you can see that version is currently 3.1.37:
https://github.com/tree-sitter/tree-sitter/blob/6bbb50bef8249e6460e7d69e42cc8146622fa4fd/cli/emscripten-version#L1C1-L1C7
Hence, what I did was run:
emsdk install 3.1.37
emsdk activate 3.1.37

And now it builds.
I'm not sure what you want to do with this bug. At the very least it seems like a good idea to update the README to explain to users how to find the right emscripten version and activate it.

I'm doing the upgrade now; been a battle, but I'm pretty close. Stay tuned

(see https://github.com/cursorless-dev/vscode-parse-tree/pull/84)

Thanks! Still would be good to document the emsdk stuff better so future contributors don't run into the same problems I did. I can send a PR to update the readme if you want.

That would be awesome

ok we're now on tree-sitter version https://github.com/tree-sitter/tree-sitter/commit/604d38e6b327ed33877e1285680b505b9484a71c as of vscode-parse-tree version 0.31.0

fwiw looks like that emscripten version file has moved. Here's where it is for the version of tree-sitter we're currently on https://github.com/tree-sitter/tree-sitter/blob/604d38e6b327ed33877e1285680b505b9484a71c/cli/loader/emscripten-version
also just wanted to confirm whether this issue is fixed for you?

Thanks for the heads up about the file move. Yes this issue is fixed for me. Feel free to close now or after I send the docs improvement when I find time.

Great. Sure, I'll leave it open until docs fix lands


CSS

CSS
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I view the study guide
THEN I see a dark blue header and footer, and four boxes with a shadow

Completed :) didn't close this at the end of the css thing lol


Feature/a1 mtdc@134 Custom MonitorExtenstion to log with java.util.logging

Custom MonitorExtension that allows to forward logs to java.util.logging (instead of using deafult ConsoleMonitor from EDC)
With that extension the connector logs are also available in App Insights Logs

Because both consumer and provider connectors are going to use this extension we introduced a new prs-connector-commons module to store common extensions there
The opportunity to contribute the extension to the EDC will be discussed, then this extension could be published as an maven artifact and the commons module won't be needed anymore
Logging configuration is in logging.properties file which is passed as a jvm parameter (-Djava.util.logging.config.file)

Very nice testing!

Thanks


Run dashing behind proxy

Hi,
I need to run the dashing server behind a proxy, so that all the ruby scripts in /jobs obtain their data from the internet through through that proxy. I set the http_proxy systemvariable in /etc/environment successfully but ruby does not seem to use it. Instead I foud some fancy code how to tell NET:HTTP to use a proxy.  Is it really true, that i need to change that in every script or am i missing some trick? I would save lots of time if there would be a simple way to tell ruby to use $http_proxy globally.......
Best regards
Luke

http://stackoverflow.com/questions/6868507/automatically-adding-proxy-to-all-http-connections-in-ruby
This is not a Dashing issue... Please close.


Hide profession specific items when using a character without the profession

I searched for "engineer", didn't find anything, and was about to make a new post. Luckily I saw this, and will instead agree!
Yes please. "this character only" should, if possible, check for engineering, so my mass-purchasing stops buying all those helms I can't use!
please and thank you!

This is no longer relevant. As any character can learn any BOE item.


v3: ionic info outputs ionic version as ionic, ionic-angular as Ionic Framework

Maybe "ionic" could also be changed to "Ionic CLI" to avoid confusion with super newb users that can'T distinguis between Ionic (CLI) and Ionic (Framework)?
In the forum every few days there are super confused users that use ionic info after they installed stuff and are confused they are not seeing the version they want to, expect to etc.
Same for ionic --versionby the way, having it output "Ionic CLI version is: 3.0.0-beta10" would help a lot.

Great input, thanks.

I think we're going to leave ionic --version alone--it serves its purpose and generally for beginners we ask for ionic info anyway. But the next version of the CLI will have more descriptive names (ionic -> Ionic CLI, cordova -> Cordova CLI). Thanks!


[QA] Avatar instantiation number goes up after destroying at random

Explorer Alpha build version:
v0.3
Issue Description:

Steps to reproduce:

Instantiate 10 avatars
Instantiate 500 more (510 at this point, it doesn't have to be 500)
Performed some actions (running, reaching walls etc.)
Clicked on randomize wearables
Destroy avatars at random

Expected behaviour:
The total number of avatars correspond to the total amount minus the ones that have been destroyed
Current behaviour:
The total number goes up
Reproduction rate:

[ ] < 10%
[ ]  < = 25%
[X]  50%
[ ]  < = 75%
[ ]  100%

Severity:

[ ] Cosmetic
[ ] Minor
[X] Medium
[ ] High
[ ] Critical
[ ] Blocker

Priority:

[ ] High
[X] Medium
[ ] Low

Platforms / OS:

[ ] Windows 11
[X] Windows 10
[ ] Mac Sonoma
[ ] Mac (specify the version on Additional Notes)

Evidence:

Additional Notes:

Solved in newest builds


[CANNOLI-193] Note dataset type in CLI command docs.

Fixes #193

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/cannoli-prb/187/
Test PASSed.


Suggest using alternative names for reserved word variants

This PR displays an additional message for improper names when they are variants of a reserved word. A name is considered a variant of a reserved word if, when stripped of underscores and converted to a lowercase string, the name matches a reserved word.
Additionally, substitutions are suggested for type (typ) and struct (structure). More substitutions can easily be added.
Example:
don't use underscores in Go names; var type_ should be type (to avoid conflict with this reserved word, choose 'typ')

I signed it!

#393

Do not merge - I recall an error in a printf (missing "%s" on line 607 I think; "%s %s should be %s" should be "%s %s should be %s%s") and I don't have time to fix it


Formal internal review of Exchange Online baseline

🗣 Description

This PR is designed to facilitate a formal review of the revised Exchange Onlinebaseline from CISA SCuBA principals and the Scuba development team. Following this review the baseline should be in a state for sending to the CISA agency formal review process.
💭 Motivation and context

A broad review by both the development team SMEs and the CISA SCuBA principals ensures a high quality update in line with agency goals and consistent policy format and styling.
✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All future TODOs are captured in issues, which are referenced
in code comments.
[x] All relevant type-of-change labels have been added.
[x] All relevant repo and/or project documentation has been updated
to reflect the changes in this PR.
[x] Tests have been added and/or modified to cover the changes in this PR.
[x] All new and existing markdown tests pass.

✅ Pre-merge checklist

[ ] Feature branch has been rebased against base branch with conflicts resolved

✅ Post-merge checklist

[ ] Ensure Exchange Online related markdown tests pass on merged branch

17 Microsoft Purview Audit
The following are suggestions / questions related to the audit policy section.

Suggest changing the name of the section to something more generic like Audit Logs or something like that instead of naming it after Microsoft's feature name.
"Activity by users with E5 licenses is logged for one year." Did you mean to say with E5 licenses is retained for one year?
MS.EXO.17.3v1 audit retention policy - In theory I understand why we have this policy, but I'm struggling with it because it doesn't seem that there is a single, standard way to "configure" it, like our other hardening policies. Technically an agency could offload their logs to a SIEM and then the SIEM would be responsible for retaining the logs per the OMB period - is that correct? If that is true, then an agency wouldn't need to follow our implementation instructions in Defender to configure and Audit Retention Policy within M365 because they would be handling that in a separate system. I think this policy might fit into the category of "the user can implement the requirement with an alternative system", which in this case would be a separate log retention system. Not sure how my discussion here changes our policy but I wanted to discuss it nonetheless.
"Audit logs SHALL be maintained for at least the minimum duration dictated by OMB M-21-31 (Appendix C)]" - Appendix C contains a lot of different types of logs and it is not obvious which ones correspond to the log types described within baseline section 17. Consider pointing the user to the specific log types which are relevant. I know that we mention the "Unified Audit Log" when discussing OMB in policies 17.1 and 17.2 but I think 17.3 should also make it clear what to look for in the OMB memo retention table.
Question about audit log retention in general. Section 17 seems to be scoped to the Unified Audit Log. What about all the other log types that we reference in AAD section 4 Centralized Log Collection? For completeness, do the baselines need an audit retention policy for those AAD logs as well?
The implementation section is not included in this baseline but it is included in Teams for policies that point to Defender. In Teams, the implementation section it is simply a sentence instructing the user to review the relevant Defender section with a hyperlink (see screenshot below). I recommend to do the same thing with EXO. I personally think it is better to have an Implementation section even if it is just to point the user to the relevant Defender section.

MS.EXO.1.1v1 instructions
In the instructions for the automatic email forwarding policy, it only references the "Default" domain policy. ScubaGear on the other hand, checks the Default and any other policies to ensure that they all have Allow Automatic Forwarding turned Off. I recommend augmenting the instructions to include the possibility that the user may have multiple policies in addition to just the Default. Maybe we just need to say, "repeat these instructions for each Remote Domain policy" or something like that.

MS.EXO.2.1v1
Recommend updating the policy name to be more descriptive of what the actual hardening configuration is - maintaining a list in an of itself isn't really what the intent is here - needs more details. How about something like: "A list of approved IP addresses for sending mail SHALL be maintained in agency's DNS SPF records"?
I also recommend updating the instructions for this policy to make it more obvious what the user is supposed to configure. For example, it instructs the user to "Identify any approved senders specific to your agency." Then what is the user supposed to do with the list of approved senders?
Also the word "senders" in the 2.1 instructions is not clear. Is that supposed to be "IP addresses" as referenced in the policy?
When I read the MS.EXO.2.2v1 instructions I get the impression that it covers both policies. In fact policy 2.1 is referenced in those instructions. Maybe it would be better to simply have a single instructions section where both policies 2.1 and 2.2 are configured together. Perhaps our standard approach of making each instruction set atomic doesn't apply in this case?

MS.EXO.3.1v1 DKIM SHOULD be enabled for any custom domain
Why is this a SHOULD policy whereas the SPF and DMARC policies are SHALL? The respective DHS BOD includes DKIM as well.

MS.EXO.4.3v1
This is a policy that is truly government specific because it instructs to set the value to<EMAIL_ADDRESS>which clearly won't apply to non FCEB organizations. I'm wondering if we should create a special note field to call this out, or simply include it in the notes.
MS.EXO.4.4v1
This is a nitpick. This policy "An agency point of contact SHOULD be included for aggregate and failure reports" is slightly different from 4.3 because 4.3 does not include the words and failure reports. If there is a reason that they differ, then do nothing. If they are supposed to be the same, then go ahead and make the language identical for consistency.

MS.EXO.5.1v1 SMTP AUTH SHALL be disabled in Exchange Online.
Not sure if the words "Exchange Online" are needed in the policy name. It might be better to remove them, unless we have standardized the usage of "in Exchange Online" with all the other policies and there is some reason for it there.
MS.EXO.5.2v1 SMTP AUTH MAY be enabled on a per-mailbox basis as needed.
To re-iterate my analysis of these types of policies, here you go. I believe that this policy is really more of an exception of policy 5.1 and not really its own hardening policy. Also the only reason I would consider keeping any references to SMTP auth is if we know of a way to configure it "securely" for specific exception mailboxes. If not, I think we should remove all references to it including any respective instructions because I don't think it contributes to the security of the tenant, at least not in any obvious way.

MS.EXO.6.1v1 and MS.EXO.6.2v1
Please read all of the comments before making any changes because there are some interdependencies.

I'm wondering if these policies should be rephrased as "Contact folders SHALL only be shared with approved domains" instead of "Contact folders SHALL NOT be shared with all domains". When I go to configure the policy in the admin portal, if I select "Share your Contacts folder", it forces me to enter a list of approved domains (see screenshot below). So technically I'm not sure there is a way that I can just turn Off contact folder sharing, however look at my third comment bullet below as well because I did potentially find another way I could configure the admin portal and still be compliant with the policy.
For 6.1 instructions step 5, I'm wondering if we augment the instructions to say "For all sharing rules under all existing policies where "Share your contacts folder" is selected, ensure Sharing with all domains is not selected". Same tweak for 6.2 instructions but it would say where "Share your calendar folder" is selected. See the respective checkboxes in the screenshot below.

I was able to also configure the settings as per the screenshot below by selecting Sharing with all domains but then deselecting both Share your contacts folder and Share your calendar folder. I believe this might achieve the purpose of the baseline policy but I haven't hands-on tested it. If this does in fact achieve the purpose of the policy, then disregard my first comment bullet. Maybe we need to augment the instructions to include this configuration?

MS.EXO.7.1v1 instructions

Step 3 assumes that the user doesn't already have an existing rule.
After step 12 I recommend adding what you want the Match sender address in message value to be set to. The default for me was the value Header.

After I created the rule as described in the instructions, it shows as Disabled in the admin portal. Could there be steps missing in the instructions? For my test rule, I had to set a specific activation time first, and then go back in and set the Enable or disable rule field to Enabled.

8. Data Loss Prevention Solutions
I'm wondering if we can combine policies 8.1 and 8.2 into a single policy. I don't see 8.1 as being specific hardening guidance - rather it states a high level requirement, that a DLP system must be used - this deviates from other policies which are specific. Maybe we can have a single policy that reads something like:
A DLP solution SHALL be used that protects PII and sensitive information, as defined by the agency. At a minimum, the sharing of credit card numbers, Taxpayer Identification Numbers (TIN), and Social Security Numbers (SSN) via email SHALL be restricted.
Also, the second sentence in policy 8.1 is not specific: "The selected DLP solution SHOULD offer services comparable to the native DLP solution offered by Microsoft." What specific services is this policy setting as the requirement? Is the protection of PII credit cards, TINs and SSNs via email, the only requirements? Are there more?

MS.EXO.9.1v1
Here's a suggested update to the policy.
There SHALL be an email filtering policy that filters attachments for risky file types. The selected filtering solution SHOULD offer services comparable to Microsoft Defender's Common Attachment Filter.
Also:

The policy mixes SHALL and SHOULD which makes the criticality ambiguous, at least to me. Is the second sentence really a SHOULD?
The second sentence for 9.1 seems like it would apply to 9.2 and 9.3 as well. Maybe it is better to move that sentence to the intro paragraph if it applies broadly to all these policies since they all see to be about attachment file types and filtering?

MS.EXO.10.3v1
Suggestion wording update to the policy for clarity below:
Email scanning SHOULD be capable of retroactively reviewing emails after they have already been delivered.
Also, I would consider changing the criticality to a SHALL since this seems like a really important feature to me.

License Requirements
I noticed that some policies do not contain the license requirements section (e.g. section 11 Phishing Protections). Is this because those policies point to Defender and the Defender baseline is responsible for stating those requirements?

MS.EXO.11.1v1 instructions
The hyperlink to Defender points to the wrong section. I would think it should point to the implementation instructions for section 2 Impersonation Protection.

Section 11 Phishing Protections licensing confusion
The following sentences were confusing: "Microsoft Defender has capabilities for all of these phishing protections. With the exception for impersonation protection, these features are available with Exchange Online Protection (EOP), which is included in all Microsoft 365 subscriptions that contain Exchange Online mailboxes."

What exactly are we trying to point out here? That if you want Impersonation Detection you need to have Defender? Consider rewording as:

These feature described in this policy section are available with Exchange Online Protection (EOP), which is included in all Microsoft 365 subscriptions that contain Exchange Online mailboxes. The only exception is impersonation protection, which is included if you subscribe to Microsoft Defender for Office 365.

Also, I would update all references to "Microsoft Defender" to be specific to the Defender flavor relevant to this baseline.

Not sure, but maybe this content belongs in the License Requirements section?

12. IP Allow Lists Implementation
With the exception of the DKIM policy, this is the only section in the entire baseline that instructs the user to go to the Microsoft 365 Defender Portal to implement the policies. Should this implementation be moved to Defender, like we did for other policies?

MS.EXO.12.1v1 instructions and MS.EXO.12.2v1 instructions
Step 8 for 12.1 and step 7 for 12.2 seem to be duplicates of each other.

13. Mailbox Auditing

"Some actions performed by administrators, delegates, and owners are logged automatically. " Not sure about the purpose of this sentence. Logged automatically by what? What does that have to do with the Mailbox Auditing feature?

MS.EXO.14.3v1
Policy states:
Allowed domains SHALL NOT be added.
Shall not be added to what? Be specific.

Link Protection

Missing period between sentence. "Microsoft Defender includes link scanning capabilities Use of Microsoft Defender is not strictly required for this purpose;"
MS.EXO.15.2v1 "Direct download links SHOULD be scanned for malware." Since section 10 is dedicated to malware, what is the reason that 15.2 is in this section? I'm fine with it - I just want to know if there is a difference between the malware policies in section 10 versus other sections.

16. Alerts

This section seems to be heavily influenced by Defender. It almost seems like it not necessarily generic (i.e. could be implemented by another product). I'm not saying that it cannot be implemented by something else - just that it appears as such based on the language used. For example the first paragraph starts with "Microsoft Defender includes several prebuilt alert policies, many of which pertain to Exchange Online." Nothing else in the intro states that a competing product can be used like in some of the other EXO policies. Policy 16.1 contains a specific list of alerts, including one named "Suspicious Connector Activity" which seems like it might be Defender-specific.
The instructions section has some invalid links to the Defender document.

I have completed my review of this baseline. Thanks for taking my questions and suggestions. I plan to go back over any specific policies that you want me to re-review (if necessary) and also help adjudicate any open questions if necessary by the end of this week. Nice work and let me know if there are any high priority discussion topics to close out on this baseline.


Issue 718: Add UUID type

Resolution for issue 718: Introduce uuid type

Added type and default value to QueryBuilder
Extended UUID datatype to conform to SQLDataType

Tested using local executable on PostgreSQL, mySQL and SQLite.
The default value of varchar(36) will allow storing of a UUID represented as a string as supported by the PostgreSQL uuid type.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Matthew Kilner seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #125 into master will decrease coverage by 0.04%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master     #125      +/-   ##
==========================================
- Coverage   85.25%   85.21%   -0.05%
==========================================
  Files          43       43
  Lines        3575     3578       +3
==========================================
+ Hits         3048     3049       +1
- Misses        527      529       +2

Flag
Coverage Δ

#SwiftKuery
85.21% <33.33%> (-0.05%)
:arrow_down:

Impacted Files
Coverage Δ

Sources/SwiftKuery/SQLDataType.swift
92.3% <0%> (-7.7%)
:arrow_down:

Sources/SwiftKuery/QueryBuilder.swift
89.65% <100%> (+0.36%)
:arrow_up:

Continue to review full report at Codecov.

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

This is a breaking change (modifies a public enum) so will need to be Swift-Kuery 2.0...

You are technically correct, but in practice this enum is only used by the various database plugins (not Kuery users), and is never switched over anyway. So I'm not sure if this is a practical concern. @EnriqueL8?

It's a concern if someone has implemented their own plugin under Swift-Kuery... which is quite unlikely.  So I think it is safe to say that this could be a minor, since tagging a major for such a small change doesn't make sense...


Scatter3d with StyleOption.Mode.Lines produces no output

Description
I try to use Scatter3d to plot 3d wireframes:
// baseline markers only that is working:
open FSharp.Plotly
let x = [19; 26; 55;]
let y = [19; 26; 55;]
let z = [19; 26; 55;]
Chart.Scatter3d(x,y,z,StyleOption.Mode.Markers, Color="rgba(127,127,127)", Dash = StyleOption.DrawingStyle.Solid, Width=4)
|> Chart.withX_AxisStyle("my x-axis")
|> Chart.withY_AxisStyle("my y-axis")
|> Chart.withZ_AxisStyle("my z-axis")
|> Chart.withMarkerStyle(Size=4)
|> Chart.withSize(800.,800.)
|> Chart.Show
Changing StyleOption.Mode.Markers to  StyleOption.Mode.Lines_Markers
The markers are now connected with solid grey lines as expected but
the axis labels and descriptions are gone.
Changing StyleOption.Mode.Markers to StyleOption.Mode.Lines
No output anymore, again the axis labels and descriptions are gone.
What is wrong here?

I published a new version and made a NuGet package (version 1.0.2). It now includes 3d-Surface-charts. I tried your code and it works with the new version. Please let me know it it works out after updating.
Kind regards,
Timo

Hi Timo, thanks for the update.
I looked at it and I think there are more issues:

Mode.Lines does not work at all
Modes.Lines_Markers does not show the axis description nor the axis tick labels anymore

Hi Daniel,
thank you for reporting that fast... I tried it again on my site and it worked out. It might be a browser related problem. Would you be so kind and have a look at the official plotly page, if you see the first 3d-line plot on top? https://plot.ly/javascript/3d-line-plots/
This picture is done using the Internet Explorer 11. Using Fsharp Plotly WPF browser control might also be an option for you...

Yes you are right is is a Chrome issue. It works well with IE and Firefox. Still trying to find a solution for it.


Truncate long messages from the Debug Sidebar

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Proposed changes
As described here, truncates long topic and string messages from the Debug Sidebar with pure CSS solution.
Preview:

Checklist

[x] I have read the contribution guidelines
[x] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[x] I have run npm run test to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

I'm happy for the topic to be truncated - less so for payload - unless there is a click to reveal all.

Unfortunately no, the expendable box is only shown if the string contains a line break (among other things).
I made a second version of this PR that uses CSS for the topic and a coded version for the string. In this second version, the expendable box is showed as soon as the string exceeds 80 characters (to do: discuss the max size)

So maybe split this PR into two - one for the topic which we all seem to agree on and one for payload which needs more thought :-)

Preview of this second version:

I set 80 characters as max length.

Nice - that seems about right to me.

Or add the clipboard icon to topic div (on mouseover)?

you can of course change the debug to show complete message and you can then see the topic in the main part of the debug - and expand it on click to see all of it. The original reason for topic to be there was for good old MQTT and I guess sometimes the topic tree may be large but in general they aren't as long as when used for SQL queries etc....

Lets split out handling topic and message properties.
I have no problem with the implementation for message properties - we can move forward with that.
My concern is the handling of topic. We append the property id and type info to the topic string - so by truncating it, that info is no longer available. A 'copy' button wouldn't help.
I personally think we can afford to have a couple lines worth of text visible, we just want to avoid overly long pieces of text. That suggests a pure CSS approach isn't appropriate here - that it should be truncating the text when building up the value to be shown:

So the above would end up something like the following. I think the max length should be enough to fill a couple lines of the default sidebar width
abcdefghiklmnop... : msg.payload : Object

We can then add a tool tip to show


HIVE-28191. Upgrade Hadoop Version to 3.4.0.

What changes were proposed in this pull request?
JIRA: HIVE-28191. Upgrade Hadoop Version to 3.4.0.
The 3.4.0 version of Hadoop has been released, and we aim for this version to be compatible with other communities. Hive is a crucial component among them. We will attempt to upgrade Hive's Hadoop dependency to 3.4.0.
Why are the changes needed?
HADOOP-17524 removed EventCount to facilitate the removal of dependencies on log4j 1.x. I referenced HADOOP-17524 and removed a portion of the code accordingly. We can refer to this link https://github.com/apache/hadoop/pull/2909 to view the changes.
Does this PR introduce any user-facing change?
no
Is the change a dependency upgrade?
Upgrade Hadoop Version to 3.4.0.
How was this patch tested?
I tried compiling the code locally and it passed, expecting it to pass CI verification.
[INFO] Reactor Summary for Hive 4.1.0-SNAPSHOT:
[INFO]
[INFO] Hive Storage API ................................... SUCCESS [  7.493 s]
[INFO] Hive ............................................... SUCCESS [  1.748 s]
[INFO] Hive Classifications ............................... SUCCESS [  0.356 s]
[INFO] Hive Shims Common .................................. SUCCESS [  2.004 s]
[INFO] Hive Shims 0.23 .................................... SUCCESS [  3.960 s]
[INFO] Hive Shims ......................................... SUCCESS [  1.301 s]
[INFO] Hive Standalone Metastore .......................... SUCCESS [  2.636 s]
[INFO] Hive Standalone Metastore Common Code .............. SUCCESS [ 26.648 s]
[INFO] Hive Common ........................................ SUCCESS [  7.782 s]
[INFO] Hive Service RPC ................................... SUCCESS [  1.406 s]
[INFO] Hive Serde ......................................... SUCCESS [  4.780 s]
[INFO] Hive Metastore ..................................... SUCCESS [  1.872 s]
[INFO] Hive Vector-Code-Gen Utilities ..................... SUCCESS [  0.189 s]
[INFO] Hive Parser ........................................ SUCCESS [  7.348 s]
[INFO] Hive UDF ........................................... SUCCESS [  1.538 s]
[INFO] Hive Llap Common ................................... SUCCESS [  2.677 s]
[INFO] Hive Llap Client ................................... SUCCESS [  2.632 s]
[INFO] Hive Llap Tez ...................................... SUCCESS [  2.597 s]
[INFO] Hive Metastore Server .............................. SUCCESS [ 17.629 s]
[INFO] Hive Query Language ................................ SUCCESS [02:55 min]
[INFO] Hive TestUtils ..................................... SUCCESS [  0.236 s]
[INFO] Hive Llap Server ................................... SUCCESS [  7.339 s]
[INFO] Hive HPL/SQL ....................................... SUCCESS [  3.640 s]
[INFO] Hive Service ....................................... SUCCESS [  6.208 s]
[INFO] Hive Accumulo Handler .............................. SUCCESS [  4.117 s]
[INFO] Hive JDBC .......................................... SUCCESS [ 21.490 s]
[INFO] Hive Beeline ....................................... SUCCESS [ 17.430 s]
[INFO] Hive CLI ........................................... SUCCESS [  4.026 s]
[INFO] Hive Contrib ....................................... SUCCESS [  2.230 s]
[INFO] Hive Druid Handler ................................. SUCCESS [01:15 min]
[INFO] Hive HBase Handler ................................. SUCCESS [  8.839 s]
[INFO] Hive JDBC Handler .................................. SUCCESS [  3.081 s]
[INFO] Hive HCatalog ...................................... SUCCESS [  0.772 s]
[INFO] Hive HCatalog Core ................................. SUCCESS [  4.615 s]
[INFO] Hive HCatalog Pig Adapter .......................... SUCCESS [  3.298 s]
[INFO] Hive HCatalog Server Extensions .................... SUCCESS [  2.796 s]
[INFO] Hive HCatalog Webhcat Java Client .................. SUCCESS [  3.283 s]
[INFO] Hive HCatalog Webhcat .............................. SUCCESS [  9.194 s]
[INFO] Hive Streaming ..................................... SUCCESS [  3.258 s]
[INFO] Hive Llap External Client .......................... SUCCESS [  2.198 s]
[INFO] Hive Shims Aggregator .............................. SUCCESS [  0.096 s]
[INFO] Hive Kudu Handler .................................. SUCCESS [  5.407 s]
[INFO] Hive Kafka Storage Handler ......................... SUCCESS [  4.510 s]
[INFO] Hive Packaging ..................................... SUCCESS [  01:12 h]
[INFO] Hive Metastore Tools ............................... SUCCESS [  0.057 s]
[INFO] Hive Metastore Tools common libraries .............. SUCCESS [  6.403 s]
[INFO] Hive metastore benchmarks .......................... SUCCESS [ 18.470 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  01:20 h
[INFO] Finished at: 2024-04-09T20:05:37+08:00
[INFO] ------------------------------------------------------------------------

I have a question. I'm trying to run unit tests locally, but I keep encountering errors like Module 'hive-standalone-metastore-common' production: java.lang.OutOfMemoryError: GC overhead limit exceeded. Is there a good way to solve this issue? My IDEA memory is already set to 12GB.
@ayushtkn Do you have any good solutions to solve this issue?

http://ci.hive.apache.org/blue/organizations/jenkins/hive-precommit/detail/PR-5187/5/tests
I find that hbase-handler related tests are failed. I think the failures  are caused by the incompatibility between hbase and hadoop3.4.0. Check HIVE-27651 which i have done about the hbase-hadoop incompatibility. This needs to make sure hbase itTest can run in Hadoop3.4.0 and maybe need hbase community to provide a compatible package with hadoop3.4.0.
ava.io.IOException: Shutting down
	at org.apache.hadoop.hbase.MiniHBaseCluster.init(MiniHBaseCluster.java:257)
	at org.apache.hadoop.hbase.MiniHBaseCluster.<init>(MiniHBaseCluster.java:110)
	at org.apache.hadoop.hbase.HBaseTestingUtility.startMiniHBaseCluster(HBaseTestingUtility.java:1185)
	at org.apache.hadoop.hbase.HBaseTestingUtility.startMiniHBaseCluster(HBaseTestingUtility.java:1232)
	at org.apache.hadoop.hive.hbase.HBaseTestSetup.setUpFixtures(HBaseTestSetup.java:112)
	at org.apache.hadoop.hive.hbase.HBaseTestSetup.preTest(HBaseTestSetup.java:78)
	at org.apache.hadoop.hive.ql.QTestMiniClusters.setup(QTestMiniClusters.java:240)
	at org.apache.hadoop.hive.ql.QTestUtil.<init>(QTestUtil.java:221)
	at org.apache.hadoop.hive.hbase.HBaseQTestUtil.<init>(HBaseQTestUtil.java:46)
	at org.apache.hadoop.hive.cli.control.CoreHBaseCliDriver.beforeClass(CoreHBaseCliDriver.java:49)
	at org.apache.hadoop.hive.cli.control.CliAdapter$1$1.evaluate(CliAdapter.java:80)
	at org.junit.rules.RunRules.evaluate(RunRules.java:20)
	at org.junit.runners.ParentRunner$3.evaluate(ParentRunner.java:306)
	at org.junit.runners.ParentRunner.run(ParentRunner.java:413)
	at org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:365)
	at org.apache.maven.surefire.junit4.JUnit4Provider.executeWithRerun(JUnit4Provider.java:273)
	at org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:238)
	at org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:159)
	at org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:377)
	at org.apache.maven.surefire.booter.ForkedBooter.execute(ForkedBooter.java:138)
	at org.apache.maven.surefire.booter.ForkedBooter.run(ForkedBooter.java:465)
	at org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:451)
Caused by: java.lang.RuntimeException: Master not initialized after 200000ms
	at org.apache.hadoop.hbase.util.JVMClusterUtil.waitForEvent(JVMClusterUtil.java:221)
	at org.apache.hadoop.hbase.util.JVMClusterUtil.startup(JVMClusterUtil.java:190)
	at org.apache.hadoop.hbase.LocalHBaseCluster.startup(LocalHBaseCluster.java:407)
	at org.apache.hadoop.hbase.MiniHBaseCluster.init(MiniHBaseCluster.java:250)
	... 21 more

http://ci.hive.apache.org/blue/organizations/jenkins/hive-precommit/detail/PR-5187/5/tests
I find that hbase-handler related tests are failed. I think the failures are caused by the incompatibility between hbase and hadoop3.4.0. Check HIVE-27651 which i have done about the hbase-hadoop incompatibility. This needs to make sure hbase itTest can run in Hadoop3.4.0 and maybe need hbase community to provide a compatible package with hadoop3.4.0.

Thank you very much for the explanation! It is very important. I will try to make Hbase compatible with Hadoop 3.4.0.

I noticed the currently closed PR from Add GraalVM Reachability Metadata and corresponding nativeTest for HiveServer2 shardingsphere#31526 side. But to be honest, isn’t https://issues.apache.org/jira/browse/HBASE-28714 related to Hadoop 3.4.0 integration on HBase side already closed? And apache/hbase:2.5.10 has been released, see https://github.com/apache/hbase/releases/tag/rel%2F2.5.10 .

@linghengqian Hive master branch uses Hbase2.5.6 :https://github.com/apache/hive/blob/c26342e0973dc3ed22b771b15b27f41b27d8eb20/pom.xml#L146
Maybe upgrading to Hbase:2.5.10 can help to upgrade Hadoop version? @slfan1989

Hi, I've just landed here after trying to bump Hadoop to 3.4.0 in Hive, and I ended up with the same exact patch of this PR, being able to build and make tests pass.
Do you think there's anything else missing to reach fully compatibility that is not yet covered by tests?
Thank you.

Hi, I've just landed here after trying to bump Hadoop to 3.4.0 in Hive, and I ended up with the same exact patch of this PR, being able to build and make tests pass.
Do you think there's anything else missing to reach fully compatibility that is not yet covered by tests?
Thank you.

Since the original author of this PR has not responded, perhaps someone needs to create a new PR containing a new version of the HBase Client. The original issue https://issues.apache.org/jira/browse/HIVE-28191 has not yet been directly claimed.
The main reason for needing to update the Hbase Client is failed integration tests. Disclaimer, I am not actually a Hive Committer, so I cannot directly update the content of the current PR.

Hi, I've just landed here after trying to bump Hadoop to 3.4.0 in Hive, and I ended up with the same exact patch of this PR, being able to build and make tests pass.
Do you think there's anything else missing to reach fully compatibility that is not yet covered by tests?
Thank you.

Since the original author of this PR has not responded, perhaps someone needs to create a new PR containing a new version of the HBase Client. The original issue https://issues.apache.org/jira/browse/HIVE-28191 has not yet been directly claimed.
The main reason for needing to update the Hbase Client is failed integration tests. Disclaimer, I am not actually a Hive Committer, so I cannot directly update the content of the current PR.

I have opened https://issues.apache.org/jira/browse/HIVE-28552 to discuss the unit test bug for the master branch.

@zhangbutao @maxgio92 @linghengqian Sorry for the late reply. I think we can work together to solve this issue. Since Hadoop 3.4.1 is about to be released, I hope we can integrate 3.4.0 into Hive first.


Feature request

1. Comment widget

https://comments.app/

2.  Search widget

https://www.algolia.com/

3. amp-auto-ads
<script async custom-element="amp-auto-ads"
        src="https://cdn.ampproject.org/v0/amp-auto-ads-0.1.js">
</script>

<amp-auto-ads type="adsense"
        data-ad-client="ca-pub-idxxxxx">
</amp-auto-ads>

Hello @LGTMXD, I have already considered all of these ideas and I will answer your requests in order:

This blog already supports a comment widgets, actually it's widget agnostic, so you could theoretically use any comment system you want and use an iframe to include it on your web site, that is waht I do on my own site (https://asur.dev/en/amperage/theme-kitchen-sink/), you can check the README to read the example I provide for Disqus, everything is public.

Algolia might not be the best solution because it would require some sort of sync step to update their indexes at the time of deploy, on the other hand GoHugo custom output formats look more promising (https://gohugo.io/templates/output-formats/) but this is tricky to implement in AMP, specially at the UI level. Consider this one a maybe :smile:

This is the most subjective of all 3, I prefer being able to set ads for myself instead of getting it automatically, this would be nice to have as a configuration [manual|auto] but this would require some time. This however is the most viable of all three.

Thank you for your ideas and stay tuned for more features. OFC pull requests are always welcome! :smiley:


Code is littered with async/await statements to keep synchronous operation

Currently the code uses await to synchronously communicate with the vscode fs api. The problem is that functions using await must be async, which then must be called with an await of its own, cascading all the way to the activation of the extension.
A better way is to pass the code that must be run after the value is returned, but this would require major rewrites and redesigns to the sections of code that deal with the file system.

I didn't actually "fix" this in the rewrite, but I'm much more ok about its use.


IoTCentral create command is not idempotent.

Describe the bug
IotCentral create command is not idempotent.
To Reproduce
Run same create command twice:
command: az iotcentral app create --name myapp000 --subdomain myapp000 -g iOTC
Response1:
{
"applicationId": "4cb23807-65c6-4bbd-a5a2-1636bb3b3bd3",
"displayName": "myapp000",
"etag": ""02000b6a-0000-0000-0000-5be48bcb0000"",
"id": "/subscriptions/c75f6a44-0c50-4e42-b7ec-
9ff5590b0944/resourceGroups/iOTC/providers/Microsoft.IoTCentral/IoTApps/myapp000",
"location": "westus",
"name": "myapp000",
"resourceGroup": "iOTC",
"sku": {
"name": "F1"
},
"subdomain": "myapp000",
"tags": null,
"template"<EMAIL_ADDRESS>"type": "Microsoft.IoTCentral/IoTApps"
}
Response2:
Try other variations of the name
Expected behavior
Should update/replace existing application and return application properties on every response.
Environment summary
Any

IoT Central has a separate CLI than IoT Hub

Closing the issue due to no response. If you still need help with this issue, please re-open.


Docker test looks like it fails on sync PRs when its actually doing what it is supposed to be doing.

This can be confusing for people. But when the Docker test skips testing because it was already tested, it makes it look like it fails
See here for example: https://github.com/jhudsl/Adv_Reproducibility_in_Cancer_Informatics/actions/runs/1791528160
I need to make it so it is clear it is skipping the test because it is not necessary, not that it is failing.

I believe this should be fixed with #436's revamping. But we will see.


Prepublish checklist line spacing is incorrect

Bug Description
Line spacing should reflect figma file (14pt/20) - Overall design should be as vertically tight as possible
Expected Behaviour
Line spacing should reflect figma file

Steps to Reproduce

Screenshots

Additional Context

Plugin Version:
Operating System:
Browser:

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

Setting status to blocked since this will be updated when we implement new designs

Setting status to blocked since this will be updated when we implement new designs

Designs have been updated, new version of pre-publish is in staging. Design to confirm.


cpuminer: 2.5.0 -> 2.5.1

Motivation for this change
Includes fixes and new features such as Bech32 support (https://github.com/pooler/cpuminer/commit/7e8960212f0055d6dafaa3c80812c50cec291de3)
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

I also request a new version here: https://github.com/pooler/cpuminer/issues/194

there is a new version now.

Updated the PR to package the just released version 2.5.1


No run parameter -PackageFilePath in deploy-webapp.ps1

Hi,
When I run the script deploy-webapp.ps1 the bash show an error over that parameter -PackageFilePath is incorrect.
I can see that PowerShell no use that parameter, what is the alternative???

Thanks for your answers

I do not believe that you need to use -PackageFilePath.

Hello @Josegc89,
Thank you for posting the issue!
As @UTKBlake mentioned, the deploy-webapp.ps1 script doesn't take a parameter named -PackageFilePath.
Feel free to reopen this issue if you see other errors.


Clarify IUriHelper GetAbsoluteUrl behavior. Fixes #9717

In #9717, there's the suggestion of a bug because uriHelper.GetAbsoluteUri didn't return the hash part of the URL. However I've confirmed there's no bug:

During prerendering, the server doesn't receive the hash part of the URL, so it's expected to be absent
Once connected, GetAbsoluteUri does return the URL including hash

So this is behaving as expected. To make this super clear, this PR adds E2E tests to show how it works:

When prerendering
When initially starting up an interactive circuit (or client-side Blazor instance)
When navigating within an interactive circuit (or in client-side Blazor)

Looks good, would be helpful if you could merge soon so I can rebase on top.

@rynowak Done


Fix malformed XML documentation in TraceEventSession.cs

While working with a TraceEventSession session, I pressed F12 on the EnableKernelProvider() method to see what exactly it did.  The documentation that was displayed to me above the decompiled method was a bunch of sentences without periods that had been mushed together into a giant paragraph that didn't make a lot of sense.  Inspection of the source code here revealed that the closing </summary> tag accidentally got placed after the parameter documentation (instead of before it).
This change fixes the malformed XML documentation, and fixes a couple other spots in the file as well while we're at it.

@brianrob An easy one to send your way.

👍  Thanks!


Cannot configure to exclude Externals

As per this question: https://stackoverflow.com/questions/42746719/exclude-externals-from-jsdoc-output
It would be good if the library could finally configure Externals not to be generated in the main view.

I have written a possible solution to you S.O question I hope it helps, sorry it is... very late.


chore(deps): bump github.com/Masterminds/sprig to 3.2.3

What this PR does / why we need it:
github.com/Masterminds/sprig is not a go 1.13 module, this is migrating to github.com/Masterminds/sprig/v3 which is one.
PR Readiness Checklist:
Complete these before marking the PR as ready to review:

[ ] the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

Signed-off-by: Matthieu MOREL<EMAIL_ADDRESS>

Hi in here we can see

Error: tests failed: knative CRD deployment failed STDOUT=() STDERR=(error: unable to read URL "https://github.com/knative/serving/releases/download//serving-crds.yaml", server reported 404 Not Found, status code=404

It seems like a part of the URL is missing, but I don't see how it would be related to my commit ?

Hi @mmorel-35 👋
Thanks for the patch! I've created an issue for ktf to track the problem in CI that you've mentioned: https://github.com/Kong/kubernetes-testing-framework/issues/454. Before that's resolved we should be able to workaround this by re-triggering the CI (which I've done).

Hi @mmorel-35 👋
Do you mind rebasing (merging main)? With #3233 we should have the problem of failing enterprise tests out of the way. 🙇


Option to select the number of images being output

Not sure if this is already an option or if it's possible to make it configurable?
For example something like below?

image foo bar (samples=2)

image foo bar (images=4)

nevermind i found the upscale command :P

Right now it is fixed, I can add an option later but found for the most part I didn't need it. To get 9 images while scanning through seeds, use (seed_search=t).


修正 [examples/vision/segmentation/paddleseg/cpu-gpu/cpp/infer.cc] 第93行末尾漏了';'

PR types(PR类型)
Bug Fix
Description
修正 [examples/vision/segmentation/paddleseg/cpu-gpu/cpp/infer.cc] 第93行末尾漏了';'

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


ci: add Apple Silicon builds

Fixes https://github.com/dunglas/frankenphp/issues/307. Closes https://github.com/dunglas/frankenphp/pull/311.

Even paying isn't option...

Larger runners are only available for organizations and enterprises using the GitHub Team or GitHub Enterprise Cloud plans.


Payment gateway

Hello
Great app im working on this but I want to ask about the payment gateways. Can I only add apple pay or I should have another payment gateway to use?

Hello @moh1122,
You can use your default gateway for processing payments with credit cards  https://help.shopify.com/manual/payments.
Apple Pay it's additional feature.

Hello
thank for your reply honestly I dont have a payment gateway yet can I add apple in app purchase instead?

We've just tested processing Apple Pay payments without payment gateway - Shopify returns error that payment gateway is not configured. So, you can't use Apple Pay without payment gateway.
You can choose any gateway which support Apple Pay - https://developer.apple.com/apple-pay/#payment-providers


Rename Cookbook and Repository

I've just used the default name suggested by github.  We need something a little more descriptive, perhaps involving the project name?

how about sti_crc_method_comparison ?

Ok, it's done.  Please note updated URLs.


how to pass the settings in cl = Client(settings) for saved cookies so that session never expires?

I followed below steps -
cl = Client()
print(cl.get_settings())

settings = {'uuids': {'phone_id': 'a3e60051-dc5b-4dd4-b089-29ab63deddcb', 'uuid': '3c6df3c5-39cc-480f-ba6d-47d999bfd59c', 'client_session_id': '861281df-776a-4909-a3b3-e0cbaa2c4b3d', 'advertising_id': 'ce7525d7-d55a-46db-8daf-ad868122018e', 'android_device_id': 'android-3e7f020e89ee73fb',
'request_id': '5100825d-f84f-46fb-85c8-72542fba5f66', 'tray_session_id': '70f690b7-b9f5-4c2a-b9a5-3d483713348d'}, 'mid': None, 'ig_u_rur': None, 'ig_www_claim': None, 'authorization_data': {}, 'cookies': {}, 'last_login': None, 'device_settings': {'app_version': '<IP_ADDRESS>.118', 'android_version': 26, 'android_release': '8.0.0', 'dpi': '480dpi', 'resolution': '1080x1920', 'manufacturer': 'Xiaomi', 'device': 'capricorn', 'model': 'MI 5s', 'cpu': 'qcom', 'version_code': '314665256'}, 'user_agent': 'Instagram <IP_ADDRESS>.118 Android (26/8.0.0; 480dpi; 1080x1920; Xiaomi; MI 5s; capricorn; qcom; en_US; 314665256)', 'country': 'US', 'country_code':1, 'locale': 'en_US', 'timezone_offset': -14400}
cl = Client(settings)

Here last_login gives None , what all argument needs to be changed while passing saved cookies, how we will get the last login?
@adw0rd ,,Please help.

Save it and get it, or I didn't understand the question?

Save it and get it, or I didn't understand the question?

When I save the settings, it gives last_login = None, it is fine as previously I was getting challenge resolver error, and sometimes the username you entered is incorrect?

@purnima110895
https://github.com/adw0rd/instagrapi/issues?q=username+your+entered
https://github.com/adw0rd/instagrapi/discussions?discussions_q=The+username+you+entered

resolved by saving the settings .


Add API docs for StaticArray

I added an example for use with uninitialized. @RX14 @jhass could you pleas re-review?


Keyset Pagination Fails with Nested Collections: ArgumentException in Sorting by Nested Property

I am encountering an issue with keyset pagination using the MR.EntityFrameworkCore.KeysetPagination library when dealing with nested collections. Specifically, I am trying to paginate PaymentRequest entities sorted by RemittanceDetail.Number, but I encounter an exception.
public class User
{
    public Guid UserId { get; set; }
    public List<Address> Addresses { get; set; }
}

public class Address
{
    public Guid AddressId { get; set; }
    public string City { get; set; }
    public string Street { get; set; }
}

Set Up the Keyset Query:

var keysetQuery = KeysetQuery.Build<User>(builder =>
{
    builder.Ascending(u => u.Addresses.First().City);
});

Sample Code

var searchQuery = _dbContext.Users.Include(u => u.Addresses).AsQueryable();
            var paginationContext = searchQuery.KeysetPaginate(keysetQuery, KeysetPaginationDirection.Forward, reference);
            var users = await paginationContext.Query
                .Take(10)
                .Select(u => new
                {
                    UserId = u.UserId,
                    Addresses = u.Addresses.Select(a => new { City = a.City, Street = a.Street }).ToList()
                })
                .ToListAsync(cancellationToken);

            paginationContext.EnsureCorrectOrder(users);
            var hasNext = await paginationContext.HasNextAsync(users);

Failure :
This fails in HasNextAsync method with the following exception :
**System.ArgumentException: Expression of type 'System.Collections.Generic.List1[<>f__AnonymousType61[System.String]]' cannot be used for parameter of type 'System.Collections.Generic.IEnumerable1[Namespace.NestedEntityType]' of method 'Namespace.NestedEntityType FirstNestedEntityType'**`
The example provided in the Readme for Loose Typing is simple, it just have a navigation property but in our case we have a collection and there is no example on how to deal with nested collections.
Could you please provide an example on how to sort with properties in the nested collection ?

Using collections in the keyset is not supported. Even if it was, it defeats the goal of using keyset pagination as it's adding a join in the base condition of the query that can't be satisfied with an index, leading to a table scan.
A solution is to create a computed column that stores Addresses.First().City and add this column to the keyset (an example of this is shown in caveats.md). This way you can index this column and you can use it in the keyset as any other column.

I'm closing this, please reopen if you still have issues.


CommonMark spec 0.28

Diff: http://spec.commonmark.org/0.28/changes.html
Changelog: http://spec.commonmark.org/changelog.txt

Released in 0.10.0: https://github.com/atlassian/commonmark-java/blob/master/CHANGELOG.md#0100---2017-09-14


Support opting out of completion notification of background jobs per job

There is currently a global option, $notify-bg-job-success to control this behavior. However, library authors may want to opt individual background jobs out of this behavior without changing a global option.
The syntax of background job (appending a &) is not easily extensible; it seems more straightforward to introduce a separate builtin command, run-bg, that support this option:
run-bg &notify-success=$false { ... }

The run-bg command can also support other options in future.

I don't see any need for a new way to run background jobs. The interactive notification of a background job status change should only apply to jobs started directly from an interactive prompt. If a job is backgrounded by a function there is no reason for the user to care if the status of that job changes. Therefore, the user should not be notified when it changes status. Which also implies the builtin (i.e., "global") notify-bg-job-success variable should be moved into the edit module.

In case it isn't obvious from my previous comment... Job status notification is only of interest to a user for jobs they explicitly backgrounded. If a function (i.e., "command") they executed chooses to background a job any notification of a change in status is only going to be confusing. The user won't have any knowledge of putting that job into the background.

@krader1961 your idea makes sense for the default behavior, but it might still be useful to have the run-bg command to choose behavior for individual commands.

@zzamboni, I'm not sure what you're proposing in your last comment. Are you proposing that run-bg be implemented as a mechanism for a user to, interactively, do the equivalent of this sequence of commands you might run in bash or ksh?:
> a-cmd &
> disown %%

Or, are you proposing that run-bg provide a mechanism to request interactive notification for jobs backgrounded by a function? If that's what you have in mind I think we need to see a real-world example where that would be useful. My contention is that the user should never see a notification for a status change in any job they did not explicitly background. In other words, if I do sleep 3 & I can expect to see a notification when that backgrounded sleep terminates. If, on the other hand, I type fn my_sleep { sleep 3 & } followed by my_sleep I do not expect, or want, to see a notification that the sleep started by that function has terminated. You might argue that I defined the function that backgrounded the external command. How would Elvish distinguish that case from something like a prompt that depends on a backgrounded helper command, via a function I didn't write, that terminates? Any job status notification for that backgrounded helper command is going to be, at best, confusing.


Update index.html

se cambia linea

aceptado

Hi @omar-cuc ,
It looks like you accidentally submitted this PR against the upstream template repo instead of your own fork: https://github.com/omar-cuc/template-python/
Regards,
Stennie


-a removes configurations that are required

After running SymbolicTM on a file that was fixed with -a I get "no match" errors, when there weren't any before the fix.
Using -a again shows that it then wants to add new configurations, e.g.
B: .@0(01)*10011 1 0(10)*1(11)*10@
Looking at the output of the first -a run I find that it commented out a configuration that could be used instead of this new one:
/ Matching B: .@(01)*10011 1 0(10)*1(11)*10@
Thanks a lot for your work on this program! I'm trying to use it to verify the proofs my ctl-decider program produces, so the files can be big and the configurations a bit messy.
Skelet36.txt
Skelet36fixed.txt
Skelet36fixedTwice.txt

I am going to investigate this. But it might take some time.

When processing the file Skelet36.txt, I see a lot of error message like:
Error: pattern is A: .@(11)*011 0 010@ matches A: .@(00)*011 0 010@

At first this looked wrong to me, but it seems okay, because they are both equivalent with: .@011 0 010@
The above commit adds to command line options:

The option -csa cleans rules like (00)*.@ into<EMAIL_ADDRESS>The option -ru removes unused rules from the output when -a is used.

If the original file is processed with the commands:
./SymbolicTM -csa -a Skelet36.txt
./SymbolicTM -a Skelet36.txt

And next with the following repeated until no warnings about unused rules are reported:
./SymbolicTM -ru -a Skelet36.txt

It results in the Sketel36.txt file with the commit. I think that this solution could further be reduced because there are many cases of 01(01)* that could be changed into (01)+ or even (01)* and it seems that there are also some rules that could be combined.

Thanks, that works!
One could certainly find a simpler set of configurations for Skelet36, but I'm happy that my decider finds something that works at all, especially with parameters that also worked for many other machines in the scan :)
Using more specific parameters I found a shorter proof and was able to clean it up with the new commit. There are presumably still a lot of reductions that could be done, but I don't think that finding those is all that important.
Skelet36v2.txt


improved arcade configuration

What does this PR do?
The arcade package was bundling megabytes on unnecessary data which is now excluded
PR Checklist

[x] Correct base branch selected? Should be develop branch.
[x] Enabled commit hook or executed ./bin/autoformat-nuitka-source.
[ ] All tests still pass. Check the Developer Manual about Running the Tests. There are GitHub
Actions tests that cover the most important things however, and you are welcome to rely on those,
but they might not cover enough.

I did a rebase for the 2.4 release, do not worry, I will resolve this for you, provided I have push rights or merge it directly otherwise for the 2.5 release cycle and likely into 2.4.1 hotfix.


add debug support

Codecov Report

Merging #11 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #11   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines          43     56   +13
=====================================
+ Hits           43     56   +13

Impacted Files
Coverage Δ

index.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

4.3.0


operators [N] [CI] log2rbac (0.0.2)

Signed-off-by: Jirka Kremser<EMAIL_ADDRESS>Thanks submitting your Operator. Please check below list before you create your Pull Request.
New Submissions

[x] Are you familiar with our contribution guidelines?
[x] Have you packaged and deployed your Operator for Operator Framework?

[x] Is your submission signed?
[x] Is operator icon set?

Updates to existing Operators (it does not apply here, it's a new operator)
~* [ ] Did you create a ci.yaml file according to the update instructions?~
~* [ ] Is your new CSV pointing to the previous version with the replaces property if you chose replaces-mode via the updateGraph property in ci.yaml?~
~* [ ] Is your new CSV referenced in the appropriate channel defined in the package.yaml or annotations.yaml ?~
~* [ ] Have you tested an update to your Operator when deployed via OLM?~
~* [ ] Is your submission signed?~
Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

/merge possible

/merge possible


Invalid redirect URI/Oauth callback failure after attaining authorization code

When I try to login using Apple oAuth by hitting the URL
http://my.example.app/api/auth/login/apple?client_id=webapp&state=ae6cfa04-2b97-443d-89d6-735ca87e1164&response_type=code&nonce=3bf581b7-59ec-4f91-9caf-bf02b9913a67&kc_idp_hint=apple&redirect_uri=https://my.example.app/api/auth/apple/callback/webapp&scope=openid email
I get the following error

Upon further inspection, it seems like it is due to the redirected authorization URL having indeed a bad URI (it uses HTTP instead of HTTPS) - https://appleid.apple.com/auth/authorize?response_mode=form_post&scope=openid+email+name&state=MHsd34r2J1DOdLgnx7G8qVbhZ7k-FGLxHM6E7qOqTDs.jKtRwBXmZUw.webapp&response_type=code&client_id=my.app.clientId&redirect_uri=http%3A%2F%2Fmy.example.app%2Fauth%2Frealms%2FAppleAuth%2Fbroker%2Fapple%2Fendpoint&nonce=GxRbWFg_lmpH3mZ_ZE2QOg
Now, if I change the redirect_uri query string parameter to ensure it is https instead of http, I am able to login with my apple ID. HOWEVER, the callback URL fails with the following error
Failed to make identity provider oauth callback: org.keycloak.broker.provider.IdentityBrokerException: No access_token from server. error='invalid_grant', error_description='redirect_uri mismatch. The code was not issued to http://api.vid.app/auth/realms/AppleAuth/broker/apple/endpoint.', error_uri='null'

I suspect this is due to the initial redirect URL being HTTP instead of HTTPS - is there a way around this? Id like for the redirected request to have the parameter for redirect_uri follow the HTTPS scheme instead, as Apple does not allow for HTTP URLs as callbacks
Or maybe I am doing something wrong... my keycloak server is configured via a helm chart using the following command - helm install auth-keycloak codecentric/keycloak --version 18.1.1

Responding to my own issue as I found the solution...
The problem was a lack of an env variable for the keycloak container PROXY_ADDRESS_FORWARDING=true. The keycloak docs make note of this over here - https://github.com/codecentric/helm-charts/tree/master/charts/keycloak#running-keycloak-behind-a-reverse-proxy
Since I installed my keycloak server using a helm chart, I had to do the following

Create a file extra-config.yaml comprising of the folllowing contents -

extraEnv: |
         - name: PROXY_ADDRESS_FORWARDING
           value: "true"

helm upgrade --reuse-values -f extra-config.yaml auth-keycloak codecentric/keycloak

The redirect URLs are now in https as expected.


PowerRename presets

Description of the new feature / enhancement
I tend to use powerrename in a few ways repetitively, it would be helpful to have presets for both the search and replace with.
If those were saved under the current options on the left, I'd love to be able to click to bring those up already. I could do this with some bat stuff but this would be more helpful visually to see it's correct, and all in one screen.
Going further, then being able to add some presets to the context menu would be amazing.
Love the tools :)
Scenario when this would be used?
Renaming certain files for one task, and others in another
Supporting information
No response

/dup https://github.com/microsoft/PowerToys/issues/25520


More documentation on signature and threads management

Describe the bug
More documentation, please
Why does the Handler method have an async? Is that mandatory? At high throughput, it leads to multiple problems. Do you think we can remove the async?
2/ Customer creates a .NET application with 60 workers. From time to time, it seems when a worker finishes a batch of 15, it waits 5 seconds before getting another batch, even if there are a lot of tasks.
When in the configuration, we specify 15 threads, is this 15 threads PER WORKER or for ALL THE 60 workers?
To Reproduce
No bug, more documentation
Expected behavior
A section in the documentation to explain if
Environment (please complete the following information):
Additional context

Hey maybe next time we can use the Discussions for such https://github.com/camunda-community-hub/zeebe-client-csharp/discussions
Sounds to me more like some questions/discussion topics and not an actual issue to fix.

Why does the Handler method have an async? Is that mandatory? At high throughput, it leads to multiple problems. Do you think we can remove the async?

The job handler supports both (async and sync) to avoid such discussions and make the implementation easy as possible.
https://github.com/camunda-community-hub/zeebe-client-csharp/blob/main/Client/Api/Worker/IJobWorkerBuilderStep1.cs#L55-L104
Async/await is a common pattern in the .net world. If you see any weird behavior I doubt it is because of this, likely it is something different.

2/ Customer creates a .NET application with 60 workers. From time to time, it seems when a worker finishes a batch of 15, it waits 5 seconds before getting another batch, even if there are a lot of tasks.
When in the configuration, we specify 15 threads, is this 15 threads PER WORKER or for ALL THE 60 workers?

Not sure I can completely follow. Would be great if can share the complete configuration. But it is not unexpected that it might take a while until it polls the next tasks, depends on the configuration, and it might be even that tasks already locked by other workers. To completely answer this I need more context and details.


Update base and add zypak-wrapper.

For any Electron apps >=v5 we need a wrapper. See: https://github.com/flatpak/flatpak/issues/3044
The new base image org.electronjs.Electron2.BaseApp seems to be maintained and has the wrapper already included.
lbry-app seems to hang when quitting after the finished shutting down event. Probably an error of zypak-wrapper.

Thank you! The test build from your PR runs perfectly fine, while the stable version of the flatpak is currently broken with:
user@mycomputer > flatpak run --branch=stable io.lbry.lbry-app
[3:1016/135730.930624:FATAL:setuid_sandbox_host.cc(158)] The SUID sandbox helper binary was found, but is not configured correctly. Rather than run without sandboxing I'm aborting now. You need to make sure that /app/lbry-app/chrome-sandbox is owned by root and has mode 4755.
/app/bin/lbry-app: line 2:     3 Trace/breakpoint trap   (core dumped) env TMPDIR=$XDG_RUNTIME_DIR/app/$FLATPAK_ID /app/lbry-app/lbry-app "$@"

Probably due to the Electron >=v5 issues. Do you have permission to merge this? I don't myself.

@jiggytom - Turns out that things are getting broken it seems. Merging this now.

Thank you guys ! @corecontingency , can we show you some appreciation?