[SPARK-23713][SQL] Cleanup UnsafeWriter and BufferHolder classes

What changes were proposed in this pull request?
This PR implemented the following cleanups related to  UnsafeWriter class:

Remove code duplication between UnsafeRowWriter and UnsafeArrayWriter
Make BufferHolder class internal by delegating its accessor methods to UnsafeWriter
Replace UnsafeRow.setTotalSize(...) with UnsafeRowWriter.setTotalSize()

How was this patch tested?
Tested by existing UTs

Test build #88342 has started for PR 20850 at commit 0379f7c.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1583/
Test PASSed.

Test build #88343 has started for PR 20850 at commit 06e7435.

Test build #88342 has finished for PR 20850 at commit 0379f7c.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
public final class BufferHolder

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88342/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88343/
Test FAILed.

Merged build finished. Test FAILed.

I think the failure of org.apache.spark.ml.r.RWrapperUtilsSuite.avoid libsvm data column name conflicting may not be related to this PR. However, I am investigating the reason for this failure.

cc @hvanhovell

retest this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1585/
Test PASSed.

Merged build finished. Test PASSed.

Test build #88347 has started for PR 20850 at commit 06e7435.

Test build #88347 has finished for PR 20850 at commit 06e7435.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88347/
Test FAILed.

Merged build finished. Test FAILed.

retest this please

Test build #88373 has started for PR 20850 at commit 06e7435.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1608/
Test PASSed.

Test build #88373 has finished for PR 20850 at commit 06e7435.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88373/
Test FAILed.

Merged build finished. Test FAILed.

This test is consistently failed. While I did not change file reader, I am investigating the reason in my environment.

@kiszk this is a good start! This is very performance critical code, can you please extend/update/run the existing UnsafeProjectionBenchmark?

Test build #88378 has started for PR 20850 at commit b696b7c.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1613/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1614/
Test PASSed.

Test build #88380 has started for PR 20850 at commit c342f0d.

Test build #88378 has finished for PR 20850 at commit b696b7c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88378/
Test PASSed.

Test build #88380 has finished for PR 20850 at commit c342f0d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88380/
Test PASSed.

@hvanhovell btw, (this is not related to this pr thought...) the most part of code in UTF8StringBuffer and BufferHolder are overlapped. So, we could clean up there, too? https://github.com/apache/spark/compare/master...maropu:CleanupBufferImpl

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1636/
Test PASSed.

Merged build finished. Test PASSed.

Test build #88411 has started for PR 20850 at commit 3637a5c.

Test build #88411 has finished for PR 20850 at commit 3637a5c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88411/
Test PASSed.

ping @hvanhovell

retest this please

Test build #88593 has started for PR 20850 at commit 3637a5c.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1762/
Test PASSed.

Test build #88593 has finished for PR 20850 at commit 3637a5c.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88593/
Test FAILed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1773/
Test PASSed.

Merged build finished. Test PASSed.

Test build #88616 has started for PR 20850 at commit 3637a5c.

Test build #88616 has finished for PR 20850 at commit 3637a5c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88616/
Test PASSed.

@hvanhovell sorry for ping again

Test build #88672 has started for PR 20850 at commit 3ad7638.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1810/
Test PASSed.

Test build #88672 has finished for PR 20850 at commit 3ad7638.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88672/
Test PASSed.

Test build #88718 has finished for PR 20850 at commit a94d470.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88718/
Test FAILed.

retest this please

Test build #88728 has started for PR 20850 at commit a94d470.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1848/
Test PASSed.

Test build #88728 has finished for PR 20850 at commit a94d470.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88728/
Test FAILed.

retest this please

Test build #88760 has started for PR 20850 at commit a94d470.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1866/
Test PASSed.

Test build #88760 has finished for PR 20850 at commit a94d470.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88760/
Test FAILed.

Test build #88787 has started for PR 20850 at commit 6caf11c.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1884/
Test PASSed.

Test build #88787 has finished for PR 20850 at commit 6caf11c.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88787/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1886/
Test PASSed.

Test build #88789 has started for PR 20850 at commit 6caf11c.

Test build #88789 has finished for PR 20850 at commit 6caf11c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88789/
Test PASSed.

Merged build finished. Test PASSed.

Test build #88801 has started for PR 20850 at commit 9dc36b7.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1893/
Test PASSed.

Test build #88801 has finished for PR 20850 at commit 9dc36b7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88801/
Test PASSed.

LGTM

@kiszk can you rerun the UnsafeProjectionBenchmark to make sure we didn't regress anywhere?
Otherwise LGTM.

@hvanhovell Here are results of UnsafeProjectionBenchmark. I have not seen the regression.
With master 529f847105fa8d98a5dc4d20955e4870df6bc1c5
OpenJDK 64-Bit Server VM 1.8.0_151-8u151-b12-0ubuntu<IP_ADDRESS>-b12 on Linux 4.4.0-66-generic
Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz
unsafe projection:                       Best/Avg Time(ms)    Rate(M/s)   Per Row(ns)   Relative
------------------------------------------------------------------------------------------------
single long                                   1638 / 1638        163.9           6.1       1.0X
single nullable long                          2375 / 2568        113.0           8.8       0.7X
7 primitive types                             5108 / 5234         52.6          19.0       0.3X
7 nullable primitive types                    7809 / 7909         34.4          29.1       0.2X

With SPARK-23713
OpenJDK 64-Bit Server VM 1.8.0_151-8u151-b12-0ubuntu<IP_ADDRESS>-b12 on Linux 4.4.0-66-generic
Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz
unsafe projection:                       Best/Avg Time(ms)    Rate(M/s)   Per Row(ns)   Relative
------------------------------------------------------------------------------------------------
single long                                   1630 / 1630        164.7           6.1       1.0X
single nullable long                          2354 / 2403        114.0           8.8       0.7X
7 primitive types                             5107 / 5174         52.6          19.0       0.3X
7 nullable primitive types                    7867 / 7938         34.1          29.3       0.2X

Merging to master. Thanks for your hard work and patience.


Equal Indicators must be equal

Expected Behavior

Equal Indicators must be equal
Actual Behavior

Equal Indicators are not equal. Not even equal to itself.
Potential Solution

Fix bug in Equal override method for BaseIndicator.
https://github.com/QuantConnect/Lean/blob/b9fcddb60394c76586426a131fa6fa09d6073390/Indicators/IndicatorBase.cs#L177
Reproducing the Problem

[Test]
public void EqualIndicatorsAreEqual()
{
    var indicator1 = new SimpleMovingAverage(5);
    var indicator2 = new SimpleMovingAverage(5);
    Assert.True(indicator1.Equals(indicator2));
}

[Test]
public void IndicatorMustbeEqualToItself()
{
    var indicator1 = new SimpleMovingAverage(5);
    Assert.True(indicator1.Equals(indicator1));
}

System Information

Local Win7
Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues

[x] I have provided detailed steps to reproduce the issue

After digging a little more in this issue the first test proposed, should fail by design, as the Equal method calls ReferenceEquals.
The second test however should pass.


hands on developing corona-virus  web App in vanilla JavaScript  using Covid-19 and Fetch API.

Topic Suggestion
hands on developing corona-virus  web App in vanilla JavaScript  using Covid-19 and Fetch API.
Proposed article introduction
Information is essential in today's world. In this article, we will make a Covid-19 web tracker application that gathers  data from Covid cases, recoveries, and deaths in a nation and showcases the information. We will make the app using the vanilla JavaScript ,Covid-19 API and the Fetch API.
Key takeaways

Understanding the importance of corona-virus information.
HTML and CSS foundation code.
Fetch and Covid-19 API  implementation.
Vanilla JavaScript complete code.

Article quality
My article will be simple and easy to understand, with relevant information on the Covid-!9 pandemic presented in a step-by-step format to serve as a guide for any web developer.
References
N/A

This topic suggestion seems quite similar to https://www.youtube.com/watch?v=nDYKRL7mwRw Therefore, it will not provide enough value to the community. Feel free to suggest another topic. Thanks


Make the implicitly exported rule consistent between checker and binder

Fixes #54342.
In the binder, when deciding if a declaration should be treated as exported or not, we consult the declaration container's flags to see if it is an "export context", meaning an ambient context where declarations are implicitly exported. In the checker, however, we treat everything in an ambient context as exported, regardless of whether we're in an ambient context where declarations are implicitly exported, resulting in us incorrectly not erroring on the .d.ts file present in #54342.

@typescript-bot run DT

Well, I guess the bot still isn't working, so we'll have to wait to see what this PR breaks in DT.

@typescript-bot run DT

@typescript-bot test this
@typescript-bot user test this
@typescript-bot user test tsserver
@typescript-bot test top100
@typescript-bot test tsserver top100
@typescript-bot perf test this

@typescript-bot test this
@typescript-bot user test this
@typescript-bot user test tsserver
@typescript-bot test top100
@typescript-bot test tsserver top100
@typescript-bot perf test this
pretty please

I am moderately suspicious that the user tests didn't net anything, but maybe that's to be expected if this only happened in ambient contexts. (I guess top100 is still running)

Because the export default feels like a red herring, I'll just mention that
declare module 'replace-in-file' {
    export function replaceInFile(config: unknown): Promise<unknown[]>;
    export {};

    namespace replaceInFile {
        export function sync(config: unknown): unknown[];
  }
}

introduces the same issue.
In an ambient module, adding an export { ... } or a similar construct like export default ... implicitly changes whether all declarations are automatically exported.
TypeScript now recognizes these unfortunately confusing semantics more consistently, and issues an error on the fact that all declarations of replaceInFile need to agree in their modifiers.
In the case where there is no export {} or export default, there is no error because everything is implicitly exported anyway.
declare module 'replace-in-file' {
    export function replaceInFile(config: unknown): Promise<unknown[]>;

