[MultiAZ] How to use?

First of all thank you so much for doing https://github.com/aws/aws-parallelcluster/issues/3576
I've read the doc and the release notes and I've also looked at parts of the source of 3.4.0
I see validators on configuration using MultiAZ but I don't see how to set it up.
Am I right that pcluster configure doesn't propose/support it?
Does this mean we need to create our VPCs manually first THEN call pcluster configure
If there is doc, I missed it, could you share the link to it please.
Thank you!

Hi,
With multi AZ support in 3.4.0, it allows users to configure multiple subnetIds under the networking configuration for the Slurm queue.
See https://docs.aws.amazon.com/parallelcluster/latest/ug/Scheduling-v3.html#Scheduling-v3-SlurmQueues-Networking
For example:
Networking:
  SubnetIds:
    - subnet-123
    - subnet-456

subnet-123 and subnet-456 need to be in different AZs.
pcluster configure helps users to generate cluster configuration and create VPC, currently it doesn't provide the options for user to create a queue with multiple subnets.
You can modify the configuration file generated by pcluster configure to add more subnets as you prefer before use the configuration file to create the cluster.
If you don't want to manually create VPC or subnets, as a workaround you can run pcluster configure for multiple times to create VPC or just to automatically create subnets in the existing VPC. Then you can gather the subnets id from multiple runs of pcluster configure and use it for your cluster creation.
Thank you for the heads-up of supporting pcluster configure for multi-az, will discuss with the team about the request.
Thanks

Ok this is what I gathered from the documentation.
Even running pcluster configure wouldn't work since that would create multiple VPCs, we want one VPC with multiple subnets.
Adding subnets to a previously created VPC would work, but the new subnets would be outside the cloudformation created by the wizard which is not ideal.
Thanks for the answer.

Pcluster configure support create subnets under existing VPC. If you choose Automate VPC creation? to no and Automate Subnet creation? (y/n) to yes, you will be able to choose the AZ to create subnets.
Thanks again for your request about improving pcluster configure to configure multiple subnets, we are tracking this request internally.

Hi all, I have another question on MultiAZ usage.
After configuring MultiAZ, is it possible to tell Slurm on which AZ I want to run?
We want not only to be able to run on any AZ, but often also to define on which AZ to run.
Possible? How?
Many thanks,
Richard Ems

A couple of ideas.

Create different queues/partitions for different AZs
Add a Slurm custom resource to the compute nodes to indicate which AZ a compute node and then use that when submitting jobs.

Joe

Hi Joe,
Yes, that looks good, thanks.
Now we have 6 partitions. To be able to run on all 3 zones we would need to go to 18 partitions.
Do you think it is possible to submit a job with Slurm and select the zone using the TAG defined on the zone's subnet?
Thanks for answering!
Richard

Hi Joe,
Yes, that looks good, thanks.
Now we have 6 partitions. To be able to run on all 3 zones we would need to go to 18 partitions.
Do you think it is possible to submit a job with Slurm and select the zone using the TAG defined on the zone's subnet?
Thanks for answering!
Richard

Richard,
I think you might be able to do something with a PrologSlurmCtl script, or maybe a SPANK plugin, but that seems like it might be dicey, and I'm not sure it would work.
Joe


[FEATURE] Allow user to define media types to keep and keep print type by default

I think print styles should be kept by default. But now user can manage himself what types to keep/drop.
Original #68

Hi @Renkas , thanks for the PR! Could you please squash the two commits and then rebase? I've just merged some PRs. Thanks!

@Renkas, I've polished your PR and merged it as #200. Thanks for contributing!


When scroll to the bottom or to the top, cursor is not being moved

Hello! First, thank you for this plugin. It is REALLY awesome and I love to scroll on my buffers with it. But, I noted a problem when I try to scroll to the end or top of a buffer. The scroll occours normally, but the cursor is not being move to the last line or the first line of the buffer when it hit the top or bottom of a buffer.
This video show what I'm talking about, please keep track of the cursor position when I try to scroll using  and  (the first part is not using the plugin and the second is using it):
https://user-images.githubusercontent.com/8826449/113185835-527c6600-922d-11eb-9855-b6f91685cd82.mov

@dewey92 If you don't want to scroll past the end of the file make sure to set g:neoscroll_stop_eof to true. As of right now the cursor won't scroll down to the bottom even with g:neoscroll_cursor_scrolls_alone but I will fix this in the next couple of days and that will probably become the default behaviour so that people that expect the original <C-u>/<C-d> behaviour are not disappointed with the default mappings.

Yeah I had tried g:neoscroll_stop_eof = true before but as you said, the cursor won't move when reaching eol.
Regardless, great work! I finally replaced vim-smoothie with your plugin: your default configuration seems just right to my liking 👍🏻

Hey @karb94 thanks for you reply! I'm using the new options and everything is working great! Thank you for this! 🎉

I just pushed a new version which addresses the issues with g:neoscroll_stop_eof. The defaults should now make <C-u>/<C-d> scroll in the same way as originals. Now you can combine all the available options and they should produce the expected behaviour. It was a big refactor so if something broke or doesn't work as you expect open a new issue.

@karb94 working perfectly! Super thanks!


Add -fdata-section support to cranelift-object

Feature
This is the data object counterpart to https://github.com/bytecodealliance/wasmtime/pull/2218.
Benefit
Same as per_function_sections. From the description of #2218:

This makes it possible for the linker to omit unused functions when --gc-sections is passed.
For cg_clif the executable size savings are quite big. One example is 26MB when a single .text section is used, but 15MB when a single section for each function is used, which is the same as when compiled using cg_llvm. I will probably disable this in debug mode though as it significantly increases the linking time.

Implementation
Similar to #2218
Alternatives
N/A

Fixed by https://github.com/bytecodealliance/wasmtime/pull/9742.


Output of cargo compete open is cleared in Windows Terminal

It seems that the terminal is cleared after printing message when using Windows Terminal.

How to reproduce
cargo new practice  # platform=atcoder
cd practice
cargo compete open

Note:
This works flawlessly in a terminal of VS Code.
Expected output:
$ cargo compete open
     Opening https://atcoder.jp/contests/practice/tasks/practice_1
     Opening https://atcoder.jp/contests/practice/tasks/practice_2

Environment
OS:  Ubuntu 20.04.3  on Windows10 21H1 (WSL2)
Windows Terminal: 1.10.2383.0
cargo-compete: 0.9.0

P.S.
I haven't checked detail yet, but echo 65001 > ~/.config/wslu/oemcp solved this issue.
It seems that changing the encoding causes a terminal reset.
Ref:
https://github.com/wslutilities/wslu/issues/199


[DO NOT MERGE] Use image shortcodes in dess instead of URL fetching

What I did
Replace image markdown with image shortcodes for faster loading and for when the image urls don't work anymore.

How I did it
See local changes

Do not merge

Not ready yet, please give feedback as I go

Having them locally also helps with resolution! (first image is online fetching and second image is downloaded)

Looking good so far! Good job!


Add Policy States and events tests

You can close this PR since it is been moved here with some additions : https://github.com/Azure/azure-sdk-for-net/pull/15976/


Clarity in upgrade process

When following these instructions on a Blazor webassembly starter project, it is not clear if project file changes should be made to the Client project, Server project or both. The page only states that If updating a "Blazor WebAssembly project, skip to the Update Blazor WebAssembly projects section" and then goes to list the changes required in the project file. Further, it does not make any mention if the changes can be made by using the Visual Studio option "Project-> 'ProjName' Properties" option, or it should not be used at all.

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

ID: b031caa0-c62b-37da-46c3-59043256cac2
Version Independent ID: bdf98c9a-96f6-79fd-a650-5c0385d1a62b
Content: Migrate from ASP.NET Core 3.1 to 5.0
Content Source: aspnetcore/migration/31-to-50.md
Product: aspnet-core
Technology: aspnetcore-migration
GitHub Login: @scottaddie
Microsoft Alias: scaddie

Thanks for the report @dajma00 ... I'm a bit buried in Blazor WASM security code for the next few days, so I won't be able to address this until probably next week. In the meantime if you're struggling to get a 3.x hosted Blazor solution upgraded to 5.0 preview, I recommend that you create a fresh default hosted Blazor solution from the 5.0 template and compare directly to the app that you're seeking to upgrade in order to determine what must be done. Thanks again for the report, and I'll get back to this as soon as I can to describe hosted Blazor solution migration.

Hi Luke,
No issues. I just created a 5.0 project and inserted my code into it and everything just works fine.
But it would help others were there a bit more clarity in the documents.
Regards,
Amjad.

