[SPARK-6443][Spark Submit]Could not submit app in standalone cluster mode when HA is enabled

3/26 update:

Akka-based:
Use an array of ActorSelection to represent multiple master. Add an activeMasterActor for query status of driver. And will add lost masters( including the standby one) to lostMasters.
When size of lostMasters equals or greater than # of all masters, we should give an error that all masters are not avalible.

Rest-based:
When all masters are not available(throw an exception), we use akka gateway to submit apps.

I have tested simply on standalone HA cluster(with two masters alive and one alive/one dead), it worked.
There might remains some issues on style or message print, but we can check the solution then fix them together.
/cc @srowen @andrewor14

Test build #28942 has started for   PR 5116 at commit da4ad2e.

This patch merges cleanly.

Test build #28942 has finished for   PR 5116 at commit da4ad2e.

This patch fails Spark unit tests.

This patch merges cleanly.

This patch adds the following public classes (experimental):

  case class SubmitDriverResponse(

  case class KillDriverResponse(

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

retest this please

Test build #29138 has started for   PR 5116 at commit da4ad2e.

This patch does not merge cleanly.

Test build #29138 has finished for   PR 5116 at commit da4ad2e.

This patch fails Spark unit tests.

This patch does not merge cleanly.

This patch adds the following public classes (experimental):

  case class SubmitDriverResponse(

  case class KillDriverResponse(

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

Test build #29146 has started for   PR 5116 at commit fa1fa80.

This patch does not merge cleanly.

Test build #29147 has started for   PR 5116 at commit 60d97a4.

This patch merges cleanly.

Test build #29147 has finished for   PR 5116 at commit 60d97a4.

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/29147/
Test FAILed.

Test build #29146 has finished for   PR 5116 at commit fa1fa80.

This patch fails Spark unit tests.

This patch does not merge 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/29146/
Test FAILed.

Test build #29156 has started for   PR 5116 at commit 2b011c9.

This patch merges cleanly.

Test build #29156 has finished for   PR 5116 at commit 2b011c9.

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/29156/
Test PASSed.

Though passed all the tests, it still doesn't work well in some cases.
I'm still working on it.

Test build #29205 has started for   PR 5116 at commit 7a881b3.

This patch merges cleanly.

Test build #29205 has finished for   PR 5116 at commit 7a881b3.

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/29205/
Test FAILed.

The UT failed because of hive compile error which would be fixed in https://github.com/apache/spark/pull/5198.

Jenkins, test this please.

Test build #29222 has started for   PR 5116 at commit 7a881b3.

This patch merges cleanly.

Test build #29222 timed out     for PR 5116 at commit 7a881b3     after a configured wait of 120m.

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

Jenkins, test this please.

Test build #29229 has started for   PR 5116 at commit 7a881b3.

This patch merges cleanly.

Test build #29229 has finished for   PR 5116 at commit 7a881b3.

This patch fails PySpark 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/29229/
Test FAILed.

Jenkins, test this please.

Test build #29235 has started for   PR 5116 at commit 7a881b3.

This patch merges cleanly.

Test build #29235 has finished for   PR 5116 at commit 7a881b3.

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/29235/
Test PASSed.

@andrewor14 Could you please take a look?

@WangTaoTheTonic there seems to be two bigger issues with this patch in its current state:
(1) Once a Master fails, it is marked as lost forever. This means the following race condition is possible: a Master fails and then recovers, and then all other Masters fail. In this case, there is still one living Master, but the code will System.exit(-1) anyway.
(2) There is not a type safe way to determine if a response is from a backup Master. The code currently searches for strings like "Can only" in the response messages to conclude that it is from a backup. This is very brittle; if someone decides to reword those messages slightly then everything will break.
To solve (1), we can have the client ping all Masters periodically or whenever a disassociation event is fired. This introduces more complexity and I would be inclined to leave it out. Instead we should add a huge comment to explain the potential race condition. Note that this condition is somewhat unlikely because the submission process is meant to be fire-and-forget instead of long-running.
To solve (2), we may need to add a field in the response messages to convey the fact that the response is from a backup. I haven't given the exact parameter a ton of thought yet, but whatever we introduce here we have to keep for backward compatibility.

@andrewor14 Thanks very much for comments.
I totally agree with you. Actually these two points are also in my main concern.
About first issue, considering client now just submits app once and exit with no retry when submit failed, even if the lost master recovers, it has never opptunity to accept request from one submit in same client. So we could add some comments here and improve it when we try to support retry in client submission.
About Issue No.2, return message is kinda weak flag too. Now I couldn't see a way that can achive backward compatibility with new parameter added. Perhaps we could make the message contents (like "Can only") common field to avoid reword one-side? Though it is a hack, we can give it a try.

Test build #29663 has started for   PR 5116 at commit 8569323.

Test build #29663 has finished for   PR 5116 at commit 8569323.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

Jenkins, test this please.

Test build #29675 has started for   PR 5116 at commit 8569323.

Test build #29675 has finished for   PR 5116 at commit 8569323.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

The exception handle logic still has some issues, and some compile problems are introduced.
I am in vacation now, will fix it as long as I'm back (maybe 3 days later).

Test build #29792 has started for   PR 5116 at commit 5d23958.

Test build #29792 has finished for   PR 5116 at commit 5d23958.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

Test build #29795 has started for   PR 5116 at commit e4f4ece.

This patch does not merge cleanly.

Test build #29796 has started for   PR 5116 at commit 979760c.

Test build #29796 has finished for   PR 5116 at commit 979760c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

Test build #29795 has finished for   PR 5116 at commit e4f4ece.

This patch passes all tests.

This patch does not merge 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/29795/
Test PASSed.

@andrewor14
I updated some exception handle logic and one unit test (from catch SubmitRestProtocolException to catch JsonParseException in L351 & L361 in StandaloneRestSubmitSuite). Not sure if it is right to modify the unit test directly. Please check, thanks.

ping? @andrewor14

Hi @WangTaoTheTonic I will look at this today.

@WangTaoTheTonic this seems to adequately address the two issues I raised previously. I think this correctly achieves what we want, but I have concerns about the duplicate code. I commented inline in more detail about this.

Test build #30393 has started for   PR 5116 at commit 9d636be.

Test build #30393 has finished for   PR 5116 at commit 9d636be.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

Test build #30394 has started for   PR 5116 at commit 35119a0.

Test build #30394 has finished for   PR 5116 at commit 35119a0.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

Test build #30410 has started for   PR 5116 at commit 220cb3c.

Test build #30410 has finished for   PR 5116 at commit 220cb3c.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

Hey andrew, I have already updated the codes per your comment, but still left one issue.
I think catching SubmitRestConnectionException in createSubmission/requestSubmissionStatus/killSubmission is more reasonable than in readResponse as we use requestSubmissionStatus to Read the response from the server and return it as a validated and we submit requests to multiple masters in createSubmission/requestSubmissionStatus/killSubmission so we should take the response there either.
Also i move the catching for ConnectionException into postJson in case one master is down.

Jenkins, test this please.

Test build #30416 has started for   PR 5116 at commit 220cb3c.

Test build #30416 has finished for   PR 5116 at commit 220cb3c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

Test build #30719 has started for   PR 5116 at commit a41de0b.

Test build #30719 has finished for   PR 5116 at commit a41de0b.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

Test build #30724 has started for   PR 5116 at commit f4f972b.

Test build #30724 has finished for   PR 5116 at commit f4f972b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

@andrewor14 I've changed and made some rebase. If everything is ok i'd like to merge it ASAP. Please take a look.

ping @andrewor14

Test build #31047 has started for   PR 5116 at commit f4f972b.

Looks like Jenkins has done his work but not posted the result here.

retest this please

Merged build triggered.

Merged build started.

Test build #31161 has started for   PR 5116 at commit f4f972b.

I think catching SubmitRestConnectionException in createSubmission/requestSubmissionStatus/killSubmission is more reasonable than in readResponse as we use requestSubmissionStatus to Read the response from the server and return it as a validated and we submit requests to multiple masters in createSubmission/requestSubmissionStatus/killSubmission so we should take the response there either.

If you look into the these methods you'll notice that the only place where it actually makes a connection to the master is in post or postJson, and both of these methods call readResponse. The current scope of catching these exceptions is too wide (we will never throw this exception outside of readResponse), and the cost we pay is duplicate code.

By the way the rest of the changes look fine to me.

Test build #31161 has finished for   PR 5116 at commit f4f972b.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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/31161/
Test FAILed.

I believe we will never throw this exception outside of readResponse

No. In postJson, it will throw a ConnectException when try to get output stream of a connection while the remote server is down. I take care of it here:

try {
val out = new DataOutputStream(conn.getOutputStream)
Utils.tryWithSafeFinally {
out.write(json.getBytes(Charsets.UTF_8))
} {
out.close()
}
} catch {
case e: ConnectException =>
throw new SubmitRestConnectionException("Connect Exception when connect to server", e)
}

I see, that seems to be the case, then I would do it in get, post and postJson instead of the higher level methods.

Sorry but I don't see benefit to push it down which still have same duplicated code.
I think we should catch SubmitRestConnectionException while send an request to one master rather than post a request to some url.  Code-wise they are same in this case, the meaning is different though.

Merged build triggered.

Merged build started.

Test build #31227 has started for   PR 5116 at commit 76fd411.

Test build #31227 has finished for   PR 5116 at commit 76fd411.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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/31227/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #31259 has started for   PR 5116 at commit 2a28aab.

Test build #31259 has finished for   PR 5116 at commit 2a28aab.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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/31259/
Test FAILed.

Ok, no problem. I will merge this first and we can always do any clean ups later if needed. I don't want to make you keep rebasing.

But first, retest this please

Merged build triggered.

Merged build started.

Test build #31303 has started for   PR 5116 at commit 2a28aab.

Test build #31303 has finished for   PR 5116 at commit 2a28aab.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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/31303/
Test PASSed.

@andrewor14 Thanks for all comments. :)

Merging into master.


refactor(device_info_plus): refactored the platform specific implementation

Description
Refactored the user facing platform specific implementation.
BREAKING CHANGE: Introduced Dart 3 features and removed platform specific implementation from user facing implementation
Related Issues
None
Checklist

[X] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[X] I titled the PR using Conventional Commits.
[ ] I did not modify the CHANGELOG.md nor the plugin version in pubspec.yaml files.
[ ] All existing and new tests are passing.
[X] The analyzer (flutter analyze) does not report any problems on my PR.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[X] Yes, this is a breaking change (please indicate that with a ! in the title as explained in Conventional Commits).
[ ] No, this is not a breaking change.

@mhadaily Can you please review it?

Thanks for your contribution, but this PR can't be accepted because of multiple reasons:

We don't plan to enforce Dart 3 for now for all users of the plugin, because Dart 3 is a fresh release an quite a lot of projects still haven't migrated to it. Plugin's functionality works fine without Dart 3 specific features.
You rolled back quite a lot of improvements, which were recently added into the plugin, including example app changes, so I assume that you didn't update your branch before making changes and pushing the code.
The value of the suggested change is questionable, considering that you suggest to bump min Dart to 3.

Based on that I am closing this PR.


[NSError] Use Error for user info providers to match the Darwin version

@swift-ci please test and merge

Base branch was modified. Review and try the merge again.

Just a merge error.

Hm, something mysterious happened to the build after I merged this, #828, and #827. I'm going to revert while we investigate.

@parkera I'm not sure what was the reason for the failure, but can those reverted PRs be re-reverted now?

Sure - can you put up another PR so we can kick off a test-and-merge?


Consider adding !important to the [hidden] style rule

Monica Dinculescu has pointed out, that the default styles for [hidden] can be easily overridden, and that it’s probably a good idea to add !important:
[hidden] { display: none !important }

This change should not break any sites (since that would mean that they were displaying hidden content), but it would protect sites from accidentally displaying hidden content.

@simevidas, interesting proposal. I’m not sure adding !important will help more than it hurts. For instance, some people may still rely on the visually hidden technique of [hidden][aria-hidden="false"], which would require unsetting the important from [hidden].
Still, this is an interesting proposal. Do we have any evidence of [hidden] not being hidden on living sites, due to developers forgetting its unusual low specificity?

Makes sense. I’ll report back if I encounter a case where the [hidden]  rule was accidentally overridden.

This makes a lot of sense. Consider this example:
ul.nav > li { display: inline; }

with:

The display: inline declaration would override [hidden].

It would without sanitize, as well. This would hijack the aria-hidden false scenario.

@jonathantneal I guess, I didn’t understand this at the time, but now I noticed that your example, [hidden][aria-hidden="false"], violates the HTML spec:

It is similarly incorrect to use this attribute to hide content just from one presentation — if something is marked hidden, it is hidden from all presentations, including, for instance, screen readers.

https://html.spec.whatwg.org/multipage/interaction.html#the-hidden-attribute
It is not correct to use hidden in combination with aria-hidden="false" to visually hide content. There is unfortunately no standard way to do this, and a custom CSS “visually hidden” class is needed.
Setting hidden means “hide from all presentations,” so applying display: none !important should not   cause issues on pages that use hidden correctly.


Enabling examples

I try to enable the lv_demo_printer example but gets the following error:
\build/../main/main.c:204: undefined reference to lv_demo_printer'`
I followed the instructions in the examples README.md file:

1.Copy lv_examples/lv_ex_conf_templ.h next to lv_examples directory
2.Rename it to lv_ex_conf.h
3.Change the first #if 0 to #if 1 to enable the file's content
4.Enable or Disable modules

Also added:
#include "lv_examples/src/lv_demo_printer/lv_demo_printer.h"
and then I call the lv_demo_printer() instead of the (original ) lv_demo_widgets()
For some reason the  lv_demo_printer() is not compiled (and none of the other examples which are not included by default)

What development kit and display are you using?


Bump version for pypy

signoff is missing and it is PyPi.

signoff is missing and it is PyPi.

fixed


Prune workflows based on changed files

Contributes to https://github.com/rapidsai/build-planning/issues/94

/merge


Document and publish provider to terraform registry

We will need to update the repo structure to match the expected layout.
https://www.terraform.io/docs/registry/providers/docs.html
We also need to check the documentation.

Some tools to consider:
https://github.com/terraform-docs/terraform-docs
https://github.com/antonbabenko/pre-commit-terraform
https://itnext.io/terraform-enforcing-doc-generation-and-code-formatting-e7dc6098b3a0

On discussion with HashiCorp, the new process is to self-publish on https://registry.terraform.io/
For the docs, we need to follow the process described at: https://www.terraform.io/docs/registry/providers/docs.html
The complete process for managing a provider is at: https://www.terraform.io/docs/registry/providers/publishing.html#preparing-your-provider

Published at: https://github.com/nirmata/terraform-provider-nirmata


ClearSessionCookie doesn't work for splitted cookies

Session cookies will be splitted if they're too large. On logout ClearSessionCookies should remove
all cookies. That does not work if cookies are splitted due to naming
Expected Behavior
ClearSessionCookies should remove all cookies set on login.
Current Behavior
When a user logs in SetSessionCookie() is responsible for setting an cookie with user information. The cookie is
created by MakeSessionCookie(...) with provider specific information. If the length of the cookie is to large, it
will be split into several cookies which are named <cookie_name>_0, <cookie_name>_1, ...
-> SET: _oauth2_proxy_0, _oauth2_proxy_1
Cookies are deleted by creating an empty cookie with be same name that has already expired. The browser will remove the
old one. The ClearSessionCookie(...) calls MakeSessionCookie(...) to create the "delete"-cookie. This time is is called
without value (to create that empty cookie) resulting in an unsplit delete-cookie.
-> UNSET: _oauth2_proxy
Possible Solution
The ClearSessionCookie() method should search the request cookies for cookies that start with OAuthProxy.CookieName and
remove them
Steps to Reproduce (for bugs)
We added a test to oauthproxy_test.go:
func TestClearSplittedCookie(t *testing.T) {
	p := OAuthProxy{CookieName: "oauth2"}
	var rw = httptest.NewRecorder()
	req := httptest.NewRequest("get", "/", nil)

	req.AddCookie(&http.Cookie{
		Name:  "test1",
		Value: "test1",
	})
	req.AddCookie(&http.Cookie{
		Name:  "oauth2-0",
		Value: "oauth2-0",
	})
	req.AddCookie(&http.Cookie{
		Name:  "oauth2-1",
		Value: "oauth2-1",
	})

	p.ClearSessionCookie(rw, req)
	header := rw.Header()

	assert.Equal(t, 3, len(header["Set-Cookie"]), "should have 3 set-cookie header entries")
}

func TestClearNotSplittedCookie(t *testing.T) {
	p := OAuthProxy{CookieName: "oauth2", CookieDomain: "abc"}
	var rw = httptest.NewRecorder()
	req := httptest.NewRequest("get", "/", nil)

	req.AddCookie(&http.Cookie{
		Name:  "test1",
		Value: "test1",
	})
	req.AddCookie(&http.Cookie{
		Name:  "oauth2",
		Value: "oauth2",
	})

	p.ClearSessionCookie(rw, req)
	header := rw.Header()

	fmt.Printf("%#v\n", header)
	assert.Equal(t, 1, len(header["Set-Cookie"]), "should have 1 set-cookie header entry")
}

Context
This bug permits the logout. If the user reloads the page she will be logged in again.
Your Environment

Version used:

The bug works in current versions

Fixed in #70 🎉


Pipenv check thinks pip is ancient, fails citing CVE-2013-5123

I've searched through existing issues, and tried to find clues in the source (which I've never cracked open before). Neither turned up anything for me, so here's an issue!
Issue description
pipenv check fails when checking pip version. It warns about an issue in version of pip older than 1.5. I've seen this on a macOS machine and a circleci docker image running pip 19.3.1 and 20.0.3 respectively.
I'm using Python 3.6, and managing python versions with pyenv.
Expected result
Since the installed version of pip is greater than 1.5, I would expect this check to pass instead of fail.
Actual result
Checking installed package safety…
37752: pip  resolved (19.3.1 installed)!
The mirroring support (-M, --use-mirrors) in Python Pip before 1.5 uses insecure DNS querying and authenticity checks which allows attackers to perform man-in-the-middle attacks. See CVE-2013-5123.

Steps to replicate
Here's a handy little script to reproduce.
mkdir pipenv_check
cd pipenv_check
pipenv --python 3.6
pipenv check

$ pipenv --support
Pipenv version: '2018.11.26'
Pipenv location: '/usr/local/Cellar/pipenv/2018.11.26_2/libexec/lib/python3.7/site-packages/pipenv'
Python location: '/usr/local/Cellar/pipenv/2018.11.26_2/libexec/bin/python3.7'
Python installations found:

3.7.5: /usr/local/bin/python3
3.7.5: /usr/local/bin/python3.7m
3.7.4: /Users/jules/.pyenv/versions/3.7.4/bin/python3
3.7.4: /Users/jules/.pyenv/versions/3.7.4/bin/python3.7m
3.6.6: /Users/jules/.pyenv/versions/645_a2/bin/python3
3.6.6: /Users/jules/.pyenv/versions/3.6.6/bin/python3.6m
2.7.16: /usr/bin/python
2.7.16: /usr/bin/pythonw
2.7.16: /usr/bin/python2.7
2.7.8: /Users/jules/.pyenv/versions/2.7.8/bin/python
2.6.6: /Users/jules/.pyenv/versions/test/bin/python
2.6.6: /Users/jules/.pyenv/versions/hadoop/bin/python
2.6.6: /Users/jules/.pyenv/versions/2.6.6/bin/python
2.6.6: /Users/jules/.pyenv/versions/2.6.6/bin/python2.6

PEP 508 Information:
{'implementation_name': 'cpython',
 'implementation_version': '3.7.5',
 'os_name': 'posix',
 'platform_machine': 'x86_64',
 'platform_python_implementation': 'CPython',
 'platform_release': '17.7.0',
 'platform_system': 'Darwin',
 'platform_version': 'Darwin Kernel Version 17.7.0: Thu Jan 23 07:05:23 PST '
                     '2020; root:xnu-4570.71.69~1/RELEASE_X86_64',
 'python_full_version': '3.7.5',
 'python_version': '3.7',
 'sys_platform': 'darwin'}

System environment variables:

PATH
__GIT_PROMPT_IGNORE_STASH
TERM_PROGRAM
PYENV_ROOT
ANDROID_HOME
TERM
SHELL
TMPDIR
Apple_PubSub_Socket_Render
TERM_PROGRAM_VERSION
TERM_SESSION_ID
__GIT_PROMPT_SHOW_UPSTREAM
USER
__GIT_PROMPT_SHOW_UNTRACKED_FILES
SSH_AUTH_SOCK
PYENV_VIRTUALENV_INIT
_
PWD
LANG
XPC_FLAGS
XPC_SERVICE_NAME
HOME
SHLVL
PYENV_SHELL
GIT_BRANCH
__GIT_PROMPT_SHOW_CHANGED_FILES_COUNT
__GIT_PROMPT_IGNORE_SUBMODULES
LOGNAME
__CF_USER_TEXT_ENCODING
PIP_DISABLE_PIP_VERSION_CHECK
PYTHONDONTWRITEBYTECODE
PIP_SHIMS_BASE_MODULE
PIP_PYTHON_PATH
PYTHONFINDER_IGNORE_UNSUPPORTED

Pipenv–specific environment variables:
Debug–specific environment variables:

PATH: /usr/local/Cellar/pipenv/2018.11.26_2/libexec/tools:/Users/jules/Library/Python/2.7/bin:/usr/local/Cellar/pyenv-virtualenv/1.1.3/shims:/Users/jules/.pyenv/shims:/Users/jules/.pyenv/bin:/usr/local/opt/gettext/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/sbin:/Applications/Wireshark.app/Contents/MacOS:/Users/jules/Library/Android/sdk/tools:/Users/jules/Library/Android/sdk/platform-tools:/Applications/microchip/xc8/v2.05/bin:/Applications/microchip/xc8/v2.05/bin
SHELL: /bin/bash
LANG: en_CA.UTF-8
PWD: /Users/jules/pipenv_check

Contents of Pipfile ('/Users/jules/pipenv_check/Pipfile'):
[[source]]
name = "pypi"
url = "https://pypi.org/simple"
verify_ssl = true

[dev-packages]

[packages]

[requires]
python_version = "3.6"

Seems like the pyup.io database is messed up:
curl -s -H "X-Api-Key: 1ab8d58f-5122e025-83674263-bc1e79e0" https://pyup.io/api/v1/safety/in
secure.json | jq .pip
[
  "<1.4",
  "<6.0",
  "<6.1.0",
  "<1.3",
  "<1.3",
  "",
  "<1.5"
]

The "" matches every version, thus the fail.

+1

+2

+3

+4

@d-me-tree, @belthaZornv, @isabelrios @AlexSunins-ECS and anybody else with the similar thoughts in mind, please don't do this ever again, anywhere. If you're affected by the issue, hit :+1: under the first comment please. Be considerate of your fellow readers/subscribers and don't spam the thread and their inboxes with meaningless messages. Thanks.

As noted in the commit above, workaround for now is
pipenv check --ignore 37752

+5

+6

@d-me-tree, @belthaZornv, @isabelrios @AlexSunins-ECS and anybody else with the similar thoughts in mind, please don't do this ever again, anywhere. If you're affected by the issue, hit +1 under the first comment please. Be considerate of your fellow readers/subscribers and don't spam the thread and their inboxes with meaningless messages. Thanks.

Also guys, do you mind to delete the comments to avoid this happening again?

I logged this yesterday: https://github.com/pyupio/safety-db/issues/2286#issuecomment-596318705

looks like they never fixed it

The fix in pyupio/safety-db#2286 should be enough to resolve this issue. I verified pipenv check is no longer complaining on my projects. Closing this issue. Thanks @thehesiod for your help!


Use a dedicated user account

Enhancement Description

One-line enhancement description (can be used as a release note): dedicated Keylime user account
Keylime Enhancement Proposal: TBC
Primary contact (assignee): @axelsimon
Enhancement target (which target equals to which milestone):

Alpha release target (x.y) TBC
Beta release target (x.y) TBC
Stable release target (x.y) TBC

Based on https://github.com/keylime/keylime/issues/181.


Fix nodes stats parsing for AWS managed Elasticsearch clusters

AWS doesn't include the path of data directories in the _nodes/stats output. This caused an error when the parser tried to extract the path to use as a label.
When parsing a list of buckets (like the list of data directories), the parser will now fall back to using bucket's position in the list as the bucket name if the expected bucket name key isn't present.
Fixes #45

Fixed in 0.11.1


Design doc for Mizar-Arktos Integration

Current design doc only includes the implementation of Mizar cniplug support for pod.
Service support is similar, will be added once integration test was done.

/lgtm

/approve


test: [M3-8445] Add new test to confirm changes to the Object details drawer for OBJ Gen 2

Description 📝
Add a Cypress integration test using mock API data to confirm changes to the Object details drawer for OBJ Gen 2.
Changes  🔄
List any change relevant to the reviewer.

Add new test in packages/manager/cypress/e2e/core/objectStorageGen2/bucket-object-gen2.spec.ts
Add new util functions in packages/manager/cypress/support/intercepts/object-storage.ts

How to test 🧪
yarn cy:run -s "cypress/e2e/core/objectStorageGen2/bucket-object-gen2.spec.ts"

As an Author I have considered 🤔
Check all that apply

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

@AzureLatte you have a conflict you'll want to resolve 👍

Test failure  delete-placement-groups.spec.ts is not related with this change.


ILX cleanup 4

Some more cleanup, this time generating the type definition(s) for a closure in IlxGen.fs rather than in the later phase.

We love technical-debt reduction :)
Getting curious if after ILX is removed, will that impact compile time in a perceptible manner?
I guess anyways the TAST -> IL is getting clearer and that is the main benefit.

@smoothdeveloper I think a full removal of the traversal would take off about 1 second out of 50 - maybe 2 or even 3 if there are other effects (reducing memory pressure, improving cache etc.).  Every bit counts.  But that's not in this PR.

Merging as we are green and this is cleanup


Refactor error handling

https://github.com/mercurius-js/cache/blob/3cebe49a0ce9a8ba613470bc85212851df684dd7/index.js#L139-L182 includes quite a few TODOs. In a few cases we are invoking the original resolver twice.

cc @sameer-coder


Unable to use trackpad navigation in editor.

Godot version
Godot 4.02 Stable
System information
Windows 11
Issue description
I am using an apple magictouch trackpad
https://www.apple.com/shop/product/MK2D3AM/A/magic-trackpad-white-multi-touch-surface
and am unable to do anything other than zoom in the editor 3d window. It seems Godot has support for multitouch gesture panning but it only enables on touchscreens. Could a tick be added to emulate touchscreen so I could use my trackpad and have it work the way it does in blender?
In Blender normal operation for this trackpad:
Two finger drag - orbit
CTRL + Two finger drag - zoom (two finger pinch/spread also works)
Shift + Two finger drag - pan
It would be great if I could adjust settings to match blender for seamless transition, but I can't find anything exposed within settings that allows me to navigate at all. I've seen the suggestion Shift+F as a band-aid and it works ok in the 3D Editor, but I can't use it within the 3D window of an open sub-scene and I'm not sure why.
Steps to reproduce
Attempt to use a multitouch single button trackpad to navigate within the 3D scene.
Minimal reproduction project
N/A

Thank you, you need add a space between the URL and "being" btw
For future passerbys.
The workaround is:
Editor -> 3D -> Use 3 Button Mouse, Orbit Modifier - "Choose a modifier key you want to use"


Add Type Validation for width and height Parameters in pdf_viewer Function

Currently, the pdf_viewer function accepts width and height parameters, which are expected to be integers or None. However, there's no explicit type checking in place.

Why the width has a default value of 700?

There is no exact reason.
I have thought that 30px is too narrow and 2000px is too wide.
We can change these values to more reasonable numbers.
I don't know which number is the bese, though.
Do you have any idea?

No, if we don't pass any default value, what will happen?


Using Camera as I2C Slave

Describe the bug
I am using the camera as a I2C slave, all of the communication is working.
As the master is battery powered, it also needs to power the cam, I tried doing that using the 5V supply on the connector, but when I put it to deep sleep, the power consumption is very high.
As I understand that the ultra low power consumption is only available when on battery power, I am now trying to power using the battery connector, but the problem is the camera does not even power on when supplied by battery.
So how can I supply power via the battery connector and have the device switch on without needing to press the physical switch.
I have tried setting GPIO33 high and low but that makes no difference, the cam wont power up when supplied by battery ?
Thanks
To reproduce
See above
Expected behavior
See above
Screenshots
No response
Environment

OS:
IDE &IDE Version:
Repository Version:

Additional context
No response
Issue checklist

[X] I searched for previous reports in the issue tracker
[X] My report contains all necessary details

Hi, thanks for helping I did see your post in the forum before I posted the issue here.
Unfortunately as far as I can see, that wont work for me, as the camera is a slave to another ESP32 Master which is doing a lot of other things, and one of them is to take a photo.
The RTC would only work if the TimerCam controls itself and decides when to wake up, but I need to have it take a photo on demand not when it wants to.

Hi @imliubo - can it be triggered with a RTC Wakeup pin ?

the  ultra low power mode actually mens turn off all power supply.  in this situation only two way to wakeup the device(RTC wakeup pin signal, or Wakeup button.),  but if you connected the 5V supply to the ext port. the TimerCAM will nerver shutdown. even you set the device to deepsleep mode(it maybe still have 20ma consumption),  I think you could try to add a mos switch to control the Camera 5V power supply.

Thanks, yes that's what I did.


Use correct wide char count when copying from SQLGetData buffer

The second parameter to out.append is a character count, not a byte count. Prior to this PR the returned string was twice as long as it should be, the latter half garbage.

Good catch! LGTM

Yep, LGTM!


Use TestProbe rather than self reference in NodeViewHolderSpec

part of #232

Coverage remained the same at 46.548% when pulling c42a0ba9bb6658ac770af09b7f2b6f18b5f750e2 on 232-nodeviewholderspec-reworked into 6de0d647afb41abe2a60a3e2681766f51e0645d4 on master.


Improve import declaration completions

import * from 'myModule'

It would be nice to have proposal at myModule position.

Duplicate of #188


Get path or fileReadStream from Meteor Files object

Hello,
It's probably a very simple question, but I just spent hours trying to find a solution ... for a project I am making (with https://github.com/huttarichard/instagram-private-api) I need to get either a path or a fileReadStream object from some image that I have stored with Meteor Files (using the S3 example).
When I insert fileRef.path as the path, it tells me it can't find the file ... however I guess I need to get the files from S3 but I don't know how to do that. Which is the right way to pass the path or a filestream for an image stored with Meteor Files? Thanks!

Hello @marcoschwartz ,
Yes, it's easy, just grab a code from interceptDownload hook in AWS:S3 example.
You should end up with something like:
import stream from 'stream';
import S3 from 'aws-sdk/clients/s3'; // http://docs.aws.amazon.com/AWSJavaScriptSDK/latest/AWS/S3.html

const s3 = new S3({
  secretAccessKey: _SECRET_,
  accessKeyId: _KEY_,
  region: _REGION_,
  httpOptions: {
    timeout: 6000,
    agent: false
  }
});

let path;
const file = FilesCollection#findOne(_id);

if (file && file.versions && file.versions['original'] && file.versions['original'].meta && file.versions['original'].meta.pipePath) {
  path = file.versions['original'].meta.pipePath;
}

if (path) {
  const opts = {
    Bucket: _BUCKET_NAME_,
    Key: path
  };

  s3.getObject(opts, function (error) {
    if (error) {
      console.error(error);
    } else {
      const dataStream = new stream.PassThrough();
      dataStream.end(this.data.Body);
      // ---> dataStream is the stream you can pass to Instagram
    }
  });

Hello @marcoschwartz ,
Have you solved it?

Yeah that works now, thanks for the help!


Adding pagination feature on Issue Page

I added pagination feature on issue page, because The current issue page does not have pagination, making it difficult to navigate through a large number of issues efficiently. I would like to have pagination added to the issue page so that users can easily navigate through long lists of issues. It should include page numbers and options to jump to the first, last, next, and previous pages.

@azizlimonu Thankyou for the feature. Before adding a feature like this, we generally raise an issue first , get assigned to work on  it and the proceed for a PR. Please make sure you are following guidelines on this project. Also are you willing to work on making the pagination feature align with the theme of the project? If yes then only we can proceed on this.

yes, can you describe more spesific about the align with the theme? thank you.

yes, can you describe more spesific about the align with the theme? thank you.

For the dark theme , keep the button matching with a deep purple theme. Use the website as a reference.


Nested object support

I was looking into the documentation and tests and I didn't anything about nested objects.
Is it supported?
new Backbone.Model({ address: { street: 'street info', zipCode: 'zipcode'  }});

It would be nice if we could register as
bindings: { '#street': 'address.street', '#zipCode': 'address.zipCode' }

At the moment I am using set to override this on model, I am not entirely sure even if I should use since I didn't find it on the documentation only through the specs. On this case i just need model update and not view update.
bindings: {
  '#street':  {
     observe: 'address.street',
     set: function(attr, val, options, config) {
         var address = this.model.get('address');
         address.street = val;
          this.model.set('address', address);
         // changing the model address doesn't trigger event change because is same object
         this.model.trigger("change:address");
         this.model.trigger("change");
      }

Duplicate of #126
See also #200 and #221

@yethee on this case I am using stickit with rendrjs.
rendr is an isomorphic librabry and their models are an extension to backbone model, so that I am unable to use Backbone Nested or Backbone Deep. :/
Don you know any solution that doesn't include changing my base models? Using mixin maybe or just with stickit code?
thanks, for the help.

You can create a helper model for the struct address in the view scope and configure separate bindings for it. This is dirty solution and may break in future versions of Backbone, but should work with Backbone 1.1.2.
Backbone.View.extend({
  bindings: {
    ...
  },
  addressBindings: {
    '#street': 'street',
    '#zipCode': 'zipCode'
  },

  initialize: function(options) {
    this.model = options.model;

    this.address = new Backbone.Model();
    // Use reference on existing object to avoid managing events
    // for sync between models.
    this.address.attributes = this.model.get('address');

    this.listenTo(this.address, 'change', function(address, options) {
      this.model.trigger('change:address', this.model, address.attributes, options);
    });
  },

  render: function() {
    ...

    this.stickit();
    this.stickit(this.address, this.addressBindings);

    return this;
  }
});

I was already thinked on this solution, but I was trying to find a better solution without having to a model. Anyway it seems a better solution than the one I have now.
@yethee Thank for your support.


add filter to ignore '.git' in CollectionDataLoader.php

See issue 488.
In CollectionDataLoader.php, appended:
Line 63: return Str::startsWith($file->getFilename(), '_');
with filter:
Line 63: return Str::startsWith($file->getFilename(), '_') || Str::startsWith($file->getFilename(), '.git');

May not suit your needs, but this is working for me and seems to be in line with the thinking of @bakerkretzmar , so thought I'd share. Thanks to @joshhanley.

Thanks! Just want to make sure this isn't a breaking change, but then looks good to me!

@bakerkretzmar I can't imagine it would break anything. The only other consideration would be to do an exact match instead of a starts with. That would ensure it only filters out .git files.
But then I just did a test if you had a .gitignore inside a collections directory, it breaks in the same way "No matching collection item handler for file: .gitignore", so maybe leaving it as starts with .git would be best.


Update README.md to refer to 0.7.0

Very trivial, though this may be useful to new users.

Thanks!


cmake simple project not working: used wasimake

Hi All,
I have been trying to test wasimake to compile a CMAKE project.
It has juts two files:

 async_test.cpp
CMakeLists.txt

#include <iostream>
#include <string>
#include <chrono>
#include <thread>
#include <future>

using namespace std::chrono;

std::string fetchDataFromDB(std::string recvdData){
    std::this_thread::sleep_for(seconds(5));

    return "DB_"+recvdData;
}

std::string fetchDataFromFile(std::string recvdData){
    std::this_thread::sleep_for(seconds(5));
    return "File_"+recvdData;
}

int main(int, char**) {
    std::cout << "App Started" << std::endl;

    auto start = system_clock::now();
    auto getFromDB = std::async(std::launch::async,fetchDataFromDB,"Data1");
    auto getFromFile = std::async(std::launch::async,fetchDataFromFile,"Data2");

    auto dataFromDB = getFromDB.get();
    auto dataFromFile = getFromFile.get();

    auto end = system_clock::now();
    auto diff = duration_cast<seconds>(end-start).count();

    std::cout << "Total time taken ="<<diff <<" Seconds" << std::endl;
    std::cout << dataFromDB +" "+dataFromFile << std::endl;

    return 0;
}

cmake_minimum_required(VERSION 3.0.0)
project(asyn_test VERSION 0.1.0)

include(CTest)
enable_testing()

add_executable(asyn_test main.cpp)

target_link_libraries(${PROJECT_NAME} pthread)

set(CPACK_PROJECT_NAME ${PROJECT_NAME})
set(CPACK_PROJECT_VERSION ${PROJECT_VERSION})
include(CPack)

I create the build folder and then cd into it and run wasimake cmake ../ from it.
Now when I run this, it gives me these errors:
-- The C compiler identification is unknown
-- The CXX compiler identification is unknown
-- Check for working C compiler: /home/anurag/.wasienv/bin/wasicc
-- Check for working C compiler: /home/anurag/.wasienv/bin/wasicc -- broken
CMake Error at /usr/share/cmake-3.13/Modules/CMakeTestCCompiler.cmake:52 (message):
  The C compiler

    "/home/anurag/.wasienv/bin/wasicc"

  is not able to compile a simple test program.

  It fails with the following output:

    Change Dir: /home/anurag/ws/cpp/async_test/test1/build/CMakeFiles/CMakeTmp

    Run Build Command:"/usr/bin/make" "cmTC_bcfdf/fast"
    /usr/bin/make -f CMakeFiles/cmTC_bcfdf.dir/build.make CMakeFiles/cmTC_bcfdf.dir/build
    make[1]: Entering directory '/home/anurag/ws/cpp/async_test/test1/build/CMakeFiles/CMakeTmp'
    Building C object CMakeFiles/cmTC_bcfdf.dir/testCCompiler.c.obj
    /home/anurag/.wasienv/bin/wasicc    -o CMakeFiles/cmTC_bcfdf.dir/testCCompiler.c.obj   -c /home/anurag/ws/cpp/async_test/test1/build/CMakeFiles/CMakeTmp/testCCompiler.c
    Traceback (most recent call last):
      File "/home/anurag/.wasienv/bin/wasicc", line 10, in <module>
        sys.exit(wasicc())
      File "/home/anurag/.local/lib/python2.7/site-packages/wasienv/commands.py", line 20, in wasicc
        return execute(wasicc_run)
      File "/home/anurag/.local/lib/python2.7/site-packages/wasienv/tools.py", line 196, in execute
        sys.exit(f(args))
      File "/home/anurag/.local/lib/python2.7/site-packages/wasienv/wasicc.py", line 14, in run
        check_program(main_program)
      File "/home/anurag/.local/lib/python2.7/site-packages/wasienv/tools.py", line 61, in check_program
        raise Exception("The program {} was not found. Is the SDK installed?\nYou can install it via: wasienv install-sdk unstable".format(os.path.basename(cmd)))
    Exception: The program clang was not found. Is the SDK installed?
    You can install it via: wasienv install-sdk unstable
    make[1]: *** [CMakeFiles/cmTC_bcfdf.dir/build.make:66: CMakeFiles/cmTC_bcfdf.dir/testCCompiler.c.obj] Error 1
    make[1]: Leaving directory '/home/anurag/ws/cpp/async_test/test1/build/CMakeFiles/CMakeTmp'
    make: *** [Makefile:121: cmTC_bcfdf/fast] Error 2

  CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
  CMakeLists.txt:2 (project)

-- Configuring incomplete, errors occurred!
See also "/home/anurag/ws/cpp/async_test/test1/build/CMakeFiles/CMakeOutput.log".
See also "/home/anurag/ws/cpp/async_test/test1/build/CMakeFiles/CMakeError.log".

Tried even
wasienv install-sdk unstable
Output: The SDK 8 is already installed

How to solve this ?

Tried even the docker image
docker run --rm -v pwd:pwdwasienv/wasienv wasimake cmakepwd``
Got response:
CMake Error: CMake was unable to find a build program corresponding to "Unix Makefiles".  CMAKE_MAKE_PROGRAM is not set.  You probably need to select a different build tool.
-- Configuring incomplete, errors occurred!
See also "//CMakeFiles/CMakeOutput.log".

Problem with even using docker:
docker run --rm -v `pwd`:`pwd` wasienv/wasienv wasimake cmake `pwd`

Error on docker:
CMake Error: CMake was unable to find a build program corresponding to "Unix Makefiles".  CMAKE_MAKE_PROGRAM is not set.  You probably need to select a different build tool.
-- Configuring incomplete, errors occurred!
See also "//CMakeFiles/CMakeOutput.log".

Ok, solved the issue.
So for some reason if we dowload using the wasi sdk using wasienv install-sdk 8 , it will not have a clang++ in it.
So download WASI_SDK 8 from there release page and manually untar it in the gin folder.
Although my code is still not working as WASI-SDK do not supports thread at the moment.
Do any one has any hack on how to run a multithreaded WASM code ?

This is still a problem ... no matter what wasi-sdk is installed ... I also tried downloading from the release page and manually installing it as mentioned above ... but still have the same result
name@DESKTOP-G3DFJ6S:/mnt/d/WASM$ wasienv default-sdk 10
name@DESKTOP-G3DFJ6S:/mnt/d/WASM$ wasimake cmake --system-information
CMake Error at /usr/share/cmake-3.21/Modules/CMakeTestCCompiler.cmake:66 (message):
  The C compiler

    "/home/name/.local/bin/wasicc"

  is not able to compile a simple test program.

  It fails with the following output:

    Change Dir: /mnt/d/WASM/__cmake_systeminformation/CMakeFiles/CMakeTmp

    Run Build Command(s):/usr/bin/make -f Makefile cmTC_6a372/fast && /usr/bin/make  -f CMakeFiles/cmTC_6a372.dir/build.make CMakeFiles/cmTC_6a372.dir/build
    make[1]: Entering directory '/mnt/d/WASM/__cmake_systeminformation/CMakeFiles/CMakeTmp'
    Building C object CMakeFiles/cmTC_6a372.dir/testCCompiler.c.o
    /home/name/.local/bin/wasicc    -MD -MT CMakeFiles/cmTC_6a372.dir/testCCompiler.c.o -MF CMakeFiles/cmTC_6a372.dir/testCCompiler.c.o.d -o CMakeFiles/cmTC_6a372.dir/testCCompiler.c.o -c /mnt/d/WASM/__cmake_systeminformation/CMakeFiles/CMakeTmp/testCCompiler.c
    Linking C executable cmTC_6a372
    /usr/bin/cmake -E cmake_link_script CMakeFiles/cmTC_6a372.dir/link.txt --verbose=1
    /home/name/.local/bin/wasicc CMakeFiles/cmTC_6a372.dir/testCCompiler.c.o -o cmTC_6a372
    wasm-ld: error: unknown file type: CMakeFiles/cmTC_6a372.dir/testCCompiler.c.o
    clang-10: error: linker command failed with exit code 1 (use -v to see invocation)
    CMakeFiles/cmTC_6a372.dir/build.make:99: recipe for target 'cmTC_6a372' failed
    make[1]: *** [cmTC_6a372] Error 1
    make[1]: Leaving directory '/mnt/d/WASM/__cmake_systeminformation/CMakeFiles/CMakeTmp'
    Makefile:127: recipe for target 'cmTC_6a372/fast' failed
    make: *** [cmTC_6a372/fast] Error 2

  CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
  CMakeLists.txt:6 (project)

Error: --system-information failed on internal CMake!
name@DESKTOP-G3DFJ6S:/mnt/d/WASM$ wasienv install-sdk 7
name@DESKTOP-G3DFJ6S:/mnt/d/WASM$ wasienv default-sdk 7
name@DESKTOP-G3DFJ6S:/mnt/d/WASM$ wasimake cmake --system-information
CMake Error at /usr/share/cmake-3.21/Modules/CMakeTestCCompiler.cmake:66 (message):
  The C compiler

    "/home/name/.local/bin/wasicc"

  is not able to compile a simple test program.

  It fails with the following output:

    Change Dir: /mnt/d/WASM/__cmake_systeminformation/CMakeFiles/CMakeTmp

    Run Build Command(s):/usr/bin/make -f Makefile cmTC_0cc13/fast && /usr/bin/make  -f CMakeFiles/cmTC_0cc13.dir/build.make CMakeFiles/cmTC_0cc13.dir/build
    make[1]: Entering directory '/mnt/d/WASM/__cmake_systeminformation/CMakeFiles/CMakeTmp'
    Building C object CMakeFiles/cmTC_0cc13.dir/testCCompiler.c.o
    /home/name/.local/bin/wasicc    -MD -MT CMakeFiles/cmTC_0cc13.dir/testCCompiler.c.o -MF CMakeFiles/cmTC_0cc13.dir/testCCompiler.c.o.d -o CMakeFiles/cmTC_0cc13.dir/testCCompiler.c.o -c /mnt/d/WASM/__cmake_systeminformation/CMakeFiles/CMakeTmp/testCCompiler.c
    Linking C executable cmTC_0cc13
    /usr/bin/cmake -E cmake_link_script CMakeFiles/cmTC_0cc13.dir/link.txt --verbose=1
    /home/name/.local/bin/wasicc CMakeFiles/cmTC_0cc13.dir/testCCompiler.c.o -o cmTC_0cc13
    wasm-ld: error: unknown file type: CMakeFiles/cmTC_0cc13.dir/testCCompiler.c.o
    clang-10: error: linker command failed with exit code 1 (use -v to see invocation)
    CMakeFiles/cmTC_0cc13.dir/build.make:99: recipe for target 'cmTC_0cc13' failed
    make[1]: *** [cmTC_0cc13] Error 1
    make[1]: Leaving directory '/mnt/d/WASM/__cmake_systeminformation/CMakeFiles/CMakeTmp'
    Makefile:127: recipe for target 'cmTC_0cc13/fast' failed
    make: *** [cmTC_0cc13/fast] Error 2

  CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
  CMakeLists.txt:6 (project)

Error: --system-information failed on internal CMake!
name@DESKTOP-G3DFJ6S:/mnt/d/WASM$

Same thing happened in the docker container ...
wasimake doesn't work !!
name@:/mnt/d/$docker run -v `pwd`:`pwd` -it wasienv/wasienv bash

root@6347e6388120:/# cmake --version
cmake version 3.17.1

root@6347e6388120:/# wasimake cmake --system-information
CMake Error: CMake was unable to find a build program corresponding to "Unix Makefiles".  CMAKE_MAKE_PROGRAM is not set.

root@6347e6388120:/mnt/d/WASM/binaryen-version_101/build# apt install make
Reading package lists... Done

root@6347e6388120:/mnt/d/WASM/binaryen-version_101/build# make --version
GNU Make 4.2.1

root@6347e6388120:/mnt/d/WASM/binaryen-version_101/build# wasienv install-sdk unstable
The SDK 8 is already installed

root@6347e6388120:/mnt/d/WASM# wasimake cmake --system-information
CMake Error at /opt/cmake/share/cmake-3.17/Modules/CMakeTestCCompiler.cmake:60 (message):
  The C compiler

    "/root/.wasienv/bin/wasicc"

  is not able to compile a simple test program.

  It fails with the following output:

    Change Dir: /mnt/d/WASM/__cmake_systeminformation/CMakeFiles/CMakeTmp

    Run Build Command(s):/usr/bin/make cmTC_25876/fast && /usr/bin/make  -f CMakeFiles/cmTC_25876.dir/build.make CMakeFiles/cmTC_25876.dir/build
    make[1]: Entering directory '/mnt/d/WASM/__cmake_systeminformation/CMakeFiles/CMakeTmp'
    Building C object CMakeFiles/cmTC_25876.dir/testCCompiler.c.o
    /root/.wasienv/bin/wasicc    -o CMakeFiles/cmTC_25876.dir/testCCompiler.c.o   -c /mnt/d/WASM/__cmake_systeminformation/CMakeFiles/CMakeTmp/testCCompiler.c
    In file included from <built-in>:1:
    /root/.local/lib/python2.7/site-packages/wasienv/stubs/preamble.h:30:9: warning: 'ESHUTDOWN' macro redefined [-Wmacro-redefined]
    #define ESHUTDOWN 0
            ^
    /root/.local/lib/python2.7/site-packages/wasienv/stubs/preamble.h:28:9: note: previous definition is here
    #define ESHUTDOWN 1
            ^
    /root/.local/lib/python2.7/site-packages/wasienv/stubs/preamble.h:116:62: warning: declaration of 'struct sigaction' will not be visible outside of this function [-Wvisibility]
    __attribute__((weak)) int sigaction(int signum, const struct sigaction *act,
                                                                 ^
    /root/.local/lib/python2.7/site-packages/wasienv/stubs/preamble.h:125:1: warning: control reaches end of non-void function [-Wreturn-type]
    }
    ^
    /root/.local/lib/python2.7/site-packages/wasienv/stubs/preamble.h:134:1: warning: control reaches end of non-void function [-Wreturn-type]
    }
    ^
    /root/.local/lib/python2.7/site-packages/wasienv/stubs/preamble.h:138:1: warning: control reaches end of non-void function [-Wreturn-type]
    }
    ^
    5 warnings generated.
    Linking C executable cmTC_25876
    /opt/cmake/bin/cmake -E cmake_link_script CMakeFiles/cmTC_25876.dir/link.txt --verbose=1
    /root/.wasienv/bin/wasicc      CMakeFiles/cmTC_25876.dir/testCCompiler.c.o  -o cmTC_25876
    wasm-ld: error: unknown file type: CMakeFiles/cmTC_25876.dir/testCCompiler.c.o
    clang-9: error: linker command failed with exit code 1 (use -v to see invocation)
    make[1]: *** [CMakeFiles/cmTC_25876.dir/build.make:107: cmTC_25876] Error 1
    make[1]: Leaving directory '/mnt/d/WASM/__cmake_systeminformation/CMakeFiles/CMakeTmp'
    make: *** [Makefile:141: cmTC_25876/fast] Error 2

  CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
  CMakeLists.txt:6 (project)

Error: --system-information failed on internal CMake!

I started off with a fresh Ubuntu 20.04.2 LTS install ...
successfully installed wasienv using the install script
successfully performed wasimake cmake --system-information without errors
successfully performed wasimake cmake -DBUILD_TESTS=OFF .. from the build folder of a project (wabt)
successfully performed wasimake make wasm-strip and produced a functioning wasm file

So, I guess the status of the build platform has a lot to do with having success with the wasienv project.
I'd consider this (#46) issue and issue #48 closed ...

the problem with the differences between clang-10 and the higher clangs seems to be linked to Ubuntu missing system dependency libtinfo.so.5 ...
name@DESKTOP-G3DFJ6S:/mnt/d/WASM/tar-1.34$ wasicc --help
/home/name/.local/lib/python3.8/site-packages/wasienv-storage/sdks/11/wasi-sdk-11.0/bin/clang: error while loading shared libraries:
libtinfo.so.5: cannot open shared object file: No such file or directory

the solution is to install it ...
sudo apt install libtinfo5


Good idea. I wonder if the data is local? Can it be synchronized across devices?

Yes, currently the data is stored locally in the browser. This is indeed a pain point, and I'm also thinking about how to make a synchronization feature, such as leveraging the capabilities of some database platforms? Let users create their own libraries and fill in the api key?Or maybe I deploy a server myself to supply the users? I haven't decided yet, if you have any good ideas you can also put forward!


Native Notebooks - navigation through notebook subtle

Steps to reproduce:

Open notebook outline
Click through the cell headings and see where the selected cell is

When I click through the outline I have a tough time finding where I actually am in the notebook.

If the cell I would like to go to is out of view, when I click it on the outline it doesn't auto-scroll such that the cell is at the top of the notebook canvas. It is somewhere in the middle of the canvas.
When I look for where it is in the canvas, it took me a long time to realize the only indicator was the run button showing up. Even after realizing that was what to look for, it is a bit small and subtle. I was expecting to see the selected state (two blue lines around the cell)

@rebornix this is because we use focus-within here:
https://github.com/microsoft/vscode/blob/69a6e6ac93745990b1e5f41ca0718228efdc7918/src/vs/workbench/contrib/notebook/browser/media/notebook.css#L244-L248
So we either should show an alternate inactive selection or always show the focus (we do this in the editor with outline in other files).

@rebornix this is because we use focus-within here:
https://github.com/microsoft/vscode/blob/69a6e6ac93745990b1e5f41ca0718228efdc7918/src/vs/workbench/contrib/notebook/browser/media/notebook.css#L244-L248
So we either should show an alternate inactive selection or always show the focus (we do this in the editor with outline in other files).

We've introduced a new color token, notebook.inactiveFocus, that will appear when a cell is focused but then focus moves away from the notebook. Helpful especially around navigating via the outline view. This color currently defaults to the notebook selected cell border color.
https://user-images.githubusercontent.com/35271042/106056203-45af8a80-60a3-11eb-87d0-fc3f30c95c40.mov

We've introduced a new color token, notebook.inactiveFocus, that will appear when a cell is focused but then focus moves away from the notebook. Helpful especially around navigating via the outline view. This color currently defaults to the notebook selected cell border color.
https://user-images.githubusercontent.com/35271042/106056203-45af8a80-60a3-11eb-87d0-fc3f30c95c40.mov


The plugin does not download the correct micromamba executable for arm/aarch64

Environment
Operating system: Linux rpi5-2CCF6726E2E4 6.6.31+rpt-rpi-2712 #1 SMP PREEMPT Debian 1:6.6.31-1+rpt1 (2024-05-29) aarch64 GNU/Linux
Prerequisite
Vscode remote SSH on Raspberry Pi 5
Install vscode-micromamba on remote
Empty repo
Steps to Reproduce
Micromamba: create environment
Relevant Debug Info
1: Syntax error: Unterminated quoted string
Expected Behavior
Micromamba environment created successfully
Actual Behavior
Create environment fails with error

The plugin seems to download micromamba executable for the wrong architecture. If I download manually micromamaba in .micromamba folder, and invoke environment creation, it completes successfully and works as expected.
For reference, I downloaded manually the release for aarch64: https://github.com/mamba-org/micromamba-releases/releases/download/1.5.9-1/micromamba-linux-aarch64
Officially there is also this script available to automate the download and install of micromamba for the correct architecture, that can be invoked like this:
"${SHELL}" <(curl -L https://micro.mamba.pm/install.sh)

Please check v0.1.24 and report

Please check v0.1.24 and report

Thank you, I can confirm it works, the correct version of micromamba was downloaded by the plugin on environment creation:
.micromamba/micromamba: ELF 64-bit LSB pie executable, ARM aarch64, version 1 (GNU/Linux), dynamically linked, interpreter /lib/ld-linux-aarch64.so.1, for GNU/Linux 3.7.0, stripped


Develop custom forms for each pathway

Feature Request
Dynamically display metadata fields and their validation by pathway.

The General pathway should maintain the current deposit process. The Scholarly Works pathway and the Data & Code pathway are described in the following issues:
#1467
#1469

Closing this because it is essentially a sub-epic of https://github.com/psu-libraries/scholarsphere/issues/1153, and the form customization work from the current backlog has been completed.


[Mukunde Brisa] - Sage - The future of ReFi and its potential impact

Thanks for submitting this tutorial for Celo Sage
Before you hit "Create Pull Request", make sure you have read the Tutorial Contribution Guidelines and reviewed the Sample Tutorial Structure.
Please confirm the following before submitting your tutorial for review -

[x] My tutorial follows the Tutorial Contribution Guidelines.
[x] The title of the Pull Request matches the title of my tutorial.
[x] I've followed the tutorial template structure.
[x] I've proofread the tutorial for spelling and grammar mistakes.
[x] I've thoroughly explained the code, with inline code comments and text explanations.
[x] I've run a linter & prettifier to format my code blocks.
[x] I've run the tutorial to test it and the code works.

NOTE: We cannot review your Pull Request if it does not meet the standard outlined above.
Reviewers - @joenyzio & @viral-sangani

Great @Ishanpathak1. Let me work on it

made the changes @Ishanpathak1


feat: cleanup tooling baseline

Motivation

If SKIP_DOCKER is set then you can never run just tools; so the requirement is to remove the use of docker to run yq
Since we are doing that, then we should reduce complexity by install oneshot tools like just in bootstrap.sh rather than adding the prebuilt repo unnecessarily.
Changes

remove prebuilt_mpr as a repo
install 'just' via gh-release-install
install 'yq' in justfile via gh-release-install
add systemctl mask or binfmt (for wslview)
add shellcheck action
wslutilities repo back behind a debian test

Testing

I tested with Debian, since my primary is Ubuntu (so I feel happier deleting Debian).

wsl -t Debian
wsl --unregister Debian
wsl --install Debian

Minimal bootstrapping :

sudo apt-get update && sudo apt-get -y install vim nfs-common unison jq direnv zip unzip net-tools git rcm
echo "$USER ALL=(ALL:ALL) NOPASSWD:ALL" | sudo tee -a /etc/sudoers.d/lenient

Testing bootstrap.sh

We install docker, because this modifies /etc/wsl.conf to include systemd on boot.

# ... clone this repo on this branch
./bootstrap.sh repos
./bootstrap.sh baseline
...
Created symlink /etc/systemd/system/systemd-binfmt.service → /dev/null.

>>> You might need to restart WSL for binfmt changes to take effect

Do the wsl --shutdown restart dance for systemd
Testing justfile etc.

This also tests the systemctl mask

export PATH=$PATH:~/.local/bin
just prepare
# select login with a web browser and it should transparently open your browser...
# no binfmt nonsense now.
just tools

Since I have 'ubuntu' installed, I realised that I can wsl --install Ubuntu-22.04 to get a separate WSL instance that is ubuntu.
The test sequence above also works.

✅ wslutilities repo does not exist (since !debian)
✅ wslu is still installed (via ubuntus own repos)
✅ binfmt isn't interfering with wslview


[QUESTION] How would you manage authentication/authorization?

Hi!
I really appreciate your work on this project.
I would like to ask you something, how would you handle authentication in this type of architecture?
Like an IdentityServer or a microservice delegated to authentication (implementing also AspNetIdentity Core)?
Thanks in advance

Hi,
For authentication, I would most likely use an Identity provider eg. Azure AD, AWS Cognito, or Auth0.
If you want to roll out your own I would look into IdentityServer from Microsoft. I would, however, recommend using one of the above.
Once the identity provider is up and running the token is validated in the ApiGateway and/or each microservice.

So should the identity server be integrated in all the microservice or only the  Gateway ???
If its only the Gateway,  then if an attacker manage to find the other microservices endpoint. then they should be able to access the resources without been authenticated.
Please @madslundt can you help as soon as possible ?
I already have an identity server setup, how do I authenticate users before accessing the resources.

So should the identity server be integrated in all the microservice or only the Gateway ???
It depends, if you are wanting to program logic based on the user being "authorised" to do something. If this is the case then you will need to handle authentication in the gateway and authorisation in the service.

If you are looking to control this based on urls e.g. not allowing people to create movies post -> /movies, then you can just add the authentication and authorisation in the gateway itself.

If its only the Gateway, then if an attacker manage to find the other microservices endpoint. then they should be able to access the resources without been authenticated.

Not true, the gateway is the only thing that is front facing, all other services are inaccessible to the outside world. If you would like an analogy, the gateway is a form of firewall if you like.

Thanks for the clarification.


Wait for fonts to be loaded in the bal-app

cy.document()
// https://developer.mozilla.org/en-US/docs/Web/API/FontFaceSet/ready
// The promise will only resolve once the document has completed loading fonts,
// layout operations are completed, and no further font loads are needed.
.then(document => document.fonts.ready)

discuss with ibm I x


Chat page

Project done !
added chat page, refactored the whole thing.
used tanstack query to prefetch messages and create mutation.
Chat data is persisted in postgres and response is streamed in using server sent events.

Move to deployment now


Set splat for DenseElementAttr when all dimensions are 1

This patch fixes a bug in constant propagation when creating DenseElementAttr for tensors whose all dimension sizes are 1.
Signed-off-by: Tung D. Le<EMAIL_ADDRESS>

Jenkins Linux ppc64le Build #1264 [push] set splat when all dimen... started at 09:43

Jenkins Linux s390x Build #2154 [push] set splat when all dimen... started at 09:42

Jenkins Linux ppc64le Build #1264 [push] set splat when all dimen... passed after 1 hr 7 min


Add auth

Required if the user has setup auth on a public box
https://machinebox.io/docs/machine-box-apis#basic-authentication

Need to add tests for for request authentication and failure of authentication. Requires checking the logs, see https://docs.pytest.org/en/latest/logging.html and example at https://github.com/home-assistant/home-assistant/blob/d9cf8fcfe82105b53aa6b4ee58a0e9633a700ca3/tests/helpers/test_entity_platform.py#L211


Remove Scene.fxaaOrderIndependentTranslucency

It was deprecated in 1.10.

@bagnell can you do this soon for 1.11?


Test oracles voting changes and updating the pool contract on testnet

Scenario:

bootstrap an aur pool with 2 oracles (oracle and ballot tokens);
run the pool for a while;
vote to change the epoch length;
update the refresh contract;

Closing due to #176 and to be continued in #177


Flutter Framework size regression

It appears a recent roll of the framework caused a size regression, about 500kb in flutter_framework_uncompressed_bytes in hello_world_macos__compile
Skia perf link:
https://flutter-flutter-perf.skia.org/e/?begin=1681270451&end=1681500982&keys=X4129c1b014a53ca2bc7747c174d61ea5&requestType=0&selected=commit%3D34371%26name%3D%252Carch%253Dintel%252Cbranch%253Dmaster%252Cconfig%253Ddefault%252Cdevice_type%253Dnone%252Cdevice_version%253Dnone%252Chost_type%253Dmac%252Csub_result%253Dflutter_framework_uncompressed_bytes%252Ctest%253Dhello_world_macos__compile%252C&xbaroffset=34370
Seems to have started with this engine roll: https://github.com/flutter/flutter/pull/124841

FYI @cbracken

This started with https://github.com/flutter/engine/commit/f09106065eaba080fa81dee7caba3a118faadc9a
That commit is bringing various Impeller code into the macOS binary.
Note that macOS release builds are not currently enabling LTO.

The increase in compressed size was ~80KB: https://flutter-flutter-perf.skia.org/e/?queries=test%3Dhello_world_macos__compile&selected=commit%3D34371%26name%3D%252Carch%253Dintel%252Cbranch%253Dmaster%252Cconfig%253Ddefault%252Cdevice_type%253Dnone%252Cdevice_version%253Dnone%252Chost_type%253Dmac%252Csub_result%253Drelease_size_bytes%252Ctest%253Dhello_world_macos__compile%252C


Orchestrate TCP connections using wait-on

Courtesy of the tip from @orefalo, adds the wait-on package to most of the foundation examples so that all services and the gateway can reliably run/restarted together in a single process without port races. Huge improvement to the later examples that start using port retries (still have a few to update).
Also cleans up nodemon parameterization so that watches only restart in response to changes in their respective subservices.

;-)

;-)


[posix] de-prioritize the added external routes

In https://github.com/openthread/openthread/pull/6782, we add routes in linux kernel to enhance border routing feature. However, the current priority of the added routes too high so it may get preferred over routes learned from RA. Hence we increase the metric to de-prioritize the added routes.

May need to check how Linux kernel assign metrics for routes, so that we can choose a more suitable metric value.


[Feature] Gradient Card Component

Description
Implement a stylish and interactive gradient card component using HTML, CSS, and JavaScript. This component showcases structured content with a vibrant gradient background, making it suitable for highlighting featured items or information on your website. Customize the card's appearance and content easily to suit different design requirements and enhance user engagement with intuitive hover effects.
Screenshots
No response
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

@thevijayshankersharma
Please attach some img to show what u want to add.

@saurabhsingh720 Here is the image

@thevijayshankersharma go ahead
I am assigning u
As admin is busy for some days
Do your task and provide your PR asap.
Then admin merge it with the proper labels .


Add Github action to manage to-triage label

Add triage label to issues without any labels
Remove triage label from issue closed as "not planned"
Add triage label to completed issue without milestone

Can be tested here

@bot-gradle test and merge

I've triggered a build for you.


Unsafe website link listed for wintersmith-tag example

The website listed on https://github.com/jnordberg/wintersmith/wiki/Plugins as an example for the wintersmith-tag plugin seems to lead to an unsafe webpage. It repeatedly spammed me with alerts.

Thanks for letting me know and feel free to edit the page, it’s a wiki :)
On 18 May 2016 at 17:26:48, Tyler Funk<EMAIL_ADDRESS>wrote:
The website listed on https://github.com/jnordberg/wintersmith/wiki/Plugins as an example for the wintersmith-tag plugin seems to lead to an unsafe webpage. It repeatedly spammed me with alerts.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub

Oh, ok. Edit made, thanks :)


Enable RISC-V platform

Open autoware with open RiSC-V chip may be a good idea.

Could you write a bit more detailed description?
For example:

How can we support it?
What ECUs can we use?
etc.

Also, since this isn't a bug but just an idea, I'll convert this to a discussion.


add leaflet.magicMarker plugin

Adding magical animation effect to a marker while loading.

@Sylvenas, I'm slightly disappointed by this, because whenever I read "magic" I expect sparkles and unicorns around!
:unicorn: :rainbow: :sparkles:


[ServiceBus] Fix forward_to and forward_dead_lettered_messages_to functionality for subscriptions

Fix forward_to and forward_dead_lettered_messages_to functionality for subscriptions by adding bearer token headers when needed.
Adds tests for this scenario as well.
changelog snippet.
fixes #15543

/azp run python - servicebus - tests

/azp run python - servicebus - tests

/azp run python - servicebus - tests

/azp run python - servicebus - tests


Customiser options disappear if too many items in section

Issue description:
If there are enough controls within an accordion section (about 95% of the height) of the customiser, the content disappears. If I zoom out on the browser, it'll reappear.
This has been addressed in the dev branch (I've tested and this problem does not occur) but I want to know if there is anything I can do to fix it from within my theme. I don't want to package the plugin with my theme, I'd rather the user keep it updated via the WP site but currently this issue is rendering parts of my theme customizer unusable.

@macaroons Most likely this is a CSS issue...
A lot has changed in the develop version so there's really no way for me to know exactly what goes on in your case as I was unable to replicate this locally.
If you can't debug this on your theme and find a quick fix, testing the develop version and reporting any bugs you find would help us get things fixed sooner and release version 3.0 which will fix - among other things - your issue as well


Captchas are missing functionality.

For times when a captcha is needed, reddit  allows one to get a new captcha (presumably if the original is unreadable by the user). Unless I'm mistaken, this functionality does not exist in prawns. If this is the case, a new captcha should be triggered by entering a special string such as a simple 1 or an empty string altogether.
Also, if I'm not mistaken, if a captcha is incorrect an error is raised and prompts the exit of the application due to this. However, the API allows one to indefinitely get a newer (potentially, somehow easier to read) captcha if the captcha is incorrect, just like in-browser functionality.

If I'm not mistaken PRAW's default behavior will prompt indefinitely while invalid captchas are sent. Exceptions are only raised if explicitly asked for. Thus one need only type an invalid captcha in order to get a new one. Please reopen if I am mistaken. Thanks for filing.

@bboe you are correct, but, currently the captchas being repeatedly prompted are prompted by the BAD_CAPTCHA caveat here, which IIRC trips off reddit's system differently than getting a new captcha via just getting the new iden of a new captcha and requesting /captcha/newiden (basically, the difference is the server recording the fact that the user made a mistake or simply can't read, and that may be important, especially if say, failing too many captchas will make the admins ratelimit you and see you as a malicious bot vs not being able to read that many captchas will maybe make the admins keep an eye on you)

