Cannot publish OSX artifacts for 2.7.2

I am using Vagrant 2.2.16 (latest version), installed with home brew on MacOs.
the script that builds the OSX python packages during the release fails
https://github.com/apache/pulsar/wiki/Release-process#macos
last lines are:
  default: adding 'pulsar_client-2.7.2.dist-info/WHEEL'
    default: adding 'pulsar_client-2.7.2.dist-info/top_level.txt'
    default: adding 'pulsar_client-2.7.2.dist-info/RECORD'
    default: removing build/bdist.macosx-10.12-x86_64/wheel
    default: + popd
    default: ~/pulsar/pulsar-client-cpp
Usage: vagrant [options] <command> [<args>]

    -h, --help                       Print this help.

Common commands:
     autocomplete    manages autocomplete installation on host
     box             manages boxes: installation, removal, etc.
     cloud           manages everything related to Vagrant Cloud
     destroy         stops and deletes all traces of the vagrant machine
     global-status   outputs status Vagrant environments for this user
     halt            stops the vagrant machine
     help            shows the help for a subcommand

probably the script does not support recent versions of Vagrant ?

The issue had no activity for 30 days, mark with Stale label.

Closed as stale.
The development of the Python client has been permanently moved to https://github.com/apache/pulsar-client-python. If it's still relevant, please open an issue there.


[Scala 3] Make sure derives can also start on the next line

I just added some small refactors to the tests, nothing major.

I just added some small refactors to the tests, nothing major.


Admin feeds

Implement an admin feed as in e.g., the Facebook 'world' feed to moderate comments, or signal in any other way. Could be useful for other users as well perhaps in new relevant content being posted.
https://github.com/tschellenbach/Feedly

done in current dashboard, but more simple


Button [disabled]=true still invokes [routerLink] - is this ok?

So if I have the following in my template:
<a mat-raised-button [disabled]="!buttonsEnabled" [routerLink]="['']">Back</a>
and when buttonsEnabled = false it follows [routerLink]. This is kind of surprise to me. Am I missing something or is this a bug?

There is no disabled attribute for hyperlinks. change for button

Man, good point! Thanks.


Fix left motion vector prediction

Left motion vector index, left_index1, is currently pointing to an
unrelated position in the picture (offset by non-integer number of
block_strides).

Fixed in current master.


PVC fails to attach during workload creation

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):

Create a single node cluster - AWS - cloud provider set
Create a PV in the cluster level named pv1
Go to the default project, create a pvc named pvc1 using the existing PV.
Volume is created successfully.
Attach the volume in a workload.
The workload fails to deploy.

Expected Result:
Attaching of volume should succeed
Actual Result:
Error seen while attaching PVC while creating workload
Warning | FailedAttachVolume | AttachVolume.Attach failed for volume "pv1" : InvalidParameterValue: Value (v1) for parameter volumes is invalid. Expected: 'vol-...'. status code: 400, request id: 5aa82c06-7784-407b-89c3
Warning | FailedAttachVolume | (combined from similar events): AttachVolume.Attach failed for volume "pv1" : InvalidParameterValue: Value (v1) for parameter volumes is invalid. Expected: 'vol-...'. status code: 400, request id: 697475bc-9884-4a40-83aa-035d125118a7
Warning | FailedMount | Unable to mount volumes for pod "wk01-c46cf7c8-zmvkj_default(5bcf07f2-ea88-48ab-9b41-7556b4185b25)": timeout expired waiting for volumes to attach or mount for pod "default"/"wk01-c46cf7c8-zmvkj". list of unmounted volumes=[vol1]. list of unattached volumes=[vol1 default-token-7m5sk]

Note:
This was seen in v2.2.8 of rancher also.
Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): rancher-master latest
Installation option (single install/HA): single

Per the rancher docs https://rancher.com/docs/rancher/v2.x/en/cluster-admin/volumes-and-storage/  & the k8s docs for ebs https://kubernetes.io/docs/concepts/storage/volumes/#awselasticblockstore.
Assuming aws & cloud-provider is set
If a Persistent Volume is created in Rancher, then an equivalent EBS volume must be created in AWS.

Create ebs volume in aws.
use volume-id from aws for creating the PV in Rancher.
Persistent volume can be used to create volumes at the project level
Volumes can be mounted into workloads
Note: All of the steps above will complete successfully until attempting to mount the volume if there is an error in step 1 and 2.

To enable dynamic provisioning of PersistentVolumes without the need to create EBS volumes every time you can define a storage class https://kubernetes.io/docs/concepts/storage/storage-classes/

Define an EBS storage class
Use this storage class when creating new persistent volumes.

On rancher:master-head
Steps:

Create an EBS volume on AWS. Note the Volume-id
Create a single node cluster - AWS - cloud provider set
Create a PV in the cluster level named pv1, add Volume-id created in Step 1 while creating this.
Go to the default project, create a pvc named pvc1 using the existing PV.
Volume Claim is created successfully.
Deploy a workload and attach this volume to the workload.

Was validated -- https://github.com/rancher/rancher/issues/22614#issuecomment-528959250


chore: fix layout and styles of source select and search bar [f-7]

This PR improves the layout styling of the top bar in the modal UI, since I think I broke some functionality in previous PRs. This is not perfect, but this is a good step in the right direction.
I think we should come to an agreement around the styling of icons. Usually what I've found works best in the past if if we fix the bounding box of icons to a fixed size, since this allows us to then add padding and set the width/etc of icons without concerning ourselves what icon we use. In this case I've gone with 16x16 to begin with, since it looks like all the icons fit inside this box. For icons which aren't exported from Figma at the right size, I've used a wrapper div (see DownArrowSvg) to bring it up to the right size.

Stacked PR Chain: f-7

PR
Title
Status
Merges Into

#124
👉 chore: fix layout and styles of source select and search bar
Pending
-

#125
*(Draft) feat(search): use hooks to style on input focus and show recent queries *
Pending
#124

#126
chore: fix className overriding
Pending
#125

#127
chore: layout top bar correctly
Pending
#126

#128
chore: fix left icon
Pending
#127

#129
chore: update source select dropdown, and api calls
Pending
#128

/annotate


error values

You should be able to create an error value with error("foo"), i.e., cast the string value "foo" to an error with the given string.  Currently this fails with an error in the cast expression.

This should be done with structured errors

This will be handled by upcoming structured errors change.


Setup: Cache cypress binary correctly for e2e testing

Caching is currently disabled for e2e and visual regression tests because the binary for cypress is not correctly cached and restored, leading to a failed workflow run. Disabling the cache means the install step roughly takes a minute instead of 5 seconds.
Find a way to correctly cache the cypress binary in a pnpm monorepo setup.

Closing as not needed at the moment.


Moving optimizely

Please rebase on top of master and fix CI.

@jelz Going to be working on cleaning up the tests now.
Cleaned up the inline oAuth flow.

If your token is expired on land in the variation container, you'll see this screen.

If you are working on the variation container and the token expires within 10 minutes, this banner will pop up. This happens in realtime, no need to reload the page.


Introduce SDK generation

Since it's already generating documentation, I think it'd be great if we could automatically generate SDK as well.
The way I'd propose is the following:
We know what the client is sending, and what the client will get through the docblocks, we could put in a limitation where people have to create some sort of interface or "model" (not relating to eloquent), which client either passes or gets.
Let's say we have a API for creating Users. We could generate a TypeScript SDK, and user won't have to write any URLs or anything. All user would do is:
import UsersApi from 'my-sdk';
UsersApi.create({
    id: null,
    first_name: 'Something',
    last_name: 'anything but something',
    email<EMAIL_ADDRESS>}).then(data => {
    console.info("User created: ", data);
}).catch(err => {
    console.error("Something wrong:" , err);
})

We could let users to hook some sort of adapters so they can generate SDK for any language, not just for JavaScript/TypeScript
How I see the flow:
I want a User's creation endpoint:
I define what kind of data will be transferred, implement everything needed, then finally generate the SDK, use it in client's end.

Hey @cyberhck, thanks for the suggestion. That would be cool. We can explore this for version 3.0 and beyond, but this is pretty low-priority, considering we have a lot of bugs and enhancements to iron out.


Update to cf-java-client 2.9.0.RELEASE

As a developer, I'd like to upgrade CFJC to 2.9.0.RELEASE when it's released.

This is addressed in the cf-deployer.


Fibonacci should only be calculated for values bigger than 2.

In this case, there should be a less or equal comparison, instead of less than.

Why not < 3 - saves the equal, or is this automatically done by the compiler?

or is this automatically done by the compiler?

No, but it would be if it was actually faster.

Actually they are both one instruction inside the cpu and there shouldn't be any difference between "greater" and "less or equal" (in Assembly JGT vs JLE).

The example is OK. Enumeration starts with 0 instead of 1. See fib(10) == 89

The example is OK. Enumeration starts with 0 instead of 1. See fib(10) == 89

Same thoughts here, closing the pull

Fib 10 is 55, fib 11 is 89.
The calculation is wrong. http://www.maths.surrey.ac.uk/hosted-sites/R.Knott/Fibonacci/fibtable.html

It just indexed from 0, instead of 1.

That doesn't matter. The calculation is not the correct one. If a beginner wants to validate the function, he gets different results than with any other fibonacci implementation.
Fibonacci is one of the basic examples when you learn about recursion and a prime example for memoization, as it always yields the same result for the same input. This shouldn't yield the wrong result.

Then fix it

I have. Therefor the pull request.
It should be <= 2.

Fib 10 is 55, fib 11 is 89.
The calculation is wrong. http://www.maths.surrey.ac.uk/hosted-sites/R.Knott/Fibonacci/fibtable.html

Your current fix is wrong

Yeah, you are right. The check for 0 is missing.
ulong fib(ulong n)
{
if (n == 0) return 0;
return n < 2 ? 1 : memoize!fib(n - 2) + memoize!fib(n - 1);
}
assert(fib(10) == 55);

Ping @hardliner66

@hardliner66 - please have a look at the @lionello's comment ;-)

+1 for the solution from @lionello

+1 for the solution from @lionello

@hardliner66 - can you rebase your PR, s.t. we can move forward? ;-)

I commited the new version :)

I commited the new version :)

Thanks - let's move forward then?

Auto-merge toggled on


Numbers - Improve Flexibility

Improve flexibility of Numbers module.

Make Strict classes open, enable overriding their methods
Remove private prime cache from PrimeNumberTools, use functional dependency injection to supply cache

Resolves #58

Assumptions that were previously valid need to be checked. It is now more likely that an error will appear, particularly in the case where a BytePrimeCache is passed as an argument.

Existing PrimeNumberTools test cases do not check large inputs that may expose errors related to cache size.

Numbers Format package updates should have been added to a different branch, and different PR. These aren't behavioral changes, but will simplify usage in Java (so does improve flexibility).

Maybe PrimeNumberTools can be renamed to Primes or PrimeTools

Prime redesign should have been done on another PR. Now, it is blocking the other updates from being part of the release.

This branch will be split into primes and non-primes updates, and this PR will be closed.


[Feature Request] Archive PyCon APAC 2015 Website

Description
Since the PyCon APAC 2015 is held by PyCon Taiwan we should also archive the website as what we have worked on other historical PyConTW websites. The historical serving websites could be archived for reducing the backward compatibility of DB.
Possible Solution
Currently, we have a repository storing all PyCon APAC 2015 pages and materials. we could first check if everything works great in the repository. After the work has been done we could continue moving the stuff from the repository to branch gh-pages.
Related Issue
#32

Solved by #33


Add exceptions to API documentation

Summary
The big change here is that QiskitError is added to the API documentation, which causes 400--500 previously failing Sphinx lookups (stemming from "Raises:" documentation) to now succeed.  This commit also corrects several other places where exceptions were not being fully documented (and in several cases makes the imports more convenient as well), and corrects a couple of places with incorrect references to exceptions.

Details and comments
This stemmed from an idle thought I had to turn on nitpicky in the Sphinx configuration, which emits a warning if any cross-reference fails to resolve.  That's probably a longer-term effort: including the ones fixed by this PR, there were 2791 warnings.  It might cause some more nuisances with release notes and removals, but we'll have to see how that goes if we decide to actually turn the option on.  The principle of making sure that most of our references resolve is good regardless, though.
qiskit.algorithms and qiskit.opflow are both deprecated but emit sizeable numbers of references, and the strategy here is probably just going to be to leave them be, since they're scheduled for removal from this repo anyway.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

8 of 8 (100.0%)  changed or added relevant lines in 8 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.03%) to 85.916%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
3
91.9%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
73084

Relevant Lines:
85064

💛  - Coveralls


{"status":"not-found"}

I installed Local API App for Homey, set the port to 500, disabled CORS and restarted the app.
If I try this request within an browser: http://172.16.x.xx:5000/dingzSwitchEvent or get://172.16.x.xx:5000/dingzSwitchEvent within Postman (with the full IP of course) the response is {"status":"not-found"}
edit: I have the new early 2023 pro
Can you help me?

Hi, can you provide a screenshot of your Homey Advanced flow?


Move towards Symbols as WeakMap keys

got halfway through review but we need to land it to change the agenda, we can continue review and introduce changes afterwards


Upgrade Linux Kernel for flatcar-3874 from 6.6.16 to 6.6.20

Subject says it all.

CI fails at build_image when running sbsign only with amd64, added on-hold.

See instead https://github.com/flatcar/scripts/pull/1735


PowerToys Run - Calculator - Random results in divisions with more than 18 decimals

Microsoft PowerToys version
0.69.1
Installation method
GitHub, PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce
While performing a calculation using a lot of decimal places to approximate a repeating decimal number if the number of decimal exceeds 18 the result is totally off.
Open power toys run and perform the following operation
120/3.3333333333333333333 (19 decimal places) = 80.6094652759 (instead of 36).
The following operation works just fine: 120/3.333333333333333333 (18 decimal places) = 36 as expected,
✔ Expected Behavior
When using more than 18 decimal digits should not return an invalid result
❌ Actual Behavior
While using more than 18 decimal digits the result seems to be random
Other Software

OS Name
Microsoft Windows 11 Business

Version
10.0.22621 Build 22621

Processor
12th Gen Intel(R) Core(TM) i7-1260P, 2100 Mhz, 12 Core(s), 16 Logical Processor(s)

This is a quirk due to the engine with overflow. Can you help us understand when in normal usage this would affect you?  We’d have to build out the same parser in the math engine I believe to deal with this

It's not a big issue for me, knowing that too many decimals cause this overflow I'll simply keep an eye out for that. I just wanted to report it.
In my opinion, a simple solution could be to truncate the number of decimals to 18 when parsing the expression to avoid the overflow and prevent the return of an incorrect result.

/dup https://github.com/microsoft/PowerToys/issues/20570


add partial support for Debian 10, and supporting changes

Pull Request (PR) description
This PR adds support for Debian 10 (and possibly variants like Ubuntu).
The defined types that rely on the SELinux Ruby gem do not work, as it is not currently shipped with the Puppet Agent package for Debian.
This PR grew to include 3 refactoring commits and 1 bugfix to support the new feature. Sorry for the overall size.
This Pull Request (PR) fixes the following issues
Fixes #309
Obsoletes and closes #300

Hi @tequeter, thanks for the awesome work. Can you take a look at the inline comments I made?

Thank you for the review, @bastelfreak ! I've commented above and pushed a new version.

thanks for your work!


Documentation is out-of-sync with Visual Studio 2022 - Preview version 17.2 Preview 6

See the workaround found in this issue: https://github.com/dotnet/maui/issues/6831, which seems to indicate that calling msbuild from the command line is no longer needed (in fact, it does not work). Instead, building the Release profile targeting Windows Machine from the VS UI nicely produces a .msix file.

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

ID: 46808586-50e1-acfa-68b3-c3ca6572dc62
Version Independent ID: 46808586-50e1-acfa-68b3-c3ca6572dc62
Content: Publish a .NET MAUI app for Windows - .NET MAUI
Content Source: docs/windows/deployment/overview.md
Product: dotnet-mobile
Technology: dotnet-maui
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Related to #352

Added the updated TFM to a PR I had open to switch from MSBuild to dotnet already: https://github.com/dotnet/docs-maui/pull/508


fix linting error

What
Checklist

[ ] I have read and agree to Contributor Guide and the Code of Conduct
[ ] I've included a screenshot or gif (if applicable)
[ ] I have tested [demo.rerun.io](https://demo.rerun.io/pr/{{ pr.number }}) (if applicable)

[PR Build Summary](https://build.rerun.io/pr/{{ pr.number }})
[Docs preview](https://rerun.io/preview/{{ pr.commit }}/docs)
[Examples preview](https://rerun.io/preview/{{ pr.commit }}/examples)
Recent benchmark results
Wasm size tracking

... of all the things it's a the the lint...


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative-extensions/eventing-writers
/assign knative-extensions/eventing-writers
Produced by: knative-extensions/knobots/actions/update-deps

/retest

/test channel-integration-tests-sasl-plain

/retest

/retest

/retest

/retest

#3412 fixes the build issues

/retest

/retest

/retest

/retest-required

/hold
not sure about https://github.com/knative-extensions/eventing-kafka-broker/pull/3410#discussion_r1366505019

/hold

/unhold

/retest

/retest

/test reconciler-tests


run image as non-root user to comply with PSP

We are using the cost-model deployment along with our own custom PodSecurityPolicies that require images to run as non-root.

Awesome, we actually had an open issue to do this here: https://github.com/kubecost/cost-analyzer-helm-chart/issues/64. We'll get this tested today. Thanks so much for the contribution!


MAINT: special: remove one libnpymath dependency; more NPY_ -> C99

Replacements:

NPY_NAN -> NAN
NPY_INFINITY -> INFINITY
NPY_PI -> M_PI
NPY_E -> M_E
npy_isnan -> isnan
npy_isinf -> isinf
npy_isfinite -> isfinite
npy_copysign -> copysign
npy_signbit -> signbit

Also cleans up a lot of Cython imports, and removes one libnpymath inclusion. That removal reduces binary size and works towards not depending on libnpymath at all - see https://github.com/numpy/numpy/issues/20880.
Things brings us down to 13 places where we still include numpy/npy_math.h.

The one failure (TestSchur.test_sort) is gh-14517, unrelated to this PR. I'll cancel CI for the second commit (please squash-merge that)

LGTM. Squash-merged as requested to keep the ball rolling, thanks Ralf, all.


How to run this code?

Hello.
I was very impressed since reading your paper and found this repository.
But I have some trouble about running this NC-ICF model, because there seems to be no executable code here at once.
Could you please upload the code for run this model?
Thank you.

Hello, author,
I read this paper of yours and I think it is very good, so I want to run it, but I can't find ml-1m and taobao, these two datasets, please the author can send these two datasets to me.


About partition flies.

Hello, how to get the partition file of Market1501 from the original dataset? Would you share the code? Thanks.

Please downloaded it from here
http://robustsystems.coe.neu.edu/sites/robustsystems.coe.neu.edu/files/systems/code/reid_benchmark/partition/Partition_market.mat

That's not what I meant. I was curious about how to get the partition file, because we want to do some experiments on our data set. Would you share the code? Thanks!


WIP, DO NOT MERGE! RHOARDOC-1393: WildFly Swarm rename to Thorntail

This is a set of tightly related changes that are tracked in the sub-tasks of RHOARDOC-1393.
Notes:

The docs/topics/wildfly-swarm directory was abandoned in favour of docs/topics/thorntail.
The new version of Thorntail was synced because the naming changes were not backported.
The references to WFS remain in the Contribution Guide topics because they're in the same commit as the sync script changes.
The guide directory was renamed to docs/thorntail-runtime.
The guide file was changed to thorntail-runtime.html and I have set up redirects from the old guide to the new one. I have tested them on the local server, but please give them go as well.
I have added an admonition at the beginning of the guide with the info about the rename.
I have added a workaround for ENTSWM-458 to the sync script, which prevents upstream syncing. It does not affect our docs delivery in any way at the moment.

Changes currently missing from this PR:

The changes to the sync script (see RHOARDOC-1394) are missing from this PR for productisation reasons. The commit with those will be provided later in a separate PR.
The new Thorntail logo is missing from the launcher landing page. I will add it when it's published.
The URLs to the community documentation will be provided when it's published.

The changes are pretty strictly organised into commits, so in my opinion, it makes sense to review individual commits as opposed to the entire change set at once.
Resolves RHOARDOC-1393, 1398, 1399, 1400, and 1358.

Will do, @gastaldi, thanks.

LGTM

Thank you, @huntley.

@Ladicek The preview is here (VPN required). I'm especially not sure of some Artifact IDs that still say swarm, so please advise.

Most of the occurences of swarm are OK, because all the Java package names are still org.wildfly.swarm.* and all the configuration keys are still swarm.*. But there are indeed some outdated pieces:

In chapter 4.1.1. Creating an application from scratch, in the listing of pom.xml, we have:

<name>Wildfly Swarm Example</name> -- that should read <name>Thorntail Example</name>;
version.wildfly.swarm -- that should be version.thorntail (the value of that property will change to something like 2.2.0.Final-redhat-00015, but we don't know the exact version number yet);
a XML comment <!-- Wildfly Swarm Fractions --> -- I'd actually remove it completely, but if not, then it needs to change to <!-- Thorntail Fractions -->.

In chapter 4.2. Deploying an existing Thorntail application to OpenShift, in the listing of pom.xml, we have:

<include>wildfly-swarm</include> -- that should read <include>thorntail-v2</include>;
<wildfly-swarm-health-check> -- that should read <thorntail-v2-health-check>.

In chapter D.23.5. MicroProfile Metrics, we have a hardcoded reference to org.wildfly.swarm:microprofile-metrics at the beginning. This needs to change to io.thorntail:microprofile-metrics. Also a few paragraphs below, the example says swarm-demo -- that should probably be changed to thorntail-demo.
The warning at the very beginning of chapter D.24. Monitor includes org.wildfly.swarm:microprofile-health. That needs to change to io.thorntail:microprofile-health.

That's all I could find.

Thank you for the feedback, @Ladicek. Remaining problems fixed in https://github.com/thorntail/thorntail/pull/1139 and https://github.com/wildfly-swarm-prod/wildfly-swarm/pull/150; please merge them, I will resync this PR when those are accepted.

Cool, thanks, will merge the PRs once the PR jobs finish.


feat: DEV-2755: Regions list grouped by labels with persistent & ordered collapse

/git merge develop


Fix 401 Unauthorized at base url for unauthenticated users

Previously, users who are not logged in (and therefore unauthenticated) would

receive a 401 unauthorized at /
they would have to access /session/new to log in first
after which they would be redirected to /.

With these commits, users who are not logged in would

be redirected to /session/new when visiting /, instead of receiving a 401 unauthorized

The behaviour for authenticated users are unchanged.

I'm not too certain if this is best practice (perhaps the logic could be implemented in the router instead of controller?)

Looks like the drop in coverage is due to there not being a 401 unauthorized response in the tests. We can probably regain the coverage once the routes are more 'populated', so there are things to be unauthorized against?
Current routes,
   page_path  GET   /                CadetWeb.PageController :index
session_path  GET   /session/new     CadetWeb.SessionController :new
session_path  POST  /session         CadetWeb.SessionController :create
session_path  GET   /session/logout  CadetWeb.SessionController :logout```

This should be implemented at middleware / Plug level instead of controller, otherwise every authenticated controller will have to implement the same logic.
Override the auth_error handler of Guardian, see
https://hexdocs.pm/guardian/Guardian.Plug.EnsureAuthenticated.html


enhancement: Dynamic loading

Load languages from the e.g filesystem

Done in https://github.com/dioxus-community/dioxus-i18n/commit/51847543013bd1dc5cdd3486fcea69e0f9887dd4


Adding pipenv support

I think this project could be made easier for new users/contributors if we utilized Pipenv, in particular https://docs.pipenv.org/advanced/#specifying-basically-anything to specify operating system specific requirements.
This way if you wanted to be a contributor and set up a dev environment it would be as simple as pipenv install --dev versus checking, then installing which operating system dependencies you need and then pip installing the development dependencies.

Hey!
I see you added support for OSX and Windows, however it seems that Linux wasn't tested?
When I try to install with pipenv on linux:
freshollie@freshollie-desktop:~/Documents/Repos/toontown-infinite$ pipenv install pyautogui
Installing pyautogui…
Collecting pyautogui
  Downloading https://files.pythonhosted.org/packages/2e/83/89b5adbc37d1bbf7b486a2c1c00e8037e6f801e8c053c4897bb82d9510c6/PyAutoGUI-0.9.36.tar.gz (46kB)
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-install-i28Tss/pyautogui/setup.py", line 6, in <module>
        version=__import__('pyautogui').__version__,
      File "pyautogui/__init__.py", line 115, in <module>
        from . import _pyautogui_x11 as platformModule
      File "pyautogui/_pyautogui_x11.py", line 7, in <module>
        from Xlib.display import Display
    ImportError: No module named Xlib.display

    ----------------------------------------

Error:  An error occurred while installing pyautogui!
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-install-i28Tss/pyautogui/

Running pipenv install Xlib and then reinstalling pyautogui fixed the error.

@freshollie, yes this is possible.. https://stackoverflow.com/a/35614580/4752883

@freshollie, yep as @alphaCTzo7G wrote you can add something like Xlib = {version = "*", sys_platform = "== 'Darwin'"}to the Pipfile to handle this. I'm happy to update the MR to include that I would just need someone else to test it for me since I only have OSX and Windows handy.

@afs2015, Currently, it seems Pyautogui uses setup.py to determine platform-neutral packages..
However, even setup.py can be modified to include python specific packages: http://setuptools.readthedocs.io/en/latest/setuptools.html#declaring-platform-specific-dependencies
Wondering what advantages do you use of adding pipenv, and piplock files, over just modifying the setup.py files? If we just keep the setup.py file, which generally works well.. and just add the platform specific settings, then wont it be simpler?
I haven't dug into pipenv too much.. Does this automatically ensure that users who use pip install Pyautogui automatically will get the platform-specific libraries as well platform-neutral libraries in one shot?

@alphaCTzo7G I have just reread everything here, and realised the functionality I was looking for was actually the dependency in the setup.py....
Sorry for the confusion. The Pipfile in the library is supposed to be used for when contributing to the library. It is not, as you said, used by pipenv when running pipenv install pyautogui.

@afs2015, do you want to modify the setup.py file and modify your PR then?
It will then help users as well as contributors easily install pyautogui on
all platforms, without having to install specific dependencies and have errors.
I can do it as well, but I dont have Mac OS. I can verify on Linux and Windows.

@asweigart what do you think?

Yes, I'm in favor of it, in general. I'm buried under other committments right now, but in general I'm open to PRs. You might have to ping/email me to get my attention on it though. Sorry about this. I'm hoping I'll have more time next week to pay attention to PyAutoGUI issues, as I have a lot of ideas for progressing on its roadmap.

I created a PR #241 to automatically install the dependencies.
Because I dont have access to the pypi repository, I created a test using
to install pyautogui using the local repository itself.
mkdir ~/tmp;
cd ~/tmp;
rm -rf env pyautogui;
git clone --single-branch -b install_dependencies_wo_user_intervention git@github.com:alphaCTzo7G/pyautogui.git;
virtualenv --python=$(which python3) env;
source env/bin/activate;
pip install -e pyautogui;
pip list;
deactivate

Can you guys check if pip list in the above sequence shows that the proper
libraries are being installed for Mac.
I have verified that it works fine for Windows, Linux(python2 and
python3)
cc @afs2015, @freshollie, @shantang96, @jtara1

The missing Linux platform dependency was fixed in 13c83ce3.


Config option for custom PATH to Plugin Directory?

Hi - would it be possible to add a config option for a custom PATH to the plugin directory/override/config, similar to BitBar?
This allows one to keep their plugins under version control/at a local directory that won't get blown away when migrating to a different system.

You can store in iCloud by using:
mkdir -p ~/Library/Mobile\ Documents/com~apple~CloudDocs/xbar/plugins
rm -rf ~/Library/Application\ Support/xbar/plugins
ln -s ~/Library/Mobile\ Documents/com~apple~CloudDocs/xbar/plugins ~/Library/Application\ Support/xbar/plugins


Subscribe (watch page) n or all users to a given page

API Endpoint or Feature
Give a bookstack admin the ability to subscribe n or all users of the wiki to a given page via API. This feature exists on a per user basis if you click on watch page, so that you can get notifications on changes.
Use-Case
we use a landing page, which is also a whiteboard, where all department news are published and we would like every user to automatically watch this page, so that every user gets a notification, if this pages changes.
Additional context
This may also be a general feature request and not api, since there could also be a subscribe button for admin users, where they could select n users to watch that page.

Thanks for the request @Karamelmar.
This is closely related to #4514, so I may close & merge this into that issue to de-duplicate.


User auto activation

Any way to enable auto activation of users that was synchronized with LDAP?

auto activation

Hello @rapososoftwares What do you mean? Are you having trouble logging in or receiving email notifications?

auto activation

Hello @rapososoftwares What do you mean? Are you having trouble logging in or receiving email notifications?

I would like to disable the need of activate the user account by confirming by email.

.

I would like to disable the need to activate the user account by confirming by email.


Change to support flexible server in production.

Purpose
We'll be updating tutorial https://learn.microsoft.com/en-us/azure/developer/python/tutorial-python-managed-identity-01 to used Flexible Server. And the user name for Flexible Server is different.
Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Get the code and follow the tutorial shown above.

Holding on this for a bit.


[Request] Ignore Pastes or Tildes

I like to copy-paste certain mods, and the addon adds an extra tilde at the start to pastes that already include one, breaking the search.
It would be nice if the addon could detect a paste (ideal) or if the paste contains a tilde already.
Thanks!

Hi,
the issue with copy-pasting was that the addon activated already upon pressing CTRL when pasting and then inserted the tilde.
I'm uploading a version right now that does not add the tilde if the query is empty, as when pressing any modifier keys. This also prevents adding another tilde when copy-pasting.
Hopefully this should fix your issue (Might take a day or so for the webstore versions to update).


SDP-488: store the Anchor Platform's tx ID when registering a receiver's wallet

What
Store the Anchor Platform's tx ID when registering a receiver's wallet
Why
So we can PATCH the AP with the success/error states later.
Checklist
PR Structure

[x] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[x] This PR does not mix refactoring changes with feature changes (split into two PRs otherwise).
[x] This PR's title starts with the name of the package, area, or subject affected by the change.

Thoroughness

[x] This PR adds tests for the new functionality or fixes.
[x] This PR contains the link to the Jira ticket it addresses.

Configs and Secrets

[x] No new CONFIG variables are required -OR- the new required ones were added to the helmchart's values.yaml file.
[x] No new CONFIG variables are required -OR- the new required ones were added to the deployments (pr-preview, dev, demo, prd).
[x] No new SECRETS variables are required -OR- the new required ones were mentioned in the helmchart's values.yaml file.
[x] No new SECRETS variables are required -OR- the new required ones were added to the deployments (pr-preview secrets, dev secrets, demo secrets, prd secrets).

Release

[x] This is not a breaking change.
[x] This is ready for production.. If your PR is not ready for production, please consider opening additional complementary PRs using this one as the base. Only merge this into develop or main after it's ready for production!

Deployment

[ ] Does the deployment work after merging?

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr44.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr44.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr44.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr44.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr44.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr44.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr44.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr44.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr44.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr44.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr44.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr44.previews.kube001.services.stellar-ops.com


Upgrade DSharpPlus to 5.0.0-nightly-02297

Draft for now because slash commands are broken.

Slash commands fixed with https://github.com/DSharpPlus/DSharpPlus/pull/1973 - moving out of draft status.


v1.13 Backports 2023-06-28

[x] #22670 (@oliver-ni)

Community contribution, requesting review from @squeed who was a reviewer on the original PR.

[x] #26476 (@julianwiedmann)

Hit trivial merge conflict, see commit message.

[ ] #26458 (@cezarygerard)

Community contribution, requesting review from @borkmann who was a reviewer on the original PR.

PRs skipped due to conflicts:

#26365 (@harsimran-pabla)
#26281 (@bprashanth)

Once this PR is merged, you can update the PR labels via:
for pr in 22670 26476 26458; do contrib/backporting/set-labels.py $pr done 1.13; done

or with
make add-labels BRANCH=v1.13 ISSUES=22670,26476,26458

/test-backport-1.13

/test-1.25-4.19


vedirect cannot open serial interface

Hello,
I was looking for a possibility to use  nodered with a victron solar charger without an additional VenusOS Raspi.
Your solution would be a easy workaround, I thought.
The problem is I have  no experience with docker. I got the 'allinone' container running, but the 'vedirect' stops with this error on the serial interface.
pi@raspberrypi:~/venus-docker-main $ ls -l /dev/serial/by-id/
insgesamt 0
lrwxrwxrwx 1 root root 13 11. Jan 19:42 usb-Prolific_Technology_Inc._USB-Serial_Controller-if00-port0 -> ../../ttyUSB0

pi@raspberrypi:~/venus-docker-main $ docker run vedirect
vedirect-dbus 3.70
000.000 ERR.serial: could not open com-port
000.000 INF.task: done in 000.000``

I hope it's just a silly mistake on my part. It would be nice if you could give me a hint.

Now it is working. somthing must have been wrong.

Got the same issue, did you do anything to fix it?

Sorry, I have no idea. Now I decided to read the values with a esp32/Arduino via VE.direct and send it direct to a InfluxDB Server.
There I don't need a Raspberry anymore. The advantage is that its much cheaper, saves power and you need no USB/RS232 Adapter.

Sorry for the late reply.
The allinone Docker image is still a work in progress, and tbh I haven't tested it myself. Since I consider that a distributed setup with containers for each service is a better approach, I think I'll just remove it from the repo.
Also, and might be related to the problem you were having, I've noticed that there was a mistake in the documentation about the VEDIRECTDEV environment variable. It should be set to the full path of the USB device inside the container, not just the file name like it was mentioned in the documentation. I've fixed it in a recent commit: https://github.com/iuriaranda/venus-docker/commit/f545f5f1dd05cae0744eefcac8abdfa19fdf7be1
@digitalwm if you're also having issues with the allinone image, I suggest you give the other individual images a try.

Thank you for for your reply, but after this problems I decided to read the VEdirect by an cheap Arduino and send the data to an InfluxDB Database. May be I will try that containers solution later.

Sorry, I have no idea. Now I decided to read the values with a esp32/Arduino via VE.direct and send it direct to a InfluxDB Server. There I don't need a Raspberry anymore. The advantage is that its much cheaper, saves power and you need no USB/RS232 Adapter.

sorry, unrelated but were you able to feed the power directly from mppt vedirect port  itself or do you use seperate power source?


Unable to close a modal containing tabs and Promise

Ionic version:  (check one with "x")
[ ] 1.x (For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[ X] 3.x
I'm submitting a ...  (check one with "x")
[ x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
Unable to close a modal containing tabs and Promise.
Expected behavior:
Steps to reproduce:

Related code:
// competition.ts => load the modal
openTeam(teamId) {
    let modal = this.modalCtrl.create(TeamTabsPage, {"id": teamId});
    modal.present();
  }

// TeamTabsPage
export class TeamTabsPage {

  team: any;

  teamTabHome = TeamPage;
  teamTabPlayers = TeamPlayersPage;
  teamTabArticles = TeamArticlesPage;
  teamTabCalendar = TeamCalendarPage;

  constructor(public navController: NavController, public navParams: NavParams, public teamProvider: TeamProvider, public viewCtrl: ViewController) {}

  ionViewCanEnter(): Promise <any> {
    return new Promise((resolve, reject) => {
      this.teamProvider.get(this.navParams.get('id'), {data: 'infos'}).subscribe(response => {
        this.team = response.team;
        resolve();
      }, error => {
        reject();
        console.log(error);
      })
    });
  }

  /**
   * Petit hack pour placer le logo de la compétition dans les tabs
   */
  ionViewDidEnter() {
    setTimeout(() => {
      let element = document.querySelectorAll('.ion-ios-team-custom');
      let parent = element[0].parentNode;
      element[0].parentNode.removeChild(element[0]);

      let img = document.createElement("img");
      img.setAttribute("class", "tab-icon-custom tab-button-icon icon icon-ios");
      img.setAttribute("src", this.team.logo);
      parent.insertBefore(img, parent.firstChild);
    }, 500)
  }

}

// TeamPage => dismiss action
closeTeam() {
    this.viewCtrl.dismiss();
  }

Other information:

ERROR Error: Uncaught (in promise): navigation stack needs at least one root page
at c (polyfills.js:3)
at c (polyfills.js:3)
at polyfills.js:3
at t.invokeTask (polyfills.js:3)
at Object.onInvokeTask (core.es5.js:4140)
at t.invokeTask (polyfills.js:3)
at r.runTask (polyfills.js:3)
at o (polyfills.js:3)
at HTMLButtonElement.invoke (polyfills.js:3)

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
li packages: (/Users/alexandre/Sites/app-mobiles/leRugbynistere/node_modules)

    @ionic/cli-utils  : 1.8.1
    ionic (Ionic CLI) : 3.8.1

global packages:

    Cordova CLI : 7.0.1

local packages:

    @ionic/app-scripts : 2.0.2
    Cordova Platforms  : ios 4.4.0
    Ionic Framework    : ionic-angular 3.5.3

System:

    Android SDK Tools : 25.2.4
    ios-deploy        : 1.9.1
    ios-sim           : 5.0.13
    Node              : v7.5.0
    npm               : 4.1.2
    OS                : macOS Sierra
    Xcode             : Xcode 8.3.3 Build version 8E3004b

Hello! Thank you for opening an issue with us! Would you be able to provide a sample application via GitHub that demonstrates the issue you are having? The sample code you provided is great, but out of context is not sufficient to duplicate the issue.
Thanks for using Ionic!

Hello !
Of course, I've created a tiny repo with the code. You can found it here : https://github.com/SquallX/ionic-demo
On index, when you click on "Article Title", you open a modal with the article content, and when you try to close the modal, ou get this error :
core.es5.js:1084 ERROR Error: Uncaught (in promise): navigation stack needs at least one root page     at c (polyfills.js:3)     at c (polyfills.js:3)     at polyfills.js:3     at t.invokeTask (polyfills.js:3)     at Object.onInvokeTask (core.es5.js:4140)     at t.invokeTask (polyfills.js:3)     at r.runTask (polyfills.js:3)     at o (polyfills.js:3)     at HTMLButtonElement.invoke (polyfills.js:3)

Excellent! Thank you. I will have a look at that.

Right now, you have the code that closes the modal existing in the child views that are shown by each tab. You can get this to work properly if you can refactor your code such that that the close that closes the modal is defined in the class for the modal itself.
That is, take this code:
  closeArticle() {
    this.viewCtrl.dismiss();
  }

and move it from tab-one.ts into modal-tabs.ts

Hi,
Thanks for your answer, when I do your manipulation, I obtain this error :
TabOnePage.html:11 ERROR TypeError: co.closeArticle is not a function     at Object.eval [as handleEvent] (TabOnePage.html:11)     at handleEvent (core.es5.js:11914)     at callWithDebugContext (core.es5.js:13206)     at Object.debugHandleEvent [as handleEvent] (core.es5.js:12794)     at dispatchEvent (core.es5.js:8814)     at core.es5.js:9406     at HTMLButtonElement.<anonymous> (platform-browser.es5.js:2687)     at t.invokeTask (polyfills.js:3)     at Object.onInvokeTask (core.es5.js:4140)     at t.invokeTask (polyfills.js:3)

OK, thx. I'll have a look.

@SquallX - right now, you have the code that does close in modal-tabs.ts (which is the correct place for it), but you have the markup that is bound to it still in the tab-one view.
<ion-header>

  <ion-navbar>
    <ion-buttons start>
      <button ion-button icon-only (click)="closeArticle()">
        <ion-icon name="close"></ion-icon>
      </button>
    </ion-buttons>
    <ion-title>{{ article.title }}</ion-title>
  </ion-navbar>

</ion-header>

You either need to refactor your design such that the header definition can be put in the modal-tabs view, or if you need to have the header defined in the child pages for any particular reason then you need to use some sort of pub-sub pattern where tab-one, tab-two, etc. emit do whatever handling they need to do and emit an event telling modal-tabs to dismiss the modal.
I suggest moving the header if possible since that is much more straight forward and only going with the pub-sub pattern if you absolutely need to.

Hi,
Thank you very much ! I understand better now.
It works perfectly =)
Thank you for your explanations.

You're welcome!! Glad to hear it is all working for you now.


Error on using feature_columns with tf.keras

System information

TensorFlow version: 1.12
Doc Link: https://github.com/tensorflow/docs/blob/b4d8d7096099c2b0a7df6a0564bf6eca8c96c4a0/site/en/tutorials/structured_data/feature_cols_keras.ipynb

Describe the documentation issue
when comment tf.enable_eager_execution(), the demo code fails.
ValueError: logits and labels must have the same shape ((32, 1) vs (32,))

This error doesn't appear in 2.0 under https://www.tensorflow.org/tutorials/structured_data/feature_columns, and we've upgraded our testing since then.  I'm closing this as fixed; thank you for bug report!


(fix) - use Record over object type for subscription operation variables

As suggested by TS and recommended ESLint rules - don't use object as a type. The object type is currently hard to use (see this issue).
This also solves the type compatibility for graphql-ws where you can use the lib in TS as you would in plain ol' JS like in #1118 .
Currently, because of this object type, you have to do this:
...
subscriptionExchange({
  forwardSubscription({ query, variables }) {
    return {
      subscribe: (sink) => {
        const dispose = wsClient.subscribe(
          {
            query,
            variables: variables as Record<string, unknown>, // 😞
          },
          sink,
        );

        return {
          unsubscribe: dispose,
        };
      },
    };
  },
}),
...

Awesome! Is this what you had in mind?

That looks great @enisdenjo, thank you!


Refactor config urls into a set

set will be reused for each configured environment

@JamesJeffryes Hold off on this for a sec, I'm going to do some branch surgery

@JamesJeffryes surgery done


frontend: AuthVisible: Validate verb before auth check

This change validates the authVerb passed into the AuthVisible component before calling getAuthorization(), which prevents components from rendering with an invalid verb.
Fixes: #2147
Testing

[X] Open a cluster in Headlamp and navigate to the Namespaces page
[X] Open frontend/src/components/namespace/CreateNamespaceButton.tsx and set the authVerb to something random (e.g. '1234')
[X] Reload the page and ensure that the create button does not show up
[X] Ensure that you see the following warning in the console:

@joaquimrocha the this.fetchAuthorization() call returns allowed: true for the invalid verb here. Maybe the rules permit all verbs without actually validating them.


refactor(rust-crane): separate main derivation config into mainDrv option

separates main derivation config to config.rust-crane.mainDrv option
remove mkDerivation options from top level
introduce config.rust-crane.source for setting the source root (this also helps differentiate mkDerivation.src since this can also hold workspace source)

I'm not sure why but the docs fail to build cause of a rust-crane.depsDrv.deps.stdenv is used but not defined. The docs of depsDrv and mainDrv build fine separately however. I tried adding stdenv to that but it did not fix anything. Afterwards I tried adding a stdenv to the deps in core/docs/default.nix and that for some reason gets rid of the error (but another error pops up). Do you have any idea what's wrong with this @DavHau ?

I'm not sure why but the docs fail to build cause of a rust-crane.depsDrv.deps.stdenv is used but not defined. The docs of depsDrv and mainDrv build fine separately however. I tried adding stdenv to that but it did not fix anything. Afterwards I tried adding a stdenv to the deps in core/docs/default.nix and that for some reason gets rid of the error (but another error pops up). Do you have any idea what's wrong with this @DavHau ?

I don't know why exactly right now. But I know that the whole dependency management is kind of annoying and I plan to refactor this soon.
It's still weird that it would still throw the error even when you set rust-crane.depsDrv.deps.stdenv directly.

separates main derivation config to config.rust-crane.mainDrv option

Why is that better? With all other modules, the top-level config defines the top-level derivation. What's special about rust that requires this change?

The reason I want it to have it's own option is just for clarity and nothing more, IMO it's more self-explanatory this way as opposed to documenting this. This would prevent people from getting confused about why their config isn't applying properly as the top-level config would only apply to the main derivation and not the dependencies derivation, potentially causing issues (and I have seen this happen first-hand so that is also why I'm more inclined towards this).

This would prevent people from getting confused

OK, seems reasonable. We can try it that way. It's just that its going to be out of sync with the other dream2nix modules. It introduces an inconsistency where some dream2nix modules have their main derivations options at the top-level and some not.
But I get the point that is is generally not clear if the top-level settings affect all dependencies or just the main one.
Maybe it will become clearer as soon as we have an overrides option, where you would override packages by name, for example:
{
  overrides.{package-name}.mkDerivation = ...;
}

package-name could either be the main package name or the name of a dependency.
Would that make it any better?

That would be great actually. This problem could would also apply when we introduce a more granular builder for Rust (or some other languages too I suppose) people would want to configure each of the package dependencies separately. That would be a good solution. For this module in specific I imagine it would export the deps derivation as ${package-name}-deps and then you could configure it via the overrides, and the main derivation in the same way.

Closing, as we plan to solve this with overrides


the stacking of buttons on confirm and upload page doesn't look good

What

The buttons in the confirm and upload button don't look good, if we could improve it something like google photos

Screenshot/Mockup/Before-After
Our App :

Google Photos :

Can we do something similar
Smartphone model

Device: [RMX 1851 Android 11]

Can I work on this?

What solution do you propose @abughalib

My simple suggestion would be 4 same-sized buttons like
[    Capture New    ] [Choose from Gallery]
[        Edit       ] [      Confirm      ]

Tried something like google photos ui as suggested by @AshAman999
Suggestions?!
https://user-images.githubusercontent.com/33085535/175482145-9cbe6b58-590e-4008-b53a-92b4961f366b.mp4

I'm not really a fan of that either, any opinion @teolemon

Somehow solved.


OOM tracking: backport #1348, #1346 and #1340 (release-1.16)

Backports #1348, #1346 and #1340.
/sig scalability
/cc jkaniuk
/cc wojtek-t

/lgtm
/approve


docs: update guidance on service accounts

Link out to cloud.google.com docs on service accounts since it is less likely to go stale.
Fixes #946

Not sure if this should be a new issue, but docs show two different ways of creating keys with gcloud format
{
  "type": "service_account",
  "project_id": "project-id",
  "private_key_id": "key-id",
  "private_key": "-----BEGIN PRIVATE KEY-----\nprivate-key\n-----END PRIVATE KEY-----\n",
  "client_email": "service-account-email",
  "client_id": "client-id",
  "auth_uri": "https://accounts.google.com/o/oauth2/auth",
  "token_uri": "https://accounts.google.com/o/oauth2/token",
  "auth_provider_x509_cert_url": "https://www.googleapis.com/oauth2/v1/certs",
  "client_x509_cert_url": "https://www.googleapis.com/robot/v1/metadata/x509/service-account-email"
}

or REST API
{
  "name": "projects/project-id/serviceAccounts/service-account-email/keys/key-id",
  "privateKeyType": "TYPE_GOOGLE_CREDENTIALS_FILE",
  "privateKeyData": "private-key",
  "validAfterTime": "date",
  "validBeforeTime": "date",
  "keyAlgorithm": "KEY_ALG_RSA_2048"
}

For this client library are both formats valid? I'm asking because I'm having auth issues with the gcloud format for this repository

Not sure if this should be a new issue, but docs show two different ways of creating keys with gcloud format
{
  "type": "service_account",
  "project_id": "project-id",
  "private_key_id": "key-id",
  "private_key": "-----BEGIN PRIVATE KEY-----\nprivate-key\n-----END PRIVATE KEY-----\n",
  "client_email": "service-account-email",
  "client_id": "client-id",
  "auth_uri": "https://accounts.google.com/o/oauth2/auth",
  "token_uri": "https://accounts.google.com/o/oauth2/token",
  "auth_provider_x509_cert_url": "https://www.googleapis.com/oauth2/v1/certs",
  "client_x509_cert_url": "https://www.googleapis.com/robot/v1/metadata/x509/service-account-email"
}

or REST API
{
  "name": "projects/project-id/serviceAccounts/service-account-email/keys/key-id",
  "privateKeyType": "TYPE_GOOGLE_CREDENTIALS_FILE",
  "privateKeyData": "private-key",
  "validAfterTime": "date",
  "validBeforeTime": "date",
  "keyAlgorithm": "KEY_ALG_RSA_2048"
}

For this client library are both formats valid? I'm asking because I'm having auth issues with the gcloud format for this repository


Disable / Enable functionality

Is there a way to avoid turning this on automatically, or selectively enabling / disabling it?
I see the Start-HumpCompletion and Stop-HumpCompletion, but they seem to only set the value of $Global:HumpCompletionEnabled (a variable which is also set to $true by default at the end of loading the module), but I don't see that variable referenced elsewhere.
Because this must work by replacing TabExpansion2 I'm wondering if there's a way I can better control that this gets enabled, and a way to disable it explicitly.
This is of most concern because of module auto-loading, which can be triggered by many things, and because the ideal of way of installing this (Install-Module) is going to put it in an auto-loading capable place.
It's of course possible to Save-Module and put it somewhere locally, but this means breaking out of using the standard way of managing modules.
For me, ideally, this functionality is disabled by default, and you must explicitly call Start-HumpCompletion to replace TabExpansion2. This can then go into the appropriate $PROFILE based on who wants it to be loaded automatically in their session.
Advantages

Proper module management (the module can be kept global and allowed to use for anyone who wants to)
Can be loaded in profiles giving control over different environments, at different scopes (admin can enabled everywhere, individuals can enable / disable at will, etc.).
Can be avoided in scheduled tasks and other invocations regardless of profile by invoking powershell.exe -NoProfile
Helper functions for adding it to a profile can be included (a la posh-git's Add-PoshGitToProfile)

Looking forward to hearing your thoughts.

@briantist - sorry for the delay in responding!
Last time I tested this, Install-Module wasn't sufficient to get the posh-HumpCompletion behaviour, I had to also add it to my $PROFILE. Are you seeing a different behaviour?


[1.x] Refresh OpenSearch nodes version in cluster state after upgrade (#865)

Description
Backport PR #865 to 1.x branch.
Fix:

As soon as OpenSearch node becomes master, it refreshes the nodes version in cluster state by comparing against their channel version. Right now, it doesn't wait for all Elasticsearch nodes to leave the cluster before refreshing the version.
There was bug in DiscoveryNode BWC serialization logic where any ES version < 7.10.2 was getting over-written with 7.10.2

Issues Resolved
#827
Check List

[ ] New functionality includes testing.

[x] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

start gradle check


Dropped messages can cause CloseChannel Objectives to stall out for a participant

If the right messages get dropped we can run into a scenario where one participant has a completed CloseChannel objective and one participant has a pending CloseChannelObjective.
My current theory is that this is due to messages containing final states being dropped. This could result in one participant having both signed final states and the other only having one.
I think the reason we only run into this with CloseChannel is that it is unique in being "pre-approved".  That means the other participant doesn't have to call ApproveObjective on the objective, meaning ensureObjective does not get called for one of the participants.

This only seems to happen if the participant B is the one to instigate the close. (Assuming the channel is opened then immediately closed). Perhaps due to turn taking?

These are local log entries of the test failing.
with-peers.log
with-peers-pretty.log

Based on the logs  I think this is what is happening

B calls close
CloseChannel objective gets sent to A
A sends final state to B
B sends final state to A but the message gets dropped
B has now both final signed states and completes the objective (no defunding happens since we are using "fake" funding)
Since A never called something to trigger ensureObjective A never attempts to get the state it is missing.

To resolve this I think we can listen for the objectiveStarted event and call ensureObjective on it.  Longer-term the wallet should look at any new objectives from the engine.pushMessage and if there are CloseChannel objectives call ensureObjective.


Netty HttpClient worker threads are not configurable

This issue belongs with the metamx/http-client project:
The Netty client worker pool uses a non-configurable Runtime.getRuntime().availableProcessors() * 2 threads. This can create an excessive number of threads on machines with a high number of cores that run a large number of Druid processes (e.g. MMs with lots of peon processes).
https://github.com/metamx/http-client/blob/master/src/main/java/com/metamx/http/client/HttpClientInit.java#L119
It would be good if this could be overridden through configuration. DruidHttpClientConfig has a numMaxThreads field (set from 'druid.global.http.numMaxThreads') which is not currently being honored and probably makes sense to use here.

ugh, yeah... that needs to be fixed by making it configurable in http-client

http-client patch in https://github.com/metamx/http-client/pull/25

http-client 1.0.5 includes that patch.

@dclim could this issue be closed?

@leventov yes, thank you


GitHub user also added when add new GitLab user under 'approved list'

Summary
GitHub user also added when GitLab user under 'approved list'
Existing GitHub user details also updated when add GitLab user
Background

Go to the corporate CLA console
Add a GitLab user under the approval list

Expected behavior
Only GitLab users should be added to in list
Screenshots

This is corner case so we will fix this in next release.

@thakurveerendras this was an issue with the same date_added value which has been resolved in in #3750 . Kind verify

The issue is not reproducible as reported on the dev site, So closing this ticket
Refer to the screenshot

Closing this ticket based on last comment.


cmake: west: Fix paths on windows

Fixes an issue on windows whereby the slashes are treated as escape characters which could cause some executions to fail.

WEST_PYTHON should not be used for python invocation. Python invocation in Zephyr should be done using PYTHON_EXECUTABLE alternatively Python3_EXECUTABLE.
So how come the backslash issue becomes an issue in the first place ? Besides the use of WEST_PYTHON as Python lookup, then a second use is where the real location of the WEST_PYTHON is compared to the real PYTHON_EXECUTABLE, in order to see if those points at the same location. But this PR adjust the WEST_PYTHON after this comparison, hence it indicates that someone is using WEST_PYTHON when in fact they should be using PYTHON_EXECUTABLE.

To ensure proper passing of WEST_PYTHON to CMake and real path comparison, it should probably be considered to ensure a posix-path is used here: https://github.com/zephyrproject-rtos/zephyr/blob/c48422f52360a6026ef64e8c13387dbd09a9c9aa/scripts/west_commands/build.py#L500


Dye not working on Silent Gear weapons/items

The sword doesn't have dye on it either.

I have the same problem. Try going to the fancy dyes github page and reporting there

I have the same problem with truly modular. Report on the silent gear github page The developer might take a while to respond here

Install the trimmable tools mod.

going to say this probably won't happen. dyes are just enchantment glints, and there's a reason why Silent Gear tools don't have enchantment glints enabled by default. leaving this open though in case I change my mind


AGConnectApp named [Default] already exits

Am getting following exception.
AGConnectApp named [Default] already exits
What does it mean?

This issue may be caused by repeated creation of AGConnectApp.
similar question for you reference, but it's in Chinese, you can translate it：https://developer.huawei.com/consumer/cn/forum/topic/0201691801128470750?fid=18

But am creating AGConnectApp to access my account,
Please check below image.

Can I confirm what changes you made in the project after you downloaded the demo?

I have Just added AGConnectAdmin SDK to my solution and added AGConnectAdmin Reference.
And try to send notification from my project using above method.

This error may be caused by the repeated creation of AGConnectApp.

This is the code for throwing an exception because the AGConnectApp already exists. You may have created AGConnectApp in other places besides the code in the screenshot which you provided. You can search for the create keyword to locate the fault.

Am getting Object reference not set to an instance of the object exception when used below code.
`await AGConnectMessaging.DefaultInstance.SendAsync(new Message()
             {

                 Notification = new Notification()

                 {

                     Title = "Test Message",

                     Body = "Detail Message",

                 },

                 Android = new AndroidConfig()

                 {

                     Notification = new AndroidNotification()

                     {

                         Title = "Notification from .NET",

                         Body = "Hello world!",

                         ClickAction = ClickAction.OpenApp()

                     }

                 },

                 Token = new List<string>() { "IQAAAACy0uRmAAAvsZe3QA42M8KS-3Dg6Dyqubcc8cTUIj-PW7tV0z82aiuXXNoNR_n_XV9HZ0BUa9VtkAi-OUNGxXN_rLYNbhgad5mgfBvdirxssQ" }

             });`

@Xuejiao-Shi
Am using above code in ASP.net Webapi,
Do i need to initialise below code some where else ?
var ag = AGConnectApp.Create(new AppOptions()

            {

                ClientId = "",

                ClientSecret = ""

            });

@Xuejiao-Shi
I fixed the problem by writing AGConnectApp.Create code in start of the application and use DefaultInstance.
Now am able to send notifications from my C# project.
Thanks for your valuable time and help.