    namespace replaceInFile {
        export function sync(config: unknown): unknown[];
  }
}


Update active-directory-functional-levels.md

Windows Server 2012R2 and everything previous is now also End of Life.  Time went by, so fast.

@robinharwood, @Xelu86
Can you review the proposed changes?
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


feat: add server-side error handling

This PR adds error handling to the Express routes. I tried to keep the logic simple and just return a 500 status code for any invalid status code returned by the PayPal APIs.
Here are a couple manual tests I did:

Invalid CLIENT_ID/APP_SECRET:

Invalid amount for create order call:

Next thing we should add is some tests :D

<EMAIL_ADDRESS>


⚠ GitHub has degraded performance

In 2617b45, GitHub (https://www.githubstatus.com/api/v2/status.json) experienced degraded performance:

HTTP code: 200
Response time: 339 ms

Resolved: GitHub performance has improved in d66f2d5.


[SMALLFIX]Use static imports for standard test utilities for class RPCFileWriteResponseTest

[SMALLFIX]Use static imports for standard test utilities for class RPCFileWriteResponseTest

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17188/
Test PASSed.


Want to help you by creating Java Fundamental CheatSheet

This will be very beneficial for beginner

Same I am also ready to help regarding creating Java Cheat Sheet.

it's already done @imgk007


XGetWindowProperty[_NET_ACTIVE_WINDOW] failed (code=1)

Contour Terminal version
Contour Terminal Emulator 0.3.3-unreleased-master-4cb0bf33
Installer source
Github: source code cloned
Operating System
Ubuntu 22.04.1 LTS x86_64
Architecture
x86-64
Other Software
No response
Steps to reproduce

Start contour

Expected Behavior
Display terminal screen without any issues
Actual Behavior
XGetWindowProperty[_NET_ACTIVE_WINDOW] failed (code=1)
xdo_get_active_window reported an error
There are no windows in the stack
Invalid window '%1'
Usage: getwindowpid [window=%1]
If no window is given, %1 is used. See WINDOW STACK in xdotool(1)

before shell prompt.
https://user-images.githubusercontent.com/29275245/183113386-d655c479-cf88-416e-8823-e0ada7e2a387.mp4
Additional notes
This issue is present only when I start a new instance of contour(when no other instance is running) from dmenu or rofi. It is not started when I start another instance and contour is already running.

What window manger are you using?

This seems to look like it happens only when xdotool is present.

Can it be something in your shell init scripts? (bashrc/zshrc)? Please try renaming them temporarily just for the sake of testing this. Can you still produce the error then?

Maybe related to this https://github.com/jordansissel/xdotool/issues/194 ,
I can reproduce the crash but I don't get that the same error 😐

Maybe related to this jordansissel/xdotool#194 , I can reproduce the crash but I don't get that same error 😐

Awesome catch! I think that might actually be a possibility for the root cause. That is: the error is thrown when the terminal window is not in focus yet, because maybe it's not even constructed nor shown yet (!).
The thing is, the terminal session is spawned as soon as possible (not intentionally, just by design), this implies that the window might not be created nor shown nor visible yet when the .zshrc / .bashrc content is being executed.
This will change though in my QML frontend refactor as there I have to work around some Qt5 bug by artifically delaying shell spawning until a safe point. This might help mitigate the issue above.
Generally speaking, this is not a bug of Contour, it is however the expectation of xdotool, that there is a window in focus (which makes sense, most of the time at least). -- that's my understanding on the whole topic so far at least.

#777 is merged. Did this issue get fixed?

@whisperity actually, I think it should.
@adgai19, can you confirm that for you?

It did. Sorry for not responding earlier. I haven't had the chance to try the newer builds since I switched to nixos

It did. Sorry for not responding earlier. I haven't had the chance to try the newer builds since I switched to nixos

No worries. In case of tweaks required to install-deps.sh, PRs are gladly taken, @adgai19. Cheers ;)


a quick question about download process yolov5 master.zip file to root

i really love your repo here, it such a great project. i have a quick question for you. before inference process, there's a process to download file like this.
Downloading: "https://github.com/ultralytics/yolov5/archive/master.zip" to /root/.cache/torch/hub/master.zip
is it really mandatory to download the file ? or you can put the file locally ? thank you.

thanks for your answer, i just curious if it's can be local or not. keep up the good work mate.