I understand your reasoning here, however, I think the surface area for making that many mistakes is incredibly small and the effects are speculative. Nevertheless if you really want a standalone endpoint for new captchas I would be happy to include it so long as it's not integrated into the existing behavior.


Add link to the Linode and Hashnode on footer area

The respective links to be added are https://www.linode.com/ and https://hashnode.com/

Closed by #4


Sense rule element's selector can't be resolved

In order to implement something like Familiar of Keen Senses, it seems like we'd need the Sense RE's selector to accept a flag set by a Choice Set, but it looks that isn't currently possible. Would it be possible to make that selector resolvable?

Turns out this wasn't really necessary to implement what I wanted, and I can't think of another use case.


AttributeError: 'Django' object has no attribute '_instance_name' when running as a standalone Python script directly

Hi, great project I love it!
Unfortunately, I am unable to get nanodjango running as a standalone script.
The documentation says:

You don’t need to use the nanodjango command - you can call app.run() from the bottom of your script, eg:

from nanodjango import Django
app = Django()
...
if __name__ == "__main__":
    app.run()

Copying this code into a file, however, gives me the following error:
% python3 app.py
Traceback (most recent call last):
  File "/Users/j0hnc/etc/code/python/django/nanodjango/app.py", line 5, in <module>
    app.run()
  File "/Users/j0hnc/etc/code/python/django/nanodjango/.venv/lib/python3.12/site-packages/nanodjango/app.py", line 372, in run
    self._prepare(is_prod=False)
  File "/Users/j0hnc/etc/code/python/django/nanodjango/.venv/lib/python3.12/site-packages/nanodjango/app.py", line 295, in _prepare
    or getattr(sys.modules["__main__"], self._instance_name) != self
                                        ^^^^^^^^^^^^^^^^^^^