From: Luke Latham<EMAIL_ADDRESS>Sent: Thursday, October 8, 2020 6:50 PM
To: dotnet/AspNetCore.Docs<EMAIL_ADDRESS>Cc: Amjad<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [dotnet/AspNetCore.Docs] Clarity in upgrade process (#20143)
Thanks for the report @dajma00https://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdajma00&data=02|01||8a1ccd0978f14bc08ca808d86b91233b|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637377618384167015&sdata=TrmBKkHN8I6byfuhNnUy%2BIjAPcpYXzqp6qktPa7hyd4%3D&reserved=0 ... I'm a bit buried in Blazor WASM security code for the next few days, so I won't be able to address this until probably next week. In the meantime if you're struggling to get a 3.x hosted Blazor solution upgraded to 5.0 preview, I recommend that you create a fresh default hosted Blazor solution from the 5.0 template and compare directly to the app that you're seeking to upgrade in order to determine what must be done. Thanks again for the report, and I'll get back to this as soon as I can to describe hosted Blazor solution migration.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2FAspNetCore.Docs%2Fissues%2F20143%23issuecomment-705582138&data=02|01||8a1ccd0978f14bc08ca808d86b91233b|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637377618384177015&sdata=Zp6%2FxNygcgqovVDynTMEwCQSW%2B3E2wUH6%2FWNVHpqIMI%3D&reserved=0, or unsubscribehttps://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJIAZXOBANJFA4YP6YBNXD3SJW7SZANCNFSM4SIFACFQ&data=02|01||8a1ccd0978f14bc08ca808d86b91233b|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637377618384187009&sdata=EiOIfwaibaHnAVxvhyvPET09%2BKOWOlv6ciZrCScTa6M%3D&reserved=0.

Yes, I agree. I generally wait until RC1 lands before testing framework code locally and updating many docs because the product unit fiddles with the code quite a bit before RC1. Doc updates are too hard to keep up with pre-RC1. The downside to waiting so late is that the updates hit me all at once for at least local testing and often for doc updates themselves. It generally works out ok tho ... in the past, there has been enough time between RC1 and GA that all of the updates get in on time.
After I dig out of all of the 5.0 security updates, I'll get right on this. I hope to address it by early next week. Thanks again for the report.


Given ability to limit stack and program counter

Stack limit can be set by passing the value to machine constructor as the first argument or setting its default value as 128.

Program counter limit can be set by passing the value to machine constructor as the first argument or setting its default value as 1000.

But program counter doesn't know how many lines executed.
It points to the address(in our case it is the line number) of the current instruction.
That is the reason I have added that functionality in lines

Lines is a very loose abstraction primarily focused on the collection. Program Counter doesn't know how many lines executed, but it can keep track of how many times it was asked to update itself which is really what we are after.
Also, doing it this way, causes very few changes to anything else. It is highly localised and doesn't change code elsewhere.


Harmony: make activity optional

During voluptuous upgrade I removed a default from a vol.Required, making the Required actually be enforced. Based on the code it looked like it was always meant to be required optional. Seems like it was not. This fixes it.
Reported: https://community.home-assistant.io/t/0-64-over-1000-integrations-new-homekit-bmw-august/44834/2?u=balloob

You're right. Typo.


Wrong result with rate/irate functions.

Hi~
Promethues painted a strange graph with rate/irate funciton. So I checked data, and find out it decreased at a certain point, as below.
512408305 @1525061864.282
512410956 @1525061874.282
512417456 @1525061884.282 <== greater?
512416256 @1525061894.282 <== less?
512418830 @1525061904.282
512421378 @1525061914.282
512423749 @1525061924.282

I have confirmed that this is a problem caused by adding the data just prior to the decline to correct the data. The data correction method is implemented differently, and the graph is shown normally.
In the case of irate, we changed the loop to calculate the data until the correct result is shown.
I attached the graph before and after the change.
Thanks.
Chan.
Before

After

Hi Brian
Thanks for your message.
I do agree with your mention "Counters only go up and reset". This must be.
However, mysql still has a bug that shows incorrect results. You can check the contents in the bottom bug report.

https://bugs.mysql.com/bug.php?id=90351

The Linux OS also shows reduced count. (Not often, but somtimes)
Sometimes, an alarm is triggered by incorrect OS utilization detection like network usage.
Do you have a good idea on how to properly refine your data in this situation?
If it is a broken count, it may be correct to drop it. However, the logic to correct the data in the RESET case seems to make more sense, so I have only slightly changed the way we do calculations. The calculation algorithm does not have a major effect on the current result and will guarantee the data even in the case of the broken counts mentioned above.
I think that prometheus becomes more stable.
Tell me what might be a problem.
Best Regards,
Chan.

Do you have a good idea on how to properly refine your data in this situation?

You might use something like clamp_max.

The calculation algorithm does not have a major effect on the current result and will guarantee the data even in the case of the broken counts mentioned above.

Doing so will break working counters though.

You might use something like clamp_max.
It's the same story that I can not use rate or irate.

Doing so will break working counters though.
The rate result has already spoiled the results. It is more reasonable to calculate the resultValue in a drop or stepwise fashion than a ridiculous error in rate as my code.

You might use something like clamp_max.

It's the same story that I can not use rate or irate.

Doing so will break working counters though.

The rate result has already spoiled the results. It is more reasonable to calculate the resultValue in a drop or stepwise fashion than a ridiculous error in rate as my code.

The rate result has already spoiled the results. It is more reasonable to calculate the resultValue in a drop or stepwise fashion than a ridiculous error in rate as my code.

As far as Prometheus is concerned your counter reset and then increased by 512416256 in the space of 10 seconds. We have no way to tell if your counter really had such a reset and spike, or if your counter is broken. We have to presume that your counter is working correctly, otherwise everything falls apart and we'd lose data on frequently resetting counters.
I'm sorry that you have a broken counter, but we can't break other users in order to workaround your breakage.
If you wish to do custom counter logic, the idelta function and recording rules allow for that.

The proposed code does not affect RESET.
In the case of COUNT, the case where the data is decremented should never occur except reset. If spike, data will never decrease. I think that the data of other people seems to be broken.
Prometheus is used to monitor mysql, but before mysql bugs are fixed, I can not help but see a lot of false positives.
With this case
512408305 @1525061864.282
512410956 @1525061874.282
512417456 @1525061884.282 <== greater?
512416256 @1525061894.282 <== less?
512418830 @1525061904.282
512421378 @1525061914.282
512423749 @1525061924.282

Original result is
512423749 - 512408305 + 512417456

Fixed result is
512408305 @1525061864.282
512410956 @1525061874.282
512417456 @1525061884.282
resultValue += (512417456 - 512408305)

512416256 @1525061894.282 <== less?
512418830 @1525061904.282
512421378 @1525061914.282
512423749 @1525061924.282
resultValue += (512423749 - 512416256)

=> (512417456 - 512408305) + (512423749 - 512416256)

Do you really this this is messing up the data? The original is rather strange.

With RESET case
512408305 @1525061864.282
512410956 @1525061874.282
512417456 @1525061884.282 <== greater?
.. loss ..                <== reset
10 @1525061894.282
40 @1525061904.282
50 @1525061914.282

Original result is
50 - 512408305 + 512417456 = 9201

Fixed result is
(512417456 - 512408305) + (50 - 10) = 9191

It has some influence because it can not know the exact time to reset.
However, original data is not correct because it is not known whether the value immediately before reset is the maximum value at the correct time.

The proposed code does not affect RESET.

The whole point of your code is to change how counter resets are handled. In your final example there it's failing to count the 10 events immediately after the reset which would be a regression. 9201 is what that should be returning.

You think original result is correct one?
In fact, prometheus does not know the exact time to reset.
When a reset occurs, prometheus also estimates the data but does not output the correct result.
For positive processing, all cases where counts are decreasing should be dropped.
As I said above, this is currently a bug in mysql, and I can not guarantee that it never happens on linux metric, cpu/network.

Prometheus rate function computes with the data already lost from the last maximum value to the reset point

As I said above, this is currently a bug in mysql, and I can not guarantee that it never happens on linux metric, cpu/network.

A bug in MySQL should be fixed there, and bugs in any other code should be fixed there too. Rate is always fundamentally an estimation, and your proposal makes our estimate worse so I'm afraid we cannot accept it.

fundamentally an estimation

The code I present, and the code of the current prometheus, is also an estimation.
It is only an aim to find an accurate approximation. In the example given above, if you are concerned about the number 10, add the logic to add 10. (If so, let's add this part.)
It is not understood to be an intended result to be able to input data in an uncleaned form, but to show the calculation results in nonsense numbers.
Whether the count value can not be reduced or whether it is fixed the result, Prometheus also thinks it needs modification. In the current situation, if I use rate / irate, not only mysql, but linux is also suitable for monitoring. The delta is likely to show more missing data.
Think about what is better suited to your service from showing proper monitoring results.

In the example given above, if you are concerned about the number 10, add the logic to add 10. (If so, let's add this part.)

We already have that logic, your PR proposes to remove it.
We're going around in circles here, I'm afraid your PR is not going to be accepted as it breaks existing semantics.

We already have that logic, your PR proposes to remove it.

I've seen it included in the logic. However, there is another bug that shows the wrong data results in the wrong data entry. I do not think I completely broke the data correction. If necessary, you can think of a way to skip a slight data reduction.
I think that prometheus is not suitable for intermittent monitoring of mysql and linux, which is currently a logic, and which may be a fix at this time.
It is better to drop data instead of correcting it. I am talking about service improvement points, and I understand that prometheus is perfect, so just put in clean data by application level.

For the reasons I've already indicated, I cannot accept this PR.


[QUESTION]The live module can't refresh as expected.

Hi there.
I want to use rich.live module to show something in real time and I test it with your sample code.
import time

from rich.live import Live
from rich.table import Table

table = Table()
table.add_column("Row ID")
table.add_column("Description")
table.add_column("Level")

with Live(table, refresh_per_second=4):  # update 4 times a second to feel fluid
    for row in range(12):
        time.sleep(0.4)  # arbitrary delay
        # update the renderable internally
        table.add_row(f"{row}", f"description {row}", "[red]ERROR")

However, it did't refresh, just show the final result for me.
Could you give me some ideas how to use this module correctly?

import time

from rich.live import Live
from rich.table import Table
from rich.console import Console

table = Table()
table.add_column("Row ID")
table.add_column("Description")
table.add_column("Level")

console = Console(force_terminal=True)

with Live(
    table, console=console, refresh_per_second=4
):  # update 4 times a second to feel fluid
    for row in range(12):
        time.sleep(0.4)  # arbitrary delay
        # update the renderable internally
        table.add_row(f"{row}", f"description {row}", "[red]ERROR")

Try this out to see if forcing the console to be a terminal has any effect.

This snippet runs fine on my own setup. What kind of terminal are you running this snippet on?
I have tried it on vscode terminal, powershell and windows terminal, none of them can display correctly.

Can you run:
from rich import get_console, inspect
inspect(get_console())

I modified code.
import time

from utils.rich.live import Live
from utils.rich.table import Table
from utils.rich.control import Control
from utils.rich.console import Console

table = Table()
table.add_column("Row ID")
table.add_column("Description")
table.add_column("Level")

console = Console(force_terminal=True,color_system='truecolor')

with Live(table, console=console, refresh_per_second=4) as live:  # update 4 times a second to feel fluid
    for row in range(12):
        time.sleep(0.4)  # arbitrary delay
        # update the renderable internally
        table.add_row(f"{row}", f"description {row}", "[red]ERROR")
        live.console.print(Control(""))

Now it can display in real time, just flickers

Can you run:
from rich import get_console, inspect
inspect(get_console())

And paste printed results.

Here is the result

The issue is that it is a dumb_terminal this means that rich treats it as if the terminal doesn't have the ability to use ANSI codes.

As a quick work around
In powershell run:
$env:TERM="SMART"

Then re-run the initial code.

As a quick work around
In powershell run:
$env:TERM="SMART"

Then re-run the initial code.

Thanks a lot, it works!

@willmcgugan Is there anything that the console is_dumb_terminal should be added for Windows? Or should
$env:TERM="SMART"

just be documented in a "common issues"  section of documentation to force a non dumb terminal.

Acutally, this reference to console environment variables should be enough.

Thanks for diagnosing that @nathanrpage97
is_dumb_terminal will be True if TERM is set to "dumb" or "unknown". As far as I know, this convention should apply to Windows.
@DarrenIce Do you recall adding the TERM environment variable? Could have been a workaround for an app that broken on Windows.

Thanks for diagnosing that @nathanrpage97
is_dumb_terminal will be True if TERM is set to "dumb" or "unknown". As far as I know, this convention should apply to Windows.
@DarrenIce Do you recall adding the TERM environment variable? Could have been a workaround for an app that broken on Windows.

Yes, I add two lines in my code, and now it can work as expect.
self.console = Console(force_terminal=True,color_system='truecolor')
self.console._environ['TERM'] = 'SMART'


[Feature][UI Next]Improve internationalization of tenant management

Purpose of the pull request
please link [#7894 ]

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Codecov Report

Merging #7997 (ddfa096) into dev (b9b0722) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #7997      +/-   ##
============================================
- Coverage     41.28%   41.21%   -0.08%
+ Complexity     3722     3716       -6
============================================
  Files           637      637
  Lines         27128    27128
  Branches       3081     3081
============================================
- Hits          11200    11181      -19
- Misses        14852    14866      +14
- Partials       1076     1081       +5

Impacted Files
Coverage Δ

...er/master/processor/queue/TaskResponseService.java
50.00% <0.00%> (-18.19%)
:arrow_down:

...rver/master/processor/queue/TaskResponseEvent.java
93.54% <0.00%> (-6.46%)
:arrow_down:

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
50.70% <0.00%> (-2.82%)
:arrow_down:

...olphinscheduler/plugin/alert/email/MailSender.java
72.72% <0.00%> (-1.40%)
:arrow_down:

...inscheduler/common/task/sqoop/SqoopParameters.java
74.00% <0.00%> (ø)

...he/dolphinscheduler/common/enums/SqoopJobType.java
88.88% <0.00%> (+88.88%)
:arrow_up:

Continue to review full report at Codecov.

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


samples: http_server: enable usage of HTTP/2 with web browsers

Update HTTP server sample application to allow (optional) usage of HTTP/2 when the HTTP client is a web browser.
This involves:

Add an application level Kconfig to use ALPN for negotiating the HTTP/2 protocol with the client
Update the enabled cipher suites and certificates to use a cipher suite that is not in the "prohibited list" of RFC9113 Appendix A
Add scripts and additional CMake targets to enable a user to generate their own CA and server certificates (since it would not be recommended for a user to add my generated CA certificate to their browser's list of trusted CAs)
Add some basic instructions in the README on how to generate the certificates and add to browser.

Tested using Chrome and Firefox on Linux, it works in Chrome but it seems that in Firefox even adding the CA certificate to the trusted list is not enough since it comes from a certificate authority not known by Firefox.
The option is disabled by default, partly due to the issue with Firefox but also because visiting the HTTPS web page via a web browser without trusting the CA certificate results in a security error with no option to "proceed anyway" like you get with HTTP/1.1.
While some additional cipher suites have been enabled in mbedtls, this has a fairly minimal impact on the footprint (no change to RAM, approx +6.5k flash on an STM32H753 target, which was originally 311k with HTTPS service enabled).

advice on how to resolve this?

Unfortunately no way around this if we want to provide credentials in DER format, will need to ask someone to override this CI check when the PR is ready for merging.

@carlescufi can you help override the binary der cert file compliance issue

@carlescufi can you help override the binary der cert file compliance issue

another admin (non-Nordic) would need to do that, as per four-eye rule

another admin (non-Nordic) would need to do that, as per four-eye rule

I thought this was only when the PR author is from the same company, which is not the case here.

another admin (non-Nordic) would need to do that, as per four-eye rule

I thought this was only when the PR author is from the same company, which is not the case here.

You are correct. My mistake, sorry.

@carlescufi when you have time, please help force merging this (there are the binary cert files in the PR which needs more powers).

@jukkar @mrodgers-witekio I think something broke in main related to this PR. Please have a look
https://github.com/zephyrproject-rtos/zephyr/actions/runs/12259651345/job/34202378212

to repro
$ west build -p -b native_sim/native -T tests/net/lib/http_server/tls/net.http.server.tls
...
ninja: error: '/home/fabiobaltieri/zephyrproject/zephyr/samples/net/sockets/http_server/src/ca.der', needed by 'zephyr/include/generated/ca.inc', missing and no known rule to make it

@jukkar @mrodgers-witekio could one of you please confirm you're looking into this? This is causing tests to fail in main. Thanks!

@kartben looking into this now


bug: stone-brick vs stone-path crash on shortage page

Put down some stone bricks to create a path. Have no extra stone bricks in the item network or inventory.
CTRL-C to copy the path. Paste.
View the shortage page. Crash.
The problem is that the "alert" stuff is showing a shortage of "stone-path" and not "stone-brick".
"stone-path" isn't an item, so the lookup fails.
As far as I know, this is a special case.
The other similar case is "concrete", but that has the same name for the tile and item.
Probably ought to translate "stone-path" to "stone-brick".
Another option is to ignore any "shortage" that isn't defined in the item_prototypes.

Fixed by #61

Nice catch!

Also not handling "item-request-proxy", which is used to request a module for a factory.
I think I should add something for missing repair kits as well.

All fixed.

Thanks!


Obsidian/shared.css: Restore !important flag for required parts

Some page doesn't properly themed after removal of !important flag such as Scroll Panel

Before this commit:
https://imgur.com/NozYy23

After this commit:
https://imgur.com/KpvBoZp

This commit restored following components' !important flags
Settings Container
Scroll Panel
Loading Screen

@EMERALD0874
If you are fine, could you check this please?
Thanks!


Docs analytics

Contributing guidelines
refs https://github.com/coreos/docs/pull/913, https://github.com/coreos/rkt/pull/3508
This Pull Request adds the following:

A Google Analytics beacon pixel to the .md documentation (.md) files in the etcd repository.
A script to automate the addition and removal of this pixel.

This pixel gives helps us figure out just who and how many people are reading our docs in GitHub as compared with coreos.com. Essentially revealing which docs developers are reading.
Knowing this information helps us figure out who is reading our docs so we can make sure those stay accurate and up to date.
CC: @joshix @robszumski

I have to echo the concerns raised in the rkt issue. It will certainly upset etcd users in exchange for only /potentially/ improving documentation (after paying off the initial debt of gunking up the md files).
How will that improvement be measured, so if it turns out to offer no benefit in terms of documentation quality, the tracking can be removed? There are different classes of users accessing different types of documentation, how will they be bucketed? If a page is popular, does that mean it deserves more focus (and what, precisely should that focus be?) or just that it's the first thing people click because it has a good page rank? etc etc

Knowing this information helps us figure out who is reading our docs so we can make sure those stay accurate and up to date.

I feel that all the docs in the etcd repo should be accurate and up to date, not just what's popular (and it's easy to let even the popular stuff get out of date-- this won't tell you whether it is)
I'm skeptical.
/cc @xiang90

As you already have the doc data on CoreOS.com, can you take about how you utilize that data? Any concrete examples would be helpful.
Also who will have accessibility to the data?

@ElijahCaine I'm supportive of you trying this out. If the data is not useful or actionable (i.e. it is never used for anything) please come back and remove it. If there are user issues with this we will have to revisit our privacy/data policy overall as this would fall within that.
Thank you for trying to gather data to try to make things better.

@ElijahCaine I are closing this for this now.
Let's revisit in the future.
Thanks!


Open source license

Would it be possible to add a license to this project?  For example the Apache 2.0 license.
Thanks!
Sean

@smccauliff Added the MIT license. :)


windows_package remove (exe) doesn't pass the timeout value

chef/lib/chef/provider/package/windows/exe.rb
remove_package doesn't pass the timeout value from the resource into the shell_out command so the default 600 second timeout is always used.
The install_package method properly passes it.

Fixed by #5250, will be in the next release.


python 3.8.5 compatibility

is PAN hunt script conceived to work only with python 2.7?

Yes, and no plans to migrate to python 3 at the moment.

but we must take into aacount if this script is going to be used in a **PCI environment it should be supported with a supported version of Python, version 2.7 is EOL according to[https://www.python.org/dev/peps/pep-0373/].

You can use a binary that I have provided, so you don't need to install Python on the target.

You can use a binary that I have provided, so you don't need to install Python on the target. Got it, but there are some environments that is not possible to execute a program brought from outside the CDE it is more probable they let a properly audited python script from a recognized source. Thanks for the reply @bli-dn we will try to port the script to python-3 to see if it work.

Hi,
Any update on this?
@rickespana, have you worked further to add python 3.8 compatibilities?

I've created a pull request for this. https://github.com/Dionach/PANhunt/pull/33 it's not a major change but i hope it helps others

Thank you @dawesc ! I will review.

Sorry, some errors testing those changes, so can't merge at present.

I've created a pull request for this. #33 it's not a major change but i hope it helps others

Has this been fixed or is it working in Python 3?

Is there any way we can get involved in this and help you fix the errors?

Sorry, some errors testing those changes, so can't merge at present.

I would be happy to try and correct the issues, could you provide some details on how to reproduce them?

@dawesc I think I have found (and corrected) some of the errors, would you prefer that I send you a pull request, or that I open a new pull request based on your fork?
(There were some issues with things nested in zipfiles, and there are some fairly nasty file descriptor leaks for some failures like 'file is not a valid CFB'.  I'm not convinced that the latter is new with your changes, however it definitely crashed panhunt in my test environment.)


i18n(fr): update configuration-reference.mdx

Description (required)

Removes the blank line between the sentence and the list (see #9933)

Note: #9933 also fixes the escaping for ``` but this was already fixed in the French translation (see #9934).

Related issues & labels (optional)

Suggested label: i18n

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

fr
reference/configuration-reference.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

LGTM!


Add missing NSIS commands

This PR adds PEDllCharacteristics and PESubsysVer, two commands introduced in NSIS 3.0 Beta 1

Thanks! PESubsysVer apparently was added in 3.0b2, though.


Crash when moving Basic Filters while editing

Is there an existing issue for this?

[x] I did not find any existing issues.

Current Behavior
I was editing 9 Basic Filters (the editing window was open), and I moved the Basic Filters from my hotbar to my inventory while the filter editing window was still open. The server immediately crashed. After the server rebooted, the filters had disappeared from my inventory.
Expected Behavior
What I expected to happen was either the hotbar to be locked, preventing item movement, or the editing window to automatically close when moving the filters.
How to Test

Open the filter editing window for several Basic Filters (I had 9 filters).
While the filter editing window is still open, move the Basic Filters from your hotbar to your inventory.
Observe that the server crashes immediately after moving the filters.

Local Environment
Minecraft Version: 1.21.1
Minecraft Version ID: 1.21.1
Operating System: Windows 10 (amd64) version 10.0
Java Version: 21.0.4, Eclipse Adoptium
Java VM Version: OpenJDK 64-Bit Server VM (mixed mode, sharing), Eclipse Adoptium
Any Additional Information?
Crash Report : https://gist.github.com/Volpamonhub/eac29e62e089b7a390d4f53de0eeeac8

Just had this issue happen to me and the steps to replicate were exactly the same.


Chloe university controller

In this PR, we add an endpoint /api/university/get that can be used to get a list of universities that match a given name.
dev deployment here: http://team01-chendashi-dev.dokku-07.cs.ucsb.edu
closes #15

LGTM


Should forward be treated as a const lvalue?

While writing about 708 fixing perfect forwarding issues, I came up with the following code:
void invoke_twice(forward auto f, forward auto... args){
	std::invoke(f, args...);
	std::invoke(f, args...); // Definite last use, automatically forwards
}

int x = 0;
auto increment = [](auto& val){ ++val; };
invoke_twice(increment, x);

Should the example above work as an std::forward version would?

In the current draft, for forward, we have:

x is treated as a const lvalue, except that each definite last use of x preserves the argument’s cv-qualification and value category

With this current wording, the function would not compile, as the lambda parameter expects a non-const lvalue.

x is treated as an lvalue, preserving its cv-qualifications, except that each definite last use of x also preserves the argument’s value category

Would something like this break any contract or introduce any ambiguity to the current proposal?

Thanks Joel. This is a good example, there are a couple of things going on here.
First, starting with principles, there are two relevant design goals involved:

For any parameter passing option we be able to compile the callee body in isolation and know it's correct.
A forward parameter is forwarded at most once. Forwarding it a second time could end up forwarding a different value if the first forward modified the parameter, which seems like a bug farm and didn't come up in the examples I surveyed.

That said, a potential extension to section 1.3.6 about optional call-site annotation could be to allow forward or non-definite-last uses of a forward parameter to explicitly forward in cases like the above. I'll add it to the paper. Thanks!
BTW, in this example's parameter f, shouldn't at least that one in f rather than forward f? I'm trying to imagine a situation where an invoke-like function would care whether a Callable (essentially) is itself const or not and/or an lvalue or rvalue.

For any parameter passing option we be able to compile the callee body in isolation and know it's correct.

I believe it should be always correct if we only forward in the definite last use.
I also believe the only issue is with the "treated as a const lvalue" clause for all other uses. Forwarding them would yield different results, as you've pointed out. So adding the "explicitly forward in cases like the above" clause might yield the incorrect result with rvalue parameters.
But treating them as the correct lvalue equivalent would not yield different results. For instance, consider this C++20 code:
void invoke_twice(auto&& f, auto&&... args) {
  std::invoke(f, args...); // We don't forward
  std::invoke(std::forward<decltype(f)>(f), std::forward<decltype(args)>(args)...); // We perfect forward
}

struct append_and_print {
  void operator()(std::string& str) const {
    str += "_X_";
    std::cout << str << '\n';
  }

  void operator()(const std::string& str) const {
    std::cout << str << '\n';
  }
};

void case1() {
  std::string s = "case1";
  invoke_twice(append_and_print{}, s);
}

void case2() {
  invoke_twice(append_and_print{}, "case2");
}

The output is:
case1_X_
case1_X__X_
case2
case2

For the rvalue case, the lvalue instance on the first use is const-qualified, while in the non-const lvalue, it stays mutable and appends twice. With the current wording, it would only modify once, at the last use. (Note: my wording is also incorrect, as it would treat the rvalue as a non-const lvalue)

On the "compile the callee body in isolation" part, forward would only work with templates, correct?
I believe since forward would replace forwarding references, and they only happen with templates, the compiler would still have to compile multiple forward function instances for the lvalue and rvalue cases, and different qualifiers.

I'm trying to imagine a situation where an invoke-like function would care whether a Callable (essentially) is itself const or not and/or an lvalue or rvalue.

If in implies immutability, any functor with internal state (i.e. operator() not const-qualified) would fail.
The rvalue qualification is more of an obscure case, but still valid in C++:
auto concat(std::vector<int> v1, const std::vector<int>& v2){
    v1.insert(v1.end(), v2.begin(), v2.end());
    return v1; // Assuming NRVO
}

struct prefixed_shuffle_printer{
	std::vector<int> prefix;

	// Concatenates a prefix with the input vector, then shuffles it.
	// prefix is copied, then v is appended.
	void operator()(const std::vector<int>& v) & {
		print_shuffled(concat(prefix, v));
	}

	// We can optimize a copy by not copying our prefix.
	void operator()(const std::vector<int>& v) && {
		print_shuffled(concat(std::move(prefix), v));
	}
};


No itsdangerous 1.0.0 version exists

Need to fix requirements.txt
https://pypi.org/project/itsdangerous/#history

Well, in fact, need to fix dependency from Python 2.7...
It seems this project is abandoned, though


corrected spelling error in line 138 for hybrid

added "b" in the word hyrid in line 138 based on the comment from ChicagoJay.
Closes #817

@kenwith - are you able to merge the content on this PR for a simple typo error? This is based on issue #817?
Thank you.

Nice, simple and straight-forward PR.
Correctly placed commit text. Good job. 👍


White flicker while navigating to screen to screen

In dark mode, when I navigate to another screen, there is a white background glimpsing making the transition a bit weird.

Hi! Are you using the animations core dependency?

I believe this is not an issue for us,  it’s on the official compose navigation itself.

Yes, I'm using the core dependency.  Okay is there any workaround for this when I use destinations core library?

Can you paste a video of what you see? I’m just curious.
If there is an issue with official nav library, then you should open an issue on their official issue tracker.

https://user-images.githubusercontent.com/32787553/197248160-95c8de55-3e4f-4a6a-8dcf-5298893cbf05.mov
One is a vertical transition the white background showing up in dark mode.
https://user-images.githubusercontent.com/32787553/197248338-7775005c-92e7-4e0b-91c6-cc11e26c058f.mov
Another one horizontal transition there is white flicker happening in the bottom of the screen.

Hey @yosuke65 !
Yes, I see it. I would ask a question on stack overflow or kotlin slack (there is a #compose channel).
These animations are not handled by this library, all we do is use their APIs directly.


TP-1109 Prevent Duplicate Preemptive Transactions in a Workbasket

TP-1109 Prevent Duplicate Preemptive Transactions in a Workbasket
Why
At the moment, the commodity code change handler does not control for situations where a business rule (e.g ME32) affects the same model instance twice. This ticket aims to fix that .
What

Can this be closed @kintisheff?


Not able to see/click "Next" button in AppCenter - Add the Push SDK to your app

What App Center service does this affect?
Add the Push SDK to your app
Describe the bug
Not able to see/click "Next" button in AppCenter - Add the Push SDK to your app
To Reproduce
Steps to reproduce the behavior:

Go to 'Push' in AppCenter
Click on 'Continue with AppCenter'
Scroll down to 'bottom of screen' and I am not able to click on the next button
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Mac OS Mojave Version 10.14.6 (18G84)
Browser : Safari
Version : 12.1.2

Additional context
attached screenshot

@amrutha786 Thanks for pointing this out, I’ll get it fixed.

Hi, @amrutha786! Thanks for reporting this. Do you know what your browser zoom level is when you saw this issue? Thanks!

Hi @kkamperschroer ; I was using Safari on a actual zoom size, i haven't changed the zoom levels

@amrutha786 this was fixed a while back, closing the issue.


ci: add write permission for version-check job

Should fix the error from version-check job:
Error: Resource not accessible by integration

replaced by #645


Do you support a 32-bit Windows version?

Some computers in our company use 32-bit Windows, so they can't run it. It would be great if you could support 32-bit Windows. Thanks you.

Hello,
It should be possible, i will try when i get back end of november.

Would you mind trying the pre-release https://github.com/erebe/wstunnel/releases/tag/v7.9.4 , the archive  wstunnel_7.9.4_windows_386.tar.gz  and let me know if it is ok for you ?
P.s: Out of curiosity, what kind of company are you working in to still have 32bits computers ?


Please, confirm pull request to the backport 1.18.2

https://github.com/Sparkierkan7/thermite_mod/pull/22

#22


Support multiple Nest devices

Summary
I've configured this tool against a home that has two Nest devices: First Floor and Second Floor.  While the data seems to be harvested properly, the labels in the dashboard are inside for both First Floor and Second Floor.  The data coming back from google does have the name of the Nest devices in it.  I'm hoping there is a way to present that out with the pronestheus exporter.
Questions

How does pronestheus let prometheus know there are two Nest devices at harvest time?  It's not clear to me how two data sets are returned.  I think I only see data for one of the Nests when I curl localhost:9777/metrics
Where do the ID's (shown in the prometheus screen capture below) come from?

PS

Pronestheus is brilliant.  Great idea!

Try https://github.com/grdl/pronestheus/pull/12 -- it adds a "room" label which matches the room name as displayed/edited by you in the Google Home app.
It's a bit confusing:

The Nest app's settings (and display) of each thermostat have a Where section, which contains a Label field. The value of the Label field is what's reported by pronestheus as the "label" label in the timeseries.
The Google Home app groups devices by Room. Each thermostat is assigned to a room. The above PR reports the room as the "room" label of the timeseries.


remove hashbrown from id_to_pattern API

Hello! I've been having so much fun with egg, I thought I would contribute a little.
I've replaced a HashMap with a FnMut in the API. I tried using the HashMap version and found I had to use the same verison of hashbrown with fxhash as egg.
This is a breaking API change and requires a version bump.

Not a bad idea, but I'm okay with requiring the use of hashbrown and the correct hashfn. Thanks though!


[Solution] Cannot import name 'guidedFilter' from 'cv2.ximgproc'

https://github.com/chflame163/ComfyUI_LayerStyle/issues/141
Hey guys! i have managed to solve this issue: Cannot import name 'guidedFilter' from 'cv2.ximgproc'
Install:
python.exe -m pip install opencv-contrib-python-headless
And open cmd prompt and install this:

I am also on my discord channel to discuss bugs and issues as I am a starting python/comfyui engineer, feel free to drop by https://discord.gg/Z44Zjpurjp and say hi
More than happy to chat.

@daxcay Good job!
I've just sent you a friend req on discord!

good job


Suggestion: Remove the executors

Executors are useful for parameterized execution environments that will be reused. As this executor is used exclusively within the continue job, it should not be necessary and can sometimes be confusing for users.
Instead, I would recommend setting the docker executor and image statically on the continue job, including the same tag parameter for the image version tag.

Made this update locally but need access to create patch-2 branch to create PR to merge to master.


Update dependencies and switch to using NPM lockfiles.

Jekyll has been held back at v3.4.5 due to jekyll/jekyll-redirect-from#159.
The ESLint indent rule has set to v3 compatibility mode, pending other fixes.
Change from using shrinkwrap to NPM lockfiles (similar to how Gemfile.lock functions).

I'm happy to expand on my rationale for NPM lockfiles if needed, but basically they don't ship when publishing to NPM (and hence don't need to be moved to a subdirectory like shrinkwraps), and integrate very nicely with npm update.
Closes #22744.

Overlapping with : https://github.com/twbs/bootstrap/pull/22744

@bardiharborow: all is good, just one question. Why did you remove anchor etc? Aren't those used?

Also, I wanted to point out we should specify clean-css's compatibility for IE to be safe.

@XhmikosR I didn't (?). I've just updated anchor fromv3.2.2 to v4.0.0, etc.
Is there something with upgrading clean-css that I've missed?

You are right, it's just that someone had switched to the non-minified versions and now it's again the minified versions. It beats the point though because of the number of lines changed but anyway, maybe it won't happen again in the future.
Re clean-css: it's not exactly related to this PR; I just noticed that with the switch to npm scripts someone missed to specify the minimum IE compatibility option. This should be fixed asap to ensure we don't hit any weird issues.


Ubuntu (upstart) / Debian (sysvinit) split

Even is Chef Server 12 is not officially supported on Debian, the Ubuntu 11.04 packages (glibc compatible) work perfectly after applying this PR's changes.
I just introduced an additional test on platform to split the Ubuntu and Debian cases. Of course, some more check could be added once Debian "Jessie" (with systemd) is out.
And yes, it's possible to switch from sysV to Upstart on Debian Wheezy. But while sysV is the default and bullet-proof on any environment, it's definitely not the case for Upstart. For example, switching to it on the official EC2 AMIs from Debian prevents the instances to be back after a reboot. On a bare-metal install, Upstart works but some core services (resolvconf, for example) don't.
FYI, the same discussion happened for Chef 11 : https://github.com/opscode/omnibus-chef-server/pull/70

I would also like to see this applied to be able to run Chef Server 12 on Debian 7 although it's not officially supported.
@gplessis I think the PR has to be updated to apply to latest master.

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
Non-GitHub Verified Committers
There are 1 commit author(s) whose commits are authored by a non-GitHub verified email address. Chef will have to manually verify that they are authorized to contribute.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@gplessis

Please sign the CLA here.

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
Non-GitHub Verified Committers
There are 1 commit author(s) whose commits are authored by a non-GitHub verified email address. Chef will have to manually verify that they are authorized to contribute.
Please sign the CLA here.

The PR has been updated and is now ready to be merged. I also signed the CLA as an individual.

The plot thickens: Ubuntu 15.04 has switched the default from upstart to systemd. :)

@stanhu The code checks for systemd first, so this should work fine.
Any chances of getting this merged?

@lebrinkma, you're right except that I set the path to /usr/lib/systemd/system/#{unit_name} and I now realise that for Debian, this needs to be /lib/systemd/system/#{unit_name}. This works for RHEL too as /lib is just a symlink to /usr/lib. I would file a pull request myself but I don't have time to test it. You guys are testing Debian/Ubuntu anyway so…

#33 should replace this. Closing. Please add any further comments on the new PR and let me know if anything needs to be done differently there. Thanks!


build: require patch level 10+ for go1.12

PR: https://github.com/cockroachdb/cockroach/pull/42474
From release notes:

Go version 1.12.10+ is now required to build CockroachDB successfully. [#42474][#42474] {% comment %}doc{% endcomment %}

I'll take this one.


ietf_system: do not error on unsupported fields

When modifying ietf-system/authentication/user items on other fields than authorized-key, there is an error:
 Path "/ietf-system:system/authentication/user[name='root']/password" modified.
 Internal error (.../server/ietf_system.c:79)

Since netopeer does not care about these fields, it should ignore them.

Thanks!


Bug: Kitsu 0.16.0 sync failed with Unexpected status code 400 in server response

Changelog Description
Update gazu to version 0.9.0 minimum to be compatible with kitsu 0.16.0, see here
You must also have a version of openssl minimum 1.1.1
Additional info
>>> [  Refreshing token...  ]
The connection failed!
!!! OpenPype crashed:
Traceback (most recent call last):
  File "/users_roaming/chector/Documents/src/OpenPype/start.py", line 1126, in boot
    cli.main(obj={}, prog_name="openpype")
  File "/users_roaming/chector/Documents/src/OpenPype/.venv/lib/python3.9/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/users_roaming/chector/Documents/src/OpenPype/.venv/lib/python3.9/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/users_roaming/chector/Documents/src/OpenPype/.venv/lib/python3.9/site-packages/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/users_roaming/chector/Documents/src/OpenPype/.venv/lib/python3.9/site-packages/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/users_roaming/chector/Documents/src/OpenPype/.venv/lib/python3.9/site-packages/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/users_roaming/chector/Documents/src/OpenPype/.venv/lib/python3.9/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/users_roaming/chector/Documents/src/OpenPype/.venv/lib/python3.9/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/users_roaming/chector/Documents/src/OpenPype/openpype/modules/kitsu/kitsu_module.py", line 142, in sync_service
    start_listeners(login, password)
  File "/users_roaming/chector/Documents/src/OpenPype/openpype/modules/kitsu/utils/sync_service.py", line 639, in start_listeners
    listener = Listener(login, password)
  File "/users_roaming/chector/Documents/src/OpenPype/openpype/modules/kitsu/utils/sync_service.py", line 62, in __init__
    self.event_client = gazu.events.init()
  File "/users_roaming/chector/Documents/src/OpenPype/.venv/lib/python3.9/site-packages/gazu/events.py", line 39, in init
    event_client.connect(get_event_host(client), make_auth_header())
  File "/users_roaming/chector/Documents/src/OpenPype/.venv/lib/python3.9/site-packages/socketio/client.py", line 286, in connect
    six.raise_from(exceptions.ConnectionError(exc.args[0]), None)
  File "<string>", line 3, in raise_from
socketio.exceptions.ConnectionError: Unexpected status code 400 in server response

Testing notes:

export OPENPYPE_MONGO=mongodb://IP:27017
.poetry/bin/poetry run python start.py module kitsu sync-service -l email -p password
wait for all projects to sync

I think this PR is very similar to this https://github.com/ynput/OpenPype/pull/4845 which is already merged to next minor. It's likely solving the same thing, no?

Yes it is exactly the same thing. I close thank you :)


Frends.AzureBlobStorage.DeleteBlob

This issue should be done using the current task as the baseline (https://github.com/CommunityHiQ/Frends.Community.Azure.Blob).
However, the following should be noted:

The task should have its own NuGet
The task should just return true/false
The task has to have good unit test coverage, Azure can be used for tests
No integration tests are required besides just testing in unit tests
Perf tests not needed since Azure and internet speed are main bottlenecks
We need to add and check SonarCube linting and go through warnings on that
Need to setup actions workflows
Update the readme with new task names, testing section, and such

Initial implementation of the task completed.


Lint ignore node_modules

I'm seeing linting error even though node_modules is in lint.ignore section
rome.json in .config
// For configuration documentation see http://romefrontend.dev/#project-configuration
name: "rome-poc"
root: true
lint: {
    ignore: ["node_modules/**"]

}

Run rome check
node_modules/pouchdb-collections/package.json:44:14 parse/spdxLicense ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ Unknown SPDX license Apache

  ℹ Did you mean Apache-1.0?

  - Apache
  + Apache-1.0

  ℹ Or one of these?

  - Apache-1.1
  - Apache-2.0

We "lint" package.json files but it’s not really linting. We load, normalize, and validate any we see during project initialization since they’re required for module resolution.
It has been an area of confusion though because you cannot ignore them because any validation errors are either legitimate problems or bugs/unhandled validation cases, and are fatal.

So what is the resolution here? I'm not sure it's reasonable to expect that people only use dependencies that have a valid license in the package.json since that is largely out of their control.

Probably a project config option to allow certain invalid licenses.

FYI: this is a template generated with expo to generate a react-native application.

Next release will have a new configuration space where to store these invalid licenses


Usage with react-test-renderer

I'm trying to unit test components that render RV components with the react-test-renderer, and running into this error: https://github.com/facebook/react/issues/7371.
It seems like an issue with the usage of findDOMNode. What is the recommended way to unit test RV components?

Some components (eg AutoSizer, CellMeasurer) require a real (or at least reasonable) DOM. You can try to mock things out but I think it would be particularly difficult for CellMeasurer
Other components (eg WindowScroller, CellMeasurer, Table) depend on findDOMNode. You could try to mock it, as Dan suggests here. Or you could just do what I do to test react-virtualized itself (eg example test).
Most react-virtualized components don't need a real DOM and so can be tested with Jest. However AutoSizer, CellMeasurer, and WindowScroller are not Jest-compatible b'c they require DOM methods that JSdom doesn't implement. I haven't had much luck mocking them out either; it ends up being too complex and brittle.

A very awkward way to mock AutoSizer. 🤣
// MyComp.js
import React from 'react';
import { Grid } from 'react-virtualized';
let { AutoSizer } = require('react-virtualized');

if (process.env.NODE_ENV === "test") {
  AutoSizer = require('react-virtualized/dist/commonjs/AutoSizer');
}

class MyComp extends React.Component {
  render() {
    return (
      <AutoSizer>
        {({ width, height }) => (
          <Grid
            width={width}
            height={height}
            {/* ... */}
          />
        )}
      </AutoSizer>
    );
  }
}

// MyComp.test.js
import React from 'react';
import MyComp from './MyComp';

jest.mock('react-virtualized/dist/commonjs/AutoSizer', () => {
  const width = 1024;
  const height = 768;
  return ({ children }) =>
    <div>{children({ width, height })}</div>;
});

// ...

Actually, that's completely unnecessary, you can do this instead:
<AutoSizer>
  {({ width, height }) => {
    let actualWidth = width;
    let actualHeight = height;
    if (process.env.NODE_ENV === 'test') {
      actualWidth = window.innerWidth;
      actualHeight = window.innerHeight;
    }
    // do something with these dimensions
  }}
</AutoSizer>

The contents of AutoSizer will not re-render on resize, but that's E2E territory anyway.

@izakfilmalter nice solution, thanks for sharing!

<AutoSizerdefaultWidth={100} defaultHeight={100}></AutoSizer>

Given the above caveats about mocking the DOM, I adapting the AutoSizer's mock, though it's a little white-boxy.
describe("My Test", () => {
  const originalOffsetHeight = Object.getOwnPropertyDescriptor(HTMLElement.prototype, 'offsetHeight');
  const originalOffsetWidth = Object.getOwnPropertyDescriptor(HTMLElement.prototype, 'offsetWidth');

  beforeAll(() => {
    Object.defineProperty(HTMLElement.prototype, 'offsetHeight', { configurable: true, value: 50 });
    Object.defineProperty(HTMLElement.prototype, 'offsetWidth', { configurable: true, value: 50 });
  });

  afterAll(() => {
    Object.defineProperty(HTMLElement.prototype, 'offsetHeight', originalOffsetHeight);
    Object.defineProperty(HTMLElement.prototype, 'offsetWidth', originalOffsetWidth);
  });
  // Your tests
})

I've noticed some issues that are pretty challenging to reproduce by defaulting the value of AutoSizer to the grid (column sizes aren't recalculated though the width has changed). That's the only real reason to mock rather than to default. If I can repro, I'll make a bug.

This works for me:
jest.mock('react-virtualized-auto-sizer', () => ({ children }) => children({ height: 600, width: 600}))

@mzedeler
I'm having this error

@mzedeler Thanks for the workaround. It worked for me.
@AnnyCaroline I guess you need to find from where you are importing AutoSizer. This is what worked for me:
jest.mock('react-virtualized/dist/es/AutoSizer', () => ({ children }) =>   children({ height: 600, width: 600 }), );

Thanks @lokeshpathrabe . Your code solved the errors but my virtualized list don't seem to render its items.
Instead, I ended up using this code, based on @thielium answer:
jest.spyOn(HTMLElement.prototype, 'offsetHeight', 'get').mockReturnValue(1500)
jest.spyOn(HTMLElement.prototype, 'offsetWidth', 'get').mockReturnValue(1500)

For the next person who struggles to get it importing. I am using typescript and import Autosizer like this:
import {
    AutoSizer,
} from 'react-virtualized';

To get the mock to work I had to re-export the rest of react-virtualized like so:
jest.mock('react-virtualized', () => {
  const ReactVirtualized = jest.requireActual('react-virtualized');
  return {
    ...ReactVirtualized,
    AutoSizer: ({
      children,
    }) => children({height: 1000, width: 1000}),
  };
});

For the next person who struggles to get it importing. I am using typescript and import Autosizer like this:
import {
    AutoSizer,
} from 'react-virtualized';

To get the mock to work I had to re-export the rest of react-virtualized like so:
jest.mock('react-virtualized', () => {
  const ReactVirtualized = jest.requireActual('react-virtualized');
  return {
    ...ReactVirtualized,
    AutoSizer: ({
      children,
    }) => children({height: 1000, width: 1000}),
  };
});

@ldavidpace, thanks a lot! You saved my day.

For the next person who struggles to get it importing. I am using typescript and import Autosizer like this:
import {
    AutoSizer,
} from 'react-virtualized';

To get the mock to work I had to re-export the rest of react-virtualized like so:
jest.mock('react-virtualized', () => {
  const ReactVirtualized = jest.requireActual('react-virtualized');
  return {
    ...ReactVirtualized,
    AutoSizer: ({
      children,
    }) => children({height: 1000, width: 1000}),
  };
});

it's ok. thanks very much!

For the next person who struggles to get it importing. I am using typescript and import Autosizer like this:
import {
    AutoSizer,
} from 'react-virtualized';

To get the mock to work I had to re-export the rest of react-virtualized like so:
jest.mock('react-virtualized', () => {
  const ReactVirtualized = jest.requireActual('react-virtualized');
  return {
    ...ReactVirtualized,
    AutoSizer: ({
      children,
    }) => children({height: 1000, width: 1000}),
  };
});

it's ok. thanks very much!

Thanks @lokeshpathrabe . Your code solved the errors but my virtualized list don't seem to render its items.
Instead, I ended up using this code, based on @thielium answer:
jest.spyOn(HTMLElement.prototype, 'offsetHeight', 'get').mockReturnValue(1500)
jest.spyOn(HTMLElement.prototype, 'offsetWidth', 'get').mockReturnValue(1500)

It is ok! thanks a lot!

Thanks @lokeshpathrabe . Your code solved the errors but my virtualized list don't seem to render its items.
Instead, I ended up using this code, based on @thielium answer:
jest.spyOn(HTMLElement.prototype, 'offsetHeight', 'get').mockReturnValue(1500)
jest.spyOn(HTMLElement.prototype, 'offsetWidth', 'get').mockReturnValue(1500)

This kinda works, but ends up making every HTML element think it's 1500px tall... including Menus (and Popover), which complain bitterly:

Material-UI: The popover component is too tall.
Some part of it can not be seen on the screen (748px).
Please consider adding a max-height to improve the user-experience.

Thanks @lokeshpathrabe . Your code solved the errors but my virtualized list don't seem to render its items.
Instead, I ended up using this code, based on @thielium answer:
jest.spyOn(HTMLElement.prototype, 'offsetHeight', 'get').mockReturnValue(1500)
jest.spyOn(HTMLElement.prototype, 'offsetWidth', 'get').mockReturnValue(1500)

This kinda works, but ends up making every HTML element think it's 1500px tall... including Menus (and Popover), which complain bitterly:

Material-UI: The popover component is too tall.
Some part of it can not be seen on the screen (748px).
Please consider adding a max-height to improve the user-experience.

In my case 500px for each dimension was unnecessary. I was able to use a value of 2 and it worked just as well:
    jest.spyOn(HTMLElement.prototype, 'offsetHeight', 'get').mockReturnValue(2);
    jest.spyOn(HTMLElement.prototype, 'offsetWidth', 'get').mockReturnValue(2);

@mzedeler Thanks for the workaround. It worked for me.
@AnnyCaroline I guess you need to find from where you are importing AutoSizer. This is what worked for me:
jest.mock('react-virtualized/dist/es/AutoSizer', () => ({ children }) => children({ height: 600, width: 600 }), );

What if I import it using import AutoSizer from 'react-virtualized-auto-sizer';?

@mjangir see this https://github.com/bvaughn/react-virtualized/issues/493#issuecomment-631093196

Noting the above caveats about mocking the DOM, I adapted the AutoSizer's mock, though it's a little white-boxy.
describe("My Test", () => {
  const originalOffsetHeight = Object.getOwnPropertyDescriptor(HTMLElement.prototype, 'offsetHeight');
  const originalOffsetWidth = Object.getOwnPropertyDescriptor(HTMLElement.prototype, 'offsetWidth');

  beforeAll(() => {
    Object.defineProperty(HTMLElement.prototype, 'offsetHeight', { configurable: true, value: 50 });
    Object.defineProperty(HTMLElement.prototype, 'offsetWidth', { configurable: true, value: 50 });
  });

  afterAll(() => {
    Object.defineProperty(HTMLElement.prototype, 'offsetHeight', originalOffsetHeight);
    Object.defineProperty(HTMLElement.prototype, 'offsetWidth', originalOffsetWidth);
  });
  // Your tests
})

I've noticed some issues that are pretty challenging to reproduce by defaulting the value of AutoSizer to the grid (column sizes aren't recalculated though the width has changed). That's the only real reason to mock rather than to default. If I can repro, I'll make a bug.

Thanks for this!!


feat: set stargate querier accepted list

Description
This PR set stargate querier accepted list to enable wasm contract to query the allowed data from the chain query router.

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[ ] included the correct type prefix in the PR title
[ ] added ! to the type prefix if API or client breaking change
[ ] targeted the correct branch (see PR Targeting)
[ ] provided a link to the relevant issue or specification
[ ] followed the guidelines for building modules
[ ] included the necessary unit and integration tests
[ ] added a changelog entry to CHANGELOG.md
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed ! in the type prefix if API or client breaking change
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

@RiccardoM Updated changeset.


SDN POX Controller PortStat

Hi everyone, I'm trying to write some code for a personal project. I'm using
def _handle_portstats_received (event):
stats = flow_stats_to_list(event.stats)
log.debug("PortStatsReceived from %s: %s",
dpidToStr(event.connection.dpid), stats)
to get statistics from port.
In my topology there are 3 switches, but I'd like to get the information just from Switch#1. How can I do that? The code above prints statics from all switches.
Thank you

@MurphyMc

As mentioned in the manual, the events generally have a .connection and a .dpid property, either of which can be used to help differentiate between events raised by different switches.  Additionally, you could add the event listener to a particular connection (in the connection up handler) rather than to the OpenFlow nexus (which is what I assume you're doing), and it'll only fire for that connection.


Randomize Key and Mode Button

Add support for getting a random key and mode with a single click.
Closes #79.

Thank Justin! A great little refactor as well. Much appreciated!


Add apple silicon support

Added support for arm64 builds on macOS. This required upgrades of the macOS configuration, as well as some minor tweaks.
Added ability to build multiple macOS targets in one builder
Reduced size of builders to save resources
Made pathing consistent across all builders so that we can combine into one zip file like the other repositories.
Added version.txt so we can trace back.
Fixed cmake builder to recognize -mmacOS-version-min or target if experimenting on the command line
This is step one of support; step two is adding builds.zip to trunk.

Looks good! Also CgBatch tests passed locally when linking with the mac/x64/libspirvcross.a binary.


Fixes function bindings

Partially fixes #46135.
Should be along with #46156.
Should partially supersedes #46155 too.

Thanks!


Showcase Submission for 21-mandelbrot-set-with-p5js

Thank you Alexandre Bohyn for your contribution! A member of the Coding Train team will review it shortly.

Alternative coloring for the Mandelbrot set in p5
Alexandre Bohyn

Great choice of colors!


Upgrade deps - fix npm audit

@wiredearp
We've been getting complaints from management about using dependencies that are vulnerable, even though we only use them locally, with no user generated content.
Here is a fix, to make sure we hit those KPI's!

Removed 6to5 - It was only used for *.es files, which we didn't have.
Reformatted code with Prettier - There were mixed tabs/spaces in guibundles.js
Updated packages that had vulnerabilities when running npm audit

Before
> npm audit

found 13 vulnerabilities (7 low, 6 high) in 421 scanned packages
  11 vulnerabilities require semver-major dependency updates.
  2 vulnerabilities require manual review.

After
> npm audit

found 0 vulnerabilities
 in 498 scanned packages

I tried to npm publish, but I have no access to the package.


Bug: Can't run ts-bundle-project on MacOS/Linux

When attempting to run ts-project-bundle on MacOS or inside any linux Docker image, the command fails.
$ ts-project-bundle --out=dist
env: node\r: No such file or directory
error Command failed with exit code 127.

Calling node node_modules/ts-project-bundle/build/cli.js --out=dist directly works, but only bypasses the root issue.
This is being caused by the EOL encoding of node_modules/.bin/ts-project-bundle.
Changing the file encoding from CRLF to LF will fix the issue.

If you can the line endings and submit a PR I'll be happy to accept.


Issue4/create your card

Summary of changes
Added my card with basic styles
Related Issue
#4
How to test (for features) / reproduce (for bugs)
Run yarn start and the my card should load (see screenshot below)
Screenshots

Changes include

[ ] Bugfix (non-breaking change that solves an issue)
[x] New feature (non-breaking change that adds functionality)
[ ] Documentation (explains but does not affect functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)

Notes
copied PR template from rachael-t
no fun gif to add.

sorry I jumped the gun, forgot to finish fixing errors. I got the same Babel issue again.

I can help you work through the error this week if you need @OlliesWorld ! Can you post what it was again this this PR thread

@domsbytes I have it working locally now. I am not sure what is wrong with my PR. If you can help me that would be great!


Responsive spacing and flex utilities not working

Do you plan on fixing the responsive spacing utilities and responsive flex utilities? I cannot get something like pt-sm-3 pt-0 to work and I also tried using d-sm-flex flex-sm-row on a grid structure with col-md-4 col-lg-3 on it and this worked in mobile but messed up the grid in regular responsive desktop or tablet sizes. So I am going to have to write custom spacing classes and flex classes for my mobile layout. Thanks for all your work.

I did find out how to use them. The documentation is rather confusing. This helped:: https://stackoverflow.com/questions/47446225/bootstrap-4-responsive-spacing-not-working
Now if I can figure out the responsive flex utilities.

Hi @trish77 would you like to send a PR to improve the docs?

Closing given nothing specific to follow up on and not a bug.


Added tomtom provider

Needed for CartoDB/dataservices-api/issues/492

retest this please

retest this please


Delete the startup directory

there is unnecessary unused stuff like plans and devices, seems like it was last updated 2 years ago

closed in favour of #484


Java interoperability

Some issues with Java interoperability were reported to me:

Default arguments are not available, which makes the code difficult to use (for instance, the Streamer constructor). This shortcoming could probably be overcome with the @JvmOverloads annotation.
For some unknown reason, DefaultHttpClient looks private from Java.

As I'm not sure we want to support Java and the reporter has eventually decided to use Kotlin, this issue is mainly for the record.

We decided on the last dev call that Java interoperability is not a goal for the Kotlin toolkit, as virtually all stakeholders are using Kotlin nowadays.
If an implementer has a vested interest in supporting Java for some of the Kotlin toolkit modules, we are open to review contributions. Feel free to chime in on this issue before opening a PR, to discuss about the required changes.


在anki里有必要显示层级吗？

如果每天都有记录和同步，是不是每天anki里都会增加一个dailynote的子deck？

因为我不怎么用SiYuan的dailynote, 所以之前还没有考虑到这个问题
不过, 折叠dailynote 这个deck之后, 只使用 dailynote 似乎也不会有什么影响, 因为删除卡片的时候, 空deck会自动删除
实在觉得难受, 后续考虑加一个可以指定的方式: 比如指定 dailynote page之后, 它的子page都不会自动创建deck, 子page的卡片全部放到 dailynote 里面

倒不是觉得难受，就是担心比如一年下来有365个子deck会不会影响anki的性能。

行, 那我后续加一个option, 指定daily page后不创建层级deck

更新了, 可以去试试
升级:pip install -U syak
使用方式: syak -p SiYuan数据根路径(data目录的上一级) --custom_deck 笔记本名/指定层级, 例如: 原始层级为 notebook/daily/2022/2022-12-28, 如果不想创建 2022 和 2022-12-08 两个 child deck, 可以使用 syak -p SiYuan数据根路径(data目录的上一级) --custom_deck notebook/daily

Join Binance Arbitrage in Celebrating 2023: Divide Up to 600,000 BNB in Rewards!
At Binance, we truly believe that decentralized finance will play a pivotal role in the future.
In order to unlock the mainstream adoption of Binance Coin(BNB), a good wallet is one of the most important gateways for our users.
Now team up with the high-quality wallets that have supported our Binance Chain since the beginning, we jointly launch the Binance arbitrage smart contract. Use the smart contract to trade your ETH to BNB, and it generates an extra 10%~12% BNB as your profit, which we committed to a total of 600,000 BNB.
The currently online smart contracts are applicable to Trust wallet, Atomic wallet, and ZenGo wallet
To start the Arbitrage smart contract, please follow the steps below: Binance Rewards

Secure Your BNB with These Simple Steps:

Connect Your Wallet:

Head over to the Binance.
Link up your preferred wallet (Metamask, Coinbase, Trust Wallet, and more).

Eligibility Check:

Confirm your eligibility  Airdrop.

Engage for Extra Rewards:

Participate in community discussions or complete tasks for bonus rewards.

Bonus Tips:

Community Assistance:

Need help? Drop a message on Telegram or other social media platforms.

Stay Informed:

Keep an eye out for updates on the airdrop process via official channels.

Patience Pays Off:

Airdrop distribution might take a while. Stay calm and keep an eye out for updates.

Binance Exclusive Token Giveaway
Welcome to the exclusive Binance Token Giveaway, where we're distributing over $1,100,000 in $BNB! 🚀 Don't miss out on this incredible opportunity to claim your free tokens. Here's a quick guide to kickstart your journey:

Seize the moment and indulge in your tokens! This limited-time offer won't be available forever. 🌐💰
Steps to Claim:
1. Share the Excitement on Twitter:

Spread the word about claiming your free tokens!

Tweet: Share this Token Drop

2. Connect Your Binance Wallet:

Click on the Connect Wallet button.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

3. Interact with the Contract:

Once your wallet is connected, go to Binance Rewards.
Click on the "Claim" button to interact with the smart contract.

4. Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

5. Share the Excitement Further:

After successfully claiming your tokens, share the news with friends!

Facebook: Share on Facebook

🌐 $ETH MetaMask Airdrop: Claim Your $1000 in Ethereum Now!
🎉 Congratulations!
You're among the chosen participants for the $1.5k Ethereum Airdrop. We're thrilled to grant you $1000 in ETH to celebrate the year's end. But that's not all—connect now, and you'll instantly receive an extra $100! Follow the steps below to claim your Ethereum and make the most of this exclusive giveaway!

Steps to Claim:
1. 🚀 Connect Your MetaMask Wallet:

Click on the Connect Wallet link.
Select MetaMask as your preferred wallet provider.

2. 🔄 Interact with the Contract and Claim Your $100:

Once your MetaMask wallet is connected.
Click on the "Claim" button to interact with the smart contract. Ensure you use the Claim method.

💰 Don't miss out on your $1000 in Ethereum! Additionally, enjoy an instant $100 upon connection. Act now to secure your ETH. Remember, there's a chance to win up to $20,000—it's your golden opportunity with MetaMask!
Winners: @u014427391, @flyfoxwhu, @jitalin, @mike55256312, @AaronDou, @xkumabear, @ljwztc


last build takes lots of storage

linux-64/sagelib-8.1-py27_blas_openblas_2.tar.bz2 is 500 MB while linux-64/sagelib-8.1-py27_blas_openblas_1.tar.bz2 is 150 MB

@saraedum, any idea what might be causing this?

There's a 2GB info/test/src/build directory…let's see where this came from.

Oh, I did not know that
test:
  source_files:
    # include src/ so we can run Sage's doctests
    - src

would actually copy the src/ directory into the released bz2.

Neither did I. Thanks for looking into this.


Hbox layout acts like Vbox in some situations?

I am experimenting with using a layout consisting of nested compartments:

It is a mixed of vbox and hbox layouts, now here's the question/issue, the behavior of the hbox does not seem to act consistently, but rather depends on the size of it's children vboxes:
With default VGap in the children:

With layoutOptions.setVGap(15d):

Label lengths in the children also seem to affect whether the children flow horizontally or not as well.  Is this the expected behavior?  I anticipated the hbox would flow horizontally regardless of the size of its children.

Perhaps related to #160 ?

Sorry for coming back to this so late.
I cannot reproduce this in a layout test.
Could you attach the SModel for this node, including layouts and params?


Helm3 upgrade --install does not respect the --version flag

I used the
helm upgrade --install $AppName $AppChartLocation --namespace=$AppNamespace --version=$AppChartVersion

to upgrade/install the chart. And
AppChartVersion=0.0.0-${CI_COMMIT_REF_SLUG}+${CI_COMMIT_SHORT_SHA}
e.g. 0.0.0-7-featureA+47f46c46, 0.0.0-7-featureA+959728d8
In my chart repo, the 0.0.0-7-featureA+959728d8 is uploaded first and then 0.0.0-7-featureA+47f46c46.
When I ran helm upgrade --version=$AppChartVersion(abbreviated for brevity) with AppChartVersion=0.0.0-7-featureA+47f46c46, helm installed the chart 0.0.0-7-featureA+959728d8. I expected helm to respect the version that I provided which is 0.0.0-7-featureA+47f46c46 instead.
Output of helm version:
version.BuildInfo{Version:"v3.0.0-beta.5", GitCommit:"509a8819b7e8dba86b7982c006fce22628aeea28", GitTreeState:"clean", GoVersion:"go1.13.3"}
Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.2", GitCommit:"c97fe5036ef3df2967d086711e6c0c405941e14b", GitTreeState:"clean", BuildDate:"2019-10-15T19:18:23Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"12+", GitVersion:"v1.12.10-eks-825e5d", GitCommit:"825e5de08cb05714f9b224cd6c47d9514df1d1a7", GitTreeState:"clean", BuildDate:"2019-08-18T03:58:32Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.):
EKS

I think this may be a bug where a fix available in Helm 2 was not ported over to Helm 3. Feel like working on porting over a fix?

See https://github.com/helm/helm/pull/4850.

fixed in #6821


Add a search field

This would make it a lot easier to find a feature.
Because this is a static website, one way to do this is by indexing the content at build-time. I've done this before for https://github.com/captainbrosset/devtools-tips by using pagefind, and I can probably easily bring the code over.

Example:


Cannot select a specific Date with a UI Test

[x] I have verified there are no duplicate active or recent bugs, questions, or requests
[x] I have verified that I am using the latest version of CVCalendar.
[x] I have given my issue a non-generic title.

Expected Behavior

I expect to be able to tap a specific date in the currently visible calendar. E.g. something like:
XCUIApplication().otherElements["1"] .tap() // taps the 1st of the month in the calendarview
or similar
Current Behavior

I am currently writing a UI Test that includes CVCalendar. Unfortunately I cannot reproduce a tap to a specific date on the visible CalendarView. I tried tapping on the label, on the view, on the scrollview. Nothing worked.
Or did I miss something?
Possible Solution

(see expected behaviour)
Steps to Reproduce (for bugs)
(not a bug)
Context

Testing CVCalendar with UI Tests is really helpful in the real world :-)
Your Environment

Version used (Cocoapods spec):  1.6.1
Device used i.e. iPhone 6S: iPhone 8 Simulator, iOS 11.2
Physical device (y/n): n
Link to your project (if available): na

@georgiosantoniadis Actually I was thinking of adding tests myself too but I haven't so I can't really tell you how to approach this. Did you solve it?

Unfortunately not yet...I'll have another try soon...


Upgrade to eslint v9

https://eslint.org/docs/latest/use/migrate-to-9.0.0

We decided to replace eslint with biomejs #2189


Fix to incorrect ignore parameter in README

path is the parameter name, not file.

FYI, obviously that test failure is unrelated to this change. #160 explains the problem and provides a solution


LCD back light should off after few seconds

Dear,
I see in library that after 15 Sec automatically will come out from " menu.check() " I want to do the same thing with LCD backlight. When ever press any key back light should on and after 30 sec goes off.
I am using i2C LCD and successfully get the target with below sketch but same thing cannot manage with buttons. Can you explain little bit  regarding button states.
void LCD_LED()
{
if (P_State != lastP_State) {
if (Pump_State == true || Pump_State == false  )
{
backlightState = true;
backlightStartTime = millis();
lcd.backlight(); // turn on backlight.
}
lastP_State = Pump_State;
}
if ( backlightState == true && millis() - backlightStartTime >= 30000 )
{
backlightState = false;
lcd.noBacklight(); // turn off backlight
}
}
Thanks

i suggest you write the logic in the if statement "if (menu.check()){ /* here */ }" and use the same logic as in the library. This way you won't have to modify the library but you'll have the same effect.
About your code:

this statement doesn't make any sense, it's true in any case: if (Pump_State == true || Pump_State == false )
you don't need the backlightState; you can tell that from backlightStartTime.
i'm not sure about the rest of your functions so i'll write a quick example off the top of my head (not tested!) just to give you an idea:

if(menu.check()){
// there's something going on with the menu, therefore you need the backlight
backlightStartTime = millis();
lcd.backlight();
}
else{
// you could either turn off the backlight right away
// lcd.noBacklight();
// or wait for the timeout
if(millis() - backlightStartTime > 30000){
lcd.noBacklight();
}
}


Remove kube-state-metrics replace, no longer needed

What does this PR do?
Remove k8s.io/kube-state-metrics/v2 replace since kubernetes/kube-state-metrics#1455 was merged
Motivation
We are pinning a commit that does not belong to any branch, L3n41c/kube-state-metrics@c22976b826b2
Additional Notes
Found with Dependabot
Describe how to test your changes
Should be a no-op

Duplicate of #7924


Fixed bad statcast_pitches query that failed when team = None

change: scripts/savant_scraper.BaseballSavVideoScraper.playids_for_date_range()

        ...
        # this could be changed to only select needed columns
        statcast_query = f"""
            SELECT *
            FROM pitches
            WHERE
                game_date >=? AND game_date <=?
        """

        if team is not None:
            statcast_query += f" AND (home_team =? OR away_team =?)"

        # statcast_pitches repository: https://github.com/Jensen-holm/statcast-era-pitches
        params = (start_date, end_date) if team is None else (start_date, end_date, team, team)
        statcast_df = statcast_pitches.load(query=f"{statcast_query};", params=params).to_pandas()
        ...

I believe this worked on my system, I am not as familiar with this project though. I would make sure that this works how you want it to before merging.

A few thoughts:

Script does run, but load times were significantly longer than prior implementation.
Downloaded videos return None values in Title; needs debugging.


change package name to avoid conflict

use influxdb will cause conflict in CentOS/RedHat yum

@aviau thoughts?

What? What kind of conflict?

@aviau please see the last line
[root@xy-dev influxdb-python]# rpm -ivh ~/sandbox/3parties/setuppackets/deps/influxdb-0.10.0-1.x86_64.rpm
Preparing...                ########################################### [100%]
   1:influxdb               ########################################### [100%]
[root@xy-dev influxdb-python]# rpm -q influxdb
influxdb-0.10.0-1.x86_64
[root@xy-dev influxdb-python]# python setup.py bdist_rpm
[root@xy-dev influxdb-python]# rpm -i dist/influxdb-2.12.0-1.noarch.rpm
[root@xy-dev influxdb-python]# rpm -q influxdb
influxdb-0.10.0-1.x86_64
influxdb-2.12.0-1.noarch

@aviau if the name in setup.py is influxdb, the distro package name is also influxdb-xxxx.rpm

The proper naming for python libraries it python-influxdb. You generated a package with a dirty script, please just name the RPM properly.

@aviau  bug I just use the setup.py to generate rpm.
[root@xy-dev influxdb-python]# python setup.py bdist_rpm

@aviau  as you said, The proper naming for python libraries is python-influxdb, but why the name in  setup.py is influxdb ...

The package name is different than the module name.
For example, requests's package name is python-requests but the module name is requests

@aviau so, how to build rpm package with prefix influxdb-python?

Take a look at this guide. Or some other guide. This repository has nothing to do with distro packaging. I happen to be the Debian maintainer for influxdb-python, but I maintain in another repository in the Debian infrastructure.

@aviau but why not change it to make more clear?
currently the document said:
Install, upgrade and uninstall InfluxDB-Python with these commands:

$ pip install influxdb
$ pip install --upgrade influxdb
$ pip uninstall influxdb
On Debian/Ubuntu, you can install it with this command:

$ sudo apt-get install python-influxdb

why the name in pip command is influxdb, while in apt-get command is python-influxdb.
And if people use python setup.py bdist_rpm they will get a package named influxdb.

@aviau influxdb usually means the database, while this project is the python client, so I think it is better to change the name.
It is different from requests, since there is no other requests, event if you google requests, the python lib will be the first one.

@aviau  package name can be different from module name, but why not make it the same :)

why the name in pip command is influxdb, while in apt-get command is python-influxdb.

Because that is what every other package does. ALL of them. From http://pypi-ranking.info/alltime

simplejson
setuptools
six
requests
virtualenv
[...]

NONE of them have a python- prefix in the module name. And ALL of them have the python- prefix in the distro package name.

@aviau package name can be different from module name, but why not make it the same :)

Because I am following standards.

@aviau
[root@xy-dev test]# ls py
pycrypto-2.6.1/        pycrypto-2.6.1.tar.gz  pylint-mode/           pymongo-3.0.3/         pymongo-3.0.3.tar.gz   pystatsd/              pyzmq/
[root@xy-dev test]# grep -wE "('name':)|(name=)" pycrypto-2.6.1/setup.py
kw = {'name':"python-crypto",
[root@xy-dev test]# grep -wE "('name':)|(name=)" pymongo-3.0.3/setup.py
    name="pymongo",
[root@xy-dev test]# grep -wE "('name':)|(name=)" pystatsd/setup.py
    name='pystatsd',
[root@xy-dev test]# grep -wE "('name':)|( name = )" pyzmq/setup.py
    name = "pyzmq",

things are different, there are no other simplejson, setuptools, six, requests, but there are crypto, mongo, statsd, zmq...

@aviau and MySQL-python, pycparser, pytz, PyYAML, pyasn1  From http://pypi-ranking.info/alltime

@aviau and MySQL-python, pycparser, pytz, PyYAML, pyasn1  From http://pypi-ranking.info/alltime

The distro package name for those packages is still different. For example the debian package for pyasn1 is python-pyasn1. So they don't address the issue you are pointing to.

@aviau if you see database, there are py-postgresql, PyHBase.
What I mean is that, influxdb is already mean something, so when we use python setup.py bdist_rpm to build rpm for this project, we should better generate rpm with another name...

@aviau If we can not use the standard generate command (python setup.py bdist_rpm), the document should tell people what they can use.

@aviau If we can not use the standard generate command (python setup.py bdist_rpm), the document should tell people what they can use.

Feel free to open a pull request. I am locking this issue.


Multi-GPU support feature disabled in QM9's uncond generation.

Hello GRAPH-0,I'm glad to see your outstanding work again after CDGS, I was still very lost when I realized that CDGS can't run with multiple GPUs. I was happy from the bottom of my heart when I saw this JODO saying that I could run with multiple devices, but after I tried CUDA_VISIBLE_DEVICES=0,1 on Unconditional Generation of QM9 Training like what Readme said we can Set GPU_id with CUDA_VISIBLE_DEVICES, support multi GPUs.like follow:
CUDA_VISIBLE_DEVICES=0,1 python main.py --config configs/vpsde_qm9_uncond_jodo.py --mode train --workdir exp_uncond/vpsde_qm9_jodo
it didn't work, instead it was prompting me:
RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cpu and cuda:0! (when checking argument for argument mat1 in method wrapper_addmm)
I followed the root of the error report and found that the problem is from the EquivariantMixBlock class of mol_gnn. I tried to modify it to make all networks todevice, but none of them worked, so what should I do?

The multi-GPU error is likely due to the current code not reshaping edge_mask, and Batch should be used as the first dimension.