Null extracted folder for artifact: ResolvedArtifact(componentIdentifier=androidx.activity:activity:1.8.0...............

gradle:6.7.1-bin
classpath 'com.android.tools.build:gradle:4.2.1'

anyone has a solution to this yet?


Input date range: disable today date for end date datepicker only

As per subject, please create an option to disable today date for end date datepicker only without disabling today date on start date datepicker.

Along these same lines, I would like to be able to have a clearBtn on the end date but not on the start date.
It seems that input-daterange ignores data- attributes on both the .input-daterange container and on the individual inputs


HBASE-25872 Add documentation for LoadBalancer about synchronization

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 28s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+1 :green_heart:
mvninstall
4m 38s
master passed

+1 :green_heart:
compile
0m 21s
master passed

+1 :green_heart:
shadedjars
9m 31s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 19s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m  5s
the patch passed

+1 :green_heart:
compile
0m 20s
the patch passed

+1 :green_heart:
javac
0m 20s
the patch passed

+1 :green_heart:
shadedjars
9m 23s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 17s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
0m 31s
hbase-balancer in the patch passed.

32m 12s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3267/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3267

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 36c4ba90b8b8 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 0955a7a22e

Default Java
AdoptOpenJDK-1.8.0_282-b08

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3267/1/testReport/

Max. process+thread count
284 (vs. ulimit of 30000)

modules
C: hbase-balancer U: hbase-balancer

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3267/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 31s
Docker mode activated.


fix chapter docker in README.md

There is some error in the instruction of how to run manim in Docker.

There is already a pr for that #586

This got fixed in #586.


[Bug] DnD a column does not move the footer

The component of the footer is not update the id correctly and affect the target of the DnD

Commit 08f325e8


In the alerts filter cannot find TestCase FQN

Affected module
The notifications module
Describe the bug
In the alerts filter cannot find TestCase FQN.
To Reproduce

Setup a tableCustomSQLQuery testcase and query the api to get the FQN under "fullyQualifiedName":

Setup an Alert (Filtering for Test Results works fine)

Try to find FQN for the testcase in the Alert filter.

Expected behavior
I would expect to be able to include/exclude Testcases based on their FQN in a notification channel.
Version:

OpenMetadata version: 1.2.4

@flikweert44 thank you for reporting. I am closing this as this is supported in 1.3.0 (the latest version released).


Included first test for GitLab code preview

Summary
This is the first test added for the GitLab code preview functionality new in v1.7.0
note: This is the first of several tests. I wanted to make a commit to work though the process.
Typical format for test cases:
Test coverage for https://github.com/mattermost/mattermost-plugin-gitlab/pull/299

Is the feature in stable branch or under review?

[x] Feature in stable branch (master, main)?

Test client

[x] Browser
[x] Desktop App (version, OS)

Test case saved: MM-T5454: Code Preview


Disabled SWC as replacement for Babel

Hello! I believe you're aware of this, so I guess this is more like a question. When I try to build the Next.js app, I get the following warning:
Process started
info  - Checking validity of types...
info  - Creating an optimized production build...
info  - Disabled SWC as replacement for Babel because of custom Babel configuration "babel.config.js" https://nextjs.org/docs/messages/swc-disabled
info  - Using external babel configuration from /Users/.../web-app/babel.config.js
Compiled successfully

May you please why we need a specific Babel configuration and if it's worth to forego SWC for it?

emotion is not yes fully supported without babel. Just need a bit of time.
But feel free to adapt.

@picosam latest news about swc.

The swcMinify: true starts to become stable with pretty much every library I throw on it and gives pretty much the same size gains than terser with a little speed gain.
Using swc with emotion is still a bit young. It doesn't work with some libs I tried and while emotion support is experimental, it seems to work fine for simple scenarios. The bundle size is a bit bigger (it does not read browserlist, so it emits es5 for now) and the speed is passing from 35s to 30secs for the nextjs-app example. Not a lot (see https://github.com/belgattitude/nextjs-monorepo-example/pull/1850)

@picosam
Nextjs 12.2 brought improvements (css-in-js/emotion supported out of the box). Not sure it's totally stable
But I will merge https://github.com/belgattitude/nextjs-monorepo-example/pull/2187 this soon.
Had to increase slightly the browser size-limit checks (few extra Kb's)


Is using sort.Sort safe when specifying more than one order column?

The Go stdlib documentation states that sort.Sort does not guarantee stability of the sorted results (sort.Stable does). Isn't stability a requirement when sorting the dataframe according to the content of multiple columns to guarantee correctness the way dataframe.Arrange is currently implemented?

You might be right. I'm curious to know if you could point an example to me that can cause a problem when using sort.Sort and how big of an impact can this be sorting performance.
My guess is that the memory copy time greatly outweights the extra overhead that O(n*log(n)*log(n)) calls to the swap function can cause, but I would like to get some numbers on this before pulling the trigger and changing sort.Sort to sort.Stable for Series.Order.

Here's an example of a test that demonstrates the issue:
func generateInts(seed int64, size, intRange int) []int {
	result := make([]int, size)
	rand.Seed(seed)
	// Random slice
	for ix := range result {
		result[ix] = rand.Intn(intRange)
	}

	return result
}

func TestDataFrame_Sort(t *testing.T) {
	size := 10
	d1 := dataframe.New(
		series.New(generateInts(1, size, 3), series.Int, "S1"),
		series.New(generateInts(1, size, 2), series.Int, "S2"))

	d2 := d1.Arrange(dataframe.Sort("S1"), dataframe.Sort("S2"))
	fmt.Println(d2.String())
	previousS1, previousS2 := d2.Elem(0, 0), d2.Elem(0, 1)
	for i := 1; i < size; i++ {
		currentS1, currentS2 := d2.Elem(i, 0), d2.Elem(i, 1)
		if !(previousS1.Less(currentS1) || previousS2.LessEq(currentS2)) {
				t.Errorf("%d: previousS1=%s, previousS2=%s, currentS1=%s, currentS2=%s",
					i-1, previousS1.String(), previousS2.String(), currentS1.String(), currentS2.String())
		}
		previousS1, previousS2 = currentS1, currentS2
	}
}

Performance of sort.Stable is notably worse than that of sort.Sort. Here's a benchmark using a dataframe with 4x100000 random integers (range 0-100000) sorted on two columns, one ascending and one descending:
BenchmarkDataFrame_Sort-2      	       5	 218578662 ns/op	50547024 B/op	     148 allocs/op
BenchmarkDataFrame_SortStable-2        3	 435763757 ns/op	50547024 B/op	     148 allocs/op
BenchmarkQFrame_Sort-2         	      30	  48992845 ns/op	  401536 B/op	       4 allocs/op

The QFrame benchmark is from a dataframe-like construct that I'm writing to support a Go implementation of  https://github.com/tobgu/qcache. Implementing that was how I stumbled over this issue. I'm not actually affected by it as a user.
For the QFrame I opted for a slightly different approach where I instead of sorting series per series i compare row by row. This is slower for single columns sorts but the time is not growing linearly with the number of columns you sort on. Something that was important to me since I currently base grouping, etc. on a sorted frame.
The performance difference between dataframe and QFrame is likely because of less allocations and less abstractions/interfaces. I'm trying to get it on par with Pandas performance-wise. It's just a prototype at the moment.
A slight optimisation that will produce correct results could be to use sort.Sort for the first column to sort and sort.Stable for the rest. I guess it's fairly common to only sort by one column.

If someone wants to submit a PR for stable sort as described by @tobgu, I'll merge it.

Woops I accidentally closed this issue sorry.


Use spaces in data keys for nicer legend labels.

This is a small change visually, but I think it looks much nicer. It is a bit odd to change the actual key of the data, as opposed to just formatting the key at display time, but I was unable to find a reasonable way to format the key for display without writing a whole new legend component.
I'd definitely understand if you feel this isn't really needed, which is why I pulled it out from the previous look-and-feel PR. :)
with this change:

before:

Yeah looks good to me. I think I'll keep this PR open to debug Cloudflare bot, if that's ok with you.


Fix indentation for Gist code samples

I modified the style used for rendering Gists in a post. The white-space
style was collapsing the white space at the beginning and end of the
line resulting in all code being left justified. I changed the style to
pre which preserves the whitespace and renders the Gist correctly.
Resolves #14

Oh, I haven't noticed this PR before. Today I will try to see))

Oh, I haven't noticed this PR before. Today I will try to see))

Your change affected other blocks, so I fixed it in my own way
After the changes

Before the changes

I removed this style from css. It didn't seem to have affected anything. Changes are already available in master

Your change affected other blocks, so I fixed it in my own way
After the changes

Before the changes

I removed this style from css. It didn't seem to have affected anything. Changes are already available in master


SD card problem with Arducam 4 cameras adapter board

I am trying to get to work 4 Cameras Adapter Board populated with 4 ArduCam mini OV2640, stacked on top of the Arduino UNO board. Everything works with up to 3 cameras. After attaching the last one (4-th) the SD card error keeps happening. The whole thing is powered by the external 5V, 4A stable power source. I tried to switch cameras to check if all modules are working properly. After adding the fourth camera it always fails. Please advice what to do.

@lesnie
Hi,
Please attach me some pictures of your hardware connection in detail

Hi,
the problem is that it is a clean thing - Original Arduino Uno board with 4 cams Adapter board on top populated with ArduCam Mini OV264 and properly formatted SD card. Everything works like it should with one, two or three cameras, fails with all four in place. Consistently. I was checking and all camera modules are good, because it doesn’t make a difference which three cameras are connected and at which place. Additional, beefy power (5V, up to 4A) provided through USB on adapter board. After removing one of the arducam modules, not important which one, the SD card can be used without a problem.

On 12 May 2018, at 03:59, UCTRONICS<EMAIL_ADDRESS>wrote:
@lesnie
Hi,
Please attach me some pictures of your hardware connection in detail
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@lesnie
We have tested our code and adapter board and it can work fine.
Don't worry and if your hardware is  defective and we will resend you a new one for free.
Please attached me your hardware connection picture.


aldehyde decarbonylase activity / octadecanal decarbonylase activity

'aldehyde decarbonylase activity' has a single child term 'octadecanal decarbonylase activity':
id: GO:0071771
name: aldehyde decarbonylase activity [2 EXP annotations]
namespace: molecular_function
def: "Catalysis of the reaction: a C(n) aldehyde = C(n-1) alkane + CO." [GOC:kad, PMID:6593720, PMID:8718622]
synonym: "decarbonylase activity" EXACT [GOC:kad, GOC:mah]
is_a: GO:0016830 ! carbon-carbon lyase activity
created_by: mah
creation_date: 2010-04-01T12:02:50Z
id: GO:0009924
name: octadecanal decarbonylase activity [0 EXP annotations]
namespace: molecular_function
def: "Catalysis of the reaction: octadecanal = heptadecane + CO." [EC:<IP_ADDRESS>, GOC:tb]
synonym: "octadecanal alkane-lyase activity" EXACT []
xref: EC:<IP_ADDRESS>
xref: MetaCyc:OCTADECANAL-DECARBONYLASE-RXN
xref: RHEA:30415
is_a: GO:0071771 ! aldehyde decarbonylase activity
The RHEA:30415 xref on GO:0009924 looks ~correct:
H+ + 2 NADPH + O2 + octadecanal = formate + H2O + heptadecane + 2 NADP+
But the EC:<IP_ADDRESS> xref on GO:0009924 is more general and looks like it should be moved to the parent GO:0071771:
name: aldehyde oxygenase (deformylating).
alt name: aldehyde decarbonylase.
def: a long-chain fatty aldehyde + H(+) + 2 NADPH + O2 <=> a long-chain alkane + formate + H2O + 2 NADP(+)
And there's a separate RHEA (RHEA:21440) for that more general reaction
a long-chain fatty aldehyde + H+ + 2 NADPH + O2 = a long-chain alkane + formate + H2O + 2 NADP+
BUT note the EC:<IP_ADDRESS>/RHEA:21440 reaction defs don't quite match that currently given in GO:0071771....
Also, might want to obsolete GO:0009924 given it's the only child of GO:0071771 and it has 0 EXP annotations?

Reviewing this, I see that the PMID:6593720 xref used in the def of 'aldehyde decarbonylase activity' is also listed on the  RHEA:21440 entry.
So I propose to go ahead with this and do the following:

[ ] move the EC:<IP_ADDRESS> xref to the parent term aldehyde decarbonylase activity
[ ] add RHEA:21440 xref to the parent term aldehyde decarbonylase activity
[ ] Re-define the parent term aldehyde decarbonylase activity based on the RHEA xref

@pgaudet Do you want to obsolete 'octadecanal decarbonylase activity'?

Hmmm.  I think these two terms are both referring to EC:<IP_ADDRESS>/RHEA:21440 and need merging:
EC:<IP_ADDRESS>
name: aldehyde oxygenase (deformylating).
alt name: aldehyde decarbonylase.
def: a long-chain fatty aldehyde + H(+) + 2 NADPH + O2 <=> a long-chain alkane + formate + H2O + 2 NADP(+)
RHEA:21440 lists PMID:6593720 and PMID:22947199.
id: GO:0071771
name: aldehyde decarbonylase activity
namespace: molecular_function
def: "Catalysis of the reaction: a C(n) aldehyde = C(n-1) alkane + CO." [GOC:kad, PMID:6593720, PMID:8718622]
synonym: "decarbonylase activity" EXACT [GOC:kad, GOC:mah]
is_a: GO:0016830 ! carbon-carbon lyase activity
created_by: mah
creation_date: 2010-04-01T12:02:50Z
id: GO:1990465
name: aldehyde oxygenase (deformylating) activity
namespace: molecular_function
def: "Catalysis of the reaction a long-chain aldehyde + O2 + 2 NADPH = an alkane + formate + H2O + 2 NADP+." [GOC:mengo_curators, PMID:22947199, RHEA:21440]
xref: EC:<IP_ADDRESS>
xref: RHEA:21440
is_a: GO:0016830 ! carbon-carbon lyase activity
created_by: tt
creation_date: 2014-08-25T15:16:43Z

Dear all,
The proposal has been made to obsolete: GO:0009924 octadecanal decarbonylase activity. The reason for obsoletion is that this represents a specific substrate of the reaction GO:0071771 aldehyde decarbonylase activity.
The proposal has also been made to obsolete GO:1990465 aldehyde oxygenase (deformylating) activity, since this represents the same reaction as GO:0071771 aldehyde decarbonylase activity.
There are no annotations to these terms; these terms are not in any subset.
You can comment on the ticket: https://github.com/geneontology/go-ontology/issues/25056
Thanks, Pascale

Thanks!


contact section is updated now on nav bar

Description 📝

Fixes/Closes: #294
contact section is updated now on nav bar
Changes 💬

High-level overview of the changes introduced.
Step-by-step breakdown of the implementation, if applicable.

Additional Notes 📒
Include any additional information or context that might be helpful for the reviewers.
Also if you are fixing the already made PR and the reviewer demands some change in it then, in each PR you have to mention the issue you have fixed in it.

updated please check @PiyushKalyanpy

You have referred to the wrong Issue!!

I am unable to detect where I'm going wrong. Smoothness while scrolling is not working properly. Otherwise everything seems fit.

Hey there,
We are closing this pull request as the changes we requested were not implemented. Please create a new PR once you implement the changes.


Closing ifstream both if its readable and unreadable

Describe the issue (if no issue has been made)
In OpenGLShader.cpp method ReadFile() ifstream isn't closed if we're unable to read from it.
PR impact
List of related issues/PRs this will solve:
None
Proposed fix
A short description of what this fix is and how it fixed the issue you described.

Both instances of close seem unnecessary, RAII will handle this fine

Both instances of close seem unnecessary, RAII will handle this fine

So RAII will clean up, when the stream goes out of scope?

ifstream.close() is now removed, and comment added for clarity.

I feel the comment is unnecessary,  RAII is one of the basics of c++ so doesn't really need to be said.

I feel the comment is unnecessary, RAII is one of the basics of c++ so doesn't really need to be said.

Good point, but as this version of Hazel also is for the community to learn, so wouldn't it be nice to have?

Well, yes RAII will take care of it, but will it still work 'fine' when I load let's say a big amount of shaders all after each other? Taking all the memory into account that this would consume during them being open? I don't expect that they will be closed before the next one will open if you leave it up to resource manager?
Either way, it doesn't feel like an overhead to close them explicitly? Maybe I am missing something?

LovelySanta, I think RAII makes sure that the destructor is being run when the end of it's scope is reached. Only if you need to catch any errors when closing the stream, then you probably need to close it explicitly.
https://stackoverflow.com/questions/748014/do-i-need-to-manually-close-an-ifstream


How to show a tab on click of a button?

Hey,
Im confused here. I need to show 2nd or third tab from first on click of the button inside the first tab. I tried to give # link of second tab  to the href but no change.
Could someone help?

With no more info the only thing we can recommend to you is to take a look at the documentation:
$('ul.tabs').tabs('select_tab', 'tab_id');
http://materializecss.com/tabs.html#method


[WIP] Update dependencies

While trying to diagnose why a number of cargo subcommands were giving Invalid parameters on this machine, I upgraded some dependencies. Unfortunately, it didn't fix the issue, so I have no idea if this works. It compiles, though!
Code changes:

env_logger::init() no longer returns an error, it panics itself.
CargoError::from has moved to the failure crate, so use that to construct our errors.

Thanks!

There was a recent bug in nightly Cargo that broke subcommands - it should be fixed in the next nightly or so.


Updating bzr to 2.6 and removing --allow-unverified flag from requirements

As mentioned in #1369 the 2.5b4 version of bzr can make problems as it is not available on PyPI. The 2.6 version however is. I assume it's save to upgrade to the newer version based on this line from the release notes:

Bazaar 2.6.0 is fully compatible both locally and on the network with 2.0, 2.1, 2.2, 2.3, 2.4 and 2.5, and can read and write repositories generated by all previous versions.

I also removed the --allow-external and --allow-unverified flags from requirements files as they might introduce security risks. The installation still works as the Travis builds will show.

See successfull travis build installing bzr: https://travis-ci.org/rtfd/readthedocs.org/builds/68298005#L653

Yay. This brings me joy.


update docs with npm version

could we please add a link to the npm page in the repo readme. I can submit a pr with the updated docs just want an A okay that's fine to do
https://www.npmjs.com/package/d3-ease

I don’t currently link to the npm website, but you can find the latest releases on the releases tab.


Add OAuth2 authorization request type for dynatrace_http_monitor resource

Hey @Dynatrace-Reinhard-Pilz and @kishikawa12
Apologies if this is already an issue or future enhancement I missed, but I'm wondering if there's any plans to extend the dynatrace_http_monitor resource to include the OAuth2 authorization request type. I noticed that exporting an http monitor resource with OAuth2 request type worked but it just defaulted to using the http request type config instead. Then I took a closer look in the http monitor resource documentation here and didn't see anything for OAuth2. The pattern for this works in the UI as documented here so it would be awesome if the provider could eventually do the same. Thanks!

Hello @ekleiner14
You have apparently stumbled across a limitiation of the Dynatrace REST API.
I just retraced your steps (using the REST API directly, not the provider). Essentially I created a HTTP monitor that uses OAuth2 and then duplicated it via REST API.
That "clone" suffers from the same shortcomings than what you've described when using the Terraform Provider.
My understanding is, that an OAuth2 Request is as a matter of fact nothing else than a normal HTTP Request - the WebUI just prepopulates the necessary properties for you as a convenience.
Based on your findings it's however clear that the WebUI must maintain a tiny bit of information that's not available via REST API - and therefore also not to the provider.
As a next step I need to reach out to Dynatrace R&D. I suspect that with a few changes in the Schema for HTTP Monitors we can add proper support for OAuth2.

Thank you @Dynatrace-Reinhard-Pilz - please keep me updated :)