AttributeError: 'Django' object has no attribute '_instance_name'

Unsurprisingly, I get the same error when I try to use pipx via pipx run app.py on the code from the documentation:
# /// script
# dependencies = ["nanodjango"]
# ///
from nanodjango import Django
app = Django()
...
if __name__ == "__main__":
    app.run()

https://github.com/radiac/nanodjango/issues/21

Thanks for reporting this! This looks like the same underlying issue as #21, should be fixed in the next release.


Port forwarding issue?!

Hello, I'm not sure if this is an issue of the Remote-Container feature or a networking issue of my local configuration. Sorry beforehand if the problem is mine (I currently can't understand it very well):
I had to run the container manually, with the -p 9000:9000 instead of the automatic-p <IP_ADDRESS>:9000:9000. I couldn't reach the flask server inside the container so I had to make it available to all interfaces... I'm not sure why.
Perhaps someone can explain to me why this happened. Thank you

@qubitron @chuxel can you explain what happens and can the sample be changed to avoid this problem?

@egamma The port is published via the appPort property, so it's unclear why this would be a problem.  If there is something going on here' it's likly either a docker issue or something we'd need to do in VS Code Remote - Containers.  It also could be OS specific since I'm not seeing it.  Let's get a bit more info, but I suspect we should move this issue to vscode-remote-release since it's likely not a sample problem.
@nffdiogosilva What OS are you running?  How many network adapters do you have? Is there anything special or interesting about the network you are on?

