feat: [M3-7445] - Public IP Addresses Tooltip and LISH Display

Description 📝
Enable Public IP Addresses for VPC-only Linodes and updated the Tooltip text.
These changes follow the discussion in which it was acknowledged that LISH does not use a Public IP Address and this was not known at the time the original design was created.
A VPC-only Linode is a Linode that has at least one config interface with primary set to true and purpose vpc and no ipv4.nat_1_1 value.
Changes  🔄

Update Public IP Addresses tooltip copy to "The noted Public IP Addresses are provisionally reserved but not assigned to the network interfaces in this configuration profile. Learn more"
Tooltip Learn more URL was not updated to the one specified in the mockup (https://www.linode.com/docs/) since our docs are now found at (https://techdocs.akamai.com/cloud-computing/docs/)
Update the LISH Console via SSH terminal command text to be displayed as enabled.

Target release date 🗓
10/14/2024
Preview 📷

Before
After

How to test 🧪
Prerequisites
(How to setup test environment)

Ensure your account has vpc customer tags
Have a VPC-only Linode

You can either create a Linode and add a VPC or edit a Linode's config so that the primary interface is VPC. (make sure the assign a public IPv4 address checkbox is unchecked)

Verification steps
(How to verify changes)

For VPC-only Linode

The Public IP Addresses and Access sections in the Linode's details page should be disabled and there should be a tooltip

For non-VPC Linodes

The Public IP Addresses and Access sections in the Linode's details page should be disabled and there should not be a tooltip.

As an Author I have considered 🤔
Check all that apply

[x] 👀 Doing a self review
[ ] ❔ Our contribution guidelines
[ ] 🤏 Splitting feature into small PRs
[x] ➕ Adding a changeset
[x] 🧪 Providing/Improving test coverage
[x] 🔐 Removing all sensitive information from the code and PR description
[ ] 🚩 Using a feature flag to protect the release
[x] 👣 Providing comprehensive reproduction steps
[ ] 📑 Providing or updating our documentation
[ ] 🕛 Scheduling a pair reviewing session
[ ] 📱 Providing mobile support
[ ] ♿  Providing accessibility support

In creating a linode to test this PR, I noticed a bug in the new Linode Create flow -- if the linode is assigned to a VPC during creation, that VPC interface should be marked as primary. See the relevant Linode Create v1 code:

https://github.com/linode/manager/blob/b1e937085586ccb36929552730b0b2f120fada32/packages/manager/src/features/Linodes/LinodesCreate/LinodeCreate.tsx#L335

That doesn't seem to be happening in the new flow as I had to resave the config after creation to trigger that setting

I can create a separate bug ticket to address this issue as I can replicate it myself.


Fix typo in scheduler creation in cljs

I believe there should be :queue instead of :trampoline as valid clause.

Nice, you are right, but a backward compatibility should be added with a clear deprecation path documented on the docstring. Add that please and I'm gladly merge this PR.

Please, can you look at the last commit if this is enough?
Thank you very much!

Awesome! Thanks!


Release 0.2.0

Holding off on this until #35 is done

We should be good to move forward with this, whenever your ready


send HTML and CSS as chunks rather than encoding them as JSON

Mitch Cooper, [May 1, 2017, 11:34 PM]:
hm

im thnkin there needs to be a better way to just send multi-line data over this transport other than encoding it into json

needs to be something like blank line means that a chunk of text is coming or something like that

Matt Barksdale, [May 1, 2017, 11:35 PM]:
tru

Mitch Cooper, [May 1, 2017, 11:47 PM]:
ok so blank line means a chunk is coming

line after blank line is the id for it

Matt Barksdale, [May 1, 2017, 11:47 PM]:
nice

Mitch Cooper, [May 1, 2017, 11:47 PM]:
how do i terminate it

???

Matt Barksdale, [May 1, 2017, 11:48 PM]:
2 blank lines

Mitch Cooper, [May 1, 2017, 11:48 PM]:
wat if the content has 2 blank line

o

Matt Barksdale, [May 1, 2017, 11:48 PM]:
what is something the content cant have

Mitch Cooper, [May 1, 2017, 11:48 PM]:
u could specify the byte length in the intro

Matt Barksdale, [May 1, 2017, 11:48 PM]:
ya tru

Mitch Cooper, [May 1, 2017, 11:48 PM]:
ok so like

<blank line>

css 239483989

<css>

<blank line>

html<PHONE_NUMBER>2938

<html>

Matt Barksdale, [May 1, 2017, 11:49 PM]:
tru

nvm


Merge #13578: [depends, zmq, doc] upgrade zeromq to 4.2.5 and avoid deprecated zeromq api functions

f1bd03eb013b96ff040a8f835e4137fbd2a38cda [depends, zmq, doc] upgrade zeromq to 4.2.5 and avoid deprecated zeromq api functions (mruddy)
Pull request description:
Upgrade the ZeroMQ dependency from version 4.2.3 to the latest stable version 4.2.5.
This PR Follows the lead of https://github.com/bitcoin/bitcoin/pull/11986.
I upgraded both patch files to correspond to the version 4.2.5 libzmq files.
I assume doing so is still necessary and correct.
Without updating the patch line numbers, things appear to work, but you get extra log messages while building depends because things don't exactly match, e.g.:
/bitcoin/depends> make zeromq
Extracting zeromq...
/bitcoin/depends/sources/zeromq-4.2.5.tar.gz: OK
Preprocessing zeromq...
patching file src/windows.hpp
Hunk #1 succeeded at 58 (offset 3 lines).
patching file src/thread.cpp
Hunk #1 succeeded at 307 with fuzz 2 (offset 87 lines).
Hunk #2 succeeded at 323 with fuzz 2 (offset 90 lines).

Updating the patches seemed cleaner, so I did it. Note that libzmq had some whitespace changes, so that's why the updated patches do too.
More info: https://github.com/zeromq/libzmq/releases/tag/v4.2.5
tags: libzmq, zmq, 0mq
Tree-SHA512: 78659dd276b5311e40634b1bbebb802ddd6b69662ba3c84995ef1e3795c49a78b1635112c7fd72a405ea36e2cc3bdeb84e6d00d4e491a349bba1dafff50e2fa5

utACK for squash merge contingent on below being fine
LGTM, but I am mildly concerned about why we don't have the patches that bitcoin has. @UdjinM6 can you maybe explain to me why we don't have those patches? Should we have them, or is it fine?

They have been dropped in https://github.com/dashpay/dash/commit/7391aad4b7a36634042707cb118cdeea2a1444c3 (another backport from Bitcoin).


Add tests

nemtom

All tasks ready to be merged, this pull request will stay open as it is a project made for the 4th lab of SzoftEng at BME 2024.


GCP cli - is this feature missing?

Hi everyone, I've seen that among the features are present both aws-cli and azure-cli, but there is no gcloud-cli.
I want to ask if such a feature exists under a different name or needs development.
Thank you in advance.

Thanks for the question @WilliamBonvini! As I believe @joshspicer addressed your question, I'll close this issue, but please let us know if you have further questions.

yes thank you, it solves my problem indeed :)


PR: Add option to use Jedi the IPython console + warning on greedy completer

This adds an  option for Jedi Code completion in the ipython shell.
It also adds a warning text for the Greedy completer, to inform users, that the current version of the Greedy completer in ipython needs an extra space to split commands.
Related Spyder-ide issues: #6803 #6700
Related Ipython Console Issues:
https://github.com/ipython/ipython/issues?utf8=✓&q=is%3Aissue+is%3Aopen+greedy+completer

Hello @hofingermarkus! Thanks for updating the PR.

In the file spyder/plugins/ipythonconsole.py, following are the PEP8 issues :

Line 482:80: E501 line too long (81 > 79 characters)
Line 484:33: E128 continuation line under-indented for visual indent
Line 484:80: E501 line too long (88 > 79 characters)
Line 485:33: E128 continuation line under-indented for visual indent
Line 485:80: E501 line too long (87 > 79 characters)
Line 486:33: E128 continuation line under-indented for visual indent
Line 488:80: E501 line too long (99 > 79 characters)
Line 489:80: E501 line too long (88 > 79 characters)
Line 490:33: E127 continuation line over-indented for visual indent
Line 492:1: W293 blank line contains whitespace
Line 497:1: W293 blank line contains whitespace
Line 508:80: E501 line too long (84 > 79 characters)
Line 512:80: E501 line too long (97 > 79 characters)
Line 600:80: E501 line too long (94 > 79 characters)

In the file spyder/utils/ipython/kernelspec.py, following are the PEP8 issues :

Line 127:80: E501 line too long (80 > 79 characters)

Updated according to change requests.
Added new updated Screenshot to PR


Add missing tests for TaskSeq.zip, mapi, map, iter, exactlyOne, tryExactlyOne and async variants

This adds missing tests for the following functions
TaskSeq.zip
TaskSeq.exactlyOne
TaskSeq.tryExactlyOne
TaskSeq.map*
TaskSeq.mapi*
TaskSeq.mapAsync
TaskSeq.mapiAsync
TaskSeq.iter*
TaskSeq.iteri*
TaskSeq.iterAsync
TaskSeq.iteriAsync
TaskSeq.empty*
TaskSeq.collect*
TaskSeq.collectSeq*
TaskSeq.collectAsync
TaskSeq.collectSeqAsync
TaskSeq.ofResizeArray

* these had existing tests that I've improved a bit on, most notably, adding mutable-data tests

Turned out that TaskSeq.zip had a nasty little bug. Well, that's why we have tests. Fixed now.
Also, this made me think how zip should behave for lazy sequences. See #32 for a follow-up.

💚 LGTM: we reached 100% coverage!!!!


poetry add --extras security requests fails with ValueError

[ ] ~I am on the latest Poetry version.~

[x] I am on the master branch of Poetry.

[x] It can be reproduced on the latest beta version of Poetry.

[x] I have searched the issues of this repo and believe that this is not a duplicate.

[x] If an exception occurs when executing a command, I executed it again in debug mode (-vvv option).

OS version and name: any

Poetry version: 1.0.0a0 — 1.0.0b1

~Link of a Gist with the contents of your~ pyproject.toml file: this bug could be reproduced with any pyproject.toml, so you can just generate a new one using poetry init.

Issue
Using poetry from master/develop branches or any recent alpha/beta release, it's not possible to add the requests package with the security extra.
$ poetry -V
Poetry version 1.0.0b1
$ poetry add --extras security requests -v
Using virtualenv: /home/mim/.cache/pypoetry/virtualenvs/test-poetry-requests-security-XPJu19SV-py3.7
Using version ^2.22.0 for requests

Updating dependencies
Resolving dependencies... (0.1s)

[ValueError]
Array has mixed types elements

Traceback (most recent call last):
  File "/home/mim/.poetry/lib/poetry/_vendor/py3.7/clikit/console_application.py", line 132, in run
    status_code = command.handle(parsed_args, io)
  File "/home/mim/.poetry/lib/poetry/_vendor/py3.7/clikit/api/command/command.py", line 119, in handle
    status_code = self._do_handle(args, io)
  File "/home/mim/.poetry/lib/poetry/_vendor/py3.7/clikit/api/command/command.py", line 167, in _do_handle
    return getattr(handler, handler_method)(args, io, self)
  File "/home/mim/.poetry/lib/poetry/_vendor/py3.7/cleo/commands/command.py", line 92, in wrap_handle
    return self.handle()
  File "/home/mim/.poetry/lib/poetry/console/commands/add.py", line 149, in handle
    status = installer.run()
  File "/home/mim/.poetry/lib/poetry/installation/installer.py", line 74, in run
    self._do_install(local_repo)
  File "/home/mim/.poetry/lib/poetry/installation/installer.py", line 189, in _do_install
    self._write_lock_file(local_repo)
  File "/home/mim/.poetry/lib/poetry/installation/installer.py", line 290, in _write_lock_file
    updated_lock = self._locker.set_lock_data(self._package, repo.packages)
  File "/home/mim/.poetry/lib/poetry/packages/locker.py", line 149, in set_lock_data
    lock['package'] = packages
  File "/home/mim/.poetry/lib/poetry/_vendor/py3.7/tomlkit/container.py", line 551, in __setitem__
    self.append(key, value)
  File "/home/mim/.poetry/lib/poetry/_vendor/py3.7/tomlkit/container.py", line 91, in append
    item = _item(item)
  File "/home/mim/.poetry/lib/poetry/_vendor/py3.7/tomlkit/items.py", line 62, in item
    i = item(_v)
  File "/home/mim/.poetry/lib/poetry/_vendor/py3.7/tomlkit/items.py", line 46, in item
    val[k] = item(v, _parent=val)
  File "/home/mim/.poetry/lib/poetry/_vendor/py3.7/tomlkit/items.py", line 70, in item
    a.append(v)
  File "/home/mim/.poetry/lib/poetry/_vendor/py3.7/tomlkit/items.py", line 778, in append
    raise ValueError('Array has mixed types elements')

Hi @sdispater,
I've already submitted PR with a fix and an explanation of this bug.
It is #1439. Did you see it?

I see you recently commited af71f6c0e585ac20a029e1eccb6501898adb6395, so I'm quite confused now :confused:
Why did you completely ignore the proposed fix?

@sdispater
Even worse, af71f6c0e585ac20a029e1eccb6501898adb6395 doesn't cover all the cases due the code causing this issue is still there.
I've updated #1439, so could you please take a look?
Thanks!

@Jamim Sorry for the confusion but while I saw the issue I did not see your related PR and I made the commit just before your comment mentionin your PR. I just commented and requested changes on your PR.


Flex items and min-width:0

What do you think of adding min-width:0 to flex items?
It helps to avoid unexpected behaviour of flex items. flex-items-and-min-width-0

Too intrusive. This lib can't make that decision by itself and it's up to the developer to add something like that.


Don't try to divide by zero if CPU scaling frequency min and max are equal

I just built and installed powerkit for the first time. When I tried to open the config GUI, it crashed with a floating point exception. I tracked down the issue to line 430 in powerkit_cpu.cpp, which calculates the current CPU frequency as a percentage within the configured scaling range. My min and max values are equal, so this leads to a division by zero. This PR handles this corner case (I interpreted it as 100% progress).

Yeah, the new UI is not finished, so there might be some issues. Thanks


Fixes for Cake 3.6 deprecations

Coverage decreased (-53.3%) to 46.667% when pulling 385926f637101e8da9bf4706ae4f64c4a11ef5e0 on jorisvaesen:master into 3f225b84dceb123d49255d34cc97d4ead7dd3f8d on andrej-griniuk:master.

Thank you!


mips64: Fixed scalar optimization bug in dzsum

Oops, sorry, did not realize we were both working on this after my blundering around last night.


[Bug] Error handling for no buying power

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
You get an error and the app breaks when you don't have enough buying power
Expected Behavior
You should get a message or value indicating you don't have enough buying power (without breaking the app)
SDK Version I encountered this issue in
Tested with alpaca-py 0.6.1
Steps To Reproduce
1. Try to buy more than you are allowed

Filled out the Steps to Reproduce section?

[X] I have entered valid steps to reproduce my issue or have attached a minimally reproducible case in code that shows my issue happening; and understand that without this my issue will be flagged as invalid and closed after 30 days.

Anything else?
No response

Hi @sshcli ,
I'm not able to reproduce your issue.
When I try to submit a similar order, I receive
APIError: {"buying_power":"176531.28","code":40310000,"cost_basis":"48640000","message":"insufficient buying power"}
Can you try your request again with 0.7.0?

Hi @haxdds
False alarm.
I can confirm that it is working as expected in both 0.6.1 and 0.7.0
Maybe it was something else causing the error or my mistake.
I'm closing this one as solved.
Thanks


Make Google.Cloud.Diagnostics.*.Snippets runable test like other library snippets

Partially addressed via #2192. Asp.Net Snippets pending.

Partially addressed via #2374.
Pending:

Asp.Net MVC Error Reporting Snippets

Asp.Net MVC and Web Api Trace Snippets.

Asp.Net Web Api Trace Snippets cannot be made runnable with OWIN hosted Web Apis because the Asp.Net tracer relies heavily on HttpContext.Current and global.asax events, none of which are available through OWIN. In fact our Tracer won't work for self hosted Asp.Net Web Apis, which might not be a huge problem in itself specially after .Net Core.
So, we still have pending:

Asp.Net MVC Error Reporting Snippets

Asp.Net MVC and Web Api Trace Snippets

This is blocked by #2297 since we'll need to set up IIS hosted applications for making the pending snippets runnable.

Closing this for now as we are reevaluating our Google.Cloud.Diagnostics.* library strategy.
See:

Google.Cloud.Diagnostics.AspNet version history.
Google.Cloud.Diagnostics.AspNetCore version history


Error Running Pull #25 Version Files On Mac

I just downloaded and tried the updated merge.py and patch.py files from the merged pull #25 in a simple python diffusers CLI pipeline.  Tomesd worked fine on my Mac with the prior versions.  The new versions give the traceback below.  Would it help you troubleshoot if I swapped out one file at a time, or are they dependent on changes in each other to work at all?
Traceback (most recent call last):
File "/Users/jrittvo/miniconda3/envs/python_playground/diffusers/gen-sd21.py", line 25, in
image = pipe(args.prompt, generator=generator, num_inference_steps=24, guidance_scale=7.5, height=768, width=768).images[0]
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/diffusers/pipelines/stable_diffusion/pipeline_stable_diffusion.py", line 676, in call
noise_pred = self.unet(
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1538, in _call_impl
result = forward_call(*args, **kwargs)
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/diffusers/models/unet_2d_condition.py", line 685, in forward
sample, res_samples = downsample_block(
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/diffusers/models/unet_2d_blocks.py", line 867, in forward
hidden_states = attn(
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/diffusers/models/transformer_2d.py", line 265, in forward
hidden_states = block(
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/tomesd/patch.py", line 105, in forward
m_a, m_c, m_m, u_a, u_c, u_m = compute_merge(hidden_states, self._tome_info)
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/tomesd/patch.py", line 38, in compute_merge
args["generator"] = init_generator(x.device)
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/tomesd/patch.py", line 19, in init_generator
return torch.Generator(device=device).set_state(torch.mps.get_rng_state())
AttributeError: module 'torch' has no attribute 'mps'

Huh, that mps is there specifically for macs, but maybe it requires a newer torch version. What torch version are you on? I can probably just default it to use cpu if mps isn't available.

Was able to grab a hold of a friend with an M1 mac and I think I found a solution to the issue.
Does the most recent commit work for you?

I set an environment variable in my python diffusers pipeline, that is necessary when I run the full Automatic1111 wrbui:
export PYTORCH_ENABLE_MPS_FALLBACK=1
And this is the full terminal output on a successful run:
(python_playground) jrittvo@M1PRO diffusers % python gen-sd21.py --prompt "photograph of a mouse" --seed "123" --save "images/mouse123.jpg"
0%|                                                    | 0/24 [00:00<?, ?it/s]/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/tomesd/merge.py:66: UserWarning: MPS: no support for int64 min/max ops, casting it to int32 (Triggered internally at /Users/runner/work/pytorch/pytorch/pytorch/aten/src/ATen/native/mps/operations/Sort.mm:39.)
rand_idx = idx_buffer.reshape(1, -1, 1).argsort(dim=1)
/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/tomesd/merge.py:104: UserWarning: The operator 'aten::scatter_reduce.two_out' is not currently supported on the MPS backend and will fall back to run on the CPU. This may have performance implications. (Triggered internally at /Users/runner/work/pytorch/pytorch/pytorch/aten/src/ATen/mps/MPSFallback.mm:11.)
dst = dst.scatter_reduce(-2, dst_idx.expand(n, r, c), src, reduce=mode)
100%|███████████████████████████████████████████| 24/24 [00:43<00:00,  1.82s/it]
Changing ONLY the merge.py and patch.py files, I get the traceback in my first post above.  You may be right that it is not actually using mps for some or all of the pipeline, but the prior tomesd file versions don't trigger any traceback.
If I try without the environment variable set, the new versions give the same traceback.  The old versions give this traceback:
NotImplementedError: The operator 'aten::scatter_reduce.two_out' is not currently implemented for the MPS device. If you want this op to be added in priority during the prototype phase of this feature, please comment on https://github.com/pytorch/pytorch/issues/77764. As a temporary fix, you can set the environment variable PYTORCH_ENABLE_MPS_FALLBACK=1 to use the CPU as a fallback for this op. WARNING: this will be slower than running natively on MPS.
I'm using torch 2.0.0.dist.  I have torch 2.1.0.dev20230407.dist on my Automatic1111.  I ca try swapping that newer package over to my python diffusers pipeline if that would be useful to you.  I also have a special torch build that brkirch bundles with his experimental Mac build of Automatic1111.  I am going to try that one in a bit, because his pieces have fixes for Macs that have not been upstreamed yet.

Sit tight for a few minutes and I'll try them.

Nope on the new new build.  I get the traceback below with both PYTORCH_ENABLE_MPS_FALLBACK=1 and PYTORCH_ENABLE_MPS_FALLBACK=0.  I assume that =0 is taking it in and out of the mix??
(python_playground) jrittvo@M1PRO diffusers % python gen-sd21.py --prompt "photograph of a mouse" --seed "123" --save "images/mouse123.jpg"
0%|                                                    | 0/24 [00:00<?, ?it/s]
Traceback (most recent call last):
File "/Users/jrittvo/miniconda3/envs/python_playground/diffusers/gen-sd21.py", line 25, in
image = pipe(args.prompt, generator=generator, num_inference_steps=24, guidance_scale=7.5, height=768, width=768).images[0]
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/diffusers/pipelines/stable_diffusion/pipeline_stable_diffusion.py", line 676, in call
noise_pred = self.unet(
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1538, in _call_impl
result = forward_call(*args, **kwargs)
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/diffusers/models/unet_2d_condition.py", line 685, in forward
sample, res_samples = downsample_block(
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/diffusers/models/unet_2d_blocks.py", line 867, in forward
hidden_states = attn(
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/diffusers/models/transformer_2d.py", line 265, in forward
hidden_states = block(
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(args, **kwargs)
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/tomesd/patch.py", line 97, in forward
m_a, m_c, m_m, u_a, u_c, u_m = compute_merge(hidden_states, self._tome_info)
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/tomesd/patch.py", line 33, in compute_merge
m, u = merge.bipartite_soft_matching_random2d(x, w, h, args["sx"], args["sy"], r,
File "/Users/jrittvo/miniconda3/envs/python_playground/lib/python3.10/site-packages/tomesd/merge.py", line 52, in bipartite_soft_matching_random2d
rand_idx = torch.randint(sysx, size=(hsy, wsx, 1), device=metric.device, generator=generator)
RuntimeError: Expected a 'mps' device type for generator but found 'cpu'

Thanks for all the information. I'll have direct access to an M1 mac in a week or two, and then I can create a natively compatible version that doesn't require PYTORCH_ENABLE_MPS_FALLBACK=1.
Otherwise, looks like the fix did make some progress. Can you try the latest commit? I think that should be all that's needed.

Yes, I'll grab it in a few minutes and let you know.  I tried the brkirch build of torch and it didn't help with our without MPS_FALLBACK enabled.  I do think that is something he has been working on though.

Your latest set works again (with MPS_FALLBACK=1).
Do you you know who brkirch is?  He seems to be an upper level developer working with Automatic1111 on Mac issues.  He has an experimantal build that has had tomesd in it for a few weeks now.  This is his repo: https://github.com/brkirch/stable-diffusion-webui/actions.  If you look at the "Actions" there, you can see what he has been working on. I think he has spent a fair amount of time recently on tomesd and Macs.

Oh huh, no I was not aware of it. I'll see if I can reach out once I start adding native support.
Thanks for all the testing! Closing the issue, as it seems to work now.

Yes, issue resolved.  Glad to be able to help in a small way.


docs: update aws_health_to_slack README for cloud

Restructure README for Better User Experience
Changes

Reorganized deployment sections to prioritize cloud usage
Separated and clarified configuration steps for different deployment methods (cloud/UI vs self-hosted)
Added clear instructions for modifying variables in both environments
Improved documentation structure and readability

Why
The previous README mixed cloud and self-hosted instructions, making it confusing for users. This restructuring:

Makes it easier for cloud users to quickly find relevant information
Keeps detailed self-hosted instructions accessible but separate
Clarifies how to configure variables in different environments
Provides a more logical flow of information

Looks good


feat: add marquee

Coverage remained the same at 32.377% when pulling e1c6582acffb4f96edbab594f71bf93e27ca3bd4 on Yang03:dev into 4586d41bd52dfc2423bf029a86578925ad427a35 on ZhonganTechENG:dev.

已线下沟通修改点


DefaultAzureCredentials fails with "Access denied" when connecting to key vault

Package Name: azure-identity
Package Version: 1.5.0
Operating System: Windows 10
Python Version: 3.8.2

Describe the bug
Visual studio code credentials not working when trying to connect to key vault (and possibly other types of resources?). If I disable the VSCode credentials, it probably uses Azure CLI credentials instead, which works fine. Note that DefaultAzureCredential used to work, but stopped for some reason (might have stopped working after I installed MS Teams?). I'm very certain that I have logged into the same account in VSCode, Azure CLI and in Windows.
To Reproduce
Setup a key vault and give yourself access. Add a secret to the key vault and run the following code
# %%
from azure.cosmos import CosmosClient
from azure.identity import DefaultAzureCredential
from azure.keyvault.secrets import SecretClient

#%%

# Fails

credential = DefaultAzureCredential()
client = SecretClient(vault_url="https://KEY_VAULT_NAME.vault.azure.net/", credential=credential)
client.get_secret("DbConnectionString").value

# %%

# Works fine

credential = DefaultAzureCredential(exclude_visual_studio_code_credential=True)
client = SecretClient(vault_url="https://KEY_VAULT_NAME.vault.azure.net/", credential=credential)
client.get_secret("DbConnectionString").value

Log output
EnvironmentCredential.get_token failed: EnvironmentCredential authentication unavailable. Environment variables are not fully configured.
ManagedIdentityCredential.get_token failed: ManagedIdentityCredential authentication unavailable, no managed identity endpoint found.
SharedTokenCacheCredential.get_token failed: SharedTokenCacheCredential authentication unavailable. No accounts were found in the cache.

Error (IDs removed)
(Forbidden) Access denied to first party service.
Caller: name=from-infra;tid=;appid=;iss=https://sts.windows.net//
Vault: ;location=

Expected behavior
The script should be able to pull the secret and print the value.
Additional context
Not sure if this is really belongs here, but I don't really know where else to direct this.

"Access denied to first party service" typically occurs with MSA accounts, i.e. user accounts from @hotmail.com, @live.com, @outlook.com. Are you signed in to VS Code with one of those accounts?

I am using vscode with outlook account and get the same error : HttpResponseError: (Forbidden) Access denied to first party service.
credential = DefaultAzureCredential()
print(credential)
client = SecretClient(vault_url=KVUri, credential=credential)
svc_pr_password = client.get_secret(secretName)

what is vscode_tenant_id ? where to find ?

vscode_tenant_id is the ID of the tenant in which you want to authenticate the user signed in to Visual Studio Code. In azure-identity 1.6.0, the credential defaults to authenticating in the user's home tenant. In azure-identity 1.7.0b1 and above, it defaults to authenticating in the tenant configured in VS Code (the "Azure:Tenant" setting, which also defaults to the user's home tenant).
You only need to specify a tenant ID when you want to authenticate in a different tenant. The value is typically a UUID like b0ae42e0-496e-4216-8811-e64c72a29c59 (which is just a random UUID, not a real tenant's ID). The correct value depends on the resource(s) you want to access. For example, if you want to access a Key Vault in tenant "A", you should set vscode_tenant_id="A". There are several ways to get a resource's tenant ID. For example, you can use the Azure CLI (e.g. for a Key Vault you want az keyvault show) or the Azure Portal (where it's sometimes called a "directory ID").


Fix iteration of dupes

If you have duplicate Clips in a Track (identical, but not instances of the same object) then range_of_child() and other methods don't work as expected. This is because we were using index() to find objects in a Composition, but index() uses the == operator instead of is.
This PR introduces index_of_child() and fixes a range_of_child() and other related methods to work properly when duplicate clips are present.
Addresses #222

I think this is related to #228.  I suspect that making these types reference types instead of value types will map better to the way people expect them to behave and clarify a lot of these kinds of things.

We will want to revisit this, but we're hitting a case where we need to integrate this now to cover some bugs.


Add BITFIELD_RO command : Using generic method for BITFIELD and BITFIELD_RO

This PR implements https://github.com/DiceDB/dice/issues/640.
It allows BITFIELD_RO and BITFIELD to use a common generic method, and uses a boolean flag to differentiate between these two commands.
This also aims to replace PR https://github.com/DiceDB/dice/pull/956, post reviews.

Hey @apoorvyadav1111 @JyotinderSingh @lucifercr07 , requesting you to review this PR at your convenience.

Not required. Ignore.


JsonInputFormatter enums

Is it possible with the utf8json JsonInputFormatter, to have it accept both enums as ints and enums as strings?  I don't see any options on the JsonInputFormatter class.  On swagger, I'm converting enums to strings.. But, in microservice communication, I want it to be able to use enums as ints..

Yes, i agree it would be good to see something like the Jil:
    [JilDirective(TreatEnumerationAs = typeof(int))]

That can be done while setting up the options.. I meant to handle both like newtonsoft does.

Currently EnumFormatter accepts both.
var apple1 = JsonSerializer.Deserialize<Fruit>("\"Apple\"");
var apple2 = JsonSerializer.Deserialize<Fruit>("1");

Console.WriteLine(apple1);
Console.WriteLine(apple2);

// Serialize Option
var v1 = JsonSerializer.Serialize(Fruit.Apple, EnumResolver.Default); // serialize as string
var v2 = JsonSerializer.Serialize(Fruit.Apple, EnumResolver.UnderlyingValue); // serialize as value

Console.WriteLine(Encoding.UTF8.GetString(v1)); // "Apple"
Console.WriteLine(Encoding.UTF8.GetString(v2)); // 1

Hmm.. I have these lines of code set up inside my startup under services.AddMvcCore..
'''
options.InputFormatters.RemoveType<Microsoft.AspNetCore.Mvc.Formatters.JsonInputFormatter>();
options.OutputFormatters.RemoveType<Microsoft.AspNetCore.Mvc.Formatters.JsonOutputFormatter>();
                options.InputFormatters.Add(new JsonInputFormatter());
                options.OutputFormatters.Add(new JsonOutputFormatter());

'''
Those formatters are from utf8Json.
In the swagger configuration, I have DescribeAllEnumsAsStrings set with SwaggerGenOptions.
The problem is when I send stuff to the API end point, it thinks the object is null if I serialize it as a value..  If I serialize it as a string, it is fine..  Am I doing something incorrectly on setting up the formatters?

Please show concrete types and concrete code.
Also, a code that can explain the problem more concisely.

Sure...  The below should be flexible to work while using other serializers.  It may just be an enhancement though..
The problem, in essence, is that utf8json only supports translating enums when you put a quote around the value, even when the value gets serialized as an integer from newtonsoft.  I can't put a serializer setting when deserializing the value because I would like it to work with both fashions.
namespace Utf8JsonEnumTesting
{
    class Program
    {
        static void Main(string[] args)
        {
            TestCombo();

            Console.WriteLine("done!");
            Console.Read();
        }

        private static void TestCombo()
        {
            Console.WriteLine("Starting TestCombo");

            FruitPicker fp = new FruitPicker();
            fp.Name = "Greg";
            fp.Fruit = Fruit.Orange;

            var defaultSerializer = new Newtonsoft.Json.JsonSerializerSettings();

            var fruitGuy = Newtonsoft.Json.JsonConvert.SerializeObject(fp, defaultSerializer);

            //error!
            var fruitGuyDeserialized = Utf8Json.JsonSerializer.Deserialize<Fruit>(fruitGuy);

            Console.WriteLine(fruitGuyDeserialized);

        }
    }

    public class FruitPicker
    {
        public string Name { get; set; }
        public Fruit Fruit { get; set; }
    }
    public enum Fruit
    {
        Orange = 0,
        Apple = 1
    }

}

@neuecc now I have a proper comment.. Originally my example was bad.. Thanks for the assistance..

Thanks, sample code is nice.
Utf8Json supports int -> enum and string -> enum.
You can deserialize
var utf8Fruit = Utf8Json.JsonSerializer.Deserialize<FruitPicker>(incomingStringFromMicroservice);
But your example shows int -> string convert.
This convert is not supported on default.
You can create custom converter.
public class DurableStringFormatter : IJsonFormatter<string>
{
    public void Serialize(ref JsonWriter writer, string value, IJsonFormatterResolver formatterResolver)
    {
        writer.WriteString(value);
    }

    public string Deserialize(ref JsonReader reader, IJsonFormatterResolver formatterResolver)
    {
        var token = reader.GetCurrentJsonToken();
        switch (token)
        {
            case JsonToken.Number:
                return reader.ReadDouble().ToString();
            case JsonToken.String:
            case JsonToken.Null:
                return reader.ReadString();
            case JsonToken.True:
            case JsonToken.False:
                return reader.ReadBoolean().ToString();
            default:
                throw new JsonParsingException("Current token is invalid. Token:" + token);
        }
    }
}

and adopt it.
public class FruitPickerGenerated
{
    public string Name { get; set; }
    [JsonFormatter(typeof(DurableStringFormatter))]
    public string Fruit { get; set; }
}

Currently can not replace formatter on global if target type is primitive.
I will try to configure it in a while.

Thanks for the reply.  Unfortunately, the class and properties are auto generated and partial properties are not supported.  So, once we regenerate the source, it will be lost.  So, that isn't really a good solution..  I have a work around by making sure the outgoing httpclient formats the json in the correct fashion...  But, it would be nice if I didn't have to do that work around.  Thanks!


Search element in a sorted matrix in C++

🚀 Feature
Hereby using this code, one can search an element in a sorted 2D matrix. If the element is found in the matrix, the code will return its position.
Have you read the Contributing Guidelines on Pull Requests?
Yes
Motivation
I found this important problem that is widely used in competitive programming missing, and hence want to add its solution.
Pitch
This problem will be very useful for understanding the basic concepts of the 2D matrix.
As a GSSoC'21 participant, I would like to add the solution to this problem in the C++ language.
Please assign it to me.

/assign


upgrade phpstan level

It's now 0.
It should go up.
Not sure of the level to target. at least 4, I guess.

Closed by : https://github.com/friends-of-presta/fop_console/pulls?q=is%3Apr+is%3Aclosed+phpstan


charles proxy doesn't work in trusty chroot

This isn't a big deal but I'm curious.  The charles proxy didn't work in my trusty chroot.  It didn't show any traffic and the proxy client couldn't talk to anything.  I had to drag out an old windows laptop (shudder). This is the first time I've had to run windows since I switched to crouton two months ago.
Anyway, this is surprising to me.  I thought the crouton networking was full-featured, and a proxy is not rocket science.  Any ideas why this wouldn't work?

Maybe need root privileges or need open ports.
I am user in some forums and we always ask to the users to give all possible info, for example if they have problems using an app then give us logs, app version, how the app was installed, how they are running the app, error messages, config files, what thing they tried to solve the problem, system config,  hardware info, network settings ,etc.
Probably I can't help but other users can help you more easy if you give more info. Just as advice.

other users can help you more easy if you give more info

Yeah, sorry about that.  I was just curious if anyone knew of an architectural problem in how crouton or chromeos does networking.  I wasn't really looking for a "fix".  Just curious.
I don't need this anymore but if anyone if lurking here is my setup.
Chromebook pixel 2015
trusty chroot, no desktop


create authenticated routes

[ ] Delete Method will delete a single like/follow from Database
[ ] Add to our likes/follows to create an array of them

Closed by #51


Download links for Linux Clients are broken

When I click Install Instructions in order to download a Linux client on https://owncloud.org/download/, I am navigated to https://software.opensuse.org/download/package?project=isv:ownCloud:desktop&package=owncloud-client
There, when I choose the option Grab binary packages directly  I receive 404 errors for any distribution other than openSUSE.
I am aware, that the packages are not hosted on owncloud.org, but I did not know where I should file this issue.

Thanks for your notice. (Un-)fortunately I can't reproduce your error. I was able to download every version of the client for every listed OS.

Sorry for reopening, definitively does not work:

$ curl "https://download.opensuse.org/repositories/isv:/ownCloud:/desktop/Debian_9.0/amd64/owncloud-client_<IP_ADDRESS>54+oc-515_amd64.deb"
<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">

<title>302 Found</title>

Found

The document has moved <a href="http://widehat.opensuse.org/opensuse/repositories/isv:/ownCloud:/desktop/Debian_9.0/amd64/owncloud-client_<IP_ADDRESS>54+oc-515_amd64.deb">here.

<address>Apache/2.4.23 (Linux/SUSE) Server at download.opensuse.org Port 443</address>

$ curl http://widehat.opensuse.org/opensuse/repositories/isv:/ownCloud:/desktop/Debian_9.0/amd64/owncloud-client_<IP_ADDRESS>54+oc-515_amd64.deb

<title>404 Not Found</title>

404 Not Found

nginx/1.14.2

We checked on different computers now and everybody is able to download the files.
Also please note that those are SUSE download mirrors so we can't control what's going on there. We try to contact them. Feel free to contact them too.


Flush pack to avoid oversized

Closes https://github.com/restic/restic/issues/4412
What does this PR change? What problem does it solve?
Reduces the possibility of packs to be larger than --pack-size
Was the change previously discussed in an issue or on the forum?
https://github.com/restic/restic/issues/4412
Checklist

[x] I have read the contribution guidelines.
[x] I have enabled maintainer edits.
[x] I have added tests for all code changes.
[ ] I have added documentation for relevant changes (in the manual).
[x] There's a new file in changelog/unreleased/ that describes the changes for our users (see template).
[x] I have run gofmt on the code in all commits.
[x] All commit messages are formatted in the same style as the other commits in the repo.
[x] I'm done! This pull request is ready for review.

The patch is finally ready. I sure have run unit tests with it, and I run this restic version with debug log enabled on my backup data: works fine and the log looks adequately

Btw, I can add some clarification to documentation about possible pack sizes depending on the --pack-size value if you like

See #4412


Middleware is sometimes not found when defined in the traefik service's docker labels

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
I'm using traefik as part of a rather simple docker compose based setup.
On the traefik service I set a label similar to this one:
- traefik.http.middlewares.default-ssl.headers.STSSeconds=3600
Another service has a label similar to this one:
- traefik.http.routers.someservice.middlewares=default-ssl
This usually works fine, the only log line is

level=info msg="Configuration loaded from flags."

and all is well.
What did you see instead?
"Sometimes" (about 1 in 10 times) after stopping, removing and restarting the traefik service, it starts up with additional errors similar to:

level=error msg="middleware "default-ssl@docker" does not exist" entryPointName=websecure routerName=someservice@docker

Since usually this seems to work (as I said, it works like 9 out of 10 times) it feels like some kind of race condition, where apparently traefik seems to load the router configuration (including the referenced middleware) before it sets up the middleware itself?
What version of Traefik are you using?
traefik             v2.8             2caeed3432ab   8 days ago     106MB

Version:      2.8.5
Codename:     vacherin
Go version:   go1.19.1
Built:        2022-09-13T15:19:09Z
OS/Arch:      linux/amd64

What is your environment & configuration?
version: '3'
[...]
services:
  reverse-proxy:
    image: traefik:v2.8
    restart: always
    [...]
    labels:
      [...]
      - traefik.http.middlewares.default-ssl.headers.STSSeconds=3600
      [...]
  someservice:
    image: [...]
    restart: always
    labels:
      [...]
      - traefik.http.routers.someservice.entrypoints=websecure
      - traefik.http.routers.someservice.middlewares=default-ssl
      [...]

If applicable, please paste the log output in DEBUG level
No response

Hi! I'm Træfiker :robot: the bot in charge of communication regulation.
Thanks for your interest in Traefik!
We dedicate the issue tracker to bug reports and feature requests only. My advanced AI has spotted that your issue might be a configuration problem or relates to something that doesn't look like a bug.
To confirm this, please join our Community Forum and reach out to us on the Traefik section.
In case I'm wrong (well, that would be embarrassing :sweat_smile:), my developers will re-open the issue and fix me!
In the meantime, you can double check Traefik's documentation.

Did a human look at this?
I created this as a bug report because this is the first time I'm seeing this issue after upgrading from v2.6. I might have just been lucky to never trigger it before, but right now it at least seems to be happening more often in any case.
Tagging @ldez to at least confirm if the bot's decision is correct.

Pinging more people because two more days passed with this issue being closed without an explanation or confirmation.
@rtribotte @ddtmachado

Pinging more people because three more days passed with this issue being closed without an explanation or confirmation.
@mpl @kevinpollet @tomMoulard @mmatur

Hello,
I confirm the bot's decision.
I recommend opening a topic on the forum.
Please stop pinging people.

Continuing at https://community.traefik.io/t/middleware-is-sometimes-not-found-when-defined-in-the-traefik-services-docker-labels/15979


Develop Resource Page: Leadership: Onboarding

This issue is for coordinating around developing the resource page for Leadership: Onboarding.
The resource page lives in the following google document for easy editing, commenting, and revision:
https://docs.google.com/document/d/1mISDVKvxOTyJNz7LQ1Nh_aJ3KRskn5qRnPZzxSQokwk/edit?usp=sharing
Once the page is in a decent shape, we will put a copy of it in the repository as a markdown file.
To work on this issue, feel free to leave a comment here and start editing the document. For questions, feedback requests, and other coordination tasks, please leave a comment here on the issue.
Please edit the google document directly and add your ideas to it. We can help with cleaning it up later.

A lot of progress is made on the document. Looks like it may be ready for a PR, @GeorgLink

Yes, I think we are at a point of diminishing returns to discuss this metric further in meetings.
PR with the new metric template would be great.

I am closing the issue as it has been labeled as a metric idea. The idea is that we can capture all metric ideas (either new metrics or additions to existing metrics). We can then look at all Metric Ideas collectively.


Remove links from compose file

Links are a legacy option and aren't necessary since compose creates a "default" network for all the services.
https://docs.docker.com/compose/compose-file/compose-file-v2/#links

Thank you for your pull request. I tried it out and it works :)


Add missing ES 2021+ features

ES2021 Or ES12

[x] replaceAll
[x] promise.any
[ ] WeakRef
[x] Numeric Separators
[x] Logical Operators

ES2022 Or ES13

[x] Top level await
[x] Class fields
[x] Array at()
[x] Error cause
[x] hasOwn
[ ] Regex match indices

ES2023 Or ES14

[x] Find array from last
[x] Hashbang syntax
[x] Symbols as WeakMap keys
[x] Change Array by Copy
[ ] https://github.com/tc39/proposal-private-fields-in-in
[ ] https://github.com/tc39/proposal-regexp-v-flag
[ ] https://github.com/tc39/proposal-resizablearraybuffer

@bnoordhuis can you pl check if there is anything missing from the list?

Looking at https://github.com/tc39/proposals/blob/HEAD/finished-proposals.md#finished-proposals, I believe we're missing:

https://github.com/tc39/proposal-regexp-match-indices
https://github.com/tc39/proposal-private-fields-in-in
https://github.com/tc39/proposal-regexp-v-flag
https://github.com/tc39/proposal-resizablearraybuffer

You guys are doing awesome work here. Here is frida-gum that implements WeakRef for QuickJS like this:
https://github.com/frida/frida-gum/blob/ed4f17176f8671af93fbbb9afdbb584fb7825bf6/bindings/gumjs/runtime/entrypoint-quickjs.js#L12

Create a ref
https://github.com/frida/frida-gum/blob/ed4f17176f8671af93fbbb9afdbb584fb7825bf6/bindings/gumjs/gumquickcore.c#L2261

deref

https://github.com/frida/frida-gum/blob/ed4f17176f8671af93fbbb9afdbb584fb7825bf6/bindings/gumjs/gumquickcore.c#L2371C24-L2371C47

unbind

https://github.com/frida/frida-gum/blob/ed4f17176f8671af93fbbb9afdbb584fb7825bf6/bindings/gumjs/gumquickcore.c#L2322
Which could make it simpler to implement WeakRef possibly.

Thanks for sharing! I did lift a couple of commits from Frida already, but hadn't seen the gum part.

Here's someone also implemented FinalizationRegistery for QuickJS: https://github.com/cashapp/zipline/blob/trunk/zipline/native/common/finalization-registry.c
I know this isn't the most perfect form but it almost gets you there.

:information_source: new upstream release: https://github.com/bellard/quickjs/commit/daa35bc1e5d43192098af9b51caeb4f18f73f9f9

I'll backport some of the fixes, but there are no new ES features other than the ones he took from this project AFAICT :-)


Support v1.3.*@dev Turkish language.

Improve dummy data and insert more
Update Address statement
Update Phone support
Insert Mobile Phone Support

Bump. Please update this PR or I'll close it within a few weeks.

Abandoned PR, closing.


[AMBARI-22012] Improve ManagedThreadPoolExecutorTest

What changes were proposed in this pull request?
I wanted to confirm that ManagedThreadPoolExecutor was not losing tasks submitted to it before it being started, so I added a unit test case.  While there, also improved the existing test a bit.
How was this patch tested?
Ran unit test (it's a test-only change).

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2851/
Test FAILed.
Test FAILured.

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2853/
Test PASSed.


Run before stop not respected by HA-Proxy

If a single container service is upgraded with run before stop ticked I would expect the proxy to keep routing services to the existing running container until stopped but instead a 503 is returned the second the upgrade starts.
I shouldn't be forced to have multiple containers to ensure continuous service is delivered, I could live with a few seconds but to drop all services immediately on upgrade seems to defeat the purpose of having the start before stop tick-box!?

haproxy?

sorry was searching for existing issue in rancher/rancher and must have stumbled onto your repo before raising issue and noticed the sec I pressed add.


Added a "filename_hook" parameter to Python interface

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[ ] Improvement
[ ] New extractor
[x] New feature

Description of your pull request and other information
Hello! I need the to sanitize the filenames myself since the existing functions do not satisfy me. Thus, I've added a way to add a hook that receives the filename generated by YoutubeDL and returns a filename that'll be used further down the line.
Reasons: with "restricted filenames", filenames are sanitized too much to the point of making the filename very ugly, and without "restricted filenames" I get characters that aren't compatible with the software I use. Also I plan to further integrate YoutubeDL with my music collection organizing scripts and such a feature would help. I already use it in my local version but I figure others could benefit from this as well.
More importantly, there are currently no ways to implement custom filtering of filenames. For example, when downloading videos straight from Youtube for the purpose of storing them, you'll sometimes want to use the title as your filename. However, the title can contain redundant or unnecessary elements, like the channel name or genre. Currently, I have to write scripts to do the postprocessing for my purposes, and that's very annoying.
The outtmpl template can't be used to, for example, replace characters in the generated filename or shorten it (though they can be used together, as my example shows), or applying a regex to strip out useless parts of the filename. If there is a way to hook a Python function so that it can process and change the filename used, I'd be happy to hear about it - the code didn't show any.
Sidenotes: The hook is very likely to get called 2 or more times (since  process_filename is called quite often), but I can't see this causing any problems if we assume the processing function to always return same x for same y (which is the case for any reasonable sanitization function).
Example usage:
def filename_hook(filename):
    filename = filename.replace(u'\u3010', u'[').replace(u'\u3011', u']')
    return filename

opts = {
"filename_hook":filename_hook,
"match_filter":print_info,
"outtmpl":"%(title)s",
"postprocessors":[{"key":'FFmpegExtractAudio', "preferredcodec":"mp3"}]
}

Sorry for reopening this, but it was closed when the reviewer, apparently, did not fully understand the purpose of this feature, and from what I found on Google the pull requests can't be reopened after closing. I also improved my pull request by further describing the usefullness of this function.

I get characters that aren't compatible with the software I use

It's surprising to hear this in 2016 that an application can't handle unicode characters. Apparently the problem is in your software, not youtube-dl

Currently, I have to write scripts to do the postprocessing for my purposes, and that's very annoying.

Hmm, isn't writing a hook function the same effort as writing postprocessing scripts?

Well, I've decided to disable restricted filenames in my non-modified setup, download a video with a Russian filename and see what fails:
Noize MC – Питерские крыши   #filename
WARNING: Requested formats are incompatible for merge and will be merged into mkv.
ERROR: unable to open for writing: [Errno 22] invalid mode ('wb') or filename: 'Noize MC \xe2\
x80\x93 \xd0\x9f\xd0\xb8\xd1\x82\xd0\xb5\xd1\x80\xd1\x81\xd0\xba\xd0\xb8\xd0\xb5 \xd0\xba\xd1\
x80\xd1\x8b\xd1\x88\xd0\xb8.f137.part'
Exception in thread Thread-1:
Traceback (most recent call last):
  File "/usr/lib/python2.7/threading.py", line 810, in __bootstrap_inner
    self.run()
  File "/usr/lib/python2.7/threading.py", line 763, in run
    self.__target(*self.__args, **self.__kwargs)
  File "ssyoutube.py", line 53, in dl_thread
    download(link)
  File "ssyoutube.py", line 59, in download
    ydl.download([link])
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl-2016.12.18-py2.7.egg/youtube_dl/Yout
ubeDL.py", line 1797, in download
    url, force_generic_extractor=self.params.get('force_generic_extractor', False))
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl-2016.12.18-py2.7.egg/youtube_dl/Yout
ubeDL.py", line 711, in extract_info
    return self.process_ie_result(ie_result, download, extra_info)
...
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl-2016.12.18-py2.7.egg/youtube_dl/Yout
ubeDL.py", line 542, in trouble
    raise DownloadError(message, exc_info)
DownloadError: ERROR: unable to open for writing: [Errno 22] invalid mode ('wb') or filename:
'Noize MC \xe2\x80\x93 \xd0\x9f\xd0\xb8\xd1\x82\xd0\xb5\xd1\x80\xd1\x81\xd0\xba\xd0\xb8\xd0\xb
5 \xd0\xba\xd1\x80\xd1\x8b\xd1\x88\xd0\xb8.f137.part'

I don't know what exactly is failing and why. Could be the FS on source drive (FAT32, not very likely to change that since I need to work with it both from Windows and Linux). Could be that I need to upgrade to Python 3, as usual. If you're interested in the problem I'm getting, I can help you debug this, but this is definitely not the point.
This is not the only problem this hook can solve. Say, you download videos in Russian - you can "transliterate" filenames (map Russian characters to their closest phonetical analog in English) to avoid having filenames in Russian all of a sudden, which trips up media players (I'm not too keen to change players/software if the experience is great otherwise) and virtual terminals (you need to set up encodings properly before you can see the filename on the terminal).
Effort writing postprocessing scripts is not effort well-spent. An issue like that is better solved when the file is created, not when "hang on, let me launch my script". Especially when you can, for example, use info_dict (available only at download time, unless you want to use even dirtier hacks) to generate an even better filename. I've got two or three Python scripts doing this "downloading, processing, recording into DB, reformatting and synchronization" thing - there's no sense to have more, honestly, not when a solution is a 3-line patch that doesn't affect anybody not using it.
So, the hook purposes:

Transliterating filenames
Filtering filenames
Generating better filenames

I think it's likely that other people will need something like this.

OK I see your point.

Especially when you can, for example, use info_dict (available only at download time, unless you want to use even dirtier hacks) to generate an even better filename.

This sounds a better approach. Allowing more flexible filename generation is better than fixing bad filenames afterwards. Could you like to have a try?

Hello! I got some useful advice in my previous PR: https://github.com/rg3/youtube-dl/pull/11482#issuecomment-267991201 . To get to my goal, I can just derive from YoutubeDL class and override the prepare_filename function. Therefore, this functionality can be achieved in the current version, so this PR is unnecessary =) Thank you everybody for the help, especially @dstftw .
@yan12125 - how do you imagine that "more flexible filename generation"? I got my solution working for me, but it's possible I could implement what you're suggesting in my free time.
The resulting (tested) implementation can be seen here: https://github.com/CRImier/ssyoutube/commit/4e549193348e86239b5f48feb4391dd69a274614

To get to my goal, I can just derive from YoutubeDL class and override the prepare_filename function.

I was proposing something like 'outtmpl_callback': my_prepare_filename, but your solution is better :)


Addon installation in Homeassistant does not allow zwavejs integration to run

Problem/Motivation

(Why the issue was filed): Followed the 6 steps found online but with additional research, for instance the proper HA URL for the server, ws://a0d7b954-zwavejs2mqtt:3000, that was hard to find.  Got the keys from the official addon.  Disabled the zwave integration.  Installed this comminity addon.  It ran in the sidebar.  Entered my port, plus the 4 keys, legacy, access control, and the S2 authorized and unauthorized keys from the old addon.

Made sure the old addon was stopped.  Enabled the integration.  Chose ws://a0d7b954-zwavejs2mqtt:3000 as the URL, and it fails ever time.  The addon runs, I can even see devices.  I then tried removing the official addon.  It kept re-installing itself, but I edited a config file to make the pairing between the integration and official addon "false".  Again, tried to enable the integration, and received failed to connect error.
Tried rebooting, same issue.
Expected behavior

(What you expected to happen): I expected the integration and the addon to run with peaceful co-existence.  But the integration will not start, meaning that my zwave homeassistant devices will not be recognized by homeassistant.

Actual behavior

(What actually happened):  The error message as above, integration would not start, and the zwave devices are not accessible from the homeassistant dashboard.

Steps to reproduce

(How can someone else make/see it happen):  I don't know, clearly it's working for some, but following the procedure precisely did not work for me.

Proposed changes

(If you have a proposed change, workaround or fix,
describe the rationale behind it)

have you enabled the websocket server in zwave-js-ui?


Custom tooltip on the remote indicator

For #126258
Let resolvers and virtual workspace providers to customize the tooltip on  the remote indicator

Adds workspaceTooltip to ResourceLabelFormatting. workspaceTooltip is a markdown string.
It is trusted (can contain commands) an supports icons ($(zap))

FYI @eamodio @chrmarti @roblourens

Needs more work...

To verify:

from the vscode-extension-samples open https://github.com/microsoft/vscode-extension-samples/tree/main/fsprovider-sample
in the package.json:

add "enableProposedApi": true,
add

 	"resourceLabelFormatters": [
 		{
 			"scheme": "memfs",
 			"formatting": {
 				"label": "${path}",
 				"separator": "/",
 				"tildify": false,
 				"workspaceSuffix": "In Memory",
 				"workspaceTooltip": "Use $(zap) with [that](command:vscode.newWindow)"
 			}
 		}
 	]

- run with F5, Use the `MemFS: Setup Workspace` command and hover over the remote indicator.
- verify that the hover shows the text (with images and links) as specified by `workspaceTooltip`


when npm run build

Description
electron-builder 19.45.4
Error: Unresolved node modules: vue
at D:\frontend\dockeron\node_modules_electron-builder@19.45.4@electron-builder\src\util\packageDependencies.ts:112:19
Steps to Reproduce

npm run build

Expected behavior: [What you expect to happen]
Actual behavior: [What actually happens]
Reproduces how often: [What percentage of the time does it reproduce?]
Versions
[node, npm, vue.js, etc.]
node 8.7.0
npm 5.4.2
vue.js 3.5.3
Additional Information
Any additional information, configuration or data that might be necessary to reproduce the issue.

I cannot reproduce this issue.  I have tried clone a new project and do npm install and npm run build.  It is working fine.
Could you provide the full logs on your console when you run install and build?
BTW, your vue.js version is 3.5.3? it does not exist.....

Okay, thank you!
------------------ Original ------------------
From: "Chang Liu";
Date: 2017年11月16日(星期四) 晚上6:12
To: "dockeron/dockeron";
Cc: "东方辰"; "Author";
Subject: Re: [dockeron/dockeron] when npm run build (#3)
I cannot reproduce this issue.  I have tried clone a new project and do npm install and npm run build.  It is working fine.
Could you provide the full logs on your console when you run install and build?
BTW, your vue.js version is 3.5.3? it does not exist.....
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


clang-analyzer fixes for clang14

This fixes clang-analyzer complaints from running clang 14 analyzer on rockylinux 9.
Correction, rockylinux 9 is clang-15, ubuntu 22.04 is clang-14.

[approve ci]

[approve ci clang-analyzer]

[approve ci autests]


[charts/redis-ha] Add a way to replace the default script for a restore init container.

Signed-off-by: Justin StGermain<EMAIL_ADDRESS>What this PR does / why we need it:
Ability to have a custom script in the restore init container.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #143

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

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

@DandyDeveloper
This is another addition we have made for some custom use cases.
I forgot to add it to the PRs yesterday.

@j771 I'm sorry it took me so long to get back to you on this.
Question on this, why this over using extraInitContainers? Which keeps this flexibility without any real confusion.

@j771 Any thoughts on the above?


Allow dot'ted field in connectFrom of $graphLookup aggregation pipeline

This relates to Issue #595. This adds capability for using dot'ed parameters in connectionFrom field of $graph aggregation pipeline.

Am pretty close to getting thru the pipeline. The code coverage is failing because my tests are not covering one additional statement, compared to the original. I could use help with which statement am I missing.  I am not able to get the details from codecove report. (It just could be that I connected my repo after this failure. Will give it a try after making some dummy changes.)

Am pretty close to getting thru the pipeline. The code coverage is failing because my tests are not covering one additional statement, compared to the original. I could use help with which statement am I missing.  I am not able to get the details from codecove report. (It just could be that I connected my repo after this failure. Will give it a try after making some dummy changes.)

@pcorpet  Re: changing test architecture.
Like to make sure I have understood you correctly. You are asking me to do the following:

Move the tests I have added into test__collection_api.py. These tests do not use live mongo server.
Add more tests in test test__mongomock that compares the behavior of mongomock with pymongo + MongoDB.

Did I get you right?

@pcorpet  Re: changing test architecture.
Like to make sure I have understood you correctly. You are asking me to do the following:

Move the tests I have added into test__collection_api.py. These tests do not use live mongo server.
Add more tests in test test__mongomock that compares the behavior of mongomock with pymongo + MongoDB.

Did I get you right?

@pcorpet I've addressed all your comments, except one, which happens to be the most significant one, in my opinion. The pending task is to add tests in test__mongomock.
I have reviewed how you have written other tests. I think it will be simple enough to translate the tests I have in "test__collections_api" into the dual comparison tests.  But it will likely take some time. I will need to switch context to my day job.

@pcorpet I've addressed all your comments, except one, which happens to be the most significant one, in my opinion. The pending task is to add tests in test__mongomock.
I have reviewed how you have written other tests. I think it will be simple enough to translate the tests I have in "test__collections_api" into the dual comparison tests.  But it will likely take some time. I will need to switch context to my day job.

@pcorpet - I added tests in test__mongomock. Was very easy. You've set up the testing architecture very well.

@pcorpet - I added tests in test__mongomock. Was very easy. You've set up the testing architecture very well.

Thank you for the hard work. Feel free to send another PR to add your name in the contributors of the README (I should have suggested that earlier, sorry).

Thank you for the hard work. Feel free to send another PR to add your name in the contributors of the README (I should have suggested that earlier, sorry).


I got diifferent results on the evaluate set.

Hello, thanks for your wonderful work. I got the pretrained model and I ran the train.py to reproduce the dynamic-multiframe-depth, but I got different results from yours(resnet18-pretrained).
Paper
Abs_rel / Sq_rel / rmse / rmse_log / a1 / a2 / a3
0.043  0.151  2.113  0.073  0.975  0.996  0.999
Own
Abs_rel / Sq_rel / rmse / rmse_log / a1 / a2 / a3
0.126  0.893  4.552  0.19  0.833  0.94  0.981
And my torch==1.10.1+cu113,torchvision==0.11.2+cu113.
All indicators are quite different from those in the paper. I did not make changes in “trian_my_resnet18.json”, just replaced “n_gpus=8” with “n_gpus=3”.

I want to know why my result is not good.
In addition to the settings in “trian_my_resnet18.json”, what details do I need to pay attention to in order to reproduce the result.

Looking forward to your reply, thank you.
Best wishes!

Hi,
Thanks for your attention to our work. The results look weird, can you double-check if the scores are from dynamic area evaluation or full-image evaluation?


Smart Object setting not work ( 1.3.2 )

Thank you very much for the new version 1.3.2, but unfortunately the “Smart object” setting still doesn’t work
I tried to find and fix this in the code myself, but there is no response to changing values, which in my opinion may be related to this. My knowledge is not enough to fix the problem.
I reported the bug earlier https://github.com/AbdullahAlfaraj/Auto-Photoshop-StableDiffusion-Plugin/issues/333#issue-1807741671
but since the new version is 1.3.2, I decided to make a separate message.

Maybe there is some simple way to solve this problem? I would appreciate your help.

I must have missed it, thanks for the reminder.
it's now fixed in https://github.com/AbdullahAlfaraj/Auto-Photoshop-StableDiffusion-Plugin/pull/378/commits/2d4bc9aea35749cd7b9ee4484c81b8895d787dfb


added jq to download packages, updated air-gapped installation doc

Description
Checklist/ToDo's

[x ] Added documentation?
[ ] Added release note entry? (  docs/release-notes.md )
[ x] Tested?

Hey,
I have added jq-package into the ansible task since it it widly used and required for ait-gapped environment setup :)
I also updated the air-gapped documentation and added more details and the usage of the oc-mirror plugin

/test

/test

@tjungbauer thanks for contribution. I rebase your master branch to devel tree. Just for information.


Automated Integration Tests

A lot of our testing is done through manual verification:

Make a change in physics.
Build the sample project.
Check changes in a few of the scenes.

This whole flow should be handled automatically in a GitHub action and branch protection rules should exist for vnext, blocking on failure.
This depends on first being able to build and run a headless sample on a GitHub runner.
Questions:

Do we use an existing testing framework?

I was able to get a prototype working with gtest, but there is a bit of legwork in writing the tests because the engine is actually running: needs a scene, a way to perform test actions, a mechanism for collecting test restults, then the results are actually tested after stopping the engine.

If we don't use gtest, how do we collect, verify, and publish the test results?

We spoke a little more about this, and maybe a simpler smoke test-based thing is sufficient for now. Rather than trying to do granular testing of each step as the engine is running, we just run a test app and check the exit code. So there's no real test framework needed, just one pass/fail check. Context/error details could be logged and we can publish that log from CI.

SmokeTest scene added. Needs #601 before adding to CI.


weapons on detect.py

Is it possible to add in weapons to the model? Or would one of the models do weapons? I downloaded 4 images from a Google search:
py detect.py --weights /content/Yolov7-Close-People-Counter/weights/yolov7x.pt --source C:\Users\Administrator\Desktop\YOLOV7-OBJECT-COUNTER-V1.2\inference\images\gun1.PNG --no-trace

No, I mean it is necessary to train the model with images. After that you can classify the weapons.

Are the datasets public to train a model? For example if the model I am using works really well how could I add to it?

You must either find or create your own datasets. By datasets I mean at least 1k different images for each class.
You can find the solution to the question of how to train your own model from the link below:
https://www.youtube.com/watch?v=kbMfgVZn05U&list=PL_Nji0JOuXg2QMohGK7wfzgJ-MavzXRHW&index=4


Request fails NTML authentication after successful CORS preflight (IE11)

We have a web application that talks to an API hosted on a different domain, using this library to polyfill fetch in order to support IE11. Since the backend is on a different domain, we have configured all the appropriate CORS headers.
When opening the app in e.g. Chrome, everything works smoothly. However, when browsing with IE11, we get a bunch of 401 Unauthorized responses from the backend. On closer inspection, I can see that for each API call, the following happens:

A CORS preflight request (OPTIONS) is sent to the endpoint, with a successful response including all the required headers:
Access-Control-Allow-Credentials: true
Access-Control-Allow-Headers: accept,content-type
Access-Control-Allow-Methods: GET
Access-Control-Allow-Origin: http://localhost:8080

A GET request to the endpoint returns a 401 Unauthorized response with
WWW-Authenticate: Negotiate, NTLM

In Chrome, the browser pops up a dialog where I enter username and password, and then proceeds to negotiate the login and issue the request again, this time successfully (and this is the only thing I see in the network tab in Chrome DevTools).

In IE11, no username/password dialog pops up, and no negotiation takes place. Instead, I just get the bare 401 response.

Is there something I need to do with this polyfill in order to make it work with NTLM handshakes?

While I'm not familiar with NTLM, this could have something to do with the withCredentials setting on the cross-origin request. See the cookies section of the documentation for an example enabling withCredentials.
Here's where we toggle the setting on the XMLHttpRequest object based on the configuration options passed into the fetch call. The browser coordinates the authentication steps, without exposing them via XMLHttpRequest, so this is as much control as we have.
You might try making the request with XMLHttpRequest, rather than fetch, to debug the issue. Removing the fetch wrapper layer will simplify testing the request and response.


Awake Utility Icons should indicate state

Description of the new feature / enhancement
The Awake Utility should hange icon according to the state.
Scenario when this would be used?
Always
Supporting information
No response

/dup #11996


[enhancement] try adding error prompt to MetaMask when adding chain that doesn't exist

Error:
MetaMask - RPC Error: Unrecognized chain ID "0x134afd7". Try adding the chain using wallet_addEthereumChain first.
{
  "code": -32603,
  "message": "Unrecognized chain ID \"0x134afd7\". Try adding the chain using wallet_addEthereumChain first.",
  "data": {
    "originalError": {
      "code": 4902,
      "message": "Unrecognized chain ID \"0x134afd7\". Try adding the chain using wallet_addEthereumChain first.",
    }
  }
}

And the RPC URL is dead:
https://dev-seed.rooch.network:443

try again

try again

Still doesn't work:

Could not fetch chain ID. Is your RPC URL correct?

The link is responsive now.


Automatically check for formatting errors in this and other repos

Is your maintenance request related to a problem? Please describe.
Not all files are automatically checked for formatting errors. I raised a few recent PRs to clean up trailing whitespace, a few extra blank lines, and replace tabs with spaces in a JSON file.
Describe the solution you'd like
A lint task should check for formatting errors in files such as doc (md), JSON, and YML files.
Prettier seems to do this on its doc (md) files, at least.
Describe alternatives you've considered
Manual cleanup with help from grep and sed
Additional context
I discovered the formatting errors while doing git log -p, which highlights the trailing spaces. This is what I sometimes use to check my recent work in a git repo.
I am raising this partially in response to @nicojs https://github.com/stryker-mutator/stryker/pull/1874#issuecomment-557493115:

@brodybits did you use a linting tool for this? If so it might be worth adding it to our linting CI/CD phase so we can prevent this from happening again

Do you want to take this task? Or should do it?

Or should do it?

That would be awesome, thanks!

Is it still valid? I have made quite a lot changes to linter so far 🤔

I think so, as discussed in #2174. I would love to work on this but am juggling some other priorities right now.

I'll close this for now. if someone wants to pick it up, feel free to do so 👍


fix(epath.Path): use __init__ for python 3.12

Addresses #607 following @Conchylicultor proposition.
This fixes the following test failures, encountered with python 3.12:
FAILED etils/epath/gpath_test.py::test_repr_gcs - assert "PosixGPath('gs:/bucket/dir')" == "PosixGPath('.../bucket/dir')"
FAILED etils/epath/gpath_test.py::test_repr_s3 - assert "PosixGPath('s3:/bucket/dir')" == "PosixGPath('.../bucket/dir')"
FAILED etils/epath/gpath_test.py::test_gcs[gcs_mocked_path0] - AssertionError: assert not True
FAILED etils/epath/gpath_test.py::test_gcs[gcs_mocked_path1] - AssertionError: assert not True
FAILED etils/epath/gpath_test.py::test_open[gcs_mocked_path0] - FileExistsError: [Errno 17] File exists: 'gs:/bucket/dataset'
FAILED etils/epath/gpath_test.py::test_open[gcs_mocked_path1] - FileExistsError: [Errno 17] File exists: '/data/soft/etils/gs:/bucket/dataset'
FAILED etils/epath/gpath_test.py::test_touch[gcs_mocked_path0] - FileExistsError: [Errno 17] File exists: 'gs:/bucket'
FAILED etils/epath/gpath_test.py::test_touch[gcs_mocked_path1] - FileExistsError: [Errno 17] File exists: '/data/soft/etils/gs:/bucket'
FAILED etils/epath/gpath_test.py::test_unlink[gcs_mocked_path0] - FileExistsError: [Errno 17] File exists: 'gs:/bucket'
FAILED etils/epath/gpath_test.py::test_unlink[gcs_mocked_path1] - FileExistsError: The operation failed because the specified path='gs:/bucket' already exists.
FAILED etils/epath/gpath_test.py::test_mkdir[gcs_mocked_path0] - AssertionError: assert not True
FAILED etils/epath/gpath_test.py::test_mkdir[gcs_mocked_path1] - AssertionError: assert not True
FAILED etils/epath/gpath_test.py::test_rename[gcs_mocked_path0] - AssertionError: assert False
FAILED etils/epath/gpath_test.py::test_rename[gcs_mocked_path1] - AssertionError: assert False
FAILED etils/epath/gpath_test.py::test_copy[gcs_mocked_path0] - AssertionError: assert not True
FAILED etils/epath/gpath_test.py::test_copy[gcs_mocked_path1] - AssertionError: assert not True

Notably, it does not fix the following:
FAILED etils/epath/gpath_test.py::test_walk[gcs_mocked_path0] - TypeError: Path.walk() got an unexpected keyword argument 'topdown'

Thank you for the fix!
For the topdown, it looks the real arg name in Python 3.12 is top_down

https://github.com/google/etils/pull/613 should fix the last issue

Thanks for taking care of it!


License for the project is not specified

Possible contributors/users would need to know what sort of license this is released under, eg GPLv3+ LGPLv3+, etc...

+1 I started working on something similar, but this is already much more complete, so I'd rather contribute to this, but not without license.


Transmission doesn't install

I'm seeing this when doing a brew cask install transmission:
[macbook/External/]> brew cask install transmission
==> Downloading https://transmission.cachefly.net/Transmission-2.84.dmg
Already downloaded: /Library/Caches/Homebrew/transmission-2.84.dmg
Error: Is a directory - <?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
<dict>
	<key>system-entities</key>
	<array>
		<dict>
			<key>content-hint</key>
			<string>GUID_partition_scheme</string>
			<key>dev-entry</key>
			<string>/dev/disk2</string>
			<key>potentially-mountable</key>
			<false/>
			<key>unmapped-content-hint</key>
			<string>GUID_partition_scheme</string>
		</dict>
		<dict>
			<key>content-hint</key>
			<string>Apple_HFS</string>
			<key>dev-entry</key>
			<string>/dev/disk2s1</string>
			<key>mount-point</key>
			<string>/private/tmp/dmg.bZq37q</string>
			<key>potentially-mountable</key>
			<true/>
			<key>unmapped-content-hint</key>
			<string>48465300-0000-11AA-AA11-00306543ECAC</string>
			<key>volume-kind</key>
			<string>hfs</string>
		</dict>
	</array>
</dict>
</plist>
Please report this bug:
    https://github.com/caskroom/homebrew-cask/issues
/usr/local/Cellar/brew-cask/0.37.1/rubylib/plist/parser.rb:66:in `read'
/usr/local/Cellar/brew-cask/0.37.1/rubylib/plist/parser.rb:66:in `initialize'
/usr/local/Cellar/brew-cask/0.37.1/rubylib/plist/parser.rb:27:in `new'
/usr/local/Cellar/brew-cask/0.37.1/rubylib/plist/parser.rb:27:in `parse_xml'
/usr/local/Cellar/brew-cask/0.37.1/rubylib/cask/system_command.rb:72:in `_parse_plist'
/usr/local/Cellar/brew-cask/0.37.1/rubylib/cask/system_command.rb:25:in `run'
/usr/local/Cellar/brew-cask/0.37.1/rubylib/cask/container/dmg.rb:23:in `mount!'
/usr/local/Cellar/brew-cask/0.37.1/rubylib/cask/container/dmg.rb:13:in `extract'
/usr/local/Cellar/brew-cask/0.37.1/rubylib/cask/installer.rb:75:in `extract_primary_container'
/usr/local/Cellar/brew-cask/0.37.1/rubylib/cask/installer.rb:36:in `install'
/usr/local/Cellar/brew-cask/0.37.1/rubylib/cask/cli/install.rb:20:in `block in install_casks'
/usr/local/Cellar/brew-cask/0.37.1/rubylib/cask/cli/install.rb:17:in `each'
/usr/local/Cellar/brew-cask/0.37.1/rubylib/cask/cli/install.rb:17:in `install_casks'
/usr/local/Cellar/brew-cask/0.37.1/rubylib/cask/cli/install.rb:6:in `run'
/usr/local/Cellar/brew-cask/0.37.1/rubylib/cask/cli.rb:80:in `run_command'
/usr/local/Cellar/brew-cask/0.37.1/rubylib/cask/cli.rb:120:in `process'
/usr/local/bin/brew-cask.rb:42:in `<top (required)>'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/usr/local/Library/brew.rb:59:in `require?'
/usr/local/Library/brew.rb:137:in `<main>'

Is it normal that I'm having both of these installed?
/opt/homebrew-cask/Caskroom/transmission/2.84
/opt/homebrew-cask/Caskroom/transmission/2.90


[BEAM-3272] Sickbay ApexRunner ParDoTranslatorTest.testAssertionFailure

This test has been flaking for some time. It has gotten much worse with Gradle, perhaps because of increased parallelism and/or stricter control over build output directories by the tool. I don't have time to try to fix it but we should unblock everyone else from getting test failures on their PRs.

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.

[x] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Since this is pretty much build breakage at HEAD, I'll take any LGTM or actually might go ahead with merging according to emergency policy.


what should go in the "runnings" property ?

MY options look like this ,
var options = {
		"html": "print.html",
		"paperSize": {
			format: 'A4',
			orientation: 'portrait',
			border: '1cm'
		},
		"deleteOnAction": true
	} ;

However, I have no idea as to what should go inside the runnings property .

I did something like this to get customized header and footer:
var runnings = "module.exports = { header: {height: '1cm', contents: function(page) { return page === 1 ? '' : '
Title
' }}, footer: {height: '1cm', contents: function(page, pages) { return '
' + page + '/' + pages + '
'}} }"
...
"runnings" : runnings

However, conversion implementation seem to be buggy. I made two fixes into code before conversion was made successfully. Fixes are found here: https://github.com/pejuam/phantom-html2pdf/commits/master

Would you mind to update the docs as well? Thanks!

@saru95 Is this solved for you with the answer from @pejuam?

Can someone please explain what the "runnings" file does and provide an example of it? This seems to be an important item that is completely undocumented, unless I'm missing something.
Thanks in advance.

@befreestudios https://github.com/bauhausjs/phantom-html2pdf#runnings-file

@dustin-H Not sure how I missed that.. (bows head in shame, lol)
BTW awesome little lib, nice job. Does exactly what we need with minimal configuration.

Thanks! Credits to @PhilippCh for initially creating this!


Hive external tables support schema change【Easy】

Requirement Description
Currently StarRocks supports querying hive external table. The table/partition and other information in hive external table are obtained from the hive metastore and cached in the StarRocks FE. Hive partitions and cached statistics such as row count and file location can be refreshed and synchronized by manually executing the refresh table syntax or enabling the StarRocks automatic incremental synchronization option.
The schema of hive table in FE cannot be changed after external table is created. Therefore, if the user's original hive table is schema changed (such as add/drop column, etc.) , the user needs to recreate the hive external table to obtain the correct result.
We need to support schema changes so that users can get correct query results without having to recreate the external table.We may support schema changes in the following two aspects:

refresh external table
Background automatic incremental synchronization hive metastore cache

Mentor
@stephen-shelby
Registration procedure
Step 1. Add a comment when you sign up. The comment can be a brief self-introduction and reason for signing up, including what you do, what you're good at, what you're interested in and your technology stack, and how familiar you are with StarRocks. Example:

I pick! My name is Johnny. I am from StarRocks and I have been working on database kernel development for 1 year. I am interested in OLAP, good at C++ programming, and have been using StarRocks for 6 months. I chose to solve this issue because I am familiar with execution engine.

Step 2. On your WeChat, add the StarRocks community assistant "StarRocks-1" as a WeChat friend and leave your GitHub ID. The assistant will invite you to a WeChat group based on the issue you picked.
Step 3. The mentor will communicate with all the participants who picked this issue and assign the task to the best match.
Before you start

Read the StarRocks GitHub workflow carefully, which provides the configurations of BE and FE and describes how to submit a pull request.
To learn more about StarRocks, read StarRocks documentation.

Pull request requirements

After you complete code development, upload the code to the StarRocks code repo as a pull request (PR).

PR title format: [SR GeekCamp] + PR title
Labels ：[Geek Camp 2022/feature]

After you submit your PR, comment on the status of your PR submission as requested under the Pinned Issue [StarRocks geek Camp 2022] page.

Comment format: "I have finished Issue #" + PR link id

My name is WangMan. I'd like to pick this issue. I started using it very early StarRocks. From version 1.14 to 2.1.  I chose to solve this problem because I am familiar with using starrocks and  I am big data development engineer from beijing  and good at Java programming and and a little knowledge of C + + programming.I hope I can contribute a little to the community. Complete the task

I pick! My name is hiliuxg. I am from Kugou Music and I have worked in big data development for 5 years . I am interested in OLAP, good at JAVA programming, and have been using StarRocks for 1 year.  I like starrocks community and hope to contribute.

I pick!! My name is ZhangRuichao. As a StarRocks deep user, I want to do my best to make the StarRocks great. so here i am.

I pick! beike olap team

I pick! My name is eric, I have been learning on database kernel development for 1 year. I am interested in OLAP, good at C++ programming. I think it is a good chance to join the StarRocks community and read the code.

I pick! My name is LeQu. I have worked in big data area for 2 years . I am interested in OLAP, good at Java programming, and have been using StarRocks for 1 year. I chose to solve this issue because I am interested in Data Lake warehouse .I think it is a good chance to join the StarRocks community and read the code. I like StarRocks and hope that StarRocks will become better and better.


Can only import API, not App

I'm not able to call falcon.App. I've seen in the code that API is a compatability patch set to be removed. falcon.API worked. I have no idea why. Looks to me like both are imported in init. Maybe something todo with Cython?
Environment:

Ubuntu 16.04 lts

Steps to reproduce:
$virtualenv --python=python3.8 venv
$source venv/bin/activate
$pip install cython
$pip install -v --no-binary :all: falcon
$pip install gunicorn
#test2.py
import falcon
app = falcon.App

$ python test2.py
Traceback (most recent call last):
  File "test2.py", line 3, in <module>
    app = falcon.App
AttributeError: module 'falcon' has no attribute 'App'

However it works if I do app = falcon.API
loving falcon otherwise! thanks for library.

Hi @NeilduToit13 !
falcon.API will be renamed to falcon.App in our next major release, Falcon 3.0.
The current stable version is Falcon 2.0.

Haha okay. Yes it did. Thanks for clarifying.

Heh, sorry for realtime editing of my answer. I happened to push it out via keyboard shortcuts a bit prematurely.
I'm going to close this issue, but I'll create another follow-up for the README.rst chicken-and-egg problem.
Besides from just giving it a try, if you're interested in the developmental version of Falcon, you can track our road towards 3.0 in this Falcon 3.0 Roadmap Gist and the 3.0 Milestone here on GH.


Adds an assertion timeout to the end-to-end tests

This is to see if this setup works on the github CI system, as I'm not having luck running things locally.  Many of these tests are failing in weird timeout situations, so this is testing to see if raising the assertion timeout for the end-to-end tests, as a whole, helps the tests succeed more consistently.
Acknowledging that this includes the ActiveMQ branch as well, so I can monitor the queues.

I can revisit this today to clean it up.

@bseeger Do you care if I go ahead and merge this?

Thanks for looking at this @little9!


Fix extra_filters in LineMulti.jsx

The "Multiple Line Charts" visualization is currently not responding to events from the "Filter Box", since the extra filters were not applied correctly. This PR fixes it.
I tested by creating a simple dashboard, and changing the time filter now works as expected.

I'll copy this one to #6838 but do not merge this one.

Copied to @superset-ui/legacy-preset-chart-nvd3 and referenced in latest master.

We actually need this fix in the v0.31 release.

yike, the merge brought in unnecessary changes, I'm going to rebase this change on top of the release branch instead.

Codecov Report

Merging #6868 into release--0.31 will increase coverage by 0.14%.
The diff coverage is 69.67%.

@@                Coverage Diff                @@
##           release--0.31    #6868      +/-   ##
=================================================
+ Coverage          56.17%   56.32%   +0.14%
=================================================
  Files                528      527       -1
  Lines              23465    23537      +72
  Branches            2777     2773       -4
=================================================
+ Hits               13182    13257      +75
+ Misses              9872     9869       -3
  Partials             411      411

Impacted Files
Coverage Δ

superset/assets/src/welcome/DashboardTable.jsx
87.5% <ø> (ø)
:arrow_up:

superset/assets/src/chart/ChartRenderer.jsx
8.47% <ø> (ø)
:arrow_up:

...rset/assets/src/dashboard/components/SaveModal.jsx
46.15% <ø> (ø)
:arrow_up:

.../assets/src/components/VirtualizedRendererWrap.jsx
100% <ø> (ø)
:arrow_up:

...explore/components/controls/ColorSchemeControl.jsx
46.66% <0%> (ø)
:arrow_up:

superset/assets/src/explore/controls.jsx
42.4% <0%> (-0.35%)
:arrow_down:

...uperset/assets/src/dashboard/components/Header.jsx
41.11% <0%> (ø)
:arrow_up:

...ts/src/visualizations/nvd3/LineMulti/LineMulti.jsx
0% <0%> (ø)
:arrow_up:

superset/assets/src/visualizations/Rose/Rose.js
0% <0%> (ø)
:arrow_up:

superset/models/core.py
83.56% <0%> (-0.26%)
:arrow_down:

... and 20 more

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 ce76560...62472dd. Read the comment docs.