Hello @ekleiner14 ,
I've received response from R&D about this.
They confirm our findings - when created via REST API the WebUI is not clearly stating, that one of the requests is being used for OAuth2 authentication for another request. They also are assuring that it doesn't affect the actual functionality.
In other words, if you export such a HTTP Monitor using the provider, that dynatrace_http_monitor resource will produce an HTTP Monitor with working OAuth2 functionality. What matters are the pre- and post-execution scripts.
Makes sense?

Hey @Dynatrace-Reinhard-Pilz - I see where I went wrong. At first the output wasn't what I expected and I had a different issue that prevented it from working. After trying again and taking a closer look with R&D's feedback I was able to get what I need. Thanks!


[HCA DX] Allow user to filter by access facet

Similar to AnVIL DX mocks:

See implementation for AnVIL DX.

HI @frano-m can  you move "Access" to under "Project" like


#13 Search for YouTube playlist

Value Proposition
As a user
I want to search for a YouTube channel to get its playlists
So that I can see each playlist and the titles and URLs of the videos in a playlist
Description

Acceptance Criteria

[ ] Input field with search bar
[ ] Clicking on the Search button, all playlists of a specific channel are shown
[ ] The URLs and titles of the videos in the playlist are shown as a list

Tasks

[ ] Create a new form with input field and search button
[ ] Fetch YouTube Data API v3
[ ] onSubmit search for channel and retrieve channel ID
[ ] Fetch with the channel ID for playlists
[ ] Fetch with the playlist ID for all playlist videos
[ ] Get all video IDs
[ ] Interpolate video IDs into YouTube embed link
[ ] Create YouTubePlaylists component
[ ] Display interpolated links and video titles in YouTubePlaylists
[ ] Show YouTubePlaylists on the create page after submitting on the Search button

Complexity
XXL

Hi Vlady,
fetchy fetchy, hm? 😁
looks good to me. 👍
cheers
R.


[BUG]Batch operations with retriable errors are not retried

Library name and version
Azure.Data.Tables 12.8.3
Describe the bug
Performing batch operations on tables (and I assume blobs) returns multipart/mixed responses. Because the OData protocol for these response types is to ALWAYS return a 202, we never fall into the ShouldRetry section of RetryPolicy, as Response.IsError is always false due to the 202.
Azure.Data.Tables, after receiving the response, does still parse these multipart responses and raises an exception when one of the inner responses fail, but if any of those are 429/500-range we have already passed the HttpPipelinePolicy.SendAsync that contains the retry logic.
Expected behavior
When an inner response of the batch response is a retriable response, I would expect a retry to have occurred.
But is this a reasonable expectation? Retrying the entire response would re-upsert the entire batch, so the ones that succeeded would be upserted again.
Actual behavior
No retries occur, and our services receive the following error from this section of code:
Operation could not be completed within the specified time.
RequestId:
Time:
Status: 500 (Internal Server Error)
ErrorCode: OperationTimedOut
Reproduction Steps
private const int TableChunkSize = 100;

// Provide a large number of entities to result in a large number of calls to Storage
// Larger table entities (bringing the batch closer to the 4MB limit) has been found
// to result in more 500s from storage. Smaller entities typically results in 429.
public Task UpsertEntitiesAsync(List<MyTableEntity> entities, string storageUri, string tableName)
{
    var tableServiceClient = new TableServiceClient(storageUri, new DefaultAzureCredential());
    var tableClient = tableServiceClient.GetTableClient(tableName);
    string partitionKey = entities.First().PartitionKey;

    // Parallel.ForEachAsync to hit it much harder
    foreach (var chunk in entities.Chunk(TableChunkSize))
    {
        var transactions = chunk.Select(e => new TableTransactionAction(TableTransactionActionType.UpsertMerge, e));
        await tableClient.SubmitTransactionAsync(transactions).ConfigureAwait(false);
    }
}

Environment
.NET SDK:
Version:           8.0.403
Commit:            c64aa40a71
Workload version:  8.0.400-manifests.e99c892e
MSBuild version:   17.11.9+a69bbaaf5
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.26100
OS Platform: Windows
RID:         win-x64

It should also be noted that attempting to build a custom RetryPolicy to parse multipart responses was incredibly difficult. Because ShouldRetryAsync is only called on IsError, I found myself having to completely copy-paste the RetryPolicy implementation because I cannot properly call ProcessNextAsync and inject myself deeper into the retry logic to make more nuanced determinations into whether or not the response was an error.
Possibly a separate issue to open for feature request?

Hi @rlrossiter, thank you for bringing this to our attention.
You are right, retrying the entire batch could lead to re-upserting already successful operations, which is not ideal. This behavior might be more of a feature request than a bug, so we will review this issue further to prioritize it accordingly.

Hi @rlrossiter - In your repro, could you share an example mix of successful and retriable status codes you are seeing? Batch transactions are guaranteed to be all-or-nothing, so it isn't possible to get partial success. What you are probably seeing is that one or more sub-requests are failing the whole batch. However, it should be possible to devise a custom RetryPolicy that retries only if all the failed sub-requests are retriable.

