HADOOP-17650. Exclude org.restlet.jee dependencies to unblock build failure with Maven 3.8.1

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 43s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
46m 15s

trunk passed

+1 :green_heart:
compile
0m 40s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
compile
0m 39s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
mvnsite
0m 47s

trunk passed

+1 :green_heart:
javadoc
0m 23s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
0m 20s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
shadedclient
62m 48s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 40s

the patch passed

+1 :green_heart:
compile
0m 34s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javac
0m 34s

the patch passed

+1 :green_heart:
compile
0m 32s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
javac
0m 32s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 37s

the patch passed

+1 :green_heart:
xml
0m  2s

The patch has no ill-formed XML file.

+1 :green_heart:
javadoc
0m 15s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
0m 15s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
shadedclient
14m 36s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
0m 47s
/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-applications_hadoop-yarn-applications-catalog_hadoop-yarn-applications-catalog-webapp.txt
hadoop-yarn-applications-catalog-webapp in the patch passed.

+1 :green_heart:
asflicense
0m 34s

The patch does not generate ASF License warnings.

83m 42s

Reason
Tests

Failed junit tests
hadoop.yarn.appcatalog.application.TestAppCatalogSolrClient

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2938/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2938

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell xml

uname
Linux 8e90aa274219 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 909329839d930ccddd663f7f316ab2c17eaec6ae

Default Java
Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2938/1/testReport/

Max. process+thread count
544 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-applications-catalog/hadoop-yarn-applications-catalog-webapp U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-applications-catalog/hadoop-yarn-applications-catalog-webapp

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2938/1/console

versions
git=2.25.1 maven=3.6.3

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.


Enable user provided service-account-signing-keys

kube-aws presently assumes that we will always use the apiservers key for signing service-accounts.  This change separates the service-account-signing-key into its own asset.
Behaviour:-

New clusters get separately generated apiserver-key.pem and service-account-key.pem keys.
Where deploying to an existing cluster the service-account-key.pem will be copied from the apiserver-key.pem file.
Where users specify their own service-account-key.pem file, that will be used.

Related issue: https://github.com/kubernetes-incubator/kube-aws/issues/1226

Codecov Report

Merging #1227 into master will increase coverage by 0.04%.
The diff coverage is 84.28%.

@@            Coverage Diff             @@
##           master    #1227      +/-   ##
==========================================
+ Coverage   36.42%   36.47%   +0.04%
==========================================
  Files          63       63
  Lines        3846     3866      +20
==========================================
+ Hits         1401     1410       +9
- Misses       2229     2239      +10
- Partials      216      217       +1

Impacted Files
Coverage Δ

core/controlplane/config/credential.go
51.35% <9.09%> (-4.65%)
:arrow_down:

core/controlplane/config/encrypted_assets.go
70.37% <98.3%> (+0.34%)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM. Great work, especially on implementing the migration plan 🎉


Chat goes off screen

The chat goes off the screen when -help is done, chat needs to be improved with a scroll bar or something to fix.

#170 fixes this.


cmd/compile: wrong declaration line in debug_info for variables captured in a closure

What version of Go are you using (go version)?

$ go version
go version devel +f376b8510e Fri Jan 3 01:36:26 2020 +0000 linux/amd64

Does this issue reproduce with the latest release?
Yes.
What did you do?
Compile:
package main

import (
	"fmt"
)

func f1(a int) {
	fmt.Printf("a: %d\n", a)
}

func main() {
	a := 1
	f := func() {
		f1(a)
	}
	f()
}

check the DW_AT_decl_line attribute of the variable a in main.main.func1
What did you expect to see?
line 12
What did you see instead?
line 14 (the first use of a inside the closure).

Note: this is not a regression.


[Feature] Support equals()&hashcode() in FlinkTableSinkBase

Search before asking

[X] I searched in the issues and found nothing similar.

Motivation
When using partial-update merge-engine, we want to reuse sink node across multiple datastreams. To achieve this, we need to override equals() for Paimon DynamicTableSink.
Solution
No response
Anything else?
No response
Are you willing to submit a PR?

[X] I'm willing to submit a PR!

The idea is good, we plan to do this but always no time... Feel free to open PR

@xiangyuf
Reusing sink node for partial-update is a good idea. We have an internally solution for this. Contributing it to the community requires opening a FLIP in the Flink community, which as Jingsong said we just couldn't find time for. If you're interested, we can share that solution with you and see if we can cooperate on that.
As far as I can see, one thing different from your plan is that, our solution does not rely on equals() && hashcode() of FlinkTableSinkBase, but would use some public fields of the sink nodes to identify those that can be reused. The benefit is that, this feature can be enabled for any connector, without relying on the connector codes to implement equals() && hashcode().

@xiangyuf Reusing sink node for partial-update is a good idea. We have an internally solution for this. Contributing it to the community requires opening a FLIP in the Flink community, which as Jingsong said we just couldn't find time for. If you're interested, we can share that solution with you and see if we can cooperate on that.
As far as I can see, one thing different from your plan is that, our solution does not rely on equals() && hashcode() of FlinkTableSinkBase, but would use some public fields of the sink nodes to identify those that can be reused. The benefit is that, this feature can be enabled for any connector, without relying on the connector codes to implement equals() && hashcode().

@xintongsong Thx for ur reply. I'd like to hear your internal solution.


possible discrepancy between the number of species on a given week (species.py vs analyze.py)

Hello. I am not sure whether I am not getting something right here or there is a discrepancy between the number of species considered by species.py and analyze.py.
python3 species.py --o list.txt --lat 41.39 --lon 2.16

says Done. 249 species on list.
python3 species.py --o list.txt --lat 41.39 --lon 2.16 --week 2

says Done. 119 species on list.
python3 analyze.py --i sample.wav --o results.txt --lat 41.39 --lon 2.16 --week 2

says Species list contains 147 species.
python3 analyze.py --i sample.wav --o results.txt --lat 41.39 --lon 2.16

says Species list contains 276 species.
I would really like to understand these numbers.
Many thanks.

Ah, good catch! species.py and analyze.py use different cut-off thresholds (0.05 and 0.03) to determine the species list. Therefore, analyze.py has more entries on the species list compared to species.py. We'll set the threshold to 0.03 for both of them to make things more consistent.


Web Automation: Waits

Description
Create a class that includes methods for all applicable waits for the Web Automation using Implicit and Explicit Waits.
Included methods should be as follows:

Wait for URL to be
Wait for URL to contain
Wait for Title to be
Wait for Title to contain
Wait for Element to be present
Wait for Element to be visible
Wait for Element to be invisible
Wait for Nested Element to be present
Wait for Nested Element to be visible -> limitation
Wait for Nested Element to be invisible -> limitation
Wait for List Element to be present
Wait for List Element to be visible
Wait for List Element to be invisible
Wait for Nested List Element to be present
Wait for Nested List Element to be visible
Wait for Nested List Element to be invisible -> limitation
Wait for Element to be clickable
Wait for Attribute Value to be
Wait for Attribute Value to contain
Wait for Text to be
Wait for Text to contain
Wait for Selection State to be
Wait for Element Count to be
Wait for Alert to be present

Dependencies

N/A

Could not implement the following due to limitations on ExpectedConditions. Other implemented wait commands can be used as workaround.

Wait for Nested Element to be visible
Wait for Nested Element to be invisible
Wait for Nested List Element to be invisible

Setting status to READY TO TEST


Change the data structure of error.type object

Codecov Report

Merging #24 (c5ff42d) into master (fb194ce) will decrease coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #24      +/-   ##
==========================================
- Coverage   95.89%   95.86%   -0.04%
==========================================
  Files           5        5
  Lines         585      580       -5
==========================================
- Hits          561      556       -5
  Misses         24       24

Flag
Coverage Δ

unittests
95.86% <100.00%> (-0.04%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/type.c
100.00% <100.00%> (ø)

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 fb194ce...c5ff42d. Read the comment docs.


Improve the function brackets completion

Discussed in https://github.com/microsoft/pylance-release/discussions/2725

Originally posted by Vincent0706 October 28, 2021
I hope the pylance can distinguish whether the function need add brackets. Sometimes, the brackets have been input and we just need input the funtion name. But when you open the function brackets completion, pylance will add the brackets everytime.
https://user-images.githubusercontent.com/82581024/139382092-2c234562-7e68-483b-a98a-566b53c379f5.mp4

I can no longer reproduce this so I'm closing it.


Unknown directory type causes deserialization failure again

Same issue as https://github.com/workos/workos-kotlin/issues/97 but with "rippling scim v2.0".
I'll reiterate here that we don't even look at this field and we would be very happy if it was just a String and not an enum.

Hey @singron,
We just released v2.11.0 of the Kotlin SDK that addresses this issue.
We're going to take a look and see what we can do to better account for new directory types that aren't present in the enum.

Thank you!


file size fixes

related to issue #15

Hi again @khoran I just made a commit in develop solving these issues, could you test it at your side please?: https://github.com/developmentforpeople/dfp_external_storage/commit/dab1b3482f4b99df67d0f4ba7d4a22465ae5ddb2
If you dont mind I will close this PR. For next time the file dfp_external_storage/dfp_external_storage/workspace/dfp_s3_storage/dfp_s3_storage.json wouldn't be needed in commit because is more a your order preference about where put the app in your workspace :).
Thanks again for your time @khoran :)

That works great, thanks! That json file wasn't meant to be included, I'll make sure to exclude that kind of stuff next time.


Login Issues

Checklist:

[x] I have checked that there are no issues with similar or the same content
YOU SHOULD CHECK CLOSED ISSUES ASWELL

[x] I have checked the FAQ (https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/wiki/FAQ) and the answer I am looking for is not there

[x] I have double checked and can reproduce the issue