@nffdiogosilva can you clarify what you mean by running the container manually? Were you debugging the app or running the app from the command line?
Note that when running the app you need to tell it to bind to host <IP_ADDRESS> instead of <IP_ADDRESS> inside the container, the launch.json file does this for you when debugging by passing the --host flag to flask run.

@egamma The port is published via the appPort property, so it's unclear why this would be a problem. If there is something going on here' it's likly either a docker issue or something we'd need to do in VS Code Remote - Containers. It also could be OS specific since I'm not seeing it. Let's get a bit more info, but I suspect we should move this issue to vscode-remote-release since it's likely not a sample problem.
@nffdiogosilva What OS are you running? How many network adapters do you have? Is there anything special or interesting about the network you are on?

I'm currently using MAC OS X: 10.14.4. Regarding my current network interfaces:
❯ ifconfig                                                                                13:32:21
lo0: flags=8049<UP,LOOPBACK,RUNNING,MULTICAST> mtu 16384
        options=1203<RXCSUM,TXCSUM,TXSTATUS,SW_TIMESTAMP>
        inet <IP_ADDRESS> netmask 0xff000000
        inet6 ::1 prefixlen 128
        inet6 fe80::1%lo0 prefixlen 64 scopeid 0x1
        nd6 options=201<PERFORMNUD,DAD>