Hi @rlrossiter - In your repro, could you share an example mix of successful and retriable status codes you are seeing? Batch transactions are guaranteed to be all-or-nothing, so it isn't possible to get partial success. What you are probably seeing is that one or more sub-requests are failing the whole batch. However, it should be possible to devise a custom RetryPolicy that retries only if all the failed sub-requests are retriable.

So we haven't been able to repro locally, as a single instance of our application hasn't been able to push storage to the point of 429/500. We only see it in our services which doesn't yet have multipart parsing to determine if it's all or nothing. All we receive from the TableRestClient is a 202, and at least one subrequest failed with 500. We're working to add metrics that parse the multipart response to see what the response count internally looks like, I can report back when we have that information.
I tried implementing a custom RetryPolicy, but because the original response is a 202, the response isn't classified as IsError, so we can't simply override ShouldRetryAsync. We also cannot inject into the ProcessNextAsync of RetryPolicy to allow us to determine IsError and then let RetryPolicy handle the retry logic.
It might be possible to put a per-retry HttpPipelinePolicy (not RetryPolicy) where I modify the response message to be the first error subresponse and let RetryPolicy then make its determination on that, but it feels quite hacky.

I'm seeing this behavior for one of our applications. We're using Azure.Data.Tables 12.8.0.
We've got code sending batched upserts like so:
public async Task UpsertBatchAsync(List<T> entities)
{
    var actions = entities.Select(e => new TableTransactionAction(TableTransactionActionType.UpsertReplace, e));
    await this.table.SubmitTransactionAsync(actions);
}

Our table client is configured for 3 retires with exponential backoff, but we're seeing the following exception almost immediately after sending the transaction. It doesn't look like any retry is happening.
Azure.Data.Tables.TableTransactionFailedException: Operation could not be completed within the specified time.
RequestId:7a392adf-b002-001c-033d-4d10fa000000
Time:2024-12-13T09:03:20.4504716Z
Status: 500 (Internal Server Error)
ErrorCode: OperationTimedOut

Content:
{"odata.error":{"code":"OperationTimedOut","message":{"lang":"en-US","value":"Operation could not be completed within the specified time.\nRequestId:7a392adf-b002-001c-033d-4d10fa000000\nTime:2024-12-13T09:03:20.4504716Z"}}}

Headers:
X-Content-Type-Options: REDACTED
Cache-Control: no-cache
DataServiceVersion: REDACTED
Content-Type: application/json;odata=minimalmetadata;streaming=true;charset=utf-8

@coreymasters-microsoft
There may be something about the entities in the batch or the partition being updated that make this batch too complex to be completed. Try sending fewer entities in the batch collection to see if there is a threshold that succeeds.

@christothes thanks for the reply.
We submit many of these batches with the same size and similar information and have for a few years. We see this transient server failure rarely, but recently implemented a retry policy on the TableClient. We were surprised to see that a retry did not occur.

@coreymasters-microsoft  -
Do you have a link to the custom retry policy? My guess is that you are expecting the API call's response code to be an error code, but most batch operations complete with a 202 status code for the envelope response, and only one or more of the multipart sub responses failed with a failed status code.
Your retry policy would have to parse the multipart response and inspect the status codes of them to reason about the batch failures as the client does here.
But to do this for each operation in a retry policy would be pretty inefficient. In addition, if you needed to re-construct the batch operation, it would be complicated to do in a retry policy. It might be better to wrap batch calls with a try/catch block that re-issues the batch as appropriate after inspecting the sub-responses

It might be better to wrap batch calls with a try/catch block that re-issues the batch as appropriate after inspecting the sub-responses

This is actually what we've done to work around the problem:
private readonly AsyncRetryPolicy _retryPolicy = Policy
            .Handle<RequestFailedException>(RequestFailedExceptionsUtils.IsTransientError)
            .WaitAndRetryAsync(settings.Value.NumberOfRetries, retryAttempt => TimeSpan.FromSeconds(Math.Pow(2, retryAttempt) * settings.Value.ExponentialBackoffDelayMultiplierInSeconds));

...

Response<IReadOnlyList<Response>> response = await _retryPolicy.ExecuteAsync(() => tableClient.SubmitTransactionAsync(batchTransactions, cancellationToken)).ConfigureAwait(false);

But isn't this doing the same thing as an HTTP-pipeline-internal retry policy should be doing? Assuming we wouldn't try and get smart around which internal batch item was failed.

The tricky thing about handling retries for batch transactions is that the whole batch may not be retriable as-is unless it is a very small subset of scenarios. For example, 429 responses or batch intermittent timeouts. For a robust solution, you'd need to inspect all the non-successful sub-response status codes and make a decision about what to do. Sometimes this may involve modifying the batch before retrying. How you decide to correct the error may be dependent on your specific scenario, so there is not a built-in default solution in the SDK.

@rlrossiter thank you for that; much appreciated. Also @christothes TY for the guidance and discussion. I think our scenario is fairly straightforward and allows us to retry the whole batch in this case.
I'm still with @rlrossiter on this one, It feels odd for the TableClient with retries configured to just not do retires in this case. I understand the nuance with batched transactions you've mentioned, however.


default kernel selection is being changed after save

