nginx ingress fails with error no service with name kube-system/default-http-backend found:

When used in kubeadm cluster with version shown below, nginx ingress controller fails with error no service with kube-system/default-http-backend found, even though the service is available.
kubeadm version: &version.Info{Major:"1", Minor:"7", GitVersion:"v1.7.3", GitCommit:"2c2fe6e8278a5db2d15a013987b53968c743f2a1", GitTreeState:"clean", BuildDate:"2017-08-03T06:43:48Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"linux/amd64"}

Ingress controller image gcr.io/google_containers/nginx-ingress-controller:0.9.0-beta.10
[dumb-init] Unable to detach from controlling tty (errno=25 Inappropriate ioctl for device).
[dumb-init] Child spawned with PID 37.
[dumb-init] Unable to attach to controlling tty (errno=25 Inappropriate ioctl for device).
[dumb-init] setsid complete.
I0824 20:10:03.237587      37 launch.go:105] &{NGINX 0.9.0-beta.10 git-9ef5330 https://github.com/kubernetes/ingress}
I0824 20:10:03.237681      37 launch.go:108] Watching for ingress class: nginx
I0824 20:10:03.237863      37 launch.go:262] Creating API server client for https://<IP_ADDRESS>:443
F0824 20:10:03.257520      37 launch.go:122] no service with name kube-system/default-http-backend found: User "system:serviceaccount:default:default" cannot get services in the namespace "kube-system". (get services default-http-backend)
[dumb-init] Received signal 17.
[dumb-init] A child with PID 37 exited with exit status 255.
[dumb-init] Forwarded signal 15 to children.
[dumb-init] Child exited with status 255. Goodbye

But the service is found when running kubectl get service --all-namespaces
default       kubernetes             <IP_ADDRESS>        <none>        443/TCP              19m
default       redis-cache-service    <IP_ADDRESS>     <none>        6379/TCP,16379/TCP   16m
default       redis-pubsub-service   <IP_ADDRESS>    <none>        6379/TCP,16379/TCP   16m
default       rethinkdb-service      <IP_ADDRESS>    <none>        28015/TCP,8080/TCP   16m
kube-system   default-http-backend   <IP_ADDRESS>   <none>        80/TCP               12s
kube-system   kube-dns               <IP_ADDRESS>       <none>        53/UDP,53/TCP        19m

I tried deploying backend service in default backend as well with no success.
Below ingress-container yaml
apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  labels:
    app: nginx
  name: nginx-deployment
spec:
  replicas: 1
  selector:
    matchLabels:
      app: nginx
  strategy:
    type: Recreate
  template:
    metadata:
      creationTimestamp: null
      labels:
        app: nginx
    spec:
      containers:
      - image: gcr.io/google_containers/nginx-ingress-controller:0.9.0-beta.10
        imagePullPolicy: Always
        name: nginx
        ports:
        - name: https
          containerPort: 443
        resources:
          limits:
            cpu: 250m
            memory: 0.5Gi
          requests:
            cpu: 250m
            memory: 0.4Gi
        args:
        - /nginx-ingress-controller
        - --default-backend-service=kube-system/default-http-backend
        - --default-ssl-certificate=default/nginx
        volumeMounts:
          - name: nginx-template-volume
            mountPath: /etc/nginx/template
            readOnly: true
      volumes:
        - name: nginx-template-volume
          configMap:
            name: nginx-template
            items:
              - key: nginx.tmpl
                path: nginx.tmpl

@anandanand84 you have RBAC enabled in the cluster. You need to use this as guide
https://github.com/kubernetes/ingress/tree/master/examples/rbac/nginx

@aledbf Is there anyway I can disable the RBAC ?

@anandanand84 you can configure RBAC to be permissive https://kubernetes.io/docs/admin/authorization/rbac/#permissive-rbac-permissions

@aledbf Thankyou I'll check

It works like charm  :)
@anandanand84 you can configure RBAC to be permissive https://kubernetes.io/docs/admin/authorization/rbac/#permissive-rbac-permissions
Thanks anandanand84


Active Directory Integrated Auth for Invoke-SqlCmd

Just wanted to see if there is an option to specify Active Directory Integrated auth when using Invoke-sqlcmd command let when connection Azure SQL Databases. This would alleviate the use of supplying Username and Password in the script

Review the docs https://docs.microsoft.com/en-us/azure/sql-database/sql-database-aad-authentication

Yes I did, but was looking for examples where ad integrated auth is used in
invoke-sqlcmd.
On Sat, Aug 17, 2019 at 2:22 PM Ilya<EMAIL_ADDRESS>wrote:

Review the docs
https://docs.microsoft.com/en-us/azure/sql-database/sql-database-aad-authentication
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PowerShell/PowerShell/issues/10353?email_source=notifications&email_token=AG4RRQ32MKPKZSUC7HLHN73QFBFYNA5CNFSM4ILYFHL2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4QR2PA#issuecomment-522263868,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AG4RRQYK53T24CB75TZCZL3QFBFYNANCNFSM4ILYFHLQ
.

@Adikota The repository is for PowerShell Core development. Invoke-sqlcmd and Azure authentication is out of the repo. Please use other resources like forums to resolve your question.


Query: Creating a module of optionally (ngx-translate)-able components.

I'm submitting a ...  (check one with "x")
[ ] bug report => check the FAQ and search github for a similar issue or PR before submitting
[x] support request => check the FAQ and search github for a similar issue before submitting
[ ] feature request

Goal: To create a module of optionally translatable components
Current behavior
I am creating a module of components that I would like to be optionally translatable. My basic requirements are:

Components in my module will use translate pipe/directive.
The components will (somehow) ship with default translations (english).
The importer of the module should not have to (directly) import @ngx-translate if they don't want to i.e. they are only planning to deliver their product in the default language, and have no interest in overriding existing translations.
However should they wish to provide translations, they should be able to.

I can't see exactly how I would achieve this given my understandings of this module and ngx.
To get the default translations working within my module (to satisfy requirements 2 and 3) my module will have to import and configure the TranslateModule. I guess this already is the problem - if the consuming application is not going to use @ngx-translate, then my module would need to .forRoot where as if it is I would have to .forChild ?
For requirement 4 I would want/expect that my consuming application can load translations with the appropriate keys specified in my components and that the translations will happen. So for example if my component defines a <p translate>LOGIN_BUTTON</p> and the application loads a translation with that key, then it should be translated as such and should override the default specified within the component.
Any advice on how to set this up would be much appreciated.
Expected/desired behavior
To be able to provide a module of optionally translatable components to the user.

Sorry for not answering the issue for such a long time.
We try to do better in the future, promised.
I am closing the issue for now - please re-open it if the issue still persists or you still need help.


[VS Code] Starting collaboration session: Invalid URI: The format of the URI could not be determined.

Error:
Starting collaboration session: Invalid URI: The format of the URI could not be determined.
Steps to Reproduce:
1.
2.

Version Data

extensionName
VSLS

extensionVersion
1.0.419

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.35.1

platformName
Windows

platformVersion
10.0.17763

Closing as a duplicate of #2314. Let's continue the conversation there. Thanks for using Live Share!


FYI MAVLink XML/XSD update to allow exponentation operator in bitmask flags

FYI We're modifying the MAVLink XSD validation file in https://github.com/ArduPilot/pymavlink/pull/920 to allow a bitmask value to be declared using the Python exponentiation operator.
This makes it easier to immediately visualize which bit is set by a flag, because you can use the syntax
<entry value="2**15" name="BIT15" />

rather than
<entry value="32768" name=" BIT15" />

This is transparent to mavgen because the parser evaluates the number before creating the generated headers. However it may affect other the rust parser
We're not updating the XML yet to allow other parsers time to update.
Any concerns, please raise on https://github.com/ArduPilot/pymavlink/pull/920

Hello @hamishwillee.
Thank you for the update. I'll integrate this functionality soon.

Solved in #65

Excellent!


Add Bundle-cleanup

Add bundle-cleanup Makefile target to remove a bundle installed via
bundle-run

/lgtm


[HOLD for payment 2023-10-13] [HOLD for payment 2023-10-12] [$250]  Distance - App crashes when tapping on the receipt thumbnail while receipt is generating

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Launch New Expensify app
Go to + > Request money > Distance > Create a distance request
Go to distance request details page
Tap on the receipt thumbnail before the map image is generated

Expected Result:
App does not crash
Actual Result:
App crashes
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[ ] Android / Chrome
[x] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.77-2
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/ad0c3860-4aba-43f4-a187-61aa818a7fbd
log.txt
Production
https://github.com/Expensify/App/assets/78819774/2ec2a46d-747b-4494-ba05-60a024f281ee
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01812bc33ae4579c27
Upwork Job ID:<PHONE_NUMBER>970191872
Last Price Increase: 2023-10-04
Automatic offers:

paultsimura | Contributor | 27024406

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Thanks @paultsimura can you raise a PR and make it ready for review please?

Fixed now

@paultsimura fix + bonus = $375 - paid ✔
@rushatgabhane review + bonus = $375 - in NewDot

@rushatgabhane please complete the checklist

I'm OOO Oct 16-23. Adding leave buddy
Status: waiting on checklist completion and possibly need to submit a regression test request

$375 payment approved for @rushatgabhane based on BZ summary.

back from OOO, @rushatgabhane how's that checklist coming along? 😁

@rushatgabhane bump

@rushatgabhane bump

I think he might be OOO 🤔 there was a post about a resort in bali today

@rushatgabhane bump, please complete so that we can close this out https://github.com/Expensify/App/issues/28814#issuecomment-1767204777

@sonialiap completed the checklist 😅

Thanks!


Auto restart doesnt work

Ram. ram gets consumed. Auto restart stops server and then it has to be manually restarted. Me want fixxy

There's not much I can do about RAM consumption. Please try https://github.com/itzg/docker-minecraft-server#enable-aikars-flags or report an issue with Mojang.
As for auto-restart, provide your container config and logs.


Make test short for FNP when testing on CPU

Perfect, please merge when passing and then I'll rebase my branch


Add query string extension trait

This commit adds query string extraction via the ExtractQuery trait, implemented for tide::Context.
Description
Added a module, querystring, which includes the ExtractQuery trait to help with validating and deserializing HTTP query parameters.
Motivation and Context
Processing query parameters is a common concern of most HTTP frameworks.
How Has This Been Tested?
Simulated requests to a Tide endpoint. This PR doesn't remove or alter existing functionality.
Types of changes

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

Checklist:

[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Heh, looks like you and @badboy had the same idea at the same time -- https://github.com/badboy/tide-query-params

He, yeah. I saw this PR after I had written my code, so I didn't bother fixing it up, adding tests and submitting it as a PR, instead just pushing the code undocumented (and unlicensed).

@yoshuawuyts Oops.
@badboy's implementation looks very similar, with a few minor differences:

I used serde_urlencoded over serde_qs,  which doesn't support nested parameters. I was concerned we'd run into this issue, which is currently un-patched in the library (a PR exists, though).

I return Result<_, tide::error::Error> instead of Option, which means this PR returns an error if parameters fail to deserialize. I suppose users can convert to Option if needed, but I'm not entirely sure which method I prefer.

This PR uses explicit lifetimes instead of serde::DeserializeOwned. I'm unsure which is best in this scenario.

I'd like to have a convenient example to show in tonight's presentation, so that was my primary goal in writing this PR. I'm happy to defer, though.

@secretfader I think what you just said makes a lot of sense. The decisions you've made made a lot of sense!
I left a few on the issue I'm not sure about, but otherwise this is looking :100:

@yoshuawuyts I'm seeing issues with the clippy lints that were just added (#![forbid(rust_2018_idioms)]) is erroring on the #[derive(Deserialize)] macro. Have you seen that before? That's the only thing preventing a successful CI run, I think.

@aturon Do we want to include this before your re-org or after?

@prasannavl This PR seems to conflict with some of the recent changes to Clippy lints, notably #![forbid(rust_2018_idioms)]. It seems like we'd prefer to favor Rust 2018 idioms. Maybe I misunderstand the point of this lint. Can you assist?

This PR seems to conflict with some of the recent changes to Clippy lints, notably #![forbid(rust_2018_idioms)]. It seems like we'd prefer to favor Rust 2018 idioms. Maybe I misunderstand the point of this lint. Can you assist?

@secretfader - I haven't look at the PR yet, but as answering your question quickly - rust_2018_idioms is the name of the lint. It makes sure you use the 2018 idioms. forbid is to indicate that the linter/compiler forbids you from NOT using 2018 idioms. :)

Alright, so this seems to be since serde_derive is still a bit problematic with 2018.  https://github.com/serde-rs/serde/blob/master/serde_derive/src/dummy.rs#L26. This wasn't a problem before since Tide didn't need to use a derive so far.
@yoshuawuyts - Should we drop the compiler lint down to deny to allow working with serde? (I personally haven't been able to find a workaround so far if serde derive is needed)

Serde also has changed usage guidelines when compiling with Rust 2018. So I tried what the issue suggested: removing the explicit dependency  on serde_derive and enabling the derive feature on Serde itself.
While it makes imports cleaner, it did nothing to satisfy the lint in question.

@prasannavl I have the Serde 2018 update ready to PR. I'll push it momentarily.

Rebased this branch against master and made the API changes suggested by @yoshuawuyts. url_query is an ideal name for this method.

I think I've resolved all feedback. This seems ready to go in, but I'd prefer to let someone else push the "merge" button.

Merged, thank you!


Fix private github repo assets link

This fixes the private repo asset download that was introduced in #23 but got removed in 04c94ee9284cef739d85aa17009164e576bccbb5 because of #26.
cc: @leo

Wonderful, thank you!


added booleans.py

Please fill in this pull request template before submitting
1. This pull request resolves #4
2. Description
I have updated booleans.py file in the required format as mentioned with proper description and a challenge.
3. Fill in checklist by marking [x]

[x] I've read the CONTRIBUTING.md
[x] I was assigned to this issue
[x] My code is formatted using required linting
[x] I've run my code locally and checked it works

@Dave3130 made some small edits for the formatting. but overall great, thanks!!

Okay !!


Axis Angle from Matrix

Does this function exist, to get the axis/angle out of a rotation matrix? I couldn't find it after scanning your header.

I ended up writing one here. I just double checked your memory layout -- I think it's transposed relative to my ordering:
0 1 2 - x axis
3 4 5 - y axis
6 7 8 - z axis

Where yours is like so.
0 3 6
1 4 7
2 5 8
x y z
| | |
a a a
x x x
i i i
s s s

void CUTE_MATH_CALL axis_angle_from_m3(m3 m, v3* axis, float* angle_radians)
{
	const float k_tol = 1.0e-8f;
	float c = 0.5f * (trace(m) - 1.0f);
	float angle = acosf(c);
	*angle_radians = angle;

	bool angle_near_zero = fabsf(angle) < k_tol;
	bool angle_not_near_pi = angle < CUTE_MATH_PI - k_tol;
	if (angle_near_zero) {
		// When angle is zero the axis can be anything. X axis is good.
		*axis = v3(1, 0, 0);
	} else if (angle_not_near_pi) {
		// Standard case with no singularity.
		v3 n = v3(m[1][2] - m[2][1], m[2][0] - m[0][2], m[0][1] - m[1][0]);
		*axis = norm(n);
	} else {
		// Angle is near 180-degrees.
		int i = 0;
		if (m[1][1] > m[0][0]) i = 1;
		if (m[2][2] > m[i][i]) i = 2;
		int j = (i + 1) % 3;
		int k = (j + 1) % 3;
		float s = sqrtf(m[i][i] - m[j][j] - m[k][k] + 1.0f);
		float inv_s = s != 0 ? 1.0f / s : 0;
		float v[3];
		v[i] = 0.5f * s;
		v[j] = m[j][i] * inv_s;
		v[k] = m[i][k] * inv_s;
		*axis = v3(v[0], v[1], v[2]);
	}
}


Evaluating html in a list

Is it possible to put a list into html?
Something like this:

      {{ hero }}

I need to show list as the text of the swal, this is what I do:
swal({
        title: 'title',
        type: 'error',
        html: '

{{err}}

'
      });

But It seems that don't evaluate {{ err }}

This is an issue relating to Angular, not SweetAlert. I'd suggest asking on relevant Angular forums/lists.

Take a look on the Angular 2+ wrapper for SweetAlert2: https://github.com/toverux/ngsweetalert2 - it's great!


XMLEventReader VS XMLStreamReader

According to this SO question XMLStreamReader is faster than XMLEventReader. We need to check whether it's true and rewrite XmlMaid implementations accordingly.

@g4s8/z please, pay attention to this issue

The job #195 assigned to @olenagerasimova/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

@g4s8 closing this issue: I do not see any significant improvements, but code looks much uglier.

Job was finished in 38 hours, bonus for fast delivery is possible (see §36)

@sereshqua/z please review this job completed by @olenagerasimova/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

@0crat quality bad

Quality is low, no payment, see §31


Play music in background instead of redirecting to YouTube

Refer to: https://www.tutorialspoint.com/How-to-add-background-music-in-HTML#:~:text=The HTML tag is,play music in the background.

Hey can i work on this issue..

can you pls assign me this issue..

Sure @Rithik14

<!DOCTYPE html>

    <title>
      HTML <bgsound> Tag
  </title>

HTML <bgsound> tag

        <bgsound src="https://media.geeksforgeeks.org/wp-content/uploads/20190531135120/beep.mp3" />

please plugin speaker/headphone to listen to the sound.

Is this correct way to answer..

Related Issue/Play music in background instead of redirecting to YouTube
fixes #39
Code snippet
<!DOCTYPE html>

    <title>
      HTML <bgsound> Tag
  </title>

HTML <bgsound> tag

        <bgsound src="https://media.geeksforgeeks.org/wp-content/uploads/20190531135120/beep.mp3" />

please plugin speaker/headphone to listen to the sound.

Is this way is correct..

If it plays the music in the browser, then yes! @Rithik14 .

pls make a PR.

Dude but this <bgsound> tag supports only for Internet Explorer not for other browsers .so can i use <audio> tag instead of <bgsound>

Hey can use audio tag..

Hey can i use audio tag

Reference

Sure.

Hey if any issues in code pls let me now

Hey if any issues in my code pls let me now

Hey @Rithik14 did you solve this?

ya dude i have solved...but i am not sure about my pull req
if any mistakes in my code pls let me now

He said that music is not playing in one of the comments in your pull request @Rithik14!

@Rithik14 - are you fine if I accept @aaryansr26 's PR?

@Rithik14 - are you fine if I accept @aaryansr26 's PR?

Fine dude

@Rithik14 - are you fine if I accept @aaryansr26 's PR?

Fine dude

I am very sorry for the inconvenience, hope you can contribute to this project.

project

No problem dude


[#109515840] Fix typo in Vagrantfile vm.box

Looks like a vim whoopsie that repeated the text three times. This was fixed
in #6, but the commits were lost just prior to merging.

LGTM


Error:Execution failed for task ':app:transformClassesWithJarMergingForDebug'. > com.android.build.api.transform.TransformException: java.util.zip.ZipException: duplicate entry: com/google/android/gms/internal/zzqx.class

You seem to have a JAR file or other library that is conflicting with this library. Please make sure all Google play and Firebase versions match exactly


fix: update l10n files and fix a couple typos

Description
This smooths out another l10n bubble (and fixes a couple typos some recent change introduced).
Task Item
#minor

Coverage remained the same at 55.783% when pulling ab5c7b369f9738416e179f5dd4d67abbd76c24ce on l10n_2020-10-16 into d3b2c4904139b342e45ba08b07888859a9a39b32 on main.


⚠ Bofrost.es has degraded performance

In e617711, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1656 ms

Resolved: Bofrost.es performance has improved in 1e57fa6 after 7 minutes.


Add Operator for running Elasticsearch in Kubernetes

Edited by Zalando
Add Zalando as using elasticsearch

Merged! Thank you @Lyr


Create jekyll-preview.yml

[x] Have you followed the contributing guidelines?
[x] Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

This adds a custom Actions workflow to allow pull requests to deploy to a preview environment on GitHub Pages.

[x] Have you followed the contributing guidelines?

[x] Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

This adds a custom Actions workflow to allow pull requests to deploy to a preview environment on GitHub Pages.


Fix missing 'this' qualifier

The 'or' and 'and' filters were not being applied to a table when selecting. The 'this' qualifier was missing which was only adding the filters to the parameter rather than the instance.

Oh - Great catch there, thank you!


Increased response time when increasing the value in ErrorHandler::memoryReserveSize

What steps will reproduce the problem?
Set ErrorHandler to high ErrorHandler::memoryReserveSize.
What is the expected result?
Response time does not change.
What do you get instead?
Increased response time.
Additional info
yii\base\ErrorHandler::register
if ($this->memoryReserveSize > 0) {
	$this->_memoryReserve = str_repeat('x', $this->memoryReserveSize); //was in 2.0.45

	//at high values MemoryReserveSize is slower than via str_repeat
	//Eg. MemoryReserveSize = 100000000 (100 MB), just for visible result
	$this->_memoryReserve = str_pad('', $this->memoryReserveSize, 'x'); //became starting from 2.0.46
}

upgrade from 2.0.45 to 2.0.48
10 MB reserved. memoryReserveSize =  10000000.

Q
A

Yii version
2.0.48

PHP version
8.1.17

Operating system

Any idea why?

It was introduced by https://github.com/yiisoft/yii2/pull/19402, and using str_pad() was supposed to be faster. But I just tested this and it is 10-20 times slower than str_repeat().


Support for SFP and Millennium for Steam

Can we have support for these 2 tools?
https://github.com/PhantomGamers/SFP
https://github.com/ShadowMonster99/millennium-steam-patcher

https://github.com/tkashkin/Adwaita-for-Steam/issues/244
https://github.com/tkashkin/Adwaita-for-Steam/issues/279


editor - sticky parameter hints

The editor input has changed but the old parameter hints still show

@alexandrudima @joaomoreno unsure to whom this belings

I have seen this too!


Evaluation metric: train or test score ?

Hi Mxnet,
I am a bit confused.
Looking at the examples and at the callback.py file, I can see that the same function param.eval_metric.get() is used for both the Test-metric and the Train_metric.
Can you clarify ? What makes the result test versus train, the used callback entry-point (batch_end_callback versus eval_batch_end_callback versus eval_end_callback)?
In other words, how to retrieve the Test score, the train score, and the train loss in a single custom callback, for example from the eval_batch_end_callback?
Many thanks,
AL

I tried using model.score inside my custom callback to calculate the train and the test accuracy scores... unfortunately this lead to an infinite fit process... If you have any suggestion :)

model is deprecated. Try moduel

Thank you Eric. I will definitely try Module.

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


iOS Build Error: Include of a non-modular header inside framework module 'BugsnagReactNative.Bugsnag'

Description
We have code that is importing <BugsnagReactNative/BugsnagReactNative.h> from this, we were calling [BugsnagReactNative start]; inside of a section of code (called from appDidFinishLaunching)
Issue

This wasn't an issue, until we upgraded one of our cocoapods dependencies (ZendeskChat) to their V2.
I'm a maintainer of react-native-zendesk-chat and I'm trying to upgrade it.

That updated pod spec has these lines:
Pod::Spec.new do |s|
  s.name         = "RNZendeskChat"
  s.version      = package['version']
  s.summary      = package['description']
  s.license      = package['license']
  s.authors      = package['author']
  s.homepage     = package['homepage']
  s.platform     = :ios, "10"
  s.source       = { :git => "https://github.com/taskrabbit/react-native-zendesk-chat.git", :tag => "v#{s.version}" }
  s.source_files = "ios/*.{h,m}"
  s.static_framework = true

  s.framework    = 'Foundation'
  s.framework    = 'UIKit'

  s.dependency 'React'
  s.dependency 'ZendeskChatSDK'
end

Environment
Library versions:
$ yarn list --pattern "(react-native|bugsnag)" # we don't have code-push
yarn list v1.22.4
├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>│  └─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>└─<EMAIL_ADDRESS>

cocoapods version (if any) (pod -v): 1.9.3 (Note that the new command for this is pod --version)

iOS/Android version(s): 12.2

simulator/emulator or physical device?: Simulator

debug mode or production?: Debug

[x] (iOS only) [BugsnagReactNative start] is present in the
application:didFinishLaunchingWithOptions: method in your AppDelegate
class?

Interesting Podfile details
platform :ios, '11.0'
require_relative '../node_modules/@react-native-community/cli-platform-ios/native_modules'

# …snipped for brevity

	use_native_modules!
	use_frameworks!

Also, we have flipper disabled!
Example code snippet
Code is just:
import <BugsnagReactNative/BugsnagReactNative.h>

// in appDidFinishLaunching:
[BugsnagReactNative start];

Error messages:

Showing All Messages
/Users/fbarthelemy/Code/TRClient5/node_modules/bugsnag-react-native/cocoa/vendor/bugsnag-cocoa/Source/Bugsnag.h:30:9: Include of non-modular header inside framework module 'BugsnagReactNative.Bugsnag': '../node_modules/bugsnag-react-native/cocoa/vendor/bugsnag-cocoa/Source/BugsnagPlugin.h'

/Users/fbarthelemy/Code/TRClient5/ios/TRClient/AppDelegate.m:23:9: Could not build module 'BugsnagReactNative'

Error is in Bugsnag.h:

Potentially related ticket: #442

Workaround is to allow non-modular headers at a Project level (temporarily -- as this is considered a problem in one of your libraries -- and probably a sign that other build weirdness will happen)

Thanks for raising @fbartho, I agree that the temporary solution is not ideal. We are going to look into a better fix by moving the import into the .m file (tbc).

@fbartho, I've just learned that a patch was applied in 2.23.7 to the missing BugsnagPlugin.h header to the podspec file.  Please can you upgrade to 2.23.7 and confirm if this fixes the issue?

@steve-nester-uk -- I tested against 2.23.9, and there's a new error:

Trying 2.23.7 now.

Problem is resolved with exactly 2.23.7 -- Do you need me to file a different Github Issue about 2.23.9 or is this sufficient?

Hey @fbartho, thanks for letting us know. We're already aware of this issue (moving to v2.23.8 or v2.23.9) you can track it here: https://github.com/bugsnag/bugsnag-react-native/issues/462


Vao restore

This fixes issue https://github.com/sensics/OSVR-RenderManager/issues/208

Jeremy, have a look at this and merge it if it looks okay to you.  It fixes https://github.com/sensics/OSVR-RenderManager/issues/208.  I tested on Windows 10 OpenGL and it runs without errors.


failed to convert convert llava-llama3 model to  hf  format

In the project: https://github.com/InternLM/xtuner/tree/main/xtuner/configs/llava/llama3_8b_instruct_clip_vit_large_p14_336, it gives an examples how to convert  llava-llama3 model to hf format:
 python ./convert_xtuner_weights_to_hf.py --text_model_id ./iter_39620_xtuner --vision_model_id ./iter_39620_visual_encoder --projector_weight ./iter_39620_xtuner/projector/model.safetensors --save_path ./iter_39620_llava
I follow it in this way:

I train llava-llama3 model with single gpu with llava_llama3_8b_instruct_qlora_clip_vit_large_p14_336_e1_gpu1_finetune.py, and get the pth named lama3_llava_pth;
convert llama3_llava_pth to hugggingface format named   llama3_llava_pth/hf;
I merge the llm_adapter and original llama3 to new llama3 with xtuner merge tool, and create ./llama3_llava_pth/merge
I try to convert format
CUDA_VISIBLE_DEVICES=4 python ./xtuner/configs/llava/llama3_8b_instruct_clip_vit_large_p14_336/convert_xtuner_weights_to_hf.py --text_model_id ./llama3_llava_pth/merge --vision_model_id ${vit} --projector_weight llama3_llava_pth/hf/projector --save_path ./llama3_llava_pth/LLava_format

however I get the error
The tokenizer class you load from this checkpoint is not the same type as the class this function is called from. It may result in unexpected tokenization.
The tokenizer class you load from this checkpoint is 'PreTrainedTokenizerFast'.
The class this function is called from is 'LlamaTokenizerFast'.
Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.
Traceback (most recent call last):
  File "/Data_PHD/****/projects/xtuner/./xtuner/configs/llava/llama3_8b_instruct_clip_vit_large_p14_336/convert_xtuner_weights_to_hf.py", line 140, in <module>
    main()
  File "/Data_PHD/****/projects/xtuner/./xtuner/configs/llava/llama3_8b_instruct_clip_vit_large_p14_336/convert_xtuner_weights_to_hf.py", line 135, in main
    convert_to_hf(args.text_model_id, args.vision_model_id,
  File "/Data_PHD/****/projects/xtuner/./xtuner/configs/llava/llama3_8b_instruct_clip_vit_large_p14_336/convert_xtuner_weights_to_hf.py", line 61, in convert_to_hf
    model = LlavaForConditionalGeneration(config)
  File "/Data_PHD/****/anaconda3_35/envs/XTUNER/lib/python3.10/site-packages/transformers/models/llava/modeling_llava.py", line 244, in __init__
    self.multi_modal_projector = LlavaMultiModalProjector(config)
  File "/Data_PHD/****/anaconda3_35/envs/XTUNER/lib/python3.10/site-packages/transformers/models/llava/modeling_llava.py", line 93, in __init__
    self.linear_1 = nn.Linear(config.vision_config.hidden_size, config.text_config.hidden_size, bias=True)
  File "/Data_PHD/****/anaconda3_35/envs/XTUNER/lib/python3.10/site-packages/transformers/configuration_utils.py", line 263, in __getattribute__
    return super().__getattribute__(key)
AttributeError: 'CLIPConfig' object has no attribute 'hidden_size'`

is there any error in my steps, thank you for you answer!

@Jason8Kang  Hi!
This PR https://github.com/InternLM/xtuner/pull/661 can solve your issue!
Meanwhile, please specific the .safetensors file for --projector_weight, like llama3_llava_pth/hf/projector/model.safetensors

@Jason8Kang
I think these warnings will not affect the usage, and we can ignore them. Right?

I also think so at first. but when I load the LLava_format in this way.
from llava.model.builder import load_pretrained_model
from llava.mm_utils import get_model_name_from_path
from llava.eval.run_llava import eval_model

model_path = "./llama3_llava_pth.bak/LLava_format"

tokenizer, model, image_processor, context_len = load_pretrained_model(
    model_path=model_path,
    model_base=None,
    model_name=get_model_name_from_path(model_path)
)

it will give warning.
/Data_PHD/***/anaconda3_35/envs/llava/lib/python3.10/site-packages/torch/nn/modules/module.py:2025: UserWarning: for vision_model.encoder.layers.8.self_attn.v_proj.weight: copying from a non-meta parameter in the checkpoint to a meta parameter in the current model, which is a no-op. (Did you mean to pass assign=Trueto assign items in the state dictionary to their corresponding key in the module instead of copying them in place?)   warnings.warn(f'for {key}: copying from a non-meta parameter in the checkpoint to a meta ' /Data_PHD/***/anaconda3_35/envs/llava/lib/python3.10/site-packages/torch/nn/modules/module.py:2025: UserWarning: for vision_model.encoder.layers.8.self_attn.v_proj.bias: copying from a non-meta parameter in the checkpoint to a meta parameter in the current model, which is a no-op. (Did you mean to passassign=Trueto assign items in the state dictionary to their corresponding key in the module instead of copying them in place?)   warnings.warn(f'for {key}: copying from a non-meta parameter in the checkpoint to a meta ' /Data_PHD/***/anaconda3_35/envs/llava/lib/python3.10/site-packages/torch/nn/modules/module.py:2025: UserWarning: for vision_model.encoder.layers.8.self_attn.q_proj.weight: copying from a non-meta parameter in the checkpoint to a meta parameter in the current model, which is a no-op. (Did you mean to passassign=Trueto assign items in the state dictionary to their corresponding key in the module instead of copying them in place?)   warnings.warn(f'for {key}: copying from a non-meta parameter in the checkpoint to a meta ' /Data_PHD/***/anaconda3_35/envs/llava/lib/python3.10/site-packages/torch/nn/modules/module.py:2025: UserWarning: for vision_model.encoder.layers.8.self_attn.q_proj.bias: copying from a non-meta parameter in the checkpoint to a meta parameter in the current model, which is a no-op. (Did you mean to passassign=Trueto assign items in the state dictionary to their corresponding key in the module instead of copying them in place?)   warnings.warn(f'for {key}: copying from a non-meta parameter in the checkpoint to a meta ' /Data_PHD/***/anaconda3_35/envs/llava/lib/python3.10/site-packages/torch/nn/modules/module.py:2025: UserWarning: for vision_model.encoder.layers.8.self_attn.out_proj.weight: copying from a non-meta parameter in the checkpoint to a meta parameter in the current model, which is a no-op. (Did you mean to passassign=Trueto assign items in the state dictionary to their corresponding key in the module instead of copying them in place?)   warnings.warn(f'for {key}: copying from a non-meta parameter in the checkpoint to a meta ' /Data_PHD/***/anaconda3_35/envs/llava/lib/python3.10/site-packages/torch/nn/modules/module.py:2025: UserWarning: for vision_model.encoder.layers.8.self_attn.out_proj.bias: copying from a non-meta parameter in the checkpoint to a meta parameter in the current model, which is a no-op. (Did you mean to passassign=Trueto assign items in the state dictionary to their corresponding key in the module instead of copying them in place?)   warnings.warn(f'for {key}: copying from a non-meta parameter in the checkpoint to a meta ' /Data_PHD/***/anaconda3_35/envs/llava/lib/python3.10/site-packages/torch/nn/modules/module.py:2025: UserWarning: for vision_model.encoder.layers.8.layer_norm1.weight: copying from a non-meta parameter in the checkpoint to a meta parameter in the current model, which is a no-op. (Did you mean to passassign=Trueto assign items in the state dictionary to their corresponding key in the module instead of copying them in place?)   warnings.warn(f'for {key}: copying from a non-meta parameter in the checkpoint to a meta ' /Data_PHD/***/anaconda3_35/envs/llava/lib/python3.10/site-packages/torch/nn/modules/module.py:2025: UserWarning: for vision_model.encoder.layers.8.layer_norm1.bias: copying from a non-meta parameter in the checkpoint to a meta parameter in the current model, which is a no-op. (Did you mean to passassign=Trueto assign items in the state dictionary to their corresponding key in the module instead of copying them in place?)   warnings.warn(f'for {key}: copying from a non-meta parameter in the checkpoint to a meta ' /Data_PHD/***/anaconda3_35/envs/llava/lib/python3.10/site-packages/torch/nn/modules/module.py:2025: UserWarning: for vision_model.encoder.layers.8.mlp.fc1.weight: copying from a non-meta parameter in the checkpoint to a meta parameter in the current model, which is a no-op. (Did you mean to passassign=Trueto assign items in the state dictionary to their corresponding key in the module instead of copying them in place?)   warnings.warn(f'for {key}: copying from a non-meta parameter in the checkpoint to a meta ' /Data_PHD/***/anaconda3_35/envs/llava/lib/python3.10/site-packages/torch/nn/modules/module.py:2025: UserWarning: for vision_model.encoder.layers.8.mlp.fc1.bias: copying from a non-meta parameter in the checkpoint to a meta parameter in the current model, which is a no-op. (Did you mean to passassign=True to assign items in the state dictionary to their corresponding key in the module instead of copying them in place?)
but when I run inference, it will give error
Loading checkpoint shards: 100%|██████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 9/9 [00:01<00:00,  6.28it/s] Traceback (most recent call last):   File "<stdin>", line 1, in <module>   File "/Data_PHD/***/projects/LLaVA/llava/eval/run_llava.py", line 115, in eval_model     output_ids = model.generate(   File "/Data_PHD/***/anaconda3_35/envs/llava/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context     return func(*args, **kwargs)   File "/Data_PHD/***/projects/LLaVA/llava/model/language_model/llava_llama.py", line 125, in generate     ) = self.prepare_inputs_labels_for_multimodal(   File "/Data_PHD/***/projects/LLaVA/llava/model/llava_arch.py", line 202, in prepare_inputs_labels_for_multimodal     image_features = self.encode_images(images)   File "/Data_PHD/***/projects/LLaVA/llava/model/llava_arch.py", line 141, in encode_images     image_features = self.get_model().get_vision_tower()(images)   File "/Data_PHD/***/anaconda3_35/envs/llava/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl     return self._call_impl(*args, **kwargs)   File "/Data_PHD/***/anaconda3_35/envs/llava/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl     return forward_call(*args, **kwargs)   File "/Data_PHD/***/anaconda3_35/envs/llava/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward     output = old_forward(*args, **kwargs)   File "/Data_PHD/***/anaconda3_35/envs/llava/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context     return func(*args, **kwargs)   File "/Data_PHD/***/projects/LLaVA/llava/model/multimodal_encoder/clip_encoder.py", line 54, in forward     image_forward_outs = self.vision_tower(images.to(device=self.device, dtype=self.dtype), output_hidden_states=True)   File "/Data_PHD/***/anaconda3_35/envs/llava/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl     return self._call_impl(*args, **kwargs)   File "/Data_PHD/***/anaconda3_35/envs/llava/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl     return forward_call(*args, **kwargs)   File "/Data_PHD/***/anaconda3_35/envs/llava/lib/python3.10/site-packages/accelerate/hooks.py", line 160, in new_forward     args, kwargs = module._hf_hook.pre_forward(module, *args, **kwargs)   File "/Data_PHD/***/anaconda3_35/envs/llava/lib/python3.10/site-packages/accelerate/hooks.py", line 290, in pre_forward     return send_to_device(args, self.execution_device), send_to_device(   File "/Data_PHD/***/anaconda3_35/envs/llava/lib/python3.10/site-packages/accelerate/utils/operations.py", line 151, in send_to_device     return honor_type(   File "/Data_PHD/***/anaconda3_35/envs/llava/lib/python3.10/site-packages/accelerate/utils/operations.py", line 83, in honor_type     return type(obj)(generator)   File "/Data_PHD/***/anaconda3_35/envs/llava/lib/python3.10/site-packages/accelerate/utils/operations.py", line 152, in <genexpr>     tensor, (send_to_device(t, device, non_blocking=non_blocking, skip_keys=skip_keys) for t in tensor)   File "/Data_PHD/***/anaconda3_35/envs/llava/lib/python3.10/site-packages/accelerate/utils/operations.py", line 167, in send_to_device     return tensor.to(device, non_blocking=non_blocking) NotImplementedError: Cannot copy out of meta tensor; no data!

@Jason8Kang
Try this script?
https://huggingface.co/xtuner/llava-llama-3-8b-v1_1-hf/discussions/1

@Jason8Kang hi,
the offical xturner/llava-llama3 can be used for lmdeploy and work well.

i get the same llava-llama3 huggingface transfer question , i just following your changed code This PR and solved the problem.
but trained llava-llama3 weight can't be used for lmdeploy pipline, and the error can be listed as following:
llama3-ft$ python pipeline_llava.py
[2024-05-08 12:43:31,652] [INFO] [real_accelerator.py:203:get_accelerator] Setting ds_accelerator to cuda (auto detect)
 [WARNING]  async_io requires the dev libaio .so object and headers but these were not found.
 [WARNING]  async_io: please install the libaio-dev package with apt
 [WARNING]  If libaio is already installed (perhaps from source), try setting the CFLAGS and LDFLAGS environment variables to where it can be found.
 [WARNING]  Please specify the CUTLASS repo directory as environment variable $CUTLASS_PATH
 [WARNING]  sparse_attn requires a torch version >= 1.5 and < 2.0 but detected 2.1
 [WARNING]  using untested triton version (2.1.0), only 1.0.0 is known to be compatible
2024-05-08 12:43:33,017 - lmdeploy - ERROR - AttributeError: 'LlavaConfig' object has no attribute 'hidden_size'
2024-05-08 12:43:33,017 - lmdeploy - ERROR - <Model> test failed!
("Checking failed with error 'LlavaConfig' object has no attribute 'hidden_size'", 'Please send issue to LMDeploy with error logs.')

and the lmdeploy code can be show as following:
from lmdeploy import pipeline, ChatTemplateConfig
from lmdeploy.vl import load_image

# from modelscope import snapshot_download, AutoModel, AutoTokenizer
# import os

### download
# modelscope下载
# model_dir = snapshot_download('xtuner/llava-llama-3-8b-hf')

#### llava-lama3-8b pipeline
### offical huggface
# pipe = pipeline('/home/fusionai/.cache/modelscope/hub/xtuner/llava-llama-3-8b-hf',
#                 chat_template_config=ChatTemplateConfig(model_name='llama3'))

### train huggingface
pipe = pipeline('/home/fusionai/project/internllm_demo/llama3/llama3-ft/train/llava_train_20240508_1/iter_1008_final_hf',
                chat_template_config=ChatTemplateConfig(model_name='llama3'))

image = load_image('/home/fusionai/project/internllm_demo/lldemploy_demo/tiger.jpeg')
response = pipe(('describe this image', image))
print(response.text)

@Jason8Kang Try this script?
https://huggingface.co/xtuner/llava-llama-3-8b-v1_1-hf/discussions/1

thanks you, it works in your script. but it still has the warning as follows. do you know the meaning of this warning, I'm not sure whether it matters?
UserWarning: for vision_model.encoder.layers.23.mlp.fc2.bias: copying from a non-meta parameter in the checkpoint to a meta parameter in the current model, which is a no-op. (Did you mean to pass assign=Trueto assign items in the state dictionary to their corresponding key in the module instead of copying them in place?) warnings.warn(f'for {key}: copying from a non-meta parameter in the checkpoint to a meta ' /Data_PHD/***/anaconda3_35/envs/xtuner/lib/python3.10/site-packages/torch/nn/modules/module.py:2025: UserWarning: for vision_model.encoder.layers.23.layer_norm2.weight: copying from a non-meta parameter in the checkpoint to a meta parameter in the current model, which is a no-op. (Did you mean to passassign=Trueto assign items in the state dictionary to their corresponding key in the module instead of copying them in place?) warnings.warn(f'for {key}: copying from a non-meta parameter in the checkpoint to a meta ' /Data_PHD/***/anaconda3_35/envs/xtuner/lib/python3.10/site-packages/torch/nn/modules/module.py:2025: UserWarning: for vision_model.encoder.layers.23.layer_norm2.bias: copying from a non-meta parameter in the checkpoint to a meta parameter in the current model, which is a no-op. (Did you mean to passassign=Trueto assign items in the state dictionary to their corresponding key in the module instead of copying them in place?) warnings.warn(f'for {key}: copying from a non-meta parameter in the checkpoint to a meta ' /Data_PHD/***/anaconda3_35/envs/xtuner/lib/python3.10/site-packages/torch/nn/modules/module.py:2025: UserWarning: for vision_model.post_layernorm.weight: copying from a non-meta parameter in the checkpoint to a meta parameter in the current model, which is a no-op. (Did you mean to passassign=Trueto assign items in the state dictionary to their corresponding key in the module instead of copying them in place?) warnings.warn(f'for {key}: copying from a non-meta parameter in the checkpoint to a meta ' /Data_PHD/***/anaconda3_35/envs/xtuner/lib/python3.10/site-packages/torch/nn/modules/module.py:2025: UserWarning: for vision_model.post_layernorm.bias: copying from a non-meta parameter in the checkpoint to a meta parameter in the current model, which is a no-op. (Did you mean to passassign=True to assign items in the state dictionary to their corresponding key in the module instead of copying them in place?) warnings.warn(f'for {key}: copying from a non-meta parameter in the checkpoint to a meta '

@Jason8Kang
I also encountered this warning, but I tested this model on mmbench dev en, and its accuracy was normal. So, I did not delve into resolving this warning.


Update sensors as soon as read

Ensure that we wait for the updated data before updating the sensors

Update sensors after data download


lxd: Cleanup storage volumes properly for VMs

If the storage pool cannot be initialized using the new storage backend,
try removing the storage pool volume the old way.
This fixes #6461.
Signed-off-by: Thomas Hipp<EMAIL_ADDRESS>

Testsuite passed

Testsuite passed


Revert "Support async predictors (#2010)"

This reverts commit a86adcda6d511723bbe55f06555ffce8095fd6ac.
This is being done primarily to tag a v0.11.6 release that does not contain the predict-time timeout issue seen with some models running v0.11.5.
After this is merged and v0.11.6 is released, PLAT-410 may be considered done.

If you'd like, I think we could probably pair on this and get a fixed (rather than reverted) version out.

@nickstenning I'd much prefer that, yes!

Update: we do have a pretty good idea of how to not have to revert this, but reverting and tagging the next release first is the right call.


[Service Bus] Investigate/fix test failures

Fix approximation assumption on delivery count with abandon when using streaming receiver

The latest test run on master has the below 10 test cases failing:

  1) Standard
       Unpartitioned Topic/Subscription
         Tests - Lock Renewal - Peeklock Mode
           "before each" hook for "Receive a msg using Streaming Receiver, lock will expire sometime after the configured time":
     OperationTimeoutError: Unable to create the amqp sender 04487311-4439-8141-b904-33a44c1b3b8f due to operation timeout.
      at Timeout.actionAfterTimeout (node_modules\rhea-promise\dist\lib\session.js:290:31)

  2) Standard
       Unpartitioned Queue
         Tests - Lock Renewal - Peeklock Mode
           renewLock() with Batch Receiver resets lock duration each time.:
     SessionCannotBeLockedError: Received sessionId 'undefined' does not match the provided sessionId 'my-session' while creating the receiver 'unpartitioned-queue-79893971-8db1-3d42-b929-b0a6faad7941'.
      at Object.translate (node_modules\@azure\amqp-common\dist\lib\errors.js:527:17)
      at MessageSession.<anonymous> (test-dist\index.js:13897:53)
      at Generator.next (<anonymous>)
      at fulfilled (node_modules\tslib\tslib.js:104:62)
      at process._tickCallback (internal/process/next_tick.js:68:7)

  3) Standard
       Unpartitioned Queue
         Tests - Lock Renewal - Peeklock Mode
           "before each" hook for "Receive a msg using Batch Receiver, wait until its lock expires, completing it now results in error":
     SessionCannotBeLockedError: Received sessionId 'undefined' does not match the provided sessionId 'my-session' while creating the receiver 'unpartitioned-queue-97b277e8-055f-5141-8646-a0c9abb1d8a7'.
      at Object.translate (node_modules\@azure\amqp-common\dist\lib\errors.js:527:17)
      at MessageSession.<anonymous> (test-dist\index.js:13897:53)
      at Generator.next (<anonymous>)
      at fulfilled (node_modules\tslib\tslib.js:104:62)
      at process._tickCallback (internal/process/next_tick.js:68:7)

  4) Standard
       Partitioned Queue
         Tests - Lock Renewal - Peeklock Mode
           "before each" hook for "renewLock() with Batch Receiver resets lock duration each time.":
     OperationTimeoutError: Unable to create the amqp session due to operation timeout.
      at Timeout.actionAfterTimeout (node_modules\rhea-promise\dist\lib\connection.js:277:31)

  5) Standard
       Unpartitioned Topic/Subscription
         Tests - Lock Renewal - Peeklock Mode
           renewLock() with Batch Receiver resets lock duration each time.:
     SessionCannotBeLockedError: Received sessionId 'undefined' does not match the provided sessionId 'my-session' while creating the receiver 'unpartitioned-topic/Subscriptions/unpartitioned-topic-subscription-53aa13a1-b6ca-0840-8f02-f7ebf7c764da'.
      at Object.translate (node_modules\@azure\amqp-common\dist\lib\errors.js:527:17)
      at MessageSession.<anonymous> (test-dist\index.js:13897:53)
      at Generator.next (<anonymous>)
      at fulfilled (node_modules\tslib\tslib.js:104:62)
      at process._tickCallback (internal/process/next_tick.js:68:7)

  6) Standard
       Unpartitioned Topic/Subscription
         Tests - Lock Renewal - Peeklock Mode
           "before each" hook for "Receive a msg using Batch Receiver, wait until its lock expires, completing it now results in error":
     SessionCannotBeLockedError: Received sessionId 'undefined' does not match the provided sessionId 'my-session' while creating the receiver 'unpartitioned-topic/Subscriptions/unpartitioned-topic-subscription-93ddeefa-a64d-4248-8c1c-1144ae090c77'.
      at Object.translate (node_modules\@azure\amqp-common\dist\lib\errors.js:527:17)
      at MessageSession.<anonymous> (test-dist\index.js:13897:53)
      at Generator.next (<anonymous>)
      at fulfilled (node_modules\tslib\tslib.js:104:62)
      at process._tickCallback (internal/process/next_tick.js:68:7)

  7) Standard
       Partitioned Topic/Subscription
         Tests - Lock Renewal - Peeklock Mode
           renewLock() with Batch Receiver resets lock duration each time.:
     SessionCannotBeLockedError: Received sessionId 'undefined' does not match the provided sessionId 'my-session' while creating the receiver 'partitioned-topic/Subscriptions/partitioned-topic-subscription-99ee8c0f-fd8c-4449-92c5-ba803d5334fc'.
      at Object.translate (node_modules\@azure\amqp-common\dist\lib\errors.js:527:17)
      at MessageSession.<anonymous> (test-dist\index.js:13897:53)
      at Generator.next (<anonymous>)
      at fulfilled (node_modules\tslib\tslib.js:104:62)
      at process._tickCallback (internal/process/next_tick.js:68:7)

  8) Standard
       Partitioned Topic/Subscription
         Tests - Lock Renewal - Peeklock Mode
           "before each" hook for "Receive a msg using Batch Receiver, wait until its lock expires, completing it now results in error":
     SessionCannotBeLockedError: Received sessionId 'undefined' does not match the provided sessionId 'my-session' while creating the receiver 'partitioned-topic/Subscriptions/partitioned-topic-subscription-fa8d63b1-5fb2-a142-be23-1267b512548b'.
      at Object.translate (node_modules\@azure\amqp-common\dist\lib\errors.js:527:17)
      at MessageSession.<anonymous> (test-dist\index.js:13897:53)
      at Generator.next (<anonymous>)
      at fulfilled (node_modules\tslib\tslib.js:104:62)
      at process._tickCallback (internal/process/next_tick.js:68:7)

  9) Streaming Receiver - Misc Tests(with sessions)
       AutoComplete removes the message from Partitioned Subscription(with sessions):

      Unexpected number of msgs found when peeking
      + expected - actual

      -0
      +2

      at Object.should.equal (test-dist\index.js:7516:35)
      at C:\GitRepos\azure-sdk-for-js\packages\@azure\servicebus\data-plane\test-dist\index.js:20982:18
      at Generator.next (<anonymous>)
      at fulfilled (node_modules\tslib\tslib.js:104:62)
      at process._tickCallback (internal/process/next_tick.js:68:7)

  10) Topic Filters -  Send/Receive messages using correlation filters of subscription
       Correlation filter on the custom properties:

      AssertionError: expected 3 to equal 4
      + expected - actual

      -3
      +4

      at Object.should.equal (test-dist\index.js:7516:35)
      at Context.<anonymous> (test-dist\index.js:22133:22)
      at Generator.next (<anonymous>)
      at fulfilled (node_modules\tslib\tslib.js:104:62)
      at process._tickCallback (internal/process/next_tick.js:68:7)

The renew lock related failures will be fixed by https://github.com/Azure/azure-sdk-for-js/pull/1104

For Topic Filters failure, here is my theory:

We run receiveBatch(10), when the subscription has 4 messages and expect the resulting array to have 4 messages
But, if there has been network issues, and more than 1 second passes after receiving 3 messages, then receiveBatch just returns the 3 it has.

cc @ShivangiReja

Latest set of results:

  1) Standard
       Partitioned Queue
         Tests - Lock Renewal - Peeklock Mode
           "before each" hook for "renewLock() with Batch Receiver resets lock duration each time.":
     OperationTimeoutError: Unable to create the amqp session due to operation timeout.
      at Timeout.actionAfterTimeout (node_modules\rhea-promise\dist\lib\connection.js:277:31)

  2) Standard
       Partitioned Topic/Subscription
         Tests - Lock Renewal - Peeklock Mode
           "before each" hook for "renewLock() with Batch Receiver resets lock duration each time.":
     OperationTimeoutError: Unable to create the amqp sender 3d5fe6cc-18d7-d940-88f0-8284840a2e85 due to operation timeout.
      at Timeout.actionAfterTimeout (node_modules\rhea-promise\dist\lib\session.js:290:31)

  3) Standard
       Unpartitioned Queue
         Tests - Lock Renewal - Peeklock Mode
           "before each" hook for "Receive a msg using Streaming Receiver, lock expires after 30 sec when auto renewal is disabled":
     OperationTimeoutError: Unable to create the amqp sender 09a36e4a-4d4e-2f49-ae8c-2ddaf1e80f30 due to operation timeout.
      at Timeout.actionAfterTimeout (node_modules\rhea-promise\dist\lib\session.js:290:31)

  4) Standard
       Unpartitioned Topic/Subscription
         Tests - Lock Renewal - Peeklock Mode
           "before each" hook for "Receives a message using Streaming Receiver renewLock() resets lock duration each time.":
     OperationTimeoutError: Unable to create the amqp session due to operation timeout.
      at Timeout.actionAfterTimeout (node_modules\rhea-promise\dist\lib\connection.js:277:31)

  5) Standard
       Partitioned Topic/Subscription
         Tests - Lock Renewal - Peeklock Mode
           "before each" hook for "renewLock() with Batch Receiver resets lock duration each time.":
     InvalidOperationError: A link to connection '1277216' $cbs node has already been opened.
      at Object.translate (node_modules\@azure\amqp-common\dist\lib\errors.js:527:17)
      at CbsClient.<anonymous> (node_modules\@azure\amqp-common\dist\lib\cbs.js:87:32)
      at Generator.throw (<anonymous>)
      at rejected (node_modules\tslib\tslib.js:105:69)
      at process._tickCallback (internal/process/next_tick.js:68:7)

  6) Streaming Receiver - Deadletter message
       UnPartitioned Subscription: deadLetter() moves message to deadletter queue:

      Unexpected number of msgs found when peeking
      + expected - actual

      -0
      +2

      at Object.should.equal (test-dist\index.js:7516:35)
      at C:\GitRepos\azure-sdk-for-js\packages\@azure\servicebus\data-plane\test-dist\index.js:20401:18
      at Generator.next (<anonymous>)
      at fulfilled (node_modules\tslib\tslib.js:104:62)
      at process._tickCallback (internal/process/next_tick.js:68:7)

  7) Streaming Receiver - Deadletter message
       UnPartitioned Subscription with autoComplete: deadLetter() moves message to deadletter queue:

      Unexpected number of msgs found when peeking
      + expected - actual

      -0
      +2

      at Object.should.equal (test-dist\index.js:7516:35)
      at C:\GitRepos\azure-sdk-for-js\packages\@azure\servicebus\data-plane\test-dist\index.js:20401:18
      at Generator.next (<anonymous>)
      at fulfilled (node_modules\tslib\tslib.js:104:62)
      at process._tickCallback (internal/process/next_tick.js:68:7)

  8) Streaming Receiver - Misc Tests(with sessions)
       AutoComplete removes the message from Partitioned Subscription(with sessions):

      Unexpected number of msgs found when peeking
      + expected - actual

      -0
      +2

      at Object.should.equal (test-dist\index.js:7516:35)
      at C:\GitRepos\azure-sdk-for-js\packages\@azure\servicebus\data-plane\test-dist\index.js:21040:18
      at Generator.next (<anonymous>)
      at fulfilled (node_modules\tslib\tslib.js:104:62)
      at process._tickCallback (internal/process/next_tick.js:68:7)

  9) Streaming Receiver - Misc Tests(with sessions)
       AutoComplete removes the message from UnPartitioned Subscription(with sessions):

      Unexpected number of msgs found when peeking
      + expected - actual

      -0
      +2

      at Object.should.equal (test-dist\index.js:7516:35)
      at C:\GitRepos\azure-sdk-for-js\packages\@azure\servicebus\data-plane\test-dist\index.js:21040:18
      at Generator.next (<anonymous>)
      at fulfilled (node_modules\tslib\tslib.js:104:62)
      at process._tickCallback (internal/process/next_tick.js:68:7)

  10) Topic Filters -  Send/Receive messages using sql filters of subscription
       SQL rule filter on the custom properties:

      AssertionError: expected 3 to equal 4
      + expected - actual

      -3
      +4

      at Object.should.equal (test-dist\index.js:7516:35)

Failures in topic filter tests are fixed with https://github.com/Azure/azure-sdk-for-js/pull/1105
Failures in streaming receiver (with sessions) are due to peek() not giving the right results when used right after sending messages.

Latest run:
Abandon/Defer/Deadletter deferred message
       Partitioned Subscription: Abandoning a deferred message puts it back to the deferred queue.:
     Error: Timeout of 65000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves<EMAIL_ADDRESS>

  2) Standard
       Partitioned Topic/Subscription
         Tests - Lock Renewal - Peeklock Mode
           "before each" hook for "Receive a msg using Streaming Receiver, lock expires after 30 sec when auto renewal is disabled":
     OperationTimeoutError: Unable to create the amqp session due to operation timeout.
      at Timeout.actionAfterTimeout (node_modules\rhea-promise\dist\lib\connection.js:277:31)

  3) Standard
       Partitioned Queue
         Tests - Lock Renewal - Peeklock Mode
           "before each" hook for "Receive a msg using Batch Receiver, wait until its lock expires, completing it now results in error":
     OperationTimeoutError: Unable to create the amqp session due to operation timeout.
      at Timeout.actionAfterTimeout (node_modules\rhea-promise\dist\lib\connection.js:277:31)

  4) Standard
       Unpartitioned Topic/Subscription
         Tests - Lock Renewal - Peeklock Mode
           "before each" hook for "Receive a msg using Batch Receiver, wait until its lock expires, completing it now results in error":
     OperationTimeoutError: Unable to create the amqp session due to operation timeout.
      at Timeout.actionAfterTimeout (node_modules\rhea-promise\dist\lib\connection.js:277:31)

  5) Standard
       Partitioned Topic/Subscription
         Tests - Lock Renewal - Peeklock Mode
           "before each" hook for "Receives a message using Streaming Receiver renewLock() resets lock duration each time.":
     OperationTimeoutError: Unable to create the amqp sender 238eaff4-9f66-fa4c-b99f-513b48df290c due to operation timeout.
      at Timeout.actionAfterTimeout (node_modules\rhea-promise\dist\lib\session.js:290:31)

  6) Streaming Receiver - Misc Tests
       AutoComplete removes the message from UnPartitioned Subscription:

      Unexpected number of msgs found when peeking
      + expected - actual

      -0
      +2

      at Object.should.equal (test-dist\index.js:7516:35)
      at C:\GitRepos\azure-sdk-for-js\packages\@azure\servicebus\data-plane\test-dist\index.js:20401:18
      at Generator.next (<anonymous>)
      at fulfilled (node_modules\tslib\tslib.js:104:62)
      at process._tickCallback (internal/process/next_tick.js:68:7)

  7) Streaming Receiver - Deadletter message
       Partitioned Subscription: deadLetter() moves message to deadletter queue:
     Error: Timeout of 65000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves<EMAIL_ADDRESS>

  8) Streaming Receiver - Deadletter message(with sessions)
       Partitioned Subscription: deadLetter() moves message to deadletter queue(with sessions):

      Unexpected number of msgs found when peeking
      + expected - actual

      -0
      +2

      at Object.should.equal (test-dist\index.js:7516:35)
      at C:\GitRepos\azure-sdk-for-js\packages\@azure\servicebus\data-plane\test-dist\index.js:21040:18
      at Generator.next (<anonymous>)
      at fulfilled (node_modules\tslib\tslib.js:104:62)
      at process._tickCallback (internal/process/next_tick.js:68:7)

For any future reference, below are the set of PRs that fix various unreliable tests
#1099
#1104
#1105
#1115
#1120

Only one failing test remainiing!!!

throws error when receving streaming data via a queueClient from a non existing namespace

@HarshaNalluru Can you take a look?

Closing this issue as I intend to open individual ones to handle the pending failures


Add a Project post for LegalAI project

Add a project post for the LegalAI project.
Needed:

Information from project members, including

Intention/mission of the project
Goals/tangible outcome of the project

https://www.nico.northwestern.edu/people/visiting-scholars.html

Miriam could help with that.


Exporting attributes of shapes

Hi,
I am quite new to this library.
I do have the task to export the attributes of any shape from a Visio file. For now I do not have found any function within class vsdx.VisioFile.Shape that would allow looping over the attributes.
My code is as follows:
from vsdx import VisioFile
with VisioFile('Test_Prozess_neu.vsdx') as vis:
# open first page
page = vis.get_page(0)
# Read all shapes
shapes = page.shapes[0].sub_shapes()
# Loop over all available shapes
for shape in shapes:
print(shape.text)
print(shape.ID)
# Now I would normally execute the function for exporting all attributes..._
Do I do something wrong or does this function simply not exist (yet)?
Thanks for some support,
Frank

Hi Frank - always good to see people using this :)
Just want to check what sort of attributes you mean - i.e. VisioFile.Shape object fields/properties or attributes of the original shape in Visio.
For the VIsioFIle.Shape object you can list attributes using shape.__dict__
So something like print(f"shape {shape.ID} {shape.__dict__}") in your loop
You might notice the the Shape.cells property when you do that, which provides a list of many of the attributes within the original Visio shape as in truncated example below:
'cells': {'PinX': Cell: name=PinX val=4.133858135787355 func=None, 'PinY': Cell: name=PinY val=10.65551182326173 func=None, 'Width': Cell: name=Width val=2.165354297053872 func=None .... and so on
Some of these have dedicated properties for getting/setting, like x and y, but many do not.
I hope that helps
Regards
Dave

Hi Dave,
Thanks for the prompt reply!
I have tested the list attributes function shape.dict but unfortunatelly it is not showing the attributes/data of the shapes I am trying to get.
I am searching for the data/attribute fields that I can view or define when I do a right click upon a shape, then choose "Data" -> "Shape Data" or "Data" -> "Define Shape Data".
Please see attached screen shot. And sorry for the german language ;-)
Any ideas how I can extract this kind of information from the Visio file?
Regards,
Frank

I see what you mean now - let me look into that

Hello Frank - I have just released v0.4.7, which introduces ShapeProperty' class, a list of these being available from Shape.data_propertiesproperty. This is a basic initial implementation - it is lacking any specific consideration of master shape inheritence and of the value type (all values will be strings). You might want to look at the test method:test_get_shape_data_properties()` for guidance.
I hope this helps with what you were trying to do,
Kind regards
Dave

Hi Dave,
sorry for coming back that late upon this.
I have installed the new version and looked into the mentioned test method.
This is my code:
process properties
shapeProps = shape.data_properties
loop over the properties
for prop in shapeProps:
print("Property value: " + str(prop.value))
The loop does at least provide the correct amount of properties, being set on a shape.
But I do struggle with getting the property_name and property_value. The function prop.value always returns "None".
Would you have any advice for me, how to query the name and value of the property?
Does it have something to do with the dictionary that you are passing to the function as a parameter?
Thanks in advance.
Regards,
Frank

Hi Frank - looks like a bug in the library code (and test for this feature) rather than anything you are doing wrong. The fix is included in release v0.4.9 that I have just pushed here and to PyPI. I'll re-open this issue so we can close when properly resolved.

Hi Frank - just a heads up that I will change the Shape.data_properties type from list to dict in the next release, so just be aware of that.
The reason for this is to make it easier to get a specific property from a shape e.g. my_shape.data_properties['my_prop_name'].value

Hi Dave,
Thanks for taking care and for the heads up.
I am going to test v0.4.9 next week.
Have a nice weekend,
Frank

Hi Dave,
I do have tested now v0.4.10. We are almost there but I would need a last information from you.
When iterating over the data properties with the following code, I do only get the last data property as output.
for key, dataProperty in shape.data_properties.items():
print(dataProperty.getattribute('name'), '->', dataProperty.getattribute('value'))
Do I have to develop the loop differently or is the dictionary only having one item?
Regards,
Frank

Hi Frank - that code seems valid, it does loop through two data items in the test file.
If I add it to test test_get_shape_data_properties() it runs fine and adds the expected output lines below...
my_property_name -> property value
my_second_property_name -> another value
Note: The properties dict is indexed by label, not name - this is because the label is visible without opening the define data popup in Visio so it's a more convenient label for a shape - do your properties have labels?
it's unrelated, but you can also loop through as below:
for prop in shape.data_properties.values():
    print(prop.name, prop.value)

Dave

Hi Dave,
You are right, the code works perfect for all new defined data properties on a shape.
However, in my test file I have used a master shape that already had defined some data properties . If I enter some text to all data properties of the master shape, only the "last" one is being shown.
I would like to attach my test Visio file here, but github does not allow for uploading Visio files. So, I do only attach a screen-shot that shows the master shape and its data properties.

If I loop through the data properties of the master shape "shape.master_shape.items()" then I do get all data properties, but not the values.
Would you have another hint for me, how to get all values of any data property of the master shape?
Ciao,
Frank

That makes complete sense now :)
I didn't take account of master shape properties - I've updated and release v0.4.11 and hopefully this will now work as expected.
Thanks
Dave

Hi Dave,
I have installed release v0.4.11 and tested it with my Visio test file. The result is still the same:

When looking at the list of the shape properties it is showing all properties from the master shape and then it is repeating the last property having a value set. In my case the last property with a value is "Status".
Any further ideas from your side?
Cheers,
Frank

Hi Frank - the properties are indexed by the label, so if the master and shape both have a property with label 'label1' the shape property should override the master shape property with same label. I suspect that is what is happening here..
There should only be on property with label of 'Status' - but I guess that is the name not the label?
The output of on the right is really helpful, if you could add label, name, value, and shape.ID to that list it might help clarify what is going on.
Also - it looks like some values it is returning are incorrect based on the screenshot - I would have expected to see 'Staging', 'FHe' etc in there
Thanks
Dave

Hi Dave,
good news, now it is working perfectly!

Thanks for all of your efforts and support!
Have a great week,
Frank

Great news, took a few attempts but learned a bit along the way so always worthwhile :)
Thanks for comfirming so quick Frank
Cheers
Dave


How to Send Message

Hi ,
I was able to read all messsge , but i am unable to send messages..
Can anyone please help me with sample code to send a Hi Message?

@rajitbansal , Hi!! im new in this proyect and i not sure to understand how to implement the library, can you please help me with a little tutorial of use?? installation, web browser connection , wich enviroment to use , how to put my number or user,etc etc (things like these) and start reading messages like youw? of course i got some skills and i think we can help each other

@rajitbansal Im new here too.. do you made it works?

Its not working on Servers , however windows os


fix: Add CSV validation for font-style

Description
Added CSV validation to fix a couple of issues with the Font Style property in multiple widgets. The plan is to avoid the CSV validation in the future. Refer the Slack thread here.
Fixes #6797
Widgets whose font style property got modified:

TextWidget
Table Widget
Select widget
multi select widget
Single select tree
multi select tree
input widget

Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Tested manually
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:green_circle: Total coverage has increased
// Code coverage diff between base branch:release and head branch: fix/font-style-issue

Status
File
% Stmts
% Branch
% Funcs
% Lines

:green_circle:
total
55.73 (0.01)
37.16 (0.05)
35.3 (0.01)
55.98 (0.01)

:red_circle:
app/client/src/utils/autocomplete/TernServer.ts
52.71 (-0.23)
40.83 (-0.84)
36.21 (0)
56.74 (-0.25)

:green_circle:
app/client/src/utils/validation/common.ts
62.3 (3.28)
60.87 (3.37)
33.33 (0)
61.67 (3.34)

:green_circle:
app/client/src/workers/validations.ts
81.18 (-0.14)
70.91 (-0.78)
78.13 (1.46)
82.06 (-0.33)

/ok-to-test sha=9eb57dd

/ok-to-test sha=9eb57dd

/ok-to-test sha=db33594

/ok-to-test sha=705db58

/ok-to-test sha=be7e388

@aswathkk In table widget sub menu Font style if we switch between JS option the selected option gets reset
https://watch.screencastify.com/v/Qd8M6OWbuKrgmYYBtyC1

/ok-to-test sha=2a8b1f9

@aswathkk For drop-down widgets error is not coming for Label Font style for random text

Looks fine to me! One note is that we should discourage ValidationTypes.CSV from being used henceforth.
@sbalaji1192 Let's use standard ValidationTypes.ARRAY instead of CSV in Table Widget V2. Can you also comment on whether we need ValidationTypes.CSV in the InputWidget?

@riodeuno We are introducing a Select type column and the developer can give options in CSV format as well. But we use the validation function to validate. Regarding the Input widget, In v2 we're not using CSV format. (ideally, we should be using it for label style property, but in V2 it's not a js property somehow. will check with @dilippitchika .)

Thanks for pointing out these issues, @Rishabh-Rathod
Migration is not possible. Because this field used to accept anything!
For example, the following value in the Font style field of the Text widget will bold and italicise the text. But, we can’t migrate this:
Anything string that contains BOLD in it. If you wanna make it {{String.fromCharCode(73)}}{{"TA"}}LIC include it as a substring

@riodeuno Now, I feel that the field accepting CSV values and showing as an array in the Expected structure might confuse the users.

What if we only accept array values?

To not break existing applications, we will simply pass through the incoming values as it is to the widget rather than passing the default value on validation failure.
This will show errors on edit mode and allow users to manually migrate to the newly accepted format without breaking the look of their application.

What if we only accept array values?
To not break existing applications, we will simply pass through the incoming values as it is to the widget rather than passing the default value on validation failure

This was the behavior even i was expecting for this case.
Ideally, any Appsmith new changes should not start showing errors to users in their apps that had no errors earlier.
If we want to go with stricter validations and show an error to the user for old valid values (which won't be valid anymore) at least we should inform users why they see such errors now and what they could do to fix it.
cc: @aswathkk

Closing this since we are disabling the JS toggle button when the value can't be converted from JS code to UI.


How to configure schema retrieval from a url?

The readme says that a schema can be retrieved from a local file or a url, however I can't find any examples or documentation showing how to retrieve a schema from a url. How's it work?

Hi John.
Once you've generated the graphql.config.json from the editor in your IDE, follow the instructions at graphql.config.json#L9
The trick is to remove the "file" property in the config json since it take precedence over the request.
Note that the url can point to a GraphQL endpoint where it posts an introspection query, or it can point at the json from an introspection query. See graphql.config.json#L11-L14
I'll update the docs with examples in the near future. Let me know if you have any questions.
/Jim.

Thanks, everything's working now. Fantastic!


Add missing documentation

Description

Motivation and Context

The project file was missing <Content Include="packages.lock.json" />
Add missing xml documentation.

How Has This Been Tested?

Screenshots
Types of changes

[ ] Improvement (non-breaking change that improves a feature, code or algorithm)
[ ] Bug fix (non-breaking change which fixes an issue with code or algorithm)
[ ] New feature (non-breaking change which adds functionality to code)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Config and build (change in the configuration and build system, has no impact on code or features)
[ ] Dependencies (update dependencies and changes associated, has no impact on code or features)
[ ] Unit Tests (add new Unit Test(s) or improved existing one(s), has no impact on code or features)
[x] Documentation (changes or updates in the documentation, has no impact on code or features)

Checklist:

[ ] My code follows the code style of this project (only if there are changes in source code).
[ ] My changes require an update to the documentation (there are changes that require the docs website to be updated).
[ ] I have updated the documentation accordingly (the changes require an update on the docs in this repo).
[ ] I have read the CONTRIBUTING document.
[ ] I have tested everything locally and all new and existing tests passed (only if there are changes in source code).
[ ] I have added new tests to cover my changes.

Related to https://github.com/nanoframework/nanoframework.github.io/pull/139

I guess this is a comprimise.
It is better to not have warnings, and as it stands, the project was inconsistant with the approach.
Surely this is also useful for the generated documentation?
Either way, let me know whether this can be merged so we can move on.

@networkfusion this PR has conflict now. Either resolves them, either close the PR and reopen it. I was even thinking to add the linter on it. That would force to clean the comments as well.


fixes bug/typo

simple fix that addresses #470.

you need to update the snapshots with cargo insta test  --review. I tried to do it for you but I don't have write permission to the PR.

thank you @jdxcode, I have since synced my forked with your commit on both branches and I expect it to work now.
If it does not work, I'll updated the snapshots the way you described


.raw and .cr2 extension support

Many of untouched pictures are in their original formats. I think the support of these file extensions would be worth knowing that these files have the data to organize them. Please notify me if this is easily doable locally.

Thanks for reaching out & raising this feature. I am working on that feature on issue https://github.com/photo-cli/photo-cli/issues/39 to process companion files together.
I am planning to release this feature on version 0.3.0 in this month. You will able to archive by putting these .raw and .cr2 values on a setting file.

thank you!

Hi @Lain2077, I thought this feature I able to make it on May but I had to change too much things on the code base thus I able to finish it now and release a new version.
Can you give it a try when you have time?
I documented the companion feature on README.
You need to update to latest version with dotnet tool install --global photo-cli --version 0.3.0 and if you don't have the .NET 8 SDK, you also need to upgrade.
I would love to get feedbacks on this feature, hope it would be useful to you.


Adding denom to counts

Description

Updated ValidatorCommission.totalShares from String to CoinStr
Updated AssetDetail.supply from String to CoinStr
Updated AssetHolder.balance to include denom
Using actualy current height instead of deriving from signing infos

closes: #137

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md)
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Wrote unit and integration
[x] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[x] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

Shares are not coins. Rest of these seem to be good though. Not sure that shares should be exposed anyway… there is a recent discussion on the discord about this actually… it’s related to validator power and shares make that easier to calculate or something.

Shares are not coins. Rest of these seem to be good though. Not sure that shares should be exposed anyway… there is a recent discussion on the discord about this actually… it’s related to validator power and shares make that easier to calculate or something.

V true. Ill revert that


Table name or alias "call" misinterpreted by formatter

Describe the bug
When an alias for a table or expression is called "call", the formatter throws an error when trying to access its fields.
Expected behavior
Running
sql = `SELECT call.id as "id" FROM c as call`;
console.log(format(sql, { language: "postgresql" }));

should work, but it doesn't.
This, however, does:
sql = `SELECT calls.id as "id" FROM c as calls`;
console.log(format(sql, { language: "postgresql" }));

Actual behavior
I got this error:
Error: Parse error at token: . at line 1 column 12
Unexpected PROPERTY_ACCESS_OPERATOR token: {"type":"PROPERTY_ACCESS_OPERATOR","raw":".","text":".","start":11}.

I assume this might happen for other keywords as well, not only "CALL".
Usage

How are you calling / using the library? As described above
What SQL language(s) does this apply to? PostgreSQL
Which SQL Formatter version are you using? 13.1, but I also tested it on 15.3

Thanks for reporting.
This happens because the formatter expects CALL to be a start of a CALL statement. Similar problem happens with other keywords that mark the start of some statement like SELECT, UPDATE, etc.
Even when that particular crash were to be fixed (we could detect that CALL is followed by .), this will still leave the problem that call inside an alias definition would get formatted on a separate line (as again, the formatter assumes it'll start a CALL-statement):
SELECT
  call.id as "id"
FROM
  c as
call

You might be interested in trying out prettier-plugin-sql-cst, which has a much better support for PostgreSQL (although that support is far from 100%) and doesn't have fundamental problems like this.


feat: honor Caps Lock status when autocompletion.

Summary
Input lowercase or uppercase keyword per user input
Testing
manual testing

The code is updated to make the completion list refresh for each typing.

Assign it to @Zhirong2022 for interactive testing since it is ready for testing now.


[jsonrpc] Fix unit test failure on json unmarshal

Description
The unit test failed due to not implemented UnmarshalJSON method. The PR fixes it.
Changes include

[x] Bugfix (non-breaking change that solves an issue)

Testing

[x] I have tested this code with the official test suite

Codecov Report

Merging #272 (6888657) into dev (a840ee8) will decrease coverage by 3.17%.
The diff coverage is 39.63%.

@@            Coverage Diff             @@
##              dev     #272      +/-   ##
==========================================
- Coverage   51.21%   48.03%   -3.18%
==========================================
  Files         117      119       +2
  Lines       17525    18484     +959
==========================================
- Hits         8975     8879      -96
- Misses       7789     8861    +1072
+ Partials      761      744      -17

Impacted Files
Coverage Δ

blockchain/storage/kvstorage/kvstorage.go
54.16% <ø> (+2.46%)
:arrow_up:

consensus/ibft/hooks.go
55.26% <ø> (ø)

consensus/ibft/pos.go
0.00% <0.00%> (ø)

helper/keccak/keccak.go
0.00% <ø> (ø)

helper/keccak/pool.go
0.00% <ø> (ø)

jsonrpc/debug_endpoint.go
36.84% <0.00%> (ø)

network/bootnodes.go
100.00% <ø> (ø)

network/connections.go
98.07% <ø> (ø)

network/e2e_testing.go
63.76% <0.00%> (-11.77%)
:arrow_down:

protocol/status.go
0.00% <0.00%> (ø)

... and 76 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


run the demo fail, hint:taobao.tddl:tddl-client:jar:5.2.8-2 is missing, which is not in mvnrepository.com.

When I run demo, it fail.
The build log shows below, and it tell me miss taobao.tddl:tddl-client:jar:5.2.8-2, but it's can't be search in mvnrepository.com.

ok. I see at it.

When I run demo, it fail. The build log shows below, and it tell me miss taobao.tddl:tddl-client:jar:5.2.8-2, but it's can't be search in mvnrepository.com.

I see at it for a moment.

I have the same problem

I have the same problem


Collecting Kubernetes Objects and Events

Looking through the chart i cant see equivalent capabilities to https://github.com/splunk/splunk-connect-for-kubernetes/tree/develop/helm-chart/splunk-connect-for-kubernetes/charts/splunk-kubernetes-objects...how should k8 events be captured? Objects are generally less interesting, although useful, but events are essential.

Thanks, @dmitryax yes I think objects and have limited uses and thus are much lower priority.  Great to hear events are being added.

https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/5047

Any update on when k8s events can be sent out the Splunk platform exporter?

+1, could we have https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/receiver/k8seventsreceiver or simliar one to send k8s event logs to Splunk cloud?

OTel k8s events receiver is now adopted by the helm chart https://github.com/signalfx/splunk-otel-collector-chart/pull/478

thanks @dmitryax , I can see kube events in Splunk cloud now. 🎉

I'm closing this issue.
We are still don't have enough evidence for adding k8s objects support. If anyone thinks that it's required, please submit another issue.