gif0: flags=8010<POINTOPOINT,MULTICAST> mtu 1280
stf0: flags=0<> mtu 1280
XHC20: flags=0<> mtu 0
en0: flags=8863<UP,BROADCAST,SMART,RUNNING,SIMPLEX,MULTICAST> mtu 1500
        ether 60:03:08:98:2d:b8
        inet <IP_ADDRESS> netmask 0xfffffc00 broadcast <IP_ADDRESS>
        media: autoselect
        status: active
p2p0: flags=8843<UP,BROADCAST,RUNNING,SIMPLEX,MULTICAST> mtu 2304
        ether 02:03:08:98:2d:b8
        media: autoselect
        status: inactive
awdl0: flags=8943<UP,BROADCAST,RUNNING,PROMISC,SIMPLEX,MULTICAST> mtu 1484
        ether 8a:85:fd:b7:1b:a8
        inet6 fe80::8885:fdff:feb7:1ba8%awdl0 prefixlen 64 scopeid 0x7
        nd6 options=201<PERFORMNUD,DAD>
        media: autoselect
        status: active
en1: flags=8963<UP,BROADCAST,SMART,RUNNING,PROMISC,SIMPLEX,MULTICAST> mtu 1500
        options=60<TSO4,TSO6>
        ether 72:00:01:85:0c:e0
        media: autoselect <full-duplex>
        status: inactive