Describe the bug
Please provide as much information as you can.
Upon saving a notebook file (specifically .dib here but the .ipynb books exhibit the same behavior on me) where creation of notebook I select "F#".
Upon first save, the code box resets to csharp. Looking at the file in a text editor shows
{"kernelInfo":{"defaultKernelName":"fsharp","items":[{"aliases":[],"languageName":"fsharp","name":"fsharp"}]}}
which should be ok. Manually changing the block back to F# then adding a few new ones, then save again.
file changes to:
{"kernelInfo":{"defaultKernelName":"csharp","items":[{"aliases":[],"name":"csharp"},{"aliases":[],"languageName":"fsharp","name":"fsharp"}]}}
This happens enough times in trying to use notebooks to be quite annoying, and has happened with .dib and .ipynb
Issues that are similar in nature are:
https://github.com/dotnet/interactive/issues/1309
https://github.com/dotnet/interactive/issues/1038
and in vscode are:
https://github.com/microsoft/vscode-jupyter/issues/4757
https://github.com/microsoft/vscode/issues/140673
https://github.com/microsoft/vscode-jupyter/issues/4757
Please complete the following:
Which version of .NET Interactive are you using? (In a notebook, run the #!about magic command. ):

Version: 1.0.520801+1e5d205f82c7f5bb1b2e5ce60c680491552cfa23
Library version: 1.0.0-beta.24208.1+1e5d205f82c7f5bb1b2e5ce60c680491552cfa23
Build date: 2024-04-08T08:18:30.0000000Z

OS

[ ] Windows 11
[ ] Windows 10
[ ] macOS
[X] Linux (Please specify distro) : Debian 12 (LMDE6)
[ ] iOS
[ ] Android

Browser

[ ] Chrome
[ ] Edge
[ ] Firefox
[ ] Safari

Frontend

[ ] Jupyter Notebook
[ ] Jupyter Lab
[ ] nteract
[X] Visual Studio Code
[ ] Visual Studio Code Insiders
[ ] Visual Studio
[ ] Other (please specify)

vscode info:

Version: 1.88.1
Commit: e170252f762678dec6ca2cc69aba1570769a5d39
Date: 2024-04-10T17:34:12.840Z
Electron: 28.2.8
ElectronBuildId: 27744544
Chromium: 120.0.6099.291
Node.js: 18.18.2
V8: 12.0.267.19-electron.0
OS: Linux x64 6.1.0-20-amd64

Screenshots
If applicable, add screenshots to help explain your problem.
A video demo is at the link of https://www.dropbox.com/scl/fi/9p7nzjarbmciyqitib63c/2024-04-18-14-19-55.mp4?rlkey=tknlky6vs9twikwbtfb3ahr13&dl=0
with the following notes:
0:01:20 saving file after selecting F# as default kernal changes the code window to C# when save is done.
0:03:52 shows the text of the example1.dib file after adding a couple new blocks and saving. It has changed some defaults to csharp.

@jonsequitur
This above issue makes Dotnet.Interactive practically unusable to anyone using any other kernel but C#.
I was looking at this code:
https://github.com/dotnet/interactive/blob/11e9be0dae01de51a59ef8e43a0d59dd9c1d3399/src/polyglot-notebooks-vscode-common/src/metadataUtilities.ts#L99C44-L99C52
It's looking like its' accessing
const kernelInfo = interactiveDocument.metadata.kernelInfo;

where in fact in .ipynb files that should be
const kernelInfo = interactiveDocument.metadata.polyglot_notebook.kernelInfo;

At least if this is the metadata that it's accessing:
{
  "metadata": {
    "kernelspec": {
     "display_name": ".NET (F#)",
     "language": "F#",
     "name": ".net-fsharp"
    },
    "polyglot_notebook": {
     "kernelInfo": {
      "defaultKernelName": "fsharp",
      "items": [
       {
        "aliases": [],
        "languageName": "fsharp",
        "name": "fsharp"
       }
      ]
     }
    }
   },
   "nbformat": 4,
   "nbformat_minor": 2
}

Can you look at it, thanks.


Ted's sas2R

add this: https://hutchdatascience.org/data_snacks/r_snacks/sas2r.html as a resource

could possibly add this too:https://hutchdatascience.org/data_snacks/r_snacks/janitor.html


[Platform] Add commissioning test for Java matter controller app

Reproduction steps / Feature
We need to validate/gate the Commissioning API for android/Java

Platform
android
Platform Version(s)
No response
Type
CI tested
(Optional) If manually tested please explain why this is only manually tested
No response
Anything else?
No response

https://github.com/project-chip/connectedhomeip/pull/23714


squashfs: fix off by one in directory counting

This was causing "corrupted directory, had 257 entries instead of max
256" errors, however this was because parseDirectoryHeader had already
added one to the count.

Rebased!


avoid warning when cargo not found

The most common case is that if cargo is not installed to  /usr but somewhere else such as /usr/local (is this the default install prefix? don't remember...), user will see cargo: command not found after sudo su executed.
P.S. sorry for previous wrong PR... there was a redundant commit.

r? @huonw
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+ 6bb187c1b9b1f2924115c0604a4b42f7fcc96e97
Thanks!

:hourglass: Testing commit 6bb187c with merge 4bc0456...

:sunny: Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64


savemat and xray DataArrays

Currently, savemat saves xray DataArrays as empty structs, even though there is an obvious representation as a ndarray (in fact np.asarray works on DataArrays).
More generally, I think savemat should check for the presence of the __array__ special method and also print a warning (... or even raise an exception) when having to convert an unrepresentable object by an empty struct.

Perhaps an even better approach would be to provide a generic function (functools.singledispatch) that would allow registering custom pre-save converters.

I think the right solution here is that savemat should do some sort of normalization utilizing np.asarray, e.g., mdict = dict((k, np.asarray(v)) for k, v in mdict.items()). Failing silently is very un-Pythonic.

asarray seems simpler than custom pre-save converters. The latter would mean that any ndarray-like data structure would need to be aware of  scipy.io. Unless I'm misunderstanding what you mean @anntzer?

The pre-save converter could default to calling asarray, but data structures with more information available (... such as xray's DataArray) could always hook it with something more complex.

Ah okay, that makes sense.


Possibility of spaces in label (ICDAR)

Summary

How to test

Checklist

[ ] I submit my changes into the develop branch
[ ] I have added description of my changes into CHANGELOG
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues)

License

[ ] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[ ] I have updated the license header for each file (see an example below)

# Copyright (C) 2020 Intel Corporation
#
# SPDX-License-Identifier: MIT

@zhiltsov-max , the segmentation task assumes that the text will be a character. Do you think you need to do a similar check there?

No, but it is a good place to raise an error, so the user could understand the problem.

Please fill the description and update changelog.


Update the concierge message wording for custom feeds

Coming from here
Problem
BUG: Concierge message after submitting custom feed uses the term commercial card feed , which we aren't using anywhere else in Newdot. Let's change this to custom, to match the product. :thread:
Solution
I'm thinking we change this message to %card network% custom card feed .
Issue OwnerCurrent Issue Owner: @mountiny

Was waiting for the confirmation of the copy


No useable private keys found

$ gpg -K
/home/megas/.gnupg/pubring.kbx
------------------------------
sec   rsa4096 2018-10-15 [SC] [expires: 2023-10-14]
      5A5E5D9XXXX4B5F93F01XXXX8CE297B6C1854
uid           [ unknown] MyName <email>
ssb   rsa4096 2018-10-15 [E] [expires: 2023-10-14]

$ gopass
It seems you are new to gopass. Do you want to run the onboarding wizard? [Y/n/q]: Y
[init] [local] Initializing your local store ...

Error: failed to run onboarding wizard: failed to init local store: failed to read user input: no useable private keys found

The gpg key was imported from another computer.

I suspect that the trust is not set correctly for the key. Trust levels full or ultimate should work afair.

You're right, I changed the trust level to ultimate and gopass accepted this key

Just had same problem, it fixed  it.
Would be nice if the error message in the CLI pointed to that direction

For those arriving here and still running into this issue, maybe with a Keybase account and/or a Yubikey, if you have several public keys that have been uploaded to Keybase or another keyserver using the same email address but some of them have expired, if you do gpg --edit-key yourEmail@domain gpg>trust 5 (for ultimate) you are probably NOT editing the correct key. After you run gpg --card-edit gpg> fetch (if you have a Yubkey and uploaded to Keybase), you need to run gpg --list-secret-keys and using the portion after the 0x of the UNEXPIRED/currently valid key (sec# if using a Yubikey or in gpg --list-keys after the 0x in the pub line), then run gpg --edit-key<PHONE_NUMBER>ABCDEF and run gpg> trust 5 on this specific key, and you shouldn't get the error anymore.
Oddly enough I was able to use the GPG key for SSH and Git cloning by just trusting the email address, but in order to get gopass to like it I had to specify the correct key and trust it specifically.


[CHANGELOG] Prepare v4.11.2

For posterity: Still think this is a bad idea. Quite disappointed we're letting users blindly fall into this trap.

@eps1lon At least, I have updated the wording to stick to what we said initially, so no change in policy :).
I believe that keeping third-party dependencies up-to-date helps with the migration. I wish we could have React 17 as a selling argument to switch to v5 but considering how little breaking changes there are (mostly stubble changes around the event), I think that most teams will try to upgrade anyway. Our examples and codesandbox demo on v4 opens with React v17.


Change Port

Hi there,
Is it possible to change the port to something else? Preferably opening it up to allow users to easily modify the port to their choosing.
Kind regards,
Rishi

The port changes to 3001 when running npm run dev but I am unable to access the web page with the following error:
TypeError [ERR_INVALID_URL]: Invalid URL
    at new NodeError (node:internal/errors:387:5)
    at URL.onParseError (node:internal/url:564:9)
    at new URL (node:internal/url:644:5)
    at parseUrl (/root/restorePhotos/node_modules/next-auth/utils/parse-url.js:17:16)
    at Object.<anonymous> (/root/restorePhotos/node_modules/next-auth/react/index.js:72:34)
    at Module._compile (node:internal/modules/cjs/loader:1120:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1174:10)
    at Module.load (node:internal/modules/cjs/loader:998:32)
    at Module._load (node:internal/modules/cjs/loader:839:12)
    at Module.require (node:internal/modules/cjs/loader:1022:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at next-auth/react (/root/restorePhotos/.next/server/pages/_app.js:43:18)
    at __webpack_require__ (/root/restorePhotos/.next/server/webpack-runtime.js:33:42)
    at eval (webpack-internal:///./pages/_app.tsx:9:73)
    at __webpack_require__.a (/root/restorePhotos/.next/server/webpack-runtime.js:97:13)
    at eval (webpack-internal:///./pages/_app.tsx:1:21)
    at ./pages/_app.tsx (/root/restorePhotos/.next/server/pages/_app.js:22:1)
    at __webpack_require__ (/root/restorePhotos/.next/server/webpack-runtime.js:33:42)
    at __webpack_exec__ (/root/restorePhotos/.next/server/pages/_app.js:75:39)
    at /root/restorePhotos/.next/server/pages/_app.js:76:28
    at Object.<anonymous> (/root/restorePhotos/.next/server/pages/_app.js:79:3)
    at Module._compile (node:internal/modules/cjs/loader:1120:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1174:10)
    at Module.load (node:internal/modules/cjs/loader:998:32)
    at Module._load (node:internal/modules/cjs/loader:839:12)
    at Module.require (node:internal/modules/cjs/loader:1022:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object.requirePage (/root/restorePhotos/node_modules/next/dist/server/require.js:88:12)
    at /root/restorePhotos/node_modules/next/dist/server/load-components.js:45:54
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Promise.all (index 1)
    at async Object.loadComponents (/root/restorePhotos/node_modules/next/dist/server/load-components.js:43:33)
    at async DevServer.findPageComponents (/root/restorePhotos/node_modules/next/dist/server/next-server.js:563:36)
    at async DevServer.findPageComponents (/root/restorePhotos/node_modules/next/dist/server/dev/next-dev-server.js:1041:20)
    at async DevServer.renderPageComponent (/root/restorePhotos/node_modules/next/dist/server/base-server.js:951:24)
    at async DevServer.renderToResponse (/root/restorePhotos/node_modules/next/dist/server/base-server.js:980:32)
    at async DevServer.pipe (/root/restorePhotos/node_modules/next/dist/server/base-server.js:407:25)
    at async Object.fn (/root/restorePhotos/node_modules/next/dist/server/next-server.js:759:21)
    at async Router.execute (/root/restorePhotos/node_modules/next/dist/server/router.js:253:36)
    at async DevServer.run (/root/restorePhotos/node_modules/next/dist/server/base-server.js:384:29)
    at async DevServer.run (/root/restorePhotos/node_modules/next/dist/server/dev/next-dev-server.js:743:20)
    at async DevServer.handleRequest (/root/restorePhotos/node_modules/next/dist/server/base-server.js:322:20) {
  input: '',
  code: 'ERR_INVALID_URL',
  page: '/'
}```

Hi @Nutlope, is this a known bug?

Hi. This issue actually has to do with his comment that auth is incomplete. You'll need to set your head to the commit he specified (1c5c8ac4f52a08f68a3091d3b21be8a65aef71f2) and then do "npm run dev". Here's how to do that:
https://coderwall.com/p/xyuoza/git-cloning-specific-commits

Thanks! Closing for now.


Windows环境下部署还是存在导入库问题

系统版本：windows server 2012
python版本：3.8
错误代码：
[2022-08-08 15:03:39,084 nonebot] ERROR: Failed to import "hoshino.modules.bili-notice-hoshino.bili_notice_hoshino", err
or: no library called "cairo-2" was found
no library called "cairo" was found
cannot load library 'C:\Program Files\UniConvertor-2.0rc5\dlls\libcairo-2.dll': error 0xc1
cannot load library 'libcairo.so.2': error 0x7e
cannot load library 'libcairo.2.dylib': error 0x7e
cannot load library 'libcairo-2.dll': error 0xc1
[2022-08-08 15:03:39,084 nonebot] ERROR: no library called "cairo-2" was found
no library called "cairo" was found
cannot load library 'C:\Program Files\UniConvertor-2.0rc5\dlls\libcairo-2.dll': error 0xc1
cannot load library 'libcairo.so.2': error 0x7e
cannot load library 'libcairo.2.dylib': error 0x7e
cannot load library 'libcairo-2.dll': error 0xc1
Traceback (most recent call last):
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python38-32\lib\site-packages\nonebot\plugin.py", line 253,
in load_plugin
module = importlib.import_module(module_path)
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python38-32\lib\importlib_init_.py", line 127, in import
_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1014, in _gcd_import
File "", line 991, in find_and_load
File "", line 975, in find_and_load_unlocked
File "", line 671, in load_unlocked
File "", line 783, in exec_module
File "", line 219, in call_with_frames_removed
File "C:\bot\HoshinoBot\hoshino\modules\bili-notice-hoshino\bili_notice_hoshino.py", line 4, in
from . import dymgr
File "C:\bot\HoshinoBot\hoshino\modules\bili-notice-hoshino\dymgr.py", line 5, in
from .res import drawCard
File "C:\bot\HoshinoBot\hoshino\modules\bili-notice-hoshino\res\drawCard.py", line 8, in
from .getImg import get_ico, get_Image, save_Image
File "C:\bot\HoshinoBot\hoshino\modules\bili-notice-hoshino\res\getImg.py", line 8, in
import cairosvg as svg
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python38-32\lib\site-packages\cairosvg_init.py", line 2
6, in
from . import surface  # noqa isort:skip
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python38-32\lib\site-packages\cairosvg\surface.py", line 9,
in
import cairocffi as cairo
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python38-32\lib\site-packages\cairocffi_init.py", line
48, in
cairo = dlopen(
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python38-32\lib\site-packages\cairocffi_init.py", line
45, in dlopen
raise OSError(error_message)  # pragma: no cover
OSError: no library called "cairo-2" was found
no library called "cairo" was found
cannot load library 'C:\Program Files\UniConvertor-2.0rc5\dlls\libcairo-2.dll': error 0xc1
cannot load library 'libcairo.so.2': error 0x7e
cannot load library 'libcairo.2.dylib': error 0x7e
cannot load library 'libcairo-2.dll': error 0xc1
[2022-08-08 15:03:39,115 nonebot] ERROR: Failed to import "hoshino.modules.bili-notice-hoshino.dymgr", error: no library
called "cairo-2" was found
no library called "cairo" was found
cannot load library 'C:\Program Files\UniConvertor-2.0rc5\dlls\libcairo-2.dll': error 0xc1
cannot load library 'libcairo.so.2': error 0x7e
cannot load library 'libcairo.2.dylib': error 0x7e
cannot load library 'libcairo-2.dll': error 0xc1
[2022-08-08 15:03:39,115 nonebot] ERROR: no library called "cairo-2" was found
no library called "cairo" was found
cannot load library 'C:\Program Files\UniConvertor-2.0rc5\dlls\libcairo-2.dll': error 0xc1
cannot load library 'libcairo.so.2': error 0x7e
cannot load library 'libcairo.2.dylib': error 0x7e
cannot load library 'libcairo-2.dll': error 0xc1
Traceback (most recent call last):
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python38-32\lib\site-packages\nonebot\plugin.py", line 253,
in load_plugin
module = importlib.import_module(module_path)
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python38-32\lib\importlib_init.py", line 127, in import
_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1014, in _gcd_import
File "", line 991, in _find_and_load
File "", line 975, in find_and_load_unlocked
File "", line 671, in load_unlocked
File "", line 783, in exec_module
File "", line 219, in call_with_frames_removed
File "C:\bot\HoshinoBot\hoshino\modules\bili-notice-hoshino\dymgr.py", line 5, in
from .res import drawCard
File "C:\bot\HoshinoBot\hoshino\modules\bili-notice-hoshino\res\drawCard.py", line 8, in
from .getImg import get_ico, get_Image, save_Image
File "C:\bot\HoshinoBot\hoshino\modules\bili-notice-hoshino\res\getImg.py", line 8, in
import cairosvg as svg
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python38-32\lib\site-packages\cairosvg_init.py", line 2
6, in
from . import surface  # noqa isort:skip
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python38-32\lib\site-packages\cairosvg\surface.py", line 9,
in
import cairocffi as cairo
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python38-32\lib\site-packages\cairocffi_init.py", line
48, in
cairo = dlopen(
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python38-32\lib\site-packages\cairocffi_init.py", line
45, in dlopen
raise OSError(error_message)  # pragma: no cover
OSError: no library called "cairo-2" was found
no library called "cairo" was found
cannot load library 'C:\Program Files\UniConvertor-2.0rc5\dlls\libcairo-2.dll': error 0xc1
cannot load library 'libcairo.so.2': error 0x7e
cannot load library 'libcairo.2.dylib': error 0x7e
cannot load library 'libcairo-2.dll': error 0xc1

我已经详细阅读 #6 和 #8 ，并按照https://www.jianshu.com/p/5f566c26334f和https://github.com/kushidou/bili-notice-hoshino/wiki/%E9%83%A8%E7%BD%B2%E6%8F%92%E4%BB%B6中的提示操作。

uppers/list.json文件是我手动创建的，重启多次依然提示错误，核心错误代码
no library called "cairo" was found
cannot load library 'C:\Program Files\UniConvertor-2.0rc5\dlls\libcairo-2.dll': error 0xc1
cannot load library 'libcairo.so.2': error 0x7e
cannot load library 'libcairo.2.dylib': error 0x7e
cannot load library 'libcairo-2.dll': error 0xc1

嗯，这个svg库引发了很多问题。我现在没有环境可以来模拟并修复你的问题，所以我计划从根本上解决掉它。

本来使用svg图像，一方面是从b站网页扒下来了这类型的文件直接用了，另一方面是预留以后的自定义元素尺寸的需求。
现在把svg直接转换成稍大的png图像保存，和调用，舍掉实时的类型转换了。

感谢回复！
我在查找资料尝试解决问题的过程中，安装过一个pycairo的第三方库，不知道对解决问题有没有思路上的帮助

感谢回复！ 我在查找资料尝试解决问题的过程中，安装过一个pycairo的第三方库，不知道对解决问题有没有思路上的帮助

与第三方库的版本、路径可能都有关系。这个SVG库我也不熟悉，属于是需要了才找来的，它所包含的诸多隐患我也没有做前期调研，最终决定还是抛弃这个库比较方便。之后再考虑svg转换的问题。

经过测试 安装requirements.txt中的插件以及pycairo以后已经
可以成功运行

经过测试 安装requirements.txt中的插件以及pycairo以后已经 可以成功运行

补充一下 需要uninstall cairosvg这个库


chore: eslint no-console rule

Summary | Résumé
No issue for this PR.

Added eslint rule for no-console.
Removed some console messages.
Changed some console messages to logMessage.
Updated PR template to remove the console.log checklist item.

Pull Request Checklist
Please complete the following items in the checklist before you request a review:

[x] Have you completely tested the functionality of change introduced in this PR? Is the PR solving the problem it's meant to solve within the scope of the related issue?
[x] The PR does not introduce any new issues such as failed tests, console warnings or new bugs.
[x] If this PR adds a package have you ensured its licensed correctly and does not add additional security issues?
[x] Is the code clean, readable and maintainable? Is it easy to understand and comprehend.
[x] Have you removed unnecessary lines such as console.log statements?
[x] Does your code have adequate comprehensible comments? Do new functions have docstrings?
[x] Have you modified the change log and updated any relevant documentation?
[x] Is there adequate test coverage? Both unit tests and end-to-end tests where applicable?
[x] If your PR is touching any UI is it accessible? Have you tested it with a screen reader? Have you tested it with automated testing tools such as axe?

@Moro-Code - for the *.js files, I was unable to import logMessage. I suggest that we ignore this rule for these files.


Magic quotes error outputted to the front-end even though disabled

On 2 production servers with 2 different SilverStripe 3.1 sites we are getting the ugly error:

Warning: get_magic_quotes_gpc support is being removed from Silverstripe. Please set this to off in your php.ini and see http://php.net/manual/en/security.magicquotes.php

Running phpinfo() in the site root shows all magic quotes settings are off.
This message originates here: https://github.com/silverstripe/silverstripe-framework/blob/3.1/core/Constants.php#L141
It seems to be the check if (function_exists('get_magic_quotes_gpc') && get_magic_quotes_gpc()) { ... } is not reliable.
Oddly magic quotes is checked for using a different technique (ini_get()) here: https://github.com/silverstripe/silverstripe-framework/blob/3.1/dev/install/install.php5#L493
Is this a better approach? I don't know. This is only happening on production servers we do not have control of so testing this is not easy.
To make the message go away we commented out the trigger_error( ... ).  We would now have to version all of /framework/ in our repo.

@digital360 @jaydenseric I'm still having this issue even though I have created a custom php.ini inside framework :( I'm getting a javascript error in my javascript console and can't use the CMS.
php.ini
magic_quotes_gpc = off
magic_quotes_runtime = Off
magic_quotes_sybase = Off
date.timezone = Pacific/Auckland
upload_max_filesize = 100M
post_max_size = 100M
memory_limit = 512M
---- error ----
Warning:  get_magic_quotes_gpc support is being removed from Silverstripe. Please set this to off in  your php.ini and see http://php.net/manual/en/security.magicquotes.php in /home/ar37com/public_html/framework/core/Constants.php on line 143

Warning:  ini_set() [ref.outcontrol]: Cannot change zlib.output_compression - headers already sent in /home/ar37com/public_html/framework/thirdparty/tinymce/tiny_mce_gzip.php on line 164

Warning:  Cannot modify header information - headers already sent by (output started at /home/ar37com/public_html/framework/core/Constants.php:143) in /home/ar37com/public_html/framework/thirdparty/tinymce/tiny_mce_gzip.php on line 177

Warning:  Cannot modify header information - headers already sent by (output started at /home/ar37com/public_html/framework/core/Constants.php:143) in /home/ar37com/public_html/framework/thirdparty/tinymce/tiny_mce_gzip.php on line 178

Warning:  Cannot modify header information - headers already sent by (output started at /home/ar37com/public_html/framework/core/Constants.php:143) in /home/ar37com/public_html/framework/thirdparty/tinymce/tiny_mce_gzip.php on line 179

Warning:  Cannot modify header information - headers already sent by (output started at /home/ar37com/public_html/framework/core/Constants.php:143) in /home/ar37com/public_html/framework/thirdparty/tinymce/tiny_mce_gzip.php on line 180
var tinyMCEPreInit={base:'/framework/thirdparty/tinymce',suffix:''};(function(e){var a=/^\s*|\s*$/g,b,d="B".replace(/A(.)|B/,"$1")==="$1";var c=
.....

@nimeso - It appears you're on a shared hosting environment, you need to make sure your hosting provider supports overriding php settings via your application.  If not, you might have to contact them.  Also, I'd try overriding within the .htaccess file using:
php_flag magic_quotes_gpc Off

@daOutlawz Still no luck at all with this one :( I talked to the hosting company and they said to try putting the php.ini file in the cms (I've now got php.ini files everywhere! lol) Still no luck at all and the hosting company is saying it's the frameworks fault... sigh :( anyone got any ideas?

@nimeso It's a hosting configuration problem and it's something they should be able to advise you how to turn off.
Also, the errors are just warnings so you should be able to turn those off too.


Kenkoooo/crawler

Codecov Report

Merging #201 into master will decrease coverage by 0.81%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master     #201      +/-   ##
==========================================
- Coverage   66.66%   65.85%   -0.82%
==========================================
  Files          17       17
  Lines         738      735       -3
  Branches      126      123       -3
==========================================
- Hits          492      484       -8
- Misses        128      136       +8
+ Partials      118      115       -3

Impacted Files
Coverage Δ

atcoder-problems-backend/src/crawler.rs
0% <0%> (ø)

atcoder-problems-backend/src/scraper.rs
29.82% <32.69%> (+29.82%)
:arrow_up:

...oder-problems-backend/src/sql/submission_client.rs
95.55% <83.33%> (-1.89%)
:arrow_down:

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 34701be...411d206. Read the comment docs.


[1.16.5] Crash when scrolling through the Mod Configs page - [ (w/ Catalogue & Configured) ]

Title ^
Scrolling down in the Mod Configs page crashes the game to desktop.
Versions:
Minecraft: 1.16.5
Forge: 36.2.19
Lootr: <IP_ADDRESS>
GIF of Crash:
https://i.gyazo.com/bf6d2a6123e2f85efc7ccc84b47bc517.mp4
Crash-Report:
https://gist.github.com/Gbergz/b9c292a4a36b32f52561b25b6ada20a1

Ah. Good old "player is null"... I constantly forget about that.


Don't add message to sharing payload if empty

For compatibility with Instagram sharing

Implemented in #28


Update nginx to use our own image

Description
This PR updates the nginx configuration to use our own custom image.
Changes include:

Adding a new .gcloudignore-nginx file
Creating a new GitHub Actions workflow for deploying nginx (deploy-nginx.yml)
Modifying the proxy configuration and deployment in Kubernetes
Adding a new nginx.Dockerfile for building our custom nginx image

Risk
Already live (ran the commands locally)
Deploy
Deploy nginx, apply infra.

Just to make sure i fully get it. Why do we need our own image?

@fontanierh the official images do not include proxy connect extension so we were relying on someone’s 5yo images. It was using a quite old nginx. It was not mandatory to use our own image but it allowed me to use the latest nginx.


The OpenAPI introspection resource returns an incorrect specification

Description:
We can add an introspection resource to an HTTP service by adding the following OpenAPI annotation:
@openapi:ServiceInfo {
    embed: true
}

When the OpenAPI contract path is not specified in the above annotation, OpenAPI compiler plugin checks that and we generate the OpenAPI specification by passing the service declaration node to the OpenAPI tool. The generate specification is added to the http:ServiceConfig annotation via a code modifier in the OpenAPI tool. Then the HTTP compiler plugin reads that field and generate a introspection resource and inject that to the existing service.
But the issue arises with the HTTP code modifier which actually adds the @http:Payload annotation to the request payload. Since this code modifier is engaged after the OpenAPI compiler plugin, the OpenAPI tool tries to generate the specification without this specific annotation which ends up in a incorrect specification.
Steps to reproduce:
Run the following service:
import ballerina/http;
import ballerina/openapi;

type Post record {
    string text;
};

@openapi:ServiceInfo {
    embed: true
}
service /text\-processing on new http:Listener(9098) {

    resource function post api/sentiment(Post post) {}
}

Use the introspection resource to retrieve the specification:
$ curl http://localhost:9098/text-processing/openapi-doc-dygixywsw

{
  "openapi" : "3.0.1",
  "info" : {
    "title" : "Text Processing",
    "version" : "0.1.0"
  },
  "servers" : [ {
    "url" : "{server}:{port}/text-processing",
    "variables" : {
      "server" : {
        "default" : "http://localhost"
      },
      "port" : {
        "default" : "9098"
      }
    }
  } ],
  "paths" : {
    "/api/sentiment" : {
      "post" : {
        "operationId" : "postApiSentiment",
        "parameters" : [ {
          "name" : "post",
          "in" : "query",
          "required" : true,
          "content" : {
            "application/json" : {
              "schema" : {
                "$ref" : "#/components/schemas/Post"
              }
            }
          }
        } ],
        "responses" : {
          "202" : {
            "description" : "Accepted"
          },
          "400" : {
            "description" : "BadRequest",
            "content" : {
              "application/json" : {
                "schema" : {
                  "$ref" : "#/components/schemas/ErrorPayload"
                }
              }
            }
          }
        }
      }
    }
  },
  "components" : {
    "schemas" : {
      "ErrorPayload" : {
        "required" : [ "message", "method", "path", "reason", "status", "timestamp" ],
        "type" : "object",
        "properties" : {
          "timestamp" : {
            "type" : "string"
          },
          "status" : {
            "type" : "integer",
            "format" : "int64"
          },
          "reason" : {
            "type" : "string"
          },
          "message" : {
            "type" : "string"
          },
          "path" : {
            "type" : "string"
          },
          "method" : {
            "type" : "string"
          }
        }
      },
      "Post" : {
        "required" : [ "text" ],
        "type" : "object",
        "properties" : {
          "text" : {
            "type" : "string"
          }
        }
      }
    }
  }
}

Note : the Post is considered as query parameter but it should be the request body.

Affected Versions:
Ballerina SwanLake Update 2201.9.0

AFAIR code modifiers execute before compiler plugin code analyzers, and those modifiers, as well as the analyzer, cannot preserve a specific order while executing. Since both HTTP and OpenAPI plugins are code modifiers, shall we check any mechanism to guarantee that the code modification has occurred? (For example, maintaining a cache or having a private flag to the service node)

Adding @azinneera for suggestions

ATM, we don't guarantee the order of execution of the modifier plugins. What is the behavior we expect here?

There is two imports openapi and http. Both have code modifiers. The requirement here is running the code modifier of the http module before running the openapi one.

We are planning to refactor the design using static resources. We need APIs to read and write the resources from compiler time. This has been tracked via: https://github.com/ballerina-platform/ballerina-lang/issues/43041


Use PyErrs for all Python-facing methods in dask_planner

As a follow up on reviews I received in #656, this PR replaces the panic calls within Python-facing methods in dask_planner and replaces them with Err(PyErr)), making modifications to signatures and logic where necessary to get this working.
This PR also adds/modifies the custom exceptions so that they can be used throughout the codebase instead of PyErr::new.

This would also resolve #510

Codecov Report

:exclamation: No coverage uploaded for pull request base (datafusion-sql-planner@f948429). Click here to learn what that means.
The diff coverage is n/a.

@@                    Coverage Diff                    @@
##             datafusion-sql-planner     #662   +/-   ##
=========================================================
  Coverage                          ?   65.71%
=========================================================
  Files                             ?       73
  Lines                             ?     3634
  Branches                          ?      748
=========================================================
  Hits                              ?     2388
  Misses                            ?     1098
  Partials                          ?      148

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

I'm going to merge this since the failures are simply codecov failing to upload. Thanks again @charlesbluca

Thanks @jdye64! Yeah for context I'm currently working on #666 to make it so Codecov uploads won't block CI


[5.1] Register the Fresh command

Closes #9159

There actually isn't meant to be a fresh command. It should be removed.

Out of interest @taylorotwell, what's the reason for removing it?

There is not really any scaffolding to be removed.
On Tue, Jun 9, 2015 at 1:30 PM, James Brooks<EMAIL_ADDRESS>wrote:

Out of interest @taylorotwell https://github.com/taylorotwell, what's
the reason for removing it?
—
Reply to this email directly or view it on GitHub
https://github.com/laravel/framework/pull/9161#issuecomment-110456357.


Misc fixes

I linked the webserver on Zulip and realized it doesn't work.
A fix for hole hovers -- now a := .. in a do-block shows the type of a on hover rather than just Type.

* This was because there is a synthetic hole there whose _value_ is the type.

The server being affected by that is a bug. Fixed in https://github.com/leanprover/lean4/pull/564.


Fixed "ok" SlackNotificationsCore.php

When you commit a change, curl_exec add an ok(some hosting providers misconfig idk)

Thanks! You also moved curl_setopt($h, CURLOPT_RETURNTRANSFER, true);, was this intentional?

Yep it was, without CURLOPT_RETURNTRANSFER set to true, curl stills printing 'ok'

Thanks! :) Merging in....


Add v-calendar package supported vue 3

I'm use this package in current project. He  look likes good. 🙂

Realy helpful package