Enhance clarity with detailed steps for reflecting changes on control plane nodes via kubeadm-config changes

This is a Feature Request

What would you like to be added
The process to reflect changes on the control plane node after editing the kubeadm-config map isn't clear and should be stated in clearly defined steps
Why is this needed
This is really vital because it's a critical process and mistakes could literally be costly.

Page related to issue: https://kubernetes.io/docs/tasks/administer-cluster/kubeadm/kubeadm-reconfigure/#reflecting-clusterconfiguration-changes-on-control-plane-nodes
/language en

/retitle Enhance clarity with detailed steps for reflecting changes on control plane nodes via kubeadm-config changes
/sig cluster-lifecycle

Page related to issue: https://kubernetes.io/docs/tasks/administer-cluster/kubeadm/kubeadm-reconfigure/#reflecting-clusterconfiguration-changes-on-control-plane-nodes

The process to reflect changes on the control plane node after editing the kubeadm-config map isn't clear and should be stated in clearly defined steps

what has to be changed in that section to make the steps clearer?

/sig cluster-lifecycle
/triage needs-information

@neolit123 the docs explained changes to the ClusterConfiguration under the apiServer, controllerManager, scheduler or etcd keys must be reflected in the associated files in the manifests directory on a control plane node. it doesn't really tell us how you can actually make those changes in the manifests directory which on my side is currently empty i guess.
there's also kubeadm init phase certs <component-name> --config <config-file>  and kubeadm init phase control-plane <component-name> --config <config-file>. is the generation of certificate mandatory when you make those changes? does reflecting the changes means writing a new manifest (say after making changes to the ClusterConfiguration)? what is the <config-file>? is the <config-file> a copy of the updated ClusterConfiguration?
I think it may make sense for experience admins but complex for newbies.

@neolit123 the docs explained changes to the ClusterConfiguration under the apiServer, controllerManager, scheduler or etcd keys must be reflected in the associated files in the manifests directory on a control plane node. it doesn't really tell us how you can actually make those changes in the manifests directory which on my side is currently empty i guess.

if the directory is empty you should not be following this guide, because this node is not being managed by kubeadm.
that is the problem....
why is the directory empty, i cannot answer, but "kubeadm init" or "kubeadm join" can create control plane nodes.
to reflect changes, there is nothing to document, IMO - just open a text editor and edit a manifest Pod yaml file.
we assume that k8s users know what a Pod yaml file is.

there's also kubeadm init phase certs <component-name> --config <config-file> and kubeadm init phase control-plane <component-name> --config <config-file>.

yes, there are such commands, but they should not be run prior to editing /etc/kubernetes/manifests files.
the page does not talk about them.

is the generation of certificate mandatory when you make those changes?

no, updating /etc/kubernetes/manifests files does not require updating certificates or running manifest regeneration commands.
does reflecting the changes means writing a new manifest (say after making changes to the ClusterConfiguration)? what is the <config-file>? is the <config-file> a copy of the updated ClusterConfiguration?

I think it may make sense for experience admins but complex for newbies.

what is a newbie in k8s terms, exactly?
we assume users can follow the denoted steps which apply to everyone on a similar level - run kubectl and kubeadm commands, edit local files, restart containers.

I feel it should have been explained better as one can simply mess up the cluster configuration if not done correctly.

still not clear to me what should be explained better.
this seems more like a request for support, and less like a docs change request.
/kind support

no, updating /etc/kubernetes/manifests files does not require updating certificates or running manifest regeneration commands. but if you want new cert SANS, you obviously must regenerate certificates.

is this the part that is not clear to you?
if yes, something that can be added is a note:

Updating cert SANs requires you to regenerate certificates after you have updated
the local manifests files for control plane components.
Once certificates are regenerated, you must restart the control plane components again.
This can be done by restarting the kubelet service with systemctl restart kubelet, or by
restarting individual containers.

@neolit123 Okay so i get maybe its because the manifests directory is empty. but then again the cluster was bootstrapped using kubeadm so i don't understand why the node won't be managed by kubeadm...but in summary, are you saying all it takes to Reflecting ClusterConfiguration changes on control plane nodes is simply just updating the associated files in the manifest directory and that's all?

I think it may make sense for experience admins but complex for newbies.

@anthonyoride i remember you posted some questions on slack related to load balancers and changing the IP of apiserver. i replied to that changing the same IPs is really not a simple task. in such cases it's much simpler to backup the cluster, create a new cluster, and restore from backup. it will introduce a few minutes of downtime, but it might save you weeks of poking at multiple problems.

@neolit123 Okay so i get maybe its because the manifests directory is empty. but then again the cluster was bootstrapped using kubeadm so i don't understand why the node won't be managed by kubeadm...

i think this node is no longer usable as is. if it was a control plane node managed by kubeadm, it no longer works as there are no control plane components on it. you must call kubeadm reset on it and kubeadm join it again...

but in summary, are you saying all it takes to Reflecting ClusterConfiguration changes on control plane nodes is simply just updating the associated files in the manifest directory and that's all?

yes, but if you have updated cert SANs you must also update certificates on each control plane node.
i do agree this is not a very noob-friendly task, that's why i suggested a new note here:
https://github.com/kubernetes/website/issues/43715#issuecomment-1784730627

@neolit123 i have actually recreated the cluster a couple of times with kubeadm reset and kubeadm join though. /etc/kubernetes/manifests directory exists on all nodes but they are all empty

actually, reading https://kubernetes.io/docs/tasks/administer-cluster/kubeadm/kubeadm-reconfigure/#reflecting-clusterconfiguration-changes-on-control-plane-nodes again, the steps seem accurate. once the config map is updated the user can pass a local config file that has the same changes. they can generate new certificates and new manifest files, as non-optional steps.

@neolit123 i have actually recreated the cluster a couple of times with kubeadm reset and kubeadm join though. /etc/kubernetes/manifests directory exists on all nodes but they are all empty

we have a variety of tests that make sure /etc/kubernetes/manifests  works...
if you are passing a --rootfs flag it can modify the k8s root dir.
closing as this is purely a support request without concrete actionable changes.
please, don't log github issues unless you have concrete proposals for something to be changed - "not enough clarity" is not a good proposal for a change.
slack, discuss.k8s,io, reddit, etc are the places to get help.
/close


Data Lake Zones folder

It will be good to include best practices and suggestions for implementing PII data and 'Right to forgotten Request' as they play a critical role.
Some of the specific comments

In the section RAW Data, it is assumed that the data can land in different folder structure for sensitive and  non-sensitive data. This might not be true for a number of scenarios and it will be likely that certain columns will be sensitive in certain datasets.
Not sure if the provided folder structure is a recommendation or just an example. There could be alternate folder structure as well and it will be good to provide pros/cons for each of those. For example, lookup tables for pseudonymization
Data Lifecycle management section can include considerations for GDPR

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

ID: 174bef87-bea0-a095-3712-2c9dcf9e9388
Version Independent ID: 9deb539b-93a8-a3d0-25c3-ebf65e5a4441
Content: Azure Enterprise Scale Analytics and AI Teams Data Lake Services - Cloud Adoption Framework
Content Source: docs/scenarios/data-management/best-practices/data-lake-services.md
Service: cloud-adoption-framework
Sub-service: ready
GitHub Login: @mboswell
Microsoft Alias: mboswell

Thanks for reporting - this issue is under review. This is a Microsoft Internal DevOps Bug ID # 461081

This is up to the individual case but has been enhanced in a future version release.
#please-close


Docs: add documentation for metatags in route config

Which scope/s are relevant/related to the feature request?
create-analog
Information
Need to add docs for defining metatags in the RouteMeta
Describe any alternatives/workarounds you're currently using
No response
I would be willing to submit a PR to fix this issue

[ ] Yes
[ ] No

Hey @TomWebwalker, definitely!
The PR is here https://github.com/analogjs/analog/pull/230
The documentation page is here: https://github.com/analogjs/analog/blob/main/apps/docs-app/docs/features/routing/overview.md
If you need more info, just let me know


raise file limit

fix: raise number of downloaded files to hotfix https://github.com/keboola/internal/issues/137

there is a special button for that


in month view，Is it possible to show all schedule without show more button？

if i set calendar height for a height large enough：

but The height of a week without tasks will also increase

it is possible to achieved one day gride heigth base on  schedules count？

@yu123yi
Unfortunately, It's not possible.


add a 'Run Flaky Tests' step that does not fail the build

This adds a step to all the builds, immediately following the main build step, which runs all the tests marked "Flaky". If this step fails, it does not fail the build. However, the TRX files and logs should be collected and published, so the Tests view should show the failing flaky tests.
Once this is working, I can fairly easily port it to EF which is using Arcade. AspNetCore will need something a little different.
Closes https://github.com/aspnet/Extensions/issues/1224

Let's see how/if this works :)

Looks like it's somewhat working. The main AzP summary page does what I'd hoped:

The tests page shows all tests passing, I need to check the scripts that collect test results.

Ahh, they didn't run at all:
The application to execute does not exist: '/home/vsts/.nuget/packages/xunit.runner.console/2.4.1/tools/netcoreapp2.0/xunit.console.dll'
=== COMMAND LINE ===
"/home/vsts/work/1/s/.dotnet/dotnet" exec --depsfile "/home/vsts/work/1/s/artifacts/bin/Microsoft.Extensions.Configuration.Tests/Debug/netcoreapp3.0/Microsoft.Extensions.Configuration.Tests.deps.json" --runtimeconfig "/home/vsts/work/1/s/artifacts/bin/Microsoft.Extensions.Configuration.Tests/Debug/netcoreapp3.0/Microsoft.Extensions.Configuration.Tests.runtimeconfig.json"  "/home/vsts/.nuget/packages/xunit.runner.console/2.4.1/tools/netcoreapp2.0/xunit.console.dll" "/home/vsts/work/1/s/artifacts/bin/Microsoft.Extensions.Configuration.Tests/Debug/netcoreapp3.0/Microsoft.Extensions.Configuration.Tests.dll" -noautoreporters -xml "/home/vsts/work/1/s/artifacts/TestResults/Debug/Flaky/Microsoft.Extensions.Configuration.Tests_netcoreapp3.0_x64.xml" -html "/home/vsts/work/1/s/artifacts/TestResults/Debug/Flaky/Microsoft.Extensions.Configuration.Tests_netcoreapp3.0_x64.html" -trait "Flaky:All=true" -trait "Flaky:AzP:All=true" -trait "Flaky:AzP:OS:Linux=true" > "/home/vsts/work/1/s/artifacts/log/Debug/Flaky/Microsoft.Extensions.Configuration.Tests_netcoreapp3.0_x64.log" 2>&1

GitHub Checks:

AzP Tests Page:

Unix builds are failing due to a permissions error (grumble...). I'll fix it
Looks good to me as long as Unix follows along after the permissions are fixed.
I'm going to leave the FlakyAttributeTests in place for now. They do mean that Extensions will always have failing flaky tests, because they are in fact tests that test the FlakyAttribute.

RE Tests that test FlakyAttributeTests: RAAS will have to start having a list of tests which it won't fail for (because they're intended to fail.).

Ah, good point. Ideally it should just be ignoring all tests from the flaky test pass, but I don't think you get that much detail when querying the APIs you're querying.

I don't follow why we would want RaaS to ignore tests from the Flaky pass. RaaS should triage them like normal and update the affected issues. Isn't that the whole point of the Flaky attribute, to still get stats about how often these fail, without blocking?

Assuming we think it will work well, that should be fine. My concern is that a flaky test will already have an issue tracking it (the attribute requires that an issue be filled in). Will RaaS be able to find and update that issue? Or will we end up with two issues?

https://github.com/aspnet/AspNetCore-Internal/issues/1867

Merging this now.
@Eilon - Be aware that RaaS may file issues in AspNetCore-Internal for the FlakyAttributeTest tests because it doesn't know they are expected to fail, and RaaS looks at all failing tests, which includes these. Of course, RaaS only runs on failing builds in master right now and those are much rarer in Extensions so they may never come up, but I wanted to bring it up.
@ryanbrandenburg is planning to fix that but we don't want to block this PR on that change. We can just disregard those issues when they appear.


Fallback to macos-13 per actions/setup-python/issues/875#issuecomment…

…-2143364001

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.7%) to 83.296%

Totals

Change from base Build<PHONE_NUMBER>:
0.7%

Covered Lines:
5406

Relevant Lines:
6396

💛  - Coveralls


build fails with unknown CMake option

Hi,
Installing the package from pypi does not work, pip fails with some obscure message.
Using a local clone, pip install . failed with -j as an unknown option to cmake, which needs a double dash before that. Thus, the following patch to setup.py fixes at least the local builds, it may also fix the pypi install:
diff --git a/setup.py b/setup.py
index 8a156d6..524ea5d 100644
--- a/setup.py
+++ b/setup.py
@@ -58,7 +58,7 @@ def cmake_setup():
         subprocess.check_call(['cmake', str(SRCDIR)],
                               cwd=BINDIR)

-    subprocess.check_call(['cmake', '--build', str(BINDIR), '-j'])
+    subprocess.check_call(['cmake', '--build', str(BINDIR), '--', '-j'])

 def meson_setup():

Cheers.

I can confirm that this was an issue for me and the solution st-bender posted fixed it.

This has been reworked in the iri2016/build.py, that autobuilds on first run rather than setup.py. This is to avoid putting executable code in setup.py and to make the package setup-platform agnostic.


Build 1.1.1 is somehow not updated on npmjs.com

I'm sorry sir but the version 1.1.1 build on npmjs has the new package.json file, but it somehow is missing all the new functions like unregisterAll etc...
Steps to reproduce:
npm i<EMAIL_ADDRESS>Trying to use it:

You should notice the files in the node_modules/dist folder appear to be outdated possibly from version 1.0.15, and is missing unregister() and unregisterAll() functions
index.d.ts

index.js

Maybe with this latest build you forgot to run npm run build before publishing 🤔?

note: I've created a PR for auto-publishing, to remove some of the headaches of publishing 👍

@QarthO thank you for 1. fixing this, and 2. making it easier to maintain in the future! 👍

@QarthO I'm very sorry but the latest version (v1.1.3) is still broken, I'll comment over on that PR for more detail

Resolved. Thanks for letting me know. Sorry about all the hassle last couple of months. Will be more on top of it now

All good man, thank you for making qHotkeys ❤  (something I feel like electron should include already, but I feel they took a very foolish stance on the matter)!


getting location updates even when the app is killed not working

Great job, but i need an update of location even app was killed by user. After killing the app notification was gone too, by the way i use real device on debuging and when i use emulator it is working properly.

Hi @Tapuyo
Thank you for opening an issue.
Usually, the problem with a real device is related to the battery monitor.
Please exclude the app and its service from any battery monitor or power-saving app.
Let me if it's working or not :)

Closing the issue due to lack of information. Feel free to open it anytime;

Closing the issue due to lack of information. Feel free to open it anytime;


Error processing your request: sourceDirectoryClient.listPaths is not a function

@azure/storage-file-datalake:

12.17.0:

Windows:

[ ] nodejs

v20.14.0:

[ ] Edge

latest:

[ ] typescript

version:

Is the bug related to documentation in

[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
When I try to run a script via azure function that decompresses tar.gz files from 1 container to another, I run into error that listPaths is not a function, but when I checked the latest documentation it is the method I can use in my script.
To Reproduce
Steps to reproduce the behavior:
Here is the full script I use
const { DataLakeServiceClient, StorageSharedKeyCredential } = require('@azure/storage-file-datalake');
const zlib = require('zlib');
const tar = require('tar-fs');
module.exports = async function (context, req) {
const accountName = process.env.ACCOUNT_NAME;
const accountKey = process.env.ACCOUNT_KEY;
const serviceClient = new DataLakeServiceClient(
    `https://${accountName}.dfs.core.windows.net`,
    new StorageSharedKeyCredential(accountName, accountKey)
);

const sourceFileSystemName = "landing";
const sourceFolderPath = "foldername";
const destinationFileSystemName = "staging";
const destinationFolderPath = "foldername";

try {
    const sourceFileSystemClient = serviceClient.getFileSystemClient(sourceFileSystemName);
    const sourceDirectoryClient = sourceFileSystemClient.getDirectoryClient(sourceFolderPath);

    console.log(typeof sourceDirectoryClient.listPaths); // Check if the function exists

    let fileList = sourceDirectoryClient.listPaths({ recursive: false });

    for await (const path of fileList) {
        if (path.name.endsWith('.tar.gz')) {
            const fileClient = sourceDirectoryClient.getFileClient(path.name);
            const downloadResponse = await fileClient.read();

            const destFileSystemClient = serviceClient.getFileSystemClient(destinationFileSystemName);
            const destDirectoryClient = destFileSystemClient.getDirectoryClient(destinationFolderPath);
            await destDirectoryClient.createIfNotExists();

            downloadResponse.readableStreamBody
                .pipe(zlib.createGunzip())
                .pipe(tar.extract(destDirectoryClient.url))
                .on('finish', () => context.log(`Successfully extracted: ${path.name}`))
                .on('error', error => context.log(`Error extracting file: ${error.message}`));
        }
    }

    context.res = {
        status: 200,
        body: "File extraction process completed successfully."
    };
} catch (error) {
    context.log(`Error: ${error.message}`);
    context.res = {
        status: 500,
        body: `Error processing your request: ${error.message}`
    };
}

};
Expected behavior
A clear and concise description of what you expected to happen.
After script is triggered in adf via foreach activity it should decompress all files in landing zone straight to staging zone
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Hi @krlkbv ,
Seems you are trying to listPaths against a DataLakeDirectoryClient instance. listPaths is under DataLakeFileSystemClient. You can list a directory by indicate the directory path in ListPathsOptions like:
fileSystemClient.listPaths({ path: dirName })

Thank you, adjusted a code and it worked


Dapp frontend errors

If the user does not update the client contract address, the user can press submit and receive an error:
Something went wrong. Error missing revert data (action="estimateGas", data=null, reason=null, transaction={ "data": "0xdd27fe90000000000000000000000000000000000000000000000000000000000000002000000000000000000000000000000000000000000000000000000000000001600000000000000000000000000000000000000000000000000000000000008000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000001c0000000000000000000000000000000000000000000000000000000000007ef40000000000000000000000000000000000000000000000000000000000017bb000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000001000000000000000000000000000000000000000000000000000000000000022000000000000000000000000000000000000000000000000000000000000000270181e203922020a7e9e959fc0aaea427901d3000682bd90b7d5f4cb2ef2cf447e3b0a615fb72250000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000406261676136656134736561716b7032706a6c683661766c766565366962326d61616e617635736333356c35676c66337a6d36726436686d666763783578656a690000000000000000000000000000000000000000000000000000000000000080000000000000000000000000000000000000000000000000000000000000480d00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000006f68747470733a2f2f646174612d6465706f742e6c69676874686f7573652e73746f726167652f6170692f646f776e6c6f61642f646f776e6c6f61645f6361723f66696c6549643d38363266623131352d643234612d346666312d613163382d6561646262626664313963662e6361720000000000000000000000000000000000", "from": "0x2A089369A955d97c44370AD1bafcB4FE47fc74fe", "to": "0xf4E0C74D76Bf324293bB3B3DA184d164d06F7664" }, invocation=null, revert=null, code=CALL_EXCEPTION, version=6.1.0)
See related Slack thread here: https://filecoinproject.slack.com/archives/C04QGFF5EDR/p1678766374041759

Me getting same after cloning

Glad that the suggested solution worked for you. Feel free to close the
issue :)
On Wed, Mar 15, 2023 at 8:00 AM Shashank Trivedi @.***>
wrote:

Just change the contract address at line 11 in
frontend/src/components/input.js with the contract address of your own
deployed contract. It worked for me.
—
Reply to this email directly, view it on GitHub
https://github.com/filecoin-project/fvm-starter-kit-deal-making/issues/2#issuecomment-1469876972,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASH4FPWHJ7JC2W3B5RQLLS3W4GVPHANCNFSM6AAAAAAV25SW6I
.
You are receiving this because you were assigned.Message ID:
@.***
com>

The original scope of the issue assumes that you do not change the frontend/src/components/input.js, since the README and code comments make clear this is what is expected from the user. However, it is useful to analyze the unhappy path. Why is this necessary? What actually goes wrong?
Perhaps some more granular logging in the frontend and client contract can help resolve this.


feat: support for Unpack[TypedDict] in task kwargs

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

16 of 16 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 97.082%

Totals

Change from base Build<PHONE_NUMBER>1:
0.02%

Covered Lines:
1098

Relevant Lines:
1131

💛  - Coveralls


2.6.x delete modifier

What is the purpose of the change
Remove extra modifiers
Brief changelog
XXXXX
Verifying this change
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[ ] Run mvn clean install -DskipTests & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Is this submission specification normal?

Codecov Report

Merging #2876 into 2.6.x will decrease coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##              2.6.x    #2876      +/-   ##
============================================
- Coverage     47.42%   47.37%   -0.05%
+ Complexity     4294     4291       -3
============================================
  Files           552      552
  Lines         24381    24381
  Branches       4314     4314
============================================
- Hits          11563    11551      -12
- Misses        11035    11046      +11
- Partials       1783     1784       +1

Impacted Files
Coverage Δ
Complexity Δ

...aba/dubbo/remoting/transport/mina/MinaChannel.java
42.25% <0%> (-11.27%)
15% <0%> (-1%)

...ba/dubbo/remoting/transport/netty/NettyClient.java
74.13% <0%> (-8.63%)
12% <0%> (-1%)

...om/alibaba/dubbo/rpc/filter/ActiveLimitFilter.java
83.33% <0%> (-5.56%)
6% <0%> (-1%)

...rpc/protocol/dubbo/telnet/InvokeTelnetHandler.java
54.21% <0%> (-3.62%)
13% <0%> (-2%)

...bo/remoting/transport/netty/NettyCodecAdapter.java
53.12% <0%> (-1.57%)
3% <0%> (ø)

...a/dubbo/remoting/transport/netty/NettyChannel.java
66.25% <0%> (+8.75%)
21% <0%> (+2%)
: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 483cd7c...0f7b0b5. Read the comment docs.

Codecov Report

Merging #2876 into 2.6.x will decrease coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##              2.6.x    #2876      +/-   ##
============================================
- Coverage     47.42%   47.37%   -0.05%
+ Complexity     4294     4291       -3
============================================
  Files           552      552
  Lines         24381    24381
  Branches       4314     4314
============================================
- Hits          11563    11551      -12
- Misses        11035    11046      +11
- Partials       1783     1784       +1

Impacted Files
Coverage Δ
Complexity Δ

...aba/dubbo/remoting/transport/mina/MinaChannel.java
42.25% <0%> (-11.27%)
15% <0%> (-1%)

...ba/dubbo/remoting/transport/netty/NettyClient.java
74.13% <0%> (-8.63%)
12% <0%> (-1%)

...om/alibaba/dubbo/rpc/filter/ActiveLimitFilter.java
83.33% <0%> (-5.56%)
6% <0%> (-1%)

...rpc/protocol/dubbo/telnet/InvokeTelnetHandler.java
54.21% <0%> (-3.62%)
13% <0%> (-2%)

...bo/remoting/transport/netty/NettyCodecAdapter.java
53.12% <0%> (-1.57%)
3% <0%> (ø)

...a/dubbo/remoting/transport/netty/NettyChannel.java
66.25% <0%> (+8.75%)
21% <0%> (+2%)
: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 483cd7c...0f7b0b5. Read the comment docs.


use this.get pattern to avoid forEach = _ember["default"].EnumerableUtils.forEach.

use this.get, instead of get(this...), so don't need to have lodash involved.

I don't get the goal of this PR. forEach is a part of ES5, and is supported by almost all modern browsers. The deprecated EnumerableUtils is not being used here.

I modified the title of this PR. It felt too passive aggressive.

Yes, forEach is fine in ES5. But the major issue is the pattern in current code when works with emberjs 2.x, it will have below line
var forEach = _ember["default"].EnumerableUtils.forEach be generated. But in 2.x it doesn't have forEach anymore, which cause exceptions.
BTW: why use get(this, ..) not this.get(...)?

@bcardarella thanks for fixing the title. Sorry about that. :+1:

More clear, the EnumerableUtils is removed at 2.x
http://emberjs.com/blog/2015/05/24/another-two-oh-status-update.html
The ArrayPolyfills and EnumerableUtils libraries are being pulled out into a library; most apps should prefer lodash or other utility libs.

@systembugtj - This library is not using EnumerableUtils or ArrayPolyfills. I believe I linked to the PR that fixed this in the issue you created.

Thanks. Looks like this is caused by ember-lazy-image, the default package downloaded from npm still refer to 1.2.5 of this lib.

@rwjblue thanks for the clarification. The issue is not from this lib. But ember-lazy-image.


Use Base.min / Base.max in MPI reductions

We can use this workaround to resolve #1922.

Thanks! Shall we also switch to macos-latest in
https://github.com/trixi-framework/Trixi.jl/blob/2ac203e7c52182e8cbd7841d58ecb1e285c96322/.github/workflows/ci.yml#L93
?

Fine with me! But I do not know why it had been set like this.

Thanks! Shall we also switch to macos-latest in
https://github.com/trixi-framework/Trixi.jl/blob/2ac203e7c52182e8cbd7841d58ecb1e285c96322/.github/workflows/ci.yml#L93
?

Fine with me! However I am not quiet clear as to why this fixed things in the past.
Also, should we not test ARM here, besides / instead of x86?

macos-latest is macos-14, which is only available with ARM - we should also delete the confusing arch specification when updating t

There are still some user-defined MPI reductions in the integration methods.

Ideally, we should also fix those.
If the current implementation already enables you to do something that has not been possible before, we can merge the fixes (without the switch to macos-latest) and fix the remaining issues later in another PR.

What do you prefer?

Indeed! This time not the operator is the problem, but the operands. E.g. where the current CI fails, we are dealing with buf::Base.RefValue{StaticArraysCore.SVector{4, Float64}}. I am not sure if any vectorial data structure would fail, but I suppose so, cf. https://github.com/JuliaParallel/MPI.jl/blob/780aaa0fdb768713a329659338a9c9cde23c41a8/src/operators.jl#L59C1-L59C110
For my current work I only need the fixes in my personal branch, where I tested this initially. So I am in favor of fixing all occurences.
I do not have a great idea though. Just reducing each entry in the vector individually would of course be an option.

Did you test the example with macos?

No, I do not have a mac. But I could try with the GH system.

That would be great 👍

It does not work. Another individual operator does not help. Instead one would need to directly generate an (MPI.jl) Op object.

This is the place where we need the vector reduction. Currently, local_integral can be a Float64 in some cases (when we compute the total entropy) or an SVector (when we compute the total mass of all conserved quantities). What I'm suggesting is to reduce collect(local_integral) instead of Ref(local_integral). That should work, shouldn't it?

Only when @vchuravy mentioned the OpWrapper I realized that it already iterates through something like a vector.
_len will be 1 in case of our SVectors, but carry the right number when using Vectors (where does this actually come from?). So, using a Vectoror reinterpreting theSVectorasPtr[Float64}` seem to make the reduction work, without a custom operator (currently tree 2d only).
Of course this is still ugly (branching for isa Real, converting the return value, allocations, ...), but this could be a way.


Enable GPU Metrics

Is your feature request related to a problem? Please describe.
we should scape gpu device metrics，used to show the resources usage for pod
Describe the solution you'd like
NVIDIA offers a solution called DCGM-Exporter，we can integrate it  as a plugin into kubegems which like node-exporter
We expect to be able to provide each container which scheduling a gpu has its own independent resource metrics.
https://docs.nvidia.com/datacenter/cloud-native/gpu-telemetry/dcgm-exporter.html#gpu-telemetry

In which version is this plugin planned to be released? v1.21.1 or v1.22.0 @LinkMaq

This feature may be more suitable release in v1.22.0

Pod GPU utilization (in %).
- record: gems_container_gpu_usage_percent
   expr: |
     DCGM_FI_DEV_TOTAL_ENERGY_CONSUMPTION

PodGPU Memory (in MB).
- record: gems_container_gpu_memory_usage_ millionbytes
   expr: |
     sum(DCGM_FI_DEV_FB_USED * on(namespace,pod) group_left(node,host_ip,pod_ip) gems_pod_status)
     by (pod,namespace,device,node,host_ip,pod_ip,modelName)

PodGPU temperature (in C).
- record: gems_container_gpu_usage_temp
   expr: |
     sum(DCGM_FI_DEV_GPU_TEMP * on(namespace,pod) group_left(node,host_ip,pod_ip) gems_pod_status)
     by (pod,namespace,device,node,host_ip,pod_ip,modelName)

PodGPU Power (in C).
- record: gems_container_gpu_usage_power
   expr: |
     sum(DCGM_FI_DEV_POWER_USAGE * on(namespace,pod) group_left(node,host_ip,pod_ip) gems_pod_status)
     by (pod,namespace,device,node,host_ip,pod_ip,modelName)

by (pod,namespace,device,node,host_ip,pod_ip,modelName)

Lost container label.

gems_container_gpu_usage_percent: GPU 使用率 [percent-1-100]
gems_container_gpu_memory_usage_mb: GPU 显存使用量 [byte-MB]
gems_container_gpu_temp: GPU温度 [°C]
gems_container_gpu_power_usage_watt: GPU功率 [W(瓦特)]

@chenshunliang


Back door adjustments

First version of exaustive search minimal admissable sets that satisfy the back door criterion.
It is based on a d-separation function from pgmpy package (https://github.com/pgmpy/pgmpy). It implements an algorithm found in "Probabilistic Graphical Models: Principles and Techniques" - Koller and Friedman. Is it possible to integrate that piece of code giving the proper credit to the coders? ( That packagee is under MIT License but I do not know much about licenses)
I have also found a polinomial time version of the minimal admissable sets algorithm described in https://arxiv.org/abs/1202.3764 by Johannes Textor and implementedin his cool project DAGitty (http://www.dagitty.net/dags.html) which is a javascript browser-based causal inference analyzer.
At the last minute I realized that you use different naming conventions for functions while I use camelCase  naming convention. I can change that later if the rest of the code is ok.

Forgot to mention, it only works with DAGs at this moment

Hello Adam,
I have finally found some time to improve the PR code. Mainly, I have

Changed the function names so that it has the same naming convention that you use.
Converted the function into a generator function as you suggested for MVP
Improved the test coverage which will be useful in order to check a polynomial time version of the same algorithm.
Temporarly added all the code that is used from pgmpy (all is stored in utils.py) so that it is easier to check how the code works.

Best,


Update copy for when RoSH info can't be retrieved

Context
When RoSH information can't be retrieved, we show guidance to the CPPs to manually copy the information from Oasys and include it in the placement consideration risk sections.
However, the guidance for HPTs is for them to manually check outside the service so they wouldn't know to check that location for the manually included information.
This change updates the copy in the RoSH section to prompt HPTs to look in the placement considerations section for this information.

Screenshots of UI changes
Before
(Taken from the Check Your Answers page, but the copy is the same as the full referral that HPTs see)

After

https://github.com/ministryofjustice/hmpps-temporary-accommodation-ui/assets/6377078/e574294a-a526-4046-80cf-f5b4166458c7
Release checklist
Release process documentation.
Pre-merge checklist

[ ] Are any changes required to dependent services for this change to work?
(eg. CAS API)
[ ] Have they been released to production already?

Post-merge checklist

[ ] Manually approve
release to test
[ ] Manually approve
release to preprod
[ ] Manually approve
release to prod

 Workaround for RoSH not pulling through - HPT


refactor(model/gateway)!: Optimize Event size

chore(model/gateway): assert Event and variant sizes
refactor(model/gateway)!: Box large Event variants
refactor(model/gateway)!: Unbox small Event variants
refactor(model/gateway)!: Remove box From<DispatchEvent>

Could you decide on and document a rule for when we should box events, should future ones be added?

Could you decide on and document a rule for when we should box events, should future ones be added?

In this PR I just boxed every variant larger than the already boxed ones and unboxed the ones smaller than Event. The optimal size of Event depends on the frequency of events, something where larger bots may see a different pattern compared to smaller ones.
I've not benchmarked before, nor can I create an accurate model of incoming Events, so I'll leave it to someone more experienced.

@7596ff I added the instructions I was talking about and probably didn't break CI.


Digital Clock

Beginner-friendly JavaScript project that builds a digital clock, coupled with attractive CSS.

Looks good going for a merge


pip install flute-kernel

Hi，when I use pip install flute-kernel， there are errors：‘pip install flute-kernel
ERROR: Could not find a version that satisfies the requirement flute-kernel (from versions: none)
ERROR: No matching distribution found for flute-kernel’，thanks a lot

Hi, thanks for trying it out!
This happens if you use a combination of OS, Python version, etc that we didn't build FLUTE with. Do you mind elaborating what your OS and Python version are?

thanks，problems sloved！

Great, glad it works!
As for your question, did you git clone the FLUTE repo? This might be caused by the system recognizing the base directory flute as the package. If that's the case, you can try running the script in a different directory, or run it inside the flute/ directory.

thanks.problems solved. but it seems that the limitations of torch is not right, the problems is
'WARNING 08-19 11:35:53 _core_ext.py:180] Failed to import from vllm._core_C with ImportError('/extra_data/cutlass/flute/flute/vllm/vllm/_core_C.abi3.so: undefined symbol: _ZNK3c105Error4whatEv')
Traceback (most recent call last):
File "/extra_data/miniconda3/envs/flute/lib/python3.10/runpy.py", line 187, in _run_module_as_main
mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
File "/extra_data/miniconda3/envs/flute/lib/python3.10/runpy.py", line 110, in _get_module_details
import(pkg_name)
File "/extra_data/miniconda3/envs/flute/lib/python3.10/site-packages/flute/init.py", line 5, in
from vllm.platforms import current_platform
File "/extra_data/cutlass/flute/flute/vllm/vllm/init.py", line 3, in
from vllm.engine.arg_utils import AsyncEngineArgs, EngineArgs
File "/extra_data/cutlass/flute/flute/vllm/vllm/engine/arg_utils.py", line 9, in
from vllm.config import (CacheConfig, DecodingConfig, DeviceConfig,
File "/extra_data/cutlass/flute/flute/vllm/vllm/config.py", line 12, in
from vllm.model_executor.layers.quantization import QUANTIZATION_METHODS
File "/extra_data/cutlass/flute/flute/vllm/vllm/model_executor/layers/quantization/init.py", line 3, in
from vllm.model_executor.layers.quantization.aqlm import AQLMConfig
File "/extra_data/cutlass/flute/flute/vllm/vllm/model_executor/layers/quantization/aqlm.py", line 11, in
from vllm import _custom_ops as ops
File "/extra_data/cutlass/flute/flute/vllm/vllm/_custom_ops.py", line 7, in
from vllm._core_ext import ScalarType
File "/extra_data/cutlass/flute/flute/vllm/vllm/_core_ext.py", line 182, in
ScalarType = torch.classes._core_C.ScalarType
File "/extra_data/miniconda3/envs/flute/lib/python3.10/site-packages/torch/_classes.py", line 12, in getattr
proxy = torch._C._get_custom_class_python_wrapper(self.name, attr)
RuntimeError: Tried to instantiate class 'core_C.ScalarType', but it does not exist!
Do you might elaborate what you meant by the following sentence? (I don't think I got what you were trying to say)

but it seems that the limitations of torch is not right

That's interesting, and very good to know. Thanks! I think FLUTE is compiled against torch==2.3.1, whereas vLLM might be using torch==2.4.0. I didn't expect the version to make any difference, but perhaps it did. In the next version of FLUTE (in a few days), we will bump the version to 2.4.0 as well. Good feedback!
I'm going to close this issue for now as I assume it's fixed. Feel free to re-open if you wish.

sorry,but I could not run succesfully, could u help slove the problems?

Ah I thought it got fixed.
The error message seems to come from this line from vllm.platforms import current_platform. This seems more like a vLLM issue? Perhaps try reinstalling vLLM?

I have reinstalled, but it still failed. which version do u have? the pip list is above.

I have 0.5.3.post1


Support for additional .exd variant

Added support for a variant of .exd files containing multiple values per cell, cells in those files will return values as an IDictionary<int, object>.
As of now the following files use this variant:

AnimaWeapon5SpiritTalk
Behavior
ClassJobResident
Credit
DeepDungeonMap5X
GatheringItemPoint
GCScripShopItem
HousingMapMarkerInfo
MapMarker
MobHuntOrder
QuestClassJobReward
Resident
TreasureSpot
VaseSlot
WeatherGroup
ZoneSharedGroup
ZoneTimeline

Wow, that is some great work!

It is great work, thank you!


sending rate

Dear Steven ，I need you help ，where can I Set the sending rate？in menuconfig

You can change the sampling rate by running idf.py menuconfig in the active_sta project (NOT the active_ap project!).
Closing old issues.


fix(Surface): update backgroundColor to use interactive tokens

Description

This is dependent on #392 to update to the new interactive tokens.
References

Testing

Confirm that Surface component looks just about the same (bite be a slightly different white color).
Automation

Checklist

[ ] all commented code has been removed
[ ] any new console issues have been resolved
[ ] code linter and formatter has been run
[ ] test coverage meets repo requirements
[ ] PR name matches the expected semantic-commit syntax

Test Execution Passed.

Test Execution Passed.

:tada: This PR is included in version @lightningjs/ui-components-v2.15.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix locking while modifying counter

Clang with thread safety does not compile because
num_nodes_since_last_loop_closure_
is modified without holding a mutex.
This fixes it.

@wally-the-cartographer merge

@wally-the-cartographer merge


Force redraw Cluster Icon and not the unclustered icons

Is there a way to force redrawing cluster icon and not the unclustered one?
on another way, I need to call buildLeafletClusterIcon without calling PrepareLeafletMarker.

Hei,
I don't think it's easy from the PruneCluster API, but you could maybe do it from a lower level point of view, using Leaflet's eachLayer method for example.


Add drug availability warning

Show warning when drug not available

Make drug typeahead behaviour similar to others

 ARV availability message during prescription


Find middle element of the linked list (two pointer approach)

Description
Create a linked list and find the middle element using fast and slow pointer approach
gssoc'22
Domain
Data Structure
Type of Contribution
Addition
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of LearnCPP.

/assign


Automation for arranging script folders in [a-e], [f-j]...... [u-z]

Script Title - folders_arranger
Brief - Currently the number of folders are increased beyond 200. So the script will create 5 folders and arrange in into them
Instructions

Create a new folder for your script and the file/folder name should be appropriate.
Create a README.md (using this template) in your folder for program Instructions
add requirements.txt if needed

Programming Language

[x] Python

Happy Coding

Hi, can I work on this? ^^ Thank you.

Hi, can I work on this? ^^ Thank you.

Sure Go ahead :)

I would like to work on this issue!

Hi @pawangeek , I have just started working on this issue in my forked repo https://github.com/rblcoder/Automation-scripts,
once I complete it, is it fine if I create a pull request?

Hi @pawangeek , I have just started working on this issue in my forked repo https://github.com/rblcoder/Automation-scripts, once I complete it, is it fine if I create a pull request?

Sure go ahead :)

Hi @pawangeek , I have created the PR, pull request!


Exposing unocss-shortcut ?

Sorry, one more demand :)
The more I explore and experiment, the more I realise that your work is really brilliant !
I want to know if you can expose the unocss-shortcut. We could import it and modify some keys as we want.
Maybe exporting this file as a unocss preset should be better for the future...

Thanks for your kind words. After using https://ui.shadcn.com/ and other similar projects,
now feel it might not be a good idea to publish the component style to NPM registry (via Unocss preset or just CSS files). Two reasons for that:

For users, it's difficult to customize the styles. For example, you would first need to learn a bunch of CSS class names in MUI or a bunch of CSS variable names in Remirror, and use the correct CSS selector priority to override them.
For maintainers, it's difficult to update. Every CSS change could be potentially a breaking change, so style updating must be careful.

I encourage users to copy and paste the component examples (as well as all the TailwindCSS/UnoCSS class names) to their projects, and customize/maintain these components themselves. You can toggle the "Code" switch to show the source code of components in the examples page.
I plan to make it easier to copy the styles by improving the following two things in the future:

Currently the origin class names (defined in unocss-shortcut.js) are shown when inspecting the website. I will fix that and only show TailwindCSS/UnoCSS class names.

I plan to add a CLI tool to install and/or update component source code to users' projects, like https://ui.shadcn.com/docs/cli

Ok. I understand your purpose, and really like your ideas. A CLI tool look like awesome!


Raise Error when Image Path is empty

Fixes https://github.com/mapillary/OpenSfM/issues/864

Hi @jonasdlindner ,
Thanks for submitting this PR.
It breaks many of the tests (see here), so please try to refine it so that all the tests are green 😄
Thanks again


[Bug] android app crashes

🐛 Bug
Android app crashed while printing out a response to my question.
To Reproduce
Steps to reproduce the behavior:

Install android app on Galaxy S22 Ultra
Ask a question
Answer interrupted by crash

MLCChat failed

Stack trace:
org.apache.tvm.Base$TVMError: InternalError: Check failed: (offset + needed_size <= this->buffer.size) is false: storage allocation failure, attempted to allocate 4202496 at offset 0 in region that is 4194304bytes
Stack trace:
  File "/Users/houbohan/tvm/src/runtime/relax_vm/memory_manager.cc", line 115

	at org.apache.tvm.Base.checkCall(Base.java:173)
	at org.apache.tvm.Function.invoke(Function.java:130)
	at ai.mlc.mlcllm.ChatModule.decode(ChatModule.java:71)
	at ai.mlc.mlcchat.AppViewModel$ChatState$requestGenerate$1$2.invoke(AppViewModel.kt:654)
	at ai.mlc.mlcchat.AppViewModel$ChatState$requestGenerate$1$2.invoke(AppViewModel.kt:653)
	at ai.mlc.mlcchat.AppViewModel$ChatState.callBackend(AppViewModel.kt:537)
	at ai.mlc.mlcchat.AppViewModel$ChatState.requestGenerate$lambda$4(AppViewModel.kt:653)
	at ai.mlc.mlcchat.AppViewModel$ChatState.$r8$lambda$lluIrcsPALEW5nCb2tohZYadhTY(Unknown Source:0)
	at ai.mlc.mlcchat.AppViewModel$ChatState$$ExternalSyntheticLambda2.run(Unknown Source:6)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:463)
	at java.util.concurrent.FutureTask.run(FutureTask.java:264)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1137)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:637)
	at java.lang.Thread.run(Thread.java:1012)

Error message:
InternalError: Check failed: (offset + needed_size <= this->buffer.size) is false: storage allocation failure, attempted to allocate 4202496 at offset 0 in region that is 4194304bytes
Stack trace:
  File "/Users/houbohan/tvm/src/runtime/relax_vm/memory_manager.cc", line 115

Expected behavior

Environment

Android
Device: Galaxy S22 Ultra

Additional context

Interesting.  This is the same error we get on Pixel 6 and 7 when trying to start the Llama2 model  (even though we have enough memory to run it).

InternalError: Check failed: (offset + needed_size <= this->buffer.size) is false: storage allocation failure, attempted to allocate 4202496 at offset 0 in region that is 4194304bytes
Stack trace:
File "/Users/houbohan/tvm/src/runtime/relax_vm/memory_manager.cc", line 115

@spectrometerHBH   (sorry the error message has your name on it  🙂  )   the new APK crashes even Qualcomm devices after a while.  Perhaps this is related to the  _0 vs _1  format switch 🤔

@Sing-Li Thanks for the input. I'll take a look.

Hi, we just updated the Android apk. https://github.com/mlc-ai/binary-mlc-llm-libs/blob/main/mlc-chat.apk
Hopefully, it fixed the OOM crash issue. It's caused by some bug introduced when we were developing multi-GPU support.


Find occurances of problematic language in this repository

Acceptance criteria: issues created to remove/update source codes and documentation to avoid problematic language

Done, nothing serious has been found.


Add publishing process

Pull Request Test Coverage Report for Build #9

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.202%

Totals

Change from base Build #8:
0.0%

Covered Lines:
1256

Relevant Lines:
1424

💛  - Coveralls


No DDL for view in Oracle on fresh <IP_ADDRESS> install

Unable to to see DDL for a view on Oracle server on fresh <IP_ADDRESS> x64 install on Windows 7.  Getting the error "-- Can't generate DDL: table editor not found for org.jkiss.dbeaver.ext.oracle.model.OracleView".

Thanks for report


Misleading return type definition for table.get_item()

Describe the bug
The current TypedDict definition for the DynamoDB table get_item() method GetItemOutputTableTypeDef contains "Item" as a required key, however, this is not true. According to the AWS documentation:

If there is no matching item, GetItem does not return any data and there will be no Item element in the response.

Currently, mypy reports any branch of code that questions the existence of the "Item" key as an unreachable code.
To Reproduce
Steps to reproduce the behavior:

Install boto3-stubs[dynamodb]
Run mypy on the following code sample

import boto3

db = boto3.resource("dynamodb")
table = db.Table("table-name")
response = table.get_item(Key={"foo": "bar"})

if "Item" in response:
    ...
else:
    # unreachable code according to mypy
    ...
...

Actual output
.../main.py:10: error: Statement is unreachable  [unreachable]

Expected output
No mypy reported errors.
Additional context
Windows 11, boto3-stubs installatalled via pipenv, boto3 version 1.34.19

Hello! Thank you for the report. I am currently working on optional keys in response StructureShapes. This looks related to #238

The issue has finally been fixed in mypy_boto3_builder 7.24.0 release. I have also released mypy-boto3-dynamodb 1.34.91 with the fix included. Please update and let me know if it works as expected.


Circuit library NLocal string circuit drawer issue

Summary
The circuit library's NLocal feature (n_local.py) has a function called __str__(self) that attempts to draw the NLocal in circuit format using the standard gates. This function fails when running aqua features that require logging, such as VQE. This PR fixes the issue.
Details and comments
Line 921 of qiskit/circuit/n_local/n_local.py has the following lines:
return transpile(self, basis_gates=basis_gates,
                         optimization_level=0).draw().single_string()

The above code fails in IQX (and more generally anywhere that the circuit drawer is set to default to mpl) because the draw() function does not return a string. The error reported is that a Figure object has no function single_string(), which makes sense.
I'm fixing it by explicitly stating draw(output='text').
I think reading the above code snippet clearly shows the issue, but if anyone's interested in reproducing the error, run this notebook in qiskit 0.19.2.

This was a duplicate of #4464

Can we backport this? Especially when using some chemistry examples and VQE this bug can come up quite quickly on IQX.


Introduce renovate

The configuration is relatively simple: automerge minor+patch updates and all GH action updates as soon as the updates are available. Exception is the awssdk, which has daily releases, so it's limited to "once per week".
For posterity, the config to let Renovate work against version-branches, restricted to patch-version bumps.

I've enabled Renovate on my Polaris fork - https://github.com/snazy/polaris/issues/6 shows what it would do.


updated wazuh agent to 4.2.2

Signed-off-by: Daniel Sutton<EMAIL_ADDRESS>

Thank you for pull requst.


SCM: clicking on staged image opens it as text file in editor

Steps to Reproduce:

add a new PNG to the project
stage it
open it

=> it opens as text file even though it is binary

We need API to provide contents for binary documents. Or possibly images, through base64.
Thoughts @jrieken ?

there is a open command with can do that

The image doesn't exist on disk. I have access to a stream of bytes. We need API to provide contents for binary documents.
The workaround I could do for now is to stream everything to a temp file and show that. Sad!

so, you want two things: 1) binary content provider and 2) openBinary which is complicated because we generally cannot open binaries. What would you expect to happen when a dll-file is staged?

I'd expect us to at least say this is a binary file which we can't open.

well, that's a start

Let's keep it in mind.

Fixed!


LocalAI intuitive module-level var names

Description
Made the module-level vars in LocalAI both more intuitive and matching OpenAI namings
Type of Change

[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

[ ] Added new unit/integration tests
[ ] Added new notebook (that tests end-to-end)
[x] I stared at the code and made sure it makes sense

Thanks!


Release v1.10.2

Is this a new chart?
Yes
What this PR does / why we need it:
Which issue(s) is this PR associated with:

#Issue_Number
https://github.com/dell/csm/issues/1280

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

[X] Chart Version bumped
[ ] Variables are documented in the chart README.md
[ ] Title of the PR starts with the chart name (e.g. [charts_dir/mychartname]) if applicable

branch conflict hence closing this


Map images to own grid

Define our own grid - something like CH1903
Make sure every image coming in is mapped onto this grid

This has been resolved


Update README.md

Personally Identifiable information will not be added a to a public example. please reframe from exposing potential security flaws in pull requests.


zstd compression in pvar file

Hi all -
It does not appear that snputils is able to handle a .pvar.zst file (which can be produced with the 'vzs' modifier in plink2). Is this something that we can work to add in to the pgen reader? It would be great. I am happy to help contribute when time permits.
Kyle

Thanks for the report! v0.2.9 includes the fix.


[feature request] Restore previous input

Could you please add an option to restore the last typed input when launched?

Can you achieve this with "Restore last input"?

It doesn't work with me.

I turned the option on.

Last input's not restoring.
Maybe it's a bug?

@hyperphysin Can you achieve this with "Restore last input"?

It doesn't work with me.

I turned the option on.

Last input's not restoring.
Maybe it's a bug?

@hyperphysin
Unfortunately, I couldn't reproduce it. Could you tell me the reproduction steps in the sandbox vault?

@hyperphysin Unfortunately, I couldn't reproduce it. Could you tell me the reproduction steps in the sandbox vault?

Weiredly, uninstalling and installing again the plugin solved the issue.
Sorry to waste your time and thank you for making this useful plugin :)

I’m glad to hear that!

@hyperphysin I also noticed "restore last input" not working for me. Turns out, that setting is effectively disabled by the "use selection words as input" setting. Disabling that one makes "restore last input" work for me.
@tadashi-aikawa I think it might make sense to maybe add to the setting description that these two setting cannot be enabled at the same time to avoid such confusion? 🙏

@hyperphysin I also noticed "restore last input" not working for me. Turns out, that setting is effectively disabled by the "use selection words as input" setting. Disabling that one makes "restore last input" work for me.
@tadashi-aikawa I think it might make sense to maybe add to the setting description that these two setting cannot be enabled at the same time to avoid such confusion? 🙏

aha! That was why. Thank you for letting me know that.

@chrisgrieser (@hyperphysin )
Thank you for the helpful information! That was completely unexpected, so I’ll try to revise it to make both options compatible👍

https://github.com/tadashi-aikawa/obsidian-another-quick-switcher/issues/259

@chrisgrieser @hyperphysin
Released in v12.0.1 👍


Flags to get compiling with GNU and clean up some copyrights

Add compiler flags for GNU.
Clean up compiler flags for Intel.
Minor code change to get things to compile with GNU
Clean up the copyright so UCAR does not appear.
Outgrads is not found so does not compile with debug flags, needed for CI @ JCSDA

Perhaps you didn't see what I wrote on Teams over the weekend, but this branch you are trying to merge to is already obsolete. I made important changes on another branch that I had pushed over to the GitHub, but had no PR on it. This was this is the branch I am planning to turn into develop. Although nothing you have here interferes w/ what I've done, I'd prefer not to merge to the branch you point to here.

@rtodling I was thinking we could have just merged into that branch and then into your branch. But we can equally do it the other way around so I merged your branch into mine and changed the base. You should see the proper base branch now.

BTW: I see there is a single line actual change in m_nc_GEOSens ... thanks


skip_dns_resolution_tests flag is undefined

skip_dns_resolution_tests is processed as a group variable by build role (src/roles/build/tasks/main.yml).
The call to populate group variables come after processing VSD variables which results in skip_dns_resolution_tests being undefined and ignored.
build/main.yml

I see what you mean. Thanks for the investigation. We missed this problem for sure. However, I do not believe the problem is one of ordering. When Ansible runs, it reads the contents of group_vars/all at the start of the run and does not update its in-memory database just because we write to group_vars/all. We could update group_vars/all first and it wouldn't make any difference. What's happening here is that the process-vars roles should be referencing the deployment variables, not the inventory variables - because the inventory variables are not yet in memory.

I think the fix is to change the following line in src/roles/common/tasks/check-dns.yml and src/roles/common/tasks/check-dns-data.yml from this:
when: not skip_dns_resolution_tests|default(false)
to this:
when: not common.skip_dns_resolution_tests|default(false)
I will code a fix and test.

My tests passed. You will see the fix, above, in 4.0.3 coming out this week.

Great, thanks!


Add @abhi-g as a member

@abhi-g is an admin for the org but not listed as a member.
This likely explains why his /lgtm's aren't working.

This change is

/assign @gabrielwen

/lgtm

Turns out this is unnecessary. admin's are obviously considered members. The reason the LGTM wasn't accepted was because it was done via the Notebook Review App. Adding it directly to the PR worked.


API client generation without requiring go-openapi modules

Can we modify the generation command so that it doesn't require go-openapi modules for our project? We have many different services and if a service needs to install multiple API clients, but those API clients rely on different versions of go-openapi packages (because maybe they clients were generated with different versions of go-swagger over time) this creates an issue with dependency management.

Maybe I could open an MR where I added some new constructors that can be used without go-openapi structs?

Any thoughts on this? Would an MR be welcome?

You can make a PR, but please don't modify the public interface besides net additions. So, without breakage :)

@casualjim here it is: https://github.com/go-swagger/go-swagger/pull/2979 let me know your thoughts and I can adjust as needed.

@casualjim made an update now I need the workflow re-approved: https://github.com/go-swagger/go-swagger/pull/2979

Fixed with #2979 merged


fix: chain endpoint still using the old one

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type

[x] Bugfix
[ ] Feature
[ ] Refactoring

What's new?

[x] PR closes #3842

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[ ] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
[ ] I've tested PR on mobile and everything seems works
[ ] I found edge cases
[ ] I've written some unit tests 🧪

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot

[ ] My fix has changed something on UI; a screenshot is best to understand changes for others.

how to test/reproduce the error:

visit snek page https://deploy-preview-4032--koda-nuxt.netlify.app/snek/gallery/659233203-3
open chrome dev tools
copy session storage value to code editor under Application > Session Storage > https://deploy-preview-4032--koda-nuxt.netlify.app
edit all string with wss://rococo-basilisk-rpc.hydration.dev to any error string wss://error.dev. put back edited JSON to Session Storage
reload the page
on the console log the API should be connected to the correct endpoint

I added a clear session also in the logout button. in case the error still occurs, try to re-login (logout button and then connect again)

how to test/reproduce the error:

this worked for me :) let's have @Jarsen136 to take a look at the code and 🚀

😍 Perfect, I’ve sent the payout
💵 $150 @ 42.94 USD/KSM ~ 3.493 $KSM
🧗 DY4SQF2iD456tH89aQtz5wv1EV3BbSW8wKKuMcwbmXaj1pM
🔗 0xee5f404314326647aae54aee4e47351e4a7f5ae00ed7c6681d317f845f89fff9
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


AD Auth doesn't work after upgrade from 2.X to 3.X

Greetings,
just installed MunkiReport 3.X on a new virtual machine with the same basic settings as my MunkiReport 2.X machine (CentOS 7). Sadly i cannot get AD Auth to work again, even with the same settings. Did something change?
Used config.php (i disabled SELinux and firewalld just to be sure...):
$conf['auth']['auth_AD']['account_suffix'] = NULL;
$conf['auth']['auth_AD']['base_dn'] = 'DC=ad,DC=domain,DC=com'; //set to NULL to auto-detect
$conf['auth']['auth_AD']['domain_controllers'] = array('ad.domain.com'); //can be an array of servers
$conf['auth']['auth_AD']['admin_username'] =<EMAIL_ADDRESS>; //if needed to perform the search
$conf['auth']['auth_AD']['admin_password'] = 'PASSWORD'; //if needed to perform the search
$conf['auth']['auth_AD']['mr_allowed_groups'] = array('Group1','Group2'); //case sensitive

We use<EMAIL_ADDRESS>to login. This returns the following error:
    An error occurred while contacting AD
    No LDAP query results for filter: [(&(objectclass=person)(objectcategory=person)(anr=user@domain.com))] in: [DC=ad,DC=domain,DC=com]

When i execute this ldap-request by hand, i also don't receive any results (seems like Windows Server 2012 AD doesn't include the mail or userPrincipleName Attribute in the anr). What request was used in MunkiReport 2.X?
When i try to use $conf['auth']['auth_AD']['account_suffix'] =<EMAIL_ADDRESS>i receive:
    An error occurred while contacting AD
    Invalid credentials

No matter if i login as user or<EMAIL_ADDRESS>

Refer to https://github.com/munkireport/munkireport-php/issues/993

Used the command in the mentioned issue, but it does still not work. adldap2 is istalled:
[servmin@HOSTNAME munki_report]$ sudo php composer.phar show -i
You are using the deprecated option "installed". Only installed packages are shown by default now. The --all option can be used to show all packages.
adldap2/adldap2           v8.1.5  A PHP LDAP Package for humans.
defuse/php-encryption     v2.1.0  Secure PHP Encryption Library
doctrine/annotations      v1.6.0  Docblock Annotations Parser
doctrine/cache            v1.7.1  Caching library offering an object-oriented API for many cache backends
doctrine/collections      v1.5.0  Collections Abstraction library
doctrine/common           v2.8.1  Common Library for Doctrine projects
doctrine/dbal             v2.6.3  Database Abstraction Layer
doctrine/inflector        v1.3.0  Common String Manipulations with regard to casing and singular/plural rules.
doctrine/lexer            v1.0.1  Base library for a lexer that can be used in Top-Down, Recursive Descent Parsers.
guzzlehttp/guzzle         6.3.0   Guzzle is a PHP HTTP client library
guzzlehttp/promises       v1.3.1  Guzzle promises library
guzzlehttp/psr7           1.4.2   PSR-7 message implementation that also provides common utility methods
hautelook/phpass          1.1.0   Portable PHP password hashing framework
illuminate/console        v5.4.36 The Illuminate Console package.
illuminate/container      v5.4.36 The Illuminate Container package.
illuminate/contracts      v5.4.36 The Illuminate Contracts package.
illuminate/database       v5.4.36 The Illuminate Database package.
illuminate/filesystem     v5.4.36 The Illuminate Filesystem package.
illuminate/support        v5.4.36 The Illuminate Support package.
league/flysystem          1.0.43  Filesystem abstraction: Many filesystems, one API.
nesbot/carbon             1.25.0  A simple API extension for DateTime.
onelogin/php-saml         v2.13.0 OneLogin PHP SAML Toolkit
paragonie/random_compat   v2.0.11 PHP 5.x polyfill for random_bytes() and random_int() from PHP 7
psr/http-message          1.0.1   Common interface for HTTP messages
psr/log                   1.0.2   Common interface for logging libraries
rodneyrehm/plist          v2.0.1  Library for reading and writing Apple's CFPropertyList (plist) files in XML as well as ...
symfony/console           v3.4.6  Symfony Console Component
symfony/debug             v4.0.6  Symfony Debug Component
symfony/finder            v3.4.6  Symfony Finder Component
symfony/polyfill-mbstring v1.7.0  Symfony polyfill for the Mbstring extension
symfony/translation       v4.0.6  Symfony Translation Component

Also i tried again to dump the autoload: sudo php composer.phar dump-autoload, and i still get the same error.
Like i said: I also tried to execute the ldap query by myself and also can't fetch a user, since Windows Server 2012 seems not to include the mail-attribute in the anr. But in MunkiReport 2.X it worked without a problem.

It looks like ADLDAP2 is complaining about your admin credentials

I have the my munkireport 2.X Instance still up, using the same credentials, using the same AD Server etc. and it works. So no credentials error.

I have the same issue, AD works in 2.x but not in 3.x. In 3.x I get only "An error occurred while contacting AD".
I also tried to change the old syntax array('') to [''], but no difference.

I'll stand up a dc and get to the bottom of this

Same problem here. 2.15 works, 3.1.1: "An error occurred while contacting Ldap Server".

Issue replicated here. taking a look

userPrincipalName isn't part of the ANR, so a NULL account suffix should probably switch out anr for userPrincipalName. Additionally, I created a new account with Windows Server 2016 and it did not receive a uid attribute, so that's probably not a reasonable filter method either with regards to AD.
In summary: no suffix = use UPN, suffix provided = anr (which includes sAMAccountName)

Fix merged into WIP branch. account_suffix = NULL means that you must specify full UPN eg<EMAIL_ADDRESS>Tested against WS2016 DC


unsupported ext tag detected

unsupported ext tag detected: EXT-QQHLS-PIC-WIDTH - #EXT-QQHLS-PIC-WIDTH:0 i do not know how to solve it

any solution?


Do not show asset when user cannot purchase corresponding contract, w…

…hen market close and dont allow start later

It throws warning about duplicate keys, i am working on it


Do not call setState if the component has already been unmounted

I have a component that navigates away from the page immediately after a successful async action. As a result I was seeing the following warning in the console:  "Warning: setState(...): Can only update a mounted or mounting component. This usually means you called setState() on an unmounted component. This is a no-op. Please check the code for the AsyncButton component.".
This pr changes it so setState is never called asynchronously if the component was unmounted before the promise resolves or rejects.

@bmac thanks. Published 0.0.6


[FLINK-11524] Solve race condition in EmbeddedLeaderService

What is the purpose of the change
Using the EmbeddedHaServices it could happen that the EmbeddedLeaderService goes into an
invalid state if two grant leadership operations are executed concurrently.
The problem was that the first operation when confirming the now outdated leader session id
would set the EmbeddedLeaderElectionService#isLeader field to false as part of the stale
confirmation message handling. Consequently, all subsequent calls to
LeaderElectionService#hasLeadership(newLeaderSessionId) would return false even though the
contender would still have the leadership.
The solution is to not set the EmbeddedLeaderElectionService#isLeader to false if the
EmbeddedLeaderService encounters a stale confirmation message.
Verifying this change

Added EmbeddedHaServicesTest#testConcurrentLeadershipOperations

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

@flinkbot approve all

Thanks for the review @zentol. Merging this PR.


Discord Login

Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Added Discord Login and fix some Github login code formatting. I don't understand why Discord doesn't give permission to access my profile name and picture.
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)
[x] Documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration:

I tested it on docker desktop and with a fresh install on ubuntu
Test Configuration:

Checklist:

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

Hi Danny, since from what I have experienced, it appears impossible to have a name and a profile image with Discord Application , we could use the email address as the name? (is it the same on chat.openai.com as well)

Hi Danny, since from what I have experienced, it appears impossible to have a name and a profile image with Discord Application , we could use the email address as the name? (is it the same on chat.openai.com as well)

To clarify, are you saying this is the default behavior on chat.openai.com?
I'm actually pretty experienced with discord development and I can tell you that they are very restrictive with their scoping. I will take a look at your code but this looks to be part of the solution, you may have looked at it already: https://stackoverflow.com/questions/68379109/javascript-discord-get-profile-picture

Looks good to me, just remove those comments and I can merge

there are now 2 minor merge conflicts from a PR i just approved but they should be easy to fix

Fixed! If you can, give me another hour because I'm testing one last thing with the login. Thanks.

Fixed! If you can, give me another hour because I'm testing one last thing with the login. Thanks.

sounds good, thanks!


Test and fix for mysql bookinfo

Fixes #1075
Allows to have both mongoldb and mysql coexisting in same test.
Note: had to rebuild the image, into gcr.io as I don’t have access to
the other. (Initially tried to just use an older image but we also
changed the protocol so that didn’t work…)
Added mysql as a dependency to package.json
Ps: as mixer tests mongodb maybe we can have this only test mysql
Add bookinfo mysql to regression tests. Fix the ratings sample app so it doesn't throw when talking to mysql

@ldemailly: Adding do-not-merge/release-note-label-needed because the release note process has not been followed.

One of the following labels is required "release-note", "release-note-action-required", or "release-note-none".
Please see: https://github.com/kubernetes/community/blob/master/contributors/devel/pull-requests.md#write-release-notes-if-needed.
Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Note: Not specific to this change.
The test for db's (mongo and mysql) which checks for number of glyphon-star is not going to find the errors in failure to access the db backend. There are following options to address this:

Either there should be a version of bookinfo output that can be distinguishable as text output or
the total number of glyphon-stars should be != 10 or
The names of the 'reviewer's' should be picked up from the db and not statically set by review's micro service.

it is publicly accessible but can/will be pushed to docker for consistency
I agree that it only test the backend if the rule works which isn’t great (thus the TODO I added last time around)

can someone put a / in front ot the lgtm :) tia!

/lgtm


create a new tab each time

the issue is it create a new tab each time , you know about the api so you can fix it easily . please help me to fix it
You: my name is rafi . so what is my name
Assistant: R afi ! Your name is ... R AF I !

You: yes , tell my name again
Assistant: I apologize , but I didn 't introduce myself earlier . Since we just started our conversation , I don 't have a name to share with you yet . Would you like to give me a name or nickname ?

also i need a lot of update using your code wait , giving the request . i hope you will love it
in the console version
Ollama Chatbot
Type 'exit' to end the conversation.
Make sure you already installed those models.
Models that you may have installed:
1. llama2-uncensored
2. llama3
3. phi3
4. wizardlm2
5. mistral
6. gemma
7. mixtral
8. llama2
9. Custom Model
Please select a model (enter the number), or press Enter for 'llama3':
Do you want to enable text-to-speech? (y/n), or press Enter to turn off: Invalid input. Text-to-speech disabled.
Do you want to see real-time output? (y/n), or press Enter for yes:

also there is a PysideGui version . its like PYQT but more open source nature.
https://gitlab.com/krafi/ollamachat where is my gitlab profile link, and a fork of the project i feel more comfortable with gitlab .

Hello,
Sorry for the late reply.
I like your approach.
The issue has been resolved, at first we were using streaming to receive
data from the AI server, now I have switched it to json objects.
Check out my latest code update so that you can update your code too.
read the documentation here,
https://github.com/ollama/ollama/blob/main/docs/api.md
Thank you
On Tue, 7 May 2024 at 10:47, kazi ar rafi @.***> wrote:

the issue is it create a new tab each time , you know about the api so you
can fix it easily . please help me to fix it
You: my name is rafi . so what is my name
Assistant: R afi ! Your name is ... R AF I !
You: yes , tell my name again
Assistant: I apologize , but I didn 't introduce myself earlier . Since we just started our conversation , I don 't have a name to share with you yet . Would you like to give me a name or nickname ?
also i need a lot of update using your code wait , giving the request . i
hope you will love it
in the console version
Ollama Chatbot
Type 'exit' to end the conversation.
Make sure you already installed those models.
Models that you may have installed:

llama2-uncensored
llama3
phi3
wizardlm2
mistral
gemma
mixtral
llama2
Custom Model
Please select a model (enter the number), or press Enter for 'llama3':
Do you want to enable text-to-speech? (y/n), or press Enter to turn off: Invalid input. Text-to-speech disabled.
Do you want to see real-time output? (y/n), or press Enter for yes:

also there is a PysideGui version . its like PYQT but more open source
nature.
—
Reply to this email directly, view it on GitHub
https://github.com/inno-waluza/OllamaChat/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUXTF3JSKBITQKB57CMDKKTZBEOUZAVCNFSM6AAAAABHLRRNYGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI4DIMBQGQZDAMA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


ERROR: Cannot uninstall 'ruamel-yaml'.

ERR Detail:
ERROR: Cannot uninstall 'ruamel-yaml'. It is a distutils installed project and thus we cannot accurately determine which files belong to it which would lead to only a partial uninstall.
Platform:Windows 10&Mac Os
Python version: 3.8.5

Try it when installing --ignore-installed ruamel.yaml

Try it when installing --ignore-installed ruamel.yaml

It works！


fix: unapplied theme on EAP

This seems to fix #108.
As the fix seems really strange, that would be cool is someone can try this before merging!

Can confirm that this fixes the issue on EAP! (tested on RustRover)


Is there Code?

Is there code for this repository?

Great work! Any expected time when the code will be released?

Added initial version :)!


Add support for indexedDB

The idea is to have support for defining how to store data (from requests to json files) straightly in the indexedDB, so developers can use and access it.

Support was added and documented.


Develop

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.25%. Comparing base (5f5143a) to head (4f192a3).
Report is 6 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #90   +/-   ##
=======================================
  Coverage   96.25%   96.25%
=======================================
  Files          17       17
  Lines         374      374
=======================================
  Hits          360      360
  Misses         14       14

Components
Coverage Δ

core
100.00% <ø> (ø)

api
90.00% <ø> (ø)

tests
100.00% <ø> (ø)


[Suggestions] Only release low CP pokemon if you already caught it

Why should we release a low CP Pokemon if it's the first time we caught it?

Currently just dumps anything lower level - you will normally always catch it again! :)

Easy to implement, snipper below works for me.
def transfer_unique_pokemon(api):
	api.get_inventory()
	response_dict = api.call()
	pokemons = []
	#print('Response dictionary: \n\r{}'.format(json.dumps(response_dict, indent=2)))
	if response_dict and 'responses' in response_dict:
		if 'GET_INVENTORY' in response_dict['responses']:
			if 'inventory_delta' in response_dict['responses']['GET_INVENTORY']:
				if 'inventory_items' in response_dict['responses']['GET_INVENTORY']['inventory_delta']:
					for item in response_dict['responses']['GET_INVENTORY']['inventory_delta']['inventory_items']:
						#print('item {}'.format(item))
						if 'inventory_item_data' in item:
							if 'pokemon' in item['inventory_item_data']:
								pokemon = item['inventory_item_data']['pokemon']
								if 'is_egg' in pokemon:
									continue
								pokemons = pokemons + [(pokemon['pokemon_id'], pokemon['cp'],pokemon['id'])]

	#Magic reverse sorts (poke_id, CP, uq_id)
	pokemons.sort(reverse=True)
	print pokemons
	pokemons_seen = []
	for pokemon in pokemons:
		if pokemon[0] in pokemons_seen:
			print('Already captured this pokemon, delete it:'+str(pokemon))
			api.release_pokemon(pokemon_id=pokemon[2])
			print('Response dictionary: \n\r{}'.format(json.dumps(response_dict, indent=2)))
			time.sleep(1.2)
		else:
			pokemons_seen = pokemons_seen + [pokemon[0]]

@rvanvenetie Can you do a pull request so everyone can profit easily?

@rvanvenetie would you like to make a pull-request for this? More than happy to implement if you do :)

Actually, I do not have the time to properly implement this atm. I've simply ignored the cp parameter, and  replaced the call to `transfer_low_cp_pokomon' with a call to snippet above.

Really want to see this one implemented. It would be nice to keep at least one of each pokemon.

Related: #142

Closing to avoid duplication spam. ON the works in https://github.com/PokemonGoF/PokemonGo-Bot/pull/274.


⚠ Bofrost.be has degraded performance

In d2f2ab6, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 895 ms

Resolved: Bofrost.be performance has improved in c2a0c0a after 9 minutes.


Solution to deprecated ui.window.locale

I found this guide really helpful and appreciate your efforts in writing this.
Since ui.window.locale is deprecated, what would be the alternative to use instead.
I made it work with WidgetsBinding.instance.platformDispatcher.locale, but I am not sure if it's the best and same solution as before.

I found this guide really helpful and appreciate your efforts in writing this.
Since ui.window.locale is deprecated, what would be the alternative to use instead.
I made it work with WidgetsBinding.instance.platformDispatcher.locale, but I am not sure if it's the best and same solution as before.

It's the solution provided by the official documentation : https://docs.flutter.dev/release/breaking-changes/window-singleton

If no BuildContext is available, the PlatformDispatcher exposed by the bindings can be consulted directly.
Code before migration:

double getTextScaleFactor() {
  return WidgetsBinding.instance.window.textScaleFactor;
}

Code after migration:

double getTextScaleFactor() {
  // View.of(context).platformDispatcher.textScaleFactor if a BuildContext is available, otherwise:
  return WidgetsBinding.instance.platformDispatcher.textScaleFactor;
}

Issue fixed by this pr: https://github.com/bizz84/localization_riverpod_flutter/pull/7


Add openapi/graphql ignore for different entity actions

We want to be able to ignore some default rest endpoints and use custom ones.
app.register(require('@platformatic/sql-openapi'), {
  ignore: {
    categories: {
       fields: ['name'],
       methods: ['POST', 'PUT']
     }
  }
})

What effect would including fields: ['name'] have in the context of the example here?

We want to be able to ignore some fields, some routes and some resolvers. Have you got a more intuitive configuration example?

I don't think this format would work because of multiple GET routes:

GET /entities
GET /entities/:id
GET /entities/:id/relation

We should be able to ignore each of them.


⚠ Jenkins Plugins Artifactory has degraded performance

In 8e9c511, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1246 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 24e8d89 after 37 minutes.


Cythonize BQM.energies

Use cython to speed up energy calculations, falling back on numpy if the extension if not built.
Test script:
import timeit

import networkx as nx
import numpy as np

import dimod

num_variables = 100
num_samples = 1000

samples = np.ones((num_samples, num_variables), dtype=np.int8)

G = nx.complete_graph(num_variables)

bqm = dimod.BQM.from_ising({}, {edge: -1 for edge in G.edges})

N = 1000

contime = timeit.timeit('bqm.to_numpy_vectors()', 'from __main__ import bqm', number=N)
print('numpy conversion time', contime)

cytime = timeit.timeit('bqm.energies(samples, _use_cpp=True)', 'from __main__ import bqm, samples', number=N)
print('cython', cytime)

nptime = timeit.timeit('bqm.energies(samples, _use_cpp=False)', 'from __main__ import bqm, samples', number=N)
print('numpy', nptime)

print('cython/numpy', cytime/nptime)

Output:
numpy conversion time 2.7262168080196716
cython 7.390854711004067
numpy 50.06847931200173
cython/numpy 0.14761492285291822

I would still like to merge this in but needing to overwrite an environmental variable for it to work properly in docker containers seems to be a problem. Documenting some approaches for those who are interested:
A) Try to determine if in a docker container and limit the number of threads using num_threads in prange docs. Of course, there doesn't seem to be a 'nice' way to do that, link, and it's docker-specific.
B) Create an opt-in approach

Decide using a function argument, e.g. bqm.energies(samples, num_threads=2) where None would let openmp choose, defaulting to 1. This just kicks the decision up stream though and does not make for portable code in my opinion.
Decide at install time, e.g. pip install dimod[openmp] whether to use the cythonized parallel code or just use the current numpy implementation. However we probably would not want to pass that up to the sdk, so only people who install dimod directly would benefit from the faster calculations.
Some sort of global variable :nauseated_face: , though that is essentially what the current OMP_NUM_THREADS environmental variable is.


Run simple build on Mesos Plugin in integration test.

Summary:
This patch introduces a simple integration test that runs a build using the Mesos plugin.
The mayor part is the introduction of a Jenkins configuration client that sets all form fields
according the the Jenkins UI.
JIRA issues: DCOS_OSS-4890

overall lgtm we can improve this test as we iterate


bug: Go Debug (Arguments) fails

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.9.1
Operating system/version
mac OS 13.5 (22G74)
Describe the bug
Hi,
When debugging Go projects, if you choose to debug (Arguments) after <lead>dc, the argument window disappears too fast then this message shows up debug adapter reported a frame at line 9 column 1, but: Cursor position outside buffer. Ensure executable is up2date

The only config changes to Lazyvim I have came from the documentation page
require("lazy").setup({
  spec = {
    -- add LazyVim and import its plugins
    { "LazyVim/LazyVim", import = "lazyvim.plugins" },
    -- import any extras modules here
    { import = "lazyvim.plugins.extras.lang.typescript" },
    { import = "lazyvim.plugins.extras.lang.json" },
    { import = "lazyvim.plugins.extras.ui.mini-animate" },
    { import = "lazyvim.plugins.extras.dap.nlua" },
    { import = "lazyvim.plugins.extras.dap.core" },
    { import = "lazyvim.plugins.extras.lang.rust" },
    { import = "lazyvim.plugins.extras.lang.go" },
    { import = "lazyvim.plugins.extras.lang.terraform" },
    -- import/override with your plugins
    { import = "plugins" },
  },

Steps To Reproduce

Create a Go project using go mod init projectname
place a debug point on any line
start the debugger with arguments dc and select Debug (Arguments)

Expected Behavior
Should allow the debugger to wait for arguments to be entered
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  "folke/LazyVim",
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

Hello @habsfanongit,
I searched using "go debug debug adapter reported a frame at line" and found this issue in the nvim-dap repository.
I scanned the issue, the label "needs:repoduction" was added recently.
That issue could benefit from the steps you described.
Best regards!

Hello @abeldekat ,
I just saw the issue and noticed that you added a mention to this one.
I initially thought this issue was caused by the Argument window disappearing quickly.
This is a different issue; I can debug without a problem if I don't use arguments.
I will try a C++ and try to reproduce the same issue you found and report it with details.
Thank you.

This isn't really a LazyVim issue. Report it to the relevant plugins instead. Closing....


NO-TICKET - kover coverage improvements

Description of Work

Updated kover to 0.7.2
Added NoCoverage annotation to exclude parts of code from coverage
Using updated PR reporter to display coverage information
Added emoji for minimal coverage state
Show files without full path

Kover still in active development and some code cannot be excluded from reports because of bugs. For example: https://github.com/Kotlin/kotlinx-kover/issues/331

Waiting for response how to get fork access to move reporter action into the p2p-org


1 UUID only and only for one user

Hello v2ray developers,
I have this problem to provide v2ray to my users, I would like to limit v2ray UUID that we use in vmess only to be used by one user not more.
Please let me know the solutions.

Hii


CocoaPod update after iOS8 tint (style) fix

As per title, please can you update the pod spec with a newer version now that the iOS8 tint style has been exposed? Thanks!

+1, please, please!

Temporary solution is to specify commit:
pod 'AFBlurSegue', :git => 'https://github.com/AlvaroFranco/AFBlurSegue', :commit => '0efebb014f68c58675b03d51e475e6c6384c1fa9'

@AlvaroFranco any update on this? can you please bump the pod version?

Sorry guys! I'm really busy with my finals

thanks man! 🍻👍🏼


[BottomSheet] Add new component

I've made a BottomSheet component, it was based of Snackbar.
It has modal, persistent, and inset props, to accomodate for most of the MD-specs. And an option to add a FloatingActionButton as prop action (string), which will render 1 FAB on the sheet with the string of the action as the icon.
Note:
The tests pass, but I get an error:

Warning: Exception thrown by hook while handling onSetChildren: Invariant Violation: Expected onBeforeMountComponent() parent and onSetChildren() to be consistent (15 has parents 0 and 13).

I'm not sure how to fix that, as I'm not very experienced with working with Mocha or Karma.
I've asked in gitter if someone was working on it, but no reply yet, so see what you do with this.

[x] PR has tests / docs demo, and is linted.

I just remembered to search for BottomSheet to see if someone was already working on it, turns out, @oliviertassinari is.
But since this was in June, I'm not sure what is happening with this.
I also see that that PR is replacing/enhancing the Drawer which is interesting, and probably better than this implementation. I don't see any implementation of a Fab yet?

Maybe we should instead of replacing Drawer, make an internal Sheet component that can slide like Both the drawer and a bottomsheet, and perhaps extend the appbar with it as well, to accomodate for expanding headers..

To add some context. This PR is related to #4732, I have hold on the development of #4732 for the next branch.

Maybe we should instead of replacing Drawer, make an internal Sheet component that can slide like

@TrySpace I agree with that direction. I have played with the demo of your PR, it's cool 👍 .
But I'm closing it, as well as #4732.
Our plan regarding moving forward is to only merge bug fixes on the master branch and continue migrating new components on the next branch. The next branch is moving away from the inline-style approach. That's changing the way components are implemented.
Helpers like withWidth have a smaller range of use-cases.
Thanks for your work.


Rosflight Baro measurement calibration

This data is pulled from a recent flight of rosflight where two other state estimators were also on-board. As you can see, the measured pressure is different than the other two pressure measurements. The INS and the Pixhawk both use GPS to estimate the barometer bias, but rosflight does not. This means that when you calibrate the altitude at the beginning of the flight, it will indeed set your altitude to zero, but your altitude estimate will not be accurate as you climb in elevation. I observed about a 10 meter altitude difference when the aircraft was at an altitude of 70 meters.

This I believe should be fixed by the new way that the firmware calibrates IMU

This was fixed by @len0rd


feature request: print manufacturer and and product in lsusb

diff --git a/util-linux/lsusb.c b/util-linux/lsusb.c
index e27aa7f31..b7b7abb99 100644
--- a/util-linux/lsusb.c
+++ b/util-linux/lsusb.c
@@ -15,7 +15,16 @@
 //config:	system and devices connected to them.
 //config:
 //config:	This version uses sysfs (/sys/bus/usb/devices) only.
-
+//config:
+//config:config FEATURE_LSUSB_STRINGS
+//config:	bool "Print vendor and product strings if they exist"
+//config:	default y
+//config:	depends on LSUSB
+//config:	help
+//config:	lsusb is a utility for displaying information about USB busses in the
+//config:	system and devices connected to them. This option prints out the
+//config:	vendor and product strings if they are available. This option
+//config:	replicates a simpler version of util-linux lsusb output.
 //applet:IF_LSUSB(APPLET_NOEXEC(lsusb, lsusb, BB_DIR_USR_BIN, BB_SUID_DROP, lsusb))

 //kbuild:lib-$(CONFIG_LSUSB) += lsusb.o
@@ -37,6 +46,16 @@ static int FAST_FUNC fileAction(
 	int product_vid = 0, product_did = 0;
 	char *uevent_filename = concat_path_file(fileName, "/uevent");

+#if ENABLE_FEATURE_LSUSB_STRINGS
+	ssize_t vlen, plen;
+	// max length standard allows, any longer and someones being naughty
+	char vendor[256] = { 0 }, product[256] = { 0 };
+	vlen = open_read_close(concat_path_file(fileName, "/manufacturer"), vendor, sizeof(vendor));
+	vendor[vlen - 1] = '\0';
+	plen = open_read_close(concat_path_file(fileName, "/product"), product, sizeof(product));
+	product[plen - 1] = '\0';
+#endif
+
 	parser = config_open2(uevent_filename, fopen_for_read);
 	free(uevent_filename);

@@ -64,7 +83,12 @@ static int FAST_FUNC fileAction(
 	config_close(parser);

 	if (busnum) {
-		printf("Bus %s Device %s: ID %04x:%04x\n", busnum, devnum, product_vid, product_did);
+		printf("Bus %s Device %s: ID %04x:%04x", busnum, devnum, product_vid, product_did);
+#if ENABLE_FEATURE_LSUSB_STRINGS
+		if(vlen) printf(" %s", vendor);
+		if(plen) printf(" %s", product);
+#endif
+		printf("\n");
 		free(busnum);
 		free(devnum);
 	}

i have been trying to convert this into toybox for lsusb as a patch for my personal use, but never figured it out.

why not simply retrieve them from the manufacterer and vendor files as seen in the busybox patch?

on another note, http://www.linux-usb.org/usb.ids doesn't change the output of lsusb.

When you send me GPLv2 code copied from busybox, I'm not gonna look closely at it.

heres a little shell script i made that prints the usb stuff and everything.
#!/bin/sh

for usb in /sys/bus/usb/devices/*; do
        [ -f $usb/uevent ] && [ -f $usb/idVendor ] && {
                eval $(cat $usb/uevent)
                read -r idproduct < $usb/idProduct
                [ -f $usb/product ] && read -r product < $usb/product
                read -r idvendor < $usb/idVendor
                [ -f $usb/manufacturer ] && read -r vendor < $usb/manufacturer
                printf 'Bus %s Device %s: ID %s:%s %s %s\n' \
                        $BUSNUM $DEVNUM $idvendor $idproduct "$vendor" "$product"
                unset idproduct product idvendor vendor BUSNUM DEVNUM
        }
done

this should sort of explain it.