en2: flags=8963<UP,BROADCAST,SMART,RUNNING,PROMISC,SIMPLEX,MULTICAST> mtu 1500
        options=60<TSO4,TSO6>
        ether 72:00:01:85:0c:e1
        media: autoselect <full-duplex>
        status: inactive
bridge0: flags=8822<BROADCAST,SMART,SIMPLEX,MULTICAST> mtu 1500
        options=63<RXCSUM,TXCSUM,TSO4,TSO6>
        ether 72:00:01:85:0c:e0
        Configuration:
                id 0:0:0:0:0:0 priority 0 hellotime 0 fwddelay 0
                maxage 0 holdcnt 0 proto stp maxaddr 100 timeout 1200
                root id 0:0:0:0:0:0 priority 0 ifcost 0 port 0
                ipfilter disabled flags 0x2
        member: en1 flags=3<LEARNING,DISCOVER>
                ifmaxaddr 0 port 8 priority 0 path cost 0
        member: en2 flags=3<LEARNING,DISCOVER>
                ifmaxaddr 0 port 9 priority 0 path cost 0
        media: <unknown type>
        status: inactive
utun0: flags=8051<UP,POINTOPOINT,RUNNING,MULTICAST> mtu 2000
        inet6 fe80::9302:1466:5c0:948d%utun0 prefixlen 64 scopeid 0xb
        nd6 options=201<PERFORMNUD,DAD>
utun1: flags=8051<UP,POINTOPOINT,RUNNING,MULTICAST> mtu 1380
        inet6 fe80::c0:5be0:4e8d:c2c2%utun1 prefixlen 64 scopeid 0xc
        nd6 options=201<PERFORMNUD,DAD>

@nffdiogosilva can you clarify what you mean by running the container manually? Were you debugging the app or running the app from the command line?

Instead of running the "docker container run" automatically, through the vs code-insiders extension,
I ran it manually from the command line with -p 9000:9000. THat's the only option that I've changed.

Note that when running the app you need to tell it to bind to host <IP_ADDRESS> instead of <IP_ADDRESS> inside the container, the launch.json file does this for you when debugging by passing the --host flag to flask run.

