Update service-fabric-cluster-creation-via-portal.md

The link for this content is broken:
c. The minumum number of VMs for the primary node type is driven by the reliablity tier you choose. The default for the reliablity tier is Silver. Read more on how to [choose the Service Fabric cluster reliability and durability](service-fabric-cluster-reliability-and-durability.md) document.

Hi @murdockcrc, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@chackdan - Chacko, could you review this public PR and submit an update to address the broken link? The target article does not exist and does not have a redirect.

@ChackDan - Second notice, please review and submit an update

Pull # 24236 submitted

@murdockcrc  - Thanks for pointing this out, author will fix in another PR.


Change compute_stats

Possible improvement, parse git status --porcelain only once. Something similar to below.
# TODO: Use porcelain for status, saves two commands
#  conflicts = 0
#  untracked = 0
#  staged = 0
#  changed = 0
#  for line in run_cmd(['git', 'status', '--porcelain']).splitlines():
    #  if line[0] in ['A', 'M']:
        #  changed += 1
    #  if line[0] in ['A', 'M']:
        #  staged += 1
    #  if line[0:2] == '??':
        #  untracked += 1

Note there may be a double counting of conflicts/changed. Investigate and make a test case to cover.


Failed minikube system wide installation on Fedora 40

I have installed minikube 0.3.0 extension on podman desktop (1.10.2) and now I would like to install the binary:
![minikube_failed_install_fedora_system_wide](https://github.com/containers/podman-desktop/assets/19164299/00d9e730-
de07-4848-bca6-5f4245379a79)
Console log:
main ↪ [minikube] Made /home/odockal/.local/share/containers/podman-desktop/extensions-storage/podman-desktop.minikube/minikube-linux-amd64 executable
VM5:2 main ↪ [minikube] Failed to install 'minikube' binary: Command execution failed with exit code 1: Command execution failed with exit code 1
console.<computed> @ VM5:2
VM5:2 main ↪ [minikube] Command execution failed with exit code 1: Command execution failed with exit code 1
    at ChildProcess.<anonymous> (/app/lib/io.podman_desktop.PodmanDesktop/resources/app.asar/packages/main/dist/index.cjs:336:13044)
    at ChildProcess.emit (node:events:518:28)
    at ChildProcess._handle.onexit (node:internal/child_process:294:12)
console.<computed> @ VM5:2
VM5:2 main ↪ [minikube] Error: Portal call failed: Failed to start command: Failed to execute child process “minikube-linux-amd64” (No such file or directory)

    at ChildProcess.<anonymous> (/home/odockal/.local/share/containers/podman-desktop/plugins/ghcriocontainerspodmandesktopextensionminikube/dist/extension.js:132:16)
    at ChildProcess.emit (node:events:518:28)
    at maybeClose (node:internal/child_process:1105:16)
    at Socket.<anonymous> (node:internal/child_process:457:11)
    at Socket.emit (node:events:518:28)
    at Pipe.<anonymous> (node:net:337:12)
console.<computed> @ VM5:2
2VM5:2 main ↪ [minikube] minikube extension is active

I run into the same problem on fedora 40
if ((system === 'linux' || system === 'darwin') && !fs.existsSync(localBinDir)) {
    if (system === 'darwin') {
      args.unshift('mkdir', '-p', localBinDir, '&&');
    } else {
      // add mkdir -p /usr/local/bin just after the first item or args array (so it'll be in the -c shell instruction)
      args[args.length - 1] = `mkdir -p /usr/local/bin && ${args[args.length - 1]}`;
    }
  }

the code must run into this condition, but the local bin dir does exist, anyway with manual cp it works.

I run into the same problem on fedora 40
if ((system === 'linux' || system === 'darwin') && !fs.existsSync(localBinDir)) {
    if (system === 'darwin') {
      args.unshift('mkdir', '-p', localBinDir, '&&');
    } else {
      // add mkdir -p /usr/local/bin just after the first item or args array (so it'll be in the -c shell instruction)
      args[args.length - 1] = `mkdir -p /usr/local/bin && ${args[args.length - 1]}`;
    }
  }

the code must run into this condition, but the local bin dir does exist, anyway with manual cp it works.

I have tried to debug the code and you are right. Even though the directory exists, the function fs.existsSync(localBinDir) returns false.
What is totally puzzling me is that if I try the same check on the cli via node, it correctly returns true. It might be a problem with a permission when called from an extension.
Also, args[args.length - 1] = mkdir -p /usr/local/bin && ${args[args.length - 1]}; is wrong, as it adds a command mkdir where arguments for cp are expected.


