Enable uninstalling requests

I was surprised to learn BayBE includes telemetry so deep in the project. It's uncomfortable importing a package that talks to the internet.
Since

"BayBE collects anonymous usage statistics only for employees of Merck KGaA, Darmstadt, Germany and/or its affiliates. The recording of metrics is turned off for all other users and impossible due to a VPN block."

these dependencies seem useless for most users?
In this PR, I moved requests to the # Attempt telemetry import section so it's no longer needed to run anything in this project.
Because of https://github.com/emdgroup/baybe/pull/181, the telemetry is no longer optional to install when using pyproject.toml? I just installed the dependencies manually. It'd be nice to have this be optional so users don't get surprised by an automatic requests installed. Does poetry not do that?

Also, is there a reason this is not mentioned in the README? It used to be, but was removed: https://github.com/emdgroup/baybe/pull/3. Why?

 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.

Hi @rickwierenga
the problem is that we have no way of making dependencies opt-out. In the ideal world we would allow installation via something like pip install baybe[notelemetry]. This would then install the code with telemetry deactivated and not require any internet packages. But to the best of my knowledge thats not possible.
As a result we decided to have internet packages in the dependencies. The reason for that is simple: no one would opt-in for telemetry, even if the only target group are our company-internal internal users.
For externals, we had hoped that

the fact that this is an open-source package where people can see exactly what data is potentially uploaded
the fact that telemetry is auto-deactivated of the company-internal page is not reachable (aka you're not in our VPN means telemetry does not send anything)
the fact that you can also deactivate telemetry in addition via environment variables as explained here
would suffice.

But I can see that if you want to be really sure you'd also want to not have any internet package in your environment. If that is what your PR enables then I'd be willing to accept it. I guess it would still require you to do some manual uninstalling after baybe installation or how are you dealing with it?
I think the telemetry mention was moved from the README because we wanted to reduce content there. But if it helps we can reintroduce the mention or at least a link to the detailed explanation.

Thanks for the quick and detailed answer. I see the points about this being open source (which is awesome!) but I think unfortunately not everyone will take the time to read it.
Why not make it opt-in?

Sounds good! I'll try this out for my experiments this night and tomorrow, but if results are good I'm happy to work on this!

Hi @rickwierenga :wave: thanks for connecting. I think @Scienfitz explained our dilemma quite accurately. To provide a little more context: baybe was originally developed purely for inhouse optimization, but we pushed to make the library open-source and were in fact successful with this plan. However, has now brought us into a sort of chimera state where the telemetry part is still "inhouse" in a sense. And I think you will understand that we can't simply drop it because the gathered internal usage stats eventually justify our work on the library – without them, there would have been no package in the first place.
That said, we are hoping to find a better solution in the mid term, ideally via opt-out. In fact, there is an ongoing discussion on extending pip capabilities in that direction, but it hasn't yet progressed very far. Let's see what the happens in the next few months.
Until then, I think your PR is a good first step 👍🏼

@rickwierenga were you able to test whether this works for you, ie uninstalling the internet deps + your code change? If you confirm I will add updates to the telemetry descriptions and merge this

Sounds good! I'll try this out for my experiments this night and tomorrow, but if results are good I'm happy to work on this!

Hi @rickwierenga, the other PR #205 that adds a telemetry section to the README is up. Ideally, we'd like to wait for your final confirmation before merging both. Can you give a quick update if everything works as expected for you?

Hi @Scienfitz, seems like @rickwierenga is currently busy with other things. Could you include that additional paragraph to the readme so that we can proceed towards merging?

@AdrianSosic will do so but I will check whether the uninstall actually enables what was claimed before I merge

@AdrianSosic will do so but I will check whether the uninstall actually enables what was claimed before I merge

Jep, makes sense. Let me know if I should do something...

nice! I like the new transparency :)

Hi @rickwierenga
This PR is accepted and ready for merge (I rebased your fork and added README and CHANGELOG entries fyi)
But if you want this PR merged with your association you still need to sign the CLA agreement, as pointed out by the bot (second comment in this PR)

Hi @rickwierenga, just briefly wanted to inform you that we plan to merge this PR before releasing 0.9.0, which will happen at some point during this week. We'd love to have your association with this PR, so I encourage you to sign the CLA the latest until tomorrow. Of course, you don't need to sign if you refuse, but then we'll probably have to merge these changes from a different branch. So hoping you will accept 🙃

@rickwierenga We recently removed the requests package alltogether in #240
As a result, this PR has become obsolete and Im closing it. Thanks nonetheless for raising the issue about potential trust issues, the README got telemetry mention again and internet packages (such as opentelemetry which I suspect still has requests as secondary dependency) can be uninstalled if desired


HDDS-4139. Update version number in upgrade tests

What changes were proposed in this pull request?
Update version number 0.6.0 to 1.0.0 in upgrade scripts and tests (content and file/directory names).
https://issues.apache.org/jira/browse/HDDS-4139
How was this patch tested?
Ran upgrade acceptance test locally.
https://github.com/adoroszlai/hadoop-ozone/runs/1022231789

Thanks @adoroszlai for find the issue.
The patch looks good.
+1.

Thanks @ChenSammi for reviewing and merging it.


HBDM Swap docs has stale field names and sample JSON from Futures docs

https://huobiapi.github.io/docs/coin_margined_swap/v1/en/#query-trigger-order-open-orders
specifies contract_type | string | true | Contract Type but the example to the right has no such field.

https://huobiapi.github.io/docs/coin_margined_swap/v1/en/#place-trigger-order : same

https://huobiapi.github.io/docs/coin_margined_swap/v1/en/#subscribe-order-data-sub : the example to the right shows a snippet JSON with contract_code and contract_type set to BTC200626 and quarter, respectively

This is a repost of the original issue in https://github.com/hbdmapi/huobi_swap_Cpp/issues/1#issue-693541867 for greater visibility

@ebenali Hi,this json config of the example for the swap has been encapsulated.eg:https://github.com/hbdmapi/hbdm_Python/blob/master/examples/huobi_swap/strategy.py#L64
The example for swap:https://github.com/hbdmapi/hbdm_Python/tree/master/examples/huobi_swap
The config example for swap:https://github.com/hbdmapi/hbdm_Python/blob/master/examples/huobi_swap/config.json#L36

The contract_type field in the JSON config is to indicate the positions of the swap.It will not be passed as parameters when posting requests.

Thanks for the quick response. What of the contract_code field, it still shows [in the docs]
(https://huobiapi.github.io/docs/coin_margined_swap/v1/en/#subscribe-order-data-sub):
{
    "op": "notify",
    "topic": "orders.btc",
    "uid": "10086",
    "ts":<PHONE_NUMBER>607,
    "symbol": "BTC",
    "contract_type": "quarter",
    "contract_code": "BTC200626",
    ...
}

Hi @ebenali It will be fixed later.Thanks for your feedback.


add an explicit tag filter

What's changed?

add an explicit tag filter.
This may help users notice the tag filter feature more easily.
from issue #776 which I forgot about half a year ago.
Checklist

[x]  I hereby agree to the terms of the HertzBeat CLA
[ ]  I have read the Contributing Guide
[ ]  I have written the necessary doc or comment.
[ ]  I have added the necessary unit tests and all cases have passed.

I made this change because I found that I forgot about this issue that was assigned to me half a year ago.
Please approve this if we still need this feature, or close it.


Backend REST API calls are blocked by pace progress bar in GCP cloud.

Hi Team,
We are using the PACE progress bar in our angular 9 app. But all of a sudden the REST API calls to the backend is blocked by pace in cloud environment and we got the below error,
Function.prototype.toString requires that 'this' be a Function
at toString (<anonymous>)
at toString (zone.js:1600)
at push../node_modules/core-js/internals/inspect-source.js.store.inspectSource (inspect-source.js:8)
at toString (redefine.js:33)
at ua (ruxitagentjs_ICA27SVfgqrux_10205201218101503.js:389)

The REST API calls are not blocked, when we remove the pace plugin. The issue occurs only in the cloud.
Environment details:
GCP cloud,
Docker,
Angular 9
Can you kindly shed some light on this!

Hi Satheesh,
I raised a similar issue : https://github.com/CodeByZach/pace/issues/509
You could try downgrading to v1.0.2 it seems to work for us.
Euan


<Transition appear> breaks click event listener

Environment

Operating System: Linux
Node Version:     v18.18.0
Nuxt Version:     3.8.2
CLI Version:      3.10.0
Nitro Version:    2.8.1
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools, vite
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-s3ebia?file=app.vue,components%2FFullscreenBackground.vue
Describe the bug
I was making a background for a modal component when I noticed a strange behaviour with the <Transition /> component, specifically when the appear prop is set.
It seems that the click event listener on an element inside <Transition appear> isn't being properly handled.
Steps to reproduce:

open the reproduction

click on the semi-transparent fullscreen div

nothing happens even though it should have set its visibility to false

remove appear from the <Transition> component in FullscreenBackground.vue

repeat the same steps & now the click is properly handled

Additional context
I've tried and I couldn't reproduce the same issue in Vue, so I assume it could be a bug in Nuxt.
No response
Logs
No response

Would you see if you can reproduce on https://stackblitz.com/github/nuxt-contrib/vue3-ssr-starter ? Sometimes these issues only reproduce on Vue SSR.

Yeah, I can reproduce it there. Didn't think of trying Vue SSR. Sorry 🙈 Will report to Vue then.

Linking https://github.com/nuxt/nuxt/issues/14715.


(tbd) Change configured_environment behavior

Puppet's communication with the node and catalog terminus included a
configured_environment option in the last release. That option sometimes
includes redundant or irrelevant information when Puppet[:environment]
has the default setting. That redundant information precludes it from
being used to identify when an agent has explicitly requested a
particular environment.
Change the behavior of configured_environment to only be set when the
environment setting is explicitly set.

/cc @pcarlisle @reidmv

:+1:


Added install logic for BlueJeans Events - "bluejeansevents"

Added install logic for currently available version of BlueJeans Events client by Verizon. This application only has one .pkg URL currently and does not have architecture specific download URLs.
Added as "bluejeansevents" after existing logic for "bluejeans", beginning on like 1720

wrong branch


experimental web apigw cleanup

I was planning on consolidating things like the following into a single var accross the entire project:
apigw_host: "http://{{ groups['apigateway']|first }}:{{apigateway.port.api}}/v1"
apigw_host_v2: "http://{{ groups['apigateway']|first }}:{{apigateway.port.api}}/v2"

However, the implications seems to be far reaching, and I am not sure the most efficient way to go about it.
Should I be retaining the v1 in all the routes? should it be all updated to v2? or the version removed entirely?
@csantanapr @mdeuser

David what you have is not directly related to apigateway.
This PR has two concepts that should be split.

apigateway experimental v1
/experimental/web  was the first implementation on what we talk today as web actions.

I think @rabbah would be happy to see /experimental/web removed. 🌮
I would delegate to him on to review and assist on this PR. 🙏
I would suggest since the majority of this PR is related to experimental/web create a new PR and move the changes to the apigateway.md to a new PR, that's the only  related change to apigateway in this PR.
Then on the new PR for the ansible:

remove apigw_host: "http://{{ groups['apigateway']|first }}:{{apigateway.port.api}}/v1"
all the environments group vars
remove from whisk.properties ansible template https://github.com/apache/incubator-openwhisk/blob/master/ansible/templates/whisk.properties.j2#L103

can def nuke /experimental and all the stuff that goes with it. go for it.

sister PR in Blue ~3909

@drcariel - the apigw_host: "http://{{ groups['apigateway']|first }}:{{apigateway.port.api}}/v1" path won't be used any more with the removal of the v1 interface.  there is likely v1-only code in the routemgmt actions (getApi, deleteApi, createApi)

Are we removing the experimental API gateway or that and experimental web actions?

PG1~1936

@drcariel applied wip label for you - what's left?

@rabbah not sure, @csantanapr asked me to change the title since he wanted me to focus on another PR.

@rabbah, looks like people are still using experimental web actions. Should at least send out a notifications above its removal in advance.


destroyResourcesOnDeletion and/or destroy not working as expected in Terraform objects

Hello,
It seems the parameter .spec.destroyResourcesOnDeletion  is not properly working as expected or maybe i'm doing something wrong. I also tried .spec.destroy: true and it seems exactly the same behaviour in tf-runner execucions, we can see the plan of a destroy detect the resources should be destroyed but then it seems don't do anything...
Error from server (NotFound): pods "account-liligo-tf-runner" not found
Error from server (NotFound): pods "account-liligo-tf-runner" not found
Error from server (NotFound): pods "account-liligo-tf-runner" not found
Error from server (NotFound): pods "account-liligo-tf-runner" not found
Error from server (NotFound): pods "account-liligo-tf-runner" not found
Error from server (BadRequest): container "tf-runner" in pod "account-liligo-tf-runner" is waiting to start: ContainerCreating
Error from server (BadRequest): container "tf-runner" in pod "account-liligo-tf-runner" is waiting to start: ContainerCreating
2023/05/16 21:23:43 Starting the runner... version  sha
I0516 21:23:44.119359       8 request.go:682] Waited for 1.000684027s due to client-side throttling, not priority and fairness, request: GET:https://<IP_ADDRESS>:443/apis/apiextensions.k8s.io/v1?timeout=32s

{"level":"info","ts":"2023-05-16T21:23:55.604Z","logger":"runner.terraform","msg":"preparing for Upload and Extraction","instance-id":""}
{"level":"info","ts":"2023-05-16T21:23:55.612Z","logger":"runner.terraform","msg":"write backend config","instance-id":"","path":"/tmp/fluxcd-cloudflare-account-liligo","config":"backend_override.tf"}
{"level":"info","ts":"2023-05-16T21:23:55.612Z","logger":"runner.terraform","msg":"write config to file","instance-id":"","filePath":"/tmp/fluxcd-cloudflare-account-liligo/backend_override.tf"}
{"level":"info","ts":"2023-05-16T21:23:55.615Z","logger":"runner.terraform","msg":"looking for path","instance-id":"","file":"terraform"}
{"level":"info","ts":"2023-05-16T21:23:55.617Z","logger":"runner.terraform","msg":"creating new terraform","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7","workingDir":"/tmp/fluxcd-cloudflare-account-liligo","execPath":"/usr/local/bin/terraform"}
{"level":"info","ts":"2023-05-16T21:23:55.622Z","logger":"runner.terraform","msg":"setting envvars","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
{"level":"info","ts":"2023-05-16T21:23:55.622Z","logger":"runner.terraform","msg":"getting envvars from os environments","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
{"level":"info","ts":"2023-05-16T21:23:55.623Z","logger":"runner.terraform","msg":"setting up the input variables","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
{"level":"info","ts":"2023-05-16T21:23:55.623Z","logger":"runner.terraform","msg":"mapping the Spec.Values","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
{"level":"info","ts":"2023-05-16T21:23:55.623Z","logger":"runner.terraform","msg":"mapping the Spec.Vars","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
{"level":"info","ts":"2023-05-16T21:23:55.623Z","logger":"runner.terraform","msg":"mapping the Spec.VarsFrom","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
{"level":"info","ts":"2023-05-16T21:23:55.640Z","logger":"runner.terraform","msg":"generating the template founds"}
{"level":"info","ts":"2023-05-16T21:23:55.640Z","logger":"runner.terraform","msg":"main.tf.tpl not found, skipping"}
{"level":"info","ts":"2023-05-16T21:23:55.641Z","logger":"runner.terraform","msg":"initializing","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
{"level":"info","ts":"2023-05-16T21:23:55.641Z","logger":"runner.terraform","msg":"mapping the Spec.BackendConfigsFrom","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
{"level":"info","ts":"2023-05-16T21:23:55.645Z","logger":"runner.terraform","msg":"[INFO] running Terraform command: /usr/local/bin/terraform version -json","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
{
  "terraform_version": "1.4.6",
  "platform": "linux_amd64",
  "provider_selections": {},
  "terraform_outdated": false
}
{"level":"info","ts":"2023-05-16T21:23:56.167Z","logger":"runner.terraform","msg":"[INFO] running Terraform command: /usr/local/bin/terraform init -no-color -input=false -backend=true -get=true -upgrade=true -force-copy -backend-config=bucket=edo-tfstate-lab-services -backend-config=prefix=tf-workspaces/fluxcd-cloudflare/account/liligo","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}

Initializing the backend...

Successfully configured the backend "gcs"! Terraform will automatically
use this backend unless the backend configuration changes.

Initializing provider plugins...
- Finding cloudflare/cloudflare versions matching "~> 4.3.0"...
- Installing cloudflare/cloudflare v4.3.0...
- Installed cloudflare/cloudflare v4.3.0 (self-signed, key ID C76001609EE3B136)

Partner and community providers are signed by their developers.
If you'd like to know more about provider signing, you can read about it here:
https://www.terraform.io/docs/cli/plugins/signing.html

Terraform has created a lock file .terraform.lock.hcl to record the provider
selections it made above. Include this file in your version control repository
so that Terraform can guarantee to make the same selections by default when
you run "terraform init" in the future.

Terraform has been successfully initialized!
{"level":"info","ts":"2023-05-16T21:23:57.737Z","logger":"runner.terraform","msg":"error from SIGINT: no such process","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
{"level":"info","ts":"2023-05-16T21:23:57.737Z","logger":"runner.terraform","msg":"error from SIGINT: no such process","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
{"level":"info","ts":"2023-05-16T21:23:57.739Z","logger":"runner.terraform","msg":"workspace select"}
{"level":"info","ts":"2023-05-16T21:23:57.740Z","logger":"runner.terraform","msg":"[INFO] running Terraform command: /usr/local/bin/terraform workspace new -no-color fluxcd-cloudflare","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
Workspace "fluxcd-cloudflare" already exists
{"level":"info","ts":"2023-05-16T21:23:58.024Z","logger":"runner.terraform","msg":"workspace new:, workspace \"fluxcd-cloudflare\" already exists"}
{"level":"info","ts":"2023-05-16T21:23:58.024Z","logger":"runner.terraform","msg":"[INFO] running Terraform command: /usr/local/bin/terraform workspace select -no-color fluxcd-cloudflare","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
Switched to workspace "fluxcd-cloudflare".
{"level":"info","ts":"2023-05-16T21:23:58.306Z","logger":"runner.terraform","msg":"error from SIGINT: no such process","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
{"level":"info","ts":"2023-05-16T21:23:58.306Z","logger":"runner.terraform","msg":"error from SIGINT: no such process","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
{"level":"info","ts":"2023-05-16T21:23:58.326Z","logger":"runner.terraform","msg":"creating a plan","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
{"level":"info","ts":"2023-05-16T21:23:58.326Z","logger":"runner.terraform","msg":"[INFO] running Terraform command: /usr/local/bin/terraform plan -no-color -input=false -detailed-exitcode -lock-timeout=0s -out=tfplan -lock=true -parallelism=10 -refresh=true -destroy","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
cloudflare_logpush_job.audit_job: Refreshing state... [id=228296]

Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
  - destroy

Terraform will perform the following actions:

  # cloudflare_logpush_job.audit_job will be destroyed
  - resource "cloudflare_logpush_job" "audit_job" {
      - account_id       = "9b10dca6d786432ba56af7c0f75c109a" -> null
      - dataset          = "audit_logs" -> null
      - destination_conf = (sensitive value) -> null
      - enabled          = true -> null
      - frequency        = "high" -> null
      - id               = "228296" -> null
      - logpull_options  = "fields=ActorID,ActorIP,ID,OwnerID,NewValue,OldValue,OwnerID,ResourceID,ResourceType,When&timestamps=rfc3339" -> null
      - name             = "audit-log" -> null
    }

Plan: 0 to add, 0 to change, 1 to destroy.
{"level":"info","ts":"2023-05-16T21:24:00.222Z","logger":"runner.terraform","msg":"error from SIGINT: no such process","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
{"level":"info","ts":"2023-05-16T21:24:00.224Z","logger":"runner.terraform","msg":"save the plan","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
{"level":"info","ts":"2023-05-16T21:24:00.291Z","logger":"runner.terraform","msg":"loading plan from secret","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
{"level":"info","ts":"2023-05-16T21:24:00.326Z","logger":"runner.terraform","msg":"running apply","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
{"level":"info","ts":"2023-05-16T21:24:00.326Z","logger":"runner.terraform","msg":"[INFO] running Terraform command: /usr/local/bin/terraform apply -no-color -auto-approve -input=false -lock=true -parallelism=10 -refresh=true","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
cloudflare_logpush_job.audit_job: Refreshing state... [id=228296]

No changes. Your infrastructure matches the configuration.

Terraform has compared your real infrastructure against your configuration
and found no differences, so no changes are needed.

Apply complete! Resources: 0 added, 0 changed, 0 destroyed.
{"level":"info","ts":"2023-05-16T21:24:02.063Z","logger":"runner.terraform","msg":"error from SIGINT: no such process","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
{"level":"info","ts":"2023-05-16T21:24:02.093Z","logger":"runner.terraform","msg":"creating outputs","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
{"level":"info","ts":"2023-05-16T21:24:02.102Z","logger":"runner.terraform","msg":"[INFO] running Terraform command: /usr/local/bin/terraform output -no-color -json","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
{}
{"level":"info","ts":"2023-05-16T21:24:02.507Z","logger":"runner.terraform","msg":"error from SIGINT: no such process","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7"}
{"level":"info","ts":"2023-05-16T21:24:02.508Z","logger":"runner.terraform","msg":"cleanup TmpDir","instance-id":"7b4d6537-8de5-48d6-b5d3-d337b73222b7","tmpDir":"/tmp/fluxcd-cloudflare-account-liligo"}
rpc error: code = NotFound desc = an error occurred when try to find container "f0db78c2198861d3a953bc971eb9d98c41ba05cd2cec462a595ab4db05fd86d9": not foundError from server (NotFound): pods "account-liligo-tf-runner" not found
Error from server (NotFound): pods "account-liligo-tf-runner" not found
Error from server (NotFound): pods "account-liligo-tf-runner" not found

Let me share the Terraform entire file here.
terraform.tar.gz
Let me also describe my environment:
kubernetes in Google, GKE: Server Version: version.Info{Major:"1", Minor:"24", GitVersion:"v1.24.10-gke.2300"
FluxCD v0.41.2
Terraform Controller v0.14.0

Could you help me fixing this?
Marc

We landed fixes for this in v0.15.0-rc, if you could upgrade.

After fix my issue with tf-runner image, I checked and this has not been fixed or maybe I'm wrong with some configuration. Let me reming my environment, basically fluxcd running version v2.0.0-rc.1 and tf-controller v0.15.0-rc.1  as you suggested

If you need something more information i can provide you

Closing as this was related to the need to upgrade.


Docs - Wrong example for toast

Custom HTML
You can pass in an HTML String as the first argument as well. Take a look at the example below, where we pass in text as well as a flat button. If you call an external function instead of in-line JavaScript, you will not need to escape quotation marks.

TOAST WITH ACTION

  var $toastContent = $('I am toast content');
  Materialize.toast($toastContent, 5000);

The above example does not show the correct code to match the description.
Thanks.

Fixed in 4fb1621


how do we "warp" gradients?

we may be able to warp vector paths, but right now we are not applying the same transformation to the gradients as we do to the outlines. I am not even sure whether we can do such a non-linear transformation to SVG linear/radialGradients...
Do we content ourselves to just warp the start/end points and radiuses and hope for the best?
Currently we are not touching them.
This is the first time this problem occurred to me.. Wish I had thought about it earlier.
cf. #5

original (pre-wave) flag emojis containing radialGradient elements:
BZ.svg
GT.svg
MX.svg
TA.svg
... and linearGradient elements:
EC.svg
FK.svg
GS.svg
GT.svg
NI.svg
PM.svg
TA.svg
VG.svg


pyvista backend pythreejs for jupyter creates poor rendering quality

files documenting the issue can be found below:

t-junction-original.pmdat.txt
t-junction-solids.ipynb.txt
(Cc. @akaszynski , @ninad-kamat )

@ninad-kamat it looks as though this area has now progressed and this issue is no longer relevant so I am closing.


No reverse() in v2.0.1

Hi, I just updated to version 2.0.1, looks like reverse() has been removed in this version.
It is possible to reverse animation in this new version?

No noway, only manually by reversing values

If I may ask, what's the point of removing reverse() method?

@amanzan Previous implementation was really poor and confusing, I decided to remove it,im sorry for inconvenience 😔

No problem, I'm using that method and it has been working just fine so far. I'm sure I'll find an alternative with this new version when I decide to upgrade. Thanks so much for this library, by the way 👍 :)

@amanzan Thank you for support 😉 well you can look how it was implemented before in commit history

@amanzan hi man, you was in here :+1:  i have simple quastion whats this below code eqivalent on new version of this library:
SupportAnimator animator =
                            ViewAnimationUtils.createCircularReveal(mRevealView, cx, cy, 0, radius);
                    animator.setInterpolator(new AccelerateDecelerateInterpolator());
                    animator.setDuration(800);

                    SupportAnimator animator_reverse = animator.reverse();

                    if (hidden) {
                        mRevealView.setVisibility(View.VISIBLE);
                        animator.start();
                        hidden = false;
                    } else {
                        animator_reverse.addListener(new SupportAnimator.AnimatorListener() {
                            @Override
                            public void onAnimationStart() {

                            }

                            @Override
                            public void onAnimationEnd() {
                                mRevealView.setVisibility(View.INVISIBLE);
                                hidden = true;

                            }

                            @Override
                            public void onAnimationCancel() {

                            }

                            @Override
                            public void onAnimationRepeat() {

                            }
                        });
                        animator_reverse.start();

                    }

Thanks

@ozodrukh could u make a gist with the implementation to reverse? Or just reverse the values rly work?

Well on Lollipop and higher where native animation is used, it looks like you don't have reverse ability.
By using compat version (Pre Lollipop) cast it to ValueAnimator and use one of this methods

ValueAnimator.setRepeatMode()
ValueAnimator.reverse()

My reverse i do in this way:
Animator animator = ViewAnimationUtils.createCircularReveal(attachmentLayout, cx, cy, 0, radius);
Animator animatorReverse = ViewAnimationUtils.createCircularReveal(attachmentLayout, cx, cy, radius, 0);


[SPARK-41821][CONNECT][PYTHON] Fix doc test for DataFrame.describe

What changes were proposed in this pull request?
Currently, the python doc test failed caused by DataFrame.describe
File "/Users/s.singh/personal/spark-oss/python/pyspark/sql/connect/dataframe.py", line 898, in pyspark.sql.connect.dataframe.DataFrame.describe
Failed example:
    df.describe(['age']).show()
Exception raised:
    Traceback (most recent call last):
      File<EMAIL_ADDRESS>line 1350, in __run
        exec(compile(example.source, filename, "single",
      File "<doctest pyspark.sql.connect.dataframe.DataFrame.describe[1]>", line 1, in <module>
        df.describe(['age']).show()
      File "/Users/s.singh/personal/spark-oss/python/pyspark/sql/connect/dataframe.py", line 832, in describe
        raise TypeError(f"'cols' must be list[str], but got {type(s).__name__}")
    TypeError: 'cols' must be list[str], but got list

This PR also adds some tests into test_connect_basic.py.
Why are the changes needed?
Let the python doc test success.
Adds test cases.
Does this PR introduce any user-facing change?
'No'.
New feature.
How was this patch tested?
N/A

ping @zhengruifeng cc @HyukjinKwon

Shall we fix TODO(SPARK-41821): Fix DataFrame.describe below? You can remove:
        # TODO(SPARK-41821): Fix DataFrame.describe
        del pyspark.sql.connect.dataframe.DataFrame.describe.__doc__

+1, please also enable the doctest

Merged to master.

@HyukjinKwon @zhengruifeng Thank you!


@azure/msal-browser vs @azure/msal for Angular SPA

Library

[ ]<EMAIL_ADDRESS>or<EMAIL_ADDRESS>[x]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>
Description
Is @azure/msal-browser the recommended library for SPA apps?
Does it replace @azure/msal in that case?
Should it be used in conjunction with @azure/msal-angular?

Is @azure/msal-browser the recommended library for SPA apps?

MSAL Browser is currently in private preview, but once it is generally available, yes, it will be the recommended library to use.

Does it replace @azure/msal in that case?

Yes, as it switches from the Implicit Flow (used by msal v1) to the Auth Code Flow w/ PKCE.

Should it be used in conjunction with @azure/msal-angular?

We will upgrade @azure/msal-angular to work with @azure/msal-browser, however, that will not happen at least until msal-browser is GA (and that work is not yet scheduled).

@jasonnutter any ETA on when we will be able to test msal-browser with AD B2C?

@CerosDev Not sure of the ETA on that, @pkanher617 @hamiltonha should be able to provide more context.

@CerosDev  B2C support is now supported, you can test by manually editing your manifest as described in the msal-browser README

Closing as the conversation is no longer pertaining to the issue.

So where we are right now???
@azure/msal-angular that has been recently released(not beta anymore) is not compatible with Safari(renew token fails), forcing users to turn off the security feature(cross-site tracking) seems like a ridiculous and controversial option.
There is no even raw ETA for 'MSAL Browser' support.
IMHO authentication flow is essential for any enterprise, there must be a stable library with cross-browser support without compromise and workarounds.
Microsoft is not a charity and that's a question of integration with Microsoft platform itself, have no idea why it's not a priority why there is no beta/alpha yet, no even dates.

So where we are right now???
@azure/msal-angular that has been recently released(not beta anymore) is not compatible with Safari(renew token fails), forcing users to turn off the security feature(cross-site tracking) seems like a ridiculous and controversial option.
There is no even raw ETA for 'MSAL Browser' support.
IMHO authentication flow is essential for any enterprise, there must be a stable library with cross-browser support without compromise and workarounds.
Microsoft is not a charity and that's a question of integration with Microsoft platform itself, have no idea why it's not a priority why there is no beta/alpha yet, no even dates.

We decided to upgrade MSAL Angular to use MSAL.js v1 (as opposed to v2, aka MSAL Browser) because MSAL.js v2 is still in development (currently in public preview), and we did not want to wait until MSAL.js v2 was stable before GAing MSAL Angular, as we know there customers waiting for MSAL Angular to work with MSAL.js v1.
I understand the frustration with Safari and ITP, and agree it is suboptimal to ask users to disable privacy settings or to often seeing popups in your application when new tokens are needed. Once MSAL.js v2 is GA and stable, we will look at upgrading MSAL Angular to use it. As noted, we haven't scheduled this work yet, but we will post an update when we know (this is dependent both on MSAL.js v2 and other priorities of our team).

The recent changes on the app registrations page in Azure: Integration Assistant (preview), sets out recommendations and evaluates the app registration for SPA application types.
The Develop tab has recommendations to "Avoid programming directly against protocols... instead, use open source library such as MSAL".
On the summary tab, one of the recommendations is "Do not enable access tokens for the implicit grant flow" (image below). There is no disclaimer that the MSAL library doesn't support Auth Code Flow and to ignore this recommendation for Angular.
Whilst the recommendations are definitely welcome, there is a serious issue for Angular that leads to a roadblock.

@liben-y I agree that is confusing, thanks for the feedback. cc: @hamiltonha

Does this fix the looping in IE11 between the app and MSAL?


2019 7 Series (RU7xxx) deadlock

The code is stuck in a loop to wait for the TV to power on, but it's already running. The property change  never occurrs.
The event loop does not seem to recognize the Tvs response.
# stuck at ctor
with samsungctl.Remote(config) as remote:
    print(remote.has_ssl())

GET /api/v2/ HTTP/1.1
Host: <IP_ADDRESS>:8001
User-Agent: python-requests/2.22.0
Accept-Encoding: gzip, deflate
Accept: */*
Connection: keep-alive

HTTP/1.1 200 OK
content-type: application/json; charset=utf-8
content-length: 1315

"device":{
    "FrameTVSupport":"false",
    "GamePadSupport":"true",
    "ImeSyncedSupport":"true",
    "Language":"en_GB",
    "OS":"Tizen",
    "PowerState":"on",
    "TokenAuthSupport":"true",
    "VoiceSupport":"false",
    "WallScreenRatio":"0",
    "WallService":"false",
    "countryCode":"DE",
    "description":"Samsung DTV RCR",
    "developerIP":"<IP_ADDRESS>",
    "developerMode":"1",
    "duid":"uuid:x",
    "firmwareVersion":"Unknown",
    "id":"uuid:x",
    "ip":"<IP_ADDRESS>",
    "model":"19_MUSEL_UHD_BASIC",
    "modelName":"UE55RU7099UXZG",
    "name":"[TV] Samsung 7 Series (55)",
    "networkType":"wireless",
    "resolution":"3840x2160",
    "smartHubAgreement":"true",
    "ssid":"ac:22:05:e4:67:9b",
    "type":"Samsung SmartTV",
    "udn":"uuid:x",
    "wifiMac":"D0:D0:03:7E:2E:A1"
},
   "id":"uuid:x",
   "isSupport":"{\"DMP_DRM_PLAYREADY\":\"false\",\"DMP_DRM_WIDEVINE\":\"false\",\"DMP_available\":\"true\",\"EDEN_available\":\"true\",\"FrameTVSupport\":\"false\",\"ImeSyncedSupport\":\"true\",\"TokenAuthSupport\":\"true\",\"remote_available\":\"true\",\"remote_fourDirections\":\"true\",\"remote_touchPad\":\"true\",\"remote_voiceControl\":\"false\"}\n",
   "name":"[TV] Samsung 7 Series (55)",
   "remote":"1.0",
   "type":"Samsung SmartTV",
   "uri":"http://<IP_ADDRESS>:8001/api/v2/",
   "version":"2.0.25"

my config:
    "host": "<IP_ADDRESS>",
    "port": 8001,
    "method": "websocket",
    "name": "r6",
    "description": "",
    "id": "",
    "token": "",
    "timeout": 1

The timeout settings does not seem to make a difference, as does method encrypted/websocket.
log:
RemoteWrapper.open()
RemoteWrapper.power()
RemoteWrapper.power => False
RemoteWrapper.power(value=True)
RemoteWrapper.power()
RemoteWrapper.power => False
RemoteWrapper.mac_address()
RemoteWrapper.mac_address => 'D0:D0:03:7E:2E:A1'
RemoteWrapper.mac_address()
RemoteWrapper.mac_address => 'D0:D0:03:7E:2E:A1'
RemoteWrapper.power()
RemoteWrapper.power => False
RemoteWrapper.open()
RemoteWrapper.mac_address()
RemoteWrapper.mac_address => 'D0:D0:03:7E:2E:A1'
RemoteWrapper.power()
RemoteWrapper.power => False
RemoteWrapper.open()

It's a deadlock because of recursive method calls.
open()
lock
power on -> power()
open() #2

The problem is still present as of today. While inspecting the code, I realized that the offending recursion to the open() method can be prevented by using configuration setting paired=True.
This workaround works even if the TV is not paired yet, at least with my TV model.
Hope it saves somebody some nerves.

Yeah, the problem is quite annoying. Especially because it's so hard to get through various indirections in the code.
Like that power attribute actually means whether the websocket connection is made (so, has very little to do with the actual power state).
And setting power sends a WOL packet and then just sleeps and then recurses to open.

I have the same TV. The problem is that it expect a TLS connection with a self-signed certificate and a token that needs to be passed after the first time the remote was permitted. The following code works for me:
`import base64
import json
import logging
import time
import websocket
import ssl
from websocket import create_connection
class SamsungTV():
def init(self, host, port = 8002, name = "MyRemote", token = None):
uri = "wss://" + str(host) + ":"+ str(port) + "/api/v2/channels/samsung.remote.control?name=" + base64.b64encode(str.encode(name)).decode('utf-8')
if token != None:
uri += "&token=" + str(token)
self.connection = websocket.create_connection(uri , sslopt={"cert_reqs": ssl.CERT_NONE})
	response = json.loads(self.connection.recv())
	# print(response)
	if response['event'] != 'ms.channel.connect':
		self.close()
		raise Exception(response)
	self.token = str(response['data']['clients'][0]['attributes']['token'])

def __exit__(self, type, value, traceback):
	self.close()

def close(self):
	if self.connection:
		self.connection.close()
		self.connection = None

def get_token(self):
	return self.token

def send_key(self, key, repeat=1):
	for n in range(repeat):
		payload = json.dumps({'method': 'ms.remote.control','params': {'Cmd': 'Click','DataOfCmd': key,'Option': 'false','TypeOfRemote': 'SendRemoteKey'}})
		self.connection.send(payload)
		# time.sleep(1)

tv = SamsungTV("INSERT-IP-ADDRESS", token=None) # of course your IP, and leave token blank on first run, second run put the string in that it printed
print("Token: " + tv.get_token())
tv.send_key("KEY_HOME")
tv.send_key("KEY_INFO")
`

This is a very old library that has not been maintained in a long time. The reason why it has not been maintained is because Samsung sucks and they were changing the API and breaking things left and right. At one point in time their own application would not even pair to the TV's because of changes they made to the firmware for the TV's. Frankly I am kind of surprised it is even working at all.
The reason why things are coded how they are is because there is no real way of being able to collect the power state from the TV's and because there are several different types of commands that have to be sent to the TV depending on the year the TV was made.


Updates on the catalog

Signed-off-by: asubedy<EMAIL_ADDRESS>Description
This PR fixes following Issues:

Center the Cloud Native Catalog Heading and sub heading2.
Moving Circuit Breaker to Resiliency from Traffic Management
Related Card is now clickable
Card chip also darkens on over now
Latest Updated and Version text added on individual page
Menu on mobile view is now right aligned.
Overflowing pattern name on card is now wrapped
This PR fixes #

Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

✅
✅
❌ Clickable Related Patterns card. (I'm using Safari).
✅
✅
✅
✅

https://user-images.githubusercontent.com/7570704/179310422-1e3cd9fc-a28d-4e84-8bc7-4e0d7ab1c69f.mov

@asubedy almost there...

@leecalcote the clickable related card issue is due to interference of other elements on the single page. Once the flex box implementation is through, the cards will be clickable easily.

That’s good to hear.


Swagger doesn't appear to work with BASE_DIR

When defining a BASE_PATH = 'netbox/' the API doesn't appear to work.

I did adding a CORS response header. Nothing is returned, just "fetching resource list". I don't see any obvious error logging support, thoughts on how to debug this further?

Note that swagger is merely the frontend documentation generator for the API. The API itself should work fine.

Does not resolve the issue, patch applied - same problem.


Display a notice if database is significantly behind current time

We check the most recent timestamp in the database and if it's more than 1 hour back from the current time, we display a notice to the user.
Phabricator task: T216727

@Samwalton9 Thanks, I have made the changes. Sure, I will write a test for it. I will raise a separate PR for that soon.

Looks good!
Sounds great :)


Caching of helm chart index prevents the most recent builds from being installed

Is this a bug report or feature request?

Bug Report

Deviation from expected behavior:
We are serving a stale cache version of the master index.yaml.  It is not a problem with the local cache, but with a cache-control header.  See this thread for some hints on the problem.
https://github.com/helm/helm/issues/2453
Expected behavior:
https://charts.crossplane.io/master/index.yaml has much more recent builds listed in it: 0.0.0-628.02f0c23
How to reproduce it (minimal and precise):
When you curl, you will see that the most recent version is 0.0.0-615.5885d89
curl -v --silent https://charts.crossplane.io/master/index.yaml 2>&1 | less
https://github.com/helm/helm/issues/2453

https://github.com/upbound/build/pull/60 should solve this problem


Ignore media query blocks.

Possible fix to #23. YMMV no extensive testing has been done except that it worked in my environment after having the same issue. I figured since only top level CQs are supported this might fly.
Additionally I've added a prepare script to the package as this help my development flow.

Closing in favor of #33


how-to summary component

Component infos
Description
We're gonna use a howtoSummary component to allow user to easily navigate between the different steps that compose a how-to.
Page related
Will be used in the how-to post view (#420)
Mockup

Examples
You can find component with similar behavior on the pages of wikifab or wikihow
Build suggestion
Each step have an id html attribut which can be the title of the step. Then the component would reference an # anchor link to this id. No need to smooth scroll or any transition for now.

Hi
Would love to work on this. Can I pick it up please?
Sumedha

Hi @sumedhan,
Yes, I add the label in-progress to the issue. Thanks !

The # link in react router causes the page to re-render. Looking for advice on how I can implement it.
this package seems like an option.
Let me know how you want me to take it forward or if there is any other recommendation.
Appreciate the guidance.

Hey @sumedhan
I had a look at react-router-hash-link and it seems like a good solution, you can go with it :)

Closed with #435


 IF YOU DO NOT FOLLOW THE FORMAT OR MAKE A GENERIC TOKEN REQUEST YOUR ISSUE WILL BE DELETED WITHOUT COMMENT
YOUR JUSTLIST MUST FOLLOW THE JSON SPECIFICATION
https://github.com/justswaporg/justlists/blob/main/example.justlists.ts
Checklist

[x] I understand that this is not the place to request a token listing.
[x] I have tested that my JustList is compatible by pasting the URL into the add a list UI at justswap.org.
[x] I understand that filing an issue or adding liquidity does not guarantee addition to the justlists website.

Please provide the following information for your token.
JustList URL must be HTTPS.
JustList URL:  https://public.web3socket.com/tron/NRY/nry.json
JustList Name: nry.json
Link to the official homepage of the JustList manager: https://www.tronenergy.li
Contract Owner TLKR4kmZYi2koGm3MDXAXfuZNbooKAEbTM
Description : DAEC is an ecosystem composed of a series of decentralized energy applications, which is used to build a (Decentralized Autonomous Energy Community) that adapts to the future
HomePage : https://www.tronenergy.li
Email<EMAIL_ADDRESS>Twitter : https://twitter.com/daecenergy

Hi hiw do I locate my crypto currency which I have lost all my credentials for please regards Ricky

Sorry, your issue will be closed, if you want to add justlist please resubmit.


update docs

several updates:

update Documenter and fix docs (by adding MacroTools dependency)
use moer xcall API
fix typos
fix refs

Thanks for this. I think the functions which have docs should generally be added as exports; we should avoid having IRTools.Inner.* references in the docs, since things should either be exported and documented or neither.

thanks for the feedback :)

I think the functions which have docs should generally be added as exports; we should avoid having IRTools.Inner.* references in the docs, since things should either be exported and documented or neither.

Okay, I can follow your direction. So how about exporting everything with documentation ?
For now I found

Base.permute!
branch!
return!
returntype, deleteblock
are not exported but documented.

Thanks!


Allow argument flags to be passed to verfier

Issue Type

Feature request

Molecule and Ansible details
ansible 2.5.0

molecule, version 2.9.0

Molecule installation method (one of):

pip

Ansible installation method (one of):

pip

Desired Behavior
Currently arguments to the verifyer are specified as a dict and passed to the verfier as --key=vlaue pairs as briefly described on: https://molecule.readthedocs.io/en/latest/configuration.html#testinfra
I have a usecase where it is neccessary to just pass switch flags to the verifyer such as -v or --setup-show without specifying any values to be passed.
I can attempt a PR which provides the functionality if it makes sense to include such an option.
I imagine something like this but am very open to any other suggestions on the layout:
verifier:
  name: testinfra
  options:
    n: 1
  flags:
    - '-v'
    - '--setup-show'

You should be using the following syntax:
verifier:
  name: testinfra
  options:
    n: 1
    v: True
   setup-show: True


Added note re ip_forward and updated dashboard url

Description

For my install (raspbian buster) I had to also set net.ipv4.ip_forward=1 to get networking working between nodes.
Also updated the alternative dashboard url to the current one.
Motivation and Context

Fixed problems accessing dashboard when it was deployed anywhere other than the master node.

[ ] I have raised an issue to propose this change (required)

How Has This Been Tested?

Tested using raspbian buster fresh install and a 3 node cluster.  Not tested on earlier versions of raspbian.
Types of changes

[ x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ x] My code follows the code style of this project.
[ x] I have tested this change and prove it works
[ x] I've read the CONTRIBUTION guide
[ x] I have signed-off my commits with git commit -s

@chrisbu : Just stumbled accross this PR when I was trying to install the dashboard.
You might also want to adjust the ClusterRoleBinding above to:
apiVersion: rbac.authorization.k8s.io/v1beta1
kind: ClusterRoleBinding
metadata:
  name: kubernetes-dashboard-admin
  labels:
    k8s-app: kubernetes-dashboard
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: ClusterRole
  name: cluster-admin
subjects:
- kind: ServiceAccount
  name: kubernetes-dashboard
  namespace: kubernetes-dashboard


AuthLogParser doesn't working

Hello YosfanEilay,
I have tried to test the tool as per the recommendations on a Windows machine but it didn't work. I have used auth.log from 2 Linux machines. Out as below:
AuthLogParser Banner
Write-Output "    ___         __  __      __                "
Write-Output "   /   | __  __/ // /    / /   ____  ____ _ "
Write-Output '  / /| |/ / / / __/ __ \  / /   / __ / __ / ' Write-Output " / ___ / /_/ / /_/ / / / / /___/ /_/ / /_/ /  " Write-Output "/_/  |_\__,_/\__/_/ /_(_)_____/\____/\__, /   " Write-Output "       ____                         /____/    " Write-Output "      / __ \____ ______________  _____        " Write-Output '     / /_/ / __ / / / _ / /        '
Write-Output "    / / // / /  (  )  __/ /            "
Write-Output "   //    _,//  //_/_/             "
Write-Output "
Write-Output "        github.com/YosfanEilay"
Write-Output "            Version: 1.0v"
Write-Output "
NotFoundHashTable
$NotFoundHashTable = @{}
Variable to store where AuthLogParser is running from.
$RunningPath = Get-Location
save path from execution and move the variable to dot source -> CreateLogCopy.ps1
$AuthLogPath = $($args[0])
if statement to check if $args[0] is empty
if ($AuthLogPath.Length -ge 1) {
}
if empty, execute instructions
else {
Write-Output "[!] Auth.Log file not found."
Start-Sleep -Milliseconds 500
Write-Output "How to execute AuthLogParser ?"
Start-Sleep -Milliseconds 500
Write-Output "+----------------------------------------------------------------------------------------+"
Write-Output "| PS C:\Users{user}\Desktop\AuthLogParser> .\AuthLogParser.ps1 C:\PATH\TO\Auth.Log\File |"
Write-Output "+----------------------------------------------------------------------------------------+"
Write-Output ""
Start-Sleep -Milliseconds 500
exit
}
Dot Sourcing -> 01-TimePatch.ps1
. "$RunningPath\03-TimePatch\01-TimePatch.ps1"
if statement to check if TimePatch is needed
if ($CreateLogCopy_Flag -eq "True") {
Dot Sourcing -> CreateLogCopy.ps1
. "$RunningPath\01-LogCopy\CreateLogCopy.ps1"
}
Dot Sourcing -> FileSummaryReport.ps1
. "$RunningPath\02-Features\01-FileSummaryReport.ps1"
Dot Sourcing -> 02-EventNameTable.ps1
. "$RunningPath\02-Features\02-EventNameTable.ps1"
Dot Sourcing -> 03-IPAddressTable.ps1
. "$RunningPath\02-Features\03-IPAddressTable.ps1"
Dot Sourcing -> 04-SSHTable.ps1
. "$RunningPath\02-Features\04-SSHTable.ps1"
Dot Sourcing -> 05-UsersGroupsActivity.ps1
. "$RunningPath\02-Features\05-UsersGroupsActivity.ps1"
Dot Sourcing -> 06-GeneralActivity.ps1
. "$RunningPath\02-Features\06-GeneralActivity.ps1"
if ($NotFoundHashTable.Values.Count -ge 1) {
Element That Does Not Exist in This auth.log File
Write-Output "
Write-Output "Element That Does Not Exist in This auth.log File"
Write-Output "+-----------------------------------------------+"
$NotFoundHashTable.Values
}
delete the auth.log copy after using it.
Start-Sleep -Seconds 1
Remove-Item -Path $AuthLogCopyLocation
Please advise what is wrong with this.

Hello @GopalSati
Let me try to help, on your post I can only see a print of some of the code
Can you please explain what was the issue and maybe screenshot what happened when you executed the tool on a log?

@GopalSati
You can screenshot the error you get?

There is no error message, just get the output which was already shared in
the initial email. I haven't seen the results as per the video you shared.
On Wed, 10 Jan, 2024, 00:45 Eilay Yosfan (DFIR), @.***>
wrote:

@GopalSati https://github.com/GopalSati
You can screenshot the error you get?
—
Reply to this email directly, view it on GitHub
https://github.com/YosfanEilay/MasterParser/issues/1#issuecomment-1883636075,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKBCTJDZFC2UCSZLLYHLXETYNWJO5AVCNFSM6AAAAABBTHSXG6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBTGYZTMMBXGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

@GopalSati
Can you please explain how do you run it?
What is the argument you write on the PowerShell command line to execute the script ?
And are you running as administrator ?

@GopalSati Do you still have issues in MasterParser v2.0 ?


fix: update @smoya/multi-parser to v5.0.6

Closing in favor of https://github.com/asyncapi/cli/pull/1365 as @derberg requested.


Get base directory from Zotero settings

This is my suggestion concerning https://github.com/hanschen/rofi-zotero/issues/2.
It adds the option to specify the profile name in the case of multiple users. Furthermore, but probably not that importantly it allows to specify the Zotero dotfile location.

Thanks a lot for your contribution! I've been a bit too busy lately to take a proper look, but I'll try to review the changes soon.

This seems to work well. Thanks a lot for your contribution!


chore: move behaviors to @stardust/accessibility

Warnings

:warning:
There are no updates provided to CHANGELOG. Ensure there are no publicly visible changes introduced by this PR.

:warning:
New package.json added: packages/accessibility/package.json. Make sure you have approval before merging!

:warning:
Package (or peer) dependencies changed. Make sure you have approval before merging!

Changed dependencies are detected.
Changed dependencies in packages/react/package.json

package
before
after

@stardust-ui/accessibility
-
^0.39.0

  Generated by :no_entry_sign: dangerJS


QueryException during artisan migrate refresh

The tables associated with the roles/permissions are deleted in the wrong order. When running php artisan migrate:refresh exceptions get thrown:
[Illuminate\Database\QueryException]
SQLSTATE[23000]: Integrity constraint violation: 1217 Cannot delete or update a parent row: a foreign key constraint fails (SQL: drop table `roles`)

[PDOException]
SQLSTATE[23000]: Integrity constraint violation: 1217 Cannot delete or update a parent row: a foreign key constraint fails

in the migration file create_roles_permissions_tables.php the order needs to be switched from:
Schema::drop('roles');
Schema::drop('permissions');
Schema::drop('permission_role');
Schema::drop('role_user');

to
Schema::drop('permission_role');
Schema::drop('role_user');
Schema::drop('roles');
Schema::drop('permissions');

I put this change in a pull request.

@GregoryHGS Accepted :) Thank you.


Tweak log file names for consistency between Mono and non-Mono logs

Avoid spaces in Mono log file names.
Use a .log extension for Mono logs, just like non-Mono logs.
Use periods to separate hours/minutes/seconds for non-Mono logs.

#21895 made me realize there was a dedicated Mono logging system, so I took the time to make this more consistent. :stuck_out_tongue:

Thanks!

Cherry-picked for 3.2.4.


Added the documentation for Basics of Visual Design: Part 2 #1315

Explained how hue, chroma, value, saturation, tones, tints and shades affect the way we perceive colors.
Fixes Basics of Visual Design :Part 2 #1315

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] Java#Script
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[x] UI/UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

hey commit to the right branch

hey, can u pls tell which branch I have to commit.

hey, can u pls tell which branch I have to commit.

UI_UX


Version for Ctrader

Hi,
Hope can have version for Ctrader, because this tool is good.

I am not sure that it is codeable in cTrader right now. Besides, cTrader can calculate position size based on risk in the order form, no?

but this tool is better than ctrader build in order form. can calculate faster, more easy and also can have a look the risk all position.
is ok if not possible, just a suggest.

Interestingly, some months ago I started writing a simplified version for cTrader having been inspired by the MT5 version. cTrader does include a good order form, but only the SL/TP can be set using a % risk, the lots can not. So there is a need for it and I recently saw a survey from cTrader asking about priority of new features and one of them was placing an order based on % risk. So clearly a lot of people asked. It's one reason why I have to use MT5 vs cTrader. cTrader is a nicer more modern UI but lack of % risk order form is a pain.

I will consider creating a cTrader version, but I think Spotware will implement that as a core feature faster than I get it done.

https://www.earnforex.com/ctrader-robots/cTrader-Position-Sizer/


Add automountServiceAccountToken: true to fullstack OA serviceAccount

Description
Ticket: TBD
cherry pick of #4175
How can this be tested?
same as #4175

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 64.51%. Comparing base (4215cbe) to head (b5bbd6a).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4176   +/-   ##
=======================================
  Coverage   64.51%   64.51%
=======================================
  Files         399      399
  Lines       26790    26790
=======================================
  Hits        17284    17284
  Misses       8145     8145
  Partials     1361     1361

Flag
Coverage Δ

unittests
64.51% <ø> (ø)

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

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


Fix migration for Neutron

Neutron drivers use own naming for alembic migrations, e.g.
cisco_alembic_version, aci_alembic_version, etc depending on driver.
Therefore we should check for all alembic_ tables and not only those
which start with alembic_, otherwise those get purged and next DB
migration attempt will fail since schema is applied, but migration
version ids are back to initial ones.

Looks reasonable. @rhafer ?

I tested this in a Cloud 7 to Cloud 8 upgrade yesterday and it didn't cause any trouble. Merging.


Speed up using trigrams

Before comparing the file content using the Levenshtein or Jaro distance, first compare the two files using word-level trigrams to get the general sense of their similarity. Then, use the distance metric only on files that are relatively similar by trigrams.
Resources:

https://en.wikipedia.org/wiki/Trigram_search
https://en.wikipedia.org/wiki/Trigram

With version 0.5.0, the tool now pre-selects using character-level trigrams. As a result, the search is about 10 times faster.
Word-level trigrams could produce more accurate results and might even be faster, but no library can currently calculate them.
I'm leaving this open to consider word-level trigrams in teh future.

The slice::windows method would be quite useful when implementing word trigrams.


LSTM with Masking during Backprop

I am trying to train an LSTM with variable length inputs, so I am using a Masking Layer to deal with the length differences.
When I classify a padded sample, I get classifier output for the unpadded as well as the padded time steps (the classifier seems to repeat the output of the last non-padded time step onto the padded ones). I can understand the rationale behind this and I can manually chop off the dimensions that are masked. So far so good.
I was wondering what exactly happens during minibatch training, especially in the backprop step.
I would imagine that Keras/TF ignores all masked time steps when computing the loss, since the label data is zero padded but the forward pass output is not (as seen above), correct?
Now we backpropagate the loss through time. And here is the part I don't understand: What exactly does Keras do with samples that have masked time steps during backprop?
Are the samples removed for that time step? Are they used normally, completely ignoring the masking? If so, doesn't that cause problems, e.g. when a batch only has few long samples and a lot of short ones? Would it mess up the gradient for the few long samples?
Thanks for any clarification.

I realized that this is not really an issue so I moved the discussion over to here:
https://groups.google.com/forum/#!topic/keras-users/6URjfgu_E5A


Microsoft Teams screen sharing

📝 Provide a description of the new feature
What is the expected behavior of the proposed feature?  What is the scenario this would be used?
--- When screen sharing in Microsoft Teams, I want to be able to select one of my zones as the shared content. I have an ultrawide monitor, so FancyZones is perfect to take advantage of the screen. But when I go to share in Teams, I only have the option to share my whole desktop or individual apps. I don't want to share my whole desktop because it's too wide and will look bad on screen sharing and I also have content in another zone that I don't want people to see. If I select a specific app to share, then I don't have the flexibility to move other windows into my visible sharing area. I would have to stop sharing my screen, pick the new app, and then start sharing again.
If you'd like to see this feature implemented, add a 👍 reaction to this post.

Duplicate of https://github.com/microsoft/PowerToys/issues/2774

Duplicate of https://github.com/microsoft/PowerToys/issues/2774


[Feature Request] Allow Smart Glasses to render virtual minecraft blocks/entries

Describe your idea
This feature is for people who want to create easy 3D games in minecraft (what?)
Basically, there are two ways to implement it:

Glasses will mixin on the client World class, it do nothing on the server or other players client (other players can see and interact the player who are using the Glasses normally). So it's actually a VR goggle.
Glasses will create a new dimension, and teleport the Glasses player into the dimension. And computer will operate the dimension. So what will happen is when player using the glasses he/she will disappear from the world. maybe we have to add another block(s) called VR Station (which may require FE or fuel) to make it more normal and avoid randomly teleport.

Describe alternatives you've considered if you've any
No response
Additional context
No response
Linked Issues
No response

A complete VR Mode will probably not happen, that's just too much
I'll add it to the list anyways


Add $tomcat_protocol_ssl paramater for compatibility with Jira >= 7.3.

Tomcat connector protocol changed in Jira 7.3:
https://confluence.atlassian.com/jiracore/jira-core-7-3-x-upgrade-notes-861251104.html
This will allow you to set the correct connector value (org.apache.coyote.http11.Http11NioProtocol) when using Jira 7.3 and higher.

Hi @markleary, thanks for the PR. Are you able to rebase against latest master and afterwards take a look at the failing spec tests (if they even reoccur)?

Rebased, tests now pass.

Thanks!


Construct version identifier from CMake variables

I suggest to construct a version identifier from other CMake variables so that such important information can be better reused at more places.

Pull request please.

Closing issue pending pull request.


Can go-callvis be used on windows platform?

Can go-callvis be used on windows platform?
How to use go-callvis?
windows don't support "make install".
my email is<EMAIL_ADDRESS>

@zgq25302111 the make install target refers to: go install -tags $(GO_BUILD_TAGS) -ldflags "$(GO_LDFLAGS)" $(GO_BUILD_ARGS). This is a normal Go command and should work on windows (I don't have a windows machine to verify however). The values in $() refer to variables in the makefile. You will have to manually replace them with the values you need. If you don't know what to use just remove them.


bot-builder-howto-add-media-attachments needs updated to reflect changes in the Using cards sample bot.

See section: Code sample for processing Adaptive Card input
Note: There is no ChoiceValidator method.

I checked the

See section: Code sample for processing Adaptive Card input
Note: There is no ChoiceValidator method.

I checked the documentation and at line 465 it reads as follows:
This ChoiceValidator method is placed into the Using cards sample just after the closed brace public for declaration of MainDialog:
This is followed by a C# inline sample where the ChoiceValidator is defined.
So I do not understand the comment? What is the problem?

Closing, obsolete.


highlighting the bounding box

Is there anyway we could use the bounding box values to programmatically draw a box around the extracted value to show the user where the value was pulled from

Hi @rahulmadaan30 Thanks for reaching out. Since this channel is for driving improvements towards MS Docs, could you provide the URL of the specific documentation that your feedback is applicable to? That way, we are able to make sure your feedback is addressed by the right team.

https://docs.microsoft.com/en-us/azure/cognitive-services/form-recognizer/quickstarts/python-labeled-data. I am working on form recognizer tool and my problem statement is that if sometimes the tool is not able to identify a correct field then i want a way to atleast find what value did it pick instead of the one it was originally intended for.I know bounding box provides me the coordinates of rectangle but is there anyway or documentation that i can use to draw that bounding box rectangle on the pdf

@rahulmadaan30 This channel is for driving improvements towards MS Docs, for any product related question/issue I would recommend you to create a thread on the forums- MSDN / Stackoverflow. Once you post your issue on forums, it will have visibility across the community which is a better suited audience for such types of issues.


add firebase db to project

Added a firebase DB to the project and provided instructions for how to use it locally.
Closes #12

Thanks for the reviews @swellander and @MattPereira. Great feedback. Can one or both of you see if my changes are to your liking?

@escottalexander looks great! Thanks for making the changes


Boolean column behavior

I received the following message when attempting to submit a boolean:
DEPRECATION WARNING: You attempted to assign a value which is not explicitly true or false to a boolean column. Currently this value casts to false. This will change to match Ruby's semantics, and will cast to true in Rails 5. If you would like to maintain the current behavior, you should explicitly handle the values you would like cast to false. (called from create at /usr/local/bundle/bundler/gems/intermodal-ba96eb8e595f/lib/intermodal/concerns/controllers/resource.rb:37

derp.


Spike on Typed Supply Chains

What does a supply-chain-as-a-template look like
As per this gist:
https://gist.github.com/jwntrs/9957c0e99851428b4bf52510d5902285
Some open questions to sort out:

templates are typed outside of each option. how do we mix Cluster(Source|Image|Config)Template and Cluster(Source|Image|Config)SupplyChain types under options?
How do we pass inputs into a nested supply chain

if a supply chain has a top level selector it should be useable on its own, which means it can't have undefined inputs
if a supply chain has undefined inputs it cannot be used as a stand alone supply chain, how do we validate it?

@martyspiewak @emmjohnson were we able to answer any of the open questions when doing this spike?

templates are typed outside of each option. how do we mix Cluster(Source|Image|Config)Template and Cluster(Source|Image|Config)SupplyChain types under options?

I think we proved we can do this right?


TestModelAccuracy incorrectly named

There is an error in the code.  TestModelAccuracy should be TestAccuracy in order to run the code successfully.

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

ID: b54afee7-1e3b-98f3-c25d-c0d71b82bf97
Version Independent ID: b75b9ae6-de0e-acc3-b9cd-4918422bf7aa
Content: Use PyTorch to train your image classification model
Content Source: docs/windows-ml/tutorials/pytorch-train-model.md
Product: windows
GitHub Login: @QuinnRadich
Microsoft Alias: quradic

Thank you.


Add Haskell icons

Add icons for Haskell (.hs) and Literate Haskell (.lsh) files, as well as the flask icon for Haskell Specs.
Works fine with Atom Dark and Atom Material themes, but using the One Dark theme the Haskell icons are offset a bit to the bottom:

Not sure how to fix that, I tried what was done with the .go and .java icons and explicitly set top: 0px !important; what fixed the issue on One Dark but moved the icons too high on Atom Dark and Atom Material.

Haskell icons look a little bit too small to discern ... maybe bump up the font-size a bit? See https://github.com/lee-dohm/file-type-icons/blob/master/styles/icons.less#L32

Agreed, one question though:
Applying a larger font size to the haskell-icon class will increase the font size for all files in the tree ending with .hs, even the test files, which obviously shouldn't be scaled since they use a completely different icon.
Is there any way to tell Atom to only apply the .haskell-icon class to *.hs files but not *Spec.hs files?
Alternatively I could apply a font-size to the test-file-icon class, too, but I'd rather not do that if there's any way around it.

Nevermind, figured it out.
I set the icon's font-size to 155%, which makes it approximately as wide as the other icons.
The icon still appears too low on One Dark

Thanks for this!


fix: nil pointer reference when merge commit was not generated.

Summary
fix a condition where nil pointer reference occurs
About
If the pull request is closed with a conflict, a nil reference will occur because the test merge commit is not created.
Reference

https://docs.github.com/en/rest/guides/getting-started-with-the-git-database-api#checking-mergeability-of-pull-requests

Thank you very much.


getProtos work

Return inclusion info in getProtos, remove a lot of complexity and O(n) from getProtos

@aang114 @paulinechi can we merge this?


Reassembling a valid motion photo again?

[We can also move this to the discussion tab if you enable it, just wanted to brainstorm a bit with other interested people]
I looked through the original XDA thread as well as the source code but I didn't come to a solution.
If I understand the format correctly, its a full mp4 and full jpg file just slapped together with the MotionPhoto_Data divider in the middle. So manually reassembling a motion photo from two different images should be possible.
In a first step I tried assembling the image of one motion photo and the video of another together. My S10e recognizes it as a motion photo, but when pressing play it stays at 0:00 and does not play a video. I'm guessing there is some other metadata you have to adjust?
The ultimate goal would be to combine any picture and video into a motion photo, but as a first step I would try recombinin existing motion photos.

Of course shortly after making this post I find the first step myself:
You have to adjust the Container:Item Item:Length in the xmp metadata to the new videos length.

I honestly have no idea if there's any requirements as to tags on the JPEG half or codec or metadata on the MP4 side. I do the same as the XDA code, just looking for the "MotionPhoto_Data" divider and splitting it there. I also don't plan to add motion photo creation to this program.
You might try taking the unmodified jpg and mp4 that my program spits out, and simply tacking them back together with the divider in between. If you're assembling it with multiple files and copy /b, use a hex editor to make sure your divider is plain ASCII text and doesn't have any garbage stuck on like a UTF-8 BOM at the start or a line break at the end. The result should be byte identical to the input file since I don't mess with either half. Once you have that, you might try small changes like simply re-saving the jpg half and see if that breaks it somehow.

If you have no plans to add this to this program, I will be developing on my own package, thats fine.
Just taping them back together without any changes does not work, since there is XMP metadata. But it does seem to be a google standard. Couldn't find their documentation though, maybe its not public.


Can't infer the type of instance variable '@name' of Person

In section "Instance variables type inference" of Docs we can read the following:

Did you notice that in all of the previous examples we never said the types of a Person's @name and
@age? This is because the compiler inferred them for us.
Then we see an example of what is said above:
`class Person
getter name
def initialize(@name)
@age = 0
end
end
john = Person.new "John"
john.name #=> "John"
john.name.size #=> 4`

But, what's an irony, exactly about this code compiler says that it's invalid, saying

The type of a instance variable, if not declared explicitly with
@name : Type, is inferred from assignments to it across
the whole program.
The assignments must look like this:

@name = 1 (or other literals), inferred to the literal's type
@name = Type.new, type is inferred to be Type
@name = Type.method, where method has a return type
annotation, type is inferred from it
@name = arg, with 'arg' being a method argument with a
type restriction 'Type', type is inferred to be Type
@name = arg, with 'arg' being a method argument with a
default value, type is inferred using rules 1, 2 and 3 from it
@name = uninitialized Type, type is inferred to be Type
@name = LibSome.func, and LibSome is a lib, type
is inferred from that fun.
LibSome.func(out @name), and LibSome is a lib, type
is inferred from that fun argument.

Other assignments have no effect on its type.
Can't infer the type of instance variable '@name' of Person
def initialize(@name)

So it seems like Docs say that compiler must infer the type of incoming value for an instance variable by itself, but in practice we see that compiler can't do this and we should exactly specify the type. Who's wronge? The compiler or the Docs?

Outdated docs.

Should i read API instead? Aren't they outdated?

@smellyshovel read the book to understand Crystal and its philosophy, and read the API as reference material. You'll probably find outdated bits here and there, but they're the main sources of information about Crystal.
When you find something outdated, you can send PRs to this repo for API docs, and to https://github.com/crystal-lang/crystal-book for the book. Or, if you're not sure how to fix a problem, just open issues like this one.
I'm closing this in favor of crystal-lang/crystal-book#74


BF: console scripts

Hi @mcraig-ibme, I've just come across a quirk in the oxford_asl build process - the oxford_asl_roi_stats and oxford_asl_hadamard_decode executables were not being created in $FSLDIR/bin/ due to a typo in setup.py.
I've also changed the python install command to set --prefix $[FSLDEVDIR], for local development (for FSL >= 6.0.6 releases) - if you add this sitecustomize.py file into your $FSLDIR/lib/pythonX.Y/site-packages/ directory, then $FSLDEVDIR/lib/pythonX.Y/site-packages/ to the fslpython module search path. This sitecustomize.py file does not get installed automatically at present (you can find a copy of it in $FSLDIR/etc/fslconf/), but we may update the fslinstaller.py script to do so at some point.
Thanks!

Thanks for the fixes Paul, these scripts are rarely used but better to have them installed and working!


Hadoop 18712. upgrade to jetty 9.4.51 due to cve

Description of PR
How was this patch tested?
For code changes:

[x] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[x] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
10m 37s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+0 :ok:
xmllint
0m  1s

xmllint was not available.

+0 :ok:
shelldocs
0m  1s

Shelldocs was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ branch-3.3 Compile Tests _

+0 :ok:
mvndep
15m 53s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
23m 47s

branch-3.3 passed

+1 :green_heart:
compile
17m 50s

branch-3.3 passed

+1 :green_heart:
mvnsite
25m 17s

branch-3.3 passed

+1 :green_heart:
javadoc
7m  1s

branch-3.3 passed

+1 :green_heart:
shadedclient
29m 57s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 58s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
21m 42s

the patch passed

+1 :green_heart:
compile
17m 20s

the patch passed

+1 :green_heart:
javac
17m 20s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
20m 52s

the patch passed

+1 :green_heart:
shellcheck
0m  0s

No new issues.

+1 :green_heart:
javadoc
6m 47s

the patch passed

+1 :green_heart:
shadedclient
30m 38s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
741m  2s
/patch-unit-root.txt
root in the patch passed.

+1 :green_heart:
asflicense
1m 36s

The patch does not generate ASF License warnings.

961m  1s

Reason
Tests

Failed junit tests
hadoop.hdfs.qjournal.server.TestJournalNodeSync

hadoop.hdfs.tools.TestDFSAdmin

hadoop.yarn.server.nodemanager.containermanager.logaggregation.TestLogAggregationService

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5585/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5585

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell detsecrets xmllint shellcheck shelldocs

uname
Linux 3fae56a1bd80 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.3 / ce803ef58b4b8627502593244abcc3e751ee5a82

Default Java
Private Build-1.8.0_362-8u362-ga-0ubuntu1~18.04.1-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5585/1/testReport/

Max. process+thread count
3336 (vs. ulimit of 5500)

modules
C: hadoop-project . U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5585/1/console

versions
git=2.17.1 maven=3.6.0 shellcheck=0.4.6

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

This message was automatically generated.

It isn't letting me access the test results, something broken for this run, neither it is letting me retrigger, @pjfanning can you rebase and push once?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 50s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+0 :ok:
xmllint
0m  0s

xmllint was not available.

+0 :ok:
shelldocs
0m  0s

Shelldocs was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ branch-3.3 Compile Tests _

+0 :ok:
mvndep
16m  8s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
24m  2s

branch-3.3 passed

+1 :green_heart:
compile
17m 51s

branch-3.3 passed

+1 :green_heart:
mvnsite
25m 56s

branch-3.3 passed

+1 :green_heart:
javadoc
7m  1s

branch-3.3 passed

+1 :green_heart:
shadedclient
29m 48s

branch has no errors when building and testing our client artifacts.

-0 :warning:
patch
30m  8s

Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+0 :ok:
mvndep
1m  1s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
21m 50s

the patch passed

+1 :green_heart:
compile
17m 14s

the patch passed

+1 :green_heart:
javac
17m 14s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
20m 47s

the patch passed

+1 :green_heart:
shellcheck
0m  0s

No new issues.

+1 :green_heart:
javadoc
6m 47s

the patch passed

+1 :green_heart:
shadedclient
30m 23s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
734m 30s
/patch-unit-root.txt
root in the patch passed.

+1 :green_heart:
asflicense
1m 36s

The patch does not generate ASF License warnings.

945m 40s

Reason
Tests

Failed junit tests
hadoop.fs.contract.router.web.TestRouterWebHDFSContractCreate

hadoop.yarn.client.api.impl.TestAMRMClient

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5585/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5585

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell detsecrets xmllint shellcheck shelldocs

uname
Linux 76386fd69d2f 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.3 / 0587ca8924c70b4718453f2798c42c66b9e420f9

Default Java
Private Build-1.8.0_362-8u362-ga-0ubuntu1~18.04.1-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5585/2/testReport/

Max. process+thread count
3186 (vs. ulimit of 5500)

modules
C: hadoop-project . U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5585/2/console

versions
git=2.17.1 maven=3.6.0 shellcheck=0.4.6

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

This message was automatically generated.

@ayushtkn I did a new run - there was a merge conflict that needed fixing

Failing tests passed locally
[INFO] -------------------------------------------------------
[INFO]  T E S T S
[INFO] -------------------------------------------------------
[INFO] Running org.apache.hadoop.fs.contract.router.web.TestRouterWebHDFSContractCreate
[WARNING] Tests run: 16, Failures: 0, Errors: 0, Skipped: 2, Time elapsed: 15.47 s - in org.apache.hadoop.fs.contract.router.web.TestRouterWebHDFSContractCreate
[INFO]
[INFO] Results:
[INFO]
[WARNING] Tests run: 16, Failures: 0, Errors: 0, Skipped: 2
[INFO] Running org.apache.hadoop.yarn.client.api.impl.TestAMRMClient
[WARNING] Tests run: 63, Failures: 0, Errors: 0, Skipped: 1, Time elapsed: 447.736 s - in org.apache.hadoop.yarn.client.api.impl.TestAMRMClient
[INFO]
[INFO] Results:
[INFO]
[WARNING] Tests run: 63, Failures: 0, Errors: 0, Skipped: 1

Merging, Thanx @pjfanning


HADOOP-18424. Fix org.apache.hadoop.io.ArrayWritable

When we store an array of string with "new ArrayWritable(String[] strings)", the "valueClass" will be "Text" while the actual value type is "UTF8". This will cause an error on deserialization of ArrayWritable.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 55s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
37m 43s

trunk passed

+1 :green_heart:
compile
23m 13s

trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
compile
22m  7s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
checkstyle
1m 24s

trunk passed

+1 :green_heart:
mvnsite
2m  9s

trunk passed

+1 :green_heart:
javadoc
1m 27s

trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javadoc
0m 59s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
3m  9s

trunk passed

+1 :green_heart:
shadedclient
23m 25s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m  5s

the patch passed

+1 :green_heart:
compile
22m 27s

the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javac
22m 27s

root-jdkPrivateBuild-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS> with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS> generated 0 new + 2854 unchanged - 1 fixed = 2854 total (was 2855)

+1 :green_heart:
compile
20m 37s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
javac
20m 37s

root-jdkPrivateBuild-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 generated 0 new + 2648 unchanged - 1 fixed = 2648 total (was 2649)

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
1m 21s
/results-checkstyle-hadoop-common-project_hadoop-common.txt
hadoop-common-project/hadoop-common: The patch generated 6 new + 12 unchanged - 0 fixed = 18 total (was 12)

+1 :green_heart:
mvnsite
1m 58s

the patch passed

+1 :green_heart:
javadoc
1m 18s

the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javadoc
1m  6s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
2m 54s

the patch passed

+1 :green_heart:
shadedclient
22m 38s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
18m 31s

hadoop-common in the patch passed.

+1 :green_heart:
asflicense
1m 15s

The patch does not generate ASF License warnings.

212m 51s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4809/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/4809

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux e9ef96a71075 4.15.0-191-generic #202-Ubuntu SMP Thu Aug 4 01:49:29 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 60e6cc9530836d4abcb0edd4072eee9c5fcfb86c

Default Java
Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS> /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4809/1/testReport/

Max. process+thread count
2201 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4809/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

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

This message was automatically generated.

no idea. ask owen.


Redirect after Enketo Express edit form record loads page not found

Using the new Enketo Express on zebra.ona.io, the edit form submission in Enketo Express works, but the redirect after you submit the edited record loads this error page:

The edit is successful though, and one can see the edited record on Ona.

This is a zebra issue related to the returnUrl that is called once a submission is editted

Is this in zebra or core? Where in code do/can we set the returnUrl value?

Also, according to @ivermac (and the docs, http://apidocs.enketo.org/#/post-instance) this should be return_url

I made pull request that fixes this problem which was merged and added to this release


[BEAM-2684] Upgrade to qpid proton-j 0.16.0, add timeout and additional messages to track potential issues in AmqpIOTest

Follow this checklist to help us incorporate your contribution quickly and easily:

[X] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[X] Each commit in the pull request should have a meaningful subject line and body.
[X] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[X] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[X] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[X] If this contribution is large, please file an Apache Individual Contributor License Agreement.

R: @jkff

I ran in fork mode tests until failure without problem based on this PR.

Coverage decreased (-0.02%) to 69.958% when pulling 6f3e64ac792f378e0f6657beefb91f84ccdf6502 on jbonofre:BEAM-2684-AMQPIO-TEST into 04f5bc6f805868f9ed1494128b90d33bb2fe0e66 on apache:master.

Superseded by #3714


Development

@cnagadya
@probuse
@adelinetush
@Takia5
@millymelisa97
@anyric
Please review my changes

Coverage increased (+15.9%) to 83.069% when pulling ad42703608d8ff8b15b60739aad865d4735f14cb on development into d4a3774eb16cc86bcc4408a4639cf190faabcaf6 on master.

Quite the pace! Amazing! Carry on. You deserve it!


Complex accessor not working correctly when having object attributes containing numbers

Release number or git hash: 3.2.2 (seems to also be a problem in version 4)
Web browser version and OS:

Steps to reproduce
see https://github.com/lineupjs/lineupjs/blob/master/src/provider/ACommonDataProvider.ts#L7-L26

create nested data where each item has a nested object with key value pairs, the nested objects need to have keys with numbers in them, e.g.

interface INestedDatum {
  b: number;
  b0: number; // access to this number won't work
}

interface IDatum {
  a: number;
  b: INestedDatum
}

type Data = IDatum[];

create a columns with a complex accessor (e.g. b.b, b.b0, etc.)
adding those columns to LineUp will end up in showing the results for simple and complex accessors except those complex accessors containing numbers in their name

see also https://codepen.io/lehnerchristian/pen/LYYPpLy
Observed behavior

The column with the complex accessor containing a number in the key shows undefined

Expected behavior

The column should simply show its data.
The intention to parse numbers out of the keys is likely to be used as array indices, but those should probably be encoded as b.b[0] instead of b.b0.
Furthermore an accessor of b.b0b also won't work, because the regular expression only checks if there is a number anywhere in the accessor and not at the end

we might wanna switch to https://lodash.com/docs#get to avoid this kind of issues in the future

Alternatively we could also use plain JS function.


ref(cmd): use api.NewRoutable() to instantiate Routable field

supercedes https://github.com/deis/workflow-cli/pull/170
see https://github.com/deis/controller-sdk-go/pull/73

blocked by https://github.com/deis/workflow-cli/pull/160

Current coverage is 5.35% (diff: 0.00%)

Merging #172 into master will decrease coverage by <.01%

@@            master      #172   diff @@
========================================
  Files           46        46
  Lines         3286      3289     +3
  Methods          0         0
  Messages         0         0
  Branches         0         0
========================================
  Hits           176       176
- Misses        3088      3091     +3
  Partials        22        22

Powered by Codecov. Last update 55247cf...5dadae2


Failed to resolve: com.github.Ikhiloya:NigeriaStatesAndLGAs:0.1.1

Failed to resolve: com.github.Ikhiloya:NigeriaStatesAndLGAs:0.1.1.. how do I resolve it ??

@hectorjoey kindly use it like this ::
implementation 'com.github.Ikhiloya:NigeriaStatesAndLGAs:0.1.1'

@hectorjoey could you please try what was recommended by @opezyhick. If that doesn't work, please provide more information

@Ikhiloya.. that's what I have in the application. I was thinking if it had to do with the android studio

It appears you haven't imported the library, could you please add jitpack url to your build.gradle file like this
allprojects {
repositories {
google()
centre()
//this line
maven {
url 'https://jitpack.io'
}
}

}


Add image tint key

When the image tint key is set, we render the image as a template and tint the image view
Also add slider & on/off switch to the demo application

Great, thanks!


Cannot get console to work

Hi,
I succesfully started the server and a node. But the console doesn't want to work.
/home/storm/.rvm/gems/ruby-1.8.7-p352@storming/gems/activesupport-2.3.14/lib/active_support/dependencies.rb:466:in `load_missing_constant': uninitialized constant CloudCrowd::AssetStore (NameError)
Can anyone help out?
ER

I have the same problem with the default file generate by crowd install:
:central_server:    http://localhost:9173
:max_workers:       5
:storage:           filesystem
I deleted the file to check if crowd was found it, and I think it is found, because the error when I remove it is:
crowd can't find the CloudCrowd configuration directory. Please use crowd -c path/to/config, or run crowd from inside of the configuration directory itself
I am not trying to use S3 in the example, so I think it is not the problem.. Someone can help me?

Same here

Somehow works while not requiring 'tmpdir'
https://github.com/vrybas/cloud-crowd/commit/ce5d695b096d7a4d86fbad2aaa8ddc5da38b336b