**OS: MAC OS Catalina 10.15.1 **
GPMDP Version: 4.6.1
Issue Descriptions: Can't login any more
**Steps to Reproduce:

Delete your data folder ~/Library/Application Support/Google Play Music Desktop Player. I did it because got stucked on Loading your library screen)
Open app and try to login
Login fails with info that my browser is not secure for logging in:

Couldn't sign you in
You are trying to sign in from a browser or app that doesn't allow us to keep your account secure.
Try using a different browser. Learn more

P.S. I tried to disable 2 factor auth for the google account but it didn't help
debug_info.zip

Same problem for me here, with macOS 10.15.1 Catalina
I'm stuck on

tried to delete my ~/Library/Application Support/Google Play Music Desktop Player directory.
Logged me in again
Stuck on the same screen again

same here But Linux Mint 19

Same here on Windows 10, tried allow less secure app login (https://myaccount.google.com/lesssecureapps), but no change.

There is a fix for it here and it works :-)
https://ci.appveyor.com/project/MarshallOfSound/google-play-music-desktop-player-unofficial/build/artifacts
See: https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/issues/3660

The CI build fix the splashscreen hang but I can't use the scrollbar (mouse wheel works).

Confirm, same problem here, Linux Mint 19.2

Same for me on Mint 19.2 and RHEL 7

Also a problem on Windows 10

Windows 7 also.

Same, Xubuntu 16.04.6 LTS, GPMDP 4.6.1 Debian 64-bit *.deb

Couldn't sign you in
You are trying to sign in from a browser or app that doesn't allow us to keep your account secure.
Try using a different browser. Learn more
Same issue. Windows 10.

Confirm, same problem: macOS 10.15.1 Catalina

I am closing this in favor of the pinned #3545. It appears to be now effecting more of you now that we are closer to the end of the year deadline that Google was imposing, but in theory is fixed in the next release.

I am closing this in favor of the pinned #3545. It appears to be now effecting more of you now that we are closer to the end of the year deadline that Google was imposing, but in theory is fixed in the next release.

Hi @jostrander I checked the thread and pr. I also pulled the code change which should fix the issue and rebuilt project locally but it does not fix the issue.
After checking the project's code and electron docs: https://electronjs.org/docs/api/webview-tag I think the issue here is that app uses  tag to authorize user. Which can mean project's architecture needs to be changed to use google libs to auth user instead.
I think the Issue should remain open
Thanks

@betmakh The issue is indeed that the application uses the web view tag. Unfortunately there is no other way for us to integrate with GPM and YTM as far as authentication goes.  We are simply injecting functionality into the existing web apps that Google provides via the web frame.
I'm not sure why the latest changes didn't fix the issue for you, unless Google has changed the way they check for third party "browsers" again. Either way it should be tracked in the existing issue, IMO.

Same issue on Windows 10.