[SHIRO-820] - Fix Log4j2 logs format

When using the utility, I noticed that the logs were displayed incorrectly.
I found a similar problem in Jira, but I think the [INFO ] output is correct, but the space after it is a consequence of incorrect logger settings, which I corrected.
Also update log message.

Following 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 [SHIRO-XXX] - Fixes bug in SessionManager,
where you replace SHIRO-XXX with the appropriate JIRA issue. Best practice
is to use the JIRA issue title in the pull request title and in the first line of the commit message.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Run mvn verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[x] If you have a group of commits related to the same change, please squash your commits into one and force push your branch using git rebase -i.
[ ] Committers: Make sure a milestone is set on the PR

Trivial changes like typos do not require a GitHub issue (javadoc, comments...).
In this case, just format the pull request title like [DOC] - Add javadoc in SessionManager.
If this is your first contribution, you have to read the Contribution Guidelines
If your pull request is about ~20 lines of code you don't need to sign an Individual Contributor License Agreement
if you are unsure please ask on the developers list.
To make clear that you license your contribution under the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

[x] I hereby declare this contribution to be licenced under the Apache License Version 2.0, January 2004
[x] In any other case, please file an Apache Individual Contributor License Agreement.

Interesting. looking at this, the old hasher CLI tool just gave the hash as the output. Now, we get an additional [INFO ] prefix. For parsing, leaving out the prefix would be beneficial.
Then, the -5level is probably to align output nicely:
[DEBUG] message
[INFO ] hash: …
[WARN ] a warning
[ERROR] an error

That said, I'd prefer to rather change the pattern in log4j.xml to just %msg%n for compatibility and leave the LOG.info statement as-is for easier parsing. Interesting. looking at this, the old hasher CLI tool just gave the hash as the output. Now, we get an additional [INFO ] prefix. For parsing, leaving out the prefix would be beneficial.
Then, the -5level is probably to align output nicely:
[DEBUG] message
[INFO ] hash: …
[WARN ] a warning
[ERROR] an error

That said, I'd prefer to rather change the pattern in log4j.xml to just %msg%n for compatibility and leave the LOG.info statement as-is for easier parsing.

Thanks for the answer @bmarwell!
No, before this the hash output looked like this
[INFO ] some_hash

In my opinion, the presence of a prefix is actually not correct, as was described in the SHIRO-820 issue.
I think that the best solution would be to display the hash in System.out without any prefix.
Then that will solve SHIRO-820.
In this case, there is no need to change the log4j2.xml
Just change LOG.info to System.out.println
If you agree with my decision, I'm ready to correct it.

fixed as per @bmarwell

Thank you @ksilisk for your contribution!


python311Packages.pyenphase: 1.11.4 -> 1.12.0

Diff: https://github.com/pyenphase/pyenphase/compare/refs/tags/v1.11.4...v1.12.0
Changelog: https://github.com/pyenphase/pyenphase/blob/1.12.0/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 260623 run on x86_64-linux 1

3 packages built:

home-assistant-component-tests.enphase_envoy
python311Packages.pyenphase
python311Packages.pyenphase.dist

Result of nixpkgs-review pr 260623 run on x86_64-linux 1

3 packages built:

home-assistant-component-tests.enphase_envoy
python311Packages.pyenphase
python311Packages.pyenphase.dist


Memory usage is too large

Is there any way to solve the problem of excessive memory usage?😵😵

Is the memory usage relatively stable over time? If there's an obvious memory leak, I can try to look into that. However, I doubt that there's anything I can do about high memory usage in general. Most of the memory should be completely managed by the ActionScript compiler and would be out of my hands.

Memory is stable, so that should be a problem with the ActionScript compiler. thank


Use override directive to allow passing in CFLAGS and CPPFLAGS.

Thanks to @mszczodrak for identifying that mbedtls was not building properly.

LGTM :+1:


[CI Image] Update ci_gpu

[tlcpackstaging/ci_gpu:20220617-055916-1b8f3b54c]

[ ] S0. image generated by PR 11744 ad dnnl into ci/gpu, blocked PR Pipeline executor tutorial

[ ] S1. Tag of nightly build: tlcpackstaging/ci_gpu:20220617-055916-1b8f3b54c

[ ] S2. The nightly is built on TVM commit: TVM_COMMIT: 1b8f3b54c6da0bf25427a8bd9a049d87318a1e66. Detailed info can be found here: https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-11744/1/pipeline/49