I'm not 100% sure now, but I'm almost certain that it was being bidden to <IP_ADDRESS> and not <IP_ADDRESS> (inside the container, as you say). I was trying to replicate the issue now, but I'm getting what it seems to be another error:
Setting up container for folder: /Users/ichigo/Developer/personal/vscode/vscode-remote-try-python
Run: docker exec c4e42700b33ccffee7baac995938f662d0ca6485a577adb7726def08ab8edb79 /bin/sh -c (cat /etc/os-release || cat /usr/lib/os-release) 2>/dev/null
Error response from daemon: Container c4e42700b33ccffee7baac995938f662d0ca6485a577adb7726def08ab8edb79 is not running
Run: docker exec c4e42700b33ccffee7baac995938f662d0ca6485a577adb7726def08ab8edb79 test -d /root/.vscode-remote/bin/daf71423252a707b8e396e8afa8102b717f8213b
Error response from daemon: Container c4e42700b33ccffee7baac995938f662d0ca6485a577adb7726def08ab8edb79 is not running
Installing VS Code Server for commit daf71423252a707b8e396e8afa8102b717f8213b
Run: docker exec c4e42700b33ccffee7baac995938f662d0ca6485a577adb7726def08ab8edb79 mkdir -p /root/.vscode-remote/bin/daf71423252a707b8e396e8afa8102b717f8213b_1557238588992
Error response from daemon: Container c4e42700b33ccffee7baac995938f662d0ca6485a577adb7726def08ab8edb79 is not running
Command failed: docker exec c4e42700b33ccffee7baac995938f662d0ca6485a577adb7726def08ab8edb79 mkdir -p /root/.vscode-remote/bin/daf71423252a707b8e396e8afa8102b717f8213b_1557238588992

It's like trying to exec the container directly, without running it first...

@nffdiogosilva There have been a number of improvements to the extension since this was first rased and we couldn't repro - are you still seeing this?


Websocket API playState renamed

The websocket channel playState seems to have been replaced after the initial connect with a channel named "state" that performs the same function and is not listed on the API.

@LoKaDah This appears to be a regression from an attempt at simplifying the WebSocket API code.  I'm going to split this issue into multiple parts and move it to a super.


Favorite Item should have gold outline on hover

Description
It would be nice to see some transition on the star when we hover
To Reproduce
Steps to reproduce the behavior:

Go to https://yahoo.github.io/navi/#/reports/1/view
Hover on the star button
The color is still dark gray

Expected behavior
The star outline should turn into the @favorite-gold color here https://github.com/yahoo/navi/blob/ca8bef67c77c11fe8519f9175c839d2bdd46b2bf/packages/reports/app/styles/navi-reports/components/favorite-item.less#L6
Screenshots

I'd like to take a shot at this.

Sounds good! Let me know if you need help

I'm trying to create a PR from my forked repo, but it showing previous file changes from my last PR. is that normal?

Yea so what happened here was that the first two commits from your original PR were squashed into one, and then that was applied to the master branch of the yahoo/navi repo

But in your repo, directdetour/navi, your master branch is separate from the yahoo/navi master branch so it doesn't know the PR got merged. There's different ways of updating your branch to stay in sync with ours, but since you've added on another commit we'll skip the updating part.
Essentially you'll have to

add a remote (just once) to our repo,git remote add upstream https://github.com/yahoo/navi.git

fetch the changes from our repogit fetch upstream

pull in the changes

If you want to start fresh and throw away your changes (like after you merge a PR)

git reset --hard upstream/master

It can get complicated if you try to work directly on master since every time we merge a PR it will add changes to our master and your master branch will be out of date. It might help to look into some git intro so you can make each pr from it's own branch and just keep your master branch in sync with ours

my local fork seems messed up and I wasn't able to push to my forked repo. I ended up making the previous change (with extra spaces) and the latest commit using the github web UI. I guess I could use some tips, once my PR is merged, to rebase(?) my fork and start using branching properly.

Oh yea it looks like you got stuck on part of our commit check (first it fixes up small style issues if it finds any in the files you changed, then it checks the format of your commit message to make sure it follows our standard). In this situation it doesn't like that cleanup starts lower cased
If you changed it to Update: Cleanup spaces ... then it would actually do the commit and you would be able to push it


[Feature]: Add a flag to add the namespace to the backup cluster name

Is there an existing issue already for this feature request/idea?

[X] I have searched for an existing issue, and could not find anything. I believe this is a new feature request to be evaluated.

What problem is this feature going to solve? Why should it be added?
I have multiple clusters that all have the same name but exist within different namespaces. We do this becuase we have multiple development environments for testing. For instance I have dev1, dev2, dev3 as the name space, all of them have a cluster called "api-data" within them.
We also make use of ArgoCD and Kustomize to provision these resources in K8s.
When provisioning the backup I currently have to create multiple files within kustomize that define the backup location that basically adds the name space as a unique identifier to the path. This results in a massive number of files that basically all contain the following:
spec:
  backup:
    barmanObjectStore:
      destinationPath: s3://psql-backup-storage/devX

I'd like to have a single bucket that contains all the backups. I'd also like to not have to specify a unique path in many files. The uniqueness of the cluster is actually the namespace + the cluster name. Therefore I would like to have a flag that tells the cluster to add the cluster name and namespace to the path when creating the files.
Something like this:
spec:
  backup:
    barmanObjectStore:
      includeNamespace: true
      destinationPath: s3://psql-backup-storage

With the result being s3://psql-backup-storage/<namespace>/<cluster>.
Describe the solution you'd like
I'd like a flag in the barman section that allows me to have the namespace be part of the generated path that is used to write files to the object storage.
Describe alternatives you've considered
Write the base paths manually in kustomize, which generates many many files with very little difference.
Additional context
No response
Backport?
Yes
Are you willing to actively contribute to this feature?
Yes
Code of Conduct

[X] I agree to follow this project's Code of Conduct

I have attempted to add this feature in with this PR, as you can see in the timeline: #5228


How to send a command to the server?

I start the server. I want send commands to this server - curl <IP_ADDRESS>:8081/?command=store.
How can I do it?
$loop = \React\EventLoop\Factory::create();
$socket = new \React\Socket\Server('<IP_ADDRESS>:8081', $loop);
$socket = new \React\Socket\LimitingServer($socket, null);
$socket->on('connection', function(\React\Socket\ConnectionInterface $connection) use ($socket){
       $connection->on('data', function($data) use ($connection, $socket){
         /*if(command == 'store')
               foreach ($socket->getConnections() as $connection) {
	        $connection->write('store');
	       }
         */

       });
});

$loop->run();

You probably want to have a look at https://github.com/reactphp/http instead of the socket component directly.

I believe this has been answered, so I'm closing this for now. Please come back with more details if this problem persists and we can reopen this :+1:

@kelunik  I tried using reactphp/https. But I did not understand how to connect to the existing server. How do I connect to the existing server?
$loop = \React\EventLoop\Factory::create();
$socket = new \React\Socket\Server('<IP_ADDRESS>:8081', $loop);
$server = new \React\Http\Server([$this, 'onHttpRequest']);
$server->listen($socket);

@wolrd Have you seen https://github.com/reactphp/http#quickstart-example and https://github.com/reactphp/http/tree/master/examples ? This is usually a good starting point to get an idea about how this can be used.

@clue yes.   I run 2 scripts and get an error:
Failed to listen on "tcp://<IP_ADDRESS>:8081": Address already in use
I did not understand how to connect to the existing server.

I have some workres with EventLoop. I need to send them commands.

I'm not sure I follow what you're trying to achieve, it sounds like you want to send a "command"(?) to your existing server? What kind of protocol are you using here?
If you're using HTTP, you may want to look into using https://github.com/reactphp/http-client/blob/master/examples/01-google.php or https://github.com/reactphp/socket/blob/master/examples/11-http-client.php instead.

@clue I want to send the text to the server:
$conn->write('worker start');
And I want the server to send this message back:
foreach ($socket->getConnections() as $connection) {
   $connection->write($data);
 }

How can I do it?
I did this way:
$this->loop = \React\EventLoop\Factory::create();

        // Create server socket.
        $connector = new \React\Socket\Connector($this->loop);
        $connector
            ->connect('<IP_ADDRESS>:8081')
            ->then(
                function (\React\Socket\ConnectionInterface $conn)  {
                    $conn->write('worker start');
                    $conn->on('data', function($data){
                        $this->loop->stop();
                    });
                }
            );
        $this->loop->run();

Is it right?

@wolrd Sounds about right. Are you having any issues with this code?
The above snippet will send some data to all your connections.
The below snippet will connect to your server once, send some data and then close the connection (without waiting for any incoming data).

@clue Thank you! It works.

I have one more question. How can I block the port (8081) from external requests?
$socket = new \React\Socket\Server('<IP_ADDRESS>:8081', $loop);

@wolrd Looks good to me. Are you having any issues with this code?
If you use <IP_ADDRESS>:x as a listening address, this socket will bind to your loopback/localhost interface only and as such nobody from outside your system will be able to connect to this server instance.

@clue thanks! It works:
This '<IP_ADDRESS>' is right. 'xxx.xx.xxx.xx' is wrong.


Homekit says update is available but Meross is updated

What issue do you have? Please be as thorough and explicit as possible.
Homekit says update is available but there is no update in the Meross app.  Firmware match in HomeKit and Meross app.
Details of your setup.

Do you use (1) Homebridge UI-X (2) Homebridge CLI or (3) HOOBS?

1

Which version of Homebridge/HOOBS do you have?

v4.41.2

Which platform do you run Homebridge/HOOBS on (e.g. Raspberry Pi/Windows/HOOBS Box)? Please also mention your version of Node.js/NPM if known.

Windows

Which version of this plugin (homebridge-meross) do you have? Has the issue started since upgrading from a previous version?

6.3.0

Which Meross devices do you have that are causing issues? Please include product models if applicable.
MSS110 plugs

Please paste any relevant logs below.

Hi @aeropink
Please see https://github.com/bwp91/homebridge-meross/issues/293#issuecomment-889490094 and comments going forward.
This is a known problem, the only current way around this is to manually increase the firmware value via the plugin settings.
Thanks!

My firmware in HomeKit and Meross match and it still says update is available.

My firmware in HomeKit and Meross match and it still says update is available.

This is a Home app issue, use the settings to override.

I have looked in the plugin but can not see where I can override it.  Can you please assist?

Type in a name, then a serial number, then a model, and the rest of the options will appear:
(see the bottom of the image for firmware override box)
You'll need the serial number (uuid) of the device which can be seen in the logs when the plugin starts
then set the firmware to like 999.9.9 and restart homebridge twice.

The home app is showing this message in error, even when devices' firmware is up to date.
unfortunately in the plugin's current state there is no other option but to override the firmware.

Thank you.  That worked.

In the new version of the plugin, I have changed the manufacturer that home app gets, so the firmware update incorrect alert should be fixed.
However it will require the accessory to be removed from the homebridge cache so it is re-added.
And if you try this, don’t forget to remove the override firmware version from the config, so then it should display the correct firmware version without the update alert!

Thanks.  I will check it out.


Add Extreme programming (XP) category

We are looking for links about Extreme Progamming.

Hi ! I just made a PR #14 on this subject.