Same issue here (Windows 10 / Chrome 78.0.3904.108

I found a solution to the same issue for me with this post on the related thread linked above.


obi go/rsync from a dir with spaces in it fails.

this came up for "~/Box Synch/someproject"
sym linking "~/Box Sync" to ~/BoxSync does not solve this.
index ff70c8c..84370c4 100644
--- a/obi/task/task.py
+++ b/obi/task/task.py
@@ -33,7 +33,7 @@ def room_task(room_name, task_name=None):
     if not project_name:
         abort("No name key found in the project.yaml. Please specify a project name")
     env.project_name = project_name
-    env.local_project_dir = os.path.dirname(project_config)
+    env.local_project_dir = '"%s"' % os.path.dirname(project_config)
     # Abort if we cannot find room_name in rooms
     rooms = config.get("rooms", {})
     room = rooms.get(room_name, None)

wrapping local dir in "" fixes it.

That will still fail to escape other path characters (like \ and ") correctly. Doesn't fabric support escaping command line arguments?

I did not try other characters mostly because I don't have a use case for them. The "Box Sync" folder is the only one I care about. I'll poke around in fabric some more.
Sent from Lotus Traveler
Tom Jakubowski --- Re: [Oblong/obi] obi go/rsync from a dir with spaces in it fails. (#51) ---
From:"Tom Jakubowski" notifications@github.comTo:"Oblong/obi" obi@noreply.github.comCc:"David Nixon"<EMAIL_ADDRESS>"Author" author@noreply.github.comDate:Tue, Jun 28, 2016 6:16 PMSubject:Re: [Oblong/obi] obi go/rsync from a dir with spaces in it fails. (#51)
That will still fail to escape other path characters (like \ and ") correctly. Doesn't fabric support escaping command line arguments?
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.

@tomjakubowski @jshrake
fabric does not have a function to escape characters but python does.
diff --git a/obi/task/task.py b/obi/task/task.py
index ff70c8c..7c9d86d 100644
--- a/obi/task/task.py
+++ b/obi/task/task.py
@@ -10,6 +10,7 @@ import fabric
 import os
 import stat
 import yaml
+import pipes

 from fabric.api import env  # the global env variable
 from fabric.api import (task, parallel, runs_once) #decorators
@@ -33,7 +34,7 @@ def room_task(room_name, task_name=None):
     if not project_name:
         abort("No name key found in the project.yaml. Please specify a project name")
     env.project_name = project_name
-    env.local_project_dir = os.path.dirname(project_config)
+    env.local_project_dir = pipes.quote(os.path.dirname(project_config))
     # Abort if we cannot find room_name in rooms
     rooms = config.get("rooms", {})
     room = rooms.get(room_name, None)

0001-handle-special-chars-in-local-dir.patch.gz

Tested with this sub dir
ls -la
drwxr-xr-x 3 david_nixon david_nixon 4096 Jun 27 21:28 sub dir!@#$^&*()-_+={}[]|\:;"<>,.?

Mmm… pipes.quote is deprecated so I'm wary of using it. I was thinking more along the lines of these options to fabric's run command: http://docs.fabfile.org/en/1.11/api/core/operations.html#fabric.operations.run
But this would require surgery deeper in obi's guts than this here.

I think you're thinking too hard on this. 😉 Surgery deeper on obi would not be safer than using the deprecated pipe.quote. The run command is not really used by the rsync command anyway. The obi rsync wraps the fabric rsync and I already tried adding the shell options there. They are not accepted.
When obi supports Python 3 then I think you could switch to show.quote but until then the pipes.quote or the original solution would handle the common use case of spaces in the directory name.
Sent from Lotus Traveler
Tom Jakubowski --- Re: [Oblong/obi] obi go/rsync from a dir with spaces in it fails. (#51) ---
From:"Tom Jakubowski" notifications@github.comTo:"Oblong/obi" obi@noreply.github.comCc:"David Nixon"<EMAIL_ADDRESS>"Author" author@noreply.github.comDate:Fri, Jul 1, 2016 6:02 PMSubject:Re: [Oblong/obi] obi go/rsync from a dir with spaces in it fails. (#51)
Mmm… pipes.quote is deprecated so I'm wary of using it. I was thinking more along the lines of these options to fabric's run command: http://docs.fabfile.org/en/1.11/api/core/operations.html#fabric.operations.run
But this would require surgery deeper in obi's guts than this here.
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.

If we have a little time over the next month we'll address this.

Closed by #77


More dependabot updates

Fixes https://github.com/dotnet/razor-tooling/issues/6698
Fixes https://github.com/dotnet/razor-tooling/security/dependabot/41
Fixes https://github.com/dotnet/razor-tooling/security/dependabot/8

You know as much as I do then! And since you're the only person (at the moment) this could break, your approval means the most :)


How can one test a template that uses lookup?

Keep in mind that Helm is not supposed to contact the Kubernetes API Server during a helm template or a helm install|update|delete|rollback --dry-run, so the lookup function will return nil in such a case.

Is there a way I can test the lookup function prior to deploying a chart? If not, is it really necessary to forbid helm from contacting the API server during a dry run? My expectation of a --dry-run in other software is that no state is modified but that I can see exactly what would happen during a full run. This is not the case if your chart uses lookup to fill in template values.
I would expect that during a dry run helm allows GET and HEAD requests to API server resources but not mutating methods like PUT, PATCH, and POST. If there's a need to have a network-free flow as exists today, perhaps helm template could elide the network calls while helm install|upgrade|delete|rollback --dry-run would perform the required lookups.

Duplicate of #8137. Let’s keep the conversation in one ticket. Thanks.


Check if mongodump and mongorestore exist

This checks if we have the binaries existing, so we can fail early on, instead of silently synchronizing collections without a restore first.
Closes #9

Everything you've implemented is great! There is only one thing missing - support of this validator from CLI. To implement this, you have to add this validator to supportedValidators map in ValidatorTypeMapper. Could you add it?

done


Didn't find 'mag_subset20K.txt' file in the unzipped folder  mag_papers_0

I downloaded  the mag_papers_0  file from the following link,  https://www.openacademic.ai/oag/ , but didn't find the mag_subset20K.txt used in https://github.com/alicezheng/feature-engineering-book/blob/master/09.01-05_[End-to-End_Example]_Recommender_Take_1.ipynb.
Is the mag_subset20K.txt file reconstructed from one of the source file, if so, how to build? if not, where is the right link to download this file?
Thanks

We didn't include the data files because we don't own the rights to distribute any of them.
I just created a new file from the first 20K lines of the mag_papers_0.txt file using the 'head' command. Here is a similar answer/discussion on how to do this from StackOverflow.

I downloaded the mag_papers_0 file from the following link, https://www.openacademic.ai/oag/ , but didn't find the mag_subset20K.txt used in https://github.com/alicezheng/feature-engineering-book/blob/master/09.01-05_[End-to-End_Example]_Recommender_Take_1.ipynb.
Is the mag_subset20K.txt file reconstructed from one of the source file, if so, how to build? if not, where is the right link to download this file?
Thanks

Do you have the right data? If that, would you mind to share with me?
Thanks a lot.


Username.test.js flaky snapshots

username.test.js has a flaky state in which the snapshots sometimes fail because something isn't waiting for the promises to resolve before creating the snapshot.
A solution suggested by @bryanjenningz is to run await findByText(SOME_TEXT_IN_THE_SNAPSHOT) to make it wait for some text to show in the DOM before creating/updating the snapshot.

Nice catch.


Implemented FFMPEG based encoder.

Does work rudimentary, will need some FFMPEG parameter adjustments.

Stacktrace when a Win10 client connects. Ubuntu 16.04 with libavcodec-ffmpeg56 7:2.8.6-1ubuntu2 amd64:
#0  0x00007ffff2c6eea8 in avcodec_encode_video2 () from /usr/lib/x86_64-linux-gnu/libavcodec-ffmpeg.so.56
#1  0x00007ffff69b87db in libavcodec_compress (h264=<optimized out>, ppDstData=0x7fffe9e65aa0, pDstSize=0x7fffe9e65a98)
    at /root/builds/freerdp-nightly/FreeRDP/libfreerdp/codec/h264_ffmpeg.c:259
#2  0x00007ffff69b4cb9 in avc420_compress (h264=0x7fffd885c5a0, pSrcData=pSrcData@entry=0x7fffc9fff010 "", SrcFormat=537135240, nSrcStep=nSrcStep@entry=5760,
    nSrcWidth=nSrcWidth@entry=1440, nSrcHeight=nSrcHeight@entry=900, ppDstData=0x7fffe9e65aa0, pDstSize=0x7fffe9e65a98) at /root/builds/freerdp-nightly/FreeRDP/libfreerdp/codec/h264.c:210
#3  0x00007ffff63da8f7 in shadow_client_send_surface_gfx (nXSrc=0, nYSrc=0, nHeight=900, nWidth=1440, nSrcStep=5760, pSrcData=0x7fffc9fff010 "", client=0x7fffc000e2f0)
    at /root/builds/freerdp-nightly/FreeRDP/server/shadow/shadow_client.c:816
#4  shadow_client_send_surface_update (pStatus=<synthetic pointer>, client=0x7fffc000e2f0) at /root/builds/freerdp-nightly/FreeRDP/server/shadow/shadow_client.c:1323
#5  shadow_client_thread (client=<optimized out>) at /root/builds/freerdp-nightly/FreeRDP/server/shadow/shadow_client.c:1630
#6  0x00007ffff6652322 in thread_launcher (arg=0x7fffc0045ec0) at /root/builds/freerdp-nightly/FreeRDP/winpr/libwinpr/thread/thread.c:320
#7  0x00007ffff79b96fa in start_thread (arg=0x7fffe9e66700) at pthread_create.c:333
#8  0x00007ffff56f5b5d in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:109

@SriRamanujam Works with debian stretch (libavcodec57).  Could you have a look at the arguments passed to avcodec_encode_video2 to trace the source of the crash? The input parameters are exactly the same as in my path, just the other API used.

Ah, it seems like you mistakenly pass codecDecoderContext instead of codecEncoderContext to libavcodec_encode_video2. codecDecoderContext is NULL, of course, which triggered the crash.
Fixing that, however, led to a different crash:
[12:29:21:742] [10879:12013] [INFO][com.freerdp.client.shadow] - Gfx Pipeline Opened
[libx264 @ 0x7fffd4fc11a0] using cpu capabilities: MMX2 SSE2Fast SSSE3 SSE4.2 AVX FMA3 AVX2 LZCNT BMI2
[libx264 @ 0x7fffd4fc11a0] profile High, level 3.1
[libx264 @ 0x7fffd4fc11a0] Provided packet is too small, needs to be 383
[12:29:21:777] [10879:12013] [ERROR][com.freerdp.codec] - FFMPEG: Failed to encode video frame (Operation not permitted [-1])

I still don't know why libx264 insists on being involved in this process. I added the "FFMPEG:" tag to that specific error message (libfreerdp/codec/h264_ffmpeg.c:269) just to be sure that it was in fact libavcodec that was throwing that error. The actual segfault is the exact same one that I reported in #3978 , indicating that there was no data to send.

@SriRamanujam Ah, that explains it, oops ^^ As for x264, as far as I know FFMPEG uses it, so don't worry. The errors indicate some H264 encoder settings are still not optimal.
As for the crash, would be interesting to get the missing return check that leads to this crash.

@SriRamanujam Ok, got the crash ;)

It got further this time, and no crashing, but the frame still failed to encode:
[libx264 @ 0x7fa075a1cc40] using cpu capabilities: MMX2 SSE2Fast SSSE3 SSE4.2 AVX FMA3 AVX2 LZCNT BMI2
[libx264 @ 0x7fa075a1cc40] profile High, level 4.0
[libx264 @ 0x7fa075a1cc40] Provided packet is too small, needs to be 447
[14:55:32:732] [6936:10009] [ERROR][com.freerdp.codec] - FFMPEG: Failed to encode video frame (Operation not permitted [-1])
[14:55:32:732] [6936:10009] [ERROR][com.freerdp.client.shadow] - Failed to send surface update
[libx264 @ 0x7fa075a1cc40] frame I:3     Avg QP:15.33  size:   653
[libx264 @ 0x7fa075a1cc40] mb I  I16..4: 100.0%  0.0%  0.0%
[libx264 @ 0x7fa075a1cc40] final ratefactor: 25.74
[libx264 @ 0x7fa075a1cc40] 8x8 transform intra:0.0%
[libx264 @ 0x7fa075a1cc40] coded y,uvDC,uvAC intra: 0.0% 0.0% 0.0%
[libx264 @ 0x7fa075a1cc40] i16 v,h,dc,p: 93%  0%  7%  0%
[libx264 @ 0x7fa075a1cc40] i8c dc,h,v,p: 100%  0%  0%  0%
[libx264 @ 0x7fa075a1cc40] kb/s:156.64

My guess is that the problem from before is still there, but since the program isn't crashing anymore we can see some debug output from the frame encode afterwards now.

Any updates on this PR?

@SriRamanujam Sorry, currently busy with other stuff :/

I had some free time this week, so I decided to see if I couldn't figure out some libav tuning stuff myself.
I managed to get the code into a state where both FreeRDP and Windows' RDP software decoder are able to render the stream. Windows' RDP hardware decoder for some reason renders the frames with a few seconds' delay, honestly not sure what to do about that but at least you can force software decoding in Group Policy.
I want to stress that I am by no means a libavcodec or H.264 tuning genius or anything of the sort, I only barely know what the options do and I mostly just adapted recommendations from the ffmpeg wiki article on live streaming. There is almost certainly lots more that can be done. The work is here: https://github.com/SriRamanujam/FreeRDP/commit/b656b3e2db0ab2bdf9a00de1539760b139fbd83e

@hardening Any chance merging this one? Still needs fine tuning, but at least it is there and people can start debugging the remaining stuff ;)

Yeah sure, I wanted to do it but forget it


'Undefined is not a function' error when moving StyleSheet to a different module

I have just installed react-native and created the default app by running create-react-native-app MyApp. It created App.js:
import React from 'react';
import { StyleSheet, Text, View } from 'react-native';

export default class App extends React.Component {
  render() {
    return (
      <View style={styles.container}>
        <Text>Open up App.js to start working on your app!</Text>
        <Text>Changes you make will automatically reload.</Text>
        <Text>Shake your phone to open the developer menu.</Text>
      </View>
    );
  }
}

const styles = StyleSheet.create({
  container: {
    flex: 1,
    backgroundColor: '#fff',
    alignItems: 'center',
    justifyContent: 'center',
  },
});

I moved styles to a separate module Styles.js and imported into App.js:

import StyleSheet from 'react-native';

const styles = StyleSheet.create({
    container: {
      flex: 1,
      backgroundColor: '#fff',
      alignItems: 'center',
      justifyContent: 'center',
    },
  });

module.exports = styles;

Now when running the app I get Undefined is not a function (evaluating '_reactNative2.default.create({...})')
Environment

react-native -v: 0.47.2
node -v: 8.4.0
npm -v: 5.3.0
yarn --version: 0.27.5

Then, specify:

Target Platform:

Development Operating System: Windows 7

Build tools: Just running with Expo

Actually, I just realized it was my fault - i incorrectly included StyleSheets from react module.


Checkstyle_cpp action take so much of time.

I use docker to compile the heron.

Still waiting for 11 jobs to complete:
Running (standalone):
Executing extra_action //tools/cpp:checkstyle_cpp on //third_party/zoo
keeper:zookeeper-cxx, 59306 s^M^M
Executing extra_action //tools/cpp:checkstyle_cpp on //third_party/lib
event:libevent-cxx, 59306 s^M^M
Executing extra_action //tools/cpp:checkstyle_cpp on //third_party/glo
g:glog-cxx, 59306 s
Executing extra_action //tools/cpp:checkstyle_cpp on //heron/proto:pro
to_physical_plan_cc, 59145 s
Executing extra_action //tools/cpp:checkstyle_cpp on //heron/proto:pro
to_stmgr_cc, 59144 s
Executing extra_action //tools/cpp:checkstyle_cpp on //heron/proto:pro
to_topology_cc, 59135 s
Executing extra_action //tools/cpp:checkstyle_cpp on //heron/proto:pro
to_stats_cc, 59081 s
Executing extra_action //tools/cpp:checkstyle_cpp on //third_party/jan
sson:jansson-cxx, 58959 s
Executing extra_action //tools/cpp:checkstyle_cpp on //third_party/gpe
XXXXXXXXXXXXXX

The jobs checkstyle_cpp takes near 16 hours.
It seems not so necessary to have this job.
Can I skip job like this? And how can I do this?

For us it takes around 20-30 mins to compile the entire codebase. Are you having difficulty downloading some jars?

On Sep 21, 2016, at 10:51 PM, yunfan123<EMAIL_ADDRESS>wrote:
I use docker to compile the heron.
Still waiting for 11 jobs to complete:
Running (standalone):
Executing extra_action //tools/cpp:checkstyle_cpp on //third_party/zoo
keeper:zookeeper-cxx, 59306 s^M^M
Executing extra_action //tools/cpp:checkstyle_cpp on //third_party/lib
event:libevent-cxx, 59306 s^M^M
Executing extra_action //tools/cpp:checkstyle_cpp on //third_party/glo
g:glog-cxx, 59306 s^M^M
Executing extra_action //tools/cpp:checkstyle_cpp on //heron/proto:pro
to_physical_plan_cc, 59145 s^M^M
Executing extra_action //tools/cpp:checkstyle_cpp on //heron/proto:pro
to_stmgr_cc, 59144 s^M^M
Executing extra_action //tools/cpp:checkstyle_cpp on //heron/proto:pro
to_topology_cc, 59135 s^M^M
Executing extra_action //tools/cpp:checkstyle_cpp on //heron/proto:pro
to_stats_cc, 59081 s^M^M
Executing extra_action //tools/cpp:checkstyle_cpp on //third_party/jan
sson:jansson-cxx, 58959 s^M^M
Executing extra_action //tools/cpp:checkstyle_cpp on //third_party/gpe
The jobs checkstyle_cpp takes near 16 hours.
It seems not so necessary to have this job.
Can I skip job like this? And how can I do this.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@kramasamy , all of left tasks is checkstyle_cpp. What task is this task do?
I'm in China, but I'm sure my compile env can access maven.twttr.com and http://central.maven.org/maven2/ normally.
Is there any jars from other maven central?

Checkstyle cpp ensures that the coding style is enforced. As long as you can access maven central we are good to go. Not sure what is going wrong. One possibility is to try compiling one directory at a time under .../heron and isolate what the problem is.

On Sep 21, 2016, at 11:11 PM, yunfan123<EMAIL_ADDRESS>wrote:
@kramasamy , all of left tasks is checkstyle_cpp. What task is this task do?
I'm in China, but I'm sure my compile env can access maven.twttr.com and http://central.maven.org/maven2/ normally.
Is there any jars from other maven central?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

If checkstyle cpp just to ensures the coding style.
Can I totally disable this action?

@yunfan123 - Hi, it takes me around 30-40 mins to compile the entire codebase using docker. My working environment is MacBook Pro Mid 2012 and 2.3 GHz Intel Core i7.

The checkstyle output is a red herring. What I suspect is happening is the build is taking a while to compile for other reasons, you just see the checkstyle output along the way. To verify you can comment out the bazel listener and extra_action that do the cpp checkstyles:
https://github.com/twitter/heron/blob/master/tools/cpp/BUILD#L5

Is any release compiled heron docker images? So I can try heron without compile it...

@yunfan123  - Released Page is addressed on Getting Started Guide.

@yunfan123 - there are prebuilt docker images. Just wondering what platform are you compiling it for?

@kramasamy   Ubuntu.

Ok - which version of Ubuntu are you trying to build on?

I built on Ubuntu 14.04 and it is pretty good.

@kramasamy @billonahill should we perform cpp code checking on third-party code?

No. We should only do checkstyles on code we write.

👍  I will see if we can disable cpp code style checking on third-party code.


[2579] Sort potential secondary subjects by name

Context
Secondary subjects are currently not in alphabetical order.
Changes proposed in this pull request

Sort secondary subjects before serialising them into the edit options serialiser

Checklist

[x] Make sure all information from the Trello card is in here
[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

 Secondary subject page: Sort subjects by name


add current_epoch to dumped_params

What does this PR do?

Fixes #3260 by restoring the current_epoch after finishing the batch size finder.

global_step, total_batch_idx, or maybe more needs to be reset too. I believe a better solution would be to make a pre_fit method or something in Trainer and reassign them to their default values every time .fit is called by calling it in .fit itself.

@rohitgr7 There is a Tuner class coming, #3160 in which I believe the resetting problem will be tackled.

@awaelchli will it support user-defined callbacks?

I'm not sure, but we recently added support to persist state of callbacks, so we should be able to dump their state too and restore them easily if that's the problem? For further questions better ask directly here #3160

The Tuner class looks really promising!

Just to update, the HyperTuner class is not going to happen at the moment, instead the tuner algorithms will be moved to a separate trainer.tune() method instead of being called by fit (it has already started https://github.com/PyTorchLightning/pytorch-lightning/pull/3293). So we need to fix this problem when the refactor is done.

@maxjeblick mind add a test to check that it is fixed...

@awaelchli you are probably right. I think the tuner algorithms needs an refactoring after v1.0.

Shouldn't tuner be refactored in such a way that it won't call .fit again? Then we won't need to dump and restore anything. What it should do is just suggest and user should change the values and rerun?

@rohitgr7 we use fit internally in these algorithms so we don´t have to redefine the training loop necessary to run these algorithms. However, IMO moving forward we should refactor it in such a way that tuner should just create a new trainer that can be used during tuning and then destroyed afterwards.

@SkafteNicki I am not suggesting to rewrite anything. The reason we need to dump and restore is because these attributes are not re-initialized and current workflow is something like:
trainer.tuner.lr_find()
# suggest
trainer.fit()

What I am suggesting is:
trainer.tuner.lr_find()
# suggest
# and tell user to reinitalize trainer, LM, and LDM(if any) with the updated hparams from the suggestions.

IMO moving forward we should refactor it in such a way that tuner should just create a new trainer that can be used during tuning and then destroyed afterwards.

this is good alternative but this will work for only Trainer and not for LM and LDM since there are chances that some attributes might be changed during .fit().


MongoError: connect ECONNREFUSED

Hi Bastien,
I get the following error when I follow your tutorial:
$ kubectl -n rocketchat logs rocketchat-app-658557f9c8-ppzjp

/app/bundle/programs/server/node_modules/fibers/future.js:313
                                                throw(ex);
                                                ^
MongoError: failed to connect to server [mongo-1.mongo:27017] on first connect [MongoError: connect ECONNREFUSED <IP_ADDRESS>:27017]
    at Pool.<anonymous> (/app/bundle/programs/server/npm/node_modules/meteor/npm-mongo/node_modules/mongodb-core/lib/topologies/server.js:336:35)
    at emitOne (events.js:116:13)
    at Pool.emit (events.js:211:7)
    at Connection.<anonymous> (/app/bundle/programs/server/npm/node_modules/meteor/npm-mongo/node_modules/mongodb-core/lib/connection/pool.js:280:12)
    at Object.onceWrapper (events.js:317:30)
    at emitTwo (events.js:126:13)
    at Connection.emit (events.js:214:7)
    at Socket.<anonymous> (/app/bundle/programs/server/npm/node_modules/meteor/npm-mongo/node_modules/mongodb-core/lib/connection/connection.js:187:49)
    at Object.onceWrapper (events.js:315:30)
    at emitOne (events.js:116:13)

The mongo pods are up:
NAME                              READY     STATUS             RESTARTS   AGE
mongo-0                           2/2       Running            0          3m
mongo-1                           2/2       Running            0          3m
mongo-2                           2/2       Running            0          3m
rocketchat-app-658557f9c8-ppzjp   0/1       CrashLoopBackOff   4          3m

Am I doing something wrong?

Hi @nedbase-jesse,
It indeed seems that the mongo cluster does not accept connection :
$ wget mongo-0.mongo:27027
Connecting to mongo-0.mongo:27027 (<IP_ADDRESS>:27027)
wget: can't connect to remote host (<IP_ADDRESS>): Connection refused

By investigating further, the side-car which is supposed to initiate Mongo's replicaSet has error too :
Error in workloop { MongoError: failed to connect to server [<IP_ADDRESS>:27017] on first connect [MongoError: connect ECONNREFUSED <IP_ADDRESS>:27017]
    at Pool.<anonymous> (/opt/cvallance/mongo-k8s-sidecar/node_modules/mongodb-core/lib/topologies/server.js:336:35)
    at Pool.emit (events.js:159:13)
    at Connection.<anonymous> (/opt/cvallance/mongo-k8s-sidecar/node_modules/mongodb-core/lib/connection/pool.js:280:12)
    at Object.onceWrapper (events.js:254:19)
    at Connection.emit (events.js:159:13)
    at Socket.<anonymous> (/opt/cvallance/mongo-k8s-sidecar/node_modules/mongodb-core/lib/connection/connection.js:187:49)
    at Object.onceWrapper (events.js:254:19)
    at Socket.emit (events.js:159:13)
    at emitErrorNT (internal/streams/destroy.js:64:8)
    at _combinedTickCallback (internal/process/next_tick.js:138:11)
  name: 'MongoError',
  message: 'failed to connect to server [<IP_ADDRESS>:27017] on first connect [MongoError: connect ECONNREFUSED <IP_ADDRESS>:27017]' }

I will try debugging this issue asap, thanks for reporting it 👍


fix: small typo errors

[x] This is a small change
[ ] This change has been discussed in issue #<?> and the solution has been agreed upon with maintainers.

Description:
I fixed some small typo errors

ThNks!


sp-core full_crypto is no_std incompatible

Is there an existing issue?

[X] I have searched the existing issues

Experiencing problems? Have you tried our Stack Exchange first?

[X] This is not a support question.

Description of bug
When building for no_std targets, sp-core dependency fails due to issue with libsecp256k1
$cargo build --target=thumbv7m-none-eabi

<truncated>

error[E0463]: can't find crate for `std`
  --> /home/user/.cargo/registry/src/github.com-1ecc6299db9ec823/subtle-2.4.1/src/lib.rs:88:1
   |
88 | extern crate std;
   | ^^^^^^^^^^^^^^^^^ can't find crate
   |
   = note: the `thumbv7m-none-eabi` target may not support the standard library
   = help: consider building the standard library from source with `cargo build -Zbuild-std`

See more details and proposed fix https://github.com/paritytech/libsecp256k1/pull/146
Steps to reproduce

Install appropriate targets
$cargo generate --git https://github.com/rust-embedded/cortex-m-quickstart
give test project a name, cd
add to Cargo.toml sp-core = {version = "7.0.0", default-features = false, features = ["full_crypto"]} dependency
$cargo build --target=thumbv7m-none-eabi

There was recently an issue around no_std incompatibility. Which was solved? https://github.com/paritytech/substrate/issues/12225
Maybe the problem is the target architecture.

Looks similar, but this time it's ECDSA, different crypto and different crate. Issue you refer to is about ed25519-zebra crate, while this is libsecp256k1 crate, and it still persists even in 7.0.0
With a tweak proposed above it builds nicely, should be a simple fix since the lib is also owned by Parity.

Ahh the problem is that libsecp256k1 is pulled in build-dependencies.
The solution to your problem is to add:
[workspace]
resolver = "2"

in your workspace Cargo.toml.

It does work indeed, did not notice that generate script still uses 2018 rust version. Setting rust edition to 2021 seems even better here.


remote error: tls: access denied with go get

I cannot use go get to download k8s.io/client-go/..., but I can download others such as github.com/nsf/gocode.
So I think my configuration about git and go are correct.
I don't know why this happens?
Any ideas how to fix it?
$ git version
git version 2.19.1.windows.1

$ go version
go version go1.11.2 windows/amd64

$ go env
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\zhongtao\AppData\Local\go-build
set GOEXE=.exe
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=C:\Users\zhongtao\go
set GOPROXY=
set GORACE=
set GOROOT=C:\Software\Go
set GOTMPDIR=
set GOTOOLDIR=C:\Software\Go\pkg\tool\windows_amd64
set GCCGO=gccgo
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=C:\Users\zhongtao\AppData\LoTemp\go-build041230430=/tmp/go-build -gno-record-gcc-switches

$ go get -u -v github.com/nsf/gocode
github.com/nsf/gocode (download)
github.com/nsf/gocode

$ go get -u -v k8s.io/client-go/...
Fetching https://k8s.io/client-go?go-get=1
https fetch failed: Get https://k8s.io/client-go?go-get=1: remote error: tls: access denied
package k8s.io/client-go/...: unrecognized import path "k8s.io/client-go/..." (https fetch: Get https://k8s.io/client-go?go-get=1: remote r: tls: access denied)

Works fine here.


Delete test

Codecov Report

Merging #10 (b1e92f3) into master (bf661f0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #10   +/-   ##
=======================================
  Coverage   97.55%   97.55%
=======================================
  Files          49       49
  Lines        2941     2941
  Branches      469      469
=======================================
  Hits         2869     2869
  Misses         49       49
  Partials       23       23

Flag
Coverage Δ

python-3.10
97.31% <ø> (-0.04%)
:arrow_down:

python-3.6
97.16% <ø> (+0.10%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


Add VS info and year numbers for other C# versions

ideally in the sub headers.

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

ID: 3d252158-0367-77c8-1816-dbc385c5c40b
Version Independent ID: 575872e5-6273-f8f6-e3e2-2b911cd9d009
Content: The history of C# - C# Guide
Content Source: docs/csharp/whats-new/csharp-version-history.md
Product: dotnet-csharp
GitHub Login: @erikdietrich
Microsoft Alias: wiwagn

This is a good idea @AdmiralSnyder
I've added it to our backlog, and added the "up for grabs" label for any interested community member.


Template sensors make a lot of errors in log

Make sure you run the latest version before reporting an issue. Feature requests should go in the forum: https://community.home-assistant.io/c/feature-requests
Home Assistant release (hass --version):
0.21.0dev0
Python release (python3 --version):
Component/platform:
sensor template
Description of problem:
Before I had Undefined errors for template sensors, but now I get a lot of tracebacks
Expected:
Less errors than before.
Problem-relevant configuration.yaml entries and steps to reproduce:
  - platform: template
    sensors:
      batteri_sensor_bad:
        value_template: '{{ states.sensor.gulvtemp_bad_1etg.attributes["Battery numeric"] | multiply(10) }}'
        friendly_name: 'Batteri Føler Luft/Gulv Bad 1.etg'
        unit_of_measurement: '%'

Traceback (if applicable):
16-05-30 15:19:35 homeassistant.core: BusHandler:Exception doing job
Traceback (most recent call last):
  File "/srv/hass/src/home-assistant/homeassistant/core.py", line 809, in job_handler
    func(arg)
  File "/srv/hass/src/home-assistant/homeassistant/helpers/event.py", line 51, in state_change_listener
    event.data['new_state'])
TypeError: template_sensor_state_listener() missing 1 required positional argument: 'new_state'
16-05-30 15:19:35 homeassistant.core: BusHandler:Exception doing job
Traceback (most recent call last):
  File "/srv/hass/src/home-assistant/homeassistant/core.py", line 809, in job_handler
    func(arg)
  File "/srv/hass/src/home-assistant/homeassistant/helpers/event.py", line 51, in state_change_listener
    event.data['new_state'])
TypeError: template_sensor_state_listener() missing 1 required positional argument: 'new_state'
16-05-30 15:19:35 homeassistant.core: BusHandler:Exception doing job
Traceback (most recent call last):
  File "/srv/hass/src/home-assistant/homeassistant/core.py", line 809, in job_handler
    func(arg)
  File "/srv/hass/src/home-assistant/homeassistant/helpers/event.py", line 51, in state_change_listener
    event.data['new_state'])
TypeError: template_sensor_state_listener() missing 1 required positional argument: 'new_state'
16-05-30 15:19:35 homeassistant.core: BusHandler:Exception doing job
Traceback (most recent call last):
  File "/srv/hass/src/home-assistant/homeassistant/core.py", line 809, in job_handler
    func(arg)
  File "/srv/hass/src/home-assistant/homeassistant/helpers/event.py", line 51, in state_change_listener
    event.data['new_state'])
TypeError: template_sensor_state_listener() missing 1 required positional argument: 'new_state'
16-05-30 15:19:35 homeassistant.core: BusHandler:Exception doing job
Traceback (most recent call last):
  File "/srv/hass/src/home-assistant/homeassistant/core.py", line 809, in job_handler
    func(arg)
  File "/srv/hass/src/home-assistant/homeassistant/helpers/event.py", line 51, in state_change_listener
    event.data['new_state'])
TypeError: template_sensor_state_listener() missing 1 required positional argument: 'new_state'

Additional info:
I suspect this may be some unfinished buisness regarding #2144

Huh, after rebuilding tox errors are gone??? Closing this until it is more consistent.

It was fixed in dev yesterday
On Tue, May 31, 2016, 07:45 John Arild Berentsen<EMAIL_ADDRESS>wrote:

Huh, after rebuilding tox errors are gone??? Closing this until it is more
consistent.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/issues/2180#issuecomment-222711054,
or mute the thread
https://github.com/notifications/unsubscribe/ABYJ2ovRBQPUThOhwnxxVEKSNVj_ALr3ks5qHEl4gaJpZM4Ip2eZ
.


Pointer and two dimensional array in C

Description 📜
Created documentation for issue number #924 . Fixes #924

Type of change 📝

[x] Documentation (Content Creation in the form of codes or tutorials)

Domain of Contribution 📊

[x] C/CPP

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

Thank you !! @apeksha235

Okay


youtube videos width is not matching to the app

how align youtube video in proper size?

Jetpack does this for you. Inspect this post: https://m.julienrenaux.fr/#/public/posts/2585

ok i did install jetpack plugin still facing same issue.

how to setup jetppack?

https://jetpack.com/support/shortcode-embeds/


Accounts fix

Fix to set up queries with doAPI.

Fix to be able to specify multiple id for GetAccountRelationships.

https://github.com/tootsuite/documentation/blob/master/Using-the-API/API.md#getting-an-accounts-relationships

id (can be array): Account IDs

Added test of accounts.

Changes Unknown when pulling 5d106e8942d233fd0cc0fe90e28e57c755b66190 on 178inaba:get_query into ** on mattn:master**.

:+1:


403 when trying to get master token

I'm trying to use the HA integration and came across the auth problem (https://github.com/leikoilja/ha-google-home/issues/528) so it was suggested I just try using the raw package.
When I run it with my credentials I get a 403 from the API calls:
vscode ➜ /workspaces/tmp-ha-google $ python3 get-token.py
DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): android.clients.google.com:443
DEBUG:urllib3.connectionpool:https://android.clients.google.com:443 "POST /auth HTTP/1.1" 403 None
ERROR:glocaltokens.client:[!] Could not get master token.
[*] Master token None
DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): android.clients.google.com:443
DEBUG:urllib3.connectionpool:https://android.clients.google.com:443 "POST /auth HTTP/1.1" 403 None
ERROR:glocaltokens.client:[!] Could not get master token.

[*] Access token (lives 1 hour) None

[*] Google devices local authentication tokens
DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): android.clients.google.com:443
DEBUG:urllib3.connectionpool:https://android.clients.google.com:443 "POST /auth HTTP/1.1" 403 None
ERROR:glocaltokens.client:[!] Could not get master token.
DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): android.clients.google.com:443
DEBUG:urllib3.connectionpool:https://android.clients.google.com:443 "POST /auth HTTP/1.1" 403 None
ERROR:glocaltokens.client:[!] Could not get master token.
DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): android.clients.google.com:443
DEBUG:urllib3.connectionpool:https://android.clients.google.com:443 "POST /auth HTTP/1.1" 403 None
ERROR:glocaltokens.client:[!] Could not get master token.