[ ] S3. Testing the nightly image on ci-docker-staging: TBC

[ ] S4. Retag TAG to VERSION:

docker pull tlcpackstaging/ci_gpu:20220617-055916-1b8f3b54c
docker tag tlcpackstaging/ci_gpu:20220617-055916-1b8f3b54c tlcpack/ci_gpu:20220617-055916-1b8f3b54c
docker push tlcpack/ci_gpu:20220617-055916-1b8f3b54c

[ ] S5. Check if the new tag is really there: https://hub.docker.com/u/tlcpack

[ ] S6. Submit a PR updating the IMAGE_NAME version on Jenkins

cc @masahi @areusch @driazati

working on this

@huajsj i think that CI run didn't validate properly. have to see the image validate before we can push it

I have a PR up for this https://github.com/apache/tvm/pull/11786.

thanks @masahi @areusch.


feat: operator deploys servicemonitor

This PR adds a feature to the operator to deploy a servicemonitor to monitor the kepler exporter service.
It also modifies grafana deployment script to remove its deployment of servicemonitor and rely on the
ServiceMonitor deployed by the operator.

@vprashar2929 FYI

This is such a welcome addition so we don't have to set it up and deploy it separately! I was just wondering if there are any uses cases / scenarios where users might not want or need the ServiceMonitor? If so would it be possible to make it configurable but deploy by default? Just some thoughts :)

@BGrasnick  thanks, yes, I did think of it but felt to cater for the most frequent usecase. We can definitely (based on feedback) config specific to configuring and deploying servicemonitor.


Provide initial Open API 3.0 Definition

Hand-Made Definition Available here or the raw link is available here. Note that this is a yaml file, however, it can also be json. Yaml was simply easier to write by hand.
Last updated 2022-12-11

[ ] OAuth scopes need to match actual in-use scopes
[ ] Authentication endpoint needs updated since it has now gone live

Ideally, this definition can be generated from the JSDocs eventually, and can live on a swagger.io instance on the front or backend site(s).
I recommend importing this into Postman to attempt to use it. Postman -> Ctrl + O -> File -> Downloaded file path OR Link -> Raw Link -> Show Import Settings -> Request/Response parameter generation -> Schema

and can live on a swagger.io instance on the front or backend site(s).

Done in #16


VCF reporting for haploid samples

Hi, I recently ran a bacterial sample (nanopore sequenced) through Clair CallVarBam to find variants in a gyrA gene. I used the --haploid flag and the resulting vcf file is shown below. My question is why the genotype (GT) is still reported as 1/1 (suggesting diploidity, no?) instead of only 1? Maybe I misunderstand how the genoype is reported or that the data has evidence that suggests diploidity. Thanks in advance.
##fileformat=VCFv4.1
##FILTER=<ID=PASS,Description="All filters passed">
##FILTER=<ID=LowQual,Description="Confidence in this variant being real is below calling threshold.">
##ALT=<ID=DEL,Description="Deletion">
##ALT=<ID=INS,Description="Insertion of novel sequence">
##INFO=<ID=SVTYPE,Number=1,Type=String,Description="Type of structural variant">
##INFO=<ID=LENGUESS,Number=.,Type=Integer,Description="Best guess of the indel length">
##FORMAT=<ID=GT,Number=1,Type=String,Description="Genotype">
##FORMAT=<ID=GQ,Number=1,Type=Integer,Description="Genotype Quality">
##FORMAT=<ID=DP,Number=1,Type=Integer,Description="Read Depth">
##FORMAT=<ID=AF,Number=1,Type=Float,Description="Estimated allele frequency in the range (0,1)">
##contig=<ID=gyrA>
#CHROM	POS	ID	REF	ALT	QUAL	FILTER	INFO	FORMAT	SAMPLE
gyrA	262	.	G	A	1015	.	.	GT:GQ:DP:AF	1/1:1015:188:0.7713
gyrA	491	.	T	TC	516	.	.	GT:GQ:DP:AF	1/1:516:188:0.1596

Ah, it a bug and it should be outputted as '1'. Before a fix, please consider '1/1' as '1'.

Fixed in the new version v1.1.


Split Documentation sidebar up into sections

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2236
Originally created by @DouweM on 2020-08-12 15:58:12

