Vuedraggable

ERROR
Cannot read properties of undefined (reading 'header')
TypeError: Cannot read properties of undefined (reading 'header')
at getSlot (webpack-internal:///./node_modules/vuedraggable/dist/vuedraggable.umd.js:2006:34)
at computeChildrenAndOffsets (webpack-internal:///./node_modules/vuedraggable/dist/vuedraggable.umd.js:2012:16)
at Proxy.render (webpack-internal:///./node_modules/vuedraggable/dist/vuedraggable.umd.js:2121:33)
at renderComponentRoot (webpack-internal:///./node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:1028:16)
at ReactiveEffect.componentUpdateFn [as fn] (webpack-internal:///./node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:6125:46)
at ReactiveEffect.run (webpack-internal:///./node_modules/@vue/reactivity/dist/reactivity.esm-bundler.js:221:19)
at instance.update (webpack-internal:///./node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:6256:16)
at setupRenderEffect (webpack-internal:///./node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:6266:5)
at mountComponent (webpack-internal:///./node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:6034:7)
at processComponent (webpack-internal:///./node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:5988:9)
ERROR
Cannot read properties of undefined (reading 'header')
TypeError: Cannot read properties of undefined (reading 'header')
at getSlot (webpack-internal:///./node_modules/vuedraggable/dist/vuedraggable.umd.js:2006:34)
at computeChildrenAndOffsets (webpack-internal:///./node_modules/vuedraggable/dist/vuedraggable.umd.js:2012:16)
at Proxy.render (webpack-internal:///./node_modules/vuedraggable/dist/vuedraggable.umd.js:2121:33)
at renderComponentRoot (webpack-internal:///./node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:1028:16)
at ReactiveEffect.componentUpdateFn [as fn] (webpack-internal:///./node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:6125:46)
at ReactiveEffect.run (webpack-internal:///./node_modules/@vue/reactivity/dist/reactivity.esm-bundler.js:221:19)
at instance.update (webpack-internal:///./node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:6256:16)
at setupRenderEffect (webpack-internal:///./node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:6266:5)
at mountComponent (webpack-internal:///./node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:6034:7)
at processComponent (webpack-internal:///./node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:5988:9)
ERROR
Cannot read properties of undefined (reading 'header')
TypeError: Cannot read properties of undefined (reading 'header')
at getSlot (webpack-internal:///./node_modules/vuedraggable/dist/vuedraggable.umd.js:2006:34)
at computeChildrenAndOffsets (webpack-internal:///./node_modules/vuedraggable/dist/vuedraggable.umd.js:2012:16)
at Proxy.render (webpack-internal:///./node_modules/vuedraggable/dist/vuedraggable.umd.js:2121:33)
at renderComponentRoot (webpack-internal:///./node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:1028:16)
at ReactiveEffect.componentUpdateFn [as fn] (webpack-internal:///./node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:6125:46)
at ReactiveEffect.run (webpack-internal:///./node_modules/@vue/reactivity/dist/reactivity.esm-bundler.js:221:19)
at instance.update (webpack-internal:///./node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:6256:16)
at setupRenderEffect (webpack-internal:///./node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:6266:5)
at mountComponent (webpack-internal:///./node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:6034:7)
at processComponent (webpack-internal:///./node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:5988:9)

same problem


can't save downloaded photo

I am using the telegram bot API, and i want to download a photo that was sent to my bot, and save it to my disk.
ive used GetInfoAndDownloadFileAsync method for downloading file, and then i tried to save the downloaded photo, but after saving, i can not open the saved file
here is my code:
private static async Task<byte[]> downloadFileAsync(string fileId)
        {
            Stream fileDestination = new MemoryStream();
            await botClient.GetInfoAndDownloadFileAsync(fileId, fileDestination);
            fileDestination.Position = 0;
            saveToDisk(fileDestination);
            byte[] bytes = new byte[fileDestination.Length];
            fileDestination.Read(bytes, 0, bytes.Length);
            return bytes;
        }

private static void saveToDisk(Stream fileDestination)
        {
            string fileFullPath = "F:\\My Files\\Picture\\pic.jpg";
            FileStream fileStream = System.IO.File.Create(fileFullPath, (int)fileDestination.Length);
            byte[] bytesInStream = new byte[fileDestination.Length];
            fileDestination.Read(bytesInStream, 0, bytesInStream.Length);
            fileStream.Write(bytesInStream, 0, bytesInStream.Length);
            fileStream.Close();
        }

and saved photo is attached

If you found a bug in our library please follow bug report template and provide minimum reproducible example we can work on. If you have any other issues feel free to ask others about their experiences in dealing with such issues in our support chat.


The status of subscriptions were wrong when select Modify subscription as process trigger

Date: 2016/06/02
[Test category]: creative Test
Severity：Major
[Steps to reproduce]

Login in marketplace as customer;
Click on "ACCOUNT->Processes->Add Process Trigger";
Fill the following fields "Display as", Type="Modify subscription" Target type="Web Service", Target URL="http://mockproductServer:port/oscm-integrationtests-mockproduct/NotificationService?wsdl",
check the "Suspends the execution of the process", then click on "Save" button;
Click on "HOME->Show Details->Get it now" to make a subscription, and assign customer to this subscription;
Click on "ACCOUNT->Subscriptions" to see the status of the subscription created in step 4;
Select the subscription created in step 4, click on "Configuration", modify the subscription name, then click on "Save" button;
Click on "ACCOUNT->Subscriptions" to see the status of the subscription created in step 4;
Click on "MY SUBSCRIPTIONS", select the subscription created in step 4, then click on "Use service" button.

[Expected result]
In step 5, the status should be "ready";
In step 6, some message "This action is subject to process control by your organization. Before it is finally executed, it must be approved in this process control system. Please wait for this approval." should be displayed;
In step 7, the status should be "suspended";
In step 8, the status should be "suspended", and no "Use service" button should be displayed.
[Observed result]
In step 5, the status was "ready";
In step 6, some message was displayed;
But in step 7, the status was "ready";
And in step 8, the subscription can be used successfully.

Please check.

Hi,
I think that the described behavior is correct. The checkbox "Suspends the execution of the proces" means, in this case, that modification process should be suspended not a subscription. Changes are waiting for approval but user still can use subscription without that changes.

The behavior described in [Observed result] is correct by design. Don't change this behavior.


Allow go back flag is not working in guided tour

Do you want to request a feature or report a bug?
bug
Bug Report
allowGoBack flag in advanceOnOptions of guided tour is not working as expected.
Steps to reproduce the issue

create a step in guided tour with no advanceOn parameter
Define advanceOnOptions and assign allowGoBack as false
back button is still visible.

StackBlitz template: https://stackblitz.com/edit/covalent-nsjjk5?file=app/app.component.ts
What is the expected behavior?
Back button should not be visible the allowGoBack is set to false.
Which version of Angular and Material, and which browser and OS does this issue affect?
Whatever the demo uses (all browsers)

I can reproduce this as well.
After looking a little, I found that the problem exists somewhere HERE.
I am trying to solve it. Will open a PR once I do.


Make riak_core_vnode_sup start options configurable

The default start options for the vnode supervisor don't fit every situation. I've added code to make it configurable by adding something like:
{riak_core_vnode_sup, [
{lifecycle_options, {
{simple_one_for_one, 10, 10},
[
{undefined, {riak_core_vnode, start_link, []}, temporary, brutal_kill, worker, dynamic}
]
}]
}
Which allows you to make vnodes fit the situation of your particular application (e.g. without brutal_kill etc...).

Hey, Jon.
Have you used this to change anything other then the shutdown method?

I haven't personally, no.

So what I'm wondering is if we really want a config for the entire child spec.  It seems just a config for the shutdown method is enough.  This could just be some undocumented thing that devs can make use of.  But I also wonder why the shutdown method is brutal_kill to begin with.  I wonder if this is something you'd want to be able to change per vnode type, but I don't think there is anyway to achieve that.
Anyways, I'm rambling.  Could you humor me and give me the use case where you needed this to be a timeout?


Bump DSE version to 6.8.31

What this PR does: Bump DSE default version in environment file to 6.8.31.
Which issue(s) this PR fixes: n/a
Fixes #: n/a
Checklist

[x] Changes manually tested
[ ] Automated Tests added/updated
[ ] Documentation added/updated
[x] CLA Signed: DataStax CLA

Hi @msmygit! Thank you for submitting this PR. But there is much more than is needed to upgrade DSE version Stargate uses.
The best way to see what is needed is to check
persistence-dse-6.8/README.md

and maybe have a look at a recent PR for upgrading DSE dependency, like #2251.
In addition, PR needs to be against v1 branch, since we need it to be updated for both "old" Stargate v1 (which is still needed for many environments) and then merged (Stargate team can do this) into main for Stargate v2.
I hope this helps.

Hi @tatu-at-datastax, I've created https://github.com/stargate/stargate/pull/2359 to address the v1 branch changes and have updated this PR to address other items.

OK let me try to explain this in detail.

The version updates are done in the v1 branch

this has already done in #2359, with additional fixes in https://github.com/stargate/stargate/pull/2360

Then we need to merge main and v1 so that changes done in v1 are also applied on the main branch
Once this is done, we only need to update the versions in the /apis/* (you correctly changed two needed files in this PR); however, we need no other changes!

I suggest you close this PR and open a new one for this purpose.. This new one should be similar to https://github.com/stargate/stargate/pull/2332


cmd prompt window wont pop up

So I did everything you said to do and inject the file on the login screen but no prompt window shows up and there is no indication that the mod is even working. I am playing the game on steam if that is an issue.

Steam shouldn't produce an issue, did you make sure you ran the injector as an administrator?

yes I did, I am using mabiNjector.exe

everything I did.
put PlaceHolder.ini in C:\Program Files (x86)\Steam\steamapps\common\Mabinogi
put Mabimod.dll in Desktop
used mabiNjector in administrator and rerouted the injection to C:\Program Files (x86)\Steam\steamapps\common\Mabinogi\client.exe
said it injected successfully and launched the game but no cmd prompt type window popped up and I couldn't tell or not if the mod was working.


Add Busola migrator patch job

Rewrite the virtual service migrator script for busola to Go.
Issue: https://github.com/kyma-project/kyma/issues/11795

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

/test all

/test all

/test pre-main-reconciler-protecode-guard


About StorageClass provisioner

Checking StorageClass in ARO since 4.11, it seems that the provisioner has changed to "file.csi.azure.com".
The documentation is "provisioner: kubernetes.io/azure-file". AKS has updated to "file.csi.azure.com", is there any change here?
Thanks,
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 3baa66d8-0696-9a88-dafd-567dab6d2eb5
Version Independent ID: d8e47064-b974-982b-b01b-638dac4c6831
Content: Create an Azure Files StorageClass on Azure Red Hat OpenShift 4 - Azure Red Hat OpenShift
Content Source: articles/openshift/howto-create-a-storageclass.md
Service: azure-redhat-openshift
GitHub Login: @johnmarco
Microsoft Alias: johnmarc

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

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

#please-close


Feature request: add API to add custom icons and functionality behind it to left bar.

Currently on left side bar VSCode have only 5 icons (explorer, search, git, debug, extensions).
Below this icons there is really huge empty area.
It would we great if we will have some API to add custom icons to this area and assign some command to it (as I know there is such possibility but only for bottom bar).
Some action require couple step to perform it (run some command):

open command palette
search some command and perform it
if in current command there is subcommands we need to perform couple command selection...

We can avoid this buy adding special icon with assigned action to left bar - only on click to perform some command (it will save us a lot of time during development).
For example on Atom there is extension "tool-bar" https://atom.io/packages/tool-bar that implement such functionality:

Huge empty area:

https://github.com/Microsoft/vscode/issues/710


Fix one defect of mvn command line eat error code.

Signed-off-by: Pan Li<EMAIL_ADDRESS>

Codecov Report

Merging #190 into master will increase coverage by 0.12%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #190      +/-   ##
==========================================
+ Coverage   80.46%   80.58%   +0.12%
==========================================
  Files          39       39
  Lines         911      917       +6
  Branches      114      115       +1
==========================================
+ Hits          733      739       +6
  Misses        121      121
  Partials       57       57

Impacted Files
Coverage Δ

.../spring/data/cosmosdb/core/DocumentDbTemplate.java
84.43% <100%> (+0.37%)
:arrow_up:

Continue to review full report at Codecov.

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


Refactoring of polygon editors to a common code base (v2)

Follow up of #10768.
This time more compact, cleaner and focused on just deduplicating the polygon editor code. No API changes.

This will need a rebase to solve recent merge conflicts, but before spending too much time rebase every other day, the current commits should be reviewed to confirm that it's the approach we want.

Ok I have to take a look at this.

I think the work is globally good. The function naming may be improved but I like the idea anyway.
I think it deserves a rebase.

Ok sorry for beging late, we discussed this PR in the PR monday.
Will merge as soon as it is rebased. :)

@groud rebased now.

Hum. I think this PR also needs a squash to make it a single commit. Please read how to make a squash here: http://docs.godotengine.org/en/stable/community/contributing/pr_workflow.html

@groud rebased. Is the authoring fixed now?

@groud rebased. Is the authoring fixed now?

Perfect. I forgot to merge this one... Merging then ;)


[Feature] Event with account view key in coinbase transaction if E::COINBASE_IS_PUBLIC = true.

🚀 Feature
Event with account view key in coinbase transaction if E::COINBASE_IS_PUBLIC = true.
We will need to transition this to a record view key in the future (hopefully this weekend). However, we will need this to ensure transactions are public for testnet2.
Note: In case this feature has challenges landing, we will require a view key during KYC.

This has been completed in snarkVM.


Rename rocm-> rocm-amdgpu-pro in README.md

There is no package named rocm anymore. Please update README.md.
sudo apt-get install rocm rocm-opencl-dev
To
sudo apt-get install rocm-amdgpu-pro rocm-amdgpu-pro-opencl-dev

This is not CORRECT,   ROCm is the correct driver package to install from ROCm repo at the instructions
ROCm driver is driver for the server market.   The AMDGPUpro recently added rocm functionality for only OpenCL support based on there packages.  The current ROCm packages are at https://rocm.github.io/ROCmLinuxpackages.html  should be run on the ROCm driver using these instructions  https://rocm.github.io/ROCmInstall.html


feat: adjustments for display short

will work with lens pr: https://github.com/samply/lens/pull/30

The catalogue has changed, so I will adapt this PR


ui: add progress bar when switching pages

resolve #126
What did:

Add progress bar in the top of page when switching pages

Just not sure whether this is a good way, because the NProgess is a jQuery-style lib which operates the DOM directly.
And the progress bar color is not very explicit. If we want to change that color, we need to copy the nprogress.css from node_modules and edit it.
Demo:

How about the solution react-top-loading-bar suggested in #126?

How about the solution react-top-loading-bar suggested in #126?

I glanced at its documents and example project - react-moviews, it seems need to change much code so I didn't try it. I can have a try today and see whether there are some other solutions.

New solution by  react-top-loading-bar. But there is a small issue, the loading bar works for most pages except the sign-in page. I know the reason, because single-spa:before-routing-event event is emitted before this page is loaded. So I think it can't work really globally.

Implement it by PR #661 , so close this PR.

Btw, this branch can be deleted.


build-snapshot: Remove jira ticket registration via test description in carina-utils/…

…../Jira.java, and unit test example

It is too superficial a decision.


Inital Microsoft Fabric datasource

I'm still working on this, and figuring out contributing stuff. I'll open a PR for suggestions and feedback, but I'll keep working on it until it's finished :)

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


Added version updating hook

Added version updating hook

This PR adds a version-updating pre-push Git hook to the project.
The functionality is similar to what we already have implemented for metacall/core.
The script is written in pure JavaScript to make it cross-platform.
An npm ci or npm install is required on all cloned repositories once the PR is merged.

CC: @viferga
Signed-off-by: Param Siddharth<EMAIL_ADDRESS>

@Creatoon Hey Raj! I think we still need to lint-format the /githooks directory. Although most of the project is in TypeScript, we will get the CI error until we format the scripts. So that is an update that needs to be made.
Update: I noticed the update after which we are ignoring the hooks from linting. I think that's a better decision.


thread 'main' panicked at 'failed to run git command

I was able to build sui-node from source starting with v0.9.0 up until v.0.16.0
Can't build sui-node with v.0.17.0 or v.0.18.0
Steps to Reproduce Issue
Can't build sui-node
Expected Result
sui-node binary to build correctly
Actual Result
error: failed to run custom build command for sui-node v0.18.0 (/build/sui/crates/sui-node)
Caused by:
process didn't exit successfully: /build/sui/target/release/build/sui-node-51780255154f293c/build-script-build (exit status: 101)
--- stderr
thread 'main' panicked at 'failed to run git command: fatal: not a git repository (or any of the parent directories): .git\n', crates/sui-node/build.rs:14:13
System Information
Ubuntu 20.04
Rust 1.66

You can't build from source, you've implemented a check to have a GIT_VERSION. So you must compile from a cloned git repo.


Unable to build (because of firebase update?)

Hi,
since yesterday I'm unable to build my project. No changes have been made on my side, and the build was working perfectly 3 days ago. If I remove this plugin the build is working again.
The error is this:
* What went wrong:
Execution failed for task ':app:transformDexArchiveWithExternalLibsDexMergerForDebug'.
> java.lang.RuntimeException: java.lang.RuntimeException: com.android.builder.dexing.DexArchiveMergerException: Unable to merge dex

Running gradlew with stacktrace option enabled we got this:
...
Caused by: java.lang.RuntimeException: java.lang.RuntimeException: com.android.builder.dexing.DexArchiveMergerException: Unable to merge dex
....
Caused by: com.android.dex.DexException: Multiple dex files define Lcom/google/android/gms/internal/measurement/zzkt;

I have noticed this:
....
Download https://maven.google.com/com/google/firebase/firebase-analytics/16.0.0/firebase-analytics-16.0.0.pom
Download https://maven.google.com/com/google/firebase/firebase-measurement-connector-impl/16.0.0/firebase-measurement-connector-impl-16.0.0.pom
Download https://maven.google.com/com/google/firebase/firebase-iid/maven-metadata.xml
Download https://maven.google.com/com/google/firebase/firebase-measurement-connector/16.0.0/firebase-measurement-connector-16.0.0.pom
Download https://maven.google.com/com/google/firebase/firebase-abt/16.0.0/firebase-abt-16.0.0.pom
Download https://maven.google.com/com/google/firebase/firebase-iid-interop/16.0.0/firebase-iid-interop-16.0.0.pom
Download https://maven.google.com/com/google/firebase/firebase-analytics/maven-metadata.xml
Download https://maven.google.com/com/google/firebase/firebase-messaging/17.0.0/firebase-messaging-17.0.0.aar
Download https://maven.google.com/com/google/firebase/firebase-config/16.0.0/firebase-config-16.0.0.aar
Download https://maven.google.com/com/google/firebase/firebase-measurement-connector-impl/16.0.0/firebase-measurement-connector-impl-16.0.0.aar
Download https://maven.google.com/com/google/firebase/firebase-crash/16.0.0/firebase-crash-16.0.0.aar
Download https://maven.google.com/com/google/firebase/firebase-auth/16.0.1/firebase-auth-16.0.1.aar
Download https://maven.google.com/com/google/firebase/firebase-core/16.0.0/firebase-core-16.0.0.aar
Download https://maven.google.com/com/google/firebase/firebase-perf/16.0.0/firebase-perf-16.0.0.aar
....

So seems like this could be the cause. As per this firebase releases note link, there have been a bunch of updates between May 23 and May 24.
In this stackoverflow post people are discussing a similar error, caused by another firebase update, solved by upgrading the google-services plugin. I have tried to update it, but no luck. I will try to update it to 4.0.1, released yesterday, and report back results.
If somebody has any clue to solve this, please, share it. I need to build a new version of my app urgently (you know, gdpr stuff).
Thanks in advanced.

Whats in your build gradle, are you just using + as your version?
it may be you need to go back to the versions you were using a few days ago..

@markterrill I'm not editing manually any file inside platforms folder. I'm using  cordova-android-play-services-gradle-release plugin. Futhermore, my build was working 2 days ago without any code change.
I have tried google-services 4.0.1 with no luck.
I'm out of ideas...

I'm having same issue since yesterday. Tried everything, nothing works.

More info:
Here seems to be more people affected.
I have tried to modify all firebase versions declared in this plugin to 15.+ but no luck either...

No.. the fix is SUPER EASY.
Just call "cordova clean". Thats it. It will work again after that.

I have called cordova clean multiple times, as well as platform
remove/platform add... No luck.
El vie., 25 may. 2018 21:13, Hirbod<EMAIL_ADDRESS>escribió:

No.. the fix is SUPER EASY.
Just call "cordova clean". Thats it. It will work again after that.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/arnesson/cordova-plugin-firebase/issues/705#issuecomment-392154236,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APL4ZODFThO3Kp4w3BzX1MaJXjHltgPNks5t2FfggaJpZM4UNdy-
.

Here same problem,
this plugin and cordova-android-play-services-gradle-release in the same project not work.
i have tried clean, re-add platform, force dependency to 15.+, to 16.+, i have changed cordova platform to 7.0.0, 7.1.0, 6.x with no solution... Only solution is remove cordova-android-play-services-gradle-release , the problem is that i need admob-free plugin in the same project :'(

@xfr4xx Thanks!! Is indeed a conflict with cordova-android-play-services-gradle-release. I can build again after delete that plugin. Weird indeed since that plugin is supposed to avoid this kind of problems...
If I got time, I will investigate what the cause of this conflict could be and report back to that plugin repo.
I understand that this error is not a bug with this cordova-plugin-firebase plugin, so the issue can be closed.
BTW, I have managed to solve the admob-free plugin play-services version conflict by using the preference PLAY_SERVICES_VERSION in cordova-admob-sdk as stated in its github: https://github.com/rehy/cordova-admob-sdk
In config.xml:
    <plugin name="cordova-admob-sdk" spec="^0.17.0">
        <variable name="PLAY_SERVICES_VERSION" value="15.+" />
    </plugin>

In package.json:
      "cordova-admob-sdk": {
        "PLAY_SERVICES_VERSION": "15.+"
      }

And then remove and re add android platform.

I have the same issue, without changing anything in my code.
first error was :transformClassesWithDexForDebug, which i got fixed by adding multiDexEnabled true to my build.grade defaultConfig. After that, :transformClassesWithJarMergingForDebug showed up.
cordova clean not helping
removing cordova-android-play-services-gradle-release did not help either.
removing and adding the android platform created a whole bunch of new errors, so so made a rollback
updated a bunch of plugins; updated cordova itself. nothing does help
can't get my work done. android sucks :/


[Bug]: Unable to UPDATE table variable in user-defined function

What happened?
We get the error: 'UPDATE' cannot be used within a function
This is with Babelfish 2.1
Code:
CREATE FUNCTION [dbo].[testfunc] (@FileSystemID int,@BeginDate datetime,@EndDate datetime)
RETURNS bigint AS
BEGIN
	Declare @totalFileSize bigint
	Declare @tmp table (FileSystemID int primary key,level int, RootID int, FileItemID int, sortNum varchar(8000))

...

	Update c Set SortNum = p.SortNum + right('0000000000' + convert(varchar, c.FileSystemID), 10),
		RootID = P.RootID
	From @tmp c Inner join FileSystem a on a.FileSystemID_int = c.FileSystemID
		Inner join @tmp P on a.FileSystemParentID_int = P.FileSystemID
	Where P.Level = @l - 1

Version
BABEL_1_X_DEV (Default)
Extension
No response
Which flavor of Linux are you using when you see the bug?
No response
Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct.

This is a known issue.


You can use the Rest-API - but how?

[Enter feedback here]
When you are mentioning that "Currently, PowerShell support for the new API version (2021-06-22) or the flag – DisableLocalAuth is not available. However, you can use the Rest-API with this API version to update the flag." you are not giving any explanation on how to really do it.
It would help at least that you cross reference this link
https://learn.microsoft.com/en-us/rest/api/automation/automation-account/update?tabs=HTTP
and give a sample on how to PATCH with
{
"properties": {
"disableLocalAuth": "True"
}
}
Thanks
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: d9d7b14e-482b-743d-4d1d-e1096b63fa39
Version Independent ID: a1e4ff6b-abe3-eed8-20ef-155d6aee3af7
Content: Disable local authentication in Azure Automation
Content Source: articles/automation/disable-local-authentication.md
Service: automation
Sub-service: process-automation
GitHub Login: @SnehaSudhirG
Microsoft Alias: sudhirsneha

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

@AlexCesarini - Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can review and update accordingly.

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


Set isort line_length to 99 to match with pycodestyle rule

I noticed, that there is an inconsistency in isort and pycodestyle rules, so this PR fixes that.

Let me quickly explain why we increased the soft wrap limit form 79 chars to 99 chars. We did increase the limit to avoid more hanging indentation or other weird constructs than necessary.
We all have pretty big screens these days. So there are some cases where it just makes more sense to write a longer line to improve readablity. Never the less, 79 char wraps are a good idea.
Imports are, at least in our case auto sorted and generated. PyCharm hides them in the code view by default and in any case I don't consider part of the code. That being said, does your change improve readability and does it even matter?

@codingjoe thank you for the explanation!
My thoughts here - imports are still part of the code. Even if Pycharm hides it by default (which I don't have), it is still there, and I guess, should follow same rules as the rest of the code. Also, definitely, not everyone is using Pycharm. :) In our company we have Vscode, and maybe something else. I even remember somebody was using vim, so you cannot expect to have hidden imports from everyone.
Also, it is, of course, a question of taste, and for me, since we are fine with 99 in the codebase, then I would like to avoid constructions like this if we can:
from django.db.models.fields.related import (
    ForeignKey, ManyToManyField, RelatedField
)

In any case, yes, imports are being sorted by isort, so we are not really worrying about it, that is true. So, I don't see this change as making it worse, only better (at least in terms of consistency).

@amureki I didn't want to make this about imports being part of the code or not. The important note from my side is, the recommended line limit is 79 chars + soft wrap. It's not 99. That also why all our plain texts have are wrapped at 79 chars. Because you can hardly make an argument for better readablity of a regular text for a given line.
I am happy to be convinced but currently I fail to see why imports should break the norm.

@codingjoe
Okay, I was not aware of soft wrap of 20 characters.
Plain texts, javascript, configs are different, our scope here is only a python code, to which imports are absolutely related. :)
But it is more a question of taste then, I feel them having the same rules (soft and hard) as rest part of the python codebase. That's why I made this proposal.
So it is up to you, feel free to close it!

Let's take the discussion into the team meeting. I am happy to increase the line length.

@codingjoe sure, that would be the best option! 🥇

@codingjoe hi, what about just removing the line about the line_length from config as we discussed? Would we be fine in the public packages with that?


Pick a target model from the list of supported TFLite GPU models.

There's a helpful list of the 4 models TFLite GPU supports here, including downloads of serialized tflite flatbuffers: https://www.tensorflow.org/lite/performance/gpu#supported_models_and_ops
We should pick one of these as our starting point and intentionally not try to build from source - if this is what they are telling users to download and use as a deployable it's something we should be able to import. Given these 4 models are almost all identical in terms of op composition it we can choose based on whatever is either coolest, most well-supported by our current optimization strategies, or just throw darts.

@mariecwhite recommended MobileNet SSD as the most relevant workload among the ones listed there.

Cool! We can then do a first-pass analysis to see which TOSA ops in particular we need (so importing the tflite flatbuffer into MLIR and lowering to the TOSA dialect via the PR sitting in tensorflow with the tool). That's tracked in #3949, so we can update this issue with what we find once one of us checks out the PR and runs it.

Apologies for the unsolicited input - we have internally run SSD Mobilenet V2 and V3 (and a few dozen more full networks) through the internal TOSA regression infrastructure. V2 passes with a bit exact match when running the TOSA form using the TOSA reference model  vs TFLite original output. The V3 version matches almost exactly, to within +/-1 in some elements due to an issue we identified in tfl.hardswish. It was raised in https://github.com/tensorflow/tensorflow/issues/40781 .

Thanks for bringing tensorflow/tensorflow#40781 to my attention, I replied there. +/- 1 differences are fine.

@sjarus very very welcome input! We're very excited to start plumbing things through! 👨🏼🔧

Thanks for your feedback @bjacob and @benvanik ! Of the four networks listed in the link in the first post, we've already run all but PoseNet, which is new to us.


Please outline requirements on how to migrate from J2ObjC -> J2ObjC-framework

this project is a really good idea but can you please provide some more flavor in:
"""
Unfortunatelly, this library doesn't compatable with original one in terms that there are different include paths in generated sources.
All inlcude/import with quotes was replaced with andgled/frameworked version.
"""
The generated j2objc code still has lots of
#import "IOSClass.h"
Do we need to manually patch these out of code to either:
@import j2objc;
or
#import <IOSClass.h>
Thanks!

Yes, i have implemented small python script: https://github.com/actorapp/actor-platform/blob/master/actor-sdk/sdk-core-ios/preprocess.py
That replaces imports


refactor to packetize and a whole mess of things

Hi,
This is going to be a good one!
I made a confd refactoring, changes include primarily how input parameters are treated and how to define cli arguments. Everything is configured via config structs (it helps packetizing confd #175) and cli arguments are created dynamically introspecting StructTags. Take a look to config/backend.go, config/global.go, and config/template.go. when it comes to cli handling codegangsta/cli library was used.
The most important thing in these changes is that I always follow the same pattern:

load from configuration file

for global config options [global] section in confd.toml is used.
for backend config options [etcd/consul/fs/...] section in confd.toml is used.
for template config options [template] is used in each template-defining file.

override if envar is defined and has a value
override if cli arg is provided

So the precedence order is as follows:

cli options
env var
configuration file

Also, I implemented the zero-conf mode (#183) using -tmpl-* cli arguments. Those are the same as the ones used in the configuration files. It can be rendered just one template per confd execution. Note that if confdir is provided zero-conf mode is ignored. It can be used like this
KEY=myvalue1 confd --onetime --tmpl-src "test.txt.tmpl" --tmpl-dest "test.txt" env

Yes, it is really opinionated but I think it is somehow organized and helps more than what it distrubs :-).
What do you think? @kelseyhightower @bacongobbler I would like to know your opinion on this, thanks!
PD.: This gist contains all configuration file examples and cli help outputs: https://gist.github.com/glerchundi/9b0fe054a157bc0eff24.

@glerchundi This is a huge PR, and will take a lot of time to review. It might turn out to be the case that there is to much refactoring here and I may ask that we break this up. I need to review first, but I just wanted to give you the heads up.

cool! if you just want to try it out, I compiled confd as you usually do:
https://github.com/glerchundi/confd/releases/download/v0.10.0-beta1/confd-0.10.0-beta1-darwin-amd64
https://github.com/glerchundi/confd/releases/download/v0.10.0-beta1/confd-0.10.0-beta1-linux-amd64

me again, I've created a test container which uses this confd version to load-balance http requests using nginx:
Concretely these are the files using confd:
https://github.com/just-containers/nginx-loadbalancer/blob/master/rootfs/etc/cont-init.d/01-confd-onetime
https://github.com/just-containers/nginx-loadbalancer/blob/master/rootfs/etc/services.d/confd/run

After a deeper review I'm not ready to move to a new package for managing flags. The goal right now is to clean up the backends, and a few more tests and get 1.0.0 out the door.
I think we can revisit this later, but I don't want to introduce such a big overhaul at this stage.


Dev osvr hdk2

This includes the commit from https://github.com/OpenHMD/OpenHMD/pull/74
Not sure if adding a 90° rotation around the x axis by multiplying the quaternion with
quatf rotatex = {
	.x = sqrt(0.5),
	.y = 0,
	.z = 0,
	.w = sqrt(0.5)
};

is the best way, but it works.

Quaternion rotations can be done by using the correct assignment of the quaternion values.
You should not need anything other then correct assignment of the values.
Please retry correct assignment.

I plan to learn some OpenGL soon and hopefully I will learn something about how to handle quaternions. In the meantime, I have no idea how to do that and found with google only that this is the way to go.

Turns out it was the only way.
https://github.com/OpenHMD/OpenHMD/commit/b296b2332c1864bc275531fcaba24b31128a2466


"Error: write ECONNABORTED" on deploying React App using Azure App Service extension

After logging into my Azure account from Visual Studio Code, I am trying to deploy my React app to Azure. After the deployment starts, I ran into the following error, and I have no idea what the cause is.
4:25:01 PM Demoapp: Creating zip package...
4:25:18 PM Demoapp: Starting deployment...
4:25:19 PM: Error: write ECONNABORTED

Hi @abhinandanmadaan
Is this still recurring?  It could have been just been a temporary outage.  If not, could you send me your React App so that I could try deploying it to investigate?

Yes I was able to deploy my app. And it seemed to be a temporary outage, and also a probable reason could be me sitting behind a corporate proxy. It was resolved on its own.

Thanks for confirmation!  Please reopen if you end up having issues again (but hopefully not)


Remove content/en/docs/setup/learning-environment/_index.md

This is a Bug Report

Problem:
According to the @kbhawkey 's comment,
There is an entry for this page in the _redirects file.
/docs/setup/learning-environment/     /docs/tasks/tools/  302!

Proposed Solution:
Remove content/en/docs/setup/learning-environment/_index.md
Page to Update:
~https://kubernetes.io/...~

/language en

/assign

If we remove that page, it'll also disappear from the left hand navigation. I'd prefer to leave it there with the comment about how to localize it correctly.
We could remove the page body if that helps folks?

@sftim Thank you for your comment!
According to @sftim 's comment,
I perform retitle.
/retitle Commentize/Remove page body of docs/setup/learning-environment/_index.md

This issue is resolved by the PR #28451 .


Reconnect IO log on Detach

fix(fio): bump fio to 3.37

This fixes an issue seen with fio 3.33 reporting an invalid miscompare.

Signed-off-by: Tiago Castro<EMAIL_ADDRESS>

fix(rebuild): connect the io log when detaching

When we detach a device, ensure that the io logs are connected.
This usually happens on the fault path, however this change
ensures that this happens during the detach itself and thus
hardening it against races.

Signed-off-by: Tiago Castro<EMAIL_ADDRESS>

bors merge


Progaurd rules for this library

Could you help me with coming up with a progaurd rule for this library

I ended up using this... was crazy ~_~
#FOR MATERIAL BAR CODE
#-keep class com.edwardvanraak.materialbarcodescanner.** {;}
#-keep interface com.edwardvanraak.materialbarcodescanner.** {;}
#-keep enum com.edwardvanraak.materialbarcodescanner.** {*;}
New rules for EventBus 3.0.x
http://greenrobot.org/eventbus/documentation/proguard/
-keepattributes Annotation
-keepclassmembers class ** {
@org.greenrobot.eventbus.Subscribe ;
}
-keep enum org.greenrobot.eventbus.ThreadMode { *; }
Only required if you use AsyncExecutor
-keepclassmembers class * extends org.greenrobot.eventbus.util.ThrowableFailureEvent {
(java.lang.Throwable);
}


ci: add EKS workflow

This workflow has been migrated from https://github.com/cilium/cilium-cli/ as part of the CI 3.0 initiative, and adapted based on the previous migration made for GKE.
See #15416 and #15482 for more details on the structure and adaptations made.
Triggers:

eks.yaml is triggered automatically when a comment starting with ci-eks is made on an PR. In this case, a GitHub status check is manually registered for the PR SHA commit, and will show up in the PR status checks with a link to the workflow run.
eks.yaml is also triggered automatically on merge to master. In this case a GitHub status check is already automatically registered by the push event, so we skip manual status check registering.
A commented pull_request trigger is available for workflow developers: it may be uncommented and used in test PRs for testing the workflow using the ci-run/gke label (requires write privileges as it will only work for PRs from branches in the Cilium repo, not from fork). Of course it should be left commented for the real PR.

Testing via #15468.

Working run with latest version here: https://github.com/cilium/cilium/actions/runs/690892438

Waiting for #15482 to be merged before removing draft status.

Link to test run for this version of the workflow, from #15468 : https://github.com/cilium/cilium/actions/runs/723299009


This commit fixes the two failing test cases TestEsRequired.java and

TestEsSearchOptions.java - bug#40598 and bug#40485 ( indentation in
map:with and es:optional in conversion module) and issue#68 in github
@bsrikan @grechaw please have a look, this cleans regression for functional tests.

OK I'll check these commits out and merge


Screenshots (typo)

Screenshots (typo)

Solved in https://github.com/valerio-vaccaro/HAN/commit/4aad32269005fa1c91607f86fcd94a16e4c18a02
Thanks!


Added command line option to allow several header lines

When i was working on some csv file i came across file which had 2 header lines, with this pull request i added an option where you can specify the number of header lines you want to skip.

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

Thanks!


feat(tests): adjusts e2e tests to be ran against 4.1 clusters

Short description of what this resolves:
There are some differences between Openshift 3.11 (currently used) and 4.1. This PR addressed them.
You can run this test suite against any cluster now using:
IKE_E2E_MANAGE_CLUSTER=false IKE_CLUSTER_USER=admin-user IKE_CLUSTER_PWD=admin-IKE_CLUSTER_HOST=xxx.yyy.zzz ISTIO_NS=istio-system-ns make test-e2e

ISTIO_NS is not needed and defaults to istio-system, but in case of our QE cluster is should be set to service-mesh-1
Changes proposed in this pull request:

allows specifying admin user credential through env var
starts istio only when the cluster is managed by e2e tests (defined through env var)
checking cluster version to determine how to enable required services (as they changed their names)
parsing shell scripts using shell lexer (so less error-prone then before -> less wrapping in bash -c)
shuffled helper funcs around

Fixes #225

TODO

[ ] Before merging adjust "branch protection" settings for the repo, as ci/circleci: e2e_tests does not exist anymore


M3-1871 Improve Tags styling

This PR covers few stories related to the display of tags through the app:

Show More tag wrapping
Show more popup placement
Chip markup (changed from button to div unless clickable)
Chip padding and font-size
Styling error on theme switch with Show More button

Could use some spacing inside the Show More box

Coverage increased (+0.003%) to 31.493% when pulling b859a0c6b10eddaf2192938f6bc1af64a46dc49e on alioso:M3-1871 into c5c4512d9bf4bd3674363aba9ccbcfa336cce566 on linode:develop.

@rjstires thanks, and you are certainly right but in this case this is the smallest value possible, which would not change whatever layout mode we'd be using


pool: Improve transaction (#201)

My general idea is to replace the old pending_payments queue with a database table(payment), to persist the data. make sure that afte the subtract the points, pending payment records still exists when the program exits unexpectedly, and the transaction failure retry can be achieved, finally moved confirm payment to a new task.

Thank you very much for @ppolewicz suggestion.
I don’t know if this PR is popular or not, this is a little improvement I made for this issue.
@mariano54 Do you want to ask some questions about this PR?

I agree with jack. While this looks like a very good change, people might want to do things differently. Also, it does add more code to the reference. Thank you for the PR though.


Signature overloading for arguments of type function fails

TypeScript Version:  2.1.4
In typescript 2.1.4 the compiler fails to narrow the correct signature overloading of a function when a arguments is a function which can optionally take an argument.
Code
function test(fn: () => void): void;
function test(fn: (x: number) => void): void;
function test(fn: (x?: number) => void) {
  fn(1);
};

test((x) => { });

Expected behavior:
Compile without errors, like previous version of typescript.
Actual behavior:
When compiling with --noImplicitAny I get the error error TS7006: Parameter 'x' implicitly has an 'any' type.
Also VSCode selects the correct overload:

but the type of x is any:

Duplicate #11936
The proper declaration for test looks like this
function test(fn: (x: number) => void): void;
function test(fn: () => void): void;
function test(fn: (x?: number) => void) {
  fn(1);
};

Sorry, I did not find that issue before opening this.
It's strange that the order of the overloads changes the compiler result. Still reading the other issue it seems that also before this example was not working correctly.
I'll close this.


"Complete" button does not immediately update to "next" on conditional page

Are you requesting a feature, reporting a bug or asking a question?
Bug
Reproducing
Given the JSON below, select the second Checkbox in Question 2.
Expected behaviour:

Question 3 is shown
The submit button becomes a "Next"-button immediately (to advance to page2)

Actual behaviour:

Question 3 is shown
The submit button is not changed and still displays "Complete"
When clicking on "Complete", it changes to "Next"

{
 "pages": [
  {
   "name": "page1",
   "elements": [
    {
     "type": "text",
     "name": "question1"
    },
    {
     "type": "checkbox",
     "name": "question2",
     "choices": [
      "item1",
      "item2",
      "item3"
     ]
    },
    {
     "type": "dropdown",
     "name": "question3",
     "visibleIf": "{question2} = [\"item2\"]",
     "title": "frage",
     "choices": [
      "item1",
      "item2",
      "item3"
     ]
    }
   ]
  },
  {
   "name": "page2",
   "elements": [
    {
     "type": "radiogroup",
     "name": "question4",
     "choices": [
      "item1",
      "item2",
      "item3"
     ]
    }
   ],
   "visibleIf": "{question2} = [\"item2\"]"
  }
 ]
}

Specify your

browser: Chrome
browser version: 70
surveyjs platform (angular or react or jquery or knockout or vue): react
surveyjs version: 1.0.32

@flxwu I have tried it with the latest version (1.0.53) and it works fine to me. Here is the example
Thank you,
Andrew


Is there some reference for cocktails?

I've been doing some casual reviewing of recipes, one thing I've noticed is the same recipe can vary a lot between sites.
For example, the After Dinner Cocktail specifies 60ml of lime juice. This is a lot more than other recipes I've seen.
This one states 1 large dash.
This one states the juice of one lime, and this says 1 lime = 1 fl oz = 30ml.
This one states 1cl lime juice (10ml).
Is there some cocktail authority that decides what counts as an "After Dinner Cocktail" (or anything else) or do we just do our best? I'd suggest this recipe is adjusted to have a lot less lime juice, perhaps 15ml.
I also have a bonus question: Is it ok if I alphabetically sort the data in cocktails.json, ingredients.json, and instructions.json? Or are you trying to keep these in the order in the cocktails-data.ts file that's sorted by ID?

Hi! Thank you for the questions
I'll try to follow the IBA (International Bartenders Association) standard. But I guess many recipes needs to be adjusted. In the beginning I used data from TheCocktailDB but I had to do a lot of manually work and rearrange the data structure to be able to add new features for example Imperial/Metric conversion and better language support.
I do believe that the cocktail data can and should be updated to be the "best" version of itself

For the translations files, I don't mind changing the order. When adding new cocktails I would probably find it easier to add it in the bottom but if there is some formatting command to sort in alphabetical order then its all good

I'll try to follow the IBA (International Bartenders Association) standard. But I guess many recipes needs to be adjusted. In the beginning I used data from TheCocktailDB but I had to do a lot of manually work and rearrange the data structure to be able to add new features for example Imperial/Metric conversion and better language support.
I do believe that the cocktail data can and should be updated to be the "best" version of itself

Thanks for this. It looks like the IBA cocktails are published on their site here: https://iba-world.com/iba-official-cocktail-list/
I'm guessing there is some intellectual property stuff that comes into play here. According to this, the recipe (I believe it means the ingredients and factual part of the method) can't be copyrighted, so if we write the recipe in our own words that should be fine.
According to that same source, cocktails can be patented, but anything older than 20 years should be safe.
I'll spend some time going through recipes and aligning them to IBA. I think it makes sense to use the exact recipe from IBA for IBA cocktails, since the "best" version is likely to be a matter of taste and IBA provides an authoritative source.
However, for non-IBA cocktails, we can just adjust as needed. It looks like the After Dinner Cocktail isn't IBA, but if you're happy I'll update this to have 15ml of lime juice to put it in the general vicinity of the other recipes.

For the translations files, I don't mind changing the order. When adding new cocktails I would probably find it easier to add it in the bottom but if there is some formatting command to sort in alphabetical order then its all good

Feel free to add to the bottom! My IDE keeps telling me I should sort it alphabetically, and provides an option to do it for me, and I thought it was a good idea.
However, the cocktails-data.ts file specifies the ID, and I think it makes sense to keep this in ID order. It will make it much easier to add new ones if we keep it this way.

I have a new question, but I'll close this and ask it in a new issue :slightly_smiling_face:


Relevance of masks in training a nnDetection model for nodule detection

:question: Question

Hello,
Assuming that we want to train a model for cancer nodule detection, and also, we want to use a dataset other than LIDC (say LNDb):

How do we prepare the dataset for training.  We don't have the contour coordinates specified for LNDb as is the case for LIDC.

Is it important that we merge the planar surfaces of ROIs in order to get the ground truth values?

Or can we approximate an ellipsoid just fitting the cuboid encompassing the nodule and hence obtain the ground truth values?

Thanking you
Rajesh

Hello @DSRajesh ,

Yes, it is definitely possible to approximate the nodules with elipsoids to train nndetection and the results will still be suitable - this is also what we have done for LUNA16. In some instance the segmentations could overlap (since boxes can overlap too) which need a manual strategy to be resolved.
There might some conflicting gradients when training with a mixture of exact segmentations and rough segmentation. This problem didn't come up for me yet so it is hard to comment on the exact performance degradation. I would experiment with different settings, e.g. train a model only on the subset with exact segmentation, train a model with a mixture of the annotations etc. and check their performance in the 5 Fold CV setting.

As I have described above, there is certainly some influence in the segmentation part of the model, and the multi-task learning might influence the detection performance as well. Also, there are differences with the augmentations; e.g. rotating an approximate structure and getting its axes aligned box introduces some errors in the training boxes. In case of exact segmentations, Retina U-Net outperformed its box-only counterpart (RetinaNet) consistently. Furthermore, there are multiple papers that used a segmentation approximation and showed improved results. I haven't seen literature where exact segmentations and approximations were mixed together.

Best,
Michael


Create fennel

Please add some reasonable stuff and let's merge after fixing.

This could maybe rather go into the issues section and be posted as a suggestion to write some "ingredient based search for recipies"?
But while I understand the predicament the submitter is in, I don't think this PR should be merged.


The Hugging Face link for SMPL motion samples is broken.

The Hugging Face link for SMPL motion samples is broken.

I'll resolve this problem. Thank you.

It fixed. Please check.


Add utility method to safely add shutdown hook

Description
Add CoreUtils.addShutdownHookSafely which adds a Runtime.addShutdownHook with the ability to use AccessController.doPrivileged if enabled to do so, allowing for the hook to be added if there is a SecurityManager running.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure:azure-core


Fix for #576: Change the way of how cleared relations get handled

Description
Previously, within a loop, it was checked whether a mapping had already been deleted and if so this mapping was added directly to the list of deleted relationships. This resulted in different results depending on the order of the mappings to be deleted.
By dividing this logic into two parts, the problem could be solved.
Related Issue
This PR solves #576.
How Has This Been Tested?
Test is provided with this PR and described in #576.
Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[ ] 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.

Thanks a lot for this really valuable PR!
As you should have noticed I have splittet the changes into to separate PRs and polished them a bit. I wanted to understand first what's really happening in the compile context, haven't touched that area since started to work on OGM. Feel free to chime in into the PR #581.


Update microsoft-defender-endpoint-linux.md

Fixed typos

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks

@mariusgx Please implement the feedback, if you agree. Thanks


Style Utilities : Dev-Border

How can I know that the border you are suggesting is right?
if the border renders in different colors in different screen sizes then there should be mentioned the size for every color.
you created a red-colored box where you put a tiny border in the middle. size should be specific it does not look good.
Here you use a button for expanding code. Did you use this button for every section? if not, why?
Why did you use pink color for the code section? which you are using?

The DEV_BORDER is used for development and testing/debugging purposes. So it must be which looks different from the other borders.

The border will be automatically applied to all the divisions and elements when the app is opened in the development mode.

Its target is not to look good rather to look different. so that elements in a page could be distinguished easily.

We are going add the code block to all the utilities.
The color of the elements is subjected to theme specific styling. So, it will change based on the predefined theme color.

provide me theme doc for testing
5. pending

The theme palette used for this documentation page at the time of this comment is as follows:

"common": {
"black": "#000",
"white": "#fff"
},
"type": "light",
"primary": {
"light": "#92f0a0",
"main": "#27d640",
"dark": "#038516",
"contrastText": "#fff"
},
"secondary": {
"light": "#FAFAFA",
"main": "#f50057",
"dark": "#FAFAFA",
"contrastText": "#fff"
},
"error": {
"light": "#e57373",
"main": "purple",
"dark": "#d32f2f",
"contrastText": "#fff"
},
"warning": {
"main": "#ff9800",
"light": "#ffb74d",
"dark": "#f57c00",
"contrastText": "rgba(0, 0, 0, 0.87)"
},
"info": {
"main": "#2196f3",
"light": "#64b5f6",
"dark": "#1976d2",
"contrastText": "#fff"
},
"success": {
"main": "red",
"light": "#81c784",
"dark": "#388e3c",
"contrastText": "rgba(0, 0, 0, 0.87)"
},
"grey": {
"50": "#fafafa",
"100": "#f5f5f5",
"200": "#eeeeee",
"300": "#e0e0e0",
"400": "#bdbdbd",
"500": "#9e9e9e",
"600": "#757575",
"700": "#616161",
"800": "#424242",
"900": "#212121",
"A100": "#d5d5d5",
"A200": "#aaaaaa",
"A400": "#303030",
"A700": "#616161"
},
"contrastThreshold": 3,
"tonalOffset": 0.2,
"text": {
"primary": "rgba(0, 0, 0, 0.87)",
"secondary": "rgba(0, 0, 0, 0.54)",
"disabled": "rgba(0, 0, 0, 0.38)",
"hint": "rgba(0, 0, 0, 0.38)"
},
"divider": "rgba(0, 0, 0, 0.12)",
"background": {
"paper": "#fff",
"default": "#fafafa"
},
"action": {
"active": "rgba(0, 0, 0, 0.54)",
"hover": "rgba(0, 0, 0, 0.08)",
"hoverOpacity": 0.08,
"selected": "rgba(0, 0, 0, 0.14)",
"disabled": "rgba(0, 0, 0, 0.26)",
"disabledBackground": "rgba(0, 0, 0, 0.12)"
}

As of now closing this issue


Delete branch not working

Description
When I try to delete a branch (local or local+remote) nothing happens. Not even an error message.
Neovim version
NVIM v0.8.0-dev+265-g12f45fd69
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3
Operating system and version
Ubuntu 20.04.4 LTS
Steps to reproduce

Open nvim in git folder with multiple branches
`:Neogit'
Try to delete a local branch using b + d
Choose a branch to delete

Expected behavior
The selected branch deleted
Actual behavior
Nothing happens, not even an error message, the branch isn't deleted.
Minimal config
vim.cmd([[set runtimepath=$VIMRUNTIME]])
vim.cmd([[set packpath=/tmp/nvim/site]])
local package_root = "/tmp/nvim/site/pack"
local install_path = package_root .. "/packer/start/packer.nvim"
local function load_plugins()
  require("packer").startup({
    {
      "wbthomason/packer.nvim",
      {
        "TimUntersberger/neogit",
        requires = {
          { "nvim-lua/plenary.nvim" },
          { "sindrets/diffview.nvim" },
        },
        config = function()
          print("loaded neogit")
          require("neogit").setup()
        end,
      },
    },
    config = {
      package_root = package_root,
      compile_path = install_path .. "/plugin/packer_compiled.lua",
      display = { non_interactive = true },
    },
  })
end
if vim.fn.isdirectory(install_path) == 0 then
  print("Installing neogit and dependencies.")
  vim.fn.system({ "git", "clone", "--depth=1", "https://github.com/wbthomason/packer.nvim", install_path })
end
load_plugins()
require("packer").sync()

After trying to delete a branch check the command history if the command git branch -d has the correct branch name

@CKolkey I know you've been tackling branches recently.
This issus may be of relevance

This issue is probably defunct - i've pretty much rewritten all of the branch deletion logic since this was opened.


Update config.toml

You need to rebase your repo on mine and make the link the way I described above. We will have some conflicts otherwise. Let me know if you need help with this.


Print query API version in logo

This will help identify problems when people post their output. We will be able to identify the version of the query API, useful for when implementation details may change (should most often be compatible, but older pre-releases did not have everything implemented).

@heaths,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Build vswhere 1.0.59+ge1514df141 completed (commit https://github.com/Microsoft/vswhere/commit/e1514df141 by @heaths)


Two header tags

I notice the markup includes two header tags, which seems incorrect to me- I am working on a fix, but wanted to make sure you didn't want both of these for some reason...
I am assuming this is a mistake as I have always thought that no single element should directly contain two  elements.

It depends of the context, you can have a header element inside an article for instance. Taking into account that and what you say, I decided to change the header tag for a section in the hero component and keep it in the articles.

Great- Yes, I think definitely better to just have a single header tag in this case.


Fix a false negative for Style/StringLiteralsInInterpolation for symbols with interpolation

Fixes #12059.

That was fast, thank you!
Now that I think of it, should we include RegEx interpolations here as well, for consistency?

That is a feature request, for sure, so probably better as a separate PR. Should we do that? I don't know 😄 That is also an interpolation, so probably yes.

Whatever we decide, I just did a double-take and realized that Style/StringLiterals is not working at all inside of a Regex interpolation! I'd call that a bug- I'll open a separate issue and raise the question there which error would be appropriate.

Thanks!


💡 [REQUEST] -  Embedding for text and images: is it teoretically possible?

起始日期 | Start Date
any
实现PR | Implementation PR
any
相关Issues | Reference Issues
#506
摘要 | Summary
Is it possible to extract the embeddings for images and text?
And if yes, will those embeddings lie in the same vectorial space, i.e., the embeddig of a dog's picture will be near the text "a dog"
基本示例 | Basic Example
none needed
缺陷 | Drawbacks
na
未解决问题 | Unresolved questions
na

Hello, it may need retraining at present, and the effect of direct alignment has not been tested. maybe you should look follow blog
https://modelbest.feishu.cn/wiki/NdEjwo0hxilCIikN6RycOKp0nYf?from=from_copylink


Add an entry point at runtime

Is it possible to add new entry points to an application that is already running?
I tried:
std = pkg_resources.get_distribution('setuptools')

pkg_resources.EntryPoint.parse_group(
    'distutils.commands', 'antialias = setuptools.command.alias:alias', std)

pprint(std.get_entry_map())

But the new entry point is not listed in the output.
https://stackoverflow.com/q/44661815/6683335

Basically no, because those entry points are defined only in the package metadata. It's conceivable you could install another package with additional entry points.
But if what you're seeking is to add a new command, I believe you can do that by updating the dist.Distribution.cmdclass.
It's still not clear to me why you would want to be dynamically updating distutils command classes at all. If what you're looking for is dynamic behavior, why not simply invoke that behavior (i.e. add your callables to a list and call them from there). What is it abut distutils.commands that's attracting you?


Update mark, unmark, statement commands

Closes #287

Codecov Report
Base: 70.31% // Head: 70.31% // No change to project coverage :thumbsup:

Coverage data is based on head (9354d2b) compared to base (ab1d3ff).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #288   +/-   ##
=========================================
  Coverage     70.31%   70.31%
  Complexity      713      713
=========================================
  Files           106      106
  Lines          2412     2412
  Branches        318      318
=========================================
  Hits           1696     1696
  Misses          599      599
  Partials        117      117

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


chore(main): release bin 1.15.6

:robot: I have created a release beep boop
1.15.6 (2024-09-03)
Bug Fixes

bin: use aerial-views dir to store wallpapers and set png and jpgs (20c4bf2)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

bin-v1.15.6
:sunflower:


Exporting Number Cell not recognised in Excel

When i export to XLS or XLSX, number columns are formatted as text. So i need to convert all cells to number ....
is there an option ?
i see that sheetjs was properly exporting numbers so why not the extension ?

Thanks for reporting. This extension extracts and exports the innerText of the cells in your matTable. What you have reported is actually an expected behavior however cell formatting support is in my to-do. I'll label this issue to track cell formatting feature.


propose direct dependencies

Add update_dependencies from the dependency graph (via DESCRIPTION) so that for every package we can list the complete distance 1 downstream & upstream packages. Please also add a distance to the dependency table

@maximilianmordig you can start with something as follows:
propose_staged_dependencies_config <- function(project = ".") {
  tbl <- dependency_table(project)

  tbl$package <- vapply(strsplit(tbl$repo, "/", fixed = TRUE), tail, character(1), 1)

  cur <- tbl[tbl$type == "current", ]
  nrow(cur) == 1 || stop("could not find current repo")

  fdesc <- file.path(project, "DESCRIPTION")
  dsc <- desc::desc(fdesc)

  pkgs <- dsc$get_deps()$package

  # get direct upstream
  internal_pkgs <- tbl[pkgs %in% tbl$package, ]

  # get direct downstream

  # create a new staged_dependencies.yaml file at
  yaml::write_yaml(new_sd, file = file.path(project, "staged_dependencies.yaml"))

}

@maximilianmordig I have asked @pawelru to do this issue to learn more about staged.dependencies

we need the a distance column in the dependency table based on DESCRIPTION

Hi all. I had a chance to look into this and this is my outcome:
get_descendants_distance <- function(parents_to_children, node) {
  internal_recursive_fun <- function(parents_to_children, node, distance = 0L) {
    if (is.null(parents_to_children[[node]])) {
      return()
    }
    c(
      lapply(parents_to_children[[node]], list, distance + 1L),
      unlist(lapply(parents_to_children[[node]], internal_recursive_fun, parents_to_children = parents_to_children, distance = distance + 1L), recursive = FALSE)
    )
  }
  internal_recursive_fun(parents_to_children, node) %>%
    c(stringsAsFactors = FALSE) %>%
    do.call(rbind.data.frame, .) %>%
    setNames(c("id", "distance")) %>%
    group_by(id) %>%
    summarise(distance = min(distance))
}

It should be run on internal deps variable. However, this is being determined by reading .staged.dependency.yaml files for all repos - not the DESCRIPTION file as written in the last comment

I don't find it clear what you mean by distance 1 packages. This assumes that each package is clean (according to R CMD check) so that only necessary upstream dependencies are listed in the DESCRIPTION file. Then, all packages listed in Imports or Depends in the DESCRIPTION file are distance 1 upstream packages. For the downstream packages of a package, we can go through all downstream packages according to the .staged.dependencies.yaml and check if they list the package as a dependency in the DESCRIPTION.
Is this what you meant with this issue?

@maximilianmordig  So I think if you only want direct dependencies then yes you can do what you suggest (and it of course relies on the staged_dependencies.yaml capturing all the packages you are interested in).
@waddella the issue says we only need this - or are you expecting to be able to say show me the dependencies with distance < x for some x in which case we'd need a distance

lets discuss it in our meeting today

So I think this is what we need (simplified pseudo code here):
get_dependency_structure <- function(starting_repo){

  repos_to_explore <- list(starting_repo)
  repos_explored <- list()

  R_deps <- named list() #dependencies from DESCRIPTION file go here

  while repos_to_explore not empty:

    current_repo <- repos_to_explore[1]

    checkout/copy repo into cache

    current_staged_deps <- deps in current_repo staged.yaml file
    append all current_staged_deps not in repos_to_explore or repos_explored into repos_to_explore

    R_deps[current_repo] <- suggest, imports and depends from current_repo description file
    remove current_repo from repos_to_explore

  for all R_deps remove values not in repos_explored

  # Now have a list of internal dependencies repos_explored and the true child->parent relationships
  # in R_deps

  R_deps <- topological_sort(R_deps)

  #use standard graph techniques to generate table which includes distances from starting_repo
  #(if relationship is other then distance will be NA and automation shouldn't check these repos out
  #when testing starting_repo)
  return(deps_table)
}

update_stage_dep_yaml_with_direct_deps <- function(project = "."){
  dep_structure <- get_dependency_structure()

  current_repo <- dep_structure[dep_structure$type == "current"]
  upstream_repos <- dep_structur[dep_structure$type == "upstream" & distance == 1]
  downstream_repos <- dep_structur[dep_structure$type == "downstream" & distance == 1]

  write_yaml(current_repo, upstream_repos, downstream_repos)
}

that is fine, I would still use the downstream info in the staged_dependency.yaml file if it exists.


Is reference script PlutusV2.txOutReferenceScript representation for Babbage era TxOutCompactRefScript correct?

Summary
It seems that PlutusV2 TxOut does not correspond to Babbage era TxOut. I mean we can not convert V2.TxOut to Babbage.TxOut without additional info on how scripts are stored.
I'm trying to convert Plutus V2 to Babbage Era Tx. The TxOutCompactRefScript expects TxOut to contain the full script
as it type goes:
  | TxOutCompactRefScript
       {-# UNPACK #-} !(CompactAddr (Crypto era))
       !(CompactForm (Core.Value era))
       !(Datum era)
       !(Core.Script era)

So it contains full script and it seems that we don't win in TxSize except that it's in the UTxO set arounnd TX.
But when I try to convert from Plutus V2 it has ScriptHash in txOutReferenceScript.
Should I calculate the map of ScriptHash to Script for conversion?
It seems that there is no way to store actual scripts in TxOut's on PlutusV2 level.
I see one way to keep them in datums. Are they supposed to be kept in Datums?
Steps to reproduce the behavior
Try to convert PlutusV2.TxOut to BabbageTxOut without additional info on ScriptHash to Script map.
Actual Result
It can not be converted. Or is it on purpose?
Expected Result

Describe the approach you would take to fix this
No response
System info
Manjaro Linux

The Plutus TxOut type exists for one purpose: to be the representation of transaction outputs used in ScriptContext. It is not intended to be equivalent to the real TxOut type for real transactions. It may omit information - in particular in this case we do not include the actual reference script in the ScriptContext because that would be pointless bloat, we just put the hash.

Thanks for clarification Michael!


Search query auto-completion deprecation

Since auto-complete is already implemented on Derpibooru, it will not be long before it will appear on the Furbooru. Once this feature is released, my implementation of auto-complete an be removed.

Auto-completion for Furbooru released today and extension simply conflicting with the site right now.


The "connectTimeout" parameter is not configurable

The URLConnection class has a "connectTimeout" setting that is not represented in the GitlabAPI class.
The result is a default wait of 2 minutes, which is not a productive option.
Check easily:
    @Test
    public void unitTest_20180417005924() {
        try {
            System.out.println(new Date());
            GitlabAPI api = GitlabAPI.connect("http://<IP_ADDRESS>:10080" ,"blabla");
            api.getVersion();
        } catch (Exception e) {
            System.out.println(new Date());
            System.out.println(e.getMessage());
        }
    }

Result:
Tue Apr 17 01:03:21 MSK 2018
Tue Apr 17 01:05:31 MSK 2018
Connection timed out (Connection timed out)

https://github.com/timols/java-gitlab-api/pull/298

Fixed: https://github.com/timols/java-gitlab-api/commit/58b0179c016875b21fd73224b44a12bcf46aec9b


Language switcher should be hide when Harbor is in integration mode

One more thing, the title in login page should be changed to 'VIC'

Change app title in sign-in page to 'VIC' when integration mode is enabled
Hide language switcher when integration mode is enabled

@lweitzman
Please help to verify.

i'm looking at https://<IP_ADDRESS>:8443/harbor/sign-in this one is probably old and has "Harbor" as title and the language picker.
is there another environment to see the integrated mode?


Use symbol where evailable

Inspired by react
https://github.com/facebook/react/blob/bc70441c8b3fa85338283af3eeb47b5d15e9dbfe/packages/shared/ReactSymbols.js
We currently use for the same purpose md5 strings, but we could first check if we can use symbols https://github.com/cssinjs/react-jss/blob/master/src/ns.js

wait, we still have no running tests, need to fix it, we can't release anything right now…


Allow '.' in idents for golang

Go symbols are in the format 'package.function'. Allow '.' in
ident so that you can attach uprobes to go functions.
Prior to this patch, the following would error:
$ objdump -tT /tmp/main  | grep main.doWork
objdump: /tmp/main: not a dynamic object
0000000000485360 g     F .text<PHONE_NUMBER>000049 main.doWork

$bpftrace -e 'uprobe:/tmp/main:main.doWork { printf("doWork\n"); }'
1.22: syntax error, unexpected ., expecting {

With this patch applied I am able to trace this function in a go binary.

I pushed an update to this branch implementing this as suggested by @ajor. I also added a test.

This is actually still insufficient for go. This fixes package level functions but methods on structs are more complicated. Given the following definition:
package main
type ExampleStruct struct {
}

func (s *ExampleStruct) PtrRecv() {
	fmt.Println("ptr")
}

The symbol shows up as main.(*ExampleStruct).PtrRecv.
Supporting this seems like it would be more complicated. Would you want to support escaping (\*)?

Thanks @psanford for working on this! Using your change, I was able to attach to some of my functions using uprobes. I'm hoping we can get Golang to be first class with bpftrace.
Another consideration that is relevant to this change is that Golang allows for any unicode letter or digit to be used in an identifier. For example:
package main

import "fmt"

func ⱑ() {
	fmt.Println("this works")
}

func main() {
	ⱑ()
}

This compiles to:
$ objdump -t unicode_test | grep main.ⱑ
0000000000484b10 g     F .text<PHONE_NUMBER>00006e main.ⱑ

However this doesn't appear to parse well:
$ bpftrace -e 'uprobe:/tmp/unicode_test:main.ⱑ { printf("nope\n"); }'
1.61: invalid character '�'
1.61-62: invalid character '�'
1.61-63: invalid character '�'
Attaching 1 probe...
Could not resolve symbol: /home/pivotal/Desktop/unicode_test/unicode_test:main.

Incidentally, I think developing a grammar for escapes might not be the only option we should consider. Another solution would be to just have a literal syntax. Something like this:
uprobe:/tmp/main:"ⱑ.*([{}]):" { printf("doWork\n"); }

This would allow for whatever bytes you want with 0 ambiguity. You would only have to escape the " which wouldn't be hard. It would support unicode, periods, parens, colon, whatever is a valid symbol byte.
Wildcards could just exist outside the literal:
uprobe:/tmp/main:*"literal" { printf("doWork\n"); }

Or they could exist inside the literal with special meaning and just have an escape for the literal value.
What do you think?

Thats a good point about unicode that I hadn't considered. Quoting does seem like a better option than trying to do escaping.

Quoting sounds good. Could probably just reuse the string type to do this

I pushed the change to support quoted strings. This fixes both the unicode issue and the pointer receiver issue.
In addition to updating the parser.yy I updated the AttachPoint class to have an explicit bool need_expansion that is set to false when we are in the string literal case. This short circuits the wildcard logic in add_probe so that we look for the literal '*' instead of trying to do an expansion.
I left the other attach_point form the same as before (need_expansion=true).

This looks good to me. Note that there's follow-up work to get uretprobes to work with go sanely; that's been discussed before ( https://github.com/iovisor/bcc/issues/1320 ).

Thanks!

@psanford Thanks a lot for doing this!


[stable/grafana]  Fix README.md layout of table broken.

Which issue this PR fixes

fixes #16826

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/ok-to-test

/test pull-charts-e2e

Shouldn't this PR be named [stable/grafana]? :)

/lgtm

@Eeemil @maorfr
Could you approve this?

/approve

/lgtm

/assign Eeemil

@Eeemil
Could you lgtm this?

/lgtm

Sorry, I tried :(

/assign @maorfr

/retest
/lgtm

/retest


Load from check point

Hi, thanks for sharing your great work!
Do you have any script for continuing train from saved checkpoint, it seems in train.py there isn't args for passing the checkpoint path etc., thanks!

Thanks to the excellent code.
Resume code is in  humannerf/core/train/trainers/human_nerf/trainer.py.
You can run it with
python train.py --cfg configs/human_nerf/zju_mocap/392/adventure.yaml resume True

thanks!

Thanks to the excellent code.
Resume code is in humannerf/core/train/trainers/human_nerf/trainer.py. You can run it with
python train.py --cfg configs/human_nerf/zju_mocap/392/adventure.yaml resume True


Adding Ticker websocket client, modified PublicClient to only return urls if url_only

Some other minor modifications.

Hi again @uzbit --
This sounds interesting... can you please add documentation for ticker?


NoIonicConvError from converged geometry optimizations

When I am calculating energy corrections for various defects as
pydefect efnv -d _/ -pcr perfect/calc_results.json -u ../unitcell/unitcell.yaml
But I am getting error: NoIonicConvError
I checked and found that geometry is converged.  Is there any way to get rid of this error?
INFO: -- Settings from vise.yaml:
INFO:
INFO: -- Settings from pydefect.yaml:
INFO:
INFO: Parsing data in Sn_Cl1_0 ...
WARNING: Ionic convergence in Sn_Cl1_0 is not reached.
Traceback (most recent call last):
File "/project/ktemp/anaconda3/lib/python3.9/site-packages/pydefect/cli/main_tools.py", line 54, in parse_dirs
_return = _inner_function(_dir)
File "/project/ktemp/anaconda3/lib/python3.9/site-packages/pydefect/cli/main_functions.py", line 183, in _inner
calc_results = get_calc_results(_dir, args.check_calc_results)
File "/project/ktemp/anaconda3/lib/python3.9/site-packages/pydefect/cli/main_functions.py", line 49, in get_calc_results
raise NoIonicConvError
pydefect.analyzer.calc_results.NoIonicConvError
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/project/ktemp/anaconda3/bin/pydefect", line 8, in
sys.exit(main())
File "/project/ktemp/anaconda3/lib/python3.9/site-packages/pydefect/cli/main.py", line 343, in main
args.func(args)
File "/project/ktemp/anaconda3/lib/python3.9/site-packages/pydefect/cli/main_functions.py", line 198, in make_efnv_correction_main_func
parse_dirs(args.dirs, _inner, args.verbose, file_name)
File "/project/ktemp/anaconda3/lib/python3.9/site-packages/pydefect/cli/main_tools.py", line 61, in parse_dirs
print(e.args[1])
IndexError: tuple index out of range

If the calculation is converged for sure, I don't know the reason.
Could you rerun the calculation from the converged CONTCAR file and run the same command again?

rerunning the calculation solved the error!


SSH tunnel not connect from DBeaver to latest linux distros.

System information:

Linux, macOS
DBeaver *

Connection specification:

SSH tunnel

Describe the problem you're observing:
SSH tunnel not connect from DBeaver to latest linux distros.
Steps to reproduce, if exist:
Install latest version of any linux distro (ubuntu22,fedora36,redhat8) and update it
Try to setup ssh tunnel from dbeaver connection with rsa key
Include any warning/errors/backtraces from the logs
2022-08-31 21:36:47.642 - pageant connect exception
com.jcraft.jsch.agentproxy.AgentProxyException: java.lang.NoClassDefFoundError: Could not initialize class com.jcraft.jsch.agentproxy.connector.PageantConnector$User32
        at com.jcraft.jsch.agentproxy.connector.PageantConnector.<init>(PageantConnector.java:68)
        at org.jkiss.dbeaver.model.net.ssh.SSHImplementationAbstract.initTunnel(SSHImplementationAbstract.java:110)
        at org.jkiss.dbeaver.model.net.ssh.SSHTunnelImpl.initializeHandler(SSHTunnelImpl.java:75)
        at org.jkiss.dbeaver.ui.net.ssh.SSHTunnelConfiguratorUI$7.execute(SSHTunnelConfiguratorUI.java:257)
        at org.jkiss.dbeaver.ui.net.ssh.SSHTunnelConfiguratorUI$TunnelConnectionTestJob.run(SSHTunnelConfiguratorUI.java:603)
        at org.jkiss.dbeaver.model.runtime.AbstractJob.run(AbstractJob.java:105)
        at org.eclipse.core.internal.jobs.Worker.run(Worker.java:63)
2022-08-31 21:36:47.643 - SSH: Connected with ssh-agent
2022-08-31 21:36:47.643 - Creating identity repository
2022-08-31 21:36:47.643 - Configure tunnel
2022-08-31 21:36:47.706 - Connect to tunnel host
2022-08-31 21:36:47.707 - SSH INFO: Connecting to -.-.-.- port 22
2022-08-31 21:36:47.740 - SSH INFO: Connection established
2022-08-31 21:36:47.782 - SSH INFO: Remote version string: SSH-2.0-OpenSSH_8.9p1 Ubuntu-3
2022-08-31 21:36:47.782 - SSH INFO: Local version string: SSH-2.0-JSCH-0.1.54
2022-08-31 21:36:47.782 - SSH INFO: CheckCiphers: aes256-ctr,aes192-ctr,aes128-ctr,aes256-cbc,aes192-cbc,aes128-cbc,3des-ctr,arcfour,arcfour128,arcfour256
2022-08-31 21:36:47.783 - SSH INFO: CheckKexes: diffie-hellman-group14-sha1,ecdh-sha2-nistp256,ecdh-sha2-nistp384,ecdh-sha2-nistp521
2022-08-31 21:36:47.792 - SSH INFO: CheckSignatures: ecdsa-sha2-nistp256,ecdsa-sha2-nistp384,ecdsa-sha2-nistp521
2022-08-31 21:36:47.792 - SSH INFO: SSH_MSG_KEXINIT sent
2022-08-31 21:36:47.811 - SSH INFO: SSH_MSG_KEXINIT received
2022-08-31 21:36:47.811 - SSH INFO: kex: server: curve25519-sha256,curve25519-sha256@libssh.org,ecdh-sha2-nistp256,ecdh-sha2-nistp384,ecdh-sha2-nistp521,sntrup761x25519-sha512@openssh.com,diffie-hellman-group-exchange-sha256,diffie-hellman-group16-sha512,diffie-hellman-group18-sha512,diffie-hellman-group14-sha256
2022-08-31 21:36:47.811 - SSH INFO: kex: server: rsa-sha2-512,rsa-sha2-256,ecdsa-sha2-nistp256,ssh-ed25519
2022-08-31 21:36:47.811 - SSH INFO: kex: server: chacha20-poly1305@openssh.com,aes128-ctr,aes192-ctr,aes256-ctr,aes128-gcm<EMAIL_ADDRESS>2022-08-31 21:36:47.811 - SSH INFO: kex: server: chacha20-poly1305@openssh.com,aes128-ctr,aes192-ctr,aes256-ctr,aes128-gcm<EMAIL_ADDRESS>2022-08-31 21:36:47.811 - SSH INFO: kex: server: umac-64-etm@openssh.com,umac-128-etm@openssh.com,hmac-sha2-256-etm@openssh.com,hmac-sha2-512-etm@openssh.com,hmac-sha1-etm@openssh.com,umac-64@openssh.com,umac-128@openssh.com,hmac-sha2-256,hmac-sha2-512,hmac-sha1
2022-08-31 21:36:47.811 - SSH INFO: kex: server: umac-64-etm@openssh.com,umac-128-etm@openssh.com,hmac-sha2-256-etm@openssh.com,hmac-sha2-512-etm@openssh.com,hmac-sha1-etm@openssh.com,umac-64@openssh.com,umac-128@openssh.com,hmac-sha2-256,hmac-sha2-512,hmac-sha1
2022-08-31 21:36:47.811 - SSH INFO: kex: server<EMAIL_ADDRESS>2022-08-31 21:36:47.812 - SSH INFO: kex: server<EMAIL_ADDRESS>2022-08-31 21:36:47.812 - SSH INFO: kex: server:
2022-08-31 21:36:47.812 - SSH INFO: kex: server:
2022-08-31 21:36:47.812 - SSH INFO: kex: client: ecdh-sha2-nistp256,ecdh-sha2-nistp384,ecdh-sha2-nistp521,diffie-hellman-group14-sha1,diffie-hellman-group-exchange-sha256,diffie-hellman-group-exchange-sha1,diffie-hellman-group1-sha1
2022-08-31 21:36:47.812 - SSH INFO: kex: client: ssh-rsa,ssh-dss,ecdsa-sha2-nistp256,ecdsa-sha2-nistp384,ecdsa-sha2-nistp521
2022-08-31 21:36:47.812 - SSH INFO: kex: client: aes128-ctr,aes128-cbc,3des-ctr,3des-cbc,blowfish-cbc,aes192-ctr,aes192-cbc,aes256-ctr,aes256-cbc
2022-08-31 21:36:47.812 - SSH INFO: kex: client: aes128-ctr,aes128-cbc,3des-ctr,3des-cbc,blowfish-cbc,aes192-ctr,aes192-cbc,aes256-ctr,aes256-cbc
2022-08-31 21:36:47.812 - SSH INFO: kex: client: hmac-md5,hmac-sha1,hmac-sha2-256,hmac-sha1-96,hmac-md5-96
2022-08-31 21:36:47.812 - SSH INFO: kex: client: hmac-md5,hmac-sha1,hmac-sha2-256,hmac-sha1-96,hmac-md5-96
2022-08-31 21:36:47.812 - SSH INFO: kex: client: none
2022-08-31 21:36:47.812 - SSH INFO: kex: client: none
2022-08-31 21:36:47.812 - SSH INFO: kex: client:
2022-08-31 21:36:47.812 - SSH INFO: kex: client:
2022-08-31 21:36:47.812 - SSH INFO: kex: server->client aes128-ctr hmac-sha1 none
2022-08-31 21:36:47.812 - SSH INFO: kex: client->server aes128-ctr hmac-sha1 none
2022-08-31 21:36:47.813 - SSH INFO: SSH_MSG_KEX_ECDH_INIT sent
2022-08-31 21:36:47.813 - SSH INFO: expecting SSH_MSG_KEX_ECDH_REPLY
2022-08-31 21:36:47.847 - SSH INFO: Host '-.-.-.-' is known and matches the ECDSA host key
2022-08-31 21:36:47.847 - SSH INFO: SSH_MSG_NEWKEYS sent
2022-08-31 21:36:47.847 - SSH INFO: SSH_MSG_NEWKEYS received
2022-08-31 21:36:47.847 - SSH INFO: SSH_MSG_SERVICE_REQUEST sent
2022-08-31 21:36:47.879 - SSH INFO: SSH_MSG_SERVICE_ACCEPT received
2022-08-31 21:36:47.916 - SSH INFO: Authentications that can continue: publickey,keyboard-interactive,password
2022-08-31 21:36:47.916 - SSH INFO: Next authentication method: publickey
2022-08-31 21:36:47.953 - SSH INFO: Disconnecting from -.-.-.- port 22
2022-08-31 21:36:48.055 - Auth fail
com.jcraft.jsch.JSchException: Auth fail
        at com.jcraft.jsch.Session.connect(Session.java:519)
        at com.jcraft.jsch.Session.connect(Session.java:183)
        at org.jkiss.dbeaver.model.net.ssh.SSHImplementationJsch.setupTunnel(SSHImplementationJsch.java:116)
        at org.jkiss.dbeaver.model.net.ssh.SSHImplementationAbstract.initTunnel(SSHImplementationAbstract.java:133)
        at org.jkiss.dbeaver.model.net.ssh.SSHTunnelImpl.initializeHandler(SSHTunnelImpl.java:75)
        at org.jkiss.dbeaver.ui.net.ssh.SSHTunnelConfiguratorUI$7.execute(SSHTunnelConfiguratorUI.java:257)
        at org.jkiss.dbeaver.ui.net.ssh.SSHTunnelConfiguratorUI$TunnelConnectionTestJob.run(SSHTunnelConfiguratorUI.java:603)
        at org.jkiss.dbeaver.model.runtime.AbstractJob.run(AbstractJob.java:105)
        at org.eclipse.core.internal.jobs.Worker.run(Worker.java:63)

2022-08-31 21:23:04.152 - pageant connect exception
com.jcraft.jsch.agentproxy.AgentProxyException: java.lang.NoClassDefFoundError: Could not initialize class com.jcraft.jsch.agentproxy.connector.PageantConnector$User32
        at com.jcraft.jsch.agentproxy.connector.PageantConnector.<init>(PageantConnector.java:68)
        at org.jkiss.dbeaver.model.net.ssh.SSHImplementationAbstract.initTunnel(SSHImplementationAbstract.java:110)
        at org.jkiss.dbeaver.model.net.ssh.SSHTunnelImpl.initializeHandler(SSHTunnelImpl.java:75)
        at org.jkiss.dbeaver.ui.net.ssh.SSHTunnelConfiguratorUI$7.execute(SSHTunnelConfiguratorUI.java:257)
        at org.jkiss.dbeaver.ui.net.ssh.SSHTunnelConfiguratorUI$TunnelConnectionTestJob.run(SSHTunnelConfiguratorUI.java:603)
        at org.jkiss.dbeaver.model.runtime.AbstractJob.run(AbstractJob.java:105)
        at org.eclipse.core.internal.jobs.Worker.run(Worker.java:63)
2022-08-31 21:23:04.152 - SSH: Connected with ssh-agent
2022-08-31 21:23:04.152 - Creating new SecureRandom.
2022-08-31 21:23:04.153 - No such algorithm: IDEA/CBC/NoPadding
2022-08-31 21:23:04.154 - No such algorithm: IDEA/CTR/NoPadding
2022-08-31 21:23:04.156 - Disabling high-strength ciphers: cipher strengths apparently limited by JCE policy
2022-08-31 21:23:04.242 - Client identity string: SSH-2.0-SSHJ_0.33.0
2022-08-31 21:23:04.293 - Server identity string: SSH-2.0-OpenSSH_8.9p1 Ubuntu-3
2022-08-31 21:23:04.569 - Exhausted available authentication methods
net.schmizz.sshj.userauth.UserAuthException: Exhausted available authentication methods
        at net.schmizz.sshj.SSHClient.auth(SSHClient.java:230)
        at org.jkiss.dbeaver.model.net.ssh.SSHImplementationSshj.setupTunnel(SSHImplementationSshj.java:110)
        at org.jkiss.dbeaver.model.net.ssh.SSHImplementationAbstract.initTunnel(SSHImplementationAbstract.java:133)
        at org.jkiss.dbeaver.model.net.ssh.SSHTunnelImpl.initializeHandler(SSHTunnelImpl.java:75)
        at org.jkiss.dbeaver.ui.net.ssh.SSHTunnelConfiguratorUI$7.execute(SSHTunnelConfiguratorUI.java:257)
        at org.jkiss.dbeaver.ui.net.ssh.SSHTunnelConfiguratorUI$TunnelConnectionTestJob.run(SSHTunnelConfiguratorUI.java:603)
        at org.jkiss.dbeaver.model.runtime.AbstractJob.run(AbstractJob.java:105)
        at org.eclipse.core.internal.jobs.Worker.run(Worker.java:63)
2022-08-31 21:25:04.331 - Dying because - Broken transport; encountered EOF
2022-08-31 21:25:04.331 - Disconnected - UNKNOWN

Possible workaround is decreasing ssh security on remote host:
add 2 lines to sshd config and restart it
PubkeyAuthentication yes
PubkeyAcceptedKeyTypes=+ssh-rsa

Can't reproduce it on Pop!_OS 22.04

I have a gut feeling that this is a duplicate of https://github.com/dbeaver/dbeaver/issues/14749.

https://github.com/dbeaver/dbeaver/issues/17312

https://github.com/dbeaver/dbeaver/issues/16247

Another workaround is not using the agent, but selecting "Public Key" as authentication method.

Possible workaround is decreasing ssh security on remote host: add 2 lines to sshd config and restart it
PubkeyAuthentication yes
PubkeyAcceptedKeyTypes=+ssh-rsa

I had this same problem on 22.04. I only needed to add PubkeyAcceptedKeyTypes=+ssh-rsa to /etc/ssh/sshd_config and restart sudo systemctl restart ssh

Seems it has nothing to do with DBeaver itself but with ssh configuration. Though ssh-related libraries may require upgrade, as i remember issues about missing support of the recent key types walkarounded by switching the ssh-related library explicitly.
See OpenSSH 8.2 release notes:

Security
========
 * ssh(1), sshd(8), ssh-keygen(1): this release removes the "ssh-rsa"
   (RSA/SHA1) algorithm from those accepted for certificate signatures
   (i.e. the client and server CASignatureAlgorithms option) and will
   use the rsa-sha2-512 signature algorithm by default when the
   ssh-keygen(1) CA signs new certificates.

...
Potentially-incompatible changes
================================
This release includes a number of changes that may affect existing configurations:
  * ssh(1), sshd(8): the above removal of "ssh-rsa" from the accepted CASignatureAlgorithms list.
...

It seems one doesn't even always need to change keys of the affected authorities, just upgrade both client and server software for both sides to use another sighning alorithm. See also OpenSSH 8.7 release notes:

...

Imminent deprecation notice
===========================

OpenSSH will disable the ssh-rsa signature scheme by default in the
next release.

In the SSH protocol, the "ssh-rsa" signature scheme uses the SHA-1
hash algorithm in conjunction with the RSA public key algorithm.
It is now possible[1] to perform chosen-prefix attacks against the
SHA-1 algorithm for less than USD$50K.

Note that the deactivation of "ssh-rsa" signatures does not necessarily
require cessation of use for RSA keys. In the SSH protocol, keys may be
capable of signing using multiple algorithms. In particular, "ssh-rsa"
keys are capable of signing using "rsa-sha2-256" (RSA/SHA256),
"rsa-sha2-512" (RSA/SHA512) and "ssh-rsa" (RSA/SHA1). Only the last of
these is being turned off by default.

This algorithm is unfortunately still used widely despite the
existence of better alternatives, being the only remaining public key
signature algorithm specified by the original SSH RFCs that is still
enabled by default.

The better alternatives include:

 * The RFC8332 RSA SHA-2 signature algorithms rsa-sha2-256/512. These
   algorithms have the advantage of using the same key type as
   "ssh-rsa" but use the safe SHA-2 hash algorithms. These have been
   supported since OpenSSH 7.2 and are already used by default if the
   client and server support them.
...

It looks like it is related to using JSCH for our interactions with ssh agents. This library can't work correctly with the new RSA 2 keys.

I just wanted to mention that I am reading through this issue as part of evaluating DBeaver for my needs, and this issue is a blocker for us. So please take that as a vote to move to a modern, maintained ssh library which supports secure ciphers. Weakening our security posture by allowing the old cipher would not be a suitable workaround, though I understand why others might choose to do it.
I see from a close read of #14749 that DBeaver has been dealing with the fact that its ssh libraries have stalled on cipher support (and one of them is totally unmaintained) for two years now. I'm not here to find fault or get into an argument; I offer this in a spirit of constructive criticism because I want your product to be better.
It is normal for old ciphers to be discontinued, and you can expect this to continue happening on a regular basis - it is the nature of cryptography to be subject to attacks which force continued improvement. In an ideal world, new ciphers would be supported as soon as they are standardized, without waiting for discontinued ciphers to force the issue.
I know that you can't afford to prioritize your product development solely on the say-so of one user, who may or may not ever become a paying customer, so please just take this for what it's worth. Good luck with everything!


feat: include chain state price data in the stamps endpoint response

This change is

@agazso

I think it would be best to return /stamps/price/total /stamps/price/current and /stamps/price/block.

I think it would be best to return /stamps/price/total /stamps/price/current and /stamps/price/block.

That would require having 3 separate endpoints, one for each value. What is the use case for having it like that?
If there is a use case for querying the chain state price data without postage batches, then I'd recommend having it under /stamps/price endpoint. @acud ?

I believe there is a usecase for querying chain state price date without postage batches (e.g. when I create a new batch and I want to figure out the current price to estimate what value to put).
In my previous comment, I meant that "price":{"block":4813080,"total":12,"current":1}} is better than "price":{"block":4813080,"totalAmount":12,"currentPrice":1}}, as the latter has some duplication and is longer.

I support @mrekucci here and I think that having it under one endpoint is better for various reasons and it would allow more information to be displayed to the user with less calls to the API. It would also be more informative for us for debugging purposes.


fix position of search and status bar

Large flame graphs not user friendly #702

I implemented this in 91691ce0396a79b923672bbfa1dc15c6e7e87368 in a slightly different but a similar way.