The Python script I'm running is off the README for the repo, with the only change being I've put in my username/password.
I've tried with 2FA enabled and an App Password, and I've tried turning off 2FA and using a username/password, both output the same thing.

It seems you're using python 3.10. Can you try with 3.9? Also what versions of requests and urllib3 you have installed (you can find this using pip freeze)?

You should consider upgrading via the '/usr/local/bin/python -m pip install --upgrade pip' command.
vscode ➜ /workspaces/tmp-ha-google $ python3 get-token.py
ERROR:glocaltokens.client:[!] Could not get master token.
[*] Master token None
ERROR:glocaltokens.client:[!] Could not get master token.

[*] Access token (lives 1 hour) None

[*] Google devices local authentication tokens
ERROR:glocaltokens.client:[!] Could not get master token.
ERROR:glocaltokens.client:[!] Could not get master token.
ERROR:glocaltokens.client:[!] Could not get master token.

Here's the packages:
vscode ➜ /workspaces/tmp-ha-google $ pip freeze
certifi==2022.6.15
charset-normalizer==2.0.12
glocaltokens==0.6.5
gpsoauth==1.0.2
grpcio==1.47.0
idna==3.3
ifaddr==0.2.0
protobuf==3.19.4
pycryptodomex==3.15.0
requests==2.28.0
simplejson==3.17.6
six==1.16.0
urllib3==1.26.9
zeroconf==0.38.7

@aaronpowell, seems like you are getting hit by 403 (permission denied) error, a bit of a silly question, but better safe then sorry: are you sure you credentials are valid? And when running get_tokens example, you are using just a username user in the example of<EMAIL_ADDRESS>not the full email address, right?

@aaronpowell to completely resemble out test setup with dependencies can you please install using poetry as described in contribution docs. Once you have poetry just run poetry install from project directory to create managed virtual env and install dependency packages exactly similar to ours. Note, you don't need to manually install glocaltokens package in your virtual env, when running the example code

@aaronpowell, seems like you are getting hit by 403 (permission denied) error, a bit of a silly question, but better safe then sorry: are you sure you credentials are valid? And when running get_tokens example, you are using just a username user in the example of<EMAIL_ADDRESS>not the full email address, right?

I'm not using a gmail account for this Google account, I'm using a custom domain that I've created a Google account with. I know the creds are valid as I can login via the browser with them.

@aaronpowell to completely resemble out test setup with dependencies can you please install using poetry as described in contribution docs. Once you have poetry just run poetry install from project directory to create managed virtual env and install dependency packages exactly similar to ours. Note, you don't need to manually install glocaltokens package in your virtual env, when running the example code

Do I need to clone the glocaltokens repo and set it up there?

I'm using a custom domain that I've created a Google account with.

Well, that's something. Can you try to create new google account and check if you can get master token for it? Basically you need to find any setup when it works so then you can triage the issue. It can be different machine, OS, network region, password length, etc. Then you can change one variable a time back to find what causes this.

Do I need to clone the glocaltokens repo and set it up there?

Yes, clone the repo, install Poetry (pip install poetry should be enough) and run poetry install. You can prepend commands with poetry run to use its virtualenv, e.g. poetry run python get-token.py

I'm using a custom domain that I've created a Google account with.

Well, that's something. Can you try to create new regular gmail account and check if you can get master token for it? Basically you need to find any setup when it works so then you can triage the issue. It can be different machine, OS, network region, password length, etc. Then you can change one variable a time back to find what causes this.

Do you want that added to the Google Home, or go 100% clean?

It doesn't matter, getting that master token only provides access to account itself, so it should work in all cases. But you can try both.

Dug up an old gmail account of mine and am able to get the master token for it. Can't add it to the Google Home (need to setup a phone linked to that account).
So I tried my wife's gmail account (which is in the Google Home) and it fails with the same reason my (non-gmail) account fails.
I'll try setting up poetry now.

vscode ➜ ~/glocaltokens/glocaltokens (master) $ poetry run python /workspaces/tmp-ha-google/get-token.py
ERROR:glocaltokens.client:[!] Could not get master token.
[*] Master token None
ERROR:glocaltokens.client:[!] Could not get master token.

[*] Access token (lives 1 hour) None

[*] Google devices local authentication tokens
ERROR:glocaltokens.client:[!] Could not get master token.
ERROR:glocaltokens.client:[!] Could not get master token.
ERROR:glocaltokens.client:[!] Could not get master token.

That's some progress! So now you need to find what makes these accounts different.
Btw, I also have personal google account on dedicated domain. But I use separate gmail account which only has access to Google Home for this integration to not expose my personal account. Although I can still get master token for my personal account just fine both on Linux and Mac. It has 2fac enabled and I use app password.

My primary Google account is created with Google Apps and that causes a WORLD of problems with Google Home, so this one have a dedicated account, but under my domain.
The email address has a + in it, the one that owns the home.

The email address has a + in it, the one that owns the home.

I wonder if that's the problem. I just tried with my primary Google account (which is on the same custom domain) and it was able to retrieve the token.
Because it's on Google Apps I can't add it to this Google Home, so I'll try a non-+ email.

Yep, that's a possible explanation.

This explains how to get extra logging from requests:
https://stackoverflow.com/questions/16337511/log-all-requests-from-the-python-requests-module
Fetching of master token is implemented here, you can probably modify that code and use gpsoauth directly (it also uses poetry):
https://github.com/simon-weber/gpsoauth/blob/master/gpsoauth/__init__.py#L91-L143

This is simple example of using gpsoauth:
from gpsoauth import perform_master_login

login =<EMAIL_ADDRESS>password = "app_password"

response = perform_master_login(login, password, "aabbccddeeff1122")
print("Response:", response)
print("Master token:", response["Token"])

Well, I remember that + is actually very special in email account. Basically if your account is<EMAIL_ADDRESS>then your actual account is<EMAIL_ADDRESS>and smth will be ignored but can be used for adding labels on email, sorting them by folders, etc.

Yeah, I know that + is special, it was mostly me being lazy about setting up the account, rather than creating a full mailbox.
vscode ➜ /workspaces/tmp-ha-google $ python basic.py
Response: {'Error': 'UNKNOWN_ERR'}
Traceback (most recent call last):
  File "/workspaces/tmp-ha-google/basic.py", line 8, in <module>
    print("Master token:", response["Token"])
KeyError: 'Token'

That's from the new script.
I'll create a complete mailbox and add it tomorrow then report back.

Have you tried using only actual part of the login? You shouldn't pass part with +. Basically it should be<EMAIL_ADDRESS>instead of<EMAIL_ADDRESS>

@aaronpowell did you have change to try login without + part?

The non-+ email is attached to a different Google Account, but it does work (meaning that a custom domain with Google Accounts is valid).
I'll try setting up yet another email and Google Account and add it to the home.

You don't have any accounts with +, they are different addresses which are delivered to the same account. To use it, you should strip part after + from login.

Stripping the + didn't do anything (going from<EMAIL_ADDRESS>to<EMAIL_ADDRESS>as the + really only impacts on my mail provider end.
But I did create a new Google account (homeassistant@...), attempted to get the token before adding it to the Home - that worked.
I then added it to the Home and ran the script again - it still worked, and it's listing the Google Nest devices!
I'll rerun the tests again later today to ensure there was no fluke caching happening, but it might be something to do with the way +-based email addresses are handled.

It should be just<EMAIL_ADDRESS>not<EMAIL_ADDRESS>or<EMAIL_ADDRESS>

It should be just<EMAIL_ADDRESS>not<EMAIL_ADDRESS>or<EMAIL_ADDRESS>
Not quite - I have a Google Account that uses the email<EMAIL_ADDRESS>which is linked to a legacy Google Apps instance, so is technically a work account as far as Google is concerned. That account can't be used with Google Home properly (it can own a Home but you can't add anyone else to the Home, which is a major limiting factor for family use), so I created an alias'ed email using<EMAIL_ADDRESS>which then has its own Google Account.
Either can be logged into, one has all my YouTube history, my Google search history, basically everything that Google knows about me, the other is really just for use with Google Home, sure, it could have YouTube history (and, well, it does thanks to watching YouTube on Google TV), but they are completely separate identities as far as Google is concerned.
Now maybe that's the problem, the auth process is mistakenly handling the +... segment of the email badly, but it's still a completely valid email (just like using {, }, |, #, and a bunch of other symbols in the local part). It's then up to the mail server to decide how to handle it. Gmail popularised using + to "tag" emails (Gmail calls it "task oriented email" and Exchange Online calls is "plus addressing"), but you could have an email where that is just part of it, and not doing something special at the mail server to redirect to a certain inbox.
To illustrate, I tested with<EMAIL_ADDRESS>(which isn't part of the Google Home, but the plus address is) and it gets a master token, so ✅, but it does not get the Google Home/Nest devices in the Home, so ❌.
If I then try with the new, non-+<EMAIL_ADDRESS>address (which is part of the Google Home), I can get the master token, so ✅, and it does get the list of Google Home/Nest devices in the Home, so ✅ again.
Ultimately, I think I have a super weird edge case that's probably more of a problem at Google's any than anything, and given the obscurity of it, they probably won't fix it, but I do feel like #270 might result in some false positives.

So if I understand you correctly you have Google account<EMAIL_ADDRESS>which is registered to non-google email address? Surprisingly that seems possible, otherwise I don't see how to achieve this.
Because G Suite doesn't allow to create such account:

When I try to add +test to my existing account in the login form, it says account doesn't exist:

Awesome!


better error message if kid is not found

Closes #19
I've used a similar error message to what they use here: https://digital.nhs.uk/developer/guides-and-documentation/security-and-authorisation/application-restricted-restful-apis-signed-jwt-authentication

Thank you for trying to work this out.
Have you tested the changes? Because I have doubts it works correctly. Notice you are raising within a try block, and it's being matched into this generic exception: https://github.com/dorinclisu/fastapi-auth0/blob/541af7f995e350e20409fa0d3625dac41e028e29/src/fastapi_auth0/auth.py#L160-L165 which will result in "Error decoding token" final message.
Please also note than Auth0UnauthorizedException is not suitable because it would imply the authentication succeeded yet there are missing permissions. Auth0UnauthenticatedException is more suitable for this case.
Please add a test case (I think it's ok to hardcode your token for now, which will surely have a different kid than the one set up in CI) in which you check for the relevant keyword in the message, similar to the other tests.

Thanks for the review.
You're correct. The exception was swallowed by the except Exception:.
I've corrected the bug, switched to Auth0UnauthenticatedException, tested locally that it works, and also added a test case with an Auth0 access token to a test application of mine. The exception should still be thrown once the token expires.

Thank you, now it works. I might do a bit more polishing (there's also some commented out code I want to remove) and then publish a new version asap.


Update TypeScript Wikipedia article

The Wikiepedia article wrongly mentions that TypeScript uses header files. I know they mean interfaces, but this is not the same thing. This can confuse and discourage persons investigating TypeScript. I suggest someone from Microsoft look into getting that corrected.
https://en.wikipedia.org/wiki/TypeScript:
"TypeScript supports header files that can contain type information of existing JavaScript libraries, enabling other programs to use objects defined in the header files as if they were strongly typed TypeScript objects. There are third-party header files for popular libraries like jQuery, MongoDB, and D3.js. TypeScript headers for the Node.js basic modules are also available, allowing development of Node.js programs within TypeScript.[7]"

Isn't the principle roughly the same as a C/C++ header file? It specifies the declaration and types of things that will exist at runtime (link time in C/C++) but are not part of the compilation unit. I suspect the author was just trying to express the idea using terminology the reader might already be familiar with.

TypeScript doesn't require d.ts files to transpile. They exist as a way to offer types at compile time. They are a convenience, not a requirement. Similarly interfaces are not required to compile TypeScript. You can write valid TypeScript without every using an interface. So, calling TypeScript declaration files headers is very misleading, especially if you're tring to attract Web developers. I can hardly imagine a normal Web developer wanting to write headers files for their JavaScript.

You don't need header files in C/C++ if everything is part of the same compilation unit. Same thing.

Changed it:

TypeScript supports definition files that can contain type information of existing JavaScript libraries, much like C/C++ header files describe the structure of existing object files. This enables other programs to use the values defined in the files as if they were statically typed TypeScript entities.


Describe Table

Hi everyone
Is there a describe table call for phoenix managed hbase tables.
I was looking in the source and cannot find one.
My use case is to check a table structure, families and columns before making calls to validate my query.

In sqlline.py you can do a !columns "view/table" to get a description of the columns in a particular view/table.


Optimize rocket

Pulling over @SergioBenitez's code from #26 (for easy one-click-merging^^) and updating Rocket's dependencies along the way.

Great!!


Paginate /developers

installed faker for seeding test data.
installed and configured pagy.
adds a new pagination_controller that will help paginate the elements.
stop at the last page to prevent the client from sending extra requests by using
https://github.com/A7madXatab/railsdevs.io/blob/1157dcb1f797cfdab66b3a21f191f13527e82a18/app/views/developers/index.turbo_stream.erb#L4
closes https://github.com/joemasilotti/railsdevs.io/issues/38.
for the pagination controller i took inspiration from one i built(and use in my work). Check it out here

https://user-images.githubusercontent.com/38360603/140589509-a7ff365a-15b9-4ac6-8290-169c4aaa5ed0.mov

I'm not sure if it is desirable or necessary to be this "lean", but it is actually decently straightforward to leverage Turbo for 0 custom code infinite scroll. I can give examples if that's a route you all want to take, but I understand just sticking to a stimulus controller since it's already written

My gut reaction is the less JavaScript we have to manage the better.

@A7madXatab  if you want any suggestions, let me know. I'd be happy to help. I will have a decently busy weekend though, so I'll try to respond when I can if you want any help

@joemasilotti all the issues you pointed out were resolved. Lemme know if i need to make other changes!. Thanks for the IntersectionObserver shoutout. I didn't know how to use it before.

Looking good! I left the default pagination visible in case the user doesn't have JavaScript enabled. Then, when the Stimulus controller connects, the pagination is hidden. I'll merge this when CI is green. Thanks for the PR!

Looking good! I left the default pagination visible in case the user doesn't have JavaScript enabled. Then, when the Stimulus controller connects, the pagination is hidden. I'll merge this when CI is green. Thanks for the PR!

Thanks for merging this!. Can't wait to contribute more to this repo!.


Add "open in browser" functionality

#3

Resolved the conflict and merged your change. Thanks @thedeany !
Thanks for bringing this project back to life. If you want to tackle another one, I will be very happy to help in any way.

@thedeany Once the conflict is resolved, I can push the changes

@sivahq Sorry about that. I hadn't noticed the conflict!
Thanks for merging my changes. I'll be looking at some of the other issues, and I may chat with you on how you'd want to implement some of them!

Happy to help!

Thanks for the contributions so far.


Issues with sharing chats

Switching between gpt-4 and chatgpt doesn't work.
The OpenAI key prompt is super confusing as no input box visible..Should probably not show that message in this case.

Repro(in private window):
https://chatcraft.org/c/tarasglek/X7OEaeIoke-7BEugHh6CX

I've dealt with 2. in #122.
There are a few ways to deal with 1. (i.e., not being able to switch versions in a shared chat).  The underlying problem is that we rely on the database for our data reactivity; that is, if you want the Chat to change (e.g., "show me a different version"), you have to update it in the database.  When we load a shared chat, we get it via HTTP as JSON and create a Chat in memory.  This has no reactivity (i.e,. the instance never changes), so this UI won't update.
We can fix it by doing:

Not showing the version switcher for shared chats.  A remote chat is really "read-only" so we could embrace this and only show you the versions that the author chose.
Copy the remote chat into our database and load it from there, which will make it reactive.

Doing 2. would involve creating a pull-through cache in your database from the R2 bucket.  When you request /username/id we'd load it over JSON, put it in the database using an id of username/id vs. our usual id (to differentiate chats I own vs. ones being cached), then load it from the db.
This would complicate the db a bit, since we'd have to filter out shared chats from ones you own based on ID.  We'd also have to add a bit of code to deal with the shared chat updating (e.g., update local copy in db when loading), deleting from local db if shared chat is deleted, etc.
I'm not sure how important this feature is, but it's doable with some work.  Let me know your thoughts.
I had a good chat with ChatCraft about it, if you want to read along for more technical detail: https://chatcraft.org/c/humphd/AUpJW_Jfb6ljp5ZM2N6SB

I think 1 is ok.
2 feels like overengineering

Agreed.  I'll morph this into an issue to make remote chats truly read-only.


Update all.json

Generic seed stealer, no polkadot explicitly, but it has "other"
"mneumonicactivator.com",

Dynamic CDN or fast fluxer botnet hosting

Fake uniswap/metamask scam
"online-uniswap.tech",

thank you

Always a pleasure :)


Can open an ipynb notebook integration test failure

https://dev.azure.com/monacotools/Monaco/_build/results?buildId=181930&view=logs&j=9833cdaa-2e92-5f8d-56a8-239fd18fd628&t=9bc48eac-eb66-52c8-7250-d6490c5f3a15&l=297

Unfortunately it seems not really actionable since the tests crashed or timed out partway through, so we don't have the error result from the test.


MuiCheckbox-colorSecondary class mistakenly applied to selectable row checkboxes

This issue is most apparent if you use a later verison of Material UI (I know the lib currently only supports v3, but having the incorrect class may have other side effects). If you use MUI v4 and you hover over a checkbox, you see the secondary color, but when you click the checkbox, you see the primary color:
https://codesandbox.io/s/muidatatables-custom-toolbar-yv860
This mismatch is due to how the library is setting the color for the checkbox. If you inspect the element, you find that "MuiCheckbox-colorSecondary" is being applied. The secondary color is the default if no color is provided to the Checkbox component:
https://v3.material-ui.com/api/checkbox/
The library sets the color using a class override:
https://github.com/gregnb/mui-datatables/blob/master/src/components/TableSelectCell.js#L37
This issue can be fixed if the color is instead set via the Checkbox's "color" attribute.
Expected Behavior
MuiCheckbox-colorPrimary-* class applied to selectable row checkboxes.
Current Behavior
MuiCheckbox-colorSecondary class applied to selectable row checkboxes.
Steps to Reproduce (for bugs)

Right click a selectable row checkbox and inspect it's properties

Your Environment
ClassName mismatch happens regardless of environment.

Tech
Version

Material-UI
4.2

MUI-datatables
2.6.4

React
16.8.6

browser
Firefox

etc

Should be fixed in 2.7.0.


Add support for Okta oauth2 integration

[x] Issue: #160
[x] List of related PRs. : None
[x] Project builds without any errors.
[x] Unit Tests updated (or created where needed) and all pass.
[x] You have ran manual functional “smoke” test (does product as a whole still work?).
[x] User Story meets the acceptance criteria - and passes. acceptance criteria should be understood at outset.
[x] There is a PR for update of internal documentation for the affected Project(s) - README etc.
[x] Description of the changes: #160
[x] Signed-off-by: Mahesh Lokhande<EMAIL_ADDRESS>Okta_settings.docx

@maheshlokhande1 all PRs should be targeting the develop branch. Would you be ok to update this PR?

@maheshlokhande1 all PRs should be targeting the develop branch. Would you be ok to update this PR?

Done.

Comments added. I also have some general comments:

Unit tests need to be added for Okta (you will find where we added similar tests for Zen)

Documentation needs to be updated for Okta.

Getting Started should include steps for logging in with Okta
Command reference for the login command should be updated with the new parameters
Example config in the configuration guide should include an example of Okta
Check for other places that Okta specific docs may be needed

Updated docs.


Add TencentQQ  <IP_ADDRESS>009

Microsoft Reviewers: Open in CodeFlow

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: wuwanhao sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Docs-update-link

Update footer for Training

Request from Marketing: https://tigera.atlassian.net/browse/DOCS-246.
Merge to /nightly, cherrypick to 3.19.1
Erica approved

/merge-when-ready delete-branch

OK, I will merge the pull request when it's ready, leave the commits as is when I merge it, and delete the branch after I've merged it.

I'm sorry but I failed delete the branch after merging the pull request.


Assertion failed: (hasType(node) && "Expected type to have been set!") when opening existential by default

Since https://github.com/apple/swift/pull/41996 the following code completion test case started crashing:
$ cat /path/to/test.swift
protocol Frror {}
protocol LocalizedError : Frror {}

func foo(error: Frror) {
  (error as? LocalizedError)#^COMPLETE^#
}
$ ./sourcekitd-test -req=complete -pos=5:29 /path/to/file.swift -- /path/to/file.swift

I also noticed that EnableOpenedExistentialTypes now defaults to true when parsing command line arguments but the default in LangOptions.h is still false.
A nicer debugging experience for the above bug can be gained by either

Changing EnableOpenedExistentialTypes in LangOptions.h to true (this is what I think should be done anyway) or
Adding InitInvok.getLangOptions().EnableOpenedExistentialTypes = true; in swift-ide-test.cpp:4267

And then running
$ ./swift-ide-test -code-completion -source-filename /path/to/test.swift -code-completion-token COMPLETE

rdar://92156002

Fixed by https://github.com/apple/swift/pull/59814!


migrations implementation

I'm investigating what the possibilities are to run database migrations.  You are using initContainers.  The problem with initContainers is they are associated with a pod. Let's say the deployment is scaled to 3 pods. The database migrations will run 3 times, right?
A better solution is to include one "job" in the helm chart and have helm hooks run the job during each app upgrade.
Are there yet other solutions besides those? that is the question.

Sorry for the late response. It seems I missed the notification for this issue. Regarding your point about using initContainers, you're absolutely right. Init containers run before each pod, ensuring that the migrations are applied or skipped for each pod individually. This is the reason why I chose to use initContainers - I wanted to ensure that the migrations run successfully before creating any new pods during a new version deployment. Even if a migration fails, it won't affect the deployment, and the application can still remain in a consistent state.
On the other hand, with a Job, the migration process is decoupled from the pod creation process. If a migration Job fails, it won't block the deployment or prevent the creation of new pods. This decoupling can be advantageous if you prefer to handle the migration process separately from the deployment process and have the flexibility to manage failures independently.
Thank you for pointing that out.

This doesn't resolve the issue above, but here is something I did as MySQL wasn't ready when the init ran.  I also had to put numbers in front of them as they were not running in order otherwise (I had additional inits).
1_dependency:
    enabled: true
    name: dependency
    # Creates a delay to wait for MySQL to become available before running migrations
    command: 'echo "Waiting for MySQL Connection..."; while ! nc -z releasename-mysql 3306 2>/dev/null; do sleep 1; done; echo "* MySQL is Online *"'
  2_migrations:
    enabled: true
    name: migrations
    command:  "python3 manage.py migrate"
  3_staticfiles:
    enabled: true
    name: staticfiles
    command: "python3 manage.py collectstatic --noinput;'

race condition.

Yes, it is technically possible to encounter race conditions when running migrations on multiple Django apps simultaneously. Although I haven't personally encountered any issues with my applications, which I often scale with tens of pods, it's important to consider this possibility. Ill add a job as option to use for migrations


Bump Garden Linux VM

Description
Bump Garden Linux VM to 934.4
Checklist

[ ] Investigated and inspected CI test results

Testing Performed

[ ] Green CI with the all-integration-tests label

The driver failures have already been addressed in master, so I'll go ahead and merge this.


Importer sourceMap

It could be a crazy question. But didn`t even found a mention about it. For example json or yaml importers, that mapping data structure to sass variabled in global scope, would be wonderful to pass sourceMap on this step. Or even more sensetive things like import non-sass processed css files from other modules.
Something like:
done({
  contents: 'file_transformed_to_sass',
  sourceMap: 'sourceMap_of_file_to_compiled_scss_file'
});

I don't understand the problem you're trying to solve. Could you please be more specific?

Sorry for so clumsy start.
The question is about custom importers. For example, i`ve got json importer function, that takes json, and generating scss documents like:
('vars.json') JSON:
{
  "foo": "100px",
  "bar": "200px"
}

(converted 'vars.json') to Scss:
$foo: 100px;
$bar: 200px;
// sourceMap = ****

So when i will give the content to importer with scss content inside, there is wont be even a mention about the json file in generated css file. So i`ve been thinking about it, and that would be great to pass the sourceMap data to importer with content. And when you will open a debug tools youl see your "style.scss" and "vars.json" with correct mapping to them.

This sounds like a feature request that should be made in the libsass repo

Another example would be this package, basically it makes sass import globs like @import 'components/*.scss';, but the problem is that in the browser the name of the file is *.scss instead of the actual file name.

https://github.com/sass/libsass/commit/96ff08ce8a0e724d5333179413a0bd6cfff5baa3#diff-dbf9ef646767bb67f153df33f848eda8R14
explains that libsass does not have this implemented yet


Update app for modern Django

Upgrade to support Django 1.7 and up.

+1
Sent from my iPhone

On Jul 25, 2015, at 12:23 PM, Jacob Wegner<EMAIL_ADDRESS>wrote:
Upgrade to support Django 1.7 and up.
—
Reply to this email directly or view it on GitHub.

Should now be Django 1.8 and up.
:gasp:


Type completion of argument is no longer performed at the time of the definition of the function in version 2.2-beta

We have always used.
Thank you very much.
Type completion of argument is no longer performed at the time of the definition of the function in version 2.2-beta R#9.2

Yeah....I also meet this issue, type completion in function declaration was missing if postfix installed.

Sorry, folks, fixed this in 2.2.1-beta

We made sure the fix.
Thank you very much.


list specific features that require a shared mailbox to be licensed

[Enter feedback here] Could the article be updated to list all advanced features that require a shared mailbox be licensed?

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

ID: 289f2440-9c44-7d93-20e3-baa9e9892cd9
Version Independent ID: abc4e209-a071-d28a-cb01-aa72bbaee7c0
Content: About shared mailboxes - Microsoft 365 admin
Content Source: microsoft-365/admin/email/about-shared-mailboxes.md
Service: microsoft-365-business
GitHub Login: @SKjerland
Microsoft Alias: sharik

@GHSandy Thank you for your feedback. We have added a note on what features will be available when licensed

@msbemba Thank you very much for the contribution and updating the documentation with PR. @GHSandy Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Reduce timeout of read/query methods [Cosmos DB Java SDK]

The Cosmos DB java SDK has a default fixed timeout of 65sec for read/query operations which is not configurable. This is too high as the HTTP call itself will get timed out before the timeout of 65 seconds reach. Also, as the latency of read/query operations are usually in milliseconds, so waiting for 65 seconds for timeout is very unnecessary.
In the .NET SDK, this issue was addressed by reducing the default timeout of read/query methods to 5/10 seconds (https://github.com/Azure/azure-cosmos-dotnet-v3/pull/1954).
Expecting to reduce the default timeouts in Java SDK as well or provide support to accept timeout duration as an argument or configurable.

This is done.


feat(protocol): network MessageCodec to sync

What this PR does / why we need it:
Remove unnecessary async context switch.
Which issue(s) this PR fixes:

Fixes #
Which docs this PR relation:
Ref #
Which toolchain this PR adaption:
No Breaking Change
Special notes for your reviewer:

/lgtm


tour: wrong package name

Context: https://go.dev/tour/basics/16
Please fix wrong package name from numpackage main to package main

That's probably a local edit, please click the Reset button at the bottom of the code window.
Your browser will remember any edits that were done in a previous session.
Do you still see the typo after pressing Reset?

I misunderstood the local edit. Thank you for your explanation. It works
after Reset.
On Mon, Mar 6, 2023 at 5:24 PM Daniel Frederick Crisman <
@.***> wrote:

That's probably a local edit, please click the Reset button at the bottom
of the code window.
Your browser will remember any edits that were done in a previous session.
Do you still see the typo after pressing Reset?
—
Reply to this email directly, view it on GitHub
https://github.com/golang/tour/issues/1460#issuecomment-1455865655, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAL6UAJOOWN2YQN5LBJKHGLW2W3N3ANCNFSM6AAAAAAVQDIPQ4
.
You are receiving this because you authored the thread.Message ID:
@.***>

If you have a moment, please close this issue.  Thanks.


[SPARK-44214][CORE] Support Spark Driver Live Log UI

What changes were proposed in this pull request?
This PR aims to add Driver Log UI in order to provide a better UX like Standalone Cluster.
Why are the changes needed?

Since Apache Spark 3.0.0, Spark History Server supports driver logs via spark.driver.log.persistToDfs.enabled and spark.driver.log.dfsDir.
For Apache Spark 4.0.0, this PR aims to support Live Driver Log UI page in Spark Driver UI (port 4040) via spark.driver.log.localDir. It means we can see the live driver log like the following via UI.

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass the CIs with the newly added test case.

Could you review this driver Live UI PR too, please, @viirya ?

Thank you for review, @viirya !

Thank you so much for review and approval! I verified inside K8s cluster on top of local shell environment.

Merged to master for Apache Spark 4.0.0.

Late LGTM

Thank you, @yaooqinn .