Once the new pages described in this epic have been added, I imagine it looking something like this:
- What is Meltano?
- Getting Started
- Tutorials
---
- Your Meltano project
- Plugins
---
- Installation
- Plugin management
- Configuration
- Data Integration (EL)
- Data Transformation (T)
- Orchestration
- Containerization
- Deployment in Production
- Analysis
---
- CLI reference
- `meltano.yml` reference
- Settings reference
- Architecture
---
- Getting Help
- Responsible Disclosure Policy
- Contributor Guide
- Team Handbook

View 3 previous comments from the original issue on GitLab


Panning is broken in most cases

Bug report
Codepen

Case 1 (series with numbers on x axis): https://codepen.io/jaksim/pen/QWbLoMQ
Case 2 (series with datetime on x axis): https://codepen.io/jaksim/pen/GRJKeNq

Some of the official examples that are broken

https://apexcharts.com/javascript-chart-demos/line-charts/zoomable-timeseries/ (case 2)
https://apexcharts.com/javascript-chart-demos/line-charts/line-chart-annotations/ (case 2)
https://apexcharts.com/javascript-chart-demos/line-charts/syncing-charts/ (case 2)
https://apexcharts.com/javascript-chart-demos/line-charts/stepline/ (case 1)
https://apexcharts.com/javascript-chart-demos/line-charts/gradient/ (case 2)
https://apexcharts.com/javascript-chart-demos/area-charts/spline/ (case 2)
https://apexcharts.com/javascript-chart-demos/area-charts/datetime-x-axis/ (case 2)
https://apexcharts.com/javascript-chart-demos/area-charts/negative/ (case 2)
https://apexcharts.com/javascript-chart-demos/column-charts/column-with-rotated-xaxis-labels/ (case 1)

Explanation
Case 1

Click on zoom in button once.
Switch to panning tool
Now try to drag your view to the right.
You will not be able to get the right most point into view again.

Case 2

Click on zoom in button once.
Switch to panning tool.
Try to drag your to the right.
You will be teleported to 1970.

Through bisect, I've found that Case 2 was broken by commit 0849cc3c805a64205e64ded892bbf734f5e613cd
Case 1 doesn't seem to behave very well before that commit either, though I can at least see the last data point. Just the x axis labels show ugly fractions, but that seems to be solvable from client code.

That's a major one.
Will look into it asap


A vulnerability was found with unauthorized access to some API endpoints, causing some sensitive information to leak

I found that some API endpoints could be accessed without logging in, giving away sensitive information
For example：
/api/get-resources?
/api/get-organizations?
/api/get-application?
The poc is:
curl "https://door.casdoor.com/api/get-organizations?owner=admin&p=&pageSize=&field=&value=&sortField=&sortOrder="

curl "https://door.casdoor.com/api/get-resources?owner=built-in&user=admin&p=1&pageSize=10&field=&value=&sortField=&sortOrder="

curl "https://door.casdoor.com/api/get-application?id=admin/app-built-in"

Hi @L0ading-x these batch GET APIs are not security vulnerability because the organization and application info are supposed to be publicly accessed to render the login page. The resource also has no reason to cause data leak. The image URL of that resource is always open to be accessed. However, this issue is not meaningless given that the batch API can still be restricted.

While organizational and application information should be publicly accessible, there is another interface, the user information interface, that can still be accessed without authorization
poc is:
curl "https://door.casdoor.com/api/get-users?owner=casbin&p=&pageSize=&field=&value=&sortField=&sortOrder="
The OWNER field represents the organization in the POC
When I get the organization information through the API above, I can use the owner field to display all the user information within the organization
For example:
curl "https://door.casdoor.com/api/get-users?owner=v2tl&p=&pageSize=&field=&value=&sortField=&sortOrder="
You can obtain some information about users within the organization v2TL

Similar as above, getting a user is also public.

You marked it as unfixable, but I found that access to some interfaces has been restricted


cluster provider interface definition

POC for the Discussion of #245

@jbpratt @reetasingh @ShwethaKumbla @vladimirvivien  Sorry I have been holding hostage a bunch of PRs because of this work still being pending. I have been out sick for a while and just getting back to the routine. Pushing a bunch of changes to this as we speak

Hi all 👋
I think this is going to be an import PR for future integration with cluster provides (i.e Kind, K3D, etc)
@harshanarayana  How is this PR doing? What else is needed to get it over the line.
(sorry I have been missing in action on this)

@vladimirvivien nothing more pending on my end for this for now. Waiting for a review

hey @harshanarayana @vladimirvivien, I'm curious what you think about taking a generics approach to this problem https://go.dev/play/p/bEYNyo9lucH

hey @harshanarayana @vladimirvivien, I'm curious what you think about taking a generics approach to this problem https://go.dev/play/p/bEYNyo9lucH

That's exactly what this PR is implementing isn't it?

hey @harshanarayana @vladimirvivien, I'm curious what you think about taking a generics approach to this problem https://go.dev/play/p/bEYNyo9lucH

That's exactly what this PR is implementing isn't it?

As far as I can tell, you aren't using generics but maybe I'm missing it? You maintain a map of providers and take the provider name as a parameter to fetch it out of the map.
https://github.com/kubernetes-sigs/e2e-framework/blob/575c4a4a57736e9d33a5bd2907c410751dbf6d95/pkg/envfuncs/provider_funcs.go#L85-L87
https://github.com/kubernetes-sigs/e2e-framework/blob/575c4a4a57736e9d33a5bd2907c410751dbf6d95/support/provider.go#L25
My recommendation is slightly different in that we depend on type safety rather than a map at runtime

hey @harshanarayana @vladimirvivien, I'm curious what you think about taking a generics approach to this problem https://go.dev/play/p/bEYNyo9lucH

That's exactly what this PR is implementing isn't it?

As far as I can tell, you aren't using generics but maybe I'm missing it? You maintain a map of providers and take the provider name as a parameter to fetch it out of the map.
https://github.com/kubernetes-sigs/e2e-framework/blob/575c4a4a57736e9d33a5bd2907c410751dbf6d95/pkg/envfuncs/provider_funcs.go#L85-L87
https://github.com/kubernetes-sigs/e2e-framework/blob/575c4a4a57736e9d33a5bd2907c410751dbf6d95/support/provider.go#L25
My recommendation is slightly different in that we depend on type safety rather than a map at runtime

Ah I see.. Sorry I totally missed that you were referring to generics.. We can do that with generics. But that limits the go version for someone adding this as a dependency on their libraries though.. Wouldn't it?

@jbpratt in second thought, I think that would be a great idea.. Do we need to worry about supporting go version less that 1.18? If not, I'll switch the PR to generics

@jbpratt on second thought, I think that would be a great idea.. Do we need to worry about supporting go version less that 1.18? If not, I'll switch the PR to generics

It does limit dependents to be >=1.18 but our go.mod already does that https://github.com/kubernetes-sigs/e2e-framework/blob/main/go.mod#L3 so I don't think it should be a problem! Everything older than 1.19 is EOL anyway.
Sounds good! Thank you for your work on this

func waitForControlPlane(client klient.Client) error {
	r, err := resources.New(client.RESTConfig())
	if err != nil {
		return err
	}
	selector, err := metav1.LabelSelectorAsSelector(
		&metav1.LabelSelector{
			MatchExpressions: []metav1.LabelSelectorRequirement{
				{Key: "component", Operator: metav1.LabelSelectorOpIn, Values: []string{"etcd", "kube-apiserver", "kube-controller-manager", "kube-scheduler"}},
			},
		},
	)
	if err != nil {
		return err
	}
	// an e2e provider cluster with one control-plane node will have 4 pods running the core apiserver components
	err = wait.For(conditions.New(r).ResourceListN(&v1.PodList{}, 4, resources.WithLabelSelector(selector.String())))
	if err != nil {
		return err
	}
	selector, err = metav1.LabelSelectorAsSelector(
		&metav1.LabelSelector{
			MatchExpressions: []metav1.LabelSelectorRequirement{
				{Key: "k8s-app", Operator: metav1.LabelSelectorOpIn, Values: []string{"kindnet", "kube-dns", "kube-proxy"}},
			},
		},
	)
	if err != nil {
		return err
	}
	// an e2e provider cluster with one control-plane node will have 4 k8s-app pods running networking components
	err = wait.For(conditions.New(r).ResourceListN(&v1.PodList{}, 4, resources.WithLabelSelector(selector.String())))
	if err != nil {
		return err
	}
	return nil
}

I am trying to find a generic way to setup this helper function.  Since this is not really generic the way it works today, i am wonder if I can add an extra interface function that provides a list of selectors and we can use that to perform this wait function ?

@jbpratt  With generics, do you want me to turn the existing kind cluster type to value receiver or use the pointer receiver ? Which one would work better ?

@jbpratt With generics, do you want me to turn the existing kind cluster type to value receiver or use the pointer receiver ? Which one would work better ?

Hey @harshanarayana, sorry about that I realized last night this would probably be a problem. I had a friend show me this which allows us to use the pointer receivers https://go.dev/play/p/qdFCw5w42DB I'd just be worried our structs are too large

@jbpratt With generics, do you want me to turn the existing kind cluster type to value receiver or use the pointer receiver ? Which one would work better ?

Hey @harshanarayana, sorry about that I realized last night this would probably be a problem.  About to push the changes

@jbpratt @reetasingh @ShwethaKumbla @vladimirvivien @cpanato PTAL.. I think I've done all the changes I had In my mind.

@vladimirvivien  Thanks for the review. I have updated the changes suggested. I am open to suggestion on if there is a way to simplify the generics part of pointer receiver. But I have not found one so, I will leave that as is for now. PTAL when you can.

/test pull-e2e-framework-test

@vladimirvivien  @cpanato  any idea why this might be happening ? Tests have gotten really flaky again in the last couple of days. Even the dependabot once have been failing. Most of them seem to be failing because of  Setup failure: failed to create kind cluster: exit status 1 : exit status 1.

/test pull-e2e-framework-test

@vladimirvivien  Thanks for the quick review. I have updated the changes based on your comments for kind_funcs.go. I have also ported the early implementation of the kwok from the original PR for reference. Please let me know if you want me to move that out and have a different PR later on.

@vladimirvivien Thanks for the quick review. I have updated the changes based on your comments for kind_funcs.go. I have also ported the early implementation of the kwok from the original PR for reference. Please let me know if you want me to move that out and have a different PR later on. PTAL when you can.

@harshanarayana Do we plan to remove the kwok implementation from this PR following https://github.com/kubernetes-sigs/e2e-framework/pull/239#issuecomment-1601917798 ? Not much reason in fixing it up if so.

@jbpratt  Yes. I will move that out of this PR and push a commit. Was doing just that as we speak

/test pull-e2e-framework-test


feat: added a closeButtonClass prop for base dialog

To mirror the ebayui-core-marko dialog: https://github.com/eBay/ebayui-core/blob/master/src/components/components/ebay-dialog-base/index.marko#L54

Thanks for the PR!
Could you please also update readme?


ERR when run  402-pose-estimation.ipynb  on  PC :NUC with Mesa Intel® Xe Graphics (TGL GT2)

Describe the bug
ERR when run  402-pose-estimation.ipynb  on  PC :NUC with Mesa Intel® Xe Graphics (TGL GT2)
err info :
error: undefined reference to `__builtin_spirv_intel_sub_group_block_read_uc2_p3i8()'

error: backend compiler failed build.
LLVM ERROR: Error: Attempt to redefine function: ; Function Attrs: convergent
declare spir_func <16 x i8> @_Z33__spirv_SubgroupBlockReadINTEL_uiPU3AS3c(i8 addrspace(3)*) #5
 => i8 (i8 addrspace(3)*)

OS:ubuntu20.04
PC: NUC
graph card: Mesa Intel® Xe Graphics (TGL GT2)

Hi @yangcopnuli I'm using Tiger Lake and didn't face that issue
Init OV

Infer Pose Estimation:

Perhaps you need to check whether your GPU is available before running (yours have red symbol near GPU that mine does not have)

Closing issue, feel free to re-open or start a new issue if additional assistance is needed.


Second example in the Examples section is missing the end '.  REPLACE(‘cdabcd’, ‘cd’, ‘x)

Second example in the Examples section is missing the end '.  REPLACE(‘cdabcd’, ‘cd’, ‘x)
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: e3e91116-1a5b-14d4-c739-317fcfe618e8
Version Independent ID: 5107eccc-6ef8-ea10-6364-cf7d8bcb651a
Content: REPLACE (Azure Stream Analytics) - Stream Analytics Query
Content Source: stream-analytics-docs/replace-azure-stream-analytics.md
Service: stream-analytics
GitHub Login: @Fleid
Microsoft Alias: fleide

@PaulFeaser thanks for pointing out this issue. I'll fix it today.


Lib directory not found

Hi everibody!
Please, help me!
I try to use this library but when I typing (java -jar cli.jar) I get a error message "Lib directory not found. ClamShell-Cli will exist"
How to resolve this problem?
Thansk

@Almaz-KG apologies for the late reply.  Hope you figured it out.  If not, please make sure lib directory is on the same directory as the cli.jar.

It was my mistake, so I am sorry :-)

@Almaz-KG No need to be sorry, thanks for